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

Allow color/size export #113

Closed
pragunap2e opened this issue Aug 22, 2023 · 13 comments
Closed

Allow color/size export #113

pragunap2e opened this issue Aug 22, 2023 · 13 comments

Comments

@pragunap2e
Copy link

After editing the graph, allow color/size of nodes / edges to be stored locally.

@wey-gu
Copy link

wey-gu commented Sep 5, 2023

+1, I hope this could be added, too

@pragunap2e
Copy link
Author

hi, @sim51 is this possible

@wey-gu
Copy link

wey-gu commented Sep 6, 2023

I did not examine the example file, but that information remained there as they were color-sized after the initial load.

@sim51
Copy link
Collaborator

sim51 commented Sep 6, 2023

I don't really see what you mean by "allow color/size of nodes / edges to be stored locally".

What I can say is when you change the color, size, etc of a graph, save it locally (ie export it as a gexf), and then you load it in gephi-lite, you lost what you have done in the appearance panel.
is it your issue ?

@pragunap2e
Copy link
Author

yes, for example if I change color / size and save as local graph, then it won't be present when it is reloaded. Layout changes are loadable. After saving the first graph the second one is loaded.

saving_this
saved_after

@sim51
Copy link
Collaborator

sim51 commented Sep 6, 2023

Thanks for your reply and the screenshot, the issue is well defined now.

FYI, the gexf format is not good enough right now for our use-case (we also work on the next syntax of gexf format).

On nodes, gexf has the following attributes : size, color, label and properties. When you load a graph on gephi-lite per default, we took those values.
We can do a GEXF export that override the size, color, label properties on each node, but you will loose all the initial values and the legend on the graph.
There is no way to store in GEXF (actual version) file that the color of node should be taken from a specific property of the node.

During our next sprint on gephi-lite, we will discuss the file formats that we want to use on gephi-lite.

@wey-gu
Copy link

wey-gu commented Sep 6, 2023

Thanks for the explanation, now I know more about .gexf, it makes sense that we don't override those information when they are parts of the attributes.

Looking forward to the next sprint! @sim51 , gephi-lite is already extra-amazing to me.

@pragunap2e
Copy link
Author

Thanks @sim51 is there any workaround for now, where I could reset color/ size values from the exported file.

@sim51
Copy link
Collaborator

sim51 commented Sep 6, 2023

Unfortunately, none to my knowledge

@sim51
Copy link
Collaborator

sim51 commented Sep 8, 2023

By discussing with @jacomyal , it's a bug.
We should export the visible graph in gexf, even if we lost the caption of the graph (but we don't have the choice for now).

@wey-gu
Copy link

wey-gu commented Sep 8, 2023

By discussing with @jacomyal , it's a bug.

We should export the visible graph in gexf, even if we lost the caption of the graph (but we don't have the choice for now).

Ah... seems we just do the filter of the graph imported when exporting and not add attributes of appearance from the appearanceState, if it's good to go, I can try PR to add appearance states into file when exporting, is that ok?

@jacomyal
Copy link
Contributor

jacomyal commented Sep 8, 2023

Hi @wey-gu,
Really sorry, I was with @sim51 today and we discussed this issue and I had a bit of time, I didn't see your latest message first. The fix is pushed and deployed.
You seem to have already taken a good look into the code though, if you see smelly things in the codebase or want to help testing the app or writing more tests, your contributions will be warmly welcome :)

@jacomyal jacomyal closed this as completed Sep 8, 2023
@wey-gu
Copy link

wey-gu commented Sep 8, 2023

Hi @wey-gu,

Really sorry, I was with @sim51 today and we discussed this issue and I had a bit of time, I didn't see your latest message first. The fix is pushed and deployed.

You seem to have already taken a good look into the code though, if you see smelly things in the codebase or want to help testing the app or writing more tests, your contributions will be warmly welcome :)

WOW thanks for the quick change and warm welcome!

I really like gephi-lite and will definitely try to contribute in the future!

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

4 participants