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

Bump robot version to v1.8.3 and address robot errors #206

Merged
merged 3 commits into from
Jun 16, 2023
Merged

Conversation

caufieldjh
Copy link
Collaborator

@caufieldjh caufieldjh commented Jun 16, 2023

This isn't the most recent version of robot, but it is the most recent version that works.
Something strange happens between v1.8.3 and v1.8.4 causing OWL parsing to fail.

@caufieldjh caufieldjh changed the title Bump robot version to v1.9.4 Bump robot version to v1.9.4 and address robot errors Jun 16, 2023
@caufieldjh
Copy link
Collaborator Author

Getting errors in transforming to obograph json from OWL

@caufieldjh
Copy link
Collaborator Author

I think there's an issue with the way the owl is being parsed:

In bfo, used this value for version: versionIRI
Don't have this version of bfo yet - will transform.
Could not parse OWL definitions enough to locate any imports.
No imports found for bfo.
Completed download from <MagicMock name='get_url()' id='140150636974528'> to /tmp/bfohspsfkui.
Moving from /tmp/bfohspsfkui to data/bfo/2019-08-26/bfo.owl.
ROBOT preprocessing: relax bfo
Relaxing /tmp/bfohspsfkui to /tmp/tmpwon_asiu_bfo_relaxed.owl...
Complete.
Before relax: 0 lines. After relax: 15 lines.

@caufieldjh
Copy link
Collaborator Author

caufieldjh commented Jun 16, 2023

This test error does not occur on a local run of pytest. Weird.
Or they don't, that is, until I update robot to v1.9.4.
With that version, the relaxed bfo produced is empty because the mocked bfo is an empty file.

@caufieldjh
Copy link
Collaborator Author

Works with robot:

  • 1.8.2
  • 1.8.3
    but fails on v1.8.4 and subsequent versions.

Don't think it's the java version - I'm on v17 locally.

@caufieldjh caufieldjh changed the title Bump robot version to v1.9.4 and address robot errors Bump robot version to v1.8.3 and address robot errors Jun 16, 2023
@caufieldjh caufieldjh merged commit c7356ef into main Jun 16, 2023
5 checks passed
@caufieldjh caufieldjh deleted the update_robot branch March 13, 2024 20:59
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.

None yet

1 participant