amazon-braket / BraketSimulator.jl

Apache License 2.0
9 stars 3 forks source link

Remove Python extension precompilation and give Julia an inferrability assist #18

Closed kshyatt-aws closed 1 month ago

kshyatt-aws commented 1 month ago

Issue #, if available: N/A

Description of changes: Each new Python session creates a new C handle to the Python library, which means type conversion methods are invalidated at each new Python session. Forcing a first run at module import means the actual first simulation is fast, at the cost of a slower import. This means the Python extension precompilation is invalidated.

Testing done: Unit tests passed locally

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

General

Tests

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

codecov[bot] commented 1 month ago

The author of this PR, kshyatt-aws, is not an activated member of this organization on Codecov. Please activate this user on Codecov to display this PR comment. Coverage data is still being uploaded to Codecov.io for purposes of overall coverage calculations. Please don't hesitate to email us at support@codecov.io with any questions.