-
Notifications
You must be signed in to change notification settings - Fork 1
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
Questions about "Construct GEMs" #25
Comments
Hello wangzlgithub, Wow, good catch! This explains why a previous issue had noticed a discrepancy between the GEMs I generated for the original paper generated by the current version of FlowSig. Thank you so much! I will update FlowSig ASAP. The simple fix should be to replace line 27 with: |
Hello wangzlgithub, |
I have also have trouble with installing this package according current install mehtods. Could you provide us a simple and successful way ? Thank you very much! @axelalmet |
Hello, thanks for developing this fantastic tool.
I don't think the parameter
layer_key
play a role when the inputting data has been normailzed in Construct GEMs step:Because I saw the source codes in
flowsig/preprocessing/_gem_construction.py
as below:flowsig/flowsig/preprocessing/_gem_construction.py
Lines 20 to 31 in f86164d
The text was updated successfully, but these errors were encountered: