issue after updating from angular 14 to 15

Bug reports must include:

Operating system and version - Windows 11
Browser and version Chrome

issue
After upgrading to angular 15 from angular 14, getting below error.
./src/content/scss/vendor.scss?ngGlobalStyle:1:0 - Error: Module parse failed: Unexpected character '@' (1:0)
File was processed with these loaders:

./node_modules/resolve-url-loader/index.js
./node_modules/sass-loader/dist/cjs.js
You may need an additional loader to handle the result of these loaders.

@charset "UTF-8";
| /* after changing this file run 'npm run webpack:build' /
| /**************************

tried adding
Tried adding
  "stylePreprocessorOptions": {
          "includePaths": [
            "src", "node_modules"
          ]
        }

in angular.json
also below are package.json dependencies
"@angular/animations": "^15.2.10",
"@angular/cdk": "^15.2.9",
"@angular/common": "^15.2.10",
"@angular/compiler": "^15.2.10",
"@angular/core": "^15.2.10",
"@angular/forms": "^15.2.10",
"@angular/localize": "^15.2.10",
"@angular/platform-browser": "^15.2.10",
"@angular/platform-browser-dynamic": "^15.2.10",
"@angular/router": "^15.2.10",
"@coreui/angular": "~4.4.9",
"@coreui/angular-chartjs": "~4.4.7",
"@coreui/chartjs": "^3.1.2",
"@coreui/coreui": "~4.2.6",
"@coreui/icons": "~3.0.1",
"@coreui/icons-angular": "~4.4.7",

@smarsh-tejaskatariya this error comes from sass-loader

what about your devDependencies ?
what's the file containing "@charset "UTF-8";  - ./src/content/scss/vendor.scss ?
see also:

stylePreprocessorOptions
Angular Update Guide
CoreUI Angular Versions

Dev dependencies
"devDependencies": {
"@angular-builders/jest": "^17.0.2",
"@angular-devkit/build-angular": "^15.2.11",
"@angular/cli": "^15.2.11",
"@angular/compiler-cli": "^15.2.10",
"@babel/core": "^7.24.4",
"@babel/preset-env": "^7.24.4",
"@babel/preset-typescript": "^7.24.1",
"@types/chai": "4.3.14",
"@types/chai-string": "1.4.5",
<EMAIL_ADDRESS>"~1.10.24",
"@types/jest": "^29.5.12",
"@types/jquery": "^3.5.29",
"@types/node": "^20.12.5",
"@types/selenium-webdriver": "4.1.22",
"autoprefixer": "^10.4.19",
"babel-jest": "^29.7.0",
"chai": "5.1.0",
"chai-as-promised": "7.1.1",
"chai-string": "1.5.0",
"css-loader": "^7.1.0",
"husky": "9.0.11",
"jest": "^29.7.0",
"jest-canvas-mock": "^2.5.2",
"jest-junit": "16.0.0",
"jest-preset-angular": "^14.0.3",
"jest-sonar-reporter": "2.0.0",
"lint-staged": "^15.2.2",
"ngx-infinite-scroll": "^14.0.1",
"prettier": "3.2.5",
"protractor": "^7.0.0",
"redux-thunk": "^2.4.1",
"rimraf": "5.0.5",
"stubby": "^5.1.0",
"ts-node": "^10.9.2",
"tslint": "^6.1.3",
"tslint-config-prettier": "1.18.0",
"typescript": "4.9.5",
"util": "^0.12.5",
"xml2js": "^0.6.2"
},
vendor.scss
@import 'custom';
@import "@coreui/coreui/scss/coreui";
@import "@coreui/chartjs/scss/coreui-chartjs";
@import 'e_variables';


customize segment separator of agnoster theme

change SEGMENT_SEPARATOR of agnoster theme to a better choice

Can you post a screenshot of before and after? This looks to me like the typical issue with non-updated Powerline fonts.

So this is the terminal before:

This is the terminal after:

Yes, this looks like you have an old, incompatible Powerline font installed. The current code is correct, and you probably need to update your font. See #4091, #4083.
Have a look at the comments above that character definition. This change is exactly what it is asking people not to do.

@apjanke Thanks for you comments. You are correct. I will take a look at the comments


merging index page changes

Switching over to use the new SEO optimized index page.

Docs Build status updates of commit c3ef7d5:
:warning: Validation status: warnings

File
Status
Preview URL
Details

Odata-docs/client/v6/getting-started.md
:warning:Warning
View
Details

Odata-docs/index.yml
:white_check_mark:Succeeded
View

Odata-docs/client/v6/getting-started.md

Line 15: [Warning-multiple-h1s] Multiple H1s are not allowed. You can only have one top-level heading.
Line 141: [Warning-InvalidFileLink] Invalid file link:(~/client/v6/basic-crud-operations).

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Add support for specific branches with playbookRepos

Fixes #858

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Robusta Runner seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


No module named 'HTTPLocust'

[2019-07-26 11:06:51,269] VAM008056/ERROR/stderr: Traceback (most recent call last):
[2019-07-26 11:06:51,269] VAM008056/ERROR/stderr: File "/usr/local/bin/locust", line 10, in
[2019-07-26 11:06:51,269] VAM008056/ERROR/stderr:
[2019-07-26 11:06:51,269] VAM008056/ERROR/stderr: sys.exit(main())
[2019-07-26 11:06:51,269] VAM008056/ERROR/stderr:
[2019-07-26 11:06:51,269] VAM008056/ERROR/stderr: File "/usr/local/lib/python3.7/site-packages/locust/main.py", line 426, in main
[2019-07-26 11:06:51,271] VAM008056/ERROR/stderr:
[2019-07-26 11:06:51,271] VAM008056/ERROR/stderr: docstring, locusts = load_locustfile(locustfile)
[2019-07-26 11:06:51,271] VAM008056/ERROR/stderr:
[2019-07-26 11:06:51,271] VAM008056/ERROR/stderr: File "/usr/local/lib/python3.7/site-packages/locust/main.py", line 393, in load_locustfile
[2019-07-26 11:06:51,272] VAM008056/ERROR/stderr:
[2019-07-26 11:06:51,272] VAM008056/ERROR/stderr: imported = import_locustfile(locustfile, path)
[2019-07-26 11:06:51,272] VAM008056/ERROR/stderr:
[2019-07-26 11:06:51,272] VAM008056/ERROR/stderr: File "/usr/local/lib/python3.7/site-packages/locust/main.py", line 365, in import_locustfile
[2019-07-26 11:06:51,272] VAM008056/ERROR/stderr:
[2019-07-26 11:06:51,272] VAM008056/ERROR/stderr: imported = source.load_module()
[2019-07-26 11:06:51,272] VAM008056/ERROR/stderr:
[2019-07-26 11:06:51,272] VAM008056/ERROR/stderr: File "", line 407, in _check_name_wrapper
[2019-07-26 11:06:51,272] VAM008056/ERROR/stderr: File "", line 907, in load_module
[2019-07-26 11:06:51,272] VAM008056/ERROR/stderr: File "", line 732, in load_module
[2019-07-26 11:06:51,272] VAM008056/ERROR/stderr: File "", line 265, in _load_module_shim
[2019-07-26 11:06:51,272] VAM008056/ERROR/stderr: File "", line 696, in _load
[2019-07-26 11:06:51,273] VAM008056/ERROR/stderr: File "", line 677, in _load_unlocked
[2019-07-26 11:06:51,273] VAM008056/ERROR/stderr: File "", line 728, in exec_module
[2019-07-26 11:06:51,273] VAM008056/ERROR/stderr: File "", line 219, in _call_with_frames_removed
[2019-07-26 11:06:51,273] VAM008056/ERROR/stderr: File "/Users/yash.makkena/dev/nf2/core/locust/geosite_ips.py", line 1, in
[2019-07-26 11:06:51,273] VAM008056/ERROR/stderr:
[2019-07-26 11:06:51,273] VAM008056/ERROR/stderr: import HTTPLocust, TaskSet, task
[2019-07-26 11:06:51,273] VAM008056/ERROR/stderr:
[2019-07-26 11:06:51,273] VAM008056/ERROR/stderr: ModuleNotFoundError
[2019-07-26 11:06:51,273] VAM008056/ERROR/stderr: :
[2019-07-26 11:06:51,273] VAM008056/ERROR/stderr: No module named 'HTTPLocust'
[2019-07-26 11:06:51,273] VAM008056/ERROR/stderr:
I'm running Python 3.7.4 from Homebrew and Locust version 0.11.0. Any ideas on why this could be an issue? I've tried it with older versions of python, and I've tried reinstalling locust, but every time I try to run a locustfile I get this issue. Any ideas?

No module named 'HTTPLocust' occurs because there is no such module by that name.  It is 'HttpLocust`.

Yeah, I realized that soon after. It has the same issue with HttpLocust, though

are you importing from the locust package?
from locust import HttpLocust

Yeah I was. But I ended up fixing my issue, I had to delete all my Python 2.7 framework and it's running now. Thank you!


⚠ Bofrost.be has degraded performance

In 89558b3, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2338 ms

Resolved: Bofrost.be performance has improved in c6dc078.


Unable to create entity from test data file with more 50 related records

Describe the bug
when a test data file has more than 50 related entities getting the following error.
To reproduce
create data file which has more than 50 related entities
Expected behaviour
expected to create the entity
Screenshots

Hi @thathi. This looks to be a limitation of Dataverse's Web API.
@ewingjm are these separate requests? Maybe refactoring to execute multiple would be more efficient?

Yeah, I was looking at this recently @tdashworth. It should be possible to do a batch request as you can reference the IDs of records created in later creates. There should also be a way of parsing the response and relating these back to aliases. It's a substantial bit of work but it is possible.


Question about DrugBank prefixes

From @ericawood:

Do you have any thoughts on biolink mappings for these predicates:
DRUGBANK:aggregation_inhibitor
DRUGBANK:binding
DRUGBANK:chelator
DRUGBANK:degradation
DRUGBANK:deoxidizer
DRUGBANK:desensitize_the_target
DRUGBANK:downregulator
DRUGBANK:gene_replacement
DRUGBANK:incorporation_into_and_destabilization
DRUGBANK:inhibition_of_synthesis
DRUGBANK:intercalation
DRUGBANK:neutralizer
DRUGBANK:nucleotide_exchange_blocker
DRUGBANK:oxidizer
DRUGBANK:stablization

For DRUGBANK:binding, how about biolink:physically_interacts_with?

For DRUGBANK:downregulator, how about biolink:entity_negatively_regulates_entity?

For DRUGBANK:aggregation_inhibitor, how about biolink:decreases_activity_of?

For DRUGBANK:binding, how about biolink:physically_interacts_with?

For DRUGBANK:chelator, how about biolink:affects?

For DRUGBANK:degradation, how about biolink:affects_degradation_of?

For DRUGBANK:deoxidizer, how about biolink:decreases_molecular_modification_of?

For DRUGBANK:desensitize_the_target, how about biolink:decreases_response_to?

For DRUGBANK:gene_replacement, how about biolink:close_match?

For DRUGBANK:incorporation_into_and_destabilization, how about biolink:affects_degradation_of?

For DRUGBANK:inhibition_of_synthesis, how about biolink:entity_negatively_regulates_entity?

For DRUGBANK:intercalation, how about biolink:molecularly_interacts_with ?

I will work on the rest later. Am late to pick up the kids

Ok, sounds good. Thank you!

What about:

DRUGBANK:nucleotide_exchange_blocker -> biolink:decreases_activity_of
DRUGBANK:oxidizer -> biolink:increases_molecular_modification_of
DRUGBANK:stablization -> biolink:increases_stability_of

I'm still not sure about DRUGBANK:neutralizer. I didn't see anything about neutralization in the biolink model.

Looks good

This is incorporated into KG2 as of KG2.7.1. I'm going to close this issue.


Clean up test records from firebase

Background
BMMRO have been adding test / dummy / accidental records / records resulting from training to the firebase database.
This is causing a problem because they are appearing in the data exports, causing a lot of manual steps to remove them before import to MS Access
There are tickets in the system to allow BMMRO to keep the database clean going forward, and delete any records which have not yet been exported to Access.
Actions

[ ] Work with BMMRO to remove these records

Did this on screen share with Charlotte.
We went through how to remove records in Dev, and did this safely in the firebase console
Charlotte felt comfortable cleaning up the invalid records in Prod.
Suggested that UAT should be used for all training / testing going forward so this problem does not come up again


Add Gen V2.0.1 to MMGAT legacy method.

Download  Gen V2.0.1.json from MMGAT API & upload into legacy_mmgats folder under MRR, to load into Redis.

Can you please name the redis key with the same name as the profile that appears in the HL7 message? That would be like this:
mmg:generic_mmg_v2.0

Please leave the "." in there


Reenable bazel phase1 tests

Closed in favor of updating https://github.com/angular/angular-cli/pull/11749


Implement xComfort into Home Assistant

Hi,
Don't know if this is the right place to ask, but also don't know how to ask otherwise.
I used Google, but there are almost no results.
Is it possible to implement this into Home Assistant, and control xComfort actor from Home Assistant?
If so, how can this be done?
I already started my Home Assistant installation, along with some other automation devices (LimitlessLed and Z-wave). But now I have the possibility to get some xComfort Actors really cheap.
Can you help me with this, or should I just ignore the offer and continue using Z-wave and other supported devices?
Kind regards,
Erwin

I am running this as a separate daemon in conjunction with Home Assistant, which communicates with this via MQTT. It could probably be integrated into Home Assistant natively, but I haven't really seen any need to so far, my current setup works well.  There are no additional benefits to native support other than packaging.
However, I would be hesitant to purchase more xComfort devices (unless they were essentially free). They are completely proprietary and in my opinion are a dead end.

Thanks for your quick reply.
Actually, it is not for me, but for a friend of mine. For myself I have automated using mainly Z-wave, and he asked me to help with some easy automating at his home.
I started with a Raspberry Pi 3, Hassbian, Limitlessled bridge and some Limitlessled Led strip controllers.
It is working great.
Now I want to make it possible to dim some lights, and explained to him what kind of hardware was needed.
He said he could get some xComfort devices from his job as electrician.  If it is for free, I guess we could try and see how it works out.
Can you give me a little help about how to set this up? I have a Raspberry Pi, and I think I need a xComfort USB stick? There are 2 versions (13 &14)? What is the difference?
Can I just run your script along Home Assistant, and send commands to it? I don't know exactly how yet, but I think with a little help I could figure it out.
Thanks!


fix: typing issues of funix()

fix type annotation of funix() the decorator
add [tool.pyright] part of pyproject.toml

Closes: https://github.com/TexteaInc/funix/issues/88

Thank you very much for your kind assistance in helping us improve Funix. I will review your pull request after merging #90.


32/64 Build Notes.Win Question

We are porting our Win32 platform to Win64 (x64) and we are going to take the opportunity to also move towards the LTS OpenSSL v1.1.x version.
I can see the Notes.WIN information is still evolving.  So maybe the notes can be updated to assists folks with similar compilation scenarios.
We are going to need to build both 32 bit and 64 bit *.DLL/LIB files for the 64 bit RPC Server side and the 32 bit RPC client side of our platform.  Given you have a setup perl to do:
Perl Config VC-WIN64A
Perl Config VC-WIN32
It would appear we will need to do this in two separate folders?  Correct?

Yes.  However, you can have the two builds share the same source, by building outside of the source tree:
rem Assume the openss source is in subdirectory "openssl"

mkdir openssl-32
cd openssl-32
perl ..\openssl\Configure VC-WIN32
nmake
cd ..

mkdir openssl-64
cd openssl-64
perl ..\openssl\Configure VC-WIN64A
nmake
cd ..

Thanks you!  I will try this! :)   Good note to add for 32/64 bit build environments.

Levitte,
I did the 32/64 compile and began to recompile/edit some internal ssl tools,
I found the output for SSLeay_version(SSLEAY_PLATFORM) for a 32 bit compile was VC-WIN64A
At first I thought it was related source include folder.  I added two folders to my INCLUDE statement
 openssl-1_1_1-stable\include
 openssl-1_1_1-stable\x32\include

yet, it still output VC_WIN64A
But I found it was related to the library.   Here is my test program used the non-static libs.
``// File: C:\Files\openssl\openssl-1_1_1-stable\wctest\ListSSLversion.cpp
#include <stdio.h>
#include <windows.h>
#include <openssl/ssl.h>
#include <openssl/crypto.h>
#pragma comment(lib,"libssl.lib")
#pragma comment(lib,"libcrypto.lib")
//#pragma comment(lib,"libssl_static.lib")
//#pragma comment(lib,"libcrypto_static.lib")
void main(char argc, char argv[])
{
#ifdef _WIN64
printf("* 64 bit \n");
#else
printf(" 32 bit **\n");
#endif
if (SSLeay() != SSLEAY_VERSION_NUMBER) {
printf("! SSL DLL Version Mismatch\n");
printf("! Expected: %08X\n",SSLEAY_VERSION_NUMBER);
printf("! Loaded  : %08X\n",SSLeay());
}
printf("- SSLEAY_VERSION_NUMBER : %s\n",SSLeay_version(SSLEAY_VERSION_NUMBER));
printf("- SSLEAY_VERSION  : %s\n",SSLeay_version(SSLEAY_VERSION));
printf("- SSLEAY_BUILT_ON : %s\n",SSLeay_version(SSLEAY_BUILT_ON));
printf("- SSLEAY_PLATFORM : %s\n",SSLeay_version(SSLEAY_PLATFORM));
}
`
You will get VC-WIN64A when using the non-static library.   Switch the the static lib by changing the pragmas and it works to output VC-WIN32
Keep in mind and this was a clean 32 bit only build. I still got the VC-WIN64A,

Closing this in favor of #8556, which goes into the remaining details


Delete test/dummy/config/initializers/secret_token.rb

Merging to resolve the security issue raised.


Feat(eos_designs): add BGP prefix-list support for peer_groups

Enhancement summary
One can specify a BGP prefix-list for a discrete BGP neighbor IPv4/IPv6 address
From:
https://avd.sh/en/stable/roles/eos_designs/doc/network-services.html
bgp_peers:
  < IPv4_address or IPv6_address >:
    prefix_list_in: < prefix_list_name >
    prefix_list_out: < prefix_list_name >

But there is no such setting for a BGP peer-group, only route-maps are supported:
bgp_peer_groups:
  - name: < BGP peer group name >
    route_map_out: < route-map name >
    route_map_in: < route-map name >

As the prefix-list must be set under each address-family for the peer group this should be honored in the YAML data model.
From eos_cli_config_gen
router_bgp:
  address_family_ipv4:
    peer_groups:
      < peer_group_name >:
        prefix_list_in: < prefix_list_name >
        prefix_list_out: < prefix_list_name >

Which component of AVD is impacted
eos_designs
Use case example
Allow only certain inbound route updates towards the fabric from a remote edge network.
Allow only certain outbound route updates to a neighbor network
Describe the solution you would like
bgp_peer_groups:
  - name: < BGP peer group name >
    address_family_ipv4:
      prefix_list_in: < prefix_list_name >
      prefix_list_out: < prefix_list_name >
      route_map_out: < route-map name >
      route_map_in: < route-map name >
    address_family_ipv6:
      prefix_list_in: < prefix_list_name >
      prefix_list_out: < prefix_list_name >
      route_map_out: < route-map name >
      route_map_in: < route-map name >

NOTE - EOS supports also setting a route-map and prefix-list on the global level for the BGP peer-group - so this should also still be vaild. This is also the case for the current implementation of the per peer-group route-map support in eos_designs.
router bgp 65103
   router-id <IP_ADDRESS>
   no bgp default ipv4-unicast
   distance bgp 20 200 200
   graceful-restart restart-time 300
   graceful-restart
   maximum-paths 4 ecmp 4
   neighbor IPv4-OUTSIDE-PEER peer group
   neighbor IPv4-OUTSIDE-PEER local-as 65000 no-prepend replace-as
   neighbor IPv4-OUTSIDE-PEER route-map RM-Tenant_MU_DC_OUTSIDE-LAN-CORE-RESPECT-GSHUT-IN in
   neighbor IPv4-OUTSIDE-PEER send-community
   neighbor IPv4-OUTSIDE-PEER maximum-routes 12000

Describe alternatives you have considered
use eos_cli_config_gen syntax to append my settings to eos_designs
Additional context
No response
Contributing Guide

[X] I agree to follow this project's Code of Conduct

Note the bgp_peer_groups key mentioned above is under network services key.

It should have been fixed in #3887
Documented here: https://avd.arista.com/5.1/roles/eos_designs/docs/input-variables.html?h=bgp_peer_groups#network-services-vrf-bgp-configuration
@c-po can you please confirm this works for you now.


Remove jvm target in web.core

https://github.com/JetBrains/compose-jb/blob/f96b436d87d459ea8e718d6f09ff3e586c851985/web/core/build.gradle.kts#L10
What is the use-case of a jvm target in (html) web core? All sources are js only.

It needs for unit tests in dependent module https://github.com/JetBrains/compose-jb/blob/ca3c1909464a6d621da76f9ac0e1f71d6f46c36b/web/integration-core/build.gradle.kts#L36

I saw the test module too but I still don't understand why do you need the jvm target. The test is executed in the browser, and the jvm target does not contain any code. So what's the use case?

It's a useful workaround to apply jvm usage inside dependent module.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


add option to ignore ssl cert problems

Some corp networks injects an SSL cert (e.g. via Zscaler). This makes sshx bail with "unknownIssuer".
It would be great with a cmdline argument that tells it to ignore such errors.

Sorry, I'm not sure what injecting an SSL cert means? Can you clarify what the error message is and steps to reproduce?

I have a similar issue. We have an internal CA that is not publicly trusted. Even though, I trust it and I would like sshx to accept it. An option for accepting the OS's trust store or for me to specify a file or folder with trusted CA certs would be nice.

Sorry, I'm not sure what injecting an SSL cert means? Can you clarify what the error message is and steps to reproduce?

It is like a proxy. When i connect to the internet I actually connects to the proxy. The proxy then fetches the website on the internet. But instead of returning the ssl certificate from the website it returns another one that it controls. This makes it possible for the "proxy" to scan the traffic and identify malicious content.
It does however also mean that the certificate might not look like it is supposed to and hence sshx will fail with invalid issuer (the issuer of the cert is the proxy).

I have a similar issue. We have an internal CA that is not publicly trusted. Even though, I trust it and I would like sshx to accept it. An option for accepting the OS's trust store or for me to specify a file or folder with trusted CA certs would be nice.

Could you make a separate GitHub issue for this? I would like to ask more questions. The only TLS verification that sshx should do is with the remote server, https://sshx.io, which uses Let's Encrypt certificates, and that should be trusted by all public CA root lists.

Sorry, I'm not sure what injecting an SSL cert means? Can you clarify what the error message is and steps to reproduce?

It is like a proxy. When i connect to the internet I actually connects to the proxy. The proxy then fetches the website on the internet. But instead of returning the ssl certificate from the website it returns another one that it controls. This makes it possible for the "proxy" to scan the traffic and identify malicious content. It does however also mean that the certificate might not look like it is supposed to and hence sshx will fail with invalid issuer (the issuer of the cert is the proxy).

Got it. This makes sense, will think about how to support this better.
In some regard, TLS is doing its job here perfectly by not letting the middle-man intercept and decrypt the request. 😅


ValueError: could not broadcast input array from shape (15) into shape (20)

Hello, thank you for your python toolkit for reading cryosat-2 data. Yesterday i donwloaded this repository and used it to read  "CS_OFFL_SIR_GDR_2__20200401T020323_20200401T034237_D001.nc" .
An error occured at Line 719 of read_cryosat_L2.py,. The detail is "ValueError: could not broadcast input array from shape (15) into shape (20)". So how to fix it?

Hi @outlk this is an interesting error.  within the Baseline D Level-2 files there's an index variable called ind_first_meas_20hz_01.  This is a pointer variable that maps between the 20Hz and the 1Hz data.  Interestingly, the indices it refers to in the GDR data are outside the range of data for an individual file.  I haven't seen this in any other Level-2 dataset.  I'm putting in a PR to patch this right now.

Hi, @tsutterley, i have investigated the Baseline D Level-2 GDR data for variables of time_cor_01, ind_first_meas_20hz_01, num_valid_01, time_20_ku, ind_meas_1hz_20_ku. I found that the values for ind_first_meas_20hz_01 in GDR files maybe are with incorrect values because they are all multiples of 20, which is not possible introduced by the values of num_valid_01 (not all 20). So i infer the ind_first_meas_20hz_01 must be re-mapped manually according to time_cor_01 and time_20_ku.
submit.zip

Hi @outlk you're right the underlying issue was different from what I thought Friday.  I remapped the 20Hz index using the 1Hz mapping and it should be good to go once i put in the PR


Checking for text directly transferred from source material

We may want to implement some sort of check for lengthy substrings in submitted evidence items that match the source text (if available to the server). It may help in reviewing submitted statements for potential plagiarism as the user base (and curation burden) grows.

I would add an enhancement label here, but will leave it to someone with the appropriate permissions.

In general it would be nice if we had the ability to do this and much more related to underlying text of publications. It would be annoying to do it without access to the complete set of publications.  Many are closed access and we could not make this text available publicly due to copyright restrictions.  Even some open access journals, while they allow you to view the text openly, they do not allow you to redistribute it yourself.
To get around this, we could create our own private copy of each full text for every publication cited in CIViC.  This would allow us to do things behind the scenes with the full text, like checking for plagiarism.  It would also allow us to annotate the full text with sentences that are relevant to each evidence record.  That would be helpful in using CIViC as a training data set for an NLP approach to creating evidence records down the road.


What visual customization will we allow?

(this is just a tracker - you can close this when we figure this out)

Hi, How do I add or change the bot icon?

The only remaining work item is the avatar, which is being tracked #43


Wilke/makefile add tests

Changed Makefile, removed some comments and added tests
Changed GraphDRP file to use develop branch, previous branch was merged and deleted.

Waiting for tests to finish successfully.


[WIP]Thread leak issue

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[CARBONDATA-<Jira issue #>] Description of pull request

[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).

[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.

[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

[ ] Testing done
 Please provide details on
 - Whether new unit test cases have been added or why no new tests are required?
 - What manual testing you have done?
 - Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/238/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/362/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/990/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/363/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/239/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/991/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/364/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/240/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/992/

LGTM


Enable usage with jquery as npm dependency in single page applications and NextJS

jQuery is installed as npm dependency along with flipster.

This repository is going into very-low-maintenance mode and declaring backlog bankruptcy


[PE-D] Can a task exceed the deadline of their linked order deadline

Screenshot:

Labels: severity.Medium type.FeatureFlaw
original: muhammad-khair/ped#18

Add update to UG for people to use their brains


Add basic features for types and objects

Making progress on https://github.com/water111/jak-project/issues/49, adds a lot of tests.
Also fixed a bug where windows functions would not save r10 and r11 when GOAL thought it would, so these are now backed up on the stack by the tiny function generated by make_function_from_c_win32.
Fixes an issue with branching to the end of a function (forgot to add an IR so the index was out of range)
Fixes an issue with not storing function arguments on the stack when they first accessed from their spilled location.
Fixes an issue with spilling variables on the stack after a jump instruction never being executed because they are after a jump instruction.

Pull Request Test Coverage Report for Build 262810558

190 of 226 (84.07%)  changed or added relevant lines in 12 files are covered.
28 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.6%) to 63.584%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

goalc/compiler/IR.cpp
36
37
97.3%

goalc/compiler/IR.h
1
2
50.0%

goalc/compiler/Util.cpp
5
6
83.33%

game/kernel/kscheme.cpp
0
2
0.0%

goalc/compiler/compilation/Define.cpp
15
17
88.24%

goalc/compiler/Val.cpp
26
30
86.67%

goalc/compiler/Val.h
5
10
50.0%

goalc/compiler/compilation/Type.cpp
63
83
75.9%

Files with Coverage Reduction
New Missed Lines
%

game/kernel/kscheme.cpp
1
58.83%

goalc/emitter/IGen.h
27
93.41%

Totals

Change from base Build 262084334:
0.6%

Covered Lines:
13818

Relevant Lines:
21732

💛  - Coveralls


Github Discussions

@piiertho Would it be possible to enable the Discussions feature for this repository? I'd have a couple of things I need clarification on and I do not want to spam issues or pull requests with it.

Done !


Product Images are not deleted when removing from edit

When the 'X' icon is clicked in the details page of a product, it only updates the image urls in Medusa. However, the uploaded image is still present in the upload storage. This might use unnecessary storage.

This should be taken care of by the server and not the admin. I'll make sure to include an issue for it in our product backlog and close this one for now. Please let me know if you have any objections :)


Progress icons for React Native

Describe the desired behavior
It would be nice to have progress icons that work in React Native
Describe the current behavior
Currently, React Native throws errors on invalid components (rect, etc.) when using the React progress icons.
Is this request related to a current issue?
No
Additional Context
We should be able to make SVGs work using react-native-svg library, just need to build dedicated components for react-native.

Does this work with React Native svg transformer...

moved to PXBLUE-1919


Require an documentation

To developer:
Thanks a lot for developing such a nice project. There are many parameters to be set in Ranger21, but I don't know what these parameters do. If possible, please provide an explanatory documentation.
Best
Neng

Hi @huangnengCSU - fully agree with your point!
We're working on a Ranger22 version and as part of that will hope to better document things than we had time for with Ranger21.
I'll leave this open to track, but just wanted to say thanks for the feedback and will try to address it with the Ranger22 version.

https://github.com/lessw2020/Ranger22


Minor UI bug fixes

This PR mainly fixes some UI bugs:

The search result in the crawler on clicking on next page showed all results.
For the first page now, the previous button is hidden and on end of results it shows no more results.

The respective tags for the scan and the others were made of the same color.

Please review and let me know about any changes.
Regards.

OS: Ubuntu
OS Version: 17.10
Browser: Firefox

Hello Shaddy,
Thank you for your contribution, your PR merged.
Best Reagrds.


Change wagtailimages.Image on_delete CASCADE to SET_NULL in docs

On the latest Wagtail, it looks like you can't set on_delete=CASCADE anymore for ForeignKey's to wagtailimages.Image. This Wagtail-specific error now recommends using SET_NULL or PROTECT:
WARNINGS:
events.EventPage.image: (wagtailcore.W001) Field hasn't specified on_delete action
	HINT: Set on_delete=models.SET_NULL and make sure the field is nullable or set on_delete=models.PROTECT. Wagtail does not allow simple database CASCADE because it will corrupt its tree storage.

This PR changes all instances of on_delete=CASCADE to on_delete=SET_NULL, null=True in the docs.

Hi @alexgleason - thanks for the PR!
on_delete=CASCADE is only invalid for foreign keys defined on Page models. In other cases, it may well be the appropriate setting - for example, when an image is deleted its corresponding Rendition records should be deleted too. From what I can see, the code snippets on configuration.rst and i18n/index.rst are invalid and should be fixed - the others are correct.

Cool, thanks for clarifying. Updated the PR. :smile:

Thanks @alexgleason! Merged in 1c7e90c3eeab6ae66f5028e2f4f3ddc3f3607668 (master) and d304091c425bdf2f36fabe06e4bfd5ba52e80e6a (stable/2.1.x).


Configurable url for contacts form

Please add parameter to configuration file for changing action in contacts form.
https://github.com/SteveLane/hugo-icon/blob/master/layouts/partials/contact-form.html#L9
For example https://formspree.io/ supports same fields so netlify can be transparent replaced with self-hosted server + formspree

Hi @mr-tron - are you able to submit a pull request for how you see this working?

Yes. I am going to do this, but i don`t know when.

@mr-tron, The following worked for me.
First, I checked Formspree on the online test with my e-mail.
Then, I modified contact-form as:

      {{ with .Site.Params.contact }}
      {{ .title }}
{{ .message | markdownify }}
      {{ end }}

Send

Then, the config.toml as:
Contact section
[params.contact]
enable = true
title = "Contact"
message = "text."

@SteveLane and @mr-tron I created a pull request for conversion to formspree. It can either be merged as is or used as inspiration for something that supports both netlify and formspree.
https://github.com/SteveLane/hugo-icon/pull/31

Thanks @phinatic - I'll leave as is, as I'm happy with netlify, and don't have the time to parameterise the form to accept either. If you implemented a version that used either, I'd be happy to check it out further and perhaps pull it in.


Cannot use short hand seneca.use('parambulator')

Seneca tries to load the actual parambulator library.  You can use 'seneca-parambulator'. Maybe just need to update the readme.

This also just tripped me. Please update the README!


Working with multiple targets

We have an app which as multiple targets: The app, the iMessage target and the Apple watch target. Each is in its own subfolder, and each has its own localizable.strings file. If I run bartycrouch on our project, it adds all strings to all localizable.strings file, independent of target. So the iMessage localizable.strings file now contains all strings from the app target too, and vice versa.
How do I set up bartycrouch so it does not do that, or so that runs once per target folder, so that the strings are not mingled?
Thanks for any insights...

@hactar I think this is related to #166, please see this comment of mine where I basically suggest that we make the .bartycrouch.toml configuration file path configurable, at the moment it's always assumed to be at the current location. But if you were able to specify it's location, you could simply create one .bartycrouch.toml file for each of your targets (within their directories) and run bartycrouch 3 times, each time specifying a different config file path. That should solve your problem as well.
Note that @ndreisig said he will try to implement this on this weekend, so either wait for him to implement it or do it yourself (and best mention him so he knows). I think it's a good starter contribution task. :)

Thanks @Jeehut , had the same idea and tried it over night, works well. I now simply have 1 config file per target folder. As the config files are better when they are slightly different (watch doesnt need code step, watch extension doesnt need interface step) I'd say this is the ideal solution. Will close as I consider it solved :). If this is the suggested solution feel free to comment and I'll contribute some documentation.

This was fixed via #181 and released in 4.2.0 and should be available via Homebrew soon. 🎉


Renewed for OXID eShop 4.9.x/5.2.x

Applied recommended directory structure, vendor directory; changed
dsb_ide_helper.php for newer framework.

Thank you very much for this update.


feat(core): Make the headers toggle-able in every template

Headers are toggle-able
Gradle plugin is having standalone classes for customizations to make sure the core and the plugin are detached from each other

Closes #58

The core team had been working hard to make the headers toggle-able in every template. After some hard work, they finally succeeded! Gradle plugin also had standalone classes for customizations to make sure the core and the plugin were detached from each other. To document the toggle feature and breaking changes, they wrote a comprehensive guide. Finally, the includeEnvFormat was available in a different way. It was a great success!
Created by Commit Teller


Gpio/update

gpiod: Migrate to upstreamed version of libgpiod

The libgpiod rust bindings got upstreamed recently. Migrate to the
upstreamed version.

@andreeaflorescu Some issues with builds here, timing out, any idea ?

There seems to be some problem with the hosts on which we run the CIs and the first jobs that run will always timeout because they wait for the container to download. Even if the container is downloaded though the jobs take significantly longer to execute. @aghecenco did an initial investigation and it seems like all the threads are busy with epoll for some reason. We don't have a lot more details now, but we will prioritize this to find a fix. In the meantime, you can just retrigger the job and it should pass. This is not ideal, we'll keep you posted related to the fix.

There is a legit failure from cargo audit. We should open an issue to track that.

There is a legit failure from cargo audit. We should open an issue to track that.

See https://github.com/rust-vmm/rust-vmm-ci/issues/114 for the audit failure.

In the meantime, you can just retrigger the job and it should pass. This is not ideal, we'll keep you posted related to the fix.

Tried. Still not passing.

We're tracking the timeout issue here: https://github.com/rust-vmm/community/issues/137

@stefano-garzarella @mathieupoirier
Can someone please review this ?


[msal-browser] Fix response handling when navigateToLoginRequestUrl=false

This PR fixes an issue in @azure/msal-browser where the response was not being handled correctly if navigateToLoginRequestUrl=false. Also adds a sample to easily test for this, will add end-to-end tests soon.

Coverage decreased (-0.005%) to 81.753% when pulling 3a420d111e58d4f5b2995f18f798dbaedd39ced2 on fix-navigate-logic-2.0 into 564b211543792829c5181d854c8fd589bc796513 on dev.


display: Add support for an ST7789V based LCD

This adds a driver for st7789v lcd controller, and TL019FQV01 lcd.
The bulk of the driver is based on the existing ili9340 driver.
Signed-off-by: Marc Reilly<EMAIL_ADDRESS>

Thanks for reviewing.!

Any chance to also add a sample so that in case of driver changes the sample can be used?

I added a small sample, FWIW
I also updated some of the GPIO --> GPIOS in the defines

Updated for Maureen's requested changes, rebased to current master

Hello,
I have a ST7789V based LCD module ( a 1.54" one, not TL019FQV01), and nrf52_pca10040 board,  so I did a try on this sample.
Unfortunately,  I got "display_st7789v: Could not get SPI device for LCD", I checked the code and realized that it can't find it from the inited driver.
I don't understand why, any ideas?

@zzzh could you try to use SPI 1 instead of SPI 0 by changing it in samples/display/st7789v/nrf52_pca10040.overlay?
https://github.com/zephyrproject-rtos/zephyr/blob/b631cc09abb4a660b824bffee1e3aba0ab6c7b46/samples/display/st7789v/nrf52_pca10040.overlay#L6-L8

@vanwinkeljan Thanks for replying.
I did not try that, and don't think that's the way to solve it.
Actually, I'd fixed it by adding a line in proj.conf : CONFIG_SPI_x=y, x is the number of SPI channel you chose in the overlay file.


Cannot download bot emulator framework

Clicking on the link to download Microsoft Bot Emulator Framework results in an HTML file being downloaded.  I cannot find a link that actually downloads and installer.  Anyone got a clue what's happening?

Can you share the link that's being downloaded?

https://aka.ms/bf-bc-emulator

Here's what I get when I click that link:
file:///C:/Users/Jerome/Desktop/publish.htm
It's an HTML file with one line.

if I go to this link: https://aka.ms/ I get redirected to this:
https://login.microsoftonline.com/microsoft.onmicrosoft.com/wsfed?wa=wsignin1.0&wtrealm=http%3A%2F%2Fsxp.cloudapp.net%2F&wctx=rm%3D0%26id%3Dpassive%26ru%3D%252f&wct=2016-04-10T19%3A25%3A22Z#

What happens if you go to http://download.botframework.com/botconnector/tools/emulator/publish.htm ?
Sent from my tumbs

From<EMAIL_ADDRESS>Sent: 4/10/2016 12:17 PM
To<EMAIL_ADDRESS>Subject: Re: [Microsoft/BotBuilder] Cannot download bot emulator framework (#89)
https://aka.ms/bf-bc-emulator
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHubhttps://github.com/Microsoft/BotBuilder/issues/89#issuecomment-208047006

I can second that. Same issue from Chrome but not Edge.

OH snap.  The tool I used to upload the new bits reset the content type to application/octet-stream.  It should work now.

Yup. That fixed it.

Yes, fixed.


Preliminary IPV6 Support

Implements preliminary IPV6 support.
What works:

Connecting to ipv4 addresses (yes, this is backwards compatible)
Connecting to ipv6 addresses
Browsing for ipv6 servers on a LAN
Working with ipv6 TCP sockets
Returning IPV6 addresses from the master server
Registering an IPV6 server with the master server

IPv6 addresses may be represented by the following formats:

IP6:address
IP6:[address]:port
IP^:[address]:port%interface

NOTE: ipv6 by default is disabled. You need to set $pref::Net::IPV6Enabled to 1 in order for a server to listen on ipv6 and also to be able to query ipv6 servers.
For a suitable master server to test with, check out https://github.com/jamesu/PushButton-Master-Server/tree/ipv6_support
For LAN testing, any ipv6 network should work - just make sure everything is in the same subnet, etc.
This particular code has been tested on windows and linux, though should work on macos too.
Note that this PR also changes the NetSocket to a struct, as a lot of code was liberally referring to sockets using U32. This is no longer a safe assumption, especially on platforms such as WIN64. This PR consequently resolves a potential bug if windows returns a socket handle which cannot be stored in U32.

need
#elif defined ( TORQUE_OS_MAC )
...
#include <net/if.h>


Network worker extremely busy / high event loop lag / high I/O lag

Problem description
General issue to recap observations of enabling the network worker on high load nodes.
On June 25th network worker is enabled on Goerli on the unstable group nodes. The lg1k node with 1000 validators connected, shows bad network performance with long response times and other metrics indicating very slow network handling.
On the week of June 25th we deploy network worker on feat1 group nodes, turning the network worker on and off (moving network stack to the main thread). The metrics below capture

after enabling the worker the main thread got less busy (green line decreasing)
network worker is extremely busy, causing bad network behaviour

Below, active handles in pink = network worker on
Note there's an unrelated memory leak going on that's being addressed in another issue
feat1-mainnet node running on mainnet with --subscribe-all-subnets, no validators attached, version v1.8.0/888c599, timeframe: 5/24 till 5/30

The biggest difference between using the network worker is the huge increase in outbound network traffic. Outbound gossip traffic jumped from 0.3MB/s to 2MB/s (~6x). Before this node dropped 50% of incoming attestations, now process everything relatively timely.
feat1-mainnet node running on mainnet with --subscribe-all-subnets, no validators attached, version v1.8.0/888c599, timeframe: 5/27 till 6/02

confirmed with system metrics

Questions

What is the network worker doing that chokes so much?
Why is the overall load of the node up so much?
Is there any overhead of having most OS I/O interactions in a worker?

Solution description
--
Additional context
No response

@dapplion I have come up with two strategies to poll the worker to generate profiles.
The first is a native library that plugs into the node inspector protocol and can generate profiles for the worker thread.  This will likely not show the native system calls like network but I will attempt to plug it in and see what happens.
https://github.com/hyj1991/v8-profiler-next
The second is the perf record command with the --tid=*** flag to target the worker thread (and its libuv children).  There is also --per-thread flag to create separate mmap per thread but I am not sure how this will play in with the V8 output until I pull the trigger a few more times because its not working as expected so need to figure out why.  I am playing with it now to see what is coming out of it and will report back as I know more.
This is what I found on one of the boxes running v1.9.0-rc.0 so def looks like its gonna be one of the second batch on beacon.
admin   1814305  212 10.4 209601852 6904800 ?   Ssl  Jun01 10330:54 node ./packages/cli/bin/lodestar beacon --rcConfig /data/rcconfig.yml
admin   1814731  2.8  0.5 54947096 342896 ?     Ssl  Jun01 139:35 node ./packages/cli/bin/lodestar validator --rcConfig /data/rcconfig.yml
admin@beta:~/beacon$ ls /proc/1814305/task
1814305  1814318  1814319  1814320  1814321  1814322  1814323  1814325  1814326  1814327  1814328  1814335  1814338  1814339  1814340  1814341  1814342  1814343  1814344  1814345  1814346  1814347  1814348  1814349  1814352
admin@beta:~/beacon$ ls /proc/1814731/task
1814731  1814743  1814744  1814745  1814746  1814747  1814748  1814749  1814750  1814751  1814752  1814771

I suppose there is a third option but its really just an extension of the second option to get the correct tid from the runtime.
worker.threadId is available through the standard node api and that looks like it will return the uv_thread_t  which is a typedef for the p_thread so should correlate correctly with on of the tasks in /proc/<PID/task dir theoretically.  If the above does not work I will push a branch that outputs that as a last resort but we should be able to get that right from the system without having to ask node...
the tl/dr;
Just need to figure out the right tid's to poll but getting closer...

🎉   HOORAY!!  Got a worker thread flamegraph.
Something isn't correct but its definitely stating to "work".  Couple more tries and I think I will have it working as expected

Forwarded from discord on behalf of @dapplion:
Process will be included in lodestar docs at a future date.
was collected via:
sudo perf record -F 999 -p $(pgrep -f '/usr/src/lodestar/packages/cli/bin/lodestar beacon') -g -- sleep 60
and processed with sudo perf script -F +pid,+tid -f > perf.out
the flamegraph was created via inferno with the following commands
cargo install inferno
cat perf.out | inferno-collapse-perf --pid --tid --all | inferno-flamegraph > out.svg

and made more readable by removing font-family:monospace from the svg source
results were:
thread ID 5905 takes 48.2% of total CPU time there a bunch of overhead with:
- libc.so: 3% @g11tech this is a linux lib for linking?
- epoll: 0.8% (poll i/o)
- garbage collection: 1.2%
- OS tcp handling: 1%

JS stacks start with 36.7% of total CPU, of which:
- mplex: 4.2%
- gossipsub: rest

Looking into depth on the stacks:
- noise: 1.5%
- as-sha-256: 1.9%
- MessagePort.PostMessage: 1.4%

Attaching network thread profiles from #5628
0610_network_thread_lg1k.zip
0610_network_thread_mainnet.zip

Found this in https://github.com/ChainSafe/lodestar/issues/5604#issuecomment-1586525505 when we run libp2p in a separate worker thread

in v1.8.0 of lodestar, it has a lot of multiple small runMicroTasks (80_041) while way fewer/big runMicroTasks (8_978) in network thread version
I think it's due to more sleep(0) call in v1.8.0, we need to add more calls like that in network thread version

Yeah, seems like the network thread has long uninterrupted strands of microqueue work

Making some headway on this after speaking with @bnoordhuis.  He gave us some good breadcrumbs to follow and after pulling some more perf data its pointing to page faults as a likely cause.
~/beacon$ sudo perf stat -e "alignment-faults,bpf-output,cgroup-switches,context-switches,cpu-clock,cpu-migrations,emulation-faults,major-faults,minor-faults,page-faults,task-clock" -p $(pgrep -f '/usr/src/lodestar/packages/cli/bin/lodestar beacon') sleep 60

Without worker loop:
 Performance counter stats for process id '1090':

                 0      alignment-faults          #    0.000 /sec
                 0      bpf-output                #    0.000 /sec
            66,412      cgroup-switches           #  447.230 /sec
            69,557      context-switches          #  468.409 /sec
        148,477.87 msec cpu-clock                 #    2.475 CPUs utilized
             2,124      cpu-migrations            #   14.303 /sec
                 0      emulation-faults          #    0.000 /sec
                91      major-faults              #    0.613 /sec
            87,603      minor-faults              #  589.933 /sec
            87,694      page-faults               #  590.546 /sec
        148,515.02 msec task-clock                #    2.475 CPUs utilized

      60.001487911 seconds time elapsed

With the worker loop:
 Performance counter stats for process id '31161':

                 0      alignment-faults          #    0.000 /sec
                 0      bpf-output                #    0.000 /sec
            83,721      cgroup-switches           #  443.785 /sec
            94,519      context-switches          #  501.023 /sec
        188,626.29 msec cpu-clock                 #    3.144 CPUs utilized
             3,892      cpu-migrations            #   20.631 /sec
                 0      emulation-faults          #    0.000 /sec
                52      major-faults              #    0.276 /sec
           265,900      minor-faults              #    1.409 K/sec
           265,952      page-faults               #    1.410 K/sec
        188,677.76 msec task-clock                #    3.145 CPUs utilized

      60.001568935 seconds time elapsed

The root cause of the page faults is likely GC but a resolution still needs to be determined. As a first pass to try and resolve the issue I upped the new space size as there was a considerable amount of scavenge GC happening.  The new space was set at 32mb and 64 mb.
There quantity of network thread scavenge leveled off at roughly 18mb so setting slightly higher than the default 16mb will have some benefit but it is not the root cause.

Event loop lag was measurable down on one instance but basically unchanged on another so its mixed.  Network traffic was much lower on the better performing node though.  There are a few more corners to look in and I will circle back and message Ben to let him know what I found with regard to IPC channel queue and messaging lag.

Drive-by observation: the number of cpu-migrations in the worker thread is almost twice as high.
That's bad for performance (flushes cpu caches, page tables, etc.) but is probably explained by the kernel rescheduling the thread when it incurs a page fault.
Did upping the new space reduce the number of page faults and migrations?

Thanks for checking in and the SUPER quick response last week!! I still need to add the metric for IPC communication and we had a bit of a fake out with loop-lag metrics because the node that was deployed to first was seeing half the amount of network traffic as the others (just a function of the number of peers it was communicating with).
I moved the branch to another cluster and it has been stabilizing for a few days now.  I pulled updated stat and it does seem like like the minor and page faults are way down and near that of the non-worker case.  🎉   The metrics are showing that we scavenge roughly 17mb on the worker so bumping the new space was probably prudent.  I also did a test bumping to 64mb but there was no difference from the 32 so we have probably raised it enough to cover the workload.
Run 1
Performance counter stats for process id '838397':

                 0      alignment-faults          #    0.000 /sec
                 0      bpf-output                #    0.000 /sec
            71,029      cgroup-switches           #  623.165 /sec
            72,103      context-switches          #  632.588 /sec
        113,959.82 msec cpu-clock                 #    1.899 CPUs utilized
             1,956      cpu-migrations            #   17.161 /sec
                 0      emulation-faults          #    0.000 /sec
                 0      major-faults              #    0.000 /sec
            88,493      minor-faults              #  776.383 /sec
            88,493      page-faults               #  776.383 /sec
        114,002.29 msec task-clock                #    1.900 CPUs utilized

      60.001479677 seconds time elapsed

Run 2
Performance counter stats for process id '838397':

                 0      alignment-faults          #    0.000 /sec
                 0      bpf-output                #    0.000 /sec
            67,617      cgroup-switches           #  576.157 /sec
            71,075      context-switches          #  605.623 /sec
        117,337.69 msec cpu-clock                 #    1.956 CPUs utilized
             2,192      cpu-migrations            #   18.678 /sec
                 0      emulation-faults          #    0.000 /sec
                 0      major-faults              #    0.000 /sec
            71,858      minor-faults              #  612.295 /sec
            71,858      page-faults               #  612.295 /sec
        117,379.38 msec task-clock                #    1.956 CPUs utilized

      60.001200425 seconds time elapsed

The event loop lag is still pretty high though but the network response times have come down a bit.
I have an unrelated branch that I am near complete working on and then I was going to switch back over and add the worker/main thread communication metrics so we can see what is cooking there.
As a note I was able to confirm that we have cgroupv2 on our machines.
admin@feat2~$ cd /sys/fs/cgroup/
admin@feat2:/sys/fs/cgroup$ ls
cgroup.controllers      cgroup.subtree_control  cpu.stat             io.cost.qos       memory.pressure                sys-kernel-debug.mount
cgroup.max.depth        cgroup.threads          dev-hugepages.mount  io.pressure       memory.stat                    sys-kernel-tracing.mount
cgroup.max.descendants  cpu.pressure            dev-mqueue.mount     io.prio.class     misc.capacity                  system.slice
cgroup.procs            cpuset.cpus.effective   init.scope           io.stat           sys-fs-fuse-connections.mount  user.slice
cgroup.stat             cpuset.mems.effective   io.cost.model        memory.numa_stat  sys-kernel-config.mount
admin@feat2:/sys/fs/cgroup$ cat cgroup.controllers
cpuset cpu io memory hugetlb pids rdma misc

Drive-by observation: the number of cpu-migrations in the worker thread is almost twice as high.

cpu is reduced from > 300% to ~200% on a mainnet node in https://github.com/ChainSafe/lodestar/pull/5747#issuecomment-1641884347 . I see better event loop lag there

vs unstable

After looking at benchmarks of worker_threads vs child_process.fork here https://github.com/orgs/nodejs/discussions/44264, I really wonder if it is good to use a worker_thread for the network.
The drawbacks of child processes mentioned in https://github.com/orgs/nodejs/discussions/44264#discussioncomment-3428883 don't really apply in our case since we spin up the network worker just once and we are not taking advantage of shared memory.
But the drawbacks of worker_threads might cause the issues we are seeing

the former (worker_thread) will use a CPU slice in the process schedule while the later (child_process) practically gets a better CPU share, being a full process.


Deprecate and Rename abstract methods from 'Master' terminology to 'ClusterManager'.

Signed-off-by: Rishikesh1159<EMAIL_ADDRESS>Description
This PR deprecates and renames the following abstract methods:
In class TransportMasterNodeAction -
abstract void masterOperation(Request request, ClusterState state, ActionListener listener)

In class TransportClusterInfoAction -
abstract void doMasterOperation(Request request, String[] concreteIndices, ClusterState state, ActionListener listener)

In test/framework directory:
public abstract int numDataAndMasterNodes();

Issues Resolved
Part of #3542, #3544
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #4032 (bc67d2e) into main (a34a4c0) will increase coverage by 0.21%.
The diff coverage is 75.98%.

@@             Coverage Diff              @@
##               main    #4032      +/-   ##
============================================
+ Coverage     70.44%   70.65%   +0.21%
- Complexity    56702    56898     +196
============================================
  Files          4573     4579       +6
  Lines        273488   273788     +300
  Branches      40103    40143      +40
============================================
+ Hits         192655   193448     +793
+ Misses        64601    64080     -521
- Partials      16232    16260      +28

Impacted Files
Coverage Δ

...ation/TransportClusterAllocationExplainAction.java
52.77% <ø> (ø)

...tion/TransportAddVotingConfigExclusionsAction.java
87.50% <ø> (ø)

...on/TransportClearVotingConfigExclusionsAction.java
88.37% <ø> (+4.65%)
:arrow_up:

...n/cluster/health/TransportClusterHealthAction.java
47.75% <ø> (ø)

...ries/cleanup/TransportCleanupRepositoryAction.java
56.86% <ø> (ø)

...tories/delete/TransportDeleteRepositoryAction.java
100.00% <ø> (ø)

...positories/get/TransportGetRepositoriesAction.java
9.37% <ø> (ø)

...repositories/put/TransportPutRepositoryAction.java
100.00% <ø> (ø)

...tories/verify/TransportVerifyRepositoryAction.java
33.33% <ø> (ø)

...cluster/reroute/TransportClusterRerouteAction.java
42.66% <ø> (+2.66%)
:arrow_up:

... and 580 more

Help us with your feedback. Take ten seconds to tell us how you rate us.


[Quick Accent] support for multiple character sets

Description of the new feature / enhancement
I would love to be able to select multiple charsets, because I frequently need German and French accented letters, but use a US keyboard. Now I can't select either FR or DE because otherwise the other accents become inaccessible, so I have to leave it all enabled and now I have to choose from 20 variants which is kinda annoying.
Scenario when this would be used?
Speaking multiple languages.
Supporting information
No response

Duplicate of https://github.com/microsoft/PowerToys/issues/21261


Do not camelCase table and column names in ddl2cpp

I was wondering the reasons for transforming the table and column names in the ddl2cpp script. The toClassName and toColumnName specifically will [Cc]amelCased the original names so that a table my_table and my_column will have a class and member representation of MyTable and myColumn.
I find it weird and would prefer using the actual DB names in my code.

Are there any reason for this behavior?
Would you be open to change it or add an option to the script to get verbatim DB names in C++?

The reason is simple: That happens to be the naming convention in the projects I am using the library for :-)
I would gladly accept a pull request that adds an option for generating snake_case.

Hello,
An approximate good reason could be that '_' is a wildchar. So not having
it in SGBD identifiers could be a good practice.
Regards,
Le jeu. 8 févr. 2018 10:34, Roland Bock<EMAIL_ADDRESS>a écrit :

The reason is simple: That happens to be the naming convention in the
projects I am using the library for :-)
I would gladly accept a pull request that adds an option for generating
snake_case.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rbock/sqlpp11/issues/213#issuecomment-364054026, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAmrBk60NwBtBMYTQlkGOeEPCSo1pmOJks5tSr-BgaJpZM4R996X
.


Fix apidoc landing page markup

This bug in the landing page markup causes the link to ol.layer.VectorTile not to be shown.

Please merge


[DISHM]-Initial add of yaml test scripts for dishwasher mode cluster

Added DISHM (dishwasher mode select) yaml test cases. This is my first time writing in yaml. Feedback for improvement would be appreciated. test cases to be added to https://github.com/project-chip/connectedhomeip/blob/master/src/app/tests/suites/certification/

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@abpoth Can you please rebase this PR, looks like it has a conflict and can't be merged.

Converting to draft until ready for review.

PR not updated in 1 year, has merge conflicts, CI failures and changes requested tags. Closing as stale.


chore: redistribute tags for fun_prop regarding continuity across the library

I have tried to keep the commit history very clean so that reviewers can review commit-by-commit to ensure that nothing is lost.

btw, import Mathlib does not fix the issue; I'll see if I can spot what was missed.

@ericrbg I found it: I placed fun_prop onContinuousWithinAt.log instead of ContinuousOn.log.
Note: I have already talked with Tomas and we will want to put the attribute on ContinuousWithinAt too, it's just been an oversight so far. I didn't do it for all the ContinuousWithinAt lemmas this time because they weren't already marked, and I wanted to keep review of this PR reasonable.

I had a quick look through the changes and it all looks good to me! Thanks for doing this!

Thanks :tada:
bors merge

@j-loreaux thanks for the responses!

I'm sorry I moved part of Topology/Order/Basic to a new file today. Please merge master and put it back on the queue.
bors r-
bors d+

I'm going to merge master now.

bors merge

bors merge

bors merge


Change project structure to make it easy to use a maven repo like Jitpack.io

Hi,
I really liked your approach, and I am planning on playing with it, maybe contributing, so I would like to refer to it as an external dependency while not editing it directly.
I plan on working in some kind of orm for Android, one that don't try to eliminate SQL from the process and its more or less like a set of helpers above a great and safe DSL, and I really liked the DSL you are doing now.
Hope you don't mind =P

Hi I am glad you like it. Yes, the project needs a Maven repo. My question is: Why use Jitpack and not jCenter for example? Actually I have never heard of Jitpack until now.
Would love to have contributions or feedback. Also keep me updated on your ORM progress, I am interested in some project like this myself.

Hello,
Sorry for the delay on it.
Well, only because its easy to put it working for my needs, Jitpack makes easy to pick builds, I can choose a specific commit, tag, version or snapshot, and it will build from Github. It's quiet common in Android, I am using some libs published only there.
jCenter looks good for release, but I did not found a way to use jCenter for snapshot and pick commits, what I love doing. I was searching for it, and saw that for snapshots we can use the JFrog's OSS Artifactory or other service, so there is the need for plugins and more integrations.
Sooo, in the end, I made it compatible with both =D
Now you can publish to jCenter, and the script still works with Jitpack, any problem releasing your SNAPSHOT through Jitpack instead of another repository service?
To release to jCenter, just configure a maven.properties using the sample-maven.properties and your settings, I did not put it in local.properties, because actually I don't like mixing configurations, I can change it if you like.
You can use one of the publishConfigurations provided to upload to jCenter, there are 2, one for Core and one for SQLite module. I did not made a single command to publish all, maybe later.
What do you think?

Hi,
It looks good to me. I am going to take a few days to review this. Thanks.

Good job! Thank you.


Unable to access Sails when explicitHost or Host is configured as servers FQDN

Sails version: v0.12.13
Node version: v6.10.0
NPM version: 3.10.10
Operating system: CentOS 7.3

Seems like a duplicate. Please delete this. Thank you.


Question: Would like example of how to start normal collection & log a custom metric

I'm not clear how I initiate the SDK for automatic performance collection, and also log custom metrics
I need to pass in the ikey twice?, once for .setup() & once for .getClient()?
appInsights.setup(ikey).setAutoCollectRequests(false).start()
aiclient = appInsights.getClient(ikey)
Also, with this in the code, my program never terminates (its event triggered), is there a .stop() ?

No, you don't need to pass it twice. setup calls getClient internally and initializes it with a default set of parameters (e.g. enable request auto-collection). Something like this will work:
var appInsights = require("applicationinsights");

appInsights.setup("<your-ikey>").start();

appInsights.client.trackException(new Error("exception"));
appInsights.client.trackEvent("test");
appInsights.client.trackTrace("test");

The SDK doesn't support stop(). You can turn off specific auto-collectors (for instance setAutoCollectRequests(false)), but you don't have to stop the whole SDK.


ResetPW Command does not work

I'm using 0.16.5 Flexible Login with the case sensitive name checking turned off and I cannot reset/change passwords, I tried doing it when they are online and offline, it says the password has been changed but when they use the new password it does not log them in.
spongeforge-1.12.2-2611-7.1.0-BETA-2916

Do you have any errors in the logs?

I do not get any errors it just says password has been changed

Do you used the UUID or the name version of the reset command?

I use player names, i.e /flexiblelogin resetpw bob test, when I was using the API 5 /flexiblelogin changepassword it was working fine for me

It works, thanks for the quick fix


Define and implement reducer semantics on heterogenously-typed inputs

Our reducers have ad hoc and sometimes inconsistent behaviors when presented with inputs of different types. A few manifestations of this are below (there are surely others). We should clarify the intended semantics and implement them.

sum of a uint with an int works as expected if the int is positive, but drops it if negative:

$ echo '#1:record[key:string,val:uint32]
1:[key1;1;]
#2:record[key:string,val:int32]
2:[key1;10;]
' | zq -t "sum(val)" -
#0:record[sum:uint64]
0:[11;]
$ echo '#1:record[key:string,val:uint32]
1:[key1;1;]
#2:record[key:string,val:int32]
2:[key1;-10;]
' | zq -t "sum(val)" -
#0:record[sum:uint64]
0:[1;]

sum is not commutative

$ echo '#1:record[key:string,val:int32]
1:[key1;-10;]
#2:record[key:string,val:uint32]
2:[key1;1;]
' | zq -t "sum(val)" -
#0:record[sum:int64]
0:[-9;]
$ echo '#1:record[key:string,val:uint32]
1:[key1;1;]
#2:record[key:string,val:int32]
2:[key1;-10;]
' | zq -t "sum(val)" -
#0:record[sum:uint64]
0:[1;]

inconsistency in whether null vs unset is returned on empty inputs

$ echo '#1:record[key:string]
1:[key1;]' | zq -t "sum(val)" -
#0:record[sum:null]
0:[-;]
$ echo '#1:record[key:string]
1:[key1;]' | zq -t "avg(val)" -
#0:record[avg:float64]
0:[-;]

We discussed the items above offline as a group.
There was consensus that items 1 & 2 will definitely be perceived as buggy by end users and should be addressed. That said, the current behaviors are a reflection of the underlying implementation of expressions and it would be difficult to change the behavior without a significant overhaul of that implementation. Therefore this issue has been made part of Epic #861 such that when we overhaul the implementation we'll make sure we can cover these.
On item 3, I spent some time thinking about it and made the case that I prefer the behavior exhibited by sum() because it provides more context to downstream processors in the flowgraph. Consider:
$ cat /tmp/my.tzng
#0:record[a:int64]
0:[-;]

$ zq -t "sum(a)" /tmp/my.tzng
#0:record[sum:int64]
0:[-;]

$ zq -t "sum(b)" /tmp/my.tzng
#0:record[sum:null]
0:[-;]

That is, a field might be defined in the schema (and hence have a valid data type we could attach to the aggregation's result, even if all input values are unset), or a field might be completely absent from the schema. The null in this case communicates the latter differently than the former, which might be helpful for debugging, e.g. to help someone realize they might have made a typo in the field name or didn't know the schema for their data as well as they thought they did. The team seemed tentatively on-board with this argument, but @henridf made the valid point that we've probably not thought through all the other side-effects in downstream processors/reducers of having fields with null data types, so this probably requires more thought.

I'm reviewing this one as part of an issue scrub and can confirm that most of the problems highlighted have been addressed as of current Zed commit 08fc271. However, the TZNG format used for the input data is ancient history, so I went back and used Zed v0.32.0 to turn those into ZSON. Here's repeats of the repro steps using that as input data.
$ zq -version
Version: v1.9.0-10-g08fc271c

sum() of uint with int now works fine when the int is negative

$ cat 1a.zson
{key:"key1",val:1(uint32)}
{key:"key1",val:10(int32)}

$ zq "sum(val)" 1a.zson
11

$ cat 1b.zson
{key:"key1",val:1(uint32)}
{key:"key1",val:-10(int32)}

$ zq "sum(val)" 1b.zson
-9

sum() is now commutative

$ cat 2a.zson
{key:"key1",val:-10(int32)}
{key:"key1",val:1(uint32)}

$ zq "sum(val)" 2a.zson
-9

$ cat 2b.zson
{key:"key1",val:1(uint32)}
{key:"key1",val:-10(int32)}

$ zq "sum(val)" 2b.zson
-9

There's still some inconsistency between sum() and avg(), though...

$ cat 3.zson
{key:"key1"}

$ zq "sum(val)" 3.zson
null

$ zq "avg(val)" 3.zson
null(float64)

That is, the user-facing docs for sum() and avg() show them both returning number which is defined in Type Conventions as "either float or int". Based on this, the way avg() seems to pick a particular numeric type for its null value is perhaps defensible, though FWIW the other aggregate functions that return number such as min() and max() happen to behave the same as sum().
$ zq "min(val)" 3.zson
null

$ zq "max(val)" 3.zson
null

I'll float this topic with the team and see if there's interest in aligning the behaviors.


[InstagramBridge] Fix bridge

Commented out any code that don't work anymore.
Fix user, location (hashtag still broken :( )
Add support for using Bibliogram to handle image/video.
Add CACHE_TIMEOUT to 15 minutes.

Close #2168

Doesn't show all pictures in feed:
https://dhusch.de/rss-bridge/?action=display&bridge=Instagram&context=Username&u=dhuschde&media_type=all&format=Html

Doesn't show all pictures in feed:
https://dhusch.de/rss-bridge/?action=display&bridge=Instagram&context=Username&u=dhuschde&media_type=all&format=Html

You can use const CACHE_TIMEOUT = 10; to do that

@csisoap why did you close PR?

@csisoap why did you close PR?

Sorry about that, i'm so silly so close this. Reopen #2168 and making small change.

It stopped working on my Bridge

Ok, it is working again.

This bridge worked for me on the first try. However, after a few requests with debug on, it's not working. Instagram is blocking too many requests. They want us to login.

It is working for me with no problem

It is working for me with no problem

can you help me it's not working for me { instagram bridge }

Hi, @csisoap!
Haven't found time to review it yet, but FYI there is a plan to implement own version of media proxy or alternative. Related issue https://github.com/RSS-Bridge/rss-bridge/issues/2365

Hi, @csisoap! Haven't found time to review it yet, but FYI there is a plan to implement own version of media proxy or alternative. Related issue #2365

ok thanks

Well this uses https://bibliogram.art so it's basically a proxy. Should maybe be in another bridge or as a seprate checkbox option?

Bibliogram produces feeds too. This is redundant. If implemented this should be in a separate bridge.


Missing prequisite package 'openssl'

Even with 'openssl-devel' installed, I recieved an error while trying to run 'make e2e-tests". This was solved by installing 'openssl'

@eldar101 all commits must be signed then we can merge :pray:


🐛 [Bug] Compilation causes error: ValueError: stoi

Bug Description
Compiling the below graph causes the following error:
ValueError: stoi

It's not clear to me what the root cause is yet; I've seen this appear in several of our models internally, but this is the first time I've been able to isolate the error outside of our repo.
To Reproduce
  import torch
  import torch_tensorrt as torchtrt
  import torch_tensorrt.logging as logging

  logging.set_reportable_log_level(logging.Level.Warning)

  torch.manual_seed(0)

  DEVICE = torch.device("cuda:0")
  SHAPE = (1, 2)

  class Model(torch.nn.Module):
      def __init__(self):
          super().__init__()

      def forward(self, a):
          y = torch.squeeze(a, -1)
          return y - y

  if __name__ == "__main__":
      tensor = torch.randn(SHAPE, dtype=torch.float32, device=DEVICE)

      model = Model().eval().to(DEVICE)
      out = model(tensor)
      print(f"Model: {out}")

      model_trt = torchtrt.compile(
          model,
          inputs=[
              torchtrt.Input(shape=SHAPE),
          ],
          enabled_precisions={torch.float},
      )
      out_trt = model(tensor)
      print(f"Model TRT: {out_trt}")

      assert torch.max(torch.abs(out - out_trt)) < 1e-6

Outputs the following:
root@1ccd9cb0f739:/workspace# python /scripts/squeeze.py
Model: tensor([[0., 0.]], device='cuda:0')
WARNING: [Torch-TensorRT] - Cannot infer input type from calcuations in graph for input a.1. Assuming it is Float32. If not, specify input type explicity
Traceback (most recent call last):
  File "/scripts/squeeze.py", line 31, in <module>
    model_trt = torchtrt.compile(
  File "/usr/local/lib/python3.8/dist-packages/torch_tensorrt/_compile.py", line 115, in compile
    return torch_tensorrt.ts.compile(ts_mod, inputs=inputs, enabled_precisions=enabled_precisions, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/torch_tensorrt/ts/_compiler.py", line 116, in compile
    compiled_cpp_mod = _C.compile_graph(module._c, _parse_compile_spec(spec))
ValueError: stoi

Expected behavior
Compilation should not error out, and should output the same values as in the non-TRT version.
Environment
Ubuntu 18.04 x86-64
Built from source:

Master (11bcb98d3cd680c3c34e6cc4c4efdc7512c144cc) using TRT NGC 22.02-py3
v1.0 using TRT NGC  21.11-py3

Error appears in both.
Additional context
In this particular case, it looks like we can fix the error by adding a third dimension to the tensor so that it is squeezable, ie.
SHAPE = (1, 2, 1)

Seeing the same issue with the following script using the aforementioned containers/builds:
  import torch
  from torch import nn
  from torch.nn import functional as F
  import torch_tensorrt as torchtrt
  import torch_tensorrt.logging as logging

  logging.set_reportable_log_level(logging.Level.Warning)

  torch.manual_seed(0)

  DEVICE = torch.device("cuda:0")
  SHAPE = (128, 45)

  class Model(torch.nn.Module):
      def __init__(self):
          super().__init__()

          self.size = 9

      def forward(self, x):
          tril = torch.zeros(128, self.size, self.size).cuda()
          indices = torch.tril_indices(self.size, self.size).cuda()
          tril[:, indices[0], indices[1]] = x

          EPS = 1e-5
          tril_clone = tril.clone()
          for i in range(self.size):
              tril[:, i, i] = EPS + F.softplus(tril_clone[:, i, i])

          return tril

  if __name__ == "__main__":
      tensor = torch.randn(SHAPE, dtype=torch.float32, device=DEVICE)

      model = Model().eval().to(DEVICE)
      out = model(tensor)
      print(f"Model: {out}")

      model_trt = torchtrt.compile(
          model,
          inputs=[
              torchtrt.Input(shape=SHAPE),
          ],
          enabled_precisions={torch.float},
      )
      out_trt = model(tensor)
      print(f"Model TRT: {out_trt}")

      assert torch.max(torch.abs(out - out_trt)) < 1e-6

Traceback (most recent call last):
  File "/scripts/tril_indices.py", line 49, in <module>
    model_trt = torchtrt.compile(
  File "/usr/local/lib/python3.8/dist-packages/torch_tensorrt/_compile.py", line 97, in compile
    return torch_tensorrt.ts.compile(ts_mod, inputs=inputs, enabled_precisions=enabled_precisions, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/torch_tensorrt/ts/_compiler.py", line 119, in compile
    compiled_cpp_mod = _C.compile_graph(module._c, _parse_compile_spec(spec))
ValueError: stoi

Closing this as the immediate issue is solved but #982 will remain open as there are deeper issues hit when using partitioning

Confirmed fixed on 1.2


Error in protobuf while invoking main.py

Getting this error while using the code:
Traceback (most recent call last):
File "C:\ripper\main.py", line 22, in
from utils import extract_kid
File "C:\ripper\utils.py", line 3, in
import widevine_pssh_pb2
File "C:\ripper\widevine_pssh_pb2.py", line 33, in
_descriptor.EnumValueDescriptor(
File "C:\Python39\lib\site-packages\google\protobuf\descriptor.py", line 755, in new
_message.Message._CheckCalledFromGeneratedFile()
TypeError: Descriptors cannot not be created directly.
If this call came from a _pb2.py file, your generated code is out of date and must be regenerated with protoc >= 3.19.0.
If you cannot immediately regenerate your protos, some other possible workarounds are:

Downgrade the protobuf package to 3.20.x or lower.
Set PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=python (but this will use pure-Python parsing and will be much slower).

More information: https://developers.google.com/protocol-buffers/docs/news/2022-05-06#python-updates
Not sure what changed.

Python version is 3.9.4, if that makes any difference.

Version 4 of protobuf has just been riddled with problems
pip uninstall protobuf
pip install protobuf==3.20

Just did that and it worked. Thank you! Closing the Issue.


Reconnect to wifi

I have an issue about wifi. Indeed, after an firmware update of my router I discovered that the ESP32 didn't reconnect. I have to reboot it to see it again on the network.
I made additionals test, each time I reset my wifi or reboot my router, the ESP32 didn't reconnect.
I have a lot of devices at home (with some esp) and this is the only one with this behaviour.
How is implemented the strategy when the device lost the wifi connexion ?

Well, this one is interesting. I thought I fixed automatic WiFi reconnect in https://github.com/tspopp/AquaMQTT/commit/b21f5325526c0ba40fecddf4887106e9fb7e361a
Back then I've found several examples, which suggested doing it this way.  Sine I've never observed any issues, I thought it just works 🤷♂ Will look into this, as soon I have time.

I tried to recreate the issue yesterday: I've been playing around with a WiFi hotspot created using my Android phone. The esp is always reconnecting in case WiFi hotspot is disabled and reenabled.
Maybe it is due to the network topology or router, but it is strange, since the auto reconnect behavior seems to be the default arduino esp32 behavior anyway 🤷♂ What kind of router are you using?
Do you use the latest platform framework libraries? Maybe a pio pkg update before building helps? If there has been an issue in the arduino framework, a pkg update might resolve it...

Hello @tspopp ,
Thanks for your help.
On my side, I made additionals tests.
If I restart WiFi on my router, it succeeds to reconnect. If I reboot my router, it fails to reconnect. I see nothing on router logs.
For the difference between both WiFi restarting is the time without WiFi, as it takes 20s for WiFi restart and more than 1min for the router reboot.
I have reflashed the esp after an pio pkg update but it changes nothing.

Well, unfortunately, I can confirm the behaviour. Today, I needed to reconfigure my Ubiquity stuff and this caused a WiFi downtime of about 30s. Every device reconnected, even plenty of ESP32s (WLED, e.g.) but the AquaMQTT didn't.
Although, my console told me, it was still connected but didn't receive an IP.
According to https://esp32.com/viewtopic.php?f=19&t=39116 it could be a problem that the WiFi.setAutoReconnect function doesn't work if the station gets "properly" disconnected (AP sending a disconnect message). It only seems to work if the loss of WiFi appears suddenly and uncontrolled.
One suggests that you could actively monitor the WiFi events and act accordingly. Maybe by introducing a loop or even doing a complete reboot after x sec of no connectivity?

Hm, I hoped they address such issues within the official esp32 framework libraries and I don't need an individual hack in a loop to check for WiFi connectivity. Reboot is not an option, we would loose any configuration previously made through MQTT since previous configuration is currently not stored/restored.
I will try re-creating this issue as soon as I have some spare time. I really want to make sure the fix will address the issue and does not cause any other side effects..

I was not able to reproduce the issue, but I basically followed the advice do not trust the auto reconnect routine, just as proposed by @kopierschnitte. Therefore I added something customized which is basically sequentially checking for the WiFi connection and attempting a hard reconnect if something is wrong. I've tested this using an Android Hotspot and it was always reconnecting fine. I hope this will finally address this issue.
I will close this issue as soon as I've merged the PR, but you may feel free to reopen it in case it re-occurs.

Thanks, I have updated the Arduino with the new code, I will inform you if the problem is solved.

Hello,
After a reboot of the rooter, the arduino is still alive on the network. So, it seems solved.
Thanks a lot and have a nice day.


Fix memoryview memory leak

See https://github.com/cython/cython/issues/1638

Sorry if I'm a suspicious about this fix, but PyBuffer_Release() in CPython does this:
void PyBuffer_Release(Py_buffer *view)
{
    PyObject *obj = view->obj;
    PyBufferProcs *pb;
    if (obj == NULL)
        return;
    pb = Py_TYPE(obj)->tp_as_buffer;
    if (pb && pb->bf_releasebuffer)
        pb->bf_releasebuffer(obj, view);
    view->obj = NULL;
    Py_DECREF(obj);
}

Note the Py_DECREF(obj) at the end. I'd like to avoid replacing a memory leak with a reference leak here, even if it's only about None.

But thanks for pointing me at the right spot to keep searching. I'll commit a fix shortly.

I'm not sure PyBuffer_Release gets called, though

In Py3, yes. In Py2, Cython has its own implementation, which is where the bug was actually hiding.


Deprecate and remove useless matchers

Next matchers should be deprecated and removed in upcoming versions of mongoid-rspec:

be_paranoid_document, because of removal Mongoid::MultiParameterAttributes since mongoid version 4.0
be_multiparameted_document because of removal Mongoid::Paranoia since mongoid version 4.0
be_versioned_document because of removal Mongoid::Versioning out of mongoid
have_instance_method since it has nothing related to features of mongoid

It's makes sense. Thanks. :+1:


[Bottom Navigation Bar] Always masks content above it, even when FAB is not applyed to it

Current behavior
When Layout includes a BottomTabBar a 28px height area above it spanning the entire width of the page is masked. Items in this area or scrolling through the area disappear behind this empty masking area.

Expected behavior
items should be visible right up to the edge of the component as designed

How to reproduce it (as minimally and precisely as possible)
In Figma place component in layout with an adjacent item above it
Run the Uno plugin
Observe the preview tab
Environment

Nuget Package:
Package Version(s):
Affected platform(s):

[ ] iOS
[ ] Android
[ ] WebAssembly
[ ] WebAssembly renders for Xamarin.Forms
[ ] Windows
[ ] Build tasks

Visual Studio:

[ ] 2017 (version: )
[ ] 2019 (version: )
[ ] for Mac (version: )

Relevant plugins:

[ ] Resharper (version: )

Anything else we need to know?
We knew this was happening when a FAB TabBarItem is included in the BottomTabBar, but since it actually happening in all cases involving the BottomTabBar (aka Bottom Navigation) I'm creating a issue to bump this up the priority list.

Caused by  https://github.com/unoplatform/uno/issues/7393


Fix two small typos in UserCommands.cs

Hello,
I was reading through for examples of MutableTraceEventStackSource used in anger and spotted some small typos, I thought that I might as well submit a PR to fix it whilst I was here, thanks for the great project! :)
Thanks,
Michael Radigan

Thanks @michaelRadigan!


Please Can Put rounded Corners

Rounded Corners Please

I concur, please add rounded corners : ) love the Application though


Make Licenser plugin work with Gradle 7.1-rc-2

Gradle 7.1-rc-1 introduced some changes which caused project.extensions.create('license', LicenseExtension, project.objects, project) to return DefaultBasePluginExtension (introduced in Gradle 7.1) instead of LicenseExtension which would lead to an exception when trying to access a property of LicenseExtension on the returned instance.
The issue was traced back to using project.with { ... } and not properly resolving the instance variables and field accessors inside that block.
By moving everything out of the project.with { ... } block, the field accessor for Licenser.extension is properly resolved.

The diff is best viewed when ignoring whitespace changes. 😉
https://github.com/CadixDev/licenser/pull/30/files?diff=split&w=1

This seems more like a Gradle bug?
Happy to merge if Gradle are going to make breaking changes mid-version though :)
As always, thanks for contributing!

This seems more like a Gradle bug?

That was my first impulse too. A Gradle engineer then explained that it's an unfortunate interaction between dynamic Groovy (and the scope of closures like project.with { ... }) and the code in Licenser. 🤷

Aha, that makes sense. Will merge this shortly :)

I hope it's okay to give this a little bump as gradle 7.1 was released yesterday and some of my projects are failing to compile :/

I've bumped the Gradle version in the functional tests to Gradle 7.1.


[HOLD for payment 2024-06-28] [HOLD for payment 2024-06-24] [$250] Categorizing - New workspace button does not create new WS if previous WS is selected first

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.75-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:
Precondition:

There is a workspace with different categories than the default categories (can be obtained by disabling a few or adding new ones, for the purpose of differentiating).

Go to staging.new.expensify.com
Go to FAB > Track expense
Track a manual expense
In 1:1 DM, click Categorize it from the whisper
Click on the workspace from precondition
Select any category
Click back button twice to return to workspace selection page
Click New workspace button

Expected Result:
The list of default categories will appear because the workspace is new
Actual Result:
Categories from the previously selected workspace appear instead of new set of default categories from the new workspace.
After selecting a category, the workspace chat in the confirmation page is the previous workspace instead of the new workspace when New workspace button is selected
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/e111df8b-9e99-4e6c-8f55-19adae6cadff
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~018fccdf6165f3ba87
Upwork Job ID:<PHONE_NUMBER>708002304
Last Price Increase: 2024-06-04
Automatic offers:

tienifr | Contributor | 102686152

Issue OwnerCurrent Issue Owner: @kevinksullivan

@kevinksullivan  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to  #wave-collect - Release 1

VSB bug

I agree with proposal from @tienifr to reset participants and numberOfParticipants when we go back to IOURequestStepParticipants.
🎀 👀 🎀 C+ reviewed!

[@mollfpr] The PR that introduced the bug has been identified. Link to the PR:
[@mollfpr] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:

No offending PR. This issue seems like an edge case.

[@mollfpr] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

The regression step should be enough.

[@mollfpr] Determine if we should create a regression test for this bug.
[@mollfpr] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Precondition:
There is a workspace with different categories than the default categories (can be obtained by disabling a few or adding new ones, for the purpose of differentiating).

Open ND app
Go to FAB > Track expense
Track a manual expense
In 1:1 DM, click Categorize it from the whisper
Click on the workspace from the precondition
Select any category
Click the back button twice to return to the workspace selection page
Click the New workspace button
Verify The list of default categories will appear because the workspace is a new
👍 or 👎

All set

$250 approved for @mollfpr


BUG: UI improvement for Price Display in Today's Special

Is there an existing issue for this?

[X] I have searched the existing issues

What went down?
There can be UI improvement for the pricing display of the Today's Sepecial
issue:

no bottom padding
Original Price's text color not so subtle
Original Price's text size could be smaller bringing the Current Price more in focus

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a gssoc-24-ext contributor
[X] I have starred the repository
[X] I want to work on this issue
[X] I check the issue doesn't exist

what you are going to add any example @vaibhav-yerkar

These are my proposed designs :

if you find it appealing , let me know
and also mention whether you want the text alignment to be centred or the left aligned

@vaibhav-yerkar go for 1st design ✔


Update play-json to 2.9.1

Updates com.typesafe.play:play-json from 2.7.4 to 2.9.1.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.typesafe.play", artifactId = "play-json" } ]

labels: library-update, semver-minor

Superseded by #10.

Superseded by #10.


Ember CLI support

We should consider rewriting this plugin as en ember-cli addon and publishing it to http://www.emberaddons.com/.

https://github.com/ivanvotti/ember-cli-pusher
https://github.com/ivanvotti/ember-pusher-component

@ivanvotti Awesome! we'll check it out.

Any update on that?

I guess that's it: https://github.com/ivanvotti/ember-cli-pusher