feat: push manifest list

Related: https://github.com/dagger/dagger/issues/1371
Continue https://github.com/dagger/dagger/pull/1467

Instead of extends `#Push`, adding new task `#PushManifestList` for creating the manifest list
Refs:

https://docs.docker.com/engine/reference/commandline/manifest/

Not implatemented, just show api changes

I'm not sure it's possible to just push manifest list with Buildkit, at least, it wasn't possible when I worked on multi-platform.
You can take a look at https://github.com/dagger/dagger/pull/1467, it's a proof of concept that implement multi-platform in dagger Europa (old version but there shouldn't be much change in go code)

@TomChv
Yes. You're correct.
Have to use Control.Build to commit meta to Buildkit Worker for manifests create.

@TomChv
platform should be same as config, and need to pass from #Pull / #Dockerfile to #Push
otherwise, the arch-suffix images  created by docker manifest create will got wrong manifest list (sub manifest arch will always same as Plan.platform)

never mind.
Just hope this feature could land asap. 😂
I want to continue do next big thing, multi-arch builder.

@morlay Appreciate the effort here and agree this is an important feature. I have a couple high-level thoughts + concerns.
In terms of your use case, you mentioned that "QEMU is too slow for compiling huge project" and it seems like your intention after this PR is to add support for distributing builds across multiple Buildkit daemons for each platform (correct me if wrong). I like that idea a lot, but it touches on a lot of issues related to support for multiple Buildkit workers that I think we need to design very carefully.
Even for just this PR, we may need to do a lot more iteration to nail down the cue interface, add a lot more tests to make sure it works seamlessly with the rest of the features (including the --platform cli flag), etc.
What I'm getting at is that it might be easier to approach this problem differently to unblock your use case faster. Here's one route I could imagine:

Use the existing support for specifying --platform at the CLI to build and push images for different platforms. Concurrent executions of dagger do for each platform can be used, optionally with each one pointing to a different buildkit worker (via the BUILDKIT_HOST env var).
To assemble the manifest list, just invoke a cli tool like docker manifest, regclient, crane, etc. from a Dagger action. This could be put in the universe action repo for re-usability too.

I saw that you tried to use docker manifest for this previously. Looking quickly at the CI output you linked to and associated code, I think the issue may just be that you were not providing access to the docker daemon's socket. There's an example of how to make this work in our ci.cue here. Please let me know if I'm misunderstanding this though as I don't have a ton of context other than the logs you posted.

This is just my personal feeling though. If there's some other reasons the above won't work or you'd rather figure out how to get this support merged into core, please let me know. I just want to make sure you are aware that we may need to take our time with these features.

@sipsma

In terms of your use case, you mentioned that "QEMU is too slow for compiling huge project" and it seems like your intention after this PR is to add support for distributing builds across multiple Buildkit daemons for each platform (correct me if wrong). I like that idea a lot, but it touches on a lot of issues related to support for multiple Buildkit workers that I think we need to design very carefully.

I agree. In fact, QEMU already enough for most usages.
Multi-arch builder only need for project like envoy (I'm maintaining the multi-arch istio, https://github.com/resf/istio, the envoy compiling have to use native host)
Different from docker buildx, dagger split steps in to small actions, so builder of  dagger should support share outputs between multi-arch buildkits, instead merging all arch-matched steps in to matched buildkit,

Even for just this PR, we may need to do a lot more iteration to nail down the cue interface, add a lot more tests to make sure it works seamlessly with the rest of the features (including the --platform cli flag), etc.

It could be accept multi platforms, if when it set, it could push with the selected platform.

To assemble the manifest list, just invoke a cli tool like docker manifest, regclient, crane, etc. from a Dagger action. This could be put in the universe action repo for re-usability too.
I saw that you tried to use docker manifest for this previously. Looking quickly at the CI output you linked to and associated code, I think the issue may just be that you were not providing access to the docker daemon's socket. There's an example of how to make this work in our ci.cue here. Please let me know if I'm misunderstanding this though as I don't have a ton of context other than the logs you posted.

In github actions, it could.
Howerver, in my real-world CI, there is no docker.
Since I migrate to docker buildx with buildkits in k8s (mutli-arch cluster, gitlab-runner as tigger), I never need to use docker daemon.
This is why the PR needed, for non-docker environment
Why mv to dagger?
Just the composible actions will be make my team work better.
most developers don't want to care how the image build.
And we have to support amd64 & arm64 both, Which increase the complexity of the Dockerfile.

@morlay

In github actions, it could. Howerver, in my real-world CI, there is no docker.

That's alright actually, I was just using docker manifest as an example but there are other command line tools that can construct and modify manifest lists without needing docker. Examples I'm aware of are:

regctl
manifest-tool

Do you think any of those would work for you? If so, those tools could be wrapped in a reusable action and then put under a universe.dagger.io package.

@sipsma

Do you think any of those would work for you?

Nope. Multi-arch builds is the big feature of buildkit, I don't want to use another tool to combine manifest list.
In workflow like (webui + go service): (this should be common for projects)
%%{init:{'theme':'base'}}%%
flowchart TD

web_assets["web assets \n (only need to build on any one arch)"]
go_build_amd64["go build for amd64"]
go_build_arm64["go build for arm64"]
multi_arch_image["build multi-arch image"]

web_assets
--> go_build_amd64 & go_build_arm64
--> multi_arch_image

Without mult-arch support, we have to use
docker do push --platform=linux/amd64
docker do push --platform=linux/arm64
docker do manyfestlist

Another issue with those steps, web assets will be re-run for each arch.
With mult-arch support, we could simple run dagger do push to ship all images.
With platform declare in dagger files, it could clear actions' working platform.
So in another words,
--platform may use to filter working-platform-matched actions, instead of force defining all actions' working platform (like docker buildx did)
My recently projects, using dagger with mult-arch support (binaries: https://github.com/morlay/dagger/releases/tag/v0.3.0-beta)：
https://github.com/octohelm/cuemod/blob/main/dagger.cue
https://github.com/octohelm/kubepkg/blob/main/dagger.cue
https://github.com/innoai-tech/media-toolkit/blob/main/dagger.cue
Will make pipeline simple:
https://github.com/innoai-tech/media-toolkit/blob/main/.github/workflows/ci.yml#L28-L33

@morlay

Nope. Multi-arch builds is the big feature of buildkit, I don't want to use another tool to combine manifest list.

I completely get this and absolutely agree we want to take advantage of buildkit's builtin features in the long-term. The only concern I have right now is that this PR makes a lot of decisions on how we want to approach this problem, which means we are committing to it (or forcing us to make backwards incompatible changes in the future). That's why I thought using a separate tool could be a faster option since it would work already today (even if it has some caveats like you said).
If we can find a way to get this same functionality but only by using experimental command line flags rather than modifying our core actions in cue, that might be another somewhat quicker path (but we'd need sign-off from other team members too of course). I could imagine something like this:

We update the --platform experimental arg to accept multiple platforms. This results in builds being performed for each platform.
If multiple platforms are specified there, the push action will push a manifest list instead of just a regular single image manifest

The only problem I'm not sure about is the one you mentioned where your web assets step is platform-agnostic and thus shouldn't be repeated for different platforms. Some of the LLB ops in buildkit are platform agnostic (local sources and mergeops come to mind), so if you can provide some more details on how that web assets step is implemented, I wonder if we can find a way to get that to work too.

@sipsma

If multiple platforms are specified there, the push action will push a manifest list instead of just a regular single image manifest

It not change old way, still could use #Push & { image: "" } to push single image.
Only use #Push & { images: {} } will push manifest list with each platform but tag on the manifest list.
tag on manifest list is better than arch-suffix, it could let use change nothing of k8s manifests to deploy to k8s cluster with any arch node, even to k8s cluster with mulit-arch nodes like me.

The only problem I'm not sure about is the one you mentioned where your web assets step is platform-agnostic and thus shouldn't be repeated for different platforms. Some of the LLB ops in buildkit are platform agnostic (local sources and mergeops come to mind), so if you can provide some more details on how that web assets step is implemented, I wonder if we can find a way to get that to work too.

# For each step, it is run on docker container. so platform is the input too.
# go bulild will use the build.web's output for `go:embed`
$ npm run build.web | go build

#  when dagger do build --platform=amd64
$ npm run build.web --platform=amd64  | go build --platform=amd64

# when dagger do build --platform=arm64
$ npm run build.web --platform=arm64  | go build --platform=arm64
# the platform changed, and the cache will be hint.
# `npm run build.web` will pull node for arm64 first, then repeat run install & build.web

Like docker did, I prefer declare platform in dagger.cue, instead of use --platform to switch.
FROM --platform=${BUILD_PLATFORM} node:alpine

/not_stale

This PR is stale because it has been open 14 days with no activity. Remove stale label or comment or this will be closed in 7 days.

not stale

Hi @TomChv @shykes @aluzzardi @helderco @sipsma
Is there a conclusion？does dagger want to support this feature?

Hi @morlay, we discussed this PR the other day as a team and decided that while we all completely agree this feature is extremely important and we need to support it in the fullness of time, we currently don't have time to give its design+implementation the attention it requires, so it would be best to close it for now. To be clear, that's not to say there's anything wrong with your change, it's just that with any foundational feature like this we need to spend a lot of time validating it and considering as many future possibilities as we can to ensure it will be compatible with everything else we have planned.
The other thing we agreed on as a team is that we truly appreciate all the effort you've put into this and are very sorry that we have to reject the change for now. It's not something we do lightly since we love community contributions and adding new features like this that make Dagger better+easier to use. We just have to balance that against other practical considerations right now.
On our side, we are going to be more careful about being clear upfront on whether PRs are likely to be merged or not so we end up making the best use of everyone's time. Also though, we recommend first running proposed features+designs by Dagger maintainers and getting confirmation that it's the right time to pursue them before diving into the implementation. This is briefly mentioned in our online contributing docs, but I'll also update our CONTRIBUTING.md to make this more clear.

Hi @morlay, we discussed this PR the other day as a team and decided that while we all completely agree this feature is extremely important and we need to support it in the fullness of time, we currently don't have time to give its design+implementation the attention it requires, so it would be best to close it for now. To be clear, that's not to say there's anything wrong with your change, it's just that with any foundational feature like this we need to spend a lot of time validating it and considering as many future possibilities as we can to ensure it will be compatible with everything else we have planned.
The other thing we agreed on as a team is that we truly appreciate all the effort you've put into this and are very sorry that we have to reject the change for now. It's not something we do lightly since we love community contributions and adding new features like this that make Dagger better+easier to use. We just have to balance that against other practical considerations right now.
On our side, we are going to be more careful about being clear upfront on whether PRs are likely to be merged or not so we end up making the best use of everyone's time. Also though, we recommend first running proposed features+designs by Dagger maintainers and getting confirmation that it's the right time to pursue them before diving into the implementation. This is briefly mentioned in our online contributing docs, but I'll also update our CONTRIBUTING.md to make this more clear.

Fine. I think I have to use my forked dagger. https://github.com/morlay/dagger/releases/tag/latest


PMID:25798942

decreased nucleosome occupancy at 5S rrna
decreased nucleosome occupancy at trna
increased transcription at subtelomere
decreased protein localization to subtelomere
increased transcription at subtelomere

decreased nucleosome occupancy at 5S rRNA genes FYPO:0005915
decreased nucleosome occupancy at tRNA genes FYPO:0005916
increased transcription at subtelomere FYPO:0005917
decreased protein localization to subtelomeric heterochromatin FYPO:0005918
I did the FYPO:0005918 name to match existing protein localization terms. Maybe one of these days we and GO will make the whole thing more internally consistent ...
edit file: 5f3455d3e3e70ee21bb870057fffb9e3cb1e38c7
release: fc7f9cae4a46cd9cc0ec5a1aed6331f080510eb2


Fix load error

Context
While I was implementing this plugin, a plugin loading error occurred.

First, I tried to update v2.361.4 because Pipeline API required it
https://gyazo.konboi.com/e588d921ce117e8781b7763e22b5efca.png
However, the error still occurred. So, I updated to v2.387.3.

Unfortunately, error wasn't resolved

Finally, I updated to the Jenkins version to v2.401.3 and loading error was resolved.
Ask

Could you try to reproduce this issue on your local environment? If you cannot reproduce this issue, I'll fix my local env.
Honestly, I'm not sure these version combinations and what should we choose the Jenkins version. Could you review around that carefully?

Jenkins store everything in a directory, called "$JENKINS_HOME", including all the plugins installed. When you are developing a plugin like this, $JENKINS_HOME is ./work.
When I start Jenkins clean with empty home, I do not get any errors, which suggests you probably have accidentally updated some plugin versions. Without the logs showing the plugin loading failure, it's hard to precisely pin down why, but probably there's no point.
I suggest you start by rm -rf ./work/plugins which just wipes the plugins, forcing the plugin dev environment to start with its default set of compatible plugins. I believe that should fix the problem.

thank you, I'll try to do rm -rf ./work/plugins.

Thank you, the original error was resolved!
I'll close this PR.


Target ci/circleci failing for test TestTCPMixerFilterV1Config

I suspect this may be related to recent work on v2.

Any status update on this issue?

I believe this has been fixed as it hasn't been blocking any PRs.  Closing for now.


Adding bulletproofs and gadgets

I don't like the idea of passing tons of parameters to functions. Can we encapsulate this into a struct with a new method that takes some of them?

Ok, will try

I think the code formatting commit is a result of running cargo fmt. If that's the case, a best practice is to run cargo fmt before you commit. If you forget and do it later you can always squash the commits together too. Having two separate commits makes the review more complicated.
Also, as always please follow commit message standards: https://chris.beams.io/posts/git-commit/#seven-rules

I am wondering about the differences between this implementation of bulletproofs and dalek.
Both are in rust, much of the code here seems very similar to the dalek code, or derivative therefrom.
What are the reasons for not just using dalek?

@brentzundel Dalek's implementation uses the Ristretto curve. This one uses BLS12-381 by default but changing to other curves supported by Milagro can be done by changing a feature flag

Would it make sense to submit a PR to bring the capabilities of this codebase into dalek?

Would it make sense to submit a PR to bring the capabilities of this codebase into dalek?

The difference in the functionality of the 2 codebases comes from gadgets. Don't know if they would be interested as the gadgets don't add any functionality to the framework but are applications of the framework.

Lets put this PR on hold for 2 days as I am trying to integrate Bulletproofs in Zmix


Add support for Depth of Field

Fix #45
TODO

[x] Add functions for triangle, aperture shape sampling
[x] Add physical camera parameter uniform struct
[x] Add physical camera uniform to shader
[x] Documentation
[x] Investigate focus distance, focal length, fstop relationship
[x] Add jittered sample for DoF

add fstop, focal length integration to support bokeh scale
needs sensor gauge? Or implicit in focal length computation?

[x] Example
[x] Example finish

add background explosions / emissive particles for bokeh effect
make background color solid / gradient
add "sticky" focus?

[ ] Verify material reducer is deduping materials (background stars)

Triangular Bokeh


Keycloak Upgrade to 4.8.3 (RH-SSO 7.3.0)

Hi,
We are upgrading Keycloak to 7.3. No major feature changes expected for existing realms, you can find the documentation here https://www.keycloak.org/archive/documentation-4.8.html
We are planning to push the upgrade to SSO-DEV and have the community test it out. Scheduled deployment around first week of May. The testing period will take about one to two weeks.
If this plan does not work for you, please comment in this issue. Thank you!

While the use of the nonce parameter is optional when integrating with BC Services Card (BCSC), it is recommended.  If this upgrade provides Keycloak with the ability to support this then my vote would be to have this scheduled to allow us to proceed with the Registries/BCSC integration.

Here is the KeyCloak support ticket (https://issues.jboss.org/browse/KEYCLOAK-6147) to allow nonce in auth request which @ejarmstr requested above.

Theme updates:
initial work done on styling, waiting for design from Design System team.

I have worked with WAM team, and we have made some changes to our integration with dev, for IDIR and BCeID, so now there should only be 1 AudienceRestriction element.
Before Change:
<ns2:Conditions NotBefore="2019-07-19T16:15:18Z"
                NotOnOrAfter="2019-07-19T16:17:18Z"
                >
    <ns2:AudienceRestriction>
        <ns2:Audience>https://sso-dev.pathfinder.gov.bc.ca/auth/realms/idir</ns2:Audience>
    </ns2:AudienceRestriction>
    <ns2:AudienceRestriction>
        <ns2:Audience>https://sso-dev.pathfinder.gov.bc.ca</ns2:Audience>
    </ns2:AudienceRestriction>
</ns2:Conditions>

After changes:
<ns2:Conditions NotBefore="2019-07-19T16:20:10Z"
                NotOnOrAfter="2019-07-19T16:22:10Z"
                >
    <ns2:AudienceRestriction>
        <ns2:Audience>https://sso-dev.pathfinder.gov.bc.ca/auth/realms/idir</ns2:Audience>
    </ns2:AudienceRestriction>
</ns2:Conditions>

I will try a quick test with SSO 7.3 after business hours when I have a chance.

I've done some tests by removing the route from devops-sso-dev (service: sso-dev) and adding to devops-sso-sandbox (service: sso-sbox-22). I've tried logging in as a "new" account, as well as re-login as an existing account (Same account previously created). They both worked as expected. I think we are now unblocked to attempt an upgrade again.
PS: We still need to request the same SiteMinder/SAML changes (as above) for TEST and PROD.


Question regarding error metrics/dataset creation

I had a few questions/clarifications regarding the hdf5 dataset that was linked on the notebook:

I ran the notebook for training from scratch using the existing hdf5 and obtained a CER of ~0.09 using just a single model (and not an ensemble).
When creating the hdf5 from scratch and running the training procedure my CER is similar to the best/second best models (~0.16-0.18).

So, as far as I can see the main difference would be in the dataset generation/preprocessing steps or the tokenizer:
a. In the notebook there's a comment that the pretained models used a vocab size of 100 as opposed to 99 (95 characters + SOS/EOS/PAD/UNK tokens)- is there an additional token used here?
b. Was the generation procedure of the hdf5 that was linked/on the google drive a little different?
Thank you!

I also don't remember exactly the first iteration but I am working on a paper with the different experiments involving pre-processing steps that will help the community. I will update you once it is finalized.


Use multiprocessing to try multiple word combinations in parallel

Can try to simultaneously generate e.g. 4 boards by calling the MiniZinc script with 4 different data files and taking whichever completed first
Before this, should do the same with serial retry logic

Added in https://github.com/jonadsimon/wonder-words-generator/commit/fdebbf9dbe4ad72882cdba2de0fed9a1decc9f96
Didn't need multiprocessing after all, instead just looped over subprocess.Popen() 4x times and manually managed jobs.


Permissions problem with MacOS SMB

I'm on Mac M1 Ventura 13.5.1, I've set a 16TB USB storage in retronas + samba + samba MiSTer as pi user on my Raspberry Pi4, fixed all permissions, then I successfully log in from MAC to retronas SMB as pi, but I can only create new files / folders, not delete existing ones (for example, I want use only some games directories in "MiSTer" and not use the "saves" dir...
Logging in Rasp as pi user and deleting a dire in mister with "rm -r saves" works...
[EDIT] Ok, I've tried to delete a subfolder of "saves" like "Adam" and the problem is that Finder says that the element is "in use"...

you'll need to do this from the server side, retronas is not setup for custom dir management, running the installer again will recreate all the folders

OK, I understand that running the installer again will recreate all the folders, what I don't understand is why I can't delete some folders having the permissions to do that.

The documentation for the samba project is available here https://www.samba.org/samba/docs/
afaik this is a limitation using symlinks with samba, they are not standard directories even though they present as such on the share. This issue is not limited to mac, i have a similar issue on linux clients accessing over samba and if i find a solution i'll implement a fix but i don't know of one atm.
you can however rename the folder as a workaround so the mister doesn't find them or remove them serverside

Ok, now it's clear, thanks!


Module included in Alpine Linux and nginx-binaries

Hi, you might be interested to know that I’ve added this module to Alpine Linux – nginx-mod-http-keyval (it’s available since v3.18) and also to nginx-binaries.

Ah, I forgot about it, I’m gonna enable it.

It’s built with the Redis store now (nginx-mod-http-keyval). I’ve also really added it to nginx-binaries – I though that I already did it, but forgot to finish it before.


Building wrong Target when building for Mac Catalyst

When building for mac catalyst (which is available in the rust master branch as of some time) I faced the problem that backtrack-sys was either trying to build for the iPhone Simulator or for MacOSX but not for the new Macabi (The cc crate  seems to decide based on if there is ios or darwin in the target name). In the target configuration for Xargo there is a field called llvm-target, which essential is the target needed to be set for the CC.
The problem showed up upon linking in xcode (or generating a xcframework), where it claimed that the platform for one of the objects in the static library (state.o) was not built for Macabi.
To circumvent the problem I could, in the end, just apply the CFLAGS="-target x86_64-apple-ios13.0-macabi" to the xargo build-command (where the target is as mentioned above the llvm-target specifier for the custom target configuration for Xargo), which overrides the old compiler flags and uses the correct target. I though get warnings since the previous compiler flags are ignored (related to ios e.g.)
Cheers
Patrick

@ubamrein , Did you try the idea  that you presented?

yes this worked (except for the warnings) in the end without problems. Though Apple is restrictive regarding the APIs which are exposed on Mac Catalyst/iPhone... so I never followed this lead further


JLink ProgramUploader swallows stream output

This (failing) test demonstrates the problem:
import asyncio
import nanaimo
from nanaimo.fixtures import SubprocessFixture, FixtureManager
import re

def test_subprocess_filters():
    """
    Executing JLinkExe breaks subprocess output stream filtering.
    We replicate this by searching the output for a simple error
    which appears in stdout from JLinkExe when run from a shell, but
    is missing when run as a SubprocessFixture command:
    """

    class TestSubprocessFixture(SubprocessFixture):
        def on_construct_command(self, arguments: nanaimo.Namespace, inout_artifacts: nanaimo.Artifacts) -> str:
            return 'JLinkExe -CommanderScript MISSING_FILE_'

    loop = asyncio.get_event_loop()
    manager = FixtureManager(loop=loop)

    async def check_stdout(manager):
        subprocess_fixture = TestSubprocessFixture(manager)
        filter = SubprocessFixture.SubprocessMessageAccumulator()
        subprocess_fixture.stdout_filter = filter
        await subprocess_fixture.gather()
        return len(re.findall("Could not open J-Link Command File 'MISSING_FILE_'", filter.getvalue()))

    result = loop.run_until_complete(check_stdout(manager))
    expected_count = 1  # <-- this is correct, currently causes this test to fail
    #expected_count = 0  # <-- this is the issue, the line is lost
    assert expected_count == result

As written it produces this output:
>       assert expected_count == result
E       assert 1 == 0

I've found a few workarounds that both have the downside of buffering: switching from readline() to read(), or using communicate() in SubprocessFixture.on_gather() like this:
class SubprocessFixture(Fixture):
    …
    async def on_gather(…
        …
        proc = await asyncio.create_subprocess_shell(…
        stdout, stderr = await proc.communicate()

This is a well written bug. Thank you @karlin . I'll look into it immediately.


Fix resize issue with jquery-ui and flexbox

This PR fixes the resize issue we were experiencing with the 3D brain panel.
Background
Flexbox and jquery-ui resizable didn't play well together under the old page scheme. The solution was to transition to a new html class scheme involving the classes panel-container, panel-left, panel-right, panel-container-vertical, panel-top, panel-bottom. We nest these classes so that the main left panel belongs to both the panel-left and panel-container-vertical classes, and so on. I think the end result is more readable and intuitive. The solution does involve one weird hacky fix, which you can find in the css for #threejsbrain-with-title.
Potential issues
This changes the html class names once again. Sorry about that. Specifically, @anotherjoshsmith if this PR gets accepted before your #58, I can help with rebasing to resolve any class name conflicts in index.html.
Remaining work
We should probably toy around with the min-width and min-height fields of some of the divs in the page. I leave it up to the group whether you want that to be part of this PR or if we defer that and just create another issue for it. My personal preference is to defer because I think we need to play around with it on different screen sizes and want to get the fix to the resizing out sooner than that.

Not sure why this is showing so many commits. On my local machine, I only added three commits to what I pulled from upstream master.

Addresses #43 and #38.


How do I get access to the functionalities in Popper without styling?

I cannot import the CSS(it's a requirement I have from above), we use TypeStyle.
So basically I have to re-implement the styling in TypeStyle(and change it, of course to fit :) )
I can style the DatePicker component itself fine, but how do I access the Popper that trigger when selecting the Input element?
Basically, I get something like this now(I've added basic styling..):

Is there a boilerplate of CSS to get a working Popper out? I mean, with the options to switch month etc, and see each date in correct rows/columns? I find it difficult to work out from it without the .css already implemented.
Here's some sample code I have so far:
        const popperCSS = style(
            {
                backgroundColor: "#fff",
                color: "grey",
                border: "1px solid rgba(209,197,195,0.6)",
                display: "inline-block",
                position: "relative",
                height: 225,
                width: 200,
            },
        );

                <DatePicker
                    placeholderText="dd.mm.yyyy"
                    popperPlacement={"left-start"}
                    className={datePickerCSS}
                    popperClassName={popperCSS}
                    dateFormat="DD.MM.YYYY"
                    selected={this.state.fromDate}
                    onChange={this.handleFromDateChange}
                />

Thanks in advance.

Why can't you use CSS? Does TypeStyle somehow break the browser's support for CSS?
If you need help with TypeStyle, I don't think this is the right place.
If you're allowed to use <style> tags, dist/react-datepicker.css is all that should be needed.
If your intent is to re-implement all the existing CSS, you'll want to look at the source in src/stylesheets/, though keep in mind that would basically be a fork you'll have to maintain yourself.

I have no processor/loader for css.
I've looked at the scss source but it's a bit difficult to figure out how it would look in typestyle or just regular non-minified css.

What browser doesn't support CSS these days? Some sort of react-native thing?

Ok... And you are somehow able to load that CSS into the browser, right?
Is there some reason you're unable to load the react-datepicker CSS into the browser in the same way?

Yes, the reason is i'm on a major project and with the tech setup, it's not possible to just load pure css in the structure(to maintain consistency/development workflow).
Technical debt, and all that jazz :)


generate "!$OMP END parallel do" for Fortran tests

The Fortran hierarchical_parallelism tests were correctly generating
"!$OMP parallel do" for construct start, but construct end was being
incorrectly generated as "!$OMP END parallel for".  Fix in the same
way the start construct is handled, by replacing "for" with "do".

Thanks a lot!


[MNT] - Update thresholds in occupancy calculations

Responds to #179
Previously, it was sort of confusing the speed_threshold was a MINIMUM value, but time_threshold was a MAXIMUM value, and also, this scheme only allowed for a subset of thresholding one might want to do (not allowing, for example, for dropping entries with too short a time duration.
This PR therefor updates speed & time threshold to use explicit min_x and max_x threshold values for both, which I think is probably better.
The only sorta annoying thing is that this is a breaking change (changes the argument names), so some older code will need to be updated. Any thoughts on this are welcome!

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

Comparison is base (1f417d3) 98.75% compared to head (408b0ba) 98.78%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #185      +/-   ##
==========================================
+ Coverage   98.75%   98.78%   +0.03%
==========================================
  Files          98       98
  Lines        3690     3699       +9
==========================================
+ Hits         3644     3654      +10
+ Misses         46       45       -1

Files
Coverage Δ

spiketools/spatial/occupancy.py
98.13% <100.00%> (+0.90%)
:arrow_up:

spiketools/tests/spatial/test_occupancy.py
100.00% <100.00%> (ø)

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


Go to type definition should go to typedef

Even after the fix for https://github.com/dart-lang/sdk/issues/56844 I'm still having an issue when working on the SDK. This is the link for the base change I made to see the problem https://github.com/FMorschel/sdk/tree/go-to-type-definition-inside-expanded.
Here is a repro of that part inside pkg/analysis_server_plugin/lib/src/correction/fix_in_file_processor.dart:
return [
  ...?registeredFixGenerators.nonLintProducers[errorCode],
  ...?registeredFixGenerators.nonLintMultiProducers[errorCode]
      ?.expand((multiProducer) {
    multiProducer;
    return [];
  }),
];

In here, with the latest SDK (got mine updated today actually to see if this was solved) I can't seem to activate Go to Type Definition on either multiProducer from the parameter or the one from the useless statement.
I can activate it with no problem on registeredFixGenerators or nonLintMultiProducers.
I'm still looking at this to try and find a better repro but in the meantime, I was instructed by @DanTup to create this issue so that others can take a look as well.

I was thinking this would give me instances but it actually gives me constructors so because it is a Function there is no actual type here.

I was thinking of why I got confused there, and then I remembered that I only saw the type like a Function on the variable itself. The generics is tied to a typedef. I think in this case it could move us to the typedef definition (where it is defined by one, not where it matches). I'm not sure if that is possible but anyway, here is my proposal.

I think what you're proposing is that "Go to type definition", at least for a function type, should navigate to the typedef, when there is one. If so, I think that's reasonable.
For consistency we should probably do that for any type that was defined using a typedef.
I'm not sure whether we have that information available, but I think we do at least some times.

https://github.com/dart-lang/sdk/issues/39332 is a similar issue where showing the typedef is requested instead of the base type. I don't know if the cause or fixes are the same, but they might be.

I think I'll close this in favour of that one then since it is older and has more discussion. Thanks!

(I'm not certain one fix will fix both, one is about hovers and one is about Type Def... but they're similar enough that the fix could be the same even if it has to be made in two places)

If needed we can always reopen this again, I'll keep that in mind though

I found some problems with argument_type_not_assignable and typedefs as well. I think I'll reopen this so we have more clarity of whats needs to be done.


giveaway filters - multiple filters: add bookmarked as filter

Is your enhancement request related to a problem? Please describe.
I recently started to use giveaway bookmarks and I find them quite useful. Unfortunately at the moment I can't find a solution to filter those giveaways I already have bookmarked on the giveaway page. So if I still don't have enough points I have to scroll though and check every one if it is already bookmarked.
Describe the solution you'd like.
Add a bookmarked filter to multiple filters to allow to filter bookmarked giveaways.
Describe alternatives you've considered.
I haven't found a useful alternative so far. If there is one please let me know. Like I said at the moment I scroll through the list and check every game if it is already bookmarked.
Additional Context
Maybe it's something like "bookmark all giveaways on the page" would also do the job.

Interesting, I was sure I had already added that. lol


Use public client auth for odsp e2e flows

Description
Currently, we use the ropc flow with a confidential client to authenticate in our e2e/stress tests against odsp. This is less appropriate than using a public client flow, since our application registration really only needs to have delegated permissions.
This PR adjusts things to use a public flow--I've updated the relevant app registrations to allow both forms of auth already.
Using a public flow also means our infrastructure setup here aligns exactly with this relatively recent MSFT guidance.

Will PR checks properly validate that this works? Or is there a risk of outer-orbit pipelines failing?

No, they won't. I'm planning on verifying this with a test branch once I get PR checks passing.

Seeing one of the stages pass (at least with expected pass rate, so not failing due to auth issues) on my test branch here
I'll merge this now and get started on the cherry-picks to release branches.


"airflow" "DAG" strings in file necessary to import dag

Hello airflow team! Thanks for the awesome tool!
We made a small module to automate our DAG building process and we are using this module on our DAG definition. Our airflow version is 1.7.0.
However, airflow will not import this file because it doesn't have the words DAG and airflow on it.  Apparently there's a safe_mode that skips files without these strings.
(https://github.com/airbnb/airflow/blob/master/airflow/models.py#L233)
This safe_mode is default to True but is not passed to the process_file function, so it is always True and there's no apparent way to disable it.
(https://github.com/airbnb/airflow/blob/master/airflow/models.py#L204)
(https://github.com/airbnb/airflow/blob/master/airflow/models.py#L382)
Putting this comment on the top of the file makes it work for the moment and brought me a good laugh today 👯
#DAG airflow —> DO NOT REMOVE. the world will explode

👍

@Etiene would you mind creating a Jira issue for this? Thanks!

No worries! But where is that? Docs need updating: https://github.com/apache/incubator-airflow/blob/master/CONTRIBUTING.md#report-bugs

Added to Jira!


chore: adapt to simpleBinder removal

@Kha

Should I keep the branch in this repo as is for use in lean4, while switching this PR to a different one rebased on master?

Yeah, I think that makes the most sense.
I hit a similar issue with the tsyntax branch -- deleting it (after the rebased version was merged into master) left its commit orphaned and inaccessible, making it impossible to depend on the same version of Lake as was used for the Lean version it was a part of -- hence why I restored the branch. I encountered this issue when debugging Henrik's doc-gen update which depends on Lake and (at the time) was only updating to nightly 06-30. It needed the tsyntax Lake but that Lake did not exist in this repo.

continued in #97


chat mode --no-stream option

I am writing a plugin, and I've understood that this will tell llm to always be in --no-stream mode when these models are called:
class Chat(llm.Model):
    can_stream: bool = False

Is this what's needed and all that is needed? The plugin will only run when stream=False to manage upstream user expectations.
def execute(self, prompt, stream, response, conversation):

When debugging that method, stream correctly ends up being false, even without --no-stream when not in chat mode.
But it seems that stream=True is always sent to execute() when in chat mode, and using --no-stream via command line will disable the chat mode itself. Here are some test runs with the plugin, vs. OpenAI gpt4 alias. The behavior seems similar.
> llm "hi"
Hello! How can I assist you today?

> llm chat
Error: Sorry, this plugin doesn't support streaming yet.     # stream was set to to True by the host

> llm --no-stream chat
Hello! How can I assist you today?     # chat mode not applied but there is a greeting

> llm chat -m gpt4
Chatting with gpt-4
Type 'exit' or 'quit' to exit

> llm --no-stream chat -m gpt4
Hello! How can I assist you today?     # chat mode not applied but there is a greeting

> llm chat -m gpt4 --no-stream
Usage: llm chat [OPTIONS]
Try 'llm chat --help' for help.
Error: No such option: --no-stream Did you mean --system?

> llm chat -o verbose     # the default model does have this option in non-chat mode
Usage: llm chat [OPTIONS]
Try 'llm chat --help' for help.
Error: No such option: -o

Applying the plugin's own announced streaming boolean would fix this for me. Allowing explicit --no-stream while in chat mode is nice but I don't personally have a strong use case to offer for that, other than perhaps consistency.
Do you plan to support model options in chat mode? They are often needed with local models. The new mode is a very welcome addition, so thanks a lot for working on it.

#244 talks about chat options, that's good

I tested llm chat --no-stream manually against both GPT-4 and Llama 2, it worked as intended in both cases.


Trainer stuck

Hi !
I tried to execute the basic_training notebook on Google collab.
The trainer.train() phase is stuck in something an infinite loop for hours after printing only "#> Starting..."
Here is the code that is displayed when I interrupt the execution :
[/usr/local/lib/python3.10/dist-packages/ragatouille/RAGTrainer.py](https://localhost:8080/#) in train(self, batch_size, nbits, maxsteps, use_ib_negatives, learning_rate, dim, doc_maxlen, use_relu, warmup_steps, accumsteps)
    236         )
    237
--> 238         return self.model.train(data_dir=self.data_dir, training_config=training_config)

[/usr/local/lib/python3.10/dist-packages/ragatouille/models/colbert.py](https://localhost:8080/#) in train(self, data_dir, training_config)
    450             )
    451
--> 452             trainer.train(checkpoint=self.checkpoint)
    453
    454     def _colbert_score(self, Q, D_padded, D_mask):

[/usr/local/lib/python3.10/dist-packages/colbert/trainer.py](https://localhost:8080/#) in train(self, checkpoint)
     29         launcher = Launcher(train)
     30
---> 31         self._best_checkpoint_path = launcher.launch(self.config, self.triples, self.queries, self.collection)
     32
     33

[/usr/local/lib/python3.10/dist-packages/colbert/infra/launcher.py](https://localhost:8080/#) in launch(self, custom_config, *args)
     70         # TODO: If the processes crash upon join, raise an exception and don't block on .get() below!
     71
---> 72         return_values = sorted([return_value_queue.get() for _ in all_procs])
     73         return_values = [val for rank, val in return_values]
     74

[/usr/local/lib/python3.10/dist-packages/colbert/infra/launcher.py](https://localhost:8080/#) in <listcomp>(.0)
     70         # TODO: If the processes crash upon join, raise an exception and don't block on .get() below!
     71
---> 72         return_values = sorted([return_value_queue.get() for _ in all_procs])
     73         return_values = [val for rank, val in return_values]
     74

[/usr/lib/python3.10/multiprocessing/queues.py](https://localhost:8080/#) in get(self, block, timeout)
    101         if block and timeout is None:
    102             with self._rlock:
--> 103                 res = self._recv_bytes()
    104             self._sem.release()
    105         else:

[/usr/lib/python3.10/multiprocessing/connection.py](https://localhost:8080/#) in recv_bytes(self, maxlength)
    214         if maxlength is not None and maxlength < 0:
    215             raise ValueError("negative maxlength")
--> 216         buf = self._recv_bytes(maxlength)
    217         if buf is None:
    218             self._bad_message_length()

[/usr/lib/python3.10/multiprocessing/connection.py](https://localhost:8080/#) in _recv_bytes(self, maxsize)
    412
    413     def _recv_bytes(self, maxsize=None):
--> 414         buf = self._recv(4)
    415         size, = struct.unpack("!i", buf.getvalue())
    416         if size == -1:

[/usr/lib/python3.10/multiprocessing/connection.py](https://localhost:8080/#) in _recv(self, size, read)
    377         remaining = size
    378         while remaining > 0:
--> 379             chunk = read(handle, remaining)
    380             n = len(chunk)
    381             if n == 0:

Hey! This seems to be a multiprocessing issue. I'll try and figure out what it is, but this can often be finnicky with the upstream ColBERT lib 🤔
@shubham526 are you making sure to run your code within if __name__ == "__main__":? In both cases, the problem appears to be that the individual processes spun up by colbert-ai's multiprocessing handler don't actually ever start.

@bclavie Yes, I am running inside if __name__ == "__main__":

Could you share some more information about your environment then? This could maybe help point in the right direction! (torch version, cuda version, python version, pip freeze/overall environment (Ubuntu?))

requirements.txt
I have attached the output of pip freeze.

OS: Ubuntu 22.04 LTS.
Python version:  '3.10.10 (main, Mar 21 2023, 18:45:11) [GCC 11.2.0]'
PyTorch version: 2.0.1+cu117
CUDA version: 11.7

I tried this on another server and it worked. No idea why though.

Faced the same issue.
@jack-lac have you managed to resolve the problem?
@shubham526 do you run you source code on Google Colab?

Also facing the same issue on Google Colab.
I thought setting something along the lines of
trainer.model.config.nranks = 1
trainer.model.config.gpus = 1

would solve it, but it's not working either (stuck at "Starting"). It's totally upstream in Colbert's training code, and may be due to Colab being slightly more closed in terms of permissions. Skimming over the code, I noticed that it needs to open some ports for communication between worker processes.
I don't think wrapping the Colab code in an if __name__ == '__main__' would solve this


Replace getBoundsInWindow with getBoundsInScreen for lower SDK requirement

Flagged in https://github.com/fischman/DisableDistractions/issues/5#issuecomment-1865355645

FTR minSDK-violating API usages were flagged in Android Studio's Problems (File) tab, but sadly don't break the build so need to be looked for explicitly.


Download link still broken for script

https://docs.microsoft.com/en-us/microsoftteams/rooms/manual-update
The link to https://go.microsoft.com/fwlink/?linkid=2151817 is still broken and not working.

Thanks for the feedback. We're tracking this in #7135


Implement render preview

Users should be able to preview their effects on the Sequence Edit page. Pressing the space bar will activate the preview.
When the preview is activated:

The render preview REST service is called, with the current frame, max frame count and sequence channels passed as inputs
The REST service returns the rendered stage prop data
The audio file starts playing, and the current frame is incremented continually
Each time the current frame changes, the rendered data is shown on the stage preview

First cut is complete. Plenty more improvements can be made, but they'll go in separate issues.


Blog Posts Not In Order

On the /blog page the posts are showing things from 15+ years ago at the top of the list:

These should be in order by the date on the document, descending.

Thats just temporary, I will revert it back to descending order in my next deploy.

Thanks! I'm going to mention an article on the new site in my newsletter that goes out to 3000 people in 2 hours, so hopefully the site's mostly working (and the article is published).

FYI, I updated gatsby-node.js to boost the limit from 1000 to 10,000.

Seems helpful - adding for future reference:
https://developer.okta.com/blog/2020/02/18/gatsby-react-netlify

Thanks for sharing. ☺
I have updated below changes you requested and also all blog posts are
updated too. You're good to go for launch now. Once you point to new
domain, I will update the Google Analytics plugin as well:

Add Publish date to category page
Remove Square label "blogging" from blog post page
Change Background of square label over headerimage on all pages
Install Prism Plugin for Code Block Styling
Mailchimp Configured - Please test before launch

On Wed, May 27, 2020 at 6:52 PM Steve Smith<EMAIL_ADDRESS>wrote:

Seems helpful - adding for future reference:
https://developer.okta.com/blog/2020/02/18/gatsby-react-netlify
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/ardalis/ardalis-com-gatsby/issues/1262#issuecomment-634756633,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APM534Q6EEUE2CPW4HM3U4TRTUZNRANCNFSM4NLWYETQ
.

I'm good to launch now? Yay!
So it sounds like all I need to do is point ardalis.com at the new location (for which I will probably move my DNS hosting over to Netlify entirely).
Your 5 things listed above you will do after I update DNS? Or what is the plan for those?

Looking at the Blog tab now, these 4 posts are NOT from May 2020:

I haven't checked further, but it's important that the publish dates match what they were previously in WordPress after this migration. Can you please double-check and correct? Thanks!

I'm going to try and create separate issues for each task we both have so that we can track them individually and close this one.


Select component by routes query

What problem does this feature solve?
At the moment, it is not possible to show a different component by the query.
It should be possible to use the ? as part of the route instead of just to make parts optional, possibly by escaping it.
What does the proposed API look like?
{
  path: "/app/user\?userId=:userId",
  name: "user",
  component: UserView,
},
{
  path: "/app/user\?action=delete&userId=:userId",
  name: "user",
  component: DeleteUserView,
},

This is intentional. You can still define custom query behavior within a navigation guard or directly use a query properly and named views together to build the behavior you want.


How to open adv_script file?

When decoding to utf8, some Japanese characters are broken.

Do you mind posting an example? When I open one in Notepad++ using UTF-8, I don't see broken characters, but I do see a bunch of special characters (e.g x80). Unfortunately, I don't know the exact format of the adv_script files.


Doesn't work on AWS Elastic beanstalk

I tried to use this in an AWS Elastic Beanstalk application. Works just fine on my Mac, but when I deploy, it fails to install the sleep package.

3631 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>3632 silly lifecycle<EMAIL_ADDRESS>no script for preinstall, continuing
3633 info linkStuff<EMAIL_ADDRESS>3634 silly linkStuff<EMAIL_ADDRESS>has /tmp/deployment/application/node_modules as its parent node_modules
3635 verbose linkBins<EMAIL_ADDRESS>3636 verbose linkMans<EMAIL_ADDRESS>3637 verbose rebuildBundles<EMAIL_ADDRESS>3638 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>3639 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle false
3640 verbose lifecycle<EMAIL_ADDRESS>PATH: /opt/elasticbeanstalk/node-install/node-v6.9.1-linux-x64/lib/node_modules/npm/bin/node-gyp-bin:/tmp/deployment/application/node_modules/sleep/node_modules/.bin:/tmp/deployment/application/node_modules/.bin:/opt/elasticbeanstalk/node-install/node-v6.9.1-linux-x64/bin:/opt/elasticbeanstalk/lib/ruby/bin:/usr/local/sbin:/usr/local/bin:/usr/bin:/usr/sbin:/sbin:/bin
3641 verbose lifecycle<EMAIL_ADDRESS>CWD: /tmp/deployment/application/node_modules/sleep
3642 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'node-gyp rebuild' ]
3643 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
3644 info lifecycle<EMAIL_ADDRESS>Failed to exec install script
3645 verbose stack Error<EMAIL_ADDRESS>install: node-gyp rebuild
3645 verbose stack Exit status 1
3645 verbose stack     at EventEmitter. (/opt/elasticbeanstalk/node-install/node-v6.9.1-linux-x64/lib/node_modules/npm/lib/utils/lifecycle.js:255:16)
3645 verbose stack     at emitTwo (events.js:106:13)
3645 verbose stack     at EventEmitter.emit (events.js:191:7)
3645 verbose stack     at ChildProcess. (/opt/elasticbeanstalk/node-install/node-v6.9.1-linux-x64/lib/node_modules/npm/lib/utils/spawn.js:40:14)
3645 verbose stack     at emitTwo (events.js:106:13)
3645 verbose stack     at ChildProcess.emit (events.js:191:7)
3645 verbose stack     at maybeClose (internal/child_process.js:877:16)
3645 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:226:5)
3646 verbose pkgid<EMAIL_ADDRESS>3647 verbose cwd /tmp/deployment/application
3648 error Linux 4.4.23-31.54.amzn1.x86_64
3649 error argv "/opt/elasticbeanstalk/node-install/node-v6.9.1-linux-x64/bin/node" "/opt/elasticbeanstalk/node-install/node-v6.9.1-linux-x64/bin/npm" "--production" "rebuild"
3650 error node v6.9.1
3651 error npm  v3.10.8
3652 error code ELIFECYCLE
3653 error<EMAIL_ADDRESS>install: node-gyp rebuild
3653 error Exit status 1
3654 error Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
3654 error Make sure you have the latest version of node.js and npm installed.
3654 error If you do, this is most likely a problem with the sleep package,
3654 error not with npm itself.
3654 error Tell the author that this fails on your system:
3654 error     node-gyp rebuild
3654 error You can get information on how to open an issue for this project with:
3654 error     npm bugs sleep
3654 error Or if that isn't available, you can get their info via:
3654 error     npm owner ls sleep
3654 error There is likely additional logging output above.
3655 verbose exit [ 1, true ]

sleep is a C++ module so I guess you still have to install the toolchain for this? See: https://aws.amazon.com/blogs/compute/nodejs-packages-in-lambda/


Make an amazing style of cursor

Pull Request Checklist
#1380

[ X] I have added screenshots and videos to show before and after the working of my code.

Uploading Screen Recording 2024-10-25 002621.mp4…

[ X] I have ensured that the screen size is set to 100% while making the video.
[ X] I have synced the latest fork with my local repository and resolved any conflicts.
[ X] I have mentioned the issue number which I created before making this PR .(format to mention issue number is : fixes #(issue number) )
[ X] I understand that if any the above conditions are not met , my PR will not be MERGED .

the issue you have linked to the PR is not even related to the PR and is not even yours
2.I don’t like the cursor style, and you also made a PR without the issue being assigned to you. Why bypass the process? Please make sure to get an issue assigned before working on it. I’ll be closing this PR as the changes aren't needed.


Make launch button configuration more extensible and flexible

Description
Over time people have contributed several extensions to our launch button infrastructure. Instead of just having two options of Binder or JupyterHub, we've now also got options for Colab (and Deepnote on the way). Moreover, we've had cases where people wanted to provide launch buttons for multiple JupyterHubs, and the current setup is not extensible in this way.
Instead, we could have users configure their launch button links via a list of entries, where each entry was a dictionary that defined the configuration for that link.
Guide for implementation
Here's one example of a more configurable structure we could use for inspiration:
html_theme_options = {
  "launch_buttons": [
    {"type": "jupyterhub", "url": "hub.myorg.edu"},
    {"type": "jupyterhub", "url": "otherhub.org2.edu"},
    {"type": "binder", "url": "mybinder.org"},
    {"type": "colab"},  # Because there's only one deepnote, we don't need url
    {"type": "deepnote"},  # Because there's only one deepnote, we don't need url
    {"type": "custom", "icon": "fontawesome class", "url_pattern": "https://myorg.com/{org}/{repo}/{path_to_page}"}  # This could be a way to specify custom launch URLs if desired
  ]
}

This would allow people more flexibility and control over their launch buttons, and be an easier extension point than our currently messy combination of configuration flags and values. This is similar to how the pydata theme does their icons.

We would find this useful too


Added CA Certificate Note

Based on recent customer issue, the current behavior of Self-Hosted gateway is that it does not present all intermediate certificates in the chain to the client connecting to the gateway. This can cause SSL handshake failure for clients depending on the certs being presented. Managed gateway already presents interim certs to the clients. We have a bug open to address this in self-hosted gateway, but until thats fixed, we need add this documentation.

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

@dlepow - Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


New Map size gives false ruler length

Based on https://hunt-map.info/

Based on this tool
Seems like the map has been reduced by half, but ruler length didn't follow

Oh, you're right! That should be an easy fix, thanks.

Fixed in latest commit.  Also noticed that the coordinates under the mouse were incorrect in the status bar, so that's fixed as well.


Add Per-DS HTTP/2 and TLS Versions Support

Adds per-DS H2 and TLS version support, via ssl_server_name.yaml and sni.yaml config generation. See the docs in the PR for details.
Includes tests.
Includes docs.
Includes changelog.

[x] This PR is not related to any other Issue

Which Traffic Control components are affected by this PR?

Traffic Ops ORT

What is the best way to verify this PR?
Run tests. Run t3c with new args and verify ssl_server_name.yaml or sni.yaml is created appropriately with specified defaults, add parameters to configure a DS and run t3c and verify files are created with specified settings.
If this is a bug fix, what versions of Traffic Control are affected?
Not a bug fix.
The following criteria are ALL met by this PR

[x] This PR includes tests OR I have explained why tests are unnecessary
[x] This PR includes documentation OR I have explained why documentation is unnecessary
[x] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[x] This PR includes any and all required license headers
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Additional Information

Tested with a test box changing between an 8x and 9x profile. First on 8x saw it properly generate the ssl_server_name.yaml based on the cmd line options I used (default-client-enable-h2=true and tls-versions=1.3,1.1). Then changed it's profile to a 9x profile, saw it generate the sni.yaml (and the ssl_server_name.yaml), how the sni version had disable_h2: true, and listed all tls variants in the list. The ssl_server_name was still correct, but since this is a 9x profile it would be unused

problem fixed with latest pushes (fixed cmdline issues, duplicate files are because of params). Only issue that I see left is if you give it a bad param name for a TLS version, you end up with an empty entry in the list of tls_versions in the yaml file

@ezelkow1 Fixed. It did have a check for that, but the if { warn was missing a continue. Fixed, and added a unit test for that.


Clean some code

Here's a list of things that can be cleaned:
https://github.com/snapshot-labs/sx-core/blob/0f81eddad82008762e4e6faf0205f337fdb0481f/contracts/starknet/space/space.cairo#L331-L336
https://github.com/snapshot-labs/sx-core/blob/0f81eddad82008762e4e6faf0205f337fdb0481f/contracts/starknet/space/space.cairo#L283-L288

this should be BigInt("...")
https://github.com/snapshot-labs/sx-core/blob/0f81eddad82008762e4e6faf0205f337fdb0481f/test/starknet/shared/setup.ts#L12

this is wrong: can be put before the execute call because the whole tx will revert if there's an error anyway..
https://github.com/snapshot-labs/sx-core/blob/0f81eddad82008762e4e6faf0205f337fdb0481f/contracts/starknet/space/space.cairo#L429-L432


Add bset instruction.

Single-Bit Set (Register) instruction
Mnemonic: bset rd, rs1, rs2
Opcode : 1100110.
This instruction returns rs1 with a single bit set at the index specified in rs2.
The index is read from the lower log2(XLEN) bits of rs2.
Operation:
let index = X(rs2) & (XLEN - 1);
X(rd) = X(rs1) | (1 << index);

Please fix build + resolve merge conflicts.


Change Component Library and write component tests

Closes #73
Changes made by this Pull Request

Replace Inkline with Element Plus
More and working component tests

Definition of Done

[x] Tests are written and run successful
[x] Code is documented
[x] User Manual is updated
[x] All requirements of the user story are met

ToDo

[ ] Change font
[x] actually write component tests

We decided to abandon that (for now) because it causes some issues with the e2e tests and the new component tests do not add much value now and are redundant to the e2e tests.


Fix Queue Provider & clean code with helpers

#728 after an investigation by the hero without a cape @bastien-phi, the exact issue was found and we have a potential fix.

Cleaver very and a nice idea!

Please

https://github.com/hyn/multi-tenant/pull/749#issuecomment-474397884

Facebook longin please


fix: re-auth when cookie expires

401 relogin kplsthx

🚀

Also added a config option for change the site, it defaults to default


Fix some Unhandled promise rejections and trim name and surname

Solving this issue #1
A function trimming the user's first_name and last_name was added on the Session.ts file.
Also, some checks have been added to the setaddr and ls commands, so that they fail gracefully if some necessary steps haven't occured.

@kpapadatos friendly bump 😄


Prevent cutting line when reading from procfile.

Hello!
Thanks for making a great library- we very much enjoy it at NoRedInk! However, I ran into some trouble when my procfile had very long lines. I believe this fixes it, but of course, you will be the final judge of that. I realize sometime things are more complicated than they look!
Explanation for the fix:
ReadLine() cuts off line when it gets too long for the buffer. ReadBytes('\n') does not. Read more here: https://golang.org/pkg/bufio/#Reader.ReadLine
Thanks for reading!

Hi Tereza,
Thanks for the contribution, but the problem was a bit complex. I fixed it with this commit https://github.com/DarthSim/overmind/commit/484043abb4c16de389bfcecbf0e52bd4e8f751b7


Fix requestLimit parameter verification in CommentNode

The bug is obvious from the diff.

Sorry for the long wait time! Merging.


[W5.11][W10-A2]Paul-Henri MIGNOT

Adding the Edit command, just able to edit name, but others contact information will be easy to add to this command.

@phmignot

In general, good work on this. Keep it up!
Some comments added. Please ack & close the PR after reading comments.

Changes applied


don't allow address(0) in transferFrom

reference issue: https://github.com/code-423n4/2022-06-yieldy-findings/issues/249

🙄


Default type for Context and Handler prevents extension or override

if i define a handler like so,  and attempt to register it with other handlers, tsc will throw errors.
export const myHandler: Handler<
    unknown,
    { id: string }
> = async (
    c: OpenAPIBackendContext<unknown, { id: string }>,
    ctx: KoaContext,
): Promise<void> => {}

export const v1InternalAPI = new OpenAPIBackend({
    apiRoot: '/v1',
    definition: __dirname + '../../../api/service.v1.internal.yaml',
    // this is needed to honor the "format" field in the openapi definition
    customizeAjv: (originalAjv) => {
        addFormats(originalAjv);
        return originalAjv;
    },
});

// register your framework specific request handlers here
v1InternalAPI.register({
    postResponseHandler,
    validationFail,
    notFound,
    myHandler,
});

the errors
src/routes/v1/router.ts:27:5 - error TS2322: Type 'Handler<unknown, { id: string; }>' is not assignable to type 'Handler<any, UnknownParams, UnknownParams, UnknownParams, UnknownParams, Document>'.
  Property 'id' is missing in type 'UnknownParams' but required in type '{ id: string; }'.

27     myHandler,
       ~~~~~~~~~~~~~~~~~~~~~~~~~~~

  src/controllers/myHandler.ts:12:7
    12     { id: string }
             ~~
    'id' is declared here.
  node_modules/openapi-backend/backend.d.ts:156:9
    156         [operationId: string]: Handler;
                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    The expected type comes from this index signature.

Found 1 error in src/routes/v1/router.ts:27

what am i doing wrong here?

Handler must have 3 parameters: context, Request,Response. You can send your own parameter after those ones.
f.e. async handler(ctx: Context, req: Request, res: Response, myObject: {id: string}, mySecondObject: ...)

That would be true if i were using express. my project is using koa, which only has 2 arguments going into the handler.

src/routes/v1/router.ts:27:5 - error TS2322: Type 'Handler<unknown, { id: string; }>' is not assignable to type 'Handler<any, UnknownParams, UnknownParams, UnknownParams, UnknownParams, Document>'.
Property 'id' is missing in type 'UnknownParams' but required in type '{ id: string; }'.

The error you are getting indicates that there's some kind of disjunction in typing since your custom { id: string} parameter does not contradict the Params generic expected by the Handler:
type UnknownParams = {
    [key: string]: string | string[];
}

export type Handler<
  RequestBody = any,
  Params = UnknownParams,
  Query = UnknownParams,
  Headers = UnknownParams,
  Cookies = UnknownParams,
  D extends Document = Document,
> = (context: Context<RequestBody, Params, Query, Headers, Cookies, D>, ...args: any[]) => any | Promise<any>;

Looking at the register method,  the current signature suggests that Handler receives no generics at all:
public register(handlers: { [operationId: string]: Handler }): void;

Your custom Handler signature defines an object of a particular shape { id: string }, and while it does fit into the expected generic type { [key: string]: string | string[] }, the opposite is not the case.
It seems to me that we could overcome this issue by making the Handler in the register function aware of each handler's custom generics. The problems is that the registration is performed in bulk and I'm not 100% sure how to dynamically define generics for each handler in this case. On the other hand, doing so individually with the registerHandler method should be more or less straightforward:
registerHandler<
  RequestBody = any,
  Params = UnknownParams,
  Query = UnknownParams,
  Headers = UnknownParams,
  Cookies = UnknownParams,
  D extends Document = Document
>(
  operationId: string,
  handler: Handler<
    RequestBody = any,
    Params = UnknownParams,
    Query = UnknownParams,
    Headers = UnknownParams,
    Cookies = UnknownParams,
    D extends Document = Document
  >
): void;

Circling back to the example provided in the description:
type Body = unknown
type PathParams = { id: string }

const myHandler: Handler<Body, PathParams> = (
  context,
  ctx: KoaContext,
) => {}

api.registerHandler<Body, PathParams >('myHandler', myHandler)

Doing so connects the dots for TypeScript and will prevent type mismatch complaints.
@anttiviljami curios to hear your thoughts on the matter


Debugger variables displays TypeError when debugging async jest test function

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.82.2
OS Version: Windows 11

Steps to Reproduce:

Debug async jest function
VSCode reports the following error in the debug variables when attempting to read a class object

Error processing variables: TypeError: Cannot read properties of undefined (reading 'subtype')	at new hC (c:\Users\Brett\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\ms-vscode.js-debug\src\extension.js:114:11521)	at Up.createVariable (c:\Users\Brett\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\ms-vscode.js-debug\src\extension.js:114:3397)	at Up.createVariableByType (c:\Users\Brett\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\ms-vscode.js-debug\src\extension.js:114:3699)	at Up.createPropertyVar (c:\Users\Brett\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\ms-vscode.js-debug\src\extension.js:114:6444)	at Up.createObjectPropertyVars (c:\Users\Brett\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\ms-vscode.js-debug\src\extension.js:114:5471)	at process.processTicksAndRejections (node:internal/process/task_queues:95:5)	at async Fa.getVariables (c:\Users\Brett\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\ms-vscode.js-debug\src\extension.js:114:16640)	at async ml._onVariables (c:\Users\Brett\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\ms-vscode.js-debug\src\extension.js:159:3583)	at async Bl._onMessage (c:\Users\Brett\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\ms-vscode.js-debug\src\extension.js:71:4859)
In logs 'sharedprocess.log'
2023-09-26 17:44:22.638 [error] [uncaught exception in sharedProcess]: Cannot read properties of undefined (reading 'then'): TypeError: Cannot read properties of undefined (reading 'then')
at v.s (c:\Users\Brett\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\sharedProcess\sharedProcessMain.js:26:4731)
at v.q (c:\Users\Brett\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\sharedProcess\sharedProcessMain.js:26:4161)
at c.value (c:\Users\Brett\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\sharedProcess\sharedProcessMain.js:26:3566)
at b.w (c:\Users\Brett\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\sharedProcess\sharedProcessMain.js:21:1902)
at b.x (c:\Users\Brett\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\sharedProcess\sharedProcessMain.js:21:1972)
at b.fire (c:\Users\Brett\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\sharedProcess\sharedProcessMain.js:21:2188)
at MessagePortMain.te (c:\Users\Brett\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\sharedProcess\sharedProcessMain.js:19:40559)
at MessagePortMain.emit (node:events:513:28)
at MessagePortMain._internalPort.emit (node:electron/js2c/utility_init:2:367)

/jsDebugLogs

Please collect trace logs using the following instructions:

If you're able to, add "trace": true to your launch.json and reproduce the issue. The location of the log file on your disk will be written to the Debug Console. Share that with us.
⚠ This log file will not contain source code, but will contain file paths. You can drop it into https://microsoft.github.io/vscode-pwa-analyzer/index.html to see what it contains. If you'd rather not share the log publicly, you can email it to<EMAIL_ADDRESS>


Register custom cache backend

Register custom cache backend
Currently it is not possible to extend cache backend from third party package. This change open abilities to add custom cache backends using setup-tools.
Implementation
Whole update is really simple - just replace get_entry_map with iter_entry_points. Which will find entry_points in all installed packages.
Usage
This give us ability to add entry_points to any own package like this in setup.py:
setup(
...
entry_points = {
		'pip_accel.cache_backends': [
			'http = wood_pip_accel_backend.http'
		]
	}
)

Hi Libor and thanks for your contribution! A well reasoned and documented pull request that fixes a bug in the way that the code was originally intended to work, that's not something I see frequently! I just merged your changes and released pip-accel 0.38 to GitHub and PyPI.


Adding documentation to the Modal files #36 (#39)

Description
Added Javadoc comments to the Model folder to improve code documentation and provide better context for each method and class. This enhances code readability and helps with future maintenance by offering clear explanations for the controller logic. No related issue was referenced for this change.
Type of Change

[ ] Bug fix
[ ] New/Update feature
[x] New/Update Documentation

Additional Information

More JavaDoc to be added

Approved, looks good like the one before


Adding support to access key presses by privileged programs.

Description
My original goal was to implement vim like bindings to Hyprland, and thought about how to implement those.
However, the style of the config wouldn't really support this. Let's say we want to add a keybinding u (leader could be for example SUPER)  to go  workspaces up.
As the style of the config does not allow for dynamic (programmatic) key binds, even if Hyprland would support vim bindings in the sense of pressing multiple keys after another, you would still need to hard-code each number (in this example).
The obvious way around this would be to allow for example LUA as a second configuration language, but that would probably be a too big change with benefits to too little people (but still be a strong one as you could add dependencies very easily and expand on the functionality).
So my idea would be to just write a program that can read the keybindings from Hyprland in a separate application and use the API to handle the behavior. This could for example be done with an option "exec-once-keyboard-input" which works like exec-once, but writes all keypress behaviours to the stdin of the program.

why not use the plugin system?

https://github.com/Euro20179/HyKeys

https://github.com/Euro20179/HyKeys

that as well but it's unmaintained

why not use the plugin system?

Ye, can try that. I do personally do not know much about the plugin API and what it can try. The reason for the idea with stdin is that, as soon you secure the config files correctly, unprivileged (in a sense that programs I wouldn't want to be able to sniff on my keyboard) couldn't do so in contrast to using sending it to the IPC.

@MightyPlaza lets start with the dumbest idea possible and wrap the api in lua XD


[MSFT:14576675] Stack walker should use _AddressOfReturnAddress to detect script-entry frames.

Currently we use _ReturnAddress to identify a frame that entered the script and to skip walking of non-script frames. But _ReturnAddress is not unique to a given frame, so use _AddressOfReturnAddress instead.

@LouisLaf @meg-gupta This is the issue we discussed.


BVAL-561 Make @OverridesAttribute repeatable and documented

https://hibernate.atlassian.net/projects/BVAL/issues/BVAL-561

Can one of the admins add this person to the trusted builders? (reply with: "add to whitelist" or "ok to test")

ok to test

LGTM

+1 for merging

Merged; thanks, @kasabuta4!


Check for #StPlayground when #GTPlayground does not exist

Not sure if this is the 'optimal' way to fix this but I works for Pharo 11.0 finally :)
Resolve #3

@Dima-369  Thanks very much.


Overlord console - add enable/disable button for remote workers

This PR adds enable/disable button for remote workers listed on overlord console. Feature from #6214

This is a cool feature, but, I don't think we can accept the security risk of enabling CORS by default. How about adding an endpoint on the overlord that proxies this request to the worker?

@gianm thanks for review. You're right, that was not good solution, I have added proxy as you suggested.

Hey @blugowski - hope the review was helpful, let us know if you have any questions about further implementation!

@gianm sorry for delay and thanks for review. I'm going to push changes asap.

@gianm I have pushed changes, waiting for code review.

aw man, this seemed like a cool thing,. @blugowski / @gianm is there an interest in finishing this?

@drcrallen yea, but maybe only in new web console?

This would totally make sense as a part of the new console #6832 if someone wants to revive the API side then adding the UI would be very simple.

@vogievetsky I will add this feature in new UI.


Its a free asura scans no more waiting for six hours

Source name
asurascansfree
Source link
https://asurascansfree.com/
Source language
English
Other details
its new so i dont how how reliable it will be but it has vip chapters on it right now
Acknowledgements

[X] I have checked that the extension does not already exist by searching the extension listing and verified it does not appear there.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[X] I will fill out all of the requested information in this form.

Add a :+1: reaction to issues you find important.

Hope this gets added quickly

If information i got are correct, this site is kinda  take comics behind paywall in scanlation sites like asura and share it for free. So it have that name.


[SPARK-15400][SQL] CreateNamedStruct and CreateNamedStructUnsafe should preserve metadata of value expressions if it is NamedExpression.

What changes were proposed in this pull request?
CreateNamedStruct and CreateNamedStructUnsafe should preserve metadata of value expressions if it is NamedExpression like CreateStruct or CreateStructUnsafe are doing.
How was this patch tested?
Existing tests.

Test build #58857 has started for PR 13193 at commit 035bd20.

Test build #58857 has finished for PR 13193 at commit 035bd20.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58857/
Test PASSed.

cc @sameeragarwal for review

Maybe remove MINOR from title?

@rxin Thank you, I removed [MINOR] from title.

LGTM, is it possible to write a test for it?

+1 on adding a test!

I see, I'll try it.

Test build #58966 has started for PR 13193 at commit c9acff7.

Test build #58966 has finished for PR 13193 at commit c9acff7.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58966/
Test FAILed.

Jenkins, retest this please.

Test build #58996 has started for PR 13193 at commit c9acff7.

Test build #58996 has finished for PR 13193 at commit c9acff7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58996/
Test PASSed.

Merging in master/2.0. Thanks.


AdminUser password validation allows password to be empty

A user can currently set a password to an empty string.

e0958f6d38dadfa0bc774082d82b0bae252e4cf2


Github Page

Set up our webpage.
Troubleshot issues with page theme, added a theme.
Added the text content.
[t:120]


Adding NetCDF Input/Output File Options

Adding namelist controlled NetCDF input/output file options for improved assessment and testing of canopy-app modules.
For the input option, we need example input NetCDF file from Issue #25 first.  Hopefully, @angehung5 can help generate one based on the current input_variables.txt file.
@zmoon
@drnimbusrain

@zmoon   One thing to consider is that with NetCDF IO we will have to compile model with  netcdf-fortran libraries.

@zmoon One thing to consider is that with NetCDF IO we will have to compile model with netcdf-fortran libraries.

Yes there is that. I do think it would be valuable to keep at least maybe a single-column text input option that doesn't require netCDF.

@zmoon Yes, my thought too.
I hope this could be done by compiling with netcdf-fortran as an option (for when user wants to input/output NetCDF) rather than requirement.  I think we should keep the text file option, and add two separate input and ouput options (e.g., in_fmt and out_fmt) to namelist.
I assume we need logic in the Makefile to compile different input (e.g., canopy_intxt_mod.F90 and canopy_inncf_mod.F90) and output submodules (e.g., canopy_outtxt_mod.F90 and canopy_outncf_mod.F90) depending on if user compiling netcdf-fortran is selected as the NetCDF read/write codes will have embedded use netcdf-fortran in them.

Yeah we should be able to do that similar to the current DEBUG option, with conditionals and appending to OBJS. At some point we can move to CMake.

@zmoon @angehung5 Closing this issue.


Error message when using !play or !search

The bot used to run these commands just fine, but just this week my friends and I can't play a song not on autoplaylist.txt, rendering the commands !play and !search useless. The error message reads as follows:
ERROR: Signature extraction failed: Traceback (most recent call last):
File "C:\Users[My name]\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\extractor\youtube.py", line 1081, in _decrypt_signature
video_id, player_url, s
File "C:\Users[My name]\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\extractor\youtube.py", line 969, in _extract_signature_function
raise ExtractorError('Cannot identify player %r' % player_url)
youtube_dl.utils.ExtractorError: Cannot identify player 'https://www.youtube.com/yts/jsbin/player-vfltmLGsd/en_US/base.js'
(caused by ExtractorError("Cannot identify player 'https://www.youtube.com/yts/jsbin/player-vfltmLGsd/en_US/base.js'",))
Any ideas on how to fix this?

Same issue here

I have the same issue, however I receive the following error message:
ERROR: Signature extraction failed: Traceback (most recent call last):
File "C:\Users[My Name]\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\extractor\youtube.py", line 1162, in _decrypt_signature
video_id, player_url, s
File "C:\Users[My Name]\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\extractor\youtube.py", line 1085, in _extract_signature_function
cache_res = res(test_string)
File "C:\Users[My Name]\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\extractor\youtube.py", line 1138, in
return lambda s: initial_function([s])
File "C:\Users[My Name]\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\jsinterp.py", line 254, in resf
res, abort = self.interpret_statement(stmt, local_vars)
File "C:\Users[My Name]\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\jsinterp.py", line 55, in interpret_statement
v = self.interpret_expression(expr, local_vars, allow_recursion)
File "C:\Users[My Name]\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\jsinterp.py", line 211, in interpret_expression
raise ExtractorError('Unsupported JS expression %r' % expr)
youtube_dl.utils.ExtractorError: Unsupported JS expression 'oE"do"'
(caused by ExtractorError('Unsupported JS expression 'oE"do"'',))

same issue for me ERROR: Signature extraction failed: Traceback (most recent call last):
File "C:\Users\Axel\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\extractor\youtube.py", line 1162, in _decrypt_signature
video_id, player_url, s
File "C:\Users\Axel\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\extractor\youtube.py", line 1085, in _extract_signature_function
cache_res = res(test_string)
File "C:\Users\Axel\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\extractor\youtube.py", line 1138, in
return lambda s: initial_function([s])
File "C:\Users\Axel\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\jsinterp.py", line 254, in resf
res, abort = self.interpret_statement(stmt, local_vars)
File "C:\Users\Axel\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\jsinterp.py", line 55, in interpret_statement
v = self.interpret_expression(expr, local_vars, allow_recursion)
File "C:\Users\Axel\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\jsinterp.py", line 211, in interpret_expression
raise ExtractorError('Unsupported JS expression %r' % expr)
youtube_dl.utils.ExtractorError: Unsupported JS expression 'oE"do"'
(caused by ExtractorError('Unsupported JS expression 'oE"do"'',))

Updated dependencies?

Updated dependencies? -Grift

@Grift I ran the update_deps.bat file, but that seemed to only work for about 60 seconds before giving me the same error message again...

So it's a problem with the youtube-dl module. They pushed an update to pypi, but people seem to have trouble getting it. Try completely uninstalling it and reinstalling it:
pip uninstall youtube-dl
and say yes to the prompt, then
pip install youtube-dl

just done the pip upgrade and it says i'm already upto date
and i still get the error

I got the same result as HardUIKnow

ERROR: Signature extraction failed: Traceback (most recent call last):
File "C:\Users\holdu\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\extractor\youtube.py", line 1162, in _decrypt_signature
video_id, player_url, s
File "C:\Users\holdu\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\extractor\youtube.py", line 1085, in _extract_signature_function
cache_res = res(test_string)
File "C:\Users\holdu\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\extractor\youtube.py", line 1138, in
return lambda s: initial_function([s])
File "C:\Users\holdu\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\jsinterp.py", line 254, in resf
res, abort = self.interpret_statement(stmt, local_vars)
File "C:\Users\holdu\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\jsinterp.py", line 55, in interpret_statement
v = self.interpret_expression(expr, local_vars, allow_recursion)
File "C:\Users\holdu\AppData\Local\Programs\Python\Python35\lib\site-packages\youtube_dl\jsinterp.py", line 211, in interpret_expression
raise ExtractorError('Unsupported JS expression %r' % expr)
youtube_dl.utils.ExtractorError: Unsupported JS expression 'oE"do"'
(caused by ExtractorError('Unsupported JS expression 'oE"do"'',))
Even after pip update. tried ramking the bot. nothing

https://github.com/Just-Some-Bots/MusicBot/issues/1141

@arunesh90 What is that good for? I don't see any solutions suggested in the comments of that thread...

@Loxworth I recommend checking the other open issues.


Introduce an inline annotation

Expected behavior:

None. The compiler will decide itself if the function should be inlined. This will depend on the optimization level, the size of the function.
#[inline(always)]. This strongly suggests that the function should be inlined.
#[inline(never)]. This strongly suggests that the function should not be inlined.

I use "strongly suggests" because it is possible that inlining will fail for all kinds of reasons (it is an optimization after all) and when that happens, we shouldn't really error out.
This annotation is probably for advanced users, but I expect it to be quite useful for library authors (especially in the standard library).

This is done: https://github.com/FuelLabs/sway/pull/3185


BANNED FROM THE SERVER!

hey i am SmashCraft32 ,
i am writting bec of me getting banned. PLEASE UNBAN ME! i didnt KNOW it was bad to AFK mine !
PLEASE!!!! give me another chance!!!! PLEASE!!! this is my favourite server! i even payed in it!!! PLEASE!!!!!!
whoops, didnt know this wasnt a place for me to send something like that , ANYWAY PLEASE! help! was banned by rapson! SORRY! DIDNT KNOW!

wow

This is sponge forge a forge mod it isn't a server


Java: Introduce Modulus analysis.

This replaces the parity analysis with a more general modulus analysis.  The basis of the analysis is the same Bound class as is used in the range analysis, so the inferred facts are of the form: e = b + v (mod m) where e is an expression, b is a Bound, and v and m are integers.  The simple case of a constant congruence fact e = v (mod m) is then given by exprModulus(e, any(ZeroBound zb), v, m) .  This allows the range analysis to strengthen bounds further.  As an example, for(int i = 0; i < 3*n; i+=3) is inferred to be equivalent to for(int i = 0; i < 3*n - 2; i+=3).
The gcd predicate is currently implemented only for small numbers, but will be replaced with the built-in predicate once 1.19 is released.
The PR is structured with individually reviewable commits.

Added a commit including test to fix a regression relative to the ParityAnalysis when the array length is a constant.


fix: add window check before addEventListener

In scrollbar-width.js, the code was doing window.addEventListener even when window was undefined.
This broke simplebar in SSR environments.
This commits adds a check before trying to add the resize event.

Looking at the codebase i believe this might not be the only issue in SSR environments.
It used to work fine in older versions though.

Amazing, thank you for your contribution!

Looking at the codebase i believe this might not be the only issue in SSR environments.

I'm not sure what you are refering to but the lib is supporting SSR, this specific case is a regression due to a change in a recent PR. So this fix only should be fine I think.

👏


Where is the step-by-step?

The article states that it will show a high level step-by-step example of how to get authorized credentials. That's the only thing I cannot figure out is how to get the needed "bearer token" to use FCM, to send messages to FCM.
Can anyone tell me how this is done? I'm doing it in .Net so not in Xamarin but the C# could be the same I hope. It is possible to use .Net so where is the explanation that the article says is what's within it?

Thanks for submitting an issue and we apologize for the delayed response. The requested updates haven't been made since the creation of the issue, and the timeline for resolution may vary based on resourcing, so the issue is being moved to an internal repo to better track it.


Remove div on text area

Unsure where this came from / the reason for this. Removing the div breaks stuff on the DS. As the text area requires  styling uplifting this will be investigated but closing for now as there is a lack of detail.


Do we expect BertSum to work on text tokenized using a tokenizer other than Core-NLP?

Is it safe to assume that BERTSUM will perform without issues on input text tokenized using a tokenizer other than Core-NLP as long as we implement sentence splitting?

Hi, if the sentence splitting is correct, it should work fine. And the tokenizer is not that important, since we always use BERT's subtokenizer after the tokenization.

Thank you for confirming!

Hello,
I was wondering if you could provide a sample of what the tokenized data should look like. I am not able to install the Stanford Tokenizer from Java in Google Colab, so I am not not able to check the output files , and format the output using an alternative tokenizer.


Idea: Adopt Common Language Service Protocol

This is just an idea, but wouldn't it speed up tool adoption to adopt the language service protocol that is now used by Visual Studio Code and Eclipse Ché?
There's already a few languages supported and after trying out the Java-Integration vor Visual Studio Code yesterday (see their implementation issue) I have to say the ergonomics from a user perspective are really good.
My assumption is that adopting this protocol could speed up adoption within all those editors and IDE's considerably making Rust a first-class citizen. Of course if this proves useful then this could be the lingua franca for racer directly further reducing the moving parts, dreaming here...

Thanks for the suggestion. Since racerd was built with the express purpose of supporting YCM, and the forthcoming RLS will replace it, this doesn't make sense for the project. Once RLS support lands in YCM, racerd will become a relic.


g.live.com is blocked but needed for OneDrive to work

Hi,
I having some problem's with my OneDrive after a while I figure out that the problem was cause by my pihole blocking "g.live.com"
So I looked up in which list this domain was included.
I can't find it in any of my lists. But if I disabled "https://raw.githubusercontent.com/StevenBlack/hosts/master/hosts" the block is gone.
So is there a entry in the list I don't find which explains the block of the domain or is it a bug in the pihole?

@ts-ws  try whitelisting g.msn.com and see if that fixes the problem. Might work or might not. Let us know 👍

Hi,
yes whitelist g.msn.com have the same affect as whitelist g.live.com OneDrive is working fine after whitelist one of this domains.

Well...g.msn.com is Alias of g.live.com which proofs that Pi-Hole not only blocks the (black)listed domain(s) CNAMES but their Aliases as well which really needs to be addressed with their devs and solution to be find. Removing domains solely because it's a CNAME or Alias of the blocked domain it's unpractical and wrong as the former and later are very likely used to push advertisements and whatnot in many cases.

Did you have a problem with Onedrive website, desktop app, or mobile app? Please be specific because "Onedrive" is not very helpful.

@llacb47 I had problems with the DesktopApp.
In the settings of the DesktopApp under the Tab "Back up" is normally a menu point called "Important PC Folders" the point is completely vanished if the g.msn.com/g.live.com domain is blocked.
Settings if g.msn.com is not blocked

Settings if g.msn.com is blocked

Lars @ts-ws we don't list the g.live.com domain here.
The g.msn.com comes to us via Dan Pollock at someonewhocares.org.  You can make your case with him at<EMAIL_ADDRESS> Dan is very reasonable and he'll ultimately do the Right Thing.

Closing this now.


Not available on PyPI?

Thanks for the fantastic repo - it looks great!
I was just wondering why this isn't available on the Python package index? I think that would be a useful feature so I just wanted to check if there are any plans for this and if not, if I'm okay to fork the repository and publish on PyPI - from the license it seems so but I wanted to double check if first.
Again, thanks for a great repository for such a crucial area.

Hi @abdrysdale!
Thanks for raising this and you're definitely right. We've reviewed this with our internal team and should be able to publish this package to PyPI, so please do hold off on publishing it yourself! 😄

Just logging an update on this @abdrysdale
We put off publishing to PyPI while we did some core development work on the package. Thankfully, that's all ready to go now, so expect version 1.0.0 (and the latest 2.0.0) at some point next week.
Have a good weekend! ✨

Good morning @abdrysdale - I'm happy to say synthgauge is now pip-installable: https://pypi.org/project/synthgauge/
Thanks again for raising this issue and for using the package.

Thanks very much - really appreciate it!