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

Change in params.yaml might not work with zntrack.json #263

Open
PythonFZ opened this issue Mar 23, 2022 · 1 comment
Open

Change in params.yaml might not work with zntrack.json #263

PythonFZ opened this issue Mar 23, 2022 · 1 comment
Labels
bug Something isn't working p1-important
Milestone

Comments

@PythonFZ
Copy link
Member

For Example a zn.method being changed in the params.yaml can still cause issues with the wrong zntrack.json entry.

ttv_train_sampler:
  groups:
  method:
  - _cls: UniformEnergetic # changed from Random
  n_configurations: 
  - 30

with the following zntrack.json

"ttv_train_sampler": {
        "method": [
            {
                "_type": "zn.method",
                "value": {
                    "module": "mlsuite.configuration_selection.random"
                }
            }
        ],

Proposed solutions:

  • ...
@PythonFZ PythonFZ added bug Something isn't working p1-important labels Mar 23, 2022
@PythonFZ
Copy link
Member Author

Solution: Just don't use zn.Method when ever possible

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working p1-important
Projects
None yet
Development

No branches or pull requests

1 participant