-
Couldn't load subscription status.
- Fork 121
Added charts to OpenSourceAnalysis #411
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
Conversation
Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we dont want new folder structure, i was thinking you would be adding this to existing website as an enhancment
@sanjay-kv so i have to make changes in app. py file right?
To add eda in to that file!
Can you pls ellobrate what excetly your are expecting
@sanjay-kv can you please give me clarity where I have to add eda of open source either on note book file or in app. py file?
@sanjay-kv pls let me know where I have to make changes
everything should go to app.py
@sanjay-kv
So only code I have to add ?
And whether I have to add charts?
tbh it should work like enhancement .
means addition to the website. which includes charts and stuff. raise a another PR with readme change, I will give label for your work and close the PR
Issue no - #406
@sanjay-kv i have added eda to open source analysis .
is there any conflits .