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

minor changes #2893

Merged
merged 1 commit into from
Jul 23, 2024
Merged

minor changes #2893

merged 1 commit into from
Jul 23, 2024

Conversation

aneesh1122
Copy link

@aneesh1122 aneesh1122 commented Jul 23, 2024

  1. fixed placeholder and artist name alignment problem
  2. fixed wrong spelling of 'carousel'
  3. Improved carousel

@fast4x fast4x merged commit b38a09a into fast4x:master Jul 23, 2024
@fast4x
Copy link
Owner

fast4x commented Jul 23, 2024

Crash androidx.media3.common.IllegalSeekPositionException
in
it.fast4x.rimusic.ui.screens.player.PlayerModernKt$PlayerModern$29$6$2$3$2$2$1.invokeSuspend(PlayerModern.kt:2250)

mediaItems size is less than index passed...

@aneesh1122
Copy link
Author

Crash androidx.media3.common.IllegalSeekPositionException
in
it.fast4x.rimusic.ui.screens.player.PlayerModernKt$PlayerModern$29$6$2$3$2$2$1.invokeSuspend(PlayerModern.kt:2250)

mediaItems size is less than index passed...

That's strange. I'm not getting any errors. I'll see what's wrong

@aneesh1122
Copy link
Author

Crash androidx.media3.common.IllegalSeekPositionException
in
it.fast4x.rimusic.ui.screens.player.PlayerModernKt$PlayerModern$29$6$2$3$2$2$1.invokeSuspend(PlayerModern.kt:2250)

mediaItems size is less than index passed...

When do you get this crash?

@fast4x
Copy link
Owner

fast4x commented Jul 23, 2024

When scroll vertical carousel

@aneesh1122
Copy link
Author

When scroll vertical carousel

I never had this issue. The index of pager is also taken from queue media count so that shouldn't happen. does it happen often? Please look into this.

@fast4x
Copy link
Owner

fast4x commented Jul 23, 2024

look
https://github.com/user-attachments/assets/81f4f7bf-e425-4529-b687-ada66bee7c38

Log
2024-07-23T16:58:18.090: androidx.media3.common.IllegalSeekPositionException
androidx.media3.exoplayer.ExoPlayerImpl.setMediaSourcesInternal(ExoPlayerImpl.java:2329)
androidx.media3.exoplayer.ExoPlayerImpl.setMediaSources(ExoPlayerImpl.java:634)
androidx.media3.exoplayer.ExoPlayerImpl.setMediaItems(ExoPlayerImpl.java:592)
it.fast4x.rimusic.utils.PlayerKt.forcePlayAtIndex(Player.kt:76)
it.fast4x.rimusic.ui.screens.player.PlayerModernKt$PlayerModern$29$6$2$3$2$2.emit(PlayerModern.kt:2247)
it.fast4x.rimusic.ui.screens.player.PlayerModernKt$PlayerModern$29$6$2$3$2$2.emit(PlayerModern.kt:2245)

kotlinx.coroutines.flow.internal.SafeCollectorKt$emitFun$1.invoke(SafeCollector.kt:11)
kotlinx.coroutines.flow.internal.SafeCollectorKt$emitFun$1.invoke(SafeCollector.kt:11)
kotlinx.coroutines.flow.internal.SafeCollector.emit(SafeCollector.kt:113)
kotlinx.coroutines.flow.internal.SafeCollector.emit(SafeCollector.kt:82)
androidx.compose.runtime.SnapshotStateKt__SnapshotFlowKt$snapshotFlow$1.invokeSuspend(SnapshotFlow.kt:172)
kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:104)
androidx.compose.ui.platform.AndroidUiDispatcher.performTrampolineDispatch(AndroidUiDispatcher.android.kt:81)
androidx.compose.ui.platform.AndroidUiDispatcher.access$performTrampolineDispatch(AndroidUiDispatcher.android.kt:41)
androidx.compose.ui.platform.AndroidUiDispatcher$dispatchCallback$1.run(AndroidUiDispatcher.android.kt:57)
android.os.Handler.handleCallback(Handler.java:883)
android.os.Handler.dispatchMessage(Handler.java:100)
android.os.Looper.loop(Looper.java:224)
android.app.ActivityThread.main(ActivityThread.java:7590)
java.lang.reflect.Method.invoke(Native Method)
com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:539)
com.android.internal.os.ZygoteInit.main(ZygoteInit.java:950)

@aneesh1122
Copy link
Author

look
https://github.com/user-attachments/assets/81f4f7bf-e425-4529-b687-ada66bee7c38

Log
2024-07-23T16:58:18.090: androidx.media3.common.IllegalSeekPositionException
androidx.media3.exoplayer.ExoPlayerImpl.setMediaSourcesInternal(ExoPlayerImpl.java:2329)
androidx.media3.exoplayer.ExoPlayerImpl.setMediaSources(ExoPlayerImpl.java:634)
androidx.media3.exoplayer.ExoPlayerImpl.setMediaItems(ExoPlayerImpl.java:592)
it.fast4x.rimusic.utils.PlayerKt.forcePlayAtIndex(Player.kt:76)
it.fast4x.rimusic.ui.screens.player.PlayerModernKt$PlayerModern$29$6$2$3$2$2.emit(PlayerModern.kt:2247)
it.fast4x.rimusic.ui.screens.player.PlayerModernKt$PlayerModern$29$6$2$3$2$2.emit(PlayerModern.kt:2245)

kotlinx.coroutines.flow.internal.SafeCollectorKt$emitFun$1.invoke(SafeCollector.kt:11)
kotlinx.coroutines.flow.internal.SafeCollectorKt$emitFun$1.invoke(SafeCollector.kt:11)
kotlinx.coroutines.flow.internal.SafeCollector.emit(SafeCollector.kt:113)
kotlinx.coroutines.flow.internal.SafeCollector.emit(SafeCollector.kt:82)
androidx.compose.runtime.SnapshotStateKt__SnapshotFlowKt$snapshotFlow$1.invokeSuspend(SnapshotFlow.kt:172)
kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:104)
androidx.compose.ui.platform.AndroidUiDispatcher.performTrampolineDispatch(AndroidUiDispatcher.android.kt:81)
androidx.compose.ui.platform.AndroidUiDispatcher.access$performTrampolineDispatch(AndroidUiDispatcher.android.kt:41)
androidx.compose.ui.platform.AndroidUiDispatcher$dispatchCallback$1.run(AndroidUiDispatcher.android.kt:57)
android.os.Handler.handleCallback(Handler.java:883)
android.os.Handler.dispatchMessage(Handler.java:100)
android.os.Looper.loop(Looper.java:224)
android.app.ActivityThread.main(ActivityThread.java:7590)
java.lang.reflect.Method.invoke(Native Method)
com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:539)
com.android.internal.os.ZygoteInit.main(ZygoteInit.java:950)

You're using radio. I already told you that modern players don't work with radio.

@fast4x
Copy link
Owner

fast4x commented Jul 23, 2024

ahhhhhhhhhhhhhh you are right...
sorry I forgot, my work is keeping me very busy.

@fast4x
Copy link
Owner

fast4x commented Jul 23, 2024

Tonight I watch why it doesn’t work with the radio

@aneesh1122
Copy link
Author

ahhhhhhhhhhhhhh you are right...
sorry I forgot, my work is keeping me very busy.

These are all the problems I've found

Problems:

  1. You can't align first page with alignment/arrangement commands. You have to provide padding amount for the first page. I tried to find out thumbnail dimensions to give padding values suitable for every device but I was not able to. Therefore I added a custom slider which controls padding. everyone can set the padding according to their wish.
WsaClient_1mn2EPF2Ly.mp4
  1. play a song and put the app in background or turn off the display. wait for the next song to play and let it play for 10-15 secs. Open the player directly now. The song starts from the beginning.
Screenrecorder-2024-07-22-19-20-15-519.mp4
  1. It has problem with radio. It crashes when a song's radio is started. Start radio and change to the next song. Crash. Works fine from quick picks.
Screenrecorder-2024-07-22-19-18-27-763.mp4
  1. previous song starts playing when exit from lyrics in landscape mode
WsaClient_KuwLz1v6SV.mp4

@aneesh1122
Copy link
Author

Tonight I watch why it doesn’t work with the radio

Pls try to look at other problems too

@fast4x
Copy link
Owner

fast4x commented Jul 23, 2024

Fixed 3

@fast4x
Copy link
Owner

fast4x commented Jul 23, 2024

2, not reproduced

@aneesh1122
Copy link
Author

aneesh1122 commented Jul 23, 2024

2, not reproduced

Song is not playing from start?

Do these steps exactly

  1. play a song and skip to last 10 seconds
  2. minimize the app to background and wait for the next song to play.
  3. let the next song be played for 5-10 seconds.
  4. open app by tapping on notification bar
Screenrecorder-2024-07-23-23-12-41-119.mp4

Turn on the volume

@fast4x
Copy link
Owner

fast4x commented Jul 23, 2024

Fixed 4

@aneesh1122
Copy link
Author

Fixed 4

That's great. Thanks man.

Were you able to reproduce 2?

@fast4x
Copy link
Owner

fast4x commented Jul 24, 2024

Yes reproduced, but it's complicated to solve

@aneesh1122
Copy link
Author

Yes reproduced, but it's complicated to solve

http://github.com/fast4x/RiMusic/pull/2873#issuecomment-2244626732

See this. Maybe it will give you some ideas

fast4x added a commit that referenced this pull request Jul 27, 2024
fast4x added a commit that referenced this pull request Jul 27, 2024
fast4x added a commit that referenced this pull request Jul 27, 2024
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.

2 participants