Transformer slows down Hugo considerably about hugo

[noahcampbell]

How are you measuring the results so I can use as a baseline?

[spf13]

Rendering my site takes about 3.3 seconds. Then I comment out the transformer logic and set final to content and render the site again and it takes about 400 ms.

Steve Francia
spf13.com
@spf13

On Oct 8, 2013, at 2:28 AM, Noah Campbell [email protected] wrote:

How are you measuring the results so I can use as a baseline?

—
Reply to this email directly or view it on GitHub.

[noahcampbell]

Yikes…that is a dramatic slow down. I'll definitely want to make that faster. The transformer "chain" is currently re-parsing the html tree every time. I will make this is a single operation and then apply all transformers.

What is "set final to content" in your last reply?

[spf13]

in site.go I did this...

func (s *Site) WritePublic(path string, content io.Reader) (err error) {
s.initTarget()

if s.Config.Verbose {
fmt.Println(path)
}

//if s.Transformer == nil {
//s.Transformer = &transform.Transformer{BaseURL: s.Config.BaseUrl}
//}
//final := new(bytes.Buffer)
//s.Transformer.Apply(content, final)
final := content
return s.Target.Publish(path, final)
}

of course you could just return content instead of final, but I like to
make things easier to undo when debugging.

Steve Francia
http://stevefrancia.com
http://spf13.com
http://twitter.com/spf13

On Tue, Oct 8, 2013 at 12:22 PM, Noah Campbell [email protected]:

Yikes…that is a dramatic slow down. I'll definitely want to make that
faster. The transformer "chain" is currently re-parsing the html tree every
time. I will make this is a single operation and then apply all
transformers.

What is "set final to content" in your last reply?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/91#issuecomment-25905069
.

[spf13]

newer code is faster, but still significantly slower than before. factor of ~4x now.

[noahcampbell]

Do you have a baseline in mind?

[noahcampbell]

This has a 50% speed up. Reopen if it needs to be faster

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.