Bluetooth: UUID: Remove macros defining 16 bit values

Application should normally declare a bt_uuid with proper type and then
use bt_uuid_cmp.
Fixes #5162
Signed-off-by: Luiz Augusto von Dentz<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master    #6166   +/-   ##
=======================================
  Coverage   52.87%   52.87%
=======================================
  Files         412      412
  Lines       40278    40278
  Branches     7802     7802
=======================================
  Hits        21297    21297
  Misses      15765    15765
  Partials     3216     3216

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 8cdd9be...8b6c0b2. Read the comment docs.


IndexGet() of Array/ImmutableArray/Undefined types

Currently index-get behavior of array is to raise a run-time error. Maybe it can simplify things if array returns undefined instead.
Also, if this array retuning undefined makes sense, we may also want to implement IndexGet of Undefined type to return undefined: undefined[index] => undefined
a := []
b := a[1][2][3]      // b == undefined

c := {}
d := c.foo["bar"][1] // d == undefined

Other things to think about (for consistency):

IndexSet behavior of array: likely we'd like to keep the current behavior of raising a run-time error?
Slicing operator of array arr[low:high]: currently it raises error if out-of-bounds. should we keep it?

/cc @earncef

I think IndexSet would have to retain the current behavior of runtime error as it makes no sense to set an item at an invalid index. Perhaps we can allow empty index to append to the array, but that is a different conversation altogether.
I can think of three ways of handling the Slicing operator:

out-of-bounds error
Return only values within the valid range. For instance if low > max returns empty slice; low < max and high > max returns [low:max]
Return Undefined for invalid range. I'm not sure it makes much sense to do this though.

The upside of leaving the runtime error for IndexGet makes it consistent with IndexSet behavior. Given that index is actually out of range, and arrays can't contain objects at any random index, this makes sense. That said, I think Undefined actually takes care of the problem quite well as the programmer can decide whether to crash out or not.
The other alternative is to create some kind of an operator to deal with this and leave the behavior as is now.
One possibility is the ?? operator that I was considering. It would require some way to catch runtime exceptions in the VM and deal with them. On the plus side, I suppose an exception handling mechanism would have to be built alongside it.
The other is something similar to optional unwrapping in Swift. Something like:
a := b?[1]?.c || 5

The presence of ? would have to be passed to the IndexGet and IndexSet methods so they could handle them as preferable. The syntax looks a little clunky but is readable and is probably simpler to deal with while keeping the current behavior.

I'll experiment with array-returning-undefined scenario, just to make sure it doesn't have any other side effects, and, it works well with other language constructs. A couple other things I want to test with:

String.IndexGet
Array.IndexSet and String.IndexSet
Slicing on Array and String

I've been testing this myself and returning undefined on slicing on array/string requires concatenation for strings and bytes and the builtin functions len() and append() are changed to work with undefined.
It's not very intuitive to check whether the resulting slice is undefined before performing these actions.
Also:
a := [1, 2, 3]
b := a[1:5] // undefined here is nonintuitive

If the goal is to avoid runtime errors on slicing, then the more intuitive behavior would be to return the values within the range or when there are no values within the range, an empty slice.

Ok. I think we can try this:

Sequence types: Array, ImmutableArray, String, Bytes
IndexGet on sequence types: returns undefined if index is out of bounds
IndexSet on Array type (other sequence types are immutable): raises a run-time error if index is out of bounds
Slicing on sequence types: returns a new sequence of same type (except for ImmutableArray which returns Array) with the elements within the range (or empty sequence)
IndexGet on Undefined type: always returns undefined

Let me know what you think @earncef.

Looks good but I'll run some tests and get back to you.

Ok, I found this while I was testing. In vm.go:650:
				if highIdx < 0 {
					highIdx = numElements
				} else if highIdx < 0 || highIdx > numElements {
					return fmt.Errorf("index out of bounds: %d", highIdx)
				}

Either the else if makes no sense or the if doesn't. Not sure what the expected behavior is. Should the highIdx be 0 or numElements in the changes we discussed? I'd assume it should be 0, but it's being set to numElements right now so I'm a bit confused. There are no tests checking this that I could find.
So:
a := [1, 2, 3][0:-1] //a = [1, 2, 3]

Ah, I see what the code is doing. It's designed to handle the likes of
a := [1, 2, 3][:]

I think it might be better to rewrite that part without using highIdx as -1


Add support for the DASH role scheme (#8194)

Description of your pull request and other information

Add support for the DASH role scheme so audio can be selected based on the role.
You can select the role using a format arguments e.g. bv,main or bv,description.
Trying to work out the weighted scoring logic for selecting the best video/audio gave me a headache, so I gave up. It would be good to know how to make main a priority.
Example Output:
ID EXT RESOLUTION │   TBR PROTO │ VCODEC        VBR ACODEC     ABR ASR MORE INFO
──────────────────────────────────────────────────────────────────────────────────────────────────────────────────
11 m4a audio only │   32k https │ audio only        mp4a.40.5  32k 48k [en] DASH audio, m4a_dash, main
13 m4a audio only │   32k https │ audio only        mp4a.40.5  32k 48k [en] DASH audio, m4a_dash, description
10 m4a audio only │   96k https │ audio only        mp4a.40.2  96k 48k [en] DASH audio, m4a_dash, main
12 m4a audio only │   96k https │ audio only        mp4a.40.2  96k 48k [en] DASH audio, m4a_dash, description
9  mp4 320x180    │  160k https │ avc1.64000c  160k video only         DASH video, mp4_dash
8  mp4 640x360    │  360k https │ avc1.64001e  360k video only         DASH video, mp4_dash
7  mp4 640x360    │  480k https │ avc1.64001e  480k video only         DASH video, mp4_dash
6  mp4 640x360    │  800k https │ avc1.64001e  800k video only         DASH video, mp4_dash
5  mp4 640x360    │ 1200k https │ avc1.64001e 1200k video only         DASH video, mp4_dash
4  mp4 832x468    │ 1500k https │ avc1.64001e 1500k video only         DASH video, mp4_dash
3  mp4 1280x720   │ 2800k https │ avc1.64001f 2800k video only         DASH video, mp4_dash
2  mp4 1280x720   │ 4500k https │ avc1.64001f 4500k video only         DASH video, mp4_dash
1  mp4 1920x1080  │ 6000k https │ avc1.640028 6000k video only         DASH video, mp4_dash

Fixes #8194
Template

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Fix or improvement to an extractor (Make sure to add/update tests)
[ ] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[x] New feature (It is strongly recommended to open an issue first)

Copilot Summary

🤖 Generated by Copilot at fde2544
Summary
🎧📄🎭

Add support for parsing and using the Role element in DASH manifests to improve format and subtitle selection. Modify yt_dlp/extractor/common.py, yt_dlp/YoutubeDL.py, and test/test_InfoExtractor.py to handle the role metadata. Add a sample manifest file test/testdata/mpd/role.mpd for testing.

parse_mpd_formats
Extracts role from DASH streams
Autumn of choices

Walkthrough

Add a new test case to check the extraction of formats and subtitles with role information from a DASH manifest (link)

Use a sample manifest file test/testdata/mpd/role.mpd that contains four adaptation sets with different roles (link)

Add a line to find the Role element in the adaptation set node of the manifest (link)
Add a line to include the role value in the format dictionary for each representation node (link)
Add a line to include the role value in the subtitle dictionary for each representation node (link)
Add a set of predefined values for the Role element to the YoutubeDL class (link)
Add a branch to handle the format selection expression with a role criterion (link)
Add a column to show the role value of each format in the formats table (link)

I don't agree that core code should be modified so that bv+main works. We should simply add another format field for role so it can be filtered for (e.g. bv+ba[role=main]).
Also, why DRM example output?

@bashonly I've added a new format field instead, ba[role=main] now works

Will review when time permits. In the future, please do not force push

Hi @Diazole,
first of all thanks for this PR, exactly what i've been searching for.
But I want to use the feature for subtitles. Receiving the role value for each subtitle works with this PR, perfect.
But the filter for downloading by value seems to work well for audio/video but not for subtitles, i think this is by design, filter only applies to audio/video.
Download value=dub works, as this is audio:
yt-dlp --allow-unplayable-formats --user-agent 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36' 'https://bakery-pplus.cbs.com/l(fr,it,no,fi,da,sv,es-MX,pt-BR,es-mx,pt-br)/paramountplus/2023/09/18/2264631875979/2380360_cenc_precon_dash/stream.mpd' -f "all[role=dub]"
WARNING: You have asked for UNPLAYABLE formats to be listed/downloaded. This is a developer option intended for debugging.
         If you experience any issues while using this option, DO NOT open a bug report
[generic] Extracting URL: https://bakery-pplus.cbs.com/l(fr,it,no,fi,da,sv,es-MX,pt-BR,es-mx,pt-br)/paramountplus/2023/09/1...econ_dash/stream.mpd
[generic] stream: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] stream: Extracting information
[info] stream: Downloading 2 format(s): 17, 16
[dashsegments] Total fragments: 882
[download] Destination: stream [stream].m4a
[download]   2.6% of ~  81.75MiB at  562.12KiB/s ETA Unknown (frag 23/882)^C
ERROR: Interrupted by user

Downloading subtitles by value is not working:
yt-dlp --allow-unplayable-formats --user-agent 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36' 'https://bakery-pplus.cbs.com/l(fr,it,no,fi,da,sv,es-MX,pt-BR,es-mx,pt-br)/paramountplus/2023/09/18/2264631875979/2380360_cenc_precon_dash/stream.mpd' -f "best[role=subtitle-forced]"
WARNING: You have asked for UNPLAYABLE formats to be listed/downloaded. This is a developer option intended for debugging.
         If you experience any issues while using this option, DO NOT open a bug report
[generic] Extracting URL: https://bakery-pplus.cbs.com/l(fr,it,no,fi,da,sv,es-MX,pt-BR,es-mx,pt-br)/paramountplus/2023/09/1...econ_dash/stream.mpd
[generic] stream: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] stream: Extracting information
ERROR: [generic] stream: Requested format is not available. Use --list-formats for a list of available formats

Is there any other way to download the subtitles by their value (maybe @bashonly can give me a hint) ?
As you see in the mpd there are subtitles with value forced-subtitle and subtitle which i want to download separately.

@Haxe18 subtitles are not video formats. Your request is out of scope for this PR, and IMO it is out-of-scope for the project since you are only providing logs of DRM-protected content

o we really need a field for this? Isn't adding to format_note sufficient?

@diazole I found a way to get the main role as default via the sort order of the format list. Perhaps this justifies the existence of a separate field because it makes sorting easier than with a free text field. Please find a proposal below.
NB: The MPEG-DASH Standard has been updated to ISO/IEC 23009-1:2022. The URL specified in yt_dlp/extractor/common.py is no longer valid. So you might consider this:

The updated standard lists some more role values that you may want to include in the documentation and comments as well.
According to the standard, one Adaption Set may have assigned multiple roles. Is it worth looking into?

diff --git a/yt_dlp/utils/_utils.py b/yt_dlp/utils/_utils.py
index 699bf1e7f..2836ebb69 100644
--- a/yt_dlp/utils/_utils.py
+++ b/yt_dlp/utils/_utils.py
@@ -5320,12 +5320,12 @@ class FormatSorter:

     default = ('hidden', 'aud_or_vid', 'hasvid', 'ie_pref', 'lang', 'quality',
                'res', 'fps', 'hdr:12', 'vcodec', 'channels', 'acodec',
-               'size', 'br', 'asr', 'proto', 'ext', 'hasaud', 'source', 'id')  # These must not be aliases
+               'size', 'br', 'asr', 'proto', 'role', 'ext', 'hasaud', 'source', 'id')  # These must not be aliases
     _prefer_vp9_sort = ('hidden', 'aud_or_vid', 'hasvid', 'ie_pref', 'lang', 'quality',
                         'res', 'fps', 'hdr:12', 'vcodec:vp9.2', 'channels', 'acodec',
-                        'size', 'br', 'asr', 'proto', 'ext', 'hasaud', 'source', 'id')
+                        'size', 'br', 'asr', 'proto', 'role', 'ext', 'hasaud', 'source', 'id')
     ytdl_default = ('hasaud', 'lang', 'quality', 'tbr', 'filesize', 'vbr',
-                    'height', 'width', 'proto', 'vext', 'abr', 'aext',
+                    'height', 'width', 'proto', 'role', 'vext', 'abr', 'aext',
                     'fps', 'fs_approx', 'source', 'id')

     settings = {
@@ -5373,6 +5373,8 @@ class FormatSorter:
         'ext': {'type': 'combined', 'field': ('vext', 'aext')},
         'res': {'type': 'multiple', 'field': ('height', 'width'),
                 'function': lambda it: min(filter(None, it), default=0)},
+        'role': {'type': 'boolean', 'field': 'role',
+                 'in_list': ('main', None), 'not_in_list': None},

         # Actual field names
         'format_id': {'type': 'alias', 'field': 'id'},


[Bug] Issue with NFTs not showing

What happened?
after installing version 1.7.0, all NFTs disappeared
what to do?
Version
1.7.0
What platform are you using?
Windows
What ui mode are you using?
GUI
Relevant log output
No response

This is a duplicate of #1493 - Transferring this issue to chia-blockchain-gui repo, referencing this ticket, & closing.
This is a default setting updated in 1.7.0 but will be corrected soon. For now, please go to the GUI and try these two steps below.

Go to "Settings" and "NFT"
Disable "Hide objectionable content"

The issue isn't the default. But the NFT not being objectable


net: openthread: Added config options for NCP vendor hooks.

New config options that can be used to set vendor hooks for NCP component.
Signed-off-by: Kamil Kasperczyk<EMAIL_ADDRESS>

Actually, there's already a PR open with this change in Zephyr:
https://github.com/zephyrproject-rtos/zephyr/pull/25450
The OpenThread part was already merged, waiting for SHA update for some time now, the author isn't responsive recently.


chore: move to node:stream/web from homegrown API

this PR gets rid of our homegrown AsyncIterableQueue implementation in favour of WHATWG's Streams API.

[x] go over plugins
[ ] implement locking
[x] figure out how to check whether a writer is closed

still doesn't work, there's a rejected promise somewhere


db-extras-zynq7 writing files to wrong path

https://github.com/SymbiFlow/prjxray/blob/6d21194b569b2e320d7b1c4fe87c8db814a823b1/Makefile#L136-L140
Should be database/zynq7/harness/ not database/artix7/harness.

diff --git a/Makefile b/Makefile
index 12be9a41..948fc7f0 100644
--- a/Makefile
+++ b/Makefile
@@ -137,7 +137,7 @@ db-extras-zynq7:
        +source minitests/roi_harness/zybo-swbut.sh && $(MAKE) -C fuzzers part_only
        +source minitests/roi_harness/zybo-swbut.sh && \
                $(MAKE) -C minitests/roi_harness \
-                       HARNESS_DIR=database/artix7/harness/zybo/swbut run
+                       HARNESS_DIR=database/zynq7/harness/zybo/swbut run

 db-check:
        @true

@mithro
Created a PR with a fix (#1051), however I see you have applied the required changes for Zynq7 manually in the database.

The PR has been merged. The issue has been fixed.


Lose internet access when connected

Describe the bug
I set up Outline Server on a Ubuntu 20.04 cloud server and can successfully connect to it using the Outline Client from my Ubuntu 20.04 laptop, but once the Outline Client is connected, I have no web access from the Ubuntu laptop.
By disconnecting Outline I can again access the web from my laptop, and can access the server console over ssh, and successfully traceroute twitter.com etc, so the server does have web access. After disconnecting Outline Client, Firefox on my laptop has trouble waking up on some domains, so I am submitting this using a different browser.
With Outline connected, route -n gives:
Kernel IP routing table
Destination     Gateway         Genmask         Flags Metric Ref    Use Iface
<IP_ADDRESS>         <IP_ADDRESS>       <IP_ADDRESS>         UG    10     0        0 outline-tun0
<IP_ADDRESS>       <IP_ADDRESS>         <IP_ADDRESS>   U     0      0        0 outline-tun0
xx.xx.xx.xx  <IP_ADDRESS>    <IP_ADDRESS> UGH   5      0        0 wlp2s0
<IP_ADDRESS>     <IP_ADDRESS>         <IP_ADDRESS>     U     1000   0        0 wlp2s0
<IP_ADDRESS>    <IP_ADDRESS>         <IP_ADDRESS>   U     600    0        0 wlp2s0

Where xx.xx.xx.xx is the IP of my cloud server.
With Outline disconnected, route -n gives:
Kernel IP routing table
Destination     Gateway         Genmask         Flags Metric Ref    Use Iface
<IP_ADDRESS>         <IP_ADDRESS>    <IP_ADDRESS>         UG    0      0        0 wlp2s0
<IP_ADDRESS>       <IP_ADDRESS>         <IP_ADDRESS>   U     0      0        0 outline-tun0
<IP_ADDRESS>     <IP_ADDRESS>         <IP_ADDRESS>     U     1000   0        0 wlp2s0
<IP_ADDRESS>    <IP_ADDRESS>         <IP_ADDRESS>   U     600    0        0 wlp2s0

I recently tried setting up Wireguard on a cloud server and had the same problem, so this may be due to something local although I have since done a clean install of Ubuntu 20.04 on my laptop.
I wonder then if the issue is because I'm connecting via a mobile hotspot.
It isn't a firewall issue because I have the firewall turned off on both client and server:
ufw status # shows 'inactive' on both.
Desktop (please complete the following information):

OS: Ubuntu 20.04
Browser Firefox or Chromium
Latest versions

UPDATE:
I've just installed Outline client on two Android devices, a phone and a tablet and both have web access while the VPN is connected, so the issue is only with my Ubuntu laptop.

Same thing here, Losing connection when connecting with a fedora 33

Same thing here, Losing connection when connecting with a fedora 33

Same issued on my end but the same client (1.2.0) was working fine on Fedora 32.

On a working Fedora 32 system, I have the following routes:

When Outline is off:
$ route -n
Kernel IP routing table
Destination     Gateway         Genmask         Flags Metric Ref    Use Iface
<IP_ADDRESS>         <IP_ADDRESS>    <IP_ADDRESS>         UG    600    0        0 wlp3s0
<IP_ADDRESS>       <IP_ADDRESS>         <IP_ADDRESS>   U     0      0        0 outline-tun0
<IP_ADDRESS>    <IP_ADDRESS>         <IP_ADDRESS>   U     600    0        0 wlp3s0
<IP_ADDRESS>   <IP_ADDRESS>         <IP_ADDRESS>   U     0      0        0 virbr0

When Outline is on:
$ route -n
Kernel IP routing table
Destination     Gateway         Genmask         Flags Metric Ref    Use Iface
<IP_ADDRESS>         <IP_ADDRESS>       <IP_ADDRESS>         UG    10     0        0 outline-tun0
<IP_ADDRESS>       <IP_ADDRESS>         <IP_ADDRESS>   U     0      0        0 outline-tun0
XX.XX.XX.XX   <IP_ADDRESS>    <IP_ADDRESS> UGH   5      0        0 wlp3s0
<IP_ADDRESS>    <IP_ADDRESS>         <IP_ADDRESS>   U     600    0        0 wlp3s0
<IP_ADDRESS>   <IP_ADDRESS>         <IP_ADDRESS>   U     0      0        0 virbr0

OK, I tried another upgrade to Fedora 33 and I noticed that the route did not change however the connection error pops up again.
On Firefox, I see an error stating "PR_END_OF_FILE_ERROR" so I tried to refresh the Firefox Browser by going to Help --> Troubleshooting Information then click "Refresh Firefox" under "Give Firefox a tune-up") and now I am able to connect just fine.

I guess, I screamed victory too soon: Just a few moments later the connection started acting up and I can no longer connect again on Fedora 33 so I rolled back to Fedora 32 for now while waiting for a fix.

@OmiceyO I found a workaround: when you start the outline client connection to the outline server on Fedora 33, stop the service named systemd-resolved by using the command and the connection will get established. Fedora 33 seems to have that service enabled by default (but disabled on Fedora 32) and it disturb Outline Client from working properly.
@happybeing and @fortuna FYI...

Thanks for the tip. Just testing again I find I don't have the problem in Chromium - I can turn the VPN on and off and Chromium is fine, and I can see my IP address changed using whatismyipaddress.com.
In Firefox, all is well if I close it down and restart after I turn the VPN on or off.
I'm on Ubuntu 20.10, and don't need to do anything with systemd-resolved. It just Firefox, and is solved by restarting Firefox.

Re-opening as with the VPN connected I still don't have internet access from the terminal (e.g. ping google.com is not working unless it is disconnected). Yet as in the last post I do now have web access in the browser, including Firefox after restarting it.

@happybeing Lucky you! Your solution did not work for me, as I stated before so I guess this remains a Fedora 33 issue. I have read many posts about systemd-resolved being enabled on Fedora 33 by default, messing up any VPN connection. So I take it that this particular service is not enabled by default in your environment.

This bug still hasn't been fixed in 2022

Hi Same issue here.
When I used Windows 10, it was fine. After I updated to windows 11, the outline VPN started to have issues.
Once I disconnected Outline VPN, any of my browsers show "no internect connection." while other APP on my laptop still work.
I called Microsoft. They could not solve this issue, saying this may be a issue with Windows 11.
Anybody has some solution in the year of 2022? thanks.

This problem exists in the following versions and I tested it.
Ubuntu 20.04
Ubuntu 21.10
Ubuntu 22.04
PopOS 22.04
Please fix this bug.

@aaaamirabbas I'm pretty sure this is a systemd problem rather than a distro problem

for fix it
1 - sudo ip link set outline-tun0 down

for fix it 1 - sudo ip link set outline-tun0 down (outline-tun0 may be different check ip addr ) 2 - sudo dhclient -r eth0 (eth0 is your internet interface and may be differ check ip addr) 3 - sudo dhclient eth0 now your internet is connected (and you can start outline again) 4 - sudo ip link set outline-tun0 up

How to fix it on windows 11?

for fix it 1 - sudo ip link set outline-tun0 down (outline-tun0 may be different check ip addr ) 2 - sudo dhclient -r eth0 (eth0 is your internet interface and may be differ check ip addr) 3 - sudo dhclient eth0 now your internet is connected (and you can start outline again) 4 - sudo ip link set outline-tun0 up

I also have this problem
I did not work with Ubuntu
How to get outline-tun0 name in ubuntu 18.04?

How to fix it on windows 11?

How to fix it on windows 10?

for fix it 1 - sudo ip link set outline-tun0 down (outline-tun0 may be different check ip addr ) 2 - sudo dhclient -r eth0 (eth0 is your internet interface and may be differ check ip addr) 3 - sudo dhclient eth0 now your internet is connected (and you can start outline again) 4 - sudo ip link set outline-tun0 up

How to fix it on windows 10?

for fix it 1 - sudo ip link set outline-tun0 down (outline-tun0 may be different check ip addr ) 2 - sudo dhclient -r eth0 (eth0 is your internet interface and may be differ check ip addr) 3 - sudo dhclient eth0 now your internet is connected (and you can start outline again) 4 - sudo ip link set outline-tun0 up

How to fix it on windows 10?

This problem exists in the following versions and I tested it.
Ubuntu 20.04 Ubuntu 21.10 Ubuntu 22.04 PopOS 22.04
Please fix this bug.

+ Manjaro KDE

I have the same issue in Linux Mint 21.1 Cinnamon.

I'm facing same issue after restarting network settings on Windows 11, before that it was working fine!

Re-opening as with the VPN connected I still don't have internet access from the terminal (e.g. ping google.com is not working unless it is disconnected). Yet as in the last post I do now have web access in the browser, including Firefox after restarting it.

Outline uses Shadowsocks under the hood. It isn't really a VPN as much as an obfuscated socks5 proxy. The ping (ICMP) protocol can't be tunnelled over a socks5 proxy. Answer from reddit.
It seems "ping" command basically won't  work. It took a while for me to figure out what is going on.

I'm also having troubles with Outline client on Ubuntu 22.04. Even on the connection tab it is shown that my network doesn't have internet access. Also Ubuntu Software store is not working, can't install anything using "apt get" and internet in Chrome is also down. However, internet is alive in Firefox.
Note: everything is working as expected when I disconnect from Outline.

macOS Ventura 13.2


Munge headerlines

This fights with gendocs.  Need to think about it (and this might happen on any such markdown-strictness fix)

fixed now

rebased

@k8s-bot test this please

all greennnnnn

This catches code blocks.  Dammit.  Back to work

Now bypasses code blocks

@nikhiljindal you might want to follow this pattern or risk the same code-blocks problem

LGTM

Passed verify-gendocs.sh
cc @davidopp please merge

LGTM


User registration features

Added Registration Features, and updated .NET Framework

It seems this branch is forked from the expanded-DB-fields branch, which is why there are duplicate changes. I want to review this particular branch more fully, but I would like to see the other branch merge first so that we can get a more narrow focus. I requested changes on the other branch, so when those are addressed this branch will need to be rebased to incorporate those changes before the Login project commits.

The merge conflict should go away once the other branch is merged and this one is rebased.

I see that this is setup for MSSQL instead of Mongo; that's something that would need to be addressed before this could merge.
Before we get that far, though, how exactly can this code be tested/used? I need to know how this integrates into the project as a whole, and how to maintain/extend it moving forward. Can I get some clarification on those things?

Yes, the MSSQL setup was for initial testing and has been what I have been working on addressing for a while. I wasn't expecting to have such a difficult time switching it over, and that's my fault. It's been giving me a huge headache, but I should be coming up on a solution.
As for how it works, when you run it, it allows you to log in or sign up for an account. It hashes the password and provides authentication. Once working with MongoDB, it will store the login data in a collection. It was the idea that we could then use it to only display cards meant for each specific user's group.
As for testing, I was running a locally hosted sql express instance and connecting to that with the connection string in  appsettings.json /program.cs and running it. All of the login data was being stored, retrieved and accessed as needed to log in or register for an account.
It was the plan to have this same testing and usage process once getting it set up for MongoDB, but addressing that issue has been a wall.
I'm unsure if this is what you needed to know. Let me know if you need any additional information.

From: Derek Yohn @.>
Sent: Friday, July 15, 2022 2:07 PM
To: dyohn/mentat @.>
Cc: Collin Burnett @.>; Author @.>
Subject: Re: [dyohn/mentat] User registration features (PR #16)
I see that this is setup for MSSQL instead of Mongo; that's something that would need to be addressed before this could merge.
Before we get that far, though, how exactly can this code be tested/used? I need to know how this integrates into the project as a whole, and how to maintain/extend it moving forward. Can I get some clarification on those things?
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/dyohn/mentat/pull/16*issuecomment-1185828215__;Iw!!Epnw_ITfSMW4!vSLA7t15WQIG77C3V6dCFHe_Y9nLQBGhw8Ys_IvcoNW561dgE6K3s4glZQg3CUY65BlHQgC7bmz9hHJLzvX3RMJIUjqr$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AQZFQBSVRZVGKVHQUI4A5IDVUGZILANCNFSM52LFBDTA__;!!Epnw_ITfSMW4!vSLA7t15WQIG77C3V6dCFHe_Y9nLQBGhw8Ys_IvcoNW561dgE6K3s4glZQg3CUY65BlHQgC7bmz9hHJLzvX3ROXJK80b$.
You are receiving this because you authored the thread.Message ID: @.***>


Unit Export Showing Wrong AMI

Babs noticed that the unit export from Doorway Partners is displaying the incorrect AMI level. I confirmed this with an export this am. See the example for Sango Court (Closed) - all of the export units show 20%, but in partners they are 50 and 60%. Medium/high prio!

QA Notes:

Go to partner site and click "Export to CSV" on the listings page
In the exported unit CSV verify that the AMI percentage matches what is in the listing for each listing


Auto-add SSL to production environments ($20/mo)

SSL certificates are now free with LetsEncrypt 90-day certificates.
They require an endpoint on the domain being certified which is already supported by this plugin for rails apps on Heroku.
Another project has automated the setup and renewal process using an extra Heroku app which is by design for security reasons.
The cost of this addition is $20/mo due to SSL endpoints on Heroku.

Heroku is transitioning to free SSL endpoints:
https://blog.heroku.com/archives/2016/5/18/announcing_heroku_free_ssl_beta_and_flexible_dyno_hours


Handling new insertion on existing SCD table

The SCD library doesn't seems working on handling new rows (insertion).
For example:
Existing source table has rows with id : 1,2,3
After initial Dataform run
The table_scd_updates has rows with id : 1,2,3
When there is a new row in the source table : 1,2,3,4
The current logic doesn't handle the new row (id 4).
File code : dataform-scd/index.js
Code line :
"// Create an incremental table with just pure updates, for a full history of the table."
...
"where ${timestamp} > (select max(${timestamp}) from ${ctx.self()})"
The above logic will only insert new row when the id already exists from the initial SCD creation.
Is it intentional or bug?
How if adding additional clause e.g
"where ${timestamp} > (select max(${timestamp}) from ${ctx.self()})"
OR ${uniqueKey} NOT IN (select distinct ${uniqueKey} from  ${ctx.self()}))

I cannot reproduce the bug.
In my experiments new row (id 4) was inserted correctly.


App submission on Windows Store fails

Just for the note. Windows Store app submission fails on these error.

UTF-8 file encoding
Error Found: The UTF-8 file encoding test detected the following errors:

File plugins\cordova-plugin-x-socialsharing\tests\tests.js is not properly UTF-8 encoded. Re-save the > file as UTF-8 (including Byte Order Mark).

File plugins\cordova-plugin-x-socialsharing\www\SocialSharing.js is not properly UTF-8 encoded. Re-> save the file as UTF-8 (including Byte Order Mark).

Hi, I've removed the file and created a new one - hope that helps!

Sorry for the late response, but the issue is still there.
I noticed that those files are still missing the Byte Order Mark. By adding the UTF8-BOM encoding I got PASS from the Windows Store.

Could you add a PR for this perhaps?

Hmm... now the file encoding passes, but Windows Store validation throws another issue.

Error Found: The supported APIs test detected the following errors:
This API is not supported for this application type - Api=System.Reflection.ICustomAttributeProvider. Module=MSCORLIB, PUBLICKEYTOKEN=7CEC85D7BEA7798E. File=Newtonsoft.Json.dll.

That ddl-file seems to belong to the WP8 platform. So I began to wonder why that file is there as I'm building a Windows Universal app which I guess shouldn't even need that file?
Now I took a slightly deeper look of the "windows" platform folder that Cordova generates and I noticed that there is a folder called "plugins" which contains the whole Socialsharing plugin's repository. Is this something that is meant to be there?

I will check this later today - this file shouldn't ne attached to windows app. Only wondows phone app

https://github.com/EddyVerbruggen/SocialSharing-PhoneGap-Plugin/blob/master/plugin.xml#L82
Hmm... I think the line 82. is the reason why it copies whole src folder to the package(?)

Windows Store validation throws issue for me too.
I have problem with supported APIs.
How it fix?


javascript.suggest.names still working in object literal when disabled

VSCode Version: Version: 1.39.1
OS Version: MacOS 10.15

Steps to Reproduce:

"javascript.suggest.names": false (abc suggestion)
declare object literal e.g. var foo = {  }
when you type something inside the brackets the abc suggestion is still working

Does this issue occur when all extensions are disabled?: Yes

You are likely seeing VS Code's default word based suggestions. Set "editor.wordBasedSuggestions": false to disable these
These are separate settings because their functionality is somewhat different:

Word based suggestions only show up when JS provides no suggestions. They contain all the words in the current document.
Suggest names is provided by JS when we know you are somewhere where an identifier could be used. It should only contain symbol names from the current document


Inconsistent model source times

Describe the bug
The same request switches between different model sourceTimes over the span of a few minutes, returning wildly different forecasts (e.g. 45% vs 18% chance of rain in the hourly forecast)
Expected behavior
Consistent use of models
Actual behavior
The first request (7:36 PM) uses nbm 2024-08-15 12Z and gfs 2024-08-15 06Z, with 45% chance of rain.
The second request (7:38 PM) uses nbm 2024-08-15 15Z and gfs 2024-08-15 12Z, with 18% chance of rain.
The third request (7:41 PM) uses the same models as the first, nbm 2024-08-15 12Z and gfs 2024-08-15 06Z with 45% chance of rain.
Request 1:
{
  "latitude": 42.<redacted>,
  "longitude": -71.<redacted>
  "timezone": "America/New_York",
  "offset": -4,
  "elevation": 28,
  "currently": {
    "time":<PHONE_NUMBER>,
    "summary": "Partly Cloudy",
    "icon": "partly-cloudy-day",
    "nearestStormDistance": 12.32,
    "nearestStormBearing": 270,
    "precipIntensity": 0,
    "precipProbability": 0.58,
    "precipIntensityError": 1.67,
    "precipType": "none",
    "temperature": 19.55,
    "apparentTemperature": 20.34,
    "dewPoint": 17.49,
    "humidity": 0.86,
    "pressure": 1014.98,
    "windSpeed": 1.69,
    "windGust": 2.38,
    "windBearing": 39,
    "cloudCover": 0.65,
    "uvIndex": 0.09,
    "visibility": 16.09,
    "ozone": 348.82
  },
  "minutely": {
    "summary": "Clear",
    "icon": "clear",
    "data": [ ... ]
  },
  "hourly": {
    "summary": "Partly Cloudy",
    "icon": "partly-cloudy-day",
    "data": [
      {
        "time":<PHONE_NUMBER>,
        "icon": "rain",
        "summary": "Rain",
        "precipIntensity": 0.762,
        "precipProbability": 0.45,
        "precipIntensityError": 1.0466,
        "precipAccumulation": 0.0762,
        "precipType": "rain",
        "temperature": 21.73,
        "apparentTemperature": 23.6,
        "dewPoint": 19.04,
        "humidity": 0.85,
        "pressure": 1015.04,
        "windSpeed": 2,
        "windGust": 2.7,
        "windBearing": 350,
        "cloudCover": 0.63,
        "uvIndex": 0.09,
        "visibility": 16.09,
        "ozone": 349.87
      },
      ...
    ]
  },
  "daily": {
    "summary": "Partly Cloudy",
    "icon": "partly-cloudy-day",
    "data": [ ... ]
  },
  "alerts": [],
  "flags": {
    "sources": [
      "ETOPO1",
      "gfs",
      "gefs",
      "hrrrsubh",
      "hrrr_0-18",
      "nbm",
      "nbm_fire",
      "hrrr_18-48"
    ],
    "sourceTimes": {
      "hrrr_subh": "2024-08-15 21Z",
      "hrrr_0-18": "2024-08-15 21Z",
      "nbm": "2024-08-15 12Z",
      "nbm_fire": "2024-08-15 12Z",
      "hrrr_18-48": "2024-08-15 18Z",
      "gfs": "2024-08-15 06Z",
      "gefs": "2024-08-15 12Z"
    },
    "nearest-station": 0,
    "units": "si",
    "version": "V2.1"
  }
}

Request 2:
{
  "latitude": 42.<redacted>,
  "longitude": -71.<redacted>,
  "timezone": "America/New_York",
  "offset": -4,
  "elevation": 28,
  "currently": {
    "time":<PHONE_NUMBER>,
    "summary": "Partly Cloudy",
    "icon": "partly-cloudy-day",
    "nearestStormDistance": 0,
    "nearestStormBearing": 0,
    "precipIntensity": 0,
    "precipProbability": 0.17,
    "precipIntensityError": 1.67,
    "precipType": "none",
    "temperature": 19.53,
    "apparentTemperature": 20.27,
    "dewPoint": 17.51,
    "humidity": 0.86,
    "pressure": 1014.96,
    "windSpeed": 1.71,
    "windGust": 2.4,
    "windBearing": 34,
    "cloudCover": 0.68,
    "uvIndex": 0.29,
    "visibility": 14.02,
    "ozone": 346.85
  },
  "minutely": {
    "summary": "Clear",
    "icon": "clear",
    "data": [ ... ]
  },
  "hourly": {
    "summary": "Partly Cloudy",
    "icon": "partly-cloudy-day",
    "data": [
      {
        "time":<PHONE_NUMBER>,
        "icon": "partly-cloudy-day",
        "summary": "Partly Cloudy",
        "precipIntensity": 0.5081,
        "precipProbability": 0.18,
        "precipIntensityError": 1.0466,
        "precipAccumulation": 0.0508,
        "precipType": "rain",
        "temperature": 22.04,
        "apparentTemperature": 24.24,
        "dewPoint": 19.1,
        "humidity": 0.84,
        "pressure": 1015.04,
        "windSpeed": 1.6,
        "windGust": 2.6,
        "windBearing": 90,
        "cloudCover": 0.66,
        "uvIndex": 0.62,
        "visibility": 16.09,
        "ozone": 347.46
      },
      ...
    ]
  },
  "daily": {
    "summary": "Partly Cloudy",
    "icon": "partly-cloudy-day",
    "data": [ ... ]
  },
  "alerts": [],
  "flags": {
    "sources": [
      "ETOPO1",
      "gfs",
      "gefs",
      "hrrrsubh",
      "hrrr_0-18",
      "nbm",
      "nbm_fire",
      "hrrr_18-48"
    ],
    "sourceTimes": {
      "hrrr_subh": "2024-08-15 21Z",
      "hrrr_0-18": "2024-08-15 21Z",
      "nbm": "2024-08-15 15Z",
      "nbm_fire": "2024-08-15 12Z",
      "hrrr_18-48": "2024-08-15 18Z",
      "gfs": "2024-08-15 12Z",
      "gefs": "2024-08-15 12Z"
    },
    "nearest-station": 0,
    "units": "si",
    "version": "V2.1"
  }
}

Request 3:
{
  "latitude": 42.<redacted>,
  "longitude": -71.<redacted>,
  "timezone": "America/New_York",
  "offset": -4,
  "elevation": 28,
  "currently": {
    "time":<PHONE_NUMBER>,
    "summary": "Partly Cloudy",
    "icon": "partly-cloudy-day",
    "nearestStormDistance": 14.05,
    "nearestStormBearing": 270,
    "precipIntensity": 0,
    "precipProbability": 0.59,
    "precipIntensityError": 1.67,
    "precipType": "none",
    "temperature": 19.51,
    "apparentTemperature": 20.25,
    "dewPoint": 17.55,
    "humidity": 0.86,
    "pressure": 1014.92,
    "windSpeed": 1.76,
    "windGust": 2.44,
    "windBearing": 27,
    "cloudCover": 0.65,
    "uvIndex": 0.09,
    "visibility": 16.09,
    "ozone": 348.67
  },
  "minutely": {
    "summary": "Clear",
    "icon": "clear",
    "data": [ ... ]
  },
  "hourly": {
    "summary": "Partly Cloudy",
    "icon": "partly-cloudy-day",
    "data": [
      {
        "time":<PHONE_NUMBER>,
        "icon": "rain",
        "summary": "Rain",
        "precipIntensity": 0.762,
        "precipProbability": 0.45,
        "precipIntensityError": 1.0466,
        "precipAccumulation": 0.0762,
        "precipType": "rain",
        "temperature": 21.73,
        "apparentTemperature": 23.6,
        "dewPoint": 19.04,
        "humidity": 0.85,
        "pressure": 1015.04,
        "windSpeed": 2,
        "windGust": 2.7,
        "windBearing": 350,
        "cloudCover": 0.63,
        "uvIndex": 0.09,
        "visibility": 16.09,
        "ozone": 349.87
      },
      ...
    ]
  },
  "daily": {
    "summary": "Partly Cloudy",
    "icon": "partly-cloudy-day",
    "data": [ ... ]
  },
  "alerts": [],
  "flags": {
    "sources": [
      "ETOPO1",
      "gfs",
      "gefs",
      "hrrrsubh",
      "hrrr_0-18",
      "nbm",
      "nbm_fire",
      "hrrr_18-48"
    ],
    "sourceTimes": {
      "hrrr_subh": "2024-08-15 21Z",
      "hrrr_0-18": "2024-08-15 21Z",
      "nbm": "2024-08-15 12Z",
      "nbm_fire": "2024-08-15 12Z",
      "hrrr_18-48": "2024-08-15 18Z",
      "gfs": "2024-08-15 06Z",
      "gefs": "2024-08-15 12Z"
    },
    "nearest-station": 0,
    "units": "si",
    "version": "V2.1"
  }
}

API Endpoint
Production
Location
Massachusetts
Other details
No response
Troubleshooting steps

[X] I have searched this repository and Home Assistant Repository to see if the issue has already been reported.
[X] I have read through the API documentation before opening this issue.
[X] I have written an informative title.

Hi,
Thanks for opening this issue, it's an interesting one! What's happening here is that I have two back-end servers for redundancy that are ingesting model files, and while they're mostly in sync, there is a bit of wiggle between them, which is what you're seeing. It's an unusually large jump for precipitation probability between runs, but can sometimes happen depending on the run.
What's strange here is that the AWS load balancer out front should route requests to the same host, so shouldn't bounce back and forth like this. Can I ask if you're accessing this from behind a VPN, or how you're making the calls?

Quick additional thought: I'm pushing out v2.1.1 with an additional header giving a node-id to show which node it's from. I've wanted to add this for my own troubleshooting for a while, so this was the reason to do it!

While I'm not seeing this over a span of a couple of minutes I'm also seeing different NBM runs occasionally just querying the API in my browser.
I've also noticed that the NBM and GFS source times have gotten stuck again with the latest runs being from yesterday.
"sourceTimes": {
  "hrrr_subh": "2024-08-16 12Z",
  "hrrr_0-18": "2024-08-16 11Z",
  "nbm": "2024-08-15 12Z",
  "nbm_fire": "2024-08-15 12Z",
  "hrrr_18-48": "2024-08-16 06Z",
  "gfs": "2024-08-15 06Z",
  "gefs": "2024-08-16 06Z"
},

When I looked last night the production API was the only one which was stuck as the development endpoint was working fine but this morning both are stuck with outdated runs.

I saw it both across multiple cURL calls and from simply opening the URL in my browser. The examples pasted above were from the latter. I'm not accessing from behind a VPN or anything.
I initially noticed the difference from discrepancies between an application running on a VPS and local testing, which sounds like it's not unexpected, due to the load balancer. However it seemed like once the local instance switched backend servers, the remote one did as well on the next call. I didn't test this extensively so

I'm assuming the intermittent internal server errors and the other weird glitches on the API endpoint are due to you fixing this issue?

Yup, exactly that.  API endpoint should be stable now though!

Yup, everything seems stable now. Has this issue been fixed or should be leave it open for the weekend to see if it pops up again?

The servers were restarted Saturday night which fixed the inconsistent model source times and version number. I held off initially closing this one to make sure the source times and version number have been stable which it has. Will close this issue for now but if it pops up again we can re-open and investigate.

@alexander0042 This seems to be happening again.
Sometimes I see:
"sourceTimes": {
  "hrrr_0-18": "2024-08-26 14Z",
  "nbm": "2024-08-26 12Z",
  "nbm_fire": "2024-08-26 06Z",
  "hrrr_18-48": "2024-08-26 12Z",
  "gfs": "2024-08-26 06Z",
  "gefs": "2024-08-26 06Z"
},

and other times I see the updated run times:
"sourceTimes": {
  "hrrr_subh": "2024-08-26 20Z",
  "hrrr_0-18": "2024-08-26 19Z",
  "nbm": "2024-08-26 18Z",
  "nbm_fire": "2024-08-26 12Z",
  "hrrr_18-48": "2024-08-26 18Z",
  "gfs": "2024-08-26 12Z",
  "gefs": "2024-08-26 12Z"
},

Good catch, and fixing this now avoided an outage! I was doing some more work in support of self hosting/ improving performance by merging the syncing and response containers; however, one of the restarts sort of corrupted the file system and prevented ingests. I've restarted the misbehaving instance, so should be all synced up in about 30 minutes or so

I'll keep an eye on it thanks. If you're curious this is what the history graph shows for the NBM update time sensor:

Seems to be good now so will close.

@alexander0042 Seeing the issue again this evening. Seeing a mix of V2.2 and V2.3 but I see no difference between the two versions besides the source differences
"sourceTimes": {
      "hrrr_subh": "2024-09-13 00Z",
      "hrrr_0-18": "2024-09-12 23Z",
      "nbm": "2024-09-12 23Z",
      "nbm_fire": "2024-09-12 18Z",
      "hrrr_18-48": "2024-09-12 18Z",
      "gfs": "2024-09-12 18Z",
      "gefs": "2024-09-12 18Z"
    },
    "nearest-station": 0,
    "units": "ca",
    "version": "V2.2"

V2.3 with outdated runs and it's also missing HRRR subhourly:
"sourceTimes": {
      "hrrr_0-18": "2024-09-12 18Z",
      "nbm": "2024-09-12 15Z",
      "nbm_fire": "2024-09-12 12Z",
      "hrrr_18-48": "2024-09-12 18Z",
      "gfs": "2024-09-12 12Z",
      "gefs": "2024-09-12 12Z"
    },
    "nearest-station": 0,
    "units": "ca",
    "version": "V2.3"
  }


SSIS ODBC Destination error no schema ( no collumn info )

I have installed ODBC driver ver <IP_ADDRESS>23 ( ANSI / UNICODE ) on Windows 10 ver 19.03.
I have Visual Studio Community 16.3.6 with SSIS extentions installed.
On a dataflow Task with ODBC Destination with a DSN ( unicode ) to ClickHouse server to a database with a table on LOG engine , no collumns are present in the designer !
if collumns created manual on the ODBC Destination the execution of the datalow fails with error , no  collumn info on ODBC Destination.

Probably duplicate: https://github.com/ClickHouse/clickhouse-odbc/issues/176  (but 176 is in Russian, so let's keep both)

Hi ,
any news on this ?

Issues mentioned here, with a good chance, are fixed by https://github.com/ClickHouse/clickhouse-odbc/pull/211 once it is merged.

Please try https://github.com/ClickHouse/clickhouse-odbc/releases/tag/v<IP_ADDRESS>91216

Hi , thank you for your effords.
It seems that now I can't get a data preview , and no collumns info , as it was before.
I receive from #176  updates and it states that the problems exists because of SSIS bug !!. I'am using SSIS with 3-4 different database vendors and SSIS has no problem !!!. If you can fix the behavior of the ODBC drivers it wil be a major plus for this awesome product.

It seems that now I can't get a data preview , and no collumns info , as it was before.

Could you tell us which version of driver you were able to get column data exactly?
Also, as stated in #176, the preview will work for tables from the database that matches the database from DSN. Were you able to preview data from table of any other database with that old version? Your accurate answers will help us to roll out fixes much sooner. Thanks!


[AutoDiff] Fix cross-module JVP/VJP method thunking logic.

Fix hole in cross-module method self-ordering thunk logic.
Use mangled symbol for JVP/VJP methods in SILFunctionBuilder and during TBDGen.
Add cross-module test.
Resolves TF-619.

@swift-ci Please test tensorflow


Update IBM Cloud SDK packages & capiIBM image

What this PR does / why we need it:
Updated IBM Cloud go SDK packages.
Updated capiIbm image.
Few code changes due to the API changes on updating the SDKs.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[x] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/cc @mkumatag

/retest-required

@mkumatag removed the changed did with accountID, only one change on passing pointer for cloud connection id while creating DHCP server.
Please have a look.

/assign @enxebre

/retest-required

/approve


Update CTRL panel "+Add CTRL" widget button to match Flow Chart "+Add Node" button

This new UI for adding nodes is really nice:

We should match it in the CTRLs tab for this UI:

"+Add CTRL" should also be present and automatically switch the CTRL panel into edit mode. It's annoying and not intuitive to toggle the edit switch every time.
To make room for the "+Add CTRL" button, we'll need to create some margin on the left of the CTRLS canvas here:

should also write a docs page as part of this PR
https://github.com/flojoy-io/docs/issues/23

Here's a lo-fi mock to clarify where each widget goes in the sidebar:

came across this Figma that suggests input/output tabs on top - seems p. nice

https://www.figma.com/file/aKeN0DN5b1wy8My5Zap0Wt/Flojoy

came across this Figma that suggests input/output tabs on top - seems p. nice

https://www.figma.com/file/aKeN0DN5b1wy8My5Zap0Wt/Flojoy

Instead of adding a margin on the left, I will simply add the button on top like in the Figma :)

merged into the style-refactor :)


"A few" is not well defined for this challenge and the check is binary.

Challenge Change the Font Size of an Element has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.99 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
 Please use the help chat room or try looking through the forum for help with a specific challenge.
Happy Coding.
@dhcodes  this issue can be closed.

@ajain17, no need to keep tag @dhcodes.   Issue mods will check any issue and close it.


Can't open video renders

Describe your bug here. Mind you, if you are modding, please try testing your bug in a clean version of the engine instead, this ensures the bug is actually caused by the engine itself. Also, if you're using an older version of JSE, please try the latest action build in the Actions tab or the latest release build. The issue has probably been fixed since that version. Oh yeah, and also be sure to check the pinned 'JS Engine: Known Issues' issue, to double check if your issue hasn't already been found!
When I render a song, I can't play the rendered video because Windows complains about the codec being 'unsupported'.
Is this a problem with FFmpeg or JS Engine?
Command Prompt/Terminal/Crash logs (if existing)
No response
Have you identified any steps to reproduce the bug? If so, please describe them below in as much detail as possible. Use images if possible.
On Windows 11 23H2, the rendered video just won't play, it complains about an incompatible codec.

Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No, I just wanted to render a song but it won't let me play the rendered video. I am not trying to make a mod.
If you use 1.12.0 or earlier, did you have Optimized Chart Loading turned on?
None
Did you check for any similar issues to what you're reporting? Check, then come back here. If there is a similar issue, then do not report the issue, otherwise it will be marked as a duplicate.


Added Lightshot

Add Lightshot (https://app.prntscr.com/en/index.html) it's an easy to use screenshot tool.

OnScreen Editing
Image-Upload Feature

Apologies, shareX is much better. We'll keep it that way. :)


Bug fix for Pipek Mezey + Stability check using Jacobi sweep

Bug fix for Pipek Mezey
This PR fixes issue #1984. In particular, the missing factor of 2 in both gradient and Hessian is restored. The erroneous h_diag for exponent = 4 is fixed (as a result, some calculations with exponent = 4 require fewer cycles to converge with the new code). The new implementation supports arbitrary exponent (not limited to 2 and 4). Tests are added that examine if the new code generates gradient and Hessian that agree with finite difference.
A bug in the atomic initial guess (in boys.py) is also fixed: the molecular s1e was used for the pbc case, which is fixed now.
Stability check of PM orbitals using Jacobi sweep
The second-order CIAH algorithm along with the atomic initial guess used by the PM code can sometimes result in convergence to a local minimum with orbitals that have high symmetry. This occurs for systems as simple as the occupied orbitals of an isolated water molecule. This PR also implements a Jacobi sweep algorithm that can break the said high symmetry in the localized orbitals and help escape from local minima. The cost is roughly two calls of atomic_pops(), which is very modest. The orbitals it generates serve as a better initial guess for PM optimization. A typical use case is
mlo = PM(mol, mo_coeff)
mlo.kernel()
isstable, mo_new = mlo.stability_jacobi()
Whie not isstable:
    mlo.kernel(mo_new)
    isstable, mo_new = mlo.stability_jacobi()

Usually one Jacobi check is enough for the subsequent PM optimization to converge to a "stable" minimum (stable w.r.t. Jacobi rotations). Two localized orbitals from water/cc-pVDZ are shown below.

I think the CIAH minimization converges to the local minimum probably due to the symmetry. The initial guess in your example are symmetry adapted orbitals. If the initial guess breaks the symmetry in molecular orbitals, the CIAH code should also converge the results the same to the Jacobi sweep.

@sunqm thanks! And completely agreed on the initial guess 👍

Is  #820 also affected by this?
The issue with wrong symmetry is well-known and affects most solvers: orbitals of different symmetries can't mix in the localization. In this case the standard solution is to slightly perturb the initial guess with a random matrix. Should this be done by default?

@susilehtola Thanks for the note! I could reproduce #820 and verified that using converged PM orbitals from the this PR (which break the symmetry) as initial guess for Boys localization resolves the problem in #820. This is an indirect solution. A direct solution may use jacobi-sweep Boys to break the symmetry (PR #1648).


HTYI: Voting options are clipped if the viewport is wider than ~1600px

In a full 8-player game of #HTYI, voting options become hidden once the screen gets too wide. It feels counterintuitive that having a larger screen displays less content.

@esalling23 Did you fix this?


No such file or directory: '/opt/rd/apps/ComfyUI/custom_nodes/ComfyUI_LayerStyle/RMBG-1.4/model.pth'

same here,there's no "RMBG-1.4" folder at all..I added this folder manually and put "model.pth" file in it,still not work..

same here,there's no "RMBG-1.4" folder at all..I added this folder manually and put "model.pth" file in it,still not work..

If the file is downloaded according to the instructions and placed in the correct location, this prompt will not appear.

Thanks ,I fixed the issue,I mis-spell the file name just now...

same here,there's no "RMBG-1.4" folder at all..I added this folder manually and put "model.pth" file in it,still not work..

If the file is downloaded according to the instructions and placed in the correct location, this prompt will not appear.

Thanks,I fixed the issue.I mis-spelled the file name at first

same here,there's no "RMBG-1.4" folder at all..I added this folder manually and put "model.pth" file in it,still not work..

If the file is downloaded according to the instructions and placed in the correct location, this prompt will not appear.

I am using RUNdiffusion to use, the think I only need is to download this by using github link or download at the manager part.
May I know what should I do if I need to use it. Thank you


missing code in slices text?tour: [REPLACE WITH SHORT DESCRIPTION]

Context: https://tour.golang.org/moretypes/7
Change the title above to describe your issue and add your feedback here, including code if necessary

Shouldn't their be code follwing this text:
This selects a half-open range which includes the first element, but excludes the last one
e.g. a[0:]


Consider removing Cognitive Services from documentation

It is part of the Applied AI effort.

The relationship between the Form Recognizer service and the Cognitive Services still seems a bit vague. The service docs haven't been completely updated and Cognitive Services resources can still make use of Form Recognizer. All languages decided to keep documentation as it is for the time being, but this issue may be revisited later. Moving it to the backlog.


bump browserify to ^9.0.3

Not just for funsies -- this updates the native modules, such as the Streams API
Fixes #22

:smile_cat:


1123: Add oberallgaeu

resolves #1123
insert into regions values (<lastID>, 1, '09780','https://ehrenamtskarte.bayern.de','Oberallgäu','Landkreis', null, false);

region was created in staging and production


Enabled local_mode in solo.rb.erb

PR for #353

still missing

I'm not sure if chef-zero works as expected. AFAIK it is incompatible wich chef-solo-search and will crash chef-zero (chef-solo local mode) converges, at least this is what I'm run into when using test-kitchen with chef-zero if a cookbook dependency injects chef-solo-search (which knife-solo afaik does by default when uploading the cookbooks).
https://github.com/pspdfkit-ops/chef-solo-search/commit/02ff543f09915a497e0c2c1a5e7dbf63af9e2620

If you provide some basic steps to test it (how/what files to change) then i can test tomorrow if search is working as expected.

If nothing else, chef zero requires that roles & environment files be json rather than ruby. That's something I'd like to get ironed out (e.g., have a pre-processing step that does the conversion) before supporting local mode from knife-solo.
We also need to add logic to remove the search cookbook when running local mode.

Oh, and there's the question of what to do about node writes. When running in local mode do we want to copy the resulting node files back to the client?
If not people might be surprised when their node attributes are not saved or subsequent runs.

@matschaffer wrting results back sounds like a good idea.
If you publish the feature without that function I want not complain either. I would really like to see local mode included soon because we monkey patch knife-solo right now which is not very user friendly.
We have a lot of cookbooks who look like this:
# search for all items in the 'components' data bag and loop over them
search(:components, "cookbook:myapache AND hostname:#{hostname}") do |component|
  fail 'missing id' unless component['id']

end

They work just fine, we do not see any crashes maybe because we do not include chef-solo-search.

knife-zero for example writes back all node attributes which is nice but also some kind of over-verbose (all ohai attributes…lots of changes on each converge )

@florian-asche just create a blank cookbook with a dependency to 'chef-solo-search' in its metadata. then try to converge it using chef-zero (e.g. using test-kitchen). Due to the file structure of chef-solo-search the "upload" to the in-memory chef-server will fail (or some parts of the cookbook will be missing).

@rmoriz ouch. Perhaps the initial implementation shouldn't worry about getting node data back to the client then.
Wonder if there's something we could do to at least ensure the file on the host doesn't get clobbered so it can be used on the next run.

@matschaffer chef-client can white-list/black-list attributes it sends back to chef-server (chef-zero). automatic_attribute_whitelist  in https://docs.chef.io/config_rb_client.html

Not really. I can't merge as is since I pretty sure it'll break anyone not already using json roles & environments, potentially other issues too.
I'm all for chef zero but it'll have to be a more concerted effort.


fix(navcontroller): don't set transitioning if no transition required

Short description of what this resolves:
Inserting or removing a view other than the active view does not require a transition and navcontroller incorrectly sets itself to isTransitioning() true.
Changes proposed in this pull request:
Don't set transitioning true when not required.
Ionic Version: 3.5
Fixes: #12267

Fixed already by 48b3243689cc728fcce566b6fc32ab6f134db046.


update doc rst and add nbsphinx support

merge it first，i will calibrate the other doc structure next time


Blurry Text with anchor.set

Okay, I finally found what was causing this issue. (If it's a bug, or maybe my fault for not positioning text correctly, not entirely sure)
Here is the jsfiddle:
http://jsfiddle.net/Dillybob/u3mGL/23/
With anchor.set(0.5, 0.5)

Without it:

Just curious, as pertinent to this topic:
http://www.html5gamedevs.com/topic/15162-phasertext-smoothness-issue-text-appears-blurred/?p=85859
I am curious if this is a bug and if maybe the text should not be blurred like that regardless of the position?

I have faced same problem, may be its a bug hope it will fix soon.

Did you try setting game.renderer.renderSession.roundPixels to 'true'? For me that fixes the problem in your fiddle.

wow carst3n...
That is awesome, thank you for this. I would of never found out lol

No problem. And yeah, I've also spend quite a bit of time on this. Now I just drop it into the init/create function of every project. The flag just needs to be set once and then you can completely forget about it.
So my vote for this issue is: make roundPixels default to 'true'. Having roundPixels set to 'false' causes all kinds of strange blurriness in both WebGL and Canvas.

Nah I can't default those, sorry. Changes too much.
Also this issue isn't a result of using anchor 0.5, it's because of the resulting sub-pixel rounding. You could easily have a Text object with anchor 0.5 that renders just fine, because it's width is divisible by 2 evenly.

@carst3n Just had the same issue. You saved my day. I was working on a Retina Mac Book before and it looked fine. Then I switched to a Windows and everything was broken in terms of fonts that seem blurry. With that option enabled everything looks super nice again. Thank you so much!


Java Path value causes non-termination error in clerk/show!

Previously reported on Clojurians; recording here for posterity as requested by @mk.
Description
Using java.nio.Path objects as values in a notebook causes a notebook to never successfully evaluate. It never reaches the Clerk evaluated 'src/clerk_errors.clj' in ... ms step.
Expected behavior
The notebook renders correctly using the string representation of a Path. Failing that, an error message is at least thrown somewhere so I don't have to manually interrupt notebook evaluation.
Reproducing the issue
(ns clerk-errors
  (:require [clojure.java.io :as io]
            [babashka.fs :as fs]
            [nextjournal.clerk :as clerk]))

;; Notebook demonstrating errors Clerk has with Java values
;;

;; # `Path`

(str (fs/path "something"))
(with-out-str (println (fs/path "something")))
;; String and printed representations work fine; the `Path` itself will cause values to hang.
; (fs/path "something")

If the above fs/path call is uncommented and evaluated with clerk/show!, the notebook hangs, never evaluating. Regular evaluation works fine.

@jackrusher also asked if I observe any browser console errors when this happens. I haven't seen any alongside this issue, in part because it doesn't seem to me like values are even getting across the JVM <> notebook boundary.

@respatialized thanks for the report, I can reproduce it on my end.

The problem is nextjournal.clerk.analyzer/exceeds-bounded-count-limit? never terminating when getting a path:
https://github.com/nextjournal/clerk/blob/80c646d892d4d81a3e1b2832c713a3f50e43dbc2/src/nextjournal/clerk/analyzer.clj#L368-L378
See
(seq (.toPath (clojure.java.io/file "something")));;=> (#object[sun.nio.fs.UnixPath 0x5aa4c178 "something"])

(take 10 (tree-seq seqable? seq (.toPath (clojure.java.io/file "something"))))
;;=> (#object[sun.nio.fs.UnixPath 0x7307e110 "something"] #object[sun.nio.fs.UnixPath 0x35cacee7 "something"] #object[sun.nio.fs.UnixPath 0x3de508 "something"] #object[sun.nio.fs.UnixPath 0x4c185902 "something"] #object[sun.nio.fs.UnixPath 0xdb3ef0 "something"] #object[sun.nio.fs.UnixPath 0x630565fa "something"] #object[sun.nio.fs.UnixPath 0xc7fc937 "something"] #object[sun.nio.fs.UnixPath 0x22141f3 "something"] #object[sun.nio.fs.UnixPath 0x5b54d3f9 "something"] #object[sun.nio.fs.UnixPath 0x13b647dc "something"])

@respatialized the issue should be fixed with the commit above which just landed in main. Thanks again for the report.


Fix isStolen text to appear under device name

Fix the You have reported this device stolen ...  -text in user_settings_page.dart to appear under the device name instead of next to it. Make sure it looks nice in smaller screens. You can test it by going to Settings-page in your emulator and click Report stolen button to make the text appear.

Hi, I would like to work on this issue.

Hi, I would like to work on this issue.

Sure, go ahead.


Missing type exports

Pyright is complaining about missing type exports as part of the distributed package (well, git repo):

Hello Johan, with f29be0d, the issue regarding missing type exports in the distributed package should be fixed. Can you update to the latest version and check again?

Fixed.


should study.exists check by id?

Currently you instantiate a study by Study(id_) but you check if a study exists by Study.exists(study_title), which is confusing. What we want to do? Everything by id_, study_title or both (exists_by_id).

My vote goes to both, because we have said that cannot be duplicate study titles on the DB.

Could we just get rid of the ID then? If the study title is a natural
primary key then there isn't a need for both, right?
On Sun, Apr 19, 2015 at 2:04 PM, josenavas<EMAIL_ADDRESS>wrote:

My vote goes to both, because we have said that cannot be duplicate study
titles on the DB.
—
Reply to this email directly or view it on GitHub
https://github.com/biocore/qiita/issues/1078#issuecomment-94309395.

Not sure. I agree on the idea but in practice having FKs to the full
name might not be ideal. BTW having unique names is great!

I agree unique names are great. For this issue, however, if you look at Job and Reference objects you see that exists should never take an ID. What exists does is check, given the data it takes in, whether something already exists with that data. This Study exists is consistent with that philosophy.

That is a great point @squirrelo
The exist was used to avoid data duplicates in the DB, e.g. avoid running 2 jobs that have been applied with the same options on the same data; similarly it is used to avoid 2 studies with the same name. In some other cases, like RawData is tricky to use, probably we can use md5 sums to check if it exists or not.
That being said, I think the current implementation follows the expected behavior and this is correct. If any function needs to check for the id, it can use the private function _check_id implemented in the base class.

Closing as it seems that we agree that the current implementation is fine.


⚠ Samsung has degraded performance

In 087935e, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 3027 ms

Resolved: Samsung performance has improved in 613a49c.


loss of GAAN is nan

I believe it is caused by a_[edge_index[0], edge_index[1]]. However, the shape of edge_index is [2,0] due to the usage of NeighborLoader with num_neigh= 0 in base.py, even when the batch size is either 0 or any other number.

Therefore, a_[edge_index[0], edge_index[1]] should be changed to a_.

Thanks for pointing it out. As per Eq. 5 in the original paper, the first $A_{ij}$ represents that only the positive edges should be counted in the loss function. I have fixed the error in num_neigh. It should work now. If you have any further questions, feel free to let us know :)


word2vec Genderify

Transformation to add masculinity or femininity to each of the nouns in a sentence.

Take each nouns in a sentence, compute their word2vec vectors

add man-word2vec-vector to them, (and woman-vector)

then to replace the noun in the sentence with the closest word-vector
to (word-vector + man/woman-vector ).

Hi @RoyRin
I was assigned as one of your reviewers.
Thanks for this contribution!
I am not sure to understand why this transformation could be helpful. Do you have any reference about works related to this? I have usually seen transformations that, given a male noun, adding woman vector and subtracting man vector, results in the feminine version of the original noun, but I have never seen intuitive explainations of adding man or woman vector to a noun.
Moreover, the code prioritize manify_function to womanify_function, see the comment below.
Correctness: All checks have failed
Interface: I think you have chosen the correct interface(s).
Applicable Tasks & Keywords: Why not TEXT_CLASSIFICATION? Keywords are good.
Specificity: The change is general
Novelty: This transformation is not yet implemented in NL-Augmenter.
Adding New Libraries: inflect==5.3.0, gensim==4.0.1, spacy~=3.0.0, toolz==0.11.1
Description: The README is clear and explains what the transformation is aiming to do.
Data and code source: Missing references to related works.
Test Cases: 4 test cases have been added.
Evaluating Robustness: No robustness evaluation has been performed.
Languages other than English:   No

Hi @RoyRin: Please add your transformation name in the test/mapper.py file in the appropriate dictionary. I think the build is failing because of old code. Please pull the code and let's check if the code passes through.

Interesting tranformation, @RoyRin  please address the comments above!

Feel free to reopen after fixing the suggestions!


What would be the FOG endpoint when using a Failover Group listener name connecting from the Public internet

[Enter feedback here]
HI Team
The article does not mention clearly as to what would be the endpoint when using a Failover Group listener name
connecting from a Public endpoint. It's fine when there is a single SQL MI we can use <mi_name>.public.<dns_zone>.database.windows.net:3342
But what about when using a Failover Group FOG, what would be the FOG endpoint & port that we need to use to connect
from the Public internet ?

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

ID: efd9c5c9-0337-f834-4c5f-7199b5f1bc44
Version Independent ID: 2f8374f8-28ff-c86a-38eb-5be8ea4302af
Content: Configure public endpoint - Azure SQL Managed Instance
Content Source: azure-sql/managed-instance/public-endpoint-configure.md
Service: sql-managed-instance
Sub-service: security
GitHub Login: @zoran-rilak-msft
Microsoft Alias: zoranrilak

@Frankd4
Thanks for your feedback! We will investigate and update as appropriate.

Hello @Frankd4, failover group listeners to SQL Managed Instances at present do not extend to public endpoints.
An unofficial workaround would be to change how the two FQDNs resolve for your SQL clients:
..database.windows.net and
..database.windows.net
so that they become CNAMEs to
.public..database.windows.net and
.public..database.windows.net
Then, your clients can target the auto-failover listener and be redirected to the public endpoints instead (note that those use a different port, 3342).

Thanks @zoran-rilak-msft for clarifying this.


Adding My Profile Page

@codervivek5 can you Pleaase assign me the issue

@codervivek5 can you Please assign me the issue

its already assgned

@codervivek5 can you Please assign me the issue

its already assgned

I have sent you the out put in discord can you please check @codervivek5 So that i can push


Add/Restore HTTPS on dweb.link

https isn't working on dweb.link. I had to fall back to regular http

Can we bump priority of this?  Right now third party gateways are ahead of us: https://dapps.earth

Done!


Path for Home component

Home component is not in /src/Home.vue, it's in /src/components/Home.vue

There have been lots of changes in project structure since this issue was opened and is already fixed. @mahnouman It can be closed


Better documentation for reading websocket requests

The documentation for websocket should title this section "Inspecting the Upgrade Request" and include specific language explaining how the user may retrieve and inspect the HTTP upgrade message:
http://www.boost.org/doc/libs/1_66_0/libs/beast/doc/html/beast/using_websocket/handshaking_servers.html#beast.using_websocket.handshaking_servers.passing_http_requests

Addressed in https://github.com/boostorg/beast/commit/ab9a4c66e0cae58c7ded86224491545f3197cf5b.


Fix: Fixed issues with opening certain file formats

Resolved / Related Issues
To prevent extra work, all changes to the Files codebase must link to an approved issue marked as Ready to build. Please insert the issue number following the hashtag with the issue number that this Pull Request resolves.

Closes #12950
Closes #15164

Steps used to test these changes
Stability is a top priority for Files and all changes are required to go through testing before being merged into the repo. Please include a list of steps that you used to test this PR.

Created a zip file named "a.zip"
Created a text file named "a.zip.txt"
Set default text editor to VS Code and confirmed that the file opened correctly
Confirmed the python script @Josh65-2201 created outputs the correct folder

The drawback of this change is that it will eliminate the functionality for cycling through photos. However, since this feature is currently not working as intended, we will need to rewrite the code regardless. See issue #15409.


refactor: entity data acquisition and types

What?
Move core types from generic dictionaries to well-known types.

DclUserProfile
DclAvatarWireFormat
DclWearableEntityDefinition
DclSceneEntityDefinition

In the case DclUserProfile, methods from_godot_dictionary and to_godot_dictionary are remained for storing purporse
The previous step was done in the UserProfile but instead of being modified directly, a Dictionary wrapper was used in the GDScript code.

Codecov Report
Attention: 295 lines in your changes are missing coverage. Please review.

Comparison is base (f782181) 48.06% compared to head (61648af) 48.05%.

:exclamation: Current head 61648af differs from pull request most recent head cddf580. Consider uploading reports for the commit cddf580 to get more accurate results

Files
Patch %
Lines

.../decentraland-godot-lib/src/avatars/avatar_type.rs
48.80%
43 Missing :warning:

.../decentraland-godot-lib/src/dcl/common/wearable.rs
43.47%
39 Missing :warning:

...ust/decentraland-godot-lib/src/avatars/wearable.rs
78.52%
32 Missing :warning:

...ust/decentraland-godot-lib/src/dcl/common/scene.rs
58.57%
29 Missing :warning:

...decentraland-godot-lib/src/avatars/avatar_scene.rs
7.40%
25 Missing :warning:

rust/decentraland-godot-lib/src/dcl/js/runtime.rs
0.00%
22 Missing :warning:

...ecentraland-godot-lib/src/avatars/godot_profile.rs
41.66%
21 Missing :warning:

...godot-lib/src/realm/dcl_scene_entity_definition.rs
71.01%
20 Missing :warning:

...nd-godot-lib/src/realm/scene_entity_coordinator.rs
66.07%
19 Missing :warning:

...ntraland-godot-lib/src/auth/dcl_player_identity.rs
31.81%
15 Missing :warning:

... and 10 more

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #241      +/-   ##
==========================================
- Coverage   48.06%   48.05%   -0.01%
==========================================
  Files         144      152       +8
  Lines       17636    18109     +473
==========================================
+ Hits         8477     8703     +226
- Misses       9159     9406     +247

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

Can we use this PR to add the following base wearables?
urn:decentraland:off-chain:base-avatars:black_glove
urn:decentraland:off-chain:base-avatars:cord_bracelet
urn:decentraland:off-chain:base-avatars:dcl_watch
urn:decentraland:off-chain:base-avatars:emerald_ring

Review:
chat.gd
		var avatar = Global.avatars.get_avatar_by_address(address)
		var avatar_name = (
			avatar.avatar_name if avatar != null else DclEther.shorten_eth_address(address)
		)

avatar.avatar_name doesn't exists

Review:
explorer.gd
		panel_chat.on_chats_arrived(
			[[player.avatar.avatar_id, player.avatar.avatar_name, 0, message]]
		)

avatar.avatar_name doesn't exists


Missing exit code

https://github.com/th3-z/kf-magicked-admin/blob/bbb7a68843f5c5b4091fb340c2f646b51719ba99/magicked_admin/main.py#L74
Causes exception TypeError: required argument 'status' pos 1

solved

Sorry for that. I broke it with the last PR when tinkering with the logging.


Using boost::filesystem in Boost 1.75.0 onwards fails trying to call syscall(__NR_statx, ...)

Windows Build Number
Microsoft Windows [Version 10.0.19042.985]
WSL Version

[ ] WSL 2
[X] WSL 1

Kernel Version
Linux version 4.4.0-19041-Microsoft<EMAIL_ADDRESS>(gcc version 5.4.0 (GCC) ) #488-Microsoft Mon Sep 01 13:43:00 PST 2020
Distro Version
Ubuntu 18.04
Other Software
gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
Boost 1.75.0
Repro Steps
Download, extract and build Boost 1.75.0:
wget https://boostorg.jfrog.io/artifactory/main/release/1.75.0/source/boost_1_75_0.tar.gz
tar -zxf boost_1_75_0.tar.gz
cd boost_1_75_0
./bootstrap.sh --prefix=installed
# This call to b2 will perform some configuration checks including the statx syscall which it will find
./b2 install variant=debug --without-python --without-iostreams --without-mpi

Write a simple program status_test.cpp that uses a boost::filesystem call that relies on statx:
#include <boost/filesystem/operations.hpp>

int main(int argc, char** argv)
{
  boost::filesystem::path FilePath("/etc/fstab");
  auto Status = boost::filesystem::status(FilePath);
  return 0;
}

Compile and run the program (assumes status_test.cpp is in the same directory as the build of boost):
g++ status_test.cpp -g -Iinstalled/include -Linstalled/lib -lboost_filesystem
export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:$PWD/installed/lib
./a.out

Expected Behavior
The call to boost::filesystem::status in the sample application was successful and the program returns 0 with no errors.
Actual Behavior
The actual output of the program is a thrown exception with a misleading error message:
terminate called after throwing an instance of 'boost::filesystem::filesystem_error'
  what():  boost::filesystem::status: Function not implemented: "/etc/fstab"
Aborted (core dumped)

If you step into the call to boost::filesystem::status you will see that the program fails when trying to call the boost::filesystem statx wrapper (boost_1_75_0\libs\filesystem\src\operations.cpp(367)):
inline int statx(int dirfd, const char* path, int flags, unsigned int mask, struct ::statx* stx) BOOST_NOEXCEPT
{
  return ::syscall(__NR_statx, dirfd, path, flags, mask, stx);
}

I think the problem is that the WSL kernel headers indicate that syscall(__NR_statx, ...) will work but the function is not actually implemented.
Diagnostic Logs
No response

I think this may be a similar issue found by nodejs https://github.com/nodejs/node/issues/30860.
This WSL issue may also be related https://github.com/microsoft/WSL/issues/3023.

It looks like the authors of boost::filesystem are adding a workaround to manually disable statx support if necessary (https://github.com/boostorg/filesystem/blob/develop/build/Jamfile.v2#L66) but that is not available in any released version of boost (1.75.0 or 1.76.0) yet.
Also, it would be preferable to be able to use the same boost binary on WSL and native Ubuntu distributions with performance benefits from using statx.


Better Oauth mock for dev

Allow to input custom user data from the Oauth mock step instead of always  replying with "Patrick"

I'd rather work on making the database available easly for dev environements instead


gluestick with deeplsd

How can i use gluestick with deeplsd ?

Hi, I recommend you to use the GlueFactory repository for this. This is the training code code repository for GlueStick. There is already the possibility to use DeepLSD there. You can take the existing config file to use SuperPoint+LSD+GlueStick, and change the line "name: gluefactory.models.lines.lsd" with "name: gluefactory.models.lines.deeplsd".


Configure mini-browser to return to PowerShell 7

Summary of the new feature / enhancement
Hello,
At my organization we have separate admin accounts and windows SSO enabled using 3rd party federation. So, it's not easy to log into admin portals, and very difficult to connect to M365/AZ PowerShell modules with my admin account. Sure, a dedicated admin workstation might help, but security team has delayed that option. Generally, when I need to log into an admin portal, I have to use a private browser to log on. With PowerShell 5, I wrote a function that will open as Administrator and as my admin account, then when I connect to M365/AZ modules the mini-browser already detects my admin account. The behavior is different with PowerShell 7, it opens to my default browser even if I'm running pwsh as my admin account. Due to SSO, it doesn't work because it detects my regular account. I've been using the -UseWindowsPowerShell in PowerShell 7 as a workaround. Although, this doesn't seem to be most ideal since it's enabling lots of compatibility settings and requires PowerShell 5 - all I really need is the mini-browser so it correctly detects the account running pwsh (my admin account). I looked at about_Windows_PowerShell_Compatibility and about_PowerShell_Config but it's not clear (or maybe i missed it) if this is configurable - it doesn't appear to be. So hoping you can add an enhancement.
Thanks,
Tino
Proposed technical implementation details (optional)
Add a switch to Import-Module that allows us to call a mini-browser (such as PowerShell 5 behavior) that excludes all the other compatibility settings in -UseWindowsPowerShell.  Alternatively, provide a way to modify the behavior in powershell.config.json file.

Could you please add more simple repro steps and screenshots? Can you demo the issue with starting locally something like Notepad?

Yes, Let me get that info. I have no issues opening local apps such as notepad, it's only with cloud modules that redirect to a browser for authentication. Let me get some screenshots.

Hello, here are the steps to reproduce. I've included a side by side comparison between my admin account (C---) and my regular account (T---) in both PowerShell 5 and PWSH 7. Since this is public forum, I've blocked out some confidential information. If you would like to screen-share for a live session, let me know.
Implement Seamless SSO with Windows
We have Okta integrated with our Windows 10 systems, but I believe you can implement a similar experience with AAD Seamless SSO. The experience will be a seamless logon experience for the logged-on Windows user when attempting to authenticate into any application that supports this feature. So, while I am logged into Windows 10 as my regular user (non-admin), I cannot log into portal.azure.com or admin.microsoft.com with alternate admin credentials. The work-around for that is to open a private browser and enter admin credentials. This is more difficult to do in PowerShell/PWSH, see below for details. Additionally, PWSH is unable to log on with alternate credentials unless using compatibility feature that proxies to PowerShell (-UseWindowsPowerShell).
SSO Experience with PowerShell 5

Open PowerShell. I wrote a local function to logon as ‘Administrator’ and my admin account in same PowerShell/pwsh session, but here is the main part of code to get you started. For my regular session, I just opened PowerShell shortcut as ‘Administrator’.
$MsgPrompt = "Enter password for Admin account"
$PS_EXE_Chosen = 'PowerShell.exe' # 'pwsh.exe'
$FullUserName = 'contoso\AdminUser' #domain needed for this
[string]$ArgumentList = '-command "Start-Process ' + $PS_EXE_Chosen + ' -Verb RunAs"'
Start-Process -FilePath $PS_EXE_Chosen -Credential (Get-Credential -UserName $FullUserName -Message $MsgPrompt) -ArgumentList $ArgumentList

Connect to M365/Azure in session. In this example, import ExchangeOnlineManagement module, then connect to EXO. Notice that my regular account session detects I am logged into Windows. Even if I click ‘use another account’ in my regular session and specify my admin account, windows sso does not give me opportunity to enter password because it autologins using my account that is logged into windows (regular account).

Observe logon. In our organization, we have okta, so you will see the okta mfa prompt on my admin session. However, in my regular session it just seamlessly sso’d into EXO without allowing me to specify admin credentials.

Confirm accounts logged on. Confirm my admin session is logged in with admin account and confirm regular session did not logon as admin (even though I tried). In my admin session you can see I query another user. In my regular session you can see I my regular account does not have permission to query another user. (Note: In Exchange, default is anyone can log into PowerShell and query their own account.)

SSO Experience with PowerShell 7

Open PWSH. I wrote a local function to logon as ‘Administrator’ and my admin account in same PowerShell/pwsh session, but here is the main part of code to get you started. For my regular session, I just opened PWSH shortcut as ‘Administrator’.
$MsgPrompt = "Enter password for Admin account"
$PS_EXE_Chosen = 'pwsh.exe' # 'PowerShell.exe'
$FullUserName = 'contoso\AdminUser' #domain needed for this
[string]$ArgumentList = '-command "Start-Process ' + $PS_EXE_Chosen + ' -Verb RunAs"'
Start-Process -FilePath $PS_EXE_Chosen -Credential (Get-Credential -UserName $FullUserName -Message $MsgPrompt) -ArgumentList $ArgumentList

Connect to M365/Azure in session. In this example, import ExchangeOnlineManagement module, then connect to EXO. Here is where I need to use compatibility switch (UseWindowsPowerShell) in order to get the mini-browser in my admin session. As I understand, it is proxying a connection to PowerShell, not ideal. Also, notice that my regular account session detects I am logged into Windows and opens the default browser in a large window. Even if I click ‘use another account’ in my regular session and specify my admin account, windows sso does not give me opportunity to enter password because it autologins using my account that is logged into windows (regular account).

Observe logon. In our organization, we have okta, so you will see the okta mfa prompt on my admin session. However, in my regular session it just seamlessly sso’d into EXO without allowing me to specify admin credentials even after I input my admin account. The windows is left open and informing me to close browser.

Confirm which accounts are logged on. Confirm my admin session is logged in with my admin account and confirm regular session did not logon as admin (even though I tried). In my admin session you can see I can query another user. In my regular session you can see I do not have permission to query another user. There is an unrelated error with EXO V3 module in PWSH admin session, you can ignore (just started happening). (Note: In Exchange, default is anyone can log into PowerShell and query their own account.)

@tinojr Thanks for detailed description!
Have you tried Connect-ExchangeOnline -UserPrincipalName  <your_admin_account> fom regular session?

Yes I have, that is overridden by Windows SSO. ;-)

Yes, It'd be great to get Okta support thoughts about the issue. Or we need a repro without the Okta software.

Ok, it seems like my point is getting missed. This is not unique to okta nor to exchangeonlinemanagement module. Seamless SSO is doing what it is designed to do.  The problem is that the authentication behavior has changed between powershell.exe (5) and pwsh.exe (7), hence the known work-around that is called out here -> (https://learn.microsoft.com/en-us/powershell/module/microsoft.powershell.core/about/about_windows_powershell_compatibility?view=powershell-7.3#using-the-compatibility-feature) .  This problem with this work-around is that it proxies all connections back to powershell.exe (5), so it's a little unreliable.  I am hoping someone from powershell development team can add this as a feature request for pwsh - the ability to have the same authentication behavior as powershell.exe (mini-browser ).

The repro steps contain two problematic points:

Proxying is used. In fact, the module does not work in PowerShell 7. This means that we need to solve the problem with EXO dev group.
Third-party soft is used. I am sure that any MSFT group will ask to reproduce the problem without this software. That is why I advised to start with Okta support. (It may turn out that they do not work correctly with .Net Core.)

Only research by the Okta dev group and MSFT EXO dev group can reveal the root of the problem. Until then it is not at all clear that the problem is in the pwsh7 code.

@tinojr - apologies if you felt that we'd missed the ask, that wasn't the case and was trying to help you in finding a suitable workaround in the meantime, other than the one you've listed, as is the case with issue triaging in a repo as busy as this. Especially if the ask requires dev work then I can't say how quickly this would be added to the set of ongoing work.
There is a possibility that this is functionality in EXO module that hasn't been migrated across, and I say that as I believe that the PnP.PowerShell module (used for SharePoint and some teams stuff) does infact prompt with that mini-browser. I could be mistaken here & if so then yes there would be a need for this to be looked as a feature request, if it is at all possible & it may have been pulled due to changes in dotnet, but I don't think this to be the case.

It's a bit perplexing to me that this behavior was changed in Pwsh 7, when there was nothing wrong with using WebView 2 (aka mini-browser) in Powershell 5, perhaps someone could enlighten me?
@tinojr's situation is not unique (I'd wager most well-run organizations have separate accounts for admin access) and why I found this thread. My default web browser is Edge, and uses my 'daily' account. I use Chrome for my 'admin' account, that way I keep things nicely segregated and don't have to constantly log-in to InPrivate/Incognito windows.
Equally, this isn't unique to the Exchange Online PowerShell module, it'll affect all other Microsoft (SharePoint, Graph, etc) management modules.
All this to say, an option should be added to PowerShell 7 to set the authentication behaviour to always use Webview2, if that is the user's preference.

I've just run into the exact same issue as described above. Running PS 7.4.1. MS auth flows are pushed to edge browser that I'm authenticated to using SSO (regular account with no elevated privileges). Even if I select "sign in as a different user", SSO overrides this and persists with regular account. My only workaround at the moment is to force edge to launch in InPrivateMode by default.
Computer\HKEY_LOCAL_MACHINE\SOFTWARE\Policies\Microsoft\Edge
DWORD 32 --> InPrivateModeAvailability ; value = 2
Essentially this means I must use another browser client for regular account. This is highly undesirable behaviour.
Has any progress been made in this space? Does PS 7.5 address this issue?

@tinojr I too am facing the same exact problem you are facing with an almost identical setup. Have you been able to find a solution for this issue?

I've just started trying to use Powershell 7, and have been running into these problems as well.  I use Edge - default profile is my regular user account, I have separate Edge user profiles for admin accounts, and the connect- commands call up my non-privileged-user edge profile window.
I have NOT tried setting Edge to use the most recently opened profile to open hyperlinks, because I NEVER want to open random links with my admin profiles.
I HAVE had success with the -device option to Connect-, since it gives me a device code I can use, but it means I have to:

manually open my admin browser profile
navigate to https://microsoft.com/devicelogin
paste in the device code presented in the Pwsh window
confirm that I wanted to log into the service name
close the now-useless browser tab

vs in Windows Powershell, using WebView2, the first service I connect to pops up a standalone mini-browser I can use to enter admin credentials, subsequent service connections I just select the already-logged-in admin credential and connect.  The mini browser closes and I'm set to work.
Also, not all services have the -Device option - Connect-MicrosoftTeams is the one I ran into today that does not.
My understanding is that Edge is available cross platform, so is WebView2 also cross-platform?  Could WebView2 be called from the Powershell service admin modules in Pwsh like it is in Windows Powershell?  That would help a lot.
Barring that ideal, having a way to configure which browser / browser profile is used for service connection authorization would appear to be important for admins trying to stick to the principle of least priviledge security best practice.


usage without completion sources?

does ncm2 do anything useful without a completion source installed? imo the readme should show a 'naked' example, or say that you really need to install additional plugins.

does ncm2 do anything useful without a completion source installed

No. You need to install completion source as you need.


Raise error on passing wrong options

Rn you can pass any option to declare and it won't error out. This is a problem both because people tend to do typos and because they may think pass incorrect parameters excepting them to affect the deployment (e.g. --model). We still have mistakes regarding this in docs, and people will wonder why commands work initially, but it breaks later.
$ mlem declare env heroku staging
$ mlem declare deployment heroku app \
    --app_name=example-mlem-get-started-app \
    --mlem_is_awesome_arent_it=yes_hooman \
    --env=staging
💾 Saving deployment to app.mlem

Maybe other commands with the magic (build, serve, deploy, apply) are affected as well. We need to check this.

@madhur-tandon @mike0sv could you please TAL? I believe this will cause confusion after release, so better to address this soon.


Have the copy and raw options appear above the paste rather than below

It's a bit of an annoyance to scroll past a 500+ line paste just to be able to copy it easily to clipboard/view raw.

In general some thought could be put into the bars currently above/below pastes. My initial idea was to have as little as possible above the paste. But that was when pastes were still single-files.

It seems to me copy/raw could move above the file, and the meta information (expires at/removal link) to the bottom.

Perhaps a minimize/maximize button could be a solve? It would hide that file allowing you to view the other files, as well as bringing the copy/raw buttons up to the top.


Publish packages in CI and update to crossplane-contrib orgs

Signed-off-by: Jason Tang<EMAIL_ADDRESS>Description of your changes
Update the CI to publish to crossplane-contrib OCI repos in both DockerHub and Upbound registries.
xref https://github.com/crossplane/crossplane/issues/3350
Also updates crossplane.yaml to use the updated metadata annotations per the xpkg spec so the package will render nicely in the Upbound Marketplace.
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
Publish
➜  provider-argocd git:(jastang-update-ci-publish) ✗ VERSION=v0.1.1 BRANCH_NAME=master make -j2 publish
10:06:52 [ .. ] Pushing package xpkg.upbound.io/jastang/provider-argocd:v0.1.1
10:06:52 [ .. ] Skipping image publish for docker.io/provider-argocd:v0.1.1
Publish is deferred to xpkg machinery
10:06:52 [ OK ] Image publish skipped for docker.io/provider-argocd:v0.1.1
xpkg pushed to xpkg.upbound.io/jastang/provider-argocd:v0.1.1
10:07:04 [ OK ] Pushed package xpkg.upbound.io/jastang/provider-argocd:v0.1.1

Verify
➜  provider-argocd git:(jastang-update-ci-publish) crane manifest xpkg.upbound.io/jastang/provider-argocd:v0.1.1 | jq .
{
  "schemaVersion": 2,
  "manifests": [
    {
      "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
      "size": 1005,
      "digest": "sha256:0a99f80d95dbd4196b18482eef1f84b5407599cf276a75c03ea2b3c39cbda395",
      "platform": {
        "architecture": "amd64",
        "os": "linux"
      }
    },
    {
      "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
      "size": 1005,
      "digest": "sha256:3b14ebdc36c23ccb892ab52feb7934f3016e55262f3a14d39a6a24655edc534c",
      "platform": {
        "architecture": "arm64",
        "os": "linux"
      }
    }
  ]
}

TODO:

[ ] confirm dockerhub org crossplane-contrib is ready
[ ] configure the Actions secrets for Upbound before running CI


Securely accept line breaks into the alert

It allows to add line breaks without making the component vulnerable to xss.
I really like this component, but I required to have line breaks into my alertbox. I came across this solution that won't make it vulnerable to xss.

Hi Thanks for pulling request , I really really appreciated your effort, Thanks a bunch to be a contributor in this organization.
I would merge it ASAP after fix the test issue


Fix unclosed details element in 01_basic_component.md

This was breaking the printed version of the book (tested in Firefox and chromium).

Thanks!

Thanks!

You too! I finished reading the book this week, and my overall impressions are good so far.