-
Notifications
You must be signed in to change notification settings - Fork 34
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
Specific adaptors contribution (marenostrum3 & ecmwf) #570
Comments
Process: threat them as two separate PRs / issues. Do they both warrant a new adaptor or can the changes be integrated with the existing ones? |
Hey guys - still want to open those PRs? :) |
Hi @andre-merzky, I'm not sure if those adaptors are interesting for you. On the one hand, the On the other hand, the You can find more in the attached link: Thanks! |
Hello @joanlopez, I checked the ECMWF SAGA adaptor you have created and I find it pretty interesting. If you are still interested in opening that PR, I will keep this issue open. Otherwise, I will close it as outdated. Thank you! |
Hi @marksantcroos and @andre-merzky,
We've developed our own adaptors (starting from yours) for two clusters:
The first one is an adaptation from PBS & SLURM and the second one is an adaptation from LSF.
Do you want to add them to saga-python adaptors?
How do you want to do it? I can do a PR, but how do you want to organize it?
You can see them here:
https://earth.bsc.es/gitlab/es/autosubmit/blob/master/autosubmit/platforms/ecmwf_adaptor.py
https://earth.bsc.es/gitlab/es/autosubmit/blob/master/autosubmit/platforms/mn_adaptor.py
The text was updated successfully, but these errors were encountered: