-
Notifications
You must be signed in to change notification settings - Fork 15
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
Add Bootstrap classes to Markdown tables #152
Conversation
For some reason, a lot of lines in the changed documents had trailing whitespaces, so please ignore whitespace changes in the diff ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Markdown tables are generated without any classes, but Bootstrap requires adding `.table` classes in order for a table to be styled. Closes #151
Only if some text is out of the I've changed the text and formatting, but I'm not sure what the original documentation author meant (that is, where are |
Hi @emilmelnikov , the table looks great! As for the Object IDs in that table: There are actually no identifiers in the data (I checked :o, e.g. here in case you want to make your eyes bleed). So this is more or less arbitrary. Maybe rename the |
that's cool. Could you check whether it is actually possible to generate these fields in export files? I have no clue how MultiFrameMoves are supposed to happen. (Quickly tried, but didn't manage to generate one). Also about the energies: I would not know where these should come from in manual tracking... |
I don't really understand very well what is going on in the manual tracking code, but it seems that multi-frame merges happen on this condition: https://github.com/ilastik/ilastik/blob/a7ee78d7936ed573b10c6ff4d8733e1f126a00ea/ilastik/applets/tracking/manual/manualTrackingGui.py#L938
Yes, I looked at the code, and in manual tracking we always write 0 energy for each event. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 the tracking docs finally look somewhat bearable on mobile devices...
Markdown tables are generated without any classes, but Bootstrap
requires adding
.table
classes in order for a table to be styled.Closes #151