Skip to content

Backward Evolution Study#74

Closed
giacomomagni wants to merge 76 commits into
developfrom
feature/backward_paper
Closed

Backward Evolution Study#74
giacomomagni wants to merge 76 commits into
developfrom
feature/backward_paper

Conversation

@giacomomagni
Copy link
Copy Markdown
Collaborator

This PR is for the backward evolution study. We seek for the charm pdf below its mass threshold.

@giacomomagni giacomomagni added physics new physics features paper Paper labels Jul 1, 2021
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 1, 2021

Codecov Report

Merging #74 (707b3da) into develop (08df898) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 707b3da differs from pull request most recent head 4977739. Consider uploading reports for the commit 4977739 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           develop       #74    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files           29        44    +15     
  Lines         2143      2580   +437     
==========================================
+ Hits          2143      2580   +437     
Flag Coverage Δ
unittests 100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/eko/evolution_operator/__init__.py 100.00% <ø> (ø)
src/eko/matching_conditions/__init__.py 100.00% <ø> (ø)
src/eko/matching_conditions/n3lo/f_functions.py 100.00% <ø> (ø)
src/eko/matching_conditions/nnlo.py 100.00% <ø> (ø)
src/eko/anomalous_dimensions/harmonics.py 100.00% <100.00%> (ø)
src/eko/evolution_operator/grid.py 100.00% <100.00%> (ø)
src/eko/matching_conditions/n3lo/__init__.py 100.00% <100.00%> (ø)
src/eko/matching_conditions/n3lo/aHg.py 100.00% <100.00%> (ø)
src/eko/matching_conditions/n3lo/aHgstfac.py 100.00% <100.00%> (ø)
src/eko/matching_conditions/n3lo/aHq.py 100.00% <100.00%> (ø)
... and 12 more

Comment thread backward_paper/evolve_backward.py Outdated
Comment thread backward_paper/evolve_backward.py Outdated
Comment thread backward_paper/runner.py Outdated
Comment thread backward_paper/runner.py Outdated
tab["x"] = xgrid

# Loop over pdfs replicas
for rep, pdf in enumerate(pdfs):
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • actually: this is quite a general feature - we maybe want to lift this into banana, (by improving get_pdf, e,g,?)
  • this may also affect our run calls - the pdf argument could e.g. become a tuple (set_name, member) and if it is just a string then cast it to member=0 (as is currently)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • if you want all replicas, we could provide a helper function, that spells out the respective tuples

Comment thread backward_paper/runner.py
Comment thread backward_paper/runner.py Outdated
@felixhekhorn
Copy link
Copy Markdown
Collaborator

Workflows were failing due to 3f2271a

@giacomomagni
Copy link
Copy Markdown
Collaborator Author

The N3LO matching is fully numbified and can be evaluated within 10 minutes.

@felixhekhorn
Copy link
Copy Markdown
Collaborator

The N3LO matching is fully numbified and can be evaluated within 10 minutes.

Perfect 👍 ! did you by chance already take a look with cProfile on what the bottleneck is? (There is some preliminary stuff in yadism )

@giacomomagni
Copy link
Copy Markdown
Collaborator Author

Not yet, good suggestion, I'll look into this!

giacomomagni added a commit that referenced this pull request Jan 17, 2022
@giacomomagni giacomomagni mentioned this pull request Jan 17, 2022
@alecandido
Copy link
Copy Markdown
Collaborator

alecandido commented Jan 23, 2022

The moment we merge #83 and #85, I'd propose we'll close this branch: the snapshot has already been taken, and the two PRs should take from here everything we want to keep.

(of course, if there will be still leftovers, we'll open another PR and wait also for that one)

EDIT: e.g. we might want to import some plots or plotting features in #79

@alecandido
Copy link
Copy Markdown
Collaborator

Since we actually merged #83 do we want to close this PR? @giacomomagni @felixhekhorn

@felixhekhorn
Copy link
Copy Markdown
Collaborator

I'd suggest to wait for the acceptance of the paper

@felixhekhorn felixhekhorn marked this pull request as draft July 13, 2022 16:16
@giacomomagni
Copy link
Copy Markdown
Collaborator Author

I'd say let's close this :)

@felixhekhorn felixhekhorn deleted the feature/backward_paper branch January 5, 2023 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

paper Paper physics new physics features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants