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

Display inferred params with `papermill foo.ipynb --help` #158

Closed
wants to merge 2 commits into from

Conversation

Projects
None yet
3 participants
@thomasballinger
Copy link

commented Jul 28, 2018

Working on guessing param type from a parameters cell

plan: mirror the structure of _build_parameter_code for different languages

@thomasballinger thomasballinger changed the base branch from master to inputHelp Jul 29, 2018

@thomasballinger thomasballinger force-pushed the thomasballinger:guess-params branch from 567728a to a0212f6 Jul 29, 2018

@thomasballinger thomasballinger changed the base branch from inputHelp to master Jul 29, 2018

@thomasballinger thomasballinger force-pushed the thomasballinger:guess-params branch from a0212f6 to 8ba3e10 Jul 29, 2018

@thomasballinger thomasballinger changed the title WIP do not merge - draft of guesssing params idea Display inferred params with `papermill foo.ipynb --help` Jul 29, 2018

@thomasballinger thomasballinger force-pushed the thomasballinger:guess-params branch from 8ba3e10 to 61b643b Jul 29, 2018

show inferred parameters of notebooks from --help
$ papermill papermill/tests/notebooks/report_mode_test.ipynb --help
notebook_path: papermill/tests/notebooks/report_mode_test.ipynb
inferred parameters:
  a: int (default 1)
  b: int (default 1)

@thomasballinger thomasballinger force-pushed the thomasballinger:guess-params branch from 61b643b to 2594002 Jul 29, 2018

@rgbkrk

This comment has been minimized.

Copy link
Member

commented Aug 26, 2018

This looks like it needs a rebase, I certainly think it's a super cool idea.

@rgbkrk

This comment has been minimized.

Copy link
Member

commented Jun 26, 2019

Since this work has stalled, I'm going to go ahead and close this. I'd love to see an issue outlining this idea for another person to tackle if you've got the time @thomasballinger!

@MSeal MSeal closed this Jun 29, 2019

@rgbkrk

This comment has been minimized.

Copy link
Member

commented Jul 2, 2019

Thanks for doing the actual close @MSeal. 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.