[logstash] Add fullnameOverride setting

Added fullnameOverride to logstash by copy pasting from kibana/filebeat/metricbeat charts.

[x] Chart version not bumped (the versions are all bumped and released at the same time)
[x] README.md updated with any new values or changes
[x] Updated template tests in ${CHART}/tests/*.py
[x] Updated integration tests in ${CHART}/examples/*/test/goss.yaml

Yes, this will be released in the next one which should be 7.5.3 or 7.6.0.


make it compatible with Emotion 10

Lots of changes affecting this code mode. But I'm almost done working on it. Stay tuned :)

Exciting!


⚠ ClamAV has degraded performance

In a826db0, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 79 ms

Resolved: ClamAV performance has improved in 049d019.


[Messaging] Unable to get messaging to validate APNS payload with apns-push-type header field

[READ] Step 1: Are you in the right place?
Cloud Firestore support is provided by the @google-cloud/firestore library. Therefore the easiest and most efficient way to get Firestore issues resolved is by directly reporting them at the nodejs-firestore GitHub repo.
If you still think the problem is related to the code in this repository, then read on.

For issues or feature requests related to the code in this repository
file a Github issue.
For general technical questions, post a question on StackOverflow
with the firebase tag.
For general Firebase discussion, use the firebase-talk
google group.
For help troubleshooting your application that does not fall under one
of the above categories, reach out to the personalized
Firebase support channel.

[REQUIRED] Step 2: Describe your environment

Operating System version: iOS
Firebase SDK version: 11.11.0
Firebase Product: Messaging
Node.js version: 16.13.1
NPM version: 8.10.1

[REQUIRED] Step 3: Describe the problem
I'm attempting to set more particular fields into the messaging send() function, however I continue to get failures as messaging().send() validates the JSON.
Steps to reproduce:
What happened? How can we make the problem occur?
Any time I add apns-push-type I seem to get these failures
This will print Firestore logs to the console.
>  {"severity":"ERROR","message":"Unhandled error FirebaseMessagingError: Invalid [apns-push-type] set for apns platform\n    at Function.fromServerError (/Users/evananger/Developer/blazebite-functions/functions/node_modules/firebase-admin/lib/utils/error.js:254:16)\n    at createFirebaseError (/Users/evananger/Developer/blazebite-functions/functions/node_modules/firebase-admin/lib/messaging/messaging-errors-internal.js:35:47)\n    at /Users/evananger/Developer/blazebite-functions/functions/node_modules/firebase-admin/lib/messaging/messaging-api-request-internal.js:79:75\n    at processTicksAndRejections (node:internal/process/task_queues:96:5)\n    at async /Users/evananger/Developer/blazebite-functions/functions/node_modules/firebase-functions/lib/common/providers/https.js:458:26\n    at async runFunction (/Users/evananger/.nvm/versions/node/v16.13.1/lib/node_modules/firebase-tools/lib/emulator/functionsEmulatorRuntime.js:506:9)\n    at async runHTTPS (/Users/evananger/.nvm/versions/node/v16.13.1/lib/node_modules/firebase-tools/lib/emulator/functionsEmulatorRuntime.js:531:5)\n    at async /Users/evananger/.nvm/versions/node/v16.13.1/lib/node_modules/firebase-tools/lib/emulator/functionsEmulatorRuntime.js:694:21 {\n  errorInfo: {\n    code: 'messaging/invalid-argument',\n    message: 'Invalid [apns-push-type] set for apns platform'\n  },\n  codePrefix: 'messaging'\n}"}

Relevant Code:
function sendOrderReady(token) {
  const date = new Date();
  return admin.messaging().send({
    token,
    apns: {
      headers: {
        'apns-push-type': 'liveactivity',
        'apns-priority': '10'
      },
      payload: {
        aps: {
          timestamp: date.getTime(),
          event: 'update',
          'content-state': {
            orderCompleteDate: date.toISOString(),
            orderReady: true
          }

        }
      }
    }
  })
}

Same issue here. @devandanger Were you be able to solve it?


[PLUGIN] Create a plugin to add field to the database

Does it mean (Add Custom Field Types Using UI ) ??


debug

Hi,
I just found your article about nail-biting thanks for sharing!
Is there any chance if you can help me to debug?
I followed your article:

trained on Teachable Machine , and it was working at the end, its clearly detected my face
cloned the repo, overwrote your json and .bin files
started the webserver

But it does not recognize if i biting my nail (or just put my hands before my face).
If I put the notify(MESSAGE_STOP); before detecting, I get the nofication, so I don't think if its the browser's issue.
Is there any way to debug the detecting?
thanks

aah, I forgot to set normal/nail-biting class names :/
its working now, thanks!

I'm glad it is working @dyipon!

I haven't used it for hours but I can imagine such a scenario.
Maybe you can try increasing the updateInterval here.


Wrong $ref for path in nested dir processing in v1

In case $ref for path is pointing to a file in nested directory, internally defined refs are using wrong base directory.
While having swagger.yaml with:
paths:
  /some_path:
    $ref: './subdir/some.yaml'

and some.yaml with:
get:
  responses:
    200:
      schema:
        type: object
        properties:
          some_property:
            $ref: './some_definitions.yaml#/SomeProperty'

will try to resolve some_definitions.yaml in the swagger.yaml root directory instead of subdir

pull request with fix #1224

thanks! @ZsergUA
the PR is now merged.


Update TOC.yml (add JMS to item)

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

Merging per Angela.
@brunoborges - Thanks for your contribution.


How to calculate RMSE for a predicted a structure and a true structure?

Usually, we need to do some kind of evaluation to check the performance of the CSP algorithm.
This can be done by calculating the RMSE distance between the predicted structure and the predicted structure... Does CrySPY has such a function?

No, but CrySPY will be able to output predictive values and variance in BO. I think we can also include RMSE.

If you can add the RMSE function to it, it will be great to evalute the performance of diff algorithms...
a reference is here
https://pyxtal.readthedocs.io/en/latest/_modules/pyxtal/operations.html#distance_matrix

Even if we evaluate the structure distance between predictive and true structures (or evaluate energy difference using RMSE), we would not be able to evaluate the performance of CSP algorithms. That is only the evaluation of accuracy of the machine learning potential. CSP includes structure generation and exploration phase.

Related: https://github.com/sparks-baird/xtal2png/issues/12
See also pymatgen's StructureMatcher and get_rms_dist function.


[FLINK-25406] Let FileSystemBlobStore sync output stream

What is the purpose of the change
In order to avoid losing data the FileSystemBlobStore should call FSDataOutputStream.sync() when putting new results. This will ensure that no data is kept in caches and might be lost in case of a node crash.
This PR is based on #18191.
Verifying this change
Added BlobUtilsTest and FileSystemBlobStoreTest.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Thanks for the review @zentol. Merging this PR once #18191 is merged.


User-agent modified by cloudfront

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
djh7fyxc2u06x
Region
region=ap-south-1
Amplify Console feature
Build settings, Custom headers, Redirects
Describe the bug
I create a project and upload a zip file that consists of an index.html and add redirect rules
Source address: <*>
target address: https://abc.com/<*>
Type: 200 (Rewrite)

And it is totally working except user-agent that I am getting this at https://abc.com server
user-agent = Amazon CloudFront

Expected behavior
Expecting User-Agent
user-agent = Mozilla/5.0 (Linux; Android 6.0.1; Moto G (4)) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/90.0.4430.72 Mobile Safari/537.36

Reproduction steps
Please follow the above stepsfollow
Build Settings
No response
Additional information
How can I solve this issue? Please help.

Is there any progress on this issue?
As of now, it is impossible to use AWS Amplify Rewrites for rewrites to an external API because the user agent is modified to be Amazon CloudFront instead of the user's user agent.
When will this be fixed?

Yes, and since it's not possible to use Lambda@Edge or see the CloudFront distribution, there's no workaround.
It's important to SSR features

This feature has just been supported with our latest launch! Closing.
Check out the details here: https://aws.amazon.com/blogs/mobile/cdn-caching-improvements-for-better-app-performance-with-aws-amplify-hosting/


Remove linguistic isolation from the formula for disadvantaged for Puerto Rico

Linguistic isolation should not be used in the formula to identify DACs for Puerto Rico.

@emma-nechamkin please produce a list of census tracts that would be affected.

To confirm, linguistic iso is only relevant for the Workforce factor, correct?

So to be clear, this is the same as making PR have the same workforce/unemployment calculation as the island areas, right? @emma-nechamkin @BethMattern

Gotcha, thanks for the clarification! So, neither the island areas nor PR should use linguistic isolation for workforce calculation, BUT PR uses the 2019 ACS data like the States while the island areas use the 2010 data.  Fun fun!


fix coredump when executing VW DDL but RM is not enabled

I hereby agree to the terms of the CLA available at: [placeholder]
Changelog category (leave one):

Bug Fix

Detailed description / Documentation draft:
fix #46

 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 DDC Hardware

Free for personal use.

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[x] Named the cask according to the token reference.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.
[x] Checked there are no open pull requests for the same cask.
[x] Checked the cask was not already refused in closed issues.
[x] Checked the cask is submitted to the correct repo.

Free for personal use.

Not exactly. It's "Pay-What-You-Want". https://losttype.com/font/?name=ddc_hardware
Ping @vitorgalvao

Technically you may want to pay zero and users are expected to know what they are downloading. It’s unlikely you’ll download a font without first knowing what it is. It’s conceivable a user has already paid more than zero and wants an easy way to install via HBC.
All that said, even though that abides by the letter of the law, it probably does not abide by its spirit and they’re expecting users to contribute something. Aaron Draplin is a relatively well-known professional designer. I also think we should try to avoid fonts with restrictive licenses, especially if they’re not too well-known.
I can justify both inclusion and refusal. If I had to guess what the owners of that website would want, I’m guessing they’d prefer refusal and I’m fine with that.

Well in my homebrew-adobe tap for example, in this kind of situation I have included caveats saying something along the lines of Please consider a voluntary purchase if you like this… along with the link.
Would that be worth doing as a comprimise?

Would that be worth doing as a comprimise?

No:

To avoid flooding users with too many messages (thus desensitising them to the important ones), caveats should be used sparingly and exclusively for installation-related matters.

Thanks nonetheless for the PRs!


Not working correctly on linux

Describe the bug
Opens but functions are not working..
To Reproduce
Open/execute launchbuddy under linux with lutris
Expected behavior
Just to work out of the box like 99% of any games and apps now working like that.
First happend in LB Version
Issue first appeared in LB Version : newest

have you considered that this uses windows functions to spawn the multiple clients? what a weird thing to assume linux people have

There's a trick with Lutris. After you've installed GW2, you can copy the directory multiple times, then "install" GW2 over and over again, but after it's done, point it to the copied directory. That way you can have many accounts that have their own prefixes and don't interfere with one another. No need for Launchbuddy.

Thanks for the hint. Do you know if this needs more system resources than using it with launch buddy?
@Lukaboot GW2 is using windows functions, so no point.
best regards

It actually uses less resources because you're not running another process that's taking up memory and CPU. You're just running multiple GW2 instances. You just need more disk space because you're essentially copying the whole prefix which already has a lot of stuff in it.
I've been running like this for 5 years now without any issues. GW2 is installed in a separate directory where all my games are, and the lutris prefixes are all inside a gw2 directory for better maintainability. And in Lutris I just rename the "game" so I know which account I'm launching.

Very every nice, thank you so much ❤


'invalid image' when trying to use a private registry image

I'm getting an error like this (data somewhat sanitised):
Error: could not read 'pods/my-pod.yml'
error reading file 'pods/my-pod.yml'
invalid image 'my-private-registry:5000/my-org-name/my-app-name'

Config snippet:
# pods/my-pod.yml
services:
  my-app-name:
    image: 'my-private-registry:5000/my-org-name/my-app-name'

It's definitely a valid image name though. I can do docker run my-private-registry:5000/my-org-name/my-app-name and it works just fine. Looks like a case of overly-strict image name validation?
If that's the case, it seems that Cage is not compatible with private docker registries. Which would suck, because so far I'm really loving what Cage can do!

Hi @camjackson normally private registries are supported, but if I had to guess it's likely that the port number is throwing off cage. We'll take a look at it.

Does my-private-registry contain a real domain name with periods? I think that might be mandatory. Here's the regex that we're using:
^(?:([^/:.]+\.[^/:]+)(?::([0-9]+))?/)?(?:([^/:.]+)/)?([^/:]+)(?::([^/:]+))?$

This was based on a earlier version of the docker-compose.yml spec. So either the spec has subtly changed, or I read it wrong, or your private registry syntax is non-standard.

Here's a list of test cases showing the syntax we definitely support, from further down in the same file:
    let pairs = vec!(
        (img1, "hello"),
        (img2, "example/hello:4.4-alpine"),
        (img3, "example.com:123/hello:latest"),
        (img4, "example.com/staff/hello"),
    );

Ahhh, yeah that explains it. The registry is at companyname-registry:5000 (because of reasons), and we hack our host files to make that resolve to the right place.
So no, it's not a valid domain name. However, I just tried it in a regular old compose file, and docker-compose will quite happily start up a container from that image name.
I was going to have a go at putting a PR together for this, but cargo build gives me this (totally understand if you're not interested in providing support for my build environment, but thought you might be interested to know 😄):
   Compiling term_size v0.2.1
   Compiling aho-corasick v0.5.3
   Compiling rand v0.3.14
   Compiling lazy_static v0.1.16
   Compiling openssl-sys-extras v0.7.14
   Compiling openssl v0.7.14
   Compiling miniz-sys v0.1.7
   Compiling backtrace-sys v0.1.5
error: failed to run custom build command for `openssl v0.7.14`
process didn't exit successfully: `/Users/cjacks/code/cage/target/debug/build/openssl-5643e7fc215043a9/build-script-build` (exit code: 101)
--- stdout
TARGET = Some("x86_64-apple-darwin")
OPT_LEVEL = Some("0")
PROFILE = Some("debug")
TARGET = Some("x86_64-apple-darwin")
debug=true opt-level=0
HOST = Some("x86_64-apple-darwin")
TARGET = Some("x86_64-apple-darwin")
TARGET = Some("x86_64-apple-darwin")
HOST = Some("x86_64-apple-darwin")
CC_x86_64-apple-darwin = None
CC_x86_64_apple_darwin = None
HOST_CC = None
CC = None
HOST = Some("x86_64-apple-darwin")
TARGET = Some("x86_64-apple-darwin")
HOST = Some("x86_64-apple-darwin")
CFLAGS_x86_64-apple-darwin = None
CFLAGS_x86_64_apple_darwin = None
HOST_CFLAGS = None
CFLAGS = None
running: "cc" "-O0" "-ffunction-sections" "-fdata-sections" "-g" "-m64" "-fPIC" "-o" "/Users/cjacks/code/cage/target/debug/build/openssl-56b1b41302837114/out/src/c_helpers.o" "-c" "src/c_helpers.c"
cargo:warning=src/c_helpers.c:1:10: fatal error: 'openssl/ssl.h' file not found
cargo:warning=#include <openssl/ssl.h>
cargo:warning=         ^
cargo:warning=1 error generated.
ExitStatus(ExitStatus(256))

command did not execute successfully, got: exit code: 1

--- stderr
thread 'main' panicked at 'explicit panic', /Users/cjacks/.cargo/registry/src/github.com-1ecc6299db9ec823/gcc-0.3.38/src/lib.rs:958
note: Run with `RUST_BACKTRACE=1` for a backtrace.

Build failed, waiting for other jobs to finish...
error: build failed

Never mind, it was just an issue with openssl header locations on MacOS. I've fixed it now.
Would you be open to a PR that would allow a registry host like my-registry?

I made an attempt at this, but I suck at regex 😕 I tried this:
^(?:([^/:]+)(?::([0-9]+))?/)?(?:([^/:.]+)/)?([^/:]+)(?::([^/:]+))?$

(before, after)
But then the test fails on the first case of a bare image name: hello. Needs more thought than I'm capable of right now 🤔

FYI, I'm just going to work around this for now by changing my fake registry host to be something like my-company-registry.local. Feel free to close this issue if you don' think it's worth the effort. And sorry for the comment spam!

Oh, no problem! If anybody figures out how to write a regex that matches whatever the latest version of compose is doing, I'll be happy to merge it. (Hint: Check the docker-compose source to see their regexes and test cases.)


Generalize the script timeout uncatchable error mechanism

Script timeout bails out of the current callstack by throwing a sort of uncatchable error:

Catch blocks won't catch the error.
Finally blocks will catch the error, but it will be automatically rethrown before any opcodes in the finally block executes.
Native protected call sites will catch the error and can clean up - however, if they do anything which invoke the bytecode executor, the error will propagate again.

In some situations it'd be useful to be able to throw an uncatchable error from user code. For example, if the Duktape heap needs to be destroyed quickly, it may be useful to throw an error which cuts through all user code but allows native code to correctly free up resources etc.
There are various useful behaviors:

One might want to enable finally blocks to run so that Ecmascript code could clean up normally too. However, finally blocks should not be allowed to replace the continuation with e.g. a "return" or a "continue".
One might want to also bypass native protected call sites in some cases, but rely on finalizers for cleanup. (This won't directly work because Duktape itself relies on protected call sites always getting control on errors.)

Figure out how to generalize the mechanism, change the script timeout to use the generalized mechanism, and expose the ability throw an uncatchable error from user code.

@fatcerberus Regarding your need for a hard exit, you should actually be able to use the current script timeout mechanism for a hard exit. Simply make the user callback indicate a script timeout when you want to hard exit - that will cause the script timeout to propagate outwards in this uncatchable manner. The error will be a script timeout, which is not entirely accurate, but should be functional for now.

My current solution of performing a longjmp() out of the engine seems to work most (if not all) of the time, I specifically avoid calling back into that Duktape heap for anything other than duk_destroy_heap() because I rightly consider its stability to be compromised.  The fake script timeout would definitely be a cleaner solution though, since I don't actually know how much intermediate teardown code I might be skipping the way it's currently done.


Merge pull request #1

Merge pull request #1

Thank you for your contributions @HallLinux, All the very best on your future endeavours. Feel free to reach out to me if you need any help in doing any open source contributions. Let it be finding a project to contribute or how to take up an issue and solve it. I'd be happy to help


driver: tcpc/vbus: Add TI TSP25750 driver

Disclaimer: I'm not experienced with the USB-C stack and this is only my second PR to zephyr.
This PR adds a driver for the Texas Instruments TPS25750 USB-C PD Controller. Furthermore it includes a rudimentary driver for VBUS which gets it information also from the TPS25750. The VBUS-Driver is not ready and just for demonstration purposes part of this PR.
My problem is that in the way the current USB-C subsystem is designed, it will probably not play nicely with this chip. For instance I cannot implement the required APIs

cc_set_polarity
set_cc
set_vconn_cb
set_vconn_discharge_cb

The chip simply doesn't support this and manages this internally. In consequence if I try to run the USB sink sample the USB subsystem shuts down:
*** Booting Zephyr OS build zephyr-v3.4.0-507-g2f5068fcda0c ***
[00:00:00.001,000] <err> ucpd_tps25750: called unimplemented ucpd_set_cc(cc_pull=3)
[00:00:00.001,000] <inf> usbc_stack: Disabled
[00:00:00.001,000] <inf> usbc_stack: PE_SUSPEND
[00:00:00.001,000] <inf> usbc_stack: PRL_HR_SUSPEND
[00:00:00.001,000] <inf> usbc_stack: PRL_TX_SUSPEND
[00:00:00.001,000] <inf> power_domain_gpio: pd_3v3 is now ON
[00:00:00.002,000] <inf> power_domain_gpio: pd_5v is now ON
[00:00:00.006,000] <inf> usbc_stack: ErrorRecovery
[00:00:00.246,000] <err> ucpd_tps25750: called unimplemented ucpd_set_cc(cc_pull=2)
[00:00:00.246,000] <inf> usbc_stack: Unattached.SNK
[00:00:00.252,000] <inf> usbc_stack: AttachWait.SNK
[00:01:59.885,000] <inf> usbc_stack: Unattached.SNK
[00:02:04.274,000] <inf> usbc_stack: AttachWait.SNK

How to proceed here? Did I misunderstand something regarding how to implement TCPC API? Or is this kind of chip really not properly supported by the USB-C subsystem?

@sambhurst @galak is anyone able to help me here?

Hello @MarcelKr, the Zephyr USB-C PD stack is designed to work with Type-C Port Controllers (TCPCs). After a quick look at the TPS25750 datasheet, that device is a PD Controller; meaning it has a built in USB-C PD stack. From the datasheet:
• USB Type-C power delivery (PD) controller
– 10 configurable GPIOs
– BC1.2 charging support
– USB PD 3.0 compliant
– USB Type-C specification complaint
– Cable attach and orientation detection
– Integrated VCONN switch
– Physical layer and policy engine
– 3.3-V LDO output for dead battery support
– Power supply from 3.3 V or VBUS source
I assume you are running your code on a micro-controller that's connected to the TP25750 via I2C. If this is the case, you can create a Zephyr application that sends the proper commands to the TP25750 to implement project.
I'm not sure if this is possible but you might be able to configure the TP25750 to operate as a TCPC and use the Zephyr USB-C PD stack.

@sambhurst thanks for helping me out ;)
So if I understand you correctly here, there's currently no dedicated API in zephyr for PD controllers. I would implement a driver with a fully "custom" API then, right?
If so, is it desired to upstream it?

@sambhurst OK thanks. I'm not sure if I'm up for the task, it sounds quite big. I'll have to ask management if they'd make time for it for me. Could you provide a description of how the API should look like?

@sambhurst please revisit

@MarcelKr Sorry for the delay. I recently started working on integrating a PD Controller into a Zephyr project and it's a little more involved than I expected, and I think this effort warrants an RFC. I'll write up the RFC for you to review and decide if it's something you'd like to take on.


Create validator-functions.py

I wrote this for my HCI work but Jun Xiang said he finds this helpful :)

Can I check the purpose of your code? Can you show me a use case of your validator function?


onChange get base64

Is there a way to get the base64 image on a Event rather than using $refs, as my $refs are updated based on v-if?

Currently, there's no way get base64 but use $refs, next version will add dataUrl params to each event handler like this:
onChange(file, fileName, dataUrl) {}


Benchmarking with a large number of clients

I wonder how the goroutine/thread benchmarks compare when there are a very large number of clients.  It's reasonable for a production server to handle 2,000 up to 20,000 clients at once.  When the number of clients is extreme large, especially at the 20,000 range, how do the various benchmarks compare?

I'd love to benchmark this, but the problem is that I don't have thousands of cores lying around. At some point, all the cores dedicated to the clients will be saturated, at which point adding more clients won't actually make a difference (if anything, the performance might decrease due to context switching). I also can't move to using multiple machines, because these rates can often not be sustained over anything but loopback (mostly because the kernel can't saturate the NICs).

I suppose one option is that the client becomes non blocking and each client sends on multiple open connections.  My goal would be to see if there is a use case where lighter goroutines match up better to threads (maybe there isn't).

I'm not sure making the client non-blocking would actually make a difference. The packets would still need to be enqueued, so you'll still incur the performance overhead of a syscall per send and per receive, and those syscalls are where the clients are spending a majority of their time anyway.
I completely agree that this would be interesting for many reasons -- benchmarking thread-per-client vs a worker pool when there are many clients for example -- but I don't know how to do that in a way that will yield relevant results. My research group might be getting a 144 core machine, and with that I should be able to push the number of clients slightly higher, but unless you've got some spare 2000-core server lying around, getting higher numbers will be challenging.

This would be a really nice test too, the first tests you built can test the raw network speed, while tests with massive ammount of clients can test the nonblocking-io features.
You don't need 1 thread per connection, this is why rust has mio, c has epool and golang has this feature native.

Again, I'm not sure non-blocking I/O will actually make a difference here. At some point, all the client cores will be saturated, and the performance cannot increase regardless of how many clients you stripe across those cores.

My perspective would not be about gaining a performance increase, but measuring the language with the least performance decrease.

Ah, I see. Yes, that could work. I suspect what we'll see is that any solution using epoll will have a fairly negligible performance decrease, whereas the ones using a thread-per-client will drop drastically due to scheduling overhead. Worker pools will likely be somewhere in between.
I'll try to run experiments with 2000 and 20000 clients tomorrow.

@cep21 so how about I instead run a benchmark with log(number of clients) on the x-axis, performance on the y-axis, and then plot a separate graph for, say, 1, 10, 20, and 40 cores? That way, you'll see the trend clearly, and also how much the number of cores affect it. I can run that with 40 (one per client CPU), 80, 160, 320, 640, 1280, 2560, 5120, 10240, and 20480 clients for example?

That seems reasonable.

:+1: I'll run the benchmark as soon as the 80-core machine is available, and then post them here.

Hmm, that's strange. With a large number of clients, I'm seeing a large number of ETIMEDOUT and ECONNRESET errors from the clients, so the test never actually completes correctly. It seems like the server is overwhelmed trying to accept connections. The way to fix this is probably to have multiple threads accepting connections, but this requires non-trivial changes to every server. Alternatively, I could have the clients retry connecting to the server indefinitely, though that becomes problematic if the server crashes.. I'm open to suggestions.

Do all servers equally misbehave?  I'm curious where that breaking point is.

Another option is to increase the timeout on the client side?

Interestingly, no, it only seems to be happening for c-threaded. I wonder why -- it doesn't do anything special compared to, say, the Rust implementation, as far as I'm aware.
Well, it's kind of weird, because the default timeout in Linux for connect is 20 seconds, and I'm seeing timeouts failing long before 20 seconds have passed. This suggests that something else is going on, but I'm not quite sure what..

Ugh, for many of the tests, the majority of testing time are now simply waiting for all the client threads to connect :confused: Since accept is only called from a single thread, this takes forever. I think we'll have to implement multi-threaded accept for all the servers before it is even feasible to run this benchmark with > 10000 clients.

Results for up to 5k clients with all implementations except c-threaded:

(box titles denote number of cores)


Update version of scala, sbt, finch and travis.yml

Description

[x] Updated version of scala, sbt, finch and circe.
[x] Changed build.sbt template, moving dependencies to project/Dependencies.scala
[x] Changed oraclejdk8 to openjdk8 in .travis.yml

Has there been any motiviation to merge this PR? I've tried to use the latest version and it seems that this template is out of date.


Java source

Would it be possible to get a copy of your java source? I would like to use some of your code in a project I am doing but I need to be able to modify the tables that the java program is pushing to.

Hi,
the Java source code is inside the subfolder called "java"


Update 'main' post-24.03

Who is in charge of resolving the conflicts? Is it the release manager or the tools team?

Closing looks like all the commits we are missing were cherry-picked.


Env.variables persist between runs

Issue Type: Bug
In my launch.json for my Node.js app. I have an entry for an env. variable like:
env: { "PORT": "1234" }
If I run the app, stop it, and remove PORT from the env-object, it's still being set to the value it had during the previous run when I run the app again.
VS Code version: Code 1.66.2 (dfd34e8260c270da74b5c2d86d61aee4b6d56977, 2022-04-11T07:46:01.075Z)
OS version: Windows_NT x64 10.0.19043
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7400 CPU @ 3.00GHz (4 x 3000)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.95GB (3.75GB free)

Process Argv
--crash-reporter-id 9c139038-ec22-4ecf-b6d2-3be33fe3a365

Screen Reader
no

VM
0%

Extensions (16)

Extension
Author (truncated)
Version

hex-to-rgba
dak
1.0.0

xml
Dot
2.5.1

json-tools
eri
1.0.2

vscode-drawio
hed
1.6.4

rest-client
hum
0.24.6

vscode-peacock
joh
4.0.1

dotenv
mik
1.0.1

vscode-docker
ms-
1.22.0

python
ms-
2022.4.1

vscode-pylance
ms-
2022.4.3

jupyter
ms-
2022.3.1000901801

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.6

color-highlight
nau
2.5.0

vscode-thunder-client
ran
1.16.2

gitblame
wad
8.1.0

(1 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392:30443607
pythontb:30283811
pythonvspyt551cf:30345471
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
pythondataviewer:30285071
vscod805cf:30301675
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
pythonvsnew555:30457759
vscscmwlcmt:30465135
cppdebugcf:30475217

I can't repro this and I'm not sure why that would happen. Are you sure this happens as described? You could set "trace": true, in your launch config to get a log

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


Add .python-version

Change description
Adds a .python-version file that can be read by pyenv or uv when creating virtual environments. Should simplify setup, where the docker-runner's install-venv-all-repos.sh script can sometimes pick the wrong python if you have multiple installed.

@hamzabinkhalid sorry could you do this one as well please? 🙏


Dev

:tada: This issue has been resolved in version 2.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[#647] Add 'affiliation' to ListCollaborators.

Fixes #647.
Let me know if there is anything that needs to be changed or can be improved.
Thank you!

Hmmm...
This pull request, as well as #702 and #706, are build-failing because of an error in the coveralls, see this gist. The error occurs because encrypted environment variable $COVERALLS_TOKEN is not set in build.
Coveralls support and this environment variable were added in 89e28f7f1be 8 days ago, and no PRs since then have had passing builds.
Something is wrong with the coveralls, but I can't debug further due to the environment variable being secured.

Another potential issue is that this paragraph, which better explains the Affiliation parameter, is not in the docs:

For organization-owned repositories, the list of collaborators includes outside collaborators, organization members with access through team memberships, organization members with access through default organization permissions, and organization owners.

Should it be? Should the API be completely understandable only through by the github-go documentation, or should the developer be expected to read the API docs?

@e-beach you should be good to go now if you rebase your change against master.  It worked for my PR.

Should it be? Should the API be completely understandable only through by the github-go documentation, or should the developer be expected to read the API docs?

It's fine to leave it out. We don't expect the entire GitHub API to be understood only by looking at our godoc. We provide a subset of the docs, just the basics. Users are expected to look at GitHub API docs for in depth details.

I believe I addressed @shurcooL's comments. I suggest that you might merge #709 together with this one, as both PRs introduce breaking changes.


Add further locations

Most of these are just from brain storming.
Follow-up to: datafaker-net/datafaker#1189

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

Project coverage is 92.03%. Comparing base (b37c566) to head (972510e).
Report is 93 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1192      +/-   ##
============================================
- Coverage     92.35%   92.03%   -0.32%
- Complexity     2821     2889      +68
============================================
  Files           292      300       +8
  Lines          5609     5726     +117
  Branches        599      618      +19
============================================
+ Hits           5180     5270      +90
- Misses          275      295      +20
- Partials        154      161       +7

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

Looks great, let's merge!


[Select] Two clicks needed to change to another element

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯

Click first <Select>
Click on second <Select> (closes the first one but doesn't switch to the next one)
Click on second <Select>again

Expected behavior 🤔

Click first <Select>
Click on second <Select> (closes the first one but switches to the next one)

Steps to reproduce 🕹
Steps:

Click first <Select>
Click on second <Select> (closes the first one but doesn't switch to the next one)
Click on second <Select>again

Context 🔦
I want to have inline select components and be able to switch to either one of them with one click and not two.
Your environment 🌎
Forked demo
https://codesandbox.io/s/selectvariants-material-demo-forked-zyw54r?file=/demo.js

`npx @mui/envinfo`

  System:
    OS: macOS 12.1
  Binaries:
    Node: 17.3.0 - /usr/local/bin/node
    Yarn: Not Found
    npm: 8.3.0 - /usr/local/bin/npm
  Browsers:
    Chrome: 98.0.4758.109
    Edge: Not Found
    Firefox: Not Found
    Safari: 15.2
  npmPackages:
    @emotion/react: ^11.7.1 => 11.7.1
    @emotion/styled: ^11.6.0 => 11.6.0
    @mui/base:  5.0.0-alpha.69
    @mui/icons-material: ^5.4.1 => 5.4.1
    @mui/material: ^5.3.1 => 5.4.3
    @mui/private-theming:  5.4.2
    @mui/styled-engine:  5.4.2
    @mui/system:  5.4.3
    @mui/types:  7.1.2
    @mui/utils:  5.4.2
    @types/react:  17.0.38
    react: ^17.0.2 => 17.0.2
    react-dom: ^17.0.2 => 17.0.2
    typescript:  4.5.5

Hello @rferreira98, i just tried your codesandbox and i didn't find the error that you are talking about can you please explain it with more details?

If you open 1 of the selects, you need to click outside and perform another click on the other select to open it.
What I wanted to happen was that I clicked on one of them, it opened. And then click on the other one, without having to click twice to open it. @MahmoudMans

@rferreira98 now i understand

I want to work on this issue, is it possible to be assigned to this issue?

@MahmoudMans any news on this issue??

@rferreira98 still working on it

<Select> is implemented using the Popover, which is build on top of Modal. It is by design to create a backdrop, for disabling interaction below the modal.
I believe if you wish to use a Select component that is not implemented using Popover, I believe you will need to implement one yourself (I don't think MUI have such implementation currently), perhaps the ClickAwayListener and Popper will be useful in this implementation.

Mui Base offer a SelectUnstyled component which uses Popper instead of Popover.
It does not have this issue.


Update of system requirements

Hi all,
This PR makes some changes to the system requirements.
It's far from perfect, but probably still an improvement.
I look forward to any opinions.
Changes:

Removed "1.33GHz":
You won't get anything (new or quad-core) with a lower clock speed.

Replaced the "server future-proofed for 3-4 years" with something current
Dual socket 2014 CPU → Single socket 2023 CPU
SAS HDD → SSD
256 GB RAM → 8x 32 GB (match CPU memory channels)

CPU
"limited by GHz" → "limited by single thread performance" (clock speed != performance)
Remove the reference to a dual-socket CPU. If you don't need more than
96 cores / 2 TB RAM / 128 PCIe lanes, a dual-socket system is more of a disadvantage.

Recommended client
2GB RAM → 8 GB RAM
HDD → SSD
I really don't know what makes sense for OMERO Insight, but you don't want to work on a
current Windows or Linux desktop with just 2 GB RAM and a HDD nowadays.

Best regards,
Christian

Thanks @chdietrich


ForeignKey | Define column name of referenced class

Setup:

DBFlow Version: 4.0.0-beta1
Target Platform: Android 23
Gradle/Maven: Gradle v2.10 Android Gradle Plugin v2.0.0
Build Tools: 24.0.3
Kotlin: 1.0.3 with kapt

Issue Kind: Question
Description:
I want to reference a Model from another Model (See below).
The problem with the version below is, I get
Cannot find symbol method getFrom()
Cannot find symbol method getTo()
Cannot find symbol method getFrom()
Cannot find symbol method getTo()

@Table(database = MMHDatabase::class)
class Transaction(from: Account? = null, value: Float = 0.0f, to: Account? = null, id: Long = -1) : BaseModel() {
    companion object {
        fun get(id: Long) : Transaction? {
            return (select
                    from Transaction::class
                    where (Transaction_Table.id eq id)).result
        }
    }

    @PrimaryKey(autoincrement = true)
    var id: Long = -1

    @ForeignKey
    var from: Account? = null

    @Column
    var value: Float = 0.0f

    @ForeignKey
    var to: Account? = null
}

Corresponding generated code (Line 3 and 9):
 1public final void bindToInsertValues(ContentValues values, Transaction model) {
 2    if (model.getFrom() != null) {
 3      values.put("from_id", model.getFrom().getFrom());
 4    } else {
 5      values.putNull("from_id");
 6    }
 7    values.put("value", model.getValue());
 8    if (model.getTo() != null) {
 9      values.put("to_id", model.getTo().getTo());
10    } else {
11      values.putNull("to_id");
12    }
13  }

and (Line 3 and 9)
 1 public final void bindToInsertStatement(DatabaseStatement statement, Transaction model, int start) {
 2    if (model.getFrom() != null) {
 3      statement.bindLong(1 + start, model.getFrom().getFrom());
 4    } else {
 5      statement.bindNull(1 + start);
 6    }
 7    statement.bindDouble(2 + start, model.getValue());
 8    if (model.getTo() != null) {
 9      statement.bindLong(3 + start, model.getTo().getTo());
10    } else {
11      statement.bindNull(3 + start);
12    }
13  }

I expected it would use the get method of the primary key of the Account class which would be getId().

I got it nearly working with:
@Table(database = MMHDatabase::class)
class Transaction(from: Account? = null, value: Float = 0.0f, to: Account? = null, id: Long = -1) : BaseModel() {
    companion object {
        fun get(id: Long) : Transaction? {
            return (select
                    from Transaction::class
                    where (Transaction_Table.id eq id)).result
        }
    }

    @PrimaryKey(autoincrement = true)
    var id: Long = -1

    @ForeignKey(
        references = arrayOf(
            ForeignKeyReference(
                columnName = "from_account_id",
                columnType = Long::class,
                foreignKeyColumnName = "id",
                referencedGetterName = "getId",
                referencedSetterName = "setId",
                referencedFieldIsPrivate = true
            )
        )
    )
    var from: Account? = null

    @Column
    var value: Float = 0.0f

    @ForeignKey (
        references = arrayOf(
            ForeignKeyReference(
                columnName = "to_account_id",
                columnType = Long::class,
                foreignKeyColumnName = "id",
                referencedGetterName = "getId",
                referencedSetterName = "setId",
                referencedFieldIsPrivate = true
            )
        )
    )
    var to: Account? = null
}

same problem
  @Override   public final void bindToInsertStatement(DatabaseStatement statement, Voce model, int start) {     statement.bindLong(1 + start, model.getAnno());     statement.bindDouble(2 + start, model.getValore());     String refvaloreString = model.getValoreString();     if (refvaloreString != null)  {       statement.bindString(3 + start, refvaloreString);     } else {       statement.bindNull(3 + start);     }     if (model.getBanca() != null) {       statement.bindLong(4 + start, model.getBanca().getBanca());     } else {       statement.bindNull(4 + start);     }     if (model.getIndicatore() != null) {       statement.bindLong(5 + start, model.getIndicatore().getIndicatore());     } else {       statement.bindNull(5 + start);     }   }
but the correct code should be
  @Override   public final void bindToInsertStatement(DatabaseStatement statement, Voce model, int start) {     statement.bindLong(1 + start, model.getAnno());     statement.bindDouble(2 + start, model.getValore());     String refvaloreString = model.getValoreString();     if (refvaloreString != null)  {       statement.bindString(3 + start, refvaloreString);     } else {       statement.bindNull(3 + start);     }     if (model.getBanca() != null) {       statement.bindLong(4 + start, model.getBanca().getId());     } else {       statement.bindNull(4 + start);     }     if (model.getIndicatore() != null) {       statement.bindLong(5 + start, model.getIndicatore().getId());     } else {       statement.bindNull(5 + start);     }   }

With the combination of
referencedGetterName and referencedSetterName in conjunction with referencedFieldIsPrivate = true
I get:
@Override
  public final void bindToInsertValues(ContentValues values, Transaction model) {
    if (model.getFrom() != null) {
      values.put("from_account_id", model.getFrom().getId());
    } else {
      values.putNull("from_account_id");
    }
    values.put("value", model.getValue());
    if (model.getTo() != null) {
      values.put("to_account_id", model.getTo().getId());
    } else {
      values.putNull("to_account_id");
    }
  }

and
  @Override
  public final void bindToInsertStatement(DatabaseStatement statement, Transaction model, int start) {
    if (model.getFrom() != null) {
      statement.bindLong(1 + start, model.getFrom().getId());
    } else {
      statement.bindNull(1 + start);
    }
    statement.bindDouble(2 + start, model.getValue());
    if (model.getTo() != null) {
      statement.bindLong(3 + start, model.getTo().getId());
    } else {
      statement.bindNull(3 + start);
    }
  }

So that kind of is solved, unfortunately I have now an 'infix' modifier is required on 'eq' in 'IntProperty' problem (But seems like a different error)

can you post your model?

@xanscale: please have a look at my second post.

@paraqles i have very simple model
@Table(database = AppDatabase.class)
public class Voce extends BaseModel {

	@PrimaryKey(autoincrement = true)
	private long id;

	@ForeignKey
	private Banca banca;

	@ForeignKey
	private Indicatore indicatore;
}

how can i resolve?

@Table(database = AppDatabase.class)
public class Voce extends BaseModel {

    @PrimaryKey(autoincrement = true)
    private long id;

    @ForeignKey(
        references = @ForeignKeyReference(
                columnName = "banca",
                columnType = Long.class,
                foreignKeyColumnName = "id",
                referencedGetterName = "getId",
                referencedSetterName = "setId",
                referencedFieldIsPrivate = true
            )
        )
    )
    private Banca banca;

    @ForeignKey(
        references = @ForeignKeyReference(
                columnName = "indicatore",
                columnType = Long.class,
                foreignKeyColumnName = "id",
                referencedGetterName = "getId",
                referencedSetterName = "setId",
                referencedFieldIsPrivate = true
            )
        )
    )
    private Indicatore indicatore;
}

Assuming Banca and Indicatore are also derived from BaseModel and thier ID field is id and is of type Long. Also there must be getter and setter called setId and getId.

It is kind of ugly and not like it is stated in the comment of the corresponding code, but seems solved.

i confirm that works that workaround. I'm hoping for an official response

@paraqles  i have found another workaround
public class Indicatore extends BaseModel {
   public Long getIndicatore() {return id;}
}

i don't know if is better or not :D

Maybe, but in my example it would be:
class Account : BaseModel() {
    fun getFrom() : Int = id
}

that is totally not clear what is happening.

@paraqles i think more like this. each model class must have a method that return the primarykey that is called getMODEL, like this:
class From : BaseModel() {
    fun getFrom() : Int = id
}

@agrosner any opinion?

Maybe, but beside that, the method is named after the variable name for @ForeignKey and not after the referenced model name.
In my case the model name is Account and the Transaction model has a from (which is the source Account) and a to (target Account).
When you see the model needs a get$Model method it would be getAccount for the Account model, which is not the expected getFrom from the Transaction model @ForeignKey from.
That is the problem I see here and the only way to solve that is giving the parameters from above to the @ForeignKey annotation.
Which is not a big problem. The problem here is more the sentence in the code about the references property:

This no longer required as the library will auto-generate references for you based on the other table's primary keys.
com.raizlabs.android.dbflow.annotation.ForeignKey

From this sentence I expected, that the processor recognises that this is a reference to another Model and tries to find the @PrimaryKey of that.
I didn't expect, that it just uses the @ForeignKey variable name as a getter method.
If this reflection should be avoided, it would be ok if it took the getId as the method name.

@xanscale: you assume that the model I'm referencing is called From and not Account.
The misleading sentence here is more the one in ForeignKey.java

This is no longer required as the library will auto-generate references for you based on the other table's primary keys.

ForeignKeyReference[] references() default {};

com.raizlabs.android.dbflow.annotation.ForeignKey.java

From this sentence I expect that the processor uses reflection to find the @PrimaryKey of the referenced model and then uses the appropriate getter and not the reference holding variable name as getter on the referenced model.
If the reflection should be avoided, it can be a convention, that a model has to implement the getId or getPrimaryKey method which is more obvious what is expected from the model.

@paraqles so
class Account : BaseModel() {
    fun getAccount() : Int = id
}

i know that is a workaround and im waiting for an official response from @agrosner

I think this applies to private fields.  If you take away the private fields, this problem goes away?

@rogerhu if i remove foreignKey and relative field problem goes away.
i think this is common and big problem. no one else has the same problem?

Change your primary key columns to be public...does this issue go away?

I had that problem few minutes ago and changing from private to public solved an issue

@rogerhu package access solve issue perfectly.

Well I had the same problem and it was fixed with a public primary key. But this seems like a major bug introduced in the last version of this library right?

PACKAGE LEVEL is enough

@xanscale As I've said above, Kotlin has that same problem and everything is public by default. Workaround is somewhere in this thread, but manually writing methods to support library bugs is not really nice solution

since this is Kotlin, each property is treated private with associated getters and setters. I think it's getting confused because its calling :
 if (model.getFrom() != null) {
       statement.bindLong(1 + start, model.getFrom().getFrom());

Instead of :
 if (model.getFrom() != null) {
       statement.bindLong(1 + start, model.getFrom().getId());

I think that's the correct way you mention?

found issue! fixed in develop. Sorry for no response (I was on vacation for a little while).

Will this also work for an other @PrimaryKey than id (for example a string name)?

@agrosner this bug affect java only projects too. your fix seams to solve only kotlin. i'm wrong?


Header of Born effective charges in VASP 5.4

Hi. The header of the BORN effective charges in VASP 5.4 (old) is a bit different: "BORN EFFECTIVE CHARGES (in e, cummulative output)". The extraction of the lotosplitting file fails because of this. I just changed it to "BORN EFFECTIVE CHARGES (i" for both versions 5 and 6. In case you would like to update it.
https://github.com/flokno/tools.tdep/blob/531cf1f579927826dae8519a6506670cd21de174/tdeptools/scripts/tdep_read_lotosplitting_from_outcar.py#L8C1-L8C70

thanks! fixed in 7d81ed571442291d30478f620994543de2b35c5d


When deleting a challenge, repeat the challenge name in the confirmation dialog

Currently, the confirmation dialog just asks if you're sure:

It would be nice to repeat the action you are about to confirm. In this case, deleting a challenge. Especially on bigger / wider screens, the three dot menu can be quite a distance removed from the challenge name, so it is conceivable that the user would be less than 100% confident that they selected the challenge they wanted to act on:

This is now on maproulette.org


[ZEPPELIN-6144] Add Helm chart for deploying Zeppelin on Kubernetes

What is this PR for?
A few sentences describing the overall goals of the pull request's commits.
First time? Check out the contributing guide - https://zeppelin.apache.org/contribution/contributions.html
What type of PR is it?
Feature
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-6144
How should this be tested?

Strongly recommended: add automated unit tests for any new or changed behavior
Outline any manual steps to test the PR here.

Screenshots (if appropriate)
Questions:

Does the license files need to update?
Is there breaking changes for older versions?
Does this needs documentation?

@Reamer  Please take a look again when you have time, thank you! We have already used this Helm chart in the production environment to do interactive analysis with Spark.


Add new cop Style/FloatDivision

StyleGuide: https://rubystyle.guide/#float-division

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

@bbatsov @koic Added third config option right_coerce and kept left_coerce as default (as per style guide).

After some deliberation I agree with @koic that the best default would be something that just warns if you use to_f on both sides. Let's also update the style guide accordingly.

See also https://blog.rubystyle.guide/ruby/2019/06/21/float-division.html - unfortunately we got no feedback on it.

Thanks for your consideration. I opened a PR https://github.com/rubocop-hq/ruby-style-guide/pull/767 to rubocop-hq/ruby-style-guide repo.

@koic PR updated. Changed default style to single_coerce.


chore: restrict webpack versions to latest compatible version

PR Checklist
Please check if your PR fulfills the following requirements:

[ x ] The commit message follows our guidelines: link tbd.
[ x ] Tests for the changes have been added (for bug fixes / features)
[ x ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ x ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] tooling / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?
The latest currently supported webpack version is just below 5. Currently that limitation is not apparent.
Issue Number: #91
What is the new behavior?
The user will get a notice in the terminal when using an incompatible webpack version and it's properly documented on which versions are compatible.
Does this PR introduce a breaking change?
[ ] Yes
[ x ] No

Other information

cool. thanks a lot.
I would merge it immediately, but unfortunately the PerformanceTest fails. We should adjust something there.


abapGit Version scheme is different from any other git client

Hi guys,
we are operating GitHub Enterprise @ SAP and we are using Hubble https://github.com/Autodesk/hubble for analysis purposes. One of the Hubble reports is showing the git client versions used https://autodesk.github.io/hubble/users-git-versions. Unfortunately abapGit gets logged with versions like:
abapGit-v1.68.0-sap-1
abapGit-v1.66.0-sap-1
abapGit-v1.50.2`
whereas any other client uses a scheme where the version goes first and any additional information after that like:
2.13.6 (Apple Git-96)
2.13.5 (Apple Git-94)
2.13.1.windows.1
<IP_ADDRESS>3.gb1f2c4b
2.12.2.windows.2
2.12.2.windows.1
2.12.2 (Microsoft Windows NT 10.0.16299.0; Win
is there any way I can get you to consider having the version to go first and anything else after that because the text in front completely messes up the report?
We also logged an issue in the Hubble Project https://github.com/Autodesk/hubble/issues/160
Best
Ingo

I think we can change it, no problem
Will something like "v1.68.0-abapGit" work? Or should it be "1.68.0-abapGit"?
https://github.com/larshp/abapGit/blob/be0b9d0df454cd67f9f7f68d202a0713bc7f1603/src/http/zcl_abapgit_http.clas.abap#L208

Thanks @larshp and @IngoS11 ! However, I feel git/1.68.0-abapGit could be misleading. It would basically say you are using the original Git client code version 1.68.0 with a few abap fixes. That's how Git for Windows does the versioning. But you are not using the code of the original Git client. As far as I understand it, the abap Git client is a reimplementation like libgit2 or JGit. Please correct me if am wrong!
libgit2 and JGit use the following version schemes:
git/2.0 (libgit2 0.26.0)
git/2.0 (libgit2 0.27.0)
JGit/2.1.0-SNAPSHOT
JGit/<IP_ADDRESS>502260915-r

That means your versioning should look more like this:
abapGit/1.6.8

or
git/2.0 (abap 1.68.0)

correct, abapGit is a custom implementation of git
I think I had "abapGit/1.6.8" in some early implementations but changed it. Since some sites(like bitbucket.org) returns an error if it is not a known git client

I see. Would git/2.0 (abapGit 1.68.0) be an option? I think that expresses that you are somewhat compatible with git 2.x and your implementation is abap 1.68.0.

git/2.0 (abapGit 1.68.0) looks good to me
@IngoS11 any thoughts?

@pks-t @ethomson did I capture the intention of the libgit2 version string correctly? (see two messages 👆)

Yeah, I encourage you to do exactly that (git/2.0 (your version)).  Some hosting providers pattern match and expect git/\d in order to ensure that you're a real Git client.

(git/2.0 (your version)) would perfectly work for me. That way we could keep the original Hubble git-versions report and build ourselves another report showing the various Flavors like (Windows, Apple, Linux, ABAP etc.) or something along that line.

closing, change merged


Update colab_online.ipynb

fix the json error.

online demo still disable

When trying to download the model, there is an error: Google Drive - Quota exceeded.
Why there is no discussion in this repo?


Arbitrarily set axes

Description

Allows arbitrarily setting axes with the Viewer.axes property. This is a major, breaking change and will require extensive changes to current implementations of layers and other dimension-related features.
This is a work-in-progress. Planned changes include:

[x] implementing core axes property in viewer
[ ] matching sliders to axes properly
[x] making corresponding changes to image layers
[ ] making corresponding changes to markers layers
[ ] making changes to Viewer.update_statusBar implementation

Type of change

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

References

#83
How has this been tested?

[ ] example: the test suite for my feature covers cases x, y, and z
[ ] example: all tests pass with my change

Final checklist:

[ ] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works

Hello @kne42! Thanks for submitting the PR.

In the file gui/elements/_viewer.py, following                are the PEP8 issues :

Line 36:1: W293 blank line contains whitespace
Line 105:1: W293 blank line contains whitespace
Line 108:1: W293 blank line contains whitespace
Line 111:1: W293 blank line contains whitespace
Line 315:37: E231 missing whitespace after ','
Line 315:39: E231 missing whitespace after ','
Line 315:73: E231 missing whitespace after ','
Line 315:75: E231 missing whitespace after ','
Line 315:80: E501 line too long (96 > 79 characters)
Line 324:29: E231 missing whitespace after ','
Line 362:80: E501 line too long (85 > 79 characters)

There are no PEP8 issues in the file gui/layers/_base_layer.py !

In the file gui/layers/_image_layer.py, following                are the PEP8 issues :

Line 46:1: W293 blank line contains whitespace

I started having a look through the PR and it all seems reasonable to me. My understanding of it is that you've now added a new ddims property which has the number of displayed dimensions - for right now just 2, but in the future we will support 1 and 3, and you've added an axes property which contains a tuple of the indicies to be displayed on the x, y dimensions of the canvas dimensions depending on their order. In the with future (x,y,z) or just (x) will be supported too.  This seems like a good approach.
I think getting the dimension sliders hooked up and then making sure that transposes / dimension swaps from the notebook work for the image layers is the next priority. Right now neither worked for me.
I'm happy to then help with the markers layer and statusBar if needed :-)

I would like to hack on this topic in the next two days,
and update the sliders GUI accordingly. this PR seems
to be stale compared to  master. I would like to take this
over. We can discuss this during the meeting.

This PR has been superseded by the new dimensions model in #171
Although #171 does not add full support for rapidly switching which axes are the displayed ones it should provide a better foundation on which to implement that feature and develop further


Vue update

moved to composition API

I would like feedback on the current looking API. For Vue 2, calling .value everytime is a must. For Vue3, in the template (html) you don't need to call .value. If you guys think calling .value everytime is too much, we can merge all state variable into one big object I think.
Also, I will prepare a vue 2 demo as well so we can see how it looks

vue 2 demo can be found here: https://github.com/Picovoice/porcupine/tree/vue2-demo/demo/vue


Giving credit to the contributors

A section should be added to the readme to give credit to anyone who has helped directly or indirectly in the development of mailboxer.

Closed in https://github.com/ging/mailboxer/pull/29 thanks to @dickeyxxx


Links on a blog post aren't routed correctly in some instances

The links contained within a blog post, like in the navigation for instance will route to the wrong place. An example would be if you click one of the map links, it would take you to narco.news/blog-post-title/maps instead of going to narco.news/maps. This then those a 404 error because obviously nothing exists at that link.
This seems to only happen under particular circumstances I've noticed. If you start at the home page and navigate to /wire and then the blog post, this usually won't happen. But if you instead open up the website by visiting the blog post directly via its URL than the links loaded and generated are all wrong.

Fixed in #3

Fixed in 92ce0344518acf59374f8602f7b931f5b1beb803


Feat/address checksum

Describe your changes
Added methods for checksum addresses.
Linked issues
Closes https://github.com/software-mansion/starknet-jvm/issues/182
Breaking changes

[x] This issue contains breaking changes

Our keccak used a mask right away. This might be misleading for users. I decided to follow starknet.js and rename it to starknetKeccak. Return types are different, so there shouldn't be any mistakes in using the wrong one.

Codecov Report
Base: 70.26% // Head: 70.53% // Increases project coverage by +0.27% :tada:

Coverage data is based on head (bef5f00) compared to base (3e066c0).
Patch coverage: 100.00% of modified lines in pull request are covered.

:exclamation: Current head bef5f00 differs from pull request most recent head f3bc172. Consider uploading reports for the commit f3bc172 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #195      +/-   ##
==========================================
+ Coverage   70.26%   70.53%   +0.27%
==========================================
  Files          53       53
  Lines        1527     1541      +14
  Branches      105      109       +4
==========================================
+ Hits         1073     1087      +14
  Misses        424      424
  Partials       30       30

Impacted Files
Coverage Δ

...ain/kotlin/com/swmansion/starknet/crypto/Keccak.kt
100.00% <100.00%> (ø)

...lin/com/swmansion/starknet/crypto/StarknetCurve.kt
82.22% <100.00%> (ø)

...mansion/starknet/data/ContractAddressCalculator.kt
100.00% <100.00%> (ø)

...ain/kotlin/com/swmansion/starknet/data/Selector.kt
100.00% <100.00%> (ø)

...in/com/swmansion/starknet/extensions/BigInteger.kt
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Feat/26 api gateway icon

add a demo of API gateway to show that it works with icon network

Note
In this MR, you can check demux-sync of ICON-Loop
Enable watcher on ICON loop
Create .env file as below
app_enable_eos=false
app_enable_icon=true


Verifier fails when there is no CredentialStatus

The current verifier for VCs generated with interop profile fails -
failed to verify: {"checks":[{"check":"status","error":"credential doesn't contain status"}]}

The changes from edge-service are included as part of #244 .


Unenforced field constraints error with update_forward_refs

Checks

[x] I added a descriptive title to this issue
[x] I have searched (google, github) for similar issues and couldn't find anything
[x] I have read and followed the docs and still think this is a bug

Bug
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
        pydantic version: 1.9.0
       pydantic compiled: True
            install path: [HIDDEN]/.venv/lib/python3.8/site-packages/pydantic
          python version: 3.8.11 (default, Aug 22 2021, 20:13:49)  [Clang 11.0.3 (clang-1<IP_ADDRESS>)]
                platform: macOS-10.16-x86_64-i386-64bit
optional deps. installed: ['email-validator', 'typing-extensions']

Trying to create a recursive model with pydantic I stumbled on an unenforced field constraints error.
I created a minimal reproduction that illustrates the bug. Running the code below:
from __future__ import annotations
from typing import List, Optional
from pydantic import BaseModel, Field

class Data(BaseModel):
    field: Optional[List[Data]] = Field(
        None,
        min_items=1,
    )

Data.update_forward_refs()

model = Data(field=[Data()])
print(model)

outputs:
  File "test.py", line 5, in <module>
    class Data(BaseModel):
  File "pydantic/main.py", line 204, in pydantic.main.ModelMetaclass.__new__
  File "pydantic/fields.py", line 486, in pydantic.fields.ModelField.infer
  File "pydantic/schema.py", line 1012, in pydantic.schema.get_annotation_from_field_info
ValueError: On field "field" the following field constraints are set but not enforced: min_items.
For more details see https://pydantic-docs.helpmanual.io/usage/schema/#unenforced-field-constraints

Removing the min_items constraints:
from __future__ import annotations
from typing import List, Optional
from pydantic import BaseModel, Field

class Data(BaseModel):
    field: Optional[List[Data]] = Field(
        None,
        # min_items=1, <-----
    )

Data.update_forward_refs()

model = Data(field=[Data()])
print(model)

works as expected:
field=[Data(field=None)]

Am I right in thinking this is a bug or am I misunderstanding unenforced field constraints?

I'm running into the same issue running pydantic 1.9.0 in python 3.9.7 on linux.

Seems using from __future__ import annotations causes field annotation becomes ForwardRef["some type"], pydantic will not resolve constraints for ForwardRef.

Given the error message, it looks like this is expected. 🤔
This actually made me find a bug on V2. I've documented it on https://github.com/pydantic/pydantic/issues/5645.


Remove player joined/left messages.

To avoid metagaming and allow admins to be slightly more stealthy.

Should they be kept for admins to see?

Should they be kept for admins to see?

Maybe in admin log? IMO a system like char name (role)[ID/playername]. Or even just player and their role.
That way if it's significant you can care.

Should they be kept for admins to see?

Sounds good and fair


Add some crashers to the validation test suite.

@swift-ci Test and merge.

@shahmishal Linux PR testing is failing due to a cmake problem, but the regular CI is passing.
Configuration problem perhaps?

Can you please try clean Linux test?
Sent from my iPhone

On Oct 2, 2016, at 11:25 AM, Mark Lacey<EMAIL_ADDRESS>wrote:
@shahmishal Linux PR testing is failing due to a cmake problem, but the regular CI is passing.
Configuration problem perhaps?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@swift-ci Clean test Linux

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - fb5b023dc9af00037c935cdff64f1dc4b7f18e28
Test requested by - @rudkx

Strange, this test does not seem to be failing on Linux. I believe we're doing an asserts-enabled build for testing and we're not hitting the assert we see on macOS, either.

@swift-ci Clean test and merge.

@swit-ci Clean test.

@swift-ci Please test.

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - fb5b023dc9af00037c935cdff64f1dc4b7f18e28
Test requested by - @rudkx

@swift-ci Please clean test.

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - fb3470266ad15348fae98a30c318b7aac2bc3994
Test requested by - @rudkx

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - fb3470266ad15348fae98a30c318b7aac2bc3994
Test requested by - @rudkx

@swift-ci Please clean test.

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - fb3470266ad15348fae98a30c318b7aac2bc3994
Test requested by - @rudkx

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - fb3470266ad15348fae98a30c318b7aac2bc3994
Test requested by - @rudkx

@swift-ci Please smoke test and merge.

@swift-ci Linux smoke test.

@swift-ci Linux smoke test.

@swift-ci smoke test Linux

@swift-ci smoke test Linux


Support command line args

Add's support for tunneling command line args to the target script. Also makes the target script launch like it was invoked by node <targetScript>.
This builds on top of PR https://github.com/myndzi/wtfnode/pull/5, only this commit https://github.com/myndzi/wtfnode/commit/87f24a44997e0b2271c99e34f740567991674ad9 is necessary to review.

Can you rebase this to master? I can help if you need, I remember the first time someone asked me to rebase a PR... ha!

Also: Instead of throwing an error (includes a stack trace), this should use
  console.error('whatever');
  process.exit(1);

or something to that effect.
I understand that -- is standard for separating "my program's arguments" from "proxied program's arguments", but since wtfnode has no arguments (yet), we can defer adding this, making execution simpler for users.

I'll make those changes and update the PR.

Feedback incorporated and rebased.

Ok, done for real this time!

I didn't know about the Module.runMain() thing, that's pretty neat


Remove the type alias from the Fn1 definition.

This kind of definition prevents creating type class
instances that are parametrized by this type.

What's the problem this solves? You can't declare instances for synonyms anyway, so to get the desired behaviour it'd be an instance for (->).
This probably would be a better definition though:
type Fn1 = (->)

But really I'm not really sure why we have it at all, it doesn't seem like there's any advantage to obfuscating the normal function arrow.

I was trying to update the purescript-typeable project to use the latest version of purescript. When upgrading it I ran into the following exception:
Error found:
in module Data.Typeable
at /Users/roa/Spaces/purescript-typeable/src/Data/Typeable.purs line 174, column 3 - line 177, column 3

  Type class instances for type synonyms are disallowed.

in type class instance

  Data.Typeable.Typeable2 Fn1

I think this error is due to defining the Fn1 as a type alias and I do not know how I could get the purescript-typeable project to compile otherwise.

Ah ok. Yeah, Fn1 used to be a foreign type like this, but was changed to a synonym as it's redundant. Just changing the instance to be for (->) instead of Fn1 would fix it.

I am ok with that, but I still think that it would be more consistent if we could use Fn1 in an instance definition (the same way you can use FnX).

You can effectively use Fn1 in an instance declaration though - you just have to use Function.


Keep screen on while video is playing

Should there be an option to keep the screen on while the video is playing?
Sounds like a plan to me.

I'd consider that moot, as some would not want it. Please feel free to send a pull with a feature implementing an option to do so.


Link expired for pre-trained limited FOV model

You previously uploaded a limited FOV pre-trained model here. That link has since expired again, would you be willing to re-upload it?

Thanks for letting me know. I have put it here.

I will close the issue if there are no further questions.


[es-dev-server] Failed to resolve module specifier "lit-html"

After last update fresh bootstrap of a web component using npm init @open-wc the following errors are thrown after running npm start
Uncaught TypeError: Failed to resolve module specifier "lit-html". Relative references must start with either "/", "./", or "../".
Could not resolve module imports in /demo/inline-module-0.js?source=%2Fdemo%2Findex.html: Unable to parse the module, this can be due to experimental syntax or a bug in the parser.

Not sure what is happening but it definitely worked yesterday.

The workaround is to run npm run start:compatibility

Thanks a lot of reporting this! Apparently es-module-lexer 0.3.7 just got released which breaks a lot of things. Pinning the version to 0.3.6 makes things work again!

It works again now with es-dev-server 1.15.1

sweet! I could deal with demo page but I got worries when tests couldn't start as well.
Now it all works again <3


Missing project.html and adus.json

Folks, this is an amazing work and is quite intriguing. I've tried to deploy this repo to play around and noticed two issues:
a. Upon start the app tries to load adus.json from the IMAGE_PATH and fails if it is missing. I assume it is environment specific?
b. The main handler in the app app.py refers to project.html template but it is not present in the repo (line 35). Would you, please, upload it?

a. project.html was an unused placeholder; the backend/flask server does not serve any webpages, simultaneously run the the react.js client for that purpose.
b. Use this notebook to collect the adus.json file.


APPS-2444 - Calling refresh when init'ing the conversations recycler …

…view
Fixes initial query execution when conversation recycler view is in a ViewPager and gets a visibility changed call before init() is called.
Since the adapter is notified through a Hander message on the main thread, there won't be problems configuring the RecyclerView after the query runs.

Still need to update the release version and change log but wanted to get


Pg executor

closes https://github.com/insitro/redun/issues/78
If you want to accept this, I guess this would be the minimum left to do

[x] add CLI subcommand to start executor worker process
[x] add&fix type annotations
[ ] add instructions & docs sections for running
[ ] test internals of executor (send/receive to/from queue)
[x] test a bunch of different / missing settings in the ini file
[x] test executor with workflows

@mattrasmus this would be very helpful for us. What is the approval mechanism here?

hi @ernestum
Thanks for the review. I'm no longer working on this experiment, it was a nice starting point into understanding how this works; I ended up working on a new framework using different opinions, requirements and backends
Feel free to take over the code if you want to further pursue this feature

All right. Will do. Thanks for the huge initial chunk of work!
May I ask what other framework you ended up using @gabriel-v?

Did a custom reinterpretation of this repo's DAG definition API using async python and scylladb, seeing some great throughput out of it - will leave a comment here when we publish it if you're interested

Yes that would be great!

meanwhile you can look at https://github.com/temporalio/samples-python
we're evaluating it and it's a very strong contender


IntLiterals should be rewritten to FloatLiteral if the type is inferred to be Float

Describe the bug
When the model contains a comparison between a Float variable and a constant that's a whole number such as a == 0 the evaluation behaves incorrectly because the whole number gets translated to an IntLiteral which is not comparable to Float.
To Reproduce
Steps to reproduce the behavior:

Use the following model:

test : Float -> Bool
test a =
  if a == 0 then
    True
  else
    False

Load the model in Morphir Web.
Pass in 0 as the input.
Notice that the result is False

Expected behavior
The whole number should be translated to a FloatLiteral if the expected type is Float so that the evaluation correctly compares it.

The problem seems to be more nuanced: we already had this fix implemented and the issue only happens if we use type aliases and our fix doesn't correctly detect that the actual type is Float. This model reproduces the issue:

type alias Price = Float

test : Price -> Bool
test a =
  if a == 0 then
    True
  else
    False


added new boss-key-wayland lua script

added boss key rewrite using xdg-portals in linux

Thanks for the PR.


MAX_REVISIONS_TO_SEARCH is too small; no command line argument to override it

What steps will reproduce the problem?

I just tried running a 'Moe magic' migration from a git repository which
had 126 commits to migrate. However, due to branching history, Moe
apparently needed to review 406 revisions. As there's no commandline
argument to override MAX_REVISIONS_TO_SEARCH, I had to run a modified
version of Moe in order to migrate.

Suggest either:
- Increasing MAX_REVISIONS_TO_SEARCH by at least an order of magnitude
- Providing a command line flag to override the default

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 31 Jan 2013 at 6:14

+1 also ran into this twice for different projects already.


Missing license

Just saw your water showcase in the bevy discord and it looks great! :)
Sadly, this project has no license so it's unknown if it's fine to take inspiration from your code (which I assume is fine since you uploaded it publicly anyway).
I would suggest you add a LICENSE to your project!

I'm in the middle of a large refactor and things are kind of broken at the moment. I'll think about licenses once things are working again. For now I will leave this open. And thanks for bringing this to my attention!

I've added the license files.


Update introduction-to-device-guard-virtualization-based-security-and…

…-windows-defender-application-control.md
the proposed changes are intended to de-emphasize the term "Windows Defender Device Guard" and highlight the change we are making to better support the promotion of WDAC  in out security stack

@Justinha, can you review this PR?


The icon display of activity bar and menu bar doesn't look normal. I don't know why?

VS Code Version:VS Code Version:VSCodeUserSetup-x64-1.55.2
OS Version:win 10

Steps to Reproduce:
1.The icon display of activity bar and menu bar doesn't look normal. I don't know why?
2.

The icon display of activity bar and menu bar doesn't look normal. I don't know why?

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

This looks a similar issue to https://github.com/microsoft/vscode/issues/85872 and https://github.com/microsoft/vscode/issues/110288.
Do you have any special font software installed (like mactype)? Anything that would disable fonts from loading?

Closing in favor of https://github.com/microsoft/vscode/issues/122021

This looks like a similar issue to #85872, #110288, and #122021
Do you have any special font software installed (like mactype)? Anything that would disable fonts from loading?

Thank you for your reminder. I saw that I installed mactype. After I tried to disable it, the display was normal.


Fix mutation observer / lifecycle management on IE

This fixes the lifecycle management on IE (e.g. incorrect calls to fallback instead of created and removed).

Not tooooo much more green in this so it seems a lot has changed in their polyfill. Hopefully the performance tests check out. If so let's merge and recommend it's usage in master (soon to be 0.14)!

I created a JSPerf here: http://jsperf.com/mutationobserver-perf

I added a childList test to the jsPerf fiddle, however it seems as if the Skate polyfill does not correctly recognize all element changes.


WINDOWS 10 do not restart after use shared folders on Mac M1

Describe the issue
After sharing a directory between MAC and the VM the sharing work but with a message of "file to big" but the VM does not restart and gets stuck on reboot. The VM is a Windows 10 for x86 which worked without any problems prior to sharing. The VM has the SPICE tools installed.
Configuration

UTM Version: 4.0.6 (65)
macOS Version: 12.6 Monterey
Mac Chip (Intel, M1, ...): M1

Crash log
If the app crashed, you need a crash log. To get your crash log, open Console.app, go to Crash Reports, and find the latest entry for either UTM, QEMU, QEMUHelper, or qemu-*. Right click and choose Reveal in Finder. Attach the report here.
Debug log
For all issues related to running a VM, including crashes, you should attach a debug log. (This is unavailable for macOS-on-macOS VMs. Attach an excerpt of your system log instead.)
To get the Debug log: open UTM, and open the settings for the VM you wish to launch. Near the top of the QEMU page is Debug Log. Turn it on and save the VM. After you experience the issue, open the VM settings again and select Export Log... and attach it here.
Upload VM
If your issue is related to a specific VM, please upload the config.plist from your VM's .utm directory. To get this, right-click the VM in UTM and choose "Show in Finder". Then right-click the selected file in Finder and choose "Show Package Contents". The config.plist file is now visible. Right-click it and choose "Compress". Attach the resulting config.plist.zip file here.
You can upload the entire .utm if needed, but note this includes your VM's drive image and may contain personal data. Since Github has an attachment size limit, you may want to upload to another service such as Google Drive. Link it here.

It was a CPU error. I choose Intel instead of default


[Event Hubs] Updates to EventProcessor post UX Study

We had a UX Study for the new Event Hubs library where we saw a bigger success in user understanding the usage of the EventProcessor after we replaced the factory model for Partition Processor that returned objects with the concept of having a base class defined that the user can extend.
Though in the Javascript community one can create objects far easier than classes and would prefer the same, the ease of extending a base class and the ease of passing only the class name instead of implementing a factory which had 2 layers of indirection took the upper hand. This is especially handy in cases where Javascript users not familiar with "interfaces" try to new them up inside the factory assuming that they are classes.
This issue is to track the changes required to update EventProcessor such that

PartitionProcessor is a class implemented by the library which replaces the existing interface. This will implement the 4 methods initialize, processEvents, processError and close such that it only logs relevant information in each
User is expected to extend this class and pass the class name to the EventProcessor constructor
PartitionProcessor gets a new method updateCheckpoint() with overloads to support both eventData and the offset/sequenceNumber combination. This internally calls the new updateCheckpoint() on the partition context. This way we reduce 1 new concept that the user has to understand i.e that of the CheckpointManager
Tests and samples updated

cc @chradek, @ShivangiReja, @bterlson

Closing this as we've made changes to EventProcessor, and have further changes described in #5103


Ensure config file takes precedence over CLI

This is due to fixes in PME issue https://github.com/release-engineering/pom-manipulation-ext/issues/381

Code change itself looks fine.
Can the PME config file specify arbitrary Groovy stuff for execution? If so, it either needs to be locked out, or the whole adjust operations needs to be re-architected. The design assumption for /adjust was the executed command was controlled/limited by the system admin, not the developer of the adjusted code.

@ASzc We should probably discuss Groovy separate to this issue? Currently, be it on the cmd line or in a cfg file, a remote groovy script from a maven repository can be executed by specifying -DgroovyScripts=GAVTC. One simple option might be to provide a way of disabling the Groovy manipulator?

It can be fixed separately, yes.
The xml and json manipulators may need to be disabled as well, as those might be able to escape the current directory?


sbom: add support to arm64

Make sure to properly deploy sbomutil for the right architecture.

/lgtm


Add the possibility to convert geojson values to geo objects

I first tried to implement Into with a panic! but this trait must not fail
So I looked at TryInto which seems unstable.
I finally created a custom TryInto

Sorry for the delay! Looks good, thanks! 🎉

Sorry for the delay! Looks good, thanks! 🎉


No Typescript support

24:3 Argument of type '{ store: Store<any>; props: {}; head: { title: { inner: string; }; }; beforeCreate(this: Vue): Promise<void>; mounted(): void; }' is not assignable to parameter of type 'ComponentOptions<Vue, DefaultData<Vue>, DefaultMethods<Vue>, DefaultComputed, PropsDefinition<Record<string, any>>, Record<string, any>>'.
  Object literal may only specify known properties, and 'head' does not exist in type 'ComponentOptions<Vue, DefaultData<Vue>, DefaultMethods<Vue>, DefaultComputed, PropsDefinition<Record<string, any>>, Record<string, any>>'.
    22 |
    23 |   },
  > 24 |   head: {
       |   ^
    25 |     title: {
    26 |       inner: 'It will be a pleasure',
    27 |     },

Is it possible to add? kr.

@zvaehn
Did you manage to solve?
You can take a look at this issue:
https://github.com/ktquez/vue-head/issues/71

I closed the issue, but if you still have questions, just open again, ok?


Pipe name wasn't matching between client & server in the README.md

I guess the README.md was just out if sync, it did confuse me a bit though. This is why I created a PR for this.

Thank you @Lakritzator !


Windows Mixed Reality Commander: Add File Management support

Add support (using the Windows Device Portal Wrapper) to access the device file system. This will enable scenarios such as a proctor updating survey questions / retrieving responses, etc

This would be a very nice addition. When testing certain apps my team has developed for HoloLens, we need to upload a config file to the device in order to have the app connect to our test server properly. Additionally, we pull down crash dumps and logs from the device. Being able to upload/download device files all through the Commander would be awesome.

It is important to note that this is not full file system access. It allows access to the folders that are available to the device portal.

Yep, an important distinction. So in the device portal there is a tab for App Crash Dumps and a tab for File Explorer. Are both of those able to be accessed?

File Explorer tab functionality is what I am targeting with this change. Crash dump access should be a separate feature request

i'll have to double check if the underlying API is different (I think it is)


[Process Agent] Use global config for APIEndpoints

What does this PR do?

This PR moves the process_config.process_dd_url and process_config.additional_endpoints to the global config.
Motivation

This is part of the process agent config refactor
Additional Notes

This PR does not remove APIEndpoints from AgentConfig. For the sake of simplicity, we decided it was better to move the two config settings that are responsible for configuring APIEndpoints to the global config, and keep APIEndpoints in AgentConfig.
Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Set api_key, process_config.process_dd_url, and add an entry to process_config.additional_endpoints. Make sure that all of these values point to a valid dd org, and that all of these orgs are receiving data when you run the process check.
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] The appropriate team/.. label has been applied, if known.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the config template has been updated.

👍🏻 from docs team

Thank you very much!


Recommend to use NTP pools additionally? Prefer PHC?

For GCP it is recommended to remove the other time sources.
Should we do the same for Azure? (My Rocky 8 image uses 2.pool.ntp.org additionally)
What is the Microsoft recommendation?
Also it seems to often prefer these other clock sources usually, adding a "prefer" option to the refclock entry or adding a "server" entry for "PHC" don't seem to make any difference.
Is there any way to accomplish this?
From GCP documentation:
https://cloud.google.com/compute/docs/instances/configure-ntp#linux-chrony
"If you see a single record pointing at metadata.google or metadata.google.internal, you do not need to make any changes. If you see multiple sources, mixed between metadata.google and a public source such as pool.ntp.org, update your sources to remove any external NTP servers."
Best regards,
Benjamin

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

ID: 8f5e6687-3b97-7f25-5e7b-91265764c816
Version Independent ID: d3f3b17a-a25b-656b-314f-25002c5d5f9f
Content: Time sync for Linux VMs in Azure - Azure Virtual Machines
Content Source: articles/virtual-machines/linux/time-sync.md
Service: virtual-machines
GitHub Login: @cynthn
Microsoft Alias: cynthn

@langeschro
Thanks for your feedback! We will investigate and update as appropriate.

@langeschro
I'm going to assign this to the document author so they can take a look at it accordingly
@cynthn
Can you please check and add your comments on this doc update request as applicable.

@mabicca You have more experience with the time services, what do you think of this recommendation?

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. If you are still experiencing a problem, you might want to move this question to Microsoft Q&A: https://learn.microsoft.com/en-us/answers/tags/94/azure-virtual-machines.  #please-close

We do recommend using the PHC as one of your time sources.  It would be best to use it in conjunction with of a pool of networked time sources as a precaution in case something goes wrong with host time.
https://learn.microsoft.com/en-us/azure/virtual-machines/linux/time-sync
FWIW most endorsed Linux images on Azure are using it out of the box with Chrony.


Restructure sentence to include .NET Framework

Why:
Restructure a sentence to include .NET Framework as one of the examples as instructed #1308
What's being changed:
Updates finding-security-vulnerabilities-and-errors-in-your-code/troubleshooting-the-codeql-workflow.md
In step 5 of the "No code found during the build" section, the following statement can also apply to .NET Framework projects:

For C# projects using either dotnet build or msbuild which target .NET Core 2, you should specify /p:UseSharedCompilation=false in your workflow's run step, when you build your code. The UseSharedCompilation flag isn't necessary for .NET Core 3.0 and later.

Change this to:

For .NET Framework projects, and for C# projects using either dotnet build or msbuild that target .NET Core 2, you should specify /p:UseSharedCompilation=false in your workflow's run step, when you build your code. The UseSharedCompilation flag isn't necessary for .NET Core 3.0 and later.

Check off the following:

[x] All of the tests are passing.
[X] I have reviewed my changes in staging.
[X] For content changes, I have reviewed the localization checklist
[X] For content changes, I have reviewed the Content style guide for GitHub Docs.

@Perkles Thanks so much for opening a PR! I'll get this triaged for review ✨


nixos/modemmanager: Add qmicli and mbimcli when using fccUnlockScripts

The ModemManager shipped fccUnlockScripts are using either qmicli or mbimcli, to unlock wwan modems. These needs to be available for ModemManager if you set the fccUnlockScripts link, so instead of having every user than needs it pull it inn in different ways, we should just provide the tools if we think they might be needed.
https://modemmanager.org/docs/modemmanager/fcc-unlock
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
24.05 Release Notes (or backporting 23.05 and 23.11 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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This seems like reasonable change, and what's expected here based on upstream docs. Thanks!


Improve flexibility of network timeouts in libindy

@jovfer We need documentation for timeouts and pre_ordered_nodes params in libindy and all wrappers

Agreed that will be provided by next PR to don't block teams