Groupby limitations

[1e-to]

No description provided.

[Rubtsowa]

Put this in a separate variable to avoid code duplication

[1e-to]

I do not think it is necessary here

[Rubtsowa]

@kozlov-alexey what do you think?

[1e-to]

If we need to add or change limitations, we will have to rewrite the text from the variables back to the lines. To the fact that so clearly

[densmirn]

I vote for inserting the constant text This function may reveal slower performance than Pandas* on user system. Users should exercise a tradeoff between staying in JIT-region with that function or going back to interpreter mode. where it requires through string formatting:

performance_limitation = `This function may reveal slower performance than Pandas* on user system. Users should exercise a tradeoff
    between staying in JIT-region with that function or going back to interpreter mode.`
...
'limitations_block':
    """
    Limitations
    -----------
    - {}.
    """.format(performance_limitation),
...

[densmirn]

or

performance_limitation = `This function may reveal slower performance than Pandas* on user system. Users should exercise a tradeoff
    between staying in JIT-region with that function or going back to interpreter mode.`
...
'limitations_block':
    f"""
    Limitations
    -----------
    - {performance_limitation}
    """,
...

[densmirn]

Thanks! One more question, could you insert example of generated documentation based on these changes to the PR as a screenshot?

[1e-to]

documentation based on these