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

Fix some issues with examples. #561

Merged
merged 1 commit into from
Jan 10, 2017
Merged

Conversation

ketch
Copy link
Member

@ketch ketch commented Jan 10, 2017

  1. Automatically compile things (by calling Make) where necessary.
  2. Use absolute imports for Python 3 compatibility.

Also some miscellaneous fixes, like replacing spaces with tabs in a makefile.

This avoids some errors that can appear when building the gallery, as noted by @rjleveque .

@coveralls
Copy link

Coverage Status

Coverage decreased (-41.3%) to 27.389% when pulling 582bed3 on ketch:update_examples into 8020d18 on clawpack:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-41.3%) to 27.389% when pulling 71f9834 on ketch:update_examples into 8020d18 on clawpack:master.

1. Automatically compile things (by calling Make) where necessary.
2. Use absolute imports for Python 3 compatibility.

Also some miscellaneous fixes, like replacing spaces with tabs in a makefile.
@coveralls
Copy link

coveralls commented Jan 10, 2017

Coverage Status

Coverage decreased (-22.8%) to 45.91% when pulling a40c235 on ketch:update_examples into 8020d18 on clawpack:master.

@rjleveque
Copy link
Member

The gallery builds fine now. Only the Python3 Travis tests are failing, so I guess we are ready to merge this and release.

@ketch ketch merged commit 612b8fd into clawpack:master Jan 10, 2017
@ketch ketch removed the in progress label Jan 10, 2017
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.

None yet

3 participants