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

Add Local dev environment and sample usage #350

Merged

Conversation

plooploops
Copy link
Contributor

New Features

Breaking Changes

Bug Fixes

Improvements

  • Local Development notes and sample usage

Dependency updates

Deployment changes

@plooploops plooploops force-pushed the local-development-notes branch 3 times, most recently from 268e2b6 to 765b24e Compare July 14, 2021 00:14
@plooploops plooploops marked this pull request as draft July 15, 2021 00:28
@plooploops plooploops marked this pull request as ready for review July 15, 2021 17:39
@plooploops plooploops force-pushed the local-development-notes branch 2 times, most recently from b9b2dbb to 38b31dc Compare September 1, 2021 22:17
@PlanXCyborg
Copy link

The style in this PR agrees with black. ✔️

This formatting comment was generated automatically by a script in uc-cdis/wool.

You can try to run the following to confirm installation:

```console
python3 sample_usage.py
Copy link
Contributor

Choose a reason for hiding this comment

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

we already have a run.py that can be used to run for verification. This seem to be duplicate

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the pointer, went ahead and updated it.

@xritter1
Copy link

Apologies for the delay! We are looking into this and would like to let you know that you could merge the newest changes into your branch at your convenience. We will have to review once more and will get the new changes merged as soon as possible. Thank you for your contribution!

@UchicagoZchen138 UchicagoZchen138 merged commit 4ae2069 into uc-cdis:master Mar 7, 2022
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.

4 participants