missing nonce when running against pebble

Hi!
I have the following local test setup.

Create a key-pair for pebble via:

mkcert localhost

Run pebble via:

docker run \
		--rm -it \
		--name pebble \
		-p 14000:14000 \
		-e PEBBLE_VA_NOSLEEP=1 \
		-v ${PWD}/acme/pebble-config.json:/pebble-config.json:ro \
		-v ${PWD}/acme/localhost-key.pem:/localhost-key.pem:ro \
		-v ${PWD}/acme/localhost.pem:/localhost.pem:ro \
		letsencrypt/pebble pebble \
			-config /pebble-config.json \
			-dnsserver coredns:53 \
			-strict

with the following pebble-config.json:
{
    "pebble": {
        "listenAddress": "<IP_ADDRESS>:14000",
        "managementListenAddress": "<IP_ADDRESS>:15000",
        "certificate": "/localhost.pem",
        "privateKey": "/localhost-key.pem",
        "httpPort": 5002,
        "tlsPort": 5001,
        "ocspResponderURL": "",
        "externalAccountBindingRequired": false,
        "domainBlocklist": [],
        "retryAfter": {
            "authz": 3,
            "order": 5
        }
    }
}

Then when running the following Rust code:
use anyhow::{anyhow, Context, Error};
use instant_acme::{
    Account, AccountCredentials, Authorization, Challenge, ChallengeType, Identifier, NewOrder,
};

#[tokio::main]
async fn main() -> Result<(), Error> {
    let acme_provider_url = "https://localhost:14000";
    let acme_account_id = "blah blah";
    let acme_account_key = "blah blah";

    let acme_credentials: AccountCredentials = serde_json::from_str(&format!(
        r#"{{
            "id": "{acme_provider_url}/acme/acct/{acme_account_id}",
            "key_pkcs8": "{acme_account_key}",
            "urls": {{
                "newNonce": "{acme_provider_url}/acme/new-nonce",
                "newAccount": "{acme_provider_url}/acme/new-acct",
                "newOrder": "{acme_provider_url}/acme/new-order"
            }}
        }}"#,
    ))?;

    let acme_account = Account::from_credentials(acme_credentials)
        .context("failed to create acme account from credentials")?;

    // Get Order
    let mut order = acme_account
        .new_order(&NewOrder {
            identifiers: &[Identifier::Dns("example.com".to_string())],
        })
        .await
        .context("failed to create new order")?;

    Ok(())
}

I get the following error:
Error: failed to create new order

Caused by:
    missing data: no nonce found

I tracked it down to the fact that the initial new_order call doesn't send a nonce. Any idea if this is the expected behavior or whether the library should first retrieve a nonce via /newNonce?
Wondering if I'm just doing something wrong here when running against pebble?

Ok, I think I'm narrowing down on the problem - I am using the following to create the AccountCredentials:
let acme_credentials: AccountCredentials = serde_json::from_str(&format!(
    r#"{{
        "id": "{acme_provider_url}/acme/acct/{acme_account_id}",
        "key_pkcs8": "{acme_account_key}",
        "urls": {{
            "newNonce": "{acme_provider_url}/acme/new-nonce",
            "newAccount": "{acme_provider_url}/acme/new-acct",
            "newOrder": "{acme_provider_url}/acme/new-order"
        }}
    }}"#,
))?;

which hardcodes the endpoint paths... Pebble uses different paths on purpose... e.g. /nonce-plz.

Right, so you provided an incorrect newNonce path and the error about that is kind of opaque. See #31 for an attempt to somewhat improve the error handling here.

@djc So one thing I've noticed - when creating a new account, instant-acme will retrieve the correct URLs from the ACME server... That said, there seems to be no way at the moment to import an account without also specifying the URLs (this is because importing/exporting is just done via serializing/deserializing the AccountCredentials struct. It would be nice if there was a way to import an account just via a subset of fields, e.g the provider URL and/or the account key, and still have the crate retrieve the correct URLs from the directory.

Why do you want that? The library considers the serialized data mostly as an opaque blob, and doesn't currently keep the the provider URL around at all in the Client/Account context (because it isn't needed).

How are you storing/passing in the key? I'm open to providing API for constructing an Account with a provider URL, ID and existing key, but the main issue I saw there is that it wasn't obvious to me how people are storing their keys.

Hi @djc, we have a utility for generating credentials, create-acme-credentials that uses instant-acme just to create and output credentials to stdout. We only grab the account ID and key from this output.
We then use those for running multiple instances of a service, so that they all use the same ACME credentials, i.e:
my-service \
    --acme-provider-url "${ACME_URL}"
    --acme-account-id "${ACME_ID}"
    --acme-account-key "${ACME_KEY}"

That said, that's not enough information, we would need to provide the paths for new-nonce, new-order, etc. We could do this, although I argue that it'd be better if the library just retrieved the paths itself.

What I guess I'm confused about is why your utility only captures part of the serialized AccountCredentials instead of the whole thing? That is the way it was intended, as the documentation explains:

This opaque type contains the account ID, the private key data and the server URLs from the relevant ACME server. This can be used to serialize the account credentials to a file or secret manager and restore the account from persistent storage.

What I guess I'm confused about is why your utility only captures part of the serialized AccountCredentials instead of the whole thing?

You're correct that if we just use the opaque credentials blob, this is resolved. And I suppose we could do that. Although to be fair, it also seems reasonable to have the crate fetch the actual directory when creating an account.
I think the ideal scenario would be that the crate exposes a builder that lets you specify whatever you want.

That doesn't solve the problem of how the key should be serialized. You also didn't really answer the question of why your code doesn't store the all of the AccountCredentials today.

You also didn't really answer the question of why your code doesn't store the all of the AccountCredentials today.

For various reasons, our service can't accept a file as a cli argument, therefore we give it the account ID and key instead, which seems like a pretty common thing, just as common as providing credentials via a file.
I'm sure we could work around this limitation, e.g. provide the file contents directly, etc, we just can't have the service read the file, unfortunately, which is why I brought this up.

See if #34 makes things easier for you?


[FEATURE] Add a ChatBot Assistant

I want to add a ChatBot Assistant..
@ANSHIKA-26  please assign me this task.

I would like to work on this project , could you please assign this issue to me??

pr #894 is still working on it ig, so cant assign it to you

issue #215 ,

pr #894 is still working on it ig, so cant assign it to you

Ooo

???
On Sat, 19 Oct, 2024, 12:39 am Subhajit-2023-44, @.***>
wrote:

pr #894 https://github.com/ANSHIKA-26/WordWise/pull/894 is still
working on it ig, so cant assign it to you
Ooo
—
Reply to this email directly, view it on GitHub
https://github.com/ANSHIKA-26/WordWise/issues/1166#issuecomment-2423079452,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3GQXJIHE2XXECQRRHO4L4LZ4FMIJAVCNFSM6AAAAABQGQKPPCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRTGA3TSNBVGI
.
You are receiving this because you commented.Message ID:
@.***>


Implement a Codepoints iterator for each encoded string variant in spinoso-string

Push the branching down into the encoded variants.
Followup to https://github.com/artichoke/artichoke/pull/1678.

I'd like to work on this one, if nobody is curretly working on it 🤚


[Plan Item] Propose new Webview API

Follow up on #41047 which explored improving the webview API. Part of #28263
This iteration we'd like to move the new API to be a fully documented proposed API and also self-host the markdown extension on the new API

Initial version of API taken into proposed with #42690

Proposed API changes for next iteration:

Add a Webview.viewColum property: readonly viewColumn?: ViewColumn;.

Rename WebviewOptions.keepAlive to make it more clear what this property actually does

Rename Webview.onMessage per https://github.com/Microsoft/vscode/issues/28263#issuecomment-366036703

Move from Webview.onDidBecomeActive/Webview.onDidBecomeInactive to an API closer to window.onDidChangeActiveTextEditor.  Current proposal: window.onDidChangeActiveWebview. Would be fired with undefined when a user switches to a normal text editor. Also would be fired when a webview changes columns.

// cc @ivankravets @eamodio @pavelfeldman

I've tried using this, unfortunately I'm using localstorage and that doesn't seem to be allowed.
Uncaught DOMException: Failed to read the 'localStorage' property from 'Window': Access is denied for this document.

Deleted the comment, was using localstorage, don't need it as I can persist the webview.
👍

Can we have API for icon/favicon of the webview?

@DonJayamanne Just sent out a new proposal that introduces an onDispose event on the webview: #44307
This PR introduces a different model for webviews. There is now a 1:1 correspondence between  webviews and "webview editors". Previously, users could split webview editors, resulting in two different webview editors both backed by the same model. This is no longer possible

@mjbvz Thanks for the feature, really appreciated.
Now, in my 3D Viewer extension, I'm trying to reference files both from vscode-extension-resource (javascript and png files) and vscode-workspace-resource (a 3D model to display), but get the following error in the Webview Developer Tools

three.js:30777 XMLHttpRequest cannot load vscode-workspace-resource:/Users/slevesque/Documents/Projects/three.js/examples/models/collada/elf/elf.dae. Cross origin requests are only supported for protocol schemes: http, data, chrome, https.

I need to set the base href to vscode-extension-resource in order to load javascript and png files, and that part works fine. But it seems impossible afterward to load workspace assets from javascript code.
Any way to authorize cross reference between vscode-extension-resource and vscode-workspace-resource ?

Thanks all for the feedback! A few initial proposed changes for this month:

Rename Webview to WebviewEditor to make it clear that it is an editor
Merge vscode-extension-resource and vscode-workspace-resource schemes into vscode-resource. This should solve the cross origin problem @stef-levesque noted.
Add option to enable find in the webview. (should find be on or off by default?)
Move onDidChangeViewColumn to be a global event

This is just a first set of proposed tweaks. We'll keep refining the API throughout the month. Let me know if you have any other suggestions or feedback
Beyond webview 1.0, I'm also thinking about how the API may be able support a few additional cases:

Restoration, i.e. when vs code closes and then reopens, we restore the opened webviews automatically. Using resources uris for this would probably make the most sense. This ironically would be very similar to TextDocumentContentProvider which webview seeks to replace.

Splitting. Currently this is not supported. I think it could be added to the existing api pretty easily however.

No commitment on these, but I think it is good to think about how these features would play with the currently proposed API
/cc @jrieken

What percentage of the VSCode userbase uses the latest VSCode version?  One thing I'm worried about with switching to the new webview is that users of older vscode versions will no longer be able to use my extension.

Rename Webview to WebviewEditor to make it clear that it is an editor

👎

Please explain

As discussed, the WebView is not an editor but a UI-part that happens to be positioned like an editor but it must not be like that. Be prepared for showing web views in the bottom panel or as peek view

🤞 maybe even in the sidebar as its own activity? 😉 🙏

@jrieken Our commitment here is getting a better webview api for editors and I do not think we agreed to support webviews other contexts. I see a few options on updating the API to future proof it for such without blocking the original goal:

Rename Webview to WebviewEditor. This gives us room to later introduce a new Webview that can be used in other UI contexts. This is the easiest

Try to make the current Webview interface more generic. We still would need the concept of a WebviewEditor however for events such as onDidChangeActiveEditor (or we would need to seriously rethink those) so this ends up being very similar to the next proposal.

Split the concept of a webview out of the existing Webview (which would renamed to WebviewEditor).

export interface Webview {
    title: string;

    html: string;

    readonly onDidReceiveMessage: Event<any>;

    postMessage(message: any): Thenable<boolean>;

    ...
}

export interface WebviewEditor {
    readonly viewColumn?: ViewColumn;

    readonly webview: Webview;

    ...
}

If we go with something like the third approach, I'd be interested in exploring a webview editor API that is more like TextDocumentContentProvider but without the baggage of a text model:
interface ContentProvider {
     provideContent(uri: Uri, token: CancellationToken): Webview;
}

function registerContentProvider(scheme: string, provider: ContentProvider): Disposable

as this would let us implement restoration.

do not think we agreed to support webviews other contexts

I don't think agreement is need here but projecting what will happen in the future. The request for webviews everywhere will come (if it's not already there), we can ignore it for a while, and then we'll implement it. Knowing that, we should design for it which means don't plan for deprecation nor duplication.

This gives us room to later introduce a new Webview that can be used in other UI contexts. This is the easiest

That's planning for duplication.

e still would need the concept of a WebviewEditor however for events such as onDidChangeActiveEditor (or we would need to seriously rethink those) so this ends up being very similar to the next proposal.

I might be missing something but I believe we need activeTextEditor, activeWebview, activeTreeView, activeTerminal etc. We can do that, plus all events and the enumerable types of all xyz-views. Or we come up with something more generic, like a part which is one of the pieces, e.g. type UiPart = TextEditor | WebView | TreeView | any. With that would come one event that and one enumerable types, ala parts: UiPart[] and onDidChangeActivePart: Event<UiPart>. These are early ideas which need harding but I think it's the better approach.

webview editor API that is more like TextDocumentContentProvider but without the baggage of a text mode

Interesting. It would be similar to the pull model the tree view is using. I have discussed something similar with @sandy081 today because tree views started with the pull model (data provider) and are now mutating into a push model

Just for reference the request for webviews in other contexts does already exist: https://github.com/Microsoft/vscode/issues/43049 (there are probably other requests as well) and TBH I have a very real need for that support (a webview in at least the sidebar or panel) for a new extension I'm working on.

@mjbvz Is there a way to focus a webview when its already visible? The reveal method only seems to focus if the panel isn't already in the requested column.

This is what reveal should do. Are you on insiders?

Yeah today's. It will focus it if it changes viewcolumns but if it is already in col 3 but another tab in col 1 or 2 is focused and I reveal to col 3 it doesn't focus it

Can you open an new issue. Also are you calling it with the view column: .reveal(webview.viewColumn)?

Out right now but will do when I get back. Basically. .reveal(ViewColumn.Three) when webview.viewColumn is also 3


[AzureMonitor] update internals in test project

This PR is preparing for future work.
I need internals in the Exporter project to be visible to the Distro Tests project.
This had some unrelated conflicts, so I'm moving these changes to its own PR.
Changes

make Exporter internals visible to Distro Tests project
The DefaultPlatform class is used in both Exporter and Distro. I needed to change the name of the Distro copy to prevent name conflicts in the Disto test project.

API change check
API changes are not detected in this pull request.


Unauthorized: access token has insufficient scopes

Description
In github actions, when I pull some images from AWS-ECR, I got error message below.
unauthorized: access token has insufficient scopes

Solved
The main reason for exception is that I locate wrong url with ECR repository! So I change URL to legitimate URL


Property displayField is required while creating new content type

Property displayField is required while creating new content type and it's missing in the examples.
Otherwise after visiting the console, console set the displayField under the hood, and you get asked about saving changes (without doing any).
Btw. why you disabled adding the screens?.....

This issue had no activity for some time now, please feel free to create a new one if you still have the issue


Fix spec

Attempt to fix #2 and close #1.
Spec HTML viewable here
This attempts to fix the spec by:

spec-refactor of Promise.resolve logic so the meat of it is available as an abstract operation
use that abstract operation to produce a promise with certainty inside PromiseReactionJob, and resolve that with a thunk for the intended "argument".

@domenic any thoughts on this solution?


Use of custom parent node as scroller div

Maybe it is a usage question.
How could I use custom parent node instead of an immediate parent element as scroller div? I have used something like below, but it's not working for me. Please guide me with an example.
<InfiniteScroll parentNode={document.getElementById("pageContentHome")}>

This component does support that, though you need to override the class member for that to work. Check https://github.com/CassetteRocks/react-infinite-scroller#custom-parent-element for more info. Hint: class YourComponent extends InfiniteScroll {....
In my opinion this is a design mistake and, as one would expect with React, the function should be passed as prop - as one would expect it.


editor, Linux: make i3/4/5/686 version, or at least label the download as x64

Please compile for i386? I was unpleasantly surprised after the download that it wasn't the correct binary for my system :(

Yes, could you confirm if it's 64 bit only?

yes it is 64 bit only right now, i've updated the download page to indicate this https://github.com/processing/p5.js-website/commit/b7f299609c18c525a88a3e53e7d9c8463924c4f4

@lmccart the thing stopping me from creating a working 32 bit build is the ffmpeg library. So I should be able to make 32 bit builds but sounds won't work until those libraries are found/built which seems quite involved. This is a downside of nw.js compared to electron which has the right codec support out of the box.

I think Electron doesn't have 32bit support, in Windows at least. There's a larger problem here in that new toolkits/frameworks are nullifying 32bit out of the box and giving devs no choice.
Anyway 32bit isn't an XP issue, no one disagrees XP is a security sinkhole. Linux on the other hand produces current and maintained 32bit systems and they're valuable for the literally millions of 32bit machines still out there.


Behavior & attendance incidents graph: List month's incidents on mouseover.

This is a further improvement to #343.
Talking with Jill Geiser today, she mentioned that it would be nice to display the list of a month's incidents when we mouseover on the column for that month.
For discipline incidents, we could also do the jump-to-incident feature (#196) here via tooltip.

👍
Thanks for capturing @erose!

Prototype version of this finished and merged in https://github.com/studentinsights/studentinsights/pull/412.


chore(polar_area_chart_plugin): bump up to 9.6.0

Update Dashboard plugin to GoodData SDK v9.6.0. More information about the new SDK version can be found in GoodData docs

Build succeeded (check pipeline).

Build succeeded (check pipeline).

Sonar scan result
More detail, see in https://sonarqube-gate.intgdc.com/dashboard?id=gooddata-dashboard-plugins-gate-PR63
To scan for vulnerabilities in dependencies and run unit tests (to get coverage report in sonar) please comment your PR with 'extended check sonar'.

Build succeeded (gate pipeline).

https://checklist.intgdc.com/job/compliance/job/gooddata-dashboard-plugins-sonarqube-scan-zuul-gate/56/ : SUCCESS in 39s
https://checklist.intgdc.com/job/compliance/job/gooddata-dashboard-plugins-fossa-scanning-zuul/33/ : SUCCESS in 50s (non-voting)
https://checklist.intgdc.com/job/compliance/job/gooddata-dashboard-plugins-merge-compliance-check/57/ : SUCCESS in 9s


[BootNormal] Fix topic message being shrunk when calling custom onMes…

[BootNormal] Fix topic message being shrunk when calling custom onMessage callback
When using getMqttClient feature according to order onMessage callback are called
the homie one shring topic buffer (by adding '\0') and so following custom callback
doesn't get full topic
Related to Issue #243

Did not think about this. Thanks! :+1:


What's the content like in $LighTrack/dataset/VOT2019/list.txt file?

agility
ball2
basketball
bolt1
ants1
ball3
birds1
...
content above not ok to run

agility
ants1
ball2
ball3
basketball
birds1
bolt1
book
butterfly
car1
conduction1
crabs1
dinosaur
dribble
drone1
drone_across
drone_flip
fernando
fish1
fish2
flamingo1
frisbee
girl
glove
godfather
graduate
gymnastics1
gymnastics2
gymnastics3
hand
hand2
handball1
handball2
helicopter
iceskater1
iceskater2
lamb
leaves
marathon
matrix
monkey
motocross1
nature
pedestrian1
polo
rabbit
rabbit2
road
rowing
shaking
singer2
singer3
soccer1
soccer2
soldier
surfing
tiger
wheel
wiper
zebrafish1


add static typechecks

From one wide-ranging-dev-changes pull request to the next - now we have code style, I'm starting work on issue #56.
First step: get mypy to not throw errors on the current code, then I'll add a GH workflow for it for future branches. I've based this off the #111 PR branch, because I don't want to bother typing the entire channels cog when it's about to be removed.

once more, this time with feeling...

Do we want to use list, tuple, etc instead of List, Tuple, etc. I only learnt recently that are already included (no need for Typing) from Python 3.9 onwards,

I'm in favour of standardising and using the typing types for everything, rather than mixing and using some inbuilt types and some typing types.

I intend to get this PR merged as-is. The changes so far allow use of poetry run pyright uqcsbot to statically typecheck commands, or ... pyright uqcsbot/file.py for specific files.
My intention is to require that pyright types all new modules from here on out, including the proposed pastexams, dominos, morse, etc. rather than merge them now and just leave the typing until later.

Seems mostly fine, it appears that your poetry.lock needs to be updated though
This PR intentionally updates poetry.lock - we've got 5 new modules for typing and stubs, so I took the opportunity to bump versions on a bunch of other libraries so dependabot stops whinging.


reduced vertical spacing in default theme

this was stimulated from this particular example, where I realized that - at least for terminal output - the vertical spacing is not ideal.
before:

after:

in the current fix instead of just fixing the terminal output the vertical spacing is reduced everywhere (in a body css selector in used in default theme). Not sure if this is welcome or not (it helps packing more content vertically, it might decrease readability). I want to have a look on how this would look in nimib documentation and then decide if I want to restrict it only on terminal output (but I would need to learn the css selector that makes it work).
as a reminder, when this is ready it will need a:

[ ] changelog line

Nope, this does not work.  If I tried changing the line-height of the samp tag, it did nothing. But adding it to the pre tag seems to work. The problem is that it affects all pre tags, including the one around code blocks.
There are two solutions in my mind:

Create a output-pre class and assign it to the pre tag.

output-pre {
  line-height: 1.15;
}
--------
doc.partials["nbCodeOutput"] = """{{#output}}
<samp>{{output}}</samp>
{{/output}}"""

Inline the style directly in the mustache template

doc.partials["nbCodeOutput"] = """{{#output}}
<samp>{{output}}</samp>
{{/output}}"""

Thanks for looking into this. Found a third option, that should work (have not tested it): using pre:has(> samp) css selector, see:

https://stackoverflow.com/questions/1014861/is-there-a-css-parent-selector
https://caniuse.com/css-has

But this is used currently on 56% of users.
So I would go with one of your two options: the class seems cleaner (and more reusable) but it can break more easily since there is code in two places (eg in a new theme block output will likely not be customized but you are likely not using the nbCodeStyle). Still I think I would go with the class option.

Thanks for looking into this. Found a third option, that should work (have not tested it): using pre:has(> samp) css selector, see:

I found it as well, but given that I use Firefox both on my laptop and on my phone, it fell away quite quickly :upside_down_face:

So I would go with one of your two options: the class seems cleaner (and more reusable) but it can break more easily since there is code in two places (eg in a new theme block output will likely not be customized but you are likely not using the nbCodeStyle). Still I think I would go with the class option.

Sounds good to me :+1: I'll have to manually fix it either way in nimiSlides no matter how we do it.

implemented your class solution (naming the class nb-output). let's see how it looks in the preview...

example of before/after taken from penguins:

looks fine to me. Thoughts before I merge?

The left one looks better to me, so hope that is the new one :rofl:  :speak_no_evil:

Haha yes, it is indeed an after/before


[Bug]: Farthest Frontier: Could not load TimeManager from assembly

All of the following criteria must be met

[X] All Requirements must be installed.
[X] Changed the title so that it doesn't just says "[Bug]: "
[X] I have searched the GitHub issues for my bug, even in the closed issues.

All of the following are optional to answer

[X] Tried reinstalling the Game.
[X] Tried reinstalling MelonLoader.
[X] Tried restarting PC.
[X] Was able to see the Start Screen.

Describe the issue.
Latest.log
I keep getting the mentioned TimeManager error when launching. The game plays but does not load any mods. I have re-installed everything and have VB22 community installed.
Did you attach your log file?
Yes, I attached my log file to the text box above.

Those are issues with mods not made by us. You should contact the mod creators.


Always build iOS Archive builds in release mode

Archive builds for upload to iTunes Connect should always be built in
release mode, not doing so results in crash on startup.
Bug: https://github.com/flutter/flutter/issues/12086

There's a bit more work here to support this properly. Most importantly, even with an override of FLUTTER_BUILD_MODE and FLUTTER_FRAMEWORK_DIR, we need to ensure we've run a pod install that picks up the release-mode framework.
For now, I've updated to just emit an error with instructions and exit with a build failure. Long-term, I think the right thing to do is make the Xcode build more hermetic and reduce the dependency on a previous command-line run.

Still LG


Add a Gitter chat badge to README.md

Azollas/org.azolla.p.roc now has a Chat Room on Gitter
@Azolla has just created a chat room. You can visit it here: https://gitter.im/Azollas/org.azolla.p.roc.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

manual


update styling of file input button

This makes the file input button look hand drawn.

Thank you! Will test and merge tomorrow.

Thanks, I made a couple of tweaks to give it the correct border etc.


Enable use of wildcard location configurations

This PR is a fix for #561
It makes the location configuration work as follow. Let say the domain is foo.bar.baz.example.tld:

if a config file with the exact same name as your domain (foo.bar.baz.example.tld) is present, use it.
else try every corresponding wildcard domain, from most qualified to least qualified, first ascending through the labels to the base domain (*.bar.baz.example.tld, *.baz.example.tld, *.example.tld) then descending (foo.bar.baz.example.*, foor.bar.baz.*, foo.bar.*, foo.*). The first matching config file will be used.
if no wildcard location config file was found either, use the default location config file.

corresponding tests

Previously to achieve a wildcard SSL for all subdomains I have been using the following example. Two A records are needed to make this work. Note I do not force redirect from non-www to www.
docker run -d -e VIRTUAL_HOST=.example.com -e LETSENCRYPT_HOST=www.example.com,example.com -e<EMAIL_ADDRESS>--network=exampleproxy --name example.com -v /srv/www/nginx-data/html/example.com:/usr/local/apache2/htdocs httpd:alpine
With the changes you made (thank you for your work) am I am able to to change my current syntax to -e VIRTUAL_HOST=example.com -e LETSENCRYPT_HOST=example.com to achieve the same scenario? The issue I had before is the NGINX config had to manually modified to make this work.


Improvements and optimizations on modal windows

As suggested by @pkdash , in resource landing page, Instead of generating a modal window for every item we can use just one and pass the corresponding variables.
This will help avoid scalebility problems with the HTML.

I was thinking the same thing for anywhere we list resources, such as Groups. Let's talk. Also, is there a use-case for Signals or some sort of push notification?

I want to point out that we have starting with these optimizations with the rewriting being done as part of #3186

Stale


Allow conditionally hidden fields

The hidden/0 view callback is handy to remove sensitive fields in views. In some cases, an API may want to hide certain fields by default, but show them in some cases (in my application's case, some fields should be visible if specifically requested as part of a sparse fieldset).
It would be handy if the hidden callback accepted conn as an argument, so the application could decide what fields to hide for a particular request.
Any thoughts on this? I'm happy to open a PR for the change.
Related issue: #68

that sounds like a reasonable solution as long as you don't need to omit any relationships (really, anything outside of attributes) conditionally. just cuz the hidden/0 function only operates on fields to begin with.


With Obsidian 1.6.4, Minimal requires second click to activate some Obsidian UI elements

Is the bug present when using the default Obsidian theme?
No
Is the bug present when snippets and plugins are disabled?
Yes
Minimal theme version
7.7.3
Describe the bug
Has something changed about the way that taps/clicks are detected on iPad in 1.6.4?
There seems to be a pattern again that when I tap on a pane that does not have focus, I have to tap a second time.
I seem to recall that used to happen a long time ago and then was fixed.
I narrowed it down to happening when Minimal is enabled.
Example steps to reproduce:

Have the file browser visible
Enable Appearance > Show tab title bar
Click on a filename in the explorer to load a file
Click in the title bar to toggle from Reading to Editing, or vice versa

Expected:

mode changes

Actual:

have to click a second time to change the mode

Copied from discord.
Debug info
SYSTEM INFO:
	Operating system: ios 15.6.1 (Apple iPad13,11)
	Obsidian version: 1.6.4 (146)
	API version: v1.6.4
	Login status: logged in
	Catalyst license: vip
	Live preview: off
	Base theme: light
	Community theme: Minimal v7.7.3
	Snippets enabled: 9
	Restricted mode: on

RECOMMENDATIONS:
	Custom theme and snippets: for cosmetic issues, please first try updating your theme and disabling your snippets. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community theme and snippets.

Can you try restarting Obsidian? I am not able to reproduce this.

Apologies for not replying sooner. I have been completely tied up with the Obsidian metadata issue.


Adds the SWAT Helmet

About the PR
What it says on the tin - adds the SWAT helmets from SS13. It's still (probably) the best standalone helmet in the game, but unlike SS13 it's not spaceproof as it's good enough as-is.
Currently, the only method of obtaining is finding the NT variant in the salvage expedition reward lootpool, but later on it can probably be mapped where appropriate.
Also adds some startgear for the NT and Syndicate footsoldiers (as seen in SS13.)
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
:cl:

add: Added the extremely robust & highly coveted SWAT helmet. You can find it as a reward in salvage expeditions.

imo this is too weak for an epic expedition reward since bombsuit helmet is 50% explosion and blunt resistance

Sounds like the bombsuit helmet is just overpowered.
That aside though, since helmets stack with armor instead of working separately like they do in SS13 due to locational damage, I mostly opted to keep the values modest as otherwise people would end up being extremely difficult to kill with both armor and a helmet.
For the record, here's the basic sec helmet in SS13:
(MELEE = 35, BULLET = 30, LASER = 30,ENERGY = 40, BOMB = 25, BIO = 0, RAD = 0, FIRE = 50, ACID = 50, WOUND = 10)
And here's the SWAT helmet:
(MELEE = 40, BULLET = 30, LASER = 30,ENERGY = 40, BOMB = 50, BIO = 90, RAD = 20, FIRE = 100, ACID = 100, WOUND = 15)


Incorrect File Name(s) (Missing 's')

Summary
TimeZone settings (& other attributes/dependencies) for some of the sub-packages or sub-modules for the apps-script-samples does not load properly due to improperly names manifest files.
operating-system@build-machine:~/Documents$ find . -type f -name 'appscript.json' 2>/dev/null

./apps-script-samples-main/solutions/automations/folder-creation/appscript.json
./apps-script-samples-main/slides/SpeakerNotesScript/appscript.json

TODO
Fix the file name(s) for these two Apps Script initialization JSON files (i.e. appsscript.json not appscript.json)!:

solutions/automations/folder-creation/appscript.json
slides/SpeakerNotesScript/appscript.json

CAVEAT:
It is realized that these files might be intentionally misspelled as a means to ignore the sub-project or sub-module while keeping version control on the files, but perhaps there is a better way to accomplish such exclusions while maintaining referential integrity.
Expected Behavior
Import of the sub-packages should reflect the proper time zones and/or other dependencies when loaded into Google Workspace's Apps Script editor.
Sample URL:  https://developers.google.com/drive/v2/reference/files/insert
Description:  Instructions for importing modules and/or projects into Apps Script for use with Google Workspace.
Actual Behavior
Import of the sub-packages does not reflect the proper time zones and/or other dependencies (e.g. included libraries) when loaded into Google Workspace's Apps Script editor.
Steps to Reproduce the Problem

Traverse the source tree with your preferred tool(s).
Determine the location of all files named appscript.json
Rename the file(s) appsscript.json in the respective directories
Import the files into the Apps Script environment within Google Workspaces
Ensure that the Apps Script editor recognizes the file name(s) appropriately and applies the appropriate configuration to the subprojects.

NOTE:  File name(s) should have 3 "p" characters & 3 "s" characters

Summary
TimeZone settings (& other attributes/dependencies) for some of the sub-packages or sub-modules for the apps-script-samples does not load properly due to improperly names manifest files.
operating-system@build-machine:~/Documents$ find . -type f -name 'appscript.json' 2>/dev/null

./apps-script-samples-main/solutions/automations/folder-creation/appscript.json
./apps-script-samples-main/slides/SpeakerNotesScript/appscript.json

TODO Fix the file name(s) for these two Apps Script initialization JSON files (i.e. appsscript.json not appscript.json)!:

solutions/automations/folder-creation/appscript.json
slides/SpeakerNotesScript/appscript.json

CAVEAT: It is realized that these files might be intentionally misspelled as a means to ignore the sub-project or sub-module while keeping version control on the files, but perhaps there is a better way to accomplish such exclusions while maintaining referential integrity.
Expected Behavior
Import of the sub-packages should reflect the proper time zones and/or other dependencies when loaded into Google Workspace's Apps Script editor.
Sample URL: https://developers.google.com/drive/v2/reference/files/insert Description: Instructions for importing modules and/or projects into Apps Script for use with Google Workspace.
Actual Behavior
Import of the sub-packages does not reflect the proper time zones and/or other dependencies (e.g. included libraries) when loaded into Google Workspace's Apps Script editor.
Steps to Reproduce the Problem

Traverse the source tree with your preferred tool(s).
Determine the location of all files named appscript.json
Rename the file(s) appsscript.json in the respective directories
Import the files into the Apps Script environment within Google Workspaces
Ensure that the Apps Script editor recognizes the file name(s) appropriately and applies the appropriate configuration to the subprojects.

NOTE:  File name(s) should have 3 "p" characters & 3 "s" characters


Clarify what to do in case of conflict

I'm very happy to see this initiative, as a tool author I was looking to see if something like this existed. I'm happy it does, and I'll be adopting this for starters in Kaocha, a test runner for Clojure.
One thing that I think would be worth stating explicitly is how tools should behave if a config file exists both at the top level of the project and in . config
Reasonable options

show an error message and exit
top level takes precedence (with warning?)
.config takes precedence (with warning?)
they are merged (in a specified order), if that makes sense for the tool

I was just thinking about this myself (for RuboCop) and there it made most sense to have the top-level take precedence (just because you can have it's configuration at every level of a directory structure), but for most cases I think it'd be best to go with option 1 as the safest one.


vim 9.0.1503 file dialogs bug

Steps to reproduce

Open vim 9.0.1503 in terminal (this is the current package on FreeBSD).
Create subwindows with Ctrl+W S, Ctrl+W V. Go to new subwindows with Ctrl+W W.
Open file selection dialog with :E.
See that file browsing dialog shows up not only in currently selected subwindow but also other subwindows overriding their contents.
Note that opening file in the current subwindow does not close dialog in other subwindows.

Expected behaviour
As before file browser dialog should only show in currently selected subwindow and not in other subwindows overriding their contents.
Version of Vim
9.0.1503
Environment
OS: 13.2-RELEASE FreeBSD.
TERMINAL: Terminology-1.13.0_1.
$TERM: xterm.
SHELL: zsh.
Logs and stack traces
No response

I am unable to reproduce - I have tried reproducing this on 13.2-RELEASE by using the version in ports.  I have tried in a terminal emulator as well as the console, and terminology.  I have also tried with tag v9.0.1503 from git directly (i.e. not in ports)
Not sure what difference our system might have, what does ldd /usr/local/bin/vim look like for you?  this is what I get
[23:50:34] trdenton:vim $ ldd ./work-console/vim-9.0.1503/src/vim
./work-console/vim-9.0.1503/src/vim:
libm.so.5 => /lib/libm.so.5 (0x1a79bc539000)
libncursesw.so.9 => /lib/libncursesw.so.9 (0x1a79bc583000)
librt.so.1 => /usr/lib/librt.so.1 (0x1a79bd3c6000)
libintl.so.8 => /usr/local/lib/libintl.so.8 (0x1a79be69a000)
libpython3.9.so.1.0 => /usr/local/lib/libpython3.9.so.1.0 (0x1a79bfe66000)
libdl.so.1 => /usr/lib/libdl.so.1 (0x1a79be083000)
libthr.so.3 => /lib/libthr.so.3 (0x1a79bedf4000)
libc.so.7 => /lib/libc.so.7 (0x1a79c09d9000)
libcrypt.so.5 => /lib/libcrypt.so.5 (0x1a79bf61f000)
libutil.so.9 => /lib/libutil.so.9 (0x1a79c0e8e000)
[vdso] (0x7ffffffff5d0)

sounds like https://github.com/vim/vim/issues/12304 please update your runtime files

Thanks @chrisbra this was it! Thank you for the fix!! :-)


Enabling ICS on an interface causes WSL VM to loose access to that interface

Version
Microsoft Windows [Version 10.0.18363.1977]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 20.04
Other Software
Internet Connection Sharing
Repro Steps
To reproduce the steps below you need 3 additional interfaces on your Windows machine, although the behavior can be reproduced with only 2 interfaces.
Interface A - <IP_ADDRESS>/24
Interface B - <IP_ADDRESS>/24
Interface C - <IP_ADDRESS>/24

Start up your WSL2 VM
You should be able to ping devices  on network A, B, and C
Share your "Internet" connection from Interface A to C
a. Right-click on Interface A and go to Properties
b. Sharing tab, then Allow other network users to connect through this computer's Internet connection
c. Select Interface C
c. Click OK
Notice that within your WSL2 VM you can no longer ping devices on Network A and C, although you can still ping devices on Network B
Stop Internet Connection Sharing on Interface A
Now you can ping devices on Network A and C
Share your "Internet" connection from Interface B to C
Notice that within your WSL2 VM you can no longer ping devices on Network B and C, although you can still ping devices on Network A
Stop Internet Connection Sharing on Interface B
Now you can ping devices on Network B and C

Expected Behavior
Naively, even if an Interface is sharing its connection through Internet Connection Sharing I would expect its connection to be available to WSL2 VM. I should still be able to ping devices in the interface's network while its connection is being shared.
Actual Behavior
Devices on the network of an interface that has Internet Connection Sharing enabled are not reachable from a WSL2 VM.
Diagnostic Logs
No response

Tried reproducing this again, and the issue only surfaces with VPN interfaces. Although it would be nice to get this to work, there are a myriad of VPN options that could cause this problem. So I am closing this ticket because it will be difficult to reproduce.


action: Add commitlint action for checking commits

commitlint will force a developer to write a commit as per our guidelines
reference links:
Naming Conventions -> https://commitlint.js.org/#/concepts-commit-conventions
Rules -> https://commitlint.js.org/#/reference-rules
Signed-off-by: Nitin Goyal<EMAIL_ADDRESS>

@jarrpa Let's merge this first so we can run this action on other PR's as well

I thiiiink it looks fine? Unfortunately I don't any information to what commitlint is, where it's from, how it's configured, or what the parameters we're configuring specify. Also this seems like it's going to create a conf directory when we already have a config directory? That doesn't make much sense. Can the JSON config not live with the workflow YAML?

commitlint makes sure that the commit message is as per our guidelines, here is the reference https://commitlint.js.org/#/reference-rules.
I have created a conf directory inside the .github/workflows/ to place all action related conf files

Here are some examples

@jarrpa I am merging this one myself. We can change this later if required.


Reworking the graph_t and graph::build interface.

I'll update the details of the rework once it is complete.

graph_container_t: removed
graph_base_t: removed
graph_t is not diamond inheritance
graph_t inherits csr, coo or csc
reworked build functions
polymorphic inheritance fix on CUDA removed
supports indices -> offsets and offsets -> indices conversions


sage activation error 500

Hi, i downloaded the latest version of sage and after uploading the entire folder in the themes folder in my e wordpress i started with activation... but after activation i had an error 500. in wp-config. I put on "TRUE" the WP_DEBUG, and i found an error on src/helpers.php on line 27... how can i fix this problem?

2 things:

don't clear out the issue template that was in place when you went to create this issue. it's there for a reason... not for you to ignore and delete.
if you had followed the steps in the template, you would understand that this is a personal support request that shouldn't be posted here


[Feature Request] I18N locale type script support

Existing Component
是
Component Name
i18n
Description
Hi there, I am currently using some element ui components in my project with typescript. I am applying Internationalization to the whole project. However, when I import the several language support locale into my project. I got the Element implicitly has an 'any' type because type '{ en: typeof import("element-ui/lib/locale/lang/en"); error.
I add some simple d.ts files by myself trying to workaround it. However, I get more implicit type error later.
Could you officially add some declaration file for the i18n files?

Right now we have no plan to support this.


Provide useful information when backward sync preconditions not met

Description
When syncing a new teku+besu node, besu 22.4.0 wound up repeatedly logging:
2022-07-20 21:00:12.578+00:00 | ForkJoinPool.commonPool-worker-1 | INFO  | BackwardsSyncAlgorithm | Waiting for preconditions...

This is an entirely unhelpful log message. It should clearly explain what preconditions it's waiting on so the user knows why sync is not progressing. Looking at the code in this case I think it was that the initial sync hadn't completed.

@daniellehrner This might be useful for our logging updates

@gezero as well RE backwards sync.

Closing for backwards sync changes in 22.10.x


fix(docs): Add an example gallery under each function / class

As done in scikit-learn and skrub

If I understand correctly the issue, is it fixed by: https://github.com/probabl-ai/skore/pull/956?

Yes, this is fixed by #956


Documentation out of sync with library

This commit:
https://github.com/UAVCAN/libuavcan/commit/2843c0a35e16e24be00eddee7cbf525308025166#diff-35b7a237998b2054ec6ba90c34a0490aL74
Changes the CAN ID field so that it no longer matches the documentation:
http://uavcan.org/CAN_bus_transport_layer_specification#CAN_ID_field
This image needs to be remade since it's easy to skim past text and would confuse people.
This has also made its way into the PX4 codebase:

Change was made on 2015-04-30 08:26:25 +0300
Current version in PX4/Firmware is 988e404586900754e7e1748c87c0ec12b757cf87 made on 2015-06-07 00:07:49 +0300

Thanks, this is a known issue, a big documentation update is coming.

Great :) It seems like the CAN IDs (including MSG IDs) have jumped around a bit recently. When can we expect them to be stable?

Pretty much now. We're phasing in the last change set before a freeze of the lower-level protocol aspects.

http://uavcan.org/Specification/4._CAN_bus_transport_layer.html#id-field


RUMM-1837: Add org.json support for the event attributes

What does this PR do?
Closes #588. This change adds support of org.json.JSONArray and org.json.JSONObject classes (which come with Android SDK) in the Logger#addAttribute call.
Review checklist (to be filled by reviewers)

[x] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[x] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[x] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #792 (de794d5) into master (1cce72d) will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #792      +/-   ##
============================================
+ Coverage     88.30%   88.37%   +0.07%
- Complexity     1822     1827       +5
============================================
  Files           196      196
  Lines          6402     6413      +11
  Branches        741      746       +5
============================================
+ Hits           5653     5667      +14
- Misses          504      505       +1
+ Partials        245      241       -4

Impacted Files
Coverage Δ

...m/datadog/android/core/internal/utils/MiscUtils.kt
50.00% <100.00%> (+8.62%)
:arrow_up:

.../src/main/kotlin/com/datadog/android/log/Logger.kt
97.83% <100.00%> (+0.02%)
:arrow_up:

...droid/rum/tracking/ActivityViewTrackingStrategy.kt
86.54% <0.00%> (-3.85%)
:arrow_down:

...g/android/rum/internal/domain/scope/RumEventExt.kt
82.61% <0.00%> (-3.48%)
:arrow_down:

...oid/src/main/kotlin/com/datadog/android/Datadog.kt
75.22% <0.00%> (-0.88%)
:arrow_down:

...atadog/android/core/configuration/Configuration.kt
97.82% <0.00%> (-0.36%)
:arrow_down:

.../android/rum/internal/monitor/DatadogRumMonitor.kt
93.79% <0.00%> (ø)

...ernal/instrumentation/gestures/GesturesListener.kt
90.70% <0.00%> (ø)

...l/net/info/BroadcastReceiverNetworkInfoProvider.kt
79.38% <0.00%> (ø)

... and 6 more


page_rule ordering

How the heck are you supposed to order page_rule resources using terraform 0.13.0 and cloudflare provider 2.9.0? The solutions suggested in #187 don't work.

I also don't see how page_rule resources can be maintained over time using terraform: if you have a list of n page rules and drop a new one in the middle, it seems like the Cloudflare API will always create it as priority = 1 regardless of any potential ordering solution.

The solution in #187 is still working for us between 50 and 110 page rules across ~ 30 domains.  I'd suggest re-reading the issue and providing a reproduction test case if you're continuing to have issues.


Additional Shell Types (Telnet/SSH/COM/etc)

It would seem natural to add more types to Terminal.

Telnet Client
SSH Client
COM Terminal

For an added bonus, allow attaching WMI remote command prompt.
Other ideas:

Network Shell (netsh)
Disk Partition Shell

Essentially all of the interactive command line tools should have direct tab support in Terminal making it truly the 'go to' place for working with command lines on Windows.

Agree with this very much. It doesn't need to be as over the top as ConEmu or anything, but it should be extensible via the JSON and be able to connect via SSH (even if that's via a putty wrapper) to other Linux systems.
Oh, and one maybe missing from the OP list, I hope that it also supports:
• WSL Shell (regardless of Linux flavour).

What is a WSL shell that exists regardless of “Linux flavor”? a WSL inherently requires a distribution or some other container. Terminal already supports that.
For SSH, we ship ssh.exe inbox in Windows, which you can use as the commandline of a profile.
For everything else, this is a /dupe of #1280.


Protocol handler: fallback to public gateways if no node available

Summary
There should be an option to disable local IPFS node and run ipfs:// and ipns:// with a public gateway as a backend (#491) (or do that as an automatic fallback if no node is available).
Rationale
Given protocol handler API from libdweb (https://github.com/ipfs-shipyard/ipfs-companion/pull/533) we are able to separate security perimeter from transport layer.
IPFS is content-addressed, trustless system. It means as long we verify hash of received payload matches we can use public gateways as a fallback transport.  Due to this the feature should be enabled only for browsers with native protocol handler.
It would be a huge UX win for end user: ipfs:// and ipns:// would always work, even if the only transport allowed in local network is HTTP or user is concerned with battery/bandwidth use and just want to browse IPFS in light, read-only mode (#507).
When native protocol handler is available, read-only public gateway could be listed as an additional backend (#491).
Additional Resources

We have a semi-official list of public gateways donated by community:
https://github.com/ipfs/public-gateway-checker/blob/master/gateways.json
It could be used in round-robin fashion.

Unable to implement this right now: cross-protocol redirect via webRequest.onBeforeRequest  gets blocked.
Upstream issue: https://github.com/mozilla/libdweb/issues/74


The issue "The resource 'xxx' is not defined in the template" is not detected

Issue
We used separate templates for different resource deployment, we put APIM in apim.tempalte.json:
{
      "apiVersion": "2023-03-01-preview",
      "name": "[parameters('apimServiceName')]",
      "location": "[parameters('location')]",
      "type": "Microsoft.ApiManagement/service",
      "sku": {
        "name": "[parameters('sku')]",
        "capacity": "[parameters('skuCount')]"
      },
      "zones": "[parameters('availabilityZones')]",
      "identity": {
        "type": "SystemAssigned"
      },
}

and the diagnostics into azuremonitor.template.json:
{
      "type": "Microsoft.ApiManagement/service/diagnostics",
      "apiVersion": "2022-09-01-preview",
      "name": "[concat(parameters('apimServiceName'), '/azuremonitor')]",
      "dependsOn": [
        "[resourceId('Microsoft.ApiManagement/service/loggers', parameters('apimServiceName'), 'azuremonitor')]"
      ],
      "dependsOn": [
        "[resourceId('Microsoft.ApiManagement/service', parameters('apimServiceName'))]"
      ],
      "properties": {
        "logClientIp": true,
        "alwaysLog": "allErrors",
        "loggerId": "[resourceId('Microsoft.ApiManagement/service/loggers', parameters('apimServiceName'), 'azuremonitor')]",
        "sampling": {
          "samplingType": "fixed",
          "percentage": "[parameters('samplingRate')]"
        }
    }
}

then we have a main tempalte to deploy those two tempates, in azuremonitor.template.json we added dependsOn, but the APIM service is not defined in the current template file, the deployment will fail.
When I run the arm-ttk, it didn't report this error.
Expectation
arm-ttk should be able to report this issue.

TTK is meant to verify best-practices compliance. It does not check for syntax, dependencies, APIs, etc., for your services.
This is likely closer to what you're looking for: https://learn.microsoft.com/en-us/azure/azure-resource-manager/templates/deploy-what-if?tabs=azure-powershell
Thanks!


dev: swap_remove

Replaces remove with swap_remove for optimized removal of items in a Vec where preservation of order of the items isn't needed.

should we pin clippy for the bins to nightly-2024-09-25?

should we pin clippy for the bins to nightly-2024-09-25?

yep, done here https://github.com/paradigmxyz/reth/pull/11401


Unsafe eval in chart.js

Hello,
When including chart.js in a chrome packaged application, the following error is thrown during the graph rendering :

Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "default-src 'self' chrome-extension-resource:". Note that 'script-src' was not explicitly set, so 'default-src' is used as a fallback.

On  line 1419 of Chart.js which is the new Function()call :
function tmpl(str, data){
    // Figure out if we're getting a template, or if we need to
    // load the template - and be sure to cache the result.
    var fn = !/\W/.test(str) ?
      cache[str] = cache[str] ||
        tmpl(document.getElementById(str).innerHTML) :

      // Generate a reusable function that will serve as a template
      // generator (and which will be cached).
      new Function("obj",
        "var p=[],print=function(){p.push.apply(p,arguments);};" +

        // Introduce the data as local variables using with(){}
        "with(obj){p.push('" +

        // Convert the template into pure JavaScript
        str
          .replace(/[\r\t\n]/g, " ")
          .split("<%").join("\t")
          .replace(/((^|%>)[^\t]*)'/g, "$1\r")
          .replace(/\t=(.*?)%>/g, "',$1,'")
          .split("\t").join("');")
          .split("%>").join("p.push('")
          .split("\r").join("\\'")
      + "');}return p.join('');");

    // Provide some basic currying to the user
    return data ? fn( data ) : fn;
  };

This is quite annoying because the CSP cannot be set for chrome packaged applications. Sandboxing or other tricks could eventually be used, but I'd rather a clean solution.

@nnnick it would be useful for newbies like myself to have an example/documentation on how to fix this. I'm not sure exactly which configuration items I need to replace to solve this issue.
Also, it seems this library is supported despite @FVANCOP comment, correct?

I followed nnick's suggestion and converted the default string templates into functions to avoid the eval calls. The eval error is gone now.
I'm using version 1.0.1, but there should be similar options in newer versions:
// var chart = new ...

chart.options.tooltipTemplate = function(element) {
		return (element.label ? element.label + ": " : "") + element.value;
	}

chart.options.legendTemplate = function(chart) {
	var legendStr = "
";

	for (var i=0; i < chart.segments.length; i++)
	{
		legendStr += "
";

		if(chart.segments[i].label && chart.segments[i].value) {
			legendStr += "" + chart.segments[i].label;
		}

		legendStr += "
";
	}

	legendStr+= "
";

	return legendStr;
}

@nnnick
Hi nnNick,
I have the honour to improve a legacy angularJS codebase
and am running intoo what I suspect is this error.
The error I see in the console points to here in the minified code:
Line 13:  e.apply(b, a)
I have currently no working dev environment so cannot see where this at in the unminified code.
function e(a) {                 var b = c.console || {}                   , e = b[a] || b.log || o;                 a = !1;                 try {                     a = !!e.apply                 } catch (a) {}                 return a ? function() {                     var a = [];                     return f(arguments, function(b) {                         a.push(d(b))                     }),                     e.apply(b, a)                 }                 : function(a, b) {                     e(a, null == b ? "" : b)                 }             }
But my html is exclusively generated in a .html file.
This html file is now linked in the directive like this:
templateUrl: 'scripts/graphs/graph.tpl.html',
Do you know if and how I can turn this intoo the function syntax you proposed?
Or is there something else I should try?
Thanks in advance!
kr Tom

Hi Tom, first a couple basic questions. Do you know what version this is? And do you have a hash of the file? Is it standalone or built into another file?

@fulldecent
Hi William,
Thank you for your response.
I have in my package.json:
"dependencies": {
<EMAIL_ADDRESS>"nnnick/Chart.js#1.0.2",
I have created the hash online here https://hash-file.online/:
"Chart.min.js", with size of 57,275 kb (kilobytes), and file hash using the hashing algorithm SHA-256 has the value : b53c2627dfab90adf4c1390e721a4f4ff030ce216a10f064cf4fea4f17c1a95e.
Butthe error occurs in a bundled vendor.min.js file.
Unfortuenedly I do currently not have a setup to test chart.min.js in isolation, but maybe I could set this up.
The error I get in chrome:
"""
not available

2
vendor.min.js:2 EvalError: Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "default-src 'self' 'unsafe-inline'".
at new Function (<anonymous>)
at c (vendor.min.js:2:458169)
at d.template (vendor.min.js:2:458562)

"""
At line at c (vendor.min.js:2:458169) is the following code:
d.template = function(a, b) {
        function c(a, b) {
            var c = /\W/.test(a) ? new Function("obj","var p=[],print=function(){p.push.apply(p,arguments);};with(obj){p.push('" + a.replace(/[\r\t\n]/g, " ").split("<%").join("\t").replace(/((^|%>)[^\t]*)'/g, "$1\r").replace(/\t=(.*?)%>/g, "',$1,'").split("\t").join("');").split("%>").join("p.push('").split("\r").join("\\'") + "');}return p.join('');") : d[a] = d[a];
            return b ? c(b) : c
        }
        if (a instanceof Function)
            return a(b);
        var d = {};
        return c(a, b)
    }

Hope this helps
kr Tom


[Bug]: Non-clean korifi deployment using kapp should not fail

What happened?
So far we are mainly doing clean deploys of korifi, either on a fresh kind cluster or we are doing an undeploy before we deploy. The latter is what our PR pipeline is doing.
Sometimes however the undeploy will fail for various reasons and in these cases the next deployment is doomed to fail with an error like this: https://ci.cf-k8s.cf/teams/main/pipelines/pr-e2e/jobs/run-e2es/builds/45#L624ade13:128:131
kapp: Error: Applying update namespace/cf-k8s-controllers-system (v1) cluster:
  Updating resource namespace/cf-k8s-controllers-system (v1) cluster:
    API server says: admission webhook "namespaces.hnc.x-k8s.io" denied the request: namespaces "cf-k8s-controllers-system" is forbidden: cannot remove tree label "cf-k8s-controllers-system.tree.hnc.x-k8s.io/depth" in namespace "cf-k8s-controllers-system"; these can only be managed by HNC (reason: Forbidden)

What happens here is the following:

kapp deploy is creating the cf-k8s-controllers-system namespace
HNC labels the namespace
The second kapp deploy is trying to create the namespace again, but does not know about the HNC label, so it effectively tries to remove it
The HNC webhook blocks the label removal

If you use kubectl this error does not occur, because kubectl ignores the labels and annotations when you apply an existing object
What you expected to happen
I can kapp deploy korifi without having to first undeploy it.
Acceptance Criteria
GIVEN I kapp deploy korifi
WHEN I kapp deploy korifi again a few minutes later
THEN I do not get any errors
How to reproduce it (as minimally and precisely as possible)

kapp deploy korifi
wait for HNC to label the system namespace
kapp deploy korifi again

Anything else we need to know?
No response
Environment
Revision of codebase: main
Cloud provider or hardware configuration: any

@georgethebeatle do you think this is still valid now that we use Helm?


quick-lint-js: 2.3.1 -> 2.4.0

Description of changes
https://quick-lint-js.com/releases/
Added

Missing parentheses around a self-invoked arrow function are now reported as E0211. (Implemented by Sarah Schulte.)
E0179 is now reported if a JSX element such as  follows return (implemented by Rob Miner).

Fixed

<div> <> </> </div> (a JSX fragment inside a JSX element) no longer reports a syntax error.

Changed

Windows builds (x86, x64, ARM, and ARM64) are now built with MinGW's C and C++ runtime libraries instead of Microsoft's. This fixes a compliance issue with GPLv3.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review pr 168852 run on x86_64-linux 1

1 package built:

quick-lint-js

Reviewed points

[x] package name fits guidelines
[x] package version fits guidelines
[x] package build on x86_64
[x] executables tested on x86_64
[x] all depending packages build

Possible improvements
Comments


bug: telescope and other popups have annoying light background when transparency is off

Did you check docs and existing issues?

[X] I have read all the flow.nvim docs
[X] I have searched the existing issues of flow.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
v0.10.2
Operating system/version
Linux, alacritty.
Describe the bug
Hi there! Thanks for very eye-friendly colorscheme, it just clicks with my tired eyes. But the decision to use gray6 as a background for floats and popups like telescope is somewhat questionable. I'm sure that I'm doing something wrong, but it looks now like that:

Steps To Reproduce
Just switch on the color theme and bring up telescope
Expected Behavior
I think it would be nice to keep float backgrounds to default background color or at least add an option to do just that.
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "0xstepit/flow.nvim",
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("flow")
-- add anything else here

Hey @andviro, thanks for opening the issue! I'll work on it in the coming days, but feel free to submit any color scheme PRs in the meantime—they're more than welcome! :)

Hey @andviro, the PR should fix the issue. Thanks again for reporting it! I always use a transparent background so I didn't notice the bug :)


Fix Tippy is not showing up sometimes.

In the Stream Settings > Subscribers UI, the popover fails to open when interacting with a partially hidden row at the bottom edge of the table. This behavior is not observed on CZO.
Preliminary investigation suggests that the issue may be related to Tippy incorrectly applying the "data-reference-hidden" attribute to elements that are not actually hidden.
Discussed in #26613.

what do you mean by a partially hidden row? Could you please share a screenshot for reference?

By partially hidden, we mean when the link being clicked is at the very bottom/top of the screen such that only part of it is visible.


Basic map and one marker live demos are just showing sea

http://apneadiving.github.io/
I get the same thing when trying to use the code.

Yes I should update the example: the pic url I used doesnt exist anymore. Thats it :)


Tailwind is not purging unused styles because no template paths have been provided.

Hi,
When building my nuxt 2.12.2 project, I am getting this warning:
Tailwind is not purging unused styles because no template paths have been provided.
But the css does seem to be purged, I think? Any clues?

It puts it in a 1.6Mb js file.
Just did "nuxt generate" on a fresh install.

Added purge option to tailwind.config.js manually, seems working
module.exports = {
  purge: ["./src/**/*.html", "./src/**/*.vue"],
  theme: {},
  variants: {},
  plugins: []
};

Should it be there by default?

Yep, It isn't by default.
Proper in my case:
  purge: [
    "./pages/**/*.vue",
    "./components/**/*.vue",
    "./plugins/**/*.vue",
    "./static/**/*.vue",
    "./store/**/*.vue"
  ],

It' happened to me too, worked with the @Triloworld solution. But I think it should be added by default!!

@Triloworld's solution seems to work well! Thanks!

I've now seen that @Atinux put the recommended settings on the new tailwind.config.js
The problem is it didn't update for current projects.
The recommended settings he put are:
// tailwind.config.js
module.exports = {
  /* ... your actual config */,
  purge: {
    // Learn more on https://tailwindcss.com/docs/controlling-file-size/#removing-unused-css
    enabled: process.env.NODE_ENV === 'production',
    content: [
      'components/**/*.vue',
      'layouts/**/*.vue',
      'pages/**/*.vue',
      'plugins/**/*.js',
      'nuxt.config.js'
    ]
  }
}

Thanks @santicros, your config help me a lot. When I build or generate with the @Triloworld solution, the breakpoints like md:* don't appears on compiled files.

@dbarjs running into the same issue on my end with md:* . Disabled purge on tailwind.config.js and nuxt.config.js

I don't think this module should require having to add in a production flag for enabled.
According to the Tailwind docs, purge should only run in production and they are explicit that enabled is a manual override. So this module should not default to requiring a manual override.


Complex inner products always return zero

The outcome of
(magicl.blas-cffi::%ddot 1 (make-array 1 :element-type 'double-float :initial-element 1.0d0) 1 (make-array 1 :element-type 'double-float :initial-element -1.0d0) 1)

is -1.0d0, as expected. However, the outcome of
(magicl.blas-cffi::%zdotu 1 (make-array 1 :element-type '(complex double-float) :initial-element #c(1.0d0 0.0d0)) 1 (make-array 1 :element-type '(complex double-float) :initial-element #c(-1.0d0 0.0d0)) 1)

is #C(0.0d0 0.0d0). The resulting zero value is consistent no matter which arrays are passed as input.

The problem stems from the fact that there exist two types of application binary interfaces (ABIs) for BLAS:

On the one hand, the reference implementation of BLAS as well as OpenBLAS both follow the standard C ABI [1] and return complex values via the SSE registers XMM0 (real part) and XMM1 (imaginary part). When using those implementations, MAGICL returns the correct dot product.
On the other hand, Apple's Accelerate framework and the Intel Math Kernel library deviate from the standard [2] and modify the interface so that functions that should return a complex number instead have a void return type and an extra argument that points to the memory where the complex result will be written.

To summarize: MAGICL's interface to zdotu, cdotu, etc. is fine if the underlying BLAS is OpenBLAS or the reference BLAS but things fail as shown above when using Apple Accelerate or Intel MKL. In the latter cases, we need to define a different interface to make things work:
(CFFI:DEFCFUN ("zdotu_" %%ZDOTU :LIBRARY MAGICL.FOREIGN-LIBRARIES:LIBBLAS)
    :VOID
  (C :POINTER)
  (N :POINTER)
  (ZX :POINTER)
  (INCX :POINTER)
  (ZY :POINTER)
  (INCY :POINTER))

(COMMON-LISP:DEFUN %ZDOTU (N ZX INCX ZY INCY)
  (COMMON-LISP:DECLARE (COMMON-LISP:INLINE %%ZDOTU)
                       (COMMON-LISP:TYPE (COMMON-LISP:SIGNED-BYTE 32) N)
                       (COMMON-LISP:TYPE
                        (COMMON-LISP:SIMPLE-ARRAY
                         (COMMON-LISP:COMPLEX COMMON-LISP:DOUBLE-FLOAT)
                         (COMMON-LISP:*))
                        ZX)
                       (COMMON-LISP:TYPE (COMMON-LISP:SIGNED-BYTE 32) INCX)
                       (COMMON-LISP:TYPE
                        (COMMON-LISP:SIMPLE-ARRAY
                         (COMMON-LISP:COMPLEX COMMON-LISP:DOUBLE-FLOAT)
                         (COMMON-LISP:*))
                        ZY)
                       (COMMON-LISP:TYPE (COMMON-LISP:SIGNED-BYTE 32) INCY))
  (CFFI:WITH-FOREIGN-OBJECTS ((PTR-C :DOUBLE 2) (N-REF885 ':INT32) (INCX-REF887 ':INT32)
                              (INCY-REF889 ':INT32))
    (COMMON-LISP:SETF (CFFI:MEM-REF N-REF885 :INT32) N)
    (COMMON-LISP:SETF (CFFI:MEM-REF INCX-REF887 :INT32) INCX)
    (COMMON-LISP:SETF (CFFI:MEM-REF INCY-REF889 :INT32) INCY)
    (MAGICL.CFFI-TYPES:WITH-ARRAY-POINTERS ((ZX-REF886 ZX) (ZY-REF888 ZY))
      (%%ZDOTU PTR-C N-REF885 ZX-REF886 INCX-REF887 ZY-REF888 INCY-REF889)
      (COMMON-LISP:COMPLEX (CFFI:MEM-AREF PTR-C :DOUBLE 0)
                           (CFFI:MEM-AREF PTR-C :DOUBLE 1)))))

[1] https://github.com/hjl-tools/x86-psABI/wiki/X86-psABI
[2] https://software.intel.com/en-us/mkl-macos-developer-guide-calling-blas-functions-that-return-the-complex-values-in-c-c-code

Nice sleuthing.

Thank you.

Alright I was able to reproduce this with MKL and got a fun result:
* (magicl.blas-cffi::%ddot 1 (make-array 1 :element-type 'double-float :initial-element 1.0d0) 1 (make-array 1 :element-type 'double-float :initial-element -1.0d0) 1)
-1.0d0
* (magicl.blas-cffi::%zdotu 1 (make-array 1 :element-type '(complex double-float) :initial-element #c(1.0d0 0.0d0)) 1 (make-array 1 :element-type '(complex double-float) :initial-element #c(-1.0d0 0.0d0)) 1)
CORRUPTION WARNING in SBCL pid 8901(tid 0x7fc6591390c0):
Memory fault at (nil) (pc=0x7fc643cc728a, fp=0x7fc64bcff3f0, sp=0x7fc64bcff2c0) tid 0x7fc6591390c0
The integrity of this image is possibly compromised.
Continuing with fingers crossed.

debugger invoked on a SB-SYS:MEMORY-FAULT-ERROR in thread
#<THREAD "main thread" RUNNING {10005184C3}>:
  Unhandled memory fault at #x0.

I'll go ahead and try regenerating the bindings and give an update here.


qwen2vl 7b , how to call api

Reminder

[X] I have read the README and searched the existing issues.

System Info
docker:
transformers                      4.45.0.dev0
torch                             2.4.0
llamafactory                      0.9.0                      /app
Reproduction
vim examples/inference/sft_xd_seal.yaml
model_name_or_path: /xiedong/Qwen2-VL-7B-Instruct
adapter_name_or_path: output/saves/qwen2_vl-7b/lora/sft_xd
template: qwen2_vl
finetuning_type: lora

run:
llamafactory-cli api examples/inference/sft_xd_seal.yaml

I don't know how to call this API. My fine-tuned dataset looks like this.
[
  {
    "messages": [
      {
      "content": "你是一个擅长识别印章上文字的助手，输出json字符串给用户。",
      "role": "system"
      },
      {
        "content": "<image>识别图片里红色印章上的公司名称或单位名称（印章主文字）。",
        "role": "user"
      },
      {
        "content": "{\"印章主文字\": \"饮酒太原近似收益有限公司\"}",
        "role": "assistant"
      }
    ],
    "images": [
      "/xiedong/yinzhang/save_dst/010155.jpg"
    ]
  }
]

Expected behavior
I would like to have an example of calling the API, where I can pass in an image and text to query the large model and get a response from the multimodal model.
Others
No response

model is right:

The IP/docs contain the following information, but I still don't know how to pass in an image.
Previously, I was able to use the language model by passing in text to experiment, but now this is a multimodal model, and I want to pass in an image.
ChatGPT told me to use tools, but there is no available tutorial explaining how I should build this tools.
{
"model": "string",
"messages": [
{
"role": "user",
"content": "string",
"tool_calls": [
{
"id": "string",
"type": "function",
"function": {
"name": "string",
"arguments": "string"
}
}
]
}
],
"tools": [
{
"type": "function",
"function": {
"name": "string",
"description": "string",
"parameters": {}
}
}
],
"do_sample": true,
"temperature": 0,
"top_p": 0,
"n": 1,
"max_tokens": 0,
"stop": "string",
"stream": false
}

For api calling, please refer to the ChatGPT document. We use the same protocol.

For api calling, please refer to the OpenAI document. We use the same protocol.
curl https://api.openai.com/v1/chat/completions \
  -H "Content-Type: application/json" \
  -H "Authorization: Bearer $OPENAI_API_KEY" \
  -d '{
    "model": "gpt-4o",
    "messages": [
      {
        "role": "user",
        "content": [
          {
            "type": "text",
            "text": "What'\''s in this image?"
          },
          {
            "type": "image_url",
            "image_url": {
              "url": "https://upload.wikimedia.org/wikipedia/commons/thumb/d/dd/Gfp-wisconsin-madison-the-nature-boardwalk.jpg/2560px-Gfp-wisconsin-madison-the-nature-boardwalk.jpg"
            }
          }
        ]
      }
    ],
    "max_tokens": 300
  }'

Could you please give me a python script which can support qwen video inference?


CreateUiDefinition - Nested Templates - Secure parameters are marked as required

I have noticed with the latest version (2021.11.05) the script thinks secure param values are required in nestedTemplates.
So if I have a nested template (module in bicep) called from main.bicep ..
main.bicep
	mycoolmodule.bicep

In mycoolmodule.bicep ..
@secure()
@description('Specifies the SSH RSA public key string for the Linux nodes.')
param aksClusterSshPublicKey string

.. results in ..

NestedTemplate ...
    [-] Parameters Without Default Must Exist In CreateUIDefinition (80 ms)
        aksClusterSshPublicKey does not have a default value, and is not defined in CreateUIDefinition.outuputs

.. but obviously putting a default value in a secure value is not allowed.
Side-note: This is a bug but for my specific use-case I dont want any nestedTemplate parameters to have to have default values when not in createUiDefintion - they should only validate against at the top-level template only. I am trying to use sub-modules as a re-usable library that could be called from multiple top-level scripts that need different combinations of features. Slightly off-topic but just wanted to give the context just-in-case :)

re: the sideNote just want to make sure I understand...  when you have a bicep module (that turns into a nested deployment in the json) you don't want any defaultValues in that scenario?


botkit+bot framework+dialogflow

do you have an example for the implementation?

There are examples for Slack and Facebook in the examples directory.


Eligable_to_move

Implemented the conditional logic of skipping player if there are no available move after eating.

Looks goods to me🚀


Updated the Wechaty messages section

have opened this PR  instead of #916  .
This PR has the same contents as in PR #916 .

@huan created the same pr as in #916 ,as that PR was creating issues have created this new PR and will be closing PR #916

@huan  have closed the PR #916  and have opened this one with same contents as #916


fix(TDI-45011): BP_0.27 NetSuite TCompv0 : set mappings for transaction custom body fields.

What is the current behavior? (You can also link to an open issue here)
https://jira.talendforge.org/browse/TDI-45011
What is the new behavior?
Please check if the PR fulfills these requirements

[X] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[X] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[X] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:
Backport of https://github.com/Talend/components/pull/1787 into "maintenance/0.27" branch

Pull request test result: success :star:

inventoryTransfer is missing

It is not missing - mapping for Inventory Transfer is not present in https://jira.talendforge.org/browse/TDI-45011, so it is not available in backport either. Mapping for Inventory Transfer is being added in https://github.com/Talend/components/pull/1808 and will be merged in "maintenance/0.27" as a backport of that PR as soon as it is validated.


Add FabricBot config for automatic GitHub triaging

As in title. This will allow FabricBot to manage special labels like "needs: triage" and "needs: author response".

Wow, that's some very verbose JSON. Is the expectation that we will always have to update this by hand if we want to make changes or is there some UX to help us with this?

@cgillum: this is actually the same config as the one we host in the DF Extension: https://github.com/Azure/azure-functions-durable-extension/blob/dev/.github/fabricbot.json
My understanding is that there's a UX on the way to help manage the config. I'm actually talking with the FabricBot team to try and get onboarded to that UX. That will take time though. For now, I understand the easiest path to onboard these " new repos" is to utilize the same .json config as the one we have in the extension.


Add a default session list

Invoking nox with no command line arguments runs every session.  It would be useful if the nox.py could specify a default list of sessions to run in that case, for example
nox.default_sessions([ 'lint', 'tests' ])

CC @lukesneeringer

My suggestion is to actually make whether or not a session is a "default" session be part of session registration, e.g.
@nox.session(run_by_default=False)
def peekaboo():
    ...

This has a few limitations:

It only works if you use the @nox.session decorator; if you rely on session_foo function names, no dice. I assert this is fine.
There is no particularly good way to take a subset of parametrized sessions. So, in other words, while nox.default_sessions(["unit_tests(python_version='3.6')"]) would be easy (but eww!), it would be downright impossble if run_by_default only took a boolean. I assert this is also fine, and excluding parametrization should be done with session.skip

If we really needed this in the future, we could also make run_by_default accept a tuple with the permutations to run by default.

I could go either way on this, but just noting that pytest's parametrize has pytest.param specifically for adding more information (marks) to individual runs:
# content of test_expectation.py
import pytest
@pytest.mark.parametrize("test_input,expected", [
    ("3+5", 8),
    ("2+4", 6),
    pytest.param("6*9", 42,
                 marks=pytest.mark.xfail),
])
def test_eval(test_input, expected):
    assert eval(test_input) == expected

If we want to go down the route that @lukesneeringer is suggesting we could do:
@nox.session
@nox.parametrize('foo, bar', [
     (1, 2),
     nox.param(3, 4, run_by_default=False)
])

If we did that, would the entire kaboodle be an initial requirement, or just the more general case?

I'd strongly prefer a full solution over a partial one.
It's also perhaps useful to just adopt pytest's marks. We could just have
@nox.mark.default - if any thing is marked as default then running nox
alone will only run those sessions, if nothing is marked as default then
all sessions are run.
On Wed, Aug 30, 2017 at 2:13 PM Luke Sneeringer<EMAIL_ADDRESS>wrote:

If we did that, would the entire kaboodle be an initial requirement, or
just the more general case?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jonparrott/nox/issues/40#issuecomment-326120934, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAPUc6wNjoAYHaDkitGJArzXyGA7YWQQks5sddBkgaJpZM4PH8RN
.

I could get behind @nox.mark.default so long as if nothing is marked default, they all are.
Should we change to @nox.mark.parametrize?

Eh, I'd be okay with just nox.default and nox.parametrize. I don't think our API surface will be big enough to need to separate those into a mark namespace.

Agreed. I don't think nox needs a complete mark namespace.

I would love to have an env variable for this, tox has TOXENV, when it is defined and you run tox, only that environment is ran. For nox we could have NOXSESSION.

@stsewd that would be much easier to do than this proposal, and could be done independently.

I'm going to take a stab at implementing this, just signaling to anyone else if it's already in progress.

Closing this in favor of #141, let's move the discussion there.


Compound like Governance Contracts

Closes #48
Introduced changes
⚠ This a work in progress PR.
Introducing a governance mechanism inspired to Compound Governance, docs

The contracts are added as a workspace member of the repo.
Checklist

[x] Linked relevant issue
[ ] Updated relevant documentation
[ ] Added relevant tests
[ ] Add a dedicated CI job for new examples
[ ] Performed self-review of the code

Contracts completed, i'll start with testing

Could you include an example using this crate - thank you!

Could you include an example using this crate - thank you!

This is supposed to be a draft or, still missing some testing. I'll make it draft for now


Why inside a function?

Just wondering, why not export the string directly?

No reason I can think of! Want to make a PR? :)


Flip SSL Logic to make -Hardened the Default Setting and Add Prompt to Choose not to run -Hardened SSL Setup

Would like to have SSL setup default to running with the -Hardened setting. Also adding a prompt to prompt out of using Hardened/Internet Facing SSL setup configuration as an option.

For kicks tried just adding -Hardened to the self-signed certificate option of the Set-SslSecurity Script. Got the following output.

Believe we do a check on certificate expiration that throws this error.


linuxclock - Bugfix Add 1 to month received from Date.getMonth (0..11)

Date.getMonth() returns an integer from 0 to 11, so to display the date I think it's better to add 1.
I did not try to install the app by deploying the full website but I did test the simple change by modifying the app directly with the IDE.
For such a simple change I hope it's enough.

Sorry, just saw the failure in the build. I bumped the version in metadata.js

LGTM, nice spot!


Add avatar stack for commit like github

Feature Description

ref: https://primer.style/css/components/avatar-stack

To avoid loading artifacts, do it with a go subtemplate, not Vue.

This is not vue

This is just a reference, the actual implementation can be implemented according to the current technology stack

I know it's react. But I was just mentioning stuff like that is best done in a subtemplate.

yeah


Fix fix NPE for FileCacheValue.

Proposed changes
FileCacheValue.files may be null if there is not any files exists for some partitions.

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


AdapterH, AdapterP code

Dear Author,
Thanks for the great project.
I am trying to find AdapterH and AdapterP codes. But I can't find them in the finetune.py
I wonder where is that part
Best

Hi,
The code is here LLM-Adapters/peft/src/peft/tuners/bottleneck.py. We use different arguments to indicate which adapter to use.

Thanks a lot I will check it now!


Refactor/create Report Due Indicator function

This will eventually need refactoring in a separate function, so that is displays a number of months/years when appropriate
Originally posted by @matthieu-foucault in https://github.com/bcgov/cas-cif/pull/719#discussion_r893711134

@BCerki is this still applicable?  Seems like you already wrote it as a function?

Yes, I think this is done

@LindsayMacfarlane closing as this one has already been done.


SMPPGateway status = STOPPED

Hi,
I'm trying to get SMSServer work with a fake smpp Gateway (java OpenSmpp). My database interface is correctly configured and enquire_link seems to work on my Gateway (I can see back and forth on OpenSmpp and SMSServer).
My issue is that when I manually put a new message in smsserver_out, its status stays 'U' (When I debug, I see that it goes to 'Q' then back to 'U'). The cause is that my Gateway status is STOPPED.
Do you see any reason why the gateway status is STOPPED instead of STARTED ?
Many thanks for your help
Alex

Hi guys, looking deeper in this issue, I think I found the cause !
In JSMPPGateway.java, startGateway doesn't call super.startGateway() so that the gateway is never initialized to STARTED. I checked and confirmed that this is correctly done in ModemGateway, for example.
I simply added super.startGateway() at row 148 (juste after the session.connectAndBind(...) and it solved my issue.
Hope this helps !
Cheers
Alex


Earthdawn v3.15b - Patch

Submission Checklist
Required

[X ] The pull request title clearly contains the name of the sheet I am editing.
[ X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[ X] The pull request makes changes to files in only one sub-folder.
[ X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Character Sheet Info Roll20 Internal Use only.

Hey there @Jiboux, I perceived your pull request is trying to push a new file sheet file html called Earthdawn.htm the the earthdawn-4e folder. However, that sheet folder already have an html called ED4E.html which you should be modifying instead. Could you shed some lights on your intentions within this pull request?

Hello Miguel;
This is me getting mixed up and going to Earthdawn-4E Folder instead of Earthdawn (FASA Official) Folder... I cancel the pull and re-pull

Le 18 sept. 2023 à 02:50, Miguel Peres @.***> a écrit :
Hey there @Jiboux https://github.com/Jiboux, I perceived your pull request is trying to push a new file sheet file html called Earthdawn.htm the the earthdawn-4e folder. However, that sheet folder already have an html called ED4E.html which you should be modifying instead. Could you shed some lights on your intentions within this pull request?
—
Reply to this email directly, view it on GitHub https://github.com/Roll20/roll20-character-sheets/pull/12155#issuecomment-1722840034, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARICXJWDBVJVTHMBEQ2QSM3X27VLRANCNFSM6AAAAAA43VJLRU.
You are receiving this because you were mentioned.


Add TypedValue TryFrom implementations

Currently our optional converters are custom, but it would make for a nicer API if we could use TryFrom. Requires Rust 1.26.0

Thus @rnewman spake

Note that because Entid and Long and Timestamp are the same, there will be overloads and the transformation is lossy, but it would be convenient to be able to say:
let (name, dob, guid) = (row.next().unwrap(), row.next().unwrap(), row.next().unwrap())
let name: String = name.into()?;
let dob: DateTime = dob.into()?;
let guid: Uuid = guid.into()?;

no? Usually the type annotations will be unnecessary:
let person = create_person(name.into()?, dob.into()?, guid.into()?);

I think in addition to, not move — after all, if you really want an Entid, and there's any doubt, you should use into_entid rather than TryInto::i64.


Jaeger agent does not fit serverless FAAS (AWS lambda etc)

The pattern of using Jaeger Agents runnning on the same host does not fit (our) usecase of running (python) functions as FAAS in AWS Lambda.
The requirement of running a seperate process is a showstopper.
Is it possible to have the agent running in another host? or in some work-around this issue in some way?
Btw. looking forward to OsCon workshop!!! :-)

That's an interesting use case. The agent was designed to run as a side-car, which is a fairly common way of running infra components to avoid pulling the complexity into client libraries (and having to support that complexity in multiple languages).
Could you elaborate on your architecture? For example:

do the Python functions make RPC calls to other microservices? If so, how do you solve the discovery / routing problem?
how do you do other monitoring, e.g. metrics?  I assume AWS Lambda provide some metrics for you, but what if you wanted to use an off the shelf metrics solution like statsd or prometheus? They typically also require a side-car agent.

The micrpservices call eachother by a defined internal endpoint specific to
the microservice (thru Apigee or AWS ApiGateway) or they call functions
(AWS Lambda) directly through their scalable invocation endpoint.
About stats; we pull them from AWS or use NewRelic to expose some basics
from within the service.
My point is: FunctionAsAService is winning teritory and i can say i'm a fan
of the simplicity (development and deployment wise) however the paradigm is
to only give a function and you will only be billed for the execution time
of said function. So fast setup/teardown is key.
I am a follower of jaeger and think it is the best opentracing
implementation yet but not being able to go serverless is a killer 😊
On Sat, May 6, 2017, 20:31 Yuri Shkuro<EMAIL_ADDRESS>wrote:

That's an interesting use case. The agent was designed to run as a
side-car, which is a fairly common way of running infra components to avoid
pulling the complexity into client libraries (and having to support that
complexity in multiple languages).
Could you elaborate on your architecture? For example:

do the Python functions make RPC calls to other microservices? If
so, how do you solve the discovery / routing problem?
how do you do other monitoring, e.g. metrics? I assume AWS Lambda
provide some metrics for you, but what if you wanted to use an off
the shelf metrics solution like statsd or prometheus? They typically also
require a side-car agent.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/uber/jaeger/issues/146#issuecomment-299657894, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA1TUVVJxfw5qsGc5jlmGOWEWVe5-rDCks5r3Lx1gaJpZM4NS167
.

@sloev jaeger-collector now supports receiving spans over HTTP in both jaeger.thrift and zipkin.thrift formats, making the agent optional.
While jaeger's go and java client already support using the HTTP endpoint, jaeger's python client still lacks support. We welcome PRs and would love to help you add support to the python client.

There is an issue in the Python client repo for sending spans over HTTP (https://github.com/jaegertracing/jaeger-client-python/issues/98). I don't think we need anything else on the backend.


Only Sonoff?

Hello,
it looks interesting plugin for ewelink app.
I don‘t have Sonoff devices, but a light switch which works with the ewelink app. Does it work too with this plugin or only Sonoff?
Sent with GitHawk

If it works in ewelink it will work with this plugin

Ok thanks, i will try it!!
And the IMEI can be a fictitious UUID??

Am 24.03.2019 um 19:49 schrieb Patrik-svobodik<EMAIL_ADDRESS>If it works in ewelink it will work with this plugin
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Yep, a UUID from a generator will do the job
ne 24. 3. 2019 v 20:33 odesílatel Kohle81<EMAIL_ADDRESS>napsal:

Ok thanks, i will try it!!
And the IMEI can be a fictitious UUID??

Am 24.03.2019 um 19:49 schrieb Patrik-svobodik<EMAIL_ADDRESS>:
If it works in ewelink it will work with this plugin
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/howanghk/homebridge-ewelink/issues/21#issuecomment-475991903,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYr07UUolraFt7ssPT_fiWRqgCRDLJhPks5vZ9MNgaJpZM4cEWYT
.

Great, thanks for your answers!!

Am 24.03.2019 um 19:49 schrieb Patrik-svobodik<EMAIL_ADDRESS>If it works in ewelink it will work with this plugin
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi @Kohle81, if the switch is a simple on/off switch and it works in the ewelink app, yes it will work as this plugin emulate the iOS ewelink app.
The so-called imei can be any generated uuid. You can generate it with an online generator