-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add __str__
for major classes
#21
Comments
Subject: Request to work on issue Dear @russelldj, Best Regards |
Hi @amitsgh, Thanks for your interest in this! Please feel free to work on this and ask any questions you need. When you're ready, you can submit a PR with the changes. You are our first external collaborator. I'll try to get some style guide documentation up for you, but generally we use Out of curiosity, how did you find out about the project? Have you been able to run it? |
Hey @russelldj, I am new to this open-source contribution thing, so I generally search issues label:'good first issue'. This is how I came across this repo. I have read the readme file and checked the work on OFO website, thus decided to give it a try in contributing to it. I haven't set it up locally, but I will now, as I was waiting for the approval because I don't know how things generally works in open source. |
@russelldj, I am not clear on how to set it up locally, according to this https://github.com/open-forest-observatory/multiview-prediction-toolkit#installation, I current don't have access to |
@amitsgh, sorry for the confusion. You don't need access to Currently, you won't have access to any of the example data. @youngdjn, do you have any thoughts about what we could use as publicly-available data or if we can send something to @amitsgh? I'm happy to take the conversation offline. |
We added some formatting guidelines here. |
It would help with debugging to be able to print out a string representation that contains some useful information. For example, what files the data is loaded from and the size of major data attributes. For example, number of vertices in a mesh or number of cameras in a set.
The text was updated successfully, but these errors were encountered: