Skip to content

[GSOC] CARSUS Dashboard #440

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Nitish-Naik
Copy link
Contributor

@Nitish-Naik Nitish-Naik commented Mar 22, 2025

📝 Description

Type: | 🚀 feature | ☣️ breaking change | 🎢 infrastructure

This PR introduces Carsus Dashboard using Jinja2 to generate an HTML Report of Silicon for Levels and Lines dataframe.

📌 Resources

Execution Video (watch here): Demo
https://drive.google.com/drive/folders/1wjSPCB536eDJU3IDz-sKwUZWbxQQOsMM?usp=sharing

🚦 Testing

How did you test these changes?

  • Testing pipeline
  • Other method (describe)
  • My changes can't be tested (explain why)

☑️ Checklist

  • I requested two reviewers for this pull request
  • I updated the documentation according to my changes
  • I built the documentation by applying the build_docs label

Note: If you are not allowed to perform any of these actions, ping (@) a contributor.

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@Nitish-Naik
Copy link
Contributor Author

Hello Mentors, @jvshields @wkerzendorf can you please review
Eagerly Waiting for your response
Nitish

Copy link
Contributor

github-actions bot commented Mar 22, 2025

*beep* *bop*
Hi human,
I ran ruff on the latest commit (ae74d92).
Here are the outputs produced.
Results can also be downloaded as artifacts here.
Summarised output:

3	F811	[*] redefined-while-unused
2	E902	[ ] io-error
1	F401	[*] unused-import
1	F541	[*] f-string-missing-placeholders

Complete output(might be large):

carsus/carsus_dashboard.ipynb:cell 4:1:20: F811 [*] Redefinition of unused `Template` from cell 3, line 1
carsus/carsus_dashboard.ipynb:cell 4:2:18: F811 [*] Redefinition of unused `pd` from cell 3, line 2
carsus/carsus_dashboard.ipynb:cell 4:2:18: F401 [*] `pandas` imported but unused
carsus/io/lanlads/base.py:1:1: E902 No such file or directory (os error 2)
carsus/io/lanlads/parsers.py:1:1: E902 No such file or directory (os error 2)
carsus/io/output/base.py:267:25: F541 [*] f-string without any placeholders
carsus/io/util.py:8:8: F811 [*] Redefinition of unused `requests` from line 2
Found 7 errors.
[*] 5 fixable with the `--fix` option.

@Nitish-Naik
Copy link
Contributor Author

Hello mentor @jvshields can you please review..
So that I can draft my proposal accordingly..
Eagerly waiting for your response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants