The writings are difficult to read

The super mario 64 font is quite peculiar and reduced on a small screen with few pixels is quite difficult to read. How hard would it be to change the font?

Indeed, the font is a bit hard to read given the DSi's resolution. My original vision was to have SM64 on the DSi as true to the original as possible, including the somewhat illegible font, heh. There were plans for a "non-vanilla" branch meant for changes that optimize the experience for the DSi, but I haven't really done anything for it yet. My interest mostly lies in the porting process, so stuff like enhancements tend to fall lower on my priority list.


Fix upload to synapse bug and add admin override

Closes #55
Still a WIP. Uploading the manifest to synapse seems to be working, as does the override button...but the override manifest seems to be blank. Maybe testing with a perfect manifest to see if that works or not will help track down that issue...
edit: Nevermind, it wasn't working because I has not selected the correct folder in the project. The app should probably validate that the name of the root folder in the manifest is the same as the name of the selected folder (or better yet, just find it). But that is a distinct issue.

This is ready for testing on staging.

Just tested out on staging, and the override worked great! (Tested with uploading files to this folder on Synapse -- which worked, even when submitting a tiny error, i.e., an invalid reportMilestone number. I then fixed this error after the fact.)
Reviewing the rest of this now, real quick.

This is done! If you don't mind pulling this into shiny, @allaway, I'm currently denied permissions :(


Fix error code output for check/pytest

Fixes #1945.

This PR will fail because of failing tests from #1937 that were not caught.


flutter-devicelab-mac-23 cannot access attached iPhone

Type of Request
bug
Infrastructure Environment
DeviceLab
What is happening?
I ssh'ed to flutter-devicelab-mac-23.mtv.corp.google.com in order to create and test a new dev signing cert, following the instructions in b/256878500. When I got to step (5) there was no attached iOS device. Running flutter doctor and flutter devices (even by extending the --device-timeout to 30 seconds) shows no attached device.
Opening Xcode and checking Window > Devices & Simulators shows a record of a previous iPhone but it doesn't seem to be attached and accessible.
Steps to reproduce

Set up a port forward to mac23. Use the password recorded in Valentine:
ssh -L 5900:localhost:5900<EMAIL_ADDRESS>

Make a VNC connection by opening the Finder on your local machine, press Cmd-K, then for the URL, put vnc://localhost.

Connect as swarming with the password in Valentine.

Open the terminal and execute:
cd ~/Downloads/hello1
../flutter/bin/flutter devices

Expected results
Expected behaviour: a physical iPhone 12 is listed
Actual behaviour: only the macOS device is listed

@kentnguyen99 could you take a look at this host and see why it's not detected the iphone?

@cbracken Can you give another try? It should be working now, there is diaglog prompting on phone device require to respond.

Thanks -- taking a look!

Thanks @kentnguyen99 ! The device now shows as attached.


Fixed duplicate page slug in populatedb when the about page was edited

This fixes the duplicate key error (on slug) when an user edited the about page then run populatedb.
Pull Request Checklist

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] Database migration files are up to date.
[x] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[ ] GraphQL schema and type definitions are up to date.
[ ] Changes are mentioned in the changelog.

Here is the report for d305dc2fe26a6a8f6cefc87a0359e5d5ec701b9a (NyanKiyoshi/saleor @ fix/populatedb/page-conflict)
Base comparison is e2852d429ae7b206f23853473446b4fd5f5ee717.
**Found 1 differences!** (click me)

# api.benchmark checkout
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  add billing address to checkout     	         34	         34	             20
  add shipping to checkout            	          7	          7	              0
  checkout payment charge             	         14	         14	              0
  complete checkout                   	          6	          6	              0
- create checkout                     	         41	         48	             24

# api.benchmark homepage
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  retrieve main menu                  	          5	          5	              0
  retrieve product list               	          4	          4	              0
  retrieve secondary menu             	          5	          5	              0
  retrieve shop                       	          2	          2	              0

# api.benchmark product
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  product details                     	         13	         13	              3

# api.benchmark variant
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  retrieve variant list               	          9	          9	              2


Unable to upload own Data

Hi,
This looks like a great tool and I am looking forward to use it. However, I have not been able to upload my own Data through the Data upload tab, the refresh button doesn't switch to green - even though my data fulfills the required data syntax. I tried the specified sample file "great_srf_example.txt" and it isn't accepted either. The console throws the following error:
[01:05:30 main ERROR] Exception on /upload_view_file [POST] Traceback (most recent call last):   File "/home/tilma/.local/lib/python3.8/site-packages/flask/app.py", line 2447, in wsgi_app     response = self.full_dispatch_request()   File "/home/tilma/.local/lib/python3.8/site-packages/flask/app.py", line 1952, in full_dispatch_request     rv = self.handle_user_exception(e)   File "/home/tilma/.local/lib/python3.8/site-packages/flask/app.py", line 1821, in handle_user_exception     reraise(exc_type, exc_value, tb)   File "/home/tilma/.local/lib/python3.8/site-packages/flask/_compat.py", line 39, in reraise     raise value   File "/home/tilma/.local/lib/python3.8/site-packages/flask/app.py", line 1950, in full_dispatch_request     rv = self.dispatch_request()   File "/home/tilma/.local/lib/python3.8/site-packages/flask/app.py", line 1936, in dispatch_request     return self.view_functions[rule.endpoint](**req.view_args)   File "main.py", line 604, in upload_view_file     fn = werkzeug.secure_filename(f.filename) AttributeError: module 'werkzeug' has no attribute 'secure_filename' [01:05:30 tornado.access ERROR] 500 POST /upload_view_file (::1) 18.84ms
I am using bash on Windows 10. Any help is greatly appreciated.

Hi,
I think the data upload issue is probably due to the werkzeug/tornado
dependencies during
the setup. I am not sure if this is related to Windows or not. You can
modify the source code to make the fn variable in upload_view_file your
data path and see if this error goes away.
Please note that this software has only been tested on mac and linux
operating systems, and I am not able to assist with windows-specific
issues.
Jason
On Wed, Dec 2, 2020 at 4:13 AM tirichl<EMAIL_ADDRESS>wrote:

Hi,
This looks like a great tool and I am looking forward to use it. However,
I have not been able to upload my own Data through the Data upload tab, the
refresh button doesn't switch to green - It fulfills the required data
syntax. I tried the specified sample file "great_srf_example.txt" and it
isn't accepted either. The console throws the following error:
[01:05:30 main ERROR] Exception on /upload_view_file [POST] Traceback
(most recent call last): File
"/home/tilma/.local/lib/python3.8/site-packages/flask/app.py", line 2447,
in wsgi_app response = self.full_dispatch_request() File
"/home/tilma/.local/lib/python3.8/site-packages/flask/app.py", line 1952,
in full_dispatch_request rv = self.handle_user_exception(e) File
"/home/tilma/.local/lib/python3.8/site-packages/flask/app.py", line 1821,
in handle_user_exception reraise(exc_type, exc_value, tb) File
"/home/tilma/.local/lib/python3.8/site-packages/flask/_compat.py", line 39,
in reraise raise value File
"/home/tilma/.local/lib/python3.8/site-packages/flask/app.py", line 1950,
in full_dispatch_request rv = self.dispatch_request() File
"/home/tilma/.local/lib/python3.8/site-packages/flask/app.py", line 1936,
in dispatch_request return
self.view_functionsrule.endpoint File "main.py", line
604, in upload_view_file fn = werkzeug.secure_filename(f.filename)
AttributeError: module 'werkzeug' has no attribute 'secure_filename'
[01:05:30 tornado.access ERROR] 500 POST /upload_view_file (::1) 18.84ms
I am using bash on Windows 10. Any help is greatly appreciated.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/junjiezhujason/aegis/issues/7, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACD57HJCLEYSIRIW3FWYRV3SSYVNDANCNFSM4UKM6PAQ
.

Thank you for your answer. I didn't have any sucess with that, however I was able to solve the problem: According to this issue
(https://github.com/maxcountryman/flask-uploads/pull/28), there is a bug in flask 1.0.0, which caused this error. Downgrading with
pip3 install -U Werkzeug==0.16.0 to 0.16.0 did the trick. Thank you for your time!


Unable to import credstash module within python lambda function

Hello,
I'm trying to import credstash into a python lambda function like so:
import credstash
and access a secret:
credstash.getSecret('bla')
And I get the following error upon invocation of the lambda:
Unable to import module 'ec_lambda_adx_scanner': /var/task/cryptography/hazmat/bindings/_constant_time.so: invalid ELF header
ec_lambda_adx_scanner is my lambda function name. I'm not running in a virtual env, just deploying and invoking as usual from AWS.
Thank you in advance.
EDIT:
To give you a little more information, I am first installing credstash from a requirements.txt file via a shell script. This builds credstash locally on my Mac. I then use serverless to package up the lambdas and their dependencies and deploy. I believe the issue is that I have built a version of credstash for my Mac environment, but when I try to use that same build on lambda (Linux image) it fails.
Update: I rebuilt the credstash lib on a Linux EC2 AMI instance rather than on my mac and have now been receiving this error instead (seems to be another C lib.):
Unable to import module 'ec_lambda_adx_scanner': libffi-a7faab9c.so.6.0.4: cannot open shared object file: No such file or directory

@mfisher29 have you tried to downgrade to credstash 1.12? You problem (and I got a similar one) could be cause by this change removed build constraint on cryptography <2.0 in 1.13.3

I figured it out. I needed to build credstash on an EC2 AMI Image in a virtual env and then copy it back to my local directory so that  I could zip it up with the lambda and deploy to AWS.
Similar to these steps here:https://github.com/kiddouk/letslambda
Except with credstash 1.13.3 in the requirements.txt file instead.


cast errors returned from WrappedSession.connect

Errors returned from WrappedSession.connect need to be cast to RustString before they can be displayed.
Fixes #5965

adding @ReactorScram for visibility

@jamilbk I ended up pushing another small change on top of this that simplifies how we generate those strings.


⚠ Community has degraded performance

In 7ec4b13, Community (https://community.wrk.com/health) experienced degraded performance:

HTTP code: 200
Response time: 222 ms

Resolved: Community performance has improved in 6aaad83.


Add clang-analyze to build

I think clang-tidy runs the analyzer, too, right? See #1768.

Apparently it can, I didn't know that.
But it requires the specific clang-analyze functionality to be enabled.
Further, it looks like most clang-analyze checks aren't supported by clang-tidy:
https://clang-analyzer.llvm.org/available_checks.html
https://clang.llvm.org/extra/clang-tidy/checks/list.html
I count 11 non-OSX checks in clang-tidy, and over 25 non-OSX checks in clang-analyze.

Ah okay, than running the clang-analyze, too, seems a good idea. When does it run though? The output doesn't help me. Are you sure it would find something (and the GitHub Action check would fail)?

The "scan-build" command inter-poses the clang-analyzer as a CC and CXX, runs the analysis, and then calls the "real" compiler.
In this case, it doesn't appear that the default-enabled checks are finding anything to complain about, but other projects I work with the default checks do complain.
Do you have any specific checks that you'd like enabled?

No, the default is fine :)


Add quotes/double quotes when selection is expanded and quote keys are pressed.

Hi @Soreine @SamyPesse! Great plugin, and thank you as always for putting it out there!
On the lines of #5, another enhancement could be wrapping the expanded selection if the quote or  double quotes is pressed.

Closed in favor of #5


feature: external-lb Ability to specify a F5 bigip pool as the target endpoint instead of virtual server

Rancher Version:
Docker Version:
OS and where are the hosts located? (cloud, bare metal, etc):
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Steps to Reproduce:
Results:
Expected:
A service should be linked to a bigip pool instead of a virtualserver AND pool. This is to support scenarios where there is no default pool for a VIP but traffic is instead routed via irules or policies. Rancher doesn't need to know about the VIP, only the pool in which to place the containers.

Is there any (planned) progress on this issue?

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


Update project structure to Maven format

As this is a Maven project, it should be in appropriate single-module Maven format.

Confirmed that this still works for current Legion Objectives once the class paths are prefixed with main.java.

Kept main path as com.rw.legion.DefaultJob

@daynebatten Confirmed this still runs as expected using Home Event analytics data. Do you have the ability to merge?

wait, need to revert the README

@daynebatten Thanks!


Hashes are different (

Hi, I've tried to reproduce on torch, and torch sha256 differs from real sha256
Is it stable in the last version of pytorch?

It's been a long time since I've looked at this code and it's not exactly in the best condition. You might want to try looking at the adder branch instead of main.
The interior of the relaxed space is filled with local minima, so if you're hoping to run gradient descent on it as I was you're out of luck. Even if the surface were to smoothly interpolate between the discrete values of sha256, the problem is still non-convex.

I have another idea. But your code can be quite helpful for this.
I want to optimize the computation of sha256 by using a neural network.
@calebh  Could you write me, please?
<EMAIL_ADDRESS>I want to discuss this idea with you.


Add missing params

closes https://github.com/MicrosoftDocs/office-docs-powershell/issues/11135

@tseward

@JohanFreelancer9

@yogkumgit The PR is still draft.

@get-itips


SecretsManager Validation problem on startup of Spring Boot 2.6.6 microservice

Type: Bug
Component:
"Secrets Manager"
Describe the bug
So because of Spring Vulnerability I have to upgrade my Spring Boot from 2.2 to 2.6.6. This changes also include Cloud upgrade (2021.0.1) and of course switching to io.awspring.cloud (2.4.0).
I was trying to get this working (I created small boot application for testing after loosing whole day), and my small application started working correctly just after I included spring-cloud-starter-bootstrap (3.1.1) (as per documentation). When I tried to apply the same solution to my production microservice (running on AWS in ECS cluster), I started getting validation errors on parameter aws.secretsmanager.prefix (The prefix must have pattern of:  (/)?([a-zA-Z0-9.-]+)(?:/[a-zA-Z0-9]+)*).
Error log I get
2022-04-08 17:09:06.097 WARN [zumzum,,] 1 --- [ main] s.c.a.AnnotationConfigApplicationContext : Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'awsSecretsManagerPropertySourceLocator' defined in class path resource [io/awspring/cloud/autoconfigure/secretsmanager/AwsSecretsManagerBootstrapConfiguration.class]: Unsatisfied dependency expressed through method 'awsSecretsManagerPropertySourceLocator' parameter 0; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'smClient' defined in class path resource [io/awspring/cloud/autoconfigure/secretsmanager/AwsSecretsManagerBootstrapConfiguration.class]: Unsatisfied dependency expressed through method 'smClient' parameter 0; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'aws.secretsmanager-io.awspring.cloud.secretsmanager.AwsSecretsManagerProperties': Invocation of init method failed; nested exception is io.awspring.cloud.secretsmanager.ValidationException: The prefix must have pattern of: (/)?([a-zA-Z0-9.-]+)(?:/[a-zA-Z0-9]+)*
2022-04-08 17:09:06.097  WARN [zumzum,,] 1 --- [           main] s.c.a.AnnotationConfigApplicationContext : Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'awsSecretsManagerPropertySourceLocator' defined in class path resource [io/awspring/cloud/autoconfigure/secretsmanager/AwsSecretsManagerBootstrapConfiguration.class]: Unsatisfied dependency expressed through method 'awsSecretsManagerPropertySourceLocator' parameter 0; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'smClient' defined in class path resource [io/awspring/cloud/autoconfigure/secretsmanager/AwsSecretsManagerBootstrapConfiguration.class]: Unsatisfied dependency expressed through method 'smClient' parameter 0; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'aws.secretsmanager-io.awspring.cloud.secretsmanager.AwsSecretsManagerProperties': Invocation of init method failed; nested exception is io.awspring.cloud.secretsmanager.ValidationException: The prefix must have pattern of:  (/)?([a-zA-Z0-9.-]+)(?:/[a-zA-Z0-9]+)*
2022-04-08 17:09:06.493 ERROR [zumzum,,] 1 --- [ main] o.s.b.d.LoggingFailureAnalysisReporter :

APPLICATION FAILED TO START

Description:
Validation failed for field: aws.secretsmanager.prefix
Action:
The prefix must have pattern of: (/)?([a-zA-Z0-9.-]+)(?:/[a-zA-Z0-9]+)*
2022-04-08 17:09:06.496 ERROR [zumzum,,] 1 --- [ main] o.s.b.d.LoggingFailureAnalysisReporter :

APPLICATION FAILED TO START

Description:
Validation failed for field: aws.secretsmanager.prefix
Action:
The prefix must have pattern of: (/)?([a-zA-Z0-9.-]+)(?:/[a-zA-Z0-9]+)*
Sample
bootstrap.yml
aws:
paramstore:
prefix: /${spring.profiles.active}
secretsmanager:
prefix: /secret/${spring.profiles.active}

What's the value of ${spring.profiles.active}? Has this setup worked before with Spring Cloud AWS 2.3.4?

So the value is zumzum-dev04 (zumzum is replacement from company name).
This worked with with Spring Boot 2.2.1-RELEASE and Spring Cloud Hoxton.SR1. We haven't updated Spring since the project started (which was in 2019).

I think I found the problem... So in file io.awspring.cloud.secretsmanager.AwsSecretsManagerProperties
you have defined:
private static final Pattern PREFIX_PATTERN = Pattern.compile("(/)?([a-zA-Z0-9.\-]+)(?:/[a-zA-Z0-9]+)");
in old code (before split), this is defined as this:
private static final Pattern PREFIX_PATTERN = Pattern.compile("(/[a-zA-Z0-9.\-_]+)");
I checked. My secret-manager prefix: "/secret/zumzum-dev04" works with old implementation, but not with new.
If I change implementation, few parts of your tests stop working. The ones that have prefix "secret".
If you change it into this, then it would work for your case and mine:
private static final Pattern PREFIX_PATTERN = Pattern.compile("(/)?([a-zA-Z0-9.\-]+)(?:/[a-zA-Z0-9.\-]+)*");
Would it be possible to fix this and make new 2.4 release with this change? I think that in next few weeks you might get few more users (that will try to upgrade to newest boot and some might have similar problem).

Would it be possible to fix this and make new 2.4 release with this change? I think that in next few weeks you might get few more users (that will try to upgrade to newest boot and some might have similar problem).

Yes, will be fixed and released.

What is release timeline for this?

Once we get #303 right meaning today/tomorrow/Monday 2.4.1 should be released.

Great.
Thank you for all your great work...

Fixed in https://github.com/awspring/spring-cloud-aws/commit/f034d798f9a0dae4bb1c3256b3a36592cc89dd71


Warning when using tasks: gitversion/setup@0 and gitversion/execute@0

The following two tasks give errors in vscode, because the tasks gitversion/setup@0 and gitversion/execute@0, are not among the supported values
- task: gitversion/setup@0
displayName: "GitVersion: Install"
inputs:
versionSpec: "5.x"
- task: gitversion/execute@0
displayName: "GitVersion: Execute"
On AzureDevOps editor when creating a pipeline these tasks are valid

@savcic-milorad is this custom tasks from extensions?
If so, you can use a schema from your org by the following instruction: https://github.com/microsoft/azure-pipelines-vscode#specific-schema
and then these tasks should be valid.

These tasks can be selected when editing a pipeline in AzureDevOps, in this editor no additional schema is required.

@savcic-milorad have you signed into Azure and followed the prompts to get more accurate schema validation?
https://github.com/microsoft/azure-pipelines-vscode?tab=readme-ov-file#schema-auto-detection

Hi @50Wliu this I didn't do. I will keep this in mind for upcoming tasks involving pipelines, thanks

Thanks - closing as expected behavior, then. We only support built-in ADO tasks unless you sign in for your organization-specific schema.


Prepare script deletes files, then throws due to empty files

><EMAIL_ADDRESS>prepublish
> rimraf artifacts build cache generated

><EMAIL_ADDRESS>prepare
> bash scripts/prepare.sh

node:internal/modules/cjs/loader:1176
    throw err;
    ^

SyntaxError: /openzeppelin-subgraphs/configs/accesscontrol.json: Unexpected end of JSON input

I think this the same error as #41. Without jq working, prepare was producing empty files that are not valid json.
fdc25ebf82c158916584940e564a823f89e3db1b should fix that.


Use react-intl hooks

Part of https://github.com/ManageIQ/sources-ui/issues/134
@Hyperkid123

Codecov Report

Merging #135 into master will increase coverage by 0.14%.
The diff coverage is 91.66%.

@@            Coverage Diff             @@
##           master     #135      +/-   ##
==========================================
+ Coverage   78.01%   78.16%   +0.14%
==========================================
  Files          40       40
  Lines         514      522       +8
  Branches       78       78
==========================================
+ Hits          401      408       +7
- Misses         89       90       +1
  Partials       24       24

Impacted Files
Coverage Δ

src/components/SourceEditForm/SourceEditModal.js
23.52% <0%> (-1.48%)
:arrow_down:

src/pages/SourcesPage.js
91.42% <100%> (+0.25%)
:arrow_up:

src/components/RedirectNoId/RedirectNoId.js
100% <100%> (ø)
:arrow_up:

src/components/SourceRemoveModal.js
100% <100%> (ø)
:arrow_up:

src/components/SourcesFilter.js
100% <100%> (ø)
:arrow_up:

.../components/SourcesSimpleView/SourcesSimpleView.js
100% <100%> (ø)
:arrow_up:

src/components/AddApplication/RemoveAppModal.js
100% <100%> (ø)
:arrow_up:

src/components/AddApplication/AddApplication.js
81.08% <100%> (+0.52%)
:arrow_up:

Continue to review full report at Codecov.

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


Feature Request: Counterpart to getEnclosingCameraPosition

Since forever, the MapController used to have the following two methods:

LngLat screenPositionToLngLat(PointF)
PointF lngLatToScreenPosition(LngLat)

Now, since 0.10, there is the very useful 👍 method

CameraPosition getEnclosingCameraPosition(LngLat, LngLat, Rect)

However, a counterpart for it, to get the currently shown bounding box in LngLat coordinates is missing. The output could be any rotated trapezoid if there is tilt or rotation, so it would be something like

LngLat[] getEnclosedBoundingBox(CameraPosition, Rect)

Use Case
My use case is that I want to pull data from another source for the shown part of the map.

Actually, this can be achieved with only a few lines of code using the existing methods, so maybe not that important. So I withdraw that feature request.

Thanks for the feedback! I totally agree it's useful to be able to easily get the bounds of the current view. The trick is, like you said, what to do when the view is rotated or tilted. Getting a list of trapezoid vertices could be useful for some cases, but in many situations a bounding box would probably be more appropriate. Since there wasn't an obvious answer at the time we put off adding this feature till later.


Signature JPG is created but throws an error 'Directory not created'

Created image file .JPG along with it Directory but throws an error:
E/SignaturePad: Directory not created
E/SignaturePad: Directory not created

Should fix this? or just ignore?

invoke  external storage read and write permission


Link is wrong or section is misleading

[Enter feedback here]
This feed back is not really for this page, but for the index on the left side of this page and the entry "NIC Teaming"
This link does not take you to a page about NIC Teaming but rather to a different guide on Azure-Stack HCI Concepts requirements. Should this not be a guide in the Windows Server Networking core guide? Just like everything else on that list? Typing in the the URL according to what seems to be the convention of other items will return a 404 error, which makes me think the entry doesn't exist.  Furthermore the destination does not explain Teaming in terms of what it does, how to set it up, requirements and exceptions.

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

ID: 62f953b5-5ea4-4435-37b9-53484f75b316
Version Independent ID: 6394da95-c5cb-2e17-2ae7-82e203b16c9e
Content: Network Load Balancing
Content Source: WindowsServerDocs/networking/technologies/Network-Load-Balancing.md
Product: windows-server
Technology: networking
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

Hi @STaylor135,
Thank you for contacting Microsoft. We value your feedback.
A documentation change has been made to address this issue and will be live the next business day. The title of the table of contents entry was changed from "NIC Teaming" to "Switch Embedded Teaming (SET)," which is the current supported form, and it was moved to alphabetical order after the "Software Defined Networking" node, which also contains useful information. Here's a blog post about the different NIC Teaming technologies: https://techcommunity.microsoft.com/t5/networking-blog/teaming-in-azure-stack-hci/ba-p/1070642.
We'll now be closing this issue. Thanks again for your input, and have a most pleasant day.
#please-close


Automatically cleanup open realtime games

Abandoned realtime games are cancelled after 15 minutes.
Notes / Concerns

I've added the functionality to the GameService but I have a slight feeling this kind of behavior might warrant its own service in the future
I don't like that games are searched for by explicit values instead of some predicate, this will break every time new possible limit is added. However, I wanted to avoid grabbing all the games and going through them manually. I think the FileAdapter should support predicate filtering.

I noticed that you are using "searchOpenGames" to find games.  But the search interface is intended to be a client-side-only function.  The reason is because I wanted the query language to be easy to swap out.  Somebody else who wishes to fork this project may use their own storage engine, data adapter, and write their own front-end.  As they do so, they might find a different query language more convenient and may only need to implement it in the data adapter and front-end.  The server-side code doesn't have to change.  Rather, the server-side will ask the data adapter for data using a domain-driven name.  That way you can optimize how to fetch the data for that specific use case in a way appropriate for your storage engine.  So you might add a data adapter method called "cancelExpiredOpenGames" or something to that effect.  Notice that the method doesn't return search results.  It just gets it done.  If it was backed by MySQL, you could choose to implement it as a stored procedure so that the operation is all on the database-side.  Very efficient.

Closing this since it is inactive.


Investigate MADIS Meteorological Surface Dataset

POC: Office NWDM

POC: Office NWDM

Additional information needed. Tagging this item as Blocked.


Add number to the end of filename if it exists when duplicating

Closes https://github.com/godotengine/godot-proposals/issues/6029
This PR adds a number at the end of the suggested filename when duplicating.
Before:

After:

This was rejected in the past: #20841

@KoBeWi Thanks for pointing that out, I'd prefer to discuss that on the proposal as I think this change is very important.

I think the name shouldn't include underscore, to be consistent with Node names.

@KoBeWi Node names use the class name by default which is in Pascal Case, but file names in Godot default to snake_case (try to save any file).
Hence why there is an underscore here, otherwise it wouldn't look as nice.

Well, node name can be snake case too (you have project setting for that), but the number is still added without underscore. The files should be consistent with that. You could make the number underscore depend on preferred name style, but it should be done in separate PR.

@KoBeWi As I said the file names use snake_case by default, why are we following the convention of the node names instead of the actual file names where the number is being added?

This will just make things more inconvenient. A file is not created until you complete this dialogue, so a unique name is not needed to be shown. It's easier to adjust it to your liking when it's the same as the original.

@MewPurPur I do not see how this makes things more inconvenient, the text is selected when the dialog opens so you can just start typing out whatever name you want if you don't want the suggested one -- where is inconvenience coming from?

It makes an erroneous assumption about how the user wants to modify the original name to make it valid, which I don't like.
We could instead warn in these kinds of dialogues when the name is a duplicate one, to prevent users from wasting time altogether.

It makes an erroneous assumption about how the user wants to modify the original name to make it valid, which I don't like.

@MewPurPur I cannot think of a default that please everyone, if you have a suggestion please let me know and I would love to add it.
Nontheless I believe this is a major usability improvement, if you disagree you can also discuss that on the proposal.

I agree that reducing the steps to duplicate to one is better usability.

This is what I expected. I don't know of a better convention.
I also expect "snake_case" in filenames are per our custom in Godot Engine.
Windows 11 does

"windows"
"windows_copy"
"windows_copy_copy"

Which is an alternative.
There's also

"windows"
"windows_copy_1"
"windows_copy_2"

I think the style should match user preferences. If someone uses snake_case names, add underscore. If PascalCase, don't add one. Unfortunately the only related setting we have is scene_name_casing, but it doesn't really apply here, so a new setting might need to be added.

I agree with KoBeWi's assesment. My goal is to see if I can expedite this.

@fire I support adding a new setting for this, feel free to open another PR based on this one if you feel like it. I'm a bit busy these days so might not have time to engage.

Closing due to lack of consensus (see discussion above and the linked proposal). Nevertheless, thank you for contributing!


JavaScript Context Docs / Examples

I couldn't find any docs/examples of using lit context with modern JavaScript. All the docs are Typescript. Where can I find JavaScript examples? Thanks.
https://lit.dev/docs/data/context/

Check out https://lit.dev/playground/#sample=examples/context-basics and toggle the language to JS for a running example.
We haven't gotten around to adding any switchable code samples for those docs but it shouldn't be too difficult to use any of the code there in vanilla JavaScript by removing any type annotations or class member keywords like private.
While decorators can work in JavaScript (they are stage 3 proposal to the language), it will currently require transpilation with something like babel as it's not yet implemented in any JS runtime/browsers.
You can use any of the controllers without transpilation though. e.g. https://lit.dev/docs/data/context/#contextprovider


Update LAB_04_MFAConditionalAccessandAADIdentityProtection.md

In the portal now the new tenant creation is shifted under Manage tenant.
Module: 00
Lab/Demo: 00
Fixes # .
Changes proposed in this pull request:

This is already fixed in another update.  Closing the Request.


Modify storage

Codecov Report
Base: 69.36% // Head: 67.61% // Decreases project coverage by -1.74% :warning:

Coverage data is based on head (a78dc5a) compared to base (99f364e).
Patch coverage: 15.78% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #107      +/-   ##
============================================
- Coverage     69.36%   67.61%   -1.75%
  Complexity      421      421
============================================
  Files            77       78       +1
  Lines          1335     1371      +36
  Branches        139      142       +3
============================================
+ Hits            926      927       +1
- Misses          371      404      +33
- Partials         38       40       +2

Impacted Files
Coverage Δ

...java/seedu/address/storage/JsonAdaptedMeeting.java
0.00% <0.00%> (ø)

...du/address/storage/JsonSerializableClientBook.java
100.00% <ø> (ø)

...c/main/java/seedu/address/model/client/Client.java
77.77% <8.33%> (-19.85%)
:arrow_down:

.../java/seedu/address/storage/JsonAdaptedClient.java
90.38% <50.00%> (-9.62%)
:arrow_down:

...a/seedu/address/storage/JsonClientBookStorage.java
85.71% <0.00%> (-14.29%)
:arrow_down:

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.


Dev

Adding new quick start CDK deploy rollup flow to the dev environment for perusal

Not going to merge, just for experimental purposes.


Bug: Path mapping between a Windows scene and a Linux worker does not work as expected

Describe the bug
The c4d.RDATA_PATH (output path) appear as a relative path on Linux when it is actually a Windows path.
For example:
On Windows: P:\ahughes\_DeadlineCloudExamples\simple_c4d_naming_output\image.png
On Linux: ./P:/ahughes/_DeadlineCloudExamples/simple_c4d_naming_output/image.png
The Linux path appears to be relative and therefore the map_path call returns a relative Linux path and the entire folder structure gets recreated relative to the scene file.
This causes additional issues because certain characters cannot be represented on Windows filesystems in paths (For example, the above path that the image.png gets written out to will be as follows P:\ahughes\_DeadlineCloudExamples\P_\ahughes\_DeadlineCloudExamples\simple_c4d_naming_output\image.png because Windows turns the P: into P_)
Expected Behaviour
I would expect Cinema4D to give us a non-relative path, but I guess we have to have the adaptor work around this and detect that the path is a relative Windows path and treat it as absolute?
Current Behaviour
A relative path is returned and converted to Linux, so the entire filesystem is recreated from the scene_file
P:\ahughes\_DeadlineCloudExamples\P_\ahughes\_DeadlineCloudExamples\simple_c4d_naming_output\image.png
Reproduction Steps
Create a scene in Windows Cinema4D and set the output path to be some absolute Windows path (I believe it converts it to absolute regardless) and then render the scene on a Linux Commandline worker.
Here is my modified plugin that you can use to just test the pathing:
# Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
import os
import sys
import traceback

print('c4d python version: %s' % sys.version)
print('system paths:')
for n in sys.path:
    print(n)

import c4d

def setTake(take_name):
    doc = c4d.documents.GetActiveDocument()
    take_data = doc.GetTakeData()
    if not take_data:
        return

    def get_child_takes(take):
        child_takes = take.GetChildren()
        all_takes = child_takes
        if child_takes:
            for child_take in child_takes:
                all_takes.extend(get_child_takes(child_take))
        return all_takes

    main_take = take_data.GetCurrentTake()
    all_takes = [main_take] + get_child_takes(main_take)

    take = None
    for take in all_takes:
        if take.GetName() == take_name:
            break
    if take is None:
        print("Error: take not found: %s" % take_name)
    take_data.SetCurrentTake(take)

def parse_argv(argv):
    for arg in argv:
        if arg.find("-RdataTest") == 0:
            import c4d
            scene_file = "<PathToYourSceneFile>"
            print("Loading: {}".format(scene_file))
            doc = c4d.documents.LoadDocument(
                scene_file, c4d.SCENEFILTER_OBJECTS | c4d.SCENEFILTER_MATERIALS
            )
            c4d.documents.InsertBaseDocument(doc)
            c4d.documents.SetActiveDocument(doc)

            setTake("Main")

            doc = c4d.documents.GetActiveDocument()
            render_data = doc.GetActiveRenderData()
            print("RDATA_PATH: %s" % (render_data[c4d.RDATA_PATH]))
            return True
    return False

def PluginMessage(id, data):
    if id == c4d.C4DPL_COMMANDLINEARGS:
        return parse_argv(sys.argv)
    return False

Environment
At minimum:

Operating system: Scene file created on Windows, Worker running RockyLinux9
Version of Cinema 4D: Cinema4D R26 on Windows Cinema4D 2024 on Linux
3: Version of this package: 0.5.4

Please share other details about your environment that you think might be relevant to reproducing the bug.

Thank you @Ahuge for the GitHub issue! We really appreciate the detailed reproduction steps and the sample scene. I've added this ticket to our backlog to investigate.

I've been investigating this issue this week and have reproduced it. I'll dig into the root cause!


add Ancient Hero Uruta, Modern Hero Ayane, Future Hero Muila, Cat-Eared Vtuber Miru to recruitment system

Add 4 new characters to the recruitment system

Their IDs: 118, 119, 120, 184

The files' format is slightly changed due to my new automatic system. I hope it's ok to do that.
Some backgrounds of characters' small pictures are manually removed since there's no official source for that.
Thanks!

Looks good. Thanks for your efforts.


Auto submit form to CRTC website

Entered all information using selenium and chrome driver, right up to the recapcha prior to form submission.

This is awesome!


Suggestion to add a license to the repository

Hello Mr. Ho!
Thank you for publishing these ansible roles, they are extremely helpful for setting up raspberry pis easily and quickly!
I've noticed that the repository doesn't currently have a license. I am using your work in an academic institution for research and we would like to open source our produced code and results responsibly. Having a clear license would help us and potentially others to figure out how to use your code in our work correctly.
Would you consider adding a license to the entire repository?
Best regards,
Valentin Bauer

Hey there, thanks for reaching out. I'm glad my work has helped you in some way. Definitely, will put up a license for this.

Thank you so much!


Utilities -> Parser -> Validating Entire Model -> Typo in "Example"

What were you initially searching for in the docs?
I was trying to understand how to implement the input model parser with validations.
Is this related to an existing part of the documentation? Please share a link
https://awslabs.github.io/aws-lambda-powertools-python/latest/utilities/parser/#validating-entire-model
Describe how we could make it clearer
The example provided uses the @root_validator decorator, but the import statement is only importing validator (which is unused).
If you have a proposed update, please share it here
In the import statement, please replace validator with root_validator.

Thanks a lot @nsandar for flagging this - I've just merged and it's available in the staging docs --- it'll be available in the latest today as part of the next release
https://awslabs.github.io/aws-lambda-powertools-python/develop/utilities/parser/#validating-entire-model


[#3942]If both acl and message trace are enabled and the default topic RMQ_SYS_TRACE_TOPIC is used for message trace, you don't need to add the PUB permission of RMQ_SYS_TRACE_TOPIC topic to the acl config.

Make sure set the target branch to develop
What is the purpose of the change
If both acl and message trace are enabled and the default topic RMQ_SYS_TRACE_TOPIC is used for message trace, you don't need to add the PUB permission of RMQ_SYS_TRACE_TOPIC topic to the acl config.
#3942
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Codecov Report

Merging #3943 (cf129ad) into develop (7a5d937) will decrease coverage by 0.01%.
The diff coverage is 40.00%.

@@              Coverage Diff              @@
##             develop    #3943      +/-   ##
=============================================
- Coverage      47.64%   47.63%   -0.02%
+ Complexity      4957     4956       -1
=============================================
  Files            633      633
  Lines          42577    42582       +5
  Branches        5589     5591       +2
=============================================
- Hits           20285    20282       -3
- Misses         19786    19792       +6
- Partials        2506     2508       +2

Impacted Files
Coverage Δ

...che/rocketmq/acl/plain/PlainPermissionManager.java
75.70% <40.00%> (-0.47%)
:arrow_down:

...rg/apache/rocketmq/common/stats/StatsSnapshot.java
84.61% <0.00%> (-15.39%)
:arrow_down:

...ache/rocketmq/common/stats/MomentStatsItemSet.java
39.13% <0.00%> (-13.05%)
:arrow_down:

.../apache/rocketmq/common/stats/MomentStatsItem.java
38.09% <0.00%> (-9.53%)
:arrow_down:

...va/org/apache/rocketmq/store/FlushDiskWatcher.java
81.25% <0.00%> (-9.38%)
:arrow_down:

...va/org/apache/rocketmq/common/stats/StatsItem.java
50.00% <0.00%> (-5.00%)
:arrow_down:

...ketmq/common/protocol/body/ConsumerConnection.java
95.83% <0.00%> (-4.17%)
:arrow_down:

...a/org/apache/rocketmq/filter/util/BloomFilter.java
59.13% <0.00%> (-2.16%)
:arrow_down:

...org/apache/rocketmq/common/stats/StatsItemSet.java
43.28% <0.00%> (-1.50%)
:arrow_down:

...he/rocketmq/client/trace/AsyncTraceDispatcher.java
79.70% <0.00%> (-1.49%)
:arrow_down:

... and 10 more

Continue to review full report at Codecov.

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

if the cluster is configured with a custom trace topic, this change optimization cannot be covered.
In the new version, the message trace should use standard protocols such as OpenTelemetry. The server's solution to store queries based on message topics is not long-term.

It may be better if we skip checks on specified topics in BrokerRpcHook where we can get trace topic configs.

but if we used  BrokerRpcHook to skip the tracing topic validation, we must add some validation logic in the broker, otherwise in ACL module, so how about only skipping the default tracing topic， and adding a description for the customized tracing  topic， just letting the admin add  permission for customized tracing  topic


Dynamic tank high network usage due to logic error

Refer to my last issue about excessive rendering update packet. #2690
It seems that there's an attempt to detect fluid type change by storing them into "prevFluid". But due to logic error causing the "prevFluid" to always be null and making needRenderUpdate() to always return true thus sending rendering update packet every tick for each tank.
Refer to this line
https://github.com/aidancbrady/Mekanism/blob/master/src/main/java/mekanism/common/tile/TileEntityDynamicTank.java#119

Done, thanks!


Image Request Loop Issue

When running Reel 1.3.0 on jQuery 3.3.1 I noticed in the Chrome Inspector Network tab it has an infinite loop where it keeps making requests to the same image over and over again.
Here is the HTML
<img src="assets/thumbmail.png"    width="960"   height="540"   class="reel"   data-image="assets/sprite.png"   data-frames="21"   data-footage="1"   data-responsive="true"   data-revolution="100">
In the Network tab, it will keep making requests to sprite.png over and over. Any idea what could be causing this issue? There are no errors reported in the console though.
Also when I view the DOM in the inspector, Reel keeps adjusting the DOM constantly as it keeps refreshing the container.

Same issue, any solution?

Any updates?

.bind >.on
.trigger > var event = new $.Event(eventType, eventData); $(element).trigger(event,eventData);
3**.!!!  all  $this.width()， $this.height() 》math.ceil($this.width()) ，math.ceil($this.height())**
setup: height= set(height, +t.attr(height) || math.ceil(t.height()) || opt.height),

function globalTrigger(element, eventType, eventData) {
// console.log('element, eventType, eventData',element, eventType, eventData)
var event = new $.Event(eventType, eventData);
$(element).trigger(event,eventData);
if (!event.isPropagationStopped()) {
element.parents().each(function() {
if (event.isPropagationStopped()) {
return false;
}
$(this).trigger(event,eventData);
});
}
t.trigger(  >> globalTrigger(t,


Metadata form 7: Access control and deletion behaviour

Feature or Bugfix

Feature

Detail

When entity is deleted all attached MF are deleted
When Org or Env is deleted MF with visibility in this org|env is deleted. And all connected attached MFs
Resource policies: CREATE_METADATA_FORM (create MF with visibility inside this entity), ATTACH_METADATA_FORM
Migrations for entity owners to have all permissions

Relates

#1541

Security
Please answer the questions below briefly where applicable, or write N/A. Based on
OWASP 10.

Does this PR introduce or modify any input fields or queries - this includes
fetching data from storage outside the application (e.g. a database, an S3 bucket)?

Is the input sanitized?
What precautions are you taking before deserializing the data you consume?
Is injection prevented by parametrizing queries?
Have you ensured no eval or similar functions are used?

Does this PR introduce any functionality or component that requires authorization?

How have you ensured it respects the existing AuthN/AuthZ mechanisms?
Are you logging failed auth attempts?

Are you using or adding any cryptographic features?

Do you use a standard proven implementations?
Are the used keys controlled by the customer? Where are they stored?

Are you introducing any new policies/roles/users?

Have you used the least-privilege principle? How?

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

With the new way of handling deletes of metadata forms with deletes of resources a couple things come to mind:

(1) We still have a dependency between MF module and other resources (i.e. metadata form tables must exist for org, env, dataset deletes to work) but since we always create all tables in DB (and I do not think we plan to change this behavior in the foreseeable future) I think this is okay

(2) Right now when we create a new form or attach a new form we do not create any permissions for the user/group on the form. I

Instead we protect backend resolvers for metadata forms by either the MetadataFormAccessService.can_perform() method (which checks if owner or not of the visibility entity) OR checking if has group has ATTACH_METADATA_FORM or CREATE_METADATA_FORM on the resource itself
I see the addition of some new permissions like UPDATE_METADATA_FORM_FIELD, EDIT_METADATA_FORM, etc. that are not currently used but if we do plan on assigning these permissions on the form moving forward this would have to also be cleaned up in DB trigger (just calling out the complexity of these triggers could expand)

What if we had a generic EventSystem class (in core or base?):
class DataallEventSystem:
    def __init__(self):
        self._subscribers = {}

    def subscribe(self, event: str, callback: Callable[..., None]) -> None:
        if event not in self._subscribers:
            self._subscribers[event] = []
        self._subscribers[event].append(callback)

    def publish(self, event: str, *args: Any, **kwargs: Any) -> None:
        if event in self._subscribers:
            for callback in self._subscribers[event]:
                callback(*args, **kwargs)

And then each MF Form resource could publish on delete:
class EnviornmentService:
    def __init__(self, event_system: DataallEventSystem):
        self.event_system = event_system

    ...

    def delete_environment(self, uri: str) -> None:
        self.event_system.publish("environment_deleted", uri)

And then MetadataFromService and AttachedMetadataFormService could import event system and subscribe:
class MetadataFormService:
    @classmethod
    def subscribe(cls, event_system: DataallEventSystem) -> None:
        event_system.subscribe("environment_deleted", cls.on_delete_env)
        event_system.subscribe("organization_deleted", cls.on_delete_org)
        event_system.subscribe("dataset_deleted", cls.on_delete_dataset)

    @classmethod
    def add_dependency(self, cls):
        self.dependencies.add(cls)

    def on_delete_env(self, env_uri):
        # Delete MF Forms and Attached Forms for env_uri
    ...

And lastly __init__.py for MF Module Interface would call:
MetadataFormService.subscribe(DataallEventSystem)
AttachedMetadataFormService(DataallEventSystem)

Not sure if this pub/sub approach is best but this way we can have multiple publishers and multiple subscribers (similar to EnvironmentResourceManager but not dedicated to just Environments and more flexible)
Just thinking out loud here - not sure if this is the right approach for the ask
cc @SofiaSazonova @dlpzx @petrkalos

With the new way of handling deletes of metadata forms with deletes of resources a couple things come to mind:
* (1) We still have a dependency between MF module and other resources (i.e. metadata form tables must exist for org, env, dataset deletes to work) but since we always create all tables in DB (and I do not think we plan to change this behavior in the foreseeable future) I think this is okay

* (2) Right now when we create a new form or attach a new form we do not create any permissions for the user/group on the form. I

  * Instead we protect backend resolvers for metadata forms by either the `MetadataFormAccessService.can_perform()` method (which checks if owner or not of the visibility entity) OR checking if has group has ATTACH_METADATA_FORM or CREATE_METADATA_FORM on the resource itself
  * I see the addition of some new permissions like UPDATE_METADATA_FORM_FIELD, EDIT_METADATA_FORM, etc. that are not currently used but if we do plan on assigning these permissions on the form moving forward this would have to also be cleaned up in DB trigger (just calling out the complexity of these triggers could expand)

What if we had a generic EventSystem class (in core or base?):
class DataallEventSystem:
    def __init__(self):
        self._subscribers = {}

    def subscribe(self, event: str, callback: Callable[..., None]) -> None:
        if event not in self._subscribers:
            self._subscribers[event] = []
        self._subscribers[event].append(callback)

    def publish(self, event: str, *args: Any, **kwargs: Any) -> None:
        if event in self._subscribers:
            for callback in self._subscribers[event]:
                callback(*args, **kwargs)

And then each MF Form resource could publish on delete:
class EnviornmentService:
    def __init__(self, event_system: DataallEventSystem):
        self.event_system = event_system

    ...

    def delete_environment(self, uri: str) -> None:
        self.event_system.publish("environment_deleted", uri)

And then MetadataFromService and AttachedMetadataFormService could import event system and subscribe:
class MetadataFormService:
    @classmethod
    def subscribe(cls, event_system: DataallEventSystem) -> None:
        event_system.subscribe("environment_deleted", cls.on_delete_env)
        event_system.subscribe("organization_deleted", cls.on_delete_org)
        event_system.subscribe("dataset_deleted", cls.on_delete_dataset)

    @classmethod
    def add_dependency(self, cls):
        self.dependencies.add(cls)

    def on_delete_env(self, env_uri):
        # Delete MF Forms and Attached Forms for env_uri
    ...

And lastly __init__.py for MF Module Interface would call:
MetadataFormService.subscribe(DataallEventSystem)
AttachedMetadataFormService(DataallEventSystem)

Not sure if this pub/sub approach is best but this way we can have multiple publishers and multiple subscribers (similar to EnvironmentResourceManager but not dedicated to just Environments and more flexible)
Just thinking out loud here - not sure if this is the right approach for the ask cc @SofiaSazonova @dlpzx @petrkalos
Answering to the downsides of using database triggers:
(1) I do not think it will be a problem for our own development.
(2) If triggers go further than deletes on a dependent table - if a delete implements business logic outside of database rules (foreign keys) I think we should avoid them. The main reason is that they can be difficult to debug.
(3) Additional concern - As a developer I might find it difficult to understand how metadata forms are deleted, it would not occur to me to look into each migration script for business logic code. Could we move the specific trigger definition to the metadata forms module? In addition, for the case we need to update a trigger, if we have the trigger code in the module we could track the history more easily (using git) than searching inside the migration scripts.

Finally, regarding @noah-paige pub/sub model vs current db triggers; db triggers are good enough for MF now. We should revisit the decision:

if MF needs to be applied to new/unexpected resources (e.g. customer external modules)
if the db triggers need to implement a complex logic or need to be updated often
In those cases I think the pub/sub model is a great candidate, it would be great if we could substitute other interfaces we are currently using for module communication (Environments - Resources, Datasets - Shares) and have a  generic way to handle intermodule event dependencies

@noah-paige @dlpzx
I like the idea of pub/sub model. We can revisit EnvironmentResources as well to unify the idea of intermodule communications.
But I don't think, we should include it into MF Prs. It's quite a big work on it's own, that requires a lot of tests as well.
I tend to keep triggers for now and then revisit it with new mechanism when it's ready.

One additional bug found during testing (thanks to @rbernotas) - when a user a part of the admin group tries to view a Metadata Tab for a resource, for instance a data.all Organization, the use receives an error 'NoneType' object is not iterable back from listEntityMetadataForm

This error stems from  L144 at dataall/backend/dataall/modules/metadata_forms/db/metadata_form_repository.py (i.e. orgs = list(set(user_org_uris).intersection(set(entity_orgs_uris))))
It is because user_org_uris and user_env_uris are set to None by get_user_admin_status_orgs_and_envs_() for admin users
I think we need to do something similar to what we did for entity_org_uris and entity_envs_uris and set the following:

        user_orgs_uris =  user_orgs_uris or []
         user_envs_uris = user_envs_uris or []

in query_entity_metadata_forms

Thanks for the latest changes @SofiaSazonova. 3 pending items I am seeing that I think need to be resolved for this PR:

(1) Downgrade Migration Script introduces errors (link)
(2) DA Admins can not attach Org or Env visibility filters to an Entity (link)
(3) Too many Forms showing as available when listing for an entity because of query filters (link)

++ 1 nit comment on triggers migration (link)
Afterwards I can re-test and think should be good for this PR
Additionally - there are some additional enhancements or fixes that we may want to pick up for MF Forms. I will add to the open issue https://github.com/data-dot-all/dataall/issues/1553

@noah-paige

Downgrades: I added fixes for this migration and redshift one (discussed it with @dlpzx )
I rolled back the queries as they were a couple iterations ago. May be it's not short, but it works as it should. It's also not bulky. That solves both problems with visibility, you mentioned
About MF permissions: I just want to have all triggers in sam place, so I left it there.

Thanks for the latest changes @SofiaSazonova. 3 pending items I am seeing that I think need to be resolved for this PR:
* (1) Downgrade Migration Script introduces errors ([link](https://github.com/data-dot-all/dataall/pull/1540#discussion_r1772025624))

* (2) DA Admins can not attach Org or Env visibility filters to an Entity ([link](https://github.com/data-dot-all/dataall/pull/1540#discussion_r1772011954))

* (3) Too many Forms showing as available when listing for an entity because of query filters ([link](https://github.com/data-dot-all/dataall/pull/1540/files/f63d539a44716527fbaf533865a970402475e2cc..7a04e7d3572af0d5e4a493b63e0f094cf65754de#r1759456519))

++ 1 nit comment on triggers migration (link)
Afterwards I can re-test and think should be good for this PR
Additionally - there are some additional enhancements or fixes that we may want to pick up for MF Forms. I will add to the open issue #1553

@SofiaSazonova - latest testing

(1) Bug in Downgrade (link)
(2) Fixed
(3) Fixed
++ sounds good on keeping the trigger in migration

@noah-paige I hope, this is the last one and I haven't missed smth now.


Update status for type inference and verification

Update status of AllReduceOp in type inference and many ops in Verification.

Change in column "Type Inference" for all_reduce op only -> yes(need-revisit)
Change in column "Verification" in scale: the rule is:

Verification        Type Inference
yes                 yes
yes*                yes*
yes(need-revisit)   yes(need-revisit)
yes(need-revisit)   no
accordingly         infeasible

Based according to most recently changes mode by other contributors.

Thank you for the thorough work in putting together this PR! I've finally found several hours to review it and make sure that it matches reality. This resulted in about a dozen fixes and a few stylistic changes, so I sent them out as a separate pull request (#331) which supersedes this PR.


Adopt ord visibilities

Description
This PR introduces filtering of APIs/Events based on their visibility field (public, internal or private).
If a call is made with a token containing the internal_visibility:read scope, all APIs/Events are returned (no matter whether they are public, internal or private). If the scope is not present, only the public APIs/Events are returned.
Changes proposed in this pull request:

new claim and @EdmProtectedBy relation for the visibility field

/hold cancel


High Availability with an External DB with CockroachDB is not working

Environmental Info:
K3s Version:
k3s v1.20.4+k3s1
Node(s) CPU architecture, OS, and Version:
amd64, ubuntu 20, k3s v1.20.4+k3s1
Cluster Configuration:
2 masters
Describe the bug:
level=fatal msg="starting kubernetes: preparing server: creating storage endpoint: building kine: storage backend is not defined"
Steps To Reproduce:
High Availability with an External DB with CockroachDB
CockroachDB is supports lib/pq driver, but k3s is not running with CockroachDB

The error message suggests you've entered an invalid datasource driver. Can you confirm that your datastore-endpoint value looks like postgres://username:password@hostname:port/database-name - in particular confirm that it is postgres:// not postgresql:// or pq://or cockroachdb:// or something else like that?

postgres://username:password@hostname:port/database-name

Can you confirm the exact arguments you're passing to K3s? Redact the password if necessary. The error message indicates you're using an incorrect scheme.

v1.20.6+k3s1
postgres://root@<IP_ADDRESS>:26257/k3sdb?sslmode=disable
this is working


feat(rake) add benchmark_task

We use a lot of promises, so we're taking a look and how we can tune performance a bit. We've cooked up some benchmarks (mostly @arthurschreiber!) and I wonder, is anyone else interested in these?
You can run them with rake:
$ rake benchmark > before.txt
$ # Make some tweaks
$ rake benchmark > after.txt
$ # Compare the outputs

The output includes a memory profile and some benchmarks (oops, but the markdown + code was not good 😆 ) :

Total allocated: 4512 bytes (86 objects)
Total retained:  0 bytes (0 objects)
allocated memory by gem
  4272  promise.rb/lib
   240  other

allocated memory by file
  3600  /Users/rmosolgo/code/promise.rb/lib/promise.rb
   672  /Users/rmosolgo/code/promise.rb/lib/promise/group.rb
   240  /Users/rmosolgo/code/promise.rb/benchmark/run.rb

allocated memory by location
  1040  /Users/rmosolgo/code/promise.rb/lib/promise.rb:41
   936  /Users/rmosolgo/code/promise.rb/lib/promise.rb:58
   656  /Users/rmosolgo/code/promise.rb/lib/promise.rb:20
   520  /Users/rmosolgo/code/promise.rb/lib/promise.rb:60
   448  /Users/rmosolgo/code/promise.rb/lib/promise.rb:24
   256  /Users/rmosolgo/code/promise.rb/lib/promise/group.rb:27
   256  /Users/rmosolgo/code/promise.rb/lib/promise/group.rb:28
   160  /Users/rmosolgo/code/promise.rb/lib/promise/group.rb:37
    80  /Users/rmosolgo/code/promise.rb/benchmark/run.rb:20
    40  /Users/rmosolgo/code/promise.rb/benchmark/run.rb:11
    40  /Users/rmosolgo/code/promise.rb/benchmark/run.rb:16
    40  /Users/rmosolgo/code/promise.rb/benchmark/run.rb:6
    40  /Users/rmosolgo/code/promise.rb/benchmark/run.rb:9

allocated memory by class
  1880  Promise
  1360  Array
   520  Promise::Callback
   512  Method
   160  Promise::Group
    80  Proc

allocated objects by gem
    81  promise.rb/lib
     5  other

allocated objects by file
    69  /Users/rmosolgo/code/promise.rb/lib/promise.rb
    12  /Users/rmosolgo/code/promise.rb/lib/promise/group.rb
     5  /Users/rmosolgo/code/promise.rb/benchmark/run.rb

allocated objects by location
    26  /Users/rmosolgo/code/promise.rb/lib/promise.rb:41
    13  /Users/rmosolgo/code/promise.rb/lib/promise.rb:58
    13  /Users/rmosolgo/code/promise.rb/lib/promise.rb:60
     9  /Users/rmosolgo/code/promise.rb/lib/promise.rb:20
     8  /Users/rmosolgo/code/promise.rb/lib/promise.rb:24
     4  /Users/rmosolgo/code/promise.rb/lib/promise/group.rb:27
     4  /Users/rmosolgo/code/promise.rb/lib/promise/group.rb:28
     4  /Users/rmosolgo/code/promise.rb/lib/promise/group.rb:37
     1  /Users/rmosolgo/code/promise.rb/benchmark/run.rb:11
     1  /Users/rmosolgo/code/promise.rb/benchmark/run.rb:16
     1  /Users/rmosolgo/code/promise.rb/benchmark/run.rb:20
     1  /Users/rmosolgo/code/promise.rb/benchmark/run.rb:6
     1  /Users/rmosolgo/code/promise.rb/benchmark/run.rb:9

allocated objects by class
    34  Array
    26  Promise
    13  Promise::Callback
     8  Method
     4  Promise::Group
     1  Proc

retained memory by gem
NO DATA
retained memory by file
NO DATA
retained memory by location
NO DATA
retained memory by class
NO DATA
retained objects by gem
NO DATA
retained objects by file
NO DATA
retained objects by location
NO DATA
retained objects by class
NO DATA
Allocated String Report
Retained String Report
Warming up --------------------------------------
Promise.all with promises
2.116k i/100ms
Promise.all with values
6.040k i/100ms
Promise.resolve    77.944k i/100ms
Promise.resolve.sync    69.861k i/100ms
Promise.resolve#then    27.488k i/100ms
Promise.new#then    48.168k i/100ms
Resolve promise with resolved promise
19.472k i/100ms
Calculating -------------------------------------
Promise.all with promises
20.515k (± 5.4%) i/s -    103.684k in   5.069609s
Promise.all with values
57.950k (±11.2%) i/s -    289.920k in   5.092571s
Promise.resolve    942.389k (± 7.7%) i/s -      4.755M in   5.082667s
Promise.resolve.sync    810.369k (±11.0%) i/s -      4.052M in   5.077646s
Promise.resolve#then    270.964k (±12.7%) i/s -      1.347M in   5.065296s
Promise.new#then    458.930k (±19.9%) i/s -      2.168M in   5.036475s
Resolve promise with resolved promise
191.382k (±11.9%) i/s -    954.128k in   5.075376s
Calculating -------------------------------------
Promise.all with promises
4.504k memsize (     0.000  retained)
86.000  objects (     0.000  retained)
0.000  strings (     0.000  retained)
Promise.all with values
1.624k memsize (     0.000  retained)
32.000  objects (     0.000  retained)
0.000  strings (     0.000  retained)
Promise.resolve   112.000  memsize (     0.000  retained)
2.000  objects (     0.000  retained)
0.000  strings (     0.000  retained)
Promise.resolve.sync   112.000  memsize (     0.000  retained)
2.000  objects (     0.000  retained)
0.000  strings (     0.000  retained)
Promise.resolve#then   344.000  memsize (     0.000  retained)
6.000  objects (     0.000  retained)
0.000  strings (     0.000  retained)
Promise.new#then   312.000  memsize (     0.000  retained)
6.000  objects (     0.000  retained)
0.000  strings (     0.000  retained)
Resolve promise with resolved promise
456.000  memsize (     0.000  retained)
8.000  objects (     0.000  retained)
0.000  strings (     0.000  retained)

What do you think, are these a good tool for maintaining this project?

I think the CI fail is in the dependency, right?
** Execute metrics:mutant
warning: parser/current is loading parser/ruby21, which recognizes
warning: 2.1.8-compliant syntax, but you are running 2.1.5.
warning: please see https://github.com/whitequark/parser#compatibility-with-ruby-mri.
rake aborted!
SyntaxError: /home/travis/.rvm/gems/ruby-2.1.5/gems/mutant-0.8.13/lib/mutant/mutator/node/send.rb:40: syntax error, unexpected ':', expecting =>
          '=~':          %i[match?],
               ^

Yup, it looks like that CI failure happens on master as well.  I'll bump the minimum version for mutation testing to 2.2 to avoid the problem

Thanks!


Support for multiple items in PayPal Website Payments Standard with payment_service_for helper

Is there any plans to add support for multiple items in payment_service_for for PayPal WPS? Current solution doesn't support.

It's not on the roadmap. If a pull request is provided that adds this it will be reviewed and possibly merged in.


Feature/420

Relates to #420
Here's a first attempt at parsing frictionless schema, with the following capabilities:

read from frictionless schema json/yaml files on disk
accept a frictionless Schema object directly from memory
map field names, types, constraints directly into pandera equivalents

The code to handle this is in pandera.io, and I've created a new FrictionlessFieldParser class to convert each field in the schema accordingly.
Currently pa.io.from_frictionless_schema() is as far as I've got in terms of an API - how/where do you want this to be imported from?
Documentation will also need updating - there are docstrings for everything I've added but nothing more than that at this point.
I've also added tests in tests.io to validate this functionality.

@cosmicBboy I've swapped this over to target v0.7.0 and added an example to the docstring so documentation should be a bit clearer now.
I've just realised I branched from master rather than from 0.7.0 directly, sorry - do you want this fixing so only my commits get covered by this MR?
Please let me know if you need anything else from me! I can possibly take a look at parsing open API schema too, though I haven't used them before. It should be easy to refactor what I've done here for anyone to extend coverage to other schema quite consistently in future, too (e.g. an abstract base class for column parsing, which is extended by FrictionlessField, OpenAPIColumn etc with a consistent API for constructing a pandera schema from them).

thanks @TColl this looks fantastic! ✨

I've just realised I branched from master rather than from 0.7.0 directly, sorry - do you want this fixing so only my commits get covered by this MR?

No worries, the branch situation is still evolving as more folks start contributing to the project! I just updated the release/0.7.0 branch to be on par with master. Would you mind rebasing your changes on top of the latest release/0.7.0 branch and re-opening the PR?
Will add comments/feedback after the rebase

thanks @TColl this looks fantastic! ✨

I've just realised I branched from master rather than from 0.7.0 directly, sorry - do you want this fixing so only my commits get covered by this MR?

No worries, the branch situation is still evolving as more folks start contributing to the project! I just updated the release/0.7.0 branch to be on par with master. Would you mind rebasing your changes on top of the latest release/0.7.0 branch and re-opening the PR?
Will add comments/feedback after the rebase

I've rebased now, though it's not pretty!

huh, weird... can you try making a new PR off of this branch? I think you'll have to make a new branch in your fork https://github.com/TColl/pandera/commit/a98aeb92edda632fb2165d1c855884f8f75a2a77
it looks like https://github.com/TColl/pandera/tree/feature/420 doubles up  on the improve documentation and parse frictionless schema  commits since I made those changes to github actions file.

huh, weird... can you try making a new PR off of this branch? I think you'll have to make a new branch name from this commit in your fork TColl@a98aeb9
it looks like https://github.com/TColl/pandera/tree/feature/420 doubles up on the improve documentation and parse frictionless schema  commits since I made those changes to github actions file.

I've force-pished a clean branch over the old one and cherry-picked the changes back in from the messy one - should be good to review now 👍

@cosmicBboy after making these latest changes I'm getting pylint E1136 (unscriptable object) errors for the newly-used typing.Optional and typing.Union type hints with the current pre-commit setup - I've raised #455 to cover this

hi @TColl I think for now you can put a #pylint: disable=unsubscriptable-object comment above the places where you get this error, this is accounted for in 3.9 tests here.
I think #455 would be a good time to look at other, less opinionated linting options... I've been finding that pylint has become more and more of a burden, though maybe that's because of python's growing pains adding more features in recent versions. If we do decide to stick with pylint unpinning it will probably yield more issues, so we can localize those changes in #455.
It looks like there are also some punit test issues](https://github.com/pandera-dev/pandera/pull/454/checks?check_run_id=2267711686), are you able to reproduce them locally? See here to run the testing suite that's on par with what CI runs.

hi @TColl I think for now you can put a #pylint: disable=unsubscriptable-object comment above the places where you get this error, this is accounted for in 3.9 tests here.

I've resorted to running pre-commit once to check those are the only errors I get, then committing with a --no-verify flag to bypass pylint failing on those specific issues. Neither option is great, but adding pylint pragma always feels like a failure to me!

I think #455 would be a good time to look at other, less opinionated linting options... I've been finding that pylint has become more and more of a burden, though maybe that's because of python's growing pains adding more features in recent versions. If we do decide to stick with pylint unpinning it will probably yield more issues, so we can localize those changes in #455.

I'm a fan of pylint personally, but I think #455 is a good place to have that discussion!

It looks like there are also some unit test issues, are you able to reproduce them locally? See here to run the testing suite that's on par with what CI runs.

I get the same result locally - some of them are trivial, if annoying (nan != nan for some versions of python), and can be fixed with some updating of the unit tests themselves to be more robust. Others are more concerning to me (the NoneType object has no 'get' method ones) and will need a code tweak or two to get it sorted. I'll get this done in the next couple of days.
On a side note, I'm a big fan of #397 purely for it halving the number of nox cases to run through - that full suite takes a while to burn through!

@cosmicBboy sorry for the delay (I got a puppy, which has taken up most of my spare time!) but the unit tests are now fixed.This was mostly my unit testing not capturing various nuances between pandas versions so the actual code is largely unchanged.


Remove the tempdir dev-dependency

tempdir is being deprecated and merged with the tempfile crate. But
instead of reviewing another crate it took less time to replicate the
small amount of functionality we need in the test code ourselves.
Alternative to #1237.

This is made in presumption that all tests run inside docker always?
Because of in compare to tempfile if you run two instances of cargo test on the same machine that would cause random failures, because of different instaces  of cargo test would use the same files.

@Dushistov that is why I added https://github.com/tokio-rs/mio/pull/1239/files#diff-2e4938596e921e8ab04f6272f8dbbe3dR21-R25, to remove the temp dir in init. So only running the same test concurrently would be a problem.
At no point was/is/or will be running inside docker a requirement.


TST: add geopackage test case

Need to add a tiny geopackage with a few layers, particularly for testing sql and where parameters since these are known to behave differently on invalid inputs.

I suppose this can be closed? Should have use closes instead of references in PR.


HTML output from marked is not always valid

The following produces invalid HTML:
marked.parse('<div>')

...and if the HTML is invalid, I get this error in console:

...and all the code examples after the invalid HTML part are broken.
I guess we could either:
a) report this as bug in marked (but they could say - well, you give inavlid html in, you get invalid html out - it's a big deal mostly with React)
b) check if the produced HTML is valid and throw an error pointing to the problematic file
c) use some other parser like react-markdown

What’s more important is that it’s invalid Markdown ;-)
But this commonmark looks very promising, I’ll try it.

Right now HTML is disabled in Markdown, not sure if we need to enable it.

I think not. Maybe at some point we'll need plain HTML code blocks, but that's a different story.

Then I’ll close this.


docs(concepts): add module federation

Close https://github.com/webpack/webpack.js.org/issues/3731

@chenxsan i know its draft but was interesting to read upfront. This is a very import addition

Oh right, it lacks sort, ima follow up


Demo page is not working

Ah thanks, I'm currently updating docs and didn't realize the old one had broken.
Will update soon.

Hey @vizo , docs should be back up: https://bestguy.github.io/sveltestrap
Let me know if you see any other issues, thanks for the heads up.


Remove temporary release override

We should be good to remove this now that we're in the 20.22.6 release according to Rally

Yeah I talked to Tara, she was going to try to keep an eye on that going forward (although a Friday holiday on branching week might not come up again for a long time)


bug: fallback lsp range format modifies the wrong buffer

Neovim version (nvim -v)
neovim: 0.9.5
conform: v5.1, commit: ad2b5ec
Operating system/version
archlinux latest
What is the severity of this bug?
breaking (some functionality is broken)
Steps To Reproduce
Provide two files:
a.cc:  maybe valid cpp source is enough, formatted or not

#define A_CC

#include <iostream>
using namespace std;

int main() { cout << "Hello World!"; }

b.cc: lsp format will change the content
#define B_CC

#include <iostream>
using namespace std;

int main() {
  cout << "Hello World!";
  return 0;
}

Configuration:

clangd lsp setup for cpp
conform format with fallback to lsp for the filetype: cpp

Operations

nvim a.cc
apply conform format (run at least once)
load b.cc with :e b.cc in nvim
apply conform format (this will trigger the bug)

bug behavior base on the above operation:
For the conform format operation on b.cc:
the first one may finish success, the latter format will raise Error without running the format callback:
Error executing vim.schedule lua callback: /usr/share/nvim/runtime/lua/vim/lsp/util.lua:501: Invalid 'start_col': out of range
stack traceback:
	[C]: in function 'nvim_buf_set_text'
	/usr/share/nvim/runtime/lua/vim/lsp/util.lua:501: in function 'apply_text_edits'
	.../share/nvim/lazy/conform.nvim/lua/conform/lsp_format.lua:24: in function 'apply_text_edits'
	.../share/nvim/lazy/conform.nvim/lua/conform/lsp_format.lua:103: in function 'handler'
	/usr/share/nvim/runtime/lua/vim/lsp.lua:1393: in function ''
	vim/_editor.lua: in function <vim/_editor.lua:0>

Additional context
You may use the lazyNvim: dev  branch to apply the test, use key shot  for conform format operation. the format() wrapper is located in init/utils.lua:format().

I found one way this could happen, but it's a race condition and the repro steps you're using don't look like they would trigger it. Worth giving it a shot anyway to see if this fixes it.
Could you set log_level = vim.log.levels.DEBUG and include the output of the log output from the repro steps? That should help provide more context for what's happening.
Also, the error is just about the format changes being out of range. What makes you so certain that it's because the formatting is applied to the wrong buffer, as opposed to some other bug related to generating the text changes?

I found one way this could happen, but it's a race condition and the repro steps you're using don't look like they would trigger it. Worth giving it a shot anyway to see if this fixes it.
Could you set log_level = vim.log.levels.DEBUG and include the output of the log output from the repro steps? That should help provide more context for what's happening.
Also, the error is just about the format changes being out of range. What makes you so certain that it's because the formatting is applied to the wrong buffer, as opposed to some other bug related to generating the text changes?

It may not be a race condition, the log with DEBUG show three entry of LSP format both on a.cc:
# vi a.cc; <C-f> to format on current buffer (a.cc)
13:07:02[DEBUG] Running LSP formatter on /mnt/tmpfs/a.cc
# :e b.cc; <C-f> to format on current buffer (b.cc)
13:07:11[DEBUG] Running LSP formatter on /mnt/tmpfs/a.cc
13:07:11[DEBUG] Running LSP formatter on /mnt/tmpfs/a.cc
# :q: prompt for modification of a.cc, there be extra empty line on the tail of  a.cc

Well, seems I'm making a mistake in the format() wrapper by passing a table from module directly to different format() operations:
function M.format()
	local format_args = require('init.options').plugins.format_args

	local have_fmt, fmt_util = pcall(require, 'conform')
	if have_fmt then
		-- get current formatter names
		local formatters = fmt_util.list_formatters()
		local fmt_names = {}

		if not vim.tbl_isempty(formatters) then
			fmt_names = vim.tbl_map(function(f)
				return f.name
			end, formatters)
		elseif fmt_util.will_fallback_lsp(format_args) then
			fmt_names = { 'lsp' }
		else
			return
		end

		-- notify with noice progress api
		local noice_progress = require('noice.lsp.progress')

		local fmt_info = 'fmt: ' .. table.concat(fmt_names, '/')
		local msg_id = noice_progress.progress_msg(fmt_info)

		-- format with callback, and notify on err
		fmt_util.format(format_args, function(err)
			noice_progress.progress_msg_end(msg_id)
			if err then
				vim.notify(err, vim.log.levels.WARN, { title = fmt_info })
			end
		end)
	else
		vim.lsp.buf.format(format_args)
	end
end

Here the fmt_util.will_fallback_lsp(format_args) will insert bufnr into the table,
and the modified table is always passed to next format operation.


Ignore keypresses if ctrl/option/alt is pressed.

Addresses https://github.com/rgthree/rgthree-comfy/issues/162

I think this has a chance to disrupt people's usage who happen to be using ctrl/cmd/alt and I was trying to come up with a way to work around that but, also, I think this can just go in and see if anyone complains.
First, though, you need to run the ./__build__.py script for the JS to build.

Not sure how I missed that. Maybe it got dropped when switching between branches.


[WEB-3063] use provided date range for latest pump settings query

WEB-3063 given a start and end date, ensure that the latest pump settings fetch respects those bounds

/deploy qa2


Create first servlet with example content

Followed the step-1 tutorial to modify my first servlet.

Commits (addressing comments):
Fix placement and style of final string variable,
and remove unnecessary blank line


feat: initial observability stack

adds cortex, loki, and grafana
adds generic dashboards for cortex and loki

leaves behind comments for setting resource requests/limits and hpa, I
intend to add those after determining a baseline of utilization

:tada: This PR is included in version 1.0.0-alpha.17 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


perf(runtime-core): should not trigger child update if changed prop is model listener

fix #4204

There are edge cases where this can lead to incorrect behavior:

If the item has been replaced but has the same key
If the v-model is bound on a destructured object which has been replaced

So the de-optimization is necessary to ensure correctness.


The user should choose the font

The package should let the use choose the font he wants to use. Otherwise it can cause problem when using HarmonicBalance in practice. For example, I use a lot of unicodes in my variable naming. However, the forced computer modern font does not recognise the unicodes in ploting such that I get plotting results seen in the figure.

The solutions would not touch the font setting of Plots.jl in the function
function _set_Plots_default()
    font = "Computer Modern"
    default(linewidth=2, legend=:outerright)
    default(fontfamily=font, titlefont=font , tickfont=font)
end

You can do this via a keyword argument to plot
In your case plot(....; titlefont="Times") should do.
Any keyword arguments are passed to Plots.plot.

This should be doable via a keyword arguments, these are all passed to Plots.plot
Here `plot(...; titlefont="sans-serif")
Computer Modern is picked for consistency with the LaTeX labels

Ha but I think you misunderstand the problem. You set computer modern by default for every plot. Also the plots outside the package. So if I first plot via the HarmonicBalance package and afterwards plot a random plot like x = -5:0.1:5; plot(x, x.^2, title = "x₀"), the plot will be in the "computer modern". I think it is better to not touch the default setting over whole Plots.jl. If you want the HarmonicBalance plots to be in "computer modern" you can do it manually in the HarmonicBalance.plot function. In that way, the HarmonicBalance plots are computer modern, but the 'self-made' plots designed by the user during the session are not touched and will be by default in "sans-serif". HarmonicBalance is in that way better designed for further processing data and better compatible with other packges who, for example, plot unicode.

tldr: I wouldn't touch the 'Plots.default' function and implement that "computer modern" is the default plot font differently
But it is just a suggestion :p
, one can also just change the default font after it plotted with HarmonicBalance
. I know it is nitpick :stuck_out_tongue: , I can try to implement it differently and make a pull request. ( For clarity, "computer modern" stays default font)


Plat EFR32: Update the device name supported by JLinkGDBServer.

Codecov Report

Merging #1705 into master will increase coverage by 0.36%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1705      +/-   ##
==========================================
+ Coverage   66.87%   67.23%   +0.36%
==========================================
  Files         171      171
  Lines       21277    21279       +2
  Branches     2610     2610
==========================================
+ Hits        14228    14307      +79
+ Misses       6140     6061      -79
- Partials      909      911       +2

Impacted Files
Coverage Δ

examples/platforms/posix/random.c
84% <0%> (-12%)
:arrow_down:

src/core/api/thread_api.cpp
34.75% <0%> (-5.35%)
:arrow_down:

src/core/net/netif.cpp
57.64% <0%> (-4.12%)
:arrow_down:

src/core/api/thread_ftd_api.cpp
41.55% <0%> (-2.6%)
:arrow_down:

src/ncp/ncp_base.cpp
28.71% <0%> (-0.41%)
:arrow_down:

src/core/thread/mle_router.cpp
79.53% <0%> (ø)
:arrow_up:

src/core/common/message.cpp
96.32% <0%> (+0.45%)
:arrow_up:

src/core/thread/mesh_forwarder.cpp
78.38% <0%> (+0.48%)
:arrow_up:

src/core/net/udp6.cpp
84.76% <0%> (+0.95%)
:arrow_up:

src/core/thread/topology.hpp
89.47% <0%> (+1.31%)
:arrow_up:

... and 6 more

Continue to review full report at Codecov.

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


fixed support for a custom-defined package names via option go_package

Fixes #63

@ysugimoto I've merged your master branch there as well. Also, I've included two commits:

ba7ab0c predates any changes, it's just an ordinary starwars example fixed without any go_package changes talked about in #63.
179c5f6 however has the re-generated output with slightly modified go_package option, have a look.

Notably, the only real changes to the starwars.graphql.go are the "github.com/graphql-go/graphql"'s Name fields under it's type config. Earlier it was Name: "Starwars_... and now it is Name: "Starwarsv1_....

As for the compile-time errors, I haven't got any in the resulted output, though I haven't found the time to test it end-to-end. I do, however plan on opening another issue inspired by grpc-gateway's project. In their project there's an utility called protoc-gen-openapiv2. I'm thinking doing the same for the graphql spec. The reason being is simple, it may be wiser to leverage the existing codebases to generate all the code for you, I'm talking about gqlgen. This project looks really promising compared to others and the only thing it needs is the .graphql schema files. Analogous to protoc-gen-openapiv2, having that generator for graphql schema will let you have the versatility with other existing, more prominent projects.

@qwerty-v This PR is ready to merge so could you please update the base branch? I can merge after that.

@ysugimoto done


Rely on the request template to specify the timeout.

cc @wikimedia/services

Coverage increased (+0.5%) to 89.677% when pulling 661b672d3131c97517f8e1de5cc3d879245be01e on Pchelolo:simplify_timeout into 340cf9acc29ad19878d03b17ede54f0d320a81e2 on wikimedia:master.


Make use of Faraday::GetMethodOverride optional

I made this middleware optional, so that it can be deactivated.
It still defaults to true because that's why previous users of the library expect. We should consider switching it to false at some point.

Coverage increased (+0.002%) to 98.07% when pulling cb434c22d3d3a781933f609ae5c20c6d9f499df7 on optional-method-override into 9b79ec7dfb1726815636c9fa1b3429c57bf068fa on master.

Coverage increased (+0.002%) to 98.07% when pulling 087f7c34498e96d3ea38f39e2e1763667e335f2d on optional-method-override into 9b79ec7dfb1726815636c9fa1b3429c57bf068fa on master.

Coverage increased (+0.004%) to 98.07% when pulling 4c1f6c1c071b9e389f3d0251d0b267da31a2627f on optional-method-override into da81f45eb2e9ee8b81a4112e42f3b48218472e97 on master.

Coverage increased (+0.004%) to 98.07% when pulling 2ea35f683f94c22192d9c8909545006fae7f22fc on optional-method-override into 5cd24ca1840716668944d40b40c43383fff9744f on master.


Support for UE 4.11 preview 7

Hi,
Awesome plug in, but it does not run under 4.11, does not want to compile. Ive tried to add C++ class, but no luck.

Just FYI it ran fine on 4.10.4.
Maybe if you have that version you may want to try that .

Yes, I've try it on 4.10 and it works fine. The problem is that, I do visualizations and their is allot of cool things for static light in 4.11.

Same problem with 4.11 final build. I have an error compiling the plugin >,<

I ran into the same issue but was able to fix it. There have been changes
to the code that plugin is based on (SimpleHMD) in version 4.11. Replace
the differences in VS, then rebuild, and it should work.

Semeon

On Fri, Apr 1, 2016 at 8:20 AM, doomgrave<EMAIL_ADDRESS>wrote:

Same problem with 4.11 final build. I have an error compiling the plugin

,<

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/NivalVR/UE4Cardboard/issues/5#issuecomment-204394448

@risoms Im not good in c++, can you please specify what needs to be replaced in VS?

I'm not near a computer. The code needed to be changed will be clear when you try to build your project. Each error in this plugin will have to be replaced by the 4.11 equivalent in simplehmd.

Semeon

On Apr 1, 2016, at 09:16, Irex777<EMAIL_ADDRESS>wrote:
@risoms Im not good in c++, can you please specify what needs to be replaced in VS?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

Dear Guys i have managed to fix for 4.11 , also changed names of classes to be uniqe than simplemhd so we now have our ownplugin
here my latest fork git
https://github.com/ANGames/UE4Cardboard.git
hope it help

Fix by @engahmedfawzy was merged


Cheat is bad

Fedoraware Version Date: Most Recent
Config used:

N/A
Description:

The cheat is like a big obese woman.
Steps to reproduce:

[x] I am able to consistently reproduce this error

Steps to reproduce the behavior:

Inject the cheat
Open the menu
Sit in awe at how useless some of these features are

Additional context

Please put the cheat on a diet, tysm.

[x] I have included the steps to reproduce to the best I am able to share.

Cope

Gay

Gay

no u

Why'd u close the issue mfed its TRUE

how hard would it be to recode fedoraware from litebase?

What would that achieve tbh?

m_Write is null?

Wouldn't it be possible to simply replace m_Write with a stringstream, and then only safe this stringstream to a file, if everything else worked fine? Instead of writing to the file directly?

or just check if m_Write exists prior to writing to it (this is what I am doing and it is working fine lol)

c++ moment


Geode Released when...

Your Question
When is the new Geode 4.0.0 update:
GEODE 4.0.0:
Add Platforms Mac, Windows & Android34, Android64 // All Mods Supports 2.205/2.206/2.207

Blocked for spamming


Not an issue, but clarification needed

Hi,
I'd like to ask about SEND_WS_0() function. I know how ws2812 work and I get idea of i.e. this line
time = 3; while(time--) WRITE_PERI_REG( PERIPHS_GPIO_BASEADDR + GPIO_ID_PIN(WSGPIO), 1 );
but I don't see any delay in this while loop. So this must mean that 3 execution of WRITE_PERI_REG function take about 0.35us (T0H from ws2812 datasheet) and 8 execution of WRITE_PERI_REG take 0.8us (T0L from ws2812 datasheet) , right?
So approximately one execution of WRITE_PERI_REG take 8 CPU cycles ( 0.1us == 10MHz, ESP2866 is running with 80MHz) ?
Secondly, if I understand it correctly, ets_intr_lock(); disables interrupts to avoid situatuation when data transfer to ws2812 is not finished and something breaks it?
Thirdly, which source you recommend to check i.e. registers of ESP8266, use SPI etc?

That takes waaay more than 8 cpu cycles!  There's 1 write buffer, so it's probably more like 60 something cpu cycles!  Only instructions that deal inside the CPU core are about 1 clock per cycle.  RAM is typically 2, FLASH 5 or 6, other IO even more, yet!
I do ets_intr_lock() to disable interrupts that could mess up timing.
I don't know what you're referring to with the 3rd thing.

@cnlohr at third thing I mean some kind of datasheet which describe avilable peripherals.

I don't really know what to offer you.  The only way I find anything is by googling, a lot.  I don't have any links to datasheets or peripheral lists or anything like that.  I just google keywords like esp8266 i2s or stuff like that.


[INFO] New repository

Hello there,
I am keeping a up-to-date version at https://github.com/BoboTiG/marisa-trie. For those needing wheels (CPython & PyPy), Python 3.9 and 3.10 support, up-to-date Cython files, etc., just have a try:
$ python -m pip install -U marisa-trie-ng

@BoboTiG I'm not sure forking is a productive way forward. Happy to add you marisa-trie maintainers, if you would like to look after this package.
Wdyt @kmike?

Forking is not ideal, indeed. We should find a way to uniformize changes from other forks as well. I would be glad to help as a maintainer here, I would backport my changes here.

If so, it would be a good thing to add myself as PyPi mainainer too, to prevent having different modules on PyPi.

Cool, thank you! I've added you as a maintainer both here and PyPI.

Great, thank you :)


BUG: Added hovering effect to quick links in footer

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?

Adding hovering effect to links in quick links in footer
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

I can add hover effect on the quick links so that become quite user friendly please assign it to me


Update README.md

We should add or promote a section on how to become a contributor.

It'd be nice if there was an explicit community call out for how they can join / participate!

anything concrete in mind @ianlivingstone?

i'll merge this guy up, we can open another issue/PR for the remaining!


Makes base64 encoded images for debug grid smol

This should make the final file even smaller.
Note: I didn't commit the compiled CSS in this... for some reason it changed the version in the comment to Tachyons 4.6.3 instead of 4.7.2, for some reason... as well as some vendor prefixes. I can totally add that compiled code if you'd like! I wasn't sure what you'd prefer based on contributor CoC.

Merged this into the 4.8.0 branch. Thanks!


Provide dependencies management in collector root pom

Because there are a lot of libraries used in collector, and some PRs may add more, in order to manage those well, I proposal to use maven dependencyManagement to list all used libraries and their versions in collector root pom.
All modules in collector-* are not allowed to dependency an existed library with given version. Update version should be taken seriously.
I hope this could be provided at least before #871 merging.

OK, I finish this tomorrow.

Provided, closing.