You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'm wondering if in the off-season we expand/rewrite some of the capabilities of f1dataR to match the new capabilities of FastF1, and also be more capable of implementing changes to reflect the changes there (#198).
For example, right now, for us to add and fully use pick_drivers instead of using pick_driver takes a rewrite of our get_session_laps() (#78). Had pick_driver() instead been a standalone function (as part of a class or otherwise) it'd be much easier to just accept a vector of character strings or numbers instead of a single value.
I also wonder if having an interface that more closely matches the workflow of FastF1 will make it easier for people to duplicate or expand on analysis they otherwise see happening in Python?
Thoughts?
The text was updated successfully, but these errors were encountered:
I'm al for it. To be honest the original implementations didn't have much though put into it and I didn't know what the package would turn into. Big fan of rewriting old code
Repository owner
locked and limited conversation to collaborators
Nov 13, 2023
I'm wondering if in the off-season we expand/rewrite some of the capabilities of f1dataR to match the new capabilities of FastF1, and also be more capable of implementing changes to reflect the changes there (#198).
For example, right now, for us to add and fully use
pick_drivers
instead of usingpick_driver
takes a rewrite of ourget_session_laps()
(#78). Hadpick_driver()
instead been a standalone function (as part of a class or otherwise) it'd be much easier to just accept a vector of character strings or numbers instead of a single value.I also wonder if having an interface that more closely matches the workflow of FastF1 will make it easier for people to duplicate or expand on analysis they otherwise see happening in Python?
Thoughts?
The text was updated successfully, but these errors were encountered: