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

Make it easier to add kart specific engine sound effects #1234

Open
hiker opened this issue Mar 7, 2014 · 7 comments
Open

Make it easier to add kart specific engine sound effects #1234

hiker opened this issue Mar 7, 2014 · 7 comments

Comments

@hiker
Copy link
Contributor

hiker commented Mar 7, 2014

At the moment engine sound effect filename are hardcoded in kart properties (lines 559). Fix this so that karts can specific other engine sounds (without any more code changes).

@konstin
Copy link
Contributor

konstin commented Mar 8, 2014

I think I'll work on that in the next week. My Idea is to add a new node to karts.xml maybe called "enginge-sound" or something like that where a sound file can be specified. If a kart doesn't have the new node, the old way will be used.

@hiker
Copy link
Contributor Author

hiker commented Mar 10, 2014

Check out what's already there ;) This is a pretty trivial ticket (depending on how you approach this). Note that changes to the kart.xml file should be avoided, since it would mean we have to re-export all karts.

@konstin
Copy link
Contributor

konstin commented Mar 16, 2014

I wrote fix for this Issue, but I think I have to open a new issue about implementing special sounds for each kart, because at the moment, there're only lots of todo's and many lines of unfinished code, although there are ready-to-use .ogg's (e.g. for Tux). So here's what I did:

        if      (s == "large") m_engine_sfx_type = "engine_large";
        else if (s == "small") m_engine_sfx_type = "engine_small";
        else
        {
            if (sfx_manager->soundExist(s))
            {
                m_engine_sfx_type = s;
            }
            else 
            {
                Log::error("[KartProperties]",
                           "Kart '%s' has an invalid engine '%s'.",
                           m_name.c_str(), s.c_str());
                m_engine_sfx_type = "engine_small";
            }
        }

This could done better by changing either small and large in the kart.xml-files or renaming engine large and engine small, but both possibilities aren't very good.

@hiker
Copy link
Contributor Author

hiker commented Mar 16, 2014

Hmm - the comments/to-do's about kart-specific sounds are most likely to give each character a different sfx when it is hit, or fires something or so. So that would be unrelated to the engine sfx (and it did actually work last time I tried, but we never had enough convincing sfx to use it ... and hearing the same sfx all the time got quickly pretty annoying).

Ideally we would also cover sound effects to be packaged with addon karts, but that's then a lot of additional work, and there's no need for this at this stage.

Could you just issue a pull request for this?

@hiker hiker closed this as completed in de50ea4 Mar 18, 2014
@hiker hiker reopened this Mar 18, 2014
@hiker
Copy link
Contributor Author

hiker commented Mar 18, 2014

I reopen this ticket since indeed we should support sound effects as part of addon karts, not only pre-defined sounds in STK.

@hiker hiker added this to the 0.8.3 milestone Sep 22, 2014
@hiker
Copy link
Contributor Author

hiker commented Sep 22, 2014

See also #278, which has some example voices.

@konstin
Copy link
Contributor

konstin commented Sep 22, 2014

There are three 6s mp3, so I'ld rather close it in favor of a general audio discussion (possibly after the release)

@hiker hiker modified the milestones: 0.9.1, 0.9.2 Sep 28, 2015
@auriamg auriamg modified the milestones: 0.9.3, 0.9.2 Apr 14, 2016
@hiker hiker modified the milestones: 0.9.4, 0.9.3 Aug 29, 2017
@Alayan-stk-2 Alayan-stk-2 modified the milestones: 0.10.0, 0.11 Nov 17, 2018
@Alayan-stk-2 Alayan-stk-2 modified the milestones: 1.1, 2.0 Jun 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants