[installer] uninstalling PT requires quitting other apps

This applies to current master 4dfac020650989c598bf4eb529bfe5e136047889
The MSI is trying to uninstall components that are used by other apps.

The reason why this was happening is that launcher continues to run in the background while we try to uninstall PowerToys. If my understanding is correct, the above PR fixed it is because Launcher was no longer running in that MSI because of the missing dlls.
An easy way to repro this even without launcher is to open settings v2 and try to uninstall powerToys. It shows the same dialogue that some applications need to be closed and it doesn't show this when we close settingsv2 and try to uninstall. However, it is prominently noticed with launcher because it is always running in the background.
According to me the correct fix for this issue would be to close PowerLauncher.exe and the Settings executable before uninstalling, as is done for the PowerToys.exe.

<Property Id="MSIRESTARTMANAGERCONTROL" Value="Disable" />
<!-- Restart explorer.exe if we detect existing PowerRenameExt.dll or ImageResizerExt.dll installation -->
<util:CloseApplication Target="explorer.exe" RebootPrompt="no" TerminateProcess="0">
  EXISTINGPOWERRENAMEEXTPATH OR EXISTINGIMAGERESIZERPATH
</util:CloseApplication>
<util:CloseApplication CloseMessage="yes" Target="PowerToys.exe" ElevatedCloseMessage="yes" RebootPrompt="no" TerminateProcess="0" />

yes, we should add this too

and windowwalker

@alekhyareddy28
great finding, thanks!

@enricogior I haven't been able to repro this exact scenario where it shows other random applications. For me it only shows PowerToys settings. Did you face this while installing or while uninstalling?

I hit this installing.

@enricogior I haven't been able to repro this exact scenario where it shows other random applications. For me it only shows PowerToys settings. Did you face this while installing or while uninstalling?

I see random applications as well, but only while uninstalling. This doesn't show up for me if all PTs are exited properly.

I just installed from the farm, still hitting scenarios it where 'files are in use.'  Seems Office related.

It might be a bug in WiX/MSI/Windows, since in theory those applications are not using the dlls installed by PT, but for some reason they are considered the same dll even if they are copies.

Maybe there's a possibility that Wix is looking only at the file IDs rather than the file names? I think there were some files added in the wix installer without file IDs and maybe that is what is causing the problem? Just thinking of possible reasons why Wix would throw this error

@arjunbalgovind
yest, that is a possibility. We'll verify it. Thanks of the suggestion.

@arjunbalgovind yes, that's our theory as well.
Looks like we might just disable the FileInUse dialog or InstallValidate action altogether now. I'd like to test this solution, but I cannot reproduce the problem on installation. I have Outlook, PowerPoint and Teams open, just the same as on @crutkas screenshot, but the installation still happens w/o that dialog. Would be awesome to have a consistent way of reproducing it.
I'm also thinking of possible implications of disabling that dialog, but only the contrived and intentional scenarios come up.

WiX is the build toolset. Windows Installer (MSI) is the runtime. So WiX isn't involved in this at runtime, other than supplying the dialog. File ids are irrelevant but maybe not names.
Start with a verbose log.

@barnson

So WiX isn't involved in this at runtime

We know that, but if we do something wrong in the WiX definitions or if WiX doesn't build the installer correctly, it's still "involved".


v8 log size

Hey there -
As we've established, I'm no expert here. I just wanted to point out that the v8.log file gets really big on my hub. I looked and it was 16MB. When we're working with a mere 60MB, I thought I should bring this to your attention. I'm not sure if this log automatically deletes itself or manages it's size on it's own or what, but I manually cleared it.
I just don't want people running out of space and crashing their hub because a log file got too big.
Thanks!

Thanks for pointing this out. I'll take a look at how to deal with this
On Mon, Aug 22, 2016 at 9:52 AM bazhip<EMAIL_ADDRESS>wrote:

Hey there -
As we've established, I'm no expert here. I just wanted to point out that
the v8.log file gets really big on my hub. I looked and it was 16MB. When
we're working with a mere 60MB, I thought I should bring this to your
attention. I'm not sure if this log automatically deletes itself or manages
it's size on it's own or what, but I manually cleared it.
I just don't want people running out of space and crashing their hub
because a log file got too big.
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/danielolson13/wink-mqtt/issues/10, or mute the thread
https://github.com/notifications/unsubscribe-auth/AEs5GDItCYi4AwbP3Xh3ggx1U8V5pb-aks5qiambgaJpZM4Jp45Y
.


port https://github.com/Azure/azure-sdk-for-java/pull/6835 from v4

port https://github.com/Azure/azure-sdk-for-java/pull/6835 from v4

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - ci

/azp run java - cosmos - tests


Update README.md

add Swagger as the code generator example

We believe the current phrasing should cover Swagger - but thanks for opening a pull request. :slightly_smiling_face:


Disk usage metrics do not work in specific zone

ISSUE TYPE

Other

COMPONENT NAME
VMWARE

CLOUDSTACK VERSION
4.18

CONFIGURATION

OS / ENVIRONMENT

SUMMARY
In my environment, I use 2 zones.
However, in the "x" zone, the metrics part presents a problem when displaying disk and network data.
While in zone "y" all data is shown normally.
Do you have any specific connectivity tests to do?
In principle, the zone settings are identical, the only difference is the following:
Zone X
System VM's -> KVM (The only KVM it has are the two SystemVM)
Zone X environment uses VMware vSphere Client version 6.7.0
Zone y uses the system VM and entire environment with vSphere Client version <IP_ADDRESS>00
Remembering that the disk and network metrics just do not appear in the problem area, but the CPU and memory metrics work normally.
Zone X:

Zone Y:

That is a very special setup @vikyoto . I am afraid that reproducing your issue is going to be a stretch. You can try and right a health-check script for the VR, but I doubt that will yield related data. I would start to see if  there are any StatsCollector related errors in the logs.


Toggle to show all at once

Thank you for this very useful plugin!
Is there an option to toggle to open all at once? Same thing for close? That would be very helpful. Thanks!

I support this idea!

Hi there,
sorry for the time I took to do it, but I've just added the option: https://a11y.nicolas-hoffmann.net/hide-show/
Thanks a lot for this idea! 👍


Remove roof tarp

Find and seal the leaks where possible.

Let's attempt this maybe this Wednesday?


Dreambooth won't start: [00:51:51] WARNING  The following values were not passed to

Describe the bug
Getting this whenever I run train_dreambooth.py on Colab but it won't get passed it, it seems to freeze up and just run and not do anything with this warning and the cursor keeps blinking, until you stop the cell, then it just stops immediately when you try to run it after that. It has always worked before, and no changes on my end. Think it might be an issue with accelerate. Have no other info.
Reproduction
[00:51:51] WARNING  The following values were not passed to
Logs
[00:51:51] WARNING  The following values were not passed to

System Info
Colab notebook

I have the exact same problem 🥶

me too but it keeps generating person in the data directory

I am also facing the same issue.
I checked my google drive, the instance images uploaded were also not present in the google drive.

I've been getting the same error since yesterday, any idea?

Confirming that I am also seeing this error. Tried passing different arguments but was not able to find the issue. This happened around 2 days ago.
The only thing that I'm seeing that might have changed is that the xformers library repo had an update 2 days ago. My best guess is that an update on that broke the notebook.

Tried running the notebook while installing the previous version of xformers==0.0.17rc481 but that did not solve the issue

I think it might be a Google end problem unless anyone can confirm it's also happening on local PC. I've looked into several things and can't pinpoint it. If anyone has success please let us know here, since he might not get to until after the weekend. I just hope it's not a serious or unfixable issue.

Now it seems to running but still not displaying. I switched from the custom model I was trying to fine-tune to SD1-5, also removed the --read_prompts_from_txt argument that I usually use.  I'm not sure if any of these at least made it run or not, too many combinations of things to try. We'll see if it completes and generates samples, etc.

The display issue appears to be specific to the use of accelerate launch. I was able to bypass it by changing this:
!accelerate launch train_dreambooth.py
to:
!/usr/bin/python3 train_dreambooth.py

The display issue appears to be specific to the use of accelerate launch. I was able to bypass it by changing this:
!accelerate launch train_dreambooth.py
to:
!/usr/bin/python3 train_dreambooth.py

Will confirm that this seems directly tied to accelerate launch. Using you your above changes it does indeed return the display and seems to work as normal.
Thanks for sharing your solution, hntrssthmpsn

There was a new version of accelerate released 2 days ago, which sounds like it coincides with the onset of this issue. It's possible that just pinning accelerate to 0.17.1 would also resolve the issue.

Will see if that works, can bind a version pretty easy for anyone as dumb as me, replace line 5 of the "Install Requirements" cell with:
%pip install -q accelerate==0.17.1 transformers ftfy bitsandbytes==0.35.0 gradio natsort safetensors xformers

getting this error when i use: !/usr/bin/python3 train_dreambooth.py
"train_dreambooth.py: error: the following arguments are required: --instance_data_dir, --instance_prompt"
but i definitely ran the cell that assigned those, so not sure. also tried the 0.17.1 but still got the Warning error.

I have been using your repository to create images out of a pre-trained model, and it has been working well. However, when I tried to train a model using your repository, I encountered an error that seems to indicate a problem with the repository.
I have tried troubleshooting the issue using the steps recommended by ChatGPT, including checking my internet connection, verifying the repository URL, refreshing the page, clearing my browser's cache, and using a different repository. However, none of these steps have resolved the issue.
Upon inquiring with ChatGPT, it was suggested that I contact the repository owner to report the issue. I would appreciate it if you could investigate this issue and provide a fix as soon as possible, as it is preventing me from using your repository for my work.

getting this error when i use: !/usr/bin/python3 train_dreambooth.py
"train_dreambooth.py: error: the following arguments are required: --instance_data_dir, --instance_prompt"
but i definitely ran the cell that assigned those, so not sure. also tried the 0.17.1 but still got the Warning error.

Try it using just the  !/usr/bin/python3 train_dreambooth.py method in the run cell but not changing the accelerate version in the installation. I actually didn't change it, I just assumed it worked,  The !/usr/bin/python3 train_dreambooth.py works on it's own with no other changes. My bad.

Made a bandaid fix for now 17ff5f59d26c9abc86ccf44df240951e0c68a27d

Made a bandaid fix for now 17ff5f5

Sorry to reply the closed thread. Accelerate launch seems to work when you add the following code cell or maybe place it in train_dreambooth.py
from accelerate.utils import write_basic_config

write_basic_config()

from accelerate.utils import write_basic_config
write_basic_config()

are you saying can add this anywhere before training starts?

from accelerate.utils import write_basic_config
write_basic_config()

are you saying can add this anywhere before training starts?

Yes, make a new cell and run it right before you begin training, but you would need to change !python3 back to !accelerate launch in the dreambooth train cell, or there is no point...

2023-12-19 15:03:08.982737: E external/local_xla/xla/stream_executor/cuda/cuda_dnn.cc:9261] Unable to register cuDNN factory: Attempting to register factory for plugin cuDNN when one has already been registered
2023-12-19 15:03:08.982794: E external/local_xla/xla/stream_executor/cuda/cuda_fft.cc:607] Unable to register cuFFT factory: Attempting to register factory for plugin cuFFT when one has already been registered
2023-12-19 15:03:08.984081: E external/local_xla/xla/stream_executor/cuda/cuda_blas.cc:1515] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2023-12-19 15:03:10.289076: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
[15:03:11] WARNING  The following values were not passed to `accelerate launch` and

Facing this issue still


Try out android-emulator-runner fork

Just testing to see if performance is any better
bors try

bors retry


doc: add icons to the home page blocks

Use (free license) fontawesome icon characters to pretty up the Zephyr
home page navigation blocks added earlier.  (The fontawesome font famliy
is already included by the read-the-doc theme so we can take advantage
of that.)
Signed-off-by: David B. Kinder<EMAIL_ADDRESS>

Shippable doc build artifacts: https://builds.zephyrproject.org/zephyrproject-rtos/zephyr/13072/docs/index.html

Codecov Report

Merging #13072 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #13072   +/-   ##
=======================================
  Coverage   48.63%   48.63%
=======================================
  Files         313      313
  Lines       46436    46436
  Branches    10710    10710
=======================================
  Hits        22585    22585
  Misses      19395    19395
  Partials     4456     4456

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c96c90a...c768d3c. Read the comment docs.

FYI, you can see all the fontawesome glyphs at https://fontawesome.com/cheatsheet  and append an fa- prefix to use the glyph name.


Host a file on dnscrypt.org

I am working on self-updates for my client, and I need to download the list of available versions. It's a very small file + a minisig signature. Can you put this file on dnscrypt,org?

Hi there,
As you probably know, a copy of the dnscrypt-resolvers.csv can be downloaded here: http://updates.dnscrypt.org/dnscrypt-proxy/dnscrypt-resolvers.csv.
A cron job mirrors it daily from github, along with the signature. It's safe to do so specifically because of the signature.
Can something similar be done for your files? Have these fixes at a fixed location in your Github repository, and then I can set up a job to mirror them? How frequently should that happen?

Done. Your updates directory is now mirrored to http://download.dnscrypt.org/updates/dnscrypt-osx-client.


Fix kubectl stop rc with sequence numbers

Introduces the concept of sequence numbers for rcs fixing the behavior of kubectl stop rc in the process.
@lavalamp @bgrant0607 https://github.com/GoogleCloudPlatform/kubernetes/issues/9147
Still need to think of a good description string and look at why the conversion generating script doesn't autogenerate for rc.Spec changes (https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/api/v1/conversion.go#L30)

Is sequenceNumber the right name?  I assume this a pattern we'll use in the future for other reasons.

@smarterclayton Yeah, it's a general pattern. SequenceNumber is the most obvious name, but that doesn't mean there's not a better name possible :)
I think the interesting debate to have about it is:
a) Does it go in metadata? (IMO: yes)
b) Do we implemented for all objects? (IMO: eventually, yes-- but it doesn't have to work everywhere immediately)
c) Does this imply that there should only be a single writer of obj.Status? (IMO: Yes! This should force us to clarify our rapidly-increasing-in-complexity web of object readers & writers.)

The basic idea, of course, is to give you some idea about what version of the object is being used by the object's actuator, so that you can--in a generic way--tell when the system has quiesced.
@bgrant0607 I've forgotten why we can't just have the actuator just reiterate the resourceVersion that it's acting on? Is this only because updates to status also increase the resourceVersion, or is there another reason, too?

And adding to my earlier list--
d) Some objects may be dependent on other objects. E.g., arguably endpoints should be using the sequence number from the service object.
...now that I think about it, endpoints should really be a subresource of services, or part of service.status.

I've forgotten why we can't just have the actuator just reiterate the resourceVersion that it's acting on? Is this only because updates to status also increase the resourceVersion, or is there another reason, too?

I can't think into the future enough to say it's safe to compare the resource version, even of a single resource, against itself. can we assume a future version of the same resource will have a higher resource version even if its from a different member/shard? The sequence number going up property is useful when a client wants to know the controller has seen an update, but doesn't care if it's seen many updates after that.

On Jun 12, 2015, at 6:13 PM, Daniel Smith<EMAIL_ADDRESS>wrote:
@smarterclayton Yeah, it's a general pattern. SequenceNumber is the most obvious name, but that doesn't mean there's not a better name possible :)
I think the interesting debate to have about it is:
a) Does it go in metadata? (IMO: yes)

Agree, unless we have a need to subdivide the object into multiple sets, but it's likely those objects probably need a different pattern anyway.

b) Do we implemented for all objects? (IMO: eventually, yes-- but it doesn't have to work everywhere immediately)

Or an object which doesn't have an obvious actuator just doesn't expose it in status.  For instance, a service isn't actuated so much as  propagated.

c) Does this imply that there should only be a single writer of obj.Status? (IMO: Yes! This should force us to clarify our rapidly-increasing-in-complexity web of object readers & writers.)
—
Reply to this email directly or view it on GitHub.

The sequence number going up property is useful when a client wants to know the controller has seen an update, but doesn't care if it's seen many updates after that.

Ah, yes, thank you-- I forgot about that part. We want to allow >=, not just ==.

PTAL, moved the Status.SequenceNumber code into the controller and added some more unittests

I think this looks pretty good. I'd like to nail down whether sequenceNumber should go in metadata or in spec before we merge it, though.

I haven't thought of a concrete reason to need two sequence numbers that couldn't be solved in spec directly (container sequence, for instance).  If we add this to metadata we probably want to define it for objects which don't have clear specs (something like a user record, or an event).  Also, probably need to clarify whether it applies to spec for things like nodes (answer is probably yes, but spec is a bit less clear for that type).

On Jun 15, 2015, at 7:15 PM, Daniel Smith<EMAIL_ADDRESS>wrote:
I think this looks pretty good. I'd like to nail down whether sequenceNumber should go in metadata or in spec before we merge it, though.
—
Reply to this email directly or view it on GitHub.

See also #7328

I just posted some responses to the issue: #https://github.com/GoogleCloudPlatform/kubernetes/issues/7328#issuecomment-113018951

I would like a sequence number on a ResourceQuota so I think it's generally useful to objects outside of the replication controller.  Users can change the quota spec, but it's not enforced until it's applied to the status.  Being able to know if the enforced status is different than the desired spec is useful as I was previously looking to compute if there was a delta when I wanted to let the user know the desired quota was different than the enforced.

Our deployments, builds, and routes would benefit as well I believe.

On Jun 17, 2015, at 10:58 PM, Derek Carr<EMAIL_ADDRESS>wrote:
I would like a sequence number on a ResourceQuota so I think it's generally useful to objects outside of the replication controller. Users can change the quota spec, but it's not enforced until it's applied to the status. Being able to know if the enforced status is different than the desired spec is useful as I was previously looking to compute if there was a delta when I wanted to let the user know the desired quota was different than the enforced.
—
Reply to this email directly or view it on GitHub.

We'll definitely want this for pods and probably services, also. With loosely coupled / distributed / async. / concurrent systems, it was only a matter of time before we needed something like this badly enough to implement it.

'Pologies for the delay, got sidetracked debugging an unrelated e2e failure to confirm it's not this change. Might still update some of the comments/descriptions but my latest set of commits has a Generation embedded in the object. PTAL.

Looks pretty good. Just a couple minor comments.

Please regenerate the swagger so that shippable can run.
hack/update-swagger-spec.sh

I did run update-swagger a few times (and the new commits ended up triggering k8s bot as well), I'll need to dig into why that doesn't seem to have worked

PTAL, I'll figure out the swagger bug later, somehow verify followed by updated seems to have cleared some state and it worked.

Shippable failure is a flake because n/w latencies are really high for some reason (took ~20s for a watch in my last run). Bumped up an integration test timeout that doesn't really need to be that short anyway  with my last commit.

LGTM


Drawer vs. Swipe-back (Android)

Hey,
So I'm using the <Drawer /> as a navigation menu together with a <Navigator />. One thing I noticed is that once I've navigated and then try to open the <Drawer /> menu again (by swiping from left to right), the <Navigator /> will at the same time do the swipe-back mechanism, pushing me to the previous route.
I've tried the navigator.replace(route) instead of navigator.push(route), but then the transition animation no longer plays, and the view is just replaced instantly.
Is there a way to remove the previous route (from the stack) inside the onClose event handler? Or is that bad practice? Kinda stuck here, great if someone got any ideas!

If I had only waited two more minutes before asking...
Disabling swipe-back: https://github.com/facebook/react-native/issues/1014#issuecomment-96272360

yes, I typically disable swipe for back


Update CLI package's README.md

Description
Updates CLI package's README.md with an updated list of arguments

Created PR directly for the main since these are just documentation changes


Changing Sizes of icon paths

I am trying to resize the map_pin since the pin is so large. does any one have a clue how to change these sizes? I looked at the svg and modified the heights/width but I noticed nothing changed in the sizes. I see that Map_Pin has a path set in the .js file, other than that I'm not really sure how I can change the size of the pin. Any help would be great.

Use scale

Also you need to edit the map-icons.scss a little bit.  This is for .7 scale (I multiplied original values of width and height and font-size by .7)
.map-icon-label .map-icon {
display: block;
font-size: 17px;
color: #FFFFFF;
width: 33.6px;
line-height: 33.6px;
text-align: center;
white-space: nowrap;
}


Define Code per https://XXX-iz.impfterminservice.de

Each Server with different Locations identified by https://XXX-iz.impfterminservice.de has the same code.
Make it configurable which centers have which code.
Example:
{
"url": "https://229-iz.impfterminservice.de/impftermine/service?plz=88212",
"code": "XXXX-XXXX-XXXX",
"name": "Ravensburg"
},
{
"url": "https://001-iz.impfterminservice.de/impftermine/service?plz=89073",
"code": "YYYY-YYYY-YYYY",
"name": "Ulm"
},
{
"url": "https://003-iz.impfterminservice.de/impftermine/service?plz=88045",
"code": "ZZZZ-ZZZZ-ZZZZ",
"name": "Friedrichshafen"
},
{
"url": "https://003-iz.impfterminservice.de/impftermine/service?plz=88444",
"code": "ZZZZ-ZZZZ-ZZZZ",
"name": "Ummendorf"
}

we won't support searching in multiple groups at once as explained in #226


Unable to use Sveltekit modules in components (i.e. $path)

When trying to import Sveltekit Modules such as the $path store in a component,
❌ I get an error in Histoire (port 6006)
✅ I don't get an error in Sveltekit (port 5173).

Example: a component that displays the <URLpath />
Output: "indaco/histoire-sveltekit/issues/2"

<!-- URLpath.svelte -->
<script>
  import { page } from '$app/stores'
</script>

<span>{$page.url.pathname}</span>

sveltekit-histoire-bug.webm
Error

> oneezy@oneezy MINGW64 ~/Desktop/www/10.Repros/indaco-histoire-sveltekit-repro (main)
$ pnpm story

><EMAIL_ADDRESS>story C:\Users\oneezy\Desktop\www\10.Repros\indaco-histoire-sveltekit-repro
> svelte-kit sync && histoire dev

  ➜  Local:   http://localhost:6006/
  ➜  Network: use --host to expose
Collect stories start all
Failed to resolve import "$app/stores" from "src\lib\components\URLpath\URLpath.svelte". Does the file exist?
Error while collecting story C:/Users/oneezy/Desktop/www/10.Repros/indaco-histoire-sveltekit-repro/src/stories/URLpath/URLpath.story.svelte:
Error: Failed to resolve import "$app/stores" from "src\lib\components\URLpath\URLpath.svelte". Does the file exist?
    at formatError (file:///C:/Users/oneezy/Desktop/www/10.Repros/indaco-histoire-sveltekit-repro/node_modules/.pnpm/vite@3.1.4/node_modules/vite/dist/node/chunks/dep-6b3a5aff.js:40854:46)
    at TransformContext.error (file:///C:/Users/oneezy/Desktop/www/10.Repros/indaco-histoire-sveltekit-repro/node_modules/.pnpm/vite@3.1.4/node_modules/vite/dist/node/chunks/dep-6b3a5aff.js:40850:19)
    at normalizeUrl (file:///C:/Users/oneezy/Desktop/www/10.Repros/indaco-histoire-sveltekit-repro/node_modules/.pnpm/vite@3.1.4/node_modules/vite/dist/node/chunks/dep-6b3a5aff.js:37587:33)
    at async TransformContext.transform (file:///C:/Users/oneezy/Desktop/www/10.Repros/indaco-histoire-sveltekit-repro/node_modules/.pnpm/vite@3.1.4/node_modules/vite/dist/node/chunks/dep-6b3a5aff.js:37720:47)
    at async Object.transform (file:///C:/Users/oneezy/Desktop/www/10.Repros/indaco-histoire-sveltekit-repro/node_modules/.pnpm/vite@3.1.4/node_modules/vite/dist/node/chunks/dep-6b3a5aff.js:41103:30)
    at async loadAndTransform (file:///C:/Users/oneezy/Desktop/www/10.Repros/indaco-histoire-sveltekit-repro/node_modules/.pnpm/vite@3.1.4/node_modules/vite/dist/node/chunks/dep-6b3a5aff.js:37365:29)
Collect stories end 3952 ms
The request url "C:/Users/oneezy/Desktop/www/2.Starters/monorepo/node_modules/.pnpm/@histoire+app@0.11.7_vite<EMAIL_ADDRESS>is outside of Vite serving allow list.

- C:/Users/oneezy/Desktop/www/10.Repros/indaco-histoire-sveltekit-repro/node_modules/.pnpm/@histoire+app@0.11.0_vite@3.1.4/node_modules/@histoire/app/dist
- C:/Users/oneezy/Desktop/www/10.Repros/indaco-histoire-sveltekit-repro/node_modules/.histoire
- C:/Users/oneezy/Desktop/www/10.Repros/indaco-histoire-sveltekit-repro
- C:/Users/oneezy/Desktop/www/10.Repros/indaco-histoire-sveltekit-repro

Refer to docs https://vitejs.dev/config/server-options.html#server-fs-allow for configurations and more details.

Here's the full reproduction:
https://github.com/oneezy/indaco-histoire-sveltekit-repro

@oneezy the same happens with histoire-dev blank project too.
I would suggest to open an issue there as I did for the other issue you reported here.
Is it something feasible to you?
Thanks

@indaco yeah I had noticed that too.
I actually tested that one out first, then yours, and am currently testing @swyx 's , though he's archived his for some reason.
I'll create a bug report over at Histoire

@oneezy  can you reference the issue here once you open it there? thanks

@indaco will do, sorry had to get some rest but I will reference it here afterwards.
No, it doesn't work there but that's exactly my plan. All apps in oneezy/monorepo are Sveltekit other then histoire (only Svelte) .. So none of the Sveltekit specific things are working.

Done.

have you got this to work yet? said it was fixed but i'm still unable to get it working

Will check and keep you posted

@oneezy https://github.com/sveltejs/kit/pull/8838


On Lollipop make sure the Snackbar elevation is the same as other children

This fixes issue #109, where the Snackbar is displaying under the rest of the view. On Lollipop we need to make sure that the elevation of the Snackbar is as high as the other child view(s).

that was fast :)

will be able to publish a new version with this fix tonight as soon as I get home

Sweet!

@jonathan-caryl 2.10.8 is now published

Great


[BUG-UI/UX] fields and questions don't scroll up after record submit

Describe the bug
After submitting a form, if fields and questions are scrollable, we don't scroll up after a submit.
To Reproduce
See video
Expected behaviour
I would expect the scroll to start from scratchc
Screenshots
https://github.com/argilla-io/argilla/assets/25269220/076ab46b-3ebc-4143-9fee-7852b31cb190

Environment (please complete the following information):

OS [e.g. iOS]: MaxOS
Browser [e.g. chrome, safari]: FireFox
Argilla Version [e.g. 1.0.0]: 1.14
ElasticSearch Version [e.g. 7.10.2]:
Docker Image (optional) [e.g. argilla:v1.0.0]: 1.14

Additional context
N.A.

Fixed here -> https://github.com/argilla-io/argilla/pull/3830


refctor shape file creation

#885 create shapefiles in memory return package of streams insted of writing direct to disk

merge zip refactor to master


Fiona from wheel does not set its GDAL_DATA path

Again/still struggling with the gdal and proj stuff (re https://github.com/Toblerity/Fiona/issues/673 and https://github.com/Toblerity/Fiona/issues/691).
From what I read, Fiona should set GDAL_PATH and PROJ_LIB to its own paths if there were no such paths defined in the environment yet.
I do not have GDAL_PATH or PROJ_LIB set, but Fiona still does not work :o)
I installed the latest version from pypi at https://pypi.org/project/Fiona/#files
pip install --user --upgrade https://files.pythonhosted.org/packages/b8/55/fdcd9a45b39a4fe50d79041d1194c2ebefcc7ef44958d54ad2b815129f02/Fiona-1.8.4-cp37-cp37m-manylinux1_x86_64.whl

import os

def checkpaths():
    gdal_data = os.environ.get('GDAL_DATA')
    if gdal_data:
      print(f"GDAL_DATA is set to {gdal_data}")
      if os.path.exists(gdal_data+"/gcs.csv"):
        print("GDAL_DATA seems correct")
      else:
        print("GDAL_DATA seems BROKEN")
    else:
      print(f"GDAL_DATA is NOT set")

    proj_lib = os.environ.get('PROJ_LIB')
    if proj_lib:
      print(f"PROJ_LIB is set to {proj_lib}")
      if os.path.exists(proj_lib+"/epsg"):
        print("PROJ_LIB seems correct")
      else:
        print("PROJ_LIB seems BROKEN")
    else:
      print(f"PROJ_LIB is NOT set")

>>> checkpaths()
GDAL_DATA is NOT set
PROJ_LIB is NOT set

Importing Fiona now should have it update the paths to its own env, but it only sets the PROJ_LIB one.
>>> import fiona

>>> checkpaths()
GDAL_DATA is NOT set
PROJ_LIB is set to /home/kannes/.local/lib/python3.7/site-packages/fiona/proj_data
PROJ_LIB seems correct

This means that e.g. transformations will fail:
import fiona
from fiona.transform import transform_geom
from fiona.crs import from_epsg

eiffel_tower = {'type': 'Point', 'coordinates': (2.2945, 48.858222)}

crs_source_epsg = from_epsg(4326)
crs_target_epsg = from_epsg(25831)

transformed = transform_geom(crs_source_epsg, crs_target_epsg, eiffel_tower)
print(transformed)

prints:
ERROR 4: Unable to open EPSG support file gcs.csv.  Try setting the GDAL_DATA environment variable to point to the directory containing EPSG csv files.
ERROR 4: Unable to open EPSG support file gcs.csv.  Try setting the GDAL_DATA environment variable to point to the directory containing EPSG csv files.
ERROR 6: No translation for an empty SRS to PROJ.4 format is known.
{'type': 'Point', 'coordinates': (2.2945, 48.858222)}

If I manually set the correct GDAL_PATH before importing Fiona everything works fine:
# you MUST use a pristine Python session for this
import os

os.environ['GDAL_DATA'] = "/home/hannes/.local/lib/python3.7/site-packages/fiona/gdal_data"

import fiona
from fiona.transform import transform_geom
from fiona.crs import from_epsg

eiffel_tower = {'type': 'Point', 'coordinates': (2.2945, 48.858222)}

crs_source_epsg = from_epsg(4326)
crs_target_epsg = from_epsg(25831)

transformed = transform_geom(crs_source_epsg, crs_target_epsg, eiffel_tower)
print(transformed)

prints
{'type': 'Point', 'coordinates':<PHONE_NUMBER>8407586, 5411935.123097671)}

I got this straightened out in Rasterio, but haven't ported it here yet. Will do so for 1.8.5.

you rock

Could we use _env.GDALDataFinder (private) to mitigate this issue prior to the update (3/11/2018)?
import fiona
import os
os.environ['GDAL_DATA'] = fiona._env.GDALDataFinder.search_wheel(
    'literally any string')
......


Update functions to middlewares and final format is also http handler

Allow option for #12
Allow customisation of microtik listname @williamdes
Convert other functions into middlewares so early return is simpler
Memory consumption is a little concern due to filters, however, stress testing only shown a 6mb increase.
When I mean stress test that was 1k request per second over 10 minutes.

Also update the docker/README.md ?

Maybe merge #24 and rebase onto so we try the new workflows on this one ?

I ask another team member for a review, since they better at CI and I might miss something since I been staring at code for last 2 hours 😆
Appreciate your input on this PR, I will push for release after this PR been merged as its a major change.

If you have some mac of guys ask them if the Linux binaries we distribute run on their workstation or if I should add Darwin binaries too

Just looking at optimisations why do we need to create an ips within the microtik formatter? plus do we care about sorting strings? seems abit pointless? unless there a reason?

PS: last commit is not GPG signed thinking

I messed up the rebase 😮💨
It doesn't matter now, I now made it so when we fetch the decisions it sorts them so it should be same order everytime.

PS: last commit is not GPG signed thinking

I messed up the rebase 😮💨
It doesn't matter now, I now made it so when we fetch the decisions it sorts them so it should be same order everytime.

Awesome!
Good for the then

This is ApacheBench, Version 2.3 <$Revision: 1901567 $>
Copyright 1996 Adam Twiss, Zeus Technology Ltd, http://www.zeustech.net/
Licensed to The Apache Software Foundation, http://www.apache.org/

Benchmarking localhost (be patient)
Completed 1000 requests
Completed 2000 requests
Completed 3000 requests
Completed 4000 requests
Completed 5000 requests
Completed 6000 requests
Completed 7000 requests
Completed 8000 requests
Completed 9000 requests
Completed 10000 requests
Finished 10000 requests

Server Software:
Server Hostname:        localhost
Server Port:            4141

Document Path:          /
Document Length:        3704276 bytes

Concurrency Level:      10
Time taken for tests:   45.741 seconds
Complete requests:      10000
Failed requests:        0
Keep-Alive requests:    0
Total transferred:<PHONE_NUMBER>0 bytes
HTML transferred:<PHONE_NUMBER>0 bytes
Requests per second:    218.62 [#/sec] (mean)
Time per request:       45.741 [ms] (mean)
Time per request:       4.574 [ms] (mean, across all concurrent requests)
Transfer rate:          790874.24 [Kbytes/sec] received

Connection Times (ms)
min  mean[+/-sd] median   max
Connect:        0    0   0.0      0       0
Processing:    26   46   6.5     45      73
Waiting:       14   24   4.7     24      40
Total:         26   46   6.5     45      73

Percentage of the requests served within a certain time (ms)
50%     45
66%     48
75%     50
80%     51
90%     54
95%     58
98%     60
99%     62
100%     73 (longest request)

Is sorting responsible for the overhead?

Is sorting responsible for the overhead?

Microtik no sort 1.7ms faster per request
This is ApacheBench, Version 2.3 <$Revision: 1901567 $>
Copyright 1996 Adam Twiss, Zeus Technology Ltd, http://www.zeustech.net/
Licensed to The Apache Software Foundation, http://www.apache.org/

Benchmarking localhost (be patient)
Completed 1000 requests
Completed 2000 requests
Completed 3000 requests
Completed 4000 requests
Completed 5000 requests
Completed 6000 requests
Completed 7000 requests
Completed 8000 requests
Completed 9000 requests
Completed 10000 requests
Finished 10000 requests

Server Software:
Server Hostname:        localhost
Server Port:            4141

Document Path:          /
Document Length:        3578225 bytes

Concurrency Level:      10
Time taken for tests:   28.034 seconds
Complete requests:      10000
Failed requests:        0
Keep-Alive requests:    0
Total transferred:<PHONE_NUMBER>0 bytes
HTML transferred:<PHONE_NUMBER>0 bytes
Requests per second:    356.71 [#/sec] (mean)
Time per request:       28.034 [ms] (mean)
Time per request:       2.803 [ms] (mean, across all concurrent requests)
Transfer rate:          1246511.39 [Kbytes/sec] received

Connection Times (ms)
min  mean[+/-sd] median   max
Connect:        0    0   0.0      0       0
Processing:    18   28   3.9     28      45
Waiting:        2    4   1.4      4       9
Total:         18   28   3.9     28      45

Percentage of the requests served within a certain time (ms)
50%     28
66%     30
75%     31
80%     31
90%     33
95%     35
98%     36
99%     37
100%     45 (longest request)

plain_text no sort 1.8ms faster
This is ApacheBench, Version 2.3 <$Revision: 1901567 $>
Copyright 1996 Adam Twiss, Zeus Technology Ltd, http://www.zeustech.net/
Licensed to The Apache Software Foundation, http://www.apache.org/

Benchmarking localhost (be patient)
Completed 1000 requests
Completed 2000 requests
Completed 3000 requests
Completed 4000 requests
Completed 5000 requests
Completed 6000 requests
Completed 7000 requests
Completed 8000 requests
Completed 9000 requests
Completed 10000 requests
Finished 10000 requests

Server Software:
Server Hostname:        localhost
Server Port:            4141

Document Path:          /
Document Length:        410401 bytes

Concurrency Level:      10
Time taken for tests:   12.260 seconds
Complete requests:      10000
Failed requests:        0
Keep-Alive requests:    0
Total transferred:<PHONE_NUMBER> bytes
HTML transferred:<PHONE_NUMBER> bytes
Requests per second:    815.64 [#/sec] (mean)
Time per request:       12.260 [ms] (mean)
Time per request:       1.226 [ms] (mean, across all concurrent requests)
Transfer rate:          326972.88 [Kbytes/sec] received

Connection Times (ms)
min  mean[+/-sd] median   max
Connect:        0    0   0.0      0       0
Processing:     6   12   2.6     12      31
Waiting:        2    4   1.4      4      13
Total:          6   12   2.6     12      31

Percentage of the requests served within a certain time (ms)
50%     12
66%     13
75%     14
80%     15
90%     16
95%     16
98%     17
99%     18
100%     31 (longest request)

obviously sorting for every request will cause some overhead unless we provide some sort of caching but at that point it might be better to put a cache infront of the blocklist bouncer.

At this point I'd argue it's a good idea to add a way to disable sorting just like IPv6
This way people can have a grain of salt more performance
But it's not that much gain

At this point I'd argue it's a good idea to add a way to disable sorting just like IPv6 This way people can have a grain of salt more performance But it's not that much gain

Added option for ?nosort the default will be to sort and if they want to heavily optimise then they can find this parameter


[SignalR]: AuthorizeHelper should no-op if endpoint routing has already executed auth

Similar to the solution in #8387, we need to avoid re-running auth in SignalR if the middleware previously ran.

cc @rynowak \ @davidfowl

The work items here are:

Scoop up all attributes on the Hub and put them in the metadata (bonus: we get CorsAttribute for free!).
Remove our custom connection-level auth and use endpoint auth now.

We already did most of this work. The last piece is to skip if it already ran

Scoop up all attributes on the Hub

We did this? @BrennanConroy wasn't so sure...

https://github.com/aspnet/AspNetCore/blob/3e056db750d562826473f476e4f9b02ad76ffdd3/src/SignalR/server/SignalR/src/HubEndpointRouteBuilderExtensions.cs#L49

I would be really careful about making things skip and be smart - rather than making sure that the app is configured correctly. The change in #8387 broke stuff in MVC because the user would configure the AuthorizeFilter manually and it would now no-op.
We landed on making it an error to have [Authorize] and not have the authorize middleware when using endpoint routing.  Consider whether you can configre this behaviour based on whether or not endpoint routing is in use.

@rynowak the way I thought of doing this was to not add auth metadata to HttpConnectionDispatcherOptions as part of the extension method @davidfowl pointed. It would be analogous to how MVC would not set up a filter if you're doing endpoint routing.

OK DOpe

@pranavkm you assigned yourself to this, are you going to do it? If not, let me know and I'll find a victim person to fix it.

Sure, I'd be happy to review the change if somebody else does the PR.

Cool, just wanted to clarify the status of your assignment ;P. We'll take a look at this in preview 6

Acceptance checklist (check one item)

[ ] We decided not to take this fix.
[ ] The fix is tests-only.
[x] The fix contains product changes (check all items below).

[x] Relevant XML documentation comments for new public APIs are present.
[x] Narrative docs (docs.microsoft.com) are updated. (check one item below)

[x] The change requires a new article. An issue with an outline has been filed here: [aspnet/AspNetCore.Docs/issues/11127]
[ ] The change requires a change to an existing article. A docs PR with these changes is linked here: [PR LINK]
[ ] The change requires no docs changes.

[x] Verification has been completed. (check one item below)

[x] The change is in the shared framework and was verified against the following versions

SDK installer: [3.0.100-preview6-012234]
ASP.NET Core Runtime: [3.0.0-preview6.19303.1]
.NET Core Runtime: [3.0.0-preview6-27730-01]

[ ] The change is in an OOB NuGet/NPM/JAR package and was verified against the following version of that package: [PACKAGE ID] [VERSION]


changing targetSdkVersion to 31 make notification not fired

on config.xml
         <preference name="android-targetSdkVersion" value="31" />
I used it with value 30 and notificiation work find!
but changing it to 31, then rm android and add androoid@10, then it is not fired.
on package.json

    "cordova-plugin-fcm-with-dependecy-updated": "^4.2.0",

"cordova-plugin-fcm-with-dependecy-updated": {
        "FCM_VERSION": "19.0.0",
        "GRADLE_TOOLS_VERSION": "3.5.3",
        "GOOGLE_SERVICES_VERSION": "4.3.3",
        "SUPPORT_LIBRARY_VERSION": "28.0.0",
        "ANDROID_DEFAULT_NOTIFICATION_ICON": "@mipmap/ic_launcher"
      }

ionic info

Ionic:

   Ionic CLI          : 5.4.16 (/Users/pedrojung/.nvm/versions/node/v14.17.3/lib/node_modules/ionic)
   Ionic Framework    : ionic-angular 3.9.2
   @ionic/app-scripts : 3.2.1

Cordova:

   Cordova CLI       : 11.0.0
   Cordova Platforms : android 10.1.2
   Cordova Plugins   : cordova-plugin-ionic-webview 4.0.1, (and 30 other plugins)

Utility:

   cordova-res                          : 0.15.4
   native-run (update available: 1.7.1) : 1.5.0

System:

   Android SDK Tools : 26.1.1 (/Users/pedrojung/Library/Android/sdk/)
   ios-deploy        : 1.11.4
   NodeJS            : v14.17.3 (/Users/pedrojung/.nvm/versions/node/v14.17.3/bin/node)
   npm               : 6.14.13
   OS                : macOS Ventura
   Xcode             : Xcode 14.2 Build version 14C18

I am having a similar issue. Notifications only show up in the foreground.
As a matter of fact, for me (running android 13 on samsung S21 FE 5G), if the app is in the background and a push arrives, the app crashes!
Hopefully somebody can help with this...

I've been running into the same issue, and I've identified the cause. Testing the receipt of push notifications using an emulator running the app, I see this fatal error:
FATAL EXCEPTION: Firebase-Messaging-Intent-Handle
Process: com.my-workplace.MyAppIdentifier, PID: 8789
java.lang.IllegalArgumentException: com.tnsos.GoVoteTN: Targeting S+ (version 31 and above) requires that one of FLAG_IMMUTABLE or FLAG_MUTABLE be specified when creating a PendingIntent.
Strongly consider using FLAG_IMMUTABLE, only use FLAG_MUTABLE if some functionality depends on the PendingIntent being mutable, e.g. if it needs to be used with inline replies or bubbles.

Doing some further research, this may be caused by specifying an outdated version of the FCM library as part of the preferences when adding the plugin, but so far my attempts to update it have met with other errors. I'll continue to add information as I go.

@thepsion5 @dimitriscsd  @kotran88
This fixed it for me
https://github.com/andrehtissot/cordova-plugin-fcm-with-dependecy-updated/issues/293#issuecomment-1377487961

As a matter of fact, for me (running android 13 on samsung S21 FE 5G), if the app is in the background and a push arrives, the app crashes!

We have exactly the same issue. Notifications work fine on older Android versions but causing the app to crash on Android 12, 13 (https://github.com/freescout-helpdesk/freescout/issues/2718)

@thepsion5 Have you found the solution? This solution https://github.com/andrehtissot/cordova-plugin-fcm-with-dependecy-updated/issues/293#issuecomment-1377487961 does not work for us - still no push notifications on Android 13.

The solution definitely works.
After replacing the plugins, have you deleted your platforms and plugins folders and then built again? That needs to happen for the new plugin files to be placed in the platform.

Android platform is removed and re-added. Still push notifications work on Android 9 phone but don't work on Android 13 (notifications are allowed for the app in the phone settings).

cordova plugin remove cordova-plugin-fcm-with-dependecy-updated
cordova plugin add https://github.com/andrehtissot/cordova-plugin-fcm-with-dependecy-updated#v6.3.1    --variable ANDROID_DEFAULT_NOTIFICATION_ICON="@mipmap/ic_launcher"   --variable ANDROID_FCM_VERSION="23.0.0"   --variable ANDROID_FIREBASE_BOM_VERSION="29.0.1"   --variable ANDROID_GOOGLE_SERVICES_VERSION="4.3.4"   --variable ANDROID_GRADLE_TOOLS_VERSION="4.1.0"   --variable IOS_FIREBASE_MESSAGING_VERSION="~> 7.4.0"
cordova platform remove android
cordova platform add<EMAIL_ADDRESS>
Excerpt from config.xml:
    <platform name="android" spec="~8.1.0">
        <allow-intent href="market:*" />
        <preference name="android-minSdkVersion" value="19" />
        <preference name="android-targetSdkVersion" value="31" />
    </platform>
    <engine name="android" spec="^8.1.0" />
    <plugin name="cordova-plugin-fcm-with-dependecy-updated" spec="6.3.1">
        <variable name="ANDROID_DEFAULT_NOTIFICATION_ICON" value="@mipmap/ic_launcher" />
        <variable name="ANDROID_FCM_VERSION" value="23.0.0" />
        <variable name="ANDROID_FIREBASE_BOM_VERSION" value="29.0.1" />
        <variable name="ANDROID_GOOGLE_SERVICES_VERSION" value="4.3.4" />
        <variable name="ANDROID_GRADLE_TOOLS_VERSION" value="4.1.0" />
        <variable name="IOS_FIREBASE_MESSAGING_VERSION" value="~> 7.4.0" />
        <variable name="FCM_CORE_VERSION" value="16.0.9" />
        <variable name="FCM_VERSION" value="18.0.0" />
        <variable name="GRADLE_TOOLS_VERSION" value="3.5.0" />
        <variable name="GOOGLE_SERVICES_VERSION" value="4.2.0" />
    </plugin>

Android platform is removed and re-added. Still push notifications work on Android 9 phone but don't work on Android 13 (notifications are allowed for the app in the phone settings).

cordova plugin remove cordova-plugin-fcm-with-dependecy-updated
cordova plugin add https://github.com/andrehtissot/cordova-plugin-fcm-with-dependecy-updated#v6.3.1    --variable ANDROID_DEFAULT_NOTIFICATION_ICON="@mipmap/ic_launcher"   --variable ANDROID_FCM_VERSION="23.0.0"   --variable ANDROID_FIREBASE_BOM_VERSION="29.0.1"   --variable ANDROID_GOOGLE_SERVICES_VERSION="4.3.4"   --variable ANDROID_GRADLE_TOOLS_VERSION="4.1.0"   --variable IOS_FIREBASE_MESSAGING_VERSION="~> 7.4.0"
cordova platform remove android
cordova platform add<EMAIL_ADDRESS>
Excerpt from config.xml:
    <platform name="android" spec="~8.1.0">
        <allow-intent href="market:*" />
        <preference name="android-minSdkVersion" value="19" />
        <preference name="android-targetSdkVersion" value="31" />
    </platform>
    <engine name="android" spec="^8.1.0" />
    <plugin name="cordova-plugin-fcm-with-dependecy-updated" spec="6.3.1">
        <variable name="ANDROID_DEFAULT_NOTIFICATION_ICON" value="@mipmap/ic_launcher" />
        <variable name="ANDROID_FCM_VERSION" value="23.0.0" />
        <variable name="ANDROID_FIREBASE_BOM_VERSION" value="29.0.1" />
        <variable name="ANDROID_GOOGLE_SERVICES_VERSION" value="4.3.4" />
        <variable name="ANDROID_GRADLE_TOOLS_VERSION" value="4.1.0" />
        <variable name="IOS_FIREBASE_MESSAGING_VERSION" value="~> 7.4.0" />
        <variable name="FCM_CORE_VERSION" value="16.0.9" />
        <variable name="FCM_VERSION" value="18.0.0" />
        <variable name="GRADLE_TOOLS_VERSION" value="3.5.0" />
        <variable name="GOOGLE_SERVICES_VERSION" value="4.2.0" />
    </plugin>

The commands above are not correct. You are removing the plugin, then re-adding the same one, instead of the fork.
I am providing again the commands that will install the fork with the fix. Note that after "github.com" on the url, the username is different:
cordova plugin remove cordova-plugin-fcm-with-dependecy-updated

cordova plugin add https://github.com/marutifh/cordova-plugin-fcm-with-dependecy-updated --variable ANDROID_DEFAULT_NOTIFICATION_ICON="@mipmap/ic_launcher" --variable ANDROID_FCM_VERSION="23.0.0" --variable ANDROID_FIREBASE_BOM_VERSION="29.0.1" --variable ANDROID_GOOGLE_SERVICES_VERSION="4.3.4" --variable ANDROID_GRADLE_TOOLS_VERSION="4.1.0" --variable IOS_FIREBASE_MESSAGING_VERSION="~> 7.4.0"

Not sure about that one. Depends on your setup.
What I do know is, the configuration for this plugin (and most plugins for that matter) should now reside in package.json and not in config.xml
Cordova switched away from putting that stuff in config.xml quite a while ago. I would remove that whole  segment you have mentioned a few comments above from config.xml and I'd make sure my package.json looks ok.
For reference, my package.json, in terms of this plugin looks like this:
   "cordova": {
    "plugins": {
      "cordova-plugin-fcm-with-dependecy-updated": {
        "ANDROID_DEFAULT_NOTIFICATION_ICON": "@mipmap/ic_launcher",
        "ANDROID_FCM_VERSION": "23.0.0",
        "ANDROID_FIREBASE_BOM_VERSION": "29.0.1",
        "ANDROID_GOOGLE_SERVICES_VERSION": "4.3.4",
        "ANDROID_GRADLE_TOOLS_VERSION": "4.1.0",
        "IOS_FIREBASE_MESSAGING_VERSION": "~> 7.4.0"
      }
    },
    "platforms": [
      "android",
      "ios"
    ]
   }

Build is failing:
> Configure project :app

FCMPlugin: Support for Gradle v4 or lower is deprecated. Please upgrade to a newer version.

BUILD FAILED in 5s

/platforms/android/gradlew: Command failed with exit code 1 Error output:
FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring project ':app'.
> Could not create task ':app:processDebugGoogleServices'.
   > Cannot create a proxy class for abstract class 'GoogleServicesTask'.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

Can you please share your versions:

cordova (cordova -v)
cordova-android (cordova platform version android)
gradle (/platforms/android/gradlew -v)

Here is what we have:

phonegap: 8.2.2
cordova-android: 8.1.0
gradle: 4.10.3

After updating nodejs, cordova-android and cordova to newer versions the build succeeded and push notifications are working now on Android 13.


Implement CORS middleware

Resolves #16

Thank you for your contribution, can you take a look to router's global option handler and decide wether to use custom function or internal middleware support?
https://github.com/julienschmidt/httprouter#automatic-options-responses-and-cors

The GlobalOPTIONS handler of the httprouter library is not the thing we're looking for as I understand from this documentation. GlobalOPTIONS is just a common handler for OPTIONS type HTTP requests.
Please let me know if I wrong, I can recheck it so I can make the needed changes.

I just wrote a simple script that makes a simple GET request to our API to test the GlobalOPTIONS handler and it didn't work due to CORS policy. I think my current implementation is fine for now to go.

Thank you for your contribution 👍


Why does the OpenAPI generator add a Record<string, any> to types?

Assuming the following OpenAPI spec
openapi: 3.0.3
info:
  title: Swagger Petstore - OpenAPI 3.0
  version: 1.0.11
paths:
  /pet:
    put:
      operationId: updatePet
      requestBody:
        content:
          application/json:
            schema:
              $ref: "#/components/schemas/Pet"
        required: true
      responses:
        "200":
          description: Successful operation
          content:
            application/json:
              schema:
                $ref: "#/components/schemas/Pet"
components:
  schemas:
    Pet:
      required:
        - name
      type: object
      properties:
        id:
          type: integer
        name:
          type: string
  requestBodies:
    Pet:
      description: Pet object that needs to be added to the store
      content:
        application/json:
          schema:
            $ref: "#/components/schemas/Pet"

the open API generator will generate this type definiton for the Pet schema:
export type Pet =
  | {
      id?: number;
      name: string;
    }
  | Record<string, any>;

I would like to understand the rational behind adding Record<string, any>. For example this will have the side effect that even the properties id and name are of type any because the any-type in Record<string, any> is wider then the type number/string. This also creates further issues if the project uses strict typescript rules regarding the usage of any. E.g. doing an assignment like const foo = response.name it will raise the Unsafe assignment of an any value error.
const response = await openApiClient.DefaultApi.updatePet({
  id: 123,
  name: "super dog",
}).execute({ url: "foo.com " });

const idontexist = response.idontexist // ✅  gives the correct error that this prop does not exist
const name = response.name; // ❌ is of type "any" but should be "string"
const id = response.id; // ❌ is of type "any" but should be "number | undefined"

Is this behavior intended? If so would it make sense to make it configurable to not add the generic Record<...>-type?

Hey @sven-petersen, I will take a closer look at this. It is definitely not intended that the types of your properties have type any. From a quick glance, I can't tell you the ratio behind this decision, I have a suspicion though and will investigate it.

Ok, so the reason this is there at all is that we want to allow to have additional properties, if not stated otherwise in the spec ( aka .additionalProperties: false). However, I assume we merely confused | and &. With the latter you should get correct types. I implemented and tested this here: https://github.com/SAP/cloud-sdk-js/pull/4479.
Once this is merged, we will release a new version soon.


Add quick key inspect cli command

Issue Number
#1389
Overview

[x] I implemented a quick key inspect cli command that extracts the extended private key (64 bytes), and chain code (32 bytes), from an extended private key (96 bytes)

Comments

key inspect does currently not support inspecting public keys.

bors r+

bors r+


PRAGMA auto_vacuum = 1; isn't supported

I got:
→  pragma auto_vacuum;
AUTO_VACUUM
0
→  pragma auto_vacuum = 1;
Error: Failed to execute SQL statement: pragma auto_vacuum = 1;
unsupported statement

Is this intended and if yes, what's the reason?

Only read only pragmas are supported at this point.

Could you describe your use case @55lj please? Maybe we would be able to help.

Sorry for the delay, I'd like to have auto-vacuum like with mysql, just less maintenance. Not sure though if auto-vacuum makes as much sense as with mysql.


I have the same problem.

ERROR in ./src/config/index.js
Module not found: Error: Can't resolve './config.js' in '/home/breadcrumbs-ubuntu/Desktop/test/scdm-client/src/config'
@ ./src/config/index.js 5:13-35
@ ./src/main.js
@ multi (webpack)-dev-server/client?http://<IP_ADDRESS>:8080 webpack/hot/dev-server ./src/main.js
ERROR in ./src/templates/index.js
Module not found: Error: Can't resolve '../config/config' in '/home/breadcrumbs-ubuntu/Desktop/test/scdm-client/src/templates'
@ ./src/templates/index.js 1:13-40
@ ./src/config/index.js
@ ./src/main.js
@ multi (webpack)-dev-server/client?http://<IP_ADDRESS>:8080 webpack/hot/dev-server ./src/main.js

I need more context to help you


Reverse order of DONE column

What
This reverses the order of the DONE column so that the most recently done items are at the top.
How to review
Check out the reversing
Who can review
Not @chrisfarms

Works 👍
Though I'm not clear why this doesn't reverse the order of the other stories? Also are the stories guaranteed to be sorted?

There are multiple calls to FetchStories in that function ... each one keeps some kind of internal state I believe ... so d is only referring to "done" stories fetched.
I'm not 100% about the sort order :/ ... I think it might be updated order rather than created order

Ahh. The internal state, global variables, and short variables names are a bit confusing.


Add it to F-Droid

The points are specified here: https://f-droid.org/docs/Inclusion_Policy/?title=Inclusion_Policy

Mmmm I missed this point: https://github.com/open-coronavirus/open-coronavirus#why-did-we-create-open-coronavirus


"sunsetsha1" handler detecting TLS certificates using SHA-1 expiring during or after sunset period

A "sunsetsha1" handle was added to test for certificates using the SHA-1 signature algorithm and that  expire during or after the Google Chrome SHA-1 sunset period. Changes were made to the mitm daemon and Android client for this handler.
See Chromium blog post for more details:
http://blog.chromium.org/2014/09/gradually-sunsetting-sha-1.html

This request is intended to address issue #60.

This should probably be a data handler with the rest of the TLS check stuff in https://github.com/google/nogotofail/blob/dev/nogotofail/mitm/connection/handlers/data/ssl.py, once https://github.com/google/nogotofail/pull/83 lands you can get the Certificate chain bytes pretty easily and then check in a passive handler.
Also I should add a data handler that handles buffering/parsing TLS records since we're currently duplicating a lot of code. I'll file a blocking issue :)

I changed the sunsetsha1 handler to be a data handler and passive. I also modified it to parse the certificate chain bytes (the new handshake "Certifcate" class) when checking the certs signing algorithm.
I am accessing the Certificate.certificates list to check each cert in the chain of trust. I was hoping to access the certificate attributes using the OpenSSL API attributes (e.g. .get_signature_algorithm()). Is the a way of casting the each cert byte string to a OpenSSL certificate type? Or maybe I'm off track ...
Here's the current handler commit with the relevant line highlighted. Thanks in advance.
https://github.com/mkenne11/nogotofail/blob/241d204cb218e48576509f17c2df5743394e021a/nogotofail/mitm/connection/handlers/data/ssl.py#l130

It's OKI worked out how to cast a certificate byte string to a pyOpenSSL certificate object.
When checking the signature algorithm of each certificate in the chain - do I need to check the algorithm for just the first certificate? Or the signature of the intermediate certificates and root certificate as well?
I assume I only need to check the signing algorithm of the first cert in the certificates list (index 0) which signs the encrypted message - but I'm happy to be told otherwise :)

Probably best to only go with the leaf cert.
On Thu, Sep 10, 2015 at 5:22 AM Michael Kennedy<EMAIL_ADDRESS>wrote:

It's OKI worked out how to cast a certificate byte string to a pyOpenSSL
certificate object.
When checking the signature algorithm of each certificate in the chain -
do I need to check the algorithm for just the first certificate? Or the
signature of the intermediate certificates and root certificate as well?
I assume I only need to check the signing algorithm of the first cert in
the certificates list (index 0) which signs the encrypted message - but I'm
happy to be told otherwise :)
—
Reply to this email directly or view it on GitHub
https://github.com/google/nogotofail/pull/78#issuecomment-139218243.

Thanks Chad. I revised the handler to check only the leaf node for use of SHA-1 during or after the sunset period.
I'll keep an eye out for the data handler that handles buffering/parsing of TLS records to land, then finalize this handler as well the noforwardsecrecy handler (PR # 79).

I changed the sha-1 handler to use the new TlsRecordHandler.
I also included a change to exclude compiled Android client and test harness files (APKs) from Git, by including these in the project .gitignore files. I built the projects in Android Studio and the default build path "app/" where the APKs are created.
It worked fine for me in testing - please let me know if you spot any issues.

Firefox is alerting on intermediate certificates using SHA-1 which expire after the sunset date.
https://blog.mozilla.org/security/2015/10/20/continuing-to-phase-out-sha-1-certificates/
Currently I'm only alerting when the leaf certificate uses SHA-1 after this date. Do you think there is much value in alerting on intermediate certificates as well?

I added alerts for intermediate certificates who use SHA-1 as the signature signing algorithm. I noticed both Firefox and SSL Labs alerts on intermediate certificates.
I raised the intermediate certificate alerts as a WARNING as the risk to the client should be small. I believe for an attacker to exploit this vulnerability they would need to have a cert with the same (SHA-1) hash value as the intermediate cert and convince the root CA to sign it (but I'm not an expert :) ).

Improved variable name consistency.


python311Packages.zha-quirks: 0.0.105 -> 0.0.106

Diff: https://github.com/zigpy/zha-device-handlers/compare/refs/tags/0.0.105...0.0.106
Changelog: https://github.com/zigpy/zha-device-handlers/releases/tag/0.0.106
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 263419 run on x86_64-linux 1

11 packages built:

home-assistant-component-tests.hassio
home-assistant-component-tests.homeassistant_green
home-assistant-component-tests.homeassistant_hardware
home-assistant-component-tests.homeassistant_sky_connect
home-assistant-component-tests.homeassistant_yellow
home-assistant-component-tests.otbr
home-assistant-component-tests.zha
python310Packages.zha-quirks
python310Packages.zha-quirks.dist
python311Packages.zha-quirks
python311Packages.zha-quirks.dist


Enhancement request: make the path completion to complete relative path and not absolute path

Thanks for your work.
I just start using it and I like it.
What it's not fit to my preferences is the path completion.
It's complete, like default vim path completion <C-X><C-F> and it's an absolute path.
I think it's not very useful this absolute path completion.
Example:

one folder named test with one subfolder named subfolder
in subfolder are two files: file-one and file-two
in file-one which is in subfolder, if I type ./ it shows ./subdir/ [path] but I want to see file-one [file] and file-two [file]

Maybe it's possible to have a choice what kind of path completion to use: absolute or relative to current buffer.
Mucomplete has this possibility: mucomplete-buffer-relative-path

Good idea! I added an option bufferRelativePath to path completion. It should work as you suggested.

Works perfectly thank you.

welcome!


Support setting cursor style to vertical bar

Some terminals support escape sequences for switching the cursor from a block to a vertical bar. It would be handy to support this!

Do you happen to know any details about how this is supported? In order to consider adding this to Vty, I need to understand how widespread support for this is, and I'll also need to know whether it's detectable (e.g. in terminfo) and how it behaves in terminals that don't support it.

I realized this is a duplicate of #243, essentially, so I'm going to close it.


Bug in xor/change?

When created a reducer with that looks like model.model.array,
using action.xor('model.array') seems to not get the state from it, but sends change to the right model,
using action.xor('model.model.array') gets the right array for xoring, but then sends action to 'model.array' instead.

What does your store setup look like?

Here is how it's setup on my end:
let defaultQuoteFields = { recipients:[], signatory_id:'' }
const quote_reducers = {
quote: combineReducers({
quote: modelReducer("quote", defaultQuoteFields),
quoteForm: formReducer("quote", defaultQuoteFields)
})
}
const app = combineReducers({
quote: quote_reducers.quote
})
store = createStore(app,  applyMiddleware( thunk, raygun))
model is 'quote.quote.recipients'

Ah, so the model argument of the modelReducer and formReducer are expected to be the exact path of your model. In this case, you should change it like so:
const quote_reducers = {
  quote: combineReducers({
-    quote: modelReducer("quote", defaultQuoteFields),
-    quoteForm: formReducer("quote", defaultQuoteFields)
+    quote: modelReducer("quote.quote", defaultQuoteFields),
+    quoteForm: formReducer("quote.quote", defaultQuoteFields)
  })
}

Thanks


More comprehensive metric validation, align error message nomenclature

Metric validation error messages now list every applicable error message, to prevent the one-error-at-a-time flow we were seeing before, and have change "outcome" to "output" to align with other sources
Addresses https://github.com/trustyai-explainability/trustyai-explainability/issues/271 and https://github.com/trustyai-explainability/trustyai-explainability/issues/247
Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[ ] You have read the contributors guide
[ ] Pull Request title is properly formatted: FAI-XYZ Subject
[ ] Pull Request title contains the target branch if not targeting main: [0.3.x] FAI-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

/test trustyai-explainability-e2e


[Metrics] The unit of some block metrics seems not accurate

I notice the metrics kubevirt_vmi_storage_times_ms_total  value is a very big number, then I check the doc of  libvirt.org, find the description which the time unit is ns:
  "block.<num>.rd.times" - total time (ns) spent on reads as
                           unsigned long long.

https://libvirt.org/html/libvirt-libvirt-domain.html#VIR_DOMAIN_STATS_BLOCK

@ArthurSens  HI, would you pls take a look?

Thanks for finding that, I'll submit a PR changing the metric name.
But I'm wondering if we'd like to follow Prometheus best practices about time units, which would be in seconds. Do we really need time metrics to so precise?
WDYT @sradco?


Change StoreType type from enum to string

https://github.com/Cryptonomic/ConseilJS/blob/master/src/types/KeyStore.ts#L16 should export strings
export const MNEMONIC = 'Mnemonic';
export const FUNDRAISER = 'Fundraiser';

I opened this ticket and #40 as per @yanivkalfa ‘s request. He can help you with your request

No longer in scope.


Made small-scale corrections to documentation (e.g. broken links).

@Cohen-J-Omer welcome! :) Thanks

pushed fix.


Menu bar icon text is hard to read

In contrast, text is not so easy to read, can there be a setting to adjust it individually.

Could you be more specific?

On the left is Itsycal, and on the right is Calendar.
The text of Calendar is too thin to look blurry on the screen. Can you add a setting to adjust the size and thickness of the font?

you mean the left one is Calendr?


IOS SignatureView not found

I keep hitting this issue and the whole page not rendering.
CONSOLE ERROR file:///app/tns_modules/@angular/core/bundles/core.umd.js:5907:24: ERROR Error: Uncaught (in promise): ReferenceError: Can't find variable: SignatureView
Anyone else facing this in IOS that can help?

Not 100% clear based on that error, but can you be sure to enable skipLibCheck in your project's tsconfig.json as you typically don't want to analyze node_modules anyway.


[turborepo] prune's --docker generated "full" folder includes empty node_modules folder for each app

What version of Turborepo are you using?
1.8.3
What package manager are you using / does the bug impact?
pnpm
What operating system are you using?
Linux
Describe the Bug
When i use the command turbo prune --scope=api --docker, "out" folder gets generated. Inside /out/full/apps/api/node_modules, there are all package folders but they are empty.
So when i use the with-docker example for my use case and i copy /app/out/full to my current directory (which contains generated "json" folder with installed dependencies), my node_modules get ovewritten and they are empty so the rest of the build fails:

# Build the project
COPY --from=builder /app/out/full/  .  <------ here
RUN yarn turbo run build --filter=api...

Expected Behavior
"full" folder shouldn't contain app's node_modules so that they dont override json's folder node_modules when copied.
To Reproduce
use the following example with pnpm:
FROM node:alpine AS builder
RUN apk add --no-cache libc6-compat
RUN apk update
# Set working directory
WORKDIR /app
RUN yarn global add turbo
COPY . .
RUN turbo prune --scope=web --docker

# Add lockfile and package.json's of isolated subworkspace
FROM node:alpine AS installer
RUN apk add --no-cache libc6-compat
RUN apk update
WORKDIR /app

# First install the dependencies (as they change less often)
COPY .gitignore .gitignore
COPY --from=builder /app/out/json/ .
COPY --from=builder /app/out/yarn.lock ./yarn.lock
RUN yarn install

# Build the project
COPY --from=builder /app/out/full/ .
RUN yarn turbo run build --filter=web...

FROM node:alpine AS runner
WORKDIR /app

# Don't run production as root
RUN addgroup --system --gid 1001 nodejs
RUN adduser --system --uid 1001 nextjs
USER nextjs

COPY --from=installer /app/apps/web/next.config.js .
COPY --from=installer /app/apps/web/package.json .

# Automatically leverage output traces to reduce image size
# https://nextjs.org/docs/advanced-features/output-file-tracing
COPY --from=installer --chown=nextjs:nodejs /app/apps/web/.next/standalone ./
COPY --from=installer --chown=nextjs:nodejs /app/apps/web/.next/static ./apps/web/.next/static
COPY --from=installer --chown=nextjs:nodejs /app/apps/web/public ./apps/web/public

CMD node apps/web/server.js

Reproduction Repo
No response

I think i managed to fix it with .dockerignore file in root having this line:
**/node_modules instead of node_modules. Still, i dont think empty node_modules should be generated anyway.

@AlexandrosMelissas Thanks I had the same issue and this fixed it for me. I simply ran through the with docker example and converted it to pnpm before encountering this issue.

Still encounter the issue on 2023-12-05.
If dockerignore file in root having this line: **/node_modules instead of node_modules is the solution, please update the example  with-docker of docs


Cannot read property "startLoadWithResult" of null in Expo 49

The library is not working in Expo 49 / RN 0.72.3.

Reproduction steps:
Call useWidget

hi @AlexHayton thanks for reporting this issue this one is due to you have to versions of webview we have published an update to our package at version 2.0.2 to fix this issue , please update yours and must be fixed

Thanks!
On Sun, 28 Jan 2024 at 16:43, eslamelkholy @.***> wrote:

Closed #4
https://github.com/announcekitapp/announcekit-react-native/issues/4 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/announcekitapp/announcekit-react-native/issues/4#event-11620414797,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAPM7WMOCKRCD6UYL4PHHTYQZ53PAVCNFSM6AAAAAA3LJNDZ6VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRGYZDANBRGQ3TSNY
.
You are receiving this because you were mentioned.Message ID:
<announcekitapp/announcekit-react-native/issue/4/issue_event/11620414797@
github.com>


⚠ Samsung has degraded performance

In 501a0f0, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 3196 ms

Resolved: Samsung performance has improved in 8792239.


Release szero 0.6.0

Issue
https://github.com/bucharest-gold/szero/issues/42
Dependencies
None known. This is just an enhancement of szero to provide a extra functionality.

released
https://github.com/bucharest-gold/szero/issues/42#issuecomment-257001632


QF-3714 In Projects admin page, add a owner type filter - by user or by organization

"owner__type" for the filter would suffice functionality wise.
But the filter title would then be "By type" in the UI, which is semantically unclear, hence the SimpleListFilter subclass.

Quick question, can a team be a project owner?
If not, I will remove it from the options.

No, a team cannot own a project. Can you please attach a screenshot on PRs that are screenshotable?

No, a team cannot own a project. Can you please attach a screenshot on PRs that are screenshotable?

Very small style adjusting comment. A valid screenshot in the first message is still required before we can merge this.

done ;-)

Ready to merge!


Update zio-config to 3.0.6

Updates dev.zio:zio-config from 3.0.1 to 3.0.6.
GitHub Release Notes - Version Diff
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!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio", artifactId = "zio-config" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "dev.zio", artifactId = "zio-config" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #300.


Add serialization support to cuML's Array

(Took the liberty of merging in rapidsai/branch-0.13, which was needed for this PR)
Builds on PR ( https://github.com/rapidsai/cuml/pull/1635 ) to provide serialization support for cuML Array objects.
This implements .serialize(...) in cuML's Array by leveraging cuDF's Buffer.serialize(...) for the initial pass. Afterwards keyword arguments needed to construct cuML's Array at the other end are added to the header. Finally the frames are explicitly coerced into Buffer objects for transmission.
Doing this ensures Dask and UCX-Py can transmit cuML Arrays through without issues. While it is possible to implement .deserialize(...), this is not needed as cuML Array inherits cuDF's Buffer.deserialize(...), which handles reconstruction of subclasses internally as long as sufficient information is included (as we have done with "constructor-kwargs" here).
With serialization of cuML Arrays in place, we merely need to register this functionality with Dask. This is done by listing cuML's Array in cuml.comm.serialize as one of the supported classes. As Dask already tries to import this module, we can be sure this registration will happen when cuML and Dask are installed with sufficiently new versions.

Apologies for the somewhat messy history here. I think this is a consequence of merging in rapidsai/branch-0.13, which was needed for the serialization bits ( https://github.com/rapidsai/cuml/pull/1689 ). Only the last 2 commits are really relevant.


Update index.md

add my name.

Job d2l-zh/PR-931/1 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-931/


Move this repo to Google Archive

We need to move this repository to google archive, as we now have better storage samples here, and are using the new google-cloud-php-storage client library instead of the one used in this sample.

Done!


ajax populated fields + externally_triggered + server_side removing fields value

Hi,
First thanks for your work. 0.9 seems to fix most of our issues with yadcf...
A point od detail :
Looks like an external triggering empties all ajax loaded fileds, after the datatable request is done, when server side.
Note I'm not using yadf_data_XX to populate my fileds, but an ajax loading in plugin options.
Tested without externaly triggered function and it works great.
I can provide some test case if needed. Here is an exemple of code used :
yadcf.init(tableWalletHistorique, [ {
column_number: INDEX_EQUIPEMENT_LIEU_UO,
filter_type: "select",
select_type: "select2",
filter_default_label: "Sélectionnez...",
filter_container_id: "tableWalletHistorique_filtre_UO",
select_type_options: {
language: "fr",
templateResult: formatFilterBasic,
ajax:  {
url: "/referentiel/getequipementlieulignelist",
dataType: "json",
method: "post",
delay: 500,
processResults: function ( data ) {
...
}
}
}
} ], { externally_triggered: true } );
// Initialisation du boutton "Filtrer"
$( "#tableWalletHistorique_filtre button[id=trigger]" ).button({
	icons: { primary: "ui-icon-rafraichir" },
	text: true
}).click(function(event){
    // Stop la propagation de l'évènement, pour ne pas déclencher un rafraichissement de la page
    event.stopPropagation();
    event.preventDefault();
    // Lancement du filtrage
	yadcf.exFilterExternallyTriggered( $('#tableWalletHistorique').DataTable() );
});

Thanks!

Correction : the issue occurs ONLY if no rows are return by the datatable request.

Okay, I think I got it.
The fact is yadcf remove fileds options witch are not in datatable result set, after each ajax query.
That is a problem with my use of yadcf, because my fields are all externaly populated.
A solution could be to return in "yadcf_data_XX", the field value selected... except for multi select fields where this functionnality is wanted.
Fields will contain only one option, but will be ajax updtated on user click.
Sorry for opening this useless issue!

Sorry, but its not clear from your issue what the real problem is, please provide step to reproduce with expected results and actual results with a minimal test case (jsbin/etc)

Okay,
Before to try it on jsbin, I'll try the good old screenshot test case.
I take an externaly filtered yadcf (filter button) because the issue is more visible, but same thing happens on "automatic" filtering.
TEST CASE 1 :

http://www.hostingpics.net/viewer.php?id=681372811.png
Here a select is opened and ajax loaded.
Data sent back by ajax request are independant of datatable dataset (the reason is
=> the "AME" option is and should exist in the select, but not in dataset targeted.

http://www.hostingpics.net/viewer.php?id=507193363.png
Here I select the "AME" option and I proceed filtering

http://www.hostingpics.net/viewer.php?id=750457852.png
Here is the final result. 0 items are returned with is normal since AME option is not in dataset targeted.
select filter was emptied by YADCF (because it didn't see "AME" in datatable result I guess).
So the whole filter is empty (just like there was no filtering) but the data displayed are filtered... witch isn't logical, from the user point of vue...

TEST CASE 2 (more vicious) :
In the test case 1, where had an item ID/N° => "50230175", Ligne => "L02" (first one in table).

http://www.hostingpics.net/viewer.php?id=281588843.png
Here I filtered on Ligne "L03", witch exclude item 50230175 from result.

http://www.hostingpics.net/viewer.php?id=474522353.png
Here I add an ID filter on 50230175 (witch is not in dataset since exclude by first filtering) and proceed.

http://www.hostingpics.net/viewer.php?id=508388433.png
Here is the result. Select filter "Ligne" has been emptied.
So if I believe the YADCF/Datatable strictly displays on this view, there is no item 50230175!
Of course I could remember I first filtered on "L03" witch could had exclude 50230175... but since the user is an idiot...
There is no reset filter buttons because of externaly filtered method. But it was automatic filtered, we would have the same result, and a click on reset button of the empty filter "Ligne" would have made appear the 50230175 item... user : Why clear an empty displayed filter change the result?!

The fact is, in some cases, this behaviour is desirable : With Multiselect select2 for exemple, I'm happy a second filtering "removes" mutiselect options with are not in resultset...
That's why I was talking about an filter configuration option to allow the developer to decide if the filter should be update or not, considering the data returned...
Hope this helped, if not I will provide a web exemple...
Thanks!

And there is more.

http://www.hostingpics.net/viewer.php?id=436243771.png
In green, you can see a select filter, ajax loaded with select2 (witch also provides optgroups) using a template allowing to have a different value/text for each option (Epuisement station/gare => HTML equivalence for the exemple)
Here, select2 is in charge, nothing to do with YADCF business.
Filtering uses the option value (not label), on a user hidden column (displayed in green for the needs of the exemple).
Select label is displayed to the user (in red) : User feels he's filtering a label while he's filtering an id... machiavellian.

http://www.hostingpics.net/viewer.php?id=209658972.png
On selection, filter value/label is correct while YADCF is waiting for datatable dataset.

http://www.hostingpics.net/viewer.php?id=596761463.png
YADCF has replaced my label with the ID, witch is data it found in datatable result dataset.
The fact is, in this case, YADCF doesn't have data to "update" the field (has id but would need label, optgroup...).
So here again, an option to tell "Please YADCF, let me handle filter displaying througth plugin options and don't feel concerned by datatable data" may be usefull.

Note for this exemple, I didn't try to return option in "yadcf_data_X".
Thanks,

As a hack, I commented lines 2256 and 2257 :
// $filter_selector.empty();
// $filter_selector.append(column_data);
I need to perform some test, but seems that it fix my issue. I guess that only "solve" something for my use of YADCF (with datatable in server side mode, and independantly ajax loaded filter)...

Okay, time to conclude.
This could be concidered as an issue, or not...
I finally was able to solve the problem, that is, to use YACF with externaly ajax populated selects/multiselect.
The thing is to initialize the filter using the "data" option, giving it an array of every possible value that could be later returned by the ajax request witch will update the field.
In this way, you're sure ajax data you selected will be in the filter, when reloading the page (stateSave selection) or update the datatable.
So, a first dataset is needed for each filter (if data comme from a database, that meens a first request set). The ajax request should return only "options" witch are in the initial data array. Otherwise, issue decribed above could occur.
Great thing that 0.9 actually allows this usage of YACF, I ll try to provide an online usecase of it, before december, for posterity.
Again, thanks for work work!

Hi,
I'm glad you solved that eventually, Indeed you can use the data property to populate the filters, but I must say that using the yadcf_data_N# is the yadcf way to work with server side setup of datatables.
Closing this issue


istio-initializer fails to initialize jobs

istio-initializer fails to initialize jobs. The same job can be properly initialized with istioctl kube-inject which suggests this is a problem with the initializer specifically and not the overall injection template.

Steps to reproduce

kubectl apply -f install/kubernetes/istio.yaml
kubectl apply -f install/kubernetes/istio-initializer.yaml
kubectl apply -f https://raw.githubusercontent.com/kubernetes/kubernetes.github.io/master/docs/concepts/workloads/controllers/job.yaml

The job creation will timeout. The initializer logs show that the apiserver thinks the pod template is invalid.
E1011 16:16:52.833475       1 initializer.go:154] Job.batch "pi" is invalid: spec.template: Invalid value: api.PodTemplateSpec{ /* ... snip ... */}

Discovered in https://github.com/istio/issues/issues/93 but not specific to helm.

Issue moved to istio/istio #1359 via ZenHub


Scrollbar on received status

Scroll bar when we receive message should naturally go down to the latest message. Not sure if this feature is already there since the demo page don't have those.

auto scroll on received message fixed.


[REQUEST] Manually delete proxies on the fly

Is your feature request related to a problem? Please describe.
When I bruteforce a site without having a success key, I get responces in the "to check" tab" sent by proxies that block the site or send errors for some reasons. After several minutes, the "to check" tab is filled by these responces. To avoid that, when I see in the log that a proxy sends such a responce, I stop the runner, switch to proxy page, find the proxy, delete it and restart the runner. It could be easier
Describe the solution you'd like
In the to check tab, when you right click on a responce, add a "delete proxy".
Describe alternatives you've considered
If it's not possible to delete the proxy without stopping the runner, it's not a big deal. Stopping and restarting the runner is not that hard. It's switching the proxy page that is time consuming.

I suggest you put global ban keys for the proxies, it will solve your issue. Sorry but I don't think I'll implement this.

.

Simply make it so you check the source for the page's name or something that the page has, and if it doesn't have it, ban the proxy.


[Feature request] Search and filter

I would love to see a search function and the ability to filter for keywords. E.g. if i want to eat something with potatoes in can filter for this keyword and see all recipes with the keyword potatoe.

UX proposal:
Add Button open up dropdown to the top and select the search button or add button.
Screenshot follows.

Seems good to me. This way the UI will not be cluttered too much.


Is some of this code missing?

Slowly but surely making progress on this awesome project. I am totally new to all this and it has been quite overwhelming. Still, I am pushing on and stoked. I have uploaded everything to my node, but can't see if works because I am missing some choices on my HA states page.
QUESTION: Is it me or is the current version of the Example Home Assistant Configuration.yaml missing a lot of lines? My HA screen is missing an effects list,  on/off switch, and color selection. Can anyone check/ update what is missing please?
this is what I have so far from mixing and matching between the lines Ben has on the original video and the YAML.file on here. Any help/advice/suggestions would be hugely appreciated:
(MQTT stuff. Then:)
light:

platform: mqtt_json
name: "Porch Strip"
state_topic: "bruh/porch"
command_topic: "bruh/porch/set"
rgb_state_topic: "bruh/mqttstrip/setcolorpub"
rgb_comand_topic: "bruh/mqttstrip/setcolor"
brightness_state_topic: "bruh/mqttstrip/setbrightnesspub"
brightness_command_topic: "bruh/mqttstrip/setbrightnes"
effect: true

input_select:
porch_led_effect:
name: "Porch LED Effect"
options:
- bpm
- candy cane
- confetti
- cyclon rainbow
- dots
- fire
- glitter
- juggle
- lightning
- noise
- police all
- police one
- rainbow
- rainbow with glitter
- ripple
- sinelon
- solid
- twinkle
brightness: true
flash: true
rgb: true
optimistic: false
qos: 0
input_slider:
porch_animation_speed:
name: Porch Animation Speed
initial: 150
min: 1
max: 150
step: 10
automation:

alias: "Porch Input Effect"
initial_state: True
hide_entity: False
trigger:

platform: state
entity_id: input_select.porch_led_effect
action:
service: mqtt.publish
data_template:
topic: "bruh/porch/seteffect"
payload: '{"transition":{{ trigger.to_state.state | string }}}'

alias: "Porch Animation Speed"
initial_state: True
hide_entity: False
trigger:

platform: state
entity_id: input_slider.porch_animation_speed
action:
service: mqtt.publish
data_template:
topic: "bruh/porch/set"
payload: '{"transition":{{ trigger.to_state.state | int }}}'

I have been playing with this a ton and made a little progress. Here is the current view and code I have on HA:
What am I missing? Thanks in advance.

light:

platform: mqtt_json
name: "Porch Strip"
state_topic: "bruh/porch"
command_topic: "bruh/porch/set"
brightness: true
flash: true
rgb: true
optimistic: false
qos: 0
effect: true

input_slider:
porch_animation_speed:
name: Porch Animation Speed
initial: 150
min: 1
max: 150
step: 10
input_select:
light_mode:
name: Porch Input Effect
options:
- bpm
- candy cane
- confetti
- cyclon rainbow
- dots
- fire
- glitter
- juggle
- lightning
- noise
- police all
- police one
- rainbow
- rainbow with glitter
- ripple
- sinelon
- solid
- twinkle
automation:

alias: "Porch Animation Speed"
initial_state: True
hide_entity: False
trigger:

platform: state
entity_id: input_slider.porch_animation_speed
action:
service: mqtt.publish
data_template:
topic: "bruh/porch/set"
payload: '{"transition":{{ trigger.to_state.state | int }}}'

Hi @pushscooter,
Nope, everything you need to run this code is in the sample HA configuration in this repo. There was a switch from the MQTT Light component to MQTT JSON Light which is probably what is confusing you. The configuration for the MQTT JSON light is much simpler and involves sending all the commands together in the same string which the NodeMCU can parse and utilize.
The effect dropdown will show up in the card for the light after you turn on the toggle.
Cheers,
Ben

Gotcha, thanks so much for explaining that Ben!
Just one question though. How do I "turn on the toggle"? And what do you mean by "CARD"? (ok,ok, thats two questions...)
I am so close I can feel it! Thanks again for all your help Ben!
Dan
PS: This is what my HA looks like after making the changes you suggested.

I have found the resources that show examples, but I still feel lost (even more so actually). Can you (or anyone) specifically show me what to add to my YAML.file to show the LED on off switch (is that what you mean by toggle?) and the effect list?

Got it, kind of. Thanks soooo much for your help. I set up some groups and a card for this LED strip via your awesome videos, but the effect dropdown still is not showing up. Is there a difference between setting up a toggle and setting up a switch. If so, can you please show what I should add to my YAML to ad a toggle. I have been looking everywhere and can't find it. Here is my YAML file and what my HA looks like:
Thanks again for everything. I know I am sooooo close!!

homeassistant:
mqtt:
broker: m13.cloudmqtt.com
port: 15366
client_id: home-assistant-1
username:
password:
protocol: 3.1
light:

platform: mqtt_json
name: "Porch Strip"
state_topic: "bruh/porch"
command_topic: "bruh/porch/set"
effect: true
effect_list:

bpm
candy cane
confetti
cyclon rainbow
dots
fire
glitter
juggle
lightning
noise
police all
police one
rainbow
rainbow with glitter
ripple
sinelon
solid
twinkle
brightness: true
flash: true
rgb: true
optimistic: false
qos: 0

switch:

platform: mqtt
name: "light"
command_topic: "bruh/porch/switch/set"

input_slider:
porch_animation_speed:
name: Porch Animation Speed
initial: 150
min: 1
max: 150
step: 10
automation:

alias: "Porch Animation Speed"
initial_state: True
hide_entity: False
trigger:

platform: state
entity_id: input_slider.porch_animation_speed
action:
service: mqtt.publish
data_template:
topic: "bruh/porch/set"
payload: '{"transition":{{ trigger.to_state.state | int }}}'

group:
airstream:
name: addressable leds
icon: mdi:truck-trailer
view: yes
entities:
- group.addressable_leds
environment:
name: weather
icon: mdi:weather-windy-variant
view: yes
entities:
addressable leds:
name: addressable leds
entities:
- switch.light
- input_slider.porch_animation_speed

Got it... kind of. Thanks soooo much @bruhautomation  for your help. I set up some groups and a card for this LED strip via your awesome videos, but the effect dropdown still is not showing up.
QUESTION:
Is there a difference between setting up a toggle and setting up a switch. If so, can you please specifically show me the lines I should add to my YAML to add a toggle to access the lights. I have been looking everywhere and can't find it. Just info about switches. And my switch doesn't turn on my LEDs, or offer color selection, or the effects list.
Here is my YAML file and what my HA pages look like:
Thanks again for everything. I know I am sooooo close and so excited to finally make this happen!!

`homeassistant:
mqtt:
broker: m13.cloudmqtt.com
port: 15366
client_id: home-assistant-1
username:
password:
protocol: 3.1
light:

platform: mqtt_json
name: "Adressable LEDs"
state_topic: "airstream/addressable_leds"
command_topic: "airstream/addressable_leds/set"
effect: true
effect_list:

bpm
candy cane
confetti
cyclon rainbow
dots
fire
glitter
juggle
lightning
noise
police all
police one
rainbow
rainbow with glitter
ripple
sinelon
solid
twinkle
brightness: true
flash: true
rgb: true
optimistic: false
qos: 0

switch:

platform: mqtt
name: "on off"
command_topic: "airstream/addressable_leds/switch/set"

input_slider:
addressable_leds_animation_speed:
name: Adressable LEDs Animation Speed
initial: 150
min: 1
max: 150
step: 10
automation:

alias: "Adressable LEDs Animation Speed"
initial_state: True
hide_entity: False
trigger:

platform: state
entity_id: input_slider.addressable_leds_animation_speed
action:
service: mqtt.publish
data_template:
topic: "airstream/addressable_leds/set"
payload: '{"transition":{{ trigger.to_state.state | int }}}'

group:
airstream:
name: addressable leds
icon: mdi:truck-trailer
view: yes
entities:
- group.addressable_leds
environment_view:
name: weather
icon: mdi:weather-windy-variant
view: yes
entities:
- group.environment
addressable leds:
name: addressable leds
entities:
- switch.on_off
- input_slider.addressable_leds_animation_speed
environment:
name: environment
entities:
- sensor.yr_symbol
- sun.sun
`

got it figured out finally. So relieved!!!
The problem was trying to do this from my desktop

Sweet! Glad you got it working. Sorry again for missing your earlier message. The pings I get from GitHub often miss my inbox so I don't see them until I check back. Cheers!


CB-15932 wait 2 mins before trigger start in Environment stop-start tests

More details could be found in the related JIRA item CB-15932 with a root cause analysis why the GCP based environment stop-start tests are failing.
A TODO has been added and this commit will be reverted with the permanent fix.

could you add to the commit message why is it necessary and what would it solve?

thanks for the detailed commit message