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

BLOM fork with name BLOMGPU #81

Open
monsieuralok opened this issue Apr 21, 2021 · 9 comments
Open

BLOM fork with name BLOMGPU #81

monsieuralok opened this issue Apr 21, 2021 · 9 comments

Comments

@monsieuralok
Copy link
Collaborator

@TomasTorsvik @matsbn @MichaelSchulzMETNO Please can you create a BLOM fork with name BLOMGPU in NorESMhub? We need it for development on GPUs?

Thanks a lot
Alok

@TomasTorsvik
Copy link
Contributor

@monsieuralok @matsbn @MichaelSchulzMETNO
Hi,
I don't think it is easy (or even possible?) to fork a repository in the same organization space. Options include

I suggest to set up an ad-hoc organization for the BLOMGPU fork that can be managed separately from NorESMhub. This seems to be the easiest solution, and it should be easy to give write access without involving NorESMhub. The person who set it up will be the owner of the repo, but others can be added as admins or given write access. I can set this up if you don't want to administrate this yourself.

@MichaelSchulzMETNO
Copy link

Alok did you mean branch, maybe?

@TomasTorsvik
Copy link
Contributor

I think the idea was to make a fork. It seems that GPU acceleration will be enabled trough OpenMP in the near future (apparently, OpenACC is on the way out), and this may not be compatible with the existing OpenMP directives in BLOM. However, at the moment the plan is to continue with OpenACC a while longer and make a switch later on.

@agu002
Copy link

agu002 commented Apr 21, 2021

Alok did you mean branch, maybe?

@MichaelSchulzMETNO no, we discussed yesterday about fork. I would then fork in my account. Thanks

@agu002
Copy link

agu002 commented Apr 21, 2021

I think the idea was to make a fork. It seems that GPU acceleration will be enabled trough OpenMP in the near future (apparently, OpenACC is on the way out), and this may not be compatible with the existing OpenMP directives in BLOM. However, at the moment the plan is to continue with OpenACC a while longer and make a switch later on.

@TomasTorsvik That was the Idea. I would then fork it in my account. Thanks a lot for clarification. I noticed after forking I can change the repository name. Let see it doesn't conflict with BLOM and BLOMGPU in my account.

@TomasTorsvik
Copy link
Contributor

@monsieuralok I think it would be a good idea to make a separate organization for the BLOMGPU fork
https://docs.github.com/en/github/getting-started-with-github/types-of-github-accounts#organization-accounts

This will make it easier to give write access and admin roles to other users. If you make it in your personal account, I think it can be more challenging to manage the repo later on.

@TomasTorsvik
Copy link
Contributor

@MichaelSchulzMETNO
Copy link

Sorry, its maybe not my business, but I dont see why you need a fork and a new organisation for this...You could also fork into you private account and cooperate on that fork. I guess there are many ways to happiness.

@TomasTorsvik
Copy link
Contributor

TomasTorsvik commented Apr 21, 2021

Sorry, its maybe not my business, but I dont see why you need a fork and a new organisation for this...You could also fork into you private account and cooperate on that fork. I guess there are many ways to happiness.

Many ways indeed :)
I'm not saying this is the only way, just what I think would be easiest. Hopefully we can have some advanced support from Sigma2 (or Lumi admin, if we port to that system), and it may speed up the process if everyone has direct access to the same fork. At lest, that's my thinking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

No branches or pull requests

4 participants