AW: DO NOT REPLY [Bug 46315] fox:needs-balancing extension

2009-01-14 Thread Georg Datterl
Hi Vincent,
 
  If that's the only problem left with my patch, I'll move on to the next 
  extension...
 Yes it's looking good otherwise. Although we might get back to you about 
 writing a bit
 of documentation once the fixing of the other bugs makes this extension more 
 widely usable ^^

docwhat? There's a reason it's called code. :-)


Mit freundlichen Grüßen
 
Georg Datterl
 
-- Kontakt --
 
Georg Datterl
 
Geneon media solutions gmbh
Gutenstetter Straße 8a
90449 Nürnberg
 
HRB Nürnberg: 17193
Geschäftsführer: Yong-Harry Steiert 

Tel.: 0911/36 78 88 - 26
Fax: 0911/36 78 88 - 20
 
www.geneon.de
 
Weitere Mitglieder der Willmy MediaGroup:
 
IRS Integrated Realization Services GmbH:www.irs-nbg.de 
Willmy PrintMedia GmbH:www.willmy.de
Willmy Consult  Content GmbH: www.willmycc.de 
-Ursprüngliche Nachricht-
Von: Vincent Hennebert [mailto:vhenneb...@gmail.com] 
Gesendet: Mittwoch, 14. Januar 2009 13:11
An: fop-dev@xmlgraphics.apache.org
Betreff: Re: DO NOT REPLY [Bug 46315] fox:needs-balancing extension

Hi Georg,

Georg Datterl wrote:
 Hi Vincent,
 
 It appears that bug #46322 is on the way of properly testing this 
 extension, with the 'one line of the spanning block at the bottom of the 
 page' problem.
 Either the test will need to be tweaked in order to work around the 
 bug, at the risk of not being extensive enough, or it will have to be 
 disabled until bug
 #46322 is fixed. More in the next episode...
 
 If I understand correctly, you worry that a fixing of #46322 will invalidate 
 this test case? How about changing the test case for #46315 after fixing 
 #46322?

Yes, that's a possibility. Why not, after all...


 If that's the only problem left with my patch, I'll move on to the next 
 extension...

Yes it's looking good otherwise. Although we might get back to you about 
writing a bit of documentation once the fixing of the other bugs makes this 
extension more widely usable ^^

Thanks,
Vincent


 Mit freundlichen Grüßen
  
 Georg Datterl
  
 -- Kontakt --
  
 Georg Datterl
  
 Geneon media solutions gmbh
 Gutenstetter Straße 8a
 90449 Nürnberg
  
 HRB Nürnberg: 17193
 Geschäftsführer: Yong-Harry Steiert
 
 Tel.: 0911/36 78 88 - 26
 Fax: 0911/36 78 88 - 20
  
 www.geneon.de
  
 Weitere Mitglieder der Willmy MediaGroup:
  
 IRS Integrated Realization Services GmbH:www.irs-nbg.de 
 Willmy PrintMedia GmbH:www.willmy.de
 Willmy Consult  Content GmbH: www.willmycc.de 
 -Ursprüngliche Nachricht-
 Von: bugzi...@apache.org [mailto:bugzi...@apache.org]
 Gesendet: Dienstag, 13. Januar 2009 12:34
 An: fop-dev@xmlgraphics.apache.org
 Betreff: DO NOT REPLY [Bug 46315] fox:needs-balancing extension
 
 https://issues.apache.org/bugzilla/show_bug.cgi?id=46315
 
 
 
 
 
 --- Comment #5 from Vincent Hennebert vhenneb...@gmail.com  
 2009-01-13 03:34:11 PST --- (In reply to comment #4)
 (In reply to comment #3)
 Created an attachment (id=23098)
  -- (https://issues.apache.org/bugzilla/attachment.cgi?id=23098) 
 [details] [details]
 Second try of patch

 Please review and comment.

 Hi Georg,

 Thanks for the patch. It's almost ready to be committed, but I'd like 
 to simplify the test case a bit (remove the inline elements and the 
 Arial font specification). Unfortunately that affects line height so 
 the whole test needs to be re-worked to give to same results as 
 before. I'll probably finish that tomorrow.

 Vincent
 
 
 Vincent


AW: DO NOT REPLY [Bug 46315] fox:needs-balancing extension

2009-01-13 Thread Georg Datterl
Hi Vincent,

 It appears that bug #46322 is on the way of properly testing this extension,
 with the 'one line of the spanning block at the bottom of the page' problem.
 Either the test will need to be tweaked in order to work around the bug, at
 the risk of not being extensive enough, or it will have to be disabled until 
 bug
 #46322 is fixed. More in the next episode...

If I understand correctly, you worry that a fixing of #46322 will invalidate 
this test case? How about changing the test case for #46315 after fixing #46322?

If that's the only problem left with my patch, I'll move on to the next 
extension...

Mit freundlichen Grüßen
 
Georg Datterl
 
-- Kontakt --
 
Georg Datterl
 
Geneon media solutions gmbh
Gutenstetter Straße 8a
90449 Nürnberg
 
HRB Nürnberg: 17193
Geschäftsführer: Yong-Harry Steiert 

Tel.: 0911/36 78 88 - 26
Fax: 0911/36 78 88 - 20
 
www.geneon.de
 
Weitere Mitglieder der Willmy MediaGroup:
 
IRS Integrated Realization Services GmbH:www.irs-nbg.de 
Willmy PrintMedia GmbH:www.willmy.de
Willmy Consult  Content GmbH: www.willmycc.de 
-Ursprüngliche Nachricht-
Von: bugzi...@apache.org [mailto:bugzi...@apache.org] 
Gesendet: Dienstag, 13. Januar 2009 12:34
An: fop-dev@xmlgraphics.apache.org
Betreff: DO NOT REPLY [Bug 46315] fox:needs-balancing extension

https://issues.apache.org/bugzilla/show_bug.cgi?id=46315





--- Comment #5 from Vincent Hennebert vhenneb...@gmail.com  2009-01-13 
03:34:11 PST --- (In reply to comment #4)
 (In reply to comment #3)
  Created an attachment (id=23098)
 -- (https://issues.apache.org/bugzilla/attachment.cgi?id=23098) [details] 
[details]
  Second try of patch
  
  Please review and comment.
  
 
 Hi Georg,
 
 Thanks for the patch. It's almost ready to be committed, but I'd like 
 to simplify the test case a bit (remove the inline elements and the 
 Arial font specification). Unfortunately that affects line height so 
 the whole test needs to be re-worked to give to same results as 
 before. I'll probably finish that tomorrow.
 
 Vincent


Vincent


--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: --- You are the assignee for 
the bug.


AW: DO NOT REPLY [Bug 46315] fox:needs-balancing extension

2009-01-09 Thread Georg Datterl
Hi Vincent,

given the fact that nobody else reads our thread, fox:disable-column-balancing 
it is and I think I made all suggested changes. Now I'm trying to build a 
testcase and then move on to the next needed extension. I'm running ant target 
juni-layout-standard but get lots of error messages and warnings from all kinds 
of tests (Junit and xmlunit are present):

Test: block_font-style.xml
  [ERROR] Invalid property value encountered in font-style=obscure: 
org.apache.fop.fo.expr.PropertyException: No conversion defined obscure; 
property:'font-style' (Keine Kontextinformationen verfügbar)
  [ERROR] Invalid property value encountered in font-style=obscure: 
org.apache.fop.fo.expr.PropertyException: No conversion defined obscure; 
property:'font-style' (Keine Kontextinformationen verfügbar)
  [ERROR] Invalid property value encountered in font-style=obscure: 
org.apache.fop.fo.expr.PropertyException: No conversion defined obscure; 
property:'font-style' (Keine Kontextinformationen verfügbar)
  [WARN ] Font serif,backslant,400 not found. Substituting with 
serif,normal,400.
  [WARN ] Font sans-serif,backslant,400 not found. Substituting with 
sans-serif,normal,400.
  [WARN ] Font monospace,backslant,400 not found. Substituting with 
monospace,normal,400.

...

Test: table_table-row_empty.xml
  [INFO ] table-layout=fixed and width=auto, but auto-layout not supported 
= assuming width=100%. (Keine Kontextinformationen verfügbar)
-  ---
- Standard Error -
09.01.2009 10:18:33 org.apache.fop.layoutengine.LayoutEngineTestSuite$1 runTest
SCHWERWIEGEND: Error on block-container_absolute-position_multi-column.xml
09.01.2009 10:18:36 org.apache.fop.layoutengine.LayoutEngineTestSuite$1 runTest
SCHWERWIEGEND: Error on block-container_reference-orientation_bug36391.xml
09.01.2009 10:18:47 org.apache.fop.util.ColorSpaceCache get
WARNUNG: Color profile 'nonexistent.icc' not found.
09.01.2009 10:18:47 org.apache.fop.util.ColorUtil parseAsFopRgbIcc
WARNUNG: Color profile 'nonexistent.icc' not found. Using rgb replacement 
values.
09.01.2009 10:18:48 org.apache.fop.layoutengine.LayoutEngineTestSuite$1 runTest
SCHWERWIEGEND: Error on footnote_changing-page-bpd.xml
09.01.2009 10:18:48 org.apache.fop.layoutengine.LayoutEngineTestSuite$1 runTest

...

-  ---
Testcase: 
block-container_absolute-position_multi-column.xml(org.apache.fop.layoutengine.LayoutEngineTestSuite$1):
  Caused an ERROR
Expected XPath expression to evaluate to '1', but got '2' (XPath: 
count(//pageViewport))
java.lang.RuntimeException: Expected XPath expression to evaluate to '1', but 
got '2' (XPath: count(//pageViewport))
at org.apache.fop.layoutengine.EvalCheck.check(EvalCheck.java:83)
at 
org.apache.fop.layoutengine.LayoutEngineTester.checkAll(LayoutEngineTester.java:235)
at 
org.apache.fop.layoutengine.LayoutEngineTester.runTest(LayoutEngineTester.java:182)
at 
org.apache.fop.layoutengine.LayoutEngineTestSuite$LayoutEngineTestCase.testMain(LayoutEngineTestSuite.java:219)
at 
org.apache.fop.layoutengine.LayoutEngineTestSuite$1.runTest(LayoutEngineTestSuite.java:193)


Testcase: 
block-container_reference-orientation_bug36391.xml(org.apache.fop.layoutengine.LayoutEngineTestSuite$1):
  Caused an ERROR
Expected XPath expression to evaluate to '[0.0 -1.0 1.0 0.0 0.0 31000.0]', but 
got '[-1.0 -0.0 0.0 -1.0 3.0 15400.0]' (XPath: 
//mainReference/span/flow[1]/block[4]/block[1]/block[1]/@ctm)
java.lang.RuntimeException: Expected XPath expression to evaluate to '[0.0 -1.0 
1.0 0.0 0.0 31000.0]', but got '[-1.0 -0.0 0.0 -1.0 3.0 15400.0]' (XPath: 
//mainReference/span/flow[1]/block[4]/block[1]/block[1]/@ctm)
at org.apache.fop.layoutengine.EvalCheck.check(EvalCheck.java:83)
at 
org.apache.fop.layoutengine.LayoutEngineTester.checkAll(LayoutEngineTester.java:235)
at 
org.apache.fop.layoutengine.LayoutEngineTester.runTest(LayoutEngineTester.java:182)
at 
org.apache.fop.layoutengine.LayoutEngineTestSuite$LayoutEngineTestCase.testMain(LayoutEngineTestSuite.java:219)
at 
org.apache.fop.layoutengine.LayoutEngineTestSuite$1.runTest(LayoutEngineTestSuite.java:193)

...

Test org.apache.fop.layoutengine.LayoutEngineTestSuite FAILED

Am I doing something wrong here or are these testcases not supposed to succeed? 
 

Regards,
 
Georg Datterl
 
-- Kontakt --
 
Georg Datterl
 
Geneon media solutions gmbh
Gutenstetter Straße 8a
90449 Nürnberg
 
HRB Nürnberg: 17193
Geschäftsführer: Yong-Harry Steiert 

Tel.: 0911/36 78 88 - 26
Fax: 0911/36 78 88 - 20
 
www.geneon.de
 
Weitere Mitglieder der Willmy MediaGroup:
 
IRS Integrated Realization Services GmbH:www.irs-nbg.de 
Willmy PrintMedia GmbH:www.willmy.de
Willmy Consult  Content GmbH: www.willmycc.de 


Re: AW: DO NOT REPLY [Bug 46315] fox:needs-balancing extension

2009-01-09 Thread Jeremias Maerki
Hi Georg

No, we haven't documented the area tree format explicitely. I guess we
assume that for people going that deep into FOP the source code is
enough documentation. At any rate, there's no lack of examples once
you've run the whole test suite once. ;-)

What we have is some documentation on how to create layout engine tests:
http://wiki.apache.org/xmlgraphics-fop/HowToCreateLayoutEngineTests


On 09.01.2009 12:41:53 Georg Datterl wrote:
 Hi Vincent, hi Jeremias,
 
 I made one single minor mistake and 49 tests complained... Anyway,
 having fixed that, I'm looking at the eval tag and I guess, the xpath
 expression is evaluated against the AreaTree. Is there a documentation
 for this Tree or can I only find how I can see it and then find my way
 around in it?
 
 Regards
  
 Georg Datterl
  
 -- Kontakt --
  
 Georg Datterl
  
 Geneon media solutions gmbh
 Gutenstetter Straße 8a
 90449 Nürnberg
  
 HRB Nürnberg: 17193
 Geschäftsführer: Yong-Harry Steiert 
 
 Tel.: 0911/36 78 88 - 26
 Fax: 0911/36 78 88 - 20
  
 www.geneon.de
  
 Weitere Mitglieder der Willmy MediaGroup:
  
 IRS Integrated Realization Services GmbH:www.irs-nbg.de 
 Willmy PrintMedia GmbH:www.willmy.de
 Willmy Consult  Content GmbH: www.willmycc.de 
 -Ursprüngliche Nachricht-
 Von: Vincent Hennebert [mailto:vhenneb...@gmail.com] 
 Gesendet: Freitag, 9. Januar 2009 12:04
 An: fop-dev@xmlgraphics.apache.org
 Betreff: Re: DO NOT REPLY [Bug 46315] fox:needs-balancing extension
 
 Hi Georg,
 
 You can usually forget about the warnings, as long as you don't have the 
 message Test org.apache.fop.layoutengine.LayoutEngineTestSuite FAILED
 at the end of the output.
 Try generating PDF out of the failing testcases and visually check what's 
 wrong; simply run the following command in the test/layoutengine
 directory:
  fop -xsl testcase2fo.xsl -xml standard-testcases/{failing-test}.xml -pdf 
 out.pdf
 
 That may help you figure out what's wrong. You can also run the debugger side 
 by side, on both a non-modified version of the Trunk and the version with 
 your changes, and see where it starts to diverge.
 
 If you're stuck you can always attach an updated patch on Bugzilla, so that 
 we can have a look at it.
 
 Good luck,
 Vincent
 
 
 Georg Datterl wrote:
  Hi Vincent,
  
  given the fact that nobody else reads our thread, 
  fox:disable-column-balancing it is and I think I made all suggested 
  changes. Now I'm trying to build a testcase and then move on to the next 
  needed extension. I'm running ant target juni-layout-standard but get lots 
  of error messages and warnings from all kinds of tests (Junit and xmlunit 
  are present):
  
  Test: block_font-style.xml
[ERROR] Invalid property value encountered in font-style=obscure: 
  org.apache.fop.fo.expr.PropertyException: No conversion defined obscure; 
  property:'font-style' (Keine Kontextinformationen verfügbar)
[ERROR] Invalid property value encountered in font-style=obscure: 
  org.apache.fop.fo.expr.PropertyException: No conversion defined obscure; 
  property:'font-style' (Keine Kontextinformationen verfügbar)
[ERROR] Invalid property value encountered in font-style=obscure: 
  org.apache.fop.fo.expr.PropertyException: No conversion defined obscure; 
  property:'font-style' (Keine Kontextinformationen verfügbar)
[WARN ] Font serif,backslant,400 not found. Substituting with 
  serif,normal,400.
[WARN ] Font sans-serif,backslant,400 not found. Substituting with 
  sans-serif,normal,400.
[WARN ] Font monospace,backslant,400 not found. Substituting with 
  monospace,normal,400.
  
  ...
  
  Test: table_table-row_empty.xml
[INFO ] table-layout=fixed and width=auto, but auto-layout not 
  supported = assuming width=100%. (Keine Kontextinformationen 
  verfügbar)
  -  ---
  - Standard Error -
  09.01.2009 10:18:33 
  org.apache.fop.layoutengine.LayoutEngineTestSuite$1 runTest
  SCHWERWIEGEND: Error on 
  block-container_absolute-position_multi-column.xml
  09.01.2009 10:18:36 
  org.apache.fop.layoutengine.LayoutEngineTestSuite$1 runTest
  SCHWERWIEGEND: Error on 
  block-container_reference-orientation_bug36391.xml
  09.01.2009 10:18:47 org.apache.fop.util.ColorSpaceCache get
  WARNUNG: Color profile 'nonexistent.icc' not found.
  09.01.2009 10:18:47 org.apache.fop.util.ColorUtil parseAsFopRgbIcc
  WARNUNG: Color profile 'nonexistent.icc' not found. Using rgb replacement 
  values.
  09.01.2009 10:18:48 
  org.apache.fop.layoutengine.LayoutEngineTestSuite$1 runTest
  SCHWERWIEGEND: Error on footnote_changing-page-bpd.xml
  09.01.2009 10:18:48 
  org.apache.fop.layoutengine.LayoutEngineTestSuite$1 runTest
  
  ...
  
  -  ---
  Testcase: 
  block-container_absolute-position_multi-column.xml(org.apache.fop.layoutengine.LayoutEngineTestSuite$1):
Caused an ERROR
  Expected XPath expression