Modify

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#16448 closed defect (fixed)

Fix ImageryCompare red entries

Reported by: Don-vip Owned by: team
Priority: normal Milestone:
Component: External imagery source Version:
Keywords: compare eli jenkins Cc: stoecker, Klumbumbus, Stereo, Marc_marc

Description

The test testImageryEntries is failing for 38 consecutives builds and the number of red entries increases every day without any visible action.

If nobody's feeling responsible to fix these issues we should disable the test? Otherwise it's only useless noise for those who try to keep other things stable.

Attachments (0)

Change History (11)

comment:1 Changed 5 years ago by stoecker

ELI recently started to add many of JOSMs entries to their list as well causing again a large number of red entries. Klumbumbus and I are working on this, but given that now is holiday time I expect it will take some more weeks to resolve all these changes. It's not like it is easy. Sometimes our data is better, sometimes ELI stuff. Each entry must be examined.

And while these 38 builds are consecutive, the errors itself not. It have always been others.

comment:2 Changed 5 years ago by Don-vip

Ok thanks for the update and taking care this (ungrateful) stuff.

comment:3 Changed 5 years ago by Don-vip

Summary: Fix ImageryCompare red entries or disable integration testFix ImageryCompare red entries

comment:4 Changed 5 years ago by Klumbumbus

Not sure why this is still reported. I added the max zoom.

* Maxzoom differs (21 != null): [AU] ACTmapi Imagery 2017 - https://data.actmapi.act.gov.au/arcgis/services/actmapi/imagery2017mga/ImageServer/WMSServer?FORMAT=image/jpeg&VERSION=1.1.1&SERVICE=WMS&REQUEST=GetMap&LAYERS=0&STYLES=&SRS={proj}&WIDTH={width}&HEIGHT={height}&BBOX={bbox}

comment:5 Changed 5 years ago by stoecker

In 13997/josm:

see #16448 - dont skip zoomlevels for WMS mirrors totally

comment:6 Changed 5 years ago by stoecker

Down to 5.

comment:7 Changed 5 years ago by Don-vip

Great! :)

comment:8 Changed 5 years ago by Stereo

Cool! For the five remaining, I've added the information to ELI after checking manually, so those can be taken over.

Will some JOSM developers be at SotM in Milan? I think it'd be good to have an imagery meeting.

comment:9 Changed 5 years ago by Klumbumbus

Component: Core imageryExternal imagery source
Milestone: 18.07
Resolution: fixed
Status: newclosed

comment:10 in reply to:  8 Changed 5 years ago by Klumbumbus

Replying to Stereo:

For the five remaining, I've added the information to ELI after checking manually, so those can be taken over.

One was wrong in ELI: https://josm.openstreetmap.de/wiki/ImageryCompareIgnores?action=diff&version=484

Will some JOSM developers be at SotM in Milan?

Not me.

comment:11 Changed 5 years ago by Don-vip

Me neither, sorry.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.