Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@MakisH
Copy link
Member

@MakisH MakisH commented Feb 17, 2022

This add a few hints that are relevant for multiple tutorial cases. When one wants to visualize the results of, e.g., the multiple perpendicular flaps case, should not look into the perpendicular flap case for that.

Current state:

Screenshot from 2022-02-17 16-12-59

@uekerman or @davidscn or @IshaanDesai do you agree with the structure? Would you add anything else here?

@MakisH MakisH self-assigned this Feb 17, 2022
Copy link
Member

@davidscn davidscn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I generally agree on the hints. However, I'm not sure if this is the right place to put them. Solver specific stuff usually belongs into the solver sections. If the hints grow in the future the page might get a bit bloated.

@MakisH
Copy link
Member Author

MakisH commented Feb 17, 2022

Solver specific stuff usually belongs into the solver sections.

I understand what you mean, but we don't really have "solver sections", we mostly have "adapter sections" (for some of which we only have one page). Most of these instructions would also apply to anyone using just the uncoupled solvers. An in the tutorials they are currently more visible.

If the hints grow in the future the page might get a bit bloated.

I hope that it will not grow much more. If it does in the future, then we could also move it to a dedicated page.

Let's leave it open until the workshop and we may collect more ideas and opinions.

Copy link
Member

@IshaanDesai IshaanDesai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree that such hints are very helpful and oftentimes also necessary as I myself have sometimes struggled to understand how to visualize the results. The amount of content right now seems reasonable to be placed on the tutorials main page, but if at some point it becomes twice its size then it would be too much for the main page.

@MakisH
Copy link
Member Author

MakisH commented Feb 18, 2022

Then I am adding this already and we can get feedback during the workshop.

@MakisH MakisH merged commit 3221d59 into master Feb 18, 2022
@MakisH MakisH deleted the tutorial-hints branch February 18, 2022 09:51
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

Successfully merging this pull request may close these issues.

3 participants