Editing Polygons with Leaflet Draw

Is it possible to edit polygons with Leaflet Draw with snapping enabled?
I have it working when creating new polygons.  I have tried this code:
var polyline = L.polyline(...).addTo(map);
    polyline.snapediting = new L.Handler.PolylineSnap(map, polyline);
    polyline.snapediting.addGuideLayer(guideLayer);
    polyline.snapediting.enable();

Which doesn't work because there is no fill set.  When I replace the polylineSnap with polygonSnap it tells me it is not a function.
This is how I enabled snapping for creating a polygon:
	var guideLayers = [lakes];
	var drawControl = new L.Control.Draw({
		polygon: {
			showArea: true,
			guideLayers: guideLayers,
			snapDistance: 15,
			guidelineDistance: 10,
			shapeOptions: {
				color: shapeTypeObject[shapeType].color
			}
		}
	});
	createPolygon = new L.Draw.Polygon(map, drawControl.options.polygon);
	createPolygon.enable();

That works for creating but when I use the same method for editing it has no effect.

I have the same problem:
var drawnItems = new L.FeatureGroup();

var drawControl = new L.Control.Draw({
        draw: {
            polyline: {
                guideLayers: drawnItems
            },
            polygon: false,
            rectangle: false,
            circle: false,
            marker: {
                marker: { guideLayers: drawnItems }
            }
        },
        edit: {
            featureGroup: drawnItems
        }
    });

var routePolyline = new L.Polyline(routePoints, routePolylineOptions);
var routePolylineDrawn = drawnItems.addLayer(routePolyline);

routePolyline.snapediting = new L.Handler.PolylineSnap(Application.routes.map, routePolylineDrawn);
routePolyline.snapediting.addGuideLayer(drawnItems);
routePolyline.snapediting.enable();

Results in:
Uncaught TypeError: Cannot read property 'editing' of undefinedL.Edit.Poly.L.Handler.extend.addHooks @ leaflet.draw-src.self-3a363f7fa59b4aad45feb25ef226ed4b72b853f3bfccbaca9f91b367c0e02924.js?body=1:1154

It happens in this block:
if (!(poly instanceof L.Polygon)) {
    poly.options.editing.fill = false;
}

I use leaflet 0.7.3, draw 0.2.3 and the latest master file from snap.
Any ideas whats wrong?

I'm sorry I can't spend time to help you on this... (I don't work with these tools anymore)
Theorically it should work :)

This is a Leaflet.draw bug, the issue and workaround can be hound here:
https://github.com/Leaflet/Leaflet.draw/issues/364#issuecomment-74184767

I think #30 solve this.


Not able to get certificate, it says : Issuing certificate as Secret does not exist

Azure kubernetes:-
Hi Team,
if i do kubectl describe certificate  -n  getting the below error. Could you tell me how to fix it ?

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

ID: f53838fc-1654-fc18-7e45-7d556193f77e
Version Independent ID: 4f92bd5b-4b91-b03d-4f1d-8c31d45af0ef
Content: Use LetsEncrypt.org certificates with Application Gateway
Content Source: articles/application-gateway/ingress-controller-letsencrypt-certificate-application-gateway.md
Service: application-gateway
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

Sure sharing the link:- https://learn.microsoft.com/en-us/azure/application-gateway/ingress-controller-letsencrypt-certificate-application-gateway

@poshak-rialtes
Thanks for your feedback! We will investigate and update as appropriate.

@poshak-rialtes
For an issue like this, I'd recommend you create a support ticket since the support team will be able to respond much more quickly and have a conversation with you to figure out what could be going on.


[TOPI-ARM] Do not alter layout if layout is NHWC

@icemelon9 I think we missed this piece while working on op strategy.
Currently, anybody running NHWC conv on ARM results in a compilation failure - also reported at https://discuss.tvm.ai/t/issue-in-alter-conv2d-layout-for-arm-cpu/6383
There might be a way to improve the NHWC ARM schedule and use alter_op_layout to alter kernel layout, but it needs work. @jackwish @FrozenGene Can you take a look? Meanwhile, we can push this PR to suppress the error.

We could bring NHWC alter op layout after we improve the NHWC schedule. Currently, let us disable it like this pr. Thanks @anijain2305


Gdp 857

Major changes. Added the map. Added model change event handlers for the aoiName and aoiValues to draw the selected features of interest on the map. I am using a WMS request mostly because that's what the old GDP app did.  The majority of the interesting changes are in js/util/mapUtils.js and js/advanced/views/SpatialView.
Still to do would be to add tests for these new features. Next step will be to add the upload shapefile functionality.

:+1:


Cells executed from interactive window take multiple clicks to uncollapse

Testing #129288

Open an interactive window
Run some commands from the python file
Type pwd into the interactive window
Click the run button
🐛 Uncollapsing the newly created cell takes 4-5 clicks. This is different from the one click it takes for cells that are backed by the python file

Uncollapsing the newly created cell takes 4-5 clicks. This is different from the one click it takes for cells that are backed by the python file

@lramos15  Can you show a screen recording of this?

@rebornix Has recorded a gif over teams since my software wasn't working.


Calico Pod is Restarting on Debian10

We are trying to deploy Kubernetes 1.21.5 on Debian10 using Kubespray-2.17.0 .
Expected Behavior
All kube-system pods should come up and running.

Current Behavior
Calico-node pod is restarting continuously.

Error:
2022-02-17 07:19:23.576 [ERROR][76] felix/table.go 1004: Failed to program iptables, loading diags before panic. error=exit status 2 ipVersion=0x4 table="raw"
2022-02-17 07:19:23.579 [ERROR][76] felix/table.go 1010: Current state of iptables ipVersion=0x4 iptablesState="# Generated by iptables-save v1.8.4 on Thu Feb 17 07:19:23 2022\n*raw\n:PREROUTING ACCEPT [330909:980778210]\n:OUTPUT ACCEPT [188968:46377262]\n-A PREROUTING -d <IP_ADDRESS>/32 -p udp -m udp --dport 53 -j NOTRACK\n-A PREROUTING -d <IP_ADDRESS>/32 -p tcp -m tcp --dport 53 -j NOTRACK\n-A OUTPUT -s <IP_ADDRESS>/32 -p tcp -m tcp --sport 8080 -j NOTRACK\n-A OUTPUT -d <IP_ADDRESS>/32 -p tcp -m tcp --dport 8080 -j NOTRACK\n-A OUTPUT -d <IP_ADDRESS>/32 -p udp -m udp --dport 53 -j NOTRACK\n-A OUTPUT -d <IP_ADDRESS>/32 -p tcp -m tcp --dport 53 -j NOTRACK\n-A OUTPUT -s <IP_ADDRESS>/32 -p udp -m udp --sport 53 -j NOTRACK\n-A OUTPUT -s <IP_ADDRESS>/32 -p tcp -m tcp --sport 53 -j NOTRACK\nCOMMIT\n# Completed on Thu Feb 17 07:19:23 2022\n" table="raw"
Warning
2022-02-17 07:20:17.663 [WARNING][1330] felix/table.go 997: Failed to program iptables, will retry error=exit status 2 ipVersion=0x4 table="raw"
2022-02-17 07:20:17.679 [WARNING][1330] felix/table.go 1000: Retrying... error=exit status 2 ipVersion=0x4 table="raw"
2022-02-17 07:20:17.742 [WARNING][1330] felix/table.go 1324: Failed to execute ip(6)tables-restore command error=exit status 2 errorOutput="iptables-restore v1.8.4 (legacy): Couldn't load match rpfilter':No such file or directory\n\nError occurred at line: 8\nTry iptables-restore -h' or 'iptables-restore --help' for more information.\n" input="*raw\n:cali-from-host-endpoint - -\n:cali-to-host-endpoint - -\n:cali-PREROUTING - -\n:cali-OUTPUT - -\n-A cali-PREROUTING -m comment --comment "cali:XFX5xbM8B9qR10JG" --jump MARK --set-mark 0/0xf0000\n-A cali-PREROUTING -m comment --comment "cali:EWMPb0zVROM-woQp" --in-interface cali+ --jump MARK --set-mark 0x40000/0x40000\n-A cali-PREROUTING -m comment --comment "cali:mPIOOWmbH3iO0R90" -m mark --mark 0x40000/0x40000 -m rpfilter --invert --validmark --jump DROP\n-A cali-PREROUTING -m comment --comment "cali:8eOxmFpkWr0RjKXR" -m mark --mark 0/0x40000 --jump cali-from-host-endpoint\n-A cali-PREROUTING -m comment --comment "cali:F-nExspj_POyei0v" -m mark --mark 0x10000/0x10000 --jump ACCEPT\n-A cali-OUTPUT -m comment --comment "cali:njdnLwYeGqBJyMxW" --jump MARK --set-mark 0/0xf0000\n-A cali-OUTPUT -m comment --comment "cali:rz86uTUcEZAfFsh7" --jump cali-to-host-endpoint\n-A cali-OUTPUT -m comment --comment "cali:pN0F5zD0b8yf9W1Z" -m mark --mark 0x10000/0x10000 --jump ACCEPT\n-I PREROUTING -m comment --comment "cali:6gwbT8clXdHdC1b1" --jump cali-PREROUTING\n-I OUTPUT -m comment --comment "cali:tVnHkvAo15HuiPy0" --jump cali-OUTPUT\nCOMMIT\n" ipVersion=0x4 output="" table="raw"
Your Environment

Calico version : 3.19.2
Orchestrator version (e.g. kubernetes, mesos, rkt): Kubernetes 1.21.5
Operating System and version: Debian10
uname -a
Linux radisys-dce-setup-07-10 4.19.81-OpenNetworkLinux #1 SMP Wed Dec 22 07:37:14 UTC 2021 x86_64 GNU/Linux
lsb_release -a
No LSB modules are available.
Distributor ID: Debian
Description:    Debian GNU/Linux 10 (buster)
Release:        10
Codename:       buster
Iptables version: v1.8.2 (legacy)

It seems like your nodes might be missing some required kernel dependencies.
Can you try checking with https://projectcalico.docs.tigera.io/archive/v3.19/reference/calicoctl/node/checksystem ?

Looks like you're missing the IPv6 version of the rpfilter module.  If you're not using IPv6 you could disable our IPv6 support.

Looks like you're missing the IPv6 version of the rpfilter module. If you're not using IPv6 you could disable our IPv6 support.

Thank you for the pointers. After enabling the kernel configurations for the required modules. Am not seeing Failures in "calicoctl node checksystem" output and No errors were found in the calico logs. However calico pod is not ready.
Below are the details of the pod description and logs. Could you pls help me in understanding the issue
calico pod description
Warning Unhealthy 26m (x2 over 26m) kubelet Readiness probe failed: calico/node is not ready: BIRD is not ready: Error querying BIRD: unable to connect to BIRDv4 socket: dial unix /var/run/calico/bird.ctl: connect: connection refused
Normal Killing 25m kubelet Container calico-node failed liveness probe, will be restarted
Normal Created 25m (x2 over 26m) kubelet Created container calico-node
Normal Pulled 25m (x2 over 26m) kubelet Container image "quay.io/calico/node:v3.19.2" already present on machine
Warning Unhealthy 16m (x44 over 26m) kubelet Liveness probe failed:
Warning BackOff 6m21s (x38 over 19m) kubelet Back-off restarting failed container
Warning Unhealthy 82s (x80 over 26m) kubelet Readiness probe failed:
Calico pod log
2022-03-05 15:20:37.156 [DEBUG][73] felix/int_dataplane.go 936: Checking if we need to clean up the VXLAN device
2022-03-05 15:20:37.156 [DEBUG][73] felix/int_dataplane.go 940: VXLAN disabled and no VXLAN device found
2022-03-05 15:20:37.157 [DEBUG][73] felix/lib.go 105: Raw kernel version rawVersion="Linux version 4.19.81-OpenNetworkLinux (root@mustaq-VirtualBox) (gcc version 8.3.0 (Debian 8.3.0-6)) https://github.com/projectcalico/calico/issues/1 SMP Thu Mar 3 05:12:02 UTC 2022\n"
2022-03-05 15:20:37.157 [DEBUG][73] felix/lib.go 113: Parsed kernel version version=4.19.81
2022-03-05 15:20:37.158 [DEBUG][73] felix/xdp_state.go 111: Created new xdpState.
2022-03-05 15:20:37.158 [INFO][73] felix/int_dataplane.go 466: XDP acceleration enabled.
2022-03-05 15:20:37.243 [DEBUG][73] felix/sockmap_state.go 37: Created new Sockmap state.
2022-03-05 15:20:37.243 [DEBUG][73] felix/sockmap_state.go 191: Wiping old sockmap state.
2022-03-05 15:20:37.243 [DEBUG][73] felix/bpf.go 154: running: bpftool prog detach pinned /sys/fs/bpf/calico/sockmap/calico_sk_msg_v1 msg_verdict pinned /sys/fs/bpf/calico/sockmap/calico_sock_map_v1
2022-03-05 15:20:37.245 [DEBUG][73] felix/sockmap_state.go 194: Failed to detach sk_msg program from sockmap. error=failed to detach sk_msg prog from sockmap: exit status 255
Error: bpf obj get (/sys/fs/bpf/calico/sockmap/calico_sk_msg_v1): No such file or directory
2022-03-05 15:20:37.245 [DEBUG][73] felix/bpf.go 154: running: bpftool cgroup detach /run/calico/cgroup sock_ops pinned /sys/fs/bpf/calico/sockmap/calico_sockops_v1
2022-03-05 15:20:37.247 [DEBUG][73] felix/sockmap_state.go 198: Failed to detach sockops program from cgroup. error=failed to detach sockops prog from cgroup: exit status 255
Error: bpf obj get (/sys/fs/bpf/calico/sockmap/calico_sockops_v1): No such file or directory
2022-03-05 15:20:37.247 [DEBUG][73] felix/sockmap_state.go 202: Failed to remove sockops program. error=remove /sys/fs/bpf/calico/sockmap/calico_sockops_v1: no such file or directory
2022-03-05 15:20:37.247 [DEBUG][73] felix/sockmap_state.go 206: Failed to remove sk_msg program. error=remove /sys/fs/bpf/calico/sockmap/calico_sk_msg_v1: no such file or directory
2022-03-05 15:20:37.248 [DEBUG][73] felix/sockmap_state.go 210: Failed to remove sockmap endpoints map. error=remove /sys/fs/bpf/calico/sockmap/calico_sk_endpoints_v1: no such file or directory
2022-03-05 15:20:37.248 [DEBUG][73] felix/bpf.go 154: running: bpftool map --json getnext pinned /sys/fs/bpf/calico/sockmap/calico_sock_map_v1
2022-03-05 15:20:37.249 [DEBUG][73] felix/sockmap_state.go 214: Failed to remove sockmap program. error=failed to clear sock map: bpf obj get (/sys/fs/bpf/calico/sockmap/calico_sock_map_v1): No such file or directory
2022-03-05 15:20:37.250 [INFO][73] felix/connecttime.go 51: Running bpftool to look up programs attached to cgroup args=[]string{"bpftool", "-j", "-p", "cgroup", "show", "/run/calico/cgroup"}
2022-03-05 15:20:37.252 [DEBUG][73] felix/cleanup.go 33: Trying to clean up any left-over BPF state from a previous run.
2022-03-05 15:20:37.258 [INFO][73] felix/route_table.go 249: Calculated interface name regexp regex="^cali.*"
2022-03-05 15:20:37.258 [DEBUG][73] felix/int_dataplane.go 981: registering ManagerWithRouteTables manager=""
2022-03-05 15:20:37.258 [INFO][73] felix/ipsets.go 126: Queueing IP set for creation family="inet" setID="all-ipam-pools" setType="hash:net"
2022-03-05 15:20:37.259 [INFO][73] felix/ipsets.go 126: Queueing IP set for creation family="inet" setID="masq-ipam-pools" setType="hash:net"
2022-03-05 15:20:37.259 [INFO][73] felix/route_table.go 249: Calculated interface name regexp regex="^wireguard.cali$"
2022-03-05 15:20:37.259 [DEBUG][73] felix/int_dataplane.go 981: registering ManagerWithRouteTables manager=""
2022-03-05 15:20:37.259 [INFO][73] felix/int_dataplane.go 834: Registering to report health.
2022-03-05 15:20:37.262 [INFO][73] felix/int_dataplane.go 1623: attempted to modprobe nf_conntrack_proto_sctp error=exit status 1 output=""
2022-03-05 15:20:37.262 [INFO][73] felix/int_dataplane.go 1625: Making sure IPv4 forwarding is enabled.
2022-03-05 15:20:37.263 [INFO][73] felix/table.go 506: Queueing update of chain. chainName="cali-failsafe-in" ipVersion=0x4 table="raw"
2022-03-05 15:20:37.263 [DEBUG][73] felix/table.go 939: Would invalidate dataplane cache but it was already invalid. ipVersion=0x4 reason="chain update" table="raw"
2022-03-05 15:20:37.263 [INFO][73] felix/table.go 506: Queueing update of chain. chainName="cali-failsafe-out" ipVersion=0x4 table="raw"
2022-03-05 15:20:37.263 [DEBUG][73] felix/table.go 939: Would invalidate dataplane cache but it was already invalid. ipVersion=0x4 reason="chain update" table="raw"
2022-03-05 15:20:37.263 [INFO][73] felix/table.go 506: Queueing update of chain. chainName="cali-PREROUTING" ipVersion=0x4 table="raw"
2022-03-05 15:20:37.263 [DEBUG][73] felix/table.go 577: Incref chain chainName="cali-from-host-endpoint"
2022-03-05 15:20:37.263 [INFO][73] felix/table.go 580: Chain became referenced, marking it for programming chainName="cali-from-host-endpoint"
2022-03-05 15:20:37.263 [DEBUG][73] felix/table.go 939: Would invalidate dataplane cache but it was already invalid. ipVersion=0x4 reason="chain update" table="raw"
2022-03-05 15:20:37.341 [INFO][73] felix/table.go 506: Queueing update of chain. chainName="cali-OUTPUT" ipVersion=0x4 table="raw"
2022-03-05 15:20:37.341 [DEBUG][73] felix/table.go 577: Incref chain chainName="cali-to-host-endpoint"
2022-03-05 15:20:37.342 [INFO][73] felix/table.go 580: Chain became referenced, marking it for programming chainName="cali-to-host-endpoint"
2022-03-05 15:20:37.342 [DEBUG][73] felix/table.go 939: Would invalidate dataplane cache but it was already invalid. ipVersion=0x4 reason="chain update" table="raw"
2022-03-05 15:20:37.342 [DEBUG][73] felix/table.go 459: Updating rule insertions chainName="PREROUTING" ipVersion=0x4 table="raw"
2022-03-05 15:20:37.342 [DEBUG][73] felix/table.go 577: Incref chain chainName="cali-PREROUTING"
2022-03-05 15:20:37.342 [INFO][73] felix/table.go 580: Chain became referenced, marking it for programming chainName="cali-PREROUTING"
2022-03-05 15:20:37.342 [DEBUG][73] felix/table.go 939: Would invalidate dataplane cache but it was already invalid. ipVersion=0x4 reason="insertion" table="raw"
2022-03-05 15:20:37.342 [DEBUG][73] felix/table.go 459: Updating rule insertions chainName="OUTPUT" ipVersion=0x4 table="raw"
2022-03-05 15:20:37.342 [DEBUG][73] felix/table.go 577: Incref chain chainName="cali-OUTPUT"
2022-03-05 15:20:37.343 [INFO][73] felix/table.go 580: Chain became referenced, marking it for programming chainName="cali-OUTPUT"
2022-03-05 15:20:37.343 [DEBUG][73] felix/table.go 939: Would invalidate dataplane cache but it was already invalid. ipVersion=0x4 reason="insertion" table="raw"
2022-03-05 15:20:37.343 [DEBUG][73] felix/static.go 556: Adding workload match rules ifacePrefix="cali"
2022-03-05 15:20:37.343 [DEBUG][73] felix/static.go 272: Adding workload match rules ifacePrefix="cali"
2022-03-05 15:20:37.343 [DEBUG][73] felix/static.go 653: Adding workload match rules ifacePrefix="cali"
2022-03-05 15:20:37.343 [DEBUG][73] felix/static.go 155: Adding workload match rules ifacePrefix="cali"

Can you show me the liveness and readiness checks from the calico-node daemonset please?  I'm wondering if they're still set to 1s (k8s recently fixed those checks to actually use the configured value, and we learned that 1s is too short).


SIMSBIOHUB-331: Keycloak Library Updates

Links to Jira Tickets
https://apps.nrs.gov.bc.ca/int/jira/browse/SIMSBIOHUB-331
Description of Changes
Swap out old keycloak libraries for newer more up to date libraries.
Remove login and logout pages.
Refactor the keycloakWrapper, and update code/tests as necessary.
Testing Notes
Anything to do with logging in/out works as before.

Logging in as an IDIR/BCEID user
Logging in without access, requesting access, being granted access, etc
Logging in with a non-admin system role and only being able to access projects you have a project level role on

Logging out as an IDIR and then logging back in as a BCeID doesn't appear to work:

Ah yeah, Its because I removed that siteminder logout url piece. Hrmm, can add it back of course. It would fix that issue, but would mean that you get logged out of IDIR everywhere else as well, so kind of a win/lose situ

Logging out as an IDIR and then logging back in as a BCeID doesn't appear to work:

As discussed, Pushed up a commit to re-add the siteminder logout redirect. Seems to fix the problem.


cassandra: ignore timeout in background jobs

Coverage decreased (-0.2%) to 79.416% when pulling b105d62e946ef1cc10ec81cadc09c3ca5ff2587e on iksaif:master into 0173ac795692f74d02cba95675b9f6a299773c98 on criteo:master.

Coverage decreased (-0.2%) to 79.412% when pulling a2b56638c88fa649105070ebd0321bb511772223 on iksaif:master into 0173ac795692f74d02cba95675b9f6a299773c98 on criteo:master.

Coverage decreased (-0.8%) to 78.778% when pulling 95d996a92bf891cff551882e0d7963da2614babd on iksaif:master into 0173ac795692f74d02cba95675b9f6a299773c98 on criteo:master.

Coverage decreased (-0.9%) to 78.632% when pulling 9aa6296ad057e72720fcb8ef939b3ec83d20ef69 on iksaif:master into 0173ac795692f74d02cba95675b9f6a299773c98 on criteo:master.

Coverage decreased (-0.3%) to 79.227% when pulling d263c97ade115c9ffb11eeb3aa799cefac4bf227 on iksaif:master into 0173ac795692f74d02cba95675b9f6a299773c98 on criteo:master.


Maple build failing in master

_Instructions: Just installing clap...
Warning: if you don't fill this issue template and provide the reproducible steps the issue could be closed directly.
Environment (please complete the following information):

OS: ubuntu20.04
(Neo)Vim version: neovim 0.7.2
vim-clap version: latest to date
Have you reproduced with a minimal vimrc: havn't tried
Have you updated to the latest plugin version: yes
Have you upgraded to/compiled the latest Rust binary: yes

Describe the bug
when compiling with :Clap install-binary an error occurs, related to the source code:
error[E0515]: cannot return value referencing temporary value
--> crates/filter/src/source.rs:61:5
|
61 |       std::io::stdin()
|       ^---------------
|       |
|  _____temporary value created here
| |
62 | |         .lock()
63 | |         .lines()
64 | |         .filter_map(move |lines_iter| {
...  |
68 | |             })
69 | |         })
| |__________^ returns a value referencing data owned by the current function
|
= help: use .collect() to allocate the iterator
For more information about this error, try rustc --explain E0515.
error: could not compile filter due to previous error
warning: build failed, waiting for other jobs to finish...
error: build failed
make: *** [Makefile:6: maple] Error 101
Clap debug

            has cargo: 1
            has maple: v:null
         has +python3: 1
has py dynamic module: 0
            has ctags:
vim-clap: Ensure ctags executable is in your PATH and has the JSON output featurectags without JSON output support
        rustc version: rustc 1.59.0

    Current sync impl: Lua
     Current FileType:
Third Party Providers: []
       Global Options:
    let g:clap#autoload_dir = '/home/eopatowsky/.vim/bundle/vim-clap/autoload'
    let g:clap#provider_alias = {'gfiles': 'git_files', 'hist:': 'command_history', 'hist/': 'search_history'}
    let g:clap_background_shadow_blend = 50
    let g:clap_disable_bottom_top = 0
    let g:clap_disable_matches_indicator = v:false
    let g:clap_disable_optional_async = v:false
    let g:clap_disable_run_rooter = v:false
    let g:clap_enable_background_shadow = v:false
    let g:clap_enable_debug = v:false
    let g:clap_enable_icon = 0
    let g:clap_forerunner_status_sign = {'done': '•', 'running': '!', 'using_cache': '*'}
    let g:clap_insert_mode_only = v:false
    let g:clap_multi_selection_warning_silent = 0
    let g:clap_no_matches_msg = 'NO MATCHES FOUND'
    let g:clap_open_action = {'ctrl-v': 'vsplit', 'ctrl-x': 'split', 'ctrl-t': 'tab split'}
    let g:clap_open_preview = 'always'
    let g:clap_popup_border = 'single'
    let g:clap_preview_direction = 'AUTO'
    let g:clap_preview_size = 5
    let g:clap_providers_relaunch_code = '@@'
    let g:clap_search_box_border_style = 'nil'
    let g:clap_search_box_border_symbols = {'nil': ['', ''], 'curve': ['', ''], 'arrow': ['', '']}
  Provider Variables:
                     []

To Reproduce
Steps to reproduce the behavior:

Create the minimal vimrc min.vim:

set nocompatible
set runtimepath^=/path/to/vim-clap
syntax on
filetype plugin indent on

Start (neo)vim with command: vim -u min.vim

Type '....'

See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Try to run rustup update && cargo build --release under the vim-clap plugin directory.

@liuchengxu still fails with same res code

Interesting, never seen this before, what does rustup show say now?

says the following

found this about the "no active toolchain"
rustup install stable rustup default stable
after this rustup shows prompts:

even though, after rustup update && cargo build --release it fails with same res code, and after cargo clean and repeating the same command,  it fails the same.

Not sure what happened on your side, but you can build the binary using docker according to the instruction here https://github.com/liuchengxu/vim-clap/blob/5cd460ca66a734824d35e95f51750858bce1dc1c/INSTALL.md#linux.

This issue should be fixed in the latest master as the related code has been refactored away anyway.


Pressing "Enter" should be disabled once onSubmit is called

Hello again,
I set numInputs to be 4
Once I enter the 4 digits, it runs onSubmit.
However, If I quickly press Enter, while it is running, it tries to run onSubmit again but with an empty value which is erroneous to the process. (I'm using a loader/overlay and am still able to quickly press "Enter")
Solution: Once you enter the number of digits and automatically call onSubmit. Disable pressing "Enter" key
Thank you!

Hello @m00gl3, you can use the onKeyDown prop to modify the behavior. You can look for event of enter key and prevent the submission in your code itself.
Or please share your code so that I can help you with the direction.
This does not seem to be a bug and looks more of a help you need to achieve what you are doing. I am moving this to a discussion. Please share a minimal reproducable codesandbox or a video or share your code, so that I can help you with the right direction. Thanks.


Add unit tests for API Endpoints.

@srishi, I read through the wikimedia API Endpoint and I  realised the sample code has no tests written for the endpoints . and i would really take an a honor of writting the tests given a chance as my contribution. Thanks

@RachealN Hey, thanks for chiming in. I had a small discussion with @srish regarding this as well. Since these are very basic examples, tests for APIs are not a priority. Also, since such tests are already existing for all APIs internally in MediaWiki API implementation and testing infrastructure, it makes sense to not duplicate the efforts here. There can be plenty of other tasks that can be done - for example, more complex code samples that use multiple APIs in tandem to solve interesting use-cases


Will it work with logseq 0.10.3-alpha+nightly.20240112?

I have a problem. Everything I try doesn't work with the version in the title. I've already used the newest version, but it's still not working. Will there be a fix for auto linking? I heard that it works with version 0.8.3 or something like that, but I just want to use the current version for other plugins I use.

Automatic Linker doesnt work for me either on 0.10.3. Tested with  AppImage in Ubuntu.

Automatic Linker doesnt work for me either on 0.10.3. Tested with AppImage in Ubuntu.

Ok good i thought its a problem only I have. Cause with a downgraded version it works but doesnt make sense to downgrade.

i fixed the issue with #77

Thank you @clarkissac . But it appears the maintainer is not active for past few weeks. Hopefully he can get this merged in. Its a shame if this plugin gets broken as it is one of the important reasons I use logseq.


Compiling diesel_codegen with no features gives a less than useful error message

diesel = { version = "0.9.0", features = ["postgres"] }
diesel_codegen = "0.9.0"

#[macro_use] extern crate diesel;
#[macro_use] extern crate diesel_codegen;

infer_schema!("dotenv:DATABASE_URL");

error: `#[derive]` for custom traits is not stable enough for use. It is deprecated and will be removed in v1.15 (see issue #29644)
  --> src/main.rs:15:1
   |
15 | infer_schema!("dotenv:DATABASE_URL");
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
   = note: this error originates in a macro outside of the current crate

Is there a nice way to do this? All I can think of is something like
#[cfg(not(any(feature = "sqlite", feature = "postgres")))]
mod hello_you_need_enable_one_of_the_backend_adapters_as_cargo_feature_thank_you;

We just need to implement the derives that are behind a cfg flag to give a
more specific error when called.
On Tue, Jan 10, 2017, 9:20 AM Pascal Hertleif<EMAIL_ADDRESS>wrote:

Is there a nice way to do this? All I can think of is something like
#[cfg(not(any(feature = "sqlite", feature = "postgres")))]mod hello_you_need_enable_one_of_the_backend_adapters_as_cargo_feature_thank_you;
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/diesel-rs/diesel/issues/570#issuecomment-271586781,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABdWK7DlGUF5trvNMvzuL7fY_4ylZzuxks5rQ5OLgaJpZM4LejmM
.

This was fixed a while back when compile_error! was stabilized.


GitLab : Close one merge_requests tab after complete ICLA docusigned

Steps :

Login PCC with valid credentials
Go to easyCla GitLab page
Connect GitLab groups
Enforced project to CLA
Create PR on projects
See-PR is gated
Click on Not Covered button, See-It open in separate tab
See-Contributor console open
Now click on "Proceed as an Individual Contributor" options
Preparing CLA...page open
Click on Sign CLA button
Fill & finish docusigned

Actual Results : Two merge_requests tab open

https://images.zenhubusercontent.com/194341141/b85384b3-6ac8-473a-a5e4-3fe5bb815663/gitlab_icla.mp4

Issue is still reproducible as reported on Dev site, So making issue status to rejected by QA
https://images.zenhubusercontent.com/194341141/0e916d82-ab66-4703-b584-1e0e7fbfa359/_3318.mp4

Closing issue due to inactivity.


SignalingClient master disconnects quite soon from WSS endpoint

SignalingClient master disconnects quite soon from WSS endpoint after a few minutes. I can reconnect until the credentials finally expire and then have to reinitialize everything.
Are there any better ways for me to handle that? If I am not the only one with this issue, could we build it in to the lib?

Refer to #218.


Default Font

If Bootstrap isn't used, the font defaults to whatever the browser has for a default. Could you set the default font (in the absence of Bootstrap) to Helvetica,Arial,etc?

Right now without bootstrap it is not shown nicely as you show in issue #3 . But later is people ask that they want to use it without bootstrap I will fix the show problem and I will set the default font. Do you want to use it without bootstrap?

Yes.  Based on your project page, the only dependency was jQuery.  I thought you included Bootstrap so you could make the demo buttons look nice, but I didn't think for a moment that this plugin was also dependent on it.
Would you consider renaming the classes in case optional customization are made to the plugin (so they wouldn't affect Bootstrap)?  Or would you keep everything named the same so that Bootstrap could be easily used?  (I'd hate for any future update of Bootstrap to affect this plugin.  I got stung like this as an early adopter of jQueryUI.)

Default font will be added.

Default font is added and it is pushed on dev branch


Allow stick tilts to be configured before stick presses

I have an 8BitDo Lite, where the sticks are physically D-pads, and a stick press is configured as Stick U+D+L+R. The UI makes it impossible for me to configure stick presses and stick tilts correctly.
I think the config string should be
030000005e0400008e02000000007801,8BitDo Lite,platform:Windows,a:b0,b:b1,x:b2,y:b3,back:b6,guide:b10,start:b7,leftstick:b8,rightstick:b9,leftshoulder:b4,rightshoulder:b5,dpup:h0.1,dpdown:h0.4,dpleft:h0.8,dpright:h0.2,leftx:a0,lefty:a1,rightx:a3,righty:a4,lefttrigger:a2,righttrigger:a5,

, but have no way of importing the mapping string.

In addition, I cannot locate where these custom mappings are stored on disc. I can't fix the mistakes with the GUI, so I have to change the data directly. Where are the custom mappings?

SDL has a very helpful community, you should ask these things there: https://discourse.libsdl.org/

That doesn't address the problem.

We don't make a UI, we don't determine the SDL controller format and we don't decide where the mappings are stored. This repository just provides a database for game developers to use.


No more inferred moves?

Hi!
I was just reading your original Tornado paper (from ManLang '18), where it is mentioned that Tornado(VM) automatically infers which data moves need happening (see code below). What happened to the TaskSchedule API?
TaskSchedule s = new TaskSchedule("s0")
    .task("t0", this::add, input0, input1, output0)
    .streamOut(output0).execute()

(code sample from the ManLang '18 paper; a very similar sample is found in the VEE '19 paper)
Is it still possible to have Tornado(VM) infer the copies from the arguments, or do we always need to specify these explicitly? This was one of the bigger selling points to me, the guaranteed memory consistency...

Hi @jay-tux . The API you showed is the old one. TornadoVM has evolved to a new explicit API since version 0.15:
https://tornadovm.readthedocs.io/en/latest/programming.html
From v0.15, we require all data to be explicitly tagged. We made this decision because for many users, this was confusing (where is my data?). So, we make it explicit. However, we rethink this  frequently, In the future I would like to remove completely the need to pass any data as stream-in or out.

Thanks for the reply!


Create a non-bundled configuration profile

While creating the bundled one, create a directory with separate files and document the location of this alternative configuration profile, we can also zip it so it can be downloaded easily as the bundled one
(related with #5)

The architecture of the newly merged image should enable implementing this rather easily, but should it be ? The related client bug was fixed.

I'm running one instance in Paris, it's an influxdb instance but lately by ISP "Deutsche Telekom" is doin MITM and some sites are inaccessible like www.rt.com.
Now I'd like to just run this on my CentOS7 instance, what would I need to do?

@dalu I don't quite understand your problem, nor how it may even remotely be related to this issue

Well is that my shortcoming or yours?
Maybe you're remotely unsuitable to contribute to a repository that deal with SOFTWARE WHOS PURPOSE IS, AMONGST OTHER THINGS, TO MITIGATE MAN IN THE MIDDLE ATTACKS THROUGHT ENCRYPTION AND AUTHENTICATION. What do you think is that un-remote enough for you?

@dalu "I don't quite understand" Was the main element of my message, and "may even remotely be related to this issue" would emphasize how far I thought I was from understanding your issue. What I did not understand was whatever you were trying to say, not how openvpn works and what it is. Furthermore, I thought this emphasis would lead you to explain more thoroughly your issue, so that I could possibly help you out.
After reading and reading your message again, I think I understand what you meant. So you want to install this on a current CentOS7 instance, and that's all? If so, well you may just apply the Dockerfile manualy, and see how it turns out. It'll probably take some debug to get it working though.
"Have you read the title of this issue?"
Sure, it's about downloading the different components of the generated OpenVPN configuration separatly. It doesn't read "porting to CentOS".
It was created to consider mitigating a bug from networkmanager / a former openvpn configuration issue (see #5).
Best regards,
Somebody who doesn't even works at scaleway


Automatic code formatting plugins for VSCode + Rider

As mentioned by @Sean-Brown , and previously at CFC, we should have code formatting happen automagically, especially as the project and the potential for inconsistency grows.
Ideally, this will be something that can be used across IDEs.  I don't know if that's feasible in the .NET world though.  If you are a .NET person and know a good solution, you can take this issue and run with it!
Also of note is that we haven't settled on many code styles yet, except for using tabs for indentation rather than spaces.  There are definite inconsistencies, so a plugin that fixes everything in the project at once will result in a huge change set.  It might be better to have one that applies when files are saved.  But taking this issue might also mean herding some cats to agree on currently diverging standards.

This blog has some opinions:

the dotnet format command. They don't seem very impressed with it, but maybe it's improved since the blog was written.
StyleCop and .editorconfig. It sounds pretty decent, although the initial wrangling of a huge list of warnings sound pretty tedious.
SonarLint. Sounds similar to StyleCop.
Resharper. I've used this tool before and it's pretty nice, but it's not free.
CSharpier. Also seems similar to StyleCop. Has plugins for VS, VS Code, and Rider.

I haven't done a deep dive yet, but it seems like there should be some decent options. No matter which we go with, it'd be good to get some opinions from the other devs before committing the rules (although we can always vote to change some rules later).
No matter what tool we go with, we should probably add a pre-commit hook to do the code formatting automatically on staged files before committing, as described here.

I've looked into this and tried some of suggested extensions. I think one good option may be to configure Visual Studio with an .editorconfig (see here), and configure omnisharp to use .editorconfig for Visual Studio Code (see here). I'm not sure about Rider since I don't have it, but I would be surprised if it wasn't easy to also configure omnisharp / whatever the default formatter is to use .editorconfig. The pro of going this route is that for visual studio and visual studio code (assuming you already have the C# extension installed), there's no need to install an additional extension for formatting.

That does look promising.  Rider does support it (see https://www.jetbrains.com/help/rider/Using_EditorConfig.html#export-code-style-settings), and EditorConfig seems to have a website listing editor support at https://editorconfig.org/ .  And as Sean-Brown's research indicates, StyleCop also uses .editorconfig, as does SonarLint.
I might yet try Resharper on top of Visual Studio, but it's not a solution for this problem since it can't be used by everyone.


change fields phone and email on model customer to be unique

fix #73
Phonenumbers and Emails should be unique anyways.
Both of the fields have to be filled, so it's impossible to create an entry twice.

Coverage remained the same at 64.542% when pulling c34cdc72e6f7f7d038ed667ced101e4696138be5 on juliusbentz:fix_contact_details_should_be_unique_#73 into 1c18111c5a33b050ed8dd30d6a12897fcc672892 on MicroPyramid:master.


Cannot open options/settings in Thunderbird on macOS

I saw Issue #276 but as this issue has been resolved, I create a new issue describing the same problem, but probably with a different cause.
I cannot open the settings/options of the add-on in Thunderbrid's add-on-management.
Though, when I installed the add-on and restarted, the options opened automatically. The only way to re-open them is to remove the add-on with restart, reset the config-entries and re-install the add-on with restart.
I'm using Thunderbird v52.1.1 (64-Bit) on macOS 10.12.4 with Markdown Here v2.12.0 in German.

Hello,
same problem here under Windows 8.1, Thunderbird 52.1.1 (32-bit), Markdown Here 2.12.0.
Click on "Options" (from the main Add-ons Manager or from Markdown Here "more..."
The issue doesn't seems to be confined to MacOS...


Orbital not capable of what Billboard can do (which is not a solver)

Describe the bug
Orbital  does not have any option to follow one object while looking at another on the y-axis only. The Orbital script has all sorts SolverOrientationType modes instead, which completely define the rotation.
Orbital does not have a TransformTarget field even though it can for example look at the camera, with is an option of SolverOrientationType  and is hard coded.
Expected behavior
Orbital should use the same PivotAxis enum like Billboard does.
In addition, it should be able to look at custom targets, like Billboard does
Basically, it should integrate the capability of being billboard, which it nearly is.
Or maybe creating a Billboard Solver and just ditch Orbital
Also, there are quite a lot of enums and a couple of them are pretty much either a portion of another or a combination of them:
OrientationType
PivotAxis
RadialViewReferenceDirection
RotationConstraintType
SolverOrientationType
And why doesn't the Pivot Axis enum make use of the FlagsAttribute?

Closing in favor of larger refactor #5713


Fix installation invocation for OpenJDK

#31745 omitted updating the install invocation for the OpenJDK JRE, this fixes it

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

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

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

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

R: @liferoad


chore(main): release 1.26.1

:robot: I have created a release beep boop
1.26.1 (2024-12-11)
Bug Fixes

dropdown border and padding (0be6674)
dropdown border and padding on form (28e26a1)
dropdown border and padding on nav items (db98b23)
dropdown in form width (4666e55)
dropdown menu hover state (389b33a)
latest news (2d9ddb8)
logo height on mobile (9ebef7c)
markup and vertical spacing in signup form (0e35b1d)
select padding (ac40318)
set different overlays per breakpoint for the support banner (39c5e65)
stanadarise padding in fills and dropdowns (9dac52d)
vertical spacing in medium size for partners (d5dcf72)
wrappung text in buttons (f69e11f)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.26.1
:sunflower:


iOS complex use cases

Hi,
I rarely do Pull Requests. So if I'm doing something wrong, please tell me.
This PR adds some features I found useful in my app. I was building a virtual machine, and needed to load a ISO/Disk Image into memory chunk by chunk. Usually the images will be stored in my own app. The user should be able to pick the images using this component.
The feature I added allows me to use files in my own apps without copying them (importing) to my inbox folder. Otherwise it'd be really inefficient since the files (disk images) are rather large.

I think it would be better to add open support down the road rather than add something called "no_import".

Hi I changed it to use a string field instead. Currently the only supported modes are "import" and "open".
To support more modes, there wouldn't be any more changes to the interface. But we may need more objc code to handle each mode differently

If we implement open support we have to do it on all platforms together, Android supports the distinction between import and open as well.
And we need to actually take the rest of the API into account. Import is easy since all you need to do is fetch it to read it. But open is more complex because the expectation of opening it is that you may write it later. Which means we need to consider where the user is going to get an API to write back to the file.
I also wonder if DocumentPicker.open would be better than an option. Depending on how conceptually different cross-platform usage of import vs open is.
Basically, I don't think open is a good thing to try piling onto the list of things for v3.

Got it. I didn't understand what you meant by "open support" initially.
From my perspective, filesystem behaves very differently across ios and android. That's why I assumed it's ok to allow per-os settings.
Should I close the PR and open an issue about my feature request instead?

Yeah, an issue would probably work better to leave open for a period of time. PRs just go stale over time.


Add ArrayDesc destructor to avoid possible stack overflow

The following script failed before this PR
import gc

import mlx.core as mx

x = mx.ones(1)
for i in range(300_000):
    x = 2 * x
del x
gc.collect()

Weirdly, it failed with segmentation fault rather than stack overflow error but it may be an artifact of it being in the destructor. The stack trace was >260k levels deep so I think in general this is not too bad to do.
I measured no slowdown for Mistral inference or transformer training in comparison to main.

CC @davidkoski maybe you can try this diff as well to make sure stack issues are resolved on your end.

Weirdly, it failed with segmentation fault rather than stack overflow error but it may be an artifact of it being in the destructor. The stack trace was >260k levels deep so I think in general this is not too bad to do.

Segmentation fault is what I would expect on the C side -- it goes past the end of the stack pages and hits unavailable memory.
I confirm this fixes my use case, thank you!


Dialogue sounds (0.5.2h)

Describe your problem here.
Hello. I wanted to test the "Dialogue sound" new function they added to 0.5.2h, but the sound repeats during ALL the lines. Is there anyway to chage the sound for each line or not?
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
PlayState.hx, CoolUtil.hx, StoryMenuState.hx, MainMenuState.hx, MenuItem.hx, TitleState.hx, HealthIcon.hx and LoadingState.hx (just for customization).

It's just like that on the Editor, should work in-game fine.


Labels select popup bug

When there are many labels, the popup does not show all the content and it's impossible to scroll or close it.

It seems to be a common issue for pop-ups.


Caching Files app in memory

What's the Problem?
Files app takes a while to launch, like 10 seconds to fully load everything like drives, shortcuts, navigation bar, etc.
Solution/Idea
Cache files app in memory at the startup, so that it'll launch as fast as one-click (whatsapp UWP beta does the same).
PC specs
Windows 11
Intel i7 10th gen
M.2 SSD drive
16 GB ram

@TechyAditya we already do this with Files but due to some platform bugs, it doesn't seem to make a difference. We're hoping to get this working when we switch to WinUI.


export service in rust will overwrite the types with the same fields

Consider the following code:
#[derive(CandidType,` Clone, Serialize, Deserialize)]
pub struct User1 {
  id: String,
  name: String,
  age: u8,
}

#[derive(CandidType, Clone, Serialize, Deserialize)]
pub struct User2 {
  id: String,
  name: String,
  age: u8,
}

#[query]
#[candid_method(query)]
pub fn test1(e: User1) {
}

#[query]
#[candid_method(query)]
pub fn test2(e: User2) {
}

#[test]
pub fn generate_candid() -> () {
  use std::env;
  use std::fs::write;
  use std::path::PathBuf;

  candid::export_service!();

  let dir = PathBuf::from(env::var("CARGO_MANIFEST_DIR").unwrap());
  let dir = dir.parent().unwrap().join("candid");
  write(dir.join(format!("test.did")), __export_service()).expect("Write failed.");
}

I would expect the candid file to be
type User1 = record { id : text; age : nat8; name : text };
type User2 = record { id : text; age : nat8; name : text };
  test1 : (User1) -> () query;
  test2 : (User2) -> () query;

but the actual output is this
type User1 = record { id : text; age : nat8; name : text };
  test1 : (User1) -> () query;
  test2 : (User1) -> () query;

It seems that candid copy the struct reference and check if the same structure exists and if it does it's omitted.

This is a known behavior: https://github.com/dfinity/candid/blob/master/rust/candid/src/types/internal.rs#L83. But technically not a bug. Because Candid follows structure typing, User1 and User2 are exactly the same, it's not wrong to use User1 when the underlying Rust code uses User2. But I understand this is confusing for documentation purposes.
One workaround can be to let users provide their own did file and the build process will check if the provided did file is a subtype of the generated one.

Hey, sorry about the confusion.  We had some crossed wires internally about what was going on.
Yes, that is the default behaviour for Candid and that's understood.  What we meant to point out was there seems to be irregularity in the derive macro code for the Rust implementation of Candid.
We had ten identical structs, so we expected to either get 10 entries in Candid (if it wasn't structurally typed), or 1 (if it was - and that's cleared up now.
We got 2... or at least I did, the rest of the team can't really replicate it yet.  I have on two separate computers though.  I managed to reduce it down to the following code....
#[derive(CandidType, Deserialize)]
pub struct Cat1 {
    pub id: String,
}

#[update]
#[candid_method(update)]
async fn create_a_cat(e: Cat1) -> api::CreateResult {
    Ok("".into())
}

#[derive(CandidType, Deserialize)]
pub struct Cat {
    pub id: String,
}

#[update]
#[candid_method(update)]
async fn create_b_cat(e: Cat) -> api::CreateResult {
    Ok("".into())
}

type Cat = record { id : text };
type Cat1 = record { id : text };
service : () -> {
  create_a_cat : (Cat1) -> (Result);
  create_b_cat : (Cat) -> (Result);
}

It seems that if the first type has a longer name than the second type it creates two.  If I change the Cat to Cat2 then it works as intended with only one candid type created.

Correct. The algorithm is trying to find the shortest name at the time of the lookup. Depending on the order the compiler processes these type definitions, it may get different result. That's also why you may not be able to reliably replicate it. Again, it's still correct, but not nice.

oh thanks, this explains it.  We have a workaround where our auto-generated classes make more sense... just thought it may be a bug.

hey @chenyan-dfinity I get that's how candid works, could you have a look and maybe comment here please: https://forum.dfinity.org/t/deduplication-of-candid-types-in-rust/19079 thanks, will close this for now.


Community Maintenance

As expressed by the main developer @tomhoule, he doesn't have time to work on GraphQL-client anymore. This means the community needs to step up and take ownership of the project.
Please name yourself if you are interested in contributing to the project. We can also create a discord for better communication.

That can't happen without a transition phase. I can take time to review PRs occasionally, other people already have maintainer access to the repo and the  graphql-rust organization, so the project is not going to die. I am happy to add people who have made a few quality contributions to the maintainers.


Modulo bias in Random

When restricting the random number to a range, modulo bias is introduced. See https://zuttobenkyou.wordpress.com/2012/10/18/generating-random-numbers-without-modulo-bias/.

Follow along in #724


Regex does not working for avoiding the JavaScript Injection

Description
The provided regex pattern in the documentation [1] for detecting the JavaScript injections is not working.
[1] - https://apim.docs.wso2.com/en/latest/deploy-and-publish/deploy-on-gateway/api-gateway/threat-protectors/regular-expression-threat-protection-for-api-gateway/#:~:text=<\sscript%5Cb%5B%5E%3E%5D%3E%5B%5E%3C%5D%2B%3C%5Cs*/%5Csscript%5Cs%3E
Steps to Reproduce
Attach the following sequence to the request flow of the API.
Please find the sequence below.
<sequence xmlns="http://ws.apache.org/ns/synapse" name="regex_policy">
    <log level="custom">
        <property name="IN_MESSAGE" value="Regular_expression_policy"/>
    </log>
    <property name="threatType" expression="get-property('threatType')" value="JavaScript-Injection"/>
    <property name="regex" expression="get-property('regex')" value="<\s*script\b[^>]*>[^<]+<\s*/\s*script\s*>"/>
    <property name="enabledCheckBody" expression="get-property('checkBodyEnable')" value="true"/>
    <property name="enabledCheckHeaders" expression="get-property('enabledCheckHeaders')" value="true"/>
    <property name="enabledCheckPathParams" expression="get-property('enabledCheckPathParams')" value="true"/>
    <class name="org.wso2.carbon.apimgt.gateway.mediators.RegularExpressionProtector"/>
</sequence>

Affected Component
APIM
Version
4.2.0
Environment Details (with versions)
No response
Relevant Log Output
No response
Related Issues
No response
Suggested Labels
No response

According to the internal discussion the patch was put on hold as it it not a bug. Hence closing this issue.


Library Coordinate Guro & Accel for WF or Unity

Required Info

Camera Model
{D455 }

Firmware Version
<IP_ADDRESS>

Operating System & Version
{Windows 10 LTSC

Platform
PC

SDK Version
{ 2.43.0}

Language
{C#/unity }

Segment
{others }

Issue Description
Good afternoon, ladies and gentlemen.
I made code like this
public static class GA
    {
        struct Vector3
        {
            public float x, y, z;
        }
       public static double[] GetDataGyro(int rounding)
        {
            var pipeline = new Pipeline();
            var cfg = new Config();
            cfg.EnableStream(Stream.Depth, 1280, 720, Format.Z16, 30);
            cfg.EnableStream(Stream.Color, 1280, 720, Format.Rgb8, 30);
            pipeline.Start();

            while (true)
            {
                using (var frameset = pipeline.WaitForFrames())
                {
                    var gyroFrame = frameset.FirstOrDefault<Frame>(Stream.Gyro, Format.MotionXyz32f).DisposeWith(frameset);
                    var Gyro = Marshal.PtrToStructure<Vector3>(gyroFrame.Data);
                    return new double[] { Math.Round(Gyro.x, rounding), Math.Round(Gyro.y, rounding), Math.Round(Gyro.z, rounding) };
                }
            }
        }
}

Collected the library and tried it like this
 static void Main(string[] args)
        {
            while (true)
            {
                double[] gc = GA.GetdataShortGyro(2);
                Console.WriteLine("x{0} y{1} z{2}", gc[0], gc[1], gc[2]);

                GC.WaitForPendingFinalizers();
            }
        }

In the console application, everything works as it should. But if I use WinForms then I get the common error
ExternalException: rs2_pipeline_wait_for_frames(pipe:0000015545087C20)
Rethrow as Exception: Frame didn't arrive within 5000

in the code
using (var frameset = pipeline.WaitForFrames())

Disconnecting the camera and reconnecting does not solve my problem.
I get exactly the same thing in Unity.
Questions:
1. How to make it work in Windows Forms?
2. How to make it work in Unity?
p.s. Although it seems to me that if this works in Windows Forms then it will work in Unity too

cfg.EnableStream (Stream.Gyro, Format.MotionXyz32f);
cfg.EnableStream (Stream.Accel, Format.MotionXyz32f);

_pp = _pipeline.Start (cfg, f =>

{

var profile = f.Profile;

});

I don't know how to apply the callback.
I'm in stupor. Help me to understand
Why do I need PiplenProfile ? that _pp?

Hi @anomal3  I wonder whether the problem is with use of the highlighted FirstOrDefault type of frameset command, given the complications of accessing IMU data described in the links below).
https://github.com/IntelRealSense/librealsense/issues/4803#issuecomment-528331115
https://github.com/IntelRealSense/librealsense/issues/5291#issuecomment-563436161
The Linq Carefully script in the SDK's C# Cookbook page of C# examples illustrates how f => can be used.
https://github.com/IntelRealSense/librealsense/blob/master/wrappers/csharp/Documentation/cookbook.md#linq-carefully

Hi @anomal3 I wonder whether the problem is with use of the highlighted FirstOrDefault type of frameset command, given the complications of accessing IMU data described in the links below).
#4803 (comment)
#5291 (comment)
The Linq Carefully script in the SDK's C# Cookbook page of C# examples illustrates how f => can be used.
https://github.com/IntelRealSense/librealsense/blob/master/wrappers/csharp/Documentation/cookbook.md#linq-carefully

I followed your advice, made a callback. And now I get this

Object does not support "librealsense::composite_frame" interface! "

Ohhh  Even an example

Hi @anomal3 I wonder whether the problem is with use of the highlighted FirstOrDefault type of frameset command, given the complications of accessing IMU data described in the links below).
#4803 (comment)
#5291 (comment)
The Linq Carefully script in the SDK's C# Cookbook page of C# examples illustrates how f => can be used.
https://github.com/IntelRealSense/librealsense/blob/master/wrappers/csharp/Documentation/cookbook.md#linq-carefully

Why did you shut up @MartyG-RealSense?

I realized this is impossible

I apologise for the delay in responding further due to a high number of RealSense customer support requests at this time.

I apologise for the delay in responding further due to a high volume of RealSense customer support requests at this time. Do you wish to continue with this case please?

Yes of course. only I have no idea how to move on

Further research of your problem found a link to a past case I handled about a Unity project for accessing the IMU in Unity with D435i.
https://github.com/IntelRealSense/librealsense/issues/5993#issuecomment-596358982

This is the best solution to solve my problem

Thanks very much @anomal3 for the update!


Add Github sign up

Add GitHub signup/login option.
Should include:

[ ] Easy way to enable integration by setting an env variable with the Auth code
[ ] It should be optional to enable it
[ ] It should skip email verification process as we can rely on GitHub having a good email
[ ] Should fill fields for github url automatically
[ ] Should ask for a password once authentification for sign up has succeeded  (to allow password login as well) - can be discussed

Refactoring this. I would still like to have it available but make it easier without as many dependencies we used to have

Closing this bc of inactivity


Use the 'await' expression instead of callbacks in Redstone

The initial support for the 'await' expression in Dart has been commited, as you can read here:
https://code.google.com/p/dart/source/detail?r=39345
Using 'await' is a tremendous advantage for server side application development. Asynchronous programming becomes very easy to implement.
Can you consider to refactor Redstone to making use of it in a new branch, even if it will be released officially in Redstone only in a later version when the Dart team will complete it? I'm starting a new project using Redstone and I'd like to write from the beginning my code to make use of the 'async...await' pattern, to avoid having to rewrite most of my code. Thank you for this wonderful framework anyway!

v0.6 (which I hope to publish on github soon) uses the new async/await and enum features, which makes the code easier to understand and maintain. Although, note that it doesn't stop you from using these features in your application right now, with the current (v0.5) stable version.

The first alpha release of Redstone v0.6 is available for test: https://github.com/luizmineo/redstone.dart/tree/v0.6

Hi, in response to your previous comment, I am having issues when I use async await in 0.5.21+. The code breaks whenever a completeError is returned from a Completer instance.
Failed to execute simple_app.SimpleAppApi.login - Wrong username or password.

"Wrong username and password" is the error message returned from the completer. On the client I get a 500 error message.
Code looks like this:
// File with login functionality
Future login(Map<String, String> credentials) {
		var completer = new Completer();
		retrieveDataFromDatabase().then(() {
			// Create session token
			var sessionToken = 'blahblah';

			completer.complete(sessionToken);
		})
                .catchError(() {
			completer.completeError("Wrong username or password.");
		});

		return completer.future;
	}

And the second file:
@app.Route('/login', methods: const [app.POST])
	login(@app.Body(app.JSON) Map payload) async {

		var credentials = payload["data"];
		var loginResponse = await new AuthenticationService(user).login(credentials);
        ......

As soon as the await at var loginResponse completes with an error(completer.completeError) this exception is thrown:
SEVERE: 2015-05-25 20:21:58.510: Failed to execute simple_app.SimpleAppApi.login - Wrong username or password.
dart:isolate  _RawReceivePortImpl._handleMessage

As a result I'm not able to capture this error and craft a response detailing this. Problem does not happen when I replace completeError with complete, but I'm not too keen on that.
I understand if you are not releasing any fixes for 0.5.21+ to focus on 0.6, I'll just use the workaround. At least this is something to test for in 0.6.

0.6 was released.


Compatible with MGISEQ fastq file

Hi,
As  MGISEQ fastq file contain '/1 | /2' at the end of the read ID, after adding UMI information, it will cause an error while mapping with BWA-MEM:
[mem_sam_pe] [mem_sam_pe] paired reads have different names: "V300041225L2C001R0010000001/1:umi_TTCAC_TTAGA", "V300041225L2C001R0010000001/2:umi_TTCAC_TTAGA"
How can I deal with this problem ?
Thanks in advance !
Best,
Gerde

Could you please paste some MGISEQ fastq reads here?

Hi, sorry for not replying on time.  MGISEQ read name looks like this:
Read 1:
@v300022634L1C001R0040000003/1
TCTGAAGTCCTCCTCTCCAGAAGCCCTGCCCAAGAAGAGCCAACAGACTTCCTGAGCCGCCTTCGAAGATGTCTTCCCTGCTCCCTGGGGCGAGGAGCAG
+
F?FDCAEFCBEB@FAECC<A?D<84DFBAC>FC7E@<D>>DFFABDD:?D6=E7?@:@C<EEAA99=1CE=9EEE>>E?D/93>CED+56A>=;>@C><D

Read 2:
@v300022634L1C001R0040000003/2
ACGGTTCTGTCCTGAAGGCTGGTGCTAGAGACTCCACACAGCCACCCCCTAGCTGAGTGATCTTAAACTAGCACCTTATCCCGTGTTCTGCCGGCTTCAT
+
DDA?8<@FEAC8FD2?5C=DDD6BDE@<:CBCD@=EDE:<E?EEE?EDEBDDDEC<;=9B?3F:B5@>BEEC@5BEED68FE;@:@DD2F<E5ECAFBA@

latest fastp (version 0.20.1) already support this type fastq by adding --fix_mgi_id.
for example, ./fastp -i R1.fq -I R2.fq -o O1.fq -O O2.fq --fix_mgi_id --umi --umi_loc=per_read --umi_len=4 --umi_prefix=UMI

Thanks for all developers, it really helps !
I will close this issues, thanks !


(PUP-6765) Ensure that produce/consume uses generated pcore resource

Ensures that the export/consume logic of the Puppet::Resource::Type
uses a generated Pcore resource type and never makes an attempt to load
the corresponding Ruby type.

CLA signed by all contributors.


Rabbit publisher

Reason/Context
The initial version of this template was released with rabbit subscriber, so a logical completion would be to add a rabbit publisher

Closing this issue as this was completed by https://github.com/asyncapi/go-watermill-template/pull/137 and released as v0.2.0


Remove use of futures_io crate

It currently uses AsyncWrite for TcpStream. But we have our own versions of most useful methods already.
The only missing function is TcpStream::send_all_vectored.
Related #305.

Blocked on  #353.

Blocked on  #353.

Done in #357, commit f2dfc69d4ef5e644b43d0465820ef773a872e0c1.

Done in #357, commit f2dfc69d4ef5e644b43d0465820ef773a872e0c1.


Don't log to stdout

The package prints  to stdout. Either remove logging or add an optional logger to Client so the application can control where the output goes.

Fixed


remove should_render

Closes #846, #847
Checklist
Please update this checklist as you complete each item:

[x] Tests have been included for all bug fixes or added functionality.
[x] The changelog.rst has been updated with any significant changes.
[x] GitHub Issues which may be closed by this Pull Request have been linked.

I'm actually ok merging this without review.


Added GoogleChrome, GoogleDrive, ScannerPro, Skype, Phone, RemindersA…

…pp, Linkedin, Music, and Remote
Salam :) Just small notes:

I tried the phone scheme it works for mobile numbers without +965, however, it doesn't work for home numbers and it keeps telling me:
الرقم المطلوب تم فصله ..
While the home number I tired works fine!

I would like to share with you this useful link:
http://lightarrow.com/lifetopix/launch-urls-for-popular-apps

I tried them all, the apps work with me I added them to Appz but there are many apps didn't work and some of them I couldn't try them because they are not available in Kuwaiti App Store e.g. Kindle, So I shared it with you maybe you can try them. Because I can't add apps without testing them.
Thanks.

yay, that looks much better! :D
About the phone, you can probably leave it like that. It's a standard, like mailto, people should know how and when to use it.
About the apps, I'll try to test the US store apps and get back to you.
For now, it's time to merge this in :rocket:

Thank you so much :) I will add more apps ASAP :3

We probably need an update to the readme as well, sorry I forgot to mention that :sweat_smile:

It's ok :) Updating is in progress ..


Generalized Resize

Related to #253

by using a naked Some instead of wrapping it in error handlng in validator-rust/src/components/gaussian_mechanism.rs line 169, are you creating the risk of a null error?

Consider adding a comment in the first reference to analytic, indicating its origin. self.analytic definition?

by using a naked Some instead of wrapping it in error handling in validator-rust/src/components/gaussian_mechanism.rs line 169, are you creating the risk of a null error?

Here? https://github.com/opendifferentialprivacy/whitenoise-core/pull/286/files#diff-ac1b8c2212cba362f060ca35454ac03fR169
It is ok to use options generally for things that may or may not exist. Components may or may not have an accuracy implementation. The components that do not have accuracy implementations return Ok(None) (successfully returned no known accuracy). In the outside code, errors are cleanly handled with the error handling system, and Ok(None) responses are filtered away.

Consider adding a comment in the first reference to analytic, indicating its origin. self.analytic definition?

Was this comment supposed to be tied to code somewhere?


Bitfield-ify UART timeout field.

Commits in this PR

Bitfield-ify UART timeout field.

PR chain

#338 Fix Flash/RAM size in STM32 board config.
👉 #347 Bitfield-ify UART timeout field. 👈 YOU ARE HERE
#340 Bitfield-ify UART request field.
#348 Bitfield-ify UART status register.
#342 Bitfield-ify UART intClear register.
#343 C++ nits in uart_dma.
#344 Templatize CircularBuffer's datatype.

https://github.com/RespiraWorks/VentilatorSoftware/pull/339 had to be duplicated here because the "merge" button was clicked before that PR was the head of the chain.  This is identical to that one.


Motrix下载报错#22

软件及系统如下,看了其他相关Issues,没找到Mac OS相关的解决配置

蹲个解决方案

My problem was the same as yours.
II ran into this problem when I downloaded the openSUSE iso.
url: https://mirrors.bfsu.edu.cn/opensuse/tumbleweed/iso/openSUSE-Tumbleweed-DVD-x86_64-Snapshot20201202-Media.iso
I don't know how to do that.

me, too

Mozilla/5.0 (Macintosh; Intel Mac OS X 11_1_0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/87.0.4280.141 Safari/537.36

@weiguozhao wei
@Xeonpoint X
需要自己填写User-agent 把上面的贴进去就可以了

Mozilla/5.0 (Macintosh; Intel Mac OS X 11_1_0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/87.0.4280.141 Safari/537.36

@weiguozhao wei
@Xeonpoint X
需要自己填写User-agent 把上面的贴进去就可以了

Mozilla/5.0 (Macintosh; Intel Mac OS X 11_1_0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/87.0.4280.141 Safari/537.36

@weiguozhao wei
@Xeonpoint X
@glntlq
需要自己填写User-agent 把上面的贴进去就可以了

Thank you for providing me with ideas to solve the problem.
I didn't expect that even downloading things like this would have user agent restrictions.

Mozilla/5.0 (Macintosh; Intel Mac OS X 11_1_0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/87.0.4280.141 Safari/537.36

@weiguozhao wei
@Xeonpoint X
@glntlq
需要自己填写User-agent 把上面的贴进去就可以了

Thank you for providing me with ideas to solve the problem.
I didn't expect that even downloading things like this would have user agent restrictions.

Thx

Thx


Fixed camel casing for dataattrs and dataattr.

Added missing space in {{ dataAttr.key/val }}

veldig bra. mangler en changelog entry. noe sånn som:
changed
multiple patterns: changed all variables named dataattr to camel case, dataAttr

Changelog oppdatert.


Computed attributes are only serialized if they have a corresponding regular attribute with the same name.

Laravel Version: 7.x
PHP Version: any
Database Driver & Version:

Description:
https://github.com/laravel/framework/blob/7.x/src/Illuminate/Database/Eloquent/Concerns/HasAttributes.php#L185
            // We want to spin through all the mutated attributes for this model and call
            // the mutator for the attribute. We cache off every mutated attributes so
            // we don't have to constantly check on attributes that actually change.
            if (! array_key_exists($key, $attributes)) {
                continue;
            }

The ! here in line 185 is erroneous, this means it only serializes computed attributes IF they have a corresponding attribute with the same name.  Therefore, the cache is only populated if it's already populated.
Steps To Reproduce:

Sounds like an optimization. Feel free to open up a discussion or send in a PR.

KK. Actually I think I misread the code. Please delete / ignore this, sorry!


Poor error location for duplicate-response

The error is very far away from where I would expect it to be (in return location).
Playground Link
That said, I wonder if this should even be an error. As an alternative, any return types with the same status code are unioned together, i.e. these two examples are equivalent when A or B have no status code or the same status code:
op foo(): A | B;
op foo(): { @body: A | B }

pri: 0


Migrate to Gatsby v2

Deploy preview for bettertw failed.
Built with commit 8785fe386de307d735ba25cdb123cb9a5be834a4
https://app.netlify.com/sites/bettertw/deploys/5c69243f431a76000a99a9ed


Info added

Description
Add the error info to let the user know the change in https://github.com/deepjavalibrary/djl/pull/1869.
Relavant issue: https://github.com/deepjavalibrary/djl/issues/1800

Codecov Report
Base: 72.08% // Head: 69.96% // Decreases project coverage by -2.12% :warning:

Coverage data is based on head (5bc8ab5) compared to base (bb5073f).
Patch coverage: 68.25% of modified lines in pull request are covered.

:exclamation: Current head 5bc8ab5 differs from pull request most recent head 3a20b0c. Consider uploading reports for the commit 3a20b0c to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2022      +/-   ##
============================================
- Coverage     72.08%   69.96%   -2.13%
- Complexity     5126     5948     +822
============================================
  Files           473      587     +114
  Lines         21970    26283    +4313
  Branches       2351     2860     +509
============================================
+ Hits          15838    18390    +2552
- Misses         4925     6511    +1586
- Partials       1207     1382     +175

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

...rc/main/java/ai/djl/modality/cv/output/Joints.java
71.42% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...main/java/ai/djl/modality/cv/output/Rectangle.java
72.41% <0.00%> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

...odality/cv/translator/BigGANTranslatorFactory.java
33.33% <0.00%> (+8.33%)
:arrow_up:

.../cv/translator/InstanceSegmentationTranslator.java
0.00% <0.00%> (-86.59%)
:arrow_down:

...nslator/InstanceSegmentationTranslatorFactory.java
7.14% <0.00%> (-11.04%)
:arrow_down:

.../cv/translator/SemanticSegmentationTranslator.java
0.00% <0.00%> (ø)

... and 500 more

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

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


OS X: Focus is not retained if fullscreen, with multiple windows, after cmd+tabbing

VSCode Version: April 2020 (version 1.45)
OS Version: Catalina MacOS

Steps to Reproduce:

Open two windows of VSCode (actual instances, not tabs).
Fullscreen the second window (ctrl+cmd+f)
cmd+tab to another app and back
You'll see it's focussed on the non-fullscreen original window.

Does this issue occur when all extensions are disabled?: Yes
Quite annoying. Perhaps it's just 'how macs handle fullscreen', and there's nothing to be done but worth reporting as it does strike me as a bug.

VSCode is built on top of a large number of components that are not maintained by the VSCode team directly. This includes:

Electron as the cross platform UI framework
node.js for most integration into the OS
Chromium as runtime and rendering engine (HTML, CSS, JS)
and after all, the operating system VSCode is running on

Reading this issue, it seems to originate not from VSCode itself but rather one of these upstream components. As such, in order to resolve it, it would need to get fixed in the upstream component. I will go ahead and close this issue and kindly ask to report this issue to the upstream component if possible:

for UI, OS integration: https://github.com/electron/electron
for Rendering: https://bugs.chromium.org/p/chromium/issues/list
for file IO: https://github.com/nodejs/node

Thanks for understanding.


Elements multiple selection

Feature to select more than one element. This will be very useful, since it's actually happens very often, when we want to change 3 elements at the same time.

@QuiiBz Have some progress with this feature. I'm thinking about inputs on the right panel. My solution to different colors and texts is just to display inputs of every element. What do you think about this? Just placing all inputs is fine, or we need another solution?

I think we should just show the value if it's the same, and "Mixed" / an empty one if they are different. Duplicating inputs will make it harder to understand what's going on.
Looking forward to your PR!

Now if we select two text with different color, we will see just blank color. Also, I think we need to place textarea for content if there is only one selected text.

Yup that's what I was thinking too. Amazing!


This fix the wrong size of layers

Its the first time i mess with gimp plugins but it kinda works

fixes #2


Update game over message

the pull request process was quite easy  and simple to use and understand . the pull request help to pull changes


Fix: bonfire-where-are-thou

Add a additional test, to make it a bit more difficult to use string comparison to pass the tests.
sts. At the moment the provided code will pass the test:
collection.map(function(obj){
      if(JSON.stringify(obj).includes(JSON.stringify(source).replace(/[{}]/g, ''))) arr.push(obj)
})

:+1: Looks good, thanks!


fix(mimetype): fix missing charset on certain non-text mime types

There was an issue where UTF-8 files were interpreted in other charset because their content-type not specified the UTF-8 charset and the HTML had other encoding.
In our case a JS file was uploaded to S3 with "content-type: application/javascript" without charset=UTF-8 and it caused issues on www env because it uses iso-8859-1.
resolve WMA-456
Co-authored-by: bmori<EMAIL_ADDRESS>

LGTM, thanks for removing the unused dep

:tada: This PR is included in version 2.4.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


interrupt does not wait for spawned process to finish

When interrupt is set to true and a process is spawned (spawn: true) there is a race condition.

on interrupt task.complete() is called and sends SIGINT to the spawned process
it then does not wait until the child process is really dead, but instead continues execution
the (new) process gets started (while the old one is still running)
the previous process finishes, causing all callbacks to fire and messing up the state of the task object

Though this can be mediated by some simple flags it is still a problem and can lead to resource problems, eg. when the spawned child sets up a listening socket while the previous child is still running. Also the output buffers are gone in this case.
To fix the issue it is required to layout all the complete() functions (taskrun.complete() and taskrunner.complete()) to work with callbacks, which is a major pain (I tried, but eventually gave up due to the complexity of the change).
However this is imho a major bug which leads to all kind of strange behaviour.

Looks like the problem here is that the interrupted task is not supposed to call done(), but grunt when interrupted returns status code 0 rather than 130, so done is called.


[REQUEST] Adding lutris support

Is your feature request related to a problem? Please describe.
No, it's not.
Describe the solution you'd like
I would like to make it so that there is support for lutris in the launcher itself in gnu/linux distributions. This would create some convenience in installing and running games.
Describe alternatives you've considered
Haven't looked into it yet
Additional context
No response

Related to #438, #279 and #299. We do have a pull request that aims to allow us to run games using Lutris itself but Linux is not having a good amount of attention by the maintainers.

To be honest we should aim to allow running games going straight to Wine or Proton but as we can't even fix the games installation I don't it happening.

To be honest we should aim to allow running games going straight to Wine or Proton but as we can't even fix the games installation (#304) probably not gonna happen soon.

So it's cool if from the program itself will be run, but it will be more convenient through lutris which has a lot of settings and support for other emulators (I know that wine/proton is not an emulator). I'll close this request then, thanks for the reply!


TypeError, line 111

File "/Users/mak/Downloads/suggestion-master/test.py", line 17, in <module>
    suggestion = Suggestion(text_field, dataset=DATASET)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/mak/Downloads/suggestion-master/suggestion/__init__.py", line 27, in __init__
    self._setup()
  File "/Users/mak/Downloads/suggestion-master/suggestion/__init__.py", line 111, in _setup
    self._dropdown.build(self.widget)
TypeError: Viewable.build() takes 1 positional argument but 2 were given

Please fix this error

Hi !
It seems that you don't have the latest version of Viewable. I recommend to install  Suggestion in a new virtual environment with pip install suggestion.
You can also upgrade Suggestion with pip install suggestion --upgrade --upgrade-strategy eager.
Once updated or newly installed, you can play with the demo interface: python -m suggestion.demo. It is a graphical user interface with 3 fields. A field to fill in a dictionary of words and two other fields (tkinter.Entry widget and tkinter.Text widget) to type text to see how Suggestion behaves.
I will probably update today the READMEs of both Suggestion and Viewable.
Let me know if this comment helps you!


feat(codegen): starcoder2

use VLlmDecoder and left-right codegen for starcoder2

can you also enable the 3b and 7b versions for starcoder2? thx

can you also enable the 3b and 7b versions for starcoder2? thx

done


Gpl976 create source plate event

Closes GPL976
Changes proposed in this pull request:

Add support for create partial source plate event (in progress)

[ ] Add implementation for all other event properties + tests
[ ] Add error condition behaviour (what happens when params do not validate from user point of view)
[ ] Add SS plate creation message support
[ ] Service client for cherrytracker
[ ] Service client for mongo
[ ] Cherrytracker implementation
[ ] Refactor tests / remove code duplicated or not needed

Closed as all changes are in the epic branch: https://github.com/sanger/lighthouse/pull/380/files


Payment summary

@sanjaypoyzer @lizziebruce @AbiBarkerOxfordshire
Changes for Oxfordshire localisation: Paying offline / outside of transaction #658 are done, please see the screenshots below
/pre-payment
Payment summary
When we’ve processed and approved your application, we’ll email you an invoice with a link to pay online.
You will be invoiced for:
[show all rows except cost]
We’ll calculate the cost of your permit. The invoice we send you will show the amount you need to pay.
NEXT

@Artgov Could you resolve the conflicts on this one? Thanks


ISAM create sysaccounts question

I'm having trouble creating an ISAM sysaccount that is assigned to groups. I can see there is a parameter but cannot see an example showing the expected format. Can someone show me an example please?

using this role (/base/configure_sysaccounts) , adding a user that is member of an EXISTING group would be like this.
sysaccounts:
- sysaccount_action: create
id: sample_user
password: "{{ sample_user_password }}" # password should be secured with ansible-vault
groups:
- samplegroup1
- samplegroup2
However, I do see that there's code missing in the modules to modify groupmemberships and the "modify" function for a user only changes the password.
Note that it may better to use "management authorization" with LDAP users/groups instead. This allows you to manage the access in an identity management system you probably already have, and allows your admin users to login with an id that they can manage themselves (in LDAP).

Thanks! I had been trying to use create_sysaccount but the formatting used when providing a list of groups seems to be the same
Not sure if it's my understanding of your reply or what you wrote, but I figured it out whilst trying to make that role work that I needed to list the groups like this. I was surprised to see I could list two different groups under the same heading in the way I did too, but it works so not complaining!
groups:
- id: ""
- id: ""
Although I've used ISAM (v5.1+) for some time, I'm new to ansible and relatively inexperienced with YAML so it's probably just my understanding :( Probably going to get sick of me asking questions until I get familiar enough with YAML and the docs to figure everything out myself :-s


Make InlineQueryResultVideo title optional

Is there a real reason to keep it required? I have to use InlineQueryResultMpeg4Gif for videos to make it look standardized with titleless InlineQueryResultPhoto and InlineQueryResultGif.

Currently, videos are shown as articles, because they can be big and often can't be downloaded when used as a result in inline mode. Therefore, the title is required for them.


Change compute measures video loading

We want to make the measures load on click

Video now loads when name is clicked on in ListBox holding all names available to be viewed


Form validation on fallback mode

The form validation on recaptcha works just fine on most modern browsers. However, I noticed that when it's using the fallback option, it's returning a form validation error for recaptcha when the user copy & paste into the input.

Did a little bit more debugging and I found out that with the fallback option, it's not calling the callback from the POST. I'm guessing this is by design. I'll just have to figure another way around in the meantime.

This ticket is a duplicate of https://github.com/VividCortex/angular-recaptcha/issues/46. By applying the discussed patch the fallback mode works fine.

Closing as duplicate then. Please reopen if i'm wrong.


Simplified Chinese (zh_CN) for YOURLS 1.7

https://github.com/ZvonimirSun/YOURLS-zh_CN
Simplified Chinese files for YOURLS 1.7。
I found the zh_CN files on YOURLS/awesome-yourls is still pointed to guox/yourls-zh_CN which is out of date and for YOURLS 1.6。

Thanks for that @ZvonimirSun!
Could you then open a pull request on https://github.com/YOURLS/awesome-yourls with this remplacement? 👍

@LeoColomb Sure, I have done that now.
https://github.com/YOURLS/awesome-yourls/pull/36

Thanks! Closing this in favor of that PR.


Add lorem ipsum type text block with character count

Description
It would be good to be able to determine a character count for a block of text. This could be super useful for testing max/min validation on inputs
Proposed solution
We could either update an existing function to take an optional characterCount param (maybe text or words). Or we could create a new function entirely?
E.G
function text(characterCount = 200): string {

Alternatives considered
No response
Do you want to create a pull request?
Yes

Yes, we can do text({ characterCount: number })

Ok cool. If you assigned this to me, i'll get that implemented

You can make the changes now according to our new code style, please


Multiple Sensors Running in Interrupt Example

It would be nice if the "HelloWorld_Interrupt" code worked with multiple sensors. It looks like it is close. It has a "SATELLITES_MOUNTED" definition and "begin" and "off" for the satellites, but only ever initializes, starts, and reads measurements for the "top" sensor.

Hello @JGarrett7 ,
we did not implement the interrupt code for multiple sensors because by default the interrupt pins for the satellites are not routed on the Arduino header. You must close some solder bridges to get the interrupt pins of the satellites routed on the connector. So, I prefer to avoid releasing a code that does not work out-of-the-box with the default hardware configuration of the shield. The reason why you find some code for the satellites in this example is because if you keep the satellites mounted, you must disable them using the xshut pins to get the sensor on board of the shield working properly in interrupt mode; otherwise you would have an hardware conflict. So, in this case you must uncomment the "SATELLITES_MOUNTED" define in the sketch. If you do not have the satellites mounted, you can keep the "SATELLITES_MOUNTED" define commented and everything should work without any issue.
Best Regards,
Carlo


Updating version and CHANGELOG for new release: v0.3.6.

Codecov Report

Merging #206 into master will not change coverage by %.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #206   +/-   ##
=======================================
  Coverage   95.98%   95.98%
=======================================
  Files          45       45
  Lines        2389     2389
=======================================
  Hits         2293     2293
  Misses         96       96

Impacted Files
Coverage Δ

statick_tool/__init__.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


On denying permissions for first time and trying to allow it again it shows ReTry?

Description
On denying permissions for first time and trying to allow it again it shows ReTry.
I was expecting it to ask for permission again, instead of showing ReTry and sending the user to app settings screen.
After going through other apps with camera permission, there it asks for permission from within the app to access camera and does not send the user outside of the app to settings -> apps.
How to repeat issue and example
Install a new app.

Click on camera icon
deny permissions
click on camera icon again
it shows to retry

Solution
How can i get the same dialog for allowing the user to allow camera and gallery access instead of sending them to settings page.
Additional Information

React Native version: 0.42.3
Platform: Android
Development Operating System: MacOS
Dev tools: Android Studio 2.2.3

Use this solution.
https://github.com/react-community/react-native-image-picker/issues/330


New version: Baulk.Baulk version 4.3.3

Created by 🥟 Dumplings in workflow run #4189.
Logs
No ReleaseNotes (en-US) for version 4.3.3
Updated: 4.3.2 → 4.3.3
Submitting WinGet manifests

Microsoft Reviewers: Open in CodeFlow

Hi @SpecterShell,

This PR's version number 4.3.3 has 3 parameters (sets of numbers between dots - major, minor, etc), which is less than the current manifest's version <IP_ADDRESS>1, which has 4 parameters.

Does this PR's version number exactly match the version reported in the Apps & features Settings page? (Feel free to attach a screenshot.)
(Automated response - build 876)


Add wallet height to bottom toolbar

What current issue(s) does this address, or what feature is it adding?
Monitoring wallet height requires getting all the wallet info which can be slow depending on how many address/tokens/contracts exist in the wallet
How did you solve this problem?
Added the current wallet height to the bottom toolbar in the same format neo-cli uses (wallet height/db height/blockchain height)
How did you make sure your solution works?
Manual testing
Are there any special changes in the code that we should be aware of?
No
Please check the following, if applicable:

[ ] Did you add any tests?
[x] Did you run make lint?
[x] Did you run make test?
[x] Are you making a PR to a feature branch or development rather than master?
[x] Did you add an entry to CHANGELOG.rst? (if not, please do)

Wallet._current_height seems to always be one higher than the db/blockchain heights. Does this track? Should I modify the height to subtract 1 for display purposes to avoid confusion?

Coverage remained the same at 86.571% when pulling 792d0d774814d4dfbb44b16e118e3d7625102319 on hal0x2328:development into b8105d0c35b5231494d153a6e416f683c07b5f2a on CityOfZion:development.

I actually encountered this issue yesterday as well in my asyncio branch.
If you could change this https://github.com/CityOfZion/neo-python/blob/fe90f62e123d720d4281c79af0598d9df9e776fb/neo/Wallets/Wallet.py#L653-L662
to this
            while self._current_height < Blockchain.Default().Height and (block_limit == 0 or blockcount < block_limit):

                block = Blockchain.Default().GetBlockByHeight(self._current_height)

                if block is not None:
                    self.ProcessNewBlock(block)
                else:
                    break

                blockcount += 1

and a single wallet rebuild will be enough
at least that way we avoid confusion when looking at the toolbar code

Ok, made the requested change

👍


[Bug]: Display still after deleting partition

ODC version
odc423
OB version

What happened?

What did you expect to happen?
No Q partition exists after deletion
How can we reproduce it (as minimally and precisely as possible)?
Delete partitions after creating a multi partition table
Anything else we need to know?
No response
Cloud
No response

Creating a partition will not display it either

Fixed In Commit d159ed302f3b0dc56ea571c7166e65cf8a72ae94
------
咏和
Sun Dec 10 10:59:51 2023 +0800

pass


[Android] Notification not being received on device when app in background

Notification is not being received on Android device when app is in background.
Notification is received when app is in foreground.

What version of RN and react-native-fcm are you running?
RN: 0.53.0
react-native-fcm: 13.3.1

What device are you using? (e.g iOS9 emulator, Android 6 device)?
Samsung Galaxy S7 edge, Android 7 device

Is your app running in foreground, background or not running?
Background

Help, please.

Hmm, so it seems as though the device does receive the notification but the notification dropdown doesn't display... it just goes straight into the notification list. You can see it when you swipe down the screen.
Does anyone know how to make the notification show up on the screen when the device receives it?
The Notification setting for the App is on and is not set to Silent.

In your server or where ever you use to send the notification, use custom notification instead of normal notification format. You can find custom notification format in example folder

let me know if @kyonzz 's solution doesn't work


deck: 1.40.1 -> 1.40.2

Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Done.
Let's see if I can still use the merge bot after all the checks are done.

Result of nixpkgs-review pr 344763 run on x86_64-linux 1

1 package built:

deck

Result of nixpkgs-review pr 344763 run on x86_64-darwin 1

1 package built:

deck

My recollection is that mergebot can only be used for PRs created by r-ryantm, but I can hold off on merging this PR so you can give it a try.

My impression of the merge bot is to allow maintainers to quickly release/fix their packages.
If it only works for PRs created by r-ryantm, the benefit of using the merge bot is quite limited.  Maintainers has no control when this bot creates the PR after all.
Furthermore, I did not expect to wait so long for ofborg to complete all checks... it has been 8 hours and still in the queue...

@NixOS/nixpkgs-merge-bot merge

My recollection is that mergebot can only be used for PRs created by r-ryantm, but I can hold off on merging this PR so you can give it a try.

You are right... merge bot can not be used for this PR.


redis-py-cluster support pipeline in cluster mode.

see title, thank U.

@rfyiamcool Issues is not a tool to ask general questions about supported features of the lib. I would suggest you read the docs before ask questions. Here for example http://redis-py-cluster.readthedocs.io/en/master/pipelines.html Otherwise you can always ask on the gitter chat channel for help and general questions.


Update before publishing to crates.io

Related to #35

changed visibility to only export compile functions
added minor doc comments
removed redundant Cargo.toml info
updated README.md

Was this requested by anyone? When did we decide on that? @Arcticae

Was this requested by anyone? When did we decide on that? @Arcticae
I see a possible reasoning behind that, just wanna know the your decision process

hello! we at Spacehard requested, it would help us with integrating in the devnet :)


Pass used tokens and cost to Segment Inoke Agent event

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

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

🔖 Feature description
LangChain passes tokens used and cost in the agent callback. Aggregate these values for each invocation and pass to Segment Invoked Agent event.
📝 Additional Context
No response
Acknowledgements

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

@elisalimli can you take a look at this?

@homanp We can't get the usage parameters if the streaming is enabled. Is it ok for us if we pass the cost parameters if the streaming is not enabled?
Reference:
https://github.com/langchain-ai/langchain/issues/3114#issuecomment-1695705394

@homanp We can't get the usage parameters if the streaming is enabled. Is it ok for us if we pass the cost parameters if the streaming is not enabled?
Reference:
langchain-ai/langchain#3114 (comment)

So only when streaming is enabled?

Yeah, the usage, cost parameters is only returned when streaming is disabled

Wondering if there is some way for us to count tokens ourselves @elisalimli I'm pretty sure this is what LiteLLM does.

Yeah!
https://cookbook.openai.com/examples/how_to_count_tokens_with_tiktoken
We can count it and return an approximate cost price.

@elisalimli let's do that, please have a look at how LiteLLM does it. They have count for every LLM.


QR Code Sharing

Add in a QR code button for quick sharing between devices.
Really useful for when writing something on the PC and then using a phone to

This has been implemented in commit #0bc4942924e3680120a9d8979f27c25892f82a3d


Correct typo in README file

Codecov Report

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

@@           Coverage Diff           @@
##           master     #106   +/-   ##
=======================================
  Coverage   92.14%   92.14%
=======================================
  Files           9        9
  Lines         433      433
=======================================
  Hits          399      399
  Misses         34       34

Continue to review full report at Codecov.

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


doesn't work , still show nothing on panel

windows 64
terminal 0.14.5

same for OSX
was compiled against a different Node.js version using
NODE_MODULE_VERSION 46. This version of Node.js requires
NODE_MODULE_VERSION 53. Please try re-compiling or re-installing
the module (for instance, using `npm rebuild` or`npm install`). Error: The module '/Users/radjivcarrere/.atom/packages/terminal-plus/node_modules/pty.js/bin/darwin/x64.node'
was compiled against a different Node.js version using
NODE_MODULE_VERSION 46. This version of Node.js requires
NODE_MODULE_VERSION 53. Please try re-compiling or re-installing
the module (for instance, using `npm rebuild` or`npm install`).
    at process.module.(anonymous function) [as dlopen] ```

Any idea? Cheers

same here both on mac 10.13.1 and windows 10 atom 13.1.1 node-gyp is installed

same here linux manjaro atom 1.22.1x64 black panel with blinking cursor without prompt line. Can't input any text.

I don't think they still maintain this package anymore but I still had the issue, maybe you can try platformio-ide-terminal? It is a fork of terminal-plus and it works for me.