Crash On Initial Install

Installing this on the 5th box here (2 Win, 1 Pi, 1 Android) and ran into a problem on the 3rd Win
Downloaded, unzipped and moved resulting folder to C:/ProgramFiles
Tried to install 3x, by double clicking on SyncTrayzor.exe; same thing each time.  No uninstall option available, so I followed instructions.  I had left the install folder under Downloads on the other 2 Win installs, but that was kinda messy - the only diff I can see.  Maybe it doesn't like the way Windoze locked down that directory?
HW
Fairly recent system (LT 3 yrs); kept cleanish and updated
msi z-390 mobo
Intel i5-9600
@3.7gmsi z390
16 GB ram
250gb m2 ssd; ~50% used
2TB spinners mirrored; ~10% used
OS
win10 pro 64
version: 22H2
build: 19045.2364
exp: Windows Feature Experience Pack 120.2212.4190.0
from the window shown after crash:
Version: <IP_ADDRESS>; Variant: Portable; Arch: Amd64
Path: C:\Program Files\SyncTrayzorPortable-x64\SyncTrayzor.exe
System.UnauthorizedAccessException: Access to the path 'C:\Program Files\SyncTrayzorPortable-x64\data\config.xml.tmp' is denied.
at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
at SyncTrayzor.Utils.AtomicFileStream..ctor(String path, String tempPath) in C:\projects\synctrayzor\src\SyncTrayzor\Utils\AtomicFileStream.cs:line 20
at SyncTrayzor.Services.FilesystemProvider.CreateAtomic(String path) in C:\projects\synctrayzor\src\SyncTrayzor\Services\FilesystemProvider.cs:line 47
at SyncTrayzor.Services.Config.ConfigurationProvider.SaveToFile(Configuration config) in C:\projects\synctrayzor\src\SyncTrayzor\Services\Config\ConfigurationProvider.cs:line 421
at SyncTrayzor.Services.Config.ConfigurationProvider.LoadFromDisk(Configuration defaultConfiguration, Boolean& hadToCreate) in C:\projects\synctrayzor\src\SyncTrayzor\Services\Config\ConfigurationProvider.cs:line 191
at SyncTrayzor.Services.Config.ConfigurationProvider.Initialize(Configuration defaultConfiguration) in C:\projects\synctrayzor\src\SyncTrayzor\Services\Config\ConfigurationProvider.cs:line 99
at SyncTrayzor.Bootstrapper.Configure() in C:\projects\synctrayzor\src\SyncTrayzor\Bootstrapper.cs:line 168
at Stylet.BootstrapperBase.Start(String[] args)
at System.Windows.Application.OnStartup(StartupEventArgs e)
at System.Windows.Application.<.ctor>b__1_0(Object unused)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
log file:
2023-05-20 10:33:27.3579 #10508 [Info] SyncTrayzor.Bootstrapper: SyncTrazor version <IP_ADDRESS> (Amd64) started at C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\SyncTrayzor.exe (.NET version: Unknown (528372))
2023-05-20 10:33:27.3891 #10508 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: LogFilePath: C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\logs
2023-05-20 10:33:27.4047 #10508 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: SyncthingBackupPath: C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\syncthing.exe
2023-05-20 10:33:27.4047 #10508 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: ConfigurationFilePath: C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\data\config.xml
2023-05-20 10:33:27.4047 #10508 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: ConfigurationFileBackupPath: C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\data\config-backups
2023-05-20 10:33:27.4047 #10508 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: CefCachePath: C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\data\cef\cache
2023-05-20 10:33:27.4047 #10508 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: DefaultSyncthingPath: C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\data\syncthing.exe
2023-05-20 10:33:27.4047 #10508 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: DefaultSyncthingHomePath: C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\data\syncthing
2023-05-20 10:33:27.4047 #10508 [Debug] SyncTrayzor.Services.Ipc.IpcCommsClientFactory: Checking for other SyncTrayzor processes
2023-05-20 10:33:27.4047 #10508 [Debug] SyncTrayzor.Services.Ipc.IpcCommsClientFactory: Did not find any other processes, or they all responded with an error
2023-05-20 10:33:27.4204 #10508 [Info] SyncTrayzor.Services.Config.ConfigurationProvider: Loaded default configuration: <Configuration ShowTrayIconOnlyOnClose=False MinimizeToTray=False CloseToTray=True ShowDeviceConnectivityBalloons=False ShowDeviceOrFolderRejectedBalloons=True SyncthingAddress=localhost:8384 StartSyncthingAutomatically=True SyncthingCommandLineFlags=[] SyncthingEnvironmentalVariables=[] SyncthingDenyUpgrade=False SyncthingPriorityLevel=Normal Folders=[] NotifyOfNewVersions=True LatestNotifiedVersion= ObfuscateDeviceIDs=True UseComputerCulture=True SyncthingConsoleHeight=100 WindowPlacement= SyncthingWebBrowserZoomLevel=0 LastSeenInstallCount=0 SyncthingCustomPath= SyncthingCustomHomePath= ShowSynchronizedBalloonEvenIfNothingDownloaded=False DisableHardwareRendering=False HideIntelXeWarningMessage=False EnableFailedTransferAlerts=True EnableConflictFileMonitoring=True ConflictResolverDeletesToRecycleBin=True PauseDevicesOnMeteredNetworks=True HaveDonated=False IconAnimationMode=DataTransferring OpenFolderCommand=explorer.exe "{0}" ShowFileInFolderCommand=explorer.exe /select, "{0}"LogLevel=Info>
2023-05-20 10:33:27.4360 #10508 [Info] SyncTrayzor.Services.Config.ConfigurationProvider: Configuration file C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\data\config.xml doesn't exist, so creating
2023-05-20 10:33:27.4360 #10508 [Warn] SyncTrayzor.Services.Config.ConfigurationProvider: Syncthing doesn't exist at C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\data\syncthing.exe, so copying from C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\syncthing.exe
2023-05-20 10:33:27.4738 #10508 [Debug] SyncTrayzor.Services.AutostartProvider: Have read/write access to the registry
2023-05-20 10:33:27.7588 #10508 [Debug] SyncTrayzor.Services.Conflicts.ConflictFileWatcher: Refreshing conflicted files. Found 0 from 0 options
2023-05-20 10:33:28.2380 #10508 [Info] SyncTrayzor.Syncthing.SyncthingProcessRunner: Starting syncthing: C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\data\syncthing.exe
2023-05-20 10:33:28.7929 #10508 [Info] SyncTrayzor.Services.UpdateManagement.UpdateChecker: No updates available
2023-05-20 10:33:30.5538 #10508 [Info] SyncTrayzor.Syncthing.SyncthingProcessRunner: Starting syncthing: C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\data\syncthing.exe
2023-05-20 10:38:27.7780 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 8MiB
2023-05-20 10:38:28.1569 #10508 [Info] SyncTrayzor.Services.UpdateManagement.UpdateChecker: No updates available
2023-05-20 10:43:27.8072 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 8MiB
2023-05-20 10:48:27.8164 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 8MiB
2023-05-20 10:53:27.8420 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 8MiB
2023-05-20 10:58:27.8469 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 8MiB
2023-05-20 11:03:27.8547 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 9MiB
2023-05-20 11:07:45.6556 #10508 [Warn] SyncTrayzor.Syncthing.Devices.SyncthingDeviceManager: Unexpected device connected: KDVYNEY-TK7YOIR-76FVI3T-POZ3GNP-6NFWH6D-3AZ6WJR-UMOY57V-HMENYAE, address <IP_ADDRESS>:22000. It wasn't fetched when we fetched our config
2023-05-20 11:07:55.9283 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state scan-waiting. Defaulting to Idle
2023-05-20 11:07:55.9283 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state scan-waiting. Defaulting to Idle
2023-05-20 11:08:25.5794 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 9MiB
2023-05-20 11:12:13.2698 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state scan-waiting. Defaulting to Idle
2023-05-20 11:12:13.2698 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state scan-waiting. Defaulting to Idle
2023-05-20 11:13:25.5867 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 9MiB
2023-05-20 11:18:25.6052 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 9MiB
2023-05-20 11:23:25.6230 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 9MiB
2023-05-20 11:27:08.3665 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state sync-waiting. Defaulting to Idle
2023-05-20 11:27:08.3753 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state sync-preparing. Defaulting to Idle
2023-05-20 11:27:08.3753 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state sync-waiting. Defaulting to Idle
2023-05-20 11:27:08.3903 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state sync-preparing. Defaulting to Idle
2023-05-20 11:27:12.5126 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state sync-preparing. Defaulting to Idle
2023-05-20 11:27:12.5126 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state sync-preparing. Defaulting to Idle
2023-05-20 11:27:18.3801 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state scan-waiting. Defaulting to Idle
2023-05-20 11:27:18.3801 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state scan-waiting. Defaulting to Idle
2023-05-20 11:27:37.0641 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state sync-waiting. Defaulting to Idle
2023-05-20 11:27:37.0641 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state sync-preparing. Defaulting to Idle
2023-05-20 11:27:37.0641 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state sync-waiting. Defaulting to Idle
2023-05-20 11:27:37.0641 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state sync-preparing. Defaulting to Idle
2023-05-20 11:27:37.1868 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state sync-preparing. Defaulting to Idle
2023-05-20 11:27:37.1868 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state sync-preparing. Defaulting to Idle
2023-05-20 11:28:25.6316 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 9MiB
2023-05-20 11:30:04.8073 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state scan-waiting. Defaulting to Idle
2023-05-20 11:30:04.8073 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state scan-waiting. Defaulting to Idle
2023-05-20 11:33:25.6386 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 9MiB
2023-05-20 11:38:25.6458 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 9MiB
2023-05-20 11:43:25.6364 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 9MiB
2023-05-20 11:48:25.6644 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 9MiB
2023-05-20 11:49:20.2081 #1244 [Info] SyncTrayzor.Bootstrapper: SyncTrazor version <IP_ADDRESS> (Amd64) started at C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\SyncTrayzor.exe (.NET version: Unknown (528372))
2023-05-20 11:49:20.2550 #1244 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: LogFilePath: C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\logs
2023-05-20 11:49:20.2550 #1244 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: SyncthingBackupPath: C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\syncthing.exe
2023-05-20 11:49:20.2550 #1244 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: ConfigurationFilePath: C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\data\config.xml
2023-05-20 11:49:20.2550 #1244 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: ConfigurationFileBackupPath: C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\data\config-backups
2023-05-20 11:49:20.2550 #1244 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: CefCachePath: C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\data\cef\cache
2023-05-20 11:49:20.2550 #1244 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: DefaultSyncthingPath: C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\data\syncthing.exe
2023-05-20 11:49:20.2550 #1244 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: DefaultSyncthingHomePath: C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\data\syncthing
2023-05-20 11:49:20.2550 #1244 [Debug] SyncTrayzor.Services.Ipc.IpcCommsClientFactory: Checking for other SyncTrayzor processes
2023-05-20 11:49:20.2550 #1244 [Info] SyncTrayzor.Services.Ipc.IpcCommsClientFactory: Found process with ID 10508 and location C:\Users\Dave\Documents\Downloads\SyncTrayzorPortable-x64\SyncTrayzorPortable-x64\SyncTrayzor.exe
2023-05-20 11:51:20.4723 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state scan-waiting. Defaulting to Idle
2023-05-20 11:51:20.4723 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state scan-waiting. Defaulting to Idle
2023-05-20 11:53:25.6819 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 9MiB
2023-05-20 11:53:45.2063 #10508 [Warn] SyncTrayzor.Services.Metering.MeteredNetworkManager: Unable to fetch device state for device ID GI6RZXC-T7PMUQW-7OMONLZ-GVFIVUP-FLJITKL-GQ5RPFP-VDSAS7O-MR6MMAK. This should not happen.
2023-05-20 11:56:38.9072 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state scan-waiting. Defaulting to Idle
2023-05-20 11:56:38.9072 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state scan-waiting. Defaulting to Idle
2023-05-20 11:58:25.6867 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 9MiB
2023-05-20 11:59:48.8710 #10508 [Warn] SyncTrayzor.Services.Metering.MeteredNetworkManager: Unable to fetch device state for device ID GI6RZXC-T7PMUQW-7OMONLZ-GVFIVUP-FLJITKL-GQ5RPFP-VDSAS7O-MR6MMAK. This should not happen.
2023-05-20 12:03:25.6816 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 9MiB
2023-05-20 12:08:25.7055 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 9MiB
2023-05-20 12:13:25.7172 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 9MiB
2023-05-20 12:18:25.7204 #10508 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 177MiB. Private Memory Size: 127MiB. GC Total Memory: 9MiB
2023-05-20 12:20:15.3312 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state scan-waiting. Defaulting to Idle
2023-05-20 12:20:15.3312 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state scan-waiting. Defaulting to Idle
2023-05-20 12:20:36.2686 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state sync-waiting. Defaulting to Idle
2023-05-20 12:20:36.2686 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state sync-preparing. Defaulting to Idle
2023-05-20 12:20:36.2686 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state sync-waiting. Defaulting to Idle
2023-05-20 12:20:36.2686 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state sync-preparing. Defaulting to Idle
2023-05-20 12:20:36.2864 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state sync-preparing. Defaulting to Idle
2023-05-20 12:20:36.2864 #10508 [Warn] SyncTrayzor.Syncthing.Folders.FolderStateTransformer: Unknown folder sync state sync-preparing. Defaulting to Idle

You are trying to use the portable version inside Program Files. That's not how things are supposed to be done 😉. If you want to use the portable version, you should put it in a location where no Admin access is required (e.g. Desktop). On the other hand, if you do want to have the program located inside Program Files, then you should delete the the folders you copied there manually and just use the normal installer to install the program (i.e. SyncTrayzorSetup-x64.exe instead of SyncTrayzorPortable-x64.zip).


Integrate cargo semver-checks into the release workflow

The following PRs add semver checks to the CI
https://github.com/awslabs/smithy-rs/pull/2518
https://github.com/awslabs/smithy-rs/pull/2530
This should be integrated into our release workflow to give us some assurance when pushing frequently to release branches.

I don't think we actually want this since cargo-semver-checks breaks on crate boundaries. This would just end up blocking releases for backwards compatible changes.


Add configs to control how ratios are displayed

Addresses user feedback concerning the readability of recipe input/output counts at some zoom levels.

Uses quilt-config to load in json5 settings
Adds configurable icon and tooltip display
Changes default behavior to show the output count on icon, and the ratio in the tooltip

New default behavior


webui: dynamic: collapse process / parent ID into process name when displaying matches by process

e.g.

I'm not sure the existing PID/PPID column will be of much use to users and removing would tighten up the UI. Open to discussion

This is completed.


Start with hmm file

Description of feature
Add the possibility to specify an hmm file that both the reference and query sequences are aligned to.

Fixed in bf804698b71c80738bb32dd216541d300eaf2699.


Revert "Move user_data to nk_console_top_data"

Reverts RobLoach/nuklear_console#66

PTAL @ #71 - I have it working pretty well, I think, and we can still get rid of the user_data field on nk_console. IMHO the approach I've taken is definitely nicer than the old user_data style which IIRC I added as a quick and dirty way to give callbacks some state to work with


New-PSSession error "subsystem request failed on channel 0" when using SSH key authentication

Reproduction Steps
New-PSSession -SSHTransport -UserName SpockAdmin -KeyFilePath /Users/tsulli/Spocks_Private_Key.pem -HostName x.x.x.x

Symptom

New-PSSession : [x.x.x.x] The background process reported an error with the following message: The SSH client session has ended with error message: subsystem request failed on channel 0.

NOTE: I'm able to successfully SSH into the system using the ssh command line client from Mac.
Fix
Make sure your sshd_config contains the Subsystem configuration as documented in the following article:
https://docs.microsoft.com/en-us/powershell/scripting/core-powershell/ssh-remoting-in-powershell-core
After modifying the sshd_config, you also need to restart the SSH daemon on your Windows instance: Restart-Service -Name sshd.
Closing immediately, as I found the fix. This is being posted purely for community documentation purposes, so others can benefit from my learnings.

sshd_config is located in %ProgramData%\ssh AND NOT where OpenSSH is installed.

Subsystem	powershell pwsh.exe -sshs -NoLogo -NoProfile

Borken Link:
https://docs.microsoft.com/en-us/powershell/scripting/learn/remoting/ssh-remoting-in-powershell-core?view=powershell-7.1


prop-types compatibility

right now the addon does not recognize the propTypes if using the new prop-types module.
import {PropTypes} from 'react'

import PropTypes from 'prop-types'

so is this in the works? are PR's welcomed?

I'm seeing this issue again in v3.2.13.

Please open an issue on the main repository @TimGraf
https://github.com/storybooks/storybook
Please before you make an issue remove all node_modules and lockfiles and re-install.
Likely the issue will be resolved. If not, please do make an issue and we can take a look what's wrong.
Thanks!


Adds error handling and watchdog noticing to both scripts

We are experiencing some SQL errors which is resulting in large batches of reportbacks not being updated. This PR adds a try catch to make sure the script continues running and we watchdog any rbid/sid's that throw this error to allow for further investigation

what's the opposite of DRY?
:+1:

Uh, PERFECTION!

hahahahaha


tensorflow-gpu using only 10% of my GPU

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10 (1809)
TensorFlow installed from (source or binary): binary (pip install tensorflow-gpu)
TensorFlow version (use command below): b'v1.10.0-rc1-19-g656e7a2b34' 1.10.0
Python version: 3.6.8
CUDA/cuDNN version: 9.0
GPU model and memory: GTX 960 2GB

Describe the current behavior
While trying to train a neural network with my GTX960 after installing tensorflow-gpu, and choosing my GPU with the below code, I can see on the Windows task manager that it's only using about 10% of the GPU, and thus making it way slower than training it with the CPU.
config = tf.ConfigProto()
config.gpu_options.allow_growth = False
config.gpu_options.per_process_gpu_memory_fraction = 1
session = tf.Session(config = config)

with tf.device("/device:GPU:0"):
    model = Sequential()
    ...

Describe the expected behavior
The GPU should be used almost if not entirely.
Code to reproduce the issue
you'd need the whole neural network code plus the datasets and the rest...
Other info / logs

python predictor_LSTM_all.py
Using TensorFlow backend.
2019-03-15 17:38:18.822616: I T:\src\github\tensorflow\tensorflow\core\platform\cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2
2019-03-15 17:38:19.109114: I T:\src\github\tensorflow\tensorflow\core\common_runtime\gpu\gpu_device.cc:1405] Found device 0 with properties:
name: GeForce GTX 960 major: 5 minor: 2 memoryClockRate(GHz): 1.2785
pciBusID: 0000:01:00.0
totalMemory: 2.00GiB freeMemory: 1.64GiB
2019-03-15 17:38:19.117459: I T:\src\github\tensorflow\tensorflow\core\common_runtime\gpu\gpu_device.cc:1484] Adding visible gpu devices: 0
2019-03-15 17:38:19.527190: I T:\src\github\tensorflow\tensorflow\core\common_runtime\gpu\gpu_device.cc:965] Device interconnect StreamExecutor with strength 1 edge matrix:
2019-03-15 17:38:19.532341: I T:\src\github\tensorflow\tensorflow\core\common_runtime\gpu\gpu_device.cc:971]      0
2019-03-15 17:38:19.535210: I T:\src\github\tensorflow\tensorflow\core\common_runtime\gpu\gpu_device.cc:984] 0:   N
2019-03-15 17:38:19.538575: I T:\src\github\tensorflow\tensorflow\core\common_runtime\gpu\gpu_device.cc:1097] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 2048 MB memory) -> physical GPU (device: 0, name: GeForce GTX 960, pci bus id: 0000:01:00.0, compute capability: 5.2)
Epoch 1/10
2019-03-15 17:38:21.045228: I T:\src\github\tensorflow\tensorflow\core\common_runtime\gpu\gpu_device.cc:1484] Adding visible gpu devices: 0
2019-03-15 17:38:21.049900: I T:\src\github\tensorflow\tensorflow\core\common_runtime\gpu\gpu_device.cc:965] Device interconnect StreamExecutor with strength 1 edge matrix:
2019-03-15 17:38:21.054218: I T:\src\github\tensorflow\tensorflow\core\common_runtime\gpu\gpu_device.cc:971]      0
2019-03-15 17:38:21.057053: I T:\src\github\tensorflow\tensorflow\core\common_runtime\gpu\gpu_device.cc:984] 0:   N
2019-03-15 17:38:21.060264: I T:\src\github\tensorflow\tensorflow\core\common_runtime\gpu\gpu_device.cc:1097] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 2048 MB memory) -> physical GPU (device: 0, name: GeForce GTX 960, pci bus id: 0000:01:00.0, compute capability: 5.2)
15709/15709 [==============================] - 39s 2ms/step - loss: 33.3039 - acc: 0.1775

(using the CPU only takes about 7-10 seconds to finish the first epoch while here it shows how it takes almost 40 seconds with the GPU.

What is the batch size you are using while training? Too small batch size makes GPU training inefficient.
In addition, are you using tf.keras? Are you using LSTM or CuDNNLSTM? The latter is much, much faster!

What is the batch size you are using while training? Too small batch size makes GPU training inefficient.

I was using a batch size of 20 as my dataset is 15700 samples long only. I just tried a batch size of 1000 and the GPU usage has gone up to 20%, so thanks for that, I had no idea. (I'm gonna have to rise the epochs as well if i want similar results tho)

In addition, are you using tf.keras? Are you using LSTM or CuDNNLSTM? The latter is much, much faster!

Yes, I'm using Keras, and I'm currently using normal LSTMs so I'm going to do some research to thsoe CuDNNLSTM and see if I can modify my code slightly to use them, thanks again!

wow @tlkh I'm in your debt for telling me about cuDNNLSTM. Not only it's faster than LSTM on CPU (which is going to save me quite some time over 111 sensors that I have to train and predict), but it's also going to save my poor CPU of quite a few rough hours.

Can you check with GPU-z. I discovered last week that windows task manager shows max 14%, but it is 100% utilization most of the time on GPU-z.

Yup @realiti4 , using GPU-z does show way higher GPU usage, and while it's only about 40% it probably is because i'm using a small Batch Size for the training.

Hello @ismaelestalayo  , i have the some problem  ,when i run my script python ,  tensorflow-gpu using only 10% of my GPU , please tel me how can be resolve it , i see in you are using LSTM or CuDNNLSTM , please tell me how , thanks in advance.

Im using OS  Linux Ubuntu 18.04,  i have this problem since 2 week , please help me,

Hello @ismaelestalayo , i have the same problem ,when i run my script python , tensorflow-gpu using only 10% of my GPU , please tel me how can be resolve it , i see that you are using LSTM or CuDNNLSTM , please tell me how , thanks in advance.

Sorry to tell you that I did not "fix" it per se (if it even was a bug or error of Tensorflow).
As you can see in my previous message, GPU-Z actually showed a 40% load on the GPU which is way more significant, and the other 60% missing could partly be because of very small batch sizes which are not optimal as another user suggested in this very issue.
Lastly, if you're using LSTM, I recommend you trying CuDNNLSTM, as it's way more efficient and fast (while getting the same results as with the basic LSTMs).

Hi
Thanks for bringing up this issue. When I try to increase the batch size over 14 the training process is crushing. Any idea why?

Hi
Thanks for bringing up this issue. When I try to increase the batch size over 14 the training process is crushing. Any idea why?

First of all, what do you mean by "crushing"? And second, there's no fixed value that works for every dataset, the batch size is just the number of examples in each one of the epoch. The higher the batch size, the more memory you'll need

By crushing I mean training process is failing at the very start of it.
GPU RTX 2080
Ram 32gib
I saw people with worse GPUs setting batch size to 1,000. I can set it to 14 and training process took 4 days to get to 1.5 in the loss function.

By crushing I mean training process is failing at the very start of it.
GPU RTX 2080
Ram 32gib
I saw people with worse GPUs setting batch size to 1,000. I can set it to 14 and training process took 4 days to get to 1.5 in the loss function.

yep, it's weird to crush with such high specs, I'd suggest you Google the specific error that TensorFlow or Python returns. it might be specific to the size of your dataset not accepting high batch sizes or something, I can't tell just by the sheer specs alone.

By crushing I mean training process is failing at the very start of it.
GPU RTX 2080
Ram 32gib
I saw people with worse GPUs setting batch size to 1,000. I can set it to 14 and training process took 4 days to get to 1.5 in the loss function.

I think its a memory overcommitment issue. The solution is to increase the memory size from the default settings. The following link might help you (it worked for me) :
https://stackoverflow.com/questions/57507832/unable-to-allocate-array-with-shape-and-data-type

I have similar problem, but with performance glitch. On low-end PC the step duration is 4ms. With added GPU NVIDIA T4, which is quite expensive, the GPU is used at 25% only and speed improvement is from 4ms per step to 3ms per step. It is improvement of only 25% when adding GPU that costs thousands of dollars?

What is the batch size you are using while training? Too small batch size makes GPU training inefficient.
In addition, are you using tf.keras? Are you using LSTM or CuDNNLSTM? The latter is much, much faster!

I've been working on this for a while but raising my batch size fixed everything. Thank you very much!


Create SECURITY.md

@simontrinh0911

Excuse me, dear friend, I have been involved in the pi network project for
several years with enthusiasm and made many friends interested in the
program, but unfortunately, because my country is under sanctions, I must
be punished instead of my country, if this sanctions has nothing to do with
us people.
That's why I don't have the possibility of kyc. If you think of a way that
will not be unfair to us, help me to pass the kyc stage.
My username is rezasodi@
Referral link minepi.com/rezasodi
Aba Facebook, I connected to pi network with the username rezasodi
In the test phase, I also created a wallet and did the test transfer, but
when to the stage
kyc
I arrived, unfortunately, my country was not able to participate in it due
to the embargo. I was very interested in this project for a long time, but
my horoscope is in a gloomy country, and if anyone can give me a way to
cooperate with your little brother so that I can pass the kyc stage, thank
you. Always carry with you.
Thank you very much for your support, dear friends
Thank you in advance for your cooperation
در تاریخ دوشنبه ۳ اکتبر ۲۰۲۲، ۰۰:۱۵ Abduljlil @.***>
نوشت:

@.**** approved this pull request.
—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/59#pullrequestreview-1127691324,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASAWWU4LKTAKXVWSVU4NILLWBHX6FANCNFSM6AAAAAAQHFH5UE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Why my pi can't Join KYC Please can you help me with KYC right?

My friend i need KYC nima banyi ba
On Sun, Oct 23, 2022, 12:26 PM Jamilu Abubakar Sadiq <
@.***> wrote:

Why my pi can't Join KYC Please can you help me with KYC right?
—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/59#issuecomment-1288090431,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASHVNUYLRS7SBTKVYP4IIQLWEUOE7ANCNFSM6AAAAAAQHFH5UE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Link your phone number with aadhar card

رضا جان من خودم ایرانی هستم اما تابعیت دوگانه دارم چطور میتونم کمکت
کنم،اتفاقا الان ایرانم یه تماس بگیر00989134256698
On Sat, 8 Oct 2022, 15:26 gholamsodani01, @.***> wrote:

Excuse me, dear friend, I have been involved in the pi network project for
several years with enthusiasm and made many friends interested in the
program, but unfortunately, because my country is under sanctions, I must
be punished instead of my country, if this sanctions has nothing to do with
us people.
That's why I don't have the possibility of kyc. If you think of a way that
will not be unfair to us, help me to pass the kyc stage.
My username is rezasodi@
Referral link minepi.com/rezasodi
Aba Facebook, I connected to pi network with the username rezasodi
In the test phase, I also created a wallet and did the test transfer, but
when to the stage
kyc
I arrived, unfortunately, my country was not able to participate in it due
to the embargo. I was very interested in this project for a long time, but
my horoscope is in a gloomy country, and if anyone can give me a way to
cooperate with your little brother so that I can pass the kyc stage, thank
you. Always carry with you.
Thank you very much for your support, dear friends
Thank you in advance for your cooperation
در تاریخ دوشنبه ۳ اکتبر ۲۰۲۲، ۰۰:۱۵ Abduljlil @.***>
نوشت:

@.**** approved this pull request.
—
Reply to this email directly, view it on GitHub
<
https://github.com/pi-apps/pi-explorer/pull/59#pullrequestreview-1127691324
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ASAWWU4LKTAKXVWSVU4NILLWBHX6FANCNFSM6AAAAAAQHFH5UE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/59#issuecomment-1272302196,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQMNJ7VV4CUYOZEQSJOHDILWCFOQJANCNFSM6AAAAAAQHFH5UE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Is this correct?

How I can complete my KYC. ?

I want to complete my KYC

I think Yes
On Sat, Dec 10, 2022, 12:32 PM rookie @.***> wrote:

Is this correct?
—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/59#issuecomment-1345159951,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3FRNLNSJX5JKR6DTSI2EOTWMQTJFANCNFSM6AAAAAAQHFH5UE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Phone number link with aadhar card

Yes I have.
On Sat, Dec 10, 2022, 6:53 PM V1e2n3k4a5t6e7s8w9a10r11l12u13 <
@.***> wrote:

Phone number link with aadhar card
—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/59#issuecomment-1345264343,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3FRNLLKI6UARHX7O3UJGFTWMR74RANCNFSM6AAAAAAQHFH5UE
.
You are receiving this because you commented.Message ID:
@.***>


Add Helm Chart Tests using bats-detik framework

Proposal
Overview
Add comprehensive Helm Chart Test capabilities to Perses helm charts using the bats-detik testing framework.
Motivation
Currently, Perses helm chart lack automated testing. Adding helm chart tests will improve reliability and catch potential issues earlier.
Proposed Solution
Implement helm chart tests using bats-detik framework, which offers several advantages:

Kubernetes-native testing capabilities with built-in assertions
Simple, human readible syntax for writing tests
Declarative test definitions using YAML
Built-in retry mechanisms for handling eventual consistency
Strong support for validating Kubernetes resources

CI Integration
Will update GitHub Actions workflow to include helm testing:
name: "Helm lint and tests"
jobs:
  helm-lint-test:
    runs-on: [ default ]
    steps:
      - name: Create kind cluster
        uses<EMAIL_ADDRESS>
      - name: Run chart integration tests
        run: ct install --config .github/configs/helm-test.yaml

Prior Art
We're already successfully using bats-detik for the helm charts in our kubernetes-based platform. Reference implementation can be found here: https://cloudoperators.github.io/greenhouse/docs/user-guides/plugin/plugin-tests/
If this proposal is accepted then I will create a Pull Request 😀
CC: @nicolastakashi @Nexucis

I really love that idea, we're using it in the OTel helm chart as well 👏🏽

I don't have a strong opinion on this topic. I would say if it is improving the stability of the helm chart and the tests are easy to maintain, then sure we should do that :)


Move current website to org

This will be closed when a new repository at necx.github.io reflects what is currently at arfc.github.io/necx.

done. Website present at necx-org.github.io. Source branch holds the source. Master branch holds the html. The README needs some work.


python312Packages.ms-active-directory: 1.14.0 -> 1.14.1

Diff: https://github.com/zorn96/ms_active_directory/compare/refs/tags/v1.14.0...v1.14.1
Changelog: https://github.com/zorn96/ms_active_directory/releases/tag/v1.14.1
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 341323 run on x86_64-linux 1

4 packages built:

python311Packages.ms-active-directory
python311Packages.ms-active-directory.dist
python312Packages.ms-active-directory
python312Packages.ms-active-directory.dist


Test get_modloader_store()

Ste — Yesterday at 23:52
hmm get_modloader_store in the verbosity seems to return null when in the editor...? I always get level debug. I guess debug is the standard level for the editor, but it also prints as null 🤔

Have to check if the fix from #201 is actually just a glorified return of the non ModLoaderStore values.

The issue is

As @Qubus0 said the Engine.get_singleton() does actually only work for Build in singletons?!
https://godotengine.org/qa/95097/engine-has_singleton-engine-get_singleton-behaving-expected

Most of the ModLoaders work is done in the _init() function where we can't access autoloads or the SceneTree

I ultimately scrapped the work on this and reverted the changes done by #201.
Instead I moved the required util functions into there own files in setup/.
I will closes this after merging the PR


fix pylint unused-wildcard-import warnings

W0614 - Warns when an imported module or variable is not used from a 'from X import *' style import.
Only import what is used.

Hi @gyorb
[Assumption : by wildcard imports you mean import * ]
I was looking into this and I noticed a few things as below :

As per my noob checking, the codebase doesn't have any such wildcard imports , apart from the below in Contributiing.md :

from codeCheckerDBAccess.ttypes import *
So adding an enable in our .pylintrc should be enough.

Also the above API imports mentioned in CONTRIBUTING.md seem obsolete , since most of the codebase actually uses the below standard : from codechecker_api.codeCheckerDBAccess_v6 import codeCheckerDBAccess

These might be wrong insights and any guidance from you on this would be great :)

Hi @meghajain-1711,
Yes I meant that the module contents should not be imported like this import *.

If there is no such import that would be great we try to avoid it.
The issue is still open because this pylint warning is not checked automatically, there are multiple pylintrc files (for CodeChecker and the smaller tools) which could be extended with the unused-wildcard-import rule so they are automatically checked by travis or in the development environment with the make pylint command.
./tools/codechecker_report_hash/.pylintrc
./tools/report-converter/.pylintrc
./.pylintrc
./analyzer/tools/statistics_collector/.pylintrc
./analyzer/tools/merge_clang_extdef_mappings/.pylintrc

Yes the example in the contribution guide is a bit outdated because there was a bigger refactoring for the api modules and we forgot to update the documentation. The modules are not imported like that anymore and update there would be great too, with the example you mentioned.
Do you plan to work on this?

I would love to work on it , if you could please assign it to me :)
Thanks a ton for your clarification and guidance !

No problem let us know it you have further questions!

Thanks @gyorb ! No issues as such, I have updated the pylintrc and contributing docs in my workspace , will be opening a PR soon.
I did not find relevant documentation related to how to test these.
Even though I found this sample json : tools/report-converter/tests/unit/pylint_output_test_files/simple.json.
Kindly guide me to any testing documentation that I might have missed,

There are multiple make targets which you can execute in the top of the repository during the development. There is a short guide for the test targets. Probably the test and/or the contribution guide could be extended with the make pylint and make pycodestyle targets. Those should be executed first during development, however the other test targets like test_analyze execute them before actually executing the other tests so they are not left out.
After updating the pylintrc files you can execute the make pylint command which will only execute pylint on the repository and report if there are any warnings.
The file you mentioned tools/report-converter/tests/unit/pylint_output_test_files/simple.json is just a test file for the pylint report converter. Multiple analyzer and linter outputs can be converted to be stored to the web server.

Hi @gyorb ,
Thanks for all your guidance.
I have updated the pylintrc as we discussed above, tested it and submitted a pullrequest.
I am creating a separate commit with documentation updates :

Contributing.md has API reference updated
Updating make pylint and make pycodestyle in contributing.md and test docs

Hope thats fine :)

Hi @meghajain-1711,
Thanks for your contribution!
Yes, two separate commits with the changes you mentioned would be great.


Question about Routinator's release testing

Hi there,
We use Routinator in our RPKI validation service, and we are trying to add automated release acceptance testing (RAT) for this service. I guess Routinator also has qualification tests before each release (e.g. tests to verify that Routinator is not missing any ROAs) - can you give us some ideas of what tests you have and how they work? We would love to learn about the tests that already exist so that we don't have to repeat them in our own RAT.
Thanks in advance!

Apologies for the late reply, we are busy working on the memory issue.
I’m afraid we currently have no automated test suit for acceptance testing. We have some plans to build tooling that allows you to descriptively define an RPKI data set and the expected results but we just haven’t had time yet to actually build this.
That said, if you’d be willing to make whatever you end up with available, we’d be more than happy to include this in our continuous integration flows and also help maintain and expand them.


Works aggregations without a query term are not filtered for visibility

This is a minor buglet, as non-visible documents shouldn't have any values to aggregate, but the aggregations might be slightly more efficient if we fix it.
Currently, in a Works search, the visibility filter is treated as part of the query that looks for the query term.
If there is no query term, then that query is omitted, including the visibility filter.
This means that the aggregations are operating over a document set containing 1.7Million more records than they need to.

It may be best to fix this as part of https://github.com/wellcomecollection/catalogue-api/issues/677
This requires adding a filter (which is currently in the query document), to the template/params to be inserted in the absence of a query term.
The better solution would be for the template to insert the query document (without this filter) into a must clause in a top-level bool, with the (pre) filters in a filter clause.

At some other time, this may be a job for a more complex ingestor and the constant_keyword feature.
Rather than putting these 1.7M documents in the indexed index, we could only put visible ones in there, and put the non-visible ones elsewhere.
We would have to handle the case where a quondam visible document becomes non-visible (suppress/delete/merge).  This would require a deletion from the visible index alongside the insertion into the non-visible index.

Actually, they are filtered for visibility, but with a clause in each filter aggregation, which is inefficient.  I have implemented https://github.com/wellcomecollection/catalogue-api/issues/677 in https://github.com/wellcomecollection/catalogue-api/pull/717 which fixes that.


Header row has too many columns

I just noticed that when I split the resulting Grin.tsv file using tabs, I end up with 25 columns in the header row...  as opposed to the 13 in the data rows.  Might be a formatting thing.   I looked into it, and I cannot seem to figure out why it's happening.
Command I was using:  cat newest_GRIN_scrape.tsv |awk '{FS="\t";print NF}'

Interesting!
I took a look at the grin-16k.tsv.gz related to release 0.1.3 counted expected amount of tabs tabs (see below, ^I are tabs). Also, the file opened ok in LibreOffice spreadsheet with tab separator.
$ cat -t grin-16k.csv | head -n 1
taxon id^Itaxon name^Idescriptor id^Idescriptor name^Idescriptor definition^Imethod id^Imethod name^Iobserved value^Iaccession id^Iaccession number^Iaccession name^Icollected from^Icitations

This makes me suspect that awk is counting too many field separators in the header file.
Please confirm that you can reproduce my findings.

Can confirm your findings.  I also opened the file with Textwrangler, and saw that there were the correct number of tabs in the header row before posting this issue.  I can't explain why awk is having this issue.

Thanks for checking. Suggest to close issue.


Why

PS D:\work\examination> hover run
hover: Downloading engine for platform windows-debug_unopt at version 92ae191c17a53144bf4d62f3863c110be08e3fd3...
Download completed in 5.62s
Download completed in 2.31s
windows-debug_unopt at version 92ae191c17a53144bf4d62f3863c110be08e3fd3...
How to update?

flutter channel beta
flutter upgrade
hover bumpversion

@pchampio
hover bumpversion
hover: Using engine from cache
go get github.com/go-flutter-desktop/go-flutter@latest: git ls-remote -q https://github.com/go-flutter-desktop/go-flutter in D:\SDK\Go\Path\pkg\mod\cache\vcs\36c8c1c2d8d23519cd4615ae43323bfee7cfa7720be81cabbab32c7f3855b88e: exit status 128:
fatal: unable to access 'https://github.com/go-flutter-desktop/go-flutter/': SOCKS5 read timeout
hover: Updating go-flutter to @latest version failed: exit status 1

sad

hover bumpversion
hover: Using engine from cache
go get<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>invalid version: git fetch -f origin refs/heads/:refs/heads/ refs/tags/:refs/tags/ in D:\SDK\Go\Path\pkg\mod\cache\vcs\da0444fc0003f84a590bd6521553c0531086778bca13ec63ace9e0dd258f4b51: exit status 128:
fatal: early EOF
fatal: The remote end hung up unexpectedly
fatal: index-pack failed
error: RPC failed; curl 18 transfer closed with outstanding read data remaining
hover: Updating go-flutter to @latest version failed: exit status 1

go env -w GOPROXY=https://goproxy.cn,direct

thanks


Shared configuration for SDK-style projects?

I was wondering if the shared configuration approach also works for SDK-style projects? I can't seem to get those projects to honor the settings in the shared .editorconfig file. And I can't see the line
<Import Project="..\packages\Microsoft.CodeAnalysis.FxCopAnalyzers.2.6.3\build\Microsoft.CodeAnalysis.FxCopAnalyzers.props" Condition="Exists('..\packages\Microsoft.CodeAnalysis.FxCopAnalyzers.2.6.3\build\Microsoft.CodeAnalysis.FxCopAnalyzers.props')" />

in the csproj file.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8b2b086b-cda0-2741-2b2b-b2d7e87d0ac9
Version Independent ID: 29465605-4eda-76c5-4254-8976bc17ca37
Content: Configure FxCop analyzers using editorconfig - Visual Studio
Content Source: docs/code-quality/configure-fxcop-analyzers.md
Product: visual-studio-windows
Technology: vs-ide-code-analysis
GitHub Login: @jillre
Microsoft Alias: jillfra

@lothrop -- Kerry, thanks for your question. It looks like you're having an issue with the product itself, rather than an issue with the documentation. Here are a few options where you might consider asking your question:

https://social.msdn.microsoft.com/Forums/vstudio
https://www.stackoverflow.com
https://www.reddit.com/r/VisualStudio/

As well, you might also consider using the Report a Problem tool to report the issue.
I am closing this issue now.  You are welcome to @ mention me for any followup.
We hope to hear from you again.


japanese plugin

YS-jack
YS-jack commented 1 hour ago
Can translates text of most if not all widgets into japanese, can also chat in japanese. can use DeepL api to translate thing more accurately.
This plugin is targeted for Japanese players who can't read English but want to play this game.
Currently some widgets are set not to be translated because I havent provided a translation for that part, and bad translation can cause confusion.
Can selectively choose not to translate if desired in config section.

Please don't close and open new prs. Push commits to the same branch.

sorry about that, i wont

oh wait, i closed it somehow


Failed to open link. Cannot match graph identifier logseq_notebook to any linked graph.

[Issue]
Logseq failed to open the URI with an error message:

Failed to open link. Cannot match graph identifier logseq_notebook to any linked graph.

Search results are correct.
[Env]
fire_seq_search_server v0.0.5
fireSeqSearch for Firefox v0.0.6
Logseq v0.6.9
Windows 10 21H1

Hi @Explorare , thank you for reporting this issue. May I ask for a few more questions about it?

How did you launch fire_seq_search_server? PowerShell? What's the command you were using?
I assume you can see the correct titles on the web page. Is my guess correct?

There is a parameter --notebook_name, I added it, but not documented it yet. https://github.com/Endle/fireSeqSearch/blob/79af3837cab46decdef1fe8bb34145dcb2f9bd77/fire_seq_search_server/src/main.rs#L36
Could you please try
.\fire_seq_search_server.exe --notebook_path C:\Users\li\logseq_notebook --notebook_name LogseqNotebookName

I tried to run the server through cmd and PowerShell 7, and got the same result.
$ .\fire_seq_search_server.exe --notebook_path C:\Users\explo\Documents\Logseq --notebook_name Logseq
[INFO ] save metas
[INFO ] Preparing commit
[INFO ] Starting merge  - [Seg("274d9eca"), Seg("10596cb9"), Seg("0e5df311"), Seg("41750180"), Seg("9e666f97"), Seg("619b8dc7"), Seg("e4e13444"), Seg("32388e75")]
[INFO ] Prepared commit 284
[INFO ] committing 284
[INFO ] save metas
[INFO ] Running garbage collection
[INFO ] Garbage collect
[INFO ] Server::run; addr=<IP_ADDRESS>:3030
[INFO ] listening on http://<IP_ADDRESS>:3030
[INFO ] Searching nlp transformer
[INFO ] Found doc addr DocAddress { segment_ord: 0, doc_id: 39 }, score 15.701874
[INFO ] Found doc addr DocAddress { segment_ord: 5, doc_id: 20 }, score 10.664612
[INFO ] Found doc addr DocAddress { segment_ord: 3, doc_id: 24 }, score 10.265078
[INFO ] Found doc addr DocAddress { segment_ord: 2, doc_id: 0 }, score 10.25067
[INFO ] Found doc addr DocAddress { segment_ord: 5, doc_id: 27 }, score 8.030888
[INFO ] Found doc addr DocAddress { segment_ord: 1, doc_id: 5 }, score 5.200621
[INFO ] Found doc addr DocAddress { segment_ord: 4, doc_id: 5 }, score 2.7373874
[INFO ] Found doc addr DocAddress { segment_ord: 5, doc_id: 11 }, score 1.9310763
[INFO ] Found doc addr DocAddress { segment_ord: 7, doc_id: 11 }, score 1.0827819
[INFO ] Search result ["{\"title\":[\"Transformer Neural Network\"]}","{\"title\":[\"Natural Language Processing\"]}","{\"title\":[\"Research Proposal\"]}","{\"title\":[\"@exploring transformer-based contextual knowledge graph embeddings_ how the design of the attention mask and the input structure affect learning in transformer models\"]}","{\"title\":[\"Response\"]}","{\"title\":[\"Color\"]}","{\"title\":[\"clips.What is a Knowledge Graph_\"]}","{\"title\":[\"hls__Holmström_2021_Exploring Transformer-Based Contextual Knowledge Graph Embeddings\"]}","{\"title\":[\"Knowledge Graph\"]}"]

Click the link in browser won't bring the windows of Logseq to front. And the error message showed up.
I've tried to re-index the graph but didn't help. Tried to create a new repo but didn't help.
There is a new line append to log of Logseq every time I click the link
[2022-05-21 08:21:56.076] [info] open-url {:url "logseq://graph/logseq_notebook?page=Transformer%20Neural%20Network"}

And the URI for a page that Logseq provided is
[2022-05-21 08:33:04.305] [info] open-url {:url "logseq://graph/Logseq?page=Research%20Proposal"}

So I guess the variable logseq_notebook didn't get the right value?

My bad! Sorry for my foolish bug.
The fix is in v0.0.8. Just wait Mozilla to approve my update https://addons.mozilla.org/en-US/developers/addon/fireseqsearch/edit

Commit 62b70ef didn't fix the issue correctly. The value of notebook_name is set to notebook_path by mistake.
[2022-05-21 11:51:58.521] [info] open-url {:url "logseq://graph/C:%5CUsers%5Cexplo%5CDocuments%5CLogseq?page=Natural%20Language%20Processing"}

I forgot to append the --notebook_name option. With the correct value of notebook_name been set, it works as intended. Thanks for the hotfix.


Update table on the STLT Facilities & Providers page to use the new and improved Table component

Problem statement
As part of the STLT Dashboard MVP, we need to implement the design of the table on the Facilities & Providers page. This implementation is strictly visual, and we'll have a separate ticket for hooking it up to the API afterward.
What you need to know

Latest Figma File

Acceptance criteria

[ ] Table should use the new and improved Table component
[ ] Table should sort on all columns
[ ] Table should paginate -- handled by API now
[ ] Table should refresh data when filter button is clicked
[ ] Clicking on ReportId will take you to the existing Report Details page /report-details/

Do not link to Name within rows for now.

To do

[ ] Update Table to use the new and improved Table component
[ ] Table should sort on all columns
[ ] Table should paginate -- handled by API now
[ ] Table should refresh data when filter button is clicked
[ ] Facility Type column needs to apply styling (facilityTypeDisplay)

Penny seems to have implemented this already. So this ticket may not be needed.
Will need to check once the API is integrated into page.

May be completed in the existing work. Will check after it gets completed.

Adding a needs-info label until we get an update from @penny-lischer as to whether this is necessary.

No longer needed. Table component already in. Styling will be updated by #10626.


Use predefined zoom levels

Pull Request Description
Fixes one subtask of #9354
https://github.com/enso-org/enso/assets/6566674/3418d31e-2d80-48b2-ba8b-9e000b09cf14
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[x] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[x] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

Previously we rounded to a zoom level before stepping. This was because some zoom methods, like OS X pinch-zoom, make it easy to make fine changes to the zoom level. If the user then zooms with a stepwise method like clicking the button, if we advance to the next step without considering how close we might be already, our change to the floating point scale factor could make little or no observable difference.
This might be a matter of opinion, but an action like clicking a zoom button has a clear intent and I think it's odd if the UI doesn't appear to react much.

I think this change allows some code simplification:

Applied first two, I still think directedClamp is useful, because it is used in other places and we have quite a few ways of changing zoom, which potentially could lead to value outside of range. (we even export zoom value, so technically anyone can edit it at any time.
I also introduced a small threshold value so that we skip zoom levels which are too close. It can require some more tweaking, but works seemingly fine.


14811- Confirm Business Information step

Issue #: /bcgov/entity#14811
Description of changes:
Restoration - Confirm Business Information step

4 addresses (reuse existing component)
save/resume this data
unit tests

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

Updated package version. Merging now


Astro support?

I’m currently using js-beautify to format astro files with its HTML formatter, which works fine for the HTML part.
The problem occurs when it formats Astro syntax inside --- blocks.
For example, having this file
---
const app = {
  theme: "dark"
}
---

  {
    app.theme == "dark"
    ? "Dark mode activated"
    : "Dark mode is not activated"
  }

results in
---
const themes = {
blue: "sdf"
}
---

  {
  themes.blue == "bg-sky-600"
  ? "The theme is blue"
  : "The theme is not blue"
  }

currently my best approach is to use unformattedContentDelimiter and set it to ---. However, as you can see, the {} inside the HTML part needs to be indented by one level.
My idea is to treat any syntax between the --- blocks and {} as normal JavaScript, just like it is inside a


Would adding support for discrete tweening make sense?

There are many values that are not tweenable, for example, a u8. It would be cool to be able to define using keyframes the values that a function with a discrete output should take for a given floating point input between 0 and 1. This would mean that I could animate non floating point values.
Conceptually I could have converted the discrete value like a u8 into a f32, and then truncated the decimal after the tweening to get the discrete result. But that requires additional work for the user of the API - if they have structs consisting of both floats and discrete types, they would ideally like to have that float use #[derive(CanTween)] instead of manually implementing CanTween by tweening all the floats and then taking all the u8s, converting to floats, tweening, and then casting back to a discrete type

I accomplished this by adding two newtypes on a T - one which tweens them by jumping to the next value in easing, and another that only works on primitive ints that will jump to intermediate ints depending on the time.
Would it make sense to provide these newtypes so that users don't have to write them themselves?

LMK if this looks good and I'll make a PR
use keyframe::num_traits::{AsPrimitive, Float, FromPrimitive};
use keyframe::CanTween;

/// Wraps any `T: Copy`, making it tweenable. Easing jumps from the starting
/// value to the target value when the blend value (`time`) hits `100%` or `1.0`.
///
/// For integer-style types, its more likely that you want to use [`IntTweenable`].
#[derive(Clone, Copy, Debug, PartialEq, Eq)]
pub struct JumpTweenable<T>(pub T);
impl<T> CanTween for JumpTweenable<T> {
    fn ease(from: Self, to: Self, time: impl Float) -> Self {
        if time.trunc().is_zero() {
            from
        } else {
            to
        }
    }
}

/// Makes any primitive int `T` tweenable, by tweening to the intermediary integer
/// values between the two points.
#[derive(Clone, Copy, Debug, PartialEq, Eq)]
pub struct IntTweenable<T: keyframe::num_traits::PrimInt + AsPrimitive<f32> + FromPrimitive>(pub T);
impl<T: keyframe::num_traits::PrimInt + AsPrimitive<f32> + FromPrimitive> CanTween
    for IntTweenable<T>
{
    fn ease(mut from: Self, to: Self, time: impl Float) -> Self {
        let diff: f32 = (to.0 - from.0).as_();
        let scaled_diff: u32 = (time.to_f32().unwrap() * diff) as _;
        from.0 = from.0 + FromPrimitive::from_u32(scaled_diff).unwrap();
        from
    }
}

#[cfg(test)]
mod tests {
    use super::*;
    use keyframe::ease;
    use keyframe::functions::{Linear, Step};

    #[test]
    fn test_jump_tween() {
        let t1 = JumpTweenable(-2);
        let t2 = JumpTweenable(8);

        assert_eq!(ease(Linear, t1, t2, 0.0), t1);
        assert_eq!(ease(Linear, t1, t2, 0.1), t1);
        assert_eq!(ease(Linear, t1, t2, 0.9), t1);
        assert_eq!(ease(Linear, t1, t2, 1.0), t2);

        assert_eq!(ease(Step, t1, t2, 0.0), t1);
        assert_eq!(ease(Step, t1, t2, 0.1), t1);
        assert_eq!(ease(Step, t1, t2, 0.49), t1);
        assert_eq!(ease(Step, t1, t2, 0.50), t2);
        assert_eq!(ease(Step, t1, t2, 0.9), t2);
        assert_eq!(ease(Step, t1, t2, 1.0), t2);
    }

    #[test]
    fn test_int_tween() {
        let t1 = IntTweenable(-2);
        let t2 = IntTweenable(8);

        // ---- Test Linear tween ----
        assert_eq!(ease(Linear, t1, t2, 0.0), t1);

        assert_eq!(ease(Linear, t1, t2, 0.09).0, -2);
        assert_eq!(ease(Linear, t1, t2, 0.10).0, -1);
        assert_eq!(ease(Linear, t1, t2, 0.11).0, -1);

        assert_eq!(ease(Linear, t1, t2, 0.89).0, 6);
        assert_eq!(ease(Linear, t1, t2, 0.90).0, 7);
        assert_eq!(ease(Linear, t1, t2, 0.91).0, 7);

        assert_eq!(ease(Linear, t1, t2, 1.0), t2);

        // ---- Test Step tween ----
        assert_eq!(ease(Step, t1, t2, 0.0), t1);
        assert_eq!(ease(Step, t1, t2, 0.1), t1);
        assert_eq!(ease(Step, t1, t2, 0.49), t1);
        assert_eq!(ease(Step, t1, t2, 0.50), t2);
        assert_eq!(ease(Step, t1, t2, 0.9), t2);
        assert_eq!(ease(Step, t1, t2, 1.0), t2);
    }
}

Looks good to me!
Any reason why 100% is the "jump" point for JumpTweenable rather than, say, 50%? I guess it doesn't matter since you can use Step to make the value jump at 50% anyway.
Regardless, create a PR and I'll merge it. Not sure if tests should be included since the rest of the library has no test coverage right now.

Yeah the choice of the value was arbitrary and can be changed by composing it with step. Sou figured this way was most straightforward


Enhancement add dedicated page for JMESPath

Description
Suggest to expand our documentation a bit with some additional queries and filters for using JMESPath. By moving our current samples to a dedicated page and adding some additional samples we can help our users who read the docs to find their way around how to do more complex filtering / sorting etc with JMESPath.

@pnp/cli-for-microsoft-365-maintainers  if you agree I would like to pick this one up.

Fine for me. Was about to prepare something.

Great suggestion. I'm sure folks will find it very helpful 👏

@plamber feel free to suggest things if you feel I am missing samples!

Hey @appieschot, are you still working on this?

Yes, almost there!


Aborted (core dumped) in tf.math.unsorted_segment_mean

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
No
Source
binary
TensorFlow version
tf 2.14
Custom code
Yes
OS platform and distribution
Ubuntu 20.04
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/compiler version
7.5.0
CUDA/cuDNN version
12.0
GPU model and memory
No response
Current behavior?
Under specific input, tf.math.unsorted_segment_mean encounters "Aborted (core dumped)".
Standalone code to reproduce the issue
import tensorflow as tf

data=tf.constant(value=np.random.randint(0,100,size=(3, 2)), shape=(3, 2), dtype=tf.int32)
segment_ids=3
num_segments=3597855484
tf.math.unsorted_segment_mean(data=data, segment_ids=segment_ids, num_segments=num_segments)

Relevant log output
2024-03-20 06:38:50.497225: F ./tensorflow/core/util/gpu_launch_config.h:129] Check failed: work_element_count >= 0 (0 vs. -697111812)
Aborted (core dumped)

Hi @Zoeeeeey ,
I tried on Colab and getting crash due to Low Memory resource.
2024-03-20 09:41:37.338368: W tensorflow/tsl/framework/cpu_allocator_impl.cc:83] Allocation of<PHONE_NUMBER>6 exceeds 10% of free system memory.
Could you please confirm your RAM size and also could you please test with latest TF version i.e TF2.16v.Since it is possible that few bugs might resolved in latest releases and thus saves our time.

It seems there is integer overflow happening with UnsortedSegmentSum Op which internally being used by tf.math.unsorted_segment_mean. Repro code snippet below & reference gist.
data=tf.constant(value=np.random.randint(0,100,size=(3, 2)), shape=(3, 2), dtype=tf.int32)
segment_ids=3
segment_ids = tf.convert_to_tensor(segment_ids)
num_segments=3597855484

tf.raw_ops.UnsortedSegmentSum(data=data, segment_ids=segment_ids, num_segments=num_segments)

Hi, thank you for your explanation.
In addition, I upgraded the version of TensorFlow and made some attempts, and this issue showed some different behaviors.
Actually, I want to ask, does the code hint Aborted (core dumped) indicate that this is a bug in the current version?

Actually, I want to ask, does the code hint Aborted (core dumped) indicate that this is a bug in the current version?

This must be a check fail that needs to be fixed with proper exception.


Updgrate etcd or move to something else

We initially used the in-cluster etcd because it was easy and have since moved to our own etcd2 Pod for Workbench configuration.   This is problematic on OpenStack because of a known isse with etcd on NFS.  We either need to upgrade to etcd3 or consider moving to another store.  We don't do anything fancy that could be done with sqlite, mysql, Mongo, etc.

Fixed by https://github.com/nds-org/workbench-helm-chart/pull/10


Standalone Crawler gives error for redis

When trying to use it as stand alone crawler without redis it gives error for redis connection.
If you run this:
crawler = CobwebCrawler.new(:cache => 600)
statistics = crawler.crawl("http://www.pepsico.com")
You will get error Redis::CannotConnectError: Error connecting to Redis on <IP_ADDRESS>:6379 (ECONNREFUSED)
While according to documentation, it should run without requiring redis in this case. Thanks

sudo apt-get install redis-server would help you (yes, it's for linux).


✨Clicking on Images opening in new tab.

@adamcogan @theixdev @pierssinclairssw @calumjs @bradystroud
Clicking on an image in a rule will  open the image in open a new tab.
Is this expected behavior?  What should the behavior behind image clicking be?

[x] Discuss appropriate solution.
Copy the image clicking behaviour from https://uxdesign.cc/ux-considerations-when-defining-your-website-imagery-2055bda527ef  (Number 3)

[x] Implement solution.

AB#64642

Tests failed.  Had issues with each of the different out of the box Gatsby zoom components.
Will need to find alternative solutions.

[https://github.com/SSWConsulting/SSW.Rules/issues/729](External Image)


Chunk upload support using TusD service.

This PR will add chunk file upload support in Zulip server and webapp.
Related Discussions -

https://chat.zulip.org/#narrow/stream/6-frontend/topic/File.20upload.20in.20Chunks
https://chat.zulip.org/#narrow/stream/3-backend/topic/Chunk.20file.20upload

First Commit -
dependency: Add tusd as dependency for supporting chunk file upload.
The official v1.11.0 release of TusD dosn't support Ubuntu20.04 and
Debian11, because the binaries are compiled using Ubuntu22.0. That's why
we did our own fork release which compiles binaries using Ubuntu20.04
and works on all supported operating systems by Zulip.
Second Commit -
run-dev: Start TusD server on Developemnt environment.
S3_REGION required if S3 is used for file upload because it is
required in TusD.
Left out work - Make S3_REGION required in setting.
Third Commit -
models: Add file_id field to the Attachment model.
This commit is like a patch to overcome lack of feature in v1 of TusD.
Sending custom data in blocking hooks or taking path_id in pre-create hook will
end requirement of this.
Fourth Commit -
upload: Implement URL to get file path using file_id.
Fifth Commit -
tusd: Add TusD hook handler endpoint.
Currently hanles two hook type pre-create (To verify that user is
authenticated and file size is in limit.) and pre-finish(Moves the
file to proper location and creates attachment row.).
For moving file in S3, there is no optimal method. So we are making copy
of the file with new key and than deleting old key.
After migrating to TusD v2, We will pass path_id in pre-create hook and
this way we will not need to move the file.
Sixth Commit -
upload: Use @uppy/tus at client side to upload a file.
This commit replaces the previoud XHRUpload plugin with Tus to utilize
new chunk upload feature. Chunk size is 25MB, but later we can take this
from server.

The pending tasks for this -

Write documentations for chunk-upload main URL.
Write a tool to cleanup the failed and leftover chunks in tusd folder where all the files get stored initially.
TusD writes a .info file along with main file, we are not dealing with that currently.

Will add more as come to mind...

@zulipbot add "buddy review"
@zulipbot add "mentor review"
@vrongmeal , @Joelute @palashb01 This PR is ready for first round of review.

Thank you @alexmv for detailed reviews, I will try to address all the reviews and let you know when it will be ready for next round of review.

Since we think that will be bounded time, part of me wonders if we should even document that API.

Agree. It seems they will release v2 soon, and the docs will be of no use as I will be working on this till v2 releases so removing API docs, as it is only filling changelog.md and of no use.

@brijsiyag thanks for all the work on this! I did a detailed review of everything but the test suite, and I think this is making very nice progress; I've posted feedback on all the details I noticed.
Aside from these mostly small comments, I think the next experiment worth trying is whether it's possible to rework this by pointing our requirements.txt at the tusv2 branch in upstream's repository and seeing if that lets us cut out the messy parts (You could consider just opening that as a separate clean PR if that feels easier, or trying adding a bonus commit on top that switches). If so, I could imagine our being able to at least test-deploy this one chat.zulip.org without the disruption of a database migration once we've settled the security model issues and start getting some experience running this.

@timabbott, Thank you for detailed reviews. I will update the code as per your suggestions.
Also I would like you to take a look of https://github.com/tus/tusd/issues/964#issuecomment-1609224366 and let me know what are your thoughts on using tusd v2 ?

Yeah let's build on top of v2 given that!

I've rebased and re-pushed.  This version chops off the upload-for-import commit from the tip; I'll push a separate PR with that. This has a couple minor UI issues for failure conditions which I don't think bear me prioritizing right now, but should be otherwise fine to merge.
Cleanup of partial uploads is unlikely to cause much space build-up, and can be cleaned up with ./manage.py delete_old_unclaimed_attachments -C


feat: Check if settings file is empty

Issue being fixed or feature implemented
Fixes issue #5497.
What was done?
Checks if settings file is empty, and deletes it if that's the case.
It will will be generated with default value {} afterwards.
How Has This Been Tested?
Running Dash Qt on regtest masternode with --nocleanup and ./src/qt/dash-qt --regtest --datadir=
Breaking Changes
No
Checklist:

[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 added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation
[x] I have assigned this pull request to a milestone (for repository code-owners and collaborators only)

Will this fix whatever causes the empty file to be written? Or it just allows recovering automatically?


Not able to move or resize the Visual Studio Code Editor Window

VSCode Version: Code 1.11.1 (d9484d12b38879b7f4cdd1150efeb2fd2c1fbf39, 2017-04-06T13:58:12.954Z)
OS Version: Windows_NT ia32 6.1.7601
Extensions:

Extension
Author
Version

vscode-nupkg
eridem
0.0.4

Steps to Reproduce:

1.Lauch the VS Code
2.All operations works fine except resizing and maximizing the window

@senthilkumar79 is that a Windows insiders version?


Fixes Issue # 23589: enable dm.use_deferred_removal and dm.use_deferred_deletion by default.

Fixes #23589
Signed-off-by: Shishir Mahajan<EMAIL_ADDRESS>

What this patch does is:

If the user start dockerd without passing these options, dm.use_deferred_removal and dm.use_deferred_deletion would be enabled by default.
---> If these options are not supported by kernel or libdm, we will only log a warning message to daemon and not blow up. i.e we ll start the daemon, since user did not pass these options. We don't want daemon to blow up on different linux distros, just because these options are not supported, and user did not explicitly passed these options.

If the user start dockerd by explicitly passing these options:

dockerd --storage-opt dm.use_deferred_removal=true dm.use_deferred_deletion=true
```.
It will enable these options just the way it does today.
    ----> if these options are not supported by kernel or libdm, we will throw an error and stop the daemon     start, Just the way it does today. The user will  assume responsibility for passing these options.

Shishir

ping @thaJeztah @rhvgoyal

If somebody is running docker daemon in host mount namespace they need to enable deferred removal and deferred deletion otherwise expect lot of errors of device being busy.
Now even docker upstream has changed their systemd unit file to run daemon in host mount namespace. So it makes sense to enable this feature by default.
When daemon is running in host namespace, it is really easy to lean container mount points into other namespaces. Run a container and just do "unshare -m bash" and now tried to exit that container and it will fail. Enable deferred device removal and container exit will succeed. Mount point still has leaked but once the original bash shell exits, device will be cleaned up.

@thaJeztah any updates on this one ?

What's the side-effects of making this the default? Why did we provide the option to begin with rather than just turning it on?

@cpuguy83
The side effects of making this the default is less errors :)
Everytime user tries to exit a container, when the mount point is leaked. dm.use_deferred_removal and dm.use_deferred_deletion will defer the device removal and deletion, and the user will be able to exit the container without any errors.
I am not 100% sure why we @rhvgoyal made this an option to begin with. But my best guess would be there were not too many errors related to this initially. Now that docker daemon is running in host mount namespace, you will see a lot of mount points leaked and hence a lot of errors on container exit. I guess enabling these options by default will provide a better user experience, rather than user getting an error everytime he/she exits the container.

But there must be negative trade-offs otherwise it should not have been optional.

@cpuguy83
ping @rhvgoyal he would be the best person to answer this.

@cpuguy83 It was just defensive programming. This was so new feature, I kept it off by default. Wanted to first see if it works reasonably well or not.
Having said that, we have figured out a kernel issue with older kernels and that breaks deferred deletion. Upstream and newer kernels don't have that issue.
So deferred removal we can enable by default and I think we might have to put some smart checks to figure out if deferred deletion works well on this kernel or not and if it does, then enable it.

Here is the issue I am talking about.
Do following.

mkdir foo
mount --bind foo foo
In separate terminal window do "unshare -m bash"
In original terminal window "umount foo"
rmdir foo

This last step succeeds with upstream and fedora kernels but fails on centos kernel with message device is busy.
What this means is that when container deletion happens, it will try to remove container rootfs dir and if that directory is still mounted on in another namespace then container deletion will fail.
Shishir, is it possible to put a check during initialization to test this and enable deferred deletion accordingly?

ping @shishir-a412ed @rhvgoyal what's the status on this one?

@thaJeztah
What @rhvgoyal mentioned in the above comment (https://github.com/docker/docker/pull/23590#issuecomment-228814790), the problem here is deferred deletion is only supported on upstream and fedora kernels, but fails on some kernels like centos.
I discussed this with him, and we feel it would be a bit hard to check at runtime if deferred deletion is supported by the underlying kernel. If YES, then only we enable dm.use_deferred_deletion option in docker daemon.
Open to suggestions on how to fix this.
Shishir

@shishir-a412ed would it produce an error that we can detect, and would be able to fall back if it's not supported?

@thaJeztah We can fall back to deferred deletion being off by default if we have the capability to determine what's the kernel behavior. I think we will have to write some dynamic check in docker daemon to determine following.

Create a dir
Bind mount that dir onto itself
Create another mount namespace so that this mount point leaks into that mount namespace.
Unmount dir from oringal mount namespace
Try to delete dir in original mount namespace.

If that dir removal succeeds, that means kernel is new enough to support dir removal even if dir is a mount point in some other namespace. Otherwise not. And that will determine whether deferred deletion should be enabled by default or not.

@rhvgoyal that sounds like adding a lot of complexity :cry:

@thaJeztah yep, it is not very simple and right now I can't think what's the other possible method.

Was thinking if it's a packaging issue (i.e., set those defaults in a systemd-drop-in file), but not sure we want to go that path 😞

I have implemented a C program
https://github.com/shishir-a412ed/Algorithms/blob/master/src/deferredDeletion.c
that checks if deferred deletion is supported by the underlying kernel or not.
We can call this in Cgo or exec the binary. and based on the response if deferred deletion is supported by the underlying kernel, we can enable it by default.
@thaJeztah @cpuguy83 WDYT ?
Shishir

During the maintainers review meeting we discussed this issue of devicemapper config/setup. It seems that already there are a significant number of setup steps that are going to remain manual for a truly proper devicemapper setup.  The only out of the box capability the daemon has is to use loopback which we all know is strongly discouraged except for simple local test/dev where data loss/performance are not a consideration.
Given all that, and adding the apparent complexity of trying to "query" the system's capabilities (e.g. requiring another binary that has to be maintained), it's hard to see a strong need to add this complexity given a real use of devicemapper will already require understanding the docs, creating the right disk configuration, and selecting the startup parameters appropriate for the system being configured.
Is it possible this test tool could be made available and added to the docs for easy configuration for an admin who doesn't want to go understand the kernel and libdm versioning manually as an alternative?
I know in IBM where we have some heavy use of devicemapper this would not have solved our DM configuration issues as testing showed that we actually had backlevel versions of libdm that didn't support deferred removal, so auto-setting the capability would not have helped.

Can we test this outside of docker and setup the systemd unit file to do it properly, or use docker-storage-setup to figure out whether the flag should be set?

@estesp
I agree, the devicemapper setup is quite complex and would always require the admin to do manual steps and go through the docs. So that he can not only just get the right configuration, but also the configuration that is right for him/her.
The main idea to enable dm.use_deferred_removal and dm.use_deferred_deletion be default was to minimize errors users were facing. But we later realized that deferred deletion is not supported by all the kernels, hence a way to detect whether it is supported or not came out as a natural requirement.
Let me know what do you think of this (This is somewhat similar to what you are suggesting)

Host the test tool (https://github.com/shishir-a412ed/Algorithms/blob/master/src/deferredDeletion.c) in a separate github repository.

Update the docker docs to point to that repository link. What the docs would say:
a) Docs should explain the problem Statement:
If a user is running docker daemon in host mount namespace, there is a high probability that container rootfs mount points can get leaked into another mount namespace. Which means if he tries to do docker rm $ContainerID he might get an error device busy.
b) Docs should propose a solution:
To avoid this error the user can enable these options (dm.use_deferred_removal and dm.use_deferred_deletion) at daemon start time. However deferred_deletion is not supported by all kernels so he can use this tool to first detect whether his kernel support this functionality or not.

c) If this tool returns TRUE (0 error code) then docs should tell how to enable these options.
WDYT ?
Shishir

Closing this PR, as we have handled this in docker-storage-setup (d-s-s).
Shishir


missing dev dependancies (babel-core, babel-loader)

babel-core and babel-loader are missing in the dev dependancies. these are needed to run ./node_modules/webpack/bin/webpack.js -d

committed and pushed


Missing Files

Thank you! this is very useful, can you try again? i think i fixed that error

Yes it works again. Thank you very much this looks beautiful

Happy to hear that! sorry if the code is a mess, you can change the colors in awesome/utils/colors.lua, all the bar configuration is in themes/nordish/theme.lua and if you want to change your avatar in the menu, change this line to the path of your image


Exposing an External IP Address to Access an Application in a Cluster

I successfully installed dind-cluster on my laptop using 1.8 sh. I then followed steps from k8s tutorial:
https://kubernetes.io/docs/tutorials/stateless-application/expose-external-ip-address/
It is possible to access the service from inside the cluster.
However, external port never gets assigned, EXTERNAL-IP stays forever in pending:
sava@DellXPS:/mnt/c/k8s/helloworld$ kubectl get services my-service
NAME         TYPE           CLUSTER-IP     EXTERNAL-IP   PORT(S)          AGE
my-service   LoadBalancer   <IP_ADDRESS>   <pending>     8080:31339/TCP   21m

Could somebody give me a hint if / how would be possible to expose LoadBalancer service on the host with running kubeadm-dind-cluster? Could https://github.com/Mirantis/k8s-externalipcontroller be used?

for LB I think you have to try ingress
I actually trying to do something similar, I've created ingress for my service and now I have an endpoint:
kubectl get ingress --all-namespaces
NAMESPACE   NAME          HOSTS              ADDRESS      PORTS     AGE
che         my-ingress   <IP_ADDRESS>.nip.io   <IP_ADDRESS>   80        22m

but that endpoint <IP_ADDRESS> is not accessible from the the host :(
if your external-ip is in pending state it probably means that there is nothing to handle that so k8s-externalipcontroller could help, give it a shot

The easiest way to access service is to use service type=NodePort


Add login and signup

Added login and signup auth flow...
When submitting the forms, it calls fetch and passes the data to the api. When the response is returned, it sets the response in AsyncStorage so it can be read by the app, and also handles redirecting to the right pages.

Looks good :+1: , I can look more into the fetch / response.json() issue and try to get the error message from the API successfully


Globalize some of the utility functions from _gamemode_fd_events.nut

Just a small pr to globalize some of the utility functions found in gamemodes/_gamemode_fd_events.nut so custom wave event registration functions can be added more easily without needing to modify the script itself. Also, VS code had a field day removing whitespaces and replacing with newline apparently...

PR not needed anymore, included features together with the Full Wave Data Rework update, gamemode_fd_events.nut is not even file used by the game anymore.


Remove trailing whitespaces

I noticed multiple trailing whitespaces in the different files. I open this issue ti link it to the MR I will make in futur.
Just to make this repo nicer :-)
I like it by the way. Many thanks to those contributing on it !

Hi, I just don't understand why my PR has been closed ?

Issue closed cause no maintainers looking at MR nor issues.


[FIX] - sap.m.RadioButtonGroup throws unhandled exception

When adding a selection handler which modifies the RadioButtons and the selected radio button in the RadioButtonGroup, an unhandled exception occurs.

The fix is to only focus the DOM reference if it exists.

Fixes: https://github.com/SAP/openui5/issues/3971

Fixed by 7dcef696d02b924a246e482c10a64e4f33c012c3


🐞[bug]: the retro text and the sign-up form are overlapping

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
The retro text ("Retro Vibe is Good Vibe") and the sign-up form are overlapping, causing both elements to appear on top of each other. This makes the form difficult to use and the text hard to read.
Expected behavior
.The retro text should be clearly visible without obstructing or overlapping with the sign-up form.
.The sign-up form should be easily accessible and visually distinct, with no interference from background text or other elements.
Add ScreenShots

What browsers are you seeing the problem on?
Chrome
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24EXTD contributor
[X] I have starred the repository

@iwanturequity that is feature , their no overlapp , please create someother issue


test error

I got this error.
Traceback (most recent call last):
File "main.py", line 290, in
main()
File "main.py", line 45, in main
evaluate(args)
File "main.py", line 258, in evaluate
output = utils.color_match(output, style_v)
File "/media/hdd1/donghyun/armor/PyTorch-Multi-Style-Transfer/experiments/utils.py", line 150, in color_match
src_flat_nrom_trans = matSqrt(dst_flat_cov_eye) * matSqrt(src_flat_cov_eye).inverse * src_norm
RuntimeError: mul() received an invalid combination of arguments - got (builtin_function_or_method), but expected one of:

(float other)
didn't match because some of the arguments have invalid types: (builtin_function_or_method)
(Variable other)
didn't match because some of the arguments have invalid types: (builtin_function_or_method)

How can I fix this?

do you resolve it, I face similar problem
src_flat_cov_eye = src_norm @ src_norm.t() + Variable(torch.eye(3).cuda())

Yes. In my case, I resolved it with this modification.
src_flat_nrom_trans = matSqrt(dst_flat_cov_eye) * matSqrt(src_flat_cov_eye).inverse() * src_norm

which file do you modify?
@donghyunyoo

the paper said support "Brush Stroke Size Control." do you know how to evaluate in different brush stroke size
@donghyunyoo

@suke27
did you solve it?
src_flat_cov_eye = src_norm @ src_norm.t() + Variable(torch.eye(3).cuda())

not yet

@donghyunyoo I make the same modification as you, but then a new error occurs as follow:
RuntimeError: The size of tensor a (3) must match the size of tensor b (655360) at non-singleton dimension 1
Did you ever encounter this error? I don't know if I could change the data formation of this  script.

@gwenguo090006
I just ran into the same error, which seems to originate from the latest commit (didn't have any error before that).
The error comes from  utils.color_match that is now used right before saving the output in main.py.
A quick fix is for now to comment out line 258 in main.py :
output = utils.color_match(output, style_v)
It seems to work for me

@LoicEm
yes, your solution is feasible. And then I step into the color_match function, I think it is used to render  the output image further. The error occurs when the multiplication between two matrices . I don't know why this happened, I don' think it's a confusion of the author between element product and matrix product. But when I modify the element product to matrix product, it works. However, the result is a little different from the case when you just comment out line 258 but it's also a style-transfer result of the original image.


Java input

i just now realized that input needs to be provided before running the program.
This links to an issue i previously opened concerning a local copy of openJDK to run Java locally, but even if server-side execution is the only way would it not be possible to ask for input in a behavior identical to a desktop environment (meaning not upfront / before running the program)?

though a local distribution of Java could render this issue meaningless / solved.

though a local distribution of Java could render this issue meaningless / solved.

if not something similar to what replit did could be done, they use server-side execution but ask for input in a desktop like manner

Thank you for the suggestions. I intend to bring a local runtime of Java to Code App.
These projects should be good references:

https://github.com/PojavLauncherTeam/openjdk-multiarch-jdk8u
https://github.com/PojavLauncherTeam/jdk17u

Thank you for the suggestions. I intend to bring a local runtime of Java to Code App.
These projects should be good references:

https://github.com/PojavLauncherTeam/openjdk-multiarch-jdk8u

https://github.com/PojavLauncherTeam/jdk17u

these both look very promising, glad you're looking into it.
Thanks for your hard work and quick responsiveness.

@bummoblizard would you happen to be able to guess an estimated date after which local Java support should be realized?

@bummoblizard would you happen to be able to guess an estimated date after which local Java support should be realized?

Hi. I'm focusing on fixing the existing issues. I will work on it after that.

Appreciate it, i found this App that has a local Java 9 runtime that works. (I have tested it without internet) To use it you have to pay around 3€ but it works.
I'm not fond of the User experience and UI at all but it still could end up helping you implement a local Java runtime yourself.
https://apps.apple.com/app/id992268206

Local OpenJDK is added in https://github.com/thebaselab/codeapp/commit/39155744844ad472f0b79c714230a8691a7832bb. However, interactive input is still not supported.

Fixed in https://github.com/thebaselab/codeapp/issues/1036


SR-2262 Generated Xcode projects should leave the boilerplate framework Info.plist files out of the project hierarchy

https://bugs.swift.org/browse/SR-2262 Generated Xcode projects should leave the boilerplate framework Info.plist files out of the project hierarchy

I don't know if this is in any way controversial or not, but to me it's pretty obvious cleanup.  Please comment if you disagree.

@swift-ci Please test

The macOS test seems to have failed for unrelated JUnit-related issues.

Seems reasonable to me, since they are not authored data.


Minor change reg assign this.state

Updated state and lifecycle section to read this.state can be assigned in constructor or
as a class variable.

@gaearon do you think it's work document the class prop syntax here? It's getting pretty common in the wild and undeclared state is a common source of bugs as people start omitting the constructor in favor of bound class props.

@alexkrolick thanks for the review! I was actually not aware that class props were not a standard yet. I saw them being used in several courses on eggheadio, blogs, twitter etc. and thought it was a standard and hence gave the PR as I couldn't find it in the docs.

Class properties are now a stage 3 proposal and the semantics for public properties at least are pretty well established: https://github.com/tc39/proposal-class-fields.
Maybe according to the contributing guidelines, you could say it's still too early to talk about them in the React docs, but actually they are already mentioned elsewhere, for example here:

If you are using the experimental public class fields syntax, you can use class fields to correctly bind callbacks
https://reactjs.org/docs/handling-events.html

And actually that suggestion is arguably much more potentially problematic than the suggestion of using a class property to set initial state. In the future it looks like a better alternative to arrow functions in class properties to bind methods will be to use a decorator for this (but the Babel plugin for the latest decorators proposal is still in development). For details, see https://github.com/tc39/proposal-decorators/issues/54#issuecomment-369813610


pitch segment doesn't work

As reported in https://github.com/Shirakumo/harmony/issues/23 , pitch segment doesn't work.

It seems that make_pitch_data incorrectly allocates memory regions, I was able to fix it by increasing size in 4 times.

I've known about this for ages and never got around to fixing it. Thanks for picking up my slack!


Change the text on this screen

Replace "Let's discover everything web 3 has to offer" by : "Explore the world of true identity ownership."

on talao branch

not done


remove package juzna/intellij-latte

deprecated https://componette.com/juzna/intellij-latte/
idea: button in footer for "report addon" ?

Addon is archived. :-)
Marked as nice-to-have for report addon.

[ ] report addon


icons are not getting reflected

Here are the steps to reproduce
Step 1: Generate a d2 file which contains an icon, check out the attachment please
Step 2: Create a template file, check out the attachment please
dictim.edn.txt
in2.d2.txt
out2.d2.txt
Step 3: Generate the d2 file by using the dictim commands
Step 4: Compile the d2 to svg icons are missing.

icon is an attribute.
This occurs because in the command line tool in apply-template functionality, after the parse phase and just before the template is applied, it's currently hard-coded to remove any attributes from the parsed d2.
this is fixed in f0e9986 (but currently unreleased)
Now existing attributes are retained and template source attributes merged over them, unless the -r option is explicitly invoked in which case all attributes are removed just after the parse phase, as per the previous behaviour.


Automatic new line at the end of file

I think the current version doesn't allow that feature. It will satisfy my git repository if you could add this.

I have a patch for this - should it be the default, or should it be an option? @sschaef / @fommil ?

It needs to be optional. We need to stay backwards compatible.

Great. Feel free to assign to me if you like - I'll put in a patch shortly.

@jkinkead thank for for all your recent work on scalariform, it really needed a hero to come along and get it moving again! :smile:

@jkinkead FYI this is the goal for 1.0 https://github.com/scala-ide/scalariform/milestones/1.0 and might answer your question above. Backwards compatibility is absolutely mission criticial. It can impact teams upgrading their IDE, with scalariform regressions becoming major operational issues.

Thank you for taking charge of this project! I don't have the time to do the real project maintenance work - but I'm happy to contribute patches! We use this a ton at AI2, and it's great to have regular releases being made again . . . that I'm not in charge of. ;)


Add candidates to CDU

Update json to have candidates to CDU
Fix description and description_source for coalition
Fix populate script

Note: to repopulate you need to do:
Add to db/seeds.rb:
Candidate.destroy_all
Party.destroy_all

Then run:
bundle exec rake db:seed

This will delete all entries in those tables. Now you can remove these lines from db/seeds.rb.
Finally run:
bundle exec rake data:populate

Não resultou comigo @luisiul359 , já tinhamos tentado isto ontem.
Tentei novamente e nada…