Sync main

Will trigger release PR

Codecov Report
Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (8a02062) 83.29% compared to head (6a48862) 83.18%.
Report is 49 commits behind head on main.

:exclamation: Current head 6a48862 differs from pull request most recent head 330c604. Consider uploading reports for the commit 330c604 to get more accurate results

Files
Patch %
Lines

src/lib/encoder.ts
75.00%
0 Missing and 4 partials :warning:

src/index.ts
75.00%
1 Missing :warning:

src/lib/utils.ts
50.00%
0 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #369      +/-   ##
==========================================
- Coverage   83.29%   83.18%   -0.11%
==========================================
  Files          19       19
  Lines        1221     1231      +10
  Branches      276      282       +6
==========================================
+ Hits         1017     1024       +7
  Misses        110      110
- Partials       94       97       +3

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


Class constructor MDCRippleFoundation cannot be invoked without 'new'

Hello,
I have this problem with vuejs and laravel.
Specifically in the ripple class:
export class RippleBase extends MDCRippleFoundation {   static get MATCHES() { .......}}
When I navigate to a page, and execute this code :
export const RippleMixin = {   mounted() {     this.ripple = new RippleBase(this)     this.ripple.init()   } }
My console show the folllowing output :

Is there any way to solve this?

@rsjavier14
which version of vue-mdc-adapter ?
can you share your package.json dependencies ?

Hey @stasson ,
thanks for answering. I am using version 0.17.0, I leave the dependencies:
 "devDependencies": {
    "autoprefixer": "^9.1.0",
    "babel-core": "^6.26.3",
    "babel-loader": "^7.1.5",
    "babel-preset-env": "^1.7.0",
    "babel-preset-stage-3": "^6.24.1",
    "css-loader": "^1.0.0",
    "extract-text-webpack-plugin": "^4.0.0-beta.0",
    "file-loader": "^1.1.11",
    "mini-css-extract-plugin": "^0.4.1",
    "node-sass": "^4.9.2",
    "postcss-loader": "^2.1.6",
    "resolve-url-loader": "^2.3.0",
    "sass-loader": "^7.1.0",
    "vue-loader": "^15.3.0",
    "vue-style-loader": "^4.1.1",
    "vue-template-compiler": "^2.5.17",
    "webpack": "^4.16.5",
    "webpack-cli": "^3.1.0"
  },
  "scripts": {
    "dev": "webpack --watch --mode=development"
  },
  "dependencies": {
    "classnames": "^2.2.6",
    "font-awesome": "4.7.0",
    "material-components-web": "^0.38.0",
    "material-icons": "^0.2.3",
    "vue": "^2.5.17",
    "vue-mdc-adapter": "^0.18.1"
  }

Delete all the dependencies, reinstalled them and reconfigured everything and it works. I could not explain what the problem was. The issue can be closed. Thank you


Set ABAP Language Version in settings page of repository

The concept of having a repository attribute for the ABAP Language Version (ALV) has been discussed and suggested in #6154.
PR #6346 implemented a basic functionality that determines the ALV in the current system and negotiate it further with the ALV from the settings (on repository level).
During testing your implementation that adopts the above described changes for object types CLAS, INTF and other source-based ones tracked in #6426, we found that we cannot set the ALV of a repository via abapGit. With that I mean, we cannot set the ALV in the settings page of the repository, e.g. in this screen

I'm in the process of some refactoring of repo settings and related UI.
I can add the ALaV setting afterward as an experimental feature.

I see, there is already an issue for tracking this feature.
➡ Closing this issue here.


Add and run Docker image for etcd

Add and run Docker image for etcd

This works well!
lauramartin@integration-docker-management-1:~$ sudo docker ps
CONTAINER ID        IMAGE                         COMMAND                  CREATED             STATUS              PORTS                              NAMES
c1094814fa6c        quay.io/coreos/etcd:v3.0.13   "etcd --listen-client"   3 minutes ago       Up 3 minutes        <IP_ADDRESS>:2379-2380->2379-2380/tcp   etcd
lauramartin@integration-docker-management-1:~$ sudo nc -vz localhost 2379
Connection to localhost 2379 port [tcp/*] succeeded!

After discussion reverted my extra commit. /cc @deanwilson


GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Java PreCommit

Run Java PreCommit

Thanks @TobKed :)


Issue #6449 fixed

Issue Reference
#6449

Create default vuejs object of download links if empty

I will create a fresh PR for this issue, so I am closing this pr


If running in editor, always disable window mode features ("always on top"/"transparent"/"clickthrough")

I sometimes accidentally click play in editor scenes, the windows gets all messed up and I have to restart Unity.
It'd be nice to have an option to disable the window options while in editor mode (always on top/transparent/clickthrough).
I can't think of any cases where it should be interesting to have them enabled on editor.
A warning could be displayed instead.

Thank you, I added this in https://github.com/screenshakes/Desktopia/commit/bd6f2e5cfbf7fa6849c456ff679331d3de9a7eba.


Add Apple to list of users

usage noted here https://support.apple.com/en-ca/guide/security/sec469d47bd8/web

XProtect has been using YARA for a long time, but we are adding companies to the "who's using YARA" list when the request comes from someone representing the company. Unfortunately nobody from Apple has reached out asking for this, in fact I'm very surprised that they mentioned YARA explicitly in this article. This is the first time I see that they say publicly that XProtect uses YARA under the hood.

@plusvic okay, thanks for the explanation!


Nette 2.3 compatibility

I have changed required version of Nette to 2.3.*. There are no BC breaks.

Similar to #16

Sorry, you are right.


re-initialize after data has changed

Is there any way to update the slider with new data after a $http request? I am using ng-repeat to iterate through the data i get from the $http request, however the view doesn't get updated after the initial request.

@marcheluis Could you share your solution?

You should be able to set a watch on the model for updates and then within the callback, execute swiper's update method. You could even append the updates, but a caveat might be the slide index if you are using that.

For anyone who ever has this issue, here is a codepen implementing @brh55 solution
http://codepen.io/dmastag/pen/zqzWGK


Model 5.0 Artifact

Model 5.0 Artifact

Description

Category:
artifact
JIRA issue: MIC-4459
Research reference:
https://vivarium-research.readthedocs.io/en/latest/models/risk_effects/child_growth_failure/cgf_2021/index.html#risk-effect-cgf

Changes and notes

Added PAF for CGF from CSV from RT
make stunting data
add underweight RR
Verification and Testing

Artifact builds and works in simulation

One thing to note about the input data--The risk effects are on incidence and EMR; the docs suggest that GBD data  comes in terms of CSMR, and therefore must be converted to EMR, but the relative risk data appears to already be in terms of EMR as an affected entity. Being that that is so, there is not additional transformation to do there, but we should ensure with Ali or Syl that we aren't missing something there.


[0.21] Adding new image for the migrator

Signed-off-by: Matthias Wessendorf<EMAIL_ADDRESS>We need to build the linux container for the "post install" job. We use the vendored lib for that.

/lgtm


POD_CIDR not found on metadata.google.internal

The instruction to set POD_CIDR=$(curl -s -H "Metadata-Flavor: Google" \
http://metadata.google.internal/computeMetadata/v1/instance/attributes/pod-cidr)
errors out with a vague message.
The error message I get from running this is:
The requested URL <code>/computeMetadata/v1/instance/attributes/pod-cidr</code> was not found on this server. That's all we know.
Is there a fix for this?

Issue was using the controller instead of the worker node.


Fix #230, added hardcoded strings in string.xml

Summary of changes
I simply added few strings in the string.xml and there were two tags with different key but same value, so I also corrected that.
Screenshots or a brief video showing the change in action:
Please make sure these boxes are checked before submitting your pull request - thanks! Guide

[x] Code is formatted.

[x] Run the lint tests with ./gradlew lintDebug and make sure BUILD is SUCCESSFUL

[x] If new feature is implemeted then it is compatible with Night theme too.

[x] Commit messages are well-written.

Automated message from Dropbox CLA bot
@abhaymaniyar, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here.

Oh sorry! my bad! @saketkumar95

Automated message from Dropbox CLA bot
@abhaymaniyar, thanks for signing the CLA!

@abhaymaniyar No problem! :)

superseded by #424


Use of Vitessce App with localhost not working

I am using Vitessce App in the latest beta version.
http://beta.vitessce.io/app/
I am uploading a config file in JSON format and the URLs from localhost. However, the App keeps loading without providing any results. I also tried to change the URLs in the example file with URLs from my local server, and it keeps loading.
Do you have any idea why this is happening?
Please find attached the config file I am using. It is set to work on a local server and uses the example files (cells.json and cell-sets.json) you use in your example and follows the format of your example config file.
Many thanks for considering my request.
config-test.json.tar.gz

@demiKar Thank you for your interest in Vitessce!  Do you have CORS enabled? Can you look at your network requests in your browser to see what the errors are?

Hi @ilan-gold. Thank you for your quick response. I am still having issues with the local server. I enabled the CORS, but it is still not loading.
I am trying the vitessce-r version now. I had several issues loading the package. I tried on several PCs to run your example, and the code is still loading.
I would also like to ask whether it can load a Giotto object in the R version. If so, is there any documentation I should follow for spatial transcriptomic analysis?
Many thanks for considering my request.

@demiKar In which web browser are you experiencing this issue? Google Chrome recently restricted public websites from accessing private (including localhost) resources: https://developer.chrome.com/blog/private-network-access-update/
Until we figure out a better long-term solution, you may need to override this restriction by disabling it in chrome://flags/#block-insecure-private-network-requests

Until we figure out a better long-term solution, you may need to override this restriction by disabling it in chrome://flags/#block-insecure-private-network-requests

Another workaround is to use a different browser (Firefox or Safari) for the time being, since the access isn't blocked by default.


Prototype a Flask alternative

Flask may have significant performance issues, due to Werkzeug's approach to the Python 2 > 3 conversion. Its syntax is pretty great, though for the purposes of just writing REST endpoints it has more features and overhead than needed.
Possibilities:

Migrate to Bottle: Very similar to Flask, lighter weight. However, code is difficult to read, has some quality issues.
Migrate to Bobo: Similar to Flask. Not very popular.
Migrate to Falcon:  Great performance, but syntax would require significant code changes, probably offers much more features than needed.
Migrate to wheezy.web:  Excellent performance. Very new. Syntax is more in the web.py style, which would require some reworking.
Build a custom framework on top of WebOb: Decent performance. Significant work. WebOb is not simple.
Build a custom framework on top of wheezy.http: Excellent performance. Significant work, but the base library is rather simple. Not as battle tested as other strategies.
Build a completely custom framework: Can build exactly what is required, and nothing more. Large amount of work. Would require reinventing the wheel quite a bit.

For custom framework options, see http://ains.co/blog/things-which-arent-magic-flask-part-1.html


There should be a task to install all binaries of osv-scanner of a particular version

Implemented in version 2.0


Windows Command Scripts CleanUp

RESTART now STARTS if the proxy is not already running (I see you changed it to Stop/Start; I went with a test where it will start if not already running and restart otherwise).
Single call to CD instead of calling in two places.
IF ERRORLEVEL instead of using the pseudo-variable.
No need to specify DNSCrypt-Proxy.exe path since current directory searched first.
ECHO OFF & Enable Extensions.
ECHO. instead of echo'ing double quotes (which actually prints the quotes).
New Stop script.

Merged, thank you!


version up

Coverage remained the same at 93.636% when pulling e75b4cae31dca948d218f1c454203ba71f207192 on versionup into 1b078986b0df251a8da4ca03a0a789b6cfcccc1f on master.


Shared my portfolio

Added my portfolio in the repo.
Visit: https://shubhayumajumdar.tech

@shubhayu-64 Are you an SSOC participant?

Yes

Yes

Sorry I can't merge your PR then. You can create PRs after the contribution period starts.


[Automated] Update eventing-istio-eventing-kafka-broker nightly

/assign @knative-sandbox/eventing-wg-leads
/cc @knative-sandbox/eventing-wg-leads
Produced via:
for x in eventing-kafka-broker.yaml eventing-kafka-controller.yaml eventing-kafka-source.yaml eventing-kafka-channel.yaml eventing-kafka-sink.yaml; do
  curl https://storage.googleapis.com/knative-nightly/eventing-kafka-broker/latest/$x > ${GITHUB_WORKSPACE}/./third_party/eventing-kafka-broker-latest/$x
done

Details:

/approve


radiomics.uk - Bad gateway Error code 502

Dear pwoznicki,
Great work with RadiomicsHub!
Your website seems to be down: Bad gateway Error code 502

Best
Simon

Thank you Simon and sorry for the invonvenience. You should now be able to reach it again (https://radiomics.uk/).
Best regards, Piotr


bench: add split to irmin-pack trace replay

This PR adds the new split function to the trace reply in bench/irmin-pack.
We will likely want to update these benchmarks/trace replays to follow the changes that @Ngoguey42 is making to our lib_context replay benchmarks, but this change is needed to for the GC in the replay to continue working since split is needed for the new chunk-based GC to function correctly.

Codecov Report

Merging #2135 (87fc55e) into main (3dd2231) will decrease coverage by 0.07%.
The diff coverage is 52.32%.

@@            Coverage Diff             @@
##             main    #2135      +/-   ##
==========================================
- Coverage   68.21%   68.13%   -0.08%
==========================================
  Files         134      134
  Lines       16031    16042      +11
==========================================
- Hits        10935    10931       -4
- Misses       5096     5111      +15

Impacted Files
Coverage Δ

bench/irmin-pack/trace_replay.ml
42.10% <0.00%> (-0.58%)
:arrow_down:

src/irmin-pack/unix/pack_index.ml
63.88% <ø> (ø)

src/irmin-pack/unix/gc_worker.ml
6.06% <16.00%> (+2.30%)
:arrow_up:

src/irmin-pack/unix/chunked_suffix.ml
85.84% <52.17%> (-10.63%)
:arrow_down:

src/irmin-pack/unix/file_manager.ml
83.92% <75.00%> (-0.26%)
:arrow_down:

src/irmin-pack/unix/dispatcher.ml
73.97% <86.66%> (-1.03%)
:arrow_down:

src/irmin-pack/unix/gc.ml
79.16% <90.90%> (-2.98%)
:arrow_down:

src/irmin-pack/unix/ext.ml
62.25% <100.00%> (-0.15%)
:arrow_down:

src/irmin-pack/unix/import.ml
82.35% <100.00%> (+2.35%)
:arrow_up:

... and 9 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thanks!


[Snyk] Upgrade ace-builds from 1.32.4 to 1.34.2

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to upgrade ace-builds from 1.32.4 to 1.34.2.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 12 versions ahead of your current version.

The recommended version was released on a month ago.

Release notes

Package name: ace-builds

1.34.2 - 2024-05-27package 27.05.24

1.34.1 - 2024-05-24package 24.05.24

1.34.0 - 2024-05-22package 22.05.24

1.33.3 - 2024-05-21package 21.05.24

1.33.2 - 2024-05-13package 13.05.24

1.33.1 - 2024-04-23package 23.04.24

1.33.0 - 2024-04-12package 12.04.24

1.32.9 - 2024-03-29package 29.03.24

1.32.8 - 2024-03-22package 22.03.24

1.32.7 - 2024-03-01package 01.03.24

1.32.6 - 2024-02-07

1.32.5 - 2024-01-30

1.32.4 - 2024-01-29

    from ace-builds GitHub release notes

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Already updated to 1.32.4


Co-op host progress successfully transferred to dedicated server, but friends are being assigned new GUIDs and must create new characters.

Hello, I managed to follow the guide and made it so that I as the host of my group's co-op world was able to transfer my progress to our dedicated server (which we bought from a third-party server hosting website), however my friends aren't keeping the same GUIDs they had from our co-op world when they join the dedicated servers, and they are forced to create new characters. I noticed that new GUIDs are being assigned to them because after one of my friends (who had previously played in co-op) joined and they got to the character creation screen, once I stopped the server there was a new SAV file in the Players folder. My question is, how can I make it so that my friends retain their previous GUIDs upon joining the server so that they can keep their previous progress? I am aware that another person had this same issue and it was solved by removing the -nosteam launch option on their dedicated server, however because we are using a third party hosting site, I don't have access to remove the launch option (if it exists). Does anybody who has encountered this problem have a solution?
I have heard of others who apparently ran the python script on their friends' SAV files from the co-op server and converted them to their new GUIDs and then uploaded the converted SAV files to the Players folder in the dedicated server, however so far I have not had any luck with this (my friend joined but had an infinite loading screen). Did this method work for anyone who has tested it?

Thank you for the detailed issue. I'm working on it at the moment, but this is a duplicate of https://github.com/xNul/palworld-host-save-fix/issues/26 so I'm going to close this one. If you would like to stay up to date on the progress of this bug, please subscribe to that issue.


(NeoVim) ALEFix append ^M^M to each line in a dos format file

Information
NVIM v0.3.1 (This doesn't happen in regular Vim)
Build type: Release
Python version 3.7
Operating System: Manjaro Linux
:ALEInfo
aleinfo.log
What went wrong
ALEFix append 2 ^Ms to each line.
Reproducing the bug
The fileformat is dos. I don't want to change the fileformat because my colleagues are using Windows.
To reproduce this, you can create a file with some code in it and convert it to dos type with unix2dos.
Then ALEFix it.

What kind of file were you trying to fix, and what did you run on the file? I tried using the trim_whitespace fixer in a dos-formatted file, and I didn't see the same issue.

I repeated it on a Python file with these code.
#!/usr/bin/env python
# -*- coding: utf-8 -*-

import os

print(1)

After unix2dos and ALEFix, I got this:

let g:ale_fixers = {
      \  '*': ['trim_whitespace'],
      \  'c': ['clang-format'],
      \  'javascript': ['prettier', 'importjs'],
      \  'sh': ['shfmt'],
      \  'python': [
      \    'autopep8',
      \    'isort',
      \  ],
      \ }

my vimrc https://github.com/Karmenzind/dotfiles-and-scripts/blob/master/home_k/.vimrc#L476

@Karmenzind Okay, give that a try. I changed two things.

Now calls to writefile() convert multiple carriage returns on the ends of lines to just one carriage return.
Carriage returns are removed before calling setline(), which adds them back in by itself.

Fixing files which are formatted for dos will probably have the side-effect of also converting the file to dos format, but that shouldn't be a huge issue.

It has been fixed
Thank you


Crash when opening godot 3.4

Godot version
3.4 stable
System information
Ubuntu 20.04.3 AMD Ryzen 5 2500U
Issue description
When I open 3.4 for the first time, I get a black window for a second, and then this error:

Godot Engine v3.4.stable.official.206ba70f4 - https://godotengine.org
OpenGL ES 3.0 Renderer: AMD Radeon(TM) Vega 8 Graphics (RAVEN, DRM 3.36.0, 5.6.11-050611-generic, LLVM 12.0.0)
OpenGL ES Batching: ON

================================================================
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v3.4.stable.official (206ba70f4b2a3eac4455bb21bd7c8af76285bd86)
Dumping the backtrace. Please include this when reporting the bug on https://github.com/godotengine/godot/issues
[1] /lib/x86_64-linux-gnu/libc.so.6(+0x46210) [0x7f3851448210] (??:0)
[2] godot() [0x131237f] (??:0)
[3] /usr/lib/x86_64-linux-gnu/libLLVM-12.so.1(+0x34ad1fa) [0x7f384bf301fa] (??:0)
-- END OF BACKTRACE --
================================================================
Aborted (core dumped)

I was able to run Godot 3.3.4
Steps to reproduce
Download Godot 3.4 editor and run it from the command line
Minimal reproduction project
N/A

@jul2040 Can you try to get a full backtrace using a debug build? This will tell you where the issue comes from.

Do you happen to be using windows 11? I'm getting a crash in 3.4 on windows 11 and when I've downgraded back to 3.3.4 that version is also crashing on loading any project. If I get some time I'll try and run a debug build, but if you are on 11 that some recent update could be the culprit.

@rob-mur You're probably running into a different issue: #54076. This is not a crash, just the Windows Terminal killing the application when it shouldn't.
OP's issue is an actual crash on Linux though.

Ah, you are quite correct - apologies!

@Calinou
I tried running a debug build, and it worked fine, then I re-downloaded a build from the godot website, and it worked fine.
I think the original file got corrupted when I downloaded it.


vNext insider BC: "The property 'Value' cannot be found on this object. Verify that the property exists and can be set." during container creation

Describe the issue
When creating container for current insider vNext versionm the creation ends with this error>
The property 'Value' cannot be found on this object. Verify that the property exists and can be set.
at <ScriptBlock>, C:\Run\SetupConfiguration.ps1: line 106
at <ScriptBlock>, C:\Run\navstart.ps1: line 155
at <ScriptBlock>, C:\Run\start.ps1: line 384
at <ScriptBlock>, <No file>: line 1Error

Scripts used to create container and cause the issue
$containerName = 'test'
Windows credentials for the host computer.'
$auth = 'AAD'
$sasToken = 'xxxx'
$artifactUrl = Get-BcArtifactUrl -storageAccount 'bcinsider' -type 'Sandbox' -country 'us' -select 'Latest' -sasToken $sasToken
New-BcContainer `
    -accept_eula `
    -containerName $containerName `
    -auth $auth `
    -artifactUrl $artifactUrl `
    -authenticationEMail<EMAIL_ADDRESS>`
    -updateHosts

Full output of scripts
BcContainerHelper is version 4.0.8
BcContainerHelper is running as administrator
Hyper-V is Enabled
UsePsSession is True
Host is Microsoft Windows Server 2019 Standard - ltsc2019
Docker Client Version is 20.10.22
Docker Server Version is 20.10.22
Removing Desktop shortcuts
Downloading application artifact /sandbox/22.0.52293.0/us
https://bcinsider.azureedge.net/sandbox/22.0.52293.0/us?xxxxxxx
Downloading C:\Users\xxxxx\AppData\Local\Temp\c2c252a9-0df4-4c7b-a609-8ff06e89086d.zip
Using WebClient
Unpacking application artifact to tmp folder using 7zip
Fetching all docker images
Fetching all docker volumes
Using image mcr.microsoft.com/businesscentral:10.0.17763.3770
Creating Container test
Style: sandbox
Multitenant: Yes
Version: 22.0.52293.0
Platform: 22.0.52192.0
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.17763.3770 (ltsc2019)
Host OS Version: 10.0.17763.3770 (ltsc2019)
Using process isolation
Using locale en-US
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Files in C:\ProgramData\NavContainerHelper\Extensions\test\my:
- AdditionalOutput.ps1
- MainLoop.ps1
- SetupVariables.ps1
- updatehosts.ps1
Creating container test from image mcr.microsoft.com/businesscentral:10.0.17763.3770
aaf99bb4f553f198f5c9dde4d2df86346ed8e778ca1f2e9218efaa5eb5624e7a
Waiting for container test to be ready
Using artifactUrl https://bcinsider.azureedge.net/sandbox/22.0.52293.0/us
Using installer from C:\Run\210-new
Installing Business Central
Installing from artifacts
Starting Local SQL Server
Starting Internet Information Server
Copying Service Tier Files
c:\dl\sandbox\22.0.52293.0\platform\ServiceTier\Program Files
c:\dl\sandbox\22.0.52293.0\platform\ServiceTier\System64Folder
Copying PowerShell Scripts
c:\dl\sandbox\22.0.52293.0\platform\WindowsPowerShellScripts\Cloud\NAVAdministration
c:\dl\sandbox\22.0.52293.0\platform\WindowsPowerShellScripts\WebSearch
Copying Web Client Files
c:\dl\sandbox\22.0.52293.0\platform\WebClient\Microsoft Dynamics NAV
Copying ModernDev Files
c:\dl\sandbox\22.0.52293.0\platform
c:\dl\sandbox\22.0.52293.0\platform\ModernDev\program files\Microsoft Dynamics NAV
Copying additional files
Copying ConfigurationPackages
C:\dl\sandbox\22.0.52293.0\us\ConfigurationPackages
Copying Test Assemblies
C:\dl\sandbox\22.0.52293.0\platform\Test Assemblies
Copying Extensions
C:\dl\sandbox\22.0.52293.0\us\Extensions
Copying Applications
C:\dl\sandbox\22.0.52293.0\platform\Applications
Copying Applications.US
C:\dl\sandbox\22.0.52293.0\us\Applications.US
Copying dependencies
Copying ReportBuilder
Importing PowerShell Modules
Restoring CRONUS Demo Database
Setting CompatibilityLevel for tenant on localhost\SQLEXPRESS
Exporting Application to CRONUS
Removing Application from tenant
Modifying Business Central Service Tier Config File for Docker
Creating Business Central Service Tier
Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
Starting Business Central Service Tier
Importing license file
Copying Database on localhost\SQLEXPRESS from tenant to default
Taking database tenant offline
Copying database files
Attaching files as new Database default
Putting database tenant back online
Mounting tenant database
Mounting Database for default on server localhost\SQLEXPRESS with AllowAppDatabaseWrite = False
Sync'ing Tenant
Tenant is Operational
Stopping Business Central Service Tier
Installation took 431 seconds
Installation complete
Initializing...
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is test
PublicDnsName is test
Using AccessControlService Authentication
Creating Self Signed Certificate
Self Signed Certificate Thumbprint ACB630C07A810C6CD5B377B631E0BA8D24BA9784
DNS identity test
Modifying Service Tier Config File with Instance Specific Settings
The property 'Value' cannot be found on this object. Verify that the property exists and can be set.
at <ScriptBlock>, C:\Run\SetupConfiguration.ps1: line 106
at <ScriptBlock>, C:\Run\navstart.ps1: line 155
at <ScriptBlock>, C:\Run\start.ps1: line 384
at <ScriptBlock>, <No file>: line 1Error
Using artifactUrl https://bcinsider.azureedge.net/sandbox/22.0.52293.0/us
Using installer from C:\Run\210-new
Installing Business Central
Installing from artifacts
Starting Local SQL Server
Starting Internet Information Server
Copying Service Tier Files
c:\dl\sandbox\22.0.52293.0\platform\ServiceTier\Program Files
c:\dl\sandbox\22.0.52293.0\platform\ServiceTier\System64Folder
Copying PowerShell Scripts
c:\dl\sandbox\22.0.52293.0\platform\WindowsPowerShellScripts\Cloud\NAVAdministration
c:\dl\sandbox\22.0.52293.0\platform\WindowsPowerShellScripts\WebSearch
Copying Web Client Files
c:\dl\sandbox\22.0.52293.0\platform\WebClient\Microsoft Dynamics NAV
Copying ModernDev Files
c:\dl\sandbox\22.0.52293.0\platform
c:\dl\sandbox\22.0.52293.0\platform\ModernDev\program files\Microsoft Dynamics NAV
Copying additional files
Copying ConfigurationPackages
C:\dl\sandbox\22.0.52293.0\us\ConfigurationPackages
Copying Test Assemblies
C:\dl\sandbox\22.0.52293.0\platform\Test Assemblies
Copying Extensions
C:\dl\sandbox\22.0.52293.0\us\Extensions
Copying Applications
C:\dl\sandbox\22.0.52293.0\platform\Applications
Copying Applications.US
C:\dl\sandbox\22.0.52293.0\us\Applications.US
Copying dependencies
Copying ReportBuilder
Importing PowerShell Modules
Restoring CRONUS Demo Database
Setting CompatibilityLevel for tenant on localhost\SQLEXPRESS
Exporting Application to CRONUS
Removing Application from tenant
Modifying Business Central Service Tier Config File for Docker
Creating Business Central Service Tier
Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
Starting Business Central Service Tier
Importing license file
Copying Database on localhost\SQLEXPRESS from tenant to default
Taking database tenant offline
Copying database files
Attaching files as new Database default
Putting database tenant back online
Mounting tenant database
Mounting Database for default on server localhost\SQLEXPRESS with AllowAppDatabaseWrite = False
Sync'ing Tenant
Tenant is Operational
Stopping Business Central Service Tier
Installation took 431 seconds
Installation complete
Initializing...
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is test
PublicDnsName is test
Using AccessControlService Authentication
Creating Self Signed Certificate
Self Signed Certificate Thumbprint ACB630C07A810C6CD5B377B631E0BA8D24BA9784
DNS identity test
Modifying Service Tier Config File with Instance Specific Settings
The property 'Value' cannot be found on this object. Verify that the property exists and can be set.
at <ScriptBlock>, C:\Run\SetupConfiguration.ps1: line 106
at <ScriptBlock>, C:\Run\navstart.ps1: line 155
at <ScriptBlock>, C:\Run\start.ps1: line 384
at <ScriptBlock>, <No file>: line 1
New-BcContainer Telemetry Correlation Id: ee906cc2-33e4-4cbf-95bb-8b0e5649dda1
Initialization of container test failed
At C:\Users\adm_ksacek\Documents\WindowsPowerShell\Modules\BcContainerHelper\4.0.8\ContainerHandling\Wait-NavContainerReady.ps1:42 char:17
+ ...             throw "Initialization of container $containerName failed" ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (Initialization of container test failed:String) [], RuntimeException
    + FullyQualifiedErrorId : Initialization of container test failed

Additional context
The problem is that the script is trying to set config value ClientServicesFederationMetadataLocation which is not existing in the vNext version anymore.

I guess this is only when you use AAD auth. right?

You can add
-myscripts @('https://raw.githubusercontent.com/microsoft/nav-docker/master/generic/Run/210-new/SetupConfiguration.ps1')

Until a new generic image is shipped

@kine Have you been able to fix the issue with the latest bcinsider? After adding the SetupConfiguration.ps1 script, the container is created but the client is not able to open a session.
I get this error in the browser:
You cannot sign in due to a technical issue. Contact your system administrator.

That error is typically because you have a multitenant container and forgot to specify tenant?

Maybe download the eventlog and see what the cause is?

@freddydk We have the same Problem with AAD Container. Eventlog doesn't say anything.

@kine Have you been able to fix the issue with the latest bcinsider? After adding the SetupConfiguration.ps1 script, the container is created but the client is not able to open a session. I get this error in the browser:
You cannot sign in due to a technical issue. Contact your system administrator.

If you are creating the container with AAD auth, you need to create the AAD app and set appropriate settings inside the container like:
Set-NAVWebServerInstanceConfiguration -WebServerInstance BC -KeyName AadAuthorityUri -KeyValue 'https://login.microsoftonline.com/<tenantid>'
Set-NAVWebServerInstanceConfiguration -WebServerInstance BC -KeyName AadApplicationId -KeyValue '<appid>'

https://learn.microsoft.com/en-us/dynamics365/business-central/dev-itpro/administration/authenticating-users-with-azure-ad-overview

I think they are available on New-BcContainer:
New-BcContainer @params `
    -containerName $containerName `
    -accept_eula `
    -artifact $artifactUrl `
    -auth AAD `
    -Credential $credential `
    -licenseFile $licenseFile `
    -updatehosts `
    -AuthenticationEMail $AadCredential.UserName `
    -AadTenant $aadTenant `
    -AadAppId $AdProperties.SsoAdAppId `
    -AadAppIdUri $appIdUri `
    -dns 'hostDNS' `
    -myScripts @('https://raw.githubusercontent.com/microsoft/nav-docker/master/generic/Run/210-new/SetupConfiguration.ps1') `
    -additionalParameters @("--env customNavSettings=ExcelAddInAzureActiveDirectoryClientId=$($AdProperties.ExcelAdAppId)")

I gonna log another issue, it does not seem directly linked with this one. Sorry for messing things :o

The above script (from a few days ago) now works for BC 22 as well.
See #2876 for more information.
A new generic image version will be out in a few days with this fix (meaning you don't need the -myscripts anymore)

Next containerHelper version will auto-patch SetupConfiguration if you are using AAD and generic <IP_ADDRESS> or earlier with 22.x and above
Real fix is with generic <IP_ADDRESS>

Shipped in 4.0.14 and Generic image <IP_ADDRESS>


Bounded loops (WIP)

Adds a new search strategy called bfs-bounded that truncates paths if they have been targeted by a JUMPI more then JUMPDEST_LIMIT times (currently set to 4).
Still needs testing and possibly adjustments.
Solve #1048 for the most part (it's still quite slow on byte array copy because memory operations are slow, but with the number of iteration reduced the time Mythril spends on this should become reasonable).

This can actually be placed in a plugin right?

I guess? Joran's suggestion was to put it into a strategy.

Maintaining a global counter for all the states may not be a good idea right? Like if there are 3 states in worklist, all hit the loop, then self._jumpcount[tx][address] ++ happens globally right? are the transaction stacks deepcopied each time?

Yes, the strategy counts jumpdests globally (per transaction) for the whole symbolic execution run, _jumpcount is a member of the BFSBoundedLoopsStrategy class instance, how is that related to transaction stacks? Sorry if I'm not following correctly :)

What if there are 10 states in worklist which enter the same JUMPI condition(which is an if condition rather than a loop), In that case there might be a problem right?

In what cases would that happen? The only case I can think of is complex if-statements if they compile in certain ways?
if (a == 0 | a == 1 | a == 2 | a == 3 | a == 4) {

} else {

}

Would be interesting to have some concrete examples where it causes issues.

...
if()
{}
else {}
// there will be 2 states here

for(i=0; i<num; i++){}
// there will be 2*JUMPI_COUNT states here

if(condition) //some states above might get pruned here.
{}
 ...

It might be uncommon but still might be possible :cry: .(uncommon but likely to be encountered given that we analyze lot's of contracts per day)

Ok, I confirmed that this is exactly the case. With this strategy we can detect this:
pragma solidity ^0.5.0;

contract Lol {

	function lol(uint256 a) public {

		if ((a == 0) || (a == 1) || (a == 2) || (a == 3)) {
		   if (a == 3) {
		      selfdestruct(msg.sender);
		  	}
		 }
	}
}

But not this:
pragma solidity ^0.5.0;

contract Lol {

	function lol(uint256 a) public {

		if ((a == 0) || (a == 1) || (a == 2) || (a == 3) || (a == 4)) {
		   if (a == 4) {
		      selfdestruct(msg.sender);
		  	}
		 }
	}
}

Although, in this case, you could say that it is very similar to a loop iterating over a. I guess with this strategy we'll miss cases whenever the code searches for a specific value in a variable or some complex combination of conditions.
The most straightforward way to deal with this IMO is to select a proper bound (perhaps a bit higher than 4?) and accept that we can miss some paths of there's an unbounded amount of values, in which case fuzzing would be more appropriate (and this is something we could hand over to Harvey)?

I'd add a state annotation which tracks which instructions have been executed over a given path.
Additionally, I'd say this should be an extending strategy (similar to how the coverage strategy is implemented).
That way we don't have to implement the same code all the time

@norhh I think a search strategy is perfect for this use case
I'd use a plugin if we extend the behaviour of laser in some way ( for example with the coverage ).
A search strategy interface is limited, making it easier to implement and more simple ( I hope )

something like this might fail

contract Lame {

    uint bata;

    function addCreditors(uint a) public returns (bool) {
        if(a == 0)
        {
            bata += 1;
        }
        else {
            bata -= 1;
        }
        // there will be 2 states here
        uint i;
        for(i=0; i<2; i++){
            a+=1;
            bata += a;
        }

        if(a == 3) // some states might get pruned here
        {
            selfdestruct(msg.sender);
        }
    }

}

I'd add a state annotation which tracks which instructions have been executed over a given path
That should do the work.

@JoranHonig .I was saying something like the coverage strategy plugin ,We can either place it as a plugin or an extensible class, as long as we can have a cool flag for it.

something like this might fail
contract Lame {

Yeah, basically another variant where we'd get state explosion because a`bata` are incremented in a loop and each value of a could potentially trigger a different path. My intuition is that Harvey could deal with this better? @wuestholz

symbolic execution shouldn't have a problem with this contract

@b-mueller Yeah, Harvey deals with path/state explosion a bit more gracefully. The main reason is that it explores the different paths and states less systematically. This avoids getting stuck in one part of the state space by exploring only its surrounding states.

I'd add a state annotation which tracks which instructions have been executed over a given path

That sounds reasonable. Not sure if refactoring this to a plugin is better, I'll finish up the class fist :)

The difference between a plugin and a search strategy is that a plugin has an initialization phase where it is allowed to introduce hooks in the laser vm.
A search strategy doesn't introduce hooks, but is initialized with the work list & called each time the executor wants to know a state to work on.
I don't think a plugin is necessary for this case. The coverage based search strategy is based in a plugin because it is dependent on that plugin. I'm not sure whether this is the best way to go about indicating the dependency, but this is what I decided to add at that point.

I rewrote this to use a StateAnnotation. Not sure how extending strategies are meant to be used / invoked?

@JoranHonig I don't see CoverageStrategy used anywhere in the code. How are extended strategies supposed to be selected? Do we still need to add CLI support for those?

I refactored this into an extension similar to the coverage strategy and added a method to laser for loading it "on top of" its existing strategy. Also added a --loop-bound CLI argument with a default value of 4 which is passed through to SymExecWrapper.
Should be ready for review now...


FFMPEG hardware acceleration is not working

FFMPEG hardware acceleration is not working

Make sure, video graphics card drivers are fully up to date. head into ersatzTV settings on the ffmpeg file and edit the profile and make sure that the hardware acceleration is enabled.
![ScreenHunter_431 Oct  05 15 54](https://github.com/user-attachments/assets/71adaab6-ee63-491f-a0ab-56e0c5f19

1a1)

FFMPEG hardware acceleration is not working

It's impossible to help without any details. Please paste your full ETV version (found below the left menu), list which video card you are trying to enable acceleration on, and list the symptoms of hardware acceleration "not working."


bluetooth no longer asks to access aplay when paired

I am building master. This is on Arch Linux aarch64 wiith bluez 5.55-1. I am using both bluealsa service files from the wiki.
Previously (with v2.1.0), I would get 2 questions at pair time if I wanted to use the bluealsa aplay device. I cannot remember what they were but thet were long numbers and in order for me to play from my phone to the RPi, I had to answer yes to both other wise it would not work.
Now with the 'master' git based package build, I ensure that the bluealsa-aplay is up and running:
● bluealsa-aplay.service - Bluealsa audio player
     Loaded: loaded (/etc/systemd/system/bluealsa-aplay.service; enabled; vendor preset: disabled)
     Active: active (running) since Sat 2020-09-12 12:51:04 CDT; 15min ago
       Docs: https://github.com/Arkq/bluez-alsa/
   Main PID: 4084 (bluealsa-aplay)
      Tasks: 1 (limit: 1067)
     CGroup: /system.slice/bluealsa-aplay.service
             └─4084 /usr/bin/bluealsa-aplay -Dhw:0,1 00:00:00:00:00:00

Sep 12 12:51:04 rune64 systemd[1]: Started Bluealsa audio player.

My /etc/default/bluealsa-aplay is:
OPTIONS="-Dhw:0,1"
But now when I connect, All I can do it share my contact list from my phone. No longer any audio devices show up.
[bluetooth]# show
Controller 43:45:C0:00:1F:AC (public)
        Name: rune64
        Alias: rune64
        Class: 0x00480000
        Powered: yes
        Discoverable: no
        DiscoverableTimeout: 0x000000b4
        Pairable: yes
        UUID: A/V Remote Control        (0000110e-0000-1000-8000-00805f9b34fb)
        UUID: Audio Source              (0000110a-0000-1000-8000-00805f9b34fb)
        UUID: PnP Information           (00001200-0000-1000-8000-00805f9b34fb)
        UUID: Headset AG                (00001112-0000-1000-8000-00805f9b34fb)
        UUID: Handsfree Audio Gateway   (0000111f-0000-1000-8000-00805f9b34fb)
        UUID: A/V Remote Control Target (0000110c-0000-1000-8000-00805f9b34fb)
        UUID: Generic Access Profile    (00001800-0000-1000-8000-00805f9b34fb)
        UUID: Generic Attribute Profile (00001801-0000-1000-8000-00805f9b34fb)
        UUID: Device Information        (0000180a-0000-1000-8000-00805f9b34fb)
        Modalias: usb:v1D6Bp0246d0537
        Discovering: no
        Roles: central
        Roles: peripheral
Advertising Features:
        ActiveInstances: 0x00 (0)
        SupportedInstances: 0x05 (5)
        SupportedIncludes: tx-power
        SupportedIncludes: appearance
        SupportedIncludes: local-name

isn't headset AG the device? Why does it no longer ask for me to allow a connection?
this is with bluez 5.55-1 on Arch I no longer have bluez-libs 5.54 to back date it.

Isn't headset AG the device

No
If you want to play A2DP streams from your phone through bluealsa-aplay, you need to start bluealsa with -p a2dp-sink
then bluetoothctl will show
UUID: Audio Sink                (0000110b-0000-1000-8000-00805f9b34fb)

If you want HFP voice connection from the phone, you need to start bluealsa with -p hfp-hf
then bluetoothctl will show
UUID: Handsfree                 (0000111e-0000-1000-8000-00805f9b34fb)

You can , of course, start bluealsa with both options (and others)

oops, sorry for the confusion on my part.
I'm back in business. Will upgrade back to a release version when available.

this was a user error and has been resolved.


Use Selected Perimeter Modal Doesn't Dismiss

When you click the button "Use Selected Perimeter", there is a modal that pops up. When you click "Ok", the modal doesn't dismiss and there is no additional feedback when clicking Ok.
I'd suggest just dismissing the modal at that point, or if there is no interaction needed, you could use a toast instead. I think this could be used across the app where at the moment you don't get immediate feedback that clicking some of the buttons did something.
Another option, which might just be a misunderstanding of how to use the app, is typically when you click "Use Selected Perimeter", are you not wanting to then go to the main page of the app and actually use it? Should we take the person to that page at that time, or at least give them a button that takes them there?

Yes it could take some time (depending on your machine) until selected perimeter is loaded. Some stuff is going on in background. I'll check if I can add a wait spinner or something else to give a feedback to user

I've yet to see it ever dismiss, no matter how much time I give it. I'm running this in docker, so I've watched in the logs to see what is going on, and it just never seems to go away.

Can you share Cassandra log when you clicked load selected perimeter

I just updated to the latest changes in master and now it is going away for me. Still think that anytime you have an action that triggers something to happen on the server that might take some time that we should provide some feedback to the user that their action was received and is in progress (spinner, toast, etc), then trigger an error in the case it wasn't successful.
Going to close this ticket out since it is at least going away.


[Feature Request] Additional commands

-list, /list - To get list of currently pinned apps
-fix, /fix - To fix broken pins (remove pins in taskband REG that point to missing .lnk files)

It's a good idea, I'll try to find the time to look into it.


[MB-4524]  Job Runner: Approved Payment Request Fetcher

Description
This PR is the first code addition to creating a job runner to handle payment request that have been approved by the TIO and need to be sent along to Syncada via GEX. This PR specifically is focused on creating a method on the job runner that queries for all payment request that have a status of reviewed
Setup/Review Notes
The job runner isn't fully complete at this point. For this PR please review the query and unit test to ensure they are correct
Code Review Verification Steps

[ ] If the change is risky, it has been tested in experimental before merging.
[ ] Code follows the guidelines for Logging
[ ] The requirements listed in
Querying the Database Safely
have been satisfied.
Any new migrations/schema changes:

[ ] Follow our guidelines for zero-downtime deploys (see Zero-Downtime Deploys)
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] User facing changes have been reviewed by design.
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change
Job Runner Proposal explains more about the approach used.

Messages

:book:
:link: MB-4524

  Generated by :no_entry_sign: dangerJS against 1c3de7d49efcd5354ad7bd3707e43892fd3f8d0d


Params for loading pretrained basenet or checkpoints.

What is the difference between the three parameters?
--base-net
--pretrained-ssd
--resume

--base-net is the mobilenet backbone (i.e. no SSD in it yet)
--pretrained-ssd is a pretrained SSD model that was previously trained with train_ssd.py.   --resume is essentially the same minus some hyperparameter settings.  In most cases --pretrained-ssd seems to work better.

Thank you dusty.

Good afternoon, I am training a custom object detection model using VGG16-SSD, instead of Mobilenet SSD for better accuracy. I have trained the model, but i could not load it. Can you please help me out to load my model and run detection.


Standalone build

Plugin Version or Commit ID
adb2d90
Unity Version
2021.3.1f1
Your Host OS
Ubuntu 18.0.4
Target Platform
Linux Standalone
[Windows Only] Visual Studio C++ and Windows SDK Version
No response
[Linux Only] GCC/G++ and GLIBC Version
No response
[Android Only] Android Build Tools and NDK Version
No response
[iOS Only] XCode Version
No response
Command Sequences
python build.py build --desktop gpu
Log

Additional Context
I have successfully compiled it on my Ubuntu Machine. When I try the same code on another machine, it throws the following error,
DllNotFoundException: mediapipe_c assembly:<unknown assembly> type:<unknown type> member:(null)
Mediapipe.UnsafeNativeMethods..cctor () (at Packages/com.github.homuler.mediapipe/Runtime/Scripts/PInvoke/UnsafeNativeMethods.cs:29)

What am I missing?

See https://github.com/homuler/MediaPipeUnityPlugin/wiki/Installation-Guide#dllnotfoundexception.
You may want to link OpenCV statically (--opencv cmake).

@homuler thank you . I tried it but it's still throwing the same error! :( Am I missing something? I tried python3.9 build.py build --desktop gpu --opencv cmake -v

What is your host machine's OS (maybe Ubuntu 18.04?) and what is your target machine's OS?
In general, libmediapipe_c.so won't work on a machine if the GLIBC version is less than that of the machine used to build it.

@homuler Yes, the other machine is Ubuntu 18.0.4 while the mediapipe is built on Ubuntu 20.0.4. In Ubuntu 18.0.4, it has got glibc v2.27 while Ubuntu 20.0.4 has got glibc v2.31. any possible fixes?

Please download the built package from the release page or use Docker Ubuntu container to build it.
As mentioned in the document, you should always prefer Docker.

:warning: If your GNU libc version in the target machine is compatible with it in the container, always prefer Docker.


Fix IAM V2 bug for new A1 data collector token entries

:nut_and_bolt: Description: What code changed, and why?
Since this commit https://github.com/chef/automate/commit/a2e1a87bc749cd89d5011c82b5379d6fdf8fdc7f we have been enforcing project permissions validation on token entries. When the authn server starts up, if it's configured with a legacy A1 data collector token, it will first check if it exists, then insert a token with the config specified value if it doesn't.
However, on IAM V2, it performs validation that the subject from context has permissions on the projects specified. In this case, we do not wish to validate projects because 1) we are just inserting the token in the default project (will always exist) and 2) this is a domain specific operation being initiated by the server which shouldn't need to round-trip to authz to ask for permission.
:athletic_shoe: How to Build and Test the Change
hab studio enter
build components/authn-service
start_all_services
echo '[auth_n]
  [auth_n.v1]
    [auth_n.v1.sys]
      [auth_n.v1.sys.service]
        a1_data_collector_token = "LEGACY_TOKEN_HERE"' >> a1.toml
chef-automate config patch a1.toml
sl | grep authn-service
# verify that token exists

:white_check_mark: Checklist

[x] I have read the CONTRIBUTING document.
[ ] Tests added/updated?
[ ] Docs added/updated?
[x] All commits have been signed-off for the Developer Certification of Origin.

Fixed #2955


feat(atom-with-hash): allow optional setHash

Problem
Changing the hash using window.location can be in conflict with the next router. To reproduce this issue within a next app:

Use atomWithHash somewhere in the app using { replaceState: true }
Navigate to the page with atomWithHash (using next router)
Trigger a change that will invoke setting the hash via atomWithHash
Navigate in browser going back in history
Try and navigate in browser going forward
Will behave unexpectedly changing the URL, but not changing the page

This might also be unexpected behavior from the perspective of the next router. I'm not entirely sure. But I can imagine other routers possibly also having problems with this.
Solution
Changing the setHash function fixed this issue so my proposed solution using this commit's changes are:
import Router from 'next/router'
import { atomWithHash } from 'jotai-location'

atomWithHash('key', 'default', {
  subscribe: (callback) => {
    Router.events.on('hashChangeComplete', callback)
    return () => {
      Router.events.off('hashChangeComplete', callback)
    }
  },
  setHash: (searchParams) => {
    Router.replace({ hash: searchParams.toString() })
  },
})

What I don't like about these changes
I'm not entirely happy with this API as using replaceState with setHash makes replaceState unused. But for now I wanted to make minimal changes.
what do you think?

Hi, thanks so much for making suggestions for this library!
I wasn't aware the issue with nextjs. Wonder if there are some other nextjs users out there.
Thought replaceState option was suggested by one of nextjs users.
Anyway, if it doesn't work, we are happy to add an option to support such cases.

But for now I wanted to make minimal changes.

I think we can be creative to have new ideas.
Maybe:
setHash?: 'default' | 'replaceState' | ((params: URLSearchParams) => void)

params can be string.
We still could have replaceState option with a deprecation message for a while before completely removing it.
Would be nice to know how you and others think.
Let me pin some users how has contributed to atomWithHash before in https://github.com/pmndrs/jotai/pulls?q=atomWithHash.
@notsidney @austinwoon @jakkku

I think I've figured out what might be going wrong with the Next router when setting the hash in the default setHash.
When replaceState = false
Here the hash is set with window.location.hash = hash which will trigger the hashchange event. I would assume that the next router is listening to that and in this case it works as expected.
When replaceState = true
Here the hash is set with window.history.replaceState. This unfortunately does not cause a hashchange event as described here (this is the docs for pushState but the behavior is the same as this Stackoverflow answer says).
I would assume it has something to do with this behavior. Not sure.

Sounds good.
@V-Mann-Nick Would you like to work on it?

Sure will do. I will make the edits to this PR by end of week.

@dai-shi I've made the changes as proposed.

Good point @dai-shi. I committed your suggestion.

Published: https://www.npmjs.com/package/jotai-location/v/0.3.0
Thanks for your contribution! Please feel free to work on any other improvements in coding, making examples, and documenting, if you are interested.


Fix/404

Related Ticket(s)
#443
Description
This fixes the 404 page with the updated links, and also the background color issue.
Changelog
Changed

404 page

URL to preview: https://ibmdotcom-library.s3.us-south.cloud-object-storage.appdomain.cloud/deploy-previews/584/404/index.html

@JennySanchez @jacobottesen-dgc the 404 redirect is live on EI server. Can you approve and merge this change when you get a chance?


Yosegi-81 The value is in a row that is null.

What is the necessity of this update? What is updated?
Data is corrupted due to an encoding bug.
The code was fixed because the last index of Not null has not been updated.
How did you do the test? (Not required for updating documents)
I have created a unit test that reproduces this bug and confirmed that it passes.

LGTM


Build(deps-dev): Bump @testing-library/dom from 8.18.1 to 8.19.0 (#4174)

Pull request opened by the merge tool on behalf of #4174

Merged in dc84e63bb5f047a1379b830d2df831f9cb19c1f1


get_budgets returns only the most recent month, not the history for the past year

The json data returned from mint.com contains the full history of budget data, but the output from mintapi only shows the current month.
I'm not familiar with Python, but I suspect it is related to one of these sections of setting up and filling out the return data. I feel like there is a loop missing...
    # Make the skeleton return structure
    budgets = {
        'income': response['data']['income'][
            str(max(map(int, response['data']['income'].keys())))
        ]['bu'],
        'spend': response['data']['spending'][
            str(max(map(int, response['data']['income'].keys())))
        ]['bu']
    }

    # Fill in the return structure
    for direction in budgets.keys():
        for budget in budgets[direction]:
            budget['cat'] = self.get_category_from_id(
                budget['cat'],
                categories
            )

Is the MintAPI working for you right now?  I cannot get it to get transactions.  See my bug report.  Thx.

I didn't try get transactions but the default call, (accounts,) and budgets
worked for me.
On Sunday, 17 January 2016, mrfiat<EMAIL_ADDRESS>wrote:

Is the MintAPI working for you right now? I cannot get it to get
transactions. See my bug report. Thx.
—
Reply to this email directly or view it on GitHub
https://github.com/mrooney/mintapi/issues/66#issuecomment-172331767.

--
Ashley Bennett
www.HiAsh.com

I also wanted a budget history so I can trend my budget actuals in my local monthly budget tool.
I looked into this and worked a solution.  Currently, the 'budgets' dictionary in the api is being created just with the max month number (ie most current). I implemented a 'hist' argument in 'get_budget()' that allows for a select-able range from 1-12 months.
I have it ready to go, just need to get set up to create a Pull Request. Do I have to be approved somehow to do this?


[EUWE] bundle update for Travis runs

Backport of the changes required to get tests running in Travis.  Based on:
https://github.com/ManageIQ/manageiq/pull/16387
https://github.com/ManageIQ/manageiq/pull/16385
https://github.com/ManageIQ/manageiq/pull/16372

I'm having trouble getting the bundler cache working on this branch.  I'll try to get that sorted out in another PR

I've restarted the build several times, but there appear to be many sporadic failures on this branch preventing tests from passing.
@jrafanie @simaishi I think we should merge this change and look into the sporadic failures separately.

Agreed! @jrafanie Are you good with merging?

👍


wpf x86  Exception: Library e_sqlite3 not found

1.2.0 has no errors and can be used. Why?

1.2.0 has no errors and can be used. Why?

some sqlite3 dll missed?

I solved it by doing this:
https://github.com/praeclarum/sqlite-net/issues/1099


Updated the locator for one of test failure

Updated the selector
Issue ticket number and link
Type of change 🧩

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[1 ] Other (please explain in the description section above)

Checklist before requesting a review

[ ] I have performed a self-review of my own code
[ ] I have double checked this work does not include any hardcoded secrets or passwords
[ ] I have made corresponding changes to the documentation
[ ] I have provided details on how I have tested my code
[ ] I have referenced the ticket number above
[ ] I have provided a description of the ticket
[ ] I have included unit tests to cover any testable code changes

Make sure to use git rebase -i main instead of git merge main :)
Here's a guide on how to do it: https://www.atlassian.com/git/tutorials/rewriting-history/git-rebase

Sure, will do


Parsedown does not escape * character inside of backticks

Note: Please see referenced issue.
If given the following input...
*(Test `a*b` test`.)*

... Parsedown will actually parse it into something like...
(Test `ab test.)*

Whereas it should be parsed into...
(Test a*b test`.)

This is in line with how, for example, GitHub renders it: (Test a*b test`.)
Here is a shot from the online Parsedown demo displaying the issue (thanks, @flaviocopes!):

Among other things, noted here already 😉 https://github.com/erusev/parsedown/issues/434 (see: "Issues elsewhere").
@erusev Parsedown will have to change its parsing strategy for inline elements to fix this one (assign priority to inline code – since escapes for most characters don't (and shouldn't) work there). Inline code is a bit of a special case when it comes to priority of inline markers – I think the order of first occurrence based approach is correct for other markers though.
In terms of correctness – I'm not even sure that the commonmark reference parser has been equipped to deal with some cases of element nesting ¯\_(ツ)_/¯

@aidantwoods Thanks for the link!
Looks like that bug has been open for around 3 months now. Is it being actively worked on?


xcframework

Hello do you have xcframework to run in simulator? I use SignalR-Client-Swift in custom swift framework and I put this framework in subproject. Can I run it on simulator ?

No, this repository doesn't have xcframeworks but is a Swift Package. If you're absolutely convinced you need an xcframework you should be able to create it yourself from sources. Examples are meant to be run in simulator.

Assuming the question has been answered.


long or unsigned long not supported

I get an error if I use long or unsigned long argument:
../node_modules/nbind/include/nbind/v8/ValueObj.h: In instantiation of ‘static nbind::WireType nbind::BindingType<ArgType>::toWireType(ArgType) [with ArgType = long int; nbind::WireType = v8::Local<v8::Value>]’:
../node_modules/nbind/include/nbind/v8/Caller.h:76:63:   required from ‘static nbind::WireType nbind::MethodResultConverter<ReturnType>::toWireType(ReturnType&&, Bound&) [with Bound = UiColorButton; ReturnType = long int; nbind::WireType = v8::Local<v8::Value>]’
../node_modules/nbind/include/nbind/v8/Caller.h:94:3:   required from ‘static nbind::WireType nbind::Caller<ReturnType, nbind::TypeList<Args ...> >::callMethod(Bound&, MethodType, NanArgs&) [with Bound = UiColorButton; MethodType = long int (UiColorButton::*)(); NanArgs = const Nan::FunctionCallbackInfo<v8::Value>; ReturnType = long int; Args = {}; nbind::WireType = v8::Local<v8::Value>]’
../node_modules/nbind/include/nbind/signature/MethodSignature.h:34:3:   required from ‘static void nbind::MethodSignature<PtrType, Bound, ReturnType, Args>::callInner(V8Args&, NanArgs&, Bound*) [with V8Args = const Nan::FunctionCallbackInfo<v8::Value>; NanArgs = const Nan::FunctionCallbackInfo<v8::Value>; PtrType = long int (UiColorButton::*)(); Bound = UiColorButton; ReturnType = long int; Args = {}]’
../node_modules/nbind/include/nbind/signature/BaseSignature.h:198:46:   required from ‘static void nbind::TemplatedBaseSignature<Signature, ReturnType, Args>::callInnerSafely(V8Args&, NanArgs&) [with Bound = UiColorButton; V8Args = const Nan::FunctionCallbackInfo<v8::Value>; NanArgs = const Nan::FunctionCallbackInfo<v8::Value>; Signature = nbind::MethodSignature<long int (UiColorButton::*)(), UiColorButton, long int>; ReturnType = long int; Args = {}]’
../node_modules/nbind/include/nbind/signature/MethodSignature.h:38:53:   [ skipping 2 instantiation contexts, use -ftemplate-backtrace-limit=0 to disable ]
../node_modules/nbind/include/nbind/signature/MethodSignature.h:16:7:   required from ‘static Signature& nbind::TemplatedBaseSignature<Signature, ReturnType, Args>::getInstance() [with Signature = nbind::MethodSignature<long int (UiColorButton::*)(), UiColorButton, long int>; ReturnType = long int; Args = {}]’
../node_modules/nbind/include/nbind/BindDefiner.h:77:28:   required from ‘void nbind::BindDefiner<Bound>::addMethod(const char*, MethodType) [with Signature = nbind::MethodSignature<long int (UiColorButton::*)(), UiColorButton, long int>; MethodType = long int (UiColorButton::*)(); Bound = UiColorButton]’
../node_modules/nbind/include/nbind/BindDefiner.h:86:12:   required from ‘void nbind::BindDefiner<Bound>::addMethodMaybeConst(const char*, ReturnType (Bound::*)(Args ...)) [with Signature = nbind::MethodSignature; ReturnType = long int; Args = {}; Bound = UiColorButton]’
../node_modules/nbind/include/nbind/BindDefiner.h:123:22:   required from ‘nbind::BindDefiner<Bound>& nbind::BindDefiner<Bound>::function(const char*, MethodType, Policies ...) [with MethodType = long int (UiColorButton::*)(); Policies = {}; Bound = UiColorButton]’
../src/UiColorButton.cc:45:2:   required from ‘BindInvokerUiColorButton<Bound>::BindInvokerUiColorButton() [with Bound = UiColorButton]’
../src/UiColorButton.cc:41:1:   required from here
../node_modules/nbind/include/nbind/v8/ValueObj.h:59:3: error: request for member ‘toJS’ in ‘arg’, which is of non-class type ‘long int’
   arg.toJS(construct);

It appear nbind does not recognized as a basic type it and try to implement it as object, requiring a toJS member to convert it.
I get this while implmenting UiColorButton on libui-node on this branch

You're probably on a 64-bit platform and JavaScript doesn't have 64-bit integers. Problem is then, what should we convert to? Options are:

Double, potentially losing the lowest 11 bits of precision.
Integer, potentially losing the highest 32 bits.
Custon bignum type holding a lower and higher half (would need implementing).

Which one do you think would work in your case? In issue #8 is listed a plan to support the last one of those options.

Custon bignum type holding a lower and higher half (would need implementing).

I think this could be the better choice, maybe there are some usable library on NPM?
Thinking more of how to implement my use case (Color representation) it could be better to use a custom class to pass by value that hold r,g,b,a values...

Usually rgba values fit in uint32_t which would be a cross-platform way to refer to the type. If each color channel is not an integer between 0-255 then maybe an std::Array<double, 4> (or substitute uint16_t for double, whatever is the most appropriate) would be a good choice of type on the C++ side and efficient to transfer between the languages.

libui function actually receive color as 4 double arguments. The problem is that in the getter it's using always 4 arguments passed by reference. So I thought to compact them in single value.
Later I thought that could be better to use a custom class. std::Array<double, 4> would probably be simpler to implement, but I like the ability to refer to individual color components by name instead that
indexing an array.

In that case please take a look at value types:
https://github.com/charto/nbind#using-objects
Basically you'd have a JavaScript class:
function RGBA(r, g, b, a) {
  this.r = r;
  this.g = g;
  this.b = b;
  this.a = a;
}

Coord.prototype.fromJS = function(output) {
  output(this.r, this.g, this.b, this.a);
}

binding.bind('RGBA', RGBA);

and a C++ class:
NBIND_CLASS(RGBA) {
  construct<double, double, double, double>();

  getset(getR, setR);
  getset(getG, setG);
  getset(getB, setB);
  getset(getA, setA);
}

Then you should be able to pass them around (by value, not reference or pointer) and everything should look nice in both languages.

In that case please take a look at value types:

Already given! Was thinking to use exactly that. But thank you for your example I will copy-paste it 😄

Color implemented! The by-value stuff works perfectly! Thank you for your help!

The custom 64-bit bignum is now mostly implemented in the v0.3 branch but there's an additional problem... int64_t which is long long in my GCC now gets converted but I'm not sure what to do about plain long... It would normally become a JavaScript number on 32-bit platforms and an object on 64-bit platforms. This will surely create portability issues... I could force it to always be an object but that seems slow and impractical when not actually needed, especially when targeting asm.js.

That's good news @jjrv! One option could be to let the user choose, may with a property in config file?

This should now be done, just missing documentation.


Environment variables are not accessible via c.env in Bun

What version of Hono are you using?
3.11.4
What runtime/platform is your app running on?
Bun - 1.0.15
What steps can reproduce the bug?
Save an Environmental Variable in ENV File .env .env.local etc.,
Try to read the value from the Hono using c.env
$ cat .env
FOO=world
BAR=hello$FOO

$ cat src/index.ts
import { Hono } from 'hono'
const app = new Hono()
app.get('/', (c) =>  c.json({ hono: `${c.env.FOO}`, bun: `${process.env.FOO}`  }) )
export default app

$ curl http://localhost:3000
{"hono":"undefined","bun":"world"}

What is the expected behavior?
Environmental Variables should be available in c.env in Hono as they are available in process.env
What do you see instead?
c.env is empty object without any values even though there are many Environmental variables.
Additional information
Attaching Whole c value
{"c":{"env":{},"_var":{},"finalized":false,"req":{"routeIndex":0,"bodyCache":{},"raw":{},"path":"/c"}}}

Hi @insumanth
In Bun, you can use the env() from hono/adapter to get variables:
https://hono.dev/helpers/adapter#env

Thanks for pointing it.
I was trying to convert Existing Cloudflare Worker to Run on Bun and faced this issue.
It was a little unclear in the Hono context env docs docs. I think it is better to mention that this works only in Cloudflare Workers.
If you think the same, I would love create a PR with the changes in honojs/website

@insumanth

If you think the same, I would love create a PR with the changes in honojs/website

Please do!

Created Pull Request with Changes at honojs/website#175

@insumanth
Thanks! I'll close this issue.


[SPARK-45009][SQL][FOLLOW UP] Turn off decorrelation in join conditions for AQE InSubquery test

What changes were proposed in this pull request?

Turn off decorrelation for predicate subquery in join conditions for an insubquery test in AdaptiveQueryExecSuite.
Why are the changes needed?

The test depends on an assumption that the query won't be decorrelated, but doesn't validate that the query plan actually has InSubquery in the plan, so would silently lose test coverage.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Test only change.
Was this patch authored or co-authored using generative AI tooling?

No.

Test only change - @cloud-fan could you review?

thanks, merging to master!


CleanCouchPoller thread getting stuck on FNAL submit6 agent

Impact of the bug
WMAgent
Describe the bug
Jen has reported been reporting heartbeat issues with TaskArchiver (CleanCouchPoller thread). Usually we are not that concern with heartbeat because for most of the time, those alerts go away with time once the thread/component completes its heavy cycle.
However, having a quick look at the TaskArchiver component log, this is the last entry for that thread:
2022-05-23 14:57:01,991:140192324904704:INFO:CleanCouchPoller:Cleaning up wmbs and disk

which means that this thread is actually getting stuck when executing this DAO (Workflow.GetDeletableWorkflows):
https://github.com/dmwm/WMCore/blob/5e654f59a28cb95eb0759272f49ad4c7a33d731c/src/python/WMComponent/TaskArchiver/CleanCouchPoller.py#L380
How to reproduce it
Not sure, submit7 is behaving, so we cannot even say it's common to MariaDB-based agents.
Expected behavior
Other than this poor performance issue to be addressed in a different ticket: https://github.com/dmwm/WMCore/issues/8478
this thread should be able to complete its cycle such that workflows can be properly archived within the agent.
Additional context and error message
None

This agent is not even running anymore, and we didn't see this issue with the other 2 FNAL agents that were being heavily used as well. Instead of dealing with this specific issue, I'd suggest to make it part of https://github.com/dmwm/WMCore/issues/11166
I am closing this out, but we can come back to this if the same problem reoccurs in the FNAL nodes.


No data passed on deeplink clicked. (Resume state or closed state)

Hi,
I have an app in development and before deeplinks were working fine. From past one week it has stopped passing data but app launches.
Data that I get on each launch
{"+clicked_branch_link":false,"+is_first_session":false}
Here is my configuration of system.
ionic info
Ionic:
ionic (Ionic CLI)  : 4.5.0 (/usr/local/lib/node_modules/ionic)
Ionic Framework    : ionic-angular 3.9.2
@ionic/app-scripts : 3.1.9
Cordova:
cordova (Cordova CLI) : 8.1.2<EMAIL_ADDRESS>Cordova Platforms     : android 8.0.0, ios 4.5.5
Cordova Plugins       : cordova-plugin-ionic-keyboard 2.1.3, cordova-plugin-ionic-webview 1.2.1, (and 19 other plugins)
System:
Android SDK Tools : 26.1.1 (/Users/amitrai/Library/Android/sdk)
ios-deploy        : 1.9.4
NodeJS            : v9.11.2 (/usr/local/bin/node)
npm               : 5.6.0
OS                : macOS Mojave
Xcode             : Xcode 9.4.1 Build version 9F2000
Cordova info
Node version: v9.11.2
Cordova version: 8.1.1
Plugins:
adobe-mobile-services,apptentive-cordova,branch-cordova-sdk,call-number,cordova-open-native-settings,cordova-plugin-camera,cordova-plugin-crop,cordova-plugin-device,cordova-plugin-file,cordova-plugin-firebase,cordova-plugin-inappbrowser,cordova-plugin-ionic-keyboard,cordova-plugin-ionic-webview,cordova-plugin-network-information,cordova-plugin-privacyscreen,cordova-plugin-screen-orientation,cordova-plugin-sim,cordova-plugin-splashscreen,cordova-plugin-statusbar,cordova-plugin-telerik-imagepicker,cordova-plugin-whitelist,cordova.plugins.diagnostic,es6-promise-plugin,ionic-plugin-deeplinks
Android platform:

The "android" command is deprecated.
For manual SDK, AVD, and project management, please use Android Studio.
For command-line tools, use tools/bin/sdkmanager and tools/bin/avdmanager

Running /Users/amitrai/Library/Android/sdk/tools/bin/avdmanager list target
Available Android targets:==============] 100% Fetch remote repository...
id: 1 or "android-23"
Name: Android API 23
Type: Platform
API level: 23
Revision: 3
id: 2 or "android-24"
Name: Android API 24
Type: Platform
API level: 24
Revision: 2
id: 3 or "android-25"
Name: Android API 25
Type: Platform
API level: 25
Revision: 3
id: 4 or "android-26"
Name: Android API 26
Type: Platform
API level: 26
Revision: 2
id: 5 or "android-27"
Name: Android API 27
Type: Platform
API level: 27
Revision: 3
id: 6 or "android-28"
Name: Android API 28
Type: Platform
API level: 28
Revision: 6
iOS platform:
Xcode 9.4.1
Build version 9F2000

I think I have the same issue (tested in Android only).
The only semi-useful thing in that data is a link like this one:
branchcordova://open?link_click_id=link-SOMEIDHERE
But all my data is lost. It works fine with Branch links though.

This is really annoying, it's either branch or deeplinks, not cool

Ok in my case it was the facebook4 plugin that was the culprit

@etiennea I am having this same issue on resume. I also have the facebook4 plugin. How did you fix this? Thanks!

How do you guys solve this ?
I'm just receiving {"+clicked_branch_link":false,"+is_first_session":false} in platform.ready and platform.resume
ENV:
ionic 4
"branch-cordova-sdk": "4.1.3"
"@ionic-native/branch-io": "^5.26.0"


[gardening][NSObject] Move operators into types

@swift-ci test and merge

Base branch was modified. Review and try the merge again.

Just a merge error.


Format some characters which are full-width

problem
When I input some content like Traditional Chinese and pretty render the markdown table would be not pretty because of the these full-width characters.
I wrote a test like that:

BTW, I used your plugin of joplin, thanks for your great work and I'm looking forward to this feature :)

PR merged, thanks! :)

PR merged, thanks! :)

Hi @FelisDiligens, I tried this in the web demo page, and saw the issue is still there.

And after updated to the new version in my Joplin, seems also doesn't work, anything else I missed ?

Hi @Stainless5792,

I tried this in the web demo page, and saw the issue is still there.

Sorry, I haven't updated the demo page yet.

And after updated to the new version in my Joplin, seems also doesn't work, anything else I missed ?

I noticed that too... depending on the used font family, full-width characters are between 1 and 2 "regular" characters long. The code just assumes, that a full-width character is exactly 2 regular characters or 2 spaces long. It works great in a terminal (see the green line in the reply2future's picture), but not well in Joplin.
As a workaround, you may want to test a few font families...
Here's "Hack":

And here's "Ubuntu Mono":

Here it aligns properly but with Hack it doesn't... not sure why.

Although Hack is monospace font, but it doesn't work well with Chinese characters. ref
It looks like the problem about joplin HTML display. I have not idea about that.

@FelisDiligens @reply2future So appreciated for your replies ! I will give it a try!


URLSearchParams and URL referenced globally instead of as properties of 'global' parameter

https://github.com/lifaon74/url-polyfill/blob/d74be6fe18d6790ccc4b64ebd4527b17810186c4/url-polyfill.js#L169-L171
Since we are checking if URLSearchParams is present in the global variable, we should also then instantiate URLSearchParams from the global variable. So instead of new URLSearchParams it should be new global.new URLSearchParams, since URLSearchParams is not necessarily the same as global.URLSearchParams.
Fix:
 if (!('URLSearchParams' in global) || (new global.URLSearchParams('?a=1').toString() !== 'a=1')) {
   polyfillURLSearchParams();
 }

This actually seems to be an issue throughout the codebase, where instead of using global.URLSearchParams and global.URL, the values are being accessed directly at global scope as just URLSearchParams and URL
https://github.com/lifaon74/url-polyfill/blob/d74be6fe18d6790ccc4b64ebd4527b17810186c4/url-polyfill.js#L173
https://github.com/lifaon74/url-polyfill/blob/d74be6fe18d6790ccc4b64ebd4527b17810186c4/url-polyfill.js#L252
https://github.com/lifaon74/url-polyfill/blob/d74be6fe18d6790ccc4b64ebd4527b17810186c4/url-polyfill.js#L298

V1.1.5 released, thanks a lot to you.


Rename the *.paml.cs file of a Windows item is not working

If I want to rename a MainWindo.paml item it's ok, so I would also like to rename the MainWindo.paml.cs but I can not.

Indeed, this is a problem. We need someone with some Visual Studio extension skills to fix this I think...

Migrated this issue to the VS extension repository.


Support for direct qBittorrent integration

Would like to request an integration directly with qBittorrent.
Use Case: torrent files uploaded (usually season packs) that are not supported by the starr apps.
These would then be unpacked with unpackerr to allow the files to be picked up via manual import or directory monitor.

You can use the folder watcher feature for that purpose.

It seems to be only looking for net new changes. Not torrents that have already completed.
Thanks,
Kyle Burk
Please excuse any spelling or grammar errors as this was sent from my iPhone.

From: David Newhall @.>
Sent: Thursday, March 23, 2023 12:58:32 AM
To: Unpackerr/unpackerr @.>
Cc: Kyle Burk @.>; Author @.>
Subject: Re: [Unpackerr/unpackerr] Support for direct qBittorrent integration (Issue #276)
You can use the folder watcher feature for that purpose.
—
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FUnpackerr%2Funpackerr%2Fissues%2F276%23issuecomment-1480642349&data=05|01||a0484d2b98184115143508db2b63a275|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638151479141182081|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=%2FXf3a9UOk9ZKRqMGuOeFyTeUygWM1BlTFziUBsLMcuI%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAE645UOUG6J6KRBDGFTIB3TW5PRARANCNFSM6AAAAAAWEXQD4Y&data=05|01||a0484d2b98184115143508db2b63a275|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638151479141182081|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=hikXOiDYGAbvioBMT0%2FUGoBUIrYJ6vCxD0u1%2B6wDfto%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>

It seems to be only looking for net new changes. Not torrents that have already completed.

Correct. If you want to extract things that are already on disk with unpackerr, simply rename them and it will extract them.

Also I dependent on how you're file watching it's likely not compatible with some file sharing tech.

Integrating with qbit wont solve this problem.
Qbit doesn't store any information about extracted files like Starr apps, so it would require unpackerr remembering what it has already extracted. Your request is not trivial, and doesn't add much functionality beyond what's already available.

Some of the limitations may be able to be overcome by integrating with something like flood (flood.js.org). This would also provide coverage for additional clients.

It is a sonarr limitation in this case. There is no automation with series packs and likely never will be. This also means manual intervention is required no matter what.
I would not expect an app that works with sonarr to do things sonarr doesn't even handle or support. The sonarr integration is just that, so it works as it should.
The alternative is a folder watcher which also does what is being asked of a manually grabbed item that isn't supported via api.
It is alot of work to go against the intended work flow to help users who grab things that are not even going to work (even if they where not packed as well).
Just my 2 cents anyways

I didn't even catch the original statement about season packs. Now that I understand the problem you're trying to solve, I can safely say that it's really difficult to achieve that outcome. If Unpackerr integrated with qbit it's now going to extract everything qbit downloads, but then what logic? It gets complicated. Does it also check with the starr app to see if it's imported so it can delete it? That logic wont work with season packs since they never get imported. What if it extracted a game or app? I suppose you don't want that deleted after extraction. Do you just turn off clean-up entirely? In which can you can run find | unrar in cron and get the same result.
There's nothing unpackerr can do that will make sonarr import a season pack.

Is a qbit integration still something you'd use even though it can't solve the problem of sonarr importing season packs?

i'm using the folder watcher currently and its working well.


Create a controller function to get all organizations

#I want to work on this

@gauravdas014  how to add up this? like there's no predefined model,route for the organization stuff

The organization model is being created. For now assume there's an organization model. So create the function to get all organizations. Make the routes as well. Follow REST architecture.

Okay!!


Add npm install bcrypt-nodejs to readme?

The current README does not make it clear how to install this module.
Can I add:
npm install bcrypt-nodejs

To clarify this for people who arrive at the GitHub repo rather than finding through npm?

I vote for this!


Backend deployment documentation and front-end deployment documentation， addresses cannot be accessed normally

https://github.com/apache/incubator-dolphinscheduler/tree/1.2.1

sorry for reply so late，the document are as follows:
https://dolphinscheduler.apache.org/en-us/docs/1.2.0/user_doc/quick-start.html
https://dolphinscheduler.apache.org/zh-cn/docs/1.2.0/user_doc/quick-start.html

some error has been corrected, thanks for telling us


Waybar 0.10.3, libcava 0.10.1, segfaults with cava module enabled in bar configuration

Tested with --buildtype=debug.
waybar -l trace:
[2024-05-16 02:15:30.380] [debug] Try expanding: $XDG_CONFIG_HOME/waybar/config
[2024-05-16 02:15:30.380] [debug] Try expanding: $XDG_CONFIG_HOME/waybar/config.jsonc
[2024-05-16 02:15:30.380] [debug] Try expanding: $HOME/.config/waybar/config
[2024-05-16 02:15:30.380] [debug] Try expanding: $HOME/.config/waybar/config.jsonc
[2024-05-16 02:15:30.380] [debug] Found config file: $HOME/.config/waybar/config.jsonc
[2024-05-16 02:15:30.380] [info] Using configuration file /home/origincode/.config/waybar/config.jsonc
[2024-05-16 02:15:30.385] [info] Discovered appearance 'dark'
[2024-05-16 02:15:30.385] [debug] Try expanding: $XDG_CONFIG_HOME/waybar/style-dark.css
[2024-05-16 02:15:30.385] [debug] Try expanding: $XDG_CONFIG_HOME/waybar/style.css
[2024-05-16 02:15:30.385] [debug] Try expanding: $HOME/.config/waybar/style-dark.css
[2024-05-16 02:15:30.385] [debug] Try expanding: $HOME/.config/waybar/style.css
[2024-05-16 02:15:30.385] [debug] Found config file: $HOME/.config/waybar/style.css
[2024-05-16 02:15:30.385] [info] Using CSS file /home/origincode/.config/waybar/style.css
[2024-05-16 02:15:30.393] [debug] Output detection done: DP-1 (GSM LG ULTRAGEAR 0x0001D493)
[2024-05-16 02:15:30.395] [warning] Mapping is not an object
fish: Job 1, './build/waybar -l trace' terminated by signal SIGSEGV (Address boundary error)

gdb waybar:
(gdb) bt
#0  __GI_strlen () at ../sysdeps/x86_64/multiarch/strlen-sse2.S:142
#1  0x0000555555593d2f in std::char_traits<char>::length (__s=0x0) at /usr/include/c++/13.2.0/bits/char_traits.h:409
#2  0x00005555557f0e34 in std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::replace (this=0x7fffffffb4f0, __pos=0, __n1=16, __s=0x0)
    at /usr/include/c++/13.2.0/bits/basic_string.h:2221
#3  0x00005555557ee2f5 in waybar::modules::Cava::Cava (this=0x555555cc3570, id="", config=...) at ../src/modules/cava.cpp:15
#4  0x0000555555591fde in waybar::Factory::makeModule (this=0x7fffffffc8c0, name="cava", pos="modules-left") at ../src/factory.cpp:311
#5  0x0000555555618ade in waybar::Bar::getModules (this=0x555555addda0, factory=..., pos="modules-left", group=0x0) at ../src/bar.cpp:495
#6  0x00005555556190e0 in waybar::Bar::setupWidgets (this=0x555555addda0) at ../src/bar.cpp:543
#7  0x00005555556167b2 in waybar::Bar::Bar (this=0x555555addda0, w_output=0x555555bca1c0, w_config=...) at ../src/bar.cpp:279
#8  0x0000555555631a74 in std::make_unique<waybar::Bar, waybar::waybar_output*, Json::Value const&> () at /usr/include/c++/13.2.0/bits/unique_ptr.h:1070
#9  0x000055555562523a in waybar::Client::handleOutputDone (data=0x555555bca1c0) at ../src/client.cpp:83
#10 0x00007ffff65a6052 in ffi_call_unix64 () at ../src/x86/unix64.S:104
#11 0x00007ffff65a4917 in ffi_call_int (cif=cif@entry=0x7fffffffcf70, fn=fn@entry=0x5555556250ee <waybar::Client::handleOutputDone(void*, zxdg_output_v1*)>, rvalue=<optimized out>,
    rvalue@entry=0x0, avalue=<optimized out>, closure=closure@entry=0x0) at ../src/x86/ffi64.c:673
#12 0x00007ffff65a504d in ffi_call
    (cif=cif@entry=0x7fffffffcf70, fn=0x5555556250ee <waybar::Client::handleOutputDone(void*, zxdg_output_v1*)>, rvalue=rvalue@entry=0x0, avalue=avalue@entry=0x7fffffffd040)
    at ../src/x86/ffi64.c:710
#13 0x00007ffff7f7b6e1 in wl_closure_invoke (closure=closure@entry=0x555555b22f00, flags=flags@entry=1, target=<optimized out>,
    target@entry=0x555555adcfe0, opcode=opcode@entry=2, data=<optimized out>) at ../src/connection.c:1025
#14 0x00007ffff7f77be8 in dispatch_event (display=display@entry=0x5555559af380, queue=0x5555559af470) at ../src/wayland-client.c:1631
#15 0x00007ffff7f79534 in dispatch_queue (queue=0x5555559af470, display=0x5555559af380) at ../src/wayland-client.c:1777
#16 wl_display_dispatch_queue_pending (display=0x5555559af380, queue=0x5555559af470) at ../src/wayland-client.c:2019
#17 0x00007ffff7f7958c in wl_display_dispatch_pending (display=<optimized out>) at ../src/wayland-client.c:2082
#18 0x00007ffff76da044 in _gdk_wayland_display_queue_events (display=<optimized out>) at ../gdk/wayland/gdkeventsource.c:201
#19 0x00007ffff76a83f9 in gdk_display_get_event (display=0x5555559d3480 [GdkWaylandDisplay]) at ../gdk/gdkdisplay.c:442
#20 0x00007ffff76e2a12 in gdk_event_source_dispatch (base=<optimized out>, callback=<optimized out>, data=<optimized out>) at ../gdk/wayland/gdkeventsource.c:120
#21 0x00007ffff6a95498 in g_main_dispatch (context=context@entry=0x5555559dbe60) at ../glib/gmain.c:3344
#22 0x00007ffff6a985a7 in g_main_context_dispatch_unlocked (context=0x5555559dbe60) at ../glib/gmain.c:4152
#23 g_main_context_iterate_unlocked (context=context@entry=0x5555559dbe60, block=block@entry=1, dispatch=dispatch@entry=1, self=<optimized out>) at ../glib/gmain.c:4217
#24 0x00007ffff6a98bac in g_main_context_iteration (context=context@entry=0x5555559dbe60, may_block=may_block@entry=1) at ../glib/gmain.c:4282
#25 0x00007ffff6cfb42d in g_application_run (application=0x5555559a4fe0 [gtkmm__GtkApplication], argc=<optimized out>, argv=<optimized out>) at ../gio/gapplication.c:2712
#26 0x00005555556275de in waybar::Client::main (this=0x55555591d780, argc=1, argv=0x7fffffffde48) at ../src/client.cpp:287
#27 0x0000555555611ef0 in main (argc=1, argv=0x7fffffffde48) at ../src/main.cpp:106

Seems to be related to config file loading?

Yeah it seems to be something wrong when $XDG_CONFIG_HOME is empty.

Yeah, need to add validation for $XDG_CONFIG_HOME.

Hi @OriginCode  , fix is merged. Please try again with the master branch

Hi @OriginCode , fix is merged. Please try again with the master branch

Yeah it doesn't segfaults now as it just cannot find the file lol.
You might want to consider implement the spec to check $HOME/.config as a fallback. There are distros that does not manually set $XDG_CONFIG_HOME.

Checking your point...

Hi @OriginCode , thank you for your feedback. Related merge request is created to the original cava upstream. See karlstav/cava#565. Once it's done I'll do sync with libcava and bump libcava version.

Hi @OriginCode , please check with the new commits. Now it should be fine. If it's so, please close the issue

@LukashonakV It still doesn't recognize $XDG_CONFIG_HOME when the variable is empty :(
Also there seems to be some weird issue in the log (see the path in the error message, it's not the same as in my config):

@LukashonakV It should look for $HOME/.config as fallback to $XDG_HOME_CONFIG.

Hi @OriginCode , I fixed the issue when libcava expands user variables and constructs file path for opening. Please update waybar or libcava-0.10.3 and let me know about results. Thanks

Hi @OriginCode , I fixed the issue when libcava expands user variables and constructs file path for opening. Please update waybar or libcava-0.10.3 and let me know about results. Thanks

I still get [error] Error loading config. Unable to open file '${XDG_CONFIG_HOME}/cava/cava.conf', exiting... with master branch waybar and libcava 0.10.3.

@OriginCode you should use $XDG_CONFIG_HOME. NOT ${XDG_CONFIG_HOME}

@OriginCode you should use $XDG_CONFIG_HOME. NOT ${XDG_CONFIG_HOME}

Both don't work on my end. :(

This error means you have not configured xdg  variable

This error means you have not configured xdg variable . env | grep -i xdg

As I mentioned before, according to https://specifications.freedesktop.org/basedir-spec/latest/index.html#variables when the variable is not set, it should look up $HOME/.config as a fallback :)

This error means you have not configured xdg variable . env | grep -i xdg

As I mentioned before, according to https://specifications.freedesktop.org/basedir-spec/latest/index.html#variables when the variable is not set, it should look up $HOME/.config as a fallback :)

Actually, I'm still thinking of fallbacks... the reason,  to me it  seems obvious it's the user responsibility to provide correct content into the API. When the fallback happened, it was not clear which config now is used

This error means you have not configured xdg variable . env | grep -i xdg

As I mentioned before, according to https://specifications.freedesktop.org/basedir-spec/latest/index.html#variables when the variable is not set, it should look up $HOME/.config as a fallback :)

Actually, I'm still thinking of fallbacks... the reason, to me it seems obvious it's the user responsibility to provide correct content into the API. When the fallback happened, it was not clear which config now is used
So, for now, fallback mechanic is not considered.

No pressure, at least $HOME works fine for now. I know it's a weird to implement fallback for some environment variables. Just saying that XDG spec is followed and used widely by Linux distributions, it may be weird to users using the variable in the way the spec describes. Thank you for fixing the segfault ;)


[WIP] Allow the passing of a custom init.vim path.

This PR should allow the defining of a custom init.vim path in the Oni config file.
I need to check into some error checking, ie if an incorrect path is given, as well as some testing outside of Windows (Mac or Linux users would be useful, since I've only got Window + a few Linux VMs).
Also, is it acceptable to assume the user will sufficiently escape the path they provide?
Ie if they are on Windows and their init.vim is at "C:\Users\CrossR\init.vim", they would need to set it with escaped back slashes as follows:
"oni.customInitVimPath" : "C:\\Users\\CrossR\\init.vim".
Also since this is applied as an argument at startup, setting a custom path isn't applied until restarting oni currently. Is it worth looking into applying it straight away, or should that be a separate PR?

This could help with #436.

Cool, thanks for submitting this @CrossR ! I definitely think it makes sense to be able to override the init.vim path. There are some cases where users would prefer to have a completely separate init.vim between Oni/Neovim, so this helps accomplish that.
The only concern I have is that the configuration settings we have today are already confusing - oni.loadInitVim, oni.useDefaultConfig, and oni.customInitVimPath. I'd like to see if we could consolidate the oni.loadInitVim and oni.customInitVimPath configuration settings, if possible (since setting oni.customInitVimPath would imply oni.loadInitVim is true).
A couple of options I'm thinking about:

Change oni.loadInitVim to support string | boolean. If it is true, we would just load init.vim from its usual place. If it is a string, we'd load it from the custom path.
Only support custom paths - We could make oni.customInitVimPath be the way to using an init.vim, whether it is in the default place or not, and deprecate oni.loadInitVim. The downside is it would hose users that use oni.loadInitVim, we'd have to make it clear we are deprecating that value and how to upgrade - it would also be a bit more inconvenient for users who pick up their config in the default place, because they might have to look up where that is.

I'd lean towards the first one, just because it is simplest... but I'm sure there are other ways we could consolidate this and simplify the relationship between oni.loadInitVim/oni.customInitVimPath. Let me know what you think!

The only concern I have is that the configuration settings we have today are already confusing - oni.loadInitVim, oni.useDefaultConfig, and oni.customInitVimPath. I'd like to see if we could consolidate the oni.loadInitVim and oni.customInitVimPath configuration settings, if possible (since setting oni.customInitVimPath would imply oni.loadInitVim is true).

Yep, totally agree!
I did find myself thinking this as I was coding it, but wasn't too sure if it was the right way to go, though I hadn't considered letting oni.loadInitVim accept either a string or a boolean. That makes the logic a lot cleaner, and clears up the user config.
I'll have a look at implementing it now.

Consolidated those two options together now and seems to work well!
Also swapped to using the built in open where I can, and updated some variable names to be more appropriate.
There is still some oddness with the interactions with oni.useDefaultConfig, but only the same that is in the live version, and this is outlined in the Wiki, so its possibly an issue to raise later.

Excellent, looks great! I just tried it and it works well - thanks for making the changes!

There is still some oddness with the interactions with oni.useDefaultConfig, but only the same that is in the live version, and this is outlined in the Wiki, so its possibly an issue to raise later.

Definitely, I'd like to see how to consolidate this at some point. But as you mentioned at least this change doesn't introduce any new issues, the problems are pre-existing 😄
I'll go ahead and bring this in. I made a quick edit to the wiki to detail that a string path can be accepted, but feel free to revise.