Edit Token Value from the Inspect Tab

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

It is frustrating not able to edit token value from a inspect tab. We have a white label product and we want to enable our clients to apply their branding/theming on our product. Token Studio enables to select any UI element and see what tokens are applied. If a user wants to edit token value:

First, they need to find the applied token from inspect tab,
Copy the token name
Go to Tokens tab, and find the token
Since the search is not a global search, they need to find the right token set on the left menu.

Describe the solution you'd like
If a user can edit the applied token value from the inspect tab, it would be great. So when you inspect, you can tap a token name and see Edit Token pop up.
Additional context

I run an internal usability study, to understand if we can use Token Studio for theming. 6 out of 6 participants mentioned they would like to able to edit token from an inspect tab. They also mentioned since the search is on top of all the tokens sets, they expect it will perform the search globally, not only on selected token set. Another big issue, if the token value is updated-but not applied, in the inspect tab, they see the updated token value, but since the UI is not updated, they see the old value, it was very confusing for participants.

This request has been moved to our new feedback tool. Go check out https://tokensstudio.featurebase.app/ to create a free account. You can upvote and comment on existing posts, make a new request, and subscribe to be notified when requests move into our roadmap.

Here's the URL to the featurebase post that is linked to this ticket: https://tokensstudio.featurebase.app/p/enhance-inspect-page-in-the-plugin


Newsletters: add 119 (2020-10-14)

[ ] Update topics for content additions and edits, Tuesday @harding
[ ] Update releases/RCs, Tuesday @harding
[x] Bitcoin Core PR Review Club summary @jnewbery or @jonatack
[x] Bitcoin Core #19954 @adamjonas
[x] Eclair #1537 @dongcarl

Note: per this discussion, we're going to try making updates to the topic index in the newsletters that add the noteworthy content.  Reviewers not interested in those updates can skip reviewing those additions (or anything else you want to skip, of course!).  Please let me know if you think this is creating too much churn on the PR or has other problems---I'm happy to try something else.

Looks good to me so far. I don't have any suggested changes.

JoinMarket release: https://github.com/JoinMarket-Org/joinmarket-clientserver/blob/master/docs/release-notes/release-notes-0.7.1.md

I've made a few small changes to the @jonatack and @adamjonas sections. Everything else looks good to me. Thanks all!

Squashed commits and merged. @harding @jonatack @dongcarl @adamjonas  thanks for your contributions this week. @jnewbery and @jonatack also thank you for your reviews and edits. Another newsletter in the books 🥂

Thanks @bitschmidty! There wouldn't be a newsletter at all if it was wasn't for your pulling/squashing/pushing/merging/publishing/tweeting every week. Thanks for all your work :beers:


Update readthedocs domain

For security reasons they've switched the hosted docs from .org to .io.

This trivial change is ready to merge. (So much so that I'm tempted to merge it myself ... :expressionless: )

This trivial change is ready to merge.

You make a fair point! Thanks @rhattersley, consider it done :smile:
Out of interest, to say nothing of the scope of this particular ticket, did you happen to check whether we have any other instances of readthedocs.org within the Iris docs?

did you happen to check whether we have any other instances of readthedocs.org within the Iris docs?

No... because I had my stupid-doofus hat on. :unamused:


Git gutter decorations not showing unless I see a diff or change its property in the settings.json

VSCode Version: Code 1.19.2 (490ef761b76b3f3b3832eff7a588aac891e5fe80, 2018-01-10T15:57:21.071Z)
OS Version: Linux x64 4.14.13-1-ARCH
Extensions:

Extension
Author (truncated)
Version

shell-launcher
Tyr
0.1.6

githistory
don
0.2.3

javadebugger
don
0.1.5

gitlens
eam
7.5.3

file-icons
fil
1.0.7

Lisp
mat
0.1.8

python
ms-
0.9.1

cpptools
ms-
0.14.5

vetur
oct
0.11.5

java
red
0.16.0

trailing-spaces
sha
0.2.11

java-imports-snippets
tus
0.0.1

vim
vsc
0.10.9

vscode-todo-highlight
way
0.5.11

change-case
wma
1.0.0

(1 theme extensions excluded)

Steps to Reproduce:

Open VSCode and change something in a file (git gutter decoration does not appear)
Open the diff view of the changes and closing it makes decorations appear.

Reproduces without extensions: Yes

I'm on stable 1.19.3, still no gutter decorations for git changes!

1.20.0 - working OK now:))


How can I test it on real time video stream?

Thank you for your great work!
I was wondering how can I test it on a real-time video stream. (e.g use my camera)

Thanks for your question. It will depend on the platform you are using to get the data from the camera. For example ROS could be the platform to load the network and also the stream data.  You will need to create a buffer variable in it to save the previous and current frame data and use them for prediction. As the streaming platform is different for different people, we do not provide the code in a specific case. We might provide a example to show how to use a streaming platform with our model in future work.


BUG: Testimonial carousel text

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The text inside the testimonial block is colliding with the outer div and the profile picture is getting cropped.

Expectation :  The size of the profile pictures need to be decreased a bit so that it could fit in the whole div
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@Its-Aman-Yadav Hey please assign me this issue


Please create flutter.googlesource.com/third_party/libpng

Part of https://github.com/flutter/flutter/issues/39532
@godofredoc

Done:
https://flutter-review.googlesource.com/admin/repos/third_party/libpng


String out of boundaries in CIGAR

The issue with StringOutOfBoundException in VarDictJava is also reproduced in Perl, but without crashing (because Perl works differently with negative indexes in the string).
https://github.com/AstraZeneca-NGS/VarDictJava/pull/126
VarDict Perl examples: Lines 1097, 1100
Correctly checked in other lines, e.g. line 1147

The PR was merged. I'll close the issue.


NoopTracer ported from Java

This is a port from Java's opentracing-noop package. It currently has the following deviations:

The Java version has interfaces for every type and defines the INSTANCE fields on them. However, C# does not have interface variables!

I think this makes the interfaces completely unusable/unnecessary. I therefore currently don't have them and I define the INSTANCE fields on the types themselves.

In the Java version, only the interfaces and the NoopTracerFactory are public. Since I currently don't have the interfaces, I made all classes public. However, the INSTANCE properties are internal and the tracer can still only be used through NoopTracerFactory so the behavior should be the same.
In the Java version, NoopSpanBuilder extends NoopSpanContext. I'm not sure if this is still necessary - I created an issue for this in the Java repository: https://github.com/opentracing/opentracing-java/issues/255

There's a debate in #55 about how we should structure these projects. This PR currently uses a separate project to be on par with Java but if #55 concludes differently, I will change it here as well.
Any feedback is more than welcome :)

Looks great! I think the interface/class part makes sense (it was always caused me curiosity why NoopSpan and NoopSpanContext are exposed to the public in Java, but that's another thing ;) )
LGTM 👍

Current status:

I'm still waiting for a reply in opentracing/opentracing-java#255 regarding NoopSpanBuilder extending NoopSpanContext. Shouldn't be a blocker though, as we could always add that later if it's really required.
I moved the types back into the main "OpenTracing" assembly to see what it looks like (see #55). I still like that the most and it follows the example of non-java OpenTracing projects. Will wait for some more feedback regarding this.
There's a comment regarding the potentially confusing name NoopTracerFactory.Create which always returns the same instance. Not sure if we should change this?!

fyi: I'll probably merge this on Tuesday morning (UTC) if there's no contrary comments.

There's a comment regarding the potentially confusing name NoopTracerFactory.Create which always returns the same instance. Not sure if we should change this?!

I think this is fine - in the end, it's supposed to be a no-op Tracer anyway, and changing it will deviate us from the Java API.


Convert Patterns to YAML

YAML could be a much cleaner solution for both patterns and the white/black lists. Just have to sort out the how.

go-yaml: For YAML Parsing
go-bindata: Embedding YAML to compile in


Installing PushPin on AWS

Hi!
I am running my application on AWS elastic beanstalk and Can I know the steps needed to run Pushpin service on AWS?

Hi, I've installed Pushpin on EC2 manually and with Ansible, by simply using apt as described https://pushpin.org/docs/install/. However I've never used beanstalk. I suggest asking about that in the https://gitter.im/fanout/pushpin in case others know.


add windows ssse3,sse4_1,sse4_2 detection for non avx path

This  PR allows to detect >sse3 for non-avx processors by using only windows methods. Which brings identical detection windows caps as of linux or macos.

BTW this detection may superseed WESTMERE arch detect https://github.com/google/cpu_features/blob/8eb944f55d768a3c0250a1e80bb94de7b2560971/src/impl_x86_windows.c#L48
so it might be removed at all (and   _WIN32_WINNT compile time dependency seems fragile)

BTW this detection may superseed WESTMERE arch detect
https://github.com/google/cpu_features/blob/8eb944f55d768a3c0250a1e80bb94de7b2560971/src/impl_x86_windows.c#L48
so it might be removed at all (and _WIN32_WINNT compile time dependency seems fragile)

ah, my bad, when I was working on this problem I only referred to the Microsoft docs. You are right, there is no point in _WIN32_WINNT, it can be removed. Thanks for the PR 👍

Thx @ajax16384 and @toor1245 !

Created PR to update IsProcessorFeaturePresent docs https://github.com/MicrosoftDocs/sdk-api/pull/1251


Support debug events

I'm planning to work on support for getting/deleting events as documented here. I added an issue in Jira but this appears to be slightly more active, so tracking it here.

Closing since doc mentions not supporting these in SDK.


#1003 eslint error

Fixes Issue

closes #1003
Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.

what checks i have to do? is it mergeable?  as it is showing all checks are not passed now its showing all checks are passed .. 🤔

@TechSpiritSS hey can you please merge my request?

Hey your branch is giving conflicts..kindly resolve them


One codebase, every platform???

I am presently evaluating Dioxus for an upcoming web application.  I would like to also support Desktop, TUI, and eventually mobile.
The Dioxus homepage says in big bold letters: "One codebase, every platform".  This strongly implies that I can have a single codebase that runs on every (supported) platform.   Based on this claim, I decided to try out Dioxus before any other rust web framework.  I have been following the Dioxus guides.
Yet dx new insists that I choose a single platform/renderer.   And after doing so, if I run dx init it gives an error that Cargo.toml already exists.
I followed the guide and created the hackernews example app.  It uses the Web renderer.  There is no mention how to make it use the Desktop renderer or TUI, or mobile, or fullstack.   My expectation (from the homepage) is that I should be able to (easily) wire up multiple renderers in order to target different platforms from the same underlying codebase.
Likewise, I looked in the examples directory and didn't see anything obvious about how to do it.   Or if its even possible.  Nor did I find mention in the docs.
The project homepage seems to be bragging loudly about this functionality... so where is it?
I'd be grateful for any pointers to examples or documentation of this functionality.
Or if it is not possible and one truly must choose a single platfform per codebase, I'd like to know that also.

@dan-da, you probably should probably move this to  Q&A discussions. It strikes me as a question.

Yet dx new insists that I choose a single platform/renderer. And after doing so, if I run dx init it gives an error that Cargo.toml already exists.

Choosing the platform in dx new mostly just adds a feature to the dioxus crate. Support for multiple platforms at once in the template is tracked in https://github.com/DioxusLabs/dioxus-template/issues/9

I followed the guide and created the hackernews example app. It uses the Web renderer. There is no mention how to make it use the Desktop renderer or TUI, or mobile, or fullstack. My expectation (from the homepage) is that I should be able to (easily) wire up multiple renderers in order to target different platforms from the same underlying codebase.

The hackernews guide intentionally does not use any libraries that are platform specific. It will work with desktop, fullstack, web or liveview.

Likewise, I looked in the examples directory and didn't see anything obvious about how to do it. Or if its even possible. Nor did I find mention in the docs.

We don't have great documentation for this currently, but you can actually run many of the examples on different platforms with these commands:
dx serve --platform web --features web --example _ -- --no-default-features
dx serve --platform fullstack --features fullstack --example _ -- --no-default-features
dx serve --platform liveview --features liveview --example _ -- --no-default-features

@onweru on the contrary, it strikes me that this is a documentation issue/omission at the very least.  "One codebase, one platform" is a main selling point on the homepage yet is not even discussed in the guide or docs, afaict.
@ealmloff thx for the comments.  very helpful.

The hackernews guide intentionally does not use any libraries that are platform specific. It will work with desktop, fullstack, web or liveview.

happy to hear that.  It seems the guide should mention it, and how to run for each platform.
How about TUI and mobile?   If it doesn't work for those why not, ie what should my expectations be?   I'm trying to (constructively) highlight that I don't see any guidance in the docs/guide for what platforms can be expected to work simultaneously or not, and this is confusing for a newcomer.   What constraints do I need to adhere to in order to work on all supported platforms?   Eg, I don't think I can expect TUI to have all the controls that Web and Desktop do.

We don't have great documentation for this currently, but you can actually run many of the examples on different platforms with these commands:

Thanks for this!   Ok, I put this in my Cargo.toml:
dioxus = { version = "0.5", features = ["web", "fullstack", "desktop", "liveview"] }

and tried to run the hackernews demo:
$ dx serve --platform fullstack --features fullstack --example _ -- --no-default-features
| 💼 Waiting to start building the project...
error: none of the selected packages contains these features: fullstack
build desktop done
Error: 🚫 Serving project failed:

Caused by:
    0: I/O Error: No such file or directory (os error 2)
    1: No such file or directory (os error 2)

Not sure where to go from here.
Also, I don't see any feature-flag available for TUI...?

ok, I've made some progress.  I'm using ubuntu 22.04.  I've been able to build the hackernews app (from the guide) with platform=desktop.   But I still get an error trying to run it.
My Cargo.toml now contains:
dioxus = { version = "0.5", features = ["web", "fullstack", "desktop", "liveview", "router"] }

I am building with the command:
dx build --platform desktop  -- --no-default-features

To get this far, I had to run the following commands:
$ sudo apt install libwebkit2gtk-4.1-dev   build-essential   curl   wget   file   libssl-dev   libayatana-appindicator3-dev   librsvg2-dev

the above was from Tauri docs linked to by the dioxus "Getting Started" page, under Desktop platform.  So docs were helpful there.
I still got a link error, complaining about -ldxo.   So I had to install libxdo-dev.
$ sudo apt install libxdo-dev

I don't see this mentioned anywhere in the "Getting Started" page.
Ok, so with that out of the way, I try to run the desktop program and get an error:
./target/debug/hacker-news
thread 'main' panicked at /home/danda/.cargo/registry/src/index.crates.io-6f17d22bba15001f/js-sys-0.3.69/src/lib.rs:6013:9:
cannot call wasm-bindgen imported functions on non-wasm targets
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

eh?   what now?

Experimenting, I just tried:
$ dx build --platform tui  -- --no-default-features
error: invalid value 'tui' for '--platform <PLATFORM>'
  [possible values: web, desktop, fullstack]

Ok, so --platform supports web, desktop, fullstack.    What about liveview, tui and mobile?   How does one target those?    color me confused.

a bit more progress.  flailing about, I learned that the wasm-bindgen error is indicating the app isn't being run in a proper wasm environment.  This led me to dx bundle, which seems to be designed to create sucn ah environment.
But when I try it, I hit yet another error:
 $ dx bundle
build desktop done
Adding assets from dist to bundle
thread 'main' panicked at /home/danda/.cargo/registry/src/index.crates.io-6f17d22bba15001f/dioxus-cli-0.5.4/src/cli/bundle.rs:193:13:
Failed to bundle project: BundlerError(
    Error {
        context: "Failed to build data folders and files",
        source: BundlerError(
            Error {
                context: "Failed to copy resource files",
                source: IoError(
                    Os {
                        code: 36,
                        kind: InvalidFilename,
                        message: "File name too long",
                    },
                ),
            },
        ),
    },
)
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

filename too long.   seriously?    and of course it doesn't bother to mention which file.

How about TUI and mobile? What constraints do I need to adhere to in order to work on all supported platforms? Eg, I don't think I can expect TUI to have all the controls that Web and Desktop do.

TUI only supports a limited number of widgets and flexbox layout. Mobile supports a full webview, but it is more difficult to set up currently (see the mobile reference for more information)

Ok, I put this in my Cargo.toml:
dioxus = { version = "0.5", features = ["web", "fullstack", "desktop", "liveview"] }

You need to put each platform under a feature in your cargo.toml:
dioxus = { version = "0.5" }

[features]
desktop = ["dioxus/desktop"]
liveview = ["dioxus/liveview"]
fullstack = ["dioxus/fullstack"]
server = ["dioxus/axum"]
web = ["dioxus/web"]

$ dx serve --platform fullstack --features fullstack --example _ -- --no-default-features
| 💼 Waiting to start building the project...
error: none of the selected packages contains these features: fullstack
build desktop done
Error: 🚫 Serving project failed:

Caused by:
    0: I/O Error: No such file or directory (os error 2)
    1: No such file or directory (os error 2)

Not sure where to go from here.

The _ is a placeholder for the example you want to run. You can try dx serve --platform fullstack --features fullstack --example readme -- --no-default-features

Also, I don't see any feature-flag available for TUI...?

Dioxus TUI has been moved to the blitz repo and may not be supported as a first class renderer in future releases


Patch horizontalAdvance for older Qt versions

Description
This PR let's us hobble along with accidental support for Qt <5.11 for a little longer, by preventing an attribute error.

We're getting a lot of test fails here, are they all unrelated?

yeah, they're all segfaults.  one directly from the new threading tests in notification manager.  should look at that separately ithink


[BUG] EXPLAIN fails for subqueries

EXPLAIN
SELECT * FROM (SELECT * FROM $astronauts)

fails with this error: __repr__ returned non-string (type QueryPlanner)

fixed


[Experiment] Enum annotation sugar

Seems not a big feature(2-4 lines for core), but a nice sugar.
Fixes #16464

@typescript-bot pack this.

Thanks!

This experiment is pretty old, so I'm going to close it to reduce the number of open PRs.


add deu and sakarya to domain list

Sakarya university web url: https://www.sakarya.edu.tr/
Dokuz eylul university web url: https://www.deu.edu.tr/
Sakarya university ceng web url: https://cs.sakarya.edu.tr/
Dokuz eylul university ceng web url: https://ceng.deu.edu.tr/

@Saitpstc I am sorry, the domain ogr.deu.edu.tr is removed from our trusted domains list because the school allows to register an email address there even if you are not a student of the school. If you are the student of this school, please apply with official document (OFFICIAL DOCUMENT tab at https://www.jetbrains.com/shop/eform/students).


'Max data disks' is a confusing term

I have just had this point raised by a customer.  They had assumed that 'Max data disks' did not include the operating system disk as this is not used for data.  They suggested that 'Max disks' would be clearer and less open to misunderstanding, and I agree.  Is there some reason for using the term 'Max data disks' or can it be changed?

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

ID: d11ed461-c687-fc39-0320-ccd60b403e06
Version Independent ID: 50017c9f-a8dd-c0ab-7154-57e1665dc547
Content: Dv3 and Dsv3-series - Azure Virtual Machines
Content Source: articles/virtual-machines/dv3-dsv3-series.md
Service: virtual-machines
Sub-service: sizes
GitHub Login: @joelpelley
Microsoft Alias: jushiman

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

We are currently cleaning up old issues and closing out items that are greater than 90 days old. If an issue is still present, please open a new feedback item on the document so we can prioritize correctly.
#please-close


sysroot and qtrpi_binary

Hi,
I cannot download sysroot and qtrpi_binary in common.sh. Where can I find these files?
Thanks a lot!

Hi there,
The old website is down. I have to re-upload the binaries somewhere else.
I'll keep you posted when this will be done.
Until then, you can still generate yourself the binaries with the init-qtrpi-full.sh script.

Hi,
Thank u very much! I'll try generate the binaries by myself.


Cannot use mingw32-make on this project

I can't use mingw32-make on this project. I did download the submodules, but I get this in the terminal:
 process_begin: CreateProcess(NULL, uname, ...) failed. make: process_begin: CreateProcess(NULL, uname -m, ...) failed. make: 'cut' is not recognized as an internal or external command, operable program or batch file. '[' is not recognized as an internal or external command, operable program or batch file. The system cannot find the path specified. -f was unexpected at this time. The system cannot find the path specified. -f was unexpected at this time. process_begin: CreateProcess(NULL, uname, ...) failed. make[1]: process_begin: CreateProcess(NULL, uname -m, ...) failed. make[1]: 'cut' is not recognized as an internal or external command, operable program or batch file. '[' is not recognized as an internal or external command, operable program or batch file. The system cannot find the path specified. -f was unexpected at this time. The system cannot find the path specified. -f was unexpected at this time. process_begin: CreateProcess(NULL, uname, ...) failed. make[1]: process_begin: CreateProcess(NULL, uname -m, ...) failed. make[1]: 'cut' is not recognized as an internal or external command, operable program or batch file. '[' is not recognized as an internal or external command, operable program or batch file. The system cannot find the path specified. -f was unexpected at this time. The system cannot find the path specified. -f was unexpected at this time.
If someone could help me compile this project that would be awesome 😁

The supplied Makefile is intended for Linux.
Since you’re on Windows you should use the supplied VisualStudio solution to build the library.

I can't use mingw32-make on this project. I did download the submodules, but I get this in the terminal:
`
process_begin: CreateProcess(NULL, uname, ...) failed.
make:
process_begin: CreateProcess(NULL, uname -m, ...) failed.
make:
'cut' is not recognized as an internal or external command,
operable program or batch file.
'[' is not recognized as an internal or external command,
operable program or batch file.
The system cannot find the path specified.
-f was unexpected at this time.
The system cannot find the path specified.
-f was unexpected at this time.
process_begin: CreateProcess(NULL, uname, ...) failed.
make[1]:
process_begin: CreateProcess(NULL, uname -m, ...) failed.
make[1]:
'cut' is not recognized as an internal or external command,
operable program or batch file.
'[' is not recognized as an internal or external command,
operable program or batch file.
The system cannot find the path specified.
-f was unexpected at this time.
The system cannot find the path specified.
-f was unexpected at this time.
process_begin: CreateProcess(NULL, uname, ...) failed.
make[1]:
process_begin: CreateProcess(NULL, uname -m, ...) failed.
make[1]:
'cut' is not recognized as an internal or external command,
operable program or batch file.
'[' is not recognized as an internal or external command,
operable program or batch file.
The system cannot find the path specified.
-f was unexpected at this time.
The system cannot find the path specified.
-f was unexpected at this time.
`
If someone could help me compile this project that would be awesome 😁


docs: add missing config reference

fixes: https://github.com/foundry-rs/book/issues/313

Would like some feedback on the configuration keys

I left the following commented out in the toml like #solc_version

wasn't sure how to represent None in toml

# Switch optimizer components on or off in detail
#optimizer_details = None
# Url of the rpc server that should be used for any rpc calls
#eth_rpc_url = None
# Etherscan API key
#etherscan_api_key = None
# Only run test functions matching the specified regex pattern
#test_pattern= None
# Only run test functions that do not match the specified regex pattern
#test_pattern_inverse = None
# Only run tests in contracts matching the specified regex pattern
#contract_pattern = None
# Only run tests in contracts that do not match the specified regex pattern
#contract_pattern_inverse = None
# Only run tests in source files matching the specified glob pattern
#path_pattern = None
# Only run tests in source files that do not match the specified glob pattern
#path_pattern_inverse = None
# The block gas limit
#block_gas_limit = None
# The memory limit of the EVM (32 MB by default)

There is no null value in TOML, i guess leaving it commented is the best option, the recommendation seems to be to include it only if needed.


integrate pika_exporter into pika repo

Integrate the feature of pika_exporter into the pika repository. Exporter release with the pika version，and the exporter of each version only supports collecting pika metrics of this version.

The first stage is to confirm the pika metrics

[ ] Most of the pika metrics are obtained by parsing the pika's INFO command information. The Pika version is based on the latest master branch, need to confirm the metrics of this version.
[ ] In addition to INFO command information, pika will also add metrics with command execution time and rocksdb related metrics(#1218). It is necessary to confirm the metrics details and export method of pika for these metrics.
[ ] Group metrics for more flexible control of collection granularity.

The second stage is to develop the program

[ ] Coding for exporter, and combine it with pika's current deployment method.
[ ] Pika exporter is a prometheus exporter, and the industry usually uses grafana as the display panel of prometheus, so can consider providing an out-of-the-box grafana dashboard template.
[ ] As above, it is also possible to provide alertmanager rules template.

1 对 metrics 指标进行分组，每个分组是否监控可以配置开关； 2 宗志在 #29 这个比较那个老的 issue 里面也提到过，能否配置告警模板，在监控工作完成之后，根据模板推进告警工作。不过告警是紧贴相关具体系统 如 zabbix 的，此项工作待议。 3 以后这种 issue，不管是题目还是内容，尽量使用英文。

ok

List pika metrics base on the latest master branch. See here
To be added: command execution time and rocksdb related metrics


Deno deploy action needs upgrade to latest Deno version

When running from the default Deno version installed by the setup-deno action, the deploy action fails with:
error: The source code is invalid, as it does not match the expected hash in the lock file.
  Specifier<EMAIL_ADDRESS>  Lock file: /home/runner/work/trg/trg/deno.lock
Error: Process completed with exit code 10.

as seen in

https://github.com/kkga/ter/actions/runs/5376914498/jobs/9754708635#step:4:163

Pinning Deno in the workflow run to the latest available version, which was working locally, helped to mitigate the issue in my fork.
       - name: Setup Deno
-        uses<EMAIL_ADDRESS>+        uses: denoland/setup-deno@v1
+        with:
+          deno-version: "1.34.3"

Reference:

https://github.com/almereyda/trg/blob/1a9346396915caf607b54b758c2e90125660eea8/.github/workflows/deno-deploy.yml#L21-L24

Pinning Deno in the workflow run to the latest available version, which was working locally, helped to mitigate the issue in my fork.

Thanks, but why didn't you just submit a PR? :)

My branch has other changes #44 and I didn't have the time for cherry-picking the commits that lead to this line.


clean up the control plane

We can now use  clusteradm to destroy  a managed cluster. I will be also helpful to cleanup the hub cluster use cluteradm also.

@qiujian16 Is seems liking this issue #50 , right ?
Also, when do the user take this cleanup action:

After the clusteradm init
After the clusteradm unjoin <managed cluster>

Please correct me, Thanks

ah yes, the both are the same. I meant we can clean after clusteradm init

@qiujian16 ,OK,  I will add a new option(clean or destroy) for this cleanup action on the Hub. Please let know which one is suitable to us，Thanks

I think maybe clean

@qiujian16 , you can close this issue, Thanks


Does this extension support code-server?

When I try to install it on code-server, I got the message below:

The 'Open Remote - SSH' extension is not available in code-server for the Web. Click 'More Information' to learn more.

Hi @kaiyuhou, code-server is just the server side (installed on the remote), this extension is meant to be installed in the vscode desktop forks like vscodium (installed locally) and it will install the compatible vscode-server on the remote automatically.
Can you explain a little more about you use case?

If you want to access your remote machine using vscodium (local vscode) then install this extension
if you want to access your remote machine using the browser then keep using code-server

Thanks for your quick response!
I would consider code-server  as a "locally" installed vscode on the remote machine. The only difference between code-server and vscodium is the front-end UI of code-server is browser-based. (Please correct me if I am wrong.)
Since the remote machine (where code-server is installed) has the SSH client, it should have the ability to SSH into a target remote.

So you have remoteMachine1 with code-server and want to connect to remoteMachine2 (and I guess remoteMachine2 cannot be exposed to the internet) and access both of them using the browser? I think that's a really special case, so what you want is:

connect with ssh to remoteMachine2 and install code-server running in port X
create an ssh tunnel from remoteMachine1 to remoteMachine2 in port X and expose remoteMachine1 port X or setup a reverse proxy :thinking:
connect using the browser

This extension uses a special vscode API to connect the local ui to the remote machine, in your case that's not needed as they are independent each code-server serves it's own front end resources

Thanks for your response.
I now think I may misunderstand the target use case of remote ssh externsion.

Suppore we have a vscodium in the local desktop1 with the ssh extension.
We access the remoteMachine1 by ssh extension to access the project1.
Now, we can access the files of project1 as we accesing the files of local projects.
I initially thought when we run and debug the project1, the project1 runs on the local machine. (So, we just use remoteMachine1 as the file storage, and leverage the horsepower of desktop1 to compile). Now, I thought I am wrong.

Back to my case:

remoteMachine1 is a powerful machine with code-server.
remoteMachine2 is not that powerful, so it even cannot support a self-hosted code-server.
Now, I think, the externsion I want to find is to help me access the files in remoteMachine2 as the local file of remoteMachine1.

I initially thought when we run and debug the project1, the project1 runs on the local machine. (So, we just use remoteMachine1 as the file storage, and leverage the horsepower of desktop1 to compile). Now, I thought I am wrong.

yeah that's not correct, on the contrary all the heavy stuff is done on the remote machine

Now, I think, the externsion I want to find is to help me access the files in remoteMachine2 as the local file of remoteMachine1

Maybe this extension fits you use case

This extension uses a special vscode API to connect the local ui to the remote machine, in your case that's not needed as they are independent each code-server serves it's own front end resources

Hi @jeanp413 , you mentioned a SPECIAL API, right? When we try to enable Open Remote SSH in code-server, it warns that This extension is disabled because it is defined to run only in code-server for the Desktop.
May I know what the api is? Do you think the code-server's lack of  support for the special api the is root cause of this problem?

Thank you for your quick response. I changed the value of extensionKind and repacked it but it didn't work in code-server still, I will give a feedback to the code-server team.
Just to be clarity, whant I do NOT want to connect to a code-server instance via remote-ssh. Instead, I want to install the remote-ssh as an extension of a code-server instance and use it to connect other machines, just like what we do in a desktop distrubtion. :)

Just to be clarity, I do NOT want to connect to a code-server instance via remote-ssh. Instead, I want to install the remote-ssh as an extension of a code-server instance and use it to connect other machines, just like what we do in a desktop distrubtion

Just to double-check I'm hearing you're right:

code-server is running on a machine
you access it install this extension
now you want to use the extension to remot-ssh into another machine (maybe a remote vm somewhere?)

is that right?

If that were the intended use case why not just use ssh jump hosts?

Yes, it is.

If that were the intended use case why not just use ssh jump hosts?

well, I assume you are talking about accessing a code-server instance that runs on a innet-net vm via ssh jump host.
Firstly, I have multiply vms and installing the code-server many times in different vms is not an interesting work. Yes, coder/coder might be a life saver, but syncing settsings and extensions is another problem.
Secondly, many users and I access code-server instances with iPad, it is not so easy to keep a ssh jump host alive in the backgroud, unless the ssh tunnel is maintained by code-server itself, just like what ms vscode server does.

@Creling nice - I wasn't even aware of that use case. Also setting that up on more than a couple of remote hosts is painful and should be dealt with differently.

@Creling nice - I wasn't even aware of that use case. Also setting that up on more than a couple of remote hosts is painful and should be dealt with differently.

Glad to get your understanding :)

Yes, it is.

Cool! @jeanp413 I don't know what's needed to support this in code-server, but if it's something we (the code-server team) can help with, please let me know!

Just to double-check I'm hearing you're right:

code-server is running on a machine
you access it install this extension
now you want to use the extension to remot-ssh into another machine (maybe a remote vm somewhere?)

is that right?

Yes, it is.

I'm still not understanding correctly, this extension is not meant to connect a remote machine to another remote machine, if you are using code-server you are already connected to a remote machine.
As I said before, if you want to access remote machine B using code-server then just install it in there, this extension does the same in the end it installs the vscode/codium server (code-server is the same just with the web bits so you can connect from the browser) on the remote machine and tunnel the connection locally to desktop vscode.
If you don't want to install code-server on remote machine B, then maybe this extension vscode-sshfs is a better fit for your use case.

if you are using code-server you are already connected to a remote machine.

I guess it depends on how you define "remote machine." I have code-sever installed on my local macOS laptop and I'd want to use an extension like this one to connect to another remote machine.

If you don't want to install code-server on remote machine B, then maybe this extension vscode-sshfs is a better fit for your use case.

Awesome! We'll recommend that moving forward. Thanks!

if you are using code-server you are already connected to a remote machine

FWIW I use code-server locally because I prefer my editor to be a browser tab. I am currently running everything locally, but for some tasks I'd like to connect to a remote host.

Hey, I want to go over my requirements to prove that remote ssh isn't meaningless on code-server. First of all, due to the internal network of the server, I usually need remote ssh 3 machines on window vs code to reach the server environment I need, but now I hope to use code-server on the first machine at the beginning, so I only need to carry an ipad with me for office. However, code-server alone cannot directly connect to the final server for code compilation, file viewing and other operations. Therefore, I think the use of emote ssh in code-server is very important. Is there any way to achieve the effect I want?

@Breeze-Zero for your use case one way would be to install code server on the remote machine and then just create an ssh tunnel from the local machine to the port coder server is running

@jeanp413 The use-case that has not been defined here is Chromebooks (without running Linux).

Chrome OS is based on Linux kernel.

@jeanp413 The use-case that has not been defined here is Chromebooks (without running Linux).

Chrome OS is based on Linux kernel.

How is this relevant? It's an ARM variant, so is FreeBSD and MacOS, doesn't mean you can run the same code.
You have to run a virtual Linux container inside ChromeOS to use VSCode locally. This is not what we want. We want to run remote, but also have the option of running Open Remote on that remote host.

@Breeze-Zero for your use case one way would be to install code server on the remote machine and then just create an ssh tunnel from the local machine to the port coder server is running

I understand, however, if that's possible, no one would be raising this issue.
The issue exists because for many reasons it can't (server ops disagree, resource constraints, change request approval etc.)
There is also an use case where coder is installed on android and turning an android phone into a coding environment, since some android devices are advertised as such.


Maximum Flutter Version support for GetX4?

What is the maximum flutter version support with Getx4?
Anyone using above 3.19?
I am currently stuck at 3.16.9, wandering if it is okay to upgrade to higher flutter version?

Flutter 3.22.0 also can use


botocore 1.31.81 or greater is breaking change for celery tasks using sqs

Describe the issue
celery tasks using botocore 1.31.81 or greater does not execute tasks from sqs queue
the messages are posted as encoded json.
this is the example of how this is being posted onto sqs queue
# generate celery task message
    msg_id = str(uuid.uuid4())

    msg_envelope = {
        "content-encoding": "utf-8",
        "content-type": "application/json",
        "headers": {},
        "properties": {
            "body_encoding": "base64",
            "correlation_id": msg_id,
            "delivery_info": {
                "exchange": "",
                "routing_key": ""
            },
            "delivery_tag": ""
        }
    }

    msg_body = {
        "task": task_name,
        "args": task_arg,
        "kwargs": task_kwargs,
        "id": msg_id,
        "retries": 0
    }
    msg_envelope["body"] = base64.b64encode(json.dumps(msg_body).encode('utf8')).decode('utf8')
    msg = base64.b64encode(json.dumps(msg_envelope).encode('utf8')).decode('utf8')
    enrtry = {

        'MessageBody': msg,
        'Id':msg_id
    }
  response = sqs.send_message_batch(
                QueueUrl=QUEUE_URL,
                Entries=batch_entries
            )

Is there a new structure we need to follow for posting encode json documents?
Links
https://boto3.amazonaws.com/v1/documentation/api/latest/guide/sqs-examples.html

Hi @stevegaunt thanks for reaching out. In botocore 1.31.81 support for using the AWS JSON-1.0 protocol with SQS was introduced per the CHANGELOG. Here are some FAQs about that protocol: https://docs.aws.amazon.com/AWSSimpleQueueService/latest/SQSDeveloperGuide/sqs-json-faqs.html.
This may affect existing workflows under certain circumstances. To better understand this - could you share your full code snippet and debug logs (with any sensitive info redacted) by adding boto3.set_stream_logger('') to your script?

(Note: I am not from the botocore team)
@stevegaunt yes, this is expected since Celery is not compatible with the newly released botocore. My recommendation is to adding a constraint to avoid bringing the latest botocore version.


Org contributions

Odilia looks like a very interesting project! Although I'm not a visually impaired linux user, I would like to help out as I believe that accessibility is a huge gap that we need to bridge in our community.
If anyone could point me to a todo list, I'd love to try and contribute.
Sincerely,
Aakash Sen Sharma

Hey! We'd love contributuons!
Since you're already supporting us through the swhkd crate, I'd love some help with input. This is my primary focus. The other two main contributers handle most of everything else.
Currently, our adaption of the code supports Option for keysym, with capslock as a modifier instead of a key. This is so capslock + b/a/o can be used for switching modes (think like Vim modes, but for interacting with your screen reader).
I'm super jazzed about your involvement! I'd gladly explain anything you need. We're working on better documentation as we speak, but I'd love to fill in any gaps.
Thanks again! Can't wait for a PR 👍

Feel free to join our Matrix channel if you want.
Join my space Odilia Screen Reader https://matrix.to/#/#odilia-screen-reader:matrix.org

Oh I forgot to mention what you could help me with.
It would be nice if we could get the reversed key problems fixed. I mentioned it in the thread on swhkd but I'll mention it here again:
If I have a keybinding capslock+f and I type f + capslock, then capslock gets passed through to the window manager, my caps gets turned on and the f key is tepeatedly typed by the input daemon.
Can't quite figure out why this is. Any help would be greatly appreciated.

On Mon Aug 15, 2022 at 11:30 AM IST, Tait Hoyem wrote:

Since you're already supporting us through the swhkd crate, I'd love some help with input. This is my primary focus. The other two main contributers handle most of everything else.

Perfect, I'll try and help out with sohkd.

Currently, our adaption of the code supports Option for keysym, with capslock as a modifier instead of a key. This is so capslock + b/a/o can be used for switching modes (think like Vim modes, but for interacting with your screen reader).
Ok so caps should be treated as modifier, I'll get on it asap.

On Mon Aug 15, 2022 at 11:33 AM IST, Tait Hoyem wrote:

Feel free to join our Matrix channel if you want.
Join my space Odilia Screen Reader https://matrix.to/#/#odilia-screen-reader:matrix.org

Done!

On Mon Aug 15, 2022 at 10:11 PM IST, Tait Hoyem wrote:

It would be nice if we could get the reversed key problems fixed. I mentioned it in the thread on swhkd but I'll mention it here again:
If I have a keybinding capslock+f and I type f + capslock, then capslock gets passed through to the window manager, my caps gets turned on and the f key is tepeatedly typed by the input daemon.
I haven't been able to reproduce this yet. I will investigate this.

Just going to close this issue, but so far your contributions have been invaluable and I hope we can continue to collaborate through the swhkd project.


[powerdns] flip default to postgres, fix postgres dependent chart

Special notes for your reviewer:
Found a bunch of weird stuff that I overlooked...
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [radarr])

Dependent docker image doesn’t support Postgres. Holding off on this merge till https://github.com/psi-4ward/docker-powerdns/pull/23 is complete.

Looks like that PR is nearly 2 years old, should we close this issue and remove postgres from the chart?

@onedr0p -- going to adjust the chart to use the image created by @naps in #94

Closes #94

This PR should be rebased. after that, @carpenike is it ready for review?

Not yet... apparently i'm not setting the postgres password properly.

Not yet... apparently i'm not setting the postgres password properly.

I see in the action logs that the powerDNS pod is failing due to livenessProbe failures. The postgresql pod takes some time to boot up, maybe it hasn't properly connected to the database in time (not that I see it trying to do that in the logs though)?

@bjw-s -- I'm wondering if it's due to the init process from PowerDNS not running / creating the zone...
Trying to back-port current config into 5c64c19.

@bjw-s -- I'm wondering if it's due to the init process from PowerDNS not running / creating the zone...
Trying to back-port current config into 5c64c19.

Likely due to the fact that the new docker image doesn't deploy the webserver so the probes were failing. Flipping to tcp port

This should get a working deployment of PowerDNS without the Web API. Will take a crack at that next ones @naps confirms core functionality.


fix problem error about miss parametre in finetune-cli.py

hi @SWivid thank you very much for the  update  , one think  i forget to update the last commit in file finetune-cli.py
because give you error miss parametre --finetune
thank you

i just cache all error like this with get message where is the problem
and i just update the gpu problem

quick quetion can use --mixed_precision in accelerate ?


update weekly

Coverage increased (+0.001%) to 75.828% when pulling ea08254b105e2e2b2b7a3d9786c97d89670fdff7 on SirDimas:dev into 5ca0db7248954719a426c4f8da064d07fe929696 on danliris:dev.


Starred entity icons should be available via some API

Feature Suggestion
#9909 removed the existing favoriteEntityIcon as we through it was the wrong kind of abstraction. It inlined the existing usage, which is suboptimal as it will lead to duplication. We should provide some way to render the desired icons.
Possible Implementation
One suggestion was to add FavoriteEntity.Icon, another way forward could be to add these icons as system icons that are available in the app. If we go down the system icon route we also need to decide how to handle the coloring of the icons, whether it should be embedded in the icon itself or for example rendered using the theme palette.
Context
https://github.com/backstage/backstage/pull/9909#discussion_r817849761

Close in favor of #10784


[BUG] Failed to connect to git.rancher.io port 443

Rancher Server Setup
Rancher version: 2.9.3
Installation option (Docker install/Helm Chart): Docker install using k3s
)
Custom: Running a Docker command on a node.
Describe the bug
Bug description: After suspending the Ubuntu VM where Rancher is installed and resuming it, I am unable to access Rancher. I checked the Docker logs of the Rancher server and it shows an error related to connection issues, preventing Rancher from starting correctly.
To Reproduce
Steps to reproduce the behavior:
Install Rancher on the Ubuntu VM using Docker.
Suspend the VM and then resume it.
Check the Docker logs on the Rancher server and notice the error messages.
Result
Rancher fails to restart after resuming the VM. The Docker log contains the following error:
[ERROR] error syncing 'c-m-kmlnv228': handler cluster-deploy: cannot connect to the cluster's Kubernetes API, requeuing
[INFO] Stopping cluster agent for c-m-kmlnv228
[ERROR] failed to start cluster controllers c-m-kmlnv228: context canceled
Expected Result
Rancher should restart successfully after suspending and resuming the VM, without any connection issues.
Screenshots

The catalog failure is probably unrelated ( at least directly), but may hint at a problem with the network in the vm not correctly restored after the suspend.  Note that both k3s ( and rancher in docker which uses k3s behind the scences) and docker depend on a stable network config as they install lots of  netfilter rules. If the suspend/resume do change the network, you probably need to restart docker/k3s.


"Number of channels seleted" indicator

As suggested by Mel.

LeoV suggested using Selection: Bars X Channels instead as "Sum" is unclear

This is now in AbyssBox, whether or not its good enough to port is up to you to decide


IndPreBelow.mkBelow failure

The function IndPredBelow.mkBelow fails to generate brecOn for the following inductive predicate.
set_option trace.Meta.IndPredBelow true in
inductive TransClosure (r : α → α → Prop) : α → α → Prop
  | extends : r a b → TransClosure r a b
  | trans_left : r a b → TransClosure r b c → TransClosure r a c

The trace contains
[Meta.IndPredBelow] failed to prove brecOn for TransClosure.below
    couldn't solve by backwards chaining (Lean.Meta.IndPredBelow.maxBackwardChainingDepth = 10): case a
    α : Sort u_1
    r : α → α → Prop
    motive✝ : (a a_1 : α) → TransClosure r a a_1 → Prop
    a✝³ a✝² : α
    : TransClosure r a✝³ a✝²
    : ∀ (a a_1 : α) (x : TransClosure r a a_1), TransClosure.below x → motive✝ a a_1 x
    a b c : α
    : r a b
    a✝ : TransClosure r b c
    a_ih✝ : TransClosure.below a✝
    ⊢ r a a✝³

Thus, we cannot use structural recursion in the following definition
def trans' {a b c} : TransClosure r a b → TransClosure r b c → TransClosure r a c
| .extends h₁       => .trans_left h₁
| .trans_left h₁ h₂ => .trans_left h₁ ∘ trans' h₂

This issue has been reported at
https://leanprover.zulipchat.com/#narrow/stream/270676-lean4/topic/Recursive.20definition.20fails/near/301780366

@DanielFabian Do you have time to take a look?
Thanks,
Leo

At first glance it looks like unprovable non-sense. So it's possible that it applied a bad tactic some steps earlier or something about the goal is wrong. I'll try to understand a bit better what is going on

here's roughly what's going on:
We first do a few introductions, apply the IH, induction and apply constructors. Up to this point, I think it looks still ok. This we create a bunch of goals to solve:

     α : Sort u_1
     r : α → α → Prop
     motive✝ : (a a_1 : α) → Ex.TransClosure r a a_1 → Prop
     a✝² a✝¹ : α
     : Ex.TransClosure r a✝² a✝¹
     : ∀ (a a_1 : α) (x : Ex.TransClosure r a a_1), Ex.TransClosure.below x → motive✝ a a_1 x
     a b : α
     : r a b
     ⊢ r a b

     α : Sort u_1
     r : α → α → Prop
     motive✝ : (a a_1 : α) → Ex.TransClosure r a a_1 → Prop
     a✝³ a✝² : α
     : Ex.TransClosure r a✝³ a✝²
     : ∀ (a a_1 : α) (x : Ex.TransClosure r a a_1), Ex.TransClosure.below x → motive✝ a a_1 x
     a b c : α
     : r a b
     a✝ : Ex.TransClosure r b c
     a_ih✝ : Ex.TransClosure.below a✝
     ⊢ α

     α : Sort u_1
     r : α → α → Prop
     motive✝ : (a a_1 : α) → Ex.TransClosure r a a_1 → Prop
     a✝³ a✝² : α
     : Ex.TransClosure r a✝³ a✝²
     : ∀ (a a_1 : α) (x : Ex.TransClosure r a a_1), Ex.TransClosure.below x → motive✝ a a_1 x
     a b c : α
     : r a b
     a✝ : Ex.TransClosure r b c
     a_ih✝ : Ex.TransClosure.below a✝
     ⊢ r a ?b

     α : Sort u_1
     r : α → α → Prop
     motive✝ : (a a_1 : α) → Ex.TransClosure r a a_1 → Prop
     a✝³ a✝² : α
     : Ex.TransClosure r a✝³ a✝²
     : ∀ (a a_1 : α) (x : Ex.TransClosure r a a_1), Ex.TransClosure.below x → motive✝ a a_1 x
     a b c : α
     : r a b
     a✝ : Ex.TransClosure r b c
     a_ih✝ : Ex.TransClosure.below a✝
     ⊢ Ex.TransClosure r ?b c

     α : Sort u_1
     r : α → α → Prop
     motive✝ : (a a_1 : α) → Ex.TransClosure r a a_1 → Prop
     a✝³ a✝² : α
     : Ex.TransClosure r a✝³ a✝²
     : ∀ (a a_1 : α) (x : Ex.TransClosure r a a_1), Ex.TransClosure.below x → motive✝ a a_1 x
     a b c : α
     : r a b
     a✝ : Ex.TransClosure r b c
     a_ih✝ : Ex.TransClosure.below a✝
     ⊢ Ex.TransClosure.below ?a✝

     α : Sort u_1
     r : α → α → Prop
     motive✝ : (a a_1 : α) → Ex.TransClosure r a a_1 → Prop
     a✝³ a✝² : α
     : Ex.TransClosure r a✝³ a✝²
     : ∀ (a a_1 : α) (x : Ex.TransClosure r a a_1), Ex.TransClosure.below x → motive✝ a a_1 x
     a b c : α
     : r a b
     a✝ : Ex.TransClosure r b c
     a_ih✝ : Ex.TransClosure.below a✝
     ⊢ motive✝ ?b c ?a✝

this all looks solvable to me. I'm not perfectly sure about the last goal, but I think if we apply backwards chaining, we could solve it.
The problem, I believe is the second goal that simply wants an α. And there 5 different objects of type α around. So I think what's happening is that the prove simply picks the first but that then effectively chooses the metavariable and it modifies the other goals rendering some of them unprovable!
When we then subsequently ask closeGoal to prove the goal, things go south:
[Meta.IndPredBelow] closeGoal:
    case a
    α : Sort u_1
    r : α → α → Prop
    motive✝ : (a a_1 : α) → Ex.TransClosure r a a_1 → Prop
    a✝² a✝¹ : α
    : Ex.TransClosure r a✝² a✝¹
    : ∀ (a a_1 : α) (x : Ex.TransClosure r a a_1), Ex.TransClosure.below x → motive✝ a a_1 x
    a b : α
    : r a b
    ⊢ r a b
[Meta.IndPredBelow] closeGoal:
    case b
    α : Sort u_1
    r : α → α → Prop
    motive✝ : (a a_1 : α) → Ex.TransClosure r a a_1 → Prop
    a✝³ a✝² : α
    : Ex.TransClosure r a✝³ a✝²
    : ∀ (a a_1 : α) (x : Ex.TransClosure r a a_1), Ex.TransClosure.below x → motive✝ a a_1 x
    a b c : α
    : r a b
    a✝ : Ex.TransClosure r b c
    a_ih✝ : Ex.TransClosure.below a✝
    ⊢ α
[Meta.IndPredBelow] closeGoal:
    case a
    α : Sort u_1
    r : α → α → Prop
    motive✝ : (a a_1 : α) → Ex.TransClosure r a a_1 → Prop
    a✝³ a✝² : α
    : Ex.TransClosure r a✝³ a✝²
    : ∀ (a a_1 : α) (x : Ex.TransClosure r a a_1), Ex.TransClosure.below x → motive✝ a a_1 x
    a b c : α
    : r a b
    a✝ : Ex.TransClosure r b c
    a_ih✝ : Ex.TransClosure.below a✝
    ⊢ r a a✝³

Notice how the last goal has ⊢ r a a✝³ as goal as opposed to ⊢ r a ?b as before. And now it's unprovable.

So, in other words, I don't think we can greedily apply backwards resolution. I think we need to exhaustively try all possible proves for each goal.
This may sound terrible, but if we first check for each goal if we can uniquely prove using backwards chaining and delay it otherwise, we wouldn't pay that much most of the time. And, indeed, in this case, we wouldn't pay anything. Because if we first proved the goal ⊢ r a ?b, that would constrain the ?b in such a way, that the other goal would then become uniquely provable, I think.

+1 on this issue.  Ran into the same thing (reported on Zulip as well.)
Any progress?

Together with @DanielFabian we looked into this, and it turns out we can solve this by deleting code:
@@ -310,9 +311,6 @@ where
         args.back.withApp fun ctor _ => do
         let ctorName := ctor.constName!.updatePrefix below.constName!
         let ctor := mkConst ctorName below.constLevels!
-        let ctorInfo ← getConstInfoCtor ctorName
-        let (mvars, _, _) ← forallMetaTelescope ctorInfo.type
-        let ctor := mkAppN ctor mvars
         m.apply ctor
     return mss.foldr List.append []

The reason is that the apply tactic by default returns the mvars in a more useful order (non-dependent first), and thus we then get ?b resolved by unification and don't have to search for it.
Not sure if this will solve all problems of this form, but probably a fair number of them.


add migration to remove duplicate eosr from prod

What does this pull request do?
Removes a duplicate end of service report in prod.
There are two rows for the referral: 0900496a-d6f1-4c78-9c15-8f0eeacfb53f
eosr ids: 6c678352-e3e0-4358-a06b-ffe738303b51 , 22f417f8-7faf-498a-bd3b-6d0e0cedb71f
6c678352-e3e0-4358-a06b-ffe738303b51 has a row in the end_of_service_report_outcome table, therefore the change is to remove the other eosr that was created which does not.
What is the intent behind these changes?
There are currently two end of service reports for a referral in prod. This will cause issues as there should only ever be one. Initially brought to attention upon an error running the sp performance report.

@rgforsyth @tomsmyers Will there be a follow-up PR to add the constraint to prevent this happening again?

Thanks for doing this!


Fixed links to new location of documation

Category

[x] Content fix
[ ] New article

What's in this Pull Request?
Looks like the linked-to docs have been moved to https://pnp.github.io/powershell/cmdlets/*
If that's correct, these changes will fix them.

Docs Build status updates of commit 3a283b3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/declarative-customization/site-design-pnppowershell.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit b1b5fb5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/declarative-customization/site-design-pnppowershell.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Ability to create annotated git tags?

I noticed the function terminus site code and its support for log, branches, branch-create, diffstat and commit.  Is it possible for it to support annotated git tags in the future?

Tags are created whenever you do a deploy. We currently don't annotate them, but we've talked about it. Especially now that we have the deploy log built into the dashboard, this makes a lot more sense.

+1

Git tags can be created locally via git and pushed to the platform. Commands in site code are intended to drive functionality within the platform, not to be a fully-featured wrapper for Git.


test(lints): Ensure unused optional dep fires for shadowed dep

This is a way to have an unused optional dependency before 2024 edition.
In prior editions, it is currently a hard error.
I'm tempted to switch that hard error to this lint in prior editions but at minimum, we need to make sure we don't make changes that cause this to revert back to a hard error on 2024+

Thanks!
@bors r+

:hourglass: Testing commit a47d41eca48eff9e2ef1be5569613ddb09cae78f with merge 8f3b72a977591b980355cd7b4b3ba3246e0327de...

:boom: Test timed out

@bors retry

:hourglass: Testing commit a47d41eca48eff9e2ef1be5569613ddb09cae78f with merge 3e89630c350ffadc67657484ef022213ad067921...

:sunny: Test successful - checks-actions
Approved by: ehuss
Pushing 3e89630c350ffadc67657484ef022213ad067921 to master...


Fix docstring typing issue TimeSeriesCrossValidation

Close #21.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@673fb82). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #32   +/-   ##
=========================================
  Coverage          ?   56.28%
=========================================
  Files             ?       41
  Lines             ?     1773
  Branches          ?        0
=========================================
  Hits              ?      998
  Misses            ?      775
  Partials          ?        0

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 673fb82...23bfc30. Read the comment docs.


[Failing Test]: Python MongoDB integration test failing

What happened?
Since Feb 3, affecting at least beam_PostCommit_Python_MongoDBIO_IT and beam_PostCommit_Python37. The error message is
File "/home/jenkins/jenkins-slave/workspace/beam_PostCommit_Python_MongoDBIO_IT/src/build/gradleenv/417525523/lib/python3.10/site-packages/pymongo/topology.py", line 226, in _select_servers_loop
07:42:57     raise ServerSelectionTimeoutError(
07:42:57 pymongo.errors.ServerSelectionTimeoutError: localhost:27326: [Errno 111] Connection refused, Timeout: 30s, Topology Description: <TopologyDescription id: 63e247326405e8d75060cc60, topology_type: Single, servers: [<ServerDescription ('localhost', 27326) server_type: Unknown, rtt: None, error=AutoReconnect('localhost:27326: [Errno 111] Connection refused')>]>

Error connecting to the test container. there is no code change around the date that test started failing. Possibly breaking change upsgtream.
Issue Failure
Failure: Test is consistent failing
Issue Priority
Priority: 2 (backlog / disabled test but we think the product is healthy)
Issue Components

[X] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[X] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

looks like jenkins workers have mongo service setup on localhost:27017 before but no longer available. Not sure how this worked

downgrade to 6.0.3 the test succeeded: #25369 indeed some breaking change in mongo 6.0.4. However, run the docker and test locally (mac M1) it succeeded.

error message on jenkins:
pymongo version: pymongo-3.13.0-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (516 kB)
13:42:38 pymongo.errors.ServerSelectionTimeoutError: localhost:27614: [Errno 111] Connection refused, Timeout: 30s, Topology Description: <TopologyDescription id: 63e29b7f096b207554610dd3, topology_type: Single, servers: [<ServerDescription ('localhost', 27614) server_type: Unknown, rtt: None, error=AutoReconnect('localhost:27614: [Errno 111] Connection refused')>]>
13:42:38


Fix deployment of generated pages with github pages

Currently, the pages are deployed but the index is the README and not the generated index.html as expected since #4

According to some research, github native build of jekyll websites doesn't accept every plug-ins. Thus, we need to build the pages locally and the deploy it.
References:

https://github.com/avillafiorita/jekyll-datapage_gen/issues/28

This may be a solution:
# .github/workflows/github-pages.yml
name: Build and deploy Jekyll site to GitHub Pages

on:
  push:
    branches:
      - main # or master before October 2020

jobs:
  github-pages:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2
      - uses: actions/cache@v2
        with:
          path: vendor/bundle
          key: ${{ runner.os }}-gems-${{ hashFiles('**/Gemfile') }}
          restore-keys: |
            ${{ runner.os }}-gems-
      - uses<EMAIL_ADDRESS>  # Choose any one of the Jekyll Actions
        with:                                # Some relative inputs of your action
          token: ${{ secrets.GITHUB_TOKEN }}


Engine builds failing on failure to find python

The failure is flaky. It looks like some machines have python and some don't.
Here's an example https://ci.chromium.org/ui/p/flutter/builders/prod/Linux Production Engine Drone/42854/overview
Marking P0 because the engine tree is closed, and rolls are blocked by failing presubmits.

Dart SDK -> Flutter engine auto-roller also failed because of this issue: https://github.com/flutter/engine/runs/10976153663. Linux, Mac and Windows bots are affected.

https://flutter-review.git.corp.google.com/c/recipes/+/38282


[Text Analytics] Implementing agreed-on renamings

This PR does a bunch of renamings that the crew agreed on, mainly:

use action instead of task
use operation instead of job
HealthcareEntityDataSource  -> EntityDataSource
EntityDataSource.id -> EntityDataSource.entityId
HealthcareResult -> AnalyzeHealthcareEntitiesResult and its variations
AnalyzePollerLike -> AnalyzeBatchActionsPollerLike
entityRecognitionTasks -> recognizeEntitiesActions, entityRecognitionPiiTasks -> recognizePiiEntitiesActions, keyPhraseExtractionTasks -> extractKeyPhrasesActions

@maririos I plan to change the representation of each action results in another PR.

/azp run js - ai-text-analytics - tests

/azp run js - ai-text-analytics - tests


Kustomize installation test failing

What happened?
The kustomize test have been failing for some time.

Steps to reproduce
No response
Relevant log output
The logs contains:

  No IntegrationPlatform resource in default namespace
  Camel K Integration Platform removed from namespace default
  Camel K Config Maps removed from namespace default
  Camel K Registry Secret removed from namespace default
  Camel K Platform Kamelets removed from namespace default
  Camel K Platform Camel Catalogs removed from namespace default
  Camel K Operator removed from namespace default
  Camel K Role Bindings removed from namespace default
  Camel K Role Bindings removed from namespace kube-public
  Camel K Roles removed from namespace default
  Camel K Service Accounts removed from namespace default
  Camel K Custom Resource Definitions removed from cluster
  Camel K Cluster Role Bindings removed from cluster
  Camel K Cluster Roles removed from cluster
  Using alternative make directory on path: /tmp/camel-k-105473497/install
      operator_test.go:51:
          No future change is possible.  Bailing out early after 0.000s.
          Expected
              <int>: 2
          to match exit code:
              <int>: 0
      test_util.go:67: Output from exec command:
          make[1]: Entering directory '/tmp/camel-k-105473497/install'
          make[1]: Leaving directory '/tmp/camel-k-105473497/install'

      test_util.go:68: Error from exec command:
          Makefile:181: *** *** CRD API FAILURE: ERROR: CRD API version 'v1' is not supported. ****.  Stop.

      dump.go:48: -------------------- start dumping namespace test-b1622017-46be-4dba-ba71-d3438fb4d9eb --------------------
      test_support.go:2742: Error while dumping namespace test-b1622017-46be-4dba-ba71-d3438fb4d9eb: the server could not find the requested resource (get integrations.camel.apache.org)
  Unable to retrieve the operator version: no matches for kind "IntegrationPlatform" in version "camel.apache.org/v1"
  Error: the server could not find the requested resource (get integrationplatforms.camel.apache.org)
  Camel K cluster setup completed successfully

Camel K version
main branch

@squakez I will take care of it.


Please Help!!!

Hello Sir,
Sorry, If I am asking a noob question. Actually your script is running very well with Python 3.6.4 installed on my system. But I am currently learning Python Programming, Due to which I have to install ANACONDA PYTHON on my system but whenever I install Anaconda on my system, Script starts to give following errror:
udemy-dl.py: error: the following arguments are required: course
With only python installed, script is running very well but with Anaconda above error starts to appear. This error only comes with Anaconda either it is stand alone or with python. Only way for me to run this script is stand alone python without Anaconda. PLEASE HELP ME TO RUN THIS SCRIPT WITH ANACONDA ALSO!!!
Thank You.

install docker to use Anaconda =)... no?

@tofanelli I have installed Docker but I don't know what to do next. Can't I run this script with docker? if yes, then please tell me the next steps. Is it necessary to run Anaconda with Docker to run this script?
If there is a simple way to run this script while Anaconda installed on system then please please tell me. extremely sorry if I am annoying you with these noob questions but please help if you can, to run this script in a simple way with anaconda installed.
Thank You.

@thedtech you are facing that error because course URL is positional argument so it need to be provide either way. i really didn't tested the script with Anaconda installation so i 'm sorry my brother i cannot help you with that. i would really appreciate if some else can.

@thedtech .... docker is a way to virtualize an OS inside your computer with some cool things... you will need go to docker website, find a container with python, go to your terminal, ask for download and install and then use it

@thedtech i 'm considering that the problem is solved for you and closing the issue but other can also suggest there solution to this issue.


rename discussion

I want to come up with a suitable name, now that vue3 has been out for so long, it can't always be called vue3-pixi
so let's discuss it here
There are currently several suitable options:

@vuepix/core (because @ vuepixi has already been registered)

@vue-pixi/core

To be honest, just call it whatever you think is cool. I don't think names are all that important.
I'd choose something along the lines of your name, like:
pixiry.js - a Lightweight and flexible Vue 3 library for creating PixiJS applications.
airy.js - a Lightweight and flexible Vue 3 library for creating PixiJS applications.
or something. The slogan or description should be simple and fitting, but it already perfect, so it's all good.
By the way, it would be cool if you set up a Discord to talk about this kind of stuff with people in general. I'd like to join there and talk with others or ask a few questions or discuss best practices

sure, thank you for your support, you are right

You can now join the discussion at https://discord.gg/mRguTPnsB9. I will add it to the website later. I'm not very familiar with Discord, so feel free to give me any suggestions you may have.


How can a user "demand" a trace to be sampled

Opening this as a placeholder for outstanding issue of force-sampled traces, aka "debug" flag.

[ ] need to have an ability to set this flag before the root span is started, since many tracers make sampling decisions upfront
[ ] need to agree on the name ("debug" or "force-sampling") and means of setting this flag

While vendors will implement their own sampler and trace data collectors, I personally think it would be great to have the spec define a common denominator or maybe just a "hint".
Right now, Sample: yes/no is done by samplers and then passed downstream, this is done so that the trace is consistent.
A better alternative could be to use something like trace-levelwhich indicates how much data a trace shall collect. 0 could be identical to sample = off, but other values like 5 could be basic trace data (spane attributes) and 10 extended trace data.

We do this by sending additional HTTP headers with the request that are evaluated during the creation of the first/root span, it then decides about no tracing, "production traffic tracing" and developer tracing. But this requires the information to be available inside createSpan()

@CodingFabian that sounds good.  How about tracePriorityHint with some defined range of integer values?
@beberlei agreed this interface would need to make the information available to the tracer at the time of createSpan call (or equivalent).
My vote would be as extended optional parameter to startTrace (or equivalent)?

(Quick preface: if there's a discussion on this elsewhere that I've missed, please point it out - I don't mean to cause rehashing of existing discussion!)
Is there a clear use case for why this needs to be part of the OpenTracing spec and not an implementation detail? I'm not saying there isn't -- I just would like to keep the spec as trim as possible if it can be made implementation-dependent (and still accomplish everything it needs to).  This sort of has the "feel" of a feature rather than part of the must-support spec to me.
Per the "set this flag before the root span is started" requirement, can this be done as part of the implementation-dependent initialization? That implementation specific config needs to happen somewhere, so can the flag / config option be up to the implementation to determine how to specify?
As for cross-process propagation of this configuration, I have been assuming that OpenTracing 1.0 supports only a system that using the same underlying tracing implementation in every process / service. This means that if a process configured with Tracing Implementation XYZ receives a HTTP request, it can assume Tracing Implementation XYZ (or no implementation) was being used when the HTTP request was sent -- and make implementation-specific assumptions about what is / is not going to be in the HTTP headers. In other words, how the flag is specified across processes again can be implementation dependent.
Apologies if I'm missing something obvious!

@bcronin No you are not missing an important point. my argument is that implementations need to do that anyways, so it would be great for interop to spec it. Your argument that it is not required for the spec is of course valid.
If interop is not a concern for 1.0, my argument is less valid. But my basic assumption is that somebody willing to go through the pain of agreeing on a spec has interop in mind.

To clarify, I'm in favor of deferring this until after 1.0 as mentioned on the chat this morning.  My hope is that by having a bit of discussion about interfaces in this thread, we can provide some perspective on interop for Tracer developers concerned with this topic such that Tracers will be more amenable to reificiation in API down the road.
Agree with @bcronin about ability to rely in implementation-specific for now, though because it may be request-specific it would have to happen potentially at each entry point of tracing, not just at Tracer initialization.  (This is the type of learning we'll get from some actual implementations before OT API.)

That implementation specific config needs to happen somewhere, so can the flag / config option be up to the implementation to determine how to specify?

The thing is, as an application developer, I should not be exposed to any tracer implementation specific details, that's the point of OpenTracing. From what I hear it's a fairly common practice to allow developers to have a way to specify that they want a particular trace sampled regardless of the tracer's default sampling policy. If I want to switch the tracing implementation for another one, I should not be forced to rewrite the app code that deals with 'debug'  traces.
Note that we're not talking about standardizing how the debug flag is specified in the request (headers or what not), that's the end application's business, but once the app decided it needs to set a debug flag, OpenTracing should have a way to do that in a vendor-neutral manner.

One problem with the current API differentiating between startTrace and joinTrace* is, that every service needs to implement the code deciding between start+join and potentially debug or not (levels/priorities) and do this in a way that is secure, so that user facing services cannot be DDoSed by users requesting traces with very high level of details.
Unless you pass the information to the Tracer outside of the OpenTracing API, there is little possibility to let the vendor make all this decisions. Which removes burden from developers to possibly re-implement this over and over again. It can also make it easier to enforce
I am wondering if startTrace(operationName, blobOfAttributes) (similar how the python api already allows to pass tags) would allow more flexibility for the vendor to decide on things.

I am wondering if startTrace(operationName, blobOfAttributes) (similar how the python api already allows to pass tags) would allow more flexibility for the vendor to decide on things.

One thing we've said a few times is that we have another ticket #22, which requires somewhat similar extension of the API to accept externally generated timestamps, so the solution to "debug" trace might be similar to what we come up with in #22.

@bcronin @CodingFabian the "interop", as in between different tracing systems, is a non-goal for OpenTracing. As such, this ticket is not trying to say anything at all about how the "debug" flag is propagated between processes, just like in general OpenTracing API has nothing to say about how span ID is propagated. By "how" I mean "in what form, on the wire", because OT does have vendor-neutral API to encode a span into intermediate representation (binary or text) that an RPC framework can use to include on the wire.

Would it be sufficient to decide on the canonical name of a trace annotation key which represents debug-ness? This reminds me of some of the const declarations we've talked about putting in an extensions namespace for various languages.
As for sequencing, I don't think this issue should block a 1.0 release, but I also don't mind discussing it in the meantime.

@yurishkuro you are right about seperating (de)serialization of context into another object, so maybe its not blobOfAttributes, but startTrace(operationName string, context map[string]string) and then something else is responsible to create the map of context.

@bensigelman yes, it could be sufficient provided that we have a way to set the tags ahead-of or during root span creation.
@beberlei I was thinking more like
func startTrace(operationName string) Span
func startTraceWithOptions(operationName string, options *Options) Span

type Options struct {
    context map[string]string
    // other stuff, in the future
}

With this approach, the forced sampling can be a proper attribute in the Options struct, rather than a tag

@bensigelman I realized that the problem may be simpler than I initially thought. I assumed that the tracer immediately needs to know that a trace is sampled, as otherwise it might already short-circuit storing certain events/tags. But in practice, when I implemented the tracer, it would work just fine as long as the "debug" flag is set immediately after creating the root span. That seems like a reasonable compromise for the end user, if you want to have everything recorded on a debug trace, you have to mark it as such asap.
Of course, if tracers choose to implement non-sampled traces with a completely different, less capable class like NoopSpan (as I think @tschottdorf was trying to), then it won't work, but that seems like the wrong approach for the tracer to take anyway, especially because no-op span cannot propagate TraceAttributes, which should be unaffected by the sampling.

That seems like a reasonable compromise for the end user

I agree.

Of course, if tracers choose to implement non-sampled traces with a completely different, less capable class like NoopSpan (as I think @tschottdorf was trying to), then it won't work, but that seems like the wrong approach for the tracer to take anyway, especially because no-op span cannot propagate TraceAttributes, which should be unaffected by the sampling.

What I want to avoid is putting log events into a trace that I know I'm not interested in. I have no trouble using the same Span for such traces, but I don't want every call to LogEvent to accumulate data into the Span when I know for a fact I won't use it. That and just not yet wanting to fork standardtracer (anticipating its change in the face of current interface discussions) are the reasons for using a separate NoopSpan currently, but I do see that as a temporary crutch. It's easy enough to flag a Span such that it   ignores LogEvent calls.

@tschottdorf mainly just FYI, I think standardtracer is a fine idea (almost as much for documentation purposes as coding purposes) but will almost certainly move out of opentracing-go in the near future... ultimately it's a distraction for callers of OT (who are the priority, ultimately).
Also, as long as StandardTracer knows about sampling, it would be easy enough to have non-sampled Span instances turn everything aside from codec tasks and perhaps Trace Attribute calls into noops...

Oh, @yurishkuro, good to know. I still like the idea of supporting tags at Span construction time as a base API, but nice to know it's not a requirement for any particular use case we have today.

I'm curious whether people think that Tags (a la #50) are an acceptable way to force sampling. It would literally "work", but part of me thinks that sampling should not need to be exposed in the Tags.

It seems slightly hacky but then upfront sampling may not make sense for
all tracer types. What's the alternative? Some Netflix
Method ShouldTrace on Span?
On Wed, Feb 10, 2016, 00:55 bhs<EMAIL_ADDRESS>wrote:

I'm curious whether people think that Tags (a la #50
https://github.com/opentracing/opentracing.github.io/pull/50) are an
acceptable way to force sampling. It would literally "work", but part of me
thinks that sampling should not need to be exposed in the Tags.
—
Reply to this email directly or view it on GitHub
https://github.com/opentracing/opentracing.github.io/issues/43#issuecomment-182209929
.

just read through all this.. a particularly interesting thread. Where before-the-fact sampling occurs (ex in zipkin tracers), knowing sampled state (or a range like proposed here) up front indeed gives a lot of implementation flexibility. If it is set via options, then implementations don't have to deal with mutability of that state later.
On the idea in general, the only thing that makes this api affecting is that the user is contributing to the sampling decision. Ex the user requests a certain level (likely to be most detail), and the internals would choose to honor that or not.
PS Zipkin's debug flag goes all the way to the collector tier, so we should certainly note strongly this is a request, not an assertion of a level.

With https://github.com/opentracing/opentracing-go/pull/50 the first checkbox is done.
For the naming, @bensigelman proposed this on gitter:

my two cents: the portable way to describe this is in terms of importance to the developer/operator of the system. so how about sampling.priority; implicit default value is 0. OT impls can do what they please with nonzero values; the naive thing would be to treat sampling.priority > 0 as “please sample this”.

Considering this closed by sampling.priority
https://github.com/opentracing/opentracing-go/blob/master/ext/tags.go#L25

Re-opening to fix typos in the doc and also add constant to Python/Go with the corresponding doc strings.

oops, clicking the last check-box auto-closed the issue

ps here's an image of "level" which was discussed at the distributed tracing workshop, particularly by @CodingFabian... similar to what was done here

@yurishkuro i think this issue can be closed, its implemented in go and python and is part of the specification on the page, even in examples.


Allow for standalone tables

We currently only define table types as immutable
views of data owned by table collections/tree sequences.
To support standalone tables:

Define a new type, OwningXTable for row type X.
impl std::ops::Deref for each new type.
The deref target is XTable.
The public API is add_row and add_row_with_metadata.
Probably also need copy function.

Thanks to @momolangenstein for suggesting the Deref pattern.

After a bit of hacking:

This is much harder than I'd thought!
The lifetime MutationTable<'a> is the culprit.
One path may be to try to get rid of that lifetime,
but then we probably want to refactor TableCollection
to ensure that lifetimes are properly coupled.

It is looking like this requires some significant
back end changes.

change metadata-related macros to work on pointers
to tables rather than on references.

change all table types to store pointers rather than reference.
This change allows dropping lifetime generics from the table types.

work out implications for code like this:
let m = tables.mutations();
drop(tables);
m.num_rows(); // currently doesn't compile due to lifetime on tables

See comments over in #265 re: a plan of action.

#265 has a working pattern now.
There is no need for a new type abstracting out "do we own the pointer or not".
Rather, standalone tables contain both an MBox and an instance of the immutable table type.
There are, however, several side effects that affect the back end:

Initialization order of the C types becomes tricky.
(This is not unexpected -- these are C structs w/many fields.)
Our current use of macros to "easily" build the rust
wrappers for types will break.
This is not a big deal -- the manual implementations are easy.
Our current table types do not allow initialization with
std::ptr::null().
To keep memory safety in place and to make the new code sane,
these tables should get a new pub(crate) new_null() -> Self
function that wraps a null pointer and a pub(crate) set_pointer(...)
to set the pointer as needed.

Yay, progress.

Here are some further design options:

// START OF PRELUDE

use std::ops::Deref;
use std::ptr::NonNull;

mod ll_bindings {
    pub struct tsk_edge_table_t {}
    pub struct tsk_table_collection_t {
        pub edges: tsk_edge_table_t,
    }
}
type MBox<T> = Box<T>;

struct TableCollection {
    inner: MBox<ll_bindings::tsk_table_collection_t>,
}

struct EdgeTable<'a> {
    table_: &'a ll_bindings::tsk_edge_table_t,
}

trait TableAccess {
    fn edges(&self) -> EdgeTable;
}

impl TableAccess for TableCollection {
    fn edges(&self) -> EdgeTable {
        EdgeTable::new_from_table(&(*self.inner).edges)
    }
}

impl<'a> EdgeTable<'a> {
    fn new_from_table(edges: &'a ll_bindings::tsk_edge_table_t) -> Self {
        EdgeTable { table_: edges }
    }
}

// END OF PRELUDE

// Option 1

struct OwnedEdgeTableV1 {
    edges: MBox<ll_bindings::tsk_edge_table_t>,
}

impl OwnedEdgeTableV1 {
    pub fn borrow(&self) -> EdgeTable<'_> { // returns EdgeTableV2<'a>
        EdgeTable::new_from_table(&self.edges)
    }
}

// Option 2

struct OwnedEdgeTableV2 {
    edges: MBox<ll_bindings::tsk_edge_table_t>,
}

#[repr(transparent)] // needed so the transmute on the newtype is safe
struct EdgeTableV2<'a> {
    table_: &'a ll_bindings::tsk_edge_table_t,
}

impl Deref for OwnedEdgeTableV2 {
    type Target = EdgeTableV2<'static>;

    fn deref(&self) -> &Self::Target { // returns &'a EdgeTableV2<'static>
        unsafe { std::mem::transmute(&self.edges) }
    }
}

// Option 3

#[repr(transparent)]
struct EdgeTableV3 {
    table_: NonNull<ll_bindings::tsk_edge_table_t>,
}

impl EdgeTableV3 {
    // this would be just for convenience to hide the unsafe
    /* private*/ fn as_ref(&self) -> &ll_bindings::tsk_edge_table_t {
        // Safety: only constructed with ref
        unsafe { self.table_.as_ref() }
    }

    pub(crate) fn ref_from_ref<'a>(edges: &'a ll_bindings::tsk_edge_table_t) -> &'a Self {
        // Safety: NonNull<T> has the layout of *mut T,
        //         matching the layout of *const T,
        //         matching the layout of &T
        unsafe { std::mem::transmute(edges) }
    }
}

struct OwnedEdgeTableV3 {
    edges: MBox<ll_bindings::tsk_edge_table_t>,
}

impl Deref for OwnedEdgeTableV3 {
    type Target = EdgeTableV3;

    fn deref(&self) -> &Self::Target { // returns &'_ EdgeTableV3
        EdgeTableV3::ref_from_ref(&self.edges)
    }
}

trait TableAccessV3 {
    fn edges(&self) -> &EdgeTableV3;
}

impl TableAccessV3 for TableCollection {
    fn edges(&self) -> &EdgeTableV3 { // returns &'_ EdgeTableV3
        EdgeTableV3::ref_from_ref(&(*self.inner).edges)
    }
}

Closed by:
#269
#278
#279
#282
#268
#267
#273
#281
#283
#284
#285
#286


Database seed for acceptance testing

Issue/Task Number: 393
Overview
This PR adds a new option to the mix seed command in order to generate the needed data to run acceptance tests
Changes

Add mix seed --test --yes command

Implementation Details

--test seeds 2 admins and a base account
--yes skips all prompts
Add a few environment variables to limit the use of this seed

Usage
To run the test seed in local, use:
E2E_ENABLED=true mix seed --test --yes

Note that I suggested -y, --yes, --assume_yes for my familiarity with apt-get arguments.

Would be nice if @sirn has a quick eye on this since this is his rewritten version of the seed, and it's to do with CLI.


Implement Trust Authorization and Update PermissionVerifier to check "network"

Hi @agunde406 Could you give a high level overview of what is happening in each of the handlers and why?

Descriptions added to Authorization handlers


【bug】what's wrong ? Can not upgrade stack version.

Upgrade version not support.

Hi,
Can you check the Ambari server logs.
Best regards,
Lucas


commons-math3-api-plugin

Repository URL
https://github.com/jonesbusy/commons-math3-api-plugin
New Repository Name
commons-math3-api-plugin
Description
Looking at https://github.com/search?q=org%3Ajenkinsci+"commons-math3"&type=code&p=1 some recent plugin bundle similar dependency
Worth to have an API plugin.
GitHub users to have commit permission
@jonesbusy
Jenkins project users to have release permission
jonesbusy
Issue tracker
GitHub issues

Security audit, information and commands
The security team is auditing all the hosting requests, to ensure a better security by default.
This message informs you that a Jenkins Security Scan was triggered on your repository.
It takes ~10 minutes to complete.
Commands
The bot will parse all comments, and it will check if any line start with a command.
Security team only:

/audit-ok => the audit is complete, the hosting can continue :tada:.
/audit-skip => the audit is not necessary, the hosting can continue :tada:.
/audit-findings => the audit reveals some issues that require corrections :pencil2:.

Anyone:

/request-security-scan => the findings from the Jenkins Security Scan were corrected, this command will re-scan your repository :mag:.
/audit-review => the findings from the audit were corrected, this command will ping the security team to review the findings :eyes:.
	It's only applicable when the previous audit required changes.

Only one command can be requested per comment.

(automatically generated message, version: 1.29.8)

The Jenkins Security Scan did not find anything dangerous with your plugin, congratulations! :tada:

:bulb: The Security team recommends that you are setting up the scan in your repository by following our guide.

Bom version is missing ${jenkins.baseline}
https://github.com/jonesbusy/commons-math3-api-plugin/blob/main/pom.xml#L45
should be bom-${jenkins.baseline}.x

/hosting host


Reconnects are buggy when performed in rapid succession

If you disable and enable Wifi/network rapidly, the reconnecting logic can get into a loop disconnecting/reconnecting.

closed with https://github.com/learn-co/atom-socket/pull/2


shader subroutine supported?

Hi,
Is subroutine supported in bgfx? If so, is there any examples of it? Thanks!
https://www.khronos.org/opengl/wiki/Shader_Subroutine

@adpa

@apda


Improve and redesign mouse Input Event

Mouse input Event
@pixelpicosean recently and Myself a couple of month ago both put effort in making trackpad support better.
This issue should sum up the ideas to discuss them. There are a couple of discussions in pr's which talk about the same topic but I though a dedicated place would be nice.
current implementation:
most mouse events are just "hacked" into the mouse button event. With custom mouse buttons like:
scrollUp /  ScrollDown / scrollLeft / scrollRight and recently Magnify. Precision input is than achieved by a float called factor which is set to the value how far the mouse was scrolled/magnified.
All this made sense When trackpads were simpler and mouses only used scroll wheels which had increments and fired events on each increment. So the wheel actually behaved like a button.
Current Issues:
the issue with this implementation is that it is hard to figure out what event to use for scrolling and magnifying, Because mouse button does not seem to be the most obvious thing (if you don't know about it)
In addition to that it does not fit at all into the new introduced class based event system. Each event type should be another class. not the same class with different types.
Proposal for redesign:
1. move different mouse events to classes
Events all of them Inherit From InpuEventMouse so you have access to position...
InputEvnetMouseScroll
    Vector2 direction
InputEvnetMouseGesturePinch
    float zoom
InputEvnetMouseGestureRotate
    float angle
InputEvnetMouseGestureSwipe
    Vector2i direction
    float speed
InputEventMouseButton
    int button
    bool pressed
...

This system is much more straight forward to use:
comparison:
example for moving node with a trackpad scroll
old:
if event is InputEventMouseButton:
    if event.buttonIndex == MOUSE_BUTTON_SCROLL_LEFT:
        node.position.x -= event.factor
    else If event.buttonIndex == MOUSE_BUTTON_SCROLL_RIGHT:
        node.position.x += event.factor
    if event.buttonIndex == MOUSE_BUTTON_SCROLL_UP:
        node.position.y -= event.factor
    else If event.buttonIndex == MOUSE_BUTTON_SCROLL_DOWN:
        node.position.y += event.factor

new:
if event is InputMouseScroll:
    node.position += event.direction

example for sizing node with a Pinch gesture
old:
if event is InputEventMouseButton:
    if event.button_index == MOUSE_BUTTON_GESTURE_PINCH:
        node.size += event.factor

new:
if event is InputEventMouseGesturePinch:
    node.size += event.zoom

2. Add device enum
in InputEventMouse:
enum Device{
    DEVICE_TRACKPAD
    DEVICE_MOUSE
    ... and maybe drawing table or touchscreen (although that is handled by compleatly different events)     or special apple mouse with touch ;)
}
InputEventMouse
    Device device

This allows to determine the input device and execute different actions for different devices.
example
scroll up in 3d could be panning or orbiting when it is emitted by a touchpad and zoom when it is emitted by a mouse
scroll down in 2d: panning when using DEVICE_TRACKPAD and zoom DEVICE_MOUSE
if event is InputEventMouseScroll:
   if event.device == DEVICE_MOUSE:
        viewport.zoom += event.direction.y
   else if event.device == DEVICE_TRACKPAD:
       vieport.scrollOffset += event.direction

To Discuss

of course as always everything (it's godot community, ppl like feedback ;) ).
naming: InputEvnetMouseGesturePinch or InputEvnetMousePinch

is the Gesture keyword necessary?

Is a swipe gesture actually beneficial or should it just be implemented with scroll left/right and some threshold logic (I love using swiping to switch scripts so in general that would be a nice feature. although I think it might be better to implement it as an signal listener for controls. Since every swipe also needs setup when it triggers...)
naming of direction, zoom, angle. What would you type in when trying to get the scroll value of a class named InputEventMouseScroll ?

the code I posed is more or less pseudocode. I don't event know if I used correct class and constant names... (pls don't judge)

reference: - (void)scrollWheel:(NSEvent *)theEvent     {        if(([theEvent momentumPhase] != NSEventPhaseNone) || [theEvent phase] != NSEventPhaseNone))        {           //theEvent is from trackpad                   }        else         {          //theEvent is from mouse        }     }
to figure out if the input is coming from mouse or trackpad on OSX

#36953 is adding a more extensive input gestures system. If that pull request is merged, we may consider this proposal fulfilled.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Feat: getAddress() to Offer Use-Case

Pull request related to this trello task

:tada: This PR is included in version 1.1.0 :tada:
Your semantic-release bot :package::rocket: