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

Maintainer needed (experience with SU2) #16

Open
MakisH opened this issue May 3, 2020 · 5 comments
Open

Maintainer needed (experience with SU2) #16

MakisH opened this issue May 3, 2020 · 5 comments

Comments

@MakisH
Copy link
Member

MakisH commented May 3, 2020

Dear preCICE community, dear SU2 community,

as you see, this repository is currently not actively maintained, but it is also not abandoned. As the preCICE developers have limited experience with SU2, we would very happily accept contributions to keep this project up-to-date.

If you have experience with SU2, maintaining the adapter (e.g. updating for new SU2 versions, commenting on issues, reviewing pull requests) would be quite easy and you would have all the help you need from our side. Due to the low demand for this adapter and the low frequency of SU2 releases, the time demand would be also low and, of course, on your own pace. Any help appreciated, for as long as you like.

Contribute to a quickly growing free/open-source project, be part of an enthusiastic community. Simply comment here if you can help. 🤗

@NAnand-TUD
Copy link

@MakisH I will have a look and evaluate the code a bit. I will let you know accordingly if I can be helpful. Thanks!

@Pranjan2
Copy link

@MakisH I am following up on our conversation from Gitter. I will go through the new releases of SU2 and probably some of the tutorials you have for other adapters. Let me know if you would like to meet sometime. I am looking to develop or perhaps extend Rusch's adapters to newer versions of SU2 and run some large-scale test cases over a cluster with detailed profiling studies specific to SU2, precice and probably some FEA solvers.

@Pranjan2
Copy link

@MakisH how would you like me to share the adapter files ? I have started working on them and would like to regularly push to a repo/branch and have it ready for the moderators to review once done.

@uekerman
Copy link
Member

uekerman commented Apr 2, 2022

Easiest is probably if you push the changes to a branch in your fork of this repo and then open a PR.
Let us know if this isn't clear. Happy to help.

@MakisH
Copy link
Member Author

MakisH commented Apr 6, 2022

@Pranjan2 sorry for only reacting now. As said already, you can just fork and open pull requests.

Best would be to submit multiple small pull requests from your fork to the develop branch of the adapter. It is important that the pull requests are small, to be able to review them. It is also important that you don't commit to your master branch, so that you can easily incorporate the changes from the reviews.

Since multiple people have expressed interest to help, keeping the pull requests small and frequent will be very helpful.

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

No branches or pull requests

4 participants