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

adds Compat package and fixes tuplecopalyse errors and Dict warnings #15

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

milktrader
Copy link
Member

You don't need to merge this if it's a distraction, I just submitted it because I'm on bleeding-edge Julia and this fixes the load failures and the deprecation warnings. I've commented out the offending lines rather than delete them also.

@multidis
Copy link
Contributor

Still hesitating on this one due to:

  • unknown performance effect on compiler
  • Reactive modifications yet to be merged (?)

@milktrader
Copy link
Member Author

Yep, no problem. This will have the correct code for v0.4 for future use once we get to that version.

@multidis
Copy link
Contributor

Let's keep this open for v0.4 transition even though the other Compat-PR was merged.

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.

2 participants