-
Notifications
You must be signed in to change notification settings - Fork 9
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
[DevX boostrap] cut down closure size of iohk/devx #21
Comments
I was planning to start by giving a try to @lf- experiment: https://jade.fyi/nixcon2022/ The idea would be to copy-paste content of JSON output generated inside https://mercurytechnologies.github.io/looking-glass-viewer/ visualizer! And to obtain them with the command:
… where
That I could reproduce using only
|
This works better:
… and attached the different outputs: |
Add so on the JSON output that could be visualized through https://mercurytechnologies.github.io/looking-glass-viewer/: … obtained with, e.g.:
|
Heavy closure (potentially removable?) found while playing with the tool (using
|
Hi! The issue you're having with the invalid paths is that Nix is issuing the worst possible error for not having built it yet. That's most likely what's up :) |
Oh! Thanks for the explanation, that's somehow what I understood but with the still unlogical fact that running And also thanks again for your so handy tool! Having an interactive thing is really useful when we have to inspect the huge number of derivations that contain a closure. Maybe having some extra overkill features like the possibility to mark with colours the graph while playing with it, or to filter out nodes under a given size in the interface would be great. Happy to find extra time in the future to contribute a bit to it. 🙂 |
Yeah, for sure, one feature I was wanting to implement but didn't get the chance is search (which actually exists in one of the examples for sigma.js); IMO the prototype there should be rewritten in react or similar; it was already running into "writing plain js is unpleasant" by the time i got the data table built. Since I am done my internship at Mercury, it's probably best to build a new looking glass viewer forked off the last one, since I am not sure what the reviews situation is given it's not really an official project that is intended to be maintained into the future. It's possible that it's building a different derivation or something similarly insane; maybe --debug helps? i know that i would generally |
We should analyse the size of the closure needed for:
… and similar (also
--minimal
).And then figure out why they are too large and what could (or should) be dropped.
The text was updated successfully, but these errors were encountered: