Update onboarding text to match design

Titles are not bold enough in all steps of onboarding. In design the text is bolder.
Update all titles on all onboarding screens to match Figma: https://www.figma.com/file/mBlqs6l6I2d3ihX80Mswm4/Casper-Wallet-–-Extension?node-id=1838-7567&t=Yk42orPjZoVYxEDp-0

Also, remove the underline from the word never on this screen (it looks like a link)

@adammake I reviewed the design and our implementation, font-weight is the same as in the design. This may be a problem on Yulia's side. I talked with her about this and this issue is only in her Chrome browser, but in Safari all fine.
Should we remove the underline from the word? We previously had a ticket about adding this underline

@ost-ptk lets remove underline, per Yulia's guidance.

Done #649

Titles are properly bold now (build 1.3.0), though word "never" is still underlined, thus moving this ticket back to "In Progress".

@piotrwitek I checked develop branch and all works as expected. I think in the build that you posted these changes are missing. Can you take a look?

@ost-ptk correct, this fix was merged after the build was created

@karatnyk it'll be fixed in the next release  build

Verified on latest 1.3.0 build as fixed, marking this ticket as "Done".


QUIC: Fix SSL_export_keying_material()

Fixes https://github.com/openssl/openssl/issues/23560

Thanks for the fast fix. I can confirm that this change fixes the problem for me.
The CI failure seems unrelated:

2024-02-13T11:53:16.3010428Z /usr/bin/ld: final link failed: No space left on device

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged to the master and 3.2 branches. Thank you.


(maint) Switch to different subnet-id for different availability zone

This switches us to availability zone us-west-2a to work on c3.large
capacity issues in us-west-2b.
Signed-off-by: Ken Barber<EMAIL_ADDRESS>

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/705/

@pljenkinsro retest this please

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/706/


Fix readme typo

Removed template because this is a very minor change.

Thanks for the catch. I've also found some other typos. I'll fix them together in one commit


Solution is invalid

Imprting solution zip and getting the following error:
The solution file is invalid. The compressed file must contain the following files at its root: solution.xml, customizations.xml, and [Content_Types].xml. Customization files exported from previous versions of Microsoft Dynamics 365 are not supported.
All files are contained in the zip file

Had to unzip files then make certain the xml files were in the root and rezip and import. that resolved the issue
https://support.microsoft.com/en-us/topic/-the-solution-file-is-invalid-error-occurs-in-dynamics-365-e4a96d92-ccd0-b704-e1bf-c013d3a3fec5


Font error

[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 56b8cbdd-0a23-9f66-6f22-90317472e990
Version Independent ID: c4dc5b5d-f391-fe19-d9d3-aeae9c9f5e7a
Content: Myanmar Text font family - Typography
Content Source: typographydocs/font-list/myanmar-text.md
Product: typography
GitHub Login: @alib-ms
Microsoft Alias: alib

Thank you for sharing the discrepancies with us. Some of the points you mentioned contradict the specific guidelines we received during the font creation process. For instance, we opted for rotated forms instead of squashed forms based on feedback from native speakers. Additionally, these forms are depicted in UTN11: Representing Myanmar in Unicode

However, I think following are issues in our font that should be fixed:

The nga letter in Mon is encoded U+105A ၚ and not U+1004 င as in Burmese. Independently, these
characters look very different. But in the context of something occurring below the character, the Mon nga (U+105A) loses its tail. Thus a Mon kinzi is encoded using U+105A U+103A U+1039. Since this is visually identical to the Burmese kinzi, where confusability is a concern (for example in International Domain Names), the Mon kinzi will not be usable and a misspelling may be necessary to get the desired name. In addition, the medial form of Mon nga is simply the tail: ္ၚ
/(U+1039 U+105A).
https://www.unicode.org/notes/tn11/UTN11_4.pdf


delete branch

often is the case after merging colleagues' PR that it is time to remove their branch from where the PR was sent. :clap:

I believe this should be on the git helper to git push -u remote_name :branch_name however sometimes we have the remote_name be specified so it will have to be gush b:d -u cordoval  and the default name should be the current branch or gush b:d -u cordoval explicit_branch_name

it is not always necessary so it should be left out.


Jenkins collector does not collect any data.

2015-11-23 18:20:00,007 INFO  c.c.d.collector.HudsonCollectorTask - ------------------------------
2015-11-23 18:20:00,007 INFO  c.c.d.collector.HudsonCollectorTask - http://<IP_ADDRESS>:8181
2015-11-23 18:20:00,007 INFO  c.c.d.collector.HudsonCollectorTask - ------------------------------
2015-11-23 18:20:00,071 INFO  c.c.d.collector.HudsonCollectorTask - Fetched jobs                0s
2015-11-23 18:20:00,080 INFO  c.c.d.collector.HudsonCollectorTask - New jobs           1        0s
2015-11-23 18:20:00,083 INFO  c.c.d.collector.HudsonCollectorTask - New builds         0        0s
2015-11-23 18:20:00,083 INFO  c.c.d.collector.HudsonCollectorTask - Finished                    0s
You can see above output from jenkins. Jenkins has 3 project s and some new builds(I run it continually), but  it can't find new builds
my property file is like this.  Do I have to use the username and apikey?  How to set them?
#Jenkins auth - set username/apiKey to use HTTP Basic Auth (blank=no auth)
jenkins.username=
jenkins.apiKey=
Thank you!

@amitmawkin,  Do we have some guide about the jenkins setting?    although the jenkins collector works well, but the dashboard can't work for there is no build to select.("select a build job" has no any options to select).  thank you!
thank you!

When i query the mongo, I found there are some data under dashboard database, although I use admin database,  it seems that Jenkins use the dashboard database whenever your properties file use other database name,  I guess it is hard code


Question: Exercise Sample with Wear Compose

Hello,
Are you planning to show how to use properly the exercise sample with wear compose?
I don't know how to use Android Compose the right way with this example. I started with the Exercise Sample and I replaced xml views by Android Compose. However, I still keep the ExerciseFragment with almost all variables. The only thing I get rid of is _binding and binding.
The thing I don't think is right in my code is that I store some functions from the fragment in my viewModel to call it in my screen (the composable function).
To show you this, in onViewCreated of the ExerciseFragment, I did this :
    viewModel.endExerciseCallback = {
            startEndExercise()
        }

        viewModel.pauseResumeCallBack = {
            pauseResumeExercise()
        }

        viewModel.forceStartExerciseCallback = {
            forceStartExercise()
        }

        viewModel.closeServiceCallBack = {
            stopChronometer()
            unbindViewsFromService()
        }

Can you explain me what's the right way to integrate the Exercise Sample using wear Compose without a mix of the 2 system?
Thank you!

Hi,
Yes, an updated sample using Wear Compose is in progress. Our approach is creating an instance of the ExerciseService in the ViewModel and binding to it in the ViewModel as well. Also in the ViewModel, we have all of the appropriate exercise methods, i.e:
fun pauseExercise() = exerciseService?.pauseExercise()
fun endExercise() = exerciseService?.endExercise()
Then, we access the ViewModel only within the NavHost, and reference the various exercise functions from within the NavHost, passing them into the appropriate screen, i.e:
ExerciseScreen(            onPauseClick = { composableScope.launch { viewModel.pauseExercise() } },                 onEndClick = { composableScope.launch { viewModel.endExercise() } },                 onResumeClick = { composableScope.launch { viewModel.resumeExercise() } },                 onStartClick = { composableScope.launch { viewModel.startExercise() } },                 serviceState = serviceState
Hope this helps! The updated sample should hopefully be published by November, let me know if you have any questions in the meantime.

Thank you breanatate, it's clearer now :) I'll wait to see the whole example.
If I can ask you a question, can you look at my question on SO about saving the state and restore it when user go back to the app? Here's the link : https://stackoverflow.com/questions/73742226/how-to-save-and-update-a-state-in-a-foreground-service
I'll be glad you look at it! Thank you 🙏

Breana,
do you know when you will publish the updated sample with exercise and wear compose? I'm waiting with anticipation. :-)

Any updates on this? Thank you

It's in progress--apologies for the delay. If there's a specific question, I may be able to answer it in the meantime @akrulec and @storatle

Any documentations on how to actually bind a lifecycle service (any service) in Compose/ViewModel would be much appreciated. It's considered an anti-pattern right now: https://medium.com/androiddevelopers/viewmodels-and-livedata-patterns-antipatterns-21efaef74a54

No need for apologies @breanatate, I see you are working on it. It looks good :-)

I have found a recommendation from the WearOS about services and ongoing activity: https://developer.android.com/codelabs/ongoing-activity#3 . However, when I try to run ExerciseService like that, the HealthServiceClient dies when the app is in the background:
W/ServiceConnection: Binder died for client:HealthServicesExerciseClient
W/ServiceConnection: WCS SDK Client 'HealthServicesExerciseClient' disconnected, retrying connection. Retry attempt: 0
    java.util.concurrent.CancellationException
        at androidx.health.services.client.impl.ipc.internal.ServiceConnection.lambda$cleanOnDeath$0$androidx-health-services-client-impl-ipc-internal-ServiceConnection(ServiceConnection.java:308)
        at androidx.health.services.client.impl.ipc.internal.ServiceConnection$$ExternalSyntheticLambda0.binderDied(Unknown Source:2)
        at android.os.IBinder$DeathRecipient.binderDied(IBinder.java:305)
        at android.os.BinderProxy.sendDeathNotice(BinderProxy.java:685)

I use Koin dependency injection:
fun provideHealthServicesClient(context: Context): HealthServicesClient {
    return HealthServices.getClient(context)
}

I create the HealthServicesManager as suggested in this sample:
class HealthServicesManager
constructor(healthServicesClient: HealthServicesClient, coroutineScope: CoroutineScope) {
    private val exerciseClient = healthServicesClient.exerciseClient

And my ExerciseService is a lifecycle service -- could this be the problem here?
class ExerciseService : LifecycleService() {
    private val healthServicesRepository: HealthServicesRepository by inject()
    private val healthServicesManager: HealthServicesManager by inject()

Thanks,

Hi @akrulec thanks for pointing that out. Yes, not ideal or recommended--still playing around with overall structure and want to be sure it's set up correctly before it goes live.
I think the LifecycleService is the right call, but the error you sent has been reported recently from a few other developers. Can you share the Health Services version so that I can investigate if it's the same issue?
adb shell dumpsys package com.google.android.wearable.healthservices | grep -E "versionCode|versionName"

adb shell dumpsys package com.google.android.wearable.healthservices | grep -E "versionCode|versionName"
    versionCode=70695 minSdk=30 targetSdk=30
    versionName=<IP_ADDRESS>6506339
    ```

@akrulec That version is close to a year old. Can you update it and try again?
You'll need to pair with a phone, and "transfer" an account from the phone from the watch so that Play Store on the watch will install the update.
If you're using Android Studio on an Intel Mac try this device (not sure whether it applies to other machines):

And this image:

(The more recent API 33 image has some problems.)
You should end up with:
$ adb shell dumpsys package com.google.android.wearable.healthservices | grep -E "versionCode|versionName"
versionCode=168741
versionCode=70695
versionName=<IP_ADDRESS>0530839
versionName=<IP_ADDRESS>6506339

Thank you for the response! I'm running the demo in my emulator watch on Android Studio. It is Wear OS Small Round API 30. There is no option to create a devices with a system image that would have API 31.

@akrulec You need to select the Wear emulator, and then click "Pair Wearable". This will attempt to pair with an emulated phone (but that phone needs to be running API 31, not 33).

From what I understood the wear OS device was running a version of healthservices that was too old. My phone device is an actual android device that runs Android version 12, so API 32, and it does not have the healthservices at all:
adb devices
List of devices attached
R5CRB0RSNCD	device.  <- phone
emulator-5554	device.  <- emulator watch

nogavicka@totoro android-client % adb -s R5CRB0RSNCD shell dumpsys package com.google.android.wearable.healthservices | grep -E "versionCode|versionName"
nogavicka@totoro android-client % adb -s emulator-5554 shell dumpsys package com.google.android.wearable.healthservices | grep -E "versionCode|versionName"
    versionCode=70695 minSdk=30 targetSdk=30
    versionName=<IP_ADDRESS>6506339

The binder crash I see is being reported on the emulator wear OS device:
W/ServiceConnection: Binder died for client:HealthServicesExerciseClient
W/ServiceConnection: WCS SDK Client 'HealthServicesExerciseClient' disconnected, retrying connection. Retry attempt: 0
    java.util.concurrent.CancellationException
        at androidx.health.services.client.impl.ipc.internal.ServiceConnection.lambda$cleanOnDeath$0$androidx-health-services-client-impl-ipc-internal-ServiceConnection(ServiceConnection.java:308)
        at androidx.health.services.client.impl.ipc.internal.ServiceConnection$$ExternalSyntheticLambda0.binderDied(Unknown Source:2)
        at android.os.IBinder$DeathRecipient.binderDied(IBinder.java:305)
        at android.os.BinderProxy.sendDeathNotice(BinderProxy.java:685)

I am running Android Studio Electric Eel | 2022.1.1
Build #AI-221.6008.13.2211.9477386, built on January 10, 2023
Runtime version: 11.0.15+0-b2043.56-8887301 x86_64
Thank you

@akrulec You need to update the Health Services on the watch. To do this you need to pair it with a phone, by selecting the "Pair Wearable" function as shown here: https://github.com/android/health-samples/issues/51#issuecomment-1412981600. Then Play Store on the watch will be able to process updates.

Thank you for the explanation. I had no idea that's how the services got updated on the emulator. I paired the watch, but no update happened. So I navigated to the PlayStore in the emulator, and found Health Services, and selected update. Turns out, that now I have both version installed. I tried restarting the emulator, and that didn't help.
adb -s emulator-5554 shell dumpsys package com.google.android.wearable.healthservices | grep -E "versionCode|versionName"
    versionCode=168739 minSdk=30 targetSdk=30
    versionName=<IP_ADDRESS>0530839
    versionCode=70695 minSdk=30 targetSdk=30
    versionName=<IP_ADDRESS>6506339

I tried uninstalling, but it only removed one. It throws an error when trying to remove the other one:
adb -s emulator-5554 uninstall com.google.android.wearable.healthservices
Success

adb -s emulator-5554 shell dumpsys package com.google.android.wearable.healthservices | grep -E "versionCode|versionName"
    versionCode=70695 minSdk=30 targetSdk=30
    versionName=<IP_ADDRESS>6506339

adb -s emulator-5554 uninstall com.google.android.wearable.healthservices
Failure [DELETE_FAILED_INTERNAL_ERROR]

It looks like something is wrong with the underlying package names/versioning. It would be nice if the right version of the services would be available in the emulator with the next android studio update.

@nogavicka When there's two versions listed, the older one is typically a system version that cannot be removed. It should just be dormant though—with the new version in place, do you still get the error you reported at https://github.com/android/health-samples/issues/51#issuecomment-1411124197?

(And yes, we know it's not ideal to have to update health services like this, and are working on a fix.)

yes! if i update each time i reset the emulator, i don't run into the error anymore. thank you

Closing this issue as the Compose-version is published. We'll continue to iterate and optimize--please let us know if you have any feedback!


New menu structure

It might look better if we removed some of the buttons from the program itself, and keep them in the "Main menu"
So what i suggest is that we remove "Reboot", "Options" and "Mode switch". These functions are still accessable from the "Main menu" and it will give more room on the display for more visuals and informations. + i think it looks cleaner.

That's a good idea. I planned to redesign the gui anyway.


[BUG] Mask is being clipped when the recorded T1 is far off center compared to MNI152NLin2009cAsym

Resolved in 4e6732d


WinGetCommandLineInterfaces Group Policy Exception for COM API Calls

[WHY:]
Currently, the policy check within Command::Execute, which is executed for all command implementations, also inadvertently blocks COM API calls due to inheritance from the command class.
[FIX:]
Allow COM API calls while the WinGetCommandLineInterfaces group policy is disabled.
The fix involves bypassing the policy check for calls originating from COM API, identified by a context flag set in the overridden Execute method in COMCommand.
This pull request is being closed as there is another one addressing the same issue https://github.com/microsoft/winget-cli/pull/4293/:
[TODOs:]

Test run to make sure fix works as expected

[ ] I have signed the Contributor License Agreement.
[ ] This pull request is related to an issue.

Microsoft Reviewers: Open in CodeFlow

Hello  @Madhusudhan-MSFT , could you please advise when this fix will be released, and will it require an update to the Intune management engine ?


Missing Data Types

Please make sure your examples inclue each and every data type supported by the platform.
Currently, the Basic Create example inclues:
var data =
{
"name": "Sample Account",
"creditonhold": false,
"address1_latitude": 47.639583,
"description": "This is the description of the sample account",
"revenue": 5000000,
"accountcategorycode": 1
}
Lookups get handled in their own section, but missing are:
Multi-Select Option Set
Image
Date and Time
File
I'm leaving this comment on the createRecord page, but the same is true for the updateRecord page here: https://learn.microsoft.com/en-us/power-apps/developer/model-driven-apps/clientapi/reference/xrm-webapi/updaterecord

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 75f70840-ac77-de34-b797-233a4ff56d37
Version Independent ID: e6b3d742-fe79-7442-0cae-d5dcba71f55f
Content: createRecord (Client API reference) in model-driven apps - Power Apps
Content Source: powerapps-docs/developer/model-driven-apps/clientapi/reference/Xrm-WebApi/createRecord.md
Service: powerapps
Sub-service: mda-developer
GitHub Login: @adrianorth
Microsoft Alias: aorth

Thanks for your comment, @devlinl. This issue has been assigned to the doc author for review.

@devlinl
I don't think the documentation about Xrm.WebApi.createRecord or Xrm.WebApi.updateRecord needs to be where the format for setting these data types needs to be documented. The payload should be exactly the same payload you send to the Web API, and this documentation should reference that.
I can see that there is a mention of a deprecated method that sets the lookup without @odata.bind. That's not standard. But I expect everything else is.
This topic should cover any exceptions to that, such as the deprecated way of setting a lookup. Or if the offline behavior is different.
So, it is good feedback and there is a gap, but I think it should be covered in Web API content, such as Create a table row using the Web API
I will create a workitem for this. (for my reference: 3147069). When that work is done, these topics can link to it.
This information is available in different places,
For example:

Setting choices values
Use image data with records
Use file column data, but I don't think you will be able to use file column data with Xrm.WebApi.createRecord or Xrm.WebApi.updateRecord. Perhaps with multiple calls using Xrm.WebApi.online.execute starting with the InitializeFileBlocksUpload Action

@devlinl
Appreciate the background. That helps a lot.
If you know where to look, you can find some documentation about this here Using OData APIs to submit user local date and time values, but this isn't a place where people would expect to find it.  Moving it out to a place where people will discover it is important.
I'd love to get your thoughts on the content that exists. As I move it, I want to take the time to ensure it is accurate and relevant.

I took a look at that page... There is some description of the different behaviors between setting a Date Time value in the UI vs via the API, but it seems to be more about how the platform handles Time Zone conversion for User Local Date Times... specifically, that the UI does the conversion, and the API does not.
So, while I agree that this particular information might be better presented elsewhere (not under the heading of "Work with table definitions using code"), it still doesn't address the difference in the expected input format of the DateOnly behaivor.

Thank you. Working on it.


Makefile: fix 'view' goal

This commit fixes this error:
$ make view
...
ALL TESTS PASSED.
gtkwave testbench_vcd testbench.gtkw
GTKWave Analyzer v3.3.78 (w)1999-2016 BSI
Error opening  .vcd file 'testbench_vcd'.
Why: No such file or directory
Makefile:20: recipe for target 'view' failed
make: *** [view] Error 255
Signed-off-by: Antony Pavlov<EMAIL_ADDRESS>

This reverts 75830805. See commit message of that commit for rationale.
I've now renamed "testbench_vcd" to "test_vcd" and removed "make view" in commit ce862f0.


Suggestion - Test with Historical Live Data

Hi,
As I learned from #18 , the TESTNET data is very limited.
It would be nice to be able to run the bot against past LIVE data for more results and also not have to wait 5 minutes each time for small changes.

This is implemented.


OTA Callback changes for custom and secondary processor jobs v2

OTA Callback changes for custom and secondary processor jobs v2
Description

Modifying the OTA Agent to use callback structure instead of directly calling PAL functions. This allows customers to pass in custom callbacks for the PAL functions.
Important note: This commit has been tested but should not be merged into any branches - it is still missing documentation and has a few issues. Creating PR so that code can be reviewed

Updated agent to pass file ID in callback signatures that did not have file context.
Created default callbacks for the new signatures to keep the code backwards compatible.
Saved the file ID in the OTA agent context so that it can be passed to callbacks.
Renamed OTA_AgentInit_v2 to OTA_AgentInit_internal
Self test check does not check for version numbers if the file ID is not 0 - it just updates the image state to eOTA_ImageState_Testing
Added code to check for custom job callback return code and respond back with a success.

Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

Closing PR to create a new one with changes requested


Cannot retrieve 'qid' field from DMatrix

It doesn't seem to be possible to retrieve the 'qid' field of a DMatrix, neither as an integer field, nor as a float field.
Example:
import numpy as np, xgboost as xgb
rng = np.random.default_rng(seed=13)
X = rng.standard_normal(size=(100,10))
y = rng.integers(1, size=100)
qid = rng.integers(3, size=100)
qid = qid[np.argsort(qid)]
dm = xgb.DMatrix(X, label=y, qid=qid)

dm.get_uint_info("qid")

XGBoostError: [20:23:13] [/workspace/src/data/data.cc:628](https://untitled+.vscode-resource.vscode-cdn.net/workspace/src/data/data.cc:628): Unknown uint32 field name: qid

dm.get_float_info("qid")

XGBoostError: [20:23:26] [/workspace/src/data/data.cc:619](https://untitled+.vscode-resource.vscode-cdn.net/workspace/src/data/data.cc:619): Unknown float field name: qid

Since one can get the group when supplying qid, I guess it should be possible to retrieve qid too.

The qid is transformed into groups internally.

Just in case it's useful, I'm working on learning to rank to have XGBoost sort the data automatically based on QID for training. The sort is not needed for prediction on evaluation, I don't have a fully working prototype yet.


Add support for the use of .. in structs

Motivation
Currently, when a struct is instantiated every field must be specified.
As a user I may not care about every field when creating a new struct and thus it would be nice to have a default value take place for the fields that are not specified.
This would be done using the .. syntax similar to Rust.
struct MyStruct {
    field1: type1,
    field2: type2,
    field3: type3,
}

let my_struct = MyStruct { field2: value2, .. };

Related: #1568

The above doesn't quite work -- what should the other fields be initialised to?  We don't support having uninitialised values in Sway, just like Rust.
What Rust does have and Sway doesn't yet is initialising one struct with the fields of another, so the above might be:
struct MyStruct {
    field1: type1,
    field2: type2,
    field3: type3,
}

let struct_a = MyStruct { field1: value1, field2: value2, field3: value3 };
let struct_b = MyStruct { field2: other_value2, ..struct_a };  // Take field1 and field3 from `struct_a`.

The above doesn't quite work -- what should the other fields be initialised to? We don't support having uninitialised values in Sway, just like Rust.
What Rust does have and Sway doesn't yet is initialising one struct with the fields of another, so the above might be:
struct MyStruct {
    field1: type1,
    field2: type2,
    field3: type3,
}

let struct_a = MyStruct { field1: value1, field2: value2, field3: value3 };
let struct_b = MyStruct { field2: other_value2, ..struct_a };  // Take field1 and field3 from `struct_a`.

In my example the default values would be set per type since each value is set to 0 for each type.
E.g. a b256 would be 0x000000..., a bool would be false, ints would be 0 etc.

OK, that would kinda work for now.  One day we may add reference types, which simply can't safely have a default initialiser.  And what if it actually doesn't make sense for a value to be zero in a certain context?  Like the divisor in a rational?
In your case what would be best is to add a constructor method, probably called default() which initialises the struct to those zero values and then you can mutate it from there.  If it truly makes sense for it to be possible to initialise only a single field in a struct and have defaults for the others then you could make a special new(value2) constructor which does that for you.

OK, that would kinda work for now. One day we may add reference types, which simply can't safely have a default initialiser. And what if it actually doesn't make sense for a value to be zero in a certain context? Like the divisor in a rational? Why should false be the default for bool? Or any enum? What's the default for Result?
In your case what would be best is to add a constructor method, probably called default() which initialises the struct to those zero values and then you can mutate it from there. If it truly makes sense for it to be possible to initialise only a single field in a struct and have defaults for the others then you could make a special new(value2) constructor which does that for you.

Those enums would just not implement a default value, ie you would be forced to specify that value in a struct.
IIRC how Rust does it is just have a default trait, and if it makes sense for a trait to have a default value, implement that trait, if theres no logical default value, just dont. So consequently you can skip out on having to fill out default values on structs by using the .. notation and specify the values that are required to be specified


feat: make blob(binary) type working

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
Main changes:

Test blob (binary) type with MySQL protocols and sqlness.
If we can't extract literal values from insert statement, execute the statement by query engine.
Fixed some issues while implementing.

Checklist

[ ]  I have written the necessary rustdoc comments.
[x]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)

@MichaelScofield PTAL


Update best-practices.md

@marcelodiiorio : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@marcelodiiorio - Thanks for the fixes.
Verified the updates; merging.


:sparkles: Add team invite links

We don't support sending email in self-hosted Sequin yet. By adding team invite link support, we make it easy for users to invite other team members to their Sequin team.

#690  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Cannot edit API

hello..i. using 0.6.4 version. when im try to edit API, it`s not show the api detail with error

angular.js:14199 TypeError: Cannot read property 'version' of undefined
at ChildScope.$scope.isRequestPathOrUriStripped (apis-controller.js:42)
at Array.fn0 (eval at compile (angular.js:15126), :4:1063)
at expressionInputsWatch (angular.js:16274)
at Scope.$digest (angular.js:17828)
at Scope.$apply (angular.js:18102)
at angular.js:19950
at completeOutstandingRequest (angular.js:6045)
at angular.js:6324

. But when i`m clicking with new tab..the api detail was appear.

Hello @phenom1711 ,
I can't seem to be able to reproduce this bug.
Could you please provide more info, like the steps you followed until the issue occurred?

after first installation:

login to konga dashboard
choose edit APIs, then click name of API
doesn`t show API details like this

if i inspecting error...it`s show

angular.js:14199 TypeError: Cannot read property 'version' of undefined
at ChildScope.$scope.isRequestPathOrUriStripped (apis-controller.js:42)
at Array.fn0 (eval at compile (angular.js:15126), :4:1063)
at expressionInputsWatch (angular.js:16274)
at Scope.$digest (angular.js:17828)
at Scope.$apply (angular.js:18102)
at angular.js:19950
at completeOutstandingRequest (angular.js:6045)
at angular.js:6324

i`m using chrome 57.x

Still can't reproduce this issue.
Have you defined an active connection in settings?
Is your current connection / node displaying correctly at the bottom-right side of the page?
The error you are getting implies that no active connection is defined thus Cannot read property 'version' of undefined

hai pantsel...i think you`r right. i still dont understand why it can be happed.


uploaded file link was not pass to  src

response should be parsed before you can use the "link" property

Good catch, it looks like the response-parsing step was unintentionally removed in #1. Not sure how I missed that problem and let it through. Fixed up now though. 👍


[docs-v2] Broken Links List

Reproduction steps
I ran a linkchecker on https://docs-v2.trinsic.id and found these broken links.  This issue is to track their completion.
Will be tracking completion in this notion doc (lines 7-21) and post as a single PR (will update here once started)
URL        '../learn/platform/intro.md'
Name       'Learn More'
Parent URL https://docs-v2.trinsic.id, line 1852, col 1
Real URL   https://docs-v2.trinsic.id/learn/platform/intro.md
Check time 2.716 seconds
Result     Error: 404 Not Found
URL        '../learn/concepts/ecosystems.md'
Name       'Learn More'
Parent URL https://docs-v2.trinsic.id, line 1855, col 1
Real URL   https://docs-v2.trinsic.id/learn/concepts/ecosystems.md
Check time 2.770 seconds
Result     Error: 404 Not Found
URL        '../reference/index.md'
Name       'Explore API'
Parent URL https://docs-v2.trinsic.id, line 1858, col 1
Real URL   https://docs-v2.trinsic.id/reference/index.md
Check time 2.937 seconds
Result     Error: 404 Not Found
10 threads active,    24 links queued,  197 links in 231 URLs checked, runtime 15 seconds
URL        './intro.md'
Name       'key concepts'
Parent URL https://docs-v2.trinsic.id/learn/, line 1799, col 19
Real URL   https://docs-v2.trinsic.id/learn/intro.md
Check time 3.034 seconds
Result     Error: 404 Not Found
URL        '../proto/index.md#serviceoptions'
Name       'ServiceOptions'
Parent URL https://docs-v2.trinsic.id/reference/, line 1974, col 30
Real URL   https://docs-v2.trinsic.id/proto/index.md#serviceoptions
Check time 3.331 seconds
Result     Error: 404 Not Found
URL        '../reference/services/wallet-service.md'
Name       'Wallet Service'
Parent URL https://docs-v2.trinsic.id/learn/platform/intro/, line 1903, col 4
Real URL   https://docs-v2.trinsic.id/learn/platform/reference/services/wallet-service.md
Check time 3.043 seconds
Result     Error: 404 Not Found
URL        '../reference/services/credential-service.md'
Name       'Credential Service'
Parent URL https://docs-v2.trinsic.id/learn/platform/intro/, line 1907, col 4
Real URL   https://docs-v2.trinsic.id/learn/platform/reference/services/credential-service.md
Check time 3.227 seconds
Result     Error: 404 Not Found
10 threads active,     8 links queued,  214 links in 234 URLs checked, runtime 21 seconds
URL        '../reference/services/template-service.md'
Name       'Template Service'
Parent URL https://docs-v2.trinsic.id/learn/platform/intro/, line 1911, col 4
Real URL   https://docs-v2.trinsic.id/learn/platform/reference/services/template-service.md
Check time 3.145 seconds
Result     Error: 404 Not Found
URL        '../reference/services/trust-registry-service.md'
Name       'Trust Registry Service'
Parent URL https://docs-v2.trinsic.id/learn/platform/intro/, line 1916, col 4
Real URL   https://docs-v2.trinsic.id/learn/platform/reference/services/trust-registry-service.md
Check time 3.058 seconds
Result     Error: 404 Not Found
URL        '../reference/services/provider-service.md'
Name       'Provider Service'
Parent URL https://docs-v2.trinsic.id/learn/platform/intro/, line 1919, col 4
Real URL   https://docs-v2.trinsic.id/learn/platform/reference/services/provider-service.md
Check time 2.739 seconds
Result     Error: 404 Not Found
URL        './standards.md'
Name       'Learn More'
Parent URL https://docs-v2.trinsic.id/learn/platform/intro/, line 1922, col 4
Real URL   https://docs-v2.trinsic.id/learn/platform/intro/standards.md
Check time 3.230 seconds
Result     Error: 404 Not Found
URL        '/reference/services/provider-service.md#add-webhook'
Name       'AddWebhook'
Parent URL https://docs-v2.trinsic.id/learn/platform/webhooks/, line 1906, col 52
Real URL   https://docs-v2.trinsic.id/reference/services/provider-service.md#add-webhook
Check time 3.251 seconds
Result     Error: 404 Not Found
URL        '/cli/index.md'
Name       'CLI'
Parent URL https://docs-v2.trinsic.id/learn/platform/webhooks/, line 1906, col 153
Real URL   https://docs-v2.trinsic.id/cli/index.md
Check time 3.199 seconds
Result     Error: 404 Not Found
URL        '/reference/services/account-service.md#authorize-webhook'
Name       'AuthorizeWebhook'
Parent URL https://docs-v2.trinsic.id/learn/platform/webhooks/, line 1977, col 47
Real URL   https://docs-v2.trinsic.id/reference/services/account-service.md#authorize-webhook
Check time 3.433 seconds
Result     Error: 404 Not Found
URL        '/learn/platform/wallets.md'
Name       'Wallets'
Parent URL https://docs-v2.trinsic.id/walkthroughs/vaccination/, line 2139, col 22
Real URL   https://docs-v2.trinsic.id/learn/platform/wallets.md
Check time 3.114 seconds
Result     Error: 404 Not Found
Reproduction Code
docker run --rm -it -u $(id -u):$(id -g) ghcr.io/linkchecker/linkchecker:latest https://docs-v2.trinsic.id

Fixes have been completed and tested both manually and with the same script that found them:
docker run --rm -it -u $(id -u):$(id -g) ghcr.io/linkchecker/linkchecker:latest https://trinsic-docs-pr-fb02d50379fedebd13e6944cb72beb75ee794b20.netlify.app/


[Bug]: MistralAI model "mistral-medium" as ReAct Agent

Bug Description
I use MistralAI as ReAct agent with tools
llm = MistralAI(
  model="mistral-medium",
  safe_mode=True
)

agent = ReActAgent.from_tools(
  tools=tools,
  llm=llm,

But medium model didn't work as expected.
mistral-small - works fine and can call tools
mistral-medium - didn't call tools and return some hallucinations
Version
0.9.39
Steps to Reproduce

Setup llm with mistral-medium model
Run ReAct agent with tools
Check observation and tool result

Relevant Logs/Tracbacks
No response

This isn't really a llama-index issue -- this is more of an issue with open-source models in general. Their agent-like reasoning capability is just note quite as good
You could maybe try to prompt engineer the react prompts, but I've had limited success with this. In general, open-source models are far behind for agent use-cases


Cannot process same PDF file twice

We're facing a big issue right now where we can't process the same PDF twice. The first time I try to parse it, it works fine then the second time I'm getting this error:
[{'detail': 'Result not found. Check job status to see if it has completed.', 'job_id': '66cf8baf-923d-43bb-8d7f-d3f95a89a79c', 'file_path': 'pdf-document.pdf'}]

solution and explanation is here : https://github.com/run-llama/llama_parse/issues/278

@tkcoding I saw that solution and adding the invalidate_cache = True to my parser object did not fix the code. Reference below the code I am using:
parser_gpt4o = LlamaParse(
        api_key=environ.get("LLAMA_CLOUD_API_KEY"),
        result_type="markdown",
        verbose=True,
        language="en",
        gpt4o_mode=True,
        gpt4o_api_key=environ.get("LLAMAPARSE_OPENAI_API_KEY"),
        ignore_errors=False,
        invalidate_cache=True,
    )

document = parser_gpt4o.get_json_result(file_path)

I'm able to cause the issue. Below is the full code. While the jobs are marked as SUCCESS, I'm unable to download the results.
Parse
Python Code
from os import environ
import sys

file_path = sys.argv[1]
parser_gpt4o = LlamaParse(
    api_key=environ["LLAMA_CLOUD_API_KEY"],
    verbose=True,
    gpt4o_mode=True,
    gpt4o_api_key=environ["LLAMAPARSE_OPENAI_API_KEY"],
    ignore_errors=False,
    invalidate_cache=True,
    do_not_cache=True,
)
try:
    print("Markdown")
    print(parser_gpt4o.load_data(file_path))
except KeyError as e:
    print("Error: ", e)
print("JSON")
print(parser_gpt4o.get_json_result(file_path))

Shell Output
Markdown
Started parsing the file under job_id a45e687b-7901-43ec-ab79-de9407e977c8
Error while parsing the file '/Users/areichert/Desktop/pronto-forms.pdf': 'text'
Error:  'text'
JSON
Started parsing the file under job_id 78df4a75-bfaf-47c7-a347-05f6db0a8ec5
[{'detail': 'Result not found. Check job status to see if it has completed.', 'job_id': '78df4a75-bfaf-47c7-a347-05f6db0a8ec5', 'file_path': '/Users/areichert/Desktop/pronto-forms.pdf'}]

Job Marked As Success But Cannot Download.
curl -X 'GET' \
  'https://api.cloud.llamaindex.ai/api/parsing/job/78df4a75-bfaf-47c7-a347-05f6db0a8ec5' \
  -H 'accept: application/json' \
  -H "Authorization: Bearer $LLAMA_CLOUD_API_KEY"
echo
curl -X 'GET' \
  'https://api.cloud.llamaindex.ai/api/parsing/job/78df4a75-bfaf-47c7-a347-05f6db0a8ec5/result/markdown' \
  -H 'accept: application/json' \
  -H "Authorization: Bearer $LLAMA_CLOUD_API_KEY"

{"id":"78df4a75-bfaf-47c7-a347-05f6db0a8ec5","status":"SUCCESS"}
{"detail":"Result not found. Check job status to see if it has completed."}%

@tkcoding I saw that solution and adding the invalidate_cache = True to my parser object did not fix the code. Reference below the code I am using:
parser_gpt4o = LlamaParse(
        api_key=environ.get("LLAMA_CLOUD_API_KEY"),
        result_type="markdown",
        verbose=True,
        language="en",
        gpt4o_mode=True,
        gpt4o_api_key=environ.get("LLAMAPARSE_OPENAI_API_KEY"),
        ignore_errors=False,
        invalidate_cache=True,
    )

document = parser_gpt4o.get_json_result(file_path)

I couldn't replicate this issue by using the same setup you have there, I tried to run the document three times and it went through OK.
Did you try other document ?

Did you try other document ?

I found it reproducable with any .txt document

@jaikb @emorling @adreichert
I think I figured out what happened .
Basically it failed at sending the document to gpt4o which is kind of "deprecated" by now.
The old 'gpt4o_mode=True' doesnt work as expected anymore. Dev team have introduced new args which should be working for your test case .
I tested out on a .txt and it threw connection exception when processed second time, after I change to vendor_multimodel args as below then it's working now.
LlamaParse(
            verbose=True,
            ignore_errors=False,
            invalidate_cache=True,
            do_not_cache=True,
            vendor_multimodal_api_key=environ["LLAMAPARSE_OPENAI_API_KEY"],
            vendor_multimodal_model_name = 'gpt4o_mini',
            use_vendor_multimodal_model=True
        )

This is now solved in production without the need to invalid_cache=True


khd does not see itself in the path

On macOS Sierra 10.12 16A319, I installed khd 1.0.1 with brew install koekeishiya/khd/khd and started it with brew services start khd.
This key shortcut didn't do anything:
lcmd - o : khd -p "- left"

Instead, it produced this log output in /tmp/khd.err:
/bin/bash: khd: command not found

That's surprising, because Activity Monitor shows that khd is running as my login user, and the $PATH prepared by my .bashrc includes khd's location, /usr/local/bin.
Specifying the path worked around the problem:
lcmd - o : /usr/local/bin/khd -p "- left"

But it seems like that should not be necessary, since the example .khdrc does not specify the path to khd.

Unable to reproduce on my end. Not sure what's up with that.

Having the same issue above :(
I use zsh myself not sure if it matters?

Could this maybe have to do with the commands being run through /bin/bash, even if another default shell is set using chsh? With zsh for example $PATH is obviously not set in .bashrc but rather in .zshrc.
Is there a way to get the default shell as set through chsh on macOS?

The default shell should be set in the env variable $SHELL.

Then I'd suggest reading that variable and running the commands through it, assuming zsh/fish/whataver -c works similarly to bash -c.

I know zsh supports -c, but I have no experience with other shells

fish does as well, just looked it up. In case it doesn't work people can always open new issues I suppose? Seems like a change worth doing to me.

Done, see #10

I'm still running into the same problem even after #10.

It seems the issue has to do with launchd not setting the correct environment variables when starting khd. Adding some debugging printfs shows that $PATH is /usr/bin:/bin:/usr/sbin when khd is started.
Updating com.koekeishiya.khd.plist to set the correct PATH env var solved the issue for me. eg. assuming khd is at /usr/local/bin/khd
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<dict>
    <key>Label</key>
    <string>com.koekeishiya.khd</string>
    <key>ProgramArguments</key>
    <array>
        <string>/usr/local/bin/khd</string>
    </array>
    <key>EnvironmentVariables</key>
    <dict>
        <key>PATH</key>
        <string>/usr/local/bin:/usr/bin:/bin:/usr/sbin</string>
    </dict>
    <key>RunAtLoad</key>
    <true/>
    <key>KeepAlive</key>
    <true/>
    <key>StandardOutPath</key>
    <string>/tmp/khd.out</string>
    <key>StandardErrorPath</key>
    <string>/tmp/khd.err</string>
    <key>Sockets</key>
    <dict>
        <key>Listeners</key>
        <dict>
            <key>SockServiceName</key>
            <string>3021</string>
            <key>SockType</key>
            <string>dgram</string>
            <key>SockFamily</key>
            <string>IPv4</string>
        </dict>
    </dict>
</dict>
</plist>

One solution would be sudo launchctl config <user> <path> $PATH, but that only applies after rebooting and affects every application run through launchctl.
Just to make sure, does sudo launchctl procinfo <pid> only show the default $PATH as well?

I have done neither of the above changes and it still sets my path properly..weird

Same here, after this change everything works perfectly! If patch isn't set
correctly check that you have defined the path in the right configuration
file for your shell, because executing a command with the -c flag doesn't
read all configuration files
--
Enrico Ghirardi
On 7 October 2016 at 11:38:13, Åsmund Vikane<EMAIL_ADDRESS>wrote:

I have done neither of the above changes and it still sets my path
properly..weird
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/koekeishiya/khd/issues/9#issuecomment-252197052, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAFOF6oq2GPjZHzS-9zm26xI90Dhw92rks5qxhMFgaJpZM4KIZiW
.

I have done neither of the above changes and it still sets my path properly..weird

Same. Don't get his issue either, but setting the path globally for launchctl is a workaround that should do the trick for him.

I had the same issue. Took me a while to figure out why nothing was working.
I then found in /tmp/khd.err that khd could not find khd and kwmc.
Running launchctl config and setting the path seems to have helped.
I'm using zsh and khd + kwm installed with Homebrew into /usr/local/bin.

@cHoco lauching khd via my shell works fine, it's just lanchd that's having troubles.
Just out of curiosity what version of macOS are you guys on? I just recently updated to Sierra, and I wonder if that's why my launchd config is broken.

I am getting the same error in the logs.
having trouble with the syntax of this command
sudo launchctl config <user> <path> $PATH
what do i actually enter in the terminal?
sudo launchctl config scott /usr/local/bin $PATH

Replace <user> with your username and <path> with what you want $PATH to be set to.

@scottgrayson What was the actual command you used for launchctl? I can't seem to work it out

FWIW, having the same problem. I think it's safe to say it's a combination of the Sierra update and users running zsh.
Thanks @scottgrayson for the specific syntax. I think, just to be safe, you would want to include anything else potentially in your path?
sudo launchctl config user path $PATH

I think when I looked at the default path SET in launchctl it was the same as I pasted minus the /usr/local/bin, so I just added that. But yes, can be a better option to use the full PATH.

weird... for some reason switching to master from 1.1.0 from homebrew made it work for me.

I haven't updated to Sierra yet, so it might not be something specific to
the newest version.
On Fri, Oct 21, 2016 at 6:07 AM, Clayton Mullis<EMAIL_ADDRESS>wrote:

FWIW, having the same problem. I think it's safe to say it's a combination
of the Sierra update and users running zsh.
Thanks @scottgrayson https://github.com/scottgrayson for the specific
syntax. I think, just to be safe, you would want to include anything else
potentially in your path?
sudo launchctl config user path $PATH
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/koekeishiya/khd/issues/9#issuecomment-255282981, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABVkiIQyLTHAwhX6UiB0Sheu2AANaFuPks5q2CxcgaJpZM4KIZiW
.

--
MK

both bash -c "echo $PATH" and zsh -c "echo $PATH" show /usr/local/bin. But my stderr is showing that it can't find the commands. I used homebrew to install. I'm starting it via brew services start kwm and brew services start khd. If I call khd in my shell, it works as expected though, so khd & && disown is how I'm launching it for now. (A bit afraid of setting up the launchd until I have it configured how I want).
$which kwmc
/usr/local/bin/kwmc

Fwiw, I was having this issue also - khd and kwmc both weren't found using a bash shell. I'm on OS X El Capitan (10.11.6), setting $PATH in .profile. Fixed it with the launchctl solution (I made launchctl use my full path as suggested by @claymullis ). Thought my having the issue might contribute to finding the root of the problem since I'm not using zsh or on Sierra.

The fix from @nanzhong has been added to the brew formula.


mmap file

I added a memory mapped file option for the model file. Since the model files have to be copied out of the apk due to assets directory readonly status and inability to create the FileChannel directly, it is assumed that you have done so to internal storage with the same context that is passed into the overloaded constructor. Internal storage eliminates problems of SD card not being present or M runtime permissions requests but is also in danger of being full etc. Expansion files are the only other option, but this requires a download from the play store, which is an extra step. I therefore did the most simple thing.

Can one of the admins verify this patch?

@andrewharp The worry point is when to delete the file that was copied. Although this does not affect the buffer, obviously the mapped file won't be there forever either. Ideally, this heavy create operation involving the big graph file is done off heap avoiding IO exactly once and the classifier object persisted for further usage. Of course all of this is taken care of outside of the TFII in my work. I certainly could add copy and delete methods if you insist but would not want to encourage repeated copy, or for that matter repeated create. I would love to work on adding persistence and feel safe to copy/delete once and then null out the buffer.
I am fixing the typo and whitespace edits. Tell me what you think about the delete.

I am using the google standard template to reformat code of 2 indent, 4 continuation, 8 tab in AS.  Line 82 where it adds/deletes a line is an example that I cannot seem to get rid of.  If you know what is happening I would appreciate a suggestion as to get rid of that behavior.

@andrewharp  Did you review the changes?

@andrewharp I changed it to copy the file from res/raw to cache temporarily then map it and delete it the copy.  Even with the copy, this version takes the create to .6 seconds on average, from 1.6 seconds that I previously benchmarked using the assets/ and IO.  It requires that you pass in the model file without the pb extension as a  resource id and of course store the file in res/raw.

@yifeif  Hi: I changed my solution but it seems Andrew is busy. Is it possible to re-assign it?

@andrewharp this version works best in terms of average speed.

@yifeif  Can you give me a hint as to how I work this through with @andrewharp  to get it finished?  I don't know what he wants in addition to what I have changed.

@cloudbank Apologies for the delay, up against some deadlines internally.

@andrewharp NP.  I don't mind at all as long as you eventually get back to
me.  : )
Best regards,
Sabine G. Vogel
On Mon, Sep 25, 2017 at 3:16 PM, Andrew Harp<EMAIL_ADDRESS>wrote:

@cloudbank https://github.com/cloudbank Apologies for the delay, up
against some deadlines internally.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/12922#issuecomment-332029044,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACLqpqKr5HFcd_P9KnDzZ7d9YrGzQoIkks5smCY0gaJpZM4PRzkg
.

@andrewharp Do you have any estimate (next week?) when you will be able to finish this?

@cloudbank I still have reservations about limiting this to a res/raw implementation, would like the questions mentioned before clarified:

Does noCompressExtensions not solve the compression problem for assets/?
Why does it give you an OOM error when mapping from the cache dir, but not from assets/?

@andrewharp
-no compression either using gradle or bazel leads to OOM.  It is
unreasonable I think to not make every effort to compress such a large file
in a 100MB limit and kind of violates best practices.  I tried to map the
uncompressed file directly and ran out of memory right away.  I am building
the java and native libraries with bazel and then using them in a gradle
built app.  We want people to build stuff with it, right?
-using the res/raw and cache files dir doesn't run out of memory.  It might
run a little faster than using assets and cache since res are indexed.  I
will do a benchmark for both and get back to you.  If it is not a big
difference and you really want assets dir I can move it back there.
-I have not given up on trying to map the file without copying but

every SO thread, and advice about this says it is not possible from
expert level people including Pete Warden and Mark Murphy.
I have tried every reasonable approach without getting into a hack so
it might be best to start with this approach and if I, or someone else
stumbles upon a better solution, it can be improved.  Given the natural way
android is set up right now it seems improbable that copying can be avoided.

Best regards,
Sabine G. Vogel
On Wed, Sep 27, 2017 at 11:34 AM, Andrew Harp<EMAIL_ADDRESS>wrote:

@cloudbank https://github.com/cloudbank I still have reservations about
limiting this to a res/raw implementation, would like the questions
mentioned before clarified:

Does noCompressExtensions not solve the compression problem for
assets/?
Why does it give you an OOM error when mapping from the cache dir,
but not from assets/?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/12922#issuecomment-332613959,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACLqptmrY2axZzCVTdzDA96kJ1J_DCMWks5smpU2gaJpZM4PRzkg
.

@andrewharp I benchmarked it and the res/raw solution comes back on average in just under a second with a best time average around .5 seconds.  The assets solution never comes back any where near that fast on average, with around 1.2-1.3seconds.  Could you live with the idea that if you want to use the overloaded create that uses a memmap, you need to move your files over to res/raw?  Not that big a deal.  You also have to remove the file extension, and I could add a check for that.

@yifeif Can you give me a hint as to how I work this through with @andrewharp to get it finished? I don't know what he wants in addition to what I have changed.

@cloudbank we always try to look at the PRs as soon as possible, but sometimes we might also have internal deadlines to meet and can have some delay. Thanks again for your patience.
@andrewharp could you let @cloudbank know whenever you get a chance? Thanks a lot!

@cloudbank I'm not sure I understand why no compression leads to OOM. Where is the 100mb limit coming from? For a properly trained model, the entropy should be high enough that you won't get any significant compression gains anyway. For reference this is the unzip -v output for inception in tensorflow_demo.apk:
 Length   Method    Size  Cmpr    Date    Time   CRC-32   Name
--------  ------  ------- ---- ---------- ----- --------  ----
 3771239  Stored  3771239   0% 1980-01-01 00:00 9225f2cc  assets/conv_actions_frozen.pb
29083865  Defl:X 25434578  13% 1980-01-01 00:00 ac57f5c3  assets/ssd_mobilenet_v1_android_export.pb
  563897  Defl:X   257084  54% 1980-01-01 00:00 7d7f4ff3  assets/stylize_quantized.pb
53884595  Stored 53884595   0% 1980-01-01 00:00 6d3b8d15  assets/tensorflow_inception_graph.pb

Compression was not deliberately turned off here, zip just decided it wasn't worth it for 2/4 graphs. The savings from the other two are very minimal overall.
What I think might be ideal would be to have the res/raw path as you have it now, and also the assets/ path if the user has not chosen to compressed the pb file. Then the user automatically gets the improvement if they set things up the default way with assets (e.g. the default classifier demo will be improved), and get even more speed you've found if they take the time to move things to res/raw.

@andrewharp
I was just thinking to change it to detect compressed or not myself...great
idea. I will change it over the next few days.
Best regards,
Sabine G. Vogel
On Wed, Oct 11, 2017 at 1:16 PM, Andrew Harp<EMAIL_ADDRESS>wrote:

@cloudbank https://github.com/cloudbank I'm not sure I understand why
no compression leads to OOM. Where is the 100mb limit coming from? For a
properly trained model, the entropy should be high enough that you won't
get any significant compression gains anyway. For reference this is the
unzip -v output for inception in tensorflow_demo.apk:
Length   Method    Size  Cmpr    Date    Time   CRC-32   Name

3771239  Stored  3771239   0% 1980-01-01 00:00 9225f2cc  assets/conv_actions_frozen.pb
29083865  Defl:X 25434578  13% 1980-01-01 00:00 ac57f5c3  assets/ssd_mobilenet_v1_android_export.pb
563897  Defl:X   257084  54% 1980-01-01 00:00 7d7f4ff3  assets/stylize_quantized.pb
53884595  Stored 53884595   0% 1980-01-01 00:00 6d3b8d15  assets/tensorflow_inception_graph.pb
Compression was not deliberately turned off here, zip just decided it
wasn't worth it for 2/4 graphs. The savings from the other two are very
minimal overall.
What I think might be ideal would be to have the res/raw path as you have
it now, and also the assets/ path if the user has not chosen to compressed
the pb file. Then the user automatically gets the improvement if they set
things up the default way with assets (e.g. the default classifier demo
will be improved), and get even more speed you've found if they take the
time to move things to res/raw.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/12922#issuecomment-335933716,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACLqphn-f7Ie3UiuI1E-RHsH7bKFtUtJks5srSIKgaJpZM4PRzkg
.

@cloudbank any update?

working on it.....sorry have been busy with work...
I really want to finish it soon, so I will carve out some time ASAP.
Best regards,
Sabine G. Vogel
On Tue, Nov 7, 2017 at 1:00 PM, Martin Wicke<EMAIL_ADDRESS>wrote:

@cloudbank https://github.com/cloudbank any update?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/12922#issuecomment-342619505,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACLqpsG_lVruq5VnduIve4y4yIP3cHXtks5s0MTRgaJpZM4PRzkg
.

@andrewharp prior to Android 2.3, any compressed asset file with an uncompressed size of over 1 MB cannot be read from the assets, but then who uses prior to 2.3?  I am using gradle with compression and both my graphs are compressedly by 25 and 53 % in the apk.  I do not see any way to control whether or not the files are to be compressed---even if I do get into the build programmatically, obviously from your input it may not matter. So, I will check both places, and cross their fingers that mem map will open the file.

@andrewharp @martinwicke I had to upgrade bazel,ruby, AS 3.o, build tools... I reset my fork because I was so far behind on time and  I wanted a clean slate.  The new PR is  #14351

Thanks, closing this one.


Add code formatter into Makefile

Let's go here: https://github.com/basho/tools.mk

@ksauzz @kuenishi ping


Plugin 'UELlama' failed to load because module 'UELlama' could not be loaded

I'm having issues implementing the plugin. I've come from using the original UELlama plugin which worked but I had issues with packaging the project. I'm pretty novice and I'm a bit confused on things.
I'm running the Oculus source build UE5.3 on Windows and when I try to follow the instructions to build using cmake I get the error CMakeLists.txt doesn't exist.
So I build through the Visual Studio IDE but when I try to run UE I get:

Plugin 'UELlama' failed to load because module 'UELlama' could not be loaded.  There may be an operating system error or the module may not be properly set up.

If you're using this branch I recommend to first use the releases found here: https://github.com/getnamo/Llama-Unreal/releases/tag/v0.3.0 they contain the correct compiled dlls that should make it drag and drop to use in blueprint only. Use the .7z link.

So I tried that release, followed the instructions and just put it in the plugins folder, but still the same issue.
Is there anything else I could be missing?
Is it because my project is being built on the visual studio IDE?
I have tried building and rebuilding the project etc.
Have tried on a different PC with a different project, all the same error.

If you're building from source (Oculus5.3), you may possibly need to recompile the project with the plugin in it so it generates new binaries to match your custom engine

NB: if you're using the CUDA branch, you may be missing CUDA 12.2 runtimes. CPU-only should work though.

I am having same exac error.
My specs Windows 10 and GTX4080.
What I did is downloaded  https://github.com/getnamo/Llama-Unreal/releases/tag/v0.3.0
Added that to clean project UE5.3 Plugin Folder
And soon as I run project that error popup:

Plugin 'UELlama' failed to load because module 'UELlama' could not be loaded. There may be an operating system error or the module may not be properly set up.

Same issue here tried the same steps, let me know if there is a fix. For now I'm going to try implement some of the changes myself but would be great if this could get resolved.

Does the windows CPU only build work for anyone?

Does the windows CPU only build work for anyone?

Yeah I tried it, its an issue loading the module by the seems, it the moment in the process of extracting it from the module see if it can build then.

It's a static build so it's possible it has a hidden dll dependency that works on my system. Need to do a dll build for CPU and cuda and try again

What version of Cuda are you using, I have a feeling im not on the right version

What version of Cuda are you using, I have a feeling im not on the right version

Ive gotta go to work now :) Night shift. I will try the CPU one tomorrow ASAP. The best error I got so far is :
TLDR , dont think Im getting a .lib file somewhere
C:\Users\skyog\Documents\GitHub\ToTheMoonReLlama\Binaries\Win64\UnrealEditor-ToTheMoon.patch_1.lib and object C:\Users\skyog\Documents\GitHub\ToTheMoonReLlama\Binaries\Win64\UnrealEditor-ToTheMoon.patch_1.exp
llama.lib(ggml.obj) : error LNK2019: unresolved external symbol __imp_strdup referenced in function gguf_add_tensor
C:\Users\skyog\Documents\GitHub\ToTheMoonReLlama\Binaries\Win64\UnrealEditor-ToTheMoon.patch_1.exe : fatal error LNK1120: 1 unresolved externals.

Update : Got CPU working with a little work around :), took it out of the module into the main source files. @oivio @aleistor222 would you like the version I've modified. Going to do  the same with the GPU one most likely tomorrow evening Ill let you know if I can it to work.

Doing a bit of a refactor before I make another build. Will test that one on other pcs to debug what's failing on startup.

I've now cut it down to an issue in the Commons Folder, literally tested the build with and without it. And the issue only happens when those files are included.

Did a little bit of poking around. Found the Module loading issue is coming from the common.cpp and idented some of the functions that are causing it. Linked a video of me all the functions that are causing the issues.

These are the ones I found instantly but there could be some more. I haven't a clue why its happening but I hope it helps in any way.
llama_tokenize
llama_token_to_piece
llama_detokenize_spm
llama_detokenize_bpe
llama_should_add_bos_token

Did a little bit of poking around. Found the Module loading issue is coming from the common.cpp and idented some of the functions that are causing it. Linked a video of me all the functions that are causing the issues.
https://www.youtube.com/watch?v=v1Mr1am2Zp8
These are the ones I found instantly but there could be some more. I haven't a clue why its happening but I hope it helps in any way.
llama_tokenize
llama_token_to_piece
llama_detokenize_spm
llama_detokenize_bpe
llama_should_add_bos_token

Try https://github.com/getnamo/Llama-Unreal/releases/tag/v0.4.0 with the CPU only build to see if it works out of the box. NB this has the refactor so if you used blueprints off of the old api you'll need to re-wire those

v0.4.0 has the same error message when launching

yea, same for me. I can confirm that with log:
[2024.02.07-16.10.49:730][  0]LogWindows: Failed to load 'H:/Unreal Projects/Llama/Plugins/Llama-Unreal/Binaries/Win64/UnrealEditor-LlamaCore.dll' (GetLastError=1114)
[2024.02.07-16.10.49:730][  0]LogPluginManager: Error: Plugin 'Llama' failed to load because module 'LlamaCore' could not be loaded.  There may be an operating system error or the module may not be properly set up.
[2024.02.07-16.11.27:950][  0]Message dialog closed, result: Ok, title: Message, text: Plugin 'Llama' failed to load because module 'LlamaCore' could not be loaded.  There may be an operating system error or the module may not be properly set up.

I'm just in the process of using GFLAG to see what dll isn't getting loaded.

Just to confirm v0.4 is failing with cuda = false in build.cs for you guys?

Yeah Cuda is false again I think it links back to the changes to the functions now being in the commons folder. Idk it seems to me like it is missing a windows DLL file.

Ok for my investigating it isn't a windows DLL. On the call stack is it saying loading the module correct "LlamaCore.dll" that what the logs say I have a feeling it was from when you Refactored the name to Llamacore as v.2.4 it hadn't been done yet and its the version that I can get running. I'm currently looking into how the Plugin is set up.

Apparently building llama.cpp yourself locally can resolve this. Hints at static lib config maybe?

See https://github.com/getnamo/Llama-Unreal/pull/10 for an alternative path. Thanks to @ellllie-42 pr you can now specify a LLAMA_PATH and use your standard CUDA_PATH if you want a dev friendly custom build env. If those fail or you have local cuda libs it will default to local paths as before.
This doesn't solve portability problem of builds yet.

I built llama.cpp using the same settings as the cuda and cpu build on the read me and tested it on my system its working with cuda and I can use gpu offloading by itself. What do I need to copy to the plugin for it to work? I tried copying the cudart.lib cublas.lib and cuda.lib to the cuda folder,editing the build.cs to enable it, and replacing the ggml_static.lib and llama.lib with the one from my build/release folder but I am still getting the same error. I tried to rebuild the solution and deleting the binaries of the plugin but that didnt work either. I was looking at a unity version(llmunity) they use llamafile(https://github.com/Mozilla-Ocho/llamafile) would that help this plugin too?

i'm also having the same issue.

Did you use the tempfix associated with this issue?
On Tue, May 21, 2024, 4:42 PM SlySeanDaBomb @.***>
wrote:

i'm also having the same issue.
—
Reply to this email directly, view it on GitHub
https://github.com/getnamo/Llama-Unreal/issues/7#issuecomment-2123671382,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGGIIKOKZ3AINFVN2FXZ3W3ZDPSWBAVCNFSM6AAAAABBYJCDWSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRTGY3TCMZYGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Did you use the tempfix associated with this issue?
…
On Tue, May 21, 2024, 4:42 PM SlySeanDaBomb @.> wrote: i'm also having the same issue. — Reply to this email directly, view it on GitHub <#7 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/BGGIIKOKZ3AINFVN2FXZ3W3ZDPSWBAVCNFSM6AAAAABBYJCDWSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRTGY3TCMZYGI . You are receiving this because you were mentioned.Message ID: @.>

i didn't know there was one, where/what is it?

https://github.com/getnamo/Llama-Unreal/pull/10
On Tue, May 21, 2024, 5:34 PM SlySeanDaBomb @.***>
wrote:

Did you use the tempfix associated with this issue?
… <#m_-2477502457224046517_>
On Tue, May 21, 2024, 4:42 PM SlySeanDaBomb @.> wrote: i'm also having
the same issue. — Reply to this email directly, view it on GitHub <#7
(comment)
https://github.com/getnamo/Llama-Unreal/issues/7#issuecomment-2123671382>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGGIIKOKZ3AINFVN2FXZ3W3ZDPSWBAVCNFSM6AAAAABBYJCDWSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRTGY3TCMZYGI
https://github.com/notifications/unsubscribe-auth/BGGIIKOKZ3AINFVN2FXZ3W3ZDPSWBAVCNFSM6AAAAABBYJCDWSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRTGY3TCMZYGI
. You are receiving this because you were mentioned.Message ID: @.>
i didn't know there was one, where/what is it?
—
Reply to this email directly, view it on GitHub
https://github.com/getnamo/Llama-Unreal/issues/7#issuecomment-2123713052,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGGIIKPN56VEDJU7IGCCB33ZDPY27AVCNFSM6AAAAABBYJCDWSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRTG4YTGMBVGI
.
You are receiving this because you were mentioned.Message ID:
@.***>


An error occurred trying to clone ...

I just tried this framework and got this error during installation.
npm i -g adonis-cli
sam@DESKTOP-KQQ0A3T MINGW64 ~/lab
$ adonis new adonis
    _       _             _         _
   / \   __| | ___  _ __ (_)___    | |___
  / _ \ / _` |/ _ \| '_ \| / __|_  | / __|
 / ___ \ (_| | (_) | | | | \__ \ |_| \__ \
/_/   \_\__,_|\___/|_| |_|_|___/\___/|___/

√ Your current Node.js & npm version match the AdonisJs requirements!

× An error occured while trying to clone master version of adonisjs/adonis-app blueprint.
Command failed: git clone -b master --single-branch https://github.com/adonisjs/adonis-app.                                                                                git C:\Users\zorg\Google Drive\lab\adonis
fatal: Too many arguments.

usage: git clone [<options>] [--] <repo> [<dir>]

    -v, --verbose         be more verbose
    -q, --quiet           be more quiet
    --progress            force progress reporting
    -n, --no-checkout     don't create a checkout
    --bare                create a bare repository
    --mirror              create a mirror repository (implies bare)
    -l, --local           to clone from a local repository
    --no-hardlinks        don't use local hardlinks, always copy
    -s, --shared          setup as shared repository
    --recursive           initialize submodules in the clone
    --recurse-submodules  initialize submodules in the clone
    -j, --jobs <n>        number of submodules cloned in parallel
    --template <template-directory>
                          directory from which templates will be used
    --reference <repo>    reference repository
    --reference-if-able <repo>
                          reference repository
    --dissociate          use --reference only while cloning
    -o, --origin <name>   use <name> instead of 'origin' to track upstream
    -b, --branch <branch>
                          checkout <branch> instead of the remote's HEAD
    -u, --upload-pack <path>
                          path to git-upload-pack on the remote
    --depth <depth>       create a shallow clone of that depth
    --shallow-since <time>
                          create a shallow clone since a specific time
    --shallow-exclude <revision>
                          deepen history of shallow clone by excluding rev
    --single-branch       clone only one branch, HEAD or --branch
    --shallow-submodules  any cloned submodules will be shallow
    --separate-git-dir <gitdir>
                          separate git dir from working tree
    -c, --config <key=value>
                          set config inside the new repository
    -4, --ipv4            use IPv4 addresses only
    -6, --ipv6            use IPv6 addresses only

Can u share the version of Git on your machine?

$ git --version
git version 2.11.1.windows.1

Would u mind upgrading it to the latest version, this seems to be quite a old one

Thanks. I went with express yesterday.

You still should not close it. There is nothing worse than a library that does not work out of the box. I consider this important issue. Your library not to work because not (node, npm) but git version is not good

Yes since git is used to clone the repo. You can clone it manually and then do npm install.
Also I am less interested in taking weird feedback, even when I am trying to help you with the process. You should stick to express

I think this should be in bold... You should stick to express
If you don't want to use up to date tools it's not Adonis fault. 😉

@RomainLanz I use everything from python, php and node stack ... just fine with my current version of git.

Of course. 😉

just fine with my current version of git.

You just overreact when we just tell you that you should update your git version.


Add Door Interaction

Implement interaction with doors: doors that open and close when interacted with and locked doors that play a sound effect.

How doors interact with light should also be decided here.

As a small addition to this issue, add a trigger to re-enable the door the player comes through when they return to the house.


scenes

Hello,
Love the module. One question. Have you worked out the issue with scenes?

Scenes should be pretty solid at this point, have you run into an issue?

On Jul 15, 2017, at 1:52 PM, Chibale11<EMAIL_ADDRESS>wrote:
Hello,
Love the module. One question. Have you worked out the issue with scenes?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/mthyden/PhillipsHueSIMPLSharp/issues/3, or mute the thread https://github.com/notifications/unsubscribe-auth/AEluUspSifjDMn00tIASg8YWnObGBewJks5sOSaXgaJpZM4OZH5W.

	Yes using version 1.2 from git hub. Is that the most current?

	Afiba Edwards

On Thu, Jul 20, 2017 at 11:34 AM -0400, "Monte"<EMAIL_ADDRESS>wrote:
Scenes should be pretty solid at this point, have you run into an issue?

On Jul 15, 2017, at 1:52 PM, Chibale11<EMAIL_ADDRESS>wrote:

Hello,

Love the module. One question. Have you worked out the issue with scenes?

—

You are receiving this because you are subscribed to this thread.

Reply to this email directly, view it on GitHub https://github.com/mthyden/PhillipsHueSIMPLSharp/issues/3, or mute the thread https://github.com/notifications/unsubscribe-auth/AEluUspSifjDMn00tIASg8YWnObGBewJks5sOSaXgaJpZM4OZH5W.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

whats on github is the latest.

On Jul 20, 2017, at 8:36 AM, Chibale11<EMAIL_ADDRESS>wrote:
Yes using version 1.2 from git hub. Is that the most current?
Afiba Edwards
On Thu, Jul 20, 2017 at 11:34 AM -0400, "Monte"<EMAIL_ADDRESS>wrote:
Scenes should be pretty solid at this point, have you run into an issue?

On Jul 15, 2017, at 1:52 PM, Chibale11<EMAIL_ADDRESS>wrote:

Hello,

Love the module. One question. Have you worked out the issue with scenes?

—

You are receiving this because you are subscribed to this thread.

Reply to this email directly, view it on GitHub https://github.com/mthyden/PhillipsHueSIMPLSharp/issues/3, or mute the thread https://github.com/notifications/unsubscribe-auth/AEluUspSifjDMn00tIASg8YWnObGBewJks5sOSaXgaJpZM4OZH5W.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/mthyden/PhillipsHueSIMPLSharp/issues/3#issuecomment-316742287, or mute the thread https://github.com/notifications/unsubscribe-auth/AEluUoBMz-Vt5R00iX8j2lIplskSWzf9ks5sP3P-gaJpZM4OZH5W.

	Ok great I'll try again.

	Afiba Edwards

On Thu, Jul 20, 2017 at 11:39 AM -0400, "Monte"<EMAIL_ADDRESS>wrote:
whats on github is the latest.

On Jul 20, 2017, at 8:36 AM, Chibale11<EMAIL_ADDRESS>wrote:

Yes using version 1.2 from git hub. Is that the most current?

Afiba Edwards

On Thu, Jul 20, 2017 at 11:34 AM -0400, "Monte"<EMAIL_ADDRESS>wrote:

Scenes should be pretty solid at this point, have you run into an issue?

On Jul 15, 2017, at 1:52 PM, Chibale11<EMAIL_ADDRESS>wrote:

Hello,

Love the module. One question. Have you worked out the issue with scenes?

—

You are receiving this because you are subscribed to this thread.

Reply to this email directly, view it on GitHub https://github.com/mthyden/PhillipsHueSIMPLSharp/issues/3, or mute the thread https://github.com/notifications/unsubscribe-auth/AEluUspSifjDMn00tIASg8YWnObGBewJks5sOSaXgaJpZM4OZH5W.

—

You are receiving this because you authored the thread.

Reply to this email directly, view it on GitHub, or mute the thread.

—

You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/mthyden/PhillipsHueSIMPLSharp/issues/3#issuecomment-316742287, or mute the thread https://github.com/notifications/unsubscribe-auth/AEluUoBMz-Vt5R00iX8j2lIplskSWzf9ks5sP3P-gaJpZM4OZH5W.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

you get this working?

Have not had a chance to work on it. Major projects going on. Will update
you when I get to look at it again. ;Thanks so much for following up.
On Wed, Aug 9, 2017 at 11:20 AM, Monte<EMAIL_ADDRESS>wrote:

you get this working?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mthyden/PhillipsHueSIMPLSharp/issues/3#issuecomment-321288181,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AczcOiGnu_R3wIF_fDaV-4J8BYNZt2lZks5sWc41gaJpZM4OZH5W
.


Add test token collection.

Tokens for use for test purposes.

Codecov Report

Merging #231 into develop will increase coverage by 0.01%.
The diff coverage is 95.23%.

@@              Coverage Diff              @@
##             develop     #231      +/-   ##
=============================================
+ Coverage      88.39%   88.41%   +0.01%
+ Complexity      1886     1885       -1
=============================================
  Files            118      118
  Lines           6715     6724       +9
  Branches        1078     1073       -5
=============================================
+ Hits            5936     5945       +9
  Misses           715      715
  Partials          64       64


wg-runtime Status Update 2020-06-22

What was accomplished
Lessons learned
What's next

What was accomplished

Fixit!

Set and Map are now allowed
Fixed intrinsic layout in IE11
Stripped more code from ESM
Fixed future issues with SSR rendering amp-img

Deferring buildCallback merged, waiting for experiment to start
TLOU2

Lessons learned

Laptop cameras suck

What's next

Resuming work on Bento
Would like to help with ESM efforts
TLOU2

What was accomplished

intersect-resources going to 10% of prod this week
Make cross-browser tests on SL blocking again

Lessons learned

Browser disconnections are at least partly Karma's fault

What's next

Ramp intersect-resources to 100% if things look good

What was accomplished

Various gulp performance fixes

What's next

Restoring rrule support to amp-date-picker
IE11 integration tests for runtime
Running v0 in Strict mode


Improvements to help documentation content

Going through the help documentation pages again I made a few edits but also wanted to raise two issues for discussion:
https://civic.genome.wustl.edu/#/help
1.) On the evidence statement tab, the 'Text summary' entry in the table of components of an evidence statement and the introductory paragraph at the top are somewhat conflating these things: (a) the manually created evidence statement text for a single evidence item from a single publication, and (b) the synthesis of multiple evidence statements to create a variant summary, and to a lesser extent (c) the gene level summary.
In general I think we could do a better job of explaining the relationship between the components of a single evidence statement/item/record, the summary text for that single evidence statement, the variant summary that synthesizes N evidence statement for a single variant, and the gene summary that synthesizes across multiple variants of a gene.  I think maybe the table in this tab show be focused on the components of a single evidence statement only.  And actually labeling this table as 'evidence statement' is itself confusing.  Maybe better to describe this is an evidence item or record, where the statement (summary text) is but one component.
2.) I'm not sure about our use of the concept of journal impact factor in the documentation for Star Ratings.  Should we be publicly endorsing the use of impact factor in this way?  A lot of things influence what journal scientists get their papers published in.  I don't think impact factor of the journal is that well correlated with the quality of individual publications in the journal.  Expert scientists/clinicians should evaluate the evidence at face value. Maybe the impact factor of the journal is one of many factors they subjectively consider, hopefully a relatively minor factor, possibly so minor that it is not worth mentioning specifically in the official CIViC documentation.

I changed the Evidence Statement tab to Evidence Item (as that's the term we're using elsewhere in the app). It looks like the other concerns mentioned here have been addressed as well so I'll close this.


Ability to detect tailwindcss

Wappalyzer currently does not detect if a webpage is using tailwindcss. It would be great if it had this feature.

Hey mate could u send link on which site does the wappalyzer not find ?

Hey mate could u send link on which site does the wappalyzer not find ?

The following site uses tailwindcss but wappayzer does not detect it.

The following site uses tailwindcss but wappayzer does not detect it.

@Lilkedus Please provide a URL.


Unguarded array access using subscript operator can lead to segfault

In src/ocl/convolutionocl.cpp:1389 (and similarly line 1000):
            auto&& kernels = handle.GetKernels("miopenConvolutionFwdAlgoDirect", network_config);
#if(!defined(__GNUC__) || defined(__clang__)) // w/a for segfault in gcc 5.4.0
            const
#endif
                auto num_kernels = kernels.size();
            auto kernel          = kernels[0];

There needs to be a check that the num_kernels is not 0, otherwise accessing kernel[0] leads to a segfault. This check is present in a similar situation in line 3362 in the same file.
The reason why there was no kernel found in my case was probably because the handle had an improper value:
$21 = (miopen::Handle &) @0x7ffefdfe9c00: {<miopenHandle> = {<No data fields>}, impl = std::unique_ptr<miopen::HandleImpl> containing 0x7ffefdff43e0, geo_map = std::unordered_map with 0 elements}

However, there still needs to be a check to prevent the segfault and have a more graceful exit.

This problem will never occur if MIOpen is used in documented way - FindConvolution() and then Convolution(). We'll fix that, however when time permits.

@jithunnair-amd please try again with the latest MIOpen and report back.


'Error resetting password: Invalid token

Subject: 'Error resetting password: Invalid token
Description:
Attempted to reset my password, as i forgot it.
Results:
Cannot login, nor can i reset my password.
Backend version: develop-7c33e18d
Frontend version:
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Page: https://couchers.org/complete-password-reset?token=hteao-VoVuuaM-YQxzltllCCtaCBwFN3ocZcjblH3Ug=
User:

not enough info or can't reproduce


Cannot use v-on directives on

I would like to use the v-on directive on the images. Unfortunately, these get ignored after render.
Is there a way to make them work?
<carousel>             <img v-on:click="myClickMethod" src="https://placeimg.com/200/200/any?1">        </carousel>

Still not working


EPIC: Guided deposit (target experience)

Need to break this up
Story
As a new user
I want an easily understood way to deposit the right assets in order to trade
So that I feel confident in depositing the right assets and am ready to trade more quickly
Acceptance Criteria

[ ] There is a clear call to action to deposit from the wallet
[ ] When I have no funds in my wallet already, I can see an explanation of how to deposit i.e. I need to deposit funds from an Ethereum wallet
[ ] If I don't have an Ethereum wallet, there is a link to get one https://ethereum.org/en/wallets/find-
[ ] There is a stepped flow to connect my Ethereum wallet
[ ] I can see the types of supported Ethereum wallets
[ ] I can select one and it triggers the connection request from my wallet
[ ] There is a way link to Trade on Console if / when I have deposited funds

Upon successful connection

[ ] There is a way to see my connected key and disconnect
[ ] I can see a view of the funds in my Ethereum wallet
[ ] I can see my total balance per asset
[ ] I can see how to deposit each i.e. there is a call to action
[ ] There is a way to get more funds i.e. a link to uniswap
[ ] I can see which markets each of my assets is currently traded in on Console at a glance e.g. in a lozenge / tag
[ ] There is a way to go see each of these markets on Console e.g. they are linked
[ ] There is a way to see a full list of the assets currently being traded on Vega
[ ] When I select an asset for deposit, there is a way to first of all approve that asset for use in my Ethereum wallet
[ ] If I select an asset that is not currently traded on Vega, I am warned of this

Upon selecting to approve the asset

[ ] I can see visual feedback to go approve the asset in my Ethereum wallet
[ ] When I've approved, I can see visual feedback that my approval is being confirmed by the Ethereum network
[ ] Once confirmed, I can see visual feedback that I have approved deposits of this asset type and to what limit
[ ] There is a way to see the confirmation on Etherscan i.e. a link

Upon successful approval of the asset to deposit

[ ] There is a way to complete my deposit from here
[ ] I can enter the amount I want to deposit
[ ] I can see my total balance
[ ] I can easily toggle to a different asset to deposit
[ ] There is feedback if I try to deposit more than my balance
[ ] There is a way to easily deposit the maximum amount of that asset
[ ] There is a way to understand deposit limits i.e. a list of the remaining deposit allowance, Ethereum deposit cap and  link to https://docs.vega.xyz/mainnet/concepts/assets/deposits-withdrawals to understand more
[ ] There is a way to cancel the deposit, returning to the list of assets

Upon depositing

[ ] There is visual feedback through awaiting confirmation(s), processing deposit, to transaction confirmed

Tasks

[ ] Detailed sketch
[ ] Visual design
[ ] Frontend tasks:

[ ] Implement UI
[ ] Integrate with backend
[ ] Write unit tests

[ ] Backend tasks:

[ ] TBC
[ ] Write unit tests

[ ] Write end to end tests
[ ] Code review
[ ] Team verify acceptance criteria met
[ ] Exploratory QA testing
[ ] Capture bugs and new edge cases in tickets (this one or new TBD)
[ ] Write automated regression tests for the A/Cs (expanding on A/Cs where possible)
[ ] Add story and A/Cs to spec doc
[ ] Review with stakeholders
[ ] Release

Sketch
Latest wireframse:
https://www.figma.com/file/8SXfX86E1Qf6Gr15tykrDT/Guided-deposit-flow?type=design&node-id=31%3A357&mode=design&t=DmdT7ki6knHzIzHV-1
https://miro.com/app/board/uXjVM2wamp8=/?moveToWidget=3458764559544280107&cot=14

Design
This is how far I've gotten with the designs for this: Figma link ↳
Notes
Reference Uniswap for UX patterns.

Per onboarding prioritisation assigning this to @joellecharlotte now because @ckpringle says his diagram is now enough for the proper design to start.

SQUID


feat: update reactjs.org to react.dev improve blog banner image

Please describe the changes this PR makes and why it should be merged:
I updated every instance of https://reactj.org to https://react.dev, however, I LEFT the instance of the HOC(Higher Order Component) on the README.
Status

[ ] Code changes have been tested against prettier, or there are no code changes
[ ] I know how to update typings and have done so, or typings don't need updating

Semantic versioning classification:

[ ] This PR changes the codebase

[ ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)
[ ] This PR changes the internal workings with no modifications to the external API (bug fixes, performance improvements)

[x] This PR only includes non-code changes, like changes to documentation, README, etc.

Resolves issue #333

LGTM

LGTM, merging


Use min-max normalisation for heat stress

Closes #592

@Rosejoycrocker why do you reckon only heat_strees was hugely affected by this change? Any hypotesis?

@Rosejoycrocker why do you reckon only heat_strees was hugely affected by this change? Any hypotesis?

Wave stress was already min-max normalised, there were just indentation changes because of the new formatter- is that what you mean?

@Rosejoycrocker why do you reckon only heat_strees was hugely affected by this change? Any hypotesis?

Wave stress was already min-max normalised, there were just indentation changes because of the new formatter- is that what you mean?

Oh, of course. Yes, that's what I meant! :)

Hey @Zapiano, there's a typo in what I pushed, sorry. I'll fix it and reopen the PR

Hey @Zapiano, there's a typo in what I pushed, sorry. I'll fix it and reopen the PR

Hey, it's not just the typo, you also have to fix the order of the elements clause ? if_result : else_result


Religion Related books and some technolgy books

@gva-cse Please star the repository and make sure to follow me (https://github.com/Karamraj/).


Invalid value for output name field on validation

Hi all,
I'm currently updating the N2V model and noticed a problem with the validation.
This is the error I am getting
python -m pybio.spec verify-spec .\model.yaml {'outputs': {0: {'name': ['Invalid input.']}}}
This is the model file. Adding '' to the output name does not help.
`outputs:

name: activation_11/Identity
axes: byxc
data_type: float32
data_range: [-inf, inf]
halo: [0, 22, 22, 0]
shape:
offset: [0, 0, 0, 0]
scale: [1.0, 1.0, 1.0, 1.0]
reference_input: input
postprocessing:

name: scale_linear
kwargs:
axes: yxc
gain: [15374.962]
offset: [47944.324]`

I have transferred the issue because it relates to the validation that was moved here.

I suppose it is resolved, please reopen if needed.


Add Proxy Authorization Support

A new class is auth.py adds the Proxy-Authorization header to a Request object.
If a proxy is specified in the request, the send() method is going to look for authentication info in the proxy URL string and set up the proxy auth from there.
None of this works with out the changes to urllib3 in #293, however.

Thanks a ton for this!

(Great pull request btw — honored to be your first :)


ExplicitValueFinder

Hi, i was trying out your analyzer for detecting cryptograpfic misuse and wanted to use ExplicitValueFinder to check some other methods too so i did some testing and found that ExplicitValueFinder has problems detecting that i use ECB mode or constant IV when im passing them as arguments to the function.

Yes, the ExplicitValueFinder is for intra-procedural use only.
If you want to track inter-procedural values, use data-flow analysis.

Could u point me to the modules that can help me accomplish that?

This link might help you to understand how to retrieve values from data-flow analysis:
https://github.com/arguslab/Argus-SAF-playground/blob/master/src/main/scala/org/argus/play/random/RetriveStringFromSourceOrSink.scala

thanks you for providing the example, but how do i resolve defsite when its out of the scope of the method and the value is passed as an argument, for example if the url was passed as an argument to the get() method

The data flow is inter-procedural, so if its a string defined in another method and passed to this one, the context of defsite will be the creation site of such string.

Ye sorry. i thought defsite wasnt returning me the location. Im closing this, thanks.


Rule   int32 gorm:"column:rule;default:200"

Your Question
model define default:200, db.create(&m), Will automatically query once，
[2020-07-13 11:49:29]  [89.22ms]  SELECT rule FROM evfs_storage  WHERE (req_id = '0x3d207a1fd830d11c82e77b02db1def9a717d0da8a18ebcf53aa10c4f20cdbbb0')

V2 won't query that, V1 you can disable this by removing gorm:force_reload_after_create from create callbacks

github.com/go-gorm/gorm<EMAIL_ADDRESS>这个是2.0地址么

https://github.com/go-gorm/gorm/wiki/GORM-V2-Release-Note-Draft-CN


ViaField/ ViaFieldIndex does not propagate validation errors along the specified field path access

Expected Behavior
When using .ViaField for validation, it should propagate validation errors along the specified field path.
For example, I am expecting to see the error message containing invalid value * params[param1].value if validation is used ViaField/ViaFieldIndex:
errs = errs.Also(config.ValidateEnabledAPIFields(ctx, "object type parameter", config.BetaAPIFields).ViaFieldIndex("params", i))

*(Note that ViaFieldIndex has similar issue for usage since is just combining the fields with indexing)
Actual Behavior
The validation error has not been propagated as expected by ViaField(Task)
Steps to Reproduce the Problem
This has occurred in couple of validations in existing pipeline v0.52.x . For example at Pipeline Validation.

Deploy the v0.52.x pipeline
Run the existing Pipeline

apiVersion: tekton.dev/v1
kind: Pipeline
metadata:
  name: clone-cleanup-workspace
spec:
  workspaces:
    # common workspace where git repo is cloned and needs to be cleanup after done
    - name: git-source
  tasks:
    # Clone app repo to workspace
    - name: clone-app-repo
      taskRef:
        name: git-clone-from-catalog
      params:
        - name: url
          value: https://github.com/tektoncd/community.git
        - name: subdirectory
          value: application
      workspaces:
        - name: output
          workspace: git-source
  finally:
    # Cleanup workspace
    - name: cleanup
      taskSpec:
        params:
        - name: tests
        steps:
          - name: check-commit-initialized
            image: alpine
            script: |
              if [[ ! $(params.tests[1]) ]]; then
                exit 1
              fi
      params:
      - name: tests
        value: string-value
    - name: check-git-commit
      params:
        - name: commit
          value: $(tasks.clone-app-repo.results.commit)
      taskSpec:
        params:
          - name: commit
        steps:
          - name: check-commit-initialized
            image: alpine
            script: |
              if [[ ! $(params.commit) ]]; then
                exit 1
              fi

You will see:
Error from server (BadRequest): error when creating "examples/v1/pipelineruns/pipelinerun-with-final-tasks.yaml": admission webhook "validation.webhook.pipeline.tekton.dev" denied the request: validation failed: indexing into array parameters requires "enable-api-fields" feature gate to be "alpha" or "beta" but it is "stable":

instead of it starting with the field path spec.finally[0] for Pipeline validation.

[investigating] This shall either be a misuse of the ViaField or a corner case for config errors being covered.


Number of channels value is always equals 3

Hello!
I'm trying to initialize model like:
    model = Unet(input_shape=(25, 25, 1),                  classes=90,                  activation='softmax')
But I'm getting an error:

Traceback (most recent call last):
File "C:/Users/Daemon/PycharmProjects/atDNARelationshipDetector/train.py", line 90, in
model = build()
File "C:\Users\Daemon\PycharmProjects\atDNARelationshipDetector\mymodel.py", line 119, in build
activation='softmax')
File "C:\Users\Daemon\AppData\Local\Programs\Python\Python35\lib\site-packages\segmentation_models_init_.py", line 34, in wrapper
return func(*args, **kwargs)
File "C:\Users\Daemon\AppData\Local\Programs\Python\Python35\lib\site-packages\segmentation_models\models\unet.py", line 226, in Unet
**kwargs,
File "C:\Users\Daemon\AppData\Local\Programs\Python\Python35\lib\site-packages\segmentation_models\backbones\backbones_factory.py", line 103, in get_backbone
model = model_fn(*args, **kwargs)
File "C:\Users\Daemon\AppData\Local\Programs\Python\Python35\lib\site-packages\classification_models\models_factory.py", line 78, in wrapper
return func(*args, **new_kwargs)
File "C:\Users\Daemon\AppData\Local\Programs\Python\Python35\lib\site-packages\keras_applications\vgg16.py", line 99, in VGG16
weights=weights)
File "C:\Users\Daemon\AppData\Local\Programs\Python\Python35\lib\site-packages\keras_applications\imagenet_utils.py", line 316, in _obtain_input_shape
'input_shape=' + str(input_shape) + '')
ValueError: The input must have 3 channels; got input_shape=(25, 25, 1)
Process finished with exit code 1

It looks like number of channels value does nothing.

Hi, could you please, read more carefully readme and previous issues before starting a new one :)

input_shape H and W should be divisible by 32.
if you set input_shape not equal 3, provide encoder_weights = None

Thank you for answer!
I've misunderstood this row in docs:

just pass (None, None, C) to make your model be able to process images af any size, but H and W of input images should be divisible by factor 32.
and I thought, that it means "only if H and W are None, than input images must be divisible by factor 32, otherwise input images may be any size" :)


Adds Travis-CI build instructions

So that GitHub will autocompile PathVisio with each patch and report on that.

Looks like Denise already implemented this in the repo.