Cannot install on Win 10

I get this error trying to install you project. have you removed this from npm registry?
C:\Users\pmeni_000\Documents\twilio\sms-cli>npm install -g sms-cli
npm ERR! Windows_NT 10.0.14393
npm ERR! argv "C:\\Program Files (x86)\\nodejs\\node.exe" "C:\\Program Files (x86)\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "install" "-g" "sms-cli"
npm ERR! node v6.9.4
npm ERR! npm  v3.10.10
npm ERR! code E404

npm ERR! 404 Registry returned 404 for GET on https://registry.npmjs.org/sms-cli
npm ERR! 404
npm ERR! 404  'sms-cli' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\pmeni_000\Documents\twilio\sms-cli\npm-debug.log

I'm sorry that the docs are somewhat misleading. I haven't published this project to npm yet, you can only install it by npm install sms-cli@derhuerst/sms-cli for now. Will publish it.

Published<EMAIL_ADDRESS>

Thanks for prompted response. Now there is a problem with<EMAIL_ADDRESS>I tried 0.1.0 and 0.1 but these give same issue
C:\Users\pmeni_000\Documents\twilio\sms-cli>npm install -g<EMAIL_ADDRESS>npm ERR! Windows_NT 10.0.14393
npm ERR! argv "C:\Program Files (x86)\nodejs\node.exe" "C:\Program Files (x86)\nodejs\node_modules\npm\bin\npm-cli.js" "install" "-g"<EMAIL_ADDRESS>npm ERR! node v6.9.4
npm ERR! npm  v3.10.10
npm ERR! code ETARGET
npm ERR! notarget No compatible version found<EMAIL_ADDRESS>npm ERR! notarget Valid install targets:
npm ERR! notarget 0.1.0
npm ERR! notarget
npm ERR! notarget This is most likely not a problem with npm itself.
npm ERR! notarget In most cases you or one of your dependencies are requesting
npm ERR! notarget a package version that doesn't exist.
npm ERR! notarget
npm ERR! notarget It was specified as a dependency of 'sms-cli'
npm ERR! notarget
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\pmeni_000\Documents\twilio\sms-cli\npm-debug.log

<EMAIL_ADDRESS>should work. I haven't published<EMAIL_ADDRESS>yet, will fix it.

Published<EMAIL_ADDRESS>please try again.

Getting closer but getting Error: Cannot find module 'window-size'. See below.
Is that the correct window-size as there are more than one?
FYI - your README.md says "todo" for how to run this
C:\Users\pmeni_000\Documents\twilio\sms-cli>node .
C:\Users\pmeni_000\Documents\twilio\sms-cli>sms help
module.js:471
throw err;
^
Error: Cannot find module 'window-size'
at Function.Module._resolveFilename (module.js:469:15)
at Function.Module._load (module.js:417:25)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (C:\Users\pmeni_000\AppData\Roaming\npm\node_modules\sms-cli\ui.js:13:27)
at Module._compile (module.js:570:32)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
C:\Users\pmeni_000\Documents\twilio\sms-cli>npm install --save window-size
<EMAIL_ADDRESS>C:\Users\pmeni_000\Documents\twilio\sms-cli
`--<EMAIL_ADDRESS>C:\Users\pmeni_000\Documents\twilio\sms-cli>sms help
module.js:471
throw err;
^
Error: Cannot find module 'window-size'
at Function.Module._resolveFilename (module.js:469:15)
at Function.Module._load (module.js:417:25)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (C:\Users\pmeni_000\AppData\Roaming\npm\node_modules\sms-cli\ui.js:13:27)
at Module._compile (module.js:570:32)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)

I guess there will be another thing broken now. :P Published<EMAIL_ADDRESS>please try once more.

We are getting there. Now F-Fetch and D-Delete work, but R-Respond and C-Compose do not and give the following error:
F to fetch | ctrl+D to leave | C to compose
(node:49456) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): TypeError: Cannot read property 'Symbol(Symbol.iterator)' of undefined
The way I run this is (after running install):
c:\twilio\sms-cli>node .
c:\twilio\sms-cli>sms

Figured it out - in ui.js query: you have the following line:
const cmd = (process.env.EDITOR || 'nano')
In windows I do not have 'nano' so I did the following:
c:\twilio\sms-cli>set EDITOR=nodepad.exe
and then re-ran the program and now it woks for C and for R

Just to recap for Windows:
Installing
cmd
npm install -g<EMAIL_ADDRESS>Usage
#load editor and twilio account information to environment
#env.bat contains the following
#set TWILIO_SID=ACxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
#set TWILIO_TOKEN=29yyyyyyyyyyyyyyyyyyyyyyyyyyyyyy
#set TWILIO_NUMBER=+1aaafffnnnn
#set EDITOR=nodepad.exe
env.bat
#run index.js
node .
#run sms help (does not say much)
sms -help
#run sms system
sms
In notepad type message (or number), save and exit

I'm willing to make this work on Windows. But I don't have a Windows machine to test on.
If i call notepad.exe from the command line, will the command run until I close the window? Also, will I be able to pass a file to edit?

I already got this working. I put notes on githup. Thanks for all your help
On Feb 16, 2017 2:39 PM, "Jannis Redmann"<EMAIL_ADDRESS>wrote:

I'm willing to make this work on Windows. But I don't have a Windows
machine to test on.
If i call notepad.exe from the command line, will the command run until I
close the window? Also, will I be able to pass a file to edit?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/derhuerst/sms-cli/issues/1#issuecomment-280436134,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVVeSRxsR5ly8T50iOgfdHmTwbKNtndIks5rdKYDgaJpZM4L-uwy
.

Can you confirm it's working when installing from master?
# either
npm i -g sms-cli@derhuerst/sms-cli
sms

# or
git clone https://github.com/derhuerst/sms-cli.git
cd sms-cli
npm install
./bin.js

I published<EMAIL_ADDRESS>to npm.


Setting up firewall on storage account level is not relevant.

Currently, the firewall settings for the storage account changed, and below does not apply anymore:
Allow trusted Microsoft services
If you have configured your Storage Account to allow access from selected networks, you need to add an exception to allow Azure Monitor to write to the account. From Firewalls and virtual networks for your storage account, select Allow trusted Microsoft services to access this storage account
what's more on the list there is no permission to allow access for log analytics/azure monitor as described below:
https://docs.microsoft.com/en-us/azure/storage/common/storage-network-security?tabs=azure-portal#trusted-access-for-resources-registered-in-your-subscription
I am trying to set it up in east us 2

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

ID: 5daf3365-76b8-b98f-7fc8-51f520e6276b
Version Independent ID: 93a216a6-fd66-cc0a-9346-58955bd0a7ba
Content: Log Analytics workspace data export in Azure Monitor (preview) - Azure Monitor
Content Source: articles/azure-monitor/logs/logs-data-export.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @yossi-y
Microsoft Alias: yossiy

You are correct, experience has changes in storage and article should be updated as well.
Can you look here?
Is this settings helpful?

@docloud-eu
Thanks for your feedback! We will investigate and update as appropriate.

@yossi-y - Thank you for providing your insights.
@docloud-eu  - I hope @yossi-y has answered your question.
Like Yossi has mentioned, the doc will be updated as it is prioritized, we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

Just updating that the export article was updated with the new storage experience.
Thanks,


[test] workflow with fork

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Close because done testing


Incorrect Lovelace/Ada amount sent alongside NFT

I think that I have found a bug in the Nami wallet. I am participating in an NFT project that requires an interactive proof for registering an NFT for the game mechanics. The procedure requires that I form a transaction whereby I send the NFT and a specific amount of Lovelace to myself, and the transaction is subsequently queried. The process fails as of recently because Nami appears to increase by 1 Lovelace the amount that is sent in the transaction alongside the native Cardano asset. I am providing screenshots of the transaction as seen in my Nami wallet and of the related Discord post in the_ Champions Block _bot-based registration process. They channel is at https://championsblock.io/ .

Hi. Do you know which version (indicated as a tag herehttps://github.com/berry-pool/nami/tags) you are using?

3.2.5

Looks like that one is the latest. https://github.com/berry-pool/nami/releases/tag/3.2.5

Talk to Cardano Support on the Live support page so that your issue can be looked into.


⚠ Contemporanea B2B has degraded performance

In 191b8b3, Contemporanea B2B ($SITE_CNTMB) experienced degraded performance:

HTTP code: 200
Response time: 5292 ms

Resolved: Contemporanea B2B performance has improved in 673e2ae.


Extend Module to allow "css-class per validator"

Began working on the API around the ValidationService
it's just a rough bit of code for now - to get your thoughts/input on the API (ValidationService) structure, once that is decided/talked through then I can implement all the sensible checks around "addCustomValidator" etc
Send me questions/improvements and I'll do my best to get back to you ASAP 😄

Sorry, was busy for a few months. I'll take a look at it soon.

@ry8806 Unfortunately it looks as though you will need pull the latest from master into your branch and resolve conflicts before this can me merged. More than  likely due to my changes which introduced a new injected parameter into the validation service constructor/function. :smile:

@ShaneYu I created a temp branch from the commit when @ry8806 forked it and merged into that branch. To get the changes @ry8806 actually made, I first had to get the clear picture of actual changes so I removed the noise from the PR by commit https://github.com/sagrawal14/bootstrap-angular-validation/commit/67cddd3983f5e16fb4a214b6e52bc98b48f0a163 (by changing the indentation) and fixed up some JSHint errors https://github.com/sagrawal14/bootstrap-angular-validation/commit/cd7021bd58cc3cfecfc9da704c6f90e404f627b5
Now, I can see the actual changes made by @ry8806 in here master...1.0.4-ry8806
I now have to sqash all the commits to let it rebase with master.


Drop ok, err, some (both kinds), none from language

Alan v0.2 doesn't really need these functions for the optional and fallible types. You can just Maybe(val) to be equivalent to some(val), Maybe{myType}(void) to be equivalent to none (a bit more awkward, type inference should eventually support Maybe(void) for none), Fallible(val) for ok(val) and Fallible{myType}(errorVal) for err(errorVal) (again, type inference may help here). And the Maybe and Fallible "functions" could be bound to ? and ! postfix operators to turn that into just val?, void? (assuming inference works as desired), val!, and errorVal! (again assuming inference works as desired) which is even shorter.
At the same time, many languages have a different meaning for some, as an array method for determining if any of the elements of the array passes the provided check function. Alan since v0.1 includes a has function that accepts a value directly and returns true if that value is present in the array. This is like a shorthand for a check function that performs an equality check on the value, so I decided to expand has to also accept check functions, which therefore made it equivalent to some in those languages.
Since this language is aimed at both crowds, completely eliminating these terms from the language may be initially confusing, but should hopefully be less confusing than having a known term with a different meaning.

The Windows machine decided to install updates (probably) and took itself out, while I accidentally screwed up my ssh access just before going on a long trip, so... I guess I will have to spot-check Windows every so often in a VM?


Add 'char' and 'byte' data type support

Purpose
Add 'char' and 'byte' data type support at ballerina language level.
Resolves #5235
Goals
This will add language level support for 'char' and 'byte' datatypes. This also provides support for byte[] and char[] array types.
Approach
Both char and byte can be represented as int values internally at BVM/CPU level. So by using the integer register space already defined for boolean type at BVM/CPU level, char and byte was added. So at any time, the values of char or byte will be kept as integer at VM/CPU level and when the relevant value is required (such as for casting and conversions or printing, etc), the integer value will be converted to the require type.
Due to the above reason, most of the instruction code defined for boolean could be used for char and byte operations (such as move, gload, gstore, fieldload, fieldstore, etc) as-well. And at CPU level, the same integer registers were used for boolean, char and byte type and their values.
But for array types, I had to introduce different operation codes and process them separately at CPU level keeping performance in mind. Ballerina char[] uses Java int[] and Ballerina byte[] uses Java byte[] underneath.
Documentation
Some of the supported operations as test cases for byte and char are as follows.
char globalChar;
byte globalByte;

function testCharValue() (char){
    char a;
    a = 'D';
    return a;
}

function testCharParam(char b) (char) {
    char a;
    a = b;
    return a;
}

function testGlobalChar(char b) (char){
    globalChar = b;
    return globalChar;
}

function testIntToCharCast(int b) (char){
    char a = b;
    return a;
}

function testCharToIntCast(char a) (int) {
    int b = a;
    return b;
}

function testCharToFloatCast(char a) (float) {
    float b = a;
    return b;
}

function testCharToAnyCast(char i) (any, char){
    char a = i;
    type typeOfa = (typeof a);
    return typeOfa, a;
}

function testCharArrayToAny(char[] caIn) (any, any) {
    any caOut = caIn;
    any typeOfca = (typeof caIn);
    return typeOfca, caOut;
}

function testIntToCharExplicitCast(int b) (char) {
    char a = (char)b;
    return a;
}

function testIntToCharConversion(int b) (char){
    var c = <char>b;
    return c;
}

function testCharToIntConversion(char b) (int){
    char c = <int>b;
    return c;
}

function testFloatToCharConversion(float f) (char) {
    var c = <char>f;
    return c;
}

function testCharToFloatConversion(char f) (float) {
    var c = <float>f;
    return c;
}

function testCharArray() (char[]){
    char[] ca = ['w', 'T', '4'];
    return ca;
}

function testCharArrayAssignment(char[] cArrayIn) (char[]) {
    char[] cArray;
    cArray = cArrayIn;
    return cArray;
}

function testCharArrayZeroLength() (int) {
    char[] a = [];
    return (lengthof a);
}

function testCharArrayLength() (int) {
    char[] a = ['d','v','f','k'];
    return (lengthof a);
}

function testCharArrayIncreaseSize() (int){
    char[] a = ['d','v','f','k'];
    a[9] = 'e';
    return (lengthof a);
}

function testCharArrayOfArray() (int, int) {
    char[][] aa = [['w', 'd', 'f', 'r'], ['h', 't', 'y'], ['u', 'j', 'l']];
    int a = (lengthof aa);
    int b = (lengthof aa[0]);
    return a, b;
}

function testCharBinaryOperation(char a, char b, char c) (boolean, boolean) {
    boolean b1 = (a == b);
    boolean b2 = (a == c);
    return b1, b2;
}

function testCharBinaryNotEqualOperation(char a, char b, char c) (boolean, boolean) {
    boolean b1 = (a != b);
    boolean b2 = (a != c);
    return b1, b2;
}

function testCharMapValues() (map){
    map m = {};
    map addrMap = {code:'S', allowed:'Y', name: "Some Name"};
    var code, _ = (char)addrMap["code"];
    addrMap.continue = 'N';
    var a = addrMap.remove("allowed");
    return addrMap;
}

struct Person {
    string name;
    int age = -1;
    Person parent;
    char gender;
}

function testCharStructFields () (string, char, string, char) {
    Person p1 = {};

    Person p2 = {name:"Jack", age:20, parent:p1, gender:'M'};

    p1.name = "Anne";
    p1.age = 35;
    p1.gender = 'F';

    return p1.name, p1.gender, p2.name, p2.gender;
}

function testWorkerWithCharVariable() {
   worker w1 {
     int a = 10;
     char b = 'E';
     a, b -> w2;
     b <- w2;
   }
   worker w2 {
     int a = 0;
     int b = 15;
     char c = 'G';
     a, c <- w1;
     c -> w1;
   }
}

function testStringToCharArray(string b) (char[]) {
    return b.toCharArray();
}

function testByteValue() (byte){
    byte a;
    a = 34;
    return a;
}

function testByteParam(byte b) (byte) {
    byte a;
    a = b;
    return a;
}

function testGlobalByte(byte b) (byte){
    globalByte = b;
    return globalByte;
}

function testIntToByteCast(int b) (byte){
    byte a = b;
    return a;
}

function testByteToIntCast(byte a) (int) {
    int b = a;
    return b;
}

function testByteToAnyCast(byte i) (any, byte){
    byte a = i;
    type typeOfa = (typeof a);
    return typeOfa, a;
}

function testByteArrayToAny(byte[] caIn) (any, any) {
    any caOut = caIn;
    any typeOfca = (typeof caIn);
    return typeOfca, caOut;
}

function testIntToByteExplicitCast(int b) (byte) {
    byte a = (byte)b;
    return a;
}

function testIntToByteConversion(int b) (byte){
    var c = <byte>b;
    return c;
}

function testByteToIntConversion(byte b) (int){
    byte c = <int>b;
    return c;
}

function testByteArray() (byte[]){
    byte[] ba = [12, 24, 7];
    return ba;
}

function testByteArrayAssignment(byte[] cArrayIn) (byte[]) {
    byte[] cArray;
    cArray = cArrayIn;
    return cArray;
}

function testByteArrayZeroLength() (int) {
    byte[] a = [];
    return (lengthof a);
}

function testByteArrayLength() (int) {
    byte[] a = [4, 6, 27, 75];
    return (lengthof a);
}

function testByteArrayIncreaseSize() (int){
    byte[] a = [4, 6, 27, 75];
    a[9] = 56;
    return (lengthof a);
}

function testByteArrayOfArray() (int, int) {
    byte[][] aa = [[4, 6, 27, 75], [3, 7, 1], [5, 32, 98]];
    int a = (lengthof aa);
    int b = (lengthof aa[0]);
    return a, b;
}

function testByteBinaryOperation(byte a, byte b, byte c) (boolean, boolean) {
    boolean b1 = (a == b);
    boolean b2 = (a == c);
    return b1, b2;
}

function testByteBinaryNotEqualOperation(byte a, byte b, byte c) (boolean, boolean) {
    boolean b1 = (a != b);
    boolean b2 = (a != c);
    return b1, b2;
}

function testWorkerWithByteVariable() {
   worker w1 {
     int a = 10;
     byte b = 3;
     a, b -> w2;
     b <- w2;
   }
   worker w2 {
     int a = 0;
     int b = 15;
     byte c = 7;
     a, c <- w1;
     c -> w1;
   }
}

Product level documentation on value types section is also updated including byte and char value types.
Automation tests

Unit tests have been added at ballerina/tests/ballerina-test/src/test/java/org/ballerinalang/test/types and the samples used for the test cases are found at ballerina/tests/ballerina-test/src/test/resources/test-src/types for the relevant types.

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Samples
Samples are added to the product level documentation as ballerina-by-examples.
Test environment
Mac OSx, JDK 1.8.0_101
Learning
It was great learning curve to understand the end to end architecture and operational model of ballerina runtime. Starting from compiler (grammar, parser, codegen) on when the program gets complied and saved as byte-code and then at BVM/CPU level when the program gets executed using the given byte-code instruction, it was a great learning experience.

We have decided not to include this feature. Thanks for the PR. Hope that it was a good learning experience on the intrinsics of the BVM.


Added QueueSizeLimit to ElasticsearchSink.

Can you pull this against the dev branch?

I checked that other PR and no it does not cover QueueSizeLimit,so my PR is still relevant.
I have tried to change the base branch for my PR from serilog:master to serilog:dev, but it added whole bunch of other commits to my PR. I don't know Git well enough to sort it out.

Is this pr dead? QueueSizeLimit is what is needed to prevent memory hogging when ElasticSearch goes down.
I can open up a new one if it :)

Branch is pretty old and no movement on it. The feature request is certainly valid and do see #119
So let's see if we can implement this.


Make it Themable

I love how simple this component is.
It would be nice if I could theme is easily, e.g. without having to override a bunch of CSS. The positioning styling could be split out from the content styling.

3 themes available now in v1.* - positioning is global though. Late to the party but hey ho!


What's the difference of datasets used in calculating closed-set setting and open-set setting's Many、Mediumn and Few-shot

@zhmiao Dear, after carefully reading the code, I found both the train and test's known type data will be used to calculate the closed-set setting and open-set setting's Many、Mediumn and Few-shot, but why the results are slightly different？ e.g. the many-shot in closed-set is 44.7 while is 44.6 in open-set， I really feel confused about it. May you help to explain it， thanks.

@jchhuang  This is because under open setting, some of the samples from the known classes were misclassified as unknown classes, and thus reduced the final numbers.


Bumped @google-cloud/spanner in pollerCore to ^7.0.0

This dependency is out of date.  Let's update it in order to bring in the latest features.

Thanks for the PR! Are there any particular benefits of introducing this new version of the Spanner client that you have in mind? In general we wouldn't look to upgrade a dependency solely because a new version is available, unless the version currently in use was extremely outdated and/or EOS, because of a security vulnerability, or there was some other relevant and desirable new functionality in the new version (partially but not only because there is a non-trivial testing effort involved).

While SemVer major version dependency updates will remain manual, Dependabot will now handle minor and patch updates. If there are any specific new features of the Spanner client libraries that would be beneficial for your use-case and/or issues solved by moving to the most recent major version, it would be great if you could please open a new issue (or reopen this one) with the details.


Ensure correct starting values for spendable income

When the game starts in round 0, the spendable income is the initial savings (or debt, for that matter) from the welfare type that has been allocated to the player.
The round income and living costs are only applied in round 1. This was wrongly inserted into the database, where the spendable income and living costs were also applied in round 0.

This error has been repaired. The facilitator app now shows just the initial savings for:

players who have not logged in
players who are in the round 0 wait screen

The player also sees this correctly (player t1p8 from above):

After player 8 above logs in, the spendable income is 40 k:

This is correct: 5k savings from round 0, 65 k income, 30 k living costs.

This is also shown to the player after logging in:

The initial conditions of the game (round 0 -> round 1 transition) look much more consistent this way.


OAuth request no longer made with a default User Agent

Expected Behavior
A request is made to the token endpoint with a User Agent
Actual Behavior
Fails to authenticate with the OAuth server (403 error)
The token request is being made without a User Agent. Without a user agent the request is rejected by our WAF. This is an issue that was fixed about a year ago in https://github.com/Kong/insomnia/issues/6425
Reproduction Steps
Existing request that worked with 9.3.1
Auth settings
OAuth 2.0
Grant Type: Resource Owner Password Credentials
Credentials - Basic Auth
Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
No response
Insomnia Version
9.3.2
What operating system are you using?
macOS
Operating System Version
Sonoma 14.5
Installation method
Upgrade
Last Known Working Insomnia version
9.3.1

What's the status of this option in your Preferences?

Unchecked
The Auth works in 9.3.1 but not 9.3.2

@ocasta fix for this will be available in the next patch release


Broke in Django 5.0?

Trying to deep dive into why, but it broke a bunch of unit tests when I upgrade to django 5.
The symptoms I see, are
 print(list(some_model.the_sorted_m2m_field.all()))
Shows the items in a random order (sometimes the right, sometimes not). But if I look at
some_model.the_sorted_m2m_field.all().query I see  ORDER BY ("the_join_table".sort_value) ASC
So something that is showing up .query is not actually showing up in the results?  Very confused..
Update 1:  My some_model has an order_by.  That is making the sql sent change from ordering by the sortedm2m to the generic table sorting..
Update 2: In my debugger, I see SortedRelatedManager try to _apply_rel_ordering, but the queryset is already ordered and it gets ignored..
Update 3: If I shove in my code .extra(order_by=[XX]) where XX is the exact code the plugin is generating, all is well and it works.

Tests fail too:
Found 50 test(s).
Creating test database for alias 'default'...
System check identified no issues (0 silenced).
.....F..................F.......F.................
======================================================================
FAIL: test_prefetch_related_sorting (sortedm2m_tests.test_field.TestCustomBaseClass.test_prefetch_related_sorting)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/django-sortedm2m/sortedm2m_tests/test_field.py", line 185, in test_prefetch_related_sorting
    self.assertEqual(get_ids(shelf.books.all()), get_ids(books))
AssertionError: Lists differ: [1, 2, 3] != [1, 3, 2]

First differing element 1:
2
3

- [1, 2, 3]
+ [1, 3, 2]

======================================================================
FAIL: test_prefetch_related_sorting (sortedm2m_tests.test_field.TestSortedManyToManyField.test_prefetch_related_sorting)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/django-sortedm2m/sortedm2m_tests/test_field.py", line 185, in test_prefetch_related_sorting
    self.assertEqual(get_ids(shelf.books.all()), get_ids(books))
AssertionError: Lists differ: [1, 2, 3] != [1, 3, 2]

First differing element 1:
2
3

- [1, 2, 3]
+ [1, 3, 2]

======================================================================
FAIL: test_prefetch_related_sorting (sortedm2m_tests.test_field.TestStringReference.test_prefetch_related_sorting)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/django-sortedm2m/sortedm2m_tests/test_field.py", line 185, in test_prefetch_related_sorting
    self.assertEqual(get_ids(shelf.books.all()), get_ids(books))
AssertionError: Lists differ: [1, 2, 3] != [1, 3, 2]

First differing element 1:
2
3

- [1, 2, 3]
+ [1, 3, 2]

----------------------------------------------------------------------
Ran 50 tests in 1.079s

FAILED (failures=3)
Destroying test database for alias 'default'...

@mgorny Any leads yet?  I spent a solid half day trying to reverse engineer it, but not seeing the change.  This is blocking my django 5 upgrade, so something I will have to get sorted eventually..

I'm sorry but I haven't been investigating it, and I don't think I will.

TLDR : Avoid using prefetch_related on SortedManyToManyFields until the problem is fixed.
I ran into the same problem and came up with a potential fix. Replacing the SortedRelatedManager.get_queryset() implementation with the following seems to solve the problem:
def get_queryset(self):
    try:
        queryset = self.instance._prefetched_objects_cache[self.prefetch_cache_name]
    except (AttributeError, KeyError):
        queryset = super().get_queryset()

    if len(queryset.query.order_by) == 0:
        queryset = self._apply_rel_ordering(queryset)

    return queryset

instead of
def get_queryset(self):
    try:
        return self.instance._prefetched_objects_cache[self.prefetch_cache_name]
    except (AttributeError, KeyError):
        queryset = super().get_queryset()
        return self._apply_rel_ordering(queryset)

Basically we make sure that the ordering is set with _apply_rel_ordering except if the query was already explicitly sorted with order_by. The early return for prefetched objects is no longer ordered correctly for some reason that I could not precisely point out.
If a maintainer could confirm that this is a valid approach I'd be happy to open a pull request. In the meantime avoinding to prefetch_related on SortedManyToManyFields solves the problem, with an obvious performance penalty.

I think, that the problem is changing method get_prefetch_queryset to get_prefetch_querysets in related_descriptors.py in Django 5.0.
So _apply_rel_ordering is not called anymore and there is not a part ORDER BY ("table1_table2_field".sort_value) ASC in generated SQL.
The workaround is to add extra(order_by=[...]) to the queryset in prefetch_related call.
Pseudo code example:
Django 4.2 version (standard and simple):
MyModel.objects.filter(...).prefetch_related('m2mfield')

Django 5.0 solution:
from django.db.models.query import Prefetch

prefetch_qs = M2MModel.objects.all().extra(order_by=['app_mymodel_m2mfield.sort_value'])
MyModel.objects.filter(...).prefetch_related(Prefetch('m2mfield', queryset=prefetch_qs)))

I don't like it so much. It is so low-level, hardcoded tables and sort_value field name. But it works.

Thank you @mokys, that seems indeed to be the source of the problem. The definitive solution for Django 5 would then be to implement get_prefetch_querysets in SortedRelatedManager. @clintonb @AlanCoding @relrod I see that you are listed as django-sortedm2m maintainers on Jazzband's website. I'd be happy to help if you give a sign of life :)

I did a very dirty copy paste
https://github.com/listing-mirror/django-sortedm2m/commit/fd9f87a49fb714b1adcfe50fb2b9e2d5dc7029a1
per @mokys
and it works great / passes all 5,000 unit & UI tests

Thank you @mokys, that seems indeed to be the source of the problem. The definitive solution for Django 5 would then be to implement get_prefetch_querysets in SortedRelatedManager. @clintonb @AlanCoding @relrod I see that you are listed as django-sortedm2m maintainers on Jazzband's website. I'd be happy to help if you give a sign of life :)

@Pamoi sorry for not responding sooner. In the future, feel free to open a pull request.


Cache data

Cache data to prevent high load on XMPP server due to many API requests.

Implemented by https://github.com/ThomasLeister/mod_serverinfo/commit/bed917ec426275ce469cbd59b3577a06450136d9


feat(climc): add SSH through EIP in server-ssh

What this PR does / why we need it:

[ ] add SSH through EIP in server-ssh

Does this PR need to be backport to the previous release branch?:
UNKNOWN

/cc @zexi
/area climc

/hold

/hold cancel
/lgtm
/approve


Add background updates

Basically just add a service worker that automatically updates the information, that way we don't have to send a request every time the user opens the popup. Plus, we could add the alerts/hazards to the service worker so that we can enable badges and notifications

Plus, if we do this first, we don't even need to do #11 , because that issue is a worse way of doing it


selectionchange event

Here is a implementation of the selectionchange event.
Specification:
https://w3c.github.io/selection-api/#selectionchange-event

I added a new GlobalEventHandlers I hope it's what you where thinking. However the spec and the real world doesn't seem to align. The selection change event is only firing on document not on element, htmlelement or window. https://jsfiddle.net/nc290odu/ but according to the spec here it should be available on https://w3c.github.io/selection-api/#dom-globaleventhandlers-onselectionchange
So not sure what we do here do we follow the spec or or do we follow the real world. The first PR was more real world the new PR is more spec world. However they don't mention element so maybe the selectionevent shouldn't be available there.


MariaDB: Views-Folder is empty

System information:

Operating system (distribution) and version: Debian Buster
DBeaver version: 21.2.3
Additional extensions: None

Connection specification:

Database name and version: 10.5.12-MariaDB-1:10.5.12+maria~buster-log
Driver name: Don't know, the default driver DBeaver loads automagically
Do you use tunnels or proxies (SSH, SOCKS, etc)? No

Describe the problem you're observing:
I created a view named "v_schluessel_historie" but it is not displayed in the Views-Node in the Database Navigator. I can find the view by issuing a SQL-command or by right-clicking on "Views" and choose "Browse from here".

Steps to reproduce, if exist:
Connect to a MariaDB.
Create any view.
Try to locate the view in the Views-Node in the Database Navigator.
Include any warning/errors/backtraces from the logs

None
By the way thank you for providing this fine tool. It's very helpful and well done.

@zongo811 it seems from your screenshot, you have a filter applied in DB Navigator. Try to clear the filter field, so the view should become visible

Silly me, thanks for the quick response.


Inconsistent sever text filename in synchronizeText

Everytime I type in an HTML document.

I get this issue whenever the Collaborate plugin is installed. Using latest Git (https://github.com/Codiad/Codiad/commit/98d992acf18ed240edd4ea317d862d4469bbed39) of both plugins and editor.


S3 buckets

Terraform for preview staging and production s3 buckets.
See README.md for more info

Can we put a line in the S3 section of the manual (https://alphagov.github.io/digitalmarketplace-manual/aws-resources.html#s3-buckets) to say which role you need to switch to for each environment, in order to view the buckets defined here? I was double checking what we had, and before I'd switched role I could view a whole bunch of old buckets (and the public bucket, which we definitely need for at least one Jenkins script) that I don't think are defined anywhere in this repo.

@katstevens

Do we need to explicitly configure ACL as part of the policy, or should we update the Cloudtrail stuff to match this?

Yeah we should update the old style definition to use computed IAM policy document data sources. It doesn't actually make any difference to the end product but the policy document style is way more readable.

Can we put a line in the S3 section of the manual (https://alphagov.github.io/digitalmarketplace-manual/aws-resources.html#s3-buckets) to say which role you need to switch to for each environment, in order to view the buckets defined here?

Sure, we'll need to keep this up to date when we change them though. I'll see if I can link to the relevant groups in creds 🤷♀

Also re the old buckets, yeah, they're in the main account, and probably shouldn't be, but they're out of scope of this PR (But we should do something about them soon.)


feat(tsconfig): support a command to show differences between the project tsconfig and this tool built-in tsconfig

e6e1e35bb0a46b5d0be08035cc4c5031ab00ada4


Feature request: Comments in markup

Add support for having inline comments in the markup eg  //TODO: Insert X here after fixing issue #y or maybe having urls to source code etc referenced in the markup for maintainability.

#comment was the one I didn't try, and it works.


Need swagger definition/better interaction examples

[Enter feedback here]
This page would be much more useful if there was a swagger schema provided as well as more examples on how to interact with the API via postman. when looking for ingestion API swagger, there seems to be only a non-graph openAPI definition lying around which is no longer documented on azure site nor seems to be functional.

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

ID: 8c0d5ed1-e4ca-a250-2a64-099e8ece9c7a
Version Independent ID: 93503a7d-e50f-fcfa-35ee-aa973697a01f
Content: Product Ingestion API for the commercial marketplace
Content Source: articles/marketplace/product-ingestion-api.md
Service: marketplace
Sub-service: partnercenter-marketplace-publisher
GitHub Login: @iqshahmicrosoft
Microsoft Alias: iqshah

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

#reassign:amhindma

#reassign:varsha-sarah

Thank you for your feedback @levdavid. Addition of better examples and tools to improve integration with the Product Ingestion API are in our future roadmap. If there are any specific questions related to the API, please feel free to reach out to me on<EMAIL_ADDRESS>

#please-close

is there an eta for the improvement @varsha-sarah ?

Hi @levdavid I don't have an exact ETA at the moment, but we are targeting for these improvements to happen by June 2023.


Fix the background scrolling effect

Describe the bug
On the app, after clicking/tapping on Connect Wallet, the overlay does not prevent the background from scrolling. This issue also occurs when the Mobile Menu is open on any mobile device.
To Reproduce
Steps to reproduce the behavior:

Go to app.indexcoop.com
Click on Connect Wallet
Scroll down to the bottom of the menu in the overlay, then continue scrolling and notice the background scrolling effect.

Expected behavior
The background scrolling effect should be disabled while an overlay is present. This provides a cleaner and more professional experience for users.
Desktop (please complete the following information):

OS: All
Browser: All
Version: All

Smartphone (please complete the following information):

Device: All
OS: All
Browser: All
Version: All

If you've worked it already, go ahead. If not, I'd hold off


babilified code fails at runtime on browserified library

All details here:
https://github.com/wnr/element-resize-detector/issues/71#issuecomment-280743474
❯❯❯ yarn list babel-core babel-preset-babili babili-webpack-plugin element-resize-detector react-sizeme                                                                                ⏎ feature/phantomjs ⬆ ✱
yarn list v0.20.3
├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>
Offending browserified code:
{"./utils":2}],2:[function(require,module,exports){
"use strict";

var utils = module.exports = {};

utils.getOption = getOption;                                   // Uncaught ReferenceError:?

function getOption(options, name, defaultValue) {
    var value = options[name];

    if((value === undefined || value === null) && defaultValue !== undefined) {
        return defaultValue;
    }

    return value;
}

Point of failure to run:
        , function(V) {
            "use strict";
            var Y = V.exports = {};
            Y.getOption = G                 // Uncaught ReferenceError: G is not defined
        }

As I state in the other issue, this ./utils appears to be injected by browserify, not by the project itself (I tried an experiment and renamed a potential conflicting method with the same results).

I just confirmed the same webpack build works with uglify.

When I try that function, I get the output as
function foo(a, b) {
    "use strict";

    var e = b.exports = {};

    e.getOption = function (f, g, h) {
        var i = f[g];

        return (void 0 === i || null === i) && void 0 !== h ? h : i;
    };
}

Do you have a minimal repro case ?

I don't have a repro case, other than to say that I was using babili-webpack-plugin on an app and narrowed it down to the element-resize-detector dist file.  The only thing curious about this file is that the library declares a getOption in src, and there are two in the dist file:

browserified getOption
library 'getOption'

They use browserify so I assume that it is injecting this additional breaking (or possibly conflicting) method into the file.
This may not be a bug in babili, but actually a bug in the browserified dist file.  Do you see anything wrong with the dist file?  Any reason it won't load at runtime.

I'm not able to reproduce this. It is possible that some of the fixes that went to master might have fixed it. Can you try it with latest master?


: TypeError: () got an unexpected keyword argument 'fan_exclusive_play'

hi ,
now it is not possible to use this mopidy extension because some api changes at website happened  few days ago , hearthis.at have now added function for exclusive content for "fan subscribers"
is 22 13:47:23 ts-hpt620quadcoretc mopidy[9450]:     return loop.run_until_complete(main) lis 22 13:47:23 ts-hpt620quadcoretc mopidy[9450]:   File "/usr/lib/python3.9/asyncio/base_events.py", line 642, in run_until_complete lis 22 13:47:23 ts-hpt620quadcoretc mopidy[9450]:     return future.result() lis 22 13:47:23 ts-hpt620quadcoretc mopidy[9450]:   File "/usr/lib/python3.9/site-packages/mopidy_hearthis/hearthis_search.py", line 119, in _get_feed_async lis 22 13:47:23 ts-hpt620quadcoretc mopidy[9450]:     return await hearthis.get_feeds( lis 22 13:47:23 ts-hpt620quadcoretc mopidy[9450]:   File "/usr/lib/python3.9/site-packages/pyhearthis/hearthis.py", line 182, in get_feeds lis 22 13:47:23 ts-hpt620quadcoretc mopidy[9450]:     return list(map(HearThis._json_to_track, cast_list(json_data))) lis 22 13:47:23 ts-hpt620quadcoretc mopidy[9450]:   File "/usr/lib/python3.9/site-packages/pyhearthis/hearthis.py", line 141, in _json_to_track lis 22 13:47:23 ts-hpt620quadcoretc mopidy[9450]:     return SingleTrack(**json_dict, user=User(**user_dict)) lis 22 13:47:23 ts-hpt620quadcoretc mopidy[9450]: TypeError: <lambda>() got an unexpected keyword argument 'fan_exclusive_play'

Hi, thank's for the info. I'll fix that.

Ok, should be fixed.

works!


LFS-743: As an API consumer, I can access the human-readable answers in addition to the answer value in the form JSON

LFS-743

Shouldn't the displayValue for file be:
0: "Vmax.txt"
1: "test 1.txt"

Free text is not included in displayedValue:

The order in value and display value arrays is not the same. Also, should content also be presented in an array?

The content ile should NOT be processed in any way. It is assumed to be opaque binary content.

retested

retested


Using docker build makes torch cuda is_available to False!

Describe the bug
I've GPU attached in my system. I tried to run Dockerfile, and expected that the image will be built with GPU support. But it isn't.
root@00921839c781:/anomalib# nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2021 NVIDIA Corporation
Built on Mon_Oct_11_21:27:02_PDT_2021
Cuda compilation tools, release 11.4, V11.4.152
Build cuda_11.4.r11.4/compiler.30521435_0

root@00921839c781:/anomalib# nvidia-smi
bash: nvidia-smi: command not found

root@00921839c781:/anomalib# python
>>> import torch
>>> torch.cuda.is_available()
>>> Flase

To Reproduce

build image: docker build -t test .
run it: docker run -it test

Expected behaviour
As it's based on nvidia-cuda, I expected it would work out of the box if GPU is attached in the system.
Hardware and Software Configuration

OS: [Ubuntu, OD]
NVIDIA Driver Version [470.57.02]
CUDA Version [e.g. 11.4]
CUDNN Version [e.g. v11.4.120]
OpenVINO Version [Optional e.g. v2021.4.2]

Additional context

https://discuss.pytorch.org/t/userwarning-cuda-initialization-the-nvidia-driver-on-your-system-is-too-old-found-version-10010/141547

@ashwinvaidya17, will you be able to have a look at this?

@innat have you tried passing --gpus=all to docker run command? You can read more here: https://openvinotoolkit.github.io/anomalib/guides/developing_on_docker.html#run-the-docker-image?

Yes, I tried that already. I tried to run this docker file in GCP ; (Architecture: x86_64, Docker version 20.10.18, build b40c2f6). GPU is attached and necessary drived is installed (auto). Is there anything else need to care manually regarding NVIDIA Container Toolkit? In your env, is it working (cuda.is_available() -> True ? inside the docker?)

That's weird. I just build a new image and it is working. From your screenshot I can see that you passed the gpu flag at the end. Try passing it before the -it flag.


Bake stuck on "Waiting for nodes to initialize"

Hello,
After creating a project inside Unity, I sent the project to bake using Azura. The Azura Bake status is stuck on waiting for Nodes to initialize on the Acoustics window.
After locating the job using the ID, I noticed that the project is active, but not progressed past 0.00% completion after 1 hour.
Any advise?

Sounds like there was a problem with initializing the nodes in the pool. Can you look at the pool on your Batch account and see what the node status is?
On the bake tab inside Unity, can you share the toolset version that you used?

Hello,
After doing some research, I have concluded that I need to request a certain "quota" from Azure. According to the Project Acoustics help page - "By default, Azure Batch accounts are allocated a small number of cores, on a VM-size not supported by the Project Acoustics plugins. You will need to request a quota increase for at least one of the Fsv2- or H-series compute nodes"
I am pretty certain this is the issue.
Many thanks

From: Mike Chemistruck @.>
Sent: 14 September 2021 17:18
To: microsoft/ProjectAcoustics @.>
Cc: Danreeves12 @.>; Author @.>
Subject: Re: [microsoft/ProjectAcoustics] Bake stuck on "Waiting for nodes to initialize" (#72)
Sounds like there was a problem with initializing the nodes in the pool. Can you look at the pool on your Batch account and see what the node status is?
On the bake tab inside Unity, can you share the toolset version that you used?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/ProjectAcoustics/issues/72#issuecomment-919305533, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVL55YUVZBBE57ZVJCGS5QLUB5YUPANCNFSM5EAKBQFA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I agree, insufficient quota can cause this. Let us know if you have any further issues.

I appreciate the help.
How many should I request? I blindly requested 5.
Cheers

From: Mike Chemistruck @.>
Sent: 14 September 2021 17:35
To: microsoft/ProjectAcoustics @.>
Cc: Danreeves12 @.>; Author @.>
Subject: Re: [microsoft/ProjectAcoustics] Bake stuck on "Waiting for nodes to initialize" (#72)
I agree, insufficient quota can cause this. Let us know if you have any further issues.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/ProjectAcoustics/issues/72#issuecomment-919318838, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVL55YQSANEJAXOVWMATXS3UB52T5ANCNFSM5EAKBQFA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Depends on how big your scenes are and how fast you want the bakes back. I believe you request quota in terms of number of cores, so 5 wouldn't get you even a single F8 instance. If you wanted 5 nodes, you'd need to request 40 cores.

Thank you again. I've sent in another request for 40.
Really appreciate your time. I'll message tomorrow with an update.
Cheers

From: Mike Chemistruck @.>
Sent: 14 September 2021 17:42
To: microsoft/ProjectAcoustics @.>
Cc: Danreeves12 @.>; Author @.>
Subject: Re: [microsoft/ProjectAcoustics] Bake stuck on "Waiting for nodes to initialize" (#72)
Depends on how big your scenes are and how fast you want the bakes back. I believe you request quota in terms of number of cores, so 5 wouldn't get you even a single F8 instance. If you wanted 5 nodes, you'd need to request 40 cores.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/ProjectAcoustics/issues/72#issuecomment-919324500, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVL55YQVHO6KFYMYLX6PKGLUB53ORANCNFSM5EAKBQFA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Graylog additional header

OVH hosted Graylog instance (https://www.ovh.com/fr/data-platforms/logs/) works on authentication. The TCP payload must include a specific string X-OVH-TOKEN: authentication (could be a header too).
README.md reads This script supports a special syntax to send additional properties; when you log a JSON encoded object in the Message field it Unmarshalls it for you but I don't see how to make a use of this to solve above issue.

I don't mind helping you implement this feature - but I need more specific technical documentation. The GELF format uses gzip compressed messages over udp - there is no notion of a 'header'.

OVH expects JSON object within packets to contains a specific key (_X-OVH-TOKEN) whose value is a secret write token (an uuid).

So, did you try what happens when you log a message with that token? It might already work fine:
$ echo '{"_X-OVH-TOKEN": "TOKEN", "Message": "Test"}' | logger

Yes, but only for specific entries instead of all entries selected by journalctl.
But I end-up considering logging from PHP instead of systemd itself because of the "one-newline-per-log-entry" limitation of PHP-FPM when logging over syslog.


Accept CACHE_MISS for compilation tasks on M1 mac

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


How to upgrade AMI to latest version in a controlled manner?

Description
How can the docs be improved?
I was going for the following setup:

Specify amiSelectorTerms: [{"alias": "al2023@latest"}] in EC2NodeClass
Specify expireAfter: 720h in NodePool with the hope of nodes being recycled every 30 days to restart with latest AMI version

However, later I realised this will not work as I expected, because description for "alias" field states that new AMI will cause drift, in which case, specifying expireAfter for the sake of update is pointless, since the nodes will be recycled immediately after new AMI becomes available. The existing docs are not clear about this scenario.
My question is how to actually implement the desired scenario, and is it even possible. Updating related docs would make sense too, as I could only find out about drift after looking at CRD spec.
Even more, the description of expireAfter field suggests that it should be used for this case:

expireAfter
ExpireAfter is the duration the controller will wait
before terminating a node, measured from when the node is created. This
is useful to implement features like eventually consistent node upgrade,
memory leak protection, and disruption testing.

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

That's a good callout on the expireAfter docs, I'm going to guess that wording was from a pre-drift. This is no longer a recommended way to cycle nodes for AMI upgrades, this should be orchestrated by Drift. amiSelectorTerms is specified under Special Cases on Drift in the disruption docs; values in amiSelectorTerms are dynamically resolved and if a new AMI is resolved for a term the node will be drifted. This applies to alias in the same way it applies to other terms (e.g. tags).

With v1, drift can't be disabled.
We tag new AMIs for Karpenter to use, let existing nodes naturally consolidate/refresh/expire and bring up new nodes with the newer AMI.
It was a gradual rollout and we felt safer than replacing all nodes at once.
@jmdeal is there a way to keep the above/previous workflow?

Since you found a solution I'm going to close this issue, feel free to reopen if needed.


Update yq syntax and fix saas file location

The image used for yq was change to be the golang version which has a completely different syntax to the original python version. The entry point for the container is also not correct hence needing the extra yq.
The path to discover what version of the operator promoted to production has changed in app-interface, this has been updated.

/lgtm
/approve


Improve log stack trace

Stack trace does not properly print commands in many cases.

Fixed as of https://github.com/MrDrMcCoy/bblib/commit/d4fe1b9d26656d56b5fd8f31ecac7d38f75c1e40#diff-954d98dd7ab494755a551066b5ce3638
With help from https://stackoverflow.com/questions/49505696 .


Beautify some code examples in README

Type: docs
Just beautify some of the code examples in README

Even a Pandemic has passed since I opened this PR, so let's close :)


Can the shaka player stream HLS media from a VAST file rather than the IMA SDK pass it to the native player?

Test Build Versions: I've tested on the demo Shaka player and our local version - V4.3.0.
Tested Environments:
Chrome on Windows
Firefox on Windows
Safari on Mac
Chrome on Mac
Chromium on Tizen
Chromium on WebOS
I went through the shaka player internal code relating to ads management. It looks like the shaka player has a wrapper around the IMA SDK, but the actual selection of media file and playback of ad is handled by the IMA SDK itself.
Question: Can shaka player play an HLS stream ad from the VAST response. In my testing I have found that if there is only an HLS stream entry under MediaFiles, then Chrome on Windows is unable to play it, whereas environments like Safari on iOS and chromium on WebOS are able to play it. This could be because of 2 reasons:

The IMA SDK performs validation of whether the video element can directly play the stream or not. If this validation fails, playback is not attempted.
The playback is done directly by the video element. (I believe this is the case since I see on the demo shaka player that it is assigning the video element with src=<media_url_retrieved_from_vast_xml>)
If the element cannot play it, we get the error from IMA SDK that the player does not have the capability to play linear assets.

We're basically trying to figure out if shaka player can take care of adaptive streaming of ads on all browsers.

Shaka on the main branch (the change is not in any release yet, but is scheduled for 4.11), supports VAST and VMAP playback without IMA intervention and uses by default DASH or HLS, the only limitation of this implementation is that there is no event tracking.

Okay so you are confirming that dash or HLS playback would work only on environments that natively support them right? And this is because the Ad playback is handled by the native player and not shaka player in the current versions.

No, I'm saying that what IMA does is not up to us. But in 4.11 there will be a non-IMA mode that will support this. But the way the two modes work is different.

So today, would the IMA select a MediaFile from VAST response and pass it on to the shaka player for ad playback?
In simpler words, the question is whether the shaka player would play the ad or the native player?

IMA always use the native player.

Perfect! Thanks for clarifying. 👍


Avoid using exception filters for AOT runtimes

LLVM does not currently have IR to represent filter expressions and mono relies on LLVM IR when AOTing. Removing the exception filters allow these methods to function properly in AOT and mixed (AOT+interpreter) modes for WebAssembly

@aspnet-hello  triage https://dev.azure.com/dnceng/public/_build/results?buildId=182664&view=logs

@lewing thanks!


(Azure CXP) https://github.com/MicrosoftDocs/azure-docs/issues/98251

Docs Build status updates of commit a9a8fc5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

includes/active-directory-msi-cross-tenant-cmk-overview.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

#sign-off

@rwike77 Please review and merge this PR.

@karavar
Is this PR still waiting for approval from @rwike77 or is it ready to merge?
Thanks.

#please-close


Fix incorrect return in build step scripts

Context
Previously we had wrong return statements in build step scripts. This results in some errors (see https://github.com/gradle/gradle-private/issues/1749). Now we change all these returns to exits.

Thanks @wolfs !


Categories/Tags/filter

Add the tags and filtering behaviours as soon as possible, this will not only be good in the docs, but also for the examples!

see https://github.com/Ircam-RnD/waves/issues/6 too


GStreamer capsfilter missing

Describe the bug
Gstreamer (via gst-launch) runs fine, with the exception that the capsfilter is missing:
For example:
gst-launch-1.0 alsasrc device=merged_mono !     audioconvert !    audioresample ! alsasink device=hw:1
runs just fine, while
gst-launch-1.0 alsasrc device=merged_mono !     audioconvert !    audioresample ! capsfilter caps=audio/x-raw, rate=16000, channels=1, format=S16LE ! alsasink device=hw:1
says, WARNING: erroneous pipeline: no element "capsfilter"
Also  gst-inspect does not list the filter
[moritz@monix:~]$ gst-inspect-1.0  | grep caps
autoconvert:  autoconvert: Select converter based on caps
autoconvert:  autovideoconvert: Select color space converter based on caps
debug:  capssetter: CapsSetter

Steps To Reproduce
Steps to reproduce the behavior:

Install gstreamer via environment.systemPackages (gst_all_1.gstreamer gst_all_1.gst-plugins-base gst_all_1.gst-plugins-good gst_all_1.gst-plugins-ugly  gst_all_1.gst-plugins-bad)
Run gstreamer (gst-launch and gst-inspect) as indicated above

Expected behavior
The plugin should be available and functional
Additional context
Tested this on aarch64 platform (Raspberry Pi 4) and on ordinary x86_64
Notify maintainers
@ttuegel @matthewbauer
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"aarch64-linux"`
 - host os: `Linux 5.15.32, NixOS, 22.05 (Quokka), 22.05pre372961.e10da1c7f54`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.8.0`
 - channels(moritz): `""`
 - channels(root): `"nixos, nixos-hardware, nixos-unstable"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

I have some more details on the case now. When installing gstreamer via configuration.nix, it somehow lacks an important plugin file:
gst-launch-1.0 (it's a wrapper script in nixos) successfully 'loads' most plugins automatically by adding  '/run/current-system/sw/lib/gstreamer-1.0' to the GST_PLUGIN_SYSTEM_PATH_1_0 environment variable.
However /run/current-system/sw/lib/gstreamer-1.0 lacks two important plugin files: libgstcoreelements.so and libgstcoretracers.so
When I install gstreamer (in addition to the system-installed gstreamer+plugins) with nix-shell -p, the environment variable GST_PLUGIN_SYSTEM_PATH_1_0 is preset in the normal bash environment and includes the path to the otherwise missing plugins.
I have no clue why these two files are not in /run/current-system/sw/lib/gstreamer-1.0 where all the other plugin files reside in. Another way to fix the issue, would be ot propagate the variable GST_PLUGIN_SYSTEM_PATH_1_0 in the user environment (again, I have no clue, why this variable is set correctly in the nix-shell environment, but not in my normal system).
Please, let me know if you have any suggestions on how to best workaround this issue.
To 'illustrate' the issue more concretely, I add a CLI session below that shows the issue pretty clearly:

[I] moritz@mopad ~> GST_DEBUG=2 gst-launch-1.0 autoaudiosrc !     audioconvert !   audioresample  !  audio/x-raw, rate=16000, channels=1, format=S16LE !         udpsink host=<IP_ADDRESS> port=11111
0:00:00.004420933 905045      0x16540f0 WARN     GST_ELEMENT_FACTORY gstelementfactory.c:701:gst_element_factory_make_with_properties: no such element factory "capsfilter"!
0:00:00.004429972 905045      0x16540f0 ERROR                default gstutils.c:2161:gst_element_link_pads_filtered: Could not make a capsfilter
0:00:00.004441782 905045      0x16540f0 ERROR           GST_PIPELINE gst/parse/grammar.y:767:gst_parse_perform_link: could not link audioresample0 to udpsink0 with caps audio/x-raw, rate=(int)16000, channels=(int)1, format=(string)S16LE
WARNING: erroneous pipeline: could not link audioresample0 to udpsink0 with caps audio/x-raw, rate=(int)16000, channels=(int)1, format=(string)S16LE
[I] moritz@mopad ~ [1]> nix-shell -p gst_all_1.gstreamer

[nix-shell:~]$ echo $GST_PLUGIN_SYSTEM_PATH_1_0
/nix/store/frafzrcldrfsxkawjbxdm7hrpszx7qp9-gstreamer-1.20.1/lib/gstreamer-1.0:/nix/store/frafzrcldrfsxkawjbxdm7hrpszx7qp9-gstreamer-1.20.1/lib/gstreamer-1.0

[nix-shell:~]$ ls /nix/store/frafzrcldrfsxkawjbxdm7hrpszx7qp9-gstreamer-1.20.1/lib/gstreamer-1.0
libgstcoreelements.so  libgstcoretracers.so

[nix-shell:~]$ GST_DEBUG=2 gst-launch-1.0 autoaudiosrc !     audioconvert !   audioresample  !  audio/x-raw, rate=16000, channels=1, format=S16LE !         udpsink host=<IP_ADDRESS> port=11111
Setting pipeline to PAUSED ...
Pipeline is live and does not need PREROLL ...
Pipeline is PREROLLED ...
Setting pipeline to PLAYING ...
New clock: GstPulseSrcClock
Redistribute latency...
0:00:00.030307234 906508      0x1de2120 WARN         audio-resampler audio-resampler.c:274:convert_taps_gint16_c: can't find exact taps
Redistribute latency...
^Chandling interrupt.
Interrupt: Stopping pipeline ...
Execution ended after 0:00:02.555674144
Setting pipeline to NULL ...
0:00:02.584285230 906508      0x1de2580 WARN                audiosrc gstaudiosrc.c:227:audioringbuffer_thread_func:<autoaudiosrc0-actual-src-puls> error reading data -1 (reason: Success), skipping segment
Freeing pipeline ...

Somehow I had this working at one point, but it seems that installing obs-studio over the weekend caused something to clear out a variable, I guess. Any insights on moving forward would be great, right now get-launch-1.0 can't find the tee element

For now, as a temporary fix, I've added a location of libgstcoreelemets.so and libgstcoretracers.so found via fd libgstcoreelements.st /nix/store | grep gstreamer-1.20.3 to the GST_PLUGIN_SYSTEM_PATH_1_0 env variable in my shell configuration (.zshrc in my case) and it's now working as expected. Feels very hacky, but it's doing the trick for anyone else who comes across this.

Another way to fix the issue, would be ot propagate the variable GST_PLUGIN_SYSTEM_PATH_1_0 in the user environment (again, I have no clue, why this variable is set correctly in the nix-shell environment, but not in my normal system).

Installing libraries system-wide is not supported: https://nixos.wiki/wiki/FAQ#I_installed_a_library_but_my_compiler_is_not_finding_it._Why.3F
While GStreamer has stable ABI, the plug-ins may depend on various libraries and if those are not compatible with a program loading the plug-in, it may cause issues.
I am not even sure why does gst-inspect-1.0 see plug-ins in NIX_PROFILES. Those should not be picked up by apps so there is no reason to have them picked up by the tool either.

However /run/current-system/sw/lib/gstreamer-1.0 lacks two important plugin files: libgstcoreelements.so and libgstcoretracers.so

gst_all_1.gstreamer has a bin output so unless you explicitly use gst_all_1.gstreamer.out in environment.systemPackages, the bin output is what will end up in the environment.
But seriously, I would recommend using nix-shell for testing and wrapping whatever programs you use so that they are always in sync with the libraries used by GStreamer.


Use envconfig to reduce the amount of boilerplate code

There is a well-supported package github.com/kelseyhightower/envconfig that does the same: assigns struct values based on env vars.
If we add it, we can remove a lot of boilerplate code.

Nice, thank you


RFC: Saturating and checking integer wrapper types

Implement two wrapper-types in std::num which provide two different types of behavior on overflow: saturating arithmetic and always checked arithmetic with the latter signalling a thread panic on overflow.
Rendered

Doesn't borrow ideas from #1530 ?

@SoniEx2 This is just the wrapper-types themselves, not how they are implemented (eg. the proposed WrappingOps trait from #1530 ). I try to avoid the issue about specific operators since that has been brought up before and would probably become the main point people would bikeshed about if it was included in the RFC.
As for traits to identify if a type is wrapping or not, that could be put in the num crate since it has other traits which allow you to be generic over different integer types.

:+1: on changing the name to avoid confusion with checked_*.

I want a Checked type which gives None if you get an overflow. This would correspond to the checked_* methods and probably give more efficient code than the Checked type proposed here anyway.

I would prefer to get an Option back too. Might I suggest making a proof-of-concept crate?

:bell: This RFC is now entering its final comment period :bell:
The @rust-lang/libs team is inclined to close this RFC. It's unfortunately been inactive for quite some time now and it seems that the desire here may not be enough to move into the standard library yet. It would likely be worth prototyping this on crates.io and seeing how it plays out there before moving.
@rust-lang/libs if you could check off your name below or write a comment below to the contrary:

[x] @alexcrichton
[ ] @aturon
[ ] @brson
[ ] @BurntSushi
[ ] @kimundi
[ ] @sfackler

Ok, the libs team has decided to close, so closing. Thanks regardless though for the RFC @m4rw3r!


Rework Animations and Character For realistic feel

I am Working on making realistic character and animations for the system
this may take time Week or two I don't know :)

I am now using animations from mixamo


Updating KE Table of contents structure, Subscriptions structure

Signed-off-by: Uma Mukkara<EMAIL_ADDRESS>
Why:

What's being changed:

Check off the following:

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


python311Packages.bellows: 0.39.0 -> 0.39.1

Diff: https://github.com/zigpy/bellows/compare/refs/tags/0.39.0...0.39.1
Changelog: https://github.com/zigpy/bellows/releases/tag/0.39.1
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 319914 run on x86_64-linux 1

18 packages built:

home-assistant-component-tests.homeassistant_green
home-assistant-component-tests.homeassistant_hardware
home-assistant-component-tests.homeassistant_sky_connect
home-assistant-component-tests.homeassistant_yellow
home-assistant-component-tests.otbr
home-assistant-component-tests.zha
python311Packages.bellows
python311Packages.bellows.dist
python311Packages.universal-silabs-flasher
python311Packages.universal-silabs-flasher.dist
python312Packages.bellows
python312Packages.bellows.dist
python312Packages.universal-silabs-flasher
python312Packages.universal-silabs-flasher.dist
python312Packages.zha
python312Packages.zha.dist
zigpy-cli
zigpy-cli.dist


Use same postgres version for docker that we have in terraware-server, use local directory for database volume

Current dependencies on/for this PR:

main

PR #2048   👈

This stack of pull requests is managed by Graphite.


Cargo doesn't know which to use because multiple target files found at..

Hi!
FROM rust AS planner
WORKDIR app
COPY . .
RUN cargo chef prepare --recipe-path recipe.json

#####################################################
#################### BUILD LAYER ####################
#####################################################

FROM rust as builder
WORKDIR /build

COPY --from=planner /app/recipe.json recipe.json
# Build dependencies - this is the caching Docker layer!
RUN cargo chef cook --release --recipe-path recipe.json

COPY . /build

RUN cargo test --lib --release

RUN cargo build --release --bin mypackage

Caused by:
  cannot infer path for `mypackage` bin
  Cargo doesn't know which to use because multiple target files found at `src/main.rs` and `src/bin/mypackage.rs`.
The command '/bin/sh -c cargo test --lib --release' returned a non-zero code: 101```

Cargo.toml
[[bin]]
name = "mypackage"

[[bin]]
name = "stub"
path = "src/stub.rs"
required-features = ["local"]

Not sure what I am doing wrong in here. The setup worked fine before using cargo chef.
Thanks

Where is the entrypoint for mypackage? src/main.rs or src/bin/mypackage.rs?

Having the same issue and the entry point is src/main.rs in my version src/bin/mypackage.rs doesn't exist in the source code and appears to purely be a skeleton artifact from the chef process.

Having the same issue and the entry point is src/main.rs in my version src/bin/mypackage.rs doesn't exist in the source code and appears to purely be a skeleton artifact from the chef process.

My setup is the same.

 Do they match?

 Do they match?

Yes they do match (mypackage). Exact the same test command used to work fine prior using cargo chef.

It should be fixed in 0.1.38!

Can confirm it works!


feat: [M3-6646] -  Strip whitespace on blur for username and email textfields

Description 📝
This change comes from a reporting of the "SOA email is not valid" validation error message on the Create Domains page (https://cloud.linode.com/domains/create) if a blank space is left in the email section after an inputted email.
The cause is that we don't currently strip extra whitespace on TextFields. For all email and username inputs, leading and trailing whitespace is stripped when a user focuses away from the input field.
Major Changes 🔄

Leading and trailing whitespace is stripped from user input in the following places:

Domains:

Create Domain email field (http://localhost:3000/domains/create)
Edit Domain drawer email field (http://http://localhost:3000/domains)
Edit SOA Record drawer email field (http://localhost:3000/domains/{id})

Account:

Add a User drawer username and email fields (http://localhost:3000/account/users)
User Profile username and email fields (http://localhost:3000/account/users/{username})
Display Settings username and email fields (http://localhost:3000/profile/display)

Managed:

Add and Edit Credentials drawer username field for Managed accounts (http://localhost:3000/managed/credentials)
Add and Edit Contacts drawer email field for Managed accounts (http://localhost:3000/managed/contacts)
Note: we recently added trimming for SSH keys (M3-6473), and that functionality should still work in this PR. (http://localhost:3000/profile/keys)

Adds a booleantrimmed prop to the SingleTextFieldForm and TextField components, which is used in the blur event handler to trim whitespace.

Preview 📷

Before
After

How to test 🧪

How to setup test environment?
How to reproduce the issue (if applicable)?

Test any/all of the textfields mentioned above and note the error messages displayed when the user enters input with a leading or trailing spaces and then focuses away from the field.

How to verify changes?

Test the username, email, and SSH key fields throughout the app -- see above for a list. When entering input with leading or trailing whitespaces and focusing away from the field, no error message should display. When the form is submitted, the action should complete successfully.
Use the MSW for Managed.

How to run unit or e2e tests?

yarn test TextField

The Domain page is deleting the whitespace for me when I leave focus. However, the Account users page is not:
https://github.com/linode/manager/assets/139489745/6ba654e2-de1b-47f1-85f5-486e943721a9

The Domain page is deleting the whitespace for me when I leave focus. However, the Account users page is not:
Screen.Recording.2023-08-04.at.9.28.19.AM.mov

However the username trim is working on the "Add a User" drawer, so it seems like its just the email.

The Domain page is deleting the whitespace for me when I leave focus. However, the Account users page is not:
Screen.Recording.2023-08-04.at.9.28.19.AM.mov

However the username trim is working on the "Add a User" drawer, so it seems like its just the email.

If you console.log the email value within the render function, it is getting trimmed properly. It might be an issue with the value in the input field not getting updated properly.

Thanks @bnussman-akamai and @tyler-akamai. In both cases, it does look like the value itself is being trimmed, but the displayed text field value doesn't reflect that. This is odd because it's happening with two email fields, but not username fields, in places where both fields are in the same form, call essentially the same function onBlur, and then exhibit different behavior. I'm taking another 👀  to see what I missed here with emails.

Thanks @bnussman-akamai and @tyler-akamai. In both cases, it does look like the value itself is being trimmed, but the displayed text field value doesn't reflect that. This is odd because it's happening with two email fields, but not username fields, in places where both fields are in the same form, call essentially the same function onBlur, and then exhibit different behavior. I'm taking another 👀 to see what I missed here with emails.

The cause of this was the type=email prop being set on some, but not all, email input fields. Browsers provide some input validation on fields with a type of email, so the value was getting trimmed (even prior to this PR), but the display value was not updating because, as far as the component was concerned, there had been no changes to the value. The kind of hacky workaround for this is to update the value to a dummy value, then change it back to the input value to trigger the rerender. Not ideal and open to other thoughts on this, but I kept type=email for the sanitization and added a check for the email type in the blur handler functions to make sure the display value gets updated and the form behavior is consistent across fields.

Awesome, glad you were able to find it!

@abailly-akamai - Thanks for the feedback. A few changes since last viewed:

Cleaned up the blur event handlers
Updated the formilTrimUtil unit test to use mock values (thank you, the expect statement was tripping me up there when I'd thought about how to mock functions)
Searched through files again and discovered the Edit SOA Record drawer and Add Credentials drawer were missing trim behavior on blur and corrected that
Partially reverted the change not to pass an onBlur handler (for username textfields, which are not validated with type=email) (https://github.com/mjac0bs/manager/commit/276cf5804d7bc0ac87a51654d56ae9da87304e49)... without the handler, form state doesn't update so although the trimming happens on the text field, the form can't submit without error. Let me know if you've got any concerns here or I've missed something.

In my testing, all listed form fields now have the same trimming behavior on blur and can submit a trimmed field without error.

Looks like I'm reviewing this after all the minor tweaks have been addressed. All appropriate fields trim the whitespace as discussed in this ticket.


[OpenAI][Archboard-Feedback] Add overload constructor in ChatMessage

Add an overload constructor,
`ChatMessage(ChatRole role, String content)
This is a more common scenario.

We have Chat[xxx]Message to differential each role now. So it is not longer needed


Start markdown

H1

I added a header named as H1 !

I named the header as H1 !

I named the header as H1 !

I created a H1 header


wtf every time i update a config

It crashes everytime i update the current config

same

I think it's related to one of my issue I opened #172

Got the same, cfg if it matters: https://pastebin.com/bHCgWqnv
Steps to reproduce:

Launch/inject f-ware in to TF2 as normal.

On main menu, edit some settings in your config (for me its consistently crashes by enabling multipoint using my config above).

Save config > Crashes game to desktop after confirming the save.

After restarting/reinjecting, the config that crashed you has been corrupted/reset to default, meaning if you didn't back up your config it's gone entirely.


igh-ethercat: init at 1.5

Description of changes
This will add the tools and kernel modules for the IgH Ethernet Master for Linux (https://etherlab.org/en/ethercat/)
# To get both the tools and the modules:

boot.extraModulePackages = [ config.boot.kernelPackages.igh-ethercat-master ];

environment.systemPackages = with pkgs; [
    config.boot.kernelPackages.igh-ethercat-master
  ];

Is this the correct way or should the tools and modules be added separately? The tooling binaries still need the correct kernel headers to build.

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

The release model of this seems to be some sort of rolling release, with 1.5.2 being the latest actual release ~9 years ago. I added a timestamp to the version number that will be updated with the commit date of the built release.

@SuperSandro2000 Do you have time to review this again?

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1723

The IgH ethercat master has now been merged via #298581