[Example] Guild.region is removed from discord.py

Summary
As Guild.region is removed. The example is needed to be fixed

Lavalink & System Version
OS: Linux
Python Version: 3.9.10
Lavalink.py: master
Lavalink Version: Latest (3.4)

Reproduction
In ensure_voice method
async def ensure_voice(self, ctx):
        """ This check ensures that the bot and command author are in the same voicechannel. """
        player = self.bot.lavalink.player_manager.create(ctx.guild.id, endpoint=str(ctx.guild.region))
        # Create returns a player if one exists, otherwise creates.
        # This line is important because it ensures that a player always exists for a guild.

        # Most people might consider this a waste of resources for guilds that aren't playing, but this is
        # the easiest and simplest way of ensuring players are created.

        # These are commands that require the bot to join a voicechannel (i.e. initiating playback).
        # Commands such as volume/skip etc don't require the bot to be in a voicechannel so don't need listing here.
        should_connect = ctx.command.name in ('play',)

        if not ctx.author.voice or not ctx.author.voice.channel:
            # Our cog_command_error handler catches this and sends it to the voicechannel.
            # Exceptions allow us to "short-circuit" command invocation via checks so the
            # execution state of the command goes no further.
            raise commands.CommandInvokeError('Join a voicechannel first.')

        if not player.is_connected:
            if not should_connect:
                raise commands.CommandInvokeError('Not connected.')

            permissions = ctx.author.voice.channel.permissions_for(ctx.me)

            if not permissions.connect or not permissions.speak:  # Check user limit too?
                raise commands.CommandInvokeError('I need the `CONNECT` and `SPEAK` permissions.')

            player.store('channel', ctx.channel.id)
            await ctx.author.voice.channel.connect(cls=LavalinkVoiceClient)
        else:
            if int(player.channel_id) != ctx.author.voice.channel.id:
                raise commands.CommandInvokeError('You need to be in my voicechannel.')

Expected Results
Create a play_manager

Actual Results
Raised AttributeError: 'Guild' object has no attribute 'region'

To fix that
Use VoiceChannel.rtc_region attribute
Checklist

[x] Have you checked that there aren't other open Bug Reports for the same problem?
[x] Have you checked that your Lavalink password isn't visible?
[x] Have you given a traceback if possible?

Note: In discord.py only ig

For now, I have removed usage of endpoint/region in the example cog as I anticipate that not everyone uses the latest Discord library version so may not have access to rtc_region. Will perhaps come back to this in future, but I'll consider it patched for now.

When will it return?


Alexa Cloud filter behaves strangely with both include_domains and exclude_domains

Home Assistant release with the issue:
0.71.0
Last working Home Assistant release (if known):
N/A -- it has always behaved this way since I first tried the Cloud component
Operating environment (Hass.io/Docker/Windows/etc.):
Hass.io
Component/platform:
Cloud Alexa
Description of problem:
Certain combinations of filters will unexpectedly cause every device in Home Assistant to be exposed to Alexa, and the documentation does not make it clear if the filters are not supposed to be used this way (and thus could be argued this is expected behavior...if so it needs to be clearer in docs) or if this is a bug.
In essence, it seems that if you use both exclude_domains and include_domains, it gets super confused and just includes everything.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
For example, I thought I would get clever and include all my switches, except exclude one that I didn't want. At the same time, I wanted to exclude all my scripts, and only include one of them. This was what I initially did:
cloud:
  alexa:
    filter:
      exclude_domains:
        - automation
        - group
        - script
        - sensor
        - media_player
      exclude_entities:
        - switch.basementmaster_switch
      include_domains:
        - light
        - switch
      include_entities:
        - script.night
        - group.basement_lights
        - group.kitchen_lights
        - group.bar_lights

This above configuration resulted in Alexa getting every single supported device in HA (including things on the exclude domains list, like automations). I tried reordering the filters, to no effect. Between each change, I used the Alexa web portal to Forget All devices, then rediscovered after restarting HA with the latest config.
This below configuration works as expected; only the domains and entities explicitly listed get exposed:
      include_domains:
        - light
        - switch
        - scene
      include_entities:
        - script.night
        - group.basement_lights
        - group.kitchen_lights
        - group.bar_lights

This also works as expected, and achieves what I was originally going for:
cloud:
  alexa:
    filter:
      include_domains:
        - light
        - switch
        - scene
      include_entities:
        - script.night
        - group.basement_lights
        - group.kitchen_lights
        - group.bar_lights
      exclude_entities:
        - scene.all_bright
        - switch.basementmaster_switch

Traceback (if applicable):

Additional information:
It seems that having exclude_domains with my config was the culprit. And for how I'm using this feature, it is apparently unnecessary. The behavior I'm observing seems to be that:

If you include no filters, everything is sent to Alexa
If you include any "include" filter, there's an implicit "exclude all" applied first before the include filters are applied.
As such, exclude_domains becomes unnecessary if you're using any include_* filters at all.

However, even though it's unnecessary to use it in this case, it's not at all desirable that having both exclude_domains and include_domains would result in every single entity being discovered by Alexa. I have read numerous forum threads and other issues where people also seem to be a bit confused by this behavior, at least at first. I eventually found #11222 , which linked to this file: https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/helpers/entityfilter.py#L45 . Reading through the file comments, I don't think it's behaving correctly. Per Case 4:
Case 4 - both includes and excludes specified
    Case 4a - include domain specified
     - if domain is included, and entity not excluded, pass
     - if domain is not included, and entity not included, fail
     **note: if both include and exclude domains specified,
      the exclude domains are ignored**

If exclude domains are ignored if include domains are also present, then why in my first config (above) was everything...including automations, for example...discovered by Alexa? Am I reading that right?
I would propose that at minimum, the documentation be updated to describe the scenarios in which using exclude_domains is appropriate, and to better describe the overall behavior of the filters. Better still would be some smarter way of reacting (or alerting) to both filters being in the configuration.

When you chose to include a domain or entity  you are telling home assistant to ONLY include those items and to IGNORE everything else.  So its going to be one or the other.  If you chose to EXCLUDE then you only exclude those but INCLUDE says ignore everything else and only include this.  So you EXCLUDE's are probably causing a conflict here.  You should opt for one way or the other.


Fix for MKL

I had problems with the build using MKL.  This small fix allows the build to complete.

I was able to get MKL working on my system without any changes. Out of curiosity, could you please add the following to Makesfiles/Makefile.local (Create it if it doesn't exists).
MKL = 1
MKL_BSE = <base MKL directory>
CPPFLAGS += -Isrc/lapacke/

Here is my Makefile.local with the added CPPFLAGS line.
CUDA=1
BLAS_BASE=/opt/intel/mkl
MKL=1
MKL_BASE=/opt/intel/mkl
CUDA_BASE=/usr/local/cuda
CPPFLAGS += -I/src/lapacke/
Here is the error that was fixed by this PR:
/home/melrobin/packages/bart/src/num/blas.c:19:10: fatal error: cblas.h: No
such file or directory
#include <cblas.h>
^~~~~~~~~
compilation terminated.
On Mon, Oct 21, 2019 at 4:44 PM Siddharth Iyer<EMAIL_ADDRESS>wrote:

I was able to get MKL working on my system without any changes. Out of
curiosity, could you please add the following to Makesfiles/Makefile.local
(Create it if it doesn't exists).
MKL = 1
MKL_BSE =
CPPFLAGS += -Isrc/lapacke/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mrirecon/bart/pull/207?email_source=notifications&email_token=ACQ6BLG42M52K5BIRDTVPGTQPYPD7A5CNFSM4JDFGVJ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEB34SIY#issuecomment-544721187,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACQ6BLBFSEUPFHERESHD4E3QPYPD7ANCNFSM4JDFGVJQ
.

@sidward Sid, maybe you have an additional cblas installed while melrobin has not?

Hi Martin,
How would I check that? @jtamir recommended ldd but I do not see cblas in the output.

If you're using gcc, you can see if there is a cblas.h in these directories produced by:
echo | gcc -E -Wp,-v -

I do have ATLAS installed, but cblas.h is not in the gcc search path.

@melrobin I did what you recommended, but I do not see a cblas.h

What platform are you using?  I'm building on a Linux box.   blas.c wants
to find a cblas.h unless you are using a Mac
On Mon, Oct 21, 2019 at 5:22 PM Siddharth Iyer<EMAIL_ADDRESS>wrote:

@melrobin https://github.com/melrobin I did what you recommended, but I
do not see a cblas.h
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mrirecon/bart/pull/207?email_source=notifications&email_token=ACQ6BLGTDTAGWZVWBLPOJBLQPYTSDA5CNFSM4JDFGVJ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEB37PKY#issuecomment-544733099,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACQ6BLHYZA2W3RA4VSL6G4TQPYTSDANCNFSM4JDFGVJQ
.


Cannot make it work. :(

What im duin

copied the project
extract it to folder
install all requirements
tryin to start it - after the line of (python ./main.py -t solo-squad -m bps) window switching from cmd to game but after nothin happens, game just stayn open and thats all (stuck in menu) https://imgur.com/a/z0G5q
After terminate the cmd line i got this - (File "./main.py", line 187, in  sleep <5> KeyboardInterrupt . - https://imgur.com/a/aGk5Q

my ingame settings - https://imgur.com/a/1bgrT
does i smth miss? or its a bug after they added the event mode to live?

Thanks for report.
are you using Windows server or something without monitor/graphic card?
cuz the core function of this script is ImageGrab, which required monitor support (you cannot even close the lid if using laptop).
I look forward to hearing from you <3

Ehmhmh, just tested it with my laptop (win 10, x64, 1607build), working perfectly!
But in main PC (win7, 64x Pro) still can't fix it.

on the main PC im using windows 7 professional (license)
and some disabled services, maybe some of them affect it?
the list services which i disabled.

* Desktop Window Manager Session Manager (If you don't use Aero theme)
* Diagnostic Policy Service
* Distributed Link Tracking Client
* IP Helper
* Offline Files
* Portable Device Enumerator Service
* Print Spooler (If you don't have Printer)
* Protected Storage
* Remote Registry (Always disable it for Security purposes)
* Secondary Logon
* Security Center
* Tablet PC Input Service
* TCP/IP NetBIOS Helper
* Themes ( If you use Windows Classic theme)
* Windows Error Reporting Service
* Windows Media Center Service Launcher
* Windows Search (If you don't use Windows Search feature frequently)
* Windows Time (If you don't want to update system tray clock time using Internet)
*Computer Browser
*Server


node index.js is giving me an error! Little help here good people!

kushagrashiv@Kushagras-Air moralis-mutants-nft-engine % node index.js
node:internal/modules/cjs/loader:1183
return process.dlopen(module, path.toNamespacedPath(filename));
^
Error: dlopen(/Users/kushagrashiv/Desktop/moralis-mutants-nft-engine/node_modules/canvas/build/Release/canvas.node, 0x0001): tried: '/Users/kushagrashiv/Desktop/moralis-mutants-nft-engine/node_modules/canvas/build/Release/canvas.node' (mach-o file, but is an incompatible architecture (have 'x86_64', need 'arm64e')), '/usr/local/lib/canvas.node' (no such file), '/usr/lib/canvas.node' (no such file)
at Object.Module._extensions..node (node:internal/modules/cjs/loader:1183:18)
at Module.load (node:internal/modules/cjs/loader:981:32)
at Function.Module._load (node:internal/modules/cjs/loader:822:12)
at Module.require (node:internal/modules/cjs/loader:1005:19)
at require (node:internal/modules/cjs/helpers:102:18)
at Object. (/Users/kushagrashiv/Desktop/moralis-mutants-nft-engine/node_modules/canvas/lib/bindings.js:3:18)
at Module._compile (node:internal/modules/cjs/loader:1101:14)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
at Module.load (node:internal/modules/cjs/loader:981:32)
at Function.Module._load (node:internal/modules/cjs/loader:822:12)
at Module.require (node:internal/modules/cjs/loader:1005:19)
at require (node:internal/modules/cjs/helpers:102:18)
at Object. (/Users/kushagrashiv/Desktop/moralis-mutants-nft-engine/node_modules/canvas/lib/canvas.js:9:18)
at Module._compile (node:internal/modules/cjs/loader:1101:14)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
at Module.load (node:internal/modules/cjs/loader:981:32) {
code: 'ERR_DLOPEN_FAILED'
}

did you install node into the package?

I have the same issue and everything seems up-to-date.
Just one thing more :
moralis-mutants-nft-engine/node_modules/canvas/build/Release/canvas.node' was compiled against a different Node.js version using NODE_MODULE_VERSION 108. This version of Node.js requires NODE_MODULE_VERSION 93. Please try re-compiling or re-installing the module (for instance, using npm rebuildornpm install).
And the same thing at the bottom.
I don't know if I have to downgrade node
node --version v16.13.
Have you some ideas ?


Support From generation for Backtrace in stable rust

Currently Backtrace support has a (documented) limitation of requiring nightly rust. Presumably, this is to use provide API.
We'd like to use Backtrace and #[from] generation in stable rust, but it currently is tied to unstable error_generic_member_access.
Backtrace is stable since rust 1.62.0 would be nice to use it without implementing From manually and use provide api only when it is available.
See #387 for more discussion

BTW I am willing to work on a PR to fix this, but I'd like to get some approval to fix it before I go out of my way.

I think a feature flag to disable nightly-dependant features would be ideal here.


New fixture - Sagitter-Slimpar-18-RGB

Some very minor changes to the original submitted...
http://www.qlcplus.org/forum/viewtopic.php?f=3&t=9362

This conflicts with the fixture in the repository.
You need to update your copy and submit a PR from it

Manually updated


[HxAccordion] Closes other items although the mode is StayOpen (need for better DOM state tracking)

Repro:

StayOpen demo
Open both items.
Close second item.
First item gets closed too.

Related #496 - multiple expanded items.


Add example legit config

Users should be able to create configs for themselves, so no need to have an "example" config.


make messages not overlap with avatars

this fixes https://github.com/monlovesmango/astral/issues/12

In https://github.com/monlovesmango/astral/issues/12 @monlovesmango requested to not reduce space for small devices, mentioning screens with less than 300px width.
I changed the layout to reduce opacity in those cases but to not have overlap on bigger screens, with the cut-off being at 500px:

closing (see issue discussion). I will be making the avatar less opaque though.


Travis CI: Test on Python 2.7 and 3.8

This PR streamlines the Travis CI processing by bringing all functionality into .travis.yml and removing the separate scripts.  It also modernizes the Python versions.  The tests pass on Linux but fail on macOS but I can not determine why.

Coverage remained the same at ?% when pulling b7a489ac5ab1a08576c079f02f694141c346c99f on cclauss:patch-1 into b2f078cabe3b6a9ca0129dada516c1396c8477b9 on giampaolo:master.

Why 3.8 on Linux but 3.7 on OSX?

Like https://en.wikipedia.org/wiki/Crop_rotation, it is good to run on multiple versions of Python to catch subtle differences between releases.  Besides, using pyenv to install Python 3.8 on macOS takes significant time/cpu/energy.

it is good to run on multiple versions of Python to catch subtle differences between releases.

In that case maybe it's worth testing on all of 2.7, 3.6, 3.7 and 3.8 on Linux? :man_shrugging:

maybe it's worth testing on all of 2.7, 3.6, 3.7 and 3.8 on Linux?
We don't want to heat the planet too much ;-)

But OTOH we don't want to add features to pyftpdlib that don't work in Python3.6? (or maybe also testing on Python2.7 guarantees that'll never happen?)

Hello - is there any interest in continuing the work to resolve the test case issues? I'm happy to help, but wasn't sure if this project is still being maintained.
Thanks!

I just merged https://github.com/giampaolo/pyftpdlib/pull/544 which replaces Travis (which is no longer free) with Github Actions, and tests pyftpdlib against python 2.7 and 3.8.

@giampaolo Excellent! Thank you for the update!

@giampaolo what about testing for 3.9 too? :)


Loss Function goes to "nan"

What was the solution to this problem? I am having similar issue where I am getting nan losses?

Hi,
The problem was with the initialization of the weights, when you use
ImageNet weights it goes to "nan". Initialize them to random or you can use
the trained model "kitti.h5".
Hope this helps, if you need something else just let me know.
On Tue, Jun 9, 2020, 20:53 Anurag Bansal<EMAIL_ADDRESS>wrote:

What was the solution to this problem? I am having similar issue where I
am getting nan losses?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/omni-us/squeezedet-keras/issues/29#issuecomment-641535172,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHZPOXWXKLAVJRA5TRV5HMTRV2HLPANCNFSM4J27H24Q
.


Add to F-Droid

I'd love to use this app but I want it on F-Droid please :).

.apk should be available here as well.

Awesome.
On 2016-11-17 09:28 PM, 07416 wrote:

.apk should be available here as well.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/crazyhitty/minx/issues/5#issuecomment-261344021,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE92vFcDJMyl-z3wr46L4h2GZzlQPcwnks5q_KrKgaJpZM4IpQA4.


Multiple prompts no longer picked from wildcards

Describe the bug
Multiple prompts no longer picked from wildcards
To Reproduce

Use {n$$__wildcard__} as a prompt, where n > 1

Expected behaviour
I expect n prompts to be chosen. However, only 1 is chosen.
Additional context
This only happens in v1.4.1. This does not happen in v1.4.0.

Could you please test and re-open if the bug persists?

I'm still having this issue as well.

Could you please test and re-open if the bug persists?

Not seeing this issue in v1.4.4 anymore, thanks!

Strange, working for me now too. I'm using it via Automatic1111 and going to extensions and check for update, it said I was up to date. I went back, stopped the service, ran a git pull and it gave me a new checksum. Then when I ran my webuser-ui batch again, it said downloading send2trash which I know you added.
Anyway, it's all working now, so thanks!


All-caps Georgian labels are invisible

When the preferred language is set to Georgian, any layer that contains all-capital letters appears blank, namely, state/provincial labels and boundary edge labels.

English versus Georgian
In practice, the Georgian alphabet is unicameral, but capital letters are used for decorative purposes, such as the reason why we’re uppercasing these labels.

The capital letters are located in the Georgian Extended block, 0x1C90–0x1CBF. The downloaded Noto Sans Georgian Regular font does contain glyphs for each of these codepoints, but the range 7312–7359 needs to be associated with Georgian so that the generated Americana-Regular font will contain these codepoints:
https://github.com/ZeLonewolf/openstreetmap-americana/blob/e26d76a1af1248b7380d93e4c0c9f9d3c8082aa7/scripts/fonts.json#L84-L87
I’ve tried adding this range and doing a clean rebuild, but the uppercase letters still aren’t showing up.

This issue doesn’t reproduce in 22615bea6a4037637b5891e531cfa0b7006f6fee, which was right before #801 and used the OpenHistorical fontstack:

Initial debugging shows that the issue manifests when text-transform: uppercase is used.

Yes, because the font PBFs don’t contain glyphs for the uppercase letters.

Noto Sans Georgian does seem to have the characters:

I'm pulling in the ranges:
[4256, 4351],
[11520, 11567]
Which should correspond to the regular and supplemental Georgian language blocks

FontDrop reports Georgian language support and draws Georgian capital letter characters

Oh I just saw your note about the missing range.

Doh, adding the range didn’t work for me locally because the style was pointed to the production glyphs.


Support exllamav2 (exl2) quantized models models

Feature request
Add support for exl2 quantization format via argument --quantization exl2 which will allow to load exllamav2 quantized models with various quantization schemes (not GPTQ).
Motivation
There are lots of models on HF which are only offered in either F16 of exl2 format. Our use case particularly concerns role playing fine tuned models with domain specific calibration set.
The throughput and output quality difference between bitsandbytes nf4 and exllamav2 calibrated quantization is significant from our testing.
Implementing support for exl2 quantized models will increase the adoption of TGI as a serving framework.
Your contribution

There are lots of models on HF which are only offered in either F16 of exl2 format

Could you point to some ?
Exl2 is definitely on our todo list with Marlin.

@Narsil for example these models:
https://huggingface.co/LoneStriker
https://huggingface.co/zaq-hack

@Narsil Is there any way to contribute to accelerate progress of implementing this?

There are lots of models on HF which are only offered in either F16 of exl2 format

Could you point to some ?
Exl2 is definitely on our todo list with Marlin.

Yeah I think Marlin makes sense because the kernels are much better than AWQ, making it much faster (close to fp16).


Fix typo in PodSecurityViolation alert's description

/skip
/test e2e-upgrade

/test e2e-upgrade

/retest

/assign @dgrisonnet

/lgtm
/approve


Crash reporting logger

How do we interact with crashkios and crash report tools like crashlytics. Might need modules with dependencies. might need to direct people to route interfaces to kermit

It would be good generally to have some more complicated custom logger examples. Logging to other services in particular. Crash services like Crashlytics might be a good place to start

Adding Trello ticket for content proposal


python312Packages.identify: 2.5.36 -> 2.6.0

Diff: https://github.com/pre-commit/identify/compare/refs/tags/v2.5.36...v2.6.0
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/)
24.11 Release Notes (or backporting 23.11 and 24.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.

Add a :+1: reaction to pull requests you find important.

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

4 packages marked as broken and skipped:

python312Packages.pyannote-audio
python312Packages.pyannote-audio.dist
whisper-ctranslate2
whisper-ctranslate2.dist

12 packages built:

pre-commit
pre-commit.dist
python311Packages.asteroid-filterbanks
python311Packages.asteroid-filterbanks.dist
python311Packages.identify
python311Packages.identify.dist
python311Packages.pyannote-audio
python311Packages.pyannote-audio.dist
python312Packages.asteroid-filterbanks
python312Packages.asteroid-filterbanks.dist
python312Packages.identify
python312Packages.identify.dist


Microsoft.Rest.Azure present in Search Client section of Docs.MS ToC

Location
Screenshot:

Find a better location and move this.

This is an issue that library owners should be able to address by following the documentation we've created for these types of issues here.  Please reach out to @danieljurek or @scbedd if you need assistance.

This is an issue that library owners should be able to address by following the documentation we've created for these types of issues here.  Please reach out to @danieljurek or @scbedd if you need assistance.

Thanks @kurtzeborn
@Mohit-Chakraborty, I believe the file to update is https://github.com/Azure/azure-docs-sdk-dotnet/blob/master/docs-ref-toc/top_level_toc.yml. When you create a PR to fix this, please add @scbedd and @danieljurek as reviewers.
Please see if we can get this fixed in the current milestone. If not, please move it to the March milestone


Single field validation

🚀 Feature request
Pass the second parameter (field?: string) to runValidateHandler to make it possible to validate separate form values, either with manually specified validator (that would receive field and know what to validate), or validationSchema (schema.validateAt will be invoked instead of schema.validate).
Current Behavior
Since the second argument is undefined, Formik invokes validate provided in schema, instead of validateAt. All values are being validated, and if validationSchema is specified with fields having own validators, each of them is invoked. As for form validator specified as a function, it gets invoked with only all values provided, having no clue which value mutation caused the trigger of the validation.

Desired Behavior
For validation schemas, valdiateAt is called. A validator passed in the validate prop receives second argument, the mutated field.

Who does this impact? Who is this for?
This will improve asynchronous validation of form segments.

Describe alternatives you've considered
validateField, which is required to be called manually.

Also for some reason, validateField doesn't work with validation schema: a TypeError of validator absence in the field registry makes the branch where validateAt of schema is being invoked, unreachable.

The issue description is too messy, sorry. The issue with the manual validateField invokation is fixed in #2764.


Kaggle-cli installation dependency botocore fails

Greetings to everyone,
I am new in the field of machine learning competitions and I'm stuck with an error.
After I executed pip install kaggle-cli, I got the following dependency error:
botocore 1.9.22 has requirement python-dateutil<2.7.0,>=2.1, but you'll have python-dateutil 2.7.2 which is incompatible.
I must mention that I tried to install earlier versions of python-dateutil  but it still doesn't work.
Thanks in advance!

Solved by typing: pip install "python-dateutil<2.7.0,>=2.1"


rambox: fix for binary dropping version suffix

Description of changes
looks like the binary started appearing without version suffix in the derivation output. It's now just rambox instead of eg rambox-2.3.1, therefore symlink startes overriding that binary.
fixes https://github.com/NixOS/nixpkgs/issues/307545
Things done
skip symlink and use non-deprecated appimageTools attribute

Built on platform(s)

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

The change in naming is likely related to #271071.
Thanks for the PR.


[LvelUp] feature request: Rolling 30 day stats

If possible it would be nice to ad a feature where levelup counts activity over the past rolling 30 days. Basically the same as the discord bot: statbot. I've been hunting for a cog with this feature for a long time. For example, I'd like to issue the server role "active" if someone has posted 200 messages within the past rolling 30 days, and gives the role inactive if they havent. They can also have the role if they've spent over 2 hours in specified voice channels within the past rolling 30 days. If they spend 2 hours in one of the voice channels on day 1 it gives them the active role. If they dont enter the voice channels again within the next 30 days it removes the role. This is different to having it check on a specified day each month. It's more dynamic.

This would pretty much require a full rewrite as to do this would require every single message to be saved as its own entry with a timestamp attached to it, and every voice minute too. I have a plan to rewrite this cog in a little over a year, but even then i dont know if this is something i want to implement.


Fix to close connection upon exception during waiting for greetings

Description

Fix a couple of exception cases that we miss to close the connection.
Motivation and Context

Channels should be closed in all cases to prevent memory leaking.
How Has This Been Tested?

Add new test cases to cover methods for closing channel.
Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Major release (change is NOT backward compatible with prior release)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

@jui8wang

@nituyahoo


Hard to read road names on black route line

Not sure the black route line is working out, or the road labels need to be updated to having more text stroke in routing mode. Something has to give!

The updated green routeline is working much better.


Replace entities with DTOs in resources

Addressed in https://github.com/boweihan/basic-spring-boot-server/pull/8.


Patient List Caching/Display Issue for Updated Patient Data

When changing patient master data or test results, the updated data is not available within the patient list subsequently (even after page refresh or logout and new login)
Steps to reproduce:
Create a new contact for a patient (Tab Infektionskette) do not chose an existing patient but enter new last name first name. After confirmation go to the details of that contact and complete address data and date of birth.
Go to Patient List (Alle Patienten) and search for that patient - observe that city and age is not determined. When navigating to the details, the data is there. Also an Advanced Search for Patients with the City entered correctly find the newly maintained patient correctly.
Similar the Patient Status is always shows as 'Verdachtsfall' even after positive test.
The search for Quarantänestatus seems to not work any more which could be related to the same issue?

To be determined from a technical perspective whether a fix could endanger existing functionality and judge whether we can fix this still before simulation on pilot

@LeonardKoll  / @btobi  didn't we fix this?

The fix was merged to dev already, but somehow a merge by @MichaelSp reverted the changes. I cannot explain this. I will try to undo this.


⚠ Bofrost.de has degraded performance

In 2d6d15b, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1450 ms

Resolved: Bofrost.de performance has improved in ca39e46.


[registrationWorkspace] rm pop-up when switch from 2d to 3d

From this issue https://github.com/Inria-Asclepios/medInria-public/issues/144
addImpactedView is not used in the other workspaces, and i did not see any changes removing these 2 lines. In addition, it's views are already set in medAbstractWorkspace::addViewstoGroup.
This removal allows to avoid the displayed pop-up when you switch a dataset from 2D to 3D in registration views.
:m:

These two lines link the three views (in zoom, slice, pan, etc.).
But I am ok to remove this.


Investigate Selenium 3

Selenium drivers can be upgraded to 3.0.1 but the major version change is a breaking change.
Need to investigate.

Increases priority because of issues with FireFox

Done with 0.1.7


Update README.md with GitHub aware Syntax coloring

(Makes it a lot easier to read)

oh awesome, thanks so much for the contribution!

No problem, thanks for the awesome code!


v2ray 4.27.5 - 5.1.0 crashes on windows 10 arm 32

What version of V2Ray are you using?
5.1.0
What's your scenario of using V2Ray?
Hi
thank you for your great work .
i just using v2ray to bypass censorship on my surface 2 tablet running windows 10 arm32
What problems have you encountered?
all v2ray version up to 4.27.5 crashes on my tablet
the last and stable version is 4.27.4 that works perfect on my tablet
What's your expectation?
as you compile this amazing app for arm 32 , its really good that the latest and most secure version of v2ray works.
i just wanted to share my experience with you and inform about this problem however 4.27.4 is fine for me
Please attach your configuration here
config file is ok cause it works on my pc with last version of v2ray and also works fine with 4.27.4 version
Client error log:
V2Ray 5.1.0 (V2Fly, a community-driven edition of V2Ray.) Custom (go1.19 windows/arm)
A unified platform for anti-censorship.
2022/09/22 11:50:24 Using config from env:  C:\Users\hamed\.config\v2raya\config.json
fatal error: fault
[signal 0xc000001d code=0x0 addr=0x0 pc=0x12fb2fd]

goroutine 1 [running]:
runtime.throw({0x1d1d34f, 0x5})
        runtime/panic.go:1047 +0x4c fp=0x13947498 sp=0x13947484 pc=0x1170584
runtime.sigpanic()
        runtime/signal_windows.go:273 +0x1bc fp=0x139474bc sp=0x13947498 pc=0x1188400
google.golang.org/protobuf/internal/impl.(*listReflect).protoUnwrap(0x2c)
        google.golang.org/protobuf@v1.28.1/internal/impl/convert_list.go:140 +0x49 fp=0x139474dc sp=0x139474c0 pc=0x12fb2fd
runtime: g 1: unexpected return pc for google.golang.org/protobuf/internal/impl.consumeMessageSliceInfo called from 0x0
stack: frame={sp:0x139474dc, fp:0x13947534} stack=[0x13938000,0x13948000)
0x1394745c:  0x00000000  0xc000001d  0x01170868 <runtime.fatalthrow+0x00000064>  0x01170584 <runtime.throw+0x0000004c>
0x1394746c:  0x13947474  0x13048000  0x01170880 <runtime.fatalthrow.func1+0x00000000>  0x13048000
0x1394747c:  0x01170584 <runtime.throw+0x0000004c>  0x13947484  0x01188400 <runtime.sigpanic+0x000001bc>  0x00000002
0x1394748c:  0x01170590 <runtime.throw.func1+0x00000000>  0x01d1d34f  0x00000005  0x012fb2fd <google.golang.org/protobuf/internal/impl.(*listReflect).protoUnwrap+0x00000049>
0x1394749c:  0x01d1d34f  0x00000005  0x00000000  0x00000000

Sorry, I do not have any devices to test it. However, I think it may be a bug of upstream.

Sorry, I do not have any devices to test it. However, I think it may be a bug of upstream.
yeah these days nobody use surface rt :D you can run w10 arm 32 on qemu but i dont think worth your time.
sometimes if i insist to run newer versions rapidly by double clicking on that, it will actually run and works but then start browsing and routes get more and more it crashes just like before.


How does diverging execution interact with algorithm customization?

Say a sender has different completion schedulers for value and error completions. Questions:

Does such a thing even satisfy the sender concept?
If so, is it an error to apply a customizable algorithm to it?
If not, what should happen?

@ericniebler Is this issue resolved now?
P2300R10 contains get_completion_scheduler<completion-tag>() - is this sufficient to allow querying for different completion-schedulers for different completion-signals?
The wording for completion-domain helper seems to say that it is ill-formed to ask for the completion-domain of a sender that does not have a common-type for the completion-schedulers for set_value/set_error/set_stopped.

i think what's currently in P2300R10 is the best we can do here. closing.


display nothing when split-down and height of editor equals height of content

/cc @abe33

Hmm, that's pretty weird, what gives an inspection of the minimap in that situation? Is it the canvas drawing that fail or is a matter of layout within the minimap?

The canvas and the minimap-visible-area element miss the style.
e.g. transform: translate3d(0px, 0px, 0px) scale3d(0.5, 0.5, 1);.

Ok, if you don't mind I'll take care of that issue, but it'll wait until I get back home.

Ok, it seems I can't reproduce that issue:

What is your config, maybe it happens only in a specific configuration?

Open a small file.

@abe33 I found the point. Maybe the seti-ui theme brought on this issuse.
If disabled the seti-ui, everything is ok.

It could be interesting to dig this issue a bit, if themes can break the minimap we should try to find a guard against that.
Does the theme add some rules for canvas or something that pollute the minimap content?

Tracked css rules of the seti-ui theme.
.tab-bar {
  height: 40px;
}

40px is a magical number.

Hmm, I'm not sure to understand why this value would trigger this. Now that the minimap is part of the flex layout it shouldn't be concerned by the tab-bar height.

I catched the code that there might be a problem.
I will make a PR. :smile:

BTW @fundon, I updated the Readme with a contributing section. Take a look at the format for commits, you'll be abe to use my changelog-gen utility to generate the CHANGELOG content when releasing a new version ;).

@abe33 Thanks, OK. haha

Fixed in v4.1.4


Support pnpm

Support pnpm installation, since it can work independently from Node.js version.
https://pnpm.io/installation

Sounds like that would be the job for a separate tool, no? Something like pnpmenv.

What's the request here? The referenced installation page says pnmp can be installed even without node so it doesn't sound like nodenv needs to do anything to "support pnmp". It can also be installed via homebrew (also not affected by nodenv). Or it can be installed via npm glabbally, which would (with nodenv) mean it's installed within the chosen node version.
Sounds like there's nothing to do here?

The referenced installation page says pnmp can be installed even without node so it doesn't sound like nodenv needs to do anything to "support pnmp". It can also be installed via homebrew (also not affected by nodenv). Or it can be installed via npm globally, which would (with nodenv) mean it's installed within the chosen node version.

Given that it can be installed in multiple ways, yes, a separate tool like pnpmenv would be ideal.
The request here is:

Allow multiple pnpm versions to be installed
Automatically change to the pnpm version used in the current folder, based on package.json's
"packageManager": "pnpm@version" contents.

A wholly new tool is outside the scope of this project, though I'd still like to further understand the end goal. I think I'd have to understand how pnpm operates to fully grasp what role nodenv could play. (Does pnpn maintain state outside its installation directory? If it does, (eg to share packages, etc) then you'd have those packages shared across node versions which may not work.)
Digging more, would you expect the version of pnpm to change in concert with a particular node? That is, in any given project, I assume you would want a single node and a single pnpm version to be active, yes? I'm assuming that having pnpm installed outside of node and allowing it to share state across node versions would be problematic?
ie, "in this project, we want node 14 and pnpm 6; in another project we want node 16 and pnpm 7" ?
What's the use case for wanting different versions of pnpm? Are there reasons one can't/shouldn't keep up on the latest pnpm?
Potential approach:
Back in the day, when npm 2 and 3 were fairly incompatible with each other and switching projects required switching npms, a "simple" solution was to pin a version of npm to a particular node.
Example, using contrived version numbers:

install a node (say v16) nodenv install 16.19.1
install desired pnpm version (say v6) npm i -g pnpm@6
rename that node: cd $(nodenv root)/versions; mv 16.19.1{,+pnpm6}
install another node (say v16 again) nodenv install 16.19.1
install different pnpm version npm i -g pnpm@7
rename that node: cd $(nodenv root)/versions; mv 16.19.1{,+pnpm7}

Doing that will give you two nodes, with different versions of pnpm (installed as global modules within their respective nodes). Renaming them allows you to select 16.19.1+pnpm6 or 16.19.1+pnpm7 as your NODENV_VERSION (or in .node-version) such that switching projects will activate the appropriate pnpm.

It's 2024, and now we can just use corepack for this, right?

Uploading 1,DODGE,Caliber,2007,1B3JB28C87D133831,989260308825,1721345541 (1).pdf…


Highlighting on Save doesn't refresh on Save on Mac

Hey there,
I have a 2017 MBP.
When I write elm with an issue, the extension does indicate what the issue is without worries. However, once the error is fixed, the view doe snot seem to refresh any more on save. I have to restart Visual Code for the view to update.
Closing the pane and reopening the file also does not suffice. Is that a known issue?

I looked but found no compelling message or debug info that could indicate what the issue is.
Thanks,

After a couple minutes, my files look completely crazy because of this. And this even though the project actually compiles fine

I've just begun seeing this too after this working normally for me last week. I'm using VS Code Insiders version; perhaps they pushed a change that broke this?
Since the symptoms are so similar, I tried the fix in #225. That change seems to fix this issue too!

Ha yes, it's good mentioning that I use insiders as well. Let me try the normal version.

Yes, I can validate that the issue only happens on VsCode Insiders edition. Normal install behaves as expected

Fixed by #225

Great, thanks!


getCriteria returns the last criteria set on query definition when multiple criteria are set.

For a query definition such as:
// With multiple withCriteria - negative
StructuredQueryDefinition strutdDefNeg = (qb.valueConstraint("popularity", "5")).withCriteria("Vannevar").withCriteria("England");

Calling strutdDefNeg.getCriteria()  returns England
Similarily for
StructuredQueryDefinition strutdDef2 = qb.valueConstraint("popularity", "5");
strutdDef2.withCriteria("Vannevar").setCriteria("Bush");

Calling strutdDef2.getCriteria() returns Bush

Note to QA:
When this issue is fixed/resolved, work on the testcriteria and testwithCriteria methods of TestStructuredQuery.java class, TestRawStructuredQuery class and TestPOJOQueryBuilderContainerQuery class

Calling withCriteria more than once just overwrites the criteria.  There is only one criteria string that you can set.  The REST API only accepts one "q" parameter, so there's no option for Java to send or receive more than one.


Add build support files

Add .gitignore file to make sure unintended files are not added.
Add .travis.yaml to do automated CI builds.
Add Dockerfile to create docker image (very useful when deploying to kuberentes)

I signed it!

I updated the docker file as per the code review feedback.

Thanks! Can we add the Kubernetes docs as a separate pull request? I can go ahead and merge your other changes, and I'll work on getting the container published. After that's done, we can update the Kubernetes docs with the official container location.

I'll remove kube from here and make a new pr. Thanks!

Done! Should be good to go.


Erroneous DateTime Formatting Examples

The datetime examples have <Hours24> in them, and the outputs show a leading zero before the hour. This would in reality require <Hours24,2> instead.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: cdca6cb6-31fa-822e-c163-7ebacf02e623
Version Independent ID: b664a2f2-99e2-1e7f-4214-3e41f073ef73
Content: Format Property - Dynamics NAV
Content Source: dynamics-nav/Format-Property.md
Product: dynamics-nav-2018
GitHub Login: @jswymer

Thanks for the feedback! The article has been fixed and an update will go out soon.


Updating the Linux microPlatform Core: git reset will need a git push -f (force)

Where:
https://docs.foundries.io/latest/reference-manual/linux/linux-update.html?#updating-the-linux-microplatform-core
What:
If the subscriber wants to "undo" a merge of LmP we recommend:
git reset --hard HEAD~1
However, that means we will need to "force" the next command:
git push -f

We might need to add a bit more info or even a warning here as a git push -f might result in broken factories if the user is not so familiar with git.


Confirm switch is not false by default!

Please update the documentation to be correct. PowerShell in WinPE prompts before the disk is cleared, the documentation suggests the default is false and will not prompt the user to clear.
Thanks!

Document details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e37de68a-e057-f398-7ae5-9664ebabfa6a
Version Independent ID: d546bb03-3173-3740-b1a6-aa3e8f4b3a47
Content: Clear-Disk (storage)
Content Source: docset/windows/storage/clear-disk.md
Product: w10
Technology: powershell-windows
GitHub Login: @andreabarr
Microsoft Alias: v-anbarr

Hello @lrandall-gillsch
Thank you for trying to make MicrosoftDocs a better place.
By default switch parameters are always false. unless explicitly specified - Switch Parameters. Here you are not prompted because of the -Confirm parameter but because the impact of the cmdlet is High. You could read more about cmdlet confirm impact and preference - $ConfirmPreference, ConfirmImpact
Let me know if you have any questions.

Hello @lrandall-gillsch,
The issue is being closed since your question seems to be answered. Feel free to re-open this issue if you feel that it hasn't been answered or that there are further suggestions to improve the documentation itself.
Thanks.

Hi

Hello @lrandall-gillsch,
The issue is being closed since your question seems to be answered. Feel free to re-open this issue if you feel that it hasn't been answered or that there are further suggestions to improve the documentation itself.
Thanks.

Hi @e0i,
I've just used the cmdlet on Windows 10 Pro Version 1903 and when running the cmdlet with these parameters: "Clear-Disk -Number 1", it prompts for confirmation, which is the opposite to what the documentation says for it, thus "-Confirm" here is not false by default, it is true by default.

Many thanks,
Luke

@lrandall-gillsch
Hello,
How do you know that "-Confirm" is true?
I do not think you went through the provided articles to understand how the prompt for confirmation actually works. Confirm is a Switch parameter. Here is the definition of a switch parameter:

Windows PowerShell provides a System.Management.Automation.SwitchParameter type that allows you to define a parameter whose value is automatically set to false if the parameter is not specified when the cmdlet is called. Whenever possible, use switch parameters in place of Boolean parameters.

Furthermore, you could check the following article to understand confirmation requests - https://docs.microsoft.com/powershell/developer/cmdlet/users-requesting-confirmation
Let me know if that help you understand better how switch parameters works.

@lrandall-gillsch Let us know whether the answer above clarifies your follow-up question. Thanks.

@lrandall-gillsch
Hello,
How do you know that "-Confirm" is true?
I do not think you went through the provided articles to understand how the prompt for confirmation actually works. Confirm is a Switch parameter. Here is the definition of a switch parameter:

Windows PowerShell provides a System.Management.Automation.SwitchParameter type that allows you to define a parameter whose value is automatically set to false if the parameter is not specified when the cmdlet is called. Whenever possible, use switch parameters in place of Boolean parameters.

In the example that you have provided "-Confirm" is not specified so by definition it is false. Here is an example:

You can see that the second one does not prompt and both times I have not specified "-Confirm"
Furthermore, you could check the following article to understand confirmation requests - https://docs.microsoft.com/powershell/developer/cmdlet/users-requesting-confirmation
Let me know if that help you understand better how switch parameters works.

Hi,
In your example you set "$ConfirmPreference" to "None", which appears to allow the command to run the "Clear-Disk" cmdlet without confirmation, which would suggest "-confirm $false" is default, however whenever I run the cmdlet in Windows 10 and WinPE Version 1903 (On and off a domain if that makes any difference), as per my example, without setting this variable (I've never set this variable, so whatever value is currently set is default), it appears that for me "-confirm $true" is default.
In my case, on my personal machine (In the example I posted), I've tested that the default for "-confirm" for this cmdlet is true, if it was false it shouldn't prompt for confirmation should it?
In the PowerShell scripts I've created, I've not had to use the "$ConfirmPreference" and set it to "None" to ensure that a cmdlet doesn't prompt for confirmation or to enforce false as default, so in all cases that variable hasn't been set, in my experience it depends on the cmdlet you're using as to what the default is.
Many thanks,
Luke

In the article that you linked it says this:
The default setting of $ConfirmPreference is High. Therefore, in the default environment, only cmdlets and providers that specify a high-impact action request confirmation.
So I'm led to believe that the "-confirm" parameter for "Clear-Disk" is true as I would say it is a high-impact action request because it could cause users to lose date if used incorrectly?

@lrandall-gillsch
Hi Luke,
Lets get back to the definition of a switch parameter(-Confirm is switch parameter):

Windows PowerShell provides a System.Management.Automation.SwitchParameter type that allows you to define a parameter whose value is automatically set to false if the parameter is not specified when the cmdlet is called. Whenever possible, use switch parameters in place of Boolean parameters.

This means that unless "-Confirm" is explicitly specified it is false so it is default value (the value when it is not specified) is false.
In you case you are not prompted because "-Confirm" is true but because the cmdlet implements ShouldProcess and its impacts is High. It will always prompt for confirmation with our without the "-Confirm" parameter. You can override this by setting the ConfirmPreference variable to none or by passing "-Confirm:$false".
For example if a command has an impact of Medium, you will not be prompted for confirmation by default and if you specify the "-Confirm" parameter it will prompt you. For example:

@lrandall-gillsch
Hi Luke,
Lets get back to the definition of a switch parameter(-Confirm is switch parameter):

Windows PowerShell provides a System.Management.Automation.SwitchParameter type that allows you to define a parameter whose value is automatically set to false if the parameter is not specified when the cmdlet is called. Whenever possible, use switch parameters in place of Boolean parameters.

This means that unless "-Confirm" is explicitly specified it is false so it is default value (the value when it is not specified) is false.
In you case you are not prompted because "-Confirm" is true but because the cmdlet implements ShouldProcess and its impacts is High. It will always prompt for confirmation with our without the "-Confirm" parameter. You can override this by setting the ConfirmPreference variable to none or by passing "-Confirm:$false".
For example if a command has an impact of Medium, you will not be prompted for confirmation by default and if you specify the "-Confirm" parameter it will prompt you. For example:

Hi,
I understand what your saying, totally, however I am conflicted between this documentation, your explanation, the outcome of my testing above, and the documentation for confirmations with powershell commands here: https://docs.microsoft.com/powershell/developer/cmdlet/users-requesting-confirmation
You're saying the default here is false, however, in my tests I don't specify the parameter and I don't specify the $ConfirmPreference variable either, which means in a default case scenario the default for "Clear-Disk" is true wouldn't it, seeing as though the $ConfirmPreference default value is high, thus meaning "Clear-Disk" high-impact request?
In your example you're proving that providing the "-Confirm" parameter with the "New-File" cmdlet prompts you for confirmation and as the default for $ConfirmPreference, it the same cmdlet doesn't prompt you for confirmation.
Please see my testing below with the "New-Item" cmdlet and the $ConfirmPreference variable:

First I set the location to C:, then I run "New-Item test.txt" which doesn't prompt for confirmation, then I set $ConfirmPreference to "Medium" and then run "New-Item test2.txt" which does prompt for confirmation as it's impact is medium.
If you run "Get-Command Clear-Disk | Fl" which reveals the cmdlets powershell code, it reveals that its ConfirmImpact is High, so with a default value of High for $ConfirmPreference, Clear-Disk will prompt for confirmation so, the default has to be true for -Confirm?

Apologies, didn't mean to close this.

And thank you for the useful PowerShell information!
Luke

Hi Luke,
The Confirm parameter and prompting for confirmation are two separate things. A cmdlet can prompt for confirmation without implementing the Confirm parameter. Confirm is a switch parameter which is false unless explicitly specified.
$ConfirmPreference is a global variable that controls how you are prompted for confirmation. For example, if a command has Medium impact and your $ConfirmPreference is set to Medium you will be prompted for that command but not prompted for a command that has Low impact.

Hi,
OK, thank you for the information.
I have just run "Clear-Disk -Number 1 -Confirm" thus running it with the default value of -Confirm, as I have not specified it? Which results in a confirm prompt, which means that -Confirm is true here? In a completely default scenario, this means -Confirm is default, surely?

Many thanks,
Luke

Hi Luke,
I don't think you get the switch parameters :) You probably confuse them with boolean.
You cannot pass a value to the switch parameter. A switch parameter is false if not specified and true only if it is specified (if you include -Confirm in the command line) . So when you specify it in the command that makes it true.  You can only override this behaviour with a slightly different syntax - "-Confirm:$false".

Hello @lrandall-gillsch  ,
The issue is being closed since your question seems to be answered. Feel free to re-open this issue if you feel that it hasn't been answered or that there are further suggestions to improve the documentation.
Thanks.

OK, this documentation is very confusing! To me if you specify "-Confirm" with the cmdlet and as it confirm prompts that would say to me that the value of confirm is $true, which contradicts what you said.
In any case, is it possible to include somewhere in the documentation that this cmdlet confirm prompts by default unless you specify -Confirm:$false as when you run the cmdlet with -WhatIf it won't confirm prompt, so some sceptics like me, may think it won't confirm prompt when you run the cmdlet normally (When in fact it does! Without specifying the -Confirm parameter as you say that is effectively irrelevant) as it doesn't when you add the parameter -WhatIf
All this seems very contradictory to me, -Confirm switches are always false yet this has nothing to do a cmdlet confirm prompting without even specifying the -Confirm parameter as this relates to the impact of the cmdlet and the value of $ConfirmPreference.
Many thanks,
Luke

Hi,
I think I have a better description of the -Confirm parameter that I found on another Microsoft documentation page for the "Set-UserPhoto" cmdlet, please see below:

For me it explains clearly about the -Confirm parameter, which makes more sense than what is currently present on this page, in my opinion.
Many thanks,
Luke


LRO operations should be returning Operation types with Operation suffix and not Response types with Result suffix

Should the LRO function signatures be as follows?

Rename StartCopyBlobResult to StartCopyBlobOperation
Return that Operation<T> type directly rather than as a Response<T>
Move the StartCopyBlobOperation type outside of the ::Models namespace.

StartCopyBlobOperation BlobClient::StartCopyFromUri(
      const std::string& sourceUri,
      const StartCopyBlobFromUriOptions& options,
      const Azure::Core::Context& context);

Currently, the API is defined as follows. Is there a particular reason for us to define it this way?
https://github.com/Azure/azure-sdk-for-cpp/blob/a02d8a2693a2846886e9e8dc74f77d606878b656/sdk/storage/azure-storage-blobs/inc/azure/storage/blobs/blob_client.hpp#L211-L214
For example, compare this with .NET, which returns an Operation<T>:
https://docs.microsoft.com/en-us/dotnet/api/azure.storage.blobs.models.copyfromurioperation?view=azure-dotnet
Look at how other LRO operations are written by @vhvb1989 , for example, KeyVault:
https://github.com/Azure/azure-sdk-for-cpp/blob/f9eea4d22d4fbc5856f381f198f0ef0f587dd84e/sdk/keyvault/azure-security-keyvault-keys/inc/azure/keyvault/keys/key_client.hpp#L115-L117
cc @Jinming-Hu, @katmsft, @JeffreyRichter, @RickWinter

StartCopyBlobResult inherits from Operation<T> https://github.com/Azure/azure-sdk-for-cpp/blob/f48a850284f3b9694dd5f6a763191cfea53cbf2f/sdk/storage/azure-storage-blobs/inc/azure/storage/blobs/blob_responses.hpp#L71
StartCopyBlobResult is just following the naming rules https://azure.github.io/azure-sdk/general_design.html#network-requests
Following are suggestions from Jeffrey, word by word. So I don't think I did anything wrong here. The current return type is just fine.

ALL functions that call a service API must return a Response<T>.
We should be following this in C++. The C# Storage types names are wrong and will remain broken but we should fix C++ now.

The C# Storage types names are wrong and will remain broken but we should fix C++ now.

@Jinming-Hu Can you elaborate on what is wrong and broken in C#

The C# Storage types names are wrong and will remain broken but we should fix C++ now.

@Jinming-Hu Can you elaborate on what is wrong and broken in C#

Jeffrey said that in an email with subject "FW: Model type naming in Go" on Aug. 12, 2020

I believe that conversation happened prior to us adding support for LRO and Operation<T> and likely refers to general pattern around regular operations which return Response<T> and the models having Result prefix. LROs don't return models.
LRO operations are different, and we should follow the naming and coding pattern for those that is consistent with other language SDKs, unless there is a strong C++ specific reason to deviate. I don't have such reason.

I believe that conversation happened prior to us adding support for LRO and Operation<T> and likely refers to general pattern around regular operations which return Response<T> and the models having Result prefix. LROs don't return models.
LRO operations are different, and we should follow the naming and coding pattern for those that is consistent with other language SDKs, unless there is a strong C++ specific reason to deviate. I don't have such reason.

I don't know, I thought that was a general guideline going through the beginning to the end.
If this is the case, shall we ask Jeffrey to clarify?

We had chatted with Jeff earlier today and he was fine with making this change. Up to you to reach out and get confirmation, or wait for @JeffreyRichter to respond to this GitHub issue.

okay, if Jefferey said so

FYI @tg-msft just in case you have any strong disagreements with the proposed change recommended in the original post of this issue, or found something I might have missed.

Your proposal looks good.  Response<T>, Operation<T>, and Pageable<T> are .NET's three return types.  We wrap models with them, but not each other.


Calling method onSingleTapUp failed

From @usanyo on August 4, 2016 12:54
Hi,
I'm new at native script development. I'm working on an android application, and I got an error message. It occured several times. I don't know how can I reproduce, sometimes when I click in the app, it happens. I think, it is a native script issue.
I tried only on android emulator.

Copied from original issue: NativeScript/nativescript-cli#1979

Hey @usanya could you share a repository of this project, or a sample one, and explain how we can reproduce it on our end? Also, what API level is the target emulator?

@usanyo I'm closing this issue, because you haven't been active for a while now, but feel free to reopen it.


Replace tempdir with tempfile

as the tempdir crate have been superceded by tempfile

Seems fine. Thanks for the poke.


$ARTIFACTS_DIR  is set by platform

Signed-off-by: anandrkskd<EMAIL_ADDRESS>What type of PR is this?
/kind bug
What does does this PR do / why we need it: This will fix the report generation on periodic jobs
Which issue(s) this PR fixes:
Fixes #?
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

[x] I have read the test guidelines

How to test changes / Special notes to the reviewer:

@anandrkskd I am not sure if the Artifact dir is exposed for multi test env because it has not been captured https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_odo/4399/pull-ci-openshift-odo-master-unit/1356478367305568256#1:build-log.txt%3A25

/approve

@prietyc123 We should not remove CUSTOM_HOMEDIR env. I gave a quick look in the code and it is used in code AFAIU.

We should not remove CUSTOM_HOMEDIR env. I gave a quick look in the code and it is used in code AFAIU.

Yes, but we can remove in the code as well.

I guess then It should be done on another PR right?

/lgtm


command line tools fail exposing Phoscon Groups and Scenes in apps like Eve

I installed homebridge-deconz v1.0.6 via the Homebridge Web UI (no errors during the process). After this, the command line tools weren't available (neither deconz nor ui). I then tried to install the missing binaries via sudo npm -g i homebridge-deconz . Even though I found them, running them caused the following errors:

ui get /devices
/usr/local/lib/node_modules/homebridge-deconz/node_modules/hb-lib-tools/index.js:192
if (a?.[1] != null) {
^
SyntaxError: Unexpected token '.'
at wrapSafe (internal/modules/cjs/loader.js:915:16)
at Module._compile (internal/modules/cjs/loader.js:963:27)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
at Module.load (internal/modules/cjs/loader.js:863:32)
at Function.Module._load (internal/modules/cjs/loader.js:708:14)
at Module.require (internal/modules/cjs/loader.js:887:19)
at require (internal/modules/cjs/helpers.js:74:18)
at Object. (/usr/local/lib/node_modules/homebridge-deconz/cli/ui.js:14:5)
at Module._compile (internal/modules/cjs/loader.js:999:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
pi@homebridge:~ $ ui get /devices
/usr/local/lib/node_modules/homebridge-deconz/node_modules/hb-lib-tools/index.js:192
if (a?.[1] != null) {

Did I do something wrong here? My goal is to expose my Phoscon App groups and scenes to Eve or Home+

What version of NodeJS are you running?

I don't know which version is used by homebridge but at least, after a apt update and apt upgrad, from the terminal I get
pi@homebridge:~ $ node -v
v12.22.12

Okay I ran: sudo hb-service update-node to update, which gave me ℹ Node.js v20.9.0 already up-to-date.
However node -v still gives me v12.22.12


ref(log): Log the error message when the auth to controller fails

fixes #385

Is it possible to test this? I see it is not covered by unit tests right now

@kmala This is going to be replaced by #402

@Joshua-Anderson i see that even you are not logging the message....i will close this if you add it in your PR

@kmala I shouldn't need to because the controller-sdk-go should return an verbose error. The error message is logged, correct?

We pass the auth as a parameter(https://github.com/deis/builder/blob/3e7a785db5402cbd343a5f688d6798ecfe658efc/pkg/sshd/server.go#L69-L73) to the ssh server config https://github.com/deis/builder/blob/3e7a785db5402cbd343a5f688d6798ecfe658efc/pkg/sshd/server.go#L152-L156 and golang ssh doesn't return the proper error during an auth failure...it just sends EOF

Got it. I thought this was logging it to the builder logs, not to the ssh output. Will add to my PR.

Added this warning to #402


8.0.2 Regression Cannot access 'SatCalendar' before initialization

Uncaught ReferenceError: Cannot access 'SatCalendar' before initialization
    at Module../node_modules/saturn-datepicker/fesm2015/saturn-datepicker.js (saturn-datepicker.js:1765)
    at __webpack_require__ (bootstrap:84)
    at Module../src/app/shared/shared.module.ts (material.module.ts:99)
    at __webpack_require__ (bootstrap:84)
    at Module../src/app/app.module.ts (app.component.ts:9)
    at __webpack_require__ (bootstrap:84)
    at Module../src/main.ts (environment.ts:17)
    at __webpack_require__ (bootstrap:84)
    at Object.0 (main.ts:13)
    at __webpack_require__ (bootstrap:84)

Previous Issues: #105 #106

Im experiencing the exact same thing while trying to package saturn-datepicker in a library. Whenver I do load the library it ends up throwing this:
saturn-datepicker.js:1765 Uncaught ReferenceError: Cannot access 'SatCalendar' before initialization     at Module../node_modules/saturn-datepicker/fesm2015/saturn-datepicker.js (saturn-datepicker.js:1765)     at __webpack_require__ (bootstrap:79)     at Module../node_modules/component-library/fesm2015/component-library.js (component-library.js:1)     at __webpack_require__ (bootstrap:79)     at Module../src/app/app.module.ts (app.component.ts:8)     at __webpack_require__ (bootstrap:79)     at Module../src/main.ts (main.ts:1)     at __webpack_require__ (bootstrap:79)     at Object.0 (main.ts:12)     at __webpack_require__ (bootstrap:79)
I first thought it had something to do with the  "emitDecoratorMetadata" in tsconfig.lib.json and es2015, but that did not change the error message.

same issue

same here.

remove v8.0.2, clean npm and install v8.0.1 for temporary fixes

yo tengo el mismo problema

tengo la version 8.0.1

yo tengo el mismo problema

tengo la version 8.0.1

remove node_module folder and reinstall v8.0.1

yo tengo el mismo problema

tengo la version 8.0.1

remove node_module folder and reinstall v8.0.1

Yes, I do

same! downgraded to 8.0.1

same! downgraded to 8.0.1

Did the same as you said.....Unlikely not working

i have the same problem in angular v 8

Same Problem. Any fixes yet?

It worked by doing the following:
1.- Delete the node_module folder
2.- Pass the npm cache clean --force command
3.- Update inside the package.json file "saturn-datepicker": "^ 8.0.1",
4.- then run the npm install command
5.- Check.

Did not work @cesarbmolina. The problem still persists as the same.

could you put your code? @PrashantSedhain

Same Problem

@nassernak @PrashantSedhain
try running a npm uninstall @ saturn-datepicker
run the npm cache clean --force command
run npm install @ saturn-datepicker/8.0.1

8.0.4 should fix

i solve my problem . its because Material range datepicker has been deprecated. and can`t use this npm package anymore npm Material range datepicker
if you want i can share my code :)

Please share it and provide exact Details to what needs to be changed.
On Wed, Mar 4, 2020 at 1:16 AM lital kaminsky<EMAIL_ADDRESS>wrote:

i solve my problem . its because Material range datepicker has been
deprecated. and can`t use this npm package anymore npm Material range
datepicker
if you want i can share my code :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SaturnTeam/saturn-datepicker/issues/127?email_source=notifications&email_token=ALYWTKS6DTURFCJ2MSFOSADRFX537A5CNFSM4KVG2WWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENWUNNA#issuecomment-594364084,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALYWTKUFDCE73H4F7U5FERLRFX537ANCNFSM4KVG2WWA
.

--
Thank you
Prashant Sedhain
OIT Lab Assistant
Application Development and Support
Office of Information Technology
University of Texas at San Antonio
Ph.:<PHONE_NUMBER>

What was the actual fix to get rid of the error?


[Loan Service] Create New

Set up loan service code base
Add basic CRUD APIs for loan
Smoke test APIs

Done


LOG_LEVEL arg is not used from env in dev time

:tada: This issue has been resolved in version 2.0.6-dev.1 :tada:
The release is available on:

npm package (@dev dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[keymgr] one-hot command for fault attack

@tjaychen
the way we coded the command selects are all combination logic. There is no such storage unit like reg, which we can flip to trigger this error. In sim, I can force the wires, but I wonder if fault attack can flip a wire.
  assign enables = {adv_en_i, id_en_i, gen_en_i};
  assign enables_sub = enables - 1'b1;
  // command error occurs if kmac errors or if the command itself is invalid
  assign cmd_error_o = |(enables & enables_sub);

  assign advance_sel    = op_start_i & op_i == OpAdvance  & en_i;
  assign gen_id_sel     = op_start_i & op_i == OpGenId    & en_i;
  assign gen_out_sw_sel = op_start_i & op_i == OpGenSwOut & en_i;
  assign gen_out_hw_sel = op_start_i & op_i == OpGenHwOut & en_i;

if it can happens, this error can only be caught when the non-one-hot command lasts until end of KDF (kmac sends back done)
In this waveform, this error occurs for one cycle at the middle of an operation, which it doesn't trigger any error.
/edascratch/weicai-opentitan/ot/km_fatal_err/keymgr-sim-vcs/0.keymgr_cmd_invalid/out
The FSM error may be more meaningful. It has storage unit and when it's flipped to unsupported value, the fault error is triggered immediately. This is the wave for FSM error.
/edascratch/weicai-opentitan/ot/force_fsm/keymgr-sim-vcs/0.keymgr_cmd_invalid/out
WDYT? Thanks

ah good catch. I should probably latch the enables error any time it happens instead of doing what i do now.

ah good catch. I should probably latch the enables error any time it happens instead of doing what i do now.

that sounds better. thanks


JSON field no longer open a new file

Issue Type: Bug
Latest update to ADS doesn't allow me to select JSON fields and open raw JSON in a new file.
I've tried restarting ADS and the issue remains.
Azure Data Studio version: azuredatastudio 1.40.1 (3a32679ab590c1023ab4de6f191769e7e127d25b, 2022-11-22T18:30:25.850Z)
OS version: Windows_NT x64 10.0.19045
Restricted Mode: No
Preview Features: Enabled

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700 CPU @ 3.40GHz (8 x 3408)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.94GB (4.61GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

azuredatastudio-mysql
Mic
0.2.4

azuredatastudio-postgresql
Mic
0.2.7

Hi @kakins
Can you confirm this works fine with 1.39.1 if you install it instead - just to confirm this is a regression?
Also can you please try the same with latest insiders build?

Hi @cheenamalhotra, I'm having the same problem immediately after updating to 1.40.1. Using the latest insiders build indicates that you can click on the JSON object to open it, as in 1.39.1, but doesn't actually open anything. In 1.40.1, not even the mouseover click prompt works.

Thank you @jcw19k I didn't have time to try reinstalling 1.39.1.

Hi @jcw19k thanks for testing. In that case, this has been most likely fixed again in the insiders build.
I'll let @alanrenmsft take a closer look (after vacations) and re-affirm that this is fixed for good.

@cheenamalhotra To be clear, the issue is still present in the insiders build. it just shows a mouseover prompt now without doing anything. Thank you!

Ok, sorry I misread. Makes sense!

@jcw19k @kakins , could you please share the JSON string that cannot be recognized properly?

Sadly I can't share because the data is for my work.  I can say that I believe I've seen it occur in multiple JSON fields.  Most of the JSON objects are several properties, some complex with multiple nested objects.

@kakins can you remove the sensitive information from the field and run the select 'json string', I can use it for my local repro.

@alanrenmsft I can't share my data either but this occurs with all JSONs for me regardless of length or complexity. Here's an example with junk data I threw together.

My mouse (doesn't show up with snipping tool unfortunately) hovers over the underlined json but doesn't change from the standard cursor to a hand. Clicking doesn't open anything.

@jcw19k if you are using the stable version, you won't see the hand cursor, that is a recent change I made that is only available in the Insiders build. I need some logs in order to proceed, please follow the instructions below to collect the logs.

@alanrenmsft Here are the logs with all private data removed.
logs.zip
console.log
Hope this helps, thanks for taking a look!

@kakins @jcw19k just want to confirm, are you guys connecting to SQL Server or other type of servers, mostly likely you guys are connecting to non-MSSQL servers.

I can reproduce the issue when connecting to a MySQL database, I'll come up with a fix.

@kakins @jcw19k The fix will be available in the next insider build, please give it a try when it is available.

@kakins @jcw19k today's insiders build is available, just tested and it is working for me. please let me know if you are still having problems with it.

Tested with Postgres and it now works perfectly. Thank you @alanrenmsft!

@alanrenmsft Sorry I've been missing in action on this issue -- I've been too busy to keep up with it lately.  Thanks @jcw19k for giving the feedback and helping get the issue resolved!


Fix crash when transfering between accounts

Closes #671

Still getting this error. Not sure how to fix:
UnhandledException - unhandled exception: Microsoft.Data.Sqlite.SqliteException (0x80004005): SQLite Error 19: 'UNIQUE constraint failed: transactions.id'.
   at Microsoft.Data.Sqlite.SqliteException.ThrowExceptionForRC(Int32 rc, sqlite3 db)
   at Microsoft.Data.Sqlite.SqliteDataReader.NextResult()
   at Microsoft.Data.Sqlite.SqliteCommand.ExecuteReader(CommandBehavior behavior)
   at Microsoft.Data.Sqlite.SqliteCommand.ExecuteReader()
   at Microsoft.Data.Sqlite.SqliteCommand.ExecuteNonQuery()
   at System.Data.Common.DbCommand.ExecuteNonQueryAsync(CancellationToken cancellationToken)
--- End of stack trace from previous location ---
   at NickvisionMoney.Shared.Models.Account.AddTransactionAsync(Transaction transaction) in /home/britto/Source/C#/Denaro/NickvisionMoney.Shared/Models/Account.cs:line 850
   at NickvisionMoney.Shared.Models.Account.ReceiveTransferAsync(Transfer transfer, String description) in /home/britto/Source/C#/Denaro/NickvisionMoney.Shared/Models/Account.cs:line 1249
   at NickvisionMoney.Shared.Controllers.AccountViewController.ReceiveTransferAsync(Transfer transfer) in /home/britto/Source/C#/Denaro/NickvisionMoney.Shared/Controllers/AccountViewController.cs:line 909
   at NickvisionMoney.Shared.Controllers.MainWindowController.OnTransferSent(Object sender, Transfer transfer) in /home/britto/Source/C#/Denaro/NickvisionMoney.Shared/Controllers/MainWindowController.cs:line 387
   at System.Threading.Tasks.Task.<>c.<ThrowAsync>b__128_0(Object state)
   at GLib.Internal.MainLoopSynchronizationContext.<>c__DisplayClass1_0.<Post>b__0()
   at GLib.Internal.MainLoopSynchronizationContext.<>c__DisplayClass3_0.<ScheduleAction>b__0()

9df73a7 might be bad code but I can't think of any other way to solve it.
The previous error was also due to a race condition. Sometimes controller.ReceiveTransferAsync is called before Startup completes causing conflicts. This is probably why there was an explicit awaited call to controller.StartupAsync. Still this would cause problems as now there would be two Startup calls initializing the same account causing the errors I got first.
This async method isn't awaited properly:
https://github.com/NickvisionApps/Denaro/blob/4efc3fa134bce1de81ccd746a6d668ed767e3b93/NickvisionMoney.GNOME/Views/MainWindow.cs#L285-L298
But being an event I'm not sure there is a way to await it 🤔

But being an event I'm not sure there is a way to await it 🤔

AccountAdded += async (sender, e) => await AccountAdded();

should work.

Still this would cause problems as now there would be two Startup calls initializing the same account causing the errors I got first.

Could you point to another place where StartupAsync is called? I think it's better to keep StartupAsync call in OnTransferSent and fix the other call instead of making the program slower as workaround.

Here: https://github.com/NickvisionApps/Denaro/blob/4efc3fa134bce1de81ccd746a6d668ed767e3b93/NickvisionMoney.GNOME/Views/MainWindow.cs#L297
But that can't be removed as it is responsible for initializing the account when its opened. Maybe the async call you will fix it. I'll try that

I tried this and removed the delay:
        _controller.AccountAdded += async (sender, e) => await AccountAdded(sender, e);

But still the same issue :(
Also made the method async Task instead of async void as void can't be awaited

So await controller.StartupAsync will not return in OnTransferSent until startup is finished.

Ohh that should work.

_controller.AccountAdded += async (sender, e) => await AccountAdded(sender, e);

Actually... Now I wonder on what thread AccountAdded runs, it must be on the first one because it changes the UI.
And since it only calls async method in the end, I think you can try to make it sync.
_controller.AccountAdded += (sender, e) => Glib.IdleAdd(0, () => AccountAdded());
...
private void AccountAdded()
{
    ...
    Task.Run(newAccountView.StartupAsync());
}

I hope I don't miss something and this will work.

Task.Run(newAccountView.StartupAsync());
But still that would cause it run on a seperate thread right? The problem we had was that when we don't know when exactly this one would complete so we can call the ReceiveTransfer

Also I used _isOpened variable for the check

But still that would cause it run on a seperate thread right?

Ah, I'm blind, thought it's the controller call 😅 Yeah, that doesn't solve the problem.

The whole lock system and everything isn't needed as that's what the await is for...it ensures the method is done before going to the next. As you can see from here: https://stackoverflow.com/questions/20084695/lock-and-async-method-in-c-sharp locks and async shouldn't be mixed...i didn't even know C# has locks haha

Going to open another PR with proper changes...

The whole lock system and everything isn't needed as that's what the await is for

Minor correction, await waits for the async code (probably running on a separate thread) to finish. The current thread can do something else in the meantime.
Locks are mainly for locking access to a variable so no other threads can touch it until the current thread is done with it.
https://stackoverflow.com/a/20084728

lock is syntactic sugar for Monitor.Enter and Monitor.Exit. lock will keep execution in the method until the lock is released. It does not function like await in any way, shape or form.


add datapartition count limit for datanode.

if too many dp on one datanode, datanode's load will became very high, and even maybe not available

default limit is 3000


Duplicate / missing font declarations

I noticed that we seem to either declare some fonts redundantly or are missing font declarations for CYK languages.
Redundant Declarations

[x] .monaco-workbench .terminal-message-widget @Tyriar this came in via https://github.com/Microsoft/vscode/commit/7bcdf81179a6575082c9536b9c1abb32a4ef6a6f and its not clear to me why the rule in shell.css is not good enough
[ ] SuggestEnabledInput @sandy081 @roblourens seems to be used in settings editor and extensions view, its not clear to me why the font has to be declared like that
[x] .context-view.monaco-menu-container and .menubar .menubar-menu-items-holder.monaco-menu-container and .monaco-shell .monaco-menu-container .monaco-menu @sbatten maybe a leftover from when the context view was not rooted within the shell? I would think we can remove the 2 rules in the widget itself (provided you drop in a rule for the standalone editor) and then we can even remove the third rule in shell.css possibly?

Missing Declarations

issue reporter
process explorer

@RMacfarlane both seem to declare the font as in shell.css (which is right because these are in separate windows) but miss the language specific overrides it seems (e.g. :lang(ja), :lang(ko), lang(zh-Hans). If we support localization in these 2 windows as well, we need to have the same font rules. I also noticed that some fonts seem to be declared redundantly (e.g. html and textarea)
//cc @jrieken

No longer needed for the terminal 👍

@roblourens I am not sure I can follow, the declaration of fonts for the suggest input seems to match with the workbench font and not the editor font?

@rebornix do you have any idea for how to handle this for the editor? Language-specific font-family overrides in css: https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/electron-browser/media/shell.css#L20

I have no idea about this.

This is good for a while now, we share the default font from here:
https://github.com/microsoft/vscode/blob/0681925277df77878322d2ad789b746f462a08c9/src/vs/workbench/browser/style.ts#L195


Open a welcome dialog after user signed up

Pull Request Description
Closes: enso-org/cloud-v2#1475
Important Notes

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[ ] The documentation has been updated, if necessary.
[ ] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[ ] All code follows the
Scala,
Java,
TypeScript,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
[ ] Unit tests have been written where possible.

QA - Works on first sign up.


Backend Error: 404 Resource not Found

Describe the bug
Searching hangs and never returns a response. Upon checking the backend container, I can see the following error
error: Unhandled Rejection at: [object Promise], reason: Error: 404 Resource not found
I believe I must have incorrectly set up the config for Azure OpenAI but I can't figure out where I went wrong.
To Reproduce
I have provided the Azure OpenAI:

model name
API key
baseURL: https://{my-resource-name}.openai.azure.com/

Expected behavior
Search function to return a result.
Screenshot

Please remove the / at the end of the base URL, save your settings and try again.

Thanks. Tried again but got the same error.

try adding "/v1 " at the end of base url

Not working here, too. Still getting
 error: Unhandled Rejection at: [object Promise], reason: Error: 404 Resource not found


Exploring de-coupled transformers for optimized computation

jaydesen created the following on Feb 08:
Aim is to explore how good/bad decoupled transformers are for OneQA tasks such as retrieval, reader and so on. Can they be practically used to optimize performance in oneQA deployment in across tasks?
Original issue: https://github.ibm.com/ai-foundation/PrimeQA/issues/24

On Feb 15, mrglass commented:
Initial testing of a combined DC-BERT / ColBERT model shows poor performance. ColBERT's use of the [MASK] token to expand the query seems to be a problem for cross attention rerankers.

The experiments didn't lead to a better system for neural IR.


W. Ross Ashby

on cybernetics, there is this good old book from him on organization behavior.

https://www.amazon.com/gp/product/0412200902/ref=oh_aui_detailpage_o00_s00?ie=UTF8&psc=1


Handle merging items stacks in the inventory

Grouping two items stacks of the same type in the inventory is not supported

looks like it still work client side, but when you disconnect and reconnect items will not be splitted anymore


Feature request: Proxy endpoints

I love this simple web server and I use it a lot for my demo projects. However I often find that I want to use some public API. However, since they are hosted on another domain, I would get CORS issues. If this webserver would support a simple proxy it would make my demo projects a lot more powerful.
Usage idea:
nws -x api:https://somehost/somedir/api

This would proxy all requests to localhost:3030/api to https://somehost/somedir/api.

:+1: old reference https://github.com/knpwrs/nws/pull/5 :)


It would be great if there is a runnable example for testing

Great job! Please consider adding some runnable examples

You can use the test.html to try this tool in web browser.


Unknown CMake command "mlir_tablegen"

Could you please add doc on building and running examples ?  Thanks.
1 clone the repo
2 mkdir build & cd build
3 cmake ..
-- The C compiler identification is GNU 11.3.0
-- The CXX compiler identification is GNU 11.3.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
CMake Error at include/triton/Conversion/TritonGPUToSPIRV/CMakeLists.txt:2 (mlir_tablegen):
Unknown CMake command "mlir_tablegen".

Hello @jinz2014 ,
We are having a PR at https://github.com/intel/intel-xpu-backend-for-triton/pull/36, you could refer to that first.


documentation

Hi I am totally new in programing.
Could you pleas add some basic documentation, like  how to call new project?
I have found some tutorial on Youtube how to call new P5 project, but terrifying to go for such information to Youtube :)
It would be faster to know it from info page.
thanks for your plugin

Hi,
Apologies for just getting back to you.
If you have the extension installed in VSCode just use CTRL+SHIFT+P to
bring up the command palette and type in "Create P5 Project", the
extension's command should come up, click on it and it'll ask for a name
for the project.
Hope this helps
Thanks
-Alexander
(+44) 7305 670 959
@.***
www.ultamatum.dev
www.github.com/ultamatum
On Sun, Mar 20, 2022 at 12:52 PM barbarerik @.***>
wrote:

Hi I am totally new in programing.
Could you pleas add some basic documentation, like how to call new project?
I have found some tutorial on Youtube how to call new P5 project, but
terrifying to go for such information to Youtube :)
It would be faster to know it from info page.
thanks for your plugin
—
Reply to this email directly, view it on GitHub
https://github.com/ultamatum/P5-Project-Creator/issues/3, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKJKM6HMU3RAN3D74JYZDDVA4NP3ANCNFSM5RFO6CAQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Will add this to README when I get a minute


implement website login

implement a website login with a database (preferably sqlite)

User Management has been added with 4e35c2a.


AccessDeniedException when using ADOT with an EKS cluster

I'm receiving an error with a basic setup of ADOT, so probably I'm missing something. I just created a new EKS cluster, adding ADOT as addon. Next step was to add a ClusterConfig like this
apiVersion: eksctl.io/v1alpha5
kind: ClusterConfig

metadata:
  name: develop
  region: us-east-1

iam:
  withOIDC: true
  serviceAccounts:
    - metadata:
        name: adot-collector
        namespace: testnamespace
      attachPolicyARNs:
      - "arn:aws:iam::aws:policy/AmazonPrometheusRemoteWriteAccess"
      - "arn:aws:iam::aws:policy/AWSXrayWriteOnlyAccess"
      - "arn:aws:iam::aws:policy/CloudWatchAgentServerPolicy"

after that I created the following OpenTelemetryCollector using the sidecar mode
apiVersion: opentelemetry.io/v1alpha1
kind: OpenTelemetryCollector
metadata:
  name: develop-collector-xray
spec:
  mode: sidecar
  resources:
    requests:
      cpu: "1"
    limits:
      cpu: "1"
  serviceAccount: adot-collector
  config: |
    receivers:
      otlp:
        protocols:
          grpc:
            endpoint: <IP_ADDRESS>:4317
          http:
            endpoint: <IP_ADDRESS>:4318

    processors:
      batch:

    exporters:
      logging:
        loglevel: debug
      awsxray:
        region: 'us-east-1'

    service:
      pipelines:
        traces:
          receivers: [otlp]
          exporters: [awsxray]
      telemetry:
        logs:
          level: debug

I added the annotation
sidecar.opentelemetry.io/inject: "true"
to my pod definition. I started the application using the java agent and passing the required env variables
ENV OTEL_EXPORTER_OTLP_ENDPOINT=http://localhost:4317
ENV OTEL_RESOURCE_ATTRIBUTES=service.namespace=test-be,service.name=test-be
ENV AWS_REGION=us-east-1
ENV OTEL_METRICS_EXPORTER=otlp
CMD java -javaagent:/app/bin/aws-opentelemetry-agent.jar -jar /app/bin/registry.jar

Once started I can see the injected sidecar pod when tracing doesn't work and from the logs I can see the following error
2023-01-26T12:05:36.665Z	debug	awsxrayexporter@v0.66.0/awsxray.go:70	response error	{"kind": "exporter", "data_type": "traces", "name": "awsxray", "error": "AccessDeniedException: \n\tstatus code: 403, request id: c3c8ff28-18c5-4c2c-a5b5-e48b93b020c4"}
2023-01-26T12:05:36.665Z	debug	awsxrayexporter@v0.66.0/awsxray.go:74	response: {

}	{"kind": "exporter", "data_type": "traces", "name": "awsxray"}

I'm probably missing some authorization somewhere but I don't have idea where because I followed the official guideline
https://docs.aws.amazon.com/eks/latest/userguide/opentelemetry.html
Any ideas?
Thanks

Looks like that IRSA is not working as expected. You want to make sure that the service account adot-collector that you created via eksctl is in the same namespace as the ADOT collector.

Yes I already checked, it's in the same namespace.
The only additional thing (but I don't know if it could be a problem) is that I have a constraint in my account so in every IAM role I must add a permission boundary. But of course I added it to the ClusterConfig otherwise I cannot create it (I didn't report it in the example). I don't know if this constraint can block the standard flow in other parts, but in the service account is present.

Oh the wonderful world of permission boundaries.

Yes I know :(

Not sure if we have the complete picture, knowing this now. Two options: if you have Enterprise support, please cut us a ticket via your TAM or SA.

Unfortunately on this account we a Basic plan for the moment

If not, I'd work from left, that is, check: serviceaccount - > pod -> IAM role or try out a different mode (deployment).

For check what you mean? Anyway now I tried using deployment mode and it works. For the development purposes it's ok but we would like to use the sidecar mode. Is it possible something missing inside pod configuration?

Please note that we offer support via GitHub on a best effort basis, so could take some time (hence, suggesting the support route).

I know of course ;)

Anyway now I tried using deployment mode and it works. For the development purposes it's ok but we would like to use the sidecar mode. Is it possible something missing inside pod configuration?

Interesting. Let me look into this (note that the add-on is using upstream OpenTelemetry operator) and get back to you.
Would you mind expanding on why you prefer sidecar over deployment or other non-sidecar modes?

Interesting. Let me look into this (note that the add-on is using upstream OpenTelemetry operator) and get back to you.

great

Would you mind expanding on why you prefer sidecar over deployment or other non-sidecar modes?

It's a consideration based on a previous environment with Jaeger where we switched from a single collector (sometimes it had problems but I really don't remember the specific cause) to a sidercar container. Of course we can evaluate different mode if it works :)

Thanks for the context @fpaparoni and I would recommend to evaluate other modes, yes. Depending on your workload (number of pods, using sidecar mode can be a rather resource intensive option).

Hi @fpaparoni, can you confirm that the Collector and the Pod you are annotating are in the same namespace? That may be a reason why the sidecar mode doesn't seem to be working.

Yes in both modes Collector, Pod and Service Account are in the same namespace. Deployment now works, if I switch to sidecar i receive an AccessDeniedException

Hey @fpaparoni, any updates here? Were you ever able to get sidecar deployment of the Collector working?

We are using the deployment mode without problems and never switched back to sidecar. If it can be useful I can make some specific tests

We are using the deployment mode without problems and never switched back to sidecar. If it can be useful I can make some specific tests

I see - I've been trying to replicate your issue with no luck, but I haven't involved permission boundaries at all so that might be where the issue lies.
Also, when you say you can make specific tests, what are you referring to? What tests do you think would be useful to create?

I was thinking about looking at specific logs if useful, anyway we are now using without problems deployment mode and we won't come back


New solc release, breaks imports of solcx

py-solc-x Version: 0.2.1
solc Version: none
Python Version: 3.6
OS: Ubuntu

What was wrong?
Because the current latest release 0.4.26 does not provide a solc-static-linux binary yet, all imports of solcx fail, because the __init__ function will try to install 0.4.26 if no version is installed.
Example:
  File "/root/.local/lib/python3.6/site-packages/solcx/__init__.py", line 27, in <module>
    install_solc()
  File "/root/.local/lib/python3.6/site-packages/solcx/install.py", line 83, in install_solc
    _install_solc_linux(version)
  File "/root/.local/lib/python3.6/site-packages/solcx/install.py", line 144, in _install_solc_linux
    _wget(download, binary_path)
  File "/root/.local/lib/python3.6/site-packages/solcx/install.py", line 130, in _wget
    "Downloading solc from {}".format(url)
  File "/root/.local/lib/python3.6/site-packages/solcx/install.py", line 118, in _check_subprocess_call
    **proc_kwargs
  File "/usr/lib/python3.6/subprocess.py", line 291, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['wget', 'https://github.com/ethereum/solidity/releases/download/v0.4.26/solc-static-linux', '-O', '/root/.local/lib/python3.6/site-packages/solcx/bin/solc-v0.4.26']' returned non-zero exit status 8.

Could there be some error handling for this as this seems to happen with every new release?
Thanks.

Yeah, definitely can add an error handler. Thanks for bringing it to my attention.
Can you confirm - is it typical that a new solidity release doesn't initially include the static binary but adds it shortly after?

is it typical that a new solidity release doesn't initially include the static binary but adds it shortly after?

Yes. That is my personal experience. It often takes a few hours before the static binary is added.

I've added the get_available_solc_versions method as discussed in #7. I'm using that to determine which version of solc to install if none is present.  Should fix this issue since it won't go for the newest release if there's no binary available yet.
Thanks for opening these issues!  As soon as #6 is merged I'll release a new version on pypi.


Folder Gathering Test Failing

The test at test/Resources/folder.gathering.manifest.json is currently failing.

Looks like a legitimate failure.