-
Notifications
You must be signed in to change notification settings - Fork 7
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 of bug #140 #141
Fix of bug #140 #141
Conversation
Can you add a short section to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
very nice! I just have a few very small change requests which I added as single line comments
Note: You don't have to fix (all) codeclimate issues. Howver, the docs should work, so this is actually a problem: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a short section to release_note.md
using a new section on the top: ## [Unreleased]?
Very good job, thanks! |
This pull request should close issue #140 the bug of missing vector attribute when saving/loading from disk.
I've made a new issue for the same bug in the matrix class (see issue #142).