OpenAPI 3 / 3.1 handling of binary content

Description
OpenAPI 3 has dropped the file type. It looks like this is not reflected here.
When using type: file in a schema:
"content": {
  "application/zip":  {
    "schema": {
      "type": "file"
    }
  }
}

the corresponding typescript type is created as (Blob | File). Which is the expected behaviour for OpenApi v2 but the above is not a valid OpenAPI 3 / 3.1 spec as the type: file is not supported anymore.
In OpenAPI 3:

Content transferred in binary (octet-stream) MAY omit schema

"content": {
  "application/zip":  {}
}

Or use type: string with contentEncoding keyword.
"content": {
  "application/zip":  {
    "schema": {
      "type": "string",
      "contentEncoding": "base64"
    }
  }
}

However, this will generate typescript types of either unknown or string.
OpenAPI specification (optional)
{
  "openapi": "3.1.0",
  "paths": {
    "/foo": {
      "get": {
        "responses": {
          200: {
            "content": {
              "image/png":  {
                "schema": {
                  "type": "string",
                  "contentMediaType": "image/png",
                  "contentEncoding": "base64"
                }
              }
            }
          }
        }
      }
    }
    "/bar": {
      "get": {
        "responses": {
          200: {
            "content": {
              "application/zip":  {}
            }
          }
        }
      }
    }
  }
}

Configuration
No response
System information (optional)
No response

Thanks for reporting @PaulStryck, will check

Hey @PaulStryck, mind adding what's the expected result here?

Thanks for looking into it.
I would expect to get (Blob | File) type annotations when using OpenAPI v3 specs.
Currently these type annotations are produced:
export type GetFooResponse = (Blob | File);
export type GetBarResponse = (Blob | File);

when using the following OpenAPI spec:
{
  "openapi": "3.1.0",
  "info": {
    "version": "1.0.0",
    "title": "Test",
    "license": {
      "name": "Apache 2.0",
      "identifier": "Apache-2.0"
    }
  },
  "servers": [
    {
      "url": "http://localhost/"
    }
  ],
  "paths": {
    "/foo": {
      "get": {
        "responses": {
          200: {
            "description": "get foo",
            "content": {
              "image/png":  {
                "schema": {
                  "type": "file"
                }
              }
            }
          }
        }
      }
    },
    "/bar": {
      "get": {
        "responses": {
          200: {
            "description": "get bar",
            "content": {
              "application/zip":  {
                "schema": {
                  "type": "file"
                }
              }
            }
          }
        }
      }
    }
  }
}

"type": "file" is only allowed in OpenAPI v2.
The correct OpenAPI v3 spec produces:
export type GetFooResponse = string;
export type GetBarResponse = unknown;

with the following OpenAPI spec:
{
  "openapi": "3.1.0",
  "info": {
    "version": "1.0.0",
    "title": "Test",
    "license": {
      "name": "Apache 2.0",
      "identifier": "Apache-2.0"
    }
  },
  "servers": [
    {
      "url": "http://localhost"
    }
  ],
  "paths": {
    "/foo": {
      "get": {
        "responses": {
          200: {
            "description": "get foo",
            "content": {
              "image/png":  {
                "schema": {
-                 "type": "file
+                 "type": "string",
+                 "contentMediaType": "image/png",
+                 "contentEncoding": "base64"
                }
              }
            }
          }
        }
      }
    },
    "/bar": {
      "get": {
        "responses": {
          200: {
            "description": "get bar",
            "content": {
              "application/zip":  {
-               "schema": {
-                 "type": "file"
-               }
              }
            }
          }
        }
      }
    }
  }
}

Any response object with an omitted schema object for a "file like" mime type should generate a (Blob|File) type and set the Blob.type property accordingly, when parsing the response.
Any response with a populated schema object for a "file like" mime type should either generate a string type, or a (Blob|File) type and parse the base64 string to a Blob object directly in the request method. The latter is possibly a question of personal taste. I'd prefer parsing to Blob.

Let's start with the initial iteration which I know won't make you happy just yet. For application/zip I made it as you described, although we could discuss that later too. I want to focus on the image/png response. The specification defines it as string, it will come back as a base64 encoded string, and the appropriate TypeScript type for that is also string, which is what gets generated. Is any of this incorrect?
If not, it seems your main desire to generate Blob | File is to preserve the output with prior versions. I would need to understand what's happening there. How do you go from string to a file? What are you doing with this response in your application?


Kata deploy 1.2 update

automated and improved tarball has slightly different format.  Update kata-deploy to make use of the new dockerfile (which uses this tarball).

/cc @jcvenegas


where did the "--strategy" be defined?

I downloaded the newest codes, and try to train a model using moco, but it reported an error:
main_pretrain.py: error: unrecognized arguments: --strategy ddp
I try to find the parser argument named "--strategy", but I didn't find it.  Is it a bug?

Hey @ludysama you need to update pytorch lightning versions.

Thank you， it works。


metrics-generator metrics don't contain juju_ prefix in their labels

Bug Description
When enabling metrics-generator to generate span-based metrics, we inject juju toplogy into those metrics, but they get injected without juju_ prefix
To Reproduce
1- Deploy Tempo HA https://discourse.charmhub.io/t/topic/15489
2- Deploy prometheus-k8s
3- integrate coordinator with prometheus over remote-write-endpoint
4- Deploy grafana-k8s
5- integrate grafana and prometheus over grafana-source
6- open grafana dashboard
7- open sources and check metrics that start with span-, you'll see labels missing the prefix
Environment
microk8s 1.28
juju 3.5.4
Relevant log output
N/A

Additional context
No response

Should use label_matcher_dict


Support BMS DR-JC03?

Describe the bug
I'm trying gather all things together for:
Victron MultiPlus 48/5000/70 + Cerbo GX + Battery LiFePO4 with BMS DR-JC03.
I've installed your app, and connected
Cerbo GX-USB <-> USB-RS485 Adapter with 3 pins (A/B/Ground) <-> RS485-RJ-45 on battery
On RJ-45(RS485):
1- RS485B
2- RS485B
3- GND
(I made this according to this manual. It's not so suitable, but didn't find better.)
Am I wrong somewhere?
To Reproduce
Steps to reproduce the behavior:

Go to Cerbo GX terminal
Install dbus-serialbattery
Connect the battery
See logs: tail -f /data/log/dbus-serialbattery.ttyUSB0/current
See error

ERROR:SerialBattery:ERROR >>> No battery connection at /dev/ttyUSB0

Expected behavior
Logs shall show success connection.
Screenshots
BMS

 BMS DR-JC03 Photos

VenusOS (please complete the following information):

Device type: Cerbo GX
Firmware Version: 2.92 (latest firmware for 2022-12-09)

Battery/BMS (please complete the following information):

BMS/Battery type: ??? / LFP
Cells: 16
Interface: 2xRJ45-RS485, 1xRJ45-RS232

Additional context

This is not a BMS that is supported by die driver. I cant's find any communication protocol information on their website, so I can't say if it would be possible to add.

Hi @Louisvdw ,
Thanks for your quick response.
But how these 3 wires to RJ45 (RS495) shall be connected to a battery via RS485/RJ45?
Is there some standard or something?
As the solution 1-A+ / 2-B- / 3-GND does not work, I found this adapter USB-RS485/RJ45
https://www.amazon.com/RS-485-Adapter-Serial-BCBL-01-Communication/dp/B08GLFBTZW
According to the picture: Green-GND, Red-A+, Black-B-.

So, on RJ-45(RS485):

GND
RS485 A+
RS485 B-

Error No battery connection at /dev/ttyUSB0

 Full log
```
@400000006393172b34f8ebb4 INFO:SerialBattery:Starting dbus-serialbattery
@400000006393172b35055f34 INFO:SerialBattery:dbus-serialbattery v0.141
@400000006393172b350f80f4 INFO:SerialBattery:Testing LltJbd
@400000006393172c0a5dffd4 ERROR:SerialBattery:>>> ERROR: No reply - returning
@400000006393172c0adb0b8c INFO:SerialBattery:Testing Ant
@400000006393172c1bc2d59c ERROR:SerialBattery:>>> ERROR: No reply - returning
@400000006393172c1c403744 ERROR:SerialBattery:>>> ERROR: Incorrect Data
@400000006393172c1c4a2254 INFO:SerialBattery:Testing Daly
@400000006393172c2e399094 ERROR:SerialBattery:>>> ERROR: No reply - returning
@400000006393172c2ebfd3ac INFO:SerialBattery:Testing Daly
@400000006393172d0421ced4 ERROR:SerialBattery:>>> ERROR: No reply - returning
@400000006393172d0496df94 INFO:SerialBattery:Testing Jkbms
@400000006393172d157be2b4 ERROR:SerialBattery:>>> ERROR: No reply - returning
@400000006393172d15f0d434 INFO:SerialBattery:Testing Lifepower
@400000006393172d276c182c ERROR:SerialBattery:>>> ERROR: No reply - returning
@400000006393172d27e00fac ERROR:SerialBattery:>>> ERROR: Incorrect Data
@400000006393172d27ece8bc INFO:SerialBattery:Testing Renogy
@400000006393172d38d95a34 ERROR:SerialBattery:>>> ERROR: No reply - returning
@400000006393172d3957487c INFO:SerialBattery:Testing Renogy
@400000006393172e0e88f474 ERROR:SerialBattery:>>> ERROR: No reply - returning
@400000006393172e0f071d54 INFO:SerialBattery:Testing Ecs
[...]
@400000006393173419ad9db4 ERROR:SerialBattery:ERROR >>> No battery connection at /dev/ttyUSB0
```

That's a web page of my battery: 512KWH-48V-lifepo4

The wires does not matter if the driver does not talk the language of that battery.
If this battery can do CANbus you might not need the serial battery driver.

I got no CANbus input, unfortunately.

The wires does not matter
But I have to establish right connection first, right?

Yes, you do need to get the correct wire connection first. But there are no standard for that. You will have to check your battery manual which pins are used.
But even if the correct pins are used and the connection is valid, the driver will not know how to talk to this battery unless it use the same BMS as one of the supported BMS.

But there are no standard for that.

That's a shame :(

You will have to check your battery manual which pins are used.

No manual from XD battery vendor. :( 🤦
Thanks for this note, L!

Contacted with the XD Battery company. I was told, it can be connected with their battery (w/o programming).
They promised to help to make it work. My hands are itching to see the manual.
I'll bring updates...

Diese Batterie funktioniert mit dem Treiber nicht. Du musst folgende Dinge ausführen. https://we.tl/t-pgNDHMLJ7y (the link for the battery manual, pc software, BIN code , upgrade video)
1.upgrade the battery by the BIN code first.
2.battery connect victron by CAN bus (the RJ45 cables PIN change for CAN)
3.master battery dip 5 and 6 on
4.victron setting pylon working model

@joggl, thanks for the answer. Why it's not in english (completely)? :-/
What this link about? I see "Introducing portraits". I guess, you added wrong link.

Didn't get it. What does "BIN code" mean?
2,3) Yes, 5 & 6 pins must be up. For some reason it works only via CAN bus which is configured as BMS CAN.

That's the answer from the battery vendor. This manual helped me to connect my battery.

Probably, will add detailed manual later.

Hello undkirby, i am from Germany, i try to write in english. I have a battery with the same BMS from an other battery maker. The program from louis doesn't work with this BMS. In the delivered version of my battery the BMS can not comunicate with victron and I have to install an Update on my battery ( this means bin-code). In the link are the updatefile, a PC-software to bring the update on the battery and choose some settings, a movie who show the way of update and settings and a manual (PDF). I had to go this way and no my Raspberry Pi with venus os comunicate with the battery. Best Regards

Hello @joggl,
it looks like you did a piece of work!
So, the link either is broken or invalid or not allowed to see from my side. I see the following:

Transfer expired
Sorry, this transfer has expired and is not available any more

I assume, something went wrong... =) Maybe you have another source of this video.
My questions to your comment:

Where this update (installed on your battery) is coming from?
How to did you install it?
Which version do you have now?

BMS can not communicate with victron

It somehow communicates with my Cerbo. Does it work via BMS Can on your side?
I have "Settings > Services > VE.Can port > Can Bus profile" = "CAN-bus BMS (500 kb/s)".

This is the manual of my battery. The update for my battery I get from my seller. I only connect 2 cables on my can-hat.

NEW-48V LIFEPO4 BATTERY USER MANUAL.pdf

I don't understand how i can do the Video in GitHub. So i do it this way .

Best Regards

 Kirby Rs<EMAIL_ADDRESS>hat am 29.01.2023 20:07 CET geschrieben:

Hello @joggl,
it looks like you did a piece of work!
So, the link either is broken or invalid or not allowed to see from my side. I see the following:
Transfer expiredSorry, this transfer has expired and is not available any more

I assume, something went wrong... =) Maybe you have another source of this video.
My questions to your comment:
Where this update (installed on your battery) is coming from?
How to did you install it?
Which version do you have now?

BMS can not communicate with victron

It somehow communicates with my Cerbo. Does it work via BMS Can on your side?I have "Settings > Services > VE.Can port > Can Bus profile" = "CAN-bus BMS (500 kb/s)".

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

I don't understand how i can do the Video in GitHub. So i do it this way .
I assume, youtube would be the best place ;)

We started to collect all available information (manuals, firmware version and so on) over here https://github.com/LeoWinterDE/BMS. Feel free commit any new firmware version or guide, thanks.

This is the manual of my battery. The update for my battery I get from my seller. I only connect 2 cables on my can-hat (CANH and CANL -> Pin 4+5).  NEW-48V LIFEPO4 BATTERY USER MANUAL.pdf

Hello @joggl,
it looks like you did a piece of work!
So, the link either is broken or invalid or not allowed to see from my side. I see the following:

Transfer expired
Sorry, this transfer has expired and is not available any more

I assume, something went wrong... =) Maybe you have another source of this video.
My questions to your comment:

Where this update (installed on your battery) is coming from?
How to did you install it?
Which version do you have now?

BMS can not communicate with victron

It somehow communicates with my Cerbo. Does it work via BMS Can on your side?
I have "Settings > Services > VE.Can port > Can Bus profile" = "CAN-bus BMS (500 kb/s)".

We started to collect all available information (manuals, firmware version and so on) over here https://github.com/LeoWinterDE/BMS. Feel free commit any new firmware version or guide, thanks.

I'm not allowed to commit directly, but this here seems to be an older version of the protocol: https://www.photovoltaikforum.com/core/attachment/77134-pylon-lfp-battery-communication-protocol-rs485-v2-8-20161216-pdf/

We started to collect all available information (manuals, firmware version and so on) over here https://github.com/LeoWinterDE/BMS. Feel free commit any new firmware version or guide, thanks.

This is the manual of my battery. The update for my battery I get from my seller. I only connect 2 cables on my can-hat (CANH and CANL -> Pin 4+5).  NEW-48V LIFEPO4 BATTERY USER MANUAL.pdf

Hello @joggl,
it looks like you did a piece of work!
So, the link either is broken or invalid or not allowed to see from my side. I see the following:

Transfer expired
Sorry, this transfer has expired and is not available any more

I assume, something went wrong... =) Maybe you have another source of this video.
My questions to your comment:

Where this update (installed on your battery) is coming from?
How to did you install it?
Which version do you have now?

BMS can not communicate with victron

It somehow communicates with my Cerbo. Does it work via BMS Can on your side?
I have "Settings > Services > VE.Can port > Can Bus profile" = "CAN-bus BMS (500 kb/s)".

Hi. After all of this information i still have a question :) I am using a raspberry pi with venus OS. So this type of bms works only with waveshare can pi hat ? I have a USB -Rs485 adapter and i failed to connect this bms with raspberry pi. So i have to buy a waveshare can pi hat ? The adapter i use right now is the one in the picture, please see the link down below. Thanks.
[https://user-images.githubusercontent.com/127730319/224959954-41155586-4ca1-456b-9dff-6ebe6ba1ebe7.jpg]

@MarzaBogdan i have only been able to run it with a pi Hat on the venus os, but technically it should also work with a RS485 USB adapter - but I haven't seen this yet.

@MarzaBogdan i have only been able to run it with a pi Hat on the venus os, but technically it should also work with a RS485 USB adapter - but I haven't seen this yet.

I have to install or activate any driver on venus os ?

@MarzaBogdan i have only been able to run it with a pi Hat on the venus os, but technically it should also work with a RS485 USB adapter - but I haven't seen this yet.

I have to install or activate any driver on venus os ?

This pi hat from waveshare runs without any additional drivers on venus os.

@MarzaBogdan i have only been able to run it with a pi Hat on the venus os, but technically it should also work with a RS485 USB adapter - but I haven't seen this yet.

I have to install or activate any driver on venus os ?

This pi hat from waveshare runs without any additional drivers on venus os.

Ok thanks, I will try again with my rs485 adapter and if it still not working i will buy that pi hat waveshare. The thing about my rs485 adapter is i have only 3 ping A B and GND. On wave share i see H L B A.
Thanks for your quickly response

For me it only works the can hat too. I have to Insert a few words at config.txt. See on waveshare wiki.
https://www.waveshare.com/wiki/RS485_CAN_HAT


[WIP][ZEPPELIN-3600] Change logger level dynamically

What is this PR for?
Enabling users to change the level of a specific logger to investigate Zeppelin server's status.
What type of PR is it?
[Feature]
Todos

[x] - Add api to change the level of logger
[x] - Add exception handler

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-3600

How should this be tested?

http --session ~/session.txt --form POST http://localhost:8080/api/login userName=admin password=password1
http --session ~/session.txt POST http://localhost:8080/api/admin name=org.apache.zeppelin.rest.AdminRestApi level=DEBUG

Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? May be

Will merge if there's no discussion
https://travis-ci.org/jongyoul/zeppelin/builds/402472726

maybe it's better to clarify that we are adding the Admin REST API in this change and also allow logging lever to be changed?

Will merge it


Issue on run_matmul

Describe the bug
Currently I'm do some inference on my laptop using this python package. I found an issue described below.
I use the following code to test the result for classical matrix multiple operation:
import torch
from intel_npu_acceleration_library.backend.runtime import run_matmul
a = torch.randn(1000, 1000, dtype=torch.float16)
b = torch.randn(1000, 1000, dtype=torch.float16)
c1 = run_matmul(a, b.transpose(0, 1))
c2 = torch.matmul(a, b)
assert c1.equal(c2)
The difference between c1 and c2 is huge!
I don't know why?
Test environment:
Ubuntu 24.04, Intel Arc Graphics with NPU enabled.
I checkout this commit from github and install it locally: 3e3dee344e98eee6134c74059c0f218a988dd545
Also I have installed required NPU drivers and it can leverage NPU well because the inference is much faster than pure CPU.

Few questions:

What driver version do you have?
Do you have the same behaviour in this example?

I suspect the issue here is in the b.transpose(0, 1) part

The driver versions are as below:
/usr/lib/x86_64-linux-gnu/libze_intel_gpu.so.1.3.29735
/usr/lib/x86_64-linux-gnu/libze_intel_vpu.so.1.8.0
/usr/lib/x86_64-linux-gnu/libze_loader.so.1.17.44
/usr/lib/x86_64-linux-gnu/libze_tracing_layer.so.1.17.44

I've tried this example, the result is also different:
Here is my code:

def run_matmul(inC, outC, batch):
# Create both inputs
X1 = np.random.uniform(-1, 1, (batch, inC)).astype(np.float16)
X2 = np.random.uniform(-1, 1, (outC, inC)).astype(np.float16)

mm = MatMul(inC, outC, batch, profile=False)

intel = mm.run(X1, X2)
intel = torch.tensor(intel)

tor = torch.matmul(torch.tensor(X1), torch.tensor(X2).transpose(-2, -1))
print(tor.equal(intel))

Because the shape for torch.matmul() is different with run_matmul(), so I transpose it.


Uml 879 add a lpa pact

Purpose
Utilises Chris' work to implement Pact mock testing of the Add an LPA to an account feature.
Fixes UML-879
Approach
I moved all the LPA context steps to a new context file and added a trait that implements pact testing features. That trait is used in the LPA context file by the 2 functions needed to test this part of the ticket.
It does need an alteration to the underlying pact behat extension that has been submitted as an upstream change (https://github.com/Raffers/pact-behat-extension/pull/1).
We will need to pull the above repo into the MoJ account and setup the code to use that fork once that is in place.
Learning
Upstream code does some weird things sometimes.
Checklist

[x] I have performed a self-review of my own code
[ ] I have added relevant logging with appropriate levels to my code
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[x] I have added tests to prove my work
[ ] The product team have tested these changes

Coverage remained the same at 81.735% when pulling 2a911bae7ab102d1b829fe3f299298a18714ae1e on UML-879-add-a-lpa-pact into 13852aff4bd9e1fa8da53d3e6b3679cb16966649 on master.


Prevent feedback loops through parser errors

The new parser repeats part of the comment in its error messages so it can highlight the syntax error. This is great, except for the fact that in production the comment posted by Hoff also comes in as a webhook event. And since the error message contains the repeated merge command, Hoff would enter a feedback loop where it complains about parsing errors in its own comments. The tests now also simulate this webhook event to make sure this doesn't happen again.
Hoff being able to respond to itself is a feature since it's used to automate deployments in certain repositories. This approach uses a simple HTML comment that is added to parser error messages. If the comment is found in any message, Hoff won't try to parse it for commands. This is also useful in other situations where Hoff may post arbitrary output that could contain its command prefix.
This fixes #223.

Thanks for the review! @OpsBotPrime merge

Pull request approved for merge by @robbert-vdh, rebasing now.

CI job :yellow_circle: started.


Use a non-capturing group for literals

Saves some memory and improves performance a bit.

According to the benchmarks introduced in #7, it improves the performance and reduces the number of allocations. But most importantly, it makes tmpl.Regexp().FindStringSubmatch() usable (otherwise results are "polluted" by matching literals).
Benchmark results:
Running tool: /usr/local/bin/go test -benchmem -run=^$ -bench ^(BenchmarkRegexpMatch)$ -v -count=1

goos: darwin
goarch: amd64
BenchmarkRegexpMatch
BenchmarkRegexpMatch-8   	 1000000	      1015 ns/op	       0 B/op	       0 allocs/op
PASS
ok  	_/Users/dunglas/workspace/uritemplate	1.082s

Running tool: /usr/local/bin/go test -benchmem -run=^$ -bench ^(BenchmarkRegexpFindAll)$ -v -count=1

goos: darwin
goarch: amd64
BenchmarkRegexpFindAll
BenchmarkRegexpFindAll-8   	  906996	      1246 ns/op	     128 B/op	       2 allocs/op
PASS
ok  	_/Users/dunglas/workspace/uritemplate	1.202s

Thanks for the series of your contributions!
Due to breaking changes to the matching sub-groups this pull request will introduce, I will tag this revision as the v2.0.0.

Thanks for the tag. I'll probably open a new PR this afternoon to fix https://github.com/dunglas/uri-template-tester/issues/3. It may introduce another BC break.


Fix issue if  tag has an id attribute but no href

I have a post in Ghost that has an id attribute directly on the anchor tag, but no href. Previously this would generate a build error. With this fix the <a> tag will be rendered with the id attribute, but withRewriteGhostLinks() will no longer try to parse an undefined href.

Thanks for adding this exception handling. Nice!

@all-contributors please add @nring for code


Include extras

From what I can tell from looking at the database the extras of a game are not included in a download, are they? If not, this would be a nice extra, maybe even offering a way to select what extras to include.

You're right, they're not downloaded. I'm planning to include them but currently I don't have much time for my side projects and the extras will take a while to do.

No worries, I know how it is with side projects.
I ended up using LGOGDownloader for my specific need.


Update to Bevy 0.9

@aevyrie I prepared a small PR which updates some of the deprecated Bevy APIs. Let me know if that's ok, any feedback appreciated.

use spawn() instead of spawn_bundle()
use insert() instead of insert_bundle()
use new DefaultPlugins API

I split the HighlightablePickingPlugins from being a part of DefaultPickingPlugins I was not sure what would be a better alternative to merging two plugin groups in the PluginGroup::build trait implementation. Is that ok or do you have a suggestion on what to do there?

Thank you!

@olegomon could you please read this https://github.com/aevyrie/bevy_mod_picking/issues/138 and comment on the thread so we can relicense as MIT + Apache?


safe delete issue

Trying to delete a key in my app's secret that has a typo in it - unfortunately it contains a colon. How can I use safe delete pathway:keyname when my keyname includes a colon?
So far have tried a combination of: containing the keyname in double quotes, single quotes, and with a backslash in front of the colon in the keyname. None of these have worked.
Thank you!

The colon issue was a little more widespread than just the delete command. This, along with some other oddities around escaping colons and carets, have now been addressed in v1.8.0. tldr; you can prepend the : with a \... like \:, and safe will now treat it like a normal, non-separating colon.


Make issues first class entities (#187)

Applicable Issues

Issue #187
Description of the Change

Description of change follows from the initial description found in issue #187 prior to any feedback from the Eiffel community.
Alternate Designs

A possible alternative to introducing new issue related events would be to use the existing EiffelFlowContextDefinedEvent. We decided to introduce new event types because we felt the information conveyable by a FlowContext wasn't sufficient. A uri would be necessary to uniquely identify an issue out side of Eiffel, and attaching other data such as tracker is very useful. Coercing these into a FlowContext doesn't fit.
Benefits

Same benefits as listed in the first comment of issue #187 prior to any feedback.
Possible Drawbacks

Same possible drawbacks as listed in the first comment of issue #187 prior to any feedback.
Sign-off

Developer's Certificate of Origin 1.1
By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.
Signed-off-by: Jaden Young<EMAIL_ADDRESS>

I should point out that the current set of commits bumps the patch versions of IV and SCC, but should probably bump at least the minor versions instead.

I've removed ISM/IA from the PR. What remains is essentially pulling the fields from  data.issues out of SCC/IV and into ID, and adding semantically equivalent links from SCC/IV to ID. There's also an added optional data.title field on ID as a nice-to-have for human consumption.

:+1:

Making good progress! Will you propose the other events (EiffelIssueStatusModifiedEvent and EiffelIssueAssignedEvent) in this PR, too?

I'd prefer that ISM and IA would come in a later PR. It it not as obvious to me that we want those events in the protocol. I would need to be convinced of the need of those additional events before I'd be happy to propose them being added.

I don't see any more comments coming in, so let's move forward on this. Let's merge the new ID event and hold on other Issue events until we sort out #190 (anyone want to take a stab at this? - would be greatly appreciated, I'm rather strapped for time right now). We need a link from README.md to ID. Other than that, we should be good to go?
If you have any objections, speak now and so on...

Ping @jaden-young. Would you be willing to add the README link so I can merge, or would you rather that I added it?

My apologies, the link has been added to the README.


Use correct component in the README

I believe It should be UploadField here instead of Uploader :)

Thanks... I just rewrote this so there may be some minor issues. Already using the field in production. Will have the uploader there too in a week:)


📎 Deprecate the CLI option --indent-size and create --indent-width

Description

create a new --indent-width CLI option;
log a deprecation diagnostic if --indent-size is provided;
log a warning diagnostic if --indent-size and --indent-width are provided, and use the latter;
include tests

I will try to do this

Should I use/create a diagnostic struct for this? Where should I check the provided option?
I'm thinking of doing based on #107 if you have any tips or better.

Should I use/create a diagnostic struct for this? Where should I check the provided option?

Yes, you'll have to create a new diagnostic.
https://github.com/biomejs/biome/blob/ef3af45b4cb775ba4556151a3184d77ac93c2761/crates/rome_cli/src/diagnostics.rs#L23-L60
You could create a generic DeprecatedArgument that accepts a generic message.

I'm thinking of doing based on #107 if you have any tips better.

Yes that's right!

Should I create an array to store all the deprecated arguments, in order to leave it generic for future uses?

Should I create an array to store all the deprecated arguments, in order to leave it generic for future uses?

It's possible that in the future we don't deprecate an option for another, that's why a generic message is more future proof

When I run the CLI command (cargo run --bin biome -- --help), it gives the following error: thread 'main' has overflowed its stack.
I'm using Windows.

To run the CLI, do
cargo biome-cli-dev --help

I tested this command now, and it also gave the same error.

Oh, I guess the previous windows users didn't use this command 😔 not sure how to fix that, I don't use windows
You can still write a test, they are very reliable, and we have many of them

I'm sorry I didn't ask for this information sooner
What would be the best or correct way to add this option? I tried putting it in the formatter configuration in the rome_service crate, but that breaks tests that apparently don't use the indent-size option through the CLI.
I checked the deprecated option in the or_diagnostic function equal to #107.
Is there anything I forgot to do or did wrong?

What would be the best or correct way to add this option? I tried putting it in the formatter configuration in the rome_service crate, but that breaks tests that apparently don't use the indent-size option through the CLI.

That's the correct place. I'm not sure what you mean when tests break, maybe the best is opening a draft PR so we can look a it, when code at hand.

Explaining better: the way I used to check the indent-size option (very similar to #107) apparently takes the settings from the configuration file, which causes the diagnostic to be run even without pass the deprecated CLI option.
If you still don't understand, I committed an early version (without tests) of what I tried to do, and I can make a draft pull request.

I understand now. Still, it would be great to see some code, so I can better understand the implementation, and give suggestions. It's fine there aren't any tests, that's what draft PRs are also good for: get feedback

I first thought this was a good idea. However, I am starting to wonder if it is still. Prettier and .editorConfig use indent_size.

width is more in line with the meaning of the word given its context, because it's applied on a line. And it's also in line with line_width.


App closed and Note was not saved in draft

During a frantic and clumsy attempt to select text from the Notepad using my Chromebook, and with Control+C...., the Notepad app closed and the Note wasn't saved as draft when I re-opened the app.
I think I might have selected some combination of keys causing the app to close like that.
What are the shortcuts for closing the app without the Note being saved in drafts, (if such a shortcut even exists)?
I guess I might have closed the Note, which I did not save, and closed the app in quick succession as a result of my clumsy keyboarding.
This has only happened once.
If I run the app, and close it using task manager, the Note is in draft, but not Saved in the list of saved Notes. To actually save the Note, I need to click the floppy disk save icon.
I guess there must be a shortcut that will close the active Note without saving and deleting the temporary draft, and close the app?
OR
The app crashed, deleting the temporary draft, and closing the App?
UPDATE:
1 - I copied text (plain text with UL list without styling, the styling CSS was removed during pasting into Note, which was copied from a website) and pasted into a new Note
2 - I copied one item of text from Note and minimised Notepad App to view Chrome browser window, pasted text into text field on website
3 - Re-focused window (clicked on tray to maximise, Chromebook)...
3.1 - Notepad crashed, any Draft notes deleted automatically.
Was it still my clumsy control+c, control+v keyboarding? Not sure. I don't believe it was.
I have about 50 saved notes, but just small bits of texts, that's all.
Basically, when I re-focus the app to copy the next snip of text, it closes and any drafts are deleted. That's what I found.
UPDATE 2
I have managed to repeat the issue:
1 - open App (Chromebook)
2 - copy arbitrary text (mouse pad select + control+c) into new Note without saving Note
3 - focus Google Chrome window via the Chromebook app tray
4 - paste text into somewhere else, anywhere
5 - go back to Notepad by selecting the app icon from the tray, copy more text (mouse pad + control C), refocus Chrome window, paste
6 - go back to Notepad, and with the same text selected, I use shift+left arrow key, to fine select text from the same line
7 - Notepad closes, I re-open Notepad, drafts deleted

And, I've no idea how related, or not, this may be - however, not being able to remove the DELETE button from the main interface can be entirely frustrating...
Also, for example: if having enabled "Ask before saving", from the Settings (in order not to automagically save changes, if making a mistake), then saving will also prompt in a similar fashion as deleting - the dialogs may have a different title, and text, but it's the same box - and when moving quickly, it's rather possible to delete something, instead of saving it, by tapping in the wrong place, meh.
Ok, thanks for your time!..🙂

And, I've no idea how related, or not, this may be - however, not being able to remove the DELETE button from the main interface can be entirely frustrating...
Also, for example: if having enabled "Ask before saving", from the Settings (in order not to automagically save changes, if making a mistake), then saving will also prompt in a similar fashion as deleting - the dialogs may have a different title, and text, but it's the same box - and when moving quickly, it's rather possible to delete something, instead of saving it, by tapping in the wrong place, meh.
Ok, thanks for your time!..🙂


There is no way of selecting multiple files without left click on the mouse

When i try to select multiple files away from each other, i tried pressing on ctrl, moving then pressing the space bar, but the file was not selected. the only way to do it is to press on the left click on the mouse.
app version : <IP_ADDRESS>

@ZoroNasr You can click the bottom button in the right and select everything

No, i don't want to select everything, i want to select like 3 or 4 files, and i can't do everything using the keyboard, i have to use the mouse unlike the normal explorer where u can go to the files with the arrows, and click on the space bar to select them. that is what i meant !!

@ZoroNasr You can do that by pressing shift while you use the arrow keys.

@ZoroNasr Can you add your feedback to #9?

Well, no, if i press the shift key, it will choose all the files in the row, i was to select specific files 3 or 4 rows on top, i can click on the file, hold on to control, and then i need to move the mouse up and then left click on it. on the normal explorer, u can simply click on space bar when u reach the file u want to select, and it gets selected. I hope you got my point. Simply imagine not using the mouse at all, and you want to select only 2 files away from each other inside the same folder between 100 other files. Thanks

@ZoroNasr We are tracking feedback related to mouse and keyboard shortcuts in #9. Please add your feedback there so that we can keep track of it.


lyxal: Watch scalacube.com

lyxal requests the watch of the watch_keyword scalacube\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Dragonrage in Charcoal HQ

scalacube\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.


Able to set release_component_type to null.

JIRA: PDC-888

I don't think this is the right fix. Based on discussion with @sochotnicky on IRC, type should not even be optional and there should rather be a default value (most likely RPM).

Basically fixed by this bug's description. I will keep this change here until we meet together in next meeting.

:+1:

Good.


Custom colors do not work

at source code lines :221
const color1 =
!this.color1 && this.color1 !== 0 ? 0x8bf329 : this.color1,
color2 = !this.color2 && this.color1 !== 0 ? 0x298bf3 : this.color2,

I fixed the issue with this commit. It seems to be working for me.
But I refactored to just add the defaults to the props: color1: { default: 0x8bf329, type: [String, Number], required: false }.

Thanks. Now it works


_submodules causes IntelliSense to be confused

Hi,
In version 3.11.0, I am unable to get IntelliSense (autocomplete in IntelliJ's PyCharm)  from the plum_azure module.
This worked fine in version 3.9.2.
The problem seem related to how plum_azure/__init__.py is generated; the module names are listed in _submodules, but changing its name to __all__, the IntelliSense works as expected.
In Python, __all__ is a special variable in __init__.py files.
Is it possible to rename _submodules to __all__ again?

@sindre-nistad Thanks for reporting this. We made this change recently with the intention of improving Intellisense so it's unfortunate that you are seeing the opposite. Can you please give me some more information about the specific behavior that has changed?

@komalali I've attached four images of PyCharm's autocomplete when _submodules are used.
In the second image, there are some correct references in the storage module, but that is likely because I've used those in various parts of the project (PyCharm tries to assume what is available based of what has been typed before).
Note: pulumi_azure is referer to as azure in this particular project.

Autocompletion of the pulumi_azure module, with _submodules

Autocompletion of the pulumi_azure.storage module, with _submodules

Autocompletion of the pulumi_azure module, with __all__

Autocompletion of the pulumi_azure.storage module, with __all__

Thanks @sindre-nistad that's super helpful. Here's the PR where we removed __all__ to give you some context behind why: https://github.com/pulumi/pulumi/pull/4873
So the solution here would require solving for that problem too, which probably just means that all exported values, including the classes, need to be listed in __all__.
Can you tell me how you're importing pulumi_azure here? Is it with import pulumi_azure as azure?


[AutoDiff] [Sema] Fix missing access-level mismatch diagnostic in -enable-testing builds.

When a derivative function is internal and its original function is
public, there should be an error when the derivative is not @usableFromInline.
This patch fixes a bug where the error does not appear in SwiftPM debug
build configurations (or any configurations with -enable-testing).
ValueDecl::getEffectiveAccess() should not be used when we perform
formal access checking for AutoDiff-related declaration attributes because
it will treat all internal declarations as public when -enable-testing is
enabled. Instea we should use a combination of ValueDecl::getFormalAccess()
and ValueDecl::isUsableFromInline().
Also, add a RUN line with -enable-testing to all AutoDiff declaration
attribute test files.
Resolves SR-13500 (rdar://68336300).

@swift-ci please test


Support VK_KHR_dynamic_rendering_local_read

Description
VK_KHR_dynamic_rendering_local_read (DRLR) closes the gap left by VK_KHR_dynamic_rendering (DR) where input attachments couldn't be used. As the Vulkan API moves towards dynamic rendering, usage of DRLR will increase and DR will eventually replace now-legacy VkRenderPass objects.
This change adds support for DRLR to RenderDoc. At a high level, DRLR allows the render pass color attachments to be mapped to "locations". Since the rest of the API does not use these remapped locations (notably, blend and vkCmdClearAttachments use the attachment index), I opted for RenderDoc to continue referencing the attachment by its index in the UI. However, where location matters (i.e. connection to the shader), the mapping is specified in the UI:

Additionally, an attachment can be mapped to VK_ATTACHMENT_UNUSED, in which case writes to it are automatically disabled. In this case, RenderDoc would tag the attachment as [disabled] and automatically set the Write Mask to ____:

DRLR additionally includes a mapping of attachments to InputAttachmentIndex decorations in the shader. For depth/stencil, the shader is no longer required to specify this decoration at all, in which case the corresponding API (VkRenderingInputAttachmentIndexInfoKHR) would also specify nullptr for the mapping. If VK_ATTACHMENT_UNUSED is specified in the mapping, the attachment is not expected to be used as input attachment. In the code, an additional flag for depth and stencil is present to differentiate between nullptr and VK_ATTACHMENT_UNUSED, with the format being an "implicit" mapping.
Notably, RenderDoc does not actually care about input attachment indices, but rather uses set/binding information to read from input attachments (e.g. during shader debugging), so the input index mapping information only ends up being used for the html export from the pipeline tab.
Fixes #3341

@Themaister the Pipeline State tab also has an "Export to Fossilize database" functionality. Is there something I need to manually implement for that?


Add decentraland extension

Description

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi Raycast Team, why is CI failing? I took the screenshots with built-in Window Capture

@pernielsentikaer done!


[Bug] Run CenterPoint inference on mini-val set two times, but results are different.

Prerequisite

[X] I have searched Issues and Discussions but cannot get the expected help.
[X] I have read the FAQ documentation but cannot get the expected help.
[X] The bug has not been fixed in the latest version (dev-1.x) or latest version (dev-1.0).

Task
I'm using the official example scripts/configs for the officially supported tasks/models/datasets.
Branch
main branch https://github.com/open-mmlab/mmdetection3d
Environment
sys.platform: linux
Python: 3.10.12 (main, Nov 20 2023, 15:14:05) [GCC 11.4.0]
CUDA available: True
numpy_random_seed:<PHONE_NUMBER>
GPU 0: Tesla T4
CUDA_HOME: /usr/local/cuda
NVCC: Cuda compilation tools, release 12.2, V12.2.140
GCC: x86_64-linux-gnu-gcc (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
PyTorch: 2.1.0+cu121
PyTorch compiling details: PyTorch built with:
  - GCC 9.3
  - C++ Version: 201703
  - Intel(R) oneAPI Math Kernel Library Version 2022.2-Product Build 20220804 for Intel(R) 64 architecture applications
  - Intel(R) MKL-DNN v3.1.1 (Git Hash 64f6bcbcbab628e96f33a62c3e975f8535a7bde4)
  - OpenMP 201511 (a.k.a. OpenMP 4.5)
  - LAPACK is enabled (usually provided by MKL)
  - NNPACK is enabled
  - CPU capability usage: AVX2
  - CUDA Runtime 12.1
  - NVCC architecture flags: -gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_80,code=sm_80;-gencode;arch=compute_86,code=sm_86;-gencode;arch=compute_90,code=sm_90
  - CuDNN 8.9.2
  - Magma 2.6.1
  - Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CUDA_VERSION=12.1, CUDNN_VERSION=8.9.2, CXX_COMPILER=/opt/rh/devtoolset-9/root/usr/bin/c++, CXX_FLAGS= -D_GLIBCXX_USE_CXX11_ABI=0 -fabi-version=11 -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -DNDEBUG -DUSE_KINETO -DLIBKINETO_NOROCTRACER -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DSYMBOLICATE_MOBILE_DEBUG_HANDLE -O2 -fPIC -Wall -Wextra -Werror=return-type -Werror=non-virtual-dtor -Werror=bool-operation -Wnarrowing -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-unused-parameter -Wno-unused-function -Wno-unused-result -Wno-strict-overflow -Wno-strict-aliasing -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=old-style-cast -Wno-invalid-partial-specialization -Wno-unused-private-field -Wno-aligned-allocation-unavailable -Wno-missing-braces -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Werror=cast-function-type -Wno-stringop-overflow, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_DISABLE_GPU_ASSERTS=ON, TORCH_VERSION=2.1.0, USE_CUDA=ON, USE_CUDNN=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=1, USE_NNPACK=ON, USE_OPENMP=ON, USE_ROCM=OFF,

TorchVision: 0.16.0+cu121
OpenCV: 4.8.0
MMEngine: 0.10.2
MMDetection: 3.3.0
MMDetection3D: 1.4.0+fe25f7a
spconv2.0: False

Reproduces the problem - code sample
I was using the original codes for CenterPoint and inference/evaluation
Reproduces the problem - command or script
I just ran the mmdet3d in colab script
config = 'configs/centerpoint/centerpoint_voxel0075_second_secfpn_head-dcn-circlenms_8xb4-cyclic-20e_nus-3d.py'
checkpoint = 'checkpoints/centerpoint_0075voxel_second_secfpn_dcn_circlenms_4x8_cyclic_20e_nus.pth'

!python tools/test.py $config $checkpoint --cfg-options test_dataloader.dataset.metainfo.version='v1.0-mini'

Reproduces the problem - error message
No error information.
Additional information
I ran this script/command twice, but I got two slightly different results. It seems the inference is non-deterministic. Do you have any ideas why?

The TTA RandomFlip3D is causing the issue. Additionally test_mode=True is also required in LoadPointsFromMultiSweeps for it to be deterministic.
Update your test pipeline to the following:
test_pipeline = [
    dict(
        type='LoadPointsFromFile',
        coord_type='LIDAR',
        load_dim=5,
        use_dim=5,
        backend_args=backend_args),
    dict(
        type='LoadPointsFromMultiSweeps',
        sweeps_num=10,
        test_mode=True,  # make sure this is set to true
        backend_args=backend_args),
    dict(
        type='MultiScaleFlipAug3D',
        img_scale=(1333, 800),
        pts_scale_ratio=1,
        flip=False,
        transforms=[
            dict(
                type='GlobalRotScaleTrans',
                rot_range=[0, 0],
                scale_ratio_range=[1., 1.],
                translation_std=[0, 0, 0]),
            # dict(type='RandomFlip3D'),  # comment out this line
            dict(
                type='PointsRangeFilter', point_cloud_range=point_cloud_range)
        ]),
    dict(type='Pack3DDetInputs', keys=['points'])
]


[FEATURE] Add a hits_only option to _search

Is your feature request related to a problem? Please describe.
Coming from https://github.com/opensearch-project/k-NN/issues/969, the response from OpenSearch is incredibly verbose. For example, examining results requires the following code:
for hit in results['hits']['hits']:
    print(hit['_source'])

in Java
SearchResponse<JsonNode> searchResponse = client.search(searchRequest, JsonNode.class);
            for (int i = 0; i < searchResponse.hits().hits().size(); i++) {
                logger.info(searchResponse.hits().hits().get(i).source().toString());
            }

Too many hits.
Describe the solution you'd like
Add a hits_only=true option that collapses the response. When hits_only is specified, only the nested hits are returned.
curl
  --request POST \
  --url "../my-index/_search?hits_only=true"
  --header 'Accept: application/json'
  --data
  {
    "vector1":[0.1, 0.2, 0.3],
    "k":1
  }

[
    '_id': 'vec1',
    '_score': 1.0,
    '_source': {
      'vector1': [0.1, 0.2, 0.3],
      'metadata': {
      'key1': 'value'
    }
]

@macohen AFAIK no, this would be a generic feature that would cut down the response to only the data


Docs: identify CNCF project network add-ons

Indicating CNCF project status in the list of network add-ons, to help end users understand which network add-ons are part of the CNCF family

I'm fine with this.
/lgtm

/assign @bradtopol
For approval, now it has the LGTM label (I hope I'm doing this right!)


Compatibility tests: Use correct test cases object and remove invalid test case

PR Summary
The $genericVerCases test case object was not being used due to what seems to have been a copy-paste error.
One test case actually did not work because having a version string with not dot would throw here
I could add the following negative test for it but was wondering why we cannot make it accept single character versions?
        It "Does not parse version string '7' due to missing dot" {
            { [Microsoft.PowerShell.CrossCompatibility.PowerShellVersion] '7' } | Should -Throw
        }

In the case of PowerShell Core we wouldn't even need our own implementation and just use System.Management.Automation.SemanticVersion, which can actually parse versions without a dot.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] Make sure all .cs, .ps1 and .psm1 files have the correct copyright header
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed and/or updated documentation
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

I could add the following negative test for it but was wondering why we cannot make it accept single character versions?

I think that's worth doing.
That class exists to allow JSON serialisation of versions across PowerShell versions, so it's not really necessary in the tests.
Thanks for fixing this @bergmeister!

OK, I will merge this and open a 2nd PR later to add the capability of parsing a single character version


How to set the default chart time?

Hi,
there were many requests to implement a default timerange for the charts.
https://github.com/alexjustesen/speedtest-tracker/issues/1855
https://github.com/alexjustesen/speedtest-tracker/issues/1858
https://github.com/alexjustesen/speedtest-tracker/issues/1449
It seems you implemented this:
https://github.com/alexjustesen/speedtest-tracker/pull/1669
For the life of me, I cannot figure out how to get this to work. It's not explained anywhere, do I add something to my environment variables, if yes, what?
Sorry if this is written down somewhere, but I cannot find it. It's not in the docs, as far as I'm aware.
Thanks.

That one was closed and not merged. As we needed to get some other things out of the way first to make it easier and more flexible with other parts of the application.
It will come in the future, no eta yet

I close things that might be duplicates or similar just to keep the issues manageable. That's why we also reference issues to keep that lineage.


equals operator for element mishandles nil

Classes with issues:
page-object\platforms\selenium_webdriver\element.rb
page-object\platforms\watir_webdriver\element.rb
They both have an override for ==(other) as follows:
def ==(other)
  element == other.element
end
The problem is a test such as
expect(my_div != nil).to be true
fails with a null pointer error because it tries to invoke .element on nil.
The workaround is to flip the operands and write 'nil != my_div' but that isn't ideal.
There should be logic within this method to check if other is nil, and handle that case accordingly.

This is a good point. This method will fail if other is anything that doesn't respond_to? :element

Just out of curiosity, when would you want to test if a PageObject::Element is nil? I would have thought one would be checking if the element exists or is visible.
Personally, I think I would have preferred to write (which would avoid the error):
expect(my_div).not_to be nil

The point is if you are checking if an element is the same as another
element, you have to null check it first so you don't get an error.
On Fri, Apr 10, 2015, 2:30 PM Justin Ko<EMAIL_ADDRESS>wrote:

Just out of curiosity, when would you want to test if a
PageObject::Element is nil? I would have thought one would be checking if
the element exists or is visible.
Personally, I think I would have preferred to write (which would avoid the
error):
expect(my_div).not_to be nil
—
Reply to this email directly or view it on GitHub
https://github.com/cheezy/page-object/issues/244#issuecomment-91645879.

Sorry, I understood your point about other not responding to element.
My question was on the the assumption that expect(my_div != nil).to be true was literally the test being run in their test suite (rather than a test to prove a bug in the page object framework). Given that my_div is a PageObject::Element, it would never be nil. That is why I was asking why they were doing that test.

This should be fixed in the next release.


Fetch by tag or all images

Hello ,
i'm don't find a way to get all images from my cloudinary by tag. maybe i'm missing somthing ?

Searching for the same. Please tell me if you find a Solution @sandro-git . I will tell you mine if I find one.

I have a Solution / Workaround for this.
Working with Tags. Just replace it with your name.
<template>

      <cld-image :public-id="post.public_id" responsive loading="lazy" />

</template>

i was inspired with your solution and i come to this one which doesn't need button i think he is reactive.
`async asyncData({ $axios }) {
const images = await $axios.$get(
'https://res.cloudinary.com/vr-cafe/image/list/Jeux.json'
)
return { images }
},
computed: {
imageComputed() {
return this.images.resources.map((image) => {
return this.$cloudinary.image.url(image.public_id)
})
},
},

sorry i'm not fluent in making pretty code in github. ;)

@sandro-git
Nice Solution. I am still struggling get the Additional Metadata or Tags from the Cloudinary API.
Thats why I needed a Refresh button for testing. But implemented now your Solution :)
Happy Coding

It’s my next step getting metadata. I let you know when I get it. Thanks for helping me. It’s good to help each other. Happy coding too.

i dont understand.
if i have the url in order to put it into
async function () {
const images = await this.$cloudinary.image.url('url' <----here!)
this.data = images
}

then why would I need this module? for transformations?
i dont get it.
i NEED the to fetch url, if i have the url why would i fetch it.
i need all the urls of all my images.
thats the whole point.
in the gridsome version of this module you can just ask using graphql for all the images that contain a word (like a folder name) in their public_id.
why cant this be achieved in nuxt?
i need to pass a varaible with a folder name into the the async function.
this is boggling me.

has anyone found a way to get all the urls in cloudinary with this module?
the gridsome version of this plugin allows to do this very simply.
is it not possible in Nuxt?

Hey, I am closing this issue as it is targeting the v1 version and it is really old (more than one year stale)


Section Headings

Upon reading the comments in the code, I am unclear on how to remove the bubble on the section headings.   Personally, I would like to remove the bubble effect altogether, but I have not had any success in resolving how to remove the bubble.  If this could be documented better if feel it would give people more freedom to work with the section headings.
In the following defines that the first #1 and second #2 characters of the heading are to be replaced with round bubbles in alternating text colours.
% [...]
% the following reads from line 252, in twentysecondscv.cls
%%%%%%%%%%%%%%%%%%%%%
% Section Color box %
%%%%%%%%%%%%%%%%%%%%%
\newcounter{colorCounter}
\def\@sectioncolor#1#2{
  {%
  % Switch between blue and gray
   \round{#1#2}{%
% [...]

However, removing #1 and #2 will break the heading, and inserting a #0 breaks the heading also.
Commenting out the \round{} also has no benefit.

I was facing the same problem and was able to resolve it. Look at these lines and change them to the following:
\newcommand*\round[2]{%
  \tikz[baseline=(char.base)]\node[anchor=north west, inner sep=1.6pt, minimum size=5.5mm,
    text height=3.6mm](char){#1} ;}


Incorporate ipyreact

ipyreact seems to be a better way of bridging between jupyter widgets and react.
https://github.com/widgetti/ipyreact
Move the frontend over to this model,  probably by just porting the code in https://github.com/widgetti/ipyreact/blob/master/src/widget.tsx
I don't want the react 16/18 testing (this is built on react 18).  I also don't want the dynamic build step that ipyreact gets from anywidget... good for dev,  adds complication to reading the code for a primarily packaged repo.

Done.  https://github.com/paddymul/buckaroo/commit/3ef051c0610132f47d83ef7dd7fd1ebd134fa53b
There are no more manually coded event listeners.


Just don't use this lib.

The last release was in 2020.
In my experience, there was plenty of bugs, which all lead to this lib...

If anyone has a better, and maintained suggestion,
feel free to leave a suggestion.

Hello @predrag-codetribe, have you seen react-native-pager-view?

Yes, have seen it but after the migration...
I migrated from leecade/react-native-swiper to https://github.com/gusgard/react-native-swiper-flatlist
because the migration process was easy,
and to avoid some bugs that I have encountered leecade/react-native-swiper.
I also enocuntered some minor bugs with  https://github.com/gusgard/react-native-swiper-flatlist,
like the onChangeIndex={({ index }) => setPage(index)} sometimes getting index out of sync...
But managed to write some hacky code to fix that.
  const [page, setPage] = useState(0)
  const isLastPage = page === pack.length - 1

  useEffect(function HackToKeepThePageUpToDate() {
    // sometimes the onChangeIndex will not be called, thus the page is not updated
    // this useEffect should address that, and the page should always be up to date
    const interval = setInterval(() => {
      if (!swiperRef.current) return
      const index = swiperRef.current.getCurrentIndex()
      setPage(index)
    }, 1000)
    return () => clearInterval(interval)
  }, [])

Certainty I am open to find the a swipper component that "just works".

If anyone has a better, and maintained suggestion, feel free to leave a suggestion.

@predrag-codetribe Try this:
https://www.npmjs.com/package/@zzzzk/react-native-swiper


Loading bar appearing when pushing to nested $scope variable

I am using version 0.9.0 and AngularJS version 1.6.1. As far as I can tell, it affects all browsers. I have a scope object that contains a few arrays of objects, like so:
$scope.myObject = {
  arr1: [],
  arr2: [
    {
      property: "data",
      nestedArr: [
        {
          name:"data"
        }
      ]
    }
  ]
};

On the front-end, data is being pushed into the nestedArr in a repeater.

{{ parentObj.property }}

    <input type="text" ng-model="obj.name" placeholder="Name" />

  Add Object

Now, the issue is that when you add an object into the nested array (either in using the method above or through the controller utilizing a function), it causes the loader to appear even though nothing is being passed to a resource or through an $http request, all that is being manipulated is the $scope variable. The loading bar is not triggered when you use the same method to pass something into the parent object; myObject.arr2 in this example. Im not really sure what to make of this or how to prevent it.

After posting this I realized that I had a misconfiguration in a directive that was appended with parentObj.nestedArr.push({}) that caused it to make a new request to my API every time instead of loading cached data.


no module named 'neptune.experiments'

I use 'conda list' to check the module which has been installed. I find that I have installed the right version of package. But why am I still getting errors?
neptune-client            1.6.1              pyhd8ed1ab_0    conda-forge

Hello, me too, can you solve it? Thanks.

Hello，I'm using the version of neptune-client==0.16.1 in the requirements.txt file.


Update Docker Hub image

Would it be possible to update the Docker Hub image?
Ideally it'd be nice to have it deployed in CI.
Thanks

I would love to have this! Would you be able to help? Maybe a workflow already exists we can re-use?


add support for trillertv

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Computer location is in US, but VPN is set to UK to access region-locked video
Provide a description that is worded well enough to be understood
When attempting to download region-locked videos over a VPN, I receive HTTP Error 503: Service Temporarily Unavailable.  The video plays correctly in browser when using the VPN, and I have --cookies-from-browser in my yt-dlp command.  I've also verified that the cookies are downloaded correctly by closing Chrome completely before running the command.
The download also fails under the same error if I use "--cookies-from-browser firefox" instead of Chrome.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
yt-dlp -vU https://www.trillertv.com/watch/aew-dynamite-episode-50-21/2pagg/ --cookies-from-browser chrome --check-formats
[debug] Command-line config: ['-vU', 'https://www.trillertv.com/watch/aew-dynamite-episode-50-21/2pagg/', '--cookies-from-browser', 'chrome', '--check-formats']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [89a161e8c] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.19045-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: ffmpeg 2024-07-07-git-0619138639-full_build-www.gyan.dev (setts), ffprobe 2024-07-07-git-0619138639-full_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.07.04, curl_cffi-0.5.10, mutagen-1.47.0, requests-2.32.3, sqlite3-3.35.5, urllib3-2.2.2, websockets-12.0
[debug] Proxy map: {}
Extracting cookies from chrome
[debug] Extracting cookies from: "C:\Users\Trevor\AppData\Local\Google\Chrome\User Data\Default\Network\Cookies"
[debug] Found local state file at "C:\Users\Trevor\AppData\Local\Google\Chrome\User Data\Local State"
Extracted 2967 cookies from chrome
[debug] cookie version breakdown: {'v10': 3008, 'other': 0, 'unencrypted': 20}
[debug] Request Handlers: urllib, requests, websockets, curl_cffi
[debug] Loaded 1829 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[generic] Extracting URL: https://www.trillertv.com/watch/aew-dynamite-episode-50-21/2pagg/
[generic] 2pagg: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] 2pagg: Extracting information
[debug] Looking for embeds
[debug] Identified a JSON LD
[generic] Extracting URL: https://cdn.fite.tv/ims-secure-videos/AEW/4061121_aew-dynamite-episode-50-21.mp4#__youtubedl_smuggle=%7B%22force_videoid%22%3A+%222pagg%22%2C+%22to_generic%22%3A+true%2C+%22referer%22%3A+%22https%3A%2F%2Fwww.trillertv.com%2Fwatch%2Faew-dynamite-episode-50-21%2F2pagg%2F%22%7D
[generic] 2pagg: Downloading webpage
ERROR: [generic] Unable to download webpage: HTTP Error 503: Service Temporarily Unavailable (caused by <HTTPError 503: Service Temporarily Unavailable>)
  File "yt_dlp\extractor\common.py", line 740, in extract
  File "yt_dlp\extractor\generic.py", line 2384, in _real_extract
  File "yt_dlp\extractor\common.py", line 909, in _request_webpage

  File "yt_dlp\extractor\common.py", line 896, in _request_webpage
  File "yt_dlp\YoutubeDL.py", line 4165, in urlopen
  File "yt_dlp\networking\common.py", line 117, in send
  File "yt_dlp\networking\_helper.py", line 208, in wrapper
  File "yt_dlp\networking\common.py", line 340, in send
  File "yt_dlp\networking\_requests.py", line 365, in _send
yt_dlp.networking.exceptions.HTTPError: HTTP Error 503: Service Temporarily Unavailable

trillertv is not a supported site. I'm assuming you'd like to make a site support request for it?
I'm geo-blocked, so I have a couple of questions:

You mentioned passing --cookies-from-browser. Does the site require an account to watch content?

Can you confirm that the content is not DRM protected? You'll need to disable DRM in your browser to check this

I'm sorry for the confusion - I saw "Triller" in the list of supported sites and assumed that extended to trillertv.
Yes, the site requires an account to watch.  I confirmed that the content is not DRM protected by disabling the Firefox addon.

All good. The confusion is understandable. triller is a tiktok clone, unrelated to trillertv

Seconding this request, I currently just get 503 "Request is from unauthorized domain" errors, even when importing cookies and using --referer set to www.trillertv.com and variations of it.
They do have free videos that could be used for initial testing, a current example:  https://www.trillertv.com/video/afc-knockout-stage-promo/2pf28/


Issue 42

Related Issue

Fixes #42 .

Proposed Changes

Changed advance_pdf_viewer to native_pdf_view to avoid pubspec conflicts
added push notification serrvice

Additional Information

Any additional information about the change

Checklist

[x] Tested
[x] No Conflicts
[x] Change In Code

ScreenVideo
https://user-images.githubusercontent.com/59207688/117425173-2ea0f400-af40-11eb-83d1-c2ccbcb44dee.mp4

Hey, @himanshusharma89, kindly review this Pr. The push notification is working and rebasing with the master branch is also done!


Ensure modifierKeys config is set before reading

This PR ensures the modifierKeys config is set before trying to reading the options from it, preventing a TypeError from being thrown.

modifierKeys is set by default, the problem is that it's being overridden by a !== null check - which is fixed through something like #760.

The extra checks probably don't hurt :wink:
But I'll close in favor of #760.


Return error strings with "trivial" closure.

Basically, this just returns an error string as a second return value, if a problem occurred loading the mo file.
This follows the Lua idiom established by e.g. io.open().
The client can now handle error conditions, if they care, rather than living with a mysterious, silent failure.
I also beefed up the first check for a valid path; type(mo_file) ~= "string") will catch nil as well as other types.
I checked that this file compiles and handles path errors reasonably.
I don't know the .mo format well enough to exercise the other error clauses, but they're pretty straightforward.

Coverage remained the same at 100.0% when pulling f6edddc5cb434b588369cbc165c6ec32de94fe2c on IonoclastBrigham:patch-1 into 3bc2b5f6075783656e296c24c8feaff0ba85fee1 on starius:master.


Bump alpine image in Provisioner

Description
Changes proposed in this pull request:

Bump alpine image in Provisioner

/test all

/retest

/retest


When scraping compiled binaries why env is ignored?

When scraping pyd files it runs python with -E flag, which ignores environment variables. For other modules I can set .env file which can be quite sophisticated. I have no control over binary bindings. They often can import other stuff or and do some additional logic in init.py file.
this is the line I am referring to:
https://github.com/microsoft/python-language-server/blob/8ff80697a22b958100b9d12cec1a308b61dc7d10/src/Analysis/Ast/Impl/Modules/CompiledPythonModule.cs#L36
Could at least that be somehow an option? In practice it is really hard to setup autocompletion for binary modules working.

.env files aren't loaded by the language server, and the VS Code extension doesn't pass things from that file down other than PYTHONPATH.
IIRC we don't load the environment so that we don't open a vector for even more odd behavior; there are a slew of variables that may change the module resolution we expect and we'll end up scraping the wrong module. You may want to try removing that and see if it does fix your case.
Note that the scraper is intended for libraries, not user code, so I wouldn't try to use it for anything other than compiled code you aren't planning on editing.

Thank you for such a quick response!
yes PYTHONPATH is basically what is needed in that case. Removing -E fixes our issues.
I didn't fully understood last part sorry. We use scrapper implicitly to autocomplete compiled code. We have a lot of C++ bindings as .pyd files. Really cool part about MS python language server that it manages to generate stubs for those and show intellisense. But if say this .pyd expects other modules imported, it can't do so. Often .pyd file importing wrapped in init.py file which also could import additional stuff, like in usd library: https://github.com/PixarAnimationStudios/USD/blob/master/pxr/usd/usd/init.py

My comment on "libraries, not user code" was about if you had a project that was partially in C/C++ and partially in Python, and wanted to edit the C++ code and immediately see the changes in the same workspace in Python. The current scraping mechanism isn't set up to do that.
We can play with omitting -E, we'd just have to do some testing to see what the other impacts are.

What do you mean by immediately? I also noticed that those cached .pyi files are not updated that often. I had to clean cache by hand to test new changes. Is that what you imply?
Follow up question, is there a way to generate pyi files ourselves and let python server use those?

What do you mean by immediately? I also noticed that those cached .pyi files are not updated that often. I had to clean cache by hand to test new changes. Is that what you imply?

Yes, that is what I mean. We have no way to know that some change in some non-python file would mean that some other pyc/pyd file needs updating (nor would we know how to build it to update it, and so on).

Follow up question, is there a way to generate pyi files ourselves and let python server use those?

There isn't an "automatic" way, but the recommended way would be to place pyi files next to your compiled modules when they're distributed such that tooling doesn't need to do any work to find them. You can run the scraper directly if you want (note that its output is not a "true" stub, it's much closer to python code and can't be processed by other tools), stubgen (if it works), hand write known APIs with the correct types, etc. If you had say foo/bar/compiled.pyd, you could have foo/bar/compiled.pyi and we'll be able to just go to the stub.


export interface Fastify Request has _supabaseClient instead of supabaseClient

Right now the interface inside the declare module is wrong
it should be modified from
declare module "fastify" {
export interface FastifyRequest {
_supabaseClient: SupabaseClient;
supabaseUser: User;
}
}
to this
declare module "fastify" {
export interface FastifyRequest {
supabaseClient: SupabaseClient;
supabaseUser: User;
}
}

I just opened a pull request that addresses this: #38


[P3] Implement pruning of ECR images

Summary
We can store about 58 versions of each image in ECR for free (50GB limit / 171MB per image * 5 images). We should implement some automated pruning of older images so we don't start to exceed the 50GB limit.

The migrate container is actually 275MB rather than 175MB, since it include devDependencies. We may be able to remove those dependencies (e.g. progress for showing progress bars in the CLI) to get it down to 175MB.
We may also be able to implement some cleanup scripts locally which use the AWS CLI, and just run those occasionally (every 50 releases) instead of implementing automated pruning.


feat(build): add the 'ignore-annotations' option for esbuild

Some libs may provide wrong 'side-effect' annotations. As a result, esbuild will remove the side-effect code unexpectedly.
For this reason esbuild provides an option named 'IgnoreAnnotations':

It will still do automatic tree shaking of unused imports, however, since that doesn't rely on annotations from developers. Ideally this flag is only a temporary workaround.

ecomfe/zrender(used by apache/echarts) is a realworld example.  Code below was removed unexpectedly:
 registerPainter('canvas', CanvasPainter);
 registerPainter('svg', SVGPainter);

And I have opened an issue: https://github.com/ecomfe/zrender/issues/927
These packages' newer versions may fix it but the old versions could not work without this option.

LGTM


Broken link in documentation

http://www.soulbroken.co.uk/blog/2011/10/produce-a-checkstyle-report-for-doc-block-validation-with-docblox/ referenced in http://www.phpdoc.org/docs/latest/for-users/reporting/phpdoc-errors.html has been taken down.

@mvriel WHat exactly needs to be done here? I'd love to try my hand at it but I can't understand what needs to be done here.

One could add https://web.archive.org/web/20131023172150/http://www.soulbroken.co.uk/2011/10/produce-a-checkstyle-report-for-doc-block-validation-with-docblox/
Though I am not sure if this still is an issue...

This is no longer an issue.


Feature Request: Explanation in documentation about power of additional special tokens

In train.py there are segments where special tokens are added to the training flow, such xml tags are predicted as a single token rather than a sequence of characters
I added custom special tokens for my internal document classification needs and the accuracy boost was really good. It took some time in digging (around a day) to understand that having special tokens according to the needs increases perfomance
So it will be better if there is a section wherein this is explained in the readme
Thanks for creating this phenomenal model though!!!
Edit :
I can add them in a PR to make your life easy

Can you share some examples on how you deal with the special tokens?
That would be really useful.

Due to an NDA I cannot exactly share my document classes, but donut's own internal train code performs this
https://github.com/clovaai/donut/blob/master/train.py#L66-L70
If you look at this line , this line is responsible for adding special tokens. So i created special tokens according toy use case and there was a good accuracy boost
Before implementing the same, the accuracy was a bit low along with some json decoding issues and random charecters showing up

I'll second the need to document the use of special tokens. Both the why and how.


MC_44: Mountains go beyond the chart area if kept max limit of x-axis at minimum number

RELEASE: v0.6.0
SEVERITY: Major
MODULE: Mountain Chart
OS/BROWSER: Windows 7 / Chrome 45.0
STEPS TO REPRODUCE:

Navigate to URL: http://static.gapminderdev.org/vizabi/release/v0.6.0/preview/mountainchart.html
Mountain Chart page will be loaded
Click on axes						                                                                          4) Fill any minimum value i.e. 10, 15 etc. in “Max” text filed of X-axis limits
Note that label of menu is appeared as GO BIG

ACTUAL RESULTS:
Mountains of chart is not giving their values for countries that have values greater than kept in max limit of x-axis
EXPECTED RESULTS:
There’s should be some validation check i.e. max value should be at least at 50-100 as all countries value fall in this range
SCREENSHOT:

i.e. max value should be at least at 50-100 as all countries value fall in this range

But if user wants to see only those countries, where salary is in range 1-10 and etc?

Agreed with @IncoCode. It should allow min and max to be anything, even if not all countries are included.


Add docs for methods on browser added by the runner

Namely, getProcessedConfig and forkNewDriverInstance

See https://github.com/angular/protractor/pull/2295

This is done.


Add default pagination size

Resolves: #178

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.77%. Comparing base (6460943) to head (420538d).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #181   +/-   ##
=======================================
  Coverage   99.77%   99.77%
=======================================
  Files         164      164
  Lines        2657     2671   +14
  Branches      613      600   -13
=======================================
+ Hits         2651     2665   +14
  Misses          6        6

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests
JS Bundle Analysis - Avoid shipping oversized bundles


Add user agent for azure so the azure cli can pick it up

TODO: Determine if it's possible to set the user agent of the azure resource provider

Here's a search of user agent in the azure rm provider. AWS implements a customizable user agent so I have hope we can too! https://github.com/search?q=repo%3Ahashicorp%2Fterraform-provider-azurerm user agent&type=code

Sounds like we just need to set the standard azure user agent env var and the azure resource will pick it up, same for aws as well.
https://github.com/hashicorp/terraform-provider-azurerm/issues/2393
We can have the terraform mixin inject these env vars (like the az and aws mixins do today, but with opt-out) when their respective cloud resource providers are used.


Invalid escape code in ld+json script tag

What version of Hugo are you using (hugo version)?
Hugo Static Site Generator v0.59.1/extended windows/amd64 BuildDate: unknown
Does this issue reproduce with the latest release?
Yes

Hugo generate invalid JSON in ld+json script tag when escaping Unicode/special character.
When validating you get the error Bad escape code \x
The expected escape code for JSON is \u0000

How to produce
content/_index.md

---
layout: sample
---

layouts/_default/ sample.html

{{ $title := "a & b" }}

output

expected output

Probably similar/related to https://github.com/gohugoio/hugo/issues/6695

Closing as duplicate of #6695


Implementing the additional checkout object fields (and more)

Hi, thanks for the great package - really great to see some existing work being done on lemonsqueezy!
Looking at the  lemonsqueezy docs, I see that the create checkout method has quite a few additional fields which aren't current being exposed by the lib.
Are there any plans to add support for these in the future? I'm specifically looking to change the product_options.redirect_url field so that I can link people to my own site.
I would imagine adding these fields now would be a breaking change, so these might need to be added through a refactor, adding golang structs with the omitempty JSON annotation, rather than manually crafted maps that aren't currently in use. (Eg; I faced some errors earlier for not specifying the button colour, even though this isnt something that was necessary for me, and assumedly would have been fine had the field not been specified)
How does that sound? Happy to open a PR for this if there is any interest.

Hello @PaulSonOfLars
While looking at the code again, I see the redirect_url field here https://github.com/NdoleStudio/lemonsqueezy-go/blob/main/checkout.go
Am I missing something?

Yep I see now @PaulSonOfLars

This is why i suggested reusing the JSON struct passed in as params, with added json and omitempty tags, to make go's marshalling work for us instead; skipping the manual map entirely :)

I like this idea. Feel free to create the PR. I promise to review this promptly.

Happy to close this since the PR is in. Thanks @AchoArnold!


better audio freq accuracy and allow application-controlled logs

Hello,
Thank you for your work around libdaisy in rust. This was really helpful for me.
This pull request contains 2 modifications:

allow an application to provide its own logs and panic handler
increase the accuracy of the audio frequency

Regards

Thanks @mtthw-meyer !


Builder Class

So I've been reading this book about Java best practices and have been really digging the discussion on Builder classes. Very powerful. However, a full-blown Builder class has certain characteristics that would cause of lot of change depending on the class. But with FXTrayIcon, I was able to implement a Builder class without losing any original functionality. The class constructors have the same mandatory parameters as FXTrayIcon. The only difference is that in the second constructor, I add detection for Mac and then instantiate with a 26,26 icon as those are much better on macs than the 16,16.
Once the Builder class is instantiated, it creates a new instance of FXTrayIcon then basically continues to build on that instance until the build call is made at which point it only passes that instance of FXT back to the application.
Besides being convenient, there are some benefits to having this class. For example, addTitleItem and addExitItem are booleans that only affect future change, whereas, in Builder style, it will exist in FXT by the end of the sentence.

		URL url = Form.class.getResource("FXIconRedWhite.png");
		FXTrayIcon trayIcon = new FXTrayIcon.Builder(Switcher.getDefaultStage(), url)
				.menuItem("Load EML Files", e -> loadEMLFiles())
				.menuItem("Load Class Files", e -> loadClassFiles())
				.separator()
				.menuItem("Save Active Message", e -> saveActiveEmail())
				.menuItem("Save All Messages", e -> saveActiveEmail())
				.menuItem("Save Master HTML", e -> Helper.saveHTMLFile(masterHTMLString))
				.separator()
				.menuItem("Generate Master HTML", e -> createFinalHTML())
				.separator()
				.menuItem("Exit Program", e -> System.exit(0))
				.show()
				.build();

The current way of doing the exact same thing would be like this:
		URL url = Form.class.getResource("FXIconRedWhite.png");
		MenuItem menuLoadEML = new MenuItem(		"Load EML Files");
		MenuItem menuLoadClass = new MenuItem(		"Load Class Files");
		MenuItem menuSaveMessage = new MenuItem(	"Save Active Message");
		MenuItem menuSaveMessages = new MenuItem(	"Save All Messages");
		MenuItem menuSaveHTMLFile = new MenuItem(	"Save Master HTML");
		MenuItem menuGenerateHTML = new MenuItem(	"Generate Master HTML");
		MenuItem menuExit = new MenuItem(			"Exit Program");
		menuLoadEML.setOnAction(e->{
			loadEMLFiles();
		});
		menuLoadClass.setOnAction(e-> loadClassFiles());
		menuSaveMessage.setOnAction(e-> {
			saveActiveEmail();
		});
		menuSaveMessages.setOnAction(e-> {
			Helper.saveAllToClassFiles(emailList);
		});
		menuGenerateHTML.setOnAction(e->createFinalHTML());
		menuSaveHTMLFile.setOnAction(e->{
		});
		menuExit.setOnAction(e-> System.exit(0));

		FXTrayIcon traycon = new FXTrayIcon(Switcher.getDefaultStage(),url,26,26);
		traycon.addMenuItem(menuLoadEML);
		traycon.addMenuItem(menuLoadClass);
		traycon.addSeparator();
		traycon.addMenuItem(menuSaveMessage);
		traycon.addMenuItem(menuSaveMessages);
		traycon.addMenuItem(menuSaveHTMLFile);
		traycon.addSeparator();
		traycon.addMenuItem(menuGenerateHTML);
		traycon.addSeparator();
		traycon.addMenuItem(menuExit);
		traycon.show();

With the Builder class, it is technically two lines of code, including the show() method whereas the current way, it. is close to 30 lines of code.
You don't have to create MenuItems and then pass them into FXTI, all you need to do is use the menuItem Build option and pass in a string for the label or a String and an Event Handler., you can also call show from the Builder sentence, and set any and every possible option in that one line.
I also added two example classes to the test package. One which was a copy of one you already had, only using the Builder class and the other one is a demo of switching icons after instantiation ... with a splash of color ☺.
That purple chain link icon was difficult to see in a dark theme so I made new icons to see how they would look and of the ones I made the red icon seems to work best in both light and dark mode.
Here are the icons.

@EasyG0ing1 If you're done with changes, I'd like to merge this. It looks great. Please confirm before I do. Thank you!

@dustinkredmond - Yeah, I'm done .. I was just working through those external tests you have set up ... it was failing them for little things ... dotting I's .. .crossing T's ... etc.
:-)

@dustinkredmond  - I added one blurb to the README at the top... that will be my last change ... for real this time. :-)

Perfect, looks good. I'll go ahead and merge and shoot for a release by week's end. As always, thanks for your contributions!


[Code Health] Migrate Gitlab templates and drop .gitlab folder.

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
No response
Expected Behavior
No response
Steps To Reproduce
No response
Anything else?
No response
What browsers are you seeing the problem on?
No response

Should I go ahead with this?


Update logging to use user-friendly formatting

Use k8s.io/kubernetes/pkg/kubelet/util/format package for more user-friendly formatting of logging

LGTM, small incremental change.
One thing, you added a package to imports so the BUILD file must be updated. You can run ./hack/update-bazel.sh from main Kubernetes directory.

@pablitoergosum If you want to have it under the k8s memory manager PR, you should create your PR vs feature-memory-manager-working branch.


Add Tail Sampler action to aggregate processor

Description
Add tail sampler action to aggregate processor
This processor keeps state, by collecting all events in a group, across different aggregation periods and takes action on events only after the aggregation group is idle for more than waitPeriod time. At this point, if the group has an error as defined by error_condition then all events collected are sent as concluding events. If there is no error, then a probabilistic sampling is done based on the configured percent value.
Example configuration for the processor -
 processor:
    - aggregate:
        identification_keys: ["id"]
        action:
          tail_sampler:
            percent: 20
            wait_period: "10s"
            error_condition: "/status == 10"
        group_duration: "5s"

Resolves #2572
Issues Resolved
#2572
Check List

[ X] New functionality includes testing.
[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[X ] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2497 (b366fc9) into main (2e37e23) will decrease coverage by 0.14%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main    #2497      +/-   ##
============================================
- Coverage     93.39%   93.26%   -0.14%
+ Complexity     2122     2120       -2
============================================
  Files           251      251
  Lines          5923     5923
  Branches        480      480
============================================
- Hits           5532     5524       -8
- Misses          264      275      +11
+ Partials        127      124       -3

see 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

merging for now. Will address minor comments from Taylor in a different PR.


Fix Travis Build

FYI http://www.smartjava.org/ is down at the moment @josdirksen

Thanks for the update. Looking into it right now. Seems an issue with my DNS provider :(

Thanks a lot @ReadmeCritic
@josdirksen just in case, your article is here too: https://dzone.com/articles/10-practical-docker-tips-for-day-to-day-docker-usa it is safer to put that link but it might bring less traffic to your site. Your call.

Well it took some effort, but we're back. @ReadmeCritic if you'd be so nice to put the link to my site back again :)

@josdirksen done. https://github.com/veggiemonk/awesome-docker/commit/b86e72ff762d56e5f30f7d8cc5d4c13e0968d972

Thanks!
On Tue, Apr 12, 2016 at 8:07 PM, Julien Bisconti<EMAIL_ADDRESS>wrote:

@josdirksen https://github.com/josdirksen done. b86e72f
https://github.com/veggiemonk/awesome-docker/commit/b86e72ff762d56e5f30f7d8cc5d4c13e0968d972
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/veggiemonk/awesome-docker/pull/168#issuecomment-209034796


help

ImporteError:cannot import name "kinefx utils"from kinefx(C:\PROGRA~1/SIDEED~1/..../python3.9libs\kinefx_init_.py)

all these tools very useful.thanks. don not know how to effect use

OK now.thanks  so beautiful tools


[WIP] Update documentation for Timer Mocks

Summary
The Timer Mocks guide in the documentation is currently out of date with examples based on the old legacy timers. Very few of the examples work anymore, and this seems to generate quite a few bug reports and support requests in the issue tracker and across various other channels.
My goal with this PR is to bring it up to date, and also change the structure slightly to drive home some of the more important points early on.
The PR contains some minor tweaks to the website to accommodate this:

CSS for supporting line highlighting in docusaurus
CSS to darken the background color of Note style admonitions in dark mode, as it used a very bright grey.

Preview
A preview is available here. Feedback is welcome :-)
Todo
Please note that this is a "work in progress"

[x] Basic structure
[x] Code example for enabling fake timers
[x] Code example for recursive timers
[ ] Code example for advancing by time
[ ] Code example for stepping through timers
[ ] Move examples/timer -> examples/timer/legacy
[ ] Add new code examples under examples/timer/modern
[ ] Add F.A.Q.s -- Suggestions, anyone?
[ ] Consider tweaking colors of highlighting and 'Note' admonition
[ ] Fine tune the language

Test plan
n/a
Supersedes #11324, #10964, #10022
Closes #11713

Codecov Report

Merging #11731 (f25caff) into master (fdc74af) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #11731   +/-   ##
=======================================
  Coverage   69.01%   69.01%
=======================================
  Files         312      312
  Lines       16335    16335
  Branches     4734     4734
=======================================
  Hits        11273    11273
  Misses       5034     5034
  Partials       28       28

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 fdc74af...f25caff. Read the comment docs.

(sorry, misclicked)

The PR contains some minor tweaks to the website to accommodate this:

CSS for supporting line highlighting in docusaurus
CSS to darken the background color of Note style admonitions in dark mode, as it used a very bright grey.

Could you separate those out into another PR so this one can be just the changed text?

Could you separate those out into another PR so this one can be just the changed text?

Yep, can do; I just thought it would make sense to keep them here while it's a draft so that it can be previewed together. Since afaik line highlighting or the admonition in question aren't used elsewhere so far. I would suggest that I cherry pick these out when the PR is all tied up and ready.
I also noticed today that my F.A.Q. answer is inaccurate; since it's still possible to spy on the global yourself. So I will adjust that and try to think of another way to encourage the point I was trying to make re: decoupling from implementation details

Lost track of this, sorry @sigveio! I really like these changes, would you be interested in reviving this PR?

No worries, @SimenB - and sorry for not following up myself lately. If this is still needed I can try and pick the ball back up. :-) Very happy to see that you also continued work on #11778 while I was AWOL. Thanks! ❤


Sample rate hard-coded at 44100Hz

The sample rate of ChuckMain node is hard-coded at 44100Hz. The frequency would be inaccurate if the project setting is using sample rate other than 44100Hz.

@gewang As a workaround, I created static functions for getting and setting sample rate for all ChucK main instances. I'm still looking for a way to read the sample rate that UE is using.

when you create your operator you have InSettings, you can get the sample rate from that: InSettings.GetSampleRate()


Ignore tags not on dev or main

Description of change
This will ignore tags not on dev and main (e.g. support branches)
Type of change
Add an x to the boxes that are relevant to your changes.

[X] Bug fix (a non-breaking change which fixes an issue)
[ ] Enhancement (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Fix

How the change has been tested
Describe the tests that you ran to verify your changes.
Make sure to provide instructions for the maintainer as well as any relevant configurations.
Change checklist
Add an x to the boxes that are relevant to your changes.

[X] I have followed the contribution guidelines for this project
[X] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

For context: hotfix versions released on support branches (like support/wasm-v0.4.1) appear in the changelog but all related PRs from that branch are missing. E.g. when releasing 0.5.0 on dev, 0.4.1 would appear in its changelog but have no changes since its merge commits are on the support branch and not merged back into dev.


chore(ci): rename integration test job

Stack:

#2657 ⬅
#2650

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is