Ticket #3669 (closed defect: fixed)

Opened 12 months ago

Last modified 11 months ago

image scaling broken in Labyrinth (mapa metales)

Reported by: walter Owned by: manuq
Priority: Unspecified by Maintainer Milestone: 0.96
Component: Labyrinth Version: Git as of bugdate
Severity: Unspecified Keywords: 12.1.0, patch, olpc-test-passed
Cc: Distribution/OS: Unspecified
Bug Status: Unconfirmed

Description

Line 126 of ImageThought.py is passing float values to simple_scale.

a simple patch of casting the floats to ints will fix the problem.

self.pic = self.orig_pic.scale_simple(int(pic_w), int(pic_h), scale)

Version 12 running on 11.3.1

Change History

Changed 12 months ago by garycmartin

  • owner changed from garycmartin to manuq
  • status changed from new to assigned

Changed 12 months ago by manuq

  • keywords 12.1.0, patch added
  • status changed from assigned to closed
  • resolution set to fixed
  • milestone changed from Unspecified by Release Team to 0.96

Seen in 12.1.0 too.

TestCase:

  • start Labyrinth, add an image node
  • try to resize the node, activity hangs

Thanks for the patch, tested, reviewed, and pushed as: 2723459fffc670bcf23e79dc1558974c9e973dbc .

Changed 12 months ago by manuq

  • keywords patch, olpc-test-pending added; patch removed

Changed 11 months ago by greenfeld

  • keywords olpc-test-passed added; olpc-test-pending removed

Image scaling works in 12.1.0 os16/Labyrith-12.1.

Note: See TracTickets for help on using tickets.