fix redirect: use replaceWith propagate changes to codePath

Fixes back button issue

I don't think any changes are needed. This fixes the problem completely.

hmmm recent files has some behaviour which is unexpected. This also occurs on staging
https://github.com/cardstack/boxel/assets/8165111/bfa6f10e-7b5f-4986-9a3f-f391cb0c24ec

hmmm recent files has some behaviour which is unexpected. This also occurs on staging
Solutioneering wud be to addRecentFiles if it has an extension.
Screen.Recording.2023-09-14.at.8.54.53.PM.mov
EDIT: the blinking of author to author.gts has to do with a race condition of the addRecentFiles in the file resource and the addRecentFiles in the operator mode

This was resolved by https://github.com/cardstack/boxel/pull/648

Actually do we expect this redirect to be testable? For example, clicking the definition container which fetches author content but goes thru a redirect. I was thinking of adding a test like this but the fetch doesn't follow the redirect automatically.
I notice that the TestRealmAdapter is delegating to realm.handle that will return status of 302 in getCardSourceRedirect. But when I receive a 302, it goes back to the file resource without following the redirect.
In the normal app, because no URLHandlers are registered, this request goes directly to the native fetch. and the 302 is succesfully followed. And I receive a 200 going back to the file resource.
If it's suposed to be testable what is the best way to handle this in the TestRealmAdapter?

Actually do we expect this redirect to be testable? For example, clicking the definition container -- which fetches author content but goes thru a redirect. I was thinking of adding a test like this but the fetch doesn't follow the redirect automatically.
I notice that the TestRealmAdapter is delegating to realm.handle that will return status of 302 in getCardSourceRedirect. But when I receive a 302, it goes back to the file resource without following the redirect.
In the normal app, because no URLHandlers are registered, this request goes directly to the native fetch. and the 302 is succesfully followed. And I receive a 200 going back to the file resource.
If it's suposed to be testable what is the best way to handle this in the TestRealmAdapter?

i think the way you would test it would be to use the DOM tests in the realm-server workspace. since that uses a real fetch against real servers.

Actually do we expect this redirect to be testable? For example, clicking the definition container -- which fetches author content but goes thru a redirect. I was thinking of adding a test like this but the fetch doesn't follow the redirect automatically.
I notice that the TestRealmAdapter is delegating to realm.handle that will return status of 302 in getCardSourceRedirect. But when I receive a 302, it goes back to the file resource without following the redirect.
In the normal app, because no URLHandlers are registered, this request goes directly to the native fetch. and the 302 is succesfully followed. And I receive a 200 going back to the file resource.
If it's suposed to be testable what is the best way to handle this in the TestRealmAdapter?

i think the way you would test it would be to use the DOM tests in the realm-server workspace. since that uses a real fetch against real servers. these particular tests are specificlaly around testing the ember routing for realm servers which is totally the area this change is concerned with

this is looking really good. it would be neat if you were able to write a DOM test in the realm server that could test this update.

Yea. I actually tried adding the dom test but it didn wanna redirect somehow I think theres other things I may need to add. I'll handle it in this ticket


Error: Cannot find module 'vscode-gulp-watch'

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

VS Code Version: latest
OS Version: macos 13.0.1 (22A400)
node: v16.19.0
yarn

Steps to Reproduce:

git clone https://github.com/microsoft/vscode.git
yarn
Failure step: ./scripts/code.sh

/gulp/bin/gulp.js
[watch-client    ]     at Function.Module._resolveFilename (node:internal/modules/cjs/loader:995:15)
[watch-client    ]     at Function.Module._load (node:internal/modules/cjs/loader:841:27)
[watch-client    ]     at Module.require (node:internal/modules/cjs/loader:1067:19)
[watch-client    ]     at require (node:internal/modules/cjs/helpers:103:18)
[watch-client    ]     at Object. (/Users/joao.costa/dev/vscode/build/lib/watch/index.js:6:73)
[watch-client    ]     at Module._compile (node:internal/modules/cjs/loader:1165:14)
[watch-client    ]     at Object.Module._extensions..js (node:internal/modules/cjs/loader:1219:10)
[watch-client    ]     at Module.load (node:internal/modules/cjs/loader:1043:32)
[watch-client    ]     at Function.Module._load (node:internal/modules/cjs/loader:878:12)
[watch-client    ]     at Module.require (node:internal/modules/cjs/loader:1067:19) {
[watch-client    ]   code: 'MODULE_NOT_FOUND',
[watch-client    ] error Command failed with exit code 1.
[watch-client    ] info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
[watch-extensions] Error: Cannot find module 'vscode-gulp-watch'

Did you build after running yarn? https://github.com/microsoft/vscode/wiki/How-to-Contribute

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

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


Toolbar goes behind the status bar on login screen

Is it a ?

[x] Bug Report

[ ] Feature Request

[ ] Chore

Bug Report
Actual Behaviour
When typing in any field, the toolbar goes behind the status.
Expected Behaviour
It should stay stuck at the top
Steps to reproduce it
Type in any text field in login screen
Screenshots of the issue

@iamareebjamal  here I can't see the bug, is it android version specific?

Maybe depends on screen size

@iamareebjamal Can you please specify the Android device and version you were using when you faced the bug?

Marshamallow. Device doesn't matter, anything above lollipop should have same behaviour

@iamareebjamal I checked it on Motorola E3 power running 6.0 and Xiomi Redmi Note 4 running 7.0 and it worked fine on both.

Try to scroll the UI, or directly click on Base URL. The keyboard will push the UI under status bar

@iamareebjamal Can we set the fitSystemWindows to true in the root layout? This would solve the issue.

@sridharjajoo Yes, I'm sending PR for the same.


VSCODE for WSL BUG！！！！！

The program appears flower screen!!!!!!!!!!!!  for WSL plugin

/needsMoreInfo

You would better use the issue template provided by the community and should provide basic information about your questions. Otherwise people won't be able to help you.


⚠ ClamAV has degraded performance

In 43a1bc9, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 91 ms

Resolved: ClamAV performance has improved in 85f529e.


Allow emojis

Is your feature request related to a problem? Please describe.
To get a quick information which part of my smart home is affected from the message it would be great when emojis could be used.
Describe the solution you'd like
CallMeBot already supports this:
https://www.callmebot.com/uncategorized/how-to-use-emoticos-with-the-api/
Problem seems to be the encoding of the text.
Maybe there could be another state for already encoded text ? Then there could be entered the emoji with the text already encoded.

Please check GitHub version


Send email to creator when project/collection is enabled

Send email to creator when project/collection is enabled
Template:
Dear <project name> team,

We are glad to let you know that your collection/project is now active and can be accessed at <link_to_collection/project>

Best Regards
ArtZero Support Team

Account free emailjs.com allow 2 template only, need upgrade premium to get 6. @nad128668 @phoebehoang85

Change the email content sent to creators @langdon0003
Dear  team,
Thank you for creating a new Collection/Project  on ArtZero.io

project address:
email:
Telegram ID:
We will contact you within 48 hours for further support.

Best Regards,
ArtZero Support Team


[FEATURE REQ]Extend AzureEngSemanticVersion to Support other Language Versions format

This should probably be done after settling on a standard naming for preview versions in all languages.
AzureEngSemanticVersion currently does not properly support version formats for all languages.
Some Java Track 1 packages with version like 1.0.1-beta is not supported.
When the version is a preview ToString() method only works for dotnet.
IncrementAndSetToPrerelease() also supports dotnet specifically
The class seems to have been built for with the dotnet version format in mind but right now it is being used for other languages.

this has been addressed with this PR https://github.com/Azure/azure-sdk-tools/pull/935/files


[Models][UX]: Design Icon for ArgoCD Component - Cluster Workflow Template

Description
Meshery is a powerful platform that contains over 200 integrations for managing cloud-native applications. MeshMap is an extension of Meshery that visually represents these integrations and components. Currently, most of the icons used in MeshMap have the default shape of a circle. However, to improve visual clarity and consistency, we need to design custom icons for MeshMap components.
Desired Behavior
A new Icon for  ** Cluster Workflow Template** needs to be created. This issue calls for a volunteer to draft a new Icon.
Instructions for Designing a Component Icon

Create two versions of the icon one is white and another one is colored
Provide the icon in SVG format
Make a PR after adding the icon inside the new-icon folder present in the meshery repository
Make sure the icon for the component is different from the Model icon

How to check whether the icon you designed is the same as the Model icon or any other component icon?

Go to the playground -> search for the component you are trying to design an icon for and check whether the icon that you are designing is already present for another component or not if not then you can make a PR.
If you don't have access to Playground then you have to fill out a signup and get verified before you access the Playground by the time you get access, you can go to integrations page in layer5.io site and search for the component and see whether an icon is already present or not.

Contributor Guides and Handbook

MeshModel Repository
🖥 MeshModel Overview
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

I am interested in helping with this issue. Can I be assigned?


Better messages about stages: add dimg and artifact names

dimg or artifact name is printed at the beggining of build log. Stages or commands can be verbose and in case of error user see only this:
verbose output from commands
>>END STREAM

Another case is command that print messages and then hanging. User needs to scroll up to know which dimg or artifact cause troubles.
It will be better if message about starting stage will contains dimg or artifact name. Something like this
  Install group
    Install [test-app]                                                                       [BUILDING]
Mon Apr 23 09:11:47 UTC 2018
      Launched command: `date && mkdir /app && echo Start > /app1/start.sh`
    Install                                                                                [FAILED] 1.28 sec

or like this
  Install group
    Install                                                                                [BUILDING]
      dimg: test-app
Mon Apr 23 09:11:47 UTC 2018
      Launched command: `date && mkdir /app && echo Start > /app1/start.sh`
    Install                                                                                [FAILED] 1.28 sec

More issues about messages: #392

<dimg name>: <stage name>:

app: before_install_artifact;
or just before_install_artifact for nameless dimg.

and also reject messages like Install group and Setup group


Create Credential Error

In reference to
https://docs.microsoft.com/en-us/azure/sql-database/sql-database-managed-instance-restore-from-backup-tutorial
When following these steps, I got as far as uploading the backup file, but when I connected to the Azure Database and ran the Create Credential command, I got an error telling me that the command is not supported on this version of SQL Server. What should I do next?

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

ID: 472c6d00-ec8b-01c5-64cd-1951fc3a0834
Version Independent ID: cbc409b0-b37d-f477-a8a7-a25c4de844f2
Content: Restore a backup to Azure SQL Database Managed Instance
Content Source: articles/sql-database/sql-database-managed-instance-restore-from-backup-tutorial.md
Service: sql-database
GitHub Login: @bonova
Microsoft Alias: bonova

@cjh15hub Thank you very much for your inquiry to find a resolution to your issue. Have you deployed an Azure SQL Database instance or have you deployed an instance of Azure SQL Database Managed Instance? The Managed Instance option is under Private Preview, where you need to be 'approved' to participate in the preview period. If you do not recall going through this approval process, you are most likely using an instance of Azure SQL Database (single database) and this version of Azure SQL does not support all the features found in the Managed Instance version.
Feature comparison: Azure SQL Database versus SQL Server
Azure SQL Database Managed Instance T-SQL differences from SQL Server

SQL Server (on-premise) is the full feature product.
Azure SQL Database Managed Instance is the Azure SQL option that has most of the SQL Server functionality.
Azure SQL Database has a subset of functionality found in Azure SQL database Managed Instance.
SQL server deployed to an Azure VM has the same functionality as SQL Server (on-premise).

The tutorial you are following is intended for an Azure SQL Database Managed Instance and will not work with Azure SQL Database.
Please let me know if you have additional questions.
Regards,
Mike

Thank you Mike. Yes I see that I was following a tutorial for the wrong version, I think. Although I don't know where to go from here. I wanted to move a backup of an old database to Azure so that a colleague can move a subset of the data to Data Lake. Is there a guide for migrating/ restoring a database to Azure so that I can connect services to it?

I found that I should have been using the Data Migration Assistant. This accomplished what I needed.


Error while trying to install ruby gem

After running gem install jekyll-lunr-js-search I get the following error
Building native extensions.  This could take a while...
ERROR:  Error installing jekyll-lunr-js-search:
ERROR: Failed to build gem native extension.
current directory: /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/ext/libv8

/Users/an-le/.rbenv/versions/2.2.3/bin/ruby -r ./siteconf20160304-43163-1rv3vcr.rb extconf.rb
creating Makefile
Compiling v8 for x64
Using python 2.7.10
Using compiler: /usr/bin/c++ (clang version 6.1.0)
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/libtool: file: /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/atomicops_internals_x86_gcc.o has no symbols
ld: library not found for -lgcc_s.10.5
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make[1]: *** [/Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/preparser] Error 1
make: *** [x64.release] Error 2
/Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/ext/libv8/location.rb:36:in block in verify_installation!': libv8 did not install properly, expected binary v8 archive '/Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/tools/gyp/libv8_base.a'to exist, but it was not found (Libv8::Location::Vendor::ArchiveNotFound) 	from /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/ext/libv8/location.rb:35:in each'
from /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/ext/libv8/location.rb:35:in verify_installation!' 	from /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/ext/libv8/location.rb:26:in install!'
from extconf.rb:7:in `'
GYP_GENERATORS=make
build/gyp/gyp --generator-output="out" build/all.gyp
-Ibuild/standalone.gypi --depth=.
-Dv8_target_arch=x64
-S.x64  -Dv8_enable_backtrace=1 -Dv8_can_use_vfp2_instructions=true -Darm_fpu=vfpv2 -Dv8_can_use_vfp3_instructions=true -Darm_fpu=vfpv3 -Dwerror=''
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/allocation.o
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/atomicops_internals_x86_gcc.o
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/bignum.o
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/bignum-dtoa.o
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/cached-powers.o
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/conversions.o
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/diy-fp.o
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/dtoa.o
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/fast-dtoa.o
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/fixed-dtoa.o
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/once.o
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/preparse-data.o
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/preparser.o
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/preparser-api.o
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/scanner.o
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/strtod.o
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/token.o
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/unicode.o
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser_lib/src/utils.o
LIBTOOL-STATIC /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/libpreparser_lib.a
CXX(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/obj.target/preparser/preparser/preparser-process.o
LINK(target) /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS>/vendor/v8/out/x64.release/preparser
extconf failed, exit code 1
Gem files will remain installed in /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/libv8-<IP_ADDRESS> for inspection.
Results logged to /Users/an-le/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/extensions/x86_64-darwin-15/2.2.0-static/libv8-<IP_ADDRESS>/gem_make.out

@ntnlbd The error log appears to point to installation of the libv8 Gem failing.
It could be related to the following issue that has been raised: https://github.com/cowboyd/libv8/issues/202

I had the same problem. This solved it for me: https://github.com/cowboyd/libv8/issues/205


RPC Server Implementation

Added docs for presentation layer entities.
Added new example with custom (not Nunavut generated) "ping" service. Communication between two such nodes looks like this:

Local  node ID: 42
Remote node ID: 47
-----------
Running...
🔴 Server received 'Ping' req (id=1005, from_node_id=47).   @      565 ms
🠊  Client sending  'Ping' req (id=1001,   to_node_id=47).   @     1000 ms
 🠈 Client received 'ponG' res (id=1001, from_node_id=47).   @     1000 ms, Δ 208 us
🔴 Server received 'Ping' req (id=1006, from_node_id=47).   @     1566 ms
🠊  Client sending  'Ping' req (id=1002,   to_node_id=47).   @     2000 ms
 🠈 Client received 'ponG' res (id=1002, from_node_id=47).   @     2000 ms, Δ 195 us
🔴 Server received 'Ping' req (id=1007, from_node_id=47).   @     2565 ms
🠊  Client sending  'Ping' req (id=1003,   to_node_id=47).   @     3000 ms
 🠈 Client received 'ponG' res (id=1003, from_node_id=47).   @     3000 ms, Δ 246 us
🔴 Server received 'Ping' req (id=1008, from_node_id=47).   @     3566 ms
🠊  Client sending  'Ping' req (id=1004,   to_node_id=47).   @     4000 ms
 🠈 Client received 'ponG' res (id=1004, from_node_id=47).   @     4000 ms, Δ 173 us
Done.
-----------
Stats:
worst_callback_lateness  = 203 us
call_duration_stats_mean = 205.5 us
call_duration_stats_std  = ± 26.5189 us (±12.9%)

Also:

renamed examples so that their name describes what it does

@lydia-at-amazon @pavel-kirienko Guys, please review. Thank You!


az: error: unrecognized arguments: --password

Hi Team,
Creating service principal command is failing on the argument --password
az ad sp create-for-rbac --name "yourAzureServicePrincipalName" --password yourSecurePassword
However, it works without adding password, it's automatically generated (as also mentioned in the document).
Also, it works the same without quotation marks on the name, despite what is mentioned in the document.
See the screenshot for the 3 scenarios:

Not sure if this is a documentation bug or the argument has changed or no longer used?

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

ID: 4fd46b73-61b1-0abc-1e8d-7e14f1893712
Version Independent ID: 94b1c084-a988-f125-1127-feb890fb8bb7
Content: Tutorial - Deploy from GitHub to Azure App Service using Jenkins
Content Source: articles/jenkins/deploy-from-github-to-azure-app-service.md
Service: jenkins
GitHub Login: @TomArcherMsft
Microsoft Alias: tarcher

@saifbadran9 Thanks for the feedback. It looks like this is an older doc that we need to update.

@saifbadran9 This article has been updated to reflect the change to the az ad sp create-for-rbac command. It should be live tomorrow. Thanks again for the heads-up.


Theme colors do not load (immediately) on SP listpage or site contents page

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Developer environment
Linux
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[X] 💥 Google Chrome
[X] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

browser version: Chrome 97.0.4692.99
SPFx version 1.13.1/1.13.0
Node.js version 14.18.1

Describe the bug / error
When developing a search bar in the page header using an application customizer extension I noticed that spfx would not load the SharePoint provided theme when i navigate to a list or the site contents page. It would always use the default light blue theme.
But it applies the correct theme to components that render at a later point e.g. after a asynchonous fetch.

On "normal" site pages everything is working as expected. (See below)

Steps to reproduce

Create SPFX Solution using yo (mine contains webparts and extensions)
Create an extension that uses Placeholder API to add a header to SP pages

this.context.placeholderProvider.tryCreateContent(PlaceholderName.Top)

Apply theme using getTheme().palette.<desired_color> to any element that is rendered immediately

import { Stack, getTheme } from '@microsoft/office-ui-fabric-react-bundle';
// Start react functional component
<Stack horizontalAlign="center" styles={{ root: {backgroundColor: getTheme().palette.themeDarkAlt}}}>
// more DOM elements

Look at the header background color on a page and a list. On a list page it will have the default theme and on a page it will be temed propperly
Add an asynchronous call and delay rendering of the themed elements in the header. See how it applies the corrent theme now.

Expected behavior
The correct theme should be applied to SPFX solutions on all sites where they can be placed, including lists and the site contents page.

Hi @Oshimani - the theme is loaded asynchronously, and it's completely fine if the getTheme method you mentioned returns "initial" theme.
I would recommend you to use ThemeProvider as described here: https://docs.microsoft.com/en-us/sharepoint/dev/spfx/web-parts/guidance/supporting-section-backgrounds#use-the-background-color-awareness-in-react-based-web-parts

@AJIXuMuK thank you for this hint.
I am sorry to tell you that this did not solve my problem. In fact this code produced confusing errors when I tried to copy it as is.
this.context.serviceScope.consume(ThemeProvider.serviceKey as any);

I had to add as any to this expression to get it past the typescript compiler.
Now for the next problem:
I importet getTheme from @microsoft/office-ui-fabric-react-bundle. It does not react to changes to the theme. However components from office-ui-fabric-react do update when I change the theme.
Can you tell me the difference between getTheme from @microsoft/office-ui-fabric-react-bundle and office-ui-fabric-react please?
Just to make clear: I have been using getTheme to access the selected theme in SharePoint for quite a while now. The only difference is, that I have an extension that will also be visible on lists and only here the problem shows.
I get quite confused with the different packages provided in spfx scaffolding project. There are multiple definitions for types like ITheme, getTheme, ThemeProvider just to name a few.
Is there a guide that describes in detail how and when to use which definition?
Thanks for your help so far!

Can you tell me the difference between getTheme from @microsoft/office-ui-fabric-react-bundle and office-ui-fabric-react please?

The only difference is version. office-ui-fabric-react-bundle is based on office-ui-fabric-react and provide as set of resources that are used on all SharePoint pages.
There might be difference in when exactly the module/library is loaded, and it could affect the provided value.

I am sorry to tell you that this did not solve my problem. In fact this code produced confusing errors when I tried to copy it as is.
I'm attaching a simple example of Application Customizer that is using ThemeProvider with no need for any.

I get quite confused with the different packages provided in spfx scaffolding project. There are multiple definitions for types like ITheme, getTheme, ThemeProvider just to name a few.
Is there a guide that describes in detail how and when to use which definition?

I believe you use libraries like PnP React Controls or some other 3p modules.
These libraries can have references on older versions of SPFx. And if you import the type from a reference from this library - you can get a typing issue.
ac-themes.zip
Please, let me know if the explanation and the same resolve you issue.

Thanks for this example. I did some testing with it.

It does work fine on site pages
It does not work on list pages, there the following error occurs

But refreshing the page after changing the theme makes it work for one refresh. There seems to be a race condition of some sort. Since the placeholder top and bottom features are available on list pages I think they should work just the same.

I believe you use libraries like PnP React Controls or some other 3p modules.
These libraries can have references on older versions of SPFx. And if you import the type from a reference from this library - you can get a typing issue.

I do not use any other UI-Components Library other than the preinstalled UI Fabric in this extension. However I use the PnP property pane controlls in a webpart that is part of this solution but I'm very careful to import components from @microsoft/office-ui-fabric-react-bundle

Thanks @Oshimani for the additional info.

It does not work on list pages, there the following error occurs
Yeah, it was more an example that you don't need to cast serviceKey to any. I haven't added any conditions or checks for null.
But, there was actually a bug with the initialization in some cases.
The needed changes were made and should be rolled out WW in the next couple of week.
After that you should be able to use ThemeProvider with no issues.

this should be rolled out ww by now.


Add a test_synthetic_data.py module using scope-generated TPFs

This issue describes a project which I would like @gully to work on when he returns from leave (and I may be gone).
In brief, I would like us to use @nksaunders' excellent scope package to generate three synthetic TPF files, and then use those synthetic files to add a new unit test module. Specifically, I propose the following two-phase plan:
Phase 1: Generate three synthetic TPF files
The first step is to use scope to generate three synthetic TPFs with the following requirements:

The three TPFs shall each contain a single star with K2-like noise and motion.
The three TPFs shall use different input light curves:

synthetic-k2-flat.targ.fits shall use a flat light curve;
synthetic-k2-sinusoid.targ.fits shall use a low-frequency sinusoid (e.g. 5-day period);
synthetic-k2-planet.targ.fits shall use a short-period Neptune-sized planet (e.g. Kp = 14; planet period = 5 days; transit depth = 2000ppm).

The TPFs shall include the input star position and light curves in the FITS file, e.g. as part of an extra "ground truth" extension, so that we can easily verify signal recovery.  Scope may already do this out of the box. If not, I recommend preparing a scope PR to add this feature.
The TPFs shall be sufficiently small to be added to the git repository, i.e. a few hundred Kb max.
The TPFs shall be added to the Lightkurve repository under lightkurve/tests/data/synthetic alongside a README file which documents how the files were generated.

Phase 2: Add unit test module
Once the three synthetic files are generated, the second step will be to create a new test module called lightkurve/tests/test_synthetic_data.py which performs the following sanity checks:

Can each of the three input light curves be recovered to a good level of accuracy after applying SFFCorrector?
Same for PLDCorrector?
Can the sinusoid period successfully be recovered from synthetic-k2-sinusoid.targ.fits using our LombScarglePeriodogram class?
Can the Neptune-sized planet period and depth be recovered from synthetic-k2-planet.targ.fits using our BoxLeastSquaresPeriodogram class?
Can the star positions be recovered to a good level of accuracy using tpf.estimate_centroids()?

There are a lot more tests and data challenges we can imagine executing using scope, but let's start with the simple TPFs and tests described above!

Progress so far:

[x] The three TPFs shall each contain a single star with K2-like noise and motion.

The three TPFs shall use different input light curves:

[x] synthetic-k2-flat.targ.fits shall use a flat light curve;
[x] synthetic-k2-sinusoid.targ.fits shall use a low-frequency sinusoid (e.g. 5-day period);
[x] synthetic-k2-planet.targ.fits shall use a short-period Neptune-sized planet (e.g. Kp = 14; planet period = 5 days; transit depth = 2000ppm).

[x] The TPFs shall include the input star position and light curves in the FITS file, e.g. as part of an extra "ground truth" extension, so that we can easily verify signal recovery. Scope may already do this out of the box. If not, I recommend preparing a scope PR to add this feature.

[x] The TPFs shall be sufficiently small to be added to the git repository, i.e. a few hundred Kb max.

[x] The TPFs shall be added to the Lightkurve repository under lightkurve/tests/data/synthetic alongside a README file which documents how the files were generated.

Ok, I completed Phase 2 in the ongoing pull request.

Closed by the above PR


Script to update to llvm syncpoint

Rather than updating TF to HEAD and updating the LLVM commit to match,
this script uses the LLVM commit as a synchronization point, which is
more in keeping with our current setup (we've had the LLVM update
feature turned off for a while now anyway).
This will later expand to other repositories that synchronize with LLVM
(mlir-hlo, llvm-bazel, etc.). I'm holding off on generalizing the script
much until we have those use cases, so the TF stuff is pretty hardcoded.
Here's an example TF PR from the script:
https://github.com/google/iree/pull/3154

This change is

git isn't detecting it, but the script is branched from update_tf_llvm_submodules.py

Reviewed 4 of 4 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @ScottTodd)

oooo did we find a different review system?

Urgh reviewable stuck things in the PR description :-/ I don't like that


Add 'action' parameter to the Component.

Description:
As a new user implementing Google reCAPTCHA, I noticed that the ‘action’ parameter is missing. The ‘action’ parameter plays a crucial role in tracking user-initiated events and metrics. Without it, we won’t have meaningful insights into user interactions.
Suggested Action:
Add the ‘action’ parameter to the reCAPTCHA implementation. And expose this as a parameter to the Component.
Benefits of Adding the ‘action’ Parameter:
Better visibility into user behavior through reCAPTCHA dashboards.
Improved monitoring for different forms of attacks (e.g., falsification or duplication of actions).

Thanks for your contribution! Sorry I've been busy these 3 weeks. I reviewed your PR and requested some changes.

@babaktaremi I'm not the PR creator, but I see no request for changes. Can you please consider making adjustments and releasing this? Thanks.

@babaktaremi I'm not the PR creator, but I see no request for changes. Can you please consider making adjustments and releasing this? Thanks.

Hi. Hope you are doing well. I requested 2 changes about 2 months ago that are still pending. I will work on the changes myself and hopefully will have the new version of the package this week

Version 5.0.0 is uploaded and will be available in few minutes. @Sire
thanks for your contribution! @devreepiet


Bump dependency on containers

@aisamanra Ping?

As a Hackage trustee I made a revision: https://hackage.haskell.org/package/config-ini-<IP_ADDRESS>


Decode a Morse code message: Feature/docs and morse decoder

🚩We have made the following changes to complete Decode a Morse code message
Here is a summary of what has been done
Type of files changed/added
, PNG
In this new Branch, We have ➕ADDED➕ the succeeding *📂 folders and/or **📄 files to complete this milestone's requirements
*📂 root/

**📄 morse_decoder.rb ➕

We have added three methods: for decoding a char, a string, and a message.

*📂 /assets/

**🖼 morse-decoder-logo.png ➕
- We have added an image logo

We have ✏MODIFIED✏ the succeeding *📂 folders and/or 📄** files
*📂 root/

**📄 README.md ✏

We have modified the documentation to follow Microverse guidelines

**📄 LICENSE ✏

We have added @mwismann.

Thank you for taking the time to review this PR ⭐

If you require additional information or have any questions, please contact me on Slack as Arturo (Arthur) Emanuel Guerra Iturres, Mathías Wismann We'll be happy to help you out. 🎯

Hi @Reem-lab Reem! so good to see you again! 😃, thank you for reviewing this PR! We wish you a great week! Cheers! 🥳


OCPBUGS-23976: Stop creating unwanted keda namespace

Cherry-pick of https://github.com/kedacore/keda-olm-operator/pull/235
Checklist

[X] Commits are signed with Developer Certificate of Origin (DCO)

/jira refresh

/lgtm


Llama3-8b 🦙

For whoever attempts to convert llama3 original weights to maxtext compatible version for personal use. Here are some suggestions I made to make it work as a end user.
First, delete"from jax.experimental.pallas.ops import attention as pallas_attention"
from Maxtext/layers/attention.py under this branch, or checkout this file from main.
Next, in end_to_end/tpu/llama3/8b/1_test_llama3_8b.sh, it is better to change /tmp/ in
"gcloud storage cp -r ${META_CHECKPOINT_PATH} /tmp/" to a local folder to avoid possible access permission error.
For me I switched it to ${HOME}. And make sure to change corresponding folder /tmp/meta-ckpt  in
"JAX_PLATFORMS=cpu python3 MaxText/llama_or_mistral_ckpt.py --base-model-path /tmp/meta-ckpt --maxtext-model-path ${BASE_OUTPUT_PATH}/${MODEL_VARIATION}/scanned_chkpt --model-size ${MODEL_VARIATION} from /tmp/meta-ckpt"
to the same designated folder, and subfolder name should be the same as subfolder  of ${META_CHECKPOINT_PATH}.
Also, when run "export BASE_OUTPUT_PATH=/path/to/GCS/bucket", don't include "/" after your bucket.
After made these modifications, you can also covert maxtext compatible weights to huggingface compatible safetensors for serving by doing ,
git checkout origin/anisha-orbax2hf -- MaxText/llama_or_mistral_orbax_to_huggingface.py
Make sure to double check function load_hf_models(model_size). And change if states to "llama3-8b".


Is there a way to add the scrollbar globally without manually adding it to every project or html element ?

Im working with micro-frontend applications and I came across simplebar and I implemented a working example in one of the Reactjs projects but the thing is Im not sure thats the best way to go forward because every application will need to have the same scrollbar and I'm afraid other developers would forget to include it in their project so I'm wondering if there is a way to have it globally added so that every project would have it. most of the projects are written in ReactJS and there might be some that will be written in other frameworks.
Thank you

Closing this as I got it done.

What is the solution?


Reading config fails if no microservcies are configured

https://github.com/IdentityPython/SATOSA/blob/34c5836e1a4e616348ac4fe029eff8d4d82df733/src/satosa/satosa_config.py#L47
If no microservcies are configured in proxy.yaml, SaToSa will not start nor construct SAML metadata as the above module fails to load plugin configuration:
File \"/home/debian/vopaas/virtualenv/lib/python3.5/site-packages/satosa/scripts/satosa_saml_metadata.py\", line 43, in create_and_write_saml_metadata,
    satosa_config = SATOSAConfig(proxy_conf),
File \"/home/debian/vopaas/virtualenv/lib/python3.5/site-packages/satosa/satosa_config.py\", line 50, in __init__,
    for config in self._config.get(key, []):,
TypeError: 'NoneType' object is not iterable"], "stdout": "", "stdout_lines": []

FYI, a way around this is to use the YAML syntax for empty list instead of letting the YAML parser put in the None value:
MICRO_SERVICES: []


No visible @interface for 'RCTRootView' declares the selector 'setReactPreferredFocusedView:'

Description
/Users/headless/tmp/AwesomeProject/node_modules/react-native/React/Views/RCTTVView.m:181:32: No visible @interface for 'RCTRootView' declares the selector 'setReactPreferredFocusedView:'
related to that commit https://github.com/facebook/react-native/commit/c92ad5f6ae74c1d398c7cd93d5c4c50da0ca0430. Before that I don't have this error.
@javache
Reproduction

Create new project with cocoapods and install react-native pod
update react-native

by this tutorial
https://github.com/skv-headless/AwesomeProject
Solution
Still looking
Additional Information

React Native version: master
Platform: [iOS simulator, iphone 5]
Operating System: [MacOS]

Maybe a solution to this would be to exclude TVView*s  from the podspec? And make that a separated target to include those

@ptomasroos I think that is right, or wrap RCTTVView in pragmas so it's not included in non-tvOS builds.

I've provided a fix in #11667

This can now be closed. Solution is merged in master


Loading in multiple datasets at once creates a cube while paint fails

When importing in multiple files in order to create a time variable like this:

Not only does nothing render and give a paint fail error, the bounding box, which should normally be very rectangular, with a low z dimension becomes a cube. Previously this didn't happen before until I was given an installer of the latest test version.

Helpful additional information
(Please click check boxes AFTER submitting ticket)

[ ] Did VAPOR crash?
[x] Did you get wrong results?

Impact

[ ] High - User productivity significantly degraded
[ ] Medium - User productivity partially degraded
[ ] Low - User productivity trivially degraded

To Reproduce
Steps to reproduce the behavior. For example:

Go to File -> Import -. NET-cdf
select multiple files.
select okay
See error

I've used the same datasets I sent days prior to replicate this bug.
Desktop

OS and version: Redhat Enterprise Linux 7
Version: Vapor3-3.2.0.alpha-hdf5-1.10.1

Duplicate of #1938


0.2.1 hasn't been released to Hackage

0.2.1 has a release here, dated 2022-04-11: https://github.com/fugue/goldplate/releases/tag/v0.2.1
But it seems it has been forgotten to push it to Hackage.
I'd say current master can be released to Hackage as 0.2.1 and the tag can be moved.  The only difference seems to be the bump to aeson: https://github.com/fugue/goldplate/compare/0.2.1...master

Candidate: https://hackage.haskell.org/package/goldplate-0.2.1/candidate

If moving the tag isn't a good idea, we can also release <IP_ADDRESS> (PVP even allows <IP_ADDRESS>).


ci: move from prettier to forge fmt

Motivation
This pull request is to address the issue outlined here
Change Summary
The changes in this pull request:

add fmt configurations for forge
apply forge fmt to source files
add fmt as a pre-commit hook and to github ci
remove prettier packages

Merge Checklist
Choose all relevant options below by adding an x now or at any time before submitting for review

[x] The PR title adheres to the conventional commits standard
[x] The PR has been tagged with change type label(s) (i.e. documentation, feature, bugfix, or chore)
[x] The PR's changes adhere to all the requirements in the contribution guidelines
[x] This PR does not require changes to the Farcaster protocol

Additional Context
N/A

The CI is failing one of the tests.
I cloned the main repo and ran the forge test on the main branch and it's displaying the same error as the CI 🤔

Thanks for the PR, that seems unrelated so I'll merge this in for now and handle that later


feat(RHTAPREL-358): remove releasestrategy field

This change removes the releasestrategy field from collect-data and from pipelines using this tasks. In addition, extraData is not collected anymore and data is collected instead. This change is triggered by a release-service PR containing both of those changes.

This change has to be on hold until the release-service change is merged.

Merging with broken tests as we have a cycle dependency (it depends on the release-service API change which also depends on this tasks through E2E). @johnbieren agreed on this as well.


(fix) fix pyproject.toml

Summary
Fix tool.setuptools.packages.find field in pyproject.toml. Otherwise in local build mode with pip install ., python system fails to locate liger_kernel.

@wizyoung I was thinking I became dumb after midnight yesterday as I faced this issue 😄


[Off-Canvas and Sticky] Sticky Header and Off-Canvas Menu Functionality Issues

Using Foundation 6 v6.2.3, my site has a header that uses the Sticky plugin and Off-Canvas for the menu.
Test case:
http://codepen.io/darinronne/pen/RGybxk
In Edge, Chrome and Firefox:
Issue 1

Toggle the menu open.
Slowly start to scroll down. You will see the top bar jump to the left, creating a gap between the top bar and the off-canvas menu.
When you scroll back to the top, the top bar will shift back to the right, closing the gap.

Issue 2

Toggle the menu open.
Scroll down the page so that the top bar is no longer in view.
Click on the content to close the off-canvas menu.
The top bar will reappear, but it will be shifted to the left.

Issue 3
Firefox is not jumping to the top of the page when the off-canvas is opened.
In IE 10+:
Issue 4

Toggle the menu open.
Start to scroll down. The top bar stays sticky. In Edge, Chrome and Firefox the top bar scrolls with the page.
Close the off-canvas menu (when further down the page). The top bar will stay shifted to the left, and won't correct itself until you scroll back to the top.

Issue 5

Start further down the page, then toggle the off-canvas open. The page does not jump to the top. Also the top bar does not shift over. It fills the entire width of the browser and overlaps the off-canvas menu.
Scroll to the top of the page. The top bar will shift to the left where it should be.

What did you expect to happen?
I expected the sticky header with off-canvas menu to work consistently across browsers.
I would think it should work like this:

Toggle open off-canvas menu.
The page jumps to top with the menu open. The top bar is in the correct position and scrolls with the page.
Closing the off-canvas menu returns the top bar to its starting position.

Thanks.

Off canvas and sticky should now play nicely for 6.3, as off canvas has been rewritten.
Release candidate is out today, with the full release scheduled for the 15th December.
Closing this, but if you still have issues with 6.3 please re-open.


Update available TLS ciphers to include all available ciphers in Go.

I had a need to use specific TLS ciphers but found many ciphers were missing support in Traefik. This PR is simply intended to add all available TLS ciphers defined in Go 1.7.

@timoreimann Absolutely. The documentation for this can be found for latest Go version at https://golang.org/pkg/crypto/tls/#pkg-constants (displays 1.8 values) . This changes as Go is updated so for a more "permalink" reference it may be better to reference the source code in GitHub:

Go 1.7 TLS Ciphers https://github.com/golang/go/blob/release-branch.go1.7/src/crypto/tls/cipher_suites.go#L269-L285
Go 1.8 TLS Ciphers https://github.com/golang/go/blob/release-branch.go1.8/src/crypto/tls/cipher_suites.go#L369-L390

I would be happy to update this for 1.8 as well, which has some nice additions.

@timoreimann I also added the ciphers for 1.8. See #1276.

@kekoav could you rebase and resolve the merge conflicts?

@kekoav any chance for an update?

I tried to fix this myself but ended up making things worse. :-(
Reverted back to the previous state. @kekoav, we need your help.

Seems be a duplicate of #1276

Closing this PR in favor of #1276.


Compile error with ns:  "unknown global in strict mode: ns"

I'm getting compile error when trying to evaluate namespace definition.
(ns examples.core         // This doesn't work!
  (:gen-class))

 (+ 1 2) => 3           // This works

Throws below error!
; eval (current-form): (ns examples)
; Compile error in /home/shinto/Shinto/Personal/Learn/Development/Clojure/clojure-by-example/app/examples/src/examples/cor
;   unknown global in strict mode: ns
;
;   (:gen-class))
;
; * Try looking to see if there's a typo.
; * Try using the _G table instead, eg. _G.ns if you really want a global.
; * Try moving this code to somewhere that ns is in scope.
; * Try binding ns as a local in the scope of this code.

Running  :ConjureCljSessionFresh  gives
; Sponsored by @ketansrivastav _
; --------------------------------------------------------------------------------
; localhost:44444 (connected): app/examples/.nrepl-port
; --------------------------------------------------------------------------------
; localhost:44444 (disconnected): app/examples/.nrepl-port

I am trying to connect to the nREPL server running inside a container at port 44444.
$ telnet localhost 44444
Trying <IP_ADDRESS>...
Connected to localhost.
Escape character is '^]'.
Connection closed by foreign host.

The session refresh and telnet logs suggest the session is getting disconnected, I guess! What might be the reason, I can clearly see the nREPL server running in headless mode.
I was trying to use docker to bring up a readymade developer environment for Clojure with NeoVim. It's not ready yet, it can be found here https://github.com/cshintov/clojure-by-example. Might need a little bit of work to make it run on other than my system. I've hard coded uid/gid in Dockerfile, etc.

Hmm this looks like you're trying to evaluate Fennel/Lua within a Clojure buffer. Maybe your filetype is fennel or your Conjure config is telling it to eval Fennel code from .clj files?

I have seen a very similar problem in evaluating a Clojure namespace.

The code is from clojure project created using lein. It might be the second scenario. How do I fix it?
I didn't do any additional configuration. Just ran nrepl server in the container and started evaluating.
One interesting thing I were able to evaulate (+ 1 2) even without running the nrepl server.

(+ 1 2) will work fine since it's evaluating your Clojure code as Fennel and (+ 1 2) works in either language. The error is a clue, it's a Lua error, all of this is Lua related.
; * Try looking to see if there's a typo.
; * Try using the _G table instead, eg. _G.ns if you really want a global.
; * Try moving this code to somewhere that ns is in scope.
; * Try binding ns as a local in the scope of this code.

So you're not evaluating as Clojure and I'm not sure why 🤔 do you have syntax highlighting enabled so Neovim knows what language you're working with?
Can you reproduce the same error with the built in babashka auto REPL or a Clojure CLI nREPL or even one from running lein repl? If it works fine with those then we know it's the REPL but I suspect it's some Neovim configuration, but I'm completely guessing as to what setting could be doing this. I have no idea right now. It could be extra Conjure config, it could be filetype things or even to do with syntax highlighting.
Conjure has no reason to eval clojure as fennel unless it is told to really, do you have any config for Conjure around swapping filetypes or some extra config for fennel / clojure to improve highlighting?

I have extra config, I'm new to Clojure and I'm just starting out using it with Neovim. So it's all default config.
I came to know the existence of fennel through this plugin actually. And never worked with Lua. So I don't know what's causing neovim/conjure to think a ".clj" file as Lua/Fennel.
Is there anyway to debug this, anywhere conjure logs or even inspect its the current config?

I was trying to avoid installing clojure/lein on the host and use the container instead. But now I did install them on the host and it is working without any issues. So it's not Conjure/Neovim, rather my setup that's causing the problem!

I'm kinda stumped on why it would select the wrong language 🤔 Conjure doesn't try to work out the language, it just trusts the "filetype" which is determined by the neovim config. The only debug tooling is let g:conjure#debug = v:true but that won't show you how it decides the language really, might have soooome hints?
If you see it again, I'd like to see what :set filetype displays, that's the only thing I can think to check really.

@cshintov I suspect this is an issue with the file paths in the Docker container being different to your host.
See g:conjure#path_subs


[FEATURE] ML Model Access Controls (ml-commons plugin)

What are you proposing? In a few sentences, describe the feature and its core capabilities.
We are building model access controls for the ml-commons plugin, so that our users can govern who can perform actions on each ML model that is managed by our plugin.
Which users have asked for this feature? (research, proposals, requests or anecdotes. Include links to GitHub Issues, Forums, Stack Overflow, Twitter, Etc.)
We’ve proactively decided to add this feature because we believe security is a job zero priority.
What is the developer experience going to be? Does this have a REST API? If so, please describe the API and any impact it may have to existing APIs. In a brief summary (not a spec), highlight what new REST APIs or changes to REST APIs are planned. as well as any other API, CLI or Configuration changes that are planned as part of this feature.
The planned solution will resemble the same design used by the alerting and anomaly detection plugins to support resource-level access controls. This design revolves around attaching backend roles to users and model groups, which are collections of versions of a particular model. The design is described in further details in the description about the user experience. In short, users will be able to access the models that share a common backend role.
Here is the documentation for the anomaly detection plugin, which provides examples of how the security plugin APIs can be used to configure backend roles for detectors. Our plan is to support the same design by allowing users to create backend roles in the same way by using the same APIs. We will add APIs within the ml-commons plugins to create, modify and delete model groups, which will include the ability to associate backend roles to these model groups.
Are there any security considerations? What is the security model of the new APIs? Features should be integrated into the OpenSearch security suite and so if they are not, we should highlight the reasons here.
This feature enhances security.
Are there any breaking changes to the API? If Yes, what is the path to minimizing impact? (example, add new API and deprecate the old one)
No
What is the user experience going to be?
The user experience will be the similar to what we provide in other plugins like anomaly detection and alerting. For example, the anomaly detection plugin’s process is configure detector-level access controls is described here.
The high-level flow to limit a user’s ability to access specific models and define what actions they can perform on those model's as an admin is as follows:

[Already Supported] Configure a role that limits the permissions the user has to perform specific actions on models. This involves configuring permissions to invoke the specific model management APIs within the ml-commons plugin below. For instance, one might create a model builder role that allows a user to get model information, search and invoke a model or a MLOps role that provides permissions to invoke all these APIs.

https://opensearch.org/docs/latest/ml-commons-plugin/api/#get-model-information
https://opensearch.org/docs/latest/ml-commons-plugin/api/#load-model
https://opensearch.org/docs/latest/ml-commons-plugin/api/#unload-a-model
https://opensearch.org/docs/latest/ml-commons-plugin/api/#search-model
https://opensearch.org/docs/latest/ml-commons-plugin/api/#delete-model
https://opensearch.org/docs/latest/ml-commons-plugin/api/#predict

[Already Supported] The admin can then create a backend role and associate roles to these backend roles. A natural way to configure the backend role is to represent an organizational group. For instance, you might create a backend role to represent a data science team within your organization and associate the model builder role to that backend role.

[New Functionality] If a model group doesn’t already exist to manage a collection model versions, a user who has the required permissions can create a model group. One or more backend roles can be associated to a model group. By default, the user that creates the model group is designated as the owner and this user’s backend roles will be attached to the model by default. The user has the option to specify a subset of backend roles. This provides the model owner access to the models that they create or import into an OpenSearch cluster. An admin type user will have the ability to reassign the model owner and change the access controls as needed.

Once the above steps are setup, a user will be limited to model API operations on the models with a common backend role. Specifically, the expected logic used to govern a user’s access is as follows:

A user has access to all the model versions within a model group if they share one more or more backend roles with the model group.

A user is authorized to perform actions on a model group based on the aggregate permissions. For instance, if a user shares two backend roles with a model group where one provides search permissions and the other provides deployment permissions, the user will be able to perform both search and deployment actions on the models.

If the user is directly associated with a role that provides permissions to perform ml-common model API operations, they will also be authorized to perform those actions. For instance, if a user has the permissions to delete models through a role that is directly associated with a user, they will be able to perform delete operations on any model group that shares a common backend role with the user. In other words, the user is authorized to perform the aggregate permissions provided by all the roles associated with common model groups and through the roles they’re directly associated with.

In summary, backend roles are used to filter what model groups a user could perform actions on. What actions a user can perform on these model groups are the aggregate permissions from all the associated roles. This logic is consistent with the design used by other plugins. However, there are limitations which have been outlined in the open questions section.
Are there breaking changes to the User Experience?  Will this change the existing user experience? Will this be a breaking change from a user flow or user experience perspective?
No
What will it take to execute? Are there any assumptions you may be making that could limit scope or add limitations? Are there performance, cost, or technical constraints that may impact the user experience? Does this feature depend on other feature work? What additional risks are there?
Additional security features come at the cost. Additional access control checks will be required and the overhead can increase API execution time.
Any remaining open questions? What are known enhancements to this feature? Any enhancements that may be out of scope but that we will want to track long term? List any other open questions that may need to be answered before proceeding with an implementation.
As described, a user has access to a model as long as they share one or more backend roles with the model. We’ve selected this policy because it is consistent with other plugins and simple to support. However, we would like to know the demand to support more advanced policies.
We are aware of certain limitations with the current design. For instance, it’s difficult to decrease a user’s scope of access at scale because it becomes difficult for admins to track a large number of roles and permissions and how they’re associated to models and users.
Secondly, it’s difficult to model the access controls for large organizations. For instance, if I wanted to control access based on teams and job responsibilities, the current design requires the admin to manage approximately #teams X #job responsibilities backend roles. For instance, I would need to model backend roles like TeamA-ModelBuilder and TeamA-MLOps.
If we had the ability to attach backend roles to allow or disallow access, we could model the backend roles like Team A, Team B, ModelBuilder and MLOps and attach the some combination of backend roles associated with teams and job responsibilities to each model group. This would enable large organization to define backend roles in a more scalable and intuitive way.
If there are users that have more advanced requirements, we like to know more about it.

This feature released in 2.9 https://opensearch.org/docs/latest/ml-commons-plugin/model-access-control/


[SQL Lab] Copy partition to clipboard doesn't work in left SQL Lab panel

The action "Copy partition to clipboard" on the left side of SQL Lab doesn't work.
Expected results
record is in clipboard
Actual results
nothing (old value remains, even though bottom of the page says "copied to clipboard!"
Screenshots

How to reproduce the bug

Go to 'SQL Lab'
In left panel select schema and table
Click down arrow to see details of the table
Click on "copy" next to latest partition info
See error

Environment
(please complete the following information):

superset version: master

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

cc @yousoph

Any updates here yet?

@junlincc Can you assign this to me?

@etr2460 Is someone from Airbnb already working on this? I just found out that I'll need to install all the backend infrastructure (Presto or Hive) to simulate the partitions. If you already have this environment working it will be fairly easy to fix the bug.
In TableElement.jsx:
renderWell() {
    const { table } = this.props;
    let header;
    if (table.partitions) {
      let partitionQuery;
      let partitionClipBoard;
      if (table.partitions.partitionQuery) {
        ({ partitionQuery } = table.partitions);
        const tt = t('Copy partition query to clipboard');
        partitionClipBoard = (
          <CopyToClipboard
            text={partitionQuery}
            shouldShowText={false}
            tooltipText={tt}
            copyNode={<i className="fa fa-clipboard" />}
          />
        );
      }
...

I simulated on the client side a table with partitions and partitionQuery and the copy and paste is working.

@michael-s-molina no one in airbnb was working on this. Thanks!

Github issue triaging:: bug still there (latest screenshot)

Assigning to @michael-s-molina since you were interested way back when. This probably needs re-validation by now.

Assigning this to @betodealmeida who may have the necessary infrastructure already installed.

CC @bkyryliuk and @nytai who also have some Presto/Hive chops, if I'm not mistaken.

Closing this as stale since it's been silent for so long, and we're trying to steer toward a more actionable Issues backlog. If people are still encountering this in current versions (currently 3.x) please re-open this issue, open a new Issue with updated context, or raise a PR to address the problem. Thanks!


Investigate JDK8/JRE8 Alpine Action Failure

The Github Action for JDK/JRE8 for alpine is failing
This is due to Alpine:latest container moving to 3.21, which causes issues for JDK/JRE8
This PR pins the dockerfile to 3.19.

The error in the GHA shows as this.. which appears to be because of a dependency no longer present in Alpine 3.21,  this dependency is only required at build time, and is related to the apk build command. This doesn't impact the installation of a JDK built on an older version, deployed on Alpine 3.21 ) tested in an 3.21 alpine container.
>>> ERROR: temurin-8-jdk*: create_apks failed
>>> ERROR: temurin-8-jdk: rootpkg failed

> Task :jdk:alpine:packageJdkAlpine FAILED
FAILURE: Build failed with an exception.

* Where:
Build file '/home/runner/work/installer/installer/linux/jdk/alpine/build.gradle' line: 104

* What went wrong:
Execution failed for task ':jdk:alpine:packageJdkAlpine'.
> Process 'command 'docker'' finished with non-zero exit value 1

* Try:
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Exception is:
org.gradle.api.tasks.TaskExecutionException: Execution failed for task ':jdk:alpine:packageJdkAlpine'.


Switch to current pywin32 package

pypiwin32 is deprecated
This might affect some recent CI failures

Thanks, merging!


[Peek] WindowActivationState checks are added for focus and close aft…

…er losing focus.

Summary of the Pull Request

PR Checklist

[ ] Closes: #26366
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

Do we having a tracking issue for this

Do we having a tracking issue for this

A new issue opened and added to PR.

I'll be adding a Setting for enabling/disabling this behavior.

I'll be adding a Setting for enabling/disabling this behavior.

I am also in favor of adding a Setting light dismiss. It can be very frustrating especially since it is not a window's normal behavior.

I've added a setting for this, off by default:

Also added a Service to the Peek.UI to get the settings, now that some settings influence it.


Title : Run EvaDB on Postgres

Summary : Made changes to provide an option that would run EVA on top of Postgres server instead of SQLite. Created a CircleCI Job to validate the changes. Currently the job is only run for Python 3.10, we can add more configurations if needed.
Also fixed the UTs that were failing on CircleCI runs in default master.
Test Plan : UT
Reviewers : @jarulraj @gaurav274

Circle CI link : https://app.circleci.com/pipelines/github/Chitti-Ankith/eva/19

Overall LGTM! Fix the linter issue and address a minor comment. Thanks!

Thanks for the review @gaurav274 !


Reduce bundled size

This should have no impact on functionality, only bundle size. Based on my local testing, the current bundle is ~2.32kb min+gzip. With these changes it is ~2.11kb min+gzip. There are a plenty more opportunities for reducing the size, but I didn't want to rename any input parameters/css variables or remove any log messages since I assumed you wanted to keep those (but based on my testing, if in addition to these changes, you also remove the warning about particle counts greater than 300, it would be ~1.9kb min+gzip)
I tried to only make changes that didn't have a very negative impact on readability.. but you might disagree and should of course feel free to ignore this PR if you prefer the code as currently written.

This is not necessarily ready to merge.. I need to do a bit more testing to ensure I didn't miss anything.

Thanks for this! Im working on an action version of this. These changes will help immensely in that


retry logic for SQL transactions

Hi,
we are experiencing a lot of 4xx (404, 401) from mobile app / curl clients on key uploads in the past weeks
Client is getting in about 50% of the cases:
{“error”:“Internal Server Error”,“code”:“internal_error”}
# or
{"error":"error loading health authority config: authorizedapp: failed to read a.b.c from database: get authorized app: starting transaction: write failed: write tcp <IP_ADDRESS>:47284-\u003e<IP_ADDRESS>:5432: write: connection reset by peer","code":"unable_to_load_health_authority"}

Eventually it always ends up with a 200 success, but the error rate is very high.
with very minimal load (15 db connections maximum)
server is logging these
db=main,user=notification LOG: could not receive data from client: Connection reset by peer

Even though it looks like some network between Cloud Run and CloudSQL might be a bit laggy / unreliable sometimes, from this code I assume there is no retry logic implemented. We are currently implementing retry logic on clients and I'll try to debug the CloudSQL connection but it'd be helpful if the server had some kind of retry logic as well. Wdyt?
https://github.com/google/exposure-notifications-server/blob/d9c3fd849921fc7d54126f2e7952b45154504dda/internal/authorizedapp/database/authorized_app.go#L152
Thanks!

Can you either update past this commit or set DB_POOL_HEALTH_CHECK_PERIOD to 1m on your services?
We believe this should precent the connection resets.

I'll try to set the ENV var. Thanks for the quick response!

looks like there are 2 items that need to be set
https://github.com/google/exposure-notifications-server/commit/b1c4a0d9e3db0091a2246360c11af56a7ec18a2c

see that commit, you can set those manually or upgrade and get the defaults

We cut release v0.5.1 that contains all of these settings as default.
Please let us know if this is still occurring after you’ve updated past that point. Thanks!

@mikehelmick thanks for the support, after setting everything you proposed, it seems like we stopped getting 4xx errors due to this healthcheck issue (we started getting them for different reason but that should be solved too). We'll be upgrading soon past the v0.5.1 so that should be resolved too.
Thanks again!

/close


[profiler] Add memray profiler to compaction

This PR adds memray profiler in compaction.

overal, LGTM

LGTM. Could you add some of the profiling output to your PR description?


Update support.md for EF 6.5

relates to https://github.com/dotnet/ef6/issues/2237

@AndriySvyryd FYI

Thanks for your contribution!


Null when copying with string constant after editing the string constants in the library properties

JabRef version
Our development version
Operating system
GNU / Linux
Details on version and operating system
No response
Checked with the latest development build (copy version output from About dialog)

[X] I made a backup of my libraries before testing the latest development version.
[X] I have tested the latest development version and the problem persists

Steps to reproduce the behaviour

I ran the program with idea's run
I paste the example entry by the user's issue
I manually input the string constant
4 . I copy the entry by clicking on it and Ctrl-C
5, Pasting it into an editor and the null occur
Rerun with the idea run, by closing the window and clicking the run again
Copying and paste works fine now
Add another dummy string constant
Null appears again

Appendix
No response

The parsedSerilization is null when setting the content.

After adding on string constant, the copy result in
null@Article{Claudepierre,
  author  = {Claudepierre, S. G.},
  journal = grl,
}

Closed by #14


s3import: Import images and metadata into single collection

WIPP recently updated their backend to support addition of metadata to an image collection (issue #60 from WIPP-backend). The s3import plugin should be updated to import metadata along with images. Currently, there is an option to import either images or metadata, but this can now be done in a single step instead of generating two separate image collections for images and metadata.

We will be deprecating this plugin.


view crashes after open

Since BC <IP_ADDRESS> views don't work anymore. In my example, I'm extending the "Job List" and try to add a view which only filters on Status = Completed. The other on a boolean "Canceled"
After clicking the view, the Webclient crashes and needs a restart. When I'm on my custom page, the view work fine. Is this a bug or could it be a wrong definition?
Here is my code:
pageextension 50000 "LOGPJJob List" extends "Job List" //89
{
    views
    {
        addlast
        {
            view(LOGPJCompleted)
            {
                Caption = 'Completed';
                Filters = where(Status = Const(Completed));
            }
            view(LOGPJCanceled)
            {
                Caption = 'Canceled';
                Filters = where(LOGPJCanceled = Const(true));
            }
        }
    }
}

I BC <IP_ADDRESS> the views worked fine.

@SarahBirrer, I cannot completely re-use the snippet you provided because I don't know how the field LOGPJCanceled is defined, so I had to remove this filter. However, publishing this code worked fine for me and I did not experience any crash:

As requested by our template for Github issues, can you provide us with:

the version of the AL extension that you are using,
the version of BC that you are testing against,
a self contained project that can be used to reproduce the issue?

@qutreson
The Code of the Field "LOGPJCanceled" is:
        field(50000; LOGPJCanceled; Boolean)
        {
            Caption = 'Canceled';
            DataClassification = CustomerContent;
            trigger OnValidate()
            begin
                if LOGPJCanceled = true then
                    Validate(Status, Status::Completed);
            end;
        }

I'm using tAL Lanugage v5.0.320540
the version of BC that you are testing against,
a self contained project that can be used to reproduce the issue?
the version of BC that you are testing against,
a self contained project that can be used to reproduce the issue?

@SarahBirrer, thank you for your answer, but I am still not able to reproduce the issue.
Here is the code that I have used based on your previous comments:
pageextension 50100 "LOGPJJob List" extends "Job List"
{
    views
    {
        addlast
        {
            view(LOGPJCompleted)
            {
                Caption = 'Completed';
                Filters = where(Status = Const(Completed));
            }
            view(LOGPJCanceled)
            {
                Caption = 'Canceled';
                Filters = where(LOGPJCanceled = Const(true));
            }
        }
    }
}

tableextension 50100 MyExtension extends Job
{
    fields
    {
        field(50000; LOGPJCanceled; Boolean)
        {
            Caption = 'Canceled';
            DataClassification = CustomerContent;

            trigger OnValidate()
            begin
                if LOGPJCanceled = true then
                    Validate(Status, Status::Completed);
            end;
        }
    }
}

This is how the view 'Completed' looks like:

This how the view 'Canceled' looks like:

When navigating from one view to the other, or when navigating back to the default view 'All', I do not experience any crash.
Can you verify this scenario on a build coming from our latest developer previews to see if the issue reproduces there?
If that still reproduces, can you share a video recording or some screenshots of the issue experienced in the web client?

This issue has been closed because it needs more information and has not had recent activity. Please open a new issue if you believe this has not been resolved and reference the current issue.


[CSL-2221] Fix OS X installer path check

Now UpdaterData is algebraic data type with different constructors for Windows and OS X. I think that approach brings more clarity to the purpose of OS dependent files. All required paths check for that config have been Implemented.
Also, a hash check of downloaded (but yet not installed) update has been implemented in the launcher.
I don`t know what to do if pattern matching on OS name or fromJust failed (It should not). In a current implementation, it throws an error, but it should not be in production, of course. Universum 1.0.1 has a bug function for that case, but It seems you want 0.9.0.

@Dolgakov what will this code do if ran on linux?

@cleverca22 Now It will just skip an update routine. It seems there is no updater implementation for Linux (At least for Daedalus).

@Dolgakov i just finished testing the update installing process on linux, and now need to test downloading

Closed because this PR is just too old.


Double click not working on safari ios

You can reproduce also it in the demo page of your website, the "cell edit" example using "dbclick".
On Safari on IOS the double tap does not work, so the editing does not start.
Maybe related to a know safari problem ? -> https://www.google.com/search?q=safari+double+tap&oq=safari+double+tap

Having Same issue.


Feature Request: Sticky Headers

Would be nice to make the header sticky, so that during the scrolling a user can see which date he picked. Otherwise great job!!!

We could add one, but then with default styles it would look like this. As you can see, when using React Native's Animated API, they move with a significant delay. This could be resolved with the help of react-native-gesture-handler and react-native-reanimated, but libraries are simply too big to consider adding them. If only there were a way to check if these libraries are already present in the project… 🤔
I will add this feature in v2 if this performance is sufficient for you.

I Made a workaround currently. I added an absolute view and gave the timeContainer an opacity


Display

Here are some things it would be nice to have on the pendant display

[ ] #3
[ ] Metric vs imperial indicator
[ ] Current height
[ ] Target height
[ ] Move delta
[ ] Move speed (Fast vs slow)
[ ] Mode indicator (Jog vs Position)
[ ] Preset (number in the corner?)
[ ] Tool Offset
[ ] Step Size

Position will involve first setting the desired position and then hitting a run button. Jog will directly couple the encoder to height. Instant adjustment. So instead of showing target height show "Mode: Jog".


Would you consider to extend Augmentor in 3D image segmentation?

Hi, would you consider to extend Augmentor in 3D? I work in the biomedical image segmentation. In this field, we usually meet 3-D MRI, CT images. While, there is no convenient tool to augment 3D images for the segmentation tasks. I just used your tool, I feel very easy to use.
Best.

@Limingxing00 you could use TorchIO which is specifically focussed on 3D medical image augmentation

Thank you! It looks interesting.


Expectations for with, not adding display precision for Time and DateTime

Based on #331, #416, and f9ee8f9, I would expect that a call made to with comparing a Time or DateTime to output the same display precision as a non-with test would. Unfortunately it doesn't. :disappointed:
Here's an example:
context 'test' do
  class Fake; def blah(time); end; end

  it 'fails nicely' do
    expect(Time.now).to eq Time.now
  end

  it 'fails poorly' do
    expect_any_instance_of(Fake).to receive(:blah).with(Time.now)
    Fake.new().blah(Time.now)
  end
end

And the output:
1) test fails nicely
   Failure/Error: expect(Time.now).to eq Time.now

     expected: 2015-02-19 00:09:28.123861000 +0000
          got: 2015-02-19 00:09:28.123845000 +0000

     (compared using ==)

     Diff:

   # ./spec/fake_spec.rb:10:in `block (5 levels) in <top (required)>'

2) test fails poorly
   Failure/Error: Fake.new().blah(Time.now)
     #<Fake:0x007fb3f8f395b8> received :blah with unexpected arguments
       expected: (2015-02-19 00:09:28 +0000)
            got: (2015-02-19 00:09:28 +0000)
   # ./spec/fake_spec.rb:20:in `block (5 levels) in <top (required)>'

I'd love to provide a PR for this, but can't find the seam to insert this code. If someone could point me in the right direction, that'd be awesome. :smile:

This is definitely something we should fix, and thanks for offering to work up a PR!
There are some other known situations where we should show the increased precision -- see rspec/rspec-expectations#611.
To use the same formatting logic in rspec-mocks and rspec-expectations, it'll need to go in rspec-support. The differ has already been moved there.  I think we should extract the format_object method from eq:
https://github.com/rspec/rspec-expectations/blob/08c11431771b7755d2a402746b0fe4f8aec8d4ff/lib/rspec/matchers/built_in/eq.rb#L38-L48
...and move that into some rspec-support helper method.  Then we should go through rspec-expectations and rspec-mocks and find all places where we use inspect and consider changing to that new helper method.  This is one of those little things that will make a big difference to users but can be a bit tedious, so if you wanted to take that on, that would be fantastic :).

Started on this work, and I'm finding myself having to write many tests like this set in built_in/eq_spec.rb. If someone has a suggestion on how to test that the OutputFormatter is being used more simply, that would be greatly appreciated! :smile:


Validate column order when creating Hive tables

Partition columns for Hive tables are always at the end because Hive works that way. When creating a table, the creation should fail if the columns are not in the final order (having CREATE TABLE silently rearrange columns is quite unexpected).
We should use a new error code for this: HIVE_COLUMN_ORDER_MISMATCH
This should fail because the partition column is listed first, but will be at the end in the final table:
CREATE TABLE test (
  ds varchar,
  x bigint)
WITH (partitioned_by = ARRAY['ds']);

This should fail because the partition columns are listed in a different order than in the partition specification:
CREATE TABLE test (
  x bigint,
  type varchar,
  ds varchar)
WITH (partitioned_by = ARRAY['ds', 'type']);

It's a good feature.

What about CREATE TABLE AS SELECT? Should they fail as well?


Is it possible to support async layout?

It's currently possible to crash react-virtualized by setting a very high number of rows (eg 1,000,000,000) and then dragging the scrollbar quickly so that millions of rows are passed by at once. In this case react-virtualized will try to just-in-time measure all of the rows being scrolled past and it will likely fail.
The only solution to this, I think, requires changing the layout algorithms to support async operations. I have not thought about this much so I don't have a sense of how much work this would be. In theory though, if this were done, it may be possible to perform some of the heavy layout crunching in a web worker and further speed up scrolling smoothness.
Filing this away for further thinking. It's likely pretty complicated and I don't have the time to tackle it at this moment.

Hi @bvaughn, have you consider also the use of throttling here?
If the layout algorithm is not fast enough to complete in less than 16ms per frame, maybe it's better to drop some calculations and only stick to the last one.
The problem I see for moving to a worker is that you'll need to keep both UI thread and worker in sync. So here you'd have the same problem plus a bit overhead for the serialization of data. So I can imagine the situation of UI sending lots of layout tasks to the worker and the worker not being able to complete them in the right time. By the moment worker sends back the results, they are no longer valid for the UI, as it'll be far ahead from that position by then.
Anyway this is just hypothesis. It'd be interesting to implement a small prototype with both approaches. I can help you if you want 😉. From my experience, moving from a sync world to an async one is a pita.
What do you think?

I have not put much thought into throttling, no. But it does seem like something worth considering! :)
Agreed the async could be a PITA.
This issue was mostly just opened as a placeholder for people to toss out ideas. Thanks for adding some thoughts. :)

Closing this for now because I don't foresee myself having time to pick it up in the near future and having open issues on the issues list isn't a good feeling. May pick it back up at some point. :)


PowerShellExecutionPolicy doesn't allow multiple 'CurrentUser' assignments

Problem description
Powershell 5.1 DSC doesn't allow you to specify the same ExecutionPolicy for ExecutionPolicyScope 'CurrentUser' but different PsRunAsCredentials, meaning you can't assign it to multiple specific users.
Verbose logs
Test-ConflictingResources : A conflict was detected between resources '[PowerShellExecutionPolicy]Builder (::8::5::PowerShellExecutionPolicy)' and '[PowerShellExecutionPolicy]Publisher
(::9::5::PowerShellExecutionPolicy)' in node 'localhost'. Resources have identical key properties but there are differences in the following non-key properties: 'PsDscRunAsCredential'.
Values 'System.Management.Automation.PSCredential' don't match values 'System.Management.Automation.PSCredential'. Please update these property values so that they are identical in both
cases.
At line:289 char:9
+         Test-ConflictingResources $keywordName $canonicalizedValue $k ...
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [Write-Error], InvalidOperationException
    + FullyQualifiedErrorId : ConflictingDuplicateResource,Test-ConflictingResources
Compilation errors occurred while processing configuration 'FailSample'. Please review the errors reported in error stream and modify your configuration code appropriately.
At C:\Windows\system32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.psm1:3917 char:5
+     throw $ErrorRecord
+     ~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (FailSample:String) [], InvalidOperationException
    + FullyQualifiedErrorId : FailToProcessConfiguration

DSC configuration
{
  Param([Parameter(Mandatory)][PSCredential]$Builder, [Parameter(Mandatory)][PSCredential]$Publisher)
  Import-DscResource -ModuleName ComputerManagementDsc
  Node localhost
  {
    PowerShellExecutionPolicy Builder { ExecutionPolicy='RemoteSigned'; ExecutionPolicyScope='CurrentUser'; PsDscRunAsCredential = $Builder }
    PowerShellExecutionPolicy Publisher { ExecutionPolicy='RemoteSigned'; ExecutionPolicyScope='CurrentUser'; PsDscRunAsCredential = $Publisher }
  }
}

Sample -Verbose -ConfigurationData:@{AllNodes=@(@{NodeName='localhost'; PsDscAllowPlainTextPassword=$true})}

Suggested solution
Add PowerShellUserExecutionPolicy which doesn't take scope and requires a credential.
PowerShellUserExecutionPolicy Builder
{
  ExecutionPolicy = 'RemoteSigned' ; Credential = $builder
}
PowerShellUserExecutionPolicy Publisher
{
  ExecutionPolicy = 'RemoteSigned' ; Credential = $publisher
}

Operating system the target node is running
OsName               : Microsoft Windows 10 Pro
OsOperatingSystemSKU : 48
OsArchitecture       : 64-bit
WindowsVersion       : 2009
WindowsBuildLabEx    : 19041.1.amd64fre.vb_release.191206-1406
OsLanguage           : en-US
OsMuiLanguages       : {en-US}

PowerShell version and build the target node is running
PSVersion                      5.1.19041.1682
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.19041.1682
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

ComputerManagementDsc version
Name                  Version Path
----                  ------- ----
ComputerManagementDsc 8.5.0   C:\Program Files\WindowsPowerShell\Modules\ComputerManagementDsc\8.5.0\ComputerManage
mentDsc.psd1

In my case, we already have machine and user policies, these two users have a need to specifically allow differently policies unique to their two accounts. Not sure what the point of specifying a run-as credential is, if using different credentials doesn't change who 'CurrentUser' is.

Would it be possible to make a composite resource that adds a property Credential as key property. The credential property could then maybe be added to PsDscRunAsCredential? 🤔

(Just playing catch up after a very hectic "holiday", heh)
That's not possible because Credential just isn't part of the underlying separators, so the composite would simply cause the problem - at least, my attempts to make one seemed to bear this out.


Jest is collecting coverage on manual mocks

Should the files in mocksPattern be excluded from coverage?

I don't think it makes sense to exclude them. They are code, not tests.
tbh I don't quite understand why people want to exclude tests from coverage in the first place, but it seems to be a pretty big deal to some. Every test file should have 100 % coverage and if not, that's a good signal that something in a test file isn't being executed properly. But oh well, I do what the majority demands :)


Clear error after send button disabled and enchance error catches

Closes #462
Also included a very obscure catch wherby the send button is disabled if there is and amount in the invoice but it does not equal amountSats() invoice().amount_sats !== amountSats() unsure if/how this would ever occur as having the ability to edit the amount is only possible if there is no amount set in the invoice. If it seems unneccessary I'm happy to remove it
This video below is only possible because of some altered logic isAmountEditable={true}
Kazam_screencast_00003.webm

@benalleng I wanted to clean up some of this logic and figured it would be easiest to just commit on top of your pr because I'm cleaning up my own mess.
basically it was working to run side effects during the recomputation of a reactive value like const insufficientFunds = () => but it's confusing / hard to reason about. it meant that multiple "reactive" things were fighting over the state of error().  now it should be clearer which of these are effectful functions vs reactive values.
would appreciate you checking this over to make sure we're still catching all the same errors we want to be. it was working for me with on-chain and amountless invoice so I feel pretty good about it

@futurepaul tack! Went through and it resets properly. reached each possible error and then cleared upon changing the send amount. using a createEffect makes a lot more sense! Good call.
How did you make an amountless invoice for signet?

@benalleng you can do this to get an amountless invoice
curl -X POST https://faucet.mutinynet.com/api/bolt11 -H 'Content-Type: application/json' -d '{}'


How to Improve the Efficiency of Face Landmarker Detection on Android Systems.Thank you.

Hi, I am using your Face Landmarker Detection project on a Huawei Honor V8 model, running the demo you provided, in LIVE_STREAM mode with setDelegate(Delegate.GPU), and trying to process 256 video frames, with a frame size of 480*640, the total time taken is about 27 seconds. The total time consumed is about 27 seconds, which is an average of 106 milliseconds per frame. How can I improve his efficiency so that he can give the result faster and the total time consumed can be less than 15 seconds?
    private fun returnLivestreamResult(
        result: FaceLandmarkerResult,
        input: MPImage
    ) {
        if (result.faceLandmarks().size > 0) {
            position++
            val finishTimeMs = SystemClock.uptimeMillis()
            val inferenceTime = finishTimeMs - result.timestampMs()

            timeCount += inferenceTime

            if (position == 256) {
                Log.d(TAG, "256 the total frame elapsed time is $timeCount"
                + "，the average time taken was ${timeCount / 256}")
            }
            faceLandmarkerHelperListener?.onResults(
                ResultBundle(
                    result,
                    inferenceTime,
                    input.height,
                    input.width
                )
            )

        } else {
            faceLandmarkerHelperListener?.onEmpty()
        }
    }

Probably just not use mediapipe and go with Tensorflow directly.
You just need two models and really just one if you can detect fast yourself.
That will spare you from all the unnessarily complexity the graph, calculators and particular dependencies and as a bonus will make your binaries smaller.


Looking forward to get involved and contribute to the community 🎉

First Name
Pulkit
Last Name
Singh
Discord Username (if applicable)
Pulkit Singh#8120
Let us understand your motive.
Just to interact and work more with communities :)

Hey @PulkitSinghDev 👋🏻
Thanks for requesting an invite! Welcome to SidHacks Community 🎉
Before going ahead, I would want you to complete the below To-Do list.
Your To-Do List ✔

Accept the invitation sent to you via mail.
Change the appearance of this organization to "Public" from the people list here.
Join our Discord Server here.

Accept the Rules from #📓〡rules channel.
Introduce yourself to our community in 👋〡introduction channel.

Feel free to share your excitement on socials with #SidHacksCommunity. You can also tag me @siddharth_hacks.

Close this issue once you complete all the above points.

Looking forward to interacting with you in our community! Welcome once again! 🤓


[DOC] Broken link to search-analyzers

What do you want to do?

[ ] Request a change to existing documentation
[ ] Add new documentation
[x] Report a technical problem with the documentation
[ ] Other

Tell us about your request.
https://opensearch.org/docs/latest/analyzers/search-analyzers/ is linked from at least two places in the documentation, such as here, but the link just goes to the documentation home.

Weird, I just clicked that link again at it works now...

I guess it was just a me problem. Seems to work now. Closing this issue.


Client revenue flaky rspec test fixes

Notion card
https://www.notion.so/saeloun/Fix-the-failing-flaky-tests-df90a065c21d4d0bb3bcb328dd127102
Summary

Fixed the flaky test on the client revenue controller
Refactored  invoices spec
Changed "during" scope of invoice model from "due_date" to "issue_date"

Preview

Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to
not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Checklist:

[x] I have manually tested all workflows
[x] I have performed a self-review of my own code
[x] I have added automated tests for my code

@swaathi GitHub CI is passing. Need your approval to merge this PR.

@abinashpa Please resolve conflicts


Fix readthedocs build

Google is sending people here instead of GitHub pages, so it should be kept up to date if possible.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 58.79%. Comparing base (9c8d0c2) to head (a494d5e).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #195   +/-   ##
=======================================
  Coverage   58.79%   58.79%
=======================================
  Files         108      108
  Lines       19840    19851   +11
=======================================
+ Hits        11664    11672    +8
- Misses       8176     8179    +3

Flag
Coverage Δ

unittests
58.79% <ø> (+<0.01%)
:arrow_up:

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

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

Instead, set up redirects from readthedocs to github pages :+1:


Create ISSUE_TEMPLATE.md

Adding an issue template to help user writing relative issue

Codecov Report

Merging #4 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #4   +/-   ##
=======================================
  Coverage   95.25%   95.25%
=======================================
  Files          62       62
  Lines         969      969
  Branches      258      258
=======================================
  Hits          923      923
  Misses         40       40
  Partials        6        6

Continue to review full report at Codecov.

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


Update to newest version of Angular

Should update the project to the newest version of Angular.

Fixed in: https://github.com/mathisGarberg/angular-folder-structure/pull/100


Limit number of cats that can appear in an adopter/foster/cat dropdown. 5 most recent?

(if this hasn't already been done, of course)

whatever the limit is decided... do this for ALL associations, if we do it

if you do this, display number of associations on the card (so its not misleading that list maxes out at 5)


Canvas upgrade11

I did some quick testing, but it could use more.

@jisqyv
I did some quick testing, but it could use more.

@halatmit I would suggest not incrementing the component version number again. There are no projects that have seen version 10.
-Jeff

I've backed out the version increment on my copy and am testing it now.

@jisqyv
I removed the extra versiosn upgrade.  WARNING: I did not test this.

LGTM, MERGED


Update Move Contract Display

This is a merge of current main with @Andrew47 's draft PR for issue #1043 .
Now, Move contracts should display like this:

To test, start the explorer and visit the stdlib:
http://localhost:3000/objects/0000000000000000000000000000000000000002

original PR:
https://github.com/MystenLabs/sui/pull/1535