-
Notifications
You must be signed in to change notification settings - Fork 39
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
Bind "androidx.health:health-services-client" #825
Comments
Thanks for the feedback.
😄 "production ready" is quite elastic term. Previews are stable enough for bindings. It is not about stability, but resources (where I live day is 24 hrs too)
I get you, but cannot promise anything.
Which app if I may know? I work on such app myself as a side project and when I find some time and that is - rarely. |
Thanks, @moljac Yes, this is a part time project for me too. https://play.google.com/store/apps/details?id=com.limegreenv.geartracker What is your app, is it written in dotnet? is it in the store? Do you have a link? I'm also interested 😉 Thanks |
You are welcome.
Cool I will check it.
Currently it is several appsand yes in .NET (Xamarin.Forms and now rewriting in MAUI). I was simply solving some problems when I was coaching my daughters. I was skiing coach. Lara Cvjetko https://www.ijf.org/judoka/29418 https://www.ijf.org/wrl?category=12 https://www.ijf.org/wrl_olympic?category=12 Nina https://www.ijf.org/judoka/48861 Jana https://www.ijf.org/judoka/64009 One of the apps was similar your app. Basic activity tracker, but I have some morphologic/anthropometric (not only) calculations too (BMI, Ponderal index,... Somatotype). Another app was for Somatotype and even ML.net. I did research paper on that. https://pdfs.semanticscholar.org/c78b/43ee7415b047b1d78bde519a1300819a6b2b.pdf https://www.scitepress.org/PublishedPapers/2019/83680/pdf/index.html Then one is of periodization (planing and programming) app. Actually data (competitions) in CSV + algorithm as console app. I dream to merge those apps into one and publish it.
No. I wish.
It is in private repo on gitlab. Here is smaller part public, but it is mess... https://github.com/HolisticWare-Applications/Ph4ct3x Even readme is centuries old. I think some apps might be there. Not sure anymore. Now I am sad/angry. I must clean that up.
|
Oh nice, you do have a lot going on! I'm sure there's still plenty of spare time for bindings though, just cut back on some sleep 😉 |
This is actually going to become pretty big. Many health and study research programs are out there evaluating these projects right now. This particular package is pure kotlin with many generics which has made it tough to bind. @Redth This was package I've been talking about |
PR in progress |
Hey @moljac - The binding does generate, but the connect client uses a bunch of kotlin generics which only translate to Java.Lang.Object on the .NET side. This was causing all sorts of issues in my binding and seems to be an issue with yours as well. |
True.
What kind of issues? Not sure if I will have resources (time) to do manual interventions. And yes this is the drawback of our tooling. |
I can't get it to cast to equivalent type, so I can work with the actual data. I can get more concrete examples.
You don't need to quote this for my benefit. I've been around the Xamarin community long enough to know the underfunding going on. My point about this is that if the binding doesn't provide the necessary functionality, you'll be dealing with more support issues downstream. |
@moljac Try doing a ReadRecords off a IHealthConnectClient. I can't find an implementation of IContinuation for kotlin in the stdlib. How do you create a read request as the class is marked abstract? I got my binding a little farther along with "new ReadRecordsRequest(IKClass, filter, null, true, 2000, "")" but there doesn't seem to be a way get IKClass. The implementation is large and likely not something we're suppose to implement anyhow. I appreciate that timing isn't on your side, but this binding doesn't appear to actually be in a usable state. |
This is why binding previews is no fun. Google moves types in released artifacts and that causes troubles. This is worse. |
Again I am not SDK API expert.
Doesn't inheriting/extending/subclassing/specializing abstract class, implementing abstract stuff and adding behavior work?
|
We're did this end up? Is there a binding that can be released soon? |
Well there are issues with build. Those packages revealed some new fun stuff.
Nugets are built. Not sure if those are accessible from public/outside. Even when everything is green I am not sure that packages will be published. |
hmmm, what does this mean as far as timing/availability? |
If the bindings are getting this hard, where does this leave cross platform development using dotnet? What's the MS model for making bindings, is it supported internally or is it left to the end users/contributors? Any update on whether this will get over the line anytime soon? Sorry for my ignorance, but when I set out to use dotnet-android on my project I assumed I'd be able to use third party libraries with minimal effort. Thanks again, just looking to see where things stand. |
Not sure. Trying to do my best to juggle all my tasks. These bindings introduced new issues and trying to solve them. |
To answer 1st part: not all bindings are that hard. Some are. Check To answer 2nd part: Well I am certainly not the one to answer that.
We have constant discussion about that. Xamarin.Components repo is maintained only if special request comes in. All dependencies are in AX and GPS-FB-MLKit, all together 600+ packages. PRs and contributions are welcome.
I hope so. In
Rethorical question: size of the set called 3rd party libraries (in mathemartical sense) would be? Where do we draw the line?
You are welcome. Even when this PR is done. It will not be merged, because it is in preview and internal builds do not have nuget feeds where users could test preview packages. So you'll have to at least build it locally. THanks. |
Thanks for the update, it sounds like you have a lot on your hands. Should I build locally now, or wait until these isdues have been resolved? I.e. Will it work? |
Tons. We had issues with nuget filesizes on nuget.org (nuget gallery). It is solved now, so I'll be working on this PR soon.
Will it work? Maybe with probability closer to No. If you can check which packages must be bound maybe you could exclude those extras that cause issues until I implement the fix. |
Thank you, I'll wait patiently 👍 |
Sheat. Now I got guilt feeling and I am not being sarcastic. I'll do my best to fix it ASAP. |
Any news? With the upcommig Wear OS 5 and fitness/health apps being the main reason most ppl buy Wear OS wearables and apps.... and, this lib is optimised for better battery life |
@moljac, any progress? |
A bit. We removed classic support and I am stuck with regular updates and new issues with R8. In this PR #835 I have bound 3 health artifacts from 2
As I am not being SDK expert it seems to me like some might be deprecated, but there is no notice (or I did not find any yet), so when adding nugets to app during test I get tons of ACW errors with duplicate/multiply-defined classes. Seems like https://maven.google.com/web/index.html?q=health#androidx.health:health-connect-client might be deprecated. Error:
|
yup It would be nice to have such comment here: https://maven.google.com/web/index.html?q=androidx.health or at least metadata |
are we close? :-) |
Not sure... |
Description
package link: https://maven.google.com/web/index.html?q=health#androidx.health:health-services-client:1.0.0-rc01
Although it's RC, it says 'Production Ready', and it looks like it's in continual development and for some reason never gets to release... so I assume it's stable enough for binding?
The problem we have is that Samsung are cripling foreground services and access to location and health data on Wear OS without consuming via this library. I have a fitness app in the Play Store for Wear OS written in net8-android and with out this it's slowly being crippled, which makes dotnet dev for WearOS unviable.
The text was updated successfully, but these errors were encountered: