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.

Sign up for GitHub

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

Jump to bottom

Previewing music crashes BEE2.4 #1035

Closed
Overwatch-C-17 opened this issue Dec 9, 2018 · 19 comments
Closed

Previewing music crashes BEE2.4 #1035

Overwatch-C-17 opened this issue Dec 9, 2018 · 19 comments
Labels
App Crash Something is causing the app to crash Done in next release This has been addressed in the next release

Comments

@Overwatch-C-17
Copy link

Overwatch-C-17 commented Dec 9, 2018 •
edited by LautaroL20
Loading

Crash
@LautaroL20 LautaroL20 added the App Crash Something is causing the app to crash label Dec 10, 2018
@LautaroL20
Copy link
Member

Which music were you trying to test?

@Overwatch-C-17
Copy link
Author

I was testing two music. First I test Bouncy Gel music, then BEE crash. Second I try it again but this one is chamber music Self Esteem Fund, BEE crash again.

@LautaroL20 LautaroL20 changed the title [4.32.0] Testing a music cause BEE to crash [4.32.0] Testing any song from "Bounce Repulsion Gel" will make the entire interface crash!!! Dec 10, 2018
@LautaroL20 LautaroL20 changed the title [4.32.0] Testing any song from "Bounce Repulsion Gel" will make the entire interface crash!!! PRL32 Testing any song from "Bounce Repulsion Gel" will make the entire interface crash!!! Dec 10, 2018
@LautaroL20 LautaroL20 changed the title PRL32 Testing any song from "Bounce Repulsion Gel" will make the entire interface crash!!! PRL32 CRASH: Testing any song from the Bee´s interface will make it crash!!! Dec 10, 2018
@LautaroL20
Copy link
Member

It doesn´t matter which song do you test, it will make the entire bee´s interface crash. All existing songs of beemod causes it.

@Overwatch-C-17
Copy link
Author

Yes true

@LautaroL20 LautaroL20 changed the title PRL32 CRASH: Testing any song from the Bee´s interface will make it crash!!! PRL32 CRASH: Previewing any song from the Bee´s interface will make it crash!!! Dec 10, 2018
@vrad-exe vrad-exe changed the title PRL32 CRASH: Previewing any song from the Bee´s interface will make it crash!!! [4.32.x] Previewing music crashes BEE2.4 Dec 23, 2018
@vrad-exe vrad-exe pinned this issue Dec 23, 2018
@vrad-exe vrad-exe unpinned this issue Dec 23, 2018
@vrad-exe
Copy link
Member

vrad-exe commented Dec 23, 2018 •
edited
Loading

Unrelated to the actual post, but apparently you can pin issues now? Just happened to have this issue open when I noticed it.
image
Might come in handy for things like...this, actually. And maybe BEEmod/BEE2-items#95 as well.

@LautaroL20
Copy link
Member

Yes, the option seems to be available for all the Bee´s members which is good. This issue should be pinned indeed.

@LautaroL20 LautaroL20 pinned this issue Dec 23, 2018
@QuentinMarical
Copy link

Same problem

BEE2 Error!

An error occurred:
Traceback (most recent call last):
File "F:\Git\BEE2.4\src\sound.py", line 200, in play_sample
File "C:\Users\Spencer Brown.virtualenvs\BEE2\lib\site-packages\pyglet\media_init_.py", line 1429, in load
File "C:\Users\Spencer Brown.virtualenvs\BEE2\lib\site-packages\pyglet\media_init_.py", line 1405, in load
File "C:\Users\Spencer Brown.virtualenvs\BEE2\lib\site-packages\pyglet\media\avbin.py", line 238, in init
File "C:\Users\Spencer Brown.virtualenvs\BEE2\lib\site-packages\pyglet\media\avbin.py", line 196, in f
OSError: exception: access violation reading 0x00000000

@LautaroL20 LautaroL20 mentioned this issue Jan 13, 2019
Closed
@mgperkow
Copy link

I got this one, too. Exactly the same error message and traceback as the others who reported it here.

@arminelec
Copy link

arminelec commented Jan 17, 2019 •
edited
Loading

I have the same issue on 4.33.1. With the same file path references. I chose to comment here, even though this ticket says 4.32.x (since the other duplicates are closed).

@LautaroL20 LautaroL20 changed the title [4.32.x] Previewing music crashes BEE2.4 Previewing music crashes BEE2.4 Jan 18, 2019
@LautaroL20
Copy link
Member

LautaroL20 commented Jan 18, 2019 •
edited
Loading

arminelec said: "even though this ticket says 4.32.x"

That is why i don´t like putting a number version in the title, because it can lend to confusion... Since even the latest release has this same issue.

@Endermage77
Copy link

Endermage77 commented Jan 19, 2019 •
edited
Loading

Can also confirm I have same porblemo

I wonder what went wrong with it?

@vrad-exe vrad-exe mentioned this issue Jan 31, 2019
Closed
@TeamSpen210 TeamSpen210 added the Done in next release This has been addressed in the next release label Feb 3, 2019
TeamSpen210 added a commit that referenced this issue Feb 3, 2019
@TeamSpen210
Fix #1035: Previewing packed music crashes.
e1ee453 
The music is a Path object in this case, which fails
an isinstance(str) check in pyglet making the func return
None. None becomes null pointer in ctypes, crash.
@vrad-exe vrad-exe mentioned this issue Feb 22, 2019
Closed
@LautaroL20 LautaroL20 mentioned this issue Feb 23, 2019
Closed
@iTom33ro
Copy link

iTom33ro commented Mar 6, 2019

But how to solve this problem, because i have same crash

@LautaroL20
Copy link
Member

You will have this issue fixed when the Pre-Release 34 comes out.

@LautaroL20
Copy link
Member

You can not fix it by yourself since the file sound.py does not exists in the public releases, otherwise you could edit it easily with notepad++ based on this commit: e1ee453. However, you will have this issue fixed when the Pre-Release 34 comes out.

@TeamSpen210
Copy link
Member

Actually you can just copy that .py file into the bin folder, and it'll override the zipped one.

@LautaroL20
Copy link
Member

Interesting. Go ahead everyone!

@iTom33ro
Copy link

iTom33ro commented Mar 6, 2019

which .py file?

@beepster4096
Copy link

Actually you can just copy that .py file into the bin folder, and it'll override the zipped one.

This works! Thanks for the temporary solution until 34 comes out.

@TeamSpen210 TeamSpen210 mentioned this issue Mar 10, 2019
Merged
@LautaroL20
Copy link
Member

LautaroL20 commented Mar 10, 2019 •
edited
Loading

Pre-Release 34 Has now been released! Download links below:

Interface: https://github.com/BEEmod/BEE2.4/releases/tag/2.4.34.1
Packages: https://github.com/BEEmod/BEE2-items/releases/tag/v4.34.0

@LautaroL20 LautaroL20 unpinned this issue Mar 10, 2019
@LautaroL20 LautaroL20 added Done in next release This has been addressed in the next release and removed Done in next release This has been addressed in the next release labels Mar 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
App Crash Something is causing the app to crash Done in next release This has been addressed in the next release
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
10 participants
@arminelec @TeamSpen210 @LautaroL20 @mgperkow @Overwatch-C-17 @vrad-exe @beepster4096 @Endermage77 @QuentinMarical @iTom33ro