Skip to content

Conversation

wholmgren
Copy link
Member

@mikofski here's a partial fix for #242. I tried using a fancy table but it didn't make a difference. So, I moved the simple table out of the parameter description and into the notes section, which does improve the rendering:

http://wholmgren-pvlib-python-new.readthedocs.io/en/fixsnldoc/modules.html#pvlib.pvsystem.snlinverter

Maybe something about the RTD theme? In any case, I think this is an improvement and will merge soon unless there are objections or better ideas.

Here's the fancy table format that I tried.

    +--------+------------------------------------------------------------+
    | Column |  Description                                               |
    +========+============================================================+
    | Pac0   | AC-power output from inverter based on input power         |
    |        | and voltage (W)                                            |
    +--------+------------------------------------------------------------+
    | Pdc0   | DC-power input to inverter, typically assumed to be equal  |
    |        | to the PV array maximum power (W)                          |
    +--------+------------------------------------------------------------+
    | Vdc0   | DC-voltage level at which the AC-power rating is achieved  |
    |        | at the reference operating condition (V)                   |
    +--------+------------------------------------------------------------+
    | Ps0    | DC-power required to start the inversion process, or       |
    |        | self-consumption by inverter, strongly influences inverter |
    |        | efficiency at low power levels (W)                         |
    +--------+------------------------------------------------------------+
    | C0     | Parameter defining the curvature (parabolic) of the        |
    |        | relationship between ac-power and dc-power at the          |
    |        | reference operating condition, default value of zero gives |
    |        | a linear relationship (1/W)                                |
    +--------+------------------------------------------------------------+
    | C1     | Empirical coefficient allowing Pdco to vary linearly       |
    |        | with dc-voltage input, default value is zero (1/V)         |
    +--------+------------------------------------------------------------+
    | C2     | Empirical coefficient allowing Pso to vary linearly with   |
    |        | dc-voltage input, default value is zero (1/V)              |
    +--------+------------------------------------------------------------+
    | C3     | Empirical coefficient allowing Co to vary linearly with    |
    |        | dc-voltage input, default value is zero (1/V)              |
    +--------+------------------------------------------------------------+
    | Pnt    | AC-power consumed by inverter at night (night tare) to     |
    |        | maintain circuitry required to sense PV array voltage (W)  |
    +--------+------------------------------------------------------------+

@wholmgren wholmgren added this to the 0.4.1 milestone Sep 13, 2016
@mikofski
Copy link
Member

This is great! Sorry I haven't been contributing.

@dacoex
Copy link
Contributor

dacoex commented Sep 14, 2016

@wholmgren seems like a RTD CSS issue. But haven checked with the native Sphinx theme.

@wholmgren
Copy link
Member Author

Tests are failing because of unrelated forecast module problems, so going ahead with the merge.

@wholmgren wholmgren merged commit f67ecff into pvlib:master Sep 14, 2016
@wholmgren wholmgren deleted the fixsnldoc branch October 22, 2016 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants