Update extensionsupport.go

Some CA software issue CRLs with the issuing distribution point set to critical. As such, this breaks the plugin. This pull request adds the respective OID to the exclusion list. This also falls inline with what is written in comments.

Unfortunately, it appears I can't verify if the patch works or not, other than going off what I can read from the source code, since xcaddy/go mod download keeps downgrading the version from the patch version to v1.4.0, despite the source code replacement.
Bluddy brilliant software, Google!!!

Hi i can not merge this in current state. In general critical extensions need to be processed if they can not be processeed it needs to fail. So the current implementation is correct as IDP is not handled.
However for IDP there is some exception defined in the RFC
https://datatracker.ietf.org/doc/html/rfc5280#section-5.2.5
Although the extension is    critical, conforming implementations are not required to support this    extension.  However, implementations that do not support this    extension MUST either treat the status of any certificate not listed    on this CRL as unknown or locate another CRL that does not contain    any unrecognized critical extensions.
So in that case still the logic needs to be implemented what should happen with certificates in "unknown" state and also to detect that a certificate has "unknown state". Palo alto implemented it this way that you have an option to disallow unknown certificates. Like you can see here:
https://knowledgebase.paloaltonetworks.com/KCSArticleDetail?id=kA10g000000CldJCAS
So similar implementation would be at minimum needed. You change basically suggest to ignore the unhandled extension which makes it non RFC conform.
It would be interesting to know in which way the IDP is used in your CA.

Yes, it could be processed, but this makes it inherently redundant, as it is possible that we could have retrieved the CRL from that IDP to begin with as part of the verification process.
Regardless, I could make further changes - such as adding the needed logic to check those IDPs - but that will have be done around the middle of next month at the very latest, when I have my server up and running, as well as the needed environment for Golang.
14 Feb 2024 11:28:14 Gr33nbl00d @.***>:

Hi i can not merge this in current state. In general critical extensions need to be processed if they can not be processeed it needs to fail. So the current implementation is correct as IDP is not handled.
However for IDP there is some exception defined in the RFC
https://datatracker.ietf.org/doc/html/rfc5280#section-5.2.5
Although the extension is critical, conforming implementations are not required to support this extension. However, implementations that do not support this extension MUST either treat the status of any certificate not listed on this CRL as unknown or locate another CRL that does not contain any unrecognized critical extensions.
So in that case still the logic needs to be implemented what should happen with certificates in "unknown" state and also to detect that a certificate has "unknown state". Palo alto implemented it this way that you have an option to disallow unknown certificates. Like you can see here:
https://knowledgebase.paloaltonetworks.com/KCSArticleDetail?id=kA10g000000CldJCAS
So similar implementation would be at minimum needed. You change basically suggest to ignore the unhandled extension which makes it non RFC conform.
It would be interesting to know in which way the IDP is used in your CA.
—
Reply to this email directly, view it on GitHub[https://github.com/Gr33nbl00d/caddy-revocation-validator/pull/12#issuecomment-1943579141], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AD7G5GDM7X3HPBPA3VC5WILYTSNU5AVCNFSM6AAAAABDCYT42GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBTGU3TSMJUGE].
You are receiving this because you authored the thread.
[Tracking image][https://github.com/notifications/beacon/AD7G5GBUFH4VUEMHCS73CADYTSNU5A5CNFSM6AAAAABDCYT42GWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTT3CVAK.gif]

The main problem with this extension is: It is used for multiple things. First it is used to locate the crl and also define alternative download solutions (mirrors). But it can also be used to split a single crl in multiple crls. For example seperate crls for CA revocations and for end entity revocations. Or split by revocation reason. To have some reasons go to one crl and the other to another crl.
This makes it quite complicated to implement. I guess thats the reason why it is not implemented fully in most implementations and thats also the reason why an "exception" is stated in handling this critical extension. Which would allow a partial implementation.
Maybe we could support crls with indirect flag set to false and without restrictions on reason code/attribute and so on. So basically support:
onlyContainsUserCerts      false
onlyContainsCACerts        false.
onlySomeReasons            "NOT SET"
indirectCRL                        false
onlyContainsAttributeCerts  false
I guess your ca does only use the extension to define download location of crl. It does not use the "splitting" functionality.


How to update every 0.1 s / 100ms

Yeah so how can I update the server every 0.1s or 100ms?

https://github.com/freyacodes/Lavalink/blob/eee27d4170c92abaa502100dc2c7582b168ee082/LavalinkServer/application.yml.example#L17

I tried it and it is every 5 seconds. I am trying to update the clients every 0.1 s

Just try setting the interval to 0.1?

Just try setting the interval to 1s? (1 second is the lowest)

No. didn't work I get an error when I pit 0.1 or 0,1

Just try setting the interval to 1s? (1 second is the lowest)

No. didn't work I get an error when I put 0.1 or 0,1

1 second?
playerUpdateInterval: 1 # How frequently to send player updates to clients, in seconds

1 second is too high I need 0.1 or 0.2s
Moquent @.***> schrieb am Mi., 22. Sept. 2021, 04:51:

Just try setting the interval to 1s? (1 second is the lowest)
No. didn't work I get an error when I put 0.1 or 0,1
1 second?
playerUpdateInterval: 1 # How frequently to send player updates to clients, in seconds
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/freyacodes/Lavalink/issues/522#issuecomment-924538293,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APVQKCUR6LT6ID4YLY3BUDDUDFACFANCNFSM5EO6PZ4Q
.

I think you're making a poor design choice, instead of making lavalink spam you with updates you should track the player state locally (increment the player timer locally every n milliseconds and correct it with update events sent by lavalink)


WARNING: unable to extract channel id

Checklist

[ X ] I'm reporting a broken site support issue
[ X ] I've verified that I'm running youtube-dl version 2019.04.30
[ X ] I've checked that all provided URLs are alive and playable in a browser
[ X ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ X ] I've searched the bugtracker for similar bug reports including closed ones
[ X ] I've read bugs section in FAQ

Verbose log

[debug] System config: [] [debug] User config: [] [debug] Custom config: [] [debug] Command-line args: [u'--verbose', u'-a', u'/home/trekjunky/Documents/downloadlist'] [debug] Batch file urls: [u'https://youtu.be/pSthYb85Jeo', u'https://youtu.be/21GH0QuKF-I', u'https://youtu.be/xgwWkwOac-Y', u'https://youtu.be/Lqi8pGR6yLM', u'https://youtu.be/wq_yGaTUp24'] [debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8 [debug] youtube-dl version 2019.04.30 [debug] Python version 2.7.14 (CPython) - Linux-4.12.14-lp150.12.58-default-x86_64-with-glibc2.2.5 [debug] exe versions: ffmpeg 3.4.4, ffprobe 3.4.4 [debug] Proxy map: {} [youtube] pSthYb85Jeo: Downloading webpage [youtube] pSthYb85Jeo: Downloading video info webpage WARNING: unable to extract channel id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output. [debug] Default format spec: bestvideo+bestaudio/best WARNING: Requested formats are incompatible for merge and will be merged into mkv. [debug] Invoking downloader on u'https://r3---sn-vgqsrnez.googlevideo.com/videoplayback?id=o-AKn9my16mpBH7VpZX-qUBxqdJJuBdyVgL1oYPv-O65EZ&aitags=133%2C134%2C135%2C136%2C137%2C160%2C242%2C243%2C244%2C247%2C248%2C278&itag=137&source=youtube&requiressl=yes&mm=31%2C26&mn=sn-vgqsrnez%2Csn-p5qlsnzd&ms=au%2Conr&mv=m&pl=41&ei=NR7RXNSWD8POwQGYrIj4Dw&initcwndbps=2688750&mime=video%2Fmp4&gir=yes&clen=224656875&dur=588.045&lmt=1557171119712418&mt=1557208578&fvip=6&keepalive=yes&c=WEB&txp=4432432&ip=2601%3A408%3A780%3A2e25%3A%3Ab73d&ipbits=0&expire=1557230229&sparams=ip%2Cipbits%2Cexpire%2Cid%2Caitags%2Csource%2Crequiressl%2Cmm%2Cmn%2Cms%2Cmv%2Cpl%2Cei%2Cinitcwndbps%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&signature=843B30A3EE3577D2AC1C2718866B78F55B3D43BA.E22E84526E6CEB0F59C4469D6D8CF31C3B536542&key=yt8&ratebypass=yes'

Description

When beginning to download the video, there is a warning about extracting the channel ID and then goes to convert the video to mkv format

I see the same.
git clone https://github.com/ytdl-org/youtube-dl
cd youtube-dl
PYTHONPATH=. python3 -m youtube_dl 'https://www.youtube.com/user/chilledsanity/videos'

shows
[youtube:user] chilledsanity: Downloading channel page
[youtube:playlist] UUJ6KZTTnkE-s2XFJJmoTAkw: Downloading webpage
[download] Downloading playlist: Uploads from Accursed Farms
[youtube:playlist] UUJ6KZTTnkE-s2XFJJmoTAkw: Downloading page #1
[youtube:playlist] playlist Uploads from Accursed Farms: Downloading 190 videos
[download] Downloading video 1 of 190
[youtube] UKsH7xklMr4: Downloading webpage
[youtube] UKsH7xklMr4: Downloading video info webpage
WARNING: unable to extract channel id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
[download] Destination: May Videochat 2019-UKsH7xklMr4.mp4
[download]  17.3% of 1.11GiB at 64.32MiB/s ETA 00:14^C
ERROR: Interrupted by user

I tried with the long version of the URL which I copied from the Browser's address bar and got this:
~/bin> youtube-dl -v https://www.youtube.com/watch?v=N5xyr1_DhpA [debug] System config: [] [debug] User config: [] [debug] Custom config: [] [debug] Command-line args: [u'-v', u'https://www.youtube.com/watch?v=N5xyr1_DhpA'] [debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8 [debug] youtube-dl version 2019.04.30 [debug] Python version 2.7.14 (CPython) - Linux-4.12.14-lp150.12.58-default-x86_64-with-glibc2.2.5 [debug] exe versions: ffmpeg 3.4.4, ffprobe 3.4.4 [debug] Proxy map: {} [youtube] N5xyr1_DhpA: Downloading webpage [youtube] N5xyr1_DhpA: Downloading video info webpage WARNING: unable to extract channel id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output. [debug] Default format spec: bestvideo+bestaudio/best WARNING: Requested formats are incompatible for merge and will be merged into mkv. [debug] Invoking downloader on u'https://r6---sn-vgqsknee.googlevideo.com/videoplayback?id=o-ALenPzuo5tbOEVkStn6OGKFHuahPDwpA_e1zok7aZCpd&aitags=133%2C134%2C135%2C136%2C137%2C160%2C242%2C243%2C244%2C247%2C248%2C278&itag=137&source=youtube&requiressl=yes&mm=31%2C26&mn=sn-vgqsknee%2Csn-p5qs7nee&ms=au%2Conr&mv=m&pl=41&ei=9STRXM2eEPGNir4P_5aH6Aw&initcwndbps=1932500&mime=video%2Fmp4&gir=yes&clen=214099177&dur=1087.933&lmt=1557198256881382&mt=1557210271&fvip=6&keepalive=yes&c=WEB&txp=5535432&ip=2601%3A408%3A780%3A2e25%3A%3Ab73d&ipbits=0&expire=1557231957&sparams=ip%2Cipbits%2Cexpire%2Cid%2Caitags%2Csource%2Crequiressl%2Cmm%2Cmn%2Cms%2Cmv%2Cpl%2Cei%2Cinitcwndbps%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&signature=8A6E37335CE566E4687C604C4A89EF74E0F47BB4.B498E65B5E1DAAD0E292993493D74CE702D93C7C&key=yt8&ratebypass=yes' [download] Destination: Embassy Protectors Assaulted (Plus Viewer Questions)-N5xyr1_DhpA.f137.mp4 [download] 100% of 204.18MiB in 01:07 [debug] Invoking downloader on u'https://r6---sn-vgqsknee.googlevideo.com/videoplayback?id=o-ALenPzuo5tbOEVkStn6OGKFHuahPDwpA_e1zok7aZCpd&itag=251&source=youtube&requiressl=yes&mm=31%2C26&mn=sn-vgqsknee%2Csn-p5qs7nee&ms=au%2Conr&mv=m&pl=41&ei=9STRXM2eEPGNir4P_5aH6Aw&initcwndbps=1932500&mime=audio%2Fwebm&gir=yes&clen=17305644&dur=1087.961&lmt=1557197357248102&mt=1557210271&fvip=6&keepalive=yes&c=WEB&txp=5531432&ip=2601%3A408%3A780%3A2e25%3A%3Ab73d&ipbits=0&expire=1557231957&sparams=ip%2Cipbits%2Cexpire%2Cid%2Citag%2Csource%2Crequiressl%2Cmm%2Cmn%2Cms%2Cmv%2Cpl%2Cei%2Cinitcwndbps%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&signature=EE3AEE3D91FFB51F91168E3E244D13FD35C3F2F8.C3CAEFD0FE8D267BB15AF7F9A39F3F752124AB8E&key=yt8&ratebypass=yes' [download] Destination: Embassy Protectors Assaulted (Plus Viewer Questions)-N5xyr1_DhpA.f251.webm [download] 100% of 16.50MiB in 00:05 [ffmpeg] Merging formats into "Embassy Protectors Assaulted (Plus Viewer Questions)-N5xyr1_DhpA.mkv" [debug] ffmpeg command line: ffmpeg -y -loglevel 'repeat+info' -i 'file:Embassy Protectors Assaulted (Plus Viewer Questions)-N5xyr1_DhpA.f137.mp4' -i 'file:Embassy Protectors Assaulted (Plus Viewer Questions)-N5xyr1_DhpA.f251.webm' -c copy -map '0:v:0' -map '1:a:0' 'file:Embassy Protectors Assaulted (Plus Viewer Questions)-N5xyr1_DhpA.temp.mkv' Deleting original file Embassy Protectors Assaulted (Plus Viewer Questions)-N5xyr1_DhpA.f137.mp4 (pass -k to keep) Deleting original file Embassy Protectors Assaulted (Plus Viewer Questions)-N5xyr1_DhpA.f251.webm (pass -k to keep)

same problem here


Install location requires sudo

Brief Issue Summary
I would like to install my project to /usr/local using the install CMake target. On my (Mac) system this requires sudo permissions. Instead of asking for permissions, CMake Tools simply fails. I can run sudo ninja install from the command-line. However, if CMake has to build a binary during this step, it will be written with the owner set to root. Then, if I make another target within CMake Tools, e.g. all, it will fail because it cannot overwrite the file.
Expected:
Somehow CMake Tools asks for my password so that I can install to /usr/local.
Other Notes/Information
I admit this may be a difficult problem to solve well. The reason I would like to install to /usr/local is that I use BATS as my test framework, and since Apple added System Integrity Protection (SIP) to Mac OS X, child processes no longer receive the DYLD_LIBRARY_PATH variable in their environment. This means my BATS tests cannot find my shared libraries, unless they are installed to a system path such as /usr/local 😡
If anyone has a better suggestion as a workaround, I am all ears.

Recently code introduced ability to save files as administrator. Wonder if could implement similar feature on all platforms.
Only issue though is to detect when it is needed...

Yup, that's exactly the problem I thought of once I'd had a while to think. Do not consider this high priority! I have found another work-around (re-sourcing .bash_profile in my BATS tests).

Any update on this feature request?

We were not able to get to this yet but we would happily review and test a PR from the community, if anyone is willing to spend the time to implement this.

Any updates on this feature? I almost always install to a location that requires sudo privileges

would like to see this also.

I imagine that this feature request is somewhat related to: https://github.com/microsoft/vscode-cmake-tools/issues/532

It looks like https://github.com/microsoft/vscode-cmake-tools/issues/532 is implemented, which makes this issue even more relevant. @gcampbell-msft Would you consider triaging this issue now?

@Zingam Thanks for following up.
As a workaround, are you able to simply open your project in VS Code as a sudo user?


Feature request: allow non-persistent attributes or/and stores

First, thanks for this library, it's a very nice and easy to use alternative to more traditional options! 👍
I'm submitting a ...

[ ] bug report
[x] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary
I'd like to have a way to mark either a store or an attribute of a state non-persistent so that it would always be based on the provided initial value. Currently for example something like appInitReady: false which gets toggled to true in init() action is currently stored as true on disk and the logic needs to be rewritten in order to work w/ tauri-pinia. There are many cases where empty / predefined initial values would be useful.
Initial ideas:

allow marking non-persistent attributes in config (something in the lines of await tauriPinia({ excludedAttrs: ['debug', 'isInited', 'createdAt'] }))
offer an api to mark store not to be persisted (in config or maybe easier via a state attribute like dontPersist: true)

New version released, see changelog.
Sorry for the wait, lots of things to do.
Should you have any more ideas, will be happy to implement them.

This is open source, you shouldn't stress about self-imposed schedules. The feature looks great, Big Thanks for the addition! 🙏


Can we stop AWS from further usage if it goes beyond (ex: 1000$) X amount

I'm afraid that this request is out of scope for git-secrets. AWS has published documentation about billing alerts using CloudWatch.


Revert "Don't require xcodegen settings preset files. All configuration should be (#84)"

This reverts commit ed81cc27408a06f230db649950a16f41c3f0d2ec.
This commit caused build errors in Pinterest that weren't tested in the
OSS CI. This PR is simply documentation of the issue.
The outcome of addresssing this PR is to understand why it happened,
and finding the best solution to address it.

I'm not sure if we should merge this commit, although, internally we're going to be running on this branch until the issue is resolved.

What’s the actual error?

Hey Rahul, I can help testing / look into it more soon. Here’s the steps reproduce:

pull XCHammer
run make release, and install the app
build the iOS app
the build errors catch divergence in compiler settings

I’d suggest implementing a more rigorous test system ( build validation ) as a follow up to prevent regressions like this going forward.

@jerrymarino interesting, lets investigate tomorrow

Closing this out as we have an intrim solution for xcconfigs. Long term this will need more effort and thinking


FR: Hide "newer version installed" extensions

When looking at the list of available extensions, there are a bunch that can show as "newer version installed."  Since these are installed, they should not show up on the available list, especially since they can't be downgraded/removed from that list.

wrong repo


Add remark about required Analyzer features for PHRASE() & NGRAM_MATCH()

Added important note for analyzers to be used with NGRAM_MATCH  and PHRASE functions

@Dronplane Does it return incorrect results if the Analyzer doesn't have the features enabled, or does it raise an error?

It just won't find anything


Seperate edge M and edge N

edge N still use SVW settings

Tests passed, the failed tests are docker issues.


Inconsistent import behaviour of experimental modin.pandas

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): any
Modin version (modin.__version__): 0.8.1
Python version: any
Code we can use to reproduce:

import os
os.environ['MODIN_EXPERIMENTAL'] = 'True'

import modin.pandas as pd
print(f'pd.read_sql from: {pd.read_sql.__module__}')

import modin.experimental.pandas as pd
print(f'pd.read_sql from: {pd.read_sql.__module__}')

Describe the problem

Reproducer output:
pd.read_sql from: modin.pandas.io
UserWarning: Thank you for using the Modin Experimental pandas API.
Please note that some of these APIs deviate from pandas in order to provide improved performance.
pd.read_sql from: modin.experimental.pandas.io_exp

Expected behaviour: pd.read_sql should be from experimental IO in both cases.

Outdated


Failed to set global recorder: SetRecorderError { .. } when using axum-test

First and foremost thanks for the nice crate. It works as expected.
The issue I'm reporting is only concerning my integration tests which output the following error: /.cargo/registry/src/index.crates.io-6f17d22bba15001f/axum-prometheus-0.7.0/src/lib.rs:852:48: Failed to set global recorder: SetRecorderError { .. }.
Is there a way to disable setting the recorder for testing or another recommended way to resolve this?

Something similar came up recently in https://github.com/Ptrskay3/axum-prometheus/issues/59. What do you think about the solutions I mentioned there? (Let's note that BaseMetricLayer is available since 0.7.0)

Since I don't need metrics in tests I wouldn't mind removing that layer for the test but I don't know an elegant way to do this. I tried implementing BaseMetricLayer via this example. However I can only see my custom metrics and not the predefined ones from this crate. Is it not possible to use both?

My code:
use axum_prometheus::{metrics_exporter_prometheus::PrometheusBuilder, BaseMetricLayer};
use metrics_util::MetricKindMask;
use std::net::{IpAddr, Ipv4Addr, SocketAddr};
use std::time::Duration;

pub struct Metric {
    pub name: &'static str,
    pub description: &'static str,
}

pub const CONCURRENT_USERS: Metric = Metric {
    name: "concurrent_users",
    description: "This metric tracks the number of concurrent users, defined as the number of session that made a request in the last five minutes",
};

pub const GAUGES: [Metric; 1] = [CONCURRENT_USERS];

pub fn metrics(port: &u16) {
    println!("initializing metrics exporter");

    PrometheusBuilder::new()
        .idle_timeout(
            MetricKindMask::COUNTER | MetricKindMask::HISTOGRAM,
            Some(Duration::from_secs(10)),
        )
        .with_http_listener(SocketAddr::new(
            IpAddr::V4(Ipv4Addr::new(0, 0, 0, 0)),
            port.to_owned(),
        ))
        .install()
        .expect("failed to install Prometheus recorder");

    for name in GAUGES {
        register_gauge(name)
    }
}

let prometheus_layer = utils::create::metrics(&3002);

 let app = configure_router(app_state.clone())
        .with_state(app_state)
        .route("/fast", get(|| async {}))
        .route(
            "/slow",
            get(|| async {
                tokio::time::sleep(std::time::Duration::from_secs(1)).await;
            }),
        )
        .layer(prometheus_layer)
        .layer(session_manager_layer)
        .layer(TraceLayer::new_for_http())
        .layer(localisation_service)
        .layer(axum::extract::DefaultBodyLimit::max(
            constants::MULTIPART_SIZE_LIMIT,
        ))
        .nest_service("/assets", get_service(ServeDir::new("assets")))
        .nest_service("/style", get_service(ServeDir::new("style")))
        .nest_service("/scripts", get_service(ServeDir::new("scripts")))
        .layer(BaseMetricLayer::new());

Is it not possible to use both?

It's definitely possible.
From first glance in that example I see both
let prometheus_layer = utils::create::metrics(&3002);
// cut ..
.layer(prometheus_layer)

and
 .layer(BaseMetricLayer::new());

I don't know what utils::create::metrics is doing, but this look suspicious. If you use BaseMetricLayer, you don't need to add any other layers related to this crate.
In case this doesn't fix the issue, then unfortunately I can't really tell what else may be wrong without creating a reproducible example myself then trying it. I think I'll definitely invest some time into improving the testing experience, because from the recent issues it semms this crate doesn't have a great story when it comes to testing.

Thank you very much for this personalized example! Now I can see all metrics as expected but am still experiencing some issues with tests. Even tho nothing is running on the specified ports (3000 for server, 3002 for metrics) it's outputting that the address is already in use. Any idea on how to resolve this?

I fixed it by re-reading the other issue you referenced before and implementing INIT.call_once. Thanks for the help!
Here is my init function for future reference:
pub fn init(port: &u16) {
    INIT.call_once(|| {
        PrometheusBuilder::new()
            .with_http_listener(SocketAddr::new(
                IpAddr::V4(Ipv4Addr::new(0, 0, 0, 0)),
                port.to_owned(),
            ))
            .install()
            .expect("failed to install Prometheus recorder");

        for name in &GAUGES {
            register_gauge(name);
        }
    });
}

Great! I'll open an issue regardless, since I think enough people complained about this before. #65


Fix errors in README

If an arrow function is turned into a block it does not implicitly return as is the case when the arrow function is not a block. Using the old example code in the README would result in a DOM level error originating at partial-observation-widget.js#L158 caused by an attempt to insert a null.
I've also fixed a minor bug caused by not calling .initialize, as well as a small typo in a comment.

Thanks a lot!


The length of the table changed while iterating over it

#v0.19: {200: "a lot", 1: "one", 2: "two", 3: "three", 20: "a lot"}
#v0.20: Error: unhandled exception: the length of the table changed while iterating over it [AssertionError]
https://github.com/coffeepots/odbc/blob/9af9e7388153972515349becb4c6c378f99e2d8b/odbc/odbcconnections.nim#L56

@mprkins I've been trying to reproduce this with no luck. When does it happen, is it at program exit or when running? The line is called by the GC finaliser for connections and these are kind of an old way of doing things in Nim now. Are you running queries on different threads?

@coffeepots
hmm I dont think I am doing anything special.

Nim Compiler Version 1.0.2 [Windows: amd64]
Compiled at 2019-10-13
Copyright (c) 2006-2019 by Andreas Rumpf
git hash: f16bb4d85b724f03b41c4dd104b439fa96c2fda4
active boot switches: -d:release

code to reproduce
`import ./odbc/odbc, json, strformat
var
con = newODBCConnection()
qry = newQuery(con)
con.driver = "SQL Server Native Client 11.0"
con.host = r""
con.port = 1433
con.database = ""
con.integratedSecurity = false
con.userName = ""
con.password = ""
qry.statement = fmt"select * from version"
if not con.connect:
echo "Cant connect"
var results = qry.executeFetch
echo results.toJson.pretty
`

C:\nim_code\TestCode>test_connector.exe
C:\nim_code\TestCode\test_connector.nim(72) test_connector
C:\nim_code\TestCode\odbc\odbc.nim(303) executeFetch
C:\nim_code\TestCode\odbc\odbc.nim(183) fetch
C:\nim_code\TestCode\odbc\odbc.nim(165) fetchRow
C:\nim_code\TestCode\odbc\odbc\odbctypes.nim(172) initSQLData
C:\Nim\lib\system\assign.nim(243) FieldDiscriminantCheck
C:\Nim\lib\system\fatal.nim(39) sysFatal
Error: unhandled exception: assignment to discriminant changes object branch; compile with -d:nimOldCaseObjects for a transition period [FieldError]
C:\nim_code\TestCode\test_connector.nim(72) test_connector
C:\nim_code\TestCode\odbc\odbc.nim(307) executeFetch
C:\Nim\lib\pure\collections\tables.nim(669) finaliseConnections
C:\Nim\lib\system\assertions.nim(27) failedAssertImpl
C:\Nim\lib\system\assertions.nim(20) raiseAssert
C:\Nim\lib\system\fatal.nim(39) sysFatal
Error: unhandled exception: C:\Nim\lib\pure\collections\tables.nim(669, 13) len(t) ==     L the length of the table changed while iterating over it [AssertionError]
with the -d:nimOldCaseObjects switch i get the same error though

Ah I think this has been fixed recently, initSQLData should now look like this.
If you fetch the latest version is the issue still present? Your example run to the point of reporting that it can't connect for me.

Thanks a lot!!!
I did git clone again the odbc repo and it seems okay now.
Much appreciated.


docs: CRD Definitions (API Documentation)

Is there documentation for the CRD's such as the PrometheusRule?
I can't see to find a definitive guide to what fields are available, valid values, and behaviors.
In the readme (https://github.com/coreos/prometheus-operator?ts=4) it says:

To learn more about the CRDs introduced by the Prometheus Operator have a look at the design doc.

The design doc does not describe the CRD's though
https://github.com/coreos/prometheus-operator/blob/master/Documentation/design.md?ts=4

I suspect there are a couple possible outcomes:

Documentation does exist, and so the action for this issue would be to update relevant README's to provide links

or

Documentation does not exist, so the action would be to create them and update relevant README's to provide links

Here :) https://github.com/coreos/prometheus-operator/blob/master/Documentation/api.md
What do you think we could have done differently to make this easier to find?

Documentation does exist, and so the action for this issue would be to update relevant README's to provide links

If you have any suggestion, please contribute them! :)


arch-install-scripts: init at 24

Motivation for this change
arch-chroot from the Arch Install Scripts is a handy tool to quickly chroot into another Linux Installation.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg build arch-install-scripts


Transitive dependency pinning

Would it be possible to implement transitive dependency pinning, something similar to proposed NuGet implementation?

Would it be possible to implement transitive dependency pinning, something similar to proposed NuGet implementation?

I can only second that request.
We are stuck with dotnet 5.0.1XX because transitive pinning got turned off without any alternative for reactivation.
 CPVM was THE solution for us, and we were overjoyed to see this feature, but now, in its current state, it's pretty useless to us (again).

Hi @jeffkl, thanks for your reply. I think there is a misunderstanding, the implementation for CPVM in dotnet 5.0.1XX (nuget 5.8) was working perfectly well for us. It just got destroyed in nuget 5.9 with no way to get it back.
Before 5.9, you could write a package version into Directory.Packages.props and it would be respected in all projects below, also if that package was "just" a transitive dependency in the project at hand.
From 5.9 onwards, this project will restore any version it want of that package, until you manually add the transitive dependency as a "fake" direct dependency.
So, yes, there was no auto-magic to resolve e.g. "Newtonsoft.Json" to the same version everywhere, but once you specified its version in the central file "manually", it would resolve to that version everywhere.
Internally, as a build test if you like, we parse all restored projects via "dotnet list packge --include-transitive" and fail if the resolved versions diverge for any package. Before 5.9, I could then just go and pin the version and be done with it.
In 5.9 (and later), I would also have to make it a direct dependency everywhere where it's currently used implicitly. And that is basically unmaintainable.

Again, Microsoft.Build.CentralPackageVersions is only specifying versions of top-level/non-transitive dependencies with some fancy MSBuild logic and implementing enforcement to ensure users don't try to add versions to PackageReference items in projects.  But until NuGet itself considers versions for packages that aren't at the top level, then Microsoft.Build.CentralPackageVersions can't inject this logic.  It could look up the transitive closure on the fly and inject the explicit reference but that would be very slow.
We'll need to wait until NuGet understands out of the box that a particular version should be used transitively.

@lazy8 and @jeffkl  For clarity, Transitive Pinning was a feature in NuGet for a while, but it was removed, which will be why it worked in NuGet 5.8 but not in 5.9 onwards.
https://github.com/NuGet/Home/issues/10132
https://github.com/NuGet/NuGet.Client/pull/3719
https://github.com/NuGet/Home/issues/10389
https://github.com/NuGet/NuGet.Client/pull/4025

Closing as won't fix since this SDK can't control the dependency graph created by NuGet.  The built-in central package version management in NuGet might support this in future release.

@avivanoff

since NuGet now supports transitive pinning, what do we need to do to take advantage of it while using Microsoft.Build.CentralPackageVersions?

We have no plans to support transitive pinning in Microsoft.Build.CentralPackageVersions.  Since shipping Microsoft.Build.CentralPackageVersions I've actually joined the NuGet team and am working on the built-in central package management.  My plan is to deprecate Microsoft.Build.CentralPackageVersions in favor of that implementation instead and hopefully have time to write a migration tool.


Enable callbacks from sklearn interface.

Simple patch to enable mid-train callbacks to be used in XGBRegressor and XGBClassifier.

Codecov Report

Merging #2650 into master will increase coverage by 0.12%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2650      +/-   ##
==========================================
+ Coverage   37.64%   37.77%   +0.12%
==========================================
  Files          82       82
  Lines        7254     7254
  Branches      710      710
==========================================
+ Hits         2731     2740       +9
  Misses       4385     4385
+ Partials      138      129       -9

Impacted Files
Coverage Δ

src/data/simple_dmatrix.cc
57.33% <0%> (+0.66%)
:arrow_up:

src/data/sparse_page_dmatrix.cc
88.52% <0%> (+1.63%)
:arrow_up:

src/metric/multiclass_metric.cc
82% <0%> (+2%)
:arrow_up:

src/objective/regression_obj.cc
89.07% <0%> (+2.18%)
:arrow_up:

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 39adba5...f387b46. Read the comment docs.

Added a new commit to fix tests failures when sklearn not available. Please let me know if this should be squashed to one commit instead.

Note sure how to fix the codecov/project test failure - as far as I can see all the extra lines I added are covered by tests. Any advice welcome...

this code need to be rebased against recent master


Update README.md

Ok so with some help from @AliTopal89 I figured out what the problem was with my installation. I've updated the README accordingly.

thanks for the contribution!


Transparent navigation bar causes UI issues on pre-Q devices

Thanks for such a convenient library.
Wanted to report that the way you suggest in the README file works good for screens with no navigation bar, but for devices with good old navigation bar (like we have in Android versions before 9) setting navigation color to transparent causes issues with these navigation bars making navigation buttons almost invisible.
Here is a screenshot taken from your sample app on Android 7:

I guess it should be mentioned in README that such transparent navigation bar should be set only for values-29 as it is mentioned in Google`s doc: https://developer.android.com/training/gestures/gesturenav#transparent-bars
I will open a PR quit later mentioning it if you don't mind.

Thanks for reporting this. Transparent navigation bar can cause contrast issues on some [v23..29) device indeed. Unfortunately this issue is device specific and can only be fixed for all devices by applying opaque black color instead of transparent one, which is not always wanted.
I added the Navigation-Bar-Transparency wiki page explaining the issue and showing how to solve it. Main readme page was updated as well. No changes to the library or demo app are actually needed IMHO, because the library doesn't control the color of the navigation bar. It's a responsibility of developers using the library.
Hope it helps.

Thanks. I am quite late with the changes I wanted to add into README.
Yes, I understand that this is device specific. Just wanted to give a heads up, since previously there were no mentions of such possible issues, so it could confuse some devs.

As to updates to sample app - I would change theme to have this color setting for the navigation bar to values-v29, since some devs might just look at you sample as reference for their code base.
So just a minor addition to play safe even for those, who don't read manual carefully 🙃

Fair enough. I've just pushed update to navigation bar color for v23 and new v29 colors.xml

Wow. Thanks for such a quick update.

You're welcome and thanks for highlighting the issue! Closing it now.


upgrade debian & rust stable version, drop cargo-chef

mostly based on https://github.com/thermondo/salesforce-event-relay/pull/103, https://github.com/thermondo/salesforce-event-relay/pull/112

@pcrock-thmdo thanks for the improvement! ❤


Error while trying to install vagrant-libvirt

running
CONFIGURE_ARGS="with-libvirt-include=/usr/include/libvirt with-libvirt-lib=/usr/lib64" vagrant plugin install vagrant-libvirt
yields
(...)
domain.c: En la función ‘ruby_libvirt_domain_init’:
domain.c:5760:29: error: ‘VIR_DOMAIN_QEMU_AGENT_COMMAND_BLOCK’ (...)

while attempting to build ~/.vagrant.d/gems/2.6.2/gems/ruby-libvirt-0.7.1/ext/libvirt ruby-libvirt, Maybe beacuse the system installed libvirt (6.6.0, fedora 33) is not compatible?
Any help, comments, suggestions are welcome.
Thank you

Looks like you may have created this issue on the wrong repository?

Maybe? Thanks for the answer nevertheless. The thing is, the command CONFIGURE_ARGS="with-libvirt-include=/usr/include/libvirt with-libvirt-lib=/usr/lib64" vagrant plugin install vagrant-libvirt runs successfully if I use the upstream Vagrant package installed under /opt and applying the known workarounds for it to work with Fedora and Similar distributions

I think I would suggest either the vagrant or vagrant-libvirt repositories, since it is dealing with those specifically. This repository, while admittedly generalizes to hashicorp, has not been tested to work with vagrant or any plugins that vagrant might use.


Fixes a python "No such file or directory" error

This output shows the error that this PR resolves. My system does not have "loop" OR "modinfo" so the check for it fails. This resolves the check error.
https://pastebin.com/4wqVCPcZ

I fixed this in https://github.com/emilsvennesson/script.module.inputstreamhelper/pull/78


Global Catalog access is needed to enable Azure Active Directory Seam…

…less Single Sign-On
Without access to the GC the scripts to enable Azure Active Directory Seamless Single Sign-On seen at https://docs.microsoft.com/en-us/azure/active-directory/hybrid/tshoot-connect-sso#manual-reset-of-the-feature do not work because the cmdlets report "[time] [  9] [INFORMATIONAL] GetDesktopSsoComputerAccount: Searching in global catalog(forest) and ad.dcsadvisory.com for computer account AZUREADSSOACC" and then fail to connect. Once GC ports are open this works immediately. Therefore proposing that this is added to this table

@brianreidc7 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

This is going to be included in an overall update to the doc. This item has been captured. At this point we can close this as it will probably be another month or so. We need to address the images as well. Thanks!
#please-close


vault : stop vault-temp container fails with "The conditional check 'inventory_hostname in groups.vault' failed"

Is this a BUG REPORT or FEATURE REQUEST? (choose one):
Bug report
Environment:
Ubuntu Xenial with dist upgrades.
Pull group_vars/all.yml and group_vars/k8s-cluster.yaml from gist: https://gist.github.com/israelshirk/c5b14843c43920ce940027e7097bc086
deploy.sh:
#!/bin/bash

set -e
set -x

rm -rf playbooks

kubespray aws -y --config kubespray.yml --nodes 2

pushd playbooks
git reset --hard cfea99c4ee56f8d7afe176fdc4a172f610bdc333
popd

rm -rf playbooks/inventory/group_vars
cp -a group_vars playbooks/inventory/

IPS=$(cat playbooks/nodes_instances.json | jq -r .[].public_ip)
for IP in $IPS; do
	ssh-keygen -R $IP
	ssh-keyscan -H $IP >> ~/.ssh/known_hosts
	ssh ubuntu@$IP -- "sudo DEBIAN_FRONTEND=noninteractive apt-get -y update && sudo DEBIAN_FRONTEND=noninteractive apt-get -y dist-upgrade && sudo DEBIAN_FRONTEND=noninteractive apt-get -y install python"
done

# Use -r on Linux.
sed -E -i'' -e 's/(ansible_ssh_host[^\s]*)/\1   ansible_ssh_user=ubuntu/g' playbooks/inventory/inventory.cfg

kubespray deploy -y --aws --ubuntu --config kubespray.yml

.kubespray.yml (config):
# Common options
# ---------------
# Path where the kubespray ansible playbooks will be installed
# Defaults to current user's home directory if not set
kubespray_path: "/current_directory/playbooks"
# Default inventory path
kubespray_git_repo: "https://github.com/kubespray/kubespray.git"
# Logging options
loglevel: "info"

aws_access_key: "mine mine mine"
aws_secret_key: "mine mine mine"
key_name: "Israel Zerrtech Laptop"
ami: "ami-6e1a0117"
masters_instance_type: "t2.medium"
nodes_instance_type: "t2.medium"
etcds_instance_type: "t2.medium"
masters_instance_profile_name: "kubespray"
nodes_instance_profile_name: "kubespray"
etcds_instance_profile_name: "kubespray"
security_group_name: "Kubernetes"
security_group_id: "sg-12473b6f"
assign_public_ip: True
vpc_subnet_id: "subnet-7973c531"
region: "us-west-2"
tags:
  - type=k8s
  - branch=development
  - persist=true

Version of Ansible (ansible --version):
2.4.0, OS X

Kubespray version (commit) (git rev-parse --short HEAD):
master as of 46cf6b77cf614b2233d1443a02c5efb8e649930d
Network plugin used:
cloud
Copy of your inventory file:
[kube-master]
k8s-denizen-iv9wa
k8s-denizen-fjhcb

[all]
k8s-denizen-j5tk6		ansible_ssh_host=<IP_ADDRESS>   ansible_ssh_user=ubuntu
k8s-denizen-90fdl		ansible_ssh_host=<IP_ADDRESS>   ansible_ssh_user=ubuntu
k8s-denizen-m4cre		ansible_ssh_host=<IP_ADDRESS>   ansible_ssh_user=ubuntu
k8s-denizen-fjhcb		ansible_ssh_host=<IP_ADDRESS>   ansible_ssh_user=ubuntu
k8s-denizen-iv9wa		ansible_ssh_host=<IP_ADDRESS>   ansible_ssh_user=ubuntu

[k8s-cluster:children]
kube-node
kube-master

[kube-node]
k8s-denizen-iv9wa
k8s-denizen-fjhcb

[etcd]
k8s-denizen-iv9wa

Command used to invoke ansible:
See above deploy.sh
Output of ansible run:
Full output of deploy.sh: https://gist.github.com/israelshirk/c5b14843c43920ce940027e7097bc086
Anything else do we need to know:
Reverting 0b60201a1eb3a72161ef3e2c82ababd50f1af1ce fixes the issue.

I just made a repo of this so my setup is a little more straightforwards:
https://github.com/israelshirk/kubespray-example
Just fill in your AWS creds; for the security profile I used the following JSON (obviously a security issue if you look at it for 10 seconds or more, but it's a test env):
{
    "Version": "2012-10-17",
    "Statement": [
        {
            "Effect": "Allow",
            "Resource": "*",
            "Action": "sts:AssumeRole"
        },
        {
            "Effect": "Allow",
            "Action": [
                "ec2:*"
            ],
            "Resource": [
                "*"
            ]
        },
        {
            "Effect": "Allow",
            "Action": [
                "elasticloadbalancing:*"
            ],
            "Resource": [
                "*"
            ]
        },
        {
            "Effect": "Allow",
            "Action": [
                "route53:*"
            ],
            "Resource": [
                "*"
            ]
        },
        {
            "Effect": "Allow",
            "Action": "s3:*",
            "Resource": [
                "arn:aws:s3:::kubernetes-*"
            ]
        },
        {
            "Effect": "Allow",
            "Action": "s3:*",
            "Resource": [
                "arn:aws:s3:::kubernetes-*"
            ]
        },
        {
            "Effect": "Allow",
            "Action": "ec2:Describe*",
            "Resource": "*"
        },
        {
            "Effect": "Allow",
            "Action": "ec2:AttachVolume",
            "Resource": "*"
        },
        {
            "Effect": "Allow",
            "Action": "ec2:DetachVolume",
            "Resource": "*"
        },
        {
            "Effect": "Allow",
            "Action": [
                "route53:*"
            ],
            "Resource": [
                "*"
            ]
        },
        {
            "Effect": "Allow",
            "Action": [
                "ecr:GetAuthorizationToken",
                "ecr:BatchCheckLayerAvailability",
                "ecr:GetDownloadUrlForLayer",
                "ecr:GetRepositoryPolicy",
                "ecr:DescribeRepositories",
                "ecr:ListImages",
                "ecr:BatchGetImage"
            ],
            "Resource": "*"
        }
    ]
}

Hi, you need add [vault] group in your inventory,
like this:
[kube-master]
k8s-denizen-iv9wa
k8s-denizen-fjhcb

[all]
k8s-denizen-j5tk6		ansible_ssh_host=<IP_ADDRESS>   ansible_ssh_user=ubuntu
k8s-denizen-90fdl		ansible_ssh_host=<IP_ADDRESS>   ansible_ssh_user=ubuntu
k8s-denizen-m4cre		ansible_ssh_host=<IP_ADDRESS>   ansible_ssh_user=ubuntu
k8s-denizen-fjhcb		ansible_ssh_host=<IP_ADDRESS>   ansible_ssh_user=ubuntu
k8s-denizen-iv9wa		ansible_ssh_host=<IP_ADDRESS>   ansible_ssh_user=ubuntu

[k8s-cluster:children]
kube-node
kube-master

[kube-node]
k8s-denizen-iv9wa
k8s-denizen-fjhcb

[etcd]
k8s-denizen-iv9wa

[vault]
k8s-denizen-iv9wa

@chapsuk thanks for the help, that got it fixed right up.  I ended up using this since it it's a scripted setup -
echo -e "\n[vault]\n" >> /tmp/playbooks/inventory/inventory.cfg
cat playbooks/inventory/inventory.cfg | grep k8s- | head -n2 | uniq -u >> playbooks/inventory/inventory.cfg


Reloading Oni with multiple instances makes the UI appear in the last instance.

Hey, sorry for the bad title, couldn't find a short way to explain this.
Steps to reproduce:

Open two or more Oni instances
Reload any oni instance but the last
the the reloaded one stays blank and the last one contains both UIs, like so:

This adds up, the more instances of Oni you reload.

Hey @muream!
Can I check what version of Oni you are running?
I made a change to how we reference the editors in the main process, so I think this should be fixed on master. (@Akin909 had a similar issue over here: #1696)
If that isn't the case, we should look into it some more.

Oh, I only searched the open issues, I'm running 0.3.1.

I just tested this in master and can confirm that it does not occur anymore


Support toggling the display of approved PRS

I would like to view the pr's that iv'e already approved in the list

Expected in upcoming version 4.0.0

Eta?
On Wednesday, June 22, 2016, Nitai J. Perez<EMAIL_ADDRESS>wrote:

Expected in upcoming version 4.0.0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CommonRaven/stash-notify/issues/3#issuecomment-227796100,
or mute the thread
https://github.com/notifications/unsubscribe/AFl9RH2H2JFlWkqnzK-ZqOg8BEvKYqdZks5qOV_DgaJpZM4I78-Q
.

--
Ido Schachter
Software developer
[image: ironSource] http://www.ironsrc.com
email<EMAIL_ADDRESS>mobile +972-52-605-2282
fax +972-77-5448273
28 Lilienblum St., Tel Aviv
ironsrc.com http://www.ironsrc.com
[image: linkedin] https://www.linkedin.com/company/ironsource [image:
twitter] https://twitter.com/ironsource [image: facebook]
https://www.facebook.com/ironSource [image: googleplus]
https://plus.google.com/+ironsrc
This email (including any attachments) is for the sole use of the intended
recipient and may contain confidential information which may be protected
by legal privilege. If you are not the intended recipient, or the employee
or agent responsible for delivering it to the intended recipient, you are
hereby notified that any use, dissemination, distribution or copying of
this communication and/or its content is strictly prohibited. If you are
not the intended recipient, please immediately notify us by reply email or
by telephone, delete this email and destroy any copies. Thank you.

With a bit of luck under 2 weeks, but no promises at this stage. :)

Update: It looks like Stash does not provide an endpoint for PRs which were approved but not merged, so this isn't going to be implemented in v4.0.0 as expected. If such an endpoint is found, please share it and I will add the functionality. Pull requests are also welcome.

Switching status to won't fix. Currently seems impossible in stash.


Flags or environment variables are not being enforced

Hi,
I have set up a minio instance locally and trying to upload modules to it via boring-registry.
I have the following .hcl file in the module folder:
metadata {
  namespace = "tier"
  name      = "tls-private-key"
  provider = "aws"
  version   = "0.1.0"
}

But whenever I run the following command:
boring-registry upload module   --storage-s3-bucket=terraform-registry-test \
  --storage-s3-region=eu-east-1 \
  --storage-s3-pathstyle=true \
  --storage-s3-endpoint=http://localhost:9000 .

It comes with the following error:
operation error S3: CreateMultipartUpload, exceeded maximum number of attempts, 3, failed to sign request: failed to retrieve credentials: failed to refresh cached credentials, no EC2 IMDS role found, operation error ec2imds: GetMetadata, exceeded maximum number of attempts, 3, request send failed, Get "http://<IP_ADDRESS>/latest/meta-data/iam/security-credentials/": dial tcp <IP_ADDRESS>:80: i/o timeout: failed to upload module

The error indicates that the AWS client tried all authentication methods and tries to use the EC2 IMDS role as a last resort, which ultimately failed.
I could reproduce the error with the following docker-compose.yaml file:
version: '3.7'

services:
  minio1:
    image: quay.io/minio/minio:RELEASE.2023-08-09T23-30-22Z
    command: server --console-address ":9001" http://minio1/data{1...2}
    ports:
      - "9000:9000"
      - "9001:9001"
    environment:
      MINIO_ROOT_USER: minioadmin
      MINIO_ROOT_PASSWORD: minioadmin
    healthcheck:
      test: ["CMD", "curl", "-f", "http://localhost:9000/minio/health/live"]
      interval: 30s
      timeout: 20s
      retries: 3
    hostname: minio1
    volumes:
      - data1-1:/data1
      - data1-2:/data2
volumes:
  data1-1:
  data1-2:

Uploading modules failed. What I had to do was to create an access key in MinIO. I could then pass the access key and the secret key as environment variables to the upload command:
$ AWS_ACCESS_KEY_ID=myAccessKeyId AWS_SECRET_ACCESS_KEY=mySecretAccessKey go run ./... upload module --storage-s3-bucket=terraform-registry-test --storage-s3-region=eu-east-1 --storage-s3-pathstyle=true --storage-s3-endpoint=http://localhost:9000 /path/to/my/modules

Let me know if that helps :)

Worked perfectly, thank you for making an example of how the command would look like with the ACCESS variables.


Use request library - AGJS-302

Adds the request library

@sebastienblanc @edewit if you guys have time, perhaps just a quick sanity check for this

@matzew @lfryc if you guys get bored, perhaps give this a gander :)

:+1: less is better


WIP: Add AgainstInvalidEnumArgument & AgainstInvalidEnumArgumentProperty

In support of #108.

@adamralph added signatures for review / confirmation. Let me know if those make sense.
Also, where the heck am I adding this? I saw the acceptance test was off of .NET 3.5 but it doesn't look at first glance like those files are linked. Will I eventually be copying the code to various places once it's passed acceptance testing?

I guess breaking the build a few times is a good way to get started. :smile:

@SeanKilleen the signatures I had in mind are those I wrote in https://github.com/liteguard/liteguard/issues/108#issuecomment-116256648 - what's your opinion on them?
BTW - [ValidatedNotNull] isn't required since we have the struct constraint.
The multi-targetting is awkward. Ultimately you need to copy the changes into all the different versions of Guard.cs in all the projects :unamused:. You can get everything working on the .net 35 project first though.

@adamralph sorry, that feedback on ValidatedNotNull is exactly what I meant I was looking for. Thanks!

BTW - I've just pushed a change to master which means there are now only two versions of Guard.cs.

I'll pull it in -- thanks!

Closing, since we agreed to withdraw #108.


[New Port Request] libvncserver

Library name: libvncserver
Library description: LibVNCServer/LibVNCClient are cross-platform C libraries that allow you to easily implement VNC server or client functionality in your program.
Source repository URL: https://github.com/LibVNC/libvncserver

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now.


Model files tab can't open

Describe the bug
Open a model and navigate to files tab, it will redirect to 404 page. I also noticed the API call 500 error
It seems the last commit is nil but the code not handle it
Environment
CSGHub Version: latest
OS: MacOS
Hardware: M2 Pro
Launch:  vscode
Screenshots

Additional context

Seems the repository is an empty repository, which with no commits. I'll fix this bug later.
Thanks for your report! @ttlittletree


Error when run create-bridge command for iOS/Objective-C

Error: ENOENT: no such file or directory, open '.../ios/ExampleBridge.h'

Thanks for reporting! Can you please provide a little more context? Were you running the command in an existing RN project?

I was running out of the project. Problem solved. Thanks :)


GH-1963 fix for language tag, off-by-one and improved test

GitHub issue resolved: #1963
Briefly describe the changes proposed in this PR:

Improved test
Fix issues with language and datatypes not being decoded
Fix incorrect generation of incorrect triples

hmz, issue in repository-compliance test...


Device detected by clinfo but not by tensorflow

I'm not sure if this is a "supported architectures" issue, or if there are more details I should give. What do you think?
$ ./run_tests.sh
+ cd examples
+ pushd 2_BasicModels
/.../TensorFlow-Examples/examples/2_BasicModels /.../TensorFlow-Examples/examples
+ python linear_regression.py
gpu_manager->VisibleDeviceCount is 0
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py", line 972, in _do_call
    return fn(*args)
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py", line 950, in _run_fn
    self._extend_graph()
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py", line 999, in _extend_graph
    self._session, graph_def.SerializeToString(), status)
  File "/usr/lib/python3.5/contextlib.py", line 66, in __exit__
    next(self.gen)
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/framework/errors.py", line 463, in raise_exception_on_not_ok_status
    pywrap_tensorflow.TF_GetCode(status))
tensorflow.python.framework.errors.InvalidArgumentError: Cannot assign a device to node 'GradientDescent/learning_rate': Could not satisfy explicit device specification '/device:GPU:0' because no devices matching that specification are registered in this process; available devices: /job:localhost/replica:0/task:0/cpu:0
         [[Node: GradientDescent/learning_rate = Const[dtype=DT_FLOAT, value=Tensor<type: float shape: [] values: 0.01>, _device="/device:GPU:0"]()]]

$ clinfo
Number of platforms                               1
  Platform Name                                   Intel(R) OpenCL
  Platform Vendor                                 Intel(R) Corporation
  Platform Version                                OpenCL 1.2 LINUX
  Platform Profile                                FULL_PROFILE
  Platform Extensions                             cl_khr_icd cl_khr_global_int32_base_atomics cl_khr_global_int32_extended_atomics cl_khr_local_int32_base_atomics cl_khr_local_int32_extended_atomics cl_khr_byte_addressable_store cl_khr_depth_images cl_khr_3d_image_writes cl_intel_exec_by_local_thread cl_khr_spir cl_khr_fp64
  Platform Extensions function suffix             INTEL

  Platform Name                                   Intel(R) OpenCL
Number of devices                                 1
  Device Name                                     Intel(R) Core(TM) i7-4710HQ CPU @ 2.50GHz
  Device Vendor                                   Intel(R) Corporation
  Device Vendor ID                                0x8086
  Device Version                                  OpenCL 1.2 (Build 25)
  Driver Version                                  <IP_ADDRESS>
  Device OpenCL C Version                         OpenCL C 1.2
  Device Type                                     CPU
  Device Profile                                  FULL_PROFILE
  Max compute units                               2
  Max clock frequency                             2500MHz
  Device Partition                                (core)
    Max number of sub-devices                     2
    Supported partition types                     by counts, equally, by names (Intel)
  Max work item dimensions                        3
  Max work item sizes                             8192x8192x8192
  Max work group size                             8192
  Preferred work group size multiple              128
  Preferred / native vector sizes
    char                                                 1 / 16
    short                                                1 / 8
    int                                                  1 / 4
    long                                                 1 / 2
    half                                                 0 / 0        (n/a)
    float                                                1 / 8
    double                                               1 / 4        (cl_khr_fp64)
  Half-precision Floating-point support           (n/a)
  Single-precision Floating-point support         (core)
    Denormals                                     Yes
    Infinity and NANs                             Yes
    Round to nearest                              Yes
    Round to zero                                 No
    Round to infinity                             No
    IEEE754-2008 fused multiply-add               No
    Support is emulated in software               No
    Correctly-rounded divide and sqrt operations  No
  Double-precision Floating-point support         (cl_khr_fp64)
    Denormals                                     Yes
    Infinity and NANs                             Yes
    Round to nearest                              Yes
    Round to zero                                 Yes
    Round to infinity                             Yes
    IEEE754-2008 fused multiply-add               Yes
    Support is emulated in software               No
    Correctly-rounded divide and sqrt operations  No
  Address bits                                    64, Little-Endian
  Global memory size<PHONE_NUMBER> (3.858GiB)
  Error Correction support                        No
  Max memory allocation<PHONE_NUMBER> (987.7MiB)
  Unified memory for Host and Device              Yes
  Minimum alignment for any data type             128 bytes
  Alignment of base address                       1024 bits (128 bytes)
  Global Memory cache type                        Read/Write
  Global Memory cache size                        262144
  Global Memory cache line                        64 bytes
  Image support                                   Yes
    Max number of samplers per kernel             480
    Max size for 1D images from buffer            64727232 pixels
    Max 1D or 2D image array size                 2048 images
    Max 2D image size                             16384x16384 pixels
    Max 3D image size                             2048x2048x2048 pixels
    Max number of read image args                 480
    Max number of write image args                480
  Local memory type                               Global
  Local memory size                               32768 (32KiB)
  Max constant buffer size                        131072 (128KiB)
  Max number of constant args                     480
  Max size of kernel argument                     3840 (3.75KiB)
  Queue properties
    Out-of-order execution                        Yes
    Profiling                                     Yes
    Local thread execution (Intel)                Yes
  Prefer user sync for interop                    No
  Profiling timer resolution                      1ns
  Execution capabilities
    Run OpenCL kernels                            Yes
    Run native kernels                            Yes
    SPIR versions                                 1.2
  printf() buffer size                            1048576 (1024KiB)
  Built-in kernels
  Device Available                                Yes
  Compiler Available                              Yes
  Linker Available                                Yes
  Device Extensions                               cl_khr_icd cl_khr_global_int32_base_atomics cl_khr_global_int32_extended_atomics cl_khr_local_int32_base_atomics cl_khr_local_int32_extended_atomics cl_khr_byte_addressable_store cl_khr_depth_images cl_khr_3d_image_writes cl_intel_exec_by_local_thread cl_khr_spir cl_khr_fp64

NULL platform behavior
  clGetPlatformInfo(NULL, CL_PLATFORM_NAME, ...)  No platform
  clGetDeviceIDs(NULL, CL_DEVICE_TYPE_ALL, ...)   No platform
  clCreateContext(NULL, ...) [default]            No platform
  clCreateContext(NULL, ...) [other]              Success [INTEL]
  clCreateContextFromType(NULL, CL_DEVICE_TYPE_CPU)  No platform
  clCreateContextFromType(NULL, CL_DEVICE_TYPE_GPU)  No platform
  clCreateContextFromType(NULL, CL_DEVICE_TYPE_ACCELERATOR)  No platform
  clCreateContextFromType(NULL, CL_DEVICE_TYPE_CUSTOM)  No platform
  clCreateContextFromType(NULL, CL_DEVICE_TYPE_ALL)  No platform

You need a device with Device Type set to GPU. Yours says CPU?

It says: CL_DEVICE_TYPE_CPU ...
Respectfully,
בכבוד רב,
Inon Sharony.
ינון שרוני.
On Sat, Aug 19, 2017 at 11:59 AM, Hugh Perkins<EMAIL_ADDRESS>wrote:

You need a device with Device Type set to GPU. Yours says CPU?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hughperkins/tf-coriander/issues/60#issuecomment-323518980,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGYsS81z9TCXvVKSg0QV9BqwiosX4IFaks5sZs42gaJpZM4O8Rul
.

Oh. Youre the guy that posted a LinkedIn message that was "like"d by thousands :)
Welcome :)
Ok, so, modern Intel CPUs often contain GPUs.  Which is not the 'CPU' bit of the CPU itself, but an extra component, inside the CPU. It's a bit confusing :-P
So, it turns out that your cpu does in fact have a GPU inside it, it's an HD4600: https://ark.intel.com/products/78930/Intel-Core-i7-4710HQ-Processor-6M-Cache-up-to-3_50-GHz

Then the next question is: does your HD4600 GPU, inside your 4710HQ CPU, support OpenCL?  The page above doesnt say.  But it sounds modernish: I used to have an HD4000, and that supported OpenCL, so let's see...
Googling for 'wikipedia hd', we get https://en.wikipedia.org/wiki/Intel_HD_and_Iris_Graphics#Capabilities . This shows that Haswell cpus have OpenCL 1.2 GPUs:

At this point, I conclude: you're missing the driver :) .  You probably need to install a driver from the Intel website, eg something like ...hmmm... I can only find for windows https://downloadcenter.intel.com/product/97501/Graphics-for-5th-Generation-Intel-Processors .  oh right: you need Beignet :)

https://www.freedesktop.org/wiki/Software/Beignet/
https://github.com/intel/beignet

(Note that you're very late to the party though; I'm working on other things at the moment; meanwhile tf-coriander is missing a cudnn replacement. I started working on one at https://github.com/hughperkins/coriander-dnn , but never quite got round to plugging it in to tf-coriander. I dont think it's tons of work. If I did have a moment, it'd probably take me ~40-80 hours. But of course if someone else does it, they have to learn everything from scratch, so could easily be 4-8 times that.
If you can find someone who could be interested in helping with that, I'm happy to assist them, meet them in Hangouts etc.
The impact of not having cudnn currently is that convolutions run on CPU (not the GPU part of the CPU, the CPU bit).
)

Wow, thanks Hugh!
Yeah, the response to that post surprised me too. I was so upset that
TensorFlow doesn't support an open standard for GPGPU!
As for my particular case, I do have an AMD GPU in addition to the Intel
one which is built into my motherboard. It's a Radeon R9 M265X. Pretty
low-scale by today's standards, but It supports OpenCL, and it's
frustrating that I can't use it with TensorFlow out of the box (as opposed
to an nVidia card, I expect). I'm not even sure how much help it would be,
considering I went and paid for a quad-core CPU.
I guess I need to re-check my driver installation. I tried using
Ubuntu-based dockers, which is where I got the stacktrace and clinfo output
in my original post.
Maybe I should address AMD support?
Respectfully,
בכבוד רב,
Inon Sharony.
ינון שרוני.
On Sat, Aug 19, 2017 at 5:15 PM, Hugh Perkins<EMAIL_ADDRESS>wrote:

(Note that you're very late to the party though; I'm working on other
things at the moment; meanwhile tf-coriander is missing a cudnn
replacement. I started working on one at https://github.com/
hughperkins/coriander-dnn , but never quite got round to plugging it in
to tf-coriander. I dont think it's tons of work. If I did have a moment,
it'd probably take me ~40-80 hours. But of course if someone else does it,
they have to learn everything from scratch, so could easily be 4-8 times
that.
If you can find someone who could be interested in helping with that, I'm
happy to assist them, meet them in Hangouts etc.
The impact of not having cudnn currently is that convolutions run on CPU
(not the GPU part of the CPU, the CPU bit).
)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hughperkins/tf-coriander/issues/60#issuecomment-323535892,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGYsS3bqJJOdRO2NmE9siI29z2TlKGbGks5sZxhBgaJpZM4O8Rul
.

Yeah, the response to that post surprised me too. I was so upset that
TensorFlow doesn't support an open standard for GPGPU!

:)
Yes, impressive to write something that went so viral :)

I guess I need to re-check my driver installation. I tried using
Ubuntu-based dockers, which is where I got the stacktrace and clinfo output
in my original post.

Docker is not very OpenCL/GPU friendly.  Docker does work ... with NVIDIA GPUs :-P . I'm not saying Docker cant be tweaked to work with AMD GPUs, but I've never heard of that being possible. For NVIDIA GPUs, you need some special additional drivers, eg https://github.com/NVIDIA/nvidia-docker , or at least pass the drivers through, using --device option to Docker, like https://hub.docker.com/r/hughperkins/cltorch-nvidia/  But I've never heard of this being possible for AMD GPUs.
Your easiest options for AMD GPUs will probably be to use the AMD GPU directly from your OS, so one of:

install an OS that supports the AMD GPU drivers, and /or
find AMD GPU drivers for your current main OS

If it was me, well .... so... I got into OpenCL, since I had a laptop with an Intel CPU, with an HD4000 inside, and I thought it was so cool that the CPU had a GPU inside, and wanted to play, and of course it wont work with CUDA, so I wrote https://github.com/hughperkins/DeepCL from scratch, incrementally, over ~6 months, so that I could play with using the HD4000 GPU :)
Later on though, I found that the Intel GPU, whilst fun, is not something I'd ever train an ml model on: aws works well for that, or at least, an NVIDIA GPU.  There are no AMD cloud-enabled GPUs around that I can find.
Currently, I think that whilst it'd be good to have competition for NVDIA GPUs, to keep them on their toes, I'm not sure that AMD will be that competition, at least, not in a big way. I think that something like the Nervana TPUs might be a more realistic competition possibly?  https://cloud.google.com/blog/big-data/2017/05/an-in-depth-look-at-googles-first-tensor-processing-unit-tpu

Agreed. On all points ;-)
--
Inon Sharony
ינון שרוני
On Aug 19, 2017 8:47 PM, "Hugh Perkins"<EMAIL_ADDRESS>wrote:

Yeah, the response to that post surprised me too. I was so upset that
TensorFlow doesn't support an open standard for GPGPU!
:)
Yes, impressive to write something that went so viral :)
I guess I need to re-check my driver installation. I tried using
Ubuntu-based dockers, which is where I got the stacktrace and clinfo output
in my original post.
Docker is not very OpenCL/GPU friendly. Docker does work ... with
NVIDIA GPUs :-P . I'm not saying Docker cant be tweaked to work with AMD
GPUs, but I've never heard of that being possible. For NVIDIA GPUs, you
need some special additional drivers, eg https://github.com/NVIDIA/
nvidia-docker , or at least pass the drivers through, using --device
option to Docker, like https://hub.docker.com/r/
hughperkins/cltorch-nvidia/ But I've never heard of this being possible
for AMD GPUs.
Your easiest options for AMD GPUs will probably be to use the AMD GPU
directly from your OS, so one of:

install an OS that supports the AMD GPU drivers, and /or
find AMD GPU drivers for your current main OS

If it was me, well .... so... I got into OpenCL, since I had a laptop with
an Intel CPU, with an HD4000 inside, and I thought it was so cool that the
CPU had a GPU inside, and wanted to play, and of course it wont work with
CUDA, so I wrote https://github.com/hughperkins/DeepCL from scratch,
incrementally, over ~6 months, so that I could play with using the HD4000
GPU :)
Later on though, I found that the Intel GPU, whilst fun, is not something
I'd ever train an ml model on: aws works well for that, or at least, an
NVIDIA GPU. There are no AMD cloud-enabled GPUs around that I can find.
Currently, I think that whilst it'd be good to have competition for NVDIA
GPUs, to keep them on their toes, I'm not sure that AMD will be that
competition, at least, not in a big way. I think that something like the
Nervana TPUs might be a more realistic competition possibly?
https://cloud.google.com/blog/big-data/2017/05/an-in-depth-
look-at-googles-first-tensor-processing-unit-tpu
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hughperkins/tf-coriander/issues/60#issuecomment-323537662,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGYsS2uCEe0nfI4peUdSCLg59VqJGVDVks5sZx_IgaJpZM4O8Rul
.


Fixed del to allow deletion of falsy values in array

Modified del API to allow deletion of falsy values in array item.
Before: del([''], 0)).to.deep.equal([''])
After: del([''], 0)).to.deep.equal([])

Coverage increased (+0.007%) to 98.951% when pulling 893c0be4e3dc77f1ab5b1eb0aa524b50bafd0ac7 on ftliou:fix-del into 0df5e928caac75dbfd08f908216693240295eb6f on mariocasciaro:master.

Coverage increased (+0.007%) to 98.951% when pulling 893c0be4e3dc77f1ab5b1eb0aa524b50bafd0ac7 on ftliou:fix-del into 0df5e928caac75dbfd08f908216693240295eb6f on mariocasciaro:master.

Thanks for the PR! Could you please fix the linting errors, please, so the build doesn't fail?

Coverage increased (+0.007%) to 98.951% when pulling 9a2fb0aa388a4434c13823e85b3716db599879e4 on ftliou:fix-del into 0df5e928caac75dbfd08f908216693240295eb6f on mariocasciaro:master.

Coverage increased (+0.007%) to 98.951% when pulling 9a2fb0aa388a4434c13823e85b3716db599879e4 on ftliou:fix-del into 0df5e928caac75dbfd08f908216693240295eb6f on mariocasciaro:master.

Oops sorry. Done :)

Thanks a lot!!!


Add install-zfs-dkms-module example

Add install-zfs-dkms-module example

[x] Add install-zfs-dkms-module example Containerfile and readme
[x] Add install-zfs-dkms-module example to main readme
[x] Add install-zfs-dkms-module github workflow file
[x] Test build install-zfs-dkms-module example

Looks like this is failing because fedora-cores stable has kernel version 6.4.7 and the ZFS package requires a maximum kernel version of 6.3.999.
ZFS does this on purpose because the kernel symbols can change and they have to ensure that they work with each version of the kernel.
This will automatically start working once the ZFS package is updated to allow for kernel 6.4.x.
Until then, I'm going to mark this as a draft PR.

I guess we only run CI when someone changes the example, so merging this would not throw a bunch of failures at us every time fedora updates the kernel to a version not supported by ZFS.

This works now with the recent release from the upstream zfs project. Thanks for the awesome example.

@twiest anything new and cool you have done to your Containerfile that you want to update here before I merge it?


Rename "api" workspace to "data-registry"

See: https://api3workspace.slack.com/archives/C05S589E7B4/p1695730568361939

We decided not to.


Add supportedThreadFeatures, threadVersion and SupportedWiFiBands attributes to some reference apps

Update the zap file of some reference apps to enable

supportedThreadFeatures
threadVersion
SupportedWiFiBands

attributes of the network commissioning cluster
The affected apps are :

all-clusters-app
all-clusters-minimal-ap
lock-app
lighting-app
lit-icd-server-app

NOTE this enables both Wifi and Thread specific clusters for validation purposes regardless of the platform network technology. This should not be the normal configuration of a product and that is why it is not applied to all sample apps - Open to better suggestion about

@jmartinez-silabs the cirque failure may be a real failure. I see:
Using PASE connection
2024-01-10 05:57:01,495 [PythonMatterControllerTEST] ERROR 'ValueDecodeFailure' object has no attribute 'networks'
Traceback (most recent call last):
  File "/home/runner/work/connectedhomeip/connectedhomeip/src/controller/python/test/test_scripts/python_commissioning_flow_test.py", line 183, in <module>
    main()
  File "/home/runner/work/connectedhomeip/connectedhomeip/src/controller/python/test/test_scripts/python_commissioning_flow_test.py", line 170, in main
    raise ex
  File "/home/runner/work/connectedhomeip/connectedhomeip/src/controller/python/test/test_scripts/python_commissioning_flow_test.py", line 148, in main
    asyncio.run(flow.commission(commissioning.Parameters(
  File "/usr/lib/python3.8/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
    return future.result()
  File "/home/runner/work/connectedhomeip/connectedhomeip/src/controller/python/test/test_scripts/example_python_commissioning_flow.py", line 49, in commission
    await self.network_commissioning(parameter=parameter, node_id=node_id)
  File "/usr/local/lib/python3.8/dist-packages/chip/commissioning/commissioning_flow_blocks.py", line 217, in network_commissioning
    if network_commissioning_cluster_state.networks:
AttributeError: 'ValueDecodeFailure' object has no attribute 'networks'
ERROR:PythonMatterControllerTEST:'ValueDecodeFailure' object has no attribute 'networks'
Traceback (most recent call last):
  File "/home/runner/work/connectedhomeip/connectedhomeip/src/controller/python/test/test_scripts/python_commissioning_flow_test.py", line 183, in <module>
    main()
  File "/home/runner/work/connectedhomeip/connectedhomeip/src/controller/python/test/test_scripts/python_commissioning_flow_test.py", line 170, in main
    raise ex
  File "/home/runner/work/connectedhomeip/connectedhomeip/src/controller/python/test/test_scripts/python_commissioning_flow_test.py", line 148, in main
    asyncio.run(flow.commission(commissioning.Parameters(
  File "/usr/lib/python3.8/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
    return future.result()
  File "/home/runner/work/connectedhomeip/connectedhomeip/src/controller/python/test/test_scripts/example_python_commissioning_flow.py", line 49, in commission
    await self.network_commissioning(parameter=parameter, node_id=node_id)
  File "/usr/local/lib/python3.8/dist-packages/chip/commissioning/commissioning_flow_blocks.py", line 217, in network_commissioning
    if network_commissioning_cluster_state.networks:
AttributeError: 'ValueDecodeFailure' object has no attribute 'networks'

@andy31415 yes it seems like a real error.  I will try it tomorrow and fix it. I didn't get time to get back to it today

After discussing with @tcarmelveilleux, I reverted the config changes to the sample apps other than all-cluster-apps for now as we want to avoid users copying non-spec compliant configs from sample apps.
We are looking into ways to ensure that the attribute list matches the device network type in a compliant way. In combination with a valid feature-amp attribute, compliance tests should be able to validate devices.


spack arch returns debian

C:\Program Files\Spack 0.16.0>spack arch
test-debian6-core2

Fixed by https://github.com/spack/spack/pull/21428


python 3.9 will break pydlm?

Looks like an easy fix. Want a PR?
/pydlm/modeler/dynamic.py:18: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated since Python 3.3, and in 3.9 it will stop working
from collections import MutableSequence

Should have been fixed.


DOC: explain deprecation

@Eric89GXL

Something went terribly wrong with the commit history

yep. looks like you created the new branch from where we left the spoc branch at the codding sprint.

Yes, I'm on it

@Eric89GXL can we do python code blocks?

Yep, just doing :: at the end of a line will render the following blocked
text as code

Codecov Report

Merging #4172 into master will increase coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4172      +/-   ##
==========================================
+ Coverage   86.02%   86.15%   +0.13%
==========================================
  Files         354      354
  Lines       63933    63933
  Branches     9739     9739
==========================================
+ Hits        54999    55084      +85
+ Misses       6245     6156      -89
- Partials     2689     2693       +4

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 8207646...4e30ce1. Read the comment docs.

@kingjr I pushed the fix I requested.
merge if you're happy

oups... like this?

Yep. See it renders just fine:
https://4544-1301584-gh.circle-artifacts.com/0/home/ubuntu/mne-python/doc/_build/html/contributing.html#deprecating

beautiful !


update poopshow to latest

Updated most of the customizable theme parts to params.
Proper pull request this time :)

Done.


Fixes incorrect forwarding of touchesCancelled

The overwrite of touchesCancelled forwards to super.touchesMoved.

Ugh thanks for fixing @fonkadelic 😅 LGTM


[doc] Update doc with steps for custom metric collection

Description of the change:
This PR updates the user documentation with the procedure
to collect operator specific metrics using the metrics package
provided by SDK.
Motivation for the change:
Currently, many of the operator authors/owners are unaware
of the procedure to leverage metrics package provided by
SDK to collect operator specific metrics. This PR would update
the documentation with the required steps to enable operator
specific custom metrics collection using the functions provided
by the SDK.

@camilamacedo86 - I have updated the PR and addressed your comments. Please have a look at it.

Hi @@varshaprasad96,
Really thank you for your contribution. However, could you please check the review comments?

I am extremely sorry for the delay. I will make the changes and update the PR.

Hi @varshaprasad96,
Really thank for your contribution 🥇.
It is very important and helpful.

Hi @varshaprasad96,
It still missing some points to address in the order we are able to move forward and merge it. Could you please check them?

Hi @varshaprasad96,
Currently, the project is looking for to contribute with k8s docs and address the users check them when the feature is based on its implementation. So, after a deeper analysis in this PR shows that these details could be better addressed in https://book.kubebuilder.io/reference/metrics.html#publishing-additional-metrics. See that its docs will redirect the users to check this document.
Then, because of this, we are closing this PR. However, please feel free to re-open if you think that something should be done here instead of.

Hi @camilamacedo86,
Thanks for the update. I see that the process is being explained in the kubebuilder docs.
Just curious to know - the doc states that the custom metrics could be registered by declaring the global variables and registering them using MustRegister function of Prometheus. Wouldn't this collide with the default metrics, as both would get registered in the same port unless specified explicitly?
Maybe I am missing something, but just got this doubt when I saw the doc ! :)


feat(only_logs): deprecate only_logs flag

Description
Remove only_logs flag from Prowler
Checklist

Are there new checks included in this PR? Yes / No

If so, do we need to update permissions for the provider? Please review this carefully.

[x] Review if the code is being covered by tests.
[x] Review if code is being documented following this specification https://github.com/google/styleguide/blob/gh-pages/pyguide.md#38-comments-and-docstrings
[x] Review if backport is needed.

License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

We are not going to deprecate this, we will refactor it for the API to use it.


Publish 1.0 release

After https://github.com/getditto/DittoAndroidTools/issues/80 is merged and we are ready to publish in a less manaul way, we want to move towards a coordinated publish of all the tools and align the versions of each tool to be the same. This is a great time to make a 1.0 release.

@zmarkan did this today 🎉


New TTS multilingual model with voice cloning

I love this webui!
Coqui released XTTS that will be ideal for sonitranslate
https://huggingface.co/spaces/coqui/xtts

Hi @4eJIoBek1
I'll definitely explore how I can implement it effectively to enhance the sonitranslate experience. Your input is much appreciated!
Thank you

Bark tts good too

Another simpler option is to imitate the tone of the voice, to distort without changing the duration
https://stackoverflow.com/questions/40564451/how-to-change-pitch-and-tempo-together-reliably-with-ffmpeg

I made a simple example of how you could look for the audio section and detect the tone of the voice and change the tone created to imitate the original voice:
https://gist.github.com/b4zz4/312bb5c17668f9e600c57fd0857c8640
This function already has the displacement in the second of start and end.
Something positive would be to separate the voice from the ambient sound using spleeter (or something similar) to just use voices without ambient sound.


Improve Readme and convert newline to Unix style

The readme had formatting problems, the wget pointed to the wrong URL.
Using the git-splits without chaning the newline style (from \r\n to \n) resulted in a error saying that /bin/bash^M is not a valid interpreter.

Thanks for your help @ilario . I updated the README with Markdown fixes and added the wget fixes. See PR10 fix and linted markdown and PR 8


Remove globals?

Hi @tjdevries, will it be possible to remove the globals definition below? I'm in the process of writing a theme using colorbuddy and my nvim config uses some of the globals (especially the single letter ones. For example, g is vim.g )
Seems like this is a common occurrence in similar nvim configurations I've looked at.
https://github.com/tjdevries/colorbuddy.nvim/blob/cb9e044ebf0b690a56ebb4018cf300703f8dbec1/lua/colorbuddy/init.lua#L34-L47

Fixed :) I just removed them. It was from a time where I was a lot less of a thinker haha


Includedefault_memory in memory overhead

Description
Currently, the pod overhead is defined by the following code:
			// Use same values for Pod Overhead as upstream kata-deploy using, see
			// https://github.com/kata-containers/packaging/blob/f17450317563b6e4d6b1a71f0559360b37783e19/kata-deploy/k8s-1.18/kata-runtimeClasses.yaml#L7
			Overhead: &nodeapi.Overhead{
				PodFixed: corev1.ResourceList{
					corev1.ResourceCPU:    resource.MustParse("250m"),
					corev1.ResourceMemory: resource.MustParse("350Mi"),
				},
			},

This is consistent with the way downstream does it (except that the comment needs updating, the correct link would be https://github.com/kata-containers/kata-containers/blob/main/tools/packaging/kata-deploy/runtimeclasses/kata-runtimeClasses.yaml#L9, and we need to explain that 350Mi was closer to what we measured in OpenShift, upstream uses 160Mi).
However, the memory overhead seems largely insufficient, since the default Kata configuration indicates 2048M. This leads to situations where qemu or virtiofsd can get killed due to hitting the cgroups memory ceiling, which is computed by the orchestrator by summing the overhead and the limits for the containers.
The problem is documented with some additional details here: https://issues.redhat.com/browse/KATA-1501.

/remove-lifecycle rotten


trial_keeper log does not flush

Short summary about the issue/question:
nni_log doesn't flush. Log can be sometimes missing (still in buffer). Very confusing.
Brief what process you are following:

How to reproduce it:

nni Environment:

nni version:
nni mode(local|pai|remote):
OS:
python version:
is conda or virtualenv used?:
is running in docker?:

need to update document(yes/no):
Anything else we need to know:

https://github.com/microsoft/nni/pull/2156


Problems parsing timestamps

Hi,
our gpsd does not return an ISO formatted date for timestamps (only tried TPV)  but instead the epoch seconds as a floating value. Does it make sense to add that case as a fallback to the AbstractResultParser.parseTimestamp or is our gpsd out of spec? As a workaround I subclassed the original ResultParser.
Best regards
Lars

I will look into it. What setup are you using?
Maybe the timestamp format can be made configurable.

We have gpsd 2.95 (protocol version 3.3). At the end I added the output of the POLL command.
Beside the format of the timestamp the poll-structure has a "timestamp" element while in the code the result parsers try to read a "time" element.
Maybe it's possible to have only one parser that can handle all gpsd versions so that the client code must not be changed in case of using a different gpsd.
I could not find a setting for the gpsd to change the timestamp format.
{"class":"VERSION","release":"2.95","rev":"2015-12-04T15:55:02","proto_major":3,"proto_minor":3}
{"class":"POLL",

"timestamp":1500970324.416,

"active":1,"fixes":[{"class":"TPV","tag":"GLL",
"device":"/dev/ttymxc2","time":1500970334.000,"ept":0.005,"lat":52.493798333,
"lon":13.384416833,"alt":71.900,"epx":31.964,"epy":13.796,"epv":30.797,"track":0.0000,
"speed":0.004,"climb":0.000,"eps":63.93,"mode":3}],"skyviews":[{"class":"SKY","tag":"GLL",
"device":"dev/ttymxc2","xdop":2.13,"ydop":0.92,"vdop":1.34,"tdop":1.03,"hdop":2.32,"gdop":2.87,
"pdop":2.68,"satellites":[{"PRN":2,"el":0,"az":0,"ss":28,"used":false},
{"PRN":12,"el":51,"az":89,"ss":35,"used":true},
{"PRN":24,"el":18,"az":156,"ss":38,"used":true},
{"PRN":25,"el":85,"az":337,"ss":36,"used":true},
{"PRN":31,"el":32,"az":307,"ss":39,"used":true}]}]}]}

The code change was in this commit: https://github.com/taimos/GPSd4Java/commit/7244f1673faac20c25082cfd34862c204fa877b8
Since then only gpsd >= 3.5 was supported.  I will have a look to see if some automagic is possible. Is there a reason to use such an old version?

Unfortunately there is a reason... we have to use an older embedded linux that can't be updated.
I thought for older gpsd versions there is the LegacyResultParser, but doesn't seem to work either.
It's ok if the client doesn't want to support old gpsd versions because I can work around it, but maybe you could point that out in the README.

I think supporting an older version is currently not planned if you have a way around it.
But would https://github.com/taimos/GPSd4Java/tree/deduplicate_code be of any help? So subclassing the ResultParser would be much easier.

Now the parsers look much clearer ;) But does it make sense to keep the LegacyParser since you don't want to support older versions? E.g. it doesn't use the correct timestamp field ("time" instead of "timestamp" - or maybe the "timestamp" version it too old to support?).
If you want to keep it maybe you could add the legacy code to the regular ResultParser and use the old code as fallback, something like this:
if (jsonObjectHasField("sky")) {
   doSomethingWithField("sky"); // new version
} else {
   doSomethingWithField("skyviews"); // old version
}

With that the user doesn't have to care about anything - it just works without changes for all supported gpsd versions. You could also add a constructor with host + port, without parser (and without the backward compatible IOExceptions ;))

Ok, I implemented it. Do you have the chance to test the version in the branch?

Yes, thanks, I can test against our old gpsd.
Maybe you could get rid of the ugly IOException for the new constructor since it cannot be thrown anymore because of the reconnect-fix in 1.9. I guess you have to keep it for the old constructors that already existed in 1.8 for binary backward compatibility. But it would be nice to not have them in the new code.

Ok, I removed the exceptions on the new constructors and left it on the old one.

Everything works so far - except the original problem this ticket was for ;)
A fallback to parse the text as a double in the AbstractResultParser.parseTimestamp() would be nice if the formatter raises an exception.

if (json.has("time")) {
	poll.setTimestamp(this.parseTimestamp(json, "time"));
} else if (json.has("timestamp")) {
	poll.setTimestamp(json.optDouble("timestamp", Double.NaN));
}

It should read the timestamp field and parse it as double, or did I misunderstand your format?

All fields representing timestamps come as double values, not only the "timestamp" field in the poll.

OK, I added a fallback to double.

Thanks, works ;)
One thing. With the if/else-if it is now possible, that the setter is not being called. Especially for list properties it's now possible that you have a null list at the end (if both field alternatives are not present). Before you always had a non-null list, even if the field was not present. Maybe replace the else-if with a simple else?

If the field is not present wouldn't this lead to the setter being called with null, too?

I guess it depends on the type. At least for list objects the setter is called with an empty list if the field is not present (parseObjectArray() does that).
poll.setSkyviews(this.parseObjectArray(json.optJSONArray("skyviews"), SKYObject.class));

ok, will add another else block to set an empty list. Then it is explicit.

even better ;)

comitted. If this is it I will merge the complete branch to get all changes to master.

Yes, that's it - and the not planned support for older versions it (at least partially) there ;)
Thank you!


How to handle tinymce content with react

Suppose we have some user content that is linking, for example, a product in a web shop. Now a click on thet link will make the whole js app downloaded again. It would be great if react could handle that as well. Can that be done?

@ceracera
I spent couple of days and can't solve it.

The idea was to replace links in HTML with React Router NavLink.
For now it's not a priority issue for me.


Api url?

I cant find the api URL is this is not the same as on https://github.com/dsmrreader/dsmr-reader? Correct?

You mean how to add this repository to home assistant? Check the Readme on https://github.com/sanderdw/hassio-addons

No i do use an Remote datalogger. I need to configure an url for the remote and https://homeassistant.local doesn't work

@domoticaDude, sorry i missed the update. Did you fix it in the mean time?
https://homeassistant.local is most likely not correct, you want to send it to another DSMR instance?
These are my settings for example:

What I did was expose the applications port 80 on port 7777 of the host. like so:

Then you can use http://homeassistant.local:7777 as your base url.. Not sure if there's a better way :)


scripts: pyocd: Add pack option to support device custom pyocd packages.

Add pack option support to support device with custom pyocd packages.
pyocd have an option for specific pack when run command.
But our project not support it.
Some vendor provide pack without upload to official CMSIS-Pack list
Use pack option we can specific the pack file like this:
board_runner_args(pyocd "--target=gd32a503vd" "--pack=${ZEPHYR_HAL_GIGADEVICE_MODULE_DIR}/${CONFIG_SOC_SERIES}/support/GigaDevice.GD32A50x_DFP.1.0.0.pack")

Why is this required? Once a pack is installed, there's no need to specify it every time.

I refer to pyocd document: Manual pack usage and did not find a way to install pack method except config pack path in pyocd.yml file.
I think only Managed packs can install directly, but not every pack in CMSIS-pack list.
In addition to Gigadevice's gd32a503, there are other vendors who also provide pack files only on their official websites, without uploading to CMSIS-pack. (e.g. artery, etc.)
The same problem occurs when maintaining the out-of-tree devices from these vendors.
I believe adding the pack option to pyocd would be the good way to solve this problem.

why do we need to host packs when there's a repo?

I think this question is not directly related to this PR. About host pack file or not in hal repo, I suggest discussing it at https://github.com/zephyrproject-rtos/hal_gigadevice/pull/30.
gigadevice provide GD32A50x_DFP on their website but they don't upload it to  CMSIS-pack list,
so it can't install directly.
Each user can flash and debug their devices only after downloading the pack file from the gigadevice website and configuring their `pyocd.yml'.

The GD32A503 series release at 2022 September release CN, release EN. I think this just about the cooperate, there have a publish time dely for new pack list on Keil.
So I recommand use others to instead, like jlink, and wait it sync at Keil.

I think --pack should be added for the following reasons.

This is an option supported by the pyocd program that is not available in the zephyr project due to argument checking.
To be honest, when I first met the problem runner pyocd received unknown arguments, it took me some time to notice that pyocd in zephyr and the debugger program pyocd were two different programs with similar arguments. This inconsistent behavior can be confusing to the user.
Not only for in-tree devices, but when maintaining out-of-tree devices, you may encounter cases where the vendor does not upload the package to the CMSIS-pack list.
Maybe for Gigadevice this is a release delay, but for some vendors it may never be released.
You can set the pack option to be correct via the pyocd configuration file. But currently only absolute path references are supported, no environment variables are allowed. So I think the --config option is not enough.
I think adding the --pack option is easy, just pass it to the pyocd program and that's enough.

This is an option supported by the pyocd program that is not available in the zephyr project due to argument checking.
To be honest, when I first met the problem runner pyocd received unknown arguments, it took me some time to notice
that pyocd in zephyr and the debugger program pyocd were two different programs with similar arguments. This
inconsistent behavior can be confusing to the user.

To get runners description of current build board, use below command.
west flash --context
Also, update the test case in zephyr/scripts/west_commands/tests/test_pyocd.py

Have you looked into using --tool-opt for this instead?

Have you looked into using --tool-opt for this instead?

You are right, --tool-opt works well.
I missed this option before.

board_runner_args(pyocd "--target=gd32a503vd" "--tool-opt=--pack=${ZEPHYR_HAL_GIGADEVICE_MODULE_DIR}/${CONFIG_SOC_SERIES}/support/GigaDevice.GD32A50x_DFP.1.0.0.pack")


Compilation fails if output directory does not exist

https://github.com/nanoc/nanoc/pull/946#issuecomment-250913802

Fixed in #949.

👍 Thanks!


delete directory .DS_Store

this dir is not needed

Congrats on your first useful PR ;)
.DS_Store should also be added to .gitignore to avoid accidentally adding it again..

Succes! All is OK!