FPGAwars / apio

:seedling: Open source ecosystem for open FPGA boards
https://github.com/FPGAwars/apio/wiki
GNU General Public License v2.0
801 stars 137 forks source link

Assorted apio blocking items #453

Open zapta opened 1 week ago

zapta commented 1 week ago

Hi @cavearr, I am nearing the end of the planned feature list and next I will focus on testing and cleanup. Here are a few action items for you that blocks the cleanup.

  1. [DONE] Build oss-cad-tools release 0.2.0 so I can switch the apio dev to it (should not affect apio prod)

  2. Make icestudio using apio.ini so we can make it required and do a bunch of related clean ups.

  3. If icestudio uses apio time please switch to apio report so we can delete apio time.

  4. if icestudio uses apio install or apio uninstall, switch to apio packages so we can delete them.

  5. If icestudio uses apio verify switch to apio lint so we can delete it.

If you have any questions please let me know.

Obijuan commented 1 week ago
  1. Build oss-cad-tools release 0.2.0 so I can switch the apio dev to it (should not affect apio prod)

I've tried to build it but the scripts gives an error:

https://github.com/FPGAwars/tools-oss-cad-suite/actions/runs/11792202730/job/32845461579

When creating the window package it is run out of space: "No space left on device" Not sure if it is a temporal error or not. I will try to build it later. If the error persist, I will create a release and we can upload the packages manually

@zapta Sorry for no answering all your questions. During this quarter I have to give a lot of lectures at the university and I have no time to spend on Apio. But I will have more free time in a month or so. My stutends use Icestudio 0.12 a lot (with apio 0.9.5). Once the lectures are finished, I will test apio and icestudio for the next release cycle (the new version is the one I will use with the students on the next course)

cavearr commented 1 week ago

Hi @zapta , I recommend that your students use the latest wips https://downloads.icestudio.io

This week I started uploading many changes that stabilize and fix UI/UX bugs and in a few days it will integrate your celery changes.

I am going to integrate into icestudio this week the new tools that celery will include by default.

Regarding the actions problem, I am already in the process of solving it, actions have a maximum size, if you cover it you have to go to other options (external workers hosted in your infrastructure and things like that) I am going to try to solve it by deleting temporary files during the build.

Thanks for the great work on Apio.

cavearr commented 1 week ago

@zapta , @Obijuan 0.2.0 release are ready!!! 💯

cavearr commented 1 week ago

I think today at the end of the day i'm relasing a new wip with your changes.

zapta commented 1 week ago

Hi @cavearr, sent you https://github.com/FPGAwars/apio/pull/454 to switch apio dev from my repo to the FPGAwars release you created. After you will update your apio, run that command to update the package and review the repo and version it uses.

zapta commented 4 hours ago

Regarding item 2, some of the deprecated stuff we can delete there are the flags --size --type --pack, but it's possible that they don't work anyway and therefore can be deleted now.

See https://github.com/FPGAwars/apio/issues/468 for details.