package.json script clean doesn't work on windows

"clean": "rm -rf dist *.tgz *.zip" only works on linux

with shx it should work cross-platform

Used rimraf instead of rm -rf.


url is null

When creating a new user, the provided LinkedIn and GitHub links appear as null in the initial response body. Will validate to ensure the urls are stored properly.

Link to code changes can be found in branch 3-url-is-null


Add feedback link in footer

I threw in the envelope 📩 instead of the rocket 🚀. Mailto goes to labs@ with a simple, but generic subject. At the moment, displays the same on main and beta.
I like @androotubelli suggestion of trying this and seeing whether users actually use it.

@austinjpaul I like your ideas! Went with it. Can always change if we need to...


Update activeadmin_quill_editor.gemspec

Tested with activeadmin 2 and its working. Removing version from gemspec

@gabrieldesar: open-ended dependency is not a good practice for gems and AA2 is still in alpha - so currently I prefer to stay on the 1.x

I created a "aa2" branch to support "2.0.0.alpha"


map analysis

This PR is a mean average precision analysis of the cell death classes

All comments addressed and I fixed the p-value "wonkiness" by changing the null_distribution size :)


stucked creating the volume

Hi @quofelix @sttts,
I tried the kubernets-complete example. First issue was that I had to change the mountpoint of the host directory form /devices to /devices/dev1 in both:
https://github.com/quobyte/quobyte-deploy/blob/master/kubernetes/quobyte-complete/registry-bootstrap-pod.yaml#L48
https://github.com/quobyte/quobyte-deploy/blob/master/kubernetes/quobyte-complete/registry-rc.yaml#L50
Otherwise it seems that /devices/dev1 is not recognized as  mount point and the start of the registry pod fails.
After solving this I am now finally stucked at https://github.com/quobyte/quobyte-deploy/tree/master/kubernetes/quobyte-complete#create-a-volume .
The result I am getting is:
kubectl exec -it --namespace=quobyte registry-bootstrap -- qmgmt -u api:7860 volume create cluster root root BASE 0777
Operation denied by API service because: 'NOT_ENOUGH_RESOURCES/POSIX_ERROR_NONE: Could not create volume cluster because available metadata device do not satisfy the requested replica configuration: 1, got distinct devices [] out of list[] available devices. / ' Code: -20
error: error executing remote command: Error executing command in container: Error executing in Docker Container: 249

It seems that the metadata pod is not picking up the mountpoints of the other pods in my case. Logs of the metadata pods all show:
[ W | a                    | deviceStatus... | 118 | May 24 17:17:19.792 ] there are currently no data devices available
[ W | a                    | deviceStatus... | 118 | May 24 17:17:29.798 ] there are currently no data devices available
[ W | a                    | deviceStatus... | 118 | May 24 17:17:39.803 ] there are currently no data devices available
[ W | a                    | deviceStatus... | 118 | May 24 17:17:49.809 ] there are currently no data devices available
[ W | a                    | deviceStatus... | 118 | May 24 17:17:59.814 ] there are currently no data devices available
[ W | a                    | deviceStatus... | 118 | May 24 17:18:01.199 ] there are currently no data devices available
[ W | a                    | deviceStatus... | 118 | May 24 17:18:11.295 ] there are currently no data devices available

I did moify the MountFlags of the docker daemon as descibed in:
https://github.com/quobyte/quobyte-deploy/blob/master/kubernetes/Quobyte_readme.md#mountflags
and verified that it worked on all the nodes by executing:
core@core06 ~ $ systemctl show docker.service | grep MountFlags=
MountFlags=1048576

The devices are marked as online:
kubectl exec -it --namespace=quobyte registry-bootstrap -- qmgmt -u api:7860 device list
Id Host                                     Status                Disk Used Disk Avail  Services
3 registry-6pclc                           ONLINE                    17 GB     240 GB  REGISTRY
1 registry-bootstrap                       ONLINE                    19 GB     240 GB  REGISTRY
2 registry-fsoyh                           ONLINE                    17 GB     246 GB  REGISTRY
4 registry-hzlq6                           ONLINE                    17 GB     240 GB  REGISTRY

Coreos Version is: CoreOS stable (899.17.0)
As I said I am stucked. Is there anything I am doing obviously wrong?
As far as I can tell it seems that the mounts of the registry containers are not seen in the metadata container. But I don't know why and what I need to do to change this.
If i execute a mount inside the metadata container I see only its own mounts even if there is a registry container running on the same node:
core@core02 ~ $ # get all quobyte containers
core@core02 ~ $ docker ps | grep quobyte
eb8e0f758bd8        quay.io/quobyte/quobyte-server:1.2.10            "/bin/bash -xec 'sed "   37 minutes ago      Up 37 minutes                                  k8s_quobyte-metadata.e9a3b333_metadata-y7sgl_quobyte_4edf595b-21d3-11e6-a679-901b0e93232b_3cdbcb2a
4d39049cec4f        gcr.io/google_containers/pause:2.0               "/pause"                 37 minutes ago      Up 37 minutes                                  k8s_POD.6059dfa2_metadata-y7sgl_quobyte_4edf595b-21d3-11e6-a679-901b0e93232b_9721f108
c66c7b66970c        quay.io/quobyte/quobyte-server:1.2.10            "/bin/bash -xec 'sed "   37 minutes ago      Up 37 minutes                                  k8s_quobyte-metadata.56a145f2_data-jz7gf_quobyte_4953caea-21d3-11e6-a679-901b0e93232b_f3edbbf6
f4bca29a10df        gcr.io/google_containers/pause:2.0               "/pause"                 37 minutes ago      Up 37 minutes                                  k8s_POD.6059dfa2_data-jz7gf_quobyte_4953caea-21d3-11e6-a679-901b0e93232b_0722a6e4
bd923e2a9000        quay.io/quobyte/quobyte-server:1.2.10            "/bin/bash -xec 'sed "   48 minutes ago      Up 48 minutes                                  k8s_quobyte-registry.36244138_registry-6pclc_quobyte_d65715de-21d1-11e6-a679-901b0e93232b_bfe9b78b
e688eb3cd6a6        gcr.io/google_containers/pause:2.0               "/pause"                 48 minutes ago      Up 48 minutes                                  k8s_POD.d8224cbf_registry-6pclc_quobyte_d65715de-21d1-11e6-a679-901b0e93232b_35e265d7
core@core02 ~ $ # execute mount in registry container
core@core02 ~ $ docker exec -ti bd923e2a9000 mount
overlay on / type overlay (rw,relatime,lowerdir=/var/lib/docker/overlay/38c2b075d4ec64afbcf01a7fbb1f31949297a23aac0de03db3825300575aa50d/root,upperdir=/var/lib/docker/overlay/bd923e2a90007066d7a284c957d0e01c67d2da9a9812f9566d6f15481d432109/upper,workdir=/var/lib/docker/overlay/bd923e2a90007066d7a284c957d0e01c67d2da9a9812f9566d6f15481d432109/work)
proc on /proc type proc (rw,nosuid,nodev,noexec,relatime)
tmpfs on /dev type tmpfs (rw,nosuid,seclabel,mode=755)
devpts on /dev/pts type devpts (rw,nosuid,noexec,relatime,seclabel,gid=5,mode=620,ptmxmode=666)
sysfs on /sys type sysfs (ro,nosuid,nodev,noexec,relatime,seclabel)
tmpfs on /sys/fs/cgroup type tmpfs (rw,nosuid,nodev,noexec,relatime,seclabel,mode=755)
cgroup on /sys/fs/cgroup/systemd type cgroup (rw,nosuid,nodev,noexec,relatime,xattr,release_agent=/usr/lib/systemd/systemd-cgroups-agent,name=systemd)
cgroup on /sys/fs/cgroup/freezer type cgroup (rw,nosuid,nodev,noexec,relatime,freezer)
cgroup on /sys/fs/cgroup/net_cls,net_prio type cgroup (rw,nosuid,nodev,noexec,relatime,net_cls,net_prio)
cgroup on /sys/fs/cgroup/perf_event type cgroup (rw,nosuid,nodev,noexec,relatime,perf_event)
cgroup on /sys/fs/cgroup/memory type cgroup (rw,nosuid,nodev,noexec,relatime,memory)
cgroup on /sys/fs/cgroup/blkio type cgroup (rw,nosuid,nodev,noexec,relatime,blkio)
cgroup on /sys/fs/cgroup/cpu,cpuacct type cgroup (rw,nosuid,nodev,noexec,relatime,cpu,cpuacct)
cgroup on /sys/fs/cgroup/cpuset type cgroup (rw,nosuid,nodev,noexec,relatime,cpuset)
cgroup on /sys/fs/cgroup/devices type cgroup (rw,nosuid,nodev,noexec,relatime,devices)
/dev/sda9 on /devices/dev1 type ext4 (rw,relatime,seclabel,data=ordered)
/dev/sda9 on /dev/termination-log type ext4 (rw,relatime,seclabel,data=ordered)
tmpfs on /run/secrets/kubernetes.io/serviceaccount type tmpfs (ro,relatime,rootcontext=system_u:object_r:var_lib_t:s0,seclabel)
/dev/sda9 on /etc/resolv.conf type ext4 (rw,relatime,seclabel,data=ordered)
/dev/sda9 on /etc/hostname type ext4 (rw,relatime,seclabel,data=ordered)
/dev/sda9 on /etc/hosts type ext4 (rw,relatime,seclabel,data=ordered)
shm on /dev/shm type tmpfs (rw,nosuid,nodev,noexec,relatime,context="system_u:object_r:svirt_lxc_file_t:s0:c253,c520",size=65536k)
mqueue on /dev/mqueue type mqueue (rw,nosuid,nodev,noexec,relatime,seclabel)
core@core02 ~ $ # execute mount in metadata container
core@core02 ~ $ docker exec -ti eb8e0f758bd8 mount
overlay on / type overlay (rw,relatime,lowerdir=/var/lib/docker/overlay/38c2b075d4ec64afbcf01a7fbb1f31949297a23aac0de03db3825300575aa50d/root,upperdir=/var/lib/docker/overlay/eb8e0f758bd8923c2574e11976d82b6156940ea1865a775d7db23375c0a90956/upper,workdir=/var/lib/docker/overlay/eb8e0f758bd8923c2574e11976d82b6156940ea1865a775d7db23375c0a90956/work)
proc on /proc type proc (rw,nosuid,nodev,noexec,relatime)
tmpfs on /dev type tmpfs (rw,nosuid,seclabel,mode=755)
devpts on /dev/pts type devpts (rw,nosuid,noexec,relatime,seclabel,gid=5,mode=620,ptmxmode=666)
sysfs on /sys type sysfs (ro,nosuid,nodev,noexec,relatime,seclabel)
tmpfs on /sys/fs/cgroup type tmpfs (rw,nosuid,nodev,noexec,relatime,seclabel,mode=755)
cgroup on /sys/fs/cgroup/systemd type cgroup (rw,nosuid,nodev,noexec,relatime,xattr,release_agent=/usr/lib/systemd/systemd-cgroups-agent,name=systemd)
cgroup on /sys/fs/cgroup/freezer type cgroup (rw,nosuid,nodev,noexec,relatime,freezer)
cgroup on /sys/fs/cgroup/net_cls,net_prio type cgroup (rw,nosuid,nodev,noexec,relatime,net_cls,net_prio)
cgroup on /sys/fs/cgroup/perf_event type cgroup (rw,nosuid,nodev,noexec,relatime,perf_event)
cgroup on /sys/fs/cgroup/memory type cgroup (rw,nosuid,nodev,noexec,relatime,memory)
cgroup on /sys/fs/cgroup/blkio type cgroup (rw,nosuid,nodev,noexec,relatime,blkio)
cgroup on /sys/fs/cgroup/cpu,cpuacct type cgroup (rw,nosuid,nodev,noexec,relatime,cpu,cpuacct)
cgroup on /sys/fs/cgroup/cpuset type cgroup (rw,nosuid,nodev,noexec,relatime,cpuset)
cgroup on /sys/fs/cgroup/devices type cgroup (rw,nosuid,nodev,noexec,relatime,devices)
/dev/sda9 on /devices type ext4 (rw,relatime,seclabel,data=ordered)
/dev/sda9 on /dev/termination-log type ext4 (rw,relatime,seclabel,data=ordered)
tmpfs on /run/secrets/kubernetes.io/serviceaccount type tmpfs (ro,relatime,rootcontext=system_u:object_r:var_lib_t:s0,seclabel)
/dev/sda9 on /etc/resolv.conf type ext4 (rw,relatime,seclabel,data=ordered)
/dev/sda9 on /etc/hostname type ext4 (rw,relatime,seclabel,data=ordered)
/dev/sda9 on /etc/hosts type ext4 (rw,relatime,seclabel,data=ordered)
shm on /dev/shm type tmpfs (rw,nosuid,nodev,noexec,relatime,context="system_u:object_r:svirt_lxc_file_t:s0:c691,c956",size=65536k)
mqueue on /dev/mqueue type mqueue (rw,nosuid,nodev,noexec,relatime,seclabel)
core@core02 ~ $ # see if docker has MountFlags=shared
core@core02 ~ $ systemctl show docker.service | grep MountFlags=
MountFlags=1048576

Thanks in advance for helping
Cornelius

You need at least one data and one metadata device and service.

Sorry for the confusion about mounts not beeing visible in other containers. I got this totally wrong and should have thought deeper about it.
The reason for this not working properly is that quobyte changed the way it scans for volume markers. The volume marker needs do be of the mount point, in a subdirectory it is not working.
Thanks to Silvan at quobyte for helping me out and clarifying things.
The only problem left for me is that the client-nsenter pod was not working for me due to shared library problems and the workaround I used without nsenter is running but mounting the quebyte cluster storage only in the container and not in the host system.

I assume the issue is resolved? If not feel free to reopen it.


Version info

This relates to #131
I realized after I made the script that not all of the exercise tests have the version info in them, or perhaps they don't have it on a line that also includes the text problem-specifications.
I definitely think there should be some kind of standard for representation of the version info in the test files.  If this is done, then the script I wrote can be used to check for missing versions, and to automatically update the .meta/version files from the convention in the test files.
Let me know what you think.

So, I recently learned that .meta/version is not the only way to represent the version information to the Dashboard project, so if you all want to use4 another convention for it just let me know, and I will write some stuff to handle it.

I think it's great that you've taken the effort so I'm merging this.  If the approach has problems we can always fix it then right?

No problem.  I have work to do on both the bash and scheme tracks, but I like Racket (I started using it for some projects too :heart_eyes: ), so I figure it is a good idea to try helping out over here.

It's a lovely language 😄  but thanks for the help, always appreciated 👍


Dialer: show headsup on incoming call even if headsup are off and tic…

…ker on
New behaviors:

you are not in dialer and get a call: system shows headsup thus doesn't change
the foreground app you were on
you are in dialer and get a call: dialer shows incallui without headsup
you are in dialer and call someone: dialer shows incallui without headsup

Change-Id: I2a750d357c20c112f6d5cd8fab87784a2e53f7b9

Conflicts exist

Fixed them. You can merge now.

Sending a new one with revert first so closing this one.


Add skeleton content for G-Cloud 9

The dates are taken from the provisional G-Cloud 9 timetable detailed here: https://digitalmarketplace.blog.gov.uk/2017/01/19/g-cloud-9-a-provisional-timetable/
The declaration questions are taken from Digital Outcomes and Specialists 2.
There are a few basic service questions that will allow us to submit very basic "services" on each of the three G-Cloud 9 lots.

LGTM 👍  couple of comments but no real dealbreakers. Merge when ready.


Please add IRESPO token from irespotokens smart contract as default

Our project is making an ICO in 2 weeks.
We will be grateful if IRESPO token from irespotokens smart contract is added to greymass wallet as default.

Once you have created your token, feel free to add it yourself! See the header Add to the table and replace with your information:
https://github.com/greymass/customtokens

I'm sure someone will get it added as it's released, and the torrent of "where airdrop?" starts flooding in 😁

Done! Thank you! Nicely done with storing the tokens in smart contract and an option to add on your own!


Implement SOCKS4

It turns out that Java has built-in support for SOCKS requests, but it only supports version 4.  We should add SOCKS4 support to SocksToRtc, so that we can handle this request.
Lack of support for SOCKS4 has caused us to pull in a SOCKS5 library into the Android build, inflating the APK and increasing the complexity of the codebase.

I daresay adding support for SOCKS4 will increase the complexity of the codebase more than adding a JAR to the build but it will help with the build process - and if it's not much work to add support, then sure. In any case, great to have this tracked.
From skimming what seems to be the definitive SOCKS4 spec (no RFC?) it may not be a lot of work to add support for SOCKS4 - but that SocksToRtc to RtcToNet link could be a nuisance:
http://www.openssh.com/txt/socks4.protocol

FWIW, my expectation is that this would not involve any changes to RtcToNet.

@bemasc @albertolalama any need to keep this open?

I don't think so. jsocks, our third party SOCKS5 library, is working fine.

This would be needed if we wanted to use DNS-over-HTTPS as uProxy's DNS.  We no longer have a motivation for this if we move to the proposed AutomapHosts method of DNS resolution.


LocalFree should be take void* or IntPtr as its parameter

Actual behavior
The LocalFree method is declared by CsWin32 as taking an nint:
[DllImport("KERNEL32.dll", ExactSpelling = true, SetLastError = true)]
[DefaultDllImportSearchPaths(DllImportSearchPath.System32)]
internal static extern nint LocalFree(nint hMem);

Expected behavior
The method takes a pointer. Not an arbitrary pointer-sized integer. While this is just semantics, it is more intuitive to use IntPtr or (even better) void* for the parameter type.
As it is, I have to cast pointers to nint to call this method.
Same for the return type.

Blocked by https://github.com/microsoft/win32metadata/issues/1296

Fixed in #889


chore: validate custom app configuration

While all FOCA configuration is automatically validated within FOCA, the same is currently not true for any custom configuration. Either validate the configuration in proWES (via pydantic models) or implement a mechanism that allows passing a model module to FOCA so that it can validate custom configuration as well (preferred).

Closed with #55


[Support] <Error sudo ./openvpn-install.sh>

In raising this issue, I confirm the following:
{please fill the checkboxes, e.g: [X]}

[X] I have read and understood the contributors guide.
[X] The issue I am reporting can be replicated.
[X] The issue I am reporting can be is directly related to the pivpn installer script.
[X] The issue I am reporting isn't a duplicate (see FAQs, closed issues, and open issues).

Issue
attempting to sudo ./openvpn-install.s on a Raspberry Pi 4 with Kali 2020.4 32 bit OS.
hostnamectl
Static hostname: kali
Icon name: computer
Machine ID: 315928cc6d65415aab45a9abd81a6b01
Boot ID: 6e756a4e1f5e4774bcf9ce2c023c7fe0
Operating System: Kali GNU/Linux Rolling
Kernel: Linux 4.19.127-Re4son-v7l+
Architecture: arm
Have you searched for similar issues and solutions?
(yes/no / which issues?)

Yes, however there are no returned searches when key words Kali + Raspberry are used
Console output of      curl -L install.pivpn.io | bash
  Debian 9 or higher is required to use this installer.
This version of Debian is too old and unsupported.

Console output of      pivpn add or pivpn add nopass
  Output Here

Console output of      pivpn debug
 pivpn: command not found (because installation fails)

Have you taken any steps towards solving your issue?
Followed installation instructions found online, youtube videos

Hi, I am not really sure what you are really trying to achieve and what documentation you have seen.. but certainly was not ours!
Kali is not supported, and not at all recommended for this,
Please read the documentation at https://docs.pivpn.io
Closing.


support stable compiler on tool chain with prebuilt binaries in mac

In mac, if instaling through
curl -L https://sp1.succinct.xyz | bash

get rustc --version : 0.
want it to be up with stable version :)

@rkdud007 - The mac toolchain build should be updated now to rustc 1.75.  Please rerun your sp1up script.
I'm going to close this issue.  Feel free to re-open if you still have issues!


Difficulty setting

There should be an ability to select / set the game difficulty.
Currently I think it should affect the game speed.
Also this task involves score table changes, because (as I think) we should have separate tables or sections for different game difficulties. Also affects #108 but I hope noone has touched that task yet :)

We could also adjust the game difficulty with the different AIs, probably? >.> #72


HHH-16500 Avoid unnecessary reflection in StreamDecorator

See https://hibernate.atlassian.net/browse/HHH-16500

Stream decorators should be removed. If you want to contribute, we'd welcome a PR that removes all the classes.

You have to search for the JIRA ticket for this, but essentially, a Stream has a close method which the user must invoke to close underlying resources. That's simply how the JDK specifies this, and we follow the model. Trying to be smart and closing IO after executing terminal operations is not the way to go.


Wrong redirection on grafana in monitoring

bug:
Steps to reproduce (least amount of steps as possible):

Setup cluster monitoring

Waiting for success
Click on any icon grafana in rancher dashboard

Expected:
Open dashboard with context

Actual:
Display grafana index page

only if cluster name local
and when i click by icon system redirect to another page
https://rancher/k8s/clusters/local/api/v1/namespaces/cattle-prometheus/services/http:access-grafana:80/proxy/d/FnxG5TYiz/nodes
https://rancher/api/v1/namespaces/cattle-prometheus/services/http:access-grafana:80/proxy/?orgId=1
redirect code: https://github.com/rancher/rancher/blob/master/pkg/api/steve/proxy/proxy.go#L39
Other details that may be helpful:
Environment information

Rancher version: v2.5.5
Installation option: install a cluster with rke

Cluster information

Cluster type: Imported
Machine type (cloud/VM/metal) and specifications (CPU/memory):
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.3", GitCommit:"1e11e4a2108024935ecfcb2912226cedeafd99df", GitTreeState:"clean", BuildDate:"2020-10-14T12:50:19Z", GoVersion:"go1.15.2", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.2", GitCommit:"59603c6e503c87169aea6106f57b9f242f64df89", GitTreeState:"clean", BuildDate:"2020-01-18T23:22:30Z", GoVersion:"go1.13.5", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

Client: Docker Engine - Community
 Version:           20.10.2
 API version:       1.41
 Go version:        go1.13.15
 Git commit:        2291f61
 Built:             Mon Dec 28 16:17:43 2020
 OS/Arch:           linux/amd64
 Context:           default
 Experimental:      true

Server: Docker Engine - Community
 Engine:
  Version:          20.10.2
  API version:      1.41 (minimum version 1.12)
  Go version:       go1.13.15
  Git commit:       8891c58
  Built:            Mon Dec 28 16:15:19 2020
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.4.3
  GitCommit:        269548fa27e0089a8b8278fc4fc781d7f65a939b
 runc:
  Version:          1.0.0-rc92
  GitCommit:        ff819c7e9184c13b7c2607fe6c30ae19403a7aff
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

@izaac @SvenDowideit @gliptak

@sowmyav27
https://github.com/rancher/rancher/pull/31171

The bug is reproduced on rancher v2.5.6 HA setup with Montiroing v1 chart 0.2.1
Steps:

install Rancher v2.5.6 in HA mode
enable monitoring v1 0.2.1 on the local cluster
Click on the Grafana icon on the node

Results:

the Grafana Home page shows instead of the Node page

Reopen the issue because the bug still exists on the latest v2.5-head

After digging into this change a bit more and digging into the UI code deeper, the UI change that was put in won't fix this issue. The change put in will only effect the istio-nav on the project level istio page. The images shown in the issue reference the k8s-component-status component which pulls the urls off the current clusters monitoringStatus.grafanaEndpoint property. I'm not sure why the istio-nav component references statically built URL's but I suspect it may need to be updated to pull the url from the current clusters monitoringStatus.grafanaEndpoint.
After talking with @aiyengar2 offline I think agree with his sentiment that the change for this may be better suited for the backend.  @aiyengar2 can I get your thoughts on this?

Any updates on this issue?


Sliver bottom sheet doesn't allow sticky sliver magic

Providing MultiSliver widget to WoltModalSheetPage with SliverPersistentHeaders inside should allow them to stick their headers below bottomsheet top bar. Right now content scrolls behind top bar and sticks there where it's hidden because top bar overlaps it. Turning isTopBarVisibleWhenScrolled to false allows us to see it, but top bar is gone.
How it should work:
https://github.com/woltapp/wolt_modal_sheet/assets/61657553/d9039a49-a08e-42b5-8262-17e7a1850309
How it actually works:
https://github.com/woltapp/wolt_modal_sheet/assets/61657553/77079aab-0b79-4589-8a79-938d910844ca

The problem seems to be related to the fact that the composition in WoltModalSheetLayout uses Stack. The viewport of the main scrollable content is the whole space we have. topBarAnimatedBuilder as well as sabAnimatedBuilder don't affect this space at all and are painted over this space. It also affects the last element in the list (this element is hidden by sabAnimatedBuilder overpainting). I see two options to fix it: to find a workaround for detecting the current state of builders' sizes and reduce the space of content (some kind of emulation of these builders being in the scrollable area), or try to change the approach to layout in WoltModalSheetLayout and really to put them into a scrollable area (big changes). @ulusoyca what do you think about it?

Hi @KamilSztandur. Might using the new isTopBarLayerAlwaysVisible field present in this PR #31 help with this issue? According to your intended design, it seems like the top bar should always be visible.


Export PACKIT variables identifying repo and brach

When testing PR on Github in Packit CI, we know PACKIT_COMMIT_SHA but we do not know git repo and branch containing this commit. These should be also exported.

Thanks @kkaarreell for creating the issue.
I am checking what env. vars we set:
https://github.com/packit/packit-service/blob/56ee0595429c6577aca3933a787d9206f6f0a087/packit_service/worker/testing_farm.py#L169-L182
But only not None values are actually sent:
https://github.com/packit/packit-service/blob/56ee0595429c6577aca3933a787d9206f6f0a087/packit_service/worker/testing_farm.py#L184-L186
And here is an example of TF requests: https://api.dev.testing-farm.io/v0.1/requests/bef4cbfb-3928-448d-b3b5-b862f5abd8e6
And only a few are set:
"variables": {
    "PACKIT_COMMIT_SHA": "b1eb01629b6af035dc95d6815ca4731922fb0fa4",
    "PACKIT_DOWNSTREAM_NAME": "keylime",
    "PACKIT_DOWNSTREAM_URL": "https://src.fedoraproject.org/rpms/keylime.git",
    "PACKIT_FULL_REPO_NAME": "keylime/keylime",
    "PACKIT_PACKAGE_NAME": "keylime",
    "PACKIT_TARGET_SHA": "d85baf35f95881795cb0f9786c2a9aebdd857bcd",
    "PACKIT_UPSTREAM_NAME": "keylime",
}

So, @kkaarreell, you can set upstream_project_url to get the PACKIT_UPSTREAM_URL filled (for now).
But, we ideally need to:

[ ] send the PACKIT_UPSTREAM_URL even if not set in the config (we know it, self.project can be used)
[ ] send the branch names as PACKIT_TARGET_BRANCH and PACKIT_SOURCE_BRANCH (naming can be changed)

After some time playing with that, I'll add a few new variables that will be set only for PR jobs:
{
  "PACKIT_SOURCE_SHA": self.source_branch_sha,
  "PACKIT_TARGET_SHA": self.target_branch_sha,
  "PACKIT_SOURCE_BRANCH": self.source_branch,
  "PACKIT_TARGET_BRANCH": self.target_branch,
  "PACKIT_SOURCE_URL": self.source_project_url,
  "PACKIT_TARGET_URL": self.target_project_url,
}

So, it will look like this:
"variables": {
                    "PACKIT_FULL_REPO_NAME": "packit-service/hello-world",
                    "PACKIT_UPSTREAM_NAME": "hello-world",
                    "PACKIT_UPSTREAM_URL": "https://github.com/packit-service/hello-world",
                    "PACKIT_DOWNSTREAM_NAME": "hello-world",
                    "PACKIT_DOWNSTREAM_URL": "https://src.fedoraproject.org/rpms/hello-world.git",
                    "PACKIT_PACKAGE_NAME": "hello-world",
                    "PACKIT_COMMIT_SHA": "0011223344",
                    "PACKIT_SOURCE_SHA": "0011223344",
                    "PACKIT_TARGET_SHA": "deadbeef",
                    "PACKIT_SOURCE_BRANCH": "the-source-branch",
                    "PACKIT_TARGET_BRANCH": "the-target-branch",
                    "PACKIT_SOURCE_URL": "https://github.com/someone/hello-world",
                    "PACKIT_TARGET_URL": "https://github.com/packit-service/hello-world",
                },

Regarding the PACKIT_UPSTREAM_URL project, I will leave it as is for now.


Fix volume mounting issue in MySQL image (FINERACT-1294)

Description
Fixed "Access Denied" issue came from MYSQL container which was caused by the order of volume mounting configured in mysql-deployment.yml.
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[x] Write the commit message as per https://github.com/apache/fineract/#pull-requests

[x] Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

[ ] Create/update unit or integration tests for verifying the changes made.

[x] Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

[] Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/api-docs/apiLive.htm with details of any API changes

[x] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

@xurror @vorburger Please review

@xurror @vorburger Please review

See my comments in FINERACT-1294.

See my comments in FINERACT-1294.


Add a function for /coins/list coingecko endpoint #20

Can you open separate PR for separate endpoints?

Done!

Please do similar changes, I mentioned on vs_currency PR.

Closes: #61


refactor IMDB tutorial

closes #190

I think, to be consistent we should already include the train/dev /test split here. What do you think?

I think, to be consistent we should already include the train/dev /test split here. What do you think?

I was very surprised to see that the tutorial ends with just the preprocessing and does not do the split, so it is a good idea imo. @agsedova should we add this?
Also i recall seeing the code for this split somewhere. Would you have an idea where this was?

I found this is in a notebook, could be useful:
`
from sklearn.model_selection import train_test_split
new_data_dir = "/Users/andst/devel/sentence_nf/data/imdb"
rest_df, test_df, rest_rule_matches_sparse_z, test_rule_matches_sparse_z = train_test_split(imdb_dataset_raw, rule_matches_sparse, test_size=5000, random_state=42)
train_df, dev_df, train_rule_matches_sparse_z, dev_rule_matches_sparse_z = train_test_split(rest_df, rest_rule_matches_sparse_z, test_size=5000, random_state=42)
dump(train_df, os.path.join(new_data_dir, "train_df.pbz2"))
dump(dev_df, os.path.join(new_data_dir, "dev_df.pbz2"))
dump(test_df, os.path.join(new_data_dir, "test_df.pbz2"))
dump(train_rule_matches_sparse_z, os.path.join(new_data_dir, "train_rule_matches_sparse_z.pbz2"))
dump(dev_rule_matches_sparse_z, os.path.join(new_data_dir, "dev_rule_matches_sparse_z.pbz2"))
dump(test_rule_matches_sparse_z, os.path.join(new_data_dir, "test_rule_matches_sparse_z.pbz2"))
dump(mapping_rules_labels,  os.path.join(new_data_path, "mapping_rules_labels_t.pbz2"))
`

I think, to be consistent we should already include the train/dev /test split here. What do you think?

I was very surprised to see that the tutorial ends with just the preprocessing and does not do the split, so it is a good idea imo. @agsedova should we add this?

It is definitely a good idea :)


Change border color to be inverted depending on wallpaper

Requested by jsherry47 on DeviantArt.
Change the color of the border accodingly to the background. This could be done via the Chameleon plugin.

Not sure the fesability but would it be possible to generate a complentary color based on the region of the wallpaper it is in? It may match better

It would technically possible, but it might use a lot more CPU since it has to crop it first.

You might find it useful to only grab colors from a cropped region of an image. You can do so with the CropX, CropY, CropW, and CropH options. These set the position and dimension of the rectangle to crop out.
https://github.com/socks-the-fox/chameleon

I would have to look at Sox's code to be sure but it may be faster since it has to iterate over a smaller section of the image.
The bigger issue would be we would actually have to calculate the complentary color and it may look out of place sometimes I guess. Shame chameleon does not have color palette options similar to Android.

Marco what if we replaced chameleon with an in house color gen that generated colors similar to android palette system? I have thought about making one like that in the past and it may generate a better foreground color than chameleon and would also generate a complementary color which will help make sure it does not blend in too much.
Example: http://jariz.github.io/vibrant.js/

I'm gonna close this one as this is currently not possible without a new plugin as Chameleon doesn't take dynamic variables as CropX and CropY.
If we are gonna change the plugin in the future, we will reopen this issue to keep track of it.


Re-exporting types is incompatible with --isolatedModules=true

See this thread about the incompatibility of the two features: https://www.reddit.com/r/typescript/comments/e9span/isolatedmodules_prevents_exporting_a_type_from_a/
It'd be nice to have a flag that simply causes these types to not be re-exported (I'm not totally sure why they are in the first place). Workaround is just to delete the line that re-exports, for example, with sed:
sed -i "/^export [{]/d" $file.validator.ts

Thanks @hjfreyer.
This was an issue for me aswell and your workaround works like a charm.

I'm using a mac so when running the app locally I needed to do this to run the same command sed -i '' "/^export [{]/d" $file.validator.ts note the '' after the -i flag.
I hope hjfreyer's suggestion is considered at some point.

Same problem. Thanks for the workaround @hjfreyer


Add whereami

Add whereami: CLI tool for getting user's geolocation information

Hey @alebcay, glad you liked! Thanks :smile:


Defining Object in module breaks vitest

Describe the bug
If a user does something like so:
export const Object = 1

Vitest will throw an error of:
TypeError: Object.defineProperty is not a function

I consider this bug because user could should not be able to crash Vitest like this.
Reproduction
As described
System Info
Doesn't matter

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

This is a bug in Vite: https://github.com/vitejs/vite/issues/15485

This is already tracked on Vitest too https://github.com/vitest-dev/vitest/issues/4829, so can we close?


More columns than values in query response

Hello,
I am sure this is a duplicate of something somewhere... but here goes:
Using mean() in a query breaks the response format: there are more columns than values in the query response.
With mean():
Query:
SELECT host, mean(zzz) as hello, host, value as value, zzz, host FROM cpu_value
Response:
{"series": [{"values": [["1970-01-01T00:00:00Z", 34545]], "name": "cpu_value", "columns": ["time", "host", "hello", "host", "value", "zzz", "host"], "tags": {"host": "server01"}}]}
Without mean():
Query:
SELECT host, zzz as hello, host, value as value, zzz, host FROM cpu_value
Response:
{"series": [{"values": [["2015-08-10T14:03:44.833892297Z", 34545, 2.34, 34545]], "name": "cpu_value", "columns": ["time", "zzz", "value", "zzz"], "tags": {"host": "server01"}}]}
easy reproduction:
This is the script I used to test this:
from influxdb import InfluxDBClient
import json

client = InfluxDBClient('localhost', 8086, 'root', 'root', 'example')
resp = client.query('DROP measurement cpu_value')

points = [
    {
        "measurement": "cpu_value",
        "tags": {
            "host": "server01",
            "region": "us-west",
            "type_instance": "idle",
        },
        "fields": {
            "value": 0.64,
            "zzz": 34545,
        }
    },
    {
        "measurement": "cpu_value",
        "tags": {
            "host": "server01",
            "region": "us-west",
            "type_instance": "idle",
        },
        "fields": {
            "value": 2.34,
            "zzz": 34545,
        }
    }

]

client.write_points(points)
QUERY = """SELECT host, zzz as hello, host, value as value, zzz, host FROM cpu_value"""
resp = client.query(QUERY)
print json.dumps(resp.raw)

See the influxdb-python issue: https://github.com/influxdb/influxdb-python/issues/228

@corylanou
Can you be more precise? What in the output don't you like?

Not sure if you noticed it, but the issue here is that host is a tag. So it should not be included in the columns list.
Not that i'm against it it being included. But if it's included, include it in the values too!

I just fixed a bug with the alias (which is shown in your last query - https://github.com/influxdb/influxdb/pull/3564).  I'm also making changes right now to the select behavior that if you put a tag in the select clause, it comes back as a column, not as another grouped set of output (https://github.com/influxdb/influxdb/issues/3549).
This may address your concerns, I'm not sure.

if you put a tag in the select clause, it comes back as a column

Great. Ping me here when this is merged and I will test it. I think that will fix the issue.

@aviau the code @corylanou referenced has been checked in and is in the nightly build. Can you try to repro with the latest?

With the same test, this is the new result:
{"series": [{"values": [["2015-08-19T14:18:34.794532536Z", "server01", 34545, "server01", 0.64, 34545, "server01"]], "name": "cpu_value", "columns": ["time", "host", "hello", "host", "value", "zzz", "host"]}]}

Thanks!


Bump yaml from 2.1.3 to 2.2.2 (for main-v0)

Relates to #396
Summary
Bump transitive dependency yaml from 2.1.3 to 2.2.2 because of CVE-2023-2251, this time for the main-v0 branch.
It's a development dependency only, so no release is necessary.

This will be merged despite the failing "Check / CodeQL" job. The failure is due to the same issue as was resolved in #339 (which hasn't been apply to this branch). Given the nature of this change I'm confident the CodeQL job wouldn't have brought up any novel issues.


[FLOC-2331] in_parallel error handling

Fixes https://clusterhq.atlassian.net/browse/FLOC-2331 (sub-task of https://clusterhq.atlassian.net/browse/FLOC-2327) in the master.  Forward-port of #1637.
Has an extra change from the 1.0.0 release since that branch hasn't been merged yet.  Don't merge this until #1634 is merged.

Merged forward after merging 1.0.0 back into master.
The changes now look identical to those that have been merged into release/flocker-1.0.0.
So I'll merge this when the unit tests have passed.

Actually no.
This branch seems to be missing some things that are in the release branch.
engineering > in_parallel-error-handling-FLOC-2330
This branch of in_parallel error handling seems to be missing the _superlative function that I can see in release-maintenance/flocker-1.0.0/in_parallel-error-handling-FLOC-2330

https://github.com/ClusterHQ/flocker/pull/1639

vs

https://github.com/ClusterHQ/flocker/pull/1640

I won't merge this yet.

Thanks @exarkun

The changes in this branch do now look identical to those already merged into release/flocker-1.0.0
The new unit tests have passed.

So I'll merge.


SQL Injection

I know this is a getting started demo, but surely all examples should advocate the use of parameters?
e.g.
var sqlQueryText = "SELECT * FROM c WHERE c.LastName = @name";
CosmosSqlQueryDefinition queryDefinition = new CosmosSqlQueryDefinition(sqlQueryText).UseParameter("@name", "Andersen");

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

ID: 71e51f1b-ce31-9f0d-317d-574f97566b7d
Version Independent ID: 71cbef81-ca57-cbfb-63be-b949346d3155
Content: Build a .NET Core console app to manage data in Azure Cosmos DB SQL API account (SDK Version 3 Preview)
Content Source: articles/cosmos-db/sql-api-dotnet-core-get-started-preview.md
Service: cosmos-db
GitHub Login: @deborahc
Microsoft Alias: dech

@s-leonard Thank you very much for the insightful feedback. For the most part, all quickstarts and tutorials are based upon static values. Not sure if this is for simplicity or if there is a possibility to also include more advanced examples with these sample scripts. I am going to assign this to the content owner to evaluate and update as appropriate.

Minor addition, but in step 5, "In the NuGet tab, click Browse, and type Microsoft.Azure.Cosmos in the search box.", it should also mention that the 'include prerelease" checkbox should be checked - mostly because by default it's not, and it just caught me off-guard. Makes the documentation slightly easier to follow.

Very old case. Closing. Please create a new issue is the documentation needs to be addressed.


Consider i2c without a timeout function

The p_timeout approach is silly. Yes, its helpful for clock stretching case, but its very rare to find devices that use clock stretching. If most devices supported clock stretching, then having the API take a timeout callback would be acceptable. But its so rare that most code is just passing hal::never_timeout(). p_timeout is also problematic for i2c devices that use DMA. Because this has to be polled, when should the DMA code be running? It could just spin while it waits for DMA to finish processing, but we loose out on the capability to put the device to sleep or switch tasks by using the polling option. Overall, it is likely in the future that we will either make hal::i2c_v2 with an hal::io_waiter in place of the p_timeout or eliminate the parameter all together and recommend that i2c implementations accept an io_waiter as an input parameter. Not sure

Remove timeout, don't use clock stretching devices
After discussing this with members of the libhal team, almost everyone agrees that we should move away from passing around time outs. And if you have to deal with a device that performs clock stretching, our advice would be "don't" and if you still need to use it, we consider that the bus is no longer deterministic and other external things will be necessary to provide some determinism such as a means to power off the device if it holds the bus down for too long. Like an output pin connected to the devices reset line.


Feature 1

In this repository, we have done the following

Add Books to a book list
Delete Books from the book list
Save data to local storage

Hi @Yordinia and @Caren-Koroeny,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
Raising a PR is not enough as a contribution to the project. Kindly ensure there are commits from all coding partners.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


request timed out while waiting for response

version:2.1.0
Agent:
error:
web application directory [/usr/local/tomcat2/webapps/pinpoint-agent-2.1.0] has finished in [435] ms
13-Sep-2020 20:19:49.397 INFO [main] org.apache.coyote.AbstractProtocol.start Starting ProtocolHandler ["http-nio-8098"]
13-Sep-2020 20:19:49.405 INFO [main] org.apache.coyote.AbstractProtocol.start Starting ProtocolHandler ["ajp-nio-9009"]
13-Sep-2020 20:19:49.407 INFO [main] org.apache.catalina.startup.Catalina.start Server startup in 504 ms
09-13 20:19:50.050 [nfoSender-Timer] WARN  c.n.p.p.AgentInfoSender                  -- request timed out while waiting for response.
09-13 20:19:53.053 [nfoSender-Timer] WARN  c.n.p.p.AgentInfoSender                  -- request timed out while waiting for response.
09-13 20:19:56.056 [nfoSender-Timer] WARN  c.n.p.p.AgentInfoSender                  -- request timed out while waiting for response.
09-13 20:19:59.059 [nfoSender-Timer] WARN  c.n.p.p.AgentInfoSender                  -- request timed out while waiting for response.
09-13 20:20:02.002 [nfoSender-Timer] WARN  c.n.p.p.AgentInfoSender                  -- request timed out while waiting for response.

check your collector log for errors and also network connectivity between the agent and your collector.


Improper spelling on "Dscirdo authetication" label

Defect: The "Dscirdo authenication" label is spelled incorrectly.
Location: The "Dscirdo authenication" page
Recommend fix: change "Dscirdo authenication" to "Discord Authentication"
Criticality: Low

This is a non-issue.


--split filters out non-test files in tests/

I have files in tests like tests/fixtures/blog-posts.js. When I use --split=4 --split-file=<1,2,3,4>, they get filtered out of some runs, causing test failures.

This is the culprit. The intent is that test helpers should be included in all bundles so I filter them out, but this means that that anything that is not a test helper gets broken up as if it is a test. Obviously this is incorrect.
We can reverse the logic to assume anything that is not a test should be included in all bundles. A PR would be welcome, otherwise I will try to get to this sometime this week. (Also need to write tests for this at some point.)

Released v0.1.1, should be fixed.

:tada:


Alter warning message in _verify_python3_env

This warning could easily cause confusion on py3 only programs.
raise RuntimeError('Click will abort further execution because Python 3 '
                       'was configured to use ASCII as encoding for the '
                       'environment.  Either run this under Python 2 or '
                       'consult http://click.pocoo.org/python3/ for '
'mitigation steps.' + extra)

Simply changing the last sentance to 'Consult http://click.pocoo.org/python3/ for mitigation steps.' + extra'  should make it work well enough as a last resort error message.

It might be good to even just split the function up so that the output is separate from the input. That'd make it easier for folks to use the locale detection, but still provide links to their own documentation.

The part you are referring to is more targeted towards developers than end users, and the click docs page is too. I think it would make sense to do the change you propose and split the docs page into sections for devs and users each.

I think the message should be changed either way, because some users are gonna run into it because the devs forgot to implement their own logic (or call the new split function I talked about) to handle this case.

Fair enough, please open a new PR
On 22 September 2016 21:46:36 CEST, Johnny Robeson<EMAIL_ADDRESS>wrote:

I think the message should be changed either way, because some users
are gonna run into it because the devs forgot to implement their own
logic (or call the new split function I talked about) to handle this
case.
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/pallets/click/issues/659#issuecomment-249008290

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

Fixed in commit PR #661


Problems with scrolling

I am trying to use List, something like this
<AutoSizer>
  {({ height, width }) => (
    <List
      width={width}
      height={height}
      rowHeight={48}
      rowRenderer={({ index }) => (

Item {index}

      )}
      rowCount={rows.size}
    />
  )}
</AutoSizer>

But instead of smoothly scrolling I get terribly jerky on chrome 61.0.3163.100 https://www.screencast.com/t/I7yRYcu71
on FF 55.0.3 I had empty space at the end https://www.screencast.com/t/JOT1Bswz
Without AutoSizer the problem doesn't disappear.
I'm using the latest versions of react and react-virtualized.
What am I doing wrong?
Who faced such a problem?

Same problem here. Introduced with 9.10.0. Downgrading to 9.9.0 fixes the problem.

Same here

Unfortunetely, even rolling back to version 9.9.0, and then 9.0.0 and 8.11.4, didn't solve the problem. I used version 7 for a long time and wanted to try the latest version in the new project. On version 7 there is no such problem. I have no ideas.

@yuristrelets That's because you should pass style and key (not index) props to your div
      rowRenderer={({ style, key, index }) => (

Item {index}

      )}

@TrySound You're right! My fault, I didn't see that it's necessary to pass style ad key. Downgrading to version 9.9.0 and changing rowRenderer solved the problem. Thank you!

I have experienced this problem with v9.10 downgrading to 9.9.0 fixed the issue.

We are seeing the same issue on 9.10.1 as well.

Guys here is the fix https://github.com/bvaughn/react-virtualized/pull/825

Fixed by #825
Will be released momentarily as 9.11

react-virtualized ^9.20.1 still has this issue.
After add some dummy style it is resolved.
Please reopen this issue or at least mark "style" as a compulsory parameter to avoid this.
env:
mac 10.13.4
chrome 67.0.3396
node 9.2.0
yarn 1.7.0

Facing the same issue in the react-virtualized ^9.21.0 also :(
My code's snap like:
<List
width=800  height=50  rowHeight=600
rowRenderer={
({ index, style, key, ...rest }) => {
return (
<Component
key={key}
style={style}
{...this.props.items[index]}   />
)
}
}
rowCount={items.length}
/>
Env:
Windows10
yarn version v1.12.3
Chrome Version 71.0.3578.98
Please suggest me if I'm missing anything here.

I have the same issue as @yuristrelets<EMAIL_ADDRESS>on Chrome 75.0.3770.100. Any news? Taking into account last 3 unresponded comments.

Same problem

Same here, Chrome Version 76.0.3809.132 (Official Build) (64-bit). Safari works fine.
<EMAIL_ADDRESS>

Adding one more to the pile. I am still seeing this problem in Chrome on 9.21.2.

I used this component and modified it
https://github.com/bvaughn/react-virtualized/blob/master/source/Table/defaultRowRenderer.js

Was experiencing this on Chrome with 9.21.2.
Adding style={{width: "100%", height: "100%"}} to the <AutoSizer> component itself has fixed this for me weirdly.

I'm experiencing this issue with the AutoSizer and virtualized table. Sadly style tags haven't helped me and I'm experiencing a very jerky table. Has anyone figured out more about whats going on here?

In case anyone is having this issue, passing the style down to whatever actual div is rendered for the row is what fixed this for me.
I had originally passed down the style property to the row component, but I didn't use that property anywhere in the row.
I then passed it down to the base row component which passed the style to the actual div creating the row.
Hope this helps anyone else.

facing same issue, using "react-virtualized": "^9.22.3"

I also

I also face the same issue, can anyone help?

still reproducing

@bvaughn Can you please update if this issue is fixed with any updates, as we're facing the same issue where scrolling from top to bottom, screen is getting blank for fewer milliseconds distorting the user experience.

any update on this??


[SeaTunnel #967][seatunnel-connector] Add neo4j spark source

Purpose of this pull request

add neo4j source
Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

Note that this neo4j seems to be an open source agreement of GNU. Will this conflict with the Apache open source agreement? If there is no conflict, should a declaration file be added? @CalvinKirs

Note that this neo4j seems to be an open source agreement of GNU. Will this conflict with the Apache open source agreement? If there is no conflict, should a declaration file be added? @CalvinKirs

you are right, which is not compatible with AL2.

Sorry, I will close this PR first. Thanks again for your contribution.

i had check it before i am writing code , this jar  was on Apache-2.0 License.

sorry, I read the wrong repository.
This is my rough check result
Apache XBean :: ASM 6 shaded (repackaged)
From: 'OW2' (http://www.ow2.org/)

asm (http://asm.ow2.org/) org.ow2.asm:asm:jar:6.2.1
License: BSD  (http://asm.ow2.org/license.html)
asm-analysis (http://asm.ow2.org/) org.ow2.asm:asm-analysis:jar:6.2.1
License: BSD  (http://asm.ow2.org/license.html)
asm-commons (http://asm.ow2.org/) org.ow2.asm:asm-commons:jar:6.2.1
License: BSD  (http://asm.ow2.org/license.html)
asm-tree (http://asm.ow2.org/) org.ow2.asm:asm-tree:jar:6.2.1
License: BSD  (http://asm.ow2.org/license.html)

From: 'The Apache Software Foundation' (http://www.apache.org)

Apache XBean :: ASM Util (http://geronimo.apache.org/maven/xbean/4.10/xbean-asm-util) org.apache.xbean:xbean-asm-util:bundle:4.10
License: The Apache Software License, Version 2.0  (http://www.apache.org/licenses/LICENSE-2.0.txt)

@wuchunfu @CalvinKirs CC


remove unhandled key:val tags before rendering

I use a synchronisation script that syncs my todo.txt file with a remote provider (todoist). The script adds some specific tags (mostly ids) that shouldn't be rendered in task title (according to todo.txt specs).
These change allow to strip all these unhandled tags from being in the task title.

Can't pull this as it is.
There is already a regex for recognizing key-val pairs: REG_TODO_EXT. The correct way to get rid of extension tokens that weren't recognized is to add an else statement into the the parse_task_str func here: https://github.com/zagortenay333/timepp__gnome/blob/3be06b0677bdfe6f220a61f4b26f411762d20f30/sections/todo/task_item.js#L431
The other place where you're removing them is redundant.
Also, please don't use tabs for this project; it's 4 spaces.


Suppress "Checking for Hue Bridge firmware updates…" in node-red-log

Thank you very much for the recent update 4.x. Migration to this went smooth in my setup!
After running this for a while I've found out  there is an info message written to the node-red-log each five minutes:
...
12 Jan 12:24:57 - [info] [hue-bridge:Philips hue II] Checking for Hue Bridge firmware updates…
12 Jan 12:29:58 - [info] [hue-bridge:Philips hue II] Checking for Hue Bridge firmware updates…
12 Jan 12:34:58 - [info] [hue-bridge:Philips hue II] Checking for Hue Bridge firmware updates…
...

Describe the solution you'd like
I have recognized there is a checkbox in the bridge node configuration property dialog

Guess the intention is, if this is switched off, the check for update shouldn't be performed.
Please suppress the info msg mentioned above, at least in case this property is switched off. We all know which update cadence is provided be Philips. So it is not really helpful having  so much useless information documented in the log.
Additional context
I'm running a rsyslog service and the log size increases over the time to much witch less meaningful information.

You are right, I will suppress this information completely in the next version. For now what I meant with fixed is, that HueMagic will not output this message that frequently as on the versions before.


Puppet scripts for wso2-obbi worker and dashboard profiles

Purpose

The updated puppet scripts for OBBI 2.0.0

Goals

Describe the solutions that this feature/fix will introduce to resolve the problems described above

Approach

Describe how you are implementing the solutions. Include an animated GIF or screenshot if the change affects the UI (email<EMAIL_ADDRESS>to review all UI text). Include a link to a Markdown file or Google doc if the feature write-up is too long to paste here.

User stories

Summary of user stories addressed by this change>

Release note

Brief description of the new feature or bug fix as it will appear in the release notes

Documentation

Link(s) to product documentation that addresses the changes of this PR. If no doc impact, enter “N/A” plus brief explanation of why there’s no doc impact

Training

Link to the PR for changes to the training content in https://github.com/wso2/WSO2-Training, if applicable

Certification

Type “Sent” when you have provided new/updated certification questions, plus four answers for each question (correct answer highlighted in bold), based on this change. Certification questions/answers should be sent to<EMAIL_ADDRESS>and NOT pasted in this PR. If there is no impact on certification exams, type “N/A” and explain why.

Marketing

Link to drafts of marketing content that will describe and promote this feature, including product page changes, technical articles, blog posts, videos, etc., if applicable

Automation tests

Unit tests

Code coverage information

Integration tests

Details about the test cases and coverage

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes/no
Ran FindSecurityBugs plugin and verified report? yes/no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes/no

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

Migrations (if applicable)

Describe migration steps and platforms on which migration has been tested

Test environment

List all JDK versions, operating systems, databases, and browser/versions on which this feature/fix was tested

Learning

Describe the research phase and any blog posts, patterns, libraries, or add-ons you used to solve the problem.

Please fix the commit message of the commit 268f997. Since it has more than 50 characters in length limit the commit message to a shorter message and add a description explaining the commit. https://chris.beams.io/posts/git-commit/#limit-50

Please fix the commit message of the commit 268f997. Since it has more than 50 characters in length, limit the commit message to a shorter message and add a description explaining the commit. https://chris.beams.io/posts/git-commit/#limit-50


AttributeError: 'NoneType' object has no attribute 'pause'

AttributeError: 'NoneType' object has no attribute 'pause' occured
while using decorator ignore_function  as decuments ignore_function suggested.
It seemd that the default tracer was not initialized. And if I pass a tracer  as param into ignore_function it worked well.
from viztracer import VizTracer, ignore_function

@ignore_function
def f():
    n = 0
    for i in range(1, 100):
        for o in range(1, 20):
            n += (i+0)
    print(n)

with VizTracer(max_stack_depth=3, ignore_c_function=True):
    f()

traceback error:
Traceback (most recent call last):
  File "C:/Users/PycharmProjects/oneway_test/test.py", line 12, in <module>
    f()
  File "D:\virtual\Envs\venv37\lib\site-packages\viztracer\decorator.py", line 17, in ignore_wrapper
    tracer.pause()
AttributeError: 'NoneType' object has no attribute 'pause'

I'm working with python3.7.7 and platform is [MSC v.1900 64 bit (AMD64)] on win32

This is a valid bug. The cause is a commit a couple of days ago when I tried to avoid direct call to the underlay structure.
@ignore_function needs a tracer object now, and it will try to read the undocumented global_tracer first, which will be set if viztracer is executed through command line.
For now, passing the tracer into @ignore_function could be the temporary solution. I will work on some utilities to register global tracer explicitly because this might not be the only case where a function requires the tracer.

I realized that I actually documented in the decorator API that @ignore_function should only be used without arguments from command line, but I forgot to mention it in the general introduction. However, I made this work with inline usage as well in #43. If you still have issues, please tell me and reopen this.


Better errors if place_module cannot place a module.

See issue #339

This looks looks great, and I understand what you mean now. Thank you!


Creating a switch to disable ICMP Ping instead of the default - allow <IP_ADDRESS>/0

Currently kube-aws includes a rule in its security group to allow ping from all traffic (<IP_ADDRESS>/0). Unless this is necessary for a reason I have not foreseen, it is an inherent security risk, violates CIS compliance, and is probably superfluous if SG's are managed appropriately.
Therefore, I have created a switch in cluster.yaml to allow removal of this rule. I have left it enabled by default but am conscious that anyone upgrading needs to ensure this is in their cluster.yaml, so am happy to adjust the logic so that there's no impact unless you explicitly ask for it to be disabled.
I don't think I've missed any other spots where this rule is used.

Hey @HarryStericker, you need to sign the CLA before I can merge this :)

Yup. I signed it last time I made commits to this repo, but somethings fudged. Before i fixed it though, I wanted to get eyes on the change and make sure it's sane!

Clsoing this PR. Raised a clean one, since cla is in a funk.


Vector format for design files

The design references in https://github.com/MicrosoftDocs/msteams-docs/tree/master/msteams-platform/assets/downloads appear to be all bitmap, making them unsuitable for importing into Sketch, Adobe Illustrator, etc.
Please upload the vector originals so that we can put these resources to use.

I asked our designers if this was practical and unfortunately it's not. Our source files have all sorts of stuff in them that's unrelated to what we published here.


Add a dynamic dependancy in order for a test to run

Describe the feature:
Due to the nature and design of many estates some features only exist on certain servers. To make this more dynamic and extendable as well as portable and reuseable. that allows and enables the same config to be run across entire estates rather than individual files per possible server setup.
Problem:
We only need to confirm some server spec if a certain value is true
e.g.
1/ if auditd is installed
2/ then run the following tests.
It could be seen as setting a variable on the fly and ensure the variable value as part of the test. This would also mean certain test would need to run in order. Although as per #742 the subsequent tests may reuse the variable multiple times (example tries to show this too).
This feels more like a programming feature but could extend it usage.
Describe the solution you'd like
A possible way,
add a new option of discovery which runs prior to any other tests to capture and set dynamic variables used by the server spec tests.
These could even be in a dedicated file ensuring that the run before all others.
An example could be (this is very ansible like i am aware)
Example 1
file name: discovery.yml
discovery:
  package:
    auditd:
      installed: true
      register: auditd_installed

sometest1.yml
{{ if .Discovered.auditd_installed }}
file:
  /etc/auditd/audit.conf:
    exists: true
    contains:
     - 'some string = 1'
    meta:
      test: 1
 {{ end }}

sometest2.yml
{{ if .Discovered.auditd_installed }}
file:
  /etc/auditd/audit.conf:
    exists: true
    contains:
     - 'enable = true'
     meta:
      test: 2
 {{ end }}

Aware this only gives a boolean value rather than a discovered output. It would be nice if you could capture the other outputs and assign a value. e.g.
Example 2
file name: discovery.yml
discovery:
  package:
    auditd:
      installed: true
      register: auditd_installed
      version: 1.1

sometest1.yml
{{ if .Discovered.auditd_installed.version < "1.1" }}
file:
  /etc/auditd/audit.conf:
    exists: true
    contains:
     - 'some string = 1'
    meta:
      test: 1
 {{ end }}

sometest2.yml
{{ if eq .Discovered.auditd_installed.version >= "1.1" }}
file:
  /etc/auditd/audit.conf:
    exists: true
    contains:
     - 'enable = true'
     meta:
      test: 2
 {{ end }}

Describe alternatives you've considered
This could met by using the command test module, but then that is just running a shell script. It doesn't add the efficiency and consistency in the modules already developed.
Thanks again for this project.

Thanks for the thorough explanation.
Reflecting on this, let's approach it from the other end and see where the gaps are.
Today goss can leverage external discovery tools as follows:

$ goss --vars <(ohai) validate
$ goss --vars <(facter -j) validate

doc (for the above): https://github.com/aelsabbahy/goss/blob/master/docs/manual.md#examples-2
These tools are dedicated to system discovery and have been around for quite some time, thus having quite a bit of maturity and a rich feature set (ex. pluggability). The same can be said for ansibles fact engine and its plugins.
To better understand this enhancement, where do those tools fall short today as an end user?

Perhaps there's a middle-ground opportunity here:

goss --vars <(goss validate -g discovery.yaml --format discovery) validate

hi @ekelali
Thats a great solution. It allows goss to be the one required tool (which is perfect in restricted environments).

@uk-bolly Does this mean this can be closed now?
Should there be a documentation update to show this example?

--format discovery does not exist in Goss today. Someone would have to implement that before this ticket is closed.
There's --format json but I don't think that meets @uk-bolly need.


Add matching support for \p{Lc}

This is defined in UAX#44 as being equivalent to Lu | Ll | Lt. Also add support for \p{L&} which is the PCRE spelling of \p{Lc}.

@swift-ci please test


Go To Symbol In File

I often find myself needing to point another developer to a specific function or other symbol, which is frequently located in a file containing numerous other definitions. Usually, I would approach this by sending them a relative file path along with the symbol name. For example:

relative file path src/utils.js
symbol name capitalize

Equipped with this information they could:

copy the file path their clipboard
raise VSCode
execute Go to file command (usually bound to ctrl+e)
paste
press Enter
execute Go to symbol command (in my case it is bound to ctrl+shift+o)
type in the before mentioned symbol name

Instead it would be convenient if the Go to file command input could accept a path with a symbol simplifying the above steps. Alternatively, to avoid overloading Go to file a new command my be introduced, for example Go to symbol in file.
The syntax for the enhanced file path could be filePath@symbolName. In the context of the example above: src/utils.js@capitalize.

You can type this but you cannot just paste such a path.


add default username and password on login page

add a comment like  on login page:
Login as Admin with:
admin
dotadmin
Login as User with:
<EMAIL_ADDRESS>password

You can add a drop-down there maybe to select if you login as
an user or an admin

let's login only as admin
and use datatables


upgrade httpx for snapshot_reporter

What does this change?
Part of https://github.com/wellcomecollection/platform/issues/5796
https://github.com/wellcomecollection/catalogue-api/security/dependabot/43
This upgrades httpx to the latest version
Checklist

[ ] Does this patch need a change to the documentation?
[ ] Do you need to update the Catalogue API Swagger?

How to test
There are no tests 😢
Wait until the morning to see the report in #wc-search-alerts?
How can we measure success?
One fewer critical vulnerability in the catalogue-api
Reports are posted in #wc-search-alerts in the morning ☕
Have we considered potential risks?
This is just the part of the snapshots that posts the reports so not very risky, in the grand scheme of things

I am not sure this get deployed automatically?


fix windows login-agent resource not found window redirection issue

@sameeragunarathne do we still need this? please verify

we don't need this exclusion anymore.


Would like to see support for XMLSerializer object and document.implementation.createDocument and eq...

Would like to see support for XMLSerializer object and document.implementation.createDocument and equivalents for ID.

This issue was imported from Google Moderator
Moderator votes: +3
Issue added by Bron on 2014-09-03
To vote this issue up or down, simply include +1 or -1 in your comment.

Available at https://caniuse.com/mdn-api_xmlserializer and https://caniuse.com/mdn-api_domimplementation_createdocument


Install java fails with error

brew cask install java fails with error;-
Error: pkg source file not found: '/opt/homebrew-cask/Caskroom/java/1.8.0_66-b17/JDK 8 Update 66.pkg'

Please always follow the reporting bugs guide. To do so, edit the top post and then comment after this one so I can check back.
We’ve just undergone a major change, so please also brew uninstall --force brew-cask && brew update.
Closing in the meantime, because a number of people never report back and it’s useless to keep the issue open in that case and without all the required information.


Add test reporting for cucumber tests

Purpose
$subject

Codecov Report
Patch coverage has no change and project coverage change: +55.26% :tada:

Comparison is base (27e03f1) 24.84% compared to head (291134b) 80.10%.
Report is 8 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1490       +/-   ##
===========================================
+ Coverage   24.84%   80.10%   +55.26%
===========================================
  Files         266        9      -257
  Lines       31091      548    -30543
  Branches     4830      246     -4584
===========================================
- Hits         7724      439     -7285
+ Misses      23101      109    -22992
+ Partials      266        0      -266

Flag
Coverage Δ

adapter
?

common-controller
?

config-deployer-service
?

idp-domain-service
80.10% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 257 files with indirect coverage changes

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


Use MARIADB_ Environment Variable Equivalent for Mariadb

In MariaDB, MARIADB_* environment variables have a higher preference to MYSQL_* variants.
The project only lists the MARIADB_* variants on the Docker hub page. The Knowledge Base also shows a clear bias towards MARIADB_* variants.
MYSQL_* environment variables work with MariaDB and there is no indication that this will change soon. I would therefore understand if you wish to retain MYSQL_* variants.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Go: add support for empty tags-set when loading saved model

There is a check in saved_model.go that does not allow empy tag set, however there are models without tags that work perfectly in the library without providing them (they are optional in the Python library too, for example).
Apparently the check was added in #36466, though instead of throwing an error on an empty set, a nil value can be passed in these cases and the C library will not complain. Otherwise, the pointer to the first element is passed as before.

Hi @gharibian Can you please review this PR ? Thank you!

Hi @gharibian Can you please review this PR ? Thank you!

Hi @gharibian Can you please review this PR ? Thank you!

Hi @gharibian Can you please review this PR ? Thank you!

Hi @gharibian Can you please review this PR ? Thank you!

Hi @gharibian Can you please review this PR ? Thank you!

Hi @gharibian Can you please review this PR ? Thank you!

A test is failing, so we cannot merge this. Can you take a look please?
tensorflow/go/saved_model_test.go:100: LoadSavedModel() failed with an empty tags set: Could not find meta graph def matching supplied tags: {  }. To inspect available tag-sets in the SavedModel, please use the SavedModel CLI: `saved_model_cli`

@mihaimaruseac I've added a version of half_plus_two with the tags removed. Can you please try to run the test again?
I cannot seem to run it on my ARM64 Mac.


[5.3] Adding hasIndex to the schema builder

Attempt two, this time in the 5.3 branch.  Thanks @GrahamCampbell.

Your getDoctrineIndexes method @return block says it returns an Index, but then you treat it like a collection or something by using map later in the code. I'm confused how this would even work?

It returns an array of Index, just missing [].

Please resend this when it's ready. It appears to have bugs right now. doc types are wrong and you appear to be calling map on a plain array.


data-tip only support string?

I have structure layout issue, so I use 'data-tip' to pass content into <ReactTooltip  />, like , but it will show [object object]. If I put data-tip="
test
", it will get the same result [object object]. Is it possible to add the feature?

Is this what you are looking for? That's already implemented. Check the spec.

cool! It works, thank your

@duckscofield How did you get this to work? I'm having the same problem. I tried the html={true} as posted above but I get the same result.

@paulcredmond
In my case, I use the code like as bellows:
<YourComponentName
...
data-for="one id name"
data-html
data-tip={

}
...
/>

Thanks for the reply @duckscofield
Mine is a little more complicated as I have unparsed tags, but I figured it out using the following:
 const unParsedHtml = "<p>Thank you for your message</p>";
 const htmlDecode = (input) => {
      const e = document.createElement('div');
      e.innerHTML = input;
      return e.childNodes.length === 0 ? "" : e.childNodes[0].nodeValue;
 }

<ReactTooltip class="react-tooltips" place="right" effect="solid">

</ReactTooltip>

It doesn't work for me either. Displays [object Object]. Tried setting data-html to true. But still no luck.

Found this thread - https://github.com/wwayne/react-tooltip/issues/329. The data-tip doesn't support jsx.


[Feat Req] Add a picker that would output a formatted citation

For people using telescope-bibtex for note taking (in markdown for instance), it would be nice to be able to format the citation directly (following some of the most common conventions, such as APA, Chicago, etc).
That way, it would be extremely simple to generate notes of that sort:
## My note
Lorem ipsum dolor sit amet, consectetur adipiscing elit.[@citekey].

## References
[@citekey]: Name F., 2022, "My awesome source", Journal of Neovim, Paris, France

Some configuration variables could be left to the user, such as:

the choice of the convention: APA, Chicago, ...
the output format: only_citation, [@citekey]: citation, ...

So, you whish to produce the text unter References by parsing more details from the entry?
If so, this should be relatively easy to acomplish. I will have a look at it (hopefully this weekend) after we merge the changes to the pickers from #31

I love the idea of Telekasten. Currently I still use vimwiki because I got exams around the corner and do not want to move all my notes, but definitly will check it out some time.
Telescope-bibtex should not need to do extra formating of entries if I understood this correctly? It just extracts the fields and display those specified by the convention or the user and maybe adds extra characters like (year) or "title".
The plan would be to support the formats you mentioned (APA, Chicago and maybe one more) out of the box and then provide a user specific template.

Telescope-bibtex should not need to do extra formating of entries if I understood this correctly? It just extracts the fields and display those specified by the convention or the user and maybe adds extra characters like (year) or "title".

Yes. Scape the bibtex entry for the selected citation, concatenate all information scraped all in a long string accordingly with the style chosen in the options and then paste that.

The plan would be to support the formats you mentioned (APA, Chicago and maybe one more) out of the box and then provide a user specific template.

The best future-proof solution would be to implement CSL (Citation Style Language), which is the "standard" with all citation styles. This is basically what Zotero, Mendelay, etc use. It seems that a lua implementation exists (see here) but I have not tested it. That would support all styles out-of-the-box. But this may also be a bit trickier to implement.
In the meantime, I think that implementing 2-3 very common styles and add the possibility to have user-defined template would likely satisfy most use-cases.

The best future-proof solution would be to implement CSL (Citation Style Language), which is the "standard" with all citation styles. This is basically what Zotero, Mendelay, etc use. It seems that a lua implementation exists (see here) but I have not tested it. That would support all styles out-of-the-box. But this may also be a bit trickier to implement.
In the meantime, I think that implementing 2-3 very common styles and add the possibility to have user-defined template would likely satisfy most use-cases.

This sounds like a good plan. Having the common styles implemented to have this feature for people to use it and then try to make CSL work.

As you are open to the idea, I will try to find some time this week to do a basic implementation of the feature.

That would be great. I will do what I can to help.

Solved by #33


Restaurants demo produces two suggestions

Steps to repro (hopefully):

load shell_0_6_0/web-shell
type 'restaurants' into search box

Two suggestion are produced.
StrategyExplorer isn't working for me atm so it's hard to root-cause the ambiguity. Thought I would note it here so we can record any interesting discussion. :)

These suggestions differ in slot assignments, maybe the recipe needs to be a bit more specific about the slots.
One recipe is:
recipe RestaurantsDemo
  create #event as handle0 // Event {...}
  create as handle1 // [Restaurant {...}]
  create #restaurant as handle2 // Restaurant {...}
  create as handle3 // Geolocation {...}
  create as handle4 // [~a] // [Description {...}]
  slot 'rootslotid-modal' #modal as slot3
  slot 'rootslotid-root' #root as slot4
  AnnotationMultiplexer as particle0
    annotation = handle0
    hostedParticle = ReservationAnnotation
    list <- handle1
    consume annotation as slot1
  DetailSlider as particle1
    selected = handle2
    consume modal as slot3
      provide content as slot2
  FindRestaurants as particle2
    location <- handle3
    restaurants = handle1
  Geolocate as particle3
    location -> handle3
    consume root as slot4
  ReservationForm as particle4
    descriptions -> handle4
    event = handle0
    restaurant <- handle2
    consume action as slot0
  RestaurantDetail as particle5
    restaurant <- handle2
    consume content as slot2
      provide action as slot0
  SelectableTiles as particle6
    list <- handle1
    selected = handle2
    consume root as slot4
      provide action as slot5
      provide annotation as slot1
      provide tile as slot6
  TileMultiplexer as particle7
    hostedParticle = RestaurantTile
    list <- handle1
    consume tile as slot6
  description `[demo] find restaurants and make reservations near ${FindRestaurants.location}`

and the other is:
recipe RestaurantsDemo
  create #event as handle0 // Event {...}
  create as handle1 // [Restaurant {...}]
  create #restaurant as handle2 // Restaurant {...}
  create as handle3 // Geolocation {...}
  create as handle4 // [~a] // [Description {...}]
  slot 'rootslotid-modal' #modal as slot4
  slot 'rootslotid-root' #root as slot5
  AnnotationMultiplexer as particle0
    annotation = handle0
    hostedParticle = ReservationAnnotation
    list <- handle1
    consume annotation as slot1
  DetailSlider as particle1
    selected = handle2
    consume modal as slot4
      provide content as slot2
  FindRestaurants as particle2
    location <- handle3
    restaurants = handle1
  Geolocate as particle3
    location -> handle3
    consume root as slot5
  ReservationForm as particle4
    descriptions -> handle4
    event = handle0
    restaurant <- handle2
    consume action as slot0
  RestaurantDetail as particle5
    restaurant <- handle2
    consume content as slot2
      provide action as slot3
  SelectableTiles as particle6
    list <- handle1
    selected = handle2
    consume root as slot5
      provide action as slot0
      provide annotation as slot1
      provide tile as slot6
  TileMultiplexer as particle7
    hostedParticle = RestaurantTile
    list <- handle1
    consume tile as slot6
  description `[demo] find restaurants and make reservations near ${FindRestaurants.location}`

Would you like me to try tweak the recipe?

Thanks for the PR, but the reason I marked this Developer Weirdness is because we have to discuss the ergonomics. The goal isn't a system that can be made to work, but a system that works smoothly, so the developer can concentrate on the novel problem they are solving and not get stuck managing arcana.

It looks slightly better now, when I removed most of the slot bindings. The only ones necessary are related to action slot - I suspect this is needed because there are 2 particles that provide slot named action, so there are multiple possibilities for the action consumer.
I agree, there still is a lot of opportunities to improve in this area.


add team ranks to scoreboard

add a teams current ranking to event cards:

old games should display old rank.
future games should display current rank.

closed via 1243ebc


Make Examples Live?

It'd be really useful to have the example live, even hosted under github.

Maybe it's time we just moved the examples to their own repo and they can live on github pages. Less code in the project is :+1: from me

I think the code is broken, component is not defined in the file, and unlikely a function:
https://github.com/dekujs/deku/blob/master/examples/events/index.js#L26
var MouseDown = component(function(props, state, send){
  function down() {
    send({ down: true });
  }
  function up() {
    send({ down: false });
  }
  var classes = {
    'active': state.down === true
  };
  return dom('div.box', { class: classes, onMouseDown: down, onMouseUp: up }, [
    'Mousedown / Mouseup'
  ]);
});


Add support for struct arguments in dtrace probes

It would be nice to be able to see the contents of any structure, not just it's address.

Consider the following API:
typedef struct {
    int hash;
    const char *definition;
} entry_t;

char *copy_entry_definition(entry_t *entry, int len);

One should be able to describe entry_t type in the API definition (or somewhere globally) like so:
"entry_t" : [
    {"field_name": "hash",       "field_type": "int"},
    {"field_name": "definition", "field_type": "string"},
]

and receive a report similar to this from Cuckoo:
          API          |               Arguments                   |     Status
-----------------------|-------------------------------------------|------------
copy_entry_definition  | entry: {hash => 971, definition = "hey"}  | SUCCEED
                       |     n: 128                                |

WIP branch: struct-argument.

In other news: I've decided to go with YAML for type descriptions:
kkk_t:
    native: No
    struct:
        abc: "double *"
        hfa: "size_t"
        sss: "char *"

unsigned long long: &unsigned-long-long
    printf_specifier: "%llu"
    native: Yes

"void *": *unsigned-long-long

I believe it's more human-readable than JSON.


Updated to meteor 1.2.1

Needed to update to latest version for some changes coming soon.

@davidlago you had mentioned some problems with meteor's Heroku build pack. Does the Meteor version affect it at all?

Yes @radhikamalik... it scares me a bit, we should try to do a test deployment to heroku (setting up a different app) and see if it works.

I'll try setting up on a separate heroku instance.  @davidlago I'll see if I hit any issues with the current build pack on a seperate heroku instance.  I also see that there is the Horse buildpack, I'll try it out and report back.

Did a deploy to my local heroku instance http://mbta-ninja-staging.herokuapp.com/ which is working well.  Followed our deployment doc on the readme and didn't really run into any issues. @davidlago Do you remember what issues you hit?

Ok now it's ready to go, Fixed and squashed my changes into the one commit.  Heroku instance is up and running fine. http://mbta-ninja-staging.herokuapp.com

Thanks Patrick! The problem we faced a couple of months ago prevented the application from starting up on heroku with that build pack. So long as it is up an running in your branch, and it looks like it is, I think we should be all set. Thanks so much!!

On Dec 1, 2015, at 11:22 PM, Patrick Greenwell<EMAIL_ADDRESS>wrote:
Ok now it's ready to go, Fixed and squashed my changes into the one commit. Heroku instance is up and running fine. http://mbta-ninja-staging.herokuapp.com http://mbta-ninja-staging.herokuapp.com/
—
Reply to this email directly or view it on GitHub https://github.com/codeforboston/mbta-ninja/pull/150#issuecomment-161177072.


SASL: "verbose no" does not disable verbose mode

On ZNC 1.6.3 the new verbose yes|no command for the SASL module does not allow me to disable verbose mode:

<Me> Verbose no
<*sasl> Verbose: true

This is on Debian 7.11.
Ref: #770 & 66daadfee8f7500311be73ea0480b9eb7aad6b82

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/35848050-sasl-verbose-no-does-not-disable-verbose-mode?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github).

https://github.com/znc/znc/blob/ae803769c2e51831777aa8dc327d8b36a13e1f4c/modules/sasl.cpp#L79
Replace m_bVerbose = sLine.ToBool(); with m_bVerbose = sLine.Token(1, true).ToBool();. No, I'm too lazy to submit a pull request.
(The code tries to turn "verbose no" into a boolean, but it should only try to turn "no" into a boolean)

As a workaround for 1.6.3, /znc reloadmod sasl would disable the verbose mode.

Awesome! Thanks for the extremely fast movement in this.


Implementation of WYSIWYG Editor / Colored Text

I have 2 issues I am working on:

I'd like to implement a WYSIWYG Editor based on Markwon.

Switching from Plaintext to rendered Markdown is no problem.
I like to have one view with buttons/functions so the user does not need to write Markdown or switch the views
How to achieve that? Is there anything possible with the use of Markwon?

I'd like to be able to change the text color of normal text and to save it. Markdown is not capable nor designed to implement something like color. Are there any suggestions on how to achieve that? Maybe something like adding custom attributes/values to the markdown file/string and render them with a custom Markwon parser?

Hello @NKnuelle !

WYSIWYG editor requires a lot of work. Well, good one for sure. As I see it Markwon can help you for initial render. Meaning that markdown is used only to persist/transmit user input, but actual representation uses something different. For example your custom spans. After user finished his/her input you just create a markdown from it and use it in the future.
There is also a possibility to use Markwon for continuous  parsing/rendering when user input changes. But. You won't be able to just setText after each change as actual input will feel and act wrong from user perspective (jumping text, loosing cursor position, freezes, etc). So you will have to come up with some diffing functionality that will apply changes to user input without modifying EditText.

It depends on your needs. If you want to change all text color in one widget I would recommend using TextView#setTextColor.
If you want to change portions of input you could also create own markdown syntax for this functionality, implement own parser and create a proper markdown visitor to render it.
You also could create a special HTML tag (or use existing), after all valid HTML is valid markdown. Create own HTML tag handler and use it along with custom HtmlRenderer.

Hi @noties!
I just mentioned I owe an answer.
Thanks for your detailed reply! At the moment I am using this library: Android HTML WYSIWYG and change some portions of it to my needs. This works quite good for now.
I like Markdown at the idea behind it. But I think it is not designed for a WYSIWYG user experience - It's just a simple and easy to learn Markup language that can be used to format text so that  no WYSIWYG editor is needed. But I need something different in my project and so that html editor seems to fit the most.
Thank you for this library - especially the documentation! It's something I miss on the library I use now.. :)


Fix code format

Remove tab between function name and parentheses

@hraz-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change. @markamos

Looks good; merging.


Error: GL_INVALID_ENUM at: ImGui_ImplOpenGL3_CreateFontsTexture

in SHA1 931a12e, Debian 9.9, AMD 64-bits, g++-6 (Debian 6.3.0-18+deb9u1) 6.3.0 20170516,
When starting tangram (I exported the API key), I have this runtime error and no displayed map:
TANGRAM map.cpp:874: setup GL
ERROR hardware.cpp:51: glGetString( GL_EXTENSIONS ) returned NULL
TANGRAM hardware.cpp:77: Hardware max texture size 16384
TANGRAM hardware.cpp:78: Hardware max combined texture units 192
TANGRAM map.cpp:207: resize: 1024 x 768
TANGRAM main.cpp:28: Base URL: file:///home/qq/tangram-es/build/
GL Error: GL_INVALID_ENUM at: ImGui_ImplOpenGL3_CreateFontsTexture
WARNING sceneLoader.cpp:687: Can't find data source touch for layer touch
WARNING tileManager.cpp:204: add source osm
TANGRAM scene.cpp:481: Fetch font https://fonts.gstatic.com/s/opensans/v13/O4NhV7_qs9r9seTo7fnsVLO3LdcAZYWl9Si6vvxL-qU.woff
TANGRAM scene.cpp:481: Fetch font https://fonts.gstatic.com/s/opensans/v13/wMws1cEtxWZc6AZZIpiqWALUuEpTyoUstqEm5AMlJo4.woff
TANGRAM scene.cpp:481: Fetch font https://fonts.gstatic.com/s/montserrat/v7/zhcz-_WihjSQC0oHJ9TCYL3hpw3pgy2gAi-Ip7WPMi0.woff
TANGRAM scene.cpp:427: Fetch texture file:///home/qq/tangram-es/build/res/img/poi_icons_32.png
TANGRAM scene.cpp:312: Prefetch tiles for View: 1024.000000x768.000000 / zoom:16.000000 lon:-74.009764 lat:40.705327
TANGRAM scene.cpp:432: Received texture file:///home/qq/tangram-es/build/res/img/poi_icons_32.png
TANGRAM scene.cpp:485: Received font: https://fonts.gstatic.com/s/montserrat/v7/zhcz-_WihjSQC0oHJ9TCYL3hpw3pgy2gAi-Ip7WPMi0.woff
TANGRAM scene.cpp:485: Received font: https://fonts.gstatic.com/s/opensans/v13/O4NhV7_qs9r9seTo7fnsVLO3LdcAZYWl9Si6vvxL-qU.woff
TANGRAM scene.cpp:485: Received font: https://fonts.gstatic.com/s/opensans/v13/wMws1cEtxWZc6AZZIpiqWALUuEpTyoUstqEm5AMlJo4.woff
TANGRAM map.cpp:186: ASYNC DISPOSE OF OLD SCENE - 1
TANGRAM scene.cpp:312: Prefetch tiles for View: 1024.000000x768.000000 / zoom:16.000000 lon:-74.009764 lat:40.705327
LOADED Font: Open Sans Italic size: 16
LOADED Font: Open Sans Regular size: 16
LOADED Font: DejaVu Sans Book size: 16
GL Error: GL_INVALID_VALUE at: ImGui_ImplOpenGL3_RenderDrawData
GL Error: GL_INVALID_VALUE at: ImGui_ImplOpenGL3_RenderDrawData
GL Error: GL_INVALID_OPERATION at: ImGui_ImplOpenGL3_RenderDrawData
GL Error: GL_INVALID_OPERATION at: ImGui_ImplOpenGL3_RenderDrawData

Hmm that's an interesting set of errors. It's not obvious to me what the issue could be. @Lecrapouille what OpenGL driver are you using?

My config:
OpenGL version: 4.3 (Core Profile) Mesa 13.0.6
GLSL version: 4.30
I have a fresh Debian 9.9 updated. Mesa 13.06 is a little old (2 years old) but this is not the problem
When I posted issue https://github.com/tangrams/tangram-es/issues/1980 It worked (I mean I saw the map). Now I just reverted to this commit d1e2f787ae01e3d7294673af26061527f17bc41a and its working back.
I'll try to do a git bisect. I'll give feedback later.

Sorry, I tried again and the bug disappeared. Since yesterday I have rebooted my computer. Probably the bug produced because I tried an experimental GitHub project messing my GPU or when I added the API key for the first time.

PS: I understand now what happened: I tried another github application and for using it, I add to export:
export MESA_GL_VERSION_OVERRIDE=4.5
export MESA_GLSL_VERSION_OVERRIDE=450

before launching it. Then I did not use a new terminal when I tried tangram-es. So the bug can be reproduced (of course in a new terminal these variables are no longer set and tangram workes well).

I reopen it, in case your are interesting by it, else feel free to close it.


Command Rejected

At FROM, for some servers, I get 550 Command Rejected even though manually writing 'MAIL FROM:xxx@xxxxx" says Ok.

There was some issue with my reverse IP, so it is fixed now.


improve POST fleet_telemetry_config error 404 vin not_found for driver

Fleet telemetry was working for me, but now I'm receiving a 404 vin not found when trying to (re)configure it on a vehicle which was working before.
endpoint: POST /api/1/vehicles/fleet_telemetry_config
reply: http 404
{
  response: null,
  error: '<REDACTED VIN> not_found',
  error_description: ''
}

are you sure you still have access to that VIN from your account?

Discovered that this error is happening because I'm trying to configure fleet telemetry using a Tesla account with DRIVER (non-owner) access to the vehicle.
Per #117 it would be great if drivers could enable fleet telemetry on a vehicle.
One factor in my confusion is #116 , which meant I could delete the fleet telemetry config using the driver account but could not reconfigure it.
Some suggestions for the devs:

the error message could be improved eg "unable to configure fleet telemetry with driver access". The current message is misleading because a driver account does have visibility of the vin
the docs could be improved to clarify that this endpoint is only available to accounts with owner access

@jbanyer DRIVER access will be supported soon.  We'll publish the change to the announcements section

@jbanyer how did you end up solving this? I am using my own vehicle for testing, and I had previously setup 3rd party, so I assumed fleet telemetry would be setup already.

Ensure your virtual key has been added to the vehicle you intend to configure. To add your virtual key to the vehicle, redirect the owner to https://tesla.com/_ak/your-domain.com. If using authorization code flow, the owner of the vehicle must have authorized your application with vehicle_device_data scope before they are able to add your key.

Either way, I also added my virtual key using the first approach key, and I was able to confirm that the key was added to my vehicle by checking the vehicle settings and the iOS app notification. However, I'm still getting the same error. Any ideas?

Any way, I also added my virtual key using the first approach key, and I was able to confirm that the key was added to my vehicle by checking the vehicle settings and the iOS app notification. However, I'm still getting the same error that my VIN is not_found. Any ideas?

@amirhmk are you attempting to configure fleet telemetry using a Tesla account which is the OWNER of the vehicle? My problem (and this ticket) was because I was trying to use a Tesla account with DRIVER access to configure fleet telemetry, which doesn't work currently.

Yes @jbanyer, I am using my own vehicle and account to get the fleet telemetry to work, but getting the exact same error as you. Now I'm wondering if my whole TLS / CSR setup is correct or even matters at this stage since I'm getting the same error if I change the config in some way.