silent-renew request status canceled

We are facing a problem frequently when silent-renew happening, below are the details.
When iframe started requesting for silent-renew, request going smoothly but at the time of return response it is showing pending and immediately canceling the request and showing status as canceled.
Below is the silent-renew scenario giving error while testing against Azure B2C

Oidc-client application -> Azure B2C : 302 status and redirecting to Google Auth
Google Auth: 302 status and redirecting to Azure B2C
Azure B2C -> Application: Status canceled

We also attaching fiddler traces for both success and failure scenarios for silent-renew, can you please check and let us know when this can happen.
PS: we increased silent-renew timeout also but no use of that.
Silent-renew Fiddler Traces.zip

You seem to have something similar to my issue https://github.com/IdentityModel/oidc-client-js/issues/172

You will need to debug this yourself. I don't support Azure B2C -- that's a Microsoft's product. If you find a bug in this FOSS library, then feel free to report it.

Currently we are trying Azure B2C with hello.js and it is supporting this silent renew feture without fail. As many times as and it is not failing even single time, but yes oidc-client library failing frequently. We assume it is the issue related to iframe which is trying to load requests and one of them failing. I am not sure what is the actual issue but as we tried same feature with hello.js it got worked with out fail.


records: remove use of old vocabularies

Needs https://github.com/inveniosoftware/invenio-rdm-records/pull/745


可不可以添加cocoapods依赖呢

你好，
首先感谢作者的无私奉献，这里我有一点意见：
我的项目使用cocoapods进行项目的第三方库的依赖，本项目能否也添加到cocoapods托管呢？

There is no .podspec file available. You could create one yourself and then in your pod file point to that. Of course it would be better if a .podspec was included in the Reflect repository. Maybe @CharlinFeng can create one?


digdag reschedule

Fix https://github.com/treasure-data/digdag/issues/996
Also, supported digdag reschedule --skip-to now to make it easy to skip all disabled, but enqueued jobs. ref. https://github.com/treasure-data/digdag/issues/995
Most of implementations are referred from digdag enable.
I did not add tests because I could not find any existence tests 😢
DISCUSSION
Are there any standard ways to handle errors in a batch command?
I mean that, assume there are schedules with id 1 to 8 in a sample project, and schedules with id 1, 3, 7 result in
error: Request conflicted: {"message":"Specified time to skip schedules is already past","status":409}

with digdag reschedule <schedule-id> --skip-to now.
With the implementation of this PR, digdag reschedule <project-name> --skip-to now may stop at the 1st error occurred, and rest of schedules are not rescheduled.
To achieve my demand at https://github.com/treasure-data/digdag/issues/995, I want to apply --skip-to all schedules in a project anyway, which should result like
error: Request conflicted: {"schedule_id":1, "project": "sample", "message":"Specified time to skip schedules is already past","status":409}
error: Request conflicted: {"schedule_id":3, "project": "sample",  "message":"Specified time to skip schedules is already past","status":409}
error: Request conflicted: {"schedule_id":7,  "project": "sample", "message":"Specified time to skip schedules is already past","status":409}

But, I do not know correct ways to merge theses three errors into one error.
If this is difficult by design, I will drop digdag reschedule <project-name>. Please let me know.
EDIT: Removed digdag reschedle <project-name> batch support for now. Please let me know if there is a good way to handle errors in a batch request.

Added acceptance tests. Ready to get reviewed.


automate "replaces:" version in config/....csv.yaml file

The following file now has a "replaces:" stanza that points the previous version.

config/manifests/bases/susql-operator.clusterserviceversion.yaml

Currently this version number is maintained by hand, It would be nice to automate this.

(depending on the ci.yaml updateGraph setting, "replaces" might not be needed...

It is looking like with updateGraph=semver-mode we don't need "replaces".  Closing.


Enhance MappesCondition - use custom conditionDescriptionWithStatus

Summary
MappesCondition is a Wrapper-Condition like Join.
It should override conditionDescriptionWithStatus to print out also all inner Conditions using the DescriptionWithStatus
Example

##from Join
  @Override
  public Description conditionDescriptionWithStatus(T actual) {
    Description descriptionsWithStatus =  innerCondition.conditionDescriptionWithStatus(actual);
    String prefix = status(actual).label + " " + descriptionPrefix() + PREFIX_DELIMITER;
    String suffix = SUFFIX_DELIMITER;
    return new JoinDescription(prefix, suffix, descriptionsWithStatus);
  }

The default match method could also give better results by buildMappingDescriptionAFTER executing the condition.matches()

// FROM
  @Override
  public boolean matches(FROM value) {
    TO mappedObject = mapping.apply(value);
    String desc = buildMappingDescription(value, mappedObject);
    describedAs(desc);
    return condition.matches(mappedObject);
  }

// TO
  @Override
  public boolean matches(FROM value) {
    TO mappedObject = mapping.apply(value);
    boolean result=condition.matches(mappedObject);
    String desc = buildMappingDescription(value, mappedObject);
    describedAs(desc);
    return result
  }

If this is okay for you I will come up with an PR;

I'm not sure I follow, the proposed change does not pass the result to the buildMappingDescription, how would it give a better description?
can you show an example?


[Feature Request] Move Backglass Manager

alk7mist — Yesterday at 10:19 PM
How about adding "Backglass Manager" to the left margin on the Home screen maybe under "Tables" button (as an alternate way to get there instead of having to right click a table)?  And in Backglass Manager itself, would it be possible to add the same Playlist filter that's available in the Home Screen?  Thank you!

Fixed with 3.4.0


check_value() fails when using wait_num_rising_edge(clk, 1) instead of wait until rising_edge(clk)

Hey everyone,
Note: I am using ModelSim INTEL FPGA STARTER EDITION 10.6d Revision: 2018.02 for my simulation.
I am using a simple testbench found on github that is using UVVM. I changed some things to adapt it to my project, and I have found an issue, but I don't know how it can happens. The issue occurs there:
for il in 0 to 267 loop
      wait_num_rising_edge(clk, 1);
      -- wait until rising_edge(clk);
      check_value((((encodeout_r /= decodein_n_r) and (encodeout_r /= decodein_p_r))), false, tb_warning,  "Check encoding", C_SCOPE);
    end loop;

All the signals are well defined and assigned just after the wait above.
The issue is that when I am using the wait_num_rising_edge(clk, 1) procedure, my test fails

But you can see in the note that the result is correct. However, when I am using the native wait until rising_edge(clk) procedure, I don't have the issue (in fact I have it, but only on the first value tested and it is correct too, so still weird that this can happen).
When I take a look at the procedure wait_num_rising_edge, it is define like that:
-- Wait for number of rising edges
    if num_rising_edge /= 0 then
      for i in 1 to num_rising_edge loop
        wait until rising_edge(clk);
      end loop;
    end if;

So, I don't get how there can be any trouble. If you need more details about my files, ask me!
Br

Hi @guilbaudl
I can not see exactly why you get different results. Does the test with await_num_rising_edge() method fail on the first iteration of the loop? And with the regular 'await until rising_edge()' you only have a failing test at the initial loop iteration? As you have already pointed out there are not many changes in behavior from the UVVM method and the regular 'await' statement, but there is a delta cycle wait in the UVVM method. Could you try to add that to your code? I.e.
for il in 0 to 267 loop
wait until rising_edge(clk);
wait for 0 ns;
check_value(((encodeout_r /= decodein_n_r) and (encodeout_r /= decodein_p_r)), false, tb_warning,  "Check encoding", C_SCOPE);
log('Loop iteration=' & to_string(il));
end loop;
Br,
Marius

Hi @mariuselv
The test with the UVVM method wait_num_rising_edge() fails at the the second iteration, but this is the first one after the reset

With the regular wait until rising_edge() there is just this fail at the second iteration.
When adding wait for 0 ns, I have the same issue than when I use the UVVM method, you though well. However, when I write wait for clk_period, I get the same result than if I just write wait until rising_edge()
It seems a bit dark to understand how these errors could occur considering that the values stored in the signals are correct

Hi,
This seems to be a delta cycle problem, i.e. one or more of the encodeX / decodeX are assigned a new value on the same delta cycle as the check - after the rising edge of the clock. Open the simulations in wave view and check when the values get updated.
Br,
Marius

Hi,
Did you find a solution to your problem or UVVM?
-- Marius and Espen


AzurePowerShell@3 - Failed to Initialize Existing Az modules

New issue checklist

[X] I searched for existing GitHub issues
[X] I read pipeline troubleshooting guide
[x] I checked how to collect logs

Task name
AzurePowerShell
Breaking task version
5
Last working task version
5
Regression Description
Since today, our pipelines are broken with the following warnings & errors:
##[warning]Failed to Initialize Existing Az modules, fallback to installation

##[error]Failed to create access policy: The term 'Set-AzKeyVaultAccessPolicy' is not recognized as the name of a cmdlet...

Environment type (Please select at least one enviroment where you face this issue)

[ ] Self-Hosted
[X] Microsoft Hosted
[ ] VMSS Pool
[ ] Container

Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Azure DevOps Server Version (if applicable)
No response
Operation system
windows-2022
Relevant log output
##[warning]Failed to Initialize Existing Az modules, fallback to installation
Install-Module -Name Az.Accounts -RequiredVersion 2.19.0 -Force -AllowClobber -ErrorAction Stop -SkipPublisherCheck
Import-Module -Name Az.Accounts -Global -PassThru -Force -RequiredVersion 2.19.0
##[warning]Both Az and AzureRM modules were detected on this machine. Az and AzureRM modules cannot be imported in the same session or used in the same script or runbook. If you are running PowerShell in an environment you control you can use the 'Uninstall-AzureRm' cmdlet to remove all AzureRm modules from your machine. If you are running in Azure Automation, take care that none of your runbooks import both Az and AzureRM modules.

## Initializing Azure Complete
## Beginning Script Execution
##[error]Failed to create access policy: The term 'Set-AzKeyVaultAccessPolicy' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

Full task logs with system.debug enabled
No response
Repro steps
No response

Hi @QuickScoP3s
Thanks for reporting! Could you please share the complete debug logs of the pipeline at<EMAIL_ADDRESS>

@v-schhabra Apologies for the delay, sent the logs to your email
Thanks!

This occurs because you have not installed / imported / loaded the Az.Keyvault module which contains the Set-AzKeyVaultAccessPolicy cmdlet, which should be the reason why your code does not recognize the command as the name of cmdlet.

Note about Windows 2022 image
In the MS-hosted windows-2022 image the following software exists:
https://github.com/actions/runner-images/blob/main/images/windows/Windows2022-Readme.md
In this we can see the following:

In Azure Powershell module 2.1.0 we can under Package Details that module Az.Keyvault is installed with version 1.2.0:

So the Az.Keyvault module should already exist on the image, but with an really old version, I have no idea why they are not loaded in your code though.

Closing this issue as cx is using task AzurePowerShell@5 and it is working fine for them.


simplify earmark to @

From usability perspective, it is problematic to write a long earmark correctly every time. Mistakes happen. The system should support detecting those mistakes. Therefore, we propose @ to be used as the default earmark, so that it will be removed from the name and signature. This way it is easier for the developer to detect if the signature line is included: is there a @ or not. When there is a @ and the module path is hastily written and wrong, the line is still included into the docs, and thus made easier to spot for improvement.
By contrast, without a clear indicator like @, a hastily typ(o)ed package name, like yamdoc, yandog, vs yamdog, would not be included into the docs and would not cause any error. To ensure that every feature has an entry in the docs, the developer would need to go through each unit one by one with much more cognitive burden. Therefore it is better to have a clear and simple signal in the code that shouts "include this line into the docs!!!"

I played around with the @-syntax... It does not feel good. It loses something. It is extra syntax. It lures us towards relative signatures. Relative paths hide the context. It is a great power but also a great source of problems. Relative syntax forces the parsing to be aware of the context to form the full signature. To give an example of a problem, there can be multiple paths to the same module. If relative signatures were used, the developer needs to understand how the import order affects the signature paths. For another, the relative signatures need to be coupled absolute syntax, for custom signatures and intended hierarchies. Too much hassle, too much dependency on the project structure. Dependency is a source of problems. Let us praise the explicit markup regardless the manual labour it may cause. It simplifies things so much.


🐛 Fix virtual scrolling issue in Game Archives screen, causing only a limited amount of games to be rendered

Fixed in v3.3.2


Page is blank(npm run build)

Before opening a new issue, please take a moment to review our community guidelines to make the contribution process easy and effective for everyone involved.
Description
A clear and concise description of what the bug is.
Steps to reproduce
Steps to reproduce the behavior:
(Add link to a demo on https://jsfiddle.net or similar if possible)
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Versions

React-Boilerplate:
Node/NPM:
Browser:

This is empty. Closing.


Hotel search criteria recursion error

explicitly specifying the argument name in the JMS annotations causes it to throw recursion error.
AvailabilityStatus should be string
set cdata to false

This change is

A new inspection was created.

The inspection completed: No new issues

@djaney Thanks :)


[Bug]: "selectedOptionValue" in TreeSelect widget does not work as expected

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Even when we clear the selected option in a TreeSelect widget, the value of the same is still stored.
Note:

This happens when there is a Default Value mentioned for the TreeSelect widget.
This is happening only in release and not in Prod.

https://www.loom.com/share/efbfe26332a948f5b9d63917229b8e25
Steps To Reproduce

D&D TreeSelect & Text widgets.
Bind the Text widget to the TreeSelect widget such that the Text property of the Text widget is {{TreeSelect1.selectedOptionValue}}.
Now clear the selected option in the TreeSelect widget.

Expected: The text in the Text widget should also disappear.
Public Sample App
No response
Version
Cloud

This issue is still prevalent on Release hence reopening it.
https://www.loom.com/share/b82061c625904198a7292c7dd890a42a

Fixed already, in https://github.com/appsmithorg/appsmith/pull/16401


Add a control over amount of characters in 'before' and 'after' context windows

This PR adds an ability to control how many characters are being added into the context windows of before and after the cursor.
Sometimes files are way too large, and model's context window is not ready for that;
Or running an inference on that large chunk of text causes OOM problems.

Closes #23

I've refactored the code to make it more concise and added the new fields to the docs, thanks!


CASMINST-3664: Store loftsman ship log in its own configmap, and remove loftsman.io/previous-data annoation

What type of PR is this?
Bug fix.
What this PR does / why we need it:
The loftsman.io/previous-data annotation that is added to the ship result configmaps in the loftsman namespaces has a size limit fo 256KB. When the size of a manifest and loftsman log from a previous run have a combined size of 256KB or more, then loftsman will fail to (re)deploy a manifest with the same name.
In this PR the loftsman.io/previous-data annotation is has been removed, and loftsman will no longer keep track of the previous contents of the ship result configmap. As we have hit situations were the size of the generated core-services manifest has exceeded 256KB. After a ship operation the loftsman logs are now stored in their own config map
This allows for a manifest and its log file to grow up to 1MB in size (this is a limit imposed by k8s and ETCD for the size of a single configmap). In my testing the size of the manifest and loftsman log will be approximately the same.
Which issue(s) this PR fixes or finishes:

CASMINST-3664

Testing
Local testing
For local testing I was using a manifest with the size fo 343KB.
[~/Documents/Work/Repos/Github/loftsman]$ ls -lh manifest.big.yaml
-rw-r--r-- 1 rysjostran 343K Dec  9 12:53 manifest.big.yaml

The 1.1.0 version of loftsman is able to deploy a manifest that is 343KB in size if it hasn't been deployed before, since there is no data needed to be persisted in the loftsman.io/previous-data annotation
[~/Documents/Work/Repos/Github/loftsman]$ ./bin/loftsman-darwin-amd64 --version
loftsman version 1.1.0
[~/Documents/Work/Repos/Github/loftsman]$ ./bin/loftsman-darwin-amd64 ship  --charts-repo https://charts.bitnami.com/bitnami --manifest-path manifest.big.yaml
<truncated>
 chart=nginx command=ship namespace=default version=9.5.16
2021-12-09T16:45:58-06:00 INF Ship status: success. Recording status, manifest, and log data to configmap loftsman-sls in namespace loftsman command=ship

When running loftsman ship again the 1.1.0 version of loftsman fails to run due to exceeding the max allowed annotation size.
[~/Documents/Work/Repos/Github/loftsman]$ ./bin/loftsman-darwin-amd64 ship  --charts-repo https://charts.bitnami.com/bitnami --manifest-path manifest.big.yaml
2021-12-09T16:46:38-06:00 INF Initializing the connection to the Kubernetes cluster using KUBECONFIG (system default), and context (current-context) command=ship
2021-12-09T16:46:38-06:00 INF Initializing helm client object command=ship
         |\
         | \
         |  \
         |___\      Shipping your Helm workloads with Loftsman
       \--||___/
  ~~~~~~\_____/~~~~~~~

2021-12-09T16:46:38-06:00 INF Ensuring that the loftsman namespace exists command=ship
2021-12-09T16:46:38-06:00 INF Loftsman will use the charts repo at https://charts.bitnami.com/bitnami as the Helm install source command=ship
2021-12-09T16:46:38-06:00 INF Running a release for the provided manifest at manifest.big.yaml command=ship
2021-12-09T16:46:38-06:00 ERR  error="Error creating ship configmap loftsman-sls in namespace loftsman: ConfigMap \"loftsman-sls\" is invalid: metadata.annotations: Too long: must have at most 262144 bytes" command=ship

The version of loftsman from this PR is able to successfully deploy this large manifest:
[~/Documents/Work/Repos/Github/loftsman]$ go run . ship --charts-repo https://charts.bitnami.com/bitnami --manifest-path manifest.big.yaml | head                        *[CASMINST-3664-v2]
2021-12-09T16:50:18-06:00 INF Initializing the connection to the Kubernetes cluster using KUBECONFIG (system default), and context (current-context) command=ship
2021-12-09T16:50:18-06:00 INF Initializing helm client object command=ship
         |\
         | \
         |  \
         |___\      Shipping your Helm workloads with Loftsman
       \--||___/
  ~~~~~~\_____/~~~~~~~

2021-12-09T16:50:18-06:00 INF Ensuring that the loftsman namespace exists command=ship

<truncated>

 chart=nginx command=ship namespace=default version=9.5.16
2021-12-09T16:49:57-06:00 INF Ship status: success. Recording status, manifest to configmap loftsman-sls in namespace loftsman command=ship
2021-12-09T16:49:57-06:00 INF Recording log data to configmap loftsman-sls-ship-log in namespace loftsman command=ship

I verified the contents of the 2 configmaps that loftsman created contained the expected data:
[~/Documents/Work/Repos/Github/loftsman]$ kubectl -n loftsman get cm                                                                                *[CASMINST-3664-v2]
NAME                    DATA   AGE
loftsman-sls            2      6m9s
loftsman-sls-ship-log   1      2m12s

Wasp testing
On wasp I used successfully deployed the cray-hms-rts helm chart with the version of loftsman from this PR.
I also tested running loftsman 1.1.0 and then loftsman 1.2.0 (from this PR) and verified that the loftsman-rts configmap only contained the manifest and the ship status, and that the loftsman-rts-ship-log contained the loftsman log.
To simulate a downgrade of loftsman I tested running version 1.2.0 and then 1.1.0. I verified that RTS was deployed successful, and loftsman didn't error out due to the changes in the ship result config map loftsman-rts.

Declined in favor of: https://github.com/Cray-HPE/loftsman/pull/43


AppBarButton and Button behave differently when interacting with them.

Current behavior
There are some differences in the behavior of both controls. For instance, when hovering the AppBarButton does not react as it is the case with the regular Button. There is also no difference in the appearance of the AppBarButton when it is enabled or disabled. This makes it difficult for the user to know if the associated command is available or not. There are other visual states, like 'Pressed' and 'Focused' where no effects can be applied.
Expected behavior
The AppBarButton should react to user interaction and changes on the enabled state.
How to reproduce it (as minimally and precisely as possible)
ButtonAndAppBarButtonBehaveDifferently.zip

Open the attached solution and compile it.
Execute it in any head given.
Hover the AppBarButton and the Button to see the difference.
Check and uncheck the CheckBox to see the different behaviors of both controls.

Environment
Nuget Package:
NuGet package(s): Uno.WinUI
NuGet package version(s): Uno.WinUI 5.1.80
Package Version(s):
Affected platform(s):

[X] iOS
[X] Android
[X] WebAssembly
[X] UWP
[X] MacOS

Anything else we need to know?

The Material AppBarButton styles were mostly there for the native NavigationBar commands but I agree that if we are going to set them as the implicit style for AppBarButton then they should be a complete style. Right now the template is extreme barebones.
Would be good to bring the visual states and structure from the original Fluent style and use the Material resources.
Thanks for the report @Suriman!


subkey inspect error

I try to inspect bip44 path, but failed.
➜  subkey inspect "quick random crucial empty hurry diagram story grid afford eternal few finish idle rhythm romance explain snack ripple work soon execute glare know embrace//44//354//0/0/0" -n polkadot --scheme Ecdsa
Invalid phrase/URI given

It seems polkadot seed is not the same as bitcoin. Is there detail explaination about address generating?

AFAIK we use bip39.

I can confirm this is from the use of the --scheme Edcsa , without the flag a key is generated as expected.

@NukeManDan why is that a bug?

Unless I am missing why the derivation should not be available in using ECDSA.

I guess whatever tag is correct, please update as you see fit - this was my best guess. 😅


Handle grantPermission for API versions <33

ok to me

@marutifh Have you had time to review this pull request?

Looks okay to, merging changes.


issue after i run rc_driver.py

I have a broken pipe error every time i run Rc_driver.py , i didn't have this problem when i run training and testing codes , all tho the part of code responsible of connection  is the same

Can you post the error on your computer as well as on raspberry pi?
Have you tried to run rc_driver_nn_only.py? Does it give you error as well?

The rc_driver_nn_only.py work just fine after i change the host to " self.server_socket.bind(("<IP_ADDRESS>", port))"
in the other hand the rc_driver.py don't work as well , i got this error
Traceback (most recent call last):
File "rc_driver.py", line 188, in
ts.start()
File "rc_driver.py", line 181, in start
self.video_stream(self.host, self.port1)
File "rc_driver.py", line 170, in video_stream
s = socketserver.TCPServer((host, port), VideoStreamHandler)
File "/home/siraj/miniconda3/envs/auto-rccar/lib/python3.7/socketserver.py", line 452, in init
self.server_bind()
File "/home/siraj/miniconda3/envs/auto-rccar/lib/python3.7/socketserver.py", line 466, in server_bind
Exception in thread Thread-1:
Traceback (most recent call last):
File "/home/siraj/miniconda3/envs/auto-rccar/lib/python3.7/threading.py", line 917, in _bootstrap_inner
self.run()
File "/home/siraj/miniconda3/envs/auto-rccar/lib/python3.7/threading.py", line 865, in run
self._target(*self._args, **self._kwargs)
File "rc_driver.py", line 174, in sensor_stream
s = socketserver.TCPServer((host, port), SensorDataHandler)
File "/home/siraj/miniconda3/envs/auto-rccar/lib/python3.7/socketserver.py", line 452, in init
self.server_bind()
File "/home/siraj/miniconda3/envs/auto-rccar/lib/python3.7/socketserver.py", line 466, in server_bind
self.socket.bind(self.server_address)
OSError: [Errno 99] Cannot assign requested address
self.socket.bind(self.server_address)

OSError: [Errno 99] Cannot assign requested address

@Sirajabudabe Address is taken by your previous run, make sure to exit the program, or you can close the terminal/pycharm, then the address will be available.

how did u solved this problem？


updated mysql driver.

There is an older version of the github.com/go-sql-driver/mysql driver which sets allowNativePasswords=false by default. This breaks with hosted mysql service such as Azure MySQL, which requires native passwords to be allowed. Updating the driver to a version which contains this change will set the native passwords to be allowed by default.
Signed-off-by: Mike Lloyd<EMAIL_ADDRESS>

Is there a plan to get this done and eventually update the go-mysql driver for notifications app?
We have received several reports regarding failure to connect to Azure MySQL server and hence notifications app crashed upon start-up.
Error messages:
2019-07-30T10:57:20.169-04:00 [APP/PROC/WEB/0] [ERR] /tmp/gobuildpack.gopath226365599/.go/src/github.com/cloudfoundry-incubator/notifications/main.go:15 +0x136 2019-07-30T10:57:20.169-04:00 [APP/PROC/WEB/0] [ERR] main.main() 2019-07-30T10:57:20.169-04:00 [APP/PROC/WEB/0] [ERR] /tmp/gobuildpack.gopath226365599/.go/src/github.com/cloudfoundry-incubator/notifications/application/mother.go:40 +0x23c 2019-07-30T10:57:20.169-04:00 [APP/PROC/WEB/0] [ERR] github.com/cloudfoundry-incubator/notifications/application.NewDBProvider(0xc420016068, 0x21, 0x55671b46e2aa, 0x1, 0x0, 0x3, 0xc420084300, 0x7f, 0xc420022013, 0xa7, ...) 2019-07-30T10:57:20.169-04:00 [APP/PROC/WEB/0] [ERR] goroutine 1 [running]: 2019-07-30T10:57:20.169-04:00 [APP/PROC/WEB/0] [ERR] panic: This server only supports the insecure old password authentication. If you still want to use it, please add 'allowOldPasswords=1' to your DSN. See also https://github.com/go-sql-driver/mysql/wiki/old_passwords

@scottgai this work is being handled via https://github.com/cloudfoundry-incubator/notifications/pull/31


Find out standard used for piping calculations

this will influence documentation of calc_form and calc_Y

The standard is EN 15316 @daren-thomas

did you update the documentation?
On Monday, 4 April 2016, JIMENOFONSECA<EMAIL_ADDRESS>wrote:

Closed #50
https://github.com/architecture-building-systems/CEAforArcGIS/issues/50.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/architecture-building-systems/CEAforArcGIS/issues/50#event-613097370


Add basic stats with breakdown of channels counts

What changed in this PR and why
Added a tab in the admin page for stats. This page currently simply has the count of contributors by channel.
It does not filter out for active contributors or anything else currently.
Whats not included in this PR and why
Currently, it seems pretty limited what we can easily achieve with administrate gem. Our css styles do not get loaded or applied and we cannot connect to our stimulus controllers without overriding administrate's js files, it seems. Have not tried. It might be nice to have a pie chart that displays this info.
Maybe, we could display info on our dashboard page? Is it only interesting for admin? If we wanted to scope it to admin only, we could easily add a flag to display it only to admin.
Closes #1861

lgtm - but I don't get what app/dashboards/stat_dashboard.rb is for

lgtm - but I don't get what app/dashboards/stat_dashboard.rb is for


Using lazy loading resulting in W3C html validation errors

Using Swiper with lazy-loading enabled results in errors when trying to validate html on https://validator.w3.org/ even though you provide fallback images on src attribute.

Error: Bad value  for attribute src on element img: Must be non-empty.
From line 104, column 37; to line 104, column 267
<img itemprop="contentUrl" src="" data-src="/images/..." class="swiper-lazy"

HTML validation doesn't have much sense in HTML5, if you really care about (for some reason) you can't use lazy loading, or pass some placeholder 1x1 px image

If you say so.
But like I said I'm getting this validation error even when using a placeholder image.
HTML:

<img itemprop="contentUrl" src="placholder.gif" data-src="/images/real-image.jpg." class="swiper-lazy"

results in

Error: Bad value for attribute src on element img: Must be non-empty.


Roll engine 0602dbb27547..8eaddd606e49 (11 commits)

https://github.com/flutter/engine/compare/0602dbb27547...8eaddd606e49
git log 0602dbb27547cd03d8f8e816b7995cace854f7e6..8eaddd606e499ac2601e254b285046575b7d5d17 --no-merges --oneline
8eaddd606 Add web integration test to build_and_test_host (flutter/engine#9218)
a7fb7da34 Roll src/third_party/dart 9f2f5adb64..6d608fb52b (5 commits)
041c79114 Roll src/third_party/skia fe18de506097..14c8ca93db18 (11 commits) (flutter/engine#9231)
086b5a48d move webOnlyScheduleFrameCallback off of window (flutter/engine#9222)
8a6bad6ef Roll src/third_party/dart 40ef0c6d9f..9f2f5adb64 (4 commits)
91ee78068 Roll src/third_party/skia 6faf8d662af8..fe18de506097 (1 commits) (flutter/engine#9228)
828191990 Roll src/third_party/dart 6e0d978505..40ef0c6d9f (7 commits)
9dafb40a2 Roll src/third_party/skia 0e8362655a66..6faf8d662af8 (1 commits) (flutter/engine#9225)
e3b8b6120 Roll src/third_party/skia 3431d9d1dcc6..0e8362655a66 (2 commits) (flutter/engine#9223)
e80df363d Keyboard support for embedded Android views. (flutter/engine#9203)
2ec6dada6 Roll src/third_party/skia 6e4fee8c812e..3431d9d1dcc6 (17 commits) (flutter/engine#9221)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (bmparr@google.com), and stop
the roller if necessary.

PullRequest is not longer mergeable. Closing it.

PullRequest is not longer mergeable. Closing it.

Commit queue failed; closing this roll.

PullRequest is not longer mergeable. Closing it.


fix munmap bug in the windows

fix munmap bug in the windows

Thank you for the PR!  Looks good other than some minor nits I would like to see changed before merging.

Awesome! +1 Thanks!

+1 @runner-mei can't believe I did those mistakes :-)


Loading custom font file not working

I am getting either of the two problems below, even after specifying the correct public path.
Call to undefined function Intervention\Image\Gd\imagettfbbox()

or
Intervention \ Image \ Exception \ NotSupportedException
Internal GD font () not available. Use only 1-5.`

I also viewed this issue but could not get any help : https://github.com/Intervention/image/issues/149 Can anyone find out the cause of such errors?
My code is :
$img->text($text, 275, 200, function($font) {
            $font->file(public_path('assets/fonts/Alpaca.ttf'));
            $font->size(60);
            $font->color('#fdf6e3');
            $font->align('center');
            $font->valign('middle');
            $font->angle(45);
        });

Call to undefined function Intervention\Image\Gd\imagettfbbox()

I would say for some reason the GD function imagettfbbox is missing on your system. Maybe GD is not installed completely or not installed at all?

Thank you for your reply.
The problem though was that my server had GD support enabled but Freetype was not installed.
Its working like a charm now.

I installed Freetype, but how do I activate it for PHP?

If you are on a Mac (like I was), MAMP will do the heavy lifting for you. That was what I did at this particular case : after installing Freetype, use MAMP as the local development server.

I'm using Homestead and Vagrant.

I'm using Homestead and Vagrant.

It has to be installed inside the vagrant box. You can SSH inside your machine via 'vagrant ssh'.

I've solved this in a docker image with
apt-get install -y libpng-dev libfreetype6-dev libjpeg62-turbo-dev && \
docker-php-ext-configure gd --with-freetype-dir=/usr/include/ --with-jpeg-dir=/usr/include/ && \
docker-php-ext-install -j$(nproc) gd


Combine Invite and Elector Table

Right now, we need to deal with corner cases where an invite without an associated elector.  This is silly, because an invite by itself does not even refer to a specific election.
We should add the fields of invite to elector, and then delete elector.


Add public API introspection of the element schema for z.ZodSet

Currently, z.ZodSet lacks any public API to access the schema of the set elements. I'm not sure if any is an oversight, but it would be a great addition.
Currently z.ZodArray, z.ZodRecord, and z.ZodMap all support such features. Doesn't seem to be too great an ask for a similar feature for z.ZodSet.

You can access zodSet._def.valueType?


Normalize email address on sign[up/in] workflows

When this PR be merged, when a user tries to register with one of this email addresses<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>she will be registered with<EMAIL_ADDRESS>(now the app registers the three addresses as three different users). The app will lowercase the email address and strip Gmail alias tricky characters.
The forgot module will have the same behavior.
After this, users previously registered with aliases or caps will become unaccessible.
Closes #689 and #736.

What are we going to do with the apps that already have users with multiple accounts but same gmail email?
I think it's really weird to discriminate gmail. Is there really a security improvement? Is not sufficient with lowercase: true at the model?

Let's make it configurable. By default it should be disabled but we should allow enforcing of this so official deployments don't have to cope with endless accounts per email.

Also, we may add a 'visible' email field and a 'normalized' email field. Checks would be against the normalized field. This would be required just because mongo can't index virtual fields.

Also, make sure that STAFF config emails are beign normalized too.

We decided to do it with a config

What's the status here? We need to fix #689

Will be merged here #905


Add ARM tests on Travis

I have updated Travis' YAML file to add emulated tests for ARMV6 and ARMV8 architectures using the Alpine Docker images.
The idea has come from @brada4 (see comment in #1861). Because the emulated builds take a lot of time, I have allowed the jobs to fail.
This is just an initial attempt to provide such a support in Travis. Please feel free to comment, improve/modify or even close the PR as you wish.

Do you see the same arm_neon.h failure in local builds of ARMV8 ? I wonder if this failure is real, as there was some recent controversy over how much of the thunderx2 optimizations are generic enough to allow using them for the ARMV8 target.

Maybe it is possible to sabotage slow tests by not installing gfortran and make the builds finish in time? It is like 10% of time still left to do at that point of failure if it was x86_64

Doubt it - he lready dropped LAPACK and CBLAS from the build if I read that correctly.

Do you see the same arm_neon.h failure in local builds of ARMV8 ? I wonder if this failure is real, as there was some recent controversy over how much of the thunderx2 optimizations are generic enough to allow using them for the ARMV8 target.

Uhm... I can try, when I have time. However, I will do an emulated build locally, as I do not have an ARM64 device at hand. I do use my RPi2s for ARM32v7 builds when needed for my project. As for the other thing, i.e., the controversy, I do not have much information (and I am not knowledgable, at all, when it comes to architectures). But what I can see there is that it might be a clang related thing (the build starts and proceeds with gcc). It might be worth trying with some debian:stretch-slim image --- I have had some problems with musl-c libraries and their transition packages and headers in alpine. Maybe I am missing some package there, too.

Maybe it is possible to sabotage slow tests by not installing gfortran and make the builds finish in time? It is like 10% of time still left to do at that point of failure if it was x86_64

I had thought about the same thing. I have created an organization openblas at Docker Hub, ownership of which I can happily hand over to any of the contributors here, and I will create and push the alpine images which have the dependencies preinstalled. Then, changing the Dockerfile to have as fewer layers as possible could make at least the builds finish.

Doubt it - he already dropped LAPACK and CBLAS from the build if I read that correctly.

Correct. That was intentional, as I had thought that the BLAS part could be more than enough to test if the builds proceed.

MUSL is tested in other alpine linux tests , does not hurt to test extra. Debian (and glibc) might be 1c bigger to stuff in 50 minutes we have... I think first thing that does not time out is already a revolution....

Well, base image for Alpine is ~5MB whereas Debian's slim images are around 17MB. But you are probably right in that when we bundle them together with glibc and friends, they will eventually add up and exceed the size of Alpine by far.
Anyways... It is a pity that the builds timeout at 99% :(

Maybe restricting the build to the static libopenblas would help (though this would drop the completeness check with gensymbol/linktest.c)

Either way worth keeping as an open PR for reminder and retrying when ARM code changes a lot..

BUILD_SHARED=NO does not get us past 98/99 percent either. Trying with make -j 3 now in my fork although I doubt it is i/o-bound.

According to pkgs.alpinelinux.org, arm_neon.h should be available in /usr/lib/clang/5.0.1/include as part of the clang-dev package on aarch64.

According to pkgs.alpinelinux.org, arm_neon.h should be available in /usr/lib/clang/5.0.1/include as part of the clang-dev package on aarch64.

I have updated the images --- now they have clang-dev. Please feel free to make PR's to aytekinar/openblas-alpine to modify the Docker image as needed. In fact, I have also invited you, @martin-frbg, as a collaborator to the repo.

Thank you. The clang-aarch64 build no longer errors now, though obviously it still times out like the others. I am still experimenting with running more than 2 concurrent make jobs in my fork - make -j 3 looked like it made the two gcc-based builds  run to completion but I am no longer sure if it was actually using 3 threads or if the setting was overruled by getarch and I was just lucky.

How about -j$(nproc)? Do you think overloading is better?

It will choose -j 2 automatically for the two cores provided by travis, still testing if -j3 or -j4 would really help.

The way emulator could work jit-ing chains of ARM code - doubling ncpus will keep compiler code more time in RAM and JIT-ed, and tests are highly undesirabe as fresh advanced code to be jitted again.

Seems 3 concurrent make jobs do not help. With the latest changes to the docker environment the ARM32 gcc job manages to finish in a bit over 47 minutes. ARM64 gcc sometimes barely makes it as well while the two clang-based jobs consistently time out at around 90 percent.

In a year it will fit in timeout...

Perhaps we could just live with the incomplete builds for now, but it looks to me as if the allow_failures is not parsed as intended right now - I do not see it mentioned in the log, and the entire CI run gets flagged as failed which is probably not what we want.

I have tried adding matrix: in front of allow_failures as it seems to me, as per the documentation, that Travis might be checking for matrix.allow_failures, which we had not had before (your naming convention has jobs instead of matrix in the top part). Let's see if this time it is parsed properly. Then we can merge this PR?

The current setup uses something called "build stages", but as I understand allow_failures  should work there as well.

I think it depends from where the wind blows....
Probably dumping gfortran and dynamic library can make gcc version through most of the time I think it must be rested for the time travis gets speedier computers....

@aytekinar looks like the "env" is not matched for some reason, but matching on "name" works.

The separation seems to be a travis bug (or missing feature) for "build stages" (saw a bug open for it but did not keep the number or link). Haven't tried with "matrix" but looks like I got it to work with stages. Now to see if it is "-names" or "-services" that actually did the trick.

I have removed stage: test from the YAML file, which seems to have fixed the problem. Finally, it is working. I am not sure if you definitely need stage: test tag, as the YAML file seems to contain only one stage, anyways. Then, we might try appending
stages:
  - test

to exactly specify what stages there are.

The alternative would appear to be to keep using "jobs" where you switched to "matrix" at some point

The alternative would appear to be to keep using "jobs" where you switched to "matrix" at some point

... while keeping stage: test everywhere?

The alternative would appear to be to keep using "jobs" where you switched to "matrix" at some point

... while keeping stage: test everywhere?

Yes, at least that works for me - I copied your original PR to my fork for experimenting, and what I now have is just your original additions with "-name" in place of "-env" in the allowed_failures. (job still running so not sure if it was name or services keyword that did the trick - though we might as well use both)

Weird. I could not get them working without removing stage: test from everywhere. Apparently, jobs and matrix are simply aliases, so it should not change much. However, I could only get the separation between build jobs and allowed failures by removing the stage tag.
It's your call. You can simply add your version here from your branch.

Misunderstanding actually - "my" version does not produce the nice separation, it just adds a comment naming the allowed failures. As the "stage: test" does not appear to serve any purpose (now at least),
removing it to get a cleaner look of the results panel is alright I think.


Container stderr appears on rkt run stdout

When using containerized applications in a pipeline, it is important not to merge stderr with stdout I think.  However, that is what currently happens.
This is somewhat related to #3639, where I reported that the new --stdout, --stderr options don't actually work (!)

What did you expect to see?
Separate stdout and stderr from rkt run
What did you see instead?
Output was all merged on stdout

You forgot to report how you are running it. By default, both stdout and stderr go to journald. If you are running with --interactive there is a TTY involved, which has a different semantics for input and output (and no concept of stdout/stderr).

I wasn't using --interactive.  Here's my command line.
# /usr/bin/rkt --insecure-options=image run --net=host --volume home,kind=host,source=/home/guestsi docker://ruby --mount volume=home,target=/home/guestsi --exec /home/guestsi/junk/hello-error-world

Perhaps my solution will be to use rkt attach with those custom stdin/stdout/stderr modes, as per #3639

Please note that those are still experimental. If you are only interested in logs, I suggest to grab the entries from journald directly. Metadata should tell you where an entries is coming from (perhaps not stdout/stderr, that may require a small patch in journals itself).

Now resolved, as per #3639


createDeployment task parameter should be optional

In createDeployment task parameter is required though a default value is specified.
But its optional in the github api.
It could be optional with "deploy" as the default value.

Thanks for pointing this out. Should be fixed in the next release.


New Zealand urban route shields

New Zealand’s Traffic Control Devices Manual provides for urban routes marked by a black-on-white pentagon.

So far, no urban routes have been mapped as route relations.
Originally posted by @1ec5 in https://github.com/ZeLonewolf/openstreetmap-americana/issues/379#issuecomment-1156915732

So far, no urban routes have been mapped as route relations

Actually, 29 urban routes in Auckland, Hamilton, and New Plymouth have been mapped as relations, but without a network tag. In order for us to render these shields, we need them to be tagged with something like network=NZ:urban or network=NZ:UR.

Hi, New Zealander here 👋 As @1ec5 noted, all urban routes are mapped, just without a network tag. (there's also one in Christchurch)

we need them to be tagged with something like network=NZ:urban or network=NZ:UR

I'm happy to make this change if no one else does. Does anyone here have a suggestion for the best tag value, to be consistent with other countries?

Side note: There are a few routes that use the white urban-route shield, but are not in "urban" areas. (e.g. Route 70, Route 72, Route 40, Route 67). This is beacuse they were downgraded from (red) state highway shields.
I'm not sure whether these weird routes should also get tagged as network=NZ:Ur[ban], or whther we need another network tag with identical rendering.

I'm happy to make this change if no one else does. Does anyone here have a suggestion for the best tag value, to be consistent with other countries?

Thank you! There are two predominant styles:

network=NZ:urban would be consistent with countries that have relatively simple route designations that are homogenous across the country. The part at the end is a keyword that describes the kind of route according to typical OSM naming conventions.
network=NZ:UR would be consistent with countries that have more complex route designations needing disambiguation, for example provinces with their own unique shield designs, or bannered routes.

It isn’t a big deal either way as long as it’s consistent. With the existing tags like network=NZ:SH, it looks like New Zealand has been taking the latter approach, so network=NZ:UR would be more consistent, if not as self-documenting.

Side note: There are a few routes that use the white urban-route shield, but are not in "urban" areas. (e.g. Route 70, Route 72, Route 40, Route 67). This is beacuse they were downgraded from (red) state highway shields.
I'm not sure whether these weird routes should also get tagged as network=NZ:Ur[ban], or whther we need another network tag with identical rendering.

Good to know. If I’m reading the TCD manual correctly, these are urban route indicators for urban routes, which are designated by road controlling authorities. It doesn’t look like it particularly matters whether the route traverses an urban area.

All 42 urban routes now have network=NZ:UR :)

Fixed in #473.


Using StaticResource causes an invalid exception

Hi
This is a brilliant nuget!!!
I am sure its working as designed but using on existing app when lots of values that are defined  int the App.Xaml.
if you have something like this in your App.Xaml
  <OnPlatform
          x:Key="LargeSize"
          x:TypeArguments="x:Double"
          Android="18"
          iOS="20" />

and then  you try to do this
FontSize="{markups:OnScreenSize
                        Small='{StaticResource SmallSize}',
                        Medium='{StaticResource DefaultSize}',
                        Large='{StaticResource LargeSize}'}"

Than it will crash with a cast exception.
Is there a way that  it could possibly handle a value defined in the app.xaml as a static resource?
Many thanks

found a workaround  - use your onscreesize inside app.xaml

Hi,glad you find a workaround, but I possibly investigate this issue further later anyway to come up with an actual fix.
Thanks for posting.


Tone.Players not working as expected

Repro:
Example working in @13.x: https://jsfiddle.net/1qtb0x3z/4/
Example not working in @latest: https://jsfiddle.net/1qtb0x3z/5/
Also, I think Typescript type for Players.get() is incorrect, I think it's coming from ToneWithContext class where the get() method receives no arguments - however the Players.get() requires passing the name of the player in.
Not sure if this warrants a separate issue or if it's related?
Hope this is clear enough

Thanks for the issue. This is caused by a breaking API change between 13 and 14. Players.get('name') is now Players.player('name'). This change makes Players more consistent with the rest of the classes which return the object's properties when you call get.

would be cool to get this updated: https://tonejs.github.io/docs/r13/Players


optimize(Packages): use mmap to read large files

Using less syscalls and processing data in memory makes getNumStringsImpl ~400% faster.
Some benchmark highlights:
On my Ubuntu 22.04 laptop, with 1408 dpkg packages.
Run on (8 X 2400.01 MHz CPU s)
CPU Caches:
  L1 Data 32 KiB (x4)
  L1 Instruction 32 KiB (x4)
  L2 Unified 256 KiB (x4)
  L3 Unified 8192 KiB (x1)
Load Average: 0.12, 0.06, 0.01
---------------------------------------------------------------------
Benchmark                           Time             CPU   Iterations
---------------------------------------------------------------------
BM_getNumStringsImpl           256691 ns       256694 ns         2517
BM_getNumStringsLegacyImpl    1080674 ns      1080676 ns          636

Can't we use FF_AUTO_CLOSE_FD?

Also, all these early returns should fall back to the old function

Also, all these early returns should fall back to the old function

I don't think fdopen can work if fstat already failed.
mmap is a bit complicated, and there are a few errors where it can fall back. This kind of situation is quite rare. The only thing that is more likely to happen is mmap failure due to no enough memory. Is it really worth falling back here?

I am actually not familiar with mmap.
Did you test ffReadFileBuffer? How does ffReadFileBuffer compare to mmap?

I am actually not familiar with mmap.
Did you test ffReadFileBuffer? How does ffReadFileBuffer compare to mmap?

I'll add a benchmark with ffReadFileBuffer.

Also, you should use benchmark fastfetch itself, not pull things out. I suggest hyperfine. If the benchmark result differs not much, I prefer use the simpler implementation.

I am actually not familiar with mmap.
Did you test ffReadFileBuffer? How does ffReadFileBuffer compare to mmap?

BM_getNumStringsImpl                    1400988 ns      1400979 ns          476
BM_getNumStringsLegacyImpl              6468152 ns      6468036 ns          108
BM_getNumStringsWithffReadFileBuffer    1839101 ns      1839052 ns          376
Seems like 30% slower than mmap.
the code:
static uint32_t useFFReadBuffer(const char* filename, const char* needle)
{
    FF_STRBUF_AUTO_DESTROY content = ffStrbufCreate();
    if (!ffReadFileBuffer(filename, &content))
        return 0;
    uint32_t count = 0;
    size_t needleLength = strlen(needle);
    char *iter = content.chars;
    while ((iter = strstr(iter, needle)) != NULL)
    {
	++count;
        iter += needleLength;
    }
    return count;
}

What about ReadFileBuffer?

What about ReadFileBuffer?

single module:

entire command:

Not as much fast as mmap, but is still faster than stdio.

You can use hyperfine './ff-stdio -s package' './ff-buffer -s package' './ff-mmap -s package'
I prefer ff-buffer since its implementation is the simplest.

You can use hyperfine './ff-stdio -s package' './ff-buffer -s package' './ff-mmap -s package'
I prefer ff-buffer since its implementation is the simplest.

Faster than mmap may be an inaccuracy.
By the way, my benchmarking environment only has 1410 packages, whether other distros could have 2000+ packages. Theoretically the more packages you have the more significant the performance difference.
If u prefer ffReadBuffer, I'll change to it later, and getNumStringsLegacyImpl can be deleted.

If u prefer ffReadBuffer, I'll change to it later, and getNumStringsLegacyImpl can be deleted

Please do it.
And you may use memmem instead of strstr, since lengths of both string are known.

Please do it.

Done. Please take a look.


ScheduledWorkflow only processing the first LaunchScheduleItems collection item?

Question
I'm. trying to define more than one LaunchScheduleItems but current code simply ignores all items which are not the first
Additional context

Current code

it looks like a logic problem however at the moment I am overloaded so it will take some time until I reach this case.  However PR are acceptable and totally possible from your side judging from the question.

@apobekiaris thanks for your fast response
BTW, I'm also a little overloaded for a PT and also I've my own classes (which I've copy pasted some time ago); what I've just written after you've confirmed, is this (hope this helps)
 protected override bool NeedToStartWorkflow(IObjectSpace objectSpace, ScheduledWorkflow workflow)
        {
            var needStart = false;
            foreach (var schedule in workflow.LaunchScheduleItems)
            {
                switch (schedule.StartMode)
                {
                    case StartMode.OneTime:
                        if (NeedToStartOneTime(workflow, schedule))
                            needStart = true;
                        break;
                    case StartMode.Min:
                        if (NeedToStartMin(workflow, schedule))
                            needStart = true;
                        break;
                    case StartMode.Daily:
                        if (NeedToStartDaily(workflow, schedule))
                            needStart = true;
                        break;
                    case StartMode.Weekly:
                        if (WeeklyDayMatch(schedule))
                        {
                            if (workflow.LaunchScheduleItems.Count > 0 && NeedToStartWeekly(workflow, schedule))
                                needStart = true;
                        }
                        break;
                }
            }
            return needStart;
        }

thnks for the patch I will update the source when I review this case.

Closing issue for age. Feel free to reopen it at any time.
.Thank you for your contribution.


Embeddable Unique Visitors Counter

Like this image except nicer-looking
And also a little 88x31 size one
Or like this

Yes, that's correct! The biggest issue with having a normal visits counter is when I reload my webpage after updating it, it would count as a hit and update the counter. That's not a very accurate representation of how many unique people have visited the web page. An 88 by 31 button is a small button, ex. these I have designed. ![caddy](https://github.com/user attachments/assets/7d5d9367-aa04-4348-9883-e3745b644991) They would be easy to integrate into my webpage since I also have a series of other 88 by 31 buttons, however they do not have to be 88 by 31 pixels, they just have to have the width factor of 88 by 31. I use a DPI monitor and fractional scaling meaning that if the image is 88 by 31 pixels it's actually larger than that, or so I believe. Sorry if I'm being hard to understand.

But also, larger and more verbose widgets would be useful for websites that don't look like they came from the early 2000s.

Let's move this chat in the discussions tab on github!


[Idea] Fast dial option to select lines in ui

It would be great to have a fast dial option to select lines in the ui, like -quick-match in unite.

Well, I don't like the feature.
You can use/create the move plugin.

Please use qselect.vim for it.
https://github.com/Shougo/qselect.vim


Please validate rp2040.svd against CMSIS-SVD.xsd

When passing rp2040.svd to a xml schema validator against CMSIS-SVD, it appears that there's many issue, like mandatory field missing, incorrect order of elements in a sequence etc.
Please validate your document.

there's many issue, like mandatory field missing, incorrect order of elements in a sequence etc.

Yup, a few open-issues regarding SVD already, see https://github.com/raspberrypi/pico-sdk/search?q=svd&state=open&type=issues
Unfortunately fixing up our internal tooling to generate a "better" SVD file isn't at the top of our priority-list at the moment. (We're all pretty busy!)

Please validate your document.

As mentioned here the SVD file did originally pass the SVDConv validation, but I wasn't aware at the time that there were additional things that SVDConv doesn't catch.

There is a validated SVD file for the RP2040  available for download
If there is still an issue with the file you can contact me directly<EMAIL_ADDRESS>File also attached:
RP2040.svd.zip (compressed as github does not accept *.svd as attachment)

Please note the SVD file is automatically generated from other files, as such the issue is not in the SVD file, but in the generator.
If you can instead provide a list of the 'class' of changes that are required, this may make it easier to achieve.
Thanks
Gordon

@ghollingworth If you read the first message in this issue(by @juju2013) you will see a link to an online validation tool, as well as a link to the SVD schema(*.xsd). You can just put the linked xsd and the RP2040 SVD in that web site and you will get all the issues reported . If you want to do it offline then there are several XML validation tools available fro free.  For me it was as easy as doing xmllint --noout --schema CMSIS-SVD.xsd RP2040.svd that then gives you the complete list of issues in the SVD file. With that you can start changing the "generator".
Do you have a link to the public repository of the "generator" ? If so I would love to take a look and help fix these issues. As the people as Raspberry Pi are so busy that should be a welcome help, right?
Lurch said they are pretty busy. There is no benefit for anybody if Raspberry Pi staff invests time into fixing there SVD generator if there is already a working open source SVD generator available. They should spend their time in making the products better.
Also fixing the issues is not needed anyway. They can just use the chipselect generated file. Or they can use the generator that chipselect uses to generate the file. That generator is open source. As everything chipselect.
If I get access to those "other files" you mention then I can update the SVD to the changes from those files. I could even give chipselect an interface / API for Raspberry Pi to push in the updated information to then pull out the updated SVD. If you don't trust me you can setup a private chipselect instance. I will help you set that up no problem.
I'm just offering my help here. If you don't like me you can take the open source chipselect stuff from my github repos and use that to fix your issues. you can also use the XML schema validator tool of your choice and look into the chipselect SVD to see how to fix the issues it will report on the official Raspberry Pi file.
Also please excuse me if I messed with your issue process, but I did not only post the chipselect file here to help Raspberry Pi. I also want to help the people that report the issue by suggesting them a solution. A working SVD might not help you or the proprietary generator, but maybe it helps juju2013 with her pysvd project or whatever else was her need for a complaint file.

Thanks, but the generator is not open source because it's connected very closely to the verilog IP.  I'll get someone to look at it in the next couple of weeks.

Although it passes the validator you linked, SVDConv reports a lot of errors and warnings when asked to produce a header.
As Gordon has explained, we will see if we can improve our generator and make an updated RP2040.svd
Clearly we must be sure that the header definitions produced from it are still correct.
Thanks.

@andygpz11 Initially I did not test it with SVDConv.
Validating against the schema only tests that the XML complies to the rules laid out in the schema. Those rules mostly specify allowed tag names and data ranges.
I now checked with the SVDConv tool. And yes it reports issues. Those are stuff like:
*** INFO M317: Raspberry Pi--RP2040.svd (Line 30)    Interrupt <description> not set.
My file is created from the RaspberryPi SVD file. So if there was no description in there, then I also have no description. So this is basically independent of the Convert tables to SVD format but more a information missing kind of issue.
*** INFO M347: Raspberry Pi--RP2040.svd (Line 45)    Field 'AINSEL' (width < 6Bit) without any <enumeratedValue> found.
This seems to go into the same direction. I compared it with the data sheet and the SVD has basically the same description for this field. But the SVDConv tool requires that all valid values need to be listed separately (enumeratedValue) This information is just missing.
If someone can point me to where I can get the missing information then I would fix all these issues.

Hi @JustAnother1, thank you for your offer of help but as Gordon explained we want to improve our generator.
We expect to just press a button so that documentation is produced from a canonical source.
If we "fix-up" autogenerated files we will have to do this each and every time we edit any of the register descriptions in RTL and want to generate/push a revised RP2040.svd
Sorry but this approach makes no sense; Gordon has already explained why we can't share the tooling.
We do hope to produce a revised RP2040.svd in due course.
May I ask if this is actually causing you a problem, other than you would like to be able to fix it yourself?
Thanks.

I agree that it makes most sense to add the missing documentation to the source files that are then used to generate the documentation. My suggestion was that chipselect could be part of what happens once you press that magic button.
The SVD file is intended to be used for debugging. Tools like SEGGER Ozone use the information from the SVD file to know which addresses belong to Special Function Registers and to decode the information read from such addresses.
If the SVD file is not valid (as in does not comply to the Schema) than all software that tries to use it as debugging help or other uses might run into problems. Like a mp3 player software that should read in a jpeg file.
Please keep in mind that I did not report this issue and that this issue is not the only one SVD related that has been reported.
I just saw an opportunity to help you. I can understand why you want to solve the issue yourself, but I don't know how I can help with that.

good news:
$ xmllint --noout --schema CMSIS-SVD.xsd rp2040-gh-1076.svd rp2040-gh-1076.svd validates

On the other hand ARMs SVDConv still has lot to say,..

can this be closed?

I think not quite.  I want to check the flow to ensure a "fresh" SDK contains an SVD that's as "good" as the one linked here.
I will do this in the next few days.

updated svd merged into develop


Fortran scanner misses comments in some cases

This is to document, in an issue, a limitation of the Fortran Scanner. The scanner itself does describe the limitation in comments, but not why it exists.
In searching for USE and INCLUDE statements when scanning Fortran source, regexes are used, which cannot deal with the combination of multiple semicolon-separated statements on one line which includes comment marks.  These lines from the unit test (SCons/Scanner/FortranTests.py) all give the wrong result:
!     USE modia ; use modib  # expect nothing, get modib
      USE mod14 !; USE modi  # expect mod14, get both
      USE mod15!;USE modi  # expect mod15, get both
      USE mod16  !  ;  USE  modi  # expect mod16, get both
; USE modi  # expect nothing (??), get modi

The regex considers the semicolon as the start of a new bit of text to scan, so in each case, what comes before has no effect.  The regexes are applied in multiline mode, FWIW. It doesn't matter where the comment mark is or how much whitespace there is, the comment appears to "end" at the semicolon.  The fifth example should (apparently) be considered a syntax error (and thus ignored?), but as scanning starts on the blank after the semicolon, there is no complaint.
Python's re module allows only fixed look-behind patterns, so there is no legal way to express "semicolon if not preceded by ! and possibly some other stuff", doing so will produce an error from the re module. There also doesn't seem to be a simple way to express "if the line begins with a comment, don't do anything more with it". It's not hard to write a bit of regex that says ignore from a character until the line ending, but interleaving that with the already fairly complex regex in use is something else.
From some digging around the internet, it seems that it's possible to encode this without a look-behind, at the cost of creating a considerably more complex regex pattern - that might be something to explore; we seem to be lacking that level of expertise.  The non-stdlib regex module can reportedly do look-behind with a variable-length pattern; a simple attempt to code it for there gave no error, but didn't help.
Probably we should find a place to document this and suggest that the easiest workaround, if it causes problems (it's not clear there's a real-world problem here, just a broken test when a change was made in the scanner module), is to "not do that" - so instead of:
      USE mod14 !; USE modi

do:
      USE mod14
!     USE modi

One suggestion from Discord was to pre-process the file to scan to remove comments, not sure how easy this is. Can comment marks appear in a line inside some other construct such that they are not considered comment indicators?

To follow up on the final paragraph of the issue, in order to strip comments, we'd need to understand the syntax involved.  The complication, as always, is context: if it's in a string (though Fortran does not use that terminology) it should not be considered a comment indicator.
From the standard:

<IP_ADDRESS> Free form commentary
The character “!” initiates a comment except where it appears within a character context. The comment extends to the end of the line. If the first nonblank character on a line is an “!”, the line is a comment line. Lines containing only blanks or containing no characters are also comment lines. Comments may appear anywhere in a program unit and may precede the first statement of a program unit or follow the last statement of a program unit. Comments have no effect on the interpretation of the program unit.

<IP_ADDRESS> Fixed form commentary
The character “!” initiates a comment except where it appears within a character context or in character position 6. The comment extends to the end of the line. If the first nonblank character on a line is an “!” in any character position other than character position 6, the line is a comment line. Lines beginning with a “C” or “*” in character position 1 and lines containing only blanks are also comment lines. Comments may appear anywhere in a program unit and may precede the first statement of the program unit or follow the last statement of a program unit.

Note that the existing regex does not attempt to recognize the fixed form "C or * in column 1".
and

3.21 character context
within a character literal constant (7.4.4) or within a character string edit descriptor (13.3.2)

And for the scanner issue, which incolves inline statement termination, here's the wording:

<IP_ADDRESS> Free form statement termination
If a statement is not continued, a comment or the end of the line terminates the statement.
A statement may alternatively be terminated by a “;” character that appears other than in a character context or in a comment. The “;” is not part of the statement. After a “;” terminator, another statement may appear on the same line, or begin on that line and be continued. A sequence consisting only of zero or more blanks and one or more “;” terminators, in any order, is equivalent to a single “;” terminator.

<IP_ADDRESS> Fixed form statement termination
If a statement is not continued, a comment or the end of the line terminates the statement.
A statement may alternatively be terminated by a “;” character that appears other than in a character context, in a comment, or in character position 6. The “;” is not part of the statement. After a “;” terminator, another statement may begin on the same line, or begin on that line and be continued. A “;” shall not appear as the first nonblank character on an initial line. A sequence consisting only of zero or more blanks and one or more “;” terminators, in any order, is equivalent to a single “;” terminator.


Cannot read properties of undefined (reading 'originalError')

Issue workflow progress

Progress of the issue based on the
Contributor Workflow

[ ] 1. The issue provides a reproduction available on
Github,
Stackblitz
or
CodeSandbox

Make sure to fork this template and run yarn generate in the terminal.
Please make sure Mesh package versions under package.json matches yours.

[ ] 2. A failing test has been provided
[ ] 3. A local solution has been provided
[ ] 4. A pull request is pending review

Describe the bug
A real error being masked by an exception in graphql-yoga's result-processor if more than one source used.
Seems like the problem is caused by graphql-jit's own version of GraphQLError which could have (and having) undefined in extensions, while in a lot of places it is treated as non-nullable, as in graphql-js.
For some reason, this problem does not occur when only one source present.
#6059 looks like another instance of this bug, though the one I'm reporting reproduces with the latest mesh version, after the fix.
To Reproduce Steps to reproduce the behavior:
I've uploaded a gist inspired by @TomKlaverAH reproduction 🙏 :
https://gist.github.com/Flamefork/2f00268dfb31a2b27dca5c5fc1e1692d
You can run it in one go with docker using:
docker run -it --rm --entrypoint=/bin/bash node:18 -c 'curl -sL https://gist.github.com/Flamefork/2f00268dfb31a2b27dca5c5fc1e1692d/raw | bash -s'

Expected behavior
The scenario is intentionally erroneous, the problem is that the expected outcome of this erroneous request should be a proper error
HTTP/1.1 200 OK

{"errors": [
  {"message": "Variable \"$input\" got invalid value { bogus: \"Res\" }; Field value.text of required type String! was not provided."},
  {"message": "Variable \"$input\" got invalid value { bogus: \"Res\" }; Field \"bogus\" is not defined by type MutInput."}
]}

but instead mesh responds with a generic exception in a response processing code
HTTP/1.1 500 Internal Server Error

{"errors":[
  {"message":"Cannot read properties of undefined (reading 'originalError')"}
]}

(formatted responses a bit for readability)

Also related: https://github.com/dotansimha/graphql-yoga/issues/2967

I'm getting this with a since swagger source behind mesh. Quite annoying as I end up getting a 500 from mesh, but the backend source worked fine.

I've noticed other people with the same issue here: https://github.com/zalando-incubator/graphql-jit/issues/234
@ardatan is there a way to just run without graphql-jit ?

@ardatan I'm also running into issues where I'm receiving an undefined error for originalError. I reproduced a codesandbox here: codesandbox
After running the command: yarn start, I opened postman and ran this somewhat invalid query:

Here's the request body that I ran:
{
    "operationName": "MyMutation",
    "query": "mutation MyMutation {update_test(input: {name: \" \"})}",
    "extensions": {}
}

Hopefully this helps resolve the issue.

Unfortunately I am not able to reproduce it even if I installed the dependencies and updated the start script in package.json;
https://codesandbox.io/p/devbox/pedantic-shannon-h9tpgk?file=%2Fpackage.json%3A7%2C23

Fixed in the upstream package;
https://github.com/dotansimha/graphql-yoga/pull/3157


Compaction commands for pulsar-admin

Adds an compact and compaction-status command in the persistent
subcommand of pulsar-admin.
This command differs from the compact-topic CLI (part of pulsar
command), in that it triggers compaction to occur via the REST api,
which causes the compaction to run in the broker process. The
compact-topic CLI spawns a new process to run the compaction.
Patch also fixes batch detection, which was broken (uncovered by a
change in the defaults), and recycles some metadata that wasn't being
recycled previously.

retest this please // BrokerServiceThrottlingTest.testLookupThrottlingForClientByBrokerInternalRetry


Import external libraries, local scope

Please provide us with the following information:

OS?

Linux (ubuntu).

Versions.

Please run ng --version. If there's nothing outputted, please run in a Terminal: node --version and paste the result here: angular-cli: 1.0.0-beta.17
node: 6.3.1
os: linux x64

Mention any other details that might be useful.
I imported external libraries for angular-cli.json as follows:

"../node_modules/jquery/dist/jquery.min.js",
"../node_modules/bootstrap/dist/js/bootstrap.min.js"

etc...
And it works very well, but these are library global scope.
How to include libraries that only used in a component (local scope)?
for exmaple "/plugins/datatables/jquery.dataTables.min.js" the use in a single component
Thanks

Thanks! We'll be in touch soon.

Hi @juanfeorar I do not think that is possible. You can create separate module and export providers. but still it will be in window scope.

Since jquery plugin need access to jquery, you should add them in the scripts as well. Then they will be in the global scope.


Location gloss no longer needed with external configuration

We are moving the responsibility for this functionality to Spine-O-matic through a configuration file


Added redirect uris ENV setting to web clients

Definition of Ready

[x] I am happy with the code
[x] Short description of the feature/issue is added in the pr description
[ ] PR is linked to the corresponding user story
[ ] Acceptance criteria are met
[ ] All open todos and follow ups are defined in a new ticket and justified
[ ] Deviations from the acceptance criteria and design are agreed with the PO and documented.
[x] No debug or dead code
[x] My code has no repetitions
[ ] Critical parts are tested automatically
[ ] Where possible E2E tests are implemented
[x] Documentation/examples are up-to-date
[x] All non-functional requirements are met
[x] Functionality of the acceptance criteria is checked manually on the dev system.

Currently web clients are only working with predefined redirect uris, changed it in a way that web clients are taking those uris from ENV settings.

Can you please use a semantic release  PR title as per contributing guidelines: https://github.com/zitadel/oidc/blob/main/CONTRIBUTING.md#make-a-pr


Add test from #202

Adds the test from #202, now passes on Python 3.5-3.8.
Closes #202.

The reported issue talks about handling of NA() values. How is this test related to that?

Good question! In any case, https://github.com/jazzband/tablib/issues/202#issuecomment-181284179 does say that test passed on 2.7 and failed on 3.4.

Replaced by https://github.com/jazzband/tablib/pull/415.


subctl gather: Create sub-directory for each cluster

What would you like to be added:
For subctl gather, create a sub-directory for each cluster instead of embedding the cluster name  in the file name.
Why is this needed:
The consensus is this will make it easier to review and search the files when troubleshooting.

Do we want to backport this (at least to 0.13)?


As a Data owner/Data manager I want to be attributed for my data ..so that I remain connected to the data if there are questions or kudos to gain

original id#5

Needs clarification and discussion.

counterpart to data user story


full-bundle-tests.yaml fail on self-hosted runners

Bug Description
At the moment, the behavior of full-bundle-tests.yaml on self-hosted runners is really flaky. We have seen them failing with the following errors:

The following happens during Setup operator environment step.

/usr/bin/sudo snap install core
  error: cannot install "core": Post "https://api.snapcraft.io/v2/snaps/refresh":
         dial tcp [18](https://github.com/canonical/bundle-kubeflow/actions/runs/7098729518/job/19405504840#step:8:19)<IP_ADDRESS>:443: connect: connection refused

This happens during bundle deployment and while bundle has been deployed and it shows missing for all charms that it waits for. wait_for_idle acts like no charm has been deployed to the model it observes.

INFO     juju.model:model.py:2947 Waiting for model:
  admission-webhook (missing)
  ... (missing)

The action never starts. At the time of writing, this action has been in queue for 1 day and 10 minutes.

The above scenarios happen after we 've seen them running successfully (almost - one UAT failed), and with no change to the tests themselves whatsoever.
To Reproduce
Run the action https://github.com/canonical/bundle-kubeflow/actions/workflows/full-bundle-tests.yaml
Environment
self-hosted runners
Relevant Log Output
.

Additional Context
No response

thank you @orfeas-k
also noting that the one UAT we observed to fail was training-integration and it's being tracked in https://github.com/canonical/charmed-kubeflow-uats/issues/50


Add builtinFuncs to ExecutionState.Funcs as desired to reduce memory usage.

Required for all non-trivial PRs

[x] Rebased/mergable
[x] Tests pass
[x] CHANGELOG.md updated
[x] Sign CLA (if not already signed)

@nathanielc this PR aims to reduce memory consumed by builtinFuncs. Currently, there are 80+ build in funcs, but they won't be used in most tickscripts. Please help to review this PR.
Thanks a lot.


Clicking gvim close window button is interrupted by File Changed dialog.

What steps will reproduce the problem?
1. Open a file in gvim.
2. Unfocus the gvim window.
3. Change file outside of gvim.
4. Click "close window" button on window.

What is the expected output? What do you see instead?
Expect: window closes immediately.
Actual: closing is interrupted by dialog "W11 Warning: File ... has changed
since editing started [OK] [Load File]".

This dialog is meaningless if the buffer is clean, since the next action will
be to quit.

What version of the product are you using? On what operating system?
OS: Linux with XFCE4 (Xubuntu).
$ gvim --version
VIM - Vi IMproved 7.4 (2013 Aug 10, compiled Feb  9 2015 16:31:56)
Included patches: 1-622
Modified by<EMAIL_ADDRESS>Compiled by<EMAIL_ADDRESS>Huge version with GTK2 GUI.  Features included (+) or not (-):
+acl             +farsi           +mouse_netterm   +syntax
+arabic          +file_in_path    +mouse_sgr       +tag_binary
+autocmd         +find_in_path    -mouse_sysmouse  +tag_old_static
+balloon_eval    +float           +mouse_urxvt     -tag_any_white
+browse          +folding         +mouse_xterm     +tcl
++builtin_terms  -footer          +multi_byte      +terminfo
+byte_offset     +fork()          +multi_lang      +termresponse
+cindent         +gettext         -mzscheme        +textobjects
+clientserver    -hangul_input    +netbeans_intg   +title
+clipboard       +iconv           +path_extra      +toolbar
+cmdline_compl   +insert_expand   +perl            +user_commands
+cmdline_hist    +jumplist        +persistent_undo +vertsplit
+cmdline_info    +keymap          +postscript      +virtualedit
+comments        +langmap         +printer         +visual
+conceal         +libcall         +profile         +visualextra
+cryptv          +linebreak       +python          +viminfo
+cscope          +lispindent      -python3         +vreplace
+cursorbind      +listcmds        +quickfix        +wildignore
+cursorshape     +localmap        +reltime         +wildmenu
+dialog_con_gui  +lua             +rightleft       +windows
+diff            +menu            +ruby            +writebackup
+digraphs        +mksession       +scrollbind      +X11
+dnd             +modify_fname    +signs           -xfontset
-ebcdic          +mouse           +smartindent     +xim
+emacs_tags      +mouseshape      -sniff           +xsmp_interact
+eval            +mouse_dec       +startuptime     +xterm_clipboard
+ex_extra        +mouse_gpm       +statusline      -xterm_save
+extra_search    -mouse_jsbterm   -sun_workshop    +xpm

Please provide any additional information below.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 20 Jul 2015 at 7:05

I don't understand the issue.  After I've modified a file which is opened in gVim from a terminal, as soon as I focus back gVim, I immediately get the W11 warning.  I don't even have the possibility to click on a "close window" button.  I have to click on "OK" or "Load File" in the popup displaying W11.

Same for me.  I think it's always good to warn the user that something unexpected is happening.  It's easy enough to click OK if you don't care.


Starting the code server from VS Code Insiders should automatically install pre-release versions of extensions

Starting the tunnel (code-insiders tunnel) and clicking the link to access the tunnel opens the insiders version of VS Code. However, it installs stable version of the Python (and Jupyter) extension, which is not compatible with VS Code Insiders so it fails to activate.
Related: https://github.com/microsoft/vscode-python/issues/20531

However, it installs stable version of the Python (and Jupyter) extension, which is not compatible with VS Code Insiders so it fails to activate.

May I know who is installing these extensions? I do not think VS Code automatically installs any extensions.

It's done automatically for me. I was thinking it might be settings sync, but I actually have the pre-release version of these extensions..

It's done automatically for me. I was thinking it might be settings sync, but I actually have the pre-release version of these extensions..

Thats strange. We do not install extensions automatically. Settings Sync is also not turned on automatically. Are there steps to reproduce?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Timothe Gypteau assignment

Second part of Lab 4 assignment

Hi Thimothe, no need to open a new PR every time you push. But you're done with the assignment so let's call it a day :+1:


doc: emphasize trade-off between versionCheckHook and testers.testVersion

Description of changes
I'm not a fan of the recent push to migrate to versionCheckHook. This is a decision better left to each package maintainer, rather than drive-by fixups or some notion of prescribed "best practice".

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Related case: when i expanded --replace to --replace-{fail,warn,quiet}, i did not prescribe any best practice. I only made the implicit explicit and provided options, letting the community decide.


chore: use mathport output in FieldTheory.SplittingField.Construction

The mathlib3 version was manually backported from the mathlib4 version. We use here the mathport output.

bors merge


add build step to ci

part of gatsby migration

this is not needed because of https://github.com/nuxdie/frontendweekend/pull/37/files#diff-b9cfc7f2cdf78a7f4b91a753d10865a2R7


Test protobuf-java 3.25.3

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Was this patch authored or co-authored using generative AI tooling?

test first

Abandon this upgrade, as there are no changes in Java code:

https://github.com/protocolbuffers/protobuf/compare/v25.1...v25.3


Bug: Files and folders don't resolve in Xcode until build or next open

Describe the bug
When first generating an Xcode project, some files and folders don't resolve in Xcode. They're red in the project navigator, files aren't available to quick-open, search results from those files don't show up etc.
If I then build a target, the folders resolve and are no longer red, but the files are still red and not resolved.
If I then close and re-open the Xcode project, the files now resolve.

First Open
After Build 1
After Build 2
After Re-Open

Reproduction steps

Clone https://github.com/jpsim/swiftlint-bazel-example
Generate the Xcode project: bazel run :swiftlint_xcodeproj
Open the Xcode project: xed .
Notice the unresolved folders and files in the project navigator
Build the swiftlint scheme
Notice the folders are now resolved
Close & re-open the Xcode project
Notice the files are now resolved

Expected behavior
Files and folders should resolve correctly when first opening a generated Xcode project.
Versions

Bazel: 5.2.0
rules_xcodeproj: 1cc0245336bf17d0a8483da21048faeb14ac53a9
rules_apple: 1.0.1
rules_swift: 1.0.0

Additional context

This is odd. The folders should all be there, but the files are expected to be red now (we now only pre-generate entitlements as part of project generation), and the only files that will get generated are the ones for the target being built.
I'll dig into this today, and see why we aren't generating the folders correctly here.

I see it now. As always, thanks for the bug report!


Update private static readonly capitalization and add some readonly modifiers where reasonable

Overview
It's that time in the release cycle where I try to catch up on all my stray branches and random changes I've made to make sure everything gets into the release.
This one's going into mrtk_dev, not stabilization.
Updates some static readonly and const capitalization to match our coding guidelines. Also converts some static readonly private fields to const where possible and adds some readonly modifiers to private fields that are treated that way.

/azp run


Add support for Django 2.X

Fixes #15

Codecov Report

Merging #18 into master will decrease coverage by 20.07%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           master      #18       +/-   ##
===========================================
- Coverage   46.05%   25.97%   -20.08%
===========================================
  Files           2        2
  Lines          76       77        +1
  Branches        9        9
===========================================
- Hits           35       20       -15
- Misses         38       56       +18
+ Partials        3        1        -2

Impacted Files
Coverage Δ

django_cookies_samesite/middleware.py
25.00% <100.00%> (-20.34%)
:arrow_down:

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 47106d4...9afbd8b. Read the comment docs.

Thanks @jotes any plan to cut a new release soon? Let me know if I can help

@sfdye Hey, I'm merging this now and release it tomorrow (hopefully around noon). Sorry for so late response.

Gentle bump

Hey,
The new version was released a couple of hours ago :-)
On Wed, 18 Mar 2020 at 13:04, Liuyang Wan<EMAIL_ADDRESS>wrote:

Gentle bump
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/jotes/django-cookies-samesite/pull/18#issuecomment-600583741,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACDAYTHVYU3ABHHK7QRRCM3RIC2D3ANCNFSM4LKTS6ZQ
.

--
Kind regards
Jarek "jotes" Śmiejczak, a python programmer by day, a mozillian by heart...
Homepage: http://jotes.work, Github: http://github.com/jotes
Mobile: +48693027040, Mozillian: http://mozillians.org/u/jotes

Sweet, I missed that. Thank you!


Plugin causing Neovim to start slowly

Hey @WhoIsSethDaniel,
Love the plugin thanks for mentioning it in the issue. I have since set it up to install all my required tools but unfortunately it seems to have performance issues when installing lots of tools.
My configuration is here:
https://github.com/alanjjenkins/chezmoi-dotfiles/blob/master/private_dot_config/nvim/lua/plugins.lua
https://github.com/alanjjenkins/chezmoi-dotfiles/blob/master/private_dot_config/nvim/lua/plugin-config/mason.lua
https://github.com/alanjjenkins/chezmoi-dotfiles/blob/master/private_dot_config/nvim/lua/plugin-config/mason-tool-installer.lua
It seems like Neovim is blocking on startup waiting for Mason Tool Installer to install / update everything on every startup.
It seems this needs to be setup as a delayed async task to me (although you may already be doing this (I haven't read the code yet)). I believe this is possible using plenary.
Note that I might have commented my plugin block for this when you look at my plugins to restore the speed of startup (the lag when doing git commits is getting on my nerves).
Thanks,
Alan Jenkins

All I can say is that it doesn't block for me. I can start editing right away while the tools are being installed (I see the updates in the messages).
I may need more information to help you.

You may try deactivating all plugins but mason and mason-tool-installer to see if that changes things.

I copied your mason-tool-installer config to my own config and brought up nvim. FWIW I had no problems. Most tools were installed (a few weren't because I was missing some prerequisites), but over several minutes it installed around 75 tools while I was doing my normal work within nvim.

Another thing to try: limit your list of tools to 1, wipe the mason install directory, and start nvim. Once the one thing is installed type :messages and cut and paste the entire output here. It may give some insight into what is going on.  I'd try commenting out all plugins but mason and mason-tool-installer first...just to eliminate the possibility of interference of some sort with one of your other plugins.

@WhoIsSethDaniel thanks will give these things a go later once I finish work.

I'm seeing this as well. If I set auto_update = false, the startup lag goes away.

is it lag or is it essentially blocking until all services are installed (I think this latter is what the initial report is saying)?  Either way I find this baffling as I don't see either one.  It also doesn't make much sense for auto_update to affect installation since auto_update only comes into effect when there are updates.
I have one idea that may fix this, but since I don't see the problem it's very difficult to say.

 I've not looked into it further though.
Running the installer code similar to the suggestions here for copilot.lua might be a valid approach: https://github.com/zbirenbaum/copilot.lua

@dsully I'm already deferring until VimEnter, which is the first entry in the link you provide.  VimEnter starts very late in the startup process. If it's truly blocking then it doesn't matter where you start it will block eventually.
How many items are you installing?  The OP had around 75 tools that were being installed.

However I'm not doing a defer_fn(). I'm don't see how that would help, but I can try it.

I went ahead and added a start_delay option. It's documented in the README.md.  By default it's 0, but if you set it to, say, 5000 it will delay starting installation for 5 seconds. This corresponds with the 1) option in the link you provided.  Try different values, perhaps start with a very high value, like 8000 or 10000. See if it helps at all. If it doesn't a detailed report on what exactly happened would be appreciated.

Setting a 3 second delay removed the startup lag / blocking for me. Thanks!

Closing. The start_delay has helped and there has been no feedback since.


Deploy a Contract

Audit https://docs.celo.org/developer-guide/start/hellocontracts

Where to learn more about truffle console?

Where to get started with Docker?

Start the node: Port 8545 blocked by fullnode docker image

follow logs - any reason to do it? How to quit?

"Migrations" -- Returned error: authentication needed: password or unlock.


What tool did you use for annotations?

Seems like a disaster to label each frame by hand.

I used RectLabel


How to get the handle or DLL substring for a specific process using AHK?

Hi there,
I'm just wondering if you have a script that can check for the handle or the DLL substring of a specific process? For example, the screenshot below shows me using Process Explorer to check the hardware ID (0000013c) of my microphone. If the microphone is in use, then the audiodg.exe process will be referencing this. I just need to somehow use AHK to check this for me.

Can you test it?
https://github.com/jNizM/ahk-scripts-v2/blob/main/src/ProcessThreadModule/GetProcessHandles.ahk

Sorry, just saw this. Would it be possible for you to convert this to AHK v1 instead?


[FLINK-27845] support Column Type Schema Convert

Related to FLINK-27845
support Column Type Schema Convert

This PR has some tests I will add as soon as possible before 6.19

Hi @complone
Before submitting a PR, it is best to discuss clearly in JIRA what it is that needs to be done and what the general idea is.
I'll close this one, if we make further progress, we'll continue working on the code.

你好@complone 在提交 PR 之前，最好在 JIRA 中讨论清楚需要做什么，大致思路是什么。 我会关闭这个，如果我们（你和我）取得进一步的进展，我们将继续处理代码。

I will try to sort out some source code analysis articles about flink-table-store and discuss it later


[json] [0.10.8] Wrapping JSON with double quotes replaces the text

In <= 0.10.5, you could highlight text in a JSON file and wrap it in double quotes:

In 0.10.8, the highlighted text is now replaced with a double quote rather than wrapped:

Seen in VSCode 0.10.8, 64-bit, on both Ubuntu 14.10 and Ubuntu 15.10.
No errors in dev console. No change in behaviour when VSCode is started with the --disable-extensions flag.

@alexandrudima Is this related to #3114 ? I just pushed a fix to programmatically setting the __characterPairSupport. Let me know if that's not the right approach.

Yeah, that's the only way for now. I'm looking into bringing characterPairSupport in the configuration json file