Skip to content

Update visual cases - #3638

Merged
hadley merged 1 commit into
tidyverse:masterfrom
yutannihilation:fix/update-visual-tests
Nov 24, 2019
Merged

Update visual cases#3638
hadley merged 1 commit into
tidyverse:masterfrom
yutannihilation:fix/update-visual-tests

Conversation

@yutannihilation

Copy link
Copy Markdown
Member

It seems these are diffs about coloubar legends, though they doesn't seem different to my eyes.

The latest build passes (cb0eaa0) on 6 days ago, so I suspect this might be related to some update of scales.

@yutannihilation yutannihilation left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Sorry, I don't know how to share the visual diffs. So, I copied data:image/png;base64,... part from the diff into a HTML file and viewed it on browser. They looks the same to my eyes.

<html>
  <head>
    <style>
      img {
	  width: 100px;
      }
    </style>
  </head>
  <body>
    
<img src='data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABIAAABXCAYAAAAJWxWIAAAABmJLR0QA/wD/AP+gvaeTAAABjUlEQVRYhc2Yy3EdMRADG9SUHZsjcVzKyZH4YFtLcqDDcwaDgzaArh6Any3qnHcz/Oz+Udr/phwsEwGVobQ/xiCA4pkbYVEkjCSKQEa4QyCcygiKE8jIhEaTQqPRFPvvnCNCIJtyACSJ8vNnDLpAsX+PQQiqEyBDeX9PCFF9vs2NJMq75hxDdQBkoPq8zY1erc1BWJQDILGo3msutKD6zEFqZUDYVD9z0JIoB4xOm+pHY5AUau2V0Z4bLd4yoF5kWjOmzk2AoPYNnUe3A61JISOJOgGjtqlz5+eRbep0ACSoHTASZDKSO5PRWitj5A7VbzsTNsuhsFvUjuz+kJFXakG2qOPALXKhdo85rxPyBEA4ZCSFjNbLaB62ZeqM3yGA/moZdao1BLUDGanJhL1IjbZSo53Upk2FTUM9KaPDnZNY1FnPnGNRlwAIqKMAaHXKSNQNGBkyRq+blv11jBpTxwEjieqA0f+f0bmRENV9xiCWqJvI6Jg6dw7Cop4OLEhBfXTgzd+itvfPMYn96xMzVA/gKz5U8AAAAABJRU5ErkJggg=='/>
<img src='data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABIAAABXCAYAAAAJWxWIAAAABmJLR0QA/wD/AP+gvaeTAAABjUlEQVRYhc2YwZUVMQwEq716EBuRENfmRCQcgB3b6j18MlAfdgKoV2rJ8jzrnHcz/Oz+Udr/phwsEwGVobQ/xiCA4pkbYVEkjCSKQEa4QyCcygiKE8jIhEqTQqXRFPvvnCNCIJtyACSJ8vNnDLpAsX+PQQiqEyBDeX9PCFF9vs2NJMq75hxDdQBkoPq8zY1eXZuDsCgHQGJRvddcaEH1mYPUyoCwqX7moCVRDhidNtWPxiAp1LVXRntutHjLgHqR6ZoxdW4CBLVvaB/dDnRNChlJ1AkYtU2dO99HtqnTAZCgdsBIkMlI7kxGa62MkTvUftuZsFkOhd2iduT0h4y8UgPZoo4Dt8iF2j3mvDbkCYBwyEgKGa2X0Txsy9QZv0MA/dUyakPtRGmkSjOZsBchI1YoI53UoVWqa6mBbEFt7pzEos565hyLugRAQB0FQKtTRqJuwMiQMXrdtOyvY9SYOg4YSVQHjP7/jM6NhKjuMwaxRN1ERsfUuXMQFvV0YCAF9dGBN3+L2t4/xyT2r08udw/fYGKV5AAAAABJRU5ErkJggg=='/>

<br />
<img src='data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABIAAABXCAYAAAAJWxWIAAAABmJLR0QA/wD/AP+gvaeTAAABE0lEQVRYhc2YwRHDMAgEF4Q9frmvVJKGUw/k4RK8D7kAzbHHCeSYM4a3X8ynuOv1OcRQXOf7g4DiEhTNUJxWaceSFB2WohIUhaVohmIJihiKSuGcoCijtKBYhiLLNQ32YLmGxCiSYhkRsVzL7WB7rlmMsBjFUOROjAaKNDq7d4tIW6VlWPanpOix35priv0ao7AYjcQIjZEV2gzpGsFyjZYYdUqlaYy0zm4r/YlVmtWQbcHe9KpVsmYdpDEa7RppqzTLNawBibVEPOkXFKU1IMPaalvrbG0ZtWCPC9uw39pGQGIUlmvjbiPaFNGeWTuN7LAuf22KeLC1XxrWcz2srGnvNSzX2gptWqHF6qN+7P8Kgn5/vNJD8shvDpsAAAAASUVORK5CYII='/>
<img src='data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABIAAABXCAYAAAAJWxWIAAAABmJLR0QA/wD/AP+gvaeTAAABGklEQVRYhc3Yy3XDUAgE0AFhJ6v0lUrScOp5eOESdBdWAZxhPsBT7bM2d7/a38nP3K6T2ky+n/cLJZl8AURZVKg2k8cFAG0mD4VoAKJSiHYzufp+oSSTC7SWUqo1QvTmSLTGCgX5KEHyV2dyiYgUQtSK7FUcbSGOUpm0UI3OI4GIcbRRHB0aEUD2qUwaFOpCiKqV/GqLvOUXrTGOVGjdOmIcKdVacZTNpMU6Uuk/LUP7Uc4+Kv39cYY8ypCtDNmK7LCDXanG1tGqiOSo1qxq4oZU95EbbKvSrxbkYc5mx2ijMbL0CSEM2eoJEXr6iUKrBhubR6VC6+5stY6sap9FNvulwUZt1Dxi6VeqHRXaVqGN8tF5y/8HAP2/AM7iPvzuGKsSAAAAAElFTkSuQmCC'/>

<br/>

<img src='data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABIAAABXCAYAAAAJWxWIAAAABmJLR0QA/wD/AP+gvaeTAAABE0lEQVRYhc2YwRHDMAgEF4Q9frmvVJKGUw/k4RK8D7kAzbHHCeSYM4a3X8ynuOv1OcRQXOf7g4DiEhTNUJxWaceSFB2WohIUhaVohmIJihiKSuGcoCijtKBYhiLLNQ32YLmGxCiSYhkRsVzL7WB7rlmMsBjFUOROjAaKNDq7d4tIW6VlWPanpOix35priv0ao7AYjcQIjZEV2gzpGsFyjZYYdUqlaYy0zm4r/YlVmtWQbcHe9KpVsmYdpDEa7RppqzTLNawBibVEPOkXFKU1IMPaalvrbG0ZtWCPC9uw39pGQGIUlmvjbiPaFNGeWTuN7LAuf22KeLC1XxrWcz2srGnvNSzX2gptWqHF6qN+7P8Kgn5/vNJD8shvDpsAAAAASUVORK5CYII='/>
<img src='data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABIAAABXCAYAAAAJWxWIAAAABmJLR0QA/wD/AP+gvaeTAAABGklEQVRYhc3Yy3XDUAgE0AFhJ6v0lUrScOp5eOESdBdWAZxhPsBT7bM2d7/a38nP3K6T2ky+n/cLJZl8AURZVKg2k8cFAG0mD4VoAKJSiHYzufp+oSSTC7SWUqo1QvTmSLTGCgX5KEHyV2dyiYgUQtSK7FUcbSGOUpm0UI3OI4GIcbRRHB0aEUD2qUwaFOpCiKqV/GqLvOUXrTGOVGjdOmIcKdVacZTNpMU6Uuk/LUP7Uc4+Kv39cYY8ypCtDNmK7LCDXanG1tGqiOSo1qxq4oZU95EbbKvSrxbkYc5mx2ijMbL0CSEM2eoJEXr6iUKrBhubR6VC6+5stY6sap9FNvulwUZt1Dxi6VeqHRXaVqGN8tF5y/8HAP2/AM7iPvzuGKsSAAAAAElFTkSuQmCC'/>

  </body>
</html>

<g clip-path='url(#cpMC4wMHw3MjAuMDB8NTc2LjAwfDAuMDA=)'><text x='349.15' y='568.24' style='font-size: 11.00px; font-family: Liberation Sans;' textLength='5.50px' lengthAdjust='spacingAndGlyphs'>x</text></g>
<g clip-path='url(#cpMC4wMHw3MjAuMDB8NTc2LjAwfDAuMDA=)'><text transform='translate(13.04,286.70) rotate(-90)' style='font-size: 11.00px; font-family: Liberation Sans;' textLength='5.50px' lengthAdjust='spacingAndGlyphs'>x</text></g>
<rect x='679.54' y='233.09' width='34.98' height='101.72' style='stroke-width: 1.07; stroke: none; fill: #FFFFFF;' clip-path='url(#cpMC4wMHw3MjAuMDB8NTc2LjAwfDAuMDA=)' />
<g clip-path='url(#cpMC4wMHw3MjAuMDB8NTc2LjAwfDAuMDA=)'><image width='17.28' height='86.40' x='679.54' y='248.41' xlink:href='data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABIAAABXCAYAAAAJWxWIAAAABmJLR0QA/wD/AP+gvaeTAAABE0lEQVRYhc2YwRHDMAgEF4Q9frmvVJKGUw/k4RK8D7kAzbHHCeSYM4a3X8ynuOv1OcRQXOf7g4DiEhTNUJxWaceSFB2WohIUhaVohmIJihiKSuGcoCijtKBYhiLLNQ32YLmGxCiSYhkRsVzL7WB7rlmMsBjFUOROjAaKNDq7d4tIW6VlWPanpOix35priv0ao7AYjcQIjZEV2gzpGsFyjZYYdUqlaYy0zm4r/YlVmtWQbcHe9KpVsmYdpDEa7RppqzTLNawBibVEPOkXFKU1IMPaalvrbG0ZtWCPC9uw39pGQGIUlmvjbiPaFNGeWTuN7LAuf22KeLC1XxrWcz2srGnvNSzX2gptWqHF6qN+7P8Kgn5/vNJD8shvDpsAAAAASUVORK5CYII='/></g>

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

left: before, right: after

Screenshot from 2019-11-24 17-29-47

<g clip-path='url(#cpMC4wMHw3MjAuMDB8NTc2LjAwfDAuMDA=)'><text x='349.15' y='568.24' style='font-size: 11.00px; font-family: Liberation Sans;' textLength='5.50px' lengthAdjust='spacingAndGlyphs'>x</text></g>
<g clip-path='url(#cpMC4wMHw3MjAuMDB8NTc2LjAwfDAuMDA=)'><text transform='translate(13.04,286.70) rotate(-90)' style='font-size: 11.00px; font-family: Liberation Sans;' textLength='5.50px' lengthAdjust='spacingAndGlyphs'>x</text></g>
<rect x='679.54' y='233.09' width='34.98' height='101.72' style='stroke-width: 1.07; stroke: none; fill: #FFFFFF;' clip-path='url(#cpMC4wMHw3MjAuMDB8NTc2LjAwfDAuMDA=)' />
<g clip-path='url(#cpMC4wMHw3MjAuMDB8NTc2LjAwfDAuMDA=)'><image width='17.28' height='86.40' x='679.54' y='248.41' xlink:href='data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABIAAABXCAYAAAAJWxWIAAAABmJLR0QA/wD/AP+gvaeTAAABE0lEQVRYhc2YwRHDMAgEF4Q9frmvVJKGUw/k4RK8D7kAzbHHCeSYM4a3X8ynuOv1OcRQXOf7g4DiEhTNUJxWaceSFB2WohIUhaVohmIJihiKSuGcoCijtKBYhiLLNQ32YLmGxCiSYhkRsVzL7WB7rlmMsBjFUOROjAaKNDq7d4tIW6VlWPanpOix35priv0ao7AYjcQIjZEV2gzpGsFyjZYYdUqlaYy0zm4r/YlVmtWQbcHe9KpVsmYdpDEa7RppqzTLNawBibVEPOkXFKU1IMPaalvrbG0ZtWCPC9uw39pGQGIUlmvjbiPaFNGeWTuN7LAuf22KeLC1XxrWcz2srGnvNSzX2gptWqHF6qN+7P8Kgn5/vNJD8shvDpsAAAAASUVORK5CYII='/></g>

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

left: before, right: after

Screenshot from 2019-11-24 17-29-28

<g clip-path='url(#cpMC4wMHw3MjAuMDB8NTc2LjAwfDAuMDA=)'><text x='352.81' y='568.24' style='font-size: 11.00px; font-family: Liberation Sans;' textLength='5.50px' lengthAdjust='spacingAndGlyphs'>x</text></g>
<g clip-path='url(#cpMC4wMHw3MjAuMDB8NTc2LjAwfDAuMDA=)'><text transform='translate(13.04,286.70) rotate(-90)' style='font-size: 11.00px; font-family: Liberation Sans;' textLength='5.50px' lengthAdjust='spacingAndGlyphs'>y</text></g>
<rect x='686.87' y='233.09' width='27.65' height='101.72' style='stroke-width: 1.07; stroke: none; fill: #FFFFFF;' clip-path='url(#cpMC4wMHw3MjAuMDB8NTc2LjAwfDAuMDA=)' />
<g clip-path='url(#cpMC4wMHw3MjAuMDB8NTc2LjAwfDAuMDA=)'><image width='17.28' height='86.40' x='686.87' y='248.41' xlink:href='data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABIAAABXCAYAAAAJWxWIAAAABmJLR0QA/wD/AP+gvaeTAAABjUlEQVRYhc2Yy3EdMRADG9SUHZsjcVzKyZH4YFtLcqDDcwaDgzaArh6Any3qnHcz/Oz+Udr/phwsEwGVobQ/xiCA4pkbYVEkjCSKQEa4QyCcygiKE8jIhEaTQqPRFPvvnCNCIJtyACSJ8vNnDLpAsX+PQQiqEyBDeX9PCFF9vs2NJMq75hxDdQBkoPq8zY1erc1BWJQDILGo3msutKD6zEFqZUDYVD9z0JIoB4xOm+pHY5AUau2V0Z4bLd4yoF5kWjOmzk2AoPYNnUe3A61JISOJOgGjtqlz5+eRbep0ACSoHTASZDKSO5PRWitj5A7VbzsTNsuhsFvUjuz+kJFXakG2qOPALXKhdo85rxPyBEA4ZCSFjNbLaB62ZeqM3yGA/moZdao1BLUDGanJhL1IjbZSo53Upk2FTUM9KaPDnZNY1FnPnGNRlwAIqKMAaHXKSNQNGBkyRq+blv11jBpTxwEjieqA0f+f0bmRENV9xiCWqJvI6Jg6dw7Cop4OLEhBfXTgzd+itvfPMYn96xMzVA/gKz5U8AAAAABJRU5ErkJggg=='/></g>

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

left:before, right: after

Screenshot from 2019-11-24 17-29-10

@hadley

hadley commented Nov 24, 2019

Copy link
Copy Markdown
Member

You can click this button — Screenshot 2019-11-25 at 7 10 48 AM — it often doesn't work the first time for me, but if you toggle back to source diff and then again to rich diff it does work.

@hadley

hadley commented Nov 24, 2019

Copy link
Copy Markdown
Member

This is probably due to some subtly different calculation by farver

@hadley
hadley merged commit 9ec8fee into tidyverse:master Nov 24, 2019
@yutannihilation

Copy link
Copy Markdown
Member Author

but if you toggle back to source diff and then again to rich diff it does work.

Oh, thanks for this tips! (But, this time this doesn't work for me...) I'll try next time :)

@yutannihilation
yutannihilation deleted the fix/update-visual-tests branch November 25, 2019 00:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants