Loading time performance regression?

Is there going to be a patch release for this today?

cc @RaananW

i’ll release 5.5.6 in an hour or two.

4 Likes

It was a hell of a ride :slight_smile: Thanks for your patience :smiley:

3 Likes

I’m sorry to have to say this, but the problem does not seem to be fixed in 5.5.6

Here it works

Here i simply removed the snapshot param which goes to version 5.5.6

and again I have load times of ~8sec

Was the fix properly applied?

These are the times I am getting -

But the PR you referenced (i.e. the PR the snapshot you have used ) is not yet merged, so it’s not applied in 5.5.6:

Still test disable meshes because gltf loaders them them on later by deltakosh · Pull Request #12487 · BabylonJS/Babylon.js (github.com)

I’ll merge this soon. Is that the fix for the issue?

Yes this will fix it.
I thought this was going out in version 5.5.6 :sweat_smile:

1 Like

I’ll see when we repeat a new patch. Probably will do that later today

My bad I thought I merged it but I did not (I’m getting old apparently)

1 Like

@RaananW I really need to get a patch release out with the fix,
do you think you are going to be able to get one out today?

1 Like

Sure, 5.6.0 will be released in an hour (and a bit :blush:)

1 Like

@Deltakosh @RaananW sorry Im wayyyy late to the party. I was out all last week and wasnt able to follow up. We will test on 5.6.0, the PG example works so I think we are good. Thank you all.

1 Like