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

Vine: Jupyter Notebook Integration and Best Practices #3760

Open
btovar opened this issue Apr 5, 2024 · 3 comments
Open

Vine: Jupyter Notebook Integration and Best Practices #3760

btovar opened this issue Apr 5, 2024 · 3 comments
Labels
bug For modifications that fix a flaw in the code. enhancement TaskVine

Comments

@btovar
Copy link
Member

btovar commented Apr 5, 2024

Users of notebooks often re-evaluate cells. If a cell redefines a vine manager, two undesirable things may happen:

  • If port is set, then the cell raises an exception because the manager cannot bind to the port.
  • If the manager is free to choose another port, then many manager instances run at the same time.

Ideally, the Manager constructor would check if a manager on a port already exits and simply warn the user that a previous manager definition is being used. (Or some mechanism like that.)

@btovar btovar added bug For modifications that fix a flaw in the code. enhancement TaskVine labels Apr 5, 2024
@dthain
Copy link
Member

dthain commented Apr 8, 2024

I think we need to think more comprehensively about how to use notebook environments to deploy distributed/parallel applications. As you point out, it doesn't make sense to reinstantiate the manager in every cell. Maybe it belongs in the notebook kernel instead? We have pending CSSI proposal on notebook/workflow integration, I'll send it to you separately...

@btovar
Copy link
Member Author

btovar commented Apr 8, 2024

Sounds good!

@dthain dthain changed the title vine: jupyter notebook manager redefintion Vine: Jupyter Notebook Integration and Best Practices Jun 12, 2024
@dthain
Copy link
Member

dthain commented Jun 12, 2024

There would appear to be a variety of ways to use TaskVine within a notebook, some of which work, and others which do not. At a minimum, we should document and provide an example of how TV can be used correctly, with some warnings re things that don't work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug For modifications that fix a flaw in the code. enhancement TaskVine
Projects
Status: No status
Development

No branches or pull requests

2 participants