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

test_machines Is Failing on master #291

Open
torrinba opened this issue Feb 22, 2024 · 3 comments
Open

test_machines Is Failing on master #291

torrinba opened this issue Feb 22, 2024 · 3 comments

Comments

@torrinba
Copy link
Collaborator

For some reason it looks like the svn setup with the machine mappings is now failing. I didn't see this problem last time I ran the tests and believe everything I am doing is the same (this was run with make test on Iris using the omfit/unstable environment).

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
TestOmasMachine.test_machines
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
svn: 'https://github.com/gafusion/omas.git/trunk/omas/machine_mappings' path not found
svn: 'https://github.com/gafusion/omas.git/branches/dummy/omas/machine_mappings' path not found
svn: 'https://github.com/gafusion/omas.git/trunk/omas/machine_mappings' path not found
ERROR

======================================================================
ERROR: test_machines (test_omas_machine.TestOmasMachine)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "omas/omas/tests/test_omas_machine.py", line 47, in test_machines
    assert omas_rcparams['tmp_omas_dir'] in machines(self.machine, 'master')
  File "omas/omas/omas_machine.py", line 548, in machines
    raise NotImplementedError(f'Machine mapping file `{machine}.json` does not exist')
NotImplementedError: Machine mapping file `d3d.json` does not exist

----------------------------------------------------------------------
Ran 64 tests in 143.150s

FAILED (errors=1, skipped=10)
make: *** [test] Error 1

There's a similar test in test_remote_machine_mappings which will also fail if this one is bypassed. Does anyone have an idea why this wouldn't work? How important is this?

@AreWeDreaming
Copy link
Collaborator

The d3d.json stores all the mappings of DIII-D data to IMAS (except for efit). So it is quite important. That being said I think it is completely autogenerated, so I am not sure why it doesn't appear on its own. This is my limited understanding, @orso82 would know more.

@torrinba
Copy link
Collaborator Author

torrinba commented Feb 22, 2024

The tests that use d3d.json from the local omas version also pass just fine. It's only when omas tries to fetch it from master that the tests fail. I don't understand the use case for this either

See lines 47 and 68 in omas/tests/test_omas_machine.py for where this fails. All the uses of d3d.json prior to these pass

Copy link

Stale issue message

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants