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.

Sign up for GitHub

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

Jump to bottom

melt error message for bad measure.vars could be more informative/specific #6512

Open
tdhock opened this issue Sep 20, 2024 · 3 comments · May be fixed by #6767
Open

melt error message for bad measure.vars could be more informative/specific #6512

tdhock opened this issue Sep 20, 2024 · 3 comments · May be fixed by #6767
Assignees
@Divendra2006
Labels
beginner-task reshape dcast melt

Comments

@tdhock
Copy link
Member

tdhock commented Sep 20, 2024

Right now we have the following error message, which is ok:

> melt(data.table(x1=1,x2=2,y1=3,y2=4),measure.vars=c("x1","x2","z1"))
Error in melt.data.table(data.table(x1 = 1, x2 = 2, y1 = 3, y2 = 4), measure.vars = c("x1",  : 
  One or more values in 'measure.vars' is invalid.

it would be even better if we could tell the user specifically which entries are invalid, and how to fix the issue, for example "One or more values in 'measure.vars' is invalid; please fix by removing [z1]"
@KyleHaynes KyleHaynes mentioned this issue Dec 23, 2024
Closed
@venom1204
Copy link
Contributor

venom1204 commented Jan 1, 2025 •
edited
Loading

@tdhock, is this project still relevant? I’m looking forward to working on it.
can you guide me regarding this issue.

@tdhock
Copy link
Member Author

tdhock commented Jan 6, 2025

yes this is still relevant.
to find the code that should be fixed, please try using github search bar:
https://github.com/search?q=repo%3ARdatatable%2Fdata.table+%22One+or+more+values+in+%27measure.vars%27+is+invalid%22&type=code

@Divendra2006
Copy link

Hi @tdhock ,

I noticed that this issue has been open for over three weeks without much progress. I’d be happy to take it over and work on a solution.

Could you assign this issue to me? Thank you!

@Divendra2006 Divendra2006 linked a pull request Jan 26, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Divendra2006 Divendra2006

Labels
beginner-task reshape dcast melt
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Enhance error message for invalid measure.vars in melt Divendra2006/data.table
3 participants
@tdhock @Divendra2006 @venom1204