ARM: Move CMSIS out of main tree

Introduction
This is a proposal to move the CMSIS out of the main Zephyr repository.
Problem description
CMSIS is one of the two "hal" layers that are still part of the main Zephyr tree, residing inside the /ext folder.
Proposed change
We would like to remove the ARM CMSIS "hal" layer from the /ext directory of the Zephyr main tree and instead, place it in a "hal" repository in the Zephyrproject organization, which, ideally, will mirror the ARM CMSIS repository.
Detailed RFC
The repository content could be fetched using the west tool, as we do for all other HAL layers.
Dependencies
Not applicable.
Alternatives
Keep having the CMSIS as part of the main zephyr repository.

FYI @galak @stephanosio @MaureenHelm @carlescufi @erwango
Is this something we should try to agree upon and do for v2.3 release?

FYI @galak @stephanosio @MaureenHelm @carlescufi @erwango
Is this something we should try to agree upon and do for v2.3 release?

There was a "requirement" a while back that one should be able to build for qemu without any additional modules/repos. This is the reason we keep CMSIS in the tree. Perhaps @nashif can comment since I recall him being the one that first introduced this concept (which kind of makes sense to me as well).

It is not documented, anywhere that we need to build QEMU without additional repositories (to the best of my knowledge, of course)
Anyway, this does not even hold, today, at least for qemu_cortex_m0 (Nordic) and qemu_cortex_m3 (TI) that, I believe, require vendor HALs.

Anyway, this does not even hold, today, at least for qemu_cortex_m0 (Nordic) and qemu_cortex_m3 (TI) that, I believe, require vendor HALs.

Not for qemu_cortex_m3. The UART driver is in-tree.

Ah, OK. Are you sure this Soc/Board does not need TI HALs at all?
In any case the argument holds for the M0 Qemu

Unless there is something wrong with my current understanding of how things work, HALs (including CMSIS) need not reside in the main repository.
The first thing that CI does is to west update and this means that everything (including the to-be-modular-ised CMSIS) will get pulled before the CI builds tests and samples.

Unless there is something wrong with my current understanding of how things work, HALs (including CMSIS) need not reside in the main repository.
The first thing that CI does is to west update and this means that everything (including the to-be-modular-ised CMSIS) will get pulled before the CI builds tests and samples.

Yes, but the idea here was that downstream distros of zephyr that do not necessarily include those HALs can still run the qemu basic kernel tests.

Ah, OK. Are you sure this Soc/Board does not need TI HALs at all?

Almost sure.

In any case the argument holds for the M0 Qemu

Right, but this is new.

Re: CMSIS

CMSIS consists of many different components (e.g. Core, DSP, NN), and the current Zephyr EXT CMSIS only includes the CMSIS-Core component.
CMSIS codebase, including all the applicable components, is quite large and converting it into a module is a reasonable approach to take.
There is already a pending PR to add the CMSIS-DSP component (https://github.com/zephyrproject-rtos/zephyr/pull/21600) and it has been requested the CMSIS be moved out into a separate repo/module for this.
Simply mirroring and patching the upstream CMSIS repository may not suffice for modularisation due to the following reasons:

There will be many unnecessary components included (e.g. CoreValidation, DAP, Driver, RTOS, RTOS2).
The CMSIS-DSP (digital signal processing) and CMSIS-NN (neural network), which are primarily of interest, use CMake-based build system and this will be directly in conflict with the Zephyr build system (unless we nuked them all in the patches, but this will not be crucial to maintain in the long term).

@carlescufi FYI there will be one more QEMU platform that will probably require external HAL (STM32) support; the Cortex-M4F: #23185.

@stephanosio @ioannisg I have no objections to going forward with this. We should create a new repo for CMSIS that contains all the other applicable components as well (DSP in particular is of interest to many). I would however like to hear from @MaureenHelm, @galak and @nashif before we go ahead.

@stephanosio note that for CMSIS-DSP we will need to settle this first:
https://github.com/zephyrproject-rtos/zephyr/issues/7516
A discussion is ongoing at the board level regarding this.

@stephanosio note that for CMSIS-DSP we will need to settle this first:
#7516
A discussion is ongoing at the board level regarding this.

@carlescufi In #21600, I have already integrated CMSIS-DSP to the Zephyr build system (with full Kconfig support for customisation) in the source code form and linking binary blobs will not be necessary; I suppose #7516 won't be relevant in that case?
In terms of licensing, all CMSIS components (including the CMSIS-DSP) are licensed Apache 2.0.

I'd ask if we need the qemu_cortex_m3 platform at all.  Is mps2_an385 sufficient?

I'd ask if we need the qemu_cortex_m3 platform at all. Is mps2_an385 sufficient?

@galak https://github.com/zephyrproject-rtos/zephyr/issues/22870

@carlescufi In #21600, I have already integrated CMSIS-DSP to the Zephyr build system (with full Kconfig support for customisation) in the source code form and linking binary blobs will not be necessary; I suppose #7516 won't be relevant in that case?

@stephanosio maybe I'm missing something, so CMSIS-DSP is offered both in source code and library form in the CMSIS repo? I thought it was only available as libraries.

I'd ask if we need the qemu_cortex_m3 platform at all. Is mps2_an385 sufficient?

Agreed with @stephanosio. Once Cortex-M4 is supported in qemu, m3 is not required at all.

@stephanosio maybe I'm missing something, so CMSIS-DSP is offered both in source code and library form in the CMSIS repo? I thought it was only available as libraries.

Yep, the upstream CMSIS repository contains both the source code and pre-built libs for the CMSIS-DSP.
The source code can be found here:
https://github.com/ARM-software/CMSIS_5/tree/develop/CMSIS/DSP/Source

I'd ask if we need the qemu_cortex_m3 platform at all. Is mps2_an385 sufficient?

We tried to remove it when we enabled mps2_an385 but there was some resistance due to some drivers and network tests depending on it if I remember correctly. IMO we should consolidate this and cleanup many of the qemu duplicates we have now, also for x86.

if we are going to remove cmsis and put it in a module, we should also remove the altera hal at the same time and probably other items in ext and complete abolish "ext/"

For ARMv7-M things are pretty clear; we don't need qemu_cortex_m3, we have mps2_an385 for a Cortex-M3 and we can make it run without MPU. We want to have Cortex-M4F - that's what #23185 will do.

if we are going to remove cmsis and put it in a module, we should also remove the altera hal at the same time and probably other items in ext and complete abolish "ext/"

Any particular reason these things need to happen "at the same time"? CMSIS and altera-hal are totally unrelated. Agree with abolishing /ext if possible - just that I am more keen of doing things in steps

We need to sync whatever the CMSIS HAL repo would be w/TFM's CMSIS usage as well.

@stephanosio @ioannisg I will move forward with this. I think it should be done in steps:

Create the repo
Populate it with CMSIS-Core
Move CMSIS-Core from ext/ to the repo
Populate the repo with CMSIS-DSP
Add CMSIS-DSP support in Zephyr


HTTP Error 500

Hi there
I'm having an issue with Tonic. I'm following this tutorial : http://labs.tonik.pl/theme/docs/installation/
Everything runs smooth, but when I try to activate the theme, it become HTTP error 500, and I cannot open wp-admin. So I have to delete tonic theme folder.
I'm trying to increase PHP memory limit, but still no luck. I'm using MAMP by the way, with PHP 7.1.6.
Any help would be greatly appreciated
Thanks!
[UPDATE]
After activate WP_DEBUG and WP_DEBUG_LOG, I get this error instead :
Parse error: syntax error, unexpected '}', expecting \\ (T_NS_SEPARATOR) in /Applications/MAMP/htdocs/web/wordpress/wp-content/themes/tonictheme/src/helpers.php on line 3

Are you using ^2.0.0 version of the starter?
It looks like you are using old version ^1.0.0 where you have to init starter with CLI before activating.

Thanks for the response
I'm installing using Composer.. So, where I can find the newest version?
Thank you

You can install it with composer with composer require tonik/theme. Afterwards just make sure that in composer.json file you have ^2.0.0 version

Thank you! :)


Running slowly when the form has many input controls

When the form has many controls, for instance 10 which should be validated and 150 controls which
shouldn't  be validated then the validation engine will check every of ( 10 + 150) = 160 controls.
It takes about 5 seconds.
Could you please add a way in which we will be able to override default "INPUT_SELECTOR".
Validator.INPUT_SELECTOR = ':input:not([type="submit"], button):enabled:visible'
For the large form we will be able to mark useful controls with some css-class or etc.
Validator.INPUT_SELECTOR = ':input.useful_flag_css:not([type="submit"], button):enabled:visible'

Yup, you already can by overriding $.fn.validator.Constructor.INPUT_SELECTOR

@1000hz Thanks!
It would be great to see such features in the manual


[Bug]: Connecting a swarm manager / worker gets stucked and doesn't allow deploy unless it's a standalone server

Description
After configuring a swarm cluster, and trying to add the manager and worker nodes on Coolify, the "Validate & Install" process get's stuck on "Docker minimum version", which actually I got v26 of docker. (Tried with v25 and upper versions between 24 and 26 which is the latest)
I tried setting up on Ubuntu 20, 22 and 24 LTS versions as the dependencies of the project rely on docker itself.
The process seems to work fine, it validates everyting but the "Docker minimum version" check keeps loading. I even left it for 24 hours as i couldn't find any logs on /data/coolify/logs.
On worker nodes, the error is clear "this is not a swarm manager, try docker swarm init", even if its a worker. Setting a swarm cluster with all manager nodes also not possible because of "Docker minimum version" check.
After some time, closing the right panel the server eventually appears as "validated" but it's impossible to deploy anything on it.
I tried setting coolify on a separated instance from the swarm cluster ( 1 manager, and 3 workers) and even as a service on a swarm cluster but the swarm nodes verification keeps getting stuck.
Minimal Reproduction (if possible, example repository)

Create a Key, and add it as authorized_key for the server willing to configure.
Add new server, configure IP/Domain and mark Manager node
Click on Validate & Install
all checks go green except "minimum docker version" which hangs forever

Same for worker node but different error. (no swarm manager, try swarm init).
Exception or Error
Could not find any logs on /data/coolify/logs
Container logs are not useul as its a server configuration issue between coolify and docker swarm.
Version
v4.0.0-beta.297

Facing the same issue... It also avoids being able to deploy databases

Same issue here: v4.0.0-beta.319

I have the same issue too

Still facing the same issue here...

The issue of not being able to deploy seems to come from the fact that compose files, on which databases and premade templates are based upon, need heavy changes to work on swarm, which prevent them from running there. Swarm support is still experimental, as someone said on the discord:

I'd say that Docker swarm is 30% supported. Left and right there are hard breaks in code for docker swarm, so functionality is not always implemented. Give it a try I'd say


Exceeded query limit reply, but Google console

When using the Populartimes populartimes.get function I am getting a "You exceeded your Query Limit for Google Places API Web Service, check https://developers.google.com/places/web-service/usage to upgrade your quota."
(Small area's work before getting the API error.
When checking the usage on the Google Developers Console, I don't see any quota exceeding it's limit. (see picture: https://imgur.com/a/eoBG9zB )
Could it be that the amount of threads/ workers is set too high (Eventhough the request per seconds limit has not reached yet according to Google)?
What is the best way to limit the requests per second/ limit the threads in the crawler?
I guess this post is old (old quota's and limits), but maybe related?
https://stackoverflow.com/questions/14014074/google-maps-api-over-query-limit-per-second-limit

@vermaat this is probably because google requires you to fill out billing information regardless of if you have or have not reached the quota.

Hi JoshiahParry,
Thanks for the reply. I have my bank account connected to the API. As you can see in the image (https://imgur.com/a/eoBG9zB ) I am not even close to the set limits.

@vermaat Where you able to find any possible solution to this problem?


building wot seems failing randomly

2024/10/07 21:25:52 🔗 listening at http://localhost:3355
2024/10/07 21:25:55 🌐 building web of trust graph
2024/10/07 21:26:11 🫂 total network size: 1219
2024/10/07 21:26:11 🔗 relays discovered: 156
2024/10/07 21:26:11 🌐 pubkeys with minimum followers:  227 keys
2024/10/07 21:26:11 📢 subscribing to inbox

and just a few min later
2024/10/07 21:26:58 🌐 building web of trust graph
2024/10/07 21:27:14 🫂 total network size: 13696
2024/10/07 21:27:14 🔗 relays discovered: 444
2024/10/07 21:27:14 🌐 pubkeys with minimum followers:  9799 keys
2024/10/07 21:27:14 📢 subscribing to inbox

the log does not provide any more information but i assume just like with blastr it just randomly fails on any relay and stops moving forward.

there must be a log, can you please provide the output of sudo journalctl -u haven -n 1000

It’s docker, and what i pasted is part of decker log - unless i am missing setting of some logging level Sent from my iPhoneOn 8 Oct 2024, at 16:03, Barry Deen 'notifications at github.com' @.***> wrote:
there must be a log, can you please provide the output of sudo journalctl -u haven -n 1000
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

I see. please retry without docker. docker support will be removed next week in 0.4.0 as outlined in #33
unfortunately nobody was willing to provide docker support and it's causing too many issues
closing for now, please reopen if you continue to get this issue running it bare metal

lol, oh man
you make it too difficult for me :)
On Oct 8 2024, at 7:59 pm, Barry Deen 'notifications at github.com' @.***> wrote:

I see. please retry without docker. docker support will be removed next week in 0.4.0 as outlined in #33 (https://github.com/bitvora/haven/pull/33)
unfortunately nobody was willing to provide docker support and it's causing too many issues
closing for now, please reopen if you continue to get this issue running it bare metal
—
Reply to this email directly, view it on GitHub (https://github.com/bitvora/haven/issues/30#issuecomment-2400600816), or unsubscribe (https://github.com/notifications/unsubscribe-auth/ABHOAM4ZF5DWTNPZNWCMLSLZ2QTQ7AVCNFSM6AAAAABPQ2EIFOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBQGYYDAOBRGY).
You are receiving this because you authored the thread.

@eleaner https://github.com/HolgerHatGarKeineNode/haven-docker
and pre-built docker images here: https://hub.docker.com/r/holgerhatgarkeinenode/haven-docker

you guys are amazing!
On Oct 8 2024, at 11:56 pm, The Ben 'notifications at github.com' @.***> wrote:

@eleaner (https://github.com/eleaner) https://github.com/HolgerHatGarKeineNode/haven-docker
and pre-built docker images here: https://hub.docker.com/r/holgerhatgarkeinenode/haven-docker
—
Reply to this email directly, view it on GitHub (https://github.com/bitvora/haven/issues/30#issuecomment-2400956794), or unsubscribe (https://github.com/notifications/unsubscribe-auth/ABHOAM6HBUX4QYKHFRU3Y5DZ2RPJHAVCNFSM6AAAAABPQ2EIFOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBQHE2TMNZZGQ).
You are receiving this because you were mentioned.


Added sharing option to Tumbir and Pinterest

Hacktoberfest notes:

due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
please read our contributing guidelines
be sure to check the output of Travis-CI for linter errors
if this is your first open source contribution, make sure it's not your last!

What does this PR do?
Add Resource(s) | Remove Resource(s) | Add info | Improve Repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Not a duplicate
[ ] Included author(s) if appropriate
[ ] Lists are in alphabetical order
[ ] Needed indications added (PDF, access notes, under construction)

I just added sharing option to Tumblr and Pinterest (2 Lines) and line-breaks between section unchanged.
Some have 2 line-breaks between sections and some have 1 line-breaks. so i didn't change it

Please propose this in the Issues section - I don't think either Tumblr or Pinterest are a good targets for use as they're very picture oriented and we have no pictures. We can reopen if the community wants it.


fix: apply adapter-cloudflare trailing slash fix to adapter-cloudflare-workers

fixes https://github.com/sveltejs/kit/issues/10937
This PR applies the same adapter-cloudflare fix from https://github.com/sveltejs/kit/pull/8733 which I forgot to add to adapter-cloudflare-workers in the past. This should fix the trailing slash issue in the aforementioned issue where the trailing slash of /base/ was being stripped too early and therefore, would never match with the prerendered page /base/.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

@benmccann please don't mind if I tag wrong person, I'm not sure who is a code owner of cloudflare-adapter, but could it be that you can merge that pull request?

Yeah, I've been rather busy prepping for Svelte Summit this Saturday, but I'll get back to reviewing PRs after that


feat(notes panel): add ui outline for notes panel

Summary

Test plan

A picture of a cute animal (not mandatory but encouraged)

Hi @sidag95, thank you for putting this up and sorry for the late follow.
I like the direction of the PR! How would you like to proceed?
I think the tab order should be Preview, Notes, i18n, WDYT?
For the buttons we'll need to use the same styling as other CMS buttons, e.g.

Because the way we ship the CMS (see https://github.com/netlify/netlify-cms/issues/4967 and https://github.com/netlify/netlify-cms/issues/2026) we'll need to class components and not react hooks.

@sidag95 are you still interested in moving this forward?

@sidag95 are you still interested in moving this forward?

Hey @martinjagodic, at this moment, it would be difficult for me to proceed with this :/


maybe can update you notebook with the eval function, and how to validate it with testing dataset?

@MheadHero
Yes, I will provide a complete notebook.

@MheadHero
Hello, I have updated the notebooks with mAP evaluation updates.
Please close the issue if this solves what you are looking for.


region accounts

So API endpoint currently is http://api.dotmailer.com which works fine as this is in alias to http://r1-api.dotmailer.com but more regions coming soon. USA(r2) is already live which means that the http://api.dotmailer.com will not work for region 2.
Curent regions :
http://r1-api.dotmailer.com (UK)
http://r2-api.dotmailer.com (USA)
Which region to use for an api user can be retreive from getAccountInfo(the only api call that is valid for any endpoints)

@romanpitak do you have any feedback on this PR ;) ?


Rename option params so they start with "allow_" ?

Would it be better if we rename reserved to allow_reserved (similarly for other option params) to more clearly convey whether something is allowed or not?  I thought it was a bit confusing when I looked at my own examples e.g. at #13 .

Or reservable.

And also validates :my_url_attribute, url: true  -- I'd rename url: boolean to something like  is_url, is_url_type. Because "url" implies a none, a value such as "http://example.com".

Actually, in Rails other validators are named in the same format, therefore I think:

url: true | { some_settings } is ok
reserved -> allow_reserved

Pushed to the branch "develop".


feat(fun_prop): command to print out fun_prop theorems

Command #print_fun_prop_theorems HAdd.hAdd  prints out all fun_prop theorems associated with addition.

Command useful for maintenance of fun_prop attribute. Allows you to to see all the theorems for a given function thus spotting duplicated and bad theorems more easily.
This will also be useful on deciding priority for new fun_prop theorems once priority is supported.

Oh, please update PR description to include the failure cache.

bors r+

Did you see I also left 2 review comments?

Snap I missed that, I just installed the GitHub mobile app and I'm all confused by it.

Feel free to cancel, or do a tiny follow-up PR.

How do I cancel?

bors r-

bors merge


Titles are not importing in the correct order

Here is a list of multiple titles:
https://gist.github.com/cam156/0eedd5b00a0087c5390400cb65b77058
We should likely apply the title metadaat again on all these records since we are seeing records with them out of order.

https://github.com/psu-stewardship/scholarsphere/blob/develop/app/actors/curation_concerns/actors/generic_work_actor.rb for live data

Here's the process I used to fix the mis-ordered titles in production-new:
https://gist.github.com/awead/45c15e78381c60804db7e2aef06ac802


ValueError: cannot reshape array of size 4800 into shape (240,10)

Hello,
The code didn't work when I using
F = audioFeatureExtraction.stFeatureExtraction(x, Fs, 0.050Fs, 0.025Fs);
which says
Traceback (most recent call last):
File "", line 1, in
File "pyAudioAnalysis/audioFeatureExtraction.py", line 577, in stFeatureExtraction
curFV[2] = stEnergyEntropy(x)                    # short-term entropy of energy
File "pyAudioAnalysis/audioFeatureExtraction.py", line 51, in stEnergyEntropy
subWindows = frame.reshape(subWinLength, numOfShortBlocks, order='F').copy()
ValueError: cannot reshape array of size 4800 into shape (240,10)
Could you give me any suggestion to solve it? Thanks for your time.

The code works when I use
F = audioFeatureExtraction.stFeatureExtraction(x[:, 0], Fs, 0.050Fs, 0.025Fs)
It seems that the audio with two channels is not work. What should I do, any examples for this kind of audio?
Thanks

These functions are designed for single channel audio signals. I am not sure if extracting a feature for multiple channels does even exist. You can merge the audio first, by for example averaging the channels or extract the features for each channel. It depends on the meaning of the channels. That is if they are captured by multiple microphones for example.

Yup. Apparently taking the mean of the two channels works. @scut-salmon

you can use two channels   .wav file

audioBasicIO.stereo_to_mono(signal) does the averaging. See https://github.com/tyiannak/pyAudioAnalysis/blob/master/pyAudioAnalysis/MidTermFeatures.py#L328-L329


I would Love to Provide Solution to this Question

Aim
I would Love to Provide Solution of this Problem Under LGMSOC in C++
Details
Pashmak decided to give Parmida a pair of flowers from the garden. There are n flowers in the garden and the i-th of them has a beauty number bi. Parmida is a very strange girl so she doesn't want to have the two most beautiful flowers necessarily. She wants to have those pairs of flowers that their beauty difference is maximal possible!
Your task is to write a program which calculates two things:

The maximum beauty difference of flowers that Pashmak can give to Parmida.
The number of ways that Pashmak can pick the flowers. Two ways are considered different if and only if there is at least one flower that is chosen in the first way and not chosen in the second way.

Problem Link : https://codeforces.com/contest/459/problem/B
Programming language

[ ] C
[ ] C++
[ ] Java
[ ] Python

Sorry we have closed this issue first you have to work on one issue

Sorry we have closed this issue first you have to work on one issue

I have Merged it sir can I Work on this Issue now ?


[mesheryctl] Remove the release executable file

/cc @leecalcote

Build checks failing. Re-running jobs.

Error in taskq version. They told me its fixed bow

Build checks failing. Re-running jobs.

Build should be fixed now 🤞

Oh, very good, @kushthedude. Re-running build checks on this PR now...

No luck, yet, @kushthedude

🤞

🤞

No luck, yet, @kushthedude

Pulling the master did the job, will tell those guys to make a new release. It is good to go @leecalcote


WIP: First take at udp filter chain

Signed-off-by: Sumukh Shivaprakash<EMAIL_ADDRESS>For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Description: Adds a filter chain to the udp listener stack.
Risk Level: Low
Testing: Added unit tests for new code and ran existing //test/...
Docs Changes: In progress
Release Notes: In progress
Adds feature for #6748
[Optional Deprecated:]

@mattklein123 - This is a first draft, if you could take a quick peek to see if things are in the right direction. Will start working on adding the test coverage this week.

I would like to see a full integration test with a fake filter for this PR though. Can you work on that? Thank you!

@mattklein123 - Just to clarify, does this integration test you are referring to live inside this repo? Ensuring you are not referring to the external repo with the DNS filter because that may take more time as it needs the RFC implemention.

Just to clarify, does this integration test you are referring to live inside this repo?

Yes sorry, in this repo.  I think the equivalent of https://github.com/envoyproxy/envoy/blob/master/test/integration/echo_integration_test.cc for UDP would be great.

Would like to hear from @mattklein123 and @alyssawilk on https://github.com/envoyproxy/envoy/pull/6912#discussion_r283945445

Just to clarify, does this integration test you are referring to live inside this repo?

Yes sorry, in this repo.  I think the equivalent of https://github.com/envoyproxy/envoy/blob/master/test/integration/echo_integration_test.cc for UDP would be great.

I see that the current echo filter lives inside the product code. Should the udp echo also live there?

@mattklein123 - I see that the current echo filter lives inside the product code. Should the udp echo also live there?

I would probably just put it in the test directory, then we don't need to bother with documentation, etc.

@sumukhs is this still WIP or is it ready for final review? If so can you update the PR title? Thank you!
/wait-any

@sumukhs is this still WIP or is it ready for final review? If so can you update the PR title? Thank you!
/wait-any

Sorry, this has been done for the past few days. Waiting for a signoff. Updated the title now.

Looks good over all! Just a few nits.

Thanks for the in-depth review @danzh2010 . Just pushed an update


Schema Migration fix for newer mainnet schema data

Goal
The goal of this PR is to update the schemas migration with respect to the newer data on chain.
Part of #2006
Discussion

Only reset the name list as needed
Add in schema 19 so that it is re-added at the top of the list

here is the command I ran to check the migration against the mainnet. You need to compile the runtime with frequency and try-runtime flags
cargo build --features frequency,try-runtime
RUST_LOG=debug try-runtime --runtime ./target/debug/wbuild/frequency-runtime/frequency_runtime.wasm on-runtime-upgrade  live --uri wss://1.rpc.frequency.xyz:443 --pallet System --pallet Schemas


ProjGeographicToGeocentric is set to use Projected/Geocentric rather than Geographic/Geocentric

Thank you for making this plugin.  I believe I've found an error in GeoReferencingSubsystem.cpp at line 269...
ProjGeographicToGeocentric = GetProjection(ProjectedCRS, FString("EPSG:4978"));
ProjectedCRS should be GeographicCRS
After changing this I was able to successfully project geographic coordinates to geocentric coordinates and back.

Hi tmek. Thanks for your feedback !
Indeed, this was a silly copy/paste error. I can't publish the fix myself right now, but I'll make it done.
BTW, there is also a small mistake related to axis convention (left->right handed).
It should be :
void UGeoReferencingSubsystem::EngineToProjected(FVector EngineCoordinates, FLargeCoordinates& ProjectedCoordinates)
{
ProjectedCoordinates = FLargeCoordinates(
EngineCoordinates.X * 0.01 + OriginCoordinates.X,
-EngineCoordinates.Y * 0.01 + OriginCoordinates.Y,
EngineCoordinates.Z * 0.01 + OriginCoordinates.Z);
}
void UGeoReferencingSubsystem::ProjectedToEngine(FLargeCoordinates ProjectedCoordinates, FVector& EngineCoordinates)
{
EngineCoordinates = FVector(
ProjectedCoordinates.X - OriginCoordinates.X,
-(ProjectedCoordinates.Y - OriginCoordinates.Y),
ProjectedCoordinates.Z - OriginCoordinates.Z) * 100;
}


CLI should not encode API response schema

I've experience several instances where a schema change in the controller must be reflected in the CLI to properly render the expected output on the command line because the current CLI implementation encodes the types too strictly and unnecessarily - in every case I've encountered so far where I've made a schema change to add or remove a field, the CLI would have been just fine if it treated the response from the controller opaquely.
We should refactor the API respond handling in the CLI so that it uses more generic JSON object interface rather than strict types that just get in the way.
I've also found several instances where the CLI is not in sync with the controller (Evidenced by several defects I've opened for these in recent days) and the CLI injects properties that should not exist (and hence confusing).

I was bit by this as well and it's not fun. In fact, if I hadn't checked with RR, I would have wasted a lot of time on chasing a "bug" in my code, while it was really a schema change for activations which is not reflected in the cli.

@ioana-blue, see https://github.com/openwhisk/openwhisk/pull/1566 for the PR that will address your issue.

Issue moved to apache/incubator-openwhisk-cli #148 via ZenHub


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 456 ms

Resolved: ClamAV performance has improved in 9ebe139 after 5 minutes.


SC integrity check

Worth looking to see what happened here:
LEGACY SUPPLY CHAIN S
Notice date: 2023-10-29
Location: eCrvhiacrele IsIt o("nLegacy"), SC
Jobs: 47

Another:
THIVEWORKS ADMINISTRA
Notice date: 2023-11-30
Location: tSivtaet eSweridviec -e sM, LuLltiCple, SC
Did the PDF columns move, perhaps?


Create a shared component for Tempus key-value message content in JSON

Parent of #297
Create a shared component that generates Tempus key-value message content in JSON that is accepted by the Connect, Attribute and Telementry (time and depth) messages for both Devices and Gateways

This is no longer needed.


Fixed #1772

Corrected basic configuration sample for ASP.NET Core following @phatboyg's comments.

I think this is fixed.


Fix server error on blog post for 3.1

Won't this break 3.2?

Just tested in 3.2, and it works as advertised :)


Upgrade to GraalVM 21.0.0

To be released Jan 19, 2021.

See related #410 issue.

See related #410 issue.

Let's use -H:+InlineBeforeAnalysis by default based on #410 findings in order to allow Spring and Tomcat to use utility methods for native image detection.

Let's use -H:+InlineBeforeAnalysis by default based on #410 findings in order to allow Spring and Tomcat to use utility methods for native image detection.

See related https://github.com/spring-projects/spring-framework/issues/25795 issue. I will test that all our sample works with -H:+InlineBeforeAnalysis enabled by default before implementing it.

See related https://github.com/spring-projects/spring-framework/issues/25795 issue. I will test that all our sample works with -H:+InlineBeforeAnalysis enabled by default before implementing it.


Rework transaction behaviour for YouTube operations

Codecov Report
Attention: 19 lines in your changes are missing coverage. Please review.

Comparison is base (a1bf48b) 3.51% compared to head (81c5aa1) 3.50%.

:exclamation: Current head 81c5aa1 differs from pull request most recent head 5e4d14f. Consider uploading reports for the commit 5e4d14f to get more accurate results

Files
Patch %
Lines

server/app/shows/[show_id]/youtube/actions.ts
0.00%
19 Missing :warning:

Additional details and impacted files
@@           Coverage Diff            @@
##            main    #180      +/-   ##
========================================
- Coverage   3.51%   3.50%   -0.01%
========================================
  Files         75      75
  Lines       2078    2081       +3
  Branches     411     411
========================================
  Hits          73      73
- Misses      2005    2008       +3

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


gnmic never closes connection for Target.Client

In our long-running process, we are using gnmic as a library. We are connecting to Arista EOS and we've noticed that connections are never closed:
sh-4.2# netstat -no |grep 6030
tcp6       0      0 <IP_ADDRESS>:6030     <IP_ADDRESS>:53896     ESTABLISHED keepalive (10.10/0/0)
tcp6       0      0 <IP_ADDRESS>:6030     <IP_ADDRESS>:53894     ESTABLISHED keepalive (9.84/0/0)
tcp6       0      0 <IP_ADDRESS>:6030     <IP_ADDRESS>:53898     ESTABLISHED keepalive (10.35/0/0)
tcp6       0      0 <IP_ADDRESS>:6030     <IP_ADDRESS>:53892     ESTABLISHED keepalive (9.33/0/0)
tcp6       0      0 <IP_ADDRESS>:6030     <IP_ADDRESS>:53890     ESTABLISHED keepalive (8.81/0/0)

All we do is basically this in a loop:
conf := config.New()
conf.GetTargets()
tgt := target.NewTarget(conf.Targets[socket])
tgt.CreateGNMIClient(ctx)
tgt.Get(ctx, req)
tgt.Stop()

It looks like conn.Close() is never executed for the selected connection.

Hi,
Thanks for taking the time to write this issue.
First let me say that using gnmic as lib needs some work and proper documentation.
The conn.Close() you are pointing at is intended to close the secondary connection ( when the primary one is successful ) if the target configuration has a comma separated list of addresses. Is it the case for your targets?
tgt.Stop() is meant to stop a gnmi connection used to run a subscribe RPC,
if you need to close the connection the gnmi client uses for other RPCs, canceling the context ctx should do it.

First let me say that using gnmic as lib needs some work and proper documentation.

Yeah, we've noticed :D. It's a great CLI, though!

The conn.Close() you are pointing at is intended to close the secondary connection ( when the primary one is successful ) if the target configuration has a comma separated list of addresses. Is it the case for your targets?

No, we have only one address. Btw, In case of more than one address I believe there is a small chance for a race condition there. I mean, one more gorutine can send a connection through the channel before done is closed.

tgt.Stop() is meant to stop a gnmi connection used to run a subscribe RPC, if you need to close the connection the gnmi client uses for other RPCs, canceling the context ctx should do it.

According to grpc module documentation, when DialContext  is used in a non blocking way(like here) ctx does not act against the connection.
If it did, this cancel would make GNMIClient unusable.

Btw, In case of more than one address I believe there is a small chance for a race condition there. I mean, one more gorutine can send a connection through the channel before done is closed.

You are right, this was added with the assumption that only one of the connections will be successful.
I can add a step that checks if the target has a gNMI client, if true close the "secondary connection"

According to grpc module documentation, when DialContext  is used in a non blocking way(like here) ctx does not act against the connection.
If it did, this cancel would make GNMIClient unusable.

Maybe I misunderstood your requirement, if this is a long running process, there is no need to recreate a new gNMI client for each RPC. tgt.Get() will reuse the existing gNMI client tgt.Client.
You will just need to supply a new ctx to each tgt.Get()
I agree that there is no way to gracefully close the underlying gRPC connection right now, so what I can do is:

Adding the used gRPC connection as a field under target.Target
Adding a target.Close() method that closes the gRPC connection if it's not nil
Rename target.Stop() to target.StopSubscriptions()

Do you think this will make it more useful for your use case ?

Adding a target.Close() method that closes the gRPC connection if it's not nil

This sounds good

Can you give the last release (0.21.0) a try?
I added the target.Stop() method as well a target.ConnState() method that returns the connection state as a string.
here is a sample of how it can be used to a Get and a Set RPCs:
package main

import (
	"context"
	"log"
	"time"

	"github.com/AlekSi/pointer"
	"github.com/karimra/gnmic/target"
	"github.com/karimra/gnmic/types"
	"github.com/karimra/gnmic/utils"
	"github.com/openconfig/gnmi/proto/gnmi"
)

func main() {
	ctx, cancel := context.WithCancel(context.Background())
	defer cancel()

	// build the targe configuration
	tc := &types.TargetConfig{
		Name:       "srl1",
		Address:    "srl1:57400",
		Username:   pointer.ToString("admin"),
		Password:   pointer.ToString("admin"),
		SkipVerify: pointer.ToBool(true),
		Timeout:    10 * time.Second,
	}

	// create the target
	t := target.NewTarget(tc)

	// parse path
	p, err := utils.ParsePath("/system/name/host-name")
	if err != nil {
		panic(err)
	}
	//
	for i := 0; i < 10; i++ {
		log.Printf("iteration %d", i)
		// create a gNMI client
		err := t.CreateGNMIClient(ctx)
		if err != nil {
			panic(err)
		}
		log.Printf("target %q connectivity state: %q", t.Config.Name, t.ConnState())

		// send gNMI GetRequest
		getRespone, err := t.Get(ctx, &gnmi.GetRequest{
			Path:     []*gnmi.Path{p},
			Encoding: gnmi.Encoding_JSON_IETF,
		})
		if err != nil {
			panic(err)
		}
		log.Printf("target %q connectivity state: %q", t.Config.Name, t.ConnState())
		log.Printf("gNMI GetResponse: %s", getRespone)

		// change the target hostname
		hostname := "\"srl1\""
		if i%2 == 0 {
			hostname = "\"srl2\""
		}
		setResponse, err := t.Set(ctx, &gnmi.SetRequest{
			Update: []*gnmi.Update{
				{
					Path: p,
					Val: &gnmi.TypedValue{
						Value: &gnmi.TypedValue_JsonIetfVal{
							JsonIetfVal: []byte(hostname),
						},
					},
				},
			},
		})
		if err != nil {
			panic(err)
		}
		log.Printf("target %q connectivity state: %q", t.Config.Name, t.ConnState())
		log.Printf("gNMI SetResponse: %s", setResponse)
		// close the target connection
		err = t.Close()
		if err != nil {
			panic(err)
		}
		log.Printf("target %q connectivity state: %q", t.Config.Name, t.ConnState())
		time.Sleep(100 * time.Millisecond)
	}
}

I believe this was solved, feel free to reopen if you see that this behavior is still there.


Add Chainspect to Showcase

👋 Hello there! Welcome. Please follow the steps below to tell us about your contribution.

Please complete a Checklist
Fill in all sections of the template
Click "Create pull request"

Checklist

[x] I have read the How to Contribute.
[x] I have read the Showcase Requirements
[x] I have run yarn build after adding my changes without getting any errors.

Showcase addition

Title: Chainspect
Description: Chainspect is an independent blockchain analytics platform focused on tracking technical metrics for the most widely used blockchains, including Cardano. We track Real-Time TPS, Max Recorded TPS, Max Theoretical TPS, Block Time, TTF, Governance Model, and Launch Date for 40+ chains, including Cardano.
Website: https://chainspect.app/
Source: N/A
Tags:

analytics

Hey @rphair,
Thank you for your feedback and for taking the time to explore our platform. I’d like to address your concerns:

I don't think this project illustrates anything about Cardano from either a development or a user perspective.

Chainspect is an analytics platform that does not emphasize any particular blockchain. We track over 40 blockchains, allowing users to compare them easily. By including Cardano, we offer valuable insights into how it performs against other major blockchains in metrics such as TPS, Block Time, TTF, etc. I believe this comparative analysis is beneficial for the Cardano community.

The home page refers to Ethereum, Bitcoin, Solana, Polygon, Avalanche, Algorand, and Fantom (with particular emphasis on Ethereum) without the word Cardano appearing even once, even in the images.

That's true! We mention Ethereum, Bitcoin, and Solana on our chain page as the biggest players in the field while using Ethereum as a neutral example. However, Cardano has a dedicated chain page on Chainspect (https://chainspect.app/chain/cardano) and multiple comparison pages (e.g., https://chainspect.app/compare/cardano-vs-ethereum, https://chainspect.app/compare/cardano-vs-bitcoin).
But I understand your concerns! To better showcase Cardano, I propose the following changes in src/data/showcases.js:

Change the website link from https://chainspect.app to https://chainspect.app/chain/cardano. It'll lead directly to the Cardano chain page.
Use a screenshot of the Cardano chain page as a preview image to focus on Cardano rather than other networks.

Additionally, we have recently added Cardano to the "Proudly Collaborating With" section on our landing page, ensuring it gets mentioned prominently.
What do you think of these changes? I look forward to your response! Thank you again for your time and consideration 🙏

@fromaline first note that my lack of approval doesn't disqualify your project from being included in the portfolio.  There are several other reviewers and only 3 affirmative responses would be necessary in order to confirm the proposed listing.
Since you've pressed for more detail I have to say how unhelpful I think these "blockchain comparative" sites are in general.  Metrics like TPS and TVL so far have mainly been used to promote industry media favourites, which pushes the crypto world onto a path towards centralisation and chauvinism.
I would happily support a link to the /cardano page if you had any way — apparently not possible with your model which includes only mainstream features — if it went into any detail about information density in transaction flow, smart contract efficiency, and features & design possibilities of Cardano's uncommon eUTxO model resulting from transaction determinism.
You are comparing Cardano to the world on terms that only serve its entrenched competitors... while the purpose of this site is to emphasise Cardano's distinctive features and teach the community of users and developers to use & contribute to that uniqueness.  If you could find a convincing way of doing that, according to the details above, then you would have my full support.

@rphair, thanks for sharing your thoughts — I understand your points and where they come from!
To be honest, we are getting similar requests for almost every blockchain we integrate into Chainspect 😅
But we are always forced to respectfully disagree and keep our metrics standardized. Otherwise, we end up with a bunch of erroneous data, which can't be understood by an average blockchain user.
In other words, there is always a tradeoff between standardization and creating custom, blockchain-specific metrics, which are not comparable to one another.
Hope it makes sense 🙏


FPS

Hi, what is the FPS you achieved on real time and you are directly using detection or face detection with classification?
Thanks for the info.

With the Image classification, I was able to achieve ~ 20 to 30 FPS.

With the Image classification, I was able to achieve ~ 20 to 30 FPS.

How were you able to do just classification without detection in the frame as it shows a bounding box around the face?

I've tried both face detection and classification, but it turns out that, classification gave more accurate results for this project.

I've tried both face detection and classification, but it turns out that, classification gave more accurate results for this project.

One thing out of curiosity I am asking you is that did you just do classification and draw the bounding boxes that are what you are trying to say, if so if you can please enclose the model to this mail id that would be of great help<EMAIL_ADDRESS>


Check authorization, record limit, and validate association arguments in to_any_through_sql_cross_table associations

Adhering to Cenzontle's standard. The field resolvers, and possibly so called "infix" functions, e.g. add_person, must do the following checks and adjustments:

check authorization
validate association arguments
diminish record limits

implemented and merged into branch
https://github.com/ScienceDb/graphql-server-model-codegen/tree/issue107-refactoring
commit:
https://github.com/ScienceDb/graphql-server-model-codegen/commit/928ac59685ff1c89d7eebb7f613ccb6aa8461f13


Issue with edu account

Hello
I have an office 365 account with my university. Im not allowed to generate an app password and for account authentication sharepoint takes me first to my university login website (message “Taking you to your organization's sign-in page”). I believe this is a limitation when trying to use the proxy to add the hyper backup task.
Are there any workarounds?
thanks
2021-01-04T01:07:56.817Z proxy:info Proxy server listening: { address: '::', family: 'IPv6', port: 3000 }
2021-01-04T01:18:25.281Z proxy:error [623422/7546] An error occurred during user authentication: { RequestError: Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443
    at new RequestError (/usr/src/app/node_modules/request-promise-core/lib/errors.js:14:15)
    at Request.plumbing.callback (/usr/src/app/node_modules/request-promise-core/lib/plumbing.js:87:29)
    at Request.RP$callback [as _callback] (/usr/src/app/node_modules/request-promise-core/lib/plumbing.js:46:31)
    at self.callback (/usr/src/app/node_modules/request/request.js:185:22)
    at Request.emit (events.js:198:13)
    at Request.onRequestError (/usr/src/app/node_modules/request/request.js:881:8)
    at ClientRequest.emit (events.js:198:13)
    at TLSSocket.socketErrorListener (_http_client.js:392:9)
    at TLSSocket.emit (events.js:198:13)
    at emitErrorNT (internal/streams/destroy.js:91:8)
    at emitErrorAndCloseNT (internal/streams/destroy.js:59:3)
    at process._tickCallback (internal/process/next_tick.js:63:19)
  name: 'RequestError',
  message:
   'Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443',
  cause:
   { Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443
       at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:56:26)
     errno: 'EAI_AGAIN',
     code: 'EAI_AGAIN',
     syscall: 'getaddrinfo',
     hostname: 'login.microsoftonline.com',
     host: 'login.microsoftonline.com',
     port: 443 },
  error:
   { Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443
       at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:56:26)
     errno: 'EAI_AGAIN',
     code: 'EAI_AGAIN',
     syscall: 'getaddrinfo',
     hostname: 'login.microsoftonline.com',
     host: 'login.microsoftonline.com',
     port: 443 },
  options:
   { simple: false,
     strictSSL: false,
     json: true,
     form: { login<EMAIL_ADDRESS>},
     uri: 'https://login.microsoftonline.com/GetUserRealm.srf',
     method: 'POST',
     callback: [Function: RP$callback],
     transform: undefined,
     resolveWithFullResponse: false,
     transform2xxOnly: false },
  response: undefined }
2021-01-04T01:18:30.303Z proxy:error [628494/9234] An error occurred during user authentication: { RequestError: Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443
    at new RequestError (/usr/src/app/node_modules/request-promise-core/lib/errors.js:14:15)
    at Request.plumbing.callback (/usr/src/app/node_modules/request-promise-core/lib/plumbing.js:87:29)
    at Request.RP$callback [as _callback] (/usr/src/app/node_modules/request-promise-core/lib/plumbing.js:46:31)
    at self.callback (/usr/src/app/node_modules/request/request.js:185:22)
    at Request.emit (events.js:198:13)
    at Request.onRequestError (/usr/src/app/node_modules/request/request.js:881:8)
    at ClientRequest.emit (events.js:198:13)
    at TLSSocket.socketErrorListener (_http_client.js:392:9)
    at TLSSocket.emit (events.js:198:13)
    at emitErrorNT (internal/streams/destroy.js:91:8)
    at emitErrorAndCloseNT (internal/streams/destroy.js:59:3)
    at process._tickCallback (internal/process/next_tick.js:63:19)
  name: 'RequestError',
  message:
   'Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443',
  cause:
   { Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443
       at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:56:26)
     errno: 'EAI_AGAIN',
     code: 'EAI_AGAIN',
     syscall: 'getaddrinfo',
     hostname: 'login.microsoftonline.com',
     host: 'login.microsoftonline.com',
     port: 443 },
  error:
   { Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443
       at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:56:26)
     errno: 'EAI_AGAIN',
     code: 'EAI_AGAIN',
     syscall: 'getaddrinfo',
     hostname: 'login.microsoftonline.com',
     host: 'login.microsoftonline.com',
     port: 443 },
  options:
   { simple: false,
     strictSSL: false,
     json: true,
     form: { login<EMAIL_ADDRESS>},
     uri: 'https://login.microsoftonline.com/GetUserRealm.srf',
     method: 'POST',
     callback: [Function: RP$callback],
     transform: undefined,
     resolveWithFullResponse: false,
     transform2xxOnly: false },
 response: undefined }

Have you tried to use a different AUTH_TYPE?

Have you tried to use a different AUTH_TYPE?

I read about it on your wiki but I’m unsure on which one to use. Can you provide additional info so I can read and give it a try?

I read about it on your wiki but I’m unsure on which one to use. Can you provide additional info so I can read and give it a try?

basic-to-sharepoint-auth-http-proxy is using node-sp-auth (v2.5.7) for sharepoint authentication. You could e.g. give form-based authentication (fba) a try: Set the AUTH_TYPE environment variable to fba and then use your university-OneDrive username/password for basic authentication.

basic-to-sharepoint-auth-http-proxy is using node-sp-auth (v2.5.7) for sharepoint authentication. You could e.g. give form-based authentication (fba) a try: Set the AUTH_TYPE environment variable to fba and then use your university-OneDrive username/password for basic authentication.

Thanks. Will try and report back

Thanks. Will try and report back

hello
i have tried today with the fba as auth_type and no luck so far :(. here is my log
2021-01-06T17:52:39.966Z proxy:error [165724/5894] An error occurred during user authentication: { RequestError: Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443
    at new RequestError (/usr/src/app/node_modules/request-promise-core/lib/errors.js:14:15)
    at Request.plumbing.callback (/usr/src/app/node_modules/request-promise-core/lib/plumbing.js:87:29)
    at Request.RP$callback [as _callback] (/usr/src/app/node_modules/request-promise-core/lib/plumbing.js:46:31)
    at self.callback (/usr/src/app/node_modules/request/request.js:185:22)
    at Request.emit (events.js:198:13)
    at Request.onRequestError (/usr/src/app/node_modules/request/request.js:881:8)
    at ClientRequest.emit (events.js:198:13)
    at TLSSocket.socketErrorListener (_http_client.js:392:9)
    at TLSSocket.emit (events.js:198:13)
    at emitErrorNT (internal/streams/destroy.js:91:8)
    at emitErrorAndCloseNT (internal/streams/destroy.js:59:3)
    at process._tickCallback (internal/process/next_tick.js:63:19)
  name: 'RequestError',
  message:
   'Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443',
  cause:
   { Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443
       at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:56:26)
     errno: 'EAI_AGAIN',
     code: 'EAI_AGAIN',
     syscall: 'getaddrinfo',
     hostname: 'login.microsoftonline.com',
     host: 'login.microsoftonline.com',
     port: 443 },
  error:
   { Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443
       at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:56:26)
     errno: 'EAI_AGAIN',
     code: 'EAI_AGAIN',
     syscall: 'getaddrinfo',
     hostname: 'login.microsoftonline.com',
     host: 'login.microsoftonline.com',
     port: 443 },
  options:
   { simple: false,
     strictSSL: false,
     json: true,
     form: { login<EMAIL_ADDRESS>},
     uri: 'https://login.microsoftonline.com/GetUserRealm.srf',
     method: 'POST',
     callback: [Function: RP$callback],
     transform: undefined,
     resolveWithFullResponse: false,
     transform2xxOnly: false },
  response: undefined }

2021-01-06T17:52:44.983Z proxy:error [170741/7810] An error occurred during user authentication: { RequestError: Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443
    at new RequestError (/usr/src/app/node_modules/request-promise-core/lib/errors.js:14:15)
    at Request.plumbing.callback (/usr/src/app/node_modules/request-promise-core/lib/plumbing.js:87:29)
    at Request.RP$callback [as _callback] (/usr/src/app/node_modules/request-promise-core/lib/plumbing.js:46:31)
    at self.callback (/usr/src/app/node_modules/request/request.js:185:22)
    at Request.emit (events.js:198:13)
    at Request.onRequestError (/usr/src/app/node_modules/request/request.js:881:8)
    at ClientRequest.emit (events.js:198:13)
    at TLSSocket.socketErrorListener (_http_client.js:392:9)
    at TLSSocket.emit (events.js:198:13)
    at emitErrorNT (internal/streams/destroy.js:91:8)
    at emitErrorAndCloseNT (internal/streams/destroy.js:59:3)
    at process._tickCallback (internal/process/next_tick.js:63:19)
  name: 'RequestError',
  message:
   'Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443',
  cause:
   { Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443
       at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:56:26)
     errno: 'EAI_AGAIN',
     code: 'EAI_AGAIN',
     syscall: 'getaddrinfo',
     hostname: 'login.microsoftonline.com',
     host: 'login.microsoftonline.com',
     port: 443 },
  error:
   { Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443
       at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:56:26)
     errno: 'EAI_AGAIN',
     code: 'EAI_AGAIN',
     syscall: 'getaddrinfo',
     hostname: 'login.microsoftonline.com',
     host: 'login.microsoftonline.com',
     port: 443 },
  options:
   { simple: false,
     strictSSL: false,
     json: true,
     form: { login<EMAIL_ADDRESS>},
     uri: 'https://login.microsoftonline.com/GetUserRealm.srf',
     method: 'POST',
     callback: [Function: RP$callback],
     transform: undefined,
     resolveWithFullResponse: false,
     transform2xxOnly: false },
  response: undefined }

hello
i have tried today with the fba as auth_type and no luck so far :(. here is my log
2021-01-06T17:52:39.966Z proxy:error [165724/5894] An error occurred during user authentication: { RequestError: Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443
    at new RequestError (/usr/src/app/node_modules/request-promise-core/lib/errors.js:14:15)
    at Request.plumbing.callback (/usr/src/app/node_modules/request-promise-core/lib/plumbing.js:87:29)
    at Request.RP$callback [as _callback] (/usr/src/app/node_modules/request-promise-core/lib/plumbing.js:46:31)
    at self.callback (/usr/src/app/node_modules/request/request.js:185:22)
    at Request.emit (events.js:198:13)
    at Request.onRequestError (/usr/src/app/node_modules/request/request.js:881:8)
    at ClientRequest.emit (events.js:198:13)
    at TLSSocket.socketErrorListener (_http_client.js:392:9)
    at TLSSocket.emit (events.js:198:13)
    at emitErrorNT (internal/streams/destroy.js:91:8)
    at emitErrorAndCloseNT (internal/streams/destroy.js:59:3)
    at process._tickCallback (internal/process/next_tick.js:63:19)
  name: 'RequestError',
  message:
   'Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443',
  cause:
   { Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443
       at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:56:26)
     errno: 'EAI_AGAIN',
     code: 'EAI_AGAIN',
     syscall: 'getaddrinfo',
     hostname: 'login.microsoftonline.com',
     host: 'login.microsoftonline.com',
     port: 443 },
  error:
   { Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443
       at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:56:26)
     errno: 'EAI_AGAIN',
     code: 'EAI_AGAIN',
     syscall: 'getaddrinfo',
     hostname: 'login.microsoftonline.com',
     host: 'login.microsoftonline.com',
     port: 443 },
  options:
   { simple: false,
     strictSSL: false,
     json: true,
     form: { login<EMAIL_ADDRESS>},
     uri: 'https://login.microsoftonline.com/GetUserRealm.srf',
     method: 'POST',
     callback: [Function: RP$callback],
     transform: undefined,
     resolveWithFullResponse: false,
     transform2xxOnly: false },
  response: undefined }

2021-01-06T17:52:44.983Z proxy:error [170741/7810] An error occurred during user authentication: { RequestError: Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443
    at new RequestError (/usr/src/app/node_modules/request-promise-core/lib/errors.js:14:15)
    at Request.plumbing.callback (/usr/src/app/node_modules/request-promise-core/lib/plumbing.js:87:29)
    at Request.RP$callback [as _callback] (/usr/src/app/node_modules/request-promise-core/lib/plumbing.js:46:31)
    at self.callback (/usr/src/app/node_modules/request/request.js:185:22)
    at Request.emit (events.js:198:13)
    at Request.onRequestError (/usr/src/app/node_modules/request/request.js:881:8)
    at ClientRequest.emit (events.js:198:13)
    at TLSSocket.socketErrorListener (_http_client.js:392:9)
    at TLSSocket.emit (events.js:198:13)
    at emitErrorNT (internal/streams/destroy.js:91:8)
    at emitErrorAndCloseNT (internal/streams/destroy.js:59:3)
    at process._tickCallback (internal/process/next_tick.js:63:19)
  name: 'RequestError',
  message:
   'Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443',
  cause:
   { Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443
       at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:56:26)
     errno: 'EAI_AGAIN',
     code: 'EAI_AGAIN',
     syscall: 'getaddrinfo',
     hostname: 'login.microsoftonline.com',
     host: 'login.microsoftonline.com',
     port: 443 },
  error:
   { Error: getaddrinfo EAI_AGAIN login.microsoftonline.com login.microsoftonline.com:443
       at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:56:26)
     errno: 'EAI_AGAIN',
     code: 'EAI_AGAIN',
     syscall: 'getaddrinfo',
     hostname: 'login.microsoftonline.com',
     host: 'login.microsoftonline.com',
     port: 443 },
  options:
   { simple: false,
     strictSSL: false,
     json: true,
     form: { login<EMAIL_ADDRESS>},
     uri: 'https://login.microsoftonline.com/GetUserRealm.srf',
     method: 'POST',
     callback: [Function: RP$callback],
     transform: undefined,
     resolveWithFullResponse: false,
     transform2xxOnly: false },
  response: undefined }

Looking at your logs again, they show an EAI_AGAIN-error. The host login.microsoftonline.com cannot be resolved, the DNS request times out. Maybe you have no/limited network connectivity on the system where you run the proxy? Does nslookup work? How do you run the procy?

Looking at your logs again, they show an EAI_AGAIN-error. The host login.microsoftonline.com cannot be resolved, the DNS request times out. Maybe you have no/limited network connectivity on the system where you run the proxy? Does nslookup work? How do you run the procy?

Thanks for the tip. I’m running the proxy on my synology ds218+ and docker. I installed the image through the GUI process.
my NAS has internet and dns connectivity. Could I be missing something under docker setup?

Thanks for the tip. I’m running the proxy on my synology ds218+ and docker. I installed the image through the GUI process.
my NAS has internet and dns connectivity. Could I be missing something under docker setup?

The last logs you posted show, that the underlying node-sp-auth library does not receive an expected XML-Response in the script file AdfsHelper.js. The code of node-sp-auth is transpiled, so the line numbers don't match - as far as I can tell, the log entry "at config_1.request.post.then.xmlResponse (/usr/src/app/node_modules/node-sp-auth/lib/src/utils/AdfsHelper.js:28:23)" refers to this line in the source: https://github.com/s-KaiNet/node-sp-auth/blob/4afc20ae845c6d451185cc9f2660ddca0fc3cdf3/src/utils/AdfsHelper.ts#L34
It is possible that for your university OneDrive for Business/Sharepoint was configured in a way that does not allow any of the authentication methods supported by node-sp-auth. Also see https://github.com/s-KaiNet/node-sp-auth/issues/90 for a possible reason.
To further investigate the issue, I would suggest to install Node.js locally and try to authenticate with a minimal script using the latest version of node-sp-auth. Unfortunately I don't have the time to investigate this issue further myself.

Encountered same problem but later found that it's pi-hole blocking the connection somehow.
Set the DNS manually to <IP_ADDRESS> on DiskStation, restart container and everything run smoothly afterwards.


LMDB: introduce a typealias for mode_t

mode_t is a non-portable extension to C (POSIX).  Introduce a new typealias ModeType that represents the underlying type that is used. Provide an alternate definition for the default mode on Windows as it does not have non-user permissions on a file but rather relies on proper ACLing.

@swift-ci please test

CC: @franklinsch


Aggregate CNP/ANP access to admin/edit and view ClusterRoles

Aggregate the CRUD permissions to "admin" and "edit" ClusterRoles so that any user granted these roles may have edit permissions. In addition to this, aggregate the READ-ONLY permissions to "view" ClusterRole, so that any user granted this role may be able to view the CNP/ANP resources.

Codecov Report

Merging #1206 into master will decrease coverage by 14.47%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #1206       +/-   ##
===========================================
- Coverage   56.12%   41.65%   -14.48%
===========================================
  Files         105       74       -31
  Lines       11550    10509     -1041
===========================================
- Hits         6483     4377     -2106
- Misses       4495     5778     +1283
+ Partials      572      354      -218

Flag
Coverage Δ

#integration-tests
?

#unit-tests
41.65% <ø> (+0.06%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

 Is this for another ServiceAccounts? I found ServiceAccount antrea-controller and antrea-agent are not bound with ClusterRole view/admin/edit.

/test-all

/test-networkpolicy /test-e2e

LGTM, guess we should do this to Traceflow CRD too? @gran-vmv @lzhecheng

Sure. Is this for another ServiceAccounts? I found ServiceAccount antrea-controller and antrea-agent are not bound with ClusterRole view/admin/edit.

Please check the PR description and related K8s docs for its purpose.

/test-e2e /test-networkpolicy

LGTM, guess we should do this to Traceflow CRD too? @gran-vmv @lzhecheng

Sure. Is this for another ServiceAccounts? I found ServiceAccount antrea-controller and antrea-agent are not bound with ClusterRole view/admin/edit.

Please check the PR description and related K8s docs for its purpose.

here is the doc i referred @gran-vmv  https://kubernetes.io/docs/reference/access-authn-authz/rbac/#aggregated-clusterroles

/test-conformance


Scan barcode from image

Enable scanning a barcode from an image selected from the photoroll
Use case:

a user (like @tacitefood) has taken many product photos (usually photo of barcode, then several photos)
he should be able to select a photo with a barcode
the app should detect the barcode in the image, and switch to the classic add new product view (or to the product if it exists) to let him do further work (like add the other images)

@teolemon  i am working in this .

@teolemon I've implemented this as a part of feature https://github.com/openfoodfacts/openfoodfacts-androidapp/issues/868.
However i'm blocked on creating a PR by a missing file on master. Will have the PR once i'm unblocked.


None check

Coverage increased (+0.02%) to 79.66% when pulling f700a31101c9926ae78d9d62dc04a0d499c0c350 on none-check into d36e11fc023ab3dbb22887f81621b9111b343748 on master.

Coverage decreased (-0.34%) to 79.29% when pulling bc5ae92cfe6c3c0c310832a4d5bb3467f5e66133 on none-check into d36e11fc023ab3dbb22887f81621b9111b343748 on master.


Segmentation probability maps depend on CPU or GPU used for inference

System information.

Have I written custom code (as opposed to using a stock example script provided in Keras): colab
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
TensorFlow installed from (source or binary): Binary
TensorFlow version (use command below): TF2.5
Python version: 3.8
GPU model and memory: Nvidia 3080

Describe the problem.
I wrote a model for semantic segmentation in tensorflow. The model were trained in a CPU and the predictions were also done in a CPU. After some time I run the predictions again in a GPU machine and I noticed that there were some very small differences in the probability maps (from the 4th-5th decimal value). In addition, I wanted to check if the batch size affected the prediction distribution (I initially thought that there were no influence).
I ran a small experiment describing these four situations:
A1) Model Trained in CPU and Predictions run in CPU --> Batch Size = 1
A2) Model Trained in CPU and Prediction run in CPU --> Batch Size = 64
B1) Model Trained in CPU and Predictions run in GPU --> Batch Size = 1
B2) Model Trained in CPU and Prediction run in GPU --> Batch Size = 64
I observed that :

Training the model and running the predictions in a CPU provides the same results regardless the batch size.

Batch size has an effect (although small) when training the model in a CPU and running the predictions in a GPU

Predictions provided by a CPU and a GPU don't match (very small differences). This can be relevant if the cutoff threshold is placed at 0.55, the CPU will classify a given pixel as a background pixel (0.549857<0.55) , whereas the GPU will classify it as a foreground pixel (0.550076>0.55) if the image was part of a larger batch.
Running the GPU predictions again, provide different probabilities (very small differences again), whereas CPU provides always the same probabilities.

Here is the link to the colab to reproduce the feature. I inluded the model I used for that (the enitre model was saved) and some images that I have used.
So my main questions are:

Is this behaviour something normal that is expected to happen?
Why this behavior happens? Is this related to this?
How can I avoid this to happen? So that the Predictions run in a GPU always provide the same probabilities as the CPU) and regardless of the batch size?

Describe the current behavior.
Differences in the probability maps based on when the inferences is done with a CPU or a GPU.
Some Results:
Probability maps when running the inference in GPU (there is a difference between using a batch size of 1 and using >1)
Batch Size=1
[[0.05668435 0.10867791 0.04235671 ... 0.22491932 0.12363011 0.17167044]
[0.04555672 0.03822105 0.03054511 ... 0.42176178 0.11014031 0.0701196 ]
[0.01282059 0.01109215 0.03718    ... 0.38060975 0.16944014 0.09439405]
...
[0.0309867  0.01612546 0.03472591 ... 0.04758538 0.0287854  0.02922609]
[0.09070282 0.01980974 0.04191865 ... 0.08519646 0.02937398 0.02979605]
[0.08077859 0.02201488 0.04651429 ... 0.07312316 0.03979398 0.08936194]]
Batch Size>1
[[0.05668447 0.10867798 0.04235688 ... 0.22491935 0.12363014 0.17167065]
[0.04555689 0.03822101 0.03054521 ... 0.42176208 0.11014052 0.07011983]
[0.01282063 0.01109215 0.03718007 ... 0.3806101  0.1694407  0.09439442]
...
[0.03098667 0.01612546 0.03472591 ... 0.04758547 0.02878542 0.02922612]
[0.09070284 0.01980972 0.04191868 ... 0.08519673 0.02937401 0.02979604]
[0.08077855 0.02201487 0.04651425 ... 0.07312333 0.03979404 0.08936189]]
Probability maps when running the inference in CPU (there is NO difference between using a batch size of 1 and using >1)
Batch Size=1
[[0.05668434 0.10867806 0.04235679 ... 0.22491916 0.12363013 0.1716703 ]
[0.04555671 0.03822112 0.03054515 ... 0.42176208 0.11014024 0.07011949]
[0.01282062 0.01109214 0.03717988 ... 0.3806093  0.1694399  0.09439388]
...
[0.03098666 0.01612545 0.03472593 ... 0.04758537 0.02878539 0.02922607]
[0.09070285 0.01980974 0.04191878 ... 0.08519644 0.02937397 0.02979606]
[0.08077864 0.02201489 0.04651435 ... 0.07312308 0.039794   0.08936188]]
Batch Size>1
[[0.05668434 0.10867806 0.04235679 ... 0.22491916 0.12363013 0.1716703 ]
[0.04555671 0.03822112 0.03054515 ... 0.42176208 0.11014024 0.07011949]
[0.01282062 0.01109214 0.03717988 ... 0.3806093  0.1694399  0.09439388]
...
[0.03098666 0.01612545 0.03472593 ... 0.04758537 0.02878539 0.02922607]
[0.09070285 0.01980974 0.04191878 ... 0.08519644 0.02937397 0.02979606]
[0.08077864 0.02201489 0.04651435 ... 0.07312308 0.039794   0.08936188]]

Unfortunately GPU computations are often nondeterministic and susceptible to small numerical differences, e.g. https://github.com/tensorflow/tensorflow/issues/2732
I believe @reedwm is exploring fully-deterministic computation on GPUs for TensorFlow, though I'm not sure what the timeline for that looks like.

The plan is that there will be a flag to make all GPU ops deterministic in TensorFlow 2.7. This means if you run the exact same op on the GPU multiple times, you will get the exact same results. However, even with the flag, running on the CPU will likely produce slightly different results than running on the GPU. Making CPU and GPU have identical numeric behavior is very difficult, so it is unlikely we would ever make running a model on the CPU give the exact same results as running on the GPU.
Changing the training batch size may significantly affect training results, which would be true even with infinite precision, so it is expected changing it may affect the results.
In TensorFlow 2.5 and 2.6, setting the enviornmental variable TF_DETERMINISTIC_OPS=1 will make many GPU ops deterministic, so you can try that, but some GPU ops will still be nondeterministic.

Hi both,
Thank you very much for your explanations.
I'm actually relieved that this difference is something that is expected to happen when working with GPUs.
I will try to set the environmental variable to 1 in the future.
I think my question has been successfully answered. Therefore, I close the issue.
Pedro


Error in Downloading Signing Tools when running Sign-BcContainerApp

PLEASE DO NOT INCLUDE ANY PASSWORDS OR TOKENS IN YOUR ISSUE!!!
Describe the issue
I want to sign an app, I'm running docker desktop on Windows 11. The PC is fully updated
Scripts used to create container and cause the issue
Sign-BcContainerApp -containerName bcserver -appFile "C:\ProgramData\BcContainerHelper\Extensions\bcserver\my\9altitudes_9A Document Management_<IP_ADDRESS>.app" -pfxFile "C:\Users\flemm\AL\9altitudes.pfx" -pfxPassword $pass

Full output of scripts
Copying certificate file to container
Downloading Signing Tools
Exception calling "DownloadFile" with "2" argument(s): "The operation has timed out."

Container Free Physical Memory: 11.6Gb

Services in container bcserver:
- MicrosoftDynamicsNavServer$BC is Running
- MSSQL$SQLEXPRESS is Running

Sign-BcContainerApp Telemetry Correlation Id: 27d2304b-2152-4755-89f5-526c9b716554
Exception calling "DownloadFile" with "2" argument(s): "The operation has timed out."
At C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\5.0.5\ContainerHandling\Invoke-ScriptInNavContainer.ps1
:285 char:25
+                         throw $errorMessage
+                         ~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (Exception calli...has timed out.":String) [], RuntimeException
    + FullyQualifiedErrorId : Exception calling "DownloadFile" with "2" argument(s): "The operation has timed out."

Additional context

It does happen all the time, and I have the issue on several diffent BC versions, BC14, BC, 18, BC22
Yes it used to work.
It might be related to some First run of IE, which i can't do on a headless docker container.

Sorry for the late response.
Did you manage to make this work?
I do have some workflows using Sign-BcContainerApp that still runs flawlessly - and they all run use containers with this, so I expect this to be a problem on your machine.
Maybe the container doesn't have access to DNS - the output from the container creation would tell me that.
In that case, you could try adding -dns "hostDNS" to your new-bccontainer command.


persist version (failover version) as part of timer / transfer task s…

…o as to deal with case when events are overwritten.

Coverage increased (+0.2%) to 65.572% when pulling 2a613b2befb36ec4c19028d3e70a2ec250a3a706 on ms-version into 1aecfbce9956b88816b6d3d68db37e2de242f849 on master.


Invisible arrow

Hello,
When im trying use ur's css code in external file but my arrows makes "display:none;" .
I did try almost every example from here with same result.
I want to implement thumbnail-navigator-with-arrows.  When i take arrows from thumbnail div to  general div they are visible but js doesnt work.

Hello again,
Im sorry for taking ur time, i was working on little count of thumbs so arrows were dissable.


Connector builds are failing in windows

Description:

$subject

Following are the findings related to this issue:

When running the build, the docker command fails silently and due to that FileNotFoundException is thrown when the plugin tries to extract the package bala
The docker command failure was due to having invalid environment variables in the docker.env file

Following improvements should be done to the plugin to fix this:

Log the docker command failure as an error in the console
Exclude the not-required environment variables from the docker.env file


bossbars dont show to players joining

i have the plugin installed on velocity, and i set a bossbar for a specific backend. But if i leave and rejoin, the bossbar goes away

This plugin is old and no fixes will be made to the already released version (besides it is impossible to compile).
I am not sure when the next version will be ready


Add support for french language, needs discussion.

Reference to #522
This pull request adds support for the french language.
I need your help on this issue though as I'm unsure about what some settings do : #1130

I hope you now understand what we're delivering with these settings. Do not hesitate to ask if you have additional questions :)

Looks good to me. Waiting for another review...

If you want to, you can add French here. https://github.com/vscode-icons/vscode-icons/blob/4d382f12dca035ae3ebd8f1bb7468fa002a63be6/README.md#contributing-with-translations

@MonsieurMan Awesome! Just hold the PR a little longer as we may need to do some modifications to it based on your suggestion.

Holding this until #1130 is closed.


Distributions/ledger account clarity

There are no details provided about the ledger account and we faced issues when trying to use this functionality.  Looking back at the documentation for AX 2012, this is called dimensions and there is a bit more information about how "If the file you import has financial dimensions, you need to include all of the financial dimensions with the proper naming, otherwise an error message will appear stating that the ledger dimension is invalid. You’ll then need to either correct the financial dimensions or remove the columns from the file.".  This pointed us to a solution whereby we have columns for each dimension in the file, with the correct naming.  I feel that having a little more information on this page for Ledger Account would help people to avoid this confusion.

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

ID: 53131fa0-be2f-1d0a-1e2f-653fc703f5fc
Version Independent ID: ae64bc34-046a-820e-54bd-62e2a70cc4f4
Content: Plan for one-time vendors in the public sector
Content Source: articles/finance/public-sector/plan-one-time-vendors-public-sector.md
Service: dynamics-ax-applications
Product: ****
Technology: ****
GitHub Login: @ShylaThompson
Microsoft Alias: brpotter

@PeterJarret, thanks for providing this feedback!
@roschlom, please review.

Bug 422667 has been logged to track this issue.

The topic has been updated in response to the feedback. Thanks again for the input.


Added references to extension INF

AddComponent should only be used in Extension INFs. Adjusted the documentation to reflect that.

@NabilFates,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


About filter expressions in the [analysis] field

Hi everyone,
There is a problem with the filter expressions that can be set into the [analysis] section. I am trying to make a more precise filtering of the data adding &&(ANGSEP(RA_src,DEC_src,RA_SCZ,DEC_SCZ) <50 ) to the filter, and it fails verification. I could not find in the code where is the verification part.
I tried this same expression with the normal Fermi Science tools and it works.
Thanks.
Pere.

Hi,
I also try and it is not working.I am not sure that ANGSEP is defined or recognized. a workaround can be to use gtselect with a ROI of 50degrees
cheers
david

Within the Sciente Tools ANGSEP is included. What I think is that within Enrico it is not recognized.
The solution that you suggest is not what I want.
I need to select time intervals in which my source of interest is at an angular distance from the center of the Fermi FOV lower than 50 deg. In the ROI selection you choose how wide your FOV will be, but there might be events recorded large off-axis angles, which are not interesting for what I want to do.
Thanks anyway for your reply.
Cheers,
Pere.

This seems to be a bug in the config validator. The configspec only says:
	filter = string(default='(DATA_QUAL>0)&&(LAT_CONFIG==1)')

so any string should be valid. When tested manually with the is_string function, the complete string validates fine:
>>> from enrico.extern.validate import Validator, is_string
>>> value = 'DATA_QUAL==1&&LAT_CONFIG==1&&ABS(ROCK_ANGLE)<52&&(ANGSEP(RA_src,DEC_src,RA_SCZ,DEC_SCZ)<50)'
>>> is_string(value) == value
True

However, the full config validation with the above value set as [analysis]/[filter] fails:
>>> from enrico.extern.configobj import ConfigObj, flatten_errors
>>> from enrico.extern.validate import Validator
>>> config = ConfigObj('test.conf',configspec='default.conf')
>>> validator=Validator()
>>> results = config.validate(validator, copy=True)
>>> results['analysis']['filter']
False

I'll try to understand what the config validator is doing in addition to calling is_string to see why it fails.

The above only happens on the SciTools Python distribution, not for the other ones on my system...

OK, found the reason it is failing. The ConfigObj reader parses that string as a list of strings, not a string:
In [9]: config['analysis']['filter']
Out[9]:
['DATA_QUAL==1&&LAT_CONFIG==1&&ABS(ROCK_ANGLE)<52&&(ANGSEP(RA_src',
 'DEC_src',
 'RA_SCZ',
 'DEC_SCZ)<50)']

The way to avoid the item being read as a list of strings rather than a string is to put quotes (single or double) around the string. Therefore, the config entry:
filter = DATA_QUAL==1&&LAT_CONFIG==1&&ABS(ROCK_ANGLE)<52&&ANGSEP(RA_src,DEC_src,RA_SCZ,DEC_SCZ)<50)

should be changed to:
filter = "DATA_QUAL==1&&LAT_CONFIG==1&&ABS(ROCK_ANGLE)<52&&ANGSEP(RA_src,DEC_src,RA_SCZ,DEC_SCZ)<50)"

@pmunar -- can you confirm that this works?

Yes, it works!
Thanks @zblz  !


chore(kinds): merge into mono repo

What this PR does / why we need it:

Merges kinds repo into ec
https://github.com/replicatedhq/airgap/pull/189
https://github.com/replicatedhq/kots/pull/4854
Which issue(s) this PR fixes:

Does this PR require a test?

Does this PR require a release note?

Does this PR require documentation?

If we merge embedded-cluster-kinds into EC, will kots be able to import things properly still?

If we merge embedded-cluster-kinds into EC, will kots be able to import things properly still?

@laverya It seems to work
https://github.com/replicatedhq/kots/pull/4854


Enable sample loader info logging

We want to enable the sample loader info logs to see if this constant stream of log lines stops the concourse container connection from dying (or at least give us some insight into when/how it is dying).
https://trello.com/c/xNdUzmdK/801-why-does-concourse-stop-outputting-performance-test-logs-during-30-million-test-1-day

 Why does concourse stop outputting performance test logs during 30 million test (1 day)


Problem with creating image

Hello,
I'm having trouble creating a blank canvas with a background colour:
jimp = new Jimp(144, 144, 0xffdddddd, function (error, canvas) {
   console.log(error, canvas);
});

Problem is, I get this error:
/Users/haydenbleasel/Projects/favicons/node_modules/jimp/index.js:49
    else throw error;
         ^

Error: cb must be a function
    at Jimp.throwError (/Users/haydenbleasel/Projects/favicons/node_modules/jimp/index.js:47:43)
    at new Jimp (/Users/haydenbleasel/Projects/favicons/node_modules/jimp/index.js:91:31)
    at createImage (/Users/haydenbleasel/Projects/favicons/index.js:42:24)
    at /Users/haydenbleasel/Projects/favicons/index.js:61:21
    at Object.async.forEachOf.async.eachOf (/Users/haydenbleasel/Projects/favicons/node_modules/async/lib/async.js:233:13)
    at generateFavicons (/Users/haydenbleasel/Projects/favicons/index.js:55:19)
    at /Users/haydenbleasel/Projects/favicons/index.js:79:17
    at Object.async.forEachOf.async.eachOf (/Users/haydenbleasel/Projects/favicons/node_modules/async/lib/async.js:233:13)
    at module.exports (/Users/haydenbleasel/Projects/favicons/index.js:76:15)
    at Object.<anonymous> (/Users/haydenbleasel/Projects/favicons/test.js:3:1)

This is also probably because my colour is a string rather than a raw number-hex i.e.
"0xffdddddd" => 0xffdddddd

Nevermind sorry, this is the result of passing in width and height parameters that are not integers.


fix megatron_patch_path

PR type

[x] Bug Fix

PR information
When the environment variable PAI_MEGATRON_PATCH_PATH is set, the following error occurs:
UnboundLocalError: local variable 'megatron_patch_path' referenced before assignment.

This error occurs because the variable megatron_patch_path is not declared in this context.
The change made was to assign the value of the environment variable PAI_MEGATRON_PATCH_PATH to megatron_patch_path.
Experiment results
Paste your experiment result here(if needed).

thanks


Move to SharpCompress

https://github.com/adamhathcock/sharpcompress

Done with #19
Thanks @DanTheMan827 !


updates?

Is this program dead? I sent an email a while ago with a feature request and never got an answer. If you don't plan on working on the project anymore could you release the source here?

The project page and beta release have been posted now.
So the project is alive... and maybe kicking? :-)


Support a-z and 0-9 in variable names

Signed-off-by: Darren Shepherd<EMAIL_ADDRESS>

@dnephin Can you confirm that the list of valid character is correct?  It is now a-z, A-Z, _ and 0-9.

r'[_a-z][_a-z0-9]*' is the pattern (used with _re.IGNORECASE), so I believe that is correct

@dnephin Updated to not allow starting with a number

Is this for env-vars? You may be interested in https://github.com/docker/docker/pull/16608 (and the linked issue)

@thaJeztah nope, not for env-vars, this is for variables in the compose file (it just happens we use very similar syntax).

LGTM


Setting collapsed: true in data breaks stateful tree-menu

I am setting collapsed true in the initial data array. When I render the component I am unable to expand the node. Without setting collapsed true I can expand and collapse as expected.
data= [
  {label: 'node1', collapsed: true, children: [
    {label: 'node2'},
    {label: 'node3'},
  ]}
]

<div>
  <TreeMenu
    stateful={true}
    onTreeNodeClick={onTreeNodeClick}
    onTreeNodeCollapseChange={onTreeNodeCollapseChange}
    onTreeNodeCheckChange={onTreeNodeCheckChange}
    expandIconClass="fa fa-chevron-right"
    collapseIconClass="fa fa-chevron-down"
    data={data} />
</div>

I suspect the stateful collapsed variable is being overwritten by the props collapsed variable in tree_node but couldn't quite figure it out from glancing at the project code.
(Sorry I was signed in as the wrong github user when I first posted this issue!)

@adailey14 this should be fixed in most recent version - please let me know if that doesn't work for you.


CPU Support

I'd like to run the inference pass on Mac OS(Mojave), a system that runs AMD/Radeon, rather than NVIDIA/Cuda.
Would there be a quick fix to run the inference pass without GPU acceleration but on CPU resources instead?
Being rather new to pytorch, I'm starting at the stacktrace andlooking at my first error in models/pix2pix.py in the line that declares a cuda model.
new_model = torch.nn.parallel.DataParallel( new_model.cuda(), device_ids=range(torch.cuda.device_count()))

I can try to make the appropriate changes , but might need help figuring what to replace this with.

Unfortunately I don't have a lot of guidance to give here as I haven't experimented with a CPU version. It might be as simple as removing the cuda() calls, but I suspect pytorch has some hidden way to bite you there.

I ran the davis test script. Every time I would run into a cuda is not enabled error, I would notice that the code has .cuda() appended to it, like you mentioned.
I removed all .cuda() as I ran into them one by one on the stack trace. The script finally started working after I reached the last one.  I only had to edit models/pix2pix.py and models/networks.py.
Behold:

Great, glad it works!


TrimBot - Fix the version for dependent libraries

Describe the bug
JSON Schema updated a major version causing havoc.

resolved


Show the connected peers at the top of the Add Channel list by default

These we have an active connection with so are most likely to result in successful channel opening

This should be ideally be re-worked to not block on the peer lookup on startup, because it is not a strictly required addition of information. That said, if LND is available, the peer list should be available, and the data access improvement applies to the other lookups that aren't strictly necessary for initial display.

needs a rebase

I'm rebased and updated this PR in https://github.com/LN-Zap/zap-desktop/pull/1076. Closing as a duplicate


Added feedback to dnd, fixed drop area bug and refactored style

File picker does not accept all file types, e.g. *.md, *.rdf, *.owl and *.klc didn't work. All of them have mime types of text/plain or text/xml.

It seems this problem is Windows-specific. On other systems the OS gives the browser enough information to provide a mime-type, on Windows this is not the case, since it is very file-extension-dependent.
Unfortunately, at the time drop compatibility is checked the extensions of the files are not accessible.
I would suggest we change the accepted mime type field for general files from "*/*" to "" and leave it at that.


Option to exclude files

I couldn't find documentation or code related to excluding certain directories. Looks like it is missing.
This will be useful in cases where there are generated dart files, eg. In case the project uses google's protocol buffers, there will be dart files generated by the compiler, protoc.

@vyshnavsdeepak Could you try 0.2.0-dev.0 with glob pattern to exclude the files following the glob syntax

Closed, if any problem exists with the glob pattern, please create a new issue.

@hyiso I also can't exactly find where we should put these globs? Also, these globs only seem to match, not exclude?

@bawahakim See the glob syntax

If it starts with ^ or !, the construction will instead match all characters
not mentioned. For example, test[^a-z].dart will match test1.dart but not
testa.dart.

How can we apply a set of "include" and "not include"? E.g. I want to include all files in lib/**/*.dart but not in lib/**/__generated__/*.dart? Tried various things like
lint_staged:
  'lib/**/*.dart': dart format --fix && dart fix --apply && dart analyze --fatal-infos
  '!lib/**/__generated__/**/*.dart':

But the generated files still get linted

Maybe you need to split the configuration into multi glob, as the package glob does not support a simple configuration to apply both include and exclude
just like
lint_staged:
  'lib/{foo, bar,baz}/**.dart': your script
  'lib/xxx/{dirs exclude __generated__}/**/*.dart': your script


Switch to a SLE12 admin node for cloud6+

regrdless of my comment above
+1

Merging since otherwise mkcloud is not getting green