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

lib v2 #59

Open
niqdev opened this issue Apr 18, 2018 · 11 comments
Open

lib v2 #59

niqdev opened this issue Apr 18, 2018 · 11 comments
Labels
enhancement Exciting New Features

Comments

@niqdev
Copy link
Owner

niqdev commented Apr 18, 2018

As proposed by @amadeu01 we should remove rx, guava and old libs no longer needed

@niqdev niqdev added the enhancement Exciting New Features label Apr 18, 2018
@niqdev
Copy link
Owner Author

niqdev commented Apr 18, 2018

@amadeu01 I have no time for this but feel free to create a feature branch and discuss here your proposal for a better clean interface for the lib, if you are interested of course

@amadeu01
Copy link
Contributor

@niqdev Thanks for mention me.
My schedule is quite busy at the moment, but I can have a couple of hours in the middle of the night haha.

I'll bring the ideas here, so we can think of something.

@amadeu01
Copy link
Contributor

  1. I think it is impossible the library be pure Java due to the use of Android SDK, which is fine. But, I could have a core write only in Java which you give only the bitmaps to be used as the user want.

  2. We could use Kotlin.

  3. As a new version of the library could have behaviour tests or unit tests. I do not have any idea how could write tests for this kind of library. So, instead of writing some documentation. The user can go to the test and learn how to use the library.

@niqdev
Copy link
Owner Author

niqdev commented Apr 18, 2018

+1 for Kotlin and tests, if we can make it more functional, even better. I'm not up to date with the latest changes on Android SDK unfortunately and I never used Kotlin, but I know Scala and I think I should be able at least to follow your changes.

It would be really nice if we could also clean the "core" code which the library wraps like MjpegInputStreamDefault and MjpegViewDefault. If I remember correctly the native C code is not fully integrated and maybe we can even think to remove it. #32 reference an alternative library that it could potentially be used also

@amadeu01
Copy link
Contributor

We could also implement some travis circleci for run tests and tasks.

I have followed a tutorial about tests, and then I thought would be nice to put some travis and codecov on it amadeu01/ingredisearch

@amadeu01
Copy link
Contributor

@niqdev I guess functional approach is really nice, I once we do not have to save anything on disk, or sql, or something like that. (Perhaps, buffer the video) When can only consume data and pass other data to some function which is willing to consume that data.

@niqdev
Copy link
Owner Author

niqdev commented Apr 19, 2018

We can reuse some of the configs here for circleci and coveralls if you want. By the way this lib originally born in that project and then I decided to extract it, this is the reason behind those libraries. Now, to get rid of dagger, guava or simply upgrade rx from that app requires to much effort unfortunately.

@amadeu01
Copy link
Contributor

@niqdev Have you seen that there will be some changes on NDK and ABI Builds on Android studio 3.2 ? I do not know if it is going to have any bad impact to the library, however we can have look at the documentation

@niqdev
Copy link
Owner Author

niqdev commented May 21, 2018

sorry for the late response, if we are going to integrate only SimpleMjpegView we can get rid of the NDK

@amadeu01
Copy link
Contributor

@niqdev I've never worked with NDK in depth. And, I would like to face some trouble and learn with it haha.

So, we could make a "wrapper for the ndk". I mean, we could make other library, that would be only native library. (Code in C/C++).

Then we could integrate it into this library.

Kotlin

I was thinking, if we use kotlin, we could use coroutines
which I think could make life easy when get rid of rxjava

@niqdev
Copy link
Owner Author

niqdev commented May 24, 2018

If that's the reason +1 if we need NDK, about your link as I said I'm not familiar with Kotlin, so it's up to you

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

No branches or pull requests

2 participants