Skip to content

Add Publication List#1900

Merged
dpanici merged 15 commits intoPlasmaControl:masterfrom
singh-jaydeep:adding_pubs
Sep 17, 2025
Merged

Add Publication List#1900
dpanici merged 15 commits intoPlasmaControl:masterfrom
singh-jaydeep:adding_pubs

Conversation

@singh-jaydeep
Copy link
Copy Markdown
Collaborator

  • Creates file pubs_list.rst listing publications which use DESC (up to 9/8/25)
  • Adds pubs_list.rst to the table of contents in index.rst

Resolves #1838

- Creates file pubs_list.rst listing publications which
  use DESC (up to 9/8/25)
- Adds pubs_list.rst to the table of contents for documentation

Resolves PlasmaControl#1838
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 8, 2025

Memory benchmark result

|               Test Name                |      %Δ      |    Master (MB)     |      PR (MB)       |    Δ (MB)    |    Time PR (s)     |  Time Master (s)   |
| -------------------------------------- | ------------ | ------------------ | ------------------ | ------------ | ------------------ | ------------------ |
  test_objective_jac_w7x                 |    7.06 %    |     3.782e+03      |     4.049e+03      |    266.94    |       34.35        |       31.12        |
  test_proximal_jac_w7x_with_eq_update   |    0.99 %    |     6.794e+03      |     6.861e+03      |    67.33     |       159.40       |       158.11       |
  test_proximal_freeb_jac                |    0.24 %    |     1.318e+04      |     1.321e+04      |    31.29     |       78.23        |       77.22        |
  test_proximal_freeb_jac_blocked        |    0.57 %    |     7.603e+03      |     7.647e+03      |    43.70     |       70.31        |       68.38        |
  test_proximal_freeb_jac_batched        |   -0.07 %    |     7.560e+03      |     7.555e+03      |    -5.54     |       71.21        |       68.58        |
  test_proximal_jac_ripple               |   -1.53 %    |     7.647e+03      |     7.530e+03      |   -117.07    |       69.81        |       68.91        |
  test_proximal_jac_ripple_spline        |   -1.50 %    |     3.498e+03      |     3.445e+03      |    -52.59    |       71.12        |       72.73        |
  test_eq_solve                          |    0.38 %    |     2.050e+03      |     2.058e+03      |     7.78     |       124.15       |       124.37       |

For the memory plots, go to the summary of Memory Benchmarks workflow and download the artifact.

@rahulgaur104
Copy link
Copy Markdown
Collaborator

Thanks for doing this, Jaydeep!
Could you also add the following preprint to the list?
https://arxiv.org/abs/2505.04211

@rahulgaur104 rahulgaur104 added the run_benchmarks Run timing benchmarks on this PR against current master branch label Sep 8, 2025
@singh-jaydeep
Copy link
Copy Markdown
Collaborator Author

singh-jaydeep commented Sep 8, 2025

Thanks for doing this, Jaydeep! Could you also add the following preprint to the list? https://arxiv.org/abs/2505.04211

Yes, will do! I'll check to see if there are other papers I've missed.

- Creates file pubs_list.rst listing publications which
  use DESC (up to 9/8/25)
- Adds pubs_list.rst to the table of contents for documentation

Resolves PlasmaControl#1838
ddudt
ddudt previously approved these changes Sep 8, 2025
@YigitElma YigitElma requested review from a team, YigitElma, dpanici, f0uriest, rahulgaur104 and unalmis and removed request for a team September 9, 2025 16:00
Copy link
Copy Markdown
Collaborator

@dpanici dpanici left a comment

Choose a reason for hiding this comment

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

@rahulgaur104
Copy link
Copy Markdown
Collaborator

Did the Type One paper actually use DESC?

Also, I think it would be even better if we can put a link to the publication list in README.rst and/or the docs.
Otherwise, it would be left to be explored by the users.

@f0uriest
Copy link
Copy Markdown
Member

Agree about adding the list to the readme, maybe just as a direct ..include? or maybe formatted as a table like the one shorter list there now?

Also wondering if we want to split the list into "papers about desc" ie ones by us that describe new features being added, vs "papers that use desc" to get other results but don't really add anything new to the code.

papers about desc, numbered from the existing list: 1, 3-10, 12, 13, 15, 19

-Adds additional papers
-Standardizes the formatting for names, using et al. if >10 authors.
singh-jaydeep and others added 6 commits September 10, 2025 11:52
Divides list into papers which extend DESC, and those which apply it without significant additions to the code.
Changing from "list" format to "citations"
Adds a link to the relevant page in the docs
Copy link
Copy Markdown
Collaborator

@dpanici dpanici left a comment

Choose a reason for hiding this comment

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

read the docs seems to be failing?

-Reverted to a numbered list from footnote style, in case this was causing the read the docs build error
-Added some CSS to style the list
@singh-jaydeep
Copy link
Copy Markdown
Collaborator Author

singh-jaydeep commented Sep 11, 2025

Hmm...I think the list formatting was causing an error. Will see if this works.

@YigitElma
Copy link
Copy Markdown
Collaborator

FYI, sometimes debugging the docs build locally is faster (you don't wait for the runner to pick up, etc). You can follow https://github.com/PlasmaControl/DESC/tree/master/docs#readme to build docs locally.

@singh-jaydeep
Copy link
Copy Markdown
Collaborator Author

Oh makes sense, thanks!

@YigitElma YigitElma requested review from ddudt and dpanici September 17, 2025 17:45
@dpanici dpanici merged commit c48d439 into PlasmaControl:master Sep 17, 2025
26 checks passed
@singh-jaydeep singh-jaydeep deleted the adding_pubs branch September 18, 2025 00:16
DMCXE pushed a commit to DMCXE/DESC-OOPS that referenced this pull request Oct 14, 2025
- Creates file pubs_list.rst listing publications which use DESC (up to
9/8/25)
- Adds pubs_list.rst to the table of contents in index.rst

Resolves PlasmaControl#1838

---------

Co-authored-by: Yigit Gunsur Elmacioglu <102380275+YigitElma@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run_benchmarks Run timing benchmarks on this PR against current master branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make list of publications that use the code

6 participants