AttributeError: Number of inputs is not equal 1 for slice layer

Hi @nerox8664!
I tried to convert an ONNX model to Keras, but I received the following error
Traceback (most recent call last):
  File "onnx2Keras.py", line 6, in <module>
    k_model = onnx_to_keras(onnx_model, ['input/input_data'],name_policy='renumerate')
  File "...\onnx2keras-env\lib\site-packages\onnx2keras\converter.py", line 174, in onnx_to_keras
    keras_names
  File "...\onnx2keras-env\lib\site-packages\onnx2keras\reshape_layers.py", line 264, in convert_slice
    raise AttributeError('Number of inputs is not equal 1 for slice layer')
AttributeError: Number of inputs is not equal 1 for slice layer

I converted this model  https://github.com/YunYang1994/tensorflow-yolov3 from Tensorflow to ONNX and now I need the model in Keras.
You can access the ONNX model here: https://drive.google.com/file/d/1ad3EDCILJf1xq8z0J39ZC1LAMAzI9zxV/view?usp=sharing
Thanks!

Hi @nerox8664 @emanuelcovaci !
I also tried to convert the same ONNX model to Keras and I received exactly the same error. The slice layer received 4 inputs, three of which seem to be weights.
Thanks!

Hello @emanuelcovaci @andreimihalcea97.
I fixed some errors related to slice layer. It now works with 4 inputs (with assumption that axes are [0, 1, ... ndims] array).
Unfortunatelly, @emanuelcovaci, your model still can't be converted due to ONNX Resize layer

@nerox8664 The onnx-tensorflow has a resize layer implementation at https://github.com/onnx/onnx-tensorflow/blob/master/onnx_tf/handlers/backend/resize.py
Can you use this and add it to onnx2keras?


Add Swagger support to enable easier development of backend apis

User Story

As a developer
I can use swagger-ui to access backend endpoint easily
So that it is easier to do backend development

DIscussion
You may use this PR from proj-ucsb-courses-search as a roadmap to know what to do
https://github.com/ucsb-cs156-s21/proj-ucsb-courses-search/pull/224/files
ACs

[ ] There is a URL that can be accessed e.g. /swagger-ui/index.html to bring up the backend swagger web page
[ ] We can interact with backend apis through that web page.

Estimate at 10 points


Breadcrumbs lead to 404s if an ancestor dir has no index.md page

Someone reported this issue to me about the page at https://handbook.sourcegraph.com/strategy-goals/strategy:

Btw, funny thing but the handbook sometimes drops an extra “/strategy/“ in the link urls for that section, thus sometimes returning 404s while navigating back and forth.

I repro'd it on these links (red arrow in screenshot). (I am confirming with the person that this is what they saw, too.)

In this case, it leads to https://handbook.sourcegraph.com/strategy-goals/strategy/use_cases, which 404s because there is no index.md document in that directory.

Similar issue:

Start from "Working with use cases" (tried direct url and from handbook search)
Click on the link "use cases" right on the first paragraph
Click on any use case on the list, they all have the extra "/strategy/" on the url.
404

More over, if you copy+paste the "use cases" link from step 2 on an incognito window straight away, you get the same 404 following on the links for each use case.
https://user-images.githubusercontent.com/119822/152061885-c491adde-4dc7-42e3-bc9e-07796395488c.mov

@rafaelrebolleda Thank you!


Broken dependencies when running "npm i"

npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found: @typescript-eslint/parser@undefined
npm ERR! node_modules/@typescript-eslint/parser
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| ^6.0.0-alpha" from<EMAIL_ADDRESS>npm ERR! node_modules/@typescript-eslint/eslint-plugin
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See /home/runner/.npm/eresolve-report.txt for a full report.

npm ERR! A complete log of this run can be found in:
npm ERR!     `/home/runner/.npm/_logs/2024-01-06T03_40_21_920Z-debug-0.log```

Thanks for reporting this. I'll take a look and see what I can do. For the meantime, feel free to run --force to download the dependencies and run the bot.

I found a way around, just delete all the dependencies and reinstall them, I think the versions are a little old.


Pass project version to frank-doc

Could you compare to the master into this PR? The current head of master already contains this addition

Done. @gvanbrakel you are right that this change was already in. Now there is no change left in this PR.


ModelClass_Table gives error

Hey guys,nice job on DbFlow..Am new at this and wanted to try it out..After creating my Model as LastMessage and wanted to perform a query, Android studio marks LastMessage_Table as error..What could be wrong? Am using DBFlow 4.0.0-beta5 .
Here is my model.
` @Table(database = MessagesDatabase.class)
public class LastMessage extends BaseModel {
@Column
@PrimaryKey
int id;

@Column
String recipient;

@Column
String messageType;

@Column
String messageTextBody;

@Column
Date timeStamp;

public void setRecipient(String recipient) {
    this.recipient = recipient;
}

public void setMessageType(String messageType) {
    this.messageType = messageType;
}

public void setMessageTextBody(String messageTextBody) {
    this.messageTextBody = messageTextBody;
}

}
Here I tried getting a single lastmessage for a given recipient
LastMessage lastMessage = SQLite.select().from(LastMessage.class).where(LastMessage_Table.recipient.is("u898981")).querySingle();
Android studio marks  LastMessage_Table as error. Any help would be greatly appreciated. All the same thanks for this awesome library.

I assume the error says "Could not find reference LastMessage_Table" or similar, right?
Did you import the table class like import my.package. LastMessage_Table;?
Did you try to Invalidate the cache of Android Studio. Helped me already a few times. (File -> Invalidate caches -> Invalidate and Restart)

yeah make sure you build project first and then try it. If not, ensure incremental compilation is off.


Remove once the Attestation Credentials storage mechanism is updated.

false

https://github.com/project-chip/connectedhomeip/blob/f0f5de0d79ae1c51d7fddbf70900ebac0acd899a/src/credentials/examples/DeviceAttestationCredsExample.cpp#L23-L28

This issue was generated by todo-issue based on a TODO comment in f0f5de0d79ae1c51d7fddbf70900ebac0acd899a.

Hi Evgeni - this TODO came from your PR https://github.com/project-chip/connectedhomeip/pull/12568. Is it needed for 1.0? If not please remove the label


Sample feedback: control-sra-batching/js

Isn't there a call to googletag.display(adUnits) missing?

Should we use a call to googletag.display(adUnits)  before calling refresh?

Hi @renebaudisch ,
Thanks for clarifying. As you can tell from the fact that the sample works, calling display before refresh isn't technically required. However, doing so is considered best practice, as mentioned in the reference documentation for the refresh method:

For proper behavior across all browsers, calling refresh must be preceded by a call to display the ad slot. If the call to display is omitted, refresh may behave unexpectedly.

I'm double checking with the team here to make sure that warning is still accurate, and will update either the documentation or the sample accordingly.


[BEAM-2899] Move the Job Server into a Submodule

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

This permits separation of the Job Server and the Java Runner which
submits to it, while both being part of the 'ReferenceRunner'

retest this please

retest this please

R: @kennknowles

Coverage increased (+0.003%) to 69.591% when pulling 19861ecd4b2d05a223f3eb4cd6f6aa9f2d74461c on tgroh:reference_runner_module into 1001cf1ab7d3382e862374da5e4cade303528956 on apache:master.

Seems reasonable. I guess I just hope that the dependency for a user is reasonable and the main entry point is simple to understand, when it gets to that point.

My plan is to name the java PipelineRunner 'beam-runners-reference', which is basically as easy as it is possible to be.

LGTM


console never open

i tried to embed this in my mainwindow
from pyqtconsole.console import PythonConsole
app = QApplication([])
console = PythonConsole()
console.show()
console.eval_in_thread()
#console.eval_in_queue
it won't open. any idea?

Need more info on this.

what is your OS / python version / pyqtconsole version?
what output do you get in the terminal?
what framework is your main window written in?
can you give more context for how you insert the code?
does one of the examples work if you execute them standalone (without the rest of your application)?

From your description it's hard to say without further context. If I were to guess, I can see two potential problems:

if you inserted the above snippet as is into some other widget, one problem could be that you're creating a second QApplication here, which you should not do
if you're doing something different, do you have some place that processes messages (app.exec_())? message processing is needed to show windows


Site navigation missing

I posted about this in an Owning a Home pull request, but wanted to log an issue. The site-wide navigation for Sheer sites appears to be missing.

Fixed!


Sd3 batch parallel

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: chang-wenbin:x: “chang-wenbin”“chang-wenbin” seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Rule 20.12: Improve performance

Description
This commit optimizes the from-where-select clause of this query, which was highlighted as one of the slowest predicates for our C query suites.
This commit makes the following changes:

Avoid repetition of m.getParameter(i), which caused a cross product
Change the way we identify "further expanded macro"
Modify the isExcluded predicate to reference the macro invocation not the macro.

@rvermeulen I think you wrote this rule originally, so I would appreciate a review. In particular, the change from
  exists(Macro furtherExpandedMacro |
    mi.getUnexpandedArgument(i).matches(furtherExpandedMacro.getName() + "%")
  )

to
not mi.getUnexpandedArgument(i) = mi.getExpandedArgument(i)

Achieves what you originally intended.
@mbaluda @jsinglet this addresses one of the slowest performing predicates that was highlighted in https://github.com/github/codeql-coding-standards/pull/226, although the slow performance is related to the upgrade itself.
Change request type

[ ] Release or process automation (GitHub workflows, internal scripts)
[ ] Internal documentation
[ ] External documentation
[x] Query files (.ql, .qll, .qls or unit tests)
[ ] External scripts (analysis report or other code shipped as part of a release)

Rules with added or modified queries

[ ] No rules added
[ ] Queries have been added for the following rules:

rule number here

[x] Queries have been modified for the following rules:

Rule 20.12

Release change checklist
A change note (development_handbook.md#change-notes) is required for any pull request which modifies:

The structure or layout of the release artifacts.
The evaluation performance (memory, execution time) of an existing query.
The results of an existing query in any circumstance.

If you are only adding new rule queries, a change note is not required.
Author: Is a change note required?

[x] Yes
[ ] No

🚨🚨🚨
Reviewer: Confirm that format of shared queries (not the .qll file, the .ql file that imports it) is valid by running them within VS Code.

[ ] Confirmed

Reviewer: Confirm that either a change note is not required or the change note is required and has been added.

[ ] Confirmed

Query development review checklist
For PRs that add new queries or modify existing queries, the following checklist should be completed by both the author and reviewer:
Author

[ ] Have all the relevant rule package description files been checked in?
[ ] Have you verified that the metadata properties of each new query is set appropriately?
[ ] Do all the unit tests contain both "COMPLIANT" and "NON_COMPLIANT" cases?
[ ] Are the alert messages properly formatted and consistent with the style guide?
[ ] Have you run the queries on OpenPilot and verified that the performance and results are acceptable?As a rule of thumb, predicates specific to the query should take no more than 1 minute, and for simple queries be under 10 seconds. If this is not the case, this should be highlighted and agreed in the code review process.
[ ] Does the query have an appropriate level of in-query comments/documentation?
[ ] Have you considered/identified possible edge cases?
[ ] Does the query not reinvent features in the standard library?
[ ] Can the query be simplified further (not golfed!)

Reviewer

[ ] Have all the relevant rule package description files been checked in?
[ ] Have you verified that the metadata properties of each new query is set appropriately?
[ ] Do all the unit tests contain both "COMPLIANT" and "NON_COMPLIANT" cases?
[ ] Are the alert messages properly formatted and consistent with the style guide?
[ ] Have you run the queries on OpenPilot and verified that the performance and results are acceptable?As a rule of thumb, predicates specific to the query should take no more than 1 minute, and for simple queries be under 10 seconds. If this is not the case, this should be highlighted and agreed in the code review process.
[ ] Does the query have an appropriate level of in-query comments/documentation?
[ ] Have you considered/identified possible edge cases?
[ ] Does the query not reinvent features in the standard library?
[ ] Can the query be simplified further (not golfed!)

🤖 Beep Boop! gcc/c/x86_64 Matrix Testing for this PR has been completed. See below for the results!

PACKAGE              : Preprocessor2
TEST_PASS            : True
SUITE                : MISRA-C-2012
COMPILE_ERROR_OUTPUT :
QUERY                : MacroParameterUsedAsHashOperand
COMPILE_PASS         : True
TEST_DIFFERENCE      :
RULE                 : RULE-20-12

🤖 Beep Boop! clang/cpp/x86_64 Matrix Testing for this PR has been completed but I didn't find anything to test!

🤖 Beep Boop! gcc/cpp/x86_64 Matrix Testing for this PR has been completed but I didn't find anything to test!

🤖 Beep Boop! clang/c/x86_64 Matrix Testing for this PR has been completed. See below for the results!

TEST_DIFFERENCE      :
QUERY                : MacroParameterUsedAsHashOperand
SUITE                : MISRA-C-2012
COMPILE_PASS         : True
TEST_PASS            : True
RULE                 : RULE-20-12
PACKAGE              : Preprocessor2
COMPILE_ERROR_OUTPUT :

🤖 Beep Boop! Matrix Testing for this PR has been completed. If no reports were posted it means this PR does not contain things that need matrix testing!

Updated the issue body - I meant that the poor performance was not related to the upgrade!


Port-Forwarding does not keep connection alive

Describe the bug
Port-Forwardings get broken after a few minutes if there is no traffic.
To Reproduce
Steps to reproduce the behavior:

Start port-forwarding of HTTP service
Open browser and ensure it's working
Relead browser tab after 5 minutes or so

Expected behavior
Port-Forward is kept alive or is automatically recreated
Environment (please complete the following information):

Lens Version: 3.5.0
OS: Win 10 x64 Prof 1819

(respectful ping)

Running into this as well, also not clear how to recover from this beyond restarting Lens. Clicking the service links tries to reuse the broken ports


updated .gitignore file + support for ios-nocodesign-9-2

As in the title. At your disposal for further details.

Thanks, merged. Toolchain table updated: https://github.com/ruslo/polly/commit/c343d0ad3fe7810c3482c84e2a8b1e8ce8685861?diff=unified


Update to v1.6.7

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Looks like this is still failing with the same errors as the other PR. I was hoping the newer compilers would fix this, but evidently not. I'm not sure what the solution is

@synapticarbors The kernel library is too old. Nothing new compilers or a newer libc can fix.


Using partial templates

Hi,
I was wondering if it's possible to use partial templates from within a template.
The following documentation indicates that it should be possible with ejs.  I suspect my path might be incorrect. I'd appreciate it if you could point me in the correct direction.
File Structure:

Hello.ejs.t

I am running the following in the generation folder:
hygen c-sharp efcore-context new --basePath "app/contect" --fileName "Test1"
I have also tried:
<%- include('_templates/c-sharp/efcore-context/new/partials/user', {name: "Laredo"}) %>
and <%- include('partials/user', {name: "Laredo"}) %>

I found a stack overflow reply which said that one needs to use the full path in Ubuntu. Still investigating.
<%- include('/home/laredo/dev/code-gen/generation/_templates/c-sharp/efcore-context/new/partials/user.ejs.t') -%>
This worked for me, but it isn't ideal.


Fix injection point in BasicAuthSecurityFilter for tomcat7

@mswiderski @ederign recent change/refactoring switched to constructor injection in this filter which doesn't work on tomcat7. so, this is switching back to field injection.

+1 and apologies for all this trouble :(

@mswiderski no worries, I blame CDI :)

master: aa0ac06185fcbcc10fc3b6af9a82c3682ca6bdf4


Feature request: disable tab

Hi!
Thanks for the project, finding this handy!
Have a usecase when I need to disable tab from set of tabs on some condition so interested if this is a decent usecase/feature request.

Need it too.
Thx

Just ended passing a className from Tabs to resulting dom https://github.com/sudodoki/react-simpletabs/commit/9a4bf7c98a19ec8b29d1df2120c719cef2168d83#diff-be3075e36834673d63173d3497c17afcR93


Warn if the root filesystem is too small

Add a warning if the root filesystem is too small (https://github.com/coreos/fedora-coreos-tracker/issues/586), either because the underlying storage is explicitly specified, or because the root partition is constrained by an additional partition.

We've decided to defer this until we have https://github.com/coreos/fedora-coreos-tracker/issues/759 because otherwise it's impossible in many cases for Butane to know that it's looking at the root disk. There are some cases where it could (e.g. if root reprovisioning/mirroring is requested), but the common case of just adding a "trapping partition"  wouldn't be detectable without some dubious heuristics.

This issue might belong in ignition? Similar to https://github.com/coreos/ignition/issues/1397#issuecomment-1154390522
With this change https://github.com/coreos/fedora-coreos-config/pull/1621

Ignition is distro-independent and doesn't know the partition layout constraints of the OS it runs on, so this needs to go in Butane in the fcos variant.

Add a warning if the root filesystem is too small (coreos/fedora-coreos-tracker#586), either because the underlying storage is explicitly specified, or because the root partition is constrained by an additional partition.

I wanted to ask a clarifying question, do we consider root constrained if there is a partition in front of it aka /var on number 5 ?
or is it only constrained if the filesystem has been specified and there is a partition in front of it ?
In speaking with @jlebon it is fairly common ask to have a /var partition after root for us.

It's common to have a /var partition after the root partition, but we should only warn if that would make the root partition too small.  In sum, the values we need to check are:

The sizeMiB of the root partition, if specified
The startMiB of any following partition.  If this is unspecified (0) and the root sizeMiB is also unspecified, then the following partition will prevent the root partition from growing at all, and we should warn.


couldn't determine metrics port from configuration

Hi Team,
we have installed ADOT add-on EKS with neccessary service role and  instrumented our nodejs applications with otel.
we are trying to use aws x-ray for collecting traces and deployed  collector-xray as deployment.
But we are getting below operator logs.
{"level":"error","ts":"2023-07-24T05:28:22Z","logger":"controllers.OpenTelemetryCollector","msg":"Cannot create liveness probe.","opentelemetrycollector":{"name":"my-collector-xray","namespace":"amazon-cloudwatch"},"error":"service property in the configuration doesn't contain extensions","stacktrace":"github.com/open-telemetry/opentelemetry-operator/pkg/collector.Container\n\t/workspace/pkg/collector/container.go:127\ngithub.com/open-telemetry/opentelemetry-operator/pkg/collector.Deployment\n\t/workspace/pkg/collector/deployment.go:55\ngithub.com/open-telemetry/opentelemetry-operator/pkg/collector/reconcile.Deployments\n\t/workspace/pkg/collector/reconcile/deployment.go:39\ngithub.com/open-telemetry/opentelemetry-operator/controllers.(*OpenTelemetryCollectorReconciler).RunTasks\n\t/workspace/controllers/opentelemetrycollector_controller.go:223\ngithub.com/open-telemetry/opentelemetry-operator/controllers.(*OpenTelemetryCollectorReconciler).Reconcile\n\t/workspace/controllers/opentelemetrycollector_controller.go:211\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.15.0/pkg/internal/controller/controller.go:118\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.15.0/pkg/internal/controller/controller.go:314\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.15.0/pkg/internal/controller/controller.go:265\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.15.0/pkg/internal/controller/controller.go:226"}
{"level":"info","ts":"2023-07-24T05:28:22Z","logger":"controllers.OpenTelemetryCollector","msg":"couldn't determine metrics port from configuration, using 8888 default value","opentelemetrycollector":{"name":"my-collector-xray","namespace":"amazon-cloudwatch"},"error":"missing port in address"}

This is the collector configuration
`apiVersion: opentelemetry.io/v1alpha1
kind: OpenTelemetryCollector
metadata:
name: my-collector-xray
spec:
mode: deployment
resources:
requests:
cpu: "1"
limits:
cpu: "1"
serviceAccount: adot-collector
config: |
receivers:
otlp:
protocols:
grpc:
endpoint: <IP_ADDRESS>:4317
http:
endpoint: <IP_ADDRESS>:4318
processors:
  batch/traces:
    timeout: 1s
    send_batch_size: 50

exporters:
  awsxray:
    region: ap-south-1

service:
  pipelines:
    traces:
      receivers: [otlp]
      processors: [batch/traces]
      exporters: [awsxray]

Please help us.`

this is collector config file. Please guid us.
`apiVersion: opentelemetry.io/v1alpha1
kind: OpenTelemetryCollector
metadata:
name: my-collector-xray
spec:
mode: deployment
resources:
requests:
cpu: "1"
limits:
cpu: "1"
serviceAccount: adot-collector
config: |
receivers:
otlp:
protocols:
grpc:
endpoint: <IP_ADDRESS>:4317
http:
endpoint: <IP_ADDRESS>:4318
processors:
  batch/traces:
    timeout: 1s
    send_batch_size: 50

exporters:
  awsxray:
    region: ap-south-1

service:
  pipelines:
    traces:
      receivers: [otlp]
      processors: [batch/traces]
      exporters: [awsxray]

`


Added integration tests and travis config for deploying the app.

I have configured travis to just run 1 job at a time for this repo.

@lesv Can you take a look?

LGTM


Build a harness for web platform tests.

This imports results from a few major browsers so we don't waste time
trying to pass tests that we shouldn't necessarily pass.

The diff of this PR crashes mobile Chrome. Hah.
On Sat, May 2, 2015, 2:52 PM Jesse Wilson<EMAIL_ADDRESS>wrote:

swankjesse wants to merge 1 commit into master from
jwilson_0502_web_platform_tests_ha:
This imports results from a few major browsers so we don't waste time
trying to pass tests that we shouldn't necessarily pass.
You can view, comment on, or merge this pull request online at:
https://github.com/square/okhttp/pull/1600
Commit Summary

Build a harness for web platform tests.

File Changes

M okhttp-tests/pom.xml
https://github.com/square/okhttp/pull/1600/files#diff-0 (5)
A
okhttp-tests/src/test/java/com/squareup/okhttp/WebPlatformTestRun.java
https://github.com/square/okhttp/pull/1600/files#diff-1 (77)
A
okhttp-tests/src/test/java/com/squareup/okhttp/WebPlatformUrlTest.java
https://github.com/square/okhttp/pull/1600/files#diff-2 (152)
A
okhttp-tests/src/test/java/com/squareup/okhttp/WebPlatformUrlTestData.java
https://github.com/square/okhttp/pull/1600/files#diff-3 (155)
A
okhttp-tests/src/test/resources/web-platform-test-results-url-chrome-42.0.json
https://github.com/square/okhttp/pull/1600/files#diff-4 (1341)
A
okhttp-tests/src/test/resources/web-platform-test-results-url-firefox-37.0.json
https://github.com/square/okhttp/pull/1600/files#diff-5 (1341)
A
okhttp-tests/src/test/resources/web-platform-test-results-url-safari-7.1.json
https://github.com/square/okhttp/pull/1600/files#diff-6 (1341)
A okhttp-tests/src/test/resources/web-platform-test-urltestdata.txt
https://github.com/square/okhttp/pull/1600/files#diff-7 (342)
M pom.xml https://github.com/square/okhttp/pull/1600/files#diff-8
(2)

Patch Links:

https://github.com/square/okhttp/pull/1600.patch
https://github.com/square/okhttp/pull/1600.diff

—
Reply to this email directly or view it on GitHub
https://github.com/square/okhttp/pull/1600.

LGTM. Looks like SPDY3 tests are flaking?

Yeah, flaky tests are (sadly) expected. @nfuller made things a bit more strict recently.

Working towards https://github.com/square/okhttp/issues/1486


update to latest main and add signal-dashboard endpoints

@krivard the only relevant new changes are the two files in  src/server/endpoints. The rest is just a sync

cc @benjaminysmith
but also @sgratzl see switch to counts in https://github.com/cmu-delphi/delphi-epidata/pull/458
you two should combine your efforts

but also @sgratzl see switch to counts in #458

done thx for checking this

Thanks Sam! I think it's fine if we get out of sync with API changes for now given that we are rapidly iterating. I think a good first step is getting the Flask API for the dash endpoint up in some form, and then we can sync stuff over.

@sgratzl I'm good to merge this now unless you want to keep accumulating?


Creation of dynamic property anlutro\LaravelSettings\SaveMiddleware::$settings is deprecated

Started getting the following warnings in the logger.

Creation of dynamic property anlutro\LaravelSettings\SaveMiddleware::$settings is deprecated in /vendor/anlutro/l4-settings/src/SaveMiddleware.php on line 33

PHP Version: 8.3
Laravel Settings Version: 1.4.0

thanks for reporting! I'll release this in 1.4.1 :)


Fix uninitialized constant

The test failure occured in #49 test:
🔗 https://github.com/masutaka/compare_linker/actions/runs/10349442581/job/28644229198

An error occurred while loading ./spec/lib/compare_linker/formatter/markdown_spec.rb.
Failure/Error: require 'factory_bot'
NameError:
uninitialized constant #Class:ActiveSupport::Delegation::Inflector
# ./vendor/bundle/ruby/3.3.0/gems/activesupport-7.2.0/lib/active_support/delegation.rb:47:in `generate'
# ./vendor/bundle/ruby/3.3.0/gems/activesupport-7.2.0/lib/active_support/core_ext/module/delegation.rb:161:in `delegate'
# ./vendor/bundle/ruby/3.3.0/gems/factory_bot-6.4.6/lib/factory_bot/definition_hierarchy.rb:3:in `class:DefinitionHierarchy'
# ./vendor/bundle/ruby/3.3.0/gems/factory_bot-6.4.6/lib/factory_bot/definition_hierarchy.rb:2:in `module:FactoryBot'
# ./vendor/bundle/ruby/3.3.0/gems/factory_bot-6.4.6/lib/factory_bot/definition_hierarchy.rb:1:in `<top (required)>'
# ./vendor/bundle/ruby/3.3.0/gems/factory_bot-6.4.6/lib/factory_bot.rb:8:in `<top (required)>'
# ./spec/spec_helper.rb:2:in `<top (required)>'
# ./spec/lib/compare_linker/formatter/markdown_spec.rb:1:in `<top (required)>'

But it does not reproduce in the local environment.

But it does not reproduce in the local environment.

After deleting Gemfile.lock, the problem was reproduced in the local environment.


Move viglesiasce to emeritus

Thank you for everything!!
More info here:
https://lists.cncf.io/g/cncf-helm/topic/stepping_down_as_an_org/82949900

Thank you for all your hard work and contributions to Helm over the years, @viglesiasce! Wishing you and your new family all the best.


[CT-431] Support for Pyspark driver

Describe the feature
Add a forth connection option to the dbt-adapter. This forth connection would create a pyspark context and utilize the spark.sql(<sql>) function to execute sql statements.
Describe alternatives you've considered
The alternative is to run your own thrift server which is difficult to setup.
Who will this benefit?
A pyspark context gives you control over the spark application configuration and amount of resources to use in the spark cluster. It also enables the user to register custom pyspark UDF functions and to create custom views based on pyspark Dataframes.

control spark job creation
register custom pyspark UDF
register custom views based on pypsark Dataframes

Are you interested in contributing this feature?
Yes for sure. I have an implementation of this feature which I would like to contribute to this project.
Here's how it works from the user's point of view.
Specify a new connection method pyspark
profiles.yml
project1:
  target: dev
  outputs:
    dev:
      type: spark
      method: pyspark
      python_module: spark.spark_context

The python_module points to a python file found in the PYTHONPATH. In this case spark/spark_context.py. The pyspark adapter will call the create_spark_context() function found in this file. The user can thus create their own spark context (either a local instance or one that leverages a cluster). This hook also lets you create custom pyspark UDF registrations.
spark/spark_context.py
def create_spark_context():
    return SparkSession.builder.getOrCreate()

A second hook is available in the the sources. This hook lets the user register custom pyspark Dataframe as an sql view df.createOrReplaceTempView(<view name>). The user is free to create any Dataframe logic they need.
sources.yml
sources:
  - name: custom
    tables:
    - name: raw_users
      meta:
        python_module: models.staging.raw_users

The python_module is loaded the same way except that the hook function has a different signature.
def create_dataframe(spark, start_time, end_time)
Using pyspark you can work around the limitations of SparkSQL when reading datafiles. For example using pyspark you can load the schema of the json you are going to read. Thus avoiding sparks schema discovery process and making sure that the data is read in the schema you want.
models/staging/raw_users.py
def create_dataframe(spark, start_time, end_time):
    data = < load schema from a file>
    schema = StructType.fromJson(json.loads(data))

    df = (spark
	.read
	.schema(schema)
	.format("json")
	.load(f"{WAREHOUSE}/{YEAR}/{MONTH}/{DAY}/users/*.logs")
	)
    return df

The implementation of this feature consist of a new PysparkConnectionWrapper which executes sql statements via the spark context
def execute(self, sql, bindings=None):
    self.result = self.spark.sql(sql)

And a new method on the SparkRelation which register pyspark dataframes as sql views.
def load_python_module(self, start_time, end_time):
    path = self.meta.get('python_module')
    module = importlib.import_module(path)
    create_dataframe = getattr(module, "create_dataframe")
    df = create_dataframe(spark, start_time, end_time)
    df.createOrReplaceTempView(self.identifier)

Registration of pyspark views is initiated by a modified source macro
{% macro source(source_name, identifier, start_dt = None, end_dt = None) %}
    {%- set relation = builtins.source(source_name, identifier) -%}
    {%- if execute and (relation.source_meta.python_module or relation.meta.python_module) -%}

        {%- do relation.load_python_module(start_dt, end_dt) -%}

        {%- do return(relation.identifier) -%}
    {% else -%}
        {%- do return(relation) -%}
    {% endif -%}
{% endmacro %}

created a pull request https://github.com/dbt-labs/dbt-spark/pull/308

@cccs-jc Really interesting! Thanks for opening the issue, and the accompanying PR.
We just merged a new connection method (session) that can leverage a PySpark session (#279). I'd be very curious to get your eyes on that implementation (and @JCZuurmond's eyes on yours, in #308). What are the crucial differences between the two approaches? For my part, I'm not convinced that dbt's role should be to manage the Spark session's instantiation/configuration, versus simply being able to connect to one that already exists.
As always, the questions are:

Which generally-applicable guardrails do we want to keep in place, (separation of concerns between dbt + database "infrastructure")
Where does Spark blur those boundaries, in a way that gets us more than what's available elsewhere (all-on-one-machine "unit" testing of macro/materialization/etc functionality)

models/staging/raw_users.py

Neat :) We're definitely thinking about what it might look like to implement "dataframe" support in dbt + Spark/Databricks. I don't think a dbt-managed pyspark is session is the way we'd want to go, though.

@JCZuurmond's implementation is very close to mine. The main differences are that mine has these additional features
A hook to create custom spark session

In this hook you can create a spark context which specifies RAM/CPU to use from a cluster, configure iceberg catalogs etc
In this hook you can also register custom UDF which can be use in SQL transformations

The only way I know how to register UDF via SQL requires the python package to be installed in a pre-existing server which is very inconvenient.

A hook to register a custom view

This hook lets you register a pyspark dataframe as a dbt source. This lets you work around the limitations of SparkSQL. For example using pyspark you can load the schema of the json you are going to read. Thus avoiding spark's schema discovery process and making sure that the data is read in the schema you want.

Creating views in SQL (dbt-externals) is possible but it has limitations which the programmatic approach does not.

Being able to create custom python UDF and views is important to us. Curious to know what alternatives you are thinking about?

@cccs-jc : Great minds think alike! 💯

A hook to create custom spark session.

You solve this by first creating the Spark session with the config and configuration that you prefer. Then, SparkSession.builder.getOrCreate() will get this context for you. There is no hook needed.

A hook to register a custom view

Why not do this outside of dbt or right before you call dbt programmatically with the Spark session approach. The example you give about the json schema is solved by using dbt external tables.
If you have improvements to the current implementation, feel free to add changes to the session module!

Our initial proof of concept used the approach you mention. That is a modified dbt launch script which creates a spark context and registers views. The in the dbt adapter retrieve the session and execute sql.
The issues with this approach are:

You need a modified dbt launch script so if you want to use the vscode-power-dbt extension you need a modified dbt-core package.
Since dbt-adapter retrieves the spark session and is already aware of spark sessions why not have it use a hook to create one.
Registering views can take a bit of time. When you have a few dozen views and you launch dbt for a specific model why register these views. Perhaps the model you run does not even use any of the views. Seems to make more sense to create the views as they are needed. Thus the hook into the source processing.

I'm aware of dbt-external tables however it's not as flexible as pyspark code. We built pyspark libraries to create dataframe out of our raw data (hundreds of tables) and would like to leverage them within dbt.
The improvement I would like to contribute is hooks into spark session creation and view creation.

Hi @cccs-jc , excuse me for the late reply. I want to help you with solving your issue whilst finding a way to not add too specific code the code base. Note that I am not able to approve PRs, that is up to the dbt folks.
spark session creation
There are other ways to configure your Spark session. You could do this on the cluster level, e.g. when using Databricks. Or in spark-defaults.conf.
Will that help you?
Hooks
I like dbt hooks, but adding hooks to one connection method is confusing -> Why only for the Spark session connection? Also, I prefer an API that is agnostic to what the hook is about, i.e. not create_dataframe but run_pre_hook. I created an issue for this on dbt core.
Will that help you?
dbt-external-tables
I understand that pySpark feels more flexible. Though, I think you can implement a lot in Jinja. I have not tested Azure Kusto, you might be able to translate the example in the docs to Spark SQL.
dbt-external-tables is pushing the limits of dbt already. dbt is a SQL based transformation tool, the fundamental assumption is that there are source models present to transform. This is generally true in database world, but in data warehouse world the assumption is slightly different: data is stored at various locations (e.g. files or Azure kusto) w.o. having the necessary source tables referencing the data. When using dbt-spark I recommend you to thoroughly think about how you create your source data models. dbt-external-tables helps, it might not cover all your cases, I do not always use it, I consider to use something else than dbt to create the sources.

We do not close the Spark session. I think we should add this to the Cursor.close

@cccs-jc : Do you want to contribute closing the Spark connection? And will that solve your questions?

I don't have the time to contribute this at the moment. Does it resolve my questions yes and no. I can work around the limitation of the spark context creation but still like to be able to create "views" so  hook in the source creation is still of interest.
For now I'll keep our fork of dbt-spark which includes these hooks. Maybe these will become available in a more general form in dbt core as a pre-hook or something similar.

Hi all. Is the spark session still not closed? I could potentially help out with this.
@JCZuurmond In the cursor class, is the dataframe that's used and its associated spark session safe to close when the cursor is closed?
e.g. when Cursor.close() or the cursor's __exit__ function is called, is that the end of that spark session life cycle or is it still needed?

Oh  nevermind I wasn't aware that https://github.com/dbt-labs/dbt-spark/pull/308 would potentially resolve this. Let me know if I'm wrong.
Otherwise, the active spark session can be easily pulled off of the dataframe itself (though I don't know if it's necessarily safe to close it right away).

@kbendick : I do not expect that we will merge #308 , if you want you could add the closing of the cursor. I think it makes sense to close the session in the Cursor.close. @jtcohen6 : Are the connections generally closed on the cursor or on the connection?

@kbendick : I do not expect that we will merge #308 , if you want you could add the closing of the cursor. I think it makes sense to close the session in the Cursor.close. @jtcohen6 : Are the connections generally closed on the cursor or on the connection?

It really depends if the Spark session associated with the dataframe inside of the cursor is going to be used any more once the cursor is exhausted / closed.
If there’s tests I can just open a PR and we’ll see?

Sounds like a plan!
And the data frame is not used outside the class. I think it fits dbt's standard workflow: after a cursor is closed not call any methods on the cursor anymore.

Sorry for the late reply on this.
Got side tracked for a bit. I will go ahead and open a PR just trying to close the Spark Session when the cursor is closed.
Given how shared the Spak Session is (typical applications have one Spark Session throughout their whole lifetime but I can't say for sure yet about dbt-spark), so I'm not sure if this approach is really going to work, but i think it is worth investigating and giving a try.

If Spark 2.x weren't supported, we could simply use SparkSession.getActiveSession() but that method was added in Spark 3.0.0.
I'm correct in my understanding that Spark 2.4.x is supported still?

@kbendick We are planning to add functionality in a forthcoming version of dbt-spark that will require Spark v3 (https://github.com/dbt-labs/dbt-spark/pull/342), if we can unblock ourselves in the upgrade for CI. I see you put a comment on https://github.com/dbt-labs/dbt-spark/pull/349#issuecomment-1157245193 offering to help, which is much appreciated :)

@kbendick : Are you still interested in creating the PR?

I don't really have time at the moment. I thought the close cursor was added already..?


Bedpe revisions (WIP)

This is just to reveal some in-progress changes in and around the coordinate transformations between VCF -> BEDPE -> VCF. This will need to be integrated with testing changes before being able to merge.

@davidlmorton - I manually edited the expected files in vcftobedpe. Mirroring them over after that for bedpetovcf was not terribly successful. In the process of conversion, INFO fields are reordered, the POS field is added, QUALS are converted to floats, reference alleles are changed to N and alt alleles are simplified to match the SVType of the variant. I believe you had a strategy to get around this that involved multiple conversions. We may need to do that. I filed issues for things that seemed questionable.

@abelhj - I am curious if the errors we're suddenly seeing on Travis (libgfortran.so.1: cannot open shared object file: No such file or directory) are the same problems you were running into when getting your classifier tests up and running?
@davidlmorton - Given the difficulty we're having in getting those to run. I'm thinking perhaps we should disable the test and get this merged in. Then we could have another PR specifically to try to fix the tests. If you have time this morning, I'd love to go over this PR with you and talk about the test changes I made last night as well as possible improvements to what I have here.

This pull request is large and tests are failing on Travis. Local tests are passing so I am going to merge.


CIV-10126 Set Up Lip Claim Issue After Payment

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/CIV-10126
Change description
Set Up LiP Claim Issue After Payment
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

Could you please put a screenshot of the Camunda diagram with your changes?


Documentation for Application Insights and Phone Factor technical profiles

Can you please add documentation for the Application Insights and Phone Factor technical profiles?
These technical profiles are in common use but it's difficult to find a complete description for them.

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

ID: fe70bb3e-884f-a8db-eee7-ae98cf653b93
Version Independent ID: d8647921-91df-9c37-8cd9-d44ce938dd9d
Content: About technical profiles in Azure Active Directory B2C custom policies
Content Source: articles/active-directory-b2c/technical-profiles-overview.md
Service: active-directory
GitHub Login: @davidmu1
Microsoft Alias: davidmu

@chrispadgettlivecom Thanks for the feedback! We are currently investigating and will update you shortly.

@chrispadgettlivecom Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

A workitem has been opened in the documentation queue to track the addition of this content. Because there is no change required to the referenced article, please close.
#please-close


Switch button firing change event 3 times in BS 4.5.0

I have tried this issue under version 4.5.0 and the method seems working but the change event is fired 3 times on each button click
https://codepen.io/DK568/pen/MWYxWMO
try to click the button
Originally posted by @dkwebtec in https://github.com/twbs/bootstrap/issues/30077#issuecomment-636015174

@Lausselloic is this the same issue as #30849?

it's not the same issue but yes event is propagate to childrens inside the label. I could give a try to handle that

#31000 targets v4-dev, but what about this change in v5?

@Lausselloic can you confirm please that this issue does not affect v5? I believe we removed most of the Button JS so this doesn't seem to apply to v5.

Sorry for the late reply, that doesn't affect v5 at time, there's a need to backport @patrickhlauke fixes (https://github.com/twbs/bootstrap/pull/28952 and https://github.com/twbs/bootstrap/pull/28834) those accessibility fixes are not present's in v5 I will give a try to port it

Fixed by #31000, thanks @Lausselloic!


Unused activity_main.xml layout file

The file 'android-CardView/Application/src/main/res/layout/activity_main.xml' is unused in the application. I believe it can be taken out to simplify the layout folder and possibly decrease confusion for some new developers.

Thanks for your report - the file is indeed obsolete. (It is automatically included as part of the release process.) We'll look at removing it in the future.

(Internal: 19085565)


Feature: Photo Uploads

Implements a working test component for photo uploads: UI, Redux and backend integration.

Code looks good, ready to add it in.

Code looks good, ready to add it in.


Bug: Hub Network Name in Complete design - inherits the "root-id" from the YAML

Is there an existing issue for this?

[x] I have searched the existing issues

Infrastructure as Code Type? (Required)
terraform
PowerShell Module Version (Optional)
7.2
Bootstrap Module Version (Optional)
4.1.4
Starter Module? (Required)
terraform - complete
Starter Module Version (Optional)
4.5.0
Input arguments of the ALZ-PowerShell-Module (Optional)
No response
Debug Output/Panic Output (Optional)

Expected Behaviour (Required)
When editing the "complete" configuration, the usage of the name element for the Hub Virtual Network appears to be overlooked
  configure_connectivity_resources:
    settings:
      hub_networks:
        - config:
            address_space: ["<IP_ADDRESS>/24"]
            location: ${starter_location}
            name: vnet-customhubname-region-01
            resource_group_name: rg-customrgname-region-01

Actual Behaviour (Required)
the hub virtual network is created with the Root ID element prepended, "mg-root" for example, and the remaining naming convention "-hub-northeurope" for example.
Other configurations, config-hub-and-spoke-vnet-multi-region.yaml for example, use a different set of elements. Is one preferened to the other?
Steps to Reproduce (Optional)
No response
Important Factoids (Optional)
No response
References (Optional)
No response

Hi @faolain-git. Thanks for the feedback. We are focussing our efforts on the Azure Verified Modules starter module, which will contain all the functionality in the other starter modules. As such we are unlikely to fix this at this time.
If it is important and urgent for you, we'd be happy to review a PR.
Thanks

Thank you Jared.

Hi @faolain-git. Thanks for the feedback. We are focussing our efforts on the Azure Verified Modules starter module, which will contain all the functionality in the other starter modules. As such we are unlikely to fix this at this time.
If it is important and urgent for you, we'd be happy to review a PR.
Thanks


Enable batch_size > 1 for multi_training

Hello,
this change allows for training of the temporal transformer with batch sizes of larger than one.
When trying to use larger batch sizes before, it threw a shape mismatch exception in the temporal transformer code section.
I mainly changed the shapes from e.g. [15, 1120, 256] for the configuration with 14 reference frames to
e.g. [2,15, 1120, 256] before the transformer. The transformer input is then e.g. [2, 4200, 10] etc.

Hi thank you for your code.
Did you also change TransVOD++ code for batch size greater than 1?


FIX: ASB-2176 - Dynamic link in nav bar - Relevant representations (registration comments)

Describe your changes
https://pins-ds.atlassian.net/browse/ASB-2176?atlOrigin=eyJpIjoiNTMwYTdiZGMxODM0NDRlYTgyMmMyODIzNmUyNWJkYmQiLCJwIjoiaiJ9
Type of change 🧩

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Other (please explain in the description section above)

Checklist before requesting a review

[ ] I have performed a self-review of my own code
[ ] I have double checked this work does not include any hardcoded secrets or passwords
[ ] I have made corresponding changes to the documentation
[ ] I have provided details on how I have tested my code
[ ] I have referenced the ticket number above
[ ] I have provided a description of the ticket
[ ] I have included unit tests to cover any testable code changes

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

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

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


Assign ID to headers with CustomHTMLRenderer

I used ToastUI's customHTMLRenderer to assign an id attribute to all headers equal to the sanitized version of the text of that header.
a simplified version of this can be found in the customHTMLRenderer docs linked above, but my version is a little more extensive in the way it sanitizes the text as it removes all non-alphanumerical characters. Here are some examples of the sanitization:

"$$!!£!HeADinG \\1!  !" -> "heading-1"
"Slightly more 'normal' heading 2" -> "slightly-more-normal-heading-2"
you can test this yourself with this jsfiddle

the sanitized text is then set as the header's id attribute, which allows for linking in markdown, and therefore tables of content and header tagging (#34). This is very similar, in fact practically the exact same, as how it is done in github, as shown below:
[link to section 1](#section-1) links to header with text that sanitizes to "section-1"
so for instance # Section 1 or ### SECTION 1!
The use of customHTMLRenderer is a really powerful addition, as it allows to add any parameters that you want to add in the future, like latex support (#61)
I tested this on my homeserver which runs in docker and everything ran smoothly and as expected. Perhaps some more testing could be done to see if the sanitizer works as intended for all titles, but because flatnotes is selfhosted this doesnt create any security flaw and as it stands now this solution works for everyday users as most people dont need fancy symbols in their headers.

Hey @pbogre, this looks great! As soon as I get a chance I'll have a look and try and get it merged in. It would be a great addition.

was just testing this now and reinstalled the image, but i got a console error saying that customHTMLRenderer wasn't defined correctly. Silly me. the latest commit fixes that and has also been tested by declaring customHTMLRenderer inside the data function.


Jackson compatibility

Ensure compatibility with jackson versions 2.6.x, 2.7.x and 2.8.x and spring versions 4.1.x, 4.2.x and 4.3.x

👍

:+1:


Game engine hangs in scene with StaticBody2D with a WorldBoundaryShape2D

Describe the bug
When using the Rapier2D in a scene containing a StaticBody2D with a CollisionShape2D whose shape property is set to a WorldBoundaryShape2D, the game hangs upon loading the scene.
To Reproduce

Create a new Godot project.
Install Rapier2D from the AssetLib and make it the physics engine for the project.
Add a StaticBody2D with a CollisionShape2D child whose shape is set to WorldBoundaryShape2D
Launch the scene.

Expected behavior
The game shouldn't hang / freeze.
Environment (please complete the following information):

OS: Mac OS
Version Sonoma 14.4.1
Godot Version v4.2.1.stable.official [b09f793f5[
Type SIMD

Example project(zip)
WorldBoundary2DHang.zip

Can confirm same issue on Windows 10, Godot 4.2.2.stable, using the same engine (Rapier2D with SIMD).

Thanks for the report, will investigate.

Oh my, it's because of the fluid pipeline. It seems that the world boundary being so big, it generates some weird shape or something when coupling with fluids.

I'll open an issue on salva repo for now and disabling world space for fluids.

Hm, looking at this, it might occur for bigger shapes too. Right now I register all shapes with fluid pipeline for all shapes using dynamic contact sampling. There should be a way for us to specify what objects should integrate with fluid pipeline and which not, but I also like the simplicity of now where it all just works. Hm.

Thank you for the quick fix! I was wondering if it had to do with the volume of the body, cool to confirm. Out of curiosity, is there an easy way to install the main branch version into my godot project?

Copy paste the addons folder into your project. Thats the only way, afaik.

Eg. here is also described: https://github.com/appsinacup/godot-rapier-2d?tab=readme-ov-file#installation


Tune some kernel configs for aarch64

Remove CONFIG_NF_CONNTRACK_PROCFS
Remove CONFIG_TRACE_IRQFLAGS
Remove CONFIG_TRACE_IRQFLAGS_NMI
Remove CONFIG_IRQSOFF_TRACER
Remove CONFIG_PREEMPTIRQ_TRACEPOINTS

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] If you are adding/removing a .spec file that has multiple-versions supported, please add @microsoft/cbl-mariner-multi-package-reviewers team as reviewer (Eg. golang has 2 versions 1.18, 1.21+)
[x] Ready to merge

Summary
Tune some configs for performance on aarch64.
Change Log

Remove CONFIG_NF_CONNTRACK_PROCFS
Remove CONFIG_TRACE_IRQFLAGS
Remove CONFIG_TRACE_IRQFLAGS_NMI
Remove CONFIG_IRQSOFF_TRACER
Remove CONFIG_PREEMPTIRQ_TRACEPOINTS

Does this affect the toolchain?
YES due to change in kernel-headers package version.
Associated issues
N/A
Links to CVEs
N/A
Test Methodology

Pipeline build id: 557110

@jslobodzian @rlmenge Let's revert that change for now.

[like]  Rachel Menge reacted to your message:

From: Eric Desrochers @.>
Sent: Tuesday, May 7, 2024 4:48:59 PM
To: microsoft/azurelinux @.>
Cc: Rachel Menge @.>; Mention @.>
Subject: Re: [microsoft/azurelinux] Tune some kernel configs for aarch64 (PR #8899)
@jslobodzianhttps://github.com/jslobodzian @rlmengehttps://github.com/rlmenge as discussed let's revert that change for now.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/azurelinux/pull/8899#issuecomment-2098889963, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACOY4VSQVOH7QDXCDNZT643ZBEAXXAVCNFSM6AAAAABGXZ5NYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJYHA4DSOJWGM.
You are receiving this because you were mentioned.Message ID: @.***>


Audio visualizer only works for left sound channel

Expected behavior:
Audio visualizer supposed be responding for all sound channels
Actual behavior:
Only left channel triggers visualizer,sounds from other channel was not reflected
How often does this problem occur?
This problem occurs every time
What is the version of Aurora you are experiencing this issue with?
My version is beta0.6.3-488
Does the previous version of Aurora have this issue?
No idea

Fixed thanks to #1282


Wrong permissions attached to SQS queue subscribed to a SNS topic

I have a SQS queue that is subscribed to two different SNS topics. Here's a small sample of my code:
    const admissionFormSubmittedTopic = new sst.aws.SnsTopic(
      'AdmissionFormSubmitted'
    )
    admissionFormSubmittedTopic.subscribeQueue(
      queues.CreditAnalysisRequested.arn
    )
    admissionFormSubmittedTopic.subscribeQueue(
      queues.UpsertPatientOnCelcoinRequested.arn
    )
    admissionFormSubmittedTopic.subscribeQueue(
      queues.UpsertPatientOnCelcashRequested.arn
    )
    const guarantorAdmissionFormSubmittedTopic = new sst.aws.SnsTopic(
      'GuarantorAdmissionFormSubmitted'
    )
    guarantorAdmissionFormSubmittedTopic.subscribeQueue(
      queues.GuarantorCreditAnalysisRequested.arn
    )
    guarantorAdmissionFormSubmittedTopic.subscribeQueue(
      queues.UpsertPatientOnCelcoinRequested.arn
    )

The problem is that the permissions of the "UpsertPatientOnCelcashRequested" queue are wrong. Here's what I have on the AWS Console:
{
  "Version": "2012-10-17",
  "Statement": [
    {
      "Effect": "Allow",
      "Principal": {
        "Service": "sns.amazonaws.com"
      },
      "Action": "sqs:SendMessage",
      "Resource": "arn:aws:sqs:sa-east-1:476738463198:doutor-sim-prod-UpsertPatientOnCelcoinRequestedQueue",
      "Condition": {
        "ArnEquals": {
          "aws:SourceArn": "arn:aws:sns:sa-east-1:476738463198:doutor-sim-prod-GuarantorAdmissionFormSubmittedTopic"
        }
      }
    }
  ]
}

So the messages that arrives on the "AdmissionFormSubmitted" topic are not being sent to this queue. This is currently breaking my app.
Can someone help me with this? Thanks.

I'm not following. How is UpsertPatientOnCelcashRequested created? And what permissions were you expecting it to have?

yeah I see your error. It's caused by each subscribe() call try to update the queue's policy.
Let me think about the fix.


Frozen bootup after freebsd logo on usb

After created an USB boot, its boot until trminated the freebsd logo, when mouse is loaded on screen the bootup get Frozen AND does not work anymore, my hw:
Laptop amd ryzen 7, Nvidia optimus (Radeon 10 AND Nvidia gtx1650), 32gb ram AND m.2 disk

Hello @MatiasCarter. Could you please try with ghostbsd.org and see whether that works?


"Failed - Server problem" when trying to download

Hello,
I'm following [this meteorpad] (http://meteorpad.com/pad/2hjNqmwHjDvkxvLC5/Leaderboard), which has been very helpful. My application has a pretty similar logic, an export button that must be downloaded performing some operations when building the sheet.
Although it works flawlessly in your example (it does download), when I port to my app I'm gettin "Failed - Server Problem" during the download. I do get the local path, for instance: /gridfs/temporaryFiles/164f1128a50093f5f43ada8f
I'm kinda lost if I'm missing some package or initial configuration. I don't have GridFS, for example, only those required in the pad.
What could I be missing here?
PS: I'm using Google Chrome (download doesn't work even in your example when running on Firefox. But this isn't a concern right now).

Can you share your code in any way and i'll have a look?
I know about the issue with Firefox, it has to do with the way I initiate the download which is not really cross-browser, but i never actually got to finding the best way to do it.

This is my method (server.js):
Meteor.methods({
	downloadExcelFile: function() {
		var Future = Npm.require('fibers/future');
    	var futureResponse = new Future();

    	var excel = new Excel('xlsx'); // Create an excel object  for the file you want (xlsx or xls)
    	var workbook = excel.createWorkbook(); // Create a workbook (equivalent of an excel file)
    	var worksheet = excel.createWorksheet(); // Create a worksheet to be added to the workbook

	    worksheet.writeToCell(0,0, 'Apontamentos');
	    worksheet.mergeCells(0,0,0,1); // Example : merging files
	    worksheet.writeToCell(1,0, 'Nome');
	    worksheet.writeToCell(1,1, 'Produção');

	    var row = 2;
	    Apontamentos.find().forEach(function(a) {
	    	console.log(a);
	    	worksheet.writeToCell(row, 0, a.safrista_id);
	    	worksheet.writeToCell(row, 1, a.production);
	    	row++;
	    });

	    workbook.addSheet("Apontamentos", worksheet);

	    mkdirp('tmp', Meteor.bindEnvironment(function (err) {
	    	if (err) {
        		console.log('Error creating tmp dir', err);
        		futureResponse.throw(err);
      		}
      		else {
        		var uuid = UUID.v4();
        		var filePath = './tmp/' + uuid;
        		workbook.writeToFile(filePath);

        		temporaryFiles.importFile(filePath, {
          			filename : uuid,
          			contentType: 'application/octet-stream'
        		}, function(err, file) {
          			if (err) {
            			futureResponse.throw(err);
          			}
          			else {
            			futureResponse.return('/gridfs/temporaryFiles/' + file._id);
          			}
        		});
      		}
    	}));

    	return futureResponse.wait();
	}
});

This is my both.js:
temporaryFiles = new FileCollection('temporaryFiles',
  { resumable: false,   // Enable built-in resumable.js upload support
    http: [
      { method: 'get',
        path: '/:_id',  // this will be at route "/gridfs/temporaryFiles/:_id"
        lookup: function (params) {  // uses express style url params
          return { _id: params._id};       // a query mapping url to myFiles
        }
      },
      { method: 'post',
        path: '/:_id',
        lookup: function (params) {
          return {
            _id: params._id
          }
        }}
    ]
  }
);

And in my template:
Template.safristaPopover.events({
	"click #export": function() {
		Meteor.call('downloadExcelFile', function(err, fileUrl) {
      		console.log(fileUrl);
      		var link = document.createElement("a");
      		link.download = 'Apontamentos.xlsx';
      		link.href = fileUrl;
      		link.click();
    	});
	}
});

I had the same error and fixed it by adding the temporaryFiles.allow part from the same meteorpad.
temporaryFiles.allow({
    insert: function( userId, file ) {
      return true;
    },
    remove: function( userId, file ) {
      return true;
    },
    read: function( userId, file ) {
      return true;
    },
    write: function( userId, file, fields ) {
      return true;
    }
  });

@rafaelquintanilha Did @skastr0 answer help you?

Hi everyone,
I am having the same problem here. I am seeing the excel file here:
/path/to/.meteor/local/build/programs/server/tmp/c05db38b-52ed-49fa-b5dd-0f0cac2e8ea2

but nothing is being downloaded. There is no /gridfs/temporaryFiles anywhere though - is that possibly part of my problem? I assumed that isn't a real filesystem path, but rather a database reference to the file in the tmp path shown above.
I have the temporaryFiles.allow all set up. I even tried adding back in insecure and autopublish, but it made no difference.
Thanks for any suggestions on how to fix.

If you have the temporaryFiles.allow set up and the name of the excel file I assume opening an url
/gridfs/temporaryFiles/c05db38b-52ed-49fa-b5dd-0f0cac2e8ea2 should work.
If not then maybe it's something with the way you create the fileCollection? Or try adding ?filename=.xls(x) to the url.

Sorry about that - as usual, you're just about to give, you make a comment on an issue, then figure it out - it was a problem with how I was calling the download method from the client.
It case it helps someone else, it is critical to use this call the downloadExcelMethod from the client like this. I just overlooked how the server method relied on link.x items to appropriately trigger the download.
    Meteor.call('downloadExcelFile', function(err, fileUrl) {
      console.log(fileUrl);
      var link = document.createElement("a");
      link.download = 'Players leaderboard.xlsx';
      link.href = fileUrl;
      link.click();
    });

Thanks for a great tool!


multi-input multi-output regression

Hello,
thanks for this nice way to interface fortran and keras!
I have a question about running a multi-input (or even single-input) and multi-output model. In this case, should I use an ensemble?
For example:
input = temperature
output = [concentration1, ..., concentration100]
What to do in this case? Also, I do need any average of the results when using ensembles. And also "Put the names of the model files" what does it mean? It assumes that my original network has been already splitted in many single outputs?
Thank you,
Kind regards,
Lorenzo Campoli

Can you clarify what you mean by multi-output? This could mean two different things:

If you mean your output vector is of length 100, this will be taken care of by the conversion.
Your output has multiple branches

See here for details on the difference and how FKB handles them
For ensembles first you must convert the keras files to Fortran format using h5_to_txt. Then you put the names of those txt files in a single file which is then provided to the ensemble. This way it knows which networks you want in the ensemble.

Hello,
in my case is (1), the output vector has length 100. So, in this case ensembles are not necessary.
Thank you.


How to freeze some heads for finetuning on different datasets?

For example, I want to freeze the classification and detection heads and only train the mask head.

This is the same as what you do to freeze some pytorch modules.

@hellock I know how to do it in normal PyTorch script. But in mmdetection, there is a lot of encapsulation and I am wondering if there is any easy way to do so?
Do I have to modify tools/train.py?

Currently there are no easy way to do so and you need to modify the optimizer (and/or the model).

@yxchng Hi, have you solve the issue? Could you offer some specific instructions?


[Bug]: Repeated answer history problems occur when using Agent

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Branch name
main
Commit ID
c3b2a1
Other environment information
No response
Actual behavior
When I have a conversation using the default WebSearch Assistant templates, previous questions are answered repeatedly

Expected behavior
No response
Steps to reproduce
The parameters are not changed, and the prompt word uses the default prompt word.

Additional information
No response

It depends on the NLP capability of LLM.

thx,question

It depends on the NLP capability of LLM.

What size model are you using? Because this situation occurred when I used deepseek-chat in the demo, and a similar situation occurred when using qwen2-7b-instruct locally.

gpt-4


Updated dependency friendsofphp/php-cs-fixer

Questions
Answers

Description?
Updated dependency friendsofphp/php-cs-fixer

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?
N/A

How to test?
All is :green_circle:

Thank you @Progi1984


PG Json functions

Will this support row_to_json, json_build_object, json_agg etc in the future?

Hey, potentially it is possible if I can query PG json field's schema (I believe mysql8 supports that as well)
Not sure if we can workout advanced functions like json agg. I can take a look


Update pl-PL.json

Coverage increased (+4.7%) to 92.162% when pulling 64f0c0e22114ed3b69b40518bab90359aff6a74d on Kun2210:master into 6c2c57af470c168ec3a792dd5288a26db3e575ee on MarshallOfSound:master.


Compiler error with Tuples

Severity	Code	Description	Project	File	Line	Suppression State
Error	CS8137	Cannot define a class or member that utilizes tuples because the compiler required type 'System.Runtime.CompilerServices.TupleElementNamesAttribute' cannot be found. Are you missing a reference?	Parquet(net451), Parquet(netstandard1.6)	C:\Users\richa\Documents\Projects\parquet-dotnet\src\Parquet\File\Page.cs	84	Active

Hi @azurecoder try to do nuget restore on the solution. System.ValueTuple comes as extra nuget package for C# 7.

Ah, perfect, told you I was old school!


Adds --max-workers flag to the exec-entrypoint helm command

Description of the change:
Adds --max-workers flag to the exec-entrypoint helm command
Motivation for the change:
Currently the helm operator runs with just 1 worker. Reconciling helm releases can sometime take >1 second, using a single worker will not be able to keep up for more than 60 CRs of the same type.

Hi @estroz,

Max concurrent reconciler configuration applies to controllers in general, so this change should apply to the Ansible entrypoint as well.

Are you saying that it should be done for Ansible too? Am I right? So, this implementation already exist for Ansible. See: https://github.com/operator-framework/operator-sdk/blob/master/pkg/ansible/controller/controller.go#L92. And it is in the AnsibleFlags which means that is done for the entrypoint and run --local commands. See: https://github.com/operator-framework/operator-sdk/blob/master/pkg/ansible/flags/flag.go#L29

Happy Monday everyone. Thanks for looking at this @estroz . As @camilamacedo86 was mentioning, the feature already exists for Ansible. I made sure to use the same arg name to keep it consistent. Let me know if there is anything else I could improve on this PR.

Yep, I added this initially to the Ansible side. @razvan-wework thanks great work.


slice plugin: add check to transaction status

Don't allow the slice plugin to intercept the transaction if a prior plugin has changed the http transaction status to something non zero.

Cherry-picked to v9.0.x branch.

This has merge conflicts, and we'll need a new 8.1.x PR. Also, feel free to recommend / suggest other PRs that this depends on, such that it makes the back porting easier.


Seems to be limited to 100 instead of 1000?

Describe the bug
This is possibly not a PRAW bug, but I can't think of anywhere else to ask about it. I recently re-ran my program to delete blacklisted comments after not having done so far a while, and noticed that it refuses to give me comments past a limit of 100. I have my normal limit set to 500 in my config, and that used to work; it would count through my last 500 comments.
However, this time, I'm getting lines in my log which show it's getting the right limit from my config, but it only counts to 100, as evidenced by  the second log in Code/Logs. My code (also in Code/Logs) is unchanged from before, so nothing should be running any different.
Yet the program only goes back to a maximum of 100 comments. If I set it to less than 100, say 50, it works perfectly and counts only that 50, so it's being limited to 100 somehow.
To Reproduce
Steps to reproduce the behaviour (I don't know if it will be reproduced, I have no clue why it's happening):

Initialise Reddit instance
Call comments with limit=(any number greater than 100)
It only counts 100

Expected behaviour
I'd expect it to count through all 500, as it used to.
Code/Logs
2021-01-01 01:12:28 - 75/500 comments checked successfully. shows it's getting the right limit from the config
2021-01-01 01:12:35 - Counted this cycle: 100 shows it's not counting right
for comment in reddit.redditor(gvars.config["user"]).comments.new(limit=gvars.config["limit"]): is where it counts
doLog(f"{counted}/{1000 if gvars.config['limit'] == None else gvars.config['limit']} comments checked successfully.", gvars) produces the first log message
System Info

OS: Slackware 14.2, Linux Kernel 4.4.217
Python: 3.7
PRAW Version: 7.1.0

Have you deleted comments in such a way in the past? If so, it's possible there are no more comments available to delete. By available, I mean the listing has already been depleted of comments. I say this because when you delete comments, it does not retroactively add older comments on to the tail end of the listing.
You can easily verify this by visiting https://old.reddit.com/user/REDDITOR/comments?limit=100 and see how far back Reddit allows you to go. That's the source of truth.
As a result, I suspect this issue has nothing to do with PRAW, and is instead the behavior of Reddit's listings so I will close this issue. If, however, I am mistaken, or you have more information to add, please follow up.

Have you deleted comments in such a way in the past? If so, it's possible there are no more comments available to delete. By available, I mean the listing has already been depleted of comments. I say this because when you delete comments, it does not retroactively add older comments on to the tail end of the listing.
You can easily verify this by visiting https://old.reddit.com/user/REDDITOR/comments?limit=100 and see how far back Reddit allows you to go. That's the source of truth.
As a result, I suspect this issue has nothing to do with PRAW, and is instead the behavior of Reddit's listings so I will close this issue. If, however, I am mistaken, or you have more information to add, please follow up.

So this is very strange. I ran it for the first time today after receiving this, it counted back to 400. I ran it a few times, deleting about 300 comments total. Then it was counting back 529 (which I confirmed is to the new end of the listing).
The listing was definitely the full 1000 full when I first ran it and had the problem with 100, because I hadn't run the program in a long time. What it deletes is "claim" and "done" comments relating to the Transcribers of Reddit, and since I do several transcriptions a day, these one-time bot interactions get posted to my profile several times per day (recently, over 50 times a day because I've had little else to do). Thus, new comments that fit the deletion requirement continually get added to the front of the listing.
Now, I hadn't run the program to delete these in several weeks (probably a month, to be honest), so the listing was absolutely stuffed with them - even checking that URL today, I initially found comments that should've been being deleted going right up to the very back of the full 1000.
And as I've said, the more times I ran it the more the amount it was actually counting extended, which was strange. And now, of course, it's once again counting to the end of the listing as it used to. All in all, I'm very confused about it.

So this is very strange. I ran it for the first time today after receiving this, it counted back to 400. I ran it a few times, deleting about 300 comments total. Then it was counting back 529 (which I confirmed is to the new end of the listing).
The listing was definitely the full 1000 full when I first ran it and had the problem with 100, because I hadn't run the program in a long time. What it deletes is "claim" and "done" comments relating to the Transcribers of Reddit, and since I do several transcriptions a day, these one-time bot interactions get posted to my profile several times per day (recently, over 50 times a day because I've had little else to do). Thus, new comments that fit the deletion requirement continually get added to the front of the listing.
Now, I hadn't run the program to delete these in several weeks (probably a month, to be honest), so the listing was absolutely stuffed with them - even checking that URL today, I initially found comments that should've been being deleted going right up to the very back of the full 1000.
And as I've said, the more times I ran it the more the amount it was actually counting extended, which was strange. And now, of course, it's once again counting to the end of the listing as it used to. All in all, I'm very confused about it.

It could have been a momentary issue with Reddit's cached listings for that user. I'm glad to read that things appear to be working now, however.

It could have been a momentary issue with Reddit's cached listings for that user. I'm glad to read that things appear to be working now, however.


User is not automatically scrolled to last message after sending a message

Describe the bug

User is not automatically scrolled to his last send message on sending a message
To Reproduce

Go to any chatroom
Scroll up a little
now send a message
Nothing happen

Expected behavior

After sending a message user should be automatically scrolled to his last send message
Additional context

I would like to work on this issue

@GOVINDDIXIT Like we discussed on the past (June last year to be more precise) on our community channel this is a not desired thing to do.
You can find more information about the reason here: https://open.rocket.chat/channel/androidnativeapp?msg=ceGsrnJFtoFz3E6Lt
Thank you.


first error: CANNOT_INSERT_UPDATE_ACTIVATE_ENTITY,   dlrs_ContactTrigger: execution of AfterUpdate  caused by: System.NoAccessException: Entity is not api accessible

Error Type: Batch Apex error
Error Date: 2016-11-04 03:10:08
Message: "First error: Update failed. First exception on row 0 with id 003A000000uVn8nIAC; first error: CANNOT_INSERT_UPDATE_ACTIVATE_ENTITY, dlrs_ContactTrigger: execution of AfterUpdate caused by: System.NoAccessException: Entity is not api accessible Class...."
Context: npsp__RLLP_OppContactRollup_BATCH
Stack Trace:
null

The above error started appearing this week, despite releasing no new Apex code recently.
What is the best way to debug and fix the error? We are running the latest versions of both Declarative Lookup Rollup Summaries Tool and the Nonprofit Success Pack.

We've been getting the same thing, it seems to have started when the Nonprofit Success Pack was updated on 2nd November, we've had this error message every night since then. I've been receiving it from two different orgs where we are using both DLRS and NPSP.
npsp__RLLP_OppContactRollup_BATCH is an Apex Class scheduled to run each night, and I also seem to always get an error for npsp__RLLP_OppSoftCreditRollup_BATCH.
I can see from release notes that they've changed how the error messages are generated, but I'm not much of an expert so not really sure what I'm looking for! https://github.com/SalesforceFoundation/Cumulus/releases
If there's anything I can help with to fix please let me know!

Hey Folks,
Have you followed the instructions to change the scheduled jobs user in the NPSP? Located here: https://powerofus.force.com/articles/Resource/NPSP-Edit-or-Reschedule-NPSP-Scheduled-Jobs. There are some issues with the NPSP shadow user, so switching to an administrator account is required.

Yes spotted that earlier! And it seems to have solved the issue completely. Thanks.

I've been having this issue with some of my NPSP orgs, but I don't see what I am supposed to do with the above mentioned link? All of the users in my scheduled jobs are "Active", and I'm not wanting to change the scheduled time of any of my jobs.... I just want to stop the errors :-) What am I missing??

@StephanieKing if you've tried @cdcarter's issue then i suspect it's something described in this issue. What org edition are you running?

Yes, this issue doesn't seem to help me, I'll check #360 , thanks!

Closing this, as its a duplicate of another issue. Look for updates on https://github.com/afawcett/declarative-lookup-rollup-summaries/issues/360.

Closing this, as its a duplicate of another issue. Look for updates on https://github.com/afawcett/declarative-lookup-rollup-summaries/issues/360.


Support non-gz compressed files in the archives check

pgbackrest supports several different compression methods, so
check_pgbackrest should do the same.
Regexp taken from COMPRESS_TYPE_REGEXP in the backrest source code.
In passing, properly escape the . character in the regexp so it doesn't
match "anything".

Merged in the branch planned for next release. Thanks !


Make vendored the default branch of this repository

Since master is just the untouched code from the original skim repository, it makes more sense to make the actual branch, vendored, the default branch.

This sounds fine, but imagine skim decides to start development again, I want to be able to merge their new PRs incrementally.  I'm not sure this is possible with a fork, but I think works with a branch.  So I'd like to make sure I don't have to accept all changes at once, or I can merge to the local master and pull incrementally from there.
Do you know any better about how this works?

I don't mean you should move the work to the master branch. What I mean is to go to the repository's settings and under the General tab change Default branch from master to vendored. What this will do is:

Change the default branch displayed in https://github.com/kimono-koans from master to vendored.
When cloning the repository, vendored will be checked out by default instead of master.
When forking the repository, the "Copy the master branch only" checkbox will change to "Copy the vendored branch only"

You'll still have a master branch that will remain the same as the upstream master branch, and you could still use that branch to sync with the original Skim if they start development again. All that change will do is make GitHub treat vendored as the main development branch of this repository.

Okay, it's been done.  We will see how it works out.


Yeoman generator

The current process is to download a copy of the app and start hacking. It would be better if this was a Yeoman generator so people could start with
npm install shopify-react-starter
yo shopify-react-starter

We could then allow further customization during the installation process.

Will be included in version 3.1


The option 'App Configuration Store' on the 'Search for resources of type' panel is unchecked after restarting Storage Explorer

Storage Explorer Version: 1.29.0-dev
Build Number: 20230329.4
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 22.04/MacOS Ventura 13.2.1 (Apple M1 Pro)
Architecture: ia32/x64
How Found: From running test cases
Regression From: Previous release (1.28.1)
Steps to Reproduce

Launch Storage Explorer -> Install Azure App Configuration extension for Storage Explorer.
Open the panel 'Search for resources of type:'.
Check the option 'App Configuration Store'.
Restart Storage Explorer -> Open the panel 'Search for resources of type:'.
Check whether the option 'App Configuration Store' is checked or not.

Expected Experience
The option 'App Configuration Store' is checked.

Actual Experience
The option 'App Configuration Store' is unchecked.

Additional Context
This issue doesn't reproduce for other types.

The behavior described is expected. Instead of always selecting all options, Storage Explorer remembers the selected search options each time it starts a search. In this case, if you select the App Configuration Store option, start a search and then close Storage Explorer, it should be selected then.

Hi @JasonYeMSFT
thank you for the response!
if you select the App Configuration Store option, start a search and then close Storage Explorer, it should be selected then.  -> It works as the description.
For other options, such as Resource Groups (Disks), will be selected after restarting Storage Explorer, regardless of whether the option is unchecked or checked. So, should we keep a consistent between 'App Configuration Store option' and other type options?

@v-xianya I checked the code. The logic to determine whether a search option is enabled is consistent for all services. If no search has been run, Storage Explorer by default checks all the options. If a search has been run, the previously selected options are checked. If you go to the settings file, manually remove this entry "tree.search.checkedTypes" and redo the repro steps I believe you will see the App Configuration option enabled.

Verified this issue on the build main 20230407.2. Fixed.
The option 'App Configuration Store' on the 'Search for resources of type' panel is checked after restarting Storage Explorer


Folded code icon in editor is wrong

Fold some code
The ... button at the end of the line is not correct

Maybe the encoding got mangled during the build?

Where do you see this? What build and platform?
I can't see it in the latest insiders, neither on Linux, Windows nor the web.
Also there hasn't been any changes to the css file in the last years.

Actually, I think devtools might be messing up that file. It only happens after I open devtools...

Yeah, I saw it on two different platforms at different times and thought there was something real, but no, this is bug in devtools' CSS pretty printer...

dup of https://github.com/microsoft/vscode/issues/145897


Update refined, refined-cats to 0.10.2

Updates

eu.timepit:refined
eu.timepit:refined-cats

from 0.10.1 to 0.10.2.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (0.10.1).
You might want to review and update them manually.
CHANGELOG.md
Gemfile.lock

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "eu.timepit" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "eu.timepit" }
}]

labels: library-update, early-semver-minor, semver-spec-patch, old-version-remains, commit-count:1

Superseded by #1716.


Twitter has moved the end points for its API to 1.1 from 1

I'm getting 410 responses because the library is making requests to this end point:
api.twitter.com/1/statuses/home_timeline.json

when it needs to be
api.twitter.com/1.1/statuses/home_timeline.json

now.
I'm not sure if you're still taking pull requests for this project? If so I can update the URL scheme pretty easily and send it to you?
Cheers, Jason.

Hey, sure, I'll accept a pull request. I should probably see if anything else needs to be modified.


Add cupy.mask_indices

As part of https://github.com/cupy/cupy/issues/6078, adding the mask_indices API.

Would you also add mask_indices to docs/source/reference/indexing.rst so that the  document of mask_indices is generated?

Thanks for reviewing @takagi! I've addressed your comments. Please let me know if you think this PR needs anything else!

/test

Thanks! Let me push a fix to your repo as it is a minor change, and now I've kicked the CI. Your PR is going to be merged soon after CI passes.

/test mini


Add ‘revo’ and ‘reincarnated’ as prefixes for the GFEs

ex ‘revo dmeta’ -> 3535 instead of 643
‘revo ilm’ -> 3236 instead of 1951

closed by #145


Added release notes generation to release wf

Issue link

RHOAIENG-7136
What changes have been made

Added generateReleaseNotes: true as per the documentation for the release action the SDK uses -> https://github.com/ncipollo/release-action
Verification steps

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] Testing is not required for this change

/approve


WebCam based Eye-Tracker

How to use webcam Eye-Tracker (GazePointer: https://sourceforge.net/projects/gazepointer/ ) with the Gaze Interaction Library ?

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

ID: 6cc2d6d6-2316-bb10-5dc7-12c0b97e02d3
Version Independent ID: f4065046-8d78-785a-1369-40a749d63147
Content: Gaze interactions - Windows UWP applications
Content Source: windows-apps-src/design/input/gaze-interactions.md
Product: uwp
Technology: design
GitHub Login: @Karl-Bridge-Microsoft
Microsoft Alias: kbridge

Hi @szydej - please see https://support.microsoft.com/en-us/help/4043921/windows-10-get-started-eye-control#supported-devices for a list of supported devices.


Unmount everything and dispose States when host activity dies

Looking at #19159, it seems reasonable looking at the API doc to expect the State.dispose to be called when the FlutterActivity is finishing/being destroyed. Yet it never gets called and there's an inbalance between things registered and/or created during initState() and things unregistered and/or released during dispose().
Should we:

Have FlutterView also handle onDestroy() (and equivalent on iOS)
Have a new AppLifecycleState for finishing
Have the widget framework unmount and dispose everything on onDestroy

or at least have the State.dispose() API doc reference AppLifecycleState if we don't want to do this automatically?
cc @Hixie @jason-simmons
also cc @matthew-carroll since this scenario will surface more frequently with add-to-app

One thing that needs to be considered here is the possibility that Flutter is used across Activitys.  I haven't looked much at that use-case, so I'm not sure how much of that scenario is about persisting the Dart process vs persisting Flutter hierarchy.  But let's not immediately assume that Flutter should disappear when an Activity or FlutterView is destroyed - if that's what is being suggested, then someone needs to make that case very carefully and deeply.
Don't forget, for example, that not all Widgets are UI.  Consider an AudioPlayer widget.  The UI may have disappeared, but the audio should probably keep playing.  Now, half of that story is on the platform side, but it doesn't necessarily mean you want to lose the Flutter state related to that Widget/Element.
  Then the developer can opt-in/opt-out.

WRT to #19159 - if the true problem of that ticket is a failure to unregister, why would the ticket require 20 Activity restarts to develop the issue?  I might expect a memory leak in that scenario, but #19159 says the issue is an ANR which is a CPU lockup, not a memory pressure issue.
I'd like to understand more clearly why this disposal problem is supposedly causing that other issue...

https://github.com/flutter/engine/pull/5640/files seems to be making the case of calling out backgroundable 'hosts' explicitly but making it an explicit signal (which may be auto-hooked) sounds great.

Generally speaking you can't rely on the OS doing anything graceful to your app anyway, FWIW, so depending on dispose being called is asking for trouble. Consider what happens when the user pulls the battery, or the OS kills your app for using too much memory, or whatnot.

I think with add-to-app, we should start to change how we think about platform interactions and pull parts out from the "OS doing things to your app" group into "mixed navigation" group. Related discussion (internal).
In your pulling the battery scenario, we'd be in the same situation as Flutter to Flutter navigation (no automatic calls to dispose) and native to native navigation (no automatic calls to onDestroy).
But otherwise, I think we should make mixed navigation API convenience a first class citizen too as opposed to our present alternative (which would involve an encapsulated Flutter view logic handling dispose() in case it's navigated away from inside Flutter and also having a custom FlutterFragment which handles onDestroy and have a platform channel with a custom listener registration mechanism on the Flutter side that views need to register to to perform the same dispose() task when it's navigated away from natively).
Whether they both trigger the same function (dispose()) and whether the native navigating-away triggers it automatically is debatable like Matt is saying, but I don't think the present state is sufficient.

@xster would you mind putting together a few sequence diagrams to show what we have now vs what you're recommending?  I'm having a difficult time comprehending the recommendation in paragraph form.

To clarify, I definitely agree that we should be calling dispose methods and so on during shutdown (otherwise I'd have suggested closing the bug!). I'm just saying that we shouldn't encourage people to rely on it.

Hi Flutter Team, did you find time to work on this issue, yet? I have been running into similar issues rgd. the App State while Androids' debug setting  "Don't keep activities" is turned on.


"aws_appautoscaling_target" showing changes to tags in each plan, because of ignore_changes on 'tags_all'

Description
Due to a known issue with tags on the appautoscaling_target resource, as described in the Terraform documentation, this module includes a lifecycle rule to ignore changes to tags_all. This rule is defined in the main.tf file:
resource "aws_appautoscaling_target" "this" {
  count = local.create && var.autoscaling_enabled && !local.is_serverless ? 1 : 0

  max_capacity       = var.autoscaling_max_capacity
  min_capacity       = var.autoscaling_min_capacity
  resource_id        = "cluster:${aws_rds_cluster.this[0].cluster_identifier}"
  scalable_dimension = "rds:cluster:ReadReplicaCount"
  service_namespace  = "rds"

  tags = var.tags

  lifecycle {
    ignore_changes = [
      tags_all,
    ]
  }
}

In my project, I use a combination of local tags and default_tags, and I instantiate the terraform-aws-rds-aurora module from within a custom module. Every time I run a plan, Terraform attempts to apply new tags added since the initial resource creation. However, due to the ignore_changes lifecycle rule, these tags are excluded from tags_all, which leads to the following issues:

Repeated Plan Outputs: Every Terraform plan shows the same tags to be applied, despite those tags not being included in the remote state due to the lifecycle rule.

Persisting Change Notifications: Even after applying, the plan still shows the same pending changes in every consecutive run.

Workaround Attempts
I've tried tainting and reapplying the resource, which correctly adds the tags and removes the change notification in future plans. However, this is not a viable solution as the tags are updated frequently, and tainting the resource for every change is impractical given the large number of resources using this module.
Proposed Solution
Would it be possible to make the lifecycle rule for ignore_changes on tags_all optional, or provide a way to remove it entirely? This would allow the tags to propagate to the remote state without requiring repeated tainting and redeployment.
Versions

Module version [Required]: 9.3.1 (originally used, but tried with latest 9.10.0 and still same issue)

Terraform version: v1.5.7

Provider version(s): v5.75.0

Reproduction Code [Required]
module "aurora" {
  source  = "terraform-aws-modules/rds-aurora/aws"
  version = "9.3.1"

  name          = var.cluster_name
  database_name = var.database_name

 [...]

  tags = merge(var.tags, { "Name" : "Aurora", "rds-db-id": "rds-id-1" })

}

Steps to reproduce the behavior:
The Terraform plan output shows:
  ~ resource "aws_appautoscaling_target" "this" {
         id          = "cluster:rds-app-development"
      ~ tags         = {
            "Cluster"        = "rds-app-development"
            "Environment"    = "development"
            "Name"           = "Aurora"
          + "rds-db-id"      = "rds-id-1"
        }
        # (8 unchanged attributes hidden)

        # (1 unchanged block hidden)
    }

Expected behavior
The tags should be updated after applying, or add tags to ignore_changes lifecycle.

I would just taint the autoscaling target and let it be replaced so it gets the ARN and this issue is resolved properly

I would just taint the autoscaling target and let it be replaced so it gets the ARN and this issue is resolved properly

As I explained in the "Workaround Attempts" section, I tried following the taint process, even though my resources already had ARN. However, if I change the tags once again after the autoscaling target got replaced, it will fall under the same situation, where the new tag is not being added to tags_all, so the tag addition is shown in every consecutive plan.

I was wondering if any update on this topic. Please refer to my last comment that explains why tainting the resource is not a viable option.