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

WIP: Update1.x #29

Closed
wants to merge 22 commits into from
Closed

WIP: Update1.x #29

wants to merge 22 commits into from

Conversation

merl-dev
Copy link

This will run on Julia 1.x, skipped about a dozen tests. Some thought given to performance but nothing implemented here. Will likely continue to work on it this week.

@Evizero Evizero self-requested a review January 27, 2019 18:37
@merl-dev
Copy link
Author

merl-dev commented Feb 3, 2019

  • Most skipped tests fixed.
  • Examples running and generated workbooks.
  • Documenter won't complete on error UndefVarError: include not defined

@merl-dev
Copy link
Author

merl-dev commented Feb 3, 2019

  • Note that it builds successfully on nightly, but not on julia 1.0.3. I develop on 1.1. These errors seem to be due to commit 6516b85 where I removed patch code for OffsetArrays.no_offset_view and IdentityUnitRange which made it's way into Base. The latter failure makes sense, the former doesn't.
  • The reason for the OSX failure on nightly is something I've noted sporadically on linux: tst_scale.jl eager-fixed parameter tests. I can't reproduce the error as it happens only rarely.

@@ -1,3 +1,3 @@
build/
site/
src/generated/*
#src/generated/* # this should be removed
Copy link
Owner

Choose a reason for hiding this comment

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

why?

Copy link
Author

Choose a reason for hiding this comment

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

I have a bunch of items that will be removed when the upgrade is complete

Copy link
Owner

Choose a reason for hiding this comment

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

the thing is this causes a couple of files to be commited that shouldn't be

Copy link
Author

Choose a reason for hiding this comment

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

Yes I am aware ... have been holding back more edits, mostly doc related.

@simonschoelly
Copy link

@merl-dev Are you still working on this PR?

@merl-dev
Copy link
Author

@merl-dev Are you still working on this PR?

Not presently

@johnnychen94
Copy link
Collaborator

johnnychen94 commented Dec 16, 2019

I'm starting to continue the update work #27 and here in #31 , so I guess this PR is deprecated now and only serves for reference usage, will be closed when it successfully updates to 1.x

@johnnychen94 johnnychen94 added this to the v0.6 milestone Dec 19, 2019
@johnnychen94
Copy link
Collaborator

johnnychen94 commented Jan 29, 2020

close this by #31

Although there're some bugs need to be fixed, branch jc/1.x is now 1.x-compatible

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