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

feat: add CSVSummary callback #173

Merged
merged 14 commits into from
Nov 20, 2020
Merged

feat: add CSVSummary callback #173

merged 14 commits into from
Nov 20, 2020

Conversation

sebastianJaeger
Copy link
Contributor

@sebastianJaeger sebastianJaeger commented Nov 20, 2020

Added functionality to save and append intermediate results to a csv value to have a history of parameter values. Contributes to #159

@redeboer redeboer changed the title add functionallity to write intermediate results to csv feat: add CSVSummary callback Nov 20, 2020
src/tensorwaves/optimizer/callbacks.py Outdated Show resolved Hide resolved
@redeboer redeboer added this to the Release 0.1.3 milestone Nov 20, 2020
@redeboer redeboer merged commit fe61e94 into epic/99-export-fit Nov 20, 2020
@redeboer redeboer deleted the add_csv_saving branch November 20, 2020 16:37
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 <sjaeger@ep1.rub.de>
Co-authored-by: spflueger <spfluege@gmail.com>
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