[css] CSS completion by writing value first

Imagine I want to space evenly my flex box items. And I don't remember the name of the property which value can be space-evenly or simply I want to write it quicker.
I would write "spaceevenly" and Tab and it would autocomplete: justify-content: space-evenly;

It's an interesting idea, not sure it fits well in code completion, which is mostly about completing what you have already typed.
If someone want to work on this, I would suggest to do it in an extension.


Create Image with Url not working properly

We don't officially support Swift for the SDK currently. However, we're happy to accept pull requests which fix issues related to Swift!

I am trying it through a bridging file like other apis, they all are working fine.


Experiment: use dask for parallelism

This is experimental work (from a while ago) that uses dask to do graph walks in parallel.

@PantelisElinas said (summarised, any errors are mine):

random walk sampling worked well on a multicore machine even with a non-configuration dask client, and gave a reasonable speed-up
subgraph sampling with graphsage worked, but resulted in a slow down (due to overhead of coordinating the parallelism), because the bottleneck is the data transfer of the sampled subgraphs, not the sampling

Closing this PR, because we're unlikely to revisit/rebase this for a while.


Overall report status badge (quarterly and annual)

Card: https://github.com/bcgov/cas-cif/issues/532
Note to self: check Project Contacts Form with errors happo shots after the rebase; there were extra fields for some reason

Card: #532 Note to self: check Project Contacts Form with errors happo shots after the rebase; there were extra fields for some reason

@BCerki Mattheiu updated contact form e2e tests here and it should be fine now.

@matthieu-foucault, I've applied suggestions and rebased


[Tensor] Add optional output tensor for tensor concatenation

This PR adds an optional feature in Tensor::cat to pass the output tensor to the function.
This change allows the user-given tensor to store the result of the concatenation without creating a new tensor.
Changes proposed in this PR:

Add optional argument output (the output tensor) to the cat function.
Add negative test cases for tensor concatenation.

Self-evaluation:

Build test: [X]Passed [ ]Failed [ ]Skipped
Run test:   [X]Passed [ ]Failed [ ]Skipped

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2703. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

I think it's a good direction for improvement.
Q) However, is it changing the entire signature?
Rather than immediately changing the function's signature, it may be better to add a new parm by overloading it and leaving the existing function as it is.
Overloading it to maintain compatibility, the old function is left out, and the internal functionality is changed into a new form.

Thank you for pointing it out! To explain, the function Tensor::cat() is overloaded and does not change the function signature. However, the concat() function is changed but will not require any exterior changes since it is only used internally.


Add User property to Chat Completion request

The Chat Completion API supports a User property, as described here: https://platform.openai.com/docs/api-reference/completions/create#completions/create-user.

A unique identifier representing your end-user, which can help OpenAI to monitor and detect abuse.

This property must be added to ChatGptOptions.

What about adding it to the ChatGptParameters class? It should be useful to be able to modify the property based on some condition: for examples in the documentation it says that it would be best to create one per user.
Putting on the ChatGptParameters class will give the developer the option to populate it on every call (maybe checking the authenticated user) rather than one per application

If you need to popolate it on every call, you can use the AddChatGpt overload that provides a ServiceProvider and that is called for every request. With this method, you can change even the API key (assuming, for example, that each authenticated user has a different key).


Make nvm install automatically use the installed version

Description of the new feature / enhancement
This will be consistent with the nvm-sh/nvm, which automatically uses the installed version. NVM4W does not do this automatically atm but gives a suggestion to nvm use x.x.x.
Scenario when this would be used?
For developers jumping across platforms, this is a subtle difference to notice/remember. One could do nvm install and get going with it, but didn't even realize it was pointing to the old version.
Supporting information
Output of nvm on WSL
Downloading and installing node v20.13.0...
Downloading https://nodejs.org/dist/v20.13.0/node-v20.13.0-linux-x64.tar.xz...
################################################################################################################# 100.0%
Computing checksum with sha256sum
Checksums matched!
Now using node v20.13.0 (npm v10.5.2)

Output of nvm on Windows
Downloading node.js version 20.13.0 (64-bit)...
Extracting node and npm...
Complete
npm v10.5.2 installed successfully.

Installation complete. If you want to use this version, type

nvm use 20.13.0

I give my permission to make a withdrawal


⚠ Bofrost.at has degraded performance

In 0ca97b1, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1614 ms

Resolved: Bofrost.at performance has improved in 2b91385 after 6 minutes.


How to verify hard adder in FPGA

Hi,I try to implement hard adder in FPGA arch，How can i verify the arith mode? Do i need to change the yosys script to map verilog benchmark to adder?
Thanks.

@lnyyyy You are right. Currently, our yosys scripts do not contain the carry mapping feature.


Implement VM groups through proper groups

VM group should be implemented through proper groups, not through VM tags.
This issue covers the following requirements:

[REQ_VM_CREATE_013] A user should be able to set VM group (VM creation)
[REQ_VM_DTLS_004] A user should be able to manage VM groups (VM details sidebar)
[REQ_VM_LIST_004] A user should be able to filter the VM list by VM groups

This issue includes:

Removing the old code (tags)
Unit tests

To test:
REQ_VM_CREATE_013
REQ_VM_DTLS_004
REQ_VM_LIST_004


cleanup: job name and other CI vars

Description
WIP
Checklist

[ ] Investigated and inspected CI test results

Testing Performed

[x] default CI
[ ] default CI with all-tests
[ ] /test openshift-* with all-tests
[ ] /test race-condition-qa-e2e-tests with all-tests

/test

/test gke-race-condition-qa-e2e-tests


consider adding a function to do an ls of a draft deposition

done in https://github.com/zenodraft/zenodraft/tree/dbd7bbb68a18877b677f85ff8af503963e1bfcfa, closing


windows: implement file locking

I added file locking for Windows. Some links to reference constant sources are included.
Tested manually using an example program that simply opens a DB and waits for signal, against Windows XP (sadly, my target platform). According to MSDN this should work on future platforms also, see referenced documentation.
The timeout support was derived from flock_unix.go.

See #420 for why I wasn't able to run the whole test suite on my target platform.

@raggi Thanks for digging into this one. If you can successfully run with the test suite with a lower MaxValueSize (as mentioned in #420) then I'll merge this in.


Module Not Found error from dependency in requirements.txt

I am trying to test a function using the functions-framework that has dependencies inside a requirements.txt file but I am getting an error that the module is not found after running the functions-framework --target=main command.

** Edit the functions-framework only seems to work when I run pip install functions-framework in my virtual env. Why would it not work with a functions-framework installation in my main environment?

Hi @phil-epl, the framework won't handle installing your dependencies for you. You'll need to have both the framework (pip install functions-framework) and your functions dependencies (pip install -r requirements.txt) installed into whatever environment you're using.
Let me know if you have any other questions!

Hi @phil-epl, the framework won't handle installing your dependencies for you. You'll need to have both the framework (pip install functions-framework) and your functions dependencies (pip install -r requirements.txt) installed into whatever environment you're using.
Let me know if you have any other questions!

Closing as there's nothing to do here.


Child library nth-check is vulnerable to CVE-2021-3803 even with the latest<EMAIL_ADDRESS>

There is a dependent library nth-check in react-scripts which is vulnerable to CVE-2021-3803. The mend scan is reporting this vulnerability for our project.
--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>| --<EMAIL_ADDRESS> |   --<EMAIL_ADDRESS>|     --<EMAIL_ADDRESS> |       --<EMAIL_ADDRESS>nth-check beyond v2.0.1(including) is available and safe from this vulnerability.
To fix this we have upgraded to the latest version for react-scripts (as shown above) still this dependency is not upgraded. Can you please take some action and upgrade this transitive dependency inside the parent library react-scripts at the earliest possible.

Vulnerable react-scripts 5.0.1 makes our product vulnerable. Please upgrade the react-scripts 5.0.1 at the earliest possible with transitive dependencies security patches. It should have the updated transitive dependency for nth-check by upgrading nth-check/1.0.2 to latest available version nth-check/2.1.1.

We are having the same issue in our project. Please upgrade this child dependency ASAP.

A few of our applications are blocked because of this as well. :(

This is also causing me and my team huge problems.

Do we have some update on this.

An "Inefficient Regular Expression Complexity" vulnerability is not of any concern in a build tool. Feel safe to waive this security issue. Also look at this pinned post.

This seems to be a dup of #12146, and would be resolved by #11174, https://github.com/facebook/create-react-app/pull/12026,  or https://github.com/facebook/create-react-app/pull/12026
It appears that migrating from v5 to v6 of @svgr/webpack requires following a migration guide, so the work may be non-trivial.

#11174 is the correct answer, just move react-scripts out of dependencies and into devDevependencies and run npm audit --omit=dev


unbound diagnostics ruby -> python

@fichtner How is work on this one going?

not at all, away for a client the next few days.

Ad, many thanks for taking care of this!

you're welcome ;)


Add message filter for datasets as well

Story
Add message filter for datasets as well
Context
Right now, curators cannot filter on records that have a message yet.
They are still figuring out what their workflow would be, which is why they did not have this filter yet.
Current situation

Desired situation

was already implemented, but facet never added to the page


AVP doesn't resolve values from AWS Secretsmanager

I'm trying to configure the AVP to read data from my AWS SecretsManager. I have two AWS accounts: test and production (one SecretsManager for each account) but only one Helm Chart.
In test all works fine but in production if I create the Application I get this error:
Error: Replace: could not replace all placeholders in Template: AccessDeniedException: User: arn:aws:iam:::user/avp is not authorized to perform: secretsmanager:GetSecretValue on resource: arn:aws:secretsmanager:eu-west-1:*:secret:test/myapp-m5nXFb
It is correct because the Chart contains a link to the SecretsManager of the test account.
The file values.yaml
...
vault:
aws:
arn: "arn:aws:secretsmanager:eu-west-1:******:secret:test/myapp-m5nXFb"
...
If I try to override the Helm values adding to the Application manifest  these lines:
source:
helm:
parameters:
- name: "vault.aws.arn"
value: "arn:aws:secretsmanager:eu-west-1:****:secret:prod/myapp-h15kMC"
ArgoCD creates the application without any error, but it doesn't resolve the value through the SecretsManager. The resulting k8s deployment manifest is:
spec:
  containers:
      env:
        - name: KC_DB_URL
          value: >-
            jdbc:postgresql://<path:arn:aws:secretsmanager:eu-west-1:****:secret:prod/myapp-h15kMC#MY-SECRET>:5432/

It seems that changes in the Application manifest cause AVP not running: even if I force an Hard Refresh the plugin doesn't start.
Do you have any suggestion?
Thanks

It looks like the error is that your user doesnt have the permissions to get the secret value. AccessDeniedException: User: arn:aws:iam:::user/avp is not authorized to perform: secretsmanager:GetSecretValue on resource: arn:aws:secretsmanager:eu-west-1:*㊙test/myapp-m5nXFb. So i would check your users permissions as well as the permissions set on the secret itself. Doesn't seem to be an issue with AVP itself.

Hi werne2j, thanks for your response. The "AccessDeniedException" is correct because my chart contains the "arn" of the test environment, but if I try to override the value in Application manifest with the correct "arn" then the AVP is not called (I don't see the  logs for the sidecar container), the Application is created but the placeholders "<path:arn:aws....." are never replaced.
So if I don't change the Application manifest then AVP is called (AccessDeniedException is raised) while if I change it (there aren't log in sidecar container and the placeholders are never replaced) AVP is not called

Ah, I see. It looks like you are using source helm while trying to do the override. That won’t work because you’re not using the built in helm. This should help https://argo-cd.readthedocs.io/en/stable/operator-manual/config-management-plugins/#using-environment-variables-in-your-plugin you have to your the plugin source and provide env vars or Paramus depending on the version of Argo cd you are using.

Thanks werne2j I solved using environment variables.


Do we need to require a user to have conda configured in rc files?

On some systems, it makes sense for users to rely on system python and only use conda for certain tasks. E.g. someone who spends a lot of time running old code may want a default environment that uses python 2, and rather than set up (base) in that manner is content to use the out-of-the-box python environment.
I removed the conda initialization from my .tcshrc file on the VM where I test using tcsh, and when I try to use jupyter-forward to connect I get and error about being unable to find jupyter:
$ jupyter-forward VMtcsh
********************************************** Authenticating **********************************************
Authenticating user (mlevy) from client (cgdm-chaohu) to remote host (localhost)
✅ The client is authenticated successfully
***************************************** Verifying shell location *****************************************
✅ Using shell: /bin/tcsh
************************************** Running jupyter sanity checks ***************************************
/bin/tcsh -c "which jupyter"
jupyter: Command not found.
*************************** ❌ Terminated the network 📡 connection to localhost ***************************

Is there some way to catch that the issue is actually that I commented out
# >>> conda initialize >>>
# !! Contents within this block are managed by 'conda init' !!
if ( -f "$CONDA_PREFIX/etc/profile.d/conda.csh" ) then
    source "$CONDA_PREFIX/etc/profile.d/conda.csh"
else
    setenv PATH "$CONDA_PREFIX/bin:$PATH"
endif
# <<< conda initialize <<<

(note that $CONDA_PREFIX is not defined without sourcing that file). One possible solution would be to pass an optional --conda-location argument; between that argument and self.shell maybe it would be possible to load conda before trying to do anything else?

Relatedly, does jupyter-forward support having a configuration file somewhere on the remote machine? It seems like that would be a good place to store the value for --conda-location instead of aliasing the command or having to provide it every time you try to connect.


Internal: Update local test check

What changed?
Visual tests were not always running locally as expected. This updates old Bash script.

Can you give more context on the change? What's different?


Added new baseline for text extents

Matches UV-CDAT/uvcdat#2053

ahem @doutriaux1 this was apparently not merged yet.


OkHttp.Builder#setQueryParameter uses too optimistic encoding

The following code throws an exception when HttpUrl tries to build a java.net.URI, which is overly strict…
      String jsonObject = "{}";
      OkHttpClient client = new OkHttpClient();
      Request request = new Request.Builder()
          .url(HttpUrl.parse("http://www.example.com")
              .newBuilder()
              // HttpUrl.Builder doesn't properly encode curly braces (at least from java.net.URI's point of view)
              .setQueryParameter("config", jsonObject)
              // this works, but URLEncoder!
              //.setEncodedQueryParameter("config", URLEncoder.encode(jsonObject))
              .build())
          .get()
          .build();
      Response response = client.newCall(request).execute();

Regrettably, OkHttp should use a more extended set of characters to encode, similar to the 2.5.0 change with URL encoding.

Does it work with OkHttp 2.5.0 ?

(We avoid java.net.URI internally for this reason)

2.5.0 didn't change this as far as I can tell. The exception happens in OkHttp code, where java.net.URI is indeed used:
java.lang.IllegalStateException: not valid as a java.net.URI: https://www.example.com?config={}
at com.squareup.okhttp.HttpUrl.uri(HttpUrl.java:336)
at com.squareup.okhttp.internal.http.RouteSelector.resetNextProxy(RouteSelector.java:135)
at com.squareup.okhttp.internal.http.RouteSelector.<init>(RouteSelector.java:71)
at com.squareup.okhttp.internal.http.RouteSelector.get(RouteSelector.java:76)
at com.squareup.okhttp.internal.http.HttpEngine.connect(HttpEngine.java:321)
at com.squareup.okhttp.internal.http.HttpEngine.sendRequest(HttpEngine.java:245)
at com.squareup.okhttp.Call.getResponse(Call.java:267)
at com.squareup.okhttp.Call$ApplicationInterceptorChain.proceed(Call.java:224)
at com.squareup.okhttp.Call.getResponseWithInterceptorChain(Call.java:195)
at com.squareup.okhttp.Call.execute(Call.java:79)

Yup. Ugh! If we escape it just for the benefit or URI, maybe that makes everything better.

Hard to escape legacy code, isn't it? ;-)

Fixed with https://github.com/square/okhttp/pull/1959


Added Action to Run Tests

This PR adds an action that downloads the released packages and the relative package from source to run the qiskit-neko tests. A new folder was created in .github with the addition of the action.yml file.
Another file package.json was added.

The first commit is the initial commit and needs testing and editing accordingly.

Thanks for the review! I have made a commit where I have tried to address the comments.

The action seems to be working well on the demo repository! What should be the next steps to follow?

I have also updated the delta parameter in test_neural_network_classifier.py. It seemed to have been passing in only half of the cases. I ran this a few times locally and have updated it with the appropriate new values.


Make it optional to use managed compressions

Make it optional to use managed compressions, from for instance Parquet.net(to increase performance).

@aloneguid I dont know why the tests are failing, no changes where made to the native code at all

@aloneguid conflicts now solved should be ready to go.

just trying something in #13

closing in favour of #13, thanks for your help on this!


[FEATURE] Viewing Asset Information in AAS-UI

Is your feature request related to a problem? Please describe.
To effectively compete with Package Explorer, the AAS-UI should have the capability to view both AAS properties and Asset Information. Currently, only the AAS ID, Description, and Global Asset ID are visible.
Describe the solution you'd like
Enhanced visibility of AAS properties and Asset Information within the AAS-UI.
Describe alternatives you've considered
Continue using Package Explorer.

Note: For small devices/small windows with a width <= 1280px, the first column (AAS list) is hidden. In this case, the GUI does not display any information about the AAS/asset

**Currently, only the AAS ID, Description, and Global Asset ID are visible.

Regarding the asset:
In the details view (i) the Asset Kind, Asset Type as well as the Thumbnail are also already shown
Regarding the AAS:
In the AAS List only the displayName respectively idShort and the ID is shown.
In the details view (i) the displayName, idShort, ID and the description is shown.
From my point of view just the following information are missing:
For the asset just the specificAssetId is missing
For the AAS just the administration information is missing
@JS-Aibel: Is there anything else missing?
@aaronzi
Proposal:
(1) Switch order of AAS and asset information in the details card
(2) Making detailsCard scrollable, so that there is enough space for further AAS information

Thank you for your reply, @seicke

In the details view (i) the Asset Kind, Asset Type as well as the Thumbnail are also already shown

I can now see the assetType in the detail view. However, the value does not include a property description; it simply states the values, for example, Instance for assetType and Type for the kind. When I hover over, the popup shows assetKind: Instance.
Could you please clarify if there should be a description for these properties or just the values?

Regarding assetKind:
There is no further description in the GUI. With respect to IDTA 01001-3-0-1 there are only the following possible values: https://admin-shell.io/aas/3/0/AssetKind
Regarding assetType:
There is also no further description in the GUI. With respect to IDTA 01001-3-0-1 it is just an identifier (string): https://aas-core-works.github.io/aas-core-meta/v3/AssetInformation.html
Note: In #96 we try to find a solution for my above proposal.

Sorry for the confusion! By “description,” I meant the name of the property, not a literal description. In the details view (i), it shows Instance instead of assetKind: Instance. This makes it unclear whether the value displayed is an assetType or a kind.

Here you can see the Asset Informatino of a "real" AAS:

I personally would not need any further information at this point about what Instance  or Rectangular connector ... means, I would find it rather annoying:


Remaining Bidirectional RNN merge modes

After https://github.com/tensorflow/swift-apis/pull/946 there is support for BiRNN with concatenate and sum mode. It would be awesome to have remaining merge modes from keras: ave, mul and none.

Could someone assign this issue to me, please?

Thanks!


useFunction() does not work when promise is returned and function is not async

If you create a useFunction() with the form:
useFunction(this, (state, ...args) => { return this.store.query(...) }, () => [...]);
it will not work correctly.  The function must be declared async as in:
useFunction(this, async (state, ...args) => { return this.store.query(...) }, () => [...]);

Resolved by https://github.com/NullVoxPopuli/ember-resources/pull/81


[html5] Account for null buffer.data in ImageCanvasUtil.createImageData()

Reproducible with this code, which seems to stop after the ~first frame and doesn't draw the diagonal red line as expected:
class Main extends Sprite {

	var bmd:BitmapData;
	var i:Int = 0;

	public function new ()
	{
		super();

		bmd = new BitmapData(100, 100);
		addChild(new Bitmap(bmd));

		stage.addEventListener(Event.ENTER_FRAME, onEnterFrame);
	}

	public function onEnterFrame(e):Void {
		i %= Std.int(bmd.rect.width);
		bmd.setPixel32(i, i, 0xFFFF0000);
		i++;
	}
}

The issue is probably localized here: https://github.com/openfl/lime/blob/80e034721085a05dc0cfa2ab71de58d94594fda2/lime/graphics/utils/ImageCanvasUtil.hx#L177-L188
I've fixed it locally by adding this at the end of ImageCanvasUtil.createImageData()
		} else if (buffer.data == null) {
			buffer.data = new UInt8Array ((buffer.__srcImageData.data.buffer:js.html.ArrayBuffer));
		}

but I'm not sure it's the correct fix.

Mmmhh... It seems to be a timing issue/race condition somewhere, because if I call the setPixel loop from inside a for all works fine.
My temp fix is obviously not the right thing to do, as that will recreate the buffer every frame.
Trying to debug with chrome dev tools and stepping through code I get a random number of pixels drawn (but the first one is always shown). Then at some point data gets set to null somewhere (by sync() probably) and all breaks.
Ideas?

What about this at the end of ImageCanvasUtil.convertToCanvas()?
		} else if (buffer.data == null && buffer.__srcImageData != null) {
			buffer.data = cast buffer.__srcImageData.data;
		}

It seems to work for me but, again, I don't know the implications and if it's the proper fix.

Thanks, I just had the same problem, your last fix is working for me here :smile:

np, thanks.


merge: v1.14.3

📑 Description

@coderabbitai summary

✅ Checks

[ ] Added test cases for my code

Q: missing session file driver fix.

@devhaozi Good catch, added.


Flutter web usePathUrlStrategy bug, loading infinite

Steps to reproduce
Go directly to http://localhost:58703/user/user_name/Jhon
It won't work.
Comment out the usePathUrlStrategy(); and everything will work normally
A point of attention to reproduce the same error when reaching the url by f5 on the page or by entering the browser url to reload
Expected results
That the application is using usePathUrlStrategy() http://localhost:58703/user/user_name/Jhon
Actual results
The Application simply keeps loading infinitely
Code sample
Code sample
https://github.com/eduardohr-muniz/go_router_poc

Screenshots or Video

Screenshots / Video demonstration

https://github.com/eduardohr-muniz/go_router_poc
Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.24.0, on macOS 14.6.1 23G93 darwin-arm64, locale pt-BR)
[✓] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2024.1)
[✓] VS Code (version 1.92.2)
[✓] Connected device (3 available)
[✓] Network resources

This is a known issue being tracked at https://github.com/flutter/flutter/issues/152995. It's fixed on Flutter master channel.
Closing this issue as a duplicate, so please follow up on it for further updates.


feat: better errors for remote SDK

Adds nicer errors to remote SDK, that expose useful properties like request_id and status_code.
Makes sure the Python tracebacks print nicely by mapping the source field from a Rust error to the __cause__ field.

Here's a comparison of the error messages before and after. A few points I think are improved:

We don't have the nested prefixes like RuntimeError: Runtime error: ... that come from wrapping errors by formatting them to a string for the new message.
All the client errors provide a request id in the output message
The actual status code is always provided.
The retry error has the underlying error provided above.

Error - No retry
Response is 507 status with "Internal Server Error" body.
Before:
RuntimeError: Runtime error: Internal Server Error

After:
lancedb.remote.errors.HttpError: Http error: (request_id=214db021-cc8e-4f57-8bd7-176999c13064) 507 Insufficient Storage: Internal Server Error

429 Error with retry
Before:
RuntimeError: Runtime error: Request failed after 2 retries with status code 429 Too Many Requests

After:
lancedb.remote.errors.HttpError: 429 Too Many Requests: Try again later

The above exception was the direct cause of the following exception:

lancedb.remote.errors.HttpError: Http error: (request_id=f8f9a923-4a36-459b-8ed7-15522c2ee4d5) 429 Too Many Requests: Try again later

The above exception was the direct cause of the following exception:

lancedb.remote.errors.RetryError: Hit retry limit for request_id=f8f9a923-4a36-459b-8ed7-15522c2ee4d5 (request_failures=2/2, connect_failures=0/3, read_failures=0/3)


[-] Unable to extract login-form form

hello i dont now why but I have a problem when i start the script

is good i delete and download the script

@slider973 take a look #184 . you should wait for some time or upgrade all of the packages required to run the script.


JSON File not recognized as JSON File

Hi
I'm wondering receiving this error when I try to use this extension. The file appsettings.json is a JSON File. Does it have to be in UNIX Format or what else could be the problem?
2021-12-01T12:12:25.4379293Z ##[section]Starting: Substitute variables
2021-12-01T12:12:25.4505667Z ==============================================================================
2021-12-01T12:12:25.4506038Z Task         : Substitute Variables
2021-12-01T12:12:25.4506336Z Description  : Apply variable substitution to JSON and YAML files.
2021-12-01T12:12:25.4506626Z Version      : 1.1.0
2021-12-01T12:12:25.4506840Z Author       : happytal
2021-12-01T12:12:25.4508454Z Help         : https://github.com/happytal/substitute-variables
2021-12-01T12:12:25.4508840Z ==============================================================================
2021-12-01T12:12:25.7608599Z Substituting variables in C:\inetpub\BudgetToolAD\appsettings.json
2021-12-01T12:12:25.7689542Z ##[error]File C:\inetpub\BudgetToolAD\appsettings.json is neither JSON nor YAML
2021-12-01T12:12:25.7750050Z ##[section]Finishing: Substitute variables
Regards,
Christian
appsettings.json.txt

Hi!
The issue here is that the file is not standard JSON because it contains comments. The task relies on the default Node.js JSON parser which cannot parse it.
I am not sure if commented JSON files are quite common, but it seems some editors allow them. For exemple for appsettings.json files which are good candidates of files being processed by this plugin. I added support of commented JSON files using https://github.com/vitaly-t/decomment. It means comments will be stripped from the resulting file but at least the variable substitution will work.
This feature is deployed in version 1.2.0. It should already by available on the Azure DevOps Marketplace.

Thanks for the quick reply and update to fix this issue!
Unfortunately something is missing. It seems that the Marketplace version 1.2.0 still contains version 1.1.0. When I run the task, the error message still shows version 1.1.0 for this extension and the error is the same.

I forgot to update the task version when publishing. I guess this is why it kept the 1.1.0 version and you still have the error. I just published version 1.2.1, hopefully it will fix the issue.

Fantastic! This update fixed the issue!!
Many thanks


⚠ Jenkins Plugins Artifactory has degraded performance

In 24aa2ff, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1364 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 9b569d7 after 12 minutes.


Implements helper classes to retry asynchronous operations

This PR introduces a helper class (AsyncRetryUnaryRpc<>) to implement asynchronous operations with retries.  noex::Table::AsyncApply() uses this new class. The unit test is extended to cover the multiple retry scenarios.

This change is

Ping.

PTAL.


[Unity][nn.Module] Introduce operator empty

This PR introduces an operator op.empty in the nn.Module frontend. It helps us to create an uninitialized memory from the memory pool, which could be used as temporary scratchpad memory to handcrafted operators.

CC: @MasterJH5574 @yzh119


tensorflow-master version  compile build_all_ios.sh  error ( armv7 compilation failed)

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug or a feature request.
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
TensorFlow installed from (source or binary):
TensorFlow version (use command below):
Python version:
Bazel version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:
Exact command to reproduce:

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.

It looks like you hit enter prematurely? Feel free to open a new bug if you're still having a problem. Or reopen with the description you intended to enter, please.


duden: 0.18.0 -> 0.19.1

Description of changes

Closes https://github.com/NixOS/nixpkgs/pull/292064 and https://github.com/NixOS/nixpkgs/pull/291670
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.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[ ] Fits CONTRIBUTING.md.

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

@linuxissuper are you interested in co-maintainership?

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

@linuxissuper are you interested in co-maintainership?

yes, thanks for asking
should I create a new MR for this or do you just add my name?

Result of nixpkgs-review pr 292193 run on aarch64-darwin 1

2 packages built:

duden (python311Packages.duden)
duden.dist (python311Packages.duden.dist)

@linuxissuper are you interested in co-maintainership?

yes, thanks for asking should I create a new MR for this or do you just add my name?

I will just add your tag to maintainers list.

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

2 packages built:

duden (python311Packages.duden)
duden.dist (python311Packages.duden.dist)

https://nix.dev/guides/best-practices.html#recursive-attribute-set-rec

Thanks for merging, @leona-ya, much appreciated 🙂


Push for Next Thursday

I'm going to be speaking at Technically a Coffee briefly on Thursday of next week to talk about the Trolley Tracker. I don't know exactly what I'm going to say because we haven't gone longer than a day with both trolleys running the whole time.
That being said, I'd still like to give folks who are there the chance to download the apps and be testers if nothing else. I think there should be some warning going on with the app letting folks it's still in the test phase or something like that as well.
Is this doable? Do we need to meet for anything before then?
Let me know.
Also, anyone who can attend that Technically a Coffee please do come out. I'd love to spotlight anyone who worked on the project.  http://sched.co/48Pt
Double also, there are still spots on the trolley for the the Thursday brewery tour if folks are around and interested. http://sched.co/482x
Wanted to add @bikeoid @austin9350 @Linktheoriginal into this message and see if y'all have any thoughts.

How do you feel about using the open beta opt-in link for the Android client? (https://play.google.com/apps/testing/com.codeforgvl.trolleytrackerclient) I'm hesitant to push a live version because it opens up reviews and if the tablets aren't running consistently and the schedules aren't 100% nailed down (managed by Greenlink) then the app will be prone to negative reviews. That takes care of the warning too, by making it opt-in.
Your call though.

I think the Beta is the right way to go. Evan had some issues signing up through that process earlier, but he was able to install it today.
I think it is worth getting a couple of people to test the Beta signup process so we can explain that properly to people.

I think the confusion stems from having different Google accounts and being
signed in as one account during the opt-in and another on the play store.
Can't believe Google hasn't streamlined that more.
On Wed, Sep 9, 2015 at 6:24 PM, Adam Gautsch<EMAIL_ADDRESS>wrote:

I think the Beta is the right way to go. Evan had some issues signing up
through that process earlier, but he was able to install it today.
I think it is worth getting a couple of people to test the Beta signup
process so we can explain that properly to people.
—
Reply to this email directly or view it on GitHub
https://github.com/codeforgreenville/trolley-tracker/issues/74#issuecomment-139062112
.

Jim's feedback on signing up through the Beta process "Installed the app. The only thing that was mildly confusing was that when I hit the "Become a Tester" button on the first page I was expecting that to do the actual installation. When I got to the next page and it provided details about how to install it took me reading the text to realize I'd only accepted to be a tester and that I still needed to install the app as a second step." Good note, we can try to do some explaining to folks both in person in and in copy to help with that.

I’ll see if I can make it to the meeting on Thursday. Where can I see this beta sign up process?
On Sep 10, 2015, at 11:54 AM, Adam Gautsch<EMAIL_ADDRESS>wrote:

Jim's feedback on signing up through the Beta process "Installed the app. The only thing that was mildly confusing was that when I hit the "Become a Tester" button on the first page I was expecting that to do the actual installation. When I got to the next page and it provided details about how to install it took me reading the text to realize I'd only accepted to be a tester and that I still needed to install the app as a second step." Good note, we can try to do some explaining to folks both in person in and in copy to help with that.
—
Reply to this email directly or view it on GitHub.

https://play.google.com/apps/testing/com.codeforgvl.trolleytrackerclient You need to be signed in to Google to see the signup process.

Got it. I already signed up for this before... I think that the process was
fairly straight forward. Are there any objections to just having a small
FAQ on the marketing site on how to signup for Beta access?
On Fri, Sep 11, 2015 at 9:19 AM, Adam Gautsch<EMAIL_ADDRESS>wrote:

https://play.google.com/apps/testing/com.codeforgvl.trolleytrackerclient
You need to be signed in to Google to see the signup process.
—
Reply to this email directly or view it on GitHub
https://github.com/codeforgreenville/trolley-tracker/issues/74#issuecomment-139543861
.


Twitter POST requests failing

All of my Twitter POST requests are failing. I've followed the examples for authenticating:
self.oauth = OAuth1Service(
name='twitter',
consumer_key=consumer_key,
consumer_secret=consumer_secret,
request_token_url='https://api.twitter.com/oauth/request_token',
access_token_url='https://api.twitter.com/oauth/access_token',
authorize_url='https://api.twitter.com/oauth/authorize',
base_url='https://api.twitter.com/1.1/'
)
self.session = self.oauth.get_session( (access_token_key, access_token_secret) )
self.session.post('lists/members/create_all.json', params={'list_id':list_id, 'owner_screen_name':owner_screen_name, 'screen_name':chunked_screen_names_str})

AttributeError: 'NoneType' object has no attribute 'items'

The GET requests all work correctly.
I also tried to use an auth_session:
self.oauth.get_auth_session(access_token_key, access_token_secret, method='POST')
but received this error:

KeyError: 'Decoder failed to handle oauth_token with data as returned by provider. A different decoder may be needed. Provider returned: \n\n  The access_token method must be called with a request_token\n  /oauth/access_token\n\n'

I have to supply "data={}" to these calls or it fails like that for me too.


Add config option to skip local link checks

We currently can only ignore remote links based on regex, but there are use cases where users might also want to ignore local links.

With the prometheus-operator project, we have a use case where it would be handy to disable validation for internal links. The context is https://github.com/prometheus-operator/prometheus-operator/pull/4899 which refactors the generation of the API documentation.
The resulting MarkDown file is mostly HTML behind the scenes with <a href="..."> anchor links. As a result, mdox fails validating pages that reference anchor links in the api.md page because it can only process "true" MarkDown syntax.


MONDO:0021421 and MONDO:0021533[Merge]

Mondo term (ID and Label)
MONDO:0021421 , carcinoid tumors, intestina[l] (note there is a typo, missing L at the end of this name)
Reason for deprecation
UMLS maps this OMIM record to the same CUI at the NCIT and SNOMED_CT concept on the term below
Term to be merged with
MONDO:0021533, intestinal neuroendocrine tumor G1
Your nano-attribution (ORCID)
If you don't have an ORCID, you can sign up for one here

Additional X-refs that can be added
UMLS CUI: C0349535
MeSH C562842


Refactored code to improve readability and understand complex logic

Thanks, but I don't see any particular improvement here.


badge code

Coverage remained the same at 100.0% when pulling 6f1fabb271193ad57b22cc6d5790b5efe24eeea7 on badges into ec43ce664d7eea6961891dd3a05ed0ec5ecd4732 on develop.


[env] Add --env and --env-file flags

Summary
Adds --env and --env-file flags that can be used with run shell and services
If both flags are specified they are merged with --env taking precedence.
environment variables passed in with flags take the highest level of precedence, replacing any env variables set by nix, devbox.json, or plugins.
TODO: The flag code is a bit brittle and repetitive, I think we can consolidate the common flag registration.
How was it tested?
devbox run -e FOO=BAR echo \$FOO
devbox services up -e HTTPD_PORT=8083

Should also help with #1215, since we can set the NIX_HARDENING_ENABLE variable with the -e flag


Error in BuildingSystems.Buildings.Constructions.Windows.SlidingWindow

Applying Dymolas (2017 FD01) checkModel() to BuildingSystems.Buildings.Constructions.Windows.SlidingWindow in pedantic mode throws following error:
The model BuildingSystems.Buildings.Constructions.Windows.SlidingWindow component radTra1to2 is structurally singular.
when assuming inputs to be known,
and that non-physical connectors will impose suitable constraints.

issue #68 solved. Running Example of for model class SlidingWindow added


⚠ Bofrost.es has degraded performance

In 49612d2, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1559 ms

Resolved: Bofrost.es performance has improved in 99cba68 after 7 minutes.


Associations not in controlled vocab

Note that the source data contains emof:association that do not have a match in the controlled vocab. To be investigated.

Could be caused by ~

~ are now handled, all values are mapped:

measurementType
measurementTypeID
measurementValue
measurementValueID
measurementUnit
measurementUnitID

association
https://vocab.ices.dk/services/rdf/collection/Association
associated with cetaceans
https://vocab.ices.dk/services/rdf/collection/Association/11

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
associated with fish shoal
https://vocab.ices.dk/services/rdf/collection/Association/10

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
associated with fish trap
https://vocab.ices.dk/services/rdf/collection/Association/101

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
associated with fishing vessel
https://vocab.ices.dk/services/rdf/collection/Association/26

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
associated with floating litter
https://vocab.ices.dk/services/rdf/collection/Association/15

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
associated with floating seaweed
https://vocab.ices.dk/services/rdf/collection/Association/17

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
associated with front
https://vocab.ices.dk/services/rdf/collection/Association/12

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
associated with land (e.g. colony)
https://vocab.ices.dk/services/rdf/collection/Association/28

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
associated with line in sea
https://vocab.ices.dk/services/rdf/collection/Association/13

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
associated with observation base
https://vocab.ices.dk/services/rdf/collection/Association/18

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
associated with offshore platform
https://vocab.ices.dk/services/rdf/collection/Association/23

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
associated with oil slick
https://vocab.ices.dk/services/rdf/collection/Association/16

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
associated with or on buoy
https://vocab.ices.dk/services/rdf/collection/Association/22

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
associated with other vessel
https://vocab.ices.dk/services/rdf/collection/Association/21

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
associated with set net
https://vocab.ices.dk/services/rdf/collection/Association/100

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
associated with shallows or sand banks
https://vocab.ices.dk/services/rdf/collection/Association/29

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
deliberately approaching observ. base
https://vocab.ices.dk/services/rdf/collection/Association/20

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
drive hunt MSFA participant
https://vocab.ices.dk/services/rdf/collection/Association/59

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
flying between (turbine) foundations
https://vocab.ices.dk/services/rdf/collection/Association/200

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
flying or swimming near wind turbine (foundation)
https://vocab.ices.dk/services/rdf/collection/Association/117

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
inside wind farm
https://vocab.ices.dk/services/rdf/collection/Association/110

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
multi-species feeding association (MSFA) participant, beater
https://vocab.ices.dk/services/rdf/collection/Association/55

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
multi-species feeding association (MSFA) participant, joined by others
https://vocab.ices.dk/services/rdf/collection/Association/51

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
multi-species feeding association (MSFA) participant, joining flock
https://vocab.ices.dk/services/rdf/collection/Association/52

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
multi-species feeding association (MSFA) participant, no further details
https://vocab.ices.dk/services/rdf/collection/Association/50

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
multi-species feeding association (MSFA) participant, scrounger type
https://vocab.ices.dk/services/rdf/collection/Association/53

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
multi-species feeding association (MSFA) participant, social feeder
https://vocab.ices.dk/services/rdf/collection/Association/56

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
multi-species feeding association (MSFA) participant, solitary diver
https://vocab.ices.dk/services/rdf/collection/Association/54

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
next to sandbank
https://vocab.ices.dk/services/rdf/collection/Association/103

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
on sandbank
https://vocab.ices.dk/services/rdf/collection/Association/102

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
sitting on marking pole or stick
https://vocab.ices.dk/services/rdf/collection/Association/25

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
sitting on observation base
https://vocab.ices.dk/services/rdf/collection/Association/19

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
sitting on offshore platform
https://vocab.ices.dk/services/rdf/collection/Association/24

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
sitting on or near floating wood
https://vocab.ices.dk/services/rdf/collection/Association/14

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
sitting on wind turbine (foundation)
https://vocab.ices.dk/services/rdf/collection/Association/116

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
type II multi-species feeding association (MSFA) participant
https://vocab.ices.dk/services/rdf/collection/Association/57

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/

association
https://vocab.ices.dk/services/rdf/collection/Association
type III multi-species feeding association (MSFA) participant
https://vocab.ices.dk/services/rdf/collection/Association/58

http://vocab.nerc.ac.uk/collection/P06/current/XXXX/


Tree view of possibilities

A tree view of the choices would be very helpful.

So how exactly would this look? You got an example?
Like would it be a nested directory listing? A graph? Would it replace the entire lower part of the front page or be another way to display that front page data?


Gauge chart tweaks

A few tweaks to change if possible:
Is it possible to make the England and Region label appear under the gauge so no value labels are cut off?
Make England and region label and value line the same colour e.g blue and grey or two shades of blue?
The green colours on the gauge are very similar - adjust them slightly?
Dial blurry on the individual NEET and Not Known charts - might just be to do with the position of the dial on the chart though.

Issues resolved.


Convert cncf-cla to bytedance-cla

Convert cncf-cla to bytedance-cla

I thought we didn't enable cla plugin and this should not be applied?

I thought we didn't enable cla plugin and this should not be applied?

I just think that we need a cla label for bytedance instead of cncf cla. Maybe we could write our own cle_bytedance plugin in the future.

Why robot take such long period time to enable ci-robot...
Let me figure out the cause...

/close


format file content

Hi @4change, please sign EasyCLA first which will allow us to review your PR,
You can find about EasyCLA here - https://docs.linuxfoundation.org/lfx/easycla
and the steps to sign EasyCLA are here - https://docs.linuxfoundation.org/v2/easycla/contributors/individual-contributor
You can start your signing EasyCLA from here - https://api.easycla.lfx.linuxfoundation.org/v2/repository-provider/github/sign/18706487/51478266/38056/#/?version=2

/close
No CLA for over a month.


Remove unnecessary null pointer checks

An extra null pointer check is not needed in functions like the following.

AudioShaderStream
ProjectParser::Open

Thank you so much for the tip - didn't know about that!
I hopefully removed all unnecessary checks. While doing this I've also realized I have used glDeleteShader instead of glDeleteProgram on couple of shader programs. Thanks!

Thanks for your small source code improvement.


Check event's source before logging

Very useful scripts, however when using them together with other input framework scripts (like [extract-all-files] (https://github.com/LiamRandall/BroMalware-Exercise/blob/master/solutions/extract-all-files.bro)) I got fake notices.

Good catch-- you're right that if you have multiple scripts using the input framework, any of those would trigger this. Out of curiosity, why did you chose to use the strcmp function over something like this?
if ( name == "torlist" )

Haha, no reason…

On 17 Aug 2015, at 15:18, jshlbrd<EMAIL_ADDRESS>wrote:
Good catch-- you're right that if you have multiple scripts using the input framework, any of those would trigger this. Out of curiosity, why did you chose to use the strcmp function over something like this?
if ( name == "torlist" )
—
Reply to this email directly or view it on GitHub https://github.com/CrowdStrike/cs-bro/pull/2#issuecomment-131797739.

Fair enough!


chore(deps): update dependency prettier to v1.14.1

This Pull Request updates dependency prettier from v1.14.0 to v1.14.1

Release Notes
v1.14.1
Compare Source
diff

JavaScript: add parens for unary in bind (#4950)
JavaScript: format angular jasmine it("should ...", fakeAsync(() => { ... correctly. (#4954)
JavaScript: Revert this/super blacklist for function composition heuristic (#4936)
JavaScript: no extra space on Flow interface method named static (#4910)
JavaScript: no extra line break in destructed assignment of ternary (#4932)
Flow: print ObjectTypeInternalSlot with both flow/babel parsers (#4869)
TypeScript: no invalid output for ImportType in TypeReference (#4939)
YAML: do not throw on duplicate merge key (#4931)
YAML: no duplicate comments in mappingValue (#4931)
YAML: print end comment in nested mapping correctly (#4918)
YAML: do not put singleline values on a separate line from the key (#4916)
YAML: prefer dash as document separator (#4921)
API: update support info for Flow (#4943)
CLI: ignore .git, .svn and .hg directories (#4906)
CLI: support TOML configuration files (#4877)

This PR has been generated by Renovate Bot.

Coverage remained the same at 93.878% when pulling dde65efd3a1fbcfcddce76af0a80dd8db45ec8ae on renovate/prettier-1.x into 2473a41f1981f20cc72bc261f1f9ce240fa29cfa on master.

Coverage remained the same at 93.878% when pulling dde65efd3a1fbcfcddce76af0a80dd8db45ec8ae on renovate/prettier-1.x into 2473a41f1981f20cc72bc261f1f9ce240fa29cfa on master.


bloop: 1.4.5 -> 1.4.6

Motivation for this change
Update bloop to latest version
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ x Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I'm trying to update to nixpkgs-20.09, and getting this when updating/reinstalling bloop...
  wanted: sha256:1i9kh4h5w3gr4w6rf5m7xpfqwps91mfddl1zk25v7f6vsraayam0
  got:    sha256:1wcgrnbjrhn9v6lx6bm2g9ddx7zjbdbrb04i22mbqmp384j8d7fj

I'm on mac. What could be happening here?

Hey, I might have made a mistake during the update. I do not have a mac to test it on so I get the sha256 from the files. Let me check that.

Yes, It seems I did make a mistake. Sorry for that, I'll fix it right away.


Feat(azure function): Support saving unpublished entity changes

Features
This changes the azure function to update the database when content is saved or auto-saved as long as it isn't published.
To test

go to the contentful webhook for dev
go to Settings
tick create, save, auto-save
disable the queue receiver for the dev function in the azure portal
Run your own locally and try making changes to content when its published/unpublished and check the db

🥇
Jolly good. Very clean simple fix. I'd add E2E tests for further safety, but from my testing seems good.
I wonder if it's worth moving ShouldIgnoreMessage to the MappedEntity class now, since that actually contains the majority of the info needed for the method (except the UsePreview field in ContentfulOptions), but that's more work for no real gain.
Anyway, jolly good.


Add express bullets

e.g. 6, 7

Added in https://github.com/louh/mta-subway-bullets/commit/217c662fe84cb4daae20d2af87f1426dc642bc34


No help for permissions

I don't know how to make it so that Visitors get lower permissions and Members get higher permissions. And the help command doesn't give you a list of commands.

@CraftedPixel The permissions system is designed in such a way that you simply make a group (Called Visitor and Member in this case), link a physical role to it if you wish and add and remove permissions as you wish. You will also find that doing _help will list the commands.
Please join our support guild if you have any further queries: http://discord.me/flarebot


Basket Line Reference with already encoded utf-8 options

In an ecommerce model with Options a Basket Line Reference is created by its options representation encoded as utf-8. It raises a problem when option value is already encoded in utf-8.
    def _create_line_reference(self, product, stockrecord, options):
        """
        Returns a reference string for a line based on the item
        and its options.
        """
        base = '%s_%s' % (product.id, stockrecord.id)
        if not options:
            return base
        return "%s_%s" % (base, zlib.crc32(repr(options).encode('utf8')))

I imagine this encoding is done because only basket.AttributeLine value supported is charField. In extended models with more rich Attribute values it is a problem.

Sorry forgot python3 bytes enconding

I suppose the issue can be closed, since related PR was merged.

Closed since related PR already merged.


Fix build trigger for tags

The tags we use don't have v in them.

Fixes for #494

Will need to manually kick of the github actions for tags 4.1.6 and 4.2.4

Going to try to delete and push tags to see if it will kick off builds

@jcormier you should simply first switch to on dispatch and manual dispatch the builds till we found a good pattern that would make less noise we could create a extra repo to test the builds and when that works simply copy paste.

I don't have permissions to trigger dispatch builds.  But that's a good idea, I could set up things in my fork of the project and test various configs.


Updated README.md with UP Squared Board setup

Signed-off-by: Yugansh Mohan<EMAIL_ADDRESS>

LGTM. I'll merge it once the build is done.


[feature request] Allow URL subpaths

Hi,
it could be very useful to allow to expose the following components in subpaths:

The web UI
The gRPC api
The REST API

Moreover, in the case of gRPC and Web UI, subpaths could allow to expose both on the same port.
Allowing subpaths is extremely useful for example when using k8s, and also when exposing behind a reverse proxy.
Regards

I agree this can be useful. Would you like to create a pull-request for this?


can't connect db in container (node 16)

Discussed in https://github.com/prisma/prisma/discussions/14970

Originally posted by Atsuyoshi-Funahashi August 24, 2022
Hi. I am currently creating an application in Next.js and using mysql for the DB.
I am building the environment with docker. Originally I was using node version 17.6.0 and it was working fine.
Problem
I have lowered my node version to 16.13.0 in order to install NextAuth.
After that, when I launch the container with docker compose up --build, the following error occurs during migration.
# error
Error: P1001: Can't reach database server at `mysql`:`3306`
Please make sure your database server is running at `mysql`:`3306`

Since this did not occur when node version was 17, I would like to know how to solve this problem.
Version
node  16.13.0
prisma 4.2.1
mysql 5.7

reproduction method
1  create next app
2 install and init prisma
3 Copy-paste the file below.
4 docker compose up --build

File
Dockerfile, docker-compose.yml, run.sh, and .env are as follows
## Dockerfile
FROM node:16.13.0

RUN mkdir /test
WORKDIR /test
COPY . /test
RUN yarn add create-next-app
COPY run.sh /usr/bin/run
RUN chmod +x /usr/bin/run
EXPOSE 3000

CMD ["run"]

## docker-compose.yml
version: '3'
services:
  db:
    image: mysql:5
    platform: linux/x86_64
    container_name: mysql
    ports:
      - '3306:3306'
    environment:
      - MYSQL_ALLOW_EMPTY_PASSWORD=yes
    volumes:
      - ./tmp/db:/var/lib/mysql
  app:
    build: .
    volumes:
      - .:/test
    ports:
      - '3000:3000'
    restart: always
    depends_on:
      - db

This is schema.prisma.
generator client {
  provider = "prisma-client-js"
}

datasource db {
  provider = "mysql"
  url      = env("DATABASE_URL")
}

model users {
  id               Int       @id @default(autoincrement())
  email            String    @unique
 # other value
}

This is seed.ts.
import { PrismaClient } from '@prisma/client';
const prisma = new PrismaClient();
async function main() {
    await prisma.users.createMany({
        data: [
            {
                email<EMAIL_ADDRESS>            },
            {
                email<EMAIL_ADDRESS>            },
        ],
    });
}

main()
    .then(async () => {
        await prisma.$disconnect();
    })
    .catch(async e => {
        console.error(e);
        await prisma.$disconnect();
        process.exit(1);
    });

This is run.sh. Please set under the project root.
# run.sh
#!/bin/sh

bash -c "sleep 30 && yarn -s run prisma migrate dev --name init && yarn install && yarn dev "

This is .env
# .env
MYSQL_HOST = 'mysql'
MYSQL_DATABASE = 'database_name'
# MYSQL_USER = 'root'
MYSQL_ROOT_PASSWORD = ''
DATABASE_URL="mysql://root:@mysql:3306/database_name"

```</div>

This looks like a docker networking issue. Try refrencing the docker service db directly in the .env file:
MYSQL_HOST = 'mysql'
MYSQL_DATABASE = 'database_name'
# MYSQL_USER = 'root'
MYSQL_ROOT_PASSWORD = ''
- DATABASE_URL="mysql://root:@mysql:3306/database_name"
+ DATABASE_URL="mysql://root:@db:3306/database_name"

I am going to close this because I don't see it as a bug, please look into docker networking for further info.

Thank you for your comment.
I've tried this change, however it doesn't works.

@Atsuyoshi-Funahashi @danstarns
I am experiencing the same problem.
I am using node:16.17.0 Docker container. 😢

I don't know if this is related, but I added connect_timeout and now I can connect stably.
DATABASE_URL="mysql://docker:docker@db:3306/docker?connect_timeout=30"

Same error here.
If i run the prisma steps (generate & migrate) directly in a package.json's script (after docker compose build), it works!
I think also that is an internal docker network error...

I don't know if this is related, but I added connect_timeout and now I can connect stably.
DATABASE_URL="mysql://docker:docker@db:3306/docker?connect_timeout=30"

What is the equivalent in PostgreSQL?


Create 1on1.md

Addresses: https://yusa.atlassian.net/browse/DS-296

Merged in order to not miss this @froboy


Make lnurl-auth with lndhub compatible to BlueWallet

BlueWallet is adding lnurl-auth support. \o/ BUT they are not using the same secret to create the key as we do.
We also use an Alby specific key (which was a mistake initially made)
To be compatible with BlueWallet, meaning that lnurl-auth with the same lndhub account in Alby and BlueWallet has the same result we have to update the way we create the key and signature.
Updating the code should be easy, but we need to think about a way to not break it for existing users. Existing users would need a way to login using the old mode.
Ideally this is a setting on the "website".
something like:
if website.useOldLnurlAuthMode {
  // use old key generation / signing method
} else { // this is the default then
  // use new key generation and signing method compatible with BlueWalet
}

Code in BlueWallet:
https://github.com/BlueWallet/BlueWallet/pull/4493/files#diff-8aaf0739679ead0a7c5e8cd819fad19bd972ccc1e8ef4dd641af75b7dd87c82cR672
-> https://github.com/BlueWallet/BlueWallet/pull/4493/files#diff-ec4c16453004f9b974f5659081ec5eafd3f4f5ae6a009fbc1bbe953a2de8b9f0R298-R306
used secret: https://github.com/BlueWallet/BlueWallet/blob/6f87a73a52ba648efa3cf33202bf965432b9054a/class/wallets/lightning-custodian-wallet.js#L113

BlueWallet only uses login and password: https://github.com/BlueWallet/BlueWallet/pull/4493#discussion_r812952383


Add Type Declaration Files for Terra Components

Issue Description
The support for TypeScript, and type checking in general, is becoming increasingly widespread across the JavaScript and ReactJS community, as it makes JS code way more predictable and less bug-prone. Currently, Terra components are not compatible with any consumers that wish to use TypeScript or static type checking. My team has begun implementing type checking in our code base because we recognize the benefits and enhancements it provides us (advanced autocomplete, in-line documentation, go to definition, auto import). However, our main roadblock has been the fact that Terra components do not have type declarations, so the TypeScript compiler cannot accurately assess the components.
Therefore, we propose that Terra add Type Declaration Files for all components in terra-core, terra-clinical, and terra-framework. Since the size of these libraries isn't too large, we don't foresee a major time commitment being required for this change.
For more information about type declaration files, please reference the following links:
https://blog.mariusschulz.com/2014/05/19/using-typescripts-type-definition-files-to-get-tooling-support-for-plain-javascript
https://definitelytyped.org/guides/creating.html
Issue Type

[x] New Feature
[x] Enhancement
[ ] Bug
[ ] Other

Expected Behavior
Each Terra component has a corresponding type declaration which consumers can access and use within their type-checked projects.

We've discussed this topic amongst the team and have found that type checking isn’t a problem that we feel needs to be solved for beyond what we're doing with react prop-types.
While there are benefits to adding TypeScript to Terra, there are also trade-offs that come with it. Here are a few concerns we have with TypeScript: potential lock-in as JavaScript evolves, worry about building up a large codebase using TypeScript, only to have the ECMAScript spec introduce conflicting keywords and type features, limiting available tools, increased build tool complexity / reduced build tool compatibility.
By using TypeScript, it limits the tools available to use. Instead of using ESLint, which has grown to become the de facto JavaScript linter, we'd have to use TSLint, which doesn't have the type of backing behind it compared to ESLint.
We've also found compatibility issues between TypeScript and Lerna, the tool we use to manage the monorepo.
An exerpt from https://medium.jonasbandi.net/here-is-why-you-might-not-want-to-use-typescript-50ab0d225bdd expands on the concern of lock-in with TypeScript.

When you start using TypeScript features (types, generics, enums, access modifiers …), your code does not run directly in a JavaScript runtime any more.
Your project becomes dependent on TypeScript. For a big code base written in TypeScript it would be a lot of work to get back to plain JavaScript.

The problem might arise in the long term. The current strength of TypeScript is that it is so closely aligned to JavaScript. This enables easy integration with the huge JavaScript ecosystem. Currently you can use TypeScript with almost every JavaScript library and framework. The TypeScript team clearly stated that this is also the goal for the future. This means new JavaScript features are also compatible with TypeScript.

However in a far future TypeScript might decide to deviate further away from JavaScript. This might impact the ease of integration with the JavaScript ecosystem a lot. Then TypeScript would just become another language that compiles to JavaScript, there are already a lot of those …

Another danger is that certain TypeScript features might clash with future JavaScript features and TypeScript has to change. This happened before with modules (I think TypeScript handled it gracefully, but there is still some friction in TypeScript aroundnamespace and module).

The next clash that is on the horizon is JavaScript introducing private fields.

Based on this, we do not have plans to add TypeScript to Terra.
While our team is not interested in adding TypeScript to Terra, you could add types for Terra components to your project. Here are a few resources for how that could be done:

https://www.detroitlabs.com/blog/2018/02/28/adding-custom-type-definitions-to-a-third-party-library/
https://medium.com/@chris_72272/migrating-to-typescript-write-a-declaration-file-for-a-third-party-npm-module-b1f75808ed2

Actually, the goal was never to get Terra to convert their code base to TypeScript. My team isn’t actually moving to TypeScript as well. The goal was to add type declarations via documentation, so that the code can still be type-checked without being converted to TypeScript.
The reason why we didn’t just add the type declarations ourselves is because it would be very difficult for us to keep up with every new Terra release and check through the components to see if any props have been added, removed, or changed. Other open source projects have a larger consumer community of TypeScript developers who consistently work together to keep the typings up to date, but I’m not sure this is the case with Terra currently. We believe it would be easier for the Terra team in charge of the release to quickly update the type declarations because they know exactly which props of which components will be changing as part of the release. With that being said, we could add the initial typings, and then the Terra team could assist in keeping them up-to-date.
I also wanted to address a few of the arguments that were brought up as negatives for TypeScript.
TypeScript has consistently added full support for new EcmaScript features by phase 3 of their development/release cycle. I believe Terra wouldn’t use the new features until phase 4. Also, the TypeScript team goes to the regular TC39 meetings about upcoming EcmaScript features, so they are major stakeholders and know the exact features that will be added in the future.  TypeScript now has major influence in the JavaScript community. It is actually interesting that a feature was proposed to EcmaScript and got turned down in the preceding TC39 meeting because it would clash with TypeScript. With that being said, the room for conflicts is very small due to the technical architecture of TypeScript. TypeScript is only the “declaration” while the plain JavaScript (the “implementation”) remains exactly the same.
The TypeScript team has stated that the entire premise of TypeScript and it’s purpose will always be to be a thin layer of static typing on top of JavaScript. The goal is to always be Javascript, but with types. If they diverge from that, it would be a completely different language than what it is right now which wouldn’t make sense.
The ESLint team created typescript-eslint-parser so it is not required to use TSLint at all. ESLint doesn’t work on JSX files anyways, so we have to use babel-eslint for JSX, and babel 7 has full support for TypeScript.
There are many projects and companies that use Lerna and TypeScript together. I’m not sure what compatibility issues were found.
These articles are good to read:
https://basarat.gitbooks.io/typescript/content/docs/why-typescript.html
https://eng.lyft.com/typescript-at-lyft-64f0702346ea
https://redditblog.com/2017/06/30/why-we-chose-typescript/
https://slack.engineering/typescript-at-slack-a81307fa288d

To me types vs no types comes down to personal opinion and what you're used to. I personally don't see a lot of value in adding type checking to dynamic languages, but I know that others do. The tipping point with this issue is that it would increase maintenance work and complexity in the terra library and thats not something we're willing to commit to.


App Transport Security Settings

Hi Avia team,
you forget add to the guide
.plist file
App Transport Security Settings
Cheers

Added, thanks!


refactor(aria_metadata): generate ARIA metadata from specification

Summary
Our ARIA metadata are currently handwritten and mostly based on ARIA 1.1 (some of them are in ARIA 1.2).
Some of these metadata are erroneous or incomplete.
I first started to update them manually. However, this took too much time and it is error-prone.
The aria-query NPM package provides a JSON file with the ARIA metadata.
This seems to be manually updated.
I also noticed several issues and some lack of fidelity with the specification (they mix related and base concepts for example).
Thus, I decided to write a script that extracts ARIA metadata directly from the specification.
The script is written in JavaScript and relies on happy-dom NPM package that parse the HTML page of the ARIA specification.
I chose happy-dom because it has less dependencies than jsdom and linkedom. It is also more used than linkedom.
This is a best effort approach, however the generated data seems pretty good (I tried for the three available versions of ARIA).
I placed the script in packages/aria-data.
I updated the biome_aria_metadata build.rs script to generate its code from aria-data.json.
For now, aria-data.json is symlinked to packages/aria-data/aria-data-1-2.json.
I used the prettyplease crate (widely used in the community) to format the output of build.rs.
I think it is a better alternative to starting a process to run rustfmt.
Now that we have a pretty reliable ARIA metadata source, we will be able to transfer more stuff from biome_aria to biome_aria_metadata.
Test Plan
CI must pass.

@ematipico I chose happy-dom because it has less dependency than linkedom and has more NPM downloads. Thanks for the suggestion!

A DOM library like happy-dom or jsdom is perfectly fine and gets the job done, but just wanted to make a note that for most scraping needs of mostly static content you don't really need a DOM, and you can get by with a simple XML/HTML parser like cheerio. This is normally at least some orders of magnitude faster, and there are some tools that simplify this further with a scraping-focused API like surgeon.
All of this said, if this is something that just needs to run manually every once in a while, a DOM library is perfectly fine. However, if there are plans to run this frequently (e.g. in CI for every PR), then I'd strongly suggest switching to a parser approach.

We will rarely run this job, considering that it's computed from source.

@DaniGuardiola
Thanks for the pointers.
Indeed, cheerio could be a better option.
I decided to keep happy-dom because it is good enough and as you said it is rarely run.
The script is there for reproducibility and to be able to extract data once ARIA 1.3 is stable.


Jwk part one

bors r+

bors r+

bors r+

bors r+

bors merge

bors retry

bors merge

bors ping

bors merge

bors cancel

bors r+

bors r+

bors r+

bors r+

bors r+


[bug]: LND rejecting message number 7?

Background
We've got at least one report of LND disconnecting when it received (unknown msg) 7: https://github.com/ElementsProject/lightning/issues/6065#issuecomment-1455172259
This happens when people turn on the experimental-peer-storage option in core lightning.
Your environment
admin ~ ฿ lncli getinfo
{
"version": "0.15.5-beta commit=v0.15.5-beta",
"commit_hash": "c0a09209782b1c62c3393fcea0844e095c25046b",
"identity_pubkey": "037f66e84e38fc2787d578599dfe1fcb7b71f9de4fb1e453c5ab85c05f5ce8c2e3",
"alias": "HODLmeTight",
Steps to reproduce
I can probably send one manually, though we are doing to change our code to only send to peers if they advertize the peer-backup feature for now.
Expected behaviour
Nothing, it's OK to be odd.
Actual behaviour
Disconnection and eventual unhappiness :(

Odd (no pun intended lol), we should just log that and keep chugging: https://github.com/lightningnetwork/lnd/blob/master/peer/brontide.go#L1484-L1491 and also have for some time. The log line is expected, but if that's what caused a disconnect, there should be another log below like EOF or w/e.
We should definitely not be disconnecting, but a feature bit could be used in theory to only send to those that know of the feature bit, which might also be useful in finding peers in the wild that support the peer storage feature as well.

Red herring then?  OK, something else is going on, thanks!


Mark testbot-raspberrypi tests as required

These are not required at the moment as they are not passing.


Option to "Show captions"

Hello,
I would like to set "Show captions" by default on init.
Is there any option that I'm missing?
Thanks in advance!

@srdjank Apologies for the slow response here.  We may be able to work through adding this in an upcoming release.
The idea would be that the window opened to the aside of the viewer could be set to a "Captions" tab by default?

Thanks for your response.
That would be great also, but I was thinking to set inside player option to show captions by default.


feat: added TryFrom<[u32; 4]> to Command/Transfer TRB structs

Added TryFrom<[u32; 4]> to Command/Transfer TRB structs with Rsvdz checks.

Could you update the branch to the latest main?

Thank you, and sorry, but rustfmt's check failed. Could you reformat the code?

Oh, I forgot that, sorry.
Now, I've done!

Thank you for your contributions. I'll release a new version soon.


for tensorflow API changes

I'd like to leave some fixes on tensorflow API changes, maybe v1.1.

Awesome! Thanks a lot! 👍


Add units array default value

This allows to omit the units array when defining a level without units.

Codecov Report

Merging #179 into master will decrease coverage by 0.08%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #179      +/-   ##
==========================================
- Coverage   92.35%   92.26%   -0.09%
==========================================
  Files          85       85
  Lines        1072     1073       +1
  Branches      179      180       +1
==========================================
  Hits          990      990
- Misses         67       68       +1
  Partials       15       15

Impacted Files
Coverage Δ

packages/warriorjs-core/src/loadLevel.js
89.28% <0%> (-3.31%)
: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 1b9d1df...9ccdc80. Read the comment docs.


scikit-image: init at 0.11.3

I believe it would be better to use dashes to unify regular and attribute names. I haven't changed anything (yet), as we have scikitlearn in there already...

@vcunat we actually discussed the naming a bit here at the sprint. Personally, I prefer to use lowercase everywhere and remove dashes and underscores. Others feel more for keeping the name exactly as how it is on PyPI, for several reasons.

Oh, I've missed the discussion then, unfortunately. We have this documented actually for general case http://nixos.org/nixpkgs/manual/#sec-package-naming. There might be good reasons for python stuff to have different guidelines, like compatibility with PyPI, I don't know (not really using python myself).


dbt-bigquery supports and enforces primary key constraints

Preview
What are you changing in this pull request and why?
@bnaul called out here that some documentation isn't in alignment with the source code (which asserts that dbt-bigquery supports and enforces primary key constraints).
Here are some other docs that should all be in alignment, but aren't:
Model contracts: Platform constraint support
https://docs.getdbt.com/docs/collaborate/govern/model-contracts#platform-constraint-support

constraints: Platform-specific support
https://docs.getdbt.com/reference/resource-properties/constraints#platform-specific-support

Checklist

[x] Review the Content style guide and About versioning so my content adheres to these guidelines.
[ ] I've checked that the preview renders correctly
[ ] I've checked that that code behaves as described

Primary (and foreign) keys are supported but not enforced in BQ, so I think you only want the first change

Primary (and foreign) keys are supported but not enforced in BQ, so I think you only want the first change

Thank you!
Our source code is currently asserting that they are enforced (?!), so I've reached out to our engineering team for clarification. Guessing that we'll want to update the source code to be this instead:
        ConstraintType.primary_key: ConstraintSupport.NOT_ENFORCED,
        ConstraintType.foreign_key: ConstraintSupport.NOT_ENFORCED,

Once that clarification comes in, will try to update all the different documentation pages and source code so that they are all in alignment.

ah interesting, they most definitely are not 😅


feat(Exception): ReThrow any exception encountered while getting the lock and ensure the releasing of consumer.

See #24
Checklist

[x] The pull request branch is in sync with latest commit on the amoerie/keyed-semaphores development branch
[x] I have included unit tests
[ ] I have updated CHANGELOG.MD
[x] I am listed in the CONTRIBUTORS.MD file

Description of changes in this pull request
Manage properly any exceptions thrown by the waiting of the lock and be sure to propagate them after we release the lock to be sure the lock is always released after usage.

Thanks again. I'll play around with this. :-)


Represent files as Path instead of str

Many GGShield functions work with files from the local filesystem. These functions could benefit from using Python pathlib to represent files as Path instead of str. The benefits would be the following:

make the code more self-explanatory: a Path argument type is more precise than an str
simplify code: reading and writing files is simpler with Path
potentially catch bugs

👋 hey @agateau-gg is that cool if I give this a go, please?

wave hey @agateau-gg is that cool if I give this a go, please?

Sure, go for it! 👍🏻

hey @agateau-gg I just want to make sure I'm on the right page before refactoring. Is this a good example of what you were referring to that should be change to the Path type from str? https://github.com/DataDavD/ggshield/blob/3a04c006698db80bdb0ff26dc63105e7dcb7e785/ggshield/core/config/utils.py#L26
Thanks in advance for verifying!

Yes, that's a good example :+1:

Hi,
Since @DataDavD's last message is three months old, I would like to give it a try if it's OK to everyone.
@agateau-gg is this first commit what you have in mind? let me know if I can contenue with the rest of the refactoring

Hi, Since @DataDavD's last message is three months old, I would like to give it a try if it's OK to everyone.

Hi, sincewe haven't heard from @DataDavD, you are welcome to give it a try 👍🏻.

@agateau-gg is this first commit what you have in mind? let me know if I can contenue with the rest of the refactoring

Your commit is a good start, but it's changing the signature of load_yaml_dict() without taking advantage of the fact that the function now receives a Path instance. You can change it to  use path.is_file() instead of os.path.isfile(), and path.open() instead of open(path).


Recent changes to indexer properties in spreads are causing pain

Proposal
I use the following pattern a lot in my code to combine old data with fresh new data
type Data = {
  [key: string]: number,
}

const oldData: Data = {
  "1": 100,
  "4": 200,
}

const newData: Data = {
  "1": 110,
  "3": 160,
}

const combinedData: Data = {
  ...oldData,
  ...newData
}

This code now gives the error
Cannot determine a type for object literal [1]. `Data` [2] cannot be spread because the indexer string [3] may overwrite properties with explicit keys in a way that Flow cannot track. Can you spread `Data` [2] first or remove the indexer?

Since all the objects have no explicit properties (they all use indexers), then the error flow gives off makes no sense. As far as i can see, there is no way to use this pattern without littering my code with $FlowFixMe's.
Flow should understand that since there are no explicit properties to overwrite, and every property is of the same type, then therefore multiple object spreads shouldn't cause an error.
Use case
https://flow.org/try/#0C4TwDgpgBAIghsOUC8UDeAoKUDaBrCEALigGdgAnASwDsBzAXRJoFcBbAIwgoBpt+B2DAF8MGAMYB7GuSiSANgBN4iEiqSpM2AEQBGbSV0AGIzyxRtAFgNQATCbOiJ02TQgB3dWoQb05vTa6xmY6AMyBAGymImJSMsBQUpy0EMo+3ogoftgAdHkKaYghUHk5bp4+IkA

As far as i can see, there is no way to use this pattern without littering my code with $FlowFixMe's.

I agree this sure looks like a bug, and I hope someone on the Flow team sees it and fixes it. I think it should be possible to do better than this for a workaround, though.
The strategy I'd recommend is to concentrate the fixmes in a small number of helper functions. For example:
const combineData = (old: Data, new: Data): Data => ({
  ...old, ...new,
}: $FlowFixMe);

Depending on how you use this pattern in your actual code, you might have several functions along this line, or you might make a more generic version, or a combination.
Separately, you might edit the title of this issue to be more specific to the particular problem you're seeing. That'll be helpful for any reader, and there's a possibility it will make it more likely that a member of the Flow team notices the report and looks into it.

This works in the latest version of Flow (look at your Try Flow link again)


micr tessdata

There is mcr.traineddata available already. Is it worth to include the same from the below link in the listed traineddata which will be useful
I have downloaded 'https://groups.google.com/forum/#!msg/tesseract-ocr/obWI4cz8rXg/9glPuTdRX5EJ'

traineddata provided by 3rd party should be maintained by 3rd party (there of outside of project repository)


Remove unnecessary HTML tags (Typo "[ipport=]"→"[ipport=]")

https://docs.microsoft.com/en-us/windows/win32/http/show-sslcert
https://github.com/MicrosoftDocs/win32/blob/docs/desktop-src/Http/show-sslcert.md
#PingMSFTDocs

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


change scss file names from style to the name of what it is

After discussing with @dmiller9911 we think that having the name of the component for the scss file would be beneficial. For example instead of button/styles.scss we would have pf-c-button.scss
This task is to change all the scss file names.

We decided to have the stylesheet name match the component name without the pf-[clud]- prefix, or match the main handlebars partial filename. So AboutModalBox/styles.scss changes to about-modal-box.scss


on up/down arrow, set the input only if data.val is a string

I'm not sure what to do with this fix; it looks like corejavascript/typeahead is no longer maintained (see Mar 28 2018 comment on https://github.com/corejavascript/typeahead.js/pull/181).
This uses my forked version of corejavascript/typeahead.js with an update to leaves input unchanged when navigating through object results with arrow keys: see https://github.com/corejavascript/typeahead.js/pull/184

@kielni  can you merge with master and commit again.  I think travis might start passing again.


{Misc} Remove some Python 2.7 legacy code

Description
Removed a bit of Python 2.7 code that I found by searching for ImportError and Python 2.7.
There are still quite a fair pieces of Python 2.7 compatibility around. sys.version_info  is a good place to start.
Note that I also saw a fair chunk of https://docs.python.org/3/library/datetime.html#datetime.datetime.utcnow in the code

This checklist is used to make sure that common guidelines for a pull request are followed.

[X] The PR title and description has followed the guideline in Submitting Pull Requests.

[X] I adhere to the Command Guidelines.

[X] I adhere to the Error Handling Guidelines.

Misc

/azp run

@wangzelin007 @jiasli I won't add more to this PR. _serialization.py is weird.
To summarize better what this PR contains:
I fixed some of the warnings from the pyupgrade section from ruff.
https://docs.astral.sh/ruff/rules/open-alias/
https://docs.astral.sh/ruff/rules/os-error-alias/
https://docs.astral.sh/ruff/rules/native-literals/
https://docs.astral.sh/ruff/rules/unnecessary-future-import/
Apart from that, I have grepped all over the code to remove ImportError for obvious Python 2 libraries; mostly that was urllib.
This one can also be fixed, but I didn't do so. There are two matches:
https://docs.astral.sh/ruff/rules/replace-stdout-stderr/
This one finds a few matches, but pylint doesn't complain since it was fixed. Maybe because those few instances are a bit special.
https://docs.astral.sh/ruff/rules/unicode-kind-prefix/

@atombrella, first, thanks for the great work. Do you mind creating separate PRs for each change, such as the IOError deprecation, so that we can easily review your PR? Thanks again!

@atombrella, first, thanks for the great work. Do you mind creating separate PRs for each change, such as the IOError deprecation, so that we can easily review your PR? Thanks again!

@jiasli :) I've opened some PRs that should be more granular in terms of changes.
I'll close this PR.


Non-reactive queries with useQuery

If you want to use TanStack Query polling etc. with a Convex query you should be able to — you can today with the http client, at least this should be documented.

This works for actions now, useQuery(convexAction(api.foo.bar, {})) is a good old non-reactive polling React Query thing.


Plunker demo not working

I reached the link:
https://plnkr.co/edit/JcG8uO9nIfSGMEKdLf0Y?p=preview (from the page https://www.npmjs.com/package/angular2-select)
and it is not working at all.
I fixed the main issue by changing the path to CDN.
In config.js:
...
packageNames.forEach(function(pkgName) {
var packageName = pkgName.substring(pkgName.indexOf('/') + 1);
packages[pkgName] = { main: '/bundles/' + packageName + '.umd.js', defaultExtension: 'js' };
});
...
See: https://plnkr.co/edit/o6PffQegv4BW6jUju5lA?p=preview

Plnkr is fixed.

Still not working or have you not updated readme wit latest link ?


Resource Pack not working

The resource pack isnt working so the dragons are just phantoms with no custom models nor custom texture pls help

You need a working LibsDisguises plugin for the dragons to look correct.
If this was a resource pack problem you'd see flying horse armor instead :)


"opengraph: false" erroring

Originally: https://github.com/haydenbleasel/gulp-favicons/issues/23

Hi,
Everything is now working beautifully, the only slight issue is when I disable the Open Graph image (which I need to handle independently to get the article image rather than a favicon image), but setting opengraph: false in the icons parameters causes the following error:

C:\Autumn Earth\Dropbox\autumn-earth\site\node_modules\favicons\helpers.js:253 Reflect.deleteProperty(request.favicon_design, 'open_g ^ ReferenceError: Reflect is not defined at Object.configure (C:\Autumn Earth\Dropbox\autumn-earth\site\node_modules\favicons\helpers.js:253:25)

Any idea? I can set the Twitter card image to false without any problem - it just seems to be the Open Graph that causes the issue.

...

I have run into the same issue. It only occurs when online is set to true. Any set of icons can be set to false, not just opengraph for the issue to occur. Running v5.1.1 of node.

CC: @johnholtripley @IanCaunce

Update: This happens by setting any icon to false with online: true. It spits out Reflect is not defined which is strange as we're running ES6 all over the place. Potentially related issue:

The cause is that after compilation step browserify wraps every source file into:

module.exports = function() {
  "use strict";
  (source file)
};

In other words it forces to use strict mode where this is undefined. To solve the issue I just used

(function() {
  var global = this || window;
  ...
})();

Added harmony-reflect polyfill, should be fixed in<EMAIL_ADDRESS>


Add empty data directive warning

Fixes #516
Also known as parameterless dX, this isn't normally intended for ROMs. Enabled by -Wall.

I'd reword the warning to "db/dw/dl directive without data", because as written right now, it seems like what is missing is a data directive, not data itself.
Otherwise, LGTM.

@aaaaaa123456789 Done.


[DFT][CMake] Fix inclusion of public include directories when linking against a DFT backend library

Description
This PR ensures that linking against backend such as target_link_library(myApp PRIVATE onemkl_dft_cufft correctly includes the oneMKL public include directories.
Fixes https://github.com/oneapi-src/oneMKL/issues/442
Checklist
All Submissions

[x] Do all unit tests pass locally? Attach a log.

onemklinteldft.txt
onemklrocmdft.txt
onemklcufftdft.txt
onemklcudadft_rebase.txt for after rebase, including CT example with MKLCPU DFT and CUFFT.
Tests build and run with portFFT, but some test cases failed. The changes in this PR should not be related to these problems.

[N/A] Have you formatted the code using clang-format?

Bug fixes

[ish] Have you added relevant regression tests?

The compile-time DFT example has been modified to depend on the described functionality.

[x] Have you included information on how to reproduce the issue (either in a
GitHub issue or in this PR)?

This affects only the build, so pinging @vmalia.

I'm told I pinged the wrong person - sorry vmalia! @mkrainiuk


Update zh_cn.json

The scope of the translation includes all commits about en_us.json from Oct 11, 2023 to Feb 4, 2024.
The number of zh_cn.json lines is strictly equal to en_us.json

To be honest I've been looking for your mod for a long time.
The GitHub repository name is CreateCasing.
The description of mcmeta is also CreateCasing.
Even mod.toml says the name of the mod is CreateCasing.

I even went to curseforge and looked for CreateCasing for a long time and couldn't find anything about it.

Which language is it?
The mod is named Create Encased because it sounds better than CreateCasing


Fix flaky test

need to add 1 to count the inode added in the @Before

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2157/
Test PASSed.


threads(n) get rid of that n!

Hi madison,
I am very excited of your python module. But can you change the code so that I can use threads() as decorator but not set an n. So the number of threads append dynamicly to the threadpool?
Thank you very much.

It seems hard because of ThreadPoolExecutor. If you don't set n in madisonmay's program it raises TypeError, and if you don't set it in ThreadPoolExecutor it dafaults to 1. I'll keep an eye on this, is a small and powerfull mini-module.

You can simulate the "number of threads append dynamicly to the threadpool" behavior by using a very high n (it's passed through as the max_workers argument to ThreadPoolExecutor.__init__()).  However, I don't think it makes sense to behave this way by default.

could async_add = threads(n)(add)  help?
python rookie