[firebase_auth] Method swizzling problem with phone auth iOS

I'm having an issue with firebase_auth verifyPhoneNumber on iOS.
I followed all requirements on documentation. APN works fine (FCM can be received). But whenever verifyPhoneNumber called, it always fail with exception message:

If app delegate swizzling is disabled, remote notifications received by UIApplicationDelegate need to be forwarded to FIRAuth's canHandleNotification: method

I did pod upgrade to firebase_auth 6.3.0 and flutter to 1.7.11-pre43
still got the same problem. Android build works fine.

@kroikie Hi, I think this should be an open issue.

@kroikie Hi, I think this should be an open issue.

@collinjackson Seems like this is still an issue. I'll reopen here so we can track resolution efforts.

Firebase Auth + APN works on development mode but I get token mismatch when building release

Any solution regarding to this problem?

We had this problem for months, but it appears to have been fixed with the latest version (6.0.9). We used to get token mismatch error on iOS on release mode, but not anymore.

@umutseven92 I use firebase_messaging 6.0.9 for a while now, still there is an issue in IOS 12, I get token mismatch. It does work with the fix that someone made a long time ago: https://stackoverflow.com/questions/57395948/flutter-firebase-phone-auth-always-returns-token-mismatch-on-ios
but I want to see this fix implemented in new versions that also have other security fixes and such.

I solved this problem. It was related APN setting(SandBox Token) not version problem.
Thanks :)

@LeeLoHoon,
Were you able to get Firebase Phone Auth work in iOS with Firebase Cloud Messaging?
Both in Simulator & Real device?
Did you ever encounter Token mismatch error?

@cielo
Hi, Yes i did. I got Token mismatch too. But, when i fixed AppDelegate.swift, i solved this error.
Here is my AppDelegate.swift Code
import UIKit
import Flutter
import Firebase
import FirebaseAuth
import UserNotifications
import FirebaseInstanceID
@UIApplicationMain
@objc class AppDelegate: FlutterAppDelegate {
override func application(
_ application: UIApplication,
didFinishLaunchingWithOptions launchOptions: [UIApplicationLaunchOptionsKey: Any]?
) -> Bool {
FirebaseApp.configure()
GeneratedPluginRegistrant.register(with: self)
return true
}
override func application(_ application: UIApplication, didRegisterForRemoteNotificationsWithDeviceToken deviceToken: Data) {
let firebaseAuth = Auth.auth()
firebaseAuth.setAPNSToken(deviceToken, type: AuthAPNSTokenType.sandbox)
}
override func application(_ application: UIApplication, didReceiveRemoteNotification userInfo: [AnyHashable : Any], fetchCompletionHandler completionHandler: @escaping (UIBackgroundFetchResult) -> Void) {
    let firebaseAuth = Auth.auth()
    if (firebaseAuth.canHandleNotification(userInfo)){
        print(userInfo)
        return
    }
}

}

@LeeLoHoon Thanks. I finally got it to work in a different way, but it works now.  Cheers

@cielo
My pleasure :)

@cielo what was the different way that you made it work?

Well, I do not have any APN setting overrides in my project.
For me,

I re-created iOS App registered in Firebase console,
updated FirebaseAuth/FirebaseCore/FirebaseMessaging to latest versions
re-created & uploaded APN Auth Authentication Key in Firebase Console Settings - Cloud Messaging.

This made Firebase iOS Phone Authentication to work in both built for development/production in real device.
Also, I do not know if it matters, but I am using Apple Development for Code Signing Identity.
However, in iOS simulator, I am still having a phone verification issue with error like below for reCAPCHA workflow.
2020-02-02 23:47:32.469357-0800 Runner[83534:726911] flutter: verifyPhoneNumberError - {"error":{"code":403,"message":"Requests from this ios client application <empty> are blocked.","errors":[{"message":"Requests from this ios client application <empty> are blocked.","domain":"global","reason":"forbidden"}],"status":"PERMISSION_DENIED"}}
My API Key Credentials is restricted to my application bundle id (set in Cloud Console), and above error gets thrown during reCAPTCHA workflow. When my API Key is unrestricted, I do not encounter above error.
So something fishy is going on in firebase_auth such that it is not sending bundle id information to reCAPCHA endpoint. For this issue, I have created a separate issue.

ios simulator works only with preconfigured test phone numbers, for me that works fine.

@Antonimo I see. Is your iOS API key (from Google Cloud console) restricted to your app identified by bundle id? or is it unrestricted at this moment?
My phone auth with iOS simulator works fine if it is unrestricted, but it does not if API key is restricted.

Hello! i was wondering, is this something current? (as in the latest flutter release )I am having the same issue and @LeeLoHoon solution seems to worked out. Should I do this every time I'm deploying on iOS ?

@sebastianalcala
Hi, My solution is also applied to IOS.
Did you try this?

@LeeLoHoon Thanks. I finally got it to work in a different way, but it works now. Cheers

@cielo , how did you get it to work? Please describe your solution

import UIKit
import Flutter
import Firebase
import FirebaseAuth
import UserNotifications
import FirebaseInstanceID
@UIApplicationMain
@objc class AppDelegate: FlutterAppDelegate {
override func application(
_ application: UIApplication,
didFinishLaunchingWithOptions launchOptions: [UIApplicationLaunchOptionsKey: Any]?
) -> Bool {
FirebaseApp.configure()
GeneratedPluginRegistrant.register(with: self)
return true
}
override func application(_ application: UIApplication, didRegisterForRemoteNotificationsWithDeviceToken deviceToken: Data) {
let firebaseAuth = Auth.auth()
firebaseAuth.setAPNSToken(deviceToken, type: AuthAPNSTokenType.sandbox)
}
override func application(_ application: UIApplication, didReceiveRemoteNotification userInfo: [AnyHashable : Any], fetchCompletionHandler completionHandler: @escaping (UIBackgroundFetchResult) -> Void) {
    let firebaseAuth = Auth.auth()
    if (firebaseAuth.canHandleNotification(userInfo)){
        print(userInfo)
        return
    }
}

}

Hi @LeeLoHoon - I tried this solution but  I am getting this error:
..../ios/Runner/AppDelegate.swift:4:8: error: no such module 'FirebaseAuth'
import FirebaseAuth

@Abosrie
OK, try this again.
https://medium.com/rocket-fuel/using-multiple-firebase-environments-in-ios-12b204cfa6c0

Did you try above thing?
Then, try this again.
https://stackoverflow.com/questions/41709912/error-could-not-build-objective-c-module-firebase

Did you try above thing?
Then, try this and do it again.
https://stackoverflow.com/questions/41709912/error-could-not-build-objective-c-module-firebase

Yes! I tried your solution and fixed the import error also from the link you have suggested but unfortunately, still, the auto verification is not happening :(

I tried the above solutions and it seemed to work but then a client with iPhone 7- iOS 13.3.1 got token mismatch

Here is a process when you want to use dev DB and prod DB.
1.For auto switching, refer to this page's Separate Firebase Environments — An Alternate Approach
https://medium.com/rocket-fuel/using-multiple-firebase-environments-in-ios-12b204cfa6c0
2.Change AppDelegate.swift code with this.
import UIKit
import Flutter
import Firebase
import FirebaseAuth
import UserNotifications
import FirebaseInstanceID

@UIApplicationMain
@objc class AppDelegate: FlutterAppDelegate {
  override func application(
    _ application: UIApplication,
    didFinishLaunchingWithOptions launchOptions: [UIApplicationLaunchOptionsKey: Any]?
  ) -> Bool {
        FirebaseApp.configure()
        GeneratedPluginRegistrant.register(with: self)
        return true
  }
    override func application(_ application: UIApplication, didRegisterForRemoteNotificationsWithDeviceToken deviceToken: Data) {
        let firebaseAuth = Auth.auth()
        firebaseAuth.setAPNSToken(deviceToken, type: AuthAPNSTokenType.sandbox)

    }
    override func application(_ application: UIApplication, didReceiveRemoteNotification userInfo: [AnyHashable : Any], fetchCompletionHandler completionHandler: @escaping (UIBackgroundFetchResult) -> Void) {
        let firebaseAuth = Auth.auth()
        if (firebaseAuth.canHandleNotification(userInfo)){
            print(userInfo)
            return
        }
    }
}

If you have an error with Firebase import or something, refer to this page.
https://stackoverflow.com/questions/41709912/error-could-not-build-objective-c-module-firebase

These are everything I can give you.

@cielo
Hi, Yes i did. I got Token mismatch too. But, when i fixed AppDelegate.swift, i solved this error.
Here is my AppDelegate.swift Code
import UIKit
import Flutter
import Firebase
import FirebaseAuth
import UserNotifications
import FirebaseInstanceID

@UIApplicationMain
@objc class AppDelegate: FlutterAppDelegate {
  override func application(
    _ application: UIApplication,
    didFinishLaunchingWithOptions launchOptions: [UIApplicationLaunchOptionsKey: Any]?
  ) -> Bool {
        FirebaseApp.configure()
        GeneratedPluginRegistrant.register(with: self)
        return true
  }
    override func application(_ application: UIApplication, didRegisterForRemoteNotificationsWithDeviceToken deviceToken: Data) {
        let firebaseAuth = Auth.auth()
        firebaseAuth.setAPNSToken(deviceToken, type: AuthAPNSTokenType.unknown)

    }
    override func application(_ application: UIApplication, didReceiveRemoteNotification userInfo: [AnyHashable : Any], fetchCompletionHandler completionHandler: @escaping (UIBackgroundFetchResult) -> Void) {
        let firebaseAuth = Auth.auth()
        if (firebaseAuth.canHandleNotification(userInfo)){
            print(userInfo)
            return
        }
    }
}

My error got resolved by this as well but then my notification handling stopped working, do you have any idea regarding the same. Below is my app delegate code
`import UIKit
import Flutter
import Firebase
import FirebaseAuth
import UserNotifications
import FirebaseInstanceID
@UIApplicationMain
@objc class AppDelegate: FlutterAppDelegate {
override func application(
_ application: UIApplication,
didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?
) -> Bool {
// iOS 12 support
if #available(iOS 12, *) {
UNUserNotificationCenter.current().requestAuthorization(options:[.badge, .alert, .sound, .provisional, .providesAppNotificationSettings, .criticalAlert]){ (granted, error) in }
application.registerForRemoteNotifications()
}
// iOS 10 support
if #available(iOS 10, *) {
UNUserNotificationCenter.current().requestAuthorization(options:[.badge, .alert, .sound]){ (granted, error) in }
application.registerForRemoteNotifications()
}
// iOS 9 support
else if #available(iOS 9, *) {
UIApplication.shared.registerUserNotificationSettings(UIUserNotificationSettings(types: [.badge, .sound, .alert], categories: nil))
UIApplication.shared.registerForRemoteNotifications()
}
// iOS 8 support
else if #available(iOS 8, *) {
UIApplication.shared.registerUserNotificationSettings(UIUserNotificationSettings(types: [.badge, .sound, .alert], categories: nil))
UIApplication.shared.registerForRemoteNotifications()
}
// iOS 7 support
else {
application.registerForRemoteNotifications(matching: [.badge, .sound, .alert])
}
FirebaseApp.configure()
GeneratedPluginRegistrant.register(with: self)
return super.application(application, didFinishLaunchingWithOptions: launchOptions)

}
override func application(_ application: UIApplication, didRegisterForRemoteNotificationsWithDeviceToken deviceToken: Data) {
let firebaseAuth = Auth.auth()
firebaseAuth.setAPNSToken(deviceToken, type: AuthAPNSTokenType.unknown)
}
override func application(_ application: UIApplication, didReceiveRemoteNotification userInfo: [AnyHashable : Any], fetchCompletionHandler completionHandler: @escaping (UIBackgroundFetchResult) -> Void) {
    let firebaseAuth = Auth.auth()
    if (firebaseAuth.canHandleNotification(userInfo)){
        print(userInfo)
        return
    }
}

}
`

Here is a process when you want to use dev DB and prod DB.
1.For auto switching, refer to this page's Separate Firebase Environments — An Alternate Approach
https://medium.com/rocket-fuel/using-multiple-firebase-environments-in-ios-12b204cfa6c0
2.Change AppDelegate.swift code with this.
import UIKit
import Flutter
import Firebase
import FirebaseAuth
import UserNotifications
import FirebaseInstanceID

@UIApplicationMain
@objc class AppDelegate: FlutterAppDelegate {
  override func application(
    _ application: UIApplication,
    didFinishLaunchingWithOptions launchOptions: [UIApplicationLaunchOptionsKey: Any]?
  ) -> Bool {
        FirebaseApp.configure()
        GeneratedPluginRegistrant.register(with: self)
        return true
  }
    override func application(_ application: UIApplication, didRegisterForRemoteNotificationsWithDeviceToken deviceToken: Data) {
        let firebaseAuth = Auth.auth()
        firebaseAuth.setAPNSToken(deviceToken, type: AuthAPNSTokenType.unknown)

    }
    override func application(_ application: UIApplication, didReceiveRemoteNotification userInfo: [AnyHashable : Any], fetchCompletionHandler completionHandler: @escaping (UIBackgroundFetchResult) -> Void) {
        let firebaseAuth = Auth.auth()
        if (firebaseAuth.canHandleNotification(userInfo)){
            print(userInfo)
            return
        }
    }
}

If you have an error with Firebase import or something, refer to this page.
https://stackoverflow.com/questions/41709912/error-could-not-build-objective-c-module-firebase

These are everything I can give you.

Using that firebase works but it breaks onesignal, any thoughts ?

This worked
import UIKit
import Flutter
import Firebase
import FirebaseAuth

@UIApplicationMain
@objc class AppDelegate: FlutterAppDelegate {
  override func application(
    _ application: UIApplication,
    didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?
  ) -> Bool {
    GeneratedPluginRegistrant.register(with: self)
    return super.application(application, didFinishLaunchingWithOptions: launchOptions)
  }

    override func application(_ application: UIApplication, didRegisterForRemoteNotificationsWithDeviceToken deviceToken: Data) {
      Auth.auth().setAPNSToken(deviceToken, type: .prod)
    }
    override func application(_ application: UIApplication,
        didReceiveRemoteNotification notification: [AnyHashable : Any],
        fetchCompletionHandler completionHandler: @escaping (UIBackgroundFetchResult) -> Void) {
      if Auth.auth().canHandleNotification(notification) {
        completionHandler(.noData)
        return
      }
    }
}

@cielo
Hi, Yes i did. I got Token mismatch too. But, when i fixed AppDelegate.swift, i solved this error.
Here is my AppDelegate.swift Code
import UIKit
import Flutter
import Firebase
import FirebaseAuth
import UserNotifications
import FirebaseInstanceID

@UIApplicationMain
@objc class AppDelegate: FlutterAppDelegate {
  override func application(
    _ application: UIApplication,
    didFinishLaunchingWithOptions launchOptions: [UIApplicationLaunchOptionsKey: Any]?
  ) -> Bool {
        FirebaseApp.configure()
        GeneratedPluginRegistrant.register(with: self)
        return true
  }
    override func application(_ application: UIApplication, didRegisterForRemoteNotificationsWithDeviceToken deviceToken: Data) {
        let firebaseAuth = Auth.auth()
        firebaseAuth.setAPNSToken(deviceToken, type: AuthAPNSTokenType.unknown)

    }
    override func application(_ application: UIApplication, didReceiveRemoteNotification userInfo: [AnyHashable : Any], fetchCompletionHandler completionHandler: @escaping (UIBackgroundFetchResult) -> Void) {
        let firebaseAuth = Auth.auth()
        if (firebaseAuth.canHandleNotification(userInfo)){
            print(userInfo)
            return
        }
    }
}

Thanks for this! This worked for me. Annoying that the documentation for the plugin has the incorrect setup code and I had to dig to find it here.
The other hard thing is before I put this change in, things worked fine in Android and in iOS simulator, but not on iOS physical device.

If your flutter Runner app is in Objective-C and @import FirebaseAuth; doesn't work even after a pods install and you want your hobby project to "just run already":
-(void)application:(UIApplication *)application didRegisterForRemoteNotificationsWithDeviceToken:(NSData *)deviceToken {
    id auth = [NSClassFromString(@"FIRAuth") performSelector:@selector(auth)];
    NSLog(@"didRegisterForRemoteNotificationsWithDeviceToken auth: %@", auth);
    [auth setAPNSToken:deviceToken type:FIRAuthAPNSTokenTypeUnknown];
}

-(void)application:(UIApplication *)application didReceiveRemoteNotification:(NSDictionary *)userInfo {
    id auth = [NSClassFromString(@"FIRAuth") performSelector:@selector(auth)];
    if ([auth canHandleNotification:userInfo]) {
        NSLog(@"didReceiveRemoteNotification userInfo: %@", userInfo);
    } else {
        NSLog(@"didReceiveRemoteNotification could not handle notification");
    }
}

I tried out what @LeeLoHoon said above, but it did not work for me until I made the following changes -
override func application(_ application: UIApplication, didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?) -> Bool {
            FirebaseApp.configure()
            GeneratedPluginRegistrant.register(with: self)

            return super.application(application, didFinishLaunchingWithOptions: launchOptions)
        }

    override func application(_ application: UIApplication, didRegisterForRemoteNotificationsWithDeviceToken deviceToken: Data) {
        let firebaseAuth = Auth.auth()
        firebaseAuth.setAPNSToken(deviceToken, type: AuthAPNSTokenType.prod)
    }

    override func application(_ application: UIApplication, didReceiveRemoteNotification userInfo: [AnyHashable : Any], fetchCompletionHandler completionHandler: @escaping (UIBackgroundFetchResult) -> Void) {
        let firebaseAuth = Auth.auth()

        if (firebaseAuth.canHandleNotification(userInfo)){
           completionHandler(.noData)
            return
        }

    }

basically on canHandleNotification one needs to call the completionHandler. This worked for me.

I think there is something underlying that is related to the firebase_messaging issue I linked above. This has plagued me for a long time now and i can't seem to narrow down what fixes/breaks it.
I may have found a winning combination of things to let firebase_auth and firebase_messaging work finally.
Contrary to what all of the github issues say, do add this to your AppDelegate.swift didFinishLaunchingWithOptions:
 if #available(iOS 10.0, *) {
    UNUserNotificationCenter.current().delegate = self as? UNUserNotificationCenterDelegate
}

Do not add this to your Info.plist:
FirebaseAppDelegateProxyEnabled
NO

Do add this to AppDelegate.swift (just like the solution above from @abhriyaroy):
    override func application(_ application: UIApplication, didRegisterForRemoteNotificationsWithDeviceToken deviceToken: Data) {
        let firebaseAuth = Auth.auth()
        firebaseAuth.setAPNSToken(deviceToken, type: AuthAPNSTokenType.unknown)
    }

    override func application(_ application: UIApplication, didReceiveRemoteNotification userInfo: [AnyHashable : Any], fetchCompletionHandler completionHandler: @escaping (UIBackgroundFetchResult) -> Void) {
        let firebaseAuth = Auth.auth()
        if (firebaseAuth.canHandleNotification(userInfo)){
            // print(userInfo)
            completionHandler(.noData)
            return
        }
    }

That should solve it. Firebase auth should be able to use the push notification verification prior to requesting notification permission, and after requesting notification permission firebase messaging should work now both in the foreground and background.
I think there is some kind of timeout related to Firebase here because while testing this, I can't seem to "break" it by setting it back, it just keeps working but after a few days it will be broken again.
I just wish I knew why this requires hacking up AppDelegate.swift like this, and I wish it worked out of the box following the instructions on pub.dev. I have spent so many hours on this issue pulling my hair out.

Do not add this to your Info.plist:
FirebaseAppDelegateProxyEnabled
NO
But this is only possible if firebase messaging is the only notification plugin, right? So if I further want to use the flutter_local_notification plugin this is not applicable

Do not add this to your Info.plist:
FirebaseAppDelegateProxyEnabled
NO

But this is only possible if firebase messaging is the only notification plugin, right? So if I further want to use the flutter_local_notification plugin this is not applicable

I haven't tested, but I think if you want to do that, then you have to use this for didReceiveRemoteNotification:
override func application(_ application: UIApplication, didReceiveRemoteNotification userInfo: [AnyHashable : Any], fetchCompletionHandler completionHandler: @escaping (UIBackgroundFetchResult) -> Void) {
  let firebaseAuth = Auth.auth()
  if (firebaseAuth.canHandleNotification(userInfo)){
      // print(userInfo)
      completionHandler(.noData)
      return
  }

  // https://firebase.google.com/docs/cloud-messaging/ios/receive#handle_messages
  Messaging.messaging().appDidReceiveMessage(userInfo)
}

Hey all 👋
Our rework of the firebase_auth plugin as part of the FlutterFire roadmap was published over a week ago with a ton of fixes and new features. Please could you try the new version and see if this is still an issue for you? If it is then please submit a new up to date GitHub issue so it's easier to track.
For help migrating to the new plugins please see the new migration guide: https://firebase.flutter.dev/docs/migration


fixes RUN-1015

fix DOM.getBoxModel

https://linear.app/replay/issue/RUN-1015/getboxmodel-is-returning-different-values-between-firefox-and-gecko

bring in Inspector{DOM,CSS,Network}Agents
generic CBORToJSON templates and functions
WIP CSS.getAppliedRules
WIP Pause.getObjectPreview for CSS

NOTE: The name of the branch diverges from the issue being fixed, because even though it fixes RUN-1015, it is mostly relying on WIP for RUN-981 (CBORToJSON + Inspector{DOM,CSS,Network}Agent).

I did a cursory review on this.  There's a bunch of extra stuff being pulled in to make the box highlighting work, which would take a long time to evaluate thoroughly.  Gonna do a build/test run instead.


ValueError: ZIP does not support timestamps before 1980

When run = exp.submit(config=est), it generates error "ValueError: ZIP does not support timestamps before 1980"

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

ID: 4bee4b9a-ce0f-f98b-3160-d6c4a40ef3ac
Version Independent ID: a0475a9e-47f6-2803-8707-fac54ba898dd
Content: Image classification tutorial: Train models - Azure Machine Learning service
Content Source: articles/machine-learning/service/tutorial-train-models-with-aml.md
Service: machine-learning
Sub-service: core
GitHub Login: @sdgilley
Microsoft Alias: sgilley

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

@kenakamu Could you please confirm if you are using azure notebooks and if your project folders and timestamps show the current values? For example as seen in this screen shot.

The error you received might be because your notebook VM or server might not be having the current date/time and hence the error while running the command.

@kenakamu we will now proceed to close this thread. If there are further questions regarding this matter, please tag @RohitMungi-MSFT in your reply. We will gladly continue the discussion and we will reopen the issue.

I am having this same issue in Notebook VMs trying to run = experiment.submit(estimator)
/data/anaconda/envs/py35/lib/python3.5/site-packages/azureml/_execution/_commands.py in _make_zipfile_exclude(project_object, zip_filename, exclude_function)
565                 relative_path = os.path.normpath(os.path.join(relative_dirpath, name))
566                 if not exclude_function(full_path):
--> 567                     zf.write(full_path, relative_path)
568             for name in filenames:
569                 full_path = os.path.normpath(os.path.join(dirpath, name))
/data/anaconda/envs/py35/lib/python3.5/zipfile.py in write(self, filename, arcname, compress_type)
1443         if isdir:
1444             arcname += '/'
-> 1445         zinfo = ZipInfo(arcname, date_time)
1446         zinfo.external_attr = (st[0] & 0xFFFF) << 16      # Unix attributes
1447         if isdir:
/data/anaconda/envs/py35/lib/python3.5/zipfile.py in init(self, filename, date_time)
336
337         if date_time[0] < 1980:
--> 338             raise ValueError('ZIP does not support timestamps before 1980')
339
340         # Standard values:
ValueError: ZIP does not support timestamps before 1980

Is this something that is still being worked on or is there a workaround?  @RohitMungi-MSFT

@cassieview - what version of the SDK is your notebook VM running?   I just tried it on a notebook VM running Azure ML SDK Version:  1.0.48 and do not see a problem; the tutorial notebook runs with no errors.

I am running Azure ML SDK Version:  1.0.53

@sdgilley I changed my version to Azure ML SDK Version:  1.0.48 and tried again - it still failed.

I created a new workspace and it works fine in the new workspace. When you create a new workspace vm it autocreates a user folder with your username. In the workspace with the issue - the user folder said it was created 50 years ago. In the new workspace the autocreated user folder had the correct create date. I think the issue was either the VM in the old workspace or some configuration bug that has since been fixed. Either way my issue is resolved now! Thanks!


move iptables related code from runv to hyperd

runv does not setup port mapping, which is a hyperd daemon function.
Let's move it back to make things easier.

Based on the discussion this morning, this pr is updated, which moves iptables related code out from runv.
And the code is moved to hyperd via hyperhq/hyperd#652

This PR could be considered as a clean up. Any comments @gao-feng ?

LGTM


Add propositional functions

Propositional functions are propositional formulas up to semantic equivalence.
@JamesGallicchio I thought about your question of whether we want to use this or { f: PropAssignment ν → Bool | dependsOnFinitelyManyVars f } as the mathematical model, in particular in cases where we have a function but don't want to construct a prop. formula. I think the two definitions should be equivalent, but the current one is easier to make because it doesn't require defining dependsOnFinitelyManyVars first (I will add that in a later PR). We can deal with the non-formula issue later by proving that any function depending on a finite amount of vars can be encoded as a formula, essentially by picking an ordering on the variables it depends on and then doing iterated Shannon expansions (or constructing a non-reduced OBDD, if you want).
EDIT: Well, to be fair this one requires defining PropForm whereas the latter one doesn't. Maybe the other one is better. Hmmmm..

For future self: another potential model is (S : Finset V) x' ((\forall v \in S, Bool) -> Bool)
I have not thought deeply about it. But it is nicely self contained, and makes it easy to express partial assignment substitution.

Ok, we are back down to 1 CNF type.
I specialized all the old code to ICnf, but most of it can be generalized once we have classes for "variables can be iterated over" and "variables can map to/from Nat".
Notably, I hijacked PropForm for input to the Tseitin implementation. This made some of the encoding less pretty, because we don't have notation for PropForms yet.
I wasn't sure whether to add back notation; the old notation frequently needed type annotations to disambiguate from Prop, which was a small headache.


get_gambl_metadata looks in the wrong place for a case set file

This is probably an easy fix. This file is in the GAMBL repo but in a remote computer the code is still looking for it in the GSC path. Can this be updated to use the GAMBL repo prefix from the active config?
metadata_fl_dlbcl = get_gambl_metadata(case_set = "tFL-study")
Error: '/projects/rmorin/projects/gambl-repos/gambl-rmorin/data/metadata/raw_metadata/gambl_tFL_manual.tsv' does not exist.

Just looking into this. I fixed this but noticed there are some other files used in this case set. Do you know if they are synced to the remote computer? I'll implement the similar repo prefix for them so same error won't  come up again
gambl_transformations = suppressMessages(read_delim("/projects/rmorin/projects/gambl-repos/gambl-rmorin/data/metadata/raw_metadata/gambl_transformation.txt", delim = " "))

fl_transformation_meta = suppressMessages(read_tsv("/projects/rmorin/projects/gambl-repos/gambl-rmorin/shared/gambl_fl_transformed.tsv"))

curated = suppressMessages(read_tsv("/projects/rmorin/projects/gambl-repos/gambl-rmorin/data/metadata/raw_metadata/clin_review_fl.tsv"))

This one also seems to be out of sync with master and is not in version control (not on GAMBL master): /projects/rmorin/projects/gambl-repos/gambl-rmorin/shared/gambl_fl_transformed.tsv


Google Fonts Integration

Hi,
I tried to integrate Google font with subsets as described here:
https://github.com/aristath/kirki/wiki/Google-Fonts-Integration
but the output does nothing. What sholud I do to use Google fonts with subsets?

Can you please post the code you're using here?

There was an error on the format of the output argument on that wiki page.
I fixed that now so please check it again. :+1:

Thank you, it's working now.


Issue with DetectInvalidImageUrls

Hi @MichaelTamm / @knorrium
Awesome library to detect layout bugs
Seems some issue with DetectInvalidImageUrls . Found Few Layout invalid Image URL's however when I typed the source URL on the browser it returned status code 200 and able to see that image
On Web in Network tab it shows 200
However, layout bug description shows
Detected visible  element with invalid src attribute "image.png" -- HTTP server responded with: 403 Forbidden

URL: website url

Hmm, HTTP status 403 means "Forbidden" -- I guess you did authenticate in your browser before you types in the URL and therefore your browser sends the authentification information ("Authenication" header or cookie) together with the request, but Fighting Layout Bugs does not and therefore gets the 403 response.


feat: support multi-exponentiations in packed format (PROOF-877)

Rationale for this change
Improve the performance of multi-exponentiations with multiple outputs of varying bit lengths by adding a packed interface.
What changes are included in this PR?

Adds a new packed_msm method to MsmHandle.

Are these changes tested?
Yes

:tada: This PR is included in version 3.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add gzip capability to HTTP endpoints

Fixes https://github.com/influxdb/influxdb/issues/1449

Makes sense, but is there no way to unit-test this? Surely there is.

Unit tests added.

Looks great, +1, assuming all is good even without the header. Thanks for the tests.


enable different package support in workspaces

This option adds an ability to build and run targets from different member packages/crates in workspaces.
Workspaces contain several crates and it is a common situation when integration tests are located in a separate crate.

Thanks!


Argocd server default readiness and liveness probe is too short

What is version of KubeSphere DevOps has the issue?
v3.4.0
How did you install the Kubernetes? Or what is the Kubernetes distribution?
kubekey
What happened?

Default timeout is 1s
Need change to 10s.
Relevant log output
No response
Additional information
No response

https://github.com/kubesphere/kse-installer/pull/365/files
https://github.com/kubesphere-sigs/ks-devops-helm-chart/pull/119/files

Issue fixed in kse v3.5.0-alpha.1

/close


[SPARK-24373][SQL] Add AnalysisBarrier to RelationalGroupedDataset's child

What changes were proposed in this pull request?
When we create a RelationalGroupedDataset we set its child to the logicalPlan of the DataFrame we need to aggregate. Since the logicalPlan is already analyzed, we should not analyze it again. But this happens when the new plan of the aggregate is analyzed.
The current behavior in most of the cases is likely to produce no harm, but in other cases re-analyzing an analyzed plan can change it, since the analysis is not idempotent. This can cause issues liek the one described in the JIRA (missing to find a cached plan).
The PR adds an AnalysisBarrier to the logicalPlan which is used as child of RelationalGroupedDataset.
How was this patch tested?
added UT

Test build #91158 has started for PR 21432 at commit 361fee8.

Probably similar change should be also in KeyValueGroupedDataset. It also uses logicalPlan without AnalysisBarrie

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3587/
Test PASSed.

Test build #91158 has finished for PR 21432 at commit 361fee8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91158/
Test PASSed.

@TomaszGaweda Yes, we should also fix KeyValueGroupedDataset .

cc @mgaido91 Could you add a test suite to check all the external APIs in KeyValueGroupedDataset  and RelationalGroupedDataset are covered by the fix?

Test build #91188 has started for PR 21432 at commit e2132c9.

Test build #91188 has finished for PR 21432 at commit e2132c9.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91188/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3611/
Test PASSed.

Test build #91190 has started for PR 21432 at commit e2132c9.

Test build #91190 has finished for PR 21432 at commit e2132c9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91190/
Test PASSed.

@mgaido91 Thanks for fixing this.

Btw, let's also mention KeyValueGroupedDataset in the title and description?

thanks @viirya, I forgot. Now it is updated, thanks.

Test build #91199 has started for PR 21432 at commit 68e398d.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3618/
Test PASSed.

Test build #91199 has finished for PR 21432 at commit 68e398d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

thanks, merging to master/2.3!


Add more MPPTs in Solar Options

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior

Expected behaviour
Is it possible to see more MPPTs?
For now it is a maximum of 4, but can this be increased to 8 or higher?
I would like to display my solar panels individually.
Possible Solutions

Mode
Both
Context / Reason

Hi it has been requested before but due to the limited space not possible until I get around to designing a wider version of the card.

Also discussed https://github.com/slipx06/sunsynk-power-flow-card/discussions/291. Adding this to ideas


Long stream of error messages related to is_deleted and _version columns when running the mysql docker compose used for example usage

When running this docker compose  https://github.com/Altinity/clickhouse-sink-connector/blob/develop/sink-connector-lightweight/docker/docker-compose-mysql.yml, using the build from https://github.com/Altinity/clickhouse-sink-connector/issues/612 we see a long stream of error messages appear:

I have a same problem in Postgre for example:
2024-08-07 07:37:33.826 INFO  - getTableEngineUsingSystemTables ResultSetcom.clickhouse.jdbc.ClickHouseResultSet@37807356
2024-08-07 07:37:33.836 INFO  - * INSERT QUERY for Database(metrika) *: insert into products(PRODUCT_ID,NAME,DESCRIPTION,PRICE) select PRODUCT_ID,NAME,DESCRIPTION,PRICE from input('PRODUCT_ID UInt64,NAME String,DESCRIPTION String,PRICE UInt64')
2024-08-07 07:37:33.841 ERROR - ******** ERROR: Database(metrika), Table(products), ClickHouse column NAME not present in source **********
2024-08-07 07:37:33.841 ERROR - ******** ERROR: Database(metrika), Table(products), Setting column NAME to NULL might fail for non-nullable columns **********
2024-08-07 07:37:33.841 ERROR - ******** ERROR: Database(metrika), Table(products), ClickHouse column DESCRIPTION not present in source **********
2024-08-07 07:37:33.841 ERROR - ******** ERROR: Database(metrika), Table(products), Setting column DESCRIPTION to NULL might fail for non-nullable columns **********
2024-08-07 07:37:33.841 ERROR - ******** ERROR: Database(metrika), Table(products), ClickHouse column PRICE not present in source **********
2024-08-07 07:37:33.841 ERROR - ******** ERROR: Database(metrika), Table(products), Setting column PRICE to NULL might fail for non-nullable columns **********
2024-08-07 07:37:33.842 ERROR - ***** ERROR inserting Batch Database(db_paperless), Table(products) ***************
java.sql.SQLException: Cannot set null to non-nullable column #2 [NAME String]
at com.clickhouse.jdbc.SqlExceptionUtils.clientError(SqlExceptionUtils.java:73) ~[app.jar:?]
can you help me?

Sorry, I made a bad table in the system and everything broke down.


Server.FindUsers is searching for exact match despite that being set to false.

Server.FindUsers(expression, false); will search for exact match only.
Expected behavior when exactMatch = false should be equal to User.Name.ToLower().Contains(expression.ToLower()) while it seems to behave as User.Name.ToLower() == expression.ToLower() In other words, It is not case sensitive, but it won't find partial name.
Example of successful search to find user "Rhea the Cookie"
expression = "rhea the cookie";
expression = "Rhea the Cookie";
Example of unsuccessful search:
expression = rhea
expression = Rhea
Suggested solution (please excuse syntax issues, typing it right here..)
public IEnumerable<User> FindUsers(string name, bool exactMatch = false)
{
    if( string.IsNullOrEmpty(name) )
        throw new ArgumentNullException(nameof(name));

    if( exactMatch )
        return _users.Values.ToList().FindAll( u => u.Name == name );

    return _users.Values.ToList().FindAll( u => u.Name.ToLower().Contains(name.ToLower) );
}

ps. sorry but i can't submit pull right now...

The FindXXX methods will be removed in 1.0. A separate issue could be made for something like a Search extension method maybe? We'll start looking into extension methods as 1.0 draws closer to launch.


fix(openapi): add migration to change 'trigger_properties' to object

What is the motivation for this pull request?
fix(openapi): add migration to change trigger_properties to object
Relates to https://github.com/braze-community/braze-php/issues/37
What is the current behavior?
trigger_properties is string
What is the new behavior?
trigger_properties is object
Checklist:

[x] Conventional Commits
[ ] Tests
[ ] Documentation

Release:

https://github.com/braze-community/braze-specification/pull/48#issuecomment-1867418308


[Builder] Moved DataStorage init and configuration to builder

Added code to initialize Datastorage in builder
Signed-off-by: Sunchit Sharma<EMAIL_ADDRESS>Description
Changes made to move Data Storage Conf. to Builder
Fixes  #137
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested ?

Change the configutartion.toml with the ips of a Machine B, place them in /var/edge-orchestration/datastorage/ folder
Run Edge Orchestration on Machine A. Using ./build.sh
On Machine B run edgex using ./edgex-launch
The Mongo DB can be verified for device profile and readings.
Metadata values attached for reference

Test Configuration:

Firmware version: Ubuntu 16.04
Edge Orchestration Release:  Coconut

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Thank you for the review. I went through all the changes suggested and would follow the required changes and structure and try to re raise a PR.

I am closing this PR and raising another as many changes on this code have been overwritten.


Add expires property

See: https://github.com/digitalbazaar/multikey-context/issues/2

In https://www.w3.org/ns/credentials/v2 expires is defined as https://w3id.org/security#expiration. This does not match my expectation from https://w3c.github.io/vc-data-integrity/vocab/security/vocabulary.html#expires. Can somebody clarify?

Hmm, I see from the linked comment that some update to the security vocab changed ...#expiration to ...#expires -- we'll want to figure out what's going on there. I see that the data integrity context uses ...#expiration as all of the incubated work has: https://www.w3.org/ns/data-integrity/v1. It's also the case that the VC v1 context uses ...#expiration: https://www.w3.org/2018/credentials/v1.

@HelgeKrueger - so I think this PR is almost ready to merge, if you could accept the suggested change from Dave.

I just approved the requested changes. Does this match the expectation?

+1 from me. @dlongley ?


Removed ExecutionMode.scala and all references

This process has been documented as no longer being necessary.
Daffodil-1814

Right, so I can see that master contains SDE.scala with windows line endings. That seems to be the root of the problem. So I'm starting to wonder if we need a commit to fix that first? Or do we push this? Sorry its such a pain, but I've been elected to be the windows (always small w) guy.

SDE.scala in master has LF line endings for me, and git on linux doesn't do any munging of line endings linke winows can, which confuses things. And I've checked out your PR branch, and then SDE.scala has CRLF line endings. So it definitely seems to me that this PR is changing from LF to CRLF, which we don't want.
Ususally git on windows has the core.autocrlf configuration option set to true. This makes it so that when you checkout a branch, git wil convet the line endings to CRLF (which probably explains why it looks like a CRLF to you). And then when you make a commit, git will change them back to LF. So this all happens transparently to the user. But for some reason, it seems like git isn't converting the the CRLF back to a LF for this file. I have no idea why it wouldn't do that. All your other PR's have been fine. Maybe something is weird about this one file?
Note that it's also not a bad thing that you're using windows. It's great to have a diversity of build environments so tha we are more likely to catch environment specific issues. And it'll be good if we can figure out this issue so that we know how to fix if if more windows developers join the project and hit this problem.

Very interesting. I need to go deeper into this. I down loaded the zip of the repo to by pass git, and see that SDE.scala does not contain 0D. So git is having fun with me. Weird that it’s just this file. Stay tuned. And thanks.

On Aug 28, 2020, at 9:10 PM, Rick Steinberger<EMAIL_ADDRESS>wrote:
SDE.scala in master has LF line endings for me, and git on linux doesn't do any munging of line endings linke winows can, which confuses things. And I've checked out your PR branch, and then SDE.scala has CRLF line endings. So it definitely seems to me that this PR is changing from LF to CRLF, which we don't want.
Ususally git on windows has the core.autocrlf configuration option set to true. This makes it so that when you checkout a branch, git wil convet the line endings to CRLF (which probably explains why it looks like a CRLF to you). And then when you make a commit, git will change them back to LF. So this all happens transparently to the user. But for some reason, it seems like git isn't converting the the CRLF back to a LF for this file. I have no idea why it wouldn't do that. All your other PR's have been fine. Maybe something is weird about this one file?
Note that it's also not a bad thing that you're using windows. It's great to have a diversity of build environments so tha we are more likely to catch environment specific issues. And it'll be good if we can figure out this issue so that we know how to fix if if more windows developers join the project and hit this problem.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/apache/incubator-daffodil/pull/412#issuecomment-683286201, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABF4TW6QZTHRHGCEXP3V34LSDD2SLANCNFSM4QOJ73LQ.

Rick, I use a Windows 10 (winver 1909) laptop too.  I made git checkouts using Cygwin git for a long time using the following snippet inside my ~/.gitconfig:
[core]
        safecrlf = true
        autocrlf = input

However, I now develop in Windows Subsystem for Linux (WSL) almost exclusively since the WSL experience (including Microsoft Terminal) has improved to the point where I feel like I'm using both Ubuntu 20.04 (inside Microsoft Terminal except for Emacs and IntelliJ IDEA which are running in their own windows using the VcXsrv X server) and Windows 10 at the same time.  The only concession I've made for Windows in my Ubuntu 20.04 environment's ~/.gitconfig is to tell git to ignore files' executable bit in case a Windows program has created a file in my git checkout or my checkout is in a Windows filesystem instead of an Ubuntu filesystem:
[core]
        filemode = false

You may want to try WSL too which will allow you to mix and match your favorite development tools as you wish.  For example, I also run Visual Studio Code natively on Windows with a plugin that performs remote compilations and debugging inside WSL.

Should all be correct now. Please let me know if you still see issues. It turned out to be a combination of config setting in git vs Intellij. I now have get configured with git config --global core.autocrlf false to do no conversion. And Intellij configured to use only LF, at the project level. That should do it. But, I will certainly keep an eye on it. All documented in the New Developer doc.


Integrate remote translog download on failover

Signed-off-by: Ashish Singh<EMAIL_ADDRESS>
Description
Solves #5637
Issues Resolved
Solves #5637
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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

https://github.com/opensearch-project/OpenSearch/issues/5670 for writing integrations tests since much of the change is on integration.

Codecov Report

Merging #5699 (3276c06) into main (28e9b11) will increase coverage by 0.15%.
The diff coverage is 71.42%.

:exclamation: Current head 3276c06 differs from pull request most recent head 3a56b49. Consider uploading reports for the commit 3a56b49 to get more accurate results

@@             Coverage Diff              @@
##               main    #5699      +/-   ##
============================================
+ Coverage     70.98%   71.14%   +0.15%
- Complexity    58669    58784     +115
============================================
  Files          4763     4763
  Lines        279945   279952       +7
  Branches      40418    40418
============================================
+ Hits         198731   199179     +448
+ Misses        64988    64551     -437
+ Partials      16226    16222       -4

Impacted Files
Coverage Δ

...java/org/opensearch/index/shard/StoreRecovery.java
65.74% <0.00%> (ø)

...c/main/java/org/opensearch/index/IndexService.java
74.15% <40.00%> (-0.17%)
:arrow_down:

...in/java/org/opensearch/index/shard/IndexShard.java
70.76% <100.00%> (-0.05%)
:arrow_down:

...rg/opensearch/index/translog/RemoteFsTranslog.java
70.83% <100.00%> (-2.90%)
:arrow_down:

...h/index/translog/transfer/FileTransferTracker.java
88.23% <100.00%> (+0.73%)
:arrow_up:

...org/opensearch/index/shard/IndexShardTestCase.java
93.82% <100.00%> (-0.24%)
:arrow_down:

...pensearch/client/cluster/RemoteConnectionInfo.java
0.00% <0.00%> (-73.18%)
:arrow_down:

...a/org/opensearch/client/cluster/ProxyModeInfo.java
0.00% <0.00%> (-60.00%)
:arrow_down:

...a/org/opensearch/client/cluster/SniffModeInfo.java
0.00% <0.00%> (-58.83%)
:arrow_down:

...a/org/opensearch/tasks/TaskCancelledException.java
50.00% <0.00%> (-50.00%)
:arrow_down:

... and 497 more

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

#5670 for writing integrations tests since much of the change is on integration.

One unit test can be added to IndexShardTests. We create a shard with remote translog settings and verify class of the translog factory in the EngineConfig.

Just for my understanding, for failover, do we not need any changes in recovery path except having RemoteStoreTranslogRepository initialised properly? Or are you planning to do follow up changes?

#5670 for writing integrations tests since much of the change is on integration.

One unit test can be added to IndexShardTests. We create a shard with remote translog settings and verify class of the translog factory in the EngineConfig.

Yes, added.

Just for my understanding, for failover, do we not need any changes in recovery path except having RemoteStoreTranslogRepository initialised properly? Or are you planning to do follow up changes?

This is enough. Since earlier the engine was NRTReplicationEngine with InternalTranslogFactory, we have to bootstrap the engine with the correct type. The changes have been done in bootstrap such that whenever the Engine bootstraps, the segments and translog are downloaded from remote store. However, that is also the default behaviour. This behaviour is helpful for failover and should stay the same. We will, however, have optimisation done to make sure that we are downloading on so much that is required.

Changes LGTM.


Set the scene render mode to intermediate before texture.render

When i tried to do screenshot on scenes containing instances, the first screenshot worked well, everything was displayed, as you can see below.

But when i tried to do screenshot a second time some meshes disappeared, all of them are instances.

When doing a screenshot using render target for the first time, the scene
intermediate rendering is set to false at the end of the
texture.render().
Next time the function is called it makes all instances being avoided
during the render.
This PR is essentially a hotfix and should be edited in order to fix the bug in a correct way.

It looks like the bug is fixed with 4.1.0 beta4

Ok thanks! closing then ;)

I did some more testing and just realized it's not fixed in beta-4 could you reopen it @deltakosh ?

It should be from the latest nightly and so in the next npm version. We worked on the same issue yesterday with @deltakosh #7219


return a 404 for .well-known

Server getting hammered by requests to .well-known. It always hits the database and locks up database, memory, etc.

Not needed with nginx changes


Can't open the D435 color camera using LabVIEW

Required Info:
Camera Model - D435
Operating System & Version - Win 10 64bit
Language - LabVIEW 2017 64bit
Objectives:
I would like to open the D435's color camera using LabVIEW
My problem :
I would like to open the D435's color camera using LabVIEW, and I using the 'RealSense_HelloWorld_Depth_and_Color.vi' downloaded from https://github.com/IntelRealSense/librealsense/tree/master/wrappers/labview, but the program alerted me to an error, which said :
Error 1097 occurred at Call Library Function Node in libRealSense3.lvlib:rs3 open multiple sensor.vi->libRealSense3.lvlib:rs3 start all 2.vi:2930001->RealSense_HelloWorld_Depth_and_Color.vi
Possible reason(s):
LabVIEW:  An exception occurred within the external code called by a Call Library Function Node. The exception might have corrupted the LabVIEW memory. Save any work to a new location and restart LabVIEW.
I located the problem in rs3 open multiple sensor.vi, maybe the realsense2.dll didn't contain the function which can open the color camera? Could you help me solve this problem? I have to use LabVIEW due to work.
Thanks
Joe

It is because that demo is trying to open the LEFT stereo camera color channel, and not the third RGB sensor. The D435 is only monochrome so can’t open as a color channel.

Hi eyhtqd, Do you need more help on this?

Hi eyhtqd, Do you need more help on this?

Yeah, I still haven't solved the problem. Could you tell me how to solve it?

Hi eyhtqd, Do you need more help on this?

Yeah, I still haven't solved the problem. Could you tell me how to solve it?

If you want to stream the color camera then you need to look at one of the other hello world demos. Try to run them all.
The hello world you chose says it was written for the D415, which uses color sensors for stereo and it therefore opens a “left color camera” which is the stereo camera. I hope that helps.


Default export is not working with typescript

Summary:
Using typescript, if I import with import ReactModal from 'react-modal'; then ReactModal is undefined. It usually means that the module has no default export.
import * as ReactModal from 'react-modal'; works but does not satisfy the definitions from<EMAIL_ADDRESS>Steps to reproduce:

test.ts
import ReactModal from 'react-modal';
console.log(ReactModal);

ts-node test.ts
Expected behavior:
Should log a component but logs undefined

@testerez we have recently release a patch for fix the imports. Can you check if this happens on latest version?

Tested with version 2.3.2 which seems to be the latests.

Is it still failing?

.babelrc#L4 This was recently added. It shouldn't cause any conflict, but you can try to remove it and see if it helps...

I have the same problem. I tried:

.babelrc#L4 This was recently added. It shouldn't cause any conflict, but you can try to remove it and see if it helps...

It dit not help.
I am using typescript with the versions:
"react": "15.4.2",
"types/react": "15.0.38",
"react-modal": "2.3.2",
"types/react-modal": "2.2.1",
my code looks like this:
import ReactModal from 'react-modal';
<ReactModal isOpen={true} contentLabel='Hello world'>
Hello World
 </ReactModal>
The error I get is:

Uncaught Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in. Check the render method of CardInputContainer.

When I use: import * as ReactModal from 'react-modal';
I get the compile error:

Error TS2604: JSX element type 'ReactModal' does not have any construct or call signatures.

I hope you can help me with this.

@MikeMolIS Perhaps, the types/react-modal to keep sync with new releases (?). I understand how the type provider works, but I'm not familiar with this repo. Can you post the links to this repo?

Here it is: https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/react-modal/index.d.ts
But the definitions does not change anything to the output. They just help giving warnings if the typing is incorrect. Be aware that babel does some magic with the imports:
import foo from 'foo' means that you import the default export of 'foo'. If foo has no default export foo should be undefined. But Babel add some glue so if foo has no default export you will get *. Typescript is more strict and does not do that. It could explain why import ReactModal from 'react-modal'; works with babel but not with typescript.

The confusing babel output.
exports.default = _Modal2.default;
module.exports = exports['default'];

Not using defaults just makes the world a happier place,
without a default people can still just rename the var if needed using:
import {ReactModal as RModal} from "react-modal" so the default syntax doesn't add anything.
(Well you can skip two { } i guess).
So the most comfy fix would be to skip the default.
Then it would work for everyone.
Background:
exporting 'exports.default' rather than 'exports' #706

Any update on the same ?
Due to usage of babel-plugin-add-module-exports, code spit out is incompatible with webpack 2.2. One way to fix this is defined in this gist.

@bogas04 Sounds great. This is still open, they have changed on their side (DefinitelyTyped), but if we can provide the correct way, it would be nicer. Can you PR this?

This is still happening #768

Importing ReactModal with default exports:
import * as ReactModal from 'react-modal';

Still got problems with this

@ru-web-designer You should import with import ReactModal form 'react-modal';.
Or...see what this logs:
import * as ReactModal from 'react-modal';
console.log(ReactModal);


[b3sum] Init with rayon thread pool seems slow

Before going anywhere: b3sum is an incredibly fast tool, compared to currently existing solution.
I was testing it with small files, and it seems that md5sum is faster. I was wondering if it was a matter of binary size, but running valgrind --tool=callgrind on it reveal that it is more the execution of std::sys::unix::thread::guard::init that is probably triggered by https://github.com/BLAKE3-team/BLAKE3/blob/b776e8f32885b58b850c03ccdb66c72dfb404459/b3sum/src/main.rs#L591
I tried with --num-threads 1 but as the code is written, this is still executed.
Is it on purpose? For now it is up to 3 times slower than md5sum for small files. I guess that if the number of threads is 1, rayon could be excluded entirely?

Hmm, it might make sense to lazily initialize the thread pool, so that we only pay this cost if it's actually used. I think this will be easy to do, and if so then why not.
That said, I haven't put very much effort into optimizing the "many small files" scenario. (The one thing we do is skip memory mapping under 16 KiB.) If we really wanted to optimize that scenario, we'd probably need to do a multithreaded directory traversal similar to what ripgrep does. That's a pretty substantial architectural overhead for an uncommon scenario, and most applications that actually care about this scenario will probably want to do things in-process rather than shelling out to b3sum.

I'm seeing some strange results when I try to benchmark this. Here's a script that runs b3sum and md5sum repeatedly against a small file:
#! /usr/bin/env python3

import subprocess
import tempfile
import time

def measure_command(command):
    times = []
    for _ in range(100):
        start = time.monotonic()
        subprocess.run(command, stdout=subprocess.DEVNULL)
        end = time.monotonic()
        times.append(end - start)
        # time.sleep(0.001)
    times.sort()
    median = times[len(times) // 2]
    microseconds = round(median * 1_000_000)
    return microseconds

with tempfile.NamedTemporaryFile() as f:
    f.write(b"hello world")
    f.flush()
    for exe in ["b3sum", "md5sum"]:
        microseconds = measure_command([exe, f.name])
        print(f"{exe:>6}: {microseconds:>4} μs")

The results are pretty noisy, but here's a representative example:
 b3sum:  879 μs
md5sum:  524 μs

So it looks like md5sum is faster. However, if I uncomment the 1 ms sleep in the script above, I get totally different results:
 b3sum:  969 μs
md5sum: 1415 μs

With small sleeps in between runs, the measured runtime of md5sum almost triples. I have no idea why that might be, and I'm curious whether you have any idea what I'm measuring. This is on an up-to-date x86_64 Arch Linux laptop with an i5-1145G7 CPU.

I tend to use hyperfine for that kind of benchmarks:
echo "hello world" > /tmp/test.txt
hyperfine "b3sum /tmp/test.txt" "md5sum /tmp/test.txt"

Here are my result an a slow machine (Intel Atom D2550):
Benchmark #1: b3sum /tmp/test.txt
  Time (mean ± σ):       6.9 ms ±   1.3 ms    [User: 2.5 ms, System: 6.7 ms]
  Range (min … max):     5.6 ms …  11.4 ms    299 runs

Benchmark #2: md5sum /tmp/test.txt
  Time (mean ± σ):       2.7 ms ±   0.7 ms    [User: 1.7 ms, System: 1.3 ms]
  Range (min … max):     2.0 ms …   7.6 ms    589 runs

  Warning: Command took less than 5 ms to complete. Results might be inaccurate.
  Warning: Statistical outliers were detected. Consider re-running this benchmark on a quiet PC without any interferences from other programs. It might help to use the '--warmup' or '--prepare' options.

Summary
  'md5sum /tmp/test.txt' ran
    2.59 ± 0.82 times faster than 'b3sum /tmp/test.txt'

But you're right about over optimizing: it seems useless. I think that just skipping the thread pool is a good start. Implementing a parallel directory traversal is not the goal for this tool. Instead, find … | xargs should be used. And usually, when there is a lot of multiple files, the list is already known, like RPM or DEB package content.

Here's what I see with hyperfine:
Benchmark 1: b3sum /tmp/test.txt
  Time (mean ± σ):       0.1 ms ±   0.4 ms    [User: 0.6 ms, System: 0.8 ms]
  Range (min … max):     0.0 ms …   4.2 ms    613 runs

Benchmark 2: md5sum /tmp/test.txt
  Time (mean ± σ):       0.4 ms ±   0.5 ms    [User: 0.8 ms, System: 0.8 ms]
  Range (min … max):     0.0 ms …   1.5 ms    827 runs

Summary
  'b3sum /tmp/test.txt' ran
    2.94 ± 10.45 times faster than 'md5sum /tmp/test.txt'

I'm guessing these measurements are dominated by OS cache behavior, more than anything the programs themselves are actually doing, but that's still just a guess.

@oconnor663 commented on 24 avr. 2022, 21:45 UTC+2:

I'm seeing some strange results when I try to benchmark this. Here's a script that runs b3sum (unchanged) and md5sum repeatedly against a small file:
So it looks like md5sum is faster. However, if I uncomment the 1 ms sleep in the script above, I get totally different results:

I tested with my slow server. I changed the script to iterate 1000 times. Without the sleep I have:
b3sum: 7427 μs
md5sum: 3352 μs

And with the sleep, I have:
 b3sum: 7302 μs
md5sum: 3302 μs

So for me, it looks consistent. It seems that benchmarking fast commands is not an easy task :wink:


arch/xtensa: Automatically generate interrupt handlers (finally)

The script to generate the _soc_inthandlers.h header has been run manually for years, only because I was a cmake novice at the time and unsure how to integrate it into the build.  So every new platform has to find the script and template file and figure out how to generate the file.  And in a few cases it looks like we've tried to EDIT the resulting files in the tree.
Let's finally do this right.  The file is now dropped (for every xtensa platform) as a "xtensa_handlers.h" file, and there is a Kconfig to control whether the original/manual file or the new one is used by the platform code.  We can migrate the other platforms slowly as people have time to validate.

I include the integration for dc233c only as that's all I can smoke test easily.  Other platforms can set the kconfig and remove their own _soc_inthandlers.h at leisure.

Quick ping for review.  @dcpleung and @ceolin in particular want to look at this.


Please add support for Xiaomi Redmi 9

Hello to the respected developers of Magisk Hide Props Config. I immediately apologize for writing here, not on Xda-Developers (since I have trouble entering).
Thank you for understanding!
Manufacturer: Xiaomi
Device: Redmi 9
Codename(s): lancelot (without NFC); galahad (with NFC)
Security Patch date: 2020.11.01
AndroidDump (galahad): https://git.rip/dumps/redmi/galahad
Model(s): M2004J19AG and M2004J19PI - without NFC; M2004J19C - with NFC
Props:
ro.product.model (galahad, NFC)- M2004J19C
ro.product.manufacturer - Xiaomi
ro.build.fingerprint - Redmi/galahad_id/galahad:10/QP1A.190711.020/V<IP_ADDRESS>.QJCIDXM:user/release-keys
Thank you in advance!

No worries, submitting prints here works great as well.
I've tested and I managed to get a print from that Android Dumps link to pass CTS. Not the one you've submitted here though...
Will add that one to the list in the next update.

Thank you, and tell me when you plan an update?

Added in 309dcf9e2ba6b93df256269654cfd01fba76f456


Refactor lower implementation to improve readability

Summary or solution description
Refactored lower built-in method.
Part of an overall refactoring to unify code generator logic and built-in methods generation.
How to Reproduce
https://github.com/CityOfZion/neo3-boa/blob/12aa115d6258a0f4488c7a350b5243362b6422da/boa3_test/test_sc/string_test/LowerStringMethod.py#L4-L6
Tests
https://github.com/CityOfZion/neo3-boa/blob/12aa115d6258a0f4488c7a350b5243362b6422da/boa3_test/tests/compiler_tests/test_bytes.py#L1161-L1185
https://github.com/CityOfZion/neo3-boa/blob/12aa115d6258a0f4488c7a350b5243362b6422da/boa3_test/tests/compiler_tests/test_string.py#L912-L941
Platform:

OS: Windows 10 x64
Python version: Python 3.7+

Task linked: CU-864eq2tfm Refactor lower implementation to improve readability


Dev angel coronado quintero final delivery

What does this PR do?
Unit Testing Mocks
Movie Reviews Module in MVVM

Where should the reviewer start?
In Reviews, all the way down to every module
Code review checklist Code Review Rubric

DRY Principle (Don’t repeat yourself)

[ ] Excellent
[x] Good
[ ] Adequate
[ ] Developing

Yagni Principle (You Aren't Gonna Need It)

[ ] Excellent
[x] Good
[ ] Adequate
[ ] Developing

KISS Principle (Keep It Simple, St...d)

[x] Excellent
[ ] Good
[ ] Adequate
[ ] Developing

Remove unused code

[ ] Excellent
[x] Good
[ ] Adequate
[ ] Developing

Hard-coded values

[ ] Excellent
[x] Good
[ ] Adequate
[ ] Developing

Language style guide & coding conventions

Wizeline Style Guide

RayWenderlich Style Guide

[ ] Excellent
[x] Good
[ ] Adequate
[ ] Developing

Architecture / Design Pattern

[ ] MVC
[ ] MVP
[x] MVVM
[x] VIPER

Architecture Implementation

[ ] Excellent
[x] Good
[ ] Adequate
[ ] Developing

SOLID

Single Responsibility Principle

[ ] Excellent
[x] Good
[ ] Adequate
[ ] Developing

Open Closed Principle

[ ] Excellent
[x] Good
[ ] Adequate
[ ] Developing

Liskov substitutability principle

[x] Excellent
[ ] Good
[ ] Adequate
[ ] Developing

Interface segregation

[x] Excellent
[ ] Good
[ ] Adequate
[ ] Developing

Dependency Inversion principle

[ ] Excellent
[x] Good
[ ] Adequate
[ ] Developing

Testing

[ ] Excellent
[ ] Good
[ ] Adequate
[x] Developing

Force unwrapping !

[x] Excellent
[ ] Good
[ ] Adequate
[ ] Developing

Mutable vs immutable property (Right use of let  and var)

[ ] Excellent
[x] Good
[ ] Adequate
[ ] Developing

Retain cycles (Capture list)

[ ] Excellent
[x] Good
[ ] Adequate
[ ] Developing

Code Documentation

[ ] Excellent
[ ] Good
[ ] Adequate
[x] Developing

hahahaha dos cosas Angel

El PR debe ser desde el upstream wizelineacademy/ios-bootcamp-project-2021 y no desde este fork
El PR debe ser hacia development-Angel-Coronado-Quintero desde una feature branch


Same namespace used for both users and items

I just noticed this, and it's surprising that nobody pointed it out before.
This can create some conflicts, especially if numbers are used for user & item IDs.
I will attempt to fix it myself in my existing PR.
Edit: done. #41

@gabe-alex Wow, yeah huge that no one pointed that out. Fixed it in my own. Sorry I didn't see your fix in time. I'm going to be actively working on the project again. Are you still working on it? Thanks for all the work thus far. I'll work on getting some of the fixes in. Take a look at the current master.

Lemme know if you're still interested. Lots of cool features to build.

@gabe-alex All done https://github.com/guymorita/recommendationRaccoon/commit/1e6c0728233c23ace186b91eee8d579d8f243717


Can you give an option to export .svg animation file from bodymovin in future?

What do you mean by .svg animation?
Like an svg strip?

Yeah i mean any possibility to export a svg file from after effects?
please go through this link : https://css-tricks.com/guide-svg-animations-smil/
It would be greate if we get after effects animations in css code (svg also) just like animate cc is capable of exporting. It will revolutionize web animations in the entire industry. once please check it out.

Unfortunately all those options fall short.

SMIL is not adopted by all browsers and even those that support it have a lot of issues.
CSS can work for some animations, but it's very limited.
As far as I know, Animate CC exports frame by frame information of the animation, and that's something I'd rather not do.
If this made sense at some point I would consider it but for now it's too much work for little gain.
For now the only way to support many of AE's features is to use Javascript.

Yes, ofcourse what you said is absolutely true.
But since css is the interface design code, which loads before javascript, I have found that css animations load faster and occupy less space when compared to the speed at which json files are doing the job.
also many experts warned me not to use json files in larger numbers.
what would you suggest if i had to make animated icons? is it better to do it in json format or svg?

Yes, if I could leverage CSS for animations I'd definitely do it.
It runs on a different thread and enables more optimizations by browsers.
But shape interpolation, filter animation, expressions, trim paths, repeaters, parametric shapes, motion paths are impossible to support with CSS.
When the WAAPI, which brings the best of the two worlds together, gets to a more mature state I'll definitely use it as the underlying engine of animations.

ok ya got your point. But anyway its a great start to be frank.. will be waiting for much more ! Thank you.


About Dataset Format

Thank you for your good work !
I followed your instructions to reproduce the results. However, I encountered an issue with the dataset format downloaded from the link in your repository. Your code requires CSV files and an image folder, but the files I obtained after extracting the downloaded package are in pickle format.
What steps can I take to convert the pickle files to the required format for your code?

Thanks for your issue!
MiniImagetNet needs CSV files to split datasets, We follow SnaTCHer and use the same split of Ravi & Larochelle. You can download the split file here and use pickle to preprocess pickle files into images or just download the original images here.

"Thank you for your response.
I have resolved that issue. Additionally, I have another question.
How many GPUs did you use to train the model?"

Sorry for the late response.
We use a single NVIDIA 3090 to train our model.

The issue has been resolved, so I will close it here.
Thank you very much!


Rounding law support

This PR introduces support for Rounding Laws. Rounding Laws are the rules pacehrh uses to determine if and when computed values are rounded. The Rounding Law is set with the SetRoundingLaw() function. There are three rounding laws - "early", "late", and "none" - and the differences between them are described in the documentation to SetRoundingLaw().
The first applications of RoundingLaw = "none" (which preserves full decimal precision) are to population predictions, allowing more precise unit testing of population functions.
This work is in support of issues #67 and #133, but doesn't close them.

Good catch! I’ll repair shortly.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Rui Han @.>
Sent: Tuesday, October 4, 2022 2:52:28 PM
To: InstituteforDiseaseModeling/PACE-HRH @.>
Cc: Charles Eliot @.>; Author @.>
Subject: Re: [InstituteforDiseaseModeling/PACE-HRH] Rounding law support (PR #194)
@rhanIDM commented on this pull request.

In pacehrh/tests/testthat/test-001-pace_config.Rhttps://github.com/InstituteforDiseaseModeling/PACE-HRH/pull/194#discussion_r987227188:

@@ -199,13 +199,16 @@ test_that("Global configuration: check-and-load", {
test_that("Global configuration: SetGlobalStartEndYears", {
e <- pacehrh:::GPE

withr::defer(e$startYear <- originalStartYear)

local_vars("startYear", envir = e)
local_vars("endYear", envir = e)
local_vars("endYear", envir = e)

Are line 203 and 204 duplicates?
—
Reply to this email directly, view it on GitHubhttps://github.com/InstituteforDiseaseModeling/PACE-HRH/pull/194#pullrequestreview-1130462833, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMVD2PDNBZ3YNEQMW3TWTN3WBR4GZANCNFSM6AAAAAAQ4OGRCA.
You are receiving this because you authored the thread.Message ID: @.***>


add a very simple test case for tagging, and fix the missing import

hey Joe! the Travis CI build failed b/c there was an error in the travis config, which I just fixed. can you pull my last commit into your branch, so that I can make sure everything passes before pulling this in?

cool, thanks @JoeGermuska!! :grinning:


All view should emit doc._id

For the next data-system version, request 'all' should emit doc._id as key.

Thanks!


Roadmap 2024

Use case
Not sure where to put this, but in the last years around January, it was always nice reading the roadmap for the upcoming year.
Currently it is still showing 2023: https://github.com/flutter/flutter/wiki/Roadmap
Proposal
Is there a new version for this document in the works?

I believe the best place for this would be discord.
I understand one is being worked on but it is not available yet. The wiki entry will be updated when it is available.
You may reach out to Hixie on discord for further information.
Closing from here as the roadmap will be updated when it is ready.
Thank you


Update autoUpdater smoke tests for new API

This will need to be forcefully merged and published manually because the tests won't pass until https://github.com/electron/electron/pull/11925 is merged and that PR won't be merged until this is released 👍

@MarshallOfSound in order to publish this, we'll need to manually change version in package.json (without making the change land on master) because semantic-release expects version to be 0.0.0-development

@zeke I can do that 👍 Gonna merge this in to master now and manually release from my local 😄
Heads up all tests will be failing for a while 😆


Release v4.1 isn't available in npm

Release v4.1 isn't available in npm

Release v4.1 isn't available in npm
It is there, but npm install still installs v4.0

@mdo: you have marked 4.1.0 as next. It should be as latest.

https://docs.npmjs.com/cli/dist-tag

Should be all cleared up now!

@XhmikosR Heads up, I opened a PR in our team repo to fix my shipping doc so I don't screw this up next time :).


feat: remove triple new lines in generated output

I understand this is a very subjective thing, but I've been going through and deleting the third new line from all my generated change logs manually, so I wondered if you would consider making it a change in the gem.
As an example, it changes this:

### v1.13.0 (2017-10-28)

#### Features

* **gems**
  * update to pact-mock_service 2.5.1	 ([1b0ed8b](/../../commit/1b0ed8b))

into this:

### v1.13.0 (2017-10-28)

#### Features

* **gems**
  * update to pact-mock_service 2.5.1	 ([1b0ed8b](/../../commit/1b0ed8b))

I tried just doing a gsub in the code after generating it, but the way the Writer is constructed, opening the file with r+ in the constructor seems to lock the file somehow. I can't reproduce it in the tests however. If you're not keen on making this change, I might do a PR to fix the locking problem instead.

It's ok, I like it 😄
Thanks 👍

Thanks! Ping me when you've done a release so I can update all my projects. I've been slowly adding it to all the ruby pact projects that I work on, and liking it. I'd love the breaking changes to also be included as the js one does ;)


inconsistent integration_tool behavior after put

Hi!
So I was using checkout for integration_tool after the release and I spotted a weird behavior during the put phase where it updates the PR state.
A picture says a thousand words so here's the screenshot.

So the weird behavior is after the PR status update, it repulls again. This is a pretty standard concourse behavior, but the weird part is, the pr resource is now doing a merge when it pulls.
Is there a way to make the re-pulls consistent with the initial get?
I tried setting integration_tool on the puts but it just errors out saying that it doesn't recognize the integration_tool param for puts.

actually my bad!
forgot about the get_params skip downloads so please disregard!


Document install & update

The procedure for installing or updating code-debug is unclear. Starting with this page:
https://marketplace.visualstudio.com/items?itemName=webfreak.debug
There is a green "Get Started" button at the top of the page which takes you to some good instructions (https://code.visualstudio.com/docs/editor/extension-gallery?pub=webfreak&ext=debug) on how to install code-debug. The first time I installed code-debug, I clicked on that button and installed without too much trouble. However, later I wanted to try to reinstall, and I clicked on the "Get Started" link on the right side of the webfreak.debug page under "Resources." That took me to https://github.com/WebFreak001/code-debug , and I wasn't sure how to proceed. I clicked "Download ZIP" but then couldn't figure out what to do with the zip file.
I would suggest several things to avoid confusion:

On the webfreak.debug page, in the main text body section, insert a new, first section before the "Debug" section titled "Installation" with the text "To install this extension to Visual Studio Code, please follow the instructions at https://code.visualstudio.com/docs/editor/extension-gallery?pub=webfreak&ext=debug ."
Rename the "Get Started" link under "Resources" to "Github page".
Include a "HowToInstall" file in the zip file that contains instructions on how to install or update from the zip file (or if that's not practical, refer to the ext=debug page).
Do something on the github code-debug page to discourage downloading the zip file and encourage following the proper installation instructions.

um the link names (Get Started, etc) are generated by vscode so I can't change them. I could add a quick description to install GDB but installing any extension in vscode is the same with ext install <name>, so I probably won't add that

Well, the Resources / Get Started link is certainly confusing. Are you saying I should complain to Microsoft/vscode?
Anyway, I don't think it would hurt to follow the other suggestions. For example, adding the "Installation" heading and text would give the user a 2/3 chance (instead of a 1/2 chance) of finding the right installation instructions. :-)

Hm sure, gonna just add that to the README. I dont know if you can even modify the sidebar so the README is also fine

I suggest you stick with my original suggestion, and make the Installation instructions: "To install this extension to Visual Studio Code, please follow the instructions at https://code.visualstudio.com/docs/editor/extension-gallery?pub=webfreak&ext=debug ." The instructions at that link worked for me the first time.
On the other hand, the installation instructions you just added to the README were confusing and led me astray. I tried ctrl-shift-P, filled in the line so it said "ext install debug" and hit ENTER. Code did some stuff, and then reported: "Info Cordova Tools 1.0.1 was successfully installed. Restart to enable it. Restart Now." I don't think I wanted Cordova Tools!
Please, don't assume your users will be very familiar with VSCode, Node.js, or the Microsoft or Linux way of doing things. I think most of your users will NOT be very familiar with some of those things. So please make your instructions VERY clear and step-by-step.

Also, on the page https://github.com/WebFreak001/code-debug , I would suggest adding after "Native debugging for VSCode" near the top of the page, "Please read the README (link) section below for installation and usage instructions."
The problem is that the README is below the list of files, and may not be visible if you're looking at the top of the page. A visitor might just grab the zip file (as I did) and try to figure out how to install it.

@WebFreak001 not sure if you saw the above two comments since you already closed this issue.

The new Readme is now on the vscode page, so I don't need to link it everywhere because it's the first thing on the page. I'm gonna add some more information and maybe a Screenshot that shows it when I'm at home


unexpected keyword argument in textworld package

Getting this error whenever I start to play with the textworld.  $alfworld-play-tw.
Error : TypeError: init() got an unexpected keyword argument 'wrappers'

Hi. What is the full stacktrace for this error?

Hi, I followed the steps for installation as given. Whenever I try to play with the TextWorld by writing the command $alfworld-play-tw I get the error

Hi, everytime I pip install the above TextWorld package, I get the following error :

I however directly used,  $pip install textworld
Is it that reason why the above error is showing up?

Hi, everytime I pip install the above TextWorld package, I get the following error :

I however directly used, $pip install textworld
Is it that reason why the above error is showing up?

I have the same error. Waiting for the solution ...


README suggestion

After some experimentation I've been able to successfully integrate this into my application, thank you!
After the "Configuration" section and before the "Capture" section in the README's Quick Start can I recommend adding a simple example of how to start a segment and subsegment for those developers who aren't working with HTTPS based services:
// Start a segment:
ctx, seg := xray.BeginSegment(context.Background(), "my-service-name")

ctx, subSeg := xray.BeginSubsegment(ctx, "Some unit of work")
// ...
// some unit of work here
// ...
subSeg.Close(nil)

// Close the segment
seg.Close(nil)

@alexbilbie , we released a new version of SDK which contains your suggestions. Thank you for your suggestion.


Fix popup menu

Fixes popup menu dimensions, padding, color, and font, backports horizontal gravity and anchor overlapping support, and a few other bugfixes/enhancements.
https://openedx.atlassian.net/browse/MA-1270

@bguertin @miankhalid please review

Not an expert @ Custom Views and especially PopupMenu, but still PR looks solid :+1:

@bguertin, would you also like to review this, or should I just merge it? cc @aleffert

Looks fine, but there's a conflict (Probably since Akiva swapped the icon shown in the toolbar?) :+1:

Yes, I swapped around the popup styles in PR #381, which conflicts with the style edits here. I'll merge it.


fix code style

Fixed code style
While going through modern-js-cheatsheet I've realized that there are some minor styling mistakes with the code samples.
I've tried to keep everything consistent by putting single space after the // comment. Hope it looks cleaner now. 👍

Yup it definitly looks better 👍 I will pay attention to that in future PR's, thanks !


Expired certificate on license page.

The page https://creativecommons.org/licenses/by/4.0/ includes a link to the piwik tracking application ( https://stats.creativecommons.org/piwik/piwik.js ), however the SSL certificate for this server has now expired and so browsers won't load it.

Thank you for reporting this!
This was fixed with the move to Let's Encrypt on stats.cc .


Thoughts: Change assembly versioning to match imgui

I'm considering changing ImGui.NET's assembly versioning to follow the versioning of imgui's -- at least partially. It probably makes sense to use a scheme like x.y.z, where x.y is the imgui version (currently 1.53), and z is a patch version for minor fixes made to ImGui.NET itself. This would make it a lot easier to tell which version of imgui you are getting with a particular version of ImGui.NET, since you currently need to crawl through the commit history to do that.
The only downside is now there will be an unusual bump from 0.4.0 to 1.53.0. Not a big deal, but it might be confusing initially.

I will likely switch to this convention when the 1.60+ update happens.

I'm following this from now on, and the newest versions will start from 1.65.0 (Dear ImGui 1.65).


Prep 0.9.6 | /workflows/release-pr

Hey just FYI @krisbitney the plugin tests are failing here with the new ipfs-uri-resolver-plugin changes we added, timeout cancelation doesn't seem to be working if I'm not mistaken... resolution is working fine though so going to push this through.


Implement MongoDB

Value Proposition
As a user, I want to have access to my stored data from any device/location so that can use them independently.
Description
Set up connection to database and persist data in there
Acceptance Criteria

[ ] User have access to his stored data independently of the device/location

Tasks

[ ] Create branch feat/database
[ ] Create new DB & Collection
[ ] Insert example array in collection
[ ] Add .env.local file
[ ] Add db.Connect.js in lib
[ ] Add models folder/files

nothing to say. Perfect and clean


⚠ Bofrost.at has degraded performance

In 3674c13, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1428 ms

Resolved: Bofrost.at performance has improved in e999c8e after 8 minutes.


include terminal chat in panel chat intro

The editor's inline chat is referenced for discoverability in the panel chat's intro. We should do something similar for terminal chat.

We want this wording to be very concise, I'm not sure we want to mention terminal (or notebooks) here as it's still just inline chat which is already called out.

The current wording - with only an action to open inline chat in the editor - might lead a user to believe that's the only one.

Asking for feedback in #ux

We've not heard many opinions here and those that we've heard have been no


Minimize allocations

This patch uses Vec::with_capacity rather than Vec::new to minimize the work
of the memory allocator. Have used tris.len() in two places to make the size
self-documenting.

Thank you for this optimization tip


Add support for python 3.8

Codecov Report

Merging #45 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #45      +/-   ##
==========================================
+ Coverage   89.49%   89.52%   +0.03%
==========================================
  Files          93       93
  Lines       12019    12058      +39
  Branches     1084     1089       +5
==========================================
+ Hits        10756    10795      +39
  Misses       1094     1094
  Partials      169      169

Impacted Files
Coverage Δ

openid/dh.py
100.00% <0.00%> (ø)

openid/test/test_trustroot.py
100.00% <0.00%> (ø)

openid/consumer/consumer.py
95.49% <0.00%> (+0.01%)
:arrow_up:

openid/extensions/pape.py
96.58% <0.00%> (+0.05%)
:arrow_up:

openid/server/server.py
91.58% <0.00%> (+0.07%)
:arrow_up:

openid/extensions/ax.py
89.59% <0.00%> (+0.07%)
:arrow_up:

openid/extensions/sreg.py
92.68% <0.00%> (+0.09%)
:arrow_up:

openid/extensions/draft/pape2.py
94.59% <0.00%> (+0.09%)
:arrow_up:

openid/message.py
91.74% <0.00%> (+0.10%)
:arrow_up:

openid/consumer/discover.py
95.09% <0.00%> (+0.12%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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


enforce two-factor usage outside whitelisted ip ranges.

Previously two-factor authentication was an opt-in per user. There are situations where two-factor auth is required to access a system.
This patch adds a configuration option:
  two_factor_authenticator:
    whitelist:
      - <IP_ADDRESS>
      - <IP_ADDRESS>/24
      - ["<IP_ADDRESS>", "<IP_ADDRESS>"]

This option will enforce two-factor authentication for every login attempt coming from an ip outside the specified whitelist.
An informative text is displayed if the second factor has not been configured yet. The two-factor setup must happen from a whitelisted address.
Note:
This might be a niche use-case that only we are facing. I decided to prepare the PR nonetheless and let you decide wether this is something that you might be interested in getting back upstream.

Yeah, this is probably not a very common use case. I would however like a feature that forces users to setup two-factor authentication on first login/before redirecting to the service (regardless of the IP range).


Can't assign any shortcuts in preference menu v0.11.1

Describe the bug
Can't assign any shortcuts in preference menu.
To Reproduce

Delete existing shortcut
Click out to deselect Shortcut field
Try to click Shortcut field again

Expected behavior
It will select Record Shortcut field to input new shortcut.
Screenshots

Desktop (please complete the following information)

macOS version: Big Sur 11.1
Safari version: 14.0.2
Pass for macOS version: 0.11.1
How was Pass for macOS installed (Github release, built self, ...): homebrew

Additional information

Clicking Record Shortcut field changes to Press shortcut for one sec and then goes back to previous state.
After double clicking Record Shortcut field it changes to Press shortcut, but then you can't type any combinations, only plain text value.
If you write in plain text, deselect field, click on delete x button, then you can write in new shortcut.

Thank you for pointing out this issue. I can reproduce this and have a look into this asap.

This is now fixed in v0.11.2.


remove read_visium_data, add docs for segmentation features

Types of changes

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

Description

remove not needed and unused squidpy/_io.py
add documentation for segmentation feature props

How has this been tested?

.
Closes

Looks great! 2 minor pet peeves:

I think here https://github.com/theislab/squidpy/pull/273/files#diff-825981da66fa773bb4837ca9fc5a9da899f85a1244ecf2ba6ae246ec648ef4c0R275 you can use :ref:`Returns`
the list format should be

- `'area'` - number of pixels of segment.

, i.e. dash instead of color and . at the end.

Looks great! 2 minor pet peeves:

I think here https://github.com/theislab/squidpy/pull/273/files#diff-825981da66fa773bb4837ca9fc5a9da899f85a1244ecf2ba6ae246ec648ef4c0R275 you can use :ref:Returns
the list format should be

- `'area'` - number of pixels of segment.

, i.e. dash instead of color and . at the end.

Ok, I have changed list formatting and added the ref to Returns. I was wondering if that exists, thanks!
Do you have any reason for preferring - over :?

Do you have any reason for preferring - over :?

Personally, I think it creates more distinction between the value on the left and the description on the right (because of the 1 extra space on the left and that - is wider than :).

Thanks for the PR @hspitzer , looks great! I think it can be closed after the checks pass.


Create unit tests for init_mongoDB, server_on, imageUpload, pullImage

No unit test needed for pullImage since it's just a GET request


Add automatic deployment to AWS

There is still a senza template in the repository. The whole deployment process should be automated by something like Lizzy. Each commit should trigger a new deployment.

I configured Jenkins to use lizzy-client to deploy, but for now I'm getting an error from lizzy server:
Requesting new stack.. Deployment failed: 400 Client Error: BAD REQUEST for url: https://lizzy-mentoring.mentoring.zalan.do/api/stacks.

calling senza with same parameters deploys properly to AWS.

Closing as it's deployment related and will be handled in GHE


Generate README for selected Organization

Before this change, links in the README are always generated as OCA/repo_slug despite the user may have answered a different organization to the question

What's the organization slug? If you are creating https://github.com/OCA/server-tools, then write "OCA" here.

Tests fail probably due to https://github.com/ekalinin/nodeenv/issues/324.


KeyValueStoreAssociations missing in CloudFront CreateFunction

Describe the bug
Per AWS Boto3 documentation https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/cloudfront/client/create_function.html
Parameter in FunctionConfig is missing. Looking at the definition the parameter is not listed. Resulting in an Parameter Validation error
   "FunctionConfig":{
      "type":"structure",
      "required":[
        "Comment",
        "Runtime"
      ],
      "members":{
        "Comment":{
          "shape":"string",
          "documentation":"
A comment to describe the function.
"
        },
        "Runtime":{
          "shape":"FunctionRuntime",
          "documentation":"
The function's runtime environment. The only valid value is cloudfront-js-1.0.
"
        }
      },
      "documentation":"
Contains configuration information about a CloudFront function.
"

Expected Behavior
Expected behavior, it should accept
response = client.create_function(
    Name='string',
    FunctionConfig={
        'Comment': 'string',
        'Runtime': 'cloudfront-js-1.0'|'cloudfront-js-2.0',
        'KeyValueStoreAssociations': {
            'Quantity': 123,
            'Items': [
                {
                    'KeyValueStoreARN': 'string'
                },
            ]
        }
    },
    FunctionCode=b'bytes'
)

Current Behavior
ParamaterValidation error:
    raise ParamValidationError(report=report.generate_report())
botocore.exceptions.ParamValidationError: Parameter validation failed:
Unknown parameter in FunctionConfig: "KeyValueStoreAssociations", must be one of: Comment, Runtime

Reproduction Steps
Create function from boto3 cloudfront client
Possible Solution
No response
Additional Information/Context
No response
SDK version used
1.34.60
Environment details (OS name and version, etc.)
Mac OS 14.2.1 (23C71), python 3.11

Hi @colask8, thanks for reaching out.  I was not able to reproduce this behavior using the sample code.  Could you provide debug logs of the behavior?  You can get debug logs by adding boto3.set_stream_logger('') to the top of your script, and redacting any sensitive information.  Any additional environment details you can provide would also be helpful.  Specifically, are you using Lambda in this at all?  Thanks!

Hi @RyanFitzSimmonsAK,
Thanks for assistance. After your reply I went to verify the packages in my python venv and found out python3 and python3.11 didnt point to same python site-packages. Issue was one of them had botocore 1.28.x
Managed to upgrade to botocore[crt] and the issue is gone.
Apologies for inconvenience.
KR,
Stojan - Cola


💭 Android: Frameprocessor

Question
When I use useFrameProcessor in android, the frame.width is always greater than frame.height. Is this correct? In IOS I see the frame.height is greater than frame.width.
What I tried
No response
VisionCamera Version
4.0.1
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar questions in the issues page as well as in the discussions page and found none.

Yes.

It is correct. That's because the Frame is rotated on Android (camera sensors are just rotated).
See orientation.

@mrousavy Just want to confirm on your comment, even if the camera is in portrait we will get orientation as Landscape-left in android?

Yes because it's the Frame's orientation.
Not the phone's orientation.

Is their any way I can make it Portrait?

what lol
the sensor orientation is landscape

🫣Okay, thanks for the quick response. I think I should get more native knowledge.