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

UPDATED DOCS #65

Closed
wants to merge 37 commits into from
Closed

UPDATED DOCS #65

wants to merge 37 commits into from

Conversation

vaibhawkhemka
Copy link

@vaibhawkhemka vaibhawkhemka commented Feb 22, 2021

Updated Installation of package "pickle" and minor fixes.
Please review this. @orensbruli @pbustos @adityaaggarwal97

@orensbruli
Copy link
Collaborator

@vaibhawkhemka Thank you for your contribution.
You are committing to the master branch and we all work on the development one as said in the README on the main page. Check if some of the changes you are suggesting apply to the development branch.

@vaibhawkhemka
Copy link
Author

vaibhawkhemka commented Mar 22, 2021 via email

@orensbruli
Copy link
Collaborator

You are right! My fault. I guess I'm looking too quick into the PRs!

@orensbruli
Copy link
Collaborator

Also, for the next time, I recommend making smaller PR. It's easy to get your files reviewed for someone related.

@orensbruli
Copy link
Collaborator

I see several new files in your PR. Are those for a new component? What is it for? Is it for some specific GSoC 2021 idea? Have you already tested?

@vaibhawkhemka
Copy link
Author

vaibhawkhemka commented Mar 22, 2021 via email

@orensbruli
Copy link
Collaborator

Perfect!
And whose are the mentors of that idea? @pbustos and @adityaaggarwal97 ? If they are we will wait until they review this. If not, I suggest you to invite them to review this PR.
Thank you!

@vaibhawkhemka
Copy link
Author

vaibhawkhemka commented Mar 22, 2021 via email

@vaibhawkhemka
Copy link
Author

vaibhawkhemka commented Mar 23, 2021 via email

@@ -12,7 +12,7 @@ This section assumes the user has already installed the RoboComp core library an

Before using the component, the user needs to install the necessary libraries:
```
pip install numpy opencv-python pickle scikit-learn==0.22.2.post1
pip install numpy opencv-python pickle-mixin scikit-learn==0.22.2.post1
Copy link
Collaborator

Choose a reason for hiding this comment

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

What is this change for?

Copy link
Author

Choose a reason for hiding this comment

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

This is an updated command for installing pickle library

@orensbruli
Copy link
Collaborator

So, this PR is obsolete

Hello @orensbruli, I have added all the final updated files for depth estimation in robocomp-robolab in a new PR ("MONOCULAR DEPTH ESTIMATION USING RGB IMAGES") to make it easier for testing them to give the real time prediction of depths. Thanks On Mon, Mar 22, 2021, 2:34 PM vaibhaw khemka @.> wrote:

Mentors are Mohamed shawky and Luis Vicente Calderita for "Monocular Depth Estimation from RGB signals". I will invite them for reviewing the PR. Thanks On Mon, Mar 22, 2021, 2:21 PM Esteban Martinena Guerrero < @.
> wrote: > Perfect! > And whose are the mentors of that idea? @pbustos > https://github.com/pbustos and @adityaaggarwal97 > https://github.com/adityaaggarwal97 ? If they are we will wait until > they review this. If not, I suggest you to invite them to review this PR. > Thank you! > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <#65 (comment)>, > or unsubscribe > https://github.com/notifications/unsubscribe-auth/ALDB2OU56F3DAAT2IQCP75TTE4AJJANCNFSM4YBKJY4A > . >

Is this PR obsolete? Is the title right? It's a full component or documentation PR?
Thank you.

@orensbruli orensbruli mentioned this pull request Mar 28, 2021
@vaibhawkhemka
Copy link
Author

Yes, this PR is obsolete now. Sorry for the confusion. The component part has been merged into https://github.com/robocomp/DNN-Services and documentation related to "Pickle and Argument parser" has been updated to #66. Thus, Closing this PR.

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.

2 participants