Skip to content

Issues caused by the frontend rework #177

@bwintermann

Description

@bwintermann

Issues that need fixing after we merged #118 into dev.

  • Turns out that, apparently, Updated Frontend and Settings #118 accidentally changed the default location of finn_deps to ./finn_deps instead of ~/.finn/finn_deps. This should be changed back for the next release.
  • In case that, for example, unzip is not available, the dependency update crashes without specifying the exact error. We should check for all tools that are required (unzip, wget, git, etc.)
  • Not a bug, but the more strict deserialization caught a typo in a bench config, fixed in Fix typo in dut file #179
  • When executing commands such as finn deps update, due to the default value of finn_build_dir, a local FINN_TMP directory will be created where the finn command was executed.
  • Cannot restart from intermediate models because QONNXs' ModelWrapper does not accept Path objects
    • Same goes for the bench configs in finn bench
  • Some articles in the wiki still need updates

Slightly related:

  • Add fitting default values to make local / non-CI based execution of finn bench smoother (Unless there are for example reasons why a CI_PIPELINE_ID is needed, etc. @fpjentzsch?)

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions