[BUG] Vite dev server hangs on initial page load

Describe the bug
Vite's dev server seems to hang when running a fresh project. It seems to happen on the first page load, where the initial SSR response is returned, the client loads, and then Vite detects new dependencies so it starts a reload.
The page never fully seems to reload. Requests are hanging in the Network tab (see screenshot).
We also sometimes see errors related to headers being sent during streaming & the standard "Can't use Switch outside Router" which is usually a red herring #64
To Reproduce
Steps to reproduce the behaviour:

Create a new Hydrogen project, or run in the monorepo with the latest Vite dev server & starter template
Start the dev server
Observe the behavior

Expected behaviour
The dev server should start up and be snappy. Any dependency refreshes should happen instantly.
Screenshots

Additional context
Add any other context about the problem here. eg.

Hydrogen version: 0.6.0
Node version: v16
Vite version: 2.6.13

@frandiox Curious if you're seeing this and if you have any ideas!
My initial thoughts:

Something related to a newer version of Vite? Monorepo is locked at 2.6.2 and it doesn't seem to happen then
Something related to the app tree in the new starter template?
Something related to #108?

Also, I am seeing some server components e.g. ShopifyServerProvider get loaded in the client, as well as some hefty libraries like graphql. This happens when we re-export server components in the same places we pull components for the client module.
However, I refactored a bit and was still able to reproduce, so I don't think it's related.

Also, I am seeing some server components e.g. ShopifyServerProvider get loaded in the client, as well as some hefty libraries like graphql. This happens when we re-export server components in the same places we pull components for the client module.
However, I refactored a bit and was still able to reproduce, so I don't think it's related.

This kinda helps?
import {defineConfig} from 'vite';
import hydrogen from '@shopify/hydrogen/plugin';

import shopifyConfig from './shopify.config';

// https://vitejs.dev/config/
export default defineConfig({
  plugins: [hydrogen(shopifyConfig)],
  optimizeDeps: {
    include: ['@headlessui/react'],
  },
});

Not sure though.

This only started happening later in the day on Friday so something related to https://github.com/Shopify/hydrogen/pull/108 seems plausible 🤔

Confirmed that adding optimizeDeps: {include: ['@headlessui/react']}, to vite.config.js in the starter template fixes our issue in the short term. IMO this should unblock us for Monday morning.
That gives us time to investigate the real issue which is: when Hydrogen's starter template includes dependencies that need optimization after they're loaded via e.g. client components, Vite attempts to refresh the page, and then the server stops responding, flat-out.
Super mysterious. I added console.log to the very top of the Hydrogen middleware, and on the refresh attempt it doesn't even make it to the log point. So something is really screwy.
I don't think it was something introduced on Friday, it's just that we added headlessui/react to our client-side stack recently, which is what makes us suddenly aware of the issue.

@jplhomer Ah I see, so that's why I didn't see this earlier. I normally test in the branch for #7 where this dependency is already optimized. I'm going to cherry-pick that commit and add it to main branch 👍

@jplhomer I think this is not happening anymore even without the optiimizedDeps after https://github.com/Shopify/hydrogen/pull/168 and https://github.com/Shopify/hydrogen/pull/144 were merged so I'm closing this for now 👍

@Shahzaib342 I cannot reproduce this anymore. Can you try running the dev server with yarn dev --force? Just to make sure this is not Vite caching something. Otherwise, try removing node_modules and update to latest Hydrogen 0.8.0.

@Shahzaib342 I cannot reproduce this anymore. Can you try running the dev server with yarn dev --force? Just to make sure this is not Vite caching something. Otherwise, try removing node_modules and update to latest Hydrogen 0.8.0.

Hi @frandiox thanks for the reply. I tried "yarn dev --force" and also tried to change Hydrogen to 0.8.0. But still getting same issue. My node version is v16.13.1 and npm version is 8.2.0.
I am following this documentation https://shopify.dev/custom-storefronts/hydrogen/getting-started on Ubuntu 20.04.3 LTS.

For some reason, my laravel 9 project all of the sudden just keeps loading... i stashed all changes, and it continued? how can vite just blow up?


Fixed incorrect filename in config

Fixes issue #464

@georgezhao2010 please review, thank you!

I don't see hacs.json in your forked repo, so not sure if my pull request is applicable. On the other hand, I see my fix in diff on https://github.com/georgezhao2010/midea_ac_lan/compare/master...wuwentao:midea_ac_lan:master. Not sure why the file is gone though. Maybe it was removed in some later commit and that doesn't show in that view.
Either way, thanks for picking this up. I've installed the integration manually by checking out my own fork on the device, but I don't have the time to maintain it (accept pull requests and such), so glad that you do :-).

I don't see hacs.json in your forked repo, so not sure if my pull request is applicable. On the other hand, I see my fix in diff on master...wuwentao:midea_ac_lan:master. Not sure why the file is gone though. Maybe it was removed in some later commit and that doesn't show in that view.
Either way, thanks for picking this up. I've installed the integration manually by checking out my own fork on the device, but I don't have the time to maintain it (accept pull requests and such), so glad that you do :-).

Why you don't see hacs.json in diff:
as most of the PR in current repo try to fix this input error, and we have fixed it in https://github.com/wuwentao/midea_ac_lan
so you can directly access it and check the file, it should be exist and already have right value now
in additional, there is too many code changes by python-black and python-isort code format tool, also fixed python flake8 with old source code,  you can check the detail in my repo about the changes.
now it should be more script checks rule in new fork.
you can continue use your fork if you don't need any new update.


Amend install / uninstall functionality

Themes can be "detected" i.e. loaded into the container
Themes can be "installed" i.e. we'll create a table to hold the installed themes, these are ones that have been "installed" by the theme manager, this will involve running an "instal()" method on the theme base.
This will allow us to publish things like screenshots here.
Themes can be "activated" i.e. this is the active theme, when a theme is activated, the current active theme will be "deactivated".

The above model will also work to get round the issue of "published" views / assets and knowing whether to delete them or not.
Assets / views will not be deleted on "deactivate" but when a theme is "uninstalled"


Add option to remove space before '/>'

Could you please add an option to disable insertion of the extra space before the slash in self-closing tags?

option added


[scalardl-ledger] Support cert-manager in ScalarDL Ledger chart

Description
This PR updates ScalarDL Ledger chart to support cert-manager.
Please take a look!
Related issues and/or PRs

https://github.com/scalar-labs/docs-internal-orchestration/pull/24

You can see the documents that describe how to use these new features in this PR.

https://github.com/scalar-labs/helm-charts/pull/262

If you use self-signed CA, this chart deploy self-signed CA. And, Scalar Envoy chart refer it to generate certificate for Envoy.

Changes made
Mainly, this PR adds/updates the following features (manifests):

Deploy a Certificate resource for cert-manager by using charts/scalardl/templates/ledger/certmanager.yaml.

Deploy self-signed CA and create certificate for ScalarDL Ledger if you don't specify any Issuer resources in issuerRef configuration.

If you deploy Scalar Envoy as a sub chart of this ScalarDL Ledger chart without issuerRef configuration, the Envoy chart will use self-signed CA that is deployed by this chart to create certificate for Envoy.

[ScalarDL Ledger chart (this chart)] ---(Deploy)---> [Issuer (self-signed CA)] <---(Refer via issuerRef configuration)--- [Certificate resource] <---(Deploy)--- [Envoy chart]

Update deployment.spec.template.spec.volumes and deployment.spec.template.spec.containers.volumeMounts to remove subPath configuration.

Cert-manager creates a secret resource that includes private key and certificate, and we use them by mounting them on pods. And, cert-manager automatically renew (update) the certificate in the secret resource before the certificate will be expired. However, if we use subPath to specify the mounted file name, Kubernetes does not apply the changes of the secret resource. So, to take advantage of cert-manager's automatically renew feature, we should not use subPath. This is because I removed subPath from the volumeMounts configuration.

Update file names of private key and certificate.

From the perspective of cert-manager specification (strictly, TLS Secrets specification of Kubernetes), we must use the fixed names for private key and certificate files. This is because cert-manager creates a secret resource that includes private key and certificate files with fixed names based on the TLS Secrets specification. So, I updated the file names in existing manifests.

cert.pem -> tls.crt
key.pem -> tls.key
ca.pem -> ca.crt

Checklist

[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have updated the documentation to reflect the changes.
[x] Any remaining open issues linked to this PR are documented and up-to-date (Jira, GitHub, etc.).
[x] Tests (unit, integration, etc.) have been added for the changes.
[x] My changes generate no new warnings.
[x] Any dependent changes in other PRs have been merged and published.

Additional notes (optional)
N/A
Release notes
Support cert-manager in ScalarDL Ledger chart. You can manage private key and certificate for TLS connections by using cert-manager.

@supl
Thank you for your review and question!

I don't know if it's possible. I am curious if the CA can sign the root certificate by itself?

Sorry, I'm not sure whether we can do it or not...
However, I created those two Issuer resource manifests for self-signed CA by following the official document of cert-manger.
https://cert-manager.io/docs/configuration/selfsigned/
And, it seems that kube-prometheus-stack adopts the same way, which is described in the above cert-manager documents. (In other words, this way might be a general/popular way to generate a self-signed certificate by using cert-manager.)
https://github.com/prometheus-community/helm-charts/blob/kube-prometheus-stack-58.2.2/charts/kube-prometheus-stack/templates/prometheus-operator/certmanager.yaml
Also, I don't have any clear motivations to adopt a different way from the cert-manager's official document at this point. (Vice versa, I think there might be some risks something like unexpected errors by adopting a different way from the cert-manager's official document.)
These are why I adopt the current way (there are two Issuer resources *-self-signed-issuer and *-ca-issuer) in this chart.

@kota2and3kan
Thank you for the explanation!
Understood. There's no problem.
I am just curious if it's even possible 😄

@supl
Thank you! I keep the current implementation as is!
Your concern makes sense to me. Because when I saw the official document first, I was also very confused... But, this chart deploys all self-signed CA related resources under the abstraction by Helm Chart.   So, I guess users can use it with less confusion than using cert-manager directly by themselves (Although they might be confused even if this implementation...).


Tagline, Global eyebrow, and Global eyebrow actions

As we continue our work on building the React component library we are currently working on adding the Tagline and Global eyebrow components.
Based on the SBL layouts we will need components that match:

a-tagline
m-global-eyebrow
m-global-eyebrow_actions

Existing components (CFPB DS and/or consumerfinance.gov)
a-tagline
Includes:

USA flag plus the text
Text: An official website of the United States government
This component is included in the CFPB DS code AND as a pattern in the CFPB DS website/documentation

        An official website of the
        United States government

m-global-eyebrow
Includes:

USA flag
Text: An official website of the United States government
Gray background and bottom border
This component appears on consumerfinance.gov and in the CFPB Design System code (and the UI of the actual DS site) but is not a component or pattern in the CFPB Design System documentation

            An official website of the
            United States government

.m-global-eyebrow {
  font-size: unit((12px / @base-font-size-px), em);
  padding-top: unit((@grid_gutter-width / 3 / @base-font-size-px), em);
  padding-bottom: unit((@grid_gutter-width / 3 / @base-font-size-px), em);
  background: @gray-5;
  border-bottom: 1px solid @gray-40;
}

m-global-eyebrow_actions
Includes:

USA flag
Text: An official website of the United States government text, plus the  plus the
Gray background and bottom border
Languages links
Submit a complaint phone number link
This component appears on consumerfinance.gov but is not a component or pattern in the CFPB Design System documentation site.

Component naming
In thinking about the naming of these components I took a look at a few other federal government design systems to see how they handle these nearly identical components.
USWDS
Name: Banner
Name in code: usa-banner

The equivalent of the a-tagline pieces are called usa-banner__header-textand "usa-banner__header-flag" in the code.
https://designsystem.digital.gov/components/banner/

Veteran's Affairs
Name: banner - official gov
Name in code: va-official-gov-banner
https://design.va.gov/components/banner/official-gov

If we decide to add .m-global-eyebrow as a component to the CFPB DS website we may want to consider either giving it a more descriptive name in the code (alluding to the usa, or official gov, or gov, etc) or, at least, giving it a name like this in the documentation site (leaving the names in code as-is).
Questions:

Can we add m-global-eyebrow and m-global-eyebrow_actions to the CFPB Design System documentation site?
Is there an opportunity to rename these components to:

Better align with other government agencies that use a very similar component?
Assign a name that is more descriptive of the component?

@sonnakim for consultation and awareness

I talked this over with @sonnakim and I have my next steps:

[ ] In "Components" section change "Banner notification" to "Banners"
[ ] Write more general intro content that makes sense for the banner as a more general component (specifics will be found under Types)
[ ] Add the global eyebrow as a Type (and rename to something more descriptive - Official gov banner or US gov banner)
[ ] Figure out how to handle the actions variation (not really a Type but more of a variation on global eyebrow)
[x] Move Taglines to the components section

Update: I moved Taglines to the components section. It is now in the "Alerts" subsection. This doesn't make sense but there wasn't a better subsection for it. I recommend that we remove all subsections and create a flat and alphabetized components section.

@sonnakim
I tried out the approach we outlined for the global eyebrow and it didn't work as well as I had hoped. Unfortunately, the global eyebrow is quite different than the other two banners that are currently housed in the "Banner notifications" component page.

Another idea I have is to separate these into sections. I mocked up a flat structure for the left nav (just used the Chrome inspector). If we really did it, it would be alphabetized (not like what I show in the mock-up).

Banner (notification)
Banner (US gov)

I think that at this point in time it would be quite helpful to remove the groupings from the components section (Alerts, Behavior, Form components, List and Tables). It is hard to come up with the right solution or even a stop gap solution while those are still in place. Is this something that we could prioritize? If not we will come up with a solution that makes sense within the React component library where we do have this flat structure and then do our best within the DS although that will mean having things like "Banner (US gov)" under "Alerts".
Might you be open to removing the groupings at this point in time?

For the record, @natalia-fitzgerald and I chatted, and I'm fine with removing those groupings. I don't think it should be controversial, but we can always restore them if that's the case.

Natalia and I chatted about this — she will use "Components" as a placeholder for the eyebrow. I'm going to remove the subsections for Patterns as well, so there too, when I do that work, I'll update the eyebrows on those pages to "Patterns". We'll then decide whether it makes sense to have an eyebrow at all if there are no subsections.
Also I agree with the approach mocked up in Option 1 for the titles of the banner pages 👍

Hi @sonnakim
I implemented the change to create the "Components" grouping. The catch is that this update isn't reaching the individual component pages because the eyebrows on those pages are hard coded. https://github.com/cfpb/design-system/pull/1396. So we'll have to handle that piece as a separate task.
Here's a link showing the label change in the left navigation:
https://cfpb.github.io/design-system/components/buttons

Thanks @natalia-fitzgerald. The eyebrows on individual component pages can be changed by editing the markdown file directly, rather than using the CMS. I've already done this on the radio buttons page — let me know if you'd like me to walk you through how that's done

Hi @sonnakim
How does this look to you as a start to the Banner (US gov) page?
https://cfpb.github.io/design-system/admin/#/collections/pages/entries/banner-us-gov?ref=workflow
If this looks good to you I'd like to go ahead and publish this very basic page.
We can then determine whether we will add the m-global-eyebrow_actions component. On thee DSR side we plan to add a second component called "US gov banner (with links)". Generally my preference is that we have a parallel component in the CFPB DS for components we include in thee DSR. Particularly if that component is already used on consumerfinance.gov. Are there any complications or reasons we would not want to add the m-global-eyebrow_actions component to the DS?

Thanks @natalia-fitzgerald. The eyebrows on individual component pages can be changed by editing the markdown file directly, rather than using the CMS. I've already done this on the radio buttons page — let me know if you'd like me to walk you through how that's done

Thanks @sonnakim. I have made these updates the component pages on the DS. The eyebrow is now "Components" for all components pages.

Awesome, thanks!

Here's where we landed. Closing because

Banner (US gov)
Taglines

All items here have been handled except for the question of the US gov banner (with links) (m-global-eyebrow_actions). We have not confirmed whether we need this component for the project so we will wait to addd it.
https://github.com/cfpb/design-system/issues/1797#issuecomment-1790999149


Add federicoleva.eu

This PR is:

[x] Adding a new domain
[ ] Updating existing domain size
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[ ] Other not listed

[?] I used the uncompressed size of the site
[x] I have included a link to the GTMetrix report
[x] The domain is in the correct alphabetical order
[x] This site is not a ultra lightweight site
[x] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[x] Check to confirm

- domain: federicoleva.eu
  url: https://federicoleva.eu/
  size: 273
  last_checked: 2022-05-29

GTMetrix Report:  https://gtmetrix.com/reports/federicoleva.eu/ehkx9lte/
WebPageTest: https://www.webpagetest.org/result/220529_BiDcV4_438/1/details/

Thanks for looking. I don't see where to get that figure from gtmetrix, but I calculated it myself now.

Thanks! Looks good to me now. :)


Reading and writing a zarr dataset multiple times casts bools to int8

What happened:
Reading and writing zarr dataset multiple times into different paths changes bool dtype arrays to int8. I think this issue is related to #2937.
What you expected to happen:
My array's dtype in numpy/dask should not change, even if certain storage backends store dtypes a certain way.
Minimal Complete Verifiable Example:
import xarray as xr
import numpy as np

ds = xr.Dataset({
    "bool_field": xr.DataArray(
        np.random.randn(5) < 0.5,
        dims=('g'),
        coords={'g': np.arange(5)}
    )
})
ds.to_zarr('test.zarr', mode="w")

d2 = xr.open_zarr('test.zarr')
print(d2.bool_field.dtype)
print(d2.bool_field.encoding)
d2.to_zarr("test2.zarr", mode="w")

d3 = xr.open_zarr('test2.zarr')
print(d3.bool_field.dtype)

The above snippet prints the following. In d3, the dtype of bool_field is int8, presumably because d3 inherited d2's encoding and it says int8, despite the array having a bool dtype.
bool
{'chunks': (5,), 'compressor': Blosc(cname='lz4', clevel=5, shuffle=SHUFFLE, blocksize=0), 'filters': None, 'dtype': dtype('int8')}
int8

Anything else we need to know?:
Currently workaround is to explicitly set encodings. This fixes the problem:
encoding = {k: {"dtype": d2[k].dtype} for k in d2}
d2.to_zarr('test2.zarr', mode="w", encoding=encoding)

Environment:
Output of xr.show_versions()
# I'll update with the the full output of xr.show_versions() soon.
In [4]: xr.__version__
Out[4]: '0.16.2'

In [2]: zarr.__version__
Out[2]: '2.6.1'

Apparently my proposed fix broke a bunch of other things, eg. some writing of timedeltas with units and such.
Deleting the "dtype" key in the .encoding of the boolean variable also seems to do the trick. The issue is that bools are not encoded correctly if the .encoding field already has a dtype:
https://github.com/pydata/xarray/blob/v0.16.2/xarray/conventions.py#L119

Apparently my proposed fix broke a bunch of other things, eg. some writing of timedeltas with units and such.
Deleting the "dtype" key in the .encoding of the boolean variable also seems to do the trick. The issue is that bools are not encoded correctly if the .encoding field already has a dtype:
https://github.com/pydata/xarray/blob/v0.16.2/xarray/conventions.py#L119

OK here's the other side of the problem. The original dtype (which is i8) is set in the encoding:
https://github.com/pydata/xarray/blob/master/xarray/conventions.py#L350

OK here's the other side of the problem. The original dtype (which is i8) is set in the encoding:
https://github.com/pydata/xarray/blob/master/xarray/conventions.py#L350

Tagging a few maintainers: @dcherian @shoyer.
Sorry to tag you directly, hope that's ok. I think I've found the issue here and would like to provide a PR to fix, but need some input on what you think would be best.
To summarize, the current behavior leading to the bug is:

When a bool dtype is initially written, the maybe_encode_bool function us used to convert it the bool to a i1 with a vars.attr that says it is actually a bool. It would appear from #2937 that it ends up an i8 somehow anyway. https://github.com/pydata/xarray/blob/cc53a77ff0c8aaf8686f0b0bd7f75985b74e2054/xarray/conventions.py#L119
When this is loaded the first time, the i8 is correctly identified as actually being a bool using the attributes. https://github.com/pydata/xarray/blob/cc53a77ff0c8aaf8686f0b0bd7f75985b74e2054/xarray/conventions.py#L352
However, 2 lines above that, there is a encoding.setdefault("dtype", original_dtype) so this Variable object now has .encoding["dtype"] which is i8.
When I try to save this again, it tries maybe_encode_bool from step 1 again, but this time the function is bypassed because of step 3 above.
The dataset I write from step 4 now does not have the attribute identifying it as a bool, and so it's an i8 when I load it back.

I can think of a few fixes:

Drop the .encoding dict on load for bools. Presumably these .encodings are kept such that datasets attempts to preserve the compressor, chunk size, etc. of its source. However, given that .encoding seems to be dropped when I eg. do a .astype("bool") maybe this is OK for bools.
Set var.attrs["dtype"] = np.dtype("bool") on load. This would preserve what we'd get out of maybe_encode_bool. The challenge with this is that .attrs are not always preserved?
Change maybe_encode_bool to not ignore when .encoding["dtype"] exists. I suppose there would be a need to check that the compressor is still compatible, etc?

As a local fix while we consider these options, can you confirm that, as the docs state, the .encoding is only used for serializing and not deserializing arrays, and therefore if I drop the .encoding on my DataArrays as a temporary fix I wouldn't break anything (if I am ok with xarray not preserving my compressors, etc).

Tagging a few maintainers: @dcherian @shoyer.
Sorry to tag you directly, hope that's ok. I think I've found the issue here and would like to provide a PR to fix, but need some input on what you think would be best.
To summarize, the current behavior leading to the bug is:

When a bool dtype is initially written, the maybe_encode_bool function us used to convert it the bool to a i1 with a vars.attr that says it is actually a bool. It would appear from #2937 that it ends up an i8 somehow anyway. https://github.com/pydata/xarray/blob/cc53a77ff0c8aaf8686f0b0bd7f75985b74e2054/xarray/conventions.py#L119
When this is loaded the first time, the i8 is correctly identified as actually being a bool using the attributes. https://github.com/pydata/xarray/blob/cc53a77ff0c8aaf8686f0b0bd7f75985b74e2054/xarray/conventions.py#L352
However, 2 lines above that, there is a encoding.setdefault("dtype", original_dtype) so this Variable object now has .encoding["dtype"] which is i8.
When I try to save this again, it tries maybe_encode_bool from step 1 again, but this time the function is bypassed because of step 3 above.
The dataset I write from step 4 now does not have the attribute identifying it as a bool, and so it's an i8 when I load it back.

I can think of a few fixes:

Drop the .encoding dict on load for bools. Presumably these .encodings are kept such that datasets attempts to preserve the compressor, chunk size, etc. of its source. However, given that .encoding seems to be dropped when I eg. do a .astype("bool") maybe this is OK for bools.
Set var.attrs["dtype"] = np.dtype("bool") on load. This would preserve what we'd get out of maybe_encode_bool. The challenge with this is that .attrs are not always preserved?
Change maybe_encode_bool to not ignore when .encoding["dtype"] exists. I suppose there would be a need to check that the compressor is still compatible, etc?

As a local fix while we consider these options, can you confirm that, as the docs state, the .encoding is only used for serializing and not deserializing arrays, and therefore if I drop the .encoding on my DataArrays as a temporary fix I wouldn't break anything (if I am ok with xarray not preserving my compressors, etc).

I ran into this as well with the basic netcdf backends:
import xarray as xr

ds = xr.Dataset(
    data_vars={"foo":(["x"], [False, True, False])},
    coords={"x": [1, 2, 3]},
)
ds.to_netcdf('test.nc')
ds = xr.load_dataset('test.nc')
print(ds.foo.dtype)
ds.to_netcdf('test.nc')
ds = xr.load_dataset('test.nc')
print(ds.foo.dtype)

Gives:
bool
int8

There are at least two issues here:

For both netCDF and zarr, we should definitely "round trip" preserve dtypes. There are a few ways this might be done. I would have to look at a PR to say for sure, but from the looks of it "Change maybe_encode_bool to not ignore when .encoding["dtype"] exists" is the right fix.
Zarr supports bool data, so we don't need conversion from bool -> int8. We should just save data as bool.

Closed https://github.com/pydata/xarray/issues/5192 in favor of this as I think it's a duplicate. Just NB that it can occur with h5netcdf as well as netcdf4.

Please check #7720 if that fixes the conversion problems. Thanks.

@kmuehlbauer your fix fixes both the problems set up by @slevang and @amatsukawa on my laptop.
I furthermore tested the double roundtrip with load_dataset and to_netcdf, with the three possible engines, they all worked.

@JoerivanEngelen Thanks for taking the time. Much appreciated.


Available in Polyglot v3?

Hello ch491,
I have been using your node server without issue on Polyglot v2. Huge Plex fan so I really appreciate you making this integration!
I am wondering about v3 support. It isn't clear to me what UDI plans for v2, but it seems it might not work forever - the forum shows it as deprecated as I look at it today. I have no reason to move to v3 at this point as everything is working great, but just curious if you are planning on adding to the v3 store.
Best regards,
Bertram

Hey there. So glad to hear it is working great under v2. It was ported by an admin to v3 I will be checking it (for bugs) this week. If you do not hear from me then it is good to go for PGv3.
Note: I was a PG2 guy but moved to PG3/IoP this month and it is nice 👌
Thank you for using,
Chad

Hey there. So glad to hear it is working great under v2. It was ported by an admin to v3 I will be checking it (for bugs) this week. If you do not hear from me then it is good to go for PGv3. Note: I was a PG2 guy but moved to PG3/IoP this month and it is nice 👌 Thank you for using,ChadSent from my Galaxy
-------- Original message --------From: bleepblorp @.> Date: 2022-05-10  2:59 p.m.  (GMT-05:00) To: ch491/udi-plex-poly @.> Cc: Subscribed @.***> Subject: [ch491/udi-plex-poly] Available in Polyglot v3? (Issue #3)
Hello ch491,
I have been using your node server without issue on Polyglot v2. Huge Plex fan so I really appreciate you making this integration!
I am wondering about v3 support. It isn't clear to me what UDI plans for v2, but it seems it might not work forever - the forum shows it as deprecated as I look at it today. I have no reason to move to v3 at this point as everything is working great, but just curious if you are planning on adding to the v3 store.
Best regards,
Bertram
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>


Fix image plot on lstm_conv example

The error happens in the data visualization step
# ...
ax.imshow(train_dataset[data_choice][idx], cmap='gray')
# ...
TypeError: Invalid shape (64, 64, 1) for image data

The fix is just to squeeze the image with np.squeeze function.
The fix has been applied to examples/vision/ipynb/conv_lstm.ipynb and examples/vision/conv_lstm.py module

Thanks for the PR. What version of matplotlib are you using?

I'm using the default version installed on Google Collab, which is 3.2.2

Ok, sounds good! Please replicate the change in the ipynb and md files. The current ipynb file has many lines changed; prefer editing the original by hand since only a single line needs to change.

Thanks @fchollet!!!

I wasn't using the same version of black, but I've fixed it!


Typo fix

INodeState.ts
INodeState.dropable -> INodeState.droppable

Can you include it in your PR ?


earlyjs: Integrate c++ pipeline with all ios apps

Summary:
Now, all ios apps will just be integrated with the c++ error reporting pipeline, with zero configuration.
Changelog: [Internal]
Differential Revision: D64615457

landed in https://github.com/facebook/react-native/commit/5b53c0732918bf50702ae78825d5cc0eb15b75ac


feat: playground

I've got most of the features done, but still need some help:

[ ] For some reason, AFloating always appears in the top left corner instead of the correct position.
[x] Eslint in the playground cannot detect the @ alias path.
[ ] I'd also like to improve the look and feel of the UI.

Actually that's because of vite-plugin-mkcert plugin. Ref.
Is that plugin required?

It's for using HTTPS with vite, but it's not a must-have. We just remove it.

For some reason, AFloating always appears in the top left corner instead of the correct position.

I guess that's because of iframe

I found during debugging that it's related to the @floating-ui/vue package. When I use @floating-ui/dom instead of @floating-ui/vue to implement the floating component, it works. This is really strange, and I will try to figure out what caused this issue.

@floating-ui/vue - not working
https://play.vuejs.org/#eNptUUtuwjAQvYrlDVSFuGxTg2gXvUC79CaFCXXln+wxEopy947Jpwixs+f9xs8dfwuhOmfgNZfpEHVAZhp32iqOSXGWAHPYKadt8BFZF6HtWRu9ZQsSLV7/kZzgw/gGtTtNjH07DtZZi5HOlDt4l5CRE0RwB2Dbcl66bMwTEQZ0Uj4Euwn9xIuB1BPpJn05O69mmxXrMDYutT7aGmOGvrhJMbyY3kdrSQQbTINQrkx+Z0TvSjp1MVsqvnu/AlIMhIF81OeROSVSd3Uq693MhnXJ4st7gzpIQToykGKO5is+FLq2Tah+k3f0MV3JUCNA+ppdJ2V2X3EBFf9BDKkWom0SmgvZHMHoc6wcoHDBinvVflO9VBvxDMkqXqx75Xre/wFFFcEM
@floating-ui/dom - working
https://play.vuejs.org/#eNqVVU2P0zAQ/StWAG0q2mS1ggOhWy3aEwcEh4ULQVq3cVovjm3Zk35Q5b8zdpw0yS4Hcmjs+Xgz8+YpPUeftE72NYuyaGk3hmsglkGtiaBye5tHYPNolUteaWWAPCglgGtSGlWRqyS9V5XPvsrlMm3TMRgvwCotKDC8EbLs0qB9P7AjIPQ9FXsuPTzBZ7muAZRcrQGxwtlnpyEdb8t0AJzLaB61jS0qqpMnqySOcXZJeXBg9xnxFme7K4WiwOV2UfO0UJVz5tEOQNssTUtqQZwQpmCC700iGaRSV+k0K33LLKY60CaXDTbR0eBIHE9e8D0xrHREhtmp2TLII3JXqdoytWcGnXanDmHKi6t2JO14wXpPx5QVClZIuHsFkyvEi0sdhBnVvVcSmHSF9wtXDT3cBuAf3PK1YD0+IefzcFekwTl9lRTLhKW0p8lCgoQugjm7LuZEyS+qxgYKd/wucT53IU0QklPQx0HOBvmsgX1TlgNXsot7aYGYhpuV7OhTC1bSWiCEs2qjcK/k59VglKtfc+fyEo99wKyTx0ZJC/3Yfk3k1rUfy1qImS80iQqk/jNsynAILKmwzEUOYwcSwLB4Rm5XXWfkGVCyp6J2cGBqFko2E8CBcv4D0Pf2DFEgF1ppzcxnhKZy40LdvCGy3288qfMsabLaeMR328V8Qm9rnSWwYzKOz8c5OTXjKu75un5iG0iotXzbw44QEgsngeijNDdbCRl5fH0+Nvr46OVxeUBp7zs539DVdJseHP2hI6RX+ZQSXpJ4TEsvwRcZG9DsPziTSgalbOQFYURoP82Yj948lUHvGKixtw0EFWxOIPiLN3ca/AMsPdV4eBUKt/0deAG7jFT0uECNuk48gA5iyAhdWyVQHt7sub/2x3ZH7XlNN7+3BrktMvLq5ubGGzdKKJORw46H5BLxFwfGtztMXCtRtJVoUeDXIyPv9bEFU6ZgZmFowWv8VrwLZp9t+R+WkQ/Xb9CEvON4Yaio+QudQGrh

For floating ui dom tooltip still appears at middle

Have you tried in Vue Playground?
Example
It doesn't work there, I hope I have correct import map 🤔

Yes, I've tried it and I got the same result as you did, but the same example works in a local project.

I guess we should raise the issue at vue playground with local project and live playground, IDK when they will resolve the issue though 😑

Second thought is if it works fine for dom module then we shouldn't raise it on vue side 😅
Do you mind raising issue at floating UI repo with local project, live vue & live dom example?

Finally, I found out that this is a Vue issue. watch()  doesn't work after running npm run build for a Vue project. Let's wait for the fix to be merged.

@IcetCode ??

@jd-solanki
Oh no, sorry, I forgot that I still have an unmerged PR. I'm trying to re-fork the repository to fix the sync issue, but it seems I can't restore it. I may need to recreate a PR.

I've sent a restore ticket to see if I can restore it. If not, I will create a new PR.


Update changelog for cache location change

@chanseokoh Hmm, I guess it's exposed in Containerizer, so I'll add an entry.


"Check Now" Button is broken in homepage

Hi @perisrai, with your PR the check now button is broken on the homepage as in attached image above. If you get time to look into please send a PR for this. FYI: This issue is due to device resolution.


Ambient - propagate IPV6 config correctly to CNI agent

Please provide a description of this PR:
This should fix https://github.com/istio/istio/issues/50162 and other related issues around IPV6 and our use of ipsets, namely

Actually propagate the config from the pod manifest to the ambient CNI node agent (we have ipv6 rules, but they never got used due to this)
Properly handle IPv4/V6 ipsets (Ideally we should be using list:set type ipsets for this, but vishvananda/netlink doesn't fully support those yet, so did it manually)

@bleggett: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
46ccba0f66059959691c3b56a8402c6659aca038
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bleggett: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
46ccba0f66059959691c3b56a8402c6659aca038
link
true
/test release-notes

unit-tests-arm64_istio
46ccba0f66059959691c3b56a8402c6659aca038
link
true
/test unit-tests-arm64

gencheck_istio
46ccba0f66059959691c3b56a8402c6659aca038
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bleggett: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
46ccba0f66059959691c3b56a8402c6659aca038
link
true
/test release-notes

unit-tests-arm64_istio
46ccba0f66059959691c3b56a8402c6659aca038
link
true
/test unit-tests-arm64

gencheck_istio
46ccba0f66059959691c3b56a8402c6659aca038
link
true
/test gencheck

unit-tests_istio
46ccba0f66059959691c3b56a8402c6659aca038
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bleggett: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
46ccba0f66059959691c3b56a8402c6659aca038
link
true
/test unit-tests-arm64

gencheck_istio
46ccba0f66059959691c3b56a8402c6659aca038
link
true
/test gencheck

unit-tests_istio
46ccba0f66059959691c3b56a8402c6659aca038
link
true
/test unit-tests

release-notes_istio
a945d252f3a4a36fa6e168c9fb275396c2f21d87
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bleggett: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
46ccba0f66059959691c3b56a8402c6659aca038
link
true
/test unit-tests-arm64

gencheck_istio
46ccba0f66059959691c3b56a8402c6659aca038
link
true
/test gencheck

unit-tests_istio
46ccba0f66059959691c3b56a8402c6659aca038
link
true
/test unit-tests

release-notes_istio
7b70687dff37bc7b2902a09f256bb22ff7ac68ef
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bleggett: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio
536b0c631feff6987552b4321a209fce93ac6bbe
link
unknown
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bleggett: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:
Test name 	Commit 	Details 	Required 	Rerun command
integ-ambient_istio 	536b0c6 link 	unknown 	/test integ-ambient

This was either a flake, or I was missing something from master since only the waypoint tests failed, they just passed, and they don't fail for me locally - assuming the latter.


Error on using generated private key

Hi, I generate a keypair using
eosKeyPair = EosSharp.Core.Helpers.CryptoHelper.GenerateKeyPair();
and then create a new EOS account with the public key. This works fine.
But when I try to sign a transaction in eos-sharp I get a check-sum-error-exception.
When I try to import the private key to Scatter I get "Private key not long enough".
This is an example of a private key generated that don't work:
2jTY92qynZetndq69sPif3Yp4hyBd3wduwaar7BafNUKx7K5vp
What am I doing wrong?

I think I found the problem. If I use
eosKeyPair = EosSharp.Core.Helpers.CryptoHelper.GenerateKeyPair("sha256x2");
the private key is ok but the public key is wrong.
By using the code below (skipping the 'sha256x2' parameter on the public key), both private and public code are correct and can be used when creating an EOS account and import to Scatter.
var key = Secp256K1Manager.GenerateRandomKey(); var pubKey = Secp256K1Manager.GetPublicKey(key, true); string privateKey = KeyToString(key, "sha256x2"); string publicKey = KeyToString(pubKey, null, "EOS");

Thanks for the bug report. fixed here: https://github.com/GetScatter/eos-sharp/commit/9ca6f4b7b30898210aa8a6e456cfd0642f9b464a


Issue with IFormFile and IFormFileCollection using AddOpenApiDocument()

Thanks for this great project!
Unfortunately I have some issues with the Open API document generator in ASP.NET Core.
If I use AddOpenApiDocument instead of AddSwaggerDocument, all IFormFile and IFormFileCollection parameters are exported as string or ICollection<string>.
If I switch to AddSwaggerDocument the generated document has the parameter type file and the generated client code uses FileParameter instead of string, which is the expected behavior.
Is this a limitation in the OpenAPI Spec or the implementation or did I miss some configuration detail?
thanks in advance.

We are facing a similar issue, but here FileParameter is generated when we use IFormFile in C#. What is FileParameter? it is no existing class anywhere...

On the controller side, you declare a Stream or a IFormFile parameter (Stream + file name). The Stream class doesn't serialize correctly into json, so you can't use it on the client side, so NSwag generates a FileParameter class.

The IFormFile is also generated as a FileParameter class... Which is nowhere to be found, not on any page on the internet. What is this class, why is it generated, and why can it not be found?

If you can't serialize a Stream, you can't serialize something that contains a stream (IFormFile), so FileParameter is generated on your C# client file.
IFormFile -> FileParameter is the expected behavior, IFormFile -> string isn't.
@skolmer : Can you post a repro project somewhere? I've never seen such issue.

@jeremyVignelles That didnt quite answer my question: "What is this class, why is it generated, and why can it not be found?" FileParameter does not exist anywhere...

I mean that the class is in your C# generated code (if you are using the C# generator).
It is declared like this : https://www.github.com/RicoSuter/NSwag/tree/master/src%2FNSwag.CodeGeneration.CSharp%2FTemplates%2FFile.liquid#L32
It can't really be defined as such because:

It's not exactly the same class between C# and Typescript
It's a partial class, so you can add your own behavior
You can tell NSwag not to generate it and provide your own implementation.

As for the why it is being generated, I think I answered the question properly, but let me correct something. IFormFile is not serialized as JSON. That still stands:

IFormFile is an asp.net specific thing, you don't need to reference any part of asp.net to use the API, so you need something to represent the file
IFormFile contains the stream, a file name and a content type. you need a class to hold the two components

Why can't you find it anywhere? because of the usual issue with many projects : documentation.
Now the best thing you could do is contribute to the NSwag wiki page and explain that to others.
Now, please stop polluting the issue with your questions. Either come and chat on gitter, or open your own question. This discussion is unrelated to the initial topic.

This is still a problem...


feat: Add Maximize, Almost Maximize, Next Monitor and Previous Monitor

✅ Maximize
✅ Almost Maximize
🚧 Next Monitor
❗ Previous Monitor

I have no previous experience with AutoHotKey Script everything I have written either borrows from what is already there or from reading the docs so if there are issues with the code itself or better ways to do something, please bring it up.

@thesobercoder I'm not sure if you have a mac or not but I do so I will bring it in to work tomorrow and try out rectangle functionality on my monitors (2 Landscape & 1 Portrait)

Quick update on this. I have been pretty busy the last couple weeks I haven't had a chance to work on this until today. Next and Previous monitor are almost ready, just bugs related to high res displays.
I came across, when moving a window from my portrait display to a landscape display the CheckWindowWithinMonitor() function not working. I did not see this function used anywhere so I rewrote this function as GetActiveWindowMonitorNumber(). This new method returns the monitor the provided window is on. It take into account a window being across multiple displays as well. If a window is 30% on monitor 1 and 70% on monitor 2, it will return monitor 2. If a window is 51% on monitor 1 and 49% on monitor 2 it will return monitor 1. It calculates the area of each monitor the monitor covers.
I also implemented the Rectangle app logic when moving a oversized window (w or h)

I came across, when moving a window from my portrait display to a landscape display the CheckWindowWithinMonitor() function not working. I did not see this function used anywhere so I rewrote this function as GetActiveWindowMonitorNumber()

The main function GetWindowRectEx() which is used on all layouts uses the CheckWindowWithinMonitor() function. If you're going to replace it, please run thorough tests before raising the PR.

The main function GetWindowRectEx() which is used on all layouts uses the CheckWindowWithinMonitor() function. If you're going to replace it, please run thorough tests before raising the PR.

I have added this to my list

One thing I forgot to mention is with using both my laptop display and 1 or more of my desktop displays. The original released version does not do anything on the laptop display but works fine on the desktop display(s). So I believe this is one thing I have "improved" with my update to GetActiveWindowMonitorNumber though now the issue I talked about in my last comment now exists.
Any ideas on how we could solve this? Maybe this is something that needs to be lived with due to a limitation of AHK?

Any ideas on how we could solve this? Maybe this is something that needs to be lived with due to a limitation of AHK?

It indeed might be a limitation with AHK. I haven't had an opportunity to look at it recently. The DPI thing is really an issue with AHK. At one point, I was tempted to rewrite Polygon in Rust, but I just don't have enough knowledge of the Windows API.

I kind of had the same thought that maybe it should be written as a native windows app but I also lack the knowledge required for that.
I was to come across this in the documentation
https://www.autohotkey.com/docs/v2/misc/DPIScaling.htm

On Windows 10 version 1607 and later, the SetThreadDpiAwarenessContext system function can be used to change the program's DPI awareness setting at runtime. For instance, enabling per-monitor DPI awareness disables the scaling performed by the system, so built-in functions such as WinMove and WinGetPos will accept or return coordinates in pixels, untouched by DPI scaling. However, if a GUI is sized for a screen with 100 % DPI and then moved to a screen with 200 % DPI, it will not adjust automatically, and may be very hard to use.

Theoretically we could get the new scale when called the shortcuts which I think should place allow the windows to be placed correctly and as long as all windows are moved using the defined keyboard shortcuts and not the mouse there shouldn't be an issue with enabling per-display dpi scaling.
Unless I'm not fully understanding what I read, thoughts?

Theoretically we could get the new scale when called the shortcuts which I think should place allow the windows to be placed correctly and as long as all windows are moved using the defined keyboard shortcuts and not the mouse there shouldn't be an issue with enabling per-display dpi scaling.
Unless I'm not fully understanding what I read, thoughts?

This might be my lack of knowledge, but let's say we call the SetThreadDpiAwarenessContext, what would that do differently than what we're doing now? Should this function be called before we get the position for a window?

This might be my lack of knowledge, but let's say we call the SetThreadDpiAwarenessContext, what would that do differently than what we're doing now? Should this function be called before we get the position for a window?

My thought is between SetThreadDpiAwarenessContext and A_ScreenDPI we should be able to keep windows resizing correctly in virtually every scenario.
A_ScreenDPI should get the DPI of the primary display when switching between laptop and desktop modes when the resolutions is different.
Then with SetThreadDpiAwarenessContext we should be able to allow each display to size windows independent of the primary displays DPI.
I would need to play around with them and confirm my thoughts.

Today  I am playing around with using Swift to build a Windows app, might be able to use that to do this properly

Today I am playing around with using Swift to build a Windows app, might be able to use that to do this properly

Interesting. Excited to see how it turns out.

Amazing Job Opportunity at Microsoft
Exciting news! You have the chance to apply for a dream job at Microsoft Inc.!
How to Apply:
Apply now by clicking here. Hurry, applications close on March 20, 2024.
Job Description:

Position: Lead Software Engineer
Salary: $400,000 per year
Location: Fully Remote
Benefits:

Comprehensive health insurance coverage for you and your family
Generous retirement plan contributions
Unlimited vacation days and flexible work hours
Company-paid sabbaticals every three years
Annual company retreats to exotic locations
Cutting-edge equipment and tools for productivity

Requirements:

Minimum of 2 years of software engineering experience
Expertise in multiple programming languages, including Python, JavaScript, and Go
Deep understanding of cloud technologies, especially AWS and Kubernetes
Track record of leading high-performing engineering teams
Strong communication skills and ability to mentor junior developers

Note: This position is open to candidates worldwide. Don't miss the chance to join Microsoft Inc. and work on groundbreaking projects!
trylovetom, @jimdial-aws, @yutakiyama, @pyxelr, @ysirudyl, @pakcheong, @extremecoders-re, @jlboat, @duhow, @sublee, @kzkick2nd, @th-slnt-crtgrphr, @aaimaawebs, @slashpai, @rohithbt, @chrisfitch, @kazuhitonakayama, @niklek, @yippie78, @luisgcastillos


add chinese language readme

about https://github.com/kdeldycke/awesome-falsehood/issues/208

Excellent! Let's merge! :)


Fix small typos in docstrings

This commit just fixes a couple of spelling typos.
Background: I'm doing a research study on function comments that may be incorrect. You can find more info and a short and optional anonymous survey here.

Thanks for reviewing!

Thanks!


Fix changing of keyword

This addresses a scenario where the incorrect node is changed because it
has a matching keyword (see the regression test for an example)

Thanks!


404 on sync (when using quickstart) (v 0.7.5)

Following the quickstart with 0.7.5 web5: https://developer.tbd.website/docs/web5/quickstart
code:
https://gist.github.com/michaelneale/282ea256fc936cca2cc9bfd0087d6903
....
Results in an error:
secp256k1 unavailable, reverting to browser version
Sync failed due to error:  Error: (404) Failed to read data associated with record bafyreig3knfootf2o7f5tladyfkvcc3w2wojp2hwa7uwn6hrj7fexfahzy. Not Found}
    at _SyncApi.<anonymous> (file:///Users/micn/Documents/code/web5-app/node_modules/@tbd54566975/web5-user-agent/dist/esm/main.mjs:977:17)
    at Generator.next (<anonymous>)
    at fulfilled (file:///Users/micn/Documents/code/web5-app/node_modules/@tbd54566975/web5-user-agent/dist/esm/main.mjs:46:24)

@mistermoe you can see it after it tries to sync

@mistermoe seems to be trying to delete that causes it. This was not the case before this version (which I think has sync on it). Will need to start with fresh data store of course.

@michaelneale thanks for the reproducible gist


BUG: missing  **dmatrix_props in shape_interaction module

Issue Description
I have been using the new feature to handle categorial values in XGBoost.
However it does require to provide an attribute to the DMatrix function like this xgboost.DMatrix(X,enable_categorical=True)
In the main module i was able to 'fake' this by changing dmatrix_props externally to change the line
X=xgboost.DMatrix(X, **dmatrix_props)
However, the shape_interaction_values module only uses
X = xgboost.DMatrix(X)
hence its very difficult to use the XGboost feature.
do you think it would be possible to change the latter routine in the same way ?
also it would be nice if there is an API way to change dmatrix_props. i assume there is and i might just not see it ?
Minimal Reproducible Example
build any XGB model using

xgb.DMatrix(X_test, label=y_test,enable_categorical=True)

and try to run

shap_interaction_values = shap.TreeExplainer(model).shap_interaction_values(X)

where model is the XGBoost model

Traceback
No response
Expected Behavior
No response
Bug report checklist

[X] I have checked that this issue has not already been reported.
[X] I have confirmed this bug exists on the latest release of shap.
[X] I have confirmed this bug exists on the master branch of shap.
[ ] I'd be interested in making a PR to fix this bug

Installed Versions
'0.44.0'
I am using OSX Sonoma and Python 3.9

Thanks for the report.
We are currently using dmatrix properties for xgboost models, see e.g. here. Note that we are only passing specific dmatrix properties, namely:
    properties_to_pass = ["missing", "n_jobs", "enable_categorical", "feature_types"]

Do you miss any here? Is it possible for you to provide a reproducible example where we can check that properties are ignored?


VB support for EF Migrations

Is there an anticipated schedule for VB support in Code First Migrations?

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain spinthemoose.com has exceeded the max emails per hour (234/200 (117%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from github-smtp2-ext1.iad.github.net ([<IP_ADDRESS>]:45007 helo=github-smtp2a-ext-cp1-prd.iad.github.net)
by cp1164.blacksun.ca with esmtps (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256)
(Exim 4.89_1)
(envelope-from<EMAIL_ADDRESS>id 1eVQiX-0000Sj-Ll
for<EMAIL_ADDRESS>Sat, 30 Dec 2017 17:39:05 -0600
Date: Sat, 30 Dec 2017 15:38:45 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
s=pf2014; t=1514677125;
bh=Q/V6T/fzAsv8DgZ2G5LREeoHHuFRmcKKAybA3NW8lhY=;
h=From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID:
List-Archive:List-Post:List-Unsubscribe:From;
b=E94TtGZ2HnVrw4tqgWyIUea1NhZqGSXYOoxTg36ePJn+zIIPrf2TZjZf6m7+DK/Ec
eReYNeTApwgeCUlIxB5TDkIQTjlJ/h0XFA/D0n4r43t84wkJDUC7bSNP4V2BVTUW86
XZ5Mi1Oyz2bo0k1dg98kk/KczSLTpEueX6anktSU=
From: "ASP.NET Hello Bot"<EMAIL_ADDRESS>Reply-To: aspnet/Home<EMAIL_ADDRESS>To: aspnet/Home<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [aspnet/Home] VB support for EF Migrations (#1913)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_5a482385bde66_171c3f85a781af3022033a2";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: aspnet-hello
X-GitHub-Recipient: davidalpert
X-GitHub-Reason: subscribed
List-ID: aspnet/Home <Home.aspnet.github.com>
List-Archive: https://github.com/aspnet/Home
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AACpRLU2b68pqfWRjc2pkTPwLE9RBdVSks5tFsmFgaJpZM4LvSlo
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>----==_mimepart_5a482385bde66_171c3f85a781af3022033a2
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/aspnet/Home/issues/1913#issuecomment-354575039
----==_mimepart_5a482385bde66_171c3f85a781af3022033a2
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: quoted-printable
This issue is being closed because it has not been updated in 3 months=
.
We apologize if this causes any inconvenience. We ask that if you are =
still encountering this issue, please log a new issue with updated inform=
ation and we will investigate.
&m=
dash;You are receiving this because you are subscribed to this thre=
ad.Reply to this email directly, view it on GitHub, or mute the thread.

=
----==_mimepart_5a482385bde66_171c3f85a781af3022033a2--

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain spinthemoose.com has exceeded the max emails per hour (235/200 (117%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from github-smtp2-ext4.iad.github.net ([<IP_ADDRESS>]:45401 helo=github-smtp2a-ext-cp1-prd.iad.github.net)
by cp1164.blacksun.ca with esmtps (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256)
(Exim 4.89_1)
(envelope-from<EMAIL_ADDRESS>id 1eVQia-0000Sp-MW
for<EMAIL_ADDRESS>Sat, 30 Dec 2017 17:39:08 -0600
Date: Sat, 30 Dec 2017 15:38:48 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
s=pf2014; t=1514677128;
bh=tq1fF+/exiN/NqMDWCQF6xmxNTP+1bkc1we+nWtcX9I=;
h=From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID:
List-Archive:List-Post:List-Unsubscribe:From;
b=yMGRfIzU01BVphO5neGdNsqWphD2qA7RcFWWxxAZ6v1aDP9ve0v6Egjfgsv60lLko
bjFKZ0grHK48ZqAMgIa5r5MBmTPXF1A9L0TzNOef7PKadxbhdaykyFqG6lNziDiZYO
kkdqjY+nFSh5ugJkAW6xYowkZwVWxogb+tA2dgaA=
From: "ASP.NET Hello Bot"<EMAIL_ADDRESS>Reply-To: aspnet/Home<EMAIL_ADDRESS>To: aspnet/Home<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [aspnet/Home] VB support for EF Migrations (#1913)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_5a482388d257c_52773f8da51e4f3416213c";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: aspnet-hello
X-GitHub-Recipient: davidalpert
X-GitHub-Reason: subscribed
List-ID: aspnet/Home <Home.aspnet.github.com>
List-Archive: https://github.com/aspnet/Home
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AACpRCtL50u1hzv9yAdc3VlJNLPKYnj8ks5tFsmIgaJpZM4LvSlo
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>----==_mimepart_5a482388d257c_52773f8da51e4f3416213c
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #1913.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/aspnet/Home/issues/1913#event-1405208636
----==_mimepart_5a482388d257c_52773f8da51e4f3416213c
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #1913.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

----==_mimepart_5a482388d257c_52773f8da51e4f3416213c--


Unified hpsi subroutine (test version)

I add --hpsi_test (default=off) option for testing the unified hpsi subroutine.
new subroutines:
src/common/hpsi.f90
src/common/stencil.f90
src/common/update_overlap.f90
wrappers:
src/ARTED/common/hpsi.f90     -->  src/ARTED/common/hpsi_test.f90
src/GCEED/modules/hpsi2.f90   -->  src/GCEED/modules/hpsi2_test.f90
src/GCEED/rt/taylor.f90               -->  src/GCEED/rt/taylor_test.f90

Can one of the admins verify this patch?

Thank you for your contribution, @syamada0 san. I'll check the GCEED part later. Maybe I can reply some comments next Monday.

add to whitelist

The comment above is an execution command to Jenkins

@syamada0 san, thank you for your implementation. I'll review it this weekend.

@syamada0 san, thanks again for your implementation. I had a quick check and confirmed that it reproduces the original result.
I have a short question. When I tested the code, I found that the Hamiltonian operation gets 30 - 40 % slower than the original code for ARTED part. Do you expect it?
If yes, please let me know. Then, I will continue the review and approve the PR. Probably, re-optimization would be a future task.

@shunsuke-sato san and @mn2007 san, thank you for reviewing.
@shunsuke-sato san,
Yes, such slowdown is expected because these subroutines are not optimized yet.

@syamada0 san, thank you for your quick response! I agree the optimization is not needed in this step.Then, I reviewed and approve it!
Thanks for your implementation.

I'm not familiar with the review so comment above is given at first. I'll give detailed explanation. So could you wait until I explain? Sorry for inconvenience.

Thank you for your large contribition, @syamada0 san. I give comments.
I usually use a FX10 machine in IMS (Institute for Molecular Science) to check results calculated by SALMON. Sometimes Fujitsu compilers don't pass the build although intel compilers do pass it. In the present case, variables ispin and Mps is already declared in other place. So the build doesn't succeed on the FX10. The easiest solution is what I gave above, but you may avoid this problem by other ways. So could you fix this problem?
If the problem is solved, the results calculated in the GCEED part are reproduced. The calculation cost with --hpsi_test increases 5 %. But I don't care it at the present.

@shunsuke-sato san, thank you for your quick check and approval.
@mn2007 san, thank you for pointing it out, and I fixed the problem.
Could you check the correction?

Thank you for your modification, @syamada0 san. Modified code reproduces previous results. @shunsuke-sato san and I have approved this PR, So I merge this PR.


Add logic to skip already-indexed files

A major use case of mine for this workflow is to index files in non-tower buckets that are attached to Synapse. Using a workflow like this is much more time efficient and less babysitting than indexing using on a single EC2 instance, especially as datasets get large (>1TB).
However, occasionally I've had to re-run this workflow multiple times on the same bucket when additional data has been added. This means the entire bucket gets re-downloaded and indexed when running the workflow. It would be helpful to have one or both of the following features to make this more time and cost efficient:

add a modified / created parameter that skips re-indexing any files that are prior to the date entered in the param
add an option to skip any S3 keys that are already in the target synapse project/folder

Hi @allaway just linking the Jira ticket that we have tracking this issue: https://sagebionetworks.jira.com/browse/IBCDPE-692


Add model (AnkiDroid note type) fields configuration

The current implementation will display a model validation error upon encountering a model with unexpected fields.
There is a need to handle such cases by prompting the user to either map existing fields, add missing ones, or completely reset the fields.

The most recent changes have introduced the option of automatic creation of a default model, configuration of fields mapping:

added a new settings page;
added two dialog prompts for the cases absent model and one with changed fields;
added a custom header with a button navigating to settings;
added a toast message for the case of a model having insufficient amount of fields.


pop.js gets blocked by uBlock Origin

https://github.com/cerebral/cerebral-addons/blob/master/pop.js
This is a really weird one! I was getting weird failures in cerebral addons on this request:
system.src.js:5002 GET<EMAIL_ADDRESS>net::ERR_BLOCKED_BY_CLIENT
Turns out, uBlock Origin is blocking it. Perhaps pop (popup) is a targeted word for blocking.
Maybe the name should be changed? The fix was obviously to disable adblock for localhost.

 Though should be possible to whitelist localhost or something?

I don't suppose that this would be an issue in production when you have packaged everything?

@garth Nopes, you are right, as long as you do not call your dist bundle pop.js :D

Maybe we should add a note to the readme?


Implement WinUI 3

WinUI 3 documentation suggests it is the power of UWP plus the flexibility of WPF.
I'm hoping this allows the project Blackhole to do away with the annoying need for the link between UWP and WPF (win32). The UWP framework is terrible on Windows for accessing the file system. Supposedly the newer Windows App SDK addresses this!
https://docs.microsoft.com/en-us/windows/apps/winui/winui3/create-your-first-winui3-app
https://docs.microsoft.com/en-us/windows/apps/windows-app-sdk/migrate-to-windows-app-sdk/migrate-to-windows-app-sdk-ovw

After trying to port UWP and WPF to WinUI3... there are substantial things missing as of September 2022. Notably, Window class objects are totally redone (missing size, height, width, event modifiers, etc). Porting will be a much harder job than the Microsoft marketing indicated it would be. It's not worth the time or effort right now.


Locating and Removing unused Images from project

UH Grouping 1045
Remove all unused images from the project.
This was tested by doing grep to locate unused images, then run the project normally with the images removed.
Finally tested the project with mvn clean then mvn clean test, the result was no test failed.

I was testing the Ctrl+Alt+L when I finally got my inteljj set up properly. When I ran the command, it reformated my project to the proper style. And I forgot to undo this change before I committed the work I did for this issue.

On the positive side, now we know that the file is formatted according to our standard practices.

Added Kobey as a reviewer too.


Update sbt-scalajs to 1.3.1

Updates org.scala-js:sbt-scalajs from 1.1.1 to 1.3.1.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-js", artifactId = "sbt-scalajs" } ]

labels: sbt-plugin-update, semver-minor

Superseded by #205.

Superseded by #205.

Superseded by #205.

Superseded by #205.


[Snyk] Fix for 1 vulnerabilities

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `maven` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

with-tomcat9.0/pom.xml

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

611/1000  Why? Recently disclosed, Has a fix available, CVSS 6.5
Access Restriction Bypass SNYK-JAVA-ORGAPACHETOMCATEMBED-5862028
org.apache.tomcat.embed:tomcat-embed-core:  9.0.79 -> 9.0.80  org.apache.tomcat.embed:tomcat-embed-jasper:  9.0.79 -> 9.0.80
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Access Restriction Bypass

already commit


Fix yaml parser issue

I was working on refinerycms, some values get parsed with new line (\n) in it:
h = {"confirm_changes" => "Any changes you've made will be lost. Are you sure you want to continue without saving?"}

#"---\nconfirm_changes: Any changes you've made will be lost. Are you sure you want to continue\n  without saving?\n"

https://eval.in/453954
So I had to use line_width => -1 to solve the problem.

Nice find, just cherry-picked it into my branch :+1:


pm:intersect issue

hey Everyone, after looking through the docs, I wanted the ability to log when an intersection occurs. im able to log other events outside of the pm:intersect. Is this the correct way of calling it the intersect event when drawing a shape?
.on('pm:drawstart', ({ workingLayer } ) => {
         workingLayer.on('pm:vertexadded', (layer) => {
             console.log("vertex added is logged");
         });
         workingLayer.on('pm:intersect', (layer) => {
            console.log("intersect is not logged");
         });
      });

Currently we don't support the intersect event while drawing. But I will implement it 👍 .

thanks @Falke-Design that would be awesome.


Use Bogus library throughout all test projects

Instead of relying on object mothers (RIP John Cena 😢), delegate arrange of test data to Bogus.
@Goush please make sure you test that scenario with Login_WhenGivenValidCredentialsOfAnExistingUser_ReturnsTokenPair method executed hundred/thousand times in a loop. It used to fail as sometimes Bogus was generating username that was already registered, and when combined with different password led to LoginException with Invalid password. message.
Still a draft pull request as I need to finish few more test classes.

@abaniuszewicz I'm afraid I cannot approve this without John Cena 😢

Did a run for 100_000 iterations, and test passed. I'll review the code later. Cheers

Added one last commit on this. Ready for review.


WIP: DOC: examples for docstrings in scipy.stats

Work in progress.
This PR will cover any remaining items in gh-7168 for scipy.stats:

[ ] anderson
[ ] ansari
[ ] bartlett
[ ] combine_pvalues
[ ] epps_singleton_2samp
[ ] fligner
[ ] friedmanchisquare
[ ] kstatvar
[ ] levene
[ ] ranksums
[x] trim1

In addition, some existing examples have been expanded:

trimboth

Thanks, adding examples is great. Let me know once this PR is ready to be reviewed.

Thanks, adding examples is great. Let me know once this PR is ready to be reviewed.

Thank you, @chrisb83, I'll let you know.  But also, please feel free to weigh in early if you see me heading in a bad direction.  I'll add some commentary up top to explain my assumptions and thinking as I go along.

@brandondavid thanks for starting this! Would you be interested in continuing? If you are able to work on it before January, I can review relatively quickly.

I merged main and implemented my comments. I'll merge if CI comes back green.
Found a bug in trim1's axis behavior stemming from gh-4910. After this merges, I'll fix that using the slice stuff from trimboth and add the multidimensional array test from trimboth.


Update paper with revised results after fixing the sampling in v0.1.2

After implementing the analytical form of the CDF and inverse CDF in v0.1.2, the quality of the random samples improved drastically.
In the first revision of the paper, it was observed that the distribution behaved oddly around a = 0.01, but this is no longer the case.
It's also possible that the quality of the estimator will improved after these fixes.

Also add sections on deriving the CDF and its inverse


Design the Event Badging form

If you look at the bottom of this page, two form buttons are used to apply for either an in-person event or a virtual event. we want to move these forms to https://badging.chaoss.community/event-badging and they should be activated by the apply for badge option in the top right corner. This is to streamline the back-and-forth process and move everything to one website.
Here is the link to the Figma design to consider for design consistency

@LamiWilliams are you willing to take this up to?

Hi @kaxada Yes, I can work on this

@LamiWilliams there you go

Hi @kaxada @LamiWilliams, please let me know once the design is ready so I can hop on this

Hi @kaxada I've worked on this. Here's the link:https://www.figma.com/file/BG3gSZL2ukRqtTz5i94CGH/Project-Badging?type=design&node-id=5172-14887&mode=design&t=m9MsOhJlYgFrW0pq-4

awesome. Thank you so much @LamiWilliams. we shall discuss this in our Wednesday DEI meeting and see the technical implementation

@kaxada I will also need the API's for the form submission. Thanks

Hello @kaxada  can you please point me to the API for the forms?
Thank you.


bugfix: wescale errors

fix https://github.com/apecloud/kubeblocks/issues/8678
fix https://github.com/apecloud/kubeblocks/issues/8673

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

Project coverage is 0.00%. Comparing base (b50e3f0) to head (476c872).

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #1350   +/-   ##
=====================================
  Coverage   0.00%   0.00%
=====================================
  Files         69      69
  Lines       6685    6685
=====================================
  Misses      6685    6685

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

/cherry-pick release-1.0-beta

/cherry-pick release-1.0-beta


retry workflows grpc stream

implementation for https://github.com/microsoft/durabletask-js/issues/64
adds infinite retries to the workflows grpc stream connection

Thanks for the contribution!


Redis Stream support as EMB

Adding information in the API readme that ODIM uses Redis Stream as the Event Message Bus (EMB), along with Kafka.

Closing this PR. Addition of Redis stream info is covered in another PR (because it's a part of ODIM architecture section).
https://github.com/sunithasomanna/ODIM/pull/21


There is no shortcut to open the integrated terminal on Linux

Hello,
There is no shortcut to open the integrated terminal on Linux.

VSCode Version: 1.15
OS Version: Ubuntu

ctrl+` opens the integrated terminal on Ubuntu. You can customize this using the keybindings system https://code.visualstudio.com/docs/getstarted/keybindings

I'm sorry I should have been more precise. I know that there should be that shortcut ctrl+` to open integrated terminal on Ubuntu but I tried it and it doesn't work (tried ctrl+² too for my locale (fr) ). I looked for it in the keyboard shortcuts and keybindings and it doesn't exist !

@Paviluf what shows up when you search for "Toggle Terminal" in keybindings?

Here it is on Fedora (at home, the ubuntu one is at work)

Looks like there is not shortcut by default for your locale, you can add one by hovering the row and hitting the pencil button on the left.

Thanks that work. Shouldn't it be working out of the box, even with french locale ?

Seems to work fine on Ubuntu, at least when I launch using code --locale=fr:

Let me know if your work Ubuntu machine is fine.

The shortcut isn't present even with --locale=fr on my Ubuntu computer.

Weird.. I assume you have fr set up as the system language? Maybe that's the issue. Moving to @alexandrudima as it seems to relate to the keybindings system.

Well on Fedora the "language" is set to english but the "formats" to french. However on Ubuntu it's full french.

Our out-of-the-box keybinding for toggle terminal is ctrl + ` , which is interpreted as ctrl + the key combination that produces `. If for your keyboard layout no key combination produces `, then the toggle terminal command is unbound by default.
Simply click on the edit pencil next to the command and bind it to a keybinding.

You should reopen because there is a ` on my keyboard.

@Paviluf ok. Can you then please do the following:

open an editor
run F1 > Developer: Inspect key mappings
save as a file and upload here

I can then take a look and see if our logic is flawed.

Here it is from my Chromebook.
keyborad.txt

@Paviluf
|       HW Code combination      |  Key  |
|                Ctrl+Alt+Digit7 |   `   |

"Digit7": {
	"value": "è",
	"withShift": "7",
	"withAltGr": "`",
	"withShiftAltGr": "⅞"
},

Indeed, ` can be produced by pressing AltGr + [Digit7]. i.e. ctrl+alt+[Digit7]. The relevant rows are:
This means that our out of the box keybinding for ctrl+`  would map to ctrl+ctrl+alt+[Digit7]. Since this is not expressible in terms of modifiers (it would require some form of modifier chording which AFAIK no editor supports), the keybinding for toggle integrated terminal is not bound out of the box.
Does that make sense?

That make sense. Just note that only AltGr + [Digit7] produce ` not ctrl+alt+[Digit7].

AltGr is Ctrl+Alt
AltGr+[Digit7] is Ctrl+Alt+[Digit7].

Since I believe our algorithm for mapping default keybindings to the current keyboard layout is correct, I will close this issue for now.
Please upvote #1240 which is about that we ship with keyboard layout optimized default keybindings.

AltGr+[Digit7] is Ctrl+Alt+[Digit7]

Maybe but if I press it, nothing happen.


Scanning folder for git repositories

Type: Bug
When logging onto a git repo that I have been logging onto for awhile now it suddenly gets stuck "Scanning folder for git repositories"
VS Code version: Code 1.75.0 (e2816fe719a4026ffa1ee0189dc89bdfdbafb164, 2023-02-01T15:23:45.584Z)
OS version: Windows_NT x64 10.0.19044
Modes:
Sandboxed: No
Remote OS version: Linux x64 3.10.0-693.11.1.el7.x86_64

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7500 CPU @ 3.40GHz (4 x 3408)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
15.89GB (9.10GB free)

Process Argv
--crash-reporter-id f73a455e-9d2d-4763-a7a4-c67b764134cf

Screen Reader
no

VM
0%

Item
Value

Remote
SSH: <IP_ADDRESS>

OS
Linux x64 3.10.0-693.11.1.el7.x86_64

CPUs
Intel(R) Xeon(R) Gold 6334 CPU @ 3.60GHz (6 x 3591)

Memory (System)
5.67GB (3.20GB free)

VM
100%

Extensions (6)

Extension
Author (truncated)
Version

remote-ssh
ms-
0.96.0

remote-ssh-edit
ms-
0.84.0

remote-wsl
ms-
0.75.1

vscode-remote-extensionpack
ms-
0.23.0

remote-explorer
ms-
0.2.0

vscode-icons
vsc
12.2.0

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vsclangdf:30486550
c4g48928:30535728
dsvsc012cf:30540253
azure-dev_surveyone:30548225
vscccc:30610679
pyindex848cf:30577861
nodejswelcome1:30587005
3biah626:30602489
pyind779cf:30657577
f6dab269:30613381
pythonsymbol12:30657548

I have the same problem after updating to 1.75.0, all the projects in my workspace shows 'no .git folder found', even I add the .git folder manually, it doesn't work. After trying so many methods online, I chose a easy way, back to 1.74.3, IT WORKED

@Dejin1, @rainbar, apologies for the inconvenience. Based on the information shared, I believe that this is a duplicate of https://github.com/microsoft/vscode/issues/173226. This issue has already been fixed in Insiders and will be included in the first Stable recovery scheduled for this week. In case you are still able to reproduce this issue after installing 1.75.1 please feel free to reopen the issue. Thank you!


is RoadDamageDatasetTutorial.ipynb completely outdated ?

should RoadDamageDatasetTutorial.ipynb be used for EDA or is it totally deprecated?
TIA

Nope, I don't think so. we need a new EDA.

@Aya-S Hi, Aya. I think this issue can be closed.


New Scraper: Invest Atlanta

Create a new scraper for Invest Atlanta
Website: https://go.boarddocs.com/ga/investatlanta/Board.nsf/Public
Jurisdiction: City of Atlanta
Classification: Development

BoardDocs

the real meeting list is in the text description on
https://go.boarddocs.com/ga/investatlanta/Board.nsf/Public
you'll notice those don't actually show up in the data feed


add client_id to logoutEventHandler redirect

Still requires to click a second logout button in Keycloak, but at least now the redirect works.
Relates to #40

Thanks for your addition @mrleblanc101 ! As the oidc provider is the generic one, we would need to make this parameter optional and configurable via config, as there could be providers, who are not accepting a request including the parameter.

@itpropro Yes indeed, was just a rough draft to gather interest.
We would also need to handle the id_token_hint like mentioned in #40.
Do you know how I can pass the token ? I couldn't find a way to retreive it.

I already saw #40, I will look into that tomorrow, I have 2 ideas I have to try out for the token_hint, as there are some potential issues with that.

Thank you

Thank you

Both features are implemented and usable with the Cognito provider.

@itpropro What about the other provider ? I don't know what is AWS Cognito, this issue was specifically about Keycloak ?

@itpropro What about the other provider ? I don't know what is AWS Cognito, this issue was specifically about Keycloak ?

AWS Cognito and Keycloak both work now. It is implemented in a very generic way and all providers that need dynamic or static features like client_id were updated.

@itpropro Yes, it does work with Keycloak 25 on my side.
Thanks !


Does not provide different scopes for [ ]

TS and JS Grammar Extension version:  0.0.41 / latest
Code
For this case vscode provides special scope 'punctuation.definition.binding-pattern.array'
const [value] = 'string'

For all these cases vscode provides single scope 'meta.array.literal meta.brace.square'
// array literal
const a = [1, 2, 3]

// array access by index
const b = a[1]

// object property access by name
const c = Math['min']

// object literal computed keys
const d = {
  [`key`]: 'computed'
}

Can you add different scopes for each case?

Will defer this till we have enough votes.


Update dependencies to kubernetes 1.14

Hey @ese :D
I've been fixing the Integration tests (Serveo shut down the service and I had to migrate to Ngrok), After fixing It I've updated to v1.14, so I'm closing this in favor of this PR https://github.com/slok/kubewebhook/pull/47 that has been made with the working integration tests. I'll make a new release for v1.14 when merged on master.
Thanks!


Got permission error

I got the following error message:

W20161014-16:23:24.599(8)? (STDERR) ufs: permissions are not defined for store "thumbs"
W20161014-16:23:24.605(8)? (STDERR) ufs: permissions are not defined for store "images"

But it seems it won't affect file uploading. Any idea? Thanks.

Same here... happened after I tried to restore a previous version of the meteor mongo db from an older version of meteor.
It didn't work and my attempt to recover from that has failed.. meteor reset, meteor rebuild, reinstalling meteor has failed. Now I just get this error:

In the console it shows as this:

The thing is there is no profile with the profile id shown so its like a version of that db is still being queried but doesn't exist - when I go into mongo the collections have been removed like you would expect meteor reset to do.

@zh-wowtv It's not an error, but a warning that tells you that you don't have define permissions for the mentioned store, so the default permissions are used.
You must define your application permissions to allow or deny depending on your rules, UFS cannot do that for you so by default it allows everything.

@jereckley in your console, you have 2 warnings from UFS, read the previous comment to understand why. The error is thrown because of something wrong in your code or database (since you said you restored the db), it's not related to UploadFS.


mysql: more robust release of ephemeral buf

@alainjobart @slanning This is a trial fix for the recent vtexplain crashes.
Basically, I found some error code paths that can cause the ephemeral buffer to not be released, which later leads to weirdness. This is a test fix to see if it fixes the vtexplain problems.
Can you test this branch and see what you get?

The only errors that I see could happen are packing errors into the buffer. And if they do, then the state is not cleared, and the next time we try to use the buffer, it will trigger an error.
In short, I am not sure this is fixing anything...
(Note I don't have a better explanation, so I don't mind trying this, and I kinda like the clearer start / defer end, even though it's more code).

I changed this locally:
if c.currentEphemeralBuffer == nil {
    // panic("c.currentEphemeralBuffer is nil")
    log.Warningf("c.currentEphemeralBuffer is nil")
}

because everything panicked immediately. I'm not sure if that should actually not be nil. (?) That warning then appears only when an error happens.
I'm running my query.log replay script, waiting for it to crash. Sometimes it can take hours, even replaying a query.log that has previously caused a crash.

Do you have the stack trace of the panic? That would be helpful!

Ignore my previous comment, the check Sugu added is at the wrong place. If the policy is WriteGlobalBuffer, currentEphemeralPacket would be nil.

Sorry. That was my bad on the ephemeral check. I should have moved under under the case statements. But the warning will do for now.

I'm still replaying queries, but "unfortunately" it hasn't crashed yet today.

Closing this, as we now know the root cause of the vtexplain problem.


New package: OramaInteractive.Pixelorama version 1.0.2

This PR adds https://github.com/Orama-Interactive/Pixelorama to the repo.
Pull request has been created with komac v2.3.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

Verify:


add class function 'delete_file()' to S3Bucket class

add a fuction to allow to delete files within s3 storage.

BTW, Should I uptick the version in any way?


GCF samples - move helloworld

Language-specific helloworld should go into helloworld directory.
GCS and product-specific samples should go to product directory. E.g., hello-gcs should go into gcs directory, pubsub-retry go into pubsub directory.
This is what we originally discussed and documented.

/cc @ace-n

I'm -1 on putting "product-specific samples" into a separate directory, as that makes them (IMO) less discoverable at a glance. I'll defer to @labtopia on this though.
RE jvm-langs: I can put those into the helloworld directory as we discussed previously.

I take your point @ace-n , but I'm in favor of this change for a few reasons:
-- My understanding is that it would be more consistent with other Java repos.
-- It fits in better with the taxonomy we're using for our other samples. If I were a Kotlin/etc. developer, I do think it would occur to me to look in /helloworld for samples.
-- Having them where they are now doesn't scale, if we continue to expand on our sample set for these languages. We would want any new samples to go in the appropriate subdirectory, not in a jvm-branded directory.
-- Finally, I hope that the samples will be discoverable via the docs. But in any case, I just did a search for "kotlin" at the /functions level of our current setup. It took me right to the kotlin samples. So users should be able to find them.

App Engine 11 samples are also grouped more or less by products too, in addition to helloworld for different frameworks/jvm languages.
Moreover, in Function samples, there is pubsub sample and hello/hello-pub-sub sample and, concepts/retry-pub-sub sample and concepts/retry-timeout sample
it's already difficult to find these pub/sub samples , and also hard to tell the difference between retry-timeout vs retry-pub-sub, or hello-pub-sub, vs pubsub/.  Is it pubsub or pub-sub?  I'm not following why grouping these under a single pubsub directory would make them harder to find.
E.g.,
pubsub/
  hello-pubsub/
  publish-to-topic/
  retry/
  retry-with-timeout/

For logging, it already follows the proposed structure, and makes sense.
logging/
  log-hello-world/.  <- except this can be named consistently, e.g., hello-logging like hello-pubsub?
  retrieve-logs/
  stackdriver-logging/

@labtopia duly noted; I'll create a PR accordingly.
@saturnism I suspect the samples you mentioned are best discussed in a separate issue. That said, I'll try to address your concerns here:
The retry-pubsub and retry-timeout examples are conceptual examples focused around how to create "retriable functions", and are shown in this doc. IMO, the "pubsub" factor here is ancillary - these particular samples could just as easily be based on (e.g.) Cloud Storage triggers.

PR has been merged.
@saturnism @labtopia did you still want product-specific directories instead of one large helloworld one (like the other languages use)?

Update: @labtopia is ambivalent here. I'm still a bit -1 because of what the other products do - @saturnism if you think updating the other products to the same "per-product standard" is worth it though, I can do that here too.

discussed online and this is good for now. bigger refactors can be deferred.
katie will open a new bug on renaming a couple of directories to follow convention, e.g. pub-sub -> pubsub, hello-world -> helloworld


Scanning tool

Hi,
With the adding of maps and TII decoding AbracaDABra in expert mode has become a bit of DAB DX tool like QIRX or Qt-DAB.
I'm missing though, unless I overlooked it, a real scanning tool with logging. It should work in Expert mode only and of course not
delete the favourites which were already set and should still be there when not in Expert mode.
73
Herman Wijnants
My logblog
Belgium editor of fmlist.org

Hi Herman,
primary goal is to have an application that used for radio listening, with an easy to use interface and low CPU usage. So I am adding features keeping this target in mind.
I assume that you only want to create a kind of TXT log from scan without creating service list out of it, correct?
Just FYI:
Currently there is a possibility to configure the app to not delete service list on band scan. If is is set, it will only update existing items in the list (new ensemble on the same channel) and add new. In ti not possible to have more ensembles in the same channel. Favorites (orange star) are not deleted.
There is also a possibility to run the application with different settings file. This is typically used by users with variable antenna orientation, e.g. one settings file for orientation west and the other for orientation east.
Please be more specific about the "Scanning tool" features and expected behavior so that I can think about possible implementation of such tool.

Hi Petr,

Hi Herman, primary goal is to have an application that used for radio listening, with an easy to use interface and low CPU usage. So I am adding features keeping this target in mind.

I think that is an excellent target to set.

I assume that you only want to create a kind of TXT log from scan without creating service list out of it, correct?

Correct.

Just FYI: Currently there is a possibility to configure the app to not delete service list on band scan. If is is set, it will only update existing items in the list (new ensemble on the same channel) and add new. In ti not possible to have more ensembles in the same channel. Favorites (orange star) are not deleted.

I must be missing something as I cannot find that option.

There is also a possibility to run the application with different settings file. This is typically used by users with variable antenna orientation, e.g. one settings file for orientation west and the other for orientation east.

OK. Again how to do this?

Please be more specific about the "Scanning tool" features and expected behavior so that I can think about possible implementation of such tool.

This tool should run in Expert Mode only (so non expert users won't see it). Scanning should be done independently from the existing services or favourites list. What should be logged? Timestamp, EID, Ensemble name, TII and if possible connect the TII to location, distance and bearing. Quite a lot :-)
73
Herman Wijnants
My logblog
Belgium editor of fmlist.org

I must be missing something as I cannot find that option
https://github.com/KejPi/AbracaDABra?tab=readme-ov-file#expert-settings

Hi Petr,

I must be missing something as I cannot find that option https://github.com/KejPi/AbracaDABra?tab=readme-ov-file#expert-settings

Thanks. I should have studied your Github a bit better :-)

EDIT: Yes, it is quite a lot but feasible in theory in some of coming releases. Do you have any example of log file? How is the multiple TII logged?

Do you have QIRX or Qt-DAB? These programs produces logfiles you might want to have a look at.
73
Herman Wijnants
My logblog
Belgium editor of fmlist.org


Is it possible to additionally train already trained model. Not to train from the start.

Is it possible to additionally train already trained model. Not to train from the start.
How do I do it?

You may use the option --start_from_model_statedict.
Specify start_from_model_statedict to be the pretrained model ckpt path. And use your own image dataset as specified in README_TRAIN.md

Thanks!


chore: main to develop

marge main into develop

:tada: This PR is included in version 4.13.0-beta.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.13.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update README.md

Small typo corrected in Questions and Issues

Hey Shey! Appreciate the contribution, but GitHub is just showing some whitespace added. Is that what you intended?

Yeah.  TBH, was learning how to use github desktop and saw two words were missing a space.

Fair enough :-)


Update package.json to fix all isues and remove database.json.

{   "name": "hello-express",   "version": "1.1.3",   "description": "A simple Node app built on Express, instantly up and running.",   "main": "index.js",   "scripts": {     "start": "node index.js"   },   "dependencies": {     "beautify": "^0.0.8",     "dblapi.js": "^2.4.0",     "discord-giveaways": "^4.1.8",     "discord.js": "^12.2.0",     "express": "^4.17.1",     "moment": "^2.27.0",     "ms": "^2.1.2",     "http": "^0.0.0"   },   "engines": {     "node": "12.x"   },   "repository": {     "url": "https://glitch.com/edit/#!/hello-express"   },   "license": "MIT",   "keywords": [     "node",     "glitch",     "express"   ] }

You should open a PR

didn't work
so i opened this

Fixed