-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Cython smp #1735
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
Cython smp #1735
Conversation
695b2ba to
b3f3391
Compare
|
should I add @9a24f0 to .mailmap or copying.md? |
heinezen
left a comment
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.
I've made comments to the SMX file only, but they apply to the SMP file in the same way. The graphics seem to convert fine, so there are only structural changes left to do.
If you based your changes on their code, then yes, you have to add them to copying.md. |
|
@heinezen I've restructured the changes, please take a look. passing a variable of type Variant through the base class constructor e.g. |
|
@jere8184 I've managed to remove the inheritance from the classes used by the fused |
|
I'll do a few performance test before I merge. Hopefully we didn't make the code slower accidentally :D |
Nice |
|
DE2 media export time on this MR: DE2 media export time on It's probably safe to say that the difference is negligeable :P |
heinezen
left a comment
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.
I added documentation to the classes, so it should be much easier to work on the cython files from now on :)
Closes #1370.