enhance add rewrite rule SplitJoinForNullSkew

What problem does this PR solve?
add transform rule:
 LogicalJoin(type:left join, hashConjuncts:t1.a=t2.a)
    +--Plan1(output:t1.a)
    +--Plan2(output:t2.a)
  ->
  LogicalUnion
    +--LogicalFilter(t1.a is null)
      +--Plan1
    +--LogicalJoin(type:left join, t1.a=t2.a)
      +--LogicalFilter(t1.a is not null)
        +--Plan1
      +--Plan2

only support left join with only 1 hash join conjuncts , and without other join conjuncts.
Since there is sometimes null value skew on the join key, which can lead to prolonged execution times, the join is split into two parts based on whether the join key is null. This can accelerate the query.
Release note
None
Check List (For Author)

Test

[x] Regression test
[x] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

run buildall

run buidall

run buildall

run p0

run buildall

run buildall


Visulaization should be done using vectors

Visualization for node exploration is to be done by drawing vectors from the current node to all possible children and not per pixel.

Pull request #8 brings this update


Source Control: View: Can't switch back to side by side after switching to inline

At first glance, this seems to be fixed in insiders bulid version 1.31.0-insider. Will re-open in case I find out this is not true.
Source Control: View: Can't switch back to side by side after switching to inline

VSCode Version: 1.30.0 (1.30.0)
OS Version: High Sierra 10.13.6

Steps to Reproduce:

Enter Source Control from activity bar
Click file
Top right, click ••• menu
Click "Toggle Side by Side View"
(probably not necessary) Go about your life for a day?
Try to toggle back

Does this issue occur when all extensions are disabled? Yes

Not a duplicate of the bot's suggestion

Possible duplicate of #65016

/duplicate #65016


Fix chrome link

The links are version specific, and I cant find a way to get a fixed URL :/
The prev link was to a version that was removed..

Thank you!


Alternative Sign In

Please provide us with the following information:

It takes quite some steps (including 2FA app) for user to set up Microsoft Entra account. Would email with one time code be feasible? Is there's even any Entra ID article that could be followed? Thanks.
This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [x] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Any log messages given by the failure

Expected/desired behavior

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

azd version?

run azd version and copy paste here.

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

That is possible with Entra External ID. We have that implemented in another simpler repo here:
https://github.com/Azure-Samples/openai-chat-app-entra-auth-builtin
I also presented it during our Build session:
https://build.microsoft.com/en-US/sessions/b5636ca7-64c2-493c-9b30-4a35852acfbe?source=sessions
We should be able to bring it into this repo by bringing in the changes to the auth scripts and the Bicep files. I will work on it.


Little bit of this, little bit of that...

Change log

Remove default styles and scripts (we will implement a CLI for this in the near-future)
Clean up default view markup
Change Title.php to Post.php to suggest a more organized workflow (Supersedes #2320 – content of #2320 would be reorganized to documentation)
Create an App composer to show an example of a global variable
Add align-wide and responsive-embeds to add_theme_support by default
Remove deprecated soil-jquery-cdn from add_theme_support
Clean up docblocks project-wide
Update linting to match the latest PSR-12 standards
Add an appropriate #app wrapper to make the project ready for frameworks like Vue
Add laravel-mix-purgecss for purgeCSS in production
Add laravel-mix-copy-watched for asset watching & versioning
Reduce the footprint of the default webpack.mix.js
Remove unused phpcs.xml
Move sage-directives and sage-svg to suggests
Bump dependencies
Add a nice boi as an author

???

Should we swap add_editor_style() to enqueue_block_editor_assets?
I prefer public/ instead of dist/. thoughts? it looks nice.
I prefer css/ & js/ instead of styles/ & scripts/. thoughts? shorter url, shorter in config, looks nice. thats what actually matters in the end, things just have to look nice. functionality is just an illusion.
I personally think we can scrap eslint & stylelint out of the box and add it back with CLI presets. There's an argument to lint what is in the repo but there is not much there.
readme & changelog
hi austin

I prefer styles and scripts for source files because if the directories are called css and js then I expect to find those file extensions within, where in actuality I might find Sass, TypeScript, etc. Directory names that describe the function, not the form, seem more logical to me.

I prefer styles and scripts
Yeah I'm okay with either, really. I was just throwing it out there again. I understand your reasonings, and there are no arguments against them. I just don't find anything about it personally off-putting and saw it as an attractive change in Laravel. Indeed, an opinionated one.

webpack.mix.js should probably be formatted according to the ESLint rules. Also, ASI is great and all (I used to write my JS that way), but it is considered hazardous by TC39:

As new syntactic features are added to ECMAScript, additional cases requiring explicit semicolons emerge over time. As such, consistently explicit semicolon use is recommended.

See: https://github.com/WebReflection/about-no-semicolons

The mix config isn't linted so I wasn't thinking about it – I'll add them back shortly.


fix:#2769

fix:(#2769 )

yd I've been testing this patch for a long time and it's greatly improved stability of hyprland/workspace module.

Thanks!


Error when running python src/picai_baseline/unet/plan_overview.py

Hi, thanks for the code!
I meet the following error during running python src/picai_baseline/unet/plan_overview.py.
Could you help to figure it out?
Preparing fold 0..
Traceback (most recent call last):
  File "src/picai_baseline/unet/plan_overview.py", line 51, in <module>
    lbl = sitk.GetArrayFromImage(sitk.ReadImage(str(preprocessed_data_path / 'labelsTr' / f'{subject_id}.nii.gz')))
  File "/research/d4/gds/zwang21/anaconda3/envs/surgical/lib/python3.8/site-packages/SimpleITK/extra.py", line 346, in ReadImage
    return reader.Execute()
  File "/research/d4/gds/zwang21/anaconda3/envs/surgical/lib/python3.8/site-packages/SimpleITK/SimpleITK.py", line 8015, in Execute
    return _SimpleITK.ImageFileReader_Execute(self)
RuntimeError: Exception thrown in SimpleITK ImageFileReader_Execute: /tmp/SimpleITK/Code/IO/src/sitkImageReaderBase.cxx:97:
sitk::ERROR: The file "workdir/nnUNet_raw_data/Task2201_picai_baseline/labelsTr/10133_1000135.nii.gz" does not exist.

BTW, I just uncomment the code in src/picai_baseline/prepare_data.py like following:
if mha2nnunet_settings_path.exists():
    print(f"Found mha2nnunet settings at {mha2nnunet_settings_path}, skipping..")
else:
    # generate mha2nnunet conversion plan
    Path(mha2nnunet_settings_path.parent).mkdir(parents=True, exist_ok=True)
    generate_mha2nnunet_settings(
        archive_dir=mha_archive_dir,
        annotations_dir=annotations_dir,
        output_path=mha2nnunet_settings_path,
    )

    # read mha2nnunet_settings
    with open(mha2nnunet_settings_path) as fp:
        mha2nnunet_settings = json.load(fp)

    # note: modify preprocessing settings here
    mha2nnunet_settings["preprocessing"]["matrix_size"] = [20, 256, 256]
    mha2nnunet_settings["preprocessing"]["spacing"] = [3.0, 0.5, 0.5]

    # save mha2nnunet_settings
    with open(mha2nnunet_settings_path, "w") as fp:
        json.dump(mha2nnunet_settings, fp, indent=4)
    print(f"Saved mha2nnunet settings to {mha2nnunet_settings_path}")

Hi @Kyfafyd,
It appears you are missing an annotation (or multiple). Did you run prepare_data.py until completion, and did this run without issues? You can run the same script again to check, it will skip over the steps that are already complete.
Your modification of prepare_data.py to be used for the U-Net baseline appears correct.
Kind regards,
Joeran


Postgresql Flexible Server doesn't support sku "B_Standard_B2ms"

Service team confirmed there is a known bug for B_Standard_B2ms. So for now Postgresql Flexible Server doesn't support this sku. Service team would fix it in next release.

Is there any update on this?

Doesn't support Standard_B1ms either

is this bug Fixed??

Are any skus supported for Postgresql at this time? I have tried:
Standard_E4ds_v5
Standard_E2s_v3
Standard_B2ms
And they all fail (List of skus derived from az postgres flexible-server list-skus -l eastus -o table, eastus is my region).

@neil-yechenwei it is blocking us to deploy our workload any ETA cc @sunilagarwal , @sr-msft, @niklarin.


[draft] Update UnitTest.cpp

THIS PROJECT IS IN DEVELOPMENT MODE. Any pull requests should merge to branch dev, otherwise will be rejected immediately.
Describe your changes
Issue ticket number and link
Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] I have checked readme and add important infos to this PR (if it needed).
[ ] An improved cpp-test/lua-test is not needed (explain why not, thanks).

Conclusion: android, apple use old stdlib for c++ without jthread


JwtBearer and Facebook Authentication

I've got both these setup:
services.AddAuthentication(options =>
                {
                    options.DefaultAuthenticateScheme = JwtBearerDefaults.AuthenticationScheme;
                    options.DefaultChallengeScheme = JwtBearerDefaults.AuthenticationScheme;

                })
                .AddJwtBearer(bearerOptions =>
                {
                    bearerOptions.TokenValidationParameters = TokenParameters(Environment);
                    bearerOptions.Events = new JwtBearerEvents();
                })
                .AddFacebook(facebookOptions =>
                {
                    facebookOptions.AppId = Configuration["Authentication:Facebook:AppId"];
                    facebookOptions.AppSecret = Configuration["Authentication:Facebook:AppSecret"];
                    facebookOptions.Scope.Add("email");
                    facebookOptions.Fields.Add("name");
                    facebookOptions.Fields.Add("email");
                    facebookOptions.Events = new OAuthEvents
                    {
                        OnCreatingTicket = context =>
                        {
                            return Task.CompletedTask;
                        },
                        OnRemoteFailure = context =>
                        {
                            return Task.CompletedTask;
                        }
                    };
                });

I know I have set defaults, but I'm guessing that only intervenes when it can't understand the Authorization header.
So my JavaScript does this:
    facebookLogin() {
        const { history } = this.props;

        window.FB.login(loginResponse => {
            debugger;

            fetch('/api/values', {
                method: 'POST',
                headers: {
                    'Authorization': `Facebook ${loginResponse.authResponse.accessToken}`
                }
            })
                .then(res => {
                    if (res.status === 401) {
                        history.push('/login');
                        return null;
                    }

                    console.log(res.text());
                });
        })
    }

So, it should be pushing the Authorization=Facebook {access_token} to the backend, and I expect it to then use the FacebookAuthenticationHandler to do authentication.
A) I can't understand from your source code where the auth scheme is pulled out of the request.
B) The log output is saying it is trying to do Bearer authentication (which I guess is doing this because I've set the default to be that)
So how do I get them playing nice together with this setup (and where on earth is the auth scheme being pulled out)
debug

Microsoft.AspNetCore.Authentication.JwtBearer.JwtBearerHandler:Information: AuthenticationScheme: Bearer was challenged.

[Authorize] uses the DefaultChallengeScheme when you don't specify a scheme in the attribute.  There's no way for a remote auth scheme like Facebook to sign in via a bearer token, only cookies supports sign in of the built in auth schemes

I'm not trying to sign in via Facebook, I've already done that using Fb.login(), I'm just trying to get the Facebook OAuth Handler to verify that the token it has received is legit.... and then get the creds from the Facebook Api. If that isn't the proper flow - how do you get facebook to accept a callback url that is the Facebook Options Callback?

I don't believe the OAuth handlers are designed to be able to validate tokens that you got yourself in this way, @Tratcher is there any way he can reuse anything to accomplish this?

No, our oauth providers don't work like that. They only support the fully interactive redirect flow. Once you have retrieved the token in javascript you can use it directly to get the user information.

Is there proper documentation on getting this flow to work?!

The JS flow is unrelated to the server, you just use the Facebook JS SDK in client code.

Thats the part I'm having trouble with, I'm using the FacebookJs SDK, but that isn't invoking the oauth callback.

Which callback would that be? That should all be local to the client, e.g. in a pop up or IFrame, it shouldn't be invoking a callback on your server.

It would appear that the external logins only seem to work with MVC, not with a SPA. With MVC there is a flow to do ChallengeAsync where you pass in the provider (a.k.a Facebook) - and I'm guessing from there the provider does all the work with redirecting etc.
So I'm guessing that in order to use the FB i would have to write my own middleware that used the FB access token endpoint to authorise a token sent from the client.
Otherwise I just have a button that invokes a middleware to do a ChanllengeAsync and pass in the provider - which seems a bit disjointed to me.

Why a middleware vs a Controller Action?

Because I'm working in the SPA world, so my ASPNET app is just a REST service. Secondly I don't like using the attributes, I prefer the middleware approach where the path of my requests and responses are very prescribed. I don't see why this offering can't be in middleware and why it has only been build for the MVC world, but there you go

Note that the access_token from facebook does not contain anything that you can validate on your server. Your only way to validate it is to send it to a facebook REST API and see if it succeeds. For instance you could send it to the user information endpoint. This is like skipping to the last step in the existing facebook handler.
https://github.com/aspnet/Security/blob/54e4f6834e6fa0f1cf2cc8c2599674ffe017c890/src/Microsoft.AspNetCore.Authentication.Facebook/FacebookHandler.cs#L27-L46


Added built in collection fixtures

refs https://github.com/TryGhost/Arch/issues/25

The instance should have two built-in collections "index" and "featured". These have been filled through in-memory tricks before, now they should come pre-populated through fixtures mechanism.

🤖 Generated by Copilot at cfdde0a
This pull request adds two built-in collections, index and featured, to the Ghost database schema and data. It uses a migration script and a fixtures file to create the collections and removes the redundant code from the collections service. The built-in collections allow users to easily access all posts or only featured posts.

There are flaky tests coming from ghost/core/test/e2e-api/admin/collections.test.js - they fail 20% of a time when run on MySQL locally. Possible reason for the failure is a race condition in the event processing inside of the collections service.

about the use of an empty filter for the "latest" collection 🤔 What do you think about using a filter of -id:null id:-null or something like that, which would include all of the posts, but it allows our validation to be as simple as "All automatic collections have a filter"

@allouis I feel like this is one of those that seems like a good idea but would add future questions like "why is this thing even here?" from anybody reading through the codebase. Pros: we would save an extra if statement(s). Cons: add a confusing statement, which would need extra explanation, which would not be visible in the DB schema but rather somewhere in comments inside of the collections service.
I'd leave it as is. Unless there are more good reasons to go down this path.


Remove personal admin emails

We should use a mailing list on Google Apps instead of listing personal emails of admins in the environment variable.

I think @petschekr mentioned this - we would lose any potential audit trail if we combined all admins into one.


Feature request: Generalize strip_option

Generalize strip_option to a generic attribute accepting a method and an argument list, which the setter method would proxy to.
Example
To build a struct with a Cell, we do not want to require the user to construct the Cell themselves.
#[derive(TypedBuilder)]
struct Foo {
    #[builder(transform = Cell::new(bar: u32))]
    bar: Cell<u32>,
}

The new attribute here is (in decl macro notation) transform = $path:path( $($name:pat : $ty:ty),* ).
A builder method is generated to require the specified parameters $name* and pass them to the function in $path, whose return value is used to set the field.
An equivalent notation for strip_option is transform = ::std::option::Option::Some(field: Type).
Alternatives
Use a trait to standardize this to avoid attribute parsing Example:
trait BuilderParam<U> : Sized {
    fn convert(self) -> U;
}

Then applying #[builder(transform)] on field: Type would generate
fn field(mut self, field: impl BuilderDelegate<Type>) -> Builder<...> {
    Builder {
        field: field.convert(),
        ...self
    }
}

I can write a patch for this if you find this appropriate.

Syntax-wise, I'd rather have something like this:
#[derive(TypedBuilder)]
struct Foo {
    #[builder(transform = |bar: u32| Cell::new(bar))]
    bar: Cell<u32>,
}

It'll be easier to parse because it's legal Rust syntax, it'll be more straightforward to determine the argument, and it'll allow having multiple arguments:
struct Point {
    x: f64,
    y: f64,
}

#[derive(TypedBuilder)]
struct Foo {
    #[builder(transform = |x: f64, y: f64| Point { x, y })]
    point: Point,
}

I'm also wondering whether or not this should be under setter (#[builder(setter(transform = ...))])


Use Rosetta2 as a binfmt_misc handler on macOS Ventura

I just learned that on macOS Ventura you can register Rosetta2 as a binfmt_misc handler inside a VM running in arm64 mode. That should mean that instead of using qemu-static emulation we should be able to run amd64 containers "compiled" into arm64, which should provide a big speed boost for code that is not available natively for arm64 (like mssql):
https://developer.apple.com/documentation/virtualization/running_intel_binaries_in_linux_vms_with_rosetta
Originally posted by @jandubois in https://github.com/lima-vm/lima/issues/889#issuecomment-1153010671

This needs implementing Virtualization.framework support (#889) right?
Or is it technically and legally possible to run Rosetta2 inside QEMU?
What is the license of the Rosetta2 binfmt_misc?

This needs implementing Virtualization.framework support (#889) right?

I don't know; it depends on if we can get VirtioFS support without it.
Or if it will be possible to somehow bridge between the VirtioFS volume and qemu in some other way. I suspect not, as event notification will be a challenge, but it doesn't hurt to try...
I suspect that the binfmt_misc handler is just copying stuff to the Rosetta volume; Rosetta will perform the translation on the host and put the results back into the volume, so binfmt_misc can execute it from there.

What is the license of the Rosetta2 binfmt_misc?

I didn't think about it, but we would not distribute the software; it would come directly from Apple. So how would calling this from inside qemu be any different from running any other part of the OS from inside any kind of application. Since we don't link it to qemu, but just call it from a kernel hook, I don't see any problems there either.

Just a side note, Rosetta works great in UTM, when I launch VMs using Virtualization.framework.
I think theoretically, Rosetta should work in QEMU, but it's intended to be used with Virtualization.framework. The Rosetta binary is provided in a VirtioFS volume.
See UTM docs on Rosetta


Setup -- Display FSMO , Site of Exchange server

Is your request related to a problem? Please describe.
New request
Describe The Request
Display FSMO information and alert if the Exchange box is of different site
Netdom Query FSMO
Nltest /dsgetsite for site information or Get-Exchangeserver |ft name,site or Check DynamicSiteName from registry
HKLM\System\CurrentControlSet\Services\Netlogon\Parameters
If the site is hardcoded for any reason then SiteName key
Additional context
Add any other context or screenshots about the feature request here.

Right now the way this works, we will display this information if we aren't on the latest build of Exchange.
Change: 056b92a0ff4116699fd74e14bda72a99fdaabc8d
Output Example:

Being that this is already checked in and this information is only required useful when we need to do /prepareAD, this issue can be closed.


the get disk path name logic is not robust enough.

from the logic, we make assumption the lun is sequence, and the os will
assign the sdb/c/d/e/f.... to lun i,i+1,i+2,
but I think it's not robust.
we should think about to use the scsi host path:
ls /dev/disk/by-path/
acpi-VMBUS:01-scsi-0:0:0:0        acpi-VMBUS:01-scsi-0:0:0:0-part2  acpi-VMBUS:01-scsi-0:0:0:3  acpi-VMBUS:01-scsi-0:0:1:0-part1
acpi-VMBUS:01-scsi-0:0:0:0-part1  acpi-VMBUS:01-scsi-0:0:0:2        acpi-VMBUS:01-scsi-0:0:1:0

say this case:
"options":{"endpoint":"http://<IP_ADDRESS>:25250","password":"d0xq7urnjbl4s7w7hawm","user":"agent"}},"disks":{"system":"/dev/sda","ephemeral":{"host_device_id":"{f8b3781b-1e82-4818-a1c3-63d806ec15bb}","lun":"0","path":"/dev/sdc"},"persistent":{"caching:None;disk_name:bosh-disk-data-769fc7be-61c5-4c95-a548-14bcb798e25b;resource_group_name:andliu-cf-vmss-3-rg":{"host_device_id":"{f8b3781b-1e82-4818-a1c3-63d806ec15bb}","lun":3,"path":"/dev/sdf"}}
sda      8:0    0     3G  0 disk
-sda1   8:1    0     3G  0 part / sdb      8:16   0    50G  0 disk  -sdb1   8:17   0    50G  0 part
sdc      8:32   0    10G  0 disk
|-sdc1   8:33   0   3.4G  0 part [SWAP]
`-sdc2   8:34   0   6.7G  0 part /var/vcap/data
sdd      8:48   0    24M  0 disk
sde      8:64   0     5G  0 disk
but actually the lun:3 is sde.

suggest to remove this logic, because it's not useful in the new stemcell.


Is kube-apiserver's "--runtime-config" should be used in webhook mode?

This is a...

[ ] Feature Request
[x] Bug Report

Problem:
I want to open apiserver's webhook mode,and the website docs says I should set --runtime-config=authorization.k8s.io/v1beta1=true.
But when I run kube-apiserver --help | grep runtime,It seems that this config will remove in the future:
$ kube-apiserver --help | grep runtime
      --runtime-config mapStringString   A set of key=value pairs that describe runtime configuration that may be passed to apiserver. <group>/<version> (or <version> for the core group) key can be used to turn on/off specific api versions. api/all is special key to control all api versions, be careful setting it false, unless you know what you do. api/legacy is deprecated, we will remove it in the future, so stop using it.

So,Should I use this or not?
And which runtime-config is opened default?
Proposed Solution:
Page to Update:
https://kubernetes.io/docs/reference/access-authn-authz/webhook/

v1.12

centos 7.5

just api/legacy  group will be removed not  runtime-config

just api/legacy  group will be removed not runtime-config
（⊙ｏ⊙）哦

@Pingan2017 （⊙ｏ⊙）哦

可以用curl {master ip}:port 来查看哪些api是打开的

可以用curl {master ip}:port 来查看哪些api是打开的

哦哦，谢谢大佬，原来这个玩意儿就是启用或关闭api资源组的

/close


terminal screen cheese

I have an extension that evaluates some selected text in node in the terminal
As soon as I select the terminal (click) its content scroll up and I cannot get it back

    let terminal: vscode.Terminal;
    vscode.commands.registerCommand('extension.runInNode', () => {
        let {selection, document} = vscode.window.activeTextEditor;
        let text = document.getText(selection);

        if (!terminal) {
            terminal = vscode.window.createTerminal({ name: 'nodejs - scratch page'});
        }

        terminal.sendText('node');
        terminal.sendText(text);

    });

		"menus": {
			"editor/context": [
				{
					"command": "extension.runInNode",
					"when": "editorHasSelection"
				}
			]
		}

Cannot repro in Ubuntu 16.04 on bash, what platform/shell are you on?

Difficulty reproducing so moving to September. If this is a bug I is very likely a low-repro bug that's present in the terminal, not an issue with the API.

Closing this off as it has not been reported for 2 weeks and can't repro. It's also very likely an upstream issue.


Path handling is different and tests fail in Windows

Summary
The way lookups are done through the jsonapi (queryHost) behaves differently on Windows than on Linux.
Accordingly, a number of the jsonapi tests fail on Windows.
Unfortunately, I don't have time to dig into this. FWIW, the only manifestation I can see is that the browser extension fails to find matches in some cases.
For example, when the browser is on account.ubnt.com:

in Windows: no entries
in Linux: colemickens/ubnt.com is returned

Also,
--- FAIL: TestRespondMessageQuery (1.95s)
    api_test.go:305:
        	Error Trace:	api_test.go:305
        	            				api_test.go:246
        	            				api_test.go:242
        	            				api_test.go:75
        	Error:      	Expect "["awesomePrefix\\foo\\bar"]" to match "\["awesomePrefix/foo/bar"\]"
        	Test:       	TestRespondMessageQuery
    api_test.go:305:
        	Error Trace:	api_test.go:305
        	            				api_test.go:246
        	            				api_test.go:242
        	            				api_test.go:81
        	Error:      	Expect "["awesomePrefix\\fixed\\yamllogin","awesomePrefix\\fixed\\yamlother"]" to match "\["awesomePrefix/fixed/yamllogin","awesomePrefix/fixed/yamlother"\]"
        	Test:       	TestRespondMessageQuery
    api_test.go:305:
        	Error Trace:	api_test.go:305
        	            				api_test.go:246
        	            				api_test.go:242
        	            				api_test.go:87
        	Error:      	Expect "[]" to match "\["awesomePrefix/b/some.other.host","awesomePrefix/some.other.host/other"\]"
        	Test:       	TestRespondMessageQuery
    api_test.go:305:
        	Error Trace:	api_test.go:305
        	            				api_test.go:246
        	            				api_test.go:242
        	            				api_test.go:93
        	Error:      	Expect "[]" to match "\["awesomePrefix/other.host/other"\]"
        	Test:       	TestRespondMessageQuery
    api_test.go:305:
        	Error Trace:	api_test.go:305
        	            				api_test.go:246
        	            				api_test.go:242
        	            				api_test.go:105
        	Error:      	Expect "[]" to match "\["somename/github.com"\]"
        	Test:       	TestRespondMessageQuery
    api_test.go:305:
        	Error Trace:	api_test.go:305
        	            				api_test.go:246
        	            				api_test.go:242
        	            				api_test.go:111
        	Error:      	Expect "[]" to match "\["somename/ubnt.com"\]"
        	Test:       	TestRespondMessageQuery
FAIL
FAIL	github.com/gopasspw/gopass/pkg/jsonapi	4.326s

Steps To Reproduce
Try to run jsonapi tests on Windows.
Expected behavior
They pass.
Environment

OS: Windows 10
OS version: Version 1809 OS Build 18277.1000
gopass Version: HEAD dec28c39...
Installation method: source

Actually, this fixes the issue: https://github.com/gopasspw/gopass/pull/935

@colemickens Could you try to run all the tests (make travis) with this fix applied on windows and report back? That would help a lot.

There's some reasons why I hadn't done this. I can't run make in windows, the tests seem to rely on more OS-specific stuff, the tests seem to need GOPASS_BINARY set at times, I can't remember how to get go to give me a summary of test results and I'm going cross-eyed from all the output go test spewed at me...
Also there seem to be packages whose tests that don't even compile? Some of them panic too. I'm not sure what the state of the tests are in Linux, but they're not in good shape on Windows.
I did try just give this a shot for the last ~40 minutes... but due to what I perceive as deficits in the testing infrastructure, I can't really even venture a good guess as to whether the tests are overall better or overall worse with #953 applied. As an example, #953 makes good sense and fixes real world usage of gopass/gopassbridge, but it doesn't even fix the jsonapi tests due to how they work (string-wise path comparisons, it initializes a full new pass store on every case in every test). In general the tests are in a bad state in Windows. Some don't compile, some panic, etc.
Unfortunately, I'm not sure I can offer much more confidence in #953 than my original anecdotal comments, at least right now.

I just ran go test (go list .\pkg\...) on a Windows computer, and the log file is about 932 lines.
There are so many tests failing that it's complicated to see what's going on.
As a fun note, when using the patch proposed in #935 these tests fail:
?   	github.com/gopasspw/gopass/pkg/tree	[no test files]
=== RUN   TestFolder
--- FAIL: TestFolder (0.00s)
	folder_test.go:41:
			Error Trace:	folder_test.go:41
			Error:      	Not equal:
			            	expected: "gopass\nÔööÔöÇÔöÇ foo (template)\n    Ôö£ÔöÇÔöÇ bar\n    Ôö£ÔöÇÔöÇ baz.b64 (binary)\n    ÔööÔöÇÔöÇ zab.yml (yaml)\n"
			            	actual  : "gopass\nÔö£ÔöÇÔöÇ foo/bar\nÔö£ÔöÇÔöÇ foo/baz.b64 (binary)\nÔööÔöÇÔöÇ foo/zab.yml (yaml)\n"

			            	Diff:
			            	--- Expected
			            	+++ Actual
			            	@@ -1,6 +1,5 @@
			            	 gopass
			            	-ÔööÔöÇÔöÇ foo (template)
			            	-    Ôö£ÔöÇÔöÇ bar
			            	-    Ôö£ÔöÇÔöÇ baz.b64 (binary)
			            	-    ÔööÔöÇÔöÇ zab.yml (yaml)
			            	+Ôö£ÔöÇÔöÇ foo/bar
			            	+Ôö£ÔöÇÔöÇ foo/baz.b64 (binary)
			            	+ÔööÔöÇÔöÇ foo/zab.yml (yaml)

			Test:       	TestFolder
	folder_test.go:55:
			Error Trace:	folder_test.go:55
			Error:      	Not equal:
			            	expected: []string{"zab/foozen"}
			            	actual  : []string{"zab/foo/bar", "zab/foozen", "zab2/foo/baz", "zab2/foo/zen/baz"}

			            	Diff:
			            	--- Expected
			            	+++ Actual
			            	@@ -1,3 +1,6 @@
			            	-([]string) (len=1) {
			            	- (string) (len=10) "zab/foozen"
			            	+([]string) (len=4) {
			            	+ (string) (len=11) "zab/foo/bar",
			            	+ (string) (len=10) "zab/foozen",
			            	+ (string) (len=12) "zab2/foo/baz",
			            	+ (string) (len=16) "zab2/foo/zen/baz"
			            	 }
			Test:       	TestFolder
	folder_test.go:66:
			Error Trace:	folder_test.go:66
			Error:      	Not equal:
			            	expected: []string{"zab", "zab/foo", "zab2", "zab2/foo", "zab2/foo/zen"}
			            	actual  : []string{}

			            	Diff:
			            	--- Expected
			            	+++ Actual
			            	@@ -1,7 +1,2 @@
			            	-([]string) (len=5) {
			            	- (string) (len=3) "zab",
			            	- (string) (len=7) "zab/foo",
			            	- (string) (len=4) "zab2",
			            	- (string) (len=8) "zab2/foo",
			            	- (string) (len=12) "zab2/foo/zen"
			            	+([]string) {
			            	 }
			Test:       	TestFolder
	folder_test.go:76:
			Error Trace:	folder_test.go:76
			Error:      	Not equal:
			            	expected: []string{"zab", "zab/foo", "zab2", "zab2/foo"}
			            	actual  : []string{}

			            	Diff:
			            	--- Expected
			            	+++ Actual
			            	@@ -1,6 +1,2 @@
			            	-([]string) (len=4) {
			            	- (string) (len=3) "zab",
			            	- (string) (len=7) "zab/foo",
			            	- (string) (len=4) "zab2",
			            	- (string) (len=8) "zab2/foo"
			            	+([]string) {
			            	 }
			Test:       	TestFolder
=== RUN   TestFormat
--- FAIL: TestFormat (0.00s)
	tree_test.go:78: Format mismatch:
		---
		gopass
		Ôö£ÔöÇÔöÇ a
		Ôöé   Ôö£ÔöÇÔöÇ b
		Ôöé   Ôöé   ÔööÔöÇÔöÇ c
		Ôöé   Ôöé       Ôö£ÔöÇÔöÇ d
		Ôöé   Ôöé       ÔööÔöÇÔöÇ e
		Ôöé   Ôö£ÔöÇÔöÇ g
		Ôöé   Ôöé   ÔööÔöÇÔöÇ h
		Ôöé   ÔööÔöÇÔöÇ f
		ÔööÔöÇÔöÇ foo (E:\tmp\foo)
		    Ôö£ÔöÇÔöÇ bar (E:\tmp\bar)
		    Ôöé   ÔööÔöÇÔöÇ baz
		    ÔööÔöÇÔöÇ baz
		        ÔööÔöÇÔöÇ inga
		---
		gopass
		Ôö£ÔöÇÔöÇ foo (E:\tmp\foo)
		Ôö£ÔöÇÔöÇ foo/bar (E:\tmp\bar)
		Ôö£ÔöÇÔöÇ a/b/c/d
		Ôö£ÔöÇÔöÇ a/b/c/e
		Ôö£ÔöÇÔöÇ a/f
		Ôö£ÔöÇÔöÇ a/g/h
		Ôö£ÔöÇÔöÇ foo/bar/baz
		ÔööÔöÇÔöÇ foo/baz/inga
		---
=== RUN   TestFormatSubtree
--- FAIL: TestFormatSubtree (0.00s)
	tree_test.go:94:
			Error Trace:	tree_test.go:94
			Error:      	Received unexpected error:
			            	Entry not found
			            	github.com/gopasspw/gopass/pkg/tree/simple.(*Folder).FindFolder
			            		E:/golang/src/github.com/gopasspw/gopass/pkg/tree/simple/folder.go:206
			            	github.com/gopasspw/gopass/pkg/tree/simple.TestFormatSubtree
			            		E:/golang/src/github.com/gopasspw/gopass/pkg/tree/simple/tree_test.go:93
			            	testing.tRunner
			            		C:/Go/src/testing/testing.go:777
			            	runtime.goexit
			            		C:/Go/src/runtime/asm_amd64.s:2361
			Test:       	TestFormatSubtree
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
	panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x38 pc=0x6435a6]

goroutine 7 [running]:
testing.tRunner.func1(0xc0420fa2d0)
	C:/Go/src/testing/testing.go:742 +0x2a4
panic(0x688c40, 0x8520c0)
	C:/Go/src/runtime/panic.go:505 +0x237
github.com/gopasspw/gopass/pkg/tree/simple.TestFormatSubtree(0xc0420fa2d0)
	E:/golang/src/github.com/gopasspw/gopass/pkg/tree/simple/tree_test.go:96 +0x266
testing.tRunner(0xc0420fa2d0, 0x6f1620)
	C:/Go/src/testing/testing.go:777 +0xd7
created by testing.(*T).Run
	C:/Go/src/testing/testing.go:824 +0x2e7
FAIL	github.com/gopasspw/gopass/pkg/tree/simple	0.060s

while using the hardcoded unix separator make them pass, but that's probably because the tests are meant to pass on unix:
=== RUN   TestFolder
--- PASS: TestFolder (0.00s)
=== RUN   TestFormat
--- PASS: TestFormat (0.00s)
=== RUN   TestFormatSubtree
--- PASS: TestFormatSubtree (0.00s)
=== RUN   TestGetNonExistingSubtree
--- PASS: TestGetNonExistingSubtree (0.00s)
PASS
ok      github.com/gopasspw/gopass/pkg/tree/simple```


Ability to access app instance via static method

This is useful when requiring services from app's container in middlewares.

Actually I :-1: this PR.

Static is evil. Also, this is a microframework, I don't think such registry bundled into the core makes sense.
This PR implements multiple features: the static registry and replaces HTTP method usage with constants. They should be in separate PRs.
If you call setName, the Application object still remains in the registry indexed by the old name, but internally holding the new name, as it is the same reference. It is a super-inconsistent state.
Separation of concerns: the application to be it's own factory??

I think there are better ways to handle this and they'll be addressed before a release.
At a cautionary glance, the middlewares could quite easily be resolved from the container to allow for DI proper within those classes.
@sagikazarmark as for it being a microframework, I think it started that way but it is now aimed at being more of a bootstrap for any sized application. Would you agree @alexbilbie ?

@philipobenito even if it's true (what I could easily accept) the rest of the issues are still there.
I cannor even see why this would make sense at all. And even if it does, there are better ways (as you also said).

I should point out my comments above relate to the new repository.

New repository?

https://github.com/protonphp/proton

Completely missed that.

So my bad on commenting here, was unaware I was watching the repo.
However, my point stands, I don't think this is the solution to the problem at hand.

We have bigger plans for the project there once we both have time to have a proper chat about it.

:+1:

@philipobenito Any hope on that "bigger chat"? I, and I am sure others, are anxiously awaiting a v2 of Proton.


Is to_opengl a must for sampling env map?

https://github.com/fudan-zvg/PVG/blob/3ea4b176b03e901800df9e95edda00ec66b3a795/scene/envlight.py#L9
Hi! The ray direction is already in the world coordinate, and I am confused about why we need to do this conversion. Could you explain it?
Looking forward to your help and reply

Hi, I've noticed that nvdiffrast uses the y-axis upward, while we use the z-axis, just to align with nvdiffrast. I think it's also okay without this conversion.

Thanks!


Hip support with variable rate compression

This PR contains the HIP version of variable rate compression.
The follow command was used to configure with CMake:
cmake .. \
-DZFP_WITH_HIP=ON \
-DZFP_WITH_OPENMP=OFF \
-DCMAKE_C_COMPILER=hipcc \
-DCMAKE_CXX_COMPILER=hipcc

LGTM


feat(share/shrex/nd): add swamp test for shrex nd

Overview
Feel free to suggest more test cases if you have any ideas
Closes https://github.com/celestiaorg/celestia-node/issues/2119

Codecov Report

Merging #2227 (2b8689d) into main (e8ffd64) will increase coverage by 0.21%.
The diff coverage is 67.74%.

@@            Coverage Diff             @@
##             main    #2227      +/-   ##
==========================================
+ Coverage   55.81%   56.02%   +0.21%
==========================================
  Files         216      216
  Lines       14130    14149      +19
==========================================
+ Hits         7886     7927      +41
+ Misses       5454     5434      -20
+ Partials      790      788       -2

Impacted Files
Coverage Δ

share/availability/discovery/discovery.go
77.59% <0.00%> (+1.34%)
:arrow_up:

share/availability/discovery/metrics.go
47.05% <57.14%> (+0.63%)
:arrow_up:

share/p2p/shrexnd/server.go
66.21% <75.00%> (-1.14%)
:arrow_down:

nodebuilder/share/config.go
53.12% <100.00%> (+1.51%)
:arrow_up:

nodebuilder/share/constructors.go
94.33% <100.00%> (+0.58%)
:arrow_up:

share/availability/discovery/backoff.go
86.79% <100.00%> (+1.07%)
:arrow_up:

... and 4 files with indirect coverage changes

Fantastic PR @walldiss 🚀
I believe you can close this issue https://github.com/celestiaorg/celestia-node/issues/2119 with this PR

During a random run of TestShrexNDFromLightsWithBadFulls, got this fatal:
2023-07-31T14:43:18.569+0200	DEBUG	share/getters	getters/cascade.go:115	cascade: launching getter #0
2023-07-31T14:43:18.570+0200	ERROR	ipld	ipld/namespace_data.go:247	could not retrieve IPLD node	{"namespace": "00000000000000000000000000000000000000186d38b0311dc2a45a12", "pos": 0, "err": "nmt node not found"}
2023-07-31T14:43:18.570+0200	ERROR	ipld	ipld/namespace_data.go:247	could not retrieve IPLD node	{"namespace": "00000000000000000000000000000000000000186d38b0311dc2a45a12", "pos": 0, "err": "nmt node not found"}
2023-07-31T14:43:18.570+0200	ERROR	ipld	ipld/namespace_data.go:247	could not retrieve IPLD node	{"namespace": "00000000000000000000000000000000000000186d38b0311dc2a45a12", "pos": 1, "err": "nmt node not found"}
2023-07-31T14:43:18.570+0200	ERROR	ipld	ipld/namespace_data.go:247	could not retrieve IPLD node	{"namespace": "00000000000000000000000000000000000000186d38b0311dc2a45a12", "pos": 0, "err": "nmt node not found"}
2023-07-31T14:43:18.570+0200	FATAL	ipld	ipld/namespace_data.go:305	target namespace outside of boundaries of links at depth: 3


HDDS-11306. OM support system audit log

What changes were proposed in this pull request?
system audit log framework is added for OM. This is added with having log for startup of OM. This can be used further to added logs for below operation:

internal request like open file cleanup, purge directory, ...
other operation critical for debugging

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-11306
How was this patch tested?

verified logs over docker and below is the result,

Thanks for adding this @sumitagrawl this will be great to have. Are we planning on adding other messages like leader change and open key cleanup in this PR or followup tasks?

Thanks for adding this @sumitagrawl this will be great to have. Are we planning on adding other messages like leader change and open key cleanup in this PR or followup tasks?

Will be doing in followup PR

Thanks @errose28 for review. Thanks @sumitagrawl for the patch.


Vault showing as empty

Vault space is beiong shown as empty despite being full i have reloaded all accounts and tried everything i can think if

I also have this blank vault issue. Paging @jakcodex for insight on this bug (assuming they have it).

I was having this issue yesterday as well, but as of today it seems to have fixed itself?

@TheScootz Glad to know it works now. I'll check later today just to make sure.
@RyanLMc, here's an explanation on reddit (that is in turn copied from the Discord) on why Muledump is/was not working.

Yikes this is a late reply. To summarize, this was a bug on Deca's side from a change they had made and subsequently fixed.


Suggestion: Improve IntelliSense responsiveness

I've noticed that compared to other editors, IntelliSense can be quite slow to respond. I've added some timing to various parts of ProvideCompletionsAsync and ProvideCompletionsAsyncAux. It looks like for large files, the vast majority of the time is spent in ParseAndCheckDocument and GetDeclarationListInfo. It may be useful to add some metrics to see just how long is actually spent in ParseAndCheckDocument as opposed to queuing waiting to be serviced by the reactor.
[F#][INFO 12:17:21 PM]  ********** COMPLETION **********
[F#][INFO 12:17:21 PM] GetTextAsync: 0 msec
[F#][INFO 12:17:21 PM] GetCompilationDefinesForEditingDocument: 0 msec
[F#][INFO 12:17:21 PM] shouldProvideCompletion: 0 msec
[F#][INFO 12:17:21 PM] TryGetOptionsForEditingDocumentOrProject: 0 msec
[F#][INFO 12:17:21 PM] GetTextVersionAsync: 0 msec
[F#][INFO 12:17:23 PM] ParseAndCheckDocument: 1829 msec
[F#][INFO 12:17:23 PM] ParseAndCheckDocument: 102 msec
[F#][INFO 12:17:23 PM] GetLinePosition etc: 0 msec
[F#][INFO 12:17:23 PM] GetPartialLongNameEx: 0 msec
[F#][INFO 12:17:24 PM] GetDeclarationListInfo: 717 msec
[F#][INFO 12:17:24 PM] More within ProvideCompletionsAsyncAux: 0 msec
[F#][INFO 12:17:24 PM] ProvideCompletionsAsyncAux: 830 msec
[F#][INFO 12:17:24 PM]  ********************************

Anyway I hope this can point some people more knowledgeable than me in the right direction to start making some optimisations :) As always I'm happy to help

#2422
the two topics are opened before,i read autocompletionprovider code,it's a basic now.
lonide use pattern match,c# use dozens of provider type  to help this.
i do find the same bug #2962 ,#2822

In short, we always try to get fresh file check results and wait for them for 1 second, then, if there are no results, we use cached ones. This is the behavior @dsyme proposed to avoid glitches in completion list on small files. In practice I find it annoying and I suggest to add an option to always use cached results, and make it on by default. This logic is here https://github.com/Microsoft/visualfsharp/blob/master/vsintegration/src/FSharp.Editor/LanguageService/FSharpCheckerExtensions.fs#L33 Maybe you can find bugs in it as well.

@saul As an experiment, try to remove waiting logic and always use cached results to see if it solves you issues with "slow intellisense".

Oh, it's 2 seconds, not one...

@vasily-kirichenko I think it's improved... the problem is the time it takes for IntelliSense varies so wildly it's hard to say for certain.
I agree that we should have an option page checkbox "Allow stale completion results (faster but less accurate)"

@saul I think that's a good idea, yes.

Hi folks, I have tested a variety of code completion delays on a number of users. A rough guideline of the results is as follows:
< 300ms -> Types are the best thing ever
300ms - 1000ms -> Types are OK
1000ms - 2000ms -> I'm vaguely aware that my life is wasting away one second at a time (general sadness inside)

2000ms -> I need to consider a career change / I might as well use Python (It takes me less time to google it)

In other words; if anything takes > 2 seconds you might as well not bother.

I wasn't sure if this was just because of the size of the FSharp.Core solution (that's the only solution I dare use VS2017 on) but if this is par for the course even on smaller solutions... not great.

@isaacabraham I chatted with @moloneymb and he's working in a 20,000 line file.  So size is an issue here

@dsyme People can work with such big files? Interesting. Do we have somewhere a list containing ideas where performance can be improved in code?

There's been a lot of performance work in the past year (some of it specific to completion lists), so it may be worth rebaselining performance here. The April 30th, 2017 experience (VS 15.0!) is very very different to the April 10th, 2019 experience.

I think we can close this out. Sooooo many issues related to IDE perf and specific to generating a completion list have been resolved since this was filed, and things are generally in a pretty good place now.


index html

clear data class and cookies

[x] Have you followed the contributing guidelines?
[x] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

close all pull


LINQ Include Query with .Take(int) results in pulling all results from the database

Steps to reproduce
Models:
public class Blog
{
    public int BlogId { get; set; }
    public string Title { get; set; }

    public List<Post> Posts { get; set; }
}

public class Post
{
    public int PostId { get; set; }
    public string Title { get; set; }
    public string Content { get; set; }

    public int BlogId { get; set; }
    public Blog Blog { get; set; }
}

The issue
When I query these 2 tables using the following LINQ Expression:
db.Blogs.OrderByDescending(m => m.BlogId).Take(10).Include(m => m.Posts).ToList();

Produce the following SQL that selects All Posts from the table.
SELECT `p`.`PostId`, `p`.`BlogId`, `p`.`Content`, `p`.`Title`
FROM `Posts` AS `p`
WHERE EXISTS (
    SELECT 1
    FROM `Blogs` AS `m`
    WHERE `p`.`BlogId` = `m`.`BlogId`
    LIMIT 10)
ORDER BY `p`.`BlogId` DESC

Incorrect # of results (should be 20 since I have 2 posts per blog in my test case)
Showing rows 0 - 24 (26280 total, Query took 0.0003 seconds.)

As you can imagine, this is quite an issue when the Posts table has 1000's of posts.
Further technical details
EF Core version: 1.0.1
Operating system: Ubuntu 16.04
I am using the Pomelo.EntityFramework.MySql driver (I am a contributor to the project).
The joining behavior in question comes from the IncludeExpressionVisitor.cs

Why doesn't the 2nd relational query simply use the Primary Keys from the 1st query?  So if my first query is:
SELECT `m`.`BlogId`, `m`.`Title`
FROM `Blogs` AS `m`
ORDER BY `m`.`BlogId` DESC
LIMIT 10

And my Result Set is:
13215	MySql Wiki Blog
13214	MySql Wiki Blog
13213	Entity Framework Wiki Blog
13212	MySql Wiki Blog
13211	MySql Wiki Blog
13210	Entity Framework Wiki Blog
13209	MySql Wiki Blog
13208	Entity Framework Wiki Blog
13207	MySql Wiki Blog
13206	Entity Framework Wiki Blog

Then why isn't the Relational Query:
SELECT `p`.`PostId`, `p`.`BlogId`, `p`.`Content`, `p`.`Title`
FROM `Posts` AS `p`
WHERE `p`.`BlogId` IN (
    13215,
    13214,
    13213,
    13212,
    13211,
    13210,
    13209,
    13208,
    13207,
    13206
)

This would achieve cut way down on the network transfer to the DbDataReader since the result set would only contain the records that were needed

When I remove these lines: IncludeExpressionVisitor.cs#L324-L366, the innerJoinSelectExpression in the else{} clause provides much better performance.  Generated SQL is:
SELECT `p`.`PostId`, `p`.`BlogId`, `p`.`Content`, `p`.`Title`
FROM `Posts` AS `p`
INNER JOIN (
    SELECT DISTINCT `m`.`BlogId`
    FROM `Blogs` AS `m`
    ORDER BY `m`.`BlogId` DESC
    LIMIT 10
) AS `m0` ON `p`.`BlogId` = `m0`.`BlogId`
ORDER BY `m0`.`BlogId` DESC

Result set is the proper size (20 Records, my 10 blogs have 2 Posts per blog) and Query Performance is still very fast:
Showing rows 0 - 19 (20 total, Query took 0.0003 seconds.)

I feel like this should be the default behavior.  I'm a little confused as to why the canGenerateExists is the default, is there a good reason?

dupe of #6530


Confusing install instructions in README

The README provides the following How to Use instructions:

Install this package
Then run npm install to update, build, and link the packages
Add the docusaurus-lunr-search plugin to your docusaurus.config.js
Then run docusaurus swizzle
Then build your Docusaurus project
Serve your application

However, if you skip step 4 everything works fine. If you attempt to edit the files by step 4 they aren't used due to the plugin being loaded from the npm module (step 3).
Question: how to swizzle and customize? Once I swizzle those files don't get used.
Suggestion: make it more clear that swizzling is not needed to use this plugin.
Thanks.

I can verify swizzling is not needed. Most likely this is suggesting a person might want to customize.
See https://docusaurus.io/docs/swizzling
Once swizzled you wrap it in your src/ or provide a new implementation as I can tell from reading the docs, but that is not a plugin issue, or this plugins specific thing.

So, I'm trying to use this on my docker, I've done local with swizzling (followed all the steps) and worked normally, but now on my docker I'm doing without the swizzle and it's not working, but I don't think the non-swizzling is the problem, is it?

Read me updated


Test out new uses of Barycentric classes

Closed by https://github.com/AztecProtocol/aztec-packages/pull/1897


Add pull request filter by head and base

According to https://developer.github.com/v3/pulls/#list-pull-requests this change allow to filter PR by head and base.

Awesome! Thanks for this contribution :)


Remove hardcodded Turbulent specific behaviors

In the good spirit of open-source, these hard-codded behaviors such as : https://github.com/turbulent/substance/blob/db0fccc8aec250054dd788e54e3f0f537cf4af24/substance/command/hatch.py#L52-L59
should be turned into variables that can be set in the config letting everyone benefit from these "shortcuts".

Related to #24


Fix ondemand test

Fix aad ondemand test failure that when the test is scheduled, test user need to be a new one.

/azp run java - spring - tests

/azp run java - spring - tests

/check-enforcer override


ZEPPELIN-2993. Code Refactoring of ZEPPELIN-1515 follow up

What is this PR for?
This is a refactoring PR of ZEPPELIN-1515. Because hadoop's FileSystem API not only works with hdfs, but also other hadoop compatible filesystem. So in this PR I rename it to FileSystemNotebookRepo
What type of PR is it?
[Refactoring]
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-2933

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

@prabhjyotsingh Could you help review it ? Thanks

Thanks @prabhjyotsingh Will merge it if no more comment


JSON parse error thrown in RPC error handler

Hello! 👋
We've been using this module in a production system and started to see these errors from about one week ago.

We're using v2.5.3, which we updated to 28 days ago according to our lockfile.
The system is a docker container built from node:6.9-alpine (so node v6.9).
The stack trace is as follows:
SyntaxError: Unexpected end of JSON input
    at Object.parse (native)
    at buildCustomError (/usr/src/app/node_modules/dropbox/src/rpc-request.js:9:29)
    at responseHandler (/usr/src/app/node_modules/dropbox/src/rpc-request.js:32:17)
    at Request.callback (/usr/src/app/node_modules/dropbox/node_modules/superagent/lib/node/index.js:703:3)
    at IncomingMessage.<anonymous> (/usr/src/app/node_modules/dropbox/node_modules/superagent/lib/node/index.js:922:12)
    at emitNone (events.js:91:20)
    at IncomingMessage.emit (events.js:185:7)
    at endReadableNT (_stream_readable.js:974:12)
    at /usr/src/app/node_modules/async-listener/glue.js:188:31
    at _combinedTickCallback (internal/process/next_tick.js:74:11)
    at process._tickDomainCallback (internal/process/next_tick.js:122:9)

Can you shed any light on the cause of the error? Unfortunately, I can't trace back to the contents of the HTTP response that was issued by Dropbox servers since the SDK caused our service to crash before we logged out the error.
Cheers

Thanks for the report! Have you been able to figure out how to reproduce this on demand by any chance?
In any case, if you can share the relevant code that would be helpful too.

We haven't been able to reproduce on-demand unfortunately. The errors appear to be transient.
The code is as follows: (this._dbx is a new Dropbox({ accessToken: user.token }))
const fn = this._cursor ? 'filesListFolderContinue' : 'filesListFolder'
const arg = this._cursor ? { cursor: this._cursor } : { path: '', recursive: true }
const backoff = new this._Backoff(this._backoffOpts)

this._dbx[fn](arg)
  .then((res) => this.handleApiResponse(res))
  .catch((err) => this.handleApiError(err, backoff, req))

So in summary we're using the api to list the entire contents of the Dropbox account, and also to respond to webhook changes, saving the cursor to track progress.

It looks like this would be the offending commit: https://github.com/dropbox/dropbox-sdk-js/commit/155cef7505b6b5a84844642f68dc68aab9ac37be
It is possible that our updated lockfile didn't make it into production until a week ago, which would explain why we only just started seeing the errors.

Thanks!


Add gpt-researcher tool implementation

Thank you for contributing to LangChain!

[ ] PR title: "community: add GPT-Researcher tool"

[ ] PR message: Delete this entire checklist and replace with

Description: GPTResearcher – an LLM-powered autonomous agent that dives deep into web, local, or hybrid sources to deliver comprehensive research on any topic. Now available as a LangChain Tool, seamlessly integrate it into ANY agent ecosystem and elevate your AI workflows.
Issue: N/A
Dependencies: gpt-researcher (https://pypi.org/project/gpt-researcher/)
Twitter handle: I don't have twitter but a mention of my LinkedIn would help - https://www.linkedin.com/in/makesh-srinivasan/

[ ] Add tests and docs: If you're adding a new integration, please include

a test for the integration, preferably unit tests that do not rely on network access,
an example notebook showing its use. It lives in docs/docs/integrations directory.

[ ] Lint and test: Run make format, make lint and make test from the root of the package(s) you've modified. See contribution guidelines for more: https://python.langchain.com/docs/contributing/

Additional guidelines:

Make sure optional dependencies are imported within a function.
Please do not add dependencies to pyproject.toml files (even optional ones) unless they are required for unit tests.
Most PRs should not touch more than one package.
Changes should be backwards compatible.
If you are adding something to community, do not re-import it in langchain.

If no one reviews your PR within a few days, please @-mention one of baskaryan, efriis, eyurtsev, ccurme, vbarda, hwchase17.

Hey! This adds a net-new community integration or feature, which has been replaced by dedicated integration packages. I'll close this PR, and would recommend reopening with just docs updates, as well as registering your package in libs/packages.yml! We'll be able to review simple PRs that only modify these two things much faster :)
Here's the guide, and if you have questions, feel free to leave them in the comments on those pages so others can see them! https://python.langchain.com/docs/contributing/how_to/integrations/


Enable compressed HTTP calls by default

As @shaneaevans has pointed out, we can shave off some network transfer costs when requesting compressed HTTP responses from the API.
By default, Zyte API returns uncompressed responses and only returns compressed responses when explicitly asked to do so via headers:
curl \
   --user API_KEY: \
   --header 'Content-Type: application/json' \
   --header 'Accept-Encoding: gzip' \   # 👈 this one
   --output 'out' \
   --data '{"url": "https://example.com/foo/bar", "browserHtml": true}' \
   https://api.zyte.com/v1/extract

Currently, only the User-Agent is set in the headers (code reference).
We'll need to:

decide on which Accept-Encoding value to use as the default on the header,
provide an easy way for users to easily change this, and
have the client automatically decompress the response if compressed.

We should definitely use compression, and make the client send the right Accept-Encoding header. Is it the case client is not sending it now (this is surprising to me)?

Based on https://github.com/aio-libs/aiohttp/issues/5219, I think you are right and we are doing it already.
Although maybe we could make brotli a dependency, to improve performance further.

The aiohttp.request has compress=False as the default (reference) with "deflate" being set.. Our current usage of aiohttp.request does not appear to be using the parameter (reference).
What do you think about passing this as the default? Accept-Encoding: br, gzip (assuming that brotli is installed as a main dependency)
On the other hand, we could also make brotli an optional dependency and would only send Accept-Encoding: br, gzip if installed. Otherwise, Accept-Encoding: gzip.

The aiohttp.request has compress=False as the default (reference) with "deflate" being set.. Our current usage of aiohttp.request does not appear to be using the parameter (reference).

compress=False is for compressing requests, not for decompressing responses; I think that's a separatediscussion topic. It also would need support from the server, I'm not sure if it'd work or not.

What do you think about passing this as the default? Accept-Encoding: br, gzip (assuming that brotli is installed as a main dependency). On the other hand, we could also make brotli an optional dependency and would only send Accept-Encoding: br, gzip if installed. Otherwise, Accept-Encoding: gzip.

If brotli is installed, aiohttp starts to add br to Accept-Encoding automatically, we don't need to do anything here. Documenting that installing brotli is good could help. Though I'm not sure if the server supports brotli or not.

compress=False is for compressing requests, not for decompressing responses; I think that's a separatediscussion topic. It also would need support from the server, I'm not sure if it'd work or not.

When testing it out in curl, the server supports compressed responses in gzip, deflate, and br when asked for via the Accept-Encoding Request Header.
On the other hand, the request body is automatically decompressed by aiohttp by the default auto_decompress=True parameter value in the client (reference).

I'm not sure I follow :) Is there anything for us to implement, or can we close this ticket?

My apologies, I've thought that aiohttp is not compressing it by default. Turns out it's indeed sending using gzip. Confirmed it manually by debugging the code.


Fix GitLab file picker spinner disappearing too early

This MR fixes https://github.com/jgraph/drawio/issues/2031 by making sure that the spinner remains shown when repos are being loaded.
I've done this by having a counter to keep track of the number of ongoing requests, so we can hide the spinner when they're all done (rather than hiding it whenever any of the requests finish).
I've only edited selectRepo. I think in theory selectRef and selectFile might have similar problems, but the number of files and refs is likely to be far smaller, so in practice they're probably not an issue.
Testing
I've tested this code on a local draw.io deployment integrated with a large private Gitlab instance, and the spinner shows for the correct amount of time. This makes the file picker functional again.

Thanks for the report. This will be fixed in the next release. (Merged via dev repo with minor fixes.)


ITE eSPI driver expecting OOB header also along with OOB data from app code - espi_it8xxx2_send_oob() & espi_it8xxx2_receive_oob

ITE eSPI driver expecting OOB header also along with OOB data from app code - espi_it8xxx2_send_oob() & espi_it8xxx2_receive_oob.
In espi_oob_msg_packet structure cycle_type, tag_len_bit_11_8 & len_bit_7_0 fields are OOB header. Currently ITE driver is such a way that app code has to fill/prepare OOB header also along with OOB data.
But in other EC vendors driver this OOB header data is taken care at driver level itself for both send & receive APIs.

ITE eSPI driver expecting OOB header also along with OOB data from app code - espi_it8xxx2_send_oob() & espi_it8xxx2_receive_oob.
In espi_oob_msg_packet structure cycle_type, tag_len_bit_11_8 & len_bit_7_0 fields are OOB header. Currently ITE driver is such a way that app code has to fill/prepare OOB header also along with OOB data.
But in other EC vendors driver this OOB header data is taken care at driver level itself for both send & receive APIs.


[sparse] correctly handle units in sparsify argspecs

With #7537, this is the last change required to seamlessly sparsify a jaxopt logistic regression

Good ideas! I'll take a look at that next week


[Feature]: global synchronization

Suggestion Description
Rocm has cooperative_groups.this_grid().sync()? (This is from nvidia cooperative groups)
I mean Global Synchronization in the grid, it is like cooperative_groups.this_grid().sync()
also:
Does rocm also have shared memory?
That you can use with “asynchronous copies,” where what interests me the most is the ability to move data from the device to shared memory without going through the registers.
Operating System
No response
GPU
No response
ROCm Component
No response

Cooperative groups are supported, including this_grid.sync().
HIP, like Cuda, has shared variables.  HIP doesn't currently support some of the Cuda async functions.


npm install is broken

With current versions in package.json it's not possible to install dependencies:
npm ERR! peerinvalid The package<EMAIL_ADDRESS>does not satisfy its siblings' peerDependencies requirements!
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>|| ^15.0.0
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>|| ^15.0.0
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>|| ^15.0.0
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>|| 0.14.x || ^15.0.0-0 || 15.x
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>

@skrobek
You can't start the project with this versions. You need to upgrade your npm and node programs.
if you are using brew on OSX then you can use following commands to upgrade them.
brew update
brew upgrade node
npm install -g npm
Have a nice day.

Note:
You can find minimum version of npm in the package.json.

@skrobek
Can you let me know the state of the starting project after you upgrade npm and node programs. Then I can close the issue.
Have a nice day.

This issue is related by npm and node version. That's why I am closing the issue.


v3.0.3 has no binaries

Summary
When trying to install the v3.0.3 CLI there are no binaries on the release for this version.
Diagnostics
Viewing the releases tab on GitHub shows no binaries whilst there are some for all other releases.
wget https://github.com/argoproj/argo/releases/download/v3.0.3/argo-darwin-amd64.gz
--2021-05-13 11:22:19--  https://github.com/argoproj/argo/releases/download/v3.0.3/argo-darwin-amd64.gz
Resolving github.com (github.com)... <IP_ADDRESS>
Connecting to github.com (github.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 301 Moved Permanently
Location: https://github.com/argoproj/argo-workflows/releases/download/v3.0.3/argo-darwin-amd64.gz [following]
--2021-05-13 11:22:19--  https://github.com/argoproj/argo-workflows/releases/download/v3.0.3/argo-darwin-amd64.gz
Reusing existing connection to github.com:443.
HTTP request sent, awaiting response... 404 Not Found
2021-05-13 11:22:20 ERROR 404: Not Found.

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

uploaded


 There are no errors in the log, and I have my config file below. I have tried npm serveronly , same thing just in my browser. I am running a Raspberry Pi 4b with Raspbian 10 using the Budgie Desktop. I cannot afford to reinstall right now, because then I would have to reinstall all my modules and I don't have time, so I would appreciate every possible solution besides re-installation.
Modules
MMM-Cursor
MMM-AssistantMk2
WallberryTheme
MMM-Hotword

config.js
/* Magic Mirror Config Sample
 *
 * By Michael Teeuw http://michaelteeuw.nl
 * MIT Licensed.
 *
 * For more information how you can configurate this file
 * See https://github.com/MichMich/MagicMirror#configuration
 *
 */

var config = {
	address: "localhost", // Address to listen on, can be:
	                      // - "localhost", "<IP_ADDRESS>", "::1" to listen on loopback interface
	                      // - another specific IPv4/6 to listen on a specific interface
	                      // - "", "<IP_ADDRESS>", "::" to listen on any interface
	                      // Default, when address config is left out, is "localhost"
	port: 8080,
	ipWhitelist: ["<IP_ADDRESS>", "::ffff:<IP_ADDRESS>", "::1"], // Set [] to allow all IP addresses
	                                                       // or add a specific IPv4 of <IP_ADDRESS> :
	                                                       // ["<IP_ADDRESS>", "::ffff:<IP_ADDRESS>", "::1", "::ffff:<IP_ADDRESS>"],
	                                                       // or IPv4 range of <IP_ADDRESS> --> <IP_ADDRESS> use CIDR format :
	                                                       // ["<IP_ADDRESS>", "::ffff:<IP_ADDRESS>", "::1", "::ffff:<IP_ADDRESS>/28"],

	language: "en",
	timeFormat: 12,
	units: "imperial",

	modules: [
		{
			module: "alert",
		},
		{
			module: "updatenotification",
			position: "top_bar"
		},
		{
			module: "newsfeed",
			position: "bottom_bar",
			config: {
				feeds: [
					{
						title: "CNet",
						url: "http://news.cnet.com/2547-1_3-0-20.xml"
					},
					{
						title: "Android Authority",
						url: "http://www.androidauthority.com/feed/"
					},
{
						title: "Engadget",
						url: "https://www.engadget.com/rss.xml"

					},
				],
				showSourceTitle: true,
				showPublishDate: true,
				broadcastNewsFeeds: true,
				broadcastNewsUpdates: true
			}
		},
{
	module: "MMM-Hotword",
	position: "top_left",
	config: {
		chimeOnFinish: false,
		mic: {
			recordProgram: "arecord",
			device: "plughw:1"
		},
		models: [
			{
				hotwords    : "smart_mirror",
				file        : "smart_mirror.umdl",
				sensitivity : "0.9",
			},
		],
		commands: {
			"smart_mirror": {
				notificationExec: {
					notification: "ASSISTANT_ACTIVATE",
					payload: (detected, afterRecord) => {
						return {profile:"default"}
					}
				},
				restart:false,
				afterRecordLimit:0
			}
		}
	}
},
{
  module: "MMM-AssistantMk2",
  position: "top_left",
  config: {
    ui: "classic2", // or "Classic", "Classic2"
    assistantConfig: {
      projectId: "integration-service-6339c", // Required to use gaction.
      modelId: "integration-service-6339c-mirrorx-v8kg0x", // (OPTIONAL for gaction)
      instanceId: "harmony-home-mirrorx", // (OPTIONAL for gaction)
      latitude: 51.508530,
      longitude: -0.076132,
    },
    responseConfig: {
      useHTML5: true, // sound render by HTML5
      useScreenOutput: true,
      useAudioOutput: true,
      useChime: true,
      timer: 5000,
      myMagicWord: false,
      //Your prefer sound play program. By example, if you are running this on OSX, `afplay` could be available.
      playProgram: "mpg321", //needed if you don't use HTML5 sound render
      chime: {
        beep: "beep.mp3",
        error: "error.mp3",
        continue: "continue.mp3",
        open: "Google_beep_open.mp3",
        close: "Google_beep_close.mp3",
      },
      // false - animated icons, 'standby' - static icons only for standby state, true - all static icons
      useStaticIcons: false
    },
    micConfig: { // put there configuration generated by auto-installer
      recorder: "arecord",
      device: "plughw:1",
    },
    customActionConfig: {
      autoMakeAction: false,
      autoUpdateAction: false, // in RPI, gaction CLI might have some trouble.(current version should be 2.2.4, but for linux-arm, Google haven't updated) so leave this as false in RPI. I don't know it is solved or not.
      actionLocale: "en-US", // At this moment, multi-languages are not supported, sorry. Someday I'll work.
    },
    recipes: [ "with-MMM-Hotword.js", "with-MMM-TelegramBot.js" ],
    profiles: {
      "default": {
        profileFile: "default.json",
        lang: "en-US"
      }
    },
    addonsConfig: {}, //describe addons configuration
  },
},
{
    module: "WallberryTheme",
    position: "fullscreen_below", // Required Position
    config: {
      unsplashAccessKey: "####################", // REQUIRED
      collections: "343012" // optional - leave empty for a random photo
    }
  },
  // WB-clock adds local time (Optional Module)
  {
    module: "WallberryTheme/WB-clock",
    position: "top_bar", // highly suggest using top_bar position
    config: {
      localCityName: "Hello ------", // optional

    }
  },
  // WB-weather adds weather via DarkSky API (Optional Module)
  {
    module: "WallberryTheme/WB-weather",
    position: "bottom_bar",  // Highly suggested location
    config: {
      // See "Configuration options" for more information.
      daysToForecast: "3",
      darkSkyApiKey: "---", // REQUIRED
      latitude:   0.0, // REQUIRED
      longitude: 0.0 // REQUIRED

    }
},

{
		module: 'MMM-Cursor',
		config: {
			// See 'Configuration options' for more information.
		}
	},

	]

};

/*************** DO NOT EDIT THE LINE BELOW ***************/
if (typeof module !== "undefined") {module.exports = config;}

Please help me.
All API keys and personal details have been removed from the file.

Ok, I figured it out from a little debugging I did myself. It is the Google Assistant module, and it isn't something I have the time to try and fix myself. So far, the developer hasn't even answered the questions of others who are having the same problem.

@android9pie2   can you close this issue now?


Only parse docs blocks contents when reading md files (#988)

Fixes #988
Also, related: Fix docs blocks parsing issues
Rename documentation node type to docs so we can filter on it (is this breaking?)
Fix block extractor bug with macros/docs that contain quotes
Fix block extractor bug with expressions

Just uploaded a readme.md file that is meant to trigger the actual issue, I failed to git add it before!


Star Ratings

The star ratings need to be whole stars and not fractions. Also, 1 star is the minimum.  You need to ask where this is defined in the grading rubric.

The star ratings are now only decremented for a whole star and won't go below one.


Loading BERT checkpoints into TLM from TensorFlow

We have a solution for loading BERT checkpoints into PyTorch 8 mile layers directly from huggingface checkpoints by modifying the keys in the pytorch state dict.  What would be better is to use this code to then re-write the checkpoint in TLM NPZ format so it can be loaded into TensorFlow 8 mile layers as well.  The NPZ files can be hosted on dropbox and has the advantage that the same checkpoint can be used in pytorch and TF

This PR makes TF support the same options as PyTorch including layer norm epsilon and T2T-style LN (after) https://github.com/dpressel/mead-baseline/pull/537

Waiting on checkpoint generation for verification


Dashboard

what happened to the dashboard?  I know Alex got to busy with school, but were you able to create anything there?

schools


unable to load ?

i have a project around 7 modules(files), when i got a .exe file in windows, with python2.7 and vs2012, it can't start up succeed. the error is like below:
D:\Projects\dms\dmc>dmc.exe
loading dmc...
nexe.js:12
throw new Error("unable to load " + path);
^
Error: unable to load dmc
at _require (nexe.js:12:19)
at module.exports (nexe.js:50:20)
at _require (nexe.js:14:49)
at nexe.js:2421:9
at nexe.js:2423:3
at NativeModule.compile (node.js:896:5)
at Function.NativeModule.require (node.js:865:18)
at Function.Module._load (module.js:297:25)
at Module.require (module.js:364:17)
at require (module.js:380:17)
the 'loading dmc...'  is the debug output of my dmc.js module, which appear in the first line of the js file.

This is included in the latest version, so this is fixed.


Adds scripts to modify the preprocessed data through CLI

This is needed to be able to update the preprocessed data of the EMP studies already present on the DB

Coverage increased (+0.1%) to 78.35% when pulling 618da3dfddac3964f02b208971ef9fc97f99daa8 on josenavas:update-ppd into 16d3ed603c7d8b65b2315e4bf46c798a3d6ae0ad on biocore:master.

:+1:

Coverage increased (+0.24%) to 78.49% when pulling 8a8a8c02d668c8e676a3aa1aaaba39faed5598b7 on josenavas:update-ppd into 16d3ed603c7d8b65b2315e4bf46c798a3d6ae0ad on biocore:master.

This should be ready for another review!

Just a few comments, but looks good overall.

@josenavas this build has legitimate failures.

Coverage increased (+0.17%) to 78.6% when pulling e7ed54f4c61cabf255c2fedcbb457806771daeb4 on josenavas:update-ppd into 49e2e49136383ad20a6257e50043644fdedeb476 on biocore:master.

Thanks for the heads up @ElDeveloper
This should be ready to go!

No problem, :+1:

Can I have more one reviewer here??


Fedora Repository

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I think the RPM package needs to be manually updated.
Describe the solution you'd like, including relevant patches or source
A clear and concise description of what you want to happen.
Having a package on RPMFusion. Unlike official Fedora repositories, they don't have a problem with codecs.
Or a custom repository.
Additional Notes
Add any other context or screenshots about the feature request here.
The documentation for uploading packages to RPMFusion is here.

AFAIK noone of current contributors of Thorium uses Fedora regularly.
So maintainance resources are pretty much limited here.

I use Nobara (built on Fedora), and updates are handled by going to GitHub releases and simply installing the latest .rpm file (software takes care of it). Sure, something could be made to let you know that there's a new Thorium release out, but the process is already fairly straightforward.
I'd try out something that can notify you when a repo has a new release: https://github.blog/changelog/2018-11-27-watch-releases/ or https://akomljen.com/an-easy-way-to-track-new-releases-on-github/

I use Nobara (built on Fedora), and updates are handled by going to GitHub releases and simply installing the latest .rpm file (software takes care of it). Sure, something could be made to let you know that there's a new Thorium release out, but the process is already fairly straightforward.
I'd try out something that can notify you when a repo has a new release: https://github.blog/changelog/2018-11-27-watch-releases/ or https://akomljen.com/an-easy-way-to-track-new-releases-on-github/

Interesting, but it isn't as convenient as a dnf update and I guess there's some sort of tradeoff with dependency analysis.
Hmm, is it possible to create some sort of simple proxy that wraps Github Releases API and acts like package manager repositories?
I'll try this.

I use Nobara (built on Fedora), and updates are handled by going to GitHub releases and simply installing the latest .rpm file (software takes care of it). Sure, something could be made to let you know that there's a new Thorium release out, but the process is already fairly straightforward.
I'd try out something that can notify you when a repo has a new release: https://github.blog/changelog/2018-11-27-watch-releases/ or https://akomljen.com/an-easy-way-to-track-new-releases-on-github/

Interesting, but it isn't as convenient as a dnf update and I guess there's some sort of tradeoff with dependency analysis.
Hmm, is it possible to create some sort of simple proxy that wraps Github Releases API and acts like package manager repositories?
I'll try this.

Let me know how this went, as an openSUSE user I am very curious about this

I use Nobara (built on Fedora), and updates are handled by going to GitHub releases and simply installing the latest .rpm file (software takes care of it). Sure, something could be made to let you know that there's a new Thorium release out, but the process is already fairly straightforward.
I'd try out something that can notify you when a repo has a new release: https://github.blog/changelog/2018-11-27-watch-releases/ or https://akomljen.com/an-easy-way-to-track-new-releases-on-github/

Interesting, but it isn't as convenient as a dnf update and I guess there's some sort of tradeoff with dependency analysis.
Hmm, is it possible to create some sort of simple proxy that wraps Github Releases API and acts like package manager repositories?
I'll try this.

Let me know how this went, as an openSUSE user I am very curious about this

I didn't really do anything. It should theoretically be doable, but I don't know if I want to drown myself in DNF or zypper specifications.
For this particular package though, I don't think you need a repository. Chromium apparently implements everything in one package and there's no dependency that Fedora doesn't already have. I imagine you can just download the zip file.

+1 for Fedora COPR or RPMFusion

Same here having t o update thorium manually is annoying, especially since chrome can do it.
We have package managers for a reason.

I second this.


should getVarNames have includeData argument?

A variable can be a mix of data and non-data, so not clear to me this should be an option.

Legitimate question.  To add to the questions,
(1) I found no instances in the entire package of calling getVarNames() which use the includeData argument.
(2) It looks like it should error out anytime includeData=FALSE, the opposite of the default value.  It uses an object tm, which is not defined anywhere I can find:
allData <- unlist(lapply(mget(ans, envir = tm$isDataEnv, inherits = FALSE, ifnotfound = TRUE), all))
Upon testing, it does error out, if you ever provide includeData=FALSE.
Being that the implementation doesn't work, and it's never used anywhere, I'm tempted to comment it out in the code base;  or change it so that a developer's warning is issued, if that case is ever invoked.
Other thoughts, @paciorek or @perrydv ?

Additionally, this includeData argument, and the error-prone implementation, were added to the getVarNames() method by @perrydv , in commit b5acc90df29926cd8261a79371dc61e0cdf63a45, in April 2015, with commit message "fixing getVarNames and default handling of nimCopy", in case that helps jog memory.

I don't recall the history of this option, but I agree it is evidently defunct.  Let's try removing it.  As long as we are not code-generating and constructing the string "includeData" by pasting ;-), your search on all uses in the code base, which I verified, seems sound.  I agree about tm being an obvious symptom of being defunct.
On principle it doesn't seem like a bad option to provide, with the intention that if any node in a variable is data then that variable would be omitted if includeData = FALSE.  But since we don't use it let's try removing it.

I actually more agree with Chris's hesitations about this, that any sort of includeData argument doesn't make sense, with our notion of "variable".  Maybe this can become another great NIMBLE debate.

Well at least we're agreed to remove it for now.

Done.  Commented it out, with an explanatory comment.
Nice catch @paciorek.  Closing issue.


Rename package to @foxglove/regl-worldview

Update all package name references from regl-worldview to @foxglove/regl-worldview.

What's the status of this?

Oh, I didn't realize I never merged it. Unknown status, needs someone to push it over the finish line.


Prepare Bundle: FAILED - Error response from daemon: dockerfile parse error line 28: unknown instruction: EXPORT

hi @zodern
thank you for your supports on this community, I ran into this error today when I was trying to deploy my old meteor app buy using meteor mup and the new docker image zodern/meteor:root

Could you please help me look at it and point me out how to fix it? thank you very much.
P/S: Does anyone else encounter this issue and know how to fix it?

I'm sorry, it's my mistake, I was putting the wrong command in the buildInstructions. I'll close this issue.


dead link to https://napari.org/docs/plugins/for_plugin_developers.html

🐛 Bug

The link is mentioned here: https://napari-plugin-engine.readthedocs.io/en/latest/
See also: documentation for writing a napari plugin.
Is the documentation still somewhere?

The current page is here: https://napari.org/plugins/stable/for_plugin_developers.html
The legacy versions documentation follows a pattern of https://napari.org/docs/<version>/plugins/for_plugin_developers.html, with the most recent one here: https://napari.org/docs/dev/plugins/for_plugin_developers.html

thanks @chili-chiu, since those are the docs for the plugin engine itself, could you open an issue there?
https://github.com/napari/napari-plugin-engine


[FE]: Some way to bulk edit DefectSuite for a bunch of Defects

Description:
(details of what and how should work)
As title says
Additional Context & Ad Hoc Info:
(Add any other context about the problem here.)
https://teams.microsoft.com/l/message/19:261159a7-5979-45a9-a6c5-bbc0f04e8204_7f757c7c-1c63-45e8-9eac-ad22ae7b915e@unq.gbl.spaces/1713874660372?context={"contextType"%3A"chat"}
It is a Terminal thing, no UI required, dependent on #60 & #77

Will be doable with a release of 3.4 Terminal ERs #60

BETA testing instructions
To add multiple Defects to a DefectSuite, run below commands in RVI terminal:
cd <DS_ID>
list <DE_ID_1> <DE_ID_2> <DE_ID_3> | xargs { echo -p "Defects=%" | patch }

All 3 DEs should now be added to DefectSuite

Validated


localhost:8988 message, update required

On Windows 7 64bit, after starting the server and connecting to localhost:8988 , this message appear:
Upgrade required
Tried different browsers, doesn't help.

Thanks for your note. As a linux guy I have to admit I tested only on Windows 10. Are you using the latest server 0.6.0 (https://github.com/tekartik/serial_wss/releases/tag/0.6.0). And what happens when you run and start the server then launch the sample terminal (http://ws.tekartik.com/serialwssterminal/)


chore(main): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2024-01-10)
Bug Fixes

add node types (#3) (392d5da)
remove ts config (#2) (91735e2)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/achieveagility/nx-publish/releases/tag/v1.0.0 :sunflower:


Colors plugin example not working

Following the example for colors plugin does not add the buttons:
$('#editor')
.trumbowyg({
  // You can only add one of foreColor/backColor
  btnsAdd: ['foreColor', 'backColor']
});

However, using btns instead works:
btns: [...,
  ['foreColor', 'backColor']
]

Using btns requires listing all other buttons.

Hi!
In fact, since v2 stable that option has changed, I need to update all examples and improve documentation.
Thanks for your feedback :)

oh please update documentation. Its not encouraging to read

WTF are btnGrp-* buttons?

Well I managed now to get on, but I can not change the colors as I did before using
$.trumbowyg.opts.colors = [];

Exemple fixed in 2.8.0 which will be released soon.


FR: auto-localization

Hey, I just installed your addon and it seems to work really well! Thanks a lot for your work!
I was wondering if it would be possible to update the fetched information based on various site locales.
I for example am using a website with two locales and while Algolia Places can provide localized location data the addon lets me only choose a default for all sites.
I don't know if this is possible, however such a feature would be greatly appreciated.

I've added this for 1.0.2, if you leave the language empty in the field config, it should use the selected site's locale instead. Let me know if it works for your case!

Cool! I'll give it a try asap


Issue building docker image from Dockerfile

PIP install PyYAML is failing with an error due to SSL cert. I was able to get around it adding
by adding trusted host. But then it resulted in a different error.
--trusted-host pypi.org  --trusted-host pypi.python.org --trusted-host files.pythonhosted.org
HTTP error 403 while getting https://files.pythonhosted.org/packages/a3/65/837fefac7475963d1eccf4aa684c23b95aa6c1d033a2c5965ccb11e22623/PyYAML-5.1.1.tar.gz#sha256=b4bb4d3f5e232425e25dda21c070ce05168a786ac9eda43768ab7f3ac2770955 (from https://pypi.org/simple/pyyaml/

I cannot reproduce the issue you describe. Just built from scratch in a clean repo; this sounds like a networking issue on your end.

Yep, that right. Did not realize that it was my company networking issue. Thank you.


Document setup steps for local development with subgraphs

Add any environment variables that need to be set
Document any additional steps that need to be taken or details to be aware of

Will be added to the relay docs subgraph example


Package requested always from uplink after maxage

Well, I guess not exactly always, but close.
Here is a log from version v2.1.1 with the default 2 minute maxage:
Verdaccio doesn't need superuser privileges. Don't run it under root.
 warn  --- config file  - /verdaccio/conf/config.yaml
2017-02-14T09:49:00.370Z warn  --- http address - http://<IP_ADDRESS>:4873/
2017-02-14T09:49:08.517Z http  --> 200, req: 'GET https://registry.npmjs.org/lodash', bytes: 0/167638
2017-02-14T09:49:08.575Z http  <-- 200, user: undefined, req: 'GET /lodash', bytes: 0/443696
2017-02-14T09:49:25.345Z http  <-- 200, user: undefined, req: 'GET /lodash', bytes: 0/443696
2017-02-14T09:50:06.242Z http  <-- 200, user: undefined, req: 'GET /lodash', bytes: 0/443696
2017-02-14T09:51:03.365Z http  <-- 200, user: undefined, req: 'GET /lodash', bytes: 0/443696
2017-02-14T09:51:06.733Z http  <-- 200, user: undefined, req: 'GET /lodash', bytes: 0/443696
2017-02-14T09:51:09.527Z http  --> 304, req: 'GET https://registry.npmjs.org/lodash', bytes: 0/0
2017-02-14T09:51:09.555Z http  <-- 200, user: undefined, req: 'GET /lodash', bytes: 0/443696
2017-02-14T09:51:15.048Z http  --> 304, req: 'GET https://registry.npmjs.org/lodash', bytes: 0/0
2017-02-14T09:51:15.073Z http  <-- 200, user: undefined, req: 'GET /lodash', bytes: 0/443696
2017-02-14T09:51:19.648Z http  --> 304, req: 'GET https://registry.npmjs.org/lodash', bytes: 0/0
2017-02-14T09:51:19.671Z http  <-- 200, user: undefined, req: 'GET /lodash', bytes: 0/443696
2017-02-14T09:51:24.575Z http  --> 304, req: 'GET https://registry.npmjs.org/lodash', bytes: 0/0
2017-02-14T09:51:24.611Z http  <-- 200, user: undefined, req: 'GET /lodash', bytes: 0/443696
2017-02-14T09:51:27.265Z http  --> 304, req: 'GET https://registry.npmjs.org/lodash', bytes: 0/0
2017-02-14T09:51:27.289Z http  <-- 200, user: undefined, req: 'GET /lodash', bytes: 0/443696

What happens here is that during the two minutes of maxage the uplink is not requested. After the maxage every request causes a request to the uplink.
My guess is that response 304 does not cause the fetched timestamp to be updated correctly.

The problem is that get_package in lib/up-storage.js does not pass the 304 response as _sync_package_with_uplinks in lib/storage.js expects.

@ilponyy what do you mean with "maxage"? I have no idea what is that.

conf/full.yaml uplinks:
    # maximum time in which data is considered up to date
    #
    # default is 2 minutes, so server won't request the same data from
    # uplink if a similar request was made less than 2 minutes ago
    #maxage: 2m

I run into this issue.
Is there a way to set something like maxage: Infinit ?
As I want to use this tool to improve the development process of my team, whenever somebody causes a package to be fetched, I don't want it to be fetched again.
If needed, I can always reset this cache myself.

@cmdelatorre I don't get it. Do you want to limit fetch a package only once?

@juanpicado sorry that I was not clear. When I say

I don't want it to be fetched again

I mean that I don't want the cache to fetch that package from the oficial npmjs.org again. Since it was already fetched once and it is now in the cache, every time we install it, the verdaccio cache is serving it all-right.
My use case is this: we are several devs on one project. Once installed, the node_modules directory uses 1GB of space. We install the same set of dependencies often, with little changes over time. So, using Verdaccio as a cache for those stable set of packages is a real improvement.
Thanks for the good work.

@cmdelatorre
You mean this? ---> #275
Here are a lot of possible, base on package manager's cache not work at all.

if you are using latest npm or yarn with lock file, and you remove it after package.json update, your problem will resolved by issue listed above. verdaccio will always try get package info from upstream to keep version list update.  After this issue has been implement, verdaccio will not get package info before it expired.

If you haven't remove it, package manage should direct download package tarball rather than query package info so it should't send any request to upstream.

If you are using old version npm and no lock file, npm will request all package info every time and verdaccio will get it from upstream, and if package have any new patch and minor release, package manager will download the latest one so cache won't work, it will resolve by issue listed above. but i suggest you upgrade to a more recent version to increase speed before we done it.
But if you decide keep at old version package manager, package cache will not always work because it will expire, and you can find it's still slow after package info expired

/cc @juanpicado