Reset inverted labels cache on epoch change

Description
The database cache epoch advances when series are deleted from the
database. When connector detects a cache epoch change, it invalidates
the series cache, as it may contain values which reference the deleted
rows in the series table.
During the database series deletion, unused labels may also be deleted.
This means that we must also invalidate the inverted label cache, as it
could also reference deleted rows in the _prom_catalog.label table.
Merge requirements
Please take into account the following non-code changes that you may need to make with your PR:

[x] CHANGELOG entry for user-facing changes
[ ] Updated the relevant documentation

If unused labels are dropped I think we also need to take care of labels cache (not just inverted labels cache). Check labels_reader.go

That is correct, but stale entries in the labels cache are much less problematic so we decided to only address the inverted labels cache for now.


It's impossible to add new lines by releasing a document.

Issue: It's impossible to add new (valid) lines by releasing an order or an invoice.
If we do a manual release before posting it's all ok.
If we post an invoice, CU80 calls this internal function:
LOCAL ReleaseSalesDocument(VAR SalesHeader : Record "Sales Header")
inside this function it checks a return value to update its internal "TempLines":
  LinesWereModified := ReleaseSalesDocument.ReleaseSalesHeader(SalesHeader,PreviewMode);
  IF LinesWereModified THEN
    RefreshTempLines(SalesHeader);

If we subscribe CU414.OnBeforeReleaseSalesDoc(VAR SalesHeader : Record "Sales Header";PreviewMode : Boolean) we cannot change the return value "LinesWereModified" because parameter has not passed.
If we subscribe CU414.OnAfterReleaseSalesDoc(VAR SalesHeader : Record "Sales Header";PreviewMode : Boolean;LinesWereModified : Boolean) LinesWereModified parameter is not "VAR".
So we can add new lines to a sales document but CU80 will not posts them.
Same problem on Purchase side (CU90&CU415)
Tested on NAV2018CU3

Hi,
I will pass your feedback on to the relevant team.
br/Gert

+1

Is it OK for you if I will make LinesWereModified  as VAR in this event? CU414.OnAfterReleaseSalesDoc(VAR SalesHeader : Record "Sales Header";PreviewMode : Boolean;LinesWereModified : Boolean)


Add logging to DeleteCommand

Codecov Report

Merging #99 (2ecd0d3) into master (3e82ef2) will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master      #99      +/-   ##
============================================
+ Coverage     74.30%   74.37%   +0.07%
- Complexity      483      484       +1
============================================
  Files            81       81
  Lines          1432     1436       +4
  Branches        163      163
============================================
+ Hits           1064     1068       +4
  Misses          322      322
  Partials         46       46

Impacted Files
Coverage Δ

...va/seedu/address/logic/commands/DeleteCommand.java
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Typo

Why all class names in events are called ChannelCreate. I think this should be retyped
Example :
https://github.com/Derpinou/discord-antiraid/blob/main/src/events/guild/guildBanRemove.ts

Copy pasta problems, I´ve didn't notice that, if you want fix this, you can fork and open a pull request, thanks for this report


Sails.js, MongoDb: timeout when trying to connect; possible to use the connection pool?

Sails version: 1.0.0-36
Node version: 6.10.2
NPM version: 3.10.10
Operating system: Windows 10

Related StackOverflow question: https://stackoverflow.com/questions/45198292/sails-js-mongodb-timeout-when-trying-to-connect-possible-to-use-the-connectio
Sails server:

Windows 10 64-bit
Sails.js 1.0.0-38
Node.js 6.10.2

MongoDB server:

Centos 7 on Virtualbox, host networking only
Mongodb 3.4.4, listening to all interfaces, no auth configured

I can successfully lift Sails, connect the datastore, and use the models without any problem.
However, this test will result on a timeout:
  let mongo = require('mongodb');

  let uri = 'mongodb://'  + process.env.MONGO_SERVER + ':' + process.env.MONGO_PORT + '/sails';

  mongo.MongoClient.connect(uri, function(err, db) {
    if(err){

      return res.serverError(err);
    }
    return res.json("open");

  });

{ MongoError: failed to connect to server [<IP_ADDRESS>:27027] on first connect [MongoError: connect ETIMEDOUT <IP_ADDRESS>:27027]
    at Pool.<anonymous> (....\node_modules\mongodb-core\lib\topologies\server.js:329:35)
    at emitOne (events.js:96:13)
    at Pool.emit (events.js:188:7)
    at Connection.<anonymous> (....\node_modules\mongodb-core\lib\connection\pool.js:280:12)
    at Connection.g (events.js:291:16)
    at emitTwo (events.js:106:13)
    at Connection.emit (events.js:191:7)
    at Socket.<anonymous> (....\node_modules\mongodb-core\lib\connection\connection.js:187:49)
    at Socket.g (events.js:291:16)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:188:7)
    at emitErrorNT (net.js:1281:8)
    at _combinedTickCallback (internal/process/next_tick.js:80:11)
    at process._tickDomainCallback (internal/process/next_tick.js:128:9)
  name: 'MongoError',
  message: 'failed to connect to server [<IP_ADDRESS>:27027] on first connect [MongoError: connect ETIMEDOUT <IP_ADDRESS>:27027]' }

I am using the exact same server information used on the datastore.
So what could be wrong here?
And as important: is it possible to use the global Sails connection pool instead of creating a new connection? How? I have searched the documentation but cannot find specific references.

Closing this issue because I was definitely wrong. I was using port 27027 instead of 27017. Datastore worked because there was a fallback to the correct port.


[BUG] Unable to install UnigetUI with scoop

Please confirm these before moving forward

[X] I have searched for my issue and have not found a work-in-progress/duplicate/resolved issue.
[X] I have tested that this issue has not been fixed in the latest (beta or stable) release.
[X] I have checked the FAQ section for solutions.
[X] This issue is about a bug (if it is not, please use the correct template).

UniGetUI Version
3.1.2
Windows version, edition, and architecture
Microsoft Windows 11 Home, 10.0.22631, x64
Describe your issue
When trying to install UnigetUI from the extras bucket on Scoop, the installer fails because innounp is unable to extract the files.
innounp.log
(I had also tried to extract v3.1.2 installer with innounp v0.50 and it failed with the following error message The setup files are corrupted or made by incompatible version. Maybe it's not an Inno Setup installation at all. (00492DAA))
I had looked through the build_release.bat and I'm not sure which version of innounp UnigetUI is built with if that could be the issue.
Steps to reproduce the issue
scoop bucket add extras
scoop install extras/wingetui

UniGetUI Log
[2024-10-21 16:35:43]    __  __      _ ______     __  __  ______
                        / / / /___  (_) ____/__  / /_/ / / /  _/
                       / / / / __ \/ / / __/ _ \/ __/ / / // /
                      / /_/ / / / / / /_/ /  __/ /_/ /_/ // /
                      \____/_/ /_/_/\____/\___/\__/\____/___/
                          Welcome to UniGetUI Version 3.1.2
[2024-10-21 16:35:43]
[2024-10-21 16:35:43] Version Code:  3.12
[2024-10-21 16:35:43] Encoding Code Page set to 65001
[2024-10-21 16:35:44] Using bundled GSudo at C:\Users\legor\AppData\Local\Programs\UniGetUI\Assets\Utilities\gsudo.exe
[2024-10-21 16:35:46] Loaded language locale: en
[2024-10-21 16:35:48] Randomly-generated background API auth token for the current session: s0jbmstjye1jpqqs2n6yvc74wee34p5w1bucixtwob46mihgmt27uw99766x3cid
[2024-10-21 16:35:48] Api running on http://localhost:7058
[2024-10-21 16:35:48] Lang files were updated successfully from GitHub
[2024-10-21 16:35:50] UniGetUI is up to date
[2024-10-21 16:35:50] Downloaded new icons and screenshots successfully!
[2024-10-21 16:35:52] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: Winget
                      █ Enabled: True
                      █ Found: True
                      █ Fancy exe name: winget.exe
                      █ Executable path: C:\Users\legor\AppData\Local\Microsoft\WindowsApps\winget.exe
                      █ Call arguments:
                      █ Version:
                      █   System WinGet CLI Version: v1.8.1911
                      █   Using Native WinGet helper (COM Api)
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[2024-10-21 16:35:52] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: Cargo
                      █ Enabled: True
                      █ Found: True
                      █ Fancy exe name: cargo.exe
                      █ Executable path: C:\Users\legor\scoop\apps\rustup-gnu\current\.cargo\bin\cargo.exe
                      █ Call arguments:
                      █ Version:
                      █   cargo 1.78.0 (54d8815d0 2024-03-26)
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[2024-10-21 16:35:53] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: Npm
                      █ Enabled: True
                      █ Found: True
                      █ Fancy exe name: npm
                      █ Executable path: C:\WINDOWS\system32\windowspowershell\v1.0\powershell.exe
                      █ Call arguments:  -NoProfile -ExecutionPolicy Bypass -Command npm
                      █ Version:
                      █   10.9.0
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[2024-10-21 16:35:53] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: .NET Tool
                      █ Enabled: True
                      █ Found: True
                      █ Fancy exe name: dotnet tool
                      █ Executable path: C:\Program Files\dotnet\dotnet.exe
                      █ Call arguments: tool
                      █ Version:
                      █   8.0.403
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[2024-10-21 16:35:53] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: Pip
                      █ Enabled: False
                      █ THE MANAGER IS DISABLED
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[2024-10-21 16:35:53] Starting scoop cleanup...
[2024-10-21 16:35:53] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: Chocolatey
                      █ Enabled: False
                      █ THE MANAGER IS DISABLED
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[2024-10-21 16:35:55] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: PowerShell
                      █ Enabled: True
                      █ Found: True
                      █ Fancy exe name: powershell.exe
                      █ Executable path: C:\WINDOWS\system32\windowspowershell\v1.0\powershell.exe
                      █ Call arguments:  -NoProfile -Command
                      █ Version:
                      █   Name                           Value
                      █   ----                           -----
                      █   PSVersion                      5.1.22621.4249
                      █   PSEdition                      Desktop
                      █   PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
                      █   BuildVersion                   10.0.22621.4249
                      █   CLRVersion                     4.0.30319.42000
                      █   WSManStackVersion              3.0
                      █   PSRemotingProtocolVersion      2.3
                      █   SerializationVersion           <IP_ADDRESS>
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[2024-10-21 16:35:56] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: Scoop
                      █ Enabled: True
                      █ Found: True
                      █ Fancy exe name: scoop
                      █ Executable path: C:\WINDOWS\system32\windowspowershell\v1.0\powershell.exe
                      █ Call arguments:  -NoProfile -ExecutionPolicy Bypass -Command scoop
                      █ Version:
                      █   Current Scoop version:
                      █   859d1db5 chore(release): Bump to version 0.5.2 (#6080)
                      █
                      █   'main' bucket:
                      █   c5d09c1bf temporal-cli: Update to version 1.1.1
                      █
                      █   'extras' bucket:
                      █   df775a156 youtrack: Update to version 2024.3.47707
                      █
                      █   'sysinternals' bucket:
                      █   6b2b695 sysmon: Update to version 15.15
                      █
                      █   'nerd-fonts' bucket:
                      █   c6d9e8d2 SarasaGothic: Update to version 1.0.22
                      █
                      █   'java' bucket:
                      █   5620947d microsoft21-jdk: Update to version 21.0.5
                      █
                      █   'games' bucket:
                      █   f1ac08a37 pcsx2-dev: Update to version 2.1.217
                      █
                      █   'charmbracelet' bucket:
                      █   0d11064 Scoop update for mods version v1.6.0
                      █
                      █   'pillager' bucket:
                      █   99b8b33 Scoop update for pillager version v0.8.1
                      █
                      █   'rsteube' bucket:
                      █   0fbd71d Scoop update for carapace-bin version v1.0.7
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[2024-10-21 16:35:56] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: PowerShell7
                      █ Enabled: True
                      █ Found: True
                      █ Fancy exe name: pwsh.exe
                      █ Executable path: C:\Users\legor\scoop\shims\pwsh.exe
                      █ Call arguments:  -NoProfile -Command
                      █ Version:
                      █   PowerShell 7.4.5
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[2024-10-21 16:35:56] LoadComponentsAsync finished executing. All managers loaded. Proceeding to interface.
[2024-10-21 16:35:58] Scoop cleanup finished!
[2024-10-21 16:35:59] Found 11 installed packages from Cargo
[2024-10-21 16:35:59] Found 0 available updates from .NET Tool
[2024-10-21 16:36:00] Found 5 installed packages from .NET Tool
[2024-10-21 16:36:00] Dependency Scoop-Search for manager Scoop is present
[2024-10-21 16:36:02] Found 4 available updates from Cargo
[2024-10-21 16:36:03] Ignored updates database was never loaded, so it is going to be loaded now
[2024-10-21 16:36:03] Found 0 available updates from PowerShell
[2024-10-21 16:36:03] Dependency Git for manager Scoop is present
[2024-10-21 16:36:03] Found 0 installed packages from PowerShell
[2024-10-21 16:36:04] Dependency cargo-update for manager Cargo is present
[2024-10-21 16:36:04] Found 207 installed packages from Winget
[2024-10-21 16:36:04] Dependency .NET Tools Outdated for manager .NET Tool is present
[2024-10-21 16:36:07] Found 4 installed packages from PowerShell7
[2024-10-21 16:36:09] Found 1 available updates from Npm
[2024-10-21 16:36:10] Found 199 installed packages from Scoop
[2024-10-21 16:36:11] Found 2 available updates from Winget
[2024-10-21 16:36:12] Found 15 installed packages from Npm
[2024-10-21 16:36:12] Backup saved to C:\Users\legor\.config\packages\EHDGE installed packages.ubundle
[2024-10-21 16:36:14] Found 199 installed packages from Scoop
[2024-10-21 16:36:19] Found 0 available updates from PowerShell7
[2024-10-21 16:36:26] Upgradable scoop package not listed on installed packages - id=tableplus
[2024-10-21 16:36:26] Upgradable scoop package not listed on installed packages - id=wingetui
[2024-10-21 16:36:26] Found 5 available updates from Scoop
[2024-10-21 16:36:26] Loaded operation parameters for package id=codeql on manager Scoop and operation Update: update main/codeql
[2024-10-21 16:36:26] Loaded operation parameters for package id=typstyle on manager Cargo and operation Update: install-update typstyle
[2024-10-21 16:36:27] Loaded operation parameters for package id=choose on manager Cargo and operation Update: install-update choose
[2024-10-21 16:36:27] Loaded operation parameters for package id=gimp on manager Scoop and operation Update: update extras/gimp
[2024-10-21 16:36:27] Loaded operation parameters for package id=zeitfetch on manager Cargo and operation Update: install-update zeitfetch
[2024-10-21 16:36:27] Loaded operation parameters for package id=vivid on manager Cargo and operation Update: install-update vivid
[2024-10-21 16:36:27] Loaded operation parameters for package id=Discord.Discord on manager Winget and operation Update: update --id "Discord.Discord" --exact --source winget --accept-source-agreements --disable-interactivity --silent --include-unknown --accept-package-agreements --force
[2024-10-21 16:36:27] Loaded operation parameters for package id=python on manager Scoop and operation Update: update main/python
[2024-10-21 16:36:27] Loaded operation parameters for package id=glzr-io.glazewm on manager Winget and operation Update: update --id "glzr-io.glazewm" --exact --source winget --accept-source-agreements --disable-interactivity --silent --include-unknown --accept-package-agreements --force
[2024-10-21 16:36:27] Loaded operation parameters for package id=sqlite on manager Scoop and operation Update: update main/sqlite
[2024-10-21 16:36:27] Loaded operation parameters for package id=telegram on manager Scoop and operation Update: update extras/telegram
[2024-10-21 16:36:28] Loaded operation parameters for package id=jsdoc on manager Npm and operation Update: install<EMAIL_ADDRESS>jsdoc --global
[2024-10-21 16:36:30] An error occurred while retrieving the icon for package codeql
[2024-10-21 16:36:30] System.Net.Http.HttpRequestException: Response status code does not indicate success: 404 (Not Found).
                         at System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
                         at UniGetUI.Core.IconEngine.IconCacheEngine.GetCacheOrDownloadIcon(Nullable`1 _icon, String ManagerName, String PackageId) in C:\SomePrograms\WingetUI-Store\src\UniGetUI.Core.IconStore\IconCacheEngine.cs:line 143
                         at UniGetUI.PackageEngine.PackageClasses.Package.LoadIconUrlIfAny() in C:\SomePrograms\WingetUI-Store\src\UniGetUI.PackageEngine.PackageManagerClasses\Packages\Package.cs:line 200
[2024-10-21 16:36:37] An error occurred while retrieving the icon for package telegram
[2024-10-21 16:36:37] System.Net.Http.HttpRequestException: Response status code does not indicate success: 404 (Not Found).
                         at System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
                         at UniGetUI.Core.IconEngine.IconCacheEngine.GetCacheOrDownloadIcon(Nullable`1 _icon, String ManagerName, String PackageId) in C:\SomePrograms\WingetUI-Store\src\UniGetUI.Core.IconStore\IconCacheEngine.cs:line 143
                         at UniGetUI.PackageEngine.PackageClasses.Package.LoadIconUrlIfAny() in C:\SomePrograms\WingetUI-Store\src\UniGetUI.PackageEngine.PackageManagerClasses\Packages\Package.cs:line 200
[2024-10-21 19:54:55] Found 0 screenshots for package Id=python
[2024-10-21 19:54:57] Loaded details for package python on manager Scoop
[2024-10-21 19:54:57] Loaded operation parameters for package id=python on manager Scoop and operation Update: update main/python
[2024-10-21 19:54:59] Loaded operation parameters for package id=python on manager Scoop and operation Update: update main/python
[2024-10-21 20:36:03] UniGetUI is up to date
[2024-10-21 20:44:26] Found 0 screenshots for package Id=MartiCliment.UniGetUI.Pre-Release
[2024-10-21 20:44:35] Loaded details for package MartiCliment.UniGetUI.Pre-Release on manager Winget
[2024-10-21 20:44:35] Loaded operation parameters for package id=MartiCliment.UniGetUI.Pre-Release on manager Winget and operation Uninstall: uninstall --id "MartiCliment.UniGetUI.Pre-Release" --exact --source winget --accept-source-agreements --disable-interactivity --version "3.1.2" --silent

Package Managers Logs
Manager WinGet with version:
System WinGet CLI Version: v1.8.1911
Using Native WinGet helper (COM Api)

——————————————————————————————————————————

Logged native task on manager Winget. Task type is ListSources
Process start time: 2024-10-21 16:35:52
Process end time:   2024-10-21 16:35:52

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Logged subprocess-based task on manager Winget. Task type is RefreshIndexes
Subprocess executable: "C:\Users\legor\AppData\Local\Microsoft\WindowsApps\winget.exe"
Command-line arguments: " source update --disable-interactivity"
Process start time: 2024-10-21 16:35:56
Process end time:   2024-10-21 16:35:58

-- Process STDOUT
 ...

Return code: SUCCESS (0)

——————————————————————————————————————————

Logged native task on manager Winget. Task type is ListInstalledPackages
Process start time: 2024-10-21 16:35:56
Process end time:   2024-10-21 16:36:04

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Logged native task on manager Winget. Task type is ListSources
Process start time: 2024-10-21 16:35:58
Process end time:   2024-10-21 16:35:58

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Logged native task on manager Winget. Task type is ListUpdates
Process start time: 2024-10-21 16:36:04
Process end time:   2024-10-21 16:36:11

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Logged native task on manager Winget. Task type is LoadPackageDetails
Process start time: 2024-10-21 20:44:26
Process end time:   2024-10-21 20:44:35

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Logged native task on manager Winget. Task type is LoadPackageVersions
Process start time: 2024-10-21 20:44:26
Process end time:   2024-10-21 20:44:31

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Relevant information
No response
Screenshots and videos
No response

The scoop unigetui package is broken, but is not updated by me, so I can't really fix it.
However, I will try to contact the maintainers to see if I can get it fixed

The main reason is that UnigetUI uses Inno (6.3.0) for packaging, and there is no third-party tool to support unpacking. Also the UnigetUI installer does not support unpacking in the /Dir=$dir /portable way.
I would like to release the .7z package directly as an attachment.

This issue will be fixed once https://github.com/ScoopInstaller/Nonportable/pull/362 is merged


rustfmt produces invalid syntax for struct with unmatched quote in trailing block comment

Describe the bug

This code:
struct Foo {
    field: u8, /* test " */
}

is turned into this by running cargo fmt
struct Foo {
    field: u8, /* test " */
               }
}

A version without the unmatched quote is formatted correctly:
This code:
struct Foo {
    field: u8, /* test */
}

is not changed
To Reproduce
run rustfmt on the above code.

Expected behavior
rustfmt should not produce the erroneous closing brace

Meta

rustfmt version:

rustfmt 1.4.36-nightly (7de6968 2021-02-07)
cargo 1.52.0-nightly (c68432f1e 2021-03-02)
rustc 1.52.0-nightly (234781afe 2021-03-07)

From where did you install rustfmt?:

rustup

How do you run rustfmt:

- cargo fmt
- rust-analyser in vscode

I am working on this issue.

Derived from the original example, struct S /*"*/; led to
thread 'main' panicked at 'bad span: `;`: `struct S /*"*/;`', src\tools\rustfmt\src\source_map.rs:52:13

RUST_BACKTRACE=full returns <unknown> for me currently so isn't very useful.
stack backtrace:
   0:     0x7ff78aac42de - <unknown>
   1:     0x7ff78aadb89c - <unknown>
 ...

nightly-x86_64-pc-windows-msvc (default)
rustc 1.53.0-nightly (07e0e2ec2 2021-03-24)

This is a duplicate of a known issue that's been fixed in source and is pending release. Leaving it open for now as a reminder to myself to add test cases for the provided snippets.

@calebcartwright I would like to add the test cases and do the back-porting for this issue if it's not already done by someone else.

Hello @calebcartwright I would like to add the test cases and do the back-porting for this issue if it's not already done by someone else.

Great thanks! IIRC #4320 was the PR that resolved the underlying issue so you may want to start there


octodns: init at 1.2.1

Description of changes
This PR adds octodns and some of it's providers
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[X] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

sorry @infinisil I messed up a rebase that's why you got pinged, fixed it :see_no_evil:

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2812

Are these really libraries, that can be consumed by everyone? If not octodns should live outside python-packages and hold these modules in its own package set, that is derived from the original one.

yes they provide library functions, but I don't know if anyone is using them.

The README for octodns-hetzner does not inspire confidence in that they intend for this library to be used by anything besides octodns itself.

I know that octodns itself at least can be. Would this be a blocker for you or is this just a preference?

I know that octodns itself at least can be

But does anyone actually do that? Does upstream advertise that as a feature?

Would this be a blocker for you or is this just a preference?

The package set moves fast and breaks things. There is no possibility to override anything when inside the python package set. That's why applications commonly live outside the package set.
For me this is a blocker, because I don't want that kind of application breakage within the package set, because it creates unresolvable constraints.

Does upstream advertise that as a feature?

only for using it in providers

For me this is a blocker, because I don't want that kind of application breakage within the package set, because it creates unresolvable constraints.

then this should probably be a policy thats written down in nixpkgs, because there is a few applications in the python package set which probably wouldn't need to be.
Does it have to be it's own package set or is it fine for you if I just throw it into pkgs/by-name?
I'll try to do changes this weekend.

One approach  could look like this:
{ lib
, python3
, withHetzner ? true
}:

let
  python = python3.override {
    packageOverrides = self: super: {
      octodns-hetzner = self.callPackage ./plugins/octodns-hetzner.nix { };
    };
  };
in
python3.pkgs.buildPythonApplication {
  pname = "octodns";
  # ...

  propagatedBuildInputs = [
    # ...
  ]
  ++ lib.optional withHetzner python.pkgs.octodns-hetzner;
}

Or look at other python apps with a plugin system. Beets and poetry come to mind.

Not sure why octodns and octodns-providers instead of octodns/providers.

Is there anything left to do here?


Hong Kong shouldn't have area code

Hi,
In Hong Kong there is no area code, only the country code (+852) and the direct phone number (e.g: 2854 4866), but the libphonenumber parse the first 4 digits into area code.

How may I reproduce the issue using the demo?
When I enter "2854 4866" and "HK" I don't see anything about an area code.
Which part of this is unexpected?

Please help us understand this issue.
We added contribution guidelines at CONTRIBUTING.md.
Please ensure you go through the checklist there and use the template so we have the information we need to fix this.

Closing as we can't reproduce the issue. Please re-open if you have clarifications.


empty volume driver when creating server from snapshot

Steps to reproduce:

create a new VCS1 server based on docker image
create snapshot of server main volume
create new VCS1 server based on snapshot
ssh into the new server
run docker ps

Result: Docker 1.12.2 is running using the '' storage driver.
On the original docker server, the output of the docker ps command is: Docker 1.12.2 is running using the 'aufs' storage driver.

When creating the image did you wait for all IO operations to end before shutting down. C2s like to cache the IO and write the ext4 metadata to the disk in the background. (Tip i use iftop and iotop to monitor that)

Hi @opichon,
When you create the snapshot the bootscript is not saved, can you check it on your new server and change it to use the docker bootscript ?

Thanks for this. But there seems to be 2 docker bootscripts: x86_64 4.8.3 docker #1 and x86_64 4.5.7 docker #4. The latter is the bootscript currently in use on the original server.

You can use x86_64 4.8.3 docker #1


Sort messages before finishing analysis

Right now the output when running istioctl x analyze depends on what
order the analyzers run in, which isn't ideal. This adds a Sort function
to the Messages struct which will sort by the fields that happen to
occur on the log line right now, then calls it in the
analyzingdistributor. Fixes #17809.
Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[X] User Experience
[ ] Developer Infrastructure

Hi @zerobfd. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

@zerobfd: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
69bf53618e3513d19c1bb872f5a7e2f2a0ceb9d6
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zerobfd: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
69bf53618e3513d19c1bb872f5a7e2f2a0ceb9d6
link
/test lint_istio

unit-tests_istio
69bf53618e3513d19c1bb872f5a7e2f2a0ceb9d6
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/cherrypick release-1.4

@howardjohn: once the present PR merges, I will cherry-pick it on top of release-1.4 in a new PR and assign it to you.

In response to this:

/cherrypick release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@zerobfd: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-mixer-k8s-tests_istio
abf54073fe59bd592a2505445bdf38f415e5428e
link
/test integ-mixer-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@howardjohn: new pull request created: #18004

In response to this:

/cherrypick release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


[assemble] Native image distribution skipped because there are "no matching artifacts"

Discussed in https://github.com/jreleaser/jreleaser/discussions/1220

Originally posted by klausbrunner February 26, 2023
Trying out JReleaser 1.5.0 for #1196, and I'm wondering if I've misunderstood the change. I'm building native images for 3 VM types on a GH Actions matrix build, which works fine. I then proceed to collect the target artifacts from all those machines in a separate job and run jreleaser:full-release.
So far, I've had to explicitly list the artifacts to pick up in the distribution, which worked but was a bit verbose. My understanding was that thanks to #1196, these would be picked up automatically, but I'm getting:
Warning:  == JReleaser ==
Warning:  distribution.solarpos-native was disabled because there are no matching artifacts

Here's the relevant config for the assembly:
                            <nativeImage>
                                <solarpos-native>
                                    <active>ALWAYS</active>
                                    <executable>${project.artifactId}</executable>
                                    <mainJar>
                                        <path>target/${project.artifactId}-${project.version}.jar</path>
                                    </mainJar>
                                    <args>
                                        <arg>--no-fallback</arg>
                                        <arg>--verbose</arg>
                                    </args>
                                </solarpos-native>
                            </nativeImage>

And here's how target/jreleaser looks on the machine that's running jreleaser:full-release (just before starting it):
    ├── jreleaser
    │   ├── assemble
    │   │   ├── solarpos
    │   │   │   └── java-archive
    │   │   │       ├── solarpos-0.1.2-SNAPSHOT.zip
    │   │   │       └── work
    │   │   │           └── solarpos-0.1.2-SNAPSHOT
    │   │   │               ├── LICENSE
    │   │   │               ├── bin
    │   │   │               │   ├── solarpos
    │   │   │               │   └── solarpos.bat
    │   │   │               └── lib
    │   │   │                   └── solarpos-0.1.2-SNAPSHOT.jar
    │   │   └── solarpos-native
    │   │       └── native-image
    │   │           ├── jars
    │   │           │   └── universal
    │   │           │       └── solarpos-0.1.2-SNAPSHOT.jar
    │   │           ├── solarpos
    │   │           ├── solarpos-0.1.2-SNAPSHOT-linux-x86_64.zip
    │   │           ├── solarpos-0.1.2-SNAPSHOT-osx-x86_64.zip
    │   │           ├── solarpos-0.1.2-SNAPSHOT-windows-x86_64.zip
    │   │           ├── solarpos.build_artifacts.txt
    │   │           └── solarpos.exe
    │   ├── output.properties
    │   └── trace.log

I would have expected that the ...linux/osx/windows-x86_64-zip files would be found automatically and made part of the distribution, but that's not happening. Am I missing some crucial config here?

There's a bug in NativeImageAssemblerResolver, line 63 should be removed https://github.com/jreleaser/jreleaser/blob/e2d815d708f37f1d86c21a17125e13b72ff789d1/core/jreleaser-model-impl/src/main/java/org/jreleaser/model/internal/validation/assemble/NativeImageAssemblerResolver.java#L63


Type-level witnesses of transformations

We've discussed ways to ensure that all other transformations that a given one depends upon do indeed run before the given one. Preferably, at type level, so that the type also serves as a documentation.
I've been hacking a little bit on this today, and this is a proof-of-concept that I've come up with:
{-# LANGUAGE DataKinds #-}
{-# LANGUAGE PolyKinds #-}
{-# LANGUAGE StandaloneKindSignatures #-}
{-# LANGUAGE TypeFamilies #-}
{-# LANGUAGE TypeOperators #-}
{-# LANGUAGE UndecidableInstances #-}

import Data.Kind

-- Tags for different transformations,
-- think 'data EtaExpanded', 'data Monomorphized' and so on.
data Foo
data Bar
data Baz

-- A set of transformation tags is basically a list of types.
type Tags = [Type]

-- A transformation on type 'a' is a function 'a -> a'
-- along with the list of tags of transformations that it requires, and a list of tags it provides.
--
-- A transformation that's ready to be run has 'requires ~ '[]',
-- which might even be made a constraint on the function executing the pipeline.
type Xform :: Tags -> Tags -> Type -> Type
newtype Xform requires provides a = Xform (a -> a)

-- Kinda-set-union that doesn't care about introducing duplicates
type family (∪) (xs :: [a]) (ys :: [a]) :: [a] where
  '[] ∪ ys = ys
  (x ': xs) ∪ ys = x ': (xs ∪ ys)

-- Deleting all copies of a given val… err, type from a list of types
type family Del (xs :: [a]) (d :: a) :: [a] where
  Del '[] _ = '[]
  Del (d ': xs) d = Del xs d
  Del (x ': xs) d = x ': Del xs d

-- Set subtraction
type family (\\) (xs :: [a]) (ys :: [a]) :: [a] where
  xs \\ '[] = xs
  xs \\ (y ': ys) = Del xs y \\ ys

-- Left-to-right composition of transformations.
-- If we apply a transformation requiring r₁ and producing p₁
-- and then apply one requiring r₂ and producing p₂,
-- the composition requires r₁ ∪ (r₂ \\ p₁) and produces p₁ ∪ p₂.
--
-- It's tedious but easy to see that this is also associative at type level too,
-- as expected.
(&) :: Xform r1 p1 a
    -> Xform r2 p2 a
    -> Xform (r1 ∪ (r2 \\ p1)) (p1 ∪ p2) a
Xform f1 & Xform f2 = Xform $ f2 . f1

-- Some examples
t11 :: Xform '[] '[ Foo ] a
t11 = Xform id

t12 :: Xform '[ Foo ] '[ Bar ] a
t12 = Xform id

t13 :: Xform '[ Bar ] '[ Baz ] a
t13 = Xform id

t1All = t11 & t12 & t13
-- ghc gladly infers
-- t1All :: Xform '[] '[Foo, Bar, Baz] a
-- indeed, this is a correct transformation that can be run

t21 :: Xform '[] '[ Foo ] a
t21 = Xform id

t22 :: Xform '[ Baz ] '[ Bar ] a
t22 = Xform id

t23 :: Xform '[ Bar ] '[ Baz ] a
t23 = Xform id

t2All = t21 & t22 & t23
-- ghc gladly infers
-- t2All :: Xform '[Baz] '[Foo, Bar, Baz] a
-- indeed, this transformation has 'Baz' requirement unsatisfied due to 't22'

I'll see how well it integrates with the rest of pirouette tomorrow.

While I like this idea, we've decided against doing something like that a while ago for two reasons:

Lowers the complexity threshold for people to get into the codebase
Keeps pirouette as modular as possible: the engineer gets to pick which transformations should be applied, not the compiler.

That being said, and with pirouette becoming more and more domain specific (i.e., the PR about the execution semantics independent symbolic evaluation failing to deliver), I can see how it will help solidify the codebase and help catching refactoring mistakes.
Personally, I'm always all-in for some type-fu! :muscle:

I like the fact that the type system helps us in applying the right transformations and I think that's relevant when transformation assume another one was previously done (for instance, assuming the terms will be in prenex form).
In terms of this attempt, it's very interesting, but I am wondering whether we can do better or not, because it seems to me that the type checker here does not enforce needed transformation to be called, but rather it gives in the type of the resulting transformation whether or not some additional constraints should be satisfied, that is when the first parameter of Xform is non-empty.
Could we somehow enforce it ?
To answer this question, I think we should look at :

how Plutarch handles constraints in Value (namely NonZero and Positive)
how optics uses Haskell's type system to handle the "subtyping" of optics depicted for instance in https://d1bv04v5uizu9x.cloudfront.net/assets/img/optics-downloadable/png/Optics downloadable_04.png

Is there anything we can learn from these to improve this solution?

Looks harmless enough if we are not storing transformations inside lists or collections where the type indices would make things tricky. @0xd34df00d made a good job at coming with a simple scheme.
But I'm still unconvinced that this is worth the trouble. When maintaining code I would expect to concern myself more with the reason of a dependency between transformations than with potentially missing a dependency when preparing a pipeline.

I'd be happy to drop this as well, so up to you folks as I'm obviously biased towards some type-level fun.
For the record, regarding @mmontin's question, the idea is that the type of the function that runs the pipeline is explicitly monomorphic, along the lines of
xform :: Xform '[] provides a -> a -> a
xform (Xform f) = f

Then, if you try to run something that has unsatisfied dependencies, GHC will complain along the lines of:
    • Couldn't match type: '[Prenex]
                     with: '[]


chore(connlib): emit INFO logs for resource changes

When operating just the headless client, it is currently impossible to know, when resources become activate / inactive. To fix this, we add INFO logs every time we activate or deactivate a resource. This should also prove useful when debugging issues with customers because we now have a timestamped record of what resources were active at that time.

Converting to draft because it is stacked.


Travis-CI Build - [/retest and /test perticular JOB] on PR support for CI-BOT

Travis-CI Build Trigger on PR changes for

/retest - Retest CI Build on PR changes
/test jobname - test CI build on flaky JOB support

What type of PR is this?
/kind feature
What this PR does / why we need it:
This PR will the add the feature for Trigger the Build on PR changes and Trigger a perticualr Build.
Which issue(s) this PR fixes: #7
Special notes for your reviewer:
None

@edisonxiang @m1093782566 , can we start adding LICENSE head on all files ?
cc @pavan187

Hey @sids-b
IMO, It is not urgent to add the license.
We can add it later:)

@m1093782566 @sids-b @edisonxiang did the review comments rework, please check and approve the PR.


Add support for Amazon s3 bucket package storage "paths"

Has anyone tried distributing rez packages using s3 buckets?
Has anyone tried using other cloud storage systems to distribute rez packages?
Amazon s3 buckets are a fairly inexpensive, accessible, and secure way to distribute files to multiple locations, especially compared to some of the other cloud storage solutions.  Plus, they support billing to the receiver, allowing one group to publish buckets, but not have to be on the hook for all the access and transfer.

That is something I really want to look into it. It would be great if other people had experience with it :)

When I get some time, I'm going to try to create a package repository plugin for s3.  There was some discussion on the google group about possible local caching in the future, and this might be a good instance to try it out.

@bpabel Do you think we could setup a little "meeting" with interested people to see if we can come up with a set of common requirements? Maybe @nerdvegas have an opinion on this. I have the feeling that if we all put an effort into it, we could have something pretty cool and flexible.
What do you guys think?

@JeanChristopheMorinRodeoFX I'm open to that, but it's probably a bit premature for that until there's at least a working proof of concept.  If you have any ideas or thoughts in particular, feel free to add them to this ticket.
I think the first step would just be to implement the existing Package Repository API -- https://github.com/nerdvegas/rez/blob/master/src/rez/package_repository.py -- using s3 buckets, even if it's slow and inefficient, and then maybe talk about ways to extend rez with things like caching to improve performance.

Hello @bpabel and @JeanChristopheMorinRodeoFX what is the current status of adding support for S3 buckets as package repositories? Did any of you play with the idea further?
As we are also exploring the possibility of using S3 buckets as a non-file based target for rez-cp I will be interested to hear more details and help towards building a proof of concept.
Are you still willing to have a small meeting or a more in-depth chat via slack or something? I would love to hear your insight before diving deeper into this.

Has anything changed here? PoC/researches/etc.

@alexey-pelykh nothing for now. But there's been some positive momentum recently. For example, https://github.com/nerdvegas/rez/discussions/1242.

Indeed! I would guess it would be a major refactor prior to actually diving into the S3 subject?

Imho yes, otherwise you'd be shoehorning in that functionality.
A
On Fri, Mar 4, 2022 at 12:49 AM Alexey Pelykh @.***>
wrote:

Indeed! I would guess it would be a major refactor prior to actually
diving into the S3 subject?
—
Reply to this email directly, view it on GitHub
https://github.com/nerdvegas/rez/issues/571#issuecomment-1058059717, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMOUSTIYZRTVANI4WNKO3TU6C7QNANCNFSM4GT46IWQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Linking to #1380

Well, the previous PR was closed. New PR is https://github.com/AcademySoftwareFoundation/rez/pull/1453.


obs-studio: show the actual version instead of 0.0.1

Motivation for this change
The application otherwise just shows v0.0.1
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)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] 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.

Looks useful. Once merged, should be cherry-picked into release-20.03 as well.


First pass at mesh to mesh

Draft PR for adding Mesh to Mesh regridding functionality and testing.
Looking to make sure that all of the test work and cover all the additions, there may also be new things I need to add so draft for now.
Many test failiures expected.

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

By the way, I think the benchmarking failure has already been fixed in main (#297) so I think it ought to pass if you do a rebase.


Feature coredata

Fixes #15

[x] Unit Tests
[ ] Code review

@sinnerschrader-mobile/ios to be code reviewed

Fixes #15


Image zoom not working for dynamic data

When we try to get dynamic data of images (from ngx store), zoom doesn't work anymore...

Could you provide an example of what's not working? Do you get any errors in the console?

In my situation I need change image when thumb click. So I decided reinitialize ngx-image-zoom after image link changed by setting a timer. This was fixed the issue temporarily.
<div class="image-container" *ngIf="show_main_image === true">
    <ngx-image-zoom
        [thumbImage]="image"
        [fullImage]="image_big"
    ></ngx-image-zoom>
</div>

And in .ts file by default is show_main_image = true
onThumbClick() {
        this.show_main_image = false;
        setTimeout(
            () => {
                this.show_main_image = true;
            }, 100);
}


NVSHAS-9424: The redirections of goToNv functionalities are not completed yet

Completed redirection links of impact modal briefs of vulnerabilities page as manager.
ImpactModalWorkloadBrief

ImpactModalNodeBrief

Looks good!


Register provider via policy Part Six

Services

paloaltonetworks
postgresql
postgresqlflexibleserver
providerhub
recoveryservices
recoveryservicesbackup
resourcehealth
resourcemover

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

/azp run java - paloaltonetworks - mgmt - tests

/azp run java - postgresql - mgmt - tests

/azp run java - postgresqlflexibleserver - mgmt - tests

/azp run java - providerhub - mgmt - tests

/azp run java - recoveryservices - mgmt - tests

/azp run java - recoveryservicesbackup - mgmt - tests

/azp run java - resourcehealth - mgmt - tests

/azp run java - resourcemover - mgmt - tests

API change check
API changes are not detected in this pull request.

/azp run java - paloaltonetworks - mgmt - tests

/azp run java - postgresql - mgmt - tests

/azp run java - postgresqlflexibleserver - mgmt - tests

/azp run java - providerhub - mgmt - tests

/azp run java - recoveryservices - mgmt - tests

/azp run java - recoveryservicesbackup - mgmt - tests

/azp run java - resourcehealth - mgmt - tests

/azp run java - resourcemover - mgmt - tests

/azp run java - resourcehealth - mgmt - tests


[Link] Add role="link" and remove href from Link when it's disabled

It is still possible to disable a link by following 3 steps:

remove the href attribute so that it can no longer receive the focus
add a role="link" so that it is always considered a link by screen readers
add an attribute aria-disabled="true" so that it is indicated as being disabled
HTML

and we can remove custom onClick handler where we calling preventDefault
 Disabled link
Check out Scott O'Hara's excellent article on the subject: Disabling a link.

Hi, thank it works like a charm.


SparkCore Init & ReadFile implementation

This PR implements ReadFile algebra for SparkCore connector. Details in https://github.com/quasar-analytics/quasar/issues/1253

PR ready to review!
There are still few things to be improved (below). Those will be added asap, however the "main" functionality is ready and will not suffer from those changes.
Missing bits are:

write a test  pass test open file -> read chunk until empty vector -> close == file content, could anyone willing to PP on it?
tests should call sc.stop() regardless of whether they've passed or fail (there can be only one SC in the single JVM)
SparkContextIO should be just type alias to ReaderT
ReadFile.sparkCursorL should get  better name (still thinking on the name :) )

@jr: rdy right?

A couple of minor comments, but otherwise LGTM :shipit:

Current coverage is 75.03% (diff: 91.66%)

Merging #1242 into master will increase coverage by 0.18%

@@             master      #1242   diff @@
==========================================
  Files           191        193     +2
  Lines         11278      11302    +24
  Methods       11009      11033    +24
  Messages          0          0
  Branches        124        124
==========================================
+ Hits           8441       8480    +39
+ Misses         2837       2822    -15
  Partials          0          0

Powered by Codecov. Last update 6c8c88a...b5eff1c

I think there are a few of @wemrysi's concerns that you did not fully address.
I think it's OK to merge, I expect it to change some more soon so I won't be too picky since this PR has been open for a while now and it would be good to have in master for everyone to look at and base work on.
I noticed that some commits have compile errors in them, so I won't merge like we talked about in the retrospective meeting today. Instead, I will mark it as ready and let you rebase and merge in order to remove the compile errors.


dag/executor:refactor Row

What have you changed? (mandatory)
This PR refactor Row  to enum， so we can easily distinguish the origin row from those generated by aggregation.
It will be useful to support Arrow chunk, since for origin row data, the type of columns comes from the column info in request, while for rows generated by aggregation the type of columns comes from the expressions of groupby and aggregate functions.
What are the type of the changes? (mandatory)

Improvement (non-breaking change which is an improvement to an existing feature)

How has this PR been tested? (mandatory)
unit tests
Does this PR affect documentation (docs/docs-cn) update? (mandatory)
no
Does this PR affect tidb-ansible update? (mandatory)
no
@breeswish @hicqu PTAL

rest LGTM.

LGTM.

/run-all-tests

/run-unit-test


Fix types + cjs/mjs imports

Re: #8

Removes default exports
Explicit exports (no export * from './foo.js')
Fixed cjs types
Separate mjs and cjs types so they resolve correctly (ATTW reports no problems now)
Bumped to 2.0.0 for breaking API change (removing default exports)

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

By the way, is there a reason you've configured esbuild here? I'm unsure if it's even needed here to be honest, if it were my project I'd probably try to make do with two tsconfig files instead and lose the big dep :)

ping @corrideat could you take a look?

Hi @minau! Sorry that it took a while to respond.
I've taken a look and I have a few comments.

Removes default exports (https://basarat.gitbook.io/typescript/main-1/defaultisbad)
Bumped to 2.0.0 for breaking API change (removing default exports)

This is perhaps my bigger concern, that it breaks the API. Unless there's something actively breaking, I'm more open to adding additional exports (in the package.json) for specific exports. This doesn't break the API and ensures that you only get the parts you need / want, even if tree-shaking fails for some reason.

Explicit exports (no export * from './foo.js') (I can't find the link right now, but tldr it's bad for ASTs and tree shaking)

Related to the above. Do you have an example of something breaking from the default exports in the current form?

Fixed cjs types (added types to package.json)

This looks good.

Separate mjs and cjs types to .d.ts and .d.cts so they resolve correctly (ATTW reports no problems now)

This may indeed be needed. I wonder if there's a better way to do this (other than copying files) but I don't think there is (other than using moduleResolution: bundler, which also cannot be forced upon users).
TL;DR: The parts related to types look fine, the parts about exports, I'm less sure about.

Hi again,
I was doing some more testing over here: https://github.com/Exact-Realty/ts-rfc8188.

Regarding default imports, they work but are reported as potentially not working by ATTW
Regarding the cts files, it seems like simple copying and renaming won't work, because those files reference other files and then tsc fails. For example,

// foo.d.ts -> foo.d.cts
export { baz } from '../bar.js';

The issue is that tsc will complain that bar.js is an ESM module and compilation will fail. So, it seems like the contents also need to be rewritten so that instead of .js it says .cjs (and of course, replacing this with a regexp works but it could break). It's quite unfortunate that that's the state of affairs.

Now, I was planning on making a new release of this package anyhow. Is there something specific that's broken with the default exports? Otherwise, I think I'll make a new release fixing the types and perhaps we can revisit breaking changes later on. I'll try to also add exports for specific parts, like in the other package I mentioned, so that there are fewer things that can go wrong when bundling.

Sorry I was offline for a few days! Yes, I see the fixed types now, thank you!
I guess you can close this PR or pick whatever you like from it :) I understand not wanting to break the default export, that's fair!


Add missing translations

Not all elements were translated yet. This branch will add the latest translated elements

Added translated elements
Added Spanish
Switched filename format for language files from en.json to en-US.json to support multiple dialects of the same language

Closed this PR. It was created for the "old" Diamond App and is now derived too far away from that codebase.


How to use it on lunarvim?

I'm trying to add ra-multiplex to lunarvim, but I'm having difficulty

hi, can you please make your question more specific? have you tried starting from the neovim configuration example? lunarvim seems to be based on neovim

Since yesterday I managed to implement ra-multiplex in lunarvim, but sometimes when I start lunarvum this error happens:

And another error also shows, as if ra-multiplex had responded to an end message with a begjn

Hi, continuing to say that I managed to resolve this bug by changing these lines in ra-multiplex's config.toml:
instance_timeout = false
gc_interval = 1 # every 10 seconds

Oh, that's interesting. Thanks for mentioning that, it might suggest some other issues with handling connects which lunarvim is more sensitive to than other editors.


Problem with QS icons on custom ROMs like AICP and AOSiP

Description
Have the same problem on AICP and AOSiP. The 6 QS icons on notification bar are way too down on AICP and way too up on AOSiP. and not padded equally. Also, the clock and date and battery and network icons, and some others are not in place and are pushed together in the centre I think. However, the QS icons are fine once the QS panel is expanded.
Steps to reproduce

Install AICP or AOSiP.
Pull down once to see the notification panel.

ROM info
Lenovo Z2 Plus on AICP and AOSiP.
Module version (build version)
All the versions upto the latest, 0.1.0_26.
Screenshots
From AICP,

From AOSiP,

Xposed log / logcat
From AICP - AICPerror.log
From AOSiP - AOSiPerror.log

Remove comment syntax. i.e. <!--     -->

I know mate, I pressed the enter key without pressing shift and that messed it up.

This should be fixed.


Complex Architecture along with Clean Architecture

It's name should be Worst Architecture instead of Clean Architecture.
No explanation required because you also know about it.

NA


Tidying up 'multi-organism behavior' children

Following up on the discussion we had on the ontology call regarding #14956
Move the following terms directly under behavior:

'aggressive behavior'
*'behavioral defense response to insect'
*'behavioral defense response to nematode'
*'multi-organism reproductive behavior' -> move under 'reproductive behavior'
*'social behavior'
*'suckling behavior' (already under 'feeding behavior)
injection of substance in to other organism -> move under 'multi-organism process'

The proposal has been made to obsolete GO:0051705 multi-organism behavior
The reason for obsoletion is that this is an unnecessary grouping term.
There are no annotations and no mappings to this term. This term is present in the goslim_pir, but so is the parent 'multi-organism process GO:0051704', so no action is needed for now.
Any comment may be added to the ticket:https://github.com/geneontology/go-ontology/issues/18810
We are opening a comment period for this proposed obsoletion. Since there are a few annotations we’d like to proceed and obsolete this term on February 17th, 2020. Unless objections are received by February 17th, 2020, we will assume that you agree to this change.
Thanks, Pascale

Could genes be annoted to this term?
GO:0002211    behavioral defense response to insect
A gene for fly swatting?

This one is funny:
*'behavioral defense response to nematode'
no annotations but the reference is http://europepmc.org/article/MED/14506883
There is strong evidence suggesting that herbivore hosts have developed the foraging skills needed to take advantage of plant properties to combat parasites and thus use behaviour as a weapon in the host-parasite arms race.
You can't really annotate specific gene products to this!

Happy to obsolete some of those !

Why would social behavior (SB) no longer be under MOP?
I don't love MOP but while it exists we should abide by it:
name: multi-organism process
def: "A biological process which involves another organism of the same or different species."

I don't like the definition as it uses a weasely 'involves' (in a different sense from how we use it for gene products). In some sense of the word, SB involves another organism.
There are 2 main readings:

a MOP has multiple organisms as participants
a MOP may have a single organism as participant, but that process is embedded in a larger process that involves another organism

If we treat social behavior as the behavior of a single organism, made in response to stimuli from another organism, with effectors directed towards that other organism, then SB is not a subclass of 1, but it is a subclass of 2, which justifies your proposed move.
I'd like to push forward with this but it may be worth reviewing MOP first, what does it really mean
Maybe we want to obsolete it too, and all the 'multi-organism P' terms
There are deeper problems here... social behavior is under BP, which is a specific objective that the organism is genetically programmed to achieve. I'm not sure everyone would agree. Humans are not fruitflies.

@cmungall
'social behavior' is now under 'intraspecies interaction between organisms', so it does indicate that multiple organisms participate.
I thought on the last ontology call we had agreed to obsolete 'multi-organism process' and re-classify everything under intra or inter-organism process.
Let me know if you want to do this differently.
Thanks, Pascale

I think you should get rid of multi organism process, it is too complicating. I notice that now everything under MOP here is "behaviour"  now except for the rather crazy
GO:0035737    JSON
injection of substance in to other organism
The process of forcing a substance into another organism, either by penetrating the skin of the other organism or by applying the substance externally to a sensitive tissue such as those that surround the eyes.
hmmmm....
so MOP can easily be removed.
(it has no direct annotations. "M-O cellular process" has a single experimental annotation)

@cmungall Note that all of the  symbiont/host stuff is now under "interaction with other species"
(multi-organism process was just weird for this, it's usually a process in one species evolved to affect another species ).

social behavior' is now under 'intraspecies interaction between organisms', so it does indicate that multiple organisms participate.

ah I didn't see that. All good, thanks


pass environment variables for stop-stl

Missing piece of #95, to prevent make stop-stl-* reporting,

WARNING: The CHRONICLE_ENV_FILE variable is not set. Defaulting to a blank string.

PR Checklist
Please complete this checklist after opening your PR

[x] I have updated documentation as necessary
[x] I have updated helm chart(s) if needed
[x] I have added tests if needed
[x] All new and existing tests are passing
[x] Any breaking changes are clearly flagged and documented
[x] I’ve included a link to any relevant ticket(s)

Will need porting back to bootstrap.


Need to check input prompts are clear enough without logger context

The worker compatibility check for example:
❌  Checking worker compatability...
[12:30:48] WARNING  Incompatible workers detected!                                                                                checks.py:202
                    22/23 workers are incompatible.
                    Only compatible workers can consume jobs from this queuer.

                    To fix, update any incompatible hosts to this git commit:
                    '685dd64f88d37a06a39c51e0094fdc9e5d3e3511':

                    Incompatible hosts:
                    {'RMA'}

Do you wish to continue? [y/n]:

if the user had set loglevel to anything higher than warning, like ERROR or CRITICAL, they would miss all the context needed to know what 'Do you wish to continue' means. Instead it should read:
❌  Checking worker compatability...
[12:30:48] WARNING  Incompatible workers detected!                                                                                checks.py:202
                    Only compatible workers can consume jobs from this queuer.

                    To fix, update any incompatible hosts to this git commit:
                    '685dd64f88d37a06a39c51e0094fdc9e5d3e3511':

                    Incompatible hosts:
                    {'RMA'}

22/23 workers are incompatible.
Do you wish to continue? [y/n]:

The line between logging and printing is a bit of funky chicken.
Logging comes with a timestamp and a level to tell the user when and severity.
This makes the most sense for events. Using it application wide is probably unnecessary.
We can use it for debugging, extra details some users may want, and for any events that arise during render.
All checks and procedures before confirming the queue should be simple prints or prompts.


add socks proxy #95, fix #81

add socks proxy for #95
add tips for rule disalbed for #81
some style adjust

LGTM


Easy to trigger OOM with docker defaultRule and dashboard

Do you want to request a feature or report a bug?
Feature
What did you expect to see?
I made an "oops" in my configuration which resulted an endless forwarding cycle (causing Traefik to get OOM killed). This was a pretty simple oops, and one I suspect others might make too. It would be nice to have some prevention - perhaps 1) a comment in the docs 2) scan the config for this as one of a small set of "known-bad" patterns and warn the user in logs 3) detect the endless forward and prevent that
What happened:

I followed the docker quickstart, and was able to get an easy setup working locally. Simple system, one container with a webserver on internal port 80
I added docker.defaultRule=Host('foo.com') and got a working domain setup without having to add any container labels
I enabled the dashboard in insecure mode
Visiting foo.com:8080 works as expected, I see the dashboard. Unfortunately, visiting foo.com:80 causes traefik to loop until it's OOM killed

I can see in a DEBUG log that the requests are forwarded to the traefik container instead of the webserver container, which makes sense because they both have the same host rule.
As a beginner to the 2.x codebase, this took a while to debug and made learning 2.x feel like a slog. Any of the remediations mentioned above would have helped. I originally thought it was my http to https redirection and spent a while learning about middlewares and load balancers before I was able to cut down my config to a much smaller MWE.

Hello,
To give you an explanation of the behavior, in this case, by default, the router's entry points on the Traefik container are traefik (8080) and http (80).
So the call to foo.com (http://foo.com:80) use this router, so you are calling Traefik (on port 80), so that create an infinite loop.
traefik (8080) and http (80) are created automacally:

traefik (8080) because you are using --api.insecure
http (80) because you didn't define an explicit entry point

A more explicit example:
version: '3'

services:
  reverse-proxy:
    image: traefik:v2.0.4
    command:
    - --api.insecure=true
    - --providers.docker
    - --log.level=DEBUG
    # - --entrypoints.http.address=:80
    # - --entrypoints.traefik.address=:8080
    ports:
      - "80:80"
      - "8080:8080"
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
    labels:
      traefik.http.routers.dashboard.rule: Host(`test.localhost`)
      traefik.http.routers.dashboard.entrypoints: traefik,http

To avoid that:

you have to set explicit entry points on the router of the Traefik container

explicit entry points:
version: '3'

services:
  reverseproxy:
    image: traefik:v2.0.4
    command:
    - --api.insecure=true
    - --providers.docker
    - '--providers.docker.defaultRule=Host(`{{ index .Labels "com.docker.compose.service" }}.localhost`)'
    - --log.level=DEBUG
    ports:
      - "80:80"
      - "8080:8080"
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
    labels:
      traefik.http.routers.dashboard.entrypoints: traefik

or/and use the api@internal service. secure-mode

api@internal service:
version: '3'

services:
  reverseproxy:
    image: traefik:v2.0.4
    command:
    - --api
    - --providers.docker
    - '--providers.docker.defaultRule=Host(`{{ index .Labels "com.docker.compose.service" }}.localhost`)'
    - --log.level=DEBUG
    ports:
      - "80:80"
      - "8080:8080"
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
    labels:
      traefik.http.routers.dashboard.service: api@internal


Databricks destination - provider created inconsistent result after apply

Describe the bug
Get the following error after applying
To Reproduce (relevant code)
resource "fivetran_group" "group" {
  name = "${var.env}_databricks"
}

resource "fivetran_destination" "dest" {
  group_id                     = fivetran_group.group.id
  service                      = "databricks"
  time_zone_offset             = "-8"
  region                       = "AWS_US_EAST_1"
  trust_certificates           = "true"
  trust_fingerprints           = "true"
  daylight_saving_time_enabled = "true"
  run_setup_tests              = "true"

  config {
    catalog               = "raw_${var.env}"
    cloud_provider        = "AWS"
    connection_type       = "Directly"
    auth_type             = "PERSONAL_ACCESS_TOKEN"
    http_path             = module.fivetran_warehouse.output.odbc_params[0].path
    personal_access_token = resource.databricks_obo_token.fivetran.token_value
    port                  = module.fivetran_warehouse.output.odbc_params[0].port
    server_host_name      = module.fivetran_warehouse.output.odbc_params[0].hostname
  }

  depends_on = [module.databricks_workspace, module.unity_catalog, module.fivetran_warehouse]
}

data "aws_rds_cluster" "fhirserver" {
  cluster_identifier = "${var.env}-fhirserver-aurora-curai"
}

resource "fivetran_connector" "fhir" {
  service  = "aurora_postgres"
  group_id = fivetran_group.group.id

  destination_schema {
    prefix = "fhir"
  }

  trust_certificates = true

  config {
    host             = data.aws_rds_cluster.fhirserver.reader_endpoint
    port             = 5454
    user             = "fivetran_databricks"
    password         = ...
    update_method    = "TELEPORT"
    always_encrypted = true
    connection_type  = "Directly"
  }
}

Expected behavior
terraform apply should work and not cause a drift after it finishes
Logs & Output (terraform apply)
fivetran_destination.dest: Still creating... [30s elapsed]
╷
│ Error: Provider produced inconsistent result after apply
│
│ When applying changes to fivetran_destination.dest, provider
│ "provider[\"registry.terraform.io/fivetran/fivetran\"]" produced an
│ unexpected new value: .config.auth_type: was
│ cty.StringVal("PERSONAL_ACCESS_TOKEN"), but now null.
│
│ This is a bug in the provider, which should be reported in the provider's
│ own issue tracker.
╵
╷
│ Error: Provider produced inconsistent result after apply
│
│ When applying changes to fivetran_destination.dest, provider
│ "provider[\"registry.terraform.io/fivetran/fivetran\"]" produced an
│ unexpected new value: .config.cloud_provider: was cty.StringVal("AWS"), but
│ now null.
│
│ This is a bug in the provider, which should be reported in the provider's
│ own issue tracker.

Plugin version:
1.1.22

@ktadiko the problem seems to be on API side. We'll fix it, so you won't need to update terraform version etc.
I'll inform you about that here when the fix will be deployed.
The problem is that API doesn't return these fields in response, but terraform provider expects it to be returned with expected values.

@ktadiko could you please try again? The fix was deployed on Fivetran API.

Fixed on API side. Closing the issue.


Commit linter allows non signed commits

The lack of signature is treated as a warning while it must be an error. Example

@shyamjesal any idea how to fix this one?


Exclude logs based on logger name

API change check
API changes are not detected in this pull request.


GHERKIN_FEATURES is not a valid win32 application

On Windows Seven, when trying to php bdd-wizard.phar --features="D:\www\behat\features" --reports="D:\www\behat\output\junit" I get the following output :
'GHERKIN_FEATURES' n’est pas reconnu en tant que commande interne ou externe, un programme exécutable ou un fichier de commandes.
('GHERKIN_FEATURES' is not a valid win 32 application)
I found no app named gherkin_features on my computer.
I'm running Behat 2.5 and I organised the following folder structure :
www
-- behat
-- bddwizard
Am I missing a dependency?
Any hint is appreciated.

Even after 6 months your problem has not been addressed !!! I am getting the same error ... just have no idea how to proceed ... HELP! someone please HELP!


hard-coded path in the pickle file

You have some hard-coded path in the pickle files, e.g., in this line: https://github.com/thusiyuan/cooperative_scene_parsing/blob/master/preprocess/sunrgbd/sunrgbd_parser.py#L116
img_info['imgrgb_path']
'/home/siyuan/Documents/Dataset/SUNRGBD_ALL/SUNRGBD/kv2/kinect2data/000002_2014-05-26_14-23-37_260595134347_rgbf000103-resize/image/0000103.jpg'
img_info['imgdepth_path']
'/home/siyuan/Documents/Dataset/SUNRGBD_ALL/SUNRGBD/kv2/kinect2data/000002_2014-05-26_14-23-37_260595134347_rgbf000103-resize/depth/0000103.png'
Could you add your code to generate the pickle files as well? Also, it seems in your preprocessed data, there is no depth images, should I download the original SUNRGBD dataset as well?

Hi,
Thank you for pointing it out.
I manually change the root temporarily as in https://github.com/thusiyuan/cooperative_scene_parsing/commit/472417abc8eb60343d848739b7ff2d5fa73e6ba2
You also need to download the original dataset, as added in README.
We will prepare the code for generating the pickles files, hopefully, will release them in next month, maybe after the ICCV deadline.
Thank you.

Thank you for fixing that. That's helpful!
One more question: in this line: img_info['seg2d'] = loadmat(img_info['seg2d_path'])['seg_2d']
'/home/siyuan/Documents/cvpr2018/data_temp/segmentation/1.mat'
Where can we download the segmentation files?

This is the segmentation map we generate for another project.
You could comment out this line.

Could you kindly share these segmentation files? I was intended to implement a function to visualize the 3d point cloud with semantic information and the corresponding 3d bounding box. Having these segmentation maps will be super helpful! Thank you for the useful code and preprocessed data!

https://drive.google.com/file/d/1T-POZBhkh1YW9W0JG2qgVSb_tnvjoHXx/view?usp=sharing
This is the segmentation file we use. You can use it after changing the seg root in the pickle file.
To be noticed, the seg results are not ground truth but inferred from a learned segmentation model. If you want the ground truth, you may need to process the original dataset.
If you want to do visualization, it should be enough.


Don't insert unicode names twice: #86

Resolves #86 -- str(new_column) of a column containing u'a\xa0b' is u'a\\xa0b' -- we already have a nice name in column_name.
With tests, py23 compatible (bug was py2 only)

Looks good to me.


chore: Make sure to keep forked application in valid state even if client exits the flow before server responds

This reverts commit c1a32c9dfe0a258f2f64aa6fcdb3c439556fa4a9. We have fixed the issue which resulted in infinite loop during the fork application here, when the subscription does not get canceled by the client
Description
Fork application is currently a slow API because it needs to create an application, fork all the pages, and then copy all the actions and collections. This process may take time and the client may cancel the request. This leads to the flow getting stopped midway producing corrupted DB objects. The PR ensures that even though the client may have cancelled the flow, the forking of the application should proceed uninterrupted and whenever the user refreshes the page, the sane forked application is available.
Fixes #10184
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

JUnit TC
Manual test

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[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

Do we know how the repeated evaluation happened last time and is it fixed now? Can you share some details on that please?

Do we know how the repeated evaluation happened last time and is it fixed now? Can you share some details on that please?

I have provided the reference what caused the repeated eval at first place in the description. You can find the details in this PR https://github.com/appsmithorg/appsmith/pull/10455
Let me know if this makes sense otherwise I'll try to describe the issue in more depth

/ok-to-test sha=e150171


how to change chest in Cosmic

how to change chest in Cosmic ?
Dont have any config for this
And how i can translate all texte for cosmic "you can open 4 chest"

Not sure what you mean by change chests. Can you explain.
I need to add the ability for that sorry.

le 27 items in cosmic choice its all chest .

I am not sure what you meant by that sorry. Could you try and explain a bit better if you could.

i send you a message ont spigot with picture


use stylus resolve url

By default Stylus don’t resolve the urls in imported .styl files, so if you’d happen to have a foo.styl with @import "bar/bar.styl" which would have url("baz.png"), it would be url("baz.png") too in a resulting CSS.
But you can alter this behavior by using --resolve-url (or just -r) option to get url("bar/baz.png") in your resulting CSS.

LGTM


Update TabNames.cs

Added "top banner" group

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


Add hotkeys for using the toolbar items

E.g. r for rotate. Now that we have the useKeyDown hook from #117 i think it’s a good time to add these.

Yep! I’d even wrap the usekeydown handler in a toolbarHotkeyHandler (or something like that) hook, so it’s clear where this logic is handled. Maybe even a calibrateToolbarKeyHandler and a projectToolbarKeyHandler so it’s menu state aware? I’m not sure!

Yep! I’d even wrap the usekeydown handler in a toolbarHotkeyHandler (or something like that) hook, so it’s clear where this logic is handled. Maybe even a calibrateToolbarKeyHandler and a projectToolbarKeyHandler so it’s menu state aware?

I like the idea of having one handler for calibrate and one for project! I think things are going to clean up nicely when we get those separated out a bit more.


changed session cookie to lax

fixes #426 I think

I think so since this is only used on /logout and all the requests to logout come from our site.


Kernel output in GPflow 2.0 with negative eigenvalues

System information:

Python version: 3.6
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): 2.0
GPflow installed from (source or binary): source
GPflow version: 2.0

There seem to be a difference between the output of the Kernel in GPflow 2.0 compared to GPy which cause negative eigenvalues further from zero:
Example:
x = tf.constant([[0.       ],
                 [0.9486833],
                 [1.8973666],
                 [2.8460498],
                 [0.9486833]], dtype=tf.float32)  # which should cause a single zero eigenvalue of the Kernel

kernel  = gpflow.kernels.RBF(lengthscale=1)
K1 = kernel.K(x)

Kern = GPy.kern.RBF(input_dim=1,        lengthscale=1)
K2 = Kern.K(x)

The eigen values are then:
tf.linalg.eigh(K1)[0]
<tf.Tensor: id=8608, shape=(5,), dtype=float32, numpy=
array([-6.8605075e-08,  1.3176124e-01,  5.1539403e-01,  1.3180454e+00,
        3.0348010e+00], dtype=float32)>

tf.linalg.eigh(K2)[0]
<tf.Tensor: id=8611, shape=(5,), dtype=float64, numpy=
array([1.75396806e-16, 1.31761230e-01, 5.15393839e-01, 1.31804469e+00,
       3.03480024e+00])>

Is there a solution for this numerical issue?
Another question that I had was if GPflow 2.0 usable only with tf.float32 at the moment or how is it possible to use it with tf.float64. If tf.float64 is also usable, do you know if tf.GradientTape is usable for training a model?
Edit:
I solved the issue by making the following change in the config.py file of the source code:
_FLOAT_VALUE = np.float32
# changed to
_FLOAT_VALUE = np.float64

Is there a better way to make such change without changing the source code, or is it a safe modification?

Hi @a-z-e-r-i-l-a - gpflow can handle both 32 bit and 64 bit computation, though of course 64 bit computation is more suitable if you require a higher precision. As this isn't an issue with gpflow itself, I'm going to close this issue. However, if you still have any questions about numerics or using gpflow (or gpflow2), then do post those on Stack Overflow, where I'm sure some kind soul will drop by to help you. SO generally provides a better forum for this kind of discussion. In particular, I recommend tagging with the gpflow tag.

hi @condnsdmatters ,
How do I use gpflow with 64 bit. so far I could only do so by the change in the source code I mentioned in the issue. Thanks.

@a-z-e-r-i-l-a float64 should be the default, as specified in the _Values enum in gpflow/config/__config__.py. You can change the type used throughout gpflow by calling gpflow.config.set_default_float(np.float32), or gpflow.config.set_default_float(np.float64) to reset it to 64 bit.


Add info - What is logged in Radix

Radix platform logges all required information and handles the logs according to compliance requirements.
All logs available for users "online" in 30 days
Logs archived for additional 60 days.
Specify this information in our Public site.
Describe how Radix CLI can be used.

Feedback from Dyna Geo team:
https://github.com/equinor/dynageo/issues/4128


Remove requirement implementations from overload

Consider the following program:
public fun main() {
  let x: UInt = 1
  _ = x.infix+(1)
}

Currently, the call to x.infix+ is building an overload set with the declarations of AdditiveArithmetic.infix+ and Uint.infix+. That is inefficient because type inference has already determined that UInt : AdditiveArithmetic, meaning that can already decide the outcome of overload resolution before invoking the solver.
Reducing the size of overload sets while checking numerical expression is crucial to avoid combinatorial explosion, especially in the presence of literals.

I would say that UInt.infix+ is more-specific than the other one, and if we got into a situation where they meant different things (I can easily imagine that with a different type), I would expect UInt.infix+ to be called in this non-generic context.

I would say that UInt.infix+ is more-specific than the other one

The current implementation of the overload solver would agree.
This issue is about removing AdditiveArithmetic.infix+ before solving the overload set when we know that the solver will always pick UInt.infix+ anyway. In other words, the goal is to prune the search tree early. Another tool we can (and do) use is to weight the choices of an overload set differently. But that only informs the heuristic guiding the search without guaranteed efficiency benefits.

if we got into a situation where they meant different things (I can easily imagine that with a different type), I would expect UInt.infix+ to be called in this non-generic context

Not sure to understand. Are you thinking about this situation?
public trait P {
  fun f() -> Int
}
public trait Q {
  fun f() -> Int
}
type A: P, Q, Regular {
  public fun f() -> Int { 0 }
  public memberwise init
}

public fun main() {
  _ = A().f()
}


Tracing AWS SDK client calls

Hi !
Moving from instrumenting with X-Ray, I was wondering if there was any support for automatic tracing around AWS SDK like the following example
svc := secretsmanager.New(session.New(&aws.Config{Region: aws.String(region)}))
xray.AWS(svc.Client)
ctx, seg := xray.BeginSegment(context.Background(), "payforadoptions")

res, err := svc.GetSecretValueWithContext(ctx, &secretsmanager.GetSecretValueInput{
	SecretId: aws.String(secretID),
})
seg.Close(nil)

Thanks

Hi @bonclay7 ,
Thanks for opening up an issue. We currently don't support Tracing AWS SDK client calls for AWS OTel Go. I will mark this issue as a feature request and I would also recommend you to create an issue in Upstream Open Telemetry Go Contrib  Repo as well.

Hi @bonclay7 ,
Thanks for opening up an issue. We currently don't support Tracing AWS SDK client calls for AWS OTel Go. I will mark this issue as a feature request and I would also recommend you to create an issue in Upstream Open Telemetry Go Contrib  Repo as well.

Thanks ! I saw this feature request open there as well https://github.com/open-telemetry/opentelemetry-go-contrib/issues/81

Thanks ! I saw this feature request open there as well https://github.com/open-telemetry/opentelemetry-go-contrib/issues/81

AWS SDK v2 support is released in upstream OTel Go repo: https://github.com/open-telemetry/opentelemetry-go-contrib/pull/621 so closing this issue.


Can't find transaction

I just installed the testnet wallet and sent some tBTC there but I just can't see them on the wallet, the transaction has already 6+ blocks confirmed, this is the txid
cec1da1d6d18f8eef3d7a5793ae7039bdc5720e79d32d70617a77be34903b941
but still don't see it there. I double checked the address and confirmations. Is there a sync problem maybe?

This is likely either a connectivity issue (no Electrum node is connected) or wallet still synchronizing, it can take a few minutes on testnet.
Unfortunately current testnet version is old and shows no connectivity or sync details, I'll update it this week so you'll be able to see what exactly is going on.

Yes, I believe there was a sync issue so I changed the Electrum node and now the transaction is there. Thanks for your answer and looking forward to an updated version.
Cheers

I would like to create channels and send payments to/from the wallet but something is not active on the testnet version. Would be nice to update testnet version.
Thanks

Testnet version 2.4.19 has been uploaded to Play store, should be available shortly.

Thanks for the update. This is what I got when opening after new version installed.
android.database.sqlite.SQLiteException: no such table: cwallet (code 1 SQLITE_ERROR[1]): , while compiling: SELECT * FROM cwallet ORDER BY _id ASC
at android.database.sqlite.SQLiteConnection.nativePrepareStatement(Native Method)
at android.database.sqlite.SQLiteConnection.acquirePreparedStatement(SQLiteConnection.java:1463)
at android.database.sqlite.SQLiteConnection.prepare(SQLiteConnection.java:901)
at android.database.sqlite.SQLiteSession.prepare(SQLiteSession.java:590)
at android.database.sqlite.SQLiteProgram.(SQLiteProgram.java:62)
at android.database.sqlite.SQLiteQuery.(SQLiteQuery.java:37)
at android.database.sqlite.SQLiteDirectCursorDriver.query(SQLiteDirectCursorDriver.java:46)
at android.database.sqlite.SQLiteDatabase.rawQueryWithFactory(SQLiteDatabase.java:2063)
at android.database.sqlite.SQLiteDatabase.rawQuery(SQLiteDatabase.java:2002)
at com.btcontract.wallettest.sqlite.DBInterfaceSQLiteAndroid$class.select(DBInterfaceSQLiteAndroid.scala:15)
at com.btcontract.wallettest.sqlite.DBInterfaceSQLiteAndroidMisc.select(DBInterfaceSQLiteAndroidMisc.scala:8)
at immortan.sqlite.SQLiteChainWallet.listWallets(SQLiteChainWallet.scala:27)
at com.btcontract.wallettest.WalletApp$.makeOperational(WalletApp.scala:201)
at com.btcontract.wallettest.MainActivity$EnsureSeed.makeAttempt(MainActivity.scala:109)
at com.btcontract.wallettest.MainActivity$EnsureAuth$$anon$1.onAuthSucceeded(MainActivity.scala:158)
at com.btcontract.wallettest.utils.BiometricAuth$$anon$1.onAuthenticationSucceeded(BiometricAuth.scala:38)
at androidx.biometric.BiometricFragment$9.run(BiometricFragment.java:907)
at android.os.Handler.handleCallback(Handler.java:938)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:246)
at android.app.ActivityThread.main(ActivityThread.java:8528)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:602)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1130)

It's probably not backwards compatible, I care about this stuff in mainnet version, but not in testnet. Just reinstall an app from scratch and it will work.

 Can you please give me a hint as where to find the wallet data and if it's possible to restore that file? I am looking for info recovering deleted apps, but I am not even sure which file to look for. I didn't really deleted the whole app, but I went to the settings menu and deleted data there.
Thanks for your help, sorry to bother you with this unrelated issue.

The important thing here is to have a recovery phrase, have you got it? If yes then just reinstall from scratch and restore using backup file in Download folder: https://sbw.app/posts/manual/#wallet-restoring
But wallet backup alone won't help since it only stores channel data (if there are any channels) and is itself encrypted with recovery phrase.
As to app un-deletion, can't say anything definite here.


Create commands for controlling decorated tabs

As of now there's not an easy way to focus a specific tab, so if you want to focus a specific tab your options are to estimate which tab number it is (if it's one of the first 8 tabs) or using tab last or tab next and maybe some repeater commands or sounds. Some people also use tab search for this purpose. All these methods are inefficient and tedious.
My idea to make all this easier is to prepend some decoration to the page title, which is the only way we have to changed the appearance of the tab. Using a combination of colors (🔴🔵🟢🟡🟣) and letters we can have a nice system that is quick to use, not unlike direct clicking.

So, for example, in this example using the command red air would focus the second tab.
Once the tabs are decorated we can use those decorators not just for focusing the tabs but also to move or close them. We could also create complex commands like close red air past yellow cap or  move blue gust and red drum before red fine. I don't know if creating such commands is overcomplicating things or if people would find them useful.

Cool idea but i wonder if that's going to lead to cursorless conflicts when we start integrating things 🤔. Maybe would be fine because actions are different? Alternately you could use some other kind of symbol, eg different emoji. Or just use letters with a prefix to avoid rango mark conflict, eg "close tab air". The consistency might be nice, eg "close tab" / "close tab air". Or "swap tab with bat", where "tab" without a mark indicates current tab, and use cursorless-style inference to infer that "bat" is a tab. Idk just stream of consciousness here 😄

What if we switch it? Instead of red air we use air red

Could work? As long as you swapped the letter with the color in the tab display obv
Would be curious to hear @andreasarvidsson's thoughts here

Yes, of course, I would swap in the tab too

@pokey I'm probably not the right person to ask since I very seldom have more tabs open that can be easily accessible by numbers. I actually don't use the tabs feature in Rango. In general having both cursorless and Rango refer to colored letters might be confusing though.

Maybe using a prefix like @pokey mentions is the right idea. In most cases is going to be the same number of syllables if we use something like tab gust. Only beyond tab 26 one would have to use three syllables.

There's no way to underline letters in the tab? Could do cursorless-style thing. But just putting letter at the beginning is prob fine

No, the only way to mark a tab is to change the document.title


[Tooltip] Tooltip (as slider ValueLabel) won't show up until I click the slider thumb

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
When overriding ValueLabel in Slider props with a custom tooltip like this
function ValueLabelComponent(props) {
  const { children, value } = props;

  return (
    <Tooltip enterTouchDelay={0} placement="top" title={value}>
      {children}
    </Tooltip>
  );
}

we observed following behavior

if there is only one thumb in the slider, everything works as expected
if there are more than one thumb, the tooltip won't show up as we hover on the thumb until we click on it. After clicking on the thumb, the tooltip will show up whenever we hover on the thumb as expected. Note: this behavior applies to each thumb separately, i.e. for each thumb on the slider, we need to click on it to make its tooltip functionally work.

Expected behavior 🤔
We should not have to click on the thumb to make tooltip work
Steps to reproduce 🕹
Steps:

Open the sandbox of this demo https://mui.com/components/slider/#customization
Verify that the 3rd slider's overridden tooltip works properly
Override the same ValueLabelComponent as ValueLabel in the 4th slider(airbnbSlider)'s components prop
should observe that the tooltip on the 4th slider won't show up until you click on the thumb

Context 🔦
No response
Your environment 🌎

`npx @mui/envinfo`
"@mui/material": "^5.1.0"
Microsoft Edge browser

This issue has been fixed in https://github.com/mui/material-ui/pull/32321
For your information, here are Codesandboxes before and after.
Codesandbox Before The Fix
Codesandbox After The Fix

As far as I know, this is not a bug, it's a support question. The reproduction provided for the issue https://codesandbox.io/s/customizedslider-demo-material-ui-forked-vh93v2?file=/demo.tsx raise what is going wrong:

The ref should be forwarded:
 interface AirbnbThumbComponentProps extends React.HTMLAttributes<unknown> {}

-function AirbnbThumbComponent(props: AirbnbThumbComponentProps) {
-  const { children, ...other } = props;
-  return (
-    <SliderThumb {...other}>
-      {children}
-
-
-
-    </SliderThumb>
-  );
-}
+const AirbnbThumbComponent = React.forwardRef(
+  (props: AirbnbThumbComponentProps, ref) => {
+    const { children, ...other } = props;
+    return (
+      <SliderThumb {...other} ref={ref}>
+        {children}
+
+
+
+      </SliderThumb>
+    );
+  }
+);

 export default function CustomizedSlider() {
   return (

once the change is done, it now works correctly: https://codesandbox.io/s/customizedslider-demo-material-ui-forked-yn4d9n?file=/demo.tsx:1276-1296.
So I have reverted #32321.

Regarding the next step, I think that we have possible options:

We explicitly add the support for tooltip children that don't forward the ref, as a fallback mechanism. But in this case, this issue is a "new feature", the PR label would need to change. We also need a follow-up to replace the added test case to cover the use case.
Or, and this is my preference, we move more incrementally, we update the demos in the docs to forward the ref.

Or, and this is what I think makes more sense, we move more incrementally. We update the demos in the docs (https://mui.com/material-ui/react-slider/#customization) to forward the ref, so that others won't face the same problem. It would feel more composable.

+1 for doing this.


Added Umbrella Projects Lesson.

Just delivering the Umbrella Projects lesson. Refs #50

:boom:  This is great @pjhampton, I'll look this over today and leave if any feedback if I have it.

You're right @doomspork! I have moved it into the advanced section.