-
Notifications
You must be signed in to change notification settings - Fork 27
Open
Labels
design-issueThis is a design issue and requires major changeThis is a design issue and requires major changedocumentationImprovements or additions to documentationImprovements or additions to documentation
Milestone
Description
In INEQR, the pixel_pos_binary needs to be a binary string y_coord+x_coord long. However, currently, the code misrepresents the binary with extra 0 padding.
piQture/piqture/data_encoder/image_representations/ineqr.py
Lines 84 to 88 in 76f1718
| for y_index, y_val in enumerate(self.pixel_vals): | |
| for x_index, x_val in enumerate(y_val): | |
| pixel_pos_binary = ( | |
| f"{y_index:0>{self.y_coord}b}{x_index:0>{self.x_coord}b}" | |
| ) |
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
design-issueThis is a design issue and requires major changeThis is a design issue and requires major changedocumentationImprovements or additions to documentationImprovements or additions to documentation