Added "az acr" commands

Commands include create/delete/show/list/update Azure container registries.
Added SDK for Python to manage Azure container registries.
Integrated repository catalog and tags commands.

Hi @djyou, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (doyou). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

/cc @SajayAntony @sivagms

/cc @mayurid

@djyou : Do you guys have a Swagger spec and generated SDKS?

Yes, we do have a swagger spec and the generated SDK is under /src/command_modules/azure-cli-acr/azure/cli/command_modules/acr/containerregistry/, which will eventually go to SDK for Python repo, but for now we haven't separated it from the CLI.

LGTM :shipit:

/cc @DavidObando

/cc @JasonRShaver

@tjprescott I believe we have addressed these issues as much as we can. Thanks for your feedback!

@tjprescott if there is anything blocking this from merging, please let us know.


Default doc links to Spark 3/Scala 2.12

Coverage remained the same at 0.0% when pulling 81c81abd1aed0ae9349a23ec834424f6c89a1093 on heuermh:doc-spark3 into 375a7af9bd918d2f5d8abce954ee56ebf1207f15 on bigdatagenomics:master.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/cannoli-prb/243/
Test PASSed.


CMake config.h

With cmake, the headers are put in prefix/include directory. Aside polution from all openblas files, there is config.h file at prefix/include/config.h. Which can trigger major issues with other compilations

Indeed the brute force GLOB_RECURSE in the toplevel CMakeLists.txt does look seriously wrong - only a very small number of headers should actually get installed, with the config.h only used as input for building openblas_config.h

This is what I have come up with so far - the unresolved part is adding the contents of the openblas_config_template.h and the final "endif" to openblas_config.h . (Redirection and concatenation does not appear to work with cmake like it does with plain Makefiles, and my awk is a bit rusty)
CMakeLists.txt

Not sure what to make of the error message from the Appveyor build on Windows -  'NF' is not recognized as an internal or external command,  operable program or batch file. Or is it simply not finding the awk program itself ?

Meh. Need help finding the right awk idiom for whatever Windows flavour of awk is used in the Appveyor environment.

Seems to work now after replacing the lengthy awk command for building openblas_config.h with a quick-and-dirty utility written in C. I simply could not come up with the right combination of quotes and escape characters to make the same awk call work on both Linux and Windows. Now I am sure there must be lots of ways to do this better - for one I do not get why I need to query LOCATION of the newly built tool to get cmake to realize it needs to use the .exe name on Windows.


bug: REGISTER PDF FROM URL fails

For this URL https://link.springer.com/content/pdf/10.1007/3-540-45937-5_6.pdf, REGISTER PDF FROM URL results in an error with the message Failed to extract text from /[...]/[jendeley download<PHONE_NUMBER>090].pdf.

Reproduced. I am trying to fix this now.

Current workaround

Save the pdf with filename of A Graph—Free Approach to Data—Flow Analysis [jendeley doi 10_1007_3-540-45937-5_6].pdf.
Register using UPLOAD PDF

I found the cause is redirect request (303) of HTTP and fixed.
> wget https://link.springer.com/content/pdf/10.1007/3-540-45937-5_6.pdf
--2023-01-25 21:12:55--  https://link.springer.com/content/pdf/10.1007/3-540-45937-5_6.pdf
Resolving link.springer.com (link.springer.com)... <IP_ADDRESS>
Connecting to link.springer.com (link.springer.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 303 See Other
Location: https://idp.springer.com/authorize?response_type=cookie&client_id=springerlink&redirect_uri=https%3A%2F%2Flink.springer.com%2Fcontent%2Fpdf%2F10.1007%2F3-540-45937-5_6.pdf [following]
--2023-01-25 21:12:56--  https://idp.springer.com/authorize?response_type=cookie&client_id=springerlink&redirect_uri=https%3A%2F%2Flink.springer.com%2Fcontent%2Fpdf%2F10.1007%2F3-540-45937-5_6.pdf
Resolving idp.springer.com (idp.springer.com)... <IP_ADDRESS>
Connecting to idp.springer.com (idp.springer.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 302 Found
Location: https://link.springer.com/content/pdf/10.1007/3-540-45937-5_6.pdf [following]
--2023-01-25 21:12:56--  https://link.springer.com/content/pdf/10.1007/3-540-45937-5_6.pdf
Connecting to link.springer.com (link.springer.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 470603 (460K) [application/pdf]
Saving to: ‘3-540-45937-5_6.pdf’

3-540-45937-5_6.pdf       100%[==================================>] 459.57K  --.-KB/s    in 0.05s

2023-01-25 21:12:56 (9.57 MB/s) - ‘3-540-45937-5_6.pdf’ saved [470603/470603]

Fixed in<EMAIL_ADDRESS>


chore: change in solc option foundry.toml

Turns out the error was on ffi / cast side.
It seems there was change on how to cast input.
Closing this PR and opening a new one


Mute / Unmute Incoming Audio

Purpose

Added new speaker icon next to the mute icon to demonstrate the mute/unmute incoming audio feature.
Does this introduce a breaking change?

[ ] Yes
[ X] No

Pull Request Type
What kind of change does this Pull Request introduce?

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

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

Join call.
When click on the mute speaker icon then the incoming audio should be muted (not call audio will be played).
When click on the unmute speaker icon then the incoming audio should start playing.

What to Check
Verify that the following are valid

...

Other Information

Mute incoming audio icon:

Unmute incoming audio icon:


"kubectl get" summary tab not working for some types

To reproduce,
Install Istio; install Istio Bookinfo sample
kubectl get gw,vs,dr,se,policy,sidecar
click on one of the returned items.
There will be a 'Kind' and 'Annotation' but nothing like the Spec.  Nothing like the kubectl describe output.
(Possibly related to https://github.com/istio/istio/issues/10872 )?
This may not be a problem with Istio CRDs, I see the same thing for kubectl get crd certificates.certmanager.k8s.io

we will be updating the summary tab in a way that does not try to replicate describe. PR in flight


Logged out Okta session page missing

If an Okta session is logged out during the session, in version v0.5.1 I saw this page which prompts for logout:

From version v1.0.0 I see some JSON instead:
{"action":"Something is wrong with your OIDC session. Please try to logout and login again","description":"The client specified not to prompt, but the user is not logged in.","error":"OIDC Error: login_required"}

Many thanks

Hi @CeriZego
thanks for reaching out and I'm sorry you are having issues with GPM. Just to better understand, is the session being closed from Okta and it is not after performing a logout from GPM, right?

Hi @ralgozino
The session is being logged out in Okta.  The when the app is used after that the error JSON is seen.
Just doing some testing after upgrading GPM.  Thanks for looking at it
Ceri

thanks for the report, we'll try to reproduce it and let you know :)

I was able to reproduce the error also with Keycloak as IDP. Ending the session IDP's side is not handled gracefully by GPM.
Basically we need to do this:
https://github.com/zamzterz/Flask-pyoidc/issues/71
Thanks again for the report @CeriZego


File titles aren't being correctly restored during HotExit

Azure Data Studio Version: master

Steps to Reproduce:

Open an untitled SQL query editor and modify the content
Close and reopen ADS

Expected: Documents are restored with the same title they had prior to closing ADS
Actual: Documents have a title based on the files content
The document below should say something like SQLQuery_1

@smartguest this seems similar to #9012 but is impacting SQL query documents.  Could you please verify this isn't related to the Edit Data changes?  And in either case, could you please help root cause this issue?  Thanks!

This doesn't appear to be related to any of the code I did, it does not use the newSqlEditor at all, and I only changed it so that the EditData Editor is affected, not this one.

Upon closer inspection, it turns out when you enter ANYTHING into the editor, it updates the title for the header, this must mean we must make the editor so it doesn't alter the header.

This appears to be fixed in Feb RC build.  I'll move to March milestone since we fixed directly in release/1.15.

When we fix this make sure we're adding tests for the title of untitled SQL editors. The tests we disabled in 1.15 should be a good place to start for what those would look like (https://github.com/microsoft/azuredatastudio/blob/master/src/vs/workbench/test/common/editor/untitledTextEditor.test.ts#L306)

FYI - this was indeed an intentional change on VS Codes part : https://code.visualstudio.com/updates/v1_42#_untitled-editor-improvements

Fixed


Update composer.json

the discuss url is for the events calendar, seems you copied it over from your other extension ;)
This PR is to fix the accurateness of extiverse.

Thanks @luceos !


CASM Monte Carlo Calculation

Lattice (Sample Attribute Type):

Lattice Matrix:

dim: [3,3]
type: float
desc: "Lattice vectors, represented as columns of a matrix"

Parameters:

type: float
dim: [6]
desc: "Lattice parameters, as [a, b, c, alpha, beta, gamma]"

Lattice System:

enum: ["triclinic", "monoclinic", "orthorhombic", "tetragonal", "hexagonal", "rhombohedral", "cubic"]

Symmetry:

type: string
desc: "Schönflies symbol"

Space Group (Sample Attribute Type):

Schönflies Space Group Symbol:

type: string

Hermann-Mauguin Space Group Symbol:

type: string

Space Group Number:

type: integer
desc: "International Union of Crystallography space group number"

Crystal Family:

enum: ["triclinic", "monoclinic", "orthorhombic", "tetragonal", "hexagonal", "cubic"]

Crystal System:

enum: ["triclinic", "monoclinic", "orthorhombic", "tetragonal", "hexagonal", "trigonal", "cubic"]

Primitive Crystal Structure (Sample):

Name:

type: string

Lattice:
type: Lattice
Space Group:
type: Space Group
File:

type: CASM prim.json type file

Number of Elements:

type: integer

Elements:

type: array of one or more elements, for example ["Zr", "O"]

Number of Components:

type: integer

Components:

type: array of one or more elements or molecules, for example ["Zr", "CH4"]

Number of Independent Composition Variables:

type: integer

Degrees of Freedom:

type: array of one of more of ["occupation", "displacment", "strain"]

Cluster Expansion Effective Hamiltonian (Sample):

Prim:

type: Primitive Crystal Structure

Basis Function Specs:

type: CASM bspecs.json file

Effective Cluster Interactions:

type: CASM eci.json file

Composition Axes (Sample):

Prim:

type: Primitive Crystal Structure

End Members:

"origin":

type: float
dim: [n_components]

"a":

type: float
dim: [n_components]

"b": (same as "a")
etc. up to n_independent_comp_vars

Formula:

type: string

Parametric Formula:

type: string

Monte Carlo Conditions (Process Parameter Type):

Temperature:

type: float
unit: K

Parametric Chemical Potential

type: float
dim: 1d

CASM Monte Carlo Calculation (Process):

Prim:

type: Primitive Crystal Structure

Composition Axes:

type: Composition Axes

Ensemble:

enum: ["grand_canonical"]

Method:

enum: ["metropolis", "lte1"]

Model:

Formation Energy:

type: Cluster Expansion Effective Hamiltonian:

Supercell Transformation Matrix

dim: [3, 3]
type: integer
desc: "Supercell lattice = Primitive lattice * transformation matrix, where lattice vectors are represented as columns"

Mode:

enum: ["incremental", "custom"]

Motif:

type: string

Initial Conditions:

type: Monte Carlo Conditions

Final Conditions:

type: Monte Carlo Conditions

Incremental Conditions:

type: Monte Carlo Conditions

Custom Conditions:

type: array of Monte Carlo Conditions

Treat array of custom conditions by making array of T and matrix of chem potential


Import/Export Data Log Files

The CDS Configuration Migration Tool often swallows warnings/errors and just runs through.  It requires manual inspection of the log file to glean what happened.  It would be great if there was a way to allow the download of the log file that is a results of a running the Import-Data action (Export-Data would be nice too, although it's less common to have issues with that process).  Currently I have to run the CDS Configuration Migration Tool manually with the same data file that I ran in a GitHub action and review my local log file to figure out what went wrong on individual executions.  This isn't feasible for upper environments.
### Tasks

PAC CLI already has --verbose for import/export data commands. It looks like it is not exposed on GitHub Action


Changed the check for mod rewrite

Description

Questions
Answers

Why?
I could not install shopware because according to it I didn't have mod_rewrite on. After some digging I found out that the $_SERVER variable doesn't have a mod_rewrite and the correct way to check for it is the apache modules function.

BC breaks?
no, works with PHP 4.3.2 and greater

Tests exists & pass?
no need.

Related tickets?
If this PR fixes an existing issue ticket, please add there url here.

How to test?
If unset the $_SERVER['mod_rewrite'] and it will still work (as you have the rewrite mod already installed).

Requirements met?
yes

Thanks @mamazu. I think this will enforce the usage of apache2 and will fail when using a different web server like nginx.
The info about mod_rewrite can be ignored when using an alternative web server.

I would try to get with ajax the sitemap.xml as example and if it response a 404 , rewrite is then not configured.  It would be server independent

We are closing this PR. The reason can be found in the comment by @janbuecker.


IntegerProperty class

implement class IntegerProperty(Property), mostly with default Property behavior , just some type casting in places.

IntegerProperty is as simple as it should be. It is just inherited from base Property with a couple of tweaks.


[IDEA] Changing Tuple ptrs to List ptrs

Is your feature request related to a problem? Please describe.

A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Changing Tuple ptrs to List ptrs will make  changing items that are held by ptrs easier

Describe the solution you'd like

A clear and concise description of what you want to happen. Write Pseudocode with Python

Tuple[Tuple[EnumType]] we will change them to List[List[EnumType]]

Describe alternatives you've considered

A clear and concise description of any alternative solutions or features you've considered. Write Pseudocode with Python
# No alternatives

Additional context

Add any other context or screenshots about the feature request here.

HERE

We will not use pointers


style(negative): cleanup imports

Removed unused imports
Organize imports

Can close this is https://github.com/longhorn/longhorn-tests/pull/1561 is merged first.

Can close this if #1561 gets merged first.

Closing.


Revert changes made when fixing #177

Hi all,
I recently noticed the behavior of the gaiohttp worker in gunicorn was behaving inconsistently with some of the others.  Specifically, it wasn't unquoting escape characters in PATH_INFO.  Looking back, I can see this behavior is intentional because of #177.
A change was made then to remove the call to unquote on PATH_INFO.  The justification was that the WSGI spec doesn't require it.  While the original PEP isn't entirely clear, if you check the wsgiref module, you'll see that PATH_INFO is indeed unquoted.
I propose reverting the changes in 0813710 and returning to the previous behavior in order to follow the wsgi reference implementation.
I'll open a pull request for this in a few moments.  Are there any objections or thoughts on this change?

I've no idea about the issue.
@Hooksie may you provide a code which fails with the current behavior?


Order of axes for CuImage.origin?

What is the order of the axes for CuImage.origin? Is it C-order or F-order? Also does it have any relationship to CuImage.dims
cc @thewtex (who raised this question)
cc @gigony (who may know the answer)

Maybe a follow up item here is adding this to the docs

@jakirkham @thewtex
The dimension order of the origin is crucial for 3D images, such as CT/MRI images.
There has been some discussion on origin, spacing, and direction at ITK's board: https://discourse.itk.org/t/images-in-physical-space-in-python/2124/15.
I don't recall the conclusion of the discussion, but it appears that establishing a standard for image metadata across Python image libraries is important.
cuCIM operates with a plugin system, where the primary plugin is the cuSlide plugin.
The cuMed plugin is intended to handle CT/MRI volumetric data, but its implementation is not yet planned; currently, we have only a stub plugin implementation for this.
Currently, for Whole Slide Images, the origin metadata is not extensively utilized and is always set to [(0.0, 0.0, 0.0)]
(https://github.com/rapidsai/cucim/blob/39f7c017f9a6eabed39214b68bfbd49b8e5434ad/cpp/plugins/cucim.kit.cuslide/src/cuslide/cuslide.cpp#L216).

Thanks Gigon! 🙏
When does this origin get set in CuImage construction? Is this part of the parse call here?
https://github.com/rapidsai/cucim/blob/46da10e05be93746910370ea5121447fbb21430e/cpp/src/cuimage.cpp#L186
Or does it happen somewhere else?

Thanks Gigon! 🙏
When does this origin get set in CuImage construction? Is this part of the parse call here?
https://github.com/rapidsai/cucim/blob/46da10e05be93746910370ea5121447fbb21430e/cpp/src/cuimage.cpp#L186
Or does it happen somewhere else?

@jakirkham
Correct, the method call would call static bool CUCIM_ABI parser_parse(CuCIMFileHandle_ptr handle_ptr, cucim::io::format::ImageMetadataDesc* out_metadata_desc) method here ( https://github.com/rapidsai/cucim/blob/39f7c017f9a6eabed39214b68bfbd49b8e5434ad/cpp/plugins/cucim.kit.cuslide/src/cuslide/cuslide.cpp#L216)


How different it is from Rest Assured framework or what is the advantage over Rest assured ?

I was using this framework for personal use and found it helpful and. But just need to know what advantage it has over the Rest assured framework. Because my entire team is already using Rest assured for Integration test and they need a reason to move on from that.

As this was an interesting question, we have raised another ticket to draw comparisons between other popular tools vs Zerocode.
Ticket is here-
https://github.com/authorjapps/zerocode/issues/206
(We will capture the points soonish. That will help all)


[Bug #3124] - clipped dropdown menu in Contents view

@pnicolli We already have some components in helpers. https://github.com/plone/volto/blob/master/src/helpers/ScrollToTop/ScrollToTop.jsx and https://github.com/plone/volto/blob/master/src/helpers/BodyClass/BodyClass.jsx
I'm ok with moving it in another place. Maybe components/theme/PopupWrapper ?

I think also that it's not a helper, but a component... I would re-place it, what do you think?
Also, but this is another question, what when we get rid of SemanticUI (eg. projects that do not use it at all? :)

Also, but this is another question, what when we get rid of SemanticUI (eg. projects that do not use it at all? :)

when we get rid of semantic-ui we'll have other, bigger, things to worry about :-)
In any case, we'd begin with a refactoring of the Contents component and move it to react-table or something like that. I think that's something we should already start doing.

@pnicolli @tiberiuichim What we agreed on this one?

If everybody agrees, we refactor it and move it to the components folder, then it's good to go?

If everybody agrees, we refactor it and move it to the components folder, then it's good to go?

Agreed

I still don't think components/theme is the right place to put it, as it's a helper component that can be used even outside Volto. I can agree that src/helpers is not the best location, but there's no other existing place for it. Maybe a new src/components/ui. But I'm fine with placing it in src/components/theme, so +1 from me.

@tiberiuichim I agree with you that it's a new kind of component, and that ideally should be "separated" from the structural ones. I also thought in having this new category in the components folder for Quanta. See
https://github.com/plone/volto/pull/3086/files#diff-9837e4844c73b95722bd37d81ab4ce85e00a07552905224cfd7d9041e62703daR1
I initially called that "core", but naming... again. We can start doing it that now, or reorder them later.

In any case, we expose components through the @plone/volto/components index, so if we decide that we need to move it later, we can do that without too much breakage. Let's put it in components/theme.

@mihaislobozeanu could you please take care of the decision? Thanks!

Resolved


Add persistence support for macOS

Added native support of macOS to the SwiftExtras module.
Added macOS as a testing target.
Updated formatting based on swift-format changes.

Codecov Report

Merging #16 into master will decrease coverage by 0.15%.
The diff coverage is 70%.

@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
- Coverage   59.74%   59.58%   -0.16%
==========================================
  Files          15       15
  Lines         390      386       -4
==========================================
- Hits          233      230       -3
+ Misses        157      156       -1

Impacted Files
Coverage Δ

...DuxExtras/Persistence/PersistStateMiddleware.swift
0% <0%> (ø)
:arrow_up:

Sources/SwiftDux/Action/ActionPlan.swift
100% <100%> (ø)
:arrow_up:

Sources/SwiftDux/State/OrderedState.swift
81.45% <57.14%> (+0.41%)
:arrow_up:

Sources/SwiftDux/Store/Store.swift
94.11% <88.88%> (-0.17%)
:arrow_down:

Continue to review full report at Codecov.

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


Add tests

Coverage increased (+0.85%) when pulling 6a7a35f524b970eac2e7337a07948c890ab07a3f on tests into dcdd49734fdc1b7b06919e7b9f6f7a2b8ffa3129 on master.


Bump bintami common from 1.x -> 2.x to avoid conflicts with other new…

…er Bitnami charts
What this PR does / why we need it:
I was having conflicts during running helm install between the Bitnami PostgreSQL and OpenLDAP charts due to different versions of the Bitnami Commons scripts 1.x versus 2.x.
In my opinion, this is a defect/deficiency with how Helm 3.13.x resolves helper scripts, this seemed to be the simplest fix
Pre-submission checklist:

[x] Did you explain what problem does this PR solve? Or what new features have been added?
[x] Have you updated the readme?
[x] Is this PR backward compatible? If it is not backward compatible, please discuss open a ticket first

@jp-gouin, is this PR worth merging?


Add OHMySQL

Project URL
https://github.com/oleghnidets/OHMySQL
Description
An Objective-C wrapper of MySQL C API.
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 8 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Warning

:warning:
Too many changes (when adding, please keep it to one project per Pull Request)

  Generated by :no_entry_sign: danger

Hm weird warning..

Thanks for contributing, @oleghnidets


Cannot delete/replace packages that fail validation

If you upload a package that isn't signed with the right certificate, the package will fail validation as expected:

But there's no way to go back and correct the issue.
First of all you can't upload the "fixed" package, because it already exists:

Second there's no option to delete the package, only unlist it:

Your only option is to increment the version number.

Duplicate of https://github.com/NuGet/NuGetGallery/issues/4678


CAB-3699: add e2e for disabled fields

Coverage remained the same at 77.125% when pulling 5de5ae1cfb8f520333931bd958d5ccc229ff8d02 on feature/cab-3699-e2e into dece5a6ac64574f326b1eb09255d326b798bd3ee on develop.


init.vim not sourced

Currently, the default init.vim is overwritten, to set the runtimepaths to pick up the plugins that are needed for Oni to function correctly.
However, this means settings aren't pulled in for custom init.vim
Need to source the user's init.vim, if it exists, once the app has started. Seems like we might want to add this in (NeovimInstance.ts)[https://github.com/extr0py/oni/blob/master/browser/src/NeovimInstance.ts] once the process has been spawned, we can source the init.vim if it exists.

Couldn't the extra runtimepaths be added through a command line option? --cmd <command> is executed before processing any vimrc, so this looks like a good way to set rtp.

Applying http://novaember.com/s/8f9453/ig79Ko.txt will append ONI's runtimepaths to the standard, but I haven't tested for any side-effects.
I also noticed a rather long startup time, not sure what that's caused by.

Thanks for the code snippet! I ported it over to a PR - #18. That's a great suggestion, I didn't know about the --cmd option, would be more robust than dynamically generating the init.vim.
I'm seeing a problem too though with the PR - it looks like Neovim is hitting an error that isn't getting rendered, so you need a keypress to activate the UI. (This might explain the long startup time in your case, too). Unfortunately, I'm not sure what that error is, so need to dig in a little more and see what's happening before bringing that PR in. Sometimes, if there's a script error on load, it doesn't make it to the UI - seems related to this issue in Neovim core: https://github.com/neovim/neovim/issues/3901

I believe this should be fixed by PR #18 - try out the oni.useDefaultConfig option and let me know.
Thanks!


3607: Fixed NJ ORC-12-3 field length too long

This PR fixed the NJ reported errors due to ORC-12.3 where the field value was too long: 47 > 30.
Test Steps:

docker-compose down
./gradlew package
docker-compose up
./gradlew reloadSettings
run:  ./prime  data --input junk/pdi-covid-19-20211221172001.csv --input-schema primedatainput/pdi-covid-19 --output-format HL7 --output-dir junk --route
Where pdi-covid-19-20211221172001.csv is a testing file that contains column "Ordering_provider_first_name" value greater than 30 chars.  In our case, it is: "SOMERSET VALLEY VISITING NURSE ASSOCIATION INC."
View the output HL7 ORC-12-3, it should contain as follows:

Changes

Include a comprehensive list of changes in this PR
(For web UI changes) Include screenshots/video of changes

Checklist
Testing

[x] Tested locally?
[x] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
[ ] (For Changes to /frontend-react/...) Ran npm run lint:write?
[ ] Added tests?

Process

[ ] Are there licensing issues with any new dependencies introduced?
[ ] Includes a summary of what a code reviewer should test/verify?
[ ] Updated the release notes?
[ ] Database changes are submitted as a separate PR?
[ ] DevOps team has been notified if PR requires ops support?

Fixes

#issue - List GitHub issues this PR fixes
https://app.zenhub.com/workspaces/onboarding--ops-62012871b0f8520011170cc6/issues/cdcgov/prime-reportstream/3607

To Be Done
Create GitHub issues to track the work remaining, if any

#issue

Specific Security-related subjects a reviewer should pay specific attention to

Does this PR introduce new endpoints?

new endpoint A
new endpoint B

Does this PR include changes in authentication and/or authorization of existing endpoints?
Does this change introduce new dependencies that need vetting?
Does this change require changes to our infrastructure?
Does logging contain sensitive data?
Does this PR include or remove any sensitive information itself?

If you answered 'yes' to any of the questions above, conduct a detailed Review that addresses at least:

What are the potential security threats and mitigations? Please list the STRIDE threats and how they are mitigated

Spoofing (faking authenticity)

Threat T, which could be achieved by A, is mitigated by M

Tampering (influence or sabotage the integrity of information, data, or system)
Repudiation (the ability to dispute the origin or originator of an action)
Information disclosure (data made available to entities who should not have it)
Denial of service (make a resource unavailable)
Elevation of Privilege (reduce restrictions that apply or gain privileges one should not have)

Have you ensured logging does not contain sensitive data?
Have you received any additional approvals needed for this change?

Pull reviewers stats
Stats for the last 30 days:

User
Total reviews
Median time to review
Total comments

MauriceReeves-usds
28
15h 19m
10

carlosfelix2
24
1h 43m
73

cwinters-usds
21
1h 38m
4

TomNUSDS
16
1h
14

jbiskie
13
1h 9m
0

brick-green-agile6
11
1h 5m
0

kevinhaube
10
1h 43m
1

jimduff-usds
10
3h 8m
2

whytheplatypus
8
57m
12

JosiahSiegel
7
59m
4

sean-usds
6
16m
2

oslynn
5
16h 45m
1

sethdarragile6
5
11h 43m
0

rhood23699
3
1h 57m
0

RickHawesUSDS
2
2h 21m
0

jorg3lopez
2
55m
0

clediggins-usds
2
1h 15m
0

ahay-agile6
1
1d 23h 57m
0

jeremy-page
1
21h 35m
0

victorialb-usds
1
2d 20h 38m
1

snesm
1
16m
0

jh765
1
2h 54m
0

@jorg3lopez this needs to be merged and/or closed. Can you verify we can merge this?


QUESTION:  On Windows, what task can be used to copy files from the drop folder using a UNC path to a local folder for testing?

Does a built-in task exist or is it necessary to create your own?

copy files is capable

Then I would respond that it is not working properly.  It complains that it cannot find the path.  I will file a bug.

This was a permissions issue.  I am running my agent on a machine that is not on the domain but am running it with same username and password as the domain user that can map the UNC path.  I had to also add just a local user with same username and password to the share and then everything worked.  Not sure if a better error message could be generated rather than just path not found.
Is this always the way permissions work.  I thought if you had a local user that had same username and password as domain user authentication would work.  This was not the case for me.

I'm not sure whether a better error message is feasible. If you fs.statSync the dir from node, what error code does it return? If ENOENT then not found is all we know.
I believe that's the way it works. Knowing the user name isn't enough, the domain is also required. Otherwise I believe it is falling back to the target computer as the "domain".


Inbound calls can still get stuck in a state where you can't decline them

Might be specific to m.call.invite's appearing in group rooms, but i thought i'd seen vdh suffer this too.  See https://vector.im/develop/#/room/!DgvjtOljKujDBrxyHk:matrix.org/$14718987041116kwLne:ralith.com for specifics

I think what I've seen is that, if Vector is backgrounded, the device is notified for the new call, but isn't notified if you pick up the call on another device. So the first device keeps ringing and is generally confused

duplicate of https://github.com/vector-im/vector-android/issues/437


Inline Assembler

Inline assembler is not supported by c2go, for example, the code fragment below.
unsigned long rdtsc() {
  unsigned int tickl, tickh;
  asm("rdtsc":"=a"(tickl),"=d"(tickh));
  return ((long)tickh << 32)|tickl;
}

Currently, having inline assembler in a program to be converted results in an error message: panic: unknown node type: 'GCCAsmStmt 0x3a991f8 <line:5:3, col:38>'.
Note that sqlite3 also contains a similar code fragment.

Note that sqlite3 also contains a similar code fragment.

Link?

You can download sqlite3 from https://www.sqlite.org/snapshot/sqlite-snapshot-201708251543.tar.gz and grep for rdtsc. It contains this fragment:
#elif (defined(__GNUC__) && defined(__x86_64__))

  __inline__ sqlite_uint64 sqlite3Hwtime(void){
      unsigned long val;
      __asm__ __volatile__ ("rdtsc" : "=A" (val));
      return val;
  }

Thanks for reporting this. It's interesting that you ran into this error when the mac and linux versions of clang on Travis did not. It must be because the preprocessor did not output this code.
Unfortunately Go does not support inline asm, so I'm not sure what the longer term fix for this is (perhaps using configuration to stub off such functions that contain asm). However, I will put a fix for the syntax parser in today.

See #229

Thanks for addressing the issue! I did not actually run into this problem on sqlite3; I'm just looking how different projects handle inline assembly. In Sulong, a C/LLVM IR interpreter on the JVM, we want to support inline assembly and have started to implement inline assembly for AMD64 by emulating its behavior in Java. For example, see the implementation of the rdtsc instructions: click me.
I'm currently doing an analysis of >1000 Github projects to determine which inline assembly instructions to implement to support as many projects as possible. rdtsc is one of the instructions that is widely used and occurs in 25% of the projects that use inline assembly. I can post a link to the study once it is completed, if you are interested.

That's interesting I would like to see the results of the study.
I'm not at all familiar with assembly. However if it is a requirement people need when converting their C code there are some more creative solutions that may work...

In according to README of project https://github.com/minio/c2goasm , this is can help.
I haven't any expertise about assembler.


Support for loadUpdateData in HSQL (CORE-1100)

See jira issue CORE-1100, this is the same as the file that's attached.

Thanks!


NooBaa CLI Re-design and Refactor

Environment info

NooBaa Version: master
Platform: Non Containerized

Actual behavior

The current src/cmd/ (CLI) components have a strange hierarchy. Accounts/buckets/whitelist management is under manage_nsfs while other NooBaa CLI commands are placed in the correct location.
The manage_nsfs help has a weird look and should be more user-friendly -

Indentation issues.
Missing info for each command.
check OC CLI for reference.
When running with no command we get invalidType error instead of help print.

Add --verbose/--quiet - NooBaa CLI should not print stderr to the user.
Manage_nsfs - The add action should be renamed to create.
Whitelist -

Currently, there is no action specified for whitelists, we always update.
We need to split the actions to get/delete/update, or remove completely and update/get/delete via editing of config.json.
change validation function to use net.blocklist API.
change the default empty string ('') default to undefined - which means that only when whitelist is undefined we will allow all, and not on empty string.

Expected behavior

We expect NooBaa CLI to -

Be user-friendly.
Not writing stderr logs to the user unless specifying --verbose.
Have a flattened hierarchy.
Account/Bucket add action should be renamed to create.

Steps to reproduce

run node src/cmd

More information - Screenshots / Logs / Other output
Example of indentation missing, missing info and stderr prints:
sudo node src/cmd/manage_nsfs account  --help
Error accrued while getting the data from /etc/noobaa.conf.d/config_dir_redirect: Error: ENOENT: no such file or directory, open '/etc/noobaa.conf.d/config_dir_redirect'
config.get_config_root - could not find custom config_root, will use the default config_root  /etc/noobaa.conf.d
load_nsfs_nc_config.setting config.NSFS_NC_CONF_DIR /etc/noobaa.conf.d
config.load_nsfs_nc_config could not find config.json... skipping
Usage:
account <action> [flags]

List of actions supported:
add
update
list
status
delete

Example of manage_nsfs invalidType error instead of help print -
sudo node src/cmd/manage_nsfs
Error accrued while getting the data from /etc/noobaa.conf.d/config_dir_redirect: Error: ENOENT: no such file or directory, open '/etc/noobaa.conf.d/config_dir_redirect'
config.get_config_root - could not find custom config_root, will use the default config_root  /etc/noobaa.conf.d
load_nsfs_nc_config.setting config.NSFS_NC_CONF_DIR /etc/noobaa.conf.d
config.load_nsfs_nc_config could not find config.json... skipping
{
  "error": {
    "code": "InvalidType",
    "message": "Invalid type, available types are account, bucket or whitelist"
  }
}

still relevant, commenting so stale bot won't close this issue


Skip SElinux recipe on non linux OSes.

Current coverage is 68.32%

Merging #223 into master will not affect coverage as of 00c1c06

@@            master    #223   diff @@
======================================
  Files            6       6
  Stmts          322     322
  Branches         0       0
  Methods          0       0
======================================
  Hit            220     220
  Partial          0       0
  Missed         102     102

Review entire Coverage Diff as of 00c1c06

Powered by Codecov. Updated on successful CI builds.

thanks for merging this, I am not sure it is sufficient to fix the windows setup so I may provide other patches in this direction.


Add LOINC Unmapped Frequency Tsv

Add_LOINC_umapped_frequencies_1232_2023-10-31

@janyaibe per email discussion, current SOP, implementation of the process in this repository and the position of the DQH group, we expect full list of StudyDescriptions as input.
I am not acting upon this PR until one of the two takes place:

DQH position is revised and we decide to amend the existing process.
Gen3 team revisits this PR to contribute full list of StudyDescription values.

@cgmeyer @janyaibe I am closing this PR per discussion at today's call.
Please submit a new PR that will add StudyDescriptions_Gen3.csv, which should contain 3 columns name Modalilty, StudyDescription and frequency. You can see example files that follow this convention here: https://github.com/MIDRC/midrc_dicom_harmonization/tree/main/in. This file should contain StudyDescriptions for all of the files (as opposed to unmapped files only).
Please let me know if there are any questions.


Add multiple consumers at once

Is there the possibility to add to the BBMetalCamera few consumers at once (maybe as an array) instead of chaining multiple .add(consumer:) function?
    public func add(consumers: [BBMetalImageConsumer]) {
        for consumer in consumers {
            lock.wait()
            _consumers.append(consumer)
            lock.signal()
            consumer.add(source: self)
        }
    }

I wrote this function, but apparently it does not work. I mean, if I init the BBMetalCamera in viewDidLoad() as follows:
let exposureFilter = BBMetalExposureFilter(exposure: -2.0)
let lutFilter = BBMetalLookupFilter(lookupTable: UIImage(named: "test_lookup.png")!.bb_metalTexture!)

bbMetalCamera.add(consumers: [lutFilter, exposureFilter, metalView])

print(bbMetalCamera.consumers) will show that the filters and the metalView are loaded as consumers, but the live view is not updated with the filters. But maybe this is due to the intrisic chain nature of the stack. I mean, each consumer added becomes the source of the next one.
What's more, as far as I understood, once a consumer is loaded in the chain, its intensity cannot be modified (for example the exposure value in a BBMetalExposureFilter), unless of generating again the whole chain with the updated value. Am I right?

I ended up with the following code:
    func addMultipleFilters(filters: [BBMetalBaseFilter]) {
        let first = bbMetalCamera.add(consumer: filters[0])
        var index = 0
        var lastFilter = first
        while index < filters.count-1 {
            index += 1
            let new = lastFilter.add(consumer: filters[index])
            lastFilter = new
        }
        lastFilter.add(consumer: metalView)
    }

which basically traduces the visual input represented by the array [Filter1, Filter2, Filter3, ...] in the chained representation:
metalCamera
-- filter1
---- filter2
------ filter(i)
-------- metalView

About adding multiple consumers
Your code addMultipleFilters(filters:) is right. We can add multiple consumers in a function. Maybe I will add this feature.
About modifying filter intensity
After a filter is loaded in the chain, we can still modify the filter intensity (for example the exposure value of BBMetalExposureFilter). The modified effect works on the new texture after modification. For example, we can slide a slider to modify the exposure value. If we are processing camera or video, we can see the modified effect immediately since camera or video transmits texture one by one. If we are processing image, we need to call transmitTexture function to transmit a new texture to see the modified effect.
var imageView: UIImageView!
var imageSource: BBMetalStaticImageSource!
var filter: BBMetalExposureFilter!

override func viewDidLoad() {
    ...

    imageView = ...

    let slider = UISlider(frame: ...)
    slider.minimumValue = -3
    slider.maximumValue = 3
    slider.value = 0
    slider.addTarget(self, action: #selector(slide(_:)), for: .valueChanged)
    view.addSubview(slider)

    imageSource = BBMetalStaticImageSource(image: image)
    filter = BBMetalExposureFilter(exposure: 0)
    imageSource.add(consumer: filter)
        .addCompletedHandler { [weak self] _ in
            guard let self = self  else { return }
            DispatchQueue.main.async {
                self.imageView.image = self.filter.outputTexture?.bb_image
            }
    }
}

@objc private func slide(_ slider: UISlider) {
    filter.exposure = slider.value
    imageSource.transmitTexture()
}

Now I think it is not necessary to add this feature to the library. Because the filter chain may be complex, and it is hard to design a function to set up a complex filter chain. So, design a function for your project. Your addMultipleFilters(filters:) function works.

Yes, I totally agree with you. Thanks for the help.


new rule - enforce that __init__.py files are empty

putting any code in __init__.py files is a bad idea, because they are unnecessarily imported when you import any modules in the same folder:
# ./package1/__init__.py
from package2.baz import baz

# ./package1/bar.py
bar = 1

# ./package2/baz.py
from package1.bar import bar
baz = 2

# ./package2/main.py
from package2.baz import baz

in this example, running python package2/main.py crashes with a circular import error, because package1.__init__ is imported even though none of the other files are explicitly importing from it.
it's absurd that the language would implicitly import a file that was not requested at all, so it would be nice if there was a rule to enforce that __init__.py files are empty.

I believe this can be merged with https://github.com/astral-sh/ruff/issues/9848.


Development: Refactor isAtLeast occurrences in client and unify spy testing

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I followed the coding and design guidelines and ensured that the layout is responsive.

Motivation and Context

#3742 refactored the client code such that exercise/course objects store the current access rights in the client. There were not used in some components.
Additionally GitHub was complaining about the checking of some spies in the client. These are fixed and unified in the code as well. (Point 11: https://docs.artemis.ase.in.tum.de/dev/guidelines/client-tests/)
Description

This PR refactores the last components to also use the exercise/course objects for the access rights.
The occurrences using the Metis service (e.g. post-header.component) is intentionally left out.
For checking the spies, we use .toHaveBeenCalledOnce()/.toHaveBeenCalledTimes(...) instead of .toHaveBeenCalled() and .not.toHaveBeenCalled() instead of .toHaveBeenCalled(0). This is unified in this PR
Review Progress

Code Review

[ ] Review 1
[ ] Review 2

I requested an Eslint rule for your changes over at https://github.com/jest-community/eslint-plugin-jest/issues/1198.

We need more approvals, one is not enough!


Resolve :compiler depepdency conflict

In this PR we remove exclude the :compiler module dependency from the protoData configuration.
This helps us to remove a duplicate :compiler artifact (which is not supposed to be there at all).

@alexander-yevsyukov, PTAL.


Link Error in Sample, 404 error core for cross-rg-deployment

In https://docs.microsoft.com/en-us/azure/lighthouse/samples
The link for the cross-rg-deployment return a 404 error Not Found
The links should be https://github.com/Azure/Azure-Lighthouse-samples/tree/master/Azure-Delegated-Resource-Management/templates/cross-rg-deployment
And not
https://github.com/Azure/Azure-Lighthouse-samples/tree/master/Azure-Delegated-Resource-Management/cross-rg-deployment

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

ID: 9a84fab5-0490-4f59-16ae-5bf88256ca9c
Version Independent ID: aab40959-c8c0-f6d0-d48f-f4d04a2fc8af
Content: Index of Azure Lighthouse samples and templates - Azure Lighthouse
Content Source: articles/lighthouse/samples/index.md
Service: lighthouse
GitHub Login: @JnHs
Microsoft Alias: jenhayes

@omiossec Thank you for your query. We will investigate and update further.

@omiossec I've reviewed the documentation and updated link. The changes should go live in few hours.

@omiossec Thanks once again for bringing this to our attention. The link has been updated now. We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Extension keeps complaining about no bazel file found.

Not all usages of Visual Studio Code has to be for working on projects which are using Bazel. Yet, when opening a directory or project which isn't using Bazel, the user keeps getting an error:

This error is excessive and contrary to the UX guide of VSCode extensions. If the extension doesn't find a Bazel file when opening a project, it should silently disable itself for this project, not annoy the user about it.

This is also happening when opening directory using Bazel but a file in bazel's output directory.

Agree - pull requests welcome 🙂


Ignore changefiles from CODEOWNERS

Beachball change files are causing everyone to get review requests

Coverage increased (+0.1%) to 83.364% when pulling e716f1a678af69d398236309e3d31b5c3b05e595 on tnorling-patch-3 into 9d1477634a3226a2e581adba5bab84b3deaa7ae9 on dev.


[CORE] Propagate fallback reason to union and shuffle exchange

What changes were proposed in this pull request?
This pr does two things

propagate fallback reason to union and shuffle exchange during validation
rename UnionExecTransformer to ColumnarUnionExec since we do not transform union in fact

How was this patch tested?
Pass CI

cc @PHILO-HE @zhouyuan thank you

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query
log/native_4489_time.csv
log/native_master_01_22_2024_19a863c0d_time.csv
difference
percentage

q1
32.70
32.31
-0.396
98.79%

q2
24.29
24.07
-0.211
99.13%

q3
36.74
37.08
0.340
100.93%

q4
37.89
38.49
0.604
101.59%

q5
68.92
68.33
-0.589
99.15%

q6
7.05
7.26
0.207
102.93%

q7
83.55
83.81
0.265
100.32%

q8
81.74
83.38
1.647
102.01%

q9
121.22
117.04
-4.176
96.55%

q10
44.86
40.47
-4.394
90.20%

q11
20.11
20.27
0.161
100.80%

q12
29.13
27.53
-1.598
94.51%

q13
45.71
46.48
0.768
101.68%

q14
22.27
15.48
-6.790
69.51%

q15
27.68
28.47
0.798
102.88%

q16
13.58
15.34
1.760
112.97%

q17
101.04
100.56
-0.472
99.53%

q18
145.65
145.52
-0.131
99.91%

q19
14.00
12.41
-1.589
88.65%

q20
27.26
26.33
-0.928
96.60%

q21
223.56
225.25
1.685
100.75%

q22
13.41
13.60
0.188
101.40%

total
1222.32
1209.47
-12.853
98.95%


cert-manager fails to work with flannel+ipsec

Environmental Info:
K3s Version:
k3s version v1.19.5+k3s2 (746cf403)
Node(s) CPU architecture, OS, and Version:
Linux node01 4.4.167-1213-rockchip-ayufan-g34ae07687fce #1 SMP Tue Jun 18 20:44:49 UTC 2019 aarch64 aarch64 aarch64 GNU/Linux
Cluster Configuration:
1 master, 4 workers. /var is on USB flash. Using the default storage backend. Cluster was initialized with --flannel-backend=ipsec.
Describe the bug:
After cert-manager is loaded, it fails to validate a simple ClusterIssuer.
I think this is because the apiserver can't contact the coreDNS pod over the encrypted flannel connection, but that's just a hunch.
Steps To Reproduce:

K3S_TOKEN=$token ./k3s.sh server --disable-local-storage --flannel-backend=ipsec on control node.
K3S_TOKEN=$token ./k3s.sh agent --server $server on worker nodes.
Install and configure system upgrade controller, velero, and longhorn.
kubectl apply -f https://github.com/jetstack/cert-manager/releases/download/v1.1.0/cert-manager.yaml
Apply this ClusterIssuer:apiVersion: cert-manager.io/v1
kind: ClusterIssuer
metadata:
  name: letsencrypt
spec:
  acme:
    email: $email
    server: https://acme-v02.api.letsencrypt.org/directory
    privateKeySecretRef:
      name: letsencrypt-account-key
    solvers:
    - http01:
        ingress:
          class: traefik

Expected behavior:
clusterissuer.cert-manager.io/letsencrypt created

Actual behavior:
Error from server (InternalError): error when creating "cert-manager-cfg.yaml": Internal error occurred: failed calling webhook "webhook.cert-manager.io": Post "https://cert-manager-webhook.cert-manager.svc:443/mutate?timeout=10s": context deadline exceeded

Additional context / logs:

My understanding is that the flannel ipsec backend is pretty broken out of the box. Have you tried it with any of the other backends? I've heard that wireguard works quite well.

My understanding is that the flannel ipsec backend is pretty broken out of the box. Have you tried it with any of the other backends? I've heard that wireguard works quite well.

Thanks for your reply. I haven't tried any other backend yet.

Thanks for your reply. I haven't tried any other backend yet.


Connect to multiple HR devices

Hi, i am really happy i found your code and that it is working :-) But i need it to collect the data from multiple devices.
Since i lack the skills to easily create that, i was hoping you can give me some pointers...
i know i will need to open other channels but simply changing the channel
heartRateMonitor = new Ant.Device.HeartRateMonitor(antDongle.Channels[1]);
will throw System.TimeoutException: 'The operation has timed out.' (when i search for devices). this only seems to work for channel 0. why is that?
So var i have introduced a new class HR which holds a HeartRateMonitor object. And MainWindow has a list of HR objects... but when i try to use multiple HeartRateMonitors the application crashes as the 'Device is currently active' :-(
according to some documentation you should be able to connect to multiple devices on 1 channel. (i think the max is 8).
i did get it to work where i initiate 8 open channels and each channel connects to 1 device ... but then the limit is 8 devices
i hope you can help me with this. i am willing to donate to any given help :-)

Hi, I have not worked on this project for quite a while. However I can reproduce your problem and I might have an idea what is wrong here. I will get back to you after some further investigation.

I updated the ant-net library with a qiuck fix. I wrongfully hard coded the channel ID for the request message to 0. However I only have one heart rate sensor right now so I was able to connect it to channel 1, but I am not able to test multiple sensors at once.

Thanks! Also for the fast reply and the quick fix! :-)
i do have 1 more question...  the search for a device (in your code) is just sending a connect request to any given device. While i would expect that the search send a broadcast to really get all the devices in the area. am i correct? i was hoping to get a list of devices, but i dont think i can do that with this code, can i?

With the current implementation a wildcard connect is executed that connects to the first device that is available. There are multiple strategies on finding ANT devices, however none of them are implemented in my ant-net library. You can find various application notes regarding the pairing and search techniques available on the ant website, however you would have to implement them yourself. Also, depending on your USB device, not all of those might be available to you. You need to check the capabilities of the device first to select the proper strategy available to you.


Trial Bronze cloud plan server not interfacing with same UI as Rocket.Chat server...

Please see this forum topic
I really like the current UI on the Rocket.Chat server but I can't show the powerusers.rocket.chat group to any of my Slack teammates who I am trying to convince to migrate to Rocket Bronze..
They would laugh in my face...
Any ideas how to get the current Rocket.Chat server UI onto my powerusers.rocket.chat group?
Thanks,
Dale

@DaleKramer I'll keep an eye on that forum topic while contacting the @RocketChat/cloud team to help you. I'm closing this issue since is not a desktop app problem.

Thanks I did not know it wasn't a desktop app problem until I finally tried it on my iPhone , sorry...

No problem! :hugs:


Implement DiscV5 packet encoding and decoding

This PR implements encoding and decoding functions for DiscV5 packets as defined in the wire protocol spec. To keep it as short as possible, most data is treated as a black box. I'm going to add encryption/decryption functions etc. in a future PR.
Unfortunately, terminology is somewhat confusing (me, at least), in particular the overloaded usage of "nonce", "token", and "tag". But changing it on our own would likely worsen the situation even further, so I've decided to stick with the variable names that are used in the spec.
To-Do

[ ] Clean up commit history

[ ] Add entry to the release notes PR

Cute Animal Picture

Closing in favor of one smaller PR per packet type, starting with #736.


Update sbt-scala-native to 0.4.16 in main

About this PR
📦 Updates org.scala-native:sbt-scala-native from 0.4.14 to 0.4.16
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-native", artifactId = "sbt-scala-native" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-native", artifactId = "sbt-scala-native" }
}]

labels: sbt-plugin-update, early-semver-minor, semver-spec-patch, commit-count:1

Superseded by #76.


Reverse order of LogBinner type parameters

LogBinner{T, N} is better than LogBinner{N, T}

AbstractArrays also have {T, N} order
One probably wants to dispatch on type T more often than on the number of levels N. Here one could then use the shorter LogBinner{T}.

Iirc I just copied the order of NTuple when I wrote the first version. I think this would be a good change


Feature Request: Ability to use Cthulhu within RemoteREPL

:sweat_smile:

To what extent is the Cthulhu.jl interface built around having direct access to stdout for display and stdin for user input? Looking at the code it seems this is somewhat assumed.
Seems there's two ways forward:

Generalize Cthulhu to allow stdout and stdin to be swapped in some way. Eg, make sure a REPL.Terminals.TTYTerminal is consistently passed around in a way we can override it (or should that be Base.TTY?). Perhaps make the TTY overridable with a task local variable. Cthulhu implicitly uses REPL.TerminalMenus.readkey to read keys from the user; eventually that needs to end up as a read to the client-side TTY.
Alternatively, invent a way for RemoteREPL to redirect and steal the global stdout and stdin of the server process. Certainly this is possible for stdout; for stdin I'm not sure how this could work. Presumably it'd need to involve a TTY.

The tricky part seems to be finding a way to detect that the server code is waiting on a read of the serverside TTY so that we can forward this read to the client. But without blocking the client TTY by trying to read from it when the server isn't waiting for input.


Use yoda syle comparison in SymfonyRequirements.php

As far as I'm aware, the Symfony code style guide requires boolean expressions to use yoda style comparisons. I've updated the SymfonyRequirements.php file to reflect this.

fabbot.io also complains about another issue:

Would you like me to fix this too?


chore: move all builtin table logic from sqlexec -> sqlbuiltins

Description
We already did this with the functions, we should do it for tables too.
sqlexec should be reserved for execution logic. sqlbuiltins reserved for builtin tables/functions/views/...

for some additional context, I found it especially confusing trying to figure out how the actual data for a table was built. Currently this happens in sqlexec, but everything else happens in sqlbuiltins.


Rendering complex Mermaid.js flowchart diagrams causes text inside nodes to be cut

I tried to use this diagram here:
graph LR

  subgraph lv0[lv0]
  direction LR
    RW[Rosary Warden]
    WoS[Weight of Sin]
  end

  lv0 -->|lv1| Re[Retribution]

  Re --->|lv2| RP[Relentless Power]
  RP -->|lv3| SS[Sacred Steel]

  lv0 -->|lv1| BP[Blood Pact]
  BP --->|lv2| De[Determination]
  BP --->|lv2| CoA[Covenant of Atonement]
  BP -->|lv1| G[Reaper Rosary]
  G -->|lv1| MS[Mystic Striker]
  G -->|lv2| BS[Blessed Steel]

  lv0 -->|lv1| CW[Crimson Warmonger]
  BS -->|lv3| RprP[Reaper Prayer]
  CW -->|lv2| CC[Crimson Cleaver]
  CC -->|lv2| K[Weight of Justice]
  CC --->|lv3| N[Crimson Glaive]

graph LR

  subgraph lv0[lv0]
  direction LR
    RW[Rosary Warden]
    WoS[Weight of Sin]
  end

  lv0 -->|lv1| Re[Retribution]

  Re --->|lv2| RP[Relentless Power]
  RP -->|lv3| SS[Sacred Steel]

  lv0 -->|lv1| BP[Blood Pact]
  BP --->|lv2| De[Determination]
  BP --->|lv2| CoA[Covenant of Atonement]
  BP -->|lv1| G[Reaper Rosary]
  G -->|lv1| MS[Mystic Striker]
  G -->|lv2| BS[Blessed Steel]

  lv0 -->|lv1| CW[Crimson Warmonger]
  BS -->|lv3| RprP[Reaper Prayer]
  CW -->|lv2| CC[Crimson Cleaver]
  CC -->|lv2| K[Weight of Justice]
  CC --->|lv3| N[Crimson Glaive]

Although it renders as intended on mermaid.live and Github I wasn't able to make nodes to acommodate its text on Nextra. I tried to change font size (both for each node and as a class), used frontmatter config, remake nodes positioning, etc.
Nextra

mermaid.live

Also, abbreviating is not a quick fix

can't reproduce, also it's an upstream issue
https://github.com/shuding/nextra/assets/7361780/b0212744-d524-47ed-af6d-9b4c354eab8b

Which was yours resolution? I got this at 1366x768

my resolution is 3456 x 2234
I tried to set your resolution in developer tools, but got same result

closing due to lack of sharing link of deployed page

Hey @dimaMachina, We are facing the exact same issue here. Please have a look.


Radio button list doesn't work

When using radio buttons in NC, they get the same name attribute across the different items. This means, that clicking a radio button in one item, removes any settings in other items.
This is because of the way Umbraco renders a radio button list. Don't know if we can do anything about it in Nested Content.
Btw, thanks for a great plugin!

@skttl Thanks for raising this. We'll take a look to explore options (or flag it as a known issue).

Finally had chance to look at this, yup - all the radio-button-lists get given the same name attribute :-(
I'm not sure what we can do about it (within NC) at the moment.  I'll leave this issue open for now and flag it as a known issue.

Closing this one off, as it's beyond the control of Nested Content. Flagged as a known issue.


Old info needs updating

Greetings Folks,
This page on the Cloud Shell needs updating for example it isn't Ubuntu but now Debian.  A lot of the software has been upgraded and changed.  New versions: PowerShell 7.1, Python 3.7, & Ruby 2.5.
Source: https://azure.microsoft.com/en-us/updates/the-azure-cloud-shell-image-has-been-updated/
Cheers,
Jeff Smalley MCT
Chicagoton

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

ID: 18fb3b37-8ba1-579c-ceb4-329c283f107a
Version Independent ID: 0ead5f4c-63a0-d782-0a89-b5d612596d89
Content: Azure Cloud Shell features
Content Source: articles/cloud-shell/features.md
Service: azure
GitHub Login: @maertendMSFT
Microsoft Alias: damaerte

@JeffSmalley -
Thanks for brining this to our attention. We will investigate it further and update you shortly.

@JeffSmalley -
Thanks for brining this to our attention. We will investigate it further and update you shortly.

@JeffSmalley -
Thanks again for bringing this to our attention. We have updated the Doc and it will be live once its merged.  We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@JeffSmalley -
Thanks again for bringing this to our attention. We have updated the Doc and it will be live once its merged.  We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


59 accounts: 0xA5b39e8438264691103Ff4a35c8034658FEE1227

0x8adc75aa643cd5d804bab300a2d5becd9a9e6b8b
0x84443f61efc60d10da9f9a2398980cd5748394bb
0x113e304d9bd13ec54929399e265d8ec723c4bbe3
0x1260af8fdbe715bcfc13b257404c07327a600819
0xf029f8f3265e77ef6e6eb71511145c5c6ba5d39d
0x209584d708633dff4c6a4d7359525bbe26d07333
0x83640976768d490f8456f50456b383a320f0baf9
0xa9d04e8b992248834a06e61e4d8602e3434ece37
0xd55929ff0f0d9e30a158baca29cd10300adca839
0x4b435b57ab0aa32f169aba415122a4a981dad2a1
0x52ca5a12c80633c6cac0504a6007a760ad9c3ff9
0x8700c81883b46922d4a9bc1edea0b77337ec0eb1
0xd2e4f9136490d6cb48eb64ceaf1c6fa165f4d2c7
0xc2890a210b1f4f7248dfb57491512bd21c53a9cf
0xdd282d17b0b2c282e4e83f2b7747017f852be4ff
0x0c5095a8837e633312d897b9f51535c5161c8b71
0x1eaa6365c625b09ef45c3b8a6f2aab43376e531f
0x9d65822e35310342294afb07f5a32347db890fed
0x292bacf82268e143f5195af6928693699e31f911
0xeecaca98f3d4851092e7089d922c61f1feadb843
0x7589b95b1c732d06d134381f16650fef4e03a1b9
0x27d530b25f13996e9c77135e361eb8bf73759d63
0x5aea111f6f1e65ffc4d3b31f3320405d3f294f05
0x436ab15705e0ed23284512770748770ee4c16155
0xc0495a7bd160c1aa2a71f9e16891079c7a9de747
0xf5668fbc7378bcc5a6f0795d7e9b5e5c3b4f7215
0xc133439da7da854f84f25685d2191464ecd3d21b
0x89b311f344ccddc54c2f7258f6e84a54390cdb89
0x9f10dbe6f9419f73117483503116105d2c7b2c15
0xbe0fa187c0d25eb1ccc4a482912d30f66565101b
0x94ddea50b0b0846d6a65404a9bd9547ee7995989
0xd67fdeeb0dea8693aac0c04efee0786c31b90cb1
0x5a1bc518e8d34979479aa9c48cc3e3c062248fa7
0x1adf825f650938a6a93f899d347f3d2cd073dc29
0xb0cf84945f0d8c7535e787c5dfbb4cfa5a672baf
0x9fefd869afc9e88aec611a2f5a0510a1be578f1b
0xbc2f73492bed65351b319ccb6fe66a8485a1cb03
0xa89f55b203e289af2c7b13dd1085ff31d3178371
0x2bfd56b4742a63778cc5c034655d5275f0c5e7e1
0xda2d6f7062e8703c79145fa16f7f9f52980c2e77
0x7ab3057b217dedaba453dab9d0b95b5803250007
0xdb4a1a616e2cc782911ec19cdb5651647f8111c9
0x6e68b398924e3da3d29714feefb82400f9993cc7
0x98a168e8ae8141e505d7d4bb9c47ba3eb1fa8e57
0x23cce7a20f4d012424253b5433576e2dd943330f
0xd8bba4d0d953fedd3dcd5a29b2b52e93215544a3
0xfb87640660b4164c1be3b45c1e6a0fb9f40c4691
0x91c6842b79ad4a0f4a03ae274af12a54ab71ee8d
0xeb7b1c0ec3d3a5170356fd21ae6ac41423e28403
0xc3f955a5737840394050e526610475407a4e1b23
0x032d80848be927200fb06b47d985f84e49977ae9
0x104d1fee1e1e3647083bfc770074f89e2d4bee4f
0x4504282c1ef82e23260f0573eaf5c0463fcc77b3
0x1a67197536c1019b732b64f9d29a1f7a3a892fa9
0xbbbed61c90d6c7f0db93820584fabe06b8570ec7
0xe51befe06f6dfdfa43665b8739eae551b165f255
0x1eda046c1aa126b459572a7e6ae329696496671f
0x489fe30ca2e433c04f15fcc1d8935c12b4d8183d
0x9d59e70d478a50833e0c834e79220e3d13775d3d
Reasoning
all created by 0xA5b39e8438264691103Ff4a35c8034658FEE1227
There are many related transactions between addresses.
Methodologyfrom @MisakaCenter
Execute at luabase.
with target as (
select *
from url('https://raw.githubusercontent.com/safe-global/safe-user-allocation-reports/main/safe_user_allocations_reworked.csv', CSVWithNames)
), transactions as (
select t1.*
from ethereum.transactions t1
join target t2
on t1.to_address = t2.safe_address
)
select distinct to_address
from transactions
where from_address = lower('0xA5b39e8438264691103Ff4a35c8034658FEE1227')

Air drop hunter 0x89930b501D06f13Aff33bC33C1d51d9C5F8a9dbb

How sure are you this is airdrop farming?
Check transaction that occur almost 2 years ago since before the spin out

First, the main address: 0xA5b39e8438264691103Ff4a35c8034658FEE1227, the most recent transfer record https://etherscan.io/txs?a=0xA5b39e8438264691103Ff4a35c8034658FEE1227&p=1, the most recent transfer happened 346 days ago to the sub-account, and the last 25 records were transferred to the sub-account through Relayer token, which is unusual in itself, unlike what a normal user should do, as shown in the figure below

Then check the earlier records, take 904 days as an example, still mainly rely on Relayer transfer

This is a premeditated wallet.

After investigating all the associated accounts, most of the interactions of the addresses were before 600 days, and the number of transfers in all accounts was less than 25, generally between 3-10, and most of the wallets were empty, or there was a small amount of gas left. , which means that there is no transfer other than interaction. Basically, it can be considered as the multi-signature address of one creator. The following are links and screenshots of some associated addresses.
https://etherscan.io/address/0xf029f8f3265e77ef6e6eb71511145c5c6ba5d39d

https://etherscan.io/address/0x83640976768d490f8456f50456b383a320f0baf9

https://etherscan.io/address/0x1eda046c1aa126b459572a7e6ae329696496671f

https://etherscan.io/address/0xc3f955a5737840394050e526610475407a4e1b23

https://etherscan.io/address/0xeb7b1c0ec3d3a5170356fd21ae6ac41423e28403

https://etherscan.io/address/0xa89f55b203e289af2c7b13dd1085ff31d3178371

https://etherscan.io/address/0x7589b95b1c732d06d134381f16650fef4e03a1b9

https://etherscan.io/address/0x5aea111f6f1e65ffc4d3b31f3320405d3f294f05

https://etherscan.io/address/0x9d65822e35310342294afb07f5a32347db890fed

@tschubotz

This is Rockside Relayer Address https://rockside.io
2 years back they created a wallet called tongue and airdrop different tokens to wallet users
This isn't Sybil
REFRENCES
https://rockside.io
https://www.tongue.fi
https://twitter.com/rockside_io/status/1320787477217640449
https://twitter.com/rockside_io/status/1328334303936319488
https://twitter.com/TongueWallet/status/1328333624584892416
https://twitter.com/TongueWallet/status/1320738611994742784

Checking the first 3 Safes
https://gnosis-safe.io/app/eth:0x8adc75aa643cd5d804bab300a2d5becd9a9e6b8b/transactions/history
https://gnosis-safe.io/app/eth:0x84443f61efc60d10da9f9a2398980cd5748394bb/transactions/history
https://gnosis-safe.io/app/eth:0x113e304d9bd13ec54929399e265d8ec723c4bbe3/transactions/history
They all look like legitimate usage.
Also what @DeveloperThing mentioned, these are likely to be wallets related to Tongue or some other wallet using Rockside.
Hence there is a quite high chance we would eliminate legit users.

Sharing some personal notes during my investigation of Tongue Wallet
Tongue wallet address 0xdb4a1a616e2cc782911ec19cdb5651647f8111c9 mentioned by https://twitter.com/blakeburrito/status/1323805940567650304
On 10/30/2020 @TongueWallet tweeted that "Monday we’ll be starting our ten day distribution of random coins with a global daily worth of $500. Available only for activated https://tongue.fi wallets". tweet link
Tx history since the Monday of 11/2/2020
https://etherscan.io/tx/0x7623eadbc90563cf86258fd25033cdbf66c72dc15fa63b3381a7515998fdb2d3
https://etherscan.io/tx/0x1e8e0c614d931131959ddd88e52d3a4827d88af89d59977f9a4c6dd3756b8262
https://etherscan.io/tx/0x2e7bb756c97b59a6f9f92bff15563c09398eace5ccd95ec04bf93116f1de37b5
https://etherscan.io/tx/0x2505696a0fe4b1bfbc10c9ae79d0278d17b01bb930a056337e5c49679ebd7762
https://etherscan.io/tx/0xa3bbe08e9883d16e7aca3bc59c3e3eacdbd763a179c2b9b91da183f1e2f5b8ac
https://etherscan.io/tx/0x882eee5600fb18bb5e557f1c2e806335f8198ee18e809e88de35190055752aee
https://etherscan.io/tx/0x8212204ea354f7121e95f3cacfe38ae5b46b587877302e70db4e7f5a54b38127
https://etherscan.io/tx/0xd2fa7c7b2021576bb756f3e96a9e85891c4692277ba19113b0040af9abf32efa
https://etherscan.io/tx/0x709e5325f4c9ec1f4715003aff766d664137b72eab7ad95b26293c073fffa9b2

Receiving small amounts of UNI, MEME, MKR, DAI airdrop is typical of Tongue wallet. I confirm that @DeveloperThing is correct


Replace core::SRange with std::span and other templated usages with std::view concept template

Description
Continuation of the suggestion in PR #470
Solution proposal
As per the title.
Additional context
After WASM/Emscripten compilation, don't want to use too many C++20 or C++23 features without knowing they work with WASM.

@pollend your wish was granted, now I use spans in a lot of places in video namespace on the vulkan_1_3 branch


Add MyMLH Setup Guide

There should be detailed instructions on how to setup MyMLH authentication for hackathons.
These are separate instructions than the MyMLH setup for development

Waiting on #291


hugginface demo

please add huggingface demo for people out of china

We are applying for free GPU resource from HuggingFace at the moment. You can try out our English demo:


Videos with no views are labeled as live

Nullable check doesn't account for 0 views and assumes null and thus live for the "views" value

You can use searchtube.


Docker image and fat jar build.sbt incorrect : Missing dependencies

I spend some times trying to use the docker image and I kept getting error on the UI, I then realized that the JAR had missing library.
So I started to try to compile a "FAT jar" but it was missing the same libraries, but sbt run was working fine.
I modified the "assembly" part in the build.sbt to get it working as follow :1234:
// Package fat jar
assemblyOption in assembly := (assemblyOption in assembly).value.copy(includeScala = true, includeDependency = true)
/* you need to be able to undo the "provided" annotation on the deps when running your spark
programs locally i.e. from sbt; this bit reincludes the full classpaths in the compile and run tasks. */
fullClasspath in Runtime := (fullClasspath in (Compile, run)).value
assemblyMergeStrategy in assembly := {
case PathList("META-INF", xs@_*) => MergeStrategy.discard
case _ => MergeStrategy.first
}
PS: Sorry I am not a developper at all so this might not be the best solution but I wanted a working "Fat jar"  and so far it is working.
Thanks for the great work, this is an awesome project !

Thanks! I realized what went wrong in docker build. I used to have spark as a direct dependency, thus java command can execute it directly. But it's no longer the case after I marked spark dependencies as provided. The simplest fix is to set docket command use sbt. I'll push a fix very soon


if override_defaults is None, passing kwargs crash

Thanks @phunter! I'll just add some test case to this first before merging... :)

Merged #789, so closing this. :)


Updated cs (Czech) translation

Commit made via Stringlate

So quick!

Oh @GitHubBot is a wonderful invention :)


Report current git sha along with Sentry errors

This will hopefully aid debugging.

Not sure, to be honest. Sentry's search and filtering seem a bit limited so that might not be possible.

Coverage increased (+49.3%) to 75.843% when pulling 5e3e7821bc256d8c5c88c1fb3ffdc009b451e83c on report-git-sha-to-sentry into 62ea22deb6fb4f826570d26f58554cfd88195666 on master.


HIVE-24667: Truncate optimization to avoid unnecessary per partition DB get operations

What changes were proposed in this pull request?

Moved DB fetch out of partition loop
Why are the changes needed?

Performance improvement
Does this PR introduce any user-facing change?

No
How was this patch tested?

Unit tests

LGTM +1 pending tests.

LGTM +1 pending tests.


Add license

This matches the license text from another MicrosoftDocs repo. If it's the wrong license for this repo, that's fine, but please add some license text to this repo. Right now it's not clear if Microsoft supports reuse of this documentation or not for projects like https://github.com/sfreed141/Direct3D12-docset
@sfreed141 FYI

@Karl-Bridge-Microsoft (apologies if you're the wrong person for this, I just saw you were the most recent person to merge a PR) could you take a look at this or assign someone else as appropriate?
I'd like to improve the project mentioned by Ben but I'm hesitant to continue if there's any legal issues that could cause it to be taken down.

Hi @benmcmorran - thanks for flagging but I don't think this is the appropriate license text. Our other public repos have some open source verbiage with links in their readme that this repo should also have. I'll update your PR with that text.

@GrantMeStrength - can you have a quick look and confirm that this repo and https://github.com/MicrosoftDocs/win32 now align?


I can't close the Gradle process

Hi,I have a probleam,I fell very strange,
I don't sure it's a bug or some configuration error of mine.

Have another Gradle instance.I try to close it.but i can't.
There is a Gradle process in my system.
Its pid is change every second,It's active when i reboot my computer,I can't close it.

So,Delete lock file of caches directory do not help.
Build any project that's i create or already exists is failure in idea.
Maybe is setting error of idea of mine,I set gradle and java environment,And setting gradle home directory for build tools of idea.
Thanks look this issue.

You're not seeing a Gradle process.  That's just grep. The command has gradle in its command-line.
You'd see the same thing if you ran ps -a | grep foobar (there's no foobar process).


Override type name if specified

Fixes a regression introduced in #356
We should override the TypeName if it's specified.
Note that there are different defaults depending
on how the options are created.

Created from configuration it defaults to logevent
Created from code it defaults to _doc

What issue does this PR address?
#364
Does this PR introduce a breaking change?
No
Please check if the PR fulfills these requirements

[x] The commit follows our guidelines
[x] Unit Tests for the changes have been added (for bug fixes / features)

Thanks for looking into this and providing a fix!


nodejs: switch from 14.x to 16.x to keep up with the lts release

Motivation for this change
See: https://nodejs.org/en/about/releases/
It's also the only version that works properly on aarch64-darwin.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

It's been fixed for node2nix: https://github.com/svanderburg/node2nix/commit/e73c616408b753c386301e1d21ac634c959a4b6f

We could do an unstable release of node2nix, but I feel it's better to keep what we have and change when node2nix makes a new release.

https://github.com/NixOS/nixpkgs/issues/144627#issuecomment-975947953 mentions that the branch off happened already. Is it time to revisit this?

btw, there is an update in progress of node2nix https://github.com/NixOS/nixpkgs/pull/146440/files
but yeah I think this can potentially be merged to staging. Whenever there is time to rebase and fix the conflict, I personally think this is mergeable, unless anyone sees an objection that I have missed.

Rebased

I just noticed, node16 fails to build on aarch64-darwin. @marsam is this normal or is this just a transitive failure ? (no worries if you don't know). Given that nixos supports darwin on tier1, we can't just merge this then ? (let me know if I'm wrong).

@happysalada it builds just fine on master branch:

https://hydra.nixos.org/eval/1725069?filter=nodejs&compare=1724929&full=#tabs-still-succeed

@bobrik thanks for checking this out. Let's merge then.

This PR cause this regression https://github.com/NixOS/nixpkgs/issues/152745 As far as I know we still lack support for nodejs 16 in node2nix which makes me wonder - What was tested before merge here? I have a feeling there might be more things broken at the moment.

I thought the latest version of node2nix supports node 16 (version 1.10). There is a current PR to regenerate and test nodePackages https://github.com/NixOS/nixpkgs/pull/146440 (still WIP).
Since it was impossible to test all the packages, if I remember correctly nodePackages were tested. We indeed missed elmPackages indeed.
I personally use
      nodePackages.prettier
      nodePackages.pnpm
      nodePackages.node2nix
      nodePackages.npm-check-updates
      nodePackages.svelte-language-server
      nodePackages.yarn

and those work like a charm.

We indeed missed elmPackages

don't worry about that. It's intentionally designed in a way it doesn't block work on other parts of nixpkgs. I'm happy to take care of it. That's being said I think there might be other packages which rely on expressions produced by node2nix using nodejs argument which might need updating. It's not just nodePackages (which are btw independent of what is defined as nodejs in nixpkgs so I think they don't even use node 16 but I might be mistaken) - there can be applications using node2nix for packaging and using nodejs argument.
Anyway I think we can deal with all the potential issues if there are any.

To clarify. I think that if intention of this PR was to update nodejs version things defined within nodePackages are using I don't think it accomplished that. This updated nodejs version for everything which uses nodejs attribute directly but I think nodePackages are using explicit attribute name which was updated in https://github.com/NixOS/nixpkgs/pull/149120 to version 14. Anyway take this with grain of salt - I did not check the code base this is just based on my memories of working with that part of nixpkgs about a year ago.
In order to update nodejs version for things that use node2nix we need to wait for support upstream. Current highest version node2nix supports is 14. However this should update nodejs for derivations that don't depend on node2nix (don't use npm dependecies, just node).


npm init solid with bare and hackernews templates results in broken apps

mkdir solid-bare
cd solid-bare
npm init solid
# bare or hackernews | SSR | ts or no ts
npm install
npm run dev -- --open

results in broken app:

The console output is:
dev.js:249 Unable to find DOM nodes for hydration key: 0-0-0-0-0-0-0-0-0-1-0-0-0-0-2-1-0-0-1-0-0
getNextElement @ dev.js:249
ErrorBoundary.tsx:27 TypeError: Cannot read properties of null (reading 'nextSibling')
    at index.jsx:4:3
    at Home (index.jsx:8:15)
    at @solid-refresh:10:42
    at untrack (dev.js:421:12)
    at Object.fn (@solid-refresh:10:28)
    at runComputation (dev.js:693:22)
    at updateComputation (dev.js:676:3)
    at createMemo (dev.js:238:10)
    at hmrCompWrapper (@solid-refresh:7:20)
    at dev.js:1351:17

Reproducible with versions:
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>

Heyy this should be fixed now! It was a vite change that broke some of our stuff, but we dealt with this. Can you try<EMAIL_ADDRESS>to confirm is the issue still persists.

works now. thank you @nksaraf!


@conda-forge-admin, please add user @timkpaine

Additional comment:
useful for maintaining pylint

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added user @timkpaine in conda-forge/astroid-feedstock#109.


Random Windows node stops working with "NetworkPlugin cni failed to set up pod ..." error

Is this a request for help?: yes

Is this an ISSUE or FEATURE REQUEST? (choose one): ISSUE

What version of acs-engine?:  v0.25.3

Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm)
Kubernetes, Windows nodes 1709
What happened:
Nodes randomly start failing in a few days of use. New pods stop being created on a particular node, with the error being Failed create pod sandbox: rpc error: code = Unknown desc = NetworkPlugin cni failed to set up pod "my-pod-648b47bcc-hj8tm_t-1ea5c65c-0824-4d63-8f5f-f8e820322e28" network: netplugin failed but error parsing its diagnostic message "": unexpected end of JSON input
Same deployment succeeds on other nodes.
kubectl get nodes returns node status as "Ready".
Restarting the node doesn't help. Recreating the cluster fixes the issue for a few days, then a random node goes bad again
What you expected to happen:
Nodes should be stable
How to reproduce it (as minimally and precisely as possible):
Create a cluster with a few Windows nodes, use it with a few pods. One of the nodes will stop creating new pods in a few days.
Anything else we need to know:
The node shows no resource stress.

Try #4046

@zhiweiv, there are 4 or 5 different suggestions given at #4046. Which one?

Stop-Service Kubelet -Force
<del azure-vnet.json file, if exist>
<del azure-vnet-ipam.json file, if exist>
delete the lock files as well.
Start-Service kubelet

I got same issue again today, the workaround worked for me.

Yes, the fix also worked in my case. The node hadn't crashed before the problem appeared though.


Added SupervisedTimeSeriesKMeans(TimeSeriesKMeans)

Hi Romain,
I wrote this Wrapper for TimeSeriesKMeans for the (very simple) supervised case of TimeSeriesKMeans and wanted to share.
I moved some of its assignments to SupervisedTimeSeriesKMeans.fit(X, y) to make the Estimator more compliant to the scikit-learn guidelines (it "almost" passes sklearn.utils.estimator_checks.check_estimator()).
I didn't want to write tests yet, as you seem to migrate to pytest currently.
Let me know what you think!

Hi, do you have a reference for this method? I had never heard of it before.

Actually, I don't..
My naive understanding of what supervised k-means is supposed to mean is this:
When we have class labels available, we don't have to look for "good" centroids, that yield "good" clusters. We can simply take the clusters that the class labels give us and calculate the centroids as the mean/barycenter/.. of each class.
Prediction is done as usual.
Maybe some reference that this works (better than just forgetting about the labels) would be good, I agree.

Then, if this does not correspond to some published method, I would suggest we do not merge this pull request but, if you are interested, you could have a pull request to tslearn-addons as the requirements for integration there are less strict. Let me know if this option suits you.
Best,
Romain

Sure, that sounds good! I just wanted to make sure you can use my code if you want to.
Should i create an empty file there with my class that imports from tslearn.clustering?

Yep!


Scene-manipulation actions don't work if the mixer isn't set to use MIDI channel 1

I've been fooling around in module code (mostly refactoring it to make it understandable to me) and happened to run across this bug.  I have no idea if/when my fooling around will be complete enough to land (let alone whether it would be accepted), so I might as well save people from this bug before then.

I’m ok with any refactoring that makes it more attainable because it’s awful right now. And the original author has abandoned it. I’d like to get this in great working order to then possibly apply the structure to the QU.

Sounds good.  I'll start getting them out the door once this is in -- starting with "autoformat all the code and enforce it on commit", which obviously is an implicit dependency of everything else.


Add X-Chain to C-Chain atomic swap steps to the tutorial, change C-Chain docs to avax RPC, update getCurrent/PendingValidators call.

Update Atomic Swap tutorial to include X-Chain to C-Chain transfers and C-Chain to X-Chain transfers.
Update C-Chain RPC endpoints from ava to avax.
Update getCurrent/PendingValidators call with correct response data.

tyvm @cgcardona


Move Excel-ELT metadata into database

Currently, the metadata for the Excel-ELT process is written to and read from a file on the users (client) machine. This information should be written to (and used from) the Materials Commons backend database.

Done.


Add Multiclass and GP machines to generated serialization tests

MulticlassLabels: algorithms set labels by set_int_label() that sets only m_labels but this way
m_current_values is not initialized and hence equals() fails. [not sure how to handle this, zeroed vectors on allocation in DenseLabels for now].
MCLDA: incomplete params registration [fixed].
QDA: incomplete params registration + SGNDArray not serialized [fixed].
MulticlassLibSVM, ScatterSVM: Error: C <= 0, in fact it's set to zero few lines above [??, ignored].
MulticlassTreeGuidedLogisticRegression, CDomainAdaptationMulticlassLibLinear: need custom initialization [ignored].
ShareBoost: apply takes features subset [ignored].
Test store_model_features true/false in kernel machines
Use Some instead of raw pointers in macros
Add SGMatrix::get_submatrix() method
@karlnapf Don't think this was what you had in mind for get_submatrix so I wait your review, GP is just initial stage, I'd need some guidance on what we want to test.

@karlnapf I want to add also the preprocessors, I think that the interesting ones are the dimension reduction preprocessors (PCA, KernelPCA, FisherLDA, missed something?), but their initializations differ so they'd need some factory method...thoughts?

Yeah good idea.
I think a first step here should be to add tests that run preprocessors with all the models (no serialization yet), that might not work ;) so maybe one after the other?

ok, so first I'll do GPs in a separate PR since it'll need some iteration to make it modular

@karlnapf when you get a chance please review the changes to multiclass, I should have addressed your comments.

cleaned up, should be mergeable


Fix TypeCollection::export_ts

It was removed in the repo refactor

This was solved by the new Language trait in https://github.com/oscartbeaumont/specta/commit/35846dfe043b4a9b192cd533d715bb27b3fbb431


[CWS] add missing defaultEventMonitorAddress on darwin

What does this PR do?
The default value was defined for linux and windows, but not darwin.
Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Test code: how to download data for a bounding box

Please take a look at the below Python code and ask any questions or make suggestions to improve the code. This is not our real code, but we will want to make a function that appears like:
get_data(bbox,type,filter)
In this case bbox is the geographic bounding area, type is the API endpoint (points, traffic signs, images, sequences), and filter would likely want to use a format with an operator (>=, ==, <, contains, etc) for the properties of the returned data, for example a map feature in the example code matching the value for a street light, but we could filter also by date > than a given input.
We also want to consider a function that would save the data to file, maybe like saveGeoJSON(path).
import mercantile, mapbox_vector_tile, requests, json
from vt2geojson.tools import vt_bytes_to_geojson

# define an empty geojson as output
output= { "type": "FeatureCollection", "features": [] }

# vector tile endpoints -- change this in the API request to reference the correct endpoint
tile_points = 'mly_map_feature_point'
tile_traffic_signs = 'mly_map_feature_traffic_sign'

# Mapillary access token -- user should provide their own
access_token = 'MLY|XXX'

# a bounding box in [east_lng,_south_lat,west_lng,north_lat] format
east, south, west, north = [-80.13423442840576,25.77376933762778,-80.1264238357544,25.788608487732198]

# list of values to filter for and keep -- update this if changing to traffic signs
filter_values = ['object--support--utility-pole','object--street-light']

# get the tiles with x and y coordinates which intersect our bounding box, MUST be at zoom level 14 where the data is available
tiles = list(mercantile.tiles(east, south, west, north, 14))

# loop through all tiles to get IDs of Mapillary data
for tile in tiles:
    tile_url = 'https://tiles.mapillary.com/maps/vtp/{}/2/{}/{}/{}?access_token={}'.format(tile_points,tile.z,tile.x,tile.y,access_token)
    response = requests.get(tile_url)
    data = vt_bytes_to_geojson(response.content, tile.x, tile.y, tile.z)

    # filter for only features matching the values in filter list above
    filtered_data = [feature for feature in data['features'] if feature['properties']['value'] in filter_values]

    # check if features are inside bbox, and push to output geojson object if yes
    for feature in filtered_data:
        if (feature['geometry']['coordinates'][0] > east and feature['geometry']['coordinates'][0] < west)\
        and (feature['geometry']['coordinates'][1] > south and feature['geometry']['coordinates'][1] < north):
            output['features'].append(feature)

# save a local geojson with the filtered data
with open('mydata.geojson', 'w') as f:
    json.dump(output, f)

Closing.


Implement an API to add migration functions and a checker for migration functions

Description

[ ] Implement Config.add_migration_function API

Config.add_migration_function(
    source_version="1.0",
    target_version="2.0",
    function=...,
    auto=False,
)

to add migration functions to the Config singleton.

[ ] Implement a MigrationSection to represent a migration function in the config.
[ ] Ignore MigrationSection on _ConfigComparator
[ ] Implement a MigrationFunctionChecker to check if there is any migration function has been assigned for a specific "source_version"→"target_version" migration when auto is False.

I would not implement the auto=True/False for now.
I believe we can have a first version without the automatic migration.
ANd once we are ready, we will open the auto parameter. What do you think ?

Agree. The auto parameter will be added with the automatic migration feature later.