gantt task time format

[cygnyx]

I'm trying this library in a day/week timeframe instead of hour/minute.

I modified render_gantt_matplotlib to add another parameter:
def render_gantt_plotly(
solution: SchedulingSolution,
fig_size: Optional[Tuple[int, int]] = None,
show_plot: Optional[bool] = True,
show_indicators: Optional[bool] = True,
render_mode: Optional[str] = "Resource",
sort: Optional[str] = None,
fig_filename: Optional[str] = None,
html_filename: Optional[str] = None,
time_format: Optional[str] = "%H:%M"
)

and later changed:
times_str.append(t.strftime("%H:%M"))
to:
times_str.append(t.strftime(time_format))

Can this be included?

[tpaviot]

t.strftime is used in the render_gantt_matplotlib function, not plotly. Are you sure about the suggested change?

[cygnyx]

Yes, I was using it for a Gantt chart. I thought it was a minor change that would increase its suitability. But I have moved on to another approach - if this is inconsistent with your project’s goals, please close without making this change.

…

On Jun 18, 2024, at 9:58 PM, Thomas Paviot ***@***.***> wrote: t.strftime is used in the render_gantt_matplotlib function, not plotly. Are you sure about the suggested change? — Reply to this email directly, view it on GitHub <#146 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABTUB2W3TSXC7IHENMWXMZLZIEFXHAVCNFSM6AAAAABH4OT62KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZXG42DKMBTHA>. You are receiving this because you authored the thread.