Skip to content

update publications, references, etc. #194

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

Merged
merged 8 commits into from
Nov 4, 2020
Merged

update publications, references, etc. #194

merged 8 commits into from
Nov 4, 2020

Conversation

rgcoe
Copy link
Member

@rgcoe rgcoe commented Oct 22, 2020

Update the DOI for the TechRxiv preprint and note acceptance of paper by Journal of Ocean Engineering and Marine Energy.

closes #190 #172

EDIT: additionally now updated References page to include project publications

@rgcoe rgcoe added the documentation Improvements or additions to documentation label Oct 22, 2020
@rgcoe rgcoe requested a review from H0R5E October 22, 2020 01:10
@rgcoe rgcoe changed the title WaveBot: update preprint DOI and paper accepted update publications, references, etc. Oct 22, 2020
Copy link
Member

@H0R5E H0R5E left a comment

Choose a reason for hiding this comment

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

The content is fine, but I think we could present it more prominently. If you want to keep it this way, then please add a subsection header for the new content.

Happy to help implement my suggestions, if they sound good to you.

Comment on lines 4 to 11
This page list supporting references for WecOptTool.
Additionally, the following references specifically focusing on WecOptTool are available:

* **Optimal control of wave energy converters** :cite:`Bacelli2014` - While written years before the development of WecOptTool, Giorgio Bacelli's doctoral thesis provides much of the framework upon which WecOptTool is based.
* **WecOptTool September 2020 webinar** :cite:`coe2020webinar` - A one hour webinar was presented on September 29th 2020. This webinar provides interested users with a tutorial and background.
* **Initial conceptual demonstration of control co-design for WEC optimization** :cite:`Coe2020` - This paper presents the basic framework of WecOptTool along with a series of examples using the Sandia WaveBot.


Copy link
Member

Choose a reason for hiding this comment

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

I'm not adverse to this content, but I think it should be on its own page, perhaps 2 pages. I would have a page for the main reference numpy style and a page for the other information called "Resources". We could even embed the webinar, if you wanted?

At very least, this needs to be given a sub-section header so that it does not get lost in the navigation.

Copy link
Member Author

Choose a reason for hiding this comment

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

@H0R5E - if you really don't mind, I would indeed appreciate it if you can show me what you mean by this.

H0R5E and others added 5 commits November 3, 2020 12:35
This commit adds two new pages, one for storing a single citation
for WecOptTool in a textual and bibtex form, ala Numpy. The
second page includes additional resources for the user. Here,
the webinar slides are embedded in a viewer, and some additional
links for further reading are also included.

Numbering was also added to the file names to ensure that the
cited references get processed and numbered in the correct order.

The build requirements are also updated in the README and travisCI
config.
use version independent DOI
use article bitex category
include journal name
@rgcoe rgcoe requested a review from H0R5E November 3, 2020 19:53
@H0R5E H0R5E merged commit d5be840 into SNL-WaterPower:master Nov 4, 2020
@rgcoe rgcoe deleted the WaveBotPaperCitationUpdate branch November 4, 2020 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

link Sept. 29 webinar slides
2 participants