Office upgrade does not work

Brief description of your issue
I have the following package which is listed twice when entering winget upgrade:
Microsoft 365 Apps for enterprise [Microsoft.Office] Version 16.0.14527.20234
Every time I enter winget upgrade --all I have the following problem:
When winget tries to update the 2 office packages, a window is displayed which is black and has no output but the office icon.
After 2-3 min the update probably completes because the window disapears and winget moves on.
After entering winget upgrade the Microsoft 365 Apps is still displayed in the list of upgradeable packages.
Maybe the updater for office could be improved as well. (Not only a black window)
I also noticed that when I open a Office App and check the version from within the app it displays that it is up to date.
So it appears that it is only updated because I use winget.
Steps to reproduce
Install Microsoft Office for Enterprise.
execute winget upgrade --all
Expected behavior
The Office packages should be updated.
Actual behavior
The feedback of the update process is bad (black window).
After the update it is still listed as upgradable in winget.
Environment
Windows Package Manager v1.1.12986
Windows: Windows.Desktop v10.0.19043.1288
Paket: Microsoft.DesktopAppInstaller v1.16.12986.0

What version of Office do you currently have installed? (winget list Microsoft.Office will tell you if you're not sure).

The Microsoft.Office package does launch a "blank window" and silently installs. It takes quite a while on my machines as well. It looks like there may be another installer or another version of an Office bootstrapper on the machine. I would also be interested in the output @jedieaston mentioned. We've got some new changes coming with our manifest schema to hopefully help eliminate these "near misses" with multiple packages that are similarly identified.

Every language of ProPlus you install will have its own item in the list of installed programs, thats how Windows and thus Geek Uninstaller for instance sees it too. So multilanguage installs maybe triggers this behavior? Just a guess.%

Yes, we're really frustrated with the state of winget being pushed by default with every win 11 installation, and we'll have to support this dumpster fire that is this package manager.
Meanwhile, winget PMs are happily getting their christmas bonus and enjoying their holidays. https://github.com/microsoft/winget-cli/issues/1684

Output from winget list Microsoft.Office:
Microsoft 365 Apps for enterprise         Microsoft.Office                       16.0.14430.20342    16.0.14527.20276 winget
Microsoft 365 Apps for enterprise - en-us O365ProPlusRetail - en-us              16.0.14430.20342
OneNote for Windows 10                    Microsoft.Office.OneNote_8wekyb3d8bbwe 16001.14326.20588.0
This image is from word:

Same here, winget upgrade --all is not working for ID "Micrrosoft.Office"
winget upgrade --all
Name                                      ID               Version          Verfügbar        Quelle
---------------------------------------------------------------------------------------------------
Microsoft 365 Apps for Enterprise - de-de Microsoft.Office 16.0.15427.20210 16.0.15427.20220 winget
Microsoft 365 Apps for enterprise - en-us Microsoft.Office 16.0.15427.20210 16.0.15427.20220 winget
2 Aktualisierungen verfügbar.

(1/1) Gefunden Microsoft 365 Apps for enterprise [Microsoft.Office] Version 16.0.15427.20220
Diese Anwendung wird von ihrem Besitzer an Sie lizenziert.
Microsoft ist nicht verantwortlich und erteilt keine Lizenzen für Pakete von Drittanbietern.
Downloading https://officecdn.microsoft.com/pr/wsus/setup.exe
  ██████████████████████████████  8.03 MB / 8.03 MB
Der Installer-Hash wurde erfolgreich überprüft
Paketinstallation wird gestartet...

It then opens a cmd window https://pasteboard.co/lAWtg223Af8f.png and completes with the message "Successfully installed" (translated) although it did not do anything. Versions of de-de and en-us remain unchanged at next execution of upgrade --all.
Erfolgreich installiert

This problem went away for me for about a month and is now back with version 16.0.15601.20230 trying to update to version 16.0.15629.20196.

I observe exactly the same behaviour of this bug:
PS C:\Users\vaclav.brozik> winget upgrade --all
Name                                      Id               Version          Available        Source
---------------------------------------------------------------------------------------------------
Microsoft 365 Apps for enterprise - en-us Microsoft.Office 16.0.15629.20258 16.0.15726.20188 winget
1 upgrades available.

(1/1) Found Microsoft 365 Apps for enterprise [Microsoft.Office] Version 16.0.15726.20188
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://officecdn.microsoft.com/pr/wsus/setup.exe
  ██████████████████████████████  7.07 MB / 7.07 MB
Successfully verified installer hash
Starting package install...
Successfully installed

PS C:\Users\vaclav.brozik> winget upgrade
Name                                      Id               Version          Available        Source
---------------------------------------------------------------------------------------------------
Microsoft 365 Apps for enterprise - en-us Microsoft.Office 16.0.15629.20258 16.0.15726.20188 winget
1 upgrades available.

I see the problem for the second time. First time MS Office got upgraded a different way and winget stopped displaying the new version for few days. Now the problem is there again with the new version available.
The most problematic part of the bug is that winget says Successfully installed although right after that it still detects the older version. So obviously the update was not "successfully installed".
The MS Office applications show version like below. Notice that the version is not exactly the same as the one detected by winget:

Microsoft® Word for Microsoft 365 MSO (Version 2209 Build 16.0.15629.20256) 64-bit
Microsoft® Outlook® for Microsoft 365 MSO (Version 2209 Build 16.0.15629.20256) 64-bit
...

The Office app confusingly shows something completely different:

Version : 18.2210.1203.0

I noticed this today.  It looks like winget wants to install the "Current Channel" but we are on the "Monthly Enterprise Channel"
https://learn.microsoft.com/en-us/officeupdates/update-history-microsoft365-apps-by-date
I'm not aware of any way to tell winget to install the Monthly Enterprise Channel.  Perhaps one day, it will see which channel you're on and pull from that.

I see something similar:
Name                                      Id               Version          Available        Source
---------------------------------------------------------------------------------------------------
Microsoft 365 Apps for enterprise - en-us Microsoft.Office 16.0.14931.20858 16.0.15726.20188 winget
1 upgrades available.

This is with Winget v. 1.4.10052.
Office apps themselves indicate that my organization is on the Semi-Annual Enterprise Channel with Office version 2202. So I suspect that the problem has to do with Winget somehow comparing that against what's available in the Monthly Enterprise Channel v. 2210.
As others have noted, installing/upgrading superficially seems to succeed, but subsequent calls to winget upgrade indicate that the version(s) have not actually changed.

FWIW: I ended up here because I wasn't sure whether my upgrade for Microsoft 365 Apps for enterprise - en-us was working, but in the end it did - it just took around 20 minutes to complete, possibly a touch longer.
Oddly, it reported the following pre-upgrade:
Installed: 16.0.15831.20208
Available: 16.0.15928.20196
but installed a newer 16.0.15831.20216.
I originally installed using the click2run installer from the office portal, so I think the current channel setting is correct for me. A second winget upgrade doesn't report any more upgrades required.

The blank installer window just hangs forever for me (I've left it for four hours, it never finishes, and doesn't appear to be busy in Task Manager). Office was initially installed by my organization.
Name                                    Id                                     Version          Available        Source
-----------------------------------------------------------------------------------------------------------------------
Microsoft 365 Apps for enterprise - en… Microsoft.Office                       16.0.15928.20282 16.0.16130.20218 winget

The biggest annoyance for me isn't that Office won't upgrade, it's that I really want to run winget upgrade --all, but that office installation, which I know is going to fail, takes 5-10 minutes to do so. I ended up creating an import file and just run winget import -i myfile.json and just have to remember to add any new installs to it.

The work on pinning should help a little here. At least, you can pin the packages that are causing trouble so winget upgrade --all will skip/ignore the pinned packages.
Office has a very complex installer mechanism, and we're still working with them on improvements to the installer flow so we can give a better experience with WinGet.

Thanks for mentioning pinning. I am using it and it is really a good workaround. I am looking forward for it being included into the regular release. How to use it now?

Install winget preview:

https://github.com/microsoft/winget-cli#development-releases
https://github.com/microsoft/winget-cli/releases

Enable the pinning feature: winget settings This opens a text editor! with the JSON configuration file:

{
    "experimentalFeatures": {
        "pinning": true
    }
}

Use it: winget pin add Microsoft.Office

How does it look like?
> winget -v
v1.5.1361-preview

> winget features
The following experimental features are in progress.
They can be configured through the settings file 'winget settings'.

Feature                       Status   Property                  Link
-------------------------------------------------------------------------------------------------------------
Show Dependencies Information Disabled dependencies              https://aka.ms/winget-settings
Direct MSI Installation       Disabled directMSI                 https://aka.ms/winget-settings
Package Pinning               Enabled  pinning                   https://aka.ms/winget-settings
Uninstall Previous Argument   Disabled uninstallPreviousArgument https://aka.ms/winget-settings
Configuration                 Disabled configuration             https://aka.ms/winget-settings#configuration
Windows Feature Dependencies  Disabled windowsFeature            https://aka.ms/winget-settings

> winget pin list
Name                                      Id                    Version          Source Pin type
------------------------------------------------------------------------------------------------
Microsoft 365 Apps for enterprise - en-us Microsoft.Office      16.0.16227.20318 winget Pinning
VMware Workstation                        VMware.WorkstationPro 16.2.5           winget Pinning

> winget upgrade
No installed package found matching input criteria.
2 package(s) have pins that prevent upgrade. Use the 'winget pin' command to view and edit pins. Using the --include-pinned argument may show more results.

We're close with the release of WinGet 1.5 (it's in Release Candidate mode). Pinning is a stable feature in WinGet 1.5

@denelon
I think this can be related to this issue, so I did not open a new one.
I tried pinning Microsoft.Office just now, got the message No installed package found matching input criteria..

Logs from command winget pin add Microsoft.Office
Click to expand
2023-08-09 12:31:36.988 [CORE] WinGet, version [1.5.1881], activity [{A9A0FB4B-6038-4986-81EB-D6DC2120E006}]
2023-08-09 12:31:36.989 [CORE] OS: Windows.Desktop v10.0.22621.1992
2023-08-09 12:31:36.989 [CORE] Command line Args: winget  pin add Microsoft.Office
2023-08-09 12:31:36.989 [CORE] Package: Microsoft.DesktopAppInstaller v1.20.1881.0
2023-08-09 12:31:36.989 [CORE] IsCOMCall:0; Caller: winget-cli
2023-08-09 12:31:36.993 [CLI ] WinGet invoked with arguments: 'pin' 'add' 'Microsoft.Office'
2023-08-09 12:31:36.993 [CLI ] Found subcommand: pin
2023-08-09 12:31:36.993 [CLI ] Found subcommand: add
2023-08-09 12:31:36.995 [CLI ] Leaf command to execute: root:pin:add
2023-08-09 12:31:36.995 [CLI ] Executing command: add
2023-08-09 12:31:37.006 [REPO] Default source requested, multiple sources available, adding all to source references.
2023-08-09 12:31:37.006 [REPO] Adding to source references msstore
2023-08-09 12:31:37.006 [REPO] Adding to source references winget
2023-08-09 12:31:37.007 [REPO] Multiple sources available, creating aggregated source.
2023-08-09 12:31:37.007 [REPO] Adding to aggregated source: msstore
2023-08-09 12:31:37.007 [REPO] Sending http GET request to: https://storeedgefd.dsx.mp.microsoft.com/v9.0/information
2023-08-09 12:31:37.078 [REPO] Response status: 200
2023-08-09 12:31:37.078 [REPO] Sending http GET request to: https://storeedgefd.dsx.mp.microsoft.com/v9.0/information
2023-08-09 12:31:37.089 [REPO] Response status: 200
2023-08-09 12:31:37.089 [REPO] Adding to aggregated source: winget
2023-08-09 12:31:37.098 [CORE] Examining extension: PFN = Microsoft.Winget.Source_8wekyb3d8bbwe, ID = IndexDB
2023-08-09 12:31:37.098 [CORE] Found matching extension.
2023-08-09 12:31:37.121 [REPO] Opening SQLite Index for ImmutableRead at 'C:\Program Files\WindowsApps\Microsoft.Winget.Source_2023.809.1041.838_neutral__8wekyb3d8bbwe\Public\index.db'
2023-08-09 12:31:37.121 [SQL ] Opening SQLite connection #1: 'C:\Program Files\WindowsApps\Microsoft.Winget.Source_2023.809.1041.838_neutral__8wekyb3d8bbwe\Public\index.db' [1, 40]
2023-08-09 12:31:37.122 [REPO] Opened SQLite Index with version [1.6], last write [2023-08-09 11:40:00.000]
2023-08-09 12:31:37.383 [REPO] Creating PredefinedInstalledSource with filter [None]
2023-08-09 12:31:37.383 [REPO] Creating new SQLite Index with version [Latest] at ':memory:'
2023-08-09 12:31:37.383 [SQL ] Opening SQLite connection #2: ':memory:' [6, 0]
2023-08-09 12:31:37.410 [REPO] Reading MSI UpgradeCodes
2023-08-09 12:31:37.412 [REPO] Examining ARP entries for Machine | X64
2023-08-09 12:31:37.434 [REPO] Examining ARP entries for Machine | X86
2023-08-09 12:31:37.458 [REPO] Reading MSI UpgradeCodes
2023-08-09 12:31:37.460 [REPO] Examining ARP entries for User | X64
2023-08-09 12:31:37.749 [REPO] Opening SQLite Index for ReadWrite at 'C:\Users\olav.birkeland\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\StoreEdgeFD\installed.db'
2023-08-09 12:31:37.749 [SQL ] Opening SQLite connection #3: 'C:\Users\olav.birkeland\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\StoreEdgeFD\installed.db' [2, 0]
2023-08-09 12:31:37.749 [REPO] Opened SQLite Index with version [1.3], last write [2022-08-26 13:57:35.000]
2023-08-09 12:31:37.775 [REPO] Sending http POST request to: https://storeedgefd.dsx.mp.microsoft.com/v9.0/manifestSearch
2023-08-09 12:31:37.787 [REPO] Response status: 200
2023-08-09 12:31:37.791 [REPO] Opening SQLite Index for ReadWrite at 'C:\Users\olav.birkeland\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\Microsoft.Winget.Source_8wekyb3d8bbwe\installed.db'
2023-08-09 12:31:37.791 [SQL ] Opening SQLite connection #4: 'C:\Users\olav.birkeland\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\Microsoft.Winget.Source_8wekyb3d8bbwe\installed.db' [2, 0]
2023-08-09 12:31:37.791 [REPO] Opened SQLite Index with version [1.3], last write [2023-08-08 13:36:11.000]
2023-08-09 12:31:37.829 [REPO] Downloading manifest
2023-08-09 12:31:37.829 [CORE] WinINet downloading from url: https://cdn.winget.microsoft.com/cache/manifests/m/Microsoft/Office/16.0.16626.20148/be42-Microsoft.Office.yaml
2023-08-09 12:31:37.844 [CORE] Download hash: 107a01ef06db860cddac0317736d6cf7dd71ae1e467c8dd6ecb144308c74c267
2023-08-09 12:31:37.844 [CORE] Download completed.
2023-08-09 12:31:37.845 [REPO] Finding installed package from available package using system reference search: Query:[none] Include:NormalizedNameAndPublisher='microsoft 365 apps for enterprise'+'microsoft corporation'[Exact] Include:NormalizedNameAndPublisher='microsoft365appsforenterprise'+'microsoft'[Exact]
2023-08-09 12:31:37.846 [REPO] Found multiple matches for available package [Microsoft.Office] in source [Microsoft.Winget.Source_8wekyb3d8bbwe] when searching for [Query:[none] Include:NormalizedNameAndPublisher='microsoft 365 apps for enterprise'+'microsoft corporation'[Exact] Include:NormalizedNameAndPublisher='microsoft365appsforenterprise'+'microsoft'[Exact]]
2023-08-09 12:31:37.846 [REPO]   Checking match with package id: O365ProPlusRetail - en-us
2023-08-09 12:31:37.846 [REPO]   Checking match with package id: O365ProPlusRetail - nb-no
2023-08-09 12:31:37.846 [REPO]   Appropriate installed package could not be determined
2023-08-09 12:31:37.846 [CLI ] No app found matching input criteria
2023-08-09 12:31:37.852 [CLI ] Terminating context: 0x8a150014 at D:\a\_work\1\s\external\pkg\src\AppInstallerCLICore\Workflows\WorkflowBase.cpp:395

Seems winget-cli struggles when an app is listed as installed twice?

winget upgrade --all looping update not sucessfull update

after period time, even its say successfully installed,

Oct 2024 and this is still an issue...

User "vbrozik" described it correctly above (some time ago).
Winget ignores which "M365 update channel" is set on the given OS and displays the latest available version from the "current channel".
However, the installation package that winget downloads and runs, it downloads the version according to the "update channel". And there might be an older version in a given channel (even if it's the latest in that channel) that it then installs.
Possible channels are:
"Current channel"
"Monthly Enterprise Channel"
"Semi-Annual Enterprise Preview"
... more here: https://www.office365versions.com/allreleases.html
The only way to prevent winget from continuing to try to update (to an older/same version) is to mark the given application as "pinned" with the command:
"winget pin add --id Microsoft.Office --verbose"
... and then use the command without "--include-pinned" and without "--force" to update, for example:
"winget upgrade --all --include-unknown --verbose"
View "pinned" applications:
"winget pin list --verbose"
or specifically:
"winget pin list --id Microsoft.Office --verbose"
Perhaps sometime in the future "winget" will follow and search for updates available only within the selected "update channel".
More info here:
https://learn.microsoft.com/en-us/microsoft-365-apps/updates/change-update-channels


Add umbrajs withdraw methods

Closes #79
Also refactors methods to take in exactly what is needed. So we no longer associate keys or a signer with the Umbra class—just a provider—and methods take keys or singers as inputs when needed

@apbendi All comments addressed in the last commit, feel free to merge if it looks good!


Buttons

Add milo buttons
Resolves: MWPW-147020
Test URLs:

Pre Migration Link: https://www.adobe.com/express/feature/image/remove-background
Before: https://main--express-milo--adobecom.hlx.page/drafts/vhargrave/buttons
After: https://buttons--express-milo--adobecom.hlx.page/drafts/vhargrave/buttons

Codecov Report
Attention: Patch coverage is 0% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 50.68%. Comparing base (195c35c) to head (169869f).

Files
Patch %
Lines

express/scripts/utils.js
0.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #23      +/-   ##
==========================================
- Coverage   52.93%   50.68%   -2.25%
==========================================
  Files           4        4
  Lines         869      870       +1
==========================================
- Hits          460      441      -19
- Misses        409      429      +20

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


crash on crafting

https://i.imgur.com/ktiolrp.jpeg cant repoduce out of this wolrd

I've also experienced this bug.
Steps to reproduce (AFAIK):

Gather material to craft an item
Craft desired item using at least one material from within your inventory (not inside of chests)

It seems this crash only occurs on some worlds that fit a certain criteria (world length or something?)
There might be other criteria to reproduce to the bug, but it's really quite elusive.
I've found that removing the PushEvent("itemlose") altogether fixes the issue, however I believe this may(?) cause some sort of continuity issue between server clients.

Fun idea, could you send me a copy of that world? The whole folder in Documents\Klei\DoNotStarveTogether\254404980

Fun idea, could you send me a copy of that world? The whole cluster folder in Documents\Klei\DoNotStarveTogether\254404980

Absolutely. Here is the cluster:
https://drive.google.com/file/d/1JMgm5wNnFfrW13r9nJXiWEdhysheuHMw/view?usp=sharing
Here are the server mods I'm using as well, in case that is helpful:

Beefalo Widget
Chest Includer
Don't Reset World
Entity Global Icon Indicator
Finder Redux
Global Positions

We were able to reproduce the error on our own. It's a client thing. When clients (including hosts on worlds with caves) attempt to create anything, the client will crash. There are also a few more mistakes we made that will require some extra fixing (refer to the example I made on my steam comment). ...... we'll let you know how this goes as we fix it.....

We fixed all the issues people were having (theoretically)... The free crafting issue and the "data" crash. Test it out and let us know if it works!

fixed


Reduces cost of using DCO

Is there anyway to make DCO fundamentally less costly to use?
I’ve seen that engineers spend a lot of time fixing DCO checks, if you factor in the merge conflicts and time lost to context switching, I’d estimate around 30m per pull request. If DCO is used on (say) 5,00,000 pull requests a year and engineers cost around $100/hr, then we’re looking at $250,000,000 spend of fixing DCO each year for the Open Source community. Maybe you think there are fewer pull requests with DCO, so maybe you think fixing it is only $100,000,00 a year? LMK if you have better numbers.
I see even experienced engineers loosing time to fixing DCO checks because there are just too many ways to fail to sign-off a commit. Just forget, use wrong args, merge in master, rebase, use Codespaces, accept a suggestion to a pull request can cause this and you only need one non-signed-off commit to fail a PR.
How can we make it less costly?

Remove DCO for all projects.
Github provides a button to sign-off commits post-hoc.
DCO check only requires one commit signed-off.
DCO check provides a button to fix commits.

What else?

I think this could be fixed buy having it respect GPG signatures, since I assume your and many other corporate teams are gpg signing commits. That's something I definitely would like and would make me a lot less frustrated when using this thing.


Update Pom

I noticed you are using my repo (notfab.net) which is no longer a thing.
Since people come asking to us every once in a while why maven is failing, I've updated your pom to match the official repository, with the latest JDA version (and LavaPlayer - which fixes bugs).

Hm, the build error is related to the CI server not understanding Java 8, so this shouldn't be a problem.


Add fake GitHub homebrew token

Fixes the buildbot master file.recurse failures we've been seeing recently.
Also updates the highstate output to increase the SNR.
See commit messages for details.
Follow-up to #460. This failure was missed because we were experiencing pip failures at the time, causing Travis bustage, and the PR was merged by hand without noticing the additional failures on Travis.
r? @larsbergstrom @edunham
cc @paulrouget

This change is

#465 should be merged first to fix the syntax error.

Also note that the scratch builder is the important Travis result; the other one (that builds the old config then the new config) will fail because the old config is busted.

@bors-servo r+

:pushpin: Commit d7494f8 has been approved by larsbergstrom

:hourglass: Testing commit d7494f8 with merge 9321ef6...

:broken_heart: Test failed - status-travis

@larsbergstrom please larsbors this, the failure is expected since the old config is busted


Exception in hkube wrapper 2.5.0.dev8

Ran a scenario from  10 min with input rate of  100 msg per sec.
got the following exception during the test run:
wrapper::INFO::WebsocketClient::got message from worker: serviceDiscoveryUpdate
Exception in thread serviceDiscoveryUpdateThread:
Traceback (most recent call last):
File "/usr/lib/python3.9/threading.py", line 954, in _bootstrap_inner
self.run()
File "/usr/lib/python3.9/threading.py", line 892, in run
self._target(*self._args, **self._kwargs)
File "/opt/venv/lib/python3.9/site-packages/hkube_python_wrapper/wrapper/algorunner.py", line 236, in handle
self._discovery_update(data)
File "/opt/venv/lib/python3.9/site-packages/hkube_python_wrapper/wrapper/algorunner.py", line 346, in _discovery_update
self.streamingManager.setupStreamingListeners(
File "/opt/venv/lib/python3.9/site-packages/hkube_python_wrapper/communication/streaming/StreamingManager.py", line 54, in setupStreamingListeners
listener = MessageListener(options, nodeName)
File "/opt/venv/lib/python3.9/site-packages/hkube_python_wrapper/communication/streaming/MessageListener.py", line 12, in init
self.adapater = ZMQListener(remoteAddress, self.onMessage, self._encoding, receiverNode)
File "/opt/venv/lib/python3.9/site-packages/hkube_python_wrapper/communication/zmq/streaming/ZMQListener.py", line 20, in init
self._worker = self._worker_socket(remoteAddress)
File "/opt/venv/lib/python3.9/site-packages/hkube_python_wrapper/communication/zmq/streaming/ZMQListener.py", line 26, in _worker_socket
worker = context.socket(zmq.DEALER)
File "/opt/venv/lib/python3.9/site-packages/zmq/sugar/context.py", line 235, in socket
s = self._socket_class(self, socket_type, **kwargs)
File "/opt/venv/lib/python3.9/site-packages/zmq/sugar/socket.py", line 58, in init
super(Socket, self).init(*a, **kw)
File "zmq/backend/cython/socket.pyx", line 328, in zmq.backend.cython.socket.Socket.init
zmq.error.ZMQError: Too many open files

Happens due to intensive change of statelss pods. Happned in the bulk feature use.
Bulk feature has been reverted, it will not happen any more. closing the issue


Spans contain column information

Check it out!
// a.sy
start :: fn {
    // åäö <-- look ma, variable length encoding!
    abc := wow()
    a := 1
}

$ NO_COLOR=1 cargo run a.sy
    Finished dev [unoptimized + debuginfo] target(s) in 0.03s
     Running `target/debug/sylt a.sy`
compile error: a.sy:3
      Failed to compile line 3
      Cannot read 'wow' in 'a.sy'
   1 | start :: fn {
   2 |     // åäö <-- look ma, variable length encoding!
   3 |     abc := wow()
                  ^^^

Error: "1 errors occured."

The compiler and type checker still only knows about lines though.

Can we propegate these changes even further. So we can get span errors in the runtime and typechecker as well?

I thought about it, but in the interest of small PRs I left it out.


app('router')->getRoutes()->match() - Dosen't work correctly when running on subdirectory

Laravel Version: 8.0.0
PHP Version: 7.3.22
Database Driver & Version: MySQL 5.6

Description:
I'm running Laravel in a subdirectory, so the base of the url is HTTP://www.domain.com/laravel
Was running on a small issue with Livewire returning 404 on ajax request, so did some digging and found where the bottleneck was, and it was this function to get the current route from the url.
// $referer = CURRENT_URL $route = app('router')->getRoutes()->match(                 Request::create($referer, 'GET')             );
And it returned the 404 response. So I did a small hack to confirm, str_replace the /laravel (subdirectory) segment from $referer, and it worked.
Steps To Reproduce:
Set up an Laravel application in a subdirectory and try the function
// $referer = CURRENT_URL $route = app('router')->getRoutes()->match(                 Request::create($referer, 'GET')             );

I don't think running a Laravel app from a subdirectory is supported
Docs: https://laravel.com/docs/8.x/installation#directory-configuration
Also some comments from core developers on previous issues:

I don't think running a Laravel app from a subdirectory is supported

That's correct. It's not.


Corporate surcharge

Context
Adds some documentation for corporate surcharging in "Optional features" section to be added when that work is completed
Changes proposed in this pull request

Adds to list of optional features and reorders that list slightly
Adds instructions as given by @alexbishop1 for adding corporate surcharges

Guidance to review
I've put up a test environment: https://govukpay-corporatesurcharge.cloudapps.digital/optional_features/#optional-features

Thanks @alexbishop1, I think I dropped the ball with this one so thanks for picking these up

Thanks for picking these up @alexbishop1, apologies for dropping the ball a bit :)
Changes in latest commits summarised per suggestions/other things I've noticed:

more active phrasing a la “You can set a different corporate surcharge for each of the following 4 types of card:”
fixed prepaid/pre-paid inconsistency
changed language re. how GOV.UK checks against numbers rather than use term "list"
fixed extraneous space/broken link/full stop error
changed language re. different types of amount to bulleted list, and added info about "amount" amount
changed language re. "you can use ... partial refund..."

@alexbishop1 I've followed the discussions in the individual comments to try to improve this, per your suggestions. The only extra thing worth mentioning is that I've had to be a bit creative to fit in with GOV.UK style in one place—rather than bullets for the responses, I'm trying h3s.
Let me know what you think (or please feel free to tell me I need to talk to someone else; I don't want to take too much of your time). There's no rush for this in any case.
Thanks again

I've changed the h3s to h4s ... the h3s looked a bit weird.

All looks good to me. I've raised the question on slack about whether we should hint that we could apply percentage if needed. But happy to go with this for now.

On reflection, it would be good if we can find a way to soften this "Each surcharge is a flat amount added to a payment, not a percentage." We visited an LA yesterday who want to use corporate surcharging but with percentages - and i think this is probably quite a common use case, as card schemes do charge on a % basis so it makes sense that a merchant just passes on this charge.

@katiebates How does it look now? I've updated in the PR and this test deployment:
https://govukpay-corporatesurcharge.cloudapps.digital/optional_features/#optional-features

I'm also not sure when to merge this as I know this feature technically isn't live yet so do please let me know when it's OK to do that.

Based on good feedback I've changed h4s at the bottom of the doc to a table so this may also be worth reviewing. The other edits I'll make shortly are just minor language ones.

I'm also not sure when to merge this as I know this feature technically isn't live yet so do please let me know when it's OK to do that.

You can merge it whenever you're ready - the majority of the work is finished and in any case, it can't be 'live' for anyone until we manually enable it anyway.

one minor thing - the docs say that the user will be informed about the surcharge. do we need to be a little clearer how this happens? (e.g. on the payment page, when the user enters the card number..?)

@katiebates I've added this line:

If a surcharge is applicable, when users enter their card details on the payment
page, the platform will inform them that a surcharge of a certain amount
will be added to the total.

It's a bit meaty but I can't think of a better solution.

@katiebates In hindsight I'm not sure we do need this edit, since we have this in the preceding sentence:

When a user enters a card number in their payment journey...

I'm not sure, though.


DL3-56 Display error message if LINEITEMs aren't successfully synced to Harmony

Description
When there's an error syncing line items the frontend must display an error.
Motivation and Context
Provides an error message when there's an error syncing line items
Fixes
Jira ticket
How Has This Been Tested?

Tested locally simulating the scenarios and tested against D2L.
Screenshots

Database changes

⚠ Deployment instructions ⚠

New ENV variables

Checklist

[X] I have reviewed the AC for this feature and my changes meet all requirements
[X] (For UI changes) I have considered the accessibility of this feature (see https://www.a11yproject.com/checklist/ for a high-level checklist)
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] Any dependent changes have been merged and published in downstream modules
[X] My change requires a change to the documentation.
[X] I have updated the documentation accordingly.
[X] I have requested a review from at least one other dev

This needs rebasing after #79

This is how it looks like after the change:

Thanks for the review @mgwozdz-unicon !


Unable to open and view contents of Blob Container

Unable to retrieve child resources.
Details:
["RestError:AuthorizationFailureThis request is not authorized to perform this operation.\nRequestId:b7039efa-401e-0020-6388-8b8c67000000\nTime:2020-09-15T17:51:25.5367916Z"]

@cjmcgnow this error message means you don't have the permissions needed to do what you are trying to do.
Can you give some more details like:

When did this happen?
What are you trying to access?
What type of authentication are you using, sign in, SAS, key?

Closing due to lack of response. Please open a new issue if you are still encountering this problem.


Preserve the pipeline's order

The pipeline gets re-sorted by length of path, and for paths of the same length, the results are non-deterministic. This non-deterministic nature means that when I check rev-manifest.json in to a VCS, it'll sometimes have random changes in order, unrelated to my current source file changes.
This behavior was introduced in c2a9e54, as part of a fix for issues with similarly-named files. (It is non-deterministic because the callback given to sort() returns 0 for same-length paths without bothering to sort them in any other way. This isn't a problem for its internal goal.)
This PR isolates the length-sorting in a shallow clone of the main file list. It also adds a simple test of this issue.
(As a separate commit, this PR also changes the location of the sort() so that it only runs once rather than re-sorting after each path is received; this should be faster, but if you don't want this change, I can remove it from the PR.)

I see that https://github.com/sindresorhus/gulp-rev/pull/113 added forced, deterministic sorting of rev-manifest.json. So this PR won't be entirely necessary once we upgrade to gulp-rev 5.1.0. However, it is still nice to be a good pipeline-neighbor for any downstream processing. And reducing the number of times this plugin sorts the list of files can't hurt either! :)


Fixed centralize logic TODOs in ec2

This PR makes the following changes:

Replaced three security_groups.filter() instances at lines 180, 197 and 379, each marked with a "Centralize logic..." TODO comment, with get_security_groups() function.
Removed "Centralize logic..." TODO comments in all previously mentioned instances.

I tested this PR by:

Launching a cluster
Adding slaves
Removing slaves
Destroying a cluster
Running "pytest" command

Fixes #377.

Thanks for taking this on @tomahawk360 and sorry about the delay in reviewing this. I've built on your work by organizing a few other instances of repeated security group logic into a couple of small functions.
The security group code overall is still messy, but I think it's a bit better with these changes.


Outdated examples/included-as-non-root-mapped using legacy Cypress

follows on from issue https://github.com/cypress-io/cypress-docker-images/issues/1086

What would you like?
Update examples/included-as-non-root-mapped to use a current version of Cypress.
Why is this needed?
The examples/included-as-non-root-mapped demonstrates how to build a new Docker image on top of cypress/included using a modified user and group. It is based on the legacy Cypress version 5.6.0 and for Docker Desktop on Mac.
Examples are not helpful if they do not demonstrate how to achieve results using current versions of Cypress. Filenames have changed and videos are no longer generated by default.

This is not an issue that I can tackle since it requires macOS that I don't use.

The example was not updated. Instead it was removed through PR #1245.


Add unclaimed reward info/Support reward withdrawing for nomination pool

Is your feature request related to a problem? Please describe.

[x] Get unclaimed reward info
[x] Support reward claiming

Version test: https://file.cdmteck.com/s/4oimk9zDJ8R5Aai/download?path=%2F&files=./koni-dev-build-0.6.9-1668497789071.zip
Validation passed for cases:

Show unclaimed balance
Claimed balance
Regression test for staking feature


Small badge problem

Hello,
When you put a group in invisible to prevent it from being seen in the list, the group's badge is not displayed on the profile
For example I'm in the admin group which is invisible but I'm looking to display the badge
Thanks

Use "show all badges in profile"

Where is it?

Look in Oxide Theme.

The option is already activated

In this case, NodeBB limits the display of the group after setting.

So there's no way to display a badge even if the group is hidden ?

@JeanBatiste73 If the group is set to "hidden" then it will not appear in either the group list or your profile. In order for someone to see it, you should deselect the option "Hidden".


Error referencing 'catppuccin-grub-theme' on system without grub

Trying to add catppuccin with the global enable flag I get the following error:
       error: The option `home-manager.users.oz.boot.loader.grub.font' does not exist. Definition values:
       - In `/nix/store/1qqg71jv43w66mhgjfkp3d91xizk1v4s-source/modules/nixos/grub.nix':
           {
             _type = "if";
             condition = false;
             content = "/nix/store/2jsy9midp0vzg9jcids65vx2xzcy712l-catppuccin-grub-theme/font.pf2";

What is odd is that my nixos config doesn't include grub (and contains no references to grub that I can find, and I get the same error if I explicitly disable grub in my config.
I'm using nixos-24.05, and have added catppuccin to my flake in my nixos config and home-manager config; and then using 'catppuccin.enable = true' in nixos config.
Any ideas?

Edited the title; the former was the (different) error I get when I enable catppuccin globally via Home Manager instead of nixos config.

From the looks of the trace, i would say it looks like your importing inputs.catppuccin.nixosModules.catppuccin somewhere inside of your home-manager config. But you only want to use inputs.catppuccin.homeManagerModules.catppuccin.

Oops, user error. Had 'catppuccin.nixosModules.catppuccin' in both nixos config and home-manager... ignore!!

Our messages crossed!


Deploy a CAP Business Application to SAP Cloud Platform

Tutorial URL: https://developers.sap.com/tutorials/cap-service-deploy.html
In step 3.2  the service instance created is not found
Pushing from manifest to org "account" / space dev as "email"...
Using manifest file c:\...path...\my_bookshop\gen\db\manifest.yaml
Getting app info...
Service instance my_bookshop-db-hdi-container not found
FAILED

even if the service is created, there is no binding to any app available

This is the same as #5026.
@thecodester please close as duplicate.


[AIP-122] Keys must be snake case.

Also, make the tests better.

Marking do not merge; this still has one bug.

Fixed my bug.


Replaced foreach loop with spread operator in Connections/PhpRedisSen…

I used the spread operator (...) to pass the ERROR_MESSAGES_INDICATING_UNAVAILABILITY array to the str_contains() function. This makes the code more concise and readable.

Passing an array as second argument is not an option when using str_contains. I think you are confusing this with the deprecated Laravel str_contains helper function, which is not used in this project.


The ChiSquareTest  function parameters are in the wrong order

evidently.analyzers.stattests.chisquare_stattest

result = sum((obs-exp)^2/exp);
In ChiSquared distribution, different order result in different results。

Hi @Coder-HHN !
Thanks for reporting!
The bug is fixed. 👩🔧 The update will be published in the next release.
#289

你好，来信已收到，将尽快回复。谢谢。


mashlib load failure on main page

Outline.expand: Unable to fetch https://localhost:8086/: Failed to load  https://localhost:8086/ Fetcher: https://localhost:8086/ Unauthorized status: 401

I think this happens when the user is not logged in.
Go to https://localhost:8086/.account/login/password/register/
But there are a lot of clicks to sign up (see #16) and after that this error still seems to happen on the site root.
Investigating.

When I log in with solidcommunity.net instead of with localhost:8086 the error changes from a 401 to a 403.
I'm going to stop debugging this on localhost and use a Digital Ocean server instead, to rule out self-signed cert issues.

@michielbdejong
You could try https://github.com/SolidOS/css-mashlib/blob/main/package.json
npm run start1 seems to work without issue

Hm, it happens on https://pivot.pondersource.com/ as well.

Indeed, npm run start1 from https://github.com/SolidOS/css-mashlib works! Thanks. I'll find out what I'm doing differently here.

The difference is that npm run start1 doesn't use the subdomains config.
If I run:
npx community-solid-server -c ./config/https-mashlib-subdomain-file.json -f ./data --httpsKey /etc/letsencrypt/live/pivot.pondersource.com-0001/privkey.pem --httpsCert /etc/letsencrypt/live/pivot.pondersource.com-0001/fullchain.pem -p 443 -b https://pivot.pondersource.com

from https://github.com/SolidOS/css-mashlib on pivot.pondersource.com then after signup https://michielbdejong.pivot.pondersource.com works, but the main page still doesn't.
The logs say:
2024-11-05T14:27:44.157Z [WebAclReader] {Primary} error: No ACL document found for root container https://pivot.pondersource.com/
2024-11-05T14:27:44.158Z [PermissionBasedAuthorizer] {Primary} warn: Unauthenticated agent has no read permissions
2024-11-05T14:27:44.246Z [HandlerServerConfigurator] {Primary} info: Received GET request for /

I'll see if I can manually create a "pod" for that main URL and put an ACL and a welcome page into it.

You could look to https://solidcommunity.net:3000/ this is a subdomain testserver
I think it works has you want
Documentation is here with the config file https://github.com/solid/solidcommunity.net/wiki/solidcommunity.net-servers#1-css-subdomain-test-server--httpssolidcommunitynet3000

sudomain and suffix do not use the same CSS config

Thanks! I also found the RootStorageLocationStrategy config variable in the CSS code which might be relevant.


add channel

the app crashes because of
compile "com.android.support:support-core-utils:26.1.0"

By the way i tested this on Android 8 and Android 7.1, and local notification are working <3

@Titozzz you can target this branch in your npm.
yarn add react-native-fcm@git://github.com/evollu/react-native-fcm#sdk-26

There are 2 reasons why the PR is not merged.

as I mentioned in #698, I can't get my sample project to compile.
facebook hasn't officially support compiling with SDK 26. merging this into main branch will force everyone to upgrade to an non-officially supported state.

I am still not getting any notification icon when the app is in background, but the app is getting the data in registerKilledListeners as per the example. Any idea how can I get the icon in the notification shade? I am using sdk-26 branch of react-native-fcm.
compileSdkVersion 26
buildToolsVersion '26.0.2'
targetSdkVersion 26

@evollu I have created a notification channel by referring this: https://github.com/invertase/react-native-firebase-docs/blob/master/docs/notifications/android-channels.md
Now I can see the channel listed in settings > apps > my-app > notifications. But still I am unable to see the notification icon in the notification tray/shade.

@evollu its finally working, needed to send channel key inside custom_notification:
"data" : {
 	"show_in_foreground": true,
 	"content_available" : true,
 	"priority" : "high",
	 "title" : "Sample title",
	 "message": "Sample message",
	 "type": "DEEPLINK",
	 "sound": "default",
	 "vibrate": 300,
         "lights":  true,
          "utm_medium": "medium",
	  "utm_campaign": "campaign",
	  "utm_content": "content",
	  "utm_term": "term",
	  "utm_source": "example",
	  "channel": "default",
	 "custom_notification" : {
	   "body" : "Sample body",
	   "color": "#F58024",
	   "title" : "Sample title",
	   "type": "DEEPLINK",
            "sound": "default",
	   "icon": "ic_notif",
	   "channel": "default",
	   "picture": "image_url"
	 },
 },

Added channel creation in registerListeners:
export function registerAppListener(openPushNotification, handleOpenURL) {
  const channel = new firebase.notifications.Android.Channel('default',
  'Notification', firebase.notifications.Android.Importance.Max)
  .setDescription('Receive notifications');

  // channel id is set as: default
  // Create the channel
  firebase.notifications().android.createChannel(channel);
}

The channel id in the notification payload should match the one given while channel creation.

@sahil290791
According to the example of the changes, your registerAppListener can be replaced with
FCM.createNotificationChannel({
      id: 'default',
      name: 'Default',
      description: 'general usage',
      priority: 'high'
})


Webserver returns wrong ContentType application/x-extension-html header

[x] I confirm that this is an issue rather than a question.

Bug report
On one system I could not open the Hyperion web UI, the browsers just show a download prompt for the html, instead of rendering it.
After digging a while I found out that it is related to the MIME type setup in the home dir, that changes what header the server returns.
A solution is in the next paragraph, main reason for this report is that others hopefully don't have to debug this as long.
Still it might be better to hardcode the content type headers, so it does not depend on users MIME type configs.
Steps to reproduce
Create the file ~/.local/share/mime/packages/user-extension-html.xml with:
<?xml version="1.0" encoding="UTF-8"?>
<mime-info xmlns="http://www.freedesktop.org/standards/shared-mime-info">
 <mime-type type="application/x-extension-html">
  <comment>html document</comment>
  <glob pattern="*.html"/>
 </mime-type>
</mime-info>

Then run update-mime-database ~/.local/share/mime, now after a hyperiond restart it shows the unexpected behavior.
To fix it again just delete the file and update/generate the mime db again.
I don't know which app created this config file initially (in my case from 2019), but it is apparently not needed (and causing issues). There is not much on the internet, I found only this report on a mail plugin, that was also solved by deleting the file. Nobody seems to actually know why it actually exists though.
What is expected?
Hyperion webserver returning proper Content-Type: text/html:
$ http http://localhost:8090/ -h
HTTP/1.1 200 OK.
Access-Control-Allow-Origin: *
Content-Length: 15989
Content-Type: text/html
Date: Sa., 11 Dez. 2021 21:16:21 UTC
Server: Hyperion Webserver

What is actually happening?
Wrong application/x-extension-html:
$ http http://localhost:8090/ -h
HTTP/1.1 200 OK.
Access-Control-Allow-Origin: *
Content-Length: 15989
Content-Type: application/x-extension-html
Date: Sa., 11 Dez. 2021 21:16:07 UTC
Server: Hyperion Webserver

System
Hyperion Server:

Build:           makepkg (GitHub-f8d78bb5/17582ac3-1638637220)
Version:         2.0.12

Hyperion Server OS:

Distribution:   Arch Linux
Architecture:   x86_64
Root/Admin:     false
Qt Version:     5.15.2
Python Version: 3.9.9
Browser:        Mozilla/5.0 (X11; Linux x86_64; rv:95.0) Gecko/20100101 Firefox/95.0

I can confirm this issue and the solution (to simply delete the file) on manjaro (arch based).
Hyperion Server:
- Build:             master (GitHub-85a83872/2efc03b6-1642870083)
- Build time:        Jan 26 2022 16:39:44
- Git Remote:        /home/nighi/sData/.cache/yay/hyperion.ng-git/hyperion.ng
- Version:           2.0.13-beta.1
- UI Lang:           auto (BrowserLang: en-US)
- UI Access:         default
- Avail Screen Cap.: framebuffer,x11,xcb,qt
- Avail Video  Cap.: v4l2
- Avail Services:    boblight,cec,forwarder,flatbuffer,protobuffer,borderdetection
- Config path:       /home/nighi/.hyperion
- Database:          read/write

Hyperion Server OS:
- Distribution:      Manjaro Linux
- Architecture:      x86_64
- CPU Model:       Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz
- CPU Type:        94
- Kernel:            linux (5.10.93-1-MANJARO (WS: 64))
- Root/Admin:        false
- Qt Version:        6.2.2
- Python Version:    3.10.1
- Browser:           Mozilla/5.0 (X11; Linux x86_64; rv:96.0) Gecko/20100101 Firefox/96.0

@Jakeler Hyperion only returns the mime.type from the request in the response.
Given that different mime-types are exchanged while rendering the page, one cannot just hard-code as you suggested.
As this is a general OS and library problem, I suggest you stay with the workaround and I will close the issue for now.


Create a ShowCase for Widgets

Pls take a look at this page. I think we should start to create a showcase like this.
http://www.material-ui.com/#/components/app-bar

Basic idea
the left side is navigation bar which contains following sections
Get Started
Introduce how to use widget lib

Pre-required
Installation
Basic Usage

Costomization
Allow user to pick required features/themes in an user-friendly way. Here we can also inject Phone object to allow real-time interaction

Features
Themes

Components
Showcase containers and components. Not sure if we need to connect Phone object here. Need to have more discussion on this.

Now, we can visit show case components on Github Pages


Redesign Add New Project page

Redesign Add New Project page similar to the image below 👇

You can contact me for images and other resources.

I can do this. Please assign me this issue.

I can do this. Please assign me this issue.

You can start working on this issue, make sure to complete this within 6 days

I can do this. Please assign me this issue.

You can start working on this issue, make sure to complete this within 6 days

@satyampgt4 She is already assigned to another issue. Please look for the other issues too. We cannot assign a single person to multiple issues at the same time.

I can do this. Please assign me this issue.

You can start working on this issue, make sure to complete this within 6 days

@satyampgt4 She is already assigned to another issue. Please look for the other issues too. We cannot assign a single person to multiple issues at the same time.

I had read the conversation of that issue so I thought problem regarding API has be solved

We cannot assign a single person to multiple issues at the same time.

I will take care of this in future

Now can I work on this issue? From where will I get that purple plane image?

Now can I work on this issue? From where will I get that purple plane image?

Ping me on Discord or Slack. I will send you that image.

@rockingrohit9639 I did this much . I don't know why project level textfield and language textfield are not even clickable. Also nothing major changes when i tried adding styling to it. Can i remove material ui fields and use html tags to add the field and styling? But the functionality will be gone then.

@rockingrohit9639 I did this much . I don't know why project level textfield and language textfield are not even clickable. Also nothing major changes when i tried adding styling to it. Can i remove material ui fields and use html tags to add the field and styling? But the functionality will be gone then.

the right thing you are doing is not to be done this way, these are the dropdown designs which will show as suggestions when user clicks on the input field

Should I then leave this issue? Because I am not very much familiar with React. I just know the  basics. And this functionality thing seems complex to me, I might mess it up.

Should I then leave this issue? Because I am not very much familiar with React. I just know the basics. And this functionality thing seems complex to me, I might mess it up.

Okay, leave it

I would like to work on this issue whenever its possible

I would like to work on this issue whenever its possible

Okay bro I will assign you when you completes the other one

Hi @ShwetKhatri2001 bro, you can start working on this issue. You are given 6 days to complete it.


[SortableGroup/SortablePosition] Bug while changing few items at the same time

Hello!
I noticed bug while changing few items.
How to repeat:

Create a simple Entity with the next fields:

/**
 * @ORM\Column(type="integer")
 * @ORM\Id
 * @ORM\GeneratedValue
 */
protected $id;

/**
 * @ORM\Column(type="string", length=255)
 */
protected $name;

/**
 * @Gedmo\SortableGroup
 * @ORM\Column(type="integer")
 */
protected $group;

/**
 * @Gedmo\SortablePosition
 * @ORM\Column(type="integer")
 */
protected $position;

Add data (id/name/group/position):
1    1-1    1    1
2    1-2    1    2
3    1-3    1    3
4    1-4    1    4
5    2-1    2    1
6    2-2    2    2
Go to the List Action
Open at the same time for editing (in new tabs) the elements with id №1, 3, 4
Change the group from 1 to 2
Save
Do the same for the items №3 and №4
As a result we have elements with wrong (same) positions

Option №2:
6. Save
7. Open the item №3
8. Refresh page (F5)
9. Change the group from 1 to 2
10. Save
11. Do the same for the item №4
As a result of the option №2 elements saving OKey.
P.S.: sorry for my English)

Closing due to age and lack of activity. If you are interested in this issue or are still experiencing any problems described here, please add a comment!


Revise and optimize default styles for form elements

We need to add Sass magic to make form elements easier updatable.

We have radio and checkbox'es done:
http://codepen.io/devbproto/pen/bNKEyV
http://codepen.io/devbproto/pen/WbyQPP
We need to add selectbox, default text field with label and validation, file upload

http://codepen.io/devbproto/pens/tags/?selected_tag=form
checkbox, radio button, file input, text field, text area, select box, validation elements basic styles added to codepen under 'form' tag.
validation is above the field, so it does not require extra styling for select boxes and text areas(scroll bar).


Fix typo in IncomingRequestTest

Description
Fixes typo in test.
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPdocs
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

Thank you @kenjis

Thank you @kenjis


Response for preflight is invalid (redirect)

0:1 Failed to load http://developer.globelabs.com.ph/dialog/oauth?app_id=xxxxxxxxxx: Response for preflight is invalid (redirect)
pls help me if you know how to solve this, I used laravel framework

It may be a CORS issue. Install and follow instructions of this laravel package https://github.com/barryvdh/laravel-cors


copy expanded launch.json command

Provide an easy way to copy the current launch.json command to clipboard so I could paste in terminal to run the desired launch without starting the debugging and show that top panel.

A launch configuration cannot be executed in a terminal directly. A debug extension must transform this into something executable first.
But you can run launch configuration without debugging with the "Start without Debugging" action (Cntl+F5).

Thanks. I find the floating debug toolbox annoying, is there a way to make it dock somewhere?

I thought vscode generates command line from this configuration, so it might be possible I could get this final commands and save them elsewhere.

Debug extensions generate the command line from the launch configurations (because only they know and understand the specific attributes of a launch config).
Try to set the setting "debug.toolBarLocation" to "docked".

Thanks.


conda env create -f environment.yml error

Ran pip subprocess with arguments:
['F:\ProgramData\Anaconda3\envs\cv\python.exe', '-m', 'pip', 'install', '-U', '-r', 'C:\Users
\Administrator\condaenv.ngu96vho.requirements.txt']
Pip subprocess output:
Collecting git+https://github.com/youngpark/cocoapi.git#subdirectory=PythonAPI (from -r C:\Users\Adm
inistrator\condaenv.ngu96vho.requirements.txt (line 5))
Cloning https://github.com/youngpark/cocoapi.git to c:\users\admini~1\appdata\local\temp\2\pip-req
-build-rn1luv66
Pip subprocess error:

@youngpark Any idea what might be causing this?

Same problem here:
Ran pip subprocess with arguments:
['C:\Users\masdc\anaconda3\envs\cv\python.exe', '-m', 'pip', 'install', '-U', '-r', 'C:\Users\masdc\source\repos\MicrosoftComputerVision\ComputerVision\condaenv.7a4lu6w5.requirements.txt']
Pip subprocess output:
Collecting git+https://github.com/youngpark/cocoapi.git#subdirectory=PythonAPI (from -r C:\Users\masdc\source\repos\MicrosoftComputerVision\ComputerVision\condaenv.7a4lu6w5.requirements.txt (line 5))
Cloning https://github.com/youngpark/cocoapi.git to c:\users\masdc\appdata\local\temp\pip-req-build-v9ez03ib
Pip subprocess error:
Running command git clone -q https://github.com/youngpark/cocoapi.git 'C:\Users\masdc\AppData\Local\Temp\pip-req-build-v9ez03ib'
fatal: unable to access 'https://github.com/youngpark/cocoapi.git/': Could not resolve host: github.com
ERROR: Command errored out with exit status 128: git clone -q https://github.com/youngpark/cocoapi.git 'C:\Users\masdc\AppData\Local\Temp\pip-req-build-v9ez03ib' Check the logs for full command output.

I was able to resolve the issue, as this is a Window Only problem:
Install Visual C++ 2015 Build Tools from [(https://go.microsoft.com/fwlink/?LinkId=691126)] with default selection.
Go to C:\Program Files (x86)\Microsoft Visual C++ Build Tools and run vcbuildtools_msbuild.bat
execute this command to erase the conda env, created when you tried to install the repository:
conda remove --name cv --all
If you want to start again from scratch , erase the complete directory where you downloaded the repo and start again with the installation process, as the documentation describes.
If you don't want to execute all the commands again to start a clean install, go to the directory where you downloaded the repository. In my case is: C:\Users\masdc\source\repos\MicrosoftComputerVision\ComputerVision
execute this command:
pip install git+"https://github.com/youngpark/cocoapi.git#subdirectory=PythonAPI"
This line is the one that was presenting error, and is the last one of the environment.yml file.
With this you will be able to install pycocotools.
Regards,
Millán


Set ctime/mtime on tracks/playlists

Use case: I want to create a backup of the music library using some variant of ls -alR ./mounted-spotifile/playlists | grep -v unwanted_meta_files > my_music_library.txt. Having access to the spotify data of when the playlist was created (A) and when a track was added to it (B) would be very nice, especially in the form of custom ctime/mtime values.
(A) is easy enough, there exists a sp_playlist_create_time(sp_playlist *playlist, int index) callback. I have not found anything for (B) yet.
Ctime/mtime is set by spotifile in spfs_entity_create()

Perhaps this issue is moot and a proper solution would be to export all playlists in XSPF (see #37 and http://www.xspf.org/xspf-v1.html#rfc.section.<IP_ADDRESS>.8)...

Exposing metadata such as creation time through stat() is exactly the kind of thing I want to do, especially when the mapping is straightforward. One key project goal is to make scripting of your music collection simple.
As a matter of fact, tracks in playlists already expose when they were added to a given playlist through their ctime. Is that what you're after in (B)?

As a matter of fact, tracks in playlists already expose when they were added to a given playlist through their ctime. Is that what you're after in (B)?

That is correct, (B)'s already implemented. Nice!
I guess that #15 is also required for the ls playlists > my_library.txt which requires some "breaking" changes. I'll look at (A).


Implementation of pandas submodule in components.guides module

The goal is to implement given submodule within the scope of components.guides module and prepare the methods that provide interface for translation. The following objectives are to achieve:

[x] preparation of en.py submodule with bases classes;

[x] preparation of pl.py submodule with inheritance;

The code was checked with the following code:
from dynpy.utilities.documents.guides import IntroToPandasGuide
IntroToPandasGuide(None)


Loading Button Animation

This is a(n):

[ ] Update to an existing Animated Button
[ ] Any Error
[x] Proposal to the Repository
[ ] Documentation / README.md changes

Details:
I want to build a button loading animation using HTML & CSS.

@Spyware007 Please assign me this usse

Hi, are you still working on this issue? If you've already made a PR with the changes that have been merged then please close the issue. If you have an open PR, please mention this issue number in your PR in order to link them.

Yes PR is already merged. Okay, I'll close the issue.


rkt triggers a bug in Docker: no space left on device (cpuset cgroup)

How to reproduce:

Boot the computer.
Start a rkt pod from a systemd unit file.
Start a Docker container from the command line. Docker fails to start a container with the following error:

> docker run --rm -ti busybox
Error response from daemon: Cannot start container ca65deb2d7972d5a1772023ad82d843bb639fcfea9d75e78e0f99fcad9e683ba: [8] System error: write /sys/fs/cgroup/cpuset/system.slice/docker-ca65deb2d7972d5a1772023ad82d843bb639fcfea9d75e78e0f99fcad9e683ba.scope/cgroup.procs: no space left on device

This issue was reported in https://github.com/coreos/bugs/issues/420 and https://github.com/opencontainers/runc/issues/133 and the fix will be in Docker 1.8.
But in order to play nice with Docker < 1.8, rkt could be modified to create the cgroups in the way that Docker < 1.8 expects them. This would add code near os.MkdirAll(appCgroup, 0755) in https://github.com/coreos/rkt/blob/master/common/cgroup/cgroup.go#L259
to set up the cgroup in a similar way to ensureParent in https://github.com/opencontainers/runc/blob/master/libcontainer/cgroups/fs/cpuset.go#L87
/cc @ecnahc515 @kayrus @iaguis

@alban what's the latest thinking on this one?

To fix the issue in the Docker side, we need both:

a Docker fix https://github.com/docker/libcontainer/pull/642 (in Docker 1.8)
a Linux fix https://github.com/torvalds/linux/commit/24ee3cf89bef04e8bc23788aca4e029a3f0f06d9 (merged in mainline, will be in Linux 4.2)

If all Linux distributions had those two fixes, there would be no need to patch rkt. But I think it would be nice to have the aforementioned workaround in rkt for the benefit of Linux distributions with either Docker < 1.8 without backport or Linux < 4.2 without backport. That work in rkt has not been started.

Thanks for the info. It's unfortunate but I think we should add the workaround here so that rkt will work seamlessly in as many places as possible.


ci: add go mod linter to lint target

Signed-off-by: bitliu<EMAIL_ADDRESS>After #743.

@Xunzhuo #743 has merged so feel free to update this PR.

@Xunzhuo LGTM after a rebase.

Codecov Report

Merging #744 (540cf79) into main (a7d1c9d) will decrease coverage by 0.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #744      +/-   ##
==========================================
- Coverage   61.41%   61.27%   -0.15%
==========================================
  Files          46       46
  Lines        5653     5653
==========================================
- Hits         3472     3464       -8
- Misses       1954     1961       +7
- Partials      227      228       +1

Impacted Files
Coverage Δ

internal/provider/kubernetes/controller.go
50.58% <0.00%> (-1.56%)
:arrow_down:

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


[analyzer] add async fix for return_of_invalid_type_from_closure

With

void g(Future<String> Function() a) {}

void f() {
  g(() => '');
}

there is an error of:

The return type 'String' isn't a 'Future', as required by the closure's context.

It would be better to offer a fix, to add async, so that we have
  g(() async => '');

A similar case for return_of_invalid_type:

Future<String> futureString() => '';

I agree, those are reasonable fixes to offer.

Here is my CL https://dart-review.googlesource.com/c/sdk/+/394361

Can I get a new review for my CL @bwilkerson? Thanks!


screen tearing

Screen looks teared while using vscode (not all the time) can be temporarily fixed by clicking/dragging the area but it tears again, later on. Not related to installed themes (happens with the default themes as well). No apparent way to reproduce the bug (sometimes alt+tab reproduces it).
Edit: moving the cursor all over the editor CAN reproduce this.
Device Drivers are up to date.

/duplicate of #117491
See https://github.com/microsoft/vscode/issues/117491#issuecomment-785054233 for links to comments with possible workarounds.


Implement terminal provider

This PR implements the provider which allows other Atom packages to launch external programs in a terminal.

Dear @jeremyramin,
We are trying to use terminal-plus for our PlatformIO https://github.com/platformio/platformio-atom-ide. We need some extra functionality to control Terminal from our package. Could you merge this PR?
Regards Ivan,
PlatformIO Team
http://platformio.org

+1
I also want to use terminal-plus from my package and implementing services would be the best way to do it

@dcalogue Take a look at our fork https://github.com/platformio/platformio-atom-ide-terminal
We implemented API and use it for http://platformio.org/platformio-ide


add fallback user name

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[x] Bugfix
[ ] Feature
[ ] Refactoring

What's new?

[ ] PR closes #<issue_number>
[ ] <brief_description_of_what_I've_added>
in profile page

the address show undefined when have not add on-chain recognition.
for example: https://kodadot.xyz/rmrk/u/ErQqbvyhYWe4h4GiK6eUhNsfBpemVZU8jkNPRp5ZEz5XpbN
no identity:

has identity:

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[x] I've tested PR on mobile and everything seems works
[ ] I found edge cases
[ ] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

pay 50 usd

😍 Perfect, I’ve sent the payout
💵 $50 @ 194.05 USD/KSM ~ 0.258 $KSM
🧗 Caiv9TbPz68q5dC8EcHu5xKYPRnremimGzqmEejDFNpWWLG
🔗 0x684bea4d06b4d40b2c247fb5ca268d69ee22d5eaeb67acce033b674cfdde4e62
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


ocamlPackages.zarith: 1.9 -> 1.10

Motivation for this change
New version, fixes some serious bugs of the previous one.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Running nix-review gives:
2 packages failed to build:
ocamlPackages.irmin-unix ocamlPackages.nocrypto

38 packages built:
alt-ergo dune-release framac fstar google-drive-ocamlfuse jackline libbap ocamlPackages.asn1-combinators
ocamlPackages.cohttp-lwt-unix ocamlPackages.conduit-lwt-unix ocamlPackages.cryptokit ocamlPackages.curly
ocamlPackages.eliom ocamlPackages.fiat-p256 ocamlPackages.gapi_ocaml ocamlPackages.git ocamlPackages.git-http
ocamlPackages.git-unix ocamlPackages.graphql-cohttp ocamlPackages.irmin-git ocamlPackages.irmin-graphql
ocamlPackages.irmin-http ocamlPackages.jwto ocamlPackages.mirage-crypto-pk ocamlPackages.ocsigen-start
ocamlPackages.ocsigen-toolkit ocamlPackages.ocsigen_server ocamlPackages.opium ocamlPackages.otr
ocamlPackages.prof_spacetime ocamlPackages.tls ocamlPackages.x509 ocamlPackages.zarith opa python27Packages.bap
python37Packages.bap python38Packages.bap why3

Where can one see the logs Théo?

Note sure if this was some transient error, but when trying to build those packages manually, I didn't get any failure.

Ok thanks!

@GrahamcOfBorg build alt-ergo framac fstar jackline

Merged into 20.09 as 4c1ae47e8cc96a7682f269c10b425d6b6c3fa413


How to recover from failed deploy when using ASG lifecycle scripts?

We just started using CodeDeploy with ASG and ELB, and we achieve zero downtime when we use register_with_elb.sh and deregister_from_elb.sh. Today I simulated a situation where a deploy step before deregister_from_elb.sh fails. In that case, the server instance in ASG stays in Standby mode and the capacity of ASG is decreased by one because deregister_from_elb.sh is never executed.
Currently, I have to manually increase the ASG capacity back to normal and move the server instance out of Standby. This isn't acceptable. Is there a way to do proper cleanup automatically after a failed deploy?
The only way that came to my mind is to add error handling logic to every deploy script I run between register_with_elb.sh and deregister_from_elb.sh scripts.

Not sure if this helps, but I've also experienced this issue in the past when a code revision is cancelled or fails.

@yyolk Interesting! I think that CodeDeploy should have a more general mechanism for running a cleanup script after an error in a deploy.
Actually that kind of functionality could be implemented to aws-codedeploy-agent. I could do a feature request there or even do an experimental pull request if it would help to prioritize the feature.
A remotely similar mechanism is a rescue block in Ansible:
http://docs.ansible.com/ansible/playbooks_blocks.html

Hi,
I have added a cleanup hook in our feature requests.
Thanks,
Amartya

is there a reason you can't run deregister_from_elb.sh firstly in your ApplicationStart, guaranteeing that it happens first? did you mean register_from_elb.sh?


How to change the default editor?

I am having a problem with the ACE editor because of its handling of diacritics. It doesn't do it properly. If you want to test, you can try हा मजकूर देवनागरी लिपीत स्वरांशचिन्हांसह लिहिण्यात आला आहे. in any of the ACE window. This is not just the case with Devanagari script, it will happen with IPA too. e.g. Try t̪ t̪ʰə d̪ə d̪ʱə n̪ə. ACE developers are probably working on it, but it seems like this issue has been reported almost ten years ago (see: https://github.com/ajaxorg/ace/issues/460) to them, but there is no concrete development till now. On that thread I found this version which works fine, but probably it isn't released yet. https://brionv.com/misc/demo/ace/build/kitchen-sink.html
OTOH I tried https://codemirror.net/ which is also an MIT licensed editor. It handles both of the above examples exactly as expected. For my needs it suits perfect. LaTeX needs to handle multiple scripts. I would appreciate if you can change the default editor to codemirror. If this is not a possibility; you can provide a switch to change the editor, it would also be pretty amazing!
Thanks 😄

Thanks for the report, I have looked in to this a bit and it seems getting a fixed ACE may be problematic,
Using codemirror must be possible in principle (also do you know if  monoco https://microsoft.github.io/monaco-editor/  is OK ?)
But currently the ACE API is used all over for setting up the page and extracting the currently edited text when you click the submit button.
So it will take a bit of time to work out (a) how to make another editor work at all and (b) think about how best to parameterise things so that you can switch editor without breaking all the current configuration points.
As far as I can see Overleaf (which also uses ACE) has the same issue?

Using codemirror must be possible in principle (also do you know if monaco https://microsoft.github.io/monaco-editor/ is OK?)

Monaco editor also seems to be good, but it says the following on it's homepage.

The Monaco editor is not supported in mobile browsers or mobile web frameworks.

So this might be a problem.

So it will take a bit of time to work out (a) how to make another editor work at all and (b) think about how best to parameterise things so that you can switch editor without breaking all the current configuration points.

I agree. There is no need to hurry, take your time.

As far as I can see Overleaf (which also uses ACE) has the same issue?

Yes, I couldn't use Overleaf for my MA thesis because of this reason :disappointed:

@niruvt  can you try
https://davidcarlisle.github.io/latexcgi/test-codemirror
I put you Marathi phrase in one of the examples at this page.
https://davidcarlisle.github.io/latexcgi/test-codemirror
Currently this is implemented as a separate runlatex-codemirror.js script but that is temporary it should be possible to merge this back in to runlatex.js  and make the editor an option, but can you see if it does what you expect?

Yes, it does what I want :smile:
Thanks

I had it working in codemirror5 but was trying to get it working for codemirror6 the new version (oficially in test release) but that seemed harder to set up and I ran out of time.  I a trying to structure runlatex.js so it can work with any editor that provides a minimal shared interface  so I may go with ace and cm5 initially and then look to add cm6 support later as I don't think I have any time to sort out cm6 at present. Sorry it's taking a while.


Implement np.trace.

Also add code to forward np.{iinfo,finfo,issubdtype,integer} to their original numpy equivalents (see #70 )
Fixes #29.

I think it's hard to answer that question in general; it depends on the hardware and on what XLA does with it. I suspect reduction is faster that matmul or gather on current XLA backends. But we can always change this later (or perhaps add XLA ops, to, say. extract matrix diagonals and let XLA decide how to do it.)


Fix for StringProperty which caused wrong dirtiniess reporting

Adding missing states for dirtiness reporting mechanism in StringProperty. This caused apps  not getting notified when a StringProperty is modified when it should.

@ruiterr @evaliyev  PTAL

@DLehenbauer Can you please merge this PR?


Move to Sort component, change routing

Who is this for?
Devs
What is it doing for them?
Moves to using the same Sort component across the site.
Closes #8912

Closing this as won't do


grpc: upstream disconnect test

Adding a test which apparently doesn't repro #13831

@yanavlasov ping?


Refactor transformer

Codecov Report

Merging #255 (e403678) into master (08d5982) will decrease coverage by 0.00%.
The diff coverage is 98.36%.

@@            Coverage Diff             @@
##           master     #255      +/-   ##
==========================================
- Coverage   91.64%   91.63%   -0.01%
==========================================
  Files          57       58       +1
  Lines        4235     4245      +10
  Branches      515      517       +2
==========================================
+ Hits         3881     3890       +9
- Misses        354      355       +1

Impacted Files
Coverage Δ

src/ts-ast-util/utilily-functions.ts
96.33% <97.87%> (+1.14%)
:arrow_up:

src/transformer/index.ts
100.00% <100.00%> (ø)

src/transformer/transformer-host.ts
95.04% <100.00%> (ø)

src/transformer/transformer.ts
93.42% <100.00%> (-2.28%)
:arrow_down:

src/webpack/plugin.ts
100.00% <100.00%> (ø)


tooltip on hover can't be removed

Describe the bug
It looks like the tooltip-on-hover feature for scatter marks is always activated, even when we don't want it.
To Reproduce
With a notebook consisting of the following code:
import numpy as np
from bqplot import pyplot as plt
import bqplot

n = 200
x = np.linspace(0.0, 10.0, n)
y = np.cumsum(np.random.randn(n)*10).astype(int)

fig2 = plt.figure( title='Line Chart')
np.random.seed(0)
p = plt.scatter(x, y)
fig2.layout.width = 'auto'
fig2.layout.height = 'auto'
fig2.layout.min_height = '300px' # so it shows nicely in the notebook
fig2

p.interactions = {'click': 'select'}
p.selected_style = {'opacity': 1.0, 'fill': 'DarkOrange', 'stroke': 'Red'}
p.unselected_style = {'opacity': 0.5}
p.tooltip = bqplot.Tooltip(fields=['x', 'y'], formats=['', '.2f'])

We can see that the tooltip on hover works, even though we didn't specify it in the interactions. For it to work we should have:
p.interactions = {'click': 'select', 'hover': 'tooltip'}

Which bqplot version were you using?
I cannot reproduce this issue on master

I just tried with master, the tooltip on hover is activated. You don't see any tooltip on hover?

Not if I don't execute p.interactions = {'click': 'select', 'hover': 'tooltip'}

That's strange, my version is 0.12.0a2.

Indeed.. Are you using JLab or the Classical Notebook?

The classic Notebook.
Actually it looks like this line activates the tooltip on hover:
p.tooltip = bqplot.Tooltip(fields=['x', 'y'], formats=['', '.2f'])
Even though we only have this (no tooltip on hover interaction):
p.interactions = {'click': 'select'}

Oooh I see now. I was running all of this in a single cell:
Now I can reproduce the issue when running this in two separate cells:
import numpy as np
from bqplot import pyplot as plt
import bqplot

n = 200
x = np.linspace(0.0, 10.0, n)
y = np.cumsum(np.random.randn(n)*10).astype(int)

fig2 = plt.figure( title='Line Chart')
np.random.seed(0)
p = plt.scatter(x, y)
fig2.layout.width = 'auto'
fig2.layout.height = 'auto'
fig2.layout.min_height = '300px' # so it shows nicely in the notebook
fig2

p.interactions = {'click': 'select'}
p.selected_style = {'opacity': 1.0, 'fill': 'DarkOrange', 'stroke': 'Red'}
p.unselected_style = {'opacity': 0.5}
p.tooltip = bqplot.Tooltip(fields=['x', 'y'], formats=['', '.2f'])

Weird that the behavior is different when you run it in two separate cells. I'll investigate a bit on this.

Yes sorry I didn't format my example well.
That's a strange behavior indeed.

It was a mistake, I forgot to remove it. I thought that removing it from the interactions was enough.
But it's not a big deal to leave things like that.

Right, sorry I'm still learning how bqplot works/should work.
Now I think I understand. If the tooltip is not bound to any mouse interaction it should never be displayed, is that it?

Yes, that's how I see it too.

I found the issue. There is a collision in method names... The reset_hover method in Mark.ts which is supposed to be used for resetting the hover event is overridden by the reset_hover in ScatterBase.ts which is used for resetting the currently hovered points.
This results in the hover event to not be removed when doing p.interactions = {'click': 'select'}

Great, and is it fixable?

Sure, PR coming :)

Thanks!


RDS - change ClusterSnapshot creationtime to UTC

Fixes #5355

Codecov Report

Merging #5387 (bf15af6) into master (3d913f8) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #5387      +/-   ##
==========================================
- Coverage   96.05%   96.05%   -0.01%
==========================================
  Files         708      708
  Lines       71000    71000
==========================================
- Hits        68200    68199       -1
- Misses       2800     2801       +1

Flag
Coverage Δ

servertests
35.83% <0.00%> (ø)

unittests
95.99% <100.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

moto/rds/models.py
94.00% <100.00%> (ø)

moto/awslambda/models.py
93.91% <0.00%> (-0.11%)
:arrow_down:

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


User folder in /nix/var/nix/profiles/per-user not created

Hi there!
I am currently building a custom Live OS and I use the Home Manager to load some files in the home folder. And... it failed :(
After some investigation, I discovered that the service home-manager-myuser.service failed to start because of the missing folder /nix/var/nix/profiles/per-user/myuser. I had the same bug when I installed NixOS for the first time.
For now, I used a workaround to create the folder before the home-manager-myuser.service launches. But I'm not satisfied with this solution.  Not the best approach :p
For those of you annoyed by this issue, I paste my workaround. Make sure you changed myuser with the appropriate username. Cheers guys!
systemd.services."before-home-manager-myuser" = {
    script = "mkdir -p /nix/var/nix/profiles/per-user/myuser";
    path = [ pkgs.coreutils ];
    before = [ "home-manager-myuser.service" ];
    wantedBy = [ "multi-user.target" ];
};

For reference, this was introduced with https://github.com/NixOS/nixpkgs/commit/4e0d6a5ff8157e6aacb1ef0b948d0b7dca36341f


Compatibility issues with django 2.2

Problem
I have been using two_factor with Django 2.1 for a while without any problems (thank you for all your work!). However, I recently upgraded to Django 2.2 and am having some compatibility issues. Whilst I am still able to create PhoneDevice objects, I can now no longer delete them (both within the shell environment and the relevant views e.g. PhoneDeleteView and DisableView). In both cases, I get the same error. For example, if I enter the following in the shell PhoneDevice.objects.first().delete() I get the following error:
Traceback (most recent call last):
  File "<console>", line 1, in <module>
  File "/home/iwilliams/.virtualenvs/twofactor/lib/python3.6/site-packages/django/db/models/base.py", line 918, in delete
    collector.collect([self], keep_parents=keep_parents)
  File "/home/iwilliams/.virtualenvs/twofactor/lib/python3.6/site-packages/django/db/models/deletion.py", line 192, in collect
    reverse_dependency=reverse_dependency)
  File "/home/iwilliams/.virtualenvs/twofactor/lib/python3.6/site-packages/django/db/models/deletion.py", line 94, in add
    if obj not in instances:
TypeError: unhashable type: 'PhoneDevice'

Possible Solution
Part of the problem seems to lie with the following method:
class PhoneDevice(Object):
   ...
    def __eq__(self, other):
        if not isinstance(other, PhoneDevice):
            return False
        return self.number == other.number \
            and self.method == other.method \
            and self.key == other.key

When the above function is commented out, I am able to remove PhoneDevice objects. Perhaps on a related note, I should note that I can add TOTPDevice and YubikeyDevice instances in Django 2.2 without a problem, both of which don't override the base __eq__ method.
Do you have any further insight into what is causing this problem/ how it can be properly resolved?
Thanks in advance!
Environment Info

Python version: 3.6.7
Django version: 2.2
django-otp version: 0.6.0
django-two-factor-auth version: 1.8.0

From https://docs.python.org/3/reference/datamodel.html#object.hash:
if it defines __eq__() but not __hash__(), its instances will not be usable as items in hashable collections.

Hmm, I wonder why that model defines __eq__? Do you know if any test fails if that method is commented out?

I'm not entirely sure how it is used, but it was introduced here: https://github.com/Bouke/django-two-factor-auth/commit/893fb23781d670ad55c0bbd5fd794bf45d27ccdb, which was added to support Django 1.7: https://github.com/django/django/commit/6af05e7a0f0e4604d6a67899acaa99d73ec0dfaa.

tests.test_views_phone.PhoneSetupTest.test_setup fails on line 58 if I remove it (which makes sense because it's asserting that a mock was called with a specific phone device)

Can confirm.  I can not remove phone devices or disable two factor.  Any time I try I receive:

TypeError at /account/two_factor/backup/phone/unregister/2/
unhashable type: 'PhoneDevice'


Realm 1.6.0 conflict with dagger

I have a module(only Realm 1.6.0 implemented) with my Database and RealmObjects,my main program has dagger and implementation of module,with this I no longer can create global variables like:
 class MainActivity : AppCompatActivity() {
    var dog: Dog= Dog()

    override fun onCreate(savedInstanceState: Bundle?) {
        ....
    }
}

it will raise error:
error: cannot access Dog
    private gr.mastersdk.database.model.Dog dog;

undeclared type variable: T
    Please remove or make sure it appears in the correct subdirectory of the classpath.

But the same code will work if I create this entity inside function like:
class MainActivity : AppCompatActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_main)

        var dog = Dog()
    }
}

Not really sure why this conflict exists,but having find any way out of it
dagger that I'm using: 'com.google.dagger:hilt-android:2.41'

Hi @DmytroTols. Thanks for the report. I can reproduce your observation in a sample project, but will have to investigate the cause of the issue.

Sure,thank you for quick response

This only seems to be an issue with incremental builds. If I add the field it fails, but after a clean build it works and continues to do so until next time the properties that references a RealmObject is changed. If I change the source of the referenced class, Doc in the same go then it works. Could be something with the classes not being available on the class path of the kapt task when doing incremental builds. I don't know why this differentiates for classes implementing RealmObject since we transform them inline ... but the output might be placed in a different build intermediate build dir  which is somehow now included in the kapt build path 👀

Seems to be a more general issue as I was able just provoke the same error without hilt in #1256

The fix is now available in 1.6.1.


Fix: sample uses String as key instead of UUID

Old requests are not working anymore
see model:
https://github.com/cap-js/incidents-app/blob/b610b8dbcdf39efe4dee6861149c644d3c5c589a/db/schema.cds#L9-L10
data:
https://github.com/cap-js/incidents-app/blob/b610b8dbcdf39efe4dee6861149c644d3c5c589a/db/data/sap.capire.incidents-Customers.csv
Unfortunately, I wrote Integer in my commit message instead of String :-(

Fixed commit message in https://github.com/cap-js/audit-logging/pull/113


updated spark 2.X version in scripts

Coverage remained the same at 71.648% when pulling cd6927c28570f35f494c31c33d8e5a03865d1d46 on akmorrow13:spark2_1 into 35bf97fc543b2943ab78f83867c6ccc6bef001f3 on bigdatagenomics:master.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/mango-prb/390/
Test PASSed.


Do you have any plan to support the optional shape argument?

Hi,
I am trying to add changes to the TFParser for few more ops (Eg.Fill) but this is not possible as the input shape argument is not Const for this op in my model. Do you have any plans to introduce support for such type of operations?

Hi @chiraggirdhar95 in the current version of ArmNN tensor shapes do need to be constant.
We've been thinking about a way of adding support for dynamically sizing fully-convolutional networks, e.g. for filters, but I don't know whether this would fit with your needs.  Could you share a bit more information about your use case?
All the best,
Matthew

I want to add changes to the Tfparser to support Fill operations(https://www.tensorflow.org/api_docs/python/tf/fill) which takes 2 arguments 1)scalar 2)shape. So the shape argument for the node Fill in my model will be fed from another node's output in the graph. So while adding changes in the TfParser how should I define changes using the current functionalities. For eg how should I set outputTensorInfo which depends on shape argument whose value is not known.

Hi @chiraggirdhar95 if the shape can be calculated statically from the model, you should be fine handling this with code in the parser, by calculating the shape and passing it to the fill node as a constant. If your model genuinely needs ArmNN tensors with dynamic shapes that change based on the input, I'm afraid that can't be supported at this time.

Closing this issue as the original question has been answered. @chiraggirdhar95 if you can let us know more about whether dynamic tensor shapes are required in your use-case, please get back in touch and we'll add it to our roadmap.
Many thanks,
Matthew


Kernels listed selecting interpreter for notebook don't match normal Python interpreter selection (and don't contain my interpreter)

Applies To

[X] Notebooks (.ipynb files)
[ ] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?

Created new Poetry environment dask-pyspy-demo and installed packages, not including jupyterlab
Refreshed window, ran "Python: Select Interpreter", picked dask-pyspy-demo:

Created a notebook, ran "Select notebook kernel".

dask-pyspy-demo does not show up. I would expect it to show up.

poetry add --group dev jupyterlab
Refreshed window again
Ran "Select notebook kernel", dask-pyspy-demo still doesn't show up.

One other thing: when restarting the window, notice how "Select Kernel" flashes a couple times to the interpreter I actually want (dask-pyspy-demo-...), then lands back on "Select Kernel":
https://user-images.githubusercontent.com/3309802/202320529-9f4360b5-c653-4054-9120-90ba855307de.mp4
VS Code Version
Version: 1.73.1 Commit:<PHONE_NUMBER>f055b99068d3688932416f2346dd3b Date: 2022-11-09T02:08:38.961Z (1 wk ago) Electron: 19.0.17 Chromium: 102.0.5005.167 Node.js: 16.14.2 V8: <IP_ADDRESS>-electron.0 OS: Darwin x64 21.6.0 Sandboxed: No
Jupyter Extension Version
v2022.9.1303220346
Jupyter logs
Visual Studio Code (1.73.1, undefined, desktop)
Jupyter Extension Version: 2022.9.1303220346.
Python Extension Version: 2022.18.2.
Workspace folder /Users/gabe/dev/dask-pyspy-demo
info 16:53:05.763: ZMQ install verified.
User belongs to experiment group 'jupyterTestcf'
User belongs to experiment group 'jupyterEnhancedDataViewer'
info 16:53:06.063: LSP Notebooks experiment is disabled -- not in treatment group
info 16:53:06.227: Got env vars ourselves faster /Users/gabe/Library/Caches/pypoetry/virtualenvs/dask-pyspy-demo-6EAlOZQU-py3.9/bin/python with env var count 44 in 137ms
info 16:53:06.229: Process Execution: > ~/Library/Caches/pypoetry/virtualenvs/dask-pyspy-demo-6EAlOZQU-py3.9/bin/python -m pip list
> ~/Library/Caches/pypoetry/virtualenvs/dask-pyspy-demo-6EAlOZQU-py3.9/bin/python -m pip list
info 16:53:06.517: Preferred Remote kernel for /Users/gabe/dev/dask-pyspy-demo/demo.ipynb is undefined
info 16:53:06.517: Find preferred kernel for /Users/gabe/dev/dask-pyspy-demo/demo.ipynb with metadata {"language_info":{"name":"python"},"orig_nbformat":4} & preferred interpreter /Users/gabe/Library/Caches/pypoetry/virtualenvs/dask-pyspy-demo-6EAlOZQU-py3.9/bin/python
info 16:53:06.540: Preferred Remote kernel for /Users/gabe/dev/dask-pyspy-demo/demo.ipynb is undefined
info 16:53:06.540: Preferred kernel .jvsc74a57bd02bc6906351481cef9ac83eabe42cb78287e09e65226f91bd31e6d8b565da21d2./Users/gabe/Library/Caches/pypoetry/virtualenvs/dask-pyspy-demo-6EAlOZQU-py3.9/python./Users/gabe/Library/Caches/pypoetry/virtualenvs/dask-pyspy-demo-6EAlOZQU-py3.9/python.-m#ipykernel_launcher is exact match or top match for non python kernels, (false, [object Object], false, false)
info 16:53:06.541: PreferredConnection: .jvsc74a57bd02bc6906351481cef9ac83eabe42cb78287e09e65226f91bd31e6d8b565da21d2./Users/gabe/Library/Caches/pypoetry/virtualenvs/dask-pyspy-demo-6EAlOZQU-py3.9/python./Users/gabe/Library/Caches/pypoetry/virtualenvs/dask-pyspy-demo-6EAlOZQU-py3.9/python.-m#ipykernel_launcher found for NotebookDocument: /Users/gabe/dev/dask-pyspy-demo/demo.ipynb
info 16:53:06.574: Got env vars with python /Users/gabe/Library/Caches/pypoetry/virtualenvs/dask-pyspy-demo-6EAlOZQU-py3.9/bin/python, with env var count 44 and custom env var count 0 in 483ms
info 16:53:06.746: Preferred Remote kernel for /Users/gabe/dev/dask-pyspy-demo/demo.ipynb is undefined
info 16:53:06.773: Registering dummy command feature

Coding Language and Runtime Version
Python 3.9.4
Language Extension Version (if applicable)
v2022.18.2
Anaconda Version (if applicable)
No response
Running Jupyter locally or remotely?
Local

@gjoseph92
I'm sorry you're running into this and thanks for reporting this issue.
I believe we've addressed this issue in the latest main branch.
Please could you install the latest pre-release version of the Jupyter extension and test this out.
You'd need to isntall VS Code insiders and the pre-release version of Python extension as well.
If the issue still persists, then this is a bug and we'll look into this, however I need a verifiaction that this doesn't exist anymore in our latest pre-release version.
Thanks

Closing this issue as its been over 4 weeks, since the information was requested.
We'll be happy to reopen the issue when the requested information has been provided.


Update docs about quill.query.tooLong

Problem
SQL runtime log messages activated via quill.binds.log are clipped. The quill.query.tooLong option was not documented yet.
Solution
Updated the docs to mention the quill.query.tooLong option in the logging section.
@getquill/maintainers

Thank you for the contribution!


Can I add to a justfile a string substitution/strip in a variable.

So I have a script I want to run in a docker container.  My just command is a docker exec  script
Now  needs to be accessible to the container.  But as it's a variable for the just command I write it
just mycommand /some/dir/host/some/common/dir
Where /some/common/dir will work in the container.
So all I really need to do is to strip off /some/dir/host before executing the script.
This is actually a just command to run a handbrake conversion script in a handbrake container.  The host has access to:
/mnt/media/tv
In the container that becomes
/media/tv
so I just need to strip off the /mnt (which I could do in my conversion script), but hey I'd like to see if just can do it.

Try the trim_start_match function, I think it does what you want. See string manipulation functions in the readme.


Proximity forest docstrings

PF needs docstrings.
Could also do with a tidy up prior to that!

I'm very tempted to assign you to this :)


Change centos image name to skip the GET image error

To avoid below error with prow jobs:
failed to perform Get Image Operation for image centos-8-stream-20gb with error [GET /pcloud/v1/cloud-instances/{cloud_instance_id}/images/{image_id}][500] pcloudCloudinstancesImagesGetInternalServerError  &{Code:0 Description:an error has occurred; please try again: unable to get image 37091d0f-a4f6-42d8-8cb5-8d71b34307c6: Unable to get Image 37091d0f-a4f6-42d8-8cb5-8d71b34307c6: Resource not found: [GET https://<IP_ADDRESS>:9292/v2/images/37091d0f-a4f6-42d8-8cb5-8d71b34307c6], error message: {"message": "No image found with ID 37091d0f-a4f6-42d8-8cb5-8d71b34307c6

\n\n\n", "code": "404 Not Found", "title": "Not Found"} Error:internal server error Message:}

@Rajalakshmi-Girish hope this is a temp fix

@mkumatag we can leave the image name as it is in the future too.
After raising of support ticket and follow-up, we can delete the image with the old image name centos-8-stream-20gb

/woof

/woof

/woof

As jobs are passing with the old image name itself, this change is not needed!

/woof

/woof