CLOUDSTACK-8935 - Cannot remove [r]VPC networks due to RTNETLINK error

This PR fixes the "sequence item 0: expected string, NoneType found" error found in the CsDhcp.py file when attempting to remo a network from a VPC.

Ping @remibergsma @wido @borisroman @miguelaferreira @karuturi @bhaisaab @DaanHoogland
Additional non-hardware required tests still running, but I would like to share the results of the ones which require hardware.
Test Environment

Management Server + MySQL on CentOS 7.1
Two KVM hosts on CentOS 7.1
Agent and Common RPMs built from source

Tests Results (hardware_required = true)
Create a redundant VPC with two networks with two VMs in each network and check default routes ... === TestName: test_02_redundant_VPC_default_routes | Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policy on RouterVM ... === TestName: test_02_routervm_iptables_policies | Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policies on VPC router ... === TestName: test_01_single_VPC_iptables_policies | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test nics after destroy ... === TestName: test_01_VPC_nics_after_destroy | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test default routes ... === TestName: test_02_VPC_default_routes | Status : SUCCESS ===
ok
Test to create Load balancing rule with source NAT ... === TestName: test_01_create_lb_rule_src_nat | Status : SUCCESS ===
ok
Test to create Load balancing rule with non source NAT ... === TestName: test_02_create_lb_rule_non_nat | Status : SUCCESS ===
ok
Test for assign & removing load balancing rule ... === TestName: test_assign_and_removal_lb | Status : SUCCESS ===
ok
Stop existing router, add a PF rule and check we can access the VM ... === TestName: test_isolate_network_FW_PF_default_routes | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_RVR_Network_FW_PF_SSH_default_routes | Status : SUCCESS ===
ok
Test to verify access to loadbalancer haproxy admin stats page ... === TestName: test02_internallb_haproxy_stats_on_all_interfaces | Status : SUCCESS ===
ok
Test create, assign, remove of an Internal LB with roundrobin http traffic to 3 vm's ... === TestName: test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | Status : SUCCESS ===
ok
Test SSVM Internals ... === TestName: test_03_ssvm_internals | Status : SUCCESS ===
ok
Test CPVM Internals ... === TestName: test_04_cpvm_internals | Status : SUCCESS ===
ok
Test stop SSVM ... === TestName: test_05_stop_ssvm | Status : SUCCESS ===
ok
Test stop CPVM ... === TestName: test_06_stop_cpvm | Status : SUCCESS ===
ok
Test reboot SSVM ... === TestName: test_07_reboot_ssvm | Status : SUCCESS ===
ok
Test reboot CPVM ... === TestName: test_08_reboot_cpvm | Status : SUCCESS ===
ok
Test destroy SSVM ... === TestName: test_09_destroy_ssvm | Status : SUCCESS ===
ok
Test destroy CPVM ... === TestName: test_10_destroy_cpvm | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 20 tests in 9543.149s

Cheers,
Wilder

Hi all,
Please keep it on hold: I tested the network deletion manually before the component/test_vpc_offerings.py was completed. Deleting the network manually worked fine, but I still have the exception with the tests. Perhaps the test needs some tuning. I will have a look into it.
Cheers,
Wilder

@wilderrodrigues ping me when you want me to run some tests!

@wilderrodrigues i'm on it when it's ready!

I'm running the new test now. Once done, I will push it and you can proceed with testing the PR.
Cheers,
Wilder

Ping @remibergsma @miguelaferreira @borisroman @DaanHoogland
It's now fixed for real!
Test create VPC offering ... === TestName: test_01_create_vpc_offering | Status : SUCCESS ===
ok
Test VPC offering without load balancing service ... === TestName: test_03_vpc_off_without_lb | Status : SUCCESS ===
ok
Test VPC offering without static NAT service ... === TestName: test_04_vpc_off_without_static_nat | Status : SUCCESS ===
ok
Test VPC offering without port forwarding service ... === TestName: test_05_vpc_off_without_pf | Status : SUCCESS ===
ok
Test VPC offering with invalid services ... === TestName: test_06_vpc_off_invalid_services | Status : SUCCESS ===
ok
Test update VPC offering ... === TestName: test_07_update_vpc_off | Status : SUCCESS ===
ok
Test list VPC offering ... === TestName: test_08_list_vpc_off | Status : SUCCESS ===
ok
test_09_create_redundant_vpc_offering (integration.component.test_vpc_offerings.TestVPCOffering) ... === TestName: test_09_create_redundant_vpc_offering | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 8 tests in 1964.378s

OK
/tmp//MarvinLogs/test_vpc_offerings_3GSCL8/results.txt (END)

I will run more tests and have an extensive report here soon.
Cheers,
Wilder

Cool. Will run the build again with this PR

Ping @remibergsma @miguelaferreira @DaanHoogland @borisroman
More tests:

2 KVM hosts on CentOS 7.1
Agent + Common built from source
Management Server + MySQL on CentOS 7.1

Test start/stop of router after addition of one guest network ... === TestName: test_01_start_stop_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test reboot of router after addition of one guest network ... === TestName: test_02_reboot_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test to change service offering of router after addition of one guest network ... === TestName: test_04_chg_srv_off_router_after_addition_of_one_guest_network | Status
 : SUCCESS ===
ok
Test destroy of router after addition of one guest network ... === TestName: test_05_destroy_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test to stop and start router after creation of VPC ... === TestName: test_01_stop_start_router_after_creating_vpc | Status : SUCCESS ===
ok
Test to reboot the router after creating a VPC ... === TestName: test_02_reboot_router_after_creating_vpc | Status : SUCCESS ===
ok
Tests to change service offering of the Router after ... === TestName: test_04_change_service_offerring_vpc | Status : SUCCESS ===
ok
Test to destroy the router after creating a VPC ... === TestName: test_05_destroy_router_after_creating_vpc | Status : SUCCESS ===
ok
Test router internal advanced zone ... === TestName: test_02_router_internal_adv | Status : SUCCESS ===
ok
Test restart network ... === TestName: test_03_restart_network_cleanup | Status : SUCCESS ===
ok
Test router basic setup ... === TestName: test_05_router_basic | Status : SUCCESS ===
ok
Test router advanced setup ... === TestName: test_06_router_advanced | Status : SUCCESS ===
ok
Test stop router ... === TestName: test_07_stop_router | Status : SUCCESS ===
ok
Test start router ... === TestName: test_08_start_router | Status : SUCCESS ===
ok
Test reboot router ... === TestName: test_09_reboot_router | Status : SUCCESS ===
ok
test_privategw_acl (integration.smoke.test_privategw_acl.TestPrivateGwACL) ... === TestName: test_privategw_acl | Status : SUCCESS ===
ok
Test reset virtual machine on reboot ... === TestName: test_01_reset_vm_on_reboot | Status : SUCCESS ===
ok
Test advanced zone virtual router ... === TestName: test_advZoneVirtualRouter | Status : SUCCESS ===
ok
Test Deploy Virtual Machine ... === TestName: test_deploy_vm | Status : SUCCESS ===
ok
Test Multiple Deploy Virtual Machine ... === TestName: test_deploy_vm_multiple | Status : SUCCESS ===
ok
Test Stop Virtual Machine ... === TestName: test_01_stop_vm | Status : SUCCESS ===
ok
Test Start Virtual Machine ... === TestName: test_02_start_vm | Status : SUCCESS ===
ok
Test Reboot Virtual Machine ... === TestName: test_03_reboot_vm | Status : SUCCESS ===
ok
Test destroy Virtual Machine ... === TestName: test_06_destroy_vm | Status : SUCCESS ===
ok
Test recover Virtual Machine ... === TestName: test_07_restore_vm | Status : SUCCESS ===
ok
Test migrate VM ... === TestName: test_08_migrate_vm | Status : SUCCESS ===
ok
Test destroy(expunge) Virtual Machine ... === TestName: test_09_expunge_vm | Status : SUCCESS ===
ok
Test VPN in VPC ... === TestName: test_vpc_remote_access_vpn | Status : SUCCESS ===
ok
Test VPN in VPC ... === TestName: test_vpc_site2site_vpn | Status : SUCCESS ===
ok
Test to create service offering ... === TestName: test_01_create_service_offering | Status : SUCCESS ===
ok
Test to update existing service offering ... === TestName: test_02_edit_service_offering | Status : SUCCESS ===
ok
Test to delete service offering ... === TestName: test_03_delete_service_offering | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 32 tests in 6656.221s

OK

Cheers,
Wilder

Create a redundant VPC with two networks with two VMs in each network ... === TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network and check default routes ... === TestName: test_02_redundant_VPC_default_routes | Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policy on RouterVM ... === TestName: test_02_routervm_iptables_policies | Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policies on VPC router ... === TestName: test_01_single_VPC_iptables_policies | Status : SUCCESS ===
ok
Stop existing router, add a PF rule and check we can access the VM ... === TestName: test_isolate_network_FW_PF_default_routes | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_RVR_Network_FW_PF_SSH_default_routes | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test nics after destroy ... === TestName: test_01_VPC_nics_after_destroy | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test default routes ... === TestName: test_02_VPC_default_routes | Status : SUCCESS ===
ok
Test to create Load balancing rule with source NAT ... === TestName: test_01_create_lb_rule_src_nat | Status : SUCCESS ===
ok
Test to create Load balancing rule with non source NAT ... === TestName: test_02_create_lb_rule_non_nat | Status : SUCCESS ===
ok
Test for assign & removing load balancing rule ... === TestName: test_assign_and_removal_lb | Status : SUCCESS ===
ok
Test to verify access to loadbalancer haproxy admin stats page ... === TestName: test02_internallb_haproxy_stats_on_all_interfaces | Status : SUCCESS ===
ok
Test create, assign, remove of an Internal LB with roundrobin http traffic to 3 vm's ... === TestName: test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | Status : SUCCESS ===
ok
Test SSVM Internals ... === TestName: test_03_ssvm_internals | Status : SUCCESS ===
ok
Test CPVM Internals ... === TestName: test_04_cpvm_internals | Status : SUCCESS ===
ok
Test stop SSVM ... === TestName: test_05_stop_ssvm | Status : SUCCESS ===
ok
Test stop CPVM ... === TestName: test_06_stop_cpvm | Status : SUCCESS ===
ok
Test reboot SSVM ... === TestName: test_07_reboot_ssvm | Status : SUCCESS ===
ok
Test reboot CPVM ... === TestName: test_08_reboot_cpvm | Status : SUCCESS ===
ok
Test destroy SSVM ... === TestName: test_09_destroy_ssvm | Status : SUCCESS ===
ok
Test destroy CPVM ... === TestName: test_10_destroy_cpvm | Status : SUCCESS ===
ok

Ran 21 tests in 10657.776s
OK
Test router internal advanced zone ... === TestName: test_02_router_internal_adv | Status : SUCCESS ===
ok
Test restart network ... === TestName: test_03_restart_network_cleanup | Status : SUCCESS ===
ok
Test router basic setup ... === TestName: test_05_router_basic | Status : SUCCESS ===
ok
Test router advanced setup ... === TestName: test_06_router_advanced | Status : SUCCESS ===
ok
Test stop router ... === TestName: test_07_stop_router | Status : SUCCESS ===
ok
Test start router ... === TestName: test_08_start_router | Status : SUCCESS ===
ok
Test reboot router ... === TestName: test_09_reboot_router | Status : SUCCESS ===
ok
test_privategw_acl (integration.smoke.test_privategw_acl.TestPrivateGwACL) ... === TestName: test_privategw_acl | Status : SUCCESS ===
ok
Test reset virtual machine on reboot ... === TestName: test_01_reset_vm_on_reboot | Status : SUCCESS ===
ok
Test advanced zone virtual router ... === TestName: test_advZoneVirtualRouter | Status : SUCCESS ===
ok
Test Deploy Virtual Machine ... === TestName: test_deploy_vm | Status : SUCCESS ===
ok
Test Multiple Deploy Virtual Machine ... === TestName: test_deploy_vm_multiple | Status : SUCCESS ===
ok
Test Stop Virtual Machine ... === TestName: test_01_stop_vm | Status : SUCCESS ===
ok
Test Start Virtual Machine ... === TestName: test_02_start_vm | Status : SUCCESS ===
ok
Test Reboot Virtual Machine ... === TestName: test_03_reboot_vm | Status : SUCCESS ===
ok
Test destroy Virtual Machine ... === TestName: test_06_destroy_vm | Status : SUCCESS ===
ok
Test recover Virtual Machine ... === TestName: test_07_restore_vm | Status : SUCCESS ===
ok
Test migrate VM ... === TestName: test_08_migrate_vm | Status : SUCCESS ===
ok
Test destroy(expunge) Virtual Machine ... === TestName: test_09_expunge_vm | Status : SUCCESS ===
ok
Test VPN in VPC ... === TestName: test_vpc_remote_access_vpn | Status : SUCCESS ===
ok
Test VPN in VPC ... === TestName: test_vpc_site2site_vpn | Status : SUCCESS ===
ok
Test to create service offering ... === TestName: test_01_create_service_offering | Status : SUCCESS ===
ok
Test to update existing service offering ... === TestName: test_02_edit_service_offering | Status : SUCCESS ===
ok
Test to delete service offering ... === TestName: test_03_delete_service_offering | Status : SUCCESS ===
ok
Test create VPC offering ... === TestName: test_01_create_vpc_offering | Status : SUCCESS ===
ok
Test VPC offering without load balancing service ... === TestName: test_03_vpc_off_without_lb | Status : SUCCESS ===
ok
Test VPC offering without static NAT service ... === TestName: test_04_vpc_off_without_static_nat | Status : SUCCESS ===
ok
Test VPC offering without port forwarding service ... === TestName: test_05_vpc_off_without_pf | Status : SUCCESS ===
ok
Test VPC offering with invalid services ... === TestName: test_06_vpc_off_invalid_services | Status : SUCCESS ===
ok
Test update VPC offering ... === TestName: test_07_update_vpc_off | Status : SUCCESS ===
ok
Test list VPC offering ... === TestName: test_08_list_vpc_off | Status : SUCCESS ===
ok
test_09_create_redundant_vpc_offering (integration.component.test_vpc_offerings.TestVPCOffering) ... === TestName: test_09_create_redundant_vpc_offering | Status : SUCCESS ===
ok
Test start/stop of router after addition of one guest network ... === TestName: test_01_start_stop_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test reboot of router after addition of one guest network ... === TestName: test_02_reboot_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test to change service offering of router after addition of one guest network ... === TestName: test_04_chg_srv_off_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test destroy of router after addition of one guest network ... === TestName: test_05_destroy_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test to stop and start router after creation of VPC ... === TestName: test_01_stop_start_router_after_creating_vpc | Status : SUCCESS ===
ok
Test to reboot the router after creating a VPC ... === TestName: test_02_reboot_router_after_creating_vpc | Status : SUCCESS ===
ok
Tests to change service offering of the Router after ... === TestName: test_04_change_service_offerring_vpc | Status : SUCCESS ===
ok
Test to destroy the router after creating a VPC ... === TestName: test_05_destroy_router_after_creating_vpc | Status : SUCCESS ===
ok

Ran 40 tests in 8310.689s
OK

LGTM, based on a set of tests that I run on this branch:
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=true \
component/test_vpc_redundant.py \
component/test_routers_iptables_default_policy.py \
component/test_routers_network_ops.py \
component/test_vpc_router_nics.py \
smoke/test_loadbalance.py \
smoke/test_internal_lb.py \
smoke/test_ssvm.py

Result:
ok
Test iptables default INPUT/FORWARD policies on VPC router ... === TestName: test_01_single_VPC_iptables_policies | Status : SUCCESS ===
ok
Stop existing router, add a PF rule and check we can access the VM ... === TestName: test_isolate_network_FW_PF_default_routes | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_RVR_Network_FW_PF_SSH_default_routes | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test nics after destroy ... === TestName: test_01_VPC_nics_after_destroy | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test default routes ... === TestName: test_02_VPC_default_routes | Status : SUCCESS ===
ok
Test to create Load balancing rule with source NAT ... === TestName: test_01_create_lb_rule_src_nat | Status : SUCCESS ===
ok
Test to create Load balancing rule with non source NAT ... === TestName: test_02_create_lb_rule_non_nat | Status : SUCCESS ===
ok
Test for assign & removing load balancing rule ... === TestName: test_assign_and_removal_lb | Status : SUCCESS ===
ok
Test to verify access to loadbalancer haproxy admin stats page ... === TestName: test02_internallb_haproxy_stats_on_all_interfaces | Status : SUCCESS ===
ok
Test create, assign, remove of an Internal LB with roundrobin http traffic to 3 vm's ... === TestName: test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | Status : SUCCESS ===
ok
Test SSVM Internals ... === TestName: test_03_ssvm_internals | Status : SUCCESS ===
ok
Test CPVM Internals ... === TestName: test_04_cpvm_internals | Status : SUCCESS ===
ok
Test stop SSVM ... === TestName: test_05_stop_ssvm | Status : SUCCESS ===
ok
Test stop CPVM ... === TestName: test_06_stop_cpvm | Status : SUCCESS ===
ok
Test reboot SSVM ... === TestName: test_07_reboot_ssvm | Status : SUCCESS ===
ok
Test reboot CPVM ... === TestName: test_08_reboot_cpvm | Status : SUCCESS ===
ok
Test destroy SSVM ... === TestName: test_09_destroy_ssvm | Status : SUCCESS ===
ok
Test destroy CPVM ... === TestName: test_10_destroy_cpvm | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 21 tests in 11171.944s

OK

And:
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=false \
smoke/test_routers.py \
smoke/test_network_acl.py \
smoke/test_privategw_acl.py \
smoke/test_reset_vm_on_reboot.py \
smoke/test_vm_life_cycle.py \
smoke/test_vpc_vpn.py \
smoke/test_service_offerings.py \
component/test_vpc_offerings.py \
component/test_vpc_routers.py

Result:
Test to update existing service offering ... === TestName: test_02_edit_service_offering | Status : SUCCESS ===
ok
Test to delete service offering ... === TestName: test_03_delete_service_offering | Status : SUCCESS ===
ok
Test create VPC offering ... === TestName: test_01_create_vpc_offering | Status : SUCCESS ===
ok
Test VPC offering without load balancing service ... === TestName: test_03_vpc_off_without_lb | Status : SUCCESS ===
ok
Test VPC offering without static NAT service ... === TestName: test_04_vpc_off_without_static_nat | Status : SUCCESS ===
ok
Test VPC offering without port forwarding service ... === TestName: test_05_vpc_off_without_pf | Status : SUCCESS ===
ok
Test VPC offering with invalid services ... === TestName: test_06_vpc_off_invalid_services | Status : SUCCESS ===
ok
Test update VPC offering ... === TestName: test_07_update_vpc_off | Status : SUCCESS ===
ok
Test list VPC offering ... === TestName: test_08_list_vpc_off | Status : SUCCESS ===
ok
test_09_create_redundant_vpc_offering (integration.component.test_vpc_offerings.TestVPCOffering) ... === TestName: test_09_create_redundant_vpc_offering | Status : SUCCESS ===
ok
Test start/stop of router after addition of one guest network ... === TestName: test_01_start_stop_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test reboot of router after addition of one guest network ... === TestName: test_02_reboot_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test to change service offering of router after addition of one guest network ... === TestName: test_04_chg_srv_off_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test destroy of router after addition of one guest network ... === TestName: test_05_destroy_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test to stop and start router after creation of VPC ... === TestName: test_01_stop_start_router_after_creating_vpc | Status : SUCCESS ===
ok
Test to reboot the router after creating a VPC ... === TestName: test_02_reboot_router_after_creating_vpc | Status : SUCCESS ===
ok
Tests to change service offering of the Router after ... === TestName: test_04_change_service_offerring_vpc | Status : SUCCESS ===
ok
Test to destroy the router after creating a VPC ... === TestName: test_05_destroy_router_after_creating_vpc | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 40 tests in 8082.832s

OK

Awesome work @wilderrodrigues ! Now the 3 tests that were failing all the time pass :-) Really happy about that! Thank you :+1:


fix: use standard serialized vector of frs in function tree cbinds

Description
Please provide a paragraph or two giving a summary of the change, including relevant motivation and context.
Checklist:

[ ] I have reviewed my diff in github, line by line.
[ ] Every change is related to the PR description.
[ ] I have linked this pull request to the issue(s) that it resolves.
[ ] There are no unexpected formatting changes, superfluous debug logs, or commented-out code.
[ ] The branch has been merged or rebased against the head of its merge target.
[ ] I'm happy for the PR to be merged at the reviewer's next convenience.

Lgtm
On Thu, Apr 20, 2023, 10:36 AM David Banks @.***> wrote:

Description
Please provide a paragraph or two giving a summary of the change,
including relevant motivation and context.
Checklist:

I have reviewed my diff in github, line by line.
Every change is related to the PR description.
I have linked
https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue
this pull request to the issue(s) that it resolves.
There are no unexpected formatting changes, superfluous debug logs,
or commented-out code.
The branch has been merged or rebased against the head of its merge
target.
I'm happy for the PR to be merged at the reviewer's next convenience.

You can view, comment on, or merge this pull request online at:
https://github.com/AztecProtocol/aztec3-packages/pull/314
Commit Summary

1b5402b
https://github.com/AztecProtocol/aztec3-packages/pull/314/commits/1b5402b261a839a7be2a1ee4a2a6618423ae93d5
use a plain old serialized vector of frs instead of doing manual
serialization logic for leaf frs

File Changes
(3 files https://github.com/AztecProtocol/aztec3-packages/pull/314/files
)

M circuits/cpp/src/aztec3/circuits/abis/c_bind.cpp
https://github.com/AztecProtocol/aztec3-packages/pull/314/files#diff-2496d2735ce4a0673043dbad1ea49b3f587eaf49a20d71564b22b3e4b06830a4
(142)
M circuits/cpp/src/aztec3/circuits/abis/c_bind.h
https://github.com/AztecProtocol/aztec3-packages/pull/314/files#diff-34a02ce73ffd12364262c25c93b85d9a3630fa4e1521ad6b904033ef751874f7
(2)
M circuits/cpp/src/aztec3/circuits/abis/c_bind.test.cpp
https://github.com/AztecProtocol/aztec3-packages/pull/314/files#diff-47a2dd92ffb322c6257ec1a891c98471db1b54be7bd0d75c39a4a3aa328e64cd
(92)

Patch Links:

https://github.com/AztecProtocol/aztec3-packages/pull/314.patch
https://github.com/AztecProtocol/aztec3-packages/pull/314.diff

—
Reply to this email directly, view it on GitHub
https://github.com/AztecProtocol/aztec3-packages/pull/314, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABIBGIQCAYMEDENIMTEPCLXCFCWBANCNFSM6AAAAAAXFTFPQM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


[1.16.2] java.lang.NullPointerException: Ticking entity

Crash: https://pastebin.com/DSQ9vPCX

I have remove erroring entities on and the crash still happens

This has been identifed and a fix is being built for this now.    Should be submitted for approval in the next few minutes.
But just in case, please provide worldseed and x,y,z location.

This should be fixed now!
Thanks for creating the issue.


Proposal: Option to mask credentials on generated YAML secrets when debug enabled

Output with debug flag prints as expected. Generated YAML shows secrets with crednetials. There sohlud be an option to disable the credentials display and still take advantage of debug feature.
Output of helm version:
v3.0.1
Output of kubectl version:
v1.16.6
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
Openstack

@bibliobeam thanks for opening - can you suggest how you'd like this implemented? (i.e. name of flag, or just does this by default, etc)

Duplicate of https://github.com/helm/helm/issues/7275.

#7275 already has a backward compatible solution, of adding an additional flag "sanitize", proposed for a similar issue. I'll look at contributing the fix as part of that issue.


Looking forward to the Light version

Is Light supported in any version? 1 or 2?

Actually v2 does have partial support for Operator Mono Light.
https://github.com/kiliman/operator-mono-lig/tree/master/ligature/OperatorMonoLig-Light/glyphs

Awesome, I'll try this later tonight! Thanks

Working great, thanks for your hard work


SSR only serves static page, never hydrates in IE11.

I'm running a basic nuxt.js website, works perfect on Chrome, Safari, Firefox and Edge.
One issue:
On IE11, it seems to only serve the static page, and the javascript bundle never hydrates the static page.
Any insight into how I might get around this issue?
Should I look into generating a static site or???

How did you solve the issue?

I convinced my boss that we didn't have to support ie11.

I don't think this is a SSR issue.
The reason is because I tried generating a static bundle and serving the site with a koa server.
The same thing happened: index.html loads on IE11, but the javascript bundles were not being loaded.
My guess is that there is something in the _nuxt/nuxt.bundle.js files that IE11 doesn't like.

Maybe IE is in need of a polyfill? Probably Map/WeakMap, MutationObserver, requestAnimationFrame, or Promises.
I know when I was trying to get Aurelia building/executing in various permutations of IE, those were the culprits, so I imagine it might be the same in Vue.

i reckon you use babel-polyfill together with useBuiltIns option in babel preset – it's not a nuxt issue.

Vuex need a polyfill for Promises on IE11...
Check the following IE11 trace error, on  click:

Yes, it was a polyfill issue. Adding in:
  build: {
    vendor: [
      'babel-polyfill'
    ],
    babel: {
      plugins: [
          ...
      ]
    },
  },

to my nuxt.config.js file, and adding require("babel-polyfill"); to layout/default.vue resolves this issue.


Support Filters for RazorPages

Right now filters are not executed for Razor Pages.
I use filters to inject HTML content in the views based on external rules.
Examples:

Widgets based on the current context (User, url, time of day) independently from the view
Inject static resources like Scripts, Styles in the Layout based on the components that are executed during the page rendering

Presumably this is means action and result filters. We support, Resource, Auth and exception filters in RazorPages as of now.

In this case we're talking about result filters. We'll likely have a story similar to action filters, but different. Result filters will probably 'just work'.

688e5189910634f3ade4f00bedd1cc1f71072cdc
Result filters added


webpackMode: "lazy-once" not supported by Webpack 4.17.2

The documentation at https://webpack.js.org/api/module-methods/#import- mentions a "lazy-once" mode option but when specifying it such as in:
import( /webpackChunkName: "moduleB-chunk"/ /* webpackMode: "lazy-once" */'./moduleB');
webpack reports:
WARNING in ./src/moduleA.js 5:0-88
webpackMode expected 'lazy', 'eager' or 'weak', but received: lazy-once.
Isn't "lazy-once" supported?
Webpack version: 4.7.12

@webpack-bot move to webpack/webpack.js.org

Let me check

Same here

Looking at code it should be possible to use "lazy-once". Pinged @sokra to have a quick look at this

It only makes sense when using an import() with expression:
import(/* webpackMode: "lazy-once" */ `./module${expr}`) // ok
import(/* webpackMode: "lazy-once" */ `./moduleA`) // not ok

The different between lazy and lazy-once is that lazy-once loads all possible modules at once, that doesn't make sense when only one module is possible.

See also the note in the documentation

Note that this only makes sense in the case of a partially dynamic statement, e.g. import(./locales/${language}.json), where there are multiple module paths that could potentially be requested.

@montogeek Since the query was answered and also the information is explicitly mentioned in docs, I guess we should go ahead and close this to just keep the count of issues in check :)


Verify &nbsp in code

Moved from GCCode: https://gccode.ssc-spc.gc.ca/iitb-dgiit/nw-ws/sgdc-cdts/issues/107
@StdGit created the issue
Some &nbsp (space) could interfere with ie rendering
We must check to remove them

@StdGit Could you provide an example? I can check it...

@MaximMartel Could you confirm this issue is still valid for version 4.0.39 ? Thanks.

Not supported. Closing issue.

Not supported. Closing issue.


Build and deploy nightly packages

We want to build and deploy binary packages in RPM & DEB format at least every day, potentially as part of every build.
We'll target at least CentOS/RHEL 7 for now, potentially 8.
Debian I'm unsure which versions make sense as I'm not too familiar with it.

First iteration has been merged, we can consider this complete for now.


Hi

sorry, 6 commits

Has this project been abandoned?  I'd like the option added in this pull request.

I personally don't use it any more (since I don't need to fetch email). If you'd like to help maintain it I'd be happy to add you.


Data for ECU-R-PRO not matching native interface

The values reported

don't match the values from the native web interface
the values for both solar panels of each inverter are always the same, even if the native interface offers independent readings.

See screenshots

(the ones from 3 to 6 are not posted for conciseness)
Original interface

same issue, waiting for support

ECU-R exposes the inverters, not the separate panels.
However, it does expose the inverters twice, once with integer data + scale factors in model 101/102/103 (single/split/multi phase), once as floats (Sunspec models 111/112/113).
I only have one inverter, so I don't know if each inverter shows up separately, or if it's all added up by the ECU-R into one big "model" object.

You need to assign each inverter a different Modbus address on the ECU-R but there is no separate panel data indeed.

Thanks


add instructions for getting secrets from batch/storage via az cli

Do you think the CLI should just link to the repo's docs on how to do this, or should it actually give instructions itself?

I think putting it in the docs is sufficient. SOmething like:

install cli
az login
the get batch keys command
the get storage keys command

But if we're gonna do this, we might as well provide instructions on how to create batch and storage on the cli too

Are you saying we have login and batch/storage key retrieval functionality on the command line?
Or just instructions on how to do those things?

I think he means just showing how to do it using the Azure Command Line (not implement it ourselves). @jiata - I think we should not ask users to install the CLI, but probably defer to the Portal Cloud Shell instead since it should be pretty quick and painless.

This might be helpful. It lists some instructions on using the CLI to create storage and batch accounts, and uses jq to pull out the keys into variables.
It'd be really cool if we could use sed to directly modify the secrets.json file with the variables we pull out of the jq queries.


[TechDocs CLI] More peace-of-mind E2E and Unit Tests

Browser-based E2E Test for techdocs-cli serve
We've run into several issues recently where bumps to Backstage packages and plugins have resulted in a non-functional preview environment when running techdocs-cli serve.
We added basic end-to-end tests for the CLI itself in backstage/techdocs-cli#55.  We should add one more case that actually loads up a headless browser and asserts that the Backstage app loads and renders as expected.
Unit tests for techdocs-cli publish
Although most publishers are quite well-covered in @backstage/techdocs-common, we don't have any validation in this repo for the publish command.  It might be sufficient to just ensure that we call publisher.publish with the expected args.

not stale


example README small typo

Automated message from Dropbox CLA bot
@lbguilherme, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here and update the thread so we can consider merging your code.

Done.

Thanks!


checkboxes don't work on German keyboard layout

with German keyboard layout
OS: Linux Mint 17.3
v.1.3

I've been able to replicate this. That's pretty unfortunate, it seems the / is under a shift key on a German keyboard, and on the place where on my keyboard a ~ is. Would it be ok to also add ctrl + ~ as a shortcut?

~ is only accessible via Alt Gr + '+' in German

any other suggestions?

could you use Alt Gr + [

Chiming in to add that on Hungarian keyboard the characters [ ] / are all under either Shift or Alt as is ~ so folding and checkboxes don't work via the keyboard.
On a Hungarian keyboard only the following special characters are available without a modifier key: , . -
My recommendation would be to keep all shortcuts to letters, that would work across all languages.
ps. great little app btw, kudos!

Chiming in to add that on Hungarian keyboard the characters [ ] / are all under either Shift or Alt as is ~ so folding and checkboxes don't work via the keyboard.
On a Hungarian keyboard only the following special characters are available without a modifier key: , . -
My recommendation would be to keep all shortcuts to letters, that would work across all languages.
ps. great little app btw, kudos!

@fonorobert Thanks! with regards to the shortcut, a letter would work for all latin script indeed, but It would need to be one that doesn't interfere with other common shortcuts (it can't be ctrl+C, for example)
any objections to alt+b or ctrl+b ?

Ctrl+B might be confusing as it's the shortcut for bold text in most word processors. Alt+B would work although it might be better to keep it on Ctrl to be consistent with the other shortcuts already in the app. Maybe Ctrl+H would be an option? Then fold/unfold could be Ctrl+J and Ctrl+K, this way most shortcuts would be on the home row for touch typists :)
On 2017. Jan 2. 21:13 +0100, Kilian Valkhof<EMAIL_ADDRESS>wrote:

@fonorobert (https://github.com/fonorobert) Thanks! with regards to the shortcut, a letter would work for all latin script indeed, but It would need to be one that doesn't interfere with other common shortcuts (it can't be ctrl+C, for example)
any objections to alt+b or ctrl+b ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (https://github.com/Kilian/fromscratch/issues/42#issuecomment-270015587), or mute the thread (https://github.com/notifications/unsubscribe-auth/AGpB_ozP3cmdFAULhNaw_gzKwUKJLOMFks5rOVpZgaJpZM4LLfYj).

I want to keep the ctrl+[ and ctrl + / shortcuts too, so I need to pick some that are not likely to become better suited for later functionality (when it comes up)

Ctrl+L for checkbox
Ctrl+K for fold/unfold?

That sounds reasonable :)
On 2017. Jan 14. 23:08 +0100, Kilian Valkhof<EMAIL_ADDRESS>wrote:

I want to keep the ctrl+[ and ctrl + / shortcuts too, so I need to pick some that are not likely to become better suited for later functionality (when it comes up)
• Ctrl+L for checkbox
• Ctrl+K for fold/unfold?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

still no checkboxes for German?

Thanks for the reminder, completely overlooked this. I have added the aforementioned Ctrl+L and Ctrl+K shortcuts, they will be in 1.4.1

released here: https://github.com/Kilian/fromscratch/releases/tag/v1.4.1

👍

Aug. 2018, 00:24 von<EMAIL_ADDRESS>

Closed > #42 https://github.com/Kilian/fromscratch/issues/42> .
—
You are receiving this because you authored the thread.
Reply to this email directly, > view it on GitHub https://github.com/Kilian/fromscratch/issues/42#event-1763811239> , or > mute the thread https://github.com/notifications/unsubscribe-auth/AAuGiclFI0vHBEw21Qtf-_L_FPto8HoOks5uMMsrgaJpZM4LLfYj> .


about the dataset format

if I got the dataset like
sentenceA sentenceB label
this is a sentence similarity / paraphrase identification task,
the label is a float number as similarity degree [0,1]
how to I use it with this project

你的这个也可以用应该，处理下数据，将原有输入换下


oni-ml: Where is the beaconing detection code?

Can someone point me in the direction of the code that does beaconing detection based on flow data?
Thanks

Howdy Thealy.
There is no beaconing-specific code. The way the spot suspicious connects analysis works is by building a probabilistic model of typical network behavior. When a network log entry is observed whose estimated probability of occurring (per the constructed model) is unusually low, it is flagged as suspicious for further analysis.

It is listed on the top level page for Apache Spot, along with side channel
data escapes, reflection attacks. I find this very confusing.
On Oct 13, 2016 3:00 PM, "NathanSegerlind"<EMAIL_ADDRESS>wrote:

Howdy Thealy.
There is no beaconing-specific code. The way the spot suspicious connects
analysis works is by building a probabilistic model of typical network
behavior. When a network log entry is observed whose estimated probability
of occurring (per the constructed model) is unusually low, it is flagged as
suspicious for further analysis.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Open-Network-Insight/open-network-insight/issues/102#issuecomment-253606675,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArI6O1tDU4IvX4I-r6LIdApKl4neG1Tks5qzn-2gaJpZM4KUNdQ
.

Sorry for any confusion Terry. Through the operational analytics and user interface these patterns (e.g. beaconing) become apparent. It is also necessary to have some context about the network and hosts to detect side channel attacks, find inappropriate data exfiltration, etc.
Once you're up and running I'd be happy to walk you through using the interface and filtering the data to find results.
Alan
Sent via the Samsung Galaxy S7 edge, an AT&T 4G LTE smartphone
-------- Original message --------
From: Terry Healy<EMAIL_ADDRESS>Date: 10/13/16 12:08 PM (GMT-08:00)
To: Open-Network-Insight/open-network-insight<EMAIL_ADDRESS>Subject: Re: [Open-Network-Insight/open-network-insight] oni-ml: Where is the beaconing detection code? (#102)
It is listed on the top level page for Apache Spot, along with side channel
data escapes, reflection attacks. I find this very confusing.
On Oct 13, 2016 3:00 PM, "NathanSegerlind"<EMAIL_ADDRESS>wrote:

Howdy Thealy.
There is no beaconing-specific code. The way the spot suspicious connects
analysis works is by building a probabilistic model of typical network
behavior. When a network log entry is observed whose estimated probability
of occurring (per the constructed model) is unusually low, it is flagged as
suspicious for further analysis.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Open-Network-Insight/open-network-insight/issues/102#issuecomment-253606675,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArI6O1tDU4IvX4I-r6LIdApKl4neG1Tks5qzn-2gaJpZM4KUNdQ
.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/Open-Network-Insight/open-network-insight/issues/102#issuecomment-253608780, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AIIW6xt9d-x196e3xvyHgX3iaiFmZLh1ks5qzoGJgaJpZM4KUNdQ.

That sounds excellent Alan. I'm making progress, working on the ML chain
now, some Spark issues.
-Terry
On Oct 13, 2016 3:36 PM, "Alan Ross"<EMAIL_ADDRESS>wrote:

Sorry for any confusion Terry. Through the operational analytics and user
interface these patterns (e.g. beaconing) become apparent. It is also
necessary to have some context about the network and hosts to detect side
channel attacks, find inappropriate data exfiltration, etc.
Once you're up and running I'd be happy to walk you through using the
interface and filtering the data to find results.
Alan
Sent via the Samsung Galaxy S7 edge, an AT&T 4G LTE smartphone
-------- Original message --------
From: Terry Healy<EMAIL_ADDRESS>Date: 10/13/16 12:08 PM (GMT-08:00)
To: Open-Network-Insight/open-network-insight <
<EMAIL_ADDRESS>Subject: Re: [Open-Network-Insight/open-network-insight] oni-ml: Where is
the beaconing detection code? (#102)
It is listed on the top level page for Apache Spot, along with side channel
data escapes, reflection attacks. I find this very confusing.
On Oct 13, 2016 3:00 PM, "NathanSegerlind"<EMAIL_ADDRESS>wrote:

Howdy Thealy.
There is no beaconing-specific code. The way the spot suspicious connects
analysis works is by building a probabilistic model of typical network
behavior. When a network log entry is observed whose estimated
probability
of occurring (per the constructed model) is unusually low, it is flagged
as
suspicious for further analysis.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<https://github.com/Open-Network-Insight/open-network-
insight/issues/102#issuecomment-253606675>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AArI6O1tDU4IvX4I-
r6LIdApKl4neG1Tks5qzn-2gaJpZM4KUNdQ>
.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub<https://github.com/
Open-Network-Insight/open-network-insight/issues/102#
issuecomment-253608780>, or mute the thread<https://github.com/
notifications/unsubscribe-auth/AIIW6xt9d-x196e3xvyHgX3iaiFmZLh1ks5qzoGJ
gaJpZM4KUNdQ>.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Open-Network-Insight/open-network-insight/issues/102#issuecomment-253615800,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArI6Ad2m9xqPanR_bOsXf8FQpalf-H0ks5qzog2gaJpZM4KUNdQ
.

Thanks @0xf00


need unlock those locking subject for the login LeetCode member

🚀 Feature Proposal
hope can unlock those locking subject for the login LeetCode member

Do you mean that you have bought premium account but cannot answer those locked problem?

yes

Thanks, it's dup with #409


chore: promote origins-cms to version 0.1.0

chore: promote origins-cms to version 0.1.0
this commit will trigger a pipeline to generate the actual kubernetes resources to perform the promotion which will create a second commit on this Pull Request before it can merge

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

Needs approval from an approver in each of these files:

OWNERS

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

@binomytech: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

verify
2d2f95313ba71e772e9ccadd6274afdb1f7f2085
link
/test verify

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.


Add golint/vet/errcheck etc checks

Could put a new target in the Makefile and/or the Travis config

done.


potential to escalate 'at risk' students

Is your feature request related to a problem? Please describe.
This is linked to an identified gap for us where the likelihood of contacting students through OnTask's electronic messaging feature is very low when working with groups of students who are displaying disengaged (at risk) behaviours
Describe the solution you'd like
IN the process of using OnTask's text conditions to identify the different cohorts of students an instructor might contact with different messages, the filtering matrix will highlight groups of students that are likely 'at risk.'  Example: in the hypothetical scenario below (see image), 3 datasets are being used to filter students: whether the student is repeating the course (blue), attending a required lab or tutorial (yellow), and the level of achievement on formative/low stakes assessments before a mid-term test (grey).

Those students that are failing the assessments, not attending as expected, and are either repeat or new students are displaying 'at risk' behaviours (red cells) and are also (in our experience to date) unlikely to respond to OnTask messages from the instructor. We would like the option for instructors to be able to 'escalate' these specific groups of students to whatever local process (mentors, student support service, Assoc Dean of Students, etc) might exist for handling or supporting disengaged/at-risk students. Within our institution, that escalation can occur via an email to a specific team or office. In the same way that OnTask currently allows an instructor to remove an individual email address from the final list of recipients, it would be ideal if an instructor can redirect a group of students (identified within the text conditions page - see image below) to an existing entity that is specifically tasked with and has the capacity to work with 'at risk' students. This also benefits the course instructor by reducing the workload represented by working with disengaged students and redirect it to the appropriate services that already exists to support these students.

Describe alternatives you've considered
Within the screenshot above, it might be ideal and simple for instructors to use an additional button (represented by the red exclamation button for clarity) at the bottom of a 'Conditions' tile that allowed them to escalate (redirect via email) the group(s) of identified students to the appropriate team/office/service.

Thanks @leichtweis for the new scenario. We haven't considered this before, but it makes perfect sense as it is another type of action that can be derived from the data.
After considering this new scenario, I have a suggestion. Instead of overloading the operations in the same action (that is, instead of adding an "additional button"), my suggestion is to allow a new type of action that instead of sending a message to the student, it sends the list of students selected by the filter to a specific email address. Under this proposal, the platform would then have an additional action called "Escalate" (or something similar) that would allow to write a simple email and insert the list of students at some point in the middle of the text. This way, the email would look something line:

Dear student services.
Here is the list of students that we have identified as "at risk" following our previous conversations:
{{ FILTER_RESULT }}
Regards.
That action then can be executed (or scheduled!) as any other with the change that upon execution a new email address (in this case student services) will be required as the destination of the message.
Does this make sense? I think it embraces the scenario you describe it, but it casts it as another action instead of overloading an existing action.
Two more aspects would fit perfectly this new frame:

You would remove the students at risk from the personalized action with a filter.

You could clone that filter condition into the new action (submit to an email) and simply negate the condition, thus selecting the students excluded from the personalised action, and including that list in the email to student services.

Comments?

@abelardopardo this makes sense and would certainly be a welcome addition to the tool from our perspective. My suggestion to highlight or tag filtered cohorts based on the tiles in the "text conditions page" because a lot of our users find that visual an easy way to get a sense of the spread of student's engagement behaviours and where they "fall" in relation to the filtered data.

Two new action types created to implement this.


General API Usability Improvements

After a discussion with @dpuccetti @mrDarcyMurphy @kofalt, this is a list of API improvements that would make frontend dev easier. This ticket could be turned into an epic when prioritized:

[ ] Send empty array for subdocument field instead of missing array (rolled into #189)
[ ] Documentation (ref #63)
[x] Ability to return all permissions for requests that return lists when user is operating as admin (rolled into #189)
[ ] Add pagination to requests that return lists

Empty subdocuments and subarrays are guaranteed per our data model. I'm surprised to hear that this is not upheld in practice. Should be easy and fast to fix. @nagem Can we make it part of the v2 migration?
[ignores the d-word] 😉
Does the permissions item refer to list queries? Should already be true for detail queries. Could be solved by a simple query param that prevents permission stripping. @rentzso Complexity?
Pagination has been on our list for a long time. I'm not opposed, but I'm also not convinced it will solve any user's problems. Filtering, for instance, will need to be moved server-side. It's certainly more complex than simply returning the first n items.
For reference, the 1627 sessions we have in the Stanford instance only amount to 80Kb, an come out in less than 200ms. I'm guessing we don't even have the right index on the perms. Should we just do pagination in the UI?

for list queries adding a query param switch should be a one line addition in containerhandler and collectionshandler.
For detail queries, the full list of permissions is already returned for superuser requests.

@rentzso, I edited the list above to specify that permissions were requested for get_alls, thanks for clarifying.

Can we make it part of the v2 migration

Sure can.

I updated https://github.com/scitran/core/issues/189 to have a clear list of what changes it includes. Let me know if we want to remove the final two checkboxes I added from this list.

I would support closing this ticket when 1 and 3 are done. Documentation is already tracked as noted above. And we may or may not open a ticket for pagination, pending a discussion with the UI team.

Since 1 and 3 are being rolled into #189, I'm also fine just closing it now. 2/4 ain't bad @dpuccetti @mrDarcyMurphy ¯\_(ツ)_/¯.

An empty array for a subdocument field won't make a large difference, we'll have to do a different (or maybe an additional) null check. Feeling ambivalent about this one.
It's tedious, but we'd all benefit from good documentation. The exercise of writing the docs and testing  the API may elucidate further ways it can be improved. Eventually a customer will be curious about it too. Luck favors the prepared.
WOOT! This will be really helpful.
I'm looking forward to this as well, but I think we all agree that it's a large-ish project that spans across teams.

So, yeah, I'm fine with closing this too.

@nagem Here's one: better error messages. While building a feature I ended up trying to add a user to a group that was already in the group. The response was:
HTTPNotFound: Element not added in list roles of container groups foo

404 seems like the wrong error code for that, and it would've been more helpful if it told me why the request was rejected not merely that it was (ie. "User is already in that group").

@mrDarcyMurphy I'm going to quote your comment in #66 where we are tracking some major error-handling work and then close this ticket.


chore(deps): update dependency tslint-plugin-prettier to v2.3.0

This PR contains the following updates:

Package
Type
Update
Change

tslint-plugin-prettier
devDependencies
minor
2.2.0 -> 2.3.0

Release Notes

prettier/tslint-plugin-prettier
v2.3.0
Compare Source
Features

support prettier@2 (#361) (be9ed18)

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

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

v3.2.3
GitHub release

Your semantic-release bot :package::rocket:


obvious mistake or is that a deliberate choice behind the screen

hey, I agree that it's an obvious mistake but is there any other valid reason or serious reason why the amount is different from the index page to the checkout page?
plan 5$ week become  5$ /month
plan 12$ week become 15$/month
I guess that this is just a parameter inside dashboard wrongly setup

Hello, it was a typo on the index.html pages. I've corrected it and changed the demo. You should see the $14 subscription redirects to checkout as $14 now. Thank you!


'let' causing variables to disappear in for-loop

This only occurs in conjunction with the es2015 plugin.
The following code
function getSum(data) {
  let total = 0;
  for (let i = 0; i < data.length; i++) {
    total += data[i];
  }
  return total;
}

gets transpiled to
"use strict";function getSum(a){for(var c=0;c<a.length;c++)a[c];return b}

which returns the undefined variable b. Seemingly total has been removed.
The same code using var i instead of let i produces the expected result
"use strict";function getSum(a){for(var c=0,b=0;b<a.length;b++)c+=a[b];return c}

Hey @michaelmcleodnz! We really appreciate you taking the time to report an issue. The collaborators
on this project attempt to help as many people as possible, but we're a limited number of volunteers,
so it's possible this won't be addressed swiftly.
If you need any help, or just have general Babel or JavaScript questions, we have a vibrant Slack
community that typically always has someone willing to help. You can sign-up here
for an invite.

If it helps, this seems to be because vars declared before a for-loop are pulled inside the for-loop declaration, which can't be done with lets because it changes their scope.
Babili knows not to pull the lets inside a for-loop without the es2015 plugin, so I'm not sure what the order of operations would be here that causes it to pull the variable inside the for-loop and then decide that it can be optimised away because it is let-scoped.

I've found that this issue has already been reported in #430 and #412.
Line 89 of merge-sibling-variables checks that the for-loop declaration is of type var, but presumably somewhere in minify-dead-code-elimination it is still believed to be of type let.

Fixed in #713


Update constants.py

Added the atoms Bromine, Chlorine, Iodine and Selenium to the ELE_TO_NUM bidict. All important Medchem atoms.

Thanks for the additions. I am slightly hesitant with this as additional constants are outside the scope of MolML. You can imagine that this could easily be expanded to all the elements.

I see where the hesitation is coming from. The only reason I added it was
because I was using this on a real dataset.
Best,
Laksh
On Wed, 31 Jul 2019, 17:57 Chris Collins<EMAIL_ADDRESS>wrote:

Merged #4 https://github.com/crcollins/molml/pull/4 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/crcollins/molml/pull/4?email_source=notifications&email_token=AKTEYJ662A3NFFCYVO7OHY3QCHAAJA5CNFSM4IIFBFJ2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOSZX3M6Q#event-2523903610,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKTEYJ4FTCSHO7ZZGZ4G55LQCHAAJANCNFSM4IIFBFJQ
.


Open source the repo

Task Summary
📅 Due date: N/A
🎯 Success criteria: Open source the repo.
This task tracks a list of TODOs for FOSSing.
To Do

[x] Change repo name to starling-integrity-api
[x] Add MIT license
[x] Verify no credentials / sensitive info are in the code and issues
[ ] Make sure it's well documented
[x] Make this repo public

@benhylau Are there any other documentation tasks that need to happen before we can mark this ticket as "done"? If yes, can you clarify what should be further documented, so that I can get it done?

I think we should doc the create API. This way, we can have clients other than starling-capture sending to this. I think it's a matter of taking from here: https://github.com/starlinglab/starling-integrity-api/issues/3#issuecomment-978061281 what actually got implemented (e.g. what error codes are actually returned, what multipart keys are actually read, etc.) and putting it in README.md or a API.md.
I think @YurkoWasHere has some ansibles for server provisioning. It may be good to add a sentence linking to that.

Also need to rename the subfolder to starlingintegrityapi from starlingcaptureapi still.

As per the conversation in https://starlinglabhq.slack.com/archives/C02LDUFLSR2/p1648247634341929, also rename the repo to remove api from the name, which is no longer accurate.

@benhylau the one remaining task in this issue is "Move content from UA data model pad to here", so I'm reassigning the ticket for you to complete and close.


In TimetableItemDetailScreen, behavior of TopAppBar is different from Figma.

Overview

With Figma , if you scroll down about the middle of the page, you see the TopAppBar, but now you have to scroll to the top of the page to see it.

Figma
actual

I've finished other tasks, so I'd like to start working on this one. 🙋


Change Heartbeat/Ephemeral combination to use a single TTL field

Drops the Ephemeral/Heartbeat config field combination to use a single TTL field. Most of the stores are using TTL's, so this is better overall.
fixes #23
replaces #24
Signed-off-by: Alexandre Beslic<EMAIL_ADDRESS>

ping @yihungjen (sorry for opening another PR and thank you for raising the issue in the first place). Let me know if that sounds good to you.

ping @aluzzardi @vieux

LGTM


Update retrieveAttributes

Don't print the intCheck console warning if no attributeId is provided.

@wesdawg , before this one gets merged, can you put brackets after the if statements to make the code cleaner? Thanks!
For example, change this:
if (attributeId != undefined)
    ...

to this:
if (attributeId != undefined) {
   ...
}


Add Material

This adds the new iOS 15+ Material ShapeStyle. They currently only work for background modifiers on the web using backdrop-filter.
Tokamak:

SwiftUI:

I added a snapshot test, but the filter doesn't seem to be screenshot correctly...


Re-enable CI in Jenkins

Ok, that's enough churn for one PR. I've disabled the failing CI again and would like to see this merged for now. Note that Jenkins might fail -- the grafana apt repository seems to be broken at the moment.

Yeah, Jenkins failures are grafana apt repository being broken right now.


Javascript over WordPress

why we are learning javascript if we can create websites through WordPress. We can modify them and edit as we want even create interactive websites.
What's the importance of learning javascript over WordPress ?

Gone through your posts and some videos. Now I understood. Thank you. I must research first before asking questions.


Generating open api document at startup

I am using NSwag.AspNetCore and NSwag.CodeGeneration.TypeScript to auto generate the Angular services for my WebApi.  However, the files are only generated when I navigate to swagger/v1/swagger.json.  However, I want the document to be compiled as soon as I run the app.  Is there any way to achieve this?
           services.AddSwaggerDocument(settings =>
            {
                settings.PostProcess = openApiDocument =>
                {
                    var clientGeneratorSettings = new TypeScriptClientGeneratorSettings
                    {
                        Template = TypeScriptTemplate.Angular,
                        PromiseType = PromiseType.Promise,
                        ClassName = "{controller}Client",
                        InjectionTokenType = InjectionTokenType.InjectionToken,
                        RxJsVersion = 6.5m,
                        BaseUrlTokenName = "BASE_URL"
                    };

                    var generator = new TypeScriptClientGenerator(openApiDocument, clientGeneratorSettings);
                    var code = generator.GenerateFile();
                    var executingAssembly = Assembly.GetExecutingAssembly();
                    var uri = new UriBuilder(executingAssembly.CodeBase);
                    var path = Uri.UnescapeDataString(uri.Path);
                    var directory = Path.GetDirectoryName(path);
                    File.WriteAllText($"{directory}/../../../ClientApp/src/app/services.nswag.ts", code);
                };
            });

What i do is generating the ts at build time:
https://github.com/RicoSuter/NSwag/wiki/NSwag.MSBuild

Thanks Rico,
I have tried that now.  Unfortunately on my startup there are some data seeding operations.  How do I not run these?  I tried setting the configuration in the variables parameter as NSwag and added a compilation symbol for NSWAG for the NSwag congiguration.  However the build doesn't seem to respect the configuartion setting.  Any ideas how I can not run a particular section of code during the nswag build?
  <Target Name="NSwag" BeforeTargets="PrepareForBuild" Condition="'$(GenerateCode)'=='True' ">
    <Exec Command="$(NSwagExe_Core30) run nswag.json /variables:Configuration=NSwag" />
  </Target>

Then in Startup.cs ConfigureServices method:
#if NSWAG
            SeedData();
#endif


How to get out of mediaCount InstagramTag?

Good day. Please tell me how to get out of mediaCount InstagramTag?

Sorry, your question is unclear.


Composer container image doesn't show exceptions in logs when Trainer crashes

** Environment **
Collecting system information...
System Environment Report
Created: 2023-05-24 19:24:21 UTC
PyTorch information
PyTorch version: 1.13.1+cu117
Is debug build: False
CUDA used to build PyTorch: 11.7
ROCM used to build PyTorch: N/A
OS: Ubuntu 22.04.2 LTS (x86_64)
GCC version: (Ubuntu 11.3.0-1ubuntu1~22.04.1) 11.3.0
Clang version: Could not collect
CMake version: version 3.26.3
Libc version: glibc-2.35
Python version: 3.10.6 (main, Mar 10 2023, 10:55:28) [GCC 11.3.0] (64-bit runtime)
Python platform: Linux-5.19.0-1025-aws-x86_64-with-glibc2.35
Is CUDA available: True
CUDA runtime version: Could not collect
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration:
GPU 0: NVIDIA A10G
GPU 1: NVIDIA A10G
GPU 2: NVIDIA A10G
GPU 3: NVIDIA A10G
Nvidia driver version: 530.30.02
cuDNN version: Could not collect
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True
Versions of relevant libraries:
[pip3] numpy==1.24.3
[pip3] pytorch-ranger==0.1.1
[pip3] torch==1.13.1
[pip3] torch-optimizer==0.3.0
[pip3] torchmetrics==0.11.3
[pip3] torchtext==0.14.1
[pip3] torchvision==0.14.1
[conda] Could not collect
Composer information
Composer version: 0.14.1
Composer commit hash: None
Host processor model name: AMD EPYC 7R32
Host processor core count: 48
Number of nodes: 1
Accelerator model name: NVIDIA A10G
Accelerators per node: 1
CUDA Device Count: 4
** To reproduce
Steps to reproduce the behavior:

make a dataset for fine tuning mpt-7b-instruct
build trainer docker image based on mosaicml/composer:latest, adding necessary prerequisites
docker run --runtime-nvidia -it --rm composer composer train.py mymodel.yaml
Note that datasets load, and it starts Building trainer... and then it exits with

Expected behavior
Fine tuning should start, or at least, when it fails, a reasonable exception / stack trace / error message should be printed.
Additional context
Dockerfile:
FROM mosaicml/composer
RUN apt-get update && apt install -y vim git
RUN pip3.10 install --upgrade pip
RUN pip3.10 install composer omegaconf
WORKDIR /mpt/
RUN git clone https://github.com/mosaicml/llm-foundry
WORKDIR /mpt/llm-foundry
RUN pip3.10 install -e ".[gpu]"
WORKDIR /mpt/llm-foundry/scripts/train

start script:
docker run --runtime=nvidia --rm -it --name train -v /opt/mpt-7b:/mpt -v /opt/mpt-7b/cache:/root/.cache train composer train.py observe-help.yaml

log:
Found cached dataset json (/root/.cache/huggingface/datasets/json/default-ef4f1a35606f162a/0.0.0/0f7e3662623656454fcd2b650f34e886a7db4b9104504885bd462096cc7a9f51)
Building trainer...
ERROR:composer.cli.launcher:Rank 3 crashed with exit code -7.
Waiting up to 30 seconds for all training processes to terminate. Press Ctrl-C to exit immediately.

stdout:
Global rank 3 (PID 104) exited with code -7
----------Begin global rank 3 STDOUT----------
Initializing model...
cfg.n_params=6.65e+09
Building train loader...
Importing preprocessing function via: `from preprocess_observe_help import o11y_help`
Building eval loader...
Importing preprocessing function via: `from preprocess_observe_help import o11y_help`
Building trainer...

----------End global rank 3 STDOUT----------

stderr:
----------Begin global rank 3 STDERR----------
Explicitly passing a `revision` is encouraged when loading a configuration with custom code to ensure no malicious code has been contributed in a newer revision.
Explicitly passing a `revision` is encouraged when loading a model with custom code to ensure no malicious code has been contributed in a newer revision.
/root/.cache/huggingface/modules/transformers_modules/mosaicml/mpt-7b-instruct/a858cfabdc6bf69c03ce63236a5e877517bb957c/attention.py:153: UserWarning: While `attn_impl: triton` can be faster than `attn_impl: flash` it uses more memory. When training larger models this can trigger alloc retries which hurts performance. If encountered, we recommend using `attn_impl: flash` if your model does not use `alibi` or `prefix_lm`.
  warnings.warn('While `attn_impl: triton` can be faster than `attn_impl: flash` ' + 'it uses more memory. When training larger models this can trigger ' + 'alloc retries which hurts performance. If encountered, we recommend ' + 'using `attn_impl: flash` if your model does not use `alibi` or `prefix_lm`.')

Loading checkpoint shards:   0%|          | 0/2 [00:00<?, ?it/s]
Loading checkpoint shards:  50%|█████     | 1/2 [00:06<00:06,  6.01s/it]
Loading checkpoint shards: 100%|██████████| 2/2 [00:08<00:00,  3.70s/it]
Loading checkpoint shards: 100%|██████████| 2/2 [00:08<00:00,  4.05s/it]
Using pad_token, but it is not set yet.
Found cached dataset json (/root/.cache/huggingface/datasets/json/default-ef4f1a35606f162a/0.0.0/0f7e3662623656454fcd2b650f34e886a7db4b9104504885bd462096cc7a9f51)

Map:   0%|          | 0/2610 [00:00<?, ? examples/s]
Map:   4%|▍         | 111/2610 [00:00<00:02, 1088.71 examples/s]
Map:   9%|▊         | 224/2610 [00:00<00:02, 1109.61 examples/s]
Map:  14%|█▍        | 377/2610 [00:00<00:02, 1053.76 examples/s]
Map:  20%|██        | 527/2610 [00:00<00:02, 1023.82 examples/s]
Map:  24%|██▍       | 635/2610 [00:00<00:01, 1032.76 examples/s]
Map:  29%|██▊       | 745/2610 [00:00<00:01, 1048.90 examples/s]
Map:  33%|███▎      | 862/2610 [00:00<00:01, 1083.13 examples/s]
Map:  38%|███▊      | 1000/2610 [00:01<00:01, 919.49 examples/s]
Map:  42%|████▏     | 1103/2610 [00:01<00:01, 942.95 examples/s]
Map:  47%|████▋     | 1214/2610 [00:01<00:01, 985.67 examples/s]
Map:  52%|█████▏    | 1362/2610 [00:01<00:01, 984.68 examples/s]
Map:  56%|█████▌    | 1464/2610 [00:01<00:01, 988.78 examples/s]
Map:  60%|██████    | 1571/2610 [00:01<00:01, 1007.56 examples/s]
Map:  64%|██████▍   | 1676/2610 [00:01<00:00, 1014.13 examples/s]
Map:  68%|██████▊   | 1781/2610 [00:01<00:00, 1021.52 examples/s]
Map:  72%|███████▏  | 1886/2610 [00:01<00:00, 1027.09 examples/s]
Map:  76%|███████▌  | 1990/2610 [00:01<00:00, 1026.62 examples/s]
Map:  80%|████████  | 2096/2610 [00:02<00:00, 866.59 examples/s]
Map:  85%|████████▌ | 2229/2610 [00:02<00:00, 869.06 examples/s]
Map:  91%|█████████ | 2376/2610 [00:02<00:00, 901.29 examples/s]
Map:  95%|█████████▍| 2475/2610 [00:02<00:00, 920.22 examples/s]
Map:  98%|█████████▊| 2570/2610 [00:02<00:00, 925.10 examples/s]

Found cached dataset json (/root/.cache/huggingface/datasets/json/default-ef4f1a35606f162a/0.0.0/0f7e3662623656454fcd2b650f34e886a7db4b9104504885bd462096cc7a9f51)

Map:   0%|          | 0/315 [00:00<?, ? examples/s]
Map:  35%|███▍      | 110/315 [00:00<00:00, 1084.28 examples/s]
Map:  85%|████████▍ | 267/315 [00:00<00:00, 1055.64 examples/s]

----------End global rank 3 STDERR----------

What is exit code -7? Why did the process die inside Trainer()? Who knows!
(The stdout/stderr printouts contain nothing better)

The bug here is "when trainer crashes, there are no logs printed."
If I don't use the container image, but instead install all dependencies locally, and start composer, it does print a helpful error message that it's running out of RAM on the 20 GB available on A10G.
I will edit the topic to clarify.
So, when the spawned processes crash for some reason, when running in docker, the composer subprocess spawner seems to not be able to harvest the exception from standard error of the subprocesses. Maybe it dies too quickly before it gets flushed out or something.

Hi @jwatte thanks for reporting this! Just to clarify, you're expecting a CUDA OOM error to be printed to the screen, correct?
We don't do anything in the Docker image that would inhibit printing of exceptions. It does look like you use the composer launcher which waits 30s for child processes to clean up, which I would think is long enough for any buffered prints to flush.
As a quick experiment, could you try setting the following env var and try to reproduce?
export PYTHONUNBUFFERED=True
For more information:
https://docs.python.org/3/using/cmdline.html#envvar-PYTHONUNBUFFERED

Have the same issue.
Output looks exactly like mine. But I don't get an OOM like bandish-shah suggests.
I set 'export PYTHONUNBUFFERED=True' and don't receive any new information for this error:
I start the training process using this image (mosaicml/pytorch:1.13.1_cu117-python3.10-ubuntu20.04) and the provided setup.py. And start it with composer: composer train.py yamls/finetune/mpt-7b_dolly_sft.yaml

Closing as this is stale and I spent some time but I'm unable to reproduce :(
If this is still an issue, please feel free to reopen! I am happy to investigate further.


why change in the default UncaughtExceptionHandler is not allowed and failes the tests?

I'm upgrading a legacy app that uses both Elasticsearch test framework (MockNode and cluster from the org.elasticsearch.test:framework:6.5.3) and EmbeddedKafka/Zookeeper frameworks. I'm facing an issue with upgrading from zoookeeper version 3.4 to version 3.5, because in v3.5 Zookeeper added the following static code to do more logging in it's NIOServerCnxnFactory:
static {
        Thread.setDefaultUncaughtExceptionHandler(new Thread.UncaughtExceptionHandler() {
                public void uncaughtException(Thread t, Throwable e) {
                    LOG.error("Thread " + t + " died", e);
                }
            });

At the same time it looks like Elasticsearch requires the use of the RandomizedRunner in order to use it in Unit tests. So, I have to run all unit tests as following:
@RunWith(RandomizedRunner.class)
@ThreadLeakScope(ThreadLeakScope.Scope.NONE)
class ElasticsearchClusterSelectorServiceSpec { ... }

However, the RandomizedRunner does not like when a default UncaughtExceptionHandler is changed! So, when I run the unit tests that run both ES and ZK now I get the following exception:
java.lang.RuntimeException: Suite replaced Thread.defaultUncaughtExceptionHandler. It's better not to touch it. Or at least revert it to what it was before. Current: class org.apache.zookeeper.server.NIOServerCnxnFactory$1
I'm assuming it is due to this code in the RandomizedRunner:
UncaughtExceptionHandler current = Thread.getDefaultUncaughtExceptionHandler();
    if (current != handler) {
      notifier.fireTestFailure(new Failure(suiteDescription,
          new RuntimeException("Suite replaced Thread.defaultUncaughtExceptionHandler. " +
          		"It's better not to touch it. Or at least revert it to what it was before. Current: " +
              (current == null ? "(null)" : current.getClass()))));
    }

So, my questions:

why is changing UncaughtExceptionHandler is causing the tests to fail? Could it at least be made configurable to fail or not to fail?
is there any other option to work around this issue? (I tried starting ZK in a separate thread but that does not help as the default handler is set for ALL threads ....)

thank you!
Marina

I replied to this already in gh-285.


Links defaulting to localhost

Hi ...
I'm trying to get the docker image setup and everything appears to work, but several default links in the base site seem to be hard coded to localhost. For example from the /ghost/ admin page, when I click on View Site, the iframe tries to load localhost:2368 instead of my actual IP or Hostname, which results in a localhost refused connection error.
Where can I change this so the default site/server uses my actual IP or Hostname?
Thanks.
OS:  Ubuntu 20.04
DOCKER:  Latest
Ghost:  Latest

@un-fuf-a-doo Ghost by default uses localhost:2368 as the URL. If you wish to change that, you'll need to pass in an environment variable called url, with the value that you wish to use.
For example, if you wanted your URL to be https://example.com, you'd start your docker container with -e url=https://example.com.
For additional configuration options please see the docs 😄

@un-fuf-a-doo Ghost by default uses localhost:2368 as the URL. If you wish to change that, you'll need to pass in an environment variable called url, with the value that you wish to use.
For example, if you wanted your URL to be https://example.com, you'd start your docker container with -e url=https://example.com.
For additional configuration options please see the docs 😄

Awesome.  That is what I was looking for.
Thank-you!

Awesome.  That is what I was looking for.
Thank-you!


Support for allcaster

Checklist

[X] I'm reporting a new site support request
[X] I've verified that I'm running youtube-dl version 2019.07.27
[X] I've checked that all provided URLs are alive and playable in a browser
[ ] I've checked that none of provided URLs violate any copyrights
[ ] I've searched the bugtracker for similar site support requests including closed ones

Example URLs

-Single video: https://allcaster.pl/serial-online/halt-and-catch-fire/S01E03
Description

Here's the output from the terminal:
youtube-dl -v 'https://allcaster.pl/serial-online/halt-and-catch-fire/S01E03'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://allcaster.pl/serial-online/halt-and-catch-fire/S01E03']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.07.27
[debug] Python version 3.7.3 (CPython) - Linux-4.19.61-1-MANJARO-x86_64-with-arch-Manjaro-Linux
[debug] exe versions: ffmpeg 4.1.4, ffprobe 4.1.4, rtmpdump 2.4
[debug] Proxy map: {}
[generic] S01E03: Requesting header
WARNING: Falling back on generic information extractor.
[generic] S01E03: Downloading webpage
[generic] S01E03: Extracting information
ERROR: Unsupported URL: https://allcaster.pl/serial-online/halt-and-catch-fire/S01E03
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 796, in extract_info
ie_result = ie.extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 530, in extract
ie_result = self._real_extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 3333, in _real_extract
raise UnsupportedError(url)
youtube_dl.utils.UnsupportedError: Unsupported URL: https://allcaster.pl/serial-online/halt-and-catch-fire/S01E03

youtube-dl does not support sites dedicated to copyright infringement.

Thx for so fast response but also sorry for taking your time.
Best regards
niedz., 28 lip 2019 o 23:02 Sergey M<EMAIL_ADDRESS>napisał(a):

youtube-dl does not support sites dedicated to copyright infringement
https://github.com/rg3/youtube-dl#can-you-add-support-for-this-anime-video-site-or-site-which-shows-current-movies-for-free
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ytdl-org/youtube-dl/issues/21921?email_source=notifications&email_token=AGFCCHORYYSG22A7YAM5EL3QBYCPXA5CNFSM4IHNNCOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD27GPRA#issuecomment-515794884,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGFCCHO3XCJCEQMJ2J3LWNLQBYCPXANCNFSM4IHNNCOA
.


Create a proper readme file for awesome Face operations

https://github.com/akshitagupta15june/Face-X/tree/master/Awesome-face-operations
Add the feature information which is implemented
This readme must contain the information regarding all the facial operations can be added here like Face swapping, Face sketch, Face ghost image etc

Can you please assign this to me?

assigning


Error using auth with emulator

Hello, trying to signup user using js sdk and nhost emulator but getting an error.
import { NhostClient } from '@nhost/nhost-js'

nhost = new NhostClient({
  backendUrl: 'http://localhost:1337'
})

await nhost.auth.signUp({
    email: payload.email,
    password: payload.password,
    options: {
      displayName: payload.name,
      allowedRoles: ['user'],
      defaultRole: 'user'
    }
  })

utils.js:574 Missing onError handler and/or unhandled exception/promise rejection for invocation 'signUp'. Original error: 'TypeError: adapter is not a function'.  Stacktrace was 'TypeError: adapter is not a function
    at dispatchRequest (chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:3416:10)
    at Axios.request (chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:3103:15)
    at Axios.httpMethod [as post] (chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:3138:19)
    at Function.wrap [as post] (chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:3832:15)
    at postRequest (chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:798:30)
    at signUp (chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:1326:16)
    at Interpreter.exec (chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:12468:98)
    at Interpreter.execute (chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:11805:14)
    at Interpreter.update (chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:11831:12)
    at chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:11711:15' Current error is 'Error: Unable to evaluate guard 'unverified' in transition for event 'error.platform.signUp' in state node 'nhost.authentication.registering':
Cannot read properties of undefined (reading 'status')'. Stacktrace was 'Error: Unable to evaluate guard 'unverified' in transition for event 'error.platform.signUp' in state node 'nhost.authentication.registering':
Cannot read properties of undefined (reading 'status')
    at StateNode.next (chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:9538:17)
    at StateNode.transitionLeafNode (chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:9405:26)
    at StateNode._transition (chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:9498:19)
    at StateNode.transitionParallelNode (chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:9443:33)
    at StateNode._transition (chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:9507:17)
    at StateNode.transition (chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:9769:32)
    at chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:12325:28
    at provide (chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:13340:16)
    at Interpreter.nextState (chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:12324:78)
    at chrome-extension://eglmfhjigjnbbfibaaallaoddlloinnc/background.js:11709:31'

Nhost version
nhost version
ℹ Nhost CLI v0.6.10 for darwin-amd64
✔ You have the latest version. Hurray!

JS sdk
"@nhost/nhost-js": "^1.1.12",

What's the "emulator" you're referring to?

Sorry, I'm referring to nhost dev

So, I think I found the reason. The issue appears because SDK internally uses axios and axios with default config won't work at the service worker because xhr as I understood isn't supported. And it needs to use another adapter e.g. https://www.npmjs.com/package/@vespaiach/axios-fetch-adapter (fetch is supported). So, it would be good to make it possible to pass more config options to axios.create...


Add a period to the end of a sentence in the documentation

Now when this line is joined with the next line (as it will be on
rubydoc.info), it won't turn into a run-on sentence.
See:

Thanks!


revert testing to using socket

switch back to using a socket instead of an output file for use in the plugin communication during testing. This should work now that we resolved the issue with python path for windows.

@eleanorjboyd did the os.environ.copy() fix the problem?


upstream updates to chips-core from @barrystyle

These are all the updates fetched from @barrystyle's repo https://github.com/barrystyle/chips for upgrading chips-core to latest bitcoin-core 0.21+ (since master branch has more updates than version 0.21 branch) version.
The relevant github issue for these updates is here: https://github.com/chips-blockchain/chips/issues/10
I have tested these updates by compiling and it syncs to the network as expected.

[ ] Task on taproot activation block height is yet to be sorted.

@who-biz @phm87 need your help reviewing this code, before merging to develop branch.

To start with I'm doing functional testing on mainnet and testnet, will post my findings here.

The following was performed on chips testnet.  I am posting below for the purpose of documentation:
Identified Issues
1. getinfo command in chips-cli does not work, if a wallet hasn't been loaded
./chips-cli -testnet getinfo
error code: -18
error message:
No wallet is loaded. Load a wallet using loadwallet or create a new one with createwallet. (Note: A default wallet is no longer automatically created)

It might make things smoother for dPoW if we returned some information about the blockchain, in any case. If a wallet hasn't been loaded, we can just use placeholder values for balance, wallet version, etc.
2. When a wallet has been loaded, getinfo does not reflect balance properly
Comparing getinfo to getwalletinfo...
Output of getinfo:
chips-cli -testnet getinfo
{
  "version": 219900,
  "protocolversion": 70016,
  "walletversion": 169900,
  "balance": 0.00000000,
  "blocks": 6175,
  "timeoffset": 0,
  "connections": 3,
  "proxy": "",
  "notarizedhash": "0000000000000000000000000000000000000000000000000000000000000000",
  "notarizedtxid": "0000000000000000000000000000000000000000000000000000000000000000",
  "notarized": 0,
  "prevMoMheight": 0,
  "notarized_MoMdepth": 0,
  "notarized_MoM": "0000000000000000000000000000000000000000000000000000000000000000",
  "difficulty": 0.01013083595512288,
  "testnet": true,
  "keypoololdest":<PHONE_NUMBER>,
  "keypoolsize": 1999,
  "paytxfee": 0.00000000,
  "relayfee": 0.00001000,
  "errors": "This is a pre-release test build - use at your own risk - do not use for mining or merchant applications"
}

Output of getwalletinfo:
chips-cli -testnet getwalletinfo
{
  "walletname": "test",
  "walletversion": 169900,
  "format": "bdb",
  "balance": 9199.99936000,
  "unconfirmed_balance": 0.00000000,
  "immature_balance":<PHONE_NUMBER>0,
  "txcount": 187,
  "keypoololdest":<PHONE_NUMBER>,
  "keypoolsize": 999,
  "hdseedid": "47f833b54ef50bc43bfbd938d119b10ccd18bf57",
  "keypoolsize_hd_internal": 1000,
  "paytxfee": 0.00000000,
  "private_keys_enabled": true,
  "avoid_reuse": false,
  "scanning": false,
  "descriptors": false
}

3. Transactions are not being cleared from mempool when mining
SHossain found this issue, and I was able to reproduce.  From appearance of logs, transactions are not being included in CreateNewBlock():
CreateNewBlock(): block weight: 860 txs: 0 fees: 0 sigops 400

Blocks being mined, despite mempool state:
chips-cli -testnet getrawmempool
[
  "0c74da539e6e61d00545d8b5634f88098006ba22d0951ccc18d06f5fd0f35aae",
  "f62da2a868405419d31c2e1bc1d91dfc206d1bc9bfc270d1e47b12bfba14ed87",
  "e12e05faeef0834e69b696fb47118a2714fad81045a5e443bb23a5967f15ad69",
  "3ef80d243597a53667c00bc428b3ed65cc9b107377500765206b89a77cddfbcc"
]

Please note that item 1 is no longer an issue, due to moving getinfo into wallet portion of RPC.
item 2 has been resolved and has a PR here: https://github.com/satindergrewal/bitcoin/pull/3.
Will begin working on mempool problems next.


feat(icons): added 4 icons of 3 different sizes for trial

Proposed Changes
Added 4 new icons with 3 different sizes in the project
Potential Breaking Changes

Acceptance Criteria

[ ] The proposed changes are covered by unit tests
[ ] The potential breaking changes are clearly identified
[ ] README.md is adjusted to reflect the proposed changes (if relevant)

I will close this branch. More work needed to be done and I got the message that we want to make all the new images available at the same time. Thanks for the reviews. Coveolove


how to create file?

I need to upload selected images and videos to my server.how can i do this?how to create file from selected items?

cordova-plugin-file-transfer
use  dataArray[index].path


os.Stat() return error for socket file in 20H2 Windows image and does not work for symlink points to a dir (in all Windows version)

While testing the new 20H2 windows image, we discovered that golang function os.Stat() always return error when checking a socket file. See the error message
os.Stat error CreateFile C:\var\lib\kubelet\plugins_registry\pd.csi.storage.gke.io-reg.sock: The file cannot be accessed by the system

Another discovery is os.Stat() does not work (no matter which version of Windows) on symlink that points to a dir.
The error is "Access is denied"

@immuzz @vrapolinario Any updates here?
It seems like this is a new regression and Windows Server 2019 and/or 1909 are not impacted.

Starting internal investigation on this.

os.Stat() seems ok for symlink on ltsc 2019 I just tested. (will confirm 20H2 later).
But for Windows container, if it checks symlink point to a different disk partition, it will not work. I put details in this doc

Quick update that we're still investigating this issue internally.

For internal MS folks: Bug filled: Bug 32905775

Sorry for the delay on this. Here's the result from the internal investigation:
The os.Stat() implementation never should have worked.  Windows had a short-term regression during which the os.Stat() implementation worked OK with socket files on some versions (Server 2019), accidentally.  Current versions (20H2 and forward) of Windows function correctly.  We strongly recommend that os.Stat() should handle the error case STATUS_IO_REPARSE_TAG_NOT_HANDLED and respond by opening the file with the FILE_OPEN_REPARSE_POINT flag specified.
            Which is approximately what this Go patch implements: os: fallback to open symlinks (Ib919e5ce) · Gerrit Code Review (googlesource.com): https://go-review.googlesource.com/c/go/+/265037
            That patch retries with FILE_OPEN_REPARSE_POINT in case of any error aside from ERROR_CANT_ACCESS_FILE – which is overly inclusive, but should solve the problem.


Add Travis test

Test helm chart in Travis, see for example https://github.com/jupyterhub/zero-to-jupyterhub-k8s/tree/0.8.2/ci

https://github.com/manics/kubernetes-omero/pull/3 added a test of the deployment, but it's missing basic tests to check OMERO.server and OMERO.web are functional


Support Swift 3.0 and Xcode 8.0 (8S128d)

I only converted syntax to Swift 3.0, uncertain whether there will be a new bug,
but it seems work well on my own project.

Thank you for the PR. I had a swift 3 branch here:
https://github.com/daltoniam/Starscream/tree/swift3
I merged those changes with the xCode 8 converter changes as well (will bring some breaking changes). Thanks again!


New command: list the list templates in a specified site

Usage
m365 spo web list template list [options]
Description
List the list templates in a specified site
Options

Option
Description

-u, --url <url>
URL of the site for which to retrieve the list templates

Examples
List all list templates in the specified site
m365 spo web list template list --url "https://contoso.sharepoint.com/sites/contoso"

Default properties

Name
ListTemplateTypeKind
Description

Additional Info
Can be retrieved by doing a GET to /_api/web/ListTemplates/

Another nice suggestion @nicodecleyre! I've got a few pointers here that are quite similar to #4616.

In the command name, let's combine the two words list and template. This will result in the following m365 spo web listtemplate list [options].
We should change the option name for url to webUrl as URL reflects on web and not on listtemplate.

Also additional comment from my side. As in the response, ListTemplateTypeKind is an int that does not give any human info let's extend the output of the command with ListTemplateTypeKindValue to which we may add the 'translated' value from this enum
https://github.com/pnp/cli-microsoft365/blob/main/src/m365/spo/commands/list/ListTemplateType.ts

Hi @nicodecleyre, I'm not sure I see the added value of this command. Checkout my comment here as well.

@martinlingstuyl also here I agree also with same comment 😜

Closing as there is no added value as far as we can see.


Fix building themes

Building themes failed when npm / gulp was not installed on local machine.
Even we download nodejs + npm how ever when using &&  in the command it  uses the local machine instead of docker image

Improved readme regarding docker image. Probally requires some spell check / rephrasing ❤

Closes: https://github.com/FOSSBilling/FOSSBilling/issues/256


Updating readthedocs

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.40%. Comparing base (f62c290) to head (43ed51d).
Report is 3 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master      #52   +/-   ##
=======================================
  Coverage   88.40%   88.40%
=======================================
  Files          14       14
  Lines        3934     3934
=======================================
  Hits         3478     3478
  Misses        456      456

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


Apply query splitting for stream query per segment

Cause it's possible, not like group-by query

Slower..


Issue/perf2usfm

one line of code to rule them all
All tests passes for the "x-morph" string type.

LGTM


move COPYING file into readme.md file

Can we remove the COPYING into ~ or put them into every readme.md file to get rid of it?
Now that we have one single galaxyp repository this would make much sense, imho.
@jmchilton ok with you? Your are probably the copyright holder?

As I understand it the existing tools have been developed under public funding and are therefore in the public domain. The copyright holders are listed in each separate tool in its respective README.
The licenses are assigned on a per-tool basis, instead of under a formally established project-wide charter.
This allows for the GalaxyP project to incorporate other tool contributions that may have more restrictive terms. The hope is that this provides the most flexibility and allows for the greatest developer freedom.
Feel free to bring this up on the mailing list if you'd like to discuss it further so more group members may participate. https://groups.google.com/a/umn.edu/forum/#!categories/galaxyp

Ok, this is fine. Just thought about getting rid of the COPYING file :)


Bug in code generator for BottomSheet style

My code:
@Destination(style = DestinationStyleBottomSheet::class)
@Composable
fun BottomSheetScreen() {

The code is correct, but I get this error:
[ksp] com.ramcosta.composedestinations.codegen.commons.IllegalDestinationsSetup: Unknown style used on TextSettingsBottomSheetScreen. Please recheck it.

The problem is in KspToCodeGenDestinationsMapper on line 498:
resolver.getClassDeclarationByName("$CORE_PACKAGE_NAME.spec.DestinationStyleBottomSheet")?.asType(emptyList())

It must replaced with:
resolver.getClassDeclarationByName("$CORE_PACKAGE_NAME.spec.DestinationStyle.BottomSheet")?.asType(emptyList())

Unfortunately I can't create MR with the fix now. Could you fix it ASAP please?

What version are you using?

The latest version - 1.6.38-beta

Something is not right here 🤔
in your example, you’re using DestinationStyleBottomSheet (no dot), and you’re saying I should replace DestinationStyleBottomSheet by DestinationStyle.BottomSheet (with dot).
That doesn’t make sense right? If you’re using without dot then for it to be recognised, also no dot would need to be used.
however if you’re really using 1.6.38-beta, then this version still uses the nested style (I.e with dot) and the ksp mapper class is correct.
So it does seem like everything is correct here. For that version you need to use the nested style (with dot).