Loading time performance regression?

Bugs
42

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

43

cc @RaananW

44

i’ll release 5.5.6 in an hour or two.

4 Likes
45

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

3 Likes
46

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?

47

These are the times I am getting -

image
image1140×138 4.47 KB

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?

48

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

1 Like
49

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

2 Likes
50

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

3 Likes
51

@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
52

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

2 Likes
53

@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