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

Add image annotations to the full report. #11

Closed
dlangenk opened this issue Aug 19, 2016 · 5 comments
Closed

Add image annotations to the full report. #11

dlangenk opened this issue Aug 19, 2016 · 5 comments
Assignees
Labels

Comments

@dlangenk
Copy link
Member

Whole image annotations are missing in the full report as requested by lenaick.

@mzur
Copy link
Member

mzur commented Aug 19, 2016

Wouldn't a separate report for image labels make more sense? I don't see how we can cram all this information in a single tabular report in a sensible way.

@dlangenk
Copy link
Member Author

This would be fine as well.

@mzur
Copy link
Member

mzur commented Aug 19, 2016

I already see the next feature request coming, since they are probably labelling the images with the substrate_type label tree. They probably don't want to see this for each image:

- bedrock
- cobbles
- gravel

But this:

- substrate_type > dominant > bedrock
- substrate_type > secondary > cobbles
- substrate_type > tertiary > gravel

@mzur mzur self-assigned this Aug 22, 2016
mzur added a commit that referenced this issue Aug 22, 2016
@mzur
Copy link
Member

mzur commented Aug 22, 2016

The image label report is now implemented as XLSX with one sheet per transect and the format:

transect_name
image.id image.filename label_1.name, label_2.name, label_3.name, ...

If this feature is urgently needed we can deploy it now. However, I'd prefer to wait for the Laravel 5.3 release (probably tomorrow) when we can finally merge #4 into master and then deploy both features by friday.

@mzur mzur mentioned this issue Aug 25, 2016
@mzur mzur closed this as completed Aug 25, 2016
@mzur
Copy link
Member

mzur commented Aug 26, 2016

@dlangenk This feature is now live.

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

No branches or pull requests

2 participants