Update in external link for .NET Core support

Instead of using download link of .NET Core to show its support, better use .NET core support policy link which has additional information and made just for that.

Current: https://dotnet.microsoft.com/download/dotnet
Expected: https://dotnet.microsoft.com/platform/support/policy/dotnet-core

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

ID: 4b88d8c4-0013-70aa-7997-a2203afd9743
Version Independent ID: 0ab99d5e-4ebd-8cb9-3d27-efe58def249f
Content: Azure Application Insights for ASP.NET Core applications - Azure Monitor
Content Source: articles/azure-monitor/app/asp-net-core.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @lgayhardt
Microsoft Alias: lagayhar

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


node 10.9: new Buffer deprecation warning

with node 10.9, needle's use of new Buffer results in the following appearing on stderr:
DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead

Thanks for the heads up!


Configurable access credentials

Needs test cases proving defaultAccessKeyId and defaultSecretAccessKey and documentation in both CLI and README.

Also want to add support for using aws-sdk credentials by default as per this comment.
I just documented the new config options in the cli options, but README instructions still needed. Also still needs tests proving defaultAccessKeyId and defaultSecretAccessKey and support for default credentials supplied by aws-sdk.

hello leontastic,
sorry that I ask myself the question here, but I do not know where I ask for a feature, which is open as a pull_request, a question!
COOL feature (Leontastic: feature/configiable-access-credentials)!
I installed it in my test version, and it works well, even multiple accounts!
What I want to expand:
I'm in AWS cli with the first dummy-account, and then want to create an account with AWS CLI (aws iam create-user --user-name Bob or aws iam create-access-key --user-name BOB)!
I have already written a new fuction that creates a second dummy-account2!
now to my question:

How do I use the koa router in "lib/Router.js", so that (aws iam create-user) does not stop and a mistake 'Not found'?

My impractical example How does it work:
When I installed the new FUNCITION, for example, in "lib/controllers/bucket.js", and then with AWS cli command (aws s3api put-bucket-website --bucket my-bucket --website-configuration File : //website.js) execute!
Result: OK, a website.xml is created in the bucket, and the new function (second dummy account2 is created)!
You can also ": tagging" instead of ": website" !!!
My conclusion is, the S3Rver will continue the commands "AWS S3API" from the KoA router, the commands "AWS IAM create-user" not !!!!!
How can I expand "aws iam create-user" and best integrate into "lib/router.js", so it works?
Need help!!!


pkg/endpoint: pass denied egress identities to UpdateNetworkPolicy

Signed-off by: Ian Vernon<EMAIL_ADDRESS>

test-me-please


New package: IG.IGClient version 1.8.5

Description

Resolve https://github.com/microsoft/winget-pkgs/issues/124685

Checks

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Manual Validation ended with:

11:30:39 AM:  > 2023-11-07 11:30:39.292 [CLI ] Package hash verification failed. SHA256 in manifest
[76e21c5dc7b4d994db07ebac964ee2b407eb2d89575112f4c7e6390a8c58b410] does not match download
[316662623639ab0e895f9502f8cbe5a207b0c9094dad247a0d48851ab6fc8bcd]

Verified locally:
PS C:\ManVal> Get-FileHash .\Installers\IGClientSetup.exe

Algorithm       Hash                                                                   Path
---------       ----                                                                   ----
SHA256<PHONE_NUMBER>39AB0E895F9502F8CBE5A207B0C9094DAD247A0D48851AB6FC8BCD       C:\ManVal\Installers\IGClientSe…


WIP: Replace JSX with Thoas for Json

Proposed Changes
Please describe the big picture of your changes here to communicate to the
VerneMQ team why we should accept this pull request. If it fixes a bug or
resolves a feature request, be sure to link to that issue.
Types of Changes
What types of changes does your code introduce to this project?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes issue #XXXX)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (correction or otherwise)
[ ] Cosmetics (whitespace, styles...)
[ ] DevOps (Build scripts, pipelines...)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating
the PR. If you're unsure about any of them, don't hesitate to ask on the
mailing list. We're here to help! This is simply a reminder of what we are
going to look for before merging your code.

[ ] I have read the CODE_OF_CONDUCT.md document
[ ] All tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if needed)
[ ] Any dependent changes have been merged and published in related repositories
[ ] I have updated changelog (At the bottom of the release version)
[ ] I have squashed all my commits into one before merging

Further Comments
If this is a relatively large or complex change, kick off the discussion by
explaining why you chose the solution you did and what alternatives you
considered, etc.

oh yay! :)

@codeadict do you consider this mergeable or WIP? (apologies for not being here earlier, was thinking the PR is a draft)

@ioolkos  its not mergeable yet. I have to fix lots of Lua tests still

@ioolkos , @codeadict: I worked a little bit on this one and "made it work" for Lua. See https://github.com/mths1/vernemq/tree/json_thoas (and then vmq_diversity_json.erl)

I was not able to get lua working with iolists, so I reverted to binary
There was a small issue in

json_to_lua(?JSON_EMPTY_OBJECT) ->
?LUA_EMPTY_TABLE;
which was driving me nuts as #{} for whatever reason matches also an non empty maps...


Still shows on startup with the checkbox unchecked

🐛 Describe the bug
The "Show this page on startup"checkbox is unchecked, but it still shows on startup every time
🌴 Visual Studio Code Version : 1.56.2
🌴 Code Settings Sync Version : 3.4.1
🌴 Standard or Insiders : Standard
🌴 Portable or Installed : Installed
🌴 OSS or Official Build : Official
🌴 Operating System : Windows
🌴 Occurs On: startup
🌴 Proxy Enabled: no
🌴 Gist Id: Not relevant I think, will be added if needed
📰 To Reproduce
Steps to reproduce the behavior:

disable show page on startup
restart vscode
see the page is still shown

💪 Expected behavior
The checkbox should work as labled
📺 Console Error Log
None

I'm having the same issue, here is my info:
🌴 Visual Studio Code Version : 1.58.2
🌴 Code Settings Sync Version : 3.4.3
🌴 Standard or Insiders : Standard
🌴 Portable or Installed : Installed
🌴 OSS or Official Build : Official
🌴 Operating System : macOS 11.0
🌴 Occurs On: startup
🌴 Proxy Enabled: no
📰 To Reproduce
Steps to reproduce the behavior:
disable show page on startup
restart vscode
see the page is still shown
💪 Expected behavior
The checkbox should work as labled
📺 Console Error Log
None

I'm also encountering this issue, it has occured in all locations that I have synced by settings to including a Windows 10 OS and a PopOS (Linux) installation.
Details for the current OS on which I am encountering this issue
NAME="Pop!_OS"
VERSION="21.04"
ID=pop
ID_LIKE="ubuntu debian"
PRETTY_NAME="Pop!_OS 21.04"
VERSION_ID="21.04"
HOME_URL="https://pop.system76.com"
SUPPORT_URL="https://support.system76.com"
BUG_REPORT_URL="https://github.com/pop-os/pop/issues"
PRIVACY_POLICY_URL="https://system76.com/privacy"
VERSION_CODENAME=hirsute
UBUNTU_CODENAME=hirsute
LOGO=distributor-logo-pop-os

Other details
:palm_tree: Visual Studio Code Version : 1.60.2
:palm_tree: Code Settings Sync Version : 3.4.3
:palm_tree: Standard or Insiders : Standard
:palm_tree: Portable or Installed : Installed
:palm_tree: OSS or Official Build : Official
:palm_tree: Operating System : Pop!_OS
:palm_tree: Occurs On: startup
:palm_tree: Proxy Enabled: no
:palm_tree: Gist Id: https://gist.github.com/jeswr/121911a5a822a52a819d540286c2c233


Update PULL_REQUEST_TEMPLATE.md

We need more detailed instructions in the template....

Reference Issue

Thanks, those are better :)

Obsolete


Feature request - to add ability for status updates to the mail

Hi
It would be really handy to have in the emails the recent status updates and who logged the updates? maybe just the most recent status?
Regards
Per

Should be possible.
I'm currently investigating which API call returns the requested information.

Found the API for the alerting parts - building proof-of-concept in my development environment to see what Zabbix actually returns (aiming to get the "Alerts" information as depicted on the problems screen).
The result will be an array that can be parsed in TWIG similar to how graphs are currently parsed as well.

Concluding on issue #41 first and will start testing what information is available.
Just to be sure what you are actually looking for: the associated list of actions as for the top right screen of the Problem overview, right?

Yep many thanks that's correct it will show the the messages (if any) that was done by the engineer and the engineers name

Do you only care about the acknowledgement messages or the full list of actions as per the screendump? (including the sending of messages).

I think it would be nice to have the message that the engineer wrote so managers and other techs knows what the engineer has/is done, but only the last one a long list when the engineer has done several things might make the email a little long to read :)

I am building a development v2.16 to see what comes out the API first. Probably will pick up all the registered messages as in TWIG one can decide to show none, 1, or all. In the end you can decide yourself which ones to show or not :-).

MVP done. New values extracted from Zabbix for consumption in TWIG template (I've added some additional fields starting with _ for convenience in processing)
    "ACKNOWLEDGES": [
        {
            "acknowledgeid": 286,
            "userid": 1,
            "eventid": 864017,
            "clock":<PHONE_NUMBER>,
            "message": "Test with severity change",
            "action": 28,
            "old_severity": 3,
            "new_severity": 1,
            "username": "moudsen",
            "name": "Zabbix",
            "surname": "Administrator",
            "_clock": "2023-08-16 18:09:59",
            "_actions": "Add message, Change severity, Unacknowledge event",
            "_old_severity": "Average",
            "_new_severity": "Information"
        },
        {
            "acknowledgeid": 285,
            "userid": 1,
            "eventid": 864017,
            "clock":<PHONE_NUMBER>,
            "message": "Second text added for testing",
            "action": 4,
            "old_severity": 0,
            "new_severity": 0,
            "username": "moudsen",
            "name": "Zabbix",
            "surname": "Administrator",
            "_clock": "2023-08-16 17:43:18",
            "_actions": "Add message",
            "_old_severity": "Not classified",
            "_new_severity": "Not classified"
        },
        {
            "acknowledgeid": 284,
            "userid": 1,
            "eventid": 864017,
            "clock":<PHONE_NUMBER>,
            "message": "Test message while acking this problem event",
            "action": 6,
            "old_severity": 0,
            "new_severity": 0,
            "username": "moudsen",
            "name": "Zabbix",
            "surname": "Administrator",
            "_clock": "2023-08-16 14:59:01",
            "_actions": "Acknowledge event, Add message",
            "_old_severity": "Not classified",
            "_new_severity": "Not classified"
        }
    ],

TWIG example
{% if ACKNOWLEDGES|length > 0 %}

{% for anAck in ACKNOWLEDGES %}

                            {{ anAck._clock }}
({{ anAck.username }}, {{ anAck.name }} {{ anAck.surname }})
{{ anAck.message }}

{% endfor %}
{% endif %}

Pending verification before publication of this release in order for you to test :-).

I've concentrated on "ACKNOWLEDGE" data only, not the "ALERT" data as the Actions window in the GUI seems to combine both in the overview. This will keep the list nice/clean/short.
Let me know if this works for you - will post a screenshot shortly how the result looks like with the above template.

That looks really good many thanks, I don't have time to test tonight but I will do some testing tomorrow.

I expect to publish v2.16 later today. I do need some work on documentation as well. LOL.

v2.16 available. Documentation updates pending but the example provisioned above should get you going into the right direction. If not, let me know and I will try to assist.
Let me know if this works for you. If it does I will create the release package for v2.16.

Looking into this by firing up the previously setup environment for testing.
I'll upgrade to your Zabbix version and will install/test the new code there.
I've not changed any logic or procedures with regards to the automatic detection of events available in the system.
Is this a clean environment by the way or the same one we've used before?
The only thing I did change was adding a new flag to the event.get procedure (besides picking up the ACK messages and pushing those into the TWIG output).
The deprecation messages are definitely not from me as I'm not using mysql to connect to Zabbix at all?
Checking my own environment to see if the same occurs with me with this Zabbix version.

Verified 6.0.19 production at my end: I do have deprecation messages not on MYSQL.
("[Thu Aug 17 11:59:45.921753 2023] [proxy_fcgi:error] [pid 231914:tid<PHONE_NUMBER>00704] [xxx:47108] AH01071: Got error 'PHP message: PHP Notice:  Parameter "/user" is deprecated. in /usr/share/zabbix/include/classes/validators/CApiInputValidator.php on line 1330'")
Upgrading to 6.0.20 later today to see whether that version does show other messages and if functionality changes there that may cause mailGraph (or other) issues.

Upgraded your test previous test environment to 6.0.20 and no issues found (tested mailGraph v2.14, v2.15 and v2.15).
Are you using a clean environment for testing mailGraph as it needs at least 1 event to start processing anyway?
Or could be a rights issue when it actually is a new environment?
I did catch a code typo for detection of "acknowledges' from the event.get call in v2.16 - will post that change in a few minutes however that is not related to the issue at hand from your end.
Can you please locate an event in your Zabbix environment and specify event_id manually to verify if something else is happening?
Please replace mailGraph.php with the newer version I'm posting in the next minute first otherwise you will hit an error. Apologies for not catching that one earlier.

Just checking as this feels MariaDB or PHP related:
PHP 8.2.9 (cli) (built: Aug  3 2023 11:39:08) (NTS gcc x86_64)
Copyright (c) The PHP Group
Zend Engine v4.2.9, Copyright (c) Zend Technologies
Server version: 10.6.15-MariaDB MariaDB Server
mailGraph stops when it does not have a valid Event ID hence the message "no data received". I could have a look to make that more sensible.
Awaiting your feedback.

I've upgraded my main systems with v2.16 and all is working fine - checking on deprecation messages in the next days.
Not just why your installation is broken? Happy to assist where I can!

Can you send me your database once more via email?
I can load it up in the existing instance of last time for you. In that database the new stuff actually works just fine so curiousity level is rising at my end.

Yep sure I sent you the information on Linkedin

I've done the following on the test box:

Put your database in
Changed admin password
Verified login to Zabbix
Changed api-user password
Upgraded mailGraph to v2.16
Set config.json for my environment
Tested with eventId = 0 and with eventId = 399
All working just fine and produced expected output as well.
Validating logs of Apache, MariaDB and Zabbix-Server as a next step.
Still not sure what is happening at your end ... grrrrr.

Zero deprecation, errors, warning or anything else in logs of Zabbix, messages, HTTPD.
I seem unable to replicate your issue ...

Shared example output with you btw direct e-mail for a set eventId but when defining to zero, the code does pick up a random eventId.

Many thanks for assiting testing this.
I had to reinstall Zabbix and attach the production DB once I felt happy that things where working, then reinstall mailGraph, then specify an eventid, I really wish that i knew what caused it (it might have saved me from working yesterday) but it works now, so I can't really complain anymore :)
Regards
Per

Glad to hear you got is sorted! I can feel the frustration all the way of not having the root cause ... :-).
Does it now also work again without a specific eventId?
Last but not least: are you happy with the new feature and can we proceed to close the request as such?

Yep it works also without specifying any eventid but if I do php mailGraph.php test then I need to specify a eventid, but it works so I am happy.
Yep feel free to close it, the new feature works really well, I show cased it for the incident management team today which is really happy about the feature, so everyone is happy here, at least until after lunch :)


Correct asset and template paths if serving from a sub directory

Changes should allow users to host in a sub directory if necessary.

Thanks!


Could not find telemetry configuration file

I am getting an error when I try to load a new telemetry format on Windows.
Publisher Configuration:
{
"Hostname": "publisher",
"Cmd": [
"--pf=./pn.json",
"--tc=./telemetryconfiguration.json",
"--ll=verbose",
"--aa"
],
"HostConfig": {
"Binds": [
"C:\iiotedge\appdata"
],
"ExtraHosts": [
"ENBZW2POCDEKEP1:<IP_ADDRESS>"
]
}
}
PC Configuration:
IoT Edge Version: iotedge <IP_ADDRESS> (3fa6cbef8b7fc3c55a49a622735eb1021b8a5963)
Publisher Version: 2.3
OS: Microsoft Windows Version 1809 (OS Build 17763.379)
Error:
[08:39:44 INF] Attempting to load telemetry configuration file from: ./telemetryconfiguration.json
[08:39:44 FTL] Loading of the telemetry configuration file failed. Does the file exist and has correct syntax? Exiting...
System.IO.FileNotFoundException: Could not find file 'C:\appdata\telemetryconfiguration.json'.
File name: 'C:\appdata\telemetryconfiguration.json'
at System.IO.FileStream.ValidateFileHandle(SafeFileHandle fileHandle)
at System.IO.FileStream.CreateFileOpenHandle(FileMode mode, FileShare share, FileOptions options)
at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options)
at System.IO.File.AsyncStreamReader(String path, Encoding encoding)
at System.IO.File.InternalReadAllTextAsync(String path, Encoding encoding, CancellationToken cancellationToken)
at OpcPublisher.PublisherTelemetryConfiguration.ReadConfigAsync() in C:\app\opcpublisher\PublisherTelemetryConfiguration.cs:line 521
Note that it is looking for the file in the wrong location.  It is looking in c:\appdata when it should be looking in c:\iiotedge\appdata.   I have placed the telemetryconfiguration.json file in both locations and get the same error.

I figured out my issue.  My binding was incorrect. it should be:
c:/iiotedge:c:/appdata
I will close now.

I figured out my issue. My binding was incorrect. it should be:
c:/iiotedge:c:/appdata


Merge decoder and encoder libs, and use dynamic linking

Significance:

Consolidates both encoder and decoder libraries into one source.
Uses dynamic linking, with all the benefits that comes with it

Testing:

Successful compile
Encoder and decoder library produced the same output

Might be a good idea to keep the static encoder and decoder libraries separate, but I'm not too sure


ORB painting

ORB light painting is done by swinging a light around in circles while slowly turning around the hand holding the light. I keep thinking there might be some way to do that with this wand and a spinner handle. We'll need to turn certain lights on on each end and maybe let them change over time. Anyway, something to think about when bored!

This would need additional lights on the very end of the strip (top and bottom) as if you spin orbs and pint the stick sideways and backwards, the LEDs of the strip can not be seen anymore. also the display needs to be off, as it would show in the orb.

True, not trivial, but it was an interesting idea. Perhaps someday when we have nothing else to do!

True, not trivial, but it was an interesting idea. Perhaps someday when we have nothing else to do!

I tried to use the MIW with single leds WS2811. Worked perfectly. When I use it in lightbar mode and put only 10 led in a ball, I can made perfectly orbs. It would be nice, if you could add an additional function in lightbar mode, so that I can adjust for 2nd controller a different color, then I could made orbs with two colors.

Cool! Orbs are fun. Send me an image.
Anyway, yes, I can probably come up with some way to do that. The code will be easy, the button interface a bit trickier. I’ll have to think of some easy way to select different colors for the two parts. It might be a better idea to split the display in half with two different colors. Then it could be done with a single controller as well.
From: tobias966 @.>
Sent: Thursday, January 20, 2022 12:11 PM
To: MartinNohr/MagicImageWand @.>
Cc: MartinNohr @.>; Author @.>
Subject: Re: [MartinNohr/MagicImageWand] ORB painting (#28)
I tried to use the MIW with single leds WS2811. Worked perfectly. When I use it in lightbar mode and put only 10 led in a ball, I can made perfectly orbs. It would be nice, if you could add an additional function in lightbar mode, so that I can adjust for 2nd controller a different color, then I could made orbs with two colors.
—
Reply to this email directly, view it on GitHub https://github.com/MartinNohr/MagicImageWand/issues/28#issuecomment-1017835122 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHQ7DEHJLJUOHVH6KKQZ2DUXBM3TANCNFSM4VLL5GGQ .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AAHQ7DGBP7ZMEFFINIKFERTUXBM3TA5CNFSM4VLL5GG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHSVO44Q.gif Message ID: @.*** @.***> >


Update tests to use test ports for the Dgraph cluster.

The tests should be updated to run against the Dgraph test cluster defined in this docker-compose.yml file: https://github.com/dgraph-io/dgraph/blob/43c8d38499fb8795e6e5bb5f6a5facf665f36d64/dgraph/docker-compose.yml

[ ]  Alpha runs on gRPC port 9180.
[X] Enterprise ACLs are enabled and require groot login (this is already done).

When changing to port 9180 there's a test that fails when using the default constructor for the client stub, which defaults to port 9080:
https://github.com/dgraph-io/dgraph-js/blob/21563532863a01a639b4ddf4848e3de3de77b3e0/src/clientStub.ts#L58-L61
To fix this we should do one of two things (or both):

Remove this test that involves the default constructor
Remove setting a default address in an empty constructor (breaking the existing API).

Setting a default address when constructing a client connection doesn't happen in any of the other official clients. It's also such a small thing that it doesn't need to be tested for, and keeping it means the tests must always run with a Dgraph cluster on port 9080.

@danielmai
this change allows specifying dgraph port via an env var:
DGRAPH_SERVER_ADDR=localhost:9180 yarn test
It also removes the constructor unit test.
Please re-open if you need anything else


Selecting the first item programmatically

How do you select the first item in the spinner?
I tried doing the following,
spinner.setOnItemListeners(..){ ... };
spinner.setSelectedIndex(2);
spinner.setSelected(true);
but it doesn't seem to be working.
Thanks!

spinner.setSelectedIndex(int) should select that item. Make sure you invoke this method after setting the items in the spinner. For example, if I set the selected index to 1 in the sample, then "Jelly Bean" would be selected instead of "Ice Cream Sandwich".
MaterialSpinner spinner = (MaterialSpinner) findViewById(R.id.spinner);
spinner.setItems("Ice Cream Sandwich", "Jelly Bean", "KitKat", "Lollipop", "Marshmallow");
spinner.setSelectedIndex(1); // <--- Set the item programmatically
spinner.setOnItemSelectedListener(new MaterialSpinner.OnItemSelectedListener<String>() {

  @Override public void onItemSelected(MaterialSpinner view, int position, long id, String item) {
    Snackbar.make(view, "Clicked " + item, Snackbar.LENGTH_LONG).show();
  }
});
spinner.setOnNothingSelectedListener(new MaterialSpinner.OnNothingSelectedListener() {

  @Override public void onNothingSelected(MaterialSpinner spinner) {
    Snackbar.make(spinner, "Nothing selected", Snackbar.LENGTH_LONG).show();
  }
});

The method setSelectedIndex just chooses the item as the first item, but I would like to actually select the item as if it would call on onItemSelected.
Thanks again.

You should just run whatever code is in your listener when you select the item. I don't know why this would be needed. If you really feel like this is necessary please provide an example.

what he means is that when he use spinner.setSelectedIndex(1);,it justchoose the item to appear as the first item,but it didn't execute the selected event.
I have the same problem,please fix that to make this project better.Thanks man.very nice Spinner!


EXTRA section

What is our strategy here for the beginning when there will not be material to load?

working on it, we already have some things in place to share!


BE-137 - There are no known instances of bodies incorporated through agreement

Signed-off-by: Elisa Kendall<EMAIL_ADDRESS>Description
Eliminated unnecessary subclasses of corporation that either reflect non-existent kinds of organizations or add clutter with respect to guarantees that can be addressed simply by linking a guarantee to the instrument of incorporation, including relationships with guarantors
Fixes: #1257 / BE-137
Checklist:

[x] I'm familiar with the FIBO developer quide. My contribution meets all the requirements described there.
[x] My contribution follows the principles of best practices for FIBO.
[x] My changes have been reconciled with latest master and no merge conflicts remain.
[x] This PR is related to exactly one issue. The issue is referenced by using a GitHub keyword such as "fixes", "closes", or "resolves".
[x] Hygiene tests have been applied by a PR with "(WIP)" in title.
[x] The issue has been tested locally using a reasoner (for ontology changes).

This was merged 2 hours and 14 minutes after the PR was made. Have you totally abandoned the 24 hour review window, @trypuz?
Allowance for review is especially important in a case like this where something is being removed from Release without even being deprecated. Since it's only removal, there is no need for excessive urgency.
And if the basis is the claim that "there are no known instances" it's even more important that people be allowed to challenge the assertion. Which doesn't actually make sense given that the removed text says "An LLP (in the UK) is an example of this, and is also a partnership"
Well in the GLEIF database there are 2804 UK LLPs using the official ELF code Z0EY (there are probably others not using that), for example Bluebay Asset Management LLP https://search.gleif.org/#/record/0XNERYKNM1SJ16DXK809

@rivettp - I think the rule was 2 approvals is enough to close it, but I don't recall.
Regardless, an LLP is not a corporation, not in the UK and not in the US.  It is NOT INCORPORATED.  We do have LLPs from around the world represented as limited liability partnerships - we have not eliminated the concept.  Only this ill formed class that has been bothering some bankers for years.  Literally. The BE issue that addressed this was years old. An LLP in the UK is formed based on the "Limited Liability Partnerships Act 2000 (in Great Britain) and the Limited Liability Partnerships Act (Northern Ireland) 2002 in Northern Ireland, with the rules governing this scheme consolidated across the UK with the Companies Act 2006, the latter coming into effect in 2009".

Have you totally abandoned the 24 hour review window

To my best knowledge "24 hour review window" has never been accepted as the principle for Open FIBO. "2 approvals" rule is explicit. So, everything is OK.


Disable Clear Guides menu item when no guides exist

Clear Guides does nothing when there are no guides available.

Thanks!


Example project

It would be cool if you had an example project showing how Bartinter works

Please, what do you mean with "Set ViewController's updatesStatusBarAppearanceAutomatically = true"??????


Add an example with pictures to README

Finally, the software is in decent enough condition to produce an example


1st draft PlatformIO support

Hi Jacob!
Quite cool calendar and weather station!!!
Have build one for myself and waiting for the parts to build one for my parents.
Did a couple of small modifications, but before asking to pull them, my biggest wish is PlatformIO support.
(I feel to old for wasting my time in a slow and uncomfortable Arduino IDE ;-))
I don't wanna break down a IDE discussion here, so I simply tried to change as less as possible, and also got it compiled as well as working.
But unfortunately I'm not very comfortable with C/CPP, that's why I only did a draft pull request now, because I think my current fix is more a workaround than a solution.
During compile (via PlatformIO) I had issues that struct tm ... was not defined correctly, cause the generic time.h wasn't loaded. That's why I renamed your time.h|c into time_misc, which worked, but I've the impression that it wasn't the right way.
Do you know a better approach/solution?
Beside this, I build my portal_calender with FireBeetle ESP32 cause I've been to impatient and closefisted to get an EZSBC ESP32 to Europe. Unfortunately I haven't found any information's how EZSBC ESP32 has to be configured for PLatformIO.
That's why platformio.ini currently only has an "[env:firebeetle32]" entry.
Do you have any information's if EZSBC's  ESP32 need a separate entry?

Hey,
I've got limited experience with PIO, but I can check out the branch and see if I can get it configured for the ezSBC. There is a very old post on the ezSBC forums from someone who planned to add support to PIO for this board, but looking through PIO's repo I can't find any mention of such a PR and the board still isn't added. But it's a pretty simple board so I don't see it being too hard to configure.
The build error is probably just due to time.h conflicting with the global time.h where tm is defined. I'm not exactly sure why that didn't happen in Arduino's build system but in any case renaming the file would work. I'd probably suggest time_util over time_misc but it'll work either way.

Hi,
quite thanks for taking the time to check my pull request.
Well, my PIO experience is mainly only in that direction that I personally feel it much more comfortable as well as faster.
The ezSBC and the FireBeetle both use an ESP-WROOM-32 module. So the difference between them is mainly the PCB and the extra parts like power management. I've hope that ezSBC might also run with my [env:firebeetle32] platformio.ini compile entry. See https://docs.platformio.org/en/latest/boards/espressif32/firebeetle32.html
If you confirm that it compiles and runs well on ezSBC, I'll modify (for sure) my change request also in regard of renaming time_misc into time_util, which by the way didn't happen in Arduino's build system because PIO allegedly has a library finder/solver. This is at least what I could find out as possible reason.
There's also a "narrowing conversion" compiler warning left, I'll (try) to fix then.

Well I got it to build on the ezsbc pretty easily using the 'esp32dev' board configuration, which sounds like the same one the arduino IDE uses so that makes sense. I did have to rename the resource directory to resources to prevent PlatformIO from crashing when initializing the project though.
I also renamed time_misc to time_util. The narrowing conversion warning was in the NTP code and I just fixed it with an explicit cast to time_t.
I'm still not an expert on PlatformIO, but unless there's anything else you think is needed for this I think this PR is good to merge.

Googled around and did a couple of tests with Serial, Serial.available(), Serial.availableForWrite(), but found no way to detect when Serial(USB) console is ready.
Look like DEBUG_DELAY is necessary.
For me it doesn't matter if in platformio.ini or config.h, but as it's more a developer than a maker screw, as well as it's only relevant to PIO env. I would leave it in the ini.


Move dockerfile under OpenLiberty folder

Need to create a OpenLiberty folder under root. Place dockerfile under that folder. All other app servers can create their own folder with dockerfile in

done


Use owner ref to calculate pod statistics in federated replicaset controller

 Issue by mwielgus
Wednesday Oct 05, 2016 at 18:19 GMT
Originally opened as https://github.com/kubernetes/kubernetes/issues/34130

Right now the controller list all pods to calculate the stats.

 Comment by dims
Wednesday Nov 16, 2016 at 14:30 GMT

This needs to be triaged as a release-blocker or not for 1.5 @mwielgus

 Comment by dims
Friday Nov 18, 2016 at 13:01 GMT

@mwielgus @matchstick all issues must be labeled either release blocker or non release blocking by end of day 18 November 2016 PST. (or please move it to 1.6) cc @kubernetes/sig-cluster-federation

 Comment by madhusudancs
Saturday Nov 19, 2016 at 19:27 GMT

Not a v1.5 release blocker.

 Comment by ethernetdan
Monday Mar 13, 2017 at 22:32 GMT

Moving to 1.7 as late to happen in 1.6. Feel free to switch back if this is incorrect.

 Comment by quinton-hoole
Friday Sep 08, 2017 at 02:45 GMT

I'm not sure that this is still an issue, but leaving open until someone can confirm definitively.

 Comment by quinton-hoole
Friday Sep 08, 2017 at 02:45 GMT

@mwielgus is no longer working on this, as far as I know, so unassigning him.  Feel free to re-assign as necessary.

cc @mwielgus


Update impl. as per decisions of meeting 2023-02-10

Also update for issue #59 of  again: support option A.

see PR #41


How do I escape backticks within code snippets?

I am trying to demonstrate using string interpolation in a typescript file, which uses the backtick character to turn on the feature. I have tried using \string/${myVariable}`` but that does not work and instead shows just the first backtick in the code snippet and then the rest of my string as plain text.

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

ID: fa622358-f5bb-1f5c-177a-081a37b6acef
Version Independent ID: 72f7fd16-a822-66c8-3a34-8905645ce5f2
Content: Markdown syntax for files, widgets, and wikis in Azure DevOps - Azure DevOps
Content Source: docs/project/wiki/markdown-guidance.md
Product: devops
Technology: devops-collab
GitHub Login: @chcomley
Microsoft Alias: chcomley

Hi @biltongza, Thanks for reaching out and sorry you're experiencing an issue. I'm following up with my PM and will be in touch ASAP. Thanks!

Hi @biltongza
Please try the following and let us know if this works.

cc: @chcomley

@biltongza, I'll be closing this issue. Please let us know if the example adramasu provided isn't helpful! Thanks!


Hello, it's down again. :0

Me again, just letting you know waifu2x is down again. Wasn't sure if you knew about this issue. Thanks!

restarted


Dash / hyphen for 'ignorePunctuation', recommend "minus sign" as well in the documentation

https://github.com/julmot/mark.js/blob/9b0efaf07b869b45cfc4815faad167940143016b/src/lib/regexpcreator.js#L67
See:
https://en.wikipedia.org/wiki/Wikipedia:Hyphens_and_dashes
Note that there are several other variants, but it would be overkill, I think:
https://www.compart.com/en/unicode/category/Pd
http://jkorpela.fi/dashes.html

Maybe should be a separate issue, but could an option be added to ignore all punctuation, e.g. using \p{P}?


fix: scope name is wrong

Problem description
L36 & 78 & 129 & 169
scope naming: connectivity_insights:policy:write -> application-profiles:write (resource is optional according to https://github.com/camaraproject/Commonalities/blob/main/documentation/API-design-guidelines.md#1161-scope-naming, otherwise it would be application-profiles:application-profiles:write)
Expected behavior

Fix per guidelines (as described above)
Alternative solution

Additional context

fixed with commit https://github.com/camaraproject/ConnectivityInsights/commit/a02c36e36d3b1267a185eb61f0cb1a631e983266


When tokensvisble enabled battle status not available

When I have runing tokens Visible at version 9 I can not run a battle, cause I can not but Chars in the Battle tracker

Only happens when you select add charakters invisible to battle status

this feature should now work again in Foundry 9 as of YTV 2.7.2 Please confirm. (it works for me)


Fixed 204 returned even after setting StatusCode in ObjectResult with Null Content

Issue - #1809 @yishaigalatzer @rynowak

:watch:

:shipit:


codestream level 5/10 bit-per-channel clarification

From what I understand from the documentation, level 5 is supposed to support up to 16 bits per channel. However with current main branch the encoder will now fail with ./lib/jxl/encode.cc:645: Codestream level verification for level 5 failed: Too high modular bit depth when encoding a 16bit uint channel. From the code, it seems that the check will fail if the bit depth is higher than 12, not 16 (https://github.com/libjxl/libjxl/blob/d382e5d5abd655f78b910f72c3be6e8f2465e087/lib/jxl/encode.cc#L580)

There are two things:

the nominal precision of the image (which is a somewhat arbitrary number in case of XYB encoding, since it is not related to how precisely the data is actually stored in the bitstream, which depends on the quantization factors that are used)
modular_16_bit, which signals if the int16_t data type can be used for modular data (e.g. for lossless, but also for quantized DC coefficients), or if the int32_t data type is needed.

For lossless encoding of uint16, the int16 data type is not sufficient. In principle the cutoff could be set at 15-bit, but the current code sets it at 12-bit so the encoder will always have the freedom to use transforms like YCoCg and Squeeze that each require an extra bit.
I hope that clarifies things.
So you can have a level 5 bitstream that represents a 16-bit image but in a lossy way (such that the actual quantized coefficients still fit in int16_t), but if you want to losslessly store a uint16 image, you'll need a level 10 bitstream. Lossless uint15 and float16 are technically possible within level 5 (if no bitdepth-increasing transforms are used), but the current encoder (in order to not reduce its freedom) will start using level 10 already at uint13 and only allows level 5 up to uint12.

Hi @jonsneyers , thank you for the explanation. In the context of the strictly lossless encoder (iiuc without xyb encoding?) handling uint14 pixels, there are still 2 points I don't clearly understand:

v0.6.1 did not require explicitly setting level 10, whereas current main branch errors out in this case. Did the 0.6.1 encoder silently switch to level 10 or did it stick with level 5 with the effect of not being to losslessly encode the whole bit range?
will there be an effect on the final size of the codestream and the compression/decompression speeds when using level 10 compared to level 5 (again, in the context of uint14)

thanks,
thomas

The 0.6.1 encoder just produced a level 10 codestream that was not marked properly as being level 10 (i.e. without a level box, so implicitly and incorrectly level 5). The libjxl decoder doesn't check the level constraints so this didn't cause problems so far, but of course moving forward we want the encoder to produce files that are properly marked.
Using level 10 only has the overhead that the box format has to be used (cannot use the abbreviated codestream-only format), with a level box. I think the overhead of the box format is 40 bytes, and the level box is 9 bytes, so the total overhead of using level 10 instead of 5 is 49 bytes. Probably that shouldn't be a big issue in use cases where the data is > 12-bit.


Reloads repeatedly on page error

Bug:
When there is some errors in the web page, the page reloads repeatedly and prevents me to see the error!

@rostamiani  Can you please mention the steps to reproduce the issue?

Closing no response


[Question] How to assume IAM role inside the escalator pod? Getting 403 despite instructions

Hello guys,
It's a wonderful project and I've almost got it working. Having followed Readme instructions in (https://github.com/atlassian/escalator/blob/master/docs/deployment/aws/README.md)
I have those ticked off:

IAM role created
IAM policy created as per (https://github.com/atlassian/escalator/blob/master/docs/deployment/aws/README.md#permissions)
--aws-assume-role-arn flag passed in the command in escalator-deployment-aws.yaml as below:

      ---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: escalator
  namespace: kube-system
spec:
  replicas: 1
  selector:
    matchLabels:
      app: escalator
      role: escalator
  template:
    metadata:
    # I'm really not sure all three are required as below: https://github.com/atlassian/escalator/blob/master/docs/deployment/aws/README.md#deployment
      annotations:
        iam.amazonaws.com/role: arn:aws:iam::XXX:role/bitbucket-pipelines-escalator-role
      labels:
        app: escalator
        role: escalator
    spec:
      serviceAccountName: escalator
      containers:
      - image: atlassian/escalator
        command:
        - ./main
        - --nodegroups
        - /opt/conf/nodegroups/nodegroups_config.yaml
        - --cloud-provider
        - aws
        # this bit: https://github.com/atlassian/escalator/blob/master/docs/deployment/aws/README.md#sts-assume-role
        - --aws-assume-role-arn
        - arn:aws:iam::XXX:role/bitbucket-pipelines-escalator-role
        name: escalator
        ports:
        - containerPort: 8080
        env:
        # this bit: https://github.com/atlassian/escalator/blob/master/docs/deployment/aws/README.md#aws-credentials
        - name: AWS_ROLE_ARN
          value: arn:aws:iam::XXX:role/bitbucket-pipelines-escalator-role
        - name: POD_NAME
          valueFrom:
            fieldRef:
              apiVersion: v1
              fieldPath: metadata.name
        - name: AWS_REGION
          value: eu-west-1
        volumeMounts:
        - name: escalator-nodegroups
          mountPath: /opt/conf/nodegroups
          readOnly: true

AWS_ROLE_ARN and AWS_REGION passed into env in above deployment, as mentioned in discovery by default credential chain in (https://github.com/atlassian/escalator/blob/master/docs/deployment/aws/README.md#aws-credentials)
finally annotations:iam.amazonaws.com/role with role arn as mentioned in the kube2iam part of Deployment instructions.

Given all that I'm still getting 403 on attempt to assume role.
AccessDenied: User: arn:aws:sts::XXX:assumed-role/eksctl-bitbucketpipelines-nodegro-NodeInstanceRole-XXX is not authorized to perform: sts:AssumeRole on resource: arn:aws:iam::XXX:role/bitbucket-pipelines-escalator-role\n\tstatus code: 403

I am missing something? Is the documentation complete?
Other sources suggest  creating OIDC Provider for the cluster (https://docs.aws.amazon.com/eks/latest/userguide/enable-iam-roles-for-service-accounts.html) I've done that with eksctl and it has no impact on it's own
Is there a specific trust relationship on the IAM role required before the escalator pod can assume it please?

Any pointers would be much appreciated. Thank you!

Thanks for giving Escalator a go @FilipSwiatczak!
Based on the following error:

AccessDenied: User: arn:aws:sts::XXX:assumed-role/eksctl-bitbucketpipelines-nodegro-NodeInstanceRole-XXX is not authorized to perform: sts:AssumeRole on resource: arn:aws:iam::XXX:role/bitbucket-pipelines-escalator-role\n\tstatus code: 403

I'd say the trust relationship isn't setup correctly between the two roles to allow eksctl-bitbucketpipelines-nodegro-NodeInstanceRole-XXX to assume bitbucket-pipelines-escalator-role.
Have a look at this page on how to allow a role to assume another role - https://nelson.cloud/aws-iam-allowing-a-role-to-assume-another-role/, it has instructions on how to allow assuming a role either in the same account or in a different account.

I'd also like to mention that documentation on how to configure a role to another assume role is going to be missing from our documentation as it will depend on the configuration of the end user's cluster/AWS accounts and we can't cater for all scenarios.

Also @awprice if escalator runs in the same node group that it controls, how can it prevent tainting it's own node and forcing escalator de-deployment? Really can't find an answer in the docs!
On scale down, using the Oldest-first approach, my setup taints the original node on which escalator pod runs first:
time="2023-10-20T16:10:42Z" level=info msg="Sent delete request to 1 nodes" nodegroup=bitbucketpipelines-on-demand-escalator
time="2023-10-20T16:10:42Z" level=info msg="Reaper: There were -1 empty nodes deleted this round"
time="2023-10-20T16:10:42Z" level=info msg="untainted nodes close to minimum (1).  Stopping cache watchers"
time="2023-10-20T16:11:08Z" level=fatal msg="main loop stopped"
rpc error: code = NotFound desc = an error occurred when try to find container "50d71de1cd6378c134bcc3870d3c378860855a379a40d3a7163cf4a913733a6a": not found%

I apologise if those are noobish questions, I'm not a kubernetes expert! (yet!)

Also @awprice if escalator runs in the same node group that it controls, how can it prevent tainting it's own node and forcing escalator re-deployment? Really can't find an answer in the docs! On scale down, using the Oldest-first approach, my setup taints the original node on which escalator pod runs first:
time="2023-10-20T16:10:42Z" level=info msg="Sent delete request to 1 nodes" nodegroup=bitbucketpipelines-on-demand-escalator
time="2023-10-20T16:10:42Z" level=info msg="Reaper: There were -1 empty nodes deleted this round"
time="2023-10-20T16:10:42Z" level=info msg="untainted nodes close to minimum (1).  Stopping cache watchers"
time="2023-10-20T16:11:08Z" level=fatal msg="main loop stopped"
rpc error: code = NotFound desc = an error occurred when try to find container "50d71de1cd6378c134bcc3870d3c378860855a379a40d3a7163cf4a913733a6a": not found%

I apologise if those are noobish questions, I'm not a kubernetes expert! (yet!)

Using instance protection like:
# protect instance on which escalator is running from termination
aws autoscaling set-instance-protection --instance-ids XXX --auto-scaling-group-name eks-bitbucketpipelines-ng-on-demand-XXX --protected-from-scale-in --region eu-west-1

also does not work and the Node is terminated after being tainted. Though if it did work it would probably leave escalator stuck trying to remove the node over and over.

@FilipSwiatczak No problem!

So while this works, it's not fully automated as I can't find a way to fetch the sts role the pod starts under from the cluster.
If you know that or how to structure that better, please share :)

We tend to use IAM roles for service accounts on EKS, as this will prevent the need to deal with node instance roles. This documentation from AWS gives a good introduction and steps to use them: https://docs.aws.amazon.com/eks/latest/userguide/iam-roles-for-service-accounts.html

if escalator runs in the same node group that it controls, how can it prevent tainting it's own node and forcing escalator re-deployment? Really can't find an answer in the docs! On scale down, using the Oldest-first approach, my setup taints the original node on which escalator pod runs first:

We avoid this by running multiple node groups in our clusters and running Escalator on a node group that isn't being scaled up/down by Escalator to prevent Escalator terminating the node that it itself is running on.
Escalator is primarily designed for scaling node groups that are running job-based workloads - so ones that will end. Escalator itself could be considered a service based workload - meaning that it will run forever. So it isn't really the sort of thing that should be run on the node groups that Escalator is scaling.

We tend to use IAM roles for service accounts on EKS, as this will prevent the need to deal with node instance roles. This documentation from AWS gives a good introduction and steps to use them: https://docs.aws.amazon.com/eks/latest/userguide/iam-roles-for-service-accounts.html

thank you @awprice ! I've followed above link and at the very end of Pod checks realised the Escalator pod does not have AWS_WEB_IDENTITY_TOKEN_FILE set.
Those docs suggest an amazon-eks-pod-identity-webhook is required to run to inject the Token but I suspect you are using kube2iam instead right? Thanks again for your patience

It appears when the escalator is deployed in a separate node-group, with custom label escalator:worker at both node and pod level, escalator does't see any cpu or mem utilisation (0). It only works when it's in the same node group for me.
apiVersion: v1
kind: ConfigMap
metadata:
  name: escalator-config
  namespace: kube-system
data:
  nodegroups_config.yaml: |
    node_groups:
      - name: "bitbucketpipelines-ng-spot"
        label_key: "escalator"
        label_value: "worker"

With this and the IAM injection issue I'm a bit stuck. Are there any more complete deployment examples in existence please?

When escalator is attempting to scale node-group different to one it's deployed in, it throws:
time="2023-10-24T10:46:18Z" level=info msg="Node IP.eu-west-1.compute.internal, aws:///eu-west-1c/ID ready to be deleted" drymode=false nodegroup=bitbucketpipelines-ng-spot
time="2023-10-24T10:46:18Z" level=error msg="failed to terminate node in cloud provider IP.eu-west-1.compute.internal, aws:///eu-west-1c/ID" error="node ip.eu-west-1.compute.internal, aws:///eu-west-1c/id belongs in a different node group than eks-bitbucketpipelines-ng-spot-id"
time="2023-10-24T10:46:18Z" level=fatal msg="node ip.eu-west-1.compute.internal, aws:///eu-west-1c/id belongs in a different node group than eks-bitbucketpipelines-ng-spot-id"

@FilipSwiatczak Some answers to your questions:

Escalator definitely works with IAM roles for service accounts, as we have it working that way at the moment and not using kube2iam. You can either add that environment variable manually to the Escalator deployment yourself or rely on something like that pod identity webhook to add it manually. Up to you, but adding it manually is a lot more simpler.
In terms of running Escalator in a different node group - this is definitely possible as we have it running this way internally. It's hard to say what the exact issue is without access to your cluster, but I would check the following things: Labels on nodes are correct, nodeSelectors on pods are correct, nodeAffinities on the pods are correct and IAM permissions are correct. The values for all of these will really depend on your environment, so I can't say what these should be set to. I'd also recommend having a read of https://github.com/atlassian/escalator/blob/master/docs/pod-node-selectors.md, as this explains how Escalator selects pods/nodes.


docs: add favicon

I took the png from the github org, ran it through https://favicon.io/favicon-converter/ and took the .ico file resulting in this:

lovely! :)


Implement local builds

Some flakes may have closed source private git repos in its inputs, which means pushing the whole flake to the remote and building there is not hermetically possible. An argument should be possible to pass that makes building occur on the local machine and nix copy the closure over to the remote.

This is implemented by 8f960401c4267f5b53e6c4886d0c4035122d2c39


fix keymap mode type error

Fixes #365

@scheatkode Thanks for raising the issue, suggesting the fix and reviewing!


Me and my friend keep disappearing

We can connect to eachother technically, and see eachother, but one person will disappear.
For example, I will stay visible on my friend's end, but they will disappear for me. Interestingly, they will stay visible for a few seconds, and only then disappear.
Similarly, if we enter a dungeon, we seem to get entirely desynced. Enemies won't be synced, and neither player can see the other, even if we leave the dungeon again.

Nevermind! Turns out my friend had downloaded an older version of the mod by accident. This is working pretty great!


HUD visual bug

When i go on a world and look at the temperature bar, it has a tint behind it i think it also occurs on the sanity bar and i had everything set to default and just survival inc without optifine.
Steps to reproduce the behavior:

Go to a world.
Look at the temperature and sanity bar.

Screenshots
https://www.mediafire.com/view/u8c1kz1rb3t02mr/Base_Profile_Screenshot_2020.07.01_-_<IP_ADDRESS>.png/file
Game Details:

Forge Version: <IP_ADDRESS>54
Additional Mods:

None

Alright so i tried the new update for the survival.inc mod and i guess its fixed so i will close the case.


[bug]: ESLint: Failed to load plugin 'next' declared in '.eslintrc'

Is there an existing issue for this?

[X] I have searched the existing issues

Current behavior

Steps to reproduce
Linting and checking validity of types  . ⨯ ESLint: Failed to load plugin 'next' declared in '.eslintrc': Cannot find module '/InfraBoard/node_modules/eslint-plugin-next/index.js'. Please verify that the package.json has a valid "main" entry Require stack: - /InfraBoard/__placeholder__.js Referenced from: /InfraBoard/.eslintrc
Environment
Production
Version
Local
Repository
InfraBoard

This issue has been marked as stale due to inactivity for more than 7 days.


Use Microsoft.NET.Sdk and iOS/Android .NET Workloads

Bump to .NET 5.0.100-rtm.20509.5
Bump $(_DotNetPackageVersion) to 5.0.0-rc.2.20475.5
Provision iOS/Android workload installers

Used the boots .NET global tool for this:
https://github.com/jonathanpeppers/boots

Microsoft.iOS/Android.Sdk -> Microsoft.NET.Sdk
Convert Xamarin.Forms app to a single, multi-targeted project.

A prototype of the default item groups / wildcards are in
Directory.Build.props
Moved logic from Directory.Build.targets to
HelloForms/Directory.Build.props.

The current error is weird, I am not getting it locally:
/Users/runner/.dotnet/sdk/5.0.100-rtm.20509.5/NuGet.targets(131,5): Invalid restore input. Duplicate frameworks found: 'net5.0, net5.0'. Input files: /Users/runner/work/1/s/HelloForms/HelloForms.csproj. [/Users/runner/work/1/s/net6-samples.sln]

I can dotnet build net6-samples.sln just fine on my Mac...


Fix for issue: https://github.com/danvk/gwbasic-decoder/issues/1

Introduce 2 parsing states: inside quote, inside REM block
Suppose 'IBM Code Page 437' encoding by default

Hi @bolner, thanks for the fix! I'm glad you've gotten value out of this tool.
Would you mind:

Adding a test file with expected output that demonstrates the fix.
At least make the encoding a parameter an argument. You're right that it should be a flag, but then that means that there should be support for flags in general :)

Hi @danvk, yes, I'll do that today or tomorrow. Later I also realized other issues: like the single quote is also an alias for the REM. Plus: one of the files I attached is broken (probably data corruption due to age :) I'll try to fix it with a hex editor on the weekend), but the program doesn't detect it, it only displays the 20% of the lines. And I have a bad feeling about some of the floating point conversions. I think sometimes it would need rounding at less digits, to show the same numbers as the original were. I could test that in an emulator.
Also it doesn't run on Python 3, only on 2. Should that stay this way, or can I change it to Python 3? :) (I think it wouldn't support 2 anymore)

Feel free to make it Python 3. In fact, please make it Python 3!

@danvk Now everything works which was planned, but I've noticed another bug. Did you know that the GAME.BAS has 7 more lines? :) Just checked it in an emulator. This problem existed before my changes as well. I'll check it.

@danvk Fixed everything, and tested with many BAS files, other than the ones in README.md. No more TODOs left :) Please have a look when you have some free time.

@danvk Committed changes.


Added Table & TableRow Support for AppendBlockChildren, RetrieveBlock…

Added Table Support.
Description
Added Model classes for Table & TableRow Block Types.
Supported Table & TableRow Block Operations:

Add Children to an existing table
Get Table Data
Get TableRows Data

Fixes # (issue)
#226 - Add Table Support
Type of change

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

How Has This Been Tested?
#226 - Compared the request body & response body sent by library with the official api postman collection for AppendBlockChildren & RetrieveBlockChildren
Added Integration tests to verify the same.
Also, tried UpdateAsync with tablerow data, but postman APIs dont seem to support, and there is no mention of it in API Changelogs.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

Hi @KoditkarVedant, Can you plz review the PR. Please let me know if any other functionality needs to be verified. I did test the Notion APIs but found only few functionalities being supported.

I almost forgot - You have missed to update the API where we update these blocks. Src/Notion.Client/Api/Blocks/RequestParams/BlockUpdateParameters

Hi @KoditkarVedant ,
Initially, I did create those 2 files TableRowUpdateBlock.cs & TableUpdateBlock.cs. But, the Notion API doesn't seem to support Update Block functionality and hence i removed them from my commits. The Table support is limited to AppendBlockChildren & RetrieveBlockChildren.

@vijayvarmaD are you planning to re-open this?


compile command fails using nim-regex 0.14.0

When updating to nitely/regex 0.14.0, nasher compile yields the following error:
nasher compile --clean
   Compiling target default
    Updating cache for target default
nasher.nim(90)           nasher
compile.nim(94)          compile
compile.nim(63)          getUpdated
regex.nim(439)           getIncludes
regex.nim(406)           find
nfamatch.nim(123)        matchImpl
nfamatch.nim(27)         submatch
nodematch.nim(134)       match
nodematch.nim(83)        isWhiteSpace
types.nim(23)            unicodeTypes
assertions.nim(29)       failedAssertImpl
assertions.nim(22)       raiseAssert
fatal.nim(49)            sysFatal
Error: unhandled exception: types.nim(23, 10) `cp.int <= 0x0010FFFF`  [AssertionError]

Current fix is to downgrade to regex 0.13.1.

Hi, was this fixed merged into 0.11.0? I am seeing it now.
± |master ?:3 ✗| → nasher compile wotr
Compiling target wotr
Updating cache for target wotr
fatal.nim(49)            sysFatal
Error: unhandled exception: types.nim(23, 10) cp.int <= 0x0010FFFF  [AssertionError]
2020-04-23 11:00:36 ⌚  radiant in ~/NWN/nwnxee-docker-template/server/modules/wotr
± |master ?:3 ✗| → nasher --version
nasher 0.11.0

The fix has been merged, but a new version has not been released yet. You can get the dev version with the fix using nimble install nasher@#head.


Beautification History (AKA "undo the beautification I just did after I already closed that tab")

Some users are experiencing problems when they beautify and then close the tab, only to realize later that the beautification was incorrect and they now must manually fix their code.
Solution:

[ ] History limit - how many items to remember in history (this can be small, like 3)
[ ] Automatically push the original text into the history
[ ] Add command to open History, which is create tabs for those 3 or so items remembered in history

I want to approach this one...
Do you already have an idea how to store them ?

Do you already have an idea how to store them ?

Since the history should be small (maybe 1-3 entries?), I was thinking store the History simply in memory (JS variable).
Take the file path as a key and map that to an array of 3 strings for history. Push next history item on save and keep the array short (3 entries max?).
{
  "FILE_PATH_1": [ "change3_last", "change2_old", "change1_oldest" ],
  ...
}

With respect to Atom-Beautify and Unibeautify, I think for now this can be implemented within Atom-Beautify (so users can get this feature ASAP) and later, once I have finished with Unibeautify migration, we can see about moving it to either Unibeautify or keeping within Atom-Beautify.

Update: The more I think about the problem:

Some users are experiencing problems when they beautify and then close the tab, only to realize later that the beautification was incorrect and they now must manually fix their code.

And really it comes down to: Users makes change, saves file, closes tab, loses Atom editor history for file because previous tab was closed.
With that in mind, I think this should be a separate Atom package entirely. One for persisting Undo History across sessions. I think this feature should be separate from Atom-Beautify and will be valuable even in other contexts.

I'm going to close this as wontfix with the expectation that someone may develop another package for Atom which fixes this UX issue 😄 . When (if) someone does develop such a package, please provide a link to it here with a comment and I can add it to the README to recommend users to take a look.

I have to disagree since if I do beatification on a file path (e.g. Right click the file/path and beautify) it would not have an Undo to Beginn with.. I might be able to hack into the save function and copy the old file before. (Have to do some research on that).
At least that is why I need this feature asap.

it would not have an Undo to Beginn with

Shoot! You are correct. Thank you for reminding me. Here is the applicable code: https://github.com/Glavin001/atom-beautify/blob/master/src/beautify.coffee#L227
So I do still think a package for Atom could (and should) exist to solve the problem of Atom's Undo History being lost, however that is not (yet) this issue here.
I haven't thought too much about this feature. What are your ideas on how you are thinking of implementing it?

I would like to replicate the file structure on beautification with file system if project root exists I would but it in a folder .atom-beautify/path/to/file.date otherwise I would store it in the plugin Folder under history something then save it somehow in the settings to get all the old files .. not so sure yet.
I still have to do some research.

I think creating a backup copy within the date is a good idea. I recommend something like https://www.npmjs.com/package/tmp or https://github.com/bruce/node-temp for creating the directory though, so it eventually gets cleaned up.


extra optimize decorator (_spec & _logger)

copy docstrings to cython code

Pull Request Test Coverage Report for Build 458

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.707%

Totals

Change from base Build 455:
0.0%

Covered Lines:
458

Relevant Lines:
464

💛  - Coveralls


Pull working  list messages at first ?

The broadway is processing some messages, and then I restart it. When the system goes online later,
the last working_list messages are left and never queued.
Should we provide a hook to queue the left working_list messages into list explicitly, or do it in producer implicitly? or any suggestions?

@fahchen Sorry for the delay! This is a good question and one I chose to not solve because, for me, it was an edge case that I couldn't solve with simple logic.
Depending on your message format in Redis, it is difficult to know which items are things that are currently being worked on or deserve to be moved back to the queue. This is true for anything using this 'durable queue' approach in Redis.
I find myself in situations where, sometimes, I have code written in another language working from the same queue. Because of that and if my messages in the list(s) do not contain any time information to determine age - It would be bad if I automatically moved the data from 'working' to back to my queue list.
I do use this library in production today and my system is pretty much running for weeks/months on end without being stopped. I monitor my working queue and have a script to move those items back from the working list to the queue list.
Having said that, I'm absolutely ok with someone implementing an optional hook and a PR to facilitate this as well. If I have cycles this week, I do plan to go in and update to Broadway 0.5 - so maybe wait on doing anything wild until you see that update.

Thanks for your reply.
My scenario is simpler than you now. Currently, I move working_list messages back to the list when my system is about going up manually.
Maybe Broadway.Producer.prepare_for_start/2 in Broadway 0.5 is a nice hook to do this, we can expose this hook for users, now it's only available for producer.


[BUG] Does it work in a multi-GPU environment?

Describe the bug
When I try to infer llm locally, only the 0th GPU is used. Is there a guide to using multi GPU?

@daegonYu
Hello.
We do not have multi-GPU environment yet, so we couldn't experiment multi-GPU environment yet.

@daegonYu
Hi
Fortunately we get multi-gpu server and investigating this issue

Okay, thank you.

Not resolve this issue yet


Fixed issue #474

canScrollHorizontally() throws NPE if the RecyclerView is not attached to the Window

Hi @KryptKode,
Why hasn't it been merged? Can we get this fix published @thagikura ?

Sorry for the delay on this. LGTM!

@thagikura Can you publish 1.1.1 version?

@thagikura We also have this problem in our app :( I would really appreciate it if you release this change

Released the library as 1.1.1. Please let me know if something goes wrong.

1e147f562fbbf1e2a83361bf7ffeffead0808818


[Flake][Fixit][Gpr_To_Absl_Logging] Fixing flaky test no_logging.cc

[Flake][Fixit][Gpr_To_Absl_Logging] Fixing flaky test no_logging.cc
Used this command to validate the fix
CC=cc bazel test --runs_per_test=100 --test_output=all -- //test/core/end2end:no_logging_test@poller=epoll1

it's not clear to me that such an error is fine during normal execution


Created functions for the data cleaning script, along with an initial version of the test files for these functions.

Fixes #59 and #55


Add support for history 3.x.x

See https://github.com/taion/react-router-scroll/issues/28#issuecomment-244503918. I don't think there's much I can do here until we have a new history v3 release.


chore: generate ClusterRole from RBAC markers

Description
This PR finalizes the migration to generating the trivy-operator clusterrole from code markers using controller-gen, and will allow us to finally close https://github.com/aquasecurity/trivy-operator/issues/204. We should probably add a few lines to the contributor documentation about how this works, with links to the marker documentation (kubebuilder docs), but that could be a follow-up PR.
I have tried to put markers in the code close to the relevant code creating the requirement for permissions. In some cases that leads to the same requirement for a permission being expressed in multiple places. But this is deduplicated by controller-gen, and IMO it makes a lot of sense to do it, since it will allow us to adjust the code and RBAC requirements from the same "page". I.e. when deleting code.
Note to reviewers:

I have granted all verbs to resources defined in trivy-operator, but I can adjust that, if you prefer to keep it as-is.
It seems like the get and list verbs gives full access to pods/log, at least AFAIK. So I removed the watch verb, which does not add anything.
I was a bit surprised to see that the operator does not need RBAC permissions to deployments, but it seems so. Some funcs in the kube package uses Deployment, but it seems like the funcs are only used from test-code. Maybe we should refactor a bit, if my investigations are correct?

While working with this, I had to dig deep into the code, so I will also suggest to close https://github.com/aquasecurity/trivy-operator/issues/187 with this PR, as I think this is as far as we get tightening RBAC with the current code and features in trivy-operator. Locking down even more, will require changes to the code and/or dropping features. So I suggest to open more specific issues for that.
Related issues

Close https://github.com/aquasecurity/trivy-operator/issues/204
Close https://github.com/aquasecurity/trivy-operator/issues/187

Checklist

[x] I've read the guidelines for contributing to this repository.
[ ] I've added tests that prove my fix is effective or that my feature works.
[ ] I've updated the documentation with the relevant information (if needed).
[ ] I've added usage information (if the PR introduces new options)
[ ] I've included a "before" and "after" example to the description (if the PR is a user interface change).

@erikgb this pr also fix the errors we see on main branch

@erikgb this pr also fix the errors we see on main branch

@chen-keinan No, I think this error is related to CRD. Please register an issue including how to reproduce. I am a bit blocked for running integration-tests on my laptop because of air-gapped environment, but I think we should have an integration test for the errors you see. So I'll do my best.

Please register an issue including how to reproduce.
errors on the main branch issue #327

1 - do you mind adding some documentation about it on this PR? I know you mentioned a followup, but keeping the changes+docs on the same PR seems like a good process for any change.

Done, PTAL!

2 - I'm seen an error related to rbacassesment when running the code here, that I don't see on the main branch?

That was an error already merged to the main branch, and fixed by https://github.com/aquasecurity/trivy-operator/pull/328.


Revert EvalIterHook logic to handle KID dict output

Codecov Report

Merging #10 (d932a01) into master (d291da2) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
- Coverage   83.90%   83.89%   -0.02%
==========================================
  Files         227      227
  Lines       13024    13027       +3
  Branches     2026     2027       +1
==========================================
+ Hits        10928    10929       +1
- Misses       1741     1742       +1
- Partials      355      356       +1

Flag
Coverage Δ

unittests
83.89% <100.00%> (-0.02%)
:arrow_down:

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

Impacted Files
Coverage Δ

mmedit/core/evaluation/eval_hooks.py
64.28% <100.00%> (+2.74%)
:arrow_up:

...ls/components/stylegan2/generator_discriminator.py
89.75% <0.00%> (-1.21%)
:arrow_down:

mmedit/datasets/pipelines/random_degradations.py
94.01% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Prevent deleting of namespace if a release exists

Problem Statement
Currently, if the user has more than 1 component installed in same namespace and they try to delete a single component using delete-namespace flag, it does no checks and deletes the namespace. This issue occured as headlamp component is installed in lokomotive-system namespace. And deleting it will result in the deletion of admission-webhook.
Offline discussion suggestions
We wanted a solution where we can install multiple components in same namespace.

Install these components in dedicated namespace
Track components installed in cluster
Check if namespace is lokomotive-system and throw error to user
Create a Validation webhook and avoid deletion of lokomotive-system namespace
Check if other helm release are installed in that namespace, if not delete the namespace (accepted soultion)

One more option was:

Only remove the namespace if it has no resources left, but that would require iterating over all resources reported by kubectl api-resources --namespaced=true which would probably be slow and maybe even problematic from API point of view.


Emoji leads to weird slug

Watch carfully the hovered link (it looks good), but when clicking, I got a weird URL. I assume that the emoji is still here, but hidden.
It seems that remark-slug is ok as you can see in the HTML. Ref https://github.com/wooorm/remark-slug/issues/2

The bug, as I found out, is caused because there’s a VARIANT SELECTOR-16 just after the emoji.
GitHub also includes it: https://github.com/MoOx/phenomic/blob/671a0cf596f184e2d1c30b77e219f4711e5e25ce/docs/content/docs/usage/gh-pages.md#-warning-for-githubio-domain
As you can see from that link, GH URL encodes the link. Which I think should be done too*.
callback('#' + encodeURIComponent(data.htmlAttributes.id));

...which can be added here
* I think this should be done here, but maybe in the HAST-related tools instead. Unsure.

https://github.com/ben-eb/remark-autolink-headings/commit/54434ffc75b3338fdb8127151ab8b8f15899a08c look good to you guys?

@ben-eb Yup!

Problem is that this test still passes if I change the line back to how it was originally. 😐

@ben-eb The tests fail because, I believe, the fixture uses  and other entities, but you can just use the literal characters (they don’t have to be encoded)

Problem is that this test still passes if I change the line back to how it was originally. 😐

How do you mean?

Tried a node modules re-install and now my local environment reports the same as Travis, so I don't know what that was about! I fixed the fixture anyway now.
It means that the test still passes regardless of me changing this line. I don't know why this is the case but it would seem to indicate that me changing the line didn't do anything...

I no longer think this is a bug, neither in remark-slug, nor in remark-autolink-headings. Rather, it’s behaviour as expected. Upon re-reading your question, @MoOx, it seems you were wondering where the #%EF... in the URL was coming from. Well: it’s because you have a VARIANT SELECTOR character in there, and it shows up in the HTML. As expected.
@ben-eb IMO this can be closed.
@MoOx I’m not sure why that variant selector is in there, either remove it if you want a pretty url, or keep it in there and deal with the URL being like this!

@wooorm It might be worth investigating into an option to strip emoji from the generated slug, which could work around this issue. That's an issue for remark-slug though so I'm going to go ahead and close this. 😄

Strip emoji from the generated slug

Well, the emoji themselves are already stripped, using the same mechanism as on GitHub and npm. If we deviate from this mechanism, it’ll break links across those vendors too.
I did some more digging, and found this article to explain a bit more.
I could see how the VS16/VS15 should possible be removed, but as GH doesn’t do it I don’t think we should do it either.

OK, fair enough. 👍

I just used the macOS native emoji picker to add this warning emoji. Nothing more... :'(

Aww, haha :')


Any reason why DANGEROUS operations like this work and surprisingly does not work in ONLY one edge case?

🔎 Search Terms
"generics", "generics type constraints", "generics record index key value change"
🕗 Version & Regression Information

This is a crash

⏯ Playground Link
https://www.typescriptlang.org/play?noUncheckedIndexedAccess=true#code/PTAEEYDpQZQYwKYDsCGAnAlge1AWlAIKgAiKSA5gmlgK4DOoWADlSgC7ZKgDuAFhnF6gAKjFAoANhKzc6AKABmNJHA5YuAWxpt2CcAAosAIwBWALlAAlBHCxoAJgB46bTBQA0oZQGskMpAB8AJSgAN5yoJGMppDeCACeoAC8EADcoCCgbPwMGAz2ZJTU9NAAcjhU1GhyAL5ycrZILtEm4MlhcfEWAOR0WBoIoJ3dNan1Wjpseoam4EHpmTC8tBL2ImKVdozaoLxUgyi5bKCbaAy0x3zIoEyHdBgU4qBoCCj26hKJbDh+XC9vH3i9UafQkCEg0nIM1asQSkG+ABksHBJAgkdwqABhQ4IfRBeYZMAAUQAGpiiQAFYQASQA8qULJ1QHlQH5uKyaBojFRQKVacJQC43ORoHJMrgJZKpdKZdLQETSsQ8LLcPUxWAAEzQeDIdDYPCgAAS+3WNxeADdkGwGKdQApqBpdigmCwkA9yKAjDt3khusdfDIeLwgUoVGpNNpdBrHAQ0OQbQAPKZIewMay2BzOVzuzw+NmBALQiyx+MhcJRQmgADqwdA7wQDGyLNtFwA-BEosYTLDEilwAswMIcic0FVztptTg+cJDdTSgBxUAorjc0ATXRrPYveoVjAKfTdYbMrhd0AAMjPWXiLCwChaPeSSRS3SQnO5aG6ZY7FcimT5LSGBIGGXeUADUiUsT1BkEQoEDWPxjg0dgpjQLI9ivFhyBodB7GgGt4lbQj2x-TsYiZFINQHUBqWOFwMCkEcql6bZjgeOgmAwKZGDvSQJDQwY2GvBAsJw79QDqOoGnUZouw1dpQk6Ho+gGQD4hGMY5HXKYNWhDV5mBaSsDBCEsChWSe3hLAkRRMF0SxHE8QJTImSrakEQRKDBWzR4t3BUAKQggAxIlMWEdVlRVSLZXlRUIplNUxQAKkSiJEtAABmUVQDS1LQECozpG4d1YEQVBMBwLURGHOgaDQJhMHuChPhIWkiTEaczQQS0kGOW17X6J0XWQd1cp9P0hjZIMgWy4BFGUVRODXSMpnSmM40TZNUysGw7CcIUcy8JAA24QIAG0AF1Cy7Yt1q-Xd9y7U6AAZzrukiWme86Hz7KihxZYhWva-lGLscc2Dw4N4gIsSJIMppji7dL2lOhSEiU-pBmGGpzo0rSEHS6F0v0qSmiM8FIUJz7WwsxFkVROy0GxOhcXxKjSXJKk6QZVTmQYSbXy5HkOv2ihRXFKKJalGKlUi+pMgAVm1Uq9RwfAmwYbg7G8BhDhOBMWFUOCSeaf4fU+WM0AsU3AS84ULuR7oFAwM42G6HG4eaLT3Qtiw838e2UmtpBzdHX7h1OMGlzsF5VE+cLIuluLVXCgAWJXdXKg0QNee5mpgihBiD5quxsf0gON44i-iWlTHk7wLA1GpxGA6S2A0kFEOW90a-MbaMz27zyFzI78wCdoq57sYvYoHvYnaXpB+6KjyhB1DfObngECkeoJ7IyzAowBM4P0PSw5ZVz3JEQ1LFpKsN8PbpedZHBpAL1CUA5QXULZeOJcTyWgA
💻 Code
// 1. Scenario - A Dangerous operation which TS allows
function mutate1(obj: Record<string, unknown>) {
    obj.key = 1; // this is dangerous. No error
}

const obj1 = {key: 'some key'};

mutate1(obj1); // Should TS error out here as it errors out when passing a readonly to non readonly

console.log(obj1.key.toLocaleLowerCase()); // EXCEPTION: key is now number NOT string.
// -------------------------- END ---------

// 2. Scenario - Here TS prevents error from happening but don't know why
function mutate2<Args extends Record<string, unknown>>(obj: Args) {
    // Why does this error out?
    obj.key = 1; // This errors out. So NOTHING can be mutated here

    if('key' in obj && typeof obj.key === 'number') {
        // NO obj keys can EVER be changed not matter the typeguard. Why????
        obj.key = 2; // It still error's out inspite of all the typeguard
    }
}

const obj2 = {key: 'some key'};

mutate2(obj2);

console.log(obj2.key.toLocaleLowerCase()); // key WILL be string here. PERFECT
// -------------------------- END ---------

/**
 * 3.
 *
 * Following Scenario 2. This surprisingly DOES NOT prevent error from happening
 * don't know why
 */
function mutate3<Args extends Record<string, unknown>[]>(obj: Args) {
    if(obj[0]) {
        obj[0].key = 1; // This DOES NOT errors out. Whyyy?
    }
}

const obj3 = [{key: 'some key'}];

mutate3(obj3);

console.log(obj3[0]?.key.toLocaleLowerCase()); // EXCEPTION: key is now number NOT string.
// -------------------------- END ---------

// 5. Scenario - this works as expected
const readonlyArr: readonly string[] = ['first'];

const mutatingArr: unknown[] = readonlyArr; // This errors out correctly
// --------- END -----

// 4. Scenario - can easily change readonly object keys
const readonlyObj = {k: 2} as const;

const mutatingObj: Record<string, unknown> = readonlyObj;
mutatingObj.k = 'string'; // No error here as well

readonlyObj.k.toFixed(2); // This WILL THROW as 'k' is no longer a number now
// ------------ END ------------

🙁 Actual behavior
1] If a function accepts an argument of type Record<string, unknown> then the function can change the value of any key to ANY value. While this makes sense within the function, but, if you pass a well structured object to this method as an argument then the function can change any key to any value and that will result in runtime error and there is no compile time error when a well structured object is passed to a function accepting Record<string, unknown>. But, if you pass a readonly array to non readonly array then it errors out properly.
2] If a function accepts the same argument type Record<string, unknown> via generics then changing ANY keys of the object within the function is not possible.
3] If the function accepts the argument of type Record<string, unknown>[] this time as a array then the function can change any key of any object within the array. This is in stark contrast to point 2 where it is not possible.
4] Readonly array cannot be assigned to non readonly array but readonly object CAN be assigned to non readonly object. Why this difference in functionality?
🙂 Expected behavior
1] Readonly object should NOT be assignable to Record<string, unknown> as any keys can be mutated within them causing dangerous side effects.
2] The behaviour of Record<string, unknown> should be the same regardless of whether it is enforced via generics or directly as a function argument type.
3] Generics constraint Record<string, unknown>[] i.e. array variant (here any key of any object within the array can be changed) does not seem to behave like generic constraint enforced via Record<string, unknown> i.e. non array variant (here no object keys can be changed)
Additional information about the issue
No response

Duplicate of #13347. Readonly objects are implicitly compatible with the mutable versions.

@MartinJohns I have highlighted many other points besides that just one point. It is not a full duplicate of the https://github.com/microsoft/TypeScript/issues/13347, only one point matches with the issues I have highlighted

This is a well-known unsoundness: A writable property should be invariant but is considered covariant by default. See also https://github.com/microsoft/TypeScript/issues/55451 and comments.

This is a crash

Also, fyi, this is not a crash. "Crash" refers to when the compiler crashes.

@abaldawa The reason why read-only arrays are not assignable to mutable arrays is explained by https://github.com/microsoft/TypeScript/issues/13347#issue-199392398:

"... ReadonlyArray has no push method, making it incompatible with Array."

(Note that readonly string[] is just a syntax for ReadonlyArray<string>:
https://www.typescriptlang.org/docs/handbook/release-notes/typescript-3-4.html#a-new-syntax-for-readonlyarray)


Can't change avatar on keybase.io

Error: "Sorry, your upload didn't work."
Chromium console: Error: Server failure: RPC error 200: Error: could not load file for sending to aws

cc @malgorithms

Is this fixed now?

yes

The error started again.
I have a profile picture in mobile application, but it does not exist on the website. Interesting situation.


Protocols dead link

Type
Site Bug Report
Feedback
With reference to the "Overview" section on this page: https://developer.tbd.website/api/web5-js/dwn/protocol/

There's a dead link on the word "Protocols".  It goes to
https://developer.tbd.website/api/web5-js/dwn/protocol/protocols which is "Page Not Found".
I believe the correct link is this: https://developer.tbd.website/api/web5-js/dwn/protocols.
Screenshots
Paste relevant images or screenshots here
developer.tbd.website URL
No response
Browser
Chrome
Relevant log output
No response

Interestingly, it works fine if the original URI does not contain a trailing /, i.e., https://developer.tbd.website/api/web5-js/dwn/protocol instead of https://developer.tbd.website/api/web5-js/dwn/protocol/
Unfortunately, I'm not sure how I landed on the latter, but this is not a bug if the starting URL contains no trailing /
Please feel free to resolve if this is not really an issue.

@shobitb Hey you are right the link is broken. The word 'Protocols' should be linking to this page: https://developer.tbd.website/docs/web5/learn/protocols. Would you like to work on this issue?

@EbonyLouis Okay, thanks for confirming. Yes I'll pick it up!

@shobitb 🥳 Thanks for catching it!

hi @EbonyLouis, I would like to work on this issue, if it is idle.

I've unassigned myself. @EbonyLouis, you can assign to @aniketh-varma .
Thanks!

I have created a PR to fix this. Please review https://github.com/TBD54566975/developer.tbd.website/pull/917

Hey @akmittal, I was waiting for the issue to be assigned to me. I think you should have also commented before raising a PR, @EbonyLouis please confirm.

Hey @akmittal, I was waiting for the issue to be assigned to me. I think you should have also commented before raising a PR, @EbonyLouis please confirm.

I've assigned to you. Correction: please make the link point to: /api/web5-js/dwn/protocols. Thanks so much!


Fix Xcode 8 error

Fixing this issue:
=== BUILD TARGET Toaster OF PROJECT Pods WITH CONFIGURATION Debug ===

Check dependencies
“Use Legacy Swift Language Version” (SWIFT_VERSION) is required to be configured correctly for targets which use Swift. Use the [Edit > Convert > To Current Swift Syntax…] menu to choose a Swift version or use the Build Settings editor to configure the build setting directly.
“Use Legacy Swift Language Version” (SWIFT_VERSION) is required to be configured correctly for targets which use Swift. Use the [Edit > Convert > To Current Swift Syntax…] menu to choose a Swift version or use the Build Settings editor to configure the build setting directly.

** BUILD FAILED **

@hypery2k Thanks!

you're welcome

Your commit is shipped with 2.0.1 release. Thanks again for you contribution :tada:


Initial checkin nuttx website github action workflow

Nuttx website github action workflow:

Checkout nuttx website
Install essential tools
Run publish.sh and push asf-site branch

Signed-off-by: liuhaitao<EMAIL_ADDRESS>

One of the github action CI in my own fork as below:
https://github.com/liuguo09/incubator-nuttx-website/pull/8/checks

@btashton please help review again. Seems comments gone after I force pushed to update PR : (
To summarize:

I remove the additional sudo apt install bundle line you commented
Keep sudo ./publish.sh, or it would fail as below:
ERROR:  While executing gem ... (Gem::FilePermissionError)
You don't have write permissions for the /var/lib/gems/2.5.0 directory.
##[error]Process completed with exit code 1.
More details in https://github.com/liuguo09/incubator-nuttx-website/pull/5/checks

I'll take another look. I run publish without sudo locally and everything ends up in the working directory. I'll fork it and take a look at what's up.

@liuguo09 Ok I finally got around to sorting out what what up using a ubuntu:18.04 container which I think is close to the github environment.  I installed the system packages that you have and then before calling publish I create a local GEM_HOME and exporting the bin path in it so the bundler application is available.
export GEM_HOME=$HOME/.gem
export PATH=$PATH:$GEM_HOME/bin

@btashton You are right. I update with the export cmds you provide before calling publish.sh and it works. As below,
https://github.com/liuguo09/incubator-nuttx-website/blob/master/.github/workflows/main.yml
But it must come with a git author email and name before git push asf-site. Now I hard coded with my own info. But I think it would be better to use some official or fake email/name instead. What's your idea?

for the author and email, prior to running publish.sh we can get them like this:
username=`git log -1 --pretty=format:'%an'`
email=`git log -1 --pretty=format:'%ae'`

for the author and email, prior to running publish.sh we can get them like this:
username=`git log -1 --pretty=format:'%an'`
email=`git log -1 --pretty=format:'%ae'`

Good idea! Update done, please take a look.


A.06.03.QoS.availability

This issue has been extracted from the issue list on:https://ies-svn.jrc.ec.europa.eu/issues/2685
Comment
This is not a test for the test framework. Continuous monitoring should be part of the local infrastructure (this applies in general for the other QoS aspects as well, but for them a one-time test in a test run are possible, but not for availability).
Proposed Change
Remove test case from ATS

I disagree with the proposed change. The ATS should include QoS aspects, but it should be clear that these are to be tested locally to the server (described as option 1 at 5.1 of TG Discovery Services). I believe this part of the ATS does not therefore form part of the requirements for the central validation service (that would be option 2 in the TG, which was to be revisited by the IOC TF).
Proposed resolution: flag this test case as 'not for central implementation'


feature(datasource/boss)

This is a slightly reworked version of @alexbaden's Boss datasource. @neurodata has been using it heavily over the past year.
KeyCloak authentication is now via a pop-up rather than redirect, and the keycloak-js library is no longer used.
There are definitely a few things we still want to look at:

promise warnings on the console while typing in hostnames
attempt to do an immediate login (like Google OAuth2) in an iframe
switch keycloak authentication workflow from implicit to authorization code

Sample URL: boss://https://api.boss.neurodata.io/collman/collman15v2/EM25K
(The data are public but a valid login is required and can be created on the KeyCloak auth page.)

Overall this looks great!
I was able to test that it worked.  The retry authorization button doesn't fully work: if you click retry while an auth tab is already open, it correctly closes the existing tab and opens a new one.  However, it shows a CANCELLED status message, and if you complete the auth in the new tab that was opened, the auth code doesn't recognize it.  However, if you then click retry again, it does work.  I noticed that the Brainmaps credential provider, that this one was based on, has the same issue with retry, though, so I'm fine with this issue not being fixed for now.

@googlebot This PR is ok with me.

Hi @jbms -- just checking in to see if there's anything else you need from us on this PR before merging.  Thanks for all your work on this - we love it!

The changes look good, thanks.  Let me know when they have been tested.

I have tested and can load BOSS data with these latest commits.  Here's a test URL: https://deploy-dev--viz-prod-neurodata.netlify.com/#!{"layers":{"bloss2018":{"source":"boss://https://api.boss.neurodata.io/bloss/bloss18/image"%2C"type":"image"}}%2C"navigation":{"pose":{"position":{"voxelSize":[3.890000104904175%2C3.890000104904175%2C50]%2C"voxelCoordinates":[40993.6484375%2C53252.57421875%2C160]}}%2C"zoomFactor":0.5965508371143481}%2C"layout":"xy"}
Also working locally


force_base_url should have better documentation

https://registry.terraform.io/providers/sonatype-nexus-community/sonatypeiq/latest/docs/resources/system_config#force_base_url
The current description just says (Boolean) Should the Base URL be forced? which is quite useless really, since it doesn't tell you anything more than the name of the option already did.
There doesn't seem to be a corresponding option in the actual IQ Server UI, so what does this flag do? When should it be set?

Looks like it's this option from the rest api docs:
https://help.sonatype.com/en/configuration-rest-api.html
Default false value is desired. If true, force IQ Server to treat all inbound requests as originating from the configured baseURL instead of relying on inbound HTTP request headers to determine this dynamically.

Setting this to true in order for HTTP requests to the server to work correctly suggests a misconfiguration of an upstream server that is reverse proxying requests to IQ Server. The upstream server should instead be sending properly formatted Forwarded headers .


Devsite 1192: Icons with Links formatting

Icons only displayed small if followed by a link.
Description
I made an if statement so that that even text gets added to the product link container div. I also enwrapped non-links in an paragraph tag with class "icon-text", so that there would be some space between the icon and the text next to it.
Related Issue

Jira Ticket
Motivation and Context
Icons should be able to have non-linked text associated with them without becoming huge.
How Has This Been Tested?
I just modified the code and checked if the DOM was intact. I looked at it on localhost and the images came out to the right size.
https://docs.google.com/document/d/1RphfPEBfnnFrOAm1nPrmdks8VTjdQRtPTgTwuVsL0Jc/edit
^This is Erin's original google doc that caused the errors
This is my copy
Screenshots (if appropriate):
Before:

After:

Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Preview: https://devsite-1192--adobe-io-website--adobe.hlx.page/test/elitesticons

It looks like this also solves this Jira Ticket. You can test using this link.

I think we need to make sure we're not breaking other columns formats:
https://main--adobe-io-website--adobe.hlx.page/#heading-3-is-the-size-with-bold
https://devsite-1192--adobe-io-website--adobe.hlx.page/#heading-3-is-the-size-with-bold
I can see the icons are misaligned here.


fix: @vuepress/client dependency in vuepress-vite

The dependency "vuepress-vite -> @vuepress/client" should not be hardcoded (as far as I understand), but always use the in-workspace version (i.e. "workspace:*")

It's a known issue of pnpm https://github.com/pnpm/pnpm/issues/4164
So I have to rollback the workspace protocol in 8230e24e03b611cc9a7b3aa8fa64483d2af6a3a3

oh, thanks for the info. I just came across this, because I had peer-dependency warnings in my project. But I wasn't able to create a minimal example for that.


feat(idempotency): expire time outed in-progress invocations

Issue number: #1038
Summary
Expire in-progress invocations caused by Lambda time outs.
Changes

Please provide a summary of what's being changed

During an invocation, we try to figure out how much time the Lambda invocation has before it expires, and save it on the idempotency record together will all the other fields.
When a new invocation with the same key arrives, we check against this timestamp to see if there's any INPROGRESS invocation past the timeout timestamp. If there is, we allow the invocation to be retried.
We also moved the squence diagrams of the idempotency module in documentation to mermaid.js.
Caveats

Only works out of the box for the @idempotent decorator, because right now it's the only way to access the Lambda context to get the remaining time left. I've added a register_lambda_context to the IdempotencyConfig class as a workaround

User experience

Please share what the user experience looks like before and after this change

Before: if a Lambda invocation times out, the invocation will get stuck until the idempotency record expires (default = 1 hour).
After: if a Lambda invocation times out, the invocation will be tried again on the next retry.
Checklist
If your change doesn't seem to apply, please leave them unchecked.

[x] Meet tenets criteria
[x] I have performed a self-review of this change
[x] Changes have been tested
[x] Changes are documented
[x] PR title follows conventional commit semantics

Is this a breaking change?
RFC issue number:
Checklist:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

View rendered docs/utilities/idempotency.md

Codecov Report

Merging #1387 (975933d) into develop (7414df7) will decrease coverage by 0.16%.
The diff coverage is 75.67%.

@@             Coverage Diff             @@
##           develop    #1387      +/-   ##
===========================================
- Coverage    99.88%   99.72%   -0.17%
===========================================
  Files          119      119
  Lines         5429     5463      +34
  Branches       620      627       +7
===========================================
+ Hits          5423     5448      +25
- Misses           2        8       +6
- Partials         4        7       +3

Impacted Files
Coverage Δ

...wertools/utilities/idempotency/persistence/base.py
96.42% <58.33%> (-2.94%)
:arrow_down:

...ws_lambda_powertools/utilities/idempotency/base.py
97.29% <77.77%> (-2.71%)
:arrow_down:

...ools/utilities/idempotency/persistence/dynamodb.py
96.70% <86.66%> (-2.00%)
:arrow_down:

..._lambda_powertools/utilities/idempotency/config.py
100.00% <100.00%> (ø)

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 c463232...975933d. Read the comment docs.

I've pushed an updated doc with the following changes:

Reordered the wording so we give the positive news first (what exactly we do we do?)
Give comfort to those using @idempotent (When do they need to care?)
Used banners to emphasize when they need to take action (what should I do?)
Moved sequence diagram for latter so they're informed first, then they can dig deeper into the mechanics

Shortening the wording on sequence diagram to fit in the final SVG
New (pushing shortly)


Ensure browser version of Svelte is used in tests

Companion PR to viamrobotics/app#3492

Sorry, not ready for review, my bad