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

fix: copy initial parameters in optimize call #174

Merged
merged 8 commits into from
Nov 20, 2020
Merged

fix: copy initial parameters in optimize call #174

merged 8 commits into from
Nov 20, 2020

Conversation

redeboer
Copy link
Member

No description provided.

@redeboer redeboer added 🐛 Bug Something isn't working 🔨 Maintenance Maintenance and upkeep improvements labels Nov 20, 2020
@redeboer redeboer requested a review from spflueger November 20, 2020 18:01
@redeboer redeboer self-assigned this Nov 20, 2020
@redeboer redeboer merged commit d2746b6 into ComPWA:epic/99-export-fit Nov 20, 2020
@redeboer redeboer deleted the deepcopy-parameters branch November 20, 2020 18:28
redeboer added a commit that referenced this pull request Jan 4, 2021
* ci: add GitHub workflow for epic branches (#144)
* ci: increase minimal coverage to 80%
* feat: add CSVSummary callback (#173)
* feat: add variable logging functionality using TensorFlow (#155)
* feat: implement YAML optimize callback (#154)
* feat: implement Loadable callback (#177)
* feat: log execute time in optimize call (#156 and #164)
* fix: copy initial parameters in optimize call (#174)
* fix: implement temporary solution for #171
* fix: remove pytest color output VSCode
* test: add additional resonance to fixture
* refactor: change fit result dict structure
* docs: use only 3 free parameters
  Speeds up CI and prevents memory problems on Read the Docs

Co-authored-by: sjaeger <[email protected]>
Co-authored-by: spflueger <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug Something isn't working 🔨 Maintenance Maintenance and upkeep improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant