Not consistent shutdowns and no way to know if shut down?

I've created a reproducible (running on 0.9.3):
use mdns_sd::{ServiceDaemon, ServiceInfo};

fn main() {
    let mdns = ServiceDaemon::new().unwrap();
    mdns.shutdown().unwrap();

    let service_type = "_mdns-sd-my-test._udp.local.";
    let instance_name = "my_instance";
    let ip = "<IP_ADDRESS>";
    let host_name = "<IP_ADDRESS>.local.";
    let port = 5200;
    let properties = [("property_1", "test"), ("property_2", "1234")];

    let my_service = ServiceInfo::new(
        service_type,
        instance_name,
        host_name,
        ip,
        port,
        &properties[..],
    ).unwrap();

    mdns.register(my_service).unwrap();

    mdns.shutdown().unwrap();
}

This example consists of creating a daemon, shutting it down then registering a service and then shutting it down again, after running it multiple times, sometimes I get panics (due to unwrapping in the example) and sometimes not, the problem here is that this is not consistent.

Console output
cosminperram@cosminsmbpro untitled % cargo r
Finished dev [unoptimized + debuginfo] target(s) in 0.02s
Running target/debug/untitled
thread 'main' panicked at src/main.rs:23:31:
called Result::unwrap() on an Err value: Msg("flume::channel::send failed: sending on a closed channel")
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
cosminperram@cosminsmbpro untitled % cargo r
Finished dev [unoptimized + debuginfo] target(s) in 0.01s
Running target/debug/untitled

Another problem would be of how could I know if the daemon has a closed channel? Looking for something simple like is_running.
Hope I haven't missed anything related these issues in docs, if I did, please state so.
Thanks for this library!

sorry I was out for a few days.  I will look into this and get back to you.

Thanks for opening this issue. I've created a PR #149 to address this. Let me know if that helps your case.

Looks great, thanks for the quick implementation.

PR is merged into main. If you can check whether it's working for you, that'll be great! If any issues, please also let me know, thanks!

Hey, I didn't had the time to do it today, I'll definitely try it tomorrow.

Nevermind my last message, I got to try status and shutdown and they work as expected, thanks, awaiting the new release (:

The new release is published ;-)


feat: add variant of StorageLockError

This PR addreses a new variant of StorageLockError

Oops... I didn't think about it. Sorry.


Add server GC benchmarks and update README

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

Comparison is base (00ae19c) 99.17% compared to head (0d9dfe4) 99.17%.

:exclamation: Current head 0d9dfe4 differs from pull request most recent head 518cf18. Consider uploading reports for the commit 518cf18 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #79   +/-   ##
=======================================
  Coverage   99.17%   99.17%
=======================================
  Files          45       45
  Lines        3269     3269
  Branches      348      348
=======================================
  Hits         3242     3242
  Misses         24       24
  Partials        3        3

Flag
Coverage Δ

Debug
98.56% <ø> (ø)

Release
99.12% <ø> (ø)

macos-latest
99.12% <ø> (+0.25%)
:arrow_up:

ubuntu-latest
98.86% <ø> (ø)

windows-latest
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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


fix-#1190/full-screen-issue

PR Fixes:
The navbar is causing the video to be cropped.
Before:
https://drive.google.com/file/d/115GaoI15Ee0-S3WgZ8s9x7sq5lIhJ8U-/view?usp=sharing
Resolves #1190
After:
https://drive.google.com/file/d/1xLNw2mg5O10dr9CvsfNg3LgdhqJ0PUkp/view?usp=sharing
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I assure there is no similar/duplicate pull request regarding same issue

what's fixed?

what's fixed?

the nav bar is causing the problem here cropping the video when you zoom 100% I just made nav bar stick to top

what's fixed?

the nav bar is causing the problem here cropping the video when you zoom 100% I just made nav bar stick to top

but video is still getting cropped.

There are 7k changes. Closing for now. Resolve the conflicts and can u comeup with latest updated pr?


A standard for usage of snake case, camel case, pascal case etc?

For example, I'm using snake case for variables and object properties, camel case for functions/methods and pascal case for classes. And it is really sad when someone else starts to use a different approach in the same project.
A rule for which case type to use when defining of variables, functions and methods etc would be highly appreciated!
Thanks!

Virtually every javascript I've seen uses camelCase for both variable and function names, and PascalCase for class names. I would love seeing this enforced in standard.
It's important that it only checks when declaring new variables though. Also, it should allow leading _ since that is by convention the way we use to mark variables as private.

:+1: for enforcing camelCase on vars/functions, PascalCase on classes, and ALL_CAPS on constants. Given that the functions that are built into JS use that naming style, I think it's reasonable to add it to standard.

I disagree with using all caps with const variables. As do the developers working on Node by the looks of it. I am using const in my node modules when requiring modules. All caps would look very messy.

I also dislike all caps for constants, but sadly everyone in the React/Redux community uses them for actions, so it'd break a lot of stuff. Of course, this is supposed to be an opinionated project so ultimately it's up to @feross.

I am using const instead of var and let everywhere in my code where it's possible, to make it more readable. It's much easier to reason about code when it is dead obvious what can change and what cannot change. And I am using ALL_CAPS only for global constants, and snake_case for local ones

@LinusU I agree with you, would like to see what breakage this would cause in the ecosystem before considering it.

I am with @grantcarthew and @ilearnio; const is used for a lot of things besides the traditional "variable holding a constant literal". It would be pretty arbitrary to enforce a different name just because one chooses to make a value immutable.

@simonratner Yes, this is a good point. I don't think we can ever assume that all const variables are the top-of-the-file or global constant kind.
But enforcing camel-case instead of snake case is more likely to work, i.e. not cause a bunch of test failures :)

Camel case variables are now enforced in standard 7.0.0-beta.0.

Any suggestion how to solve scenario where I have keys in a dataset from an API (I can't modify it) and want to use object destructuring assignment?
let {   route_id,   std_date,   std_time } = flight

I suppose this would work:
let { route_id: routeId, std_date: stdDate, std_time: stdTime } = flight

So obvious. Thanks! :)

I use snake_case to avoid the sort of mappings that @bganicky mentioned
let { route_id: routeId, std_date: stdDate, std_time: stdTime } = flight

And also, personally, I think it reads better.
I like Standard so i'm willing to make the change but It's going to be a nightmare to change all of my codebase by hand.
Anyone know of a auto formatter that supports this rule?

@mderazon use eslint with eslint-config-standard then just override the rule for camelcase.

@mderazon Yeah, I respect your decision to use eslint directly. standard has the goal of reducing ambiguity so that we can all give less feedback in PRs, so I think standardizing on camel case is a net win, even if it makes it a bit more annoying to interact with APIs that use snake_case. It helps that there's no enforcement on property names.

I wanted to complain bitterly because I had a project that used standard and was fine, and then I came back to it after months and now it's telling me to use camelCase and I thought... "Really?  Functions and methods use camelCase, but surely people use snake_case for local variables..." and then I look around (literally, I just looked through a bunch of files in express, babel, and even coffee-script) and couldn't find any snake_case.
"But I used snake_case?! Where on earth did I learn that?" I wondered. Then it hit me: python. I quickly googled "python tutorial" and the first one that pops up used snake_case:
>>> the_world_is_flat = True
>>> if the_world_is_flat:
...     print("Be careful not to fall off!")
...
Be careful not to fall **off!**

"I'm not crazy!" I thought. "I came to JavaScript from CoffeeScript; I bet I carried that over from Python!" I rushed to the CoffeeScript website to confirm, but no! The CoffeeScript docs used camelCase. "Am I going crazy?" I thought. "Where the heck did I pick up this convention?" But finally, I went to the CoffeeScript Cookbook site, and found a good many of the code snippets there used snake_case... inconsistently.
TL;DR And that's the story of how I realized I've been naming variables weird for a long time. A tale of anger, curiosity, confusion, panic, soul-searching, and (finally) self-understanding.

@wmhilton Heh, thanks for sharing your story ;-)


ESPAsync_WiFiManager and ArduinoIoTCloud libraries are incompatible.

Describe the bug
When trying to use an access point provided by ESPAsync_WiFiManager with the ArduinoIoTCloud functionality in the same Sketch, simply adding #include <ArduinoIoTCloud.h> to an ESPAsync_WiFiManager example project causes the access point web server to hang. This issue has been filed against ArduinoIoTCloud as well.
Steps to Reproduce
Add #include <ArduinoIoTCloud.h> to an ESPAsync_WiFiManager example. When the AP is accessed, the web server hangs.
Expected behavior
I expected to be able to use ESPAsync_WiFiManager to maintain a wifi connection, and when the connection was active, use ArduinoIoTCloud to control an ESP8266 node.
Actual behavior
When the Captive Portal is accessed by a web browser, the web server hangs and a blank page results.
Debug and AT-command log (if applicable)
It appears that the page is rendered correctly, but the web server hangs when it is served.
Screenshots
It's a blank page.
Information
Take any ESPAsync_WifiManager example and put the header #include <ArduinoIoTCloud.h> into the .ino and the captive web page will hang.
Please ensure to specify the following:

Arduino IDE version  1.8.15
ESP8266 D1 mini
Contextual information (e.g. what you were trying to achieve): I'd like to have the WiFi configuration provided by ESPAsync_WifiManager keep the ESP8266 connected so that I can use the Arduino IoT Cloud to control the device. So to start with I set up a ESPAsync_WiFiManager project and simply put the ArduinoIoTCloud header into the project.
Simplest possible steps to reproduce:

Steps to reproduce:

Get the most recent ArduinoIoTCloud and ESPAsync_WiFiManager libraries.
Set up an ESPAsync_WifiManager manager project such as Async_AutoConnectWithFeedbackLED
add #include <ArduinoIoTCloud.h> to the project.
It will compile, load to the ESP8266 and run.
When you try to connect to the Captured Portal, the browser will connect, the correct content appears to be delivered by viewing the source in the browser's development mode, but the page will be blank.

If NTP config is enabled in the ESPAsync_WifiManager, the captured portal web page does not render. So this comment is also advisable when using ArduinoloTCloud with ESPAsync_WiFiManager.
// Use false to disable NTP config. Advisable when using Cellphone, Tablet to access Config Portal.
// See Issue 23: On Android phone ConfigPortal is unresponsive (khoih-prog/ESP_WiFiManager#23)
#define USE_ESP_WIFIMANAGER_NTP false

Hi @kirk-clendinning
Thanks for your interest in the library.
You have to understand that ArduinoIoTCloud library is a complete ecosystems for many boards, besides ESP8266, to manage the connection to ArduinoIoTCloud, including local SSID / PWD, and many more complex features
Nobody can expect this library, managing Async WebServer and connection, to co-exist with the complex synchronous ArduinoIoTCloud library. I don't think even the synchronous ESP_WiFiManager library can do this, without spending lots of time to study / debug and have many modifications to make them working together.
Anyway, this is redundant and I won't spend time on this, and don't expect anybody will do so.
Good Luck,

In the end, it appears that the conflict had to do with the NTP implementations. By disabling that in both libraries, the combination worked correctly.
Thanks for responding.
-kirk-

On Aug 3, 2021, at 15:23, Khoi Hoang @.***> wrote:
Closed #59 https://github.com/khoih-prog/ESPAsync_WiFiManager/issues/59.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/khoih-prog/ESPAsync_WiFiManager/issues/59#event-5106702046, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADCLV3ME4RYQOVCLFWMZSALT3A63ZANCNFSM5BDMAJAA.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.

I'm amazed that you figure out the solution.
Thanks for your information and research, which will help many other users.
Best Regards,

Sadly, I did not have time to go into depth. Since the ESP8266 has no RTC, it may be that both of the libraries were trying to replace that in some way with NTP, and so there was some sort of conflict. I find the Arduino development environment a bit tedious since one cannot step for debugging. It is sometimes difficult to find issues in asynchronous libraries.
Thank you for your contributions to the community.
-kirk-

On Aug 3, 2021, at 15:35, Khoi Hoang @.***> wrote:
I'm amazed that you figure out the solution.
Thanks for your information and research, which will help many other users.
Best Regards,
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/khoih-prog/ESPAsync_WiFiManager/issues/59#issuecomment-892110340, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADCLV3I4P4SM3U2B6FV47Z3T3BAHRANCNFSM5BDMAJAA.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.


Incorrect Range on hasAbstractObjectReference data property

C2SIM Problem Report / Change Request
Submitter Name (s): Martin Johansson (entered by Curtis Blais)
Contact email address: Martin Johansson<EMAIL_ADDRESS>Submitter Organization(s): Pitch
Date of Submission: 13 February 2023
Type (Problem Report or Change Request): Problem Report
Product: C2SIM core ontology
Problem/Change Description:
Object property hasAbstractObjectReference does not have a range defined in the C2SIM Core ontology. The range needs to be set to UUIDBase.

XML schema in support of CWIX 2023 was generated for the merged Core-SMX-LOX ontology on 2/1/2023, to include the change for this issue.


Fix NetworkTools compilation after commit fc02d38b8

#1105

That's what we have now (after the fix in this PR): [ERROR] (1) D:\Dev\ProcessHacker3\src\trunk\plugins\NetworkTools\country.c(613,24): error C2198: 'PhImageListDrawIcon': too few arguments for call [D:\Dev\ProcessHacker3\src\trunk\plugins\NetworkTools\NetworkTools.vcxproj]


v0.69.1 breaks compatibility with JDK 8.

Given demo.java
public class demo {
    public static void main(String...a) {
        System.out.println("Hallo");
    }
}

Running
docker run -v `pwd`:/ws --workdir=/ws jbangdev/jbang-action:0.69.1 --java 8 demo.java

fails with
jbang --java 8 demo.java
[jbang] Building jar...
[jbang] Downloading JDK 8. Be patient, this can take several minutes...
[jbang] Installing JDK 8...
[jbang] Default JDK set to 8
Error: A JNI error has occurred, please check your installation and try again
Exception in thread "main" java.lang.UnsupportedClassVersionError: demo has been compiled by a more recent version of the Java Runtime (class file version 55.0), this version of the Java Runtime only recognizes class file versions up to 52.0
	at java.lang.ClassLoader.defineClass1(Native Method)
	at java.lang.ClassLoader.defineClass(ClassLoader.java:756)
	at java.security.SecureClassLoader.defineClass(SecureClassLoader.java:142)
	at java.net.URLClassLoader.defineClass(URLClassLoader.java:468)
	at java.net.URLClassLoader.access$100(URLClassLoader.java:74)
	at java.net.URLClassLoader$1.run(URLClassLoader.java:369)
	at java.net.URLClassLoader$1.run(URLClassLoader.java:363)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.net.URLClassLoader.findClass(URLClassLoader.java:362)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:418)
	at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:352)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:351)
	at sun.launcher.LauncherHelper.checkAndLoadMain(LauncherHelper.java:601)

Where as docker run -v pwd:/ws --workdir=/ws jbangdev/jbang-action:0.68.0 --java 8 demo.java say
[jbang] Downloading JDK 8. Be patient, this can take several minutes...
[jbang] Installing JDK 8...
[jbang] Default JDK set to 8
[jbang] Building jar...
[jbang] There is a new version of jbang available!
[jbang] You have version 0.68.0 and 0.69.1 is the latest.
[jbang] Use your package manager to update jbang to the latest version,
[jbang] or visit https://jbang.dev to download and install it yourself.
Hallo

Hmmm....that shouldn't happen! Thanks for reporting.
I noticed action build was using java11 where I didn't expect it. Wonder if default Java version changed ?
Investigating...

this is not just the action - something fishy. moving to the main repo


Add button to copy device IP from Brighterscript Language > Devices

There's only a button to open the IP on a browser, but no button to copy the IP. Would be nice to have a 1-click for this.

I'll work on this when I have time, next weekend probably. Just wanted to create an issue for tracking.

Great, thanks!
Here's the relevant section of code you'll need to modify.
https://github.com/rokucommunity/vscode-brightscript-language/blob/9deae16322d04538839c46980fde679dfae087a2/src/viewProviders/OnlineDevicesViewProvider.ts#L100


⚠ Bofrost.fr has degraded performance

In 814b505, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 891 ms

Resolved: Bofrost.fr performance has improved in 2b70547 after 311 days, 6 hours, 17 minutes.


multiple_inheritance.md

Description 📜
In this submission, an audio has been submitted explaining the concept of multiple inheritance in C++.
Fixes #5104

Type of change 📝

[x] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[ ] Doc#umentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[x] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] JavaScript_#Domain
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

Hi there! Kindly check if there are any changes that need to be made.

@vasu-1 and @Suvraneel please have a look.
And let me know in case of any changes.
Thank you!


Inheritance Tree Dunders

Dunders __base__, __bases__, __subclass__
Examples are shown as molecules

(This one is ready for review btw)

What's the point of having get_class_name methods if they're not used? 🤔

haha, good point

How do you feel about including an __mro__ example for the Ant class?


trying to install throwing an error

f1@LagMachine:~$ go get -u github.com/cybozu-go/transocks/...
package github.com/cybozu-go/transocks         imports math/bits: unrecognized import path "math/bits" (import path does not begin with hostname)
package github.com/cybozu-go/transocks         imports github.com/apparentlymart/go-textseg/v13/textseg: cannot find package "github.com/apparentlymart/go-textseg/v13/textseg" in any of:         /usr/local/go/src/github.com/apparentlymart/go-textseg/v13/textseg (from $GOROOT)         /home/f1/tutorialspoint/sample/src/github.com/apparentlymart/go-textseg/v13/textseg (from $GOPATH)
package github.com/cybozu-go/transocks         imports github.com/hashicorp/hcl/hcl/printer: cannot find package "github.com/hashicorp/hcl/hcl/printer" in any of:         /usr/local/go/src/github.com/hashicorp/hcl/hcl/printer (from $GOROOT)         /home/f1/tutorialspoint/sample/src/github.com/hashicorp/hcl/hcl/printer (from $GOPATH)
package github.com/cybozu-go/transocks         imports github.com/pelletier/go-toml/v2/internal/ast: cannot find package "github.com/pelletier/go-toml/v2/internal/ast" in any of:         /usr/local/go/src/github.com/pelletier/go-toml/v2/internal/ast (from $GOROOT)         /home/f1/tutorialspoint/sample/src/github.com/pelletier/go-toml/v2/internal/ast (from $GOPATH)
package github.com/cybozu-go/transocks         imports github.com/pelletier/go-toml/v2/internal/danger: cannot find package "github.com/pelletier/go-toml/v2/internal/danger" in any of:         /usr/local/go/src/github.com/pelletier/go-toml/v2/internal/danger (from $GOROOT)         /home/f1/tutorialspoint/sample/src/github.com/pelletier/go-toml/v2/internal/danger (from $GOPATH)
package github.com/cybozu-go/transocks         imports github.com/pelletier/go-toml/v2/internal/tracker: cannot find package "github.com/pelletier/go-toml/v2/internal/tracker" in any of:         /usr/local/go/src/github.com/pelletier/go-toml/v2/internal/tracker (from $GOROOT)         /home/f1/tutorialspoint/sample/src/github.com/pelletier/go-toml/v2/internal/tracker (from $GOPATH)
i have go version >1.7 so i dont know why i would get this error , can someone help me?

Install go 435.03M
go version
go version go1.19.3 linux/amd64

After go get, go build,  or go install,  a lot of files in ~/go ~/.cache/go-build
But NO executable found!
Now I use this https://github.com/cybozu-go/transocks/issues/29#issuecomment-913555490 5.8M
Better to use github actions  to auto-build executable file for each release!

To build an executable, just run:
go install github.com/cybozu-go/transocks/cmd/...@latest


fix(core-integration): handle subscription error

About the Contributor

This pull request is posted on behalf of the NRK.
Type of Contribution
This is a:

Bug fix
Current Behavior

When using core-integration (so gateways or other external applications):
const coreConnection = new CoreConnection(coreOptions)
await coreConnection.autoSubscribe('rundownPlaylists', myArguments)

If there is an error when subscribing (for example: a Match.test() throws, due to invalid arguments), the error is swallowed silently.
New Behavior

The error is now thrown.
Testing Instructions

Other Information

Status

[x] PR is ready to be reviewed.
[x] The functionality has been tested by the author.
[x] Relevant unit tests has been added / updated.
[x] Relevant documentation (code comments, system documentation) has been added / updated.

Should this be targetting 51, or an older release?

Should this be targetting 51, or an older release?

I'll cherry pick this fix onto r49 and r50 as well 👍


JSON decoding using regex breaks if model outputs a "

As topic suggests.
Using pydantic -> regex
It causes model to subsequently output blank as the logit mask goes on.
Is there any way to escape " produces by the model in a json field?

@aliencaocao Can you provide more details or code to reproduce your problem?

Its a common issue as gpt4 json mode also have this problem. If a json field is string, and model decides to include double quotes in this string, it prematurely ends the field as the regex will match it as an ended string, but model does not think so, so it keep trying to output something other than the next key of the json, which after logit masking by sglang, becomes all tokens that appear as blank in console. Just have any json as regex and instruct the model to output double quotes will trigger this.

I think this is an inherent issue of json string enforcement. There is no good way to tell when a model output ends a "string" since everything is a string. json string can contain any character even special format ones like \n. The model is not trained to escape chars the way json expects so there is no tell " vs \".

Its a common issue as gpt4 json mode also have this problem. If a json field is string, and model decides to include double quotes in this string, it prematurely ends the field as the regex will match it as an ended string, but model does not think so, so it keep trying to output something other than the next key of the json, which after logit masking by sglang, becomes all tokens that appear as blank in console. Just have any json as regex and instruct the model to output double quotes will trigger this.
@aliencaocao
We can use the following regular expression to match string values in JSON. I am confused why the " still exists when it is prohibited in this regular expression?  I also occured the same problem.

("(?:[^"\\x00-\x1f\x7f-\x9f]|\.)*"|null)

Its a common issue as gpt4 json mode also have this problem. If a json field is string, and model decides to include double quotes in this string, it prematurely ends the field as the regex will match it as an ended string, but model does not think so, so it keep trying to output something other than the next key of the json, which after logit masking by sglang, becomes all tokens that appear as blank in console. Just have any json as regex and instruct the model to output double quotes will trigger this.

@aliencaocao Thank you for your explanation. Do you know where the relevant code is? I hope to have a deeper understanding of this issue in sglang.


Add hash to uploading images

Ensure that all uploading images will have a unique name.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-2820 Image override when using the same name


when running bootstrap.image.upload, i got a signal 2 (then i killed it with ctrl-c)

Building generic words...
Serializing words...
Serializing JIT data...
Serializing global namespace...
Serializing special object table...
Performing word fixups...
Performing header fixups...
Image length: 657012
Object cache size: 58238
Writing image to /Users/erg/factor-pristine/boot.unix-x86.64.image...
Killed by signal 2.
^CKilled by signal 2.
Interrupted
Starting low level debugger...

Not handling EINTR correctly?

scumbag random clicks


Launcher variant

Suggestion to make a luancher version of this app with a small app drawer feature for those who just intend to use this app on handhelds, specially to gamepass gaming

Too much work


Unable to set custom home directory in DummyAuthorizer().add_user()

Thanks for the wonderful package!
However, I ran into a ValueError: no such directory when trying to run the example from the DummyAuthorizer class:
from pyftpdlib.authorizers import DummyAuthorizer
authorizer = DummyAuthorizer()
authorizer.add_user('user', 'password', '/home/user', perm='elradfmwMT')

---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[6], line 1
----> 1 authorizer.add_user('user', 'password', '/home/user', perm='elradfmwMT')

File ~/.pyenv/versions/3.10.13/envs/pixl-3.10.13/lib/python3.10/site-packages/pyftpdlib/authorizers.py:106, in DummyAuthorizer.add_user(self, username, password, homedir, perm, msg_login, msg_quit)
    104     homedir = homedir.decode('utf8')
    105 if not os.path.isdir(homedir):
--> 106     raise ValueError('no such directory: %r' % homedir)
    107 homedir = os.path.realpath(homedir)
    108 self._check_permissions(username, perm)

ValueError: no such directory: '/home/user'

Any idea what maybe causing this? Thanks!

The error message says it all no such directory: '/home/user'. :D
That's just an example from the doc. You should use a path of a directory which exists.

Thanks for the reply @giampaolo! Might be worth adding a note to the docs then to specify that :) and maybe some example code of how to create that directory?
At the moment I can only get it to work if I use "." as home directory. I'm probably missing something obvious, but it would be great if the documentation could help out a bit more. Thanks!


Update country list in shipping address if shipping method overrides allowed countries

@schmengler I cannot decipher this requirement now. Can you please elaborate it if you remember?

You can specify different allowed countries for each shipping method. So if the user selects a different shipping method, the country list should be updated accordingly

@schmengler  I think when we set the shipping address or change the shipping address, the shipping method information will be unset from the quote and methods will be refreshed again according to the shipping address. So if a shipping method is not available in the shipping country opted, then that shipping method won't be shown in the frontend.
So I assume this is already covered in that way?

No, we need it the other way around too.

when we set the shipping address or change the shipping address, [...] methods will be refreshed again according to the shipping address.

when we change the shipping method, the dropdown of available countries in the shipping address must be changed
Example: Shipping method A is selected and allows shipping only to Germany. I change it to shipping method B which also allows shipping to India. But I cannot change the shipping address because the country dropdown did not update.


nlopes/slack#232 - added missing Output method

Fixes #232

oops. my bad, should have verified post merge. technically speaking output is the only method we need the rest can all be implemented around that IIRC. we may want to look into that later just to reduce the number of methods necessary to satify the interface.

@james-lawrence no worries! and thanks @mauleyzaola for the super-fast fix! :)

Thank you guys and happy new year 🎉


Multiple Windows related fixes

These patches fix generation of unique ids and colored log on windows

LGTM
only 1 very small stylistic criticism: UniqueIDGenerator.generate_unique_id and the module level generate_unique_id having the same name can make the definition of generate_unique_id a bit confisong to read, might be better to have seperate names or just use __call__ on UniqueIDGenerator

i'm running into a error testing this:
MainThread                     ERROR    21:05:45.982839 lona.client_pre_compiler exception raised while pre compiling js client
  Traceback (most recent call last):
    File "C:\Python39\lib\site-packages\lona\client_pre_compiler.py", line 56, in compile
      template = self.jinja2_env.get_template('lona.js')
    File "C:\Python39\lib\site-packages\jinja2\environment.py", line 997, in get_template
      return self._load_template(name, globals)
    File "C:\Python39\lib\site-packages\jinja2\environment.py", line 958, in _load_template
      template = self.loader.load(self, name, self.make_globals(globals))
    File "C:\Python39\lib\site-packages\jinja2\loaders.py", line 125, in load
      source, filename, uptodate = self.get_source(environment, name)
    File "C:\Python39\lib\site-packages\jinja2\loaders.py", line 214, in get_source
      raise TemplateNotFound(template)
  jinja2.exceptions.TemplateNotFound: lona.js
Traceback (most recent call last):
  File "d:\lauri\Documents\python-projects\webpages\lona-test\main.py", line 30, in <module>
    app.run(port=8080)
  File "C:\Python39\lib\site-packages\lona\app.py", line 189, in run
    server = LonaServer(
  File "C:\Python39\lib\site-packages\lona\server.py", line 167, in __init__
    self.static_file_loader = StaticFileLoader(self)
  File "C:\Python39\lib\site-packages\lona\static_file_loader.py", line 27, in __init__
    self.discover()
  File "C:\Python39\lib\site-packages\lona\static_file_loader.py", line 166, in discover
    self.style_tags_html = self.server.templating_engine.render_template(
  File "C:\Python39\lib\site-packages\lona\templating.py", line 126, in render_template
    template = self.get_template(template_name)
  File "C:\Python39\lib\site-packages\lona\templating.py", line 96, in get_template
    template = self.jinja2_env.get_template(template_name)
  File "C:\Python39\lib\site-packages\jinja2\environment.py", line 997, in get_template
    return self._load_template(name, globals)
  File "C:\Python39\lib\site-packages\jinja2\environment.py", line 958, in _load_template
    template = self.loader.load(self, name, self.make_globals(globals))
  File "C:\Python39\lib\site-packages\jinja2\loaders.py", line 125, in load
    source, filename, uptodate = self.get_source(environment, name)
  File "C:\Python39\lib\site-packages\jinja2\loaders.py", line 214, in get_source
    raise TemplateNotFound(template)
jinja2.exceptions.TemplateNotFound: lona/style_tags.html

this might be related due to the fact i installed using pip install git+http...
how should i install this?

@laundmo: I agree. I changed UniqueIDGenerator.generate_unique_id to UniqueIDGenerator.__call__

@laundmo: This looks like a pip problem to me. Which version are you using? I use 21.2.4. This can't be caused by this pr. I didn't touch the MANIFEST.in or other packaging parts.

i believe i have found the cause of the error
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
requests 2.25.1 requires idna<3,>=2.5, but you have idna 3.2 which is incompatible.
nbconvert 6.0.7 requires mistune<2,>=0.8.1, but you have mistune 2.0.0a4 which is incompatible.
datamodel-code-generator 0.7.2 requires jinja2<3.0,>=2.10.1, but you have jinja2 3.0.1 which is incompatible.

@laundmo:  Strange. If dependencies can not be met pip installs python files but not non-python files? wtf!?

I checked this patch on:

Firefox 91.1.01
Chrome 92.0.4515.159
Windows 10 Enterprise LTSC 64 bit

My results:

Simple script is working as expected
pytest passing test  and tox -e lint okay

Anything else I could check?

@themutt: \o/ great! :) No i think that's all. I'll wait for @laundmo's feedback, then i will merge and release 1.3.1


MemoryLeak??

Environment details

OS type and version: macOSCatalina 10.15.7
Python version: python --version Python 3.8.1
pip version: pip --version pip 20.2.3 from /usr/local/lib/python3.8/site-packages/pip (python 3.8)
google-api-python-client version: pip show google-api-python-client 1.12.3

There seem to be memory leaks again with this client. Very frustrating. Here are some stack trace examples:
Stack trace
Traceback (most recent call last):
  File "manage.py", line 21, in <module>
    main()
  File "manage.py", line 17, in main
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python3.8/site-packages/django/core/management/__init__.py", line 401, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python3.8/site-packages/django/core/management/__init__.py", line 395, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python3.8/site-packages/django/core/management/base.py", line 328, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/local/lib/python3.8/site-packages/django/core/management/base.py", line 369, in execute
    output = self.handle(*args, **options)
  File "/usr/src/app/api/surveys/management/commands/survey_results_report.py", line 133, in handle
    result = service.spreadsheets().values().update(
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/discovery.py", line 1402, in methodResource
    return Resource(
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/discovery.py", line 1289, in __init__
    self._set_service_methods()
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/discovery.py", line 1338, in _set_service_methods
    self._add_basic_methods(self._resourceDesc, self._rootDesc, self._schema)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/discovery.py", line 1371, in _add_basic_methods
    fixedMethodName, method = createMethod(
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/discovery.py", line 1183, in createMethod
    docs.append(schema.prettyPrintSchema(methodDesc["response"]))
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 158, in prettyPrintSchema
    return self._prettyPrintSchema(schema, dent=1)[:-2]
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/_helpers.py", line 134, in positional_wrapper
    return wrapped(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 145, in _prettyPrintSchema
    return _SchemaToStruct(schema, seen, dent=dent).to_str(self._prettyPrintByName)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 315, in to_str
    return self._to_str_impl(self.schema)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 267, in _to_str_impl
    s = self.from_cache(schemaName, seen=self.seen)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/_helpers.py", line 134, in positional_wrapper
    return wrapped(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 108, in _prettyPrintByName
    self.pretty[name] = _SchemaToStruct(
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 315, in to_str
    return self._to_str_impl(self.schema)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 258, in _to_str_impl
    self._to_str_impl(pschema)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 299, in _to_str_impl
    self.string = "".join(self.value)
MemoryError

Traceback (most recent call last):
  File "manage.py", line 21, in <module>
    main()
  File "manage.py", line 17, in main
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python3.8/site-packages/django/core/management/__init__.py", line 401, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python3.8/site-packages/django/core/management/__init__.py", line 395, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python3.8/site-packages/django/core/management/base.py", line 328, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/local/lib/python3.8/site-packages/django/core/management/base.py", line 369, in execute
    output = self.handle(*args, **options)
  File "/usr/src/app/api/surveys/management/commands/survey_results_report.py", line 133, in handle
    result = service.spreadsheets().values().update(
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/discovery.py", line 1402, in methodResource
    return Resource(
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/discovery.py", line 1289, in __init__
    self._set_service_methods()
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/discovery.py", line 1338, in _set_service_methods
    self._add_basic_methods(self._resourceDesc, self._rootDesc, self._schema)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/discovery.py", line 1371, in _add_basic_methods
    fixedMethodName, method = createMethod(
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/discovery.py", line 1183, in createMethod
    docs.append(schema.prettyPrintSchema(methodDesc["response"]))
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 158, in prettyPrintSchema
    return self._prettyPrintSchema(schema, dent=1)[:-2]
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/_helpers.py", line 134, in positional_wrapper
    return wrapped(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 145, in _prettyPrintSchema
    return _SchemaToStruct(schema, seen, dent=dent).to_str(self._prettyPrintByName)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 315, in to_str
    return self._to_str_impl(self.schema)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 267, in _to_str_impl
    s = self.from_cache(schemaName, seen=self.seen)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/_helpers.py", line 134, in positional_wrapper
    return wrapped(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 108, in _prettyPrintByName
    self.pretty[name] = _SchemaToStruct(
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 315, in to_str
    return self._to_str_impl(self.schema)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 258, in _to_str_impl
    self._to_str_impl(pschema)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 292, in _to_str_impl
    self._to_str_impl(schema["items"])
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 267, in _to_str_impl
    s = self.from_cache(schemaName, seen=self.seen)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/_helpers.py", line 134, in positional_wrapper
    return wrapped(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 108, in _prettyPrintByName
    self.pretty[name] = _SchemaToStruct(
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 315, in to_str
    return self._to_str_impl(self.schema)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 258, in _to_str_impl
    self._to_str_impl(pschema)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 292, in _to_str_impl
    self._to_str_impl(schema["items"])
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 267, in _to_str_impl
    s = self.from_cache(schemaName, seen=self.seen)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/_helpers.py", line 134, in positional_wrapper
    return wrapped(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 108, in _prettyPrintByName
    self.pretty[name] = _SchemaToStruct(
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 315, in to_str
    return self._to_str_impl(self.schema)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 258, in _to_str_impl
    self._to_str_impl(pschema)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 267, in _to_str_impl
    s = self.from_cache(schemaName, seen=self.seen)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/_helpers.py", line 134, in positional_wrapper
    return wrapped(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 108, in _prettyPrintByName
    self.pretty[name] = _SchemaToStruct(
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 315, in to_str
    return self._to_str_impl(self.schema)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 258, in _to_str_impl
    self._to_str_impl(pschema)
  File "/usr/local/lib/python3.8/site-packages/googleapiclient/schema.py", line 299, in _to_str_impl
    self.string = "".join(self.value)
MemoryError

Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/django/db/utils.py", line 97, in inner
    return func(*args, **kwargs)
psycopg2.DatabaseError: out of memory for query result

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

Traceback (most recent call last):
  File "manage.py", line 21, in <module>
    main()
  File "manage.py", line 17, in main
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python3.8/site-packages/django/core/management/__init__.py", line 401, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python3.8/site-packages/django/core/management/__init__.py", line 395, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python3.8/site-packages/django/core/management/base.py", line 328, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/local/lib/python3.8/site-packages/django/core/management/base.py", line 369, in execute
    output = self.handle(*args, **options)
  File "/usr/src/app/api/surveys/management/commands/survey_results_report.py", line 118, in handle
    for survey_result in survey_results.iterator():
  File "/usr/local/lib/python3.8/site-packages/django/db/models/query.py", line 346, in _iterator
    yield from self._iterable_class(self, chunked_fetch=use_chunked_fetch, chunk_size=chunk_size)
  File "/usr/local/lib/python3.8/site-packages/django/db/models/query.py", line 74, in __iter__
    for row in compiler.results_iter(results):
  File "/usr/local/lib/python3.8/site-packages/django/db/models/sql/compiler.py", line 1584, in cursor_iter
    for rows in iter((lambda: cursor.fetchmany(itersize)), sentinel):
  File "/usr/local/lib/python3.8/site-packages/django/db/models/sql/compiler.py", line 1584, in <lambda>
    for rows in iter((lambda: cursor.fetchmany(itersize)), sentinel):
  File "/usr/local/lib/python3.8/site-packages/django/db/utils.py", line 97, in inner
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/django/db/utils.py", line 90, in __exit__
    raise dj_exc_value.with_traceback(traceback) from exc_value
  File "/usr/local/lib/python3.8/site-packages/django/db/utils.py", line 97, in inner
    return func(*args, **kwargs)
django.db.utils.DatabaseError: out of memory for query result

This is absolutely unacceptable and possibly the first time I have seen this number of Memory errors of this nature from a python package.

Hi @michealjroberts,
https://github.com/googleapis/google-api-python-client/issues/535 reports the same issue.
The out of memory seems to come from repeated calls to discovery.build(...).  That also has a small benefit for credentials, which will be re-used until they expire.
Copied from https://github.com/googleapis/google-api-python-client/issues/535
service = discovery.build("sheets", "v4")
sheets = service.spreadsheets()
for i in range(0, 50):
    get_responses(sheets)
    sleep(2) # no leak


[Unity][CUTLASS]  Fix circular import bug in relax cutlass backend

There exists circular import bug in tvm/python/tvm/relax/backend/contrib/cutlass.py
Code to reproduce:
from tvm.contrib.cutlass import (
    has_cutlass,
    num_cutlass_partitions,
    finalize_modules
)

Error:
File "/home/work/tmp/test.py", line 1, in <module>
    from tvm.contrib.cutlass import (
  File "/home/tvm/python/tvm/contrib/cutlass/__init__.py", line 18, in <module>
    from .build import has_cutlass, num_cutlass_partitions, finalize_modules, finalize_modules_vm
  File "/home/tvm/python/tvm/contrib/cutlass/build.py", line 28, in <module>
    from tvm import relax, relay, runtime
  File "/home/tvm/python/tvm/relax/__init__.py", line 90, in <module>
    from . import backend
  File "/home/tvm/python/tvm/relax/backend/__init__.py", line 19, in <module>
    from . import contrib
  File "/home/tvm/python/tvm/relax/backend/contrib/__init__.py", line 20, in <module>
    from .cutlass import partition_for_cutlass
  File "/home/tvm/python/tvm/relax/backend/contrib/cutlass.py", line 24, in <module>
    from tvm.contrib.cutlass.build import is_shape_valid_for_cutlass_matmul
ImportError: cannot import name 'is_shape_valid_for_cutlass_matmul' from partially initialized module 'tvm.contrib.cutlass.build' (most likely due to a circular import) (/home/tvm/python/tvm/contrib/cutlass/build.py)

I think it is better to remove from .cutlass import partition_for_cutlass from backend/contrib/__init__.py since doing import relax shouldn't cause contrib/cutlass/build.py to depend on contrib/cutlass.py. cc @yelite

Import removed from __init__


Update owners file for Azure

Update owners file for Azure

/assign @8bitmp3

Hi @8bitmp3, @sudivate, and @eedorenko are in the transitioning phase and would like to onboard new owners to the Azure area. Please guide on how can we proceed?
cc-@aronchick

/lgtm
Thanks @sudivate !
cc @RFMVasconcelos (another 👍 OWNERS update)
/assign @animeshsingh @Bobgy @joeliedtke @rmgogogo
PTAL and approve if you can, thank you!

The following users are mentioned in OWNERS file(s) but are untrusted for the following reasons. One way to make the user trusted is to add them as members of the kubeflow org. You can then trigger verification by writing /verify-owners in a comment.

yilun-msft

User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
layouts/shortcodes/azure/OWNERS

shbijlan

User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
layouts/shortcodes/azure/OWNERS

nilspohlmann

User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
layouts/shortcodes/azure/OWNERS

vsetty-msft

User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
layouts/shortcodes/azure/OWNERS

Can you please add them to kubeflow org first?

@Bobgy - Created PR https://github.com/kubeflow/internal-acls/pull/381 to add above users to the Kubeflow org

/retest

/verify-owners

/verify-owners

/verify-owners

The github sync was successful, we need to wait for @yilun-msft @shbijlan @nilspohlmann @vsetty-msft to accept kubeflow invitation.

/verify-owners

Hello @vsetty-msft Have you been added as a Kubeflow member in a separate PR? If so, can you please accept this invitation.  Thanks!! cc @sudivate @yilun-msft @shbijlan @NilsPohlmann @Bobgy

/verify-owners

/verify-owners
cc @sudivate

Please rebase master to pass conflicts

@sudivate, Please update the PR to resolve the merge conflict.

Let's close PR if no updates recently, it has been 4 months.


[ 1° ] - Release/v1.4.0 automate testdata registration: prepare and refactor shell scripts to automate testdata registration/deletion

This PR contains the following changes:

Prepare the test data upload script to upload test data for all passes (batterypass/dpp/smc/transmissionpass) to the provider setup (provider backend/edc/aas registry)
Merge existing scripts of different passes into one single script
Prepare deletion script to remove all edc assets and digital twins from the setup
Documented a readme on how to execute the scripts

What we want to achieve with this PR?
The scripts are written and refactored to speed up the registration process so that the data provider is ready for testing purposes
What is new?
Added

Added script to automate the uploading of various passes types (e.g., batterypass, transmissionpass)
Added script to delete data from the data provider

Deleted

Remove the legacy style to  register/delete the testdata

The ip checked are executed and nothing was found.

Most of the files are renamed/deleted so there are no huge changes in files.


Documentation: Add pixel values for the rem breakpoints and column sizes

Suggestion... For the grid and media queries, it might be helpful to list the equivalent pixel values for people designing or saving out graphics. Looking through the documentation (http://v4-alpha.getbootstrap.com/layout/overview/#responsive-breakpoints), it wasn't clear what the rem media query values and container widths equate to in pixels.

Somewhat related to: #17070

Submitted PR to add those :smiley:

Because rem and em are related to the base font size, I think it could be worth adding a footnote (or some other unobtrusive indicator) that the pixel values are based on the standard 1em = 16px formula. If someone changes the base font size either globally or for a particular breakpoint, the relative size of the em and rem units will be modified as well.
That said, I think this sort of touch is helpful because most people probably keep the default value for the base font size. (Of course, this is just an assumption since I don't have any real stats on the sorts of changes developers make to Bootstrap for their various sites.)

Fixed by #17146


Actuator prometheus endpoint not get registered

When I build the app with the "io.micrometer:micrometer-registry-prometheus" dependency, this is supposed to enable prometheus scrapping endpoint, (when I run the jar in JVM it is).
But when I run the binary generated by graalvm native-image feature the prometheus endpoint is not registered and not available.

@aclement Could you please have a look to this one when you have some time?

@aclement Could you please have a look to this one when you have some time?

Let's move that one to 0.9.1.

Ok we can probably add that hint in 0.9.2 then, thanks for the exploratory work.

I also added heath endpoint and info endpoint. Both are not discovered by default.
I had to add
  {
    "name":"org.springframework.boot.actuate.health.HealthEndpoint",
    "allDeclaredFields":true,
    "allDeclaredMethods":true
  },
  {
    "name":"org.springframework.boot.actuate.info.InfoEndpoint",
    "allDeclaredFields":true,
    "allDeclaredMethods":true
  }

Seems that metrics endpoint works only with this configuration
{
  "name":"org.springframework.boot.actuate.metrics.MetricsEndpoint",
  "allDeclaredFields":true,
  "allDeclaredMethods":true
},
{
  "name":"org.springframework.boot.actuate.metrics.MetricsEndpoint$MetricResponse",
  "allDeclaredFields":true,
  "allDeclaredMethods":true
},
{
  "name":"org.springframework.boot.actuate.metrics.MetricsEndpoint$Sample",
  "allDeclaredFields":true,
  "allDeclaredMethods":true
},
{
  "name":"org.springframework.boot.actuate.metrics.MetricsEndpoint$AvailableTag",
  "allDeclaredFields":true,
  "allDeclaredMethods":true
},
{
  "name":"org.springframework.boot.actuate.metrics.MetricsEndpoint$ListNamesResponse",
  "allDeclaredFields":true,
  "allDeclaredMethods":true
}

@GregoireW:
Thanks for trimming down that config. I've added a hint for the PrometheusScrapeEndpoint and upgraded the actuator-webmvc sample to verify it is reachable.
I've also added a hint for the info endpoint and upgraded actuator-webmvc sample.
I did not add a hint for the health endpoint, that's already covered, not quite sure why you had to add it? Let's deal with that in another bug if still a problem. (The actuator-webmvc hits the health endpoint in its testing)
@prettymama
I've also added those to the metrics endpoint hints we already had (and expanded actuator-webmvc to test it).
Now with all these changes I am verifying the endpoints are reachable, not verifying all potential data is available for the endpoints. Please raise new issues for further data within these that looks to be missing.

@aclement Out of curiosity, what did you add (and where) to have those hint ? there is no reflect-config.json in spring aot but in the end there is a generated file. (too lazy to check that today)
anyway I checked, Not sure why I put the health endpoint. It was working from the start.
I check quickly and 0.9.2-SNAPSHOT fixes the issue. Thank you


xml file still contains 'people' type even the 'person_types' was set to ['person']

For Caltech converting, I have set person_types to ['person'] in main:
person_types = ["person"]
parse_anno_file(vbb_inputdir, seq_inputdir, vbb_outputdir, seq_outputdir, person_types=person_types)
However, the xml file is:

@SharonZhu emm, I cannot replicate the error you met. Maybe redirect output to a new place to eliminate history residues if you run this script previously.


Issue/4500 improve module install logging

Description
Add debug logging on module install
closes #4500
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Type annotations are present
[x] Code is clear and sufficiently documented
[x] No (preventable) type errors (check using make mypy or make mypy-diff)
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
- [ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

When an inmanta compile or an inmanta export is executed. We should also print a full list of the V1 and V2 modules that are installed.

Processing this pull request

Merged into branches master in 461091db5e30abfa7e3a3ace79a8307baf07a38b

Processing #4667.

Processing #4667.


connect to halcon

if i want connect the camera(O3D3xx) to halcon software in the computer ,what should i do.

Hey @PHILCHEN666 ,
there is a halcon driver available for the O3D3xx: https://www.mvtec.com/products/interfaces/documentation/view/65-3d-camera-12-mvtecdoc-o3d3xx/ In case of any question related to this driver please get in touch with our support team


How to autoselect the first match?

Is there an easy way to auto select the first match without having to navigate through the completion list?
example of what I want to achieve:
typed "as" and the completion pops up with the first match selected "async"

As it is now, I type "as" and have to manually select the first match in the completion menu since it's not selected by default:

I saw the preselect option in config but dunno what it really does...
Can anyone help me?

You can use preselect = 'always'

Lovely! tysm!


Input sanitation

Purpose
This PR adds input sanitation functions for parsing arbitrary string input form and url values that allows input to be validated against a set of whitelisted values.
Implementation

An interface AllowSet which implements a single method IsSafe(string) bool was added. Objects that implement this interface are used to validate input.
AllowSetBytes and AllowSetStrings implement AllowSet. The former is set of allowed un-ordered characters while the latter is a set of ordered characters (strings).
The following method can be called in a request handler to safely retrieve string field values: GetStringFieldSafe.

Example Usage
handlerAllowSet = NewAllowSetChars(`abcdefghijklmnopqrstuvwxyz`, 10)

func handler(w http.ResponseWriter, r *http.Request, params map[string]string) (interface{}, error) {
    fieldValue, err := scroll.GetStringFieldSafe(r, "foo", handlerAllowSet)
    if err != nil {
        return nil, err
    }

    return scroll.Response{"message": fmt.Sprintf("Great input: %q", fieldValue)}, nil
}

@russjones What do you think about making this the default behavior? I.e. we can get rid of *Safe functions (sorry I know I suggested to add them in the first place :smile:) and add this AllowSet functionality to regular Get*Field functions. API will be clean and security will be enforced by default. We can pick some sensible defaults for AllowSets, like the ones you chose for steve.

lgtm

lgtm


Practitioners with Uppercase characters in their email won't map to Users with the same

If you create a Practitioner with an email that contains uppercase letters like
{
    "resourceType": "Practitioner",
    "meta": {
        "accounts": [
            {
                "reference": "Organization/A"
            }
        ]
    },
    "name": [
        {
            "given": [
                "Practitioner"
            ],
            "family": "A.1"
        }
    ],
    "telecom": [
        {
            "system": "email",
            "use": "work",
            "value"<EMAIL_ADDRESS>        }
    ]
}

Then create a user also using uppercase letters in the email
{
    "resourceType": "Practitioner",
    "firstName": "Practitioner",
    "lastName": "A.1",
    "email"<EMAIL_ADDRESS>    "sendEmail": "false",
    "password": "foobar"
}

These accounts won't be matched together.
I think the logic to create a user converts the email to lowercase whereas the logic to create a Practitioner does not.

Thanks for filing this @ianplunkett
The root cause is very subtle here.  My interpretation of this passage is that the Practitioner-email search parameter should be case-insensitive because the underlying element is type string

Based on this excerpt highlighted above by @codyebberson from the FHIR search spec:

If the underlying data type is string then the search is not case sensitive.

And the following paragraph:

Note: There are many challenging issues around case sensitivity and token searches. Some code systems are case sensitive (e.g. UCUM) while others are known not to be. For many code systems, it's ambiguous. Other kinds of values are also ambiguous. When in doubt, servers SHOULD treat tokens in a case-insensitive manner, on the grounds that including undesired data has less safety implications than excluding desired behavior. Clients SHOULD always use the correct case when possible, and allow for the server to perform case-insensitive matching.

At a minimum, the best path forward is to modify TokenTable.indexResource to lowercase value filed for tokens of type PropertyType.ContactPoint and of course, correspondingly, lowercase the value being queried for at search-time. This approach requires reindexing all resources.
There's an argument that could be made that we should expand the case-insensitivity beyond just ContactPoint tokens, but based on this other excerpt from the spec we should probably err on the side of being conservative and leaving other token types as case-sensitive:

Match is case sensitive unless the underlying semantics for the context indicate that the token should be interpreted case-insensitively (see, e.g. CodeSystem.caseSensitive). Note that matches on _id are always case sensitive.


AWS Lambda account ISN'T free !

I'm trying to create an AWS Lambda account but it isn't free. I keep being asked for my credit card number or some kind of paiement. What shall I do ?

That's because only one tier is free, the details are needed for if you exceed that. eg go over 1,000,000 requests or 3 million seconds of computer time.

You have to use a CC to open an account. It will not be charged anything unless you exceed a free tier or consume a service that doesn't have a free tier. You pay for what you use. For personal use, it is not likely that you would ever exceed the Lambda free tier, so you will not get charged.


Sonarr importing multiple episodes simultaneously

I use this script in conjunction with acd_cli on a VPS with very limited storage, so my download process is as follows:

Sonarr sends release to SabNZBD
SabNZBD downloads release, calls SABPostProcess.py
Video file(s) are converted to MP4, then Sonarr is called to scan folder
SabNZBD sees post-processing as finished and download "complete" as soon as Sonarr is called to scan

The problem is, since SabNZBD is seeing the download as "complete" while Sonarr is still moving/renaming the file(s), it moves onto the next download. Even if I have SabNZBD pause downloads while post-processing, it still downloads faster than Sonarr can move the files. I have another script called by Sonarr which clears out the local target directory to where episodes are moved after uploading to ACD, but the problem is when it is called for more than one episode at once, it will still be in the process of moving the second episode, but it will delete the folder structure of the directory after the first episode is moved.
Basically, for some reason Sonarr will allow the SabPostProcess.py script to alert it multiple times to scan a folder, and it won't queue those requests. It'll start moving multiple files if called multiple times. On the other hand, when using the same scripts with CouchPotato, SabNZBD will wait until that file is completely moved and renamed by CP before marking the download as "complete" and moving onto the next one. It won't do that for Sonarr.
Is there a way to "slow" Sonarr down and limit it to one file at a time?

That's more of a question for the Sonarr people than me, I'm honestly not sure. Either way though slowing it down would not be ideal. Sonarr should only be getting told about subdirectories anyway and should be capable for cleaning up the source files / download directories as it finishes accepting the file. You shouldn't need a custom script to do this and that may be why you're having issues.
If you use Sonarr's built in clean up features it will only delete the files associated with the download its finished, it wont just blanket delete your entire SAB downloads folder.


Auto-resolve DID Documents for native chain addresses

According to the DID specifications,  a DID must be resolvable [cannot find the link ATM]
In the cosmos-cash implementation a user can generate an arbitrary DID where then
she can attach it's own controllers/verification method controllers to be able to control the
did document.
But since there is no DID document associated to the user wallet address, it results
in an unresolvable DID for the user chain address
Proposal: use the did:key method to auto resolve any address in the cosmos-cash chain
(cfr. https://w3c-ccg.github.io/did-method-key/)
Expected behavior
resolve a did with the form did:key:cosmosXYZ to a did document as described in the did:key method specs

Eventually we can submit our driver to the identity foundation  to resolve our method with the universal resolver

this has bee implemented in #184  and researched in #169 the change is that the method is did:cosmos:key:... for a couple of reasons:

in a universal resolver context there can be only a driver for a method
the encoding of the key requires some manipulation to be usable in a cosmos chain


[feature request] dialogue with the model

It would be awesome to be able to conversate with the model based on the input instead of it being a fire-once approach.
I'm thinking something like having a "Conversate" prompt that opens a window in which each question-answer can be viewed.
I would love this to adjust the question asked or specify even further details for the prompts.

Nevermind, didn't spot another issue that asked for this #28. Closing.


Migrate is always generating migrations with db.Time (MySQL)

Bug description
After run first migrate, always i run npx prisma migrate dev it's creating even though it hasn't changed anything.
I believe it is related to the db.Time type of MySQL
Always generate this:
-- AlterTable
ALTER TABLE `Services` MODIFY `course_load` TIME;

How to reproduce
just start a new project
yarn add -D prisma typescript ts-node @types/node

then use schema.prisma below
and run twice:
npx prisma migrate dev

Expected behavior
No response
Prisma information
datasource db {
  provider = "mysql"
  url      = env("DATABASE_URL")
}

generator client {
  provider = "prisma-client-js"
}

model Services {
  id          Int       @id @default(autoincrement())
  course_load DateTime? @db.Time
}

Environment & setup

OS: Windows
Database: MySQL
Node.js version: v14.16.1

Prisma Version
prisma                : 2.30.0
@prisma/client        : 2.30.0
Current platform      : windows
Query Engine (Binary) : query-engine 60b19f4a1de4fe95741da371b4c44a92f4d1adcb (at<EMAIL_ADDRESS>Migration Engine      : migration-engine-cli 60b19f4a1de4fe95741da371b4c44a92f4d1adcb (at<EMAIL_ADDRESS>Introspection Engine  : introspection-core 60b19f4a1de4fe95741da371b4c44a92f4d1adcb (at<EMAIL_ADDRESS>Format Binary         : prisma-fmt 60b19f4a1de4fe95741da371b4c44a92f4d1adcb (at<EMAIL_ADDRESS>Default Engines Hash  : 60b19f4a1de4fe95741da371b4c44a92f4d1adcb
Studio                : 0.422.0

I can reproduce with 2.31.0-dev.49. Thanks for reporting.


Rheostat Functional Component

developers, you have done a fantastic job making this component work.. kudos !!!
do we have the Rheostat Component (Slider) as a functional component???

Why do you care what kind of component it is?


Installation failing

Expected Behavior
go get github.com/runatal/gotenberg-go-client/v8@latest
works
Current Behavior
go get github.com/runatal/gotenberg-go-client/v8@latest
go: github.com/runatal/gotenberg-go-client/v8@latest (v8.6.3) requires<EMAIL_ADDRESS>parsing go.mod:
	module declares its path as: github.com/dcaraxes/gotenberg-go-client/v8
	        but was required as: github.com/runatal/gotenberg-go-client/v8

Delete the go.sum file and this module from go.mod, then run go mod tidy. go clean -modcache can also help.

It is working now - thanks!


matlab-actions/run-command fails with error no such file error

To use the cache feature, I updated my workflow to use setup-matlab@v2 from setup-matlab@v1
However, with v2 the Simulink Test workflow fails since simstruct.h cannot be found.

Hi @AkashJohnSubash ,
setup-matlab@v2 now requires specifying the products that the builds requires beyond MATLAB.
You may need to update your CI .yml file to specify the products required. The products parameter takes a space-separated list of products. Spaces in the product names should be replaced with underscores (see the README  for a full explanation and examples). Here is an example that includes Simulink and Simulink Test:
    - name: Setup MATLAB
      uses: matlab-actions/setup-matlab@v2
      with:
        products: Simulink Simulink_Test

Hi @davidbuzinski ,
thanks for clarifying that. I did have a look at the Readme earlier but wasn't aware that the Simulink headers were no longer downloaded implicitly. Thanks, this did the trick and fixed my problem


Data service does not have the option to creating a query

Description:

Integration studio offers a way to create queries when configure data services (refer: https://mi.docs.wso2.com/en/4.3.0/develop/creating-artifacts/data-services/creating-data-services/#creating-a-query).
It seems the above option is not available in the mi-vscode extension.
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

Hi @gabilang,
The option to create a query is given in the diagram view of the resource.

https://mi.docs.wso2.com/en/4.3.0/develop/creating-artifacts/data-services/creating-data-services/#generate-data-service-from-a-datasource Is this feature still there in the vs-code extension? Can we create a data service directly from an existing datasource?

Hi @gabilang,
Not at the moment, but we need to work on that. Could you please create an issue for this so that we can keep track and deliver the feature at the earliest.

Created an issue to track the feature for creating data services from the existing datasources. Hence closing this.


Instruction issue in 'Get Geolocation Data' waypoint

Waypoint: Get Geolocation Data
I see that we can't read properly numbers in both latitude and longitude, here's screenshot what it looked like below.
See right side in mobile view.

I think, we can make it more readable and better visible for other new campers.
My Code:
  if (navigator.geolocation) {

  navigator.geolocation.getCurrentPosition(function(position){

  $("#data").html("Latitude: " + position.coords.latitude + "<br/>" + "Longitude: " + position.coords.longitude);
    });
  }

Thanks for the report, but this issue was resolved by PR #3986 and is pending deployment to the website.
Closing as resolved, hopefully this update will hit the website soon.
Thanks and happy coding!

Oh Thanks for quick response! :smile:


HBASE-23989 Consider 2.10.0 in our support matrix

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1290/1/console in case of problems.

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1290/1/console in case of problems.

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
 U: .

 U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1290/1/console

versions
git=2.17.1 maven=2018-06-17T18:33:14Z)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 48s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
shelldocs
0m  1s
Shelldocs was not available.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
5m 37s
master passed

+0 :ok:
refguide
4m 50s
branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m 16s
the patch passed

+1 :green_heart:
shellcheck
0m  2s
There were no new shellcheck issues.

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+0 :ok:
refguide
4m 50s
patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

_ Other Tests _

+1 :green_heart:
asflicense
0m 19s
The patch does not generate ASF License warnings.

23m 15s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1290/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1290

Optional Tests
dupname asflicense shellcheck shelldocs refguide

uname
Linux 653dd9be2614 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / ba3f02629a

refguide
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1290/1/artifact/yetus-general-check/output/branch-site/book.html

refguide
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1290/1/artifact/yetus-general-check/output/patch-site/book.html

Max. process+thread count
76 (vs. ulimit of 10000)

modules
C: . U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1290/1/console

versions
git=2.17.1 maven=2018-06-17T18:33:14Z) shellcheck=0.4.6

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1290/2/console in case of problems.

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1290/2/console in case of problems.

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
 U: .

 U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1290/2/console

versions
git=2.17.1 maven=2018-06-17T18:33:14Z)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 34s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
shelldocs
0m  0s
Shelldocs was not available.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
5m 36s
master passed

+0 :ok:
refguide
4m 54s
branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m  2s
the patch passed

+1 :green_heart:
shellcheck
0m  2s
There were no new shellcheck issues.

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+0 :ok:
refguide
4m 48s
patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

_ Other Tests _

+1 :green_heart:
asflicense
0m 19s
The patch does not generate ASF License warnings.

22m 45s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1290/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1290

Optional Tests
dupname asflicense shellcheck shelldocs refguide

uname
Linux dba0a28fbdfc 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / ba3f02629a

refguide
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1290/2/artifact/yetus-general-check/output/branch-site/book.html

refguide
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1290/2/artifact/yetus-general-check/output/patch-site/book.html

Max. process+thread count
76 (vs. ulimit of 10000)

modules
C: . U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1290/2/console

versions
git=2.17.1 maven=2018-06-17T18:33:14Z) shellcheck=0.4.6

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Ping @ndimiduk @busbey @saintstack


Paramount+ Global

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

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

Checklist

[X] I'm requesting a site-specific feature
[X] I've verified that I'm running yt-dlp version 2022.09.01 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've searched 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
Australia/Europe
Example URLs
https://www.paramountplus.com/shows/calvin-and-kaisons-play-power/video/8JEcCgqfaKcvBf6AJMFGsvT0MYte_RUm/calvin-and-kaison-s-play-power-ultimate-hero/
https://www.paramountplus.com/shows/aerial-america/video/K_8IrDKmGoTe4Mnqfti0JQuvzb8MLRCM/aerial-america-california/
Provide a description that is worded well enough to be understood
I've noticed a lot of content on international versions of P+ currently aren't available on the US version (both shows and language options). For some reason, only the US version of P+ is supported (if you try and access a UK show with a US account, it won't work, but if you try and access a Australian show with a UK account, it works just fine), any time I try to rip stuff from a international version (i.e, Mexico or Australia), it gives me a 403 error. It would be neat if you could add support for the other countries in the future.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', 'https://www.paramountplus.com/shows/blues-clues/video/YZqYwu9sJNARDyM1huzbEwC5jzeW6sXB/blue-s-clues-snack-time/']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.09.01 [5d7c7d6] (win32_exe)
[debug] Python 3.8.10 (CPython 64bit) - Windows-10-10.0.22000-SP0
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 2022-04-03-git-1291568c98-full_build-www.gyan.dev (setts), ffprobe 4.4-essentials_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.06.15, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[debug] Loaded 1670 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.09.01, Current version: 2022.09.01
yt-dlp is up to date (2022.09.01)
[debug] [ParamountPlus] Extracting URL: https://www.paramountplus.com/shows/blues-clues/video/YZqYwu9sJNARDyM1huzbEwC5jzeW6sXB/blue-s-clues-snack-time/
[ParamountPlus] YZqYwu9sJNARDyM1huzbEwC5jzeW6sXB: Downloading JSON metadata
ERROR: [ParamountPlus] YZqYwu9sJNARDyM1huzbEwC5jzeW6sXB: Unable to download JSON metadata: HTTP Error 403: Forbidden (caused by <HTTPError 403: 'Forbidden'>); please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
  File "yt_dlp\extractor\common.py", line 670, in extract
  File "yt_dlp\extractor\cbs.py", line 70, in _real_extract
  File "yt_dlp\extractor\paramountplus.py", line 95, in _extract_video_info
  File "yt_dlp\extractor\common.py", line 1030, in download_content
  File "yt_dlp\extractor\common.py", line 994, in download_handle
  File "yt_dlp\extractor\common.py", line 864, in _download_webpage_handle
  File "yt_dlp\extractor\common.py", line 821, in _request_webpage

  File "yt_dlp\extractor\common.py", line 803, in _request_webpage
  File "yt_dlp\YoutubeDL.py", line 3656, in urlopen
  File "urllib\request.py", line 531, in open
  File "urllib\request.py", line 640, in http_response
  File "urllib\request.py", line 569, in error
  File "urllib\request.py", line 502, in _call_chain
  File "urllib\request.py", line 649, in http_error_default
urllib.error.HTTPError: HTTP Error 403: Forbidden

Duplicate of #4897


[Enhancement] feature request in ov

Hello Expert,
One feature request if possible and if you could enable space key press to scroll the file as all other existing pager doing... i know you have key binding to achieve the task but it is good to have by default second if you could add ctrl + > and ctrl + < key bind to jump directly on top and bottom of page this is also available by default in all the pager programs rest all looks good so far ...
I tried the keybinding for page_up/page_down with space and other key combination but result is not what i am expecting . Second i also noticed ov line number is wasting unnecessary terminal space need to be optimised. Below screenshot you can see this space can be reduced when line number enabled.

Thank you for writing the issue.
I noticed that the settings in ov-less.yml need to be modified for > and <.
I will commit the following later.
("CTRL +>" cannot be assigned due to terminal restrictions.)
diff --git a/ov-less.yaml b/ov-less.yaml
index 8bbe4a2..08f9f23 100644
--- a/ov-less.yaml
+++ b/ov-less.yaml
@@ -93,8 +93,10 @@ KeyBind:
         - "Up"
     top:
         - "Home"
+        - "<"
     bottom:
         - "End"
+        - ">"
     left:
         - "left"
     right:
@@ -127,9 +129,9 @@ KeyBind:
     remove_all_mark:
         - "ctrl+delete"
     next_mark:
-        - ">"
+        - "alt+>"
     previous_mark:
-        - "<"
+        - "alt+<"
     set_view_mode:
         - "p"
         - "P"

The line numbers in ov (displayed with ov -n) are just the width of the largest line number column, with no extra spaces.
(In large files, the first line will appear to have spaces).
Regarding line numbers, I would be grateful if you could raise another issue if necessary.

@noborus  Thanks for looking into it. it would be good if these basic task we handled without key map binding ov-less.xml config later if someone want to change the mapping they use the config to do. I hope you come up with some idea.  regarding line width i got your point but still if my text has not 5 digit number then no point to have extra width which take but space .. line number column should be surpassed and not take much terminal space

Many of the keys that are said to be standard follow less.
If the key binding of ov is the same as less, the key assignment of the function only in ov becomes difficult.
Therefore, it is not the default.
However, for users accustomed to less, I maintain ov-less.yml.

Many of the keys that are said to be standard follow less. If the key binding of ov is the same as less, the key assignment of the function only in ov becomes difficult. Therefore, it is not the default. However, for users accustomed to less, I maintain ov-less.yml.

Okay i then i tried key-map but i am not able to get result.. will check it later no issue

I'm done with mine, so I'll close it.


[WIP] Add "Install with Edgectl" page

Add a new "Install with Edgectl" page that provides an index of the various "help" pages that provide additional guidance when installing Ambassador Edge Stack with edgectl.

@ark3 here's a draft of the new "Install with Edgectl" page. It's nearly complete except that the various help pages aren't listed in alphabetical order, the reason for which is that some of the pages currently have the same title. Could you perhaps help disambiguate them?


Add Turkish i18n config file

Made this lil' file for a project, figured I should send it upstream.

@matcornic Do you know why the wecker build fails regularly? I don't have permissions to see the pipeline

@matalo33 can you give me your username on Wercker so I can give you rights ?
The problem is because Hugo does not compile, probably because Golang env on Wercker is not in 1.11
I use the latest version of Hugo to check if the theme is still working with latest version.
source "/pipeline/script-c9e8d70b-44f7-4995-8acb-e8833497f6c1/run.sh" < /dev/null
# github.com/gohugoio/hugo/hugolib
../../gohugoio/hugo/hugolib/hugo_sites.go:72:18: h.Deps.Log.LogCountForLevelsGreaterThanorEqualTo undefined (type *jwalterweatherman.Notepad has no field or method LogCountForLevelsGreaterThanorEqualTo)
../../gohugoio/hugo/hugolib/hugo_sites.go:304:7: h.Deps.Log.ResetLogCounters undefined (type *jwalterweatherman.Notepad has no field or method ResetLogCounters)
../../gohugoio/hugo/hugolib/hugo_sites_build.go:82:21: h.Deps.Log.LogCountForLevel undefined (type *jwalterweatherman.Notepad has no field or method LogCountForLevel)

@matalo33 can you give me your username on Wercker so I can give you rights ?

Thanks, just signed up, matalo33

@matalo33 can you give me your username on Wercker so I can give you rights ?

Thanks, just signed up, matalo33

Added you

You can sign up on Netlify too for deployment

Added you. Btw, you have a twitter or something (so we can avoid flooding issues with short messages) ?

Good idea, I was thinking of how to better have discussions, twitter works :). matt____taylor


Function request?

Function request, if acceptable, realizable!
*1-2 days old dns addresses queries on the background?
*Smart (petty) ping, frequently visited web sites?
Similar functionality is used by the cfosspped program, it speeds up the loading of web pages.
https://www.cfos.de/en/cfosspeed/cfosspeed.htm
Sorry but google translate!

Hi,
I don't think this is a good idea.
First, all modern web browsers already preload frequently visited websites.
Automatically replaying previous queries also makes it easy to fingerprint your devices. From a privacy standpoint, this is really bad.
Finally, it increases the load on servers for no major benefits.
So, well... probably not, sorry :(

OK, I understand.


change link for guest mode (connect page)

📝 Changed routes:

/connect

(view full size)

Commit 87ac625a7a54ffa80628aea86549824fdd919cbc (https://thirdweb-gahypoo1d.thirdweb-preview.com).


Bug: remove-space-around-fullwidth-characters not working well with lists.

Describe the Bug
the remove-space-around-fullwidth-characters rule breaks the list markdown syntax on some situations.
How to Reproduce
before:
- 【list item 1】
- 【list item 2】

after:
-【list item 1】
-【list item 2】

Expected Behavior
the space should not be removed between the list mark - and the full-width character 【 at the beginning of the list-item content.

Thanks for reporting this issue @ben-yip.
Could you explain why fullwidth characters are being used where they do not work with markdown syntax? I am asking because the rule is behaving as expected since it is removing the spaces and knows nothing about markdown syntax itself.
I personally would expect it to remove the spaces in the above scenarios since I am not sure there is an exception to the removal of spacing around fullwidth characters in the general case.
Any clarification on how how this is valid for full width characters would greatly help me.

Also, if we are going to ignore that the fullwidth characters need space removed around them, would you expect the same behavior in block quotes, and tables? I am asking because if this is the case, I may need to go ahead and take care of these scenarios as well if possible.
Thanks for helping make this plugin better!

Thanks for replying. @pjkaufman
Personally, I am used to put my thinkings or comments inside a pair of 【】 apart from other content, and I use lists quite often.
When I write something like this:
- Lorem ipsum dolor sit amet, consectetur adipisicing elit.
- Autem eius expedita fugit magnam nisi odit porro sit suscipit totam voluptates.
- 【"Lorem ipsum" is used for testing the appearance of a layout.】

I'm expecting a complete list. If the space is omitted by this rule, the last line would end up with a normal paragraph just starting with a character - other than a sentence as a list item.
 This behavior is expected to be consistent across block quotes and table, or any other scenario supports nested lists.
If all the above make sense to you, would there be an extra toggleable switch provided for format rule to support this behavior?

I am thinking I could add it as a normal part of the rule, but adding it for block quotes and tables will just add time to how long it will take.

Do note: I am referring to fullwidth characters in tables and block quotes. If it is handle in lists, it will likely handle nesting already.

Adding as normal part of the rule would be great.
As regards the block quotes part, I mean this new behavior should also apply to lists inside a block quote. 😂 I mostly care about lists right now.

This should be accounted for with lists. There is no need to account for block quotes as they are syntactically correct with no spaces between it and its content. Tables might be a different story, but that will need to be addressed separately if the need arises.
Please let us know if master or the next release does not fix this issue.


Update to Node20

I see that #41 was just resolved with a v2 release. I have some bad news;
Node v16 is also deprecated and have been for the last 6 months :'(
Node.js 16 actions are deprecated.
Is there any chance to update to v20?

Any timeline for the 3.0 release to silence the node 16 deprecation warning on my GH Action runs?

We're waiting to merge #46 before releasing v3.
In the meantime, you can reference the master branch in your workflows to get rid of the warning:
steps:
  - name: Run the Cake script
    uses: cake-build/cake-action@master


pull request for e2e test at 12/14/2016 11:26:11

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


env setup added for simple bike repositioning

Description

env setup added for simple bike repositioning
PYTHONPATH setup added in README

Linked issue(s)/Pull request(s)

issue_number

Type of Change

[ ] Non-breaking bug fix
[ ] Breaking bug fix
[ ] New feature
[ ] Test
[ ] Doc update
[ ] Docker update

Related Component

[ ] Simulation toolkit
[ ] RL toolkit
[ ] Distributed toolkit

Has Been Tested

OS:

[ ] Windows
[ ] Mac OS
[ ] Linux

Python version:

[ ] 3.6
[ ] 3.7

Key information snapshot(s):

Needs Follow Up Actions

[ ] New release package
[ ] New docker image

Checklist

[ ] Add/update the related comments
[ ] Add/update the related tests
[ ] Add/update the related documentations
[ ] Update the dependent downstream modules usage

Codecov Report

Merging #254 (0eec846) into master (6b615d3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #254   +/-   ##
=======================================
  Coverage   64.94%   64.94%
=======================================
  Files         111      111
  Lines        5334     5334
=======================================
  Hits         3464     3464
  Misses       1870     1870

Flag
Coverage Δ

unittests
64.94% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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

Codecov Report

Merging #254 (0eec846) into master (6b615d3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #254   +/-   ##
=======================================
  Coverage   64.94%   64.94%
=======================================
  Files         111      111
  Lines        5334     5334
=======================================
  Hits         3464     3464
  Misses       1870     1870

Flag
Coverage Δ

unittests
64.94% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


[Android] Extensibility to override rendering of entire action set

Automatically generated task. Any updates to the title and body will be overwritten. Do NOT modify the labels.
Android implementation of #1690

Android was completed with #1655


update to .ensime format for 2.0

@yilinwei best place to look https://github.com/ensime/ensime-sbt/blob/2.0/src/main/scala/EnsimePlugin.scala#L685-L724
and just above it are the case class definitions.

@fommil Awesome, will take a look at it.

@yyadavalli @fommil is anybody working on supporting format 2.0 for maven? If not I would be glad to do ti.

Not that I know of. Please go ahead.

Thank you. I will go for it. I am new to the code base, can anybody point me to the specs of ensime1 and/or ensime2 format.

Thank you a lot. Will take a look at it

Pull request 32 closes this


Including headers from other packages

Hi,
This might have been answered before but I haven't found a solution in either the docs, any open pull requests or my playing around.
I wish to include files in my build form a different package.
however I can't do:
includes = ["include", "../otherpackage/include"] so am wondering how to do this within Bazel's methodology.
I have a package core with a libarary in it.
package(default_visibility = ["//visibility:public"])

cc_library(
    name = "Mylib",
    srcs = ["Mylib.c",],
    hdrs = ["Mylib.h",],
    includes = ["include", "headers"],
    deps = ["//project/dependancy:headers",],
)

in the dependancy package I have:
package(default_visibility = ["//visibility:public"])

filegroup(
    name = "headers",
    srcs = glob([
        "*.h",
    ]),
)

This doesn't work, so I'm getting something wrong.
I have tried including the headers as data = ["//project....",]
but that doesn't help, its quite likley that I'm just making a hash of things but if anyone has some quick pointers, or can add an example of such to the documentaion I'd be very appreciative.

The Google way to do this is includes rooted at the WORKSPACE level.
So MyLib can just depend on a cc_library in //project/dependancy:whatever
and then in MyLib .cc files use
#include "project/dependency/some_header.h"
As all bazel compilations have an implicit -I on the directory containing WORKSPACE.
However, if you can't do that yet, then the 'includes' attribute can be used.
But, not in the way you might think.  Includes is for how THIS library is depended on by others (not what this library depends on).
So.  In //package/dependency
cc_library(
name = "whatever",
srcs = ["whatever.c"],
hdrs = glob(["*.h"]),
includes = ["."],
)
The magic is: includes = ["."]
Now when MyLib depends on //project/dependency:whatever it will be able to reference the headers in there without extra work.

Closing the issue, feel free to reopen it if you have further questions.


Fixed URLs for HTML, CSS, JS

Fixed the redirection of URL for HTML, CSS and Javascript

@saragibby Please take a look at these translation changes too.


Update Giorgio's bio, links, and profile pic

👷 Deploy Preview for gladia processing.

Name
Link

🔨 Latest commit
bfbe53d62783198bf672eca1111105b0bcb85d1c

🔍 Latest deploy log
https://app.netlify.com/sites/gladia/deploys/63e268cd8a8b46000802c123


Should allow 80 bytes of data in OP_RETURN

Should allow 80 bytes of data in OP_RETURN, currently still limited to 40 bytes.

yes, I just merge 2H ago a PR for raising the limit. This is currently merged on master.

released on v<IP_ADDRESS>