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

Add Image support in python (.obm) #289

Draft
wants to merge 15 commits into
base: upsilon-dev
Choose a base branch
from

Conversation

Oreig403
Copy link

This pull request adds 2 functions:

  • image_size(image) which takes as argument a string (for example: "image.obm") and returns the size in form of a tuple (w,h)
  • draw_image(image,x,y,w,h) which draws the image
    As I don't know how to code the website so that it converts all of the png files to obm the conversion is only possible through the obm_builder.c file (apart from the wallpaper.obm file which goes through the website)

@Oreig403
Copy link
Author

code
programme

@github-actions
Copy link

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 821588 bytes 478157 bytes 226384 bytes 1816 bytes 228200 bytes 1301561 bytes
Head 821812 bytes 478813 bytes 226376 bytes 1816 bytes 228192 bytes 1302441 bytes
+224 bytes +656 bytes -8 bytes +0 bytes -8 bytes +880 bytes
+0.0 % +0.1 % -0.0 % +0.0 % -0.0 % +0.1 %

apps/code/catalog.en.i18n Outdated Show resolved Hide resolved
python/port/mod/kandinsky/modkandinsky_table.c Outdated Show resolved Hide resolved
Co-authored-by: Yaya-Cout <yaya.cout@ik.me>
@Yaya-Cout
Copy link
Member

It is breaking some builds, so I won't merge it for now, but I would need an improvement in the External API (Maybe moving it into Ion)

@github-actions
Copy link

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 821588 bytes 478157 bytes 226384 bytes 1816 bytes 228200 bytes 1301561 bytes
Head 821812 bytes 478813 bytes 226376 bytes 1816 bytes 228192 bytes 1302441 bytes
+224 bytes +656 bytes -8 bytes +0 bytes -8 bytes +880 bytes
+0.0 % +0.1 % -0.0 % +0.0 % -0.0 % +0.1 %

@github-actions
Copy link

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 821588 bytes 478157 bytes 226384 bytes 1816 bytes 228200 bytes 1301561 bytes
Head 821812 bytes 478813 bytes 226376 bytes 1816 bytes 228192 bytes 1302441 bytes
+224 bytes +656 bytes -8 bytes +0 bytes -8 bytes +880 bytes
+0.0 % +0.1 % -0.0 % +0.0 % -0.0 % +0.1 %

@Oreig403
Copy link
Author

Ok I will look at the breaks after. The improvement should not be too much of a problem ( just a few lines to change)

@Yaya-Cout Yaya-Cout marked this pull request as draft October 21, 2022 19:38
@github-actions
Copy link

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 821588 bytes 478157 bytes 226384 bytes 1816 bytes 228200 bytes 1301561 bytes
Head 821836 bytes 478813 bytes 226376 bytes 1816 bytes 228192 bytes 1302465 bytes
+248 bytes +656 bytes -8 bytes +0 bytes -8 bytes +904 bytes
+0.0 % +0.1 % -0.0 % +0.0 % -0.0 % +0.1 %

@github-actions
Copy link

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 821588 bytes 478157 bytes 226384 bytes 1816 bytes 228200 bytes 1301561 bytes
Head 821836 bytes 478813 bytes 226376 bytes 1816 bytes 228192 bytes 1302465 bytes
+248 bytes +656 bytes -8 bytes +0 bytes -8 bytes +904 bytes
+0.0 % +0.1 % -0.0 % +0.0 % -0.0 % +0.1 %

@Oreig403
Copy link
Author

Oreig403 commented Oct 22, 2022

It seems the n0100 build was fixed but I don't get how to fix the simulator problems

@Yaya-Cout
Copy link
Member

You need to move the External API into Ion (#290), because it crashes when flags are not set.

@Oreig403
Copy link
Author

Ok, I will try to do that later.

@github-actions
Copy link

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 821588 bytes 478157 bytes 226384 bytes 1816 bytes 228200 bytes 1301561 bytes
Head 821836 bytes 478813 bytes 226376 bytes 1816 bytes 228192 bytes 1302465 bytes
+248 bytes +656 bytes -8 bytes +0 bytes -8 bytes +904 bytes
+0.0 % +0.1 % -0.0 % +0.0 % -0.0 % +0.1 %

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.

2 participants