Skip to content
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

Clarify status of reshape2 #4908

Merged
merged 3 commits into from
Mar 8, 2021
Merged

Clarify status of reshape2 #4908

merged 3 commits into from
Mar 8, 2021

Conversation

hadley
Copy link
Contributor

@hadley hadley commented Feb 17, 2021

Minor clarification — reshape2 is not deprecated, and we are committed to keeping it available on CRAN.

@codecov
Copy link

codecov bot commented Feb 17, 2021

Codecov Report

Merging #4908 (a809515) into master (97c96b2) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4908   +/-   ##
=======================================
  Coverage   99.46%   99.46%           
=======================================
  Files          73       73           
  Lines       14414    14414           
=======================================
  Hits        14337    14337           
  Misses         77       77           
Impacted Files Coverage Δ
R/fcast.R 100.00% <ø> (ø)
R/fmelt.R 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 97c96b2...a809515. Read the comment docs.

Copy link
Member

@MichaelChirico MichaelChirico left a comment

Choose a reason for hiding this comment

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

Thanks Hadley, LGTM

@eddelbuettel
Copy link
Contributor

There is a typo in the one line the PR changes. Apologies for posting an image but I think it drives the point home:

image

This presumably meant superseded (though my instinct would still be to spell it supercede).

@avimallu
Copy link
Contributor

For completeness, I believe that the error message for data.table::dcast in the event something other than a data.frame or data.table is passed will also have to be changed:

The dcast generic in data.table has been passed a factor and will attempt to redirect to the reshape2::dcast;
please note that reshape2 is deprecated, and this redirection is now deprecated as well. Please do this
redirection yourself like reshape2::dcast(as.factor(mtcars)). In the next version, this warning will
become an error.

I'm not very well versed with the data.table source code to suggest where to change this, but just pointing it out.

@hadley
Copy link
Contributor Author

hadley commented Feb 23, 2021

I have corrected the typo, and updated all other mentions of reshape2 being deprecated (by grepping for reshape2).

@mattdowle
Copy link
Member

Thanks @hadley, will merge. Although this PR does touch code files, it's edits to messages, so I won't add you to the contributors list which would make you a copyright holder. There are one or two similar cases in the past so I'll be consistent. Any type of logic change, even just to a single line of R or C code, would be really welcome and I'd add you to contributors list at that point.

Btw, our use of the word 'deprecated' derives from your use of the word 'retired' in Dec 2017 here: hadley/reshape@83ef5d2.
To this day, reshape's README starts :

reshape2 is retired: only changes necessary to keep it on CRAN will be made. We recommend using tidyr instead.

To me, that is the definition of deprecation, but I see your #3549 (comment) pointing to https://lifecycle.r-lib.org/articles/stages.html#retired and I see that you have since retired the word retired (footnote 1 in the doc). Would it make sense to update the word 'retired' in https://github.com/hadley/reshape/blob/master/README.md to 'superseded' then?

@mattdowle mattdowle added this to the 1.14.1 milestone Mar 8, 2021
@mattdowle mattdowle merged commit 788c585 into Rdatatable:master Mar 8, 2021
@hadley
Copy link
Contributor Author

hadley commented Mar 8, 2021

@mattdowle I have no expectation of becoming a contributor based on this tiny contribution (which as you say, arises from our own confusion around messaging). I've fixed the reshape2 readme — thanks for pointing that out!

@mattdowle
Copy link
Member

@hadley I didn't think you had that expectation, sorry I didn't write that very well: I've seen comments from others who wonder why I haven't added you, so I was more documenting for them. And I was taking the opportunity to welcome you and really encourage you to contribute code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants