Alter Options hash behavior

[ronald-jaepel]

Hashing now excludes keys:

• starting with _ or
• containing __ or
• listed in {"commit_message", "push", "debug", "force"}.

Key removal is recursive.

For logs of old cadet-RDM repositories, OutputRepo.update_log_hashes can bring the log up to date.

[ronald-jaepel]

I've currently got a cold and a fever, so this PR might be a bit messy. Let me know if this solves your options problem from #33 and then we can merge.

[schmoelder]

Thanks for the quick fix. Unfortunately, I now get the following error when calling case.run_study()

  File ~/code/CADET-RDM/cadetrdm/batch_running/case.py:185 in run_study
    if not force and self.has_results_for_this_run:

  File ~/code/CADET-RDM/cadetrdm/batch_running/case.py:93 in has_results_for_this_run
    self.results_branch = self._get_results_branch()

  File ~/code/CADET-RDM/cadetrdm/batch_running/case.py:116 in _get_results_branch
    for output_branch, log_entry in output_log.items()[::-1]:

TypeError: 'dict_items' object is not subscriptable

[schmoelder]

I suggest

for output_branch, log_entry in reversed(output_log.items()):

[schmoelder]

Just realized that this wasn't even part of this PR....

[ronald-jaepel]

Cool! Again what learned :)

[schmoelder]

unfortunately, the self.output_repo.update_log_hashes() seems to ignore the Case.status attribute which leads to crashes every time I want to run multiple studies / cases at once.

[schmoelder]

don't we need to set the changes_were_made flag here, too? Otherwise, it will never actually update the version number and always try to update the log hashes.

[ronald-jaepel]

To document our insight from Discord:
changes_were_made was not set to True, so the version number of the repo was not bumped.
After that I was afraid it would not work, because it would re-do the fix because v0.1.6 is still <0.1.7, so I created a pre-release. @schmoelder correctly identified that that isn't necessary as the version update section is only performed if if self._metadata["cadet_rdm_version"] != cadetrdm.__version__:.

[schmoelder]

This crashes if len(log.entries) == 0 (i.e. a repo that has never been run).