NoClassDefFoundError com.ericsson.research.owr.OwrBridge

I am trying to develop a Google Glass App using Open WebRTC.
I have imported openwebrtc.jar and openwebrtc_bridge.jar into my project:

Why do I get this error:
E/AndroidRuntime(7826): FATAL EXCEPTION: main
E/AndroidRuntime(7826): Process: *my package name*, PID: 7826
E/AndroidRuntime(7826): java.lang.NoClassDefFoundError: com.ericsson.research.owr.OwrBridge
E/AndroidRuntime(7826): 	at *my package name*.CallActivity.onCreate(CallActivity.java:26)
E/AndroidRuntime(7826): 	at android.app.Activity.performCreate(Activity.java:5235)
E/AndroidRuntime(7826): 	at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1089)
E/AndroidRuntime(7826): 	at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2188)
E/AndroidRuntime(7826): 	at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2273)
E/AndroidRuntime(7826): 	at android.app.ActivityThread.access$800(ActivityThread.java:138)
E/AndroidRuntime(7826): 	at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1236)
E/AndroidRuntime(7826): 	at android.os.Handler.dispatchMessage(Handler.java:102)
E/AndroidRuntime(7826): 	at android.os.Looper.loop(Looper.java:149)
E/AndroidRuntime(7826): 	at android.app.ActivityThread.main(ActivityThread.java:5045)
E/AndroidRuntime(7826): 	at java.lang.reflect.Method.invokeNative(Native Method)
E/AndroidRuntime(7826): 	at java.lang.reflect.Method.invoke(Method.java:515)
E/AndroidRuntime(7826): 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:786)
E/AndroidRuntime(7826): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:602)
E/AndroidRuntime(7826): 	at dalvik.system.NativeStart.main(Native Method)
11-10 14:02:04.433: I/Process(7826): Sending signal. PID: 7826 SIG: 9

Do I have to add more than just the two .jar-files?

It seems that @nbermudezs has got OpenWebRTC to run on Glass, albeit slowly: https://github.com/EricssonResearch/openwebrtc/issues/499
Maybe he has some pointers.

@bakedCouchPotato what is the purpose of openwebrtc-bridge.jar? I didn't have to include it for the app to work. I only included openwebrtc.jar

openwebrtc-bridge.jar is used if you want to add WebRTC support to an Android WebView. When creating an app for Google Glass I would suggest sticking with the native (Java) approach.

Unfortunately I need to use WebRTC.
I am porting a web app to glass and I used the WebRTC support of API level 21+ for smartphones.
I also tried removing the libraries from the Java Build Path and adding them to the /libs folder.
This is the error that I get in the console when I try to compile:
The library 'openwebrtc_bridge.jar' contains native libraries that will not run on the device.
Additionally some of those libraries will interfer with the installation of the application because of their location in lib/
lib/ is reserved for NDK libraries.
The following libraries were found:
lib/armeabi-v7a/libopenwebrtc_bridge_jni.so

You can still use WebRTC, just not in WebViews. If you have a web application you most likely need to rewrite the interface for it anyway, since websites are pretty ugly to use on Glass.
Uninstalling and reinstalling the app might solve that last issue, seems like there's an old .so lib left.

I completely removed the app and reinstalled it, but I still get this error.

OK, so how do I create native apps for with OWR?
For the desktop app I use http://simplewebrtc.com/
Is OWR compatiple to this?


Add thousands separator to the number of search results

Changes in this PR
This is a small change to make the number more easily readable for humans.
Screenshots of UI changes:
Before

After

Codecov Report

Merging #692 (c2979e1) into main (0ee2206) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #692   +/-   ##
=======================================
  Coverage   91.18%   91.18%
=======================================
  Files          24       24
  Lines        1066     1066
  Branches       55       55
=======================================
  Hits          972      972
  Misses         80       80
  Partials       14       14

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[Feature] Actionable notifications: foreground, background, destructive

PR Type
Added options for selectable items: foreground will allow to open application after select option. destructive make button in red color;

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation  changes
[ ] Other... Please describe:

PR Checklist
For bug fixes / features, please check if your PR fulfills the following requirements:

[x] Testing has been carried out for the changes have been added
[ ] Regression testing has been carried out for existing functionality
[ ] Docs have been added / updated

Does this PR introduce a breaking change?

[ ] Yes
[x] No

pn-actions.json example
{
  "PushNotificationActions": [
    {
      "category": "news",
      "actions": [
        {
          "id": "read", "title": "Read", "foreground": "true"
        },
        {
          "id": "skip", "title": "Skip", "destructive": "true"
        },
        {
          "id": "add", "title": "Add to list"
        }
      ]
    }
  ]
}

Last 3 commits a custom solution for one app and here suddenly pushed. But foreground and destructive mode still can be good cherrypicked

Thanks I have cherrypicked the relevant commit and pushed out in the 11.0.0 release


"Error: expected array with fields" when trace a "public virtual AsyncUnaryCall" method

(AsyncUnaryCall is from gRPC C#)
(All of them have two overloads like
public virtual AsyncUnaryCall<SystemAuthorizeV3.Types.Response> AuthorizeV3Async(SystemAuthorizeV3.Types.Request request, Metadata headers, Nullable<DateTime> deadline, CancellationToken cancellationToken){}
public virtual AsyncUnaryCall<SystemAuthorizeV3.Types.Response> AuthorizeV3Async(SystemAuthorizeV3.Types.Request request, CallOptions options){}

This error will be throwed when tracing (all of simpleTrace,fullTrace,fullWithValuesTrace) them.
Il2Cpp.Tracer.fullTrace(
    domain.assemblies["Takasho.ProtobufGenerated"].image.classes[
      "Takasho.Schema.CommonFeatureset.PlayerApi.System.SystemClient"
    ]
  );
Error: expected array with fields
    at NativeFunction.<anonymous> (<anonymous>)
    at Il2CppMethod.invokeRaw (node_modules/frida-il2cpp-bridge/dist/il2cpp/structs/method.js:104:1)
    at Il2CppMethod.<anonymous> (node_modules/frida-il2cpp-bridge/dist/il2cpp/structs/method.js:112:1)
    at Il2CppObject.domain.assemblies.Takasho.ProtobufGenerated.image.classes.Takasho.Schema.CommonFeatureset.PlayerApi.System.SystemClient.methods.RecordLoginHistoryV1Async_.implementation (agent/index.ts:83:52)
    at InvocationContext.replaceCallback (node_modules/frida-il2cpp-bridge/dist/il2cpp/structs/method.js:77:1)
    at NativeFunction.<anonymous> (<anonymous>)
    at Il2CppMethod.invokeRaw (node_modules/frida-il2cpp-bridge/dist/il2cpp/structs/method.js:104:1)
    at Il2CppMethod.<anonymous> (node_modules/frida-il2cpp-bridge/dist/il2cpp/structs/method.js:112:1)
    at Il2CppObject.method.implementation (node_modules/frida-il2cpp-bridge/dist/il2cpp/tracer.js:80:1)
    at InvocationContext.replaceCallback (node_modules/frida-il2cpp-bridge/dist/il2cpp/structs/method.js:77:1)

This error will cause a in-game exception.
But if i use
  domain.assemblies["Takasho.ProtobufGenerated"].image.classes[
    "Takasho.Schema.CommonFeatureset.PlayerApi.System.SystemClient"
  ].methods.AuthorizeV3Async.implementation = function (
    request: Il2Cpp.Object,
    b: Il2Cpp.Object,
    c: any,
    d: any
  ): any {
    console.log("IdToken: " + req.fields["idToken_"].value);
    return this.methods.AuthorizeV3Async.invoke(request, b, c, d);
  };

instead, it works fine.
So is this a bug or incorrect tracer usage? I trace the whole class to intercept methods sequence, it works fine on other classes without overloads and generics.

@EtherealAO
Hi, if it's an android app and that method is invoked at app startup (without requiring play time), I can definitely investigate. What's its name?

@EtherealAO
Hi, if it's an android app and that method is invoked at app startup (without requiring play time), I can definitely investigate. What's its name?

東方ダンマクカグラ,could be downloaded at https://apkpure.com/%E6%9D%B1%E6%96%B9%E3%83%80%E3%83%B3%E3%83%9E%E3%82%AF%E3%82%AB%E3%82%B0%E3%83%A9%EF%BC%88%E3%83%80%E3%83%B3%E3%82%AB%E3%82%B0%EF%BC%89/com.dena.a12026801
AuthorizeV3Async is called when try to log in, a Japan IP may be required.
You can try Takasho.Schema.CommonFeatureset.PlayerApi.OndemandMaster.OndemandMasterClient.GetEntriesV1Async instead, which invoked at app startup.

@EtherealAO
Uhm, the app is just a blank screen to me; I can only dump it. I looked at how those functions are "encoded" (e.g. fridaSignature) and they look good to me.
Woudl you add some prints here and see what happens?
https://github.com/vfsfitvnm/frida-il2cpp-bridge/blob/master/src/il2cpp/structs/method.ts#L148
https://github.com/vfsfitvnm/frida-il2cpp-bridge/blob/master/src/il2cpp/tracer.ts#L95

Oh, I managed to make it load to the main screen. All I have to do is to press the back button instead of "OK" when it warns me I don't have the google spyware framework installed.
I'll investigate soon

It should be fixed, this what caused the issue:
https://github.com/vfsfitvnm/frida-il2cpp-bridge/commit/1922ed3734743da8da9711f378338198a7a9f28a#diff-d3919f4c535e7fff1ed6c3dcb718c10b892bf5dfd6a660279dab5e4f19377720R43


Do not unnecessarily double-check directory creation

The mkdir endpoint will report success/failure without the need to double-check the directory got created
This cuts out a round-trip + removes the need to keep /verify updated as we add new features. /verify could also be rewritten to properly use stored procedures but since it's kind of useless anyway I think it's better to just phase it out and eventually remove it entirely

This fixes latch cp <local> latch://account/.../... btw


Removing tarballs from the repository.

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[X] The toolchain has been rebuilt successfully (or no changes were made to it)
[X] The toolchain/worker package manifests are up-to-date
[X] Any updated packages successfully build (or no packages were changed)
[X] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[X] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[X] All package sources are available
[X] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json)
[X] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[X] All source files have up-to-date hashes in the *.signatures.json files
[X] sudo make go-tidy-all and sudo make go-test-coverage pass
[X] Documentation has been updated to match any changes to the build system
[X] Ready to merge

Summary

Looks like accidentally we checked-in some compressed sources into our repo. Removing them now, they are available in the blob storage.

Does this affect the toolchain?

No.
Test Methodology

Checked that the removed tarballs are available in our blob storage.

I'm surprised these got in, we have SPECS/**/*.bz2 etc... in our .gitignore. Might be worth double checking that is working as intended.

Maybe room for a GitHub action to flag these at PR?

I'm surprised these got in, we have SPECS/**/*.bz2 etc... in our .gitignore. Might be worth double checking that is working as intended.

I read this too late, but that's a very good idea. I just checked what we ignore and it seems like we are limiting ourselves to the SPECS folder only. I'm going to fix that in a separate PR in a few minutes. Thanks!


Updated readme

This would result in master being updated with vNext_0_5_0. Do this once finished with v0.5.0


create get_status_details to send more details

Create a second end point in RulesDecision that sends back all the data for a given firefighterID and timestamp.
Sample call:
/get_status_details?firefighter_id=20f8b710-dd12-4349-80ac-79dd09c50e5d&timestamp_mins=2020-11-19%2018:49:00

Response:
{
  "timestamp_mins": "2020-11-19T18:49:00.000Z",
  "firefighter_id": "20f8b710-dd12-4349-80ac-79dd09c50e5d",
  "device_id": "upkar-nodejs:09:2F:CB:51:3F:AC",
  "device_battery_level": -47.96,
  "temperature": 27,
  "humidity": 71,
  "carbon_monoxide": 126.4,
  "nitrogen_dioxide": 8.36,
  "formaldehyde": 0.23,
  "acrolein": 7.04,
  "benzene": 0.45,
  "device_timestamp": "2020-11-19T18:49:25.000Z",
  "device_status_LED": null,
  "status": 1,
  "carbon_monoxide_twa_10min": 0.0,
  "carbon_monoxide_twa_30min": 0.0,
  "carbon_monoxide_twa_60min": 0.0,
  "carbon_monoxide_twa_240min": 0.0,
  "carbon_monoxide_twa_480min": 0.0,
  "carbon_monoxide_gauge_10min": 0,
  "carbon_monoxide_gauge_30min": 0,
  "carbon_monoxide_gauge_60min": 0,
  "carbon_monoxide_gauge_240min": 0,
  "carbon_monoxide_gauge_480min": 0,
  "nitrogen_dioxide_twa_10min": 0.0,
  "nitrogen_dioxide_twa_30min": 0.0,
  "nitrogen_dioxide_twa_60min": 0.0,
  "nitrogen_dioxide_twa_240min": 0.0,
  "nitrogen_dioxide_twa_480min": 0.0,
  "nitrogen_dioxide_gauge_10min": 0,
  "nitrogen_dioxide_gauge_30min": 0,
  "nitrogen_dioxide_gauge_60min": 0,
  "nitrogen_dioxide_gauge_240min": 0,
  "nitrogen_dioxide_gauge_480min": 0,
  "formaldehyde_twa_10min": null,
  "formaldehyde_twa_30min": null,
  "formaldehyde_twa_60min": null,
  "formaldehyde_twa_240min": null,
  "formaldehyde_twa_480min": null,
  "formaldehyde_gauge_10min": null,
  "formaldehyde_gauge_30min": null,
  "formaldehyde_gauge_60min": null,
  "formaldehyde_gauge_240min": null,
  "formaldehyde_gauge_480min": null,
  "acrolein_twa_10min": null,
  "acrolein_twa_30min": null,
  "acrolein_twa_60min": null,
  "acrolein_twa_240min": null,
  "acrolein_twa_480min": null,
  "acrolein_gauge_10min": null,
  "acrolein_gauge_30min": null,
  "acrolein_gauge_60min": null,
  "acrolein_gauge_240min": null,
  "acrolein_gauge_480min": null,
  "benzene_twa_10min": null,
  "benzene_twa_30min": null,
  "benzene_twa_60min": null,
  "benzene_twa_240min": null,
  "benzene_twa_480min": null,
  "benzene_gauge_10min": null,
  "benzene_gauge_30min": null,
  "benzene_gauge_60min": null,
  "benzene_gauge_240min": null,
  "benzene_gauge_480min": null
}

You'll need to encode a space in the API call for the timestamp @lidderupk

I copy pasted from the Postman UI. I have changed it. Thanks Dan.

Fixed with PR https://github.com/Call-for-Code/Prometeo-Rules-Decision/pull/24


Initializing nDataBackBuff later

Fixing potential problem:
Deleted “final” of  nDataBackBuff;
Initialized nDataBackBuff and dataPacketBuff instead in initSystem()
nDataBackBuff would not change accordingly to
sampling rate, because system mode is not initialized at line 121, so that getSampleRateSafe() would always return 250 no matter what sampling rate, and nDataBackBuff = 3*(int) getSampleRateSafe() would always return 750.

Thanks!!!

I have so many bugs to fix lol


Return separate function for module export

To prevent circular dependency issue in some situations, this returns a new function instead of the referenced function from get-set.js
Fixes #4756

This generates the makeGetSet every time so alternatively a reference could be stored and reused.

@TremayneChrist what's going on here? I don't quite understand.

Moment breaks when serving files through polymer serve.
The polymer serve server transpiles files on the fly, depending on which browser is loading them. For IE, it converts all ES6 code to ES5 and for some reason it fails on moment.
It seems to be something to do with the circular reference:
https://github.com/moment/moment/blob/2e2a5b35439665d4b0200143d808a7c26d6cd30f/src/lib/units/year.js#L1
https://github.com/moment/moment/blob/2e2a5b35439665d4b0200143d808a7c26d6cd30f/src/lib/moment/get-set.js#L6
If you remove year.js from get-set.js it also fixes the issue.

@marwahaha I wouldn't say this PR is the ideal fix as it creates a new function on every call. It's more of a pointer to the issue. The ideal fix would be to remove the circular imports.

@marwahaha maybe #4906 could be merge-worthy :)


Always on Top

Is your feature request related to a problem? Please describe.
Sometimes I feel like having the app on the top without hiding behind the other apps when switching.
This function can help in using the localsend over the other apps without resizing or splitting the apps to work in adjacent.
Describe the solution you'd like
Possibility of an option in settings to toggle the Always on Top function.
Describe alternatives you've considered
No response
Additional context
*this feature is for desktop version.

@bachig26 use PowerToys, it has the feature of making any app appear always on top.

Yes, PowerToys should be used.


make devenv release blocker

As quite a bit people now depend on devenv.sh, I'd like to make it a release blocker.
I commit to responding and addressing any build failures within 24h.

I plan to merge this in a few days.


update to 2.4.10-SNAPSHOT

Pull Request Test Coverage Report for Build 537

0 of 0   changed or added relevant lines in 0 files are covered.
65 unchanged lines in 16 files lost coverage.
Overall coverage decreased (-0.5%) to 64.996%

Files with Coverage Reduction
New Missed Lines
%

infra-sofa-boot-starter/src/main/java/com/alipay/sofa/infra/endpoint/SofaBootVersionEndpoint.java
1
81.94%

healthcheck-sofa-boot-starter/src/main/java/com/alipay/sofa/healthcheck/startup/ReadinessCheckProcessor.java
1
75.86%

isle-sofa-boot-starter/src/main/java/com/alipay/sofa/isle/spring/listener/SofaModuleContextRefreshedListener.java
1
72.41%

healthcheck-sofa-boot-starter/src/main/java/com/alipay/sofa/healthcheck/core/HealthIndicatorCheckProcessor.java
1
70.45%

runtime-sofa-boot-starter/src/main/java/com/alipay/sofa/runtime/spring/factory/ServiceFactoryBean.java
2
71.43%

runtime-sofa-boot-starter/src/main/java/com/alipay/sofa/runtime/service/helper/ReferenceRegisterHelper.java
2
76.19%

test-sofa-boot-starter/src/main/java/com/alipay/sofa/test/runner/SofaJUnit4Runner.java
3
56.67%

infra-sofa-boot-starter/src/main/java/com/alipay/sofa/infra/config/spring/namespace/handler/SofaBootNamespaceHandler.java
3
0.0%

healthcheck-sofa-boot-starter/src/main/java/com/alipay/sofa/healthcheck/core/AfterHealthCheckCallbackProcessor.java
4
60.87%

runtime-sofa-boot-starter/src/main/java/com/alipay/sofa/runtime/spring/factory/AbstractContractFactoryBean.java
4
52.78%

Totals

Change from base Build 529:
-0.5%

Covered Lines:
1465

Relevant Lines:
2254

💛  - Coveralls


Update deployment manifests and instructions

Deployment manifests and instructions are very outdated in the project since the community has been solely updating the kube-prometheus manifests. It would be great to leverage the efforts that were made there and update the manifests that we have in this repository.

@dgrisonnet, can you help me with getting started what i need to do? thanks

Sure, so the goal of this issue is to update the deployment manifests and instructions that we have in this repository under the deploy directory based on the more up-to-date manifests in kube-prometheus that are prefixed by prometheus-adapter- in the manifests directory.
The few pointers that I could give you are that you should be able to update all the manifests and bring the new ones except for the configmap and the apiservices that we can keep as is, except for a few selectors that might need to be updated to match the new changes.
Also, it would be great to move away from the custom-metrics-apiserver resource name to prometheus-adapter which is used in kube-prometheus and is in my opinion more clear.
As for the deployment instructions, we could move the second point out of the default deployment and add a new section explaining how users can configure the adapter and enable HTTPS.
Let me know if you need any additional information and feel free to reach out to me on slack if you need any help :slightly_smiling_face:

/remove-lifecycle rotten

/remove-lifecycle stale
/unassign championshuttler

/remove-lifecycle stale

This was fixed by https://github.com/kubernetes-sigs/prometheus-adapter/pull/531.


Changed pyclass name of PyArray and PyArrayIter #3612

Resolves #3612
The issue is

The name of array.array iterator has to be array_iterator but arrayiterator in RustPython.

but it is "The name of array.array iterator has to be arrayiterator but array_iterator in RustPython", right?
and I also changed also PyArray's name,
because it looks different from CPython's.

please check your git config user.email matches to your account. It is referring different account.


Diff chunks are not minimal

Monace Playground Repro

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.59.0
OS Version: Win 10

Steps to Reproduce:
bug1.r:
{
    if (A) {
        if (B) {
            doit
        }
    }
}
C
D

bug2.r:
{
    if (A && B) {
        doit
    }
}
C
E

Compare the two files:

Note how lines 6+7 are identical to lines 4+5 - there is no need for them to appear in the diff.
Change "E" to "D" in bug2.r and see the expected result:

Somehow, the second (D/E) diff chunk has an impact on the first diff chunk, which is not necessary at all.

Looks like duplicate of #84981

@IllusionMH well, other diff chunk issues have been solved quite quickly in the past, see https://github.com/microsoft/vscode/issues/119051 or https://github.com/microsoft/vscode/issues/121436.

/assign @alexdima (a person who resolved 2 previous)
OK, wasn't aware of that changes.

This is unfortunate, it has to do with how we compute diffs when trimming whitespace is not ignored: we actually compute the diff with trimming whitespace ignored and then introduce the char-level diffs for trimming whitespace. In this case, this backfires as there was a better } that could have been matched.

Fixed. Try the monaco playground link in the first issue with the latest dev!

@hediet I have two comments:

This screenshot seems to use ignoreTrimWhitespace: true. That is not what my issue was about, but I wonder if it was better if the original line 6 was deleted rather than original line 7, because the original line 7 is an exact match with the new line 5.

My original issue was about `ignoreTrimWhitespace: false, in which case I get this:

This looks pretty much like the snapshot I posted in the original comment. What was changed is that the CD/CE stuff does not impact the first chunk any longer - however, both cases (CD/CE and CD/CD) yield the non-minimal diff for the first chunk. So while the diff algorithm is more consistent now, its diffs tend to be longer than before. That is contrary of making them "minimal" ;)

I am checking out 1.82.0, and I would expect this fix to be in there. But it seems it is not:

Line 5 should not be part of the diff IMHO.

Can you check if you use the legacy diff algorithm?

Can you check in your settings if you use the legacy diff algorithm?

Do you mean diffEditor.diffAlgorithm? I have tried both options (legacy and advanced), and I don't see a difference between the two. Sometimes, diffEditor.ignoreTrimWhitespace does not seem to have an effect, either:

I will investigate this separately - maybe my profile is broken somehow (or maybe not, see below).
In a new Profile (using the advanced algorithm), I do see a difference depending on diffEditor.ignoreTrimWhitespace:

After settings the diff algorithm to legacy, the result is similar, although now the highlights are gone and diffEditor.ignoreTrimWhitespace has stopped working again:

This is the error I see in the console:

After setting the algorithm back to advanced and reloading, the error messages don't appear any more, but diffEditor.ignoreTrimWhitespace is still/again without effect.

TLDR: advanced algorithm, with whitespaces: new line 5 should not be part of the diff chunk because it is identical to old line 7.

Based on the Playground, I think this has been fixed in 0.42.0-dev-20230805 and is broken in 0.42.0-dev-20230804. Could it be that VS Code 1.82.0 ships with an older Monaco editor version?

1.82.0 ships with an older Monaco editor version?

That cannot be.

Thanks! This looks perfect!


Added urban dictionary support

what do you do if you want to define a phrase with more than one word?
It looks like !urban foo bar will look up foo and bar separately, but not foo bar.
Maybe it makes sense to only support one phrase per command.

good point - I'll probably change it so it supports multi-word entries but not looking up multiple words at once


migrate ws to 2.x

Currently the test 1 from server spec fails with:
(node:34693) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: Not running
    1) should return a new instance

  1 failing

  1) Server should return a new instance:
     Uncaught Error: connect ECONNREFUSED <IP_ADDRESS>:43010
      at Object.exports._errnoException (util.js:1022:11)
      at exports._exceptionWithHostPort (util.js:1045:20)
      at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1087:14)```

...or switch to uWebSockets #8.


Get Professor Matching data

We need the actual data for professor matching, not just the data we generate. Producing a better schedule (according to our fitness functions) than the schedule that CCIS actually produced will show the viability of this system for generating professor matchings.

We're not going to do professor matching. The problem is too complex and requires too much human back-and-forth because people aren't willing to enumerate their preferences, and the actual preferences are often too complex to be represented as objective functions.


[MacOS] Add YQ tool to macOS images(#5144)

#Description
Add YQ tool to MacOS images
Related issue:
Fixes: https://github.com/actions/virtual-environments/issues/5144
Related issue / work item is attached
Tests are written (if applicable)
Documentation is updated (if applicable)
Changes are tested and related VM images are successfully generated

/azp run macos1015, macos11, macos12


Merges Testing

Merges QOL improvements made by @ericgaspar and fixes upgrade script along with bumping version to ynh3
Package_check results

An automatic package_check will be launch at https://ci-apps-dev.yunohost.org/, when you add a specific comment to your Pull Request: "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!"*

!testme

I am incrementing the version to be able to test on my pi by using the upgrade command. This way I can ensure that the upgrade functionality works. I'll decrement it to ynh2 before merging.

Just use --force command to  force the upgrade:
sudo yunohost app upgrade komga -u https://github.com/YunoHost-Apps/komga_ynh/tree/testing --force

ahhhhhh, it is that simple 🤦♂

This app should install. For some rather curious reason, because of my YunoHost account registered with accentuated name, the app was gives me an error 500 ... When switching to an account where the registered name is not accented, the problem goes away.

!testme

This app should install. For some rather curious reason, because of my YunoHost account registered with accentuated name, the app was gives me an error 500 ... When switching to an account where the registered name is not accented, the problem goes away.

That is indeed weird. Are you able to login to komga when you are logged into YunoHost or is issue #5 just happening to me?

This app should install. For some rather curious reason, because of my YunoHost account registered with accentuated name, the app was gives me an error 500 ... When switching to an account where the registered name is not accented, the problem goes away.

That is indeed weird. Are you able to login to komga when you are logged into YunoHost or is issue #5 just happening to me?

I did get the same issue with Firefox but the registration process went through with Chromium. I don't know what the problem is ...

This app should install. For some rather curious reason, because of my YunoHost account registered with accentuated name, the app was gives me an error 500 ... When switching to an account where the registered name is not accented, the problem goes away.

That is indeed weird. Are you able to login to komga when you are logged into YunoHost or is issue #5 just happening to me?

I did get the same issue with Firefox but the registration process went through with Chromium. I don't know what the problem is ...

Are we passing any basic auth headers for logged in users? I disabled SSOwat injection on my local installation yet the problem still persists. The weird thing is that its working flawlessly for logged out use-cases (from registrations to logging in )

You are right, for me it worked with Chromium because I wasn't logged in... maybe ping @kay0u who knows permissions better

https://github.com/YunoHost-Apps/komga_ynh/pull/7 should fix the issue

Thanks @kay0u merging them

I installed komga on my server without problems so far. Thanks @ kay0u for the help

😜

@ericgaspar the tests are failing :/ It is throwing 502 bad gateway which I assume is because it is not able to fully launch before the test checks if the url is accessible
The service komga didn't fully executed the action start before the timeout.

Can we change the timeout duration?

I think we can merge this into master because everything is working even though the automated tests are failing

You could use --timeout=60 : https://github.com/YunoHost-Apps/etherpad_mypads_ynh/blob/8c8f0c9aff78c6a376a581d4323497135b05900a/scripts/install#L331
or sleep 60

It also means  that --line_match option in not finding anything that matches

!testme

!testme

Merging PR, tests are still failing though :/
Will create new PR for tests


[{ 	"resource": "/c:/Users/HP/.vscode/extensions/github.copilot-1.89.156/package.json", 	"owner": "extension-editing", 	"severity": 8, 	"message": "This proposal cannot be used because for this extension the product defines a fixed set of API proposals. You can test your extension but before publishing you MUST reach out to the VS Code team.", 	"startLineNumber": 78, 	"startColumn": 3, 	"endLineNumber": 78, 	"endColumn": 27 }]

Type: Bug
[{
"resource": "/c:/Users/HP/.vscode/extensions/github.copilot-1.89.156/package.json",
"owner": "extension-editing",
"severity": 8,
"message": "This proposal cannot be used because for this extension the product defines a fixed set of API proposals.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Update static import to support Django 3

@stvbdn this should be superceded by #8

Thanks @stvbdn I think this has been fixed in https://github.com/kevinhowbrook/wagtailguide/commit/03a3dbf6606104e89818910296a24f0d757f31bd so closing for now


CRASHLOOPBACKOFF kubernetes pod error

**Yaml file**
apiVersion: v1
kind: Pod
metadata:
name: application-server
labels:
env: test
server: tomcat
spec:
containers:

name: gameoflife
image: karthik4u619/karthik4u619:gameoflife
ports:

containerPort: 8080

**DOCKERFILE** working in container as docker file not in pod as yaml.
FROM tomcat:8.0
LABEL Author="Karthik Kalva"
ADD https://auditbucket1.s3-us-west-2.amazonaws.com/gameoflife/gameoflife.war /usr/local/tomcat/webapps
EXPOSE 8080
CMD [ "catalina.sh","run" ]
**--ERROR MSG --**

It might be due to network setup issue in your machine try to run this file in aks or eks or try to reinstall kubeadm again , your file seems to look good and the issue might be in setup


Single line break breaks layout

A single line break \n somehow breaks the layout leading to very strange spaces. Expected behavior would be that it is just ignored.

Interesting!… Did you dive a little bit into the code to see what could possibly be the root caus of this behavior?

Hello,
Is anybody trying to add a solution in the code for this? I did not see any related commits in master or next. We get this and this is very problematic for our users:

Thank you for any feedback!

@afilp
I just manually added a method to convert single \n to double like this.
addLineBreak(content) {
        const newContent = newContent.replace(/\n/g, (m) => {
            return m + m;
        });
        return newContent;
  }

<Markdown>
                {this.addLineBreak(content)}
 </Markdown>

I guess It is the easiest way to handle this problem for now.

this happened to me in latest version
using 2 \n solved:
\n\n

@sibelius We get this data from a server fetch, so we cannot write them differently. We can only do a post-fetch replace, as @khsily proposed.
But, couldn't this problem be solved in the library level instead? So that end users like us do not have to think about it? Unfortunately, I cannot contribute to that because I do not know the internals well.
Thanks anyway!

Is there any updates with this issue?


Factor the elaborator into a separate refiner

I am still not convinced by the design of LHS, but do not have a counter-proposal, either.


Use Connect instead of gRPC

This PR swaps gRPC for Connect in both the server and CLI implementations.  For compatibility with running instances of the Perseus service, we enable the gRPC wire protocol for the client.  We have also replaced grpc-gateway with Vanguard (connectrpc.com/vanguard) for JSON/REST support in the server's API.
I have done compatibility testing using an old (gRPC) client hitting a new (Connect) server, a new client hitting an old server, and a new client hitting a new server.
The generated metrics are slightly different due to differences between the gRPC and Connect runtimes, but the things being monitored are the same.

Ran git pull --rebase origin main to sync with unrelated Dependabot updates and force-pushed


Update version: DarkfullDante.srtAlign version 0.6.1

Pull request has been created with WinGet Automation 🪟📦🤖
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978335226 2024-01-15 14:33:43.152 [CLI ] ShellExecute installer failed:<PHONE_NUMBER>

(Automated response - build 575.)

This package appears to require ASP.NET 8.0 Core or the .NET SDK 8.0.1:

Manual Validation ended with:

10:53:55 AM: Description: A .NET Core application failed. Application: srt-align.exe Path: C:\Users\User\AppData\Local\Temp\WinGet\DarkfullDante.srtAlign.0.6.1\srt-align.exe Message: You must install or update .NET to run this application.  App: C:\Users\User\AppData\Local\Temp\WinGet\DarkfullDante.srtAlign.0.6.1\srt-align.exe Architecture: x64 Framework: 'Microsoft.NETCore.App', version '3.1.0' (x64) .NET location: C:\Program Files\dotnet\  The following frameworks were found:   8.0.1 at [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]  Learn more: https://aka.ms/dotnet/app-launch-failed  To install missing framework, download: https://aka.ms/dotnet-core-applaunch?framework=Microsoft.NETCore.App&framework_version=3.1.0&arch=x64&rid=win-x64&os=win10

(Automated response - build 586.)

With the dependency, this package appears to be a DLL and 2 JSON files:

Automatic Validation ended with:

2024-01-18T20:02:34.3174763Z ##[error] One or more errors occurred. (Manifest is invalid.) Manifest is invalid. (Line: 11, Col: 1, Idx: 308) - (Line: 11, Col: 2, Idx: 309): While parsing a block mapping, did not find expected key.

(Automated response - build 615.)

@wingetbot waivers Add Validation-No-Executables

With the dependency, this package appears to be a DLL and 2 JSON files:


[mesheryctl] mesheryctl system context view <context-name>

See mesheryctl Command Tracker for current status of commands.
Related issue #2829
Current Behavior

Users can view the current-context as shown below-
❯ mesheryctl system context view
Current context: local

endpoint: http://localhost:9081
token: Default
platform: kubernetes
adapters:
- meshery-istio
channel: stable
version: latest

Desired Behavior

Users should also be able to view contexts that are not the current-context by their names-
❯ mesheryctl system context view bob
Current context: bob

endpoint: http://localhost:9081
token: Default
platform: docker
adapters:
- meshery-linkerd
channel: stable
version: latest

Contributor Resources

mesheryctl Contributing Guide
Beginner's guide to contributing to Meshery and mesheryctl
mesheryctl Command Tracker
Meshery CLI Commands and Documentation

@navendu-pottekkat  : let me take this up.

Fixed in #2880


feat: 63 user story client implement getclient

Description:
Commit Message
feat: implement get client query handler

see the guidelines for commit messages.
Checklist:

[x] I have performed a self-review of my own code
[x] I have documented my code in accordance with the documentation guidelines
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have run the unit tests
[ ] I only have one commit (if not, squash them into one commit).
[x] I have a descriptive commit message that adheres to the commit message guidelines

Please review the CONTRIBUTING.md file for detailed contributing guidelines.

Codecov Report

Merging #198 (00634b9) into main (6903734) will increase coverage by 1.67%.
The diff coverage is 98.84%.

@@            Coverage Diff             @@
##             main     #198      +/-   ##
==========================================
+ Coverage   54.86%   56.54%   +1.67%
==========================================
  Files          68       68
  Lines        6140     6292     +152
==========================================
+ Hits         3369     3558     +189
+ Misses       2771     2734      -37

Impacted Files
Coverage Δ

contracts/cosmwasm-vm/cw-ibc-core/src/lib.rs
3.22% <ø> (ø)

...osmwasm-vm/cw-ibc-core/src/ics02_client/handler.rs
95.21% <98.54%> (+9.12%)
:arrow_up:

...cosmwasm-vm/cw-ibc-core/src/ics02_client/client.rs
73.92% <100.00%> (+4.66%)
:arrow_up:

...cosmwasm-vm/cw-ibc-core/src/ics02_client/events.rs
100.00% <100.00%> (ø)

.../cosmwasm-vm/cw-ibc-core/src/ics02_client/types.rs
44.96% <100.00%> (+4.77%)
:arrow_up:

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Set default CI timeout

To avoid jobs running continuously if there's a problem, like this.
(I think the default is 24h).

Actually meant this for ruby-lsp, but we may as well do it here too.


fix: round columns percentage stats to 9th decimal

fixes #3431

Thanks for the PR @esciara, can you please add a test for this?

Was looking into it... now sure how to do it. can you point me to somewhere?

Actually I was more looking on tests/core/test_table_diff.py , since the fix is integration agnostic...

@georgesittas is the test I added (or rather changed) good enough?

This looks fine, if I understand correctly its just for display purposes and not related to the existing --decimals option


Use smartcrop for better image cropping

https://github.com/muesli/smartcrop/blob/master/README.md

This seems like a super crazy thing to do cross-compilation with, so I'm going to remove this issue for now.


onImpossible

Hi, Nice plugin! Thanks for your work.  Can you provide more details for onImpossible callback? Why it calls and what need to do to fix showing pdf doc? I have tired to use it on android.

file not found, unkwnown file extension / mime type, etc.
try to explore the code with a java script debugger, which is very easy on android with chrome.

Thanks Felix. Do i need to copy pdf files to specific folder? Will they work if they stores in random folder inside app?

I mean is it ok if i use relative urls to docs - ./static/pdfs/58a86e84f9bf1618040000b7.pdf

Always use absolute urls as supported by the cordova-file-plugin.

Thanks!


Tensor(randomUniform: ...) does not behave as documented

The documentation for Tensor<Float>(randomUniform:lowerBound:upperBound:seed:) says

Creates a tensor with the specified shape, randomly sampling scalar values from a uniform distribution between lowerBound and upperBound.

However, I have found that the shape only matches the one specified when the bounds are 0D tensors. The documentation makes no mention of this being a precondition.
The equivalent Python function has the same restriction, so perhaps this is the expected behaviour and the documentation just needs to be amended to specify it.
I had hoped that using a tensor of bounds values would produce a tensor where each value was within the corresponding bounds. That is, Tensor<Float>(randomUniform: [3], lowerBound: [0, 1, 2], upperBound: [1, 2, 3])  would give a random tensor [0..<1, 1..<2, 2..<3]. My actual use case is 3D rather than 1D.
The desired behaviour can be achieved pretty easily.
let lb = Tensor<Float>([0, 1, 2])
let ub = Tensor<Float>([1, 2, 3])
let random = lb + (ub - lb) * Tensor(randomUniform: [3])

I would suggest that ideally this should behave as I'd like it to, but if not the documentation needs to be amended to make the preconditions clear, and that the code should also have preconditions, which would make this another item to add to the list in #517.

I had hoped that using a tensor of bounds values would produce a tensor where each value was within the corresponding bounds.

Does using scalar lower/upper bounds solve your use case?
import TensorFlow
print(Tensor(randomUniform: [3, 3], lowerBound: Tensor(-1), upperBound: Tensor(1)))
// [[  0.8018838141398685,   0.5886823980640088,  -0.9940630515487747],
//  [  0.9104622604107688,  -0.2710584578836728,  0.08515388234458765],
//  [-0.31065717882801147,  -0.7031103329154349,  -0.8709943926625416]]

I hadn't even thought about non-scalar Tensor lower/upper bounds before, that seems like an advanced use case.

No it doesn't, I specifically need these individual ranges. I've solved it using the generate in 0..<1 then scale and shift technique for now, so this isn't blocking me, but it's a bit clunky.
This probably is a pretty unusual use case. I'm training against generated data, and this is part of my generation step. I first randomly generate some sample ranges, then have to randomly generate some points within those ranges, and the ranges are all different sizes.
I've got quite a lot of advanced scenarios in this project, with things like random access tensor based spline integration going on.
It would be nice to have an overload that takes Float instead of the 0D tensor in the scalar case too.

As it's an unusual case, I thought an example might be interesting, so this is a simplified version of my actual code.
// Demo, actual values come from config
let containerLowerBound: Float = -512
let containerUpperBound: Float = 512
let containerSize = containerUpperBound - containerLowerBound
let minRegionLength: Float = 16
let margin: Float = 16
let batchSize = Int(pow(2, 10))
let regionCount = Int(pow(2, 10))

// Generate uniform region lengths that all fit within the container, allowing a margin.
let regionLengths = Tensor<Float>(
  randomUniform: [batchSize, regionCount],
  lowerBound: Tensor(minRegionLength),
  upperBound: Tensor(containerSize - 2 * margin)
)

// Generate uniform region start positions such that the region fits within the container, allowing a margin.
// Coordinate space local to container.
let regionStartPositions = margin + ((containerSize - 2 * margin) - regionLengths) * Tensor<Float>(
  randomUniform: [batchSize, regionCount]
)

I would like that last bit to be able to be this instead.
// Generate uniform region start positions such that the region fits within the container, allowing a margin.
// Coordinate space local to container.
let regionStartPositions = Tensor<Float>(
  randomUniform: [batchSize, regionCount],
  lowerBound: Tensor(margin),
  upperBound: (containerSize - margin) - regionLengths
)

That's rewritten from my real code and untested, so there might be the odd little maths mistake in there, but the idea should be clear.


Develop CRC-16 Decoder Python Simulation

Create a CRC-16 algorithm in python that accepts four bytes of data (8 nibbles, 32 bits, etc) and outputs a new two byte (4 nibbles, 16 bits, etc) word of data with the message bits and a flag with the CRC decoder result. Once completed create python test functions (that use the assert keyword) to verify that the algorithm works.


1.1 Adding a User

As an admin, I want to be able to create accounts for users so users can access the dashboard.
Story points: 6
Priority:  1 (high)
Risk: Low
Child of: #8
Tasks:

[x] Create a frontend interface with fields to fill (username, email, password, role, name, DOB). (1 points)
#34
[x] Design a database schema for a user (1 points)
#35
[x] Create a backend API endpoint for account creation (2 points)
#36
[x] Trigger API call on form submission (1 point)
#37
[x] Assure error handling in the backend with error codes and message (1 points)
#38

Tests:

Unit testing: Test form validation logic and form submission
Integration testing: Test success and failure from form submission to database insertion

Add user completed

Demo of feature:
https://github.com/user-attachments/assets/2cbfcc2c-d282-406d-aae0-e8bd7547ab28


Ide highlighting

Hello,
It is not an issue, in fact, just a quality improvement. when I started using Laravel fastPaginate the idle (Phpstorm) in my case highlights my code when I use $uqery->fastPaginate() although using idle helper which is Laravel Idea.

You can use IDE-Helper package which can help.

I already using Laravel Idea which is for that.

I'm not sure how to fix this! Happy to change anything here if I need to, feel free to reopen.


fix: storybook requires postcss8 when using tailwindcss

Reopened pull request:
Used this (https://storybook.js.org/addons/@storybook/addon-postcss) guide to enforce storybook to use postcss8, as required by PostCSS plugin tailwindcss.
Fixes https://github.com/tremorlabs/tremor/issues/172

thanks for the work @jelleag, this should be fixed in the v2.


Resnet: reproducing basic performance on Fiji

I copied the resnet onnx files
https://s3.amazonaws.com/onnx-model-zoo/resnet/resnet50v2/resnet50v2.onnx
https://s3.amazonaws.com/onnx-model-zoo/resnet/resnet50v1/resnet50v1.onnx
Let's start with resnet v2
+ BASE=/opt/rocm/mivisionx/model_compiler/python
+ python3 /opt/rocm/mivisionx/model_compiler/python/onnx_to_nnir.py resnet50v2.onnx Resnet-nnir --input-dims 1,3,224,224
loading ONNX model from resnet50v2.onnx ...
converting to IR model in Resnet-nnir ...
Traceback (most recent call last):
  File "/opt/rocm/mivisionx/model_compiler/python/onnx_to_nnir.py", line 210, in <module>
    main()
  File "/opt/rocm/mivisionx/model_compiler/python/onnx_to_nnir.py", line 207, in main
    onnx2ir(onnx_model_proto, outputFolder)
  File "/opt/rocm/mivisionx/model_compiler/python/onnx_to_nnir.py", line 191, in onnx2ir
    graph = onnx_graph_to_ir_graph(onnx_model.graph)
  File "/opt/rocm/mivisionx/model_compiler/python/onnx_to_nnir.py", line 180, in onnx_graph_to_ir_graph
    graph.updateLocals()
  File "/opt/rocm/mivisionx/model_compiler/python/nnir.py", line 407, in updateLocals
    raise ValueError("reshape: mismatch detected: " + node.inputs[0] + ":" + str(input.shape) + " " + node.outputs[0] + ":" + str(param))
ValueError: reshape: mismatch detected: resnetv24_pool1_fwd:[1, 2048, 1, 1] resnetv24_flatten0_reshape0:[1, 2, 1, 1]

any help ?
Then resnet v1
`+ BASE=/opt/rocm/mivisionx/model_compiler/python

python3 /opt/rocm/mivisionx/model_compiler/python/onnx_to_nnir.py resnet50v1.onnx Resnet-nnir
loading ONNX model from resnet50v1.onnx ...
converting to IR model in Resnet-nnir ...
OK: creating IR description in Resnet-nnir/graph.nnir ...
OK: creating IR binaries in Resnet-nnir/binary ...
python /opt/rocm/mivisionx/model_compiler/python/nnir_to_openvx.py Resnet-nnir Resnet-openVXModel
reading IR model from Resnet-nnir ...
OK: reading IR description from Resnet-nnir/graph.nnir ...
OK: reading IR binaries from Resnet-nnir/binary ...
#OUTPUT-TENSOR: resnetv17_dense0_fwd 1 1000 1 1
creating C code in Resnet-openVXModel ...
creating Resnet-openVXModel/CMakeLists.txt ...
creating Resnet-openVXModel/cmake/FindOpenCL.cmake ...
creating Resnet-openVXModel/annmodule.h ...
creating Resnet-openVXModel/annmodule.cpp ...
creating Resnet-openVXModel/weights.bin ...
creating Resnet-openVXModel/anntest.cpp ...
creating Resnet-openVXModel/annpython.h ...
creating Resnet-openVXModel/annpython.cpp ...
creating Resnet-openVXModel/anntest.py ...
cd Resnet-openVXModel
mkdir -p build
cd build
cmake ..
-- Configuring done
-- Generating done
-- Build files have been written to: /home/paolo/FastMM/MIVisionX/ONNX/resnet50v1/Resnet-openVXModel/build
make
Scanning dependencies of target annmodule
[ 16%] Building CXX object CMakeFiles/annmodule.dir/annmodule.cpp.o
[ 33%] Linking CXX shared library libannmodule.so
[ 33%] Built target annmodule
Scanning dependencies of target annpython
[ 50%] Building CXX object CMakeFiles/annpython.dir/annpython.cpp.o
[ 66%] Linking CXX shared library libannpython.so
[ 66%] Built target annpython
Scanning dependencies of target anntest
[ 83%] Building CXX object CMakeFiles/anntest.dir/anntest.cpp.o
[100%] Linking CXX executable anntest
[100%] Built target anntest
`
Then I execute the test in two formats

./build/anntest weights.bin  OK: loaded 26 kernels from libvx_nn.so ERROR: invalid data (magic,size)=(0xf00dd1e1,0) in weights.bin at byte position 4 -- expected size is 37632 ERROR: failed with status = (-1) at /home/paolo/FastMM/MIVisionX/ONNX/resnet50v1/Resnet-openVXModel/annmodule.cpp#993 ERROR: annAddToGraph() failed (-1)
and
python anntest.py ./build/libannpython.so  weights.bin  ~/Downloads/balls.jpeg out  OK: AnnAPI found "input,data,1,3,224,224;output0,resnetv17_dense0_fwd,1,1000;" as configuration in ./build/libannpython.so OK: loaded 26 kernels from libvx_nn.so ERROR: annAddToGraph: failed (-1) ERROR: annCopyToInferenceInput: invalid handle INFO: annCopyToInferenceInput status -1 ERROR: annRunInference: invalid handle INFO: annRunInference status -1  Traceback (most recent call last):   File "anntest.py", line 74, in <module>     output,name,n,c,h,w = output_info.split(',') ValueError: need more than 4 values to unpack

I do not do any optimization ... I will ask about optimization shortly (fuse opt does not work for resnet or squeezenet, which is in the tutorial).
Let me know how I can help ... thanks

Another test: googlenet
https://s3.amazonaws.com/download.onnx/models/opset_9/bvlc_googlenet.tar.gz
compiled using
`BASE=/opt/rocm/mivisionx/model_compiler/python
python3 ${BASE}/onnx_to_nnir.py $2 nnir${1} --input_dims 1,3,224,224
python3 ${BASE}/nnir_update.py --fuse-ops 1 nnir${1} nnir${1}-Fused
python ${BASE}/nnir_update.py --convert-fp16 1  nnir${1}-Fused nnir${1}-Fused-2
python3 ${BASE}/nnir_to_openvx.py  nnir${1}-Fused-2 ${1}-openVXModel
cd ${1}-openVXModel
mkdir -p build && cd build
cmake ..
make
the optimization fp16 fails (I guess)python anntest.py build/libannpython.so weights.bin ~/Desktop/camera\ phone/small.jpg  out
OK: AnnAPI found "input,data_0,1,3,224,224;output0,prob_1,1,1000;" as configuration in build/libannpython.so
OK: loaded 26 kernels from libvx_nn.so
ERROR: agoVerifyGraph: kernel org.khronos.nn_extension.convolution_layer: invalid tensor meta for argument#4
ERROR: vxVerifyGraph: failed (-17)
`
I can fuse the operation but there is no advantage.

@paolodalberto For resnet ONNX mode, we currently do not support v2. We support [https://s3.amazonaws.com/download.onnx/models/opset_8/resnet50.tar.gz]. (https://s3.amazonaws.com/download.onnx/models/opset_8/resnet50.tar.gz) from https://github.com/onnx/models/tree/master/vision/classification/resnet/resnet50 which is release 1.3.
As for your question on googlenet, I haven't checked it yet. I will let you know once I look into it

let me check if I can reproduce performance for the other resnet. I will work during the weekend.
thank you.

@paolodalberto I tried the googlenet model you have specified with the latest MIVisionX and it passed our anntest. I believe you have an older version of MIVisionX since we have 32 kernels in libvx_nn.so now. Please pull the latest code. Yours specifies 26 (OK: loaded 26 kernels from libvx_nn.so)

@paolodalberto Any updates on the issue? Were you able to run the models?

@paolodalberto please reopen the issue if you are having trouble with running the model.


fix: Disable the default discovery scheduler

The default auto discovery is disabled, but the default discovery is very short (30s).  When the user enables the auto discovery, it would trigger it every 30s by default.  This value doesn't fit the most of use cases.  By modifying it to 0s, the discovery scheduler will be disabled by default.

If your build fails  due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?)
[x] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?)

Testing Instructions

New Dependency Instructions (If applicable)

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 38.72%. Comparing base (4ff1172) to head (fd3df5e).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4860   +/-   ##
=======================================
  Coverage   38.72%   38.72%
=======================================
  Files         118      118
  Lines       11064    11064
=======================================
  Hits         4284     4284
  Misses       6408     6408
  Partials      372      372

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

recheck


Translated relationship and relationship fetch default placeholder

Translated relationship and relationship/fetch default placeholder.

A new inspection was created.

Even better! Thanks @promatik , merging now. Will be tagged and released on Monday the latest.

The inspection completed: No new issues


Hacktoberfest

Hacktoberfest 2017

I'm looking for well known URL's

So how can we help ?

I am looking for more well known subdomain names like in sub.txt:
api.example.com
app.example.com
email.example.com


Long Text being cut down

Describe the bug
In ManimGL v1.6.1 with manimpango v0.4.1, when passing a long string to Text, parts outside the screen are cut.
Code:
class Textbug(Scene):
    def construct(self):
        text = "The study of this language does not require a bulky knowledge of numerous parts of mathematics; although some preliminary acquaintance with calculus and algebra is more than welcome."
        text_box = Text(text, font="Consolas").scale(0.7)
        self.add(text_box)

Wrong display or Error traceback:

@AStarySky did you ever find a workaround for this issue? The same issue seems to be present on manim Community, but I can't find a bug for it.

Umm, it’s probably the problem of manimpango, the text processing part of manim. What I choose to do is to just fallback to whatever version of manimgl that can handle this properly (I’m using v1.5.0 right now, which is pretty old). Also I remember that manimce and manimgl uses different version of manimpango, so it might not work for you.
But thank you for your attention on the problem, I hope that this problem will solve soon.(Maybe you can reach out to other developers maintaining CE)
Or just \n in every line.🫤
来自钉钉专属商务邮箱------------------------------------------------------------------
@.>
日　期：2024年12月05日 16:23:05
@.>
@.>; @.>
主　题：Re: [3b1b/manim] Long Text being cut down (Issue #1833)
@AStarySky did you ever find a workaround for this issue? The same issue seems to be present on manim Community, but I can't find a bug for it.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Also, I’m quite busy with my studies right now, so it’s been a long time since I used manim last time. Apologies for any inaccuracies.
来自钉钉专属商务邮箱------------------------------------------------------------------
@.>
日　期：2024年12月05日 16:23:05
@.>
@.>; @.>
主　题：Re: [3b1b/manim] Long Text being cut down (Issue #1833)
@AStarySky did you ever find a workaround for this issue? The same issue seems to be present on manim Community, but I can't find a bug for it.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

I did report this on manim CE as well: https://github.com/ManimCommunity/manim/issues/4043
Using Tex is a good workaround in my case.


Embeddables do not properly alias for order-by statements in query builder

If one has an entity that contains an embeddable (referenced as $priceObj):
/**
 * @ORM\Embeddable
 */
class Price
{

    const CUR_USD = 'USD';

    const CUR_CAD = 'CAD';

    const INVALID_CURRENCY_CODE_EXC_CODE = 2;

    const INVALID_CURRENCY_CODE_EXC_MESSAGE = 'An invalided currency code was provided.';

    protected $validCurrencies = array(
        0 => self::CUR_USD,
        1 => self::CUR_CAD
    );

    /**
     * @ORM\Column(type="bigint")
     */
    private $amount;

    /**
     * @ORM\Column(type="string", length=3, nullable=false)
     */
    private $currencyCode;

}

And one attempts to perform an orderBy on the amount field. Doctrine generates a proper-looking DQL statement; however, the SQL statement where the order-by'd element is using the pre-aliased select statement. This appears to cause a 'semantical error'.
PHP:
$qb = $this->_em->createQueryBuilder();
        $query = $qb->select('row')->from('ExampleDomainModel\Entity\Barcode\ProductResult', 'row');
        $query->andWhere($qb->expr()
            ->eq('row.GSIN', ':GSIN'))
            ->setParameter('GSIN', $barcode);

        $query->addOrderBy('row.priceObj.amount', 'asc');

Generated DQL:
SELECT row FROM ExampleDomainModel\Entity\Barcode\ProductResult row WHERE row.GSIN = :GSIN AND row.isNew = :a57a4da9e5537d AND row.priceObj.amount >= :a57a4da9e5677f ORDER BY row.priceObj.amount asc

Generated SQL:
SELECT b0_.id AS id_0, b0_.feedProvider AS feedProvider_1, b0_.isNew AS isNew_2, b0_.title AS title_3, b0_.description AS description_4, b0_.GSIN AS GSIN_5, b0_.createdAt AS createdAt_6, b0_.url AS url_7, b0_.imageUrl AS imageUrl_8, b0_.fpExtendedAttributes AS fpExtendedAttributes_9, b0_.priceObj_amount AS priceObj_amount_10, b0_.priceObj_currencyCode AS priceObj_currencyCode_11, b0_.location_latitude AS location_latitude_12, b0_.location_longitude AS location_longitude_13 FROM barcode_product_result b0_ WHERE b0_.GSIN = ? AND b0_.isNew = ? AND b0_.priceObj_amount >= ? ORDER BY b0_.priceObj_amount ASC

$ php composer.phar show --installed | grep doctrine
adamlundrigan/ldc-zf-oauth2-doctrine         dev-master 54c810e zf-oauth2 adapter for use with D...
api-skeletons/zf-oauth2-doctrine             1.0.5              Doctrine OAuth2 Server Adapter f...
doctrine/annotations                         v1.2.7             Docblock Annotations Parser
doctrine/cache                               v1.6.0             Caching library offering an obje...
doctrine/collections                         v1.3.0             Collections Abstraction library
doctrine/common                              v2.5.3             Common Library for Doctrine proj...
doctrine/data-fixtures                       v1.0.2             Data Fixtures for all Doctrine O...
doctrine/dbal                                v2.5.4             Database Abstraction Layer
doctrine/doctrine-module                     0.9.0              Zend Framework 2 Module that pro...
doctrine/doctrine-orm-module                 0.9.2              Zend Framework 2 Module that pro...
doctrine/inflector                           v1.1.0             Common String Manipulations with...
doctrine/instantiator                        1.0.5              A small, lightweight utility to ...
doctrine/lexer                               v1.0.1             Base library for a lexer that ca...
doctrine/orm                                 v2.5.4             Object-Relational-Mapper for PHP
hounddog/doctrine-data-fixture-module        0.0.4              Zend Framework 2 Module that pro...
phpro/zf-doctrine-hydration-module           0.1.5              Doctrine hydrators for ZF2
zfcampus/zf-apigility-doctrine               0.4.1              Doctrine for Apigility
zfcampus/zf-doctrine-querybuilder            1.4.2              QueryBuilder filters for Doctrine

Expected:
SELECT b0_.id AS id_0, b0_.feedProvider AS feedProvider_1, b0_.isNew AS isNew_2, b0_.title AS title_3, b0_.description AS description_4, b0_.GSIN AS GSIN_5, b0_.createdAt AS createdAt_6, b0_.url AS url_7, b0_.imageUrl AS imageUrl_8, b0_.fpExtendedAttributes AS fpExtendedAttributes_9, b0_.priceObj_amount AS priceObj_amount_10, b0_.priceObj_currencyCode AS priceObj_currencyCode_11, b0_.location_latitude AS location_latitude_12, b0_.location_longitude AS location_longitude_13 FROM barcode_product_result b0_ WHERE b0_.GSIN = ? AND b0_.isNew = ? AND b0_.priceObj_amount >= ? ORDER BY priceObj_amount_10 ASC

Actual:
SELECT b0_.id AS id_0, b0_.feedProvider AS feedProvider_1, b0_.isNew AS isNew_2, b0_.title AS title_3, b0_.description AS description_4, b0_.GSIN AS GSIN_5, b0_.createdAt AS createdAt_6, b0_.url AS url_7, b0_.imageUrl AS imageUrl_8, b0_.fpExtendedAttributes AS fpExtendedAttributes_9, b0_.priceObj_amount AS priceObj_amount_10, b0_.priceObj_currencyCode AS priceObj_currencyCode_11, b0_.location_latitude AS location_latitude_12, b0_.location_longitude AS location_longitude_13 FROM barcode_product_result b0_ WHERE b0_.GSIN = ? AND b0_.isNew = ? AND b0_.priceObj_amount >= ? ORDER BY b0_.priceObj_amount ASC

The key difference is that "b0.priceObj_amount" should become "priceObj_amount_10" according to the "OriginalFieldName" As "Re-mappedName"

As mentioned on IRC, might be worth looking into https://github.com/doctrine/doctrine2/blob/31a0c02b066764ec7b55eedc5f3cab78235a4e4b/lib/Doctrine/ORM/Query/SqlWalker.php#L330 and code around it, which is responsible for alias generation.

This seems to be a duplicate / slightly different version of the same issues described in #5622

I can confirm that setting
[mysqld]
sql-mode= "STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION"

Does resolve this issue.


Go Module Support?

Is this something your folks are interested in? If so I could do a PR.

I have a branch with go.mod support for this, but never pushed it.  We have started consolidating the Go libraries for NR into a single repo ( https://github.com/newrelic/newrelic-client-go ), and this is eventually on the list to go there as well so little work has been done in this repo while in that process.  The only caveat to go.mod is that golangci-lint is broken...

Good to know about the newrelic-client-go repo, funnily enough I recently wrote my own client internally to manage new relic alerts, I guess I should migrate over at some point.
There were a couple of other things I was hoping to improve with the client library (such as properly flushing all worker goroutines when flush is called). Since this library will be migrated in the not too distant future is that something that you would still want contributed back or should I just maintain my own fork?

Resolved in #11

For additional work, contributing back here would be great as the timeline for consolidation hasn't been established yet.  We're working on a few other priorities at the moment.  If you are curious around that work, we have an RFC that we would love feedback on for our Developer Toolkit public roadmap: https://discuss.newrelic.com/t/rfc-developer-toolkit-roadmap-2020/90528


Initial attempt at rendering annotations on types

I’m submitting this, as @sichvoge may be interested in it. It’s not finished yet, but I wanted to see if this was starting on the right lines…
Concentrating on the semantics first, before any styling. This implements the “simple” method from our previous discussion.
Currently it will pick up on type annotations and render them next to the type.  For type annotations that are just flags (i.e. have a nil type) it will display only the annotation name.
Limitation: when an annotation does have properties, they are shown as JSON, not in a structured way. It looks like I may need to modify raml2html’s source to add a property to the item to say that it is an annotation.  Then I can query this in item.nunjucks and render it more appropriately.  Please me know if you think this is the right way to go.
Test data: I created a very simple API for testing.

@matatk If you don't mind, can you add a snapshot of how it would look like to your OP. That will help to immediately understand the changes of this PR without having it to install :)

I'm not sure if GitHub tells you about updates; in case it doesn't, I added pictures last week. As I said, this is very early and will need much more work, but wanted to check if it was starting off on the right track, and ask the questions above about the output.

It did not tell me about your update :D MH, it's pretty hard to render annotations indeed. I will make time next week to play around with your branch, ok?

OK had some time looking into this, for me this looks like a good first step forward @matatk. @kevinrenskers any opinions on your side?


BUG Remove usage of deprecated each() and use a helper method instead

Prevents deprecation errors when running PHP7.

Don't merge, I'm going to move into ArrayLib and replace other each() calls.


Add search bar

I want to incorporate a search bar on the landing page such that users can easily find any project on the website
I will use the following tech stack to perform this task

Javascript
HTML/CSS
Bootstrap

I am a GSSOC'22 contributor, please assign this issue to me.

already assigned #1168


Fix mismatched tags in HTML example.

 was closed using

Codecov Report

Merging #637 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #637   +/-   ##
=======================================
  Coverage   71.49%   71.49%
=======================================
  Files         136      136
  Lines       12790    12790
=======================================
  Hits         9144     9144
  Misses       3646     3646

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f17096b...df34587. Read the comment docs.

Thank you, @cppguru!


PR: added missing paging to terraform provider fetch

PR added missing paging to terraform provider fetch to enable support for > 100 providers
Description

Bugfix: added missing paging to terraform provider fetch
(was only fetching first page (100) Providers)
refactored provider fetch workflow to remove the need for the hard-coded number of total snippets
(complimentary change to paging bug fix so snippets not truncated when hitting configured total_snippets amount)
pulled latest snippets and picked up missing providers
updated npm dependencies, audit fixes

Checklist

[x] The code builds correctly (npm run build:snippets produces a new and improved snippets file)
[ ] Tests for the changes have been added and passing (for bug fixes/features)
[ ] Test results are pasted in this PR (in lieu of CI)
[x] Docs have been added/updated (for bug fixes/features)
[x] Added myself/the copyright holder to the AUTHORS file
[x] Any breaking changes have been noted in the description

👋  @orrin - you're awesome for having submitted this PR and it totally missed my radar for an embarrassingly long time. I like the improvements here but just merged a conflicting update. I plan to roll in your changes and credit you in the authors for the work. Thank you!


added Model and StanModel classes

closes #6

Codecov Report

Merging #20 into master will increase coverage by 2.77%.
The diff coverage is 98.30%.

@@            Coverage Diff             @@
##           master      #20      +/-   ##
==========================================
+ Coverage   92.41%   95.19%   +2.77%
==========================================
  Files          11       11
  Lines         567      603      +36
==========================================
+ Hits          524      574      +50
+ Misses         43       29      -14

Impacted Files
Coverage Δ

viabel/tests/test_bounds.py
100.00% <ø> (+1.96%)
:arrow_up:

viabel/tests/test_objectives.py
100.00% <ø> (+4.34%)
:arrow_up:

viabel/convenience.py
96.00% <75.00%> (+25.16%)
:arrow_up:

viabel/models.py
100.00% <100.00%> (+5.55%)
:arrow_up:

viabel/tests/test_approximations.py
100.00% <100.00%> (+5.00%)
:arrow_up:

viabel/tests/test_convenience.py
100.00% <100.00%> (ø)

viabel/tests/test_models.py
97.67% <100.00%> (+1.24%)
:arrow_up:

viabel/approximations.py
91.95% <0.00%> (+0.57%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cbd28d0...5eda908. Read the comment docs.


Problem: can't build Postgres within Omnigres

prereqdir=`cd 'parser/' >/dev/null && pwd` && \
   cd '../../src/include/parser/' && rm -f gram.h && \
   ln -s "$prereqdir/gram.h" .
/bin/sh: line 0: cd: parser/: No such file or directory
make: *** [../../src/include/parser/gram.h] Error 1

Solution: don't use CDPATH

Perhaps we can unset CDPATH in the build process to avoid this? cc @piranha

So it turns out CDPATH is really weird thingie, and you can avoid this problem in zsh by doing this instead of export CDPATH:
if [[ -o interactive ]]; then
    setopt auto_cd
    cdpath=(~/dev/work ~/dev/web ~/dev/misc ~/dev)
fi

Here is an article that says "do unset CDPATH in beginning of every script": https://bosker.wordpress.com/2012/02/12/bash-scripters-beware-of-the-cdpath/


Using Latitude/Longitude instead of Coordinates when posting to API

Hi,
I'd like to use the app for sending my location every hour or so to a website.
This is the URL I have to use:
https://project-gc.com/api/UserLocation/Update/?protocol=1&uid=XXX&latitude=%LAT&longitude=%LON&acc=�C&ts=%TIMESTAMP&token=XXX&nid=XXX
UID, Token and NID are provided, but how do I get Latitude/Longitude and Timestamp out of Overland? And how is the syntax? Would %timestamp be okay or do I need to use $timestamp?

Overland doesn't have the option to include the location data in the URL like that, it only sends it in the POST body. This is what the app sends: https://github.com/aaronpk/Overland-iOS#api
It sounds like you only care about uploading a single point at a time, not the full track logs, which is on the backlog here: #121
It seems like a reasonable feature to include URL parameters in this mode, so I'll keep this issue open and try to add that for the next update.


Full WebGL rendering on MVT layer

I wanted to compare vector tile rendering in OL to MapLibre so I constructed a set of PBF vector tiles that cover zoom levels 0-14.
The two maps are here: https://hcmc.uvic.ca/people/greg/tiles/openlayers.html
and here: https://hcmc.uvic.ca/people/greg/tiles/maplibre.html
(if it helps, there's a styled version of the OL map here: https://hcmc.uvic.ca/people/greg/tiles/openlayers_styled.html)
Using the same vector tiles, the MapLibre map zooms in as far you like with no degradation in quality, but the OL map does not.
As I understand it this is the result of OL rasterizing the tiles whereas MapLibre uses WebGL rendering for everything.
I'm not clear on the issue though: is my experience a result of me not knowing what I'm doing? or is OL not able to render the vector tiles using WebGL?
Can someone explain what I'm doing wrong? And if I'm not doing anything wrong, can someone tell me if WebGL vector tile rendering is on the horizon?
I'm not sure if this is a bug report or a feature request.

A webgl renderer is being worked on, e.g. #14445 but so far there is no text rendering. To test the current state you'd need to use an unreleased v8.0.0 version, e.g ol@dev via npm.
Here is an examle

The OpenLayers map is not configured correctly. As far as I can tell, your tiles are available down to zoom level 14, but it appears you haven't set maxZoom: 14 on the VectorTile source. When you do that, you can zoom in as far as you want and will also get crisp rendering.
Also, you can avoid the configuration hassle and get the same style as for the maplibre map by using ol-mapbox-style to parse the style json:
import {apply} from "ol-mapbox-style";
import {useGeographic} from "ol/proj";

useGeographic();
apply('map', "https://hcmc.uvic.ca/people/greg/tiles/vi.json").then((map) => {
  map.getView().setCenter([-121.72, 50.69]);
  map.getView().setZoom(6);
})

I have created a codesandbox with that: https://codesandbox.io/s/withered-sky-nvn9d3.

Thanks to you both. Both responses are massively useful.


[qBittorrent] Port forwarding help

Qbittorrent.

Hi, this is more of a question. When I change my internal port to match the external port I can connect to trackers but am not open for incoming connections (the client refuses). When I keep the internal port as 6881 however it is open for connection but trackers have this port blacklisted so it doesn't work. I guess the question is do we know why this is happening and is there any way to change it?
Sorry for the poor wording, I hope this makes sense.
Thanks.

This is the current setup now. 59595 in both the internal settings and in the config file. This allows me to connect to trackers but not be open to connections (port forwarding has been set up for 59595).
When I had 59595 in the config page but had the internal qbit settings listening to 6881 I was able to be open to the connections but unable to connect to trackers as that port is blocked even though I was still portforwarding 59595 and the config file was set to that.
I hope that's explained more of my situation anyhow.

Hi, thanks. What would be the solution? Changing the default qbittorrent port from 6881 to another port ? such as 59595 ?

That should work, yes.
On Fri, 1 Jul 2022, 19:16 Alexandre, @.***> wrote:

Hi, thanks. What would be the solution? Changing the default qbittorrent
port from 6881 to another port ? such as 59595 ?
—
Reply to this email directly, view it on GitHub
https://github.com/alexbelgium/hassio-addons/issues/376#issuecomment-1172021667,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUMWJOBJQHTVN2HKRNQRLWLVR2LN3ANCNFSM52HDCZRQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'll push a new version ASAP and we'll see how that goes ;-)

That worked for me perfectly, thank you.

Great news, thanks

this ports can be changed?


Move Instance Error

https://github.com/Minestom/Minestom/discussions/1571 There is more information about this.

nvm


IE compatibility

The current plugin is not compatible with any ie version since it utilizes the Object.assign function.
Using the existing _extends function instead solves the problem.

Fixed in 1.12.5. Thank you for reporting this issue.


Add support for Occupancy sensor

Description:
This is based mostly on #14188 Tested and confirmed to work with Centralite Motion Sensor and Xiaomi Aquara motion sensors
Checklist:

[x] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass

Scratch Xiaomi Aquara motion sensors, those need a zigpy quirk fix. But Centralite motion/occupancy sensor works fine.


This runs too early in the boot process if included in an image

Rather than deploying this to an already running NixOS system, I was hoping to be able to build this into a pre-baked image that I could then deploy and boot.
Unfortunately, this doesn't seem to work, since this runs right at the beginning of stage2. At this point, SSH has never run, so there's no host key, the network is not even available to be able to reach Azure KeyVault, etc.
What do you think of having another version of this module that accepts similar config and instead creates a systemd service that can then be sequenced as part of the boot?
99] stage-1-init: resize2fs 1.45.5 (07-Jan-2020)
[  232.432835] stage-1-init: Resizing the filesystem on /dev/disk/by-label/nixos to 134152187 (4k) blocks.
[  232.432994] stage-1-init: The filesystem on /dev/disk/by-label/nixos is now 134152187 (4k) blocks long.
[  232.433343] stage-1-init: mounting /dev/disk/by-label/nixos on /...
[  232.491454] EXT4-fs (sda2): mounted filesystem with ordered data mode. Opts: (null)
[  233.555253] EXT4-fs (sda2): re-mounted. Opts: (null)
[  233.555685] booting system configuration /nix/store/jy2c5crwl63miv0k106fffldb90sccpv-nixos-system-unnamed-20.09.20200720.dirty
[  234.045090] stage-2-init: running activation script...
[  235.601159] stage-2-init: setting up /etc...
[  235.883966] stage-2-init: setting up secrets...
[  237.981769] stage-2-init: /nix/store/14lb4jzdf2j3k9a57niqzv2wi6z49yr9-sops-install-secrets-0.0.1/bin/sops-install-secrets: Error setting up gpg keyring: Cannot read ssh key '/etc/ssh/ssh_host_rsa_key': open /etc/ssh/ssh_host_rsa_key: no such file or directory
[  237.981874] stage-2-init: Activation script snippet 'setup-secrets' failed (1)
[  239.726445] systemd[1]: Inserted module 'autofs4'
[  239.872676] NET: Registered protocol family 10
[  239.873489] Segment Routing with IPv6
[  240.072141] systemd[1]: systemd 245 running in system mode. (+PAM +AUDIT -SELINUX +IMA +

Won't this be too late for services to run?

Maybe if secrets were more tied to services this could work. Than we just need to ensure to have each affected service to wait before sops run.

For now the easiest would be to have network in initrd. It's not a very nice solution though.

I was thinking the services could use systemd's mechanism and wait on the /run/secrets/... key path to exist.
I'll try to enable network in initrd, I hadn't thought to just see if it works, but it might.

Since I also want to restart systemd services conditionally in the end it cannot be a systemd service that we let manage by nixos's service activation hook. It needs to start before. However it might be possible to trigger a target or wait for the path to exists as you suggested.

It looks like boot.initrd.network.enable isn't quite sufficient. Although the VM gets a lease, the network is unreachable when sops-install-secrets tries to use it to access the authentication token service. Maybe I can curl and sleep in a loop before calling sops-install-secrets for now to see if it works.

Unfortunately, the closest I've gotten required boot.initrd.network.flushBeforeStage2 = false; and it still wound up failing due to network failures, I suspect because stage-2 started reconfiguring the interface, etc. I can now try to disable all of that, but I'm starting to think I should've gone ahead and tried the service route.

Oh, wait, I think it's DNS...

Yes. Golang cannot resolve it.

This is what I plan to implement to support networked vaults:

First start sops-nix from activation script to get the state of services that need a restart/reload.
Start a sops-nix.service that services that require a key can wait for.
Let the systemd activation script finish which will not restart regular services
Restart/Reload all services that have been not reload/restarted in 3.

I got a full demo working. It boots a VM in Azure with no unencrypted secrets, which starts up Tor and a hidden service powered by a key protected with (sops)-nix and decrypted via AZKV. This is a very cool UX with Nix.
However, with the current model, it required some hacks:

relax the constraint around gpg/ssh keys in Nix since the AZKV scenario doesn't need it (luckily sops-install-secrets was less particular and it and my patched sops worked)

(I think it should be addressed in sops-nix, probably by adding some options/values)

had to enable boot.initrd.network.enable and boot.initrd.network.flushBeforeStage2 = false which felt ... undesirable.

(sounds like you're addressing this)

work around some DNS-in-Azure pleasantries by writing /etc/resolv.conf before calling sops-install-secrets

(I'm going to have to fix this myself with other postLVMcommands... but that's unrelated and hopefully unnecessary after your changes)

Just to make sure I'm not missing something, here's what I was thinking when I was thinking systemd could orchestrate the restarts --

(on boot) use ConditionPathExists=${config.sops.secrets.foobar-key.path} to block the dependent services on boot
(on key change) use a systemd path watcher to notice that the symlink changed and auto-restart the service, ex: https://superuser.com/a/1276457

Maybe that doesn't work because you symlink the entire secret dir? Or I'm missing a caveat of what activation allows for differently?

I got a full demo working. It boots a VM in Azure with no unencrypted secrets, which starts up Tor and a hidden service powered by a key protected with (sops)-nix and decrypted via AZKV. This is a very cool UX with Nix.
However, with the current model, it required some hacks:
* relax the constraint around gpg/ssh keys in Nix since the AZKV scenario doesn't need it (luckily `sops-install-secrets` was less particular and it and my patched `sops` worked)

  * (I think it should be addressed in `sops-nix`, probably by adding some options/values)

Is there any option that azure needs to have in this case? Than I can relax the constraint if azure is enabled.

* had to enable `boot.initrd.network.enable` and `boot.initrd.network.flushBeforeStage2 = false` which felt ... undesirable.

  * (sounds like you're addressing this)

* work around some DNS-in-Azure pleasantries by writing /etc/resolv.conf before calling `sops-install-secrets`

That will be also addressed in my plan

  * (I'm going to have to fix this myself with other `postLVMcommands`... but that's unrelated and hopefully unnecessary after your changes)

Just to make sure I'm not missing something, here's what I was thinking when I was thinking systemd could orchestrate the restarts --
* (on boot) use `ConditionPathExists=${config.sops.secrets.foobar-key.path}` to block the dependent services on boot

* (on key change) use a systemd path watcher to notice that the symlink changed and auto-restart the service, ex: [superuser.com/a/1276457](https://superuser.com/a/1276457)

Maybe that doesn't work because you symlink the entire secret dir? Or I'm missing a caveat of what activation allows for differently?

I think it would watch inodes, I am not sure. In this case it would not work. However I also don't want it to restart on key change unconditionally. On activation the service might get restarted already so having another restart will be not necessary. It should just wait for the new key to be decrypted.

I'm abandoning the Sops change now (Go SDK needs changes), so we will need an option for Azure to be able to set AZURE_AUTH_MODE(="msi"). However, I think the module API should plan for a future when this is not needed. (For example, GCPKMS will not need any such variable, and will not need a gpg or sshpubkey.)

Would it make sense to have a sops.azure.enable = true; option?

Not really. Let me provide some background:

Azure (and every other cloud I'm aware of) provide multiple authentication methods for {users|services|applications}
Some providers like GCP have fairly-standardized ways of authenticating - they try a sequence of mechanisms from a single standard public SDK function (DefaultCredentials for Google) that try a specific sequence of well-defined authentication mechanisms
Azure never bothered to standardize such a mechanism cross-SDK, so sops and every application has special handling for the various Azure auth types, usually based on environment variables. In sops' cases, AZURE_AUTH_METHOD is sort of the entrypoint.
Based on the value of AZURE_AUTH_METHOD, other env vars (might) then need to be set. (Except for the in the msi scenario)

I think the sops-nix module should attempt to map to the sops CLI api as much as possible, to meet expectations. But this is hard to do without modeling the entire surface API of Sops, which for Azure includes 5+ potential environment variables, or just allowing arbitrary environment variables to be set. I don't really have any sort of recommendation here, but sops.azure.enable is not really helpful here.
At the very least, this module is very much built around assumptions about having credential material on-disk, and that won't be true for GCP/AWS/Azure/etc and so that will need to be modeled in the module API.
Let me know what you think, I'll have to think about this more to see if I have any alternative suggestion.

Not really. Let me provide some background:
* Azure (and every other cloud I'm aware of) provide multiple authentication methods for {users|services|applications}

* Some providers like GCP have fairly-standardized ways of authenticating - they try a sequence of mechanisms from a single standard public SDK function (`DefaultCredentials` for Google) that try a specific sequence of well-defined authentication mechanisms

* Azure never bothered to standardize such a mechanism cross-SDK, so sops and _every_ application has special handling for the various Azure auth types, usually based on environment variables. In sops' cases, `AZURE_AUTH_METHOD` is sort of the entrypoint.

* Based on the value of `AZURE_AUTH_METHOD`, other env vars (might) then need to be set. (Except for the in the `msi` scenario)

I think the sops-nix module should attempt to map to the sops CLI api as much as possible, to meet expectations. But this is hard to do without modeling the entire surface API of Sops, which for Azure includes 5+ potential environment variables, or just allowing arbitrary environment variables to be set. I don't really have any sort of recommendation here, but sops.azure.enable is not really helpful here.
At the very least, this module's assertions are currently built around assumptions about having credential material on-disk, and that won't be true for GCP/AWS/Azure/etc and so that will need to be modeled in the module API.
Let me know what you think, I'll have to think about this more to see if I have any alternative suggestion.

Ok. Until it is clear to me what needs to be set to support azure, I will keep the module assertion in place. When nix-sops supports Azure than there should be documentation how this is achieved.

The other issue I have with it running at activation is that it runs inside make-disk-image.nix when it is not desired (though arguably I can see this being a design problem with m-d-i). I can imagine a way to pass an extra kernel cmdline arg nosops which the activationScript could use to skip installing secrets, maybe.

The other issue I have with it running at activation is that it runs inside make-disk-image.nix when it is not desired (though arguably I can see this being a design problem with m-d-i). I can imagine a way to pass an extra kernel cmdline arg nosops which the activationScript could use to skip installing secrets, maybe.

Did not know that make-disk-image.nix relies on the activation phase. What is it using for?

I have a similar issue. My ssh key doesn't reside on the rootfs, but on a separate partition. For now, I forked your project, and turned the activation script into a systemd unit.
https://github.com/zaninime/sops-nix/commit/8af89796020155cbde4fa33a7f1f8b1531f859fa

That looks interesting, did you solve dependencies between the affected services and the secrets?

I just use the usual after / requires in the other units, to make them depend on this one. I also add RequiresMountsFor= to this unit, to ensure the FS is mounted before doing the secrets setup.

Same issues here, I'm using / on tmpfs mode and my host ssh key is in/persistent.

Add neededFotBoot = true; for my /persistent volume solved my issues : )

Hi,
I just ran into pretty much this problem.
My SSH host keys are mounted through a systemd unit generator that generates the mount unit for the mountpoint containing the keys. (Look here if you're curious).
Since the mount is not in /etc/fstab, I can't use neededForBoot to mount it during stage 2 so sops can access it.
All of my secrets are bound to systemd units, so I'd rather use RequiresMountsFor=.

@Mic92:

Since I also want to restart systemd services conditionally if secrets in the end it cannot be a systemd service that we let manage by nixos's service activation hook. It needs to start before. However it might be possible to trigger a target or wait for the path to exists as you suggested.

Services depending on a sops service to provide secrets could be restarted after it if they specified a Requires dependency on it. I don't think there's a way in systemd itself to have dependent services reload in response to a sops service restart, but it would be pretty trivial to have sops-install-secrets issue the service reloads itself, no?

 Right now, AFAICT, the system image that start with sops-nix enabled and can't active (in my case: a VM just to test the configuration, running without any networking) fails is left in a half-inactivated state.


add cluster ci

bors try

bors try-

bors try

hmm I'm getting
   18345125       any buildkit jbolewsk REQUEUE_       0:00   1:05:00      1 (JobHoldMaxRequeue)

so we might be running into job submission limits

bors try

bors try

bors try-


FEMA_COMMUNITIES constant not previously defined and results in exception when running comparison tool

Describe the bug
FEMA_COMMUNITIES variable is not previously defined.
To Reproduce
Steps to reproduce the behavior:
In the following comparison tool notebook notebook
There is a variable, FEMA_COMMUNITIES, that does not appear to be used in any of the Index named tuples.
One can find this variable by searching (CMD + F) and enter into the search field FEMA_COMMUNITIES.
One will notice it is defined as a keyword argument but not previously set.
Expected behavior
Workflow should output comparison tool without any exceptions
Desktop (please complete the following information):

OS: MacOS, Big Sur 11.6
Python Version: 3.7.4
data-pipeline 0.1.0
Poetry version 1.1.11

Good catch @ahlusar1989 -- that was introduced by this PR, and we just need to delete that index.
Deleting the lines:
        Index(
            method_name=FEMA_COMMUNITIES,
            priority_communities_field=FEMA_COMMUNITIES,
            other_census_tract_fields_to_keep=[],
        ),

Should do the trick. Want to open a PR? Thank you!!

@lucasmbrown-usds sure thing - thanks for confirming and for clarifying.


[Request] Ubuntu Touch version

Good day!
Please, make Ubuntu Touch version and publish it on open-store.io
Best regards!

We're busy enough developing an Android app. We have no plans to support other platforms.

Thanks for reply!20:06, 15 августа 2018 г., cketti notifications@github.com:We're busy enough developing an Android app. We have no plans to support other platforms.
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.


Bump to mypy 0.701 version

This fixes #75
run_test.sh actually stopped working because the python version launched a new version somewhat recently that changed the error messages https://pypi.org/project/protobuf/#history
This diff pins to the old version (3.6.1) of python-protobuf in requirements.txt to avoid this problem in the future. I plan to try to bump the version of python-protobuf explicitly in a future diff.
This diff additionally bumps the mypy pointer to the latest release. Latest release no longer supports python3.4 which ships default in trusty (travis CI default)
As such, I bumped travis CI to explicitly use xenial so that we have a newer version of python3
90% of software engineering is dependency management..
Thank you @erictraut for reporting.


404 error on changelog for self-hosting

The README file has a link to View the changelog for self-hosting, but this link does not work.

This is the correct file: https://github.com/graphql-hive/platform/blob/main/deployment/CHANGELOG.md

thanks, I'll open a PR for the readme file.

Already fixed @yanns https://github.com/graphql-hive/platform/commit/5c4b7ddffa44803d7712989ca70786df3a417a46

amazing! 😍


Geomedian notebook

Proposed changes
Hi all, here is the geomedian product notebook, based on the Africa notebook.
All feedback is welcome!! Happy to change anything.
Checklist (replace [ ] with [x] to check off)

[X] Notebook created using the DEA-notebooks template
[ X] Remove any unused Python packages from Load packages
[X ] Remove any unused/empty code cells
[X ] Remove any guidance cells (e.g. General advice)
[ X] Ensure that all code cells follow the PEP8 standard for code. The jupyterlab_code_formatter tool can be used to format code cells to a consistent style: select each code cell, then click Edit and then one of the Apply X Formatter options (YAPF or Black are recommended).
[X ] Include relevant tags in the final notebook cell (refer to the DEA Tags Index, and re-use tags if possible)
[X ] Clear all outputs, run notebook from start to finish, and save the notebook in the state where all cells have been sequentially evaluated
[] Test notebook on both the NCI and DEA Sandbox (flag if not working as part of PR and ask for help to solve if needed)
[ ] If applicable, update the Notebook currently compatible with the NCI|DEA Sandbox environment only line below the notebook title to reflect the environments the notebook is compatible with

Hey @margaretharrison , haven't forgotten this one, aiming to get to it this week! It looks awesome at first glance

P.S. Might be worth looking into why the tests are failing before merging.

Thanks for picking that up - changed.

View entire conversation on ReviewNB

Thank you again!

View entire conversation on ReviewNB


Feature Requests

Can u add the ability to save the qauntized models to storage that way if we want to use it again we don't have to keep reconverting every time we start an generation.

You should already be able to do that by setting the "-M, --mode" argument to "convert" to save the model as a quantized gguf file at the location specified with "-o, --output".

It auto saves to the main folder but I tried qauntized aura flow buy it killed when saving output

Interesting, please recompile with --config Debug, re-run with --verbose, and post the output

Interesting, please recompile with --config Debug, re-run with --verbose, and post the output

I can't post output termux crashes immediately I got 7gb ram to spare but it loads the tensors but as soon as it trys to save output it crashes. Could someone else try converting aura flow 0.3 please I need q5 or q4 and q8

I'm not familiar with aura flow but I wonder if it's based on a model type that sdcpp doesn't currently support. You could pipe the output to a file to avoid losing it when termux crashes, eg: "./sd etc etc &> foo.txt" to pipe both stdout and stderr to a file.

I'm not familiar with aura flow but I wonder if it's based on a model type that sdcpp doesn't currently support. You could pipe the output to a file to avoid losing it when termux crashes, eg: "./sd etc etc &> foo.txt" to pipe both stdout and stderr to a file.

You're right that has to be what it is it's not supported so it don't work so I can quantize sd3 I bet. Will flux and auto flow be added soon?

Gguf used less memory I can run flux q8_0 and t5xxlfp16 in comfyui on my phone so using gguf is better than safetensors

Sd3 uses flow so how come flux and aura not auto support?


Cannot use mypy for 3rd party plugins as tmt is missing py.typed marker

Would by nice if 3rd party can use tmt type declarations.
Currently it fails with src.py:17: error: Skipping analyzing "tmt": module is installed, but missing library stubs or py.typed marker  [import]
See https://mypy.readthedocs.io/en/stable/running_mypy.html#missing-library-stubs-or-py-typed-marker

Ha, .py.typed was in my initial drafts, but didn't make it to the final annotation patches.