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

Basic setup for using pixi #212

Merged
merged 12 commits into from
May 13, 2024
Merged

Basic setup for using pixi #212

merged 12 commits into from
May 13, 2024

Conversation

castelao
Copy link
Collaborator

@castelao castelao commented May 2, 2024

Setup to run on Eagle.

@castelao castelao changed the title Setup for Eagle Basic setup for using pixi May 10, 2024
@castelao
Copy link
Collaborator Author

It's not worth to setup for Eagle considering it will be out soon. This setup shouldn't affect anyone but it would help me.

@castelao castelao marked this pull request as ready for review May 10, 2024 15:44
Copy link
Member

@grantbuster grantbuster left a comment

Choose a reason for hiding this comment

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

I'm still not familiar with the pixi setup so lgtm, but how are you using this? At this point i think i'm the only person running sup3r on eagle which is fine. You and brandon should focus on getting a kestrel gpu env setup with tensorflow version pinned. I'll finish model validation on Eagle and then we'll migrate things over when kestrel GPUs are available.

Copy link
Collaborator

@bnb32 bnb32 left a comment

Choose a reason for hiding this comment

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

looks good! small comment on netcdf vs h5netcdf. also, a key part of this is some additional instructions on use though. could you add some to the README?

pyproject.toml Outdated Show resolved Hide resolved
@castelao
Copy link
Collaborator Author

... also, a key part of this is some additional instructions on use though. could you add some to the README?

Yes. I'm planning to add the instructions in a following PR. With this one here I can do some tests in kestrel, and tune it, before recommending you both on how to use it.

@bnb32
Copy link
Collaborator

bnb32 commented May 13, 2024

... also, a key part of this is some additional instructions on use though. could you add some to the README?

Yes. I'm planning to add the instructions in a following PR. With this one here I can do some tests in kestrel, and tune it, before recommending you both on how to use it.

Ok, that works.

@castelao castelao merged commit 138a006 into main May 13, 2024
7 checks passed
@castelao castelao deleted the pixi branch May 13, 2024 14:59
github-actions bot pushed a commit that referenced this pull request May 13, 2024
* Testing with Python-3.11

* No direct dependency on google-auth-oauthlib

Let's try to remove it as a direct dependency.

* It seems to run just fine in 3.11

Thanks to @bnb32 with PR #210 !

* Adding a basic pixi setup

Sync dependency tree on OSX and Linux.

* And the actual versions

* Adding test and doc environment

* Update meaningless upgrade on dynamic version of sup3r

* Ignoring .pixi

* Adding dev environment

* dev environment aggregates everyting

When developing we test and build documentation.

* Updating pixi.lock

* Removing netcdf4
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

3 participants