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

Obtain iotaa from conda-forge #415

Merged
merged 2 commits into from
Feb 26, 2024

Conversation

maddenp-noaa
Copy link
Collaborator

Synopsis

Now that iotaa is available from conda-forge, obtain it from there.

Note that this works, too:

~/git/uwtools/src % python3 -m venv uw
~/git/uwtools/src % source uw/bin/activate
(uw) ~/git/uwtools/src % pip install .
...
(uw) ~/git/uwtools/src % uw --help
usage: uw [-h] MODE ...

Unified Workflow Tools

Optional arguments:
  -h, --help
      Show help and exit

Positional arguments:
  MODE
    config
      Handle configs
    fv3
      Execute FV3 tasks
    rocoto
      Realize and validate Rocoto XML Documents
    sfc_climo_gen
      Execute sfc_climo_gen tasks
    template
      Handle templates

Note that no conda is involved there -- that's pure python / pip.

Type

  • Tooling (CI, code-quality, packaging, revision-control, etc.)

Impact

  • This is a non-breaking change (existing functionality continues to work as expected)

Checklist

  • I have added myself and any co-authors to the PR's Assignees list.
  • I have reviewed the documentation and have made any updates necessitated by this change.

Copy link
Contributor

@WeirAE WeirAE left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Congrats on getting it to conda-forge and thanks for a quick fix!

Copy link
Collaborator

@elcarpenterNOAA elcarpenterNOAA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@maddenp-noaa maddenp-noaa merged commit ef43827 into ufs-community:main Feb 26, 2024
2 checks passed
@maddenp-noaa maddenp-noaa deleted the conda-forge-iotaa branch February 26, 2024 16:12
WeirAE pushed a commit to WeirAE/workflow-tools that referenced this pull request Feb 27, 2024
WeirAE added a commit that referenced this pull request Feb 29, 2024
* edit config error message and test

* modify error text for clarity and doc text
Explicit typing doc in progress

* Updated to allow YAML as output always

* missed error documentation update

* modified representer for f90nml

* update to prod mode_config.rst for resolution

* for some reason, a space

* UW-509 docs for sfc_climo_gen driver (#412)

* Obtain iotaa from conda-forge (#415)

* Add conda badges and update pylint (#416)

* modify error text for clarity and doc text
Explicit typing doc in progress

* Updated to allow YAML as output always

* missed error documentation update

* update to prod mode_config.rst for resolution

* for some reason, a space

* Update error message and documentation

* Changes as follows below:
Moved representers to support.py
Moved OrderedDict logic to its own representer

* updated recursive function within ordereddict

* Update src/uwtools/config/support.py

Co-authored-by: Paul Madden <[email protected]>

* Update src/uwtools/config/support.py

Co-authored-by: Paul Madden <[email protected]>

* Update src/uwtools/config/support.py

Co-authored-by: Paul Madden <[email protected]>

* Update src/uwtools/config/support.py

Co-authored-by: Paul Madden <[email protected]>

* Update src/uwtools/config/support.py

Co-authored-by: Paul Madden <[email protected]>

* Update src/uwtools/config/support.py

Co-authored-by: Paul Madden <[email protected]>

* Update src/uwtools/config/support.py

Co-authored-by: Paul Madden <[email protected]>

* fix f90nml calls for cleanliness

* consolidate representers and test

* Update src/uwtools/config/support.py

Co-authored-by: Paul Madden <[email protected]>

* fix organization and doc string

* formatting

---------

Co-authored-by: Paul Madden <[email protected]>
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.

4 participants