Signature extraction failed

Hi, i have this problem.

I have any idea.

+1

I also have the same problem.

$ ./youtube-dl -v "https://www.youtube.com/watch?v=iYKe3aTc8D0"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://www.youtube.com/watch?v=iYKe3aTc8D0']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.04.26
[debug] Python version 2.7.13 - Linux-4.10.12-200.fc25.x86_64-x86_64-with-fedora-25-Twenty_Five
[debug] exe versions: none
[debug] Proxy map: {}
[youtube] iYKe3aTc8D0: Downloading webpage
[youtube] iYKe3aTc8D0: Downloading video info webpage
[youtube] iYKe3aTc8D0: Extracting video information
WARNING: unable to extract html5 player; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
[youtube] {22} signature length 40.40, html5 player None
ERROR: Signature extraction failed: Traceback (most recent call last):
File "./youtube-dl/youtube_dl/extractor/youtube.py", line 1081, in _decrypt_signature
video_id, player_url, s
File "./youtube-dl/youtube_dl/extractor/youtube.py", line 969, in _extract_signature_function
raise ExtractorError('Cannot identify player %r' % player_url)
ExtractorError: Cannot identify player u'https://www.youtube.com/yts/jsbin/player-vflppxuSE/en_US/base.js'; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
(caused by ExtractorError(u"Cannot identify player u'https://www.youtube.com/yts/jsbin/player-vflppxuSE/en_US/base.js'; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.",)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "./youtube-dl/youtube_dl/extractor/youtube.py", line 1081, in _decrypt_signature
video_id, player_url, s
File "./youtube-dl/youtube_dl/extractor/youtube.py", line 969, in _extract_signature_function
raise ExtractorError('Cannot identify player %r' % player_url)
ExtractorError: Cannot identify player u'https://www.youtube.com/yts/jsbin/player-vflppxuSE/en_US/base.js'; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "./youtube-dl/youtube_dl/YoutubeDL.py", line 760, in extract_info
ie_result = ie.extract(url)
File "./youtube-dl/youtube_dl/extractor/common.py", line 429, in extract
ie_result = self._real_extract(url)
File "./youtube-dl/youtube_dl/extractor/youtube.py", line 1642, in _real_extract
encrypted_sig, video_id, player_url, age_gate)
File "./youtube-dl/youtube_dl/extractor/youtube.py", line 1091, in _decrypt_signature
'Signature extraction failed: ' + tb, cause=e)
ExtractorError: Signature extraction failed: Traceback (most recent call last):
File "./youtube-dl/youtube_dl/extractor/youtube.py", line 1081, in _decrypt_signature
video_id, player_url, s
File "./youtube-dl/youtube_dl/extractor/youtube.py", line 969, in _extract_signature_function
raise ExtractorError('Cannot identify player %r' % player_url)
ExtractorError: Cannot identify player u'https://www.youtube.com/yts/jsbin/player-vflppxuSE/en_US/base.js'; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
(caused by ExtractorError(u"Cannot identify player u'https://www.youtube.com/yts/jsbin/player-vflppxuSE/en_US/base.js'; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.",)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

where take the OP's clip as the example.

https://pastebin.com/CG2ayHpJ
Yet another confirmation using a video from 2009

+1
https://www.youtube.com/watch?v=Ad6SxduZfis
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 969, in _extract_signature_function
raise ExtractorError('Cannot identify player %r' % player_url)
ExtractorError: Cannot identify player u'https://www.youtube.com/yts/jsbin/player-vflppxuSE/en_US/base.js'

Same here...

Why @dstftw close this Issue? Isn´t duplicate.

I have the same issue, I'm using windows exe version 04.26.2017
C:\Users\Administrator>youtube-dl -v --extract-audio --audio-format mp3 -o C:/yo
utube-downloads/%(id)s.%(ext)s http://www.youtube.com/watch?v=RgKAFK5djSk
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '--extract-audio', '--audio-format', 'mp3', '-
o', 'C:/youtube-downloads/%(id)s.%(ext)s', 'http://www.youtube.com/watch?v=RgKAF
K5djSk']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2017.04.26
[debug] Python version 3.4.4 - Windows-2012ServerR2-6.3.9600
[debug] exe versions: ffmpeg N-85588-g6108805, ffprobe N-85588-g6108805
[debug] Proxy map: {}
[youtube] RgKAFK5djSk: Downloading webpage
[youtube] RgKAFK5djSk: Downloading video info webpage
[youtube] RgKAFK5djSk: Extracting video information
WARNING: unable to extract html5 player; please report this issue on https://yt-
dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to
update. Be sure to call youtube-dl with the --verbose flag and include its comp
lete output.
[youtube] {22} signature length 40.40, html5 player None
ERROR: Signature extraction failed: Traceback (most recent call last):
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpi8nwgst1\bu
ild\youtube_dl\extractor\youtube.py", line 1081, in _decrypt_signature
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpi8nwgst1\bu
ild\youtube_dl\extractor\youtube.py", line 969, in _extract_signature_function
youtube_dl.utils.ExtractorError: Cannot identify player 'https://www.youtube.com
/yts/jsbin/player-vflppxuSE/en_US/base.js'; please report this issue on https://
yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U
to update. Be sure to call youtube-dl with the --verbose flag and include its c
omplete output.
(caused by ExtractorError("Cannot identify player 'https://www.youtube.com/yts/
jsbin/player-vflppxuSE/en_US/base.js'; please report this issue on https://yt-dl
.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to u
pdate. Be sure to call youtube-dl with the --verbose flag and include its comple
te output.",)); please report this issue on https://yt-dl.org/bug . Make sure yo
u are using the latest version; type  youtube-dl -U  to update. Be sure to call
youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpi8nwgst1\bu
ild\youtube_dl\extractor\youtube.py", line 1081, in _decrypt_signature
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpi8nwgst1\bu
ild\youtube_dl\extractor\youtube.py", line 969, in _extract_signature_function
youtube_dl.utils.ExtractorError: Cannot identify player 'https://www.youtube.com
/yts/jsbin/player-vflppxuSE/en_US/base.js'; please report this issue on https://
yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U
to update. Be sure to call youtube-dl with the --verbose flag and include its c
omplete output.
Traceback (most recent call last):
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpi8nwgst1\bu
ild\youtube_dl\extractor\youtube.py", line 1081, in _decrypt_signature
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpi8nwgst1\bu
ild\youtube_dl\extractor\youtube.py", line 969, in _extract_signature_function
youtube_dl.utils.ExtractorError: Cannot identify player 'https://www.youtube.com
/yts/jsbin/player-vflppxuSE/en_US/base.js'; please report this issue on https://
yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U
to update. Be sure to call youtube-dl with the --verbose flag and include its c
omplete output.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpi8nwgst1\bu
ild\youtube_dl\YoutubeDL.py", line 760, in extract_info
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpi8nwgst1\bu
ild\youtube_dl\extractor\common.py", line 429, in extract
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpi8nwgst1\bu
ild\youtube_dl\extractor\youtube.py", line 1642, in _real_extract
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpi8nwgst1\bu
ild\youtube_dl\extractor\youtube.py", line 1091, in _decrypt_signature
youtube_dl.utils.ExtractorError: Signature extraction failed: Traceback (most re
cent call last):
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpi8nwgst1\bu
ild\youtube_dl\extractor\youtube.py", line 1081, in _decrypt_signature
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpi8nwgst1\bu
ild\youtube_dl\extractor\youtube.py", line 969, in _extract_signature_function
youtube_dl.utils.ExtractorError: Cannot identify player 'https://www.youtube.com
/yts/jsbin/player-vflppxuSE/en_US/base.js'; please report this issue on https://
yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U
to update. Be sure to call youtube-dl with the --verbose flag and include its c
omplete output.
(caused by ExtractorError("Cannot identify player 'https://www.youtube.com/yts/
jsbin/player-vflppxuSE/en_US/base.js'; please report this issue on https://yt-dl
.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to u
pdate. Be sure to call youtube-dl with the --verbose flag and include its comple
te output.",)); please report this issue on https://yt-dl.org/bug . Make sure yo
u are using the latest version; type  youtube-dl -U  to update. Be sure to call
youtube-dl with the --verbose flag and include its complete output.

See: #12885


Update for OCaml 5.00

Only amd64 at the moment and not msvc

CI likely to fail until https://github.com/ocurrent/ocaml-version/pull/44 is merged and released.

Updated:

CentOS/Oracle Linux 7 and earlier won't ever support OCaml 5.00.0 (GCC is too old)
Express the conditions slightly differently to avoid the need for newer ocaml-version (for now)


override Swift_Transport_SendmailTransport $private $_params attribute

Hi,
I'm trying to connect to a smtp server which needs 3 options:

verify_peer = false
verify_peer_name = false
allow_self_signed = true

I noticed that _params is used in StreamBuffer and is received from Swift_Transport_SendmailTransport where they are defined in private $_params attribute.
Am I missing something or it is just not possible to override these settings?
Shouldn't one be able to override params with custom values?
Thanks

I'm sorry guys,
I just realized that there is a public setStreamOptions method which allows me to do exactly what I was trying to do.
Might come in handy for someone else.


fix!: emergency hard fork to fix masternode voting

Issue being fixed or feature implemented
Masternode voting was broken in protocol version 6 which introduced nonce validation for masternode voting. This is because we would verify against the owner identity nonce instead of the voting identity nonce.
What was done?
Changed to validate against the voting identity nonce in protocol version 7.
For protocol version 6, did the fix if we are past epoch 13 included.
How Has This Been Tested?
Not yet tested. Will be done on testnet.
Breaking Changes
This is a breaking change.
Checklist:

[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have added or updated relevant unit/integration/functional/e2e tests
[X] I have added "!" to the title and described breaking changes in the corresponding section if my code contains any
[X] I have made corresponding changes to the documentation if needed

For repository code-owners and collaborators only

[X] I have assigned this pull request to a milestone

@coderabbitai review all


Macro expansion & delimiters detail

MODSEC-314: We should add an explicit entry to all operators to tell if they support macro expansion.
For operators supporting a list of parameters (pm, within), we should explicitly state which delimiter is used).

On v3 this list is clear as the class that supports variable expansions has m_couldContainsMacro set to true.
https://github.com/SpiderLabs/ModSecurity/blob/587cbf39153fccb9cd129433609835fa83064bc4/src/operators/within.h#L34

Maybe non-developers would also want this in the doc, no?

Maybe non-developers would also want this in the doc, no?

Sure. Now we have the technical meaning to do this as oppose to before, where those couldn't be computed easily. This means that documentation will be easy. It is a know issue that we have to have a better documentation. Want to help us on that task?

As I'm working with Apache, I'm still in 2.9 ...


stock example - generation of localisation files does not work

Steps to Reproduce
I try to regenerate the i18n directory to try out the localisation process but there is no output
(see below)
This in the stock example. The process is described in the regenerate.md
It seems like the complete process relies on that process to work.
Platform is Windows 10
Logs
C:\sdks\flutter\examples\stocks>flutter --verbose  pub pub run intl_translation:extract_to_arb --output-dir=lib\i lib\stock_strings.dart
[  +25 ms] [c:\sdks\flutter\] git rev-parse --abbrev-ref --symbolic @{u}
[  +82 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[        ] origin/alpha
[        ] [c:\sdks\flutter\] git rev-parse --abbrev-ref HEAD
[  +60 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] alpha
[        ] [c:\sdks\flutter\] git ls-remote --get-url origin
[  +62 ms] Exit code 0 from: git ls-remote --get-url origin
[        ] https://github.com/flutter/flutter.git
[        ] [c:\sdks\flutter\] git log -n 1 --pretty=format:%H
[  +62 ms] Exit code 0 from: git log -n 1 --pretty=format:%H
[        ] 8f65fec5f5f7d7afbb0965f4a44bdb330a28fb19
[        ] [c:\sdks\flutter\] git log -n 1 --pretty=format:%ar
[  +62 ms] Exit code 0 from: git log -n 1 --pretty=format:%ar
[        ] 2 days ago
[ +162 ms] c:\sdks\flutter\bin\cache\dart-sdk\bin\pub.bat run intl_translation:extract_to_arb --output-dir=lib\i lib\stock_strings.dart
c:\sdks\flutter\packages\flutter\pubspec.yaml has changed since the pubspec.lock file was generated, please run "pub get" again.
[ +403 ms] "flutter pub" took 442ms.
pub finished with exit code 65

#0      throwToolExit (package:flutter_tools/src/base/common.dart:28)
#1      pubInteractively (package:flutter_tools/src/dart/pub.dart:176)
<asynchronous suspension>
#2      PackagesPassthroughCommand.runCommand (package:flutter_tools/src/commands/packages.dart:128)
#3      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:256)
<asynchronous suspension>
#4      FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:196)
<asynchronous suspension>
#5      CommandRunner.runCommand (package:args/command_runner.dart:194)
<asynchronous suspension>
#6      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:279)
<asynchronous suspension>
#7      CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:109)
#8      new Future.sync (dart:async/future.dart:222)
#9      CommandRunner.run (package:args/command_runner.dart:109)
#10     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:162)
#11     run.<anonymous closure> (package:flutter_tools/runner.dart:89)
<asynchronous suspension>
#12     AppContext._run (package:flutter_tools/src/base/context.dart:76)
<asynchronous suspension>
#13     AppContext.runInZone.<anonymous closure> (package:flutter_tools/src/base/context.dart:66)
#14     _rootRun (dart:async/zone.dart:1120)
#15     _CustomZone.run (dart:async/zone.dart:1001)
#16     runZoned (dart:async/zone.dart:1467)
#17     AppContext.runInZone (package:flutter_tools/src/base/context.dart:65)
#18     run (package:flutter_tools/runner.dart:60)
<asynchronous suspension>
#19     main (package:flutter_tools/executable.dart:43)
<asynchronous suspension>
#20     main (file:///c:/sdks/flutter/packages/flutter_tools/bin/flutter_tools.dart:16)
#21     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:263)
#22     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:151)

[  +51 ms] ensureAnalyticsSent: 43ms
[   +8 ms] exiting with code 65

C:\sdks\flutter\examples\stocks>

Flutter Doctor
C:\sdks\flutter\examples\stocks>flutter doctor
[√] Flutter (on Microsoft Windows [Version 10.0.16299.98], locale en-US, channel alpha)
    • Flutter at c:\sdks\flutter
    • Framework revision 8f65fec5f5 (2 days ago), 2017-12-12 09:50:14 -0800
    • Engine revision edaecdc8b8
    • Tools Dart version 1.25.0-dev.11.0
    • Engine Dart version 2.0.0-edge.d8ae797298c3a6cf8dc9f4558707bd2672224d3e

[√] Android toolchain - develop for Android devices (Android SDK 27.0.1)
    • Android SDK at C:\Users\ride4\AppData\Local\Android\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-27, build-tools 27.0.1
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b01)

[√] Android Studio (version 3.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b01)

[√] IntelliJ IDEA Community Edition (version 2017.2)
    • Flutter plugin version 19.1
    • Dart plugin version 172.4343.25

[√] Connected devices
    • Moto G 5 Plus             • ZY224DHFWG    • android-arm • Android 7.0 (API 24)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 7.1.1 (API 25) (emulator)

I have the same problem on Windows 8, but I get the following
C:\my-dev\Projects\flutter\flutter-master\examples\stocks>flutter pub pub run intl_translation:extract_to_arb --output-dir=lib/i18n lib/stock_strings.dart
c:\my-dev\ide\flutter\packages\flutter\pubspec.yaml has changed since the pubspec.lock file was generated, please run "pub get" again.
pub finished with exit code 65

@robinbakkerus  Can u run on verbose and see if you get more info.

Stocks example run failed.

@sethladd could we add the look label to this.

done, thanks!

This works for me now.
Can we close the issue?


Support f!(y, x) for Zygote with Zygote.Buffer

See https://fluxml.ai/Zygote.jl/stable/utils/#Zygote.Buffer
Ping @Vaibhavdixit02

PR #360 fails because

Buffer is not an AbstractArray and can't be used for linear algebra operations like matrix multiplication. This prevents it from being captured by pullbacks.

and some functions are typed f!(y::AbstractArray, x::AbstractArray). I think in general this fix would only cover a very limited subset of possible mutating operations, so I'm not sure it's worth it

Wait, I think the main thing it does, and what I had in mind asking for it was supporting in-place functions that's it. It has been quite robust in my experience and by documenting that the extent of mutation here is just in-place function calls why is this not worth it? Doesn't that mean that you'd be able to mark off the Two-argument column for Zygote here https://gdalle.github.io/DifferentiationInterface.jl/DifferentiationInterface/dev/backends/#Features?

Gonna give it another try. Hopefully all it takes is to remove a few type annotations for my test suite to pass

Buffer was discussed on Slack today and described as very hacky and buggy, so I think this feature would cause more harm than good in the end.


Spreadsheet: Clarify the location data instructions for users

When I first opened it up, I thought I had to fill in all the cells that had errors in them otherwise it wouldn't work.  Which was overwhelming. I thought "oh this won't work without input"

I suggest a few things:

Have error-checking in the auto-calculated cells. =IF(A1, A1/B1, ) so the cell remains blank if it doesn't have the proper data.
Add an explanation above the 2nd column to explain how to use it (see screenshot below for a suggested format)
Add "updated often" to the label (column E), not the empty text. Also, change it to a clear suggestion. Like "Update this every 2-3 weeks"
Move "date last updated" to the top of the data area so it's easier for the person to know they need to update it
Automatically highlight  "date last updated" in yellow/red if it's more the X days old. (Or alternately, have error text that shows up in the cell next to it.
Move the "fill in your location" column to the right. Since if someone is in the bay (like me) then most of the info is already there. But if someone needs to fill it in, it's much easier to have the labels right next to the stuff that needs to be filled in.
I think it'd be best if all the "need to update frequently" cells were right next to each other with the calculated cells groups together at the end. That would give a better user experience. (I didn't bother to do demo this in the example below because it would have messed up all your formulas elsewhere. This suggestion might not be worth the work. But thought I'd toss it in anyway.

Here's how I rigged it up

Not the most beautiful thing in the world. But you get the idea.
Here's a link if you wanna borrow from it. You can also find the error checking formulas I used: https://docs.google.com/spreadsheets/d/1KHi1kbnvWSMBDd-gKux7eYjNbL4_54uI09ZUrH8Mo_k/edit#gid=601829656
(this is many "issues" in one, hope that's okay.)

Closing in favor of #269


Redundancy in save data

The save system is effectively a bottom-up state management, where each individual piece of state is put into a monolithic state object that is then serialized and deserialized. For saving, the object is constructed by traversing each layer object and adding the data marked as persistent into the save data object, in the same position. For loading, each layer is traversed and given the value at the same location in the save data object, if it exists.
The issue comes when multiple parts of a layer both reference the same piece of persistent data. Each piece will be saved and loaded multiple times, one for each time its seen in the layers' structures. This is slightly remedied by tracking features that have already been included and skipping duplicates, but there's still significant duplication from individual pieces of state that leads to increased save file size, as well as slightly longer load times.
Perhaps this can be solved by having an object "own" the persistent data, and only the owner records the location. This has problems to solve, like the owner typically not existing when the data is created, it becoming a nuisance to project developers, and it becoming less robust to layer structure changes.

Map and Set both automatically delete duplicates, right? You might use something that uses them then.

Jakub asked about marking the "source of truth" location, and I had these thoughts:
I'd be down for that as well. For the points example, that would mean in the returned layer object you'd write points: persist(points), or something similar. I'm not a huge fan of persist as the name, since it'd be confusingly similar to the existing function persistent, but that's totally valid
that does have the issue where if I reference a property that has been marked as the source of truth, it'd have to strip away that persist call. That could probably be done in the lazy proxy though
to clarify, if I have a challenge that automatically created a persistent number for completions, I'd want it to be the source of truth, so challenge.completions would be marked as the source of truth. If I passed that to somewhere else, I'd have to "unmark" it
but since all features are lazy proxies, the lazy proxy could handle that in its getter

further conversation: https://canary.discord.com/channels/619613842141347841/1029790117075890236

This appears to be fixed, but at the cost of reset issues.

Fixed by https://github.com/profectus-engine/Profectus/commit/4207677944eb38aba5c41ec9596b09ab6bea6e1c and some follow-up commits


Chat-GPT or even Microsoft Bing Search

Description of the new feature / enhancement
Implement into the Windows Launcher the ability to request to start a chat with the new Bing AI.
Scenario when this would be used?
Instead of Google/Bing Searching it would pull up the AI chat bot of your choice.
Supporting information
Ahead of the curve and right on the wave of trending. #23143

/dup #24445


test(components): [button] cover remaining props

恭喜成为贡献者 @w2xi https://github.com/FightingDesign/fighting-design/commit/c9e96578372f40e328fedaeffe7ae6b8c9ffc232


effdet:benchmark coverage for custom devices

This PR is for https://github.com/pytorch/benchmark/pull/2374.
Previously, effdet hardcoded the CUDA device in two ways:

Initializing the check for 'cuda', only
Moving intermediate tensors to 'cuda', explicitly

To add custom device support, this PR:

Removes the check for cuda only, which will allow the device to be all custom devices that supports 'to(device)' functionality, such as 'cpu', 'xpu', and 'cuda'
Moves intermediate tensors to the appropriate devices by giving the param of devices.

@rwightman Hi Ross, could you please help review this PR? We need it for our PR for Pytorch. Thank you very much.


64-32 setup should use 32-bit pentry for calling conventions

Typically a function will have undefined4 for 32-bit and undefined8 for 64-bit as the default undefined type for arguments
On a 64-32 setup, such as mips o64/o32/n32, where the cspec pointer and long are 32-bit, why choose undefined8?
My guess is SLEIGH/PCODE related, where the PCODE may be telling you the register size is 64-bit in these cases, but when you have it defined as a specific size in the cspec, shouldn't that be the size used when possible?
I have noticed that a lot of times when it is decided to be undefined8 it will then have _X_Y notation when it is used in the function as an assignment, but if changed to undefined4 it will pick up the type that it is being assigned to.

Typically a function will have undefined4 for 32-bit and undefined8 for 64-bit as the default undefined type for arguments
On a 64-32 setup, such as mips o64/o32/n32, where the cspec pointer and long are 32-bit, why choose undefined8?
My guess is SLEIGH/PCODE related, where the PCODE may be telling you the register size is 64-bit in these cases, but when you have it defined as a specific size in the cspec, shouldn't that be the size used when possible?
I have noticed that a lot of times when it is decided to be undefined8 it will then have _X_Y notation when it is used in the function as an assignment, but if changed to undefined4 it will pick up the type that it is being assigned to.

Is the register size 8 bytes? I'm fairly certain this would be a side effect of how input/output parameter storage is specified in a cspec.

Oh yeah, the pentry... That didn't even click last night, I suppose it is honoring the pointer_size and it's really the argument maxsize attr.  I would say that's a bug those are 8 bytes.  longlong would use the multi register setup/instructions.

Current binary is 64-32_o32 BE.  I just tried a new ldef entry/cspec with maximum 4 and tried changing languages and reloading the image and it resulted in hi usage instead of lo resulting in all the uncommitted functions breaking in either case.
When testing this new ldef/cspec with a dummy C file with 8 arguments (mixed sized and mixed size return), LE seemed fine and BE was broken.
I was unable to figure out the gcc flags for m64-32_o32, but could n32.  n32.spec did hate the join, which o32.cspec has commented out already and comment, so something about this appears to be a known issue maybe?
Guessing this is why the e500 support is like the way it is on PowerPC?

unsigned int foo(unsigned long long x)
{
    return (unsigned int)x;
}
unsigned long long bar(unsigned int x)
{
    return (unsigned long long)x;
}
void test(void)
{
    unsigned int a = foo((unsigned long long)0x1122334455667788ULL);
    unsigned long long b = bar((unsigned int)0x11223344UL);
    foo(b);
    bar(a);
}

mips64-linux-gnuabi64-gcc -mabi=32 -mips64 ./test.c -c
For example in 10.2.2 mips64-32 o32, the min/max pentry with 1/8 has issues with arguments and return values as it splits usage across a0/a1 and v0/v1.  -mabi=n32 does use the full a0 and v0 register so min/max pentry of 1/8 is fine.
So maybe my question is really only MIPS:BE:64:64-32addr o32 related.

Try the following for your mips64_32_o32.cspec
        <pentry minsize="1" maxsize="8" metatype="float">
          <register name="f14"/>
        </pentry>
        <pentry minsize="1" maxsize="4" extension="sign">
          <register name="a0_lo"/>
        </pentry>
        <pentry minsize="1" maxsize="4" extension="sign">
          <register name="a1_lo"/>
        </pentry>
        <pentry minsize="5" maxsize="8" extension="sign">
          <addr space="join" piece1="a0_lo" piece2="a1_lo"/>
        </pentry>
        <pentry minsize="1" maxsize="4" extension="sign">
          <register name="a2_lo"/>
        </pentry>
        <pentry minsize="1" maxsize="4" extension="sign">
          <register name="a3_lo"/>
        </pentry>
        <pentry minsize="5" maxsize="8" extension="sign">
          <addr space="join" piece1="a2_lo" piece2="a3_lo"/>
        </pentry>
        <pentry minsize="1" maxsize="500" align="4">
          <addr offset="16" space="stack"/>
        </pentry>
      </input>
      <output>
        <pentry minsize="1" maxsize="8" metatype="float">
          <register name="f0"/>
        </pentry>
        <pentry minsize="1" maxsize="4" extension="sign">
          <register name="v0_lo"/>
        </pentry>
		<pentry minsize="5" maxsize="8">
          <addr space="join" piece1="v0_lo" piece2="v1_lo"/>
        </pentry>
        -->
      </output>

This forces the use of the lower half of all registers only, and it should work for BE or LE I think, as long as the xx_lo registers are specified.
It didn't automatically recover the parameters as the correct size at first, but it did recover that they existed and passed the correct values.
void test(void)

{
  undefined4 uVar1;
  undefined4 uVar2;
  undefined4 in_v1_lo;

  uVar1 = foo(0x11223344,0x55667788);
  uVar2 = bar(0x11223344);
  foo(uVar2,in_v1_lo);
  bar(uVar1);
  return;
}

After specifying "int foo(longlong x), the decompiler produces:
longlong test(void)

{
  int iVar1;
  longlong lVar2;

  iVar1 = foo(0x1122334455667788);
  lVar2 = bar(0x11223344);
  foo(lVar2);
  lVar2 = bar(iVar1);
  return lVar2;
}

If this works for you, we can change the calling convention, or you can submit a PR.

Thanks for taking a look ... I'll def play with it this weekend

This is looking like a 6-ft one way and half a dozen the other way.   It looks like it fixed the usage of undefined8 and used undefined4.  Which looks like its allowing it to detect and convert to a structure pointer now.
On the flip side the comparison instructions are on the 4-byte _lo register and it is wanting a 8-byte register resulting in lots (not all though?)  of CONCAT44 with the _hi.   Sometimes adding in a bunch of extraout_v0_hi stack variables.  I checked a copy of my current and loading it fresh.
I can't suggest to push this change.  The CONCAT would be too noticeable I think.  Thanks again for the XML to test, I had not thought to use the _lo like you had.
I guess I'm still lost on why the registers get endianess applied, maybe I'm just not thinking through the use cases.  If the calling  convention is wanting to use the _lo and the instructions know the _hi is the most significant 32-bit and _lo least significant of the 64-bit, why does the base register in the pentry (no _lo or _hi) default to using _hi in the function prototypes when its a pentry max of 4.

Welcome to my world!!!  Somehow we need a foolproof way of getting Ghidra to work correctly with sized pointers.


Idea 182.x doublebrackets

Thanks for your review. I'll open another pull request with the changes you requested.


feat: Add PR template to project

Why the changes were made
Help the project review process by:

adding a why section, so we understand why that PR is important and can help us prioritizing our work
adding a how section, so we can understand the thought process behind a PR
adding a testing section, so we do not only read the proposed code changes, but also test the proposed changes prior to approving a PR

Help the project quality by:

standardizing PR's descriptions
documenting processes

How the changes were made
Searched for some templates in the internet to help create the one proposed in this PR.
How to test the changes made
Read the proposed template and make sure it is clear and helps achieve this PR's goal.

to be most effective @kaovilai and @mateusoliveira43 IMHO we should align our template as closely to https://github.com/vmware-tanzu/velero/blob/main/.github/pull_request_template.md as possible. Over time the two templates should converge to be exactly the same ideally.

@weshayutin they won't be exactly the same, but the differences should coincide with actual differences in development model. For example, we don't have the DCO signoff or changelog files per PR.

@sseago 100% agree :)

/LGTM

/override ci/prow/4.12-ci-index
/override ci/prow/4.12-images
/override ci/prow/4.13-ci-index
/override ci/prow/4.13-images
/override ci/prow/4.14-ci-index
/override ci/prow/4.14-images
/override ci/prow/unit-test


Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.

I can't seem to install or use node-gyp.  Here's the output when I try to start up a Hubot locally.  This is on OSX, with node installed via homebrew:
gyp ERR! configure error
gyp ERR! stack Error: `gyp` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onCpExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/configure.js:343:16)
gyp ERR! stack     at emitTwo (events.js:87:13)
gyp ERR! stack     at ChildProcess.emit (events.js:172:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:200:12)
gyp ERR! System Darwin 14.5.0
gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /private/tmp/hubot-gordon/node_modules/hubot-wolfram/node_modules/wolfram/node_modules/libxmljs
gyp ERR! node -v v4.1.2
gyp ERR! node-gyp -v v1.0.3
gyp ERR! not ok
npm ERR! Darwin 14.5.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install"
npm ERR! node v4.1.2
npm ERR! npm  v2.10.1
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the libxmljs package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls libxmljs
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /private/tmp/hubot-gordon/npm-debug.log

Sounds like an issue on your system. Would be helpful if you provided some information about what you have tried to do to debug this.

Same. I'm on Windows 7. Fresh install of  Visual Studio 2013
Trying:
npm install gulp-cordova-bump
7996 verbose stack Error<EMAIL_ADDRESS>install: node-gyp rebuild
7996 verbose stack Exit status 1
7996 verbose stack     at EventEmitter. (C:\DevSoft\nodejs\node_modules\npm\lib\utils\lifecycle.js:214:16)
7996 verbose stack     at emitTwo (events.js:87:13)
7996 verbose stack     at EventEmitter.emit (events.js:172:7)
7996 verbose stack     at ChildProcess. (C:\DevSoft\nodejs\node_modules\npm\lib\utils\spawn.js:24:14)
7996 verbose stack     at emitTwo (events.js:87:13)
7996 verbose stack     at ChildProcess.emit (events.js:172:7)
7996 verbose stack     at maybeClose (internal/child_process.js:818:16)
7996 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:211:5)
7997 verbose pkgid<EMAIL_ADDRESS>7998 verbose cwd C:\Dev\Projects\my-project
7999 error Windows_NT 6.1.7601
8000 error argv "C:\DevSoft\nodejs\node.exe" "C:\DevSoft\nodejs\node_modules\npm\bin\npm-cli.js" "install" "--save-dev" "gulp-cordova-bump"
8001 error node v4.2.2
8002 error npm  v2.14.7
8003 error code ELIFECYCLE
8004 error<EMAIL_ADDRESS>install: node-gyp rebuild
8004 error Exit status 1
8005 error Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
8005 error This is most likely a problem with the libxmljs package,
8005 error not with npm itself.
8005 error Tell the author that this fails on your system:
8005 error     node-gyp rebuild
8005 error You can get their info via:
8005 error     npm owner ls libxmljs
8005 error There is likely additional logging output above.


Automatically parse locale

fix #13438

Checklist

[ ] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[ ] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[ ] I've updated the documentation if necessary.

Motivation and Context

Locale is handled manually by checking environment variable, should use locale charmap instead which automatically maps utf8 to UTF-8.
Description

Check if output of locale charmap matches UTF-8.

@joshdholtz Sorry, there was a type s/chapmap/charmap/. Nice findings.

That the tests pass here, shows that there is one or more tests missing that check for this error message by the way ;)
Maybe this can be rewritten so it uses locale charmap where available and the old logic where not?

I think we could add some logic here to check the OS and and check if the locale command exists but I thhiiinnnkkkk it might be more straight forward, easier to maintain, and easier to test if we just add the same kind of hardcoded check for en_US.utf8

Especially as the current version would add a which ... call for each execution of fastlane :/ We don't want to do that.

@joshdholtz What about some other cases like UTF8 or some weird cases which should not work. I think it's better to leave it for locale to handle.
@janpio Any alternatives? I think if we need to use environment variables, we need to take a look at how locale was implemented.
By the way, why do we need to check the locale? What are the implications if the locale is not UTF-8?

@janpio Any alternatives? I think if we need to use environment variables, we need to take a look at how locale was implemented.

When I suggested that, I thought about trying the locale command and handling an eventual error by looking at the ENV vars. That would get rid of the additional which, but of course not the actual locale ... call itself.

@janpio Nice, I didn't know I can do that in ruby.

Me neither, rubocop told me ;)
Some stuff left:
[02:05:51]: $ bundle exec rubocop -D
[02:05:52]: ▸ Inspecting 1032 files
[02:07:26]: ▸ ...........................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................C............................................................................................................................................................................................................................................................................................................................................................................................................................................................
[02:07:26]: ▸ Offenses:
[02:07:26]: ▸ fastlane/lib/fastlane/cli_tools_distributor.rb:19:7: C: Metrics/PerceivedComplexity: Perceived complexity for take_off is too high. [20/18]
[02:07:26]: ▸ def take_off
[02:07:26]: ▸ ^^^
[02:07:26]: ▸ 1032 files inspected, 1 offense detected

I am not too sure what the best way to fix this would be. @joshdholtz ?

@janpio I tried to reduce the cyclomatic complexity by splitting it into another function. Thanks for reminding me of the failed lint.
@joshdholtz Feel free to give any suggestion? :smile:

This should be good to merge now, what do you think @joshdholtz?

Can you take another look before merging @joshdholtz and @minuscorp please? Thanks.


[Mailing Lists] Add the rest of the mailing lists to /subscribe page

Current State of the Meshery.io site
https://meshery.io/subscribe is missing entries for all the project mailing lists.
Desired State of the Meshery.io site
Have all lists available on /subscribe:

| Meshery Developers |<EMAIL_ADDRESS>|
| Meshery Users |<EMAIL_ADDRESS>|
| Meshery Maintainers |<EMAIL_ADDRESS>|
| Meshery Security and Vulnerability Reports |<EMAIL_ADDRESS>|
| Meshery Community |<EMAIL_ADDRESS>|

Contributor Resources
The meshery.io website uses Jekyll and GitHub Pages. Site content is found under the master branch.

See the Contributing to Meshery.io Website section of the readme.md and other contributing instructions, too.
See Meshery site designs in this Figma designfile. Fill-in the Layer5 Community Member Form and join the Layer5 Community Slack for access.

Hey @leecalcote , Can I Take this ?

Sure @Zain-Muiz go ahead.

Hi @Zain-Muiz, how are we doing here?

Hey @leecalcote , We are ready here ! Having a small issue with my forked repo. Will need to re-fork. As soon as https://github.com/meshery/meshery.io/pull/523 gets merged, will have PR up for this issue as well !

@Zain-Muiz good deal. I wasn't sure that you were working through this or not. We need mailing lists published on smp-spec.io, meshery.io and getnighthawk.dev. In trying to get this behind us, I put in a few changes last night and just pushed before seeing your note. Will you review the main page and the mailing lists page to see if we need anything further here or if we can close this issue?
It would be great to get the getnighthawk mailers published on that site. Mailing lists


ParquetFileReader

Codecov Report

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

@@           Coverage Diff           @@
##           master      #98   +/-   ##
=======================================
  Coverage   55.37%   55.37%
=======================================
  Files           9        9
  Lines         186      186
=======================================
  Hits          103      103
  Misses         81       81
  Partials        2        2

Continue to review full report at Codecov.

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

thanks! Is this ready to test?

I have added an simple example in examples/parquet/parquet_reading.go. It's ok.
Maybe much more cases should be added and tested, which can also test the parquet-go :)


Repo sync for protected CLA branch

The pull request is created from main637799424116725292 to main to fix git push error for protected CLA branch

@opbld15 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


extending edges with additional shapes

Hey Guys,
at first a very nice library you have build here.
At Second:
how its possible to extend existing edges by more shapes? Your Documentation on customizing them has not enough information about things like adding more shapes or else.
I've tried to add a shape with following code:
G6.registerEdge("dataLabelsEdge", {
                options: {
                    style: {
                        stroke: "#000"
                    }
                },
                labelAutoRotate: true,
                labelPosition: 'center',
                refX: 0,
                refY: 0,
                // Response the states change
                afterDraw(cfg, group) {
                    const shape = group.get('children')[0];
                    const startPoint = shape.getBBox();
                    if (cfg.data && cfg.data.status) {
                        // the left label
                        group.addShape("text", {
                            attrs: {
                                text: 'Status: ' + cfg.data.status,
                                fill: "#595959",
                                textAlign: "start",
                                textBaseline: "middle",
                                x: startPoint.x + 10,
                                y: startPoint.y - 10
                            },
                            name: "left-text-shape"
                        });
                    }
                    console.log(group);
                },
            }, 'line');

It seems that the Position of the shape is outside of the edge-element:

startPoint.x and startPoint.y are the top-left coordinates，ref to the demo：https://codesandbox.io/s/fast-dream-d9rto

thanks for that example, it is very helpful to understand how to create own additional shapes on an extended edge.  maybe it could be useful to add this information to the documentation, too ? :)

thanks for that example, it is very helpful to understand how to create own additional shapes on an extended edge. maybe it could be useful to add this information to the documentation, too ? :)

Thanks for your suggestion, we will improve the docs soon.


Uploading EML with bounding box info returns "Please use dot . as a decimal separator"

Steps

Upload EML with bounding box

      <geographicCoverage>
        <boundingCoordinates>
          <westBoundingCoordinate>3.5176</westBoundingCoordinate>
          <eastBoundingCoordinate>7.0243</eastBoundingCoordinate>
          <northBoundingCoordinate>53.4074</northBoundingCoordinate>
          <southBoundingCoordinate>50.699</southBoundingCoordinate>
        </boundingCoordinates>
      </geographicCoverage>

Edit metadata
Save basic metadata
Error message:

Geographic Coverage section was not validated! This needs to be fixed before all sections can save successfully.

Go to Geographic coverage. Message: Please use dot . as a decimal separator
Save Geographic coverage without changing anything.
Error resolved.

Note that the bounding box coordinates already have decimal separators, so not sure what is causing this issue.

Thank you Peter. This sounds odd, I'll check

@peterdesmet I believe this is because you don't have a required description in the geographic coverage.
Please use dot . as a decimal separator is just a reminder not an error message.
In a new version (3.0-SNAPSHOT) there is an option to validate an eml you're trying to upload

Could you send me your eml file so I can check it please?

It is indeed a file with bounding box, but no taxonomic coverage description
eml.xml.txt (added txt extension to allow upload to GitHub)

Hmm, for all the bird tracking datasets I also used version 2.2 when uploading EML metadata and that worked fine (except validation maybe). Is this issue related to the version?

I think so. IPT can process EML 2.2 but result can be erratic due to differences in versions.

Is this still relevant please?


Remove default argument for API key

None is acceptable by remotivelabs-broker API.

@carlmartus is this something that should be merged?


Feature: Enhance landing page ("Loading..." text)

Is your feature request related to a problem? Please describe.
When reaching the portal the user first lands on a blank page with the "Loading..." text on the upper left corner which isn't very visual, and there is no indication that the page is actually going to redirect later.
Describe the solution you'd like

[ ] "Beautify" the page, maybe with a design close to the main portal design
[ ] Add a loading animation

Duplicate #560


syntax: pamconf: recognize '-' at the beginning

Let vim recognize - at the beginning of a line in pam.conf. See PAM.CONF(5) for the relevant documentation.

Please review it again. This time:

Files with (/etc/pam.conf) or without the first "service" field are distinguished based on their path. By default, treat the file as if it does not have a service field, since most Linux distributions ship /etc/pam.d/*, and /etc/pam.conf is not read by PAM if the other exists.
Orders are changed so that pamconfService comes last to match the first field. Otherwise pamconfType overrides the highlight group.

I haven't come up with a better way that supports both file types.
To test:
Create an etc directory anywhere and place a file named pam.conf in it, with the following content:
#%PAM-1.0
# vim: ft=pamconf:

aa-aa123 session    required   pam_env.so           user_readenv=1
aabbcc  -session    required   pam_env.so           user_readenv=1

Create etc/pam.d directory and place a file in it (name does not matter):
#%PAM-1.0

auth            include         system-login
-auth           optional        pam_gnome_keyring.so
password        include         system-login
-password       optional        pam_gnome_keyring.so use_authtok
session         include         system-login
-session        optional        pam_kwallet5.so auto_start

I got a reply from the previous maintainer:

I don’t maintain any Vim runtime files anymore.  Please consider taking over maintainership and send your patch to Bram directly.

@k-takata Could you please take another look at it?

LGTM
@brammool Could you consider including this?

Setting b:pamconf_has_service_field in filetype.vim is strange.  Let me check the file name in the syntax file.
Please check once it's updated.

Thanks for the update! The latest runtime/syntax/pamconf.vim change in e7b1ea0276cc83cd5c612f3189a174a60d57b5ed looks good to me.


Use map to insert and delete data can case memory leak

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.9.2 linux/amd64
Does this issue reproduce with the latest release?
Yes, latest release download from https://golang.org/dl/
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build363981589=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
What did you do?
Use map to insert and delete data, code like this:
package main

import (
	"fmt"
	"runtime"
	"runtime/debug"
)

func main() {
	a := make(map[int]int)

	stats := new(runtime.MemStats)
	runtime.ReadMemStats(stats)
	fmt.Printf("Before: %5.2fk\n", float64(stats.Alloc)/1024)
	previous := stats.Alloc

	for i := 1000000; i > 0; i-- {
		a[i] = 0
	}

	for k := range a {
		delete(a, k)
	}

	debug.FreeOSMemory()
	fmt.Printf("len(a)=%v\n", len(a))

	runtime.ReadMemStats(stats)
	fmt.Printf("After: %5.2fk    Added: %5.2fk\n", float64(stats.Alloc)/1024, float64(stats.Alloc-previous)/1024)

	return
}

What did you expect to see?
Memory not increased obviously
What did you see instead?
Memory increased obviously, output like this:
Before: 42.54k
len(a)=0
After: 39252.77k    Added: 39210.23k

I do not know why so many memory still exist after i delete all elements in map.

Dup of #20135.


Change node pool labels to respect node restrictions

https://github.com/banzaicloud/pipeline/issues/1964

Does not affect us, see linked issue.


It's good to add the features

1.Add the keyboard shortcut mapping.
2. Set a keyboard shortcut to make it can switch every change-case case.
such as:
press ctrl + c: apply to camel command.
press ctrl + c again: apply to constant command.

Oracle's SQL Developer has a toggle case shortcut like this (Ctrl-') and it works great. It cycles through a number of common case types. You highlight a word, hold down control, and keep pressing the single quote key until it switches to the case you want. Fast and intuitive once you get used to it. Definitely a worthwhile feature to add.


df.ta.to_utc unexpected behavior caused by JupyterLab

Which version are you running? The lastest version is on Github. Pip is for major releases.
0.3.14b0
Do you have TA Lib also installed in your environment?
yes
Did you upgrade? Did the upgrade resolve the issue?
I try upgrade, not solve.
Describe the bug
The df.ta.to_utc is a property, but it is not idempotent. When you type df.ta. then press tab for auto completion, it will cause the df.ta.to_utc be called. I don't know why this design as a property rather a method.
To Reproduce

df = ... # In JupyterLab, create a dataframe with non-UTC datetime index.

Type df.ta., then press tab.

Show the df again. If you can't see the index time zone changed, try step 2 multi-times then try step 3 again, you will see it.

Expected behavior
The df.ta.to_utc property should design as a method.
Screenshots
https://user-images.githubusercontent.com/4510984/155868723-36fe4bb3-31b2-4221-a3f9-6e392e78b42d.mp4

Hello @GF-Huang,

The df.ta.to_utc is a property, but it is not idempotent.  I don't know why this design as a property rather a method.

Yeah. Good point.
Contributions are welcome to improve the library. 😎
Kind Regards,
KJ


Update MINLPTests to v0.3

This is now registered.

See https://github.com/JuliaOpt/MINLPTests.jl/pull/8.


Prevent random attributes being set.

Not sure how/why, and might very well be something simple I'm missing!
I'm noticing that some entities I generate such as a path or text are generating SVG elements with inline attributes such as font-family, x or dx, etc... with values that are unexpected and seemingly at random.
Anyone else have this issue?  Am I missing a something fundamentally here?
It forces me to do things such as-
var var2 = target.text(function (add) {
    add.tspan(opts.variable2).addClass('slide-variable2-label')
       .y(opts.barLength).dy('1em');
}).x(v2Xoffset).attr('font-family','');

Note the clearing of font-family so I can target the element via CSS instead.

There is nothing random about that. In order to render things in a definite way, svg.js provides default values for several attributes including font-family and font-size. You can change this values.
Take a look at the SVG.defaults object (https://github.com/wout/svg.js/blob/master/src/default.js)

Did this answer resolve your issue? I'll be closing this for now.

I can see setting reasonable defaults  for attributes not accessible through CSS, but for the properties that can be targeted via style sheets it's a bit intrusive IMO.
Maybe a good compromise could be an options setting where setting display property default values could be disabled?

Some functionality of svg.js is based on these properties. E.g. to correctly calculate the leading of lines, we need a font-size to be set. One may discuss if we should use getComputedStyle but this would add one more dom dependeny and is slower. I dont know if this is worth it


Replace remaining usages of Protobuf reflection with MessageDef API

Just as the subject says, this PR addresses the respective TODOs and gets rid of remaining usages of Protobuf reflection.
Besides, it moves the TimeExts.kt file to the Chords Proto Values library where it should belong.

@dpikhulya is it still a draft?

@dpikhulya I am asking because GitHub told me the review was required.

@armiol Yeah, it was a false alarm. I've found one more thing that needs to be addressed.

@armiol Okay, addressed one more thing and it's ready for review now.


🐞Loading spinner and text is hidden behind the top menu on the Dapp/State page

Describe the bug
When opening the page for the first time (so it's not cached), the loading animation and the Loading Elements screen is hidden behind the top menu on the Dapp/State page
To Reproduce
Go to a project's Dapp/Stake page and look at the top of the list
Expected behavior
The loading animation and the text next to it should be visible
Screenshots

Web3 Scenario (please complete the following information):

Env.: Desktop
OS: macOS 10.15
Browser: Safari
Version: 13.1.2
Provider: Metamask

Reward
Discord Username and #: the4kman#4895
ETH Wallet: 0xaa8070363a48e6C245FAb811054bBF2B0B2853D7

Metamask in Safari?
I tried to recreate the behaviour, but the loader works perfectly for me, can you explain me better what happened?

I'm using Safari to reproduce this behavior, I'm using Metamask in another browser (not really relevant in this case).
I can still reproduce this issue by simply navigating to the Dapp/State page of the dfohub project. The behavior is a bit different now - currently, there is a text hidden behind the menu that says "This R&D version is no longer supported because it is lower than 0.1.".
The State page actually properly displays its contents if I navigate away and back to it - the issue only occurs when you open the page the first time.


scripthaus: init at 0.5.0

Description of changes
needed by #280566

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
[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 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.

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

Result of nixpkgs-review pr 286194 run on aarch64-darwin 1

1 package built:

scripthaus

@eclairevoyant i've tried the 'normal' way and got stuck at tests and some compilation errors. Then i messed up the code... Now after re-reading the manual i've set CGO_ENABLED = 1; as it shoud be done and (sadly) disabled tests.
Thanks for review and for mentioning nix-init, what a great tool i didn't know about!

upstream fixed tests and released new version, version bumped and enabled tests

added updateScript and tested it locally with nix-shell maintainers/scripts/update.nix --argstr maintainer raspher


RL traffic lights in sumo imported network

Hello,
I have imported a sumo network file (net.xml) and I want to train RL traffic lights in this scenario. In order to do this I have create a custom scenario (with the routes) and  a custom enviroment (based on green_wave_env) to train the RL traffic lights. But I am getting some errors.
Here is the  code I used (based on green_wave example):
HORIZON = 200

N_ROLLOUTS = 20

N_CPUS = 2

inflow = InFlows()

inflow.add( veh_type="human",
            edge="S_N",
            probability=0.03125,
            departLane="0",
            departSpeed=1)

inflow.add( veh_type="human",
            edge="N_S",
            probability=0.03125,
            departLane="0",
            departSpeed=1)

inflow.add( veh_type="human",
            edge="W_E",
            probability=0.09,
            departLane="0",
            departSpeed=1)

inflow.add( veh_type="human",
            edge="E_W",
            probability=0.03125,
            departLane="0",
            departSpeed=1)

env_params = EnvParams()
sim_params = SumoParams(render=True,sim_step=1)
initial_config = InitialConfig()
vehicles = VehicleParams()
vehicles.add('human', num_vehicles=0)

net_params = NetParams(inflows=inflow,
    template='/home/lsi04/flow/tutorials/networks/grid.net.xml',
    no_internal_links=False)

additional_env_params = {
        'target_velocity': 50,
        'switch_time': 3.0,
        'num_observed': 2,
        'discrete': False,
        'tl_type': 'controlled'  }

initial_config=InitialConfig()

flow_params = dict(

    exp_tag='green_wave',

    env_name='JavierPradoGridEnv',

    scenario='JavierPradoScenario',

    simulator='traci',

    sim=SumoParams(
        sim_step=1,
        render=False, ),

    env=EnvParams(
        horizon=HORIZON,
        additional_params=additional_env_params, ),

    net=net_params,

    veh=vehicles,

    initial=initial_config,
)

def setup_exps():

    alg_run = 'PPO'

    agent_cls = get_agent_class(alg_run)
    config = agent_cls._default_config.copy()
    config['num_workers'] = N_CPUS
    config['train_batch_size'] = HORIZON * N_ROLLOUTS
    config['gamma'] = 0.999  # discount rate
    config['model'].update({'fcnet_hiddens': [32, 32]})
    config['use_gae'] = True
    config['lambda'] = 0.97
    config['kl_target'] = 0.02
    config['num_sgd_iter'] = 10
    config['clip_actions'] = False  # FIXME(ev) temporary ray bug
    config['horizon'] = HORIZON

    flow_json = json.dumps(
        flow_params, cls=FlowParamsEncoder, sort_keys=True, indent=4)
    config['env_config']['flow_params'] = flow_json
    config['env_config']['run'] = alg_run

    create_env, gym_name = make_create_env(params=flow_params, version=0)

    register_env(gym_name, create_env)

    return alg_run, gym_name, config

alg_run, gym_name, config = setup_exps()

ray.init(num_cpus=N_CPUS + 1, redirect_output=False)
trials = run_experiments(experiments={flow_params['exp_tag']: {'run': alg_run,'env': gym_name,'config': {
                **config
            },
            'checkpoint_freq': 20,
            'max_failures': 999,
            'stop': {'training_iteration': 20,},}
    })

Here are the errors I get:
Process STDOUT and STDERR is being redirected to /tmp/ray/session_2019-06-05_11-46-50_23722/logs.
Waiting for redis server at <IP_ADDRESS>:52973 to respond...
Waiting for redis server at <IP_ADDRESS>:65510 to respond...
Starting the Plasma object store with 13.471593267000001 GB memory using /dev/shm.

======================================================================
View the web UI at http://localhost:8889/notebooks/ray_ui.ipynb?token=4c9f25316d8d55134afb8e1d6b3d7f3c710f830bf1f93175
======================================================================

== Status ==
Using FIFO scheduling algorithm.
Resources requested: 0/3 CPUs, 0/1 GPUs
Memory usage on this node: 7.1/33.7 GB

Created LogSyncer for /home/lsi04/ray_results/green_wave/PPO_JavierPradoGridEnv-v0_0_2019-06-05_11-46-51ktobyge9 ->
== Status ==
Using FIFO scheduling algorithm.
Resources requested: 3/3 CPUs, 0/1 GPUs
Memory usage on this node: 7.1/33.7 GB
Result logdir: /home/lsi04/ray_results/green_wave
RUNNING trials:
 - PPO_JavierPradoGridEnv-v0_0:	RUNNING

Error processing event.
Traceback (most recent call last):
  File "/home/lsi04/anaconda3/envs/flow/lib/python3.5/site-packages/ray/tune/trial_runner.py", line 261, in _process_events
    result = self.trial_executor.fetch_result(trial)
  File "/home/lsi04/anaconda3/envs/flow/lib/python3.5/site-packages/ray/tune/ray_trial_executor.py", line 211, in fetch_result
    result = ray.get(trial_future[0])
  File "/home/lsi04/anaconda3/envs/flow/lib/python3.5/site-packages/ray/worker.py", line 2386, in get
    raise value
ray.worker.RayTaskError: ray_PPOAgent:train() (pid=23764, host=lsi04)
  File "/home/lsi04/anaconda3/envs/flow/lib/python3.5/site-packages/ray/rllib/agents/agent.py", line 279, in train
    result = Trainable.train(self)
  File "/home/lsi04/anaconda3/envs/flow/lib/python3.5/site-packages/ray/tune/trainable.py", line 146, in train
    result = self._train()
  File "/home/lsi04/anaconda3/envs/flow/lib/python3.5/site-packages/ray/rllib/agents/ppo/ppo.py", line 101, in _train
    fetches = self.optimizer.step()
  File "/home/lsi04/anaconda3/envs/flow/lib/python3.5/site-packages/ray/rllib/optimizers/multi_gpu_optimizer.py", line 125, in step
    self.num_envs_per_worker, self.train_batch_size)
  File "/home/lsi04/anaconda3/envs/flow/lib/python3.5/site-packages/ray/rllib/optimizers/rollout.py", line 28, in collect_samples
    next_sample = ray.get(fut_sample)
ray.worker.RayTaskError: ray_PolicyEvaluator:sample() (pid=23779, host=lsi04)
  File "/home/lsi04/anaconda3/envs/flow/lib/python3.5/site-packages/numpy/core/fromnumeric.py", line 52, in _wrapfunc
    return getattr(obj, method)(*args, **kwds)
AttributeError: 'list' object has no attribute 'reshape'

During handling of the above exception, another exception occurred:

ray_PolicyEvaluator:sample() (pid=23779, host=lsi04)
  File "/home/lsi04/anaconda3/envs/flow/lib/python3.5/site-packages/ray/rllib/evaluation/policy_evaluator.py", line 368, in sample
    batches = [self.input_reader.next()]
  File "/home/lsi04/anaconda3/envs/flow/lib/python3.5/site-packages/ray/rllib/offline/input_reader.py", line 25, in next
    batches = [self.sampler.get_data()]
  File "/home/lsi04/anaconda3/envs/flow/lib/python3.5/site-packages/ray/rllib/evaluation/sampler.py", line 64, in get_data
    item = next(self.rollout_provider)
  File "/home/lsi04/anaconda3/envs/flow/lib/python3.5/site-packages/ray/rllib/evaluation/sampler.py", line 267, in _env_runner
    preprocessors, obs_filters, unroll_length, pack, callbacks)
  File "/home/lsi04/anaconda3/envs/flow/lib/python3.5/site-packages/ray/rllib/evaluation/sampler.py", line 346, in _process_observations
    policy_id).transform(raw_obs)
  File "/home/lsi04/anaconda3/envs/flow/lib/python3.5/site-packages/ray/rllib/models/preprocessors.py", line 161, in transform
    for (o, p) in zip(observation, self.preprocessors)
  File "/home/lsi04/anaconda3/envs/flow/lib/python3.5/site-packages/ray/rllib/models/preprocessors.py", line 161, in <listcomp>
    for (o, p) in zip(observation, self.preprocessors)
  File "/home/lsi04/anaconda3/envs/flow/lib/python3.5/site-packages/numpy/core/fromnumeric.py", line 257, in reshape
    return _wrapfunc(a, 'reshape', newshape, order=order)
  File "/home/lsi04/anaconda3/envs/flow/lib/python3.5/site-packages/numpy/core/fromnumeric.py", line 62, in _wrapfunc
    return _wrapit(obj, method, *args, **kwds)
  File "/home/lsi04/anaconda3/envs/flow/lib/python3.5/site-packages/numpy/core/fromnumeric.py", line 42, in _wrapit
    result = getattr(asarray(obj), method)(*args, **kwds)
ValueError: cannot reshape array of size 1 into shape (0,)

Worker ip unknown, skipping log sync for /home/lsi04/ray_results/green_wave/PPO_JavierPradoGridEnv-v0_0_2019-06-05_11-46-51ktobyge9
Attempting to recover trial state from last checkpoint.

I have been trying with differentes sumo networks files but I always get the first error and the " shape (0,) " in the second error.
Thanks for your time and regards.

hi @DMarcelAM I feel like the issue might have something to do with your definition of the observation_space or action_space variables in your environment. In order to help me reproduce your bug, would you be able to share these methods as well? Thank you!

@property
    def action_space(self):
        """See class definition."""
        if self.discrete:
            return Discrete(2 ** self.num_traffic_lights)
        else:
            return Box(
                low=-1,
                high=1,
                shape=(self.num_traffic_lights,),
                dtype=np.float32)

    @property
    def observation_space(self):
        """See class definition."""
        speed = Box(
            low=0,
            high=1,
            shape=(self.scenario.vehicles.num_vehicles,),
            dtype=np.float32)
        dist_to_intersec = Box(
            low=0.,
            high=np.inf,
            shape=(self.scenario.vehicles.num_vehicles,),
            dtype=np.float32)
        edge_num = Box(
            low=0.,
            high=1,
            shape=(self.scenario.vehicles.num_vehicles,),
            dtype=np.float32)
        traffic_lights = Box(
            low=0.,
            high=1,
            shape=(3 * self.rows * self.cols,),
            dtype=np.float32)
        return Tuple((speed, dist_to_intersec, edge_num, traffic_lights))

Ah, I see. I believe that since you are using a template, and thereby not originating the network with a set number of vehicles, self.scenario.vehicles.num_vehicles is initialized to 0, resulting in the size bug. You can consider replacing these variables with a fixed (non-zero) number and that should do the trick.
Let me know if it doesn't work and I'll be happy to iterate through this with you.

I was trying with different numbers but I'm obtaining similar errors,
I set self.scenario.vehicles.num_vehicles = 1  I get ValueError: cannot reshape array of size 2 into shape (1,).
I set self.scenario.vehicles.num_vehicles = 2  I get ValueError: cannot reshape array of size 1 into shape (2,).
I set self.scenario.vehicles.num_vehicles = 3  I get ValueError: cannot reshape array of size 1 into shape (3,).
When I set self.scenario.vehicles.num_vehicles = X  I get ValueError: cannot reshape array of size 1 into shape (X,). except with X=1

I set num_vehicles=0 in green_wave example (tot_cars=0)  and there's no error. But when I add inflows to green_wave_example I get the same error: ValueError: cannot reshape array of size 1 into shape (0,)

I think the environment you are using may include a get_state method that doesn't take into account variability in the number of vehicles, hence the shape error when you turn on inflows. You can probably debug this by printing self.state.shape or len(self.state) and seeing if the number changes. If so, I wont recommend padding the observations (see flow/envs/merge.py as an example).

Hi @AboudyKreidieh and thanks for answer my questions,
When you say that my get_state method should not take into account variability in the number of vehicles, how can I use the speed of each vehicle (the number of vehicles is still variable) as a state ?

Hi @DMarcelAM, the tricky thing is that unless you use something special, a neural network takes in a fixed number of states as its input. If the number of states is variable, you either have to:
(1) Figure out how to select a fixed number of vehicle states from the total number of vehicle states
(2) Use a neural net architecture that can adapt to the number of vehicles in the system like a transformer
(3) Convert the problem into some other format where the number of vehicle states is fixed.
So, for example, you could imagine you only include the states of the 10 vehicles closest to the intersection or something like that.

Hi @DMarcelAM, the tricky thing is that unless you use something special, a neural network takes in a fixed number of states as its input. If the number of states is variable, you either have to:
(1) Figure out how to select a fixed number of vehicle states from the total number of vehicle states
(2) Use a neural net architecture that can adapt to the number of vehicles in the system like a transformer
(3) Convert the problem into some other format where the number of vehicle states is fixed.
So, for example, you could imagine you only include the states of the 10 vehicles closest to the intersection or something like that.

Hi, I got a similar error when testing flow_mappdg on singleagent_traffic_light_grid, the error is
~/git/flow/flow/envs/base.py", line 334 in step
 self.observed_ids.update(self.k.vehicle.get_ids())
AttributeError: 'list' object has no attribute 'update'

Trying to figure out the reason.


"already exists. Is a different review already running?"

I've hit this a couple of times already with no background nix-review working. How can I force nix-review to ignore things ?
(ins)[teto@jedha:~/nixpkgs3]$ nix-review rev HEAD
/home/teto/.cache/nix-review/rev-bc84ba0d802d3095e0e64cbf19fb83e8e2840993/nixpkgs already exists. Is a different review already running?
Traceback (most recent call last):
  File "/nix/store/jyxbl0k4vv85928mhl72c70ikglz0491-nix-review-1.0.5/bin/.nix-review-wrapped", line 12, in <module>
    sys.exit(main())
  File "/nix/store/jyxbl0k4vv85928mhl72c70ikglz0491-nix-review-1.0.5/lib/python3.7/site-packages/nix_review/__init__.py", line 10, in main
    cli.main(command, args)
  File "/nix/store/jyxbl0k4vv85928mhl72c70ikglz0491-nix-review-1.0.5/lib/python3.7/site-packages/nix_review/cli/__init__.py", line 119, in main
    args.func(args)
  File "/nix/store/jyxbl0k4vv85928mhl72c70ikglz0491-nix-review-1.0.5/lib/python3.7/site-packages/nix_review/cli/rev.py", line 18, in rev_command
    with Builddir(f"rev-{commit}") as builddir:
  File "/nix/store/jyxbl0k4vv85928mhl72c70ikglz0491-nix-review-1.0.5/lib/python3.7/site-packages/nix_review/builddir.py", line 53, in __init__
    os.makedirs(self.worktree_dir)
  File "/nix/store/b75brsgimlrw33l9ykhacz5fni3vq0jn-python3-3.7.2/lib/python3.7/os.py", line 221, in makedirs
    mkdir(name, mode)
FileExistsError: [Errno 17] File exists: '/home/teto/.cache/nix-review/rev-bc84ba0d802d3095e0e64cbf19fb83e8e2840993/nixpkgs'

Also the help doesn't show all options (like -p or --builder-args)

There is currently no workaround for this. Currently the directory name of the
review will be calculated from pr number or commit hash.
You probably have a different shell open with the same nix-review hash.
If that is not the case, delete the directory.

Should be fixed in the latest release.


feat(dialog): Add showModal and close functions for dialogs

There wasn't functions to call the showModal and close functions for dialogs, so here they are !

Instead of making the function return a result we could either

introduce a cast function specifically for HTMLDialogElements and accept only them.
Use Nested types like I have tried for events https://github.com/CrowdHailer/plinth/commit/34235f1f1c4f5082828ce34edac6c8b9d2bb98de#diff-2bac0d4ff6d1d92942f4fb7ece233f64bd5db1bd02b12840631382a81669dcb6R43

I'm away for a week, so if you want other opinions try asking about this PR in the forum. I'm not 100% sure which way I prefer, there's probably good arguments either way.


Modified get column meta method to reference saved metadata

Also, field name will be reset in meta_data_free() when the statement gets destroyed

Coverage decreased (-0.1%) to 73.213% when pulling 26c25c257b9c7645dbb9af261c2d8b513b270cfe on yitam:metadata into a3456cdbcbd7434a3d6638ad54b470d35d35bd09 on Microsoft:dev.

Codecov Report

Merging #978 into dev will increase coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev     #978      +/-   ##
==========================================
+ Coverage   81.24%   81.37%   +0.13%
==========================================
  Files          25       25
  Lines        7790     7787       -3
==========================================
+ Hits         6329     6337       +8
+ Misses       1461     1450      -11

Impacted Files
Coverage Δ

...c14/x86/php-7.1.28-src/ext/pdo_sqlsrv/pdo_stmt.cpp
84.09% <0%> (+0.11%)
:arrow_up:

...php-7.1.28-src/ext/pdo_sqlsrv/shared/core_stmt.cpp
79.14% <0%> (+0.47%)
:arrow_up:

...x86/php-7.1.28-src/ext/sqlsrv/shared/core_stmt.cpp
86.72% <0%> (+0.47%)
:arrow_up:

Continue to review full report at Codecov.

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


User should be able to get the time taken for a runtime job programatically.

What is the expected feature or enhancement?
There should be a way to recover the full elapsed time a job takes from the job and/or result instance.
Acceptance criteria
And user sees the total elapsed time, i.e the time the user would be billed for on the IBM Cloud

@renier @tmittal947 are you aware of any tickets open on API side to expose this information?

NVM got it from the team, the API to expose this info is being worked upon in ntc github tickets 1144 and 1252.

I am personally fine with whatever mechanism people decide upon. However one should make sure that the time per step values correspond to the billed times where appropriate

The metrics API is ready and this issue is now unblocked. https://us-east.quantum-computing.cloud.ibm.com/openapi#/Jobs/get_job_metrics_jid

Spoke to @tmittal947 and @renier. Let's just have a job.metadata() method that returns whatever is returned by /jobs/{id}/metrics API. And then in the future we will think about returning this information in the result metadata.


Add Labels via the front-end

It should be possible for users to add labels from queue-triage-web

PR #75 opened and waiting for review/comment

Resolved by PR #75


Fatal Exception: android.app.ForegroundServiceStartNotAllowedException in Android 12 devices

Hi, I have an issue with this crash.

It's happening only with Android 12 devices.
I'm using cast sdk version 21.0.0
Do you have any idea?

This issue was resolved in Cast SDK version 21.2.0. You can find the release notes here https://developers.google.com/cast/docs/release-notes#october-10,-2022
If you still run into this issue please reopen


best_model = model yields last model run in the epoch.

Description:

The following code yields the last epoch's model not the best_model's epoch due to the python's object creation mechanism.
It creates a reference to the object and assigns that reference to the variable. This means that both variables will now point to the same object.

Bug Link
-https://github.com/MedMNIST/experiments/blob/12e9f40ad214f6f076b1672cf29fab9d2e7216cc/MedMNIST3D/train_and_eval_pytorch.py#L157
Fix
from copy import deepcopy
best_model = deepcopy(model)

Hi @karthikkurella,
Thanks for your bug report! It has been fixed now.

Hi @ThreeSRR , thanks for fixing this. How do you think this would impact the results in the paper, since, as this code suggests, the numbers in both the papers used the model from the last epoch instead of the epoch with the best validation AUC?

We have carefully checked our experiments. We did not use the same script as the one in this repository, and the results reported in both papers reflect the models with the highest validation AUC. Thanks!


Makefile: Make the flash device configurable

What?
Allow the device path to be configured during make flash.
How?
Expose a new DEVICE variable, defaulting to /dev/ttyACM0 (which is the likely path of the device on linux).

I like it. This can help with the issue we raised in the other PR @akhilgupta1093 .

To change the port, one would edit the DEVICE variable in the makefile?


Include credit limit option on account creation page

Our meals billers and membership committee members (the ones doing orientation at least) are asking that we include the question of credit limit on the account creation page. =
"Can you set it up so that the billers can add a credit limit for new accounts?  I think that would be a help.  Thanks, ?Gail [Graham]"
This is because we have a policy for different credit limits for different types of members:
$50 for single renters
$100 for renter families
$300 for owners
Some owners like to negotiate different terms with their renters, for example, they may want their renter to entirely pre-pay their bills, so their credit limit would be $0.
While we're capable of doing this today, it is a 2-step process to also go to the accounts page and edited, and the 2nd step can be easily forgotten.
Originally created by Willie Northway at 2016-09-19 00:26, migrated from redmine-#5128

Priority changed from 4 to 5

"I figured out how to do [this] and am satisfied with the two step process.
—Gail"
By Willie Northway at 2016-09-20 08:03


Capturing membership of weighted means (via Squid3 checkbox) in Report CSVs

We have talked previously about ways to rigorously link Report CSV-rows to calculated weighted means (as appended to the WM stats file). The Report CSV (whether Sample or Reference Material) ought to directly incorporate information about which rows were included in the mean and which were not (as indicated by the checkboxes for individual analyses).
For Reference Materials, the columns of primary interest are the Calibration Constants, and the options are related to the index-isotope for common Pb. So:
For Perm1: (4corr 206/238 calib constant) OR (7corr 206/238 calib constant) OR (8corr 206/238 calib constant)
For Perm3: (4corr 208/232 calib constant) OR (7corr 208/232 calib constant)
For Perms2+4: (4corr 206/238 calib constant AND 4corr 208/232 calib constant) OR (7corr 206/238 calib constant AND 7corr 208/232 calib constant)
Probably the most rigorous way to handle this is to have an extra auto-column (e.g. PB204C_PB206_U238_CALIB_CONST_MEAN) in the CSV report for each calib constant within each of the possible options, which is dedicated to reporting the status of the checkbox for that row. Within the field, there would be three possible statuses: "included" for a spot calib-constant value that was included in the mean, "excluded" for a spot that wasn't, and "NA" , which would be deployed at column-scale, covering all unused column-options.
Using Perm1 as an example, if the user chose 204Pb as their preferred index-isotope for common Pb, then the auto-column for 4corr 206/238 calib const would comprise solely "included" or "excluded" according to the checkboxes, whereas ALL rows in the 7corr 206/238 calib const and 8corr 206/238 calib const auto-columns would be labelled "NA". But those latter columns need to be there, and theoretically populatable, because it is so easy for users to simply switch the radio-button controlling the applicable common-Pb index isotope.
Remember also that in Perms2+4, calib consts function as "locked pairs", so if the user chooses 204Pb as the index isotope, then both the 4corr 206/238 calib const and the 4corr 208/232 calib const would be populated with either "included" or "excluded" according to their (independent sets of) checkboxes, whereas both 7corr 206/238 calib const and 7corr 208/232 calib const would be populated with NA throughout.
For Samples, the situation would be simpler. We would need 7 columns, covering the 7 different age-types (e.g. PB204C_PB206_U238_MEAN), but all 7 are always available for population, irrespective of the Perm used for the Reference Material. So there would always be 6 full sets of "NA" for all Samples at all times, and one set of "included" vs "excluded".
A final measure to consider would be to append a text-string of the summary stats, as currently written to the RefMat_Summary and or WM stats text-files, as the rightmost column, for each of the rows with the label "included". In the case of a user having calculated and appended multiple iterations of a mean to their stats file(s), this might help them track exactly which weighted-mean calculation is reflected by the CSV report.

Spoke with @sbodorkos and clarified that these new columns will be included in the built-in reports and available on an all-or-nothing basis for custom reports.


org.gradle.internal.classpath.Instrumented.DecoratingProperties.clone() does not return an independent instance

A call to org.gradle.internal.classpath.Instrumented.DecoratingProperties.clone() (used for example to get an isolated copy of the JVM system properties) returns a DecoratingProperties instance that uses the same delegate. Calls to setProperty(...) will change the original (and all other "cloned") instances.
Expected Behavior
Cloning should also clone the delegate Properties instance.
Current Behavior
Cloned Properties instances will be "coupled" through their delegate. Changes in one cloned instance will be visible in all other cloned instances.
Context
https://github.com/OpenAPITools/openapi-generator uses clone() to create an independent copy of the JVM system properties. This works well outside of Gradle, but fails when the generator is used in Gradle.
Steps to Reproduce
The following code should print World, but it prints Bug when Properties instances are instrumented by Gradle:
System.setProperty("Hello", "World");
Properties copy = (Properties) System.getProperties().clone();
copy.setProperty("Hello", "Bug");
System.out.println(System.getProperty("Hello"));

Workaround: Create a new Properties instance and use putAll(...) to add the original property values.
Your Environment
Gradle 6.7.1 (also happens with 7.0.2) on Windows 10

@gradle/configuration-cache could you please take a look at this?


fix dry run template payload error

Signed-off-by: sabinasiddiqi<EMAIL_ADDRESS>Hey, I just made a Pull Request!
Our team is seeing a bug in the dry run template editor. Every time we tried to run a template we would get the PayloadTooLargeError: request entity too large even when it was a small file. It appears the express version is >=4.0 and that requires the express router to use bodyParser. Making this switch fixes the issue.
Related issue here
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

I see you were unable to replicate this anymore locally. The fix surprises me a bit, since the code for express literally contains the line exports.json = bodyParser.json straight up. Is this fix still needed then?
If you want to go ahead still, the way of getting rid of the vale warning is to put the bodyparser word in backticks so it doesn't get spell checked, e.g. `bodyparser`

I see you were unable to replicate this anymore locally. The fix surprises me a bit, since the code for express literally contains the line exports.json = bodyParser.json straight up. Is this fix still needed then?
If you want to go ahead still, the way of getting rid of the vale warning is to put the bodyparser word in backticks so it doesn't get spell checked, e.g. `bodyparser`

I am going to close this PR because it looks like there was something in my local dev making it look like folders that met the size requirement were also erroring out because now I cannot replicate it either in our instance of backstage or the forked repo.
We do still have the issue that the templates we want to test are over the default size limit of 100kb - about 130 kb. Do you have any ideas or suggestions for that? @freben

@freben @sabinasiddiqi I assume there is no further continuation from here. If that is the case, can we close the related issue #12561 ? Otherwise, I can look further into this.


Remove facebook

Our facebook integration is deprecated. And after discussion comms decided that we should remove the facebook integration for user profiles. That means that it is no longer possible to login or register using facebook.

Should we tell any users who are currently using Facebook for login?

We do not track usage of login strategy, but we have 8000 users with an associated facebook account.
If users report that they miss the feature, then we could adapt to the latest facebook api changes


[TA] update testing region

We're currently testing in eastus since this region supports the gated preview features. We should revert back to the default once the features are deployed in the rest of the regions.
https://github.com/Azure/azure-sdk-for-python/pull/27174/files#diff-eb14a08a0193ff909aa7473225ab154a8560d8fe8b1ae9a7243060deea8640a9L14

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Container Registry:0.07581946,Docs:0.06727839,Cognitive Services:0.04664562'
Label prediction was below confidence level 0.6 for Model:CategoryLabels: 'Client:0.5109884,Mgmt:0.44128242,Service:0.04496284'


Planet Ship scouting

you see what kills you. If ships are on your  planet you should see them.

version bug

This is no longer an issue?


Improved starting visualization for GraphExplorer

The value of visualizing RDF as an LPG is in the reduction of information, and making it simpler to interpret data associated with meaningful nodes. The GraphExplorer widget is designed to assist in the discovery of information proximate to a specific node; expanding data allows one to 'discover' the nodes that are connected to it, and locking in nodes/edges allows users to capture interesting shapes.
In the current version of the GraphExplorer widget, the original nodes are selected from multi-select dropdowns filtered by type. The initial nodes are populated by themselves, and can be expanded as desired.
This ticket changes the initial behavior. Nodes populated in the initial GraphExplorer should have all outgoing connections pre-expanded with their corresponding data attributes. In lieu of the original buttons, this ticket also scopes in a more intuitive exploration UX.
New Behavior

Single-click: display JSON data
Double-click: expand node and lock-in (lock-in the selected node only)

As part of this ticket, we need to perform an initial investigation into node location caching, or else the visualization will be very jarring as it updates. Will probably be advantageous to cache the node data as well (good chance people will expand/collapse the same node multiple times).

This is delayed due to issues with ipycytoscape. The limitations of the ipycytoscape graph widget (no clear method, and issues updating the front-end) are preventing a clean implementation in ipyradiant. Temporary workarounds have led to further issues maintaining the ipycytoscape graphs.


Buffer_overflow_action:block flood log file too fast

Describe the bug
I'm using a buffer file with block action in case of overflow, the filesystem was filled with log multiple times per second causing a 14Go log file in less than a day and no disk space remained
Expected behavior
Less warning message about buffer full
Your Environment

fluentd 1.11.1
Debian 10
4.19.0-8-cloud-amd64

Your Configuration
<buffer>
       @type file
       flush_interval 1s
       overflow_action block
       total_limit_size 1G
       path /var/run/td-agent/td-agent.buffer
</buffer>

Your Error Log
[warn]: #0 [output_system_forward] failed to write data into buffer by buffer overflow action=:block

You can avoid this problem by specifying log related parameters.

@log_level error to disable warn level message

https://docs.fluentd.org/deployment/logging#per-plugin-log

ignore_xxx_log_interval parameters

https://docs.fluentd.org/deployment/logging#suppress-log-stacktrace-messages

Wouldn't it be better to set it by default? It causes fluentD to crush itself in terms of minutes


Interface for frequency and time dependent models

Given that #29 has landed, we can add better multifrequency support for models. We will not take a wrapper approach since unwrapping every multifrequency model would be very difficult. Instead, the current plan is to tag the parameters passed to the model themselves.
A spectral index model could look something like
struct SpectralIndex{T, F}
      specindx::T
      ref_freq::F
end

function build_params(model::TaylorSpectral, refparam, Ti, Fr)
       param = refparam*(Fr/model.ref_freq)^model.specindex
       return param
end

struct MultiDomainParam{Fr, Ti, P}
      freqmodel::Fr
      timemodel::Ti
      param::P
end

modify(Gaussian(), Stretch(MultiDomainParam(2.0, SpectralIndex(1.0, 230e9), Constant()))

This proposal has been implemented in #73


Exposing basepath as a member of IApiDescription; no longer hardcoded.

Really appreciate the work you've done so far; for this to be generic we can't have this basepath hardcoded to 'v2'.

Absolutely fine and sensible, apologies for being so absent from the project!


Option - Governance

Governance allegedly works via cmd.
Not sure if Governance is served up via API.
See if we can get a list of proposals that are open and vote somehow from the application.

This one is dead as governance has moved to ETH for now, will re-open if there's a need to add compatibility for ETH voting later!


cannot install chromium inside docker

This worked in the past but it seams it stopped working. When running make bootstrap this error occures:
Hit:1 http://security.debian.org/debian-security stretch/updates InRelease
Ign:2 http://deb.debian.org/debian stretch InRelease
Hit:3 http://deb.debian.org/debian stretch-updates InRelease
Get:4 http://dl.google.com/linux/chrome/deb stable InRelease [1811 B]
Hit:5 http://deb.debian.org/debian stretch Release
Get:6 http://dl.google.com/linux/chrome/deb stable/main amd64 Packages [1082 B]
Fetched 2893 B in 0s (7072 B/s)
Reading package lists...
Reading package lists...
Building dependency tree...
Reading state information...
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:
The following packages have unmet dependencies:
google-chrome-unstable : Depends: libdrm2 (>= 2.4.75) but 2.4.74-1 is to be installed
Depends: libgbm1 (>= 17.1.0~rc2) but 13.0.6-1+b2 is to be installed
Depends: libnss3 (>= 2:3.31) but it is not going to be installed
Depends: libu2f-udev but it is not installable
E: Unable to correct problems, you have held broken packages.
ERROR: Service 'app' failed to build: The command '/bin/sh -c apt-get update     && apt-get install -y wget gnupg     && wget -q -O - https://dl-ssl.google.com/linux/linux_signing_key.pub | apt-key add -     && sh -c 'echo "deb [arch=amd64] http://dl.google.com/linux/chrome/deb/ stable main" >> /etc/apt/sources.list.d/google.list'     && apt-get update     && apt-get install -y google-chrome-unstable fonts-ipafont-gothic fonts-wqy-zenhei fonts-thai-tlwg fonts-kacst fonts-freefont-ttf       --no-install-recommends     && rm -rf /var/lib/apt/lists/*' returned a non-zero code: 100
Makefile:77: recipe for target 'build-image' failed
make: *** [build-image] Error 1

i was able to get around this by updating the docker image.
cat Dockerfile
FROM node:18-slim as core

here i changed from node:10-slim as core to the above


New Crowdin updates

Hello @ps-jarvis!
This is your first pull request on the PrestaShop project. Thank you, and welcome to this Open Source community!

Hello @ps-jarvis!
This is your first pull request on the PrestaShop project. Thank you, and welcome to this Open Source community!

I must be dreaming, finally. :D
@Quetzacoalt91 do you think you could set this up for all native modules?

We created a new Crowdin project for the native modules that could benefit the integrated translations. So yes, this is something we can do in the future once the solution is fully implemented here.

@Quetzacoalt91 It would be good if it was in the current project, in Modules folder, so all people don't have to be reassigned to the new repository.

native modules that could benefit the integrated translations

All native modules are using the new translation system, so the only thing that is blocking is the communication betweeen crowding and github. 👍

As far as know the current integration with Github does not allow other projects to be added in a specific subfolder, the result would have displayed autoupgrade among the existing folders.
At the moment it seems safer to create a new project (https://crowdin.com/project/prestashop-official-modules) where Crowdin add each reapository at the root.


Enable audit log format options

Is your feature request related to a problem? Please describe.
Enable audit logging options for FHIR audit event. The formats should comply with one or more standards.
Describe the solution you'd like
Ability to configure the FHIR server to use a standardized audit logging format.
Describe alternatives you've considered
n/a
Additional context
n/a

Closing and will use the related issue


internal: treat default resource group normally

In order to address the issue https://github.com/pingcap/tidb/issues/40875, we should take the default resource group as a normal one.

Does we should support init default in resource manager first?


Rename packages

Renaming connect-query to connect-react-query and protoc-gen-connect-query to protoc-gen-connect-react-query. This makes it clear that these packages still very much depend on running in a react environment.
These changes will be published as v1

Should the extension for generated files remain _connectquery.ts?

That's a good question. I think it stands to reason that it should be _connectreactquery.ts, even it makes me hurt a little inside.

@timostamm how do you feel about using _cq_react.ts as a file ending instead of _connectreactquery.ts postfixes? I kinda like keeping the postfix short since we're already prefixed with the service and the method.

Closing this until we can discuss what renaming would cost us.