awesomewm config written in fennel lisp

if you want to have a look at an awesome-wm config written in lisp, I am curently daily-driving this one I wrote in fennel based on the default one but modified quite a bit
I only tell you this because I know you use emacs and such probably like lisp
hope you enjoy using awesome-wm,

Went through fennel, the tooling is awful and has a long way to go to catch up.

Went through fennel, the tooling is awful and has a long way to go to catch up.

I am using it on a daily basis
my website is technically written in it
as well as most of the software I use on a daily basis(including ones that arent meant to be configured in lua)


Improved details on setting up an Xcode project for use with whiten mobile

If you provide me a quick and dirty list of the steps needed, I can covert that into a PR.
I have been trying to follow examples in go-innovation, but the docs on https://ebiten.org/documents/mobile.html#iOS
But it's a bit vague on connecting a UIViewCobtroller to a UIView.

As this requires a general knowledge about iOS application development and might change frequently, I hesitate to add more details in the instructions. Have you succeeded to run go-innovation btw?

Yes, I can get go-innovation to run just fine.  You're right in the fact that things change fast with Apple.  Running the latest Xcode, it already does things different than what you have in go-innovation.  I'll play with it some more and would be happy to add my findings.

I was able to get it to work... I'll document some stuff as I go and can contribute if you'd like.

Sure, could you send a PR at ebiten/ebiten.org by editing the HTML under content? Thanks!


Add Safari 17

This might be a nice time to add a spot in the system for data about a potential future Safari 17.

Aren't you a day or two early? 🤓
Only kidding 0:-)
+1

Safari 17 was added in https://github.com/Fyrd/caniuse/commit/7fa607a13b560c578147e06e943223e12c52fb9c - now where is that changelog? 0:-)

there we go: https://developer.apple.com/documentation/safari-release-notes/safari-17-release-notes 🥳

Even more information: https://webkit.org/blog/14205/news-from-wwdc23-webkit-features-in-safari-17-beta/


[Feature Request]: UI without wscript.exe

The request
Hi, nice Job but are there alternative versions of the updater script without using the Windows Scripting Host?
Is your feature request related to a problem?
company policies refuse to use Windows Scripting Host

Additional information
No response

It's in the pipe #465
We are looking for a good way to remove the vbs helper

Windows Script Host is going to be deprecated with VBS.
https://learn.microsoft.com/en-us/windows/whats-new/deprecated-features
and it is recommended to disable WSH on hosts to mitigate many security threats.
I hope you will find a good solution to substitute WSH with some PowerShell/.Net feature/assembly.


stop gulp verifying ssl certificate & update user-agent

Haven't fully tested it yet but this was failing for me when loaded over HTTP, so I added a dirty check to see if we should disable ssl verification.
also updated the user-agent to a more commonly used one

Thanks, I will include this in some way in a upcoming release.

I have introduced a new environment variable to disable the ssl verification. Try it out.
SSL_VERIFICATION=false

Awesome, thanks! and thanks for the project too, happy to help out how I can


Convert expandable header to flexbox

Fixes https://github.com/cfpb/design-system/issues/1398
Changes

Converts expandable header from a float to a flexbox layout.

Testing

yarn start and view http://localhost:4000/design-system/components/expandables and resize the window and expand/collapse the expandables.

Screenshots
Before:

After:

Note: I left the o-expandable_header-right for consistency, even though it doesn't have any styles applied any more. We could consider removing it entirely.

Edit: I have removed both o-expandable_header-left and o-expandable_header-right, as we really shouldn't have classes that designate positioning, and those two elements already have o-expandable_label and o-expandable_link, so we can target those instead.


Fix: fix default commit handler

ZH CARD 1702

I haven't totally thought this through yet, but another way to fix this might be to adjust how we handle newFormData. If instead of deleting the whole key-value pair, we changed the value to null but kept the key, then the commit handler might put null in for us. (Advantages: don't have to worry about accidentally wiping data, disadvantages: side effects?).
I'm not sure how that key-value deletion happens. In the screenshot below, the first console.log is the form data before I touched anything. The next two logs happened after I deleted BC Registry ID, and there's a moment where it's undefined before it disappears entirely:

This is a valid point and I need to investigate this option more since It can have unintentional side-effects.

@Sepehr-Sobhani This definitely looks more complicated than just checking against the set columns for a table can handle. Exploring how new_form_data is updated is definitely a good place to look. We might be able to do something with the json schemas, but that may not work since not all schemas are being pulled from the database yet.

Going to create a new clean branch for this issue with the new approach.


Remove deprecated key in desktop entry

The Encoding key is deprecated.
Ref: https://specifications.freedesktop.org/desktop-entry-spec/latest/apc.html
$ cat /usr/share/applications/web-eid.desktop
[Desktop Entry]
Encoding=UTF-8
Type=Application

Exec=web-eid %F
Icon=web-eid

Name=Web eID

Categories=Qt;Office;
$ desktop-file-validate /usr/share/applications/web-eid.desktop
/usr/share/applications/web-eid.desktop: warning: key "Encoding" in group "Desktop Entry" is deprecated

@abn, thanks a lot for your contribution, merged!


Assign a puppet manifest to a group host

**Question
I have a puppet running with a Foreman node to manage different hosts.
I installed and loaded puppet-filebeat with success and have access to class parameters in my Foreman web interface. I can assign different values to my parameters and assign the module to a host group to test and eventually to production host group to deploy.
The problem is I want to manage my SSL certificate so I created a manifest that looks like the one in the Wiki and created a mount point on my puppet server to gather the certs.
I don't know how to assign my puppet manifest to a Foreman hostgroup. I am not the person that installed the puppet server / foreman node so I am quite helpless with the product.
If you have any suggestion please let me know.
Thank you for your time

If I remember correctly, Foreman only allows you to assign classes (with parameters) to hosts. You'll need to create a new profile class (assuming you're using roles and profiles) that you can assign to the nodes within Foreman.
Sorry I can't help more - it's been a long time since I've managed anything puppet using Foreman.


NS Activities > NS International Work > Look like the projects stay in ongoing status after end date. It should be closed after the end date - Maybe this is only on the DFS server?

@udaynwa this is only in DFS server..we will get this fix.


fix(composer): custom tag rendered slightly off the center

Overview
Fix for when custom tags is chosen from the Tag Style drop down menu it rendered slightly off center.
Before
https://github.com/awslabs/iot-app-kit/assets/20994167/eeb05058-a06f-4774-8a91-22f1645d9707
After
https://github.com/awslabs/iot-app-kit/assets/20994167/fa9a1081-1acf-461b-99c1-f84dacbda68d
Verifying Changes
Scene Composer
For scene-composer package changes specifically, you can preview the component in the published storybook artifact. To do this, wait for the Publish Storybook action to complete below.

Click on the workflow details
Select the Summary item on the left
Download the zip file

To run the storybook build locally, you need a local static web server:
npm install -g httpserver
cd <Extracted Zip Directory>
httpserver

Then open the website http://localhost:8080 to run the doc site.
Legal
This project is available under the Apache 2.0 License.

I pulled & built the branch since I wanted to compare the icons used in the before video since the second looked like the axis block. Unfortunately, the icons are still not centering.
https://github.com/awslabs/iot-app-kit/assets/6610619/76e5b50b-db0c-49b8-a42e-f392bace3580

@mumanity This PR is solely focused on Custom Tags and if you see the before video the Tag when Tag style switched to custom style it moves off the center a little bit.

@mumanity This PR is focused solely on custom icons. In the before video you can see the whole tag move off center when you select a custom style Tag style option. This is fixed in this PR.


Could you release 1.3.4 ?

Hi,
This PR closed in August contains some updates we need for our deployment.
https://github.com/saddle/saddle/pull/63
Its content is currently only available in 1.3.4-SNAPSHOT. Could you please release 1.3.4 so that we can get the official jar from the official repo?
Thanks!
S

Will do so now. Apologies for falling off the map.


feat: release v5.19.0

Introduces ucc-gen init command to initialize empty add-on
Adds new line to globalConfig file, so it does not conflict with pre-commit stage for the add-ons
Bumps UCC UI to v5.19.0 (introduces linters, dropdown value fix and multilevel dropdown)

:tada: This PR is included in version 5.19.0 :tada:
The release is available on:

v5.19.0
GitHub release

Your semantic-release bot :package::rocket:


Synthesize

Needs:

magic stereo zero-crossing!
the resampling that @bmcfee and @blacker were discussing in Issue #5!
tests and documentation!

The resampling is happening at loadtime:  we're keeping raw_samples for rending 44,100 audio later, and making analysis_samples at 22,050 for use in various features.
(@bmcfee, does librosa really only resample things in mono?  Or can I not read?)

(@bmcfee, does librosa really only resample things in mono? Or can I not read?)

it should work in both stereo and mono.

@bmcfee, I was getting an error in valid_audio, which requires things to be in mono, it looked like - which is why I did two resamples.

@bmcfee, I was getting an error in valid_audio, which requires things to be in mono, it looked like - which is why I did two resamples.

Dang, you're right.  It works if you're doing resample-on-load, but librosa.resample itself is mono-only.
One more thing to add to the todo if i ever get around to writing a resampler library...

I think I have the zero-crossing fixes working!  I am going to start thinking about tests for all this, but please put your wisdom / numpy tricks / etc here.

...added several tests.  Still need to make a test for the zero-crossing stuff, I think, and do some stereo vs mono cleanup.

Well, so much for that cute test.  More to work on this week.
@bmcfee, it appears that when I write out a file with librosa, I can't read it back in without getting a BackEndError from audioread - but this is crazy, because you have a passing librosa test that does exactly that.

K, I think I've got everything.  I await feedback / fixes / better syntax, etc!


Membership system issue

Hi again,
When selling a membership product and creating an account upon purchase you have made it so that if the buyer uses an existing email but is not logged in, he's not automatically logged in and a notification is sent.
I think there are several problems here:
The user is not added to the proper group and the email you are sending states that they can now access the protected content which is not correct.
Shouldn't you either modify the email to warn them of an access or add them to the proper group and send them the existing email?
Frankly the best would be to simply ask them for their password during checkout if the email already exists.

This question is much for @Mesuva as he worked on a lot of that stuff. So are you saying there's currently no check that the email may already exist as a registered user when checking out for membership stuff?
If so, then yes - what are your thoughts on doing an ajax search for that email once they've moved to the next field and displaying a nice alert a la twitter?

Actually that's not exactky it. There's a check for the email BUT if the email is already in use and the user is not logged in, the user i not added to the proper group AND receives an email saying he now has access to protected content.
Since he wasn't added to the proper group he doesn't actually have the access the email says he has.
An ajax search is a good idea. Maybe a popup login form?

Well, I'd think that if the e-mail is already in use, then we should not allow them to proceed until they login. wouldn't that solve this?

that - or enter a different email

The plan here was that if a user already exists and someone checkouts as a guest, but uses the already used email, the account with that email gets the new user groups and is sent an email that indicates this. At this point there shouldn't be any automatic login. It's only for new accounts that automatic logins would happen, otherwise you could get into other people's accounts by placing transactions.
It sounds like I've just missed something with the allocation of user groups in this case, that would just be a bug I need to address. This code simply wasn't complete on this side of things, so sorry for any confusion here.
I'm not against changing this logic to flag if accounts are already in use, or prompting for login in certain cases, I just found that the logic does work fine for my Snipcart membership add-on, so copying that code in was the easiest thing to do at this point.

@Mesuva I think the idea is not automatic login. Like you said, that would be an unacczptable security risk.
What could happen is offering to login before proceeding.
On the other hand just allocating the proper group would work as well.
In your logic you first check if a customer is a guest and if he is you check if they are already registered. If they are not already registered you register them and load a new customer object with the proper customer ID.
BUT if they are already registered you only send an email.
So when we get to the part where groups are assigned, the already existing customer is still a guest and doesn't get added to the group.
What needs to happen is you get the user ID since they are already registered and load a new customer object for them as well.
So practically your
if(!$user)
should end right after login the user and the rest of the code should apply in both situation

I think that's too complicated.
If the Email already exists as a registered Users email, then the guest has to login or use another email-address.
exactly like the Twitter-screenshot brucewyne posted already here.

yes - I think in any case, if there's an email with an account associated with it. They either need to login before continuing, or change the email to proceed.

From a coding perspective, it's actually easier to just associate the groups with the exiting user, but not log them in, and then send them an email showing they have access (and showing something on screen indicating that they need to log in).  I've got that finished up and ready to submit as a pull request.
However, I'll admit that that approach came out of necessity from the Snipcart Memberships add-on, not out of preference. It does actually work fine in practice, but I can see how it's a little different to what might be expected.
I'll look to see if there is a straightforward way to do some user account checking as part of the checkout process, without having to do too much javascript based look-up-ing.

This process has been updated with a recent merge, it now throws a warning/error when trying to place an order as a guest, using an email address that has already been used.
Simpler and more inline with the thoughts above.

I think think this is now addressed? Let me know otherwise.


Error: Could not handle config_flow.async_step_dhcp

I've been using this integration happily for quite some time now. However, I often (one to a few times per day) find log errors referring to this component, and I don't know what this means. The integration seems to work fine otherwise. I want to know if this is something that I can fix. The error is below:
2024-01-20 11:10:11.552 INFO (MainThread) [custom_components.luxtronik2] Found device with hostname '' IP 'X.X.X.X'
2024-01-20 11:10:15.556 ERROR (MainThread) [custom_components.luxtronik2] Could not handle config_flow.async_step_dhcp DhcpServiceInfo(ip='X.X.X.X', hostname='', macaddress='XXXXXXXXXXXX')
Traceback (most recent call last):
  File "/config/custom_components/luxtronik2/config_flow.py", line 256, in async_step_dhcp
    broadcast_discover_ip, broadcast_discover_port = discover()[0]
                                                     ~~~~~~~~~~^^^
IndexError: list index out of range
2024-01-20 11:10:15.654 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/discovery_flow.py", line 96, in _async_start
    await gather_with_limited_concurrency(
  File "/usr/src/homeassistant/homeassistant/util/async_.py", line 187, in gather_with_limited_concurrency
    return await gather(
           ^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/util/async_.py", line 185, in sem_task
    return await task
           ^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 880, in async_init
    flow, result = await task
                   ^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 908, in _async_init
    result = await self._async_handle_step(flow, flow.init_step, data)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 396, in _async_handle_step
    if result.get("preview") is not None:
       ^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'get'

Again, the integration seems to work fine for the most part. If there are hiccups or glitches, I am unaware of them. I'm just trying to find out if I can fix this error reporting.

I think it is the same as #216 and #220

Good catch! I think so too. I'm closing it.


PYIC-7603: upgrade express js to latest version 5.0.0

Proposed changes

Update express js to 5.0.0

What changed

Updated  express js to 5.0.0
Removed express-async-errors
Updated body parser to latest version

Issue tracking

PYIC-7603

Thanks Shiv - do you think this lets us simplify any of our error handling?
I.e. we should be able to just throw errors from our middleware (like you would in Java, say), rather than needing to catch them and call next(err).

Yes, I am creating separate PR for it. Thought to create separate PR in case if there is breaking changes in migration and need to rollback.


METRON-1915: Disable vagrant hostmanager in the development guest

Contributor Comments
This makes node1 resolve to <IP_ADDRESS> inside of the VM, instead of <IP_ADDRESS> by not modifying /etc/hosts inside the guest.  One example of where this is important is that kafka only listens on <IP_ADDRESS>:6667.  The alternative is we could configure things to point to the IP instead of node1.
I left out fastcapa since I expected this may be useful in a dual VM scenario, but I haven't spun that up in years.
Pull Request Checklist
Thank you for submitting a contribution to Apache Metron.
Please refer to our Development Guidelines for the complete guide to follow for contributions.
Please refer also to our Build Verification Guidelines for complete smoke testing guides.
In order to streamline the review of the contribution we ask you follow these guidelines and ask you to double check the following:
For all changes:

[X] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[X] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[X] Has your PR been rebased against the latest commit within the target branch (typically master)?

For code changes:

[ ] Have you included steps to reproduce the behavior or problem that is being changed or addressed?

[ ] Have you included steps or a guide to how the change may be verified and tested manually?

[ ] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
mvn -q clean integration-test install && dev-utilities/build-utils/verify_licenses.sh

[ ] Have you written or updated unit tests and or integration tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

[ ] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book? If not then run the following commands and the verify changes via site-book/target/site/index.html:
cd site-book
mvn site

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.
It is also recommended that travis-ci is set up for your personal repository such that your branches are built there before submitting a pull request.

what breaks specifically?
having all the metron ports etc point to localhost seems to me that it would also break things or have conflicts etc.

typo in PR title

Specifically my issue is that when I attempt to connect to a kafka broker as a part of the bro plugin via node1:6667 it fails because it's not listening at the IP that hostmanager inserts into /etc/hosts.

@JonZeolla Can you provide steps for validating this?

Also reproduction steps

Bump, any update?

Nothing more to add than I mentioned before.  Happy to close to clean things up


feat: replace gunicorn with uvicorn to support Windows

This PR contains changes which will make storage test bench windows compatible.

Replaced gunicorn command with uvicorn to run the test bench. Uvicorn is windows compatible.

Done changes to convert wsgi server into an asgi server as uvicorn needs asgi server.

[x] Appropriate changes to README are included in PR

Some checks are failing right now, looking into it.

I checked out your branch and tried running the java conformance suite against it.
First I needed to change the CMD in the Dockerfile to work with the new flags.
Index: Dockerfile
<+>UTF-8
===================================================================
diff --git a/Dockerfile b/Dockerfile
--- a/Dockerfile	(revision 0858d6c0e9ca790114b75598e14baf27b9584855)
+++ b/Dockerfile	(date<PHONE_NUMBER>461)
@@ -20,9 +20,8 @@

 RUN python3 -m pip install -e .

-CMD ["gunicorn", \
-      "--bind", "<IP_ADDRESS>:9000", \
-      "--worker-class", "sync", \
-      "--threads", "10", \
-      "--access-logfile", "-", \
-      "testbench:run()"]
+CMD ["uvicorn", \
+      "testbench:run", \
+      "--host", "<IP_ADDRESS>", "--port", "9000", \
+      "--access-log"]
+

During test run, it seems that creating a bucket is failing due to a JSON Decoding error (full request collapsed below)
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/flask/app.py", line 1820, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python3.10/site-packages/flask/app.py", line 1796, in dispatch_request
    return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)
  File "/opt/storage-testbench/./testbench/common.py", line 797, in wrapper
    return response_handler(func(*args, **kwargs))
  File "/opt/storage-testbench/./testbench/rest_server.py", line 216, in bucket_insert
    bucket, projection = gcs_type.bucket.Bucket.init(flask.request, None)
  File "/opt/storage-testbench/./gcs/bucket.py", line 264, in init
    metadata = cls.__preprocess_rest(json.loads(request.data))
  File "/usr/local/lib/python3.10/json/__init__.py", line 346, in loads
    return _default_decoder.decode(s)
  File "/usr/local/lib/python3.10/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/local/lib/python3.10/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Full request log
2022-10-04 18:42:21,572 CONFIG   com.google.api.client.http.HttpTransport           - -------------- REQUEST  --------------
POST http://localhost:9000/storage/v1/b?project=conformance-tests&projection=full
Accept-Encoding: gzip
User-Agent: non-test/ (TestRetryConformance/1-[return-503_return-503]-storage.objects.get-41) java-conformance-tests/ gcloud-java/ Google-API-Java-Client/2.0.0 Google-HTTP-Java-Client/1.42.2 (gzip)
x-goog-api-client: gl-java/1.8.0 gdcl/2.0.0 linux/5.18.16 gccl-invocation-id/160ef385-82d4-4831-b207-41584f7782ee
Content-Type: application/json; charset=UTF-8
Content-Encoding: gzip

2022-10-04 18:42:21,572 CONFIG   com.google.api.client.http.HttpTransport           - curl -v --compressed -X POST -H 'Accept-Encoding: gzip' -H 'User-Agent: non-test/ (TestRetryConformance/1-[return-503_return-503]-storage.objects.get-41) java-conformance-tests/ gcloud-java/ Google-API-Java-Client/2.0.0 Google-HTTP-Java-Client/1.42.2 (gzip)' -H 'x-goog-api-client: gl-java/1.8.0 gdcl/2.0.0 linux/5.18.16 gccl-invocation-id/160ef385-82d4-4831-b207-41584f7782ee' -H 'Content-Type: application/json; charset=UTF-8' -H 'Content-Encoding: gzip' -d '@-' -- 'http://localhost:9000/storage/v1/b?project=conformance-tests&projection=full' << $$$
2022-10-04 18:42:21,578 CONFIG   com.google.api.client.http.HttpTransport           - Total: 63 bytes
2022-10-04 18:42:21,579 CONFIG   com.google.api.client.http.HttpTransport           - {"name":"224219_s001-503_503-m041_bkt1","retentionPolicy":null}
2022-10-04 18:42:21,582 CONFIG   com.google.api.client.http.HttpTransport           - -------------- RESPONSE --------------
HTTP/1.1 500 Internal Server Error
date: Tue, 04 Oct 2022 22:42:21 GMT
content-length: 1281
server: uvicorn
content-type: application/json

2022-10-04 18:42:21,590 CONFIG   com.google.api.client.http.HttpTransport           - Total: 1,281 bytes
2022-10-04 18:42:21,591 CONFIG   com.google.api.client.http.HttpTransport           - {"error":{"code":500,"message":"Traceback (most recent call last):\n  File \"/usr/local/lib/python3.10/site-packages/flask/app.py\", line 1820, in full_dispatch_request\n    rv = self.dispatch_request()\n  File \"/usr/local/lib/python3.10/site-packages/flask/app.py\", line 1796, in dispatch_request\n    return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)\n  File \"/opt/storage-testbench/./testbench/common.py\", line 797, in wrapper\n    return response_handler(func(*args, **kwargs))\n  File \"/opt/storage-testbench/./testbench/rest_server.py\", line 216, in bucket_insert\n    bucket, projection = gcs_type.bucket.Bucket.init(flask.request, None)\n  File \"/opt/storage-testbench/./gcs/bucket.py\", line 264, in init\n    metadata = cls.__preprocess_rest(json.loads(request.data))\n  File \"/usr/local/lib/python3.10/json/__init__.py\", line 346, in loads\n    return _default_decoder.decode(s)\n  File \"/usr/local/lib/python3.10/json/decoder.py\", line 337, in decode\n    obj, end = self.raw_decode(s, idx=_w(s, 0).end())\n  File \"/usr/local/lib/python3.10/json/decoder.py\", line 355, in raw_decode\n    raise JSONDecodeError(\"Expecting value\", s, err.value) from None\njson.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)\n"}}

I checked out your branch and tried running the java conformance suite against it.
First I needed to change the CMD in the Dockerfile to work with the new flags.
Index: Dockerfile
<+>UTF-8
===================================================================
diff --git a/Dockerfile b/Dockerfile
--- a/Dockerfile	(revision 0858d6c0e9ca790114b75598e14baf27b9584855)
+++ b/Dockerfile	(date<PHONE_NUMBER>461)
@@ -20,9 +20,8 @@

 RUN python3 -m pip install -e .

-CMD ["gunicorn", \
-      "--bind", "<IP_ADDRESS>:9000", \
-      "--worker-class", "sync", \
-      "--threads", "10", \
-      "--access-logfile", "-", \
-      "testbench:run()"]
+CMD ["uvicorn", \
+      "testbench:run", \
+      "--host", "<IP_ADDRESS>", "--port", "9000", \
+      "--access-log"]
+

During test run, it seems that creating a bucket is failing due to a JSON Decoding error (full request collapsed below)
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/flask/app.py", line 1820, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python3.10/site-packages/flask/app.py", line 1796, in dispatch_request
    return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)
  File "/opt/storage-testbench/./testbench/common.py", line 797, in wrapper
    return response_handler(func(*args, **kwargs))
  File "/opt/storage-testbench/./testbench/rest_server.py", line 216, in bucket_insert
    bucket, projection = gcs_type.bucket.Bucket.init(flask.request, None)
  File "/opt/storage-testbench/./gcs/bucket.py", line 264, in init
    metadata = cls.__preprocess_rest(json.loads(request.data))
  File "/usr/local/lib/python3.10/json/__init__.py", line 346, in loads
    return _default_decoder.decode(s)
  File "/usr/local/lib/python3.10/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/local/lib/python3.10/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Full request log

Thanks Ben, I'll look into it.

Closing this. Will raise another PR.


Page title “Creating a single control-plane cluster with kubeadm” could mislead

This is a Bug Report

Problem:
The title of https://kubernetes.io/docs/setup/production-environment/tools/kubeadm/create-cluster-kubeadm/ used to be
“Creating a single control-plane cluster with kubeadm” and PR #14594 changed that to “Creating a single control-plane cluster with kubeadm” to match guidelines on terminology. However, the current title does not read as idiomatic English and could also mislead readers.
All Kubernetes clusters should have exactly 1 control plane. You might have 0 control planes I suppose, in which case you need to fix that so that your workloads remain managed. You can't have 2 control planes for 1 cluster. What you can do is run your control plane across multiple hosts and even zones, to provide resilience. For that reason, it isn't the control plane that is single; instead, this page explains how to deploy a cluster where the control plane runs on only one computer.
Proposed Solution:
Change the title of the page. Pick wording that makes it clear that it's a simple cluster setup and isn't suitable for keeping your cluster within a low error budget (this is because if the one control plane host fails then you lose all control loops, the persisted data in etcd, and the Kubernetes cluster API). You don't need to go into all that detail so long as the reader is aware that another page explains how to do roughly the same thing but with availability taken into consideration.
Also reword the following text from the body of the page:

Install a single control-plane Kubernetes cluster or high-availability cluster

Optionally, add or revise introductory text to clarify the limitations of what this page is teaching. For example, consider if text “a minimum viable Kubernetes cluster that conforms to best practices” is the right fit?
Page to Update:
https://kubernetes.io/docs/setup/production-environment/tools/kubeadm/create-cluster-kubeadm/
Additional Information:
/sig cluster-lifecycle
/language en
/kind cleanup
/priority backlog
/good-first-issue
If you're looking to pick this up as a first issue and would like advice, go ahead and reply - a Kubernetes contributor will help you to get started.
Also, because this issue is not especially urgent, if you have already made a few contributions to this documentation, please consider leaving this one as a starting point for someone new.

Change the title of the page. Pick wording that makes it clear that it's a simple cluster setup and isn't suitable for keeping your cluster within a low error budget (this is because if the one control plane host fails then you lose all control loops, the persisted data in etcd, and the Kubernetes cluster API). You don't need to go into all that detail so long as the reader is aware that another page explains how to do roughly the same thing but with availability taken into consideration.

my vote goes for:
"Creating a cluster with kubeadm"
and the page should link to the HA guide which is an optional extension.

Install a single control-plane Kubernetes cluster or high-availability cluster

Optionally, add or revise introductory text to clarify the limitations of what this page is teaching. For example, consider if text “a minimum viable Kubernetes cluster that conforms to best practices” is the right fit?

"Creating a minimum viable Kubernetes cluster that conforms to best practices", seems good to me.
The HA guide (link under "high-availability cluster") can be moved to under "higher availabilty" in this sentence:
"Use multiple control-plane nodes. You can read Options for Highly Available topology to pick a cluster topology that provides higher availabilty."
it's currently a bullet item here:
https://kubernetes.io/docs/setup/production-environment/tools/kubeadm/create-cluster-kubeadm/#resilience

I think I can help with this one, if it's OK
/assign

Anyone is welcome to open a pull request that makes the changes suggested, or otherwise helps get the issue done.


[Event Request] Codeunit 5772 Whse.-Transfer Release - OnCheckReopenOutbound

Hi Team,
In codeunit 5772 we need possibility to replace and avoid using standard code. Having event in procedure Reopen, would be a perfect solution :

[IntegrationEvent(false, false)]
local procedure OnCheckReopenOutbound(var TransHeader: Record "Transfer Header")

Can't you use the existing OnAfterReopen()?

Moved to OnAfterReopen()


Add case for Edge in 'IE' detection

Should fix Issue 112
Or observations so far are that loading contentDocumentData from a blob (still) doesn't appear to work and this is an example of an Edge user agent:

Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36 Edge/12.10240

@chine   Thanks for posting this! I will ask some of the other contributors to review and we can merge it as soon as the develop branch is open for business again - early next week.


Add open source docs

Adds docs covering contribution, security and code of conduct guidelines as well as Unlicense statement.

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

Comparison is base (9384ccd) 74.30% compared to head (7253f7b) 74.30%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #10   +/-   ##
=======================================
  Coverage   74.30%   74.30%
=======================================
  Files           1        1
  Lines         144      144
=======================================
  Hits          107      107
  Misses         37       37

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


flushing delays with TCP_CORK ?

I am exploring the use of TCP_CORK as a way of reducing packet fragmentation and thus bandwidth.
The flip side seems to be the increased latency (up to 200ms) in case of not having enough data for a full packet - which is a common scenario. The solution is to "uncork" and "recork" a corked connection to force sending, as described here: http://baus.net/on-tcp_cork/
Looking at Epoll package code it appears that Netty does not currently do this on a regular channel flush. I suppose it could be possible to manually call Native.setTcpCork after the flush, but I wonder if there ought to be a higher level API for this (or a flushing option) as well as a single native call that would both uncork / recork the channel in a single JNI invocation ...

@dmk23 there is nothing high-level implemented via us.

Should this be marked as a possible feature suggestion?

At least we could expose that operation somewhere in EpollSocketChannel?

@trustin you can already doing this by just cal it during flush()

Is it actually done right now? I saw the calls to Native.setTcpCork only on initial channel setup when I searched through the code - not in any place related to flushing...
It also seems to me it ought to be just an option of flush vs. something that is always enforced?

@dmk23 I may be confused but you can just do channel.config().set* for changing socket options on the fly. What issue you have ?

Yes, this can be done manually right now.
Just was wondering if it should/could be made an explicit option in the flushing API(s)... Since there is not much docs on TCP_CORK option the existing behavior / solution was not readily findable...
This is not a major thing, but might be an enhancement for clarity purposes...


UrlCapture have no getCapturedFileName method

This issue was imported from Closure Library's previous home at http://closure-library.googlecode.com
The link to the original issue is:
https://code.google.com/p/closure-library/issues/detail?id=159

Gears is obsolete


[BEAM-11862] Updates ExternalTransformFinder to consider external leaf transforms

This cherry-picks https://github.com/apache/beam/pull/14293 to 2.29.0 release branch.

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

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

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

GitHub Actions Tests Status (on master branch)

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

Run XVR_Dataflow PostCommit

XVR_Dataflow PostCommit should pass if containers are already available. I already verified Kafka.

Not available yet, if I have to do anything to make it available. I'm also happy to just re-run as part of RC verification anyhow.

Seems like it passed anyways (probably branch runs tests using HEAD containers after your script updates).

Cutting the branch leaves SNAPSHOT version but does update Dataflow v1 container (though I could go either way on this): https://github.com/apache/beam/blob/85e93ca3a01990e8da1e10af7826662012c06d74/runners/google-cloud-dataflow-java/build.gradle#L48
I don't know much about where other containers are specified. I am very open to adjusting the scripts more if it improves stabilization of releases and still works well with the idea of long-lived release branches.
I'll take another look on Monday.

Seems like you still have "2.29.0.dev" here so Python (and Python x-lang) would use HEAD containers.
https://github.com/apache/beam/blob/release-2.29.0/sdks/python/apache_beam/version.py#L20
This is different from previous versions.
https://github.com/apache/beam/blob/release-2.28.0/sdks/python/apache_beam/version.py

Yes, that is intentional. What I mean is that I have not looked up where that value is inspected to decide SDK environment container image. Most likely the Dataflow container should also not be modified when cutting the release branch.


Override CaseChecker configuration

Hi,
After getting the Kotlin & Java upgrade I encounter an issue with the override of the CaseChecker defined in the reference.conf of the core module.
I redefine two properties to change the case check in a reference.conf inside my ruleset module like this:
CaseChecker.propertyNames.allow = [${CaseChecker.cases.camelCase}]
CaseChecker.queryParameterNames.allow = [${CaseChecker.cases.camelCase}]

To make it work, in the build.gradle.kts, I added my ruleset dependency before the core one to ensure typesafe loaded mine first:
dependencies {
    val jadlerVersion = "1.3.0"

    implementation(project(":zally-ruleset-afkl"))
    implementation(project(":zally-core"))

It was working fine until the upgrade though I have no idea what exactly changed the behavior.
I know it is not a Zally's issue but I was wondering if anyone had the same need as me and if so how they manage to handle it.
Thank you for your help
Jean-Gaël or JG

Hi
I found a workaround by adding the two lines in the rules-config file inside the server module but I'd rather be able to leave it in the ruleset module if possible.

Hi @jeangael-domine ,
It is hard to say what has been changed. Java / Kotlin migration didn't touch configuration modules. The configuration Zally is implemented using typesafe config. The problem here that zally-core and your module use the same mechanism for default settings - define reference.conf file. The library doesn't guarantee in which order reference.conf files would be merged and resolved. Perhaps, after the upgrade the resolution order has been changed and now the reference.conf from zally-core merged after the reference.conf from your module. I would say you were lucky enough since this moment by using undocumented librarys' feature 😀
The workaround you have found is actually one of the solutions proposed by the config library. The final word is always from the final config.
I think in general we should review our current configuration setup especially in cases with custom rule sets which reuse parts of existing configuration. The same will happen with two existing rule sets (zally-ruleset-zally and zally-ruleset-zalando) if they will have an overlapping configuration.

Hi,
Thanks for your quick reply @vadeg
What made me think it was not completely sheer luck that it was working is that if I reset to the commit preceding the Java/Kotlin upgrade, everything was loading in the right order.
But I could not figure out what such an upgrade would impact this behavior.
Anyway it is true that I was not really confident about not having bugs in the future with my solution of changing the order of declaration in the build file.
I'll stick to this solution then even if it would have made more sense to me to be able to place the conf in the ruleset file and not the server. But the lib does not make it possible :)
Thanks for your explanations


Using spread operator may generate non-ES5 compatible code

Transpiling ECMAScript 2015 code that uses the spread operator, to ES5, using the --language_out=ES5 flag leads to a non-ES5 compatible code being generated.
This example shows a simple script which uses the spread operator. The output contains an automatically added polyfill for Object.defineProperty, which throws an exception when the third parameter, a descriptor, has either get or set property defined.
This exception is invalid in ES5 context. Probably, the Google Closure Compiler should provide different implementations for the above polyfill, according to the --language_out parameter.

@shicks Looks like this part:
https://github.com/google/closure-compiler/blob/20e8f03d4c9b67ca6654e049c74e0b4d978ab012/src/com/google/javascript/jscomp/js/util/defineproperty.js#L42
needs to be conditional?

It is conditional on the existence of Object.defineProperties, which should be defined in all ES5 browsers.  So it's just a case of useless code when you know you're compiling to ES5; it's not actually incorrect.  We could certainly be smarter about what code we inject, but currently there's no mechanism at all to inject different code based on language out.
Additionally, $jscomp.defineProperty is not actually stored back to Object.defineProperty, and we never call it directly with anything that has a getter or setter.  So this codepath is actually completely dead anyway, and might as well be deleted - we'll save 30 or so bytes from most binaries as a result.

Hello @shicks,
You've closed the issue, but there is no commit associated with it that removes the dead codepath, as you've suggested. However, currently, the presence of throw new TypeError('ES3 does not support getters and setters.'); will cause issues when Google Closure Compiler is used in the Dojo build ecosystem.
Can you please resolve the issue by either conditionally change the polyfill's contents according to the --language_out parameter, or remove the dead codepath for good.
Many thanks!

@sindilevich This was closed because your original case is not reproducible. This should only throw on browsers which do not have Object.defineProperties. If you can provide an environment in which this creates an error, we can re-open.


Fix some clashes between new and old symmetry.

#276 updated the representation of symmetry from plain strings to an enumeration. The intent was that the new SymmetryType class could handle string or enumeration representations seamlessly to minimize breakage of old code that used the string representations.
While the strings still worked with little to no modification in most places, some pieces of code with the older format flew under the radar because the problems weren't caught by the unit tests. Also, the .lower() was misplaced in the fromStr() constructor for SymmetryType in a way that would break case insensitivity.

In the grid editor armi grids when I try to make a new 1/3 grid, I get this tb:
Traceback (most recent call last):
  File "c:\users\ntouran\codes\armi-framework\armi\utils\gridEditor.py", line 368, in onNew
    self.parent.new(event)
  File "c:\users\ntouran\codes\armi-framework\armi\utils\gridEditor.py", line 1503, in new
    gridBp = dlg.getGridBlueprint()
  File "c:\users\ntouran\codes\armi-framework\armi\utils\gridEditor.py", line 1790, in getGridBlueprint
    symmetry = geometry.SymmetryType.fromStr(
TypeError: fromStr() takes 2 positional arguments but 4 were given

Thanks Nick. Should be the default constructor for SymmetryType there but I accidentally left it as fromStr()


Adds a flag, to allow the client dev to remove the production tips info

See PR https://github.com/vuejs/vue/pull/4907
Scott

As seen in https://github.com/vuejs/vue/issues/4907 @smolinari Can you close it please? 😛

Hehehe..didn't know I could close my own PRs. Learning, learning, learning...
Scott


chore(functions/file_system): Update file_system sample to use gen2 signature

Migrate https://cloud.google.com/functions/docs/samples/functions-concepts-filesystem go sample to use functions framework.
Tested by:

Running local test
Deploying and invoking function using curl
stdout:

Thanks for reviewing. Can a repo admin trigger the merge? I do not have write access to the repo.


flutter web netlify deploy Error: unable to locate asset entry in pubspec.yaml

logs
2:29:34 PM: Setting "enable-web" value to "true".
2:29:34 PM: You may need to restart any open editors for them to read new settings.
2:29:38 PM: Downloading Web SDK...                                           2,697ms
2:29:42 PM: Running "flutter pub get" in repo...                             2,173ms
2:29:42 PM: Building without sound null safety
2:29:42 PM: For more information see https://dart.dev/null-safety/unsound-null-safety
2:30:20 PM: Compiling lib/main.dart for the Web...
2:30:20 PM: Error: unable to locate asset entry in pubspec.yaml: "lib/assets/fonts/CAIRO-REGULAR.ttf".
2:30:20 PM: Target web_release_bundle failed: Exception: Failed to bundle asset files.
2:30:20 PM: Compiling lib/main.dart for the Web...                             37.9s
2:30:20 PM: Exception: Failed to compile application for the Web.
2:30:20 PM:
2:30:20 PM: ────────────────────────────────────────────────────────────────
2:30:20 PM:   "build.command" failed
2:30:20 PM: ────────────────────────────────────────────────────────────────
2:30:20 PM:
2:30:20 PM:   Error message
2:30:20 PM:   Command failed with exit code 1: if cd flutter; then git pull && cd .. ; else git clone https://github.com/flutter/flutter.git; fi && ls && flutter/bin/flutter doctor && flutter/bin/flutter clean && flutter/bin/flutter config --enable-web && flutter/bin/flutter build web --release --dart-define=FLUTTER_WEB_USE_EXPERIMENTAL_CANVAS_TEXT=true --enable-experiment=non-nullable --no-sound-null-safety
2:30:20 PM:
2:30:20 PM:   Error location
2:30:20 PM:   In Build command from Netlify app:
2:30:20 PM:   if cd flutter; then git pull && cd .. ; else git clone https://github.com/flutter/flutter.git; fi && ls && flutter/bin/flutter doctor && flutter/bin/flutter clean && flutter/bin/flutter config --enable-web && flutter/bin/flutter build web --release --dart-define=FLUTTER_WEB_USE_EXPERIMENTAL_CANVAS_TEXT=true --enable-experiment=non-nullable --no-sound-null-safety
2:30:20 PM:
2:30:20 PM:   Resolved config
2:30:20 PM:   build:
2:30:20 PM:     command: if cd flutter; then git pull && cd .. ; else git clone https://github.com/flutter/flutter.git; fi && ls && flutter/bin/flutter doctor && flutter/bin/flutter clean && flutter/bin/flutter config --enable-web && flutter/bin/flutter build web --release --dart-define=FLUTTER_WEB_USE_EXPERIMENTAL_CANVAS_TEXT=true --enable-experiment=non-nullable --no-sound-null-safety
2:30:20 PM:     commandOrigin: ui
2:30:20 PM:     publish: /opt/build/repo/build/web
2:30:21 PM: Caching artifacts
2:30:21 PM: Started saving build plugins
2:30:21 PM: Finished saving build plugins
2:30:21 PM: Started saving pip cache
2:30:21 PM: Finished saving pip cache
2:30:21 PM: Started saving emacs cask dependencies
2:30:21 PM: Finished saving emacs cask dependencies
2:30:21 PM: Started saving maven dependencies
2:30:21 PM: Finished saving maven dependencies
2:30:21 PM: Started saving boot dependencies
2:30:21 PM: Finished saving boot dependencies
2:30:21 PM: Started saving rust rustup cache
2:30:21 PM: Finished saving rust rustup cache
2:30:21 PM: Started saving go dependencies
2:30:21 PM: Finished saving go dependencies
2:30:23 PM: Build failed due to a user error: Build script returned non-zero exit code: 2
2:30:23 PM: Creating deploy upload records
2:30:23 PM: Failing build: Failed to build site
2:30:23 PM: Failed during stage 'building site': Build script returned non-zero exit code: 2
2:30:23 PM: Finished processing build request in 2m10.850085066s

without assets the website got deployed
when i added assets..its showing error

Hi @Raghupathi22
Are you sure you're providing the correct asset path in pubspec.yaml?
lib/assets/fonts/CAIRO-REGULAR.ttf
Assets are not located in the lib folder usually, if you have assets in the root directory, then remove lib/
Also, please reproduce the issue without netfify
Can you please provide your flutter doctor -v, a complete reproducible minimal code sample, and complete flutter build web --verbose logs running the minimal code sample?
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Palestinian territories

Subject: Palestinian territories
Description:
I mark I have traveled to Palestinian territories
Results:
in the map which shows the countries I have traveled to, they are coloured in, but in this case it is only the gaza strip, but not west bank that is marked in orange.
Backend version: develop-e7a6ee78
Frontend version: develop-e7a6ee78
User Agent: Mozilla/5.0 (iPad; CPU OS 17_4 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) CriOS/116.0.5845.118 Mobile/15E148 Safari/604.1
Page: https://couchers.org/profile/edit/about
User: faireality_x (24505)

This is for illustrative purposes only, and we do not control this particular set of geo-data, it’s direct from the react package.


Use pgdg main, not 9.5

PostgreSQL 9.5 is still in alpha. Use main instead so we can get to other versions.

Or you could enable both:
deb http://apt.postgresql.org/pub/repos/apt/ precise-pgdg 9.5 main


The difference between rxd-modpack-xtended and rxd-modpack

I have some questions. Is this the difference between the simple full version and the simplified version, or two different things? I want to know which one I should install?, What is the difference between them?

xtended is a lite modpack while no xtended is fully featured modpack


changes does not appear

i tried this project (0.0.4 version), it works but when i made changes in app.js file
(changed text    'Hello, '    to    'Hellox, '     ), it does not appear in browser.
i did maven clean install, restart application but i does not show.

@keremozkanvivense This repository is only for demo and should not be used as is. It contains a lot outdated dependencies.


Google Analytics (Site Kit)

Automatic insertion of Google Analytics tracking code when Site Kit is installed

Was looking into this and originally had the idea that perhaps we can add a HTML comment into the story content (e.g.<!--WEB_STORIES_ANALYTICS-->) which could then be replaced as needed in the server side, however, it doesn't seem to be possible to add a HTML comment like this via React.
It is possible to add a tag and then set it's content (dangerouslySetInnerHTML) as the comment, however, only specific tags are allowed as a children for amp-story, so can't really do that either.
I'm now thinking that the best way could be creating a new REST endpoint for analytics which would currently include the GA tag config data only, however, it would pass an array of configs for enabling adding other analytics information, too, in the future.
Thoughts? cc @spacedmonkey @dvoytenko

Hmm, just remembered that the analytics code should be rendered dynamically anyway and not statically saved to the content, so using REST endpoint for saving it directly into the content wouldn't work either.

Do we have an example of Google Analytics code that we need to insert?

@dvoytenko #616 is currently using the logic that is used for AMP Plugin by Site Kit, however, would be good to specify it specific for Stories, who should be deciding the default configuration?

@pbakaus Tagging you here as well for the default configuration (which is currently a replication of what SiteKit adds to AMP Plugin).

The default config is specified here: https://docs.google.com/document/d/1q8rkYYMKZh3R2eqkpwgGkx6QX0oSXoLYtIO6ml1BFD0/edit#heading=h.xqcejitqohdk

Ok, so if I understand it right, we just need to replace the "YOUR_GOOGLE_ANALYTICS_ID" in the amp-analytics? If so, we have a few options:

Get the YOUR_GOOGLE_ANALYTICS_ID to the client side and generate the final JSON with it on the client side. This will produce a fully-valid AMP, but would require re-saving the story if the ID changes.
Build the full config client side, but use substitution marker within. E.g. "gtag_id": "[YOUR_GOOGLE_ANALYTICS_ID]". Then we can replace this marker server-side. We could do it either during save/publish time or even serving time. Makes it easy to control server-side, but the analytics config is not truly valid.
Just output an empty <amp-analytics> tag and replace it whole on the server at save/publish or serve time. This will leave more options for controlling this server-side, especially if we decide to do it at the serving time. But we'll have to do bunch of tries for what an "empty" <amp-analytics> would work: completely empty or maybe with an empty {} config. Replacing such an HTML fragment will also be difficult, given unpredictably whitespaces.

I'd say we should start with either 1 or 2.

@dvoytenko
For 1: Note that this issue is specific to Site Kit meaning that we should track when the ID in Site Kit changes and then update all the stories when the analytics ID changes, not sure if that's the best option.

Note that it also needs to be filterable on the server-side, e.g. being able to remove the Site Kit analytics tag fully, so the second option also doesn't seem to be the best. Also, currently, the rest of the saving / server-side logic is based on the assumption that it's not necessarily WordPress, it would be better not to save any invalid content assuming that we're replacing it within WP.

For this specific Site Kit integration issue, the fully server-side approach in #616 still seems to make sense to me, it just needs updating the analytics output now, PTAL.
For the AMP Stories internal Google tag ID (non-Site Kit) global settings, we could consider option 1, however, if the GA is global through all the stories, this would mean that we'd need to re-save every single story when that changes, not sure if that's a good option either, considering that some stories might be from a previous version and have some differences? Also, triggering resave for all the stories seems potentially a bit much. This would be a good discussion under #570.

@miina, to clarify then, what's your proposal for this?

I guess also, another option, we could look for the <amp-story ...> and insert analytics tag right after it?

@dvoytenko The proposal is in #616 which looks for </amp-story> tag and inserts amp-analytics tag if the configuration is OK and Site Kit plugin is enabled. The PR should be ready for review, perhaps that's easier to look at than describing it here in text :)

oh. awesome!

Hey all, I just randomly found this Site Kit integration while browsing the plugin code. There's currently a lot of code duplicated there from Site Kit. Maybe that's intentional if you want to keep the story template completely locked for extensions (which may break AMP compatibility) - but if not, I think it would be a more elegant solution to introduce an action here, something like web_story_analytics. Site Kit could easily hook into that action.
Then all that would be left to the Web Stories plugin would be to in turn hook into Site Kit's googlesitekit_gtag_amp_opt filter to conditionally include all the story-specific events. This would overall reduce duplicate code in here and also decrease the chance for breakage from conflict, should the Site Kit plugin logic around rendering the snippet change.
cc @swissspidy

@felixarntz Thanks for the note. Yes, it's currently purposefully duplicating some code, however, there's a @todo comment in the code to note that the logic should come directly from Site Kit. The action introduced for that is web_stories_print_analytics.
The GA current implementation is partial and will be continued in #570, however, it might be good to create a separate follow-up issue for addressing this specific (Site Kit + GA) improvement separately -- will add one.

Added https://github.com/google/web-stories-wp/issues/1168


Double-check Hydra Version

Hi authors,
Thank you for providing the code base.
I am wondering what hydra version you are using. I am getting the following exception:
  File "xxx/anaconda3/envs/Crystallization/lib/python3.7/site-packages/hydra/_internal/utils.py", line 644, in _locate
    obj = getattr(obj, part)
AttributeError: module 'diffcsp' has no attribute 'pl_data'

I tried hydra-core==1.3.2 and 1.2.0. Neither can work.

Hi,
We use hydra-core==1.1.0 as the hydra version.


Cow: Watch<PHONE_NUMBER>(?#NO NorAm)

Cow requests the watch of the watch_number<PHONE_NUMBER>(?#NO NorAm). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Dragonrage in Charcoal HQ


[MUGEN][Doc] Fix tutorial links of retrieval example

Stack from ghstack:

#356
#357
-> #355
#354

Summary:
As titled.
Test Plan:
Verified that the links would work in the public domain.
Differential Revision: D40421055

cc @sophiazhi

Codecov Report

:exclamation: No coverage uploaded for pull request base (gh/langong347/65/base@2f9124a). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@                   Coverage Diff                    @@
##             gh/langong347/65/base     #355   +/-   ##
========================================================
  Coverage                         ?   93.40%
========================================================
  Files                            ?       54
  Lines                            ?     3258
  Branches                         ?        0
========================================================
  Hits                             ?     3043
  Misses                           ?      215
  Partials                         ?        0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@langong347 has imported this pull request.  If you are a Meta employee, you can view this diff on Phabricator.

@langong347 has imported this pull request.  If you are a Meta employee, you can view this diff on Phabricator.

@langong347 has imported this pull request.  If you are a Meta employee, you can view this diff on Phabricator.


Offline Use?

Hi, thanks for the great work so far!
I want to create different packages for different targets while being offline and was wondering if and how I could use nwjs-builder-phoenix in that scenario.
If all else fails I might set up a local mirror, but I don't know which files I need and where to put them (obviously, I don't want to clone a complete mirror site, only the required versions/targets).
Thanks for your support!
Ole.

You can download binaries from the mirror and save it with original filenames in ./node_modules/nwjs-builder-phoenix/caches/. As long as you don't specify a symbol for version like legacy or stable, an Internet connection shouldn't be a must have.
I didn't test it yet, but if anything doesn't work as above please report back and I will fix.

Thanks, that seems to work! :)
Best,
Ole.


Is this PSR-4 Autoload compatible ?

we were trying to integrate routing system into our small php system to test the routing performance. However, we found it really hard to load classes with namespace.
Those namespace was defined and auto-loaded as per PSR-4.
Finally, we achieved class loading by custom code in route.php. Is there any plan to make it PSR-4 compatible ?
Thank you

Are you trying to load custom classes into Flight or loading Flight classes into your app?

I am trying to load Flight Classes into my app.
Though, i got ride of this situation, but because you're the developer and read every code of this beautiful app i though i can talk with you regarding this.
Thanks.

The classes are all in standard folders so your autoloading function should be able to load them. So use \flight\net\Router would refer to the file in flight\net\Router.php.


ToLabel doesn't update on model change

Hi! I had a conversation with @labordep about this behavior that I consider a bug in ToLabel but may be not... so I'm reporting it to know @plantec opinion.
Code:
model := ToLabelModel new.
model text:
    ((String loremIpsum: 50) asRopedText
         fontSize: 20;
         bold;
         foreground: Color blue;
         yourself).
aToLabel := model onWidget.
aToLabel openInOBlSpace.

Then evaluate any of these lines:
aToLabel text foreground: Color random.
aToLabel text clearAttributes: 1 to: 6 if: [ :_ | true ].
(aToLabel text from: 3 to: 10) foreground: Color random.

And the label only refreshes after sending aToLabel textChanged.
I saw that foreground: announces BlTextAttributeAdded, and looking for references I discovered that AlbTextEditor subscribes to it, to react. So that may be the fix.
In the previous snippet, I added:
aText := aToLabel text.
aText when: BlTextStringsInserted send: #textChanged to: aToLabel.
aText when: BlTextsDeleted send: #textChanged to: aToLabel.
aText when: BlTextAttributeAdded send: #textChanged to: aToLabel.
aText when: BlTextAttributesRemoved send: #textChanged to: aToLabel.

and it was fixed.

BTW, it may make sense to create a superclass in common for these BlText change announcements.

Talking with Enzo I realized the basic BlTextElement suffers the same issue... may this behavior be intentional?
textElement :=
    BlTextElement new
        text: 'A' asRopedText;
        yourself.
textElement openInNewSpace.

textElement text
    when: BlTextStringsInserted send: #textChanged to: textElement;
    when: BlTextsDeleted send: #textChanged to: textElement;
    when: BlTextAttributeAdded send: #textChanged to: textElement;
    when: BlTextAttributesRemoved send: #textChanged to: textElement.

[ 10 timesRepeat: [
    | aRandomSize |
    aRandomSize := (SharedRandom globalGenerator nextIntegerBetween: 1 and: 10) * 30.
    textElement text fontSize: aRandomSize.
    0.5 seconds wait ] ] fork

I'm not sure. I should check further why it is not already done like that.
Btw, the model stuff should not be used. I'm not sure these classes will remain in the futur (except for ToAlbum). Maybe spec adapters will play the role of these kind of view-model

I do `aToLabel textChanged. after each text attribute modifications.
for me its make sens. It's like modification of a png image pixel matrix : the top system don't need to observe each of theses evolutions and this is better to notify manually when this is ok.

Hi,
Working in Spec-Toplo, created a subclass of ToLabel, and added this initialize:
initialize

	super initialize.

	self whenTextReplacedDo: [ :new :old |
		new
			when: BlTextStringsInserted send: #textChanged to: self;
			when: BlTextsDeleted send: #textChanged to: self;
			when: BlTextAttributeAdded send: #textChanged to: self;
			when: BlTextAttributesRemoved send: #textChanged to: self.
		old unsubscribe: self ]

to fix this problem

@labordep okay, it's the approach as a List widget, that isn't subscribed to the items collection changes... I mean, similarly, the user has the responsibility to tell the widget it was to refresh.

Yes, it's a valid objection. I could rollback if it was really bad idea. The requirements from Spec to adapt a ToLabel to be a SpLabelPresenter are:

the Spec label can be enabled or disabled, and the Toplo widget which changes foreground color of the text
the Spec label sets and retrieves Strings from the Toplo widget
First, I had an adapter for (2) and a dresser for (3), but it looked better as subclass of ToLabel

it's good to  implement spec adapters. we can see was is lacking. regarding enable/disable, it will be integrated for all widgets. The look change will be implemented through a theme/skin mechanism.
For string access, you can add it in ToLabel.
Thanks!

Anyway, continue with subclassing. finally it is good as we can immediately see what is wrong or lacking :)

@plantec do you want that I push the enable and string API to ToLabel? or I continue with this approach with Spec backend and we push them before


Support tar files for multi-arch images

Fixes #112

I am confused about how this fixes oci_image_index support for oci_tarball.

I am confused about how this fixes oci_image_index support for oci_tarball.

When the image dir contains an image index, this now bundles things up properly so that docker load will work with the tar (assuming you've enabled containerd) - it's the same format as skopeo copy docker://some:image "oci-archive:oci-archive.tar" --multi-arch all -f v2s2 would output.
The layout of the tar file is an index.json which is an index of images (which may have annotations indicating their repo tags), which point at manifests by digest (in the blobs directory), and then a blobs directory containing all of the layers (and manifests) referenced.
I've been using it to take an oci_image_index and produce a .tar file which can be docker loaded, just like oci_tarball already supports for oci_image.
(My goal is to be able to docker load an multi-arch image, as I have systems which then consume these docker load-able tars)

I am more confused now. oci_image_index doesn't produce more than one image. it just makes a multi-arch image out of multiple images and then produces a single image. and docker style tarballs don't support multi-platform images inside the tarballs.
Could you please add an example to the examples folder to demonstrate how this works?

bundles things up properly

Could you elaborate a bit more on this?

Sorry for not being clear! Hopefully the example will help. I've just pushed an example of a multi-platform image with some Go code built for Linux in both amd64 and arm64.
Assuming you've enabled containerd for a local docker install, the tarball it produces can be docker loaded and then docker.com/example:latest will be runnable. This will work whether you're on an amd64 or arm64 machine.
If you have an Apple Silicon machine, you can do this:
% bazel build //examples/multi_arch_go:image-multiarch-tar
INFO: Invocation ID: 0713b09c-525d-42e3-a825-b732f6dd581e
INFO: Analyzed target //examples/multi_arch_go:image-multiarch-tar (0 packages loaded, 0 targets configured).
INFO: Found 1 target...
Target //examples/multi_arch_go:image-multiarch-tar up-to-date:
  bazel-bin/examples/multi_arch_go/image-multiarch-tar/tarball.tar
INFO: Elapsed time: 0.339s, Critical Path: 0.13s
INFO: 1 process: 1 internal.
INFO: Build completed successfully, 1 total action
% docker load <bazel-bin/examples/multi_arch_go/image-multiarch-tar/tarball.tar
Loaded image: docker.com/example
% docker run -it --rm --platform linux/amd64 docker.com/example
yo
% docker run -it --rm --platform linux/arm64 docker.com/example
yo

From this you can see this .tar was docker load-ed and could be run on both architectures. As far as I can tell, this wasn't possbile using rules_oci before this PR.

(I think one of the key pieces of information here is that while docker style tarballs don't support multi-platform images, oci style ones do, and docker can be configured to work with oci style tarballs)

Assuming you've enabled containerd for a local docker install

is this the case for standard setup?

Assuming you've enabled containerd for a local docker install

is this the case for standard setup?

With docker desktop, there's a "features in development" flag you can enable: https://docs.docker.com/desktop/containerd/
(I'm not actually using docker desktop, I'm using a different system which understands OCI format images, but it's easy to test in docker desktop :))
I'm assuming this feature is reasonably in scope for rules_oci, given it's about oci-spec formatting rather than specifically docker interoperation.

(I think one of the key pieces of information here is that while docker style tarballs don't support multi-platform images, oci style ones do, and docker can be configured to work with oci style tarballs)

ideally, we should not implement hacks. it's the reason why we don't produce both oci and docker style tarballs in just one tarball by putting both manifest.json and index.json into the tarball.  I would assume that containerd just supports oci style tarballs and ignores manifest.json.  I don't want our API docs to say oci_image_index is supported but doesn't in standard setups.

ideally, we should not implement hacks

I wouldn't classify this as a hack... It is unfortunate that we're having to generate the index.json file ourselves, but I view it as the same level as unfortunate that we're generating a manifest.json file ourselves in this file before this PR...
Would it feel more clean if there were an attribute on oci_tarball which let you select whether you were expecting an oci or docker format tar?
I agree that it's confusing that there are two possible output formats (oci and docker), and that oci_tarball assumes you're outputting the non-oci format and only works if the inputs are non-oci format... Maybe a good solution here would be to rename oci_tarball to docker_tarball, and make a new rule named oci_tarball which only deals with the oci format?

The more I think about it, the more I think that having an output_format attr which can take values "docker" or "oci" makes sense, and that we should just error if we can't translate between the formats correctly.
So the current state of the repo would only support "docker", and would error if you gave it an oci image to tar up (e.g. an oci_image_index. With this PR, we could support oci -> oci, but we'd still error on oci -> docker or docker -> oci. In the future we may be able to do translation between the formats, but we wouldn't necessarily ever actually implement these.

Hi @illicitonion, thank you very much for working on this, it is also a feature we would really like.
We were just trying it out & we see the following:
bazel run examples/multi_arch_go:image-multiarch-tar
...
open /var/lib/docker/tmp/docker-import-3288145814/blobs/json: no such file or directory

Any idea why this would be?

Hi @illicitonion,
Thank you very much for working on this, it is also a feature we would really like. Just curious, when we were trying it out we saw the following error:
bazel run examples/multi_arch_go:image-multiarch-tar
...
open /var/lib/docker/tmp/docker-import-3371548048/blobs/json: no such file or directory

Just curious if you know why that would be. Thank you.

@thesayyn Thanks for the review, and sorry for the ridiculous delay in getting back to you.
I've added the format attr as described - for now I've only implemented oci support for image_index targets, but we could add image support as/when needed (I imagine I'll probably add it soon, but wanted to keep the PR manageable).
I've also reworked the PR slightly to minimise the diff - the old codepath now hopefully is more clearly mostly unchanged, and the new codepath is mostly just additions not modifications.
Please let me know if you have more ideas or comments! :)

Hi @illicitonion,
Thank you very much for working on this, it is also a feature we would really like. Just curious, when we were trying it out we saw the following error:
bazel run examples/multi_arch_go:image-multiarch-tar
...
open /var/lib/docker/tmp/docker-import-3371548048/blobs/json: no such file or directory

Just curious if you know why that would be. Thank you.

@kriscfoster It sounds like you're running docker without containerd enabled - if you enable it as per https://docs.docker.com/desktop/containerd/ your docker run should work.

Thank you @illicitonion, this worked!

Do we know what is needed for this to make it to main? It's working well for us but we would prefer to use it directly from a main release instead of from a fork.

Note, this will have big merge conflict against https://github.com/bazel-contrib/rules_oci/pull/385, we'll see who gets in first :)

Hopefully this is ~good to go!
@alexeagle What's your timeline for #385? I'm happy to do either rebase (it should be trivial, just every cp_f_with_mkdir becaomes an add_to_tar call (and removing the ${STAGING_DIR} prefix from each second arg), and would love to get both in as soon as we can! :)

Sorry to dig up an old issue, @illicitonion since you authored these changes and you seem to have experience in the topic, I've hit the same error as @kriscfoster but through Skaffold (which is using Docker Engine API).
More context: https://github.com/GoogleContainerTools/skaffold/issues/9217
Any idea why this wouldn't work? I would assume it should, since I'm able to podman load the image produced by oci_image_index -> oci_tarball locally. (I also assume you're using Podman, from your earlier comments)
Is there any way we could add support for multi-arch oci_tarball in Skaffold using Docker Engine API's client? Or is there a different client that supports both Docker and OCI format?

Friendly ping @illicitonion, if you have any pointers about the above message ☝🏻 🙏🏻

Sorry, I don't have any useful context here - it sounds like skaffold needs OCI format support.


[Enhancement] Add build info stating external ToughAsNails.jar dependency.

It would be good if it was mentioned anywhere that you need a ToughAsNails.jar external dependency to build DynamicSurroundings.
Likewise, it also wouldn't hurt if the empty lib folder and modified gradle.build (containing compile fileTree(dir: 'lib', includes: ['*.jar']) ) were pushed to the repo too, as modifying those manually means trouble since it makes the local git working state unclean - preventing a pull.

I went ahead and checked the JAR in the libs directory for now w/the appropriate TAN jar.  I have it on my list to clean up API dependencies a bit.  Some work I have to do.

Modified build to pull TaN from maven repository.  Should address this nicely.


Optional hqx scaling

Some sort of pixel art scaling algorithm would be neat for the user to have as an option. Hqx is one of the better-looking ones, in my opinion.
Bonus points if it uses webassembly or webgl for snappiness (not sure how CPU-intensive these algos are).
Using a background thread for scaling (web worker or similar) may be required for performance.

Self-assigned. I've made some progress with hqx in webgl.


Memory Card

almost done, but it needs doing more tests

Great to see our memory card is being translated, and the code looks good!
Please provide all the unit tests in the TS so that we can make sure our code are working as expected.
You can start from the following 3 :

https://github.com/huan/memory-card/blob/master/src/memory-card.spec.ts
1.https://github.com/huan/memory-card/blob/master/src/memory-card.multiplex.spec.ts
https://github.com/huan/memory-card/blob/master/src/storage/file.spec.ts

@huan Thx, I will follow them in next days.Actually, it has passed one test file .
And how about the questions? Especially in files.ts. it is not implement. @wj-Mcat

@wj-Mcat Please feel free to help yourself to merge a PR as the maintainer when you believe it's meet the quality of our Python Wechaty requirements with following all the Wechaty best practices.
The current setting is a PR that can be merged after it gets at lease one approval.


[SPARK-37284][BUILD] Upgrade Jekyll to 4.2.1

What changes were proposed in this pull request?

Jekyll 4.2.1 was released in September, which includes the fix of a regression bug.
https://github.com/jekyll/jekyll/releases/tag/v4.2.1
Why are the changes needed?

To keep it stable.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

CIs.

Test build #145086 has started for PR 34552 at commit 06fb2c0.

Test build #145086 has finished for PR 34552 at commit 06fb2c0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/145086/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49555/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49555/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/49555/

I briefly tested with the following manually. The generate pages look fine.
SKIP_API=1 bundle exec jekyll build
SKIP_API=1 bundle exec jekyll serve --watch


MeSH: C537666 is not a recognised xref for Becker muscular dystrophy (MONDO:0010311)

Hi
I was looking at the following MONDO term in the OWL file OWL.edition of MONDO (v1.0), it is listed as having an exactMatch to MeSH: C537666, but that ID does not exist
Becker muscular dystrophy (MONDO:0010311) it has a exactMatch to MeSH: C537666
But that ID is not found/does not exist : https://www.ncbi.nlm.nih.gov/mesh/?term=C537666
For Becker muscular dystrophy (MONDO:0010311)  should the MeSH ID be:
https://www.ncbi.nlm.nih.gov/mesh/?term=C570377
or
https://www.ncbi.nlm.nih.gov/mesh/?term=D020388
(or none of the above?)
Thanks

This xref comes from Orphanet (cc @ana-rath-orphanet and @Orphanet). Looks like it has been retired from MeSH?
I agree this should be updated to D020388, I will do so.

Thank you pointing this out. Will be corrected in Orphanet as well .
Relationship (Orphanet to MeSH) ND (as Becker is a sibling of Duchenne
and not a subtype, the main MeSH code being exact for Duchenne)
Best
Ana
Le 2020-05-27 20:32, Nicole Vasilevsky a écrit :

This xref
comes from Orphanet (cc @ana-rath-orphanet [1] and @Orphanet [2]). Looks
like it has been retired from MeSH?
I agree this should be updated
to D020388, I will do so.
--
You are receiving this because you
were mentioned.
Reply to this email directly, view it on GitHub [3],
or unsubscribe [4].

Links:
[1]
https://github.com/ana-rath-orphanet
[2] https://github.com/Orphanet
[3]
https://github.com/monarch-initiative/mondo/issues/1601#issuecomment-634858814
[4]
https://github.com/notifications/unsubscribe-auth/AKLLIYAD3BWQFP7EFUORUOTRTVMCNANCNFSM4NMEAZ3Q

@nicolevasilevsky MeSH: D020388 is for Duchenne so I think that MeSH:C570377 should be added to MONDO:0010311?

re: @nicolevasilevsky MeSH: D020388 is for Duchenne so I think that MeSH:C570377 should be added to MONDO:0010311?
Should I put this in as a separate request? Was this resolved?

@lcdaugherty I made this change, it should be available in the next release at the end of July/early Aug.
(Sorry for the delayed reply!)

@nicolevasilevsky no problem, thank you for following up and making the change.


index.html, start-login.html, end-login-html, should be in the same domain, so we should remove validate domains configuration

Please look at this issue,
https://github.com/OfficeDev/microsoft-teams-library-js/issues/431
This issue mentioned that (index.html, start-login.html, end-login-html) these pages should be in the same domain, but this docs mentioned that we can use validate domains to assign different domain for these pages, which is confusing. I think we should remove validate domains configuration

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

ID: 6d5b8b0f-3da3-8642-6edf-dc6608a830cf
Version Independent ID: f93cdabd-f6ab-29d0-f3c9-78deb21ca521
Content: Authentication flow for tabs - Teams
Content Source: msteams-platform/tabs/how-to/authentication/auth-flow-tab.md
Product: msteams
GitHub Login: @laujan
Microsoft Alias: lajanuar

@SLdragon:
If we want to authenticate all the three pages in a tab then it should be in one single domain.
And, if we want to use one of the page in other tab then it should be in another domain
Then, these two domains should be placed in Validate Domains.

Thank you, just sync with Teams SDK team, the login page must in the same domain, and other pages do not have this limitation:

The authentication pages must be in same domain and let's say you have other tab which corresponds to other domain can be added in manifest. So, overall different domains can be added in manifest but for authentication purpose all pages (index,start,end) must use same domain.  it's like if you have total 5 pages out of which 3 are auth pages and 2 are pages of different domain , so you can mention two valid domains in manifest for it to work.

I think we can update the docs to clarify this, thank you!


Reinstate missing bin/rails script

This was not meant to be deleted in:
https://github.com/alphagov/calculators/pull/180/files#r118538377
The app will not start on Heroku without this.
Also removing a redundant old rails script from Rails 3.

 Child Benefit tax calculator


test: Emacs 29.1

Test 29.1 and snapshot but allow failure for the snapshot.

I would prefer not to have builds for snapshot, atleast for Windows.

I prefer to build snapshots so people know whether it works on the latest branch or not. That's why we have the experimental flag set up. I am assured the Windows snapshot will be fixed since the other two snapshots (ubuntu and macos) have no error.  Okay, let me give this a try. Thanks for the PR!


Dev migrate to laravel 9

Migrate to Laravel 9
Bro , can you make a Develop branch ?

thanks Bro, I will make a Develop Branch

thanks Bro, I will make a Develop Branch

No probs bro. I'm working on converting current test into Pest. I might send a pull request this week.


Suggestion from InfinityPlus05#8973

"My idea for the new economy bot: adding a AH/crafting system. I want to give the shop more purpose so 75% of the inventory isn't useless.
Heres how my idea would work: lets say the shop has 25 different items. However, the shop only has 3 items every 24 hours. This would include weaker items and materials for crafting. If the materials aren't available in the shop, people can sell materials on an auction house for either a fair price or a marked up price. These materials can be combined together to make good items that aren't available for purchase from the shop. These crafted items can also be sold on the Auction house. "
Crafting, you'll need specific items as well as some money to buy an item from the store.  Meant to close #6.

 Meant to close #6.

Framework is in place; server admins and god mode users can set a requiredItems property when adding items.


python fails to load ADT since calculated base address ends up negative

I've only hit the following problem once with 12.1 firmware and couldn't reproduce it after a reboot now.
I've added some debug prints and m1n1 computes the ADT base address as 0x12c0000 - 0xffffffffff0a8000 + 0x8010a8000 = -0xfffffff7fcd40000.
The same thing works in C because it overflows back to 0x8032c0000.
I'm not sure if this is just a python issue and casting the result down to a u64 is good enough or if 0xffffffffff0a8000 is an unexpected virt_base.
TTY> CPU init (MIDR: 0x611f0221)...
TTY>   CPU part: 0x22 rev: 0x11
TTY>   CPU: M1 Icestorm
TTY>
TTY> boot_args at 0x803b3c000
TTY>   revision:     2
TTY>   version:      2
TTY>   virt_base:    0xffffffffff0a8000
TTY>   phys_base:    0x8010a8000
TTY>   mem_size:     0x1df98c000
TTY>   top_of_kdata: 0x803b40000
TTY>   video:
TTY>     base:       0x9e3a5c000
TTY>     display:    0x1
TTY>     stride:     0xa00
TTY>     width:      640
TTY>     height:     1136
TTY>     depth:      32bpp
TTY>     density:    1
TTY>   machine_type: 0
TTY>   devtree:      0x12c0000
TTY>   devtree_size: 0x58000
TTY>   cmdline:
TTY>   boot_flags:   0x0
TTY>   mem_size_act: 0x200000000
TTY>
TTY>
TTY>
TTY> m1n1 v69ed3e4-dirty
TTY> Copyright The Asahi Linux Contributors
TTY> Licensed under the MIT license
TTY>
TTY> Running in EL2
TTY>
TTY> Device info:
TTY>   Model: Macmini9,1
TTY>   Target: J274
TTY>   Board-ID: 0x22
TTY>   Chip-ID: 0x8103
TTY>
TTY> Heap base: 0x803b40000
TTY> MCC: Initializing T8103 MCC...
TTY> MCC: Initialized T8103 MCC (8 channels)
TTY> MMU: Initializing...
TTY> MMU: Adding nGnRE mapping at 0x400000000 (0x80000000)
TTY> MMU: Adding nGnRE mapping at 0x480000000 (0x80000000)
TTY> MMU: Adding nGnRE mapping at 0x580000000 (0x80000000)
TTY> MMU: Adding nGnRE mapping at 0x600000000 (0x80000000)
TTY> MMU: Adding nGnRE mapping at 0x6a0000000 (0x20000000)
TTY> MMU: Adding nGnRE mapping at 0x6c0000000 (0x40000000)
TTY> MMU: Adding nGnRE mapping at 0x700000000 (0x100000000)
TTY> MMU: RAM base: 0x800000000
TTY> MMU: Top of normal RAM: 0x9e0a34000
TTY> MMU: Unmapping TZ0 region at 0x9e0a3c000..0x9e275c000
TTY> MMU: Unmapping TZ1 region at 0x9e0a34000..0x9e0a3c000
TTY> MMU: Unmapping TZ3 region at 0x9e6e64000..0x9ffe64000
TTY> MMU: SCTLR_EL1: 30100180 -> 30901085
TTY> MMU: running with MMU and caches enabled!
TTY> display: Dummy framebuffer found, initializing display
TTY> dart: dart /arm-io/dart-dcp at 0x23130c000 is a t8020 (locked)
TTY> dart: dart /arm-io/dart-disp0 at 0x231304000 is a t8020
TTY> rtkit(dcp): booting with version 12
TTY> rtkit(dcp): unknown system endpoint 0x0a
TTY> rtkit(dcp): pre-allocated buffer (ep 0x1, dva 0xf00243000, phys 0x9e6263000)
TTY> EPIC: started interface 0 (disp0-service)
TTY> display: waited 2000 ms for display status
TTY> display: connected:0 timing_cnt:0 color_cnt:0
TTY> fb init: 640x1136 (32) [s=640] @(nil)
TTY> fb console: max rows 31, max cols 4
TTY> fb: display logo
TTY> AIC: Version 1 @ 0x23b100000
TTY> WDT registers @ 0x23d2b0000
TTY> WDT disabled
TTY> pmgr: Cleaning up device states...
TTY> pmgr: Enabling SIO_ADMA, parent of active device MCA1
TTY> pmgr: Enabling ANS2, parent of active device APCIE_ST
TTY> pmgr: Enabling DEBUG, parent of active device DEBUG_USB
TTY> pmgr: initialized, 268 devices found.
TTY> cpufreq: Initializing clusters
TTY> cpufreq: Switching cluster PCPU to P-State 7
TTY> Initialization complete.
TTY> Checking for payloads...
TTY> Devicetree compatible value: apple,j274
TTY> Unknown payload at 0x803438000 (magic: 3083702c)
TTY> No valid payload found
TTY> dart: dart /arm-io/dart-usb0 at 0x382f80000 is a t8020
TTY> USB0: initialized at 0x803f46030
TTY> dart: dart /arm-io/dart-usb1 at 0x502f80000 is a t8020
TTY> USB1: initialized at 0x803f46360
TTY> Running proxy...
readmem: 803b3c000 2e0
m1n1 base: 0x803378000
LOAD: _HDR 16384 bytes from 0 to 0
LOAD: TEXT 163840 bytes from 4000 to 4000
LOAD: RODA 32768 bytes from 2c000 to 2c000
LOAD: DATA 393216 bytes from 34000 to 34000
ZERO: 196608 bytes from 0x94000 to 0xc4000
LOAD: PYLD 0 bytes from 94000 to c4000
SKIP: 67108864 bytes from 0xc4000 to 0x40c4000
12c0000 ffffffffff0a8000 8010a8000
Fetching ADT (0x00058000 bytes)...
readmem: -fffffff7fcd40000 58000
Traceback (most recent call last):
  File "/Users/speter/asahi/git/m1n1/proxyclient/tools/chainload.py", line 37, in <module>
    sepfw_start, sepfw_length = u.adt["chosen"]["memory-map"].SEPFW
  File "/Users/speter/asahi/git/m1n1/proxyclient/m1n1/proxyutils.py", line 326, in __getitem__
    return self._adt[item]
  File "/Users/speter/.asdf/installs/python/3.10.1/lib/python3.10/functools.py", line 970, in __get__
    val = self.func(instance)
  File "/Users/speter/asahi/git/m1n1/proxyclient/m1n1/proxyutils.py", line 324, in _adt
    return adt.load_adt(self._utils.get_adt())
  File "/Users/speter/asahi/git/m1n1/proxyclient/m1n1/proxyutils.py", line 184, in get_adt
    self.adt_data = self.iface.readmem(adt_base, self.ba.devtree_size)
  File "/Users/speter/asahi/git/m1n1/proxyclient/m1n1/proxy.py", line 385, in readmem
    req = struct.pack("<QQ", addr, size)
struct.error: argument out of range

I've hit this same bug with 12.2 firmware.
Managed to reproduce it after six reboots. Masking the address to 64bits does seem to produce the correct results.

Looks like this is an artifact of using a somewhat weird virt base when we boot in raw mode. I'll just mask it out for now, seems like this doesn't break anything else.


[BUG] Azure.Identity: Too strict scopes

Describe the bug
I tried to get user_impersonation scope on arm (https://management.core.windows.net/user_impersonation) which I understand from here  is standard scope but I get an error that underscore cannot be used (I was using AzureCliCredential specifically, not sure about other credentials. I tried with default credential before and it got all the way to vs code where it obtained the token).
AzureCliCredential authentication failed: The specified scope is not in expected format. Only alphanumeric characters, '.', '-', ':', and '/' are allowed (Parameter 'scope')

Expected behavior
Accept the scope parameter.
Actual behavior (include Exception or Stack Trace)
Throws an error.
To Reproduce
var tokenProvider = new AzureCliCredential();
var token= tokenProvider.GetToken(new Azure.Core.TokenRequestContext(new[]{"https://management.core.windows.net/user_impersonation"})).Token;

Environment:

Name and version of the Library package used: Azure.Identity 1.2.3
Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects): Windows 10 .NET 5.0.100 CLI (the project is core 3.1 though).
IDE and version : VS Code 1.43.2

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

@dmladek-dtml Although there does seem to be an issue with the scope validation, it appears that you are attempting to do an On Behalf Of flow. Unfortunately, we don't yet support that, but the following issue has some information about how to work around this until we do.
https://github.com/Azure/azure-sdk-for-net/issues/16264#issuecomment-716869572
Does this work for your scenario?

(the reporter account is another account of mine to prevent any confusion)
@christothes I don't remember what exactly I was trying to accomplish, but I believe I got it working in the end for the scenario needed, possibly without the use of actual scopes, but thank you for the links.


Lock image size is hard-coded and square

Just wanted to start by saying thanks for i3lock-next! Definitely the fastest locker I've come across so far.
I was wondering if it'd be possible to make i3lock-next a little more flexible with regards to the lock image, specifically:

lookup lock image size directly in i3lock-next-helper.c
potentially adjust the ring size argument that's passed to i3lock-color

I've done step 1 already on my fork and can submit a PR if you're agreeable. Otherwise, I'm curious as to what your thoughts are.

I appreciate your kind words! I would love to implement both features, go ahead and submit a PR.

I rewrote the i3lock-next script in Python on the kcarbon-lookup_lock_image_size branch, let me know what you think.

I'm personally happier with scripting in Python and your branch/script looks good! I'm about to get on a plane though, so I didn't have a chance to test it out. This is where my ignorance comes up (aerospace background, not software), but how much slower is the Python script versus the bash script? I'd assume not enough to become perceptible to the user?

In my expert (read: definitely NOT expert) opinion, Python should be faster than Bash. Getting the image size with Python instead of with C could be slower but in my (very unscientific) testing it didn't seem slower.


Using an extended Http class with header options to deal with the controllers instead of using a Json file

Hello All,
I have a question about a change I made, I get my labels using a controller not a Json file, so I needed to initialize option in the header of each http request, so I create a new class extending the Http, to use the new Class in the I create a new Loader implementing the TranslateLoader and it uses the the extended Http Class instead of the basic. Does my solution is correct or there is an easier way to do so?
Thanks in advance.

Hello, yes it is the way to do it :-)

Thank you @ocombe  :)


Bump spring boot to 2.7.9 and spring native to 0.12.2

Pull Request Test Coverage Report for Build #191

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.237%

Totals

Change from base Build #190:
0.0%

Covered Lines:
193

Relevant Lines:
207

💛  - Coveralls


Are msexchuid and amurl immutable even if end user migrate Exchange Server?

Thank you for writing clear document. Our team understand definition, but let me ask one question.
Are msexchuid and amurl immutable even if end user migrate Exchange Server? For example, some end users may migrate their on-premises Exchange Server to Office 365. When they start to use Office 365, are msexchuid and amurl same?
The background is we refer this document and try to make UserId from Exchange IdToken for backend. If these parameter keep same, we want to use this logic. If these params changed to another after migration, we need to consider it for making UserId.

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

ID: 709d2fe0-b7b5-5008-218c-6e84f5565998
Version Independent ID: d2cb30ef-4ba0-1d0b-d681-74dbf35bb6d0
Content: Inside the Exchange identity token in an Outlook add-in - Outlook Developer
Content Source: docs/add-ins/inside-the-identity-token.md
Product: outlook
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

@exextoc Can you take a look? Thanks.

No, amurl and msexchuid will not be the same between Office365 and on-premise after migration.  If you are considering using the amurl+msexchuid as the mapping for  your addin's backend id, you will need to re-create the mapping after the user migrates.

hi @exextoc team
thanks for your response.


Update ansible to 2.11.x: rename ansible-base to ansible-core

Tool name
ansbile
Tool license
GNU GPL 3.0
Add or update?

[ ] Add
[X] Update

Desired version
2.11.x (latest)
Approximate size
No response
Brief description of tool
https://docs.ansible.com/ansible/latest/installation_guide/intro_installation.html#upgrading-from-ansible-3-or-ansible-core-2-10
Ansible base package seems to renamed from ansible-base to ansible-core so auto patching is no longer working at the moment.
URL for tool's homepage
No response
Provide a basic test case to validate the tool's functionality.
No response
Virtual environments affected

[ ] Ubuntu 16.04
[X] Ubuntu 18.04
[X] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11
[ ] Windows Server 2016
[ ] Windows Server 2019

Can this tool be installed during the build?
No response
Tool installation time in runtime
No response
Are you willing to submit a PR?
No response

Hi @petermylemans!
Thanks for notifying us about such changes. We will take a look and prepare a PR if needed.

Hi, @petermylemans
To install ansible 2.11 in runtime, you can use:
jobs:
  build:
    runs-on: ubuntu-20.04
    steps:
        - run: |
            pipx uninstall ansible-base
            pipx install ansible-core
            pipx inject ansible-core ansible
            ansible --version

The new images have been deployed with ansible 2.11.2.


Game Crashes when you use the play animation event twice.

Describe your bug here.
I don't know if anyone has gotten this issue, but. if you use the character event animation, then later on use it again. the game freezes for about 2-3 seconds and then crashes. this could cause people to lose their charting progress if they haven't saved prior.
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no.

load auto save

ok

since ur using lua i think u gotta precache ur funy character assets first
https://github.com/ShadowMario/FNF-PsychEngine/wiki/Lua-Script-API:-Precaching-Functions