Visual Studio Code blank terminal, can't enter/type anything [Windows 10]

I am facing this issue where I was working fine with Visual Studio Code, installed some extensions, and then my terminal is frozen (blank screen). I am not able to type anything inside termial.
The image describes how my terminal looks:-
https://imgur.com/a/GNTewu7
I even tried uninstalling the VS Code (as well as all the extensions), updated windows, and then reinstalled the fresh copy of VS code, but the problem is still there. I tried some suggested methods in other threads to make the terminal working again, but the terminal problem does not go away. Some of the solutions that I already tried are:-

Re-installing VS Code, updating, and restarting VS Code as well as Windows.
Changing the default Command Line Shell from CMD to Powershell and git bash and back again
Adding a new Command Line Shell
Resizing the terminal window
Zoom In/Out of the terminal
checked the "Use legacy console" option in Powershell options
changed Change inheritEnv to true as "terminal.integrated.inheritEnv": true
Inside the terminal window, Pressed Ctrl + c and hit enter
changed the explorer kind in VS Code form inherited to external "terminal.explorerKind": "external"
changing the renderer type from auto to DOM, canvas forth and back "terminal.integrated.rendererType": "canvas"

- VSCode Version:
1.44.2
ff915844119ce9485abfe8aa9076ec76b5300ddd
x64

-VSCode Info:
Version: 1.44.2 (user setup)
Commit: ff915844119ce9485abfe8aa9076ec76b5300ddd
Date: 2020-04-16T16:36:23.138Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 6.2.9200

- OS Version:
Windows 10, version 1909
**code --log trace:**  https://justpaste.it/673hy
code  --status
Version:          Code 1.44.2 (ff915844119ce9485abfe8aa9076ec76b5300ddd, 2020-04-16T16:36:23.138Z)
OS Version:       Windows_NT x64 6.2.9200
CPUs:             Intel(R) Core(TM) i5-4200U CPU @ 1.60GHz (4 x 2295)
Memory (System):  7.93GB (2.87GB free)
VM:               0%
Screen Reader:    no
Process Argv:     --log trace
GPU Status:       2d_canvas:                     enabled
                  flash_3d:                      enabled
                  flash_stage3d:                 enabled
                  flash_stage3d_baseline:        enabled
                  gpu_compositing:               enabled
                  multiple_raster_threads:       enabled_on
                  oop_rasterization:             disabled_off
                  protected_video_decode:        unavailable_off
                  rasterization:                 enabled
                  skia_renderer:                 disabled_off_ok
                  video_decode:                  enabled
                  viz_display_compositor:        enabled_on
                  viz_hit_test_surface_layer:    disabled_off_ok
                  webgl:                         enabled
                  webgl2:                        enabled

CPU %   Mem MB     PID  Process
    0      105   13036  code main
    0       84    9172     shared-process
    0       38    9916     utility
    0      213   11576     window (Settings - adv_exe_6 - Visual Studio Code)
    0       22    3760       winpty-process
    0        7    7652         console-window-host (Windows internal process)
    0        7    8760         C:\WINDOWS\System32\cmd.exe
    0       20    6228       winpty-process
    0        7     232         console-window-host (Windows internal process)
    0       44   10196         C:\WINDOWS\System32\WindowsPowerShell\v1.0\powershell.exe
    0       78    7988       extensionHost
    0       64    2840         electron_node tsserver.js
    0       35   10444           electron_node typingsInstaller.js typesMap.js
    0       60   13140         "C:\Users\ITKHAN\AppData\Local\Programs\Microsoft VS Code\Code.exe" "c:\Users\ITKHAN\AppD
ata\Local\Programs\Microsoft VS Code\resources\app\extensions\json-language-features\server\dist\jsonServerMain" --node-
ipc --clientProcessId=7988
    0       42   14272         electron_node tsserver.js
    0       22    9492       winpty-process
    0       68    5716         C:\WINDOWS\System32\WindowsPowerShell\v1.0\powershell.exe
    0        7    5724         console-window-host (Windows internal process)
    0       11   12124       watcherService
    0        7   14004         console-window-host (Windows internal process)
    0       22   13552       winpty-process
    0        7   10100         console-window-host (Windows internal process)
    0        7   11960         C:\WINDOWS\System32\cmd.exe
    0      143   13760     gpu-process
    0       15   14188     electron-crash-reporter

Workspace Stats:
|  Window (Settings - adv_exe_6 - Visual Studio Code)
|    Folder (adv_exe_6): 8 files
|      File types: js(4) stackdump(1) html(1) css(1)
|      Conf files:

Steps to Reproduce:

Install VS Code
Open the integrated terminal
Start trying to type commands

This issue still occur when all extensions are disabled

Here's how  I have solved this issue.
I unchecked the compatibility mode in VS Code properties where it says Run this program in compatibility mode for:.
But this made the terminal to open in an external window instead of being integrated in VS Code whenever I open an new terminal inside VS Code. So I also disabled the Conpty for windows terminal inside VS code settings > Terminal > Inegrated:Windows Enable Conpty
"terminal.integrated.windowsEnableConpty": false
The interesting point to note here is that this solution didn't work for me when I had my VS code with all the extensions when this problem arise for the first time. After updating the windows, reinstalling fresh copy of VS code without any extensions, it did the trick for me.
So the issue was probably with the outdated windows, or some extensions that I installed.


Implement TCP/IP in Swift

Current coverage is 71.31% (diff: 70.00%)

Merging #153 into master will decrease coverage by 0.27%

@@             master       #153   diff @@
==========================================
  Files           134        134
  Lines          6830       7151   +321
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           4890       5100   +210
- Misses         1940       2051   +111
  Partials          0          0

Powered by Codecov. Last update d0920e6...8455971

@paulofaria I've merged this up with master but a few performance tests shows a 3k req/s regression on my machine.
It's hard to profile properly but my guess is we're just doing too much in our polling loops.

The issue is probably the throws overhead and the fact that we have work on three different modules now.. TCP, IP, and POSIX.


It is required to avoid ambiguity.


A11y addon stopped working

http://storybooks-official.netlify.com/?selectedKind=UI|MenuItem&selectedStory=default&full=0&addons=1&stories=1&panelRight=0&addonPanel=%40storybook%2Faddon-a11y%2Fpanel
Expected: a warning about black text on blacky background
Actual: 0 violations 0 passes

Axe assumes default styling (white background on body).
however the background of the iframe is transparent, and below you see the color of the manager UI. (which is dark)
Axe cannot possibly know this, I think.
Is A11Y addon even enabled for that story?

I do not think this is a bug.

why the tab is present

All tabs are always present, we do not have a system to only display tabs that are relevant.
We should add such a thing, sure.

Should we maybe make iframe background white by default?

It's light on the light theme, it's dark in the dark theme. User have the ability to theme or add a preview-head.html or load css some other way to give the body inside the iframe a colour. Also they can use the background-addons.
The background addon works by adding the colour to the background of the preview (Component in the manager) So the background inside the iframe should be transparent.
I do not think we should force a white background by default.

I think that the fact that our "official examples" site contains an unreadable text on the start screen is a major issue itself

Fixed it:
https://deploy-preview-4086--storybooks-official.netlify.com/?path=/components/Components-Heading-types

The transparent background also breaks full screen mode: http://storybooks-official.netlify.com/full=1
WDYT on this point?

I think we should enable it for our own components and make sure that everything passes, so that we would be sure that storybook UI is accessible enough, at least with builtin themes.

Fixed it:
https://deploy-preview-4086--storybooks-official.netlify.com/?path=/components/Components-Heading-types

=( https://deploy-preview-4086--storybooks-official.netlify.com/iframe.html?path=/components/Components-Heading-types

Fullscreen going white is a known issue I want to fix.

I think we should enable it for our own components and make sure that everything passes, so that we would be sure that storybook UI is accessible enough, at least with built-in themes.
WDYT on this point?

Hmm, I need to think about it.

Closing this as strictly speaking the addon works, I'll improve the official example in an attempt to make it as accessible as possible.


Readme/setup for self hosting

The application looks amazing! Will there be a setup guide for self hosting the front and backend?

Yep, I intend on writing up something on how to set that up for people who do not want to use the default public server. I will make sure to mention this issue when I have that ready.

Thanks!

As discussed in Discord, I'll follow up with a Docker image and docker-compose file.

Added a Docker image, docker-compose file, and instructions for self-hosting with Docker as part of https://github.com/christoabrown/group-ironmen/pull/2.


flannel: cluster_name is undefined

I am trying to install Kubernetes, but I get stopped at the error below. I am trying to run this on Cisco OpenStack.
TASK: [flannel | configure flannel in etcd] ***********************************
fatal: [mantl-worker-003 -> mantl-control-03] => One or more undefined variables: 'cluster_name' is undefined
FATAL: all hosts have already failed -- aborting

So it seems that the variables were not being read from the kubernetes_vars.yml file. I had to set the variables under the

hosts: all
vars:
section in the kubernetes.yml file.


Add release tags to docker images published to docker hub

Can you please add tags to each release released to docker hub? So that we can control what version of DocsGPT we have deployed.

Hey, I would like to work on this issue, can you please assign this task to me?

@dartpain ,
Could you please assign this issue to me?


rewrite of _utils._check_blocks _utils._check_same_gradients,

In the spirit of #119  I rewrote them in a more stringent way and allclose now use them to do all the checks for the metadata . This implies that all the test of allclose now are on these functions. I think everything has a test (maybe not all the gradient possibilities).
What do you think?
I, of course, can create the equal functions, being the same as allclose except for the more stringent numerical checks, but I do not know if it really usefull.

_check_maps and _check_blocks seem like such fundamental functions that form the basis of most of our equistore.operations modules that I feel like they deserve their own dedicated and self-contained unit tests. Of course there would be some repetition in the unit tests of the public functions that use them, like add and allclose etc.
Is it just a rule we follow that we don't test private functions at all? That all testing of them come from the public functions that wrap them?

did the rebase added the equal function

with that we surpassed 100 tests 🎉🎉


Installing error

gzip and tar raise error when installing:
$ curl --proto '=https' --tlsv1.2 -sSf https://imdl.io/install.sh | bash -s -- --to ~/.local/bin
install: Repository:  https://github.com/casey/intermodal
install: Crate:       imdl
install: Tag:
install: Target:      x86_64-unknown-linux-musl
install: Destination: /home/user/.local/bin
install: Archive:     https://github.com/casey/intermodal/releases/download//imdl--x86_64-unknown-linux-musl.tar.gz

gzip: stdin: not in gzip format
tar: Child returned status 1
tar: Error is not recoverable: exiting now

The return code is 2.
I use Linux Mint 20.4 Cinnamon x64.

Thanks for reporting this! I just opened #509 which should fix the issue. Once it merges, deployment to GitHub pages should happen shortly, which will update https://imdl.io/install.sh.


Add CDNJS version badge in readme

Hi @una
This will add a badge that shows the lib version on CDNJS and link to its page on CDNJS!
Thanks!

Awesome, thanks!


[Task]: Prepare documentation of deployment process of Playground

What needs to happen?
Prepare a deployment strategy, reads like a recipe book and defers to convention over verbose redundant documentation
Issue Priority
Priority: 3
Issue Component
Component: beam-playground

.take-issue


Question: Dynamically display price on top of chart when user click on chart. Revert to current price when user unpresses.

I want to use the const price = LineChart.usePrice(); to have a dynamic price display on top of the chart. Once a user touches the chart, the price display should be adjusted to the price the user is currently hovering over. Once the user unpresses, it should go back to the original price. How do I do this?

I've been able to console.log the updated value with useAnimatedReaction, but I haven't been able to successful display the price change.

Figured it out.
Since it's usePrice returns an animated / shared value, you cannot use a normal Text component.
Instead, you need to create a shared value with the useDerivedValue hook from reanimated.
This, you then need to visualize using the AnimatedText component also exported from wagmi charts.

hello? can you share the code please? @julian-dotcom

Hi @cihangir-mercan.
One way to do this is to use LineChart.PriceText and formatPrice which are both exported by the library. Here I am printing the currently selected price on the chart. If no price is selected then the price from the last data point is used:
import { View } from 'react-native';
import { LineChart, formatPrice } from 'react-native-wagmi-charts';

// ... inside your component:

const formatValue = ({ value }) => {
  'worklet';
  const valToPrint = value
    ? value
    : data[data.length - 1].value.toFixed(2);

  return formatPrice({ value: valToPrint.toString() });
};

<LineChart.Provider data={data}>
  <View style={{ position: 'relative' }}>
    <LineChart.PriceText
      style={{ position: 'absolute', top: 10, left: 10 }}
      format={formatValue}
    />
    <LineChart>
      <LineChart.Path color="black" />
      <LineChart.CursorCrosshair />
    </LineChart>
  </View>
</LineChart.Provider>

https://github.com/coinjar/react-native-wagmi-charts/assets/27768821/28fdd084-a33f-454c-ab85-fe68f152cc01
Hope this helps.

Hi @cihangir-mercan.
One way to do this is to use LineChart.PriceText and formatPrice which are both exported by the library. Here I am printing the currently selected price on the chart. If no price is selected then the price from the last data point is used:
import { View } from 'react-native';
import { LineChart, formatPrice } from 'react-native-wagmi-charts';

// ... inside your component:

const formatValue = ({ value }) => {
  'worklet';
  const valToPrint = value
    ? value
    : data[data.length - 1].value.toFixed(2);

  return formatPrice({ value: valToPrint.toString() });
};

<LineChart.Provider data={data}>
  <View style={{ position: 'relative' }}>
    <LineChart.PriceText
      style={{ position: 'absolute', top: 10, left: 10 }}
      format={formatValue}
    />
    <LineChart>
      <LineChart.Path color="black" />
      <LineChart.CursorCrosshair />
    </LineChart>
  </View>
</LineChart.Provider>

Screen.Recording.2024-05-07.at.17.09.05.mov
Hope this helps.

yeah. nice one. i have found out this too. thanks.


Upgrade packages to latest tarball

Here is a list of open source packages and their corresponding version (along with the corresponding version in brew), for packages that list their version:
bash zos-version 5.2.15 brew-version 5.2.15
byacc zos-version 2.0 brew-version 20230521
expect zos-version 5.45.4 brew-version 5.45.4
getopt zos-version 1.1.6 brew-version unknown
git zos-version 2.41.0 brew-version 2.41.0
gnulib zos-version 0.1.5861-2ba7c brew-version unknown
less zos-version 632 brew-version 633
libiconv zos-version 1.17 brew-version 1.17
libpipeline zos-version 1.5.6 brew-version 1.5.7
libssh2 zos-version 1.10.0 brew-version 1.11.0
libtool zos-version 2.4 brew-version 2.4.7
libxml2 zos-version 2.9.12 brew-version 2.11.4
libxslt zos-version 1.0.0 brew-version 1.1.38
man-db zos-version 2.11.2 brew-version 2.11.2
ncurses zos-version 6.3 brew-version 6.4
sshpass zos-version 1.09 brew-version unknown
sudo zos-version 1.9.13p3 brew-version unknown
vim zos-version 9.0.1482 brew-version 9.0.1550
which zos-version 2.21 brew-version unknown
xmlto zos-version 0.0.28 brew-version 0.0.28
yq zos-version 4.33.3 brew-version 4.34.1

Notes:
byacc seems to show a different format
a few packages don't seem to have a brew package (probably a bug in the tool)
less, libpipeline, libssh2, libtool, libxml2, libxslt, ncurses, vim, yq are down-level
bash, expect, git, libiconv, man-db, xmlto are up-to-date

https://github.com/ZOSOpenTools/meta/pull/412 should cover this as we roll it out

closing since 412 covers this.


bf16: auto support

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.

🔖 Feature description
rather than having to have the user figure out if their gpu supports bf16, we can have an bf16: auto and then do detection if bf16 is supported and set bf16/fp16 appropriately.
✔ Solution
see above
❓ Alternatives
No response
📝 Additional Context
No response
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this feature has not been requested yet.
[X] I have provided enough information for the maintainers to understand and evaluate this request.

Thanks @simhallq. Closed by #1116


[Issue]:  How does Graph RAG retrieve?

Describe the issue
It seems that in the Table 3 in the paper(https://arxiv.org/pdf/2404.16130), a certain level of communities will be traversed and the answer will be obtained. But the source code seems to select the communities where the entities related to the query are located?
Steps to reproduce
No response
GraphRAG Config Used
No response
Logs and screenshots
No response
Additional Information

GraphRAG Version:
Operating System:
Python Version:
Related Issues:

Are you referring to local or global search, and do you have a code snippet that demonstrates the discrepancy?


wrap_fp16_model is not in mmdet.core

Looks like tools/test.py is not working with mmdet master as wrap_fp16_model was moved to mmcv in this commit.

Thanks for your kind report. PR https://github.com/open-mmlab/mmdetection/pull/3822 will fix this issue. We accelerate the fix process.

Fixed in https://github.com/open-mmlab/mmdetection/pull/3822


Update to latest CUDA, use NCCL_jll

This updates NCCL.jl to CUDA.jl 5, and to use the new NCCL_jll build.

Okay, this is now working locally for me: I've tried to keep the argument names/keywords args somewhat consistent with MPI.jl.
Remaining items

[ ] Set up CI: if you can set up a pipeline, I'm happy to try to figure it out. Note that NCCL appears to require at least 3 devices, see https://github.com/NVIDIA/nccl/issues/1125.
[ ] Set up Docs

@maleadt ?

We don't have enough GPUs to take 3 devices and assign them to a single worker, only for testing of NCCL.jl...

Fair enough. Okay, I think this is ready. It may need a DOCUMENTER_KEY if it doesn't already exist.

FWIW I've had no issue using libraries which use NCCL on systems with only two devices. Maybe a local environment or config issue?

I'll try setting up a 2 GPU worker.


Consider adding special support for YouTube URLs

Or possibly even yt-dlp? I notice there's a caching argument in one of the Docker files. Would be neat to use yt-dlp with basically any video URL.
Just spent a few minutes creating a FreshRSS extension that uses this. It's a nice enhancement!

@Hydrophobefireman I ended up forking this and updating it to support oEmbed along with a bunch of other features like dark/light mode: https://github.com/joestump/reader

looks awesome!


Support gradle

Since Android Studio was announced at Google I/O this year, Android Studio and gradle is getting popular and  popular (and seems standard now). I added gradle support in this PR, and hope aar binary will be uploaded to maven repo soon.

+1

Gradle is now the default build system.


How to set up the prompts for LangSam for the segmentation?

@luca-medeiros Do we need setup the prompts in a specific file and just simply set it to the text_prompt = "wheel" for example?

https://github.com/luca-medeiros/lang-segment-anything?tab=readme-ov-file#usage


Incorrect dependencies

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
This conda-forge has a number of differences in its dependencies as compared with the upstream repo.
Installed packages
N/A

Environment info
N/A

Dependencies were updated during the  of the last version: https://github.com/conda-forge/geomstats-feedstock/pull/6


Change icons depending on number of errors

Fix for issue #35. Has some minor issues like not changing the icon back if you kill ghcid with ctrl-c, but nothing major that I could find.
I added a conditional dependency on Win32 for windows users and kept all code within CPP blocks, so it should only affect windows users. I tested it on a linux box and it still builds fine there.

Awesome, thanks! I was originally thinking of constructing bitmaps etc for the icons, but reusing the message box icons is significantly nicer. I've modified the code a bit, refactored some pieces etc. The only real additional functionality is that I capture the icons before and restore them after, so it restores the icon if you do a Ctrl-C.
In limited testing today I'm already finding it useful functionality, far more than I thought I would :)


Option to control enemy spawner and turret placement

During map gen, it may seem like you have fewer biters than normal because biters are generated and placed before the maze canyon is placed. This causes them to disappear. With the right luck you might not see biters for 30-40 km.
This is evidenced by the fact that if you explore early you might see lone biters hanging out or running around, or worms doing likewise.
If it's possible biter nests should be controlled by the mod during generation to prevent this issue, and make the placement of biters more consistent.

This will be an option to control the placement of spawners and turrets at chunk generation. It will be on by default and be a map setting.


Add missing dependency on zlib

Unconditionally required here: https://github.com/cottsay/find-object/blob/635efcd6220cabc258b88c54b1ae4279035bbe31/CMakeLists.txt#L134
rosdep rules for zlib: https://github.com/ros/rosdistro/blob/eee874f940271cea17534d9cf9d87a526ba1dce2/rosdep/base.yaml#L8857-L8873
Should resolve RHEL 9 build failures on the buildfarm: https://build.ros2.org/view/Rbin_rhel_el964/job/Rbin_rhel_el964__find_object_2d__rhel_9_x86_64__binary/
Thanks!

Great! thx!


Can't build with Zig master

Currently blocked by https://github.com/ziglang/zig/issues/16336

Fixed by 632134d0aa193257fd7215b8fb73adb3bf187df2


pcre2: ftp.pcre.org looks defunct

Motivation for this change
According to pcre.org github is now the official mirror for releases.
cc @ttuegel
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Hash is still ok?

Hash is still ok?

It's possible. It's not generated code, but uploaded assets:

https://github.com/PhilipHazel/pcre2/releases/tag/pcre2-10.37

Though a simple TOFU check by breaking the hash can help verify.

Hash is still ok?

It's possible. It's not generated code, but uploaded assets:
*<EMAIL_ADDRESS>(release)](https://github.com/PhilipHazel/pcre2/releases/tag/pcre2-10.37)

Though a simple TOFU check by breaking the hash can help verify.

Verified locally and it is the same hash.
Waiting for ofborg and then merging.

Hash is still ok?

$ nix-build . -A pcre2.src --check
checking outputs of '/nix/store/12lw9q06n2j6h5x8hsamdsh3yjk2i6g9-pcre2-10.37.tar.bz2.drv'...

trying https://github.com/PhilipHazel/pcre2/releases/download/pcre2-10.37/pcre2-10.37.tar.bz2
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   658  100   658    0     0   2043      0 --:--:-- --:--:-- --:--:--  2043
100 1688k  100 1688k    0     0   779k      0  0:00:02  0:00:02 --:--:-- 1745k
/nix/store/an15rwsimzw7ncwkxxn22ahkjh73npjk-pcre2-10.37.tar.bz2

as far as I can tell, yes.


Added ISMAI as new university

Added University Institute of Maia: http://www.ismai.pt/pt

@hugoamvieira Pull request merged, thank you!


Improve CSI Storage Classes Documentation

Can we get improved documentation here? Unfortunately, the new CSI documentation is... not really very practical, and fails to address basic use cases:

It provides zero guidance for migrating workloads from the old storage classes to the new CSI-based ones.
It does not provide any overview of behavioral or semantic changes between the old classes and the new ones.
It doesn't provide the same level of useful information provided for the old storage classes in the existing documentation. For example, it doesn't cover a basic use case such as creating a Persistent Volume and specifying the azure file share information directly in the PV and not the storage class. Is it done still the same? Different?

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

ID: 7013047f-1846-c266-6759-6897b6694eb9
Version Independent ID: e98f3106-93e3-f425-ffdd-0260c8a494cf
Content: Enable Container Storage Interface (CSI) drivers on Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/csi-storage-drivers.md
Service: container-service
GitHub Login: @palma21
Microsoft Alias: mlearned

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Hi @tomasr - thanks for your feedback! The doc has been significantly updated since your comment. I will close this issue now, but will you please let me know if you're still experiencing these issues?
#please-close


Update README.md

:white_check_mark: Build Magneto 36 completed (commit https://github.com/shaynevanasperen/Magneto/commit/813e7b7f89 by @JosephWoodward)

@JosephWoodward Thanks, this is a nice improvement.


Locking warnings

Killing tasks could be too risky without human confirmation

[ ] Define a mechanism to "warn" when allocations have exceeded expected times by a long margin..

Related to #21

This requires adequate time estimation


fix c7n-mailer-replay, broken after #3017

The c7n-mailer-replay command was broken by #3017 which added support for custom template directories via a new CLI option on c7n-mailer, but didn't replicate that option for replay. This fixes that issue so that c7n-mailer-replay works again. This also decodes the message payload before printing it, since EmailDelivery was switched to utf8 encoding after replay was written.
Before:
$ c7n-mailer-replay -c mailer.yml -t -j message.json message.txt
2018-11-16 11:14:39,935 - c7n_mailer.replay - DEBUG - Reading message from: message.txt
2018-11-16 11:14:39,935 - c7n_mailer.replay - DEBUG - Read 1881 byte message
2018-11-16 11:14:39,935 - c7n_mailer.replay - DEBUG - base64-decoding and zlib decompressing message
2018-11-16 11:14:39,936 - c7n_mailer.replay - DEBUG - Loaded message JSON
Traceback (most recent call last):
  File "/home/jantman/GIT/cloud-custodian/bin/c7n-mailer-replay", line 11, in <module>
    load_entry_point('c7n-mailer', 'console_scripts', 'c7n-mailer-replay')()
  File "/home/jantman/GIT/cloud-custodian/tools/c7n_mailer/c7n_mailer/replay.py", line 121, in main
    tester.run(options.dry_run, options.print_only)
  File "/home/jantman/GIT/cloud-custodian/tools/c7n_mailer/c7n_mailer/replay.py", line 54, in run
    addrs_to_msgs = emd.get_to_addrs_email_messages_map(self.data)
  File "/home/jantman/GIT/cloud-custodian/tools/c7n_mailer/c7n_mailer/email_delivery.py", line 245, in get_to_addrs_email_messages_map
    list(to_addrs)
  File "/home/jantman/GIT/cloud-custodian/tools/c7n_mailer/c7n_mailer/email_delivery.py", line 267, in get_mimetext_message
    'template', 'default', self.config['templates_folders'])
KeyError: 'templates_folders'

After:
$ c7n-mailer-replay -c mailer.yml -T -t cais_c7n/mailer-templates -j message.json message.txt
2018-11-16 11:20:03,936 - c7n_mailer.replay - DEBUG - Reading message from: message.txt
2018-11-16 11:20:03,937 - c7n_mailer.replay - DEBUG - Read 1881 byte message
2018-11-16 11:20:03,937 - c7n_mailer.replay - DEBUG - base64-decoding and zlib decompressing message
2018-11-16 11:20:03,937 - c7n_mailer.replay - DEBUG - Loaded message JSON
2018-11-16 11:20:04,082 - c7n_mailer.replay - INFO - Would send email to<EMAIL_ADDRESS>u'bar@example.com')]
2018-11-16 11:20:04,155 - c7n_mailer.replay - INFO - Send mail with subject: "[cloud-custodian] Orphaned cloud-custodian CW Event rules in us-east-1"
<!DOCTYPE html>
<html lang="en">
...

Ohhhh yeah, I forgot that in the PR description, sorry.
c7n-mailer itself uses -t for the templates. I figured that since this is rarely used at all and likely only used manually, it would be better to unify -t between both mailer and replay, than keep compatibility with replay's old flags but end up using -t for different things in the two of them.


2.0 docs => 404 Not Found

Hi!
https://kivy.org/doc/stable/ => 404 Not Found
https://kivy.org/doc/stable-1.10.1/ => OK

Fixed now. It ran out of space.


updated_custom_token_list.json

updated_custom_token_list.json

Please fill out the application in the correct format. Incorrect formats will result in the issue being closed.

updated_custom_token_list.json


redo header styling

the header can be a little janky at times.
examples of the header being janky:

To fix this I changed the header to use flexbox rather than putting position absolute on every element.
the header is now dynamic and responsive improving the experience for all users but especially mobile users.

Looks the same to me on the live site versus your changes, even when resizing the window?

@FantasmicGalaxy this is probably due to the old CSS being cached by your browser. 7c80241 should help.


Retroactive loading of dataset is broken

For starters, the sample on the main README is broken:
follows_delta = Forgetsy::Delta.fetch('user_follows')
follows_delta.incr('UserFoo', date: 2.weeks.ago)
follows_delta.incr('UserBar', date: 10.days.ago)
follows_delta.incr('UserBar', date: 1.week.ago)
follows_delta.incr('UserFoo', date: 1.day.ago)
follows_delta.incr('UserFoo')

Each of the calls to follows_delta.incr results in a NOOP because Forgetsy::Set.valid_incr_date returns false in each case that date is explicitly included...
puts follows_delta.fetch

{"UserFoo"=>0.9997776321896626}

Thanks for spotting that! The readme had omitted the date option from the call to the factory method used to create the Delta. If you switch to version 0.2.7 I've added a replay flag which makes this simpler, so you can create a delta like so -
follows_delta = Forgetsy::Delta.create('user_follows', t: 1.week, replay: true)

And then start adding retrospective events to it. In the case of the example above it will actually return something like
{ 'UserFoo' => 0.667, 'UserBar' => 0.500 }


BB 3D calculation resulting in NaN.

Calculation of Gx and Gy returning an Inf, specifically offset due to bb is small (1e-56), when for example there is 1HO in IP1 and no LR.
Proposal to fix inside /xfields/beam_elements/beambeam_src/beambeam3d.h in line 309 an extra if statement for the case of the closed orbit being close to 0.
Example to run which produces the issue with a lattice example with a simple bb HO element:

import numpy as np
import json
import xobjects as xo
import xtrack as xt
import xpart as xp
import xfields as xf

# load hllhc json from test data
 url = 'https://raw.githubusercontent.com/xsuite/xtrack/main/test_data/hllhc15_noerrors_nobb/line_and_particle.json'

 resp = requests.get(url)
 resp_parsed = re.sub(r'^jsonp\d+\(|\)\s+$', '', resp.text)
 data = json.loads(resp_parsed)

line = xt.Line.from_dict(data['line'])
line.particle_ref = xp.Particles.from_dict(data['particle'])

line0 = line.copy() # the line is frozen otherwise

# insert bb3d element at s = ip1['s'], first find s of ip1
for ii, ee in enumerate(line0.elements):
    if line0.element_names[ii] == 'ip1':
        ip1_s = line0.get_s_elements()[ii]
        print(ip1_s)

# insert bb element at IP1
line0.insert_element(element=xf.BeamBeamBiGaussian3D, at_s=ip1_s, name='ip1_bb_HO')

# instantiate an arbitrary beam beam element inside the Gaussian3D with arbitrary parameters
bb_coupling = False
params = {}
params['phi'] = 0
params['alpha'] =  0
params['x_bb_co'] =  0
params['y_bb_co'] =  0
params['charge_slices'] = [0.0]
params['zeta_slices'] =  [0.0]
params['sigma_11'] =  1
params['sigma_12'] =  0
params['sigma_13'] =  0
params['sigma_14'] =  0
params['sigma_22'] =  1
params['sigma_23'] =  0
params['sigma_24'] =  0
params['sigma_33'] =  0
params['sigma_34'] =  0
params['sigma_44'] =  0
if not (bb_coupling):
    params['sigma_13'] =  0.0
    params['sigma_14'] =  0.0
    params['sigma_23'] =  0.0
    params['sigma_24'] =  0.0

params["x_co"] = 0
params["px_co"] = 0
params["y_co"] = 0
params["py_co"] = 0
params["zeta_co"] = 0
params["delta_co"] = 0
params["d_x"] = 0
params["d_px"] = 0
params["d_y"] = 0
params["d_py"] = 0
params["d_zeta"] = 0
params["d_delta"] = 0

newee = xf.BeamBeamBiGaussian3D(old_interface=params)
line0.element_dict['ip1_bb_HO'] = newee

tracker = xt.Tracker(line=line0)
# will get error on the closed orbit using the twiss
tw = tracker.twiss()

Code segment containing bug: https://github.com/xsuite/xfields/blob/0664de28196bf040b3be8e45edb71fd50310bd17/xfields/beam_elements/beambeam_src/beambeam3d.h#L308

Working on a fix.


release v2.0.2 w/ an iojs fix

see: https://github.com/sass/node-sass/pull/791#issuecomment-85425643
pretty please :heart_eyes_cat:

:+1:

:+1:

:+1:
Release v2.1.1. :tada:

:+1:


Building multi-arch Docker image using debian as base

In the CI I used the cartesi/machine-emulator:0.14.0-bullseye as base.
Needed to downgrade clang-tidy and clang-format to versions available in debian bullseye
Final images built in depot.dev
Minor styling changes in README

Thanks @tuler ! This will be done on the https://github.com/cartesi/server-manager/pull/6 PR (WIP).


Remove deprecated MLOperandDescriptor.type

Now latest Chrome has supported MLOperandDescriptor.dataType, we can remove deprecated MLOperandDescriptor.type.

@ibelem, @huningxin, PTAL, thanks!
cc/ @BruceDai, you may want to remove it from webnn-polyfill as well.

cc/ @BruceDai, you may want to remove it from webnn-polyfill as well.

Thanks for kindly reminder. 👍
WebNN Polyfill API has already supported MLOperandDescriptor.dataType since https://github.com/webmachinelearning/webnn-polyfill/pull/238 was landed.

cc/ @BruceDai, you may want to remove it from webnn-polyfill as well.

Thanks for kindly reminder. 👍 WebNN Polyfill API has already supported MLOperandDescriptor.dataType since webmachinelearning/webnn-polyfill#238 was landed.

Oh, so remaining the WPT tests I think.

Right. I will submit PR to fix it on WPT side. Thanks.


fix: phpstan error in v1.8.8

Description
See https://github.com/codeigniter4/CodeIgniter4/actions/runs/3197626681/jobs/5221091878
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

Now passed
https://github.com/codeigniter4/CodeIgniter4/actions/runs/3198075094

Thanks for the quick response @ddevsr! And the mention + review @datamweb. Waiting on that last test.

The PHPUnit failure looks unrelated (Predis connection failure m). Merging!


fix switching between versions

How to reproduce: Go in to 1.8, then 1.12, then 1.8 -> throws an error

fixed with https://github.com/Leg0shii/ParkourCalculator/pull/23


After update to js-controller 3.3.1 wrong types in log

iobroker 3.3.1
node v12.22.1
npm 6.14.12
mihome-vacuum 3.1.1

2021-04-30 14:41:54.307  - warn: mihome-vacuum.0 (1587) State value to set for "mihome-vacuum.0.history.allTableJSON" has wrong type "string" but has to be "object"

please delete the adapter and reinstall it. this object is fixed in 3.1.1


Added js functionality on cart and all product pages

Added js functionality to the cart and also added functionality to the products to add dynamically when clicked on add to cart. Also the prices and GST and total also gets updated as per the need, And the remove button on cart also works. Thank you for this oportunity got to learn alot while building this. The 2 js files would be cart.js and index.js

Added js functionality to the cart and also added functionality to the products to add dynamically when clicked on add to cart. Also the prices and GST and total also gets updated as per the need, And the remove button on cart also works. Thank you for this oportunity got to learn alot while building this. The 2 js files would be cart.js and index.js

If you could add a message after clicking "add to cart", an item has been added. Will have a good impact

i was thinking about making something like that... sure i will make it..

Thank you for this Nitin, i have learned so much that i had not learned while watching tutorials and was stuck on tutorial hell.


logsumexp and negative exp's

This is a feature request to allow "subtraction" for example
log(exp(a) - exp(b))
so that the function logsumexp([a, b], scale = [1, -1]) or something similar. Would be similar to scipy implementation of logsumexp
https://github.com/scipy/scipy/blob/f2ec91c4908f9d67b5445fbfacce7f47518b35d1/scipy/misc/common.py#L18

Does the log_diff_exp function cover what you're after?

It would if it had a vector signature like log_sum_exp. Any chance that can get added?

Yes, @andrjohns has added vectorized versions of:
 bessel_first_kind
 bessel_second_kind
 beta
 binary_log_loss
 binomial_coefficient_log
 choose
 falling_factorial
 fdim
 fmax
 fmin
 fmod
 gamma_p
 gamma_q
 hypot
 lbeta
 ldexp
 lmgamma
 log_diff_exp
 log_falling_factorial
 log_inv_logit_diff
 log_modified_bessel_first_kind
 log_rising_factorial
 modified_bessel_first_kind
 modified_bessel_second_kind
 multiply_log
 owens_t
 pow
 rising_factorial

These are available on develop and will be available in the upcoming 2.25 release (next week will be the release candidate).

I think they meant log_diff_exp(vector) rather than log_diff_exp(vector, real) , is that right @spinkney?

Yep, this can be defined for vectors/matrices like log_sum_exp, I put together a rough prototype on godbolt as a proof-of-concept: https://godbolt.org/z/xET4cE
I can work this up later next week

Actually what I want is an elementwise log_diff_exp. I'm not sure if that would even be faster than a loop though.

Can you clarify that a little? What signature are you after? Something like log_diff_exp(matrix, real)?

For example
vector[N] theta1;
vector[N] theta2
target += log(inv_logit(theta1) - inv_logit(theta2));

After a bit of algebra that becomes
numerator = log(exp(-theta1) - exp(-theta2);
denom = log(1 + exp(-theta1) + exp(-theta2) + exp(-theta1 - theta2));

so the numerator is log_diff_exp(-theta1, -theta2) and the denom is log_sum_exp(0, -theta1, -theta2, -(theta1 + theta2).

Yep, the elementwise log_diff_exp(vector, vector) will be available in 2.25.
Incidentally, where you have:
log(inv_logit(theta1) - inv_logit(theta2));

In 2.25, there will also be an elementwise log_inv_logit_diff, so you can just write:
log_inv_logit_diff(theta1, theta2);


liferay plugin project

Used in liferay plugin projects development.

Closing as too specialized. We have existing Java, Maven and Eclipse templates which should cover this scenario.


(1.10) [FLINK-19109][task] Emit watermarks when FileReader is closing

What is the purpose of the change
Emit watermarks while ContinuousFileReaderOperator is being closed (see FLINK-19109).
(This is a fix for 1.10)
cc: @alpinegizmo , @pnowojski
Verifying this change
This change added FileReadingWatermarkITCase.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): yes
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? no

@rkhachatryan I tested this PR with the original application that prompted the bug report, and it does not solve the problem.

@rkhachatryan I was mistaken: I have multiple 1.10-SNAPSHOT builds, and I tested against the wrong one. This PR does work correctly for this use case.

The feature is very useful, why close it? @rkhachatryan

@gabrywu this issue has already been fixed for 1.10.3 by another PR https://github.com/apache/flink/pull/13374


problem: Could not resolve "@noble/secp256k1"

I get this error when using the latest package from npm.
If I install @noble/secp256k1 I then get other errors from the ndk package when ndk uses secp256k1.

@gazhayes can you confirm which version of NDK you're using? I used to see this error but haven't see it in a while.

Ah thanks. Looks like NPM did something silly (surprise). I just updated manually to 0.6.0 and it's resolved.


rail: remove unneeded call to XGetWindowAttributes()

It is already called above "if" with the same arguments.

+1 it's a clear fix. merging.


Demo not working in Chrome 52?

I did the following:

created account
signed in
logged out
clicked "sign in"

Nothing seemed to happen. What am I missing?

Are you trying this on https://credential-management-sample.appspot.com/ or on your local?
Are you on desktop or on mobile? (this shouldn't matter)
Is your credential correctly stored? Find a key icon next to star icon in the URL bar if desktop.

@aaronshaf Do you have the "Offer to save passwords" option enabled in your Chrome settings?
That really stumped me.  I assumed the API was just broken completely in Chrome.  Finally, I realized that I needed to enable this in Chrome (I had it disabled because I use LastPass).


Installing noble on windows

Hi there,
I have problems installing noble in windows (namely windows 7, but I need to support more versions of windows). Some dependency (bluetooth-hci-socket) fails to build. Is there any easy workaround for this that can easily be applied by not so technical customers?
cheers,
Torsten

Closing this for now, discussion moved to https://github.com/sandeepmistry/node-bluetooth-hci-socket/issues/35#issuecomment-245872352.


proposal: add explicit decision doc for large changes

#33670 talks about how to identify large changes. Having done that, we should expect a large discussion. It can't be the case that you have to read the whole discussion to participate. That doesn't scale.
The goal of the discussion should be to identify all the pros and cons and tradeoffs involved in the decision. Often that makes the decision clear, but not always, especially for large ones.
We've always collected these pros and cons informally; at best we occasionally write a discussion summary. For large changes it may make sense to write a formal, explicit “decision document” that lays out the pros, cons, and tradeoffs. Then the goal of the discussion is to complete this document. Someone who wants to participate in the discussion could start by reading the document and checking whether anything is left to add.
The current proposal process description hints at the discussion feeding back into the main proposal doc, but I think it would be clearer to have a separate document that is the discussion summary / inputs to the decision / tradeoffs.
Someone proposing a large change would be responsible for incorporating discussion into the design document, or else for finding someone who will.
The same kind of decision document could be written for design drafts (maybe a pre-decision document).

Often times, digging through an extremely large issue for decision summaries to reference can be tedious. However, as you previously mentioned, Github doesn't have a way of pinning comments. Maybe an issue could have a corresponding external log of decision docs? That does however sound like some effort to maintain.

The very long discussion on the slog proposal is a great example of why having a summary doc for larger changes would be valuable for all people participating in a proposal discussion. I lost count of how many times I had to hit the "Load more..." button on that issue, in an attempt to discern whether my concern had already been voiced.


chore(nightscout): Pass token using Api-Secret header

Description
This PR changes the Nightscout app to pass a configured token using the Api-Secret header instead of using the token query param. This is safer since URLs are often kept in access logs.
This header is documented in Nightscout's Swagger doc.
I have tested this against my own instance.
Copilot

🤖[deprecated] Generated by Copilot at 61e5f1b
Summary
🔒🔄📄

Improved security and compatibility of Nightscout app by using Api-Secret header for authentication. Modified apps/nightscout/nightscout.star to send the header with http.get.

To access the Nightscout API
The old way was not very spry
So they added headers
To avoid any wedges
And used Api-Secret to fly

Walkthrough

Use Api-Secret header for authentication with Nightscout API (link, link)

@IsThisPaul If you get a chance, would you mind testing this PR against your instance?

@IsThisPaul No problem! I tested against my instance which requires tokens, and everything is working well. I just thought you'd like to verify 🙂

@IsThisPaul This is just another way to pass the token. All instances support the query param and the header, and just like before it is excluded if not set.

LMK if/when this should be merged. @IsThisPaul

@matslina It's ready to be merged!

I have not had the chance to review this but trust Gabe’s testing.
—
Paul
On Thu, Nov 30, 2023 at 7:38 AM Mats Linander @.***>
wrote:

LMK if/when this should be merged. @IsThisPaul
https://github.com/IsThisPaul
—
Reply to this email directly, view it on GitHub
https://github.com/tidbyt/community/pull/2065#issuecomment-1833901617,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYE3ZK6DKESS3IO3RZW3ZYDYHCK6BAVCNFSM6AAAAAA73BHXQ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZTHEYDCNRRG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

I just received a report of a NS 401 error from a user with a NS token. Did
the old method break?
—
Paul
On Thu, Nov 30, 2023 at 7:52 AM Gabe Cook @.***> wrote:

@matslina https://github.com/matslina It's ready to be merged!
—
Reply to this email directly, view it on GitHub
https://github.com/tidbyt/community/pull/2065#issuecomment-1833934645,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYE3ZK7K7SZ5YWO4YIJDOY3YHCMRJAVCNFSM6AAAAAA73BHXQ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZTHEZTINRUGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Interesting. I checked Nightscout source and Api-Secret has been supported for years, so I'm unsure why some users get a 401. For now, I can add a fallback to the old method if a 401 is returned.


How to use serve-index behind a reverse proxy ?

Hi,
I'd like to use serve-index behind a reverse proxy that adds a root context, i. e :
mydomain.com/root-context/my-folder --> localhost:3000/my-folder
The serve-static module works fine (if I enter the full adresse by hand, it is correctly resolved), but URLs generated by serve-index are absolute URLs, and are not taking in account the root context.
Example of url generated by serve-index:
mydomain.com/my-folder/other (should be mydomain.com/root-context/my-folder/other)
To summarize : I can access my content with direct URLs, but I can't navigate through serve-index's interface.

Why using absolute URLS and not relative ones ? For instance, the Apache module uses relative URLs when serving directory files list.
If absolutes URLs are absolutely needed, how can I configure a root context ?

Thanks for your help

Hi @alemarre, the reason this module has historically used absolute URIs in the links is because this module will respond with or without a trailing slash. For example, if you loaded mydomain.com/root-context/my-folder in your web browser, then a link to image.png would end up going to mydomain.com/root-context/image.png, not mydomain.com/root-context/my-folder/image.png, since the relative link it relative within the root-context folder since there is no trailing slash in the URL.
The way to resolve this is that you need to get your req.originalUrl set correctly before entering this module. The best way is to get it setup in Express at the top, so anything in your application trying to generate URLs or even HTTP redirects can do so using the full URL when necessary.
Depending on how your proxy works, you could possibly have the proxy add a request header X-Original-Url to the backend request and then add the following to the top of your Express application:
app.use(function (req, res, next) {
  req.originalUrl = req.headers['x-original-url'] || undefined
  next()
})

Thanks a lot for your great explanation

No problem, @alemarre!


PCM rework

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
closes #1646
What are the significant changes in functionality due to this change request?
Adding PCM validation capability for steady state problems

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[ ] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

Sorry, I cannot find the module to set random seed using utils. Could you please let me know how? Thank you.

From: Jimmy-INL @.>
Sent: Wednesday, December 1, 2021 11:36 PM
To: idaholab/raven @.>
Cc: Dongli Huang @.>; Author @.>
Subject: Re: [idaholab/raven] PCM rework (PR #1726)
@Jimmy-INL requested changes on this pull request.

In framework/Models/PostProcessors/Validations/PhysicsGuidedCoverageMapping.pyhttps://github.com/idaholab/raven/pull/1726#discussion_r760756176:

@@ -137,6 +137,7 @@ def _evaluate(self, datasets, **kwargs):
msr_mean = 0.0
msr_std = 0.01*np.std(yapp_std)
msr_numSmpl = 1000

 np.random.seed(0)

We do not use the NumPy random seed, Please use the one in the utils.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/idaholab/raven/pull/1726#pullrequestreview-821041466, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APV5HEDPCBVPGFCDXMGFYGLUO3ZTVANCNFSM5JCWXCPQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Job Mingw Test on 6bbf96d : invalidated by @joshua-cogliati-inl
restarted civet

Job Precheck on 07152d6 : invalidated by @Jimmy-INL

I have updated the pcm scripts to enable multiple responses of the Experiments/Features and fixed several issues raised in the comments. Would you please start another review process? @wangcj05 @Jimmy-INL Thank you!

Updated per your comments, changed to static responses, enabled multiple responses, and tested with linear model. Would you please take a look? @wangcj05 @Jimmy-INL Still a few issues remaining, will discuss tomorrow. Thank you!

I have updated what we discussed in yesterday's meeting. Would you please start a review of it? @wangcj05 @Jimmy-INL Thank you.

@huang714 Could you resolve the conflicting files in this PR?

@huang714 Could you resolve the conflicting files in this PR?

It shows 'Already up to data' after 'git merge devel'. I couldn't find the three conflict files now.

@huang714 white space issue, could you resolve it?

I dealt with the comments on the doc file, however, I cannot compile the user manual to check whether the changes are displayed correctly. The problem seems to be a version.tex file under raven/doc/ missing (raven_user_manual.tex line 441). Please let me know if there is any other issue. Thanks! @wangcj05 @Jimmy-INL

I dealt with the comments on the doc file, however, I cannot compile the user manual to check whether the changes are displayed correctly. The problem seems to be a version.tex file under raven/doc/ missing (raven_user_manual.tex line 441). Please let me know if there is any other issue. Thanks! @wangcj05 @Jimmy-INL

FYI: to see the problems with generating the documents, run:
./make_docs.sh --verbose

Hi @huang714, fyi, the following test failed on most of the Linux machines:

Hi @huang714, fyi, the following test failed on most of the Linux machines:
FAILED: Diff tests/framework/PostProcessors/Validation/test_validation_gate_pcm

This is because I used a system error related condition to determine whether this is singular matrix. The new condition is changed to condition number greater than 100. Let me know if there is a better way. Thank you!

All jobs on 1ec4845 : invalidated by @wangcj05
retesting

All jobs on 1ec4845 : invalidated by @wangcj05
retesting

Job Mingw Test on 1ec4845 : invalidated by @wangcj05
fetch issue

Job Test qsubs sawtooth on 060341b : invalidated by @wangcj05
FAILED: Diff tests/cluster_tests/test_mpiqsub_parameters Diff tests/cluster_tests/test_mpiqsub_nosplit

Job Mingw Test on 060341b : invalidated by @wangcj05
TimeoutError: Timed out while waiting for node to startup.

checklist is good, and tests passed.


Add master-worker paradigm

Most applications will have a master to store app states, assign work, and manage any global information.
Currently, the "controller + etcd" does part of the work.
We can make this more general to have a master task for user to extend from.

The master logic should be implemented by user. Again, the philosophy of TaskGraph is to hide low level details of user. User implements only the logic like what state to tell master, what master should tell a worker, how master save states, how master keep hot standby, etc.

I think it need to implement three new interface in assigning work of master
GetFreeWork(workType) (workID uint64, error), a node wants to get a free work of appointed type
OccupyWork(workType, workID) (ok bool, error), represents task want to occupy this work
ReleaseWork(taskType, taskID) error, release task to grab next work
(
create three etcd dir is a choice.
/FreeWorkDir/WorkType/workID  // to identify a free task
/PossessWok/taskID // to store workPath of task possess
/OccupiedWorkDir/WorkType/workID // indicate possessed work (may be useless)
).

@plutoshe
Can you give more context?

If we want to implement a work assign on master node, a slave need to ask the master which work is free, whether occupy work is successful.
And if current work is finished, this salve node wants to occupy a new work, it is necessary to tell the master node that it would like to release the current work

The first two interface mostly like the logic of WaitFreeTask(), TryOccupyTask().
The third interface is a mechanism to support node constantly grab free work to process.

Agreed.
I think this is more specific to mapreduce.
But master-worker communication is necessary.
I'm going to implement master base and worker base for user to extend:
type Master interface {
  NotifyWorker(WID, input) output
}
type Worker interface {
  NotifyMaster(input) output
}

And both master, worker task should have the following callbacks:
Notified(From, input) output

In this way, you can easily implement the communication logic, data exchange, task logic on top.

LGTM, it is more general implementation comparing to my idea.
But I think the master need to provide worker a interface to tell master how to response specific message.
type Master interface {
  NotifyWorker(WID, input) output
  Possess(input, function)
}

The master is implemented by user. Framework provides the API and helps user implement the communication details.

Ok, I know the logic. I used to think user only implement worker.

OK. After some time spent on thinking, we can actually hack on current interface and do what we want.
We can use 0 as master and others as workers.
Task#DataRequest will invoke the grpc method defined by user. Put in whatever messages needed in your definition.
After the coordination, workers should have their work.

Yeah, it is reasonable.

BTW, I have some idea about task mechanism.
A task of different type might have different life span, it could be fix by varying topology change. But I think it is not beautiful. Is there some more flexible method to solve this problem.
In addition,as for current framework, a node process a taskID until the whole framework shutdown.
However there are some node do nothing at some time due to the difference of total live task number between epochs. Is it possible to release a idle task to wait for other failed task?

Let's not care about those if they aren't blockers to mr impl.
We need to rewrite the framework at some point. Keep those in another issue if needed.

I got it


Add Tracktion Engine module and licence field

List of 3rd party JUCE modules

| Name          | Description   | Licence       |
| ------------- | ------------- | ------------- |
| [Gin](https://github.com/FigBug/Gin) | A few extras for juce (FilesystemWatcher, DownloadManager, ImageBlur etc.) | |
| [ATK](https://github.com/mbrucher/AudioTK) | An audio digital processing toolbox based on a workflow/pipeline principle | |
| [Tracktion Engine](https://github.com/Tracktion/tracktion_engine) | The DAW framework for timeline based audio applications | GPL/Commercial |```

Added with #55b7429


AttributeError: module 'main' has no attribute 'spec'

I am trying to run pandas_profiling function on a pandas dataframe on python version 3.6.4.
The package got imported successfully but when I am trying to use the function
AD_profile=pandas_profiling.ProfileReport(final_dataset)
then I get  the following error.
Traceback (most recent call last):

  File "<ipython-input-35-e667d7ef2617>", line 1, in <module>
    AD_profile=pandas_profiling.ProfileReport(final_dataset)

  File "C:\ProgramData\Anaconda3\Lib\site-packages\pandas_profiling\__init__.py", line 66, in __init__
    description_set = describe(df, **kwargs)

  File "C:\ProgramData\Anaconda3\Lib\site-packages\pandas_profiling\describe.py", line 349, in describe
    pool = multiprocessing.Pool(pool_size)

  File "C:\ProgramData\Anaconda3\lib\multiprocessing\context.py", line 119, in Pool
    context=self.get_context())

  File "C:\ProgramData\Anaconda3\lib\multiprocessing\pool.py", line 174, in __init__
    self._repopulate_pool()

  File "C:\ProgramData\Anaconda3\lib\multiprocessing\pool.py", line 239, in _repopulate_pool
    w.start()

  File "C:\ProgramData\Anaconda3\lib\multiprocessing\process.py", line 105, in start
    self._popen = self._Popen(self)

  File "C:\ProgramData\Anaconda3\lib\multiprocessing\context.py", line 322, in _Popen
    return Popen(process_obj)

  File "C:\ProgramData\Anaconda3\lib\multiprocessing\popen_spawn_win32.py", line 33, in __init__
    prep_data = spawn.get_preparation_data(process_obj._name)

  File "C:\ProgramData\Anaconda3\lib\multiprocessing\spawn.py", line 172, in get_preparation_data
    main_mod_name = getattr(main_module.__spec__, "name", None)

AttributeError: module '__main__' has no attribute '__spec__'

It would be great if someone could help me fix it.
Thanks!

I have the same issue. How did you solve it?

Not sure. I tried re-installing the python package using pip and then the error went away for a couple of days. However, it has started happening again since day before yesterday!.


Adapt to the WebRequest API in MV3

The GPC signals are not updated for the extension as we still rely on the API that has been phased out since MV3. This issue attempts to address the upgrade of the WebRequest API to correctly append the GPC signal.

I am looking into this now, working on migrating the backend to use declarativeNetRequest API. I think I have a good idea of how to do this and can give an update at the meeting tomorrow.

@ekuller, thank you for looking into this. We also need to update the logic, which we depend on when updating the GPC signals. For example, the following caching mechanism is unlikely to work since the service worker does not maintain these global variables. I can look more into this.

In order to track third party requests I had to make the service worker persistent. This is due to a bug that prevents the service worker from waking up when the webRequest listener should be called. I used a solution from stack overflow. If we decide we do not want to use this work around we will have to get rid of tracking third party web requests.
I have migrated the gpc backend for scheme 6 to use the declarativeNetRequest API, and suggest how we should migrate the backend for the other schemes below:

[ ]  Scheme 0- use updateDynamicRules API to create and manage the urls/domains on which we want to enable/disable gpc
[ ] Scheme 1 - similar to scheme 0, but if gpc is enabled on all future domains we add a universal rule as in scheme 6 and add domains whose requests we don't want to enable gpc on to the "excludedDomains" property
[ ] Scheme 2 - I think the gpc backend would be handled in the implementation of scheme 1's backend, if not, it would require the same features
[ ] Scheme 3 - after the profile is selected, we would dynamically need to add a rule that is applied to all urls and customize the "domains" property to the profile; the rule we need to be removed and updated if the profile is updated in the options page
[ ] Scheme 4 - similar to scheme 3, aside from how domains are selected
[ ] Scheme 5 - similar to scheme 1 and 3
[ ] Scheme 6 - static rule that sets gpc universally and is enabled and disabled dependent on setting (already implemented)

This is also of interest to @kalicki1 and @OliverWang13.

@bella-tassone, @chunyuema, and I met on Saturday and discussed delegation of webRequest/GPC migration. These are the tasks I have come up with for the coming week:

@chunyuema -- update dynamic rule (I think only one rule really needs to exist at a time for this purpose) when user profile changes in schemes 3-5, setting the domains condition equal to an array of the appropriate domains dictated by the disconnect list; something like:

  chrome.declarativeNetRequests.updateDynamicRules({addRules:{
    "id" : 1,
    "action" : { "type" : "modifyHeaders",
      "requestHeaders": [
          { "header": "Sec-GPC", "operation": "set", "value": "1" },
          { "header": "DNT", "operation": "set", "value": "1" }
        ]},
    "condition" : {
      "urlFilter" : "|*"
      "domains": [LIST_OF_DOMAINS_TO_SEND_GPC_TO]
    }
  }, removeRuleIds:[1]
  })

@bella-tassone -- add and remove rules that set GPC for individual urls for schemes 0-2 and learning phase of 5; use {chrome.runtime.sendMessage({greeting: "NEW RULE", d: currentDomain, id: domains[currentDomain].id})} and {chrome.runtime.sendMessage({greeting: "RM RULE",  id: domains[currentDomain].id})} . When adding domains to the domain list they should now contain two properties: domains[currentDomain].id=Object.keys(domains).length+1 and domains[currentDomain].id set to true or false dependent on whether we are sending GPC. Code that uses the values of domains[currentDomain] currently should be updated to use the values of domains[currentDomain].id (decent amount of refactoring). I can definitely help you with this.
@ekuller  -- help with Bella's large task if/when needed and implement logic/rules for when there is customized domain list, but universal setting for all future urls being visited.

declarativeNetRequest Documentation

Nice plan and teamwork!

Note from the discussion yesterday

Below is a conceptual overview of how our extension is currently working. This might be a good reference to have a quick overview of everything we have changed so far @SebastianZimmeck

The plan moving forward:

Banner UI Layer: @bella-tassone
Options page UI Layer: @ekuller
Rule Set Layer: @chunyuema

Very nice, @chunyuema! (Maybe, even put this, or a final version, in the readme as an architectural overview.)


Core Set + Rotated Cycles vs Revised Core Set Settings

I have 1 Core Set + the two rotated cycles.
Since the "Use X Core Set/s"  now only applies to the revised set, its quite difficult to know how many of each card in the revised set I actually have. Can we have separate options for the original and revised sets?
Also is there a way to filter out rotated cards?

I can answer that last question for you about filtering out rotated cards.
When you are using the deckbuilder, click the Collections tab near the top (it's right next to the Settings tab where you get the "Use X Core Set/s" option). There, if you unselect the boxes for Core Set, Genesis, and Spin, the cards in those boxes will be filtered out.

I just tested this by building a new deck, selecting my collection as only Core Set, Genesis and Spin and setting "Use 1 Core Set" and I am seeing the correct quantities displayed on the Build tab:

Is the issue that you want to use post-rotation (either FFG or NISEI) legality, but quantities from a pre-rotation product? So for example, you wouldn't see Zaibatsu Loyalty on the Build tab at all (as it only appears in the Original Core) and you'd see just two copies of Aggressive Secretary (as there are two in the Original Core, but only 1 in Revised Core and SC19)?
If so, that's a little more tricky because some people will want to build for a certain number of cores (like if playing a SC19 only event for example, or building SC19 Core Experience decks), rather than using the "Use X Core Sets" option as an extra layer over legality to help them understand what physical cards they have available to them.

Is the issue that you want to use post-rotation (either FFG or NISEI) legality, but quantities from a pre-rotation product? So for example, you wouldn't see Zaibatsu Loyalty on the Build tab at all (as it only appears in the Original Core) and you'd see just two copies of Aggressive Secretary (as there are two in the Original Core, but only 1 in Revised Core and SC19)?

yuppers, thats the one
there kind of needs to be separate use X pre-rotation core sets and use X post-rotation core sets options for post rotation


Fixing #66 setting fixed number of digits in fractions

See https://github.com/DIGGSweden/dgc-java/issues/66#issuecomment-856183106.


Should @figspec/react be a dependency instead of a devDependency?

As seen in:
https://github.com/pocka/storybook-addon-designs/blob/2e0b7631cc3a90a013c9b525436510a46af512a2/packages/storybook-addon-designs/package.json#L22
I'm asking because when using the latest alpha of storybook-addon-designs, 6.0.0-alpha.2, I had to separately install @figspec/react or my project wouldn't compile.
PS: thank you for this awesome addon! ❤

The reason behind not including the packages (@figspec/react and @figspec/components) is an attempt to reduce package dependencies: package managers download unused packages even if a user doesn't use Figspec preview.
https://storybook-addon-designs-rhgclupaa.vercel.app/?path=/story/docs-figma-figspec-readme
I may change the strategy in the future but will keep as-is in alpha releases.

Thank you @pocka, that makes total sense, I appreciate the care you put in ensuring your plugin doesn’t bloat people's builds. I'm currently trying to speed up Storybook build times on a very large codebase, so I'm thankful!


Allow closing of modal now that payment is recognised instantly. Fixes #341

Allow closing of modal now that payment is recognised instantly.
Fixes #341
http://beta.elementaryos.org/branch/close-modal

I've got some issues with alignment and height.

*sighs
Browser and Version?

Firefox 35.0.1 :-P

Doesn't do that for me. *sighs

@emersion Could you send me a print-screen that looks like this?

Okay. It was my cache or something like that. Sorry :-S


Incompatible with lwjgl3ify 1.5.1

I updated lwjgl3ify from 1.4.x and it stopped working
log: https://mclo.gs/8ihTP0j

To be fair, lwjgl3ify is sort of a very heavy handed hack that breaks compat with most mods. Breakages are the norm, more than the exception.

If you disable jar_discoverer_cache in the CoreTweaks config, the crash goes away.

Oh hey, good call.

Pretty sure this was just an issue on my side, I initialized the jar discoverer cache really early, and it caused some other classes to get loaded earlier than normal, which broke LWJGL3ify's mixins that happened to be targeting those classes.

**HE'S BACK**
Cool, thanks.
Any chance it might ever be possible to get smart moving to not crash with it eventually or it's one of those really up in the air things where both mods just change too much stuff to actually get them to work together properly?

IIRC that was a pretty simple issue on SM's side that I just never bothered to fix, will have a look at it later.


[README] Fix Clock example

Put the runTimer array in a block

I suppose it work without it as well.

Unfortunately it doesn't seem to. Recreated the example, and it throws.


PR Error #22667

Just a question
[X] bug report
[ ] feature request
[ ] question about the decisions made in the repository
[X] question about how to use this project

unsupported file modified: /dev/null

New automerge — Not eligible for automerge (click details for more information)

Token Address: EJt3iszkR6GKiV1YSyefB2GaHsXniyVAjAtS9mrxyTZR
Token Name: ColliderCoin
Token Symbol: MCCN
Logo: \assets\mainnet\EJt3iszkR6GKiV1YSyefB2GaHsXniyVAjAtS9mrxyTZR\collider_blue_coin.png
Link to the official homepage of token: https://collidercoin.com/

When I click on the issue all it says is "unsupported file modified: /dev/null".  However, all I did was add my logo, add my token, and merge.  It doesn't give me any details of what the issue is.

closed PR


[BACKLOG-24264] PDI Service/Job supporting 'result-set' capability

Bringing in POC work from @pedrofvteixeira's branch to master.
Verified and tested locally, works as expected. PDI-Service's /ktr endpoint now takes two new parameters resultSetStepName and resultSetCopyNumber
@pentaho/rogueone

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/pan/Pan.java
  engine/src/main/java/org/pentaho/di/pan/PanCommandExecutor.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change: -1.1% :small_red_triangle_down:
Complexity Change: -1.1% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change: -3.4% :small_red_triangle_down:
Line Change: -3.8% :small_red_triangle_down:

org.pentaho.di.pan.PanCommandExecutor

Branch Change: -2.9% :small_red_triangle_down:
Complexity Change: -1.7% :small_red_triangle_down:
Instruction Change: -1.9% :small_red_triangle_down:
Line Change: -2% :small_red_triangle_down:

org.pentaho.di.trans.step.StepMeta

Branch Change: -0.6% :small_red_triangle_down:
Complexity Change: -0.5% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Method Change: -1% :small_red_triangle_down:

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change: -2.1% :small_red_triangle_down:
Complexity Change: -2.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -1.1% :small_red_triangle_down:
Complexity Change: -1.3% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:
Line Change: -0.4% :small_red_triangle_down:

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Branch Change: -2.6% :small_red_triangle_down:
Complexity Change: -1.5% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.8% :small_red_triangle_down:

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change: -1.7% :small_red_triangle_down:
Complexity Change: -1.2% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

License header violations

:heavy_exclamation_mark: Copyright year is not the current year on engine/src/main/java/org/pentaho/di/pan/Pan.java. Found 2018. Was expecting 2019

:heavy_exclamation_mark: Copyright year is not the current year on engine/src/main/java/org/pentaho/di/pan/PanCommandExecutor.java. Found 2018. Was expecting 2019

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/pan/Pan.java
  engine/src/main/java/org/pentaho/di/pan/PanCommandExecutor.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change: -1.1% :small_red_triangle_down:
Complexity Change: -1.1% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.pan.PanCommandExecutor

Branch Change: -2.9% :small_red_triangle_down:
Complexity Change: -1.7% :small_red_triangle_down:
Instruction Change: -1.9% :small_red_triangle_down:
Line Change: -2% :small_red_triangle_down:

org.pentaho.di.trans.step.StepMeta

Branch Change: -0.6% :small_red_triangle_down:
Complexity Change: -0.5% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.calculator.CalculatorMetaFunction

Branch Change: -8.7% :small_red_triangle_down:
Complexity Change: -3% :small_red_triangle_down:
Instruction Change: -0.7% :small_red_triangle_down:
Line Change: -0.9% :small_red_triangle_down:

org.pentaho.di.trans.steps.datagrid.DataGridMeta

Branch Change: -2% :small_red_triangle_down:
Complexity Change: -1.7% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Method Change: -1% :small_red_triangle_down:

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -3.3% :small_red_triangle_down:
Complexity Change: -3.9% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:
Line Change: -0.4% :small_red_triangle_down:

org.pentaho.di.trans.steps.sort.SortRowsMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceinsert.SalesforceInsertMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change: -0.2% :small_red_triangle_down:


Bump minor protocol version.

Now that https://github.com/near/near-sdk-rs/pull/154 is merged we need to bump minor protocol version since it is breaking collections API by renaming Map.

I think we should CHANGELOG.md in the root as well, to put all new changes there, especially breaking


server: add server info metrics for DBasS

Signed-off-by: iosmanthus<EMAIL_ADDRESS>What problem does this PR solve?
This pull request adds server info metrics via a GaugeVec in format: (version, githash) -> startup_ts.
Problem Summary:
Related changes

Need to cherry-pick to the release branch

Check List
Tests

Manual test (add detailed scripts or steps below)

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note

server: add server info metrics for DBasS

Could you provide information about how it looks like?

Could you provide information about how it looks like?

Could you provide information about how it looks like?

Could you provide information about how it looks like?

Is 1.612B duplicate for this metric?

Is 1.612B duplicate for this metric?

Is 1.612B duplicate on this metric?

Metric itself contains the timestamp information.

Is 1.612B duplicate on this metric?

Metric itself contains the timestamp information.

Is 1.612B duplicate on this metric?

Metric itself contains the timestamp information.

But the metrics' timestamps don't stand for the server startup timestamp...

Is 1.612B duplicate on this metric?

Metric itself contains the timestamp information.

But the metrics' timestamps don't stand for the server startup timestamp...

Missing leading v in the version?

Missing leading v in the version?

/lgtm

/lgtm

/lgtm

/lgtm

/merge

/merge


install error

remix$ npm install
npm WARN deprecated<EMAIL_ADDRESS>We're super 😸  excited that you're trying to use ES2015 syntax, but instead of making more yearly presets 😭 , Babel now has a better preset that we recommend you use instead: npm install babel-preset-env --save-dev. preset-env without options will compile ES2015+ down to ES5 just like using all the presets together and thus is more future proof. It also allows you to target specific browsers so that Babel can do less work and you can ship native ES2015+ to user 😎 ! We are also in the process of releasing v7, so please give http://babeljs.io/blog/2017/09/12/planning-for-7.0 a read and help test it out in beta! Thanks so much for using Babel 🙏, please give us a follow on Twitter @babeljs for news on Babel, join slack.babeljs.io for discussion/development and help support the project at opencollective.com/babel

will take a look.
Note that if you want to build Remix IDE, you'll have to pull ethereum/browser-solidity.
see readme: https://github.com/ethereum/remix

this is a warning the installation is still ok

@owocki - I have a branch under jwm/babel-update that I'd like to make a PR on, however believe I may need access to write to the repo.

@yann300 would you be able to help @jmichaliga with repo access in order to make a PR here? cc @owocki

Also @jmichaliga would you mind clicking 'Start Work' on Gitcoin to claim this one as yours?

@jmichaliga Are you still working on this one? Want to make sure as it has now been claimed elsewhere on Gitcoin. @mavv Have you begun work on this issue yet?

@vs77bb  I solved the issue but i cannot submit the solution since i don't have any ether purchased

@mavv reach out to myself or vivek on slack and well send you enough ETH to cover gas :)


Publish pre-compiled bosh release publicly

bosh-deployment repo has a bosh.yml with a not-well-known URL to a precompiled version of bosh.
OMG OMG OMG it makes bosh create-env/bosh-init so much faster.
I didn't know until Friday that the whole implementation of precompiled releases was changed - that they are now embedded inside a normal release. This is awesome.
Could we please link to these pre-compiled URLs in bosh.io/releases?
Really, I have so many excited questions that follow on from this - can bosh.io/releases include pre-compiled releases (for latest stemcell) rather than just src release?
Does bosh finalize-release work on compiled releases? If I export a compiled release during my pipeline and finalize-release it will I get what I expect? I assume so since the compiled bits are referenced by package guids not by version numbers.
I'm so excited for a pre-compiled-by-default future.

@drnic i was just chatting with @tylerschultz last week about having finalize release work with compiled releases. that way consumption of releases continues to work via official ways (git repo with yaml receipts).

Ok lovely

Closing as the various questions are answered/covered above or in other tickets.


FunctionalTests: move JwtTokenExtractor tests to Functional tests and consume app id and password from environment

…

Pull Request Test Coverage Report for Build 73253

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 78.534%

Totals

Change from base Build 73181:
0.0%

Covered Lines:
4394

Relevant Lines:
5595

💛  - Coveralls

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.3.1


Cannot use dynamic specPattern

Issue comes from using cypress-parallel.
I have the need to segregate some test and then parallelize them.
I tried using the tag, but that didn't work since the filtering comes after the cypress-parallel spec strategy.
Seeing that it uses this library to retrieve the specPattern I tried using a dynamic approach using an env variable, but it doesn't resolve the configuration:
> cypress-parallel run --node 1:1

Error: Expected a string literal for specPattern, but got Identifier
    at parseTestingTypeObject (/home/rsaleri/projects/cypress-parallel-issue/node_modules/@badeball/cypress-configuration/dist/cypress-configuration/cypress-post10-configuration-parser.js:38:27)
    at parseTestingTypesObject (/home/rsaleri/projects/cypress-parallel-issue/node_modules/@badeball/cypress-configuration/dist/cypress-configuration/cypress-post10-configuration-parser.js:70:29)
    at parsePost10Configuration (/home/rsaleri/projects/cypress-parallel-issue/node_modules/@badeball/cypress-configuration/dist/cypress-configuration/cypress-post10-configuration-parser.js:165:44)
    at parseConfigurationFile (/home/rsaleri/projects/cypress-parallel-issue/node_modules/@badeball/cypress-configuration/dist/cypress-configuration/cypress-post10-configuration.js:191:83)
    at resolvePost10Configuration (/home/rsaleri/projects/cypress-parallel-issue/node_modules/@badeball/cypress-configuration/dist/cypress-configuration/cypress-post10-configuration.js:112:68)
    at resolveConfiguration (/home/rsaleri/projects/cypress-parallel-issue/node_modules/@badeball/cypress-configuration/dist/cypress-configuration/index.js:73:78)
    at run (/home/rsaleri/projects/cypress-parallel-issue/node_modules/@badeball/cypress-parallel/lib/cli.js:124:83)

Process finished with exit code 1

I put together a Proof of Concept here: https://github.com/rsaleri/cypress-parallel-issue

This is expected behavior, as parsing is best effort. You can use cypress-split instead, which implements a different strategy.

Thank you very much, I'll look into it


Decide how to calculate errors

One idea is to have another key in the YAML that specifies which error measure gets calculated:
error_function:
  - ibug_68_48_points_face_size

But this may be overly complicated. Perhaps the number of landmarks expected should be handled by another key? Or perhaps normalisation should be separate?
The biggest problem is something like Intraface which only returns 49 points - what if we trained on 68?

Implemented as landmark processes.


.container plugin not working out of the box

The "container" plugin that ships with Tailwind doesn't seem to be working for me.
my tailwind.js file:
 plugins: [
    require('tailwindcss/plugins/container')({
      center: true,
      padding: '1rem',
    }),
  ]

my mix file:
mix.js('resources/assets/js/app.js', 'public/js')
    .postCss('resources/assets/css/app.css', 'public/css')
    .tailwind()

I can verify that my tailwind.js file is being used (by switching color values and seeing them reflected)
I'm on 0.5.2
A co-worker set up a brand new tailwind project and verified the same issue.
This could very well be PEBCAK, but I'm kinda out of ideas.
If you don't quickly identify this as an actual bug I will dig deeper, just want to make sure this isn't currently an issue effecting everyone.

I got this issue too with latest version 0.6.1 and i have @tailwind components; in my scss

@SimonJ Make sure that @tailwind components; is behind @tailwind utilities; in your css file.

Thanks @rdbok it was the config file that needed to be changed

@tailwind components

It would be nice to add this to the docs :)

@crysfel Where else do you think it should be added? It's already explained in the installation section:
https://tailwindcss.com/docs/installation#3-use-tailwind-in-your-css

This tripped me up as well. One place where it could be mentioned is in the container section in the docs: https://tailwindcss.com/docs/container -- I remember seeing the @tailwind components directive while installing but not thinking I was using any plugins (i.e. this was a clean Tailwind installation), I ignored and forgot it.

Attempting to use Tailwind 1.1.3 with Gatsby and Emotion. The container class throws an error. Error: "Couldn’t resolve Tailwind class name: container"
Yes I have included
@tailwind base;
@tailwind components;
@tailwind utilities;
in global.css

Can you share the actual code that's throwing the error? Sounds like maybe you are using a Babel macro or some other non-official thing?

Attempting to use Tailwind 1.1.3 with Gatsby and Emotion. The container class throws an error. Error: "Couldn’t resolve Tailwind class name: container"
Yes I have included
@tailwind base;
@tailwind components;
@tailwind utilities;
in global.css

Same problem here. Gatsby, Emotion, Babel, PostCss. For files, see below. I don't know if it is emotion at fault... when I avoid using emotion and just throw a container class into a JSX div:

....when I mouse over the pt-24 style in VSS, the style resolves, but container does not.
Some details:
style.css
@tailwind base;
@tailwind components;
@tailwind utilities;

...

tailwind.config.js
module.exports = {
  theme: {
    fontFamily: {
...
    },
    extend: {
      colors: {
...
        }
      }
    }
  },
  variants: {},
  plugins: []
};

babel-plugin-macros-config.js
module.exports = {
  tailwind: {
    styled: `@emotion/styled`,
    config: `./tailwind.config.js`,
    format: `auto`,
  },
}

postcss.config.js
const tailwindcss = require(`tailwindcss`);

module.exports = {
  plugins: [
    tailwindcss(`./tailwind.config.js`),
    require(`autoprefixer`),
    require(`cssnano`)({
      preset: `default`
    })
  ]
};

Can you share the actual code that's throwing the error? Sounds like maybe you are using a Babel macro or some other non-official thing?

Hi @adamwathan, Indeed it seems related to tailwind.macro. I'm having the same issue using styled-components and tailwind. The workaround I found for now was to add the container class directly to my styled component.

import React from "react"
import styled from 'styled-components'
import tw from 'tailwind.macro';

const Main = styled.main`
  ${tw`w-full flex flex-wrap mx-auto container`}; <- with container throw an error
  ${tw`w-full flex flex-wrap mx-auto`}; <- without container doesn't throw an error

const Layout = ({ children }) => {
  <>
    {children} <- workaround add the container class directly on the component
  </>
}

export default Layout

Anyone else trying to use tailwindcss v1.2 grid utilities with tailwind.macro v1.0.0-alpha.10 and getting
Error: Couldn’t resolve Tailwind class name: grid
This seems related to the 'container' class bug...

Anyone else trying to use tailwindcss v1.2 grid utilities with tailwind.macro v1.0.0-alpha.10 and getting
Error: Couldn’t resolve Tailwind class name: grid
This seems related to the 'container' class bug...

@jeremyswann Probably due the plugin (tailwind.macro v1.0.0-alpha.10) not being updated for v1.2 and those classes not being defined in static-styles.js or dynamic-styles.js: https://github.com/bradlc/babel-plugin-tailwind-components/tree/master/src

@hacknug yeah it's pretty much depreciated, I'm now using twin.macro which is currently maintained 👍
But as it stands if you want to use 'container' you still need to use it as a class.

Maybe it's worth to add on the docs which part related to what tailwind package (utilities/base/components)


chore: bump webpack and webpack-cli dependencies

It is necessary to fix the error Cannot read property 'properties' of undefined
https://github.com/plotly/dash-component-boilerplate/issues/12#issuecomment-425040481

I just ran in to this error myself

On this branch, I got an error with babel-core
$ yarn && yarn build
yarn install v1.17.3
warning package-lock.json found. Your project contains lock files generated by tools other than Yarn. It is advised not to mix package managers in order to avoid resolution inconsistencies caused by unsynchronized lock files. To clear this warning, remove package-lock.json.
[1/4] Resolving packages...
[2/4] Fetching packages...
info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
[3/4] Linking dependencies...
warning " ><EMAIL_ADDRESS>has unmet peer dependency "babel-core@6".
[4/4] Building fresh packages...
success Saved lockfile.
$ npm run build
npm WARN lifecycle The node binary used for scripts is /tmp/yarn--1569420475859-0.4898397621835602/node but npm is using /home/psego/.volta/tools/image/node/10.16.3/6.9.0/bin/node itself. Use the `--scripts-prepend-node-path` option to include the path for the node binary npm was executed with.

><EMAIL_ADDRESS>build /home/psego/Development/OpenSource/lint-diff
> webpack

Hash: 3c7d06c3693317dfe717
Version: webpack 4.30.0
Time: 63ms
Built at: 09/25/2019 10:07:56 AM
           Asset      Size  Chunks             Chunk Names
    lint-diff.js  6.62 KiB    main  [emitted]  main
lint-diff.js.map  3.52 KiB    main  [emitted]  main
Entrypoint main = lint-diff.js lint-diff.js.map
[./index.js] 2.84 KiB {main} [not cacheable] [built] [failed] [1 error]

ERROR in ./index.js
Module build failed (from ../node_modules/babel-loader/lib/index.js):
Error: Cannot find module 'babel-core'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:636:15)
    at Function.Module._load (internal/modules/cjs/loader.js:562:25)
    at Module.require (internal/modules/cjs/loader.js:692:17)
    at require (/home/psego/Development/OpenSource/lint-diff/node_modules/v8-compile-cache/v8-compile-cache.js:161:20)
    at Object.<anonymous> (/home/psego/Development/OpenSource/lint-diff/node_modules/babel-loader/lib/index.js:3:13)
    at Module._compile (/home/psego/Development/OpenSource/lint-diff/node_modules/v8-compile-cache/v8-compile-cache.js:194:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
    at Module.load (internal/modules/cjs/loader.js:653:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
    at Function.Module._load (internal/modules/cjs/loader.js:585:3)
    at Module.require (internal/modules/cjs/loader.js:692:17)
    at require (/home/psego/Development/OpenSource/lint-diff/node_modules/v8-compile-cache/v8-compile-cache.js:161:20)
    at loadLoader (/home/psego/Development/OpenSource/lint-diff/node_modules/loader-runner/lib/loadLoader.js:18:17)
    at iteratePitchingLoaders (/home/psego/Development/OpenSource/lint-diff/node_modules/loader-runner/lib/LoaderRunner.js:169:2)
    at runLoaders (/home/psego/Development/OpenSource/lint-diff/node_modules/loader-runner/li


ID 3

Node ID : 044f788841b53ca2

confirmed:ID3
DocumentType:verified
Nationality:verified
DeviceDetails:verified
Experience:verified
Screenshot:verified

confirmed:ID3
DocumentType:verified
Nationality:verified
DeviceDetails:verified
Experience:verified
Screenshot:verified

Form Submitted, 10:40 am, Time Zone : UTC+3hours


Add URI, URL Prisms

Fix #488 , #489
Maybe there are better/simpler ways to generate Arbitrary, Cogen instances? Any feedback is welcome :)

@xuwei-k Thanks for your review.

I moved java.net.URL specific code to the jvm specific directory.
(I'm not sure if this is the best way to separate js/jvm specific code, though :sweat:)

About adding protocols, it seems that jar protocol has its own format: jar:<url>!/{entry},
and new URL("jar://host.com:123/path/to/jar") throws an exception, for example.
So I added a Gen[URL] for jar.
https://docs.oracle.com/javase/8/docs/api/java/net/JarURLConnection.html

LGTM, what do you think @xuwei-k ?

Great, thank you guys 👍