Undefined symbols for architecture x86_64

🐛 Bug Report

I’m unable to build our project for iOS with this library using Xcode 14.3 and React Native 0.72.7 (Android works as intended). I’ve tried both workarounds in the README and those on StackOverflow to no avail. I’ve also looked at the example project but haven’t spotted any differences that would cause this error.
Any help would be greatly appreciated. 🙂
To Reproduce

react-native run-ios
Build fails with error code 65

❌ Undefined symbols for architecture x86_64
❌   "_FBLinkable_NSProcessInfo_MacCatalystDetermining", referenced from:
⚠       _FBLink_NSProcessInfo_MacCatalystDetermining in libreact-native-fbsdk-next.a(RCTFBSDKSendButtonManager.o)
⚠       _FBLink_NSProcessInfo_MacCatalystDetermining in libreact-native-fbsdk-next.a(RCTFBSDKMessageDialog.o)
⚠       _FBLink_NSProcessInfo_MacCatalystDetermining in libreact-native-fbsdk-next.a(RCTFBSDKAuthenticationToken.o)
⚠       _FBLink_NSProcessInfo_MacCatalystDetermining in libreact-native-fbsdk-next.a(RCTFBSDKAppLink.o)
⚠       _FBLink_NSProcessInfo_MacCatalystDetermining in libreact-native-fbsdk-next.a(RCTFBSDKGameRequestDialog.o)
⚠       _FBLink_NSProcessInfo_MacCatalystDetermining in libreact-native-fbsdk-next.a(RCTFBSDKSettings.o)
⚠       _FBLink_NSProcessInfo_MacCatalystDetermining in libreact-native-fbsdk-next.a(RCTFBSDKGraphRequestManager.o)
❌   "_FBLinkable_NSProcessInfo_OperatingSystemVersionComparing", referenced from:
⚠       _FBLink_NSProcessInfo_OperatingSystemVersionComparing in libreact-native-fbsdk-next.a(RCTFBSDKSendButtonManager.o)
⚠       _FBLink_NSProcessInfo_OperatingSystemVersionComparing in libreact-native-fbsdk-next.a(RCTFBSDKMessageDialog.o)
⚠       _FBLink_NSProcessInfo_OperatingSystemVersionComparing in libreact-native-fbsdk-next.a(RCTFBSDKAuthenticationToken.o)
⚠       _FBLink_NSProcessInfo_OperatingSystemVersionComparing in libreact-native-fbsdk-next.a(RCTFBSDKAppLink.o)
⚠       _FBLink_NSProcessInfo_OperatingSystemVersionComparing in libreact-native-fbsdk-next.a(RCTFBSDKGameRequestDialog.o)
⚠       _FBLink_NSProcessInfo_OperatingSystemVersionComparing in libreact-native-fbsdk-next.a(RCTFBSDKSettings.o)
⚠       _FBLink_NSProcessInfo_OperatingSystemVersionComparing in libreact-native-fbsdk-next.a(RCTFBSDKGraphRequestManager.o)
❌   "_FBLinkable_UIPasteboard_FBSDKPasteboard", referenced from:
⚠       _FBLink_UIPasteboard_FBSDKPasteboard in libreact-native-fbsdk-next.a(RCTFBSDKSendButtonManager.o)
⚠       _FBLink_UIPasteboard_FBSDKPasteboard in libreact-native-fbsdk-next.a(RCTFBSDKMessageDialog.o)
⚠       _FBLink_UIPasteboard_FBSDKPasteboard in libreact-native-fbsdk-next.a(RCTFBSDKAuthenticationToken.o)
⚠       _FBLink_UIPasteboard_FBSDKPasteboard in libreact-native-fbsdk-next.a(RCTFBSDKAppLink.o)
⚠       _FBLink_UIPasteboard_FBSDKPasteboard in libreact-native-fbsdk-next.a(RCTFBSDKGameRequestDialog.o)
⚠       _FBLink_UIPasteboard_FBSDKPasteboard in libreact-native-fbsdk-next.a(RCTFBSDKSettings.o)
⚠       _FBLink_UIPasteboard_FBSDKPasteboard in libreact-native-fbsdk-next.a(RCTFBSDKGraphRequestManager.o)
❌ ld: symbol(s) not found for architecture x86_64
❌ clang: error: linker command failed with exit code 1 (use -v to see invocation)

I’ve also tried #import <FBSDKCoreKit/FBSDKCoreKit-Swift.h> instead of #import <FBSDKCoreKit/FBSDKCoreKit.h>, but I get a different error:
[shortboxed] Compiling AppDelegate.mm
❌ /Users/dsb/quicksilver-app/ios/shortboxed/AppDelegate.mm:32:12: use of undeclared identifier 'FBSDKApplicationDelegate'
  return [[FBSDKApplicationDelegate sharedInstance] application:application openURL:url options:options] ||
           ^
❌ /Users/dsb/quicksilver-app/ios/shortboxed/AppDelegate.mm:51:5: use of undeclared identifier 'FBSDKApplicationDelegate'
  [[FBSDKApplicationDelegate sharedInstance] application:application didFinishLaunchingWithOptions:launchOptions];
    ^

Expected Behavior

Build completes successfully.
Code Example

AppDelegate.mm:
#import "AppDelegate.h"

#import <Firebase.h>
#import <React/RCTLinkingManager.h>
#import <SDWebImageWebPCoder/SDImageWebPCoder.h>
#import "RNPaypal.h"
#import "RNSplashScreen.h"
#import "SDImageCodersManager.h"

#import <GoogleSignIn/GoogleSignIn.h>

#import <React/RCTBundleURLProvider.h>

#import <AuthenticationServices/AuthenticationServices.h>
#import <FBSDKCoreKit/FBSDKCoreKit.h>
#import <SafariServices/SafariServices.h>

@implementation AppDelegate

- (BOOL)application:(UIApplication *)application
            openURL:(NSURL *)url
            options:(NSDictionary<UIApplicationOpenURLOptionsKey, id> *)options
{
  if ([url.scheme localizedCaseInsensitiveCompare:self.paymentsURLScheme] == NSOrderedSame) {
    return [BTAppSwitch handleOpenURL:url options:options];
  }

  if ([[RNPaypal sharedInstance] application:application openURL:url options:options]) {
    return YES;
  }

  return [[FBSDKApplicationDelegate sharedInstance] application:application openURL:url options:options] ||
      [GIDSignIn.sharedInstance handleURL:url];

  return [RCTLinkingManager application:application openURL:url options:options];
}

- (BOOL)application:(UIApplication *)application
    continueUserActivity:(nonnull NSUserActivity *)userActivity
      restorationHandler:(nonnull void (^)(NSArray<id<UIUserActivityRestoring>> *_Nullable))restorationHandler
{
  return [RCTLinkingManager application:application
                   continueUserActivity:userActivity
                     restorationHandler:restorationHandler];
}

- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions
{
  [FIRApp configure];
  [[RNPaypal sharedInstance] configure];
  [[FBSDKApplicationDelegate sharedInstance] application:application didFinishLaunchingWithOptions:launchOptions];

  [BTAppSwitch setReturnURLScheme:self.paymentsURLScheme];

  /**
   * Deletes all Keychain items accessible by this app if this is the first time
   * the user launches the app.
   */
  // Checks wether or not this is the first time the app is run.
  if ([[NSUserDefaults standardUserDefaults] boolForKey:@"HAS_RUN_BEFORE"] == NO) {
    // Set the appropriate value so we don't clear next time the app is launched.
    [[NSUserDefaults standardUserDefaults] setBool:YES forKey:@"HAS_RUN_BEFORE"];

    NSArray *secItemClasses = @[
      (__bridge id)kSecClassGenericPassword,
      (__bridge id)kSecClassInternetPassword,
      (__bridge id)kSecClassCertificate,
      (__bridge id)kSecClassKey,
      (__bridge id)kSecClassIdentity
    ];

    // Maps through all Keychain classes and deletes all items that match.
    for (id secItemClass in secItemClasses) {
      NSDictionary *spec = @{(__bridge id)kSecClass : secItemClass};
      SecItemDelete((__bridge CFDictionaryRef)spec);
    }
  }

  // Register WebP format support
  [SDImageCodersManager.sharedManager addCoder:SDImageWebPCoder.sharedCoder];

  self.moduleName = @"myapp";
  // You can add your custom initial props in the dictionary below.
  // They will be passed down to the ViewController used by React Native.
  self.initialProps = @{};
  return [super application:application didFinishLaunchingWithOptions:launchOptions];
}

- (NSString *)paymentsURLScheme
{
  NSString *bundleIdentifier = [[NSBundle mainBundle] bundleIdentifier];
  return [NSString stringWithFormat:@"%@.%@", bundleIdentifier, @"payments"];
}

- (NSURL *)sourceURLForBridge:(RCTBridge *)bridge
{
#if DEBUG
  return [[RCTBundleURLProvider sharedSettings] jsBundleURLForBundleRoot:@"index"];
#else
  return [[NSBundle mainBundle] URLForResource:@"main" withExtension:@"jsbundle"];
#endif
}

@end

Podfile:
# Resolve react_native_pods.rb with node to allow for hoisting
require Pod::Executable.execute_command('node', ['-p',
  'require.resolve(
    "react-native/scripts/react_native_pods.rb",
    {paths: [process.argv[1]]},
  )', __dir__]).strip

# platform :ios, min_ios_version_supported
platform :ios, '13.0'
prepare_react_native_project!

# 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['NO_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

# Toggle this to true for the no-ad-tracking Analytics subspec
# https://rnfirebase.io/analytics/usage#disable-ad-id-usage-on-ios
$RNFirebaseAnalyticsWithoutAdIdSupport = true

target 'OneSignalNotificationServiceExtension' do
  pod 'OneSignalXCFramework', '>= 3.0', '< 4.0'
end

target 'myapp' do
  config = use_native_modules!

  pod 'react-native-image-crop-tools', :path => '../node_modules/react-native-image-crop-tools'

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

  use_react_native!(
    :path => config[:reactNativePath],
    # Hermes is now enabled by default. Disable by setting this flag to false.
    :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}/.."
  )

  target 'myappTests' do
    inherit! :complete
    # Pods for testing
  end

  post_install do |installer|
    # Mixing Swift and Objective-C in a react-native project may be problematic.
    # Workaround:  https://github.com/facebookarchive/react-native-fbsdk/issues/755#issuecomment-787488994
    installer.aggregate_targets.first.user_project.native_targets.each do |target|
      target.build_configurations.each do |config|
        config.build_settings['LIBRARY_SEARCH_PATHS'] = ['$(inherited)', '$(SDKROOT)/usr/lib/swift']
      end
    end
    installer.aggregate_targets.first.user_project.save
    installer.pods_project.targets.each do |target|
      target.build_configurations.each do |config|
        config.build_settings['EXCLUDED_ARCHS[sdk=iphonesimulator*]'] = "arm64"
      end
    end
    # https://github.com/facebook/react-native/blob/main/packages/react-native/scripts/react_native_pods.rb#L197-L202
    react_native_post_install(
      installer,
      config[:reactNativePath],
      :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|
        # Support for Xcode v14, which requires code signing
        # https://github.com/CocoaPods/CocoaPods/issues/11402
        config.build_settings["DEVELOPMENT_TEAM"] = "BXVH3243KA"
        if target.name == "BraintreeDropIn"
          # Fixes build issue with the Braintree Drop-in UI
          config.build_settings['IPHONEOS_DEPLOYMENT_TARGET'] = '9.0'
        end
      end
    end
  end
end

Environment

System:
  OS: macOS 13.6.1
  CPU: (8) x64 Intel(R) Core(TM) i7-1068NG7 CPU @ 2.30GHz
  Memory: 2.21 GB / 32.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 18.18.0
    path: ~/.nvm/versions/node/v18.18.0/bin/node
  Yarn:
    version: 1.22.19
    path: ~/.nvm/versions/node/v18.18.0/bin/yarn
  npm:
    version: 9.8.1
    path: ~/.nvm/versions/node/v18.18.0/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /usr/local/bin/watchman
Managers:
  CocoaPods:
    version: 1.12.1
    path: /Users/me/.rbenv/shims/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 22.4
      - iOS 16.4
      - macOS 13.3
      - tvOS 16.4
      - watchOS 9.4
  Android SDK: Not Found
IDEs:
  Android Studio: 2022.3 AI-223.8836.35.2231.10811636
  Xcode:
    version: 14.3/14E222b
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: <IP_ADDRESS>
    path: /usr/bin/javac
  Ruby:
    version: 2.7.6
    path: /Users/me/.rbenv/shims/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.72.7
    wanted: 0.72.7
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: true
  newArchEnabled: false

with apologies, others are building fine (as are my projects) so this will not result in a change in this repository and so is not an issue, it's a project-specific problem
you and others may feel free to continue discussion here (and propose a PR to the docs if anything interesting is learned) but I don't have any time personally to troubleshoot project-specific problem
I suggest you build an example from scratch (starting with npx react-native init BuildTest or similar and proving it can work, then figuring out why your project is not working


style(Label): update typings and remove propTypes

This PR is part of work for removing propTypes in production bundle (#524, #731).
Also, cleanups and updates typings for #1072.

Codecov Report

:exclamation: No coverage uploaded for pull request head (style/label-typings@d386c0b).

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 672feac...d386c0b. Read the comment docs.

💥

Released in<EMAIL_ADDRESS>


[Enhancement] Reduct the unnecessary ALWAYS_NULLABLE in nullable mode

Search before asking

[X] I had searched in the issues and found no similar issues.

Description
for many doris' functions, they have nullable mode ALWAYS_NULLABLE but in fact no need to be. many of them could be changed to DEPEND_ON_ARGUMENT or CUSTOM just like https://github.com/apache/doris/pull/27343, https://github.com/apache/doris/pull/22321 and https://github.com/apache/doris/pull/26827.
Solution
It may relate to 3 position: BE's function's implementation, function prototype of Nereids planner and function prototype of Legacy planner.
If could reduct it, reduct it.
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

l want try

l want try

sure! u can add my wechat torch-wood_ for more information


request for comments: gRPC-streaming API surfaces

gRPC streaming support has been done in GAPIC layer for nodejs, and it's in Speech API (as streamingRecognize method), but I'd like to know if there're some things to be polished for language-idiomaticity.
Note that this is for autogen layers, so API-specific conversions might not be possible (for example, separating audio config and audio data is very specific to streamingRecognize method, I think it's hard to generalize this pattern to other methods).
Cc: @stephenplusplus @callmehiphop @jmdobry
Here's what GAPIC is currently doing.
Bidi streaming
Both client and server has multiple objects. Typical example is the speech recognition -- client pushes the audio data asynchronously and server pushes the recognition results asynchronously.
This method does not take request parameter, but returns a readable/writable object stream.
var s = client.streamingRecognize()
  .on('data', function(data) { /* called every time a new item arrives. */ })
  .on('error', function(err) { /* called when an error happens. */ })
  .on('end', function() { /* called at the end of the stream. */ });

// writing requests.
s.write(req1);
s.write(req2);

// or pipe from another stream.
requestStream.pipe(s);

Response streaming (server-side streaming)
The client sends a single request object, and returns multiple objects. Typical example is query method in database like APIs (bigtable API has one, for example).
The method will take a request object as usual, but it returns a readable object stream instead of promise (or having callbacks).
var s = client.query(request)
  .on('data', function(data) { /* called every time a new item arrives. */ })
  .on('error', function(err) { /* called when an error happens. */ })
  .on('end', function() { /* called at the end of the stream. */ });

// has 'end' method to finish the stream in the middle.
s.end();

Request streaming (client-side streaming)
The client sends multiple request objects, the server returns a single response when it's done. Currently it's not used among Google APIs, but some may use in the future.
This does not take a request parameter but takes a callback, and returns a writable object stream.
var s = client.method(function(err, response) {
  if (err) {
    console.error(err);
    return;
  }
  // doThings(response);
});

// writing requests
s.write(req1);
s.write(req2);

// or pipe from another stream.
requestStream.pipe(s);

Thanks for writing this up. It all looks good, but I have a question on the last example:
var s = client.method(function(err, response) {
  if (err) {
    console.error(err);
    return;
  }
  // doThings(response);
});

The callback here is confusing to me. Working with streams, I expect to keep my event handling connected to the stream's event emitter. Can this instead emit error and response events in place of the final callback?
client.method()
  .on('error', function(err) {})
  .on('response', function(resp) {})
  .on('end', function() {});

I think it's okay to close this now.


stenc: init at 1.0.7

Motivation for this change
handle encrypted tape backups
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Running in production for years, working well. Thanks for merging!


[CS] Add a narrow hack for rdar://139234188

Currently we set FunctionRefKind::Compound for enum element patterns with tuple sub-patterns to ensure the member has argument labels stripped. As such, we need to account for the correct application level in getNumApplications. We ought to be setting the correct FunctionRefKind and properly handling the label matching in the solver though. We also ought to consider changing FunctionRefKind such that "is compound" is a separate bit from the application level.
Filed #77803 & #77804 to track properly fixing things here.
rdar://139234188

@swift-ci please test

@swift-ci please test source compatibility

@swift-ci please smoke test


We shouldn't add explicit newlines when writing errors

Steps to reproduce
On Windows
PS> throw ("1234567890"*([Console]::BufferWidth/10 + 10))

then with the mouse, select the text of the error message, paste on something like notepad
Expected behavior
no unnecessary line breaks
Actual behavior
line breaks at the buffer width
Environment data
Win10 any version

If I insert clipboard in Notepad after $error[0].Exception.Message | clip I don't see any extra newlines.
@lzybkr Please clarify that we should fix.

You're trying something other than what the repro steps say - and in this case, you will see different behavior.
Do follow my repro steps exactly (including running on Win10).

Dupe of https://github.com/PowerShell/PowerShell/issues/3813
I know this is older, but that one has more info.


Add new capability that allow purview scans to replace dummy entities

@rodrigomonteiro-gbb in the merge conflict could you keep the Kafka endpoint in the deployment script.

Closing the PR for now but will open back up once it's ready.


Opensource maintenance: Overflow

Grant category
OpenSource maintenance
Description
Set of go scripts to make it easer to run a story consisting of creating accounts, deploying contracts, executing transactions and running scripts on the Flow Blockchain. These go scripts also make writing integration tests of your smart contracts much easier.
Problem statement
Flow developers using Go to ...

tell a story
run a lot of repetitive tasks
run integration tests

Need evidenced by usage:

4 known projects using it
3 contributors
13 Github stars

Proposed solution
A Go library that makes it is to automate tests. Opensource repository available here: https://github.com/bjartek/overflow
Impact
Other Go developers can automate work and run integration tests.
Milestones and funding
Overall duration to deliver future milestones: ~6 months. Total FLOW requested: 28,000.

Milestone
Deliverables
Timeline
Risks
FLOW requested

0 - Ongoing maintenance and contribution
-
-
-
5,000 FLOW

1 - Support gas estimation
Add method for gas estimates, update CLI logging (https://github.com/onflow/flow-cli/issues/325)
~ 4 weeks
Need guidance from CLI engineers to improve logging
4,000 FLOW

2 - Consistent and complete API
Update library interface and add documentation
~ 2 weeks
-
1,500 FLOW

3 - Increase test coverage (https://github.com/bjartek/overflow/issues/19)
90%+ coverage
~ 2 weeks
-
2,000 FLOW

4 - V3 release
Prep, release, and announce at the Flow office hours and blog post
~1 week
-
500 FLOW

5 - Adoption
Get 10+ Flow projects to use Overflow
~4 weeks
-
5,000 FLOW

6 - Maintenance
Help resolve issues developers might face (not including new feature development)
3 months
-
10,000 FLOW

Team

Name
Role
Bio
Contact

Bjarte
Main contributor
-
@bjartek

Milestone completion

[x] Ongoing maintenance and contribution
[x] Support gas estimation
[x] Consistent and complete API
[x] Increase test coverage
[ ] V3 release
[x] Adoption
[x] Maintenance

I think this would make a great contribution to the community. Overflow basically saves me from tediously writing my own CLI, which is a huge time/effort saver. I hope we can get more people interested in it.

This grant proposal has been approved 🚀

Thank you <3

Overflow follow up grant
Almost all the milestones for the initail grant of overflow are done, I will release v1 soon and write a blogpost and hold an AMA about it.
In order to continue to maintain overflow I am applying for an follow up grant.
The followings milestones are things that I think would be very nice to have
== Continued supported.
In the three months since the grant launched I have helped many projects convert code from old overflow syntax to the new one. I have also commited fixes to onflow core contracts to show how the new syntax could improve the readability of the tests.
I would like to ask for a monthly fee to keep overflow a float.
The last grant gave out 10k flow (valued at 5 when the grant was given) for a total of 50k USD for 3 months. So 16666 USD per month. Since the initial support period is over I think that sum is a little high so what about 10000 a month.
This will include fixing small issues in the overflow repo that are tagged with small.
== Create memory snapshots and start from them
Flow cli has support for starting the emulator from a file. I would like to create some convenience around that so it is possible to set up a project/run an overflow story and create a memory dump of that. This can be very usfull as a debugging tool. You have an issue? can you create a dump for me?
It could also be used to create a dump of a test setup to enable very quick bootstrap of tests. The .find repo has 30 contracts now and the test overhead to start up a new emulator is seconds not miliseconds as it should be.
For this feature I would like to ask for 5000 USD
== Bootstrap new project and maintain it
When you start up a new project today you often want to get externally defined contracts and set up a bare bones skeleton. I propse to add features to overflow that enable it to be used to bootstrap the backend side of a flow project.
This includes:

creating a new github repo
get common shared code and put into project
set up NPM module of interactions (or work with interaction template)
diff the state of your local contracts with contracts on mainnet/testnet to see if your external dependencies have changed
offer simpler local version of some contracts on mainnet that is hard to use for testing USDC for instance

The end result is a tool that allow anybody that want to start with an NFT project using overflow to do so easy.
For this feature I would like to ask for 10000 USD

I belive this can be closed.


⚠ Cioos National CKAN has degraded performance

In fbb0097, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 5988 ms

Resolved: Cioos National CKAN performance has improved in 850b449 after 5 minutes.


Make this select required

I need to make required this field in a form via bootstrap like formvalidation do

I'm not sure what exactly your looking for but this isn't something this plugin or bootstrap-select does.  You can put validation on the original select (the one you use to call ajaxSelectPicker()).  Everything we do is on that original select and then we just refresh the bootstrap-select.
So not sure what you are looking for here?

Agreed, this project does not handle input element requirements nor any sort of validation. It's sole purpose is to provide the AJAX capabilities for the bootstrap-select plugin.

im looking for make one ajaxselectpicker required (mandatory) with formvalidation


Add support for LSP API

The latest IntelliJ releases add support for Language Server Protocol (LSP) for Plugin Developers.
Documentation is here and an example is here.
Would Biome be able to replace the TypeScript LSP?

The plugin already uses the LSP APIs

Would Biome be able to replace the TypeScript LSP?

That's the grand goal, yes


Mandatory Username and Password Requirement

First of all, I’d like to say how excited I am about the functionality of this plugin—it works very well and has been a great addition to my system. However, I’ve noticed a small issue that might need attention.
Currently, the plugin requires a username and password to be set, even in cases where they may not be necessary for the specific use case (localhost is allowed). This can create some friction for users who may not need or want to use these fields in certain configurations.
It would be fantastic if the username and password fields were optional, allowing for a more flexible setup depending on user preferences or system requirements.
I’ve already worked on a possible solution, and Ii like to share the code for it.
Looking forward to your feedback!
let options = {};
        if (this.username && this.password) {
            options = {
                headers: {
                    Authorization: 'Basic ' + btoa(this.username + ':' + this.password),
                }
            };
        }

Bye

Can’t test it now, but if you leave username and password empty it should work of authentication is switched off..

Just tested, works fine for a system without username/password. Just leave it empty


mediaelementplayer-events.html demo page

This page has js syntax error. On line 42 needs to have another ")" .. small bug

@owldesign This has been fixed; can you close this issue, please?


Update Volume Description in  Component to Specify Range

This pull request resolves issue #553 by updating the description field for the volume attribute in the <Audio/> component.
The new description clearly mentions that the volume range should be between 0 and 1. This clarification aims to prevent any confusion for users setting the volume and to avoid issues with unintended loudness.
Closes #553

Thanks @eerkmen ❤


[fix](alter inverted index) fix incorrect CreateTime of 'show alter' query result after fe restart

Proposed changes
Issue Number: close #xxx
Problem summary
For add or drop inverted index, when replay the logModifyTableAddOrDropInvertedIndices will new a schema change job, that has a new CreateTime, here should new a schema change job when not replay log.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 33.77 seconds
stream load tsv:          466 seconds loaded<PHONE_NUMBER>9 Bytes, about 153 MB/s
stream load json:         36 seconds loaded<PHONE_NUMBER> Bytes, about 62 MB/s
stream load orc:          66 seconds loaded<PHONE_NUMBER> Bytes, about 15 MB/s
stream load parquet:          27 seconds loaded 861443392 Bytes, about 30 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230223124516_clickbench_pr_102626.html


Add dependencies to UT instructions in README

It occurred to me while working on the UT/container stuff that our UTs require several dependencies that we don't document. Not hard to work them out from the error output, but mildly annoying.

Also python-pip, no?

Added python-pip

Thanks


Add reference to "subscribing to an event" in regards to += operator

Hi,
C# newbie here...
Coming from other languages I've always been used to += meaning "add to" and did a double-take when seeing the += operator used with a lambda on the right-hand side in the following example found at the bottom of the page:
public MainWindow()
{
    InitializeComponent();

    Loaded += (o, e) =>
    {
        this.Title = "Loaded";
    };
}

I had to go searching to find out how that works. It's explained in the C# reference on the += operator of course, but it would be helpful (for beginners like me) to copy/paste the blurb from that page over to the "Delegates and lambdas" page. I'm referring to this one:

The use of the += operator in this context is referred to as subscribing to an event. For more information, see How to: Subscribe to and Unsubscribe from Events.

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

ID: 36115425-3181-1363-dcbd-42e48db8613c
Version Independent ID: 57ca5426-2ae3-7260-cef9-e92dc9d4e856
Content: Delegates and lambdas
Content Source: docs/standard/delegates-lambdas.md
Product: dotnet
GitHub Login: @richlander
Microsoft Alias: wiwagn

+1
The doc is pretty clear until the last example.

Thanks @simshaun, that's a great suggestion! Would you like to submit a pull request for this?
We explain how to do that in the following article:
https://docs.microsoft.com/en-us/contribute/#quick-edits-to-existing-documents


User-reported unhelpful error message

Program: https://code.world/haskell#Pjw_uU0sXW4FucyYZN5_i3w
Comment:
FIX THIS PROBLEM
Message:
CodeWorld programs cannot read from the console.

Dup of #762


Error after run "npm install" command

download new version from "branches -> angular-version-update" and run "npm install" command in root
after install dependencies, hapend this error:

Hi @amin-pi,
What version of node and npm do you use?

Hi @amin-pi,
What version of node and npm do you use?

node:  v10.15.3
npm:  6.9.0

I used node version v10.16.0 and the same version of npm, everything works fine.
I can suggest you run npm run postinstall. It might help.

I use node version v10.16.0 and the same version of npm, everything works fine.
I can suggest you run npm run postinstall. It might help.

thanks,
I tested npm run postinstall, but did not make a difference.
There is still the same error.

@amin-pi, just test it on three different PC everything is work well. Maybe it is something with your environment.

@amin-pi try just to run npm i<EMAIL_ADDRESS>the problem should be here.

thanks @CubikNeRubik ,
i'm used the last commit on your branch.
it's ok. resolved the problem.


Please merge

https://github.com/solana-labs/governance-ui/pull/1965

https://github.com/solana-labs/governance-ui/pull/1965


Gradient Bug (Possibly to do with eigh).

We've been working on some code to simulate rigid bodies using JAX and came across a gradient bug (in the sense that jax.check_grads fails. Unfortunately, it was fairly difficult to minimize the repro so it's a bit long. I'm happy to iterate to try to narrow down the problem if it would be helpful.
Thanks very much for any help!
Here's the code, run using the most recent version of JAX (v 0.3.14) both on Colab and Desktop:
import jax.numpy as jnp
import jax.test_util as jtu
from jax import random
from jax import vmap, jit
from functools import partial

from collections import namedtuple

RigidBody = namedtuple('RigidBody', ['center', 'orientation'])

Array = jnp.ndarray

@partial(jnp.vectorize, signature='(q),(q)->(q)')
def _quaternion_multiply(lhs: Array, rhs: Array) -> Array:
  wl, xl, yl, zl = lhs
  wr, xr, yr, zr = rhs

  return jnp.array([
      -xl * xr - yl * yr - zl * zr + wl * wr,
      xl * wr + yl * zr - zl * yr + wl * xr,
      -xl * zr + yl * wr + zl * xr + wl * yr,
      xl * yr - yl * xr + zl * wr + wl * zr
  ])

@partial(jnp.vectorize, signature='(q)->(q)')
def _quaternion_conjugate(q: Array) -> Array:
  w, x, y, z = q
  return jnp.array([w, -x, -y, -z], dtype=q.dtype)

@partial(jnp.vectorize, signature='(q),(d)->(d)')
def _quaternion_apply(q: Array, v: Array) -> Array:
  if q.shape != (4,):
    raise ValueError('')
  if v.shape != (3,):
    raise ValueError('')

  v = jnp.concatenate([jnp.zeros((1,), v.dtype), v])
  q = _quaternion_multiply(q, _quaternion_multiply(v, _quaternion_conjugate(q)))
  return q[1:]

def transform(body: RigidBody, points: Array) -> Array:
  position, orientation = body
  return position[None, :] + _quaternion_apply(orientation, points)
transform = vmap(transform, (0, None))

def moment_of_inertia(points):
  ndim = points.shape[-1]
  I_sphere = 2 / 5
  @vmap
  def per_particle(point):
    diagonal = jnp.linalg.norm(point) ** 2 * jnp.eye(point.shape[-1])
    off_diagonal = point[:, None] * point[None, :]
    return ((diagonal - off_diagonal) + jnp.eye(3) * I_sphere)
  return jnp.sum(per_particle(points), axis=0)

def transform_to_diagonal_frame(shape_points):
  I = moment_of_inertia(shape_points)
  I_diag, U = jnp.linalg.eigh(I)
  shape_points = jnp.einsum('ni,ij->nj', shape_points, U)
  return shape_points

def _diagonal_mask(X: Array) -> Array:
  """Sets the diagonal of a matrix to zero."""
  if X.shape[0] != X.shape[1]:
    raise ValueError(
        'Diagonal mask can only mask square matrices. Found {}x{}.'.format(
            X.shape[0], X.shape[1]))
  if len(X.shape) > 3:
    raise ValueError(
        ('Diagonal mask can only mask rank-2 or rank-3 tensors. '
         'Found {}.'.format(len(X.shape))))
  N = X.shape[0]
  # NOTE(schsam): It seems potentially dangerous to set nans to 0 here. However,
  # masking nans also doesn't seem to work. So it also seems necessary. At the
  # very least we should do some @ErrorChecking.
  X = jnp.nan_to_num(X)
  mask = 1.0 - jnp.eye(N, dtype=X.dtype)
  if len(X.shape) == 3:
    mask = jnp.reshape(mask, (N, N, 1))
  return mask * X

def safe_mask(mask, fn, operand, placeholder=0):
  masked = jnp.where(mask, operand, 0)
  return jnp.where(mask, fn(masked), placeholder)

def energy_fn(body, shape_points):
  R = transform(body, shape_points)
  R = jnp.reshape(R, (-1, R.shape[-1]))
  dr_2 = jnp.sum((R[:, None, :] - R[None, :, :]) ** 2, axis=-1)
  dr = safe_mask(dr_2 > 0, jnp.sqrt, dr_2)
  e = 0.5 * jnp.where(dr < 1.0, (1.0 - dr) ** 2, 0.0)
  return 0.5 * jnp.sum(_diagonal_mask(e))

def shape_energy_fn(shape_points):
  body = RigidBody(
    jnp.array([[0.0, 0.0, 0.0],
               [0.5, 0.25, 0.15]]),
    jnp.array([[1.0, 0.0, 0.0, 0.0],
               [1.0, 0.1, 0.0, 0.0]])
  )

  shape_points = transform_to_diagonal_frame(shape_points)
  return energy_fn(body, shape_points)

points = jnp.array([[-0.5, -0.5, -0.5],
                    [-0.5, -0.5,  0.5],
                    [ 0.5, -0.5, -0.5],
                    [ 0.5, -0.5,  0.5],
                    [-0.5,  0.5, -0.5],
                    [-0.5,  0.5,  0.5],
                    [ 0.5,  0.5, -0.5],
                    [ 0.5,  0.5,  0.5]]))

# If these two lines are swapped then the test passes. The only
# difference between the two is whether or not the moment of inertia
# tensor (that is passed into `eigh`) has eigenvalues that are already ordered
# or whether they are out of order.
points = points * jnp.array([[1.0, 1.1, 1.2]])
# points = points * jnp.array([[1.2, 1.1, 1.0]])

jtu.check_grads(shape_energy_fn, (points,), 1)

This asserts with the error
Not equal to tolerance rtol=0.002, atol=0.002
JVP tangent
Mismatched elements: 1 / 1 (100%)
Max absolute difference: 0.5010784
Max relative difference: 1.2183625
 x: array(0.089806, dtype=float32)
 y: array(-0.411272, dtype=float32)

Thanks - if the gradient issue is with eigh, can we reproduce by calling check_grads on transform_to_diagonal_frame? That would pretty significantly reduce the size of the repro.

Ah, good point, it does! However, the reason I had earlier thought the error didn't show up with transform_to_diagonal_frame is that I had been testing on GPU. It seems like this bug might only show up on the CPU backend.
Here's the repro only with transform_to_diagonal_frame:
import jax.numpy as jnp
import jax.test_util as jtu
from jax import random
from jax import vmap, jit
from functools import partial

from collections import namedtuple

def moment_of_inertia(points):
  ndim = points.shape[-1]
  I_sphere = 2 / 5
  @vmap
  def per_particle(point):
    diagonal = jnp.linalg.norm(point) ** 2 * jnp.eye(point.shape[-1])
    off_diagonal = point[:, None] * point[None, :]
    return ((diagonal - off_diagonal) + jnp.eye(3) * I_sphere)
  return jnp.sum(per_particle(points), axis=0)

def transform_to_diagonal_frame(shape_points):
  I = moment_of_inertia(shape_points)
  I_diag, U = jnp.linalg.eigh(I)
  shape_points = jnp.einsum('ni,ij->nj', shape_points, U)
  return shape_points

points = jnp.array([[-0.5, -0.5, -0.5],
                    [-0.5, -0.5,  0.5],
                    [ 0.5, -0.5, -0.5],
                    [ 0.5, -0.5,  0.5],
                    [-0.5,  0.5, -0.5],
                    [-0.5,  0.5,  0.5],
                    [ 0.5,  0.5, -0.5],
                    [ 0.5,  0.5,  0.5]])

# If these two lines are swapped then the test passes. The only
# difference between the two is whether or not the moment of inertia
# tensor (that is passed into `eigh`) has eigenvalues that are already ordered
# or whether they are out of order.
points = points * jnp.array([[1.0, 1.1, 1.2]])
# points = points * jnp.array([[1.2, 1.1, 1.0]])

jtu.check_grads(transform_to_diagonal_frame, (points,), 1)

Hmm, I can't repro on CPU...

Thanks @sschoenholz for the more concise version! This repros for me on a Colab CPU runtime, as well as on my own macbook.

Thanks - if the gradient issue is with eigh, can we reproduce by calling check_grads on transform_to_diagonal_frame? That would pretty significantly reduce the size of the repro.

It's likely that in this example the bug is related with eigh. I open a new issue proving that eigh gradients can be wrong: https://github.com/google/jax/issues/10877


Additional information required on -PermanentlyClearPreviousMailboxInfo

If you use the -PermanentlyClearPreviousMailboxInfo as a parameter it does more than is described.
It will not only remove the user mail guid it will also delete the mailbox completely making it unrecoverable. Worse still it will delete an archive mailbox too. In the case of the archive mailbox it seems to leave the archive mailbox guid in place.
So it does clear the primary mailbox attributes but also permanently deletes the mailbox AND it permanently deletes an archive mailbox without removing the attributes.
Thus, it doesn't appear to work as described in the documentation.
To be fair there is a onscreen warning.

Delete all existing information about user “Jon@contoso.com"?. This operation will clear existing values from Previous home MDB and Previous Mailbox GUID of the user. After deletion, reconnecting to the previous mailbox that existed in the cloud will not be possible and any content it had will be unrecoverable PERMANENTLY. Do you want to continue?
However, the warning makes no mention of archive mailboxes. It should somewhere.

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

ID: 02635647-f6d7-5930-ddd4-1ca51577e066
Version Independent ID: 307e0743-17fa-dcd1-b642-15811e5171b8
Content: Set-User (exchange)
Content Source: exchange/exchange-ps/exchange/users-and-groups/Set-User.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@ponsaelius, OK, dire warning added.
Thanks for the feedback.
Closing


Specify Printer Columns

We can use kubebuilder annotations to specify additional columns to print with kubectl get requests. Although many of our resources use structs and lists that don't work well with this, GatewayClass.controller is one great example of a field we should add. https://book.kubebuilder.io/reference/generating-crd.html#additional-printer-columns

/assign

@robscott Is it safe to close this now?

Yep, thanks!
/close


Could you support mail gem 2.7.1?

I upgraded rails to 6.0.0 and I got the following bundler error because iso-2022-jp supports until mail 2.7.0. Could you support mail 2.7.1?
Bundler could not find compatible versions for gem "mail":
  In snapshot (Gemfile.lock):
    mail (= 2.7.1)

  In Gemfile:
    rails (~> 6.0.0) was resolved to 6.0.0, which depends on
      actionmailbox (= 6.0.0) was resolved to 6.0.0, which depends on
        mail (>= 2.7.1)

    rails (~> 6.0.0) was resolved to 6.0.0, which depends on
      actionmailer (= 6.0.0) was resolved to 6.0.0, which depends on
        mail (~> 2.5, >= 2.5.4)

    mail-iso-2022-jp (~> 2.0.8) was resolved to 2.0.8, which depends on
      mail (>= 2.2.6, <= 2.7.0)

@champierre
Some unit tests against mail 2.7.1 failed. Wait for a while, please.

@champierre Thank you for reminding me!

@kuroda Thank you very much for very quick response and action.


User completion in commit box is wrong when manually triggered

When typing @, I get suggestions for users, but once I typed a few characters pressing Ctrl D will show me issue suggestions.

I think if you scroll down in that issues list you'll see the users below the issues. Since it wasn't triggered by a trigger character we show both.

Correct, they are all there. But why is this useful behavior? In other words, why do all those suggestions appear as possibilities for the @sand input? Scrolling down, not even @sandy081 is highlighted in that case:

The bug seems to be that the filtering only happens from the moment the widget pops up (either automatically or manually). Notice this example:

The widget only filters on the moreno sequence that I type after manually opening it and even fails to properly render highlights on user names. The expected behavior is: when manually opening it, it should be filtered on my initial input (joao) and should further filter down as I type the rest (moreno). This is exactly how it works in the editor:

This occurs for all suggests, not just in the commit box.

@jrieken I think CompletionProvider would need to add some support for this. Is there a way for an extension to tell the suggest widget what text before the cursor should be used for filtering?

Is there a way for an extension to tell the suggest widget what text before the cursor should be used for filtering?

Yes, it the CompletionItem#range property.

In a codespace and desktop RemoteHub I get user suggestions in the commit box, but I can't trigger suggestions automatically or manually in web at all.

Opened https://github.com/microsoft/vscode-pull-request-github/issues/4119 to fix.


Exceptions occur when setZoomable(false) and setScaleType(FIT_XY) are used together

The picture shows only half

can you show example how you did this?


Bug in modifiers for icompares

@Siewart When creating an indicator, you have to define modifiers. These can only contain arithmetic symbols[+ \ - \ / \ * ][0-9], not doubles or information state values (so no $dialoguestate.length).

Hi Jelte, Siewart,
If this is relevant to the ARIA demos or milestones, can you put a relevant issue on the main ARIA repo? That way we can keep track of everything that needs to be done for all components of ARIA.
Dank,
Michel

It's not :)
This was originally developed as a (overly) flexible alternative to ORs in preconditions. So this is an extended feature that was not added.
Very low priority at this moment.


How to add proxy for the API call to openai

Great tool! However my default network has problem with connecting to openai.
I tried to add proxy to your script. In "openai/init.py", I add below code
#proxy = None
proxy = {
   'http': 'xxx:80',
   'https': 'xxx:80',
}

But seems still not working. Am I missing something? How can I debug it more efficient?
Need your insights, thanks a lot

Hey @mizeng!
Good question. Not really sure how it works. You could try something like this:
import openai

openai.api_key = os.getenv("api_key")
openai.proxy = # your proxy

In the text_chat.py file. But it should actually do what you were trying in the __init__.py file. Unfortunately, OpenAI's documentation is pretty bad.
Sorry that I could not help you more.

weird. below code snippet works for me.
import requests

url = "https://api.openai.com/v1/completions"

payload = "{\n  \"model\": \"text-davinci-003\",\n  \"prompt\": \"Say this is a test\",\n  \"max_tokens\": 7,\n  \"temperature\": 0\n}"
headers = {
    'authorization': "Bearer xxxx",
    'content-type': "application/json"
    }
proxy = {
   'http': 'sjd-entbc-001.corp.ebay.com:80',
   'https': 'sjd-entbc-001.corp.ebay.com:80',
}

s = requests.Session()
s.proxies = proxy
response = s.request("POST", url, data=payload, headers=headers)

print(response.text)

Strange. What error message you are getting if you use OpenAI's Python API?

goes to this exception , 'Error communicating with OpenAI: Proxy URL had no scheme, should start with http:// or https://'.
I tried add "http://" to the proxy value, but no use.
 except requests.exceptions.RequestException as e:
            raise error.APIConnectionError(
                "Error communicating with OpenAI: {}".format(e)
            ) from e

Found the issue, my fault. I didn't notice I set https_proxy in the workflow settings.

Thanks for investigating in this. Feel free to create a pull request implementing this.


Bug report: Transaction rejection error messages in sidecar.

The error messages in the sidecar related to the rejection of transaction from the node implies that execution of invalid transactions is being attempted, and failing. These should be reworked, in conjunction with changes to the error messages reported over the binary to facilitate more ergonomic use.

This PR is finished and merged here


Issues with current internal version of pipeline sky subtraction as used by DM team

arose from #58
investigating why patch boundaries are seen on stitching even with default pipeline
these were uncovered when setting up Las Vegas 360 sphere.

23/MAY/24

Robert said results indicated a bug but Aaron found  no bug.
Instead it appears the current sky subtraction is not working as DM are expecting

17/JUN/24

investigate the issue in other Photometric bands
only occurs in the red bands
checked other tracts - does not occur in every tract
submitted this to DM channel but no reply as yet.

09/JUL/24

no progress

23/JUL/24

on hold until @aemerywatkins is asked to progress since there are currently higher priorities

06/AUG/24

still on hold

24/SEP/24

still on hold
Lee says DM would like this implemented in the pipeline before 1st light with LSST Cam to enable pretty press (PR) images  so looks like it is merged into the background matching
for now though keep the tickets separate

22/OCT/24

Nate Lust working on this.
no requests from DM for Aaron to work on this
will be discussed more at Princeton meeting in December

12/NOV/24

see

28/NOV/24

agreed to close this ticket since it is effectively merged with #45 background matching


[ci skip] adding user @zwelz3

Hi! This is the friendly automated conda-forge-webservice.
I've added user @zwelz3 as instructed in #11.
Merge this PR to add the user. Please do not rerender this PR or change it in any way. It has [ci skip] in the commit message to avoid pushing a new build and so the build configuration in the feedstock should not be changed.
Please contact conda-forge/core to have this PR merged, if the maintainer is unresponsive.
Fixes #11

@johanneskoester I think this would have to go in after #10


Atomic Swap - additional logic for  "show token" button on create offer screen

According to the new design, we found some places where the user can copy token and leave creation screen without created transaction
For Example:
case 1: copy token and leave screen via button back
case 2:  copy token and leave screen via navigate to different screen
There're 2 ways how to deal with it:

Forbid to copy token on swap token pop-up and delete button copy, but it'll look strange

Add additional logic for  "show token" button
2.1 We need to add the same validator to "show token" button as it is for "copy swap token"
if at least one required field is blank  -  "show token" button should be disabled

2.2 After click "show token" button system should pre-prepare transaction to atomic swap, that means that after leave screen without clicking "publish offer", "copy swap token" we will create the transaction for swap
OR as an option instead of 2.2.
2.2  Forbid to copy token on swap token pop-up and add logic the same as for  "copy swap token" button  to "copy token" button, that means that after clicking on "copy token" button creation screen will be closed and transaction for swap will be created

fixed.
show token currently works only as information panel without the ability to copy token


snapcraft: fix build using golang 1.13+

golang 1.13 and 1.14 will both fail with different errors when
attempting go get commands.  Since full support of go.mod is
not yet available in snapcraft 3.9 (stable), this commit will
disable go modules until it can be enabled (v3.10+). Modules
are disabled by setting GO111MODULE=off in the build environment.
The current yaml specifies the go version in build-snaps.  However,
the go plugin has a property go-channel which sets the go snap
version.   The default value is latest/stable which is currently
go 1.13.  If a different version is specified in build-snaps, the
go snap will be updated/overridden to the version (implicitly) specified
by the go-channel.  An example log snippet shows this behavior:
<snip>
go (1.12/stable) 1.12.17 from Michael Hudson-Doyle (mwhudson) installed
snapd is not logged in, snap install commands will use sudo
go 1.13.8 from Michael Hudson-Doyle (mwhudson) refreshed
<snip>

This is why the behavior probably only recently manifested itself,
assuming the go snap's latest/stable was updated to point to 1.13.8.
Local builds would also work if using snapcraft v3.10 or newer (found
on candidate and edge channels).
In short:

Lock in the go version to 1.13 using go-channel.

Disable go modules until properly supported by snapcraft (v3.10+)
using GO111MODULE=off.

Signed-off-by: Chris Patterson<EMAIL_ADDRESS>How can you test this?
I tested on build.snapcraft.io using:
https://github.com/cjp256/worker/commit/e54cb08d4a439d96933b09300e76219050c350ca
With a build log:
https://build.snapcraft.io/user/cjp256/worker/850763
The build log shows that it builds, but does not release because I can't release your snap :wink:

@cjp256, thanks for the fix!


Adapt NetworkPolicys

How to categorize this PR?

/area security networking
/kind enhancement
What this PR does / why we need it:
This PR adapts the NetworkPolicys as following:

The custom pod label selectors for Prometheus is adapted similar to https://github.com/gardener/gardener/pull/7907/commits/b14706780cb74902404aa44351bd39124aee48ae.

Which issue(s) this PR fixes:
Related to https://github.com/gardener/gardener/issues/7352
Release note:

NONE

/cc @shreyas-s-rao @ashwani2k

ping @shreyas-s-rao @ashwani2k

Can we get a new release with this PR?

Sorry, I saw that this is already released with v0.6.0.


feat(compiler-sfc): improve utility type Partial and Required

The test cases were written in defineProps.spec.ts. If they were written in resolveType.spec, it would change many props assertions. It doesn't seem worth it.


add TBB to vcpkg

@a-zakir Can you expand why TBB need to be reintroduced?
TBB was introduced as a backend for parallel execution policy: https://en.cppreference.com/w/cpp/algorithm/execution_policy_tag_t
As per gcc10, execution policy should be supported by libstdc++ and shouldn't require TBB anymore.

@a-zakir Can you expand why TBB need to be reintroduced? TBB was introduced as a backend for parallel execution policy: https://en.cppreference.com/w/cpp/algorithm/execution_policy_tag_t As per gcc10, execution policy should be supported by libstdc++ and shouldn't require TBB anymore.

i don't know, i was told to do it, to restore lost perf


Issues with C++ unit tests

As described in README.md:
# Move to builddir.  Fill in <...> with the full path, e.g.
# `cmake.linux-x86_64-cpython-3.11`.
$ cd python/build/cmake<...>

# Run C++ unit tests.
$ ninja test

When I ran ninja test I encountered the error:
[0/1] Re-running CMake...
/bin/sh: 1: /tmp/pip-build-env-tqe5dgej/overlay/lib/python3.11/site-packages/cmake/data/bin/cmake: not found
FAILED: build.ninja

I think it was because pip install created a virtual environment to do the installation, and the absolute path of this temporary directory was written into build.ninja. After installation this directory was removed and the error occurred. Just use host ctest and it works well under this directory.
build CMakeFiles/test.util: CUSTOM_COMMAND
  COMMAND = cd /data03/home/hlzg/project/triton/python/build/cmake.linux-x86_64-cpython-3.11 && /tmp/pip-build-env-tqe5dgej/overlay/lib/python3.11/site-packages/cmake/data/bin/ctest --force-new-ctest-process
  DESC = Running tests...
  pool = console
  restat = 1

build test: phony CMakeFiles/test.util

Thanks @huanglanzhiguan. It fixed the problem for me.

I ran into the same issue. Could you elaborate more on how to fix it? (I am not familiar with Cmake nor Ninja)
Should I change ctest path from temporal one to host one on triton/python/build/cmake.linux-x86_64-cpython-3.11/build.ninja ? Is this enough?
Thank you for your help in advance.


fix: add a flag to save operations Notifications

Signed-off-by: harkiratsm<EMAIL_ADDRESS>Notes for Reviewers
This PR fixes #
Signed commits

[ ] Yes, I signed my commits.

@Revolyssup Context - https://app.clickup.com/t/2me48ef?comment=1429336115

Codecov Report
Base: 16.75% // Head: 12.52% // Decreases project coverage by -4.22% :warning:

Coverage data is based on head (00cc06c) compared to base (284be09).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6642      +/-   ##
==========================================
- Coverage   16.75%   12.52%   -4.23%
==========================================
  Files         115      115
  Lines       11229    11230       +1
==========================================
- Hits         1881     1407     -474
- Misses       8873     9509     +636
+ Partials      475      314     -161

Flag
Coverage Δ

gointegrationtests
?

unittests
12.52% <0.00%> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

server/handlers/meshery_pattern_handler.go
0.00% <0.00%> (ø)

mesheryctl/internal/cli/root/system/start.go
1.70% <0.00%> (-56.18%)
:arrow_down:

mesheryctl/internal/cli/root/system/status.go
1.12% <0.00%> (-53.94%)
:arrow_down:

mesheryctl/internal/cli/root/system/update.go
1.17% <0.00%> (-47.06%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
1.88% <0.00%> (-45.29%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
2.22% <0.00%> (-41.49%)
:arrow_down:

mesheryctl/internal/cli/root/root.go
24.24% <0.00%> (-40.91%)
:arrow_down:

mesheryctl/internal/cli/root/system/logs.go
3.26% <0.00%> (-36.96%)
:arrow_down:

mesheryctl/internal/cli/root/version.go
0.00% <0.00%> (-32.95%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.03% <0.00%> (-23.03%)
:arrow_down:

... and 3 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

What does this look like in terms of a general model that all events adhere to?

Even though the save is false by default still, we were seeing the snackbar that pattern has been saved during the cyto conversion, the function formatPatternOutput is being used for the save true as well as false also. so I added an additional check.

It's more of an extra check to stop the save notifications from appearing on each call.

@harkiratsm  Let's discuss this on the Meshery Dev call. Please add this as an agenda item in the meeting minutes if you would. :)

Closing as it would be fixed by @Revolyssup PR.


i18n: Generate scenario outlines correctly

If you use another language than english scenario outlines are not generated correctly. Instead of several tests only the first one is generated.
This PR fixes this issue by using the i18n map in the isOutline check.

Thanks @nonJerry !
Merged to main, will be included the nearest release.

Released in v5.4.0


Mention Ceph RBD and CephFS deprecation in the user doc to notify the users.

The Ceph RBD and CephFS intree storage driver is getting deprecated in kubernetes release 1.28 and users have to be notified about the same.

@idvoretskyi can you please review/approve this ? its been hanging here for some time and users are into a confusion in absence of this change.

Thanks @idvoretskyi 👍


BadImageFormatException thrown for dependency dll which is managed c++

Expected behavior
No exception thrown, or continue with ignoring the exception.
Actual behavior
Unhandled Exception: System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> System.BadImageFormatException: Could not load file or assembly or one of its dependencies. An attempt was made to load a program with an incorrect format.
at System.Reflection.RuntimeAssembly._nLoad(AssemblyName fileName, String codeBase,
CallStack.txt
Gauge version
Gauge version: 0.8.1

Plugins
-------
csharp (0.10.0)
html-report (3.1.0)
spectacle (0.1.1)
xml-report (0.2.0)

Could you please share some more details to help debug this?
BadImageFormatException occurs when the architecture against which an assembly to be loaded was compiled is different from the host architecture. Classic example : load an x86_64 assembly in x86 machine.
Your description does not have enough information for any of us to narrow down the cause. Perhaps if you can share a sample project which has this issue?

Sorry, I forgot to mention that. All assemblies are compiled for x64. and machine is also 64 bit.
I tried with both 32 bit and 64 bit versions of Gauge too.
I will try to create a sample project.

Closing this as an old issue. This may be fixed in the newer releases.
Will reopen if it still needs fixing.


Fix TS issues with Tabs

Fix TS issues with Tabs, part of #901

Codecov Report

Merging #964 (92587ca) into main (2b9b2fb) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #964   +/-   ##
=======================================
  Coverage   99.27%   99.27%
=======================================
  Files         114      114
  Lines         832      832
  Branches      252      252
=======================================
  Hits          826      826
  Misses          6        6

Impacted Files
Coverage Δ

components/tabs/src/atoms/tab/index.tsx
100.00% <100.00%> (ø)

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 2b9b2fb...92587ca. Read the comment docs.


Want a Release for ubuntu (linux)

You can try using wine to run it on Linux. I have no plans to support Linux.


Favicons Overlapping

Hello, this picture below depicts that favicons overlap on the headline of the article. Secondly, The favicons are not vertically positioned at centre when "Show full headline" is enabled.

duplicate of #192


PR - Root certificate authority requires escape characters on windows OS

Description
Resolve windows path being escaped by helm pkg
Type of change

This pull request fixes a bug in Radius and has an approved issue (issue link required).

Additional Description
Helm.runesUntil strips \ (x92) rune from string when converting Buffer RuneReader to []runes. This issues is outside this package. Alternatives is to import this func and remove this or mutate path before sending to parser. Care also has to be take to support users who are using "c:\\dir\\file.crt" as a hack. This only works as Helm's runesUntil strips \ by using the next rune!
fixes: Root certificate authority requires escape characters on windows OS #6946

@gpltaylor - thanks! the code change looks good. I've kicked off a functional test run, and I'll take care of merging when it passes.

Note: the codecoverage check that's failing is new and we're stilling adjusting it. Right now it fails on small changes like this one, so I'll need to bypass the check.

Merged! thanks


Feature/news

Add support for news articles
Description

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[x] 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 added tests to cover my changes.
[ ] All new and existing tests passed.

Ill fix the white text issue in the add form in a later pull request, where ill fix up the looks of the gmm, news and photos forms to be more like to login form and such, but ill rather do that all at once, so it wont be in this pr.


How to maintain multiple web socket session opposite of a single connection id in aca-py mediator for multi-Device Message Delivery

Using wallet backup and restore concept (in bifold wallet), I am using same wallet with same data in two mobile devices. In my expectation, if any agent sends a message to my wallet agent, then both of the device should get the messege (via acapy mediator). But only one device is receiving the message because of mediator's websocket session management.
So in aca-py mediator, for a connection id, I want to maintain multiple websocket session so that if any agent send message to my mobile wallet, both mobile should receive the message.
I have read a documentation in hackmd about the architecture of this session handling process, but there is not much information for clear understanding.
How can I maintain a list of sessions associated with each connection ID where each session corresponds to a different device?
I have added a simple illustration of my thought:

Unfortunately, this type of functionality is not currently supported. Private keys needed to decrypt messages sent to an instance of a wallet are not shared across wallets, and as a result, messages are specific to a device.  Further, there is not in the current mediator protocols a method for forwarding a single message to more than one wallet instance.
These types of issues do need a solution, but to this point have not been a priority.  This is something you might want to bring up at the DIDComm Working Group at the Decentralized Identity Foundation (https://decentralized.identity).


ADD: wradlib notebooks

First serve for some wradlib addons (see https://github.com/openradar/erad2024/issues/52#issuecomment-2274989778).
Will add further content later.
This also adds hvplot/datashader to the environment.
@mgrover1 I've put an additional files for the cloud repo here
ftp://ftp.meteo.uni-bonn.de/pub/radar/erad2024/wradlib/desio_dem.tif

@mgrover1 I'd appreciate, if you could merge the one file in the link above into the cloud bucket (possibly inside a wradlib folder). Thanks!

The data is uploaded @kmuehlbauer 😊

And the notebook works, too. Good time to leave for weekend.


bro, you broke our build

linking to https://github.com/nearuaguild/abstract-dao/pull/9
and https://github.com/rust-lang/rust/issues/133290 ,
i'm still not convinced this is not a compiler problem, but as per spec here https://rust-lang.github.io/rfcs/2045-target-feature.html , #[target_feature(enable = "simd128")] is only allowed on unsafe functions , not sure how up to date that spec is
https://github.com/DaniPopes/const-hex/blob/85f08142ccc5300c90faa94484b20db19388bf19/src/arch/wasm32.rs#L71-L72

and portable-simd feature wasn't called for in this use case https://docs.rs/crate/ethers-contract/2.0.14/source/Cargo.toml#24
❯ : cargo tree -i const-hex
const-hex v1.13.1
├── ethers-contract v2.0.14
│   └── near_abstract_dao v0.6.1 (/home/user/Documents/code/abstract-dao)
└── ethers-core v2.0.14
    ├── ethers-contract v2.0.14 (*)
    └── near_abstract_dao v0.6.1 (/home/user/Documents/code/abstract-dao)

More constructively: I see that calls to the functions using simd128 are gated on if has_simd128(), which is defined as:
https://github.com/DaniPopes/const-hex/blob/85f08142ccc5300c90faa94484b20db19388bf19/src/arch/wasm32.rs#L9-L12
That's the right spirit, but unfortunately, it's not guaranteed that this style of cfg-gating actually omits the functions in question when simd128 is not available. See also https://github.com/BurntSushi/memchr/issues/144 for a similar problem reported before. In general it's very likely to work most of the time, but evidently not always, especially (but not only) with opt-level = 0. The only reliable way to ensure that the wasm module doesn't contain any simd128 instructions, and thus can be loaded by engines that don't implement SIMD, is to put #[cfg(target_feature = "simd128")] on those functions. Everything else depends on compiler flags and undocumented implementation details of the compiler (e.g., how smart it tries to be about "which functions need to get codegen'd").
I don't know why check_simd128 can be defined without unsafe but I don't think it matters.

Thanks for the report. That's unfortunate considering it is the same implementation as the other architectures. I'll apply the same fix as in BurntSushi/memchr#144.
As for "without unsafe", that's because most of the intrinsics can't cause undefined behavior so they're not marked unsafe, however I can't find where this is explained.
Also I would suggest a more descriptive issue name for next time :D

Released in 1.13.2.


Avoid performing automagic CXXFLAGS detection

Hello Matthias,
Vc performs automatic detection of CXXFLAGS for the computer where it is being compiled, which breaks compilation of software on one machine to be installed on another. There is a bug on Gentoo Linux filed for this: https://bugs.gentoo.org/show_bug.cgi?id=584118.
Would it be possible to let the user choose the target architecture rather than detect it in CMake?

Isn't the TARGET_ARCHITECTURE variable enough for your purposes? Do you think it should default to 'none' instead of 'auto'?

The problem appears not when compiling Vc itself, but when compiling some other code against Vc. The current VcConfig.cmake automatically adds architecture flags into CXXFLAGS, which causes problems when one wants to compile something on one machine to run on another.
Defaulting to TARGET_ARCHITECTURE=none may be enough, but what's really important is that the VcConfig.cmake should only populate Vc-specific variables, while leaving standard variables alone, at least by default. I can always call vc_set_preferred_flags() after find_package(Vc) if that's what I really want. Maybe passing a variable to the function above that will receive the flags would be a good solution. Then, the user could call, say, vc_get_native_arch_flags(MYCXXFLAGS) or something similar.
I can probably put up a merge request with a possible solution if you want. Just let me know.

The intent of VcConfig.cmake is actually to not modify CMAKE_CXX_FLAGS but only to set Vc_DEFINITIONS, Vc_COMPILE_FLAGS, Vc_ARCHITECTURE_FLAGS, and Vc_ALL_FLAGS (which concatenates the preceding three). Anything else is an actual bug and I'd be happy to merge anything that fixes this.
BTW, are you talking about Vc 0.7 by chance? 0.7 is known to be buggy in this regard.

No, this is in Vc-1.2.0. I am using Vc for GeantV, so I am maintaining its package in Gentoo Linux.
Take a look at the bug linked above in Gentoo; the culprit seems to be AddCompilerFlag(), called from OptimizeForArchitecture.cmake.

Yes, AddCompilerFlag by default adds to CMAKE_CXX_FLAGS. But I just grepped through cmake/* (again) and did not find a single issue. OptimizeForArchitecture collects all the -m<...> flags in Vc_ARCHITECTURE_FLAGS. I would rather assume that Krita by default compiles its targets with Vc_ARCHITECTURE_FLAGS.
The thing here seems to be Gentoo specific because of the expectation that make.conf is the only place the defines the compiler flags? If so, then the Gentoo package should patch Krita to default to the make.conf flags instead of the flags determined via the cmake macro.
The user should simply compile with TARGET_ARCHITECTURE=none. This should give him the flags from make.conf only.

Vc should not touch CMAKE_CXX_FLAGS when one calls find_package(Vc). That's the same as modifying CXXFLAGS, so this is what the bug is.
As you state yourself above, then intent of VcConfig.cmake is to not modify CMAKE_CXX_FLAGS, but it does, since it includes VcMacros, which includes OptimizeForArchitecture, which calls AddCompilerFlag, which changes the CXXFLAGS by adding things to CMAKE_CXX_FLAGS by default.

Sorry, you are right, after digging through a bit more, I found the problem in calligra build system.


x/website:

What is the URL of the page with the issue?
https://go.dev/doc/tutorial/random-greeting
What is your user agent?
Firefox
Screenshot
Add an init function to seed the rand package with the current time. Go executes init functions automatically at program startup, after global variables have been initialized. For more about init functions, see Effective Go.
What did you do?
Perform the tutorial :)
What did you expect to see?
Descriptions matching the code.
What did you see instead?
Reference to old "init" code, which I guess got removed after Go 1.20, when not needed anymore.
https://pkg.go.dev/math/rand#Seed

Duplicate of #60627


Poly f louis

I created a new branch for the older branch containing polynomial embeddings since that was already out of date. I am planning to continue my work on polynomial embdeiings via this branch.

Dear @Louis-Mozart,
Thank you for your PL request. Please read gitflow to understand the main, development, and feature branch relationships. We are not allow to merge a branch into main branch. Please see https://www.atlassian.com/git/tutorials/comparing-workflows/gitflow-workflow


Is it possible to open the mindmap below?

My markdown notes tend to be on the order to 5 to 10 nested items, which is really hard to see when the screen is split left/right.
This could be fixed by allowing me to pick an up/down orientation as described in #16.  However, I'd actually prefer if I could choose how the screen is split.  Default could be "Right" as it is now, but I'd like to switch it to "Down".  Is this possible?  Or possible to add?
Also, in cases where I want a preview and mindmap it's nice to move the mindmap below

After the preview is opened to the right you can drag the tab of the preview below and Atom will split the view accordingly.
If you are a developer I would be happy to accept a pull request that would add this option.


Multiprocessing eigenvalue order

In the base version, the eigenvalues are returned in the ascending order (eig0<eig1<eig2). In the multiprocessing version, this order is reversed (eig0>eig1>eig2).
Attached a small example
stMultiprocBug.txt

Yes, descending order is more intuitive in general - that is what most other methods that use eigenvalue decomposition do. The only argument for the ascending order that I have is that some papers you mention and Vedrana's note on structure tensor use it.
In the end it doesn't matter too much, as long as it is clearly stated. My main issue with the current setup is that this ordering change isn't mentioned anywhere. I've spent a bit too much time trying to figure out why all of a sudden my results (e.g. linearity measure) are useless.

Thanks a lot for the input. I'm sorry I wasted your time - it should definitely be stated in the function docstring. I'll try to get the fixed soon!
As mentioned all probably change all functions to use descending order in version 1.0, but for now I'll probably just add the docs. I might also add the order as a function argument in the next update, but without changing the default behavior.

Does returning the eigen values in descending order change the following statement in one of the tutorials:

Note that the first axis of the vector correspond to (z, y, x), meaning that vec[0] is the length of the vector along the Z-axis, not the X-axis as you might expect.

Thank you..

First of all, sorry that I haven't gotten around to fix this issue.
@R-icntay, the order of the axes (x, y, z) or (z, y, x) is not related to issue discussed here and does not depend on whether eigen values are sorted ascending or descending.

Fixed in #15


DllNotFoundException: Lerc32

I've gotten this to work when deployed to my HoloLens, but I'm trying to simulate it from the Unity Editor with HoloGraphic Emulation. I'm getting the followubg error:
DllNotFoundException: Lerc32
Esri.PrototypeLab.HoloLens.Unity.Elevation+c__Iterator0.MoveNext () (at Assets/EsriToolkit/Mapping/Scripts/Elevation.cs:39)
UnityEngine.SetupCoroutine.InvokeMoveNext (IEnumerator enumerator, IntPtr returnValueAddress) (at C:/buildslave/unity/build/Runtime/Export/Coroutines.cs:17)

I managed to run code in editor after adding Lerc32.dll to plugins folder and select platforms all except WSA(please note, that you'll probably should use Lerc64.dll, but rename it to Lerc32.dll)


Machinery: Use proper test runner

Use a proper test runner that makes it easier to re-run failed validation tests and does error reporting in CI

New npm test time: 2 seconds.


Add offline access

Adds an option to request offline access.

Hey @bennettrogers thanks for the PR! I was able to take a quick look and am looking for your thoughts on some proposed changes I've been thinking about. Please feel free to correct me if I'm wrong on any of this as I haven't worked with Google's offline-access before.
Return a separate function for offline-access
I think that a separate function that the hook would returns for granting offline access would be a more intuitive API for library users. See below for a rough example:
const Component = () => {
  const { grantOfflineAccess } = useGoogleLogin()

  const handleClick = async () => {
    const offlineOptions = {}
    const authCode = await grantOfflineAccess(offlineOptions)

    doStuffWithCode(authCode)
  }

  return (
    <button onClick={handleClick}>Grant offline access</button>
  )
}

I think this approach comes with a few benefits:

Avoids some user-land confusion that the direct GoogleAuth methods and API kind of lend to. The actions of signing in and getting a authCode feel like different semantic actions and we can help clarify it at the library level.
Keeps the return type of signIn as it is now to avoid casting return values as shown below.

const googleUser = await signIn() as GoogleUser

Let me know what you think!

I've created a branch and PR with the above revisions in mind in #14.

#14 Looks excellent! Your approach makes way more sense than mine.
Thank you for looking at this so quickly!

Closing this in favor of #14 which has been merged and released as 0.12.0.


 not rendering on screen when cordova added

** Ionic v2 **
How to replicate the issue:
Step 1: visit this plunker http://plnkr.co/edit/w5qUFI?p=preview
(note: cordova.js has been linked, so you will get some alerts while loading. Press cancel for all alerts)
Step 2: Launch the preview
Step 3: Once puppy list got loaded, click on any puppy you like.
Step 4: It will open the detail page
Step 5: Now go back
Step 6: You can notice the bug. Puppies list is not getting rendered on screen this time. now when you click kittens segment button and then click puppies, now list got loaded.
I've checked this in both android mobile and pc browser.
It works fine either when I removed cordova.js from index.html  or remove setTimeout from home.ts for initializing puppies array. I use setTimeout because I'm using Promise for making http request in my application.

@adamdbradley @brandyscarney @jgw96 Hey there, Did you got chance to look into this issue?

Hey @dheeran1 !  Here is an example of how you would do this 👍
import {NgZone} from "angular2/core";
import {Page} from "ionic-angular";

@Page({
     template: "something using *ngFor"
 })
class AwesomeExample {

    items: any[];

    constructor(private _zone: NgZone) {
        fetch("awesomeapi.json").then((response) => {
             return response.json()
        }).then((data) => {
            this._zone.run(() => {
                this.items = data;
            })
        })
     }
}

This should fix your issue. If you would like to read more about NgZone and zones in angular2 and Ionic 2 here are some good links : https://angular.io/docs/js/latest/api/core/NgZone-class.html, http://blog.thoughtram.io/angular/2016/02/01/zones-in-angular-2.html. Thanks for using Ionic!

@jgw96  thanks for your reply. I'll check that for promise and let you know. meanwhile what is the workaround for setTimeout in given plunkr sample above.

@dheeran1 Hey! Soo i took your plunker and updated it to make it work and give you a demo of NgZone. Sorry i should have done this earlier (: haha. I also changed your setTimeout to a ES6 arrow function(https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Functions/Arrow_functions) so that you would be able to use this without having the ref variable you had made. Arrow functions allows this to bind lexically, which means you dont have to do anything like let self = this or anything like that. ES6 is awesome!

@jgw96: setTimeout (and friends) are patched by NgZone and run automatically.

We don’t have to do anything special to tell the framework that a change has happened. No ng-click, no $timeout, $scope.$apply().
If you’ve read our article on understanding Zones, you know that this works obviously because Angular takes advantage of Zones. Zones monkey-patches global asynchronous operations such as setTimeout() and addEventListener(), which is why Angular can easily find out, when to update the DOM.

http://blog.thoughtram.io/angular/2016/02/01/zones-in-angular-2.html

@jgw96 Thanks, Glad you did that and it works :-)
But I'm confused, why this happens only after linking cordova?

@dheran No problem at all! Yeah, so weve noticed some really weird issues caused by cordova that causes change detection not to be triggered by async operations. Currently the workaround for it is using NgZone.run() as i used in your plunker, but eventually when we narrow down the issue more and when the cordova team fixes this issue you will not have to do this.

Another weird thing is, it will work fine if setTimeout delay was reduced to less than 500ms

@dheran are you still having this issue with the latest version of Ionic 2 (beta.6)?

Hello! As it has been a little while since there was any activity on this issue i will be closing it for now. Thanks for using Ionic!

@jgw96 do you mean without using ngzone?

@dheran hey! yep without NgZone usage.

It's not working without ngzone. Is it working for you? Can you please update the plunk you shared earlier above?


gdsfactory/gdsfactory/pdk.py PDK class validate_layers method

This issue deals with the validate_layers method of the PDK class on line 240 in gdsfactory/gdsfactory/pdk.py. This method checks if the PDK contains the layers in the list "layers_required". The "layers_required" variable is not something the user can pass directly to the validate_layers method, instead it is a global variable defined on line 41 of pdk.py
The solution I'd like:
make layers_required an argument of the validate_layers method so we can pass the layers we require and check if the PDK has these layers.

This is a small update and I can make a PR with the changes if this is a good idea. Please provide feedback. Thoughts anyone?

Hi @alibillalhammoud . So I think the layers_required is typically used to enforce the layers required in a PDK for all features of gdsfactory to work. And my understanding is that you want to customize this for your PDK? What is your use case/motivation?

Hello @tvt173,
I was looking through the Pdk class and saw the validate_layers method perfectly suited for my use case, and also that layers_required was empty (so validate_layers is not being utilized effectively in the current version). I also did not see where layers_required was being accessed or modified outside validate_layers. I have now looked more at the history of validate_layers and saw that it was originally a pydantic validator, but was modified for the reasons described here. If we were to make a change as below, we would not change the interface of the method (compatible with existing call to validate_layers) while allowing for more generic layer validation functionality.
def validate_layers(self, layers_required: Optional[List[Layer]] = []):
        for layer in layers_required:
            if layer not in self.layers:
                raise ValueError(f"{layer!r} not in Pdk.layers {list(self.layers.keys())}")

Below is more context on what I am working on...
I am part of the OpenFASOC team. We make open-source analog circuit generators. There are some small analog cells used in OpenFASOC that we have had to manually design (see header). Moving forward, we want to automate design of small blocks needed. We are using GDSFactory for this.
Motivation for this issue: we are looking for areas where it may be useful to everyone in the open-source silicon community if we directly contribute to the tools used in our flow rather than applying new features to our codebase. Layer validation is a generic feature which may be suitable for GDSFactory.
The generators we are developing are python functions (ComponentFactory). Our goal is for the functions to accept a Pdk object so we have significant reuse of code once we define the Pdk class for a new technology. To improve reuse of code we defined a dictionary generic layer names (strings) which maps to the actual PDK layers. With this approach we would like the ComponentFactory to verify at runtime that generic layers it will access actually map to something in the PDK object, so we would want a method like 'validate_layers'.
Please let me know if this is a suitable suggestion,
thanks

sorry, i was speaking from memory and didn't take the time to check the codebase... you seem to be right that layers_required is empty right now. i think when this was implemented it was intended for that to be populated by special layers like TEXT and PORT which are expected to exist by some functions in gdsfactory, right @joamatab ?
otherwise, i have no issue with your suggestion. i would suggest making a MR for it so we can review. i do only have one small comment on your code snippet though... it's bad practice to have a mutable object as a default argument for a function. I would suggest setting the default to None and in the case of None, maybe check against that globally defined layers_required list? (assuming we actually add some layers to it...)

tvt173, Thank you for this information about mutable object default args; I will remember for future use.
See new PR with this feature.

PR looks great, thank you Ali!


LOL dataset  results were not noticeable

I tried to test the low-light enhancement task on the LOL dataset with visualize.py, and the results were not noticeable, and the image was basically unchanged, why is that? (I have tested Gopro dataset and it is good)
before:

after:

Note that if you want to test low-light image, please use the code src/visualization-Line1279-1281

lines 1279-1281:
“self.ema.to(self.device)
print("test start")
if self.condition”
   It seems that as long as `visual.py` is called, what is actually invoked is `src/visualization` at lines 1279-1281 . Therefore, it appears that calling these lines directly may not resolve the issue.

The code is modified one times, if you git recently, it should be lines 1286-1288


[Retrospective] Release version 2.0.0 (RC1)

How to use this issue?
Please add comments to this issue, they can be small or large in scope. Honest feedback is important to improve our processes, suggestions are also welcomed but not required.
What will happen to this issue post release?
There will be a discussion(s) about how the release went and how the next release can be improved. Then this ticket will be updated with the notes of that discussion along side action items.

Security had trouble getting the proper 2.0.0 build for a while, we thought we had done what was needed for the transport client removal, but ended up having to revisit it several times.  I'm not sure mechanically what wasn't working (getting the correct build, change propagation delays)

It doesn't look like component issues are being used to track release status, see several issues that do not yet have preparation phases complete (this is a sampling of them)

https://github.com/opensearch-project/OpenSearch/issues/2245
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/1279
https://github.com/opensearch-project/index-management/issues/274

Is there a new process?

Build qualifier switch from 2.0.0-alpha1 -> 2.0.0-rc1 was hard to track

Is there a new process?

No new process, just nobody ran after plugin owners this time around.

The release issue is unreadable with the description being so long and test reports hard to find relevant information to the state of the release.

How could we have found the issue on the website drop down before day of launch? CC: @CEHENKLE

https://github.com/opensearch-project/opensearch-build/issues/1892
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/1486
https://github.com/opensearch-project/opensearch-build/issues/1860
https://github.com/opensearch-project/opensearch-build/issues/1893
https://github.com/opensearch-project/opensearch-build/issues/1965
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/1398#issuecomment-1099459235
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/1401
https://github.com/opensearch-project/opensearch-build/issues/2018
https://github.com/opensearch-project/opensearch-build/issues/2040
https://github.com/opensearch-project/opensearch-build/issues/2075
https://github.com/opensearch-project/opensearch-build/issues/2076

@anirudha @praveensameneni @bbarani
Let us discuss about managing both OS and OSD plugins within the same repo.
Tagging notificationsDashboards at eb3af31759668a94727950d081e8a3a161f22918 ...
Tagging notifications-core at 482337090e9707e3e00c3538166bc62b45308d28 ...

As we can see OS 2695 use a different commit from notifications than OSD 3015 also in notifications, to build OS and OSD notifications plugins respectively.
This would cause potential discrepancies between OS and OSD build going on.
We need to discuss how to properly manage these plugins and their repos.
Thanks.

Some teams created release tags for rc1, isn't clear what should or should be done as part of post release steps in the component or centralized release issue.

Some teams created release tags for rc1, isn't clear what should or should be done as part of post release steps in the component or centralized release issue.

No team is creating tags for rc1, it is already automated from infra side.

Some teams created release tags for rc1, isn't clear what should or should be done as part of post release steps in the component or centralized release issue.

No team is creating tags for rc1, it is already automated from infra side.

Edit: I just notice security team has a github action to create tags.
I am not sure why that is the case as the tag is already automated from Infra side.
Thanks.

Open an issue to delete it.

It seems like there is expectation for OpenSearch Dashboards (+ plugins) to be prepped and sanity tested complete when OpenSearch (+ plugins) completes. That seems to be broken to me, for minor releases it is achievable, but for major releases it's shouldn't be expected. If any sanity tests found that OpenSearch was invalid then any result during OpenSearch Dashboards should be considered invalid. If we don't, and find out the regression occurs in OpenSearch but then we not spent multiple resources on OpenSearch Dashboards and OpenSearch.

Release notes should not be in the code complete section until after sanity tests are complete. I see multiple repos checking this check box off when anything could happen during sanity testing that requires them to make a change and update their release notes. Release notes can be in draft but I think there should be buffer room to ensure that release notes are the last things that checked off.

Build qualifier switch from 2.0.0-alpha1 -> 2.0.0-rc1 was hard to track

I think this was a big miss. I think we prioritized getting an alpha1 build of OpenSearch green which caused a lot of last minute changes to get an rc1 build. OpenSearch plugins needs an rc1 build to switch build no rc1 build existed so then it was a chicken and egg problem which then ended up making the Friday before the week of the initial planned released being the first day an OpenSearch 2.0.0-rc1 build was actually made. Giving initially a few days of sanity tests for OpenSearch and virtually none for OpenSearch Dashboards.

Distribution build breaks were happening constantly, it was hard to know if the build was passing or not.  The slack channel #opensearch-distribution-builds seems like it was unmonitored.

Close this one and leads to 2.0.0 GA in #2142.


can't find component [model.beans.User]

When registering a new user, I get the following error window after pressing the Register button:
ERROR!
An error occurred!
Action: home:main.process
Error: invalid component definition, can't find component [model.beans.User]
Type: expression
Details:

@VTPhil95 Sorry to hear you're having an issue. I'm afraid that error is pretty ambiguous tho - it could mean a number of different things:

You may be missing all the files needed from the repo
DI/1 may not be finding the component in your path
File permission issue(s)

etc.
What does your environment look like? Are you running this in CommandBox, or do you have this set-up as a site on a server somewhere? Are you running Windows, OSx, Linux?

@ddspringle  Thanks for the quick reply.  I'm running CommandBox on Windows here.
Essentially all I did was:

Run box install fw1-sa
Run box start
Configure an object cache and datasource
Press the Register button on the new user page and I get that error.

I've been poking at the code a bit and one thing I noticed is that if I replace this line:
https://github.com/ddspringle/framework-one-secure-auth/blob/master/home/controllers/main.cfc#L129
with this:
rc.userObj =variables.beanFactory.getBean( "User" );
I'm able to create a blank User record.

@VTPhil95 Ah. Yeah the way box is installing it with the slug is a bit off from what I expected. I'll get that fixed up.
You'll want to cd into fw1-sa before starting up Lucee w/ box server start. I suspect that is the root cause - webroot needs to be inside the fw1-sa directory to find the right path to model.beans...
That said, yes more could be done with DI/1 instead of the new model.beans.... syntax to let DI/1 find and instantiate these beans, which would probably eliminate the pathing issue altogether regardless of where webroot ends up being.
For now, try starting up from inside the fw1-sa directory and let me know if the problem goes away.

@ddspringle Starting from inside the fw1-sa directory fixed it.  Thanks for the quick help on this, I really appreciate it.

Sweet! Glad it is sorted. Lemme know if you have any other questions or need anything else!


Fails to send isMaster

I am using
"org.reactivemongo" %% "play2-reactivemongo" % "1.1.0-play28-RC3",
and I am getting below error:
[error] r.c.a.MongoDBSystem - [Supervisor-1/datapipeline] Fails to send a isMaster request to mongodb-1.mongodb-headless.mongodb.svc.cluster.local:27017 (channel #e7e125a4)
is there any configuration I need to give in connectionString or what can be the root cause. I didnt find anything in documentation

As indicated in the guide line, this tracker must not be use to ask question/for help.
Rather use the mailinglist.


[WIP] Implement server command

Implements #3

Codecov Report

Merging #24 (eedca83) into master (323a331) will decrease coverage by 1.12%.
The diff coverage is 76.66%.

@@            Coverage Diff             @@
##           master      #24      +/-   ##
==========================================
- Coverage   82.68%   81.55%   -1.13%
==========================================
  Files           3        4       +1
  Lines         179      206      +27
==========================================
+ Hits          148      168      +20
- Misses         15       22       +7
  Partials       16       16

Impacted Files
Coverage Δ

pkg/task/server/server.go
74.07% <74.07%> (ø)

pkg/task/message/message.go
100.00% <100.00%> (ø)

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 323a331...eedca83. Read the comment docs.


sudo rtupdate doesn't work?

sudo rtupdate doesn't work i try to downgrade the rtorrent but doesn't work

donee