How to compute the max pool size of sharding-proxy ?

Project: Sharding-proxy
Version: v3.1.0 & v4.x
Question
How can i count the total connections of sharding-proxy ? should I use "netstat -an|grep :3307|wc -l" to compute on the server ?
And what's more, what's the max pool size of sharding-proxy ? Does it equal to total of "MaxPoolSize" in the config-master_salve.yaml and config-sharding.yaml ? For example, if i set MaxPoolSize to 50 with master node and 40 with slave node in config-sharding.yaml, and the MaxPoolSize be set to 100 with master node and 100 with slave node in config-sharding.yaml, then the total MaxPoolSize is 290 ?
Thank you !

try SCTL (Sharding-Proxy control language)
sctl:show cached_connections | Query the number of cached physical database connections in the current TCP connection

I got the result.
mysql> sctl:show cached_connections;
+--------------------+
| CACHED_CONNECTIONS |
+--------------------+
| 0                  |
+--------------------+
1 row in set (0.00 sec)
I wonder what's the max connection of sharding-proxy, just like the "max_connections" and "max_user_connections" of mysql. Bcoz there are so many config files like "config-master_slave.yaml" and "config-sharding_yaml", and every config file include at lest two "maxPoolSize" parameters by default value 50, maybe i have 5 logic schemas to use master-slave, and 2 logic schemas to use sharding, so i have 5 config files named config-master_slave*.yaml and 2 files named config-sharding*.yaml, so what's the total pool size ?
By the way, can i set the parameter maxPoolSize with different value to different schema ?

And sharding-proxy dont support "sctl:hint show status" and "sctl:hint show table status;"
mysql> sctl:hint show status;
ERROR 11001 (S1100): 1Could not support sctl type [sctl:hint show status].
mysql> sctl:hint show table status;
ERROR 11001 (S1100): 1Could not support sctl type [sctl:hint show table status].

@yanickxia same sctl:hint command can not be supported in sharding-proxy even configurated "proxy.hint.enabled: true".
mysql> sctl:hint show status;
ERROR 11001 (S1100): 1Could not support sctl type [sctl:hint show status].
proxy.transaction.type: LOCAL
proxy.opentracing.enabled: false
proxy.hint.enabled: true
query.with.cipher.column: true
sql.show: true

It is related with your application's TPS.
This is the same question for ask connection pool.


ISSUE-34-3: Add Life Cycle Events

Allow enabling lifecycle events passing following optional values via options object.
Available options:

setLifecycleEvents (boolean), defaults to false

Cherry picked into #49


OCPCLOUD-2426: Update to k8s 1.29

update the vendor and some minor fixups related to controller-runtime and klogr.

added a fix for the generate test

/approve
/lgtm


Error: Property 'utime' does not exist on type 'typeof Deno'

I got this on Deno 1.13.2 and also on latest 1.14.1
OS: MacOS Mojave 10.14.6
$ deno run --allow-read --allow-net --unstable --config tsconfig.json src/cli.ts
# ...
error: TS2339 [ERROR]: Property 'utime' does not exist on type 'typeof Deno'. 'Deno.utime' is an unstable API. Did you forget to run with the '--unstable' flag?
    await Deno.utime(dest, statInfo.atime, statInfo.mtime);
               ~~~~~
    at https://deno.land/std@0.93.0/fs/copy.ts:96:16

TS2339 [ERROR]: Property 'utimeSync' does not exist on type 'typeof Deno'. 'Deno.utimeSync' is an unstable API. Did you forget to run with the '--unstable' flag?
    Deno.utimeSync(dest, statInfo.atime, statInfo.mtime);
         ~~~~~~~~~
    at https://deno.land/std@0.93.0/fs/copy.ts:111:10

TS2339 [ERROR]: Property 'utime' does not exist on type 'typeof Deno'. 'Deno.utime' is an unstable API. Did you forget to run with the '--unstable' flag?
    await Deno.utime(dest, statInfo.atime, statInfo.mtime);
               ~~~~~
    at https://deno.land/std@0.93.0/fs/copy.ts:135:16

TS2339 [ERROR]: Property 'utimeSync' does not exist on type 'typeof Deno'. 'Deno.utimeSync' is an unstable API. Did you forget to run with the '--unstable' flag?
    Deno.utimeSync(dest, statInfo.atime, statInfo.mtime);
         ~~~~~~~~~
    at https://deno.land/std@0.93.0/fs/copy.ts:160:10

TS2339 [ERROR]: Property 'utime' does not exist on type 'typeof Deno'. 'Deno.utime' is an unstable API. Did you forget to run with the '--unstable' flag?
    await Deno.utime(dest, srcStatInfo.atime, srcStatInfo.mtime);
               ~~~~~
    at https://deno.land/std@0.93.0/fs/copy.ts:183:16

TS2339 [ERROR]: Property 'utimeSync' does not exist on type 'typeof Deno'. 'Deno.utimeSync' is an unstable API. Did you forget to run with the '--unstable' flag?
    Deno.utimeSync(dest, srcStatInfo.atime, srcStatInfo.mtime);
         ~~~~~~~~~
    at https://deno.land/std@0.93.0/fs/copy.ts:214:10

$ deno --version
deno 1.13.2 (release, x86_64-apple-darwin)
v8 9.3.345.11
typescript 4.3.5
$ deno upgrade
# ...
$ deno --version
deno 1.14.1 (release, x86_64-apple-darwin)
v8 <IP_ADDRESS>
typescript 4.4.2

run with --unstable

run with --unstable

That's strange.

OK, I found the cause. I also needed to add this to lib section in tsconfig.json:
{
    //...
    "lib": [
      "dom",
      "deno.ns",
      "deno.unstable", // <= add this
      "esnext"
    ]
     //...
}

I suppose this issue can be closed now. Thanks!


[Is this a normal node?] (Version: [1.11.0])

Issue Description
I have now succeeded in running a ripple node through the docker image provided here!
What I'm curious about is whether it is possible to perform roles such as "wallet deposit processing, withdrawal processing, wallet creation" through my Ripple node.
We will share the current rippled.cfg information and the output result of server_info.
please. I must know this.
Thank you for reading this long article.
Supporting Files
<rippled.cfg>
[server]
port_rpc_admin_local
port_ws_public
[port_rpc_admin_local]
port = 5005
ip = <IP_ADDRESS>
admin = <IP_ADDRESS>
protocol = http
[port_ws_public]
port = 80
ip = <IP_ADDRESS>
protocol = ws
[node_db]
type=NuDB
path=/var/lib/rippled/db/nudb
fast_load=1
[ledger_history]
256
[database_path]
/var/lib/rippled/db
[debug_logfile]
/var/log/rippled/debug.log
[sntp_servers]
time.windows.com
time.apple.com
time.nist.gov
pool.ntp.org
[ips]
r.ripple.com 51235
[validators_file]
validators.txt
[rpc_startup]
{ "command": "log_level", "severity": "info" }
[ssl_verify]
1
< server_info >
{
"result": {
"info": {
"build_version": "1.11.0",
"complete_ledgers": "82009983-82052432",
"hostid": "DENY",
"initial_sync_duration_us": "164115188",
"io_latency_ms": 1,
"jq_trans_overflow": "0",
"last_close": {
"converge_time_s": 3.001,
"proposers": 35
},
"load_factor": 1,
"peer_disconnects": "12",
"peer_disconnects_resources": "0",
"peers": 21,
"pubkey_node": "n9JvTVraubatQMgeXFTjF1zThDDoiVGL8ay1kB6BGJb1qcCkQobQ",
"server_state": "full",
"server_state_duration_us": "148290438438",
"state_accounting": {
"connected": {
"duration_us": "40947155",
"transitions": "2"
},
"disconnected": {
"duration_us": "117141235",
"transitions": "2"
},
"full": {
"duration_us": "149953562686",
"transitions": "53"
},
"syncing": {
"duration_us": "135668703",
"transitions": "55"
},
"tracking": {
"duration_us": "82059896",
"transitions": "55"
}
},
"time": "2023-Aug-24 04:01:27.937383 UTC",
"uptime": 150329,
"validated_ledger": {
"age": 2,
"base_fee_xrp": 1e-05,
"hash": "2595D8EDD4071146A64326E77A9C2C33E423B6D08C59EA9AA42CBEF6D71CC28A",
"reserve_base_xrp": 10,
"reserve_inc_xrp": 2,
"seq": 82052432
},
"validation_quorum": 28
},
"status": "success"
}
}
thankyou!!!!

That looks like a normal node that should be able to successfully submit any valid transactions including ones to send & receive money on mainnet. I don't see any online deletion settings, though; you may want to Configure Online Deletion so that the size of your ledger store does not grow indefinitely. Otherwise, you might end up storing 5 TB of data per year (rough estimate).
Feel free to reopen if you have further questions.


replace haskell formatter?

Apheleia's default haskell formatter, brittany, is no longer maintained as of a bit under a year ago.
I learned this while trying to run a full make fmt-docker, which failed on the brittany test; I was unable to get the installer working and it looks like even brittany's author has had issues with installation.
I'd propose removing brittany from apheleia and replacing it with another haskell formatter.   A brief bit of looking around turned up hindent and ormolu.
I'll have a go at adding formatters for those, but I'm not a haskell dev and don't have an opinion about which should be the new default.
Any input appreciated?

Please consider setting either fourmolu or ourmolu as the default Haskell formatter, I'd say these are the most widely used ones for industrial Haskell code today.

https://github.com/radian-software/apheleia/pull/221, merged, adds those two formatters. https://github.com/radian-software/apheleia/pull/222 fixes the brittany installation (it was just a matter of the ubuntu version being too old, though I agree with the brittany author that haskell package installation is a mess). I'd like to wait at least a little time after the new formatters are added to Apheleia (say, 3 months?) before switching to make one of them the default though. Just to give people a chance to try them out before swapping it for everyone.


[Doubt] Is there a channel to ask questions about the crate ?

I'm here asking because I did not notice any places where I could ask questions about how to do certain things since reading the documentation and finding the right wording can be a pain. Is there a Discord or any other group chat where people can ask trivial questions?

Hi
There isn’t a dedicated channel for the crate unfortunately.
But you can ask your questions here if you wish.

Ok, Thanks for the answer, I'll as the questions come up.
Em sáb, 15 de ago de 2020 13:29, Mohammed Alyousef<EMAIL_ADDRESS>escreveu:

Hi
There isn’t a dedicated channel for the crate unfortunately.
But you can ask your questions here if you wish.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MoAlyousef/fltk-rs/issues/267#issuecomment-674419903,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACB3K2MYSRBXMKRX2CM5RO3SA2ZWZANCNFSM4QAKK7LA
.


Downloads give service error

If I try to download any of the installers, it says "Service error"

I'm guessing a temporary problem with Bintray? They're working fine over here.


Kubectl restart command without specific deployment will restart all deployments

What happened:
I executed kubectl --context <my-context> -n <my-namespace> rollout restart deployment and all the deployments in the namespace performed a rolling restart.
What you expected to happen:
I would expect this to return an error stating that a specific deployment name is required.
How to reproduce it (as minimally and precisely as possible):

Setup a cluster and create more than 1 deployment.
Execute kubectl --context <my-context> -n <my-namespace> rollout restart deployment
Notice that all deployments perform a rolling restart.

Anything else we need to know?:
Environment:
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"27", GitVersion:"v1.27.4", GitCommit:"fa3d7990104d7c1f16943a67f11b154b71f6a132", GitTreeState:"clean", BuildDate:"2023-07-19T12:14:48Z", GoVersion:"go1.20.6", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"24+", GitVersion:"v1.24.16-eks-2d98532", GitCommit:"af930c12e26ef9d1e8fac7e3532ff4bcc1b2b509", GitTreeState:"clean", BuildDate:"2023-07-28T16:52:47Z", GoVersion:"go1.20.6", Compiler:"gc", Platform:"linux/amd64"}

AWS EKS
MacOS: Darwin Kernel Version 22.5.0: Thu Jun  8 22:22:22 PDT 2023; root:xnu-8796.121.3~7/RELEASE_X86_64 x86_64

I think, it was intentionally designed that way but that would be better to add an example in the help text to mention this behavior.

Seems like a default behaviour that we do not find in other kubectl commands.

I think, it was intentionally designed that way but that would be better to add an example in the help text to mention this behavior.

@ardaguclu Any idea where we can confirm this?

I think, it was intentionally designed that way but that would be better to add an example in the help text to mention this behavior.

Yeah, Add an example similar to "kubectl get po".

/triage accepted
/priority backlog

Issue has been fixed;
/close


test_id:1780 is three tests in reality

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:

/kind bug

/kind enhancement

What happened:
When checking for test_id:1780 I noticed that this is in fact a test table and there are three tests involved.
What you expected to happen:
Each of the tests should have it's own test_id
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

KubeVirt version (use virtctl version):
Kubernetes version (use kubectl version):
VM or VMI specifications:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

/triage build-watcher

See the issues marked with test_id but pointing at different tests: https://github.com/kubevirt/kubevirt/issues?q=is%3Aopen+is%3Aissue+test_id%3A1780

https://github.com/kubevirt/kubevirt/issues/3824
https://github.com/kubevirt/kubevirt/issues/3811

Hi Daniel, how one can generate free ids ?
it must be of Polarion or we can add some custom ids with other label or other value ?
such as
ctest_id:1234
or test_id:c1234

Yes, I think you are correct, test ids are generated in Polarion. Hey @ILpinto, maybe you can help here?

You need to add the test id in the It section like: [test_id:1234]


Can we encrypt and decrypt the images using this library ?

I want to encrypt the image in IOS(swift 4 or Objective C) and Decrypt in C# , using this framework can any one help me out.
Thanks in advance

Yes, this should work, since my merge of #8.


Add ops.nn.dot_product_attention

I haven't check the performance yet, but I believe adding this operation will be beneficial since both torch and jax have optimized it in their codebases.
From a performance perspective, we should be able to replace _compute_attention in MultiHeadAttention with this op if the input shapes are strictly 4D.
EDITED:
It seems that CI is using jax<=0.4.30 due to python version limitation. I implemented a pure numpy version of dot_product_attention for the unit tests.
For backends:

jax: Uses jax.nn.dot_product_attention if available. Otherwise, adapts the impl from jax==0.4.33.
numpy: Adapts the impl from jax==0.4.31 (no customizable vmap)
tensorflow: Adapts the impl from jax==0.4.31 (no customizable vmap)
torch: Uses torch.nn.functional.scaled_dot_product_attention

The main barrier should be the python version.
jax>=0.4.31 only supports python>=3.10.


Add slider bars for the different parameters in Neural Style Transfer

This seems trivial and might be something I get to myself.  But having slider bars for various options would be very nice.

Hey @thistleknot,
Thanks for the feature request. I don't have any immediate plans to add support for this but I'll keep the issue open for future reference. Thanks!

Made some progress
from __future__ import print_function
import copy

import kornia
from pytorch_generative import colab_utils
import torch
from torch import nn
from torch import optim
from torch.nn import functional as F
from torchvision import models
from torchvision import transforms
from ipywidgets import interact, interactive, fixed, interact_manual
import ipywidgets as widgets

print(colab_utils.get_device())

    #return styleWeight/100

weightStyle = widgets.IntSlider(min=-10, max=30, step=1, value=10);
weightContent = widgets.IntSlider(min=-10, max=30, step=1, value=10);

ui = widgets.HBox([weightStyle,weightContent])

def f(weightStyle, weightContent):
    print(weightStyle,weightContent)

out = widgets.interactive_output(f, {'weightStyle': weightStyle, 'weightContent': weightContent})
display(ui, out)

little bit better (realized I might have been applying weights to styles incorrectly)
imSize = widgets.IntSlider(min=128, max=768, step=128, value=512);
numSteps = widgets.IntSlider(min=256, max=2048, step=256, value=1024);
styleWeight = widgets.Dropdown(
    options=[('1', 1), ('1000', 1000), ('5000', 5000), ('1e9', 1e9)],
    value=5000,
    description='Style Weight:',
)

ui = widgets.HBox([imSize, numSteps, styleWeight])

def f(imSize, numSteps,styleWeight):
    print(imSize, numSteps,styleWeight)

out = widgets.interactive_output(f, {'imSize': imSize, 'numSteps': numSteps, 'styleWeight': styleWeight})
display(ui, out)

Closing this out since there's not plans to support this in the library.

Closing this out since there's not plans to support this in the library.


Describe which exactly app is supported for Dynamics

Hello,
the document provides list of applications.
However, application "Microsoft Dynamics 365" does not exists, please explain more in detail, which exactly application is supported.
For example:
Dynamics 365 Sales is supported
Dynamics 365 Business Central is not supported

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

ID: bd934939-4dbb-3a92-f996-56b9557f49ac
Version Independent ID: 3cda6b81-26e8-c9ac-e768-d912636a461b
Content: Grant controls in Conditional Access policy - Microsoft Entra
Content Source: articles/active-directory/conditional-access/concept-conditional-access-grant.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

Thanks for the ask! Reached out to a PM contact will let you know what I find out.
#label:"awaiting-product-team-response"
#label:"assigned-to-author", "doc-enhancement", "triaged"
#reassign:@MicrosoftGuyJFlo

Adding the following per the PG in the linked PR that should publish later today.

Microsoft Dynamics 365 for Phones
Micorsoft Dynamics 365 Sales


removing vcs-detect-changes from POWERLEVEL9K_VCS_GIT_HOOKS unexpectedly removes remote repo icon

Tested on master with the default config.

Clone a repo from github.
cd into it.
Observe the presence of github icon in the vcs prompt.
Remove vcs-detect-changes from POWERLEVEL9K_VCS_GIT_HOOKS. That is, set POWERLEVEL9K_VCS_GIT_HOOKS=(git-untracked git-aheadbehind git-stash git-remotebranch git-tagname). Restart zsh.
Repeat steps 1-2.
Observe the absence of github icon in the vcs prompt.

Expected: There is a github icon in the vcs prompt in 6.
Actual: There isn't.
That being said, it's undocumented how the presence or absence of hooks affects vcs segment. It would be nice to have it documented. But either way the current behavior is very surprising.

This is fixed on next with merging #1235 . Closing for now.


Update AndroidManifest.xml

Corrected the edittext cut off on keyboard

Login Page Edit text wont be cut off when keyboard is opened


From LLM generated codes/references are not visible in the FHIR-creator from scratch form on main

On the fhirchain branch however they are

ValueString are not displayed on both

components aren't displayed as well

Bundle_26eca26f-154c-487d-a1bd-d4dcc9b301b4.zip
Bundle for reproduction


Shall we have toPicture api?

Is there any way to expose a ui.Picture from a vector graphic so we can draw it to the canvas dynamically.

You can use vg.loadPicture. https://github.com/dnfield/vector_graphics/blob/183bd5c63c9f7e5ba39e318b6709156b5a3b70a3/packages/vector_graphics/lib/src/vector_graphics.dart#L607


add new prefixes for v1.2

New prefixes includes:

id/ > PREFIX_STUDENTID
dt/ > PREFIX_DATE
d/ > PREFIX_DAY
tm/ > PREFIX_TIME
tn/ > PREFIX_TUTORIALNAME
v/ > PREFIX_VENUE
as/ > PREFIX_ASSESSMENTNAME
w/ > PREFIX_WEIGHTAGE
f/ > PREFIX_FULLMARK

Codecov Report

Merging #70 (2ffc714) into master (1672df5) will increase coverage by 0.23%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master      #70      +/-   ##
============================================
+ Coverage     60.01%   60.24%   +0.23%
  Complexity      412      412
============================================
  Files            91       91
  Lines          1533     1542       +9
  Branches        167      167
============================================
+ Hits            920      929       +9
  Misses          577      577
  Partials         36       36

Impacted Files
Coverage Δ

...ain/java/seedu/address/logic/parser/CliSyntax.java
93.33% <100.00%> (+10.00%)
:arrow_up:

Continue to review full report at Codecov.

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


graphic shadow error

I get some unwanted "shadow" when plotting results (2D axisymmetric), eg
fobj = pyansys.read_binary(rstfile)
cpos = fobj.plot_nodal_solution(0, 'x', label='Displacement', off_screen=off_screen,
                         n_colors=9, show_edges=True)

I think, I have a similar issue with the cylinder_demo:
from pyansys import examples
examples.ansys_cylinder_demo()

That's due to the shell elements showing up.  I'll try to reproduce it on my end and see if I can change the transparency of the shell elements.

Thanks. As a quick fix I solved the problem by adding "lighting=False" to plot_nodal_solution(). For 2D models (like my example) I think its better to turn of lighting anyway. It also works for ansys_cylinder_demo(), but I guess in 3D lighting might be something you want in that case.

I investigated this and it has to do with the ordering of the mesh faces generated by ANSYS.  I'd recommend setting lighting=False when this occurs.  There's not much I can do from a developer's standpoint to correct this, but thanks for noting this.


Add test to verify virt-operator release upgrade

What this PR does / why we need it:
This PR adds an important test case that simulates a KubeVirt upgrade scenario performed by upgrading the virt-operator itself.
This, in addition to the existing test that performed the upgrade by patching the KubeVirt CR with the desired release (while maintaining the virt-operator at its current version), covers the two upgrade scenarios documented in the user guide.
The goal is to be able to catch virt-operator bugs the surface only when the operator itself is upgraded, e.g. #6763.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #6785
Special notes for your reviewer:
I've tested this locally to verify that the scenario from #6763 is reproduced and fails the test - it does - the test fails after a reasonable timeout, and the virt-operator logs include the errors reported in #6763.
Here's how to reproduce:

Deploy any commit with the offending bug from #6565

git checkout 99a00e143
make cluster-sync

Execute the test and force to upgrade from v0.46.1:

KUBEVIRT_E2E_FOCUS="by.updating.virt-operator" KUBEVIRT_FUNC_TEST_SUITE_ARGS="--ginkgo.v --previous-release-tag=v0.46.1" make functest

Release note:
NONE

/assign @davidvossel @rmohr

/retest-required

/retest

/retest

/retest

/retest

/retest

/retest

@rmohr did you want to give the lgtm when you're comfortable with it?

I am.
/lgtm

/retest

I've just realized the sig-compute lanes are consistently failing on test_id:5360]should set appropriate IO modes.
In the test artifacts I can see the following error while trying to bring up a virt-launcher pod with a local block volume:
MapVolume.MapBlockVolume failed for volume \"block-disk-for-tests1\" : blkUtil.AttachFileDevice failed. globalMapPath:/var/lib/kubelet/plugins/kubernetes.io~local-volume/volumeDevices/block-disk-for-tests1, podUID: 1cf58746-883a-4f6f-b2f7-049def2c64c2: makeLoopDevice failed for path /var/lib/kubelet/plugins/kubernetes.io~local-volume/volumeDevices/block-disk-for-tests1/1cf58746-883a-4f6f-b2f7-049def2c64c2: losetup -f --show /var/lib/kubelet/plugins/kubernetes.io~local-volume/volumeDevices/block-disk-for-tests1/1cf58746-883a-4f6f-b2f7-049def2c64c2 failed: exit status 1

This test appears to be flaky anyway, but here it consistently fails across the 3 compute lanes, so this might have something to do with the fact that we reinstall the operator itself? No idea what though.
TBH, I was sure this test runs as part of the sig-operator lanes, and it seems that it was unintentionally moved to the sig-compute lanes in #5156. So as a first measure, I'll move it back to sig-operator and see if it stabilizes it.

/test pull-kubevirt-build
/test pull-kubevirt-e2e-k8s-1.22-sig-storage

/test pull-kubevirt-e2e-k8s-1.20-sig-compute-nonroot

/test pull-kubevirt-e2e-k8s-1.20-sig-compute-nonroot

TBH, I was sure this test runs as part of the sig-operator lanes, and it seems that it was unintentionally moved to the sig-compute lanes in #5156. So as a first measure, I'll move it back to sig-operator and see if it stabilizes it.

So, moving it to sig-operator lanes seems to solve the problem (test logs indicate that our testcase of interest is being run and pass).
@rmohr if you're good with this change, would you mind re-lgtm'ing this?

/lgtm

/test pull-kubevirt-e2e-k8s-1.22-sig-storage


screen burning

Hi, I'm using an Arzopa G1 Gaming, is a 144Hz portable gaming monitor, yor black frame insertion works wonders! I really like it, I have set it to 120Hz and fix emulators and games to 60 fps and the blur is gone, but... it automatically adds a image retention of some GUI elements, is there a way to fix this?? this is an IPS monitor.

This is an interaction with LCD voltage inversion algorithm; e.g. black frames assigned a specific voltage polarity, and visible frames being assigned the opposite voltage polarity.
Run long enough, it builds up a static charge on the monitors' pixels, creating the temporary image retention that requires roughly multiple of the length of BFI time in non-BFI mode, to "undo".
Some TestUFO patterns (e.g. www.testufo.com/flicker) fixes this by randomly adding a repeat refresh cycle as a phase-switch frame once every 20 seconds.  This solves the problem.
Odd-number cadences (e.g. 60fps BFI at 180Hz via ON-OFF-OFF or ON-ON-OFF cadence) are completely immune and doesn't need an intermittent phase-switch frame.


Move Skipping Helpline Call Routing instructions to a different section in documentation

I was looking for the instructions to configure Twilio to only do meeting searches and it took me a few minutes to find the instructions under Skipping Helpline Call Routing (https://yap.bmlt.app/helpline/skipping-helpline-call-routing). I was thinking maybe this documentation would be better in a different section because it's really not about Helpline/Volunteer Routing. Maybe it's better under the General section, or the Meeting Search section, or the Miscellaneous section. LOL. Just a thought.
Maybe it could be a 7.a. under Setup in General? Or just a quick note in step 7 in Setup that links to wherever else in the documentation it lives.

It was moved to general, and I added a note under 7 on the setup page.


Affirm Payload with AMS

This PR introduces a collection of ActiveModel Serializers to provide an
object oriented way for users to render the Affirm json payload for a
Spree::Order where needed.
It will also give them the possibility for easy overriding and
applying any needed customization without the need to open up the
classes by simply inherit from the Serializer and apply only the changes
needed.

[x] 100% Spec coverage
[x] line items image urls
[x] discounts object with promotions
[x] metadata rendering
[x]  rebase / squash / split and rewrite commits

Thanks @Senjai, addressed your comments.

Thanks @Senjai!


Add Spigot's TNT Priming Event

See discussion about it here
Deprecation of Paper's event reported by LG_Legacy
Changes:

Removed meta entry from paper event
Restrict paper's event to versions < 1.19
Add Spigot's TNTPrimeEvent and restrict to versions >= 1.19
Change event handler name in new event to follow the onX naming convention

Done :D

Done

Done


BUG - customers email not being added to tracking

Customer email isn't being added to the tracking email field when exported from woocommerce.
Running Version 3.0.6

Hi @Red-Cross-Knight,
Can you check on your site, Woocommerce > Settings > Shipping > Shipping Zones > Shipping Methods
Click Flagship Shipping and there will be a settings popup. Scroll down and check on the box Send tracking updates to billing email. This should include email of customers when you export shipments to Flagship.

We are not using automated quotes because the box sizing doesnt work correctly, and we process dry, live and frozen goods and things never quote properly so we utilize the 'export to flagship' option from the order.
This used to work fine until a few version ago when it just stopped working.
It looks like the plugin is relying on shipping zones/methods to get this setting and its not applying to the 'export to flagship' option.

Hi @Red-Cross-Knight,
Yes, the plugin is relying on the shipping zones/methods settings. Can you confirm that the "Send tracking updates to billing email" is checked?
If it's already been checked and the shipment still does not include the tracking emails of the customer then please send the screenshot of the Order Details to<EMAIL_ADDRESS>

Hi @Red-Cross-Knight,
Yes, the plugin is relying on the shipping zones/methods settings. Can you confirm that the "Send tracking updates to billing email" is checked?
If it's already been checked and the shipment still does not include the tracking emails of the customer then please send the screenshot of the Order Details to<EMAIL_ADDRESS>
Sent an email with more screenshots to help outline the issue.

Hi @Red-Cross-Knight ,
The new version of the plugin is now available on Wordpress. Can you try upload and verify if this issue is fixed ?

Yes it worked, oddly it disabled the plugin and I had to manually reenable it after the upgrade.


IIS scenario: stdout and stderr are not present

dotnet test this

@dotnet-bot test this please

@pan-wang when running on IIS, stdout and stderr are set to -1, which causes _dup to return -1. Calling _dup2 with -1 causes an exception rather than returning -1, so we now explicitly check for it being -1.

when WAS disable stdout and stderr when creating worker process. This is why I said dup2 does not work in the code comments and called another API.

We don't use dup2 to redirect stdout/err to the file/pipe, we use it to store/restore the previous stdout that was set by IIS. Different scenario, but you are correct that it doesn't work as expected.

with your new change, does the logging work?

@shhsu can you verify that locally and see if stdout logs are put in the windows event log if there is an exception?

Pending verification.

I can take over this PR. There are few more issues left over with calling dup2 that I didn't realize. @pan-wang was right about calling dup2 I believe. Will revert appropriately.

Going to revert to original behavior and disable tests for now.

Just to confirm: this is just a manual revert?

Yeah, we still moved the location of output redirection to another file.

@shhsu @pan-wang 🆙 📅

LGTM


Output local_filename is available before the package is available

Description
I was attempting to build the package locally once, and then use it several times in different lambda functions. That ends up looking something like this:
module "test_create_package" {
  source = "git::https://github.com/terraform-aws-modules/terraform-aws-lambda.git?ref=v4.0.2"

  create_function = false
  create_package  = true

  recreate_missing_package = false

  runtime     = "python3.8"
  source_path = "${path.module}/../../lambda/src"
}

module "test_create_function" {
  source = "git::https://github.com/terraform-aws-modules/terraform-aws-lambda.git?ref=v4.0.2"

  function_name = "foo"
  handler       = "index.lambda_handler"
  runtime       = "python3.8"

  create_package = false
  local_existing_package = module.test_create_package.local_filename
}

The problem is that the output local_filename is determined immediately at plan time, but the zip is only created in the apply phase. Which means the terraform graph determines that it is able to use the value immediately in the apply phase to create the lambda function. Which results in errors of the form:
│ Error: unable to load "./builds/4d797a72c1bf2fcf10fcee1ece41ecf3e1ec6c99ea87c4772711bc449c5bad4b.zip": open ./builds/4d797a72c1bf2fcf10fcee1ece41ecf3e1ec6c99ea87c4772711bc449c5bad4b.zip: no such file or directory
│
│   with module.test_create_function.module.lambda.aws_lambda_function.this[0],
│   on .terraform/modules/test_create_function.lambda/main.tf line 22, in resource "aws_lambda_function" "this":
│   22: resource "aws_lambda_function" "this" {

I believe this can be fixed by adding a depends_on statement to the output local_filename for null_resource.archive:
output "local_filename" {
  description = "The filename of zip archive deployed (if deployment was from local)"
  value       = local.filename

  depends_on = [
    null_resource.archive,
  ]
}

I'll be testing that and opening a pull request shortly if it does indeed fix the behavoir.

[x] ✋ I have searched the open/closed issues and my issue is not listed.

Yes, that is also an option, but module-level depends_on does some ugly things to the graph. Forcing the output to wait keeps the graph really clean.

And I did just test my fork with this change and it does indeed fix the problem...

This issue has been resolved in version 4.1.1 :tada:


Add an option to define a global default configuration

This PR adds a new option to define every modules defaults. Particularly useful when you want to completely deactivate the cache and use a custom cacheBuster, e.g:
angular
  .module('app', [
    'oc.lazyLoad',
  ])
  .config([
    '$ocLazyLoadProvider',
    function ($ocLazyLoadProvider) {
      $ocLazyLoadProvider.config({
        moduleDefaults: {
          cache: false
        }
      });
    }
  ]);

Related to #306 and #336

Historically, some loaders weren't using the cacheBuster so I am adding them one by one as I'm needing them. Let me know if I've missed one that's useful to you.

When change is merge with master branch?
I really need it.


Use a newer macosx SDK

What does the pull request do?

What is the current behavior?

What is the updated/expected behavior with this PR?

How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Avaloniaui.net with user documentation

Breaking changes

Fixed issues

We should probably change XCode version instead. Something like
/bin/bash -c sudo xcode-select --switch /Applications/Xcode_10.3.app/Contents/Developer
See
https://github.com/Microsoft/azure-pipelines-image-generation/blob/master/images/macos/macos-10.14-Readme.md#installed-sdks
https://github.com/Microsoft/azure-pipelines-image-generation/issues/886#issuecomment-487101917

We can use a specific version of xCode y. It should work either way.


Add federated auth mode for live testing

API change check
API changes are not detected in this pull request.

/azp run js - appconfiguration - tests


Set Security Profile

Hi all,
I'm using pandevice library.
My idea is the following:

for each rule with service setted to any, I would like to apply a security profile

fw = pandevice.firewall.Firewall(hostname=HOSTNAME, api_key=API_KEY)

rulebase = pandevice.policies.Rulebase()

fw.add(rulebase)

rules = pandevice.policies.SecurityRule.refreshall(rulebase)

for rule in rules:
       if 'any' in rule.service:
                 //set security_profile (like av,URL Filtering Profiles,Vulnerability Protection Profiles,etc...)

       rule.apply()

What is the method that set the security profile to a specific rule?
Thanks in advice

@EdoardoMDP
This is controlled by the following params in pandevice.policies.SecurityRule:
        group (str): Security Profile Group
        virus (str): Antivirus Security Profile
        spyware (str): Anti-Spyware Security Profile
        vulnerability (str): Vulnerability Protection Security Profile
        url_filtering (str): URL Filtering Security Profile
        file_blocking (str): File Blocking Security Profile
        wildfire_analysis (str): Wildfire Analysis Security Profile
        data_filtering (str): Data Filtering Security Profile

@shinmog
Thanks for the answer.
I tried rule.virus but it returns the security profile.
I don’t know the function that allows me to set the profile( something like virus.set(“nane”))

@EdoardoMDP
I might not be understanding your question.
If your question is, "I have a pre-existing Anti-virus security profile that I want to add to my security rule, how do I do that," then the answer is, using your code example:
for rule in rules:
    rule.virus = "my-antivirus-profile"
    rule.apply()

If your question is, "I'd like to configure a brand new anti-virus security profile (or modify an existing one), how do I do that?" then the answer is that modifying those profiles (antivirus, spyware, etc) are not yet supported by pandevice.

Thanks so much.
I need the first one!
@shinmog

Glad to help :)


Call out the different name used for the wrapper

Explicitly call out the different name used for the wrapper .pyx file vs. the other C++ sources
As a new user, I was following along and just called the .pyx file Rectangle.pyx like everything else which was kinda confusing then when in the wrapping_CPlusPlus/setup.py there's a setup(ext_modules=cythonize("rect.pyx")) and I was thinking that maybe that file was magically created somehow and was miffed.
Hoisted the blurb on why the file name needs to be different without some additional work and worded it in there.

Yes, I agree it's good to explicitly call out the name of rect.pyx before it shows up in the setup.py file. Thanks.


CUDA implementations for scatter & gather.

The implementation is very similar to @hughperkins's code in cltorch. Two things are worth pointing out:

I couldn't use the regular TensorInfo object because it flattens dimensions when possible, where as I need to keep the size and stride information as they are. I therefore added a SimpleTensorInfo struct.

I use template specialization for a small number of dimensions. On a simple benchmark (index size = 1000 x 10 x 100 and dim = 2) this reduces the runtime by 33% for gather and 42% for scatter.

thanks dominik!

@dominikgrewe just out of personal curiosity, how do these structs arrive in the kernel in CUDA?  In OpenCL, I've been allocating a memory buffer for them, just like for the underlying data itself, and copy it across, which is quite slow.  On CUDA, do they just get sent with the rest of the kernel parameters all together as a single blob?

@hughperkins they arrive on the GPU effectively as a memcpy of sizeof(args...); the structs are POD and that just works. Though, I wonder what happens for big-endian platforms and little-endian platforms when pointers are built in. Maybe the compiler in the kernel call (as part of parsing <<<>>>) handles converting whatever needs converting when building the blob.


Update workflow.yml

Added Codecov action to workflow

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Colors of budget categories change from nYNAB default to Toolkit colors randomly when clicking in budget

Name of Defective Feature
I'm also having a similar problem, and it seems to be related to nYNAB superseding the Toolkit's color formatting setting "Highlight all Negative Category Balances Red". On the budget screen when I first load nYNAB, the color for all of the budget categories is the default: orange for credit card overspending or for a transaction that will soon take the category balance below zero. However, when I click within some of the budget cells, the red color on negative cells will occasionally appear. Clicking around more will typically lead the cells to randomly change from orange to red, apparently depending on when the toolkit or nYNAB most recently refreshes the colors.
System Settings (Required)
ToolKit Version: 1.3.0
Operating System (Mac/Windows/Etc): macOS Sierra
Browser (Chrome/Firefox/Safari): Chrome
Browser Version: 61.0.3163.100
Other Extensions: Adblock Plus, LastPass, Google Hangouts
Export of Toolkit Settings (Bottom Left of Settings Page)

Settings Export

[{"key":"AccountsDisplayDensity","value":"1"},{"key":"AccountsEmphasizedOutflows","value":false},{"key":"AccountsStripedRows","value":false},{"key":"ActivityTransactionLink","value":true},{"key":"AdditionalColumns","value":true},{"key":"AdjustableColumnWidths","value":false},{"key":"AutoCloseReconcile","value":false},{"key":"BetterScrollbars","value":"0"},{"key":"BudgetBalanceToZero","value":true},{"key":"BudgetQuickSwitch","value":false},{"key":"CategoryActivityCopy","value":false},{"key":"ChangeEnterBehavior","value":true},{"key":"CheckNumbers","value":false},{"key":"ClearSelection","value":false},{"key":"ColourBlindMode","value":false},{"key":"CustomFlagNames","value":false},{"key":"DisplayTargetGoalAmount","value":"0"},{"key":"EditAccountButton","value":"0"},{"key":"GoalWarningColor","value":true},{"key":"HideAgeOfMoney","value":true},{"key":"HideReferralBanner","value":true},{"key":"ImportNotification","value":"0"},{"key":"PrintingImprovements","value":false},{"key":"PrivacyMode","value":"0"},{"key":"RemovePositiveHighlight","value":false},{"key":"RowHeight","value":"0"},{"key":"RowsHeight","value":"0"},{"key":"RunningBalance","value":"1"},{"key":"ShowCategoryBalance","value":false},{"key":"SplitKeyboardShortcut","value":true},{"key":"SquareNegativeMode","value":false},{"key":"StealingFromFuture","value":true},{"key":"TargetBalanceWarning","value":false},{"key":"accountsClearSelection","value":false},{"key":"activityTransactionLink","value":true},{"key":"betterScrollbars","value":"0"},{"key":"budgetBalanceToZero","value":true},{"key":"budgetProgressBars","value":"0"},{"key":"budgetQuickSwitch","value":false},{"key":"budgetRowsHeight","value":"0"},{"key":"calendarFirstDay","value":"0"},{"key":"categoryActivityPopupWidth","value":"0"},{"key":"checkCreditBalances","value":false},{"key":"collapseExpandBudgetGroups","value":false},{"key":"collapseSideMenu","value":false},{"key":"compactIncomeVsExpense","value":true},{"key":"currentMonthIndicator","value":false},{"key":"daysOfBuffering","value":true},{"key":"daysOfBufferingHistoryLookup","value":"0"},{"key":"editButtonPosition","value":"0"},{"key":"enableRetroCalculator","value":false},{"key":"enlargeCategoriesDropdown","value":true},{"key":"goalIndicator","value":false},{"key":"googleFontsSelector","value":"0"},{"key":"hideAccountBalancesType","value":"0"},{"key":"hideHelp","value":true},{"key":"highlightNegativesNegative","value":true},{"key":"importNotification","value":"0"},{"key":"incomeFromLastMonth","value":"0"},{"key":"l10n","value":"0"},{"key":"largerClickableIcons","value":false},{"key":"monthlyNotesPopupWidth","value":"0"},{"key":"navDisplayDensity","value":"1"},{"key":"pacing","value":"0"},{"key":"popupCalculator","value":false},{"key":"privacyMode","value":"0"},{"key":"reconciledTextColor","value":"0"},{"key":"removeZeroCategories","value":true},{"key":"reports","value":true},{"key":"resizeInspector","value":false},{"key":"rightClickToEdit","value":true},{"key":"seamlessBudgetHeader","value":false},{"key":"showIntercom","value":true},{"key":"spareChange","value":false},{"key":"splitKeyboardShortcut","value":true},{"key":"splitTransactionAutoAdjust","value":false},{"key":"swapClearedFlagged","value":false},{"key":"toBeBudgetedWarning","value":true},{"key":"toggleSplits","value":false},{"key":"toggleTransactionFilters","value":"0"},{"key":"warnOnQuickBudget","value":true}]

Steps to Reproduce the Behavior

Open nYNAB. I observe the default orange color on budget categories that have been funded by credit cards.
Click a budget category. About half of the time, the budget items that are negative appropriately turn red, as I would expect based on my toolkit settings.
If I continue to click around, the color may return to orange, and then go back to red, seemingly randomly.

Expected Behavior

I expected to see all negative categories highlighted in red, due to toolkit settings.
Actual Behavior

The negative budget categories sporadically display either red or orange.

Just a quick note -- this is something I'm actively working on to fix/improve. Hopefully coming in the next Toolkit update, or the following depending on when the next update happens.
Thanks for the bug report!

Thanks for the update! I'll keep monitoring nYNAB's behavior and look for changes.

This should be fixed with #1060.

@mirwinator - We released a new version this morning with some fixes. When you get the chance, double check if this is still an issue for you?

@falkencreative - I think everything looks good now. I artificially created some credit card overspending, and the category went red immediately rather than yellow. I also refreshed nYNAB, and the cell was still red prior to clicking anywhere. Thanks for your all's hard work!

Great! Thanks to @Twizzes for the fix.


[perl6.vim] Improper highlighting after lines with odd number of quotes

Vim syntax highlighting is not correct with lines after those containing odd number of quotes (single, in this case)
Here's a link to an example:
https://github.com/masak/yapsi/blob/master/lib/Yapsi.pm#L785
PS: I currently work around this by inserting another single quote on the same line inside a comment, like so # '

This has been fixed.


Add Schema and Transform support for NDArrayWritable

Issue Description
We currently lack an efficient column-based String Transform for converting a text field to bag-of-words (BOWs). We do have StringListToCategoricalSet, but it's not scalable to large vocabularies.
I discussed this with @AlexDBlack, and we decided the principled approach is to add NDArrayWritable as a "type" for columns. This requires support in both Schema and Transform. It will also have implications for analysis and saving to disk. When saving to, e.g., a CSV, we think the right approach is to encode the NDArray object using Base64 and then write the Base64 string into the CSV. That way when the CSV is read from disk, we can properly decode it back into an NDArray.
Once this is added, we can, for example, create a StringToBOWTransform that wraps the BOW Vectorizer, etc.
Version Information
All versions
Contributing
If you'd like to help us fix the issue by contributing some code, but would
like guidance or help in doing so, please mention it!

As discussed, @AlexDBlack

Some initial thoughts

Restrict NDArrays to the same size, and specify this size in the schema
AnalyzeSpark etc - probably just ignore NDArrayWritable columns
Threading: we'll be passing arrays between threads constantly (with something like spark). Need to make sure we don't run into any issues as a result.

Some alternative approaches:

java arrays (int[], float[] etc) Writable objects
a general-purpose array type (could be useful and much more efficient, for example, for bitsets and the like)
record binarization (i.e., binary storage format) https://github.com/deeplearning4j/DataVec/issues/77

Implemented here: https://github.com/deeplearning4j/DataVec/pull/302


New beforeUpload event - makes it possible to validate files before the upload starts

Can be used for validating files sizes - see basic example. Can also be used for more advanced size checking (by file extension).
Calling uploadingFile.setAbort() from the event will abort the upload (xhr.send is not called) and the file is removed from the queue.

@eivindr thanks alot! can you bump version in package.json also please? then I'll merge and publish to npm.

Thanks, published 1.5.4 to npm.

Trying to use this function as per the example I am getting Type 'EventEmitter<{}>' is not assignable to type EventEmitter<uploadedFile>" Type '{}' is not assignable to type 'UploadedFile' Property  'id' is missing in type '{}' during the build process

Going into node_modules/ng2-uploader/src/services/ng2-uploader.ts and changing line 87 from   _beforeEmitter: EventEmitter<UploadedFile> = new EventEmitter();
to
  _beforeEmitter: EventEmitter<any> = new EventEmitter();
Fixes the problem for now.

thanks, https://github.com/jkuri/ng2-uploader/commit/0d4433a174a2e9437351a011a29041c8d9ea76ca and published.


Packwiz curseforge export should not include the packwiz executable in overrides

I think this should be rather obvious why it's a bad idea, considering the fact that curseforge are known to reject uploads that contain executables like this. I feel it is best to be on the safe side here.

It is intended that you copy the binary to your PATH (then you wouldn't have to have it in the same directory as your pack), but this sounds like a good check to have anyway - I'll add it at the same time as fixing #7.

Sweet, thanks for the help! That also explains it. I'm using it through an alias and the variable is in the same folder while I'm testing to get used to stuff.


UUID field and admin error

My models use UUID as primary-key. I used cramer/django-uuidfield, because I got a problem using south, if I use UUID field defined in project test_tools.
If I use django_polymorphic and cramer/django-uuidfield together, I get this error, when I try to show a models list in admin interface:
Reverse for 'base_devices_basedevice_change' with arguments
'(UUID('9639a9909ed24a99ae20f2c0f5c631ba'),)' and keyword arguments '{}' not found.
1 pattern(s) tried: [u'admin/base_devices/basedevice/(\\d+)/$']

I solved it changing in PolymorphicParentModelAdmin class, line 250:
new_change_url = url(r'^(\d+)/$', self.admin_site.admin_view(self.change_view), name='{0}_{1}_change'.format(*info))

to
new_change_url = url(r'^(\w+)/$', self.admin_site.admin_view(self.change_view), name='{0}_{1}_change'.format(*info))

Have somebody a better solution??

Issue can be closed.
Adding pk_regex variable to Admin class fixes the problem with UUID primary key in admin panel.
class AttributeAdmin(PolymorphicParentModelAdmin):
    pk_regex = '([\w-]+)'
    base_model = Attribute

Thanks for providing the solution @zenzire, closing the issue.


fix(tests): make tests run locally (take 2)

Signed-off-by: Luca Guerra<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

If contributing rules or changes to rules, please make sure to also uncomment one of the following line:

/kind rule-update

/kind rule-create

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area engine

/area rules

/area tests

/area proposals

What this PR does / why we need it:
In an attempt to make it easier for people to run tests locally I messed up the bash script which currently does not report failed tests correctly. This reverts the change and adds correct instructions to the README about how to run it. As it turns out, I was missing an option or badly specified it before.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

@LucaGuerra: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: LucaGuerra
To complete the pull request process, please assign fededp after the PR has been reviewed.
You can assign the PR to them by writing /assign @fededp in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

/test tests-integration

/ok-to-test

Closing and reopening to trigger the CI
/close

@jasondellaluce: Closed this PR.

In response to this:

Closing and reopening to trigger the CI
/close

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

/reopen

@jasondellaluce: Reopened this PR.

In response to this:

/reopen

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

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

LGTM label has been added.  Git tree hash: e0998712ff02b81aff59f6bc96b99fd4d03a193f

/milestone 0.32.1

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, jasondellaluce, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,jasondellaluce]

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

/close
Closing and reopening to trigger the CI

/reopen

@leogr: Reopened this PR.

In response to this:

/reopen

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


Selecting items is super slow

v2020.10.27
bl 2.83
Was testing you version again that of sambler and noticed you added more functionality. The interface is nice but perhaps you should change it a bit.
The main issue is that it needs a refresh and then when using the select button its also slowing down. I did not check what is happening in the background and why it does that. My guess you looping over the objects once more.
EDIT
Did a quick check, bsically every button runs the same action as refresh. i think you should change that and store these settings into somekind of format. That info would be easier to retrieve and the addon would respond in milliseconds.

PS why are you redrawing meshes actually?
in the function calculate_mesh_polycount, i see you using bmesh the apply all mods get the data and then free the bmesh, thats kinda tedious. Why not use it how sambler does it, that one works instantly. Ow i see he does same thing, but only when its using MODS ON.

Thanks for your feedback. Yea I think many optimisations can be done, I learn Blender Python on the fly so this leads to this kind of weird logic :)
I can indeed add an option to enable/disable modifiers in calculation #14
About the selection I think it will take more time to handle this (as you understood, I'm not a hardcore dev ^^), feel free to suggest a pull request if you want, probably relative to #15


WeaviateMemoryStore connector

Motivation and Context

Description

This adds a WeaviateMemoryStore, which allows for memories to be stored in Weaviate.
I have some outstanding questions, please help!

Weaviate has strict naming conventions around class names (i.e. collections). What are the conventions in SK for class names?
What are the most appropriate log levels for the connector? Please can you verify these are as expected?
I have a suite of integration tests that can run against a Weaviate instance. I have implemented these integration tests before, in another project. Any guidance on how these should run for this repo? Ultimately it requires Weaviate running in a docker container.

Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows SK Contribution Guidelines (https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
[x] The code follows the .NET coding conventions (https://learn.microsoft.com/dotnet/csharp/fundamentals/coding-style/coding-conventions) verified with dotnet format
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

@microsoft-github-policy-service agree

I have rebased again. Would it be possible to merge this sooner, rather than later, if possible?
Thanks.

I have rebased again. Would it be possible to merge this sooner, rather than later, if possible?
Thanks.

Thanks for your patience. Will do our best to get this in this week. I can't see the conflicts right now to know what's remaining.

5th rebase (!)

5th rebase (!)

@codebrain -- It might be worth re-creating the PR with the checkbox of Enable edits by maintainers so we can rebase/merge main as appropriate.


Timesheet - 14/08/1996 - James Pang

Task 1
Reviewed Sanjeev's PR on project details, added some fixes to breadcrumb
Proof of Work

https://github.com/thenewboston-developers/Website/pull/2017#pullrequestreview-730040340

Time Spent
3 hours

Task 2
Added fix for projects sidemenu
Proof of Work

https://github.com/thenewboston-developers/Website/pull/2011

Time Spent
.5 hours

Total Time Spent
3.5 hours
Account Number
3443227b5f81d2e2e595c52fe0fafce0870c4c4d837a550ef084d44cb95aa4be

@jamessspanggg, The year appears to be wrong on this one.

@jamessspanggg, The year appears to be wrong on this one.

opps my bad, changed


Finish Terminal Help

Need to finish fleshing out the help in the terminal node.

Lots of print statements, but accepting at this time.


New version: Streetwriters.Notesnook version 2.6.15

Pull request has been created with komac v2.0.2 :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

(Automated response - build 684.)

For moderators
This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping @Exorcism0666, I'll get a notification.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！


binder - baltrad

Several issues here:

~baltrad commandline tools do not work in the environment~
baltrad2wradlib uses quite outdated/deprecated wradlib functionality
install problems for several baltrad components

[x] hlhdf
[x] rave
[x] beamb
[x] bropo
[x] wrwp
[x] drqc
[ ] rave_gmap

@DanielMichelson should I make the changes directly at the source repo? https://github.com/DanielMichelson/baltrad2wradlib.git

beamb is failing to build with:
configure: creating ./config.status
config.status: creating def.mk
mv: relocation error: /lib/x86_64-linux-gnu/libacl.so.1: symbol getxattr version ATTR_1.0 not defined in file libattr.so.1 with link time reference
config.status: error: could not create def.mk

FYI:
It looks like beamb is the only failing part of baltrad. But we can't be sure, since the binder build logs are not persistent (at least I do not know how to get them). I'm giving up for now.

Maybe we can export the build logs in the respective calling shell scripts?

Yeah - we need to find a better way to cache those...

This is running on pangeo binder. AFAIK that is some Debian based docker container. @scollis is that correct? They use repo2docker to build the image.
I'm not sure if the error is connected with conda? Or the Python version? (3.9). We might need to think if we want to use conda compilers.

So I can see we're on Ubuntu 18.04 and Python 3.9, but I don't know much else yet. The build for the course in Oz 2019 used conda compilers, I think. Also, I need to ask Anders about BALTRAD and Python 3.9. I doubt there are issues, but need it confirmed.

Does baltrad require CYLP be installed?

I think cylp was for pyart. But let's check this anyway

I don't recognize a CYLP dependency for BALTRAD. Can't find one in the BALTRAD build in https://github.com/openradar/AusOpenRadar2019/

@DanielMichelson @kmuehlbauer - is there a good way to test if  beamb is installed?

It's important that the first two BALTRAD modules are installed in the order listed in the postBuild script. I can see that HL-HDF is installed. Aside from running the unit tests that come together with each package, you can do simple imports to verify that the packages have installed. I can see that RAVE is not successfully installed yet, so none of the following packages will be either until RAVE is sorted out.
With this in mind, I think trying to find and save the build log will be valuable to troubleshoot.

import _pyhl
import _rave
import _beamblockage
import _ropogenerator
import _wrwp
import GmapCreator
import _dr_qc

@DanielMichelson thanks for the additional information! I am about to sign off for the weekend... I can take another look at this early next week to see if we can troubleshoot/get things running.

I've tinkered a bit with it over the weekend on my fork https://github.com/kmuehlbauer/erad2022/tree/main. It looks like there is some progress.
The good news is, that it is now possible to compile baltrad on the already running binder without problems. Currently trying to get behind the issues which prevent baltrad building on binder image creation. I'll clean-up and merge this if this is resolved.

With #19 baltrad is correctly installed into the environment (to be checked after build).

The gmap extension does not correctly install, but might be resolved with #15.

FYI: The baltrad install logs are available from the running binder in the ~/tmp/ folder.

DRQC: should be fixed by #20

@DanielMichelson FYI: I've made the baltrad2wradlib notebook work with the current binder state. I'll fix this at your fork.

Amazing work @kmuehlbauer ! I'm always happy and grateful to see someone else able to make sense out of BALTRAD.
Funny that you are devoting your energy to this now, when I was off celebrating Canada Day on Friday and @mgrover1 is off today for the 4th of July.
I will try to take advantage of the time difference and look more closely later today.

+1 😊 Looking forward to trying all this out  From: Daniel Michelson<EMAIL_ADDRESS>Monday, July 4, 2022 at 9:37 AMTo: openradar/erad2022<EMAIL_ADDRESS>Scott<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Re: [openradar/erad2022] binder - baltrad (Issue #10)Amazing work @kmuehlbauer ! I'm always happy and grateful to see someone else able to make sense out of BALTRAD.Funny that you are devoting your energy to this now, when I was off celebrating Canada Day on Friday and @mgrover1 is off today for the 4th of July.I will try to take advantage of the time difference and look more closely later today.—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>
Update: made some progress setting myself up. Forked the project and wrote a trivial Issue to confirm that I am functional again.
rave_gmap is not installing correctly due to path confusion. I will try to find a fix next.
Build logs are very spewy, but it's great that they're there! Like, why does the relevant part of the rave_gmap build start on line 3109 of the log?

The logs contain every output which the install_baltrad_rave_gmap.sh script emits. The first couple thousand lines is the output of the conda activate notebook.

@DanielMichelson How can we check if rave_gmap is correctly working? I've attached the according lines of the build log below. Can we see the mentioned path confusion there? AFAICT the error in the logs is due to the fact that the setup.py was tampered with to copy a certain Lib.pth to the relevant python folder which isn't available for some reason.
But that should be corrected by this line of code after the installation, at least that is my assesment of the situation:
https://github.com/openradar/erad2022/blob/3d76d8433c3533aaf226498c85cbb8d4d342bb9c/binder/install_baltrad_rave_gmap.sh#L32
Please note that the apache2 isn't installed and activated as in the older vagrant based courses. Maybe that's another issue.
+ export LD_LIBRARY_PATH=/srv/conda/envs/notebook/lib:/srv/conda/envs/notebook/hlhdf/lib:/srv/conda/envs/notebook/rave/lib
+ LD_LIBRARY_PATH=/srv/conda/envs/notebook/lib:/srv/conda/envs/notebook/hlhdf/lib:/srv/conda/envs/notebook/rave/lib
+ cd /home/jovyan/tmp
+ git clone --depth=1 https://github.com/baltrad/GoogleMapsPlugin.git
Cloning into 'GoogleMapsPlugin'...
+ cd GoogleMapsPlugin/
+ python setup.py install --prefix=/srv/conda/envs/notebook
WARNING: '' not a valid package name; please use only .-separated package names in setup.py
running install
/srv/conda/envs/notebook/lib/python3.9/site-packages/setuptools/command/install.py:34: SetuptoolsDeprecationWarning: setup.py install is deprecated. Use build and pip and other standards-based tools.
  warnings.warn(
/srv/conda/envs/notebook/lib/python3.9/site-packages/setuptools/command/easy_install.py:144: EasyInstallDeprecationWarning: easy_install command is deprecated. Use build and pip and other standards-based tools.
  warnings.warn(
running bdist_egg
running egg_info
creating src/RAVE_Google_Maps_Plugin.egg-info
writing src/RAVE_Google_Maps_Plugin.egg-info/PKG-INFO
writing dependency_links to src/RAVE_Google_Maps_Plugin.egg-info/dependency_links.txt
writing top-level names to src/RAVE_Google_Maps_Plugin.egg-info/top_level.txt
writing manifest file 'src/RAVE_Google_Maps_Plugin.egg-info/SOURCES.txt'
reading manifest file 'src/RAVE_Google_Maps_Plugin.egg-info/SOURCES.txt'
adding license file 'LICENSE'
adding license file 'COPYING'
adding license file 'COPYING.LESSER'
writing manifest file 'src/RAVE_Google_Maps_Plugin.egg-info/SOURCES.txt'
installing library code to build/bdist.linux-x86_64/egg
running install_lib
running build_py
creating build
creating build/lib
copying src/GmapLegend.py -> build/lib
copying src/GmapCreator.py -> build/lib
copying src/GmapColorMap.py -> build/lib
copying src/GmapLayerSettings.py -> build/lib
copying src/GenerateProductInfo.py -> build/lib
copying src/googlemap_pgf_plugin.py -> build/lib
creating build/bdist.linux-x86_64
creating build/bdist.linux-x86_64/egg
copying build/lib/GmapLegend.py -> build/bdist.linux-x86_64/egg
copying build/lib/GmapCreator.py -> build/bdist.linux-x86_64/egg
copying build/lib/GmapColorMap.py -> build/bdist.linux-x86_64/egg
copying build/lib/GmapLayerSettings.py -> build/bdist.linux-x86_64/egg
copying build/lib/GenerateProductInfo.py -> build/bdist.linux-x86_64/egg
copying build/lib/googlemap_pgf_plugin.py -> build/bdist.linux-x86_64/egg
byte-compiling build/bdist.linux-x86_64/egg/GmapLegend.py to GmapLegend.cpython-39.pyc
byte-compiling build/bdist.linux-x86_64/egg/GmapCreator.py to GmapCreator.cpython-39.pyc
byte-compiling build/bdist.linux-x86_64/egg/GmapColorMap.py to GmapColorMap.cpython-39.pyc
byte-compiling build/bdist.linux-x86_64/egg/GmapLayerSettings.py to GmapLayerSettings.cpython-39.pyc
byte-compiling build/bdist.linux-x86_64/egg/GenerateProductInfo.py to GenerateProductInfo.cpython-39.pyc
byte-compiling build/bdist.linux-x86_64/egg/googlemap_pgf_plugin.py to googlemap_pgf_plugin.cpython-39.pyc
installing package data to build/bdist.linux-x86_64/egg
running install_data
creating /srv/conda/envs/notebook/rave_gmap
creating /srv/conda/envs/notebook/rave_gmap/web
creating /srv/conda/envs/notebook/rave_gmap/web/css
copying web/css/gmlocalsearch.css -> /srv/conda/envs/notebook/rave_gmap/web/css
copying web/css/gsearch.css -> /srv/conda/envs/notebook/rave_gmap/web/css
copying web/css/lmaps.css -> /srv/conda/envs/notebook/rave_gmap/web/css
copying web/css/radar.css -> /srv/conda/envs/notebook/rave_gmap/web/css
creating /srv/conda/envs/notebook/rave_gmap/web/img
copying web/img/scl.png -> /srv/conda/envs/notebook/rave_gmap/web/img
copying web/img/logo-small-bourky.png -> /srv/conda/envs/notebook/rave_gmap/web/img
copying web/img/BALTRAD-logo-small128.png -> /srv/conda/envs/notebook/rave_gmap/web/img
copying web/img/BALTRAD-logo-small64.png -> /srv/conda/envs/notebook/rave_gmap/web/img
copying web/img/BALTRAD-logo-small.png -> /srv/conda/envs/notebook/rave_gmap/web/img
copying web/img/legendhmc.png -> /srv/conda/envs/notebook/rave_gmap/web/img
copying web/img/logo-small-chmu.png -> /srv/conda/envs/notebook/rave_gmap/web/img
copying web/img/calendar_icon.jpg -> /srv/conda/envs/notebook/rave_gmap/web/img
copying web/img/Radar2.gif -> /srv/conda/envs/notebook/rave_gmap/web/img
copying web/img/favicon.ico -> /srv/conda/envs/notebook/rave_gmap/web/img
creating /srv/conda/envs/notebook/rave_gmap/web/js
copying web/js/datetimepicker_css.js -> /srv/conda/envs/notebook/rave_gmap/web/js
copying web/js/lmaps_init.js -> /srv/conda/envs/notebook/rave_gmap/web/js
copying web/js/lmaps.js -> /srv/conda/envs/notebook/rave_gmap/web/js
copying web/js/gmap_overlay.js -> /srv/conda/envs/notebook/rave_gmap/web/js
copying web/js/radar.js -> /srv/conda/envs/notebook/rave_gmap/web/js
copying web/js/gmap_dragzoom.js -> /srv/conda/envs/notebook/rave_gmap/web/js
copying web/js/init.js -> /srv/conda/envs/notebook/rave_gmap/web/js
copying web/list_data.php -> /srv/conda/envs/notebook/rave_gmap/web
copying web/index.php -> /srv/conda/envs/notebook/rave_gmap/web
copying web/index.html -> /srv/conda/envs/notebook/rave_gmap/web
copying web/smhi-areas.xml -> /srv/conda/envs/notebook/rave_gmap/web
creating /srv/conda/envs/notebook/rave_gmap/Lib
copying src/CALIBRI.TTF -> /srv/conda/envs/notebook/rave_gmap/Lib
copying src/gmap_layer_settings.xml -> /srv/conda/envs/notebook/rave_gmap/Lib
copying README -> /srv/conda/envs/notebook/rave_gmap
copying README.ravepgf -> /srv/conda/envs/notebook/rave_gmap
copying README2 -> /srv/conda/envs/notebook/rave_gmap
copying LICENSE -> /srv/conda/envs/notebook/rave_gmap
copying COPYING -> /srv/conda/envs/notebook/rave_gmap
copying COPYING.LESSER -> /srv/conda/envs/notebook/rave_gmap
creating build/bdist.linux-x86_64/egg/EGG-INFO
copying src/RAVE_Google_Maps_Plugin.egg-info/PKG-INFO -> build/bdist.linux-x86_64/egg/EGG-INFO
copying src/RAVE_Google_Maps_Plugin.egg-info/SOURCES.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying src/RAVE_Google_Maps_Plugin.egg-info/dependency_links.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying src/RAVE_Google_Maps_Plugin.egg-info/top_level.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
zip_safe flag not set; analyzing archive contents...
__pycache__.GmapLayerSettings.cpython-39: module references __file__
__pycache__.GmapLegend.cpython-39: module references __file__
creating dist
creating 'dist/RAVE_Google_Maps_Plugin-0.1-py3.9.egg' and adding 'build/bdist.linux-x86_64/egg' to it
removing 'build/bdist.linux-x86_64/egg' (and everything under it)
Processing RAVE_Google_Maps_Plugin-0.1-py3.9.egg
creating /srv/conda/envs/notebook/lib/python3.9/site-packages/RAVE_Google_Maps_Plugin-0.1-py3.9.egg
Extracting RAVE_Google_Maps_Plugin-0.1-py3.9.egg to /srv/conda/envs/notebook/lib/python3.9/site-packages
Adding RAVE-Google-Maps-Plugin 0.1 to easy-install.pth file

Installed /srv/conda/envs/notebook/lib/python3.9/site-packages/RAVE_Google_Maps_Plugin-0.1-py3.9.egg
Processing dependencies for RAVE-Google-Maps-Plugin==0.1
Finished processing dependencies for RAVE-Google-Maps-Plugin==0.1
Traceback (most recent call last):
  File "/home/jovyan/tmp/GoogleMapsPlugin/setup.py", line 93, in <module>
    os.rename(source, dest)  # doesn't work in some environments
FileNotFoundError: [Errno 2] No such file or directory: '/srv/conda/envs/notebook/rave_gmap/Lib.pth' -> '/srv/conda/envs/notebook/lib/python3.9/site-packages/rave_gmap.pth'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/jovyan/tmp/GoogleMapsPlugin/setup.py", line 96, in <module>
    shutil.copyfile(source, dest)
  File "/srv/conda/envs/notebook/lib/python3.9/shutil.py", line 264, in copyfile
    with open(src, 'rb') as fsrc:
FileNotFoundError: [Errno 2] No such file or directory: '/srv/conda/envs/notebook/rave_gmap/Lib.pth'
Objects created: 0
Objects deleted: 0
Objects pending: 0
+ cp web/index.html /srv/conda/envs/notebook/rave_gmap/web/.
+ rm /srv/conda/envs/notebook/rave_gmap/web/index.php
+ echo /srv/conda/envs/notebook/rave_gmap/Lib/
+ cp /home/jovyan/binder/baltrad/rave_gmap/web/smhi-areas.xml /srv/conda/envs/notebook/rave_gmap/web/.
+ cp /home/jovyan/binder/baltrad/rave_gmap/web/products.js /srv/conda/envs/notebook/rave_gmap/web/.
+ mkdir /srv/conda/envs/notebook/rave_gmap/web/data
+ cp /home/jovyan/binder/baltrad/rave_gmap/web/data/cawkr_gmaps.tgz /srv/conda/envs/notebook/rave_gmap/web/data/.
+ cd /srv/conda/envs/notebook/rave_gmap/web/data
+ tar xzf cawkr_gmaps.tgz
+ rm cawkr_gmaps.tgz

Update: I figured it out, but have not committed a solution just yet. Problem is that distutils has been deprecated and one of the arguments that GoogleMapsPlugin uses in its setup.py file ("extra_path") was removed in Python 3.8. The setup.py file was admittedly a hack anyway, probably my fault originally, but the latest developments made it unusable. I now have a setup.py file that builds successfully, but it needs to be paired with the install_baltrad_rave_gmap.sh script. This solution will need to be vetted by Anders for BALTRAD in general, but I think it will do the job for ERAD 2022. Hope to get it committed later this week.

The overall baltrad build process should be fixed by #22 (with exception of gmap plugin).

Finally fixed by #25


Filter postgis function on connect

I've got a database that has postgis enabled and connecting takes seconds in-which massive is proxying the 1000+ postgis functions.
It would be great if setting these functions up was avoided either by convention or as a connect option. Actually an option to skip the loadFunctions all together would be handy in some cases.
I managed to reduce the functions.sql queries results from 1081 rows (4500ms) to 43 rows (185ms). I've added what I was playing with in-case its handy?

Ahh... You can do this, just haven't updated the README yet. My bad.
You can actually filter functions, schemas, and tables (blacklist, whitelist, etc) during the call to connect. For some trivial examples, see the tests in the file load_filter_spec.js.
You can exclude all functions, much as you suggest, like so (from the tests):
+  it('excludes functions at load whenever it is told...', function (done) {
+    massive.connect({connectionString: constr, excludeFunctions: true}, function (err, db) {
+      assert(db.functions.length == 0);
+      done();
+    });
+  });

Or you can us a function blacklist filter (in much the same way you are above) by passing either a comma-delimited string, or an array of strings, and you can use the same "wildcard" syntax as in your code above (here, again, some simple examples from the tests, but you get the idea):
+  it('excludes functions with name and schema matching multiiple blacklist patterns as a comma-delimited string argument', function (done) {
+    massive.connect({connectionString: constr, functionBlacklist: "myschema.artist_%, all_%"}, function (err, db) {
+      assert(!db.myschema.artist_by_name && !db.all_products && db.myschema.all_albums && db.AllMyProducts);
+      done();
+    });
+  });
+  it('excludes functions with name and schema matching multiiple blacklist patterns as a string array argument', function (done) {
+    massive.connect({connectionString: constr, functionBlacklist: ["myschema.artist_%", "all_%"]}, function (err, db) {
+      assert(!db.myschema.artist_by_name && !db.all_products && db.myschema.all_albums && db.AllMyProducts);
+      done();
+    });
+  });

Looks like you could take all of the pattern matches from your SQL above, and set them up in an array to pass to the connect() method similar to the test examples.
I think you may be right though, I may "steal" your pattern matches, and set up another simple switch which enables opt-in for including this specific set of functions for PostGis.
I'll also update the README about the various filtering options available for connect().
In the meantime, take a look at the test file and see how it works for you. Please let us know if you run into problems. The filtering use-cases have been tested in basic form, but have not been exercises extensively in a "real-world" setting.

Thats exactly what I was after thanks.
Feel free to close this or if you are going to auto filter postgis functions then keep it open as a reminder?

I was leaving it open as a reminder. Glad it worked for you!


Unauthorized for url: https://api.github.com/markd own/raw

 aemonge   ~/u/d/cells-hybrid-solution    master  grip pub-sub.propursal.md --export --user aem
onge --pass '*******************'
Exporting to pub-sub.propursal.html
[2018-07-11 11:37:07,038] ERROR in app: Exception on / [GET]
Traceback (most recent call last):
  File "/usr/lib/python3.6/site-packages/flask/app.py", line 1982, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/lib/python3.6/site-packages/flask/app.py", line 1614, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/lib/python3.6/site-packages/flask/app.py", line 1517, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/lib/python3.6/site-packages/flask/_compat.py", line 33, in reraise
    raise value
  File "/usr/lib/python3.6/site-packages/flask/app.py", line 1612, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/lib/python3.6/site-packages/flask/app.py", line 1598, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/usr/lib/python3.6/site-packages/grip/app.py", line 174, in _render_page
    content = self.renderer.render(text, self.auth)
  File "/usr/lib/python3.6/site-packages/grip/renderers.py", line 78, in render
    r.raise_for_status()
  File "/usr/lib/python3.6/site-packages/requests/models.py", line 939, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 401 Client Error: Unauthorized for url: https://api.github.com/markd
own/raw

versions
 aemonge   ~/u/d/cells-hybrid-solution    master  node --version ; npm --version ; grip --versi
on
v8.9.4
6.0.1
Grip 4.4.0

In case you have 2FA enabled, use personal access token as password.
https://github.com/joeyespo/grip/issues/35#issuecomment-191944696


Formatting Issue on Fragments Page

On Fragments Page, enlist in Advanced Guides in React Docs, (ref. https://reactjs.org/docs/fragments.html) there's a formatting issue as highlighted in the attached image screenshot.

Seems intentional but I agree it looks silly in a sentence. Please send a PR with your suggestion?

Yes sure. I will send a PR.

@gaearon Please review the PR.

Seems like the text has been changed and the issue is no longer reproducible. Sorry for the late turnaround time.
Thanks 😄


Fix anonymous login

Passing the the mechanism ANONYMOUS without body to the Stanza.auth method would result in an invalid :xmlel element, because the children element would be a list of a empty list, which would result in error when converting to xml string.

Coverage increased (+0.6%) to 74.754% when pulling ea09f988d413b15c23b1d0bfd7b1ce592d5f41ee on picaoao:fix-anonymous-login into 0ecef1e1d6e11f0a47a688b29e8c13c39aa6018f on scrogson:master.


{optimize  = true} doesn't work with closures (even without updating variables)?

I see that there's some new documentation re optimized derivatives in the readme, but I don't see how it applies to examples like this one:
local d = require "autograd"

function test()
    local net = nn.Sequential()
        :add(nn.Linear(2, 1))
        :add(nn.Sigmoid())

    local netFn, netParams = d.functionalize(net)

    local f = function(x)
        return torch.sum(netFn(netParams, x))
    end

    --WORKS WITH NO OPTIMIZATION
    local x = torch.rand(1, 2)
    local df = d.grad(f)
    local dx = df(x)

    --WORKS WITH OPTIMIZATION AND NO CLOSURE
    local g = function(x, ps)
        return torch.sum(netFn(ps, x))
    end
    local dg = d.grad(g)
    local dx = dg(x, netParams)

    --FAILS WITH OPITMIZATION AND CLOSURE
    local dfOptimized = d.grad(f, {optimize = true})
    local dxOptimized = dfOptimized(x)
end

With the closure, the optimized version gives this error and generated code:
return function(locals, rlocals, vlocals, objects)
local nn = require('autograd').nn
local util = require('autograd.util')
local util_fillInPlace = util.fillInPlace
local torch_sum = torch.sum
local model = model
return function(p1)
    locals[1] = objects[1].forward({ 0.4047  0.6308
[torch.DoubleTensor of size 1x2]
, 0.01 *
-9.2744
[torch.DoubleTensor of size 1]
}, p1)
    locals[2] = objects[1].backward((util_fillInPlace(rlocals[1], locals[1], 1)), { 0.4047  0.6308
[torch.DoubleTensor of size 1x2]
, 0.01 *
-9.2744
[torch.DoubleTensor of size 1]
}, p1)
    locals[3] = torch_sum(locals[1])
    return locals[2][2], locals[3]
end
end

[string "return function(locals, rlocals, vlocals, obj..."]:8: '}' expected near '0.6308'

Not sure if this is an inherent issue with the optimized version of  grad, or if there is a way around it, but thought I would check. Thanks!

With optimized mode, closures are "frozen" after the first time you run the function. Behind the scenes, we cache the function so that we don't have to recompile it again. This has the unfortunate side-effect of ignoring all changes to the closed-over variable in the future, which in this case includes the parameters of your neural net.
In general, in optimized mode

Do NOT use closures, they aren't safe.
Do NOT use variables for control flow (if-statements, for-loops) that you did not DIRECTLY pass in.

Thanks for your quick response! I understand the freezing issue, but I wasn’t sure why it would cause problems cases like this. Given that I’m taking the gradient wrt the network’s input, rather than it’s parameters, it seems like it should be ok for the parameters to be frozen at the initial values they get assigned?
One other note: no idea if there’s any possibility for second-order gradients to be supported in non-optimized mode, but this is definitely a feature I would find useful.

Don't you want to update the parameters?
Yeah, the cool features of autograd are unfortunately split non-uniformly
between direct and optimized mode. I'm working to fix this, but I do not
have an ETA.
On Mon, Jun 6, 2016 at 10:34 AM oal17<EMAIL_ADDRESS>wrote:

Thanks for your quick response! I understand the freezing issue, but I
wasn’t sure why it would cause problems cases like this. Given that I’m
taking the gradient wrt the network’s input, rather than it’s parameters,
it seems like it should be ok for the parameters to be frozen at the
initial values they get assigned?
One other note: no idea if there’s any possibility for second-order
gradients to be supported in non-optimized mode, but this is definitely a
feature I would find useful.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/twitter/torch-autograd/issues/129#issuecomment-223977267,
or mute the thread
https://github.com/notifications/unsubscribe/AAJ4j_8ktWTsy46eD5W0U2rzpsPrh1cBks5qJC_hgaJpZM4IuZoS
.

That sounds right - the numbers in the generated code are indeed the same as the parameter tensor.

Ok, cool. So, pass the parameters in as an argument, and you should be good to go.


Check "Always Output Default Values" for Repeated Fields in JSON Output

This is what we believe to be a bug in how the JSON is generated. It seems as though this print option is not respected for repeated fields. Let us know if we are mistaken!

@stepancheg anything you'd like me to update here? thanks!


marginBottom and marginTop with percentage cutting children

[x] I have reviewed the documentation
[x] I have searched existing issues
[x] I am using the latest React Native version

[iOS] When you use percentage in marginBottom or marginTop it kind cuts the children inside.
Tested with Text, and Views component
Devices:  [Simulator iPhone 5s, Simulator iPhone 7, Simulator iPhone X, Real device iPhone 6s]
[Android] Sometimes happens to the views only.
Devices: Samsung S5 mini
Sometimes even without "flex:1" property the bug still occurs. There should be a group of properties that activate this bug or not.
On React-Native V0.53.3 this bug doesn't happen.
Environment
Environment:
OS: macOS High Sierra 10.13.4
Node: 9.8.0
Yarn: 1.6.0
npm: 6.0.0
Watchman: 4.9.0
Xcode: Xcode 9.3 Build version 9E145
Android Studio: 3.1 AI-173.4720617
Packages:
react:  16.3.1
react-native:  0.55.3
Steps to Reproduce
https://snack.expo.io/rJtl8WR6G
      <View style={{ flex: 1, justifyContent:'center',alignItems:'center'}}>
   		<View style={{ marginTop: "8%"}}>
			<Text style={{ fontSize: 40, fontWeight: '700' }}>ASdsad</Text>
		</View>
      </View>

Expected Behavior
To not cut the children!
Actual Behavior
Cutting the Children!

@hramos Please Evaluate the issue.
I believe this is a real issue.

@react-native-bot open

@FlaviooLima
Yes I am also using percentages and still have an issue.
The issue started once I upgraded to React native/Expo sdk 26.
Before that, everything worked normally.

@nikolal I use react-native only and this start happening after I upgraded to v54 and v55. v53 no bugs there.
it's a shame that this is happening, and I don't understand why people are not complaining more.
I believe this is a HUGE bug.

@FlaviooLima Not only margin, but also there are problems with padding and text. I get text truncated, images that doesn't have the same size as in 0.53. Padding that sometimes works as before, sometimes not.

Happens to me with RN 0.55.4 as well. This issue should be reopened.

I also noticed this after an upgrade and while preparing a test repo for #22039.
Not sure how the bot identified this as a question rather than a bug, but that should also be addressed.

My temporary solution is to convert % to px for each component and passing it down the tree. I am building dynamic app where users are choosing width and height so I don't have control of width and height  and users must use % values. I am not sure if it will help you.
It goes something like this:
`
getDimensions = (layout) => {
const { width, height } = layout;
this.setState({ width: width });
this.setState({ height: height });
}
<View
style={this.styles.container}
onLayout={(event) => { this.getDimensions(event.nativeEvent.layout); }}

Later I calculate height based on those values. In example below, height and width are % value for current component, while parentHeight and parentWidth are values that i am passing from parents ( calculated above ).
height: parentHeight ? parseFloat(height) / 100 * parentHeight : height,
width: parentWidth ? parseFloat(width) / 100 * parentWidth : width,
`
I am rendering components dynamically using recursive loop so I auto pass these values from parents to children.
Probably not useful to manby people if you have to do this in each component.
This issue should really be fixed, especially since we know exactly when it started.

This is the same issue:  https://github.com/facebook/react-native/issues/18911

Still and issue and nothing has been done about it. All worked well before. I can confirm it still happens on 0.57. The only solution I found was to add the margin to the first element within the wrapping view. So for example:
<View style={{marginTop:'5%'}}>
    <Text>Bla</Text>
</View>
// now needs to be:
<View>
    <Text style={{marginTop:'5%'}}>Bla</Text>
</View>

However if you try to inspect the elements you still get the error:
JSON value '5%' of type NSString cannot be converted to NSNumber.
This issue should be re-opened!


added stanza for comparison (POS-tagging)

Similarly, UDpipe seems to also be lacking here

Hi @KennethEnevoldsen ,
Thanks for pointing towards the stanza models.
However, for consistency, we want to benchmark the models in the same script as the rest of the performance we report in https://github.com/alexandrainst/danlp/tree/master/examples/benchmarks
So the stanza models should be added to the banchmark sctipt first. I can open an issue with it, on less you want to add it to this pull request?

Hmm will not have the times as it stands, but feel free to create an issue. I might get to it when I have the chance

Link to new issue #93


add gws tooltips

Coverage remained the same at 82.244% when pulling de6103a7d46bbc7ba3b645cd2960d9c296a8009f on sunny4381:add-tooltips into 7b69f8dfb3cd0c0a620a46fba44d7a9a269bb2b7 on shirasagi:master.

Coverage increased (+0.003%) to 82.248% when pulling f7b17f85d63253e567802c2dfe8a2f00bf64012e on sunny4381:add-tooltips into 7b69f8dfb3cd0c0a620a46fba44d7a9a269bb2b7 on shirasagi:master.


How to pass this file value in file param UploadMedia action method?

In Waba360DialogController, UploadMedia Action method how to pass file param value, Is it base64 value or image binary in string format.
{
"file": "string", - How to pass this file value?
"contentType": "string"
}
Thanks in advance.

I mistakenly used [FromBody] for that API End, it should use [FromForm] instead.
Just pushed a fix with some for that.
Thanks!

Hi @sleekflowalexyuen https://github.com/sleekflowalexyuen
Thanks for the quick response.
I'll check the fix and close the issue as well.
Meanwhile, Are you working or any future plan to work on any front-end
platform to consume these WABA 360 Dialog APIs.
I do have a similar interest and my working on building a tool, If you have
any interest to collaborate, I'll more than happy to connect further.
Regards,
Shivam Kalra
On Wed, 6 Apr 2022 at 11:59, sleekflowalexyuen @.***>
wrote:

I mistakenly used [FromBody] for that API End, it should use [FromForm]
instead.
Just pushed a fix with some for that.
Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/yuenkik6/WABA360Dialog.NET/issues/2#issuecomment-1089871839,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEHIJ2ZSY3VDFIOI7E7MZN3VDUVLPANCNFSM5STZT6PQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

This plugin is actually used for my company SasS project called Sleekflow, and I contribute mainly for backend part.


Branch logic refactoring

to #311

thanks will take a look later today or tonight

Looks good, thanks :smile:


Fix Issue #3497: move initialization to constructor

@sigatrev,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:shipit:

This will not meet the bar for RS3...  It should go into release/1.7.

going to close this and open a new PR for 1.7 to ensure the tooling doesn't get confused.


Prevent recursive processing of us_ticker in FSL

When a ticker is scheduled to run so fast that it is pending again
before the previous event has been processed then this next event
is processed (recursively) by calling into us_ticker_irq_handle().
This can lead to a stack overflow.
This patch prevents this recursion by replacing the call to
us_ticker_irq_handler() with a call to set the interrupt to pending
again.  This allows the next timer event to be processed without
making the stack deeper.

LGTM


Can't edit Shipping details

after clicking on this shipping edit button it redirects to same order page

@peter-mw  thank you for doing this so quick
src\MicroweberPackages\Order\Models\Order.php on this file
we also need to add two more
( address2,
other_info)
fields so that form can update these two fields as well

 public $fillable = [
        'id',
        'email',
        'first_name',
        'last_name',
        'country',
        'amount',
        'payment_amount',
        'transaction_id',
        'city',
        'state',
        'zip',
        'address',
        'address2',
        'other_info',
        'phone',
        'user_ip',
        'is_completed',
        'payment_gw'
    ];

added them in dev branch


ci: merge staging to master

This is an automatic PR generated by the pipeline CI/CD. This will be automatically fast-forward merged if successful.

Pipeline Attempt on 715233275 for 16de2a653c80d936e2cbd8fde7a07f5522df1fee
https://gitlab.com/MatrixAI/open-source/js-async-cancellable/-/pipelines/715233275

Pipeline Attempt on 715304063 for 16de2a653c80d936e2cbd8fde7a07f5522df1fee
https://gitlab.com/MatrixAI/open-source/js-async-cancellable/-/pipelines/715304063

@tegefaulkes there's a failure here in:
  ● PromiseCancellable › PromiseCancellable.from › from a cancellable function with default early rejection
    expect(received).rejects.toBeUndefined()
    Received: {}
      160 |       const pC = PromiseCancellable.from(p);
      161 |       pC.cancel();
    > 162 |       await expect(pC).rejects.toBeUndefined();
          |                                ^
      163 |       await expect(p).resolves.toBe('Result');
      164 |     });
      165 |   });
      at Object.toBeUndefined (node_modules/expect/build/index.js:241:22)
      at Object.<anonymous> (tests/PromiseCancellable.test.ts:162:32)

But only affects the mac builder.
This same error affects js-timer.
Can you check if this is a regression next. I'll label this as a bug.

We tested this on the matrix-mac-1, no problems there. It's only the Mac CI/CD. @tegefaulkes is looking into this in the CI/CD by pushing a bunch of console.log traces.

If fixed, then new update for this, and have to propagate that update across the rest of the downstream repos like js-timer which also has its CI/CD blocked on this bug.

Ultimately we want to know on the Mac CI/CD why await pC rejection is just {} when we are expecting undefined and it is undefined for all other platforms.
try {
  await pC;
} catch (e) {
  console.log(typeof e);
  console.log(e);
  console.log(e.name);
  console.dir(e);
}

You may then trace backwards to look at where the reject call is being made inside PromiseCancellable, and where that object is coming from.

I've found the problem. It seems that aborting the AbortController with abortController.abort() or abortController.abort(undefined) results in the abortController.signal.reason to be a DOMException [AbortError]: This operation was aborted error. It seems that the controller is helpfully filling in the reason for us if one is not provided but ONLY in the macos CI job.
The fix I'm going with here is to use an internal symbol symbolUndefinedReason in place of undefined for the abort reason. When receiving the abort event and rejecting we convert this back to undefined. This should be portable.

I prefer you calling it cancelXReason where X can be something. This names it similar to our tasks domain in PK. Could be cancelDefaultReason or cancelUndefinedReason.

The macos had a later version of node where AbortError exists. Other current nodejs doesn't so, by doing the above, everything will behave the same. It does mean pC.cancel() and pC.cancel(undefined) will reject with undefined.
Note:
      await expect(Promise.reject(undefined)).rejects.toBe(undefined);
      await expect(Promise.reject(new Error)).rejects.toThrow(undefined);

@tegefaulkes please update and release a patch version after staging merges. Then propagate updates to all users of async-cancellable. I think js-timers and Polykey.

Pipeline Attempt on 806740243 for 0dbf46c0ae3e887649b09af6caee85dbdd1966fa
https://gitlab.com/MatrixAI/open-source/js-async-cancellable/-/pipelines/806740243

Pipeline Succeeded on 806740243 for 0dbf46c0ae3e887649b09af6caee85dbdd1966fa
https://gitlab.com/MatrixAI/open-source/js-async-cancellable/-/pipelines/806740243

The macos had a later version of node where AbortError exists. Other current nodejs doesn't so, by doing the above, everything will behave the same. It does mean pC.cancel() and pC.cancel(undefined) will reject with undefined.
Note:
      await expect(Promise.reject(undefined)).rejects.toBe(undefined);
      await expect(Promise.reject(new Error)).rejects.toThrow(undefined);

Yes, both cancel() and cancel(undefined) will cause the promise to reject with undefined.
With this merged now I'll push a patch version and update any dependants.


Release to staging

Pull Request Test Coverage Report for Build 1786

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.07%) to 96.666%

Files with Coverage Reduction
New Missed Lines
%

../models/investigation.rb
1
99.02%

Totals

Change from base Build 1752:
0.07%

Covered Lines:
3856

Relevant Lines:
3989

💛  - Coveralls


"components" can be filenames

With current configuration I can't specify files to be generated by swagger, like:
 <supportingFiles>ApiClient.java,ApiException.java,Configuration.java,JSON.java,Pair.java,StringUtil.java,TypeRef.java,ApiKeyAuth.java,Authentication.java,HttpBasicAuth.java,OAuth.java,OAuthFlow.java</supportingFiles>

"compenents" is restricted to ['apis', 'models', 'supportingFiles']

Is there a workaround?

No workaround at this time. It will be supported in the next release.


can't connect to existing account

or I'm looking over it, or it's currently not possible to pair an existing account to steam openid
great plugin otherwise

I have no idea what you are talking about. Write a proper bug report or write none at all...


Pull request with type [CREATE] for user [bobmn] and object type [TaskSequence] - titled [How to use & set Windows 10 1803+ Rollback options]

Automated pull request from the Configuration Manager community hub.

Could you add README TS step?