SyntaxError: missing parentheses

Does someone know how to fix this problem?

Can I see line 109 real quick in a photo?
On Sat, Apr 24, 2021, 9:49 AM mickbos00 @.***> wrote:

Does someone know how to fix this problem?
[image: Pycharm]
https://user-images.githubusercontent.com/83127462/115962597-7d03cb00-a51c-11eb-98c9-c3b5d436457b.PNG
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/TreborNamor/TradingView_Machine_Learning/issues/10,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOHWMRLJYV6Z2VYVY2TEUWTTKLK7DANCNFSM43QG3IMQ
.

On the functions.py file
On Sat, Apr 24, 2021, 10:00 AM Robert Roman @.***> wrote:

Can I see line 109 real quick in a photo?
On Sat, Apr 24, 2021, 9:49 AM mickbos00 @.***> wrote:

Does someone know how to fix this problem?
[image: Pycharm]
https://user-images.githubusercontent.com/83127462/115962597-7d03cb00-a51c-11eb-98c9-c3b5d436457b.PNG
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/TreborNamor/TradingView_Machine_Learning/issues/10,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOHWMRLJYV6Z2VYVY2TEUWTTKLK7DANCNFSM43QG3IMQ
.

Can I see line 109 real quick in a photo?
…
On Sat, Apr 24, 2021, 9:49 AM mickbos00 @.***> wrote: Does someone know how to fix this problem? [image: Pycharm] https://user-images.githubusercontent.com/83127462/115962597-7d03cb00-a51c-11eb-98c9-c3b5d436457b.PNG — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#10>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOHWMRLJYV6Z2VYVY2TEUWTTKLK7DANCNFSM43QG3IMQ .

On the get.py file. It might help to update your python version on the python.org website too. Most syntax errors happen because of outdated python.
…
On Sat, Apr 24, 2021, 10:00 AM Robert Roman @.> wrote: Can I see line 109 real quick in a photo? On Sat, Apr 24, 2021, 9:49 AM mickbos00 @.> wrote: > Does someone know how to fix this problem? > [image: Pycharm] > https://user-images.githubusercontent.com/83127462/115962597-7d03cb00-a51c-11eb-98c9-c3b5d436457b.PNG > > — > You are receiving this because you are subscribed to this thread. > Reply to this email directly, view it on GitHub > <#10>, > or unsubscribe > https://github.com/notifications/unsubscribe-auth/AOHWMRLJYV6Z2VYVY2TEUWTTKLK7DANCNFSM43QG3IMQ > . >

Got it fixed, but now this error arises


Libbitcoin behavior

Libbitcoin is causing some issues. Will leave this ticket open to monitor progress.

ZMQ errors

It uses ZMQ instead of JSON RPC. This is probably more suitable for push based approach (where we receive block notification) rather than our current polling model. See #43.
Every couple of hours we get an Errno::EMFILE) "Too many open files -  error. I attempted to fix this in 28e05abbf028db81d618ebf6e68502bd07d1189d by disconnecting and reconnecting after timeouts. No idea if that really helps.
In the node settings I increased query_workers from 4 to 30. I kept subscription_expiration_minutes = 10, but I don't know if that matters.

Node 1 block behind emails

It often fell 1 block behind causing a email notification. This may be a side effect of polling problems, e.g. if we're waiting for a response too long and meanwhile we poll the other nodes.
I added a few manual peers, including our Bitcoin Core 0.18 node.
Other nodes have this problem too.

Sadly we're still getting Too many open files errors every couple of hours for the rake nodes:poll_repeat task. This is a continuous process, as opposed to rake nodes:poll which spins up a new process every ten minutes through the Heroku scheduler. So  I might try removing libbitcoin from the continous poller and only poll it every 10 minutes.

8856b6e33e481adb13d4718a1ec4de8e5b66e354 drops Libbitcoin during the repeat poll rake task, so now it's updated only once every 10 minutes.

It's manageable for now.


[Bug] BetterSyntax crashing

Addon
BetterSyntax
What happened?
BetterSyntax is crashing when trying to open a channel/DM with a code block message.
What Discord version do you have this issue on?
Stable
Logs
No response
Code of Conduct

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

Fixed in latest version


Pagination QuickJumper bug

English is not my native language; please excuse typing errors.

when i input the first page or the last page, corresponding button not highlighted.

bug confirm, thank you


Maven artifact

Thanks for working on this!
Do you mind publishing this to Maven Central?

yes, that was always the plan, but I wanted some feedback before publishing it. If this is compatible enough with the original macro then I can publish it.


Can't Detect Spotify "prefs" file location.

🔍 Is there already an issue for your problem?

[X] I have checked older issues, open and closed

ℹ Environment / Computer Info
- Spotify version:1.1.84
- Spicetify version: v2.15.0

📝 Description

📸 Screenshots
No response

Not a Marketplace issue
FAQ


ERROR: unable to download video data: IncompleteRead (16 bytes read) even If I use aria2c as an external downloader

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.02.03. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
Latin America
Description
Whenever I try to download something from PlutoTV I get this error:
ERROR: unable to download video data: IncompleteRead (16 bytes read)
Even if I use aria2c as an external downloader, it is worth mentioning that I already updated yt-dlp (from source). I get this error when I try to download using this command
yt-dlp https://pluto.tv/es/on-demand/series/monster-rancher-las-ptv1/season/1/episode/tigre-del-viento-1999-1-5-ptv1
or using this command
yt-dlp --downloader aria2c --no-hls-split-discontinuity -P "Download" https://pluto.tv/es/on-demand/series/monster-rancher-las-ptv1/season/1/episode/tigre-del-viento-1999-1-5-ptv1
I have also tried this command to download the whole season:
yt-dlp -vU --check-formats --downloader aria2c --no-hls-split-discontinuity -P "Download" -o "%(series)s/%(season_number)s - %(season)s/%(episode_number)s - %(episode)s.%(ext)s" "https://pluto.tv/es/on-demand/series/atlantis-series-las"
Verbose log
C:\Users\Charles Dickens\Desktop\ty\yt-dlp-2022.02.03>yt-dlp -vU --check-formats --downloader aria2c --no-hls-split-discontinuity -P "Download" -o "%(series)s/%(season_number)s - %(season)s/%(episode_number)s - %(episode)s.%(ext)s" "https://pluto.tv/es/on-demand/series/atlantis-series-las"
[debug] Command-line config: ['-vU', '--check-formats', '--downloader', 'aria2c', '--no-hls-split-discontinuity', '-P', 'Download', '-o', '%(series)s/%(season_number)s - %(season)s/%(episode_number)s - %(episode)s.%(ext)s', 'https://pluto.tv/es/on-demand/series/atlantis-series-las']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, err utf-8, pref cp1252
[debug] yt-dlp version 2022.02.03 [28469edd7] (source)
[debug] Lazy loading extractors is disabled
[debug] Plugins: ['SamplePluginIE', 'SamplePluginPP']
[debug] Python version 3.9.7 (CPython 64bit) - Windows-10-10.0.19042-SP0
[debug] exe versions: ffmpeg git-2019-10-31-7c872df, ffprobe git-2019-10-31-7c872df
[debug] Optional libraries: Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
Latest version: 2022.02.03, Current version: 2022.02.03
yt-dlp is up to date (2022.02.03)
[debug] [PlutoTV] Extracting URL: https://pluto.tv/es/on-demand/series/atlantis-series-las
[PlutoTV] atlantis-series-las: Downloading JSON metadata
[PlutoTV] 6176be9a829284001a945303: Downloading m3u8 information
[PlutoTV] 6176be9a829284001a945303: Downloading m3u8 playlist
[PlutoTV] 6176be9a829284001a945303: Downloading m3u8 playlist
[PlutoTV] 6176be9a829284001a945303: Downloading m3u8 playlist
[PlutoTV] 6176be9a829284001a945303: Downloading m3u8 playlist
[PlutoTV] 6176be9a829284001a945303: Downloading m3u8 playlist
[PlutoTV] 6176be9a829284001a945303: Downloading m3u8 information
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[download] Downloading playlist: Atlantis
[PlutoTV] playlist Atlantis: Collected 26 videos; downloading 26 of them
[download] Downloading video 1 of 26
[debug] Default format spec: bestvideo*+bestaudio/best
[info] Testing format hls-2453
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 545
[download] Destination: C:\Users\Charles Dickens\Desktop\ty\yt-dlp-2022.02.03\Download\tmp4_vpqr10.tmp
[download] 100% of 10.00KiB in 00:00
[info] Testing format hls-2453
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 545
[download] Destination: C:\Users\Charles Dickens\Desktop\ty\yt-dlp-2022.02.03\Download\tmpz9h1i4ov.tmp
[download] 100% of 10.00KiB in 00:00
[info] 6176be9a829284001a945303: Downloading 1 format(s): hls-2453
[debug] Invoking downloader on "https://siloh-latam-aka.plutotv.net/a61d00_BBC/clip/6176be9a829284001a9452eb_The_Earth_Bull/720p/20211027_095929/hls/0-end/hls_2400.m3u8"
[hlsnative] Downloading m3u8 manifest
[hlsnative] Fragment downloads will be delegated to aria2c
[hlsnative] Total fragments: 545
[download] Destination: Download\Atlantis\1 - Season 1\1 - El Toro de la Tierra.mp4
[debug] aria2c command line: aria2c -c "--console-log-level=warn" "--summary-interval=0" "--download-result=hide" "--file-allocation=none" -x16 -j16 -s16 "--allow-overwrite=true" "--allow-piece-length-change=true" --header "User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/90.0.4430.212 Safari/537.36" --header "Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8" --header "Accept-Encoding: gzip, deflate" --header "Accept-Language: en-us,en;q=0.5" --header "Sec-Fetch-Mode: navigate" "--check-certificate=true" "--remote-time=true" "--show-console-readout=true" --dir ".\Download\Atlantis\1 - Season 1\" "--auto-file-renaming=false" "--file-allocation=none" "--uri-selector=inorder" -i "Download\Atlantis\1 - Season 1\1 - El Toro de la Tierra.mp4.part.frag.urls"
[#a94fca 1.4MiB/1.5MiB(91%) CN:1 DL:269KiB]ERROR: unable to download video data: IncompleteRead(16 bytes read)
Traceback (most recent call last):
  File "C:\Users\Charles Dickens\AppData\Local\Programs\Python\Python39\lib\http\client.py", line 559, in _get_chunk_left
    chunk_left = self._read_next_chunk_size()
  File "C:\Users\Charles Dickens\AppData\Local\Programs\Python\Python39\lib\http\client.py", line 526, in _read_next_chunk_size
    return int(line, 16)
ValueError: invalid literal for int() with base 16: b''

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Users\Charles Dickens\AppData\Local\Programs\Python\Python39\lib\http\client.py", line 576, in _readall_chunked
    chunk_left = self._get_chunk_left()
  File "C:\Users\Charles Dickens\AppData\Local\Programs\Python\Python39\lib\http\client.py", line 561, in _get_chunk_left
    raise IncompleteRead(b'')
http.client.IncompleteRead: IncompleteRead(0 bytes read)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Users\Charles Dickens\Desktop\ty\yt-dlp-2022.02.03\yt_dlp\YoutubeDL.py", line 3086, in process_info
    success, real_download = self.dl(temp_filename, info_dict)
  File "C:\Users\Charles Dickens\Desktop\ty\yt-dlp-2022.02.03\yt_dlp\YoutubeDL.py", line 2801, in dl
    return fd.download(name, new_info, subtitle)
  File "C:\Users\Charles Dickens\Desktop\ty\yt-dlp-2022.02.03\yt_dlp\downloader\common.py", line 440, in download
    ret = self.real_download(filename, info_dict)
  File "C:\Users\Charles Dickens\Desktop\ty\yt-dlp-2022.02.03\yt_dlp\downloader\hls.py", line 262, in real_download
    return fd.real_download(filename, info_dict)
  File "C:\Users\Charles Dickens\Desktop\ty\yt-dlp-2022.02.03\yt_dlp\downloader\external.py", line 40, in real_download
    retval = self._call_downloader(tmpfilename, info_dict)
  File "C:\Users\Charles Dickens\Desktop\ty\yt-dlp-2022.02.03\yt_dlp\downloader\external.py", line 159, in _call_downloader
    dest.write(decrypt_fragment(fragment, src.read()))
  File "C:\Users\Charles Dickens\Desktop\ty\yt-dlp-2022.02.03\yt_dlp\downloader\fragment.py", line 363, in decrypt_fragment
    decrypt_info['KEY'] = decrypt_info.get('KEY') or _get_key(info_dict.get('_decryption_key_url') or decrypt_info['URI'])
  File "C:\Users\Charles Dickens\Desktop\ty\yt-dlp-2022.02.03\yt_dlp\downloader\fragment.py", line 355, in _get_key
    _key_cache[url] = self.ydl.urlopen(self._prepare_url(info_dict, url)).read()
  File "C:\Users\Charles Dickens\AppData\Local\Programs\Python\Python39\lib\http\client.py", line 469, in read
    return self._readall_chunked()
  File "C:\Users\Charles Dickens\AppData\Local\Programs\Python\Python39\lib\http\client.py", line 583, in _readall_chunked
    raise IncompleteRead(b''.join(value))
http.client.IncompleteRead: IncompleteRead(16 bytes read)

It happens to me few times with Pluto TV also. You just need to re enter the command and it will restart the download at the fragment it stopped.
Happened also with Tubi TV but in this case, it restarts automatically without any human intervention.
So, what Pluto TV needs is a way to detect the error and restart the download automatically again.


Cannot flash without going into bootloader mode

While it's perfectly doable to put the board in bootloader mode and then flash, it would be really great to be able to flash it without that, like any other esp32 boards. The board isn't in sleep mode or anything, and I can even talk to it via webusb, but I can't flash it.
I'm getting this error:
CURRENT: upload_protocol = esptool
Looking for upload port...
Auto-detected: /dev/ttyACM0
Uploading .pio/build/esp32s2doit-devkit-v1/firmware.bin
esptool.py v3.1
Serial port /dev/ttyACM0
Connecting...
Traceback (most recent call last):
  File "/home/xtrinch/.platformio/packages/tool-esptoolpy/esptool.py", line 4582, in <module>
    _main()
  File "/home/xtrinch/.platformio/packages/tool-esptoolpy/esptool.py", line 4575, in _main
    main()
  File "/home/xtrinch/.platformio/packages/tool-esptoolpy/esptool.py", line 4074, in main
    esp = esp or get_default_connected_device(ser_list, port=args.port, connect_attempts=args.connect_attempts,
  File "/home/xtrinch/.platformio/packages/tool-esptoolpy/esptool.py", line 121, in get_default_connected_device
    _esp.connect(before, connect_attempts)
  File "/home/xtrinch/.platformio/packages/tool-esptoolpy/esptool.py", line 629, in connect
    last_error = self._connect_attempt(mode=mode, esp32r0_delay=False, usb_jtag_serial=usb_jtag_serial)
  File "/home/xtrinch/.platformio/packages/tool-esptoolpy/esptool.py", line 590, in _connect_attempt
    self.bootloader_reset(esp32r0_delay, usb_jtag_serial)
  File "/home/xtrinch/.platformio/packages/tool-esptoolpy/esptool.py", line 553, in bootloader_reset
    self._setDTR(False)  # IO0=HIGH
  File "/home/xtrinch/.platformio/packages/tool-esptoolpy/esptool.py", line 499, in _setDTR
    self._port.setDTR(state)
  File "/home/xtrinch/.platformio/penv/lib/python3.8/site-packages/serial/serialutil.py", line 603, in setDTR
    self.dtr = value
  File "/home/xtrinch/.platformio/penv/lib/python3.8/site-packages/serial/serialutil.py", line 473, in dtr
    self._update_dtr_state()
  File "/home/xtrinch/.platformio/penv/lib/python3.8/site-packages/serial/serialposix.py", line 715, in _update_dtr_state
    fcntl.ioctl(self.fd, TIOCMBIC, TIOCM_DTR_str)
OSError: [Errno 71] Protocol error
*** [upload] Error 1

Any ideas? Any help would be greatly appreciated, even if only to point me in the direction of what this error is and where do I start looking.

Yes. The problem is that espressif changed the way it is possible to flash over native USB and i need to update library.

Should work now with master and v1.3.3

Afraid not, still the same error :(

Strange, it works to me now.
All you have to do is to implement CDC code, so using CDC example is best way to test.

I can confirm I'm seeing basically issues with this using the CDC example and v1.3.3.
...
Uploading .pio\build\esp32s2doit-devkit-v1\firmware.bin
esptool.py v3.1
Serial port COM20
Connecting....
Traceback (most recent call last):
  File "C:\Users\Oliver\.platformio\packages\tool-esptoolpy\esptool.py", line 4582, in <module>
    _main()
  File "C:\Users\Oliver\.platformio\packages\tool-esptoolpy\esptool.py", line 4575, in _main
    main()
  File "C:\Users\Oliver\.platformio\packages\tool-esptoolpy\esptool.py", line 4076, in main
    before=args.before)
  File "C:\Users\Oliver\.platformio\packages\tool-esptoolpy\esptool.py", line 121, in get_default_connected_device
    _esp.connect(before, connect_attempts)
  File "C:\Users\Oliver\.platformio\packages\tool-esptoolpy\esptool.py", line 629, in connect
    last_error = self._connect_attempt(mode=mode, esp32r0_delay=False, usb_jtag_serial=usb_jtag_serial)
  File "C:\Users\Oliver\.platformio\packages\tool-esptoolpy\esptool.py", line 596, in _connect_attempt
    self.sync()
  File "C:\Users\Oliver\.platformio\packages\tool-esptoolpy\esptool.py", line 487, in sync
    timeout=SYNC_TIMEOUT)
  File "C:\Users\Oliver\.platformio\packages\tool-esptoolpy\esptool.py", line 425, in command
    self.write(pkt)
  File "C:\Users\Oliver\.platformio\packages\tool-esptoolpy\esptool.py", line 388, in write
    self._port.write(buf)
  File "c:\users\oliver\.platformio\penv\lib\site-packages\serial\serialwin32.py", line 323, in write
    raise writeTimeoutError
serial.serialutil.SerialTimeoutException: Write timeout
*** [upload] Error 1
====================================================================== [FAILED] Took 15.66 seconds ======================================================================The terminal process "C:\Users\Oliver\.platformio\penv\Scripts\platformio.exe 'run', '--target', 'upload'" terminated with exit code: 1.

I was going to try the same test using Arduino IDE, and noticed that the library.properties file still shows V1.3.2 so it wont update easily through library manager. Not tried manually updating the library yet.

Question, did you change Upload Mode to internal?

OK, just manually updated the library in Arduino IDE and it works as expected - can program without manually entering bootloader mode. Upload Mode Internal must be set.
So seems that the only issue is actually that this isn't working with PIO?

Great! That PR will make the PIO environment setup much less fiddly. Thank you.
Been looking more into the CDC upload again and I agree, this is not a problem with this library, it is related to the PIO Arduino ESP32 implementation. I get the same results using the standard ESP32 Arduino USB example.  I will raise an issue over on that project, explain my findings and see what they say.
As a workaround for now @xtrinch, I found that if you remove any specific upload port from your platformio.ini, attempt upload, it will fail but the ESP will go into bootloader mode, then attempt the upload a second time, it will successfully upload and reboot back into your program.

@oliverparis thanks, but I think our errors differ, my board always throws protocol error, not timeout, and it never works.

I know it may be strange, but for some reason it also depends what code is flashed.
For example i had issue and many errors trying to flash over USB when old CDC example was flashed. That example has been using USB serial and Serial1. I dont know exactly what caused the problem, but when i removed Serial1 from example it suddenly started working. No more issue with flashing over USB.

I don't use regular Serial in my code.
Also I logged the DTS & RTS signals in the tud_cdc_line_state_cb callback, and they are nowhere near correct, it mostly just receives DTS=1,RTS=1 continously. Almost like something is interfering with the setting of these signals from esptool.
But there isn't any problem setting them when connecting via a terminal, then it just receives DTS=1,RTS=1 once.

Currently arduino-esp32 and esptool is using baudrate to signal it will going into bootload mode. It is the way espressif decided to workaround it for now. No longer DTR/RTS are used for that.

Odd, that makes me a bit confused. My flashing routine crashes at the DTR sending, in the following line:
https://github.com/espressif/esptool/blob/master/esptool.py#L572
I don't see any baud rate changing in that piece of code?

Maybe you are still using old esptool.
https://github.com/chegewara/EspTinyUSB/blob/master/src/classes/cdc/cdcusb.cpp#L215-L217
I think it is in arduino-esp32 master branch and you have to update tools by running get.py or get.exe.

Oh, I didn't realize they have their own version of esptool.
[env:esp32s2doit-devkit-v1]
platform = espressif32
platform_packages =
  framework-arduinoespressif32 @ https://github.com/espressif/arduino-esp32#master
  platformio/tool-esptoolpy @ ~1.30100

So now I'm running the master branch of arduino-esp32 and if I uncomment this line https://github.com/chegewara/EspTinyUSB/blob/991acff150f33b3d81dfac4999d24590d6b984ff/src/classes/cdc/cdcusb.cpp#L205 I can flash it (although every second time, like @oliverparis suggested)

Yes. The first one is switching into bootload mode and second just perform update.
Its because old esptool is not using 1200 baudrate, instead DTR/RTS.


Mismatch in number of arguments for TreeofThoughts.tot_bfs()

Inside solve() tot_bfs() is called with args
initial_prompt, num_thoughts, max_steps, max_states, value_threshold, pruning_threshold
but in the method definition value_threshold does not appear.  This mismatch causes a crash when running example.

Hey 👋 try now please it should be fixed

Thanks!


Add a "download files" script that recognizes patterns

Allow user to specify subset of results for easy download => At present I can navigate to an experiment’s azureml/ExperimentRun/dcid.my_run_id/outputs folder, but then the options for bulk download are a bit constrained.  As a specific use case, I’d like to be able to download segmentation.nii.gz for all patients in the test dataset.  These are all under outputs/epoch_xyz/Test, but together with the individual-organ segmentations.  For head-and-neck patients, being able to download only the segmentation.nii.gz would reduce the data volume by about a factor of 20.
@kh296
AB#4218

Example of how that could be invoked:

python download_files.py --run abc_123 --pattern segmentation.nii.gz would download all files that contain "segmentation.nii.gz"
--pattern "Test/*/segmentation.nii.gz" would restrict to the test folder.
Should we make pattern effectively a regex?

The syntax looks fine to me, and making pattern effectively a regex sounds like a good idea.  It would also be nice to be able to specify a list of patterns, for example:
--patern "Test/*/segmentation.nii.gz Test/metrics.csv"
The folder hierarchy after download should probably be the same as before download.  For example:
--run abc_123 --patern "Test/*/segmentation.nii.gz" would download to
abc_123/outputs/epoch*/Test/*/segmentation.nii.gz;
--run abc_123 --patern "Test/metrics.csv" would download to abc_123/outputs/CrossValResults/*/epoch*/Test/metrics.csv .


discuss: chash key support more flexible ways

Issue description
Now chash key only support fetch from nginx variable. We need to be more flexible approach，eg: cookie, custom header and so on.
Here is an example to achieve:
upstream schema
hash_on = {
            type = "string",
            default = "vars",
            enum = {
              "vars",
              "header",
              "cookie",
              "consumer"
            },
        },
        key = {
            description = "the key of chash for dynamic load balancing",
            type = "string"
        },

eg:
local function create_chash_hash_key(ctx, upstream)
    local key = upstream.key
    local hash_on = upstream.hash_on
    local chash_key
    -- from nginx variable
    if hash_on == "vars" then
        chash_key = ctx.var[key]
    elseif hash_on == "header" then
        chash_key = ngx.req.get_headers()[key]
    elseif hash_on == "cookie" then
        chash_key = ctx.var["cookie_" .. key]
        -- TODO chash_key doesn't exist, set-cookie
    end
    if not chash_key then
        chash_key = ctx.var["remote_addr"]
    end
    return chash_key
end

hash_on type desc:

vars: fetchs nginx variable
header: custom headers or standard http headers
cookie: fetch chash key from cookie. It can be used to sticky session
consumer: hash by consumer_id

If the specified chash key is not present, use default key: remote_addr(it can also provide default settings).
everybody have a better idea?

@codjust I prefer your design  than the exist: https://github.com/apache/incubator-apisix/blob/master/lua/apisix/schema_def.lua#L261
And please send this discussion to the mailing list too, thx.

@codjust I prefer your design than the exist: https://github.com/apache/incubator-apisix/blob/master/lua/apisix/schema_def.lua#L261
And please send this discussion to the mailing list too, thx.

ok,, done.


vimPlugins: update on 2023-11-24

Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Could you explain in the vimPluginsUpdate the reason for doing so ? oneliners would be fine. For instance

fixes bug XX
I want to use version X

I am not a fan of these frequent updates, it creates some churns and because vim plugins are usually not tested, it is risky so having even a simple motivation motivates me to merge (and thanks for running the nixpkgs-review, that definietely helps too).

Ok this is very understandable.
In this case I want to get this fix from efmls-configs.
I will take the time to motivate those updates in the future.
I never do them without a reason.


feat(google-tag-manager): Allow GTM ID to be function

This PR allows GTM ID to be function returning String or Promise, as shown in readme.md examples

@McSneaky, @pi0
Do you have an idea how to implement dynamic ID's
My website is using different domains for different languages and I also use for each domain a different Google-Tag-Manager account.

@sowinski Depends if you build it once for all domains or build it separately for each domain?

@McSneaky I build it once.
I use server side rendering and have only one instance for all domains (currently ~25).
So I am looking for a solution which is doing something like that:
export default function({ req }) {

  if (req.headers.referer == "exmple.com")
    return 'GTM-156'
  if (req.headers.referer == "exmple.it")
    return 'GTM-24424'
  if (req.headers.referer == "exmple.es")
    return 'GTM-2424'
 ...
}

I think it is not possible without rebuilding the logic of the code.
The module is only called once: https://github.com/nuxt-community/modules/blob/master/packages/google-tag-manager/index.js
It need to be somewhere else.

Yea, can't use this module's settings.
But you can try to parse it out of head and forceby change it one the fly.
No idea if it will work or what side effects it might have tho

@McSneaky
Where would be the best place? I was thinking to rewrite the logic and add it to nuxtServerInit.
What do you think?

@McSneaky

But you can try to parse it out of head and forceby change it one the fly.
Can you give me a small example or a hint what you exactly mean?

@McSneaky

But you can try to parse it out of head and forceby change it one the fly.

Can you give me a small example or a hint what you exactly mean?

I haven't had change to use Nuxt for ages.. But what I meant is something like this:
// BaseLayout.vue
export default {
  created () {
     if (req.headers.referer == "exmple.com")
       this.head.scripts['gtm'].src = 'https://URL/GTM-156'
     if (req.headers.referer == "exmple.it")
        this.head.scripts['gtm'].src = 'https://URL/GTM-24424'
  }
}

It's a lot of pseudocode and won't work, but hopefully it's enough to get idea going

I changed the implementation a little bit and load the code within nuxtServerInit and it seems to work.
Would be nice if someone could refactore the logic of the  nuxtTagManager within a new function. That way user could reuse the code.


Adding CatalogId as an optional parameter

To be able to search products from other catalogs than the default one.

At the moment we are working on search module architecture refactoring and in next version this feature will be available out the box.

You can close the PR if you don't need it. No worries. I've just pushed the PR while I was doing it for myself 👍

The module is obsolete.


Add support to the HiGHS solver MathOpt

Hi,
HiGHS is a magnificent open-source solver that's able to run in parallel, clearly beating CBC in some of the tests I've done in python.
Is there any plan to integrate it into or-tools? (C++)
If not, I'd like to do it myself since the interface looks similar to CBC's. What would be the right way to start?
thanks,
Santiago
Mizux's EDIT(to have task list need to edit the first message...):

[x] Create github issue label
[ ] Incorporate it to CMake build (see below)

[x] Build HiGHS using a CMake based build (provided by project, need to test)
[ ] BUILD_HIGHS using cmake fetch()
[ ] find_package() module

[ ] Test system wide install (https://repology.org/project/highs/badges)

[ ] Incorporate it to bazel build

[ ] Build HIGHS using bazel

[ ] Integrate to MathOpt

note: my dev will be in [mizux/highs]  (to create) branch and regularly rebased on top of [master]

Yes, but we do not have the time to write the interface for it. So we will wait from some contribution for it.

You can copy the cbc_inteface.cc file

Todo list:

[ ] Add option in CMake to enable support (OFF by default maybe ON later)
e.g.
https://github.com/google/or-tools/blob/2cb85b4eead4c38e1c54b48044f92087cf165bce/CMakeLists.txt#L146-L153
[ ] Add FetchContent() section tot build it
e.g.
https://github.com/google/or-tools/blob/2cb85b4eead4c38e1c54b48044f92087cf165bce/cmake/dependencies/CMakeLists.txt#L155-L174
ref: https://github.com/ERGO-Code/HiGHS/blob/master/CMakeLists.txt
note: need to patch their CMake (no CMake config and install rule, no CTest use etc...)
[ ] Add A FindHIGHS.cmake to find a system-wide installed version
e.g. https://github.com/google/or-tools/blob/stable/cmake/FindGLPK.cmake
[ ] Add section in deps.cmake
e.g. https://github.com/google/or-tools/blob/2cb85b4eead4c38e1c54b48044f92087cf165bce/cmake/deps.cmake#L61-L65
[ ] Add a define USE_HIGHS in cpp.cmake
e.g. https://github.com/google/or-tools/blob/2cb85b4eead4c38e1c54b48044f92087cf165bce/cmake/cpp.cmake#L26-L28
[ ] Link MPSovler build to it
e.g. https://github.com/google/or-tools/blob/2cb85b4eead4c38e1c54b48044f92087cf165bce/ortools/linear_solver/CMakeLists.txt#L26

Any news about support of highs solver ? Do you work on it, SanPen ?
Latest version 1.2.2 solved one of my unsolvable mip problems by any open source solver (I maybe tried them all) including scip. Since optimal solution was found under 3 seconds, I was very satisfied and maybe there’s time to think about usage of mip solver in some cases for me.

My process was the following:

I went through ortools code and discovered that ortools modelling in C++was not as simple as ortools in python (a + b <= 3, etc) which disappointed me.
I thought that making my own ortools exactly the way I wanted was less hassle, so I did it. It took me about a week.
I integrated Highs rather easily.

So, essentially I thought that fighting a code that I don't understand would be far more complicated than building everything from scratch, so I made my own solver interface with matrix-variable operations embedded.
If we find time to do a gathering with the authors and get dummy-proof guide, I might write the Highs interface, otherwise it is too much for me.

Having first got in touch with @lperron in August of last year, the HiGHS team is now in a position to contribute to writing an OR-Tools interface to HiGHS. Not only will this get around the "not open-source to industry" issue of SCIP, but it will give OR-Tools a better MIP solver than SCIP - see http://plato.asu.edu/ftp/milp.html
That said, I'm interested to read about the experiences of @SanPen and @Mizux on this page, and it would be great if you were able to feed your knowledge into the process.

I've flagged up this exchange with @galabovaa and she will have a look at what's involved to implement the proto solver.
With your help, I'm sure that we'll make this happen

dev note:
To Fix/Patch

[ ] highs_interface.cc (currently only a copy/paste from scip_interface.cc)
[ ] HiGHS CMake fix:

[ ] Use of include(CTEST)
[ ] Why using SHARED_BUILD option then set the CMake orignal one BUILD_SHARED_LIB ...
[ ] fast build/regular build code refactoring (add_subbdirectory(src) x2 !!!)
[ ] Add install rules for config files and Target.cmake

Thanks.  @galabovaa got Bazel to build HiGHS before, so can contribute. She's busy for the next couple of weeks, though

This is a post to say hi and to confirm interest in this feature! We currently use CLP and GLOP through Java OR tools. We are grateful to everyone for all the existing contributions and excited about the opportunity to start using HiGHS soon. Let me know if there is an opportunity to help in testing.

@galabovaa is now in a position to start work on an OR-Tools <-> HiGHS interface. Can't say how long it will take, but it's definitely happening!

Awesome! Kudos for the great work :)

any updates about ORTools + Highs?

@Mizux Hello, I wonder if you folks are still working on this. I would like to move away from SCIP, due to speed on MIP, also due to potential memory leaks on SCIP. Do you folks still have plans to enable HiGHs within or-tools? Is there an approximate timing you could share?

We are, low priority though.
Not in 9.8 (coming soon). Hopefully 9.9
Laurent Perron | Operations Research | @.*** | (33) 1 42 68 53
00
Le mar. 24 oct. 2023 à 16:16, FernandoMotkraft @.***> a
écrit :

@Mizux https://github.com/Mizux Hello, I wonder if you folks are still
working on this. I would like to move away from SCIP, due to speed on MIP,
also due to potential memory leaks on SCIP. Do you folks still have plans
to enable HiGHs within or-tools? Is there an approximate timing you could
share?
—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/3116#issuecomment-1777311709,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3MWEMQXICTHJBB2DTLYA7ESRAVCNFSM5NULJT6KU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCNZXG4ZTCMJXGA4Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks for the info @lperron

It seems we already have Highs support for mathopt internally (i.e. blaze only), I need to export and clean it asap.
But as Laurent say we want to release the incoming v9.8 this week first so we won't integrate it before v9.9 (ETA around Christmas)

That would be a nice Xmas present @Mizux :)

Hello! Any news about HiGHS integration? :)

Currently cleaning HiGHs build system, the objective is to have HiGHS support working for 9.10 this week or next (finger crossed):

https://github.com/ERGO-Code/HiGHS/pull/1701

Currently some math_opt tests fail when using HiGHs 1.7.0 (internally we still use 1.4.1) trying to understand what happen

Currently some math_opt tests fail when using HiGHs 1.7.0 (internally we still use 1.4.1) trying to understand what happen

Happy to help you understand why, if you'll share HiGHS output

Currently some math_opt tests fail when using HiGHs 1.7.0 (internally we still use 1.4.1) trying to understand what happen

Happy to help you understand why, if you'll share HiGHS output

Thanks, I've create the issue #4203 with all information I have, I'm trying to investigate...


Update for Laravel 5.4

Will update to function properly with Laravel 5.4 ASAP. Just need orchestra/testbench to support 5.4 first to be able to run proper tests.

Laravel 5.4 compatibility is now ready. Read more: https://github.com/olssonm/l5-very-basic-auth/releases/tag/3.0


gRPC Service as a DLL, read access violation. grpc::g_core_codegen_interface was nullptr

Summary
When compiling a gRPC Service to a DLL, it does not work if you load it dynamically at runtime. If you link the DLL at compile time, it works fine, but loading dynamically, it causes an issue for which the error is not comprehensible to me. The server will go into an eternal loop of crashing with the following message:
Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr

See https://github.com/grpc/grpc/pull/26775 for code and steps to reproduce.
What version of gRPC and what language are you using?
$ vcpkg list
abseil:x64-windows                                 2021-03-24#1     an open-source collection designed to augment th...
c-ares:x64-windows                                 1.17.1#1         A C library for asynchronous DNS requests
grpc:x64-windows                                   1.37.0#2         An RPC library and framework
grpc[codegen]:x64-windows                                           Build code generator machinery
openssl:x64-windows                                1.1.1k#7         OpenSSL is an open source project that provides ...
protobuf:x64-windows                               3.15.8#3         Protocol Buffers - Google's data interchange format
protobuf[zlib]:x64-windows                                          ZLib based features like Gzip streams
rapidjson:x64-windows                              2020-09-14#1     A fast JSON parser/generator for C++ with both S...
re2:x64-windows                                    2020-10-01       RE2 is a fast, safe, thread-friendly alternative...
upb:x64-windows                                    2020-12-19#1     μpb (often written 'upb') is a small protobuf i...
zlib:x64-windows                                   1.2.11#10        A compression library

What operating system (Linux, Windows,...) and version?
Edition	Windows 10 Enterprise
Version	20H2
Installed on	7/13/2021
OS build	19042.1083
Experience	Windows Feature Experience Pack 120.2212.3530.0

What runtime / compiler are you using (e.g. python version or version of gcc)
$ msbuild.exe
Microsoft (R) Build Engine version 16.10.2+857e5a733 for .NET Framework
Copyright (C) Microsoft Corporation. All rights reserved.

What did you do?
I used CMake to build a single service as a shared lib (DLL). Then, at runtime in gRPC server, I use LoadLibrary to load that DLL, get a pointer to an instance, and call builder.RegisterService() passing that service instance. I verified the DLL is properly loaded and instantiated. __cdecl and DllExport are used to export the factory function without name mangling, and I can confirm this instance is alive in memory at the time it is registered. The server begins listening without any issue, but then when the client request happens, it triggers a breakpoint and loops forever with the below error message.
What did you expect to see?
A client request should be able to be processed by the listening server, a reply should be formed and sent, and the client should receive it.
What did you see instead?
Instead, we see the following issue:
Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr

The server outputs:
'greeter_server.exe' (Win32): Loaded 'C:\Users\keithl\repos\grpc\examples\cpp\helloworld\cmake\build\Debug\greeter_server.exe'. Symbols loaded.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\ntdll.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\kernel32.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\KernelBase.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\advapi32.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\msvcrt.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\sechost.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\rpcrt4.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\ws2_32.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Users\keithl\repos\grpc\examples\cpp\helloworld\cmake\build\Debug\libprotobufd.dll'. Symbols loaded.
'greeter_server.exe' (Win32): Loaded 'C:\Users\keithl\repos\grpc\examples\cpp\helloworld\cmake\build\Debug\cares.dll'. Symbols loaded.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\wsock32.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Users\keithl\repos\grpc\examples\cpp\helloworld\cmake\build\Debug\re2.dll'. Symbols loaded.
'greeter_server.exe' (Win32): Loaded 'C:\Users\keithl\repos\grpc\examples\cpp\helloworld\cmake\build\Debug\abseil_dll.dll'. Symbols loaded.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\bcrypt.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Users\keithl\repos\grpc\examples\cpp\helloworld\cmake\build\Debug\libssl-1_1-x64.dll'. Symbols loaded.
'greeter_server.exe' (Win32): Loaded 'C:\Users\keithl\repos\grpc\examples\cpp\helloworld\cmake\build\Debug\libcrypto-1_1-x64.dll'. Symbols loaded.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\crypt32.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\ucrtbase.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\user32.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\win32u.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\gdi32.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\gdi32full.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\msvcp_win.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Users\keithl\repos\grpc\examples\cpp\helloworld\cmake\build\Debug\zlibd1.dll'. Symbols loaded.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\msvcp140d.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\vcruntime140d.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\vcruntime140_1d.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\ucrtbased.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\dbghelp.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\imm32.dll'.
The thread 0x8634 has exited with code 0 (0x0).
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\mswsock.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\IPHLPAPI.DLL'.
'greeter_server.exe' (Win32): Loaded 'C:\Windows\System32\cryptbase.dll'.
'greeter_server.exe' (Win32): Loaded 'C:\Users\keithl\repos\grpc\examples\cpp\helloworld\cmake\build\Debug\other_service_shared.dll'. Symbols loaded.
Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

Unhandled exception thrown: read access violation.
**grpc::g_core_codegen_interface** was nullptr.

//// Until you manually stop it
The program '[38760] greeter_server.exe' has exited with code 0 (0x0).

There are some client and server logs that may be of help, but I didn't see anything that stood out:
D0722 14:58:04.838000000 20664 call.cc:747] {"created":"@1626991084.838000000","description":"Error received from peer ipv6:[::1]:50051","file":"C:\Users\keithl\repos\vcpkg\buildtrees\grpc\src\17cc203898-db2679e7f2.clean\src\core\lib\surface\call.cc","file_line":1068,"grpc_message":"","grpc_status":12}
D0722 14:58:04.838000000 20664 metadata_batch.cc:289] mdelem   UNREF:0000015EDD07A6D0:2->1: 'grpc-status' = '12'
D0722 14:58:04.838000000 20664 error.cc:177] 0000015EDD006430: 2 -> 1 [C:\Users\keithl\repos\vcpkg\buildtrees\grpc\src\17cc203898-db2679e7f2.clean\src\core\lib\surface\call.cc:1083]
D0722 14:58:04.838000000 20664 metadata_batch.cc:93] mdelem   UNREF:0000015EDD07BC80:3->2: ':authority' = 'localhost:50051'
D0722 14:58:04.838000000 20664 metadata_batch.cc:93] mdelem   UNREF:0000015EDD00B7E0:2->1: ':path' = '/helloworld.Greeter/SayHello'
D0722 14:58:04.838000000 20664 metadata_batch.cc:93] mdelem   UNREF:0000015EDD07BD30:3->2: 'user-agent' = 'grpc-c++/1.37.0 grpc-c/15.0.0 (windows; chttp2)'
I0722 14:58:04.838000000 20664 completion_queue.cc:789] cq_end_op_for_pluck(cq=0000015EDD073380, tag=00000017DAAFEC70, error="No Error", done=00007FF6695DB320, done_arg=0000015EDD074280, storage=0000015EDD0742F0)
D0722 14:58:04.838000000 20664 src/core/lib/iomgr/closure.h:247] closure 0000015EDD073EF0 finished
D0722 14:58:04.838000000 20664 src/core/lib/iomgr/closure.h:247] closure 0000015EDD074090 finished
D0722 14:58:04.838000000 20664 src/core/lib/iomgr/closure.h:247] closure 0000015EDD07B320 finished
D0722 14:58:04.838000000 20664 src/core/lib/iomgr/closure.h:247] closure 0000015EDD080C40 finished
D0722 14:58:04.838000000 20664 src/core/lib/iomgr/closure.h:247] closure 0000015EDD080E48 finished
D0722 14:58:04.838000000 20664 src/core/lib/iomgr/closure.h:247] closure 0000015EDD080FF8 finished
D0722 14:58:04.838000000 20664 exec_ctx.cc:43] closure 0000015EDD0815E8 finished
D0722 14:58:04.838000000 20664 src/core/lib/transport/transport.h:102] CALL_STACK 0000015EDD073FC0:0000015EDD0735E0 UNREF completion
I0722 14:58:04.838000000 20664 src/core/lib/gprpp/ref_counted.h:182] stream_refcount:0000015EDD073FC0 C:\Users\keithl\repos\vcpkg\buildtrees\grpc\src\17cc203898-db2679e7f2.clean\src/core/lib/transport/transport.h:105 unref 2 -> 1 completion
I0722 14:58:04.838000000 20664 completion_queue.cc:1320] RETURN_EVENT[0000015EDD073380]: OP_COMPLETE: tag:0x17daafec70 OK
I0722 14:58:04.839000000 20664 completion_queue.cc:1426] grpc_completion_queue_destroy(cq=0000015EDD073380)
I0722 14:58:04.839000000 20664 completion_queue.cc:1420] grpc_completion_queue_shutdown(cq=0000015EDD073380)
D0722 14:58:04.839000000 20664 exec_ctx.cc:34] running closure 0000015EDD0733C0: created [C:\Users\keithl\repos\vcpkg\buildtrees\grpc\src\17cc203898-db2679e7f2.clean\src\core\lib\surface\completion_queue.cc:547]: scheduled [C:\Users\keithl\repos\vcpkg\buildtrees\grpc\src\17cc203898-db2679e7f2.clean\src\core\lib\iomgr\pollset_windows.cc:101]
D0722 14:58:04.839000000 20664 exec_ctx.cc:43] closure 0000015EDD0733C0 finished
I0722 14:58:04.839000000 20664 init.cc:215] grpc_shutdown(void)

server:
I0722 16:36:50.019000000 29152 call_combiner.cc:151] ==> CallCombiner::Stop() [000001C1D8B8D638] [C:\Users\keithl\repos\vcpkg\buildtrees\grpc\src\17cc203898-db2679e7f2.clean\src\core\lib\surface\call.cc:1529: recv_trailing_metadata_ready]
I0722 16:36:50.019000000 29152 call_combiner.cc:157]   size: 1 -> 0
I0722 16:36:50.019000000 29152 call_combiner.cc:185]   queue empty
D0722 16:36:50.019000000 29152 call.cc:746] set_final_status SVR
D0722 16:36:50.019000000 29152 call.cc:747] "No Error"
I0722 16:36:50.019000000 29152 completion_queue.cc:789] cq_end_op_for_pluck(cq=000001C1D8B88D80, tag=000001C1D8B90280, error="No Error", done=00007FF7C161F940, done_arg=000001C1D8B90420, storage=000001C1D8B90490)
D0722 16:36:50.019000000 29152 src/core/lib/iomgr/closure.h:247] closure 000001C1D8B8DF30 finished
D0722 16:36:50.019000000 29152 src/core/lib/iomgr/closure.h:247] closure 000001C1D8B8E2F0 finished
D0722 16:36:50.019000000 29152 src/core/lib/iomgr/closure.h:247] closure 000001C1D8B8E3F8 finished
D0722 16:36:50.019000000 29152 src/core/lib/iomgr/closure.h:247] closure 000001C1D8B8E490 finished
D0722 16:36:50.019000000 29152 src/core/lib/iomgr/closure.h:247] closure 000001C1D8B8E7A0 finished
D0722 16:36:50.019000000 29152 exec_ctx.cc:43] closure 000001C1D8B8EB88 finished
I0722 16:36:50.019000000 29152 combiner.cc:200] C:000001C1D8AEA4C0 grpc_combiner_continue_exec_ctx contended=0 exec_ctx_ready_to_finish=1 time_to_execute_final_list=0
I0722 16:36:50.019000000 29152 combiner.cc:223] C:000001C1D8AEA4C0 maybe_finish_one n=000001C1D8B894F8
I0722 16:36:50.019000000 29152 chttp2_transport.cc:811] W:000001C1D8B89370 SERVER [ipv6:[::1]:63133] state WRITING -> IDLE [finish writing]
I0722 16:36:50.019000000 29152 src/core/lib/gprpp/ref_counted.h:182] chttp2_refcount:000001C1D8B89378 C:\Users\keithl\repos\vcpkg\buildtrees\grpc\src\17cc203898-db2679e7f2.clean\src\core\ext\transport\chttp2\transport\chttp2_transport.cc:1060 unref 8 -> 7 writing
I0722 16:36:50.020000000 29152 combiner.cc:265] C:000001C1D8AEA4C0 finish old_state=3
D0722 16:36:50.020000000 29152 exec_ctx.cc:34] running closure 000001C1D8B8EFD8: created [C:\Users\keithl\repos\vcpkg\buildtrees\grpc\src\17cc203898-db2679e7f2.clean\src\core\lib\channel\connected_channel.cc:76]: scheduled [C:\Users\keithl\repos\vcpkg\buildtrees\grpc\src\17cc203898-db2679e7f2.clean\src\core\ext\transport\chttp2\transport\chttp2_transport.cc:820]
I0722 16:36:50.020000000 29152 call_combiner.cc:117] ==> CallCombiner::Start() [000001C1D8B8D638] closure=000001C1D8B906C8 [C:\Users\keithl\repos\vcpkg\buildtrees\grpc\src\17cc203898-db2679e7f2.clean\src\core\lib\channel\connected_channel.cc:59: on_complete] error="No Error"
I0722 16:36:50.020000000 29152 call_combiner.cc:125]   size: 0 -> 1
I0722 16:36:50.020000000 29152 call_combiner.cc:133]   EXECUTING IMMEDIATELY
D0722 16:36:50.020000000 29152 exec_ctx.cc:43] closure 000001C1D8B8EFD8 finished
D0722 16:36:50.020000000 29152 exec_ctx.cc:34] running closure 000001C1D8B906C8: created [C:\Users\keithl\repos\vcpkg\buildtrees\grpc\src\17cc203898-db2679e7f2.clean\src\core\lib\surface\call.cc:1931]: scheduled [C:\Users\keithl\repos\vcpkg\buildtrees\grpc\src\17cc203898-db2679e7f2.clean\src\core\lib\iomgr\call_combiner.cc:99]
I0722 16:36:50.020000000 29152 call_combiner.cc:151] ==> CallCombiner::Stop() [000001C1D8B8D638] [C:\Users\keithl\repos\vcpkg\buildtrees\grpc\src\17cc203898-db2679e7f2.clean\src\core\lib\surface\call.cc:1539: on_complete]
I0722 16:36:50.020000000 29152 call_combiner.cc:157]   size: 1 -> 0
I0722 16:36:50.020000000 29152 call_combiner.cc:185]   queue empty
D0722 16:36:50.020000000 29152 metadata_batch.cc:93] mdelem   UNREF:000001C1D8AB2060:1->0: 'grpc-status' = '12'
I0722 16:36:50.020000000 29152 completion_queue.cc:789] cq_end_op_for_pluck(cq=000001C1D8B88D80, tag=0000007CA55FEF80, error="No Error", done=00007FF7C161F940, done_arg=000001C1D8B905A0, storage=000001C1D8B90610)
D0722 16:36:50.020000000 29152 exec_ctx.cc:43] closure 000001C1D8B906C8 finished
I0722 16:36:50.020000000 29152 completion_queue.cc:1227] grpc_completion_queue_pluck(cq=000001C1D8B88D80, tag=0000007CA55FEF80, deadline=gpr_timespec { tv_sec:<PHONE_NUMBER>854775807, tv_nsec: 0, clock_type: 1 }, reserved=0000000000000000)
D0722 16:36:50.020000000 29152 completion_queue.cc:961] PENDING TAGS:
D0722 16:36:50.020000000 29152 src/core/lib/transport/transport.h:102] CALL_STACK 000001C1D8B8E000:000001C1D8B8D620 UNREF completion
I0722 16:36:50.020000000 29152 src/core/lib/gprpp/ref_counted.h:182] stream_refcount:000001C1D8B8E000 C:\Users\keithl\repos\vcpkg\buildtrees\grpc\src\17cc203898-db2679e7f2.clean\src/core/lib/transport/transport.h:105 unref 3 -> 2 completion
I0722 16:36:50.020000000 29152 completion_queue.cc:1320] RETURN_EVENT[000001C1D8B88D80]: OP_COMPLETE: tag:0x7ca55fef80 OK
I0722 16:36:50.020000000 29152 completion_queue.cc:1420] grpc_completion_queue_shutdown(cq=000001C1D8B88D80)
D0722 16:36:50.020000000 29152 exec_ctx.cc:34] running closure 000001C1D8B88DC0: created [C:\Users\keithl\repos\vcpkg\buildtrees\grpc\src\17cc203898-db2679e7f2.clean\src\core\lib\surface\completion_queue.cc:547]: scheduled [C:\Users\keithl\repos\vcpkg\buildtrees\grpc\src\17cc203898-db2679e7f2.clean\src\core\lib\iomgr\pollset_windows.cc:101]
D0722 16:36:50.020000000 29152 exec_ctx.cc:43] closure 000001C1D8B88DC0 finished
I0722 16:36:50.020000000 29152 completion_queue.cc:1227] grpc_completion_queue_pluck(cq=000001C1D8B88D80, tag=000001C1D8B90280, deadline=gpr_timespec { tv_sec:<PHONE_NUMBER>854775807, tv_nsec: 0, clock_type: 1 }, reserved=0000000000000000)
D0722 16:36:50.020000000 29152 completion_queue.cc:961] PENDING TAGS:
D0722 16:36:50.020000000 29152 src/core/lib/transport/transport.h:102] CALL_STACK 000001C1D8B8E000:000001C1D8B8D620 UNREF completion
I0722 16:36:50.020000000 29152 src/core/lib/gprpp/ref_counted.h:182] stream_refcount:000001C1D8B8E000 C:\Users\keithl\repos\vcpkg\buildtrees\grpc\src\17cc203898-db2679e7f2.clean\src/core/lib/transport/transport.h:105 unref 2 -> 1 completion
I0722 16:36:50.020000000 29152 completion_queue.cc:1320] RETURN_EVENT[000001C1D8B88D80]: OP_COMPLETE: tag:0x1c1d8b90280 OK

Anything else we should know about your project / environment?
Our use case is that we are creating a pluggable architecture for gRPC server and services, wherein our team authors core gRPC services that live in a gRPC server, and other teams author other "plugin" services that can be dynamically loaded at runtime (will require a restart of the server in order to RegisterService, but will not require recompilation). We are trying to fulfill a requirement of keeping process count low by having 1 gRPC server, rather than having a server for each service (which would increase process count). This server runs on the user's local system and provides an interface into native functionality.
Related Issues
https://github.com/grpc/grpc/issues/26097
Research
About compiler warnings:

https://github.com/protocolbuffers/protobuf/blob/master/cmake/README.md#dlls-vs-static-linking
https://www.gamedev.net/forums/topic/637431-dealing-with-needs-to-have-dll-interface-to-be-used-by-clients-of-class/

If your gRPC is not working when it's being dynamically loaded, I suspect that this is most likely caused by some other factors. MSVC or the way to build or load gRPC. Sometimes thread local variables are not properly initialized with MSVC. (e.g. https://developercommunity.visualstudio.com/t/thread-local-variables-fail-to-be-initialized-when/124121)

So if you can share something we can take a look at, it'd be great. Otherwise, there is nothing we can take a look at.


format go-list do not marshal

Currently the exec.Cmd that is run contains formatting instructions for
-json and -f, in >go1.17 this just works, as -json takes precidence.
This has becomes an error in go1.17, so we should pick one and just use
that. My preferences is to have go-list do the parsing, use -f, and just
chomp the string before passing it to ioutil.ReadFile.

Could you revert the change on the go.mod? thank you

I would be happy to, but per the spec, the go directive is supposed to indicate the version a module was written with, and these changes are for the go1.17 toolchain, tested against it, and generated with it.

but this works for go <1.17, and it's not a go1.17 specific feature, the behavior doesn't change between go1.16 and go1.17.

https://github.com/golang/go/issues/44738
https://go-review.googlesource.com/c/go/+/298029/

In fact the only require change is to remove -f {{.GoMod}}, because currently it is just ignored.

I fixed the problem in bedb342ad1cd24348e76c9a2d10eaaf1c12bf31b

I cannot find the reference, but usually see the go directive used to mean "the maximum version supported by the module".

When you create a module, the go command setup to go directive to the current Go version, because you are writing the code with your current Go version, it's doesn't mean that the code works with the previous version (ex: if you are using io instead of ioutils).
Then it's weird to indicate a maximum version because it's not useful information, in some points of view it can be seen as false information because it's impossible to say that your code works with all the previous versions of Go.
For me, everybody uses the go directive to define the minimal version, I have a lot of examples.

I agree that people frequently try and use it to define a minimal version, since it can be used as a gate that "affects use of new language features" thereby enforcing some notion of backwards compatibility. However, it "was originally intended to support backward incompatible changes to the Go language", and this only works if you use it to mean I support version up to go 1.17.
I thought there was an issue about this, but we should really pin down what the version means: min, max, or something else.

There is a ticket: golang/go#30791; they does not seem to be any consensus.


Create initial rest API which will be accessible after successful installation.

Hey,
Do you have an initial code for this? Want to help a little bit.
//Muhpirat

Hi @MuhPirat
As a start, I want to say welcome ;)
We don't have initial code for that, but we were thinking to use https://github.com/APIJet/APIJet as a PHP framework for the API, but if you have something in mind please share it.
We are open for all kind of solutions. Do you want to start it?
Thanks,
Gyuner

Hey @GyunerZeki Thank you for the fast answer. Sure<EMAIL_ADDRESS>please add me. ;-)
Thanks,
Mumu

Hi @MuhPirat
You should have an invitation now. Please let me know if you don't.
Thanks,
Gyuner


Linux UI needed correction - (e.g. target host) next to the connection name

also i saw #8705 about

(e.g. target host) next to the connection name

it's very pretty feature!
but at my environment looks like

this not aligned vertically and horizontally

thanks for the bug report
it duplicates #9062 .
This issue will be closed


Extension info does not match marketplace

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

VS Code Version: 1.86.2
OS Version: Win 11 23H2

Steps to Reproduce:

Open https://marketplace.visualstudio.com/items?itemName=ms-python.mypy-type-checker

Open the same extension in VS Code

Web "Released on" matches "Published", okay.
Web "Last updated" matches "Last released"
Code "Last updated" is different

Maybe a duplicate of https://github.com/microsoft/vscode/issues/164333#issuecomment-1294867079

Last updated is when user has updated this extension.


Snapshot failing with val.forEach is not a function

Do you want to request a feature or report a bug?
bug
What is the current behavior?
So, I'm upgrading my project from an older version of Enzyme 2.x to Enzyme 3.x, keeping React 15.
I'm using enzyme-to-json to create snapshots tests in Jest, I upgraded it to 3.x as well to work with Enzyme 3.x
The problem I'm noticing is that some snapshot tests fail because of this error:
TypeError: val.forEach is not a function

I tracked this down to pretty-format, inside the printImmutable.js file.
Basically, here it expects val to be an array, but an empty object is provided:
val.forEach((item, key) => // line 84 of the compiled lib

The output from enzyme-to-json looks like this:
    {
      "type": "div",
      "props": {
        "className": "ButtonGroup"
      },
      "children": [
        {
          "type": "Button",
          "props": {
            "styles": {},
            "importance": "primary",
            "size": "regular",
            "className": "Button--active Button--first Button--last",
            "key": ".$0/.0"
          },
          "children": null
        }
      ]
    }

And it looks good to me, but I think I'm missing something...
Now, why does it happen?
p.s. actually I'm not sure if this is an Enzyme bug or a Jest one...
ps. 2:
This is my component (simplified):
import React, { cloneElement } from 'react';
import CSSModules from 'react-css-modules';

function ButtonGroup({ children, ...props }) {
  return (
    <div styleName="ButtonGroup" {...props}>
      {Children.map(children, (child, key) => {
        return cloneElement(child, { key });
      })}
    </div>
  );
}

export default CSSModules(ButtonGroup, styles);

If the current behavior is a bug, please provide the steps to reproduce and either a repl.it demo through https://repl.it/languages/jest or a minimal repository on GitHub that we can yarn install and yarn test.
It depends by Enzyme, probably enzyme-to-json and Jest.
What is the expected behavior?
The Jest snapshot should work ust fine
Please provide your exact Jest configuration and mention your Jest, node, yarn/npm version and operating system.
It's a basic create-react-app application, with Enzyme 3.x

Can you please paste output of npm ls pretty-format so we study the relevant version

└─┬<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    └─┬<EMAIL_ADDRESS>      ├─┬<EMAIL_ADDRESS>      │ ├─┬<EMAIL_ADDRESS>      │ │ └──<EMAIL_ADDRESS>deduped
      │ ├─┬<EMAIL_ADDRESS>      │ │ └──<EMAIL_ADDRESS>deduped
      │ └──<EMAIL_ADDRESS>      ├─┬<EMAIL_ADDRESS>      │ └─┬<EMAIL_ADDRESS>      │   └──<EMAIL_ADDRESS>deduped
      └─┬<EMAIL_ADDRESS>        └──<EMAIL_ADDRESS>deduped

I actually managed to avoid the error getting rid of the code that assigned styles to the cloned component.


Adding MeanAbsoluteError test to allTests

Sorry for not looking at this earlier. Another minor mistake, #182 didn't add the test to allTests.

@rxwei Sorry for so many PRs today. Could you label this with a kokoro build?

@rxwei I ran the CI tests and they all pass so this can be merged.

Thanks!


Add all clang-tidy checks, ignore all unwanted ones

adds all clang-tidy checks, then ignores any that make changes to the code in its current state

@Calinou should i do something to get a review on this, or does it just happen some time in the 4.x milestone pipeline?

I haven't used clang-tidy extensively, so I don't know if this list is suitable.
In fact, the current list already doesn't pass:
pre-commit run --hook-stage manual clang-tidy --all-files
...
725 warnings generated.
clang-tidy applied 37 of 37 suggested fixes.
424 warnings and 1 error generated.
Error while processing /home/hugo/Documents/Git/godotengine/godot/core/config/engine.h.

See https://github.com/godotengine/godot/pull/93401, which should likely be merged first. CC @Repiteo
I wouldn't enable any clang-tidy check that currently passes fine without evaluating whether it's actually a code quality guideline we want to follow. But it's a good idea to review the various checks offered by clang-tidy.

See #93401, which should likely be merged first. CC @Repiteo
I wouldn't enable any clang-tidy check that currently passes fine without evaluating whether it's actually a code quality guideline we want to follow. But it's a good idea to review the various checks offered by clang-tidy.

Is there a code quality guideline we are currently following? If so, where can i find it?
My experience with clang-tidy is that the ones that offer automatic fixing are the ones that are the simplest to enforce. The error you sow above is one caused by the compile_commands.json file not containing the path to an include

Unrelated whitespace changes in modules/gdscript/tests/scripts/.

I don't think any checks or changes should be run in thirdparty


Namespace events in .on() and .off()

I think it would be beneficial to be able to namespace events like in jQuery, where you can go u('button').on('click.namespace') so you can easily turn off just that namespaces event and not all click handlers on the button. I find this useful when I'm triggering custom events on the document in an Ajaxy site. Otherwise events start multiplying.

I think the benefit is too small for the cost it carries to implement it (both in size and complexity). Umbrella JS isn't intended to be a clone of jQuery, just keeping compatibility where it makes sense and departing from it where it doesn't and I think it's better if we keep this outside of Umbrella's scope.
I didn't totally undestand "I find this useful when I'm triggering custom events on the document in an Ajaxy site". If you are trying to define custom event names and trigger them manually, you are able to do so.

I understand not wanting to implement it. If everything gets added eventually it'd turn into jQuery.
I didn't explain what I'm doing well. i am using custom events but thought I needed namespaces to do some stuff but as I started explaining it here I think I found out how to do what I want with namespacing.
Appreciate the quick feedback. Keep up the good work.

Can I ask what you ended up doing? While not adding namespaces, maybe we could add a note to the documentation for anyone looking for namespaces.


Mitigating "missing key" log message formatting failures

This is a rough sketch for mitigating not-very-good log output as described in https://trello.com/c/qLOqQYy0/226-unable-to-format-log-message-log-messages-not-particularly-useful
No tests - I'm not sure it's worth going ahead with - will see what others think.

@risicle have made some tweaks to this and added tests. Would be good to get this in if we can.

@idavidmcdonald comments addressed

Bah. Didn't bump the version. Nevermind, it'll trickle out.


Prevent a big report to fail creating the issue

Is your feature request related to a problem?
Issue creation API has a limit of 65536 characters. Whenever the report is longer, the action fails with the following error:
Error: Validation Failed: {"resource":"Issue","code":"custom","field":"body","message":"body is too long (maximum is 65536 characters)"}
This might happen easily when the amount of analyzed PRs is over ~300.
Describe the solution you'd like
I would like the action to validate the size of the report before submission, splitting the extra details into multiple comments if necessary (or just create the issue with whatever it fits) without failure.
Describe alternatives you've considered
Alternatively, the action could support a config flag to include in the report only the first two tables with the basic stats, so all the details of each single PR are not reported.
Additional context
Thanks!

Thanks for opening this issue! I've just been putting up with this and it hadn't crossed my mind that we could make this better. 😓 Great suggestion, I've opened up a pull request to address this.

That was so fast, thank you!


torch_geometric.nn.radius gets different output/performance on GPU and CPU

Hello!
I found my pointnet++ implementation performs differently on GPU and CPU, although all the randomness has been removed. I narrowed down and locate the difference at the function "torch_geometric.nn.radius". It gives different results on GPU and CPU. By fixing the rows and cols returned by this function, CPU and GPU performances get much closer. Do you have a sense about what the potential cause is?
Thanks!

Yeah, the radius GPU kernel is non-deterministic. It will return a random permutation of nodes that lay within the given radius in case the number of neighbors in larger than max_num_neighbors. The CPU version will return the closest ones. You can "fix" that by increasing the number of max_num_neighbors or reducing the radius.

Thanks for clarifying this!


Remove start menu shortcuts on uninstall

Fixes issue #979
Brief summary of changes
Add lines to remove shortcuts from start menu upon uninstall on windows
Testing I've completed
Created installer locally ran it then ran uninstall and the shortcuts were removed. This is important in developers/testers environment where multiple installations are commonplace.
CHANGELOG.md (choose one)

no need to update because bugfix

Artifact worked as expected and removed shortcuts from start menu on uninstall (windows)


💬 [Question] Different Design of GDD & GDT Lists

💬 Question

The design when opening the entries on GDD & GDT differ. Is this on purpose? What would that purpose be?
GDD:

GDT:

The style of the GDT list has been adapted to the style of the GDD list.


help(CDK): Bootstrap call not found

What are you trying to do?
Trying to add @angular/material to project by running "ng add @angular/material".
This gives me "Bootstrap call not found"
What troubleshooting steps have you tried?
"ng add @angular/material" gives me "Bootstrap call not found"
Reproduction
Steps to reproduce:

ng add @angular/material

Environment

Angular: 12.0.3
CDK/Material:
Browser(s): -
Operating System (e.g. Windows, macOS, Ubuntu):  Windows

SchematicsException [Error]: Bootstrap call not found
at findBootstrapModulePath (\src\frontend\node_modules@angular\cdk\schematics\utils\vendored-ast-utils\index.js:451:15)
at Object.getAppModulePath (\src\frontend\node_modules@angular\cdk\schematics\utils\vendored-ast-utils\index.js:474:32)
at \src\frontend\node_modules@angular\material\schematics\ng-add\setup-project.js:65:44
at Generator.next ()
at fulfilled (\src\frontend\node_modules@angular\material\schematics\ng-add\setup-project.js:12:58)
at processTicksAndRejections (internal/process/task_queues.js:95:5)
Bootstrap call not found

The "bootstrap call" means that the schematics are looking for a method call for bootstrapModule, but they aren't finding them. How is app bootstrapped?

I believe it tries to find the main module in order to start building out the dependency graph. It is specifically looking for syntax that looks like foo.bootstrapModule so if you're doing something different, it might not find it.

I believe it tries to find the main module in order to start building out the dependency graph. It is specifically looking for syntax that looks like foo.bootstrapModule so if you're doing something different, it might not find it.

Thx it solved that issue but gave me a new error about default builder .. ended up adding it manually.


Error with import geemap on Jupyter Notebook

Environment Information
Please run the following code on your computer and share the output with us so that we can better debug your issue:
import geemap
geemap.Report()

Description
I cannot even import geemap into a Jupyter Notebook.
I get this error:
Please restart Jupyter kernel after installation if you encounter any errors when importing geemap.   File "C:\ProgramData\Anaconda3\lib\site-packages\geemap\geemap.py", line 935     if layer_manager := self._layer_manager:                       ^ SyntaxError: invalid syntax
What I Did
I tried updating all the packages in my conda environment and that did not help.

Try creating a fresh conda env to install geemap. You should not install packages to the base env. It is likely your base env is corrupted.
conda create -n gee python=3.11
conda activate gee
conda install geemap -c conda-forge

Great! Thanks so much. Will try that!

Feel free to reopen the issue if the problem persists with a new conda env


Prevent User 1 from accessing User 2's command history

Related to #130

Seems to be done jusf needs some testing


Determine how to deal with multiple audiences for jwt.Credentials

JWT bearer credentials require an audience that matches the service URL to work.
Right now, JWT credentials don't require an audience when constructing, but instead can generate a one-time token during before_request. This works, but unfortunately this is pretty awful performance-wise.
My best as to why we did this is because within one client we may actually be talking to two different audiences. Consider pubusb - there is the publisher audience (https://pubsub.googleapis.com/google.pubsub.v1.Publisher) and the subscriber audience (https://pubsub.googleapis.com/google.pubsub.v1.Subscriber).
I see two options for making this better:

Store a cache of tokens for every given audience. This has lots of problems.
Make our clients pass in credentials.with_claims(audience='...') (example here) instead of just credentials. This means that we'll essentially be using a copy of the credentials for each audience (pubsub will have two), so they will likely refresh at different times (which I don't consider a problem).

@dhermes @lukesneeringer thoughts?

I don't understand the "pretty awful performance-wise" piece you are referring to
I don't see much wrong with asking the caller to be aware of the audience. Libraries like google-cloud-python can handle this without being a burden on users.

I don't understand the "pretty awful performance-wise" piece you are referring to

Generating a new JWT for every request adds a lot of overhead. It's much better to only generate that once.

I don't see much wrong with asking the caller to be aware of the audience.

Great, I'll do what I described in (2) above. I'll remove the per-request one-time JWT logic. We can always add that back as a new credential type later if needed, but if we hit 1.0 with that behavior we're married to it.


Fixed CSV hex column

Hex column in the result tuple is not formatted and is set as decimal in the generated CSV log. This is a quick fix for the issue.

Hex column in the result tuple is not formatted and is set as decimal in the generated CSV log. This is a quick fix for the issue.

This issues solved？My csv log hex column is decimal。


GitHub workflow: automatically assign feature requests to the specified project

…ects
Signed-off-by: zhangjinpeng1987<EMAIL_ADDRESS>
What problem does this PR solve?
Automatically assign feature requests to the specified project.
What is changed and how it works?
Add GitHub workflow.
Related changes

None

Check List

No code

Side effects

None

Release note

No release note

/rebuild


fix: Fix preview image stretched in contain/cover mode

What

This PR fixes the stretch problem in PreviewView, scaled width and height was mis-calculated.
Changes

This PR changes the containerAspectRatio from W:H to H:W,  to keep the consistence to contentAspectRatio

Tested on

Android Pad, Landscape, 1280x800, Android 11
Related issues

I found it myself, did not submit the issue yet.

Hey! Thank you for your PR!
Can you maybe provide before and after screenshots so I can better understand this?

Doesn't this effectively do the same thing, just inverting division with multiplication?

cc @blancham btw

I feel like it's deeper than that, I stated in my PR that in my case I needed a certain order of (width, height) but it seems to depend on the device/ preferred orientation.
I think that if I test this code on my Vivo phone it'll not work

Doesn't this effectively do the same thing, just inverting division with multiplication?

comparing contentAspectRatio and  containerAspectRatio means nothing if you calculate them in different way...
val contentAspectRatio = contentSize.height.toDouble() / contentSize.width
val containerAspectRatio = containerHeight.toDouble() / containerWidth
if (contentAspectRatio > containerAspectRatio) {
} else {
}

in my device in landscape mode ( width > height) , the height will be stretched to be larger than the width if i put the video in a square.

Or maybe it's always given as if you are in landscape mode so the formula should depend on the orientation we give as prop

Device orientation is different than Camera orientation.
Camera sensors are pretty much always in landscape orientation, so it will be like width: 4000 height: 2000, when the screen (in portrait) is width: 2000 height: 4000.
We need to always compute this in Camera orientation, and only the view code should update to device/view orientation itself.
This is somewhat related to https://github.com/mrousavy/react-native-vision-camera/issues/1891

How about we just swap the previewTargetSize before set up previewView according sensorRelativeOrientation?
  private fun setupPreviewView() {
    // ...

    if (cameraId == null) return

    val cameraId = cameraId ?: throw NoCameraDeviceError()
    val characteristics = this.cameraManager.getCameraCharacteristics(cameraId)
    // sensorRelativeOrientation should be in [ PORTRAIT, PORTRAIT_UPSIDE_DOWN]
    val sensorRelativeOrientation = outputOrientation.toSensorRelativeOrientation(characteristics)
    Log.i(TAG, "sensorRelativeOrientation: $sensorRelativeOrientation")
    var targetSize = this.getPreviewTargetSize()
    if (!arrayOf(Orientation.PORTRAIT, Orientation.PORTRAIT_UPSIDE_DOWN).contains(sensorRelativeOrientation)) {
      targetSize = Size(targetSize.height, targetSize.width)
    }

    val previewView = PreviewView(context, targetSize, resizeMode) { surface ->
      // ...
    }
    // ...
}

For what it's worth, I did this patch on my project and it's working fine so far.
diff --git a/node_modules/react-native-vision-camera/android/src/main/java/com/mrousavy/camera/core/PreviewView.kt b/node_modules/react-native-vision-camera/android/src/main/java/com/mrousavy/camera/core/PreviewView.kt
index d462abc..e27cee2 100644
--- a/node_modules/react-native-vision-camera/android/src/main/java/com/mrousavy/camera/core/PreviewView.kt
+++ b/node_modules/react-native-vision-camera/android/src/main/java/com/mrousavy/camera/core/PreviewView.kt
@@ -56,7 +56,7 @@ class PreviewView(context: Context, callback: SurfaceHolder.Callback) : SurfaceV

   private fun getSize(contentSize: Size, containerSize: Size, resizeMode: ResizeMode): Size {
     val contentAspectRatio = contentSize.height.toDouble() / contentSize.width
-    val containerAspectRatio = containerSize.width.toDouble() / containerSize.height
+    val containerAspectRatio = containerSize.height.toDouble() / containerSize.width

     Log.d(TAG, "coverSize :: $contentSize ($contentAspectRatio), ${containerSize.width}x${containerSize.height} ($containerAspectRatio)")

I will upload a picture as soon as I can but for me the preview was horizontally squished, with the change it's not. (tested on Vivo phone)

I can confirm this fixed the problem for me on 3.6.17
When mounting camera while phone is in landscape mode it was stretched really badly, with this it is all good

Created a PR to fix this issue once and for all: https://github.com/mrousavy/react-native-vision-camera/pull/2519 💪🚀
Thanks to everyone who sponsored me on GitHub to fix this bug!! 💖
Try the PR and see if it works (by checking out that branch and running the example app)


Getting URL parameter '....' is not supported for facets or aggs using elasticsearch-persistence

I'm using elasticsearch-persistence. I have a search query like that:
search q, size: per_page, from: page-1,
              aggs: {
                  timeline: {
                      histogram: {
                          field: 'published_at',
                          time_interval: '1y'
                      }
                  }
              }

I tried to use facets as well but I get URL parameter 'aggs' is not supported.  Why is that? Thank you for your help!

That's because you're using the "simple query in URL parameter" variant, see example here: http://www.rubydoc.info/gems/elasticsearch-api/Elasticsearch/API/Actions#search-instance_method
Which is because of: https://github.com/elastic/elasticsearch-rails/blob/master/elasticsearch-model/lib/elasticsearch/model/searching.rb#L31
When you pass the search definition as a Hash, the aggregations (or facets, but those are going away in future) should be returned as part of the response.
Check out the https://github.com/elastic/elasticsearch-ruby/tree/master/elasticsearch-dsl gem for building search definitions programatically!

Thank you @karmi, it helped. I appreciate how the elasticsearch for ruby/rails is designed and also your help.

Thank you @karmi, it helped. I appreciate how the elasticsearch for ruby/rails is designed and also your help.

@infiniteiteration Thanks!, closing the issue then.


Object inherited from base class shows Object reference not set in .cshtml

[Enter feedback here]

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

ID: 140ce637-4d58-995f-c69e-b1de51f1dbb5
Version Independent ID: 9bc46ffa-60db-bdee-0984-fab183c8f503
Content: Razor Pages with EF Core in ASP.NET Core - Update Related Data - 7 of 8
Content Source: aspnetcore/data/ef-rp/update-related-data.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

My problem is that the object inherited from the base class
compiles OK but click on CreateNew throws an Error Object reference not set -<EMAIL_ADDRESS>in Courses/Create.cshtml
Holding my mouse over @Model.DepartmentNameSL shows
Select DepartmentNameModel.DepartmentNameSL { get; set; } so why would it be saying Object reference not set?
I have copied/pasted the three files, Create.cshtml and cshtml.cs and DepartmentNamePageModel.cs from Github, but still hit the error
Please help! Pitters

I am following page 7 of 8 Razor Contoso Universty tutorial

Thanks for a quick response, Rick, very encouraging.
I copied and pasted SchoolContext.cs and corrected Student to Students ! and Enrollment to Enrollments . .
but when I compile (OK) and click n Courses it now tells me that Department in Index.cshtml.cs doesn't exist :(
Course = await _context.Courses
.Include(c => c.Department)
.AsNoTracking()
.ToListAsync();
yet when I hover over c.Department, it shows "Department Models.Course.Department { get; set; } "
My Models are the same as the tutorial solution
Oh, and a very Happy New Year to you all, too!!

Further to above, I changed the field name Department, and selected CourseAssignments
(in Pages\Courses\Index.cshtml.cs.
Compiled and ran - clicked on Courses and it opened the Courses screen but the Department field was blank.
Clicked on Edit link and got the original error Object reference not set
????

Thanks for contacting us.
We don’t have the resources to invest in this area, so we are closing the issue. Should your request generate enough  👍 responses, we’ll reconsider.


Kafka spring listener spring validation on headers

I'm trying to use spring validation annotation on @KafkaListener method.
I was trying the following:
@Component
@Validated
public class Consumer {
    @KafkaListener(topics = "${app.kafka.consumer.topic}", groupId = "${spring.kafka.consumer.group-id}")
    public String consume(@Payload @Valid @NotNull RequestObj request,
                          @Header(KafkaHeaders.RECEIVED_TOPIC) @Valid @NotEmpty String topic) {
        ...
    }
}

And it doesn't seem to work, meaning the @NotEmpty annotation is not checked.
I also configured the configuration mentioned in the docs:
@Configuration
@EnableKafka
public class Config implements KafkaListenerConfigurer {

    @Autowired
    private LocalValidatorFactoryBean validator;
    ...

    @Override
    public void configureKafkaListeners(KafkaListenerEndpointRegistrar registrar) {
      registrar.setValidator(this.validator);
    }
}

But it only effect the payload part.
Is what I am trying to achieve is actually supported?
Thanks for your help.

Please don't use issues to ask questions; and please don't ask questions in multiple places; it is a waste of your time and ours.
See my answer to the exact same question on Stack Overflow.
https://stackoverflow.com/questions/70061445/kafka-spring-listener-spring-validation-on-headers/70067194#70067194


Spotify Not working

Hello I’d like to know why my Spotify GUI on 5 zig won’t work even though I have Spotify open

did u connect it to spotify? is not do /mediamod then u will get it from here


added sum aggregator

This should prevent us from having to worry about a renamed event/property mucking up reporting as we can sum both. Could add max and min ones too, though I don't think there's much call for that
e.g. @sum(page:view->count(),dwell->count())
@adambraimbridge

:+1:


Conflict with Conda - local:17: = not found

Getting the error local:17: = not found when conda is installed.
Only when i comment out source $HOME/.git-acc from my profile does it stop.
Setup:
Terminal : zsh
Device : Mac M1 Air
OS: Ventura(13.4.1)

same issue with https://github.com/tw-yshuang/Git_SSH-Account_Switch/issues/5#issue-1382390511

Thx for the quick update.
But there is no solution there to resolve it, only to use alternatives like pyenv/pipenv.
The conflict with conda is blocker for me.
Is there anyway i can help to resolve this?
p.s Awesome tool, resolved a lot of headaches for me.


1 line task clipping

As a streamer I'd like to have a setting allowing me to chose whether to clip tasks taking longer than 1 line, to better control how my cards and tasks look for the viewers

while keeping to the minimalistic design, what would you suggest?
Side-scrolling would be a bit too much as the main list already scrolls vertically.

can the task's line be wrapped within the width + padding of the card?

I will provide you with code and instructions on how you can disable the text ellipses..
a useful thing about the Ellipses is it discourages task spammers from flooding your tasklist with long text..   (example:  1 task with 40 words would make that 1 User's card very long.)  Also, it promoted equality among viewers since every task takes up the same amount of space.  1⃣ = 1⃣

That is a valid argument. Maybe the middle ground is to allow a 2 line per task and/or 140 character limit?

upon further consideration, this request is out of the scope of this project direction.
close issue request.


⚠ GitLab Server has degraded performance

In c86fba1, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 10642 ms

Resolved: GitLab Server performance has improved in 48a81df.


Q: How to fallback to a static resource if backend is unavailable

Setup: based on Create React App
"react-i18next": "^4.8.0",
"i18next": "^8.4.3",
"i18next-xhr-backend": "^1.4.2",
Problem Statement
I have a service which enables end users to configure a custom dictionary which is exposed over a rest end point (they don't have to, but are allowed to if they wish).  I would love to fallback to a "bundled" dictionary,  which we provide out of the box as a static resource if no custom dictionaries are found. The loadUrl for both these assets would be different.
Any ideas on how it can be achieved? I am not even sure if this should be a concern for XHR backend or for i18next init method.
My ideas so far
Use a custom XHR function to achieve this. I do not want to reinvent the wheel, but at the same time, that seems to be the reason behind this option to be present.
Just wondering if a better option exists ( I am new to i18next ecosystem and was able to start very fast, but I do not know the inner workings of it yet, any hint would be appreciated)

Best options i guess would be creating a own backend plugin (based on xhr) -> you do loading xhr for the custom translations from the rest endpoint and merge those with the values you provide (bundled or from a own rest endpoint).
Creating own plugin is rather easy using the exiting xhr-backend + reading https://www.i18next.com/misc/creating-own-plugins.html#backend
If you like to provide a simpler way for your customers to create own translations you might have a look at our https://locize.com

Thank you for the great hint! I will give it a shot.
Closing the issue.

Awesome...if you run into troubles - let me know


Calculating time taken to predict

I am doing a document classification task and my test set is quiet large: around 250k samples to be compared against 55k labels. Is there a way to calculate the time it takes to predict for these samples? I know it works fine since I can predict for 10k samples (even though it takes like 5-10 mins).
Issue #68 was also referred and i changed the number of threads from 10 to 50, but it still takes time.

@tharangni Hi, the time it takes in test is proportional to the size of the examples, so it takes 25x more time when you increase examples from 10k to 250k.

Awesome! Thanks for the info! Closing it .


Support for React 18

There is a npm error installing the package:
npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>

Will take a look very soon thank you

Resolved in 0.16.0


FilledLineChart stroke

I am trying to draw FilledLineChart with a solid stroke on the top.
Something like this

Is it possible to do with your lib?

This has been added in v2.3.0.
FilledLineChart(chartData: data)
                .filledTopLine(chartData: data,
                               lineColour: ColourStyle(colour: .red),
                               strokeStyle: StrokeStyle(lineWidth: 3))


Planning for future releases

As a developer, I would like to have a backlog/rough plan for all future releases, and a polished plan of the upcoming release.
Tasks to complete this story:

[ ] Wiki page of potential features
[x] Wiki page of features mockups #24 & #25 @BarsemianGeorgik
[x] Issues and task breakdown for Advanced feature No.1 (Release 2 backlog) @Keeran10
[ ] Issues and task breakdown for Advanced feature No.2 (Release 2 backlog) @Keeran10
[ ] Issues and task breakdown for Static Analysis story (Release 2 backlog)
[ ] Issues and task breakdown for Test Coverage story (Release 2 backlog)
[ ] Issues and task breakdown for Firebase APM story (Release 3 backlog)

When aforementioned tasks are completed, provide a link to the issue in the comments. No need to reference this issue. Note that these tasks can also be captured in the form of notes directly inside the backlog. When doing so, simply provide a link to the backlog.

The issues for the first advanced feature (b&w filter) have been created and placed inside the backlog.

The issues for the second advanced feature (gridlines) have been created and placed inside the backlog.

The issues for the Static Analysis tool, the Test Coverage and the Firebase APM have been created and placed inside our backlog.


Update Composer dependencies (2021-06-01-00-04)

Visual regression test passed!
View the visual regression test report


Saving a GCN-LSTM model gives tensorflow warnings about untraced functions

Describe the bug
When saving the model in the gcn-lstm demo notebook it seems to save/load OK and the predictions are the same for the loaded model; however, I get these warnings which I don't understand:
WARNING:absl:Found untraced functions such as lstm_cell_1_layer_call_fn, lstm_cell_1_layer_call_and_return_conditional_losses, lstm_cell_1_layer_call_fn, lstm_cell_1_layer_call_and_return_conditional_losses, lstm_cell_1_layer_call_and_return_conditional_losses while saving (showing 5 of 5). These functions will not be directly callable after loading.

To Reproduce
Steps to reproduce the behavior:

Open the Notebook gcn-lstm-time-series.ipynb
Execute the following

model.save("model_save_test_2.mdl")
model_ld = keras.models.load_model("model_save_test.mdl", custom_objects=sg.custom_keras_layers)

See error

Environment
Operating system: for example: Ubuntu
Python version: 3.8.3
Package versions: stellargraph==1.3.0b, tensorflow==2.4.0

#1850 (where this was first noticed) may be relevant too.

#1850 (where this was first noticed) may be relevant too.


Corrections un main files path to be able to work with grunt-injector

Thanks @leseulsteve I'm not getting much time to maintain this right now so I've made you a contributor.

No worries mate :) thanks