graspologic-org / graspologic

Python package for graph statistics
https://graspologic-org.github.io/graspologic/
MIT License
634 stars 134 forks source link

Loftusa patch 3 #1054

Closed loftusa closed 1 month ago

loftusa commented 6 months ago

@bdpedigo proposed fix to #1035. Kinda janky, but it bought my imports down to 2.35 seconds:

Screenshot 2023-12-11 at 3 04 37 PM
bdpedigo commented 6 months ago

linear_sum_transport gets called many many times during an optimization so (if i understand what this will do correctly) that import would also happen over and over again if that code is called - not sure that is viable. would ideally get at something closer to the root cause of this (for instance, why does POT need to even import tensorflow here...)

bdpedigo commented 1 month ago

closing since there hasnt been action on this but happy to revisit if there's interest