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

Option to hide "xxx is not supported" #1134

Open
hongbo-miao opened this issue Jan 24, 2025 · 3 comments
Open

Option to hide "xxx is not supported" #1134

hongbo-miao opened this issue Jan 24, 2025 · 3 comments

Comments

@hongbo-miao
Copy link

hongbo-miao commented Jan 24, 2025

Would it be a good idea to hide these info by default or provide an option? Thanks! ☺️

ldf is not supported
xls is not supported
xlsx is not supported
@danielhrisca
Copy link
Owner

This is coming from canmatrix

If it bothers you , then install the missing dependencies

@MatinF
Copy link
Contributor

MatinF commented Jan 24, 2025

@hongbo-miao alternatively consider raising this in the canmatrix github as a suggestion

@hongbo-miao
Copy link
Author

hongbo-miao commented Jan 24, 2025

I see, thanks for the info! I will ask on canmatrix side.

I would not say "bother". However, it does confuse a lot of people. For example this was asked on Reddit 4 years ago here.

I had a same question. My code is very simple which is just reading MF4 file: https://github.com/hongbo-miao/hongbomiao.com/blob/main/embedded-system/decode-can-mf4-data/src/main.py

I am curious if install extra dependencies (and which one exactly?), what would be benefits other than hide the warning? Does it only support more file formats that I actually don’t need in my case. (Because I know some libraries for example after adding some optional dependencies, it can help use GPU, or help handle large data in low memory devices.)

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

No branches or pull requests

3 participants