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

RADIUSS Tutorial Reflection #81

Closed
slabasan opened this issue Aug 17, 2023 · 2 comments
Closed

RADIUSS Tutorial Reflection #81

slabasan opened this issue Aug 17, 2023 · 2 comments

Comments

@slabasan
Copy link
Collaborator

  • Update rendered tutorial notebooks in thicket
  • Update tutorials page with materials
  • Rendered basic tutorial is using new RAJAPerf data, this shouldn't be the case yet because we want to update all
  • Use display instead of print for dataframes, will all columns of dataframe render?
@ilumsden
Copy link
Collaborator

The following PRs fix a lot of the syncing between the Thicket docs/ directory and the tutorial repo:

Also, regarding display and print, we should definitely be using display for DataFrames. However, when running in Jupyter, there is no need to manually invoke the display function. When run as Jupyter cells, the following pieces of code are equivalent:

Explicit display:

df = pd.DataFrame(...)
...
display(df)

Implicit display:

df = pd.DataFrame(...)
...
df

Also, when running in Jupyter, there is no need to manually import display from IPython.display. The display function is auto-imported by Jupyter.

@slabasan
Copy link
Collaborator Author

Several PRs resolve this issue: #121 #120 #119 #117

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

2 participants