enterprise_gateway icon indicating copy to clipboard operation
enterprise_gateway copied to clipboard

Published 20 hours ago • verified publisher icon jupyter-server

Switch from conda to venv for development env

Open bloomsa opened this issue 2 years ago • 8 comments

addresses #1294

Upon doing this, the "happy path" is simpler but overall it feels a little more frail / platform dependent. I'm curious to hear thoughts on that tradeoff or any suggestions people have seen from other repos

bloomsa avatar May 06 '23 01:05 bloomsa

Could you elaborate on the why?

lresende avatar May 09 '23 00:05 lresende

@lresende beyond what is in issue #1294?

bloomsa avatar May 09 '23 01:05 bloomsa

is there a way to make it a choice? instead of one or another?

lresende avatar May 09 '23 15:05 lresende

Or remove this topic altogether. To my earlier point, the environment is up to the dev, so should we even bother with providing these trivial helper methods at all? We can document suggestions, but do we need the make target helpers?

kevin-bates avatar May 09 '23 15:05 kevin-bates

We can document suggestions, but do we need the make target helpers?

I like this idea. I can update the docs to include suggestions for both venv and conda, but pull out the make targets altogether. On one hand, it feels kinda wrong to pull out something that already works, but my thinking behind it is that having the targets there makes it seem like conda is the only way to build the local dev environment when that isn't really the case.

Are you both okay with that solution @lresende @kevin-bates

bloomsa avatar May 10 '23 00:05 bloomsa

Hi @bloomsa - I apologize for the delay. I'm fine with documenting suggestions. I just don't really see the utility of having helper targets at this point.

@lresende - do you have a strong objection?

kevin-bates avatar May 16 '23 23:05 kevin-bates

@kevin-bates Well, it looks like we have been there: https://github.com/jupyter-server/enterprise_gateway/pull/1079

And than we came back: https://github.com/jupyter-server/enterprise_gateway/pull/1108

Do you remember why we had to come back?

lresende avatar May 17 '23 17:05 lresende

It appears we have come full circle. :smile: @bloomsa contributed https://github.com/jupyter-server/enterprise_gateway/pull/1107 to introduce only the helper targets, but then we hit a bit of a snafu due to other changes that #1108 fixed.

Given this, I think we can proceed with removing the helper targets and documenting suggestions for setting up conda/mamba and venv environments - as discussed above.

@lresende, @bloomsa - does this sound like a plan?

kevin-bates avatar May 17 '23 17:05 kevin-bates