kernel-memory icon indicating copy to clipboard operation
kernel-memory copied to clipboard

[Bug] There is no need to remove spaces here, they are needed if they remain after the decoder.

Open aropb opened this issue 9 months ago • 0 comments

There is no need to remove spaces here, they are needed if they remain after the decoder.

https://github.com/microsoft/kernel-memory/blob/1c424ed35738342ea2b3f8cae4091cb649071c49/service/Core/Handlers/TextExtractionHandler.cs#L220

It can be a PDF where the sentence goes to the next page.

But before that, you can do TrimEnd(): https://github.com/microsoft/kernel-memory/blob/1c424ed35738342ea2b3f8cae4091cb649071c49/service/Core/Handlers/TextExtractionHandler.cs#L228

aropb avatar Apr 02 '25 08:04 aropb