Can't install or search for packages from winget source, only msstore appearing

Hi there,
I am facing an issue where in user context I can only find packages from the msstore source
whereas using system context I can find both.
Any ideas on how to troubleshoot fix it ?

I have the same issue .
The cdn.winget.microsoft.com refused to connect

Can confirm I am having the same issue:
`2024-09-01 08:31:47.097 [FAIL] WindowsPackageManager.dll!00007FFB8DEF56D2: ReturnHr(1) tid(3b94) 80190193 Forbidden (403).
Msg:[D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\HttpStream\HttpClientWrapper.cpp(50)\WindowsPackageManager.dll!00007FFB8DEE481E: (caller: 00007FFB8DDF24F9) Exception(1) tid(3b94) 80190193 Forbidden (403).
]
2024-09-01 08:31:47.097 [FAIL] WindowsPackageManager.dll!00007FFB8DDA55A7: LogHr(1) tid(3b94) 80190193 Forbidden (403).
2024-09-01 08:31:47.097 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\RepositorySource.cpp(53)\WindowsPackageManager.dll!00007FFB8DF002B1: (caller: 00007FFB8DDFBC7C) LogHr(2) tid(3b94) 80190193 Forbidden (403).
Msg:[winrt::hresult_error: Forbidden (403).]
2024-09-01 08:31:47.097 [REPO] Source add/update failed, waiting a bit and retrying: winget
2024-09-01 08:31:49.132 [FAIL] WindowsPackageManager.dll!00007FFB8DEF56D2: ReturnHr(2) tid(3b94) 80190193 Forbidden (403).
Msg:[D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\HttpStream\HttpClientWrapper.cpp(50)\WindowsPackageManager.dll!00007FFB8DEE481E: (caller: 00007FFB8DDF24F9) Exception(2) tid(3b94) 80190193 Forbidden (403).
]
2024-09-01 08:31:49.132 [FAIL] WindowsPackageManager.dll!00007FFB8DDA55A7: LogHr(3) tid(3b94) 80190193 Forbidden (403).
2024-09-01 08:31:49.132 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\RepositorySource.cpp(634)\WindowsPackageManager.dll!00007FFB8DEFFF6B: (caller: 00007FFB8DD4473E) LogHr(4) tid(3b94) 80190193 Forbidden (403).
Msg:[winrt::hresult_error: Forbidden (403).]
2024-09-01 08:31:49.132 [REPO] Failed to update source: winget
2024-09-01 08:31:49.371 [CLI ] Leaf command succeeded: root:source:update`

Can confirm I am having the same issue. I can see in my network capture that TCP and TLS connectivity are fine. Some sort of cdn endpoint out there for the winget source is not functioning. If I try another machine in a different region, it's working.
`2024-09-01 08:31:47.097 [FAIL] WindowsPackageManager.dll!00007FFB8DEF56D2: ReturnHr(1) tid(3b94) 80190193 Forbidden (403). Msg:[D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\HttpStream\HttpClientWrapper.cpp(50)\WindowsPackageManager.dll!00007FFB8DEE481E: (caller: 00007FFB8DDF24F9) Exception(1) tid(3b94) 80190193 Forbidden (403). ]
2024-09-01 08:31:47.097 [FAIL] WindowsPackageManager.dll!00007FFB8DDA55A7: LogHr(1) tid(3b94) 80190193 Forbidden (403).
2024-09-01 08:31:47.097 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\RepositorySource.cpp(53)\WindowsPackageManager.dll!00007FFB8DF002B1: (caller: 00007FFB8DDFBC7C) LogHr(2) tid(3b94) 80190193 Forbidden (403). Msg:[winrt::hresult_error: Forbidden (403).]
2024-09-01 08:31:47.097 [REPO] Source add/update failed, waiting a bit and retrying: winget 2024-09-01 08:31:49.132 [FAIL] WindowsPackageManager.dll!00007FFB8DEF56D2: ReturnHr(2) tid(3b94) 80190193 Forbidden (403). Msg:[D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\HttpStream\HttpClientWrapper.cpp(50)\WindowsPackageManager.dll!00007FFB8DEE481E: (caller: 00007FFB8DDF24F9) Exception(2) tid(3b94) 80190193 Forbidden (403). ]
2024-09-01 08:31:49.132 [FAIL] WindowsPackageManager.dll!00007FFB8DDA55A7: LogHr(3) tid(3b94) 80190193 Forbidden (403).
2024-09-01 08:31:49.132 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\RepositorySource.cpp(634)\WindowsPackageManager.dll!00007FFB8DEFFF6B: (caller: 00007FFB8DD4473E) LogHr(4) tid(3b94) 80190193 Forbidden (403). Msg:[winrt::hresult_error: Forbidden (403).]
2024-09-01 08:31:49.132 [REPO] Failed to update source: winget 2024-09-01 08:31:49.371 [CLI ] Leaf command succeeded: root:source:update`

No longer occuring now. No idea what that was.

I managed to "fix" the issue by granting the user admin rights, restarting the machine and running winget source reset --force.
Is there any way to reset the source for the user in system context?

In some cases winget source update -n winget could fix the issue.


[LIVY-535] Fix non-atomic session creation

What changes were proposed in this pull request?
All steps, that include check for too many sessions and creating new
session, should be atomic operation to avoid concurrent access of
several threads to different part of this code in the same time.
https://issues.apache.org/jira/browse/LIVY-535
How was this patch tested?
If was tested manually on local virtual cluster with this different values of config option livy.server.session.max-creation: 1, 2 and 3
Then I run the following command to make sure that only allowed number of sessions were created in the same time and others were rejected:
for i in {1..5}; do curl -X POST --data '{"file": "/tmp/spark-examples-2.jar", "className": "org.apache.spark.examples.SparkPi"}' -H "Content-Type: application/json" http://localhost:8999/batches; done

Codecov Report

Merging #129 into master will increase coverage by 0.11%.
The diff coverage is 87.5%.

@@             Coverage Diff              @@
##             master     #129      +/-   ##
============================================
+ Coverage     70.98%   71.09%   +0.11%
- Complexity      925      927       +2
============================================
  Files           100      100
  Lines          5511     5512       +1
  Branches        829      830       +1
============================================
+ Hits           3912     3919       +7
+ Misses         1058     1053       -5
+ Partials        541      540       -1

Impacted Files
Coverage Δ
Complexity Δ

.../scala/org/apache/livy/server/SessionServlet.scala
67.07% <87.5%> (+0.4%)
16 <0> (ø)
:arrow_down:

...cala/org/apache/livy/scalaapi/ScalaJobHandle.scala
52.94% <0%> (-2.95%)
7% <0%> (ø)

...c/main/scala/org/apache/livy/repl/ReplDriver.scala
30.76% <0%> (-2.57%)
7% <0%> (ø)

...ain/scala/org/apache/livy/utils/SparkYarnApp.scala
78.87% <0%> (-0.71%)
33% <0%> (ø)

.../scala/org/apache/livy/sessions/SessionState.scala
61.11% <0%> (ø)
2% <0%> (ø)
:arrow_down:

...in/java/org/apache/livy/rsc/driver/JobWrapper.java
80% <0%> (ø)
8% <0%> (+1%)
:arrow_up:

...ain/java/org/apache/livy/rsc/driver/RSCDriver.java
79.23% <0%> (+1.27%)
42% <0%> (+1%)
:arrow_up:

...c/src/main/java/org/apache/livy/rsc/RSCClient.java
83.85% <0%> (+3.72%)
26% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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

Took a look at the surrounding code and this seems fine. I was worried that createSession might be blocking, but it's not.
Merging to master.

The code shouldn't be spending that much time in the synchronized block. It does do quite a lot of things, including spawning a child process, but it doesn't wait for that process to finish or anything like that. So it should be reasonably ok.
Otherwise this fix would be way more complicated. The accounting for how many things are starting is all over the place. If you want to spend time fixing that instead...

@Jassy1994 I tend to agree with @vanzin - it was a simple fix for a bug in current implementation. Also, I kept in mind that this code won't take too much time, so the waiting time should be quite acceptable. Please, let me know if it's not so in your case.


chore: updates diagram generation tests

updates diagram generation tests. silences many of the other snapshot tests for next PR
Here are the diagrams generations we are testing:

Added more test cases, too:


Add a link to documentation for setting ms-ds-consistencyguid via Powershell

It would be very helpful if this page included a link to the documentation on how to set the attribute via Powershell.
Section in question: "Permission required" (https://docs.microsoft.com/en-us/azure/active-directory/hybrid/plan-connect-design-concepts#permission-required)
Link to the documentation for the powershell cmdlet: https://docs.microsoft.com/en-us/azure/active-directory/hybrid/how-to-connect-configure-ad-ds-connector-account#configure-ms-ds-consistency-guid-permissions
It took some time to find this page without the link and it would make sense to explain not only what permission needs to be set, but also how to set it.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 4d02a10e-bd9c-a622-e341-17be6c539013
Version Independent ID: 86e2c83c-0778-5f54-2286-e69861259855
Content: Azure AD Connect: Design concepts - Microsoft Entra
Content Source: articles/active-directory/hybrid/plan-connect-design-concepts.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@JonathanBrockhausen Thanks for your feedback. We will investigate and update the thread.

@JonathanBrockhausen Thanks for your suggestion. There is a link provided in recommended content to Configure AD DS
connector Account Permissions. However, I have pushed the changes to update the document to configure permissions link as well for easy access. Changes will be reflected in document after author's approval.


Backend Web App "uvicorn" logger not working

Discussed in https://github.com/microsoft/PubSec-Info-Assistant/discussions/818

Originally posted by supplemarty August 15, 2024
The backend (and enrichment) web apps use the "uvicorn" logger as seen here:
log = logging.getLogger("uvicorn") log.setLevel('DEBUG') log.propagate = True
When I use this log variable in the backend code for debug tracing like:
log.debug("HELLO WORLD")
those log entries do not show up in the log stream NOR application insights but in the enrichment app this same pattern seems to be working fine.
I setup my own logger using the StreamHandler, and it works as I would expect. I cannot see where uvicorn is getting its logging dict configuration in the azure web app configuration. Thoughts?

Python apps run on Azure App Service for Linux use a gunicorn WSGI by default.
https://learn.microsoft.com/en-us/azure/app-service/configure-language-python
By default the logging and debugging arguments are not set in the terraform
https://github.com/microsoft/PubSec-Info-Assistant/blob/f29513590d8eaa20f1666a21e572456ab50864a0/infra/main.tf#L90
You can specify more commands in your app service where the gunicorn command is set and restart it. If you want to log them to a file you should use the access log command
https://docs.gunicorn.org/en/20.x/settings.html#accesslog
To enable debugging at a log level, you should specify it using the log level command.
https://docs.gunicorn.org/en/20.x/settings.html#accesslog
You also have the option to override this, if you wish to use uvicorn directly instead.
https://learn.microsoft.com/en-us/azure/developer/python/configure-python-web-app-on-app-service#other-frameworks-and-web-servers

Sorry if I am missing something but hhy does the uvicorn logger work for the enrichment app but not the backend web app?

Is there a specific error or statement you are not seeing that you can point to?
The backend makes use of asynchronous generators, which means logging when it happens must be proceeded by a yield statement to actually output to the log if it's inside an asynchronous generator.
I also noticed the compare and directGPT approaches do not have a logging config set, which may be a reason if it is specific to logging those approaches as a global logging level is not specified with logging.basicConfig
Logging is set at several different layers of the application, narrowing it down to a specific file or function would be helpful in debugging this.

In the backend app.py you will see the log = logging.getLogger("uvicorn") as I pointed out above. The code was written to setLevel("DEBUG"). Any calls to log.debug("HELLO WORLD") fail to show any output in the log stream for the backend web app nor app insights. I used the out of the box deployment code.

We have added this to the backlog and assigned an engineer to review.


cpp-utilities: 5.10.2 -> 5.10.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/Martchus/cpp-utilities/releases.
meta.description for cpp-utilities is: "Common C++ classes and routines used by @Martchus' applications featuring argument parser, IO and conversion utilities"
meta.homepage for cpp-utilities is: "https://github.com/Martchus/cpp-utilities"
meta.changelog for cpp-utilities is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 5.10.3 with grep in /nix/store/jpq7bf4ycvssnrg4m5za53ln24xg4a7x-cpp-utilities-5.10.3

found 5.10.3 in filename of file in /nix/store/jpq7bf4ycvssnrg4m5za53ln24xg4a7x-cpp-utilities-5.10.3

directory tree listing: https://gist.github.com/d15a19909f01017fa561013485701e8e

du listing: https://gist.github.com/7a944c773475efab0c8291ed4ca8dbff

Rebuild report (if merged into master) (click to expand)

28 total rebuild path(s)

10 package rebuild(s)

10 x86_64-linux rebuild(s)
9 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
9 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
cpp-utilities
libsForQt5.qtutilities
libsForQt512.qtutilities
libsForQt514.qtutilities
libsForQt515.qtutilities
plasma5Packages.qtutilities
syncthingtray
syncthingtray-minimal
tageditor
tagparser

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/jpq7bf4ycvssnrg4m5za53ln24xg4a7x-cpp-utilities-5.10.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cpp-utilities https://github.com/r-ryantm/nixpkgs/archive/a542ffb1243763dba0fb60eb91e1940f5cdaf84e.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/jpq7bf4ycvssnrg4m5za53ln24xg4a7x-cpp-utilities-5.10.3
ls -la /nix/store/jpq7bf4ycvssnrg4m5za53ln24xg4a7x-cpp-utilities-5.10.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

8 packages built:

cpp-utilities
libsForQt5.qtutilities
libsForQt512.qtutilities
libsForQt514.qtutilities
syncthingtray
syncthingtray-minimal
tageditor
tagparser

Maintainer pings
cc @doronbehar for testing.

Result of nixpkgs-review pr 122804 at a542ffb1 run on aarch64-linux 1

7 packages built successfully:

cpp-utilities
libsForQt5.qtutilities (libsForQt515.qtutilities ,plasma5Packages.qtutilities)
libsForQt512.qtutilities
libsForQt514.qtutilities
syncthingtray
syncthingtray-minimal
tageditor

3 suggestions:

warning: unclear-gpl
gpl2 is a deprecated license, please check if project uses gpl2Plus or gpl2Only and change meta.license accordingly.
Near pkgs/development/libraries/cpp-utilities/default.nix:29:5:
   |
29 |     license = licenses.gpl2;
   |     ^

error: no-flags-array
checkFlagsArray is only intended to be used in Bash, not as a Nix attribute.
Near pkgs/development/libraries/cpp-utilities/default.nix:23:3:
   |
23 |   checkFlagsArray = [ "LD_LIBRARY_PATH=$(PWD)" ];
   |   ^

warning: unused-argument
Unused argument: fetchpatch.
Near pkgs/development/libraries/cpp-utilities/default.nix:4:3:
  |
4 | , fetchpatch
  |   ^


Hotfix/split into parts

Some files restructured
Some functions refactored or removed.
Package based movements is done.

I think, there are still things to change, especially, refactoring on store module would be nice for second step.
I am proposing changes right now to develop-a branch, then I will try to keep changing some stuffs. But for now, I can be merged.

I suggest you to look at the changes commit by commit, instead of reviewing them all in one. @gianmarcomennecozzi


Crash when using an invalid regular expression

Issue for discussion thread https://groups.google.com/d/topic/elm-discuss/7Uwl5-usqjs/discussion
Basically, JavaScript's RegExp (which is what Elm's Regex sits on top of) can throw an exception when given an invalid string.  Examples:
RegExp("a"); // OK
RegExp("\\"); // Exception!
RegExp("("); // Exception!
RegExp("(?"); // Exception!
RegExp("(?:"); // Exception!
RegExp("?"); // Exception!
RegExp("+"); // Exception!
RegExp("*"); // Exception!

The obvious fix is to make Elm's Regex.regex return a Result (or a Maybe), but that seems like a pain to work with.  It's also an API-breaking change, which I have no problems with whatsoever, but others might.
The only other solution I can think of is to escape characters at the end of an input string until no exception is thrown.  For example:
RegExp("(?:"); // Exception!
RegExp("(?\\:"); // Exception!
RegExp("(\\?\\:"); // Exception!
RegExp("\\(\\?\\:"); // OK

It's a bit hacky though, and may not follow the principle of least surprise.  Thoughts?

In my opinion the API-breaking change (with Result or Maybe) is the better solution.

+1 for API-breaking changes. The best thing about Elm is that it's opinionated.

Duplicate of #378, which has further discussion.

Okay, closing this here as a duplicate.


[Run] To decide and remove the hardcoded pattern and maxCount values

https://github.com/microsoft/Launcher/blob/ca916deda91181ce335374b21e7f195e77861e59/src/modules/launcher/Plugins/Wox.Plugin.Indexer/SearchHelper/WindowsSearchAPI.cs#L91
Right now we have hardcoded values for the parameter and the maxSearchResults.

To decide how we want to support regex based search and allow that pattern to be passed.
To decide on the maximum number of results that we want to populate.

@alekhyareddy28 is this still relevant with all the updating we've done here?

Right now we populate a maximum of 30 results from the indexer plugin. If we want to modify that/make it customizable then yes there is work involved here. If we're good with the fixed value of 30 then it's fine to go ahead and close this.

@alekhyareddy28 is this still an issue?

/needinfo

She was one of the first devs. This is in us to verify still

The link in the post need an update :D

that was cause it was an issue we moved over, contextually this still holds.


flutter doctor reports fine in terminal but fails in VSCode (Run from command palette)

Run 'flutter doctor' from VSCode command palette:
[flutter] flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, v0.5.1, on Mac OS X 10.13.6 17G65, locale en-CN)
[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
[!] iOS toolchain - develop for iOS devices (Xcode 9.4.1)
    ✗ Missing Xcode dependency: Python module "six".
      Install via 'pip install six' or 'sudo easy_install six'.
[✓] Android Studio (version 3.1)
[✓] IntelliJ IDEA Community Edition (version 2017.2.7)
[!] VS Code (version 1.25.1)
[✓] Connected devices (1 available)

! Doctor found issues in 2 categories.
exit code 0

Run flutter doctor -v in terminal
[✓] Flutter (Channel beta, v0.5.1, on Mac OS X 10.13.6 17G65, locale en-CN)
    • Flutter version 0.5.1 at /Users/wolfg/Dev/lib/flutter
    • Framework revision c7ea3ca377 (8 weeks ago), 2018-05-29 21:07:33 +0200
    • Engine revision 1ed25ca7b7
    • Dart version 2.0.0-dev.58.0.flutter-f981f09760

[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at /usr/local/share/android-sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-27, build-tools 27.0.3
    • ANDROID_HOME = /usr/local/share/android-sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 9.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.4.1, Build version 9F2000
    • ios-deploy 1.9.2
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 26.0.1
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)

[✓] IntelliJ IDEA Community Edition (version 2017.2.7)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 21.2.2
    • Dart plugin version 172.4343.25

[!] VS Code (version 1.25.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected devices (1 available)
    • iPhone X • 73056EF7-1637-4DE2-91C6-60449C983145 • ios • iOS 11.4 (simulator)

! Doctor found issues in 1 category.

Try to run Flutter: Run Flutter Upgrade from the command palette to sync with your installation

cc @DanTup

My guess is that your PATH or environment variables are different inside VS Code compared to your other terminal (this can happen if you have terminal profile scripts that run when you open a terminal, for example, which are not run by VS Code).
Try running printenv both in the VS Code terminal and your normal terminal and compare for differences.

@a-atalla
Ran Flutter: Run Flutter Upgrade
The result is
[flutter] flutter upgrade
Upgrading Flutter from /Users/wolfg/Dev/lib/flutter...
From https://github.com/flutter/flutter
   834fb7b9a..56800d8fa  master     -> origin/master
Already up-to-date.

Upgrading engine...
Already up-to-date.

Flutter 0.5.1 • channel beta • https://github.com/flutter/flutter.git
Framework • revision c7ea3ca377 (8 weeks ago) • 2018-05-29 21:07:33 +0200
Engine • revision 1ed25ca7b7
Tools • Dart 2.0.0-dev.58.0.flutter-f981f09760

Running flutter doctor...
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, v0.5.1, on Mac OS X 10.13.6 17G65, locale en-CN)
[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
[!] iOS toolchain - develop for iOS devices (Xcode 9.4.1)
    ✗ Missing Xcode dependency: Python module "six".
      Install via 'pip install six' or 'sudo easy_install six'.
[✓] Android Studio (version 3.1)
[✓] IntelliJ IDEA Community Edition (version 2017.2.7)
[!] VS Code (version 1.25.1)
[!] Connected devices
    ! No devices available

! Doctor found issues in 3 categories.
exit code 0

@danrubel
flutter doctor reports ok both in system terminal and  VSCode terminal. How to check the environment used by VSCode?
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, v0.5.1, on Mac OS X 10.13.6 17G65, locale en-CN)
[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
[✓] iOS toolchain - develop for iOS devices (Xcode 9.4.1)
[✓] Android Studio (version 3.1)
[✓] IntelliJ IDEA Community Edition (version 2017.2.7)
[!] VS Code (version 1.25.1)
[!] Connected devices
    ! No devices available

! Doctor found issues in 2 categories.

Run printenv both in the VS Code terminal and in your normal terminal and compare for differences.

PATH in VS Code terminal, it has duplicate entries:
PATH=/usr/local/share/flutter/bin:/Users/wolfg/Library/Python/2.7/bin:/Users/wolfg/.yarn/bin:/Users/wolfg/.config/yarn/global/node_modules/.bin:/Users/wolfg/.gem/bin:/usr/local/share/android-sdk/platform-tools:/usr/local/share/android-sdk/tools:/Users/wolfg/.rbenv/shims:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin:/Applications/Wireshark.app/Contents/MacOS:/usr/local/share/flutter/bin:/Users/wolfg/Library/Python/2.7/bin:/Users/wolfg/.yarn/bin:/Users/wolfg/.config/yarn/global/node_modules/.bin:/Users/wolfg/.gem/bin:/usr/local/share/android-sdk/platform-tools:/usr/local/share/android-sdk/tools:/Users/wolfg/.rbenv/shims:/Users/wolfg/.nvm/v8.9.4/bin

PATH in normal terminal:
PATH=/usr/local/share/flutter/bin:/Users/wolfg/Library/Python/2.7/bin:/Users/wolfg/.yarn/bin:/Users/wolfg/.config/yarn/global/node_modules/.bin:/Users/wolfg/.gem/bin:/usr/local/share/android-sdk/platform-tools:/usr/local/share/android-sdk/tools:/Users/wolfg/.rbenv/shims:/Users/wolfg/.nvm/v8.9.4/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin:/Applications/Wireshark.app/Contents/MacOS

No other significant differences in the results of printenv

@wolfg1969 Sorry, I think I have misunderstood! Just to clarify:

Running from Terminal is good
Running from VS Code's terminal is good
Running from Command Palette is bad

Is that right?
If so, can you try enabling the general extension log file for Dart Code, then restart VS Code and grab a copy. In that file it should list your PATH as we had it. I think the VS Code terminal runs some of your terminal scripts, so it's possible it differs there.
If that still turns out to be the same, I'll send you a debug build of Dart Code that also logz the full environment as it is when you run flutter doctor from the palette so we can compare that.

@danrubel Yes, only running from Command Palette is bad.
I found the reason after enabling the extension log: another extension prepend a virtualenv python path. Thank you!


Github authentication error when submitting new license

The following response was sent when submitting a new license with this text:
Unexpected error while posting to github, please e…turned 401 status code - message Bad credentials\n

Below is the full text of the error:
Unexpected error while posting to github, please email the SPDX technical workgroup that the following error has occurred: Traceback (most recent call last):
  File "/spdx/src/app/views.py", line 1477, in post_to_github
    raise Exception("Post to Github returned {0} status code - message {1}".format(statusCode, jsonResponse["message"]))
Exception: Post to Github returned 401 status code - message Bad credentials

I recreated the authentication token which fixed this issue, but I now ran into the following error:
Unexpected error, please email the SPDX technical workgroup that the following error has occurred: Traceback (most recent call last): File "/spdx/src/app/views.py", line 1174, in issue submissionDatetime=now, userEmail=userEmail, notes=licenseNotes, xml=xml) NameError: name 'userEmail' is not defined

Email was removed from the model during the recent updates to the license submittal form which caused this issue.


how to convert pretrianed caffe2 R2Plus1D models to pytorch?

How can i convert the pretrained caffe2 R2Plus1D models listed [here] (https://github.com/facebookresearch/R2Plus1D/blob/master/tutorials/models.md) into a pytorch model for fine-tuning on a custom dataset? I will be happy to contribute the models back once i succeed.

Could it be possible to according to the architecture of pytorch model and caffe2 model, adjust the parameters of each layer and save?

@cdeepakroy yes, you can because models now are just sets of pickle blobs.

@cdeepakroy Hi, Have you successed in converting caffe2 model into pytorch?

How can i convert the pretrained caffe2 R2Plus1D models listed [here] (https://github.com/facebookresearch/R2Plus1D/blob/master/tutorials/models.md) into a pytorch model for fine-tuning on a custom dataset? I will be happy to contribute the models back once i succeed.

Hi, I also want to know how to  convert the pretrained caffe2 R2Plus1D models.  Have you successed？

Converting the models to pytorch is relatively straightforward but painful process (needs to be done by hand). You can find the sample model definitions here and you'd need to copy all the weights by hand.
Cheers

@bjuncek Thank you for your converted models Bruno!
But is that just really copy from the .pkl file and assign to pytorch's statedict? I don't see the weights of your converted r2plus1d_18-91a641e6.pth equal to weights of r2.5d_d18_l16.pkl from pre-trained models and somehow has 2 millions less Params.
Am I missing something? Please tell me I'm trying to convert the batch-norm C3D model. The same definition does not give me flatten 4608 but instead 8192 in_features at fc1.

So, you'd need to assign the correct weight to correct layer.
For R3D, that would look something like:
def get_blob(weights, name):
    return torch.from_numpy(weights['blobs'][name])

model.conv1.weight.data = get_blob(weights, "conv1_w")
model.bn1.weight.data = get_blob(weights, 'conv1_spatbn_relu_s')
model.bn1.running_mean.data = get_blob(weights, 'conv1_spatbn_relu_rm')
model.bn1.running_var.data = get_blob(weights,  'conv1_spatbn_relu_riv')
model.bn1.bias.data = get_blob(weights, 'conv1_spatbn_relu_b')

for i in range(0,2):
    layer_counter = 0
    model.layer1[i % 2].conv1.weight.data = get_blob(weights,'comp_{}_conv_1_w'.format(layer_counter+i))
    model.layer1[i % 2].bn1.weight.data = get_blob(weights, 'comp_{}_spatbn_1_s'.format(layer_counter+i))
    model.layer1[i % 2].bn1.bias.data = get_blob(weights, 'comp_{}_spatbn_1_b'.format(layer_counter+i))

...
...

The pre-trained models referenced in the PR (and subsequent refactoring which makes them more readable and accurate) were actually trained from scratch to match the performance of the models here, so the weights wouldn't be the same.

Hey folks we just did that for the r(2+1)d 34-layer model here:
https://github.com/moabitcoin/ig65m-pytorch
Hope that helps.

Thanks @daniel-j-h. Much appreciated!

ps, @cdeepakroy if you found this solution satisfactory, feel free to close the issue.
Many thanks!!!

merge to https://github.com/facebookresearch/VMZ/issues/86


Suggestion: moving from NetCDF4 to xarray

Having data available in NetCDF4 format is incredibly useful for both cross-compatibility with much of climate/weather analysis but also for the tools built around it. However, a lot of this analysis is more easily (and more commonly) done in xarray (docs, which builds on netCDF4 + numpy + pandas) these days rather than using the netCDF4 package itself + numpy.
I could see a benefit to the user for the default output of api.get_data(...use_opendap=True) to return an xarray.Dataset rather than a netCDF4 object.
Without changing anything else, the easiest thing to do would be to use xarray's built-in converter:
import xarray as xr
nc = api.get_data(station_ids=["tplm2",'44025'], modes=["stdmet"], start_time="2019-06-01", end_time="2024-06-01", use_opendap=True)
ds = xr.open_dataset(xr.backends.NetCDF4DataStore(nc))

But the real benefit would be something that transfers some of the metadata from the stations info into the xarray object (and therefore, to any netcdf file saved from it). Something like this:
ds = stdmet_df.reset_index().set_index(['station_id','timestamp']).to_xarray()

metas = []
for sid in ds.station_id.values:
    # Get station metadata
    smeta = api.station(sid)

    # Create empty Dataset with dimension of station_id
    ds_meta = xr.Dataset(coords = {'station_id':[sid]})

    for attr in smeta:
        if attr == 'Location':
            for dim,dirc in zip(['lat','lon'],['NS','EW']):
                # Find lat / lon in string
                dim_set = re.search(r'[0-9]*\.[0-9]*\ ['+dirc+']',smeta['Location']).group(0)
                # Turn into float (multiplied by +1 / -1 depending on whether it's N/E or S/W)
                dim_set = float(re.search(r'[0-9]*\.[0-9]*',dim_set).group(0))*(1 if dirc[0] in dim_set else -1)
                # Add to dataset
                ds_meta[dim] = (['station_id'],[dim_set])
        elif re.search(r'^[0-9]+\.{0,1}[0-9]*',smeta[attr]) is not None:
            # Assume everythign of the form '##.## lorem ipsum' is split
            # into a value and units
            # Get units as the bit after the numebr
            units = re.split(r'^[0-9]+\.{0,1}[0-9]*\ ',smeta[attr])[-1]
            # Get the value
            value = float(re.search(r'^[0-9]+\.{0,1}[0-9]*',smeta[attr]).group(0))
            ds_meta[re.sub(r'\ ','_',attr).lower()] = (['station_id'],[value])
            ds_meta[re.sub(r'\ ','_',attr).lower()].attrs['units'] = units
        else:
            # If doesn't fit either paradigm, just copy in as string
            ds_meta[attr] = (['station_id'],[smeta[attr]])

    metas.append(ds_meta)

# Concatenate across station ids
metas = xr.concat(metas,dim='station_id')

# Merge with original dataset, make coordinates
ds = xr.merge([ds,metas]).set_coords([k for k in metas])
print(ds)

Then it can either be saved as a netcdf using ds.to_netcdf(fn) or further analyzed using xarray's tools.
This is totally a suggestion and I clearly got a little sidetracked with it, so feel free to take it or leave it (but I think this will make a big difference in terms of usability in workflows using array data).
This is related to JOSS review https://github.com/openjournals/joss-reviews/issues/7406.

Thank you for the suggestion @ks905383 this looks excellent. At a minimum, the switch over to xarray as a wrapper for the netCDF4 datasets is worth implementing. I initially attempted this directly through xarray, but was unable to load the data from URL without the extra steps of reading it to a local temp file. In terms of including the station metadata, the snippet you posted speaks for itself. I'll work on implementing this, possibly with an include_metadata flag or similar. I'll also try to make sure the behavior maps well to xarray.concat.
The latter might take some time to implement, but the first will be included in the next release.

Thank you again for these recommendations! The changes are included in #52 and the release to PyPi and Conda is from #53.


Main web page has garbage paragraphs on the bottom

http://mikesofaer.github.com/stop_censorship/  has 4 paragraphs of garbage text on the bottom of the page.  I'm assuming that they were NOT intended to be there???

Hi Dan,
That text is sample text so that you can get a better sense of how a page of text looks censored.  We should at least make that clear.
Thanks for mentioning it!

Ahhhhh.... yeah, I had no clue what those paragraphs were about and thought perhaps the site had been hacked in some way (although I couldn't think of how off the top of my head).
Yes, putting some explanatory text there would be good.  This issue can obviously be closed.

Replaced the sample text, hope you like it better :-)

Belatedly I'll just say thanks!  The new sample text is awesome! :-)


Header: Nav Header

Header: Nav Header
Implement Nav Header in the Design System Storybook.

Functional Tissue Unit Explorer
Cell Distance Explorer

Behavior

Scrolls with page
Apps icon opens #557
No max width
Two width based variants:

+1920px to 1013px: Uses larger logo buttons, generally
1012px to 320px: Uses smaller logo buttons

If other navigation categories are present, the HuBMAP apps button switches to a menu button, which opens a sidenav for all navigation categories. The HuBMAP apps menu items are contained at the bottom of the sidenav.

@LibbyUX
Please review this here
The apps button wont work now, Edward is currently working on the menu


Requests for inverters with multiple inputs and DC/DC optimizers

https://sam.nrel.gov/node/70436
https://sam.nrel.gov/node/74234
https://sam.nrel.gov/node/74779
https://sam.nrel.gov/node/75182

Introduced inverters with up to four MPPT inputs in SAM 2018.11.11.


Merge rustc changes back into the miri repo

This uses https://github.com/josh-project/josh (RIIR ftw)
The first time it looks a bit odd with the merges and so, all future syncs will work just like the instructions say. I'll add instructions for the sync from miri to rustc, too, starting to do that right now.
Initial cache building now takes 5 mins 🥳

@bors r+

:pushpin: Commit 94775c4e52d6d38ef44313094643123239bdfdad has been approved by oli-obk
It is now in the queue for this repository.

I don't quite recognize our use-case in what Josh advertises itself for but if it works :shrug:
I'm not super happy about having to run a docker container for the sync, let's hope the infra team automates this soon :)

I'll add instructions for the sync from miri to rustc, too, starting to do that right now.

FWIW I did that earlier with git-subtree while testing https://github.com/rust-lang/miri/pull/2564 and it worked very well.

I'll add instructions for the sync from miri to rustc, too, starting to do that right now.

FWIW I did that earlier with git-subtree while testing #2564 and it worked very well.

yea, but I wanna make sure that josh and git-subtree work fine, and we don't make rustc-to-miri syncs harder due to git subtree in the other direction being weird (I don't think it is, but need to try it out)

Looks like bors is on strike again so I guess that gives us time to at least fix the typo I noted. ;)
@bors r-

I don't quite recognize our use-case in what Josh advertises itself for but if it works 🤷
I'm not super happy about having to run a docker container for the sync, let's hope the infra team automates this soon :)

that's just me being lazy and following the instructions. I belive a cargo install plus a short shell script may suffice, too

@bors r+

:pushpin: Commit 6deaa61a784a74ab1b3e6b863e3ec8a0c02ea17d has been approved by oli-obk
It is now in the queue for this repository.

@bors ping

:sleepy: I'm awake I'm awake

@bors r-

@oli-obk bors seems to work again... why did you r- this?

:pushpin: Commit 6deaa61a784a74ab1b3e6b863e3ec8a0c02ea17d has been approved by oli-obk
It is now in the queue for this repository.

The other PR was merged and this one didn't start

@bors r+

:bulb: This pull request was already approved, no need to approve it again.

:pushpin: Commit 6deaa61a784a74ab1b3e6b863e3ec8a0c02ea17d has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 6deaa61a784a74ab1b3e6b863e3ec8a0c02ea17d with merge 6d4ef3186087df5f8988b95ae377162acb85ad8f...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 6d4ef3186087df5f8988b95ae377162acb85ad8f to master...


fixes for #27 and fix of broker's get_wallet_balance() for live trading

Hi @Dave-Vallance
I hope the following helps. It is fixing two things at once.
First issue I had with #27 which renders bt-ccxt-store unusable on some exchanges as given params are not accepted on all exchanges. See https://github.com/ccxt/ccxt/wiki/Manual#custom-order-params. I mitigated it by trying the API call once. If unsuccessful it is not carried out in future to save API calls while still having the feature of @obiben.
The other thing is get_wallet_balance() in the broker which fails for me on a Bitfinex paper trading account if the amount of a currency is zero. eg. if all TESTUSD have been exchanged for TESTBTC, this fails.
Cheers

Hi @Dave-Vallance
In the meantime I have corrected and added more patches to my master, like 40819af, being unrelated. Please let me know if and for which fix I should create new PRs for you to make it easier.
Cheers

Thanks - looks good after review.


Investigate situation for Null/Never in inference, specifically in literals

may be related to https://github.com/dart-lang/sdk/issues/38120
This code currently gives errors:
if ((() => true)()) {}

because the inferred return type is Null, which is not assignable to bool.
This is partly an open issue regarding literal inference. And partly should be changed so that in the meantime it returns Never.

Why should it return Never? Did you mean throw something instead of true?

The title of the issue is quite wide, but the example is narrow.
And it works for me.
Is there a more wide example you are thinking about, @MichaelRFairhurst ?

@bwilkerson and @pq ran into something with this earlier...I probably summarized it into a repro that didn't actually capture it.
I thought we had not correctly implemented function return type inference, and that it always inferred a return type of Null for function literals.
Maybe the real case is this?
() { return true; }

We need to make sure that we're handling this correctly, and correctly may be that it should return dynamic, or Never, or bool.

OK, we might have an issue here.
main(a) {
  var f1 = () {
    return true;
  };

  var f2 = () {
    return a;
  };

  var f3 = () {
    throw 42;
  };
}

The type of f1 is bool Function().
The type of f2 is dynamic Function().
The type of f3 is Null Function().
The type of f3 looks wrong, it probably should be Never Function(), but I don't know where this would be written in the spec, so that I could confirm this. @leafpetersen, could you help?

@scheglov I added this to my TODO list, but won't get to this today.
This relies on the reachability analysis that is part of definite assignment/promotion.
In case it's helpful here's my intention:
A path from the entry to a function does not complete if it terminates in a throw statement or an expression statement of type Never
A path from the entry to a function returns normally if it terminates in a return statement.
So we have:

A function body for which all paths do not complete has inferred return type Never
A function body for which all paths either do not complete, or return normally, has the inferred return type given by the upper bound of the types of all of the returned expressions
A function body for which at least one path neither fails to complete nor returns normally (in other words, it falls of the end of the function body) has the inferred return type given by the upper bound of Null and the types of all of the returned expressions.

I believe the inference implementation was updated to do what the spec requires.
Specifically, we can infer Never.
https://dart-review.googlesource.com/c/sdk/+/149605


(refactor) project settings update

updated to match Xcode recommended settings. There were some target version bumps, so if you want me to revert those, just lemme know.
Signed-off-by: Michael Redig hybrids-07-baggage at icloud.com
Pull Request Checklist

[x] Pull request is based on the develop branch
[x] Pull request contains a changelog file in ./changelog.d. See https://github.com/matrix-org/matrix-ios-sdk/blob/develop/CONTRIBUTING.md#changelog
[x] Pull request includes a sign off

Codecov Report

Merging #1543 (f62d688) into develop (56a23ee) will decrease coverage by 30.26%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           develop    #1543       +/-   ##
============================================
- Coverage    42.10%   11.83%   -30.27%
============================================
  Files          515      515
  Lines        84113    84113
  Branches     37064    35901     -1163
============================================
- Hits         35418     9957    -25461
- Misses       47663    73785    +26122
+ Partials      1032      371      -661

Impacted Files
Coverage Δ

MatrixSDKTests/MatrixSDKTestsData.h
0.00% <0.00%> (-100.00%)
:arrow_down:

MatrixSDK/Crypto/Data/MXDeviceList.h
0.00% <0.00%> (-100.00%)
:arrow_down:

MatrixSDK/Crypto/Data/MXOlmSession.m
0.00% <0.00%> (-100.00%)
:arrow_down:

MatrixSDK/Data/MXPeekingRoomSummary.m
0.00% <0.00%> (-100.00%)
:arrow_down:

MatrixSDK/Crypto/Utils/MXCryptoTools.m
0.00% <0.00%> (-100.00%)
:arrow_down:

MatrixSDKTests/MXPollBuilderTests.swift
0.00% <0.00%> (-100.00%)
:arrow_down:

MatrixSDK/Space/MXSpaceChildrenResponse.m
0.00% <0.00%> (-100.00%)
:arrow_down:

MatrixSDK/Crypto/Data/MXOlmSessionResult.m
0.00% <0.00%> (-100.00%)
:arrow_down:

MatrixSDK/Crypto/Data/MXExportedOlmDevice.m
0.00% <0.00%> (-100.00%)
:arrow_down:

MatrixSDK/JSONModels/Sync/Room/MXRoomSync.m
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 238 more

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

targets reverted :)

Thanks @mredig


Upgrade package to work with dbt-utils >v1.0.0

As said in this issue https://github.com/Divergent-Insights/dbt-dataquality/issues/13, this PR is created to fix the dbt-utils dependency issues for dbt-utils version >v1.0.0. This will allow everyone who uses the latest version of dbt-utils to use your great dbt package.
Some background on the changes made to the dbt_utils.surrogate_key macro in version 1.0.0: https://docs.getdbt.com/guides/migration/versions/upgrading-to-dbt-utils-v1.0#changes-to-surrogate_key
Let me know if there's anything more I can do to help!

@wilson-urdaneta @wilson-vaquen Could you have look at this PR?


Delete plugin to do a clean install

I want to do a clean install of the plugin because I appear to have messed something up. I do not see a delete script or a delete flag in hack/build.sh. Thanks!

If you're trying to delete a released version of the plugin, you just need to delete the binary, something like which kn-quickstart to find the binary and then rm /path/to/kn-quickstart to delete it. If you used brew to install, then you'd need to use their uninstall comamnd.
If you're trying to build from source, the binaries are created in the main project directory, so you should be able to remove them with something like rm /path/to/kn-plugin-quickstart/kn-quickstart*
If you're trying to do something else, then let us know what that something else is :smile:


sortby with ascending=False should create an index

Is your feature request related to a problem?
When using sortby with ascending=False on a DataArray/Dataset without an explicit index, the data gets correctly reversed, but it is not possible to tell anymore which ordering the data has.
If an explicit index (like [0, 1, 2]) exists, it gets correctly reordered and allowes correct aligning.
Describe the solution you'd like
For consistency with aligning xarray should create a new index that indicates that the data has been reordered, i.e. [2, 1, 0].
Only downside: this will break code that relies on non-existent indexes.
Describe alternatives you've considered
No response
Additional context
No response

Would we then have a different schema for different values of ascending?
If that's correct I would vote against this; I empathize with the confusion, but then the caller can add an index, and generally we don't want different values of arguments to create different schema without a very obvious and explicit connection between them.
Tbh even if the proposal is to add the index for any value of ascending, I'd be marginally against; it's not what I'd expect as a user. But open-minded on this.
(Thanks for adding the issue @headtr1ck , hope this comes across as constructive)

I guess my understanding of indexes was: "no index" == "default index ~range(0,n)".
Therefore, if no index exists and ascending=True, no index in the output is correct.
But then again I agree that it is easy for users to add an index.

I guess my understanding of indexes was: "no index" == "default index ~range(0,n)".

Yeah — that is the case in pandas. But in xarray, we can have dims without indexes, so adding one is making a change...

Ok, then I'm fine with the current state.


Porting Scaleout sample from C# to JavaScript

Porting Scaleout sample from C# to JavaScript

@gandiddi There are some unresolved comments.

See inline comments. Also, not at all obivous, but there is a README in the samples folder that has a table of samples for each language. Likely need to update that table to include this.

Updated the table in README for JS scaleout sample


Comments on user guide

DO NOT MERGE. Marker edits for discussion items.
I've made a pass through the User Guide and only have a few small comments. I'll add more comments to some edits.

What's the path forward for this PR? Is #207 addressing some of the same issues?

What's the path forward for this PR? Is #207 addressing some of the same issues?

I tried to incorporate everything from this PR in #207. Let me know if I missed anything.

Ok I'm gonna proactively close this, but Werner, feel welcome to re-raise any things that were missed.


White Paper Describing the Model Approach and Accuracy on Benchmark Dataset

Will it possible for you to add a description the model you're using? Especially which cost function you used for multi-label classification. If possible, please also provide some accuracy measure on some standard evaluation dataset so that the user will be more confident to use it. Thanks!

Hey @YipingNUS, this is the paper that describes the architecture of the model we're using.
You can also find another application of this model here, where you can find some concrete results and numbers on a large-corpus.
Hope that helps!

Hi @jstypka, thanks for your reply. The second paper is especially helpful since I've read the first classic paper a few times already ;) I read in the following link that Keras' binary_crossentropy (which is used in your code) is kind of flawed.
https://github.com/keras-team/keras/issues/5335
Besides I'm a bit worried whether the label correlation, as well as the imbalanced data size for each label, will have an effect of the label ranking. Have you tried to compare with some other approaches of multi-label classification such as the following paper:
Zhu, Shenghuo, et al. "Multi-labelled classification using maximum entropy method." Proceedings of the 28th annual international ACM SIGIR conference on Research and development in information retrieval. ACM, 2005.

Interesting thoughts!
I don't know of any attempts for comparing it with other methods, I would be very interested in learning about that as well. And for the two problems that you mentioned, I have an unvalidated intuition that they are indeed present.
For the label correlation - I do not consider it a problem to be honest ("it's not a bug, it's a feature"). A model that can exploit label cooccurrences is much more powerful than a set of independent binary classifiers. For the argument of imbalanced data size - that is true and it's problematic. I'd be happy to hear your suggestions on how to mitigate that if you have any ideas.
Cheers!

I find the idea of this paper quite interesting. Although it's not for multi-label classification, the model is almost exactly the same. Section 2.3 talks about reducing the open space risk and learning different thresholds for each class. My intuition is it might help to alleviate the label imbalance problem as well.


Speeding up recoverstate

recoverstate was unusably slow with 700 000+ files.
Now skipping loop over filelist when read state offset == 0.

Looks simple and straight forward.  This belongs in the libdali project, which, unfortunately, is still private.  But I'll get it integrated upstream and pushed into miniseed2dmc, leaving this open until that's done.
Thanks for the optimization, much appreciated!


How to correctly modify a planner object

Hello! I want to implement an adapive exploration constant for POMCP as a function of the uncertainty over different states, as described in this paper. For that, I intend to do the following:

As the different entropy-based exploration constant parameters depend on the observation model, they can be calculated outside of the planner
There is one value per state, so it can be pased to the POMCP object's exploration_const as a list instead of a single value
In the ucb function, modify the function call to take the state being simulated by the simulation, and index _exploration_const by the index corresponding to the state

Would it be a way of implementing this without modifying the source code? I would prefer to not have to modify the library if possible. Also, I don't know if you would consider this interesting for a pull request, in which case I'd gladly discuss how to implement it in the library.

Hi, in this case, I recommend forking the repo and develop the feature as a subclass of POMCP or POUCT in a separate file, instead of directly changing the source code for them. If you'd like, feel free to open a PR then. It is unclear to me how broadly useful this is and how someone else could use this for their problem. If you make a PR, would be great to include one or more domains (in pomdp_problems) to illustrate its appeal. Thank you for considering contributing!

Thank you very much! Would this new feature and subclass need to follow the cython syntax? If yes, I would like to know if you could point me to some documentation / guidelines since I've never worked with Cython personally.

Ideally yes, but you should be able to inherit and implement your subclasses in Python. I mostly learned Cython through reading their documentation or other websites. Also, let's not worry about adding this into pomdp-py for now. I hope you can focus on solving your problem.

Sounds good to me! Thank you for your help. I'm closing this for the moment. Will re-open if there is anything else to discuss.

I started to implement a class that inherits from POMCP as you suggested:
class POMCP2(POMCP):
    """
    Implements some methods on top of the POMCP implementation.

    Attributes
    ----------

    _exploration_const_list: list or None, default=None
        List of exploration constants to be used for each depth of the tree.
        If None, all methods use the original POMCP implementation
    """

    def __init__(self,
                 max_depth=5, planning_time=-1., num_sims=-1,
                 discount_factor=0.9, exploration_const=math.sqrt(2),
                 num_visits_init=0, value_init=0,
                 rollout_policy=None, action_prior=None,
                 show_progress=False, pbar_update_interval=5,
                 exploration_const_list=None):

        super().__init__(max_depth=max_depth,
                         planning_time=planning_time,
                         num_sims=num_sims,
                         discount_factor=discount_factor,
                         exploration_const=exploration_const,
                         num_visits_init=num_visits_init,
                         value_init=value_init,
                         rollout_policy=rollout_policy,
                         action_prior=action_prior,
                         show_progress=show_progress,
                         pbar_update_interval=pbar_update_interval)

        self._exploration_const_list = exploration_const_list

    def _simulate(self,
                  state, history, root, parent,
                  observation, depth):
        if self._exploration_const_list is not None:
            breakpoint()
            if depth > self._max_depth:
                return 0
            if root is None:
                if self._agent.tree is None:
                    root = self._VNode(agent=self._agent, root=True)
                    self._agent.tree = root
                    if self._agent.tree.history != self._agent.history:
                        raise ValueError("Unable to plan for the given history.")
                else:
                    root = self._VNode()
                if parent is not None:
                    parent[observation] = root
                self._expand_vnode(root, history, state=state)
                rollout_reward = self._rollout(state, history, root, depth)
                return rollout_reward
            action = self._ucb(root, state)
            next_state, observation, reward, nsteps = sample_generative_model(self._agent, state, action)

            if nsteps == 0:
                # This indicates the provided action didn't lead to transition
                # Perhaps the action is not allowed to be performed for the given state
                # (for example, the state is not in the initiation set of the option,
                # or the state is a terminal state)
                return reward

            total_reward = reward + (self._discount_factor**nsteps)*self._simulate(next_state,
                                                                                history + ((action, observation),),
                                                                                root[action][observation],
                                                                                root[action],
                                                                                observation,
                                                                                depth+nsteps)
            root.num_visits += 1
            root[action].num_visits += 1
            root[action].value = root[action].value + (total_reward - root[action].value) / (root[action].num_visits)
            return total_reward

            # POMCP belief update
            if depth == 1 and root is not None:
                root.belief.add(state)
        else:
            return super()._simulate(self, state, history, root, parent, observation, depth)

It is very much identical to POUCT's _simulate() method, with the only difference being that the function call for self._ucb includes state. However, I am getting the following error:
Traceback (most recent call last):
  File "ssvep_exp.py", line 491, in <module>
    exp.main()
  File "/home/jtorretr/gitrepos/phd_pomdp/bci_pomdp/experiment.py", line 1186, in main
    action = policy.plan(bci_problem.agent)
  File "pomdp_py/algorithms/pomcp.pyx", line 94, in pomdp_py.algorithms.pomcp.POMCP.plan
  File "pomdp_py/algorithms/po_uct.pyx", line 233, in pomdp_py.algorithms.po_uct.POUCT.plan
  File "pomdp_py/algorithms/po_uct.pyx", line 306, in pomdp_py.algorithms.po_uct.POUCT._search
  File "pomdp_py/algorithms/pomcp.pyx", line 128, in pomdp_py.algorithms.pomcp.POMCP._simulate
  File "/home/jtorretr/gitrepos/phd_pomdp/bci_pomdp/planner/POMCP2.py", line 52, in _simulate
    if depth > self._max_depth:
AttributeError: 'POMCP2' object has no attribute '_max_depth'

But the attribute should be correctly assigned from the different __init__() methods. Any idea why this is happening?

This happens because _max_depth is a private member of POUCT.  My suggestion for now is, fork the pomdp-py repo, add property getters for private members you need (similar to this), and then build your class by inheritance.

Thank you for your answer. I did what you suggested but the error persists... here is my code for POUCT on the fork:
cdef class POUCT(Planner):

    """ POUCT (Partially Observable UCT) :cite:`silver2010monte` is presented in the POMCP
    paper as an extension of the UCT algorithm to partially-observable domains
    that combines MCTS and UCB1 for action selection.

    POUCT only works for problems with action space that can be enumerated.

    __init__(self,
             max_depth=5, planning_time=1., num_sims=-1,
             discount_factor=0.9, exploration_const=math.sqrt(2),
             num_visits_init=1, value_init=0,
             rollout_policy=RandomRollout(),
             action_prior=None, show_progress=False, pbar_update_interval=5)

    Args:
        max_depth (int): Depth of the MCTS tree. Default: 5.
        planning_time (float), amount of time given to each planning step (seconds). Default: -1.
            if negative, then planning terminates when number of simulations `num_sims` reached.
            If both `num_sims` and `planning_time` are negative, then the planner will run for 1 second.
        num_sims (int): Number of simulations for each planning step. If negative,
            then will terminate when planning_time is reached.
            If both `num_sims` and `planning_time` are negative, then the planner will run for 1 second.
        rollout_policy (RolloutPolicy): rollout policy. Default: RandomRollout.
        action_prior (ActionPrior): a prior over preferred actions given state and history.
        show_progress (bool): True if print a progress bar for simulations.
        pbar_update_interval (int): The number of simulations to run after each update of the progress bar,
            Only useful if show_progress is True; You can set this parameter even if your stopping criteria
            is time.
    """

    def __init__(self,
                 max_depth=5, planning_time=-1., num_sims=-1,
                 discount_factor=0.9, exploration_const=math.sqrt(2),
                 num_visits_init=0, value_init=0,
                 rollout_policy=RandomRollout(),
                 action_prior=None, show_progress=False, pbar_update_interval=5):
        self._max_depth = max_depth
        self._planning_time = planning_time
        self._num_sims = num_sims
        if self._num_sims < 0 and self._planning_time < 0:
            self._planning_time = 1.
        self._num_visits_init = num_visits_init
        self._value_init = value_init
        self._rollout_policy = rollout_policy
        self._discount_factor = discount_factor
        self._exploration_const = exploration_const
        self._action_prior = action_prior

        self._show_progress = show_progress
        self._pbar_update_interval = pbar_update_interval

        # to simplify function calls; plan only for one agent at a time
        self._agent = None
        self._last_num_sims = -1
        self._last_planning_time = -1

    @property
    def updates_agent_belief(self):
        return False

    @property
    def last_num_sims(self):
        """Returns the number of simulations ran for the last `plan` call."""
        return self._last_num_sims

    @property
    def last_planning_time(self):
        """Returns the amount of time (seconds) ran for the last `plan` call."""
        return self._last_planning_time

    @property
    def max_depth(self):
        """Returns the maximum depth of the MCTS tree."""
        return self._max_depth

    @property
    def agent(self):
        """Returns the agent that the planner is planning for."""
        return self._agent

I also checked whether self.max_depth existed (without the underscore), but no. The attribute cannot be accessed.

After you modify the Cython files, you need to rebuild. Cython is a compiled language. You can run make build to rebuild (see installation as developer).

Closing this now. Feel free to reopen if comes up again.


Install Jenkins plugins using password failed

TASK [geerlingguy.jenkins : Install Jenkins plugins using password.] ***********
failed: [ec2-52-213-150-14.eu-west-1.compute.amazonaws.com] (item=git-parameter) => {"changed": true, "cmd": ["java", "-jar", "/opt/jenkins-cli.jar", "-s", "http://localhost:8080/", "install-plugin", "git-parameter", "--username", "admin", "--password", "admin"], "delta": "0:00:07.937499", "end": "2016-12-25 18:32:26.503943", "failed": true, "item": "git-parameter", "rc": 255, "start": "2016-12-25 18:32:18.566444", "stderr": "Dec 25, 2016 6:32:26 PM hudson.remoting.SynchronousCommandTransport$ReaderThread run\nSEVERE: I/O error in channel Chunked connection to http://localhost:8080/cli\njava.io.IOException: Unexpected termination of the channel\n\tat hudson.remoting.SynchronousCommandTransport$ReaderThread.run(SynchronousCommandTransport.java:73)\nCaused by: java.io.EOFException\n\tat java.io.ObjectInputStream$PeekInputStream.readFully(ObjectInputStream.java:2353)\n\tat java.io.ObjectInputStream$BlockDataInputStream.readShort(ObjectInputStream.java:2822)\n\tat java.io.ObjectInputStream.readStreamHeader(ObjectInputStream.java:804)\n\tat java.io.ObjectInputStream.(ObjectInputStream.java:301)\n\tat hudson.remoting.ObjectInputStreamEx.(ObjectInputStreamEx.java:48)\n\tat hudson.remoting.AbstractSynchronousByteArrayCommandTransport.read(AbstractSynchronousByteArrayCommandTransport.java:34)\n\tat hudson.remoting.SynchronousCommandTransport$ReaderThread.run(SynchronousCommandTransport.java:59)\n\nhudson.remoting.RequestAbortedException: java.io.IOException: Unexpected termination of the channel\n\tat hudson.remoting.Request.abort(Request.java:307)\n\tat hudson.remoting.Channel.terminate(Channel.java:888)\n\tat hudson.remoting.SynchronousCommandTransport$ReaderThread.run(SynchronousCommandTransport.java:92)\n\tat ......remote call to Chunked connection to http://localhost:8080/cli(Native Method)\n\tat hudson.remoting.Channel.attachCallSiteStackTrace(Channel.java:1537)\n\tat hudson.remoting.Request.call(Request.java:172)\n\tat hudson.remoting.Channel.call(Channel.java:821)\n\tat hudson.remoting.RemoteInvocationHandler.invoke(RemoteInvocationHandler.java:256)\n\tat com.sun.proxy.$Proxy3.main(Unknown Source)\n\tat hudson.cli.CLI.execute(CLI.java:333)\n\tat hudson.cli.CLI._main(CLI.java:504)\n\tat hudson.cli.CLI.main(CLI.java:385)\nCaused by: java.io.IOException: Unexpected termination of the channel\n\tat hudson.remoting.SynchronousCommandTransport$ReaderThread.run(SynchronousCommandTransport.java:73)\nCaused by: java.io.EOFException\n\tat java.io.ObjectInputStream$PeekInputStream.readFully(ObjectInputStream.java:2353)\n\tat java.io.ObjectInputStream$BlockDataInputStream.readShort(ObjectInputStream.java:2822)\n\tat java.io.ObjectInputStream.readStreamHeader(ObjectInputStream.java:804)\n\tat java.io.ObjectInputStream.(ObjectInputStream.java:301)\n\tat hudson.remoting.ObjectInputStreamEx.(ObjectInputStreamEx.java:48)\n\tat hudson.remoting.AbstractSynchronousByteArrayCommandTransport.read(AbstractSynchronousByteArrayCommandTransport.java:34)\n\tat hudson.remoting.SynchronousCommandTransport$ReaderThread.run(SynchronousCommandTransport.java:59)", "stdout": "Installing git-parameter from update center", "stdout_lines": ["Installing git-parameter from update center"], "warnings": []}

works with no errors when  jenkins_plugin module used after the role run completed

I see the same issue using a vagrant-box bento/ubuntu-16.04. When I browse to the server, I get the page "Unlock Jenkins" for copying the initial administrator password.
How have you moved the jenkins_plugin module behind the jenkins role?

I have created a wrapper playbook which is depended on this one. My customizations are done in the wrapper playbook, after the dependency is fulfilled.

As far as I can see that happens to plugins with yet unresolved dependencies. The plugin is being installed with dependencies resolved as expected though. So the error happens after that. I tried both stable (2.32.2) and weekly (2.50) builds.
You can list all dependencies in jenkins_plugins as a workaround, though it's quite a pain.
Some issues that seem related:
https://issues.jenkins-ci.org/browse/JENKINS-23232
https://issues.jenkins-ci.org/browse/JENKINS-40056
http://stackoverflow.com/questions/36929484/jenkins-severe-i-o-error-in-channel-chunked-connection

I found a much better workaround with Ansible jenkins_plugin module, which works like a charm!
https://docs.ansible.com/ansible/jenkins_plugin_module.html

@dizeee - Ah, I'd not seen that module before. It might be good to simply use it in this role too!

@geerlingguy I'm playing around with it right now and I could make a pull request in a day or two. Do you want me to keep the current code for backward compatibility with Ansible < 2.2?

@dizeee - I'd rather just make a clean break and bump the meta/main.yml minimum version to 2.2. This change would require the role to go through a major version bump, but that's fine by me. Simplicity/maintainability > legacy support.

@geerlingguy Perfect!

I'm testing @dizeee's PR now, will hopefully be able to merge it shortly.


Convert fairseq transformer nmt to nvidia triton inference server?

❓ Questions and Help

Convert fairseq transformer nmt to nvidia triton inference server? is this possible to achieve. I wish to load and serve my fairseq transformer based model to triotn inference server
https://github.com/triton-inference-server/server

I don't believe anyone has tried.  Feel free to report back with your experience.


Экспорт дашбоардов в PDF и печать

добрый день, была бы полезна функциональность печати дашбоардов и экспорта как pdf

I've implemented PDF reports generation based on DataLens dashboards using Puppeteer. There are several major obstacles in this approach.

One should pass page width for each tab so that all elements fit into PDF page. Page height is available via .react-grid-layout.
DataLens loads widgets lazily, hence one needs to set viewport size equal to the page size so that all elements are loaded simultaneously. But in this way, a single tab with many widgets may load longer than default navigation timeout of 30 seconds, so one needs to adjust this timeout. Plus one may get errors from data source because it may not allow too many simultaneous connections.
.dash-tabs element breaks PDF page width in case of a dashboard with multiple tabs. So it should be removed from the page before executing page.pdf() (although in my opinion horizontal tabs list is pointless in case of PDF reports, so that's a good thing that they should be removed).

It is also possible to remove some buttons from web page before rendering it as PDF.


Datashader object does not update in Holoviews after update to 0.6.1

Dear datashade community,
I wrote some functions to visualize large time series dataframes using holoviews and datashader with bokeh extension enabled. Here are the main elements of the function to give you an idea of how I implemented datashader:
import holoviews as hv
from holoviews.operation.datashader import datashade

point = hv.Scatter(df)
shade = datashade(point, cmap = colormap).opts(plot = dict_ds)
layout = hv.Layout(shade).opts(plot = dict_lo).cols(1)
return layout

Basically, the these are the core elements you use in your example notebooks.
For adjusting plot options, I am using dicts with .opts and not cell magic as this turned out to be a pain when transferring the code from jupyter to python scripts as need when creating convinient plotting modules.
Today, I updated datashader from 0.6 to 0.6.1 which resulted in crashs of my plot functions (did not save the output, sorry...). Therefore, I made sure that I had holoviews in the most recent version, which it was... Thereafter, I updated bokeh from 0.12.7 to 0.12.9. The result was that the function works again but creates interactive plots with a single static datashade rendered image. A zoom in does not cause an update in the rendering...

My question, did you guys change something in the way the server updates the data when the plot gets interacted with? How can I make use of the great datashader module again?
Thx for your help!!!

There were some changes in bokeh that caused this problem. We'll try to get a hotfix release of holoviews out later today or tomorrow. I do worry about your initial issue though:

I updated datashader from 0.6 to 0.6.1 which resulted in crashs of my plot functions (did not save the output, sorry...).

Do you have some way to reproduce this?

Thx for the quick reply! Great to hear that holoviews is getting an hotfix!!!
As fix, I downgraded my modules (bokeh > 0.12.7 + datashader >0.6.0) and the plotting function worked again.
To reproduce the error, I updated only datashader 0.6.1 again:

The function is still working... So, I am not able to reproduce the issue. Maybe the kernel was in a bad state last time. Thanks anyway

HoloViews 1.8.4 with a fix for this issue was released last week and you can get it with:
conda install -c ioam holoviews


[BUG][LINUX] Deleting character result in deletion of 2 characters.

Deleting text from a textbox will result in deletion of 2 characters at once.
After some reading on flutter issues this seems to also happens on upstream but, it seems that it is fixed, but for some reason i still experience the problem even on the fixed version of flutter.
Hover doctor

Even though the hover doctor output below is using the master channel, during testing dev and beta channel also shows similar result

$ hover doctor
hover: Hover version (devel) running on linux
hover: Sharing packaging tools
hover: darwin-bundle is supported
hover: darwin-dmg is supported
hover: To package darwin-pkg these tools are required: mkbom,xar
hover: Install bomutils from your package manager or from https://github.com/hogliux/bomutils
hover: Install xar from your package manager or from https://github.com/mackyle/xar
hover: To still package darwin-pkg without the required tools installed you need to run hover with the `--docker` flag.
hover: To package linux-appimage these tools are required: appimagetool
hover: Install appimagetool from your package manager or from https://github.com/AppImage/AppImageKit#appimagetool-usage
hover: To still package linux-appimage without the required tools installed you need to run hover with the `--docker` flag.
hover: linux-deb is supported
hover: To package linux-pkg these tools are required: makepkg
hover: You need to be on Arch Linux or another distro that uses pacman as package manager to use this. Installing makepkg on other distros is hard and dangerous.
hover: To still package linux-pkg without the required tools installed you need to run hover with the `--docker` flag.
hover: To package linux-rpm these tools are required: rpmbuild
hover: You need to be on Red Hat Linux or another distro that uses rpm as package manager to use this. Installing rpmbuild on other distros is hard and dangerous.
hover: To still package linux-rpm without the required tools installed you need to run hover with the `--docker` flag.
hover: To package linux-snap these tools are required: snapcraft
hover: Install snapd from your package manager or from https://snapcraft.io/docs/installing-snapd
hover: To still package linux-snap without the required tools installed you need to run hover with the `--docker` flag.
hover: To package windows-msi these tools are required: wixl
hover: Install msitools from your package manager or from https://wiki.gnome.org/msitools/
hover: To still package windows-msi without the required tools installed you need to run hover with the `--docker` flag.
hover:
hover: Sharing flutter version
Flutter 1.26.0-2.0.pre.418 • channel master •
https://github.com/flutter/flutter.git
Framework • revision 024c49da85 (2 hours ago) • 2021-01-19 19:28:13 -0800
Engine • revision b58dbc88fc
Tools • Dart 2.12.0 (build 2.12.0-240.0.dev)
hover: Flutter engine commit: https://github.com/flutter/engine/commit/b58dbc88fce92bce9d63ee482133072edf5ede6e
hover: ⚠ The go-flutter project tries to stay compatible with the beta channel of Flutter.
hover: ⚠     It's advised to use the beta channel: `flutter channel beta`
hover: Finding out the C compiler version
gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
Copyright (C) 2017 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

hover: Sharing the content of go.mod
module github.com/raveltan/diamondv2-desktop/go

go 1.15

require (
	github.com/go-flutter-desktop/go-flutter v0.42.0
	github.com/pkg/errors v0.9.1
)
hover: Sharing the content of hover.yaml
application-name: ""
executable-name: ""
package-name: ""
license: ""
target: lib/main_desktop.dart
branch: ""
cache-path: ""
opengl: ""
engine-version: ""
hover: Sharing the content of go/cmd
go/cmd/main.go	go/cmd/options.go

Error output
There are no error only warning below:
go-flutter: no method handler registered for method 'routeUpdated' on channel 'flutter/navigation'

@pchampio How to update?

This was fixed in the latest version of go-flutter.
Please update both go-flutter.
Please update flutter the the latest beta version.


Have we properly attributed and lisenced our sample?

{
  "name": "oidc-client-js",
  "version": "0.1.0",
  "description": "This is a sample of using the oidc-client-js with OpenIddict.",
  "main": "server.js",
  "scripts": {
    "start": "node server.js"
  },
  "repository": {
    "type": "git",
    "url": "git+https://github.com/openiddict/openiddict-samples.git"
  },
  "author": "Shaun Luttin, Brock Allen & Dominick Baier",
  "license": "Apache-2.0",
  "bugs": {
    "url": "https://github.com/openiddict/openiddict-samples/issues"
  },
  "homepage": "https://github.com/openiddict/openiddict-samples#readme"
}

Not sure. What is this?

It's a project.json from a sample of using oidc-client-js. What I am wondering, is

whether to include you and Dominick's names in the author property, and
whether to use the same license that oidc-client-js used (Apache-2.0).

You should be using the npm package, so I don't think you need to make us the author of your sample. As for the license in your sample, it's up to you.

Thank you sir. BTW... great library. I spent 60 hours rolling my own before encountering yours. While I still might complete mine, it's great to have yours as an existing solution, especially for some of the tougher parts like nonce, state, and id_token validation.


#Suggestion: Chat history #Nicetohavenotimportant

I know I have closed the FAF client many a time when I had a PM waiting for me, that I never got to read. So my "nice to have" feature request is that the client remembers the chat in the open windows. And open them again when you relaunch the client. I guess this need to be togglable if it is made.
And also remember the last Aeolus history aswell.
Bonus if one can also save all chat history locally if one choose. I know eve online does this for all chats and it's quite nice if you wanna look on convos you have had way back.
Eve does it by saving it in txt in a chat folder, And naming it something like Downlord-PrivateChat-Date.txt. Or Aeolus-Chat-Date.txt
Date would here probably be a timestamp of when you closed the client. (or the chat window)
Please don't take this as anything other than something I think would be nice to have. And would that I think would be an improvement on the client. And IMO a killer feature over the old klient ^^

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

I was just thinking about how to best implement both features.
Have it write to a file in a chat history folder.
named by this something like this; datetime-name-private.txt
If you manually close a private chat it writes a unique line that says so.
Every time you open the client it finds all Aeolus chats made the past day and adds that in the client.
And it also search for all private chats that has been logged within the day that does not have the chat closed line at the bottom.
I guess it could also work with just 1 file for each Person / chatroom. just that it reads the logged time for each message to find ut how far back to collect the chat.(and again stopping at the line for closed chat) That way you can easly see a long log of a convo you have had with a friend without looking throgh several logs. Maybe even have it toggable how far back in search history it goes.
And have it named after UID instead of name, so even if someone changes name it keeps the log in one place.
This is probably either an obvious solution or an obviously stupid solution for a experienced developer :P
Mad rambelings

I'm not sure whether archiving general chat (i.e. #aeolus) makes sense, because it would indicate that you can see the whole history of this channel, which is not true. You would only archive chat messages that where send while you were online, the rest would be missing.
For private messages I see 3 different working modes:

Show all archived messages
Show all unread messages (optional extension: reopen these tabs on startup)
Don't show archived messages

I think we should close this. I dont think it would help. You can see archived chat on discord ;)


Not able to sign in successfully as it throws error/exception.

It took one day to over come this problem but was in vain.
I have checked it many times and tried with some other tenant name but that too in vain.
Please give me a detailed solution so as to get it done in one go....
Thanks.

Got it working by # removing tenant name from below lines in app.config and web.config respectively:
In ## app.config change,
from :
to     :  <add key="todo:TodoListResourceId" value="https://TodoListService" `/>
In ## web.config change,
from :
to     :

Got it working by # removing tenant name from below lines in app.config and web.config respectively:
In app.config change,
from : <add key="todo:TodoListResourceId" value="https://<teanant_name>/TodoListService" />
to : <add key="todo:TodoListResourceId" value="https://TodoListService" />
In web.config change,
from : <add key="ida:Audience" value="https://<teanant_name>/TodoListService" />
to : <add key="ida:Audience" value="https://TodoListService" />

Hey @shayazm, apologies for the confusion and extra overhead, there was an issue in the readme, the instructions should've guided you to register an app that indeed had the "https://<tenant_name>/TodoListService" AppID Uri and thus would work by simply doing the replacement.


Reuse environment vars and wait for GitHub actions to avoid races

What type of PR is this?
/kind ci
What this PR does / why we need it:
Avoiding GitHub actions races by:

Waiting for jobs running for the same CRI-O commit
Re-using the variables across the jobs

Which issue(s) this PR fixes:
None
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

@haircommander PTAL

/lgtm


backend Tensorflow vs. Pytorch

This looks very promising: Pytorch
Which seems to have several advantages from our standpoint, not the least that it's not owned by the world's largest corporate. It's very early still so it could be fruitful in that sense as well.
Please could you take a look at it and share your thoughts. We could also support two (in our case Keras and Pytorch).

A brief look - seems promising. I will go through some videos.


lvm2: 2.03.18 -> 2.03.19

Description of changes
See commit messages.
pkgsMusl.lvm2 and pkgsStatic.lvm2 still build on x86_64-linux, but I haven't run any extensive tests yet (not even all nixos tests).
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

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

This should probably target staging, no?

python3Packages.aiohttp should be an unrelated staging problem, right?

python3Packages.aiohttp should be an unrelated staging problem, right?

I really hope so


ci: refactor jobs

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows Conventional Commits
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[x] Refactoring
[ ] Code style update
[ ] Build or CI related changes
[ ] Documentation content changes

@nsbarsukov hello, please review me)


colrv1_postproc.py: set OS/2.fsType=0 to remove font embedding restrictions

The old CBDT/CBLC font similarly has no font embedding restrictions (OS/2.fsType=0), so the COLRv1 font needs to match that.
Currently we weren't changing that and the font ended up using ufo2ft's default value for OS/2.fsType which is 4 (i.e. preview and print only).
Fixes https://github.com/google/fonts/issues/5729

I haven't actually run the postproc script.. the fonts are unchanged. should I?

in fact.. it crashes lol
but for unrelated reasons from this change:
$ python colrv1_postproc.py
Updating fonts/Noto-COLRv1.ttf from fonts/NotoColorEmoji.ttf
cannot process alias fe82b -> unknown_flag
Traceback (most recent call last):
  File "/usr/local/google/home/clupo/oss/noto-emoji/colrv1_postproc.py", line 332, in <module>
    app.run(main)
  File "/usr/local/google/home/clupo/.pyenv/versions/fonttools-py310/lib/python3.10/site-packages/absl/app.py", line 308, in run
    _run_main(main, args)
  File "/usr/local/google/home/clupo/.pyenv/versions/fonttools-py310/lib/python3.10/site-packages/absl/app.py", line 254, in _run_main
    sys.exit(main(argv))
  File "/usr/local/google/home/clupo/oss/noto-emoji/colrv1_postproc.py", line 323, in main
    _add_fallback_subs_for_unknown_flags(colr_font)
  File "/usr/local/google/home/clupo/oss/noto-emoji/colrv1_postproc.py", line 192, in _add_fallback_subs_for_unknown_flags
    unknown_flag = cmap[UNKNOWN_FLAG_PUA]
KeyError: 1042475

@rsheeter do you happen to know what's going on?


port changes from 1.5 branch

Checklist before submitting

[x] Did you read the contributor guide?
[ ] Did you update any affected documentation?
[ ] Did you write any tests to validate this change?

Description
Fixes # (issue).
Review process to land

All tests and other checks must succeed.
At least one maintainer must review and approve.
If any  maintainer requests changes, they must be addressed.

Need to force merge this PR as main yet does not have the config dir.


There is no way to assign an hotkey to the new command "hide tabs" (Or even better assign a key for toggling that option)

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

VS Code Version:  1.57.0
OS Version: MacOS Big Sur

Steps to Reproduce:
There is not command for hiding tabs only for showing tabs.
Personally I think it will  be better to also have toggle. So same hotkey can be assigned to hide and show tabs.
Currently I can't assign any hottkey for showing tabs

Do you need any more info for understand the issue?

The show tabs command was intended to be an internal only command which is why it doesn't show up in the command palette. Tabs are meant to be a set once and done type setting but if you really want to toggle them on and off I'd recommend making an extension and using an existing extension like https://marketplace.visualstudio.com/items?itemName=rebornix.toggle that toggles the terminal.integrated.tabs.enabled setting.


Method to stop idle.watch()

Is there a method to stop idle.watch after a user successfully logs out?

Sorry. Yes, there is. Idle.stop().


fix broken link:https://git.k8s.io/minikube/docs/contributors/adding_…

…a_dependency.md

@CriaHu Pls try raise separate PRs for different localization teams. It is very difficult to collect approvals across teams, even for small PRs like this. Thanks.

oh,sorry.I'll resubmit.

/remove-language de
/remove-language fr
/remove-language ja
/remove-language ru

/lgtm
/approve