Unable to communicate with router SSL error

Hi!
First seems like a great intergration! Would like to test this. But not able to complete the config. Did al the steps but getting a SSL error.
Logger: custom_components.openwrt.ubus
Source: custom_components/openwrt/ubus.py:92
Integration: OpenWrt (documentation, issues)
First occurred: 16:10:27 (2 occurrences)
Last logged: 16:10:47
api_call exception: HTTPSConnectionPool(host='<IP_ADDRESS>', port=443): Max retries exceeded with url: /ubus (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate (_ssl.c:1129)')))
Logger: custom_components.openwrt.coordinator
Source: custom_components/openwrt/ubus.py:93
Integration: OpenWrt (documentation, issues)
First occurred: 16:10:27 (2 occurrences)
Last logged: 16:10:47
Device [Test] async_update_data error:
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 703, in urlopen
httplib_response = self._make_request(
File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 386, in _make_request
self._validate_conn(conn)
File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 1040, in validate_conn
conn.connect()
File "/usr/local/lib/python3.9/site-packages/urllib3/connection.py", line 414, in connect
self.sock = ssl_wrap_socket(
File "/usr/local/lib/python3.9/site-packages/urllib3/util/ssl.py", line 453, in ssl_wrap_socket
ssl_sock = ssl_wrap_socket_impl(sock, context, tls_in_tls)
File "/usr/local/lib/python3.9/site-packages/urllib3/util/ssl.py", line 495, in _ssl_wrap_socket_impl
return ssl_context.wrap_socket(sock)
File "/usr/local/lib/python3.9/ssl.py", line 500, in wrap_socket
return self.sslsocket_class._create(
File "/usr/local/lib/python3.9/ssl.py", line 1040, in _create
self.do_handshake()
File "/usr/local/lib/python3.9/ssl.py", line 1309, in do_handshake
self._sslobj.do_handshake()
ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate (_ssl.c:1129)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/requests/adapters.py", line 440, in send
resp = conn.urlopen(
File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 785, in urlopen
retries = retries.increment(
File "/usr/local/lib/python3.9/site-packages/urllib3/util/retry.py", line 592, in increment
raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='<IP_ADDRESS>', port=443): Max retries exceeded with url: /ubus (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate (_ssl.c:1129)')))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/config/custom_components/openwrt/ubus.py", line 90, in _api_call
response = await self.executor_job(post)
File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
File "/config/custom_components/openwrt/ubus.py", line 84, in post
return requests.post(
File "/usr/local/lib/python3.9/site-packages/requests/api.py", line 117, in post
return request('post', url, data=data, json=json, **kwargs)
File "/usr/local/lib/python3.9/site-packages/requests/api.py", line 61, in request
return session.request(method=method, url=url, **kwargs)
File "/usr/local/lib/python3.9/site-packages/requests/sessions.py", line 529, in request
resp = self.send(prep, **send_kwargs)
File "/usr/local/lib/python3.9/site-packages/requests/sessions.py", line 645, in send
r = adapter.send(request, **kwargs)
File "/usr/local/lib/python3.9/site-packages/requests/adapters.py", line 517, in send
raise SSLError(e, request=request)
requests.exceptions.SSLError: HTTPSConnectionPool(host='<IP_ADDRESS>', port=443): Max retries exceeded with url: /ubus (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate (_ssl.c:1129)')))
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/config/custom_components/openwrt/coordinator.py", line 243, in async_update_data
self._apis = await self.load_ubus()
File "/config/custom_components/openwrt/coordinator.py", line 232, in load_ubus
return await self._ubus.api_call("*", None, None, "list")
File "/config/custom_components/openwrt/ubus.py", line 44, in api_call
await self._login()
File "/config/custom_components/openwrt/ubus.py", line 48, in _login
result = await self._api_call(
File "/config/custom_components/openwrt/ubus.py", line 93, in _api_call
raise ConnectionError from err
ConnectionError
Logger: homeassistant.config_entries
Source: config_entries.py:370
First occurred: 16:10:47 (1 occurrences)
Last logged: 16:10:47
Config entry 'Test - <IP_ADDRESS>' for openwrt integration not ready yet: OpenWrt communication error: ; Retrying in background

I don't use any certificate, no HTTPS. What am i doing wrong? Can u help :)?

Hi,
I tried both, checkbox on/off same error
Got 4 TP Links Archer C7's v2 running on 21.02.3
Also tried root user for login. Same error

Please refer to https://www.home-assistant.io/integrations/logger/ on how to enable logging

Added to configuration.yaml
Restart HA
Checked

Same thing, nothing more then what i send before. Did i miss something?

Unable to get the right information out of my system. Closing this case.

I'm getting the same error.
api_call exception: HTTPSConnectionPool(host='<IP_ADDRESS>', port=443): Max retries exceeded with url: /ubus (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate (_ssl.c:997)')))

I use a self-signed certificate on my OpenWRT router, which will automatically upgrade any non-HTTP requests to HTTPS. I get the error whether or not I check Use HTTPS.
 Device [openwrt] async_update_data error:

Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 703, in urlopen
    httplib_response = self._make_request(
  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 386, in _make_request
    self._validate_conn(conn)
  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 1042, in _validate_conn
    conn.connect()
  File "/usr/local/lib/python3.10/site-packages/urllib3/connection.py", line 414, in connect
    self.sock = ssl_wrap_socket(
  File "/usr/local/lib/python3.10/site-packages/urllib3/util/ssl_.py", line 453, in ssl_wrap_socket
    ssl_sock = _ssl_wrap_socket_impl(sock, context, tls_in_tls)
  File "/usr/local/lib/python3.10/site-packages/urllib3/util/ssl_.py", line 495, in _ssl_wrap_socket_impl
    return ssl_context.wrap_socket(sock)
  File "/usr/local/lib/python3.10/ssl.py", line 513, in wrap_socket
    return self.sslsocket_class._create(
  File "/usr/local/lib/python3.10/ssl.py", line 1071, in _create
    self.do_handshake()
  File "/usr/local/lib/python3.10/ssl.py", line 1342, in do_handshake
    self._sslobj.do_handshake()
ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate (_ssl.c:997)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/requests/adapters.py", line 489, in send
    resp = conn.urlopen(
  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 787, in urlopen
    retries = retries.increment(
  File "/usr/local/lib/python3.10/site-packages/urllib3/util/retry.py", line 592, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='<IP_ADDRESS>', port=443): Max retries exceeded with url: /ubus (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate (_ssl.c:997)')))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/config/custom_components/openwrt/ubus.py", line 90, in _api_call
    response = await self.executor_job(post)
  File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/config/custom_components/openwrt/ubus.py", line 84, in post
    return requests.post(
  File "/usr/local/lib/python3.10/site-packages/requests/api.py", line 115, in post
    return request("post", url, data=data, json=json, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/requests/api.py", line 59, in request
    return session.request(method=method, url=url, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/requests/sessions.py", line 587, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/lib/python3.10/site-packages/requests/sessions.py", line 723, in send
    history = [resp for resp in gen]
  File "/usr/local/lib/python3.10/site-packages/requests/sessions.py", line 723, in <listcomp>
    history = [resp for resp in gen]
  File "/usr/local/lib/python3.10/site-packages/requests/sessions.py", line 266, in resolve_redirects
    resp = self.send(
  File "/usr/local/lib/python3.10/site-packages/requests/sessions.py", line 701, in send
    r = adapter.send(request, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/requests/adapters.py", line 563, in send
    raise SSLError(e, request=request)
requests.exceptions.SSLError: HTTPSConnectionPool(host='<IP_ADDRESS>', port=443): Max retries exceeded with url: /ubus (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate (_ssl.c:997)')))

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/config/custom_components/openwrt/coordinator.py", line 243, in async_update_data
    self._apis = await self.load_ubus()
  File "/config/custom_components/openwrt/coordinator.py", line 232, in load_ubus
    return await self._ubus.api_call("*", None, None, "list")
  File "/config/custom_components/openwrt/ubus.py", line 44, in api_call
    await self._login()
  File "/config/custom_components/openwrt/ubus.py", line 48, in _login
    result = await self._api_call(
  File "/config/custom_components/openwrt/ubus.py", line 93, in _api_call
    raise ConnectionError from err
ConnectionError

And new_coordinator debug lines from home_assistant.log:
2022-07-28 09:34:23 DEBUG (MainThread) [custom_components.openwrt.coordinator] new_coordinator: {'id': 'openwrt', 'address': '<IP_ADDRESS>', 'username': 'hass', 'https': False, 'port': 0, 'path': '/ubus', 'wps': False, 'password': 'xxxxxxx'}
2022-07-28 09:34:33 DEBUG (MainThread) [custom_components.openwrt.coordinator] new_coordinator: {'id': 'openwrt', 'address': '<IP_ADDRESS>', 'username': 'hass', 'https': False, 'port': 0, 'path': '/ubus', 'wps': False, 'password': 'xxxxxxx'}
2022-07-28 09:34:44 DEBUG (MainThread) [custom_components.openwrt.coordinator] new_coordinator: {'id': 'openwrt', 'address': '<IP_ADDRESS>', 'username': 'hass', 'https': False, 'port': 0, 'path': '/ubus', 'wps': False, 'password': 'xxxxxxx'}
2022-07-28 09:34:44 DEBUG (MainThread) [custom_components.openwrt.ubus] New call [http://<IP_ADDRESS>/ubus] {"jsonrpc": "2.0", "id": 1, "method": "call", "params": ["00000000000000000000000000000000", "session", "login", {"username": "hass", "password": "xxxxxxx"}]}
2022-07-28 09:35:45 DEBUG (MainThread) [custom_components.openwrt.coordinator] new_coordinator: {'id': 'openwrt', 'address': '<IP_ADDRESS>', 'username': 'hass', 'https': False, 'port': 0, 'path': '/ubus', 'wps': False, 'password': 'xxxxxxx'}
2022-07-28 09:35:45 DEBUG (MainThread) [custom_components.openwrt.ubus] New call [http://<IP_ADDRESS>/ubus] {"jsonrpc": "2.0", "id": 1, "method": "call", "params": ["00000000000000000000000000000000", "session", "login", {"username": "hass", "password": "xxxxxxx"}]}

FWIW, I just followed the instructions here and here to enable self-signed certs.


How to test integrity check

There is this problem in e2e tests now where I'm not able to write tests for functionalities that don't modify a filesystem. For example: result of integrity check is simply printed to the console. Testing this functionality in unit tests is not perfect either as it needs temp files and folders to be created. Currently e2e tests have many helper functions for this kind of stuff.
Idea 1
The idea is to capture Result from run_program in e2e test. The problem is that every error that I'd have to assert on would have to propagate to the top. I think that integrity check procedure should not fail just because verification failed. So meybe some custom Result type that would differentiate whole procedure failure from procedure result.
Idea 2
Capture whole output from the program in e2e test and regex the output to verify the result. This approach is even more e2e-ish but it's not easy to capture the output of your own program. I'd have to use dependency injection to pass streams to the run_program function and then propagate it everywhere in the program.
Idea 2 sounds great: https://jeffkreeftmeijer.com/rust-stdin-stdout-testing/
I use macros warn!, debug! etc. to print stuff but it's possible to set the Target for logger: https://docs.rs/env_logger/latest/env_logger/struct.Builder.html#method.target and I actually already set target in my e2e tests in init_logger() (create_snapshot.rs). I could try to pass there my own pipe e.g. let mut my_pipe: Vec<u8>.

// create pipe that logger will write to
let log_output = Box::new(Vec::new());
// create env_logger::Target with custom pipe
let logger_target = env_logger::Target::Pipe(log_output);
// prepare env_logger::Builder
let mut builder = env_logger::Builder::new();
// initialize the logger
builder.target(logger_target).try_init().ok();

mizeria::run_program(...)

// convert Vec<u8> to string
// ERROR: log_output is moved...
let str_output = String::from_utf8(*log_output).expect("Found invalid UTF-8");

assert!(str_output.contains("some expected log message");

https://github.com/env-logger-rs/env_logger/blob/main/examples/custom_target.rs

Capturing console output in tests was added in 575965d47eebc691d73d18921ccb4ac8f6523ff0
Using simplelog instead of env_logger should be considered. (#41)


Request to add more content

Hi Team,
Please add potential use cases, alternative services, implementation considerations, or pricing guidance.
Thanks!!
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 14acf440-88ed-186c-a3ab-5309bf7e363c
Version Independent ID: d32e5eb7-5011-dc40-e978-c6bc9f6363a5
Content: DevTest and DevOps for microservice solutions - Azure Solution Ideas
Content Source: docs/solution-ideas/articles/dev-test-microservice.yml
Service: architecture-center
Sub-service: solution-idea
GitHub Login: @EdPrice-MSFT
Microsoft Alias: pnp

thanks for bringing this to our attention. I have assigned the issue to the content author to evaluate and update as appropriate

FYI @mosabami

Hi @Pachehra
Sorry for the delay and thanks for the suggestions.
This current article is a  “Solution idea” which its goals is to keep it simple, here there is more information:
Microservice reference example implementation with AKS
Microservice Architecture
Microservice and dotnet, Architecture and ASP
I will close the ticket, but feel free to re-open if additional clarification is needed.

#please-close


Extension issue

DotJoshJohnson.xml-unresponsive.cpuprofile.txt

Issue Type: Bug
Extension Name: xml
Extension Version: 2.5.0
OS Version: Windows_NT x64 10.0.18363
VSCode version: 1.41.1

:warning: We have written the needed data into your clipboard. Please paste! :warning:

Please give this issue a descriptive title.


Add pipeline action for integration tests

Readiness checklist

[ ] I added/updated unit tests.
[x] I added/updated integration tests.
[x] I ensured that the PR title is good enough for the changelog.
[x] I labeled the PR.

I realised now someone fixed the AdminUi tests. It was a good exercise, too bad it was needless 😭


forge doc --build --root <PATH> --prefix not found error

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (28b2ae6 2023-03-04T00:10:50.272293Z)
What command(s) is the bug in?
forge docs
Operating System
macOS (Intel)
Describe the bug
forge docs subcommand states the support of --root argument.
However, when trying to provide one, the execution fails:

I just hit this running the following command:
forge doc --out ./_site

Interestingly this does not occur when I remove the dot-slash as a workaround:
forge doc --out _site

In our case, we realized the issue is that the root of our forge project is not the git root. We have the forge project in the ./contracts folder. Our workaround for the issue is to add remappings to the root of the git repository, and then run forge doc for the full repo. This is not ideal, since we may have multiple forge projects in the same repo in the future, and would like the ability to build the docs separately.


[SYCL-BLAS] fix SYCL-BLAS backend names in backend table

Description
Fixes config/build for SYCL-BLAS to use correct #define names in backend table
Checklist
All Submissions

[x] Do all unit tests pass locally? Log: out.txt
[x] Have you formatted the code using clang-format?

What was error signature before this PR?

@asrednit said he saw the following:
1: [ RUN      ] Nrm2TestSuite/Nrm2Tests.RealSinglePrecision/Column_Major_Intel_R__Xeon_R__Platinum_8358_CPU___2_60GHz
1: unknown file: Failure
1: C++ exception with description "oneMKL: Couldn't load selected backend" thrown in the test body.

It is a bit strange because I am not sure how this could have ever configured/built properly before but I remember doing it. Maybe @hjabird or @pgorlani have insights - there might be a configuration step we are missing instead of the change to the header file.

Yes, that is exactly my concern. The PR looks good, but it would good to understand how this was missed.


REUSE_DB=1 does not work on travis.ci in 1.4.2

I used django-nose's master branch to run tests, since 1.4.1 doesn't work with django 1.8. And recently master branch stopped working too, django asks whether it should destroy already existent database. It seems like django-nose detects in code that it should create database even if it is already exists.
Unfortunately it is only reproduced on travis.ci, it works fine on my local machine even with 1.4.2. So I had to stick to a specific commit, that was made before 7th October (5ad928ec94c4efeb000d3d846c00d27b250df84f). I suspect there is a problem with closing connections that was added recently, but I am not sure.
Travis environment:
python==2.7.9
django==1.8.5
django-nose==1.4.2
nose==1.3.7
MySQL-python==1.2.5
Database is created by separate script before running tests. So it always created.

In TravisCI, the django-nose test suite with --no-input, so that test databases are recreated without prompts.
Is this a public project?  Can you link to the repository and the TravisCI results?

@jwhitlock No, it is private repo.
I tried to use --noinput, but it does not work either:
Creating test database for alias 'default'...
Got an error creating the test database: (1007, "Can't create database 'test_main'; database exists")
Destroying old test database 'default'...
Got an error recreating the test database: (1217, 'Cannot delete or update a parent row: a foreign key constraint fails')

It tries to drop database anyway.


Update Getting-Started-with-Server-Core.md

updated to remove future tense.

Thanks, @doctordns !


Add configuration option for auditor method name attribute

Adds a new configuration option auditor_name_attr that accepts a Symbol containing the name of the attribute on the auditor class that returns the auditor's name to display in the UI.
Use case: our User record does not have a name field. We could define a name method on our User model class, but having to define a custom method on a class in our app for the sake of a third-party gem is a bit annoying. Instead, with this change, we can do:
config.audits1984.auditor_name_attr = :email

I've tested this change in our app and verified that it works as expected.
cc @jorgemanrubia

Hi @olivier-thatch,
Thanks for the PR. While it brings flexibility, I find the proc to extract the name a bit off. What about an option to configure the auditor_name_attribute that defaults to current name. Instead of reading #name the could would do public_send(Audits1984.auditor_name_attribute)

@jorgemanrubia Updated the PR as requested. (Your suggestion was actually my first approach for this PR :) I thought some folks might appreciate the extra flexibility, but I agree it's probably not needed.)

Thank you!

No rush, but if you could release new versions of console1984 and audits1984 with the changes I submitted, it would be greatly appreciated too :)

Hey I already did after merging the PRs! Latest versions should include the last changes.

🚀  Oops! Sorry I didn't check before asking 🤦  Thanks!


Exclude more non-functional lines from coverage

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (88e89ae) 88.71% compared to head (f6e1577) 78.50%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #360       +/-   ##
===========================================
- Coverage   88.71%   78.50%   -10.21%
===========================================
  Files          30       30
  Lines        1489     1475       -14
===========================================
- Hits         1321     1158      -163
- Misses        168      317      +149

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


Add performance section regarding a span processor to the readme

Adding a new section to the readme about the span processor - SimpleSpanProcessor vs BatchSpanProcessor and its impact on the collector performance

@oded-dd I updated the content a little bit so please re-review


Does not accept longer callsigns.

One of the recommendations here is that digipeaters should use location names for their callsigns (with their FCC issued callsigns in the comment).
When I do this with the callsign of PEPPRPIKE (which is valid), I get this:
parse_ax25addr('PEPPRPIKE') failed. [1]
Looking at ax25.c, it looks like it's hardcoded to expect a callsign of up to 6 characters, then a dash and SSID.  So when it doesn't see the dash (or a * or null), it bombs and returns 1.
The request here is that it should properly support up to 9 characters without an SSID.

AX25 specifies 7 bytes for the source address--6 bytes for the callsign and one byte for the SSID. Valid SSID values are 0-15. Your request is simply not supported by the underlying protocol.
http://www.aprs.org/doc/APRS101.PDF page 12

AX.25 limits callsigns to "3-6 characters - [0-15]" APRS has a looser requirement of 3-9 characters, and no more than two characters for the optional SSID, so yes, PEPPRPIKE is a valid APRS callsign, but not at all a valid AX.25 callsign, and there's no way to work around that. I've used callsigns like W6KWF-TS when testing something that isn't using AX.25 as the transport layer, but there's no way I could use that over VHF AX.25.
I believe APRX will accept APRS but not AX.25 callsigns when it can, but warns you that it's using a callsign it can't use to transmit with. Regardless, if you want your digipeater to be a digipeater, and not an RX-only I-gate, you need to meet the 3-6 character callsign and numeric 0-15 SSID requirement of AX.25.
Does that address your concerns?

Ah okay, that makes sense.  I only reviewed the packet format for APRS
which is 9 characters, but didn't realize that ax25 limitation.  Disregard!
Although it would be nice to have a few extra characters ;)

Dave/KD8TWG

Kenneth Finnegan wrote:

AX.25 limits callsigns to "3-6 characters - [0-15]" APRS has a looser
requirement of 3-9 characters, and no more than two characters for the
optional SSID, so yes, PEPPRPIKE is a valid APRS callsign, but not at
all a valid AX.25 callsign, and there's no way to work around that.
I've used callsigns like W6KWF-TS when testing something that isn't
using AX.25 as the transport layer, but there's no way I could use
that over VHF AX.25.
I believe APRX will accept APRS but not AX.25 callsigns when it can,
but warns you that it's using a callsign it can't use to transmit
with. Regardless, if you want your digipeater to be a digipeater, and
not an RX-only I-gate, you need to meet the 3-6 character callsign and
numeric 0-15 SSID requirement of AX.25.
Does that address your concerns?
—
Reply to this email directly or view it on GitHub
https://github.com/PhirePhly/aprx/issues/18#issuecomment-128165598.

Just curious, where are you guys seeing the 9-character limit for APRS? I've only ever found reference to 9 character callsigns in the APRS-IS spec, not in the APRS spec.

I don't think the APRS "spec" actually spells it out, but since APRS-IS is more permissive than AX.25, APRS systems should be tolerant of the 9 character limitation, so it should be in the APRS spec.

I may have been looking at the object packet format.
Tom Hayward wrote:

Just curious, where are you guys seeing the 9-character limit for
APRS? I've only ever found reference to 9 character callsigns in the
APRS-IS spec, not in the APRS spec.
—
Reply to this email directly or view it on GitHub
https://github.com/PhirePhly/aprx/issues/18#issuecomment-128174080.

APRS-IS definitely allows nine character callsigns, so you were right to think you should have been able to use that, but it wouldn't be able to transmit.

APRS-IS servers will not accept nine character callsigns. They're fine as object/item names, though.


maintaining cmake file lists in seperate files

Maintaining file lists directly in CMakeList.txt is annoying. Splitting them out into separate files which can be loaded and regenerated automatically. Changes to these files trigger a reconfigure thanks to the CMAKE_CONFIGURE_DEPENDS property

Looks like the regen script might need to sort the files to ensure consistency.


How AO and VO model are trained?

Thanks for sharing such a useful repo.
You mention that:
"We train the AO and VO models first. We then initialize the AV model with weights from the trained AO and VO models as follows: AO Audio Encoder → AV Audio Encoder, VO Video Encoder → AV Video Encoder, VO Video Decoder → AV Joint Decoder."
You use AO and VO to initialize AV for further pre-training on LRS2 pre-train set and fine-tuning on LRS-2 train set. But when you pre-train AO and VO before that initialization, do you only use the pre-train set of LRS-2 or do you pre-train on pre-train+train sets or do you pre-train on pre-train and fine-tune on train set before using AO and VO for AV model training? This is particularly important for us as we will use AO and VO model weights you shared and want to know which LRS2 sets they have seen during training.
Thanks!

The AO and VO models are initialized with random weights, then pretrained on the pretrain set and then fine-tuned on the train set. Once training for both models is complete, they are used to initialize the AV model.

Hello, after training separate audio and video models in the "audio_only" and "video_only" folders, I'm trying to use the weights from the audio_only model to initialize the AV model as pre-trained weights. However, I'm experiencing some loss of weights. How can I implement the initialization of the AV Audio Encoder using the AO Audio Encoder?


Token security

I am rewriting the token storage and validation logic in a more secure way.
Fixes # (issue)
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

@anomic30  can you please tell me why the below code is returning null.
var d=await magic.auth.loginWithMagicLink({
        email,
});
console.log(d);

@anomic30  please help me I could not understand why it is returning null.


add icons for the expand all and collapse all links.

Description

expand all icon added (chevron double up icon)
collapse all icon added (chevron double down icon)
icons storybook section included to see all icons available

References #
https://e-builderinc.atlassian.net/browse/NAV-227
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

How Has This Been Tested?

Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have checked my code and corrected any misspellings

References https://github.com/trimble-oss/modus-web-components/issues/1098

@rjavier-trimbler @jkendell89 Could you please create a new GitHub issue?  Modus needs more details for this issue for tracking purposes, and the existing Issue #1098 is a bigger story that might not be appropriate.

@rjavier-trimbler Thank you for fixing other icons like the calendar icon. A couple of comments are pending.

Great PR - thanks for the contribution!

Great PR - thanks for the contribution!


Statistics page height is hard-coded

Steps

Go to https://osf.io/4znzp/statistics/

Expected

Page is of dynamic height and scrolling is only necessary if it is larger than the height of the window

Actual

The widget area is always the same height no matter how many widgets there are/how big the widgets are.

The iframe height is set to a pixel number.
http://stackoverflow.com/questions/9162933/make-iframe-height-dynamic-based-on-content-inside-jquery-javascript
And this is generally the code that can change it:
// It's not good practice to use tag as selector, rather add an id to iframe and use that
$('iframe').load(function() {
  // get height
  var height = this.contentWindow.document.body.offsetHeight;
  // set it with js
 this.style.height = height + 'px';
 // or set it with jquery
 $(this).height(height);
});

@caneruguz ... given approach above, can this be tackled by an intern? TY

@lbanner That's a suggestion only, there may be other options but I think that will work well. Yes an intern can easily handle this. Thanks!

@caneruguz wouldn't that only run when it loads?  Meaning if a user added widgets they would have to refresh the page for it to adjust in height to reflect that change.  Not sure if this would work but I got it from the same stackoverflow link you posted:
<script type="text/javascript">
  function resizeIframe(iframe) {
    iframe.height = iframe.contentWindow.document.body.scrollHeight + "px";
  }
</script>

Then "resizeIframe" could be called when the user adds/deletes/moves/etc. stuff around so that it is always the appropriate size.

@ticklemepierce that code snippet I gave was not meant to be the solution, I don't know how and when it should run since I'm not familiar with the code, the pertinent part was the section about how you can access the content height. You can run it wherever required. Usually you need to take into account window resize as well.
$(window).resize(function(){
// ...
})

and any other action that requies it.

@ticklemepierce You want to tackle this one?

I believe @hmoco is tackling it.

I have been having difficulties getting piwik to work locally so I can test potential changes, so I will be putting this in the backburner for now.

any more luck with piwik?

I believe @GageGaskins and maybe @PatrickEGorman and @zik3qe are currently working on piwik.

It's @GageGaskins --


Python raw string broken

Description
The Python Language's raw string functionality is broken
Steps to Reproduce
print(r"\")

Expected behavior: String stops there because of the r.
Actual behavior: Atom language thinks the string continues, regardless of the r.
Versions
apm  1.9.2
npm  2.13.3
node 0.10.40
python 3.5.1
git 2.8.1.windows.1
visual studio
windows 10

Seems that this issue should be moved to the atom/language-python repo.

Oh
Will do.


Flux source controller sends notifications to wrong NS

An issue exists in the chart for flux which configures the event endpoint to flux-system namespace (see https://github.com/giantswarm/flux-app/blob/1c7d44591db11ba19737c6079c08f2838fcdda05/helm/flux-app/templates/base/deployment-source-controller.yaml#L36)
This causes issues when flux is deployed to a namespace other than flux-system
The event notification address should be configured to use $(RUNTIME_NAMESPACE) instead

Looks like this affects other deployments too - seeing the same in image-automation controller deployment


Update permissions.md

 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.

thank you, please can you sign the cla


Update aws-java-sdk-secretsmanager to 1.12.723

About this PR
📦 Updates com.amazonaws:aws-java-sdk-secretsmanager from 1.12.720 to 1.12.723
Usage
✅ Please merge!
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!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-secretsmanager" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.amazonaws", artifactId = "aws-java-sdk-secretsmanager" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #88.


feat: allow to use different locale paths for server and client

This is a small PR which allows user to specify different path for client and server scenarios. This allows for more flexible folder structure of the nextjs app, as sometimes just removing the /public/ portion of the path isn't enough. For example having the nextjs app under src folder, as result of using custom server, this way the clientLocalePath would be ./public/static/locales and the serverLocalePath ./src/public/static/locales.

https://github.com/isaachinman/next-i18next/issues/761#issuecomment-661305289


[ Gather ] Dynamic Gather value propagation

Overall looks good.

I would appreciate Concat elimination mentioned above.
Testing for #1
Please add negative tests for cases when we are not allowed to fold

All points introduced


Cannot run tests when TextEditor.insertText is called

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Due to:

https://github.com/atom/text-buffer/pull/295
https://github.com/atom/atom/issues/16956
https://github.com/atom/atom/pull/17089

It's not possible to call TextEditor.insertText in tests.
    Error: Deprecated function(s) TextBuffer.setTextInRange) were called.
      TextBuffer.setTextInRange is deprecated. The `undo` option is deprecated. Call groupLastChanges() on the TextBuffer afterward instead.
      --------------------------------------------------------------------------------------------------------------------------------------
      TextBuffer.setTextInRange -- /home/travis/atom/usr/share/atom/resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:655:16
      Selection.insertText -- /home/travis/atom/usr/share/atom/resources/app.asar/src/selection.js:491:47
      result.mutateSelectedText.selection -- /home/travis/atom/usr/share/atom/resources/app.asar/src/text-editor.js:1377:31
      getSelectionsOrderedByBufferPosition.map -- /home/travis/atom/usr/share/atom/resources/app.asar/src/text-editor.js:1425:86
      Array.map -- null:null:null
      transact -- /home/travis/atom/usr/share/atom/resources/app.asar/src/text-editor.js:1425:60

https://github.com/atom/atom/blob/107ec432e57839e4de12eb5d0034a96c750bf741/src/selection.js#L491
Steps to Reproduce

create a test
use TextEditor.insertText

Expected behavior: [What you expect to happen]
Be able to run tests when using TextEditor.insertText
Actual behavior: [What actually happens]
Tests are failing due to use of the deprecated option undo
Reproduces how often: [What percentage of the time does it reproduce?]
every time.
Versions
Since 1.25 on Linux, MacOS and Windows.
Currently on 1.28.2
Additional information

https://travis-ci.org/asciidoctor/atom-asciidoc-image-helper/jobs/407027295
https://ci.appveyor.com/project/asciidoctor/atom-asciidoc-image-helper/build/37/job/f090ftcw2l0ph2f2

Check the following:
https://github.com/atom/atom/blob/master/src/selection.js#L491


[doc] Refactored main "Getting Started" page to fix #7260

Fix stream-lines main "Getting Started" page by:

Adding a figure to show different workflows.
Detailing the exact steps needed to get a "Hello World!" demo up and running on a target of choice.
Provides pointers to additional resources about individual aspects (general, HW, SW) of the OpenTitan project.

Signed-off-by: Timothy Trippel<EMAIL_ADDRESS>

@tjaychen @rswarbrick @mundaym @alphan @imphil
please review

Thanks @timothytrippel, LGTM! I'll leave this open over the weekend to give others a chance to jump in and will merge on Monday if nothing else comes up.


Map in a Bundle

https://medium.com/the-wtf-files/the-mysterious-case-of-the-bundle-and-the-map-7b15279a794e

workaround in 2.0.0


prompt_preview_theme parse error when previewing the adam2 theme

To be clear, this issue is with previewing the adam2 theme and not with using the theme itself. I personally do not feel the need to preview themes often, so I would consider this to be a low priority issue anyways. Also, the parse error does not come up when previewing other themes. Parse error message when previewing the adam2 theme is copied below.
~/.zprezto ❯❯❯ uname -a; git pull && git submodule update --init --recursive; prompt -p adam2
Linux awl 3.13.0-48-generic #80-Ubuntu SMP Thu Mar 12 11:16:15 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
Already up-to-date.
adam2 theme:
prompt_preview_theme:14: parse error
.-(~/.zprezto)-------------------------------------------------------------------------------------------------------------------------(iong-local@awl)-
`--> command arg1 arg2 ... argn

That theme comes with Zsh, not Prezto; so, it's not a Prezto issue.


New IRVisitor test to show how we could escape HTML

Trying to implement a scenario using IR rewriting to handle HTML escaping automatically:

Literal strings are not escaped
Variables are escaped (can potentially have unsafe data)
Data from tables are escaped (can potentially have unsafe data)
IR tree is traversed and whenever there are unsafe string nodes being concatenated, those nodes are escaped.

✅ No public API change.


Assertion 'rows() == other.rows()' fails in Debug mode

The assertion 'rows() == other.rows()' fails in Debug mode inside Decimator::solveConstrainedMinimizationInequalities.
This always happens, for instance in the first test case:

test 1
Start 1: testBoundingmeshTorus
1: Test command: Debug/boundingmesh "-v" "100" "examples/torus/torus.off" "Debug/torus_decimated.off"
1: Test timeout computed to be: 9.99988e+06
1: boundingmesh: /usr/include/eigen3/Eigen/src/Core/Assign.h:505: Derived& Eigen::DenseBase::lazyAssign(const Eigen::DenseBase&) [with OtherDerived = Eigen::Matrix<double, -0x00000000000000001, -0x00000000000000001>, Derived = Eigen::Matrix<double, -0x00000000000000001, 1>]: Assertion `rows() == other.rows() && cols() == other.cols()' failed.
1/6 Test # 1: testBoundingmeshTorus ...................***Exception: Other  0.18 sec
test 2

Fixed by a4b738767cf0d
If an Eigen vector is supposed to have size zero, it should not be constructed by MatrixXd(0, 0), but better with Eigen::VectorXd::Zero(0).


xray role changes owner of artifactory folder, causing error in artifactory

Which installer: Ansible
Which product and version: ansible 2.11.2 jfrog-platform collection 7.21.12
Which operating system and version: Ubuntu 20.04
What happened:
I used this playbook to install artifactory and xray on a single server
- hosts: postgres_servers
  roles:
    - jfrog.platform.postgres

- hosts: artifactory_servers
  roles:
    - jfrog.platform.artifactory

- hosts: xray_servers
  roles:
    - jfrog.platform.xray

While artifactory started working for a while, when the xray role ran, it chowned /opt/artifactory to user xray:xray!
I had to chown -R artifactory:artifactory artifactory artifactory-pro-7.21.12 jfrog-artifactory-pro-7.21.12-linux.tar.gz
What you expected to happen:
artifactory and xray to work without spending more than a day troubleshooting

Ideally, we recommend you to try on different servers :)


Fixed: Search module error on down-cursor key if showNoResults=false

This PR fixes an unexpected error on Search module.
When the Search module built with showNoResults={false} props, the <div class="result"> element won't present in DOM. And if a user types cursor-down (keycode: 40) in search input,  then it raises an error:
Search.js:260 Uncaught TypeError: Cannot read property 'querySelector' of null
    at Search.eval [as scrollSelectedItemIntoView] (Search.js:260)
    at Search.eval [as moveSelectionBy] (Search.js:252)
    at Array.eval (Search.js:113)
    at EventSet.dispatchEvent (event-stack.development.js:164)
    at EventPool.dispatchEvent (event-stack.development.js:275)
    at eval (event-stack.development.js:341)
    at Map.forEach (<anonymous>)
    at HTMLDocument.eval (event-stack.development.js:340)

This PR can avoid that error by checking menu element existence.

Codecov Report

Merging #3938 into master will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #3938      +/-   ##
==========================================
- Coverage   99.84%   99.84%   -0.01%
==========================================
  Files         184      178       -6
  Lines        3218     3152      -66
==========================================
- Hits         3213     3147      -66
  Misses          5        5

Impacted Files
Coverage Δ

src/modules/Search/Search.js
99.55% <100.00%> (-0.01%)
:arrow_down:

src/elements/Button/Button.js
95.45% <0.00%> (-0.11%)
:arrow_down:

src/modules/Popup/Popup.js
98.71% <0.00%> (-0.04%)
:arrow_down:

src/modules/Tab/Tab.js
100.00% <0.00%> (ø)

src/views/Card/Card.js
100.00% <0.00%> (ø)

src/elements/Flag/Flag.js
100.00% <0.00%> (ø)

src/elements/Icon/Icon.js
100.00% <0.00%> (ø)

src/elements/List/List.js
100.00% <0.00%> (ø)

src/elements/Step/Step.js
100.00% <0.00%> (ø)

... and 44 more

Continue to review full report at Codecov.

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


Update README.md

wrong module name

closing in favor of https://github.com/fraserxu/gulp-html2js/pull/18


Mcho/dev

Questions
Answers

Description?
on reset or uninstall then install, modules are hooked in the wrong place

Type?
bug fix

BC breaks?
yes

Deprecations?
yes

Fixed ticket?
Fixes PrestaShop/Prestashop#20748

How to test?
Steps to reproduce the behavior:

Go to BO > Module Manager page > Search for the slider module
Click on reset
Go to FO > Home page
See error => the banner is at the last position on the home page

mistake not branch master but dev branch for PR sorry

im so sad, could you close issue in this case ?


Unify collection lists

What happened?
I saw that we have two types of components which are may doing the same ??
import CollectionList from<EMAIL_ADDRESS>import Collections from<EMAIL_ADDRESS>
<template>
  <CollectionList v-if="type === 'rmrk'" />
  <Collections v-if="type !== 'rmrk'" />
</template>

Please reproduce in steps
~
Expected Behavior
I've expected to have one reusable component across the app 🙈
What browsers are you seeing the problem on?
No response
Are you logged in?
No response
At which address did you encounter bug?
No response
Screenshots
~
Relevant log output
No response
Payment link for reward
https://kodadot.xyz/transfer/?target=<My_Kusama_Address_check_https://github.com/kodadot/nft-gallery/blob/main/CONTRIBUTING.md#creating-your-ksm-address>
Code of Conduct

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

@vikiival can we add priority to this?

@roiLeo it would get merged with #2812

meanwhile @/components/rmrk/Gallery/Collections.vue is not used anymore, IMO we can safely remove it and close this issue


PEP8 Compliance Errors

This PR cleans up the PEP8 compliance errors introduced in PRs #5 and #6.

This patch seems to inherit a bunch of extra stuff from your fork... in any case I think I fixed up the PEP8 reports when I merged your earlier changes so I don't think this is needed anymore.


changes needed to build the example on linux

Hi @inglada, thanks for the PR!
Instead of making the example more complex, I added CMake transitive dependencies for the required libraries :) Thank you very much for bringing this to our attention!

Thanks. That's OK for the dependencies, but I still have to use lower case in
find_package(sqlite3 REQUIRED)
instead of
find_package(Sqlite3 REQUIRED)

OK. Thanks!


Next release date

Hi, sorry to be presumptuous in asking, but is there a date for the next release?
There are quite a few unmerged PR's and we just spend a solid 2 weeks on a confounding bug related to this plugin (after having upgraded and cause imgCache related issues) that one of the PR's actually addresses.
Looks like last release was 3.5 months ago. Tnx

Maybe @dwieeb ?

This looks really bad Ionic. Is Capacitor the same response rate?

Please Can anybody tell us when it will be the next release, thanks ^_^

In your opinion, what are the most important PRs you would like to see merged?

@mlynch I think most people want this feature : add http(s) proxy/scheme for iOS
https://github.com/ionic-team/cordova-plugin-ionic-webview/pull/448
we still have a cors problem while using salesfore ( salesforce only accept  http/https)
Link: https://help.salesforce.com/articleView?id=snapins_chat_cors_whitelist.htm&type=5
And others like Azure:
https://github.com/ionic-team/cordova-plugin-ionic-webview/issues/349

I would agree @webteambel def 448. Also #376, #452 and #486. There are only 7 total and they all passed with no conflicts so I maybe a full merge is ok @mlynch ?

any news ? We are waiting for the new version so we can fix our problem and publish our new version of app using ionic 5.

@webteambel This is such an easy release. All tests past. Almost no information except @mlynch asking 2 weeks ago. Can I maintain this plugin if Ionic won't?

@tryhardest I have no idea, sorry :/

To shed a little light on this, we recently audited the PRs in this repo and either closed them, asked for more information, or begun the merging process. We're going to make a release as soon as https://github.com/ionic-team/cordova-plugin-ionic-webview/pull/452 is merged, which should address some of the original concerns.
This repo doesn't actually have any automated tests (it's tremendous work setting up automated testing for Cordova plugins), so it's not as simple as just merging if there are no conflicts. We have to thoroughly hand test submitted code as well as inspect it for potential security issues.
As a heads up, part of the delay here is that we're starting to focus more on Capacitor after seeing the vast benefits: cohesive, focused platform; productivity gains; faster shipping; etc. That's not to say Cordova won't be supported for the foreseeable future, but we have a limited number of resources to dedicate. We're a small team and frankly the amount of open source code we have is overwhelming sometimes, so thank you for your patience! 💙

4.2.0 was released last Tuesday


Word Cloud is hanging the browser

Have a look at this link. All browsers are hanging, including Edge.

Hi Rhuan,
We're investigating this issue and will update. Please be aware that we're aware that the WordCloud visual has performance issues when data gets big.
thx, Avi

@rhuanbarreto Thanks for your feedback.
We’ve already fixed the performance problem of WordCloud. We’re going to publish it to the Power BI DEV environment today.
The new version will be available in PROD in two weeks.


[REGRESSION]: Cannot set ChromiumSandbox launch option because of error "slowMo: expected number, got boolean"

Context:

GOOD Playwright Version: 1.40.0
BAD Playwright Version: 1.41.0
Operating System: Windows
Extra: None

Code Snippet
var playwright = await Playwright.CreateAsync();
var chromeLaunchOptions = new BrowserTypeLaunchOptions
{
  ChromiumSandbox = false,  // <-- This is the problem.
};
var chrome = await playwright.Chromium.LaunchAsync(chromeLaunchOptions);

Describe the bug
On the call to LaunchAsync when using BrowserTypeLaunchOptions that sets the ChromiumSandbox value(true or false), the following exception is thrown:
Microsoft.Playwright.PlaywrightException: slowMo: expected number, got boolean
   at Microsoft.Playwright.Transport.Connection.InnerSendMessageToServerAsync[T](ChannelOwner object, String method, Dictionary`2 dictionary, Boolean keepNulls) in /_/src/Playwright/Transport/Connection.cs:line 208
   at Microsoft.Playwright.Transport.Connection.WrapApiCallAsync[T](Func`1 action, Boolean isInternal) in /_/src/Playwright/Transport/Connection.cs:line 538
   at Microsoft.Playwright.Core.BrowserType.LaunchAsync(BrowserTypeLaunchOptions options) in /_/src/Playwright/Core/BrowserType.cs:line 56

Simply rolling back to 1.40.0 makes this code work again.
Basic console app:

Folding into https://github.com/microsoft/playwright-dotnet/pull/2794#issuecomment-1895677252


Outdated

Article is outdated. Many of the options here have moved or are renamed in the admin center

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

ID: 3ed95865-eddb-4ad1-d2e2-5bd618151c1b
Version Independent ID: ef32bb26-e78a-caad-5b41-d7e49f6ef9b4
Content: Set up multi-factor authentication for Office 365 users
Content Source: Office365-Admin/security-and-compliance/set-up-multi-factor-authentication.md
Service: o365-administration
GitHub Login: @Sirkkuw
Microsoft Alias: sirkkuw

@RobvanB Thank you for your feedback. We have made changes to the doc via PR#https://github.com/MicrosoftDocs/OfficeDocs-O365ITPro/pull/890 to reflect the correct steps.
Thanks
Sri

@RobvanB - Please refer to the author's https://github.com/MicrosoftDocs/OfficeDocs-O365ITPro/pull/890#issuecomment-570688767 in response to this issue. We hope this helps to give clarity so will close this issue.
Thank you for your contribution to make the docs better! Much appreciated!


Fixed the dependecies list

I don't think this PR is what it should be, because all it does is remove the instructions to install the dependencies.  Seems redundant, I'm going to close it.


Errors got lost on-save

Note: I'll be happy to attempt a fix PR (at some point =) if you let me know whether it's more likely in the templ-vscode extension repo or the lsp part of the templ prog.

templ v0.2.364, none of its config-settings customized yet.
go version go1.21.1 linux/amd64
VSCodium 1.82.2 (settings editor.formatOnSave and editor.formatOnType both true)
Linux (Void Linux, kernel 4.19.294_1, utterly irrelevant I'm sure)

Here's a repro file to play with (perhaps in an ideally fresh / blank workspace), initially correct:
package client

templ BarBaz() {
	<foo>if true { Bar<b>Baz</b> } </foo>
}

Now remove the very last } and diagnostics correctly shows an error. But save it and it's gone!
Now close and reopen the incorrect file. The error initially shows up correctly. But correct it and it stays! Save it and it still stays!
Now Cut the whole (just-corrected) templ and the error goes. But Paste it back in, it returns! Even renaming the templ won't help. But removing it's one line or just the if keyword will.
So it's a lil shaky yet as far as diagnostics-publishing goes  =)  mad props on this cool project either way tho!

Whoa I even did a VSC reload (workbench.action.reloadWindow) and the mistaken if: unterminated (missing closing '{ ') diagnostic just keeps on staying! Did another after repeated killall templ and it's still there. Are these persisted for some reason?!
The state of the file is the same as quoted originally in the beginning above.

The same issue also occurred during my tests for #192.

Thanks for looking into this and for highlighting the issue.
I merged an update from @joerdav today that clarifies how the for,if and switch stuff should be used. It now checks for {\n as expected after an if block - https://github.com/a-h/templ/pull/216 - so, I think this issue might be fixed in the latest commit. I couldn't replicate the issue as you describe it with the latest.
However, reading this, I'm thinking that maybe the parser should allow elements etc. after the opening brace, but that the formatter should put it on a new line.
For example, in the Go playground, if you write this...
package main

import "fmt"

func main() {
	if true { fmt.Println("test") }
}

It gets updated to:
package main

import "fmt"

func main() {
	if true {
		fmt.Println("test")
	}
}

If you're interested in seeing how the LSP is actually seeing the files, there's some info on the debugging tools in the LSP at https://github.com/a-h/templ/issues/85 - You can inspect the internal model that the LSP has, including the templ source code to Go source code mapping.
The logs are reasonably good too.

Hi folks, I've just released https://github.com/a-h/templ/releases/tag/v0.2.408 - it contains a number of changes that are likely to impact or fix this issue.
In particular...

fix: use full sync to reduce jank LSP behaviour
feat: add useful error message for incomplete go statements
fix(fmt): do not introduce additional whitespace where it affects rendering

If you're able to test again, that would be much appreciated.

I haven't had any reports of further issues since v0.2.408, so I think it's time to close this issue. Please re-open or create a new issue if there's still a problem here.
Thanks everyone!


Adds a list of requirements to the expansion request

This should be used to interpret the components provided with the request.
For example, this may be used to provide the StandardRequirements related to ParDos which otherwise cannot be derived from the components provided.
https://github.com/apache/beam/blob/f3636f32b3f5a97bdd5c8421bf3b1f702ad0980f/model/pipeline/src/main/proto/org/apache/beam/model/pipeline/v1/beam_runner_api.proto#L1668
This fixes https://github.com/apache/beam/issues/29354.

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

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

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

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

assign set of reviewers

Thanks.
I believe Go failures are unrelated: https://github.com/apache/beam/issues/29355

Requirements specify capabilities the runner must understand to execute a pipeline.
While retuning a set of requirements along with the response make sense (though I don't see that being done or these being incorporated into the final pipeline's requirements, which is a bug we need to fix) I and struggling to understand why one would send a set of "requirements" with a request.


Add references to the actual Schema, Groups and Filters to Result

This way it will be easy to access the actual Schema, Groups and Filters and all their data. It also removes duplicate code.

Closed by https://github.com/robertpainsi/filtering.js/commit/2178a9d06e19a3026e583e0a67ac8ed742c52446


useService support

Partial fix for #48 including only useService. Personally, I haven't use the useActor just yet and I am not sure how it even works. I think it's better to keep a smaller PR.

LGTM as it is 👍


Allow %IP placeholder in bluenet's host name that gets replaced with IP address

Bluenet has a host name parameter -h that can be feeded with the hostname or IP address of the hub. Since currently we don't use a zeroconf for the hub, we want the Hub to tell us its IP address. Therefor the -h parameter should replace occurrences of %IP with the actual IP address when connected to a Wi-Fi network. This said, we can feed the -h with any arbitrary content such as -h http://%IP:6543/-/ to let the mobile client know where the hub is reachable. Currently we provide the hub's full API endpoint to the -h parameter for convenience.

Implemented in #181


More explicit campfire errors

This provides more explicit campfire errors for the following cases :

If the token, rooms or account environment vars aren't defined
If campfire returns a 401 error (the token is invalid)

Looks great, thanks.


Carbon SVG Export has Broken Backgrounds

Describe the bug
The SVG export should look somewhat similar to the PNG exported version. Instead it has a white background, which isn't expected, and is almost unreadable.

Instead, it looks like this:

To Reproduce
Steps to reproduce the behavior:

Go to https://carbon.now.sh

Make sure there's some code in the view

Click export as SVG

Info (please complete the following information):
This seems to happen with any BG colour, any code, any syntax highlighting theme, including the defaults when you first navigate to the page.

OS [e.g. macOS, Linux, Windows, iOS]: Reproducible on Linux, Windows and MacOS.
Browser [e.g. Chrome, Safari, Firefox]: Reproducible on Vivaldi and Firefox.
Carbon URL [e.g. carbon.now.sh?bg=pink]: https://carbon.now.sh/

Here is a copy of the .svg that is exported. Stored within a .zip file as GitHub issues can't embed SVG files.
carbon.svg.zip

Code snippet

   const pluckDeep = key => obj => key.split('.').reduce((accum, key) => accum[key], obj)
const compose = (...fns) => res => fns.reduce((accum, next) => next(accum), res)
const unfold = (f, seed) => {
const go = (f, seed, acc) => {
const res = f(seed)
return res ? go(f, res[1], acc.concat([res[0]])) : acc
}
return go(f, seed, [])
}

Hello! I believe this may have introduced a new issue, the title bar isn't on top of the background like it should be.
Carbon Preview

Exported SVG

Attached Broken SVG
BrokenTitleBarSVG.zip


Migrate from find-java-home to jdk-utils

This would mean that we use the same strategy as in vscode-java.

@JessicaJHee  were you interested in looking into this? If not, I can take a look into it for next sprint

@JessicaJHee were you interested in looking into this? If not, I can take a look into it for next sprint

Yes I'm interested in this issue! I can take it


linux-tegra: Enable binder config for Jetson Orin Nano

As per zendesk ticket 2718. The config was previously enabled for the TX2 on L4T 32.X base-line, but the upstream 35.X did not enable it by default.
Changelog-entry: linux-tegra: Enable binder config for Jetson Orin Nano

@balena-ci I self-certify!


Issue when using home manager as a nixos module: services option does not exist

I'm in the process of writing a fully declarative configuration of my system. When I try to build it, I get the following error:
$ sudo nixos-rebuild test -I nixos-config=/cfg
building Nix...
building the system configuration...
error: The option `mine.homeConfig.services' defined in `/cfg/modules/bluetooth.nix' does not exist.
(use '--show-trace' to show detailed location information)

Now, the context.
Four files are of interest here:

/cfg/default.nix: This is the top-level configuration file.

{ config, pkgs, ...}:

{
  imports =
    [ ./machines/current
      ./modules
      <home-manager/nixos>
    ];
}

The home-manager channel is installed by the root user:
# nix-channel --list
home-manager https://github.com/rycee/home-manager/archive/release-19.03.tar.gz
nixos https://nixos.org/channels/nixos-19.03

/cfg/modules/default.nix imports all .nix files from the /cfg/modules directory and all sub-directories.
/cfg/modules/users.nix:

{ options, config, lib, ... }:

with lib;

{

  options.mine = {

    # [...]

    homeConfig = mkOption {
      type = options.home-manager.users.type.functor.wrapped;
      default = {};
      description = "Home-manager configuration to be used for the user";
    };

  };

  config = {

    # [...]

    home-manager.users."${config.mine.userName}" =
      mkAliasDefinitions options.mine.homeConfig;

  };

}

/cfg/modules/bluetooth.nix:

{ pkgs, config, lib, ... }:

with lib;

let cfg = config.mine.bluetooth; in

{

  options.mine.bluetooth.enable = mkOption {
    type = types.bool;
    default = config.mine.hardware.bluetooth;
    description = "Enable bluetooth";
  };

  config = mkIf cfg.enable {

    hardware.bluetooth = {
      enable = true;
      powerOnBoot = mkDefault false;
    };

    # Required for blueman-applet service to work.
    environment.systemPackages = [ pkgs.blueman ];
    services.dbus.packages = [ pkgs.blueman ];

    mine.homeConfig.services.blueman-applet.enable = config.mine.x.enable;

  };

}

My config is heavily inspired by @Infinisil 's config, perhaps he will have a clue as to what's going on.
EDIT: Some more info:
Output of sudo nixos-rebuild test --show-trace -I nixos-config=/cfg:
building Nix...
building the system configuration...
error: while evaluating the attribute 'activationScript' of the derivation 'nixos-system-kuba-x220-19.03.172883.1601f559e89' at /nix/var/nix/profiles/per-user/root/channels/nixos/nixos/modules/system/activation/top-level.nix:102:5:
while evaluating the attribute 'system.activationScripts.script' at undefined position:
while evaluating anonymous function at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:75:45, called from undefined position:
while evaluating the attribute 'value' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:338:9:
while evaluating the option `system.activationScripts':
while evaluating the attribute 'isDefined' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:375:5:
while evaluating the attribute 'values' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:364:9:
while evaluating the attribute 'values' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:458:7:
while evaluating anonymous function at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:350:28, called from /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:350:17:
while evaluating 'dischargeProperties' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:417:25, called from /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:351:62:
while evaluating the attribute 'condition' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:502:14:
while evaluating the attribute 'condition' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:502:14:
while evaluating the attribute 'condition' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:502:14:
while evaluating the attribute 'services.udisks2.enable' at undefined position:
while evaluating anonymous function at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:75:45, called from undefined position:
while evaluating the attribute 'value' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:338:9:
while evaluating the option `services.udisks2.enable':
while evaluating the attribute 'isDefined' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:375:5:
while evaluating the attribute 'values' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:364:9:
while evaluating the attribute 'values' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:458:7:
while evaluating anonymous function at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:350:28, called from /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:350:17:
while evaluating 'dischargeProperties' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:417:25, called from /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:351:62:
while evaluating the attribute 'condition' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:502:14:
while evaluating the attribute 'condition' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:502:14:
while evaluating the attribute 'condition' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:502:14:
while evaluating the attribute 'condition' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:502:14:
while evaluating the attribute 'mine.homeConfig.services.udiskie.enable' at undefined position:
while evaluating anonymous function at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:75:45, called from undefined position:
while evaluating the attribute 'value' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:338:9:
while evaluating the option `mine.homeConfig':
while evaluating the attribute 'mergedValue' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:370:5:
while evaluating anonymous function at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:370:32, called from /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:370:19:
while evaluating 'merge' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/types.nix:368:22, called from /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:373:8:
while evaluating the attribute 'config' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:87:25:
while evaluating 'yieldConfig' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:74:29, called from /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:73:16:
while evaluating anonymous function at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:80:24, called from /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:80:11:
while evaluating anonymous function at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:81:26, called from /nix/var/nix/profiles/per-user/root/channels/nixos/lib/modules.nix:81:13:
The option `mine.homeConfig.services' defined in `/cfg/modules/bluetooth.nix' does not exist.

When I change the name of the service in bluetooth.nix from blueman-applet to foo, the error message changes:
building Nix...
building the system configuration...
error: The option `home-manager.users.me.services.foo' defined in `/cfg/modules/users.nix' does not exist.
(use '--show-trace' to show detailed location information)

I've boiled the cause of the error down to this file:
{ config, ... }:

{
  services.udisks2.enable = config.mine.homeConfig.services.udiskie.enable;
}

If I don't import this file, the config builds fine.
So apparently I can write options from homeConfig, but I can't read them. I can work around this, but I don't understand why this happens. Is this a bug in the module system?

I've boiled the cause of the error down to this file:
{ config, ... }:

{
  services.udisks2.enable = config.mine.homeConfig.services.udiskie.enable;
}

If I don't import this file, the config builds fine.
So apparently I can write options from homeConfig, but I can't read them. I can work around this, but I don't understand why this happens. Is this a bug in the module system?

Should that not be
{
  config.services.udisks2 ...
}

AFAIK you can omit lots of things when writing a module. The "canonical" form of a module is:
{ config, ... }:

{
  imports = [
  # ...
  ];

  options = {
  #  ...
  };

  config = {
  # ...
  };
}

If your module doesn't have any imports or option declarations, you can write the contents of config inline, without the config = part. Furthermore, if you don't use any of the arguments like pkgs, config etc., you can omit the argument too.
So, this module:
{ config, lib, ... }: # args not used, can be omitted

{
  imports = []; # empty, can be omitted
  options = {}; # empty, can be omitted
  config = {
    # imports and options are empty, so the config
    # doesn't need to be wrapped in a set
    services.udisks2.enable = true;
  };
}

is equivalent to this module:
{
  services.udisks2.enable = true;
}

For a definitive reference, see the source code: https://github.com/NixOS/nixpkgs/blob/55256d644b0671a4a6155d0f462a65a8427a423e/lib/modules.nix#L118

Interesting, I did not know that. Unfortunately I do not know what is causing your issue in that case

And here is the home.nix if this could help https://gist.github.com/SCOTT-HAMILTON/a82bb8a0d93c208c744898cfb07973e9


[BUG] Pending+Expired Payments initiated by Boltz Extension aren't returned to wallet

Describe the bug
Payments that have been initiated from the Boltz Extension but expired, are not removed from wallet's ledger and not returned to the available balance.
To Reproduce
Steps to reproduce the behavior:

Go to Boltz Extension
Create a Swap Out (Lightning -> onchain) for which you apparently did not have enough liquidiity for
bonus: I restarted lnbits docker container once together with lnd -> not sure if that messed things up

Expected behavior
Cleanup Job should collect these pending and expired payments and return balance.
Screenshots

Desktop (please complete the following information):

LNbits version: commit 239f166a141fa9347ba7ab958c80cbc8804bd8b8 (HEAD -> main, origin/main, origin/HEAD)
Database postgres

Additional context
none

looking at it again, it looks like it may just be how lnbits handles failed payments currently. i think this pr would bring more clarity into this situation. https://github.com/lnbits/lnbits/pull/2537


Expose inline styles

Expose the inlineStyle to developers.
resolves #10

@leMaik sorry was working on a release last month and totally forgot this issue. Please let me know if this is your prefer approach to expose the inline style to devs.

@leMaik made changes per review request

@mirocody The great thing is: With<EMAIL_ADDRESS>(port is coming soon, see #28), due to the way withStyles and JSS work, the user will be able to override any style of this component without having props for everything. So this PR essentially makes the 0.x version and the 1.0.0 port compatible. :tada:

@leMaik Awesome so I remove the xxLandscapeStyle and slideStyle, but keep other styles. Also made change to README. please let me know further changes needed 😃

Hi @leMaik just a reminder the fix has been post for your latest comment. Thanks!

@mirocody Thank you for the reminder and the changes. :+1: I assigned this issue to me so that I don't forget this. I'll try to get this done this week-end.

Sorry for the big delay. :disappointed: The ported version for material-ui 1.0.0 will support customization using the classes property, similar to how material-ui's components do it. :+1:


Move maximum_frequency from ProtocolConfig to MeasurementSpec.

This also drops the unsupported maximum_frequency_per_user field from ReachAndFrequency and Duration MeasurementSpec messages.
Closes #159
Closes #160

This change is


Model with id "undefined" returned by collection.get(obj)

@caseywebdev The Collection::get method defined by 01d9f0f9b1e85cb21a2940902bc8ba4280c8bca9 has this odd behavior when the collection contains a model with the id "undefined": collection.get(obj) will return that model whenever obj.id is undefined. An odd edge case, I know, but believe it or not I ran into this in a production application... among other things, it means that
new Backbone.Collection([{id: 'undefined'}, {id: 'foo'}, {id: 'bar'}])

will create a collection with length 1.
What was wrong with the Backbone 1.0.0 definition of get, which explicitly checked for this case?
return this._byId[obj.id != null ? obj.id : obj.cid || obj];

Related to #2847.

Has anyone solved that problem? id "undefined": collection.get(obj)


timer now ends if no bids are placed

timer is now 10 seconds and resets after every new highest bid
now gets the current highest bid correctly when checking against a new bid


Expose web services for task lists

[ ] POST /v1/{{user_uuid}}/lists
[ ] GET /v1/{{user_uuid}}/lists
[ ]  GET /v1/{{user_uuid}}/lists/{{task_list}}
[ ]  PATCH /v1/{{user_uuid}}/lists/{{task_list}}
[ ]  DELETE /v1/{{user_uuid}}/lists/{{task_list}}


Added some ifs around the k3s server specifics

Added:
if [ -d /var/lib/rancher/k3s/server ]; then
Around some of the server specific components (the directory does not exist on agents)
This means that k3s-rancher is skipped for now  but agent nodes don't have kubeconfigs locally so will fail when trying to run that section anyway
Perhaps a kubeconfig generator might work in future?
These seem to have the necessary info to make one:
/var/lib/rancher/k3s/agent/etc/k3s-agent-load-balancer.json
/var/lib/rancher/k3s/agent/server-ca.crt
/etc/rancher/node/password

I might work on that next

Also this is my first contrib to a more publicly used script please test it (I didn't do much but it's best to be sure 😬 )

Thanks @leflambeur, I added a few changes and tested on:

Ubuntu 18.04 k3s server/agent
Ubuntu 18.04 RKE
RancherOS 1.5.5 RKE

I'll test it tomorrow for you 😊


add build and run req: libssh2

Superceeds and closes #2 .

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Upload problem mentioned in https://github.com/conda-forge/libgit2-feedstock/pull/2#issuecomment-254885011 is solved by removing the personal BINSTAR_TOKEN environment variable from CircleCI account.

No package 'libssh2' found mentioned in https://github.com/conda-forge/libgit2-feedstock/pull/2#issuecomment-254885011 and https://github.com/conda-forge/libgit2-feedstock/pull/2#issuecomment-255646577 still exists. Suspect pkconfig related issue in libssh2. Will investigate libssh2-feedstock.

I triggered CI before the fixed libssh2 was built on Travis thinking that Travis wouldn't build this PR before libssh2 anyways. Plus, Travis did not have problems and it is CircleCI we are curious about. Now, I see this on CicleCI
-- Checking for module 'libssh2'
--   Found libssh2, version 1.7.0_DEV

What is _DEV suffix? Why do we see it? 😕

What is _DEV suffix? Why do we see it? 😕

See https://github.com/conda-forge/libssh2-feedstock/pull/4#issuecomment-271057053.

As is libssh2 can be found on Travis and Circle builds, but not on Windows for libgit2 builds on my forks. When the builds are done here I'll double-check and will leave Windows builds alone. Thoughts?

Unrelated FYI have enabled CircleCI to build PRs on this repo. So it should no longer be necessary to have CircleCI enabled on your fork.


Version update

Hi
I updated to the most recent version using
pip install --upgrade boldigger-cline
However, whenever I check the version of boldigger-cline, it says it's 1.0.0. Wasn't 2.0.0 the latest release?
Thanks
nauras

Since "-" is forbidden in python modules you'd have to install it with:
pip instal --upgrade boldigger_cline

alright, got it. thanks. but the initial installation with "-" if i remember correctly?!
either way, the github page should probably be updated accordingly then.
thanks again for the help.
Nauras

done

Hi, could you re-open this please?
I had boldigger_cline v2.1.2 installed and updated to v.2.2.0 according to your instructions above. See the last part of the installation output below:
Installing collected packages: boldigger, boldigger_cline
  Attempting uninstall: boldigger
    Found existing installation: boldigger 2.1.2
    Uninstalling boldigger-2.1.2:
      Successfully uninstalled boldigger-2.1.2
  Attempting uninstall: boldigger_cline
    Found existing installation: boldigger-cline 2.1.2
    Uninstalling boldigger-cline-2.1.2:
      Successfully uninstalled boldigger-cline-2.1.2
Successfully installed boldigger-2.2.0 boldigger_cline-2.2.0

However, when I check the installed version, it still says 1.0.0. See below:
(digger_env) -bash-4.2$ boldigger-cline --version
1.0.0

Any idea what's going on here?

Hi there,
my assumption would be that you have two different versions of python installed. One where pip is pointing to, the other where boldigger-cline is pointing to.
When calling boldigger-cline you can specify which python version to use with:
"pythonX.X -m boldigger-cline" (X.X refers to the version you want to call)
Can you please check if this is solving your issue?
Alternatively, to show which version you have installed you can also run
"pip freeze"
or
"pipX.X freeze" (X.X) again for you version, to show all installed packages

Thanks for the reply. Yes, something like this must be going on here.
Running bildigger-cline vwith miniconda. Have to figure this out with our HPC people.
Thanks for pointing me in the right direction. Will close this for now, as this is probably not a boldigger-cline issue.

To me it seems as there really is an issue here. While leaving the problem with boldigger-cline in conda on an HPC aside, I just wanted to check this on my personal machine.
I uninstalled boldigger-cline:
$ pip uninstall boldigger_cline
Found existing installation: boldigger-cline 2.2.0
Uninstalling boldigger-cline-2.2.0:
  Would remove:
    c:\users\nauras\programs\python\python39\lib\site-packages\boldigger_cline-2.2.0.dist-info\*
    c:\users\nauras\programs\python\python39\lib\site-packages\boldigger_cline\*
    c:\users\nauras\programs\python\python39\scripts\boldigger-cline.exe
Proceed (Y/n)? Y
  Successfully uninstalled boldigger-cline-2.2.0

Installed again:
$ pip install boldigger_cline
Collecting boldigger_cline
  Using cached boldigger_cline-2.2.0-py3-none-any.whl (16 kB)
Requirement already satisfied: boldigger>=2.2.0 in c:\users\nauras\programs\python\python39\lib\site-packages (from boldigger_cline) (2.2.0)
.
.
.
Installing collected packages: boldigger_cline
Successfully installed boldigger_cline-2.2.0

However, when checking the version, it still says 1.0.0:
$ boldigger-cline --version
1.0.0

I just have one version of python installed on my personal machine:
$ py -0
Installed Pythons found by C:\WINDOWS\py.exe Launcher for Windows
 -3.9-64 *

So something must be up here. What am I misunderstanding?
Cheers
nauras

Fixed with version 2.2.1

Ok, so I was using the latest version after all, it shows showed the incorrect version?

Yes, now I still need to find a solution for the deviation between website and BOLDigger output which seems impossible to me, because they are retrieved from the same source. But I'll dig into this next week when I have time again :)


[docs] nodesource script is being deprecated

Describe the bug
when running the installer script for nodejs on ubuntu/debian in step 3, the following text appears in the terminal:

                           SCRIPT DEPRECATION WARNING

  This script, located at https://deb.nodesource.com/setup_X, used to
  install Node.js is deprecated now and will eventually be made inactive.

  Please visit the NodeSource distributions Github and follow the
  instructions to migrate your repo.
  https://github.com/nodesource/distributions

  The NodeSource Node.js Linux distributions GitHub repository contains
  information about which versions of Node.js and which Linux distributions
  are supported and how to install it.
  https://github.com/nodesource/distributions

there is then a 60 second wait before the script continues
Steps to reproduce

run curl -fsSL https://deb.nodesource.com/setup_18.x | sudo bash - in the terminal as described in step 3 of the setup docs
see message

Expected behavior
the script used for setting up nodejs not being deprecated
Screenshots
No response
Self-hosted instance?
Yes
Additional context
No response

The new steps seem a bit more complex, we might need to replace the Nodesource repo steps with another method. Maybe something like nvm, n, or fnm?

nvm is one of the most user friendly nodejs runtime managers i've tried but i haven't tried the other ones

nvm might be a good pick here, yeah. Main downside I can think of is that there's no Windows support, but since the bot currently doesn't support Windows (and the guide only has steps for Linux systems anyways) that shouldn't be too much of an issue.


change language_sh.lua to support .zsh

simply added .zsh to the 'files' line to highlight code in .zsh files.

Could you please also bump the plugin version in manifest.json?

Sorry, I am a noob at this. can I do that in this PR, or I must create a new PR?

Yes, you can do it in this PR by adding new commits to your master-1 branch.


[bugfix] use env and partition of testcase for json report

Fixes https://github.com/eth-cscs/reframe/issues/1502

Codecov Report

Merging #1506 into master will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1506      +/-   ##
==========================================
- Coverage   91.72%   91.72%   -0.01%
==========================================
  Files          83       83
  Lines       13029    13027       -2
==========================================
- Hits        11951    11949       -2
  Misses       1078     1078

Impacted Files
Coverage Δ

reframe/frontend/statistics.py
95.45% <100.00%> (-0.06%)
:arrow_down:

Continue to review full report at Codecov.

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


[Event Hubs]  Create clients using Azure Identity tracing getEventHubProperties() creates a span with a peer.address attribute as the FQNS failing in nightly runs

Event Hubs nightly test runs are failing with:

Error message:
expected false to be true
Stack trace:
AssertionError: expected false to be true
at Context. (/mnt/vss/_work/1/s/sdk/eventhub/event-hubs/test/public/node/client.spec.ts:121:16)
at runMicrotasks ()
at processTicksAndRejections (internal/process/task_queues.js:97:5)

For more details check here:

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1482675&view=results

I'll take this @jeremymeng since it's tracing related


whisper.cpp update

https://github.com/go-skynet/LocalAI/pull/717 Needs a manual update - this card is to track it

this is now irrelevant


Material styles overriding third party CSS framework

What is the expected behavior?
Tachyons CSS classes are able to style elements, ie <button mat-stroked-button></button>
What is the current behavior?
Tachyons CSS is being overwritten by Material Styles because the Material CSS is being loaded after Tachyons.
What are the steps to reproduce?
I have reproduced a minimal example, the code is here and you can see it on netlify here
The button text is being uppercased using the Tachyons class ttu. The button itself should have extra padding as it has the Tachyons pa4 class applied to it but this is being overwritten by the Material mat-stroked-button class. If you open devtools you can see that the bundled CSS is in a link tag before the Material CSS in a script tag, therefore being overwritten.
How can I load the material CSS after the bundled styled CSS?
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular CLI: 7.1.1
Node: 11.11.0
OS: win32 x64
Angular: 7.1.1
... animations, cdk, cli, common, compiler, compiler-cli, core
... forms, http, language-service, material, platform-browser
... platform-browser-dynamic, router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.11.0
@angular-devkit/build-angular     0.11.0
@angular-devkit/build-optimizer   0.11.0
@angular-devkit/build-webpack     0.11.0
@angular-devkit/core              7.1.0
@angular-devkit/schematics        7.1.1
@ngtools/webpack                  7.1.0
@schematics/angular               7.1.1
@schematics/update                0.11.1
rxjs                              6.3.3
typescript                        3.1.6
webpack                           4.23.1

Is there anything else we should know?

This is working as expected- the components all assume they own the styling on themselves. Any overrides of the Material Design styles would need to happen with higher specificity.