Skip to content

Conversation

macsz
Copy link
Contributor

@macsz macsz commented Mar 13, 2024

In the notebook [1] when plotting columns were not named properly, which caused errors. In this change this problem is fixed.

Additionally, the version of the DyNAS-T was bumped up to 1.5.2, which is aligned with the version installed in other available examples.

[1] examples/notebook/dynas/Transformer_LT_Supernet_NAS.ipynb

In the notebook [1] when plotting columns were not named properly,
which caused errors. In this change this problem is fixed.

Additionally, the version of the DyNAS-T was bumped up to 1.5.2,
which is aligned with the version installed in other available
examples.

[1] examples/notebook/dynas/Transformer_LT_Supernet_NAS.ipynb

Signed-off-by: Maciej Szankin <[email protected]>
Copy link

⚡ Required checks status: All passing 🟢

No groups match the files changed in this PR.


Thank you for your contribution! 💜

Note
This comment is automatically generated and updates for 360 minutes every 180 seconds. If you have any other questions, contact chensuyue or XuehaoSun for help.

@macsz
Copy link
Contributor Author

macsz commented Mar 13, 2024

Failing pre-commit check is a false-positive (spelling check run on base64 encoded image)

@chensuyue chensuyue merged commit 01d966b into master Mar 14, 2024
@chensuyue chensuyue deleted the macsz/fix_hat branch March 14, 2024 08:08
@macsz
Copy link
Contributor Author

macsz commented Mar 14, 2024

Thanks for fixing the spelling check. Could not find the previous PR that had this issue fixed to apply it to this one. Appreciated!

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.

3 participants