[BUG]: Doom 2.0.9 indent guides cause my code to jump around like it's 1992

With the latest indent guides, especially if you are only showing the active guide, the new character seems to be changing the line height.
Reproduction

Open up your config.el
Try configuring it as such:(after! highlight-indent-guides
(setq
    highlight-indent-guides-auto-enabled         nil
    highlight-indent-guides-responsive           'top
    highlight-indent-guides-delay                0
    highlight-indent-guides-highlighter-function 'j/active-guide)
(custom-set-faces!
    '(highlight-indent-guides-top-character-face
    :foreground "#DE5356")))

Reload or restart emacs

Expected
Only the active indent guide is shown, the text does not jump around depending on which lines are affected by the guide.
Actual
Code gets real jumpy anytime the indent guide shows up, but unlike the 1992 classic "Jump Around" by House of Pain, this is a bit of a bummer.

Estimated Priority
🥘 4/10 - We should probably fix this?
Environment
OS X Catalina (10.15)
Emacs 27 (emacs-plus)
Doom 2.0.9
Doom Info

emacs   version    27.0.91
        features   NOTIFY KQUEUE ACL GNUTLS LIBXML2 ZLIB TOOLKIT_SCROLL_BARS MODULES THREADS JSON PDUMPER GMP
        build      Jun 05, 2020
        buildopts  (--enable-locallisppath=/usr/local/share/emacs/site-lisp --infodir=/usr/local/Cellar/emacs-mac/HEAD-b71b8b1/share/info/emacs --prefix=/usr/local/Cellar/emacs-mac/HEAD-b71b8b1 --with-mac --enable-mac-app=/usr/local/Cellar/emacs-mac/HEAD-b71b8b1 --with-gnutls --with-modules --with-json)
        windowsys  mac
        daemonp    server-running
doom    version    2.0.9
        build      grafted, HEAD -> develop, origin/develop, origin/HEAD 8736413 2020-06-03 03:46:58 -0400
        dir        ~/dotfiles/client/doom.d/
system  type       darwin
        config     x86_64-apple-darwin18.7.0
        shell      /usr/local/bin/fish
        uname      Darwin 18.7.0 Darwin Kernel Version 18.7.0: Tue Aug 20 16:57:14 PDT 2019; root:xnu-4903.271.2~2/RELEASE_X86_64 x86_64
        path       (~/.luarocks/bin /usr/local/bin ~/.emacs.d/bin ~/.npm-global/bin ~/bin ~/.yarn/bin ~/.config/yarn/global/node_modules/.bin /usr/local/bin /usr/bin /bin /usr/sbin /sbin /usr/local/share/dotnet ~/.dotnet/tools /usr/local/MacGPG2/bin /usr/local/Cellar/emacs-mac/HEAD-b71b8b1/libexec/emacs/27.0.91/x86_64-apple-darwin18.7.0/)
config  envfile    envvar-file
        elc-files  0
        modules    (:completion (company +childframe) (ivy +childframe) :ui doom doom-dashboard doom-quit fill-column hl-todo hydra indent-guides modeline ophints (popup +defaults) pretty-code tabs treemacs vc-gutter vi-tilde-fringe workspaces :editor (evil +everywhere) file-templates fold lispy multiple-cursors snippets :emacs dired electric undo vc :term vterm :checkers (syntax +childframe) :tools direnv docker editorconfig (eval +overlay) lookup (lsp +peek) macos magit rgb tmux :lang (clojure +lsp) emacs-lisp json (javascript +lsp) markdown org (python +lsp) racket sh (web +lsp) :email (wanderlust +gmail) :config (default +bindings +smartparens))
        packages   ((anakondo) (evil-lisp-state) (fish-mode))
        unpin      (n/a)
        elpa       (n/a)

Workaround
(setq highlight-indent-guides-character ?\x2502)

Notes

Experimented with changing the face :height, it makes it a bit better, no vertical jump but the horizontal jumping still exists.
Is it a character width\height issue?
Is there another character that could be fast but not mess with text layout?

As of cc772ad this issue should be resolved. Let me know if that isn't the case and I'll reopen it. Thanks for bringing it to my attention!
Side note: I appreciate the detailed, personal issue template -- it irks me to no end when folks ignore the one Doom provides -- but yours is missing your M-x doom/info. Please include it in future bug reports.

@hlissner Thanks very much for fixing that. I apologize for not using your template. Your time is valuable too and we should be doing what we can to make it easier for you to provide support.
Next time I will start with the template and add to it! 🚀


Add ability to set destroy failed jobs on a per job basis

We recently needed the ability to selectively delete failed jobs, even though our default is to keep all failed jobs.  I noticed there was a pull request (#742) but it has been sitting for a few months without tests.  I went ahead and built out the feature with tests included and I updated the README with an example.  DJ has been a real work horse for us for a number of years now, and I appreciate all the work you've done maintaining it.

Coverage increased (+0.12%) when pulling 995ff6a793b9e410cc214229f5c66c47790b7f17 on edgibbs:allow_per_job_destroy_on_failed into 1f881569603e45b93b8b1f21ecfa85379645fe3c on collectiveidea:master.

Coverage increased (+0.12%) when pulling 995ff6a793b9e410cc214229f5c66c47790b7f17 on edgibbs:allow_per_job_destroy_on_failed into 1f881569603e45b93b8b1f21ecfa85379645fe3c on collectiveidea:master.

Coverage increased (+0.38%) when pulling 854e5539aa0e4fcb551096867517a005656b4f44 on edgibbs:allow_per_job_destroy_on_failed into 1f881569603e45b93b8b1f21ecfa85379645fe3c on collectiveidea:master.

Funnily enough I discovered a need for this just yesterday.  Yes please - would be a useful feature.

Any update on when/if this will be merged?

Right now it does not cleanly merge. Also the first half of the worker changes are not needed and is better as is.

Coverage increased (+0.24%) to 90.81% when pulling 0d44f56a231b0ca1ea0c752ac8c22424627e6908 on edgibbs:allow_per_job_destroy_on_failed into 6ade20ce9365dc0648dc5874de255215868a41a8 on collectiveidea:master.

Coverage increased (+0.24%) to 90.81% when pulling 0d44f56a231b0ca1ea0c752ac8c22424627e6908 on edgibbs:allow_per_job_destroy_on_failed into 6ade20ce9365dc0648dc5874de255215868a41a8 on collectiveidea:master.

Coverage increased (+0.24%) to 90.81% when pulling 0d44f56a231b0ca1ea0c752ac8c22424627e6908 on edgibbs:allow_per_job_destroy_on_failed into 6ade20ce9365dc0648dc5874de255215868a41a8 on collectiveidea:master.

Coverage increased (+0.32%) to 90.89% when pulling 555ad4a15e9164be81a7aba322d828affe2a9452 on edgibbs:allow_per_job_destroy_on_failed into 6ade20ce9365dc0648dc5874de255215868a41a8 on collectiveidea:master.

Coverage increased (+0.32%) to 90.89% when pulling 555ad4a15e9164be81a7aba322d828affe2a9452 on edgibbs:allow_per_job_destroy_on_failed into 6ade20ce9365dc0648dc5874de255215868a41a8 on collectiveidea:master.

Coverage increased (+0.32%) to 90.89% when pulling 555ad4a15e9164be81a7aba322d828affe2a9452 on edgibbs:allow_per_job_destroy_on_failed into 6ade20ce9365dc0648dc5874de255215868a41a8 on collectiveidea:master.

Coverage increased (+0.32%) to 90.89% when pulling 555ad4a15e9164be81a7aba322d828affe2a9452 on edgibbs:allow_per_job_destroy_on_failed into 6ade20ce9365dc0648dc5874de255215868a41a8 on collectiveidea:master.

Coverage increased (+0.32%) to 90.89% when pulling c00e5fd67b2e5b33eacfc5447673ddbba8fd5dd8 on edgibbs:allow_per_job_destroy_on_failed into 6ade20ce9365dc0648dc5874de255215868a41a8 on collectiveidea:master.

Coverage increased (+0.32%) to 90.89% when pulling c00e5fd67b2e5b33eacfc5447673ddbba8fd5dd8 on edgibbs:allow_per_job_destroy_on_failed into 6ade20ce9365dc0648dc5874de255215868a41a8 on collectiveidea:master.

Coverage increased (+0.32%) to 90.89% when pulling c00e5fd67b2e5b33eacfc5447673ddbba8fd5dd8 on edgibbs:allow_per_job_destroy_on_failed into 6ade20ce9365dc0648dc5874de255215868a41a8 on collectiveidea:master.

Coverage increased (+0.32%) to 90.89% when pulling c00e5fd67b2e5b33eacfc5447673ddbba8fd5dd8 on edgibbs:allow_per_job_destroy_on_failed into 6ade20ce9365dc0648dc5874de255215868a41a8 on collectiveidea:master.

Thanks for the feedback, I'm running to work now, but I'll make the changes tonight.

Coverage increased (+0.26%) to 90.83% when pulling 2b6b0e2e259ba8417e2f9d695c6a6ab62aa9344e on edgibbs:allow_per_job_destroy_on_failed into 6ade20ce9365dc0648dc5874de255215868a41a8 on collectiveidea:master.

Coverage increased (+0.26%) to 90.83% when pulling 2b6b0e2e259ba8417e2f9d695c6a6ab62aa9344e on edgibbs:allow_per_job_destroy_on_failed into 6ade20ce9365dc0648dc5874de255215868a41a8 on collectiveidea:master.

Coverage increased (+0.26%) to 90.83% when pulling 2b6b0e2e259ba8417e2f9d695c6a6ab62aa9344e on edgibbs:allow_per_job_destroy_on_failed into 6ade20ce9365dc0648dc5874de255215868a41a8 on collectiveidea:master.

I think I got all the changes in from the comments.  I went ahead and updated the method name to destroy_failed_jobs?.  The only issue I see there is it has a slightly different name from the Delayed::Worker.destroy_failed_jobs class attribute.


Personal vs Pooled

I cannot find any clear answer why I would use a Personal WVD instead of a Pooled WVD.  I would greatly appreciate if someone couldd explain it, or point me in the right direction.

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

ID: 4aca2613-94aa-ea90-099f-625879914c4e
Version Independent ID: 6b4fcb8e-5d64-826f-04fd-207530dda12f
Content: Windows Virtual Desktop personal desktop assignment type - Azure
Content Source: articles/virtual-desktop/configure-host-pool-personal-desktop-assignment-type.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

@BryanAdvisicon
Thanks for your Question! We are currently investigating and will get back on this.

@stgeorgi @RoopChevuri Can you please add your comments on this.

Its because ur using me as a tool
On Fri, Jun 5, 2020, 1:43 AM VikasPullagura-MSFT<EMAIL_ADDRESS>wrote:

@stgeorgi https://github.com/stgeorgi @RoopChevuri
https://github.com/RoopChevuri Can you please add your comments on this.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/56398#issuecomment-639271540,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APKSOELA24T2MPYIOHCB45LRVCAWHANCNFSM4NS3EK7Q
.

Why do yall continue to do this stuff knowing your invading my privacy your probably in this with enrique silva

@BryanAdvisicon Yes, it is the same as mentioned in this doc.
Hope the provided information is helpful.
We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.


vercel deployment

Can your APP be deployed on Vercle?

Can your APP be deployed on Vercle?

Hi, I just tested the Vercel, will prompt error: Unable to find any supported Python versions ,Vercel is suitable for front-end static project deployment, the recommended use of Zeabur direct deployment .

主页模板不好看

主页模板不好看

你可以自己改好看的


TagUI in TURBO mode at 20-30X faster in exchange for reliability - for some users

as above, below attached special version on request of some users

Download this zip file turbo_tagui.zip, unzip and copy the 2 files to overwrite what you have in tagui/src folder.
Websites and apps are not designed for superhuman speed. Using above will let TagUI's Chrome integration run at max speed, and also change the typing behaviour from character by character to all text at one go. This may cause website validation events to not trigger and work correctly. Running at max speed your laptop CPU will also become pretty hot.
Use at your own risk and backup your files beforehand!
PS - attached files are based on TagUI v6.21

Above turbo_tagui.zip will do both 1 and 2 -

run Chrome at max automation speed at 100% CPU
type text instantly instead of character by character

Below are zip files for the 2 changes separately (in case 1 of the change breaks your automation)

Turbo_Mode.zip - run Chrome at max automation speed at 100% CPU
Instant_Typing.zip - type text instantly instead of character by character

Closing issue but feel free to post anything here


Format files

as title ...
It's anoing to see many lines of diff if you edit one of this files and the editor automaticaly correct unrelated things.
this pull format files so there will be less + fix alignment of "make help"
this pull does not change any behaviour

Codecov Report

Merging #13698 (9fdc2ef) into master (00ec651) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #13698      +/-   ##
==========================================
+ Coverage   42.24%   42.26%   +0.01%
==========================================
  Files         697      697
  Lines       76616    76616
==========================================
+ Hits        32369    32383      +14
+ Misses      38930    38914      -16
- Partials     5317     5319       +2

Impacted Files
Coverage Δ

modules/queue/manager.go
62.13% <0.00%> (-2.96%)
:arrow_down:

modules/process/manager.go
72.50% <0.00%> (-2.50%)
:arrow_down:

routers/api/v1/repo/pull.go
24.84% <0.00%> (-0.61%)
:arrow_down:

routers/repo/view.go
38.11% <0.00%> (+0.64%)
:arrow_up:

services/pull/patch.go
55.68% <0.00%> (+1.70%)
:arrow_up:

modules/queue/workerpool.go
60.81% <0.00%> (+2.04%)
:arrow_up:

models/unit.go
49.31% <0.00%> (+2.73%)
:arrow_up:

services/pull/check.go
51.82% <0.00%> (+2.91%)
:arrow_up:

services/pull/temp_repo.go
29.78% <0.00%> (+3.19%)
:arrow_up:

modules/charset/charset.go
73.03% <0.00%> (+4.49%)
:arrow_up:

Continue to review full report at Codecov.

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

🚀

:rocket:


Fix the README example

There was a small tweak needed in the README for it to work.
// CC @osterman @aknysh


Windows EC2 instance detection

What does this PR do?
On windows under EC2, hostname starts with ec2amaz-.
If the system hostname starts with "ec2amaz-" we use the instance id instead of the hostname.
agent status output is:
Before
 Hostnames
  =========
    ec2-hostname: ip-172-29-3-76.ec2.internal
    hostname: EC2AMAZ-V68JKHF
    instance-id: i-0fc144c877ac77810
    socket-fqdn: EC2AMAZ-V68JKHF
    socket-hostname: EC2AMAZ-V68JKHF
    hostname provider: os

After
 Hostnames
  =========
    ec2-hostname: ip-172-29-5-33.ec2.internal
    hostname: i-017091562d278613e
    instance-id: i-017091562d278613e
    socket-fqdn: EC2AMAZ-OUBBQHG
    socket-hostname: EC2AMAZ-OUBBQHG
    hostname provider: aws

Here is the current behavior
For a new host.
Case1

set ec2.use_windows_prefix_detection at true
start the agent

=> the instance-id is used as canonical instance
If we set ec2.use_windows_prefix_detection at false, we have another host. Metrics are not sent to the previous host.

Case2

set ec2.use_windows_prefix_detection at false
start the agent

If we set ec2.use_windows_prefix_detection at true, nothing chages. Metrics are still sent to the host.

The 2 cases exist because in case1, EC2AMAZ-J9U50S5 is not an alias for the host.


vscode.env.shell always set to PowerShell instead of default profile

Issue Type: Bug
Could be related to https://github.com/microsoft/vscode/issues/132150, but I tested on Insiders and my issue (which is slightly different) is still reproducing. This only started happening in v1.60.0
Repro steps

Add this to your user settings:
"terminal.integrated.defaultProfile.windows": "Command Prompt"
Create a 'hello world' extension and add this code:
vscode.window.showInformationMessage(vscode.env.shell);
F5 the extension and run the hello world command

Expected:
env.shell is set to the command prompt
Actual
env.shell is set to PowerShell, no matter how long I wait or even if I reload VS Code

The terminal itself seems to respect the profile setting, though:

VS Code version: Code - Insiders 1.61.0-insider (aa93eefe550167de3ca29aaa3d06f721a47594a3, 2021-09-09T05:15:19.597Z)
OS version: Windows_NT x64 10.0.19043
Restricted Mode: No

System Info

Item
Value

CPUs
AMD EPYC 7452 32-Core Processor                 (4 x 2346)

GPU Status
2d_canvas: unavailable_softwaregpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: disabled_offrasterization: disabled_softwareskia_renderer: enabled_onvideo_decode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
undefined

Memory (System)
16.00GB (9.87GB free)

Process Argv
--crash-reporter-id fb927676-2903-44ae-bbc9-133db7d2893d

Screen Reader
no

VM
0%

/duplicate #132150

@Tyriar are you sure it's a duplicate? I'm aware of the linked bug, but my issue is still reproducing on Insiders. I even tried on today's version:

1.61.0-insider (user setup)
Commit: 0884da1650baa24994d23d2a3742823c1186e526
Date: 2021-09-10T08:12:39.143Z
Electron: 13.1.8
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Windows_NT x64 10.0.19043

Some related fixes got pushed in in the past few hours


Opengl accelerated

I would like to have GPU accelerated FFT.
Is it possible?
This will have applications for real time simulations and games.

I do have a specific use case, although there are many applications of FFT for 2D and 3D games, my specific use case is to implement Jos Stam's 2D fluid solver.
I am making a game for IOS and Android, that is underwater with realistic currents and eddies.

I can prototype it with fftea, and perhaps it will be fast enough with CPU.


Add button to save XML as a file

This will be useful for making records of projects in progress, easier than having to copy and paste into a new file each time you want to save. Maybe have the option to customize the filename.
Request from @MariannaPaszkowska.

Added in 6e7a42e3e554373573f7481131656da53bd4750b


Debug output in PunktSentenceTokenizer

In reference to https://github.com/nltk/nltk/commit/b3e2b410eb0031b52ae05e9d35fe275dbe8098cd, there is a print statement in the middle of nowhere, is this intended?

https://github.com/nltk/nltk/blob/develop/nltk/tokenize/punkt.py#L1072

@SanketDG Thanks for catching this! Yes, it shouldn't be there.


python3Packages.lxml: 4.4.2 -> 4.5.0

Motivation for this change
routine update
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

sorry for spam, had to target staging

@GrahamcOfBorg build systemd poetry2nix


Bug: BLE Curtain Control Broken

Describe the Bug
This seems to be a carryover of this:[ #1007 (comment)](https://github.com/OpenWonderLabs/homebridge-switchbot/issues/1007#issuecomment-2361594324)
However, with version 4.0 my BLE control is completely broken.
To Reproduce
Update from 3.8 to 4.0
Relevant Log Output
[10/14/2024, 2:23:26 PM] [@switchbot/homebridge-switchbot] This plugin generated a warning from the characteristic 'Target Position': characteristic value expected valid finite number and received "NaN" (number). See https://homebridge.io/w/JtMGR for more info.
[10/14/2024, 2:23:27 PM] [SwitchBot] Curtain3: LeftCurtain TypeError: Cannot read properties of undefined (reading 'runToPos')
[10/14/2024, 2:23:27 PM] [SwitchBot] Curtain3: LeftCurtain Retrying
[10/14/2024, 2:23:28 PM] [SwitchBot] Curtain3: LeftCurtain TypeError: Cannot read properties of undefined (reading 'runToPos')
[10/14/2024, 2:23:28 PM] [SwitchBot] Curtain3: LeftCurtain Retrying
[10/14/2024, 2:23:29 PM] [SwitchBot] Curtain3: LeftCurtain TypeError: Cannot read properties of undefined (reading 'runToPos')
[10/14/2024, 2:23:29 PM] [SwitchBot] Curtain3: LeftCurtain Retrying
[10/14/2024, 2:23:30 PM] [SwitchBot] Curtain3: LeftCurtain TypeError: Cannot read properties of undefined (reading 'runToPos')
[10/14/2024, 2:23:30 PM] [SwitchBot] Curtain3: LeftCurtain Retrying
[10/14/2024, 2:23:31 PM] [SwitchBot] Curtain3: LeftCurtain failed BLEpushChanges with BLE Connection, Error Message: "Cannot read properties of undefined (reading 'runToPos')"

Config for homebridge-switchbot
"name": "SwitchBot",
            "credentials": {
                "notice": "Keep your Token & Secret a secret!"
            },
            "options": {
                "devices": [
                    {
                        "deviceId": "...",
                        "configDeviceName": "LeftCurtain",
                        "configDeviceType": "Curtain3",
                        "connectionType": "BLE",
                        "customBLEaddress": "...",
                        "curtain": {
                            "setOpenMode": "1",
                            "setCloseMode": "0",
                            "silentModeSwitch": true,
                            "hide_lightsensor": true
                        },
                        "external": true
                    },
                    {
                        "deviceId": "...",
                        "configDeviceName": "RightCurtain",
                        "configDeviceType": "Curtain3",
                        "connectionType": "BLE",
                        "curtain": {
                            "setOpenMode": "1",
                            "setCloseMode": "0",
                            "silentModeSwitch": true,
                            "hide_lightsensor": true
                        },
                        "external": true
                    }
                ],
                "BLE": true
            },
            "_bridge": {
                "username": "...",
                "port": 59906
            },
            "platform": "SwitchBot"
        }
    ],

Device and Model
Raspberry Pi 4
Node.js Version
20.17.0
NPM Version
10.2.4
Homebridge Version
1.8.4
Homebridge Switchbot Plugin Version
4

any word on this?

I'm also having the same issue

@donavanbecker any chance you could take a look into this. Still stuck on v.3.8.0


Improve the UI of detail view of teams page.

present UI --

Assigned to Dheeraj

Thanks @DheerajMalisetty


adding data structure classes in separate files to test them

I have created all classes we will use inside our data structure.
I have made classes in separate files, for now, to test them one by one using unit test

I closed this PR because I will make the work inside for string things out with testing then will add the final result to the main branch


Feature request: pico-stdio: Provide puts_raw analogue that doesn't write newline

It would be great to have access to stdio_put_string() version that doesn't append newline at the end - that would simplify logging and similar routines, and can help in some cases with custom stdio drivers.
int putchar_raw(int c) {
    char cc = (char)c;
    stdio_put_string(&cc, 1, false, true);
    return c;
}

int puts_raw(const char *s) {
    int len = (int)strlen(s);
    stdio_put_string(s, len, true, true);
    stdio_flush();
    return len;
}

//-+-+-+-+-+-+- suggestion -+-+-+-+-+-+-//
int put_string_raw(const char *s) {
    int len = (int)strlen(s);
    stdio_put_string(s, len, false, true);
    stdio_flush();
    return len;
}
//-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-//

If that's acceptable, I can send a PR.

seems ok, but name is probably confusing maybe writes_raw is a better name

putchar_raw and puts_raw have been added. Marking this as resolved.

putchar_raw and puts_raw have been added. Marking this as resolved.

I'm a bit confused. Both putchar_raw and puts_raw were already there. The request was to add writes_raw (put_string_raw).

You're right. My misunderstanding. Lets see what @kilograham says.

fixed


Artboard tool doesn't have snapping implemented

Artboard doesn't support snapping, either it should be implemented or perhaps the option to turn on snapping should be disabled?

Snapping should be implemented for the artboard, yes.

@Sahilgill24 for tracking purposes can you please comment in this issue so I can co-assign it to you?

Hey @Keavon , was really busy with my acads that's why was not able to continue it properly

@Sahilgill24 it's all good, it looks like Hypercube kindly finished it up for you so we'll get this merged soon. Feel free to keep contributing to the project if you're so inclined when you have more free time :)


Paths must return as array, not as a concatenated string.

Expected
getPaths("<svg>...</svg>")
// ["...", "..."]

Closes with this commit https://github.com/aykutkardas/svgps/commit/3caed62545e9200827d91e14c51c6dee7da5cd4a


add:usr:search: replace %self% with the user's mention name

This allows to share and reuse the same queries across different
users. E.g. club search 'owner:%self% !is:done' to list all
uncompleted tasks.
This is the equivalent to currentUser() in JIRA. I asked
Clubhouse for this feature a long time ago.

Looks like we can now get the mention_name and the workspace_slug. That will make the config cleaner.

Released in v2.1.0


--no-env causes failure if gnu parallel is loaded from a module

Rather obvious but not intuitive for newcomers, especially because there is no warning from qbatch.

Perhaps a warning should be printed for --no-env

We could also stick a check for parallel at the start of the job script, e.g.
which parallel || { echo "GNU parallel not found. Exiting."; exit 1; }

Hrm, I like that, just need to keep an eye on my new "don't use parallel for single commands" modifications (see PR #108)

"Fixed" by adding a reasonable job error in the script


[Node.js] Add some minor new APIs

Add serializedData to notification object
Add fileName to RecorderDevice

@halton PTAL, thanks.


Is it possible to temporarily disable a personal tab in my own Microsoft TEAMS app?

I am building my own teams app and would like to disable or hide certain personal tabs of my app depending on e.g. the role of the signed in user.
Here are two examples in which case I would like to disable certain personal tabs:

User hasn't signed in yet, so I only want to show one tab, the "sign in" tab. All other tabs are redundant as they show the same information.
User doesn't have permission to view a certain tab, because access may be restricted to admins only. Instead of showing a "you have no access" banner, we would prefer to simply hide or disable (e.g. greyed-out or unclickable) that particular tab.

I couldn't find any information on whether this is possible.
Thank you for your help!

We have static tabs in personal scope and It is not possible to hide static tabs. You can request this feature on Microsoft Uservoice.

We are closing this issue, please feel free to reopen if you need any help. Thank you.


Powershell Debugger Exits

VSCode Version: Code 1.13.1 (379d2efb5539b09112c793d3d9a413017d736f89, 2017-06-14T18:21:47.485Z)
OS Version: Windows_NT ia32 10.0.15063
Extensions:

Extension
Author
Version

vscode-eslint
dbaeumer
1.2.11

beautify
HookyQR
1.1.1

mssql
ms-mssql
1.0.0

csharp
ms-vscode
1.10.0

PowerShell
ms-vscode
1.4.1

debugger-for-chrome
msjsdiag
3.1.5

code-settings-sync
Shan
2.8.1

Steps to Reproduce:
1.Intermittent problem.  The debugger just exits abnormally.  VSCode does not crash, but the debugging session is over.
2.

Reading your description it seems that the issue originates from an extension and not VS Code itself. Thereby I will close this issue and kindly ask you to report this issue against the extension repository.
Most extensions are on GitHub as well so it should be easy to file it against the extension. If you open an extension on the market place, click on the Support link to find out more.

Thanks again for your feedback!


Very large differences between cell and pack currents using Equivalent Circuit model

Discussed in https://github.com/pybamm-team/liionpack/discussions/242

Originally posted by MustafaDragonborne March 31, 2023

I simulated a simple 2p1s configuration using the ECM (not sure if the problem is ECM specific) but I see that the difference between the accumulated cell currents and the total pack current is too large. Sometimes, the pack current is much higher than the accumulated cell currents as is evident from the last graph (Pack current - accumulated cell currents) which means there is alot of extra current coming in.
This is because of the fact that the cell currents are displaced one step forward which cause the error. This difference becomes negligible if the cell currents are moved one step back with respect to the pack current. This is a bug that needs to be fixed.

Steps to reproduce:
import liionpack as lp
import pybamm
import numpy as np
import matplotlib.pyplot as plt

lp.set_logging_level('NOTICE')

# Define parameters
Np = 2
Ns = 1

# Generate the netlist
netlist = lp.setup_circuit(Np=Np, Ns=Ns)

# Define a cycling experiment using PyBaMM
experiment = pybamm.Experiment([
    'Charge at 2 A for 2 minutes',
    'Rest for 2 minutes',
    'Discharge at 1 A for 2 minutes',
    'Rest for 2 minutes'
    ],
    period='10 seconds')

# Define the PyBaMM parameters
parameter_values = pybamm.ParameterValues("Chen2020")

# Solve the pack
output = lp.solve(netlist=netlist,
                  parameter_values=parameter_values,
                  experiment=experiment,
                  initial_soc=0.5,
                  manager='casadi')

sum_cell = np.sum(output['Cell current [A]'], axis=1)
pack = output['Pack current [A]']
diff = sum_cell - pack
if np.any(np.abs(diff)>0.001):
    print('Differences in pack and cell current')
    print('Time steps', np.argwhere(np.abs(diff)>0.001).flatten())
    print(diff[np.abs(diff)>0.001])

plt.figure()
plt.plot(pack)
plt.plot(sum_cell)


Generate Helm chart release v0.1.1

includes Flux v1.4.1 and the Git known hosts customisation option

I've rebased this on master and bumped the helm-op and fluxd versions, but I note that the index file includes the old versions and I'm not sure what to do about that.

OK I figured it out: Stefan has been generating the package and updating the index from master branch (or a topic branch like this one). For that reason it's convenient to have the index here, since you need the chart files to make the package tarball anyway.
An alternative is to start in gh-pages, grab the chart files from master with git clone -b ... and make the package tarball and update the index, ready to commit. I've added a script to gh-pages to do that, and removed the index from here.
This PR is therefore just about updating the Helm chart to the more recent versions of fluxd and helm-op (the package version 0.1.1 is already published, i.e., committed to the gh-pages branch).

Wonderful, thanks @dholbach :star:


my airflow can not run my code

Can you tell me  why there is an error ?  How can I solve it?
best regards!
gxq

And I also can not install airflow==0.6


Basic Voice Command Features

📋 **Issue Title: Add Voice Command Features to Assistant **
Description
Add voice command recognition to JARVIS to support the following features:

Voice Command Recognition: Listen to user commands.
Application Opening: Open apps like Calculator, Notepad via voice.
Web Search: Perform Google searches using voice commands.
Play Music: Search and play music from YouTube via voice.
Screenshot Capture: Take screenshots with a voice command, saving them with a custom name.
Greeting: JARVIS should greet the user upon activation.

Suggested Libraries

SpeechRecognition: For voice input.
os/subprocess: For opening apps.
webbrowser: For web searches.
pytube: For playing YouTube music.
pyautogui: For capturing screenshots.

thanks @suryanshsk


Web frontend

Make a frontend for this.


pkg/search: staticcheck

Closing this, see my comment here:

https://github.com/perkeep/perkeep/pull/1405#issuecomment-894727096


adds css tooltips section in interactions

Fun with css tooltips!
The default gulp build looks like it added aria-labels and rebuilt the lightbox section.

Thanks for this PR, left a few comments!

Where are the comments @una ? I don't see them.

@josephjguerra sorry I forgot to hit "submit" with the new GH code review system


Use latest node-gyp

This gets rid of "python2-only" issues.
We also remove Node < 6 as those versions are not supported anymore. (even node 6 is not maintained)

Any idea when the v5 will be released?


Change Scroll Indicator Color

How to change the scroll indicator color, it's showing white.

same here

For iOS use:
flatListProps={{                 indicatorStyle: 'black',               }}
For Android edit Theme on AndroidManifest.xml
add in Theme
@android:color/darker_gray


Reducing the size on WASM targets

On WASM targets, this library's unicode lookup table adds roughly 50 KB to release builds.
This makes it the second biggest contributor to my binary size right now, right behind the data section.
I would like to write a PR that on WASM targets uses js-sys's toLowerCase instead of the lookup table. This may result in different behavior between WASM and other targets though.
Would a PR like this be accepted? Would it need to be behind a feature gate?

See also, the related issue in pulldown-cmark: https://github.com/raphlinus/pulldown-cmark/issues/357

I'm sympathetic to reducing the size on WASM. What were you thinking with regards to using toLowerCase? I'm not that familiar with WASM, does that work on chars?

JS does not have a character type, toLowerCase only works on strings.
The idea is to eliminate the lookup table, which is the biggest problem, and rely on the browser knowing how to correctly convert a unicode string to lowercase.
Taking a look at the code, it seems the lookup table is only used to convert characters from uppercase to lowercase, if I'm correct? An implementation approach could be to cfg the lookup table and the implementations where it's used and add separate ones for the wasm32 arch that make use of js-sys's bindings to toLowerCase.

The table is to do case folding, which as it explains, handles some cases that simply converting to lowercase does not. I don't know if the browser exposes a way to do case folding. If a way is found, using cfg is a fine way to change the internals.

Quickly checking, it does seem like toLowerCase and toUpperCase handle this correctly.
I could run simply a single character string through toLowerCase, and it should give the right result.

Turkish is an example of when upper/lower case fails. Does that pass the test?
Searching for "JavaScript case folding", I see mention of two APIs: String.prototype.localeCompare and Intl.Collator. I have not used either, so I don't know which is better to use.

I tested that specifically and it does pass the test.
localeCompare could be used to avoid doing it character-by-character, which may have a lot of overhead.
Taking that as an approach for the PR, I would re-implement the functions previously using the lookup table, to eliminate it entirely.

localCompare could be used for both PartialEq and Ord
toLocaleLowerCase could be used for Hash

Both set to "en" locale, as recommended the case folding page.
Does that look like a good solution?

I believe so!

I'll leave this issue open, despite my attempt at using js-sys to solve this not working out, as it may be worth investigating how to make the output WASM smaller if possible.

Size was greatly reduced in #35.


Electron-packager

Hi,
I've had trouble using electron-packager module with Photon's setup.

What problems?
Describe in detail the maximum you can

It was more an issue with my setup I think.
Thanks,
mcdonaghc


unreadable data when used with a phone (badges and cards alike)

See the attached screenshot. Default theme vs kibibit

Hi @anubisg1 ! thanks for opening this!
Unfortunately, this is just because of the different font, and not an actual bug. Every font affects the smallest screen size it can compensate. I don't change the font size at all from the default hassio is setting in this theme.
If you won't add the additional CSS resource (which loads the comfortaa font), and keep the default font, it will have the same size as the default font and everything will look good in your phone resolution.
On the mobile app, the default font-size is not configurable on the app itself. But it is affected by the system's text size. If you go to Settings -> Display -> text size, you can change the size of the text in your phone, which will change the size of the font in the app. The problem is, that it affects everything else on your phone as well.
I added some photos as examples at the end of this reply.
Sorry but those are the best answers I can give :-/
TL;DR:

don't add the comfortaa font to keep the regular font and it relative sizes
change the font size on your phone to change the font size in the app

Settings -> Display -> Text Size

Regular font size

When setting my phone to XXL font size

When setting my phone to XXS font size

Hi @anubisg1 ! thanks for opening this!
Unfortunately, this is just because of the different font, and not an actual bug. Every font affects the smallest screen size it can compensate. I don't change the font size at all from the default hassio is setting in this theme.
If you won't add the additional CSS resource (which loads the comfortaa font), and keep the default font, it will have the same size as the default font and everything will look good in your phone resolution.
On the mobile app, the default font-size is not configurable on the app itself. But it is affected by the system's text size. If you go to Settings -> Display -> text size, you can change the size of the text in your phone, which will change the size of the font in the app. The problem is, that it affects everything else on your phone as well.
I added some photos as examples at the end of this reply.
Sorry but those are the best answers I can give :-/
TL;DR:

don't add the comfortaa font to keep the regular font and it relative sizes
change the font size on your phone to change the font size in the app

Settings -> Display -> Text Size

Regular font size

When setting my phone to XXL font size

When setting my phone to XXS font size

thank you for your response. the reason why i posted this here is because on iphone is rendering correctly and on a much bigger and much higher pixel density on the galaxy s9 was rendering wrong. i'll check your suggestion. thanks again!

thank you for your response. the reason why i posted this here is because on iphone is rendering correctly and on a much bigger and much higher pixel density on the galaxy s9 was rendering wrong. i'll check your suggestion. thanks again!


added regex to prevent illegal characters error for Project Name

Illegal Characters Error

After entering the $ feathers generate app command, it allows the User to write a custom 'Project Name'. If the User includes spaces in the name, it results in 'error - illegal characters / ERR! Invalid Name:'.
Added regex to prevent user from writing a Project Name with spaces.
I'm new to GitHub and not sure if where I added the regex code is best practice.

Screen shot of the command $ feathers generate app , the User writing Project Name, and the according error at the bottom.

Thank you @LunettoRandall. You are right, the project name should indeed be filtered but I think it makes more sense to use the same method we already are for service names etc. instead of a custom regex.
I created a follow up PR for this in #277

Thank you!


Inverted normals on right wing when using flip_y=true

The ordering of the panel coordinates is inverted when performing a flip about the y axis. This inverts the normals. This has to be corrected using reverse on one of the axis of the matrix.

Use symmetric option instead to create symmetric wings. flip_y is retained and any corrections should be performed on the client side.


Write a document about the StableHLO serialization format

https://github.com/openxla/stablehlo/blob/main/docs/bytecode.md is a useful document, but it primarily talks about implementation details. Let's write something that's focused on the user perspective.

A combination of the recently updated https://github.com/openxla/stablehlo/blob/main/docs/bytecode.md and the recently introduced https://github.com/openxla/stablehlo/blob/main/docs/vhlo.md addresses this well.


Open-vm-tools fails to install due to missing gdk-pixbuf-xlib-2.0

Describe the bug
Open-vm-tools fails to install:
checking for gdk-pixbuf-xlib-2.0 >= 2.21.0 (via pkg-config)... no
configure: error: gdk-pixbuf-xlib-2.0 not found. Please configure without X11 (using --without-x) or install the gdk-pixbuf-xlib-2.0 devel package.
builder for '/nix/store/aryy4mq9nr08dlvmhb2prm6zpyk2dmzr-open-vm-tools-11.0.5.drv' failed with exit code 1

To Reproduce
virtualisation.vmware.guest.enable = true;

in /etc/nixos/configuration.nix
Notify maintainers
joamaki
Metadata

system: "x86_64-linux"
host os: Linux 5.4.44, NixOS, 20.09pre228622.029a5de0839 (Nightingale)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.6
channels(alex): "home-manager"
channels(root): "nixos-20.09pre229574.0a146054bdf, home-manager"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute: open-vm-tools
# a list of nixos modules affected by the problem
module: virtualisation.vmware.guest

I have a tentative fix here https://github.com/tteggel/nixpkgs/tree/fix-open-vm-tools-gdk-pixbuf-xlib

Upstream PR: https://github.com/vmware/open-vm-tools/pull/438

Thanks!


Recommended way of reacting (or listening) to StateNotifier state changes with another StateNotifier

I have a function in a StateNotifier (SN) that depends on the state of two other SN's, as well as its own state.
The state of each SN must be in the correct state before the function can execute properly and it must be called automatically each time the other two SN's state changes.
Say the structure looks like this:
final aProvider = StateNotifierProvider((ref) => ANotifier());
class ANotifier extends StateNotifier<AState> {
  // ...

  Future<void> dependentFuc(BState bState, CState cState) async {
    final currentState = state;
    if (currentState is ACorrectState && bState is BCorrectState && cState is CCorrectState){
      final res = await ///... and so on using data in each "CorrectState"
      // set state = using res from above
    }
  }
}

final bProvider = StateNotifierProvider((ref) => BNotifier());
class BNotifier extends StateNotifier<BState> {
  /// ...
}

final cProvider = StateNotifierProvider((ref) => CNotifier());
class CNotifier extends StateNotifier<CState> {
  /// ...
}

I thought about doing something like:
// change aProvider to:
final _aProvider = StateNotifierProvider((ref) => ANotifier());
final aProvider = StateNotifierProvider((ref) {
    final aNotifier = ref.read(_aProvider);
    final bState = ref.watch(bProvider.state);
    final cState = ref.watch(cProvider.state);

    aNotifier.dependentFuc(bState, cState);
    return aNotifier;
});

But, of course, that wouldn't work because aNotifier gets disposed of when the provider is recalled.
I wanted to use the clean ref.watch syntax somehow but it doesn't seem like I can?
Is the only way to set up listeners for the notifiers, store the notifiers and call the remove listener functions in dispose of ANotifier?
Like:
final aProvider = StateNotifierProvider((ref) => ANotifier(
  bNotifier: ref.read(bProvider),
  cNotifier: ref.read(cProvider),
));
class ANotifier extends StateNotifier<AState> {
  final BNotifier bNotifier;
  late final Function() _bCancel;

  final CNotifier cNotifier;
  late final Function() _cCancel;

  ANotifier({
    required this.bNotifier,
    required this.cNotifier,
  }) : super(AState.initial()){
    _bCancel = bNotifier.addListener((_) => dependentFuc());
    _cCancel = cNotifier.addListener((_) => dependentFuc());
  }

  Future<void> dependentFuc() async {
    final currentState = state;
    final bState = bNotifier.state;
    final cState = cNotifier.state;

    if (currentState is ACorrectState && bState is BCorrectState && cState is CCorrectState){
      final res = await ///... and so on using each "CorrectState"
      // set state = using res from above
    }
  }

  @override
  void dispose(){
    super.dispose();
    _bCancel();
    _cCancel();
  }
}

It's so much boilerplate code for something as seemingly simple as "call dependentFuc when BNotifier or CNotifer" change".
PS: There are functions in each notifier to set them to their "CorrectState", which are called from the UI for example.

I was pointed to https://github.com/rrousselGit/river_pod/issues/186 which solved my problem.
The aProvider can be rewritten to the following, to listen to the other providers in a concise way:
final aProvider = StateNotifierProvider<ANotifier>((ref) {
  final aNotifier = ANotifier();

  final bNotifier = ref.watch(bProvider);
  final cNotifier = ref.watch(cProvider);

  final cancelB = bNotifier.addListener((bState) {
    final cState = ref.read(cProvider.state);
    aNotifier.dependentFuc(bState, cState);
  });
  final cancelC = cNotifier.addListener((cState) {
    final bState = ref.read(bProvider.state);
    aNotifier.dependentFuc(bState, cState);
  });

  ref.onDispose(() {
    cancelB();
    cancelC();
  });

  return aNotifier;
});;


[feature] exporting task bundles

I want to share the build tasks with my other team members on a project. Using VSCode tasks, this can be done fairly efficiently. However, tasks built using overseer don't have the same portability. I know bundle files are saved in ~/.local/state/nvim/overseer, but there is no way to export bundles conveniently.
Because of this, I think having a conversion layer between overseer bundles and VSCode tasks would be nice. You can create tasks with overseer first, then export them into VSCode tasks later. We already have VSCode  tasks.json support, so I guess implementing the reverse direction would be straightforward.
I am new to the plugin, so sorry if I am missing something here.

It would actually be quite difficult and a lot of work to try to convert an arbitrary overseer task to the VSCode format. The best way to share tasks with your team on a project is the VSCode tasks.json format, and the best way to create those is unfortunately by hand. The good news is that it shouldn't be that difficult to do and the format/process is pretty well documented.


javscript should target node.js

These games are meant as simple CLI scripts (see https://github.com/coding-horror/basic-computer-games/issues/117#issuecomment-1070918597).
I therefor think javascript should target mainly node.js instead of the browser (in opposite of whats currents implemented).

So, I started porting the existing javascript implementations to be simple node-scripts.
I also introduced very basic common function like print, input and tab. (see https://github.com/coding-horror/basic-computer-games/issues/548#issuecomment-1079839924).
I decided to use modern ES-modules instead of the legacy commonCJ node-syntax. That way the scripts can easily load the common function (also in the browser). For this to work the files have a *.mjs extension.
To not loose the ability to play the games in the browser I created a simple node-terminal-emulator that lets run our node-scripts directly in the browser. For this to work I overwrote the default process.stdout/stdin of node and hocked them up to some browser mockups. I know this is very hacky but it works just fine (at least for the simple scripts I've tested).

Here is an example of the emulator in action: https://mojoaxel.github.io/basic-computer-games/00_Common/javascript/WebTerminal/terminal.html#74_Rock_Scissors_Paper/javascript/rockscissors.mjs
This way we have the best of both worlds:

✔ The JavaScript implementations get very slim and easy to ready also for beginners. All the browser-specific stuff is gone!
✔ The scripts can simply be executed and played in every shell, just like e.g. the python or perl implementations.
✔ We still have the possibility to run the games in the browser.
✔ The terminal-emulator even feels more "real" because the input fields are gone.
❌ It is no longer super easy for the user to run the games locally IN THE BROWSER, but I think that is ok ant it is still possible using the emulator.

This is only a DRAFT for now and I need to clean it up a little more over the next days.
Please let me know what you think!!

This is all fine, but we definitely want to be able to run the games in the browser as the end state for JS, somehow.. as long as that is retained it is all good!

Hi @mojoaxel  , I didn't close this on purpose. This was done automatically when the history was re-written for https://github.com/coding-horror/basic-computer-games/issues/681
I didn't see this PR :see_no_evil: As there were super many changes, I hope those were done automatically :crossed_fingers: If not and if this was a bigger amount of work, we could temporarily restore the old state.
Otherwise, I suggest you to do the following:

Manually copy the files you worked on (hopefully not that many)
delete your local clone of the repository (except for the files you were working on)
Clone again
Add your files back; make a new PR

Sorry for the inconvenience.

I didn't close this on purpose.

No problem! I'll rebase my branch.
Sadly I can not reopen this pull request (maybe @coding-horror can?). I'll propably open a new one!


Disable username / password change forms and link to issues

This three for username change:

[ ] https://github.com/hoodiehq/hoodie-server-account/issues/79
[ ] https://github.com/hoodiehq/hoodie-client-account/issues/75
[ ] https://github.com/hoodiehq/hoodie-server-account/issues/80

These two for password change

[ ] https://github.com/hoodiehq/hoodie-client-account/issues/75
[ ] https://github.com/hoodiehq/hoodie-server-account/issues/80

@gr2m The linked issues are all resolved, changing username (email) and password works. Can we close this issue?

Yes, thanks for looking into it Stefan!


Add a Scroll to top button Matched to website deisgn theme

Do you want to have the enhancement of existing game ? 😀 Describe yourself..
Hey @kunjgit @ishita-43
I would like to create a scroll to top button matched to website design theme as the home page is too lengthy it will helps users to scroll back to the top in just one click!

Describe the solution you'd like
N/A
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @MastanSayyad !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @MastanSayyad ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇

Hey can u assign this issue , i can solve it asap

yes you can comment /assign

/assign

can you please assign me this issue?


feat(FOROME-839): Added Import/Export dataset UI support

https://quantori.atlassian.net/browse/FOROME-839

I would add cursor: pointer  for "Upload file" btn on hover


Add version access-esm1.5 to um7

Closes #117.
Tested by successfully running
spack install um7@access-esm1.5^openmpi<EMAIL_ADDRESS>arch=linux-rocky8-x86_64

Merging.


Support for Laravel 5.8.*

Thanks for the update to Laravel 5.8, but I can't install it because I'm on Laravel 5.8.14.
Would it be possible to update the composer.json to accept 5.8.* or do you need to fix anything before you can do that?
Here's the error I receive when running composer require ahsankhatri/wordpress-auth-provider

Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Conclusion: don't install ahsankhatri/wordpress-auth-provider v2.1.1
    - Conclusion: don't install ahsankhatri/wordpress-auth-provider v2.1.0
    - Conclusion: remove laravel/framework v5.8.14
    - Installation request for ahsankhatri/wordpress-auth-provider ^2.1 -> satisfiable by ahsankhatri/wordpress-auth-provider[v2.1.0, v2.1.1, v2.x-dev].
    - Conclusion: don't install laravel/framework v5.8.14
    - ahsankhatri/wordpress-auth-provider v2.x-dev requires laravel/framework >=5.6 <=5.8 -> satisfiable by laravel/framework[5.6.x-dev, 5.7.x-dev, v5.8.0].
    - Can only install one of: laravel/framework[5.6.x-dev, v5.8.14].
    - Can only install one of: laravel/framework[5.7.x-dev, v5.8.14].
    - Can only install one of: laravel/framework[v5.8.0, v5.8.14].
    - Installation request for laravel/framework (locked at v5.8.14, required as 5.8.*) -> satisfiable by laravel/framework[v5.8.14].

Merged. Closed.


[ROADMAP] Automate and Revamp Flatcar Build Release Pipeline

Automate and Revamp Flatcar Build Release Pipeline
Foundational

[ ] Build the foundation Flatcar Release tool to handle the release.

Pre-Builds

[ ] Build a Release Planning SOP
[ ] Automate the triggering of the Jenkins Release Builds
[ ] Integrate a step to automate the triggering of the release tool - https://github.com/flatcar-linux/Flatcar/issues/650

Post-Builds <-> Pre Go/No-Go

[ ] Implement auto-restarts of Flaky tests to better utilize time
[ ] Automate building of the Go/No-Go document and integrate it into the release tool - https://github.com/flatcar-linux/Flatcar/issues/650

Integrate the changelog-aggregator
Integrate the CVE Generation tool

[ ] Build a Jenkins pipeline to store the meta-information of every manifest, size, content, and package difference.

Post Go/No-Go

[ ] Automate the publishing to Azure Marketplace - https://github.com/flatcar-linux/Flatcar/issues/633
[ ] Automate the publishing to AWS Marketplace
[ ] Automate the publishing to GCP Marketplace
[ ] Integrate the release steps into the release tool

Interact with the Origin
Update SDK
CF templates
Update Website

[ ] Automate Signing Updates and integrate into the Release pipeline

Community

[ ] Create a CI pipeline for when a PR is created in the Github repo(s).

I think this roadmap item overlaps with the above: https://github.com/flatcar-linux/Flatcar/issues/442


Partial block

Addresses #
Summary of changes:

Sorry, wrong upstream.  Gotta change the default :P


855 refactor databag functional arguments

Description
Pass generation as functions from the dataGenerator to generationPartitioner
the only difference between this pr and #866 is the final commit
Split Partitioning responsibility from Generation responsibility
Turn Reductive Walker into a Reductive Generator
Changes
Reductive walker is now a Reductive DataGenerator
There is a DataGenerator specifically for walking
Partitioning and recombining is now a Decorator for the DataGenerator, so is optional
Random Reductive Restarting is now a Decorator for the DataGenerator
DataGeneratorProvider now wraps Reductive or Walking with decorators
All uses of DataBag have now been replaces with GeneratedObject.
All uses of FieldValue have been repalced with DataBagValue.
All uses of DataBagValueSource have been replaced with FieldSpecSource.
DataBag, FieldValue, DataBagValueSource classes have been deleted.
(these were required to do the other refactors)
FieldSpecValueGenerator creates a single DataBagValue rather than a whole DataBag
MultiplexingDataBagSource > simplified and renamed > FieldCombiningDataBagSource
Additional notes
DataWriters now call getOrderedValues and provide an ordering
Reductive no longer uses RowSpec', or DataBagSource's
Config no longer passed to classes that don't need it
FixedFieldStrategy no longer passed to classes that don't need it
Various code that is no longer needed due to this refactor has been deleted.
Resolves #855
Resolves #872

There is some major problem with this, and it drastically affects performance.
Smaller changes can still come in, but this pr is on hold for now


It there a way to set range?

Is there a built in method to set min and max acceptable values.
Use case: I want users to only be able to select values between 1 and 5.

Will add it

Added Check Latest Commit

@ashik94vc Thanks for the quick response.
I set both initialNumber=1 and finalNumber=5 in my layout xml, and use setRange(1, 5) in Java code, yet it has no effect. I can select-minus all the way to 0 and select-plus to numbers greater than 5.

Will look into it.

Cannot be reproduced


keras: add ability to pass live into callback

Suggestion. If we decide it's good I can try to add it to all other integrations.
It makes this workflow (non obvious, not natural, less scalable, and not documented?):
logger = DvcLiveCallback(path="evaluation", report=None)
live = logger.dvclive

into:
live = Live("evaluation", report=None)

# ... somewhere down the road:

.... DvcLiveCallback(dvclive = live)

We need an ability to pass live logger instance or params, otherwise callback would initialize its own instance and will be producing reports for example, would ignore path, etc, etc
It's better to pass live to callbacks (vs asking callback to give us logger): proper order of control (code looks natural, easier to test), one live instance could be shared across multiple callbacks, etc

Codecov Report
Base: 54.38% // Head: 54.35% // Decreases project coverage by -0.03% :warning:

Coverage data is based on head (2fb6cc6) compared to base (56b3096).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #302      +/-   ##
==========================================
- Coverage   54.38%   54.35%   -0.04%
==========================================
  Files          33       33
  Lines        1596     1597       +1
  Branches      152      152
==========================================
  Hits          868      868
- Misses        724      725       +1
  Partials        4        4

Impacted Files
Coverage Δ

src/dvclive/keras.py
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@shcheklein what's the use case here?
I remember discussing this a long time ago and I think the conclusion was that it makes more sense to have a single instance (and output path) associated with the train loop.
The primary motivation was organization-wise (separate inside/outside train loop in folders) and just overall efficiency (having a out of train loop evaluation in a separate stage avoids to recompute training when changing evaluation stuff)

@daavoo Maybe you are thinking of https://github.com/iterative/dvclive/issues/207?

@daavoo use case is when you want to log certain things besides what's being tracked inside one of the callbacks, so you need to have access to the live instance.
This is the way it can be solved:
https://github.com/shcheklein/hackathon/blob/universe/train.py#L55-L56
But I think this is wrong to do it that way (wrong control order). I've outlined a few reasons for that above.

Alternatively maybe live instance can be global (similar to Python logger?) and callbacks can rely on that instance?

@daavoo @dberenbaum yes, #207 and #289 are the same / very related. I hit this bump independently.

Alternatively maybe live instance can be global (similar to Python logger?) and callbacks can rely on that instance?

I think it used to be global but we moved away from it in case users want multiple live instances (for example, they train multiple models). IMO optionally passing live into the callback (as this PR does) is the most flexible since you can choose whether you want to keep everything in one instance or not.

@daavoo use case is when you want to log certain things besides what's being tracked inside one of the callbacks, so you need to have access to the live instance.
This is the way it can be solved:
https://github.com/shcheklein/hackathon/blob/universe/train.py#L55-L56
But I think this is wrong to do it that way (wrong control order). I've outlined a few reasons for that above.

I understand the low-level motivation, my question was about the high-level use case, which in the linked example appears to be something like: "log the best value of each metric so that those are the values shown in the table".
I am ok with the P.R. but my feeling still is that the use cases for this are either workarounds of existing dvclive limitations or cases where would be a better practice to have a separate stage and/or output folder.

but my feeling still is that the use cases for this are either workarounds of existing dvclive limitations or cases where would be a better practice to have a separate stage and/or output folder.

Yes, I think I understand your feelings. My intuition though that this kind of extensibility / flexibility will be still needed (it will be very hard to fit / cover everything with callbacks / higher level APIs). Also, tbh, I don't see that much harm exposing this. We can keep learning from the way people using it and migrate scenarios inside.
I'll try to expand to all other frameworks / add docs if needed.

I think that this approach might be useful if we don't want to log everything on each step:
let's say that we want a callback to log training metrics, but only want to dump something "big" (confusion matrix for example) at the end of the training.


fix flake case TestRotateMaxBackups

Note:  There are so many flakes on TestRotateMaxBackups .  That's all because the rotate is done in a separate go routine, and there is no mechanism to know when the rotation is done. So here temporarily wait a short time 500ms.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: nmittler
If they are not already assigned, you can assign the PR to them by writing /assign @nmittler in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hzxuzhonghu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
889a615043c8fdc03f8d53d141f755051637d974
link
/test istio-unit-tests

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

@hzxuzhonghu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
24b942ba0a151619371cddbf2ee823cf3adbb4eb
link
/test istio-unit-tests

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

/test istio-unit-tests

/assign @hklai

/assign @ozevren

Perhaps merge this to release-1.1 first? We will be merging release-1.1 back to master from time to time.

It is in release-1.1, close this one.


OSX Full screen shortcut

Most applications in OSX have Control+Command+F tied to full screen, while Binary Ninja just has a dropdown for "Enter Full screen". Would it be possible to get that keybinding tied to entering and exiting full screen?

That's implemented via QT so it's slightly different than our normal keybinding system unfortunately. The good news is that for MacOS in particular the generic menu keybindings will work just fine. So if you go to settings/keyboard/shortcuts/app shortcuts you can create a mapping there.
I recommend making one for "Exit Full Screen" too if you do that so the same hotkey toggles.

Actually scratch that -- it's not QT but rather a MacOS feature I think.

Right on, thanks!


Minor changes to allow for subclassing

This there are a few minor changes to lib/regexify.rb to allow for easier use of the Regexify class as a base class.  There is a new file called examples/example.rb to demonstrate.  examples/example.rb contains three different subclasses of Regexify of varying complexity.  The monkey-patch of Regexify is just for the convenience of the test cases at the bottom.
I hope you will consider merging this into the master branch of your repository.
Thanks for letting me contribute.
-- Steve Witten

Probably because this is the first one I've ever done.  I'll re-open it.
Sorry for the newbie mistake.
-- sw
On Sun, Oct 14, 2018 at 7:20 PM Justin Léger<EMAIL_ADDRESS>wrote:

Hey Steve, is there a reason why you closed this PR?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/jusleg/regexify/pull/2#issuecomment-429688660, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACPr9fdP1FgSvXJmrLYzkrJ9GAQw64T9ks5uk_DngaJpZM4Xba9l
.

@stevewi All good. I like what I see. Perhaps we could introduced your variations as options in the main gem. What do you think?

It's your gem...  I only want to use it without having to support it 😉
I don't know what you mean by 'options in the main gem'...  Can you please
explain?  I'm afraid I'm not very familiar with gem packaging either.  Be
aware that the 'RegexifyANSI' class may generate nonsensical Regexps -- and
that the class is an example of what is possible...not what is possible and
correct (I think it needs a little more thought).
I posted a note about the changes and my motivation for them on the PR.  I
don't know if you've read it but, if not, I'd encourage you to.  It sort of
lays out a vision of what I want to accomplish with Regexify.
Regards,
Steve
On Mon, Oct 15, 2018 at 9:12 AM Justin Léger<EMAIL_ADDRESS>wrote:

@stevewi https://github.com/stevewi All good. I like what I see.
Perhaps we could introduced your variations as options in the main gem.
What do you think?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jusleg/regexify/pull/2#issuecomment-429917078, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACPr9aIrjI_FdQZJ6Ezsr0dBRWnuqv4Vks5ulLP_gaJpZM4Xba9l
.

On Mon, Oct 15, 2018 at 11:04 AM Steve Witten<EMAIL_ADDRESS>wrote:

I don't know what you mean by 'options in the main gem'...  Can you please
explain?  I'm afraid I'm not very familiar with gem packaging either.

P.S.  What I've submitted is totally backwards-compatible from a
functionality perspective with your gem.  I'm always leary of introducing
scads of incomprehensible (and possibly incompatible) options.
-- sw


Automatic update system

Evaluate how to build an auto update system. Crucial item in my view since it allows us to distribute the newest versions quickly. But beware we can shoot our self in the foot here!  I love the idea of being able to upgrade without restarts. ATITD (novel non-combat mmog from a while back) even had a "live" patching system that would update even while keeping you online.
Also need to be wary of saved world updates. I tossed some placeholder "version" props in there just as a reminder that we have to have a way to check on blocks that change radically - possible that a block gets split into multiple types, then have to look for old version and make a determination of what to split into what, etc.
This is a big one.


Is it "INT8 or FP8" with "--use_weight_only --weight_only_precision int8 --qformat fp8"

System Info
GPU - A10
Who can help?
@Tracin
Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[X] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
Hi there,
With "--use_weight_only --weight_only_precision int8 --qformat fp8", will the quantization be in INT8 or FP8?
Expected behavior
FP8 dtype when printing out weights.
actual behavior
The weights are in torch.int8.
additional notes
N/A

May I know your full cmd with above parameters？


Cucumber offline setup

Hi everyone,
Is it possible to install cucumber offline? If so, is there any documentation for that?
Best,
Tuğrul.

Please use the support forums for questions and discussions. We use GitHub issues to track bugs and contributions exclusively.


Optimize runtime for non single-module(01,02) compilation

All this changes questionable and I don't know even if it pass specs. Also optimization results was for my set of benchmarks, and I don't know if it would work for something else.
 Before it just marks it in llvm (so inline works only in current llvm module, no cross module inlining). Not sure if backtrace btw works. So now all methods marked by AlwaysInline, like headers in C++. Also mark by this AlwaysInline many small but critical methods, which have most slowdown in per module compilation. This marks was only done for methods, which I only found was to be slow, in my tests. But I am sure there also many methods which I missed.
Also add likely intrinsic which should help in non single-module optimizations.

Results of tests below:
In this PR -O2 became is a decent level of compilation, which runtime is 0.9-2 times slower than release (in 1 test even faster), but compile time is 3-4 times faster, incremental compile time: 5-10 times faster.
--release just little faster after all this optimizations, because single-module and O3 doing similar thing with inlining.
On the recompile compiler this optimizations have no visible effect.
Example of optimization from tests below:
brainfuck
Master: -O2: initial: 2.40s, incremental: 1.33s, run time: 19.71s
This branch: -O2: initial: 2.25s, incremental: 1.32s, run time: 4.90s
brainfuck runtime 4 times faster in this branch. and almost equal release result: 4.26s (which compile time is 6.74s)
Tests:
initial - is compile time with clean cache, incremental - compile same file with 1 line change, run time: benchmark run time in sec.
--release = -O3 --single-module,   = no optimization mode many modules, -O2 = O2 and many modules, -O1 = O1 and many modules.
 Also hard to optimize Tuple.includes? because it use enumerable, so thats why in base64 i replace in?.

There are multiple spec failures, so it looks like this has altered the semantics of some programs

This is really good, awesome work 👍
I think we'll need to look at the individual actions independently, though.
There is already an pending discussion about likely/unlikely intrinsices: #11910. We should continue this part there. I think there's a general consensus on supporting it in some way, but the concrete implementation is still up for debate.
I don't think there has been any dedicated discussion about inlining methods on the Crystal side, except what you already mentioned in #13505. So this would be a new discussion to start. The idea sounds definitely interesting. But there is a wide range of possibilities how it can be implemented.
Then I also see a number of other changes. Not sure what's the reason for adding parenthesis to yield expressions?
Using wrapping math operators in places where overflow is impossible should be an easy thing. I'm wondering if we could do something about this in the compiler even...

parenthesis remains because I try likely here, but somehow it compile error, so I remove it.
Optimizations which add huge impact:

this line: https://github.com/crystal-lang/crystal/pull/14225/files#diff-160d80555792cf986b3f299649b59e7b973ac45a415cafe2978905ccfacce4d1R576, each_index is a method which used in almost every cycle. And remove checked add generate better asm.
all inlines in scr/pointer.cr - Pointer used every where, so making call for it, is huge overhead.
https://github.com/crystal-lang/crystal/pull/14225/files#diff-f424630a978047aee2dd7194fe51fea5a69342ab1a9de74342b0382f01dc6211L40

Awesome work!  @kostya although I don’t understand these codes, but look at the final result, it great!

In fact, I hope the core members can raise the priority for this meaningful discussion.
Don't expect like #13464，it take eight months from proposal to merge, too long, it destroying the enthusiasm of OS contributors, it also destroying the enthusiasm of people using Crystal, In fact, I feel that if it weren’t the hard work of @kostya and @funny-falcon 's  follow-up, #13464 , It probably never been merged until now.

Shouldn’t each_with_index specialized then for common collections? At least Array already have index when it iterates, no need for other var.
I see problem in untyped offset parameter of each_with_index and wrapping addition: what if user pass UInt8 as an offset? What if we iterate external storage, so its index could be larger than 2G?

instead of AlwaysInline, we can for example calculate somehow method complexity, and inline if it small, like <5 instructions(even with all unfolded calls) and not have cycles and not self recursion. But I not see how to do it in current compiler, because it generate llvm directly without semi form.

Another idea, mark full class with AlwaysInline, like Pointer and automatically inline all calls to it.

Also there is already small inlining in generate calls https://github.com/crystal-lang/crystal/blob/master/src/compiler/crystal/codegen/call.cr#L449,  but it inline only primitives, self and getters. It also should inline setters for example.

I experiencing very slow compile std_spec in this branch in -O2, I think this is because all specs compiled in single module (because no classes), so parallel compilation have no benefit here.

@kostya indeed, the _main.bc file for make std_spec is 29MB in crystal's cache.

praise: some very interesting investigation here! Finally some numbers to back some concepts: likely/unlikely + inlining directly instead of merely hinting LLVM :100:

praise: auto-inlining low-complexity methods sounds like a great idea (still need to check for @[NoInline]). Please open issues so we can talk about these (inline in crystal + auto-inline)!

suggestion: it would be even more interesting to know which type of optimization have the most impact (and how much) :eyes:

suggestion: if adding @[AlwaysInline] to all methods in Pointer(T) and Intrinsics has such a positive impact, maybe you can open a pull request with just that? they should be called enough that LLVM should always inline them.

issue: as outlined by @funny-falcon the removed overflows in Enumerable can lead to overlooked overflows (e.g. Enumerable doesn't have to be finite); we can however consider to override the methods in objects with a finite number of iterations :+1:

for simplify testing I add this rep: https://github.com/kostya/crystal-metric
--release: 25.6282s
Master:
-O2: 106.0206s
This branch:
-O2: 30.3626s
Master + only inline pointer:
-O2: 65.6595s

My first branch was separated by commits. This is result for crystal-metric on each commit. For -O2.
By commits:

Initial, https://github.com/kostya/crystal/commit/8b5fbbacc30af58e8aba570fe64d77f752d6d200, 105.7559s
https://github.com/kostya/crystal/commit/675669e185c03e9bde33b17f6bd34db246ae261a, 104.1533s
https://github.com/kostya/crystal/commit/43514d770c972a1206d0196162ffcf6ddbb39468, 70.3383s
https://github.com/kostya/crystal/commit/5d90685231be3bf0fc0cff585fceb0830746e5d9, 70.0416s
https://github.com/kostya/crystal/commit/9e8d79228bdcb8f1c320ade1b82cc058369c66e0, 59.5174s
https://github.com/kostya/crystal/commit/1d212fb71abd75cf032e9ab1d8c4a21c732c7e81, 43.9694s
https://github.com/kostya/crystal/commit/15933d3c4644117455d13e6c2f50776d3c9fdef1, 31.0405s
Likely, https://github.com/kostya/crystal/commit/f3f3c40681a5ca3f2101cda79f0d88d3d166482b, 31.0076s
https://github.com/kostya/crystal/commit/66afa46a0fa64b18289ef8cfec72c9f5ed4b4e8a, 30.6503s
https://github.com/kostya/crystal/commit/059ec72596fd146ea08d339aaa3e75d41c53c748, 29.7027s
https://github.com/kostya/crystal/commit/3222e2811eb8240c7aca265e6bd7f0842b6b8a25, 30.3150s
https://github.com/kostya/crystal/commit/b5f590263e9bc3bac7601a067085ba9d66c355f8, 30.2319s
https://github.com/kostya/crystal/commit/9044866cfc23c5f733b9b0710ed352b3d54400d6, 30.3091s
https://github.com/kostya/crystal/commit/2bd93406a9f56377dba43b1f328fba1b3a3a9328, 29.7686s
https://github.com/kostya/crystal/commit/3ad621fc220cc1c9cdef341df02f24a043046caa, 30.2483s
https://github.com/kostya/crystal/commit/93097f7770003054555848d5757edc280a4067e2, 30.0238s
Auto inline setters, https://github.com/kostya/crystal/commit/4291b7b32676f600a5e1237d4c81a26ea0e92d3b, 29.7598s

Very nice! So, these 4 commits had the most impact:
https://github.com/kostya/crystal/commit/43514d770c972a1206d0196162ffcf6ddbb39468, 70.3383s
https://github.com/kostya/crystal/commit/9e8d79228bdcb8f1c320ade1b82cc058369c66e0, 59.5174s
https://github.com/kostya/crystal/commit/1d212fb71abd75cf032e9ab1d8c4a21c732c7e81, 43.9694s
https://github.com/kostya/crystal/commit/15933d3c4644117455d13e6c2f50776d3c9fdef1, 31.0405s
Looking at them, it's mostly inlining methods, some (un)likely and a few removed overflow checks in Enumerable. I believe what @funny-falcon and I suggested above stands?

I see one more thing to improve, which I don't know how to do, this is record.new.
record Bla, x : Int
Bla.new

Bla.new - generate call and slowdown many benchmarks, but if I just replace to Tuple, benchmarks became much faster.

Plotting this changes for crystal-metric rep. 1.11.2-my is a 1.11.2 recompiled on my MacBook m1 in release mode. Other releases downloaded from GitHub. Funny that it is have faster compile time, may be because of Rosetta, or maybe because of https://github.com/crystal-lang/crystal/issues/12060


Add built-in support for IAsyncEnumerable

Description
Adds built-in support for IAsyncEnumerable for runtimes implementing .NET Standard 2.1 or greater
Closed issues

Checklist

[ ] Reviewed the contribution guidelines
[ ] Linked the issue(s) the PR closes
[ ] Implemented automated tests and checked coverage
[ ] Provided inline documentation comments for new public API
[ ] Ran the full solution build and validation locally

@Kralizek I don't know how you managed to trigger the GitHub Actions job, but this is amazing! I was sure the jobs trigger only from the master branch. Just to think I almost gave up on .NET 6 for v5.
Anyway, back to the topic at hand. The support for IAsyncEnumerable sounds like an awesome idea. I was planning to implement it myself. Even had a PoC in #1233 . I'll have a look later today at the PR.

@aivascu I just pushed commits to this PR :P

@Kralizek I pushed commits as well and it didn't trigger 😕

Mine were better commits 😜😜😜

@aivascu thank you for fixing the other errors. Maybe we should split this PR in two parts? (One to support .net standard 2.1 and one for IAE)

@Kralizek yes I think it's worth doing it in separate PRs. Do you want to create the PR or should I cherry-pick the commits from this branch and open it?

@aivascu i think it's faster if you do it and merge it right after and i can rebase this one onto the new v5

@Kralizek opened the PR for netstandard2.1 and waiting for the pipelines to finish. In the meantime I added an issue for you to close #1325 .
Also see the checklist above.


Task Azure SQL Database Deployment shows "The remote server returned an error: (500) Internal Server Error" while from within VS it works fine

We have Azure SQL Database Deployment task in the middle of our RMO definition and it fails with error:
FilePath= Find-VstsFiles LegacyPattern D:\_work_caas\95a7cdab5\PyRq - Daily\drop\Data.Schema\bin\Release\Data.Schema.dacpac
packageFile= D:\_work_caas\95a7cdab5\PyRq - Daily\drop\Data.Schema\bin\Release\Data.Schema.dacpac
PublishProfilePath = Find-VstsFiles LegacyPattern D:\_work_caas\95a7cdab5\PyRq - Daily\drop\Data.Schema\Properties\PublishProfiles\Int.publish.xml
Publish profile path = D:\_work_caas\95a7cdab5\PyRq - Daily\drop\Data.Schema\Properties\PublishProfiles\Int.publish.xml
##[error]Status         : ProtocolError
##[error]Response       : System.Net.HttpWebResponse
##[error]Message        : The remote server returned an error: (500) Internal Server Error.
##[error]Data           : {}
##[error]InnerException :
##[error]TargetSite     : System.Net.WebResponse GetResponse(System.Net.WebRequest)
##[error]StackTrace     :    at Microsoft.PowerShell.Commands.WebRequestPSCmdlet.GetResponse(WebRequest request)
##[error]                    at Microsoft.PowerShell.Commands.WebRequestPSCmdlet.ProcessRecord()
##[error]HelpLink       :
##[error]Source         : Microsoft.PowerShell.Commands.Utility
##[error]HResult        : -2146233079

However, publishing from withing VS2015.3 shows way more meaningful error:
(58,1): SQL72014: .Net SqlClient Data Provider: Msg 50000, Level 16, State 127, Line 12 Rows were detected. The schema update is terminating because data loss might occur.
(47,0): SQL72045: Script execution error.  The executed script:
An error occurred while the batch was being executed.

Full log: https://mpsit.visualstudio.com/DefaultCollection/PIExp/_apps/hub/ms.vss-releaseManagement-web.hub-explorer?definitionId=4&releaseId=808&_a=release-logs

Release to out Daily env works fine from within both RMO and VS. However any deployment attempts to Int from RM fail with HTTP 500, and works fine from VS.

I believe this might be related to the recent outage of SQL Azure in West US region. DNS records were changed, some other internal staff I'm not really familiar with. I opened an ICM ticket to resolve that. For now all services that rely on API to manage SQL Azure are end up with HTTP 500.

@abatishchev
In response task is not giving an understandable message I will create a bug for that and will be fixed with next deployment,.
Next, you are facing issue that "The schema update is terminating because data loss might occur". In most of cases this error occurs if you did some changes in you schema in dacpac which can cause some data loss.
In your database project there is checkbox for this you can uncheck checkbox for "Block incremental deployment"  it if you surely want those changes. Refer below image.

hi Ajay, yes, I was facing various errors (from data loss to missing extended properties) which I was able to learn only from the output of VS, in contrast RMO always shows 500.

@abatishchev
Thanks for reporting this issue. We will fix this issue soon.

@abatishchev
Using RMO it was failing on creating firewall rules failed before executing your dacpac but creating firewall should also giving understandable message which is here you getting as object of System.Net.HttpWebResponse. We have created internal bug for this. Since your release is succeeding now I am closing this issue and tracing it as part of internal bug. If you again face this issue feel free to reopen this bug again.

@abatishchev
Hi Alexander,
We use REST Apis to Create Firewall Rule & post deploy Operations to Delete Firewall.
As noted by you there were 500s due to an outage, the script was failing out while trying to Create Firewall Rules.
If that part was working correctly, you would have got data loss error, which we do not suppress.
You can try that again(Since there is no outage right now) and will notice that sqlpackage.exe throws data loss error
(In case there is an issue there which you want to report , please run release with System.Debug as true & share logs)
Thanks & Regards
Rohit Batra
PS :- Although SqlPackage throws Data Loss Error , in some cases we have observed that this error is not that verbose.
This is due to the way we handle error from sqlpackage.exe errors & we have a bug opened to capture more details.

Hi Ajay and Rohit,
Thanks for detailed and thoughtful response!
Yes, the outage has been mitigated, my server loved back to the right control ring, now both Portal and RMO work fine.
P.S.
Can you please email me (alexbat@) the links to those two bugs so I could subscribe?


P256 Public Key Generator

Component
Forge
Describe the feature you would like
It would be very useful for testing of P256 signatures to have a forge function that can generate / derive the public key coordinates based on the secp256r1 curve from a random uint256 private key:
function generatePublicKeyP256(uint256 privateKey) external pure returns (uint256 publicKeyX, uint256 publicKeyY)

Maybe, we can think this further, and add an Enum for the elliptic curve used (e.g. secp256k1, secp256r1 etc.) to have the possibility to add further curves without breaking the API.

I'd love to work on this :)

I'd love to work on this :)

Assigned

Quick questions does this need to be here https://github.com/foundry-rs/foundry/blob/0b03a58b4a42b149a27e0b6cc1ff9559306f3603/crates/cheatcodes/src/utils.rs#L19 or in the forge crate if so where?

in the cheatcodes
see https://github.com/foundry-rs/foundry/blob/29e51120ac6e77a62242364a22108edea7274a2a/crates/cheatcodes/spec/src/vm.rs#L305-L307
and
https://github.com/foundry-rs/foundry/blob/29e51120ac6e77a62242364a22108edea7274a2a/crates/cheatcodes/src/evm.rs#L164-L169
for reference

Hi @protocolwhisper how is it going? Do you need any pointers or have any questions?

Hi @zerosnacks, it's going well I've just submitted the pull requests + tests :)

Can you please add the function to the vm interface in forge-std, otherwise hard to use 🙃


bug(dialog): custom reactive form control in a dialog created with ng-template throw error

Is this a regression?

[x] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
v13
Description
The custom reactive  form control in a dialog created with ng-template will throw the following error:
Error: NG01350: ngModel cannot be used to register form controls with a parent formGroup directive. Try using
formGroup's partner directive "formControlName" instead

Reproduction in StackBlitz
It has no problems in v12 and v13.
Reproduction
Steps to reproduce:

create a custom form control using ngModel
create a dialog with ng-template and a reactive form
open the dialog

Expected Behavior
Don't throw errors.
Actual Behavior
Throw the error NG01350.
Environment

Angular: 14.1.3
CDK/Material: 14.1.3
Browser(s): Chrome latest
Operating System (e.g. Windows, macOS, Ubuntu):

I'll try to solve this issue

I'll try to solve this issue

Any news?

I have the same issue, but in my case, I used cdk/dialog

Same issue here. This is a real problem because I'm using a vendor component I try to inject in a MatDialog. I can't really change that behavior.

This is so annoying. I strongly think the most reusable way of embedding pure components in dialogs is using ng-templates.
Please let us know if there is anything we can do to provide some support on this issue. @maykonpacheco

It seems like the problem has been fixed in angular 16 with this PR

It seems like the problem has been fixed in angular 16 with this PR

I think that is possible. Especially with the new reactivity model.


WIP: Codegen option to create module hierarchy

Work-in-progress PR for generating protobufs in a module hierarchy

resolves #260
resolves #438
resolves #644
prerequisite #687,

TODO:

add tests
code generate the mod.rs files I'm currently writing by hand when testing in a another repo which uses a build.rs snippet like:

let manifest_dir = PathBuf::from(env::var("CARGO_MANIFEST_DIR")?);
let out_dir = manifest_dir.join("src").join("test_pb");
codegen.out_dir(out_dir);
codegen.customize(
    Customize::default()
        .gen_mod_rs_hierarchy_out_dir_mod_name("test_pb".to_owned())
        .gen_mod_rs(false),
);

Need guidance on:

intended usage of idents, components in RustPath-like structures

Testing output in a tonic::Codec PR that I'll probably move to tonic repo once ready: https://github.com/appbiotic/code/pull/21. I'll be checking in the generated tree of mod.rs files temporarily to show progress. The proto tree seems to compile OK but haven't done comprehensive tests yet.

Sorry the project is low support.
If you have some specific question, I'll try to answer.

No worries, the project really helped me get started with ffi this past year, both as user and reading the code. Thanks for checking in on me.
Wanted to do something that made includes like a monorepo. Just switched over to cbindgen this week so I could do a header file per crate and being able to specify adding things like the dependencies includes. Doesn't have all the nice types like safer-ffi, but I only need return and free raw mut ptr structs so works ok at the moment.


Update the cycle frontier

Game Title
the cycle frontier
Status of the Anti-Cheat
😥 Broken
Tinkering Steps
The game is officially dead from 27 september
Proof of Mention
https://store.steampowered.com/news/app/868270/view/3685681434321802502

Wouldn't it be better to simply remove it then?


Challenge 03 Release

Add challenge-03/challenge-03.ipynb
Modified release date in root README

Some comments in the PR, some here:

Short intro to explain what the problem is about to those who don't know it (see below)
Don't want to push my own materials too hard, but this might be actually helpful, I'll let you decide: https://github.com/mstechly/quantum_tsp_tutorials
And you should definitely add this as a reference: https://docs.dwavesys.com/docs/latest/doc_handbook.html

Intro:
The core of this challenge is about translating a real-world problem into a mathematical representation that fits a quantum computer. In this challenge we encourage you to use two representations which even though equivalent, are used in different contexts.
The first is Quadratic Unconstrained Binary Optimization (QUBO) which is used primarily for the quantum annealers (D-Wave) and the other is Ising Hamiltonian, which is often used with an algorithm called Quantum Approximate Optimization Algorithm (QAOA).

This challenge might require spending some time going through the references (especially 1-3), as we are not aware of any "quick introductions" worth mentioning. If you have hard time understanding something, feel free to reach out for help on QOSF Slack [TODO: invite link].

Unless you have a prior background with this kind of problems, we encourage you to start from QUBO and then perhaps trying Ising Hamiltonians as well, as there are more materials about it.

This is also worth mentioning that D-Wave allows you to use their machines for free for limited amount of time, so you can try running your QUBO on a real machine after signing up [here](https://cloud.dwavesys.com/leap/signup/).

Some comments in the PR, some here:

Short intro to explain what the problem is about to those who don't know it (see below)
Don't want to push my own materials too hard, but this might be actually helpful, I'll let you decide: https://github.com/mstechly/quantum_tsp_tutorials
And you should definitely add this as a reference: https://docs.dwavesys.com/docs/latest/doc_handbook.html

Intro:
The core of this challenge is about translating a real-world problem into a mathematical representation that fits a quantum computer. In this challenge we encourage you to use two representations which even though equivalent, are used in different contexts.
The first is Quadratic Unconstrained Binary Optimization (QUBO) which is used primarily for the quantum annealers (D-Wave) and the other is Ising Hamiltonian, which is often used with an algorithm called Quantum Approximate Optimization Algorithm (QAOA).

This challenge might require spending some time going through the references (especially 1-3), as we are not aware of any "quick introductions" worth mentioning. If you have hard time understanding something, feel free to reach out for help on QOSF Slack [TODO: invite link].

Unless you have a prior background with this kind of problems, we encourage you to start from QUBO and then perhaps trying Ising Hamiltonians as well, as there are more materials about it.

This is also worth mentioning that D-Wave allows you to use their machines for free for limited amount of time, so you can try running your QUBO on a real machine after signing up [here](https://cloud.dwavesys.com/leap/signup/).


It seems that mounting a file share on a ACI behaves like mounting bind mounts rather than mounting Docker volumes.
From Docker:

If you mount an empty volume into a directory in the container in which files or directories exist, these files or directories are propagated (copied) into the volume. Similarly, if you start a container and specify a volume which does not already exist, an empty volume is created for you. This is a good way to pre-populate data that another container needs.

If you mount a bind mount or non-empty volume into a directory in the container in which some files or directories exist, these files or directories are obscured by the mount, just as if you saved files into /mnt on a Linux host and then mounted a USB drive into /mnt. The contents of /mnt would be obscured by the contents of the USB drive until the USB drive were unmounted. The obscured files are not removed or altered, but are not accessible while the bind mount or volume is mounted.

When you mount a File Share from a Storage Account into an ACI, it behaves like the second case. If this is correct, I'd be nice if it was clarified in the document. Otherwise, there might be something not working, as that is the behaviour I'm observing 😝
Thanks!

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

ID: b730fc95-951e-1acd-770d-88b3e6f2f6ef
Version Independent ID: b6445fd2-46ea-aedd-2f2c-294faa660660
Content: Mount an Azure Files volume in Azure Container Instances
Content Source: articles/container-instances/container-instances-volume-azure-files.md
Service: container-instances
GitHub Login: @dlepow
Microsoft Alias: danlep

Adding @dkkapur who might be able to assist further.

@hjlarrea thanks for the patience while I looked into this.
The statement from docker is correct and we expect it to behave like #2. I am working to update the document now.

@MicahMcKittrick-MSFT - Micah, I'll add myself to follow up on doc updates as well. I'll work with Deep to publish the clarification. Thanks for getting the right info!
#assign: @dlepow

@MicahMcKittrick-MSFT - Do you know if there are intentions of adding the possibility of mounting shares or blobs as proper Docker volumes in ACI? Thanks again!

@hjlarrea I am not personally sure. @dlepow or @dkkapur might be able to tell you though.

@hjlarrea - The updated article referring to bind mounts has been published, and I'm closing this issue. Thanks for your suggestion!
For your question about plans to mount shares/volumes as Docker volumes, I'm not aware of plans currently. Please feel free to post your idea at Azure UserVoice for others in the community to vote on.

#please-close


Two steps display at once

I'm having a problem where 2 steps display at the same time, one right under the other.  Sometimes this happens if the user clicks to edit a record & goes thru the wizard, then clicks cancel & then clicks to create a new record.
Other times, it happens a single step is very long, so the user has to scroll way down to click the wz-next button.

Can you provide a JSFiddle replicating the example? I have never experienced what you described or heard of anyone experiencing anything like that.  A working example of the error will help me debug it.

Thank you for getting back to me.  I'll try to make an example.  Our code is proprietary & uses Telerik Kendo Grid & a WebAPI back end.  There is a possibility of a CSS conflict w/ the Kendo Grid CSS.
Just to make sure I have not overlooked something obvious -- which version(s) of AngularJS is Angular Wizard compatible with?  (I'm assuming it works with the latest version of Angular, but I just want to make sure I'm not overlooking something simple.)

exemple here -> jsFiddle

Hi,
I am able to replicate the issue, and attached the sample.
This happens if we load the wz-step with templated html.
Please see the attached template.
WebSite1.zip

Hi, any progress? Could you remove the "waiting for example" label since @dreamerNcoder provided an example?

@dreanerNcoder, see my sample jsFidle,
open the debug browser, you will see that no template, the canexit funcion is called every time.

Label removed, I will look into this more in the next couple of days, thank you.