search

dataflownb / dfkernel

A kernel to support Python dataflows in the Jupyter Notebook environment
Other
119 stars 8 forks source link

Allow for explicit library export #61

Open colinjbrown opened 5 years ago

colinjbrown commented 5 years ago
screen shot 2018-12-19 at 12 35 56 pm

In the case below a user might expect that only np and n would be exported but they also get pd.

I suggest that in this case we look at the set of imports and the tuple that exits and if there is any overlap then we don't export the libraries.

This way if a user wants to be smart about writing their code they don't have to jump through any hoops to change their expectations of what will happen.

This goes along with the idea that the auto import logic that exports libraries should be able to be turned off on a per cell basis.

colinjbrown commented 8 months ago

Still relevant to the notebook extension, this is an unusual issue however as it implies that we want to do garbage collection for libraries if they're not used. Might need some weighing in if this issue actually matters or not.