Command tab 'problems' badge counter should count by filter too

The command tab problems should just show problems within a filter. So if I select a filter and inside that path is no error, it should show no counter badge:

Badge is to show the total number of problems. Not showing the badge might mislead/confuse users when the problems are filtered out.

In confuse me in its current state, anyway it could be a option on the right sie of the filter input (anicon; count all/only filtered).


keybase-gui: fix kbfs mount path check

Motivation for this change
Fixes #31261
keybase-gui did not work out of the box with the default configuration of the keybase nixos service.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

While this does look like it'll work in most scenarios it doesn't solve the overall problem in #31261 as it's now just looking at two options of hardcoded mountpoint.  A better solution would be to have the wrapper (I may have used the wrong word, if so, sorry) check for where keybase is mounted and then use that.

Success on aarch64-linux (full log)
Attempted: keybase-gui
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/qiz9p527cdg9x3pdv0wsdw7jnyqp30n6-keybase-gui-1.0.44-20180223200436.9a9ccec79
shrinking /nix/store/qiz9p527cdg9x3pdv0wsdw7jnyqp30n6-keybase-gui-1.0.44-20180223200436.9a9ccec79/share/keybase/libnode.so
shrinking /nix/store/qiz9p527cdg9x3pdv0wsdw7jnyqp30n6-keybase-gui-1.0.44-20180223200436.9a9ccec79/share/keybase/libffmpeg.so
shrinking /nix/store/qiz9p527cdg9x3pdv0wsdw7jnyqp30n6-keybase-gui-1.0.44-20180223200436.9a9ccec79/share/keybase/Keybase
strip is /nix/store/gp7fylxwn18b7pl2c18ks89hsiaxyfvf-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/qiz9p527cdg9x3pdv0wsdw7jnyqp30n6-keybase-gui-1.0.44-20180223200436.9a9ccec79/bin
patching script interpreter paths in /nix/store/qiz9p527cdg9x3pdv0wsdw7jnyqp30n6-keybase-gui-1.0.44-20180223200436.9a9ccec79
/nix/store/qiz9p527cdg9x3pdv0wsdw7jnyqp30n6-keybase-gui-1.0.44-20180223200436.9a9ccec79/share/keybase/post_install.sh: interpreter directive changed from " /usr/bin/env bash" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/bash"
checking for references to /build in /nix/store/qiz9p527cdg9x3pdv0wsdw7jnyqp30n6-keybase-gui-1.0.44-20180223200436.9a9ccec79...
/nix/store/qiz9p527cdg9x3pdv0wsdw7jnyqp30n6-keybase-gui-1.0.44-20180223200436.9a9ccec79

That's half an hour I could have spend not debugging bash quoting:)

Success on aarch64-linux (full log)
Attempted: keybase-gui
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/mx8a42iva2fgj7abyv3gscvmwy1h264k-keybase-gui-1.0.44-20180223200436.9a9ccec79
shrinking /nix/store/mx8a42iva2fgj7abyv3gscvmwy1h264k-keybase-gui-1.0.44-20180223200436.9a9ccec79/share/keybase/libnode.so
shrinking /nix/store/mx8a42iva2fgj7abyv3gscvmwy1h264k-keybase-gui-1.0.44-20180223200436.9a9ccec79/share/keybase/libffmpeg.so
shrinking /nix/store/mx8a42iva2fgj7abyv3gscvmwy1h264k-keybase-gui-1.0.44-20180223200436.9a9ccec79/share/keybase/Keybase
strip is /nix/store/gp7fylxwn18b7pl2c18ks89hsiaxyfvf-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/mx8a42iva2fgj7abyv3gscvmwy1h264k-keybase-gui-1.0.44-20180223200436.9a9ccec79/bin
patching script interpreter paths in /nix/store/mx8a42iva2fgj7abyv3gscvmwy1h264k-keybase-gui-1.0.44-20180223200436.9a9ccec79
/nix/store/mx8a42iva2fgj7abyv3gscvmwy1h264k-keybase-gui-1.0.44-20180223200436.9a9ccec79/share/keybase/post_install.sh: interpreter directive changed from " /usr/bin/env bash" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/bash"
checking for references to /build in /nix/store/mx8a42iva2fgj7abyv3gscvmwy1h264k-keybase-gui-1.0.44-20180223200436.9a9ccec79...
/nix/store/mx8a42iva2fgj7abyv3gscvmwy1h264k-keybase-gui-1.0.44-20180223200436.9a9ccec79

Success on x86_64-linux (full log)
Attempted: keybase-gui
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/9vz09f6dxpshbx2ngi86ck2i89czm4kl-keybase-gui-1.0.44-20180223200436.9a9ccec79
shrinking /nix/store/9vz09f6dxpshbx2ngi86ck2i89czm4kl-keybase-gui-1.0.44-20180223200436.9a9ccec79/share/keybase/Keybase
shrinking /nix/store/9vz09f6dxpshbx2ngi86ck2i89czm4kl-keybase-gui-1.0.44-20180223200436.9a9ccec79/share/keybase/libffmpeg.so
shrinking /nix/store/9vz09f6dxpshbx2ngi86ck2i89czm4kl-keybase-gui-1.0.44-20180223200436.9a9ccec79/share/keybase/libnode.so
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/9vz09f6dxpshbx2ngi86ck2i89czm4kl-keybase-gui-1.0.44-20180223200436.9a9ccec79/bin
patching script interpreter paths in /nix/store/9vz09f6dxpshbx2ngi86ck2i89czm4kl-keybase-gui-1.0.44-20180223200436.9a9ccec79
/nix/store/9vz09f6dxpshbx2ngi86ck2i89czm4kl-keybase-gui-1.0.44-20180223200436.9a9ccec79/share/keybase/post_install.sh: interpreter directive changed from " /usr/bin/env bash" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/bash"
checking for references to /build in /nix/store/9vz09f6dxpshbx2ngi86ck2i89czm4kl-keybase-gui-1.0.44-20180223200436.9a9ccec79...
/nix/store/9vz09f6dxpshbx2ngi86ck2i89czm4kl-keybase-gui-1.0.44-20180223200436.9a9ccec79

merge?

@puffnfresh can you take a look and merge please? Thanks!


Can't receive push notification until the app is relaunched

I'm developing  iOS 10 app with xcode 8 and swift 2.3. Push notification don`t work, but if app went to background and back to foreground - everything starts to work and works even after restart. I send notification from server like this:
{
 “to”: “/topics/general”,
  “collapse_key”: “Updates Available”,
  “priority”: “high”,
  “content_available”: true
}

If I send a notification through the firebase messaging console - everything is always working well.
Topic subscribed as in the example. I also enabled push notification in capabilities & have correct provision profile. Logs from real device (iphone se, ios 10.1):
2016-12-18 16:50:39.473212 TestApp DEV[757:157970] WARNING: Firebase Analytics App Delegate Proxy is disabled. To log deep link campaigns manually, call the methods in FIRAnalytics+AppDelegate.h. 2016-12-18 16:50:39.475522 TestApp DEV[757:157970] Firebase automatic screen reporting is enabled. Call +[FIRAnalytics setScreenName:setScreenClass:] to set the screen name or override the default screen class name. To disable automatic screen reporting, set the flag FirebaseAutomaticScreenReportingEnabled to NO in the Info.plist 2016-12-18 4:50:39.656: <FIRMessaging/INFO> FIRMessaging library version 1.2.0 2016-12-18 16:50:39.693987 TestApp DEV[757:158128] [Firebase/Core][I-COR000001] Configuring the default app. 2016-12-18 16:50:39.694 TestApp DEV[757] <Debug> [Firebase/Core][I-COR000001] Configuring the default app. 2016-12-18 16:50:39.706858 TestApp DEV[757:158129] <FIRAnalytics/INFO> Firebase Analytics v.3600000 started 2016-12-18 4:50:39.710: <FIRInstanceID/WARNING> Failed to fetch APNS token Error Domain=com.firebase.iid Code=1001 "(null)" 2016-12-18 16:50:39.710 TestApp DEV[757:] <FIRAnalytics/INFO> Firebase Analytics v.3600000 started 2016-12-18 16:50:39.715835 TestApp DEV[757:158129] <FIRAnalytics/INFO> To enable debug logging set the following application argument: -FIRAnalyticsDebugEnabled (see http://goo.gl/Y0Yjwu) 2016-12-18 16:50:39.716 TestApp DEV[757:] <FIRAnalytics/INFO> To enable debug logging set the following application argument: -FIRAnalyticsDebugEnabled (see http://goo.gl/Y0Yjwu) 2016-12-18 16:50:39.745232 TestApp DEV[757:158129] [Firebase/Core][I-COR000018] Already sending logs. 2016-12-18 16:50:39.745 TestApp DEV[757] <Debug> [Firebase/Core][I-COR000018] Already sending logs. 2016-12-18 16:50:39.908521 TestApp DEV[757:158129] <FIRAnalytics/WARNING> The AdSupport Framework is not currently linked. Some features will not function properly. Learn more at http://goo.gl/9vSsPb 2016-12-18 16:50:39.908 TestApp DEV[757:] <FIRAnalytics/WARNING> The AdSupport Framework is not currently linked. Some features will not function properly. Learn more at http://goo.gl/9vSsPb 2016-12-18 16:50:39.971807 TestApp DEV[757:158148] [Firebase/Core][I-COR000019] Clearcut post completed. 2016-12-18 16:50:39.972 TestApp DEV[757] <Debug> [Firebase/Core][I-COR000019] Clearcut post completed. User not found 2016-12-18 16:50:40.193176 TestApp DEV[757:157970] [MC] System group container for systemgroup.com.apple.configurationprofiles path is /private/var/containers/Shared/SystemGroup/systemgroup.com.apple.configurationprofiles 2016-12-18 16:50:40.195801 TestApp DEV[757:157970] [MC] Reading from public effective user settings. 2016-12-18 16:50:40.327400 TestApp DEV[757:158128] <FIRAnalytics/INFO> Firebase Analytics enabled 2016-12-18 16:50:40.327 TestApp DEV[757:] <FIRAnalytics/INFO> Firebase Analytics enabled 2016-12-18 16:50:40.461033 TestApp DEV[757:158129] [Firebase/Core][I-COR000019] Clearcut post completed. 2016-12-18 16:50:40.461 TestApp DEV[757] <Debug> [Firebase/Core][I-COR000019] Clearcut post completed. InstanceID token: ewnr0upTmMI:APA91bEeNr6CVZQ8Blr8QliZUtR0psiWo4p59lP_KAPJQ2P6s3L56o8RR1xEmLdFHF4bByCThUMvZekxNri2LLGoBoLVssGh-ieDK68poWlkonRNB8TpO_r3hodLF3tZfqA_EUWR2UsY DEVICE TOKEN = <0f2b3791 4687ef1b 9dcb147a 1610fddb 2992f0d3 37462a59 5f222c5d 58743128> 2016-12-18 4:50:47.343: <FIRInstanceID/WARNING> APNS Environment in profile: development Connected to FCM. InstanceID token: ewnr0upTmMI:APA91bEeNr6CVZQ8Blr8QliZUtR0psiWo4p59lP_KAPJQ2P6s3L56o8RR1xEmLdFHF4bByCThUMvZekxNri2LLGoBoLVssGh-ieDK68poWlkonRNB8TpO_r3hodLF3tZfqA_EUWR2UsY try to subscribe Topic Topic subscribed

Hi @alexslo are you subscribing to a single topic /topics/general?

Hello, yes I subscribing to a single topic.

Hello, yes I subscribing to a single topic.

Hi @alexslo you mentioned that everything is working well when you send from the console.

are you sending to the topic from the console?
could you add the code you are using to subscribe to the topic? In particular in what callback are you subscribing?

Yes, when I send to the topic from the console everything is working well
code code I using to subscribe to the topic:
`    func tokenRefreshNotification(notification: NSNotification) {
if let refreshedToken = FIRInstanceID.instanceID().token() {
print("InstanceID token: (refreshedToken)")
// Connect to FCM since connection may have failed when attempted before having a token.
if (UIApplication.sharedApplication().isRegisteredForRemoteNotifications()) {
print("try to subscribe Topic")
FIRMessaging.messaging().subscribeToTopic("/topics/general")
print("Topic subscribed")
}
}
}`

and
 NSNotificationCenter.defaultCenter().addObserver(self, selector: #selector(self.tokenRefreshNotification),                                                          name: kFIRInstanceIDTokenRefreshNotification, object: nil) in
func application(application: UIApplication, didFinishLaunchingWithOptions launchOptions: [NSObject: AnyObject]?)

HI @alexslo I'm not able to reproduce your issue. Couple of things to try:

Could you try the sample in this repo and see if you are experiencing the same issue.
Could you try calling connectToFcm() in the tokenRefreshNotification callback


parallel-full: fix missing files

Previously, this was not usable as a drop-in replacement for parallel since it only included the one exe. This changes to additionally symlink all files from $out as well as symlink all other outputs. So no files are missing compared to parallel, while being efficiently symlinked.
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] 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.

Something doesn't look right:
[nix-shell:~/.cache/nixpkgs-review/pr-304018]$ ls -alh ./results/parallel-full/
total 387M
dr-xr-xr-x     4 root root      5 Dec 31  1969 .
drwxrwxr-t 52161 root nixbld 662K May  4 02:43 ..
dr-xr-xr-x     2 root root     22 Dec 31  1969 bin
lrwxrwxrwx     1 root root     61 Dec 31  1969 hsx8w57vggjjvhfmhyr3aslz3j6vvlp0-parallel-20240322 -> /nix/store/hsx8w57vggjjvhfmhyr3aslz3j6vvlp0-parallel-20240322
dr-xr-xr-x     4 root root      4 Dec 31  1969 share

As if something was symlinked with the store path hash.

One week of radio silence, I'll assume review. I both manually inspected and ran a diff -r between the wrapped and unwrapped derivation, only the wrapper and the sym symlink are different. LGTM, thanks!
Result of nixpkgs-review pr 304018 run on x86_64-linux 1

3 packages built:

parallel-full
parallel-full.doc
parallel-full.man


version bump freesmug-chromium

(version 43.0.2357.124 to version 43.0.2357.130)

Thank you for the contribution. It was merged directly as 7bf5a32e25c52225e9a91e5fc59ac14d86fcaff3 to keep commit history cleaner. Your contribution is still credited to you.


High load

I have been testing different rtsp clients. OpenRTSP obviously being one. Most of the ones I have tested do leave a lot to be desired, but this library is ticking a lot of boxes for me, as I need to run it on an embedded system.
As such, i have been doing some tests, and I am somewhat surprised by the result. It seems that this library is using more resources than OpenRTSP. I was expecting the opposite to be honest. In the default setup, openRTSP uses approx 0.1s user, 0.3s system on my laptop for 30s recording, while this library uses 0.3s user and 0.3s system. System seems to be a fraction lower with this library, but user time is certainly higher. By increasing the USLEEP_UNIT I can get it down to < 0.2s, but that comes with it's own set of issues too, see below.
I am also seeing some video corruption from my Reolink C1-pro camera, the lower part of the frame seems to break up. That situation seems to improve if I reduce USLEEP_UNIT and increase the buffer size, but my laptop should have more than enough umph to handle this. I have yet to figure out why.  I am sure that this library can work, but I am unable to do so it seems. I would be willing to help out with this and to get it working, but I don't know where to start really.

In order to make the library easier for using, there is much regex using which could load much, especially for the function "GetMediaData".
Thanks for your feedback, I will try to improve the problem.

To improve performance, there are 2 kind of way:

WITHOUT WRITING HARD DISK:
Just now, I committed some code to improve performance, which lowers the use of regex and provide the API to disable the function of obtaining SPS,PPS,VPS periodically, please refer to complete_example.cpp.
If you don't change USLEEP_UNIT(default 10,000), in my test case, it will take about 7-8%(user time+sys time) more than openRtsp.
If you change USLEEP_UNIT to 13000, it will do as good as openRtsp.
If you change USLEEP_UNIT to 15000, it will do better than openRtsp.

WITH WRITING HARD DISK
In my old examples, it invokes "write" for every packet of data.
You could lower the frequency of "write" to improve the performance.
You could refer to the new "complete_example.cpp" which does well in my test case

Additional note:
I assumed that there are 25 frames of video data and "1 frame 1 packet" in 1 second.
So the interval between 2 frames is 1/25 second(40,000 us).
According to "sampling theorem", the sampling period is 20,000us. And because the 'usleep' is not precise enough, in order to get stable data, myRtspClient set it 10,000us. That is USLEEP_UNIT.
Actually, there is always "1 frame n>1 packet". So it's safe to change USLEEP_UNIT within 20,000, even bigger for different RTSP server.


I intend to include an python code programming environment in C# web application, could this tool help?

I would like to provide an jupyter notebook programming environment in an web applications, which is built in MVC architecture.
I'd just found this dotnet notebook, and I found it could program C# with notebook.
Could this dotnet notebook tool program python too?
Thank you

We don't currently support Python but it would certainly be possible to create a Python proxy kernel.


Adding descriptions in different languages

We've had a few requests for that on our Google Play reviews page - is it feasible with the help of the translations community?

Adding more fields to the title/description screen would only make more people skip it, I think.
If we implement this, how about in the gallery details view?
Alternatively,  in the gallery details view we could just put a link to the image on Commons, so that people can go there and access all of the available features there.

Currently we are able to edit description in different languages too. But on description field in Media Details, only the one in English is displayed.
Maybe a display in all languages button, which will be shown if there is a additional language exist, would solve this issue.

I am interested to work on this Issue.

Maybe a display in all languages button, which will be shown if there is a additional language exist, would solve this issue.

@neslihanturan Should I add a button displaying list of languages to select from or should I display media description according to the locale (current language of the device)?

IMO, it would be cleaner to show the description in the current locale and fall back to English if the description in that language isn't available.

IMO, it would be cleaner to show the description in the current locale and fall back to English if the description in that language isn't available.

OK then I'll do it this way :+1:

Please be aware that the information template might not always have the same format.
I am not sure whether the description format is always the same or not.
Here are some examples:
https://commons.wikimedia.org/wiki/Commons:Structured_data/Get_involved/Feedback_requests/Interesting_Commons_files#Variations_on_the_{{Information}}_template
In cases where you get a parsing error (example: malformed template), I suggest just showing the whole thing as now.

Can i contribute to this ?

Hey @its-kos. I am not working on this issue currently. You can take this up :-)

I can do Greek

@its-kos You don't have to translate the description yourself. You can use Wikimedia APIs to fetch description in user's locale.
Check them out here

This link doesnt work

I'm sorry. Please try this out https://commons.wikimedia.org/w/api.php

Ah well i might have misinterpreted what the issue was, i thought it was a translation

Are you clear with the issue now? Do you need more help with the issue?

Hi team,
This is my first time contribution.
I am interested to take up this issue, can I take it? or is someone working on it?
Also can someone help me understand the issue better.

Ah sorry this has been implemented already... also as a first contribution please choose a bug to fix, thanks :-) See https://github.com/commons-app/commons-app-documentation/blob/master/android/Volunteers-welcome!.md

also as a first contribution please choose a bug to fix, thanks :-)

While I agree that bug fixes serve better as first time contributions, I do wonder about the following statement:

Ah sorry this has been implemented already...

While we do clearly have support for adding descriptions in other languages (which is done) much of the discussion in this issue seems to be about showing other language descriptions in the media details view (which is not done yet, to my knowledge). So, this issue is in a kind of awkward state given it's misleading title and description. I believe we should either edit the issue's title and description (or) close this one and open a new one for the showing descriptions in other languages.

@sivaraam Indeed I had only relied on the word "adding" and did not re-read the whole discussion before closing, sorry about that. Master already falls back to showing other languages, and allows to enter captions and descriptions in any number of languages. You are right that this issue is kind of awkward, so better I created a new issue about showing (captions and) descriptions in other languages: https://github.com/commons-app/apps-android-commons/issues/3949


Dialog Open Bug

When a particular dialog is clicked open for the second time the screen grays out but the dialog itself doesn't show up. It doesn't seem to matter which dialog - link, table, etc., they all do this.

I am including the standard CKEditor package, however they all seem to do this:
<script src="https://cdn.ckeditor.com/4.5.8/standard/ckeditor.js"></script>
P.S. - Fantastic job on this so far.

Thanks !
Well, I tested with the sample repo (https://github.com/chymz/angular2-seed-ckeditor) on latest Chrome & FF but I can't reproduce your problem.
Can you share some code ? I think it's more a CKEditor bug...

Interesting, I feel like I might be missing something then. I installed via NPM as instructed and have the following relevant code in index.html:
    <script src="https://cdn.ckeditor.com/4.5.8/standard/ckeditor.js"></script>

    <script>
        System.config({
            packages: {
                "app": {
                    "format": 'register',
                    "defaultExtension": 'js'
                },
                "ng2-ckeditor": {
                    "defaultExtension": "js"
                }
            },
            map: {
                "ng2-ckeditor": "node_modules/ng2-ckeditor/lib/CKEditor.js"
            }
        });
        System.import('app/main').then(null, console.error.bind(console));
    </script>

Does this look right? To be honest I am new to SystemJS and wasn't quite sure about that piece. Thanks!

Closed, after further investigation this is a bug with the dialog plugin. For some reason in my build the container that dialogs get put into is not being set back to a block display after being hidden:

Obviously the first path is always being taken for some reason.


Pagination support encoded path

Following #262
Finally it's not a good idea to encode the path first you have to split the query from the path and encode.
Something like
uri = URI.new!("/team?page=:page")
iex(7)> URI.decode_query(uri.query)
%{"page" => ":page"}
iex(8)> query = URI.decode_query(uri.query) |> URI.encode_query()
"page=%3Apage"
iex(9)> %{uri | query: query} |> to_string()
""/team?page=%3Apage""

Instead I use a simple regex with a OR.

Looks great. Many thanks for contributing! 🙌


Fix broken link, and minor typos.

fixes link to configuration/#concise-configuration
some typos (hope I got the British English correct)
splitting one sentence into two for easier readability

Note:
Instead of ../../configuration.md#concise-configuration one could maybe also do /configuration.md#concise-configuration. I was not sure what you would prefer?
Also I could not fully test this. I don't know where the path configuration.md#concise-configuration gets translated into configuration/#concise-configuration.

Ace, thanks @spier! I'll add a section in CONTRIBUTING.md outlining how to check the docs, I think it's actually gonna be ../configuration.md#concise-configuration in the end but I'll merge and check locally.


Make negative timestamp to work under Windows

datetime.datetime.fromtimestamp can't work under Windows with nagative time stamp:
Traceback (most recent call last):
  File "visual.py", line 256, in <module>
    init()
  File "visual.py", line 115, in init
    onscreen_quakes[-1].load()
  File "visual.py", line 35, in load
    self.datetime  = datetime.datetime.fromtimestamp(int(self.time)/1000)
OSError: [Errno 22] Invalid argument

So use the following:
datetime.datetime(1970, 1, 1) + datetime.timedelta(seconds=timestamp)

Based to the answer from Stackoverflow

Thanks @hellofwy


Add custom fields tutorial

this isn't happening any time soon, so it isn't worth keeping the issue around


Need configurable (or unlimited) buffer size

Many command line tools I use for development output a lot of lines to stdout, having a fixed sized buffer means I can't scroll back to see the entire output.  Many modern terminals support configurable or unlimited buffer sizes (even cmd.exe was configurable!).

Something tells me that one of our other 9615 issues covers this. 😄

Found it! /dup #1410

Dupe https://github.com/microsoft/terminal/issues/1410

Thanks @wsluser, you beat me to it


Authenticating via API key not supported?

Hello!
It seems that authenticating with an API key is not supported in this client. But while the docs for nodejs-speech don't mention it as an option, the general google-cloud-node docs on authentication suggest you can authenticate with an API key for "any APIs that accept an API key." Following the pattern of passing a key attribute to the constructor (I tried with both @google-cloud/speech and the meta google-cloud package) gives Error: Could not load the default credentials. Browse to https://developers.google.com/accounts/docs/application-default-credentials for more information.
Are there any plans to make this possible for the speech client? Otherwise, perhaps that language should be adjusted.
Thanks!
Environment details

OS: macOS 10.13.3
Node.js version: 9.4.0
npm version: 5.6.0
@google-cloud/speech version: 1.0.1

Steps to reproduce

Create project, enable Google Speech API, generate API key with no restrictions.
Run code like

import speech from '@google-cloud/speech';
const sc = new speech.v1.SpeechClient({
  projectId: process.env.GCLOUD_PROJECT_ID,
  key: process.env.GCLOUD_API_KEY,
});
sc.getProjectId((err, str) => {
  console.log(err);
});

Hello,
I don't think it's possible at the moment since google-auto-auth which is used internally in<EMAIL_ADDRESS>does not seem to support that option.
There is an issue that references this as a request after the option has been added to the underlying google-auth-library-nodejs : https://github.com/stephenplusplus/google-auto-auth/issues/9
I tried the following without success :
var googleAuth = require('google-auto-auth');
const speech = require('@google-cloud/speech');
const fs = require('fs');

var gAuthClient = googleAuth({
  apiKey: 'MY_API_KEY'
})

const client = new speech.SpeechClient({
  projectId: "MY_PROJECT_ID',
  auth: gAuthClient
});

Could be my environment though :

Node.js : 9.5.0
npm : 5.6.0
@google-cloud/speech : 1.2.0

@phsultan What was the error you received?

I just received an 'Invalid grant' message from the API server, and actually, nothing in the code of google-auto-auth would allow me to authenticate using apiKey as an attribute.
@stephenplusplus, is it something that is planned to be included in google-auto-auth ? If so, would you accept PRs in that regard ?
I'm asking because of these two issues:

https://github.com/stephenplusplus/google-auto-auth/issues/9 (attempt to authenticate using apiKey, now closed)
https://github.com/stephenplusplus/google-auto-auth/issues/39 (deprecation message)

We've had an API or two that used API keys for authentication in the past, and google-auto-auth wasn't involved at all, actually. For example, in the case of Speech, we would accept it as an argument:
const speech = new Speech({ apiKey: '...' })

Then we'd give the key right to gRPC to use however it does.
However, a PR to accept a key in google-auto-auth probably wouldn't hurt, so I'd definitely take it :)

I can give a bit more feedback on this issue. From my testing, I was not able to pass the API key to gRPC in all cases. However, using google-auth-library or even axios directly to send REST requests is pretty straightforward, and mostly preserves the code patterns we have with @google-cloud/speech:

gist using google-auth-library
gist using axios

I tried to pass the API key as a key parameter to gRPC by adding it as a metadata or as a request attribute and after having modified the cloud_speech.proto file with no luck. The reason why I modified cloud_speech.proto was to try to pass the key as a URL parameter, but I'm not even sure that's the right way to go.
Here is the corresponding code : gist using gRPC
Hope this helps!

I'd be fine with using a service account as recommended in the documentation, however the real issue is that there doesn't seem to be any predefined Role for granting API access to service accounts. The only way for a service account to access the Speech-To-Text API is to give it the "Owner" role, which is a non-starter for most real-world use-cases.
Any chance we can have a more granular permission system in the future ? In the meantime, being able to use an API key would solve the issue, since API keys can be restricted to a specific API (e.g. Speech-To-Text).

We've had an API or two that used API keys for authentication in the past, and google-auto-auth wasn't involved at all, actually. For example, in the case of Speech, we would accept it as an argument:
const speech = new Speech({ apiKey: '...' })

Then we'd give the key right to gRPC to use however it does.
However, a PR to accept a key in google-auto-auth probably wouldn't hurt, so I'd definitely take it :)

Man, does this mean, that if I get/create api key for speech-to-text, I can create two speech clients, that can create two speech-to-text streams at same time?

Hi folks,
I'm just going to update this issue to let you all know that it is possible to authenticate Node.js Speech client using an API key, even though it's a little bit hacky. This gist shows the complete example, the main idea is here:
const apiKey = 'REDACTED'; // <-- use your API key here
const speech = require('@google-cloud/speech');
const {GoogleAuth, grpc} = require('google-gax');

function getApiKeyCredentials() {
  const sslCreds = grpc.credentials.createSsl();
  const googleAuth = new GoogleAuth();
  const authClient = googleAuth.fromAPIKey(apiKey);
  const credentials = grpc.credentials.combineChannelCredentials(
    sslCreds,
    grpc.credentials.createFromGoogleCredential(authClient)
  );
  return credentials;
}

const sslCreds = getApiKeyCredentials();
const client = new speech.SpeechClient({sslCreds});

Anyway, please be advised that we recommend using service account key for authentication on the server side, and OAuth2 for the client side authentication. Please consider using either of those instead of API key authentication.


[base-ui][NumberInput] Implement numberInputReducer

Part of https://github.com/mui/material-ui/issues/38514
This PR adds a numberInputReducer function that would be used to determine the internal value and inputValue of the useNumberInput hook. (I plan to integrate this with useNumberInput and useControllableReducer in a separate PR)
The logic is mostly extracted from useNumberInput with one notable change – the value can be set to an empty string when the input is completely cleared. Previously it was just undefined but it shouldn't be used as it would change the controlled/uncontrolled state (https://github.com/mui/material-ui/issues/38513)

[x] I have followed (at least) the PR section of the contributing guide.

@michaldudak Regarding managing the controlled/uncontrolled state, if a component is using useControllableReducer it wouldn't need to use the state provided by useControlled right? It just needs to show the error

Regarding managing the controlled/uncontrolled state, if a component is using useControllableReducer it wouldn't need to use the state provided by useControlled right?

Right. useControllableReducer is an equivalent of useControlled in a sense that it handles both the controlled and uncontrolled values. You don't need both useControllableReducer and useControlled in the same component.

Setting back to draft first – I will add an additional set of action types that are more detached from the underlying events/user actions and closer to how the state is being operated on, and will try integrating both

@michaldudak I've updated this ~ in the end I decided to not pass any events into this at all to keep it simpler (I was only using very few properties on the events anyway)
It's also much clearer what each action is going to do:
clamp: 'numberInput:clamp',
inputChange: 'numberInput:inputChange',
increment: 'numberInput:increment',
decrement: 'numberInput:decrement',
decrementToMin: 'numberInput:decrementToMin',
incrementToMax: 'numberInput:incrementToMax',


Feature/integration tests

In prep for a 2.0.0, rolls back to the traditional "OTHER" branch behavior, while adding integration tests.
Current coverage is ~73%, which pretty much accounts for all non-failing (error) code paths and things that don't directly interact with git.
Further development toward an additional feature in 2.0.0 (for deploying long-running branch artifacts from OTHER branches) will be done in a separate branch, in a separate PR.

This should take care of #89 . :-)


fix: Change who's -> whose

This change is

Coverage increased (+0.3%) to 92.141% when pulling 5c03dbb915721e76781e13ecc7c651149804bfd1 on jacobmischka:fix-whos into 2923515a0aec80a7ebd4b1fc3430aad859bcda8a on kbknapp:master.

Thanks!


Large memory consumption

I have a large memory consumption. It starts from a few mb, but goes to a few Gb in about 24h.
● colord.service - Manage, Install and Generate Color Profiles
     Loaded: loaded (/lib/systemd/system/colord.service; static)
     Active: active (running) since Sat 2023-03-18 17:09:14 CET; 16h ago
   Main PID: 1432 (colord)
      Tasks: 3 (limit: 18908)
     Memory: 1.4G
        CPU: 4min 20.780s
     CGroup: /system.slice/colord.service
             └─1432 /usr/libexec/colord

I'm one fresh Ubuntu 22.04 (same behaviour on ubuntu 20.04) with a bunch of dockers.
Linux alpxp-nuc 5.15.0-67-generic #74-Ubuntu SMP Wed Feb 22 14:14:39 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
How can I debug what is going on? What logs should I check?

oops. sorry )


GlobalSettings for Java, Global Plugin for Scala

3rd attempt with a single clean commit, so that it's easier for Guillaume to merge the code.
This commit includes the updated version of GlobalPlugin for Scala, the Java adapter, the Global settings for Java and an updated sample for todolist java evan that creates 3 entries as a bootstrap

Merged.


ddcutil: 1.3.0 -> 1.3.2

Description of changes
Version bump with no functional changes
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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 (clight ddcui ddcutil enlightenment)
[x] Tested basic functionality of all binary files
[x] Tested with ddcutil detect
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Note: the release doesn't have a tag on the github but it's available on the website.


Tracking reasons for requesting changes

Engagement reports can be returned to the report author for one or more reasons.
Each day the requirements owners count the reasons why reports were returned to the report authors.
All six report reviewers update a master Excel spreadsheet daily.
The requirements owners want ANET to tabulate why reports are being returned to the report author for changes.
The results for approvers who return reports for changes should also be included.

@dspCarlson Do you mean that ANET should provide an spreadsheet containing the requested changes for all engagement reports within ANET?

Maybe we should provide a link which downloads the spreadsheet?
If so, who should be able to download the spreadsheet and where should we place the link, on the dashboard?
Which information should we provide? Report link, report id, approver, reason?
I don't understand what you mean with: "The results for approvers who return reports for changes should also be included."

@maradragan When report approvers, including the administrators, request changes, they will be able to choose one or more reasons why the engagement report was returned to the report author.  This one-to-many relationship (one report to one or more reasons for request changes) will require a database change.  This change will require a GUI change for administrators/report approvers.
I don't think we need to worry about outputs (e.g. spreadsheets) yet.
Question:
I don't understand what you mean with: "The results for approvers who return reports for changes should also be included."
Answer:
Currently, only the request for changes are being tabulated by the administrators in Excel spreadsheets.  But the administrators want to also tabulate report approvers and the request for changes they make.  The administrators are going to do analytics on why reports are returned for changes for all ANET report approvers.

It is probably related to the implementation of #1574

@maradragan The requirements owners were really hot about this issue a month ago.  Now they haven't asked about the progress of this issue.  There are are about 10 reasons why reports are returned ("request changes") to report authors for edits.  Instead of writing out the reasons each time, the report approvers and administrators reviewing reports should be able to just check reasons.  For example, one reason for returning a report is the principals mentioned in the "Engagement details" are not assigned to the report in the "Attendees" data field.  Report authors can't find the principal engagement attendees in ANET so they will add anyone or the report author is simply lazy and doesn't try to find the principal attendees.

This issue is OBE.  The requirements owner is long gone.


enhancement enable memtable on sink node for broker load by session var

Proposed changes
As title.

run buildall


Tile localization and better tooltips

In an effort to improve the tilesheet extension the following plan has been laid out which will allow for translation of tilesheet icons as well as improved tooltips. Even translation into English will be useful as it will allow multiple icons on a tilesheet to share the same display name, as well as allowing custom tooltips to be assigned across the wiki.

[ ] Add a table to the database with 4 columns for entry id (matching the entry id in the existing table), language code (such as en, de, or ru), display name, and description.
[ ] Add a special page where translators can translate tilesheet icons, assigning them a display name and description that suits their language.
[ ] For invoking the tilesheet extension add tooltip parameters and having the extension output the appropriate tooltip code, instead of the wrapping template handle it. Order of priority for the tooltip is tooltip parameter at invocation, display name + description from the new table, and defaulting to the item name in the tilesheet itself.

I don't quite understand that last part. Could you clarify the priority thing a bit better?

@elifoster you should work on this, it's a pretty big flaw in the translation system as of now.
Translators shouldn't really have to translate much; in theory translations will be auto-generated based on a comparison between the tilesheet dump file, en_US.lang and other language files. I made a map for this function a while ago.

Yeah, I probably should. I still need to get my local wiki working properly with all the templates and stuff.

@xbony2 To do that automatically all you'd need is an API and then write a program to interact with that API.

Exactly.

So should the table start out like:
entry_id  |  display_name  |  description
-----------------------------------------
0         |   Item Name    |  Description

Then, you can go and add a language to its known langs, for example:
entry_id  |  display_name  |  description  |  display_name_es  |  description_es

At which point you can translate the display_name and description into es.
This is how I have started it.

No, we are not adding columns for each language.
It should instead be:

entry_id
lang
display_name
description

0
en
Item name
Description

0
es
Nombre del árticulo
Descripción


Edit Location zaps existing geolocation

Clicking the Edit Location button automatically zaps existing lat/lon (and probably alt).
(This tends to be destructive. Although the old geoloc is technically preserved via versioning,
we can't tell if a lat/lon change is deliberate or the result of the bug. It's a pain to restore the geoloc.)
Example:

goto https://mushroomobserver.org/locations/401
result:

Click "Edit Location"
result: MO automagically destroys the existing geoloc

This should be a pretty easy fix, once i can focus on it. There's probably a line that replaces the nsew bounds with those loaded from Google on map open, that needs to be moved. On Dec 17, 2023, at 7:46 PM, Joseph D. Cohen @.***> wrote:
Clicking the Edit Location button automatically zaps existing lat/lon (and probably alt).
(This tends to be destructive. Although the old geoloc is technically preserved via versioning,
we can't tell if a lat/lon change is deliberate or the result of the bug. It's a pain to restore the geoloc.)
Example:
goto https://mushroomobserver.org/locations/401
result:
Screenshot.2023-12-17.at.7.34.27.PM.png (view on web)
2. Click "Edit Location"
result: MO automagically destroys the existing geoloc
Screenshot.2023-12-17.at.7.35.03.PM.png (view on web)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

I believe this is fixed by #1729

You're right. Thanks so much for doing this.


Update spec.selector in r11s deployment manifests

Description
Updates k8s Deployment manifests for Routerlicious so spec.selector.matchLabels matches all the label values in spec.template.metadata.labels instead of just one of them.
K8 documentation is a bit confusing around this topic, it says that if several Deployments have overlapping spec.selectors, they might start to compete to manage the same pods and cause unexpected issues. However, it seems that only applies to pods created manually (with labels that match the selectors for more than one deployment/replicaSet). Pods that are created by Deployments (or technically, by the ReplicaSets created by Deployments) also have metadata.ownerReferences values which Deployments/ReplicaSets also leverage to know exactly which pods they should try to manage. For pods that have a value there, that dictates who should manage it; for pods with no value there, then the label selectors come into play and Deployments/ReplicaSets could start competing.
Since we (should) only have pods created by the Deployments themselves, the label selectors (spec.selector.matchLabels) don't really matter right now, but I wanted to update them to match the labels that the pods for each Deployment end up getting (spec.template.metadata.labels) so we further minimize the risk of weird things happening where these charts are deployed.

@hedasilv @znewton a quick cleanup-like change. Nothing urgent, just for peace of mind 😄.

Ah, good catch, it does apply there too, I missed it myself when I looked around. I'll open another PR for that one and do a double check I didn't miss any others.


Automatically paginate GQL Queries

Fixes #1696.
This change adds support for automatic pagination with GQL queries.
Since GQL query positions are determined by query string binding values (i.e. offset and limit), programmatically paging is difficult using GQL. However, the server returns the computed value of the query in the first result batch when a GQL query is executed.
By dispatching subsequent requests using the computed query and discarding the GQL query, we can page through results in the same manner as with a query object.

Would we be able to add a system test to exercise pagination please?

@dwsupplee good call; found a bug too. PTAL.


WIP Fix TO Go cdn dnssec generate to accept numbers

The docs say they're strings, but it appears Perl accepts both,
and the Portal sends numbers. This changes Go to emulate the Perl.
Fixes #2716
What does this PR do?
Fixes #2716
Which TC components are affected by this PR?

[ ] Documentation
[ ] Grove
[ ] Traffic Analytics
[ ] Traffic Monitor
[x] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] Other _________

What is the best way to verify this PR?
Steps to Reproduce in #2716
Be aware this endpoint takes a long time, reproducing will likely require increasing http and db timeouts.
Check all that apply

[ ] This PR includes tests
[ ] This PR includes documentation updates
[ ] This PR includes an update to CHANGELOG.md
[ ] This PR includes all required license headers
[x] This PR does NOT fix a serious security flaw. Read more: www.apache.org/security

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2299/
Test PASSed.

@rob05c - can you write a unit test for this?

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2303/
Test PASSed.

@rob05c - can you write a unit test for this? or an integration test? or whatever makes the most sense?
@mitchell852 https://github.com/apache/trafficcontrol/pull/2720/files#diff-548856fd9ea3ae645fc12cb9afde3e31R31

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2308/
Test PASSed.


Syntax highlighting doesn't work.

Information
Browser details https://whatismybrowser.com/w/G93E9TN
Details
After any input changes the highlights doesn't work.
Description
Well, that's because there is something wrong with  Prism.util.isActive  function.
Steps to reproduce
Just change any input.
Expected behavior
Highlight the changes.
Screenshots

Cannot reproduce in production, looks to be an issue in dev only. I wonder if some globals are being registered badly due to hot reloading.

That's probably what it is. I run the app component out of the our build workflow and works perfectly.


:arrow_up: Upgrade lodash

See  #208

awesome bro

Awesome ! You're the best !

Thank you guys ❤
As I said in the issue, I will fix other minor vulnerabilities when @vue/cli-plugin-unit-jest bump to v4 😉


Improve initial setup

Hubtty should generate the hubtty.yml configuration file if none is found.
It should provide a consistent experience with the gh CLI tool and offer to create the oauth token on first use.
https://docs.github.com/en/free-pro-team@latest/developers/apps/authorizing-oauth-apps

Hubtty now initiates the device flow to generate the oauth token and copies a default configuration file to streamline initial setup.

Hubtty now initiates the device flow to generate the oauth token and copies a default configuration file to streamline initial setup.


Cvd-contract-type - wrong hungarian translation for 'veh-acq'

We have noticed that in cvd-contract-type codlist in case of code 'veh-acq' the hungarian translation is not correct.
The english translation is:
         <Value ColumnRef="eng_label">
            <SimpleValue>Vehicle purchase, lease or rent</SimpleValue>
         </Value>

The hungarian one:
         <Value ColumnRef="hun_label">
            <SimpleValue>Járművásárlás, -bérlés vagy -bérlés</SimpleValue>
         </Value>

I think the proper hungarian translation should be like 'Járműbeszerzés, -lízing vagy -bérlés'
We use SDK 1.11.1, but it also a problem in SDK 1.12.
Could you please investigate this issue?
Thanks is advance!

Thank you for raising this issue, there does seems to be a repeated term in Hungarian.
The source of this codelist entry is in article 3 in the amendment to the CVD:
https://eur-lex.europa.eu/legal-content/HU/TXT/HTML/?uri=CELEX:32019L1161
The full sentence in Hungarian is "ajánlatkérő  szervek vagy közszolgáltató ajánlatkérők által odaítélt, közúti járművek  megvásárlására, lízingjére, bérletére vagy részletvételére vonatkozó  szerződések, amennyiben azok kötelesek alkalmazni"
Does your proposed correct align with the same words/terminology?
We can try to catch this correction SDK 1.13 and the label will later be patched to 1.11 and 1.12

Yes, our proposal complies with the legislation. Thank you very much for your support and for correcting the translation of the codelist!

Thanks for confirming the codelist correction aligns with the directive.  We will include this in SDK 1.13 and you'll see it later on in the patches to the previous SDKs.


Support the gamepad UI layout for the integrated screen checks

First of all, Thanks for a great tool!
I like to run the leaguestart campaign with a gamepad due to the small QoL features like auto gem leveling. But the integrated check for the Gamescreen don't seem completely reliable with the alternate UI art and layout. After calibrating, the test seems to have a 50% chance to be positive or negative. The inventory calibration and check seems ok, but I haven't tested it extensively. The other image based checks are of course fine.
Here's a couple of screenshots to show what it looks like:

Yes, I can look into it. Though the gamescreen check is less important when playing on a gamepad. Aside from auto-hiding clone-frames (it used to affect more features), it mainly controls what an omni-key press does. If the script detects that the player is on the "gamescreen" when the omni-key is pressed, it will assume it was pressed to use a skill bound to it and not to access a feature. If the check is negative during a press, it will initiate screen-reading to determine the context of the key-press.
So the check merely minimizes the amount of screen-reading the script does (depending on which key/skill combination is used, the omni-key might be pressed excessively) in order to improve script -- and sometimes system -- performance. Since playing on a gamepad doesn't involve much omni-key usage, it should be fine for now. To make sure you don't get too many false-negatives for that check when you do press the omni-key, you could calibrate the gamescreen check with a rare color-value (setting the camera in a way that e.g. the fireplace in Lioneye's watch fills the top-right corner of the screen).

Ok, thanks! I have also started noticing the leveling guide behaving weirdly, skipping ahead of for example Tidal Island as soon as I entered it, don't know if it's related to this or because I rerolled to a different class and reset the progress with a new build? I assume that's based on log parsing though so idk.
One more note is that ctrl-f doesn't work as usual since there's a game pad specific button combo instead, but I am able to use the buy gem filter by manually focusing the search field first and then hitting the omni-key. Not sure if that's possible to improve script-wise but you could explain it in the help tooltip as a backup plan.

I rerolled to a different class and reset the progress with a new build

As long as you use the "reset progress" in the settings menu after importing a new build, everything should work fine. If this keeps happening, please report back.

ctrl-f doesn't work as usual since there's a game pad specific button combo instead

Right, makes sense. I think you're the first to bring up gamepad support, so it never crossed my mind. I don't think AHK can send gamepad presses but I'd have to do some research to be certain. Otherwise, one would have to make do with this hybrid approach.

Yep, and I'm almost certainly a minority user here, so my expectations are low and I'll get by just fine with how it already works. It's just for the campaign after all. Already appreciate you just replying and considering it 😁

Update on the level guide automation: I think it always detects me as being in Lionseye, because it never moves to the next step except if the next step is in Lionseye.

Huh, the first thing you could check is if the client.txt in the logs folder of the game is still being written to correctly by the client. If it behaves normally, the script itself could be stuck or the game may deny access to the log-file. Either way, it's the first time I've heard of such a behavior.

Yeah, here's the last part, and it looks like this is a PoE issue :P
2023/04/08 15:02:08 73203 951bc057 [INFO Client 1820] [STARTUP] Registration in 0.484979 seconds
2023/04/08 15:02:11 75734 47bbea58 [INFO Client 1820] Changing to device "Headset Earphone (11- Arctis 7 Chat)"
2023/04/08 15:02:11 75750 951bc057 [INFO Client 1820] [STARTUP] Audio in 3.02254 seconds
2023/04/08 15:02:11 76281 b51d4f61 [DEBUG Client 1820] [RENDER] SetRendererSettings
2023/04/08 15:02:11 76281 b51d6447 [DEBUG Client 1820] [RENDER] ResizeWindow
2023/04/08 15:02:11 76281 3cd2dd07 [DEBUG Client 1820] [VULKAN] ResizeBuffers
2023/04/08 15:02:11 76281 3cd2cec0 [DEBUG Client 1820] [VULKAN] ReinitSwapChain
2023/04/08 15:02:11 76281 97399bf1 [INFO Client 1820] [VULKAN] SwapChain = 3840x2160
2023/04/08 15:02:11 76328 9d4eec8a [DEBUG Client 1820] [WINDOW] TrigggerDeviceLost
2023/04/08 15:02:11 76359 9d4eeba7 [DEBUG Client 1820] [WINDOW] TrigggerDeviceReset
2023/04/08 15:02:11 76390 b51d614c [DEBUG Client 1820] [RENDER] RestartSwapChain
2023/04/08 15:02:11 76390 9e6a0b0a [INFO Client 1820] Enumerated adapter: NVIDIA GeForce GTX 1070
2023/04/08 15:02:11 76515 9e6a1d2a [INFO Client 1820] Enumerated device for adapter: NVIDIA GeForce GTX 1070. Selected feature level: 45312. Max feature level: 45312
2023/04/08 15:02:11 76515 9e6a1363 [INFO Client 1820] Enumerated output for adapter NVIDIA GeForce GTX 1070 of \\.\DISPLAY1
2023/04/08 15:02:11 76515 9e6a0b0a [INFO Client 1820] Enumerated adapter: Microsoft Basic Render Driver
2023/04/08 15:02:11 76531 9e6a1d2a [INFO Client 1820] Enumerated device for adapter: Microsoft Basic Render Driver. Selected feature level: 45312. Max feature level: 45312
2023/04/08 15:02:11 76531 b51d650d [DEBUG Client 1820] [RENDER] ResizeDevice
2023/04/08 15:02:11 76531 9d4eec8a [DEBUG Client 1820] [WINDOW] TrigggerDeviceLost
2023/04/08 15:02:11 76531 9d4eeba7 [DEBUG Client 1820] [WINDOW] TrigggerDeviceReset
2023/04/08 15:02:11 76578 3275a75e [INFO Client 1820] [SHADER] Disable Async: 1 frames
2023/04/08 15:02:11 76578 b51d610e [INFO Client 1820] [RENDER] Disable Async: 1 frames
2023/04/08 15:02:12 77484 3cd2cec0 [DEBUG Client 1820] [VULKAN] ReinitSwapChain
2023/04/08 15:02:12 77484 97399bf1 [INFO Client 1820] [VULKAN] SwapChain = 3840x2160
2023/04/08 15:02:12 77546 9d4eec8a [DEBUG Client 1820] [WINDOW] TrigggerDeviceLost
2023/04/08 15:02:12 77593 9d4eeba7 [DEBUG Client 1820] [WINDOW] TrigggerDeviceReset
2023/04/08 15:02:15 80187 ca86d7fb [DEBUG Client 1820] Rebuilding UI::Core::Properties due to input method change
2023/04/08 15:02:15 80187 ca86fd3f [DEBUG Client 1820] Rebuilding UI::Core properties. Current input mode = '1'
2023/04/08 15:05:59 303781 ca86d7fb [DEBUG Client 1820] Rebuilding UI::Core::Properties due to input method change
2023/04/08 15:05:59 303781 ca86fd3f [DEBUG Client 1820] Rebuilding UI::Core properties. Current input mode = '2'
2023/04/08 15:05:59 303875 890d9d5f [INFO Client 1820] Async connecting to ams01.login.pathofexile.com:20481
***** LOG FILE ERROR. REOPENING *****

Don't know if the timestamps are UTC or local timezone, but nothing's been logged for at least 3 hours 😅 I'll deal with that on my own.
Good news is that this log also shows an obvious way to detect a gamepad being used, so there's a silver lining!

Well there's your problem!

I did some testing around integrating gamepad support and initially got good results. But then I noticed that AHK was causing unusually high CPU-load when the omni-key was mapped to a gamepad-key (and when there were references to gamepad-keys in general) -- to the extent that the script was generating as much load as the game-client itself. While CPU-load itself is not a big deal, I cannot rule out the possibility that this may lead to stalling or increased latency within the script's code itself.
There's also the problem that AHK is limited to receiving gamepad key-presses. It cannot send key-presses without using external gamepad drivers/emulation, so there's no easy way of sending gamepad-specific versions of the required key-combinations (e.g. CTRL-F).

Unfortunate. And even if you could send button presses through a virtual controller, I think PoE might lock input to a single device.
The gamescreen detection might still be possible to improve, though.

I also use Xbox 360 gamepad. And I used reWASD to edit what the button mean. And everything was fine for me.
For example,

I edit the "omni-key" to be "P" key,
2.1  in reWASD, "holding gamepad button B" is set to "press button B + Holding P key for 0.5 second".
2.2  in reWASD, "press gamepad button B" is set to "press button B + press P key".

Everything was solved for me at present. However, I only use iteminfo-check at present. I don't know if any other function work well or not.

Hey again! Quick update, it seems like ´Ctrl+F´ is working now even when using a gamepad, so that's no longer an issue! But I'm still having problems with unreliable screen checks, even on the syndicate board which is pretty static. This might point to it being unrelated to using a gamepad, so I will do a bit of extra experimenting tomorrow!
And I'm using a Steam controller, which also has a rich ability to rebind buttons. I set the omni key to ´Home´ since it's the least in the way of both normal usage and standard bindings.

I see. Now that I think of it, it's probably better to leave the gamescreen pixel-check uncalibrated for gamepad-usage in order to prevent false-positives. You can delete the calibration data by removing the [gamescreen] section in ini\screen checks (XXXXp).ini and then restarting the script. This will lead to omni-key presses always activating omni-key features (as opposed to whatever is normally bound to the key, e.g. middle mouse-button).
The only drawback is that automatic clone-frame toggling would not be available since that also uses the gamescreen pixel-check. If you want to make use of that, let me know -- I'll try to source a gamepad from somewhere then.


gzip: invalid header

I'm getting "gzip: invalid header" from helm/tiller when trying to install a chart from a helm repository hosted through bitbucket.io. I've also tried an Azure CDN and get the same error.
These are the basic build steps for installing helm and packaging the chart. At the end I simply add the .tgz and updated index to a repository and commit it.
It seems like when the file is fetched/downloaded using helm the gzip compression is stripped off and we are left with a tar file, however if I just use a browser to download the file the gzip compression is left intact and we can install that file local from disk just fine.
What am I doing wrong/How do I fix this?
# install helm
- wget "https://raw.githubusercontent.com/kubernetes/helm/master/scripts/get" -O /usr/local/bin/helm-install.bash
- chmod +x /usr/local/bin/helm-install.bash
- helm-install.bash -v v2.5.1
- helm init --client-only
- export HELM_PACKAGE_VERSION=0.$BITBUCKET_BUILD_NUMBER.0
- helm package chart/design-service --version=$HELM_PACKAGE_VERSION
- helm repo index --url https://buildasign.bitbucket.io/helm --merge siterepo/helm/index.yaml siterepo/helm
...  other steps to check in to repo

helm repo add buildasign https://buildasign.bitbucket.io/helm/
helm repo update
helm upgrade -i qa-design-service buildasign/design-service
Release "qa-design-service" does not exist. Installing it now.
Error: gzip: invalid header

@Strandedpirate It appears your problem is in the way the chart is (or is not) being gzipped. Debugging the code, it doesn't fail until its actually trying to load the archive from disk saying its not a gzip. Also when I try to open the fetched chart archive (.tgz) in vim, it complains that its not a valid gzipped tar.
Not sure what is causing this on your end but it looks like its not gzipped to me. Perhaps to troubleshoot you can package it with helm and then test to see if the chart is gzipped correctly (for me just opening with vim is enough but I have vim.tar compiled in apparently).

I'm going to close this ticket due to inactivity, but please re-open if this still needs to be addressed. Thanks!

Re-opening...
I'm able to reproduce that problem with latest (2.8.1) helm and repo hosted on bitbucket.
So I can confirm that package in the repo is actually gzipped file:
repo/python-0.1.0.tgz: gzip compressed data, extra field, has comment```

When I do `helm fetch` for the repo I'm getting file ~6 times bigger:

$ file python-0.1.0.tgz
python-0.1.0.tgz: POSIX tar archive

So as we can see, tarball was ungzipped in transit.

I suspect that http client that you use in helm isn't properly configured and tries to behave like a browser by unpacking gzipped stuff on the fly.

I can confirm that required headers are set properly serverside:

< Content-Type: application/x-tar
< Content-Encoding: gzip
< Content-Language: en
< Accept-Ranges: bytes
< Content-Length: 5276

@bacongobbler @jascott1  I've just realized that this issue still closed. Could you please reopen?

I have the same issue with a chart hosted on bitbucket.
helm install --name es --debug --dry-run es/
[debug] Created tunnel using local port: '43131'

[debug] SERVER: "localhost:43131"

[debug] Original chart version: ""
[debug] CHART PATH: /home/jonathan/Projects/es-helm/es

Error: error unpacking es-common-0.1.0-pre.0.tgz in es: gzip: invalid header

Is it possible that Bitbucket decompresses gzipped requests? I can't reproduce with chartmuseum or the stable/incubator charts.

Locally I'm able to untar it with tar xvf using the z option gives the error
gzip: stdin: not in gzip format

can you try a chart not hosted on bitbucket, such as one of the charts from the stable/incubator repos? If so then there's something up with hosting charts on bitbucket specifically.

I would also like to share that I am experiencing a similar "Error: gzip: invalid header" error. I am also hosting my charts via Bitbucket.

@bacongobbler can we please reopen this issue?

Uhh, this issue was never closed to begin with :)

I think @rochaporto is on the right track though. By default we use go's default HTTPClient which as @dene14 suggested, respects headers and adapts the output. It should be a pretty simple change under NewHTTPGetter to return a http.Client using a http.Transport with the DisableCompression flag set to true (and the associated unit test to cover that).
Marking this as a good first issue if someone wants to tackle this!

fixed in #4165

Still the same issue, even when file stored locally
helm version
Client: &version.Version{SemVer:"v2.11.0", GitCommit:"2e55dbe1fdb5fdb96b75ff144a339489417b146b", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.11.0", GitCommit:"2e55dbe1fdb5fdb96b75ff144a339489417b146b", GitTreeState:"clean"}
Deploy should be to azure aks cluster. helm install work well.
Issue also reproducible from azure devops (vsts) using ubuntu16.06 hosted agent.

FWIW I got this inappropriate error as well when I pointed to a yaml file instead or a chart directory.

Same error for local dep charts with helm v2.12.1 and OSX Mojave
If I build the dependencies it creates a gzipped file that can't be decompressed when installing: Error: gzip: invalid header
helm install --dry-run --debug ./Chart.yaml --values values-int.yaml
[debug] Created tunnel using local port: '52958'

[debug] SERVER: "<IP_ADDRESS>:52958"

[debug] Original chart version: ""
[debug] CHART PATH: <my_local_directory>/helm/marketplace/Chart.yaml

Error: gzip: invalid header

to simplify, testing with only one dep:
#Requirements.yaml
dependencies:
  - name: redis-ha
    version: 3.0.3
    repository: file://charts/redis-ha

file charts/redis-ha-3.0.3.tgz
redis-ha-3.0.3.tgz: gzip compressed data, extra field, has comment, original size 35840

tar -xvzf charts/redis-ha-3.0.3.tgz works fine
I don't know if it's related but  I got an extrange behaviour when updating dependencies:
$ helm dep update
Hang tight while we grab the latest from your chart repositories...
...Unable to get an update from the "local" chart repository (http://<IP_ADDRESS>:8879/charts):
	Get http://<IP_ADDRESS>:8879/charts/index.yaml: dial tcp <IP_ADDRESS>:8879: connect: connection refused
...Successfully got an update from the "coreos" chart repository
...Successfully got an update from the "stable" chart repository
Update Complete. ⎈Happy Helming!⎈
Saving 1 charts
Save error occurred:  directory <my_local_directory>/helm/marketplace/charts/redis-ha not found
Deleting newly downloaded charts, restoring pre-update state
Error: directory <my_local_directory>/helm/marketplace/charts/redis-ha not found

But that directory exists and it's readable
$ls -la <my_local_directory>/helm/marketplace/charts/redis-ha
total 40
drwxr-xr-x   6 rub  staff   192  3 ene 09:32 .
drwxr-xr-x   3 rub  staff    96  3 ene 09:44 ..
-rwxr-xr-x   1 rub  staff   479  3 ene 09:26 Chart.yaml
-rwxr-xr-x   1 rub  staff  8256  3 ene 09:26 README.md
drwxr-xr-x  10 rub  staff   320  3 ene 09:32 templates
-rwxr-xr-x   1 rub  staff  3980  3 ene 09:26 values.yaml

I am seeing this problem with helm3.
# helm dep up tenant-api
Error: error unpacking helm-chart.tgz in tenant-api: gzip: invalid header

I have been talking to bitbucket support and we made a feature request; https://jira.atlassian.com/browse/BCLOUD-19694 vote for it!

@jurgenweber I don't believe #4165 was ported over to Helm 3. If you feel like porting the fix over to Helm 3, please feel free!
Re-opening for the moment.

I got same error.
My helm repo is in S3 bucket. When I doing helm s3 reindex <repo_name> and I got error below message
helm s3 reindex jcl
traverse the chart repository: load archive from s3 object: gzip: invalid header
Error: plugin "s3" exited with error
make: *** [prepare] Error 1

well you acn fix that in s3, change the metadata, etc.

I hit similar issue with Helm (3.0.3) and Bitbucket, but the problem is this one:
Error: file '<cache>/helm/repository/passwordstore-0.7.2.tgz' does not appear to be a gzipped archive; got 'application/octet-stream'
helm.go:75: [debug] file '<cache>/helm/repository/passwordstore-0.7.2.tgz' does not appear to be a gzipped archive; got 'application/octet-stream'

The gzip is a legit one, but when helm check the content type got application/octet-stream and not x-gzip.
I think the problem is in this check in the file chart/loader/archive.go:
if contentType := http.DetectContentType(buffer); contentType != "application/x-gzip"

That should accept octect/stream too

We have random does not appear to be a gzipped archive; errors after switching to Helm 3 (3.0.2). Usually with Redis and incubator/raw charts. Subsequent runs work ok, but the error is quite annoying.
At first, it seemed to be a CI runner problem. Due to this error's random nature, it's hard to reproduce and diagnose.

yeah, I have seen that also. Not really related to the OP, should make another issue.

I am facing the same issue. But this is only appearing when one automation is running multiple helm charts together. This is the error that we are getting:
Error: file "/root/.cache/helm/repository/chartA-0.1.0.tgz" does not appear to be a gzipped archive; got "application/octet-stream"

    at ChildProcess.exithandler (child_process.js:281:12)
    at emitTwo (events.js:126:13)
    at ChildProcess.emit (events.js:214:7)
    at maybeClose (internal/child_process.js:915:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:209:5)```

According to mimesniff standard application/x-gzip has a byte pattern of 1F 8B 08, checking that manually:
$ file some.tar.gz
some.tar.gz: gzip compressed data, last modified: Tue Apr 21 08:10:11 2020, from Unix, original size modulo 2^32 1054208
$ xxd some.tar.gz | head -1
00000000: 1f8b 0800 63aa 9e5e 0003 ecbb 0558 945d  ....c..^.....X.]

We can see that byte pattern 1f8b 0800; the file in question was created with tar czvf.
the line here https://github.com/helm/helm/blob/b21b00978539ea8270e6b672bc1e6bc07af61475/pkg/chart/loader/archive.go#L88
is properly checking if the content sniffed is of correct mimetype. According to the documentation for the function here:
// DetectContentType implements the algorithm described
// at https://mimesniff.spec.whatwg.org/ to determine the
// Content-Type of the given data. It considers at most the
// first 512 bytes of data. DetectContentType always returns
// a valid MIME type: if it cannot determine a more specific one, it
// returns "application/octet-stream".

So if the underlying method deems it a generic if it can't determine what it is. The check here
https://github.com/helm/helm/blob/b21b00978539ea8270e6b672bc1e6bc07af61475/pkg/chart/loader/archive.go#L88-L96
checks if the response is of correct mime type, if not it checks the extension and if both fail then its not a yaml file. The downside of this check is that it doesn't check the file itself, only the extension. A quick way of doing it to check the byte pattern, e.g.:
package main

import (
	"fmt"
	"os"
)

func check(e error) {
	if e != nil {
		panic(e)
	}
}

// 1F 8B 08 is the byte pattern for gzipped files
func checktype(buf []byte) bool {
	return buf[0] == 0x1F && buf[1] == 0x8B && buf[2] == 0x8
}

func main() {
	f, err := os.Open("/tmp/some.tar.gz")
	check(err)

	buffer := make([]byte, 3)
	readBytes, err := f.Read(buffer)
	check(err)
	fmt.Println(readBytes, "bytes read")
	fmt.Println("is zip -", checktype(buffer))
	f.Close()
}

running:
$ go run check.go
3  bytes read
is zip - true

That way we can confirm if the contents are actually a gzipped file.

Good detective work @Shagon94. I guess the real question is, why is this intermittent and re-running helps? Sounds like an ordering problem, like the file hasn't been downloaded fully when the check runs.
I thought it might be a problem with the file getting corrupted on download but it is happening too frequently for that I think.

Hello @cablespaghetti , if you can provide any urls that show these symptoms or steps to reproduce I can look into it further, so far everything seems to be working properly.

I'm afraid I've seen only with private s3-based charts so far, using https://github.com/hypnoglow/helm-s3 and when installing multiple instances of the same chart with https://github.com/roboll/helmfile.
My current thinking is that the first call to download the chart begins to download, then the next one starts (in parallel) and because Helm can see a file with the right name in the cache directory it immediately checks the file type of a file which hasn't finished downloading yet.
Thinking aloud, the fix for this may have to be in helmfile but it's odd that it never happened with Helm 2.

FWIW, this is still a problem on helm 3.2.0 (so latest ArgoCD is incompatible) but fixed in 3.2.4. It's an ongoing issue with bitbucket.io sites decompressing ".gz" files. It seems to be related to file extension too. Helm charts with ".tgz" fail, but charts with some random extension like ".gzip" download successfully with helm.

Makes sense given it was merged into helm 3.2.2.

I don't understand this? @bacongobbler did we disable Go's transparent on-the-wire native gzip decompression because of a bug in bitbuket's server implementation?
Someone pasted the link to the problem, and there are tons of reports in the atlassian bug repo over this issue,

https://jira.atlassian.com/browse/BCLOUD-21555?jql=project in (BCLOUD%2C BSERV) AND text ~ "gzip bitbucket" ORDER BY cf[13230] ASC

From my understanding, the bug is that Atlassian decompressed on the server (on upload), and rather than re-compressing when they serve or serving as a different mime type they simply serve the decompressed file as if it was compressed. This causes the error observed above. You can see error explained in BCLOUD-21555
I would not have set DisableCompression to true. The problem is that Atlassian mistakenly marks files they intentionally decompress as compressed.
If I'm right on all, this is probably the core issue facing the Bitnami project's chart truncation. If this patch was reverted, they could have simply enabled compressed on CloudFlare (because CloudFlare like ~100% of the internet except Atlassian does not mistakenly serve decompressed data as compressed). https://github.com/bitnami/charts/issues/10539

@EvanCarroll that makes no sense - if it was actually decompressed server-side, then setting DisableCompression would have no effect. More likely, that issue you linked is the same thing as what's happening here, just that the user's browser decompressed the file during download, rather than Helm.

@pdf Right, and and why doesn't decompression work when the browser or when helm does it? Because it's decompressed server side and the mime-type the sever serves it as marks it as still-compressed. Is my read of this wrong?

@EvanCarroll yes, backwards.

@pdf ah, I see what you're saying. Let me try again! So while I had it backwards, the problem is essentially that a client that request a .tgz file on a bugged webserver will send it down with a mime-type that instructs the client to unzip on the wire. What they should do is sent it with a mime type as described here. The problem is that this patch which disables compression for everything and affects the index file Bitnami uses which is not uploaded as a compressed .tgz but as a plain yaml file. Only the packages are a .tgz. So the assumption that @dene14 said,

With helm we have very specific case - we just need to download binary file (and we know it's binary) from server, then untar it with uncompress option and that's it. We should ignore any transformations in the middle. And in our case that transformation in the middle done by HTTP client library used by helm.

Isn't strictly speaking true. Because doesn't just download the binary file, it downloads the package index/yaml too, which should allow for on-the-wire decompression so CDNs could turn on transparent compression and solve the Bitnami index problem.
Sorry, did I get it right this time. Kind of embarrassing but I'm trying to wrap my head around it.

I'd consider writing a test case to test your hypothesis. repotest is a test server package you can use to write a test case.
See also https://pkg.go.dev/net/http/httptest

I sitll get the error even if the encoding is not present;
curl -I https://helm.repo.com/index.yaml
HTTP/2 200
date: Fri, 19 Apr 2024 05:23:03 GMT
content-type: application/octet-stream
content-length: 1780
last-modified: Fri, 19 Apr 2024 05:15:03 GMT
accept-ranges: bytes
etag: "6621fdd7-6f4"
strict-transport-security: max-age=31536000; includeSubDomains

helm pull --repo https://helm.repo.com/ my-service --version 1.0.0
Error: gzip: invalid header

why?


removed null checks

fixes #5

Hi, we agree that the check is not technically necessary because free() makes that check internally, we prefer to check for that in this case (for various reasons). Also, this file is an exact copy of the MSDN sample and we prefer to keep it the same as that file.


Deployment commit for stage-stable

Merged main branch to stage-stable.
Use latest commit to update namespace ref in app-interface repo. Don't use merge commit, SHAs must be unique when images are created for each branch.

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

:exclamation: No coverage uploaded for pull request base (stage-stable@d3caa86). Click here to learn what that means.

Additional details and impacted files
@@               Coverage Diff               @@
##             stage-stable    #1068   +/-   ##
===============================================
  Coverage                ?   78.30%
===============================================
  Files                   ?       68
  Lines                   ?      673
  Branches                ?      119
===============================================
  Hits                    ?      527
  Misses                  ?      130
  Partials                ?       16

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


Fix spacelift-promex readme

The readme for installing the promex exporter is slightly incorrect. Config values are prefixed with spacelift.

@Apollorion We should also suggest installing prom CRDs, since if you start from scratch, the cluster will not have them.

Can we add a dependency on this chart to install the CRDs without installing the whole prom stack? https://github.com/prometheus-community/helm-charts/tree/main/charts/kube-prometheus-stack/charts/crds

Personally I'm not sure that's a good idea. I think it might make more sense to make deploying the custom resources optional, and to explain in the README that there's a dependency on having a working Prometheus stack installed first. I'd be super wary of installing CRDs from some other project and ending up in a crazy dependency hell.

Also, I agree with @adamconnelly I do call out in the blog post I've written that kube prom stack is a requirement.


Go-manifold verification code update .toml

Updated .toml to the newest release of go-manifold for the change in verification code

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


Can I use my custom .dotfiles with oh-my-zsh

Can I use my custom .dotfiles with oh-my-zsh?
For example https://github.com/sarbbottam/conf-files, can I use it along with oh-my-zsh?

Why not? Rc files exist outside of oh-my-zsh

Any pointer would be helpful. Thanks!

I have oh-my-zsh as a submodule in my dotfile repo: https://github.com/wulfgarpro/dotfiles
In my .zshrc, I init oh-my-zsh like so:
# Path to your oh-my-zsh configuration.
ZSH=$HOME/.oh-my-zsh

#source /etc/environment

DEFAULT_USER="james"

# Set name of the theme to load.
# Look in ~/.oh-my-zsh/themes/
# Optionally, if you set this to "random", it'll load a random theme each
# time that oh-my-zsh is loaded.
ZSH_THEME="robbyrussell"
#ZSH_THEME="agnoster"

# Which plugins would you like to load? (plugins can be found in ~/.oh-my-zsh/plugins/*)
# Custom plugins may be added to ~/.oh-my-zsh/custom/plugins/
# Example format: plugins=(rails git textmate ruby lighthouse)
plugins=(git sudo history-sync)

Thanks! I would give it a shot and update the issue.

Thanks @wulfgarpro it worked like a charm!!


Missing documentation?

I was looking to find if a loader stored an object of the loaded resources. I found examples of Loader.resources used in the PIXI docs, but couldn't find that member in the resource-loader docs. Digging into the source I found it is commented, but not showing up in the built docs?
Loader docs - No mention of member resources
Loader source - Documented  member resources
Maybe that's intentional?

Not intentional, looks like none of the Loader and Resource members variables are showing up at all. Not sure what is causing that.

I'll be revisiting this when v4 is ready for release as it will significantly change how we setup the API docs.

Fixed in v4 rc2


library not found for -lSocketRocket

I installed the cocoaPod and the cordova-plugin-advanced-websocket following the instructions, after I try build the app I got this error:
ld: warning: directory not found for option '-L/Users/Library/Developer/Xcode/DerivedData/WEBRTC-bvcpjkekfmmeqvgsnyxadxpdmjwy/Build/Products/Debug-iphoneos/SocketRocket'
ld: library not found for -lSocketRocket
clang: error: linker command failed with exit code 1 (use -v to see invocation)

SocketRocket is included as CocoaPod library as you can see here in plugin.xml:
https://github.com/HomeControlAS/cordova-plugin-advanced-websocket/blob/a235415d5a149ed832e5e76d6d60effda2ed641c/plugin.xml#L52
When you open the project in xCode, you need to add that CocoaPod to your project as this is not done automatically by cordova/ionic scripts. Why it's not added directly to project but rather to the workspace is beyond me. That's why most cordova plugin developers avoid this issue by copying the whole source of some library to their plugin and don't use CocoaPods at all.


[SMALLFIX]Java 8 Improvement: changed getWorkerCount

replace anonymous type with lambda in alluxio.master.file.AlluxioMasterRestServiceHandler#getWorkerCount

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19260/Failed Tests: 1org.alluxio:alluxio-tests: 1alluxio.zookeeper.ZookeeperFailureIntegrationTest.zkFailure
Test FAILed.


HttpFormPart has data not body

I guess It means data.
I fix body -> data

Thanks, yes, same issue, this was code written for HttpServer.


Ability to set secured query parameters more easily

Right now we need to do:
client.setExtraHeader('X-Algolia-QueryParameters', 'numericFilters=user_id%3d42');

I would like to do:
client.setSecuredQueryParameters({ numericFilters: "user_id=42" });

Be carefully, the order is important:
{ numericFilters: "user_id=42", hitsPerPage: 1 } can be encoded in "numericFilters=user_id%3d42&hitsPerPage=1" or "hitsPerPage=1&numericFilters=user_id%3d42" but only one of them is the valid key for the X-Algolia-QueryParameters signature.

Yes indeed, that's super complex actually :/

Even if we sort the keys: { numericFilters: ["user_id=42", "user_id=43"]} can be encoded in json or string format

Not sure this is a real world issue since we never had a client having it?
Otherwise, we first need to make sure that every API client encodes to JSON in alphabetical order when generating the api key.
Then on the backend side, at query time, reorder the provided secure keys and do the match.
I think ES6 standardized the key order.

If a user set { numericFilters: "(user_id=42,user_id=43)"}, we will need to parse it and convert it that case

We should maybe leave this edge case on the user side.

What will be the order for: ["a", "b", [ "c", "d"]]:  ["a", "b", [ "c", "d"]] or  [[ "c", "d"], "a", "b"] ?

Do not get the question.
Order within a JSON object stringified should really be the same for every engine (we could first do the testing ourselves before trying to fix it).
Order for a user string value inside an object: {numericFilters: "(user_id=42,user_id=43)"} is unchanged, but could be different from the moment the client generated the key. For this case I think it's an edge case.

What will be the order for: ["a", "b", [ "c", "d"]]: ["a", "b", [ "c", "d"]] or [[ "c", "d"], "a", "b"] ?

When using JSON.stringify (any engine), order should the same since arrays are ordered, whereas maps are not (but ordered internally, that's the possible bug)

This seems outdated with the new secured api key we are introducing

@vvo Any link to the new secured api key docs?

@acorncom I am currently implementing it, then we will document its usage, I will update this issue accordingly.


surface: linux 6.1.62 -> 6.1.71

Description of changes
Update surface linux kernel from 6.1.62 to 6.1.71.

It fixes a build fault below:
cpupower> applying patch /nix/store/lf6nr283p1vwm6gi5930sqmkmj614197-source/patches/6.1/0010-surface-shutdown.patch
cpupower> patching file drivers/pci/pci-driver.c
cpupower> patching file drivers/pci/quirks.c
cpupower> Hunk #1 FAILED at 6070.
cpupower> 1 out of 1 hunk FAILED -- saving rejects to file drivers/pci/quirks.c.rej
cpupower> patching file include/linux/pci.h
error: builder for '/nix/store/9j0h3djqqjzpvhinqfz675yk8szrkz5d-cpupower-6.1.62.drv' failed with exit code 1

https://github.com/NixOS/nixos-hardware/commit/249a94e715a3d6cc0217b37b0172dea1edd4afd2 updated linux-surface from arch-6.6.4-1 to arch-6.6.6-1, which also introduced the commit https://github.com/linux-surface/linux-surface/commit/944cabb9a01b3069d1f016d021f0b6b8aa78f69d for linux 6.1.
This commit has a change https://github.com/linux-surface/linux-surface/commit/944cabb9a01b3069d1f016d021f0b6b8aa78f69d#diff-87dba197d613f882797cd43bc87e0ca180ecefbc3c28bac379f1d1d9b4c1ef47 , which updated the patch files' base commit.
https://github.com/torvalds/linux/commit/c9260693aa0c1e029ed23693cfd4d7814eee6624 was introduced in linux 6.1 by https://github.com/torvalds/linux/commit/1c8f75ee92334d89f1ddada26d47f9caa955f1a4 , which is the newer 0010-surface-shutdown.patch based on, but we didn't update linux 6.1 at the same time.
Finally, boom——

What I would like to suggest is that maybe we should update and test more carefully, or an unexpected ripple effect could cause a 6.6 update to break a 6.1 build.
Things done

[ ] Tested the changes in your own NixOS Configuration
[x] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

we were about to replace this kernel version in https://github.com/NixOS/nixos-hardware/pull/831

we were about to replace this kernel version in #831

I agree the replace, and am building https://github.com/NixOS/nixos-hardware/pull/831 on my surface go to test. And I have absolutely no objection if this PR is closed directly.
I just think a bug should be logged and fixed rather than swept under the rug.

merged https://github.com/NixOS/nixos-hardware/pull/831 instead

Hi @Cryolitia thanks for testing on Surface Go

I just think a fault should be logged and fixed rather than swept under the rug.

The problem was logged in https://github.com/NixOS/nixos-hardware/issues/825 and fixed by replacing 6.1 with 6.6 in https://github.com/NixOS/nixos-hardware/pull/831
I hope this works for you and that I didn't miss anything here?


Polyfill imports can't be treeshaken

When we import polyfills they're global.
Since a build system can't know where they're used, it can't make a decision to include it or not.
You can see an example of this here:
https://github.com/alphagov/govuk-frontend/blob/de2b8db0dcc71da3d8be2d0ad2a4f0b5b728dbb6/src/components/checkboxes/checkboxes.js#L1
A potential solution would be to have a function that polyfills return that can be called.
import polyfillBind from '../../vendor/polyfills/Function/prototype/bind'
polyfillBind()

Just realised this is only a problem if using the 'all' entry point. Which currently doesnt allow a destructured import.
So will close for now, may re-open if we decide to allow a destructured import.


Can explanation be created for test data instances?

Can we use x_test in generating local feature importance
exp = Dice(d, m, method="random")
query_instance = x_train[1:2]
e1 = exp.generate_counterfactuals(query_instance, total_CFs=10, desired_range=None,
                                  desired_class="opposite",
                                  permitted_range=None, features_to_vary="all")
e1.visualize_as_dataframe(show_only_changes=True)

closing this issue as no response on this for about 2 weeks. Please re-open another one if you still have questions on this.


feat(DH): Update VPC service with API version 2020-12-15

:tada: This PR is included in version 0.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Remove unecessary fetch order(s) methods

Those methods are not working properly and are not necessary because we can do it easily with requestApi (generic) method.

https://github.com/ecomclub/ecomplus-passport-client/pull/13


Keycloak OIDC endpoint generation doesn't respect radio button

Setup

Rancher version: 2.7.2
Rancher UI Extensions: None
Browser type & version: Chrome 112 Windows 10

Describe the bug

Selecting "Generate" on the radio button in the Endpoints section of the Keycloak OIDC auth provider screen will correctly generate Issuer and Auth Endpoint text boxes from the Keycloak URL text box. If a user leaves text in the Keycloak URL box and changes the radio button to "Specify" the Issuer and Auth Endpoint text fields become enabled but revert any changes until the radio button is returned back to "Generate" and the Keycloak URL text field is cleared.
To Reproduce

Ensure radio button is set to "Generate" (default)
Fill in Keycloak URL text box
Observe that the Issuer and Auth Endpoint fields are correctly generated
Switch radio button to "Specify"
Change text in one of the text fields
Leave focus for the text field and observe that the change is reverted

Result
Text is reverted in Issuer and/or Auth Endpoint.
Expected Result
Custom changes are preserved and not overwritten when the text fields are enabled.
Screenshots

Additional context
This seems to only occur on first setup of this auth provider. I can't reproduce it on the "Edit config" page of an existing provider.

Related: https://github.com/rancher/rancher/issues/38480


Use PIC for (default) static lib

caf wrapper script defaults to static linkage against libcaf_mpi.a but if you're
building a shared lib with the caf script this fails because objects in libcaf_mpi.a
are not compiled with -fPIC. So creating shared libs fails.
The wrapper script should probably default to building and linking against the shared lib

Avg response time
coverage on master

Summary of changes
Always compile position independent code so that the caf wrapper script can build shared libs. Probably, the better solution is to link against the shared lib by default.
Rationale for changes
Allow caf to compile JSON-Fortran
Additional info and certifications
This pull request (PR) is a:

[X] Bug fix
[ ] Feature addition
[ ] Other, Please describe:

I certify that

[X] I reviewed and followed the contributing guidelines, including
- Increasing test coverage for all feature-addition PRs
- Increasing test coverage for all bug-fix PRs for which there
does not already exist a related test that failed before the PR
- At least maintaining test coverage for all other PRs
- Ensuring that all tests pass when run locally
- Naming PR  to indicate work in progress (WIP) and to attach the PR
to the appropriate bug report or feature request issue
- White space (no trailing white space or white space errors may
be introduced)
- Commenting code where it is non-obvious and non-trivial
- Logically atomic, self consistent and coherent commits
- Commit message content
- Waiting 24 hours before self-approving the PR to give another
OpenCoarrays developer a chance to review my proposed code

Fixes #635

LGTM pending CI tests passing.


Creates standard for PreTrained behavior

This changes the standard for DJL behavior with preTrained blocks. As of now, they should also start out with frozen parameters. This has been applied to the embeddings.
It was previously applied only to PyTorch, but as of now applies to all models. However, I did leave a carveout for models. It adds a boolean "wasLoaded" so that if you load a model and then create a Trainer directly from it, it will not be frozen. If you load a model and then append some new layers to it (as we have several examples of), then it will need to be unfrozen to retrain.
fixes #2351

Codecov Report
Base: 72.08% // Head: 74.37% // Increases project coverage by +2.28% :tada:

Coverage data is based on head (f118ce9) compared to base (bb5073f).
Patch coverage: 74.70% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2360      +/-   ##
============================================
+ Coverage     72.08%   74.37%   +2.28%
- Complexity     5126     6817    +1691
============================================
  Files           473      670     +197
  Lines         21970    29599    +7629
  Branches       2351     3073     +722
============================================
+ Hits          15838    22013    +6175
- Misses         4925     6086    +1161
- Partials       1207     1500     +293

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

...rc/main/java/ai/djl/modality/cv/output/Joints.java
71.42% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...main/java/ai/djl/modality/cv/output/Rectangle.java
72.41% <0.00%> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

.../modality/cv/translator/ImageFeatureExtractor.java
0.00% <0.00%> (ø)

.../ai/djl/modality/cv/translator/YoloTranslator.java
27.77% <0.00%> (+18.95%)
:arrow_up:

...ain/java/ai/djl/modality/cv/util/NDImageUtils.java
67.10% <0.00%> (+7.89%)
:arrow_up:

api/src/main/java/ai/djl/modality/nlp/Decoder.java
63.63% <ø> (ø)

... and 616 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


New components (colors not touched)

@MasterLegend2772 Please, review and merge (without homepage). I didnt touch defaults, so colors might not work with our system

need split merge to avoid unfinished homepage in main

should be able to simple merge now

 Page Designs & Overall Design


fix: Add BigInt as supported CursorType

Currently, BigInt was not being detected as a valid cursor if the @id field was of that type and there were no other @unique columns of either Int or String type.
As far as I could test everything seems to work as expected, if there is some reason the exclusion of BigInt as a valid cursor type is intentional, please let me know.

@jrangulod Could you try 1.5.1-beta.1 and tell me if it all works fine for you?

Sure @franky47, I'll be testing this today and the rest of the week and reporting back on issues if any


Is the direct boot example is considered as the First Stage Bootloader for the ESP32-C3?

Hi, I just want a clarification whether this example is the same for first stage bootloader of the ESP32 as mentioned in the API document. If yes, is it possible for the bootloader to be overwritten if I want to flash it into my ESP32C3-Mini-V1 chip? Any guidance would be very helpful.
Thank you in advance

Hi @afifahsyukri, the first stage bootloader of ESP32-C3 (and all other ESP32 series of chips) is in ROM and can not be overwritten. That ROM bootloader is capable of booting applications from Flash. The application loaded from Flash can be: the second stage bootloader, as in IDF; or a standalone application, as in this example.
Hope this helps.

May I know how may I access the first-stage bootloader source code for
ESP32-C3? Any guidance or help would be greatly appreciated.
Regards,
Afifah.
On Fri, Jun 9, 2023 at 7:48 PM Ivan Grokhotkov @.***>
wrote:

Closed #11
https://github.com/espressif/esp32c3-direct-boot-example/issues/11 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/espressif/esp32c3-direct-boot-example/issues/11#event-9483574504,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4L2MQFIM5TI5D2DSVKQGYTXKMESHANCNFSM6AAAAAAYRBBBQA
.
You are receiving this because you were mentioned.Message ID:
<espressif/esp32c3-direct-boot-example/issue/11/issue_event/9483574504@
github.com>

ROM bootloader code of the existing chips is not open source, I'm afraid. We do provide ELF files for debugging, in case that helps: https://github.com/espressif/esp-rom-elfs/releases


feat: stategraph parser

While developing in LangGraph it can be helpful for the developer to visualize the
flowchart representing the logic flow of their StateGraph.This parser reads a python
file and creates an abstract syntax tree (AST) of the StateGraph. The AST is then
translated into Mermaid charting language and saved as a PNG.
NOTE: I have never committed to an open-source project
This PR is related to a conversation I had with @hwchase17

hi @stugorf, thanks for this, we've recently added support for mermaid drawing in langchain-core, see examples here https://github.com/langchain-ai/langgraph/pull/311 so closing this one, thanks again

Thank you!
On Tue, Apr 16, 2024 at 1:50 PM Nuno Campos @.***>
wrote:

hi @stugorf https://github.com/stugorf, thanks for this, we've recently
added support for mermaid drawing in langchain-core, see examples here
#311 https://github.com/langchain-ai/langgraph/pull/311 so closing this
one, thanks again
—
Reply to this email directly, view it on GitHub
https://github.com/langchain-ai/langgraph/pull/74#issuecomment-2059901388,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKDS6MJSWP7K43TSHSGID3Y5WFIPAVCNFSM6AAAAABCSMLHLCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJZHEYDCMZYHA
.
You are receiving this because you were mentioned.Message ID:
@.***>