Skip to content

chore: rm some tk stuff#2002

Draft
maelle wants to merge 1 commit intomainfrom
switch
Draft

chore: rm some tk stuff#2002
maelle wants to merge 1 commit intomainfrom
switch

Conversation

@maelle
Copy link
Contributor

@maelle maelle commented Jul 22, 2025

not ready

SEXP l6 = PROTECT(Rf_ScalarString(PROTECT(Rf_mkChar(message))));
SEXP l7 = PROTECT(Rf_lang3(l4, l5, l6));
PROTECT(ec=Rf_eval(l7, rho));
SEXP l4 = PROTECT(Rf_ScalarReal(percent));
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@schochastics if you agree we can remove the tk (not tkplot) stuff, I will need help with this C file. I am getting many errors from the current changes.

@schochastics
Copy link
Contributor

schochastics commented Jul 22, 2025

In order to contribute, I need to first understand what this tk stuff was doing in the first place 😬

@maelle
Copy link
Contributor Author

maelle commented Jul 22, 2025

@schochastics I do wonder whether it was doing anything at all, depending on the verbose option, but let's ask in a future hackathon.

@maelle
Copy link
Contributor Author

maelle commented Mar 10, 2026

@krlmlr could you please have a look when you get a chance?

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.

2 participants