Skip to content

Conversation

@AndreiKingsley
Copy link
Contributor

Remove init/imports/renders part from Lets-Plot descriptors. They've been moved to Integration in Lets-Plot Kotlin.

A Lets-Plot Kotlin release with these changes is required before we can merge!

@ileasile ileasile changed the title Remove imports/init/renders from lets-plot descriptors (move to Integ… (DON'T MERGE ME) Remove imports/init/renders from lets-plot descriptors (move to Integ… Oct 8, 2024
@ileasile
Copy link
Collaborator

ileasile commented Oct 8, 2024

@AndreiKingsley I would suggest to create a separate module (artifact) for Jupyter support that will depend on others. It will help in three ways:

  1. It will be possible to build it for JVM target 11 without changing the jvm version for the whole project that will allow render<*>() from the API
  2. It will reduce a number of dependencies in descriptors to only one
  3. It will reduce the current versions hassle

I can help if needed

@AndreiKingsley
Copy link
Contributor Author

@ileasile let's discuss it here.

@alshan alshan changed the title (DON'T MERGE ME) Remove imports/init/renders from lets-plot descriptors (move to Integ… Remove imports/init/renders from lets-plot descriptors (move to Integ… Nov 18, 2024
@alshan alshan merged commit 10fefcc into master Nov 18, 2024
alshan added a commit that referenced this pull request Nov 18, 2024
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