-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Enhanced singlefile conversion and support for DE1 formats #1148
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
Conversation
21baf88 to
a3ec958
Compare
TheJJ
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.
awesome work!
| """ | ||
| return determine_rgba_matrix(self.pcolor, palette, player_number) | ||
| # fetch drawing instruction | ||
| cmd = self.get_byte_at(dpos) |
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.
we have to make sure this is properly resolved to a c++-only call. we can see that in the cythonized html file. otherwise this will really slow down the conversion.
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.
Wouldn't the function declaration with cdef ensure that?
| cdef class SMPOutlineFrame(SMPFrame): | ||
| """ | ||
| SMPFrame for the outline graphics. | ||
| """ |
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.
hm, i wonder why they are doing that. in any way, if the outline information is separate from the graphics info, we have to merge them in order to do our alpha-encoding.
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.
Probably because they wanted a smaller number of commands (?) Or something to do with their expanded rendering. Merging should work fine, since he outlines does not overlay with any other pixels in the main graphic.
| subtex = TextureImage( | ||
| frame.get_picture_data(palette, self.player_id), | ||
| frame.get_picture_data(main_palette, player_palette), |
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.
here we have to do the merging of the outline info and the graphics i think.
what do we do about the damage graphics? (we should likely discuss that in the chat :)
TheJJ
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.
whee
Adds more options to the "singlefile" conversion to support newer game versions and SLPs outside of DRS archives. Specifically: