FR: WatchOS Support

With the release of WatchOS 4 and an LTE Apple Watch, there’s no better time to plan support for Apple Watch. The Series 3 Watch has significantly more power, and applications will want to connect to Firebase.

Hi Matthew! Yes WatchOS4 looks awesome and we are keeping a close watch (pun intended) on this platform. We will continue to evaluate officially supporting the platform as it grows in apps and users, but until we do, we welcome any community contributions in this area.

I'm In !

+1

+1

+1

+1

+1

+1

+1

+1

+1

Watch OS 5 is here, and an addition with Firebase to WatchOS would be greatly appreciated! +1

Any update on this?

+1

2019 and still no watchOS support 😢

just watched firebase 2019. Added wearOS support. No watchOS support still.

I guess that this won't be an issue anymore with watchOS 6 since apps are independent?

I think this will be a much bigger issue. Hopefully with all of the streaming data and network framework, we can make this available ASAP!

On Jun 6, 2019, at 4:09 AM, José Jeria<EMAIL_ADDRESS>wrote:
I guess that this won't be an issue anymore with watchOS 6 since apps are independent?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/firebase/firebase-ios-sdk/issues/269?email_source=notifications&email_token=AABTJ3GQ25BNHTXFWAKTKATPZDH5NA5CNFSM4D24IZL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXCHJBI#issuecomment-499414149, or mute the thread https://github.com/notifications/unsubscribe-auth/AABTJ3AD5RVKFDXAWYDRRK3PZDH5NANCNFSM4D24IZLQ.

I think this will be a much bigger issue. Hopefully with all of the streaming data and network framework, we can make this available ASAP!
…

Surely you can add watchOS support because presumably watchOS 6 has the CFNetwork framework?

My assumption is that watchOS won't be supported until Google Analytics is supported which is gonna be a long time since we don't have UIKit for watchOS and at this point it's unlikely we ever will with the introduction of SwiftUI.

Started some investigation into this and kicked off the attempt with GoogleUtilities since it has no dependencies and is a dependency of all other Firebase SDKs - a few hurdles to start:

OCMock isn't available for watchOS, meaning a majority of our tests won't work
A larger hurdle (after disabling the test_spec in the podspec) the SystemConfiguration framework isn't available in watchOS.

This affects three subspecs: Reachability, Network, and AppDelegateSwizzler, which in turn affects Auth, Messaging, and Analytics.

It looks like it may be possible to add support for individual subspecs and maybe that's the way to pare it down further, but unfortunately this won't be as easy as just adding s.watchos.deployment_target = '5.0'
If anyone else wants to make an attempt in the meantime, you may be able to start adding support for each subspec and then creating a new watchOS target in the Example Xcode project.

@ryanwilson watchOS tests running on simulator are compiled against the iOS simulator target. This might change with standalone watch apps, but afaik hasn't yet in watchOS 6. iOS simulator testing + build targeting watchOS should be sufficient for watchOS for the time being. I made a PR to OCMock to add watchOS support, but it shouldn't be a blocker for testing.

@morganchen12 makes sense, the issue I ran into was using pod lib lint and pod gen - the watchOS bit choked when getting to the test spec that depended on the OCMock pod. I disabled the test spec to generate the project instead.
Re: SystemConfiguration not being available, it looks like [WatchConnectivity](https://developer.apple.com/documentation/watchconnectivity/wcsessiondelegate/1615662-sessionreachabilitydidchange) has a reachability function we can use to replace the one in SystemConfiguration`.

It would be really appreciated to do watchOS support at least from 6.0.

Additional FR #3453

any updates on this issue? watchOS 6 has released.

watchOS support was merged into OCMock today. In preparation for this feature, we can start running tests on watchOS simulator once OCMock makes a new release.

Any movement on this issue?

Progressing at #4016

Now that #4016 has merged with support for FirebaseMessaging and watchOS and will be part of the first release of 2020, I'm going to close this is umbrella issue.
Please open additional issues for any additional specific requested functionality for watchOS.

@paulb777 Is this supporting the Firebase Database on independent watch apps, or just Firebase Messaging? The original ticket here was so that we can make read/writes to the Realtime Database in watchOS

@mknippen It's just Firebase Messaging so far.
Going back and looking at the OP, I don't see the Firebase Database mention.
How about creating another issue specifically for Database and watchOS? A similar issue could be created for any other requested Firebase libraries for watchOS.  That will help us prioritize.
BTW, if anyone wants to work on this, the approach would be to update the relevant podspec's like #4016, working though any build issues, validating, and sending a PR.

@paulb777 @mknippen I've created a separate feature request for Realtime Database support: https://github.com/firebase/firebase-ios-sdk/issues/4556

I've created feature requests for Firebase Analytics and Firebase Crashlytics on watchOS: #4557 and #4558, respectively.


[Bugfix][VLM] Fix incompatibility between #7902 and #7230

This PR fixes an incompatibility between multimodal tensor stacking and multi-image support in LLaVA-NeXT and InternVL which causes the VLM CI to fail.
It also updates the docstrings about the input shapes which have become outdated since #7230.
The issue failed to be caught in #7902 because the changes from #7230 were not merged into its feature branch prior to merging #7902.

Thanks for the fix!

Thank you for the fix! Sorry about that.


fix(http): add missing codes to ErrorCode

This PR adds following error codes to the ErrorCode enum:
30038: Maximum number of uncompleted guild scheduled events reached (100)
50045: File uploaded exceeds the maximum size
50046: Invalid file uploaded
170001: Invalid JSON for uploaded Lottie file
170002: Uploaded Lotties cannot contain rasterized images such as PNG or JPEG
170003: Sticker maximum framerate exceeded
170004: Sticker frame count exceeds maximum of 1000 frames
170005: Lottie animation maximum dimensions exceeded
170006: Sticker frame rate is either too small or too large
170007: Sticker animation duration exceeds maximum of 5 seconds
180000: Cannot update a finished event
180002: Failed to create stage needed for stage event

Codecov Report

Merging #1291 (434abb0) into main (042ac9e) will decrease coverage by 0.15%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1291      +/-   ##
==========================================
- Coverage   31.16%   31.00%   -0.16%
==========================================
  Files         363      364       +1
  Lines       14376    14494     +118
==========================================
+ Hits         4480     4494      +14
- Misses       9896    10000     +104

Impacted Files
Coverage Δ

http/src/api_error.rs
7.59% <0.00%> (-0.61%)
:arrow_down:

.../request/channel/webhook/update_webhook_message.rs
21.90% <0.00%> (-0.55%)
:arrow_down:

http/src/routing/route_display.rs
10.66% <0.00%> (-0.39%)
:arrow_down:

http/src/request/mod.rs
85.71% <0.00%> (ø)

http/src/request/channel/message/create_message.rs
0.00% <0.00%> (ø)

...ttp/src/request/channel/webhook/execute_webhook.rs
0.00% <0.00%> (ø)

...application/interaction/create_followup_message.rs
0.00% <0.00%> (ø)

...application/interaction/update_followup_message.rs
0.00% <0.00%> (ø)

...pplication/interaction/update_original_response.rs
0.00% <0.00%> (ø)

http/src/request/attachment.rs
0.00% <0.00%> (ø)

... and 3 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 042ac9e...434abb0. Read the comment docs.


How to execute/call objects via interop?

Let's supposed I have an interop object, a result from the engine.EvaluateScript(...) method as an object. Now let's suppose that result is a JavaScript Function, I have that in C# now.
How can I:

Execute the function in the engine using the object variable that I got?
Or pass the object back to the engine then execute it as function?

For example, in Jint I can do:
var myFunction = engine.Evaluate(...);
engine.Call(myFunction, ...);

Hi @victoriaquasar,

Execute the function in the engine using the object variable that I got?

Script objects are instances of the ScriptObject class, which has methods such as InvokeAsFunction:
engine.AddHostType(typeof(Console));
var func = (ScriptObject)engine.Evaluate("(function(x) { Console.WriteLine(x); })");
func.InvokeAsFunction("Hello, world!");

You can also use C#'s dynamic type:
engine.AddHostType(typeof(Console));
dynamic func = engine.Evaluate("(function(x) { Console.WriteLine(x); })");
func("Hello, world!");

Or pass the object back to the engine then execute it as function?

No problem at all:
engine.Execute("function invoke(x) { x(...Array.from(arguments).slice(1)); }");
engine.Script.invoke(func, "Hello, world!");

Good luck!

Thanks!


feat: add public, preprocessed, and permutation to uni-stark symbolic

It has been mentioned that uni-stark may support permutation traces. If this is the case, I think the symbolic variables and expressions could be shared between uni-stark, sp1, and Valida.
Summary of changes in the PR:
uni-stark:

Add an enum Entry to describe whether the variable is a public input, preprocessed trace,  main trace, permutation trace, or permutation challenge.
Permutation challenges and public inputs have their degree_multiple set to zero.

Air:
Since it's more natural for symbolic builder and other types of builders (such as recursive builder in SP1) to have permutation challenges be a more general value other than a EF element, I have added a ExprRandEF associated type which I requited to be an AbstractField and convert to ExprEF.

Hm this standardization won't be ideal for backends, since they'll end up with some unreachable!() cases, like a backend for SP1 constraints will need to handle public values.
I see how writing very similar symbolic code in SP1 etc. could be painful though, so probably a good tradeoff.

Cleaned up the PR and also removed all the code related to permutation challenges as it seems to be too far off and unrelated.


Delete unnecessary spaces

When copying from the web page, there is an unnecessary space after the code.

@changeworld : Thanks for your contribution! The author, @Yahnoosh, has been notified to review your proposed change.

Verified the updates; merging.


Do not run wrapper task in case there is no upgrade available

I noticed that this plugin will always running the wrapper tasks, regarding of if there is an upgrade available.
However, the plugin nows already if there is an upgrade available.
https://github.com/gradle/wrapper-upgrade-gradle-plugin/blob/e60ef6bce200a2544164d054b7160c9ff51369e8/src/main/java/org/gradle/wrapperupgrade/UpgradeWrapper.java#L243-L251
Here it checks for the current release. If there is no upgrade available, it just returns *the * version (that is, usedBuildToolVersion).
Then it will only use that version to run the wrapper task. Whether if it is the same version or not.
https://github.com/gradle/wrapper-upgrade-gradle-plugin/blob/e60ef6bce200a2544164d054b7160c9ff51369e8/src/main/java/org/gradle/wrapperupgrade/UpgradeWrapper.java#L63-L65
https://github.com/gradle/wrapper-upgrade-gradle-plugin/blob/e60ef6bce200a2544164d054b7160c9ff51369e8/src/main/java/org/gradle/wrapperupgrade/UpgradeWrapper.java#L82-L90
I guess it makes sense to avoid this. It is an unnecessary waste of resources.
Especially since you also recommend running this periodically in the readme:
https://github.com/gradle/wrapper-upgrade-gradle-plugin/blob/e60ef6bce200a2544164d054b7160c9ff51369e8/README.md#L97
In case there is no higher version than the current version, just skip the task execution and thats it 🤷

This is done intentionally, since rerunning the wrapper might reveal outdated script files, missing checksums, etc.

But only the first run/first time, right?
We have to consider two different scenarios here.
The first is, you apply this plugin for the first time and the wrapper doesn't have an upgrade.
In this case the following could happen:

nothing (as everything is up to date and fine)
build scripts might be updated, checksum etc.

All following runs will not change anything! Except there is an upgrade of course.
The second is, that you apply the plugin for the first time and the wrapper does have upgrade.
In this case, well, running the wrapper tasks make sense 🤷♂
So as you can see, only in the first case and  you did something "wrong" when you added the wrapper manually, the indention behavior makes sense.
All other cases are still covered when we apply this pre-check I suggest here.
Let's imagine that it is really the case, that it fixed some build scripts. Further imagine that you run this script each week. Then you still running the wrapper task for "a long time", until Gradle do another release, without doing anything.
I just think that the intended behavior isn't necessary that important to keep it. But this is just my opinion ☺
Feel free to close this issue if I still can't convince you 🙃

I understand what you mean. We do not intend to invest more effort into figuring out if some files need updating or not than our current mechanism does.


Creating splash screen with the app icon in the root folder.

i want to contribute to this issue

@akeelshafi  you can contribute to #3 if yes then ask for assign request there.

@Satyajit0003 Logo files are uploaded in the root directory.

okk

Anu Update ??

Sir, I want to contribute to this and help solve the issue as quickly as possible.

Any Update ?? @Satyajit0003

Working on it


Cooked send's promise is absence when local result is ready.

Identified this with Sagar: In RPCManager::rpc_manager_handler(), if the sender is local sender_id == nid, toFulfillQueue, the queue for promises accepting results, is SOMETIMES empty, which causes crash. The reason is: a message is sent first and then prepare the promises in toFulfillQueue; however, a local node could response so fast that it calculated the result and try to insert its result to the promise before the promise is prepared (by locking the guard on toFulfillQueue). We need to prepare the promise before sending the message.

Closed by fixing #78 and removing possibility of a race condition in using toFulfillQueue and FulfilledList even though that race condition is infinitely rare.


OpenAPI changes for temporal modifiers.

ON HOLD pending further UX/product input. SQL engine changes are mostly finished, so not much flexibility for significant logic changes, but UI design could change quite a bit how we want to persist this in the app DB.
Notes from discussion:

We probably also want to store a flag for whether temporal modifiers are enabled for a particular criteria group section.
Consider how to handle a temporal modifier toggle. AoU collapses any criteria groups together. We could do the same or just keep the top one. This may change how we store the criteria groups (e.g. use the same criteria groups property for default/temporal first condition, or keep just a single criteria groups property and flag each group somehow).
Undecided how to handle the group by modifier. It's not supported in AoU or the current Tanagra SQL implementation. We could possibly support it in the future, but visualizations probably higher priority. Possibilities for now include ignoring it on the backend and/or flagging it somehow on the frontend. Whatever we choose, will probably need to flag this in the config somehow that it doesn't support temporal queries.

@tjennison-google FYI


vet or fix these: ERROR:nnnn expected ">>"

Overview

this is subsumed by #175
todo

[ ] improve error message!
[ ] get this file to parse (or confirm that it is a valid reject) 2020-03-eval/0008_0000bf706...

Details
Suspect these appeared after #168. We need to determine the source of these new rejects, either

to  errors in the new PDF DDL (suspect some are): and fix the DDL
we are enforcing a stricter version of PDF (or at least one that catches errors at parse time rather than run time): and document thes. and update the tests

I'm seeing this error while parsing a (apparently) valid dictionary of /Type /Page.
Files
Files where this is seen
 pdf_tests/2020-03-eval/0008_0000bf706c51686d65b06087f8d09e475c43df25a709a5b8a8c75d3d6b435eef.pdf
 pdf_tests/2020-03-eval/0012_0000ef38c57732e921e320fccf2a06004f75cfbc8071e4ea785ab77f1cf7ec9a.pdf
 pdf_tests/2020-03-eval/0015_0001c61c1d2c2b323a7d13d9ea192bab52af0a635b73b6b24cc38c57f2684367.pdf
 ...

0008: This is apparently caused by the fact that none of the font definitions in this document include a FontDescriptor field, which is both required in the spec and contains information that is important for text extraction.
Given that actual PDF processors seem to accept this, it would indicate that they implicitly use and define a default Font Descriptor. Assuming that we actually want to accept these, we should be able to do something similar without too much pain.

0008: now passes
0012, 0015: are still rejected, but versions with streams decompressed pass. Seems likely that this is caused by some issue independent from fonts.


Client Rust SDK failed to compile in macOS-12

macOS-latest workflows will use macOS-12.
I filed a PR.
I expected to see this happen: All checks are passed.
Instead, this happened: teaclave-client-sdk-macos failed.
I pinned the runner image in #661 to macos-11 and the CI passes.

Rust SGX SDK 2.0.0 cannot be compiled on macOS successfully due to intel instructions. Therefore, the client Rust SDK cannot be compiled, either. Tell us if you have needs for macOS.


Not able to fetch caption for auto-generated subtitles of a video

Not able to fetch caption for auto-generated subtitles of a video as there were some different patterns schema.

Bug-fix


Remove test-results.json from git

This file is necessary for danger-plugin-jest to work correctly, but there's really no need to check it into git and get conflicts on every single PR.

👍

Thanks a lot for contributing to styled-components! I've invited you to our organization, if you have any questions, let me know!

@mxstbr OH! I previously thought it's required. What a happy surprise that we're able to remove it :smile:

Same, it was super annoying though haha


Hosting model on own webserver - relative paths in myAppConfig?

I'm trying to use Web-LLM with a model I got from HuggingFace and created a WASM version of. I want to load that model from the same webserver that the HTML is on. So not from huggingface, and not from a custom local server that emulates huggingface. Just a plain old folder on very cheap shared hosting.
It seems that this doesn't work however:
  // Option 1: Specify appConfig to decide what models to include
  const myAppConfig = {
    model_list: [
      {
        "model_url": "models/TowerInstruct-7B-v0.1-q4f32_1-MLC/",
        "local_id": "TowerInstruct-7B-v0.1-q4f32_1",
        "model_lib_url": "models/TowerInstruct-7B-v0.1-q4f32_1-MLC/TowerInstruct-7B-v0.1-q4f32_1.wasm",
      },
    ]
  }
  const selectedModel = "TowerInstruct-7B-v0.1-q4f32_1";

Are relative paths not supported?
index.js:2280 Uncaught (in promise) Error: Cannot find global function vm.builtin.param_module_from_cache_by_name

Interested in this as well.


Added url shortener in next js

Closes:   #627

Title :  url shortner in Next.js
Name: :  Aditi Jain
Idenitfy yourself: :  GSSOC

How Has This Been Tested? ⚙

MYSELF
Screenshots 📷

Type of change ☑

What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist: ☑

[x] My code follows the Contributing Guidelines & Code of Conduct of this project.
[x] This PR does not contain plagiarized content.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] My changes generate no new warnings.

Note to reviewers 📄

@Avdhesh-Varshney sir please merge my pr


Link broken?

It looks like this link in the dependencies no longer functions (did the code get moved somewhere maybe?):
PLUMED (ANN included): https://github.com/weiHelloWorld/plumed2

Sorry I have moved it to another repository without updating README: https://github.com/weiHelloWorld/plumed_additional.  If you would like to use plumed version of ANN force, you need to checkout https://github.com/plumed/plumed2 and then copy ANN.cpp to plumed2/src/function.   Originally I implemented it as a CV, but I found it would be better to implement it as a general function.  Let me know if you have any questions.


TypeOf utility for extracting optionally narrowed types from data

function typeFromInterface<T>(def: DefOf<T>): Type<T> {
  return type(def as Infer<T>)
}
type describedProp = {
  /** Email of user */
  'email': 'email',
}
type DefOf<T> =
| T extends string ? 'string'
| T extends any[] ? [...]
: T extends Record<keyof any, any> ? {
  [K in keyof Required<T> as IsOptionalKey<T, K> ? `${K}?` : K]: DefOf<T[K]>
}


Move content from project page to districthousing.org

As a person directing someone to the project, I want to provide a short url so that it will be easy to remember.
For example, could we move our app to app.districthousing.org and our project page to districthousing.org (or www.districthousing.org)?

I'm going to close this for now. It's a good idea to make this more delineated, but until we have more info at the public URL, it won't be a priority.


Automate Building of Linux Client

This automated both the building of the game launcher, a custom Unreal Engine Development image, and using that image to build both the client build and the dedicated server image - and storing all these artifacts in Cloud Storage.
Closes #113
Wow - finally got this done after about 3 months of work!

Awesome!


Create an SAPUI5 App from the Command Line

Tutorial URL: https://developers.sap.com/tutorials/cp-cf-sapui5-local.html
Step 5: Deploy the MTA archive.
I always get this error:
Service operation failed: Controller operation failed: 502 Updating service "dest_SecondApp" failed: Bad Gateway: Error creating service "dest_SecondApp" from offering "destination" and plan "lite": Service broker error: Quota limit exceeded. Instance creation not allowed.
I tried to set 256M, 512M, 1024M.
But from a remote Web IDE, deployment is working fine with 256M.

Hi,
I understand your confusion. The term "quota" refers to the service instance in this case (and not to the memory of your app runtime. The trial account only allows you to have two "destination service instances" in use at a time. You can fix the issue by removing the services instances via the SCP cockpit.
Please let me know if it worked

Thank you very much!
This solved my problem.
Maybe this link is also useful to someone: Delete Service Instances Using the Cockpit


chore: adapt to springboot 2.4.9 - reverted junit to v4

@jacobdotcosta Why keep using junit 4 ? You have created another PR to fix the branch to use junit 5: https://github.com/snowdrop/circuit-breaker-example/pull/63.
From my point of view, the PR to use junit 5 should be ok.
However, the openshift tests fail in both PRs.
cc @aureamunoz

@jacobdotcosta Why keep using junit 4 ? You have created another PR to fix the branch to use junit 5: #63. From my point of view, the PR to use junit 5 should be the right way. However, the openshift tests fail in both PRs. cc @aureamunoz

This PR is to be abandoned. I'll close it.


New submission

Looks good, but I'm getting 1080 bytes and your filename needs to be yourdomain.com.md

FYI: https://gtmetrix.com/reports/kjvscripturejournals.com/YA4h138q/

On Aug 4, 2022, at 12:11 PM, Bradley Taunt @.***> wrote:
I'm getting 1080 bytes

Oddly enough, the compressed version is 1080 while the uncompressed version in only 990. I’m not sure why just yet.
Jeremy Sarber
https://jeremysarber.com


Fixed tech ranking

Too few countries and in too obscure conditions would get higher tech levels - most notably access to mechanical_production - and this would deadlock entire economy until 1840s. On a test example two OPMs were only countries with high enough industry score to break 0.6 required for machine parts. Another example had a single 2-state country.
I've swapped normalized rankings for a score system where all civilized nations are linearly distributed across the range (and can share spots), so this will greatly increase tech diversity and tech availability.
Uncivilized nations are not affected.

General design sounds correct. The old design was from EU3 to Vic2, where there was almost always a normal-ish distribution of tech scores (with a long tail of those who had fallen behind). Will review code momentarily.


fix s3 tag forcelly rewrited

if s3 tag contains: [tag1, tag2, tag3], but the crossplane s3 config tag
is [tag1, tag2], provider-aws will forcelly rewrite the tags with [tag1,
tag3]. This is not as expected in some situations.

provider-aws will forcelly rewrite the tags with [tag1, tag3].

[tag1, tag2] ?

provider-aws will forcelly rewrite the tags with [tag1, tag3].

[tag1, tag2] ?

yeah, thanks


swift 3.0

Adds support for Swift 3.0

@yaakovgamliel thanks for your collaboration. However, we've already migrated the library to Swift 3 in this pull request.

Np I'm starting to use the EventSource heavily at work so I'm happy to help

@yaakovgamliel that's great! let us know how it goes!!!


Oxen rebrand

Rebranding Loki -> Oxen
App compiles and runs on the ledger, images still need updating. Also need to do further testing

Loki rebranded to Oxen, Icons and images changed to Oxen assets and tested on most recent 9.0.0 dev branch


Cluster Tags for better identification

Hi guys
Target : Have grafana dashboard per cluster type, availability zone and region or other tags. I would like a label for these informations.
Why : in some organizations we can have multiple vCenters in different region or country. In each vCenter we can have different use cases or clusters type : Oracle Cluster, Customer Clusters, Test Clusters, Windows Clusters, etc. Separate clusters based on different label for capacity planning, troubleshooting, reporting purposes in dashboards should be a very usefull   feature.
How :
To do that i thought 3 options :

we can get all tags per object at the begining of the process and we can define labels for each metrics like perf metrics for vms which is more dynamic. As this modification is quite big, do you think it's a good idea  ?

rather that add dynamically tags in label, we  could define some extra_labels in the config.yml and each label has to be a tag in vcenter. The value of tag will be the label value or None if the tag is not found on the object. It is more static than the first solution and require a mapping between tag defined in vcenter and extra_labels defined in config.yml but easier to implement with no mass change in actual code.

last option and i think better one, As the tag is on cluster level, we can also implement a new collector for cluster to collect tags and maybe few metrics if needed (evc / ha) for example.
With that it will be easily to map cluster with tags (az,region, etc) and host/vm with cluster as the label cluster is already collected.
With that it will be easy to have cluster Dashboard with all information of cluster and hosts/vms

What do you think ?
Thanks for your answers

How are you running the exporter and scraping from it?

Running the exporter in a container and displays metrics in grafana.

Typically you can use telegraf to inject tags. You can also do relabels in prometheus to change information that was imported. For instance, snmp_exporter has an example where it changes the labels for targets.

do you recommend using telegraf for tags rather than doing it in the exporter ?

For now yes. I'm not sure If I will be able to get to adding tags in the near future.

I will try to do it soon and let you know


support ACME Tls Support

Manual ACME is already possible as it's just about serving a file to proof domain control.
However might be even better if we can support automated ACMA support and roll the certs like that.
We have no immediate need for it however, so this ticket is open for others to pick up.

Deuplicated by https://github.com/plabayo/rama/issues/313


new(falcosidekick): add nodeSelector, tolerations and affinity values for test connection pod

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If this PR will release a new chart version please make sure to also uncomment the following line:

/kind chart-release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area falco-chart

/area falco-exporter-chart

/area falcosidekick-chart

/area event-generator-chart

What this PR does / why we need it:
This adds the node selector, tolerations and affinity value options to the test-connection pod.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Checklist

[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] CHANGELOG.md updated

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: SEANDOUGHTY
Once this PR has been reviewed and has the lgtm label, please assign cpanato for approval by writing /assign @cpanato in a comment. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 0e2adaeceada95213e02ffd3025fc11704b97f5e

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Issif, SEANDOUGHTY
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Issif]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Implement Analytics

We need to track the status of our library in deployment. Analytics will help ensure that events are logged as desired and error rates are low.
Initial implementation will log:

Number of events logged
Number of bytes logged
Number of each type of error received

This is now implemented in 0.4.1.


K8s v1.19.9 is Out of Support for Azure

K8s v1.19.9 is out of support for Azure; v1.20.9 is their current default, and they have v1.22.2 in preview.
The prereq page indicates a requirement for 1.19.9.
I seem to be able to get a IAC and Deployment to run with v1.19.11.
The IAC will build with v.1.20.9, but the Deployment will fail:
TASK [monitoring : cluster-monitoring - deploy] ********************************
fatal: [localhost]: FAILED! => {"changed": true, "cmd": ["/tmp/ansible.guo067c2/viya4-monitoring-kubernetes/monitoring/bin/deploy_monitoring_cluster.sh"], "delta": "0:00:01.368033", "end": "2021-11-17 15:21:42.859281", "msg": "non-zero return code", "rc": 2, "start": "2021-11-17 15:21:41.491248", "stderr": "", "stderr_lines": [], "stdout": "User directory: /tmp/ansible.guo067c2\nHelm client version: 3.7.1\nKubernetes client version: v1.19.9\nKubernetes server version: v1.20.9\n\nnamespace/monitoring created", "stdout_lines": ["User directory: /tmp/ansible.guo067c2", "Helm client version: 3.7.1", "Kubernetes client version: v1.19.9", "Kubernetes server version: v1.20.9", "", "namespace/monitoring created"]}

PLAY RECAP *********************************************************************
localhost                  : ok=43   changed=11   unreachable=0    failed=1    skipped=37   rescued=0    ignored=0

This is an error the monitoring and logging team are working on that item is in this repo: https://github.com/sassoftware/viya4-monitoring-kubernetes So if you can pass the error along there.
If you have an issue with the v1.19.9 I can fix that. Please adjust the information here. Thx.

I was able to deploy with K8s v1.20.09 by removing the cluster-monitoring from the deploy command.  This will work for the interim until the monitoring app will support with a higher K8s version.
PS, I also tested by using K8s v1.19.11, which exceeds the requirements listed in the link above, but is allowed by Azure and it also failed.
Closing issue.

New issue logged, https://github.com/sassoftware/viya4-monitoring-kubernetes/issues/229

The monitoring deployment issue has been fixed with our 1.1.1 release.


fix support for datatype CHOICE

Thanks for the contribution! I've added some comments re: code style and others may have comments as well.

 (After doing that you'll need to push -f to force it.) If we merge this right now it will just show up as "my commit" in the history.

Closing this in favor of #66.


The reward for not completing the next quest stays stuck here.

The reward for not completing the next quest stays stuck here.
You can only do this by re-running the software
After restarting the software and continuing that reward

same╮(╯▽╰)╭

This is twitch drop bug, you cannot get twitch drop of Sea of thieves even use idle it on broswer.

This is twitch drop bug, you cannot get twitch drop of Sea of thieves even you use idle it on broswer.

That's right.


Hi, will export to QLinear save weights in int8?

Using tensorrt backend, will QLinear make the onnx model smaller?
I got some error when trying to save to QLinear:
deploy/common.py", line 138, in optimize_model
    assert node_detect, "Graph is illegel, error occured!"
AssertionError: Graph is illegel, error occured!

ONNX_QNN backend will save model weights in INT8, so make the model smaller.
Any model file or something to reproduce the bug ?

@Tracin  Hi, I am trying quantize a simple resnet18 model, I am using QNNX_QNN, it seems can sucssfully trace and insert FakeQuantize ops, but finally can not pass QNNX_QNN pass:
deploy/deploy_onnx_qnn.py", line 262, in format_qlinear_dtype_pass
    scale_proto = self.onnx_model.initializer[scale][0]
KeyError: '::FixedPerTensorAffine_400'

what could be the reason?
More log like:
WARNING: The shape inference of ::FixedPerTensorAffine type is missing, so it may result in wrong shape inference for the exported graph. Please consider adding it in symbolic function.
WARNING: The shape inference of ::FixedPerTensorAffine type is missing, so it may result in wrong shape inference for the exported graph. Please consider adding it in symbolic function.
WARNING: The shape inference of ::FixedPerChannelAffine type is missing, so it may result in wrong shape inference for the exported graph. Please consider adding it in symbolic function.
WARNING: The shape inference of ::FixedPerTensorAffine type is missing, so it may result in wrong shape inference for the exported graph. Please consider adding it in symbolic function.
WARNING: The shape inference of ::FixedPerChannelAffine type is missing, so it may result in wrong shape inference for the exported graph. Please consider adding it in symbolic function.
WARNING: The shape inference of ::FixedPerTensorAffine type is missing, so it may result in wrong shape inference for the exported graph. Please consider adding it in symbolic function.
WARNING: The shape inference of ::FixedPerTensorAffine type is missing, so it may result in wrong shape inference for the exported graph. Please consider adding it in symbolic function.
WARNING: The shape inference of ::FixedPerTensorAffine type is missing, so it may result in wrong shape inference for the exported graph. Please consider adding it in symbolic function.
04.19 14:34:57 INFO convert_d...py]: <function deploy_qparams_tvm at 0x7fac38196790> BackendType.ONNX_QNN
04.19 14:34:57 INFO convert_d...py]: Convert to ONNX QNN.

It is not right absolutely, any simple code to reproduce the error?
The exported ONNX model using standard onnx opset 11, should be able to inferenced via ORT.

@Tracin I am not sure my code can run on your side or not, since I modified some code in mqbench to support torch1.11, (this is the main purpose I trying mqbench for decent pytorch support).
Here is the process I runed:
extra_prepare_dict:
    extra_qconfig_dict:
        w_observer: MinMaxObserver
        a_observer: EMAMinMaxObserver
        w_fakequantize: FixedFakeQuantize
        a_fakequantize: FixedFakeQuantize
        w_qscheme:
            bit: 8
            # symmetry: False
            symmetry: true
            per_channel: True
            pot_scale: False
        a_qscheme:
            bit: 8
            # symmetry: False
            symmetry: true
            per_channel: False
            pot_scale: False
quantize:
    quantize_type: naive_ptq # support naive_ptq or advanced_ptq
    cali_batchsize: 16
    # backend: 'Tensorrt'
    backend: 'ONNX_QNN'
    # backend: 'PPLW8A16'
    deploy:
        model_name: 'r18.onnx'
        output_path: './'
        deploy_to_qlinear: true
model:                    # architecture details
    type: resnet18        # model name
    kwargs:
        num_classes: 1000
    path: /path-of-pretrained
data:
    path: /path-of-imagenet
    batch_size: 64
    num_workers: 4
    pin_memory: True
    input_size: 224
    test_resize: 256
process:
    seed: 1005

Code:
if __name__ == '__main__':
    train_loader, test_loader = prepare_dataloader()

    config_f = sys.argv[1]
    config = parse_config(config_f)
    print(config)
    # first finetune model on cifar, we don't have imagnet so using cifar as test
    model = resnet18(pretrained=True)
    model.fc = nn.Linear(512, 10)
    if os.path.exists("r18_raw.pth"):
        model.load_state_dict(torch.load("r18_raw.pth", map_location="cpu"))
    else:
        # train_model(model, train_loader, test_loader, device)
        print("train finished.")
        # torch.save(model.state_dict(), "r18_raw.pth")
    model.to(device)
    model.eval()

    if hasattr(config, 'quantize'):
        model = get_quantize_model(model, config)
        print('now model in quantized mode.')

    model.to(device)
    evaluate_model(model, test_loader)

    # evaluate
    if not hasattr(config, 'quantize'):
        evaluate_model(model, test_loader)
    elif config.quantize.quantize_type == 'advanced_ptq':
        print('begin calibration now!')
        cali_data = load_calibrate_data(test_loader, cali_batchsize=config.quantize.cali_batchsize)
        from mqbench.utils.state import enable_quantization, enable_calibration_woquantization
        # do activation and weight calibration seperately for quick MSE per-channel for weight one
        model.eval()
        enable_calibration_woquantization(model, quantizer_type='act_fake_quant')
        for batch in cali_data:
            model(batch.cuda())
        enable_calibration_woquantization(model, quantizer_type='weight_fake_quant')
        model(cali_data[0].cuda())
        print('begin advanced PTQ now!')
        if hasattr(config.quantize, 'reconstruction'):
            model = ptq_reconstruction(
                model, cali_data, config.quantize.reconstruction)
        enable_quantization(model)
        evaluate_model(model, test_loader)
        if hasattr(config.quantize, 'deploy'):
            deploy(model, config)
    elif config.quantize.quantize_type == 'naive_ptq':
        print('begin calibration now!')
        cali_data = load_calibrate_data(test_loader, cali_batchsize=config.quantize.cali_batchsize)
        # do activation and weight calibration seperately for quick MSE per-channel for weight one
        model.eval()
        enable_calibration_woquantization(model, quantizer_type='act_fake_quant')
        for batch in cali_data:
            model(batch.to(device))
        enable_calibration_woquantization(model, quantizer_type='weight_fake_quant')
        model(cali_data[0].to(device))
        print('begin quantization now!')
        enable_quantization(model)
        # print(model)
        evaluate_model(model, test_loader)
        if hasattr(config.quantize, 'deploy'):
            deploy(model, config)
    else:
        print("The quantize_type must in 'naive_ptq' or 'advanced_ptq',")
        print("and 'advanced_ptq' need reconstruction configration.")

the model traced seems normally on saved onnx with Fake ops:

But, the problems is on the QNN pass can not get pass.
And I found these comented code will raise a Assertion:

any solution to fix this?

Resnet18 can pass the test in https://github.com/ModelTC/MQBench/blob/main/test/backend/test_backend.py#L120-L131
Try chaning the model definition using in the testcase ?

@Tracin May I ask when will pytorch1.11 support? I do belive your test can run, as I mentioned before, I am runing on pytorch1.11 for upgradation on our product.
However, I found the problem caused be initializers. The initializers still old names:
def format_qlinear_dtype_pass(self):
        print(type(self.onnx_model.initializer))
        print(self.onnx_model.initializer.keys())
        for node in self.onnx_model.graph.node:
            if node.op_type in FAKE_QUANTIZE_OP:
                print(node)
                scale, zero_point, qmin, qmax = node.input[1], node.input[2], node.input[3], node.input[4]
                logger.debug(f'scale, z, qmin, qmax: {scale}, {zero_point}, {qmin} {qmax}')
                qmin = self.onnx_model.get_constant(qmin)
                qmax = self.onnx_model.get_constant(qmax)
                assert qmax - qmin == 2 ** 8 - 1, "Only 8 bit quantization support deploy to QNN."
                scale_proto = self.onnx_model.initializer[scale][0]
                if scale_proto.raw_data != b'' and scale_proto.dims[0] == 1:
                    scale_data = self.onnx_model.get_initializer(scale)
                    self.onnx_model.set_initializer(scale, scale_data.astype(np.float32), raw=False)
                zero_point_proto = self.onnx_model.initializer[zero_point][0]
                zero_point_data = self.onnx_model.get_initializer(zero_point)
                # Align sym and asym scheme.
                zero_point_data = (zero_point_data - qmin).reshape((1,))
                self.onnx_model.set_initializer(zero_point, zero_point_data.astype(np.uint8), raw=False)

initializers still old names:
dict_keys(['conv1.weight', 'conv1.bias', 'layer1.0.conv1.weight', 'layer1.0.conv1.bias', 'layer1.0.conv2.weight', 'layer1.0.conv2.bias', 'layer1.1.conv1.weight', 'layer1.1.conv1.bias', 'layer1.1.conv2.weight', 'layer1.1.conv2.bias', 'layer2.0.conv1.weight', 'layer2.0.conv1.bias', 'layer2.0.conv2.weight', 'layer2.0.conv2.bias', 'layer2.0.downsample.0.weight', 'layer2.0.downsample.0.bias', 'layer2.1.conv1.weight', 'layer2.1.conv1.bias', 'layer2.1.conv2.weight', 'layer2.1.conv2.bias', 'layer3.0.conv1.weight', 'layer3.0.conv1.bias', 'layer3.0.conv2.weight', 'layer3.0.conv2.bias', 'layer3.0.downsample.0.weight', 'layer3.0.downsample.0.bias', 'layer3.1.conv1.weight', 'layer3.1.conv1.bias', 'layer3.1.conv2.weight', 'layer3.1.conv2.bias', 'layer4.0.conv1.weight', 'layer4.0.conv1.bias', 'layer4.0.conv2.weight', 'layer4.0.conv2.bias', 'layer4.0.downsample.0.weight', 'layer4.0.downsample.0.bias', 'layer4.1.conv1.weight', 'layer4.1.conv1.bias', 'layer4.1.conv2.weight', 'layer4.1.conv2.bias', 'fc.weight', 'fc.bias'])

what might miss here?

It is difficult to reproduce your code, to make things simple, can the specific test case pass in torch 1.11?
If not, I would reproduce the bug easily, otherwise it might be wrong in other parts of code.

@Tracin You can have test on pytorch1.11. A lots of things changed  including onnx export, fx etc.
I think I know where the problem is.

@Tracin You can have test on pytorch1.11. A lots of things changed including onnx export, fx etc.
I think I know where the problem is.

It will take some time to test.
The problem is about the version or not ?

@Tracin yes. torch1.11 removed a lot of APIs used in mqbench.


🐛 [firebase_analytics] PageView pages are not tracked out of the box as a screen_view event in Analytics

Hello,
I use PageView extensively in my app. I noticed the pages that the user views are not tracked in Firebase Analytics. Any tips on how to get individual pages in a PageView tracked as a screen_view event? It seems only Navigator namedRoutes are tracked out of the box, but PageView is another frequently used method to navigate in an app.
PageView link https://api.flutter.dev/flutter/widgets/PageView-class.html
Firebase Analytics https://pub.dev/packages/firebase_analytics

Hi @giorgio79
Please see the example app for a demonstration on how to use this functionality
https://github.com/FirebaseExtended/flutterfire/tree/master/packages/firebase_analytics/firebase_analytics/example/lib
Thank you


upping model download timeout and connection values

/cc @rhatdan @lmilbaum
Addresses: https://github.com/containers/ai-lab-recipes/issues/327#issuecomment-2074934943

I think curl has also a retry flag. Would you mind looking into it if this download issue persists? Not a blocker for this PR.

LGTM


README: Update state example

Add the recently added Ethernet properties.

Coverage remained the same at 73.048% when pulling c72c20963681f09efc133c2dc2cb840db87fa34a on tyll:README into e411aca88749d4a16eb85d17ed887ec892d307c1 on nmstate:master.

Updated the commit message


NETFramework

NuGet package not installing for NETFramework

@miroiu
This seems mostly a case of publishing the version of Nodify with support for multiple frameworks.
It might be worth it to consider strong name signing the assembly, see Microsoft recommedations for more info. It should not break anything, although it could break things when going back.
If I have some time this week I can provide the basic implementation(without the keys of course 😉).

Hi,
The problem was that the latest NuGet package at that time (v1.6.1) did not include .NET framework targets.
@ruslan79k please let me know if the new version (v1.6.2) works for you.
Sorry for the delay.

@labiej
That's a good idea. I was looking into it a while back but was afraid that I may break something.

@miroiu
Sorry for not responding sooner, I've been busy.
Regarding breaking things, unless I'm missing something strong naming an assembly shouldn't break anything.
It can cause issues when the key itself changes because that will change the strong name.
I did find the time to do some more digging especially how to handle the key file itself.

Signing an open source library is recommended by Microsoft here
It is recommended to add the 'private' key to the repository here
the main takeaway is that strong naming is about avoiding DLL hell, not security.

I did implement it already, when I've updated my fork I'll create a pull request.

@labiej
No worries. Thanks for your time and the PR!
I will close this issue as it was fixed with the 1.6.2 release.


Add format option.

I found I needed to set the format of the translation string to preserve line breaks in the text (my clients can't write html!).
So I've added it in as an option inside the configuration - but it could also be done as an option on the text field itself.

Hey @aadgrant, thanks for this PR and sorry I never picked it up for the Studio v2 version of the plugin.
I'm working on releasing the Studio v3 version now. If you'd like to submit again for that version we can merge it in!


(maint) Fix test failure resulting from multiple merges

A change to the Executor's default arguments and testing of log messages
were both merged independently, introducing a test failure that wasn't
caught by PR testing. Fix the test.

CLA signed by all contributors.


Just confirming the discord message

Just posting this to confirm the discord message

I'm not a discord user, you did a mistake.

No worries, someone messaged me about build 1224 claiming to be you and I just wanted to make sure. Thank you for letting me know

Interesting ... Someone calls himself as myself?

Yes the account ID is gildor2#3518

Posted a warning message on the forum and facebook page.
Just curious what that "gildor" does there?

They only joined today, I think that they where trying to use your name to get more information from the PUBG datamining group about how the modified version of umodel (1224) works for viewing pubg mobile files


zeranoe -> gyan.dev build

Rip zeranoe

Any idea for mac os ffmpeg?

...I think change to main download page https://ffmpeg.org/download.html is best

Ty


Mappers: Increase the code point limit for SnakeYAML to 2GB

Increase the code point limit because ORT result files can exceed 1GB for large projects.
Fixes #6316.

@sschuberth @fviernau I had to switch to Integer.MAX_VALUE because 2.gibibytes.toInt() causes an integer overflow. 2.gibibytes.toInt() - 1 would work as well if you would prefer that?

2.gibibytes.toInt() - 1 would work as well if you would prefer that?

No, as IMO Integer.MAX_VALUE makes more clear that this is the maximum supported value here.


STYLING: Dropdown menu on Navbar

Total failure, wasted all day for nothing. Should've first thought about whether was gonna be needed. Anyway, at least I discovered ChakraUI and started to fall in love with it


URGENT UPDATE: fix issue with bar data after alpaca introduced adjustments

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #24

PR #25

PR #26

PR #27   👈

This stack of pull requests is managed by Graphite.

Merge activity

Feb 18, 4:21 AM EST: @raulfrk started a stack merge that includes this pull request via Graphite.


Why use a random conditional during training?

Maybe I'm wrong, but it looks to me like the conditional used for training is randomly generated... Shouldn't it be taken from the dataset?

It should be obtained from the data set, and a label is fixed this time to facilitate the visualization of the training process.

Thanks very much, that clears things up


Descriptor and shceme update

Consider redesign scheme to use inline class

Does not make sense without https://github.com/Kotlin/KEEP/pull/176 and/or some kind of compiler plugin. In order to access inner properties, we need to implicitly pass both specification for those properties and a config node. For that we need functions with two receivers.


Implement detection of oversize SCP's before the build and deployment steps.

Is your feature request related to a problem? Please describe.
The customizations pipeline takes a long time to deploy in our organization. We have many SCP's.
If an SCP exceeds 5120 bytes only then, late within the deployment, do we get notified of this problem.
Describe the feature you'd like
My team came up with two ideas to mitigate this.
The first one is an automatic minimization of the SCP json files. This reduces SCP size. Deploying a SCP using the web console has this behavior.
Second can we be notified of SCP's that will fail the deployment due to it being larger than 5120 bytes, before the build and deploy steps. For example, an extra pipeline step which does a minimization/sanity check for size. This check could auto complete if no issues were found but require manual approval if a problem related to size is found. Therefore, blocking the deployment.
Additional context
No context.

@Skripd Thank you for the feature request.
I have created an internal backlog to track this request.


Want to use vedo to display and reconstruct 3D CT images

I am an active beginner in vedo and would like to use vedo to display and reconstruct 3D CT images.
I saw a demo on the official website about the head，

I have a question about how to make a vti file like this. My image is a JPG.
Could you give me some advice, I would be grateful !!!

Hi, a JPG image is a 2D object. What you need instead is a 3D image (a vedo.Volume()).
A 3D image can be seen as a set of 2D slices one on top of the other.

Yes, I have a set of CT slices (JPG images), which are converted into 3D images. Is there a special format for vti?

I completed the rebuild but there is one small problem，When I zoom in, could I make the edges of the pixels disappear?

Sorry I missed this issue! Have you tried .linewidth(0) ?


JSON schema error while running action

On running the action, encountered the following error:
Error: no schema with key or ref "http://json-schema.org/draft-07/schema#"

Same: https://github.com/google/automotive-design-compose/actions/runs/6951328475/job/18913118404

Same: https://github.com/shipmonk-rnd/doctrine-hint-driven-sql-walker/actions/runs/7044685714/job/19172875221?pr=6


Timestamp format

Timestamp should use consistent time since epoch across all OSCP services. Seconds should be sufficient precision for SSRs.

Fixed in master, ready for test. Thanks !


just variable to be exported as environment variables (array)

Hi casey,
is it possible to export a just variable as an array? Something like this?
set export := true
ips        := '(
<IP_ADDRESS>
<IP_ADDRESS>
)'

My plan is to define this array and use it in the recipes.
Kind regards, Thomas

Currently not, and I'm not sure it's possible. just exports variables to the shell as environment variables, and I believe that it's only possible to export variables as strings, and not as lists, which are only supported by certain shells.

Most modern shells (including most versions of bash, csh, ksh, tcsh, and zsh) support arrays or lists. But the Just default shell sh (Bourne Shell) does not. In most systems sh is actually an alias to ash, bash, or dash. These shells usually run in POSIX mode when accessed as /bin/sh and the POSIX shell definition doesn't support arrays or lists so even when the native shell supports arrays it's not usually available as sh. So supporting arrays/lists by default would be problematic at best. Though maybe it could be allowed via a settings such as set allow-array-vars when changing the default shell?

@runeimp It's true that many shells support arrays. However, just is invoking the shell as a subprocess, so there would need to be some way to pass arrays through environment variables to the shell. Is that possible? I thought environment variables could only be strings.

Most shells have an option to specify a dot file to load with the shell invocation.
All shell variables are environment variables if they are exported, no? Originally they were always strings back in the AT&T UNIX days. Arrays were a feature added to later shells. Original UNIX shells didn't have arrays. Which, of course, is why the PATH variable uses semicolons as a separator character on Windows and colons everywhere else. Sort of the original CSV of the time.
I did some testing and Bash (my primary shell) had no problem with loading arrays from it's dot files. Unfortunately my environment is a bit screwy at the moment though and I could not get my other shells to successfully read new variables from their respective dot files. But I would be surprised if the majority had any problems with arrays.
The bigger issue to me is simply that the default is Bourne / POSIX Shell and many semi-popular shells don't support arrays or lists. So it's definitely tricky either way.

Apologies for dragging this one up from the past, but is there any further guidance on this?
.env:
export PROVIDER=(1, 2, 3)
``

justfile:

```just
set shell := ["bash", "-uc"]
set dotenv-filename := 'provider.env'

But it fails with the error:
error: Failed to load environment file: Error parsing line: '("1" "2" "3")', error at line index: 5

Just is at 1.35.0.
Is it expected to work, or am I reading the issue incorrectly?

I guess that export PROVIDER=(1, 2, 3) isn't valid syntax for a .env file. But more importantly, even if it was, shells that support array variables only support them for local variables, and can't import and export them.

@ross-spencer If you do export PROVIDER='1;2;3'; (Windows) or export PROVIDER='1:2:3'; (non-Windows) some shells will interpret PROVIDER as an array automatically. You seem to be using Bash which may work depending on the version. I think Bash 4 was the first version with arrays. But I could be wrong.


amazon-echo-hub-node issues with auto discovery and docker ?

I am trying to run nodered in docker and get device detection to work.
Following the examples I understand that I am to use network_mode: host
amazon-echo-hub-node is set to port 80
(It would be nice to be able to set the ip address here for people with multiple nics. I had to add the interface/ip to the http server in the js code)
My main issue and one I don't understand how others got around is this.
amazon-echo-hub-node when is runs a ssdp server will use port 1900
and as we use host networking that port will be used on the host machine
I get EADDRINUSE <IP_ADDRESS>:1900
because docker itself uses that port
ie
sudo netstat -tulpn | grep :1900
udp        0      0 <IP_ADDRESS>:1900            <IP_ADDRESS>:*                    2075/docker-proxy
udp6       0      0 :::1900                 :::*                                2083/docker-proxy
How have people got around this ?
Thanks

I can't figure this one out either at the moment.  My problem goes 1 level deeper.  I'm running the NR docker on a NAS machine.  I can't figure out how to start NR in the docker container running as root in order to even try to deploy the node to listen on Port 80.  I get permission denied, which I've read is because i'm not running as Root.  I just don't have enough experience with docker to figure this one out atm.

Related to issue #200


chore: limit requests library version under 2.32

As title says - limiting requests library version

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.59%. Comparing base (f734977) to head (85b7234).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1203   +/-   ##
=======================================
  Coverage   90.59%   90.59%
=======================================
  Files          93       93
  Lines        7015     7015
=======================================
  Hits         6355     6355
  Misses        660      660

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


Move API draft specifications into their own repository.

Need a name for this repo if anyone has any suggestions.
As a reminder these are API definitions that will be implemented by multiple engine implementations (such as infix, verity and aperture) to allow them to interoperate, but they are not required to be implemented by all engines, which is why they don't belong in the dogmatiq/dogma repo.
I'm not sure if they deserve a blahkit style name, because there's not really any code in them that makes them a "kit" for doing anything.

github.com/infix/draftspecs/envelopespec has moved to https://github.com/dogmatiq/envelopespec
github.com/infix/draftspecs/messagingspec has moved to https://github.com/dogmatiq/transportspec


add socks5 proxy 👻

Why
Right now for some network issues, I couldn't use this plugin to fetch oembed code that providers return directly. This is a very common issue for many Chinese users and it caused by the GFW.
To solve this, I have to hard code the configuration in fetchOembed.js  so that the axios could use socks5 proxy to request the data successfully.  Otherwise, I would get ETIMEDOUT errors.
original:
//<EMAIL_ADDRESS>const axios = require("axios");

const fetchOembed = async endpoint => {
  const response = await axios.get(endpoint.url, {
    params: {
      format: "json",
      ...endpoint.params
    }
  });
  return response.data;
};

module.exports = fetchOembed;

My workaround:
// modified<EMAIL_ADDRESS>const axios = require("axios");
const SocksProxyAgent = require("socks-proxy-agent");

const fetchOembed = async endpoint => {
  const proxyHost = "<IP_ADDRESS>", proxyPort = "9999";
  const proxyOptions = `socks5://${proxyHost}:${proxyPort}`;
  const httpAgent = new SocksProxyAgent(proxyOptions);
  const httpsAgent = httpAgent;
  const response = await axios.get(endpoint.url, {
    params: {
      format: "json",
      ...endpoint.params
    },
    httpAgent: httpAgent,
    httpsAgent: httpsAgent
  });
  return response.data

However, these changes will be overwritten whenever yarn updates.
Solution
The right way to solve this is by forking the repository to customized gatsby-remark-oembed.   I add a new option named useProxy  to custom the proxy that axios uses.
So I could use it like using usePrefix.
 options:{
 useProxy: {
       host: '<IP_ADDRESS>',
        port: '1234',
        // useSocks5 defaults to true
         useSocks5: false // this will use  axios proxy config instead of socks5
       }
}

If useProxy is missing, the proxy is disabled as expected.
 options:{
 useProxy:  false, // -> disable explicitly
}

Files and test
I've modified amendedOptions.js and created a file named getProxyAgent.js to handle above processing.
In order to maintain the stability of the plugin, I also created getProxyAgent.test.js and all tests were passed. In addition, some test cases were also added to other .test.js files.
code review is needed
I don't know whether some of my code is necessary to achieve this feature.

If I remember correctly from another project it is possible to "highjack" Axios globally. If so that might be a better approach so we do not have to change it in several places.

Hi @tizee, Nick here, a new maintainer on the project. Thanks for the PR.
Would you mind opening an issue for this so that we can discuss the issue in there? From what I can tell, the issue is pretty much your PR description, but it's always nice to have issues to track these things.
I haven't used Axios myself, just good old fetch @raae, but I agree that changing it in several places to use the proxy will add a bit of a maintenance burden which we should try to avoid if possible.

Hi @tizee, Nick here, a new maintainer on the project. Thanks for the PR.
Would you mind opening an issue for this so that we can discuss the issue in there? From what I can tell, the issue is pretty much your PR description, but it's always nice to have issues to track these things.
I haven't used Axios myself, just good old fetch @raae, but I agree that changing it in several places to use the proxy will add a bit of a maintenance burden which we should try to avoid if possible.

Sorry for  the late response,  the issue was due to poor network connection and I have managed to address this by setting up a VPN.  To keep it simple,  I'd like to close this PR now.

Okey, thanks for closing!


(refactor)fix_vulnerability: enable autoescape for yaml template

Signed-off-by: ksatchit<EMAIL_ADDRESS>
What this PR does / why we need it:
Adopts a more recommended way to load the jinja templates, as per jinja2 documentation:
Notes on Autoescaping

In future versions of Jinja2 we might enable autoescaping by default for security reasons.
As such you are encouraged to explicitly configure autoescaping now instead of relying on the default.

Uses select_autoescape with template type as yaml

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #889
Checklist

[ ] Does this PR have a corresponding GitHub issue?
[ ] Have you included relevant README for the chaoslib/experiment with details?
[ ] Have you added debug messages where necessary?
[ ] Have you added task comments where necessary?
[ ] Have you tested the changes for possible failure conditions?
[ ] Have you provided the positive & negative test logs for the litmusbook execution?
[ ] Does the litmusbook ensure idempotency of cluster state?, i.e., is cluster restored to original state?
[ ] Have you used non-shell/command modules for Kubernetes tasks?
[ ] Have you (jinja) templatized custom scripts that is run by the litmusbook, if any?
[ ] Have you (jinja) templatized Kubernetes deployment manifests used by the litmusbook, if any?
[ ] Have you reused/created util functions instead of repeating tasks in the litmusbook?
[ ] Do the artifacts follow the appropriate directory structure?
[ ] Have you isolated storage (eg: OpenEBS) specific implementations, checks?
[ ] Have you isolated platform (eg: baremetal kubeadm/openshift/aws/gcloud) specific implementations, checks?
[ ] Are the ansible facts well defined? Is the scope explicitly set for playbook & included utils?
[ ] Have you ensured minimum/careful usage of shell utilities (awk, grep, sed, cut, xargs etc.,)?
[ ] Can the limtusbook be executed both from within & outside a container (configurable paths, no hardcode)?
[ ] Can you suggest the minimal resource requirements for the litmusbook execution?
[ ] Does the litmusbook job artifact carry comments/default options/range for the ENV tunables?
[ ] Has the litmusbooks been linted?

Special notes for your reviewer:

By default, the sonarqube checks are not applicable in travis builds for Pull Requests (Ref: https://travis-ci.community/t/pull-request-builds-fail-with-sonar/3473)

gentle ping @ajeshbaby !


python: make toPythonApplication pure

Description of changes

Ready to retarget to staging.
This should allow use to use poetry from pythonPackages without polluting the shell and fixes nix-shell adding a bunch of things to PYTHONPATH and python to PATH when using a package that uses toPythonApplication.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Findings while testing:
Works for the new nix shell.
nix shell --expr "with import ./. {}; with python3.pkgs; toPythonApplication poetry" --impure

and not for the old one since that is sourcing stdenv.
nix-shell --expr "with import ./. {}; with python3.pkgs; toPythonApplication poetry"

This would mean we would need to remove it from https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/interpreters/python/mk-python-derivation.nix#L151= and move it to buildInputs?


updated aspects required

I had to change org.aspectj-version to 1.8.2 in order to build this

@pslack did you use STS or cmd to build? and What other changes you made in the pom or in other config files?

I confirm. I needed this to make mvn compile successful.
My setup:
❯ mvn --version
[MVNVM] Using maven: 3.2.5
Apache Maven 3.2.5 (12a6b3acb947671f09b81f49094c53f426d8cea1; 2014-12-14T18:29:23+01:00)
Maven home: /Users/Dagguh/.mvnvm/apache-maven-3.2.5
Java version: 1.8.0_65, vendor: Oracle Corporation
Java home: /Library/Java/JavaVirtualMachines/jdk1.8.0_65.jdk/Contents/Home/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "mac os x", version: "10.11.3", arch: "x86_64", family: "mac"

Sorry I didn't see this note, I was using command line to build with no changes to pom etc.  Sorry I don't have the environment I used at the time, I'll give this another run on my current setup


Nightly Build Phase 1: Migrate to new version scheme

Overview
Phase 1 of nightly builds. This part is limited to updating the version references in various files to match the new scheme, plus tweaking the codegen that consumes them.
Details

Adopt 3 release channels, with corresponding branches (takes full effect in #878)

nightly pulls from main every night (if there are changes)
beta pulls from nightly every 3 months, ~1 week before release
stable pulls from beta every 3 months

switch from null version 0.0.0+development to correct version on each channel

main: X.Y.Z-dev
nightly: X.Y.Z-nightly.YYYYMMDD.N
beta: X.Y.Z-beta.YYYYMMDD.N
stable: X.Y.Z

Changes

The various Cargo.toml files were updated so that internal dependencies have good versions (which get updated by channel_tool.py as necessary).
python/setup.cfg was reformatted so that it round-trips cleanly through configparser (tabs, ugh)
A bunch of stuff in the codegen docstrings was tweaked to match the new versioning scheme.
nightly and beta versions have a counter after the date, which isn't necessary for regular operation, but is extremely useful for testing multiple versions in a single day.
If we decide to commit the generated Python bindings (could do this in build.yml), then I think we could get rid of the @versioned decorator, as all building and doc generation happens on a channel with the versions updated.

Current dependencies on/for this PR:

main

PR #877   👈

PR #878

This comment was auto-generated by Graphite.

This PR got hashed because of a copy/paste error, closing and starting over.


chore: update GH workflows and release processes to align with SDK release process

Ticket #177
Note: Do not want to close this ticket until this pr is merged and a new version is released and gets merged in the homebrew repo.

@cprice404 - I misunderstood what I needed to do but the updates should address your feedback properly.


Automatically set IACUC protocol and Project Name from schedule

Pull Request instructions:

Please follow the update protocol
Answer the questions below in detail. Your responses will be emailed to experimenters.
If the experimenters must do anything new, provide detailed step by step instructions on the wiki
If computer maintainers need to manually update anything, provide detailed step by step instructions
Use markdown syntax in order for your comments to be rendered reliably in the email: "1." instead of "1)", use four spaces for indents.
If you use the keyword "skip email" in the title, it will skip the email updates
Merges from "develop" into "production_testing" should use the keyword "production merge" in the title for reliable indexing of updates
Merges from "production_testing" into "main" should use the keyword "update main"

Describe changes:

Uses a cached copy of the training schedule to set the IACUC protocol and Project Name when starting a new mouse
The cached copy is refreshed every 2 minutes (process running on my computer), and saved at /allen/aind/scratch/dynamic_foraging/DynamicForagingSchedule.csv
If the mouse is not on the schedule, no IACUC protocol is set, and the Project Name defaults to "Behavior Platform"
If the project description file is not present, then the Project Name defaults to "Behavior Platform"
Since the ephys rigs and 428 are not on the schedule, they will not have these fields set. But if the mouse went through training in 446/447, then it will already be set by the time the mouse goes to ephys/428

What issues or discussions does this update address?
Describe the expected change in behavior from the perspective of the experimenter

none

Describe any manual update steps for task computers

none

Was this update tested in 446/447?

[x] tested in 447

@hanhou @XX-Yin This is ready for review


no-cmx-file warnings generated for jsoo-react modules

I'm now getting these warnings when building a project based on the minimal template:
Warning 58 [no-cmx-file]: no cmx file was found in path for module React__Dom_html, and its interface was not compiled with -opaque
Warning 58 [no-cmx-file]: no cmx file was found in path for module React__Dom_svg, and its interface was not compiled with -opaque
Warning 58 [no-cmx-file]: no cmx file was found in path for module React, and its interface was not compiled with -opaque

I've updated to the latest master and created some child projects, amongst other things, but otherwise don't do anything sophisticated and have added no other OCaml dependencies. Any idea what this could be about? If nothing obvious comes to mind I can try to create a minimal repro for it.

I tried building the template as it was, and also updated it to latest commit and I can't reproduce. 😕
Is the issue happening when running make build on the template?

Thanks for trying! Yeah it happens on make build, or make watch. My guess is that it has something to do with using sub projects then. I'll try to make a repro later today.

Repro here: https://github.com/glennsl/jsoo-react-no-cmx-file-repro. Apparently creating a very basic nested library somehow causes this warning to be emitted,

Steps to reproduce:
git clone git@github.com:glennsl/jsoo-react-no-cmx-file-repro
cd jsoo-react-no-cmx-file-repro
make create-swtich
make init
make build

Seems to be a dune bug. Created an issue for it at https://github.com/ocaml/dune/issues/5486


Cutting only part of a silence

I think there should be an argument that tells whatever function cuts out the silence to leave a minimum of X seconds of silence. This would make the speaking sound more natural, preserving pauses after a sentence for example, while still cutting out some dead air.

yes, I need this too. Some way to control threshold. Because input may vary.
Where in the code could I find this? maybe I can help


Compilation aborts when trying to compile a project with Navigation Compose and AnchoredDraggableState

Describe the bug
Compilation aborts when trying to compile a project with Navigation Compose and AnchoredDraggableState, when navigation is not added to the project - the project is built normally.
None of the following functions can be called with the arguments supplied:

public constructor AnchoredDraggableState<T>(initialValue: TypeVariable(T), positionalThreshold: (totalDistance: Float) -> Float, velocityThreshold: () -> Float, snapAnimationSpec: AnimationSpec<Float>, decayAnimationSpec: DecayAnimationSpec<Float>, confirmValueChange: (newValue: TypeVariable(T)) -> Boolean = ...) defined in androidx.compose.foundation.gestures.AnchoredDraggableState

public constructor AnchoredDraggableState<T>(initialValue: TypeVariable(T), anchors: DraggableAnchors<TypeVariable(T)>, positionalThreshold: (totalDistance: Float) -> Float, velocityThreshold: () -> Float, snapAnimationSpec: AnimationSpec<Float>, decayAnimationSpec: DecayAnimationSpec<Float>, confirmValueChange: (newValue: TypeVariable(T)) -> Boolean = ...) defined in androidx.compose.foundation.gestures.AnchoredDraggableState

Affected platforms

Android

Versions

Kotlin version*: 1.9.23
Compose Multiplatform version*: 1.6.10-beta01

To Reproduce
Reproduction sample: https://github.com/sodove/anchored_repro

Add next code to project

enum class DrawerState {
    Opened,
    Closed,
}

...
val state = remember {
                AnchoredDraggableState(
                    initialValue = DrawerState.Closed,
                    anchors = DraggableAnchors {
                        DrawerState.Opened at -startAnchor
                        DrawerState.Closed at 0f
                    },
                    positionalThreshold = { distance: Float -> distance * 0.5f },
                    velocityThreshold = { with(density) { 100.dp.toPx() } },
                    animationSpec = tween(),
                )
            }
...

Add the following dependency to your commonMain source set
implementation("org.jetbrains.androidx.navigation:navigation-compose:2.8.0-alpha01")
Try to compile project

Expected behavior
The project has to build and run

Got the same issue, and not using Compose Multiplatform

@Lonchi78 The issue is about mixing 1.6 and 1.7 while using experimental API that were changed between these versions

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


A few minor updates.

Fixes #25

The MR includes a very minor fix for some things CRAN was complaining about. Need to add a tag on a previous commit and track the release.


fix extract_response_codes_ftp()

When I test Pureftpd, I meet with responses like
"250-Begin
type=pdir;sizd=4096;modify=20230818011648;UNIX.mode=0755;UNIX.uid=1001;UNIX.gid=1000;unique=100012g7e0d4; /
250 End."
To extract response code, I have to do the modification.

Thanks for your contribution @B1y0nd . Did you observe the issue for that particular server only or for other FTP servers? Have you tested your patch to ensure that it also works for other servers?

@thuanpv I haven't had much time for the experiment, so far I have only found this issue with the FTP server. Your original implementation was for the following response(LightFTP),

but there will indeed be the response(PureFTPD).

I conducted tests and they are currently successful for the two ftp servers(I observe the other FTP servers mentioned in ProFuzzBench, and their responses also have the same issue as I mentioned. Perhaps you can adopt my code for a simple test). Because I know that the correct extraction of this response code is crucial for guiding the entire fuzzing process, I propose this modification.


How can I train the model in the "3D Only" way?

Thanks for the impressive work, and i want to know how can I train the model in the "3D Only" way? When I used the command "python scripts/train_scripts/train_pretrain.py --gpu 0 --tag pretrain --use_normal --use_color" to train the model, it reported an error "RuntimeError: mat1 dim 1 must match mat2 dim 0"

Thanks for the impressive work, and i want to know how can I train the model in the "3D Only" way? When I used the command "python scripts/train_scripts/train_pretrain.py --gpu 0 --tag pretrain --use_normal --use_color" to train the model, it reported an error "RuntimeError: mat1 dim 1 must match mat2 dim 0"

Excuse me, do you know how to train the model in the "3d only" way now?


Bug report: m365 spo hubsite list --includeAssociatedSites doesn't include associated sites in the result

Description
When I use the command m365 spo hubsite list with the option --includeAssociatedSites, I would expect the result to include the associated sites from the hubsite. If we take a look at the docs  we can see that the option is only available when the output type is JSON, which is the new default type since v4, but this isn't the case. The option --includeAssociatedSites can only be used in conjunction with --output json at the moment, which is quite verbose.
Looking at the docs again. At the remarks header, it states that the output type is text by default. Which is no longer the case.
Steps to reproduce
Run the command
m365 spo hubsite list --includeAssociatedSites
Expected results
Returns a list of all the available hubsites within your tenant, including the associated sites in the result.
Actual results
Returns a list of all the available hubsites within your tenant, but doesn't include the associated sites.
[
  {
    "Description": null,
    "EnablePermissionsSync": false,
    "EnforcedECTs": null,
    "EnforcedECTsVersion": 0,
    "HideNameInNavigation": false,
    "ID": "9ff01368-1183-4cbb-82f2-92e7e9a3f4ce",
    "LogoUrl": null,
    "ParentHubSiteId": "00000000-0000-0000-0000-000000000000",
    "PermissionsSyncTag": 0,
    "RequiresJoinApproval": false,
    "SiteDesignId": "00000000-0000-0000-0000-000000000000",
    "SiteId": "9ff01368-1183-4cbb-82f2-92e7e9a3f4ce",
    "SiteUrl": "https://contoso.sharepoint.com/sites/Examplesite2",
    "Targets": null,
    "TenantInstanceId": "21d86952-3c30-45e2-bd84-a520efdbd413",
    "Title": "Examplesite2"
  },
  {
    "Description": null,
    "EnablePermissionsSync": false,
    "EnforcedECTs": null,
    "EnforcedECTsVersion": 0,
    "HideNameInNavigation": false,
    "ID": "a9dc96c0-2b64-4860-8f7c-411d143c1ccf",
    "LogoUrl": null,
    "ParentHubSiteId": "00000000-0000-0000-0000-000000000000",
    "PermissionsSyncTag": 0,
    "RequiresJoinApproval": false,
    "SiteDesignId": "00000000-0000-0000-0000-000000000000",
    "SiteId": "a9dc96c0-2b64-4860-8f7c-411d143c1caf",
    "SiteUrl": "https://contoso.sharepoint.com/sites/Examplesite",
    "Targets": null,
    "TenantInstanceId": "21d86952-3c30-45e2-bd84-a520efdbd413",
    "Title": "Example Site"
  }
]

Diagnostics
n/a
CLI for Microsoft 365 version
V5.4.0
nodejs version
16.11.1
Operating system (environment)
Windows
Shell
PowerShell
cli doctor
n/a
Additional Info
n/a

If everyone agrees that this needs to be resolved, then I can take it up. 😄


Update scalaz-core to 7.2.28

Updates org.scalaz:scalaz-core from 7.2.27 to 7.2.28.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [{ groupId = "org.scalaz", artifactId = "scalaz-core" }]

labels: semver-patch

This has been done as part of #150 .


Handling  BTA_HH EVENT: VC_UNPLUG in components/esp_hid/src/bt_hidh.c (IDFGH-5182)

Is your feature request related to a problem? Please describe.
I was working with Xbox Controller. When I am connected to the controller, and
I force a re-pairing from the controller side, the esp32 doesn't realise that
it has disconnected beyond the above message.
BTA_HH EVENT: VC_UNPLUG
Describe the solution you'd like
Handling and sending that event up to the user level so that it can be handled
with esp_hidh_event_data_t callback
Describe alternatives you've considered
Not sure what else can be done to get this event.

Hi, @edwinclement08
Could you please checkout to the 2078dfe or pull the latest master branch and see whether the issue still exists?
If yes, feel free to reopen this.
Thanks


Removing precommit hooks

These were a good idea but aren't really useful

I agree. Thanks for removing them.


sc folder structure: Classes & Guides

This changes the render.py client function to look for a 'client_subdir' in the driver's settings dict, which allows the SuperCollider docs to be rendered into two folders inside of the master outputdir: (Classes for the 'clients' & Guides for the 'topics').
I've also added to the pd, cli, and max drivers the entry in the settings dict: 'client_subdir': '' so that they will render as they were previously.
I tested it on all four and SC looks correctly changed and the other three look as they did before.

@weefuzzy any objection to me merging this? I keep having to do file shuffling...


ci: run baseline perf tests nightly

test-4.9

test-net-next

test-4.19

test-runtime

test-me-please

test-4.19

test-gke

test-me-please

test-me-please

All jenkins CI passed, need to just update the docs.

oh right, helm-docs was recently added. Maybe we can mark it as required now ? /cc @nebril @aanm


rubocops/cask: Check for correct stanza order within on_* blocks

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

Follow-up to #15211 to fix more TODOs but in a different cop.

Now that we detect correct stanza grouping within on_* blocks in Casks, correct stanza ordering in on_* blocks was the next logical step. For example, url has to come after version and sha256 in an on_macos or on_intel block for consistency with the top-level stanza order we enforce elsewhere.
Still not doing the nested on_os inside on_arch, that felt excessive for an edge case that isn't present in more than a few real Casks we have. I removed the test with that specific TODO. If anyone feels strongly I can look at it again, though.

The best I can come up with so far for the nested clobbering problem is fixing the inner stanzas first, then returning a log message if there are top level errors too asking the user to rerun the cop. That feels like awful UX.

Putting this back in draft because I need to do more thinking about what's actually the problem here and the fixes for nested stanzas.

OK, so the overwriting and infinite looping happens when determining the "correct stanza" for block inner stanzas:
                correct_stanza_index = sorted_stanzas.index(stanza)
                correct_stanza = sorted_stanzas[correct_stanza_index]

Because a stanza named :url comes 15th in the list of sorted stanzas, whether or not it's a top-level URL or an inner URL (that can be in a different place since it's a different "kind", for want of a better term), so it tries to replace the 15th item, again, again, again...
I don't know how to not do that for inner stanzas. And I am quite confused and I feel like I'm going in circles. 😓

OK, I thought I was getting somewhere last night with the reimagined approach after Markus' reply on Slack, but now I'm as confused as ever. Any ideas, folks? Is this worth it at this point?

Right, even with a few days off I officially can't figure it out and it's reached the point where I get a sinking/anxious/useless feeling when I come back to it. Thanks so much for your continued help though, Markus! ❤ If you'd like to continue with this then please do - it seems like you have an idea of what and how things could be achieved here and I wouldn't want your effort to be wasted too!

@issyl0, I did it!
Also opened https://github.com/rubocop/rubocop/pull/11858 to fix an issue I encountered here.

Also, another issue is the function autocorrect_source we use in our tests, which does not loop (to handle nested corrections). rubocop-rspec provides a expect_correction function which does this by default, which we should switch to.

@reitermarkus 🤩 🤩 🤩


⚠ Samsung has degraded performance

In 968a0e1, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2935 ms

Resolved: Samsung performance has improved in 8759a99.


The "stream" option is not documented in readme

While it's actually implemented here https://github.com/moscajs/aedes-logging/blob/12791401faf51affbbb0bc82b8c69d583b772932/index.js#L8

@Arsfiqball PR?


Removing not needed and incorrect ReferenceCopyLocalPaths references

Fixes: #142

/cc @BrennanConroy

:up::date:
found one more issue - in release build both the debug and release versions of signalrclient.lib would be included.

:shipit:


Sphinx doc

Move nbdev docs to sphinx docs, meaning that we will only program on .py files.

removed all #export cells in nbdev_nbs
other cells in nbdev_nbs were kept for nbdev_test, which is quite nice for automatic testing and debugging.

There are only a few code changes:

psm_readers: automatically detect mod_sep and underscore_for_ncterm
all unimod_ids were added into modification of MaxQuantReader and its inherited readers.

TODO: pdflatex failed in readthedocs, but this is not important.

Complete code review is not recommended as there are too many code changes.


Update TDLib & Save voice/video notes

When I run the version from upstream, it doesn't log in and instead suggests to update app (tdlib). Had to change some code to do the update.
Also added downloading voice & video notes and fix for config.humanParams.textReadSpeedWPM.

Hey, thanks for your contribution! I will review shortly.
I am not surprised about the message you got, it's been very long since i worked on this project and an update is definitely needed.

Overall good, but I would request to separate active and inactive users in the TelegramUser class and schema

Thank you for the feedback.
It's a good idea to put disabled usernames in a separate column, I'll do it one of these days.

I'll try to land that change later today, feel free to send another PR for that if I end up forgetting about it :)


importing patient zipper code from master, stripping out test executi…

…on archiving , need to reimplment as product test archiving somehow

@rdingwell can you fix the merge conflicts in this?


send_auto_takeoff_command() missing in wifiConnection.py

Hi,
I was going through the wifiConnection.py file and could not find the function 'send_auto_takeoff_command()' function which is internally called by the 'turn_on_auto_takeoff()' function. However, this function is present in the bleConnection.py file.
Was this done on purpose to support auto-takeoff feature with only BLE? I updated the wifiConnection.py file, at my end, to include this function and the throw mode seems to work fine.
However, the next time I connect the drone and create the Mambo instance, I observe that the drone stops sending the initial packets after sending the "cut-out enabled" packet.
I am able to take off and perform other operations however the drone does not send any sensor data.

I tried to ensure that each function exists in BLE and wifi but I guess I missed that one.  I'll update it shortly.  Also, I don't know why you would be dropping sensor data.  I am focusing mostly on the wifi since it sends a lot more sensor information but I've not seen it fail to send any unless you send an invalid packet (then it gets unhappy inside the drone).  I'll update the function and test shortly.

I double checked that the packets being sent are valid. If it was invalid, the functions like takeoff, flip, land etc. would not be working. I am able to execute all these operations. However, the drone remains silent in terms of sending the sensor data every 0.5 seconds.
Interestingly, I observe this only on my laptop and not on my desktop computer. I am wondering if this has something to do with the controller name/type being sent in the initial JSON packet.
Let me know if you face the same issue after executing the throw mode and creating the mambo instance again.

That is very odd.  I'll hunt it down tonight.  I don't have a drone with me at work today!

I just put the command into the wifi and it seems to work just fine.  Sends lots of sensor info to me.  Can you update and try again?

I tried to reproduce it again and this time it worked fine. The issue seems to be sporadic.
Will update you with the reproduction steps, in case I run into the issue again. Thanks!

Glad to hear it!


Error in Equation 5.149 [version 2022-04-01]

the smaller or equal symbol should be greater or equal based on KL divergence definition from equation 5.10.

fixed


Flow unable to understand trivial variable equivalence when determining refinements

I ran into a issue where flow will not recognise a type of a copied variable for conditional rendering in React:
https://flow.org/try/#0PQKgBAAgZgNg9gdzCYAoVBLAtgBzgJwBcwAlAUwEMBjYqfOLMAcn0pqYG50q4A7AZ2IAVMgA9iAXjAAKAN5gAFoSwwwAXwBcYeUpVbB+DLwDm6gJRgJAPkXKYXVFACuvGhj5gA6vgo4cZfDlbFXUtHTsAfn1CQxNzbVQwMB4BYgUKfgAJO0tg+0SwVkInfF5FDOyQgDIqsAAeEXE8iVldGDUwYCsuNVQgA
const Text = ({ html }: { html: string }) => html;

function Wrapper({ html }: { html?: string }) {
  const hasHtml = html;
  return hasHtml && <Text html={html} />;
}

Funny enough, this will work:
https://flow.org/try/#0PQKgBAAgZgNg9gdzCYAoVBLAtgBzgJwBcwAlAUwEMBjYqfOLMAcn0pqYG50q4A7AZ2IAVMgA9iAXjAAKAN5gAFoSwwwAXwBcYeUpVbB+DLwDm6gJRgJAPkXKYXVFACuvGhj5gA6vgo4cZfDlbFXUtHTsAfn1CQxNzbVQwMGBgMB4BYgUKfgAJO0tg+0SwVkInfF5CsAAyarAAHhFxQolZXRg1ZKsuNVQgA
const Text = ({ html }: { html: string }) => html;

function Wrapper({ html }: { html?: string }) {
  return html && <Text html={html} />;
}

Am I missing something?

Thanks for your quick reply, @jbrown215! I'd be interested in helping with testing when somebody picks up this task.

@rkrv I got very definite answer to same problem here: https://github.com/facebook/flow/issues/6146#issuecomment-439139904
This has been reported many times already. I think that it's because the restriction is not well documented and the use case is pretty idiomatic JS. If you know the restriction, it's very easy to avoid it and still being able write clean code. Just avoid destructing objects that has this kind of "cross dependencies"


Update README.md

Description

Fixed some minor grammatical errors in the README.md
Breaking change?

Example

Checklist

[ ] Unit tested updated code paths
[ ] Tagged 1 of @janniks or @zone117x for review

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add - hotkey for toggling the signedness of IntegerDisplayType constants

What is the feature you'd like to have?
Please fix the shortcut of changing the type of number
Is your feature request related to a problem?
NO
Are any alternative solutions acceptable?
YES
by manual right click and select the corresponding button
Additional Information:

using The ➖(minus sign - ) button
is not effect.

using The ➖(minus sign - ) button is not effect.

That is meant to cycle the type of an integer data variable in 1, 2, 4, 8 byte iirc, so it is not designed to work that way. However, your feature request is a valid one and worth considering

the latest version fixed this problem.

the latest version fixed this problem.

I do not think we actually changed anything related recently. Is it because you tested with a data variable? If it is a data variable, pressing "-" on it will indeed toggle its signedness, and it will achieve the same result that you have expected. However, if the constant is from the code and not a data variable, I believe pressing "-" should not have worked. If that is the case, we should case this open until we actually support it

the latest version fixed this problem.

I do not think we actually changed anything related recently. Is it because you tested with a data variable? If it is a data variable, pressing "-" on it will indeed toggle its signedness, and it will achieve the same result that you have expected. However, if the constant is from the code and not a data variable, I believe pressing "-" should not have worked. If that is the case, we should case this open until we actually support it

老哥说的有道理，我确实测试快捷键有生效的了，但是测试另一部分，又出现不行的。我上传一个文档。
我重新打开这个，然后上传文件过来。

the latest version fixed this problem.

I do not think we actually changed anything related recently. Is it because you tested with a data variable? If it is a data variable, pressing "-" on it will indeed toggle its signedness, and it will achieve the same result that you have expected. However, if the constant is from the code and not a data variable, I believe pressing "-" should not have worked. If that is the case, we should case this open until we actually support it

老哥说的有道理，我确实测试快捷键有生效的了，但是测试另一部分，又出现不行的。我上传一个文档。 我重新打开这个，然后上传文件过来。

Thx for confirming! But we do not need a file for this, since it is quite easy to reproduce

the latest version fixed this problem.

I do not think we actually changed anything related recently. Is it because you tested with a data variable? If it is a data variable, pressing "-" on it will indeed toggle its signedness, and it will achieve the same result that you have expected. However, if the constant is from the code and not a data variable, I believe pressing "-" should not have worked. If that is the case, we should case this open until we actually support it

老哥说的有道理，我确实测试快捷键有生效的了，但是测试另一部分，又出现不行的。我上传一个文档。 我重新打开这个，然后上传文件过来。

Thx for confirming! But we do not need a file for this, since it is quite easy to reproduce

OK

This should also work for toggling the sign of variables and parameters too.