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

Easily install missing IronPython extension package #10869

Merged
merged 15 commits into from
Jul 16, 2020

Conversation

aparajit-pratap
Copy link
Contributor

@aparajit-pratap aparajit-pratap commented Jul 9, 2020

Purpose

Easily install missing IronPython extension package.
python_package

Declarations

Check these if you believe they are true

  • The codebase is in a better state after this PR
  • Is documented according to the standards
  • The level of testing this PR includes is appropriate
  • User facing strings, if any, are extracted into *.resx files
  • All tests pass using the self-service CI.
  • Snapshot of UI changes, if any.
  • Changes to the API follow Semantic Versioning and are documented in the API Changes document.

Reviewers

@DynamoDS/dynamo

FYIs

(FILL ME IN, Optional) Names of anyone else you wish to be notified of

@aparajit-pratap aparajit-pratap marked this pull request as ready for review July 10, 2020 16:39
Copy link
Collaborator

@mmisol mmisol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some questions related to this PR and the future. Probably something we should answer as a team.

What should be the strategy for releasing future versions of the Iron Python engine package? Should 2.7.9 be the version here and whenever 3.0.0 is released we deprecate the previous version of the package in PM (if that's possible)?

We talked about removing the migration functionality in Python Epic 8. I think we should make sure to do that at a time it is no longer relevant, which would have to be some time after we deprecate the 2.7.9 version of the Iron Python engine package. I'm not sure if it should be enough to infer some info about usage from Package Manager, by tracking downloads, or if we should use analytics to cover cases where users have already installed the package and are just using it.

@@ -12,6 +12,9 @@ namespace Dynamo.PythonMigration
public class GraphPythonDependencies
{
private ViewLoadedParams ViewLoaded { get; set; }
internal static readonly string PythonPackage = "DSIronPython_Test";
internal static readonly Version PythonPackageVersion = new Version(1, 0, 7);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These will be updated once the IronPython extension is published to PM prod.

@aparajit-pratap
Copy link
Contributor Author

What should be the strategy for releasing future versions of the Iron Python engine package? Should 2.7.9 be the version here and whenever 3.0.0 is released we deprecate the previous version of the package in PM (if that's possible)?

@mmisol, as discussed I think the strategy is to add new packages for new engine versions and make them available in the PM in the future.

@aparajit-pratap aparajit-pratap added the PTAL Please Take A Look 👀 label Jul 16, 2020
@aparajit-pratap aparajit-pratap merged commit 65751e4 into DynamoDS:master Jul 16, 2020
@aparajit-pratap aparajit-pratap deleted the dyn-2787 branch July 16, 2020 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PTAL Please Take A Look 👀
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants