Starting directory as legacy cmd

When the command prompt is opened by the Win+X shortcut without administrator privilege, the working directory is %USERPROFILE%. If opened with administrator privilege, the working directory is %WINDIR%\system32.
But when the terminal is set to be the default terminal for the system and opened by the Win+X menu, if starting directory is set to ., it is always %WINDIR%\system32.
Is it possible to set the starting directory in windows terminal to make it behave like cmd?

Huh, good catch.
Just for sanity - are you on Windows 10 or 11?
It looks like the old "C:\Users\migrie\AppData\Local\Microsoft\Windows\WinX\Group3\02 - Command Prompt.lnk" shortcut MANUALLY specified the CWD as %USERPROFILE%.
But it looks like there's special magic code that redirects the pwsh / cmd links to Terminal, and we're not sure if they're doing it right 😨
We're gonna go file this internally. Thanks!

Huh, good catch.
Just for sanity - are you on Windows 10 or 11?
It looks like the old "C:\Users\migrie\AppData\Local\Microsoft\Windows\WinX\Group3\02 - Command Prompt.lnk" shortcut MANUALLY specified the CWD as %USERPROFILE%.
But it looks like there's special magic code that redirects the pwsh / cmd links to Terminal, and we're not sure if they're doing it right 😨
We're gonna go file this internally. Thanks!

It's Windows 11 that I'm using, in fact I don't remember if Windows 10 allows user to configure system default terminal 😂.
I guess there can be some user side imitation of that behavior (such as changing the shortcut to some batch script) and wanted help to get normal and explicit solutions initially. But I'm really gladder to hear that this might be fixed internally.

Internally: MSFT:49685255


Calculate delta of benchmark results

Post process JMH JSON with Java instead of Python

Codecov Report

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

@@            Coverage Diff            @@
##             master      #76   +/-   ##
=========================================
  Coverage     75.77%   75.77%
  Complexity      723      723
=========================================
  Files            78       78
  Lines          2621     2621
  Branches        233      233
=========================================
  Hits           1986     1986
  Misses          497      497
  Partials        138      138

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 b453093...c9870f0. Read the comment docs.


CLI: add classes to export argument

The main exporters have everything in place to filter by type of class to export (enum, primitive, resource, etc), but it seemed there was not yet a command-line parameter to actually specify it.  Added this parameter.

Oops, included other changes too, removingh.


Describe and implement rules on labels usage

https://kotlinlang.org/docs/reference/returns.html
add it to chapter 5

We should not prohibit nested blocks with labels, but should prohibit simple loops/lambdas where labels are used

We should not prohibit nested blocks with labels, but should prohibit simple loops/lambdas where labels are used


Figure out how to create a framework for modifying the pressure after reinitialization

Wyatt was able to put together a basic method on the Nalu side, and can likely be done on the AMR-Wind side as well. Can change BCs. Worried about the decoupling of pressure: one solver has different pressure than the other.

Met on 10/2. Dynamic pressure idea didn't work. Now have a better idea and path forward. Still unclear what to do on the Nalu-Wind side. Have already achieved much better results ignoring the Nalu-Wind pressures. Pressure needs to coincide with own description of the velocity. Michael will push forward with AMR-Wind ignoring pressures from Nalu-Wind. As Wyatt works his side out, this will get even better.

Wyatt to set up a meeting today to discuss.
Michael is testing using the pressure gradient from Nalu-Wind in AMR-Wind for the simulation.

Michael was able to run a wave problem and the results look good. Now Michael is going to run the actual milestone problem.
We determined that a lot of the reinit was not needed. Wyatt feels it would be good to have a feature that sets up meshes. We have a good path forward.


Port all CLIs to Clize and eliminate ArgParser

Addresses #GitHubissuenum
Description
Testing:

[ ] Ran tests and they passed OK
[ ] Added new tests for the new feature(s)

CLA

[ ] If a new developer, signed up to CLA

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 16
- Added 12

Complexity increasing per file
==============================
- lib/improver/cli/apply_emos_coefficients.py  2
- lib/improver/cli/nbhood_land_and_sea.py  4
- lib/improver/cli/generate_orographic_alphas.py  1
- lib/improver/cli/spot_extract.py  1
- lib/improver/tests/nbhood/test_init.py  2
- lib/improver/tests/acceptance/test_spot_extract.py  1
- lib/improver/tests/acceptance/test_generate_orographic_alphas.py  1
- lib/improver/wxcode/wxcode_utilities.py  6

Complexity decreasing per file
==============================
+ lib/improver/cli/update_grid_metadata.py  -2
+ lib/improver/tests/acceptance/test_nbhood_iterate_with_mask.py  -2
+ lib/improver/cli/nbhood.py  -11
+ lib/improver/cli/generate_topography_bands_mask.py  -3
+ lib/improver/tests/acceptance/test_generate_topography_bands.py  -1
+ lib/improver/cli/combine.py  -1
+ lib/improver/tests/utilities/test_cli_utilities.py  -1
+ lib/improver/cli/generate_topography_bands_weights.py  -2
+ lib/improver/cli/generate_landmask_ancillary.py  -2
+ lib/improver/cli/standardise.py  -10
+ lib/improver/utilities/cli_utilities.py  -4
+ lib/improver/tests/acceptance/test_update_grid_metadata.py  -1
+ lib/improver/cli/wxcode.py  -9
+ lib/improver/tests/acceptance/test_temporal_interpolate.py  -3

Clones removed
==============
+ lib/improver/cli/nbhood_land_and_sea.py  -2
+ lib/improver/cli/nbhood.py  -2
+ lib/improver/cli/generate_topography_bands_mask.py  -1
+ lib/improver/cli/generate_topography_bands_weights.py  -1
+ lib/improver/tests/utilities/test_GenerateTimeLaggedEnsemble.py  -4
+ lib/improver/tests/acceptance/test_standardise.py  -2
+ lib/improver/cli/nbhood_iterate_with_mask.py  -2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 15
- Added 13

Complexity increasing per file
==============================
- lib/improver/cli/apply_emos_coefficients.py  2
- lib/improver/wxcode/wxcode_utilities.py  6
- lib/improver/cli/nbhood_land_and_sea.py  4
- lib/improver/cli/generate_orographic_alphas.py  1
- lib/improver/tests/nbhood/test_init.py  2
- lib/improver/tests/acceptance/test_generate_orographic_alphas.py  1
- lib/improver/cli/spot_extract.py  1
- lib/improver/tests/acceptance/test_spot_extract.py  1

Complexity decreasing per file
==============================
+ lib/improver/cli/combine.py  -1
+ lib/improver/utilities/cli_utilities.py  -4
+ lib/improver/cli/nbhood.py  -11
+ lib/improver/tests/acceptance/test_generate_topography_bands.py  -1
+ lib/improver/cli/standardise.py  -10
+ lib/improver/tests/acceptance/test_update_grid_metadata.py  -1
+ lib/improver/cli/wxcode.py  -9
+ lib/improver/tests/acceptance/test_nbhood_iterate_with_mask.py  -2
+ lib/improver/tests/acceptance/test_temporal_interpolate.py  -3
+ lib/improver/cli/generate_landmask_ancillary.py  -2
+ lib/improver/cli/generate_topography_bands_mask.py  -3
+ lib/improver/tests/utilities/test_cli_utilities.py  -1
+ lib/improver/cli/update_grid_metadata.py  -2
+ lib/improver/cli/generate_topography_bands_weights.py  -2

Clones removed
==============
+ lib/improver/cli/nbhood_land_and_sea.py  -2
+ lib/improver/tests/utilities/test_GenerateTimeLaggedEnsemble.py  -2
+ lib/improver/cli/nbhood.py  -2
+ lib/improver/tests/acceptance/test_standardise.py  -2
+ lib/improver/cli/generate_topography_bands_mask.py  -1
+ lib/improver/cli/nbhood_iterate_with_mask.py  -2
+ lib/improver/cli/generate_topography_bands_weights.py  -1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 15
- Added 13

Complexity increasing per file
==============================
- lib/improver/tests/nbhood/test_init.py  2
- lib/improver/wxcode/wxcode_utilities.py  6
- lib/improver/tests/acceptance/test_generate_orographic_alphas.py  1
- lib/improver/cli/spot_extract.py  1
- lib/improver/tests/acceptance/test_spot_extract.py  1
- lib/improver/cli/nbhood_land_and_sea.py  4
- lib/improver/cli/apply_emos_coefficients.py  2
- lib/improver/cli/generate_orographic_alphas.py  1

Complexity decreasing per file
==============================
+ lib/improver/cli/nbhood.py  -11
+ lib/improver/cli/update_grid_metadata.py  -2
+ lib/improver/tests/acceptance/test_generate_topography_bands.py  -1
+ lib/improver/cli/generate_topography_bands_weights.py  -2
+ lib/improver/utilities/cli_utilities.py  -4
+ lib/improver/tests/acceptance/test_nbhood_iterate_with_mask.py  -2
+ lib/improver/cli/generate_topography_bands_mask.py  -3
+ lib/improver/tests/acceptance/test_temporal_interpolate.py  -3
+ lib/improver/cli/standardise.py  -10
+ lib/improver/cli/generate_landmask_ancillary.py  -2
+ lib/improver/cli/wxcode.py  -9
+ lib/improver/cli/combine.py  -1
+ lib/improver/tests/utilities/test_cli_utilities.py  -1
+ lib/improver/tests/acceptance/test_update_grid_metadata.py  -1

Clones removed
==============
+ lib/improver/cli/nbhood_iterate_with_mask.py  -2
+ lib/improver/cli/nbhood.py  -2
+ lib/improver/cli/nbhood_land_and_sea.py  -2
+ lib/improver/cli/generate_topography_bands_weights.py  -1
+ lib/improver/cli/generate_topography_bands_mask.py  -1
+ lib/improver/tests/acceptance/test_standardise.py  -2
+ lib/improver/tests/utilities/test_GenerateTimeLaggedEnsemble.py  -2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 15
- Added 13

Complexity increasing per file
==============================
- lib/improver/cli/apply_emos_coefficients.py  2
- lib/improver/tests/acceptance/test_generate_orographic_alphas.py  1
- lib/improver/cli/generate_orographic_alphas.py  1
- lib/improver/cli/nbhood_land_and_sea.py  4
- lib/improver/wxcode/wxcode_utilities.py  6
- lib/improver/tests/nbhood/test_init.py  2
- lib/improver/cli/spot_extract.py  1
- lib/improver/tests/acceptance/test_spot_extract.py  1

Complexity decreasing per file
==============================
+ lib/improver/cli/combine.py  -1
+ lib/improver/cli/generate_topography_bands_mask.py  -3
+ lib/improver/tests/acceptance/test_generate_topography_bands.py  -1
+ lib/improver/cli/generate_landmask_ancillary.py  -2
+ lib/improver/cli/nbhood.py  -11
+ lib/improver/cli/standardise.py  -10
+ lib/improver/tests/acceptance/test_temporal_interpolate.py  -3
+ lib/improver/utilities/cli_utilities.py  -4
+ lib/improver/tests/acceptance/test_nbhood_iterate_with_mask.py  -2
+ lib/improver/cli/update_grid_metadata.py  -2
+ lib/improver/tests/utilities/test_cli_utilities.py  -1
+ lib/improver/tests/acceptance/test_update_grid_metadata.py  -1
+ lib/improver/cli/generate_topography_bands_weights.py  -2
+ lib/improver/cli/wxcode.py  -9

Clones removed
==============
+ lib/improver/cli/generate_topography_bands_mask.py  -1
+ lib/improver/cli/nbhood_iterate_with_mask.py  -2
+ lib/improver/tests/utilities/test_GenerateTimeLaggedEnsemble.py  -2
+ lib/improver/tests/acceptance/test_standardise.py  -2
+ lib/improver/cli/nbhood_land_and_sea.py  -2
+ lib/improver/cli/nbhood.py  -2
+ lib/improver/cli/generate_topography_bands_weights.py  -1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 15
- Added 13

Complexity increasing per file
==============================
- lib/improver/cli/generate_orographic_alphas.py  1
- lib/improver/tests/nbhood/test_init.py  2
- lib/improver/cli/nbhood_land_and_sea.py  4
- lib/improver/tests/acceptance/test_spot_extract.py  1
- lib/improver/wxcode/wxcode_utilities.py  6
- lib/improver/cli/spot_extract.py  1
- lib/improver/tests/acceptance/test_generate_orographic_alphas.py  1
- lib/improver/cli/apply_emos_coefficients.py  2

Complexity decreasing per file
==============================
+ lib/improver/cli/update_grid_metadata.py  -2
+ lib/improver/tests/acceptance/test_update_grid_metadata.py  -1
+ lib/improver/tests/utilities/test_cli_utilities.py  -1
+ lib/improver/cli/generate_topography_bands_weights.py  -2
+ lib/improver/tests/acceptance/test_temporal_interpolate.py  -3
+ lib/improver/cli/combine.py  -1
+ lib/improver/tests/acceptance/test_generate_topography_bands.py  -1
+ lib/improver/tests/acceptance/test_nbhood_iterate_with_mask.py  -2
+ lib/improver/cli/generate_topography_bands_mask.py  -3
+ lib/improver/cli/wxcode.py  -9
+ lib/improver/cli/nbhood.py  -11
+ lib/improver/utilities/cli_utilities.py  -4
+ lib/improver/cli/generate_landmask_ancillary.py  -2
+ lib/improver/cli/standardise.py  -10

Clones removed
==============
+ lib/improver/tests/acceptance/test_standardise.py  -2
+ lib/improver/cli/generate_topography_bands_weights.py  -1
+ lib/improver/tests/utilities/test_GenerateTimeLaggedEnsemble.py  -2
+ lib/improver/cli/nbhood_land_and_sea.py  -2
+ lib/improver/cli/generate_topography_bands_mask.py  -1
+ lib/improver/cli/nbhood.py  -2
+ lib/improver/cli/nbhood_iterate_with_mask.py  -2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 18
- Added 15

Complexity increasing per file
==============================
- lib/improver/cli/temp_lapse_rate.py  2
- lib/improver/cli/generate_orographic_alphas.py  1
- lib/improver/cli/nbhood_land_and_sea.py  5
- lib/improver/tests/nbhood/test_init.py  2
- lib/improver/tests/acceptance/test_spot_extract.py  1
- lib/improver/cli/nbhood_iterate_with_mask.py  1
- lib/improver/cli/spot_extract.py  1
- lib/improver/cli/apply_emos_coefficients.py  2
- lib/improver/cli/__init__.py  1
- lib/improver/tests/acceptance/test_generate_orographic_alphas.py  1
- lib/improver/wxcode/wxcode_utilities.py  6

Complexity decreasing per file
==============================
+ lib/improver/cli/generate_topography_bands_mask.py  -3
+ lib/improver/cli/standardise.py  -10
+ lib/improver/tests/utilities/test_cli_utilities.py  -1
+ lib/improver/tests/acceptance/test_temporal_interpolate.py  -3
+ lib/improver/cli/combine.py  -3
+ lib/improver/tests/acceptance/test_generate_topography_bands.py  -1
+ lib/improver/cli/update_grid_metadata.py  -2
+ lib/improver/utilities/cli_utilities.py  -4
+ lib/improver/cli/probabilities_to_realizations.py  -5
+ lib/improver/cli/percentiles_to_realizations.py  -6
+ lib/improver/cli/nbhood.py  -10
+ lib/improver/cli/wxcode.py  -9
+ lib/improver/cli/generate_topography_bands_weights.py  -2
+ lib/improver/tests/acceptance/test_update_grid_metadata.py  -1
+ lib/improver/tests/acceptance/test_probabilities_to_realizations.py  -1
+ lib/improver/cli/generate_landmask_ancillary.py  -2
+ lib/improver/tests/acceptance/test_percentiles_to_realizations.py  -1
+ lib/improver/tests/acceptance/test_nbhood_iterate_with_mask.py  -2

Clones removed
==============
+ lib/improver/tests/acceptance/test_standardise.py  -2
+ lib/improver/cli/generate_topography_bands_mask.py  -1
+ lib/improver/cli/nbhood_land_and_sea.py  -2
+ lib/improver/cli/nbhood_iterate_with_mask.py  -2
+ lib/improver/cli/nbhood.py  -2
+ lib/improver/tests/utilities/test_GenerateTimeLaggedEnsemble.py  -2
+ lib/improver/cli/generate_topography_bands_weights.py  -1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 18
- Added 15

Complexity increasing per file
==============================
- lib/improver/cli/nbhood_iterate_with_mask.py  1
- lib/improver/cli/temp_lapse_rate.py  2
- lib/improver/wxcode/wxcode_utilities.py  6
- lib/improver/tests/acceptance/test_spot_extract.py  1
- lib/improver/cli/__init__.py  1
- lib/improver/cli/nbhood_land_and_sea.py  5
- lib/improver/cli/apply_emos_coefficients.py  2
- lib/improver/tests/nbhood/test_init.py  2
- lib/improver/cli/spot_extract.py  1
- lib/improver/tests/acceptance/test_generate_orographic_alphas.py  1
- lib/improver/cli/generate_orographic_alphas.py  1

Complexity decreasing per file
==============================
+ lib/improver/cli/combine.py  -3
+ lib/improver/utilities/cli_utilities.py  -4
+ lib/improver/tests/acceptance/test_nbhood_iterate_with_mask.py  -2
+ lib/improver/cli/update_grid_metadata.py  -2
+ lib/improver/tests/acceptance/test_probabilities_to_realizations.py  -1
+ lib/improver/tests/acceptance/test_temporal_interpolate.py  -3
+ lib/improver/tests/acceptance/test_update_grid_metadata.py  -1
+ lib/improver/tests/acceptance/test_generate_topography_bands.py  -1
+ lib/improver/tests/acceptance/test_percentiles_to_realizations.py  -1
+ lib/improver/cli/generate_landmask_ancillary.py  -2
+ lib/improver/cli/generate_topography_bands_weights.py  -2
+ lib/improver/cli/probabilities_to_realizations.py  -5
+ lib/improver/cli/wxcode.py  -9
+ lib/improver/cli/generate_topography_bands_mask.py  -3
+ lib/improver/cli/nbhood.py  -10
+ lib/improver/cli/standardise.py  -10
+ lib/improver/tests/utilities/test_cli_utilities.py  -1
+ lib/improver/cli/percentiles_to_realizations.py  -6

Clones removed
==============
+ lib/improver/cli/nbhood_iterate_with_mask.py  -2
+ lib/improver/cli/nbhood_land_and_sea.py  -2
+ lib/improver/tests/utilities/test_GenerateTimeLaggedEnsemble.py  -2
+ lib/improver/cli/generate_topography_bands_weights.py  -1
+ lib/improver/cli/generate_topography_bands_mask.py  -1
+ lib/improver/cli/nbhood.py  -2
+ lib/improver/tests/acceptance/test_standardise.py  -2

See the complete overview on Codacy


Newsletters: add 203 (2022-06-08)

[ ] Lede, releases/RCs, topic updates @harding (plus maybe any additional stories)
[x] Bitcoin Core PR Review Club @glozow
[x] Bitcoin Core 24408 new RPC @adamjonas

Short and sweet. Nothing from me.

Pushed a few tiny edits to @glozow and @adamjonas sections (thank you both!  I especially appreciate learning something nifty about miniscript for this week's description!), updated the RCs, and added the topic links.  Thanks also @adamjonas for the review!

Quick run through LGTM

Small change to PR review section, squashed, and merged! Thank you @harding @glozow @adamjonas


add numerically stable norm?

norm could be implemented by implementing the appropriate custom element types and calling mapreduce

This is great, but it's a distraction from the project goals, and won't be worked on unless someone needs it and requests it.

We just fixed this, but I think I accidentally pushed the changes to main :/


NixOS VM window too big to fit on physical screen

I was playing with Qubes OS and tried to install NixOS 16.09 into a VM there.
Used the graphical installer:

with defaults. After booting from the ISO, when one has to fdisk/mkfs/configure one gets a NixOS VM window that is higher than the physical screen (> 768px).
with "nomodeset". same thing.
with vga=0 on kernel commandline. same thing (it starts ok, but later switches to another mode).

The window can not be resized, so there is no chance seeing the bottom lines of the shell == where one usually inputs stuff, where nano UI asks when saving stuff, ...

modprobe.blacklist=bochs_drm

@volth thanks! I'll try that next time.

(triage) close as resolved?

I didn't retry it yet and it seems like a comment from @volth (see my thanks) is missing now.
So, unless otherwise proven, this is not solved.

is missing now.

That was a bad idea which does not work


Quantization with TFLite

What does this PR do?
This PR adds support for quantization with TFlite in optimum.exporters.tflite:

It refactors optimum/utils/preprocessing, TasksProcessors now offer a load_dataset method, which wraps the Datasets.load_dataset function (with a few additional options) and returns a processed version of the dataset for the considered task. Currently 4 tasks are supported:

Text classification
Token classification
Question answering
Image classification

The following quantization approaches are supported:

Dynamic INT8 quantization
Static INT8 quantization (requires a calibration dataset): it is possible to allow fallback to float kernels and to request for end-to-end integer computation (inputs and outputs are also integers).
Static INT8x16 quantization (requires a calibration dataset): the weights are in int8 and activation are in int16
FP16 quantization

Usage:
optimum-cli export tflite -m csarron/bert-base-uncased-squad-v1 --task question-answering --sequence_length 384 --quantize int8 --num_calibration_samples 100 bert_int8

Heavy testing is done, and the tests are decoupled in multiple github actions to make things faster.
To do in later PRs:

[ ] Document how to use quantization
[ ] Extend the TasksProcessors to all the supported tasks
[ ] Add a validation step after quantization (still needs to be determined how)

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

I think extending the documentation would be great!

Yes, will do it in later PRs, otherwise this PR will become huge.


'--once' option dosen't work well with 'docker exec -it /bin/bash'

It dosen't exit , when I set gotty to a daemon process use nohup or daemon command .

Hi, could you provide the actual commands you tried? gotty --once -w docker run -it --rm ubuntu /bin/bash worked properly on my environment.

@yudai   daemon  --name=gotty  -U  -o /home/gotty.log -P /home/ -- /usr/sbin/gotty -r -w --once docker exec -it 942d3cd41cc6 /bin/bash
thanks!

It looks working well on my env. Actually, gotty stops working after a client exits the bash process. No additional client cannot connect to gotty after that.
I assume what you want to archive requires the -r option of the daemon command. It restarts a new process once a child process exits. With this option, daemon restarts a new gotty process after the previous gotty client disconnected and the new gotty process has a new random URL.

It stops working and no additional client cannot connect to gotty, but the process of gotty still exist

Did you try with the -r option of the daemon command?
I confirmed that the process of gotty disappears after the first client exits when I don't provide the -r option.

No.  My command is daemon  --name=gotty  -U -o /home/log -F /home/pid -- /usr/sbin/gotty -r -w --once docker exec -it 3d7f353e784e /bin/bash
I want the gotty process exit when I close the client. It works for you?

Ok, I confirmed when I close the browser window (not running exit), gotty gets stuck with the log websocket: close 1001.
I'll fix the issue soon.

thank you

Confirmed that docker command ignores SIGHUP when it's launched by gotty. However, I have no idea why the process ignores the signal. Indeed, it accepts SIGTERM and quits. I need some more time to investigate the root cause of the problem.

Got a little bit progress. When ran daemon --name=gotty -U -o /home/gotty.log -P /home/ -- /usr/sbin/gotty -r -w --once docker exec -it 942d3cd41cc6 top, the process exited normally. So it seems like bash ignores SIGHUP in some circumstance?

Did you tried to call a SIGKILL after the current SIGHUP https://github.com/yudai/gotty/blob/6013e439336db11fb0bcede9bfbd7e912cdfa435/app/client_context.go#L78 ?

@moul I added a new option that configures the signal when gotty kills child processes.
Indeed, SIGKILL always works, however I'm still wondering why only docker exec bash won't exit. There must be a condition that docker or bash ignores SIGHUP.

Nohup/daemon are usually responsible of this kind of behavior by explicitly ignoring SIGHUP signals.
But in this context, docker is behind GoTTY, so the signals between the 2 processes should continue to pass
+------+   +--------------+ +-------+  +-------------+
|      |   |              | |       |  |             |
| bash |   | nohup/daemon | | GoTTY |  | docker exec |
|      |   |              | |       |  |             |
+---+--+   +--------------+ ++----+-+  +----+--------+
    |                        ^    |         ^
    |                        |    |         |
    +------------------------+    +---------+
            SIGHUP ignored       SIGHUP should pass

@moul Yes, that's the interesting point. I try sending SIGHUP directory to the docker exec process from another terminal, but it didn't work either. Mysterious...

@yudai  It issues new progress?  I have this problem too, but i found that when i closed this websocket  gotty code was block at https://github.com/yudai/gotty/blob/master/app/client_context.go#L74 and the processSend not return is this the question

I think this issue is resolved in #159. @upccup 's hint was helpful. Thanks!

Should be fixed in the master branch. Let me know when it still exists.


Receive contact us details on EMAIL instantly for faster response to user

i can implement this feature where you will receive this info instantly on mail
for instant response to user
enhance user experience

@Sawan-Kushwah assigned


Proposal: Storage of sha for files in the tree

User-Problem
Many files got/get shared between users as part of a set. So many users (particularly in the same organization) maintain copies of the same file. If those users create new dats for every file they replicate that file even though they wouldn't need to.
This is particularly important when using a central tool like datBase or hyperdrive that (also) acts as backup for the data. Those central storages could easily just download the same file once for each client.
Code-Problem
Hyperdrive doesn't store the sha information in the tree which makes it impossible to determine the content before entirely downloading a file.
Solution

Add optionally (default=true) sha property to the file-tree item: https://github.com/mafintosh/hyperdrive/blob/2a585c9a7906bd6d432ab9b3f7fd959fb8c3417c/index.js#L585-L595
Add an optional API that allows to lookup files by their sha-hash in a cache. opt.cache.get(hash, cb) (?)

Would you accept a PR that implements this?

This idea has come up before (https://github.com/datproject/discussions/issues/77#issuecomment-354969691) and i'd be in favor of it. There's some more thinking to go in to it though: which hash functions to support? We already use BLAKE2b elsewhere, but md5/sha1 are much more commonly used today for this sort of de-dupe (as opposed to cryptographic assurance). How do we make this feature "future proof", allowing evolution of which hashes are used? What is the specific additional overhead incurred to calculate this hash for large files? Is this even useful if we don't find the "inner" hashes of compressed files as well?
My current opinion is that multiple hash types should be allowed, but we default to just SHA-1 to start. By default clients would compute and add the hash, but not verify on download (rely on merkel tree for that).
I think the cache API/lookup should be done by an external library, at least to start.

has come up before

Thanks! 😍

How do we make this feature "future proof"?

My thinking is to add {hash: "blah", hashFn: "sha-1"} to the tree.

the cache API/lookup should be done by an external library

Yes, but how to trigger and weave it in? My approach would be to add a cache object to opts:
https://github.com/mafintosh/hyperdrive/blob/2a585c9a7906bd6d432ab9b3f7fd959fb8c3417c/index.js#L25
where cache would have a cache.get(${hashFn}/${hash}, cb) method to make sure its partially compatible to leveldb.

This kind of metadata is also useful in applications for establishing happens-before precedence. If you can reference the hash of a file, then you know it was created prior to the hash's publication.

I have no problem adding a flat hash of the file. Someone wants to do a PR
that adds a blake2b?
On Wed, Feb 21, 2018, 19:39 Paul Frazee<EMAIL_ADDRESS>wrote:

This kind of metadata is also useful in applications for establishing
happens-before precedence. If you can reference the hash of a file, then
you know it was created prior to the hash's publication.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mafintosh/hyperdrive/issues/203#issuecomment-367445820,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAW_VaLKzZc7vqh_pTHwPAqvGsxbWYhZks5tXHEJgaJpZM4SOIN0
.

@mafintosh a few questions:

As mentioned in the comments before: should it also store what hashing method was used?
Is it okay to be optional? With a flag in the opts?
What about the cache-lookup API?

@martinheidegger

yes, or just support blake2b for now (your call).
yes should be optional.
cache-api is out of scope for hyperdrive but def something that could have value by itself.

@mafintosh Thanks for the reply. I guess its easy to get started working on it :)

What I think about when I think about future-proofing this:  Can this be parallelized over GPUs when I REALLY need to sync.
I'm curious, too, if using factoradics as an index/hash is a way to optimize dividing up the indexing work.  See:  https://www.researchgate.net/publication/292950384_A_GPU-based_Branch-and-Bound_algorithm_using_Integer-Vector-Matrix_data_structure

Also, the Bela Ban and team of jgroups.org have done a lot of the practical ground-work around optimizing state-sync over tcp and udp.  Yes, it's Java, but the hashing and syncing logic has been optimized for decades now and includes a decent amount of wisdom to consider.  Not as new as a permutahedron, approach, tho.  ;)

See:  http://jgroups.org/demos.html
ReplicatedHashMap

I'm curious to see if a permutahedron could be used for this.


Is "custom fetch" still possible?

I was reviewing the source to see if I could use the "custom fetch" feature to somehow wrap Shopify's Admin GraphQL API with graphql-request, and noticed that the fetch param seems to be ignored vs. the created fetcher, below.
https://github.com/jasonkuhrt/graphql-request/blob/38fda1df17eaa9c50b3a6c2f0e1695c3cc507216/src/index.ts#L394-L410
I'm guessing L406 should be something like const fetcher = params.fetch ?? createHttpMethodFetcher(...);

@anulman you can see here that in the end it uses the fetch param

you’re right! i ended up using it and had no issues; forgot to come back and close this. thank you, and sorry for the false alarm 😅


[New Port Request] libcsnd6

Library name
libcsnd6
Library description
C++ bindings for the Csound API
Source repository URL
https://github.com/csound/csound
Project homepage (if different from the source repository)
https://csound.com/
Anything else that is useful to know when adding (such as optional features the library may have that should be included)
There's also libcsound64, the C API, which should be a dependency I think. There are also csound plugins, the one I'm using is the STK plugin, having a way to install that would be helpful too.

No stale


Not working with NextJS

React v18.2.0
Next v12.2.5
Node v16.16.0

Same here

Can you guys post a reproduction of the bug on https://next.new?

I see that this was close but I'm still getting this error with Next. Was there a fix? If so what was the fix?


[Py] Python API for ops with struct unpacked manually

This PR does not depend on #4666  in order to be able to merge faster.

Most of the code was already reviewed in #4670


Expose both IPv6 and IPv4 primary addresses

Resolves #30

I will rebase it on the current main state and run the tests.
Tests are wrapped in docker to have access to netbox files. Just run poetry run invoke tests.


Overlay Support

This adds overlay networking support to Interlock meaning you no longer need to publish ports.
This is a significant refactor for the extensions to allow for better use and more reliable event handling.  It also updates to no longer require a volume for the proxy containers as that was unreliable in specific configurations.  This places the configs directly in the container on update to prevent inconsistency with the volumes.
Closes #88

There is some issue with Travis that isn't using the Godep properly.  Tests pass locally.  A test image is also available at ehazlett/interlock:overlay.


Add fighter command

added a command to spawn fighters in the players hanger.

Barrenn, your script has contraband: it duplicates parts of another script. Please either submit this as completely new command (require the file with any tables you need), or modify the current one (inventory.lua).
The many indentation levels can easily be avoided by what programmers call »fail early, fail fast«. That is, check for undesired states first to return early. For example:
-if ship:hasComponent(ComponentType.Hangar) then
-	if … then
-		stuff;
-	end
-end
+if not ship:hasComponent(ComponentType.Hangar) then
+	return false, err
+end
+if not … then
+	return false, err
+end
+
+stuff;
+return true, ""

I have rebased it to new-commands. Only fixed readme file conflicts. Some of the functions and variables are not needed at this point and can be obtained from scripts/lib/cmd/, where our common files are located.
Same goes for available which was moved into completely separate command /list.

LGTM, would squash and merge.

@Barrenn thanks for your work!


🐞 Bug: Getting started dropdown icon is not working

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Getting started dropdown icon is not working(i.e. it is changing direction of arrow on dropdown)

Steps To Reproduce

Go to main page
click on Getting started button
its dropdown icon is not working(i.e. it is changing direction of arrow on dropdown)

Expected Behavior
IN DROP DOWN IT SHOULD POINT DOWNWARDDS LIKE THIS:

Screenshots
No response
Device Information [optional]
- OS:
- Browser:
- version:

Additional context
No response
Are you working on this issue?
Yes

@priyanshagra go ahead !

@tanmoysrt kindly check

sir please review my pr(https://github.com/priyanshagra/CircuitVerse/commit/f97fab21c55d91ddc1ebce139439c95f338f9fe8) of this issue i created


Add overloads to try and simplify resolveDeps workflows

Still a bit awkward, but less than before.
Semantics should be exactly the same, just moved a bunch of boilerplate into helper overloads

Sure, I think resolveLocalDep sounds great.

TBH looking at all the callsites, resolving to "this module" by default definitely seems like the right thing to do. I do think that for the bulk of users, resolving "this module's" dependencies is what they want. And it's true that occasionally people will need to aggregate across multiple modules and feed it through the same context, but that seems like a pretty advanced use case that doesn't happen that often

I have some ideas of how to make this API even better, will update the PR

@lefou took another pass at it. I think the code looks a lot nicer now; no more confusing nested T.task things, and it looks like a normal method call chain.
One downside is the normal method call now needs to throw an exception if it fails, but that's what normal method calls do, and it shouldn't be too hard to wire up try/throw/catch to report a proper error

One downside is the normal method call now needs to throw an exception if it fails, but that's what normal method calls do, and it shouldn't be too hard to wire up try/throw/catch to report a proper error

Couldn't we provide a version returning a Task, so we just add another () and get error handling for free.

One downside is the normal method call now needs to throw an exception if it fails, but that's what normal method calls do, and it shouldn't be too hard to wire up try/throw/catch to report a proper error

Couldn't we provide a version returning a Task, so we just add another () and get error handling for free?

Nevermind. The issue is that the input-parameter(s) are dynamic.

I had some idea in mind for quite a while, that we split the coursier resolution into two steps, resolution and fetching. Lots of meta-operations like finding updates, inspecting licenses, ... only require the resolution without the fetched artifacts. The resolution could also be locked and persisted for later reuse in a more reproducible setup. Since this affects resolveDeps as well, we might think it through as s whole for the next breaking Mill. We lived so long with resolveDeps, it's not time critical to change it now.

Making defaultResolver().resolveDeps(...)() return a task that you call with additional () doesn't really work, because then you cannot pass stuff to the ... that comes from within the enclosing T{...} block. To make that work, we'd need to make resolveDeps itself take a T.task{...}, then we're back to where we started in messiness.
This is pretty fundamental to Mill's multi-stage evaluation model here. defaultResolver().resolveDeps(...) is actually less flexible than the old resolveDeps(T.task{...})(), because we are moving the decision of what gets passed to resolveDeps from resolve-time to evaluation-time. In theory, with the old API you could make decisions affecting the task graph structure based on resolve-time values, while the new API the task graph structure is fixed.
Given all the callsites within the Mill codebase work just fine with the more restricted callsites, I think it's probably the right move. I'd like to get it in sooner rather than waiting for a complete re-think of the API, since it's a backwards compatible addition that fixes a pretty warty part of the Mill experience, and it doesn't block us from making more drastic cleanups later

Updated a bunch of the examples to take advantage of this, which makes them a lot simpler


Ability to add suggestion select listener

I would like to implement a feature which fills other textfields when the user picks a suggestion. Currently i am not able to do that with this addon. Ability to add suggestion select listener would be great.

I agree with tistvan. It's a must have.

In fact, I'm working on it right now ;-)

https://blog.oio.de/2015/01/17/write-simple-auto-complete-combobox-vaadin/ may be helpful ;-)

This add-on is designed to be a TextField with a String as value.
https://github.com/maxschuster/Vaadin-AutocompleteTextField/issues/12#issuecomment-277391947
I'm adding a select event that will get triggered when the user actively selects a suggestion by clicking or pressing enter. The event won't get triggered if the spells out the word without selecting a suggestion.

Is it possible to wrap in SelectionEvent the actual selected Object (in common case it's an Object with an overriden toString() method)?

The event won't get triggered if the spells out the word without selecting a suggestion.

Perfect!

I've updated the online demo with an "Add SelectListener" checkbox. If you enable it, you should see a notification in the lower right corner of the screen, when you select a suggestion.
You can find the new source code in the onselect branch.

Great job!
Are you going to back port this enhancement to versions 1 and 2? Do you need help with that? We are currently using version 1 and will probably upgrade to 2 if it's compliant with vaadin 7.9. Version 3 requires vaadin 8 and that's not an option for us so far.

I'm not sure when I'll have the time to do it. If created issue #20 for the back port.

Updated version 3.0-alpha-2 is available in the Vaadin Directory


[Enum Request] Table 21 "Cust. Ledger Entry" - Field 7000001 "Document Situation"

Hello,
we need to be able to extend the options of field 7000001 - "Document Situation" in table 21 - "Cust. Ledger Entry".
Could you please change it into an enum?
Thanks

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: .


Suggestion: Add glossary.md to tips folder

Add glossary.md to tips folder
After I watched this Video by Fireship.io, I thought that it is a good idea for my first OpenSource contribution.
A markdown file that contains a list of Programing terms with explanation and code examples if needed, and some additional resources to learn more about it.
Example :
Anonymous Function
An anonymous function is a function that was declared without any named identifier to refer to it.
Code Examples
window.addEventListener("click", function () {
    // This is An Anonymous Function
})

Additional Resources
https://en.wikipedia.org/wiki/Anonymous_function

Hi @ayoubmehd 😃, this is a pretty good idea! I think making our glossary and adding it to the tips folder is fine, but we could go beyond that and help a project that already has this type of glossary in place.
I found this project: https://github.com/do-community/developer-glossary, what do you say we contribute to it with your example "Anonymous Function" and perhaps others like asynchronous, etc 😃 ? Then we would add a link to this glossary on the tips command of EddieBot 👍
PS: I'm sure there are other projects with this type of glossary, the one I mentioned is just an option

Hi @BOLT04, Yeah I think that's better than starting from scratch 👍.

A very good idea, @ayoubmehd , and thanks, @BOLT04  for the suggestion to contribute to an ongoing repo.

Great team work 💪 , thank you all 👍

Hi @ayoubmehd 😃, were you able to open a PR for that project's glossary? If so can we close this issue?

I oppened a PR #968, Thank you @BOLT04

I was wondering since we have this on EddieBot, and were discussing about this, maybe we can have this repo as the place to keep tips and fetch the data from here to EddieBot

@Vyvy-vi I think that's a good idea

I was wondering since we have this on EddieBot, and were discussing about this, maybe we can have this repo as the place to keep tips and fetch the data from here to EddieBot

that is a good idea 👍, I believe we discussed this somewhere earlier but I couldn't find any issue mentioning it.
If you want, let's open an issue on EddieBot to discuss the development of that feature @Vyvy-vi 😃.
However, I think this issue could be developed first: https://github.com/EddieHubCommunity/EddieBot/issues/418. So that we could leverage a function to get the list of tips in the support repo, what do you think?

Having it in this repo would mean we would not need to deploy the bot for the latest updates 👍


unsupported batteryCareMode

My ID.4 was updated from 2.4 to 3.1 today, and vwsfriend seems to have trouble getting readings:
2023-06-26T18:37:58+0000:INFO:vwsfriend_base:Updating data from WeConnect
2023-06-26T18:38:02+0000:WARNING:addressable:/vehicles/<MY VIN HERE>/domains/charging/chargingCareSettings/batteryCareMode: An unsupported batteryCareMode: unsupported was provided, known values are [activated, deactivated, unknown] please report this as a bug

The SOC is not recorded in the database anymore since (edit: but the VW app is also complaining I need to agree with privacy settings in the car).
For what's worth, I'm running vwsfriend on Kubernetes using the official OCI images, and this is version 0.23.10.

Hmm, I had several other issues with the VW app, and re-paired the car to my account. After that, all my issues were gone, including this one. The error provided may be a little vague regarding to the actual cause, but that's how stuff works sometimes. Closing the issue, keep up the good work!


[Technical] Fix technical issues found when exploring the current state of the extension

Removed direct interface dependencies between services
Properly host the service using IServiceContainer + add tests (IServiceContainer mainly to test the VsSessionHost itself).
Suppress critical issues reported in SQ all of which are false posititves.
Fix non-critical issues found along the way
Fix a few test holes which I noticed when I updated the tests.

Note: Not going to push until we ship 2.1

LGTM


Undocumented behaviour in prange

Reporting a bug

[X] I have tried using the latest released version of Numba (most recent is
visible in the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).
[X] I have included a self contained code sample to reproduce the problem.
i.e. it's possible to run as 'python bug.py'.

There is some undocumented behaviour in the prange statement. Consider the following code below, which is a typical hot loop for parallelisation. When the result of a computation are reduced to a slice as below,
@numba.njit(parallel=True)
def hot_loop():

    n=10000

    result = np.zeros((n*100), np.float32)
    for i in numba.prange(n):

        # Run computations
        res = np.random.rand(100,100) @ np.random.rand(100)

        # Store result
        result[i*100:(i+1)*100] += res

    return result

it is significantly slower than an allocated array,
@numba.njit(parallel=True)
def hot_loop():

    n=10000

    result = np.zeros((n*100), np.float32)
    for i in numba.prange(n):

        # Run computations
        res = np.random.rand(100,100) @ np.random.rand(100)

        # Store allocated array
        result[i*100:(i+1)*100] += np.random.rand(100)

    return result

Results for my Intel I7 CPU are 74ms for the former, and 974 microseconds for the latter. There is no mention of this anywhere in the docs, or on other issues, but significantly impacts what you can really use prange for? The issue may be that prange is not fully supported to reduce to slices in this way, however, that doesn't seem to matter for the latter case - which is confusing and non-intuitive behaviour.

Just a thought, is it because the compiler knows the size of the allocated array beforehand? In which case iterating through the result of a computation and reducing each component to an array should work, but I can report that this also doesn't help.

Another weirder behavouir - the hot method's writing time in prange depends on the type of computation! Not just on the shape of the array it's writing, bizarre.
For example
@numba.njit(parallel=True)
def hot_loop():

    n=10000

    result = np.zeros((n*100), np.float32)
    for i in numba.prange(n):

        # Run computations
#         res = np.random.rand(100,100) @ np.random.rand(100)
        res = np.random.rand(100) @ np.random.rand(100)
        # Store result
#         result[i*100:(i+1)*100] += np.random.rand(100)
        result[i*100:(i+1)*100] += res

    return result

Also takes around 1ms on my CPU, despite 'res' containing the exact same shape/type array as a matvec in the example above...

Running the following code, creating a temporary array within the loop in which to store the result of the computation, results in the same slow behaviour as the reduction from a computation above,
@numba.njit(parallel=True)
def hot_loop(result):

    n=10000

    result = np.zeros((n*100), np.float32)

    for i in numba.prange(n):

        # Run computations
        res = np.random.rand(100,100) @ np.random.rand(100)
        res2 = np.empty(100)
        tmp[:] = res[:]
        result[i*100:(i+1)*100] = tmp

This implies that Numba is doing something along these lines for a reduction but hiding it from the user.

Thanks for the report. I think what is happening is as follows...
In this case:
@numba.njit(parallel=True)
def hot_loop():

    n=10000

    result = np.zeros((n*100), np.float32)
    for i in numba.prange(n):

        # Run computations
        res = np.random.rand(100,100) @ np.random.rand(100)

        # Store result
        result[i*100:(i+1)*100] += res

    return result

The variable res is being used after its definition (+=res) and so is "live" and it's expression must be executed for the result to be correct.
Whereas in this case:
@numba.njit(parallel=True)
def hot_loop():

    n=10000

    result = np.zeros((n*100), np.float32)
    for i in numba.prange(n):

        # Run computations
        res = np.random.rand(100,100) @ np.random.rand(100)

        # Store allocated array
        result[i*100:(i+1)*100] += np.random.rand(100)

    return result

The variable res is defined but not used and so is "dead". The result of which is that the compiler removes the statement as it has no effect, the work is therefore not present and so the function runs more quickly.
This process is called Dead Code Elimination and is an optimisation technique employed by the parallel=True compilation target.

Hi @stuartarchibald , thank you for the detailed explanation.
I tried to simplify my real codeexamples, with more complex matvecs (specifically multiple chained uses of the @ operator) where this is definitely the case, I know these are taking place as I see multiple cores being hit while running top even when parallel=False is set for the prange - implying that the underlying Blas matvec is being called in a multithreaded way.
My reasoning here was that there is a massive oversubscription of threads caused by using prange on top of numpy, when one has more than one matvec. Furthermore I wrote my own single threaded matvec function, and saw a decrease in runtime, further validating this hypothesis of thread oversubscription.
My further question for you is, as it's likely that this will be a common usecase for prange (i.e. calling numpy/blas routines which may be multithreaded over all threads invoked by prange) how does a programmer handle for this, without resorting to writing single threaded implementations of blas routines.

Hi @stuartarchibald , thank you for the detailed explanation.

No problem. For future, this sort of discussion is great to have on Numba's forum: https://numba.discourse.group.

I tried to simplify my real codeexamples, with more complex matvecs (specifically multiple chained uses of the @ operator) where this is definitely the case, I know these are taking place as I see multiple cores being hit while running top even when parallel=False is set for the prange - implying that the underlying Blas matvec is being called in a multithreaded way.

Chances are your BLAS is multithreaded. If you use Numba's TBB threading layer and also use MKL for your BLAS they are aware of each other and over subscription shouldn't occur. If you
$ conda install tbb

that should get you TBB and Numba will automatically detect this and use it. Docs on the threading layers are here: https://numba.readthedocs.io/en/stable/user/threading-layer.html.

My reasoning here was that there is a massive oversubscription of threads caused by using prange on top of numpy, when one has more than one matvec. Furthermore I wrote my own single threaded matvec function, and saw a decrease in runtime, further validating this hypothesis of thread oversubscription.
My further question for you is, as it's likely that this will be a common usecase for prange (i.e. calling numpy/blas routines which may be multithreaded over all threads invoked by prange) how does a programmer handle for this, without resorting to writing single threaded implementations of blas routines.

I think the above suggestion should fix it, if it doesn't there are environment variables that can be set to force single threaded BLAS (background info here: https://github.com/numba/numba/issues/6637#issuecomment-760460620). I'd also strongly recommend not writing BLAS routines unless you are doing research in the area simply because the existing BLAS implementations are going to be hard to beat.

Hi, I will be sure to check out the forum in the future. Cheers for the link.
I tried out the env vars suggested in the other issue, and was able to use the numpy routines without over-subscription. Thanks again for helping me out with this!

Hi, I will be sure to check out the forum in the future. Cheers for the link.
I tried out the env vars suggested in the other issue, and was able to use the numpy routines without over-subscription. Thanks again for helping me out with this!

No problem, glad you've got something that is working as expected.

Closing this question as it seems to be resolved. Thanks for using Numba!


fix(curriculum): add lesson to registration form practice project for adding method attribute

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #48002

:sparkles: :tada: AWESOME! :tada: :sparkles:
Hi @RohitPatel1122, thanks for this pull request and for contributing to the code-base for the first time. We are looking forward to more contributions from you in the future.
Cheers & happy contributing!

@Sboonny Accidentally, I removed other reviewers from the PR. Can you please add other reviewers to this, since i don't see option to do that.
Thank you !!

Asking for review doesn't speed the process, sometimes it slow it. Because we move from the oldest to the newest
Even asking me for review wasn't necessary. because I had the pr open, was checking the requested code 🤷

It's a weekend, I think another mod will review it after it as usual 👍

Congrats on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We are happy to accept these changes and look forward to future contributions. 📝


Fix Colormaps

Not specifying a colormap in mode shape plotting results in a white mesh.

Closing as I couldn't reproduce the error with the latest.


[W6.5][W14-B4]Wang Jinyi

LO-JavaFx
changed font
added prompt text for textarea on the top
added a background image
I wonder why after setting the background image to the image file, there is still a white text box on top of the background image :/

Hi Kow,
I don't know how to attach a screenshot to a reply to the review comment,
so I decided to put it here :P
[image: Inline images 1]
And I tried to take a screenshot of my codec, but it doesn't seem as clear
as on github.
So sorry to trouble you to navigate to github if you need to see my codes
to see why this white box is appearing on top of background...
Thanks!
Regards,
Jinyi
On 24 February 2018 at 11:40, Kowshik Sundararajan<EMAIL_ADDRESS>
wrote:

@kowshik-sundararajan commented on this pull request.
Good attempt! 👍 Could you attach a screenshot of the issue? It may be
because you are setting a background image for the text-area but there is
another html element above the text-area.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nus-cs2103-AY1718S2/addressbook-level3/pull/471#pullrequestreview-99100816,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcoWTkyo_rmttFyfaW77LBxt2jcuDmMzks5tX4SogaJpZM4SNvoL
.


Failed to load schema

Hi
You closed the previous issue just to let you know also two of my colleagues installed the plugin and got the same error.
Will be great if you can check it.
Not going to interfere you more
Thanks

Thanks for reaching out again, I see no issue thus the previous issue was closed. But just to be sure I will try to reproduce it on my environment later on and I’ll keep you posted.
Please stay tuned.

@gamlielalon1983 did you guys just try and use an older version? say 1.0.5?

I tried  same issue.
I uninstall the latest one and installed the previous one same
error message.
On Sun, Mar 21, 2021 at 08:41 aml11 @.***> wrote:

@gamlielalon1983 https://github.com/gamlielalon1983 did you guys just
try and use an older version? say 1.0.5?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/RoeiOfri/homebridge-palgate-opener/issues/9#issuecomment-803521714,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMOTIOM47KOG2E3NJQAXNATTEWILDANCNFSM4ZQW6ELA
.

Hi, Following your issue, configuring the plugin wasn't supported hence you got the error "Failed to load schema file".
I've fixed the issue in v1.0.7 which was just published.
Please update and try again and update me for any issues by opening a new issue.
Thanks for your attention to the issue.

Correction: Please upgrade to v1.0.8.


PO007 | Policy Naming Conventions - type indication

It is recommended that the policy name include an indicator of the policy type.

would be nice if the plugin reads a config file that has the convention names marked. We can leave it with some defaults.. but the values needs to be updated, they can update the config file and the rule  checks happen accordingly.

See https://github.com/apigeecs/bundle-linter/issues/77


Wiki not user editable?

Can you explain how I can make it user editable.  I've never worked with Wikis before.

@zacharyvincze https://help.github.com/articles/changing-access-permissions-for-wikis/

thx
On Mon, Oct 3, 2016, 5:32 AM Zachary Vincze<EMAIL_ADDRESS>wrote:

Closed #27 https://github.com/zacharyvincze/OpenForum/issues/27.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zacharyvincze/OpenForum/issues/27#event-810196782,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADvq4dlCO-pKHBSsp4V0nQe2IizBSfa9ks5qwPXCgaJpZM4KLq13
.


[CLOSED] Issue 1042 porting to 3.1, enabling all the tests pass on Linux

 Issue by Wilson-Ding1
Thursday Sep 28, 2017 at 04:25 GMT
Originally opened as https://github.ibm.com/IBM-Swift/GRMustache.swift/issues/7

 Issue by vadimeisenbergibm
Sunday Apr 02, 2017 at 14:00 GMT
Originally opened as https://github.com/IBM-Swift/GRMustache.swift/pull/7

@shmuelk please merge

vadimeisenbergibm included the following code: https://github.com/IBM-Swift/GRMustache.swift/pull/7/commits

 Comment by Wilson-Ding1
Thursday Sep 28, 2017 at 04:25 GMT

 Comment by shmuelk
Monday Apr 03, 2017 at 08:07 GMT

LGTM


[yaml] Revise yaml user label

This is more in-line with how other user labels are generated in Beam. Namely, Beam Notebooks...
https://github.com/apache/beam/blob/e5a9ab6233ec76b0aab17474c684b1fad7fe5082/sdks/python/apache_beam/runners/dataflow/dataflow_runner.py#L340-L348
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

R: @robertwb


update to 3.6.1

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Ok now it passes. Feel free to merge if you find it ok


Web API throws BadHttpRequestException

Yesterday I installed VS 2017 RC which quite messed up my previous environment (.NET Core was generally not working).
Most of the issues could be fixed by uninstalling the new .NET 1.1.0 SDK and reinstalling 1.0.1 but I have one issue remaining with Kestrel.
I have a Service Fabric application (with .NET Core Web API microservices) which ran without an issue previously but now I cannot get my REST API to work again. The request is taking really long and ends and I get this error in the debug console:
Microsoft.AspNetCore.Server.Kestrel: Information: Connection id "0HL0FNU42QORF" bad request data: "Malformed request: MethodIncomplete"

Microsoft.AspNetCore.Server.Kestrel.BadHttpRequestException: Malformed request: MethodIncomplete

Also sometimes I get this error:
Microsoft.AspNetCore.Server.Kestrel: Information: Connection id "0HL0FO6PO8FQ9" bad request data: "The input string contains non-ASCII or null characters."
Microsoft.AspNetCore.Server.Kestrel.BadHttpRequestException: The input string contains non-ASCII or null characters.
   at Microsoft.AspNetCore.Server.Kestrel.Internal.Infrastructure.MemoryPoolIteratorExtensions.GetAsciiString(MemoryPoolIterator start, MemoryPoolIterator end)
   at Microsoft.AspNetCore.Server.Kestrel.Internal.Http.Frame.TakeStartLine(SocketInput input)
   at Microsoft.AspNetCore.Server.Kestrel.Internal.Http.Frame`1.<RequestProcessingAsync>d__2.MoveNext()
Microsoft.AspNetCore.Server.Kestrel: Warning: Connection processing ended abnormally

After a while I got the idea to create a new Service Fabric and later a Web API application by using the VS templates. Both result in the same error.
I have no clue what exactly is causing the error...
Maybe I had .NET Core 1.0.0 and not 1.0.1 installed before and this is causing the issue but I cannot imagine everyone having 1.0.1 installed having this issue. Strangely I couldn't find anyone else having the same issue.

Would suggest you are referencing Kestrel 1.1.0 which has stronger request validation?

I'm referencing 1.0.0 and I didn't change the Kestrel or any other references since the last time it worked.
Even when creating a new Web API project without modifying it I get this issue.
I guess it has to be an issue with my local machine somehow...
But I don't know which applications I could have to reinstall, I already removed VS 2017 RC by now.

@FelschR I've run into the same error with .NET Core WebApi and Kestrel, after installing VS2017 RC yesterday.
Surprisingly, the error only occurs if I call my API via localhost. If I use my external ipaddress (<IP_ADDRESS>) for requests against the API everything's working fine.
I also tried multiple .NET Core SDK as well as .NET Core Runtime versions. But, the error still stays.
My current assumption is that something is modifying the local requests. Also played around with fiddler and saw that request for localhost are always tried with https. The strange thing is that Kestrel got the encrypted message which obviously results in "Malformed request: MethodIncomplete". All request going to <IP_ADDRESS> are http as expected.
For now it's a workaround for me to just use my external ip-address instead of localhost for development and testing. But, I will investigate this as soon as I got some sparetime.

@reicheltp great (or not so much) to hear that I'm not the only one having this issue.
I didn't notice that external IPs work. Sounds like a great workaround for now!
Meanwhile I started to completely remove VS 2015 and all .NET development tools and reinstall it from scratch. I'll update this post once it finishes installing.
The "Repair" option for Visual Studio in "Programs and Features" doesn't fix this issue.

The better logging suggestion HTTPS requests when HTTPS is not enabled is a dupe of #1116. It was fixed in 1.1 by #1122.
#1119 provided a clearer error message ("An SSL/TLS handshake might have been attempted on an HTTP endpoint.") based on the first byte of the request being 0x16, but we decided that wasn't generic. Now our error message is more in line with what you would see from nginx.

Damnit Chrome! @FelschR thanks for the solution!


(PUP-5694) Add per environment static_catalogs setting

Creates a boolean static_catalogs puppet setting defaulting to
false. The setting can be overridden in environment.conf, similar to
how future parser worked.
Currently the static_catalogs setting only modifies the compiler's
message when it compiles a catalog, e.g. "Compiled static catalog". In
the future, it will control whether the compiler inlines file metadata
into the catalog.

:+1: once comment discussion is resolved.

Verified Environments::EnvironmentLoader#clear_all is called via benchmarks/many_environments/benchmarker.rb:
$ bundle exec rake benchmark:many_environments:run
mkdir -p /var/folders/2q/qhdc5tpx4knd4pjr1hdskmjc0000gn/T/many_environments20160121-70259-rzvota
                 user     system      total        real
Run 1        0.000000   0.000000   0.000000 (  0.000442)
Run 2        0.000000   0.000000   0.000000 (  0.000328)
Run 3        0.000000   0.000000   0.000000 (  0.000292)
Run 4        0.000000   0.000000   0.000000 (  0.000316)
Run 5        0.000000   0.000000   0.000000 (  0.000279)
Run 6        0.010000   0.000000   0.010000 (  0.000369)
Run 7        0.000000   0.000000   0.000000 (  0.000290)
Run 8        0.000000   0.000000   0.000000 (  0.000299)
Run 9        0.000000   0.000000   0.000000 (  0.000288)
Run 10       0.000000   0.000000   0.000000 (  0.000288)
> total:     0.010000   0.000000   0.010000 (  0.003191)
> avg:       0.001000   0.000000   0.001000 (  0.000319)

As @hlindberg mentioned it is also possible that it is being called by Puppet Server.

@cprice404 I'm not seeing any references to it in https://github.com/search?l=&p=3&q=user%3Apuppetlabs+clear_all&ref=advsearch&type=Code&utf8=✓ (ignoring pe-puppet fork, crossfader, puppet-win32-ruby*, and some other gem related repos). I'm not sure if that only searches the default branch for each repo, or all branches?

@joshcooper sorry, this was kind of a drive-by on my part.  Here are the relevant places where Puppet Server interacts with Puppet's environment caching stuff:
https://github.com/puppetlabs/puppet-server/blob/0bb8584c938fb057b5bf1f677dec55a2a587b962/src/ruby/puppet-server-lib/puppet/server/environments/cached.rb#L4-L24
https://github.com/puppetlabs/puppet-server/blob/0bb8584c938fb057b5bf1f677dec55a2a587b962/src/ruby/puppet-server-lib/puppet/server/config.rb#L31-L34
So it looks like we aren't calling that particular function.  Figured it wouldn't hurt to highlight that we are interacting with the environment cache stuff in general, though.


Add options to change stream and file log level

Change file log level to INFO
Description
(Please describe what this PR do.)
The log file needs to be written more detail information. So please change the file log level to lower level (INFO).
Type of change
Please insert 'x' one of the type of change.

[ ] Bug fix (non-breaking change which fixes an issue)
[] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[x] Refactoring, Maintenance
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

@dd-jy , LGTM. :-)


Make use of fs.FS (go 1.16 and up) so that source can be loaded from diverse filesystems.

Proposal
I propose adding a Filesystem fs.FS field to Config so that the user can supply their own filesystem.
Background
I use yeagi for my fantasy console, but I also target WASM where there is no default filesystem. Currently I use Eval(string) but it means I lose all the benfits of filesystem separation gained from EvalPath() resulting in huge monoliths of unseparated code.
As I already have and use an http zipfile fs.FS in my fantasy console, being able to set that FS for yaegi allows me to lay out files like a local filesystem (even tho it's dynamically loaded from a zipfile over http) and not have to use Eval(string) anymore.
Workarounds
Eval(string), but I lose all the benefits of file separation that EvalPath() gives.
I have started a PR #1199 to discuss how I see implementation (I was curious so I started impl before creating this feature request).

fyi, this https://github.com/traefik/yaegi/pull/1201 is now merged.

@mpl Hi. 1201 doesn't relate to this particular issue. This issue is for the feature of using fs.FS in the interpreter Config (see #1199 ) rather than being for the stdlib for fs.FS.


Fixed bool config having int value

like DEBUG=1

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

:exclamation: No coverage uploaded for pull request base (main@bde12f4). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1708   +/-   ##
=======================================
  Coverage        ?   51.37%
=======================================
  Files           ?       96
  Lines           ?     3877
  Branches        ?        0
=======================================
  Hits            ?     1992
  Misses          ?     1885
  Partials        ?        0

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


Reduce duplicate naming in windows service

Moves some commonly used strings into immutable variables so that they are easier to update and extract for use with other tools. It should also make extending the service a bit easier.
Also makes indentation in the affected files consistent to 4 spaces across all lines.

As commit history needed changing I'm closing this and superseding with https://github.com/opcm/pcm/pull/135. This PR also addresses other review comments.


I am not able to login via business account

Version 1.8.2

@pradeepnegi88  there is a trick
try to download this version: 1.6
login with your email and password
than download the new version and problem wil be sloved


Implement rough MCA idea for reducing dimensionality of one hot columns

I left it to 6 because I noticed that there is a flaw in the idea I had of tracking the categorical columns. Instead, we should run a search through those columns unless it's an already large amount, and check the unique values of each of those columns to better understand how the dimensions will grow after one hot encoding. As for the components, it is not the same as PCA where you can select the variance, so really the only way to find optimal n for components is to do a grid search comparing scores from train and validation. If this is something we want to pursue, I can add those features in as well. As for now, this is a rough idea and I'll close the issue now or later depending on if we want to add none/some/all of these ideas.

Updated with PCA query issues being fixed


[Fix] Add robustness to solution verification

Motivation
This PR updates block production to correctly address malicious solutions. This fix overwrites the solution target with value calculated during solution verification. So even if a malicious solution is injected by a malicious validator, honest validators will always process them and safely convert them to honest solutions.
CI run is here.
Test Plan
Tests have been added to ensure that malicious solutions are handled correctly.

Closing this PR for now due to unifying the fix in https://github.com/AleoNet/snarkVM/pull/2521.


Apply an Interval/ROI/Mask to filter both volumes and geometries

https://gitter.im/scenerygraphics/SciView?at=5ed17ce14c9b0f060d314505
Vladimír Ulman @xulman 17:21
actually I wanted to crop everything, I'm showing both mixed -- the Volume and some graphics elems (Sphere in this case) and it would not be natural, in my case, to see half of the image (VOlume) while complete "annotation" of it (Spheres)
but it was super painful to position the box, and also suddenly SciView was responding a lot slower,... so I gave up eventually
Kyle I S Harrington @kephale 18:00
oh i see, let's revisit this because i believe we should do this in a more imglib2 way (e.g. applying an interval over a collection of mixed object types)

I think this is probably still a heavy lift, but more objects in sciview now use ImgLib2 interfaces.

This is related to https://github.com/scenerygraphics/sciview/issues/291


Runbook for MachineConfigControllerPausedPoolKubeletCA, initial machine-config-operator setup

This is for MCO-119 and corresponds to the alert added with https://github.com/openshift/machine-config-operator/pull/2802/
This is also the machine-config-operator's first runbook, so this sets up the OWNERS file.
On one hand, I feel like this is too verbose, on the other hand, given how often the situation for this alert has to be explained, I wanted to include enough "why" so the recipient of the alert fully understood the ramifications.

What is the procedure for getting this merged? I left a small change, but is there someone else who should be reviewing this?

while we're here @mburke5678 would you mind taking a quick look as well?

@jkyros I made a few small comments. Feel free to accept or ignore them as you see fit.
cc @kikisdeliveryservice

@mburke5678 thanks for taking a look. I like your wordings better and the translation issues hadn't occurred to me, thank you!

@jkyros I made a few small comments. Feel free to accept or ignore them as you see fit.
cc @kikisdeliveryservice

Thanks for the great review @mburke5678 !! Thanks for updating @jkyros! Teamwork! :)

Errors have been fixed, thanks @sinnykumari for taking a look.

/lgtm

/assign @nautilux

@NautiluX @ravitri Could you PTAL so we can merge this?

@jkyros / @kikisdeliveryservice - Sorry for the delay in reviewing this. This is a great runbook with lot of details :+1:. So far, the references we have had were the following:

KCS
SOP (private)

I think the runbook majorly covers the points mentioned in the given two references as such and we can update these references with the runbook document link ahead too. Overall, LGTM but I will wait for @NautiluX to provide his feedback too. We'll merge it soon after that :+1:


Implementing unique file names

Hi,
How would one go about implementing unique file names.
In carrierwave one can just define
def filename
generate unique  file name
end
However, with carrierwave direct the file is being uploaded directly from the front end.
Any ideas?

I know this issue is over 3 years old, but I'm running into the same problem, the unique GUID is the same for uploads from SPA.  This causes an issue when a user has a file called logo.png and another also called logo.png (a different file, locally /dirA/logo.png and `/dirB/logo.png'

So in my case the reason the GUID in the file path wasn't changing was due to the way I was generating the instance of the uploader class in javascript.  I'm using gon to access rails variable from javascript and was using it to access my uploader.  The uploader was generated when I hit the home controller and that was it.  Every time I accessed the uploader it was the same instance of the uploader class.  I was able to get around this by using gon.watch to create a new instance of the class.

This should be fixed on master


Check refs console command

A console command that allow to check and fix broken or inconsistent links.
It should allow user to specify what does he want to do with broken link:

skip
remove
set value manually

For edges in graph db it should also check consistency of edge. That both vertexes aware of the specific edge. In case if edge is not consistent auto-fixing is possible, command should allow to do that automatically.

Already implemented with repair database console command.


Preset a certain part of the picture before start

Hi Ian,
Openseadragon fascinates me. I presents  fantastic possibilities. But now I am achieving a difficulty and maybe you can help me. I want to process panoramic images in a way that the user can find the picture in a certain way.
The panorama shows, when processed, one part of the whole picture.
To enlarge the picture as much as you want it to  is no problem with the option "defaultZoom". But my problem is, however,  the enlarged picture is central. Is it possible to choose another part of the picture?
One example: When I click on the link to activate the panorama picture, it is  400 % enlarged (defaultZoom: 4). But I only want to show one special part of the picture, f.e. the one that shows the corner down left.
Is it possible - like with defaultZoom - to  preset a certain part of the picture before I start it?
Did I possibly overlook something within the documentary?
Kind regards
Franz

I recommend fitBounds for going to specific locations. If you want to go to a specific location at startup, you can do it like so:
viewer.addHandler('open', function() {
  viewer.viewport.fitBounds(bounds, true);
});

Remember that the bounds that you give (which should be an OpenSeadragon.Rect) need to be in viewport coordinates. You can learn more about that here:
http://openseadragon.github.io/examples/viewport-coordinates/

Hi Ian,
Thank you for your quick response. That seems to be the slolution!
Unfortunately I did not manage to find the definition for the variations. You refer to the part "viewport-coordinates" within the tutorial. There it is described that the variations for the start value are x and y. But unfortunantely I cannot manage how to add "bounds".
Among others I tried the following three possibilities:
version 1:
var bounds = OpenSeadragon.bounds.x(0.1);
var bounds=OpenSeadragon.bounds.y(0.1);
version 2
var bounds=x(0.1);
var bounds=y(0.1);
version 3
var bounds=(0.1, 0.1);
and then I added your sentencs:
viewer.addHandler('open', function() {
viewer.viewport.fitBounds(bounds, true);
});

Unfortunately without success. I have the feeling I have got this completely wrong.
Can you help me again?
Best regards
Franz

Whats this?
I don´t want close this .....

Apologies for the confusion. I meant the bounds variable should be a OpenSeadragon.Rect, for instance like so:
var bounds = new OpenSeadragon.Rect(0, 0, 0.1, 0.1);

Those coordinates will give you the upper left tenth of the image, assuming the image is 1 viewport coordinate wide (which is the default).

Hi Ian!
Thanks for the fast answer.
But it doesn`t work.
Did you mean to generate a retangle?

I thought:
For the point to start the panorama I need only a Ccoordininate or landmark, like (0.0, 0.5) for  startpoint.
As an exampel I send two screenshots:
No1 ist the usual startposition.
Photo No2. shows the situation which I like.
Best regards
Franz

could you help further

Just a point isn't really enough, because it doesn't set the zoom for you. You could pan to a point and also set the zoom separately, but the advantage to fitBounds is that it all happens together in one call.
As for how to figure out what the rectangles should be, the easiest thing is to open up your page in a web browser and navigate to the point of interest, then go into the browser's console and type (assuming your viewer is called viewer and is accessible globally):
viewer.viewport.getBounds();

Grab the numbers from the result, and then you can use those for your fitBounds.

Hi Ian!
Thank you very much,
now it is working!
Yeah!!!!
Here is the pre release for my Projekt:
http://www.tomaschowski.com/Panorama/Megazoom/TEST - Paris2.html
It is a very early version without anything special.
I want to create elements of storytelling in panorama. It will be a mixture between a linear story and an intuitive discover.
If you are Interested, I can give you  the link for my project if it finish. Maybe for your showcase, just in case you are interested.
Best regards,
Franz

Excellent, and yes, please do let me know when it's ready to share! :)

Hi Ian,
After a long break I finally can manage to develop the panoramas. Id like to add another panorama to the first. For both of them I have determined an image section for the start. Your script functions very well!
However, I have another problem:
The start position of the second  panorama functions. At the first panorama the determined start positions are ignored.
I have added the following:
screenshot 1: only one panorama

screenshot 2: both panoramas
and here is the code:

<div id="Agde" class="Panorama"> </div>

Franz Tomaschowski | Dorfen/Germany |
Impressum

// Panorama Paris    -->

// Panorama Agde    -->

The adress of the Site is: www.tomaschowski.com
Please, could you hlp me, once more
Best regards
ftomasch

Great to see it coming along!
BTW, you can make your code easier to read here on GitHub by wrapping it in ```.  I've taken the liberty of doing that for your comment above.
I think your big issue is that both of your viewers are named viewer, so when the open event fires you're telling only the second viewer where to go. Just give the second viewer a different variable name and update the code as appropriate and that should sort it out.

Every time the word viewer appears in the "Panorama Agde" section, change it to viewer2 (with no space). That should fix it.


in windows, it is not working

in windows, it is not working, please help

What happens when you try (does it throw an error, nothing, etc)?  Can you run the Debug command and post a gist of that.


After Windows Update PowerToys Run stopped working.

Microsoft PowerToys version
0.75.1
Installation method
GitHub, PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
After Windows Update PowerToys Run stopped working.
Even if I click PowerToys tray icon and select it in the menu, PowerToys Run not running.
My update history (Latest updates):
2023-11 Cumulative Update for Windows 10 Version 22H2 for x64-based Systems (KB5032189)
2023-11 Cumulative Update for .NET Framework 3.5, 4.8 and 4.8.1 for Windows 10 Version 22H2 for x64 (KB5032339)
Security Intelligence Update for Microsoft Defender Antivirus - KB2267602 (Version 1.401.622.0) - Current Channel (Broad)
Windows Malicious Software Removal Tool x64 - v5.119 (KB890830)
Bug report zip file:
PowerToysReport_2023-11-15-13-10-17.zip

✔ Expected Behavior
PowerToys Run to appear.
❌ Actual Behavior
PowerToys Run window not appearing.
Other Software
No response

There is a crash in the event viewer log with code c06d007e.
I am on the same version of Windows 10 with no crashes.
Are you still experiencing the problem after a reboot?
/needinfo

Thank you for your quick reply.
Correlation is not causation.
It stopped working after Windows Update. I updated my own personal PC (PowerToys Run crashes on my work PC) and it's working as expected.
It still crashes after reboot. I reinstall PowerToys (with both user and system-wide installer options) multiple times, but it keeps crashing. I could share Windows Event Log if you want.

Event viewer log is included in the bug report but doesn't provide detailed information.
Can you navigate to %LOCALAPPDATA%\CrashDumps folder and check if there is a dump named PowerToys.PowerLauncher.exe.*.dmp, It could help.
/needinfo


Return a non-empty response from setResourceCacheMaxBytes

The empty reply added to this channel in https://github.com/flutter/engine/pull/10434 ends up throwing this exception: https://github.com/flutter/flutter/blob/93c32c4ff667d82136dcc1f8ecef1b0ab993a047/packages/flutter/lib/src/services/platform_channel.dart#L319
Sorry, should have thought of this earlier. Can we add a non-empty reply to avoid the exception? Empty string instead of null?
/cc @dnfield

Sorry, looks like this triggered a new issue:
08-29 09:28:30.088 10314 10337 E flutter : [ERROR:flutter/lib/ui/ui_dart_state.cc(148)] Unhandled Exception: FormatException: Unexpected character (at character 1)
08-29 09:28:30.088 10314 10337 E flutter :
08-29 09:28:30.088 10314 10337 E flutter : ^
08-29 09:28:30.088 10314 10337 E flutter :
08-29 09:28:30.088 10314 10337 E flutter : #0      _ChunkedJsonParser.fail (dart:convert-patch/convert_patch.dart:1392:5)
08-29 09:28:30.088 10314 10337 E flutter : #1      _ChunkedJsonParser.parseNumber (dart:convert-patch/convert_patch.dart:1259:9)
08-29 09:28:30.088 10314 10337 E flutter : #2      _ChunkedJsonParser.parse (dart:convert-patch/convert_patch.dart:924:22)
08-29 09:28:30.088 10314 10337 E flutter : #3      _parseJson (dart:convert-patch/convert_patch.dart:29:10)
08-29 09:28:30.088 10314 10337 E flutter : #4      JsonDecoder.convert (dart:convert/json.dart:493:36)
08-29 09:28:30.088 10314 10337 E flutter : #5      JsonCodec.decode (dart:convert/json.dart:151:41)
08-29 09:28:30.088 10314 10337 E flutter : #6      JSONMessageCodec.decodeMessage (package:flutter/src/services/message_codecs.dart:93:17)
08-29 09:28:30.088 10314 10337 E flutter : #7      JSONMethodCodec.decodeEnvelope (package:flutter/src/services/message_codecs.dart:141:54)
08-29 09:28:30.088 10314 10337 E flutter : #8      MethodChannel.invokeMethod (package:flutter/src/services/platform_channel.dart:321:33)
08-29 09:28:30.088 10314 10337 E flutter : <asynchronous suspension>

The line before this in the stack trace was await SystemChannels.skia.invokeMethod('Skia.setResourceCacheMaxBytes', 1024 * 1024 * 512);
So it looks like the response must be valid JSON.

Ahh right because it's a json codec channel.
Ok.  This is an easy fix.


Metrics graph calculation does not consider more than one metric

To repro just add a 2nd metric -> the graph for the 2nd metric will look the same as the graph for the first metric.
My fix is only partial for the non-mysql query in Metric.php:
$query = DB::select("select {$queryType} as aggregate FROM metrics JOIN metric_points ON metric_points.metric_id = metrics.id WHERE metric_points.metric_id = {$this->id} and to_char(metric_points.created_at, 'YYYYMMDDHH24') = :timestamp GROUP BY to_char(metric_points.created_at, 'H')", [                 'timestamp' => $dateTime->sub(new DateInterval('PT'.$hour.'H'))->format('YmdH'), ]);
i.e. add an additional constraint metric_points.metric_id = {$this->id} - not sure how to do this in the mysql query construct

Heh, bit of an oversight on my part.


General question for problem set 2 (does it have to be the same research question?)

The research question respectively the hypothesis I formulated for the first problem set do not really work for the second one, since the correlation is not that strong. Can I change the research question, or would that go against the main idea you had in mind for the problem set?

I personally think that if the correlation is not that strong, that is also a statement in regards to your question. I would say you can show that, and still go into another hypothesis if you can do it with the same dataset.
Just my opinion though because there was no answer for a long time - and fyi, my dataset didn't have more than 3 variables that were really usable, so for the exercises with control variables I had to find a new dataset, new hypothesis, tidy the dataset and then calculate the fits. I assume that changes are fine as long as you document your though process..

thank you so much!


PyPi

Should I see a version of imjoy-jupyterlab-extension on PyPi (for JupyterLab 4/itkwidgets[lab] compatible installation)?
it should be version 0.1.21 but I do not see it on Pypi (ref https://pypi.org/project/imjoy-jupyterlab-extension/#history).
thanks

Hi @scrgiorgio Thanks for reaching out, I ended up remove the new release since there is a bug after I publishing it. The automatic upgrade script from jupyterlab didn't work well, and the extension does not work as expected. I need some more time to get back to the extension, super busy right now, sorry.

Thanks for the clarification

@scrgiorgio
FYI: Jupyterlab 4 is supported with the new release: https://pypi.org/project/imjoy-jupyterlab-extension/


Merge Changes Detection

closes #30

Codecov Report

Merging #32 (fe4ef86) into main (a26ed6f) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #32   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            6         6
  Lines          111       142   +31
  Branches        12        16    +4
=========================================
+ Hits           111       142   +31

Impacted Files
Coverage Δ

src/label.ts
100.00% <100.00%> (ø)

src/pulls.ts
100.00% <100.00%> (ø)

src/queries.ts
100.00% <100.00%> (ø)

src/run.ts
100.00% <100.00%> (ø)

src/util.ts
100.00% <100.00%> (ø)

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 a26ed6f...fe4ef86. Read the comment docs.


[KYUUBI #5472][AUTHZ][FOLLOWUP] Check permanent view also need support merge projection

Why are the changes needed?
To close #5472 .
Check permanent view also need support merge projection
For case SELECT count(id) FROM $db1.$view1 WHERE id > 10 we only need to check id, before this pr, we need to check id, scope.
How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Was this patch authored or co-authored using generative AI tooling?

ping @yaooqinn

Codecov Report

Merging #5542 (cb9322c) into master (ceb8453) will increase coverage by 0.03%.
Report is 2 commits behind head on master.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #5542      +/-   ##
============================================
+ Coverage     61.32%   61.35%   +0.03%
  Complexity       23       23
============================================
  Files           598      598
  Lines         34254    34259       +5
  Branches       4488     4489       +1
============================================
+ Hits          21006    21020      +14
+ Misses        11122    11113       -9
  Partials       2126     2126

Files
Coverage Δ

.../kyuubi/plugin/spark/authz/PrivilegesBuilder.scala
91.73% <100.00%> (+0.21%)
:arrow_up:

...rk/authz/ranger/RuleApplyPermanentViewMarker.scala
94.11% <100.00%> (+0.36%)
:arrow_up:

.../plugin/spark/authz/util/PermanentViewMarker.scala
85.71% <100.00%> (+2.38%)
:arrow_up:

... and 8 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Merged to master, @AngersZhuuuu could you please open a PR for branch-1.8? this are conflicts


chore: fix function name in comment

fix function name in comment

Could you sign off on the DCO https://github.com/prometheus/prometheus/blob/eb3b349024de77bb57499fddf3a7f55b449f95cf/CONTRIBUTING.md#contributing?


Revert "Update readme with vscode extension"

@Bigrig7272gg please stop creating PRs that undo other people's PRs. They will not be merged, especially if you don't include an explanation.


add threshold Schnorr examples

This PR adds a Motoko and Rust example for the use of management canister's threshold Schnorr API: signing and fetching the public key by a third canister. it also shows how such signatures can be verified.
Since the management canister API is not yet implemented for threshold Schnorr, the example canisters use a mock canister that mocks the behavior of management canister for threshold Schnorr in an INSECURE way. This will be removed as soon as the implementation is finished.

hello @altkdf. Please also update CODEOWNERS to reflect the owner of the example. Additionally, please provide a workflow that runs an end to end test. See .github/workflows directory for examples. thanks!

hello @altkdf. Please also update CODEOWNERS to reflect the owner of the example. Additionally, please provide a workflow that runs an end to end test. See .github/workflows directory for examples. thanks!

Thanks for pointing out @dfx-json, didn't notice it existed. Done.

There's one error remaining, which I encounter myself time to time as well and that is in the linking of the example. I believe this is due to the cdylib crate type and I'm not sure how to solve this. Any suggestions?
unknown-linux-gnu/lib/rustlib/x86_64-unknown-linux-gnu/lib" "-o" "/home/runner/work/examples/examples/rust/threshold-schnorr/target/debug/deps/libschnorr_example_rust.so" "-Wl,--gc-sections" "-shared" "-Wl,-z,relro,-z,now" "-nodefaultlibs"
  = note: /usr/bin/ld:/tmp/rustchzKclQ/list:4: syntax error in VERSION script
          collect2: error: ld returned 1 exit status

   Compiling pocket-ic v3.1.0
error: could not compile `schnorr_example_rust` (lib) due to 1 previous error
warning: build failed, waiting for other jobs to finish...

There's also one related question on stackexchange, but it doesn't seem to have a solution.