Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add paper from issue #182

Merged
merged 2 commits into from
Aug 24, 2023
Merged

Add paper from issue #182

merged 2 commits into from
Aug 24, 2023

Conversation

ramonpeter
Copy link
Collaborator

I added the paper from #106. In order to fit the norm I slightly updated the bibtex entry to

@article{Holmberg:2023gnn,
    author = {Holmberg, Daniel and Golubovic, Dejan and Kirschenmann, Henning},
    title = "{Jet Energy Calibration with Deep Learning as a Kubeflow Pipeline}",
    journal = {Computing and Software for Big Science},
    year = {2023},
    month = {8},
    day = {23},
    volume = {7},
    number = {1},
    pages = {9},
    issn = {2510-2044},
    doi = {10.1007/s41781-023-00103-y}
}

Key got some addition to be more unique (Holmberg:2023 -> Holmberg:2023gnn)
Changed month into proper integer (Aug -> 8)

@jraine
Copy link
Collaborator

jraine commented Aug 24, 2023

Maybe we can follow up with the author to see if the paper can also be added to the arxiv? I think inspire will miss things only submitted to CSBS.
Otherwise looks good - but as the paper has a strong focus on kubernetes perhaps most relevant in the Fast inference / deployment section.

On that note - Fast inference / deployment is a subsection of classification but should probably be top level (follows up on #123 )

@claudius-krause
Copy link
Collaborator

I support moving the subsection to the top level, we can add this item to the restructuring discussion, no need for a separate issue or PR just on this

@ramonpeter
Copy link
Collaborator Author

ramonpeter commented Aug 24, 2023

Hi @jraine, @claudius-krause
I added it to the sections the author suggested it to put them. I also added it to the deployment part too.
I guess its ready to merge now.

Indeed @deinal, it would be nice to add it to the arxive. Was there a reason you guys did not?

@claudius-krause claudius-krause merged commit 4ed28e6 into master Aug 24, 2023
2 checks passed
@claudius-krause claudius-krause deleted the add-paper branch August 24, 2023 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants