🐛 [Bug] RuntimeError with V1.3: Trying to create tensor with negative dimension

Bug Description
Torch-TensorRT Model Compilation gives runtime error when dynamic input shape is used. This error is not present in an older version of Torch-TensorRT (1.0).
To Reproduce
Code to reproduce the behavior:
import torch
import torch_tensorrt
import torch.nn as nn
import torch.nn.functional as F
import timm

class HRNetw48(nn.Module):
    def __init__(self, input_frames=2):
        super().__init__()
        base_model_name: str = 'hrnet_w48'
        input_depth: int = input_frames
        feature_location: str = ''
        self.combine_outputs_dim: int = 512
        self.upscale_mode: str = 'nearest'
        self.pred_scale: int = 8

        self.base_model = timm.create_model(base_model_name,
                                            features_only=True,
                                            feature_location='',
                                            out_indices=(1, 2, 3, 4),
                                            in_chans=input_depth,
                                            pretrained=False)

        self.combine_outputs_kernel = 1
        self.fc_comb = nn.Conv2d(720, self.combine_outputs_dim, kernel_size=self.combine_outputs_kernel)
        hrnet_outputs: int = self.combine_outputs_dim

        self.fc_cls = nn.Conv2d(hrnet_outputs, 7, kernel_size=1)
        self.fc_size = nn.Conv2d(hrnet_outputs, 2, kernel_size=1)
        self.fc_offset = nn.Conv2d(hrnet_outputs, 2, kernel_size=1)
        self.fc_distance = nn.Conv2d(hrnet_outputs, 1, kernel_size=1)
        self.fc_tracking = nn.Conv2d(hrnet_outputs, 2, kernel_size=1)
        self.fc_mask = nn.Conv2d(hrnet_outputs, 1, kernel_size=1)
        self.fc_horizon = nn.Conv2d(hrnet_outputs, 1, kernel_size=1)

    def forward(self, inputs):
        stages_output = self.base_model(inputs)

        x = [
            F.avg_pool2d(stages_output[0], 2),
            stages_output[1],
            F.interpolate(stages_output[2], scale_factor=2.0, mode="nearest"),
            F.interpolate(stages_output[3], scale_factor=4.0, mode="nearest"),
        ]
        x = torch.cat(x, dim=1)
        x = F.relu(self.fc_comb(x))
        size = self.fc_size(x)
        offset = self.fc_offset(x)
        distance = self.fc_distance(x)
        tracking = self.fc_tracking(x)
        mask = self.fc_mask(x)
        above_horizon = self.fc_horizon(x)

        return mask, size, offset, distance, tracking, above_horizon

model = HRNetw48(input_frames=2)
model = model.half().cuda()
model.eval()

print('Compiling TensorRT model ...')
model_jit = torch.jit.trace(model, torch.rand((1, 2, 512, 512)).cuda().half(), strict=False)
model = torch_tensorrt.compile(
    model_jit,
    inputs=[torch_tensorrt.Input(
        min_shape=[1, 2, 512, 512],
        opt_shape=[4, 2, 512, 512],
        max_shape=[4, 2, 512, 512], dtype=torch.half)
    ],
    enabled_precisions={torch.half},
    truncate_long_and_double=True
)
torch.jit.save(model, f"./130_hrnet48_all_2220.ts")

Expected behavior
Model compiles successfully.

Build information about Torch-TensorRT can be found by turning on debug messages

Torch-TensorRT Version: 1.3.0a0
PyTorch Version: 1.13.0a0+d0d6b1f
CPU Architecture: Intel(R) Xeon(R) CPU E5-2698 v4 @ 2.20GHz
OS (e.g., Linux): Ubuntu 20.04.5 LTS
How you installed PyTorch: Using NGC Docker Image (nvcr.io/nvidia/pytorch:22.10-py3)
Build command you used (if compiling from source):
Are you using local sources or building from archives:
Python version: 3.8.13
CUDA version: 11.8
GPU models and configuration:
Any other relevant information:

Additional context
The same code works on a docker image with Torch version: 1.10.0a0+0aef44c, Torch-TensorRT version: 1.0.0+55c3bab4, TensorRT version: 8.0.3-1, and CUDA 11.3.
Based on Open and Closed Issues, I tried out different variants of the compilation, however, nothing was successful. Details:
Code:
model = torch_tensorrt.compile(
    model_jit,
    input_signature=[torch_tensorrt.Input(
        min_shape=[1, 2, 512, 512],
        opt_shape=[4, 2, 512, 512],
        max_shape=[4, 2, 512, 512], dtype=torch.half)
    ],
    enabled_precisions={torch.half},
    truncate_long_and_double=True,
    require_full_compilation=True
)

Output:
WARNING: [Torch-TensorRT] - Input signature parsing is an experimental feature, behavior and APIs may change
ValueError: Grouped inputs currently requires partial compilation to be enabled, this restriction will be relaxed in a future release

Code:
model = torch_tensorrt.compile(
    model_jit,
    input_signature=[torch_tensorrt.Input(
        min_shape=[1, 2, 512, 512],
        opt_shape=[4, 2, 512, 512],
        max_shape=[4, 2, 512, 512], dtype=torch.half)
    ],
    enabled_precisions={torch.half},
    truncate_long_and_double=True
)

Output:
WARNING: [Torch-TensorRT] - Input signature parsing is an experimental feature, behavior and APIs may change
RuntimeError: Trying to create tensor with negative dimension -1: [-1, 2, 512, 512]

Code:
model = torch_tensorrt.compile(
    model_jit,
    inputs=[torch_tensorrt.Input(
        min_shape=[1, 2, 512, 512],
        opt_shape=[4, 2, 512, 512],
        max_shape=[4, 2, 512, 512], dtype=torch.half)
    ],
    enabled_precisions={torch.half},
    truncate_long_and_double=True,
    require_full_compilation=True,
    torch_executed_ops=["prim::TupleConstruct"]
)

Output:
ValueError: require_full_compilation is enabled however the list of modules and ops to run in torch is not empty. Found: torch_executed_ops: ['prim::TupleConstruct'], torch_executed_modules: []

Code:
model = torch_tensorrt.compile(
    model_jit,
    inputs=[torch_tensorrt.Input(
        min_shape=[1, 2, 512, 512],
        opt_shape=[4, 2, 512, 512],
        max_shape=[4, 2, 512, 512], dtype=torch.half)
    ],
    enabled_precisions={torch.half},
    truncate_long_and_double=True,
    torch_executed_ops=["prim::TupleConstruct"]
)

Output:
RuntimeError: Trying to create tensor with negative dimension -1: [-1, 2, 512, 512]

Code:
model = torch_tensorrt.compile(
    model_jit,
    input_signature=[torch_tensorrt.Input(
        min_shape=[1, 2, 512, 512],
        opt_shape=[4, 2, 512, 512],
        max_shape=[4, 2, 512, 512], dtype=torch.half)
    ],
    enabled_precisions={torch.half},
    truncate_long_and_double=True,
    torch_executed_ops=["prim::TupleConstruct"]
)

Output:
WARNING: [Torch-TensorRT] - Input signature parsing is an experimental feature, behavior and APIs may change
RuntimeError: Trying to create tensor with negative dimension -1: [-1, 2, 512, 512]

Hello - I have tested your provided script on Torch-TensorRT 1.3.0, and the model is compiling successfully on my machine. I believe the issue may be with the version of Torch-TensorRT packaged in the 22.10 Docker container. Specifically, 1.3.0a0 is not the same build as 1.3.0. The following commands, when run inside the Docker container, will install version 1.3.0, with the necessary PyTorch version as well. With the new version installed, rerun the script, and please let me know if this resolves the issue.
# Uninstall torch and torch-tensorrt
pip uninstall torch-tensorrt torch

# Reinstall torch and torch-tensorrt
pip install torch==1.13.0
pip install torch-tensorrt==v1.3.0 -f https://github.com/pytorch/TensorRT/releases/expanded_assets/v1.3.0

Thanks for the response!
I tried reinstalling torch and torch-tensorrt on the NGC docker image, and the compilation works. However, there is this warning message which shows up:
WARNING: [Torch-TensorRT TorchScript Conversion Context] - TensorRT was linked against cuDNN 8.6.0 but loaded cuDNN 8.5.0

Any thoughts on how I can update this on the docker image?
Also, with this fix, there is a problem now being caused by loading a compiled torch script model. Described in this new issue: https://github.com/pytorch/TensorRT/issues/1550#issue-1496779475


Update ExportToExcel for error due to Action class mismatch

Fixes an error using this package with Nova 4. The function can be removed, as it is identical to the base Action class function.

Laravel Nova 4.22 Action class has no method withName
You should just rename this method


Update PR Template to make sure libraries are synced

Template should ask developer to check if they've made a change to the Twig/React lib that they've also made the corresponding update to the other library. This is to make sure that the libraries are in sync.

This should just be handled with tests


Encrypted AuthContainer payload

AuthContainer is the envelope for all messages coming from the controller or sent by EVE. For the LOC case the node config should be encrypted. The best place to provide encryption is to modify the AuthContainer itself, so that the message can be both signed and encrypted (in this exact order).
This patch adds the cipherData and cipherContext members to the AuthContainer to encrypt the payload. The decrypt and validate procedure is the following:

if cipherData is valid the encrypted payload is defined in the cipherData.cipherData member, the protectedPayload is set to null.
Once EVE stack decrypts the payload, protectedPayload member is set with the decrypted data.
The signature verification procedure can follow.

CC: @deitch

I cannot comment on the content of the change (especially as Erik did that). I can comment on the toolset. It looks like yo ran this using a single step more recent minor version of protoc, but a several steps older minor version of protoc-gen-go.
Can you rerun the make proto with the right tools? Or more easily, inside the devcontainer? If you are in vscode, it should recognize it for you. Else follow the instructions in the README.

I cannot comment on the content of the change (especially as Erik did that). I can comment on the toolset. It looks like yo ran this using a single step more recent minor version of protoc, but a several steps older minor version of protoc-gen-go.
Can you rerun the make proto with the right tools? Or more easily, inside the devcontainer? If you are in vscode, it should recognize it for you. Else follow the instructions in the README.

Avi, as I said in the mail thread I can't install devcontainer. I spent quite a lot of time trying to build it from sources and also failed (I'm on linux, not on mac). I'm not using vscode. The whole procedure of 'make proto' now is inconvenient for me and I expect more complains will follow (the whole eve team is using linux, not mac).
I can rebuild the protoc-gen-go, but as I said this is very inconvenient.

@deitch @eriknordmark updated, please take a look.

LGTM but if you can pull in #15 you should be able to manually use the devcontainer on Linux.

Thanks, @eriknordmark . These changes work for me. I replied here: https://github.com/lf-edge/eve-api/pull/15/commits/3ca855b339c1a6f5e26deb3730472be45ca9d90c#r1298131945


Fixing Travis-CI build

Following the discussin in PR #15 , this fixes some issues with the Travis-CI setup, and in particular,

updates the miniconda.sh executable to the last version
removes the use of conda virtual env as this was not working: source activate pykrige-env executed but didn't do anything, the python command was still loading the default python environement not the one in miniconda (not sure why), which resulted in the scipy import issue.
also rasing an exception if any of the dependencies are not installed (in adition to the prevously added custom message) so it's easier to debug.

Fantastic, thanks for the good work guys!


Fix Compat.toml for libLLVM

I'm pretty sure we want to go the other way on this; we only want julia=1.5.

Closed in favor of https://github.com/JuliaRegistries/General/pull/12972


Electron 3.0.x: Object has been destroyed upon quit with multiple windows open

Refs: https://github.com/electron/electron/issues/14513

Fixed in beta.12


Minor code fix to make library compatible with DirectXTex October 2024

The October 2024 release of DirectXTex changed void* to uint8_t* usage for buffers. This breaks std::byte* usage.
A hotfix is being made for the library to fix this and the next version includes the updates, but a minor tweak is needed to this library to work. It should be compatible with both the old and new version.

lgtm


Use F.matmul instead of F.batch_matmul

batch_matmul is going to be unavailable in Chainer v3, or would be deprecated even if https://github.com/chainer/chainer/pull/3016 is merged.

Oops, the PR depends on the new F.matmul in v3.

I think It is better to support both v2 and v3 for now by checking the version of chainer and use F.numpy_like_matmul for v2.

Coverage increased (+0.01%) to 71.537% when pulling 5138b7339ea750785673750303eb465fa8b69947 on toslunar:use-matmul into afa7583aabfd240e7d4fc9b2bf8464e68bd1cdea on chainer:master.

Coverage decreased (-0.0003%) to 71.525% when pulling fb22da8aa8ff3b8af4e1b1619efe362ea6282c79 on toslunar:use-matmul into afa7583aabfd240e7d4fc9b2bf8464e68bd1cdea on chainer:master.

Can you merge the current master to this PR so that CI runs with both Chainer v2 and v3?

Rebased to the current master

Coverage decreased (-0.04%) to 71.665% when pulling 25849f300aad8786edbb5644742396d8ae79963e on toslunar:use-matmul into e84ea5f89f8e293969eee621baf7a512b6e95b42 on chainer:master.

Coverage decreased (-0.04%) to 71.665% when pulling 25849f300aad8786edbb5644742396d8ae79963e on toslunar:use-matmul into e84ea5f89f8e293969eee621baf7a512b6e95b42 on chainer:master.

Coverage increased (+0.04%) to 71.744% when pulling 2f8b887a2da04a381607e464571e2314600f5bbf on toslunar:use-matmul into e84ea5f89f8e293969eee621baf7a512b6e95b42 on chainer:master.

LGTM


deps!: bumping nmt to v0.19.0, breaking protobuf

Related: #2633

Changes of the name do not break protos. Changes of the type and fields reorder do.

Codecov Report

Merging #2643 (97a3881) into main (47047f3) will decrease coverage by 0.28%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2643      +/-   ##
==========================================
- Coverage   51.34%   51.06%   -0.28%
==========================================
  Files         158      158
  Lines       10559    10559
==========================================
- Hits         5421     5392      -29
- Misses       4669     4693      +24
- Partials      469      474       +5

see 7 files with indirect coverage changes

Or does the way it work not care about naming and just uses data types and offsets.

Yes

closing in favor of core bump #2646


Use base64 encoded credentials input instead of raw text

Is your feature request related to a problem? Please describe.

Security issues. This suggestion is from observing how Github action does it.

Describe the solution you'd like
Use base64 encoded credentials input instead of raw text
Additional context
If this is agreed I'll implement and make a PR.

Use base64 doesn't make any different. people can still decrypt it.
Giving a limited permission to the service account might be a better way:

Create a service account with no role & permission.
Create a Multi-Regional Bucket  and add the service account with Storage Admin Permission
Go to Strapi plugin setting page to set service account json and bucket name for upload plugin.

#12 solution // relate to


⚠ ku-s has degraded performance

In 93508dc, ku-s ($KU_API_STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 9313 ms

Resolved: ku-s performance has improved in 17ffbd3.


For 0.12.1 release

(enhancement) Add serialisation for machines. Serialization is model-specific with a fallback implementation using JLSO. The user serializes with MLJBase.save and de-serializes with machine. See the MLJBase.save doc-string for details. Model-specific serialization is defined by methods MLJBase.save(model::MyModel, ...) and de-serialization with MLJBase.restore(model::MyModel, ...) (PR #218, https://github.com/alan-turing-institute/MLJ.jl/issues/138, https://github.com/alan-turing-institute/MLJ.jl/issues/292)

Codecov Report

Merging #219 into master will decrease coverage by 0.54%.
The diff coverage is 92.00%.

@@            Coverage Diff             @@
##           master     #219      +/-   ##
==========================================
- Coverage   83.16%   82.61%   -0.55%
==========================================
  Files          30       30
  Lines        2043     1979      -64
==========================================
- Hits         1699     1635      -64
  Misses        344      344

Impacted Files
Coverage Δ

src/MLJBase.jl
100.00% <ø> (ø)

src/composition/networks.jl
67.51% <50.00%> (-1.62%)
:arrow_down:

src/composition/composites.jl
93.51% <100.00%> (+0.46%)
:arrow_up:

src/interface/model_api.jl
100.00% <100.00%> (ø)

src/machines.jl
98.46% <100.00%> (+0.21%)
:arrow_up:

src/parameter_inspection.jl
50.00% <0.00%> (-50.00%)
:arrow_down:

src/hyperparam/one_dimensional_ranges.jl
82.22% <0.00%> (-2.57%)
:arrow_down:

src/utilities.jl
75.00% <0.00%> (-0.65%)
:arrow_down:

src/distributions.jl
84.69% <0.00%> (-0.46%)
:arrow_down:

... and 12 more

Continue to review full report at Codecov.

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


ERROR [stderr] (signalfx-metrics-publisher) [opentelemetry.auto.trace ...]

Hello all
I have install the splunkt-otel-java Agent on Openshift 4.6 on an Linux Image from RedHat jboss-eap-73-openjdk11-rhel8
The Agent send Data to an SignalFF-Agent in the Cluster running as Deamonset. This agent send the Data to an Otel-Collector in the AWS and they send it to SignalFX.
Now after Start i see this ERROR messages:
[opentelemetry.auto.trace 2021-05-07 08:05:53:596 +0200] [main] INFO io.opentelemetry.javaagent.tooling.VersionLogger - opentelemetry-javaagent - version: splunk-0.10.0-otel-1.1.0
[opentelemetry.auto.trace 2021-05-07 08:05:56:713 +0200] [main] INFO com.splunk.javaagent.shaded.io.micrometer.core.instrument.push.PushMeterRegistry - publishing metrics for SignalFxMeterRegistry every 30s
08:06:14,987 DEBUG [org.jboss.as.config] (MSC service thread 1-2) Configured system properties:
[Standalone] =
	io.opentelemetry.javaagent.slf4j.simpleLogger.dateTimeFormat = '[opentelemetry.auto.trace 'yyyy-MM-dd HH:mm:ss:SSS Z']'
	io.opentelemetry.javaagent.slf4j.simpleLogger.log.muzzleMatcher = OFF
	io.opentelemetry.javaagent.slf4j.simpleLogger.showDateTime = true
	otel.exporter.jaeger.endpoint = http://localhost:9080/v1/trace
	otel.metrics.exporter = none
	otel.propagators = b3multi
	otel.resource.os.description = Linux 4.18.0-193.41.1.el8_2.x86_64
	otel.resource.os.type = LINUX
	otel.resource.process.command_line = /usr/lib/jvm/java-11-openjdk-<IP_ADDRESS>-3.el8_3.x86_64:bin:java -D[Standalone] -Xlog:gc*:file=/opt/eap/standalone/log/gc.log:time,uptimemillis:filecount=5,filesize=3M -Xms63m -Xmx250m -XX:MetaspaceSize=96m -XX:MaxMetaspaceSize=200m -Djava.net.preferIPv4Stack=true -Djboss.modules.system.pkgs=org.jboss.logmanager,jdk.nashorn.api,com.sun.crypto.provider -Djava.awt.headless=true -Xbootclasspath/a:/opt/jboss/container/wildfly/s2i/galleon/galleon-m2-repository/org/jboss/logmanager/jboss-logmanager/2.1.17.Final-redhat-00001/jboss-logmanager-2.1.17.Final-redhat-00001.jar:/opt/jboss/container/wildfly/s2i/galleon/galleon-m2-repository/org/glassfish/jakarta.json/1.1.6.redhat-00001/jakarta.json-1.1.6.redhat-00001.jar:/opt/jboss/container/wildfly/s2i/galleon/galleon-m2-repository/jakarta/json/jakarta.json-api/1.1.6.redhat-00001/jakarta.json-api-1.1.6.redhat-00001.jar:/opt/jboss/container/wildfly/s2i/galleon/galleon-m2-repository/org/wildfly/common/wildfly-common/1.5.2.Final-redhat-0...
	otel.resource.process.executable.path = /usr/lib/jvm/java-11-openjdk-<IP_ADDRESS>-3.el8_3.x86_64:bin:java
	otel.resource.process.runtime.description = Red Hat, Inc. OpenJDK 64-Bit Server VM <IP_ADDRESS>+1-LTS
	otel.resource.process.runtime.name = OpenJDK Runtime Environment
	otel.resource.process.runtime.version = <IP_ADDRESS>+1-LTS
	otel.resource.service.name = APP-TEST
	otel.resource.telemetry.auto.version = splunk-0.10.0-otel-1.1.0
	otel.resource.telemetry.sdk.language = java
	otel.resource.telemetry.sdk.name = opentelemetry
	otel.resource.telemetry.sdk.version = 1.1.0
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by com.github.os72.protobuf351.UnsafeUtil (x-internal-jar:/) to field java.nio.Buffer.address
WARNING: Please consider reporting this to the maintainers of com.github.os72.protobuf351.UnsafeUtil
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
08:18:23,443 ERROR [stderr] (signalfx-metrics-publisher) [opentelemetry.auto.trace 2021-05-07 08:18:23:442 +0200] [signalfx-metrics-publisher] WARN com.splunk.javaagent.shaded.io.micrometer.signalfx.SignalFxMeterRegistry - failed to send metrics: Unable to send datapoints
08:18:50,038 ERROR [stderr] (signalfx-metrics-publisher) [opentelemetry.auto.trace 2021-05-07 08:18:50:037 +0200] [signalfx-metrics-publisher] WARN com.splunk.javaagent.shaded.io.micrometer.signalfx.SignalFxMeterRegistry - failed to send metrics: Unable to send datapoints
{08:20:19,864} ERROR [stderr] (signalfx-metrics-publisher) [opentelemetry.auto.trace 2021-05-07 08:20:19:864 +0200] [signalfx-metrics-publisher] WARN com.splunk.javaagent.shaded.io.micrometer.signalfx.SignalFxMeterRegistry - failed to send metrics: Unable to send datapoints
{08:21:49,867} ERROR [stderr] (signalfx-metrics-publisher) [opentelemetry.auto.trace 2021-05-07 08:21:49:866 +0200] [signalfx-metrics-publisher] WARN com.splunk.javaagent.shaded.io.micrometer.signalfx.SignalFxMeterRegistry - failed to send metrics: Unable to send datapoints
{08:22:19,870} ERROR [stderr] (signalfx-metrics-publisher) [opentelemetry.auto.trace 2021-05-07 08:22:19:869 +0200] [signalfx-metrics-publisher] WARN com.splunk.javaagent.shaded.io.micrometer.signalfx.SignalFxMeterRegistry - failed to send metrics: Unable to send datapoints

This is my settings what is export on startup during a script:
  export OTEL_RESOURCE_ATTRIBUTES="service.name=APP-TEST"
  export OTEL_EXPORTER_ZIPKIN_ENDPOINT="http://${IpFromAgentOnNode}:9080/v1/trace"
  export OTEL_TRACES_EXPORTER="zipkin"
  export JAVA_OPTS_APPEND="$JAVA_OPTS_APPEND -javaagent:/opt/signalfx/splunk-otel-javaagent-all.jar"

Weierd is, i recive Data in SignalFX.
only the Error Messages are not nice.
Any idea?
Thanks and Greating

i testet it with the splunk-otel-java Agent version 0.9.0 and it works fine, without any errors.

Hi @EvilAcid,
Splunk-otel-java 0.9.0 did not contain the metrics feature, that's a new thing that was introduced in 0.10.0.
If it's possible, can you enable the signalfx-agent's debug logging and send us those logs for analysis? That would help us get to the root of the problem.
Since you're using OpenShift I cannot recommend switching to Splunk distro of OpenTelemetry Collector (we still recommend using the SFx Agent there), but there are two workarounds that you can try:

You can send metrics directly to the Splunk cloud by setting SPLUNK_METRICS_ENDPOINT and SPLUNK_ACCESS_TOKEN (see the docs just below the table);
Or you can disable the metrics exporter: export SPLUNK_METRICS_ENABLED=false.

Hello @mateuszrzeszutek  Thank you for the answer.
Our security rules do not allow me to send directly to Splunk Cloud from a pod. therefore we still have the gateway. However, I will update this in the coming days to the latest version that you propose.
In the meantime I try to set the value export SPLUNK_METRICS_ENABLED = false as you described.

@mateuszrzeszutek
Your Hint with export SPLUNK_METRICS_ENABLED=false was the solution.
No more Errors and I receive all datas. 👍  Tank you


chore: test cluster versions update

Updates versions used for the test clutser, and includes a change to use a specific kind node version rather than the kubernetes version per Kind's docs.

knative_serving v1.10.1
knative_eventing v1.10.1
contour v1.10.0
kubectl v1.27.2
dapr v1.10.0
helm v3.12.0
kind v0.19.0
kind_node v1.27.1

/kind cleanup

@zroubalik please override the linter error.

/override "style / suggester / shell"


updated the ec2 instance script

updated the ec2 instance script to allow 1433 for EKS application to connect and enabled the sql server to mixed mode
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

LGTM


allow nav poly to be directly edited via the vertices. This addresses #25459

Addressing #25459: this should now expose the vertices. Update the outline and poly to the new vertices.
Known Issue:
I found that modifying the outline in the editor doesn't change the vertices value unless you reselect the node. This is equal to the behavior in OccluderPolygon2D and CollisionPolygon2D.

Erm, tried to rebase, broke everything. Still working on resolving it. 😊

I'll close for now as it's not in a mergeable state, and it might be simpler to do a new PR based on the current master branch (thus also giving the proposal more visibility than this old PR can provide).


Refactor - extract functions from generate.rs to utils.rs

Summary
This PR further refactoring generate.rs file, it's too large. Moving some of its functions to utils.rs.

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


@NotNull Optional return type should be nullable in schema

When a method return type is @NotNull Optional<Foo> that's legit in Java since the method will always return an Optional instance, but this generates a wrong GraphQL schema, i.e. Foo! which will be null if the optional is empty.

Then we would need another annotation @NotNullNotEmpty for optionals that should generate an exclamation mark, otherwise it would not be possible to generate a non-nullable schema field.

I don't get why this additional annotation would be needed. These the cases I'm considering:

Foo should be Foo
@NotNull Foo should be Foo!
Optional<Foo> should be Foo
@NotNull Optional<Foo> should still be Foo

Other cases such as Optional<@NotNull Foo> don't make sense, since this would be an optional never empty, but nullable itself: then just don't use Optional 🙂

There's @GraphQLNonNull with a sole purpose of making things non-null in the schema, so you can always use that explicitly. @NotNull has broader semantics unrelated to GraphQL, so I agree with @heruan on @NotNull Optional.

Indeed, @GraphQLNonNull solves the problem I was mentioning.


code that draws the line under a fold header

I am looking for help modifying the code, if this is not the right place to post please point me in the right direction.
I am trying to implement a new word wrap mode which will affect folding based on indent, in the same way as the python lex, but handle wrapped header lines differently.
Basically when a long wrapped line is the header of a fold, ATM the whole wrapped line - visually a few lines - are displayed and marked with an underline.
I want to change this behavior so that in this case only the part of the header line that fits in a one screen line will be visible and underlined. The rest of the header line (the wrapped part) will fold together with the rest of the text in the fold.
I am quite lost in the folding code ATM, I am especially looking for pointer to the place where the fold header line is being underlined. Any other pointers/comments are greatly appreciated.
Thanks !

For which lexer do you want to implement this? Generally, this should be handled in Scintilla and any patches should be submitted there.

Thanks ! ok, but can you point me to the code that draws the line please ?


402: PAYMENT_REQUIRED

-This Deployment has been disabled.

+Your connection is working correctly.

+Vercel is working correctly.

402: PAYMENT_REQUIRED
Code: DEPLOYMENT_DISABLED
ID: iad1::jfcqs-1694131203088-45a5965ed064

If you are a visitor, contact the website owner or try again later.
If you are the owner, please [read this documentation section](https://vercel.com/docs/errors/platform-error-codes#deployment_disabled).

The issue has been fixed!


Component seems to be "leaking" open files

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.41.0
Python release (python3 --version):
Python 3.4.2
Component/platform:
lifx
Description of problem:
Opening files without closing them. Essentially the amount of open file descriptions growing over time until "too many open files" error cripples HASS
Expected:
Close files after opening when appropriate?
Problem-relevant configuration.yaml entries and steps to reproduce:
light:
  - platform: lifx

Traceback (if applicable):

Additional info:
I acknowledge that the Unifi component is approximately twice as bad as the lifx one, but that's something that has to be deal with separately.
https://www.dropbox.com/s/93xsk3l0xodpsxe/lsof.log.gz?dl=0

Confirmed. Rediscovering a light seems to open a new file. I will fix that.
However, your light LIFX_White_800_13c0f9_AJ is responsible for 80% of the connections. Try moving it around and if it keeps acting up, it should probably be replaced.

A proper fix involves an aiolifx update but this should work for now: https://github.com/amelchio/home-assistant/blob/lifx-file-leak/homeassistant/components/light/lifx.py

Thanks for finding a fix so quickly. I've just updated to see how it goes.
And thank you for singling out the worst offender. Ironically I replaced the previous problem light with this one. It's annoying me because I have 2 other lights that are in almost the exact same position.. Meet dining area lights 1 - 3:

I've swapped two of them around to see if there is something "special" about one of those 3 positions that causes problems.

Please let me know how it goes :)
Almost 5000 disconnects seems very excessive though. Did you investigate about duplicate IP addresses on your network?

All my addresses (apart from the gateway itself) are assigned by a Unifi gateway via dhcp. I have no static addresses and no conflicts (that I am aware of).
Did the update lifx code make it into the just released home assistant? I doubt it but I thought I would ask :P

Some devices misbehave and thus you can get IP conflicts even in an all-DHCP setup. So it's best to test, especially when you have issues :). Just run arping 192.168.1.xxx for the IP address when a light has problems and verify that you only get 1 reply.
There are no LIFX updates in HA 0.42.


Can I use replace TouchableItem with my own container component?

Expected behaviour
I implemented a custom button component and want to use it on TabBar instead of the default TouchableItem. But currently there is no easy way to do this. Could you add a prop on TabBar to let users provide their own container component? For example:
<TabBar
  renderLabelContainer={(scene, onPress) => <MyCustomButton onPress={onPress} />}
/>

Hey, I just released a new alpha 2.0.0-alpha.0 of the library. It's rewritten using react-native-gesture-handler and react-native-reanimated addresses a many platform specific bugs and performance problems.
Please try the new version and see if it addresses your issue. If not, please open a new issue following the issue template.


RETURNING clause for postgresql

This PR implements postgres RETURNING clause.
This PR implements it directly in QueryBuilder, which could be quite misleading, for other sql users, so if you have any propositions regarding how this code should be arranged I can do some refactoring

Coverage decreased (-0.5%) to 97.302% when pulling 2ed0d162d38c8a3cd3bc44c0a8b4fa7cd6cf797c on Zeliboba5:master into 08c8effeba292c28476e98a4048b377a4f3a1341 on kayak:master.

Coverage decreased (-0.5%) to 97.302% when pulling 2ed0d162d38c8a3cd3bc44c0a8b4fa7cd6cf797c on Zeliboba5:master into 08c8effeba292c28476e98a4048b377a4f3a1341 on kayak:master.

If you'd like to add a vendor specific function you can override the querybuilder class and instantiate it in the PostgreSQLQuery class' _builder function. See my last PR for an example: https://github.com/kayak/pypika/pull/109

Sorry, not sure how I closed this, didn't mean to. Re-opened.

Moved everything to postgres QueryBuilder, added some more tests and covered some more clauses, now it all looks neat.
Regarding consolidation of returning and select I think it is wrong way to go, as it still different functions, which are not fully compatible (RETURNING does not support aggregate functions, so I covered it in my new commits)

Looks good to go, just need to fix the merge conflicts. I'll merge once that's done, let me know if you need help with it.

Merged and published 0.12.0


feat(FX-4285): Sticky bottom commercial section

This PR resolves FX-4285
Quick links to different artworks: https://www.notion.so/artsy/Different-artworks-5d499dfa983c4ee2b00b2b738ca4a123
Demo
https://user-images.githubusercontent.com/3513494/204780098-7963df7d-6e17-47e3-8be3-1af939e997ca.mp4
PR Checklist

[x] I tested my changes on iOS / Android.
[x] I added screenshots or videos to illustrate my changes.
[x] I added Tests and Stories for my changes.
[ ] I added an app state migration.
[x] I hid my changes behind a feature flag.
[ ] I have prefixed changes that need to be tested during a release QA with [NEEDS EXTERNAL QA] on the changelog.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

Sticky bottom commercial section - dimatretyak

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

Question: are we waiting for the #7760 to be unblocked and you will add the changes here or we are merging both of the prs when ready?

not quite. first we have to merge these changes, because we will have to move this line of code to ArtworkLotTimer


Add features: Exit after match and benchmark only flags

I've added these features in preparation for running rana in a docker container.

--bench-only Qquit after benchmarking a single core.
--exit-after Exit the program after finding a match.

NOTE: Limitation is applied to each core/thread

Adds handler for thread completion

closing due inactivity

@grunch sorry this fell off my radar! Working on merging updates and addressing your comments in a new PR.


Add picture to recipe

Nice.

you forgot to commit the image.

I am new to pull requests in git. The picture is in a different pull request here #362


Document backup/restore for K3s embedded etcd datastore

This PR addresses this issue https://github.com/rancher/k3s/issues/2224

LGTM


F&F notifications

-‘Removed as a Keeper’. Message missing || we use Friends and Family not Trusted Contacts.

The notification text has been fixed, also earlier the share wasn't getting deleted from the old trusted contact's device on doing change contact at manage-backup, now that should work fine.

An F&F contact goes from Keeper to Other Contact when Change COntact happens.


chore: use the standard owlbot.py

Towards https://github.com/googleapis/google-cloud-python/issues/10922.
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[x] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[x] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

@parthea PTAL.


VOS_joint_finetune_R50 config

Hi, thank you for your great work! However, I wonder where is the VOS_joint_finetune_R50.yaml config file you mentioned in ModelZoo？

for the VOS task


[Service Team Request] Azure CLI support for Azure Healthcare APIs

Resource Provider
HealthcareApis
Description of Feature or Work Requested
Azure Healthcare APIs is a set of managed API services based on open standards and frameworks that enable workflows to improve healthcare and offer scalable and secure healthcare solutions.
Feature overview: https://azure.microsoft.com/services/healthcare-apis
Feature documentation: https://docs.microsoft.com/azure/healthcare-apis
Minimum API Version Required
2021-11-01
Swagger Link
Feature: https://github.com/Azure/azure-rest-api-specs/tree/main/specification/healthcareapis/resource-manager
Pull request: https://github.com/Azure/azure-rest-api-specs/pull/16108
Target Date
2022-03-15

PLR

Hi @jsntcy, I'm just wondering how this work item is progressing and if it's still estimated to complete by April 26, 2022.

Yes, we planed it in this sprint and will release it around April 26.

Hi @jsntcy / @wangzelin007 / @yonzhan, I noticed that this work item is still open and its milestone has changed. Has the release been postponed to May 24, 2022?

Sorry for replying so late.
Upgrading the new API version has changed a lot.
And I found that the old version has a part of the code that is not automatically generated by codegen.
I'm looking for Mark Baker and Peiyuan to confirm how to upgrade this part.
I will release this version as soon as possible.

Hi @jsntcy / @wangzelin007 / @yonzhan, I noticed that this work item is still open and its milestone has changed. Has the release been postponed to May 24, 2022?

Sorry for replying so late.
Upgrading the new API version has changed a lot.
And I found that the old version has a part of the code that is not automatically generated by codegen.
I'm looking for Mark Baker and Peiyuan to confirm how to upgrade this part.
I will release this version as soon as possible.


vboot: use dasharo-sdk:v1.1.2, generate arv key

See: https://github.com/Dasharo/open-source-firmware-validation/pull/237#issuecomment-2031573693

Tested:
/vboot/generate_keys keys-dir
./vboot/resign pcengines_apu2_v0.9.0.rom keys-dir

NOT tested:
any new changes in binary?
booting?

Was able to perform vboot suite on APU2/3 with this revision. So the board still boots fine after this change.


bug: Fix bug in error message when no upstream exists

Apparently this was an error message that I didn't have speced and missed this at first.
Glad I found it tonight tho!
Spec added for this case!
Also the .sleet.yml file here was updated for the most current circleci config, and to take advantage of the new username flag we can use to default the username, even if this repo is forked

Yay patch releases right after other upgrades :lol:


Not connecting to remote server!

when i connect to localhost with unity it works fine, but when I upload it to my remote server it doesn't work it returns unable to connect to remote server, but when I try to connect with this site "http://amritb.github.io/socketio-client-tool/", it works just fine
app.ts:
import { SocketServer } from "./Classes/Server";
import { WebServer } from "./Web/WebServer";
import express, { Express, Request, Response } from 'express';
const PORT = process.env.PORT || 8080;
// const PORT = 8080;

const http = require('http');
const app: Express = express();
const server = http.createServer(app);

var GameServer = new SocketServer(server);
var AdminServer = new WebServer(app);

server.listen(PORT, () => {
    console.log(`Server listening on port ${PORT}`);
});

server.ts
constructor(server: http.Server) {
        this.io = new Server(server, {
            cors: {
                origin: "*"
            }
        });

        this.DatabaseHandler = new DatabaseHandler();
        this.Connections = [];
        this.Lobbies = [];

        this.io.use(async (socket: Socket, next: any) => {
            console.log("Hit!");
            .......
        });
}

2021.3.19f1, unity editor on windows (android platform), the log just says unable to connect to remote server
socket = new SocketIOUnity(uri, new SocketIOOptions {
    Query = new Dictionary<string, string>
		{
			{"id", EncryptedId},
			{"username", EncryptedUsername},
			{"avatarURL", EncryptedAvatarURL},
		}
	,
	EIO = 4,
	Transport = SocketIOClient.Transport.TransportProtocol.WebSocket,
});

Hi, I got same issue but on Windows builds, socket called connect but no response from socket about OnConnected/OnError/...

make sure the internet permission is enabled in the manifest (maybe its the cause)
<manifest xlmns:android...>
 ...
 <uses-permission android:name="android.permission.INTERNET" />
 <application ...> ... </application>
</manifest>

do the things in this comment: #45 (comment) try this if fixed ur problem.

I'm testing in unity editor, it it doesn't work, but it is working on localhost, just the remote server that is not working

Btw this is my package.json
{
  "name": "server",
  "version": "1.0.0",
  "description": "",
  "main": "app.js",
  "scripts": {
    "run": "node dist/app.js",
    "build": "tsc -p tsconfig.json"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "dependencies": {
    "crypto": "^1.0.1",
    "ejs": "^3.1.8",
    "express": "^4.18.2",
    "mysql": "^2.18.1",
    "nodemon": "^2.0.20",
    "socket.io": "^4.5.4",
    "uuid": "^9.0.0"
  },
  "devDependencies": {
    "@types/express": "^4.17.15",
    "@types/mysql": "^2.15.21",
    "@types/node": "^18.11.18",
<EMAIL_ADDRESS>"^3.0.2",
    "tsc": "^2.0.4"
  }
}

Hi, i stopped it from reconnecting only connecting and I don't get anything back no OnConnected or OnError

Fixed the issue!
just changed to port from 80 to 3001 and it worked!


Show city and postcode in search results

The search does only include the city at the results, when the POI has  an addr:city tag. This is kind of bad, because most of POIs doesn't have the addr:city tag. So searching for media markt nürnberg results in this:

So only two results are actually in Nürnberg and no "Media Markt". There are three in the OpenStreetMap database: 786620520, 145043755 and<PHONE_NUMBER>. Those all have in common, that they don't have addr:city=Nürnberg.
Nominatim does better here by getting the city and the postcode from the relations where the POI is in: https://www.openstreetmap.org/search?query=media markt nürnberg

So to resolve this issue the Qwant Maps search should return the following when searching for Media Markt Nürnberg
Media Markt
Äußere Bayreuther Straße 80, 90491 Nürnberg
Media Markt
Ipsheimer Straße, 90431 Nürnberg
Media Markt
Reichenthaler Straße, 90473 Nürnberg

The postcode is not so important as a first step. The city is key here for the search :)
For me this is an issue which "blocks" me from using Qwant Maps as well as recommending it to other people. Because apart from this I find it one of the best OSM based maps on the internet. Vector based and clicking on POIs for getting information :)

Thanks for your relevant feedback !
We are actually working on a solution to handle this kind of admins zones in Cosmogony:

https://github.com/osm-without-borders/cosmogony/pull/67
https://github.com/osm-without-borders/cosmogony/pull/84
This should solve your problem as soon as it will be released !


Create distribution type/class for unique token payouts (see desc.)

Would like a feature to only pay a BTC address that has never been included in a distribution before. For example: We want to send a token called GOLDFOLDER to someone when they have reached 10,000,000 FAH points, or perhaps 100 days of folding. Once they receive the token once, we never want to send it again to them. This is just an idea we have, let us know how possible it is.

8 SP - Includes spec and implementation


Fix build timing issue for catkin

Changes proposed in this pull request:

Fix build dependency issue introduce with PR #204

sorry but what is it about ????
On Mon, Mar 20, 2017 at 11:41 PM, Mark Horn<EMAIL_ADDRESS>wrote:

Changes proposed in this pull request:

Fix build dependency issue introduce with PR #204
https://github.com/intel-ros/realsense/pull/204

You can view, comment on, or merge this pull request online at:
https://github.com/intel-ros/realsense/pull/219
Commit Summary

Fix build timing issue for catkin

File Changes

M realsense_camera/CMakeLists.txt
https://github.com/intel-ros/realsense/pull/219/files#diff-0 (5)

Patch Links:

https://github.com/intel-ros/realsense/pull/219.patch
https://github.com/intel-ros/realsense/pull/219.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/intel-ros/realsense/pull/219, or mute the thread
https://github.com/notifications/unsubscribe-auth/AQrq25wdnKkNGibo-u4hIqp3gX9ppQ3Pks5rnw6sgaJpZM4MjJ73
.


adding a couple details to local install guide

we need to add back in the mention of collision for the gitops and metaphor-frontend repositories


Add support for custom prefix in script name in JSR223 script engine

read attribute "#jsr223.groovy.engine.script.name.prefix" from engine scope script context
keep Script + counter to avoid name collision

Can you open your PR on the official repo: apache/groovy


System builder syntax rework

Existing PR: #2736
Original issue: #1978

Rendered.

Are we happy to fundamentally alter the end user experience of Bevy and basically make all existing docs out of date.

Yep! This change is only going to get more painful the longer we wait.

IMO there's an important bit of Future work that should be included: this would provide an excellent unified foundation for "label descriptors", a much-discussed future feature that would allow users to specify schedule and run criteria information on labels, which would then be dispatched to each system with that label.

IMO there's an important bit of Future work that should be included

By included do you mean the changes required for label descriptors be made on top of this pr before merging it?

"Future work" refers to a subsection of the RFC where you note down stuff that could happen after this RFC is accepted and implemented.
I don't think label properties are relevant here, though. It's mostly an orthogonal feature, in both API and implementation.

Alright, I accept @Ratysz's judgement on that.

I have a couple of ideas about the add_exclusive(system.exclusive_system()) API:

Changing it to add_exclusive(system): Yeeting .exclusive_system() since it is assumed that the system is exculsive.
Changing it to add_system(system.exclusive_system()): Removing add_exclusive method and specifying the exclusiveness through the builder-like pattern.

I'm quite ignorant to that part of the codebase so I don't know if the proposed APIs are possible or the implementation effort broaden too much the scope of the RFC/PR.

Changing it to add_exclusive(system): Yeeting .exclusive_system() since it is assumed that the system is exclusive.

I'd be quite happy with this, although I'd prefer add_exclusive_system for clarity. @DJMcNab, you ran into some issues with yeeting .exclusive_system though right?

@DJMcNab, you ran into some issues with yeeting .exclusive_system though right?

That was a while ago, I've fixed that. This breaks it again; haven't had the time to try and fix it yet.

I didn't even try to get rid of .exclusive_system - to be honest, my PR was mostly just about recognising the for<'a> &'a mut F: FnMut(...) was the way to remove needing .system

So how's this going. Has cart's prerequisite stuff been merged?

Fixed by #45.


Send Email Authentication Email send issues or  unclear about this.

Here i see  you use this routes
router.post('/send-verification-email', auth(), authController.sendVerificationEmail);
but I have some question. this is used where the    didn't active their account and token   validate date expires then need to send token again  .
But you use auth(). how it work? i think passport use token and check the user is authenticate or not. but here I see you use.  how the user get token without login or  validate their account?

The /send-verification-email route is used when a user wants to verify their email. The user has to be logged in for their email to be retrieved; that's why there is an auth() middleware. The access token is just used to confirm the user is logged in. The user will then get an email with a verify-email token which will verify their email once clicked.

i see you sent a verify-email and sent a token using params.
how it works and how I can get this token. I am not clear about this process.
first, when the user completes registration he gets a verifying email for email verification,  then he clicks the link and token add /verify-email?token=token, when he clicks this link he verifies this email. but how work /send-verification-email?

Please go through the code and you will get the answers to your questions. The 'Issues' section is not for getting explanation of how the code works; it's for reporting a problem with the code.
That said, you get the token from the email as I said in my previous comment.


Typescript Types?

Are there any typescript typings or any plans to make them?
If not I might whip some up.

There are not, but this would be very welcome.
If you could provide an example (even if it's just a set of instructions here!) of how to check they work and are correct to make maintain easier that would be great!

Sounds good. I'll submit a PR an a bit.
I don't know of any good way to check that they work with a JavaScript project besides manually looking over them. But they just need to be updated whenever a module, function or object signature changes.

Being tracked in #220

Yup just the last part of publishing the types missing, I'll try to give it a push 🙏🏻


Open images in new tab

Ref: https://github.com/bigbinary/neeto-planner-web/issues/1924

@AbhayVAshokan _a


Content-Type header is dropped

Using axios, I discovered this is the underlying library that handles redirects, so 👋 .
The API I'm communicating with follows JSON API which describes that clients must add Content-Type: application/vnd.api+json headers to their requests.
I have noticed though that when performing a redirect, this library will remove any header that starts with content-*.
This results in my requests failing, with a 415 Unsupported Media Type response (as per spec).
Just wondering what the intentions were behind this? Is there a workaround to "reinsert" headers such as Content-Type later?
Thanks!

Looks related to the fix implemented to fix https://github.com/olalonde/follow-redirects/issues/49. Wonder if it can be dialled back to just removing the Content-Length header?

Woops, definitely wrong to drop Content-Type. Should drop other things such as encoding etc., not just length.

No, wait, this is intended behavior.

I have noticed though that when performing a redirect, this library will remove any header that starts with content-*.

Not correct: only for a 307, where the entity must be dropped.

where the entity must be dropped.

Is that in the spec somewhere? I checked RFC 7231, but didn't find anything that specified that exactly?

I've documented this in the source code:

Automatic redirection needs to done with
care for methods not known to be safe, as defined in Section 4.2.1,
since the user might not wish to redirect an unsafe request.

— RFC 7231, $6.4
This document further says:

Early user agents split on whether the
method applied to the redirect target would be the same as the
original request or would be rewritten as GET.  Although HTTP
originally defined the former semantics for 301 and 302 (to match
its original implementation at CERN), and defined 303 (See Other)
to match the latter semantics, prevailing practice gradually
converged on the latter semantics for 301 and 302 as well.  The
first revision of HTTP/1.1 added 307 (Temporary Redirect) to
indicate the former semantics without being impacted by divergent
practice.  Over 10 years later, most user agents still do method
rewriting for 301 and 302; therefore, this specification makes
that behavior conformant when the original request is POST.

So note that this issue is not “xxx header is dropped”, but rather “non-safe methods are rewritten into GET on 301/302/303”, and this is intended behavior.

Thanks for your patience, I understand now, thanks!


Is it really only compatible with ember 4+?

Just noticed the peerDependencies are ember-source 4+, can't it also be 3.28?

I'm not sure. We only support the latest versions from the ember-cli blueprints, but it is definitely possible it works with earlier versions.


Close the sheet, but the component remains mounted with opacity 0.

Hello.
Under normal operation, when the sheet is closed, its internal components, such as the divs with classes 'react-modal-sheet-container' and 'react-modal-sheet-content', along with other internal components, disappear, causing the main sheet to take on the 'hidden' class.
However, in some instances, when the sheet is closed, it takes on the 'hidden' class but still retains the aforementioned internal components. In this case, the sheet closes, but the internal components remain mounted with opacity set to 0, causing the application to freeze since there is a layer above that is blocking the rest of the functionality.
Best regards.

Hi @jafelixeducaria  👋🏻
Unfortunately without a reproduction this is very difficult to fix.
I've added some tolerance to various value comparisons to avoid situations like this. Try updating to the latest version and hopefully that will help 🙂


Support matplotlib>=3.6

Currently some of the plotting tests fail
====================================================================== FAILURES =======================================================================
____________________________________________________________ test_plot_2D_line_rect_region ____________________________________________________________
Traceback (most recent call last):
  File "/home/vid18871/projects/scanspec/tests/test_cli.py", line 148, in test_plot_2D_line_rect_region
    assert result.exit_code == 0
AssertionError: assert 1 == 0
 +  where 1 = <Result MatplotlibDeprecationWarning('Passing the angle parameter of __init__() positionally is deprecated since Matplotlib 3.6; the parameter will become keyword-only two minor releases later.')>.exit_code
__________________________________________________________________ test_plot_3D_line __________________________________________________________________
Traceback (most recent call last):
  File "/home/vid18871/projects/scanspec/tests/test_cli.py", line 216, in test_plot_3D_line
    assert len(arrow_artists) == 4
AssertionError: assert 0 == 4
 +  where 0 = len([])

@tizayi

for the test test_plot_3D_line.
get_default_bbox_extra_artists()  which is used in the test was changed in between matplotlib 3.5.3 and 3.60.
in 3.53
`
def get_default_bbox_extra_artists(self):
"""
Return a default list of artists that are used for the bounding box
calculation.
    Artists are excluded either by not being visible or
    ``artist.set_in_layout(False)``.
    """

    artists = self.get_children()

    if not (self.axison and self._frameon):
        # don't do bbox on spines if frame not on.
        for spine in self.spines.values():
            artists.remove(spine)

    if not self.axison:
        for _axis in self._get_axis_list():
            artists.remove(_axis)

    artists.remove(self.title)
    artists.remove(self._left_title)
    artists.remove(self._right_title)

    return [artist for artist in artists
            if (artist.get_visible() and artist.get_in_layout())]

`
and in 3.60
`
def get_default_bbox_extra_artists(self):
"""
Return a default list of artists that are used for the bounding box
calculation.
    Artists are excluded either by not being visible or
    ``artist.set_in_layout(False)``.
    """

    artists = self.get_children()

    for axis in self._axis_map.values():
        # axis tight bboxes are calculated separately inside
        # Axes.get_tightbbox() using for_layout_only=True
        artists.remove(axis)
    if not (self.axison and self._frameon):
        # don't do bbox on spines if frame not on.
        for spine in self.spines.values():
            artists.remove(spine)

    artists.remove(self.title)
    artists.remove(self._left_title)
    artists.remove(self._right_title)

    # always include types that do not internally implement clipping
    # to Axes. may have clip_on set to True and clip_box equivalent
    # to ax.bbox but then ignore these properties during draws.
    noclip = (_AxesBase, maxis.Axis,
              offsetbox.AnnotationBbox, offsetbox.OffsetBox)
    return [a for a in artists if a.get_visible() and a.get_in_layout()
            and (isinstance(a, noclip) or not a._fully_clipped_to_axes())]

`
This removes a large number of artists from the bounding box calculation. The test can use get_children() directly instead to test if 4 instances _Arrow3D exist.


Use last_description instead of last_comment for Rake 11

Rake 11 removed last_comment. It should be replace last_description
see. https://github.com/ruby/rake/blob/master/History.rdoc

This is a blocking issue. Please consider to merge and release a patch version. Thank you. :+1:

@jodosha this shouldn't be blocking unless you absolutely need Rake 11, which looking at the ChangeLog, shouldn't be the case. Locking to rake ~> 10.0 should be a reasonable workaround.

@lsegal Thanks for the quick reply :tophat:. Sure, locking Rake works, but I'm sure that there are a lot of developers out there with the same problem.
Instead of let the rest of us to lock Rake to ~> 10.0, it would be nice if you can make it to work with latest Rake.
All the best. And once again, thanks for YARD. :green_heart:

Seems this was deprecated a while back, so the patch should be safe.

CI is failing as Rake 11 doesn't support Ruby 1.8.7 anymore.

@lsegal Thanks for merging this 💚 .


Some fixes and a new feature

Improve indentation around multiline assignments, comma fixup feature, plus some minor updates.

Let me know if you'd like me to create issues for the indentation fixes.

Thanks for this PR. I noticed that one unit test failed (indent assignment statement (with comment)).
How do you handle this case?
CONSTANT Vcc : SIGNAL := --logic 1 constant
'1';
CONSTANT zero4 : bit_vector(0 TO 3) :=--test
('0', '0', '0', '0');

The expected behavior would be:
CONSTANT Vcc : SIGNAL := --logic 1 constant
    '1';
CONSTANT zero4 : bit_vector(0 TO 3) := --test
    ('0', '0', '0', '0');

Sorry for the long delay, busy with other work stuff right now :/
I had failed to run the jest testcases so I missed this. I found the bug but while doing so uncovered an inconsistency in my approach that I'd like to fix before pushing another set of changes. Coming up soon!

I've added the jest into the github action. So if you merge master into your pr, github will check for you automatically.


Ability to connect multiple users to a session at the same time

Join session

@HRNasr what protocol do you refer to? RDP, VNC or SSH? It's notable that Windows RDP only allows one simultaneous session. It's a limitation of the OS.

@HRNasr I don't think that's possible for RDP + Guacd


Issue 3287: Refactoring the test_script_result table to use react js.

As per discussion with David on Slack.
(Code is not completed yet)

Merged in squashed commit 1f59bff304f31da08cda7c865baf00f302b164cf.


Spoiler prop for blocks to add a spoiler treatment

Create a blur over block contents with a spoiler prop for blocks

Can we build it as a molecule? Shouldn't be that difficult technically, just wonder if it should be a part of the Blocks or not


<EMAIL_ADDRESS>decompress error

  D:\ scoop install avidemux
WARN  Purging previous failed installation of avidemux.
ERROR 'avidemux' isn't installed correctly.
Removing older version (2.8.1).
'avidemux' was uninstalled.
Installing 'avidemux' (2.8.1) [64bit] from extras bucket
Loading Avidemux_2.8.1.VC++.64bits.exe from cache
Checking hash of Avidemux_2.8.1.VC++.64bits.exe ... ok.
Extracting dl.7z ... ERROR Exit code was 2!
Failed to extract files from C:\Users\Andrea\scoop\apps\avidemux\2.8.1\dl.7z.
Log file:
~\scoop\apps\avidemux\2.8.1\7zip.log
Please try again or create a new issue by using the following link and paste your console output:
https://github.com/lukesampson/scoop-extras/issues/new?title=avidemux%402.8.1%3A+decompress+error

#9689

So the solution is:

In the meantime you could install the nightly version avidemux-nightly from the versions bucket


Implement tile events

In particular, safe zones (amongst others) -- e.g. in the arena zone's (48) center area.
Upon first moving onto a tile (or spawning in on one) it'll show:
7642	This is a safe zone.

To the info box.

This issue was moved to ApexKO/OpenKO#56


react-native 0.69.1 - Error: No available storage method found.

I've updated react-native to 0.69.1 and after that i get the following error: "Error: No available storage method found."
I'm using the driver from https://github.com/aveq-research/localforage-asyncstorage-driver. I don't know if it's related here or in the other repository.
My code looks like:
import {driverWithDefaultSerialization} from '@aveq-research/localforage-asyncstorage-driver'
import localforage from 'localforage'

...

const initLocalForage = async () => {
    const driver = driverWithDefaultSerialization()
    await localforage.defineDriver(driver)
    await localforage.setDriver(driver._driver)
    localforage.config()
  }

It worked before with react-native 0.68.2.

I use it in Angular 14 and having same issues, but only in firefox. Chrome and chromium Browsers are all fine.

I think this would be an upstream issue with @aveq-research/localforage-asyncstorage-driver, because localForage itself, using the default drivers, is still working. Can you check with the upstream package, since we don't have any drivers that support React Native built into the localForage core package.


Proposal: Reorganize XAgent Documentation Structure

Overview
After reviewing several successful open-source projects, it's evident that a streamlined documentation structure can have significant benefits. One such exemplary project is Vue.js and Vue.js doc, which separates its core codebase from its documentation. In line with this, I propose that we reorganize XAgent's documentation in a similar fashion.
Benefits

Clarity and Focus: By separating documentation from the main codebase, contributors can easily identify where to make changes without sifting through code files.
Ease of Maintenance: A dedicated documentation repository ensures updates and changes are centralized, making it easier to manage versioning and history.
Scalability: As the project grows, the volume of documentation can increase significantly. A separate repository ensures that this doesn't clutter the main project.

Proposed Changes

Utilize the XAgent-doc Repository:

The repository for documentation has been initialized and can be found here.
Migrate any remaining or updated documentation from the XAgent repo to XAgent-doc, ensuring that the content aligns with the latest changes and updates.

Update XAgent README.md:
# XAgent

[Description of XAgent]

## Documentation

For detailed documentation, guides, and API references, please visit `XAgent-doc`.

Populate XAgent-doc Repository:

Structure the documentation similar to Vue.js doc, with clear sections, guides, API references, etc.
Ensure all links and references within the documentation are updated to reflect the new structure.
As the repository has already been created and content initially modified, continue to refine and expand upon the documentation as needed.

Next Steps

Review this proposal and provide feedback.
Upon approval, begin the migration process and make necessary updates.
Announce to the contributor community about the change for transparency and to keep everyone informed.

Your feedback is essential. Please share any thoughts, concerns, or suggestions to refine this proposal further.

Maybe put the repo under OpenBMB?

I'm ok with other stuff.


0.16 Alpha 30th Oct

Proposed change

Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:

Checklist

[ ] The code change is tested and works locally.
[ ] Local tests pass. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.
[ ] The code has been formatted using Ruff (ruff format frigate)

npm build

npm build

npm run build


Eduardo cv

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: eajose sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Add a way to quickly reset the search

Some possibilities:

a "reset" search button
make the MetaSRA logo clickable to point to a fresh search

Done 1df7a6992b4f6e1f38c3352336cdc729b7834fea


Fix double restart when --watch is used

When tns livesync <platform> --watch is executed, the application is restarted twice as typescript's watch command is compiling the sources again.
CLI starts the watch command last, after the app is already started on the device. The watch command is compiling the .ts files to .js. CLI detects these changes and restarts the app.
In order to fix this, make the watch hook to return a promise. CLI will not start "watching" for file changes until the promise is resolved.
The watch hook's promise will be resolved when the following message is printed by TypeScript compiler: "Compilation complete. Watching for file changes.".

:+1:


some minor edits

Thanks for the edits. I just have one comment and after that, we can approve and merge.

Oh ok I see. I just made those edits!

On Jun 1, 2022, at 12:09 AM, Kevin Kho @.***> wrote:
@kvnkho commented on this pull request.
In tutorials/beginner/index.md https://github.com/fugue-project/tutorials/pull/138#discussion_r886301205:

FugueSQL
-We'll show a bit of FugueSQL, the SQL interface for using Fugue. This is targeted for heavy SQL users and SQL-lovers who want to use SQL on top of Spark and Dask, or even pandas. FugueSQL that is used on DataFrames in memory as opposed to data in databases.
+We'll show a bit of FugueSQL, the SQL interface for using Fugue. This is targeted at heavy SQL users and SQL-lovers who want to use SQL in addition to Spark and Dask, or even pandas. FugueSQL is used on DataFrames in memory as opposed to data in databases.
Python and SQL are the interface and Spark and Dask are the backends so "on top of" is technically more right than "in addition", but happy to change the wording to make that clearer too.
—
Reply to this email directly, view it on GitHub https://github.com/fugue-project/tutorials/pull/138#pullrequestreview-991371276, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKAOXJFNE6HBCZ4R6UGIJZ3VM3PAJANCNFSM5XOLK7IA.
You are receiving this because you authored the thread.


DAT-336-fix-RemoveNoneOrEmptyElement-to-remove-None-element-completely

Current converter RemoveNoneOrEmptyElement will remove None key/value to empty dict, which should be remove completely.
DAT-336

@ake2l Hi Alex, I changed RemoveNoneOrEmptyElementConverter class to 2nd option that you gave, and change the test. Please check it again, thank you.


ICF results are 20% down of what is should be

Something in the implementation is wrong

Testing is being performed. Also Weka results are being double checked, just in case the configuration was wrong.

For some datasets now the ICF is correct, for others it is not.


state=SSLv2/v3 read server hello A: sslv3 alert handshake failure

Hi,
since a few days the toolbelt stopped working on my box. Any idea what's up? This is the latest version as far as i can see.
[root@SG1 pagzSitesProd]# heroku ps
Installing Heroku Toolbelt v4... !    Heroku client internal error.
 !    Search for help at: https://help.heroku.com
 !    Or report a bug at: https://github.com/heroku/heroku/issues/new

    Error:       SSL_connect returned=1 errno=0 state=SSLv2/v3 read server hello A: sslv3 alert handshake failure (OpenSSL::SSL::SSLError) (Excon::Errors::SocketError)
    Command:     heroku ps
    Plugins:     heroku-vim
    Version:     heroku-toolbelt/3.42.17 (x86_64-linux) ruby/1.9.3
    Error ID:    dfb817cac6014b34b16f2662aca0cb53

    More information in /root/.heroku/error.log

Heroku client internal error.
SSL_connect returned=1 errno=0 state=SSLv2/v3 read server hello A: sslv3 alert handshake failure (OpenSSL::SSL::SSLError)
/usr/local/heroku/vendor/gems/excon-0.45.4/lib/excon/ssl_socket.rb:120:in `connect_nonblock'
/usr/local/heroku/vendor/gems/excon-0.45.4/lib/excon/ssl_socket.rb:120:in `initialize'
/usr/local/heroku/vendor/gems/excon-0.45.4/lib/excon/connection.rb:387:in `new'
/usr/local/heroku/vendor/gems/excon-0.45.4/lib/excon/connection.rb:387:in `socket'
/usr/local/heroku/vendor/gems/excon-0.45.4/lib/excon/connection.rb:106:in `request_call'
/usr/local/heroku/vendor/gems/excon-0.45.4/lib/excon/middlewares/mock.rb:47:in `request_call'
/usr/local/heroku/vendor/gems/excon-0.45.4/lib/excon/middlewares/instrumentor.rb:22:in `request_call'
/usr/local/heroku/vendor/gems/excon-0.45.4/lib/excon/middlewares/base.rb:15:in `request_call'
/usr/local/heroku/vendor/gems/excon-0.45.4/lib/excon/middlewares/base.rb:15:in `request_call'
/usr/local/heroku/vendor/gems/excon-0.45.4/lib/excon/middlewares/base.rb:15:in `request_call'
/usr/local/heroku/vendor/gems/excon-0.45.4/lib/excon/connection.rb:233:in `request'
/usr/local/heroku/vendor/gems/excon-0.45.4/lib/excon.rb:229:in `get'
/usr/local/heroku/lib/heroku/jsplugin.rb:183:in `manifest'
/usr/local/heroku/lib/heroku/jsplugin.rb:196:in `url'
/usr/local/heroku/lib/heroku/jsplugin.rb:125:in `setup'
/usr/local/heroku/lib/heroku/cli.rb:24:in `start'
/bin/heroku:24:in `<main>'

I'm not really sure, but that version of SSL is deprecated and shouldn't be used anymore. Perhaps try reinstalling? What OS is this on?

I am running the dev env on centos5 with the default ssl.

Try reinstalling and maybe updating your ruby

alright i upgraded to ruby 2.2.3 since i am running it in rvm anyway so it doesn't impact the system and it fixed it:
[root@SG1 ~]# ruby -v
ruby 2.2.3p173 (2015-08-18 revision 51636) [x86_64-linux]
pretty odd tho, i thought 1.9.3 wasn't that old.
thanks a bunch .

it's not that 1.9.3 is old, it's that you had it compiled against an openssl that has expired ssl algorithms


aliased icons

Hi, I just tested the app, very nice an easy to use but I think it would be better to enable smoothing when scaling the icon.
I use a 1024x1024 icon as source and all smaller icons are aliased, I think smoothing would fix this.
Or maybe add a checkbox to choose smoothing option ?

Greetings,
Thank you for reporting this issue.
I have uploaded a new build of this app at: http://phantom.im/apps/IconGenerator.air
This build creates the PNG's with smoothing and high quality anti-aliasing.
Please test this build and report back if it fixed the issue.

Hi,
great I think it's much better :)

this app is great, it would be very nice to have also the support for the app screenshots ;-)

One more thing to say, for sizes below or equals to 120px, the icon start to be blurry. I tried with a bicubic interpolation and the results are a little betters for these sizes. Have a look, on the right generated image with your app, and on the left generated image with bicubic interpolation, you will see on smaller size the result is a little less blurry:

Here is the script I used for bicubic interpolation:
/**
 * Resize an Image with bicubic interpolation
 * @param width the new desired width
 * @param height the new desired height
 * @param bitmapData the original sized BitmapData
 * @return
 */
public function interpolateBicubic( width:int, height:int, bitmapData:BitmapData ):BitmapData
{
	// same size no need to resize
	if( width == bitmapData.width && height == bitmapData.height )
	{
		return bitmapData.clone();
	}

	var interpolated    :BitmapData = new BitmapData( width, height, true, 0x00000000 );
	var original_copy   :BitmapData = bitmapData.clone();

	var xFactor:Number = bitmapData.width / interpolated.width;
	var yFactor:Number = bitmapData.height / interpolated.height;

	if (xFactor > 1 || yFactor > 1)
		original_copy.applyFilter(original_copy, original_copy.rect, new Point(0, 0), new BlurFilter(1.4*(xFactor/2),1.4*(yFactor/2),BitmapFilterQuality.HIGH));

	var step:int = 0;
	while ( step < interpolated.width )
	{
		for (var y:int = 0; y < interpolated.height; y++){
			interpolated.setPixel32(step, y, getPixelBicubic32(step * xFactor, y * yFactor, original_copy));
		}
		step++;
	}

	return interpolated;
}

private function getPixelBicubic32( x:Number, y:Number, bitmapData:BitmapData ):Number
{
	var i:int = int(x);
	var j:int = int(y);

	if (((i - 1) < 0) || ((j - 1) < 0))
		return bitmapData.getPixel32(i, j);
	else if (((i + 1) >= bitmapData.width) || ((i + 2) >= bitmapData.width) ||
			((j + 1) >= bitmapData.height) || ((j + 2) >= bitmapData.height))
		return bitmapData.getPixel32(i, j);

	var dx  :Number = x - i;
	var dy  :Number = y - j;
	var asum:Number = 0;
	var rsum:Number = 0;
	var gsum:Number = 0;
	var bsum:Number = 0;

	var Rmdx:Array = new Array(4);
	var Rdyn:Array = new Array(4);
	for( var m:int = -1; m <= 2; ++m )
		Rmdx[m + 1] = A(m - dx);
	for( var n:int = -1; n <= 2; ++n )
		Rdyn[n + 1] = A(dy - n);

	for( m = -1; m <= 2; ++m )
	{
		for (n = -1; n <= 2; ++n)
		{
			var rgb :int = bitmapData.getPixel32(i + m, j + n);
			var a   :int = (rgb >> 24) & 0x0FF;
			var r   :int = (rgb >> 16) & 0x0FF;
			var g   :int = (rgb >>  8) & 0x0FF;
			var b   :int = (rgb >>  0) & 0x0FF;

			var Rres:Number = Rmdx[m + 1] * Rdyn[n + 1];
			asum += a * Rres;
			rsum += r * Rres;
			gsum += g * Rres;
			bsum += b * Rres;
		}
	}

	var alpha:int = (int)(asum + 0.5);
	if(alpha < 0)
		alpha = 0;
	else if(alpha > 255)
		alpha = 255;

	var red:int = (int)(rsum + 0.5);
	if(red < 0)
		red = 0;
	else if(red > 255)
		red = 255;

	var green:int = (int)(gsum + 0.5);
	if(green < 0)
		green = 0;
	else if(green > 255)
		green = 255;

	var blue:int = (int)(bsum + 0.5);
	if(blue < 0)
		blue = 0;
	else if(blue > 255)
		blue = 255;

	return (alpha << 24) | (red << 16) | (green << 8) | (blue << 0);
}

private static function A( x:Number ):Number
{
	var p0:Number = ((x + 2) > 0) ? (x + 2) : 0;
	var p1:Number = ((x + 1) > 0) ? (x + 1) : 0;
	var p2:Number = (x > 0) ? x : 0;
	var p3:Number = ((x - 1) > 0) ? (x - 1) : 0;

	return (1 / 6) * (p0 * p0 * p0 - 4 * (p1 * p1 * p1) + 6 * (p2 * p2 * p2) - 4 * (p3 * p3 * p3));
}

@binouze Thank you very much for your in-depth comment. I will be trying this approach as well with another PNGEncoder. I plan to create a new release this weekend.
@pitanello Thanks for the idea! Do you need the app to take screenshots from ADL or only scale them (like the launch images)?

I was thinking about feeding the app with already made screenshots and let it scale in different resolutions.

@pitanello Sure, I will research what sizes are needed and add it to next version.

@binouze Hi, I have been testing with different combinations and I think I found out the best results, you can test it in this link:
http://phantom.im/apps/IconGenerator2.air
Btw, using StageQuality.BEST caused the blurry effect. I have removed that and used another encoder, you can see the new changes in the latest commit.

Hi, the result is better than before but I think the bicubic interpolation is still better (for my icon) ;)

NOTE:
PNG is a lossless format so i don't think changing the PNGEncoder will change the result output
Maybe adding a parameter for choosing the scaling interpolation method ? (like in Photoshop when you make a resize)

Thanks for testing it. I will use your code instead and release a new build afterwards.
It's my first time working with graphics stuff (I'm more of a RCP dev), I really appreciate your input.

Happy to help,
I have made an "app" to make the icons for my games but I did it fast, all params are hard coded and I always have to modify them to generate the icons.
It's nice to see people that take time to built nice util tools like this ;)

Hi,
I have added the Bicubic Interpolation option in the new release.
I noticed when using it, it takes a bit longer to generate the icons, so it's turned off by default, but can easily be turned on in the Settings screen.
While testing it with the Launch Images it took a very long time to generate them, so I'm using the default algorithm for those.

OK, good thing to let the user decides to activate it.
Thanks


Submitting transactions constructed by cardano-wallet API endpoint are failing

Just checking...

[X] This is a cardano-wallet bug.
[X] I am using the latest cardano-wallet release.
[X] I am using the correct cardano-node version for that release of cardano-wallet.

Version
d9a6a19cd3447eb4d92b5337fe24e0e4925d3897
Platform
NixOS
Installation method
Nix
Network configuration
testnet<PHONE_NUMBER>
Context
cardano-wallet does not seem to recognize it's own encoding after signing when trying to submit.
After signing this transaction like so:
curl --header "Content-Type: application/json" --request POST --data '{"passphrase":"somepassword", "transaction":"hKYAgoJYIB96zaPcSvcb1MJAd0wXAWgnnhOIuMvzt2LK7xO4Fb3vAIJYIFovk4iEhkIefZTs/JV6puCGLzeoI2J7w2wGGMcPkFsmAA2AAYSCWB1wIJh+F3oCLkS/qzHLykvqVNAzGfT3W6ro/amTcBoAHoSAglg5AAO0uAaR3PMqFhft9AUzV4L9gz2A3qVlrLoFNYQOiTw08j5ffb1xsPEuE8y1aiycPjR29LRsKFwFGgTgqROCWDkAA7S4BpHc
8yoWF+30BTNXgv2DPYDepWWsugU1hA6JPDTyPl99vXGw8S4TzLVqLJw+NHb0tGwoXAUaAVxiUoJYOQBlAo+Rfsxa+CGhikFjJG8xXhoxQfzwpcZ8pElyDok8NPI+X329cbDxLhPMtWosnD40dvS0bChcBRo1S8H2AhoAArplAxoCk/PzDoCg9fY="}' http://localhost:8090/v2/wallets/fc685e4b0c373139720d3c755d35927d2d648d31/transactions-sign

{"transaction":"hKYAgoJYIB96zaPcSvcb1MJAd0wXAWgnnhOIuMvzt2LK7xO4Fb3vAIJYIFovk4iEhkIefZTs/JV6puCGLzeoI2J7w2wGGMcPkFsmAA2AAYSCWB1wIJh+F3oCLkS/qzHLykvqVNAzGfT3W6ro/amTcBoAHoSAglg5AAO0uAaR3PMqFhft9AUzV4L9gz2A3qVlrLoFNYQOiTw08j5ffb1xsPEuE8y1aiycPjR29LRsKFwFGgTgqROCWDkAA7S4BpHc8yoWF+30BTNXgv2DPYDepWWsugU1hA6JPDTyPl99vXGw8S4TzLVqLJw+NHb0tGwoXAUaAVxiUoJYOQBlAo+Rfsxa+CGhikFjJG8xXhoxQfzwpcZ
8pElyDok8NPI+X329cbDxLhPMtWosnD40dvS0bChcBRo1S8H2AhoAArplAxoCk/PzDoChAIGCWCByaR3v5uFgYKd0T0nv5yJlrb88TDmfd9KKvSnmHc9vK1hAFg+eVD3Cx2ZVHiLSnAPDTySTDXS/zHZRGmVTBD4XKCt5iEGxU1qYcH3twe2unFv2L3LHclQJ+E1EiBm09sjRA/X2"}

There does not seem to be an obvious way to submit this transaction other than this
Which I then attempt to use like so, which doesn't work:
curl --header "Content-Type: application/octet-stream" --request POST --data 'hKYAgoJYIB96zaPcSvcb1MJAd0wXAWgnnhOIuMvzt2LK7xO4Fb3vAIJYIFovk4iEhkIefZTs/JV6puCGLzeoI2J7w2wGGMcPkFsmAA2AAYSCWB1wIJh+F3oCLkS/qzHLykvqVNAzGfT3W6ro/amTcBoAHoSAglg5AAO0uAaR3PMqFhft9AUzV4L9gz2A3qVlrLoFNYQOiTw08j5ffb1xsPEuE8y1aiycPjR29LRsKFwFGgTgqROCWDkAA7S4BpHc8yoWF+30BTNXgv2DPYDepWWsugU1hA6JPDTyPl99vXGw8S
4TzLVqLJw+NHb0tGwoXAUaAVxiUoJYOQBlAo+Rfsxa+CGhikFjJG8xXhoxQfzwpcZ8pElyDok8NPI+X329cbDxLhPMtWosnD40dvS0bChcBRo1S8H2AhoAArplAxoCk/PzDoChAIGCWCByaR3v5uFgYKd0T0nv5yJlrb88TDmfd9KKvSnmHc9vK1hAFg+eVD3Cx2ZVHiLSnAPDTySTDXS/zHZRGmVTBD4XKCt5iEGxU1qYcH3twe2unFv2L3LHclQJ+E1EiBm09sjRA/X2' http:
//localhost:8090/v2/proxy/transactions

{"code":"bad_request","message":"DecoderErrorDeserialiseFailure 'Shelley Tx' (DeserialiseFailure 0 'expected list len or indef')"}

Figured I'd give the cardano-wallet CI a try directly, which also doesn't work:
./result/bin/cardano-wallet transaction submit hKYAgoJYIB96zaPcSvcb1MJAd0wXAWgnnhOIuMvzt2LK7xO4Fb3vAIJYIFovk4iEhkIefZTs/JV6puCGLzeoI2J7w2wGGMcPkFsmAA2AAYSCWB1wIJh+F3oCLkS/qzHLykvqVNAzGfT3W6ro/amTcBoAHoSAglg5AAO0uAaR3PMqFhft9AUzV4L9gz2A3qVlrLoFNYQOiTw08j5ffb1xsPEuE8y1aiycPjR29LRsKFwFGgTgqROCWDkAA7S4BpHc8yoWF+30BTNXgv2DPYDepWWsugU1hA6JPDTyPl99vXGw8S4TzLVqLJw+NHb0tGwoXAUaAVxiUoJYOQBlAo+Rfsxa+CGhikFjJG8xXhoxQfzwpcZ8pElyDok8NPI+X329cbDxLhPMtWosnD40dvS0bChcBRo1S8H2AhoAArplAxoCk/PzDoChAIGCWCByaR3v5uFgYKd0T0nv5yJlrb88TDmfd9KKvSnmHc9vK1hAFg+eVD3Cx2ZVHiLSnAPDTySTDXS/zHZRGmVTBD4XKCt5iEGxU1qYcH3twe2unFv2L3LHclQJ+E1EiBm09sjRA/X2

Parse error. Expecting Base16-encoded format.

Thought that what was returned by /transactions-sign endpoint would have handled the conversion for me according to the documentation. Seems like it didn't. I went ahead and base16 encoded the transaction I received from /transaction-sign endpoint as well and passed it into cardano-wallet cli and it also failed.
/result/bin/cardano-wallet transaction submit 684B5941676F4A59494239367A61506353766362314D4A41643077584157676E6E684F49754D767A74324C4B37784F344662337641494A5949466F766B346945686B4965665A54732F4A5636707543474C7A656F49324A3777327747474D63506B46736D414132414159534357423177494A682B46336F434C6B532F717A484C796B7671564E417A476654335736726F2F616D5463426F41486F5341676C673541414F307541615233504D71466866743941557A56344C39677A32413371566C724C6F464E59514F69547730386A356666623178735045754538793161697963506A5232394C52734B4677464767546771524F4357446B41413753344270486338796F57462B333042544E58677632445059446570575773756755316841364A50445479506C393976584777385334547A4C56714C4A772B4E4862307447776F5841556141567869556F4A594F51426C416F2B52667378612B434768696B466A4A47387858686F7851667A7770635A3870456C79446F6B384E50492B58333239636244784C68504D74576F736E443430647653306243686342526F315338483241686F414172706C41786F436B2F507A446F436841494743574342796152337635754667594B643054306E7635794A6C7262383854446D6664394B4B76536E6D486339764B31684146672B655644334378325A5648694C536E415044547953544458532F7A485A52476D5654424434584B437435694547785531715963483374776532756E4676324C334C48636C514A2B45314569426D3039736A52412F5832

cardano-wallet: INTERNAL ERROR: unsafeSealedTxFromBytes: DecoderErrorDeserialiseFailure "Shelley Tx" (DeserialiseFailure 0 "expected list len or indef")
CallStack (from HasCallStack):
  error, called at src/Cardano/Wallet/Util.hs:78:21 in cardano-wallet-core-2021.11.11-8b6Yzqmnz8K52rjbKp3LB4:Cardano.Wallet.Util
  internalError, called at src/Cardano/Wallet/Primitive/Types/Tx.hs:924:35 in cardano-wallet-core-2021.11.11-8b6Yzqmnz8K52rjbKp3LB4:Cardano.Wallet.Primitive.Types.Tx
  unsafeSealedTxFromBytes, called at src/Cardano/Wallet/Api/Client.hs:338:27 in cardano-wallet-core-2021.11.11-8b6Yzqmnz8K52rjbKp3LB4:Cardano.Wallet.Api.Client

How is this suppose to work?
Description
Steps to Reproduce

Use transactions-construct endpoint to create a transaction
Use transaction-balance endpoint to balance the transaction
Use trasnasction-sign endpoint to sign the transaction
Submit the transaction using either proxy/transactions endpoint or cardano-wallet transaction submit cli command
...

Expected behavior
Expect the transaction to be submitted.
Actual behavior
Errors out because transaction encoding provided by the cardano-wallet endpoint is invalid

Atm, signed transaction needs to be encoded to binary format before passing to proxy/transactions. Check Submit transaction in https://input-output-hk.github.io/cardano-wallet/user-guide/common-use-cases/pab.


airthings : guard against missing pm25 value in sensor data

Description
If the airtag does not report pm25 then the script crashes when trying to access.  This fix check for the presence of the data and if it's missing set it to -1 and then display n/a when rendering.
Copilot

🤖 Generated by Copilot at 2820be1
Summary
🐛🌬📱

Improve airthings app compatibility and formatting. Handle missing pm25 data and format it as a string in apps/airthings/airthings.star.

The airthings app shows the air quality
But some devices lack the ability
To measure pm25
So we handle that case
And format it as a string nicely

Walkthrough

Add a check for pm25 key in samples dictionary and assign -1 if missing (link)
Convert pm25 variable to 'n/a' string if it has a value of -1 for display (link)

Unrelated to this particular change, but it'd be great if someone could update the app to output an illustrative default graphic when no configuration has been provided. For previews and such.


Verify units of dx and dy as returned by bmi_noahowp%get_grid_spacing

Current behavior
Calling bmi_noahowp%get_grid_spacing returns (/model%domaingrid%dy, model%domaingrid%dx/) (i.e., via bmi_grid%spacing). This behavior is consistent with BMI 2.0 specifications. However, as the units of dx and dy are not specified in the BMI 2.0 documentation, there is ambiguity regarding the appropriateness of the current implementation.
When executed in standalone (outside NextGen), dx and dy are read-into the model via the required domain attributes NetCDF file (which is expected to have global attributes dx and dy). The example attributes file (i.e., data/NorthForkAttributes.nc) contains a subset of the NWM 3.0 grid for the North Fork (WY) CAMELS test basin; hence, dx = dy = 1 km.
Suggested changes
Verify current implementation of bmi_noahowp%get_grid_spacing or amend implementation of dx and dy as needed.

@drakest123 Care to expand upon or edit this issue?


Add missing WITH-VEC4

Probably missing as a result of d9689b9965f5a4e6ef3d2bf7df9162b3f4438431.

It's just with-vec now, no need for the special version.


Model.php Return Type Object causes Error when Saving. (Fixed with this modification)

Currently if you use your models to return data types of object instead of array, you will be hit with an error once you try to use the save() method.
When checking if its an object is_object, its looking for a non-stdClas. To resolve these in model.php you need to remove all the !
if (is_object($data) && ! $data instanceof \stdClass)
{
     $data = $this->classToArray($data);
}

Corrected by:
if (is_object($data) && $data instanceof \stdClass)
{
     $data = $this->classToArray($data);
}

Otherwise you end up with Cannot use object of type stdClass as array This needs to be fixed on the save() , insert() , and update() methods.

Looks like it needs to be updated on the update() method, not on the save() method otherwise it causes it to create new entries in the database. I haven't tested on insert() yet, but just so you're aware that it causes an error because it still thinks its an array.


deploy.py script calls register_application() even if the application exists

Information

Onefuzz version: all
OS: all

The deploy.py script calls register_application() even though the 'onefuzz-cli' app exists in the current tenant. This results in the creation of a new secret each time you invoke deploy.py if you are the Owner of the 'onefuzz-cli' in your tenant. The result is that now our 'onefuzz-cli' App Registration has hundreds, soon to be thousands of unused secrets, see the image below.
Provide detailed reproduction steps (if any)
Create a new OneFuzz instance on any tenant other than Microsoft, e.g. a personal Azure subscription or the PME tenant.
The resulting behavior is different if you are the Owner or not of the 'onefuzz-cli' app on your tenant, even though the script succeeds in both cases. Which is pretty strange.
This issue is caused by the fact that the ONEFUZZ_CLI_APP is hard coded to the Microsoft tenant. It should be changed so that if the user provides a client_id as a command line argument then script should use that app ID instead of the global variable when applying the filter (sorry, I did not test this solution):
OLD CODE:
cli_app = list(client.applications.list(filter="appId eq '%s'" % ONEFUZZ_CLI_APP))
NEW CODE:
cli_app = list(client.applications.list(filter="appId eq '%s'" % self.cli_config["client_id"]))
In my opinion, if you create a new instance using deploy.py, it should create a new app registration "onefuzz-cli-INSTANCE_NAME", instead of attempting to reuse a single one per tenant. It would be a simpler model and users will understand things clearly.
There are a couple different issue at hand here and it's not my intention to muddy the water. I just suggest they're all handled at the same time. If you have any questions please feel free to reach out over Teams directly.
SECRETS:

(it goes on for several pages)
AB#39994232

There are two concerns in this issue

The deployment scripts add a new password entry on each deployment.

This is addressed in the PR #966 - The new code limits the number of passwords for an application to two

The deployment script ignores the client_id parameter. A new PR will be created to handle that issue

The last outstanding issue has been addressed in #2305


CcStrict should not touch urn() or anonymous identity (#184)

CcStrict should not touch urn() or anonymous identity.  (https://github.com/yegor256/takes/issues/184)

Wrapped CcStrict.appyrules with condition
Tested CcStrict.decode of anonymous identity

@cyberone Thanks for your pull request, let me find someone who can review it

@carlosmiranda review this one, please

@rultor good to merge

@rultor good to merge

@carlosmiranda Thanks for your request. @yegor256 Please confirm this.

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 16min)

@elenavolokhova please, review this task for compliance with our quality rules

@rultor deploy pls

@rultor deploy pls

@davvd OK, I'll try to deploy now. You can check the progress here

@rultor deploy pls

@davvd Done! FYI, the full log is here (took me 9min)

@carlosmiranda
According to our quality rules:

Pull request description explains the solution proposed and contains a link to the original ticket it is related to.

There is no link to original ticket in description. Please, fix it.
According to our quality rules:

The code reviewer found at least three problems in the code.

There were no issues found during review. We should get used to this rule as soon as possible.
Please confirm that you will try to find at least three problems next time.

@cyberone Please link to the original ticket in the description. @elenavolokhova We will find three or more issues next time.

@carlosmiranda done

@cyberone Thanks!

@elenavolokhova All issues are addressed and everything is clear to us, we'll work better next time.

@carlosmiranda @cyberone Thank you!

@davvd Quality is acceptable.

@davvd Quality is acceptable.

@elenavolokhova thanks a lot, next time everybody should try to make it better

@carlosmiranda 10 mins was added to the account of @elenavolokhova (for QA review), in transaction 56387847. 19 mins sent to your balance (ID 56387850), many thanks! It took 47 hours and 35 mins.. you're getting extra minutes here (c=4). +19 added to your rating, at the moment it is: +4565


docs: change log updated

Description

change log updated

Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review:

[x] DEPENDENCIES are up-to-date. Dash license tool. Committers can open IP issues for restricted libs.
[x] Copyright and license header are present on all affected files

@ajithsimons  What is know defects?

@almadigabor  Please review and merge.


Update app.php

Added the chinese language.

Hi there,
Please create a single Pull Request for the translation of the BookingProduct package.


Restrict character set in tag names to allow easy encoding as HTTP headers

As it occurred in https://github.com/Wikia/opentracing-php/issues/3
The basictracer implementation in Go defines tags here:
https://github.com/opentracing/opentracing-go/blob/master/ext/tags.go

var (
	// PeerXXX tags can be emitted by either client-side of server-side to describe
	// the other side/service in a peer-to-peer communications, like an RPC call.

	// PeerService records the service name of the peer
	PeerService = stringTag("peer.service")

	// PeerHostname records the host name of the peer
	PeerHostname = stringTag("peer.hostname")

...

If one would like to put the tag names directly as HTTP headers he would need to take an extra effort to encode the dots.
It would be great if that could be avoided.

@wladekb a wider range of chars is currently allowed in tags, which are intended to be reported with the span and not intended to be serialized across the wire, as opposed to trace attributes (to be renamed to "baggage"), which are intended to be serialized across the wire and propagated in band with the trace.
Can you share a use-case for sending tag names as HTTP headers?

@wladekb any thoughts about @dkuebric's question above (which I share)?

I would consider the suggestion invalid after I got better understanding of what is the primary purpose for tags and baggage (trace attributes) and what goes where. Feel free to close it.

Sold :) Thanks.


docs: Add 404 page with sphinx-notfound-page

Closes https://github.com/meltano/sdk/issues/2038

:books: Documentation preview :books:: https://meltano-sdk--2039.org.readthedocs.build/en/2039/

A bit better:


GPflow2 likelihood tests as well as likelihoods are buggy

I tried to increase code coverage and ended up fixing some tests in #1079 - is_analytic in develop-2.0 actually returns the opposite of what it said (actual_method is fallback_method when it should be is not!), and it did not take into account that likelihoods not necessarily overwrite all of variational_expectations, predict_density, and predict_mean_and_var - but now the tests don't pass anymore. Would be great if someone else could help investigate what's going on there!

MWE is in #1083.

It's just the tests that are broken, and #1083 that fixes them is ready to merge.


Add sending of trace messages to another process

Add sending of trace messages to another process.
Add printp/1 function to pretty print redbug trace messages externally.

this got a bit stale...


Could 'gulp build' css or js in one file?

After use gulp build,will create a lot of file like:
accordion.min.css
accordion.min.js
ad.min.css
api.js
api.min.js
breadcrumb.css
breadcrumb.min.css
button.css
button.min.css

but,I am using webpack,so,I have to import them file by file.
i think is unnecessary,because when  during run gulp install  , I already select what component I need
Is there some way could build all components in one file?
like:
semantic-ui.min.js
semantic-ui.min.css

I find!
Just run gulp 'package compressed css'

close #3776

Closed #3776


[GHA][LBT] add repro cmd to PR comment

Motivation
Include the cluster test cmd used in the land-blocking test workflow to help repro failures locally.
Test Plan
Canary result

:v: @sausagee can now approve this pull request

@bors-libra r=andll

:pushpin: Commit d6ff8e4 has been approved by andll

:hourglass: Testing commit d6ff8e4d7f2265e4359c2064a2a696a35d9bb79c with merge 69ff70554a530774943c1860795f8a25abb56676...

:broken_heart: Test failed - checks-circle_commit_workflow

@bors-libra retry

:hourglass: Testing commit d6ff8e4d7f2265e4359c2064a2a696a35d9bb79c with merge a28039d04220de1bebe24636e40643feeb086542...

:broken_heart: Test failed - checks-circle_commit_workflow

@bors-libra retry

:hourglass: Testing commit d6ff8e4d7f2265e4359c2064a2a696a35d9bb79c with merge d6398cb047102217335c8073f34bc5046d88e6f4...

:broken_heart: Test failed - checks-circle_commit_workflow

@bors-libra r=andll

:pushpin: Commit 1800f44 has been approved by andll

:hourglass: Testing commit 1800f4445d3e7870cf5f24495e5748343b116a54 with merge e22bce84be64dd7ada1dabb819443ef4f86bbb12...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: andll
Pushing e22bce84be64dd7ada1dabb819443ef4f86bbb12 to master...


data transformation pipeline from base to STARTERFRAME

STARTERDATAFRAME: transformations (take a source dataframe, apply the transformations that you want to try out, put it all into 1 big nested frame with columns 'transformation.name', 'train.X', 'train.Y'

I'm working on this one, stuck on renaming the dataframes


add real-time crypto price tracker script

Pull Request: Add Real-Time Cryptocurrency Price Tracker Script
Description
This pull request introduces a Python script that uses CoinGecko public API to retrieve real-time information of a particular cryptocurrency such as current price, 24-hr price change, and historical data. Please refer to CoinGecko
coins list for the full list.
Related Issues:
Implement Real-Time Cryptocurrency Price Tracker
Changes Made:

Added main.py, requirements.txt, runtime.txt in Real-Time Cryptocurrency Price Tracker directory

Type of Change

[X] ✨ New feature (added crypto tracker functionality)

Checklist

[X] Created a new branch (feature/crypto-tracker) for this PR
[X] Followed the coding style and conventions of the project
[X] Manually tested the script for various inputs
[X] No merge conflicts

Hi @king04aman, I have updated the script with README. For your review, thanks!


Pulling packages from different Channels: bioconda, conda-forge

There's no documented way of adding channels such as bioconda and conda-forge to a project. There are a lot of packages that are only available through these.

Duplicate of https://github.com/astral-sh/uv/issues/1703


"?boolean" in Table.d.ts triggers typescript build error

Describe the bug
When I try to build my code using typescript it won't compile. I try building using "tsc -p tsconfig.json". On Typescript v4 the error I receive is "JSDoc types can only be used inside documentation comments." and in Typescript v5 the error is "'?' at the start of a type is not valid TypeScript syntax. Did you mean to write 'boolean'?"
The error is occurring in node_modules/dynamodb-onetable/dist/mjs/Table.d.ts:100:29
To Reproduce
I created this repo to reproduce the bug. https://github.com/melissarh57/dynamodb-onetable-error/blob/main/README.md
when running the command npm run build these are the output logs
`

<EMAIL_ADDRESS>build
tsc -p tsconfig.json

node_modules/dynamodb-onetable/dist/mjs/Table.d.ts:17:30 - error TS2307: Cannot find module 'aws-lambda' or its corresponding type declarations.
17 import {DynamoDBRecord} from 'aws-lambda'
~~~~~~~~~~~~
node_modules/dynamodb-onetable/dist/mjs/Table.d.ts:100:29 - error TS8020: JSDoc types can only be used inside documentation comments.
100         options?: {nothrow: ?boolean}
~~~~~~~~
Found 2 errors in the same file, starting at: node_modules/dynamodb-onetable/dist/mjs/Table.d.ts:17
`
Steps to reproduce the behavior:

Clone the repo listed above.
run 'npm run build'
see error log in console.

Expected behavior
Can import Table and compile via typescript without errors
Environment (please complete the following information):

Node Version 16
OneTable Version: 2.6.3
TypeScript Version: 5.1.6 using "strict"

Additional context
Please let me know if I missed any important info
Here is the offending code in Table.d.ts

Thank you. Fixed.


Bump dbf version to fix bug

see https://github.com/mapbox/dbf/issues/30

Found another issue and pushed another fix to this branch.
Also, I don't love the fact that to deploy this change I have to merge this PR and then just deploy an unchanged version of overyield — instead I'll push this to a better-named branch and then make a PR on the Overyield side to update which branch we use in package.json.