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.

Sign up for GitHub

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

Jump to bottom

Cut banana out of ekobox #122

Closed
alecandido opened this issue May 25, 2022 · 6 comments · Fixed by #226
Closed

Cut banana out of ekobox #122

alecandido opened this issue May 25, 2022 · 6 comments · Fixed by #226
Assignees
@felixhekhorn @alecandido @giacomomagni
Labels
dependencies Pull requests that update a dependency file refactor Refactor code
Milestone
post

Comments

@alecandido
Copy link
Member

Maybe the solution is just a forth package, ekobox and ekomark will depend on it, but at least ekobox won't depend on ekomark, and we're sure will not depend on banana.
@alecandido alecandido added this to the 0.9.3 milestone May 25, 2022
@felixhekhorn
Copy link
Contributor

@scarlehoff we were thinking about this as a solution for the problems in NNPDF/nnpdf#1529

@alecandido alecandido mentioned this issue Jun 1, 2022
Closed
5 tasks
@scarlehoff
Copy link
Member

This is in the milestone for 0.9.4 (which has been reelased) but it is not closed. What is the status? (to know whether I have to update the conda recipe or not: conda-forge/eko-feedstock#3)

@alecandido
Copy link
Member Author

The person that released v0.9.4 simply forgot to update the milestone (milestones in EKO have been just introduced, people are not familiar with them...). I'm updating it.

@alecandido alecandido modified the milestones: 0.9.4, 0.9.5 Jun 7, 2022
@felixhekhorn
Copy link
Contributor

The person that released v0.9.4 simply forgot to update the milestone (milestones in EKO have been just introduced, people are not familiar with them...). I'm updating it.

sorry 😇

@alecandido
Copy link
Member Author

No problem :)

@felixhekhorn felixhekhorn mentioned this issue Jun 23, 2022
Closed
@felixhekhorn felixhekhorn added refactor Refactor code dependencies Pull requests that update a dependency file labels Jul 11, 2022
@felixhekhorn felixhekhorn modified the milestones: 0.9.5, 0.10.1, 0.10.0 Jul 13, 2022
@alecandido alecandido modified the milestones: 0.10.3, 1.0.0, post Jan 20, 2023
@felixhekhorn felixhekhorn mentioned this issue Mar 14, 2023
Merged
@giacomomagni giacomomagni linked a pull request Mar 14, 2023 that will close this issue
Polarized ToyLH from genpdf #225
Merged
@felixhekhorn felixhekhorn mentioned this issue Mar 14, 2023
Merged
@felixhekhorn
Copy link
Contributor

the relevant PR is #226

@felixhekhorn felixhekhorn reopened this Mar 14, 2023
@giacomomagni giacomomagni removed a link to a pull request Mar 14, 2023
Polarized ToyLH from genpdf #225
Merged
@giacomomagni giacomomagni linked a pull request Mar 14, 2023 that will close this issue
Invert ekomark-ekobox dependency #226
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@felixhekhorn felixhekhorn

@alecandido alecandido

@giacomomagni giacomomagni

Labels
dependencies Pull requests that update a dependency file refactor Refactor code
Projects
None yet
Milestone
post
Development

Successfully merging a pull request may close this issue.

Invert ekomark-ekobox dependency NNPDF/eko
4 participants
@felixhekhorn @alecandido @scarlehoff @giacomomagni