Archive succeeded, exit status 1 with error, no obvious reason

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description

build_app ends with "Archive Succeeded" but then error with "Exit status: 1". I tried capturing the output of bundle exec fastlane beta using the --capture_output flag, but I got nothing on my clipboard. I'm copy pasting the last few lines of the output, but I see no obvious errors here.
Complete output when running fastlane, including the stack trace and command used

▸ Generating 'MyApp.app.dSYM'
▸ Copying /Users/apoorv/Library/Developer/Xcode/DerivedData/MyApp-cldopslihickokfuhwjgczqbidqw/Build/Intermediates.noindex/ArchiveIntermediates/MyApp/BuildProductsPath/Release-iphoneos/MyApp\ Today\ Extension.appex
skipping copy phase strip, binary is code signed: /Users/apoorv/Library/Developer/Xcode/DerivedData/MyApp-cldopslihickokfuhwjgczqbidqw/Build/Intermediates.noindex/ArchiveIntermediates/MyApp/IntermediateBuildFilesPath/UninstalledProducts/iphoneos/MyApp Today Extension.appex/MyApp Today Extension
▸ Running script 'Run Script'
▸ Running script '[CP] Embed Pods Frameworks'
▸ Running script '[CP] Copy Pods Resources'
▸ Touching MyApp.app (in target: MyApp)
▸ Archive Succeeded
ERROR [2018-11-13 13:27:45.00]: Exit status: 1
+-------------+-------------------------+
|           Build environment           |
+-------------+-------------------------+
| xcode_path  | /Applications/Xcode.app |
| gym_version | 2.108.0                 |
| sdk         | iPhoneOS12.1.sdk        |
+-------------+-------------------------+
INFO [2018-11-13 13:27:45.08]: ▸ Build system information
INFO [2018-11-13 13:27:45.08]: ▸ warning: Swift 3 mode has been deprecated and will be removed in a later version of Xcode. Please migrate "SwiftURLMatcher" to Swift 4.2 using "Convert > To Current Swift Syntax…" in the Edit menu. (in target 'SwiftURLMatcher')
INFO [2018-11-13 13:27:45.08]: ▸ Build system information
INFO [2018-11-13 13:27:45.08]: ▸ warning: Swift 3 mode has been deprecated and will be removed in a later version of Xcode. Please migrate "SwiftURLMatcher" to Swift 4.2 using "Convert > To Current Swift Syntax…" in the Edit menu. (in target 'SwiftURLMatcher')
INFO [2018-11-13 13:27:45.08]:
ERROR [2018-11-13 13:27:45.08]: ⬆  Check out the few lines of raw xcodebuild output above for potential hints on how to solve this error
WARN [2018-11-13 13:27:45.08]: 📋  For the complete and more detailed error log, check the full log at:
WARN [2018-11-13 13:27:45.09]: 📋  /Users/apoorv/Library/Logs/gym/MyApp-MyApp.log
INFO [2018-11-13 13:27:45.09]:
ERROR [2018-11-13 13:27:45.09]: Looks like fastlane ran into a build/archive error with your project
ERROR [2018-11-13 13:27:45.09]: It's hard to tell what's causing the error, so we wrote some guides on how
ERROR [2018-11-13 13:27:45.09]: to troubleshoot build and signing issues: https://docs.fastlane.tools/codesigning/getting-started/
ERROR [2018-11-13 13:27:45.09]: Before submitting an issue on GitHub, please follow the guide above and make
ERROR [2018-11-13 13:27:45.09]: sure your project is set up correctly.
ERROR [2018-11-13 13:27:45.09]: fastlane uses xcodebuild commands to generate your binary, you can see the
ERROR [2018-11-13 13:27:45.09]: the full commands printed out in yellow in the above log.
ERROR [2018-11-13 13:27:45.10]: Make sure to inspect the output above, as usually you'll find more error information there
INFO [2018-11-13 13:27:45.10]:
WARN [2018-11-13 13:27:45.10]: Lane Context:
INFO [2018-11-13 13:27:45.10]: {:DEFAULT_PLATFORM=>:ios, :PLATFORM_NAME=>:ios, :LANE_NAME=>"ios beta"}
ERROR [2018-11-13 13:27:45.10]: Error building the application - see the log above
INFO [2018-11-13 13:27:45.11]: Successfully generated documentation at path '/Users/apoorv/Code/MyApp-iphone/fastlane/README.md'
+------+------------------+-------------+
|           fastlane summary            |
+------+------------------+-------------+
| Step | Action           | Time (in s) |
+------+------------------+-------------+
| 1    | default_platform | 0           |
| 💥   | build_app        | 708         |
+------+------------------+-------------+
ERROR [2018-11-13 13:27:45.14]: fastlane finished with errors
bundler: failed to load command: fastlane (/Users/apoorv/.rbenv/versions/2.5.1/bin/fastlane)
FastlaneCore::Interface::FastlaneBuildFailure: [!] Error building the application - see the log above
/Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane_core/lib/fastlane_core/ui/interface.rb:163:in build_failure!'   /Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane_core/lib/fastlane_core/ui/ui.rb:17:in method_missing'
/Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/gym/lib/gym/error_handler.rb:73:in handle_build_error'   /Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/gym/lib/gym/runner.rb:97:in block in build_app'
/Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane_core/lib/fastlane_core/command_executor.rb:89:in execute'   /Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/gym/lib/gym/runner.rb:93:in build_app'
/Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/gym/lib/gym/runner.rb:18:in run'   /Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/gym/lib/gym/manager.rb:19:in work'
/Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane/lib/fastlane/actions/build_ios_app.rb:64:in run'   /Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane/lib/fastlane/runner.rb:253:in block (2 levels) in execute_action'
/Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane/lib/fastlane/actions/actions_helper.rb:50:in execute_action'   /Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane/lib/fastlane/runner.rb:232:in block in execute_action'
/Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane/lib/fastlane/runner.rb:227:in chdir'   /Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane/lib/fastlane/runner.rb:227:in execute_action'
/Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane/lib/fastlane/runner.rb:157:in trigger_action_by_name'   /Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane/lib/fastlane/fast_file.rb:149:in method_missing'
Fastfile:21:in block (2 levels) in parsing_binding'   /Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane/lib/fastlane/lane.rb:33:in call'
/Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane/lib/fastlane/runner.rb:49:in block in execute'   /Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane/lib/fastlane/runner.rb:45:in chdir'
/Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane/lib/fastlane/runner.rb:45:in execute'   /Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane/lib/fastlane/lane_manager.rb:56:in cruise_lane'
/Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane/lib/fastlane/command_line_handler.rb:36:in handle'   /Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane/lib/fastlane/commands_generator.rb:108:in block (2 levels) in run'
/Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/commander-fastlane-4.4.6/lib/commander/command.rb:178:in call'   /Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/commander-fastlane-4.4.6/lib/commander/command.rb:153:in run'
/Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/commander-fastlane-4.4.6/lib/commander/runner.rb:476:in run_active_command'   /Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane_core/lib/fastlane_core/ui/fastlane_runner.rb:76:in run!'
/Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/commander-fastlane-4.4.6/lib/commander/delegates.rb:15:in run!'   /Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane/lib/fastlane/commands_generator.rb:333:in run'
/Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane/lib/fastlane/commands_generator.rb:41:in start'   /Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/fastlane/lib/fastlane/cli_tools_distributor.rb:108:in take_off'
/Users/apoorv/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/fastlane-2.108.0/bin/fastlane:23:in <top (required)>'   /Users/apoorv/.rbenv/versions/2.5.1/bin/fastlane:23:in load'
/Users/apoorv/.rbenv/versions/2.5.1/bin/fastlane:23:in `<top (required)>'

Environment

✅ fastlane environment ✅
Stack

Key
Value

OS
10.13.6

Ruby
2.5.1

Bundler?
true

Git
git version 2.17.2 (Apple Git-113)

Installation Source
~/.rbenv/versions/2.5.1/bin/fastlane

Host
Mac OS X 10.13.6 (17G3025)

Ruby Lib Dir
~/.rbenv/versions/2.5.1/lib

OpenSSL Version
OpenSSL 1.0.2p  14 Aug 2018

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
10.1

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL
en_US.UTF-8
✅

LANGUAGE

fastlane files:
`./fastlane/Fastfile`
# This file contains the fastlane.tools configuration
# You can find the documentation at https://docs.fastlane.tools
#
# For a list of all available actions, check out
#
#     https://docs.fastlane.tools/actions
#
# For a list of all available plugins, check out
#
#     https://docs.fastlane.tools/plugins/available-plugins
#

# Uncomment the line if you want fastlane to automatically update itself
# update_fastlane

default_platform(:ios)

platform :ios do
  desc "Push a new beta build to TestFlight"
  lane :beta do
    build_app(workspace: "MyApp.xcworkspace", scheme: "MyApp")
  end
end

`./fastlane/Appfile`
app_identifier("com.MyApp.MyAppMobile") # The bundle identifier of your app
<EMAIL_ADDRESS># Your Apple email address

itc_team_id("REDACTED") # App Store Connect Team ID
team_id("REDACTED") # Developer Portal Team ID

# For more information about the Appfile, see:
#     https://docs.fastlane.tools/advanced/#appfile

fastlane gems

Gem
Version
Update-Status

fastlane
2.108.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.2.0

bundler
1.17.1

CFPropertyList
3.0.0

concurrent-ruby
1.1.3

i18n
0.9.5

minitest
5.11.3

thread_safe
0.3.6

tzinfo
1.2.5

activesupport
4.2.10

public_suffix
2.0.5

addressable
2.5.2

atomos
0.1.3

babosa
1.0.2

claide
1.0.2

fuzzy_match
2.0.4

nap
1.1.0

cocoapods-core
1.5.3

cocoapods-deintegrate
1.0.2

cocoapods-downloader
1.2.2

cocoapods-plugins
1.0.0

cocoapods-search
1.0.0

cocoapods-stats
1.0.0

netrc
0.11.0

cocoapods-trunk
1.3.1

cocoapods-try
1.1.0

colored2
3.1.2

escape
0.0.4

fourflusher
2.0.1

gh_inspector
1.1.3

molinillo
0.6.6

ruby-macho
1.3.1

nanaimo
0.2.6

xcodeproj
1.7.0

cocoapods
1.5.3

cocoapods-prune-localizations
0.3.1

colored
1.2

highline
1.7.10

commander-fastlane
4.4.6

declarative
0.0.10

declarative-option
0.1.0

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20180417

dotenv
2.5.0

emoji_regex
0.1.1

excon
0.62.0

multipart-post
2.0.0

faraday
0.15.3

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

faraday_middleware
0.12.2

fastimage
2.1.4

jwt
2.1.0

memoist
0.16.0

multi_json
1.13.1

os
1.0.0

signet
0.11.0

googleauth
0.6.7

httpclient
2.8.3

mime-types-data
3.2018.0812

mime-types
3.2.2

uber
0.1.0

representable
3.0.4

retriable
3.1.2

google-api-client
0.23.9

json
2.1.0

mini_magick
4.5.1

multi_xml
0.6.0

plist
3.4.0

rubyzip
1.2.2

security
0.1.3

naturally
2.2.0

simctl
1.6.5

slack-notifier
2.3.2

terminal-notifier
1.8.0

unicode-display_width
1.4.0

terminal-table
1.8.0

tty-screen
0.6.5

tty-cursor
0.6.0

tty-spinner
0.8.0

word_wrap
1.0.0

rouge
2.0.7

xcpretty
0.3.0

xcpretty-travis-formatter
1.0.0

generated on: 2018-11-13

I redacted some sensitive details here.

@fastlane-bot I have, although it isn't apparent that this issue is related to code signing.

I was able to figure this out. Apparently gym was failing because it did not have write permissions for the directory, where it looks like build logs are written to.
~/Library/Logs/gym
I was able to chmod that directory to set the appropriate permissions, but IMO this should not be an error that causes the build to fail and fastlane to stop. Just my 2 cents.
Also, the statement in the logs where I found that the permissions were messed up was formatted like a warning, and not an error, which made things more confusing, since I wasn't seeing any errors and was really confused by the exit with error.


Add revealed event

Add revealed event decoding and unit tests
Smart contract from PDS for revealed event https://github.com/flow-hydraulics/flow-pds/blob/main/cadence-contracts/IPackNFT.cdc#L27

@joshuahannan  can you merge this if it's good to go? I'm not authorized


How to handle multiple cookie Authentication schemes?

Hi, I have different cookie auth schemes to login application users vs customers who use 2 different SPA applications and oAuth clients. I keep them also separate having 2 different asp identity stores and services.
IdentityServer by default uses the default cookie scheme or the one set in the options.
Is there a way to use multiple schemes and so we can support issuing tokens for both the application users and the customers ?

Important update
This organization is not maintained anymore besides critical security bugfixes (if feasible). This organization will be archived when .NET Core 3.1 end of support is reached (3rd Dec 2022). All new development is happening in the new Duende Software organization.
The new Duende IdentityServer comes with a commercial license but is free for dev/testing/personal projects and companies or individuals making less than 1M USD gross annnual revenue. Please get in touch with us if you have any question.


Code and Refactor Subcommands

Description:
The current codebase lacks modularity, making it challenging to maintain and extend. The goal is to modularize the code by extracting smaller functions into a commonly used module. Additionally, we should refactor the main module to only include functions corresponding to subcommands, enhancing code organization and readability.
Tasks:

[ ] Identify commonly used functions and move them to a separate module.
[ ] Refactor the main module to retain only functions related to subcommands.
[ ] Update import statements and ensure all dependencies are resolved.

Hey @mAmineChniti, I would like to work on this. Can you please assign it to me?

@omibo done.


Only sanity check SQL connection etc. if necessary

If the user has set elected to skip installing SQL databases then do not worry about:

testing for a SQL connection
validating the UNC path
any other SQL specific parameters or tests that are strictly necessary for installing databases

I need to keep the SQL connection validation in place, this remains necessary because the new Sitecore instance is not guaranteed to use the same data access login as other instances. So, even though the databases have already been created, we may need to update the user mappings.


Handle sprite and display resolution change

Codecov Report

Merging #85 (bfcaa8f) into main (ad8af84) will increase coverage by 0.63%.
The diff coverage is 48.01%.

@@            Coverage Diff             @@
##             main      #85      +/-   ##
==========================================
+ Coverage   51.76%   52.39%   +0.63%
==========================================
  Files          74       47      -27
  Lines        1988     1735     -253
  Branches      220      161      -59
==========================================
- Hits         1029      909     -120
+ Misses        954      826     -128
+ Partials        5        0       -5

Impacted Files
Coverage Δ

...ter-demo/src/seagulls/space_shooter/_space_rock.py
48.33% <23.07%> (-3.67%)
:arrow_down:

...er-demo/src/seagulls/space_shooter/_ship_button.py
37.50% <28.12%> (-3.68%)
:arrow_down:

...-demo/src/seagulls/space_shooter/_shooter_scene.py
47.42% <28.57%> (-1.47%)
:arrow_down:

...c/seagulls/space_shooter/_replay_shooter_button.py
43.43% <29.41%> (-2.91%)
:arrow_down:

...e-shooter-demo/src/seagulls/space_shooter/_ship.py
42.85% <37.50%> (+1.80%)
:arrow_up:

...r-demo/src/seagulls/space_shooter/_di_container.py
74.74% <50.00%> (+0.01%)
:arrow_up:

...emo/src/seagulls/space_shooter/_game_over_scene.py
52.50% <50.00%> (+0.50%)
:arrow_up:

...rc/seagulls/space_shooter/_asteroid_missed_rule.py
66.66% <60.00%> (ø)

...oter-demo/src/seagulls/space_shooter/_blue_ship.py
66.66% <60.00%> (+2.38%)
:arrow_up:

...o/src/seagulls/space_shooter/_game_over_overlay.py
66.66% <60.00%> (-3.34%)
:arrow_down:

... and 36 more

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 ad8af84...bfcaa8f. Read the comment docs.


Publishing Timed Out!

I encounter the error "Publishing Timed Out! - The server timed out while trying to publish your changes" for all my Katas when I press the "Re-Publish" button.

I confirm the issue, I experienced the same this morning. I even tried it via the direct server, but it's the same...

Looks like they're publishing now :-)


chore(deps): upgrade jenkins-x/jx3-versions to version 1.1.28

from: https://github.com/jenkins-x/lighthouse.git

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign jstrachan
You can assign the PR to them by writing /assign @jstrachan in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


new dataset upload bug fix

Fixes bug that does not allow upload of new datasets to GCP
Corrects imports in CL
some minor typos

I'm going to go ahead and merge this


Task - Duplicate task displayed after assigning task to another user & skeleton loading

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.58-0
Reproducible in staging?: Y
Reproducible in production?: Y
**If this was caught during regression testing, add the test name, ID and link from TestRail:**N/A
Issue reported by: Applause - Internal Team
Action Performed:

Go to https://staging.new.expensify.com/
and log in
Open any chat
Create a task
Open the task details and assign the task to the user you share the task with
Navigate to the chat and open the task again

Expected Result:
Only the system message that the task was assigned to a user is displayed
Actual Result:
The duplicate task is displayed below the system message, after opening the task again the skeleton loading is displayed
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/926169bc-c7bc-400b-bc9f-3c824c40b7e2
View all open jobs on GitHub

@stephanieelliott  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #vip-vsp

PR is under active review

New PR is on staging! I will update with payment dates once this hits prod

@stephanieelliott how can I help here?

Oh sorry @twisterdotcom it looks like my little note didn't make it on here:

Reapplying the Bug label to get another BZ member on this while I am OOO til May 2. Thanks @twisterdotcom!

Heads up this is a regression from https://github.com/Expensify/App/pull/39215, so no payment is actually due for this one. So the only thing left to do here is create an issue for the regression test (assuming @nexarvo determines we should create one) and then close this issue!


chore(npm): Bump npm version for toolchain

This project uses semver, please check the scope of this pr:

[x] #patch# - backwards-compatible bug fix
[ ] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

Changes Unknown when pulling cbca5eb42caf19fab98bb36738bd73e60ee77fb5 on article/ubiquits-for-the-angular-developer into ** on master**.

Changes Unknown when pulling cbca5eb42caf19fab98bb36738bd73e60ee77fb5 on article/ubiquits-for-the-angular-developer into ** on master**.


Adding slack communities for frontend and backend

added two slack communities
Frontend Mentor community for frontend
HackerEarth Developer community for both frontend and backend

Looks good 👍🏻


bower resolutions problem

For avoid dependencies conflict, add a "resolutions" section at bower.json, after "dependencies" section like this for avoid resolutions interactive challenge when installing bower dependencies:
"resolutions": {
    "angular": "1.3.11",
    "select2": "~3.5.2"
}

I chose the last stable angular release, but you can keep the 1.2.28 version. If you also chose to upgrade, ping me for help in fix the post update bugs.

@ayr-ton I'm trying apply your fix adding a "resolutions" hash using angular 1.3.11 but I'm having this following error at dashboard:
[Error] Error: [$injector:modulerr] Failed to instantiate module miningApp due to:
[$injector:modulerr] Failed to instantiate module miningApp.auth due to:
undefined is not an object (evaluating '$httpProvider.responseInterceptors.push')
http://localhost:8888/assets/app/scripts/auth/auth.js:45:39

I'm googling it and i found this answer http://stackoverflow.com/a/21269115
Anyway, can you help me?

@avelino @ayr-ton
my last PR and this commit ab8db0992bac7f0b8ff4f8ec2a64f1178d40c775 will fix this issue. ;)


Fix regression with mariadb-version arg, fixes #1517

The Problem/Issue/Bug:
#1517: Sadly, ddev config --mariadb-version 10.1 creates a config.yaml with the dbimage...10.2... and it shouldn't even have a dbimage at all.
How this PR Solves The Problem:
In the midst of config spaghetti, this makes sure the default dbimage in config handling gets set right.
Manual Testing Instructions:
Oh, this has to be tested against every regression in its sad history.
But

ddev config --mariadb-version 10.1 should work with the .ddev/config.yaml showing the right stuff (and not having a dbimage spec)
ddev config --mariadb-version 10.2 should do the same
Manually changing to 10.1 or 10.2 in the config.yaml should work
Manually setting the dbimage for 10.1 or 10.2 should work

Automated Testing Overview:

Related Issue Link(s):
Release/Deployment notes:
I'm not sure if it's worth getting this into v1.7.0 or just telling people not to use ddev config --mariadb-version for this release. I'm not entirely sure that anybody uses it.
It's clear that interacting specifications give us enormous pain and we'll have to figure out a coherent approach to them.

Wow, so sad. I'm pretty sure that this PR addresses #1517 correctly, but in appropriately testing it you've discovered yet another round of #1374, which is truly annoying. It's obvious that we need better tests and some work on design.
I think what I'll do is add more testing to this one to address the specific problems with the ddev config action and confusion. Then I'll probably reopen #1374 or open another issue for it to chase the NewApp() problems.

Thanks for preventing a descent into chaos @andrewfrench , would love to have you look again. So many permutations. The only good thing is that correctness of ddev config behavior can  be determined just by looking at the resulting config.yaml, doesn't require a start. Correctness of the NewApp() behavior can be determined on start and depends only on the config.yaml.
We need to avoid interdependent config at all costs. I wonder how we're going to do that. I wonder if we ever needed mariadb_version.


bug: collision check fails if category has changed

we should exclude category from the collision check, since the current ML solution is inconsistent and it is a user-editable field

category is not currently included in the hash, so something else is causing duplicate transactions to be allowed. requires more investigation


Don't show tooltips with empty text

The dev can make a template that return an empty string. In that case there is no need to show the tooltip.
https://github.com/nnnick/Chart.js/pull/823

I am +1

We've recently announced a feature complete status for v1.x and will only be accepting bug fixes for v1.x. Be sure to take a look at v2.0-dev and it's new features that are either finished or currently under development.  If you see your feature is still not implemented, feel free to submit another PR to the v2.0-dev (or later) branch for your feature.


tonic options max_decoding_message_size/max_encoding_message_size

Currently its not possible to define tonic options to change max_encoding_message_size and max_decoding_message_size
I redefined them in KvClient and WatchClient (I didn't add max_encoding_message_size for watch client as I don't see any use case for it)
let etcd_client = etcd_client::Client::connect(["localhost:2379"], None).await.unwrap();

let etcd_kv_client = etcd_client
  .kv_client()
  .max_encoding_message_size(10* 1024 * 1024);
  .max_decoding_message_size(10* 1024 * 1024);

let etcd_watch_client = etcd_client
  .watch_client()
  .max_decoding_message_size(10* 1024 * 1024);

LGTM, thanks!


How to use css variables in themed primitives?

I'm struggling to find documentation on how to use light/dark mode color variables with primitives.
I can change the theme OK using the <Theme> props - this adds the correct styles to any theme components, type, form elements etc.
In the docs for primitives the example styles use hard-coded color variables that do not update with the theme change.
For example in <Popover>...
.PopoverTrigger {
  background-color: white;
  color: var(--violet-11);
  box-shadow: 0 2px 10px var(--black-a7);
}

What is the recommended way to change these color variables so that they use the themed colors?
Any help appreciated! :)

var(--violet-11) in light and dark mode does point to different values. Are you asking about using, say, var(--violet-11) in light mode, but var(--green-5) in dark mode?

Hey @vladmoroz - thanks for the response. maybe I didn't explain very well :)
In this example (Link)[https://www.radix-ui.com/primitives/docs/components/popover]:

The CSS provided sets the background as background-color: white;
What variables are available that could be used so that the background would be light in light mode and dark in dark mode.
Something like var(--background-primary) maybe? Is it somewhere in the docs do you know?

Gotcha, and to clarify, is this about Radix Primitives (unstyled components) or is this about Radix Themes usage (a component library with pre-defined styles)?

The theme colors work fine for components included in the theme component library... it's when building a custom component (or primitive) where I want to use a theme-responsive color :)

I guess I'd expect color tokens something like...
.my-thing {
  background: var(--background-primary);
  color: var(--text-primary);

  &:hover {
    color: var(--color-link);
  }
}

I'm guessing they exist but I can't find them...

I see, check out how Radix Themes colors are defined:
https://github.com/radix-ui/themes/blob/main/packages/radix-ui-themes/src/styles/tokens/color.css
You’d normally use something like:

--color-background and --color-panel for background colors
--accent-1 through --accent-12 for your accent color
--gray-1 through --gray-12 for grays
Transparent variants of the above like --gray-a5 or --accent-a8 for components that need to blend with the background

More colors, like red, green, yellow, etc., are also available directly, see Color docs
Will take a not to improve the docs for this area.


uim: disable qt support

https://github.com/NixOS/nixpkgs/pull/174634 removed qt4 but enabled the (previosly disabled) qt5, breaking the build.
Disable QT5 again by disabling qt all together.
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

cc @ericsagnes (do you still maintain this package?)

can we maybe fix the qt5 support by adding wrapQtAppsHook?

can we maybe fix the qt5 support by adding wrapQtAppsHook?

@Atemu could it be that this is why some users experience this qt5 support as broken?

I'll clarify the comment but I won't pursue actually fixing the package.
I have no idea how it works or how I'd even test it works.
All I know is that it transitively breaks mlterm and why.

I too have no idea what is the purpose of this package. Secondly, before https://github.com/NixOS/nixpkgs/commit/308baf304984fb5bb94f0b69f4b4dc8d5030405d , qt5 support was never enabled, and for sure it did not work since forever. However It is not that hard to fix it, and I would also say that its not accurate to say that qt5 support is broken in uim. Hence I propose this change:
diff --git i/pkgs/tools/inputmethods/uim/default.nix w/pkgs/tools/inputmethods/uim/default.nix
index 70d3e8e8725c..2ad8e60e8e92 100644
--- i/pkgs/tools/inputmethods/uim/default.nix
+++ w/pkgs/tools/inputmethods/uim/default.nix
@@ -44,6 +44,8 @@ stdenv.mkDerivation rec {

     ruby # used by sigscheme build to generate function tables
     librsvg # used by uim build to generate png pixmaps from svg
+  ] ++ lib.optionals withQt5 [
+    qt5.wrapQtAppsHook
   ];

   buildInputs = [
@@ -52,7 +54,7 @@ stdenv.mkDerivation rec {
   ++ lib.optional withAnthy anthy
   ++ lib.optional withGtk2 gtk2
   ++ lib.optional withGtk3 gtk3
-  ++ lib.optionals withQt5 [ qt5.qtbase.bin qt5.qtbase.dev ]
+  ++ lib.optionals withQt5 [ qt5.qtbase qt5.qtx11extras ]
   ++ lib.optional withLibnotify libnotify
   ++ lib.optional withSqlite sqlite
   ++ lib.optionals withNetworking [

We can apply the above while still disabling qt5 support by default, while removing the comment near it - because it is not correct, and the truth is that for no particular reason we disable qt5 support by default.

@doronbehar feel free to open a PR on that. As I mentioned, I'm not taking ownership of qt support in uim, I'm just restoring it to the status quo.

It's just that our approaches a bit conflicting, and my solution will also return things to their status quo.. I hope you won't mind, but I pushed here my change in addition to yours, along with a change to the comment.

I'm not a fan of adding dead code that I don't know what it does. Could you open your own PR for that?

I'm not a fan of adding dead code that I don't know what it does. Could you open your own PR for that?

I can just merge this one :)

I did test that the dead code compiles successfully, observed that binaries with -qt5 are added to result/bin, so I think it is good enough for the moment.

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

3 packages built:

mlterm
mlterm-wayland
uim

For the record^.


Set multiple cookies in one response

Hi.
I was struggling with setting multiple cookies in one response. I think the root cause is add_default_headers function from mist/internal/http module. In the process of converting list to to dict you lose the same keys (in my case set-cookie).
This updated version seems to work, but I am not sure if it doesn't have any other consequences
pub fn add_default_headers(
  resp: Response(BytesBuilder),
) -> Response(BytesBuilder) {
  let body_size = bytes_builder.byte_size(resp.body)

  let headers = [
    #("content-length", int.to_string(body_size)),
    #("connection", "keep-alive"),
    ..resp.headers
  ]

  Response(..resp, headers: headers)
}

version: 0.17.0

I obviously had some misconceptions about valid headers, and honestly after reading the RFC... I'm not particularly more confident 😅
It seems like maybe there are some headers where it's valid to send multiple with the same name?  But I can't really find information about which exactly those are... it does almost seem like most things just allow you to specify multiple.
I'm gonna try to do some more digging to find concrete answers, but at the very least only allowing one in all cases definitely seems wrong.
So thank you!  I'll address this one way or the other.


fix issue #813

Please submit your pull requests to the 'dev' branch.
When submitting a pull request, please make sure you have completed the following checklist:

[x ] Your code is flake8 compliant (Dojo's code isn't currently flake8 compliant, but we're trying to correct that)
[x ] If this is a new feature and not a bug fix, you've included the proper documentation under the /docs folder

@devGregA What do you think about this one as well?

I haven't had a chance to test. I'm a little worried that we don't have CI
tests around JIRA.
This PR makes a lot of sense to me. A couple things I'm confused on:
Overwriting status as is done in the PR doesn't make sense to me
(views.py). Status is a method that is constructed based on data about the
finding (models.py).
Callable checks if an object is null. I'm not sure what is being caught in
the else that isn't caught by the previous elif.
OH maybe it's being overwriting because otherwise we're losing the
transition from active to non-active. Would be great if we could just get
some insight into the thinking.
Greg Anderson
DefectDojo Project Leader
<EMAIL_ADDRESS>On Tue, Feb 5, 2019 at 3:14 PM Aaron Weaver<EMAIL_ADDRESS>wrote:

@devGregA https://github.com/devGregA What do you think about this one
as well?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DefectDojo/django-DefectDojo/pull/833#issuecomment-460806293,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEhYwGKX3mc0DNyVPXp627_TcCRZYfoZks5vKfRSgaJpZM4ZcWdZ
.

Hi Greg,
Thanks for taking a look into this.
Considering this part of the code belongs to the “closing” status part, I’ve just added the status that is set after you “close” the finding. There’s probably a better way to do this. However, I’m not familiarized with the code so I wasn’t sure how to do it.
product_tab = Product_Tab(finding.test.engagement.product.id, title="Close", tab="findings")
# HERE - lines below added to update the finding status in jira (when it's closed)
finding2 = finding
finding2.status = list(('Mitigated','Inactive','Verified'))
update_issue(finding2, old_status, True)
# END
Regards,
Alejandro
From: Greg Anderson<EMAIL_ADDRESS>Sent: 05 February 2019 21:43
To: DefectDojo/django-DefectDojo
Cc: Mendiondo, Alejandro; Author
Subject: Re: [DefectDojo/django-DefectDojo] fix issue #813 (#833)
I haven't had a chance to test. I'm a little worried that we don't have CI
tests around JIRA.
This PR makes a lot of sense to me. A couple things I'm confused on:
Overwriting status as is done in the PR doesn't make sense to me
(views.py). Status is a method that is constructed based on data about the
finding (models.py).
Callable checks if an object is null. I'm not sure what is being caught in
the else that isn't caught by the previous elif.
OH maybe it's being overwriting because otherwise we're losing the
transition from active to non-active. Would be great if we could just get
some insight into the thinking.
Greg Anderson
DefectDojo Project Leader
<EMAIL_ADDRESS>On Tue, Feb 5, 2019 at 3:14 PM Aaron Weaver<EMAIL_ADDRESS>wrote:

@devGregA https://github.com/devGregA What do you think about this one
as well?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DefectDojo/django-DefectDojo/pull/833#issuecomment-460806293,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEhYwGKX3mc0DNyVPXp627_TcCRZYfoZks5vKfRSgaJpZM4ZcWdZ
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/DefectDojo/django-DefectDojo/pull/833#issuecomment-460815339, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AhCwxjEET7-4i6Zyp9tqgwhC0VdXEti_ks5vKfrmgaJpZM4ZcWdZ.
This email and any attached files are confidential and may be legally privileged. It is intended solely for the addressee. Access to this email by anyone else is unauthorised. If you are not the addressee, any disclosure, reproduction, copying, distribution, or other dissemination or use of this communication is strictly prohibited. If you have received this transmission in error please notify the sender immediately and then delete the email. Alternatively you can call us. A full list of contact numbers is available on https://www.kbc.ie/contact-us
Email transmission cannot be guaranteed to be secure or error free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or contain viruses. The sender therefore does not accept liability for any errors or omissions in the contents of this message, and shall have no liability for any loss or damage suffered by the user, which arise as a result of email transmission. If verification is required please request a hard copy version.
KBC Bank Ireland plc is regulated by the Central Bank of Ireland. KBC Bank Ireland plc is a company registered in the Republic of Ireland, Company Number 40537. Registered Office: Sandwith Street, Dublin 2, Ireland.


"go mod tidy" error from sermo repo

I was looking to update the repo to work with the "tensorflow" package that has been renamed to "ts" one of the dependency projects but ran into this:

Clone the repo.
go mod tidy

You get a git "invalid version" error because something is trying to pull in<EMAIL_ADDRESS>and the repo seems to be private or deleted.

@scottmcmaster,
Thanks for your report. Please check out newly commit with go.mod clean-up.


Move repo to Dracula's official org

Hello @ichi0g0y!
I'm the creator of Dracula color scheme. I was searching for themes created by the community and I found yours. Good job by the way!
I was wondering if you are interested in transferring this repository to the official organization (https://github.com/dracula). You'll continue maintaining this project, the only difference is that we can make it more visible and include it on the site (https://draculatheme.com).
What do you think?

Hello @ zenorocha.
Previously, I tried to move the repository.
There is that conversation in # 3.
Github 's repository transfer is fine, but I do not know how to transfer APM.
Once you know the procedure, I'd like to join quickly :)

Awesome!
I sent you an invitation to join the organization, once you accept that we can continue moving forward.

Thank you, i am joined now.

Ok, now go to https://github.com/ichi0g0y/dracula-ui/settings and select Transfer.
Then, apply this template (https://github.com/dracula/template) which includes README, LICENSE, and .github.

You can also rename the repo to atom-ui.

Github's transfer is complete.
What should I do with APM?

Remove dracura-ui from my APM repository for now and Would you like to register to APM from Dracula team?

I think Atom's registry just uses the GitHub organization, so you can still publish it with your user in the same way.

Try releasing a new version to see what happens.

i'm tried apm publish minor, how about it?

Hey @ichi0g0y, it's because the theme name is set to atom-ui in the package.json file. If you might need to rename it dracula-ui.

Hi @JimmyMultani, i tried apm --rename dracula-ui. but an error occurred -> Creating new version failed: A package by that name already exists. i deleted old repository dracula-ui. but it seems that it still exists.

Hey @ichi0g0y, I moved it over for you. :)

nice work! thank you!

Close issue?


Cart Rule Issue

I created a cart rule to enable free shipping if cart price > 100 but when i enable this rule this is applied on all items even cart price is less than 100

Hi @devansh-webkul  i am getting this error when i enable this rule for guest users

Even i am getting this error on login after updating CartRule.php

@arooba499
can you check again by run composer dump-autoload;php artisan config:cache once updating the cart.php as mention in PR.

@vaishaliwebkul  i removed cache now working fine but still in case of cart item attribute[cart price ] it is not working as expected  and simply ignore the condition and set the shipping price to zero but  i tested it for cart attribute[subtotal] in this condition it is working fine.

@arooba499
can you display the condition in screenshot which is working fine as expect or which is not working??

I am talking about this condition when i set condition for cart price it is not working

as cart price must be greater than 200 for this condition and  in this case cart price is 300 but rule is not working,,,correct me if i am wrong

@arooba499
I also tested this with the same condition in the master by updating code with this PR
seems to be working fine, kindly check all the condition as shown below


Asynchronous file logging

Name
Type
Description

/async
bool
Optional. When true (default: false), there will be special worker thread spawned, which will only manage writing logs. All records will be transformed to owned, appended into the thread-safe queue. Flushing should be configured the same way as in synchronous logging with additional timed interval support (flush every N seconds for example).

Also there can be queue overloading, if disks work bad and we block on write/flush syscalls. To avoid infinite queue bloat we can suggest three alternatives:

Grow infinitely anyway. Then Blackhole won't drop any record until it is killed by OOM killer.
Constraint by queue size. If exceeded all further records will be dropped.
Constraint by total buffer size. The logic the same.

Will reborn as asynchronous sink/handler wrapper.


Add 27W USB-C Power Supply

Per https://www.raspberrypi.com/documentation/computers/configuration.html#common-bootloader-properties-chosenbootloader.
See also https://github.com/raspberrypi/documentation/pull/3267.
andrew@bookworm5:~ $ od -x  --endian=big /proc/device-tree/chosen/power/rpi_power_supply
0000000 0000 2e8a 000f 0000
0000010

Thanks to trejan on the forum: https://forums.raspberrypi.com/viewtopic.php?p=2154632#p2154394.

For the benefit of anybody else seeing this closed PR - https://github.com/raspberrypi/documentation/pull/3267 says "The first word is the VID, but the second word isn't a PID - it's a Product VDO."


[FEATURE] Alllow overriding the aws service identifier in AwsSigv4Signer

Is your feature request related to a problem?
Currently the service identifier is hard coded to es with no way to override.
https://github.com/opensearch-project/opensearch-js/blob/2.1/lib/aws/AwsSigv4Signer.js#L73
What solution would you like?
Allow overriding the default with a user provided value from opts
request.service = opts.service || 'es';

What alternatives have you considered?
rolling our own signing functionality

This makes sense. Feel free to contribute a PR.

Closing via https://github.com/opensearch-project/opensearch-js/pull/333


MRT doesn't respect parent height and just overflows

material-react-table version
v1.7.4
react & react-dom versions
v18
Describe the bug and the steps to reproduce it
MRT should respect the height of the parent element, and get smaller to fit into it, but it doesn't. If you place it in a div that is 500px tall, it just overflows out of it.
You can fix it manually in the CSS by making the paper height 100% and the table container height 100%.
Minimal, Reproducible Example - (Optional, but Recommended)
https://stackblitz.com/edit/github-zeky5k?file=src%2FTS.tsx
Screenshots or Videos (Optional)

Do you intend to try to help solve this bug with your own PR?
No, because I do not have time to dig into it
Terms

[X] I understand that if my bug cannot be reliably reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

i'm having the same problem as well and i don't know how to fix this too.
The current solution i have is using muiTableContainerProps props and set height and width in sx.
muiTableContainerProps={{                               sx: { width: '100%', height: '100%' },                             }}

Try setting layoutMode="grid" in the grid properties, that worked for me

Having the same issue, and none of the posted solutions fix it.

Also having this issue, and it's causing multiple problems, even with the solutions posted.

Having the same problem, if there are any solutions for this ?

I was also seeing this issue. I was able to fix it through applying some styles to the different accessible props. Here's the outcome

The selected div is the same as in OPs post. The two additions I made to make this height responsive:

Manually override the immediate child div style with height: 100% (red tick mark). This is done by adding the following prop muiTablePaperProps={{ sx: { height: '100%' } }}
Add calc(100% - 112px) to muiTableContainerProps prop (blue tick mark). This is done by adding the following prop muiTableContainerProps={{ sx: { height: 'calc(100% - 112px)' } }}
The 112px comes from the addition of the header and footer tabs (56px each in my case). Adjust as you see fit with your table's design.

This provides a table that's height responsive where the row display itself expands and shrinks while the headers and footers stay visible.

thanks @arcasoy .
i needed to additionally add this to index.html
<style>
  html, body {
    /* fix flash of white on reload */
    background-color: black;
  }
  html, body, #root {
    /* enable 100% height for inner containers.. */
    height: 100%;
  }
</style>

and make sure that my App container also has sx= {{height: "100%"}} set.
App.jsx
export default function App() {
    return (
        <Container
            maxWidth="false"
            sx={{ height: "100%", display: "flex", flexDirection: "column" }}
        >
            <Typography variant="h4" component="h1" gutterBottom>
                test for MRT2
            </Typography>
            <MyTable/>
        </Container>
    );
}

Going to close this as this is kinda a feature rather than a bug in order for the table container scrolling to work as expected.
You can do something like this:
<div style={{ height: '300px' }}>
    <MaterialReactTable
      columns={columns}
      data={data}
      muiTableContainerProps={({ table }) => ({
        sx: {
          height: `calc(100% - ${table.refs.topToolbarRef.current?.offsetHeight}px - ${table.refs.bottomToolbarRef.current?.offsetHeight}px)`,
        },
      })}
      muiTablePaperProps={{
        sx: {
          height: '100%',
        },
      }}
    />
  </div>

if you really need it.

Would it be useful to add this as a prop to the  component? I imagine this is a commonly utilized styling and, with how elegant @KevinVandy's solution is, I wonder if an adjustToParent: true prop (or some other name) could be added to apply the styles suggested. This might just make implementation for a new user more seamless.


[help] Could you provide the preprocessed ml-1m datasets?

Hi @YChen1993 ,
Thanks for your great effort.
As the title suggests, could you provide an extra preprocessed ml-1m datasets, or the corresponding pre-processing scripts? I meet some troubles when apply your method on ml-1m, since I don't know the way that you preprocess the dataset.
Thanks!

Hi @yeeeqichen, thanks for asking!
I just uploaded the prepared ml-1m dataset. Specifically, I use the data prepared by SASRec here. I basically just convert the original flattened data format (one interaction per row) into the nested format (one user's interactions per row). You can run the script by python3 main.py --data_name ml-1m.
Although I did not report the ml-1m results in paper, I used to run our method with some default hyper-parameters (as well as a few latest developed baselines) on ml-1m. A quick snapshot is that no significant improvements are made upon SASRec on ml-1m. You can have a try and tune the hyperparameters in a granular manner.
Please feel free to let me know if you have any questions or observations. Thanks!

Thank your so much!


[FIX]  query with empty subject should return empty and not fail

fixes #453

[X] Have you followed the guidelines in our Contributing document?
[x] Have you lint your code locally prior to submission?
[x] Have you written new tests for your core changes, as applicable?
[x] Have you successfully ran tests with your changes locally?

Does it means that now entities also supports regexp, similar to sub?

Does it means that now entities also supports regexp, similar to sub?

it hopefully should, let me know if you encounter a bug with it.


Cannot read properties of undefined (reading 'meta')

TypeError: Cannot read properties of undefined (reading 'meta')
    at http://localhost/apps/groups/assets/index.f67c0ed6.js:1150:57081
    at Array.forEach (<anonymous>)
    at http://localhost/apps/groups/assets/index.f67c0ed6.js:1150:57046
    at Object.useMemo (http://localhost/apps/groups/assets/index.f67c0ed6.js:93:59210)
    at er.useMemo (http://localhost/apps/groups/assets/index.f67c0ed6.js:12:5954)
    at tmt (http://localhost/apps/groups/assets/index.f67c0ed6.js:1150:56997)
    at gR (http://localhost/apps/groups/assets/index.f67c0ed6.js:93:53252)
    at Nae (http://localhost/apps/groups/assets/index.f67c0ed6.js:97:8857)
    at Tae (http://localhost/apps/groups/assets/index.f67c0ed6.js:97:964)
    at DSe (http://localhost/apps/groups/assets/index.f67c0ed6.js:97:887)

What were you doing in the app when you saw this error, and was it in groups or talk?

What were you doing in the app when you saw this error, and was it in groups or talk?

I believe I tried to open a group and it crashed this way (don't recall the group). I was using "new" group

No reproduction, closing for now.


Enable @:dox(hide) for types.

I kind of missed that call. Is there still an RTTI problem that we have to address?

too late now


Possibility to mock HttpClient

Is your feature request related to a problem? Please describe.
We would like to build in-memory integration tests. For this we would like to provide our own HttpClient to the DataSync client library, for example to provide mocked authentication functionality.
I saw in #887 the HttpClient was exposed, but it cannot be set.
Describe the solution you'd like
Being able to set HttpClient for the DataSync client library
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

You don't need to set the HttpClient for unit tests.
For faking, create a FakeDelegatingHandler and push that into the config for the client - we do this ourselves in our tests - see https://github.com/Azure/azure-mobile-apps/blob/main/sdk/dotnet/test/Datasync.Common.Test/Mocks/MockDelegatingHandler.cs
You could, as an alternative, provide a mocked HttpClientHandler instead (as the last item in the pipeline)
For mocking, mock the IRemoteTable or IOfflineTable instead.  You can then inject that and bypass testing at a HTTP layer.
I don't think it's a good idea to replace the HttpClient as it requires a relatively specific setup for client/server commuication.  We've provided the facilities to inject into the pipeline at the proper point.
I'm going to mark this as resolved, so it will auto-close 7 days after the final comment.

I think this is related, therefore I'm commenting here: I would like to use Microsoft.AspNetCore.Mvc.Testing package in integration tests. Ist it possible to use the HttpClient created from WebApplicationFactory.CreateClient() with DatasyncClient?
Thank you!

We do that ourselves in our test projects.  Take a look at the Integration tests to see how we set it up.


Fix popup trying to emit the non-existing signal 'cancelled' when closed

Closing any popup (in game or in the editor) spams the message Can't emit non existing signal cancelled. (This signal only exists in AcceptDialog).
This PR removes this call in the Popup class. The signal popup_hide which is sent when the popup is closed is already sent on line 74.

Thanks!


preset websockify path in novnc app

The RELATIVE_URL_ROOT environment variable will allow the access of the novnc web page from a different path like this:
http://<IP_ADDRESS>:6080/novnc_app/
But if you go in to the right hand settings in the app for websockify it states that the relative path is websockify not novnc_app/websockify
This is a problem when I try to run novnc through an nginx reverse proxy with a conf like this:
server {
    $LISTEN_443
    server_name $FQDN;

    ssl_certificate /etc/letsencrypt/live/example.com/fullchain.pem;
    ssl_certificate_key /etc/letsencrypt/live/example.com/privkey.pem;

    location /novnc_app/ {
        proxy_set_header X-Forwarded-Host \$host;
        proxy_set_header X-Forwarded-Server \$host;
        proxy_set_header X-Forwarded-For \$proxy_add_x_forwarded_for;
        proxy_buffering off;
        proxy_pass https://novnc:443/customer/;

        # Enable websockets for the noVNC console to work
        proxy_http_version 1.1;
        proxy_set_header Connection \$http_connection;
        proxy_set_header Upgrade \$http_upgrade;
    }
}

The console says that it can’t find websockify but if I edit the right hand settings to
novnc_app/websockify  it works great.
Is there a way to automate, or preset this value so it does not need to be set manually?

The RELATIVE_URL_ROOT environment variable will allow the access of the novnc web page from a different path like this:
http://<IP_ADDRESS>:6080/novnc_app/
But if you go in to the right hand settings in the app for websockify it states that the relative path is websockify not novnc_app/websockify
This is a problem when I try to run novnc through an nginx reverse proxy with a conf like this:
server {
    $LISTEN_443
    server_name $FQDN;

    ssl_certificate /etc/letsencrypt/live/example.com/fullchain.pem;
    ssl_certificate_key /etc/letsencrypt/live/example.com/privkey.pem;

    location /novnc_app/ {
        proxy_set_header X-Forwarded-Host \$host;
        proxy_set_header X-Forwarded-Server \$host;
        proxy_set_header X-Forwarded-For \$proxy_add_x_forwarded_for;
        proxy_buffering off;
        proxy_pass https://novnc:443/customer/;

        # Enable websockets for the noVNC console to work
        proxy_http_version 1.1;
        proxy_set_header Connection \$http_connection;
        proxy_set_header Upgrade \$http_upgrade;
    }
}

The console says that it can’t find websockify but if I edit the right hand settings to
novnc_app/websockify it works great.
Is there a way to automate, or preset this value so it does not need to be set manually?

The same problem here. Where did you @OhioVR edit the right hand settings to
novnc_app/websockify to make it works great? Thanks!

fixed on latest version


Eye Tracking Test Tool Works - Doesnt work on Vrchat

I'm using a custom Gary Mod world Avatar with eye tracking and a quest pro option on the avatar. But I can't get any response but closed eye lids or open eyelids.
When I use the troubleshoot program it's working perfectly via virtual desktop. Is there a step I'm missing to utilize the tool?

VRChat isn't OpenXR. So I think this program is completely irrelevant to you.

This is an eye tracking, not face tracking program. Eye tracking = the use of eye gaze for the purpose of pointing. You're not going to get eye lid info, this would require face tracking.


Vignettes

Have added some addtional documentation:
Diagram in the readme
Updated vignettes
Model equations
Can't remeber how to get the github page to build from master - is it just a github action @OJWatson ?

Pkgdown GHA yaml file isn't in this branch (or any I think - though was in master at some point). Have added it back in and am just running the pkgdown GHA test now.


feat: bring trade success screen to new trade flow

Description
Does what it says on the box - bring back the scree to the new trade flow.
Issue (if applicable)

closes #8027
Risk

High Risk PRs Require 2 approvals

What protocols, transaction types, wallets or contract interactions might be affected by this PR?

None - only change in prod code paths is removal of negative x margin for trade summary content.
Testing

Engineering

Trade summary looks good with the new trade flow flag on
Trade summary looks good with the new trade flow flag off

Operations

[ ] :checkered_flag: My feature is behind a flag and doesn't require operations testing (yet)

Trade summary looks good

Screenshots (if applicable)

Flag on

Flag off

This is awesome @gomesalexandre, thank you ser! 🙏
We probably want to update the trade summary to show the updated "ExpandedStepperSteps" content to match the style of the new trade flow (when expanded).


⚠ Feedaty widget has degraded performance

In cbcee7c, Feedaty widget (https://widget.feedaty.com/public/js/feedaty.js?merchant=10213975&style_ver=2021&data-sku=36263) experienced degraded performance:

HTTP code: 200
Response time: 1167 ms

Resolved: Feedaty widget performance has improved in 19f9165.


build: Fix grep to search only the exact symbol name

For example Init and UnInit conflict with each other.

@Dirbaio :smiley_cat:

@Dirbaio :smiley_cat:


geoutils version in outdated in conda feedstock

See at this line, the version indicated for the latest version of geoutils is 0.0.15, while we are now at version 0.1.5 and xdem's environment requires >0.1.5.
Therefore, if installing via conda-forge:
mamba install -c conda-forge geoutils -> yields version 0.1.5
mamba install -c conda-forge xdem -> forces a downgrade to 0.0.15

This is all as expected! :smile:
Latest xdem release 0.0.18 only works with geoutils 0.0.15 exactly. Only the main branch of xdem currently works with latest geoutils, in preparation for the next release!

So the conda feedstock package dependency versions should match the ones of the latest published release (tar file in the "Releases" of the repo).

Ok I'm convinced now!


Syncing issues. Websocket connection is not created for some reason

Logs in 297808

Has not been reported. Changes in the backend a long time ago must have helped.


Sergio ortiz development

Adds a map to my webapp.

I messed up with the first commit to try and handle null events. I thought a method called isEmptyorNull() existed but apparently not lol. Instead I manually checked if event.isEmpty() || event == null


Limit number of WebClient instances

Is your feature request related to a problem? Please describe.
WebClient invokes external sources to fetch some data or modify some values. Knot.x HTTP Server handles incoming requests with custom handlers. Each handler can contain some business logic that interacts with external data sources via WebClient.
Moreover, Knot.x Fragments allows defining business logic in the form of a graph. Each graph node can communicate with some external resource via WebClient. HTTP Action fetches data from APIs via GET requests.
We need to revalidate when we create WebClient instances and where we can cache some objects.
Describe the solution you'd like
Design session required.
Additional context
Related modules:

https://github.com/Knotx/knotx-data-bridge
https://github.com/Knotx/knotx-repository-connector/tree/master/http
https://github.com/Knotx/knotx-server-http

The default behaviour of @Cacheable annotation is to cache using action alias as key.
As long as an alias is unique, this should not lead to inconsistencies (requires validation).
WebClient instances can be cached on the factory level with WebClientOptions as key (transformed to JSON - equals and hashCode).

WebClient caching is introduced in #99. However, the cache size is not configurable.
Consider creating a shared WebClientCache available to all action factories (may require an API change).

Implemented in io.knotx.fragments.handler.action.http.HttpActionFactory.


[FEAT?] Add daily note support

WHAT
If I want to quick add data to daily note, it will be hard to change the date everyday, OR i should write a script for myself.
**But I think it is hard to code this feature, so this issue is just an inspiration **
HOW TO
Argenos had coded a similiar feature before. https://github.com/argenos/nldates-obsidian
HOW IT COULD BE
It would be nice that make a parameter like daily?
like obsidian://advanced-uri?vault=<test-vault>&timepath=%DD%%MM%%WW%%20Weekly%20Note&data=<text>&mode=append

Good idea. What do you want to achieve with timepath?

@Vinzent03 I don't know if it is doable, but I want to use my daily note to capture my ideas and TODO, for example , I am using my daily note to capture todo. Don't need to open Ob and then input data.
Just double click Ctrl to call listary and then input what I want.
And also, I could make daily note as a read later center, though it is not so convinent now, but I have use the feature to append URL\Text\Current Time and track and so on....
I think if it is doable, I could give you another inspiration:

Add a configurable path in setting

like: timepath:  YY-MM-DD dailynote

I still don't understand your aim with timepath. Just to tell the plugin where the daily notes are or to insert dates based on this moment format?

@Vinzent03 Hmmmm, if you have another better solution, please ignore what the timepath .... 😂

To clarify. You aim is to append text to the today's daily note?  Normal text or has it to support extra features?

@Vinzent03 Yes, my aim is to append text to the today's daily note

I hope the new release fits your needs.

@Vinzent03 it works so good ! Thanks for your hard job!


iOS/Safari: AADSTS50058

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Users get the Error AADSTS50058: A silent sign-in request but none of the currently signed in user(s) match the requested login hint.

Expected or Desired Behavior
After a user clicks OK on the alert "To view the information on this page, you need to verify your identity" this error should not appear again after the user verified his identity and reloads the page.
Observed Behavior
The popup appears again after the user successfully logged and verified his identity after clicking OK on the alert leading to a login loop on iOS/Safari.
Steps to Reproduce
SharePoint Online
SPFx WebPart fetching data from AadHttpClient, SPO search, lists, ms graph.
The issue occurs on our own production and test environment spo tenants and our customers using iOS/Safari and switching between different accounts on different tenants.
To some users this happens a lot. Clearing cache and cookies seems to only temporarily fix the issue.
Sadly we can not reproduce this behaviour consistently and could not reproduce this at all in incognito windows.

I can confirm this issue. We have a webpart which is using MSGraphClient. Everything is working as expected, except if you don't empty the safari cache explicitely on an iPhone or Mac, after a while - say 24 hours but maybe sooner - the user will be presented a popup that tells him/her To view the information on this page, you need to verify your identity. Click Here. So far - according to this issue https://github.com/SharePoint/sp-dev-docs/issues/2685 - this is by design. However, when the user clicks the button / link and after the user is redirected, unfortunately the popup re-appears. And more in general, the work-around also negatively impacts the performance (due to a number of redirects being added to the request). This error can also be reproduced on a Mac when browsing with Safari. In the developer console the following error is logged: Error: AADSTS50058: A silent sign-in request was sent but no user is signed in. The cookies used to represent the user's session were not sent in the request to Azure AD. This can happen if the user is using Internet Explorer or Edge, and the web app sending the silent sign-in request is in different IE security zone than the Azure AD endpoint (login.microsoftonline.com).
Even though it says "no user is signed in" it is possible to browse every sharepoint online site without hitting a login screen.
As the webpart is embedded on a sharepoint site, there should be no error saying "no user signed in". The MSGraphClient should handle this error internally by initiating all steps necessary to log in the user again and possible use a refresh token.

@lahuey seems you requested people submit a new issue if this was still going on in your comment here https://github.com/SharePoint/sp-dev-docs/issues/2685#issuecomment-513048328 ...

Folks, just for investigation purposes, can you please check if disabling ITP in Safari would remove the issue? https://github.com/AzureAD/microsoft-authentication-library-for-js/wiki/Known-issue-on-Safari

@lucabandMSFT I have received indeed the same request from Microsoft Premier Support. I will try on Monday when I have the necessary devices to test again available. Having said that, is it really a solution to require your users to go into their settings and make changes?

We are also seeing this issue on Mac and Windows 10.
Mobile devices as well.

Yes Its working after disabling ITP. But it would be difficult for all users to communicate or alerting user to disable that settings... specifically for top leadership.


[ur] Localize 'auto-scaling' in urdu

Need to localize auto-scaling term in urdu

Closing Issue! PR Merged


Finalize Frontend

User story

As a software developer
I want  to make sure there are no typos or broken image links
So that user has a great User Experience

Acceptance criteria

No typos or broken links
Rename buttons if needed

Definition of done

Can be demoed
(Fully tested with the unit test)
Code well structured and formatted (hold to code conventions)
Clear explaination of code
No syntax/runtime errors
Codes are peerly reviewed
Communicate issues by leaving comments on github

Renamed button Requests Overview to my Closed Review.
In this way the user has an overview of all open requests in the calendar and access the closed ones through the button.
Changed link behind the plugin avatar.
Removed some lorem ipsums and added greeting on the calendar
Task will be merged together with the refactor_calendar branch beacause the code changes made are very minor


Cloud Manager - Code Scan Report - Bug and Code Smell Issues

Required Information

AEM Cloud Service
Adobe Cloud Manager Code Scan
ACS AEM Commons Version: 5.3.2
Reproducible on Latest? Yes

ACS_Commons_CloudManager_Code_Scan.xlsx

Have you tried using ACS AEM Commons 6+ -- the 6.0.0 release adjusted for many AEM as a Cloud Service requirements.

@davidjgonzalez I noticed around 36 minor issues with ACS Commons (acs-aem-commons-ui.apps:6.6.4). How should we address them?
build_project_issues (1).csv


add health checks to controller and router charts

K8s will use each type of probe if we make them available. A successful probe means:

startup: the app is finished starting up, go ahead and try the readiness probe
readiness: the app is ready to receive incoming requests, tell the LB to begin forwarding to the pod
liveness: the app is healthy and still has capacity for more requests, keep sending and don't restart me right now

Ref: https://kubernetes.io/docs/tasks/configure-pod-container/configure-liveness-readiness-startup-probes/

There is an ER health check configuration that can be added to the er config that we use currently to verify the health of an ER & if it's connected to the controller ctrlPingCheck.  It doesn't specifically show it's "ready to receive traffic", because that could mean lot of things, since it could be connected to certain links while others are having issues/down.  This is just a general check to make sure the ER is receiving updates from the controller.
The most recent addition to the health check is linkCheck, so you can verify if the ER has a links or even a specific link to another ER, which would indicate it's ready to transport traffic from one ER to another.
To add the health check to the ER, you need to add something like this to the ER config:
healthChecks:
  ctrlPingCheck:
    # How often to ping the controller over the control channel. Defaults to 30 seconds
    interval: 30s
    # When to timeout the ping. Defaults to 15 seconds
    timeout: 15s
    # How long to wait before pinging the controller. Defaults to 15 seconds
    initialDelay: 15s
  linkCheck:
    # Number of links required for the health check to be passing. Defaults to 0
    minLinks: 1
    # How often to check the link count, defaults to 5s
    interval: 5s
    # How long to wait before running the first check. Defaults to 1s
    initialDelay: 5s

& also add a web section to the ER like this:
web:
  # name - required
  # Provides a name for this listener, used for logging output. Not required to be unique, but is highly suggested.
  - name: health-check
    # bindPoints - required
    # One or more bind points are required. A bind point specifies an interface (interface:port string) that defines
    # where on the host machine the webListener will listen and the address (host:port) that should be used to
    # publicly address the webListener(i.e. mydomain.com, localhost, <IP_ADDRESS>). This public address may be used for
    # incoming address resolution as well as used in responses in the API.
    bindPoints:
      #interface - required
      # A host:port string on which network interface to listen on. <IP_ADDRESS> will listen on all interfaces
      - interface: <IP_ADDRESS>:8081
        # address - required
        # The public address that external incoming requests will be able to resolve. Used in request processing and
        # response content that requires full host:port/path addresses.
        address: <IP_ADDRESS>:8081
    # apis - required
    # Allows one or more APIs to be bound to this webListener
    apis:
      # binding - required
      # Specifies an API to bind to this webListener. Built-in APIs are
      #   - health-checks
      - binding: health-checks

This combination would allow you to GET https://localhost:8081/health-checks
The above would produce something like this as an output:
{
    "data": {
        "checks": [
            {
                "details": null,
                "healthy": true,
                "id": "controllerPing",
                "lastCheckDuration": "4.344µs",
                "lastCheckTime": "2024-02-13T18:40:01Z"
            },
            {
                "details": [
                    {
                        "linkId": "lndLOpwd7yOcSXtCcPwWf",
                        "destRouterId": "j.LOxzd9A",
                        "latency": 3271785.96875,
                        "addresses": {
                            "ack": {
                                "localAddr": "tcp:<IP_ADDRESS>:443",
                                "remoteAddr": "tcp:<IP_ADDRESS>:61031"
                            },
                            "payload": {
                                "localAddr": "tcp:<IP_ADDRESS>:443",
                                "remoteAddr": "tcp:<IP_ADDRESS>:65235"
                            }
                        }
                    },
                    {
                        "linkId": "3W72EY2a0inbyCHIdYk6Gd",
                        "destRouterId": "f9fs.nvej",
                        "latency": 84934025.1015625,
                        "addresses": {
                            "ack": {
                                "localAddr": "tcp:<IP_ADDRESS>:443",
                                "remoteAddr": "tcp:<IP_ADDRESS>:42764"
                            },
                            "payload": {
                                "localAddr": "tcp:<IP_ADDRESS>:443",
                                "remoteAddr": "tcp:<IP_ADDRESS>:42750"
                            }
                        }
                    },
                    {
                        "linkId": "1yp3sDwqj6CHui4Zmt89wB",
                        "destRouterId": "PKud5nLtj",
                        "latency": 188746151.2265625,
                        "addresses": {
                            "ack": {
                                "localAddr": "tcp:<IP_ADDRESS>:58616",
                                "remoteAddr": "tcp:<IP_ADDRESS>:443"
                            },
                            "payload": {
                                "localAddr": "tcp:<IP_ADDRESS>:52392",
                                "remoteAddr": "tcp:<IP_ADDRESS>:443"
                            }
                        }
                    },
                    {
                        "linkId": "5wjStaisGHkT5Xu0fQrfEq",
                        "destRouterId": "bnq85xLt3",
                        "latency": 201215039.2109375,
                        "addresses": {
                            "ack": {
                                "localAddr": "tcp:<IP_ADDRESS>:443",
                                "remoteAddr": "tcp:<IP_ADDRESS>:48878"
                            },
                            "payload": {
                                "localAddr": "tcp:<IP_ADDRESS>:443",
                                "remoteAddr": "tcp:<IP_ADDRESS>:48864"
                            }
                        }
                    },
                    {
                        "linkId": "4e0RACw8dsguV43BrAsQfc",
                        "destRouterId": "u6Q1QSPulm",
                        "latency": 3872621.265625,
                        "addresses": {
                            "ack": {
                                "localAddr": "tcp:<IP_ADDRESS>:443",
                                "remoteAddr": "tcp:<IP_ADDRESS>:48868"
                            },
                            "payload": {
                                "localAddr": "tcp:<IP_ADDRESS>:443",
                                "remoteAddr": "tcp:<IP_ADDRESS>:48864"
                            }
                        }
                    },
                    {
                        "linkId": "6AmAWcfB50a2OzFvlsr1vn",
                        "destRouterId": "7fTQPzdt7d",
                        "latency": 3430749.8671875,
                        "addresses": {
                            "ack": {
                                "localAddr": "tcp:<IP_ADDRESS>:443",
                                "remoteAddr": "tcp:<IP_ADDRESS>:50371"
                            },
                            "payload": {
                                "localAddr": "tcp:<IP_ADDRESS>:443",
                                "remoteAddr": "tcp:<IP_ADDRESS>:25962"
                            }
                        }
                    },
                    {
                        "linkId": "1CnhJJ73e1AiKjoVoRy3tt",
                        "destRouterId": "oWeCqGOcJ",
                        "latency": 2906363.9140625,
                        "addresses": {
                            "ack": {
                                "localAddr": "tcp:<IP_ADDRESS>:443",
                                "remoteAddr": "tcp:<IP_ADDRESS>:56812"
                            },
                            "payload": {
                                "localAddr": "tcp:<IP_ADDRESS>:443",
                                "remoteAddr": "tcp:<IP_ADDRESS>:56796"
                            }
                        }
                    },
                    {
                        "linkId": "3vfjSpoNfoYbyqIBbT7ZKx",
                        "destRouterId": "R7nKHgLtj",
                        "latency": 66181343.484375,
                        "addresses": {
                            "ack": {
                                "localAddr": "tcp:<IP_ADDRESS>:45646",
                                "remoteAddr": "tcp:<IP_ADDRESS>:443"
                            },
                            "payload": {
                                "localAddr": "tcp:<IP_ADDRESS>:45640",
                                "remoteAddr": "tcp:<IP_ADDRESS>:443"
                            }
                        }
                    },
                    {
                        "linkId": "2MaMaVsqFNtvhRejCpyR7y",
                        "destRouterId": "s3FjWqdlS",
                        "latency": 70628346.7578125,
                        "addresses": {
                            "ack": {
                                "localAddr": "tcp:<IP_ADDRESS>:443",
                                "remoteAddr": "tcp:<IP_ADDRESS>:57722"
                            },
                            "payload": {
                                "localAddr": "tcp:<IP_ADDRESS>:443",
                                "remoteAddr": "tcp:<IP_ADDRESS>:57720"
                            }
                        }
                    },
                    {
                        "linkId": "2EVj6GaGBr1KEFXSeypc3i",
                        "destRouterId": "joI2Wqdlb",
                        "latency": 187742628.4765625,
                        "addresses": {
                            "ack": {
                                "localAddr": "tcp:<IP_ADDRESS>:38150",
                                "remoteAddr": "tcp:<IP_ADDRESS>:443"
                            },
                            "payload": {
                                "localAddr": "tcp:<IP_ADDRESS>:38136",
                                "remoteAddr": "tcp:<IP_ADDRESS>:443"
                            }
                        }
                    }
                ],
                "healthy": true,
                "id": "link.health",
                "lastCheckDuration": "120.997µs",
                "lastCheckTime": "2024-02-13T18:40:06Z"
            }
        ],
        "healthy": true
    },
    "meta": {}
}


On Firefox 24: getBBox is not a function

I get this error after following the example setup using d3 and dagre-d3 (dagre-d3 v0.2.9).
TypeError: labelSvg.node(...).getBBox is not a function @ http://mr.gy/scripts/dagre-d3.js:571

FYI to anyone else who runs into this, you can get around it by using createElementNS() instead of createElement() and passing the namespace URI http://www.w3.org/2000/svg. In my case, I was creating the <svg> element dynamically, but I had to do this with document.createElementNS("http://www.w3.org/2000/svg", "svg").

Just hit this. Turns out Firefox also throws this error when the SVG is hidden by something like display:none… or… being in a closed <details> dropdown! Dangit Firefox >_<


[release-1.9] telemetry tests: fix more loop vars in goroutines

Manual cherrypick required.
#30817 failed to apply on top of branch "release-1.9":
Applying: telemetry tests: fix more loop vars in goroutines
Using index info to reconstruct a base tree...
M	tests/integration/telemetry/stackdriver/stackdriver_filter_audit_test.go
M	tests/integration/telemetry/stackdriver/stackdriver_filter_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/integration/telemetry/stackdriver/stackdriver_filter_test.go
Auto-merging tests/integration/telemetry/stackdriver/stackdriver_filter_audit_test.go
CONFLICT (content): Merge conflict in tests/integration/telemetry/stackdriver/stackdriver_filter_audit_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 telemetry tests: fix more loop vars in goroutines
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

/close

Thanks @xichengliudui


Fork.dev integration broken by upgrade to 2.36

git version 2.36.0.vfs.0.0
This started when v2.36 version was installed - I saw that timing line up on two different machines on different days.
The gui I use, Fork.dev, has stopped working. It stays in this state permanently:

(Expected: It loads information about the repo and displays it in its UI, as it did before v2.36).
This happens 100% of the time on some repositories, and 0% of the time on other repositories.
Looking in Fork.dev logs, I see failures like the following:
?? 2022-04-27 16:03:33.129 Git request failed'status --porcelain -z --untracked-files=all':
fatal: unsafe repository ('X:/Office/src' is owned by someone else)
To add an exception for this directory, call:
git config --global --add safe.directory X:/Office/src

I can reproduce this failure from a non-elevated command prompt.
P.S. May have the same cause as this issue.

I've confirmed that following the "git config" command listed (for each affected repo) does work around the problem. But that is just a workaround, a fix is still needed.

Let me be clear that the message you see is not a bug. The fact that that warning is shown is Git working exactly as intended, as Git seems to see some dubious ownership there.
If you wish to see more information about the ownership issue, disable the safe.directory setting temporarily (e.g. via git config --global --edit), then run git status after setting the environment variable GIT_TEST_DEBUG_UNSAFE_DIRECTORIES to true.
It might be possible that Git is overreacting, but it is equally possible (I suspect that it is even more likely) that some fishy process lets your worktree be owned by an unintended user.

It is entirely possible for a feature behaving as intended to nevertheless be an unacceptable regression in user experience. This is especially common with security features.
Was this security check recently added or modified? The correlation with git upgrade, strongly suggests this (but doesn’t entirely rule out your theory either). I will follow your advice, but can’t do it right now.

From: Johannes Schindelin @.>
Sent: Friday, April 29, 2022 7:30:15 AM
To: microsoft/scalar @.>
Cc: Andrew Godfrey @.>; Author @.>
Subject: Re: [microsoft/scalar] Fork.dev integration broken by upgrade to 2.36 (Issue #520)
Let me be clear that the message you see is not a bug. The fact that that warning is shown is Git working exactly as intended, as Git seems to see some dubious ownership there.
If you wish to see more information about the ownership issue, disable the safe.directory setting temporarily (e.g. via git config --global --edit), then run git status after setting the environment variable GIT_TEST_DEBUG_UNSAFE_DIRECTORIES to true.
It might be possible that Git is overreacting, but it is equally possible (I suspect that it is even more likely) that some fishy process lets your worktree be owned by an unintended user.
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fscalar%2Fissues%2F520%23issuecomment-1113392379&data=05|01|agodfrey%40microsoft.com|779c9e5e47b94e8d34ec08da29ecc775|72f988bf86f141af91ab2d7cd011db47|1|0|637868394186780501|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=Eh6iPNZDsSi9F6qcE3uXjb5yxcCad5yVcvUsOUnM8H8%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAVRKNTV22VHRYDPKVUI6TF3VHPW7PANCNFSM5UQTUFNQ&data=05|01|agodfrey%40microsoft.com|779c9e5e47b94e8d34ec08da29ecc775|72f988bf86f141af91ab2d7cd011db47|1|0|637868394186780501|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=ABzpoJ%2Fw1GzOoW5PU8xF%2Be4S9E54Koyef%2Bq8qQP0EB0%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>


Width of whitespace

About the dejavu font at v5.1, the width of whitespace character at become larger than older version obvously.
Is it the expect rendering?

Hi,
No, it's not the expected. I regenerated the fonts and I will update them.
Thank you for pointed this out!

The new fonts with normal white spaces are in the master branch.


[baseline] as of 2.0.2, portmidi:x64-windows-static-md passes

see above, change ci.baseline.txt

Thanks for the review phoebe :)


How to disable Emmet attempting to expand a particular string?

Looking at the read me and I see "disabled_single_snippet_for_scopes" and in threads I have seen "disabled_single_snippets": "space separated list of tags goes here"
However, I'm not quite sure how to implement these. I basically want Emmet to stop trying to expand the string /**, specifically in CSS/SCSS.
What would be the corrent syntax to add to Emmet.sublime-settings? Thanks for any guidance.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/10218514-how-to-disable-emmet-attempting-to-expand-particular-string?utm_campaign=plugin&utm_content=tracker%2F35867&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F35867&utm_medium=issues&utm_source=github).

I'm having issues with the:
"disable_tab_abbreviations": true
and
"disabled_single_snippet_for_scopes":"text.html"
I want Emmet to stop expand the 'html' tag in html file,not working.

@benfrain Emmet never expands /** because it’s invalid abbreviation.
@bluedusk https://github.com/sergeche/emmet-sublime/blob/master/Emmet.sublime-settings#L65

@sergeche this is odd. I am using DocBlockr (https://github.com/spadgos/sublime-jsdocs/) and it dosn't expand /** in Sass/CSS. However, when I disable Emmet it works perfectly. Is there any part of Emmet that would be interfering?

@benfrain  Can you record a small video/gif with this bug?

Updated comment but sure, will record a GIF

That’s make sense: unlike HTML, in CSS etc. Emmet completely overrides Tab key so it’s not even passed further. There’s no easy solution because Sublime Text API does not allow passing key event from one plugin to another.
PS: I’m using DocBlockr too, in my case simply pressing Enter key after /** creates full comment block.

Ah!!!! I've been a fool :) Yes, just pressing Enter will work fine for me, thanks.

THe only setting works for me is add
"disable_tab_abbreviations": true
in sublime setting.
Any setting i made in Emmet setting won't work. Any idea why? thanks.

Did you changed Emmet.sublime-settings or Preferences.sublime-settings?


Target mode broken?

I'm not sure if I'm just misunderstanding, but I've got code that looks like this:
pointer:setTarget{actions[1]}
print (pointer:getMode())

This prints "POSITION".
Lines 85 - 94 of core/pointer/internal.lua read as follows:
	if (self._target ~= target) then
		return self
	end

	self._target = target
	self._mode = PointerMode.TARGET

	self._scene:__getInternal():onPointerTargetChanged(self._pointer)

	return self

If I'm reading that right, it seems like the problem is that the if statement is backwards.  Replacing the ~= with == on line 85 causes this error:
Error

lib/Inky/inky/core/pointer/internal.lua:244: attempt to call method '__getInternal' (a nil value)

Traceback

[love "callbacks.lua"]:228: in function 'handler'
lib/Inky/inky/core/pointer/internal.lua:244: in function 'setOverlappingElements'
lib/Inky/inky/core/scene/internal.lua:108: in function '_resolvePointerOverlappingElements'
lib/Inky/inky/core/scene/internal.lua:132: in function 'onPointerTargetChanged'
lib/Inky/inky/core/pointer/internal.lua:92: in function 'setTarget'
lib/Inky/inky/core/pointer/init.lua:54: in function 'setTarget'
ui/actionbar.lua:17: in function '_initializer'
lib/Inky/inky/core/element/internal.lua:312: in function 'initialize'
lib/Inky/inky/core/scene/internal.lua:199: in function 'render'
lib/Inky/inky/core/element/internal.lua:323: in function 'render'
lib/Inky/inky/core/element/init.lua:159: in function 'render'
main.lua:105: in function 'draw'
[love "callbacks.lua"]:168: in function <[love "callbacks.lua"]:144>
[C]: in function 'xpcall'

Which is beyond my ability to troubleshoot.

Sorry for the late response. This has been fixed in the latest version.


docs: add template for pkg stormlib and SDL_mixer

Do these docs generate some page? I didn't know they existed or I would have added them

@wheybags was added recently: https://github.com/ruslo/hunter/pull/981 (moved from wiki to be precise).

to create files with templates for packages with missing documentation file use the following script in the folder maintenance
python3 docs_create_missing_stubs.py

this creates the files maintenance/packages/only_hunter/<pkg_name>.rst
You can add these files then to docs/packages/pkg/ folder and update the __FIXME__ tags


uninitialized constant JSON::JWT

Im trying to load this gem into a rails project but for some reason when I try to use it I get an uninitialized constant JSON::JWT error.
Ive just included
gem 'json-jwt' in my Gemfile, like usual, then did bundle install. Ive never had this type of issue.
Do you have any idea why this might be happening?
Thanks

sorry, no idea.

Had the same problem. In my case, oj_mimic_json was replacing JSON const. Putting json-jwt before oj fixed the problem:
gem 'json-jwt'
gem 'oj'
gem 'oj_mimic_json'


chore(master): release 0.4.2

:robot: I have created a release beep boop
0.4.2 (2024-02-29)
Miscellaneous Chores

deps: update everything (061625d)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/indykite/terraform-provider-indykite/releases/tag/v0.4.2 :sunflower:


Issue 254 clean up spme outputs

Description
Cleaned up SPMe output by implementing appropriate equations with submodels. Also written a standard set of output variables which we can ensure is consistent across all model for a specific chemistry.
Fixes #254
Type of change
Please delete options that are not relevant.

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

Key checklist:

[x] No style issues: $ flake8
[x] All tests pass: $ python run-tests.py --unit
[x] The documentation builds: $ cd docs and then $ make clean; make html

You can run all three at once, using $ python run-tests.py --quick.
Further checks:

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works
[x] Any dependent changes have been merged and published in downstream modules

Codecov Report

Merging #258 into master will increase coverage by 0.02%.
The diff coverage is 98.75%.

@@            Coverage Diff             @@
##           master     #258      +/-   ##
==========================================
+ Coverage    97.7%   97.72%   +0.02%
==========================================
  Files          52       52
  Lines        2659     2772     +113
==========================================
+ Hits         2598     2709     +111
- Misses         61       63       +2

Impacted Files
Coverage Δ

pybamm/spatial_methods/finite_volume.py
96.87% <100%> (ø)
:arrow_up:

pybamm/models/submodels/electrode.py
98.68% <100%> (+1.38%)
:arrow_up:

pybamm/models/lithium_ion/spme.py
100% <100%> (ø)
:arrow_up:

pybamm/models/submodels/electrolyte_diffusion.py
100% <100%> (ø)
:arrow_up:

pybamm/models/lithium_ion/spm.py
100% <100%> (ø)
:arrow_up:

pybamm/models/submodels/electrolyte_current.py
100% <100%> (ø)
:arrow_up:

pybamm/discretisations/discretisation.py
98.81% <33.33%> (-1.19%)
:arrow_down:

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 dbc8711...ba2bf3d. Read the comment docs.

yea, I think users are all going to be interested to calculate and plot different things. You could have a small number of standard output variables. in the pybamm.LithiumIonBaseModel or pybamm.BaseModel class. Just put the name in the dict equated to None, then have the derived classes replace that with the actual expression.

Refactor using {"variable": None} dictionary in the base model

Thanks for the feedback on this I think it is much better now. I have added standard variables to base_model as suggested. I have also put a check in check_model_wellposedness to checks if these variables have been added. I also went ahead and did the DFN outputs aswell which was pretty simple to do.


Update is not working 0.071

I've been trying since it came out but I can't get this error out... can you help me ?
Logger: homeassistant.components.websocket_api.http.connection
Source: components/websocket_api/commands.py:226
Integration: Home Assistant WebSocket API (documentation, issues)
First occurred: 14:42:47 (1 occurrences)
Last logged: 14:42:47
[2607201512] Error updating Batmon: The command '/bin/ash -o pipefail -c pip3 install -r requirements.txt' returned a non-zero code: 1
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/hassio/update.py", line 167, in async_install
await async_update_addon(self.hass, slug=self._addon_slug, backup=backup)
File "/usr/src/homeassistant/homeassistant/components/hassio/handler.py", line 53, in _wrapper
raise HassioAPIError(data["message"])
homeassistant.components.hassio.handler.HassioAPIError: The command '/bin/ash -o pipefail -c pip3 install -r requirements.txt' returned a non-zero code: 1
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/websocket_api/commands.py", line 226, in handle_call_service
await hass.services.async_call(
File "/usr/src/homeassistant/homeassistant/core.py", line 2012, in async_call
response_data = await coro
^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/core.py", line 2049, in _execute_service
return await target(service_call)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 235, in handle_service
return await service.entity_service_call(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 876, in entity_service_call
response_data = await _handle_entity_call(
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 948, in _handle_entity_call
result = await task
^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/update/init.py", line 157, in async_install
await entity.async_install_with_progress(version, backup)
File "/usr/src/homeassistant/homeassistant/components/update/init.py", line 432, in async_install_with_progress
await self.async_install(version, backup)
File "/usr/src/homeassistant/homeassistant/components/hassio/update.py", line 169, in async_install
raise HomeAssistantError(f"Error updating {self.title}: {err}") from err
homeassistant.exceptions.HomeAssistantError: Error updating Batmon: The command '/bin/ash -o pipefail -c pip3 install -r requirements.txt' returned a non-zero code: 1

get same error in my ha

This appears to be the new 'influxdb' requirement. What hardware and OS is it? Inside HA?

Yes inside HA

Now is OK … thanks


After edit html files, VI report an issue.

Hello, I use awesome version, In last Sunday I found somethings change, when I edit a html file and save it, It popup a windows below and say:
1 abc.html|1 col 1 warning| discarding malformed <!DOCTYPE>
2 abc.html|2 col 1 warning| discarding unexpected
3 abc.html|10 col 1 warning|  lacks "alt" attribute
VI version > "VIM - Vi IMproved 7.4 (2013 Aug 10, compiled Oct 13 2016 21:22:16)"
Please help!

Some solutions:
Add let g:syntastic_html_checkers=[''] into my_configs.vim


Errors without Examples folder

Overview
If you remove the Examples folder, the toolkit fails to compile.
Due to BoundingBoxRig and ProgressExampleLaunchButton.
Also LoadingAnimation and ProgressExampleLaunchButton are part of the Examples namespace, not sure if this is intentional.
Expected Behavior
Compiles
Actual Behavior
Doesn't compile
Steps to reproduce
Create empty project, import Dev_Working_Branch, and delete the Examples folder.
Unity Editor Version
2017.3.1f1
Mixed Reality Toolkit Release Version
Dev_Working_Branch

@cre8ivepark, can you take a look at this?

Closing issues older than 180 days. If this is still an issue, please reactivate with recent information.


Grace period for inactivating a recently voted in producer is too strict

system_contract::update_elected_producers only gives a 252 second grace period since the time a new producer was added into a new proposed producer schedule before considering them for inactivation due to no recent blocks produced.
This number was likely chosen prior to the latest DPOS changes that (on average) doubled the time between proposing a new producer schedule and making that schedule active. With those changes, and with a fully active set of 21 producers, a newly elected producer may be right on the edge of the grace period. This can frequently cause some newly elected producer to be disabled (zeroing out their block signing key) in a healthy network prior to them even producing their first block.
The easy fix is to just double this grace period to 504 seconds to match the doubling of the average time between proposed schedule to active schedule with the DPOS changes. However, this bug demonstrates that our current method of disabling inactive block producers is fragile. A better way of implementing it is likely possible and we should at least explore that for an updated version of the system contract after 1.0. But for the 1.0 milestone, it is likely sufficient to just double the grace period.

On steem & bitshares this grace period was closer to 1 day.   I think it is fair to allow 1 hour.