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

Basic thermal example review #323

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

Conversation

JackB-Ansys
Copy link
Collaborator

Update to Thermal basic example to fix some issues and improve comments.

@JackB-Ansys JackB-Ansys linked an issue Jun 13, 2024 that may be closed by this pull request
Comment on lines +9 to +10
# This script will change model parameter values (geometry, winding pattern and materials),
# run magnetic calculations, extract and plot results.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# This script will change model parameter values (geometry, winding pattern and materials),
# run magnetic calculations, extract and plot results.
# This script changes model parameter values (geometry, winding pattern, and materials),
# runs magnetic calculations, and extracts and plots results.

Use present tense and Oxford comma.

#
# Perform required imports
# ~~~~~~~~~~~~~~~~~~~~~~~~
# Import ``pymotorcad`` to access Motor-CAD. Import ``pyplot`` from ``matplotlib`` to display
# graphics. Import ``os``, ``shutil`` and ``tempfile`` to open and save a temporary MOT
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# graphics. Import ``os``, ``shutil`` and ``tempfile`` to open and save a temporary MOT
# graphics. Import ``os``, ``shutil``, and ``tempfile`` to open and save a temporary MOT

#
# Perform required imports
# ~~~~~~~~~~~~~~~~~~~~~~~~
# Import ``pymotorcad`` to access Motor-CAD. Import ``pyplot`` from ``matplotlib`` to display
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Import ``pymotorcad`` to access Motor-CAD. Import ``pyplot`` from ``matplotlib`` to display
# Import ``pymotorcad`` to access Motor-CAD. Import ``pyplot`` from Matplotlib to display

Use the product name for Matplotlib to avoid a code entity of a code entity?

Comment on lines +52 to +53
# after executing the script, use the ``MotorCAD(keep_instance_open=True)`` option when opening the
# new instance.
Copy link
Member

@PipKat PipKat Jun 25, 2024

Choose a reason for hiding this comment

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

Suggested change
# after executing the script, use the ``MotorCAD(keep_instance_open=True)`` option when opening the
# new instance.
# after executing the script, use ``MotorCAD(keep_instance_open=True`` when opening the
# new instance.

Comment on lines +63 to +66
# This will also disable crucial popups, for example prompts to save data or overwrite data, or
# dialogs used to reconcile differences in material data between the database and MOT file. In
# each case the default action will be taken. This setting will persist until Motor-CAD is
# closed.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# This will also disable crucial popups, for example prompts to save data or overwrite data, or
# dialogs used to reconcile differences in material data between the database and MOT file. In
# each case the default action will be taken. This setting will persist until Motor-CAD is
# closed.
# This also disables crucial popups, such as example prompts to save data or overwrite data and
# dialogs used to reconcile differences in material data between the database and MOT file. In
# each case, the default action is taken. This setting persists until Motor-CAD is
# closed.

# Set Motor-CAD to EMag
# ~~~~~~~~~~~~~~~~~~~~~
# Show the E-Magnetic context in Motor-CAD and ensure that the BPM
# Motor Type is set.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Motor Type is set.
# motor type is set.

Comment on lines +80 to +81
# Setting up the analysis consists of setting the geometry, winding and material parameters, setting
# the calculation options and operating point, and saving the file.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Setting up the analysis consists of setting the geometry, winding and material parameters, setting
# the calculation options and operating point, and saving the file.
# Setting up the analysis consists of setting the geometry, winding and material parameters,
# calculation options and operating point, and then saving the file.

#
# Geometry setup
# ~~~~~~~~~~~~~~
# Display the **Scripting** tab and set the ``Slot_Number``, ``Tooth_Width`` and
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Display the **Scripting** tab and set the ``Slot_Number``, ``Tooth_Width`` and
# Display the **Scripting** tab and set the ``Slot_Number``, ``Tooth_Width``, and

Comment on lines +89 to +91
# To set parameter values via automation with the GUI visible, a different tab must be displayed.
# The **Scripting** tab is chosen because no parameters on this tab will be changed in this
# script. It is best practice to display the **Scripting** tab when changing input parameters by
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# To set parameter values via automation with the GUI visible, a different tab must be displayed.
# The **Scripting** tab is chosen because no parameters on this tab will be changed in this
# script. It is best practice to display the **Scripting** tab when changing input parameters by
# To set parameter values by automation with the GUI visible, a different tab must be displayed.
# The **Scripting** tab is chosen because no parameters on this tab are changed in this
# script. It is best practice to display the **Scripting** tab when changing input parameters by

Use present tense. Style guide says to avoid using "via" as a preposition.

Comment on lines +116 to +118
# * Define parameters for coil 3 using the ``set_winding_coil()`` method. Set positions to a, b, c
# etc. when using Upper/Lower paths (when Left/Right paths are selected, L or R should be used
# instead). Define the third coil to have 60 turns in the second phase.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# * Define parameters for coil 3 using the ``set_winding_coil()`` method. Set positions to a, b, c
# etc. when using Upper/Lower paths (when Left/Right paths are selected, L or R should be used
# instead). Define the third coil to have 60 turns in the second phase.
# * Define parameters for coil 3 using the ``set_winding_coil()`` method. Set positions to a, b, c, and so on
# when using Upper/Lower paths. (When Left/Right paths are selected, L or R should be used
# instead.) Define the third coil to have 60 turns in the second phase.

# Setting up the analysis consists of setting the geometry, winding and material parameters, setting
# the calculation options and operating point, and saving the file.
#
# Geometry setup
Copy link
Member

Choose a reason for hiding this comment

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

All the section headings should be set up so that they start with a simple verb and then the action. For example, the heading here would be: Set up geometry. The heading for the next section would be: Set up winding. Please change the headings to follow this format.

mcad.set_component_material("Rotor Lam (Back Iron)", "M250-35A")
# Materials setup
# ~~~~~~~~~~~~~~~
# Set the stator and rotor lamination materials. Component names are displayed in the
Copy link
Member

@PipKat PipKat Jun 25, 2024

Choose a reason for hiding this comment

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

Suggested change
# Set the stator and rotor lamination materials. Component names are displayed in the
# Set the stator and rotor lamination materials. Component names are shown in the

An intransitive verb is necessary (because subject is not doing the action).

Comment on lines +135 to +136
# *Materials* tab in Motor-CAD. Materials and their properties are defined in the Motor-CAD
# *Material database*.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# *Materials* tab in Motor-CAD. Materials and their properties are defined in the Motor-CAD
# *Material database*.
# **Materials** tab in Motor-CAD. Materials and their properties are defined in the Motor-CAD
# **Material database**.

Double asterisks for bold (UI elements).

Comment on lines +141 to +142
# Calculation options
# ~~~~~~~~~~~~~~~~~~~
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Calculation options
# ~~~~~~~~~~~~~~~~~~~
# Set calculation options
# ~~~~~~~~~~~~~~~~~~~~~~~

Comment on lines +164 to +165
# Operating point
# ~~~~~~~~~~~~~~~
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Operating point
# ~~~~~~~~~~~~~~~
# Set operating point
# ~~~~~~~~~~~~~~~~~~~

mcad.set_variable("PhaseAdvance", 45)
# Operating point
# ~~~~~~~~~~~~~~~
# Set the operating point of the machine, defined by the speed, current and phase advance. Ensure
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Set the operating point of the machine, defined by the speed, current and phase advance. Ensure
# Set the operating point of the machine, defined by the speed, current, and phase advance. Ensure

mcad.save_to_file(filename)
# Save the file
# ~~~~~~~~~~~~~
# Save the file to the temporary folder and display a message that the initialisation in complete.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Save the file to the temporary folder and display a message that the initialisation in complete.
# Save the file to the temporary folder and display a message that the initialization in complete.

American spelling.

# ~~~~~~~~~~~~~
# Save the file to the temporary folder and display a message that the initialisation in complete.
mc.save_to_file(working_folder + "/" + mot_name + ".mot")
print("Initialisation completed.")
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
print("Initialisation completed.")
print("Initialization completed.")

Comment on lines +191 to +192
# Export results to a CSV file. Results will be saved as *Export_EMag_Results.csv* in the temporary
# working directory. A try and except block are used to raise an error in the case of the export
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Export results to a CSV file. Results will be saved as *Export_EMag_Results.csv* in the temporary
# working directory. A try and except block are used to raise an error in the case of the export
# Export results to a CSV file. Results are saved in a ``Export_EMag_Results.csv`` file in the temporary
# working directory. A try-and-except block is used to raise an error in the case of the export


# %%
# Get and analyze results
# Get and analyse results
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Get and analyse results
# Get and analyze results

American spelling.

Comment on lines +211 to +212
# Retrieve the *Torque (VW)* graph data. The data type for this series is *E-Magnetics*, so use the
# ``get_magnetic_graph()`` method.
Copy link
Member

Choose a reason for hiding this comment

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

The single asterisk results in italics font. Is this what you want? It seems that both of these are code entities and should be double backticks.

Comment on lines +215 to +216
# When retrieving graph data points, use the *Graph Viewer* under *Help* in the Motor-CAD
# interface to find the series names and data types.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# When retrieving graph data points, use the *Graph Viewer* under *Help* in the Motor-CAD
# interface to find the series names and data types.
# When retrieving graph data points, use the **Graph Viewer** under **Help** in the Motor-CAD
# interface to find the series names and data types.

Double asterisk for bold to indicate a UI element.

Comment on lines +220 to +221
# Retrieve the airgap flux density graph data. This data can be found in the *Graph Viewer* in
# Motor-CAD. The data type for this series is *FEA Path*, so use the ``get_fea_graph()`` method.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Retrieve the airgap flux density graph data. This data can be found in the *Graph Viewer* in
# Motor-CAD. The data type for this series is *FEA Path*, so use the ``get_fea_graph()`` method.
# Retrieve the airgap flux density graph data. This data can be found in the **Graph Viewer** in
# Motor-CAD. The data type for this series is **FEA Path**, so use the ``get_fea_graph()`` method.

Not sure if FEA Path in bold for UI element or double backticks for code entity?

Comment on lines +225 to +230
# Only the most recently displayed harmonic graphs are available in Motor-CAD via the
# *Graph Viewer*. To retrieve harmonic data by using Motor-CAD automation, first display the
# relevant *Graphs -> Harmonics* tab. Use the ``initialise_tab_names()`` method to initialise the
# available tabs in the Motor-CAD user interface. When the tabs have been initialised, the
# ``display_screen()`` method can be used until either the Motor-CAD context is changed, or the
# instance is closed.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Only the most recently displayed harmonic graphs are available in Motor-CAD via the
# *Graph Viewer*. To retrieve harmonic data by using Motor-CAD automation, first display the
# relevant *Graphs -> Harmonics* tab. Use the ``initialise_tab_names()`` method to initialise the
# available tabs in the Motor-CAD user interface. When the tabs have been initialised, the
# ``display_screen()`` method can be used until either the Motor-CAD context is changed, or the
# instance is closed.
# Only the most recently displayed harmonic graphs are available in Motor-CAD using the
# **Graph Viewer**. To retrieve harmonic data by using Motor-CAD automation, first display the
# relevant **Graphs -> Harmonics** tab. Use the ``initialise_tab_names()`` method to initialize the
# available tabs in the Motor-CAD user interface. When the tabs have been initialized, you can use the
# ``display_screen()`` method until either the Motor-CAD context is changed or the
# instance is closed.

# ``display_screen()`` method can be used until either the Motor-CAD context is changed, or the
# instance is closed.
#
# Open the *Graphs -> Harmonics -> Torque* tab to make the torque harmonic data available.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Open the *Graphs -> Harmonics -> Torque* tab to make the torque harmonic data available.
# Open the **Graphs -> Harmonics -> Torque** tab to make the torque harmonic data available.

except pymotorcad.MotorCADError:
print("Results failed to export.")

# Retrieve the torque harmonic graph data. The data type for this series is *E-Magnetics*, so use
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Retrieve the torque harmonic graph data. The data type for this series is *E-Magnetics*, so use
# Retrieve the torque harmonic graph data. The data type for this series is **E-Magnetics**, so use

Double asterisk for bold for UI element, but double backticks if this is a code entity.


print("Simulation completed.")


# %%
# Plot results
# ------------
# Plot results from the simulation.
# Use``pyplot`` from ``matplotlib`` (imported as ``plt``) to plot graphs of the simulation results.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Use``pyplot`` from ``matplotlib`` (imported as ``plt``) to plot graphs of the simulation results.
# Use``pyplot`` from Matplotlib (imported as ``plt``) to plot graphs of the simulation results.


print("Simulation completed.")


# %%
# Plot results
# ------------
# Plot results from the simulation.
# Use``pyplot`` from ``matplotlib`` (imported as ``plt``) to plot graphs of the simulation results.
# Plot the Airgap Flux Density and the Torque (VW) results as subplots in the same figure.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Plot the Airgap Flux Density and the Torque (VW) results as subplots in the same figure.
# Plot the airgap flux density and the torque (VW) results as subplots in the same figure.

# --------------
# Exit Motor-CAD.
mcad.quit()
# Plot the Torque harmonic results as a bar chart.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Plot the Torque harmonic results as a bar chart.
# Plot the torque harmonic results as a bar chart.

Comment on lines +268 to +271
# If you want to continue working with this instance of Motor-CAD, use the
# ``MotorCAD(keep_instance_open=True)`` option when you launch Motor-CAD. If Motor-CAD is kept open,
# it is useful to restore the popup messages that were disabled earlier, using the
# ``set_variable()`` method ().
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# If you want to continue working with this instance of Motor-CAD, use the
# ``MotorCAD(keep_instance_open=True)`` option when you launch Motor-CAD. If Motor-CAD is kept open,
# it is useful to restore the popup messages that were disabled earlier, using the
# ``set_variable()`` method ().
# If you want to continue working with this instance of Motor-CAD, use
# ``MotorCAD(keep_instance_open=True`` when you launch Motor-CAD. If Motor-CAD is kept open,
# it is useful to restore the popup messages that were disabled earlier using the
# ``set_variable()`` method.

# Perform required imports
# ~~~~~~~~~~~~~~~~~~~~~~~~
# Import the required packages.
# Import ``pymotorcad`` to access Motor-CAD. Import ``pyplot`` from ``matplotlib`` to display
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Import ``pymotorcad`` to access Motor-CAD. Import ``pyplot`` from ``matplotlib`` to display
# Import ``pymotorcad`` to access Motor-CAD. Import ``pyplot`` from Matplotlib to display

# Perform required imports
# ~~~~~~~~~~~~~~~~~~~~~~~~
# Import the required packages.
# Import ``pymotorcad`` to access Motor-CAD. Import ``pyplot`` from ``matplotlib`` to display
# graphics. Import ``os``, ``shutil`` and ``tempfile`` to open and save a temporary MOT
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# graphics. Import ``os``, ``shutil`` and ``tempfile`` to open and save a temporary MOT
# graphics. Import ``os``, ``shutil``, and ``tempfile`` to open and save a temporary MOT

# %%
# Specify working directory
# ~~~~~~~~~~~~~~~~~~~~~~~~~
# Save the file to a temporary folder
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Save the file to a temporary folder
# Save the file to a temporary folder.

print("Starting initialization.")
mcad = pymotorcad.MotorCAD()
# Connect to Motor-CAD and open a new Motor-CAD instance. To keep a new Motor-CAD instance open
# after executing the script, use the ``MotorCAD(keep_instance_open=True)`` option when opening the
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# after executing the script, use the ``MotorCAD(keep_instance_open=True)`` option when opening the
# after executing the script, use ``MotorCAD(keep_instance_open=True)`` when opening the

Comment on lines +58 to +60
# This will also disable crucial popups, for example prompts to save data or overwrite data, or
# dialogs used to reconcile differences in material data between the database and MOT file. In
# each case the default action will be taken. This setting will persist until Motor-CAD is
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# This will also disable crucial popups, for example prompts to save data or overwrite data, or
# dialogs used to reconcile differences in material data between the database and MOT file. In
# each case the default action will be taken. This setting will persist until Motor-CAD is
# This also disables crucial popups, such as prompts to save data or overwrite data and
# dialogs used to reconcile differences in material data between the database and MOT file. In
# each case, the default action is taken. This setting persists until Motor-CAD is

Comment on lines +73 to +76
# Set Motor-CAD to Thermal
# ~~~~~~~~~~~~~~~~~~~~~~~~
# Show the Thermal context in Motor-CAD and ensure that the BPM
# Motor Type is set.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Set Motor-CAD to Thermal
# ~~~~~~~~~~~~~~~~~~~~~~~~
# Show the Thermal context in Motor-CAD and ensure that the BPM
# Motor Type is set.
# Set Motor-CAD to thermal
# ~~~~~~~~~~~~~~~~~~~~~~~~
# Show the thermal context in Motor-CAD and ensure that the BPM
# motor type is set.

Comment on lines +91 to +93
# .. note::
# To set parameter values via automation with the GUI visible, a different tab must be displayed.
# The **Scripting** tab is chosen because no parameters on this tab will be changed in this
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# .. note::
# To set parameter values via automation with the GUI visible, a different tab must be displayed.
# The **Scripting** tab is chosen because no parameters on this tab will be changed in this
# .. note::
# To set parameter values by automation with the GUI visible, a different tab must be shown.
# The **Scripting** tab is chosen because no parameters on this tab are changed in this

Comment on lines +100 to +101
# Thermal input data setup
# ~~~~~~~~~~~~~~~~~~~~~~~~
Copy link
Member

@PipKat PipKat Jun 25, 2024

Choose a reason for hiding this comment

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

Suggested change
# Thermal input data setup
# ~~~~~~~~~~~~~~~~~~~~~~~~
# Set up thermal input data
# ~~~~~~~~~~~~~~~~~~~~~~~~~

Again, start all headings starting with the simple verb form.

mc.set_fluid("HousingWJFluid", "Dynalene HF-LO")

# %%
# Disable the option for *Active Cooling Only*, so that the HWJ channel also cools the front and
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Disable the option for *Active Cooling Only*, so that the HWJ channel also cools the front and
# Disable the option for **Active Cooling Only** so that the HWJ channel also cools the front and

mc.set_variable("Water_Jacket_Active_Cooling_Only", 0)

# %%
# Enable the *Input h* option to allow user input of the heat transfer coefficients for HWJ cooling
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Enable the *Input h* option to allow user input of the heat transfer coefficients for HWJ cooling
# Enable the **Input h** option to allow user input of the heat transfer coefficients for HWJ cooling

mc.set_array_variable("HousingWJ_CalcInputH_A", 0, 1)

# %%
# Get fluid properties (thermal conductivity, density and dynamic viscosity) and the fluid
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Get fluid properties (thermal conductivity, density and dynamic viscosity) and the fluid
# Get fluid properties (thermal conductivity, density, and dynamic viscosity) and the fluid

Comment on lines +133 to +135
# Outputs will be updated when a steady state thermal calculation runs. Alternatively, the
# relevant tab (for example, *Input Data -> Housing Water Jacket -> Heat Transfer*) can be
# displayed - however this may not be suitable when running Motor-CAD without the GUI.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Outputs will be updated when a steady state thermal calculation runs. Alternatively, the
# relevant tab (for example, *Input Data -> Housing Water Jacket -> Heat Transfer*) can be
# displayed - however this may not be suitable when running Motor-CAD without the GUI.
# Outputs are updated when a steady state thermal calculation runs. Alternatively, the
# relevant tab (for example, **Input Data -> Housing Water Jacket -> Heat Transfer**) can be
# shown. However, this might not be suitable when running Motor-CAD without the GUI.

Slice number to get x and y coordinate values from.
time_step_number : int
Time step number to get x and y coordinate values from.
Returns
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Returns
Returns

Blank line is necessary for rendering correctly in the doc.

Comment on lines +188 to +190
Value of x coordinates from graph
y_values : list
Value of y coordinates from graph
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Value of x coordinates from graph
y_values : list
Value of y coordinates from graph
Value of x coordinates from graph.
y_values : list
Value of y coordinates from graph.

Slice number to get x and y coordinate values from.
time_step_number : int
Time step number to get x and y coordinate values from.
Returns
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Returns
Returns

Comment on lines +209 to +211
Value of x coordinates from graph
y_values : list
Value of y coordinates from graph
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Value of x coordinates from graph
y_values : list
Value of y coordinates from graph
Value of x coordinates from graph.
y_values : list
Value of y coordinates from graph.

Copy link

codecov bot commented Jul 29, 2024

Codecov Report

Attention: Patch coverage is 57.14286% with 3 lines in your changes missing coverage. Please review.

Project coverage is 84.37%. Comparing base (bae7688) to head (da07dd0).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #323      +/-   ##
==========================================
- Coverage   84.47%   84.37%   -0.11%     
==========================================
  Files          20       20              
  Lines        2003     2009       +6     
==========================================
+ Hits         1692     1695       +3     
- Misses        311      314       +3     

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

Successfully merging this pull request may close these issues.

Review the PyMotorCAD Basic Examples
3 participants