-
Notifications
You must be signed in to change notification settings - Fork 184
Update OSWEC tutorial dimensioned geometry image #621
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
Conversation
Update of OSWEC Tutorial dimensioned geometry png
nathanmtom
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The tutorial dimension image of the OSWEC has been improved and adds additional geometric description to help a new user recreate this example case.
H0R5E
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks fine if the width of the image is increased in the docs. I increased the width to 600pt and it looked like this:
Maintainers can't edit this pull request, so I created a patch that should be applied to the docs:
From 0031e8727c636753b095c177e3f9ecb12ad90880 Mon Sep 17 00:00:00 2001
From: Mathew Topper <damm_horse@yahoo.co.uk>
Date: Thu, 8 Jul 2021 12:41:32 +0100
Subject: [PATCH] Increase image width to 600pt
---
docs/user/tutorials.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/docs/user/tutorials.rst b/docs/user/tutorials.rst
index af896fe4..22a3297e 100644
--- a/docs/user/tutorials.rst
+++ b/docs/user/tutorials.rst
@@ -230,7 +230,7 @@ hinge to the base that restricts the flap in order to pitch about the hinge.
The full-scale dimensions and mass properties of the OSWEC are shown below.
.. figure:: /_static/images/OSWEC_Geom.png
- :width: 300pt
+ :width: 600pt
:align: center
+-------+---------------+
--
2.31.1.windows.1
If you save the above as a file called my.patch in the WEC-Sim root you can then run the following to apply the patch:
> cd \path\to\WEC-Sim
> git apply my.patch
It's just changing one line in the docs/user/tutorials.rst file (line 233), if you would prefer to do this manually.
Manually increasing width of geometry image to 600pt as per pull request change
|
I have resolved the merge conflict in #643. The change was made against an old version of the docs. Closing this and will merge the other. @rakkarolina, thanks for your contribution. |


Current image for geometry in OSWEC tutorial seems underdefined. Created a new image that includes important dimensions the previous image seemed to lack in hopes that this makes the geometry easier to recreate. Checked these dimensions and CAD file from issue #496 to make sure they matched.
Can make text larger or change orientations if needed.