ggplot2 icon indicating copy to clipboard operation
ggplot2 copied to clipboard

Error: Invalid index: field name 'x_start' not found

Open YUCHENG-ZHAO opened this issue 1 year ago • 5 comments

options(repr.plot.width=10,repr.plot.height=10) p1 <- ggplot(nc) + geom_sf(aes(fill=meta[ as.character(nc$cell),'cell_type']))+ theme_void() p1 image I'm trying to find a picture of this. but Error: Invalid index: field name 'x_start' not found

YUCHENG-ZHAO avatar Aug 21 '24 03:08 YUCHENG-ZHAO

@Dudemullet Thanks for the PR. According to AWS documentation, Batch job names need to respect the following constraint first character must be alphanumeric, and up to 128 letters (uppercase and lowercase), numbers, hyphens, and underscores are allowed - Can you modify your implementation to take care of this constraint.

savingoyal avatar Jan 27 '20 19:01 savingoyal

@savingoyal Of course will look into it.

It makes sense to look into all of these considerations, not just the spacing issue.

Dudemullet avatar Jan 27 '20 19:01 Dudemullet

PR for #100

savingoyal avatar Jan 28 '20 18:01 savingoyal

@Dudemullet Any updates on this PR?

savingoyal avatar Mar 04 '20 17:03 savingoyal

Sorry for the delay, was a very busy last month and am currently on vacation (no laptop) . But will look back into this on March 16

On 5 Mar 2020, at 2:14 AM, Savin [email protected] wrote:

 @Dudemullet Any updates on this PR?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

Dudemullet avatar Mar 05 '20 12:03 Dudemullet

@savingoyal Left the job_name length validation in the batch class and all the username validation sent to the batch_decorator.

As for the error messages I intentionally made them verbose but helpful but they're multiline so don't know if I got a little carried away with that.

Dudemullet avatar Mar 23 '20 00:03 Dudemullet

hello all, this seem simple enough fix, could we proceed with planning to merge it. I'd be happy to help

bhat-ganesh avatar Aug 15 '24 18:08 bhat-ganesh