stfc / PSyclone

Domain-specific compiler and code transformation system for Finite Difference/Volume/Element Earth-system models in Fortran
BSD 3-Clause "New" or "Revised" License
104 stars 27 forks source link

Refresh documentation #2543

Open sergisiso opened 5 months ago

sergisiso commented 5 months ago

Here are some ideas to update the PSyclone documentation on readthedocs, this follows two purposes:

What do you think? Any other re-organization changes that you would do?

hiker commented 5 months ago

I basically agree. The one thing is point 11: I am not sure about is PSyKE - shouldn't that just go into the (new) Instrumentation Tools? And I think that section would be more than long enough to have its own chapter (and not part of "Psyclone transformations".

Obviously, anything with psydata has a transformation ;) But I would prefer if it is listed in the "PSyclone transformations", with just a reference to the corresponding section in the Instrumentation chapter.

One other thing which might be worth addressing, since we update the docs: how do we get the right transformation (e.g. how can a HPC expert know if there is an API-specific transformation exist or not ... even I usually don't know :) )? I have started to discuss this in #978 , but never did anything about it :)

sergisiso commented 5 months ago

And I think that section would be more than long enough to have its own chapter (and not part of "Psyclone transformations".) ... But I would prefer if it is listed in the "PSyclone transformations", with just a reference to the corresponding section in the Instrumentation chapter.

Just to be clear, I didn't mean to move them in the list of psyclone transformations, just to move the page location in the menu to the group of pages called "PSyclone transformations" that maybe also needs a more generic name: image

I agree with your last point but lets discuss this on #978

hiker commented 5 months ago

Ah, I misunderstood. Then it all sounds great, thanks for designing this!!