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

OOPS atlas changes #46

Merged
merged 2 commits into from
Aug 30, 2023
Merged

Conversation

twsearle
Copy link
Collaborator

Description

Changes associated with https://github.com/JCSDA-internal/oops/pull/2331 and the refactoring of the OOPS interface to atlas.

Issues

Testing

@twsearle twsearle linked an issue Aug 21, 2023 that may be closed by this pull request
@twsearle twsearle marked this pull request as ready for review August 21, 2023 12:20
@DJDavies2
Copy link
Contributor

It seems that this branch doesn't actually work with https://github.com/JCSDA-internal/oops/pull/2331 after all, maybe something was committed to the oops branch after I tested? In any case I think this branch needs updating.

@DJDavies2
Copy link
Contributor

Here is the compile failure:

/home/d03/frwd/cylc-run/orca-jedi-46/share/mo-bundle/orca-jedi/src/orca-jedi/variablechanges/VariableChange.h:36:3: note: no known conversion for argument 1 from 'const eckit::Configuration' to 'const Parameters_& {aka const VariableChangeParameters&}'
/home/d03/frwd/cylc-run/orca-jedi-46/share/mo-bundle/orca-jedi/src/orca-jedi/variablechanges/VariableChange.h:29:7: note: candidate: orcamodel::VariableChange::VariableChange(const orcamodel::VariableChange&)
class VariableChange: public util::Printable,
^~~~~~~~~~~~~~
/home/d03/frwd/cylc-run/orca-jedi-46/share/mo-bundle/orca-jedi/src/orca-jedi/variablechanges/VariableChange.h:29:7: note: candidate expects 1 argument, 2 provided
/home/d03/frwd/cylc-run/orca-jedi-46/share/mo-bundle/orca-jedi/src/orca-jedi/variablechanges/VariableChange.h:29:7: note: candidate: orcamodel::VariableChange::VariableChange(orcamodel::VariableChange&&)
/home/d03/frwd/cylc-run/orca-jedi-46/share/mo-bundle/orca-jedi/src/orca-jedi/variablechanges/VariableChange.h:29:7: note: candidate expects 1 argument, 2 provided
gmake[2]: *** [orca-jedi/src/mains/CMakeFiles/orcamodel_hofx3D.x.dir/orcamodelHofX3D.cc.o] Error 1
gmake[1]: *** [orca-jedi/src/mains/CMakeFiles/orcamodel_hofx3D.x.dir/all] Error 2
gmake[1]: *** Waiting for unfinished jobs....
gmake[2]: *** [orca-jedi/src/mains/CMakeFiles/orcamodel_hofx.x.dir/orcamodelHofX.cc.o] Error 1
gmake[1]: *** [orca-jedi/src/mains/CMakeFiles/orcamodel_hofx.x.dir/all] Error 2

@twsearle
Copy link
Collaborator Author

@DJDavies2 I think this is just because it was out-of-step with develop. I have merged and pushed again, but I am just going to squash and merge this now anyway as we have compile failures. I guess the underlying scheduled change went in?

@twsearle twsearle merged commit 6b3cdfc into develop Aug 30, 2023
@twsearle twsearle deleted the feature/oops-atlas-interface-changes branch November 16, 2023 14:13
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.

OOPS atlas interface updates
2 participants