Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Port STI calculation code #218

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

wbinek
Copy link
Contributor

@wbinek wbinek commented Jan 22, 2019

No description provided.

@nicolas-f
Copy link
Member

Hi,
Thanks for the pull request.
Can you set the header file on nc_curve file like the other files ?
Can you add some comments on this class file, in order to explain what nc_curve do and to add a reference to the document you used to implement the STI computation ?

@wbinek
Copy link
Contributor Author

wbinek commented Jan 22, 2019

Yes, no problem. I will add such comments.

Basically the STI computation is based on IEC:60268-16:2011 standard. The NC curves are used to introduce background noise to the STI calculation (see: https://www.engineeringtoolbox.com/nc-noise-criterion-d_725.html).

@J51n1989
Copy link

Hi!, i am a universitary user of i-simpa and i need to do some STI calculations for a proyect. I see you work in a code, but i am not good whit python, is there a place where to learn about how to do it? (i mean, is that simply as paste this code in the source code of isimpa?) i'll apreciate your help here

@nicolas-f
Copy link
Member

@J51n1989 Hi, you can find @wbinek version of I-Simpa here https://ci.appveyor.com/project/nicolas-f/i-simpa/builds/31870274/artifacts , please note that we still not have validated the results of this contribution.

@elyurn
Copy link

elyurn commented Jun 25, 2021

Hello!

Do you know when the calculation of the STI will be officially supported please?
Thank you very (very) much for all your work on this opensource project.

@martinritchie
Copy link

@nicolas-f ,

Sorry to bother you with some basic questions but I am not familiar with this code base: what is the status of this PR and is it ready for merging but awaiting the final approval?

@BurakAyva
Copy link

BurakAyva commented May 12, 2024

@J51n1989 Hi, you can find @wbinek version of I-Simpa here https://ci.appveyor.com/project/nicolas-f/i-simpa/builds/31870274/artifacts , please note that we still not have validated the results of this contribution.

Hi @nicolas-f , AppVeyor has one month retention policy and it removed this build. Would you please upload I-Simpa with STI again?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants