teasit / magic-formula-tyre-tool

MATLAB GUI for Magic Formula Tyre Modeling
GNU General Public License v3.0
41 stars 7 forks source link

[BUG] "Downsampling Error for Magic Formula Tyre Tool V1.4" "Expected X to be nonempty" #4

Closed Zaga272 closed 10 months ago

Zaga272 commented 1 year ago

Hey Tom, I was trying to use the fitter of your Magic Formula Tyre Tool but unfortunately there is a problem with the downsampling. Once i click "Start Fitter" Matlab gives out an error message which says: "Error using downsample, Expected X to be nonempty" I tried it with different FSAE TTC Raw data but the error keeps showing up. So in order to reproduce the error you just have to import Cornering and DriveBrake data of your tyre and start the fitter. I also tried to change the downsampling from 1 to 5 and so on but this did not fix the problem. I just started learning about Tyre modelling and your tool helps me a lot to understand the tyre characteristics and how to work with the FSAE TTC Data. So any help to fix this problem would be much appreciated.

Here is also a screenshot of the error message: image

Btw I am using Matlab 2022b

teasit commented 1 year ago

@Zaga272 The downsampling factor is not the issue. It's that your data did not contain any relevant samples to fit the model to.

The tool currently does not give a better error description, sorry for that. Which dataset are you using?

Zaga272 commented 1 year ago

Hey, I am using the Raw FSAE TTC Data from Round 9.

Gianluca2 commented 1 year ago

@Zaga272 You need to run RUN data to make the program work

BhandeGiridhar commented 11 months ago

Hi! I've been facing a similar issue. When I try to fit for Mz0, Mx, etc modes, it states me an error that "Parameter 'UNLOADED_RADIUS' is required to fit 'Mz0'". I am not sure what to do, the data cant be incomplete now, can it? Ive tried different measurement data and still face the same issue.

teasit commented 11 months ago

Hi,

The issues are not related. In your case, the unloaded tire radius is empty in your model, which is the default. You then need to set it according to the tire spec (e.g. 0.2 metres). Search for the name in the table.

Hope that helps Tom

GiridharBhande0z @.***> schrieb am Mo., 13. Nov. 2023, 19:24:

Hi! I've been facing a similar issue. When I try to fit for Mz0, Mx, etc modes, it states me an error that "Parameter 'UNLOADED_RADIUS' is required to fit 'Mz0'". I am not sure what to do, the data cant be incomplete now, can it? Ive tried different measurement data and still face the same issue.

— Reply to this email directly, view it on GitHub https://github.com/teasit/magic-formula-tyre-tool/issues/4#issuecomment-1808754691, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGSJEVTIABH4CZV4567HUDLYEJQXXAVCNFSM6AAAAAA5M5UZZCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBYG42TINRZGE . You are receiving this because you were assigned.Message ID: @.***>

lmpsn commented 11 months ago

Hi , I'm facing the same issue as @Zaga272 but I have actually used RUN data from the Round 9 of the TTC. What would be the problem ?

Zaga272 commented 11 months ago

Hi, well, I've also used the RUN Data from Round 9 from the beginning and the problem still exists.

cabadiez commented 11 months ago

I have the same problem as zaga

teasit commented 11 months ago

@Zaga272 OK, I can reproduce the issue with some Round 9 data. Will look into it.

teasit commented 11 months ago

@Zaga272 Please try using this updated import script to see if it fixes your issue. FSAETTC_SI_ISO_Mat.zip

The file to be replaced should be located here: <path-to-tool>/src/magic-formula-tyre-library/src/+tydex/+parsers

Or navigate to the file by clicking this button:

image
Zaga272 commented 10 months ago

Hey @teasit, now it works. Thank you very much.

teasit commented 10 months ago

Alright. Glad to hear it.

Will update the script with the next release!

teasit commented 10 months ago

fixed in v1.5.1