-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add Piecewise FastSRM #50
Conversation
@thomasbazeille would you mind taking a look at that ? Let me know if you can't. |
Sorry for the (huge) delay, I'll check it ASAP |
🙋 I'm happy to pick this up ! @hugorichard I assume you don't have time to fix the merge conflicts here ; I'll plan to open a superseding PR with these changes unless I hear otherwise from you. Thanks ! |
Hello Elizabeth,
Indeed, I won't have time to merge conflicts.
Fill free to do as you wish :)
Thanks a lot for taking care of this.
Le mar. 5 sept. 2023 à 22:17, Elizabeth DuPre ***@***.***> a
écrit :
… 🙋 I'm happy to pick this up ! @hugorichard
<https://github.com/hugorichard> I assume you don't have time to fix the
merge conflicts here ; I'll plan to open a superseding PR with these
changes unless I hear otherwise from you. Thanks !
—
Reply to this email directly, view it on GitHub
<#50 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACK4DFTHSGAOZUEIIBF3EQTXY6CERANCNFSM5I7KGIYA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
cc75363
to
0e1ce05
Compare
Closing now that #70 has been merged. Thank you, @hugorichard ! |
Thx to all ! |
This PR is a copy of the piecewise SRM package of Thomas.
It uses the last version of FastSRM. This is relevant for #49