[bug] Launch failed but no error logs were generated.

Describe the bug
I am developing an application with tauri and steamworks-rs, and I forgot to copy the steam_api64.dll and steam_api64.lib files to the debug directory when I started it, so the application did not start as I wish.
npm run dev

><EMAIL_ADDRESS>dev
> tauri dev

     Running BeforeDevCommand (`npm run start`)

><EMAIL_ADDRESS>start
> ng serve

Initial chunk files | Names          |  Raw size
styles.css          | styles         | 700.77 kB |
polyfills.js        | polyfills      |  88.34 kB |
main.js             | main           |  11.65 kB |
chunk-RW2SMIYW.js   | -              |   1.98 kB |

                    | Initial total  | 802.74 kB

Lazy chunk files    | Names          |  Raw size
chunk-CANKSL6J.js   | welcome-routes |   3.96 kB |

Application bundle generation complete. [3.119 seconds]

Watch mode enabled. Watching for file changes...
NOTE: Raw file sizes do not reflect development server per-request transformations.
  ➜  Local:   http://localhost:14200/
        Info Watching D:\src\game\happy-cultivation-tauri\src-tauri for changes...
   Compiling vswhom-sys v0.1.2
   Compiling vswhom v0.1.0
   Compiling embed-resource v2.4.2
   Compiling tauri-winres v0.1.1
   Compiling tauri-build v1.5.3
   Compiling happy-cultivation-tauri v0.0.0 (D:\src\game\happy-cultivation-tauri\src-tauri)
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 5.87s

What puzzles me is that the console did not display any error logs, which caused me to waste a lot of time troubleshooting this issue.
I would like to know how to display the error logs.
Reproduction
https://github.com/qq253498229/tauri-error001
No response
Expected behavior
No response
Full tauri info output
npm run tauri info

><EMAIL_ADDRESS>tauri
> tauri info

[✔] Environment
    - OS: Windows 10.0.22631 X64
    ✔ WebView2: 126.0.2592.113
    ✔ MSVC: Visual Studio Community 2022
    ✔ rustc: 1.79.0 (129f3b996 2024-06-10)
    ✔ cargo: 1.79.0 (ffa9cf99a 2024-06-03)
    ✔ rustup: 1.27.1 (54dd3d00f 2024-04-24)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (default)
    - node: 20.10.0
    - pnpm: 9.5.0
    - yarn: 1.22.21
    - npm: 10.2.3

[-] Packages
    - tauri [RUST]: 1.7.1
    - tauri-build [RUST]: 1.5.3
    - wry [RUST]: 0.24.10
    - tao [RUST]: 0.16.9
    - @tauri-apps/api [NPM]: 1.6.0
    - @tauri-apps/cli [NPM]: 1.6.0

[-] App
    - build-type: bundle
    - CSP: unset
    - distDir: ../dist/happy-cultivation-tauri/browser
    - devPath: http://localhost:14200/
    - framework: Angular
    - bundler: Webpack

Stack trace
No response
Additional context
No response

I have tried setting RUST_BACKTRACE=1, but there are still no error logs.
$env:RUST_BACKTRACE = 1
echo $env:RUST_BACKTRACE
1

I think this is a windows issue. If you build the app and try to run the app binary from inside powershell it also silently fails. It will only complain about a missing dll if you double click on the exe. Here's one of the GitHub issues i found https://github.com/PowerShell/PowerShell/issues/16468 (auto closed by the inactivity bot but it's not resolved).


Upgrading IBM Notifier?

When SUPER is first installed it downloads and installs it's own private copy of IBM Notifier in to -
/Library/Management/super/IBM Notifier.app
As far as I can see whilst this is in a non-standard location it is a standard copy of this app.
I have therefore been perhaps too clever and worked on an AutoPkg recipe for Munki which downloads the latest IBM Notifier from the IBM GitHub repo, wraps it in an installer pkg which installs this latest copy in to the special location used by the SUPER script. I then have Munki check this location for what version is currently there and if needed runs my installer pkg to update it.
My installer pkg does work and does update this locations copy.
However it looks like something else is periodically downgrading it back to version 2.9.1 of IBM Notifier. (The current version at the time I write this is actually 3.0.3.)
Is the SUPER script installing a specific now out of date version each time it runs?
Answer - yes it is!
I tested and confirmed this by running the super-starter script and watched it replace version 3.0.3 with 2.9.1 of IBM Notifier.
Is there any particular reason why SUPER wants to use an out of date version?

Yes as I mentioned I had deliberately built a process using AutoPkg and Munki to upgrade your copy of IBM Notifier and then saw your system was downgrading it back. 🥲
I have now turned mine off since as things stand it is pointless.

FWIF... super v3 should work with the latest versions of IBM Notifier. I'm just not going to test further for super v3.

https://github.com/Macjutsu/super/releases/tag/v4.0.0-beta1


Issue to upload in App Store - ITMS-90338

Description
I'm facing an issue when trying to deliver to App Store Connect.
When I load the app, I get this email from Apple:
ITMS-90338: Non-public API usage - The app references non-public selectors in MyApp: _isKeyDown, _modifiedInput, _modifierFlags. If method names in your source code match the private Apple APIs listed above, altering your method names will help prevent this app from being flagged in future submissions. In addition, note that one or more of the above APIs may be located in a static library that was included with your app. If so, they must be removed. For further information, visit the Technical Support Information at http://developer.apple.com/support/technical/
Reading other issues with similar errors, I downgraded the 'Minimal Deployments' and added the find_and_replace code to the Podfile, but it didn't work for me.
In Android the upload to store works fine.
I'm using Xcode 14.3.

React Native Version
0.71.7
Output of npx react-native info
System:
OS: macOS 12.6.4
CPU: (4) x64 Intel(R) Core(TM) i7-4578U CPU @ 3.00GHz
Memory: 2.91 GB / 16.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 18.15.0 - ~/.nvm/versions/node/v18.15.0/bin/node
Yarn: 1.22.19 - ~/Documents/client-mobile-v2/node_modules/.bin/yarn
npm: 9.5.0 - ~/.nvm/versions/node/v18.15.0/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.12.1 - /Users/developer/.rvm/gems/ruby-3.2.2/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.2, iOS 16.2, macOS 13.1, tvOS 16.1, watchOS 9.1
Android SDK: Not Found
IDEs:
Android Studio: 2022.2 AI-222.4459.24.2221.9862592
Xcode: 14.2/14C18 - /usr/bin/xcodebuild
Languages:
Java: 11.0.18 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.2.0 => 18.2.0
react-native: 0.71.7 => 0.71.7
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce

Archive App or build in App Center.
Try to delivery in App Store Connect.
Receive an email from apple with one more issue.

Snack, code example, screenshot, or link to a repository
My Podfile:
require_relative '../node_modules/react-native/scripts/react_native_pods'
require_relative '../node_modules/@react-native-community/cli-platform-ios/native_modules'

platform :ios, '12.4'
prepare_react_native_project!

$RNMapboxMapsVersion = '~> 10.11.1'
$RNMapboxMapsImpl = 'mapbox'

# If you are using a `react-native-flipper` your iOS build will fail when `NO_FLIPPER=1` is set.
# because `react-native-flipper` depends on (FlipperKit,...) that will be excluded
#
# To fix this you can also exclude `react-native-flipper` using a `react-native.config.js`
# ```js
# module.exports = {
#   dependencies: {
#     ...(process.env.NO_FLIPPER ? { 'react-native-flipper': { platforms: { ios: null } } } : {}),
# ```
# flipper_config = ENV['nNO_FLIPPER'] == "1" ? FlipperConfiguration.disabled : FlipperConfiguration.enabled

linkage = ENV['USE_FRAMEWORKS']
if linkage != nil
  Pod::UI.puts "Configuring Pod with #{linkage}ally linked Frameworks".green
  use_frameworks! :linkage => linkage.to_sym
end

abstract_target 'appCommonPods' do
  config = use_native_modules!

  # Flags change depending on the env values.
  flags = get_default_flags()

  use_frameworks! :linkage => :static

  use_react_native!(
    :path => config[:reactNativePath],
    # Hermes is now enabled by default. Disable by setting this flag to false.
    # Upcoming versions of React Native may rely on get_default_flags(), but
    # we make it explicit here to aid in the React Native upgrade process.
    :hermes_enabled => flags[:hermes_enabled],
    :fabric_enabled => flags[:fabric_enabled],
    # Enables Flipper.
    #
    # Note that if you have use_frameworks! enabled, Flipper will not work and
    # you should disable the next line.
    # :flipper_configuration => flipper_config,
    # An absolute path to your application root.
    :app_path => "#{Pod::Config.instance.installation_root}/.."
  )
  $RNFirebaseAsStaticFramework = true

  pod 'react-native-branch', path: '../node_modules/react-native-branch'
  pod 'RNSha256', :path => '../node_modules/react-native-sha256'
  pod 'RNFS', :path => '../node_modules/react-native-fs'
  pod 'MapboxMaps', '10.11.1'
  pod 'rn-fetch-blob', :path => '../node_modules/rn-fetch-blob'
  permissions_path = '../node_modules/react-native-permissions/ios'
  pod 'Permission-Calendars', :path => "#{permissions_path}/Calendars"
  pod 'Permission-Camera', :path => "#{permissions_path}/Camera"
  pod 'Permission-LocationAlways', :path => "#{permissions_path}/LocationAlways"
  pod 'Permission-LocationWhenInUse', :path => "#{permissions_path}/LocationWhenInUse"
  pod 'Permission-LocationAccuracy', :path => "#{permissions_path}/LocationAccuracy"
  pod 'react-native-fbsdk-next', :path => '../node_modules/react-native-fbsdk-next'
  pod 'Permission-MediaLibrary', :path => "#{permissions_path}/MediaLibrary"
  pod 'Permission-Notifications', :path => "#{permissions_path}/Notifications"
  pod 'Permission-PhotoLibrary', :path => "#{permissions_path}/PhotoLibrary"
  pod 'OneSignalXCFramework', '>= 3.0', '< 4.0'
  pre_install do |installer|
    $RNMapboxMaps.pre_install(installer)

    Pod::Installer::Xcode::TargetValidator.send(:define_method, :verify_no_static_framework_transitive_dependencies) {}

    installer.pod_targets.each do |pod|
        if pod.name.eql?('RNPermissions') || pod.name.start_with?('Permission-')
          def pod.build_type;
            Pod::BuildType.static_library # I assume you use CocoaPods >= 1.9
        end
      end
    end
  end

  target 'app' do
    #Production-specific pods
    target 'appTests' do
      inherit! :complete
      # Pods for testing
    end
  end
  target 'appDevelopment' do
    #Development-specific pods
  end

  def find_and_replace(dir, findstr, replacestr)
    Dir[dir].each do |name|
        text = File.read(name)
        replace = text.gsub(findstr,replacestr)
        if text != replace
            puts "Fix: " + name
            File.open(name, "w") { |file| file.puts replace }
            STDOUT.flush
        end
    end
    Dir[dir + '*/'].each(&method(:find_and_replace))
  end

  post_install do |installer|
    $RNMapboxMaps.post_install(installer)
    react_native_post_install(
      installer,
      # Set `mac_catalyst_enabled` to `true` in order to apply patches
      # necessary for Mac Catalyst builds
      :mac_catalyst_enabled => false
    )
    __apply_Xcode_12_5_M1_post_install_workaround(installer)
    installer.pods_project.targets.each do |target|
      target.build_configurations.each do |config|
        config.build_settings['APPLICATION_EXTENSION_API_ONLY'] = 'No'
      end
    end

    find_and_replace("./node_modules/react-native/React/Base/RCTKeyCommands.m",
      "_modifierFlags",
      "_modifierEventFlags")
    find_and_replace("./node_modules/react-native/React/Base/RCTKeyCommands.m",
      "_modifiedInput",
      "_modifiedEventInput")
    find_and_replace("./node_modules/react-native/React/Base/RCTKeyCommands.m",
      "_isKeyDown",
      "_isKeyEventDown")

  end

end

target 'OneSignalNotificationServiceExtension' do
  use_frameworks! :linkage => :static
  pod 'OneSignalXCFramework', '>= 3.0', '< 4.0'
end

target 'OneSignalNotificationServiceExtensionDev' do
  use_frameworks! :linkage => :static
  pod 'OneSignalXCFramework', '>= 3.0', '< 4.0'
end

We've seen this issue in the past.  You'll need to set your build configuration to Release to avoid bundling the development tooling code that's using these API.

i've encontered similar issue after uploading my app to be tested in TestFlight
however, changing build configuration to Release still giving me same response from Apple Developer

changing build configuration to Release not work for me

after countless of trial, ive succesfully submitted to app store connect
Basically what I'm doing is changing these Build and Archive scheme to Release
Hope this helps anyone


Unable to connect to Synapse workspace on Private link using P2S VPN

I have set up synapse private link & endpoint and a VM (without Public IP) in a same subnet. Also created a P2S VPN on the same VNet.
I can connect to the VM with the Private IP. I can also access synapse through the VM, but not when I try to access directly from my laptop using P2S VPN.
Any update on this would be highly appreciated
https://docs.microsoft.com/en-us/azure/synapse-analytics/security/how-to-connect-to-workspace-with-private-links

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

ID: d50a36d1-a544-8500-7591-13dbfc78bff3
Version Independent ID: 9cf5a12a-1887-3755-b845-194d1dd2fe2f
Content: Connect to a Synapse workspace using private links - Azure Synapse Analytics
Content Source: articles/synapse-analytics/security/how-to-connect-to-workspace-with-private-links.md
Service: synapse-analytics
Sub-service: security
GitHub Login: @Danzhang-msft
Microsoft Alias: danzhang

Thanks for the feedback! We are currently investigating and will update you shortly.

Thanks for leveraging Azure docs feedback channel to raise this question. This channel is reserved for ‘Azure docs’ feedback- for sharing doc feedback and suggesting content improvements. We didn't determine any changes for this documentation upon reviewing this feedback.
If your issue is specific to a use case, we request you to please reach out in forums like Microsoft Q&A or Stackoverflow. We have our engineers and other community members who will be happy to assist. Forums have a better suited audience for such queries. Once your issue is posted in forum, please do share the forum thread link here for tracking.
If you have a document which you are following and if you think it needs an update, please let us know about that here.
We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.
Thank you


updater: use oodi api directly

Currently updater accesses oodi api through proxy. Refactor it to make a direct connection. Requires proper setup of key and cert, see the proxy code:

See student number updater
https://github.com/UniversityOfHelsinkiCS/oodikone2-backend/blob/master/src/services/doo_api_database_updater/student_list_updater.js#L49
Access to cert and key is needed for this, currentyly found in
CERT_PATH=/home/tkt_oodi/oodikone.cs.helsinki.fi/clientCert.pem
KEY_PATH=/etc/pki/tls/private/050618.key

For container the access must me managed using a volume mapping. For staging directory /home/tkt_oodi/oodikone.cs.helsinki.fi/staging/data is mapped to /data
Simillar mapping should be done for production docker-compose.yml


WIP: Add OptionsFormatter class and update to allow auto-logging of Options

This PR adds an OptionsFormatter to the ConfigOptions.  This class allows the options to be written and read in different formats.  Three formats initially supported for write are the existing Options-file properties format, the ";"-separated list of items, and writing the options to the log file.
By adding a LogOptionsFormatter, the system will be able to automatically log options to the file as they are added or removed.  Currently, this involves a secondary step of updating the Dump method which is sometimes missed.  Furthermore, some sub-classes may be skipped or not have their options printed at all.

@mrambacher - Please update when you have completed updating the branch. I will review only once the branch has all of the contents you intend to merge. Thanks

Why is the title "WIP: ..."?

@ofriedma, @mrambacher - Please complete the review and all required activities so this may be ready for merge. Thanks

@ofriedma, @mrambacher - Please complete the review and all required activities so this may be ready for merge. Thanks

Will review after #648 review

@ofriedma #648 is merged now. Please review this one. Thanks

Please resolve conflicts  @mrambacher

@ofriedma - Are you done reviewing the PR?

@ofriedma - Are you done reviewing the PR?

waiting for a fix to my comments

@ofriedma - Are you done reviewing the PR?

waiting for a fix to my comments

Comments addressed.  Please review again

@mrambacher

@mrambacher There is a failure in the compile process

License check issue.  Believe it should be fixed or a bug in the license check for files that are not licensed.

@mrambacher there is a compile error here:
../db/db_options_test.cc:64:12: error: local variable 'mutable_map' will be copied despite being returned by name [-Werror,-Wreturn-std-move]
return mutable_map;
^~~~~~~~~~~
../db/db_options_test.cc:64:12: note: call 'std::move' explicitly to avoid copying
return mutable_map;
^~~~~~~~~~~
std::move(mutable_map)
../db/db_options_test.cc:76:12: error: local variable 'mutable_map' will be copied despite being returned by name [-Werror,-Wreturn-std-move]
return mutable_map;
^~~~~~~~~~~
../db/db_options_test.cc:76:12: note: call 'std::move' explicitly to avoid copying
return mutable_map;
^~~~~~~~~~~
std::move(mutable_map)


adding a new endpoint for adding a paper to mod corpus and fixed the file_upload endpoint

to update the WFT properly.

Thanks @valearna


add srvicing tag

I'm not sure if PreReleaseVersionIteration needs to be set but I can wait and see.

[11/02/2021 10:42:33] Done.
Successfully created package 'C:\Users\test\github\dotnet-msquic\artifacts\packages\Release\NonShipping\System.Net.MsQuic.Transport.6.0.0-servicing.21551.2.nupkg'.

Nice! Thanks 😄


Change Twitter Bootstrap tabs automatically?

I am looking to have the Twitter Bootstrap tabs change on a timed sequence. I am using them kind of like a carousel. I would like the tabs to change to the next one every 10 seconds.
Here is an example: http://library.buffalo.edu
Click on the news stories to see what I mean. Any help would be appreciated.

May be this is exactly what you are looking for:
Create responsive Bootstrap nav tab carousel
http://rongmeibible.worthyofpraise.org/create-responsive-bootstrap-nav-tab-carousel/

@strapboot Please avoid necroposting.


Hard code class names for RDF::Reader & Writer tests

The reader and writer tests called described_class to define their target object. This was causing problems in "an RDF::Enumerable" examples, where described_class changes to RDF::Enumerable.
This hard codes the class, instead of using RSpec's lambda.

Good to know.


Fix inject debugger issue on react native v0.59 onward.

Necessary fix, please merge into new version ASAP


Error with inherited state in htmlbars template

I get the following error with the Ember addon when I try to extend a block with an state:

TemplateError: Can not apply multiple states at the same time from the exclusive state group ":scope[state|error]"

/* block-1.block.css */
:scope[state|error] {
    ...
}

/* block-2.block.css */
@block-reference block1 from "block-1.block.css";
:scope {
    extends: block1
}

/* block-2.hbs */

When the state is not inherited, the build is fine:
/* block-1.block.css */
:scope[state|error] {
   ...
}

/* block-1.hbs */

Hm. Thats interesting. Will take a look asap, but it may be another week before I can get it on my docket. Think you could pop up a branch with a failing test case?

Sure! This case fails:
https://github.com/dacoho/css-blocks/commit/b88d28caf7b4c5620a427bd8093a9968cd6025d9

Hi @amiller-gh, I've been reviewing this and have found that in the glimmer analysis, inherited static and dynamic attributes are being added to each element several times - once for each of the block classes.
That happens in this loop:
https://github.com/linkedin/css-blocks/blob/7079a5320f2b29f31e246c0c1476de40af35512c/packages/%40css-blocks/glimmer/src/ElementAnalyzer.ts#L218
I think this may be helpful :)

Hey @dacoho, sorry for the slow reply – great sleuthing. That seems like a solid solution, but can we maybe add the existence check to addDynamicAttr so Analyzers don't need to worry about this logic? My hope is for Analysis integration to be as straightforward as possible, if we already made this mistake in one of our own analyzers then it sounds like something that @css-blocks/core can help validate for us!
We can keep a faster model for style presence check on ElementAnalysis instead of iterating through addedStyles each time. addDynamicAttr (as well as all the other style application methods: addDynamicClasses, addStaticClass, addDynamicGroup, addStaticAttr) would use that to check if a Class or Attr has already been added. If yes, no-op, if no, add the Style(s).

I'm seeing an error with states and Glimmer component in an ember-engine that may be related.

Cannot select attributes other than states: :scope[is-on]

Component block CSS
:scope {
    font-weight: bold;
}

:scope[is-on] {
    color: red;
}

Glimmer Component hbs.
<button block:scope block:is-on={{isOn}} {{on "click" this.toggleOn}}>
    {{yield}}
</button>

"@css-blocks/ember-cli": "^0.24.0"
"ember-cli": "~3.14.0"
"ember-engines": "^0.8.3"
"@glimmer/component": "1.0.0-beta.3"

@GCheung55 You're using the new syntax for 1.0 but the version of css-blocks that you've got installed is 0.24. You can either install @css-blocks/ember-cli@next or use the older syntax in your glimmer templates. I'm working on a new release for @next with a lot of bug fixes right now... it should be out in the next day or two.

@chriseppstein switching to 1.0 worked for me. Thanks!


folder and image name changes

Last commit is good - only change on last commit was a Mac OS hidden system file.


Object type SUSH not supported, serialize

Object type BOBF not supported, serialize
Import of object ZTT_I_PROJECTS failed
Object type BOBF not supported, serialize
Import of object ZTT_I_TASKS failed
Object type SUSH not supported, serialize
Import of object D5F4A85E043C2ECB975350C9BF5299HT failed

You need to install first abapGit plugins repository.


Make host answer page look better

After players answer their guesses and the correct answer are displayed on the host screen but does not look great

Before

Looks pretty good now

https://github.com/hto-projects/friendpardy/pull/104


Facade Design Pattern

Façade design pattern creates a single class that connects to other subsystems and creates simple method calls that then call to the other classes.  One spot that we see this could be used in our project would be by creating a façade for our JDBC classes.  right now we have 3 JDBC classes: JDBCInsert, JDBCSelect, and JDBCUpdateDelete.  Say we wanted to save a favorite food and then view the favorite food.  Right now to save the favorite food we make a call to the JDBCInsert class to save the food in the database.  Then to view the favorite food we make a call to the JDBCSelect class.  Instead with a JDBCFacade class, we could make a call to the JDBCFacade class to save the food which would use the instance of the JDBCInsert to save the food.  We would then be able to use the same JDBCFacade class to then get the favorite food from the database as it would use the instance of the JDBCSelect to get the favorite food from the database.


Issue with loading OPF images from the gallery

What

https://images.openproductsfacts.org/images/products/332/968/019/7938/2.jpg

Fixed by @stephanegigandet server side


Silent failure

Gitfs seems to fail silently in a lot of cases.
e.g. if the mount path does not exist
Shouldn't there be some output if startup fails?

Hi @IvanMalison, thanks for the bug report.
Indeed, there should be some useful output for every (startup) fail.
Feel free to post other usecases with silent failures, links to code with unhandled errors or to submit a PR.


bootstrap: use portable printf instead of echo

The -n flag is not supported by the echo built-in of some versions
of sh. Use the portable printf instead.
Signed-off-by: David R. Bild<EMAIL_ADDRESS>

LGTM. Thanks @drbild !


typo in engines table

typo in the Engines table

thanks, i've signed my commit

Approved. Merging


FR: Add button to refresh "Extra Auto-Correct Source Files"

The documentation says:

The list of custom replacements is only loaded when the plugin first lints a file or when the file is added to the list of files that include custom misspellings

This means that making a change to a file that is already in the list of custom misspelling files will not work unless the Linter is reloaded or the file is removed and re-added to the list of custom misspelling files

It would be really nice if there was a button in the settings that just refreshed this list on demand. It'd be even nicer if it automatically refreshed when you modify the input files but a refresh button would be fine so you don't have to fully reload the application

This sounds fine to handle. I am not so sure about refreshing on change as that requires listening to change events. But something more akin to what Templater does where you have the refresh button to reload all the custom autocorrect files does make sense.

I am working on getting this added. I believe I have something working, but I will need to test it out and verify that it works. Once that is done, I can go ahead and get this merged. Hopefully it will be in the next release.

The change has been merged to master and should go out in the next release.


💡 [FEATURE] - Add Complete MongoDB resources

Idea Contribution

[X] I have read all the feature request issues.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Explain feature request
No response
Explain your solution
I want to add the  mongodb complete resources.
Any alternative approaches/features
No response
Additional Context
COuld you assign me @jf

no

Complete this @jfmartinz , Assign level and merge it.
thanks


(Frontend) Create comment system for posts

Create a mock system for comments that does not require a backend (but can easily use one once implemented).

Fully completed with backend implementation support included


5th party member won't show

I can't get my 5th party member to show. It shows 4 of them but not the 5th. They are 100% Owned by a player in the permissions. Any ideas?

Any errors on the console? Was the character selected by the player (i.e. the character's name shows up right by the player's name on the player's list on the bottom left)?

Fixed. It was the character hadn't been selected by the player and the default view was "All Selected". When I changed it to All then the 5th character appeared and when I logged in as the user and "selected" the character on login then it also appeared.
Thanks


chore(readme): update slack badge

A new inspection was created.

The inspection completed: No new issues


Install Instructions for CentOS 7.1

I created a gist that has instructions on how to get this working on CentOS 7.x. The only piece that is missing from the default repos is Yara and unrar. I created RPMs for those and put it into a Package Cloud, free repo (included in the instructions).
https://gist.github.com/dcode/a828034a883cc06634f9

Thanks! We've got something similar in the readme now. If you want to make any tweaks feel free to submit a PR.


Invoke capacitor sqlite plugin methods from Custom Plugin

Hi, I am working on implementing custom plugin and in that plugin I want to insert records into sqlite using capacitor sqlite plugin…
Is there any example, where we can call method of one plugin from another plugin…
I have tried, but while calling a method need to pass PluginCall object as parameter… for that i need to pass message handler, PluginCall(this.handler,pluginId,callbackid,methodname,data); what will be the values for this parameters?
Any idea will be helpful for this implementation…
Thanks in advance…
https://forum.ionicframework.com/t/invoke-capacitor-sqlite-plugin-methods-from-custom-plugin/192713

@jothipandiyanjp Thanks for looking at the plugin, I have been looking at your question and tried some ideas. I do not think it is feasible as the PluginCall wraps a call from the web layer to native where in fact you would like to access it directly in the native part of your plugin meaning not going through the bridge. So the only way i can think off is to define for the capacitor-sqlite plugin two interfaces one for being call through the bridge and one to be call from native.

@jepiqueau Thanks for your reply. Will you add this requirement in your future enhancement?

@jothipandiyanjp I am not sure as this requires a complete rewrite of the plugin and on top of this, even if you install the capacitor-sqlite plugin in the custom plugin it seems with my first attempt that you need also to install it in the app using the custom plugin, installing only the custom plugin is not enough.

@jothipandiyanjp There is also the question of handling the error messages raised by the capacitor-sqlite in your custom plugin. Are you sure that you cannot develop your app by using the two plugins. The last way would be to take the cdssUtils folder and implements it directly in your custom plugin define mDb = new SQLiteDatabaseHelper(context,dbName+"SQLite.db",encrypted,
inMode,secret,newsecret,1)
and after this to call directly the SQLiteDatabaseHelper methods
Voilà

@jepiqueau  closing this issue. I have added cdssUtils folder in my custom plugin and using it..

@jothipandiyanjp Ok, may be in the future i will do two interfaces, so it will be easier for you to update, tell me if you have some issues


Cannot generate schema from Box OpenAPI Spec

Attempting to convert from https://raw.githubusercontent.com/box/box-openapi/en/openapi.json
Validation of the OAS has been performed and passes using Spectral. The oas-validation package seems to be hanging up over a default of null in the specification at #/paths/~1tasks/post/requestBody/content/application~1json/properties/due_at (note that schema should come after /applications~1json/ which is strange)
due_at contains a default: null value which when changed to default: "" keeps the oas-validation error from occurring. However, the error "OpenAPI-to-GraphQL creation event error:  Cannot read property 'schema' of undefined" persists.
I didn't note any debug flags I could set. Any guidance on how to track this error down?

Had a quick look at this, some observations:

conversion fails at on point because during schema_builder.createFields() because the Tags fieldTypeDefinition does not have a subDefinition. There are multiple properties missing a subDefinition.

The tags property of File is defined as:

tags:
  allOf:
    - type: array
      example:
        - approved
      items:
        type: string
      minItems: 1
      maxItems: 100
      description: ...
    - x-box-field-variant: 2
    - nullable: false

Removing allOf and declaring the properties on tags does not generate an error.

The OAS is massive, 918kB in JSON. Can you break it down and isolate the offending parts? Would make debugging easier.


Add 5× settings for Ishar 1/2/3

These were configured as NTSC; I’m not in a location where I can produce 5× crop settings right now, but will set them to PAL in the meantime.

Fixed


Increasing P2H in a scenario with excess electricity should result in lower CO2 and higher renewables share

Issue https://github.com/quintel/etmodel/issues/2477 did indeed fix the "connection" between households_useful_demand_for_hot_water_for_houses_with_p2h and
households_flexibility_p2h_electricity.
However, the failed mechanical turk test which brought this bug to light still fails.
Increasing the P2H on live results in less energy use, lower CO2 emissions, less and higher share of renewables.
Increasing the P2H on Beta results in less energy use, equal CO2 emission and equal share of renewables.
I would say Live gets it right.
I'll dive into this. @antw and @ChaelKruip do you have ideas on the causes of this difference between beta and live?

Screenshots
Excess electricity without P2H (left Beta, right Live)

Excess electricity with P2H (left Beta, right Live)

Mechanical turk test
This issues originates from Mechanical Turk failure:
rspec ./spec/flexibility/flexibility_spec.rb:123 # Flexibility P2H In a scenario increasing the number of P2H units should decrease CO2 emissions
Scenario is made with excess electricity (10,000 turbines inland).
Increasing P2H electric boiler to 20% share should decrease CO2
In the Use of excess electricity chart this results in a decrease in excess electricity, but not in an increase of converted to heat for households.

I'll dive into this.

No need; I'm pretty sure I know what the problem is. I'll push a fix today.

No need; I'm pretty sure I know what the problem is. I'll push a fix today.

Sounds good :smiley:

Pushed a fix, but it's curently incomplete. It is still a problem that if you add P2H other hot water technologies will begin to show a deficit, since they're not accounting for heat demand which is being satisfied by P2H. Fixing that should be easy, but it's ETEngine we're talking about so of course it's anything but... 😣
I'll update this again once I have a fix for the deficits.

A fix is now in place. The heat chart will still show a deficit when using P2H (quintel/etengine#948) but I'm reasonably confident that there is no actual deficit in the graph. I will resolve this soon.


Docs: Implement Left side nav behavior

Once the common styling for left side nav is in place and can e reused for the rest of the sub pages the interaction can be started. After #717 is done. If I'm the one who takes this ticket I'd like to first discuss/brainstorm the possible solutions with @xak and/or @saadmhybris.

no longer valid.


Question about UUIDs vs Long (javaTypeForIDType)

Hello Etienne,
Our database schema is using Long IDs.
That's why I was setting javaTypeForIDType to true.
But I was wondering, if using random UUIDs is the better option.
Or does this make no sense, if the database/repository doesn't work with UUIDs?
Merci,
Markus

A UUID can be mapped into a Long.
So I guess you can have java code that uses UUID, and map it to a Long into that database.
But I don't know your project and its constraints.
Are you sure that all the Long IDs on your database can be mapped into a UUID, now and in the future ?
Etienne

A UUID can be mapped into a Long.
So I guess you can have java code that uses UUID, and map it to a Long into that database.
But I don't know your project and its constraints.
Are you sure that all the Long IDs on your database can be mapped into a UUID, now and in the future ?
Etienne

The idea was to create an UUID from the String
"::"
But as there is no way to reverse the direction and get the String back from the hash value of the UUID, I guess I have
either to store the UUID->rawUUIDString mapping in the service, the graphql is running in,
or to go back to javaTypeForIDType and see, if I really need UUID.
We thought UUID might be convenient in the Apollo client  when it comes to cashing.
Is my assessment correct?

The idea was to create an UUID from the String
"::"
But as there is no way to reverse the direction and get the String back from the hash value of the UUID, I guess I have
either to store the UUID->rawUUIDString mapping in the service, the graphql is running in,
or to go back to javaTypeForIDType and see, if I really need UUID.
We thought UUID might be convenient in the Apollo client  when it comes to cashing.
Is my assessment correct?

I can't answer to you on this point.
You're going way out of the scope of this plugin.

I can't answer to you on this point.
You're going way out of the scope of this plugin.


User Credential Prompt Unclear During CrystalDiskMark Installation

Describe the bug
When attempting to install CrystalDiskMark, a user credential prompt appears. After pressing "Yes," it is unclear what the user is expected to type, causing confusion and preventing the installation from proceeding.
To Reproduce
Steps to reproduce the behavior:

Go to Install
Click on CrystalDiskMark
Click Install/Upgrade
See error after download

Screenshots

+1
I'm having the same problem with Google Chrome on Windows 11, version 23H2. I've reinstalled WinGet, and it's now updated to version 1.8.1911.

Failed to install Google.Chrome.

I'm having this issue... Any hints?

same.


Don't apply effects directly from owned items to actors

Closes #2935
WIP:

[x] add a migration that does the following:

for any actors that have spells or spell scrolls with a disabled: false, transfer: true active effect, create a copy of that effect directly on the actor with the original effect as the source.
for all effects on a spell or spell scroll, disable the effect, disable transfer, and reset startTime/startRound/startTurn

[x] when applying an effect to an actor via the apply effect button in the chat, copy the effect to the actor, even if the actor owns the effect's item and transfer == true.
[x] when applying an effect to an actor, reset the effect's startTime to right now.
[x] update effects on spells/scrolls in compendiums and effects on spells/scrolls on actors in compendiums to have transfer: false

This looks good to me. I think 3.1 is the right target for this since the migration has a fairly large blast radius. We have some other AE-related things coming in 3.1 that should help contextualise this PR for users.

Thanks for taking care of this. I've approved it but it looks like it needs a rebase. Let me know if you're happy to do that, otherwise I can take care of it.

I can rebase it tonight!


Question: does client package assume the use of server package?

Hello,
Thanks for this library. First time user here, so apologies if I'm asking something dumb.
I was trying to determine from the docs whether the client library requires the use of the server library, or whether it could integrate with an existing SignalR backend.
Context: I already have a SignalR stream API that is using a channel rather than async enumerable. It doesn't seem as straightforward as I'd hoped to convert from a channel-based implementation to one based on async enumerable, and if I was to convert my backend, Fable.SignalR seems to only support the latter. So that got me wondering whether I could just integrate directly.

Hi @kentcb,
The library does indeed assume a matching backend implementation, as it assumes a very specific implementation of the hub, as well as when using MsgPack it requires a custom implementation of the IHubProtocol.
I haven't had to deal with this myself, but channels should be able to be used with this method.

Thanks for confirming


Tolerate missing leading slash for 3 part names

Description
Closes #27.
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@jthomas @joshuaauerbachwatson bumping to refresh on radar.

Closed by 0e1f6c6ecc768b1ef1c6362651a85032511ec22d.


Finish message after export is shown even if it is disabled in export settings

System information:

Operating system (distribution) and version win 10 x64
DBeaver version 6.2.3

Steps to reproduce, if exist:
1, Start export to CSV from some result query
2. Uncheck "Show finish message on finish" and start export -> finish message is shown anyway

Fixed

verified


Flutter Catalog App

Have you read the Contributing Guidelines ?
Yes
Application: Catalog App
Catalog App is a demo(UI) version of an e-commerce application where one can purchase items.
Fixes #393

Project Level :
Intermediate
Tech Stack :

Flutter(2.8.1)
Dart

Screenshot :

Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

@Kushal997-das Kindly review this PR brother.

@photon149 have a look !!


abpBrainExtraction crash

I just installed ANTsR and the problem I am having is with abpBrainExtraction.
I enter the following commands in R in terminal:

struct<-antsImageRead("/Users/Bryson/Data/ANTsR_test/MPRAGE.nii.gz")
tem<-antsImageRead("/Users/Bryson/Data/ANTsR_test/bet.nii.gz")
temmask<-antsImageRead("/Users/Bryson/Data/ANTsR_test/bet_mask.nii.gz")
bm<-abpBrainExtraction(img=struct,tem=tem,temmask=temmask)

And I get the following error:

Problem in arg list
see usage by calling antsApplyTransforms() w/o arguments
Error in thresholdImage(temmaskwarped, 0.5, 1) :
trying to get slot "dimension" from an object of a basic class ("numeric") with no slots

So I called antsApplyTransforms() w/o arguments, but it wasn't helpful to me:

antsApplyTransforms()
warpedimg<-antsApplyTransforms( fixed=img1 , moving=img2 ,
transformlist=list("my0GenericAffine.mat","my1Warp.nii.gz") )

For full mode: use standard ants call as in antsApplyTransforms full mode ...

antsApplyTransforms("-d","2","-i","r64slice.nii.gz","-o","temp.nii.gz","-r","r16slice.nii.gz","-t","./Z0GenericAffine.mat")
for full help:
use .Call( "antsApplyTransforms", .int_antsProcessArguments( c(list("--help")) ), PACKAGE="ANTsR" );

[1] 0

The "img" image is a MPRAGE from the ABIDE dataset.
The "tem" image is the bet brain extracted version of the same image(I wasn't sure what to put here).
And the "temmask" is the brain mask output from bet.
I've run the example in the manual and it runs fine. Also, I tried running abpBrainExtraction.R another time, and although it didn't give me good results (probably from a bad choice of the "tem" image), it did finish without errors. Any ideas on what I can do?

Can you provide the images you were using?
2015-06-10 17:14 GMT-04:00 BrysonBR<EMAIL_ADDRESS>
I just installed ANTsR and the problem I am having is with
abpBrainExtraction.
I enter the following commands in R in terminal:
struct<-antsImageRead("/Users/Bryson/Data/ANTsR_test/MPRAGE.nii.gz")
tem<-antsImageRead("/Users/Bryson/Data/ANTsR_test/bet.nii.gz")
temmask<-antsImageRead("/Users/Bryson/Data/ANTsR_test/bet_mask.nii.gz")
bm<-abpBrainExtraction(img=struct,tem=tem,temmask=temmask)
And I get the following error:
Problem in arg list
see usage by calling antsApplyTransforms() w/o arguments
Error in thresholdImage(temmaskwarped, 0.5, 1) :
trying to get slot "dimension" from an object of a basic class ("numeric")
with no slots
So I called antsApplyTransforms() w/o arguments, but it wasn't helpful to
me:
antsApplyTransforms()
warpedimg<-antsApplyTransforms( fixed=img1 , moving=img2 ,
transformlist=list("my0GenericAffine.mat","my1Warp.nii.gz") )
For full mode: use standard ants call as in antsApplyTransforms full mode
...
antsApplyTransforms("-d","2","-i","r64slice.nii.gz","-o","temp.nii.gz","-r","r16slice.nii.gz","-t","./Z0GenericAffine.mat")
for full help:
use .Call( "antsApplyTransforms", .int_antsProcessArguments(
c(list("--help")) ), PACKAGE="ANTsR" );
[1] 0
The "img" image is a MPRAGE from the ABIDE dataset.
The "tem" image is the bet brain extracted version of the same image(I
wasn't sure what to put here).
And the "temmask" is the brain mask output from bet.
I've run the example in the manual and it runs fine. Also, I tried running
abpBrainExtraction.R another time, and although it didn't give me good
results (probably from a bad choice of the "tem" image), it did finish
without errors. Any ideas on what I can do?
—
Reply to this email directly or view it on GitHub
https://github.com/stnava/ANTsR/issues/54.

for antsApplyTransforms , try ?antsApplyTransforms
the template should be the whole head, just like the raw T1
the mask should be the template's brain mask
this function is a fast approximation to https://github.com/stnava/ANTs/blob/master/Scripts/antsBrainExtraction.sh which has been evaluated more thoroughly
you might also try getMask which on occasion can work well despite its simplicity ....

also - just updated the documentation
https://github.com/stnava/ANTsR/blob/master/man/abpBrainExtraction.Rd#L10-L12

Thank you both for your quick responses. The link below has all of the files that I used originally and others which will be explained below.
https://drive.google.com/open?id=0B9oWACI5TJOEfm1KdzAxSG5IdlNZR3QyOEtGa3l2ZEVETHBUQXZyNDJGUGk5YXRNT2N4blk&authuser=0
First, in response to the extended explanation of the tem and temmask variables, I decided to re-run with the following code:

struct<-antsImageRead("/Users/Bryson/Data/ANTsR_test/MP1.nii.gz")
struct<-abpN4(struct)
tem<-antsImageRead("/Users/Bryson/Data/ANTsR_test/MP3.nii.gz")
tem<-abpN4(tem)
temmask<-antsImageRead("/Users/Bryson/Data/ANTsR_test/bet_MP3_mask.nii.gz")
bm<-abpBrainExtraction(img=struct,tem=tem,temmask=temmask)

Which resulted in the same error:

Problem in arg list
see usage by calling antsApplyTransforms() w/o arguments
Error in thresholdImage(temmaskwarped, 0.5, 1) :
trying to get slot "dimension" from an object of a basic class ("numeric") with no slots

I also tried getMask, but it did not give me a good result (also in the link).
One more question, how dependent is abpBrainExtraction on the quality of the template's brain extraction?

I'm getting the same issue...it looks like antsRegistration isn't actually
writing out the transforms it's supposed to.  From debugging:
Browse[2]>   mytx<-antsRegistration( tem, img, typeofTransform = c('SyN'),

initialTransform=initafffn )

Browse[2]> file.exists(mytx$invtransforms[1])
[1] FALSE
2015-06-11 11:50 GMT-04:00 BrysonBR<EMAIL_ADDRESS>
Thank you both for your quick responses. The link below has all of the
files that I used originally and others which will be explained below.
https://drive.google.com/open?id=0B9oWACI5TJOEfm1KdzAxSG5IdlNZR3QyOEtGa3l2ZEVETHBUQXZyNDJGUGk5YXRNT2N4blk&authuser=0
First, in response to the extended explanation of the tem and temmask
variables, I decided to re-run with the following code:
struct<-antsImageRead("/Users/Bryson/Data/ANTsR_test/MP1.nii.gz")
struct<-abpN4(struct)
tem<-antsImageRead("/Users/Bryson/Data/ANTsR_test/MP3.nii.gz")
tem<-abpN4(tem)
temmask<-antsImageRead("/Users/Bryson/Data/ANTsR_test/bet_MP3_mask.nii.gz")
bm<-abpBrainExtraction(img=struct,tem=tem,temmask=temmask)
Which resulted in the same error:
Problem in arg list
see usage by calling antsApplyTransforms() w/o arguments
Error in thresholdImage(temmaskwarped, 0.5, 1) :
trying to get slot "dimension" from an object of a basic class ("numeric")
with no slots
I also tried getMask, but it did not give me a good result (also in the
link).
One more question, how dependent is abpBrainExtraction on the quality of
the template's brain extraction?
—
Reply to this email directly or view it on GitHub
https://github.com/stnava/ANTsR/issues/54#issuecomment-111179765.

might try verbose output to get some insight
brian
On Thu, Jun 11, 2015 at 12:22 PM, bkandel<EMAIL_ADDRESS>wrote:

I'm getting the same issue...it looks like antsRegistration isn't actually
writing out the transforms it's supposed to. From debugging:
Browse[2]> mytx<-antsRegistration( tem, img, typeofTransform = c('SyN'),

initialTransform=initafffn )
Browse[2]> file.exists(mytx$invtransforms[1])
[1] FALSE

2015-06-11 11:50 GMT-04:00 BrysonBR<EMAIL_ADDRESS>
Thank you both for your quick responses. The link below has all of the
files that I used originally and others which will be explained below.

https://drive.google.com/open?id=0B9oWACI5TJOEfm1KdzAxSG5IdlNZR3QyOEtGa3l2ZEVETHBUQXZyNDJGUGk5YXRNT2N4blk&authuser=0

First, in response to the extended explanation of the tem and temmask
variables, I decided to re-run with the following code:
struct<-antsImageRead("/Users/Bryson/Data/ANTsR_test/MP1.nii.gz")
struct<-abpN4(struct)
tem<-antsImageRead("/Users/Bryson/Data/ANTsR_test/MP3.nii.gz")
tem<-abpN4(tem)

temmask<-antsImageRead("/Users/Bryson/Data/ANTsR_test/bet_MP3_mask.nii.gz")

bm<-abpBrainExtraction(img=struct,tem=tem,temmask=temmask)
Which resulted in the same error:
Problem in arg list
see usage by calling antsApplyTransforms() w/o arguments
Error in thresholdImage(temmaskwarped, 0.5, 1) :
trying to get slot "dimension" from an object of a basic class
("numeric")
with no slots
I also tried getMask, but it did not give me a good result (also in the
link).
One more question, how dependent is abpBrainExtraction on the quality of
the template's brain extraction?
—
Reply to this email directly or view it on GitHub
https://github.com/stnava/ANTsR/issues/54#issuecomment-111179765.

—
Reply to this email directly or view it on GitHub
https://github.com/stnava/ANTsR/issues/54#issuecomment-111192253.

several "interesting" things going on here ...

you are a victim of this issue which is rooted in ITK

good news is that issue will be resolved soon thanks to @hjmjohnson finally killing this evil curse

bad news is the fix is not yet in ....

more good news is that i added a verbose option to antsRegistration in ANTsR to help us debug issues like this more easily

bad news is there was another bug in antsRegistration (in ANTs, not ANTsR) which this issue uncovered ...

good news is that bug is fixed ....

finally - here is a workaround ( command line )

ImageMath 3 MP3close.nii.gz ClosestSimplifiedHeaderMatrix MP3.nii.gz

then do the work on the close version ... make a close version for each image.


miopenSetNdTensorDescriptorWithLayout() should support NHWC

@PeixuanZuo Currently miopenSetNdTensorDescriptorWithLayout() do not support the use of NHWC...

This is a bug or omission in TensorDescriptor::CalculateStrides(). It should observe tensorLayout == miopenTensorNHWC and calculate strides accordingly.
Originally posted by @atamazov in https://github.com/ROCmSoftwarePlatform/MIOpen/issues/2001#issuecomment-1452240988

If this support is not planned, then the library should MIOPEN_THROW.

[Attribution] @junliume @johnny-keker https://github.com/ROCmSoftwarePlatform/MIOpen/labels/bug https://github.com/ROCmSoftwarePlatform/MIOpen/labels/urgency_normal, no clear ideas about assignment.

Hi @carlushuang  and @JehandadKhan  could you take a look and suggest what might be the fix plan?

Ultimately we need to fix this API : miopenSetNdTensorDescriptorWithLayout(), I think there is no reason this API only support part of the layout since its name is ..WithLayout. Any bug we need to fix it.
Regarding to the steps for this plan:

in short term, fix the document that this API has bug for NHWC ( do we need this?)
in middle term, fix the bug in problem descriptor when calculating the layout to get a cache of the layout. (this is where NHWC layout may fail)
in long term, we need better unify the internal implementations. Currently 1) complete stride/len implementation. 2) enum for layout implementation. These 2 may have not synced together, and make us pain for maintain.  And there is a lot of discussions on this topic. #343 #767  #1532  #1946

Maybe I am wrong but it seems like fixing TensorDescriptor::CalculateStrides() as recommended at the topmost comment should resolve the issue.


Typo "Azure OpenAI service"→"Azure OpenAI Service"

https://learn.microsoft.com/en-us/fabric/data-science/open-ai
https://github.com/MicrosoftDocs/fabric-docs/blob/main/docs/data-science/open-ai.md
#PingMSFTDocs

Thanks, @hyoshioka0128 !


Flash messages do not work after loading a file from a page

We use oro platform version 2.6.*
Unfortently I can not check on new version of oro/platform.
We encountered an issue with Flash messages.
Steps to reproduce:

Download some file from the page, in our case, a link from the grid.
Open another page
Perform an action that redirects the user to another page and shows the user a flash message.

Result
Flash message isn't displayed
Expected result
Flash message is displayed on the current page
In our case, this is because, after clicking on the download link, the oroui/js/app/views/page/messages-view component receives the beforeunload event and switches to pageIsGoingToReload mode. As a result, flash messages are opened only on the next page after the one on which they should be shown.
Has anyone encountered such an issue and are there any solutions?
Thanks

Internal reference: BAP-19318


how to use the RGBD superversied on nerf?

Thanks for the excellent work！I am also researchring on how to use the RGBD superversied on nerf.  I read the code of DS-nerf ,but I found load_sensor_depth on load_llff.py actually didnot work，just like https://github.com/dunbar12138/DSNeRF/issues/14#issue-1001083662 .
So I wonder what should we change compaired with code of DS_nerf?  Could you provid us a tutorial so we can use RGBD-supervised loss in our project?
Besides,I wonder  what is simplices.npy use for? I cannot find the same file on DS-nerf.

In my project, I used synthetic depth maps (i.e. acquired by a NeRF-network and not by a sensor). This makes some things more manageable.
The depth maps I used have a range of values in [min_depth, max_depth]; therefore, loading them into the pipeline was straightforward. Have a look at this line of my load_blender.py file to bring them in the right format. For the llff data, it is similar.
Make sure you arrange them in the right order in your data loader, see link for my implementation for the use_batching=True case.
In the training loop extract the depth tensors link and use it for the computation of the loss here.
Not sure where you found simplices.npy?


Litmus Go: Add support to use go-runner(exp image) as kubelet kill helper image  (in place of pulling a seperate image)

Details:

Kubelet kill experiment pulls ubuntu:16.04 image as part of the helper image. This can be done with the go-runner image only we may not need to pull a different image for this as go-ruuner image has a base image of `ubuntu:bionic. We need to have systemd installed for running kubelet kill experiment

Some Key Points:

The Dockerfile for go runner experiment image is in litmuschaos/litmus-go repo.

The kubelet kill experiment is here in litmus-go which uses chaoslib to create helper pod.

We need to make the experiment working for the go-runner helper image.

Once it is resolved:

Raise a PR in litmus-go repository and don't forget to add the logs of the chaos pod and helper pod. For any help feel free to comment here.

@uditgaurav I want to contribute, I know a little bit docker. Can you guide me a bit in implementing this? Thanks.

sure @avats-dev feel free to try on this.
You can also join our slack community to know more about chaos engineering and litmus:
To join our slack please follow the following steps!
Step 1: Join the Kubernetes slack using the following link: https://slack.k8s.io/
Step 2: Join the #litmus channel on the Kubernetes slack or use this link after joining the Kubernetes slack: https://slack.litmuschaos.io/

i wanted to work can you assign it to me

any guide to set up litmus on okteto?

Please feel free to take this @jabhishek87  Oketo guide is here.

The go-runner requires systemd capabilities, which will increase its size. To address this, we can maintain a Ubuntu image in litmuschaos with reduced vulnerabilities by conducting a thorough scan.


fix: Update readme fixes #61

Codecov Report

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

@@         Coverage Diff          @@
##           master   #62   +/-   ##
====================================
  Coverage       0%    0%
====================================
  Files           1     1
  Lines         157   157
  Branches       11    11
====================================
  Misses        157   157

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 6158990...c30642e. Read the comment docs.

Hi @hippo00. I'm in the process of updating the example folder, but I've also run into an issue with the Text to Speech output, as I explain here.
I'm going to look more closely at that next week to hopefully get that resolved and updated, but I'm going to merge this for now as it's just documentation updates.


Update remote main

Hello @Qubiz @joostaafjes! Thanks for these changes. This repo is just a mirror of the code in our Gerrit repository located here: https://pigweed-review.googlesource.com/ That's where we accept new patches. Please read https://pigweed.dev/docs/contributing.html for guidelines on contributions. Also feel free to chat with us on our discord server if you have any questions: https://discord.com/invite/M9NSeTA

Thank you for these contributions! As mentioned by @AnthonyDiGirolamo, all Pigweed development occurs in our Gerrit instance. We can't take GitHub pull requests at this time, so I'll close this PR, but feel free to submit patches to https://pigweed-review.googlesource.com/.
See our Contributing and Getting Started docs for details. And feel free to reach out on our Discord with questions!


[docs] Fix CORS section in installation

CATEGORY
Choose one

[ ] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[x] Documentation

SUMMARY
Fix the extra CORS Dependency in Superset installation step.
TEST PLAN
https://superset.incubator.apache.org/installation.html#cors
ADDITIONAL INFORMATION
REVIEWERS
@michellethomas @mistercrunch @etr2460

Codecov Report

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

@@           Coverage Diff           @@
##           master    #8958   +/-   ##
=======================================
  Coverage   58.97%   58.97%
=======================================
  Files         359      359
  Lines       11333    11333
  Branches     2787     2787
=======================================
  Hits         6684     6684
  Misses       4471     4471
  Partials      178      178

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 1f6f4ed...c2661ff. Read the comment docs.

CORS is optional for Superset app. it's contributed by community:
https://github.com/apache/incubator-superset/pull/478
I guess most of users/companies don't need to handle cross-domain issue.

What is the purpose of this change? If the purpose is to describe the process when not installing Superset via pip, then I think it might be better to explain when this alternative approach is needed.

I feel confused by the current instruction. I am ok with all options .
Before:

After:


Initial launcher logic

What's new
Initial launcher logic
Some TODOs:

Probably cleanup some messy places, maybe (?) simplify some logic.
Classpath injection

Classpath injection

If we don't create a fatJar, we can easily manipulate this using -cp in  void launch(...) rather than using -jar


feat(component): added BranchCard

Added the BranchCard component to the molecules folder.
#29

updated with changes

updated with new changes


Make the async prober private

Problem
After we move from using the async prober to the composite prober, we should make the async prober private as it should be more of an implementation detail.
Persona:
Which persona is this feature for?
Exit Criteria
A measurable (binary) test that would indicate that the problem has been resolved.
Time Estimate (optional):
How many developer-days do you think this may take to resolve? 1
Additional context (optional)

https://github.com/knative-sandbox/eventing-kafka-broker/pull/3178
https://github.com/knative-sandbox/eventing-kafka-broker/pull/3130

/hold as this is not ready yet

/area control-plane

/assign


followNewTab is not effective

I'm submitting a ...
[ ] bug report
[ ] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary

 const browser = await puppeteer.launch({ headless: false, args: ['--window-size=1440,764', '--no-sandbox', '--disable-setuid-sandbox'] });
    const page = await browser.newPage();
    page.setViewport(defaultViewport);
    // 初始化录制器
    const recorder = new PuppeteerScreenRecorder(page, {
        followNewTab: true,
        videoFrame: defaultViewport,
        aspectRatio: '360:191'
    })
    // 开始录制，以日期为文件名
    recorder.start(`./screencast/${date}.mov`);

    await page.goto(baseUrl);
    await page.waitForTimeout(5000);

  const newPage = await browser.newPage();
        newPage.setViewport(defaultViewport);
        await newPage.goto(baseUrl);
        await newPage.waitForTimeout(4000);

    await recorder.stop();

it seems that page.on('popup') is not effective

Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)

I was playing with this library today and saw a bunch of issues including this one.  This was an easy fix and was just a programming error not covered by any tests.  I didn't add any tests, but fixed the issue.
Note that if you change the viewport size on your original page the output video is going to be based on that viewport size.  So if you use followNewTab and those new tabs have a different sized viewport, it will be squished into the viewport size of the original page.
I'm not sure exactly the intent/use-case of this feature as it isn't really documented (not to mention it didn't work)... but I would suggest only using this if you want to record a whole session starting with one window and then following tabs that are opened...where all tabs have same viewport size.
However as soon as a new tab is opened the new one is recorded.  If the original page opens multiple tabs the recording of the original window will stop and switch to each one as they are opened.  So lots of edge cases that may surprise end users of this lib.

could you tell me how would trigger page.on('popup'); Is that event defined By yourself?

It seems that page.on('popup') wasn't trigger on my computer


6229 subdomainless urls dont work with ips

This fixes #6229.

@jgoizueta , @javisantana could you CR this, please?

@jgoizueta , @javisantana ping

I can't say too much about the code but the solution looks good enough, not so many changes and fixes the issue.
One question: if I setup subdomainless urls and whatever.com as domain, can I access using bot ways, whaever.com and whaever.com ip address, right?

That kind of questions are the reason I've added you here ;-)
Yes, you can. In my tests I've kept localhost.lan in the configuration (avoiding IPs there) and tried with both localhost.lan and IP. config/initializers/session_store.rb change is caused by that, since using a fixed domain makes the cookie don't work for IP.

:es:

:thumbsup: I see no problem here


VM Support

Had talks at Kubecon 2023 with the KubeVirt team and we discussed using their stack to achieve this and it seemed to be a real possibility.


WIP: Compile resx files into CLR binary files (.resource)

This is a rough outline of the API I am looking at for the rule csharp_resx, that is responsible for implementing the behaviors of resgen.
This is a stripped down version of my working branch, as I wanted to get some feedback on the direction I am looking at for this.
Relates to #10

After looking into the satellite assemblies more, I think it'll be easier to work with if the single vs multiple resx are split into two different rules. That should make that API easier to work with when trying to assign identifiers to every resource.

Cool! I'm sorry I didn't get a chance to look at this in detail. I'll wait for the follow-up :)

P.S. a PR with just the API and an empty (print("todo")) impl could be a good first step. It will make the individual reviews smaller.


Different precompiled files with same code

When I precompile the same css in two servers I get differents css minimized. I don't know the reason of this issue,I have the same config (config.assets.css_compressor = :sass)
For example one part of the unpacked minimized css in server 1 is like this (the same that the real file) :

.c3-tooltip
{
    border-collapse:collapse;
    border-spacing:0;
    background-color:#fff;
    empty-cells:show;
    -webkit-box-shadow:7px 7px 12px -9px #777;
    -moz-box-shadow:7px 7px 12px -9px #777;
    box-shadow:7px 7px 12px -9px #777;
    opacity:0.9
}

Then in server 2 I get this (webkit-box-shadow and -moz-box-shadow were removed):

.c3-tooltip
{
    border-collapse:collapse;
    border-spacing:0;
    background-color:#fff;
    empty-cells:show;
    box-shadow:7px 7px 12px -9px #777;
    opacity:0.9
}

how are you getting that info?
Are you doing curl calls to 2 different servers and seeing different results?
Not sure how to simulate this issue, if you could provide a sample app would help out.
thanks

I have a loadbalancer with different servers and sometimes the stylesheets are were not loaded. I checked the names of the css compressed files and in one server I have a different fingerprint. Next uncompressing the files with (http://cssunpacker.com/) I realize that one of the servers when compress the css is removing some -webkit, -moz  and other syntaxis.

one server I have a different fingerprint

This means that the file content is not the same. Something is generating different files in your server.
If the code is the same and the version of the gems are the same I don't see why sprockets would generate different results. Make sure that your assets folders and dependencies are the same.

Yes, the compressed file in one server remove some lines as shown in the first comment. I just realize that most of the files that have this problem have ".css.scss" extension and files with .sass extension don't have this problem. Could be something related with this? I know that in the latest version of sass-rails the .css.scss extension is deprecated.

Don't think so. Given the same environment and the same code, I don't see how the files would be different. I'd make sure that the environment is the same in both servers first (gem versions, dependencies versions, assets code)

It's the same code and gem versions. The deploy in both servers use the same branch. In the server with the problem if I check the non precompiled assets It's exactly the same of the other server, but when I compress it, some attributes as shown in the first comment were removed.

Does that code being generated by some gem like compass or bourbon? Seems only the prefix styles are missing, but the box-shadow is there.

No, the only difference are that in the server of the problem I'm using AWS opsworks with some chef recipes and I don't have rbenv configured, It's possible that some of this is cousing this issue? (I use the default recipes of opsworks rails layer). The box-shadow is there in both servers, the compression remove the whole line "-webkit-box-shadow" and "-moz-box-shadow".Thanks, I would really appreciate any help or comments about this issue.

"-webkit-box-shadow" and "-moz-box-shadow"

In our sass code are these properties generated by some kind of mixin?
Like:
@include box-shadow(7px 7px 12px -9px #777);

Not having rbenv configured can be a reason for this yeah, if that means that global gems installed on the machine are affecting your application, but I'd guess bundler would protect you.

not, It's not generated by some kind of mixin, they are in some imports of scss from the application.sass with normal css syntax.

:confounded: so this is now even more weird. If it is just normal css syntax I can't see how this same file can be different in different servers.

yes is very weird. Finally I give up with this issue and decided to use the gem 'asset_sync' to upload the assets on s3.

RAILS_ENV=production rake assets:precompile on server dropped
  font-family: "ヒラギノ丸ゴ ProN W4", "Hiragino Maru Gothic Pro", "メイリオ", Meiryo, Helvetica, sans-serif
  -webkit-tap-highlight-color: rgba(0, 0, 0, 0)
  -ms-text-size-adjust: 100%
  -webkit-text-size-adjust: 100%

from html section
on local machine not

The same is happening to me  my css vendor prefixes are removing in production enviroment but not in my local machine. Here you can get more info http://stackoverflow.com/questions/32179803/rails-removing-css-vendor-prefixes-in-production-enviroment


UI need some fixes for translation with longer texts

I would post examples in the next comments and also do pull request with fixes for all of the mentioned problems :-)

Wallet icon is missing for Slovak translation in Firefox (in Chrome it works fine).

Text is overflowing tooltip popup.

Wallet modal tab header text will overflow in case that the translation text is long enough.

Issue solved by https://github.com/anon-real/sigma-usd/pull/35 and merged into master.


odontoblast terms

Add:
abnormal odontoblast morphology (child of abnormal dental pulp morphology MP:0013250)
DEF: any structural anomaly of the skeletogenic cell on the outer surface of the dental pulp that secretes dentin matrix, is derived from the odontogenic papilla, and develops from a preodontoblast cell
DEF SOURCE: CL:0000060, GO:0071895
Child terms:
abnormal odontoblast number
DEF: deviation from the normal numbers of skeletogenic cells on the outer surface of the dental pulp that secrete dentin matrix, are derived from the odontogenic papilla, and develop from preodontoblast cells
increased odontoblast number
DEF: greater than normal number of the skeletogenic cells on the outer surface of the dental pulp that secrete dentin matrix, are derived from the odontogenic papilla, and develop from preodontoblast cells
abnormal odontoblast differentiation
DEF: anomaly in the process by which a relatively unspecialized cell of neural crest origin acquires the specialized features of an odontoblast, a skeletogenic cell on the outer surface of the dental pulp that secretes dentin matrix, is derived from the odontogenic papilla, and develops from a preodontoblast cell
DEF SOURCE: CL:0000060, GO:0071895

Added:
abnormal odontoblast morphology MP:0030453
abnormal odontoblast number MP:0030454
increased odontoblast number MP:0030455
abnormal odontoblast differentiation MP:0030456


Use of std::ref and std::cref

When should std::ref() and std::cref() be used?
I believe they are used in std::thread(), std::bind(). Any other std:: functions they must be used with?
I think they invoke std::reference_wrapper.
But it seem mysterious that that pointers always work and without the above, references do not.

Coming in Ep380


CertificatePinner is pinning against public key rather than SPKI

It seems like CertificatePinner is extracting the public key and pinning against that, rather than pinning against the SPKI.
Pinning against SPKI is what Adam Langley's blog recommends to avoid misinterpretation attacks:

Also, we're hashing the SubjectPublicKeyInfo not the public key bit string. The SPKI includes the type of the public key and some parameters along with the public key itself. This is important because just hashing the public key leaves one open to misinterpretation attacks. Consider a Diffie-Hellman public key: if one only hashes the public key, not the full SPKI, then an attacker can use the same public key but make the client interpret it in a different group. Likewise one could force an RSA key to be interpreted as a DSA key etc.

Are there plans to move from public key to SPKI?
Thanks!

OkHttp uses the exact same pinning mechanism as Chrome.


[Question]: does it work with VIte?

I did it work with vite just changed the export for this one:
export const env: EnvType = { ...import.meta.env, ...window.env }
and in the import of html file:

forget that i found a fork for vite env and is working correctly https://www.npmjs.com/package/vite-inject-env

yes, I also found it later and used it, thanks for your answer


HttpClient throws exception form some (invalid?) www-authenticate header values

I'm experiencing this issue where HttpClient fails to parse some (invalid?) www-authenticate response header.
For example:
The result of curl -i -X POST https://api.thetvdb.com/login is:
HTTP/2 401
date: Mon, 29 Jul 2019 09:15:50 GMT
content-type: application/json; charset=utf-8
content-length: 26
set-cookie: __cfduid=d0a0c5ee3934fe0f08c3ddd136ab53e471564391749; expires=Tue, 28-Jul-20 09:15:49 GMT; path=/; domain=.thetvdb.com; HttpOnly
vary: Accept-Language
www-authenticate: JWT realm=jwt auth
x-powered-by: Thundar!
x-thetvdb-api-version: 2.2.0
expect-ct: max-age=604800, report-uri="https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct"
server: cloudflare
cf-ray: 4fddfd9498b5c2c2-FRA

{"Error":"Not authorized"}

When I try to post to the same url using HttpClient, I get an exception
import 'dart:io';

Future main() async {
  var httpClient = HttpClient();
  var request = await httpClient.postUrl(Uri.parse('https://api.thetvdb.com/login'));
  await request.close();
  httpClient.close();
}

HttpException: Failed to parse header value
#0      _HeaderValue._parse.expect (dart:_http/http_headers.dart:684:9)
#1      _HeaderValue._parse.parseParameters (dart:_http/http_headers.dart:764:15)
#2      _HeaderValue._parse (dart:_http/http_headers.dart:773:20)
#3      _HeaderValue.parse (dart:_http/http_headers.dart:626:12)
#4      _HttpClientResponse._authenticate (dart:_http/http_impl.dart:476:22)
#5      _HttpClientRequest._onIncoming (dart:_http/http_impl.dart:1093:25)
#6      _HttpClientConnection.send.<anonymous closure>.<anonymous closure> (dart:_http/http_impl.dart:1741:17)
#7      _RootZone.runUnary (dart:async/zone.dart:1379:54)
#8      _FutureListener.handleValue (dart:async/future_impl.dart:126:18)
#9      Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:639:45)
#10     Future._propagateToListeners (dart:async/future_impl.dart:668:32)
#11     Future._completeWithValue (dart:async/future_impl.dart:483:5)
#12     Future._asyncComplete.<anonymous closure> (dart:async/future_impl.dart:513:7)
#13     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#14     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#15     _Timer._runTimers (dart:isolate-patch/timer_impl.dart:391:30)
#16     _Timer._handleMessage (dart:isolate-patch/timer_impl.dart:416:5)
#17     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:172:12)

The www-authenticate header returned from the server may not be valid but it feels like HttpClient should fail gracefully so that callers still be able to get the response object and just skip authentication (just like the 'no credentials available' here).
Thanks,
Itay

I believe this works as intended.
The parser failed to understand the syntax because the response of www-authenticate is not valid. There is no way we can continue the parsing. It shouldn't have "auth" after the "real=jwt".
Here is an example (Section 3):
http://self-issued.info/docs/draft-ietf-oauth-v2-bearer.html and https://tools.ietf.org/html/rfc2617#section-2

We are facing the same issue as well. Do we have an alternative for this aside from using platform channels?

I'm facing the same issue. For 401s a server I need to interact with responds with the header: WWW-Authenticate: Bearer realm=<custom_realm_value>, error=invalid-token, error_description=Invalid bearer token.
Note that the value in realm can be any string and the header value contains spaces after commas. Which is completely valid based on my understanding.
Quoting from the RFC:

Upon receipt of an unauthorized request for a URI within the protection space, the origin server MAY respond with a challenge like the following: WWW-Authenticate: Basic realm="WallyWorld"

I don't think the HTTPClient should assume what a server will respond with. We are facing this issue which is blocking us from handling refresh tokens when a 401 response is received.


Follow-up improvements to @waku/app

Follow-up thing for #154:

[x] double check if execSync reliable enough - yes, we can safely use the one from nodejs;
[x] move examples in the repo to dedicated folder and remove usage of wakuExamples from package.json;
[x] check import ES modules - checked, we can move to it but I think it is better to refrain from it since this code used as it is by Node;
[x] configure ci to support this package; - the other thing to do is allow CI to publish new version of the package
[x] add support similarly to hardhat

NB: since I moved examples to dedicated folder - there is huge diff in the PR

Would it be better to keep the examples at the root of js-waku-examples repo and just create a new create-waku-app?

By moving everything I wanted to separate examples from other code and make it easier to identify where examples are, because when they are at the root we need to maintain a dictionary with example name and it's place in the root. We can keep them as before of course.

Is there a benefit in having the create app in same repo?

The only benefit is that in order to get them in create-waku-app we don't need to clone the repo but just take the examples and copy them into npm package.

I don't remember, so we package the examples code in the create app package? Or do we download the code from the repo upon invocation?

create-waku-package works like this: upon releasing it we clone files of the examples and push them into npm registry. If we decide to keep create-waku-app then I will need to write some code to clone examples from the repo. I would like to avoid this path since it increases distance between the create-waku-app and examples.

I tested locally and we are moving away from light-chat being the default template. eth-pm is selected by default.
Is it possible to have a menu where nothing is selected by default it can just displays (default: light-chat) so that the user can just hit "enter" and get light-chat
And then if they want they can scroll updown
Or having light-chat selected by default in the list. In this case, I think it's still be good to have a label default.
The idea being that when a user tries the first time, they don't have to make a decision.

I tested locally and we are moving away from light-chat being the default template. eth-pm is selected by default.
Is it possible to have a menu where nothing is selected by default it can just displays (default: light-chat) so that the user can just hit "enter" and get light-chat
And then if they want they can scroll updown
Or having light-chat selected by default in the list. In this case, I think it's still be good to have a label default.
The idea being that when a user tries the first time, they don't have to make a decision.

Good spot. I moved the light-chat option to the top.

have we considered writing the scripts in TS?

I thought of going with it - but it add unnecessary friction for simple package with two files.

have we considered writing the scripts in TS?

I thought of going with it - but it add unnecessary friction for simple package with two files.

I'd be in the favour of temporary friction writing code that is easy to write more code on top of in the future
cc @weboko


Thank you!

Just wanted to drop a big thank you for this crate! It saved me from some real nasty stuff. <3

@jared-mackey Thanks for reaching out and you are very welcome, I'm glad it can help you. If your code is open-source or available somewhere, I'd love to have a look. I'm interested in seeing exactly how people make use of it.

I want to do the same - thank you so much for this crate! :heart:

@lokegustafsson You are very welcome, and thanks for reaching out.
I'm glad you like the documentation, it a necessary part of a crate, but a real pain to make. Regarding the download number, I guess I'm just not the best salesman, but if you like, share and subscribe that would really help out this channel 😄


Add Inkbunny to Sources

I was just hoping that you wouldn't mind adding https://inkbunny.net/ to the list of sources seems like another good site to add to the growing list.  He didn't close it its a bot.

What "closed automatically" means.


Prevents future UT build issues addresses #2967

Description
Depending on the GitHub CI workflow executes it will invoke ./gradlew build which will run tests on random parameters every time, there are local-code's that will break a build one of them being az-Cyrl. The solution here was preventing (by explicitly writing the local to be set to en-US) Github actions building on these random parameters that will interrupt a build and lose time. Manual testing was done to prove that using this flag breaks a build, you can check the issue #2967 or you can run (locally) ./gradlew build -Dtests.local=az-Cyrl to see it breaks
TLDR:

This one line change prevents future build fails by manually changing the local code to avoid UT build failures on  -Dtests.locale=az-Cyrl if you get unlucky with that randomized build parameter

Related Issues
Resolves #2967
Check List

[X] New functionality includes testing.
[X] Commits are signed per the DCO using --signoff.
[X] Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Update: this is the specific bug location, but as noted above you should fix all of them.
https://github.com/opensearch-project/ml-commons/blob/6a6cac13ee1a009ea43a3bad147059578d2adc38/common/src/main/java/org/opensearch/ml/common/connector/ConnectorAction.java#L211

This seems like the wrong fix for the issue.
There's clearly a locale-based bug somewhere but instead of fixing it you're ignoring it for the entire build, even for tests unaffected by the issue. This would miss issues such as #1734
All of the failing tests in the linked issue are in the RestMLPredictionActionTests class, so if those tests require en-US locale that should be scoped only to those tests.
However, locale based bugs are very frequently using upper- or lower-case string transformations. This line has a toUpperCase() transformation without a locale and is very likely the bug that needs fixing by adding Locale.ROOT. (There may be others.)
https://github.com/opensearch-project/ml-commons/blob/6a6cac13ee1a009ea43a3bad147059578d2adc38/plugin/src/main/java/org/opensearch/ml/rest/RestMLPredictionAction.java#L132
A brief search of the entire repo shows several similar potential locale issues, you should probably fix them all!

https://github.com/search?q=repo%3Aopensearch-project%2Fml-commons touppercase()&type=code
https://github.com/search?q=repo%3Aopensearch-project%2Fml-commons+tolowercase()&type=code

If you want to go a step further, converting a string by upper/lower casing just for a comparison is inefficient as it creates a whole new string object which is immediately discarded. If done frequently this adds to the heap/GC load. Use .equalsIgnoreCase() when you can to avoid this extra object creation.

Hey Dan! Thanks for the insight, I agree with you I did feel skeptical making the change Ill get to work with updating the string methods to use Local.ROOT

Thanks for making the changes.

LGTM Someone should probably do an org-wide search for these. :)
(Edit: just did 172 of them! Holy cow!)

Thanks Dan, can you create issues in the impacted repos ?

LGTM Someone should probably do an org-wide search for these. :)
(Edit: just did 172 of them! Holy cow!)

Thanks Dan, can you create issues for the impacted repos ?

Yep, sometime next week.   Filtering to just Java files it's only 66.


Fix plugin

fixed the errors I encountered when running ‘discover_position’, and added the missing tests.
However I found no way to get the filtering to work correctly, nor dap.

Thanks for the fixes! If the 'not getting the filtering work' you describe was the same issue I was having, this PR into your fork might help.


Dev server will return index when file not exists

Describe the bug
request a not existed file will receive /index.html.
Reproduction
https://stackblitz.com/edit/vitejs-vite-vlvqcb?file=index.html&terminal=dev
Steps to reproduce
just visit https://vitejsvitevlvqcb-s5sf--5173--28a8ce1b.local-corp.webcontainer.io/aaa.js
System Info
System:
    OS: Windows 11 10.0.22635
    CPU: (8) x64 11th Gen Intel(R) Core(TM) i5-1135G7 @ 2.40GHz
    Memory: 2.30 GB / 15.84 GB
  Binaries:
    Node: 22.3.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.19 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 10.8.1 - C:\Program Files\nodejs\npm.CMD
    pnpm: 9.7.1 - ~\AppData\Roaming\npm\pnpm.CMD
  Browsers:
    Edge: Chromium (127.0.2651.8)
    Internet Explorer: 11.0.22621.3566
  npmPackages:
    @vitejs/plugin-react: ^4.3.1 => 4.3.1
    vite: ^5.4.5 => 5.4.5

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I would assume this is how SPA fallback works normally https://vitejs.dev/config/shared-options.html#apptype
For example, you can configure appType: "mpa" to avoid such fallback https://stackblitz.com/edit/vitejs-vite-gfcunr?file=vite.config.ts

Yep, this is an expected behavior.


Add custom model selection

New models for the F5-TTS are constantly being released, for a variety of languages (ex.: https://huggingface.co/sarpba/F5-TTS-Hun/blob/main/hun_v3/model_270000_hun_v3.pt) . I don't want to have to rewrite the code all the time, so it would be nice if a custom HF model (pt/safetensor) downloader and a model selector were included in this extension.

The vocab.txt file may also need a unique path, because in this repo it had to be replaced because of the special Hungarian accented letters:
https://huggingface.co/sarpba/F5-TTS-Hun/tree/main/hun_v4


fix order of call, disable animations, show always menu icon

E.g. when user is in page "/app/main" and clicks on "Find local chefs", he navigates to "/app/buyer" but the header would show the back button
Or if user is in page "/app/buyer", opens the side menu and navigates to "/app/settings" and then hits browser back, he would go back to "/app/buyer" but the header would have the back button only.
Now the menu icon is always shown


python37Packages.yapf: 0.27.0 -> 0.28.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/python3.7-yapf/versions.
meta.description for python37Packages.yapf is: '"A formatter for Python code."'.
meta.homepage for python37Packages.yapf is: '"https://github.com/google/yapf"

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
/bin/sh passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 0.28.0 with grep in /nix/store/v6yv3dnzjqb4l5hig4jxpyjwaji3j58j-python3.7-yapf-0.28.0
directory tree listing: https://gist.github.com/ab6dade7753af62369f27f3cf507a8dd
du listing: https://gist.github.com/f0f121c7811d9c55e91551bc2322f3f9

Rebuild report (if merged into master) (click to expand)

33 total rebuild path(s)
9 package rebuild(s)
9 x86_64-linux rebuild(s)
8 i686-linux rebuild(s)
8 x86_64-darwin rebuild(s)
8 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
python27Packages.pyls-isort
python27Packages.python-language-server
python27Packages.yapf
python37Packages.pyls-black
python37Packages.pyls-isort
python37Packages.pyls-mypy
python37Packages.python-language-server
python37Packages.yapf
vimPlugins.vim-yapf

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/v6yv3dnzjqb4l5hig4jxpyjwaji3j58j-python3.7-yapf-0.28.0 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A python37Packages.yapf https://github.com/r-ryantm/nixpkgs/archive/fb71cabd46ae6a83ad33d225a0a724f4ab095ae0.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/v6yv3dnzjqb4l5hig4jxpyjwaji3j58j-python3.7-yapf-0.28.0
ls -la /nix/store/v6yv3dnzjqb4l5hig4jxpyjwaji3j58j-python3.7-yapf-0.28.0/bin

cc @siddharthist for testing.

I tested the binary and it appears to work fine.


Build on linux-aarch64

This would be nice to have for running Docker on Apple Silicon without Rosetta.

Ahh didn't see that this is related to #53.


MediaLibrary: User sub-library doesn't show recent items

When a user has Dashboard access with ManageOwnMedia permission (not ManageMediaContent), their media items are always placed under the \Users\<username> folder structure. When such a user goes to /Admin/Orchard.MediaLibrary, they should be able to see their recent media items, but the items are not displayed, even after clicking on "Recent".
The reason behind that is the following:

Users with ManageMediaContent permission see all the media items, so when they see the recent items, that means that their root folder is the absolute root (folderPath is empty).
Users with only ManageOwnMedia permission see only their own media items, so their root folder is the user media folder (folderPath is Users\<username>).
The logic that determines whether recent items need to be displayed compares the current folder to empty string. That is always a match for media admins (since the root folder for them is empty string) when they open up the Media Library and/or click on "Recent", but never matches for users with ManageOwnMedia permission, since their root folder is not an empty string.

I've tested the following implementation locally and works exactly as it should (only small additions, no breaking changes):

IMediaLibraryService extended with the GetMediaContentItemsRecursive and GetMediaContentItemsCountRecursive methods that do exactly the same as their non-recursive counterparts, but with recursive set to true when calling BuildGetMediaContentItemsQuery (btw that parameter was never used before, because the only time its value is true, it's never evaluated due to another condition check above it).
RecentMediaItems in the AdminController uses the new recursive methods.
The current root folder is passed on from the controller to the UI logic to be able correctly determine if the current folder is user's root folder or not (while not changing the behaviour for admins).

@sebastienros I suggest this to be added to 1.9.3/1.10.

I posted this here: https://github.com/OrchardCMS/Orchard/issues/6396
This is actually caused because hasFolderPath is set to true (since it's being passed in as an empty string in the view).  This line here: https://github.com/OrchardCMS/Orchard/blob/dev/src/Orchard.Web/Modules/Orchard.MediaLibrary/Views/Admin/Index.cshtml#L99 should use string.isNullOrWhiteSpace (because on initial load, it's "" by default, not null).  In fact, it will never be null.
https://github.com/OrchardCMS/Orchard/blob/dev/src/Orchard.Web/Modules/Orchard.MediaLibrary/Controllers/AdminController.cs#L44
It defaults to "".
I've tested on my local and confirmed that changing the line
https://github.com/OrchardCMS/Orchard/blob/dev/src/Orchard.Web/Modules/Orchard.MediaLibrary/Views/Admin/Index.cshtml#L99
to hasFolderPath: @(!string.IsNullOrWhiteSpace(viewModel.FolderPath) ? "true" : "false"), resolves the issue for me on 1.10

I'd rather go with IsNullOrEmpty, but the reason behind the fix is correct - it does fix #6396, but not this one. That is because calling the API for recent items is based on the assumption that you need to load recent items when you are in the root (which is determined by checking folderPath against empty string). While that is true for users with access to the whole Media (ManageMediaContent permission), that is not correct for users restricted to their own Media folder (ManageOwnMedia permission) under Users\<username>.

Yup, you're right.

:shipit: :)

I posted a comment on the commit:
@BenedekFarkas @sebastienros
I have an issue now, with this latest change. My account is an administrator (and site admin). Prior to the change, I could see all media folders. Now I don't see any. I have the Manage Media role permission but not the Manage Own Media.
Is this as expected?

I'm testing this in our QA environment now. For some reason on Dev, I can't see any folders anymore... but on QA I still can, so going to test this update there.

I take it back, this is working as expected on QA.  Now to troubleshoot my dev environment!

I think I found what's causing your issue, but that also occurs before the change I applied as well (just tested on 1.10 before my last commit): if you use the navigation in Media Library, items and folder structure are not reloaded until you refresh the page.

It's weird because I was refreshing, clearing my cache, trying in a private browsing session, restarting debug, etc.

@sebastienros do you think it's an issue we should care about (see my previous comment)?

Is it only in 1.10?  The media library works in QA with your changes pushed out and in production without your changes pushed out... but on my local dev environment, I don't get the folder navigation list (even though the folders are there in Azure blob storage).

The change is only 1.10 according to Sébastien's instructions: since the IMediaLibraryService interface changed (with two additions), it's a breaking change if you have an implementation for that. I'll check locally with Storage Emulator!

Oh nice! good idea.

I did a test with a user that has ManageMediaContent, but not ManageOwnContent explicitly with Storage Emulator and it works for me.

Ok, that's good.  Hopefully that means it's only an issue with my dev environment and not the code.


replace hami.sh to hami.io

fix: https://github.com/Project-HAMi/HAMi/issues/177
use hami.io to replace hami.sh.

thank for fixing


True zero-downtime deployment by request draining

Currently there's no way of telling Traefik that old container is going to be stopped, so it might route requests to a container that is shutting down. I'm creating this issue to track the progress of figuring out what is the best way of implementing this.
The problem was mentioned in this StackOverflow question: https://stackoverflow.com/questions/75918681/how-to-avoid-downtime-when-using-docker-rollout-with-traefik
Current idea
The easiest way seems to fail healthchecks before the container is going to be stopped, so Traefik is not routing new requests to the unhealthy container(s). This can be achieved by adding ! test -f /drain to container healthcheck, that is "fail if there exists a file named drain in /", and docker-rollout can create this file before stopping the old container.
I'm not sure if this behavior should be hardcoded in the tool, as there might be better ways of implementing request draining for proxies other than Traefik / nginx. Implementing hook support would allow docker-rollout users to implement true zero downtime deployment in two steps:

Add && ! test -f /drain to current container healtcheck in compose file
Add a hook like --before-stop "docker exec $1 touch /drain && sleep 10" to create the file manually

https://doc.traefik.io/traefik/middlewares/http/retry/
maybe this can help ?

I like the current idea, but wouldn't the "sleep" timout need to be at least the interval of the health check times the "retry" amount for the health check? Otherwise the container might not enter the "unhealthy" state before the shutdown sequence starts.
Maybe another option (specifically for traefik): in the --before-stop hook, add the label traefik.enabled=false. (note that I have not tested how quickly traefik would pick up this change).

Thats a useful insight, thanks! We should describe the requirements for sleep in docs if we go for the hooks solution, or wait a required time in docker-rollout itself.
Docker container labels are immutable (at least for now: https://github.com/moby/moby/issues/21721), so we cannot use them to deregister the container from Traefik.

I created a sample implementation in #36, it would be amazing if someone could test it with their setup to confirm it works correctly 😄


Reduce flakyness of configureNdkBuild* tasks by setting an

Summary:
This is a follow up to
https://github.com/facebook/react-native/commit/21dd646ecaf4f17cc62f18a4abf2f9cb88a72881
The ABI specific tasks were not involved in this patch, and they're still
failing. I'm fixing them.
Changelog:
[Internal] [Changed] - Reduce flakyness of configureNdkBuild* tasks by setting an
explicit dependency on prefab - part 2
Differential Revision: D34960946

quick check - @cortinico do we need this in 68?

quick check - @cortinico do we need this in 68?

Nope I don't think so, this is for CI only essentially AND only for new architecture users. I don't think it's a release blocker as no one mentioned this (I just noticed it in a couple of occasions on Circle CI). Worst case this can end up in a point release.