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

🐈 Task: Remove the ersilia current command #1267

Closed
2 tasks
DhanshreeA opened this issue Sep 23, 2024 · 15 comments
Closed
2 tasks

🐈 Task: Remove the ersilia current command #1267

DhanshreeA opened this issue Sep 23, 2024 · 15 comments
Assignees
Labels
good first issue Good for newcomers

Comments

@DhanshreeA
Copy link
Contributor

Summary

Related to #1262

This is a fairly straightforward task. This command is implemented here.

Objective(s)

  • De-register the current command from the ersilia cli here
  • Remove the implementation for this command mentioned above.

Documentation

No response

@NKD52
Copy link

NKD52 commented Sep 28, 2024

Hey, I am interested in tackling this task.

@GemmaTuron
Copy link
Member

Hello @NKD52
Thanks for your interest. This issue has been opened for Outreachy applicants, if you are an approved outreachy applicant please first complete the designated tasks before starting to contribute code. If you are not, we'll ask you to wait until the Outreachy application period is completed as the core team is very small and we cannot onboard new contributors at this moment while running Outreachy. Thanks for your understanding!

@daud99
Copy link
Contributor

daud99 commented Oct 1, 2024

@GemmaTuron I'm an approved Outreachy applicant. Can you assign this task to me?

@GemmaTuron
Copy link
Member

Hello @daud99

Please follow the guidelines set out on the Outreachy documentation in GitBook. There is a number of tasks to be completed before being assigned a coding task. And then pick only one, as I see you have added comments to more than one, to get started with. Thanks!

@Ajoke23
Copy link
Contributor

Ajoke23 commented Oct 2, 2024

Hi @DhanshreeA . Could you please assign this task to me. I have sent my motivation statement, model output and my system specification to you on slack

@GemmaTuron @miquelduranfrigola

@daud99
Copy link
Contributor

daud99 commented Oct 2, 2024

Hi @GemmaTuron , I have already completed the task (motivation letter, model output and system specification) to @DhanshreeA .
Thanks for letting me know.

@jubril-adeyi
Copy link
Contributor

Hi, @GemmaTuron I am an approved outreachy intern and I have already completed the necessary tasks; I have sent my motivation letter to @DhanshreeA and ran the models and sent the output also,
Could you please assign this task to me

@Ajoke23
Copy link
Contributor

Ajoke23 commented Oct 2, 2024

Hi @GemmaTuron, I have received an OK to go ahead and start contributing to issues from @DhanshreeA.

Please, can this issue be assigned to me?
Can I go ahead to start?

@DhanshreeA
Copy link
Contributor Author

Hi @Ajoke23 go ahead!

@Ajoke23
Copy link
Contributor

Ajoke23 commented Oct 2, 2024

@DhanshreeA Can I go ahead to start contributing to this issues.
@GemmaTuron, I have received an OK to go ahead and start contributing. I requesting to be assigned to this issue?

Summary

Related to #1262

This is a fairly straightforward task. This command is implemented here.

Objective(s)

  • De-register the current command from the ersilia cli here
  • Remove the implementation for this command mentioned above.

Documentation

No response

@DhanshreeA
Copy link
Contributor Author

DhanshreeA commented Oct 3, 2024

@Ajoke23 could you please reconcile your work in a single PR? 😅 I see three PRs linked to this issue, please tell me which two I can close.

@Ajoke23
Copy link
Contributor

Ajoke23 commented Oct 3, 2024

@Ajoke23 could you please reconcile your work in a single PR? 😅 I see three PRs linked to this issue, please tell me which two I can close.

Hi @DhanshreeA. I tagged you to the main pull request.
The main pull request is #1284

@DhanshreeA DhanshreeA moved this from Queued to In Progress in Ersilia Model Hub Oct 5, 2024
@DhanshreeA
Copy link
Contributor Author

@Ajoke23 can we close #1282 then?

@Ajoke23
Copy link
Contributor

Ajoke23 commented Oct 5, 2024

@Ajoke23 can we close #1282 then?

Hi @DhanshreeA . Yes, you can

@DhanshreeA
Copy link
Contributor Author

Closed in #1284

@github-project-automation github-project-automation bot moved this from In Progress to Done in Ersilia Model Hub Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
Archived in project
Development

No branches or pull requests

6 participants