Re: [PR] Bug: Auto sizing not working properly due to rounding error when calculating defaultCharWidth [poi]

2024-07-07 Thread via GitHub
pjfanning commented on PR #541: URL: https://github.com/apache/poi/pull/541#issuecomment-2212515980 I created #657 as an outline -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comme

Re: [PR] Bug: Auto sizing not working properly due to rounding error when calculating defaultCharWidth [poi]

2024-07-07 Thread via GitHub
pjfanning commented on PR #541: URL: https://github.com/apache/poi/pull/541#issuecomment-2212498631 https://svn.apache.org/viewvc?view=rev&rev=1913676 was in POI 5.2.5. So far the only suggested solution is to have a configurable extra width that is added to all cell widths. There are too m

Re: [PR] Bug: Auto sizing not working properly due to rounding error when calculating defaultCharWidth [poi]

2024-07-07 Thread via GitHub
ThoSap commented on PR #541: URL: https://github.com/apache/poi/pull/541#issuecomment-2212497631 With Apache POI 5.2.3 the sizing worked properly with Excel, I'm using that version currently as newer ones are broken. I don't know about LibreOffice thoe. -- This is an automated message

Re: [PR] Bug: Auto sizing not working properly due to rounding error when calculating defaultCharWidth [poi]

2024-07-07 Thread via GitHub
centic9 commented on PR #541: URL: https://github.com/apache/poi/pull/541#issuecomment-2212496557 Changes from this PR were applied as part of Apache POI 5.3.0, see [r1913676](https://svn.apache.org/viewvc?view=rev&rev=1913676), however some manual tests showed that the actual result is dep

Re: [PR] Bug: Auto sizing not working properly due to rounding error when calculating defaultCharWidth [poi]

2024-07-04 Thread via GitHub
ThoSap commented on PR #541: URL: https://github.com/apache/poi/pull/541#issuecomment-2208897536 Was this fixed in Apache POI 5.3.0 which was released two days ago? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

Re: [PR] Bug: Auto sizing not working properly due to rounding error when calculating defaultCharWidth [poi]

2023-12-10 Thread via GitHub
niklasmelzner commented on PR #541: URL: https://github.com/apache/poi/pull/541#issuecomment-1848954307 I guess you're referring to [this bug](https://bz.apache.org/bugzilla/show_bug.cgi?id=68094)? I'll have a look at it. -- This is an automated message from the Apache Git Service. To

Re: [PR] Bug: Auto sizing not working properly due to rounding error when calculating defaultCharWidth [poi]

2023-12-10 Thread via GitHub
pjfanning commented on PR #541: URL: https://github.com/apache/poi/pull/541#issuecomment-1848945124 @NiklasMelznerExxcellent Some users are complaining that this change is still creating cells that are not wide enough. Would you have any interest in having another look? I'm wondering

Re: [PR] Bug: Auto sizing not working properly due to rounding error when calculating defaultCharWidth [poi]

2023-11-09 Thread via GitHub
NiklasMelznerExxcellent commented on PR #541: URL: https://github.com/apache/poi/pull/541#issuecomment-1803536627 Alright, thank you for addressing the issue so quickly! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] Bug: Auto sizing not working properly due to rounding error when calculating defaultCharWidth [poi]

2023-11-09 Thread via GitHub
pjfanning commented on PR #541: URL: https://github.com/apache/poi/pull/541#issuecomment-1803524931 It will be a few weeks before POI 5.2.5 is released. We are concentrating on releasing XMLBeans 5.2.0 first and once that is released, we should be in a position to start a POI release vote.

Re: [PR] Bug: Auto sizing not working properly due to rounding error when calculating defaultCharWidth [poi]

2023-11-09 Thread via GitHub
NiklasMelznerExxcellent commented on PR #541: URL: https://github.com/apache/poi/pull/541#issuecomment-1803515832 I can't see any additional failing tests (except the ones that fail due to my locale but aren't related to the issue). My example seems to work too: https://github.com/apac

Re: [PR] Bug: Auto sizing not working properly due to rounding error when calculating defaultCharWidth [poi]

2023-11-08 Thread via GitHub
pjfanning commented on PR #541: URL: https://github.com/apache/poi/pull/541#issuecomment-1801946113 I added some changes in https://github.com/apache/poi/commit/8819952b2f7210d4201a1b1d75d420abc67d8c2c I didn't want to break backward compatibility so I've added new methods and depre

[PR] Bug: Auto sizing not working properly due to rounding error when calculating defaultCharWidth [poi]

2023-11-02 Thread via GitHub
NiklasMelznerExxcellent opened a new pull request, #541: URL: https://github.com/apache/poi/pull/541 The XSSFSheet#autoSizeColumn method doesn't work properly for some specific cell values and font metrics. The following code snippet is an example for this behavior: ```java Stri