IR-based guards library

This PR adds a new library for guard conditions based on the IR. There's a shim layer intended to provide the same API and results as the existing guards library, but it's not precisely identical.
The first two changes add a RelationalInstruction class to hold some useful member predicates. The third adds the library and a copy of the tests from the old library. The fourth accepts test output and adds a second set of tests that don't go through the shim layer.

I believe the test failure is related to the internal PR for #194. @geoffw0 can you confirm?

@dave-bartolomeo it occurred to me that the IR has ConvertInstruction as a proper member of the SSA graph. Would there be a conversion to boolean on the 2 in the following code: if(2) { /* do something */ }

I believe the test failure is related to the internal PR for #194. @geoffw0 can you confirm?

Yes, the failure of CWE-119/semmle/tests/OverflowBuffer.qlref can be ignored.

it occurred to me that the IR has ConvertInstruction as a proper member of the SSA graph. Would there be a conversion to boolean on the 2 in the following code: if(2) { /* do something */ }

I don't know, but when I experimented with a nullness query for IR I found that if (myPointer) is compiled to with ConvertInstruction from PointerType to BoolType.

@jbj @rdmarsh2 For if (2), the original AST would have a conversion to bool. However, the result of that conversion would have the constant value 1, so the IR would ignore the original 2 literal and just create a Constant[1] instruction as the operand of the ConditionalBranch.

Running IRGuardsEnsure.ql on ChakraCore with the IR cached:
	IRGuardsEnsure.ql-12:#select#fffffff ................................................................. 16.1s
	IRGuardsEnsure.ql-12:#select#query#fffffffffffffffffffffffff ......................................... 8.8s
	IRGuardsEnsure.ql-12:project#Location::Location::hasLocationInfo_dispred#ffffff ...................... 2.6s
	IRGuardsEnsure.ql-9:IRBlock::IRBlock::dominates_dispred#ff ........................................... 2.5s
	IRGuardsEnsure.ql-9:SSAConstruction::Cached::getInstructionSuccessor#fff_102#join_rhs ................ 1.7s
	IRGuardsEnsure.ql-9:IRGuards::IRGuardCondition#class#fffffff ......................................... 1.4s
	IRGuardsEnsure.ql-9:IRGuards::IRGuardCondition::controlsBlock_dispred#fff#antijoin_rhs ............... 1.3s
	IRGuardsEnsure.ql-9:project#IRBlockConstruction::Cached::getInstruction#3 ............................ 1.3s
	IRGuardsEnsure.ql-9:IRBlock::IRBlock::isReachableFromFunctionEntry#f ................................. 1.3s (executed 716 times)
	IRGuardsEnsure.ql-10:Instruction::UnaryInstruction#3#fffffff ......................................... 1.2s
	IRGuardsEnsure.ql-9:IRBlock::IRBlock::getAnInstruction_dispred#ff_10#join_rhs ........................ 1.1s
	IRGuardsEnsure.ql-10:Instruction::BinaryInstruction#3#fffffff ........................................ 1.1s
	IRGuardsEnsure.ql-11:Instruction::BinaryInstruction#3#fffffff ........................................ 1s
	IRGuardsEnsure.ql-11:IRGuards::compares_eq#ffffff .................................................... 867ms (executed 5 times)
	IRGuardsEnsure.ql-9:IRGuards::IRGuardCondition::controlsBlock_dispred#fff ............................ 812ms
	IRGuardsEnsure.ql-9:IRGuards::IRGuardCondition::controlsBlock_dispred#fff#shared#4 ................... 558ms
	IRGuardsEnsure.ql-12:IRBlock::IRBlock::getLocation_dispred#ff ........................................ 526ms
	IRGuardsEnsure.ql-10:IRGuards::compares_lt#ffffff#join_rhs ........................................... 505ms
	IRGuardsEnsure.ql-12:IRGuards::IRGuardCondition::ensuresEq_dispred#ffffff_051234#join_rhs ............ 501ms```

And running ASTGuards.ql:
	ASTGuards.ql-9:Expr::Expr::toString_dispred#ff .............................................. 8.1s
	ASTGuards.ql-9:SSAConstruction::Cached::MkInstruction#fffffff_1023456#join_rhs .............. 2.4s
	ASTGuards.ql-8:SSAConstruction::Cached::InstructionTagType::toString_dispred#ff ............. 1.9s
	ASTGuards.ql-9:#select#query#fffffff ........................................................ 1.7s
	ASTGuards.ql-9:Location::Location::fullLocationInfo_dispred#ffffff .......................... 1.1s
	ASTGuards.ql-9:SSAConstruction::Cached::MkInstruction#fffffff_2#join_rhs .................... 1s
	ASTGuards.ql-9:SSAConstruction::Cached::getInstructionOperand#fff_102#join_rhs .............. 1s
	ASTGuards.ql-9:Class::Class::getCanonicalMember_dispred#fff ................................. 983ms
	ASTGuards.ql-9:SSAConstruction::Cached::MkInstruction#fffffff_26#join_rhs ................... 954ms
	ASTGuards.ql-9:IRGuards::IRGuardCondition#class#fffffff ..................................... 939ms
	ASTGuards.ql-9:exprparents_20#join_rhs ...................................................... 839ms
	ASTGuards.ql-9:Expr::Operation::getOperator_dispred#bf ...................................... 797ms
	ASTGuards.ql-9:Expr::Expr::getType_dispred#ff ............................................... 771ms
	ASTGuards.ql-9:Access::EnumConstantAccess#class#f ........................................... 750ms
	ASTGuards.ql-9:Element::unresolveElement#fb ................................................. 632ms
	ASTGuards.ql-9:Declaration::Declaration::getName_dispred#ff ................................. 530ms
	ASTGuards.ql-9:Expr::Expr::getValue_dispred#bf .............................................. 509ms

(both of those are just the predicates that took more than 500ms)

I'm considering merging the tests into one file with multiple query predicates to reduce compilation time; the cache is discarded between tests, so there's currently about 10 minutes of unnecessary recompilation of the IR library. @jbj does merging these tests sound reasonable to you?

I've raised the compilation cache issue on Slack, and improvements are coming in 1.19. That won't be out until Christmas, so it sounds reasonable to merge those tests into one file for now.

I don't see any unaddressed comments. @jbj I think this is ready to merge


Corrected broken URLs to point to updated content.

The "Start Free Trial" and "Subscribe Now" links pointed to a 404 page at Pluralsight. I have updated the URLs with destinations that make sense to the average user. The free trial URL remains focused on Scott Allen's ASP.NET MVC 5 course for promotional purposes.
If Microsoft has updated affiliated links, that might be a better option for the future.

The "free" part was there before I even started. :)  Maybe you would want to edit/remove the word "Free"
As for the other link - I'm simply just linking them to their default pricing page as you suggested?

The "free" part was there before I even started.

Yes, that's why I said "pre-existing" in my remarks.
Yes! on the link ... I agree with you that that's the best place to send cats.
The inclusion of pricing info and "free" language is an editorial decision for @Rick-Anderson and @scottaddie to work out. Our guidelines dissuade the practice. I'm just a little worker 🐝 around here, so I'm just making a passing suggestion on it. They'll let you know if they want to make changes to that content.

All good. Sorry missed the pre-existing part. Not a very good excuse on my part, but I was reading your reply on my phone while waiting for coffee! lol

I recommend just linking to 3rd party product/service description/pricing pages and allowing the 3rd party to handle their pitch and pricing descriptions.

+99 - Lets do that.

@ameyer117 I've moved on from morning ☕ to an afternoon 🍺. I'm sure the quality of my next two PR's will go UP! :smile:

Thanks @ameyer117 ... I'll get this in as soon as the build completes.


You shouldn't do this ?

You shouldn't do this and this can have to unexpected, unintended repurcusions. Proceed with caution, you have been warned!
Care to elaborate? I shouldn't use your module ?

hi @felipetesc if you look at the hyperlink there are a few comments mentioning why what I am doing to achieve hot reloading with this library is a bad idea -- hope that can answer your questions! :)


Facing Build Error while building

Building com.okswitzerland.app for device (ios-release)...
Automatically signing iOS for device deployment using specified development team in Xcode project: 257YJF2YD9
Running pod install...                                              6.0s
Running Xcode build...
Xcode build done.                                           16.3s
Failed to build iOS app
Error output from Xcode build:
↳
    ** BUILD FAILED **

Xcode's output:
↳
    Writing result bundle at path:
    	/var/folders/q0/f84pq32x2j36mms_hp33fvhr0000gn/T/flutter_tools.Tm0tDj/flutter_ios_build_temp_dirDsaNWa/temporary_xcresult_bundle

    ld: framework not found Flutter
    clang: error: linker command failed with exit code 1 (use -v to see invocation)
    ld: framework not found Flutter
    clang: error: linker command failed with exit code 1 (use -v to see invocation)
    eyJibHVlcHJpbnRfYmx1ZXByaW50VmFyaWFudCI6eyJnZW5lcmljIjp7fX0sImJsdWVwcmludF9wcm92aWRlckZpbGVQYXRoU3RyaW5nIjoiXC9Vc2Vyc1wvYXNodXRvc2h2aWtyYW1zaW5naFwvZGV2ZWxvcFwvU2FkUGFTYWQgXC9Td2lzc1wvVXNlclwvY29kZWNhbnlvbi1lOXJjbWFRcS02YW1tYXJ0LW11bHRpdmVuZG9yLWZvb2QtZ3JvY2VyeS1lY29tbWVyY2UtcGFyY2VsLXBoYXJtYWN5LWRlbGl2ZXJ5LWFwcC13aXRoLWFkbWluLXdlYnNpdGUgM1wvVXNlciBhcHAgYW5kIHdlYlwvaW9zXC9Qb2RzXC9Qb2RzLnhjb2RlcHJvaiIsInR5cGUiOnsiYmx1ZXByaW50Ijp7fX0sImJsdWVwcmludF9ibHVlcHJpbnRJZGVudGlmaWVyIjoiOEVBNjJGRUFCMjMwNzAwNTBFRkQ0RDQ1MDhENjU5RTUifQ==
    note: Building targets in dependency order
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'nanopb' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'SDWebImage' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'Reachability' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'libwebp' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'PromisesObjC' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'PhoneNumberKit' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'OrderedSet' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'Mantle' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'GTMSessionFetcher' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'FMDB' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'AppAuth' from project 'Pods')
    warning: Run script build phase 'Run Script' will be run during every build because it does not specify any outputs. To address this warning, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase. (in target 'Runner' from project 'Runner')
    warning: Run script build phase 'Thin Binary' will be run during every build because it does not specify any outputs. To address this warning, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase. (in target 'Runner' from project 'Runner')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'GoogleDataTransport' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'GoogleSignIn' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'GoogleUtilities' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'FirebaseInstallations' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 10.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'Firebase' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'FirebaseCore' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 10.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'FirebaseMessaging' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'SDWebImageWebPCoder' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'GTMAppAuth' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'GoogleSignIn-GoogleSignIn' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'FirebaseCoreInternal' from project 'Pods')
    /Users/ashutoshvikramsingh/develop/SadPaSad /Swiss/User/codecanyon-e9rcmaQq-6ammart-multivendor-food-grocery-ecommerce-parcel-pharmacy-delivery-app-with-admin-website 3/User app and web/ios/Pods/Pods.xcodeproj: warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'FirebaseCoreDiagnostics' from project 'Pods')

    Result bundle written to path:
    	/var/folders/q0/f84pq32x2j36mms_hp33fvhr0000gn/T/flutter_tools.Tm0tDj/flutter_ios_build_temp_dirDsaNWa/temporary_xcresult_bundle

Error (Xcode): Framework not found Flutter

Encountered error while building for device.

Hi @ashuviksinfoodal, please provide the following information

Your flutter doctor -v output
Your pubspec.lock & pubspec.yaml
Your Podfile & Podfile.lock
The full verbose output of the command you run i.e, (if you run flutter run  provide the output of flutter run -v in a .txt or .log file since the output will be huge. If you run flutter build ios please provide the output of flutter build ios -v)

Thank you

I'm facing the same issue:
flutter doctor -v:
[✓] Flutter (Channel stable, 3.3.10, on macOS 13.1 22C65 darwin-arm, locale en-US)
• Flutter version 3.3.10 on channel stable at /Users/lukeirvin/Development/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 135454af32 (3 weeks ago), 2022-12-15 07:36:55 -0800
• Engine revision 3316dd8728
• Dart version 2.18.6
• DevTools version 2.15.0
[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
• Android SDK at /Users/lukeirvin/Library/Android/sdk
• Platform android-33, build-tools 32.1.0-rc1
• Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 14C18
• CocoaPods version 1.11.3
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2021.3)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
[✓] VS Code (version 1.74.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.56.0
[✓] Connected device (3 available)
• iPhone 14 Pro Max (mobile) • 9264C232-AB9C-49BD-9CBE-A80FA501866B • ios            •
com.apple.CoreSimulator.SimRuntime.iOS-16-2 (simulator)
• macOS (desktop)            • macos                                • darwin-arm64   • macOS 13.1 22C65 darwin-arm
• Chrome (web)               • chrome                               • web-javascript • Google Chrome 108.0.5359.124
[✓] HTTP Host Availability
• All required HTTP hosts are available
• No issues found!

Hi @lukeirvin, your issue may be caused by something else, so kindly file a new issue and provide the information requested for in https://github.com/flutter/flutter/issues/117849#issuecomment-1368779013
Thank you


Add partial and optional support for OCIO to MaterialX

The default color management system in MaterialX has support for a very limited set of color spaces. Some commonly used spaces such as linear Rec.2020 or sRGB-encoded AP1 are not available. MaterialX is often used in applications that already use OCIO for color management and so artists are sometimes confused as to why the color spaces in a menu that is available for textures in another part of the application are not available for textures in MaterialX. In some cases, a color space may even be mathematically the same as one in the MaterialX set, but it is simply named differently. This may lead to the artist picking the wrong color space and wasting time. In situations like this, the application is able to offer a better user experience by offloading the interpretation of color space names to OCIO. This allows artists to use the same color spaces with all textures rather than having a different set for MaterialX.
However, not all OCIO color spaces are appropriate for use in a MaterialX document and this PR does not offer support for full OCIO. Supported color spaces include any of those that are mathematically compatible with one of the default MaterialX spaces.
This PR does not change where the implicit color conversions happen in a MaterialX shader, and it does not provide any additional functionality to creators of MaterialX documents, other than expanding the set of allowed color space names. The MaterialX API already allows a client to override the default mx::ColorManagementSystem, this PR illustrates how that may be done in order to take advantage of a subset of OCIO.
It should be noted that only minimal changes would be needed to adapt this PR for use with either OCIO or the nanoColor library currently under development.
This integrates the OCIO shadergen into the MaterialX shadergen to provide on-the-fly color management for OpenGL, Vulkan, Metal, and OSL.
Targeting 1.39.1. Definitely not 1.39.0.
Tested with GLSL and OSL on Linux.

Well, OSL was not as trivial as expected, but still manageable. Results are good:

And we can see the OCIO code being injected in GLSL:
vec4 ocio_color_conversion_8182d4da8df3d64c153c5e4252dbbf(vec4 inPixel)
{
  vec4 outColor = inPixel;

  // Add Gamma 'monCurveFwd' processing

  {
    vec4 breakPnt = vec4(0.0392857157, 0.0392857157, 0.0392857157, 1.);
    vec4 slope = vec4(0.077380158, 0.077380158, 0.077380158, 1.);
    vec4 scale = vec4(0.947867274, 0.947867274, 0.947867274, 0.999998987);
    vec4 offset = vec4(0.0521326996, 0.0521326996, 0.0521326996, 9.99998974e-07);
    vec4 gamma = vec4(2.4000001, 2.4000001, 2.4000001, 1.00000095);
    vec4 isAboveBreak = vec4(greaterThan( outColor, breakPnt));
    vec4 linSeg = outColor * slope;
    vec4 powSeg = pow( max( vec4(0., 0., 0., 0.), scale * outColor + offset), gamma);
    vec4 res = isAboveBreak * powSeg + ( vec4(1., 1., 1., 1.) - isAboveBreak ) * linSeg;
    outColor.rgb = vec3(res.x, res.y, res.z);
    outColor.a = res.w;
  }

  return outColor;
}

and in the OSL code:
color4 ocio_color_conversion_8182d4da8df3d64c153c5e4252dbbf(color4 inPixel)
{
  color4 outColor = inPixel;

  // Add Gamma 'monCurveFwd' processing

  {
    vector4 breakPnt = vector4(0.0392857157, 0.0392857157, 0.0392857157, 1.);
    vector4 slope = vector4(0.077380158, 0.077380158, 0.077380158, 1.);
    vector4 scale = vector4(0.947867274, 0.947867274, 0.947867274, 0.999998987);
    vector4 offset = vector4(0.0521326996, 0.0521326996, 0.0521326996, 9.99998974e-07);
    vector4 gamma = vector4(2.4000001, 2.4000001, 2.4000001, 1.00000095);
    vector4 isAboveBreak = vector4((outColor.rgb.r > breakPnt.x) ? 1.0 : 0.0, (outColor.rgb.g > breakPnt.y) ? 1.0 : 0.0, (outColor.rgb.b > breakPnt.z) ? 1.0 : 0.0, (outColor.a > breakPnt.w) ? 1.0 : 0.0);
    vector4 linSeg = outColor * slope;
    vector4 powSeg = pow( max( vector4(0., 0., 0., 0.), scale * outColor + offset), gamma);
    vector4 res = isAboveBreak * powSeg + ( vector4(1., 1., 1., 1.) - isAboveBreak ) * linSeg;
    outColor.rgb = vector(res.x, res.y, res.z);
    outColor.a = res.w;
  }

  return outColor;
}

@jstone-lucasfilm, I wanted to ask about this during our meeting today but we ran out of time. Now that Jerry has implemented Bernard's request, is it possible to move forward with this PR?  (Although this work is compatible with NanoColor, it meets a separate need, and thus seems like it could proceed without waiting for the conclusion of that initiative.)

@doug-walker Agreed that this pull request is still valuable, independent of the ultimate design of NanoColor, and I'm currently thinking that we should include it in our next MaterialX release, v1.39.2.  Right now we're in the release candidate phase for v1.39.1, and this change deserves more time than we'd have to include it in that release.


Pubsub messages do not ack

Messages are never ack'd, which causes them to be replayed on every startup.

@robertjchristian  Thanks for reporting! This should be fixed in #144, so I'm closing this for now - but please reopen this issue if that didn't solve the problem.


TOC Content Div is set to 75% and doesn't allow content to span appropriate columns

Detailed description

Describe in detail the issue you're having.
There is extra space (4 columns) on the right side of our page template. The TOC content div has max-width and flex-basis set to 75%. Should this be fixed since shadow DOM styling can't be overridden?

Is this a feature request (new component, new icon), a bug, or a general
issue?
Bug

Is this issue related to a specific component?
TOC Content DIV

What did you expect to happen? What happened instead? What would you like to
see changed?
I expected our page content to not have an extra 4 columns of space on the right side.

What browser are you working in?
Chrome

What version of Carbon for IBM.com are you using?
On AEM v.17.0 and on storybook v1.18.0

What offering/product do you work on? Any pressing ship or release dates we
should be aware of?
Afterburner / AEM

Additional information
– Jira ticket on Afterburner CMS: https://jsw.ibm.com/browse/ADCMS-1068
– Please reach out to Raveena Rastogi for further details.

To be investigated as part of Fireline.


How to debug directly in Unity?

What do you mean? Testing the TAP device when connected to a computer , using the unity player?

Yes
shahartap<EMAIL_ADDRESS>schrieb am So., 15. Apr. 2018, 09:02:

What do you mean? Testing the TAP device when connected to a computer ,
using the unity player?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TapWithUs/tap-unity-plugin/issues/1#issuecomment-381385051,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHunQx43r-CM0jz4Pv2mDoSDUzShFr0ks5tovCEgaJpZM4TTABu
.

It is in the development pipeline. Until it'll be developed, You can simulate the input using key commands.
Each tap combination is mapped into a letter that you can use.
For example, tapping only the thumb, can be simulated by reading key code A from Input class in Unity. For a list of combination-to-letter mapping you can use the Glossary section in the app TapGenius for iOS. or I can mail it to you if you wish :)
Thanks for taking part in developing with Tap! :)
Please leave this issue open , We'll update you when a the plugin is updated to work with the Unity Editor.

@alexanderzeillinger You can use the Wi-fi Debug option of Unity if you build for Android.
It display Debug Log through network via ADB.
@shahartap or @yanivgal  Are you available? Your demo scene is a bit lean. Would like to improve your project by Pull Request. In the coming days. (Just need to make it works before that ^^. I have some trouble to make your code run on my Androids.)

@JamsCenter  you're most welcome to make a pull request. What improvements are you planning on doing?

(JamsCenter)
I was planning to push some prefabs and demo scenes.
But never find the time to finish the code in a clean way.
Plus the blocking bugs of the code on this git make me look at other solutions.
https://github.com/TapWithUs/tap-unity-plugin/issues/4


Feature/header columns (#6)

Update data-type.ts

Update sample.ts

Update sample.ts

Update build.spec.ts

Update data-type.ts

Codecov Report

Merging #7 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master     #7   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           5      5
  Lines         152    150    -2
  Branches       13     12    -1
=====================================
- Hits          152    150    -2

Impacted Files
Coverage Δ

src/build.spec.ts
100% <ø> (ø)
:arrow_up:

src/data-type.ts
100% <100%> (ø)
:arrow_up:

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 40d74b0...d0f30c6. Read the comment docs.


message_edit: Close message edit UI after message is moved.

Currently, if we navigate to some other topic/stream
while the message is being moved, the message edit UI
still remains open as we do not get its row in
message_lists.current since the message has not moved yet
to the stream/topic we navigated.
Hence the correct thing to do would be to delete
the message_id from currently_editing_messages if it
exists there but we cannot find the row.
Fixes #21724.

Fixes: #21724

Screenshots and screen captures:
Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

@punchagan , @amanagr  Can you please take a look at this PR.

Can you fix the CI failure?

Ahh, sorry for missing on this, Fixed the CI !
Thanks!

Looks good to me, tagged to merge once CI passes.
Please prioritize the follow-up PR to check who did the edit -- I think it's potentially a more important bug than this one, since the consequence of that bug is potentially losing work.

Looks good to me, tagged to merge once CI passes.
Please prioritize the follow-up PR to check who did the edit -- I think it's potentially a more important bug than this one, since the consequence of that bug is potentially losing work.

Just to check, has this been done? If so, it would be great to link the PR here for the record.


AMD cards are not initialized only for Dx11 Windows

Hello, im using your project on Windows. I copied DLL files to game executable location. In Dx9 mode it just works, but for Dx11 it could not initalize graphics device and error log says: err:   D3D11CreateDevice: Failed to create swap chain.
Software information

League of legends
Is this information matter idk:
1920x1080, High settings, vsync off, fullscreen
Prefer Dx9 legacy mode is unchecked (when its checked dxvk succeeds to convert dx9 to vulkan)

System information

GPU: RX570 4GB
Driver: Radeon Software 22.1.1
Wine version: not wine Windows 10 64bit but game is 32 bit so 32bit DLL files are used.
DXVK version:

Log files

d3d9.log: 0 KB
d3d11.log: League of Legends_d3d11.log
dxgi.log: Non exists

You forgot to copy dxgi.dll.

Either that or the game refuses to load dxgi.dll from the game directory (some do this by accident, some do this as a kind of terrible anti-cheat measure), in which case there's really nothing you can do.

You forgot to copy dxgi.dll.

no, i did copy.

Either that or the game refuses to load dxgi.dll from the game directory (some do this by accident, some do this as a kind of terrible anti-cheat measure), in which case there's really nothing you can do.

Okay thanks i will try to find a way thats a game specific problem as you described.


⚠ Anthelion has degraded performance

In a251deb, Anthelion ($ATH) experienced degraded performance:

HTTP code: 200
Response time: 1889 ms

Resolved: Anthelion performance has improved in 03d633f after 412 days, 5 hours, 8 minutes.


Improve XCTAssertEqualSequences to log difference

Implements improvement suggested @timvermeulen in #100
Improves the XCTAssertEqualSequences function to log when two elements in a sequence are not equivalent or when one of the sequences is shorter than the other.
I implemented it such that as soon as it finds to unequal elements between the sequences, it'll fail and stop without looking at the remaining elements. We could try to make it more sophisticated if that's the goal.
I'm not sure how to test it, since it's a test helper utility. As it stands, XCTAssertEqualSequences was not being tested. The test suite, which heavily depends on the function still passes after the changes but that doesn't exercise the function in the failure cases or its output. I experimented with that by changing some tests to make them fail and it seemed to work fine.
The code was based on the implementation of elementsEqual from the Swift standard library.
Checklist

[ ] I've added at least one test that validates that my change is working, if appropriate
[X] I've followed the code style of the rest of the project
[X] I've read the Contribution Guidelines
[X] I've updated the documentation if necessary

New commit fixes bug and addresses feedback

@swift-ci please test


Remove extra separator when TextEdit is read only and unselectable

This PR makes the separator only appear when the following two blocks of items will be added.
Also changed calling is_selecting_enabled() to using selecting_enabled directly. This is the only place that uses the getter inside the cpp.

Before
After

It's basically the same on 3.x and the fix is the same.

Thanks!

Cherry-picked for 3.5.


[WIP][QNN] Quantized fully connected

This PR is the implementation of pre-quantized fully connected op.

There is dependency on #3779. Once it is merged the test cases will pass. will wait for it,

Another PR has been merged with these changes.


Set build-init test JVM Xmx to 1g

See https://github.com/gradle/gradle-private/issues/4010
The OOM only happens in InProcessGradleExecuter (:build-init:iembeddedIntegTest and :build-init:integForceRealizeTest), which makes me wonder if that's just insufficient memory for build-init integration tests. Let's start with increasing the memory and see if the problem gets solved/mitigated.

Your PR is queued. See the queue page for details.

I've triggered a build for you. Click here to see all failures if there's any.

I have cherry-picked to PR #27116.


Interval support in select_loop

Hi, It'll be helpful if select_loop has support to periodically do some work.
ticker := time.NewTicker(5 * time.Second)
quit := make(chan struct{})
go func() {
    for {
       select {
        case <- ticker.C:
            // do stuff
        case <- quit:
            ticker.Stop()
            return
        }
    }
 }()

I know I can do this with a separate thread and push interval events in the channel but inbuilt Interval will be much more ergonomic

It should be easy to do the same manually:
let mut tick = Instant::now() + Duration::from_secs(5);
loop {
    let to = tick - Instant::now();
    select_loop! {
        timeout(to) => {
            tick += Duration::from_secs(5);
            // do stuff
        }
        recv(quit, _) => {
            break;
        }
    }
}

If you don't find this solution satisfactory, could you suggest another by converting the Go example into your preferred Rust syntax?

Now we have channel::tick in version 0.2.0. :)
Here's translation of your Go code into Rust:
let ticker = channel::tick(Duration::from_secs(5));
let (quit_s, quit_r) = channel::bounded(0);

thread::spawn(move || {
    loop {
        select! {
            recv(ticker) => {} // do stuff
            recv(quit_r) => return,
        }
    }
});


Add hooks for deleting providers, mappings and plans, simplify abstraction around kube-client, clean up types

Resolves #170

Replaces the useAuthorizedK8sFetch with a more general useAuthorizedK8sRequest which takes a requestFn that can be anything returning a promise, and handles relevant errors in one place.
Uses this helper to create useAuthorizedK8sClient which is a drop-in replacement for useClientInstance that will use the authorized hook in each method to handle SSL errors and token expiry.

This client wrapper uses actual TS generics and introduces the IKubeResponse<T> type, which allows code handling the results of a client method call to be strongly typed. This involved some refactoring.
These new types are good candidates to be moved into the actual client itself, which would make some of the explicit type annotations I added unnecessary. https://github.com/konveyor/lib-ui/issues/33

Uses this new client abstraction for all current k8s mutations and queries we have.
Adds a mechanism to the PollingContext for temporarily speeding up the polling interval for a short time, which is useful to call right after a mutation which has a delayed response (creating providers and plans), so you don't have to wait the full 5 seconds to see the results.

I didn't put this in useMockableMutation because most of the time it is not necessary, so it's just an optional feature you can use in a particular mutation hook if it will help.
This should only be used on create and update mutations -- on a delete mutation, you should instead proactively remove the object from the queryCache like I do with providers, and if you do that, increasing the polling speed will make the object reappear before it is truly removed.

Adds useDeleteProviderMutation which deletes both the provider and the associated secret.

Because this deletes a CR that will not immediately delete its associated objects in the inventory API, its onSuccess callback does not invalidate the useProvidersQuery and make it refetch. Instead, it filters out the matching object from the queryCache, so it appears to be deleted immediately. As long as it is really deleted in the inventory before the next refetch, it is a good experience. If for some reason a delete took longer than 5 seconds, technically this would result in the provider disappearing, reappearing, then disappearing again, but that seems unlikely.

Adds useDeleteMappingMutation and useDeletePlanMutation.
Wires up these mutations to the Remove actions in the kebab menus of the respective tables.
Also fixes a bug caused by #164 where the "new mapping name" field (hidden until you check the box to save mappings for later in the plan wizard) was essentially required no matter what because the matches() regex validation was considering an empty string invalid.

Next up: edit.

@mturley So this doesn't actually do anything yet, right? It just adds the hooks but doesn't actually do the deletes? 'Cause I don't see anything happen when when I try it in demo/preview mode. Also, these will all have confirmation dialogs, right?

@mturley Also, a note on terminology... if the action is Add (providers), the corresponding action should be Remove. If the action is Create (mappings and plans), the corresponding action should be Delete. We can take care of this when we do our detailed UI walkthrough next week.

@vconzola it does actually work when you're not in preview mode, but I neglected to add confirm dialogs. I'll do that. Actually that gives me a convenient place for the message that the action is not available in preview mode, it didn't have anywhere to go otherwise.

@vconzola ah ok. I'll fix the terminology for Remove/Delete in this PR while I'm in that code since I also need to add the confirmation.


Generalize mass shift in theoretical fragmentation models

Generalize deuterium to parameterizable isotopicMassShift, (needed to be done for 15N data).

This work will be completed on issue-11. #11


New Icon Request: Password Refresh

Icon name
Password Refresh
Use cases
I need an icon for a user management system to use for refresh password functionality
Design ideas
something like this:

but that would work with the current lock and refresh icons.
Checklist

[X] I have searched if someone has submitted a similar issue before and there weren't any. (Please make sure to also search closed issues, as this issue might already have been resolved.)
[X] I have searched existing icons to make sure it does not already exist and I didn't find any.
[X] I am not requesting a brand logo and the art is not protected by copyright.
[X] I am not requesting an icon that includes religious, political imagery or hate symbols.
[X] I have provided appropriate use cases for the icon(s) requested.

Hi @pgbradbury,
I've played around with the concept, but I'm not sure any of these is production ready.
Open lucide studio
Open lucide studio
Open lucide studio

Hi @karsa-mistmere I am happy with the first one at the top, but I don't object to any of them. What are your concerns?

None of them are very legible, I think,
The first one, because I had to squish it down so much to fit within refresh's circle.
As for the others, maybe too much is lost of lock's outline for it to remain recognisable.

Hi @karsa-mistmere I am happy with the first one at the top, but I don't object to any of them. What are your concerns?

None of them are very legible, I think,
The first one, because I had to squish it down so much to fit within refresh's circle.
As for the others, maybe too much is lost of lock's outline for it to remain recognisable.

I like the 2dn variant, I do agree with the legibility issue tho.

@karsa-mistmere ok fair enough, I can't figure out how to see it at the same sizes as the other ser icons without writing some code to render the SVG next to an icon so difficult for me to judge. It could maybe work using a key instead of a lock?

Open lucide studio
Open lucide studio

Perhaps this solution will help you figure out the problem click

@karsa-mistmere Is there anything else that I need to do to move this icon into the set?


Update Website and UI

This is a mockup for what the spash-page is supposed to look like as of July 15th- other UI updates to come SOON as well...

added to lamda and s3!


Clearly define what an Experience is

We need a clear and unambiguous definition of Experience that links to other, defined concepts like channel, location, event, content, etc.

@cdegroot-adobe

and that is NOT specific to XC - but encompasses DC and CC as well...

Are we waiting for @cdegroot-adobe to start off the conversation here? Is this related to the existing experienceevent schema?

@trieloff @lrosenthol  @cmathis  @kstreeter
To be clear "experience" in this context is some sort of content or coordinated group of content (page, image, video, voice etc) that is managed and is sent to an enduser at a specific point of time based on a set of decisions made by one or more engagement applications. A marketer will have a name and maybe anid to identify the content in targeting rules and reports.
Examples:
A user lands on the web home page of a brand. The Adobe Analytics all important pageName="homepage". In a channel agnostic world the experience.name="Homepage".
A user views the product details of a product in a mobile catalog app. In the Mobile Analytics world this is a state="product details:{sku}".   In a channel agnostic world the experience.name="Product Details".
A user selects to listen to a store opening hours in an IVR over the phone. In the IVR state change this is a state="storeOpeningHours".   In a channel agnostic world the experience.name="Store Opening Hours".
It is important to have these identification details in a channel neutral location as it is hard to ferret out the channel specific paths when we are doing omni channel queries.

@cmathis yes, this came up in the experience event discussion: https://github.com/adobe/xdm/pull/143#issuecomment-371454272

Here is how I see it:

we have places where content can go. These are what we call channel-locations (#170). You can think of them as rectangles on a screen or in a time line (e.g. in video or audio channels)
the stuff that is going into these places, or is contained by them, is content
a large part of the discipline of marketing is to decide which content goes into what channel-location
you can make that assignment based on the consumer's audience (an audience is simply a set of consumers, however defined or identified) – this process is called targeting
assigning content to channel-locations across a range of channel-locations that span multiple impressions, interactions, channels, or even days, so that a consistent whole is created is the goal of many marketers, designers, and authors.
creating this consistent whole of content in channel-locations is what constitutes experience design or experience management

This gets us quite close to the definition of experience, but we need to consider some additional points.

consistency of the experience is in the eye of the consumer, so experiences need to be specific to an audience (this audience can be as broad as "everybody" or as narrow as "only the user with the ID 007")
an experience is always concrete, so in the case of an A/B/N test, where multiple candidate variants of content are competing for the placement in a specific channel-location, every experience will have only one piece of content in each channel-location
if there are multiple A/B/N tests, in multiple channel-locations, this will increase the potential number of possible experiences

In summary:

an experience

is assigned to an audience
contains one or more assignments

an assignment

relates to one channel-location (1:n)
relates to one piece of content (1:n)
relates to one set of eligibility rules (time, frequency, etc) (1:n)

The problem with those definitions are that they only apply to DMa...But consider that we have an application on the DMe called XD (eXperience Designer) that has nothing to do with anything you've written here about channels or locations or ...
Especially because we have chosen to name XDM as the "Experience Data Model" (vs. having the Extensible Data Model", which is a better fit with XMP!), we need a definition of experience that applies across Adobe.
So let's try again with a wider definition please.

@lrosenthol - Is eXperience Designer not building experiences that will be shown to actual people (an audience)?
Yes we are looking at the mechanics of consumption here, but it is not unrelated. Creators create experiences in CC, in DC and DMa we have a mix of creation and distribution and consumption analysis.
Who from CC should we bring into this discussion to fill out the panel?

@cdegroot-adobe Yes, in the wider scope, XD certainly builds "experiences" for customers.  However, those experiences have no channels or channel locations, no assignments, etc.   All those terms and concepts have no place in that world.
My issue is in our definition of what XDM stands for.  If we change that from eXperience->eXtensible, then the problem goes away.  If we insist on having it be eXperience - then the definition of eXperience needs to be VERY broad...

@lrosenthol

Yes, in the wider scope, XD certainly builds "experiences" for customers. However, those experiences have no channels or channel locations, no assignments, etc. All those terms and concepts have no place in that world.

I beg to differ. The terms might not be clearly defined, but the concepts are definitely there:

Create a low-fidelity design in Adobe XD to define the overall structure of your app or website.

"app or website" refers to the channel that your Adobe XD prototype is targeting. And like a channel in marketing, it comes with constraints like screen size, interaction mode, etc.

Define the user flow of your app or website using the prototyping tools available within Adobe XD.

"the user flow" are the various screens or stages or artboards that the prototype contains. In the terms used above, they are the channel locations (as the most general term) with example content.

That may be true, @trieloff - but as you yourself point out, the terms are completely different.  We CANNOT use XC-only terms here.  It MUST be a set of terms that apply equally across all three Adobe clouds - the Adobe Cloud Platform.

I’d like to get someone from Adobe XD to define what an experience is. I expect that the definition will be about as wide and all-encompassing as the definition we would get out of AEM, so the question is if we can find a definition that all parties can agree upon and that is useful. If not, we will end up with an experience data model that models everything about experiences, but (because of the lack of a reliable definition) does not model experiences themselves.

Which is EXACTLY why I wish we could change the X in XDM away from experience!

I think the word "Experience" is overloaded, we probably need to qualify it for our different use cases:

When assembling content for a website or mobile page we are creating a dynamic data-driven user experience.
When someone uses XD they are designing a user interactions experience, or a family of those that will be materialized using data-driven assembly as defined above.

These are the Experience Cloud and Creative Cloud variants of "Experience" that I see. If there are Document Cloud equivalent use cases they might lead to other qualifiers.
My point is that just "Experience" is too broad and vague to address these different things.


Some geo functions

This is partly to learn how all this geo stuff works, partly as I think this is useful functionality, and also to figure out a bit how the data pipelines would look like.
This actually only checks if the point is in the sea or not, it check against coastlines extracted from OSM, but does not check internal waters. So e.g. the Great Lakes all report as land.

By the way, the actual DB check here is consistently sub-1 ms
# EXPLAIN ANALYZE SELECT EXISTS (SELECT FROM land_polygons WHERE ST_Contains(geom, ST_SetSRID(ST_MakePoint(-74.0115, 40.7077), 4326)));
                                                                QUERY PLAN
------------------------------------------------------------------------------------------------------------------------------------------
 Result  (cost=20522.51..20522.52 rows=1 width=1) (actual time=0.302..0.303 rows=1 loops=1)
   InitPlan 1 (returns $0)
     ->  Bitmap Heap Scan on land_polygons  (cost=25.66..20522.51 rows=1 width=0) (actual time=0.301..0.301 rows=1 loops=1)
           Filter: st_contains(geom, '0101000020E6100000DBF97E6ABC8052C08AB0E1E9955A4440'::geometry)
           Heap Blocks: exact=1
           ->  Bitmap Index Scan on idx_land_polygons_geom  (cost=0.00..25.66 rows=716 width=0) (actual time=0.268..0.268 rows=2 loops=1)
                 Index Cond: (geom ~ '0101000020E6100000DBF97E6ABC8052C08AB0E1E9955A4440'::geometry)
 Planning Time: 0.194 ms
 Execution Time: 0.333 ms
(9 rows)

I've added a GetRegion RPC as well

This PR need not be merge until there is a use for this functionality, and until I've cleaned up the data processing a little bit.
A use should come as the community APIs are developed.

I've added layers to display regions a user has visited, lived in, surfed, and hosted people from. It's currently very ugly, it looks something like this:

I am hoping to fix up these pipelines e.g. so that they can be run in one hit on AWS on a clean Ubuntu 20.04 image once a month and the artifacts can be dropped in a publicly available S3 bucket. Or actually, a docker image would probably be better.

This also looks nice. https://github.com/markmarkoh/datamaps

This also looks nice. https://github.com/markmarkoh/datamaps


Display likes

In this pull request, the following was done:

[ ] Add like to API on heart icon click
[ ] Display the likes when the page loads
[ ] Increment the like on clicking a heart icon in the movie card

Hi @FilbertKimasa ,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights
✅ Homepage displays movies fetched from the API
✅ Increment the like on clicking a heart icon in the movie card
✅ Display the likes when the page loads
✅ Add like to API on heart icon click
✅ Descriptive PR summary
✅ Clean code
✅ Use ES6 syntax and ES6 modules.
Cheers, and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.


new: give useful error if bad path input

https://discord.com/channels/1186394868336038080/1186684706700410962/1194770635856035950

https://github.com/uqbar-dao/necdev/pull/9/commits/7cced53db5d6a0df16f361fd8ea7bff7d9d6bd70


Brave Browser can't be updated if installed via winutil.

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Install Brave via winutil.
Wait until new brave version becomes available.
Brave tells you it can't update.
You have to uninstall brave and install using their installer instead.
more information is here

Expected behavior
Brave should be able to update itself after installation via winutil (ideally).  Or support for it should be removed perhaps.
If this isn't caused by the way winutil installs brave then my apologies for wasting your time.  Please forgive me.

This is an issue that is with Winget. They should fix this in the next update, but there is nothing I can do to remedy this.


'Text' typed indicators do not work in genLinks arrays

When I try adding a 'Text' case section to genLinks, they are not being built.

Removed case that prevented link tabs from generating on Text input.


fix(debian): don't create advisories not contained in cve-xxx/dsa-xxx/dla-xxx

Description
If i understand correctly - Debian doesn't add old(EOL) releases to CVES.
e.g. CVE-2022-3358 doens't contain debian 8.
We are currently adding similar CVEs to the DB without a fixed version.
.
Fixed: Only found versions were used.
Related issues:

Close aquasecurity/trivy/issues/3194
Close aquasecurity/trivy/issues/3147

any update on when can this PR be merged?
@DmitriyLewen

Hello @nikithaguduru
I asked @knqyf263 to check it out.
After his review we will merge this PR.

It is not planned for the moment.
https://github.com/aquasecurity/trivy/issues/3194#issuecomment-1347892057


Dark/Light mode switching

Description of the new feature
Implement Dark and Light Mode detection protocol adopted by few terminals and apps (neovim being the most well known i guess)
the specification is presented here: https://github.com/contour-terminal/contour/blob/master/docs/vt-extensions/color-palette-update-notifications.md
Proposed technical implementation details
No response

This might also be a potential solution for #10639 (not so much the dark/light mode detection, but the ability to request palette update notifications).


copied files from 2017-09-01 and created a new folder 2017-10-01

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Initial commit for https://github.com/Azure/azure-rest-api-specs/pull/1781

No modification for AutorestCI/azure-sdk-for-python

No modification for AutorestCI/azure-sdk-for-node

No modification for AutorestCI/azure-sdk-for-ruby


Setting false to field showCounts not effective for multiple views in manifest json

OpenUI5 version: 1.125.1
Browser/version (+device/version): Windows 11 / Chrome 126.0.6478.127
Any other tested browsers/devices(OK/FAIL): No
URL (minimal example if possible): https://ui5.sap.com/test-resources/sap/fe/core/fpmExplorer/index.html#/controllerExtensions/multiModeExtensibility
Steps to reproduce the problem:

Set showCounts to false

What is the expected result?
We expect the table count to be hidden, while the backend odata request with $count will not be sent.

What happens instead?
The count is still there, and the the backend odata request with $count is sent.
Any other information? (attach screenshot if possible)
Actually this is an issue found in our app, and we checked ui5 example, found the same issue. So we just provide this as example

Hello @Rosaugo,
Thank you for sharing this finding.
I've created an internal incident DINC0218314.
The status of the issue will be updated here in GitHub.
Regards,
Konstantin Gogov
SAPUI5 GitHub Dispatcher


[BUG] Server Error(500)

When using an ipv6 address to access the wms system, a Server Error 500 error will occur. Looking at the cause of the system error, the troubleshooting is that the ip field in the throttle table is too short, resulting in records that cannot be inserted. The length of the ip field needs to be increased.
https://github.com/GreaterWMS/GreaterWMS/blob/d0dc20ecf587f5575a86341f2cc81c4517b785af/throttle/models.py#L3-L15

After manually adjusting the IP field length to 255, the system runs normally.

ok , tks , we will fix it


Calculate net promoter score and identify detractors and promoters

Problem statement
Given survey results, ReportStream needs an NPS score and identify organizations who are detractors and promoters.
What you need to know
Net promoter score / likelihood of referral
What is it?
Net promoter score is a customer satisfaction and loyalty metric derived from asking “how likely are you to recommend x product to others”
0 to 6 is detractor. 7 and 8 is passive. People use product but that’s it. 9 to 10 is a promoter. People who are likely to recommend product to others.
Score is calculated using percentage of promoters minus percentage of detractors.
For software as a service companies, average net promoter score in 2021 is 30, according to one study. https://www.retently.com/blog/good-net-promoter-score/
Acceptance criteria

[x] Calculate the Net Promoter Score
[x] Identify the organizations that are promoters and detractors
[x] Create a tracking MS list, share with team

To do

[ ] ...

This should be automatically done on survey monkey.

36	Reponses
32	Provided NPS
2	Detractors (0 - 6)
11	Passive (7 - 8)
18	Promoter (9 - 10)
See here  for list of organizations that are in each group:
CC: @rachelhanster @drew-usds @a-chappelear
Now need to get POC for the respective organizations which can be retrieved from the Tracking Sheet

50 - NPS for  public health authority that uses ReportStream to get data (e.g. state, local, tribal, or territorial public health
department) (n = 18 respondents out of 20)
60 PNS organization that reports test results through ReportStream (e.g. if you submit data to ReportStream via API
connection or CSV upload) (n = 10)

In https://cdc.sharepoint.com/:x:/r/teams/USDSatCDC/_layouts/15/Doc.aspx?sourcedoc={2DF9F707-5FC0-4B2B-B7B4-51D5C3D1EFD3}&file=CX Survey Data_May 2022.xlsx&action=default&mobileredirect=true in tab called outreach list of people for outreach. Decide how to enter into MS lists.

Attn: @BerniXiongA6

@victorialb-usds Of the individuals who responded, did these respondents all provide a net promotor score?

How many organizations did not respond?

I think this can be closed because @victorialb-usds completed the AC for this ticket which has informed the subsequent tickets in this Epic that @a-chappelear and I are currently working on.

FYI- @RakshindaAslam

@victorialb-usds @rachelhanster - The summarized view of quantitative and qualitative results for sharing with the RS team is here: https://cdc-my.sharepoint.com/personal/trk5_cdc_gov/_layouts/15/doc.aspx?sourcedoc={f14e451f-7a14-4f8c-80fd-4f63795a62e9}&action=edit.
Please update as necessary.

Pulling it back into the sprint backlog to synthesize qualitative results.

Synthesized qualitative and quantitative survey results by Sender and Receiver - https://cdc-my.sharepoint.com/:p:/r/personal/trk5_cdc_gov/_layouts/15/doc.aspx?sourcedoc={f14e451f-7a14-4f8c-80fd-4f63795a62e9}&action=edit


iothread support for virtio-scsi

With qemu iothreads, it is possible to have a separate thread to handle IO. This results in lower IO latencies and better throughput.

Depends on https://github.com/intel/govmm/pull/14

@sboeuf @chavafg The CI failed due to some network issue. Can you restart the build, as it looks like I cannot do it.

@amshinde done

@sboeuf @egernst @jodh-intel  @laijs @bergwolf ptal

sweet! lgtm

lgtm.
/cc @markdryan.
Also, we should run the metrics CI on it to ensure we see the expected behaviour (/cc @grahamwhaley).


Adding a test environment and updates to the welcome and endofbatch crons

Why are we making these changes?
Right now Pairing Bot has a few actions that require manual work by its maintainers. This PR seeks to automate that work and also adds a dev environment.
What changes are being made in this PR?

Adding a development branch and environment in Google App Engine to test out new Pairing Bot features without breaking the Pairing Bot for subscribed recursers.

Modifying the endofbatch cron job to only unsubscribe people who have recently left RC, versus unsubscribing all users at the end of a batch.

Adding a welcome cron job that posts a message to 397 Bridge in the 2nd week of a batch to welcome people to Pairing Bot's features.

Questions for the Reviewer

Any stylistic changes to make here?

Is the logic for checking when to unsubscribe people sound? One possible edge case would be if the weekly cron job failed silently and we didn't catch when people left RC.

I'm going to merge in these new cron jobs but have them only log what they would have done, instead of actually performing the action. I don't want to accidentally subscribe folks. If the cron jobs seem to be doing ok after a few runs, then I'll remove the log statements and have the jobs perform their actual tasks 😄


Guide to use the features of Cordova

Tried by downloading code to find device information. couldn't works folks guide me to utilize.I don't have ionic framework on my machine. My plan is to get device information with angular+ cordova. Please give solution.
Thanks in advance
Mrudula

I think this is are you looking for https://github.com/driftyco/ng-cordova/blob/master/src/plugins/device.js
Also don't forget to make calls to cordova after 'deviceready' event

Problem resolved able to achieve to find device information from cordova, but device name and uuid not coming.
Device Name: undefined
Device UUID: null
Please help me out

Device Name - device.name is deprecated as of version 2.3.0. Use device.model instead.
Device UUID - this info may help https://github.com/apache/cordova-plugin-device#deviceuuid


APP_NAME is not respected

Describe the bug
When changing app name it won't respect the name change and launch the new binary.
There's two issues:

The scripts will try and launch "My Application" despite clearly building template application:

On some projects, it won't even change the binary name compiled:

Affected platforms
Select one of the platforms below:

iOS

Versions

Kotlin version*: 1.8.22
Compose Multiplatform version*: 1.4.1
OS version(s)* (required for Desktop and iOS issues):  OSx: 13.4
OS architecture (x86 or arm64): M2 arm64
JDK (for desktop issues): Zulu JDK19

To Reproduce

Change APP_NAME
run ./cleanup.sh
./gradlew --stop
Sync project
Run ios target

App will either not run (as it try's to launch "My application", that doesn't exist), or it builds an old name "My Application" instead of "HayNet" new binary (so you see no changes...)
Expected behavior
Should respect the new name and build and run that target
Screenshots
See above
Additional context
Tried downgrading kotlin/gradle etc
Running from xcode will build and run the project correctly with the correct name

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Issue with express-handlebars (Error: ENOENT: no such file or directory, open '..\views\layouts\main.handlebars')

I get this error:
Error: ENOENT: no such file or directory, open '..\views\layouts\main.handlebars'
Whenever I add this code:
const express = require('express');
const handlebars = require('express-handlebars');
const path = require('path');

const app = express();
const hbs = handlebars.create();

app.engine('hbs', hbs.engine);
app.set('views', path.join(__dirname, 'views'));
app.set('view engine', 'hbs');

app.get('/', (req, res) => {
  res.render('main', {
    title: 'Hey There, World!',
    message: 'This is a fantastic example of Handlebars!'
  });
});

app.listen(3037);

This is from chapter 4.

In your project folder, you should (for now) have a javascript file and a 'views' folder.
Inside the views folder, create a file called 'main.hbs'. Also, create a directory called 'layouts'.
inside the 'layouts' directory, create a file called 'steam-layout.hbs' for example.
leave the 'main.hbs' file empty.
the 'steam-layout.hbs' file should contain:
`

Page: {{title}}
    Message: {{message}}

`
Notice that we need to include the variables {{title}} and {{message}}
now change your js-code to:
`const express = require('express');
const handlebars = require('express-handlebars');
const path = require('path');
const app = express();
app.set("view engine", "hbs");
app.engine(
"hbs",
handlebars({
layoutsDir: __dirname + "/views/layouts",
extname: "hbs",
})
);
app.get("/", (req, res) => {
res.render("main", {
layout: "steam-index",
title: "Hey There, World!",
message: "This is a fantastic example of Handlebars!",
});
});
app.listen(3037);`
I'm also new to handlebars but there are some great guides that explain the concept of views, layouts, and partials in detail.
check out: https://bit.ly/37sqYsu

In your project folder, you should (for now) have a javascript file and a 'views' folder.
Inside the views folder, create a file called 'main.hbs'. Also, create a directory called 'layouts'.
inside the 'layouts' directory, create a file called 'steam-layout.hbs' for example.
leave the 'main.hbs' file empty.
the 'steam-layout.hbs' file should contain:
<!DOCTYPE html>
<html>

<head>
   <title></title>
</head>

<body>
   <h1>Page: {{title}} </h1>
   Message: {{message}}
</body>

</html>

Notice that we need to include the variables {{title}} and {{message}}
now change your js-code to:
const express = require('express');
const handlebars = require('express-handlebars');
const path = require('path');

const app = express();

app.set("view engine", "hbs");
app.engine(
"hbs",
handlebars({
layoutsDir: __dirname + "/views/layouts",
extname: "hbs",
})
);

app.get("/", (req, res) => {
res.render("main", {
layout: "steam-index",
title: "Hey There, World!",
message: "This is a fantastic example of Handlebars!",
});
});

app.listen(3037);

I'm also new to handlebars but there are some great guides that explain the concept of views, layouts, and partials in detail.
check out: https://bit.ly/37sqYsu


nginx-proxy/letsencrypt integration

Is there a way to have the supervisor tag addons and homeassistant with the appropriate nginx-proxy and letsencrypt tags? This would automate the issue of certs. Example environment entries on a manually configured home-assistant docker container.

VIRTUAL_HOST=hass.${DOMAIN_NAME}
VIRTUAL_PORT=80
VIRTUAL_PROTO=http
LETSENCRYPT_HOST=hass.${DOMAIN_NAME}
LETSENCRYPT_EMAIL=${EMAIL_ADDRESS}

Those values make hass.${DOMAIN_NAME} available on port 443 which proxies you to port 80 on the hass container. The ssl certificates are auto generated.
Info links below.
https://github.com/jwilder/nginx-proxy
https://github.com/JrCs/docker-letsencrypt-nginx-proxy-companion

Even if they did, you cannot access the Docker socket, so it would still not be possible to do stuff like this.

The other docker container (nginx-proxy) accesses the docker socket. All I'm asking is the ability to have the supervisor container (which as I understand it manages the homeassistant and addon containers) include these environment variables. The other containers will monitor for those tags and auto generate certs and proxy traffic to them.

Again: None of the add-on access the Docker socket, since that is not possible in Hass.io.
So you request does not make sense. Tagging can be done, sure. But no add-on would be able to access those tags.

Ow, wait! Now I hit submit, it dawns on me. You are manually running these containers next / separate from Hass.io right?
That makes sense.

This actually could be possible as an add-on...
But it has some dependencies, which i've discussed a couple of days ago with pvizeli.
I'll get back to you as soon as possible.

My problem with this container solution is, that it make container useless. If a container have full access to docker api, he have more right as a root user and if that container also in front of the hole box... It's like a brean death of security specialist.
There are a simle solution, they I want integrate if I find time. You dock to hass.io service API and every add-on can provide his data to this service endpoint. A container with support this solutions can attach to this data and provide a proxy without full rights over the hole system.

Are you referring to the nginx container having read-only access to the docker socket? Can you elaborate on the issue that causes? I don't understand what security risk is introduced with read-only access to the socket.

It exists no read-only rights to a socket. If you have no rights to ask docker for data, you never become data they you can read and for ask data, you need write permissions.

Sorry, I can't comprehend what risk you're trying to explain.

 It stay in the dmz or is available from internet. If a container have access to the docker socket, it have the high possible rights on your system. If this container will be hacked, the attacker have more as only full access to your system. In other words, if you use a proxy container like this, you can also map the other container directly to internet and have a better protection.
But if you don't use this kind of proxy to protect your system, yes it dosn't matters.

Which container are you claiming has WRITE access to the docker socket?

https://twitter.com/Ben_Hall/status/706879493135323136

The nginx-proxy container does not HAVE to have access to the docker socket.  Can use docker-gen to remove this requirement.


[Docs] add link to and description of MeshSync in readme

Current State
Desired State

Contributor Guides and Resources

📚 Instructions for contributing to documentation

Meshery documentation site and source

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

I'll take this one😶🌫

I have merged the PR in my master branch to give a view of readme file . I didnt knew by doing so , it will close this issue 😅


International Space Station: Update for binary_sensor.iss fails

Home Assistant release with the issue: 0.100.3

Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.): Hassbian

Integration: ISS
https://www.home-assistant.io/integrations/iss/

Description of problem:
Continuous errors in log file regarding the ISS integration. It seems that max retries exceeded with url.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):
2019-10-26 21:21:04 ERROR (MainThread) [homeassistant.helpers.entity] Update for binary_sensor.iss fails
Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.7/site-packages/urllib3/connection.py", line 157, in _new_conn
    (self._dns_host, self.port), self.timeout, **extra_kw
  File "/srv/homeassistant/lib/python3.7/site-packages/urllib3/util/connection.py", line 61, in create_connection
    for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):
  File "/usr/lib/python3.7/socket.py", line 748, in getaddrinfo
    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno -3] Temporary failure in name resolution

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.7/site-packages/urllib3/connectionpool.py", line 672, in urlopen
    chunked=chunked,
  File "/srv/homeassistant/lib/python3.7/site-packages/urllib3/connectionpool.py", line 387, in _make_request
    conn.request(method, url, **httplib_request_kw)
  File "/usr/lib/python3.7/http/client.py", line 1229, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/usr/lib/python3.7/http/client.py", line 1275, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/usr/lib/python3.7/http/client.py", line 1224, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/lib/python3.7/http/client.py", line 1016, in _send_output
    self.send(msg)
  File "/usr/lib/python3.7/http/client.py", line 956, in send
    self.connect()
  File "/srv/homeassistant/lib/python3.7/site-packages/urllib3/connection.py", line 184, in connect
    conn = self._new_conn()
  File "/srv/homeassistant/lib/python3.7/site-packages/urllib3/connection.py", line 169, in _new_conn
    self, "Failed to establish a new connection: %s" % e
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPConnection object at 0xa29a64f0>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.7/site-packages/requests/adapters.py", line 449, in send
    timeout=timeout
  File "/srv/homeassistant/lib/python3.7/site-packages/urllib3/connectionpool.py", line 720, in urlopen
    method, url, error=e, _pool=self, _stacktrace=sys.exc_info()[2]
  File "/srv/homeassistant/lib/python3.7/site-packages/urllib3/util/retry.py", line 436, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPConnectionPool(host='api.open-notify.org', port=80): Max retries exceeded with url: /iss-pass.json?lat=XXXXXXXX&lon=XXXXXXXXX&n=2 (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0xa29a64f0>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/helpers/entity.py", line 268, in async_update_ha_state
    await self.async_device_update()
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/helpers/entity.py", line 448, in async_device_update
    await self.hass.async_add_executor_job(self.update)
  File "/usr/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/components/iss/binary_sensor.py", line 98, in update
    self.iss_data.update()
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/util/__init__.py", line 240, in wrapper
    result = method(*args, **kwargs)
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/components/iss/binary_sensor.py", line 120, in update
    self.is_above = iss.is_ISS_above(self.latitude, self.longitude)
  File "/srv/homeassistant/lib/python3.7/site-packages/pyiss/__init__.py", line 140, in is_ISS_above
    test = self.pass_times(latitude, longitude, altitude, 2)
  File "/srv/homeassistant/lib/python3.7/site-packages/pyiss/__init__.py", line 87, in pass_times
    params=payload, timeout=5)
  File "/srv/homeassistant/lib/python3.7/site-packages/requests/api.py", line 75, in get
    return request('get', url, params=params, **kwargs)
  File "/srv/homeassistant/lib/python3.7/site-packages/requests/api.py", line 60, in request
    return session.request(method=method, url=url, **kwargs)
  File "/srv/homeassistant/lib/python3.7/site-packages/requests/sessions.py", line 533, in request
    resp = self.send(prep, **send_kwargs)
  File "/srv/homeassistant/lib/python3.7/site-packages/requests/sessions.py", line 646, in send
    r = adapter.send(request, **kwargs)
  File "/srv/homeassistant/lib/python3.7/site-packages/requests/adapters.py", line 516, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPConnectionPool(host='api.open-notify.org', port=80): Max retries exceeded with url: /iss-pass.json?lat=XXXXX&lon=XXXXXX&n=2 (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0xa29a64f0>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution'))

Additional information:

We have no control over third-party services.


fix initialization bug for kwargs in function

Generator.function does not define self.kwargs when we don't pass that argument progamatically in functions.Single.
To reproduce this error, invoke garak's cli like below
from garak.cli import main as garak_main
from typing import List

def generate_text(
) -> List[str]:
    return "empty str"

def main():
    args = [
        "-m",
        "function.Single",
        "-n",
        f"{__name__}#generate_text",
        "-p",
        "test.Blank"
    ]
    garak_main(args)

if __name__ == "__main__":
    main()

Offered fix resolves this issue.

I have read the DCO Document and I hereby sign the DCO


Improve RTL checks

Unfortunately I can't squash the request, if you'll decide to accept the request, please squash it.
The current implementation that checks whether it's an RTL device has one specific flaw, it does not check if the developer defined specifically in AndroidManifest android:supportsRtl="false", a usecase might be that an app does not want to support RTL because it does not look good when this flag is toggled on.
I made 2 main changes:

Removed from the manifest android:supportsRtl="true" which is wrong for a library
Changed isRtl from LayoutUtils to read what is the configuration the manifest

Looks good to me, thanks!


关于集成spring-boot

Please answer these questions before submitting your issue. Thanks!
Which version of Sharding-Jdbc do you using?
Expected behavior
Actual behavior
Steps to reproduce the behavior
Please provide the reproduce example codes (such as github link) if possible.
请问要如何集成spring-boot + mybatis 了？没有提供对应的示例？

https://github.com/shardingjdbc/sharding-jdbc-example


Update main repo readme

condenses main repo readme, adds project state and licence badges
removes unused bcgovr template scripts

A quick attempt at a tighter main repo readme. I suggest a repo reorg/tidy to put the first R attempt work into a folder, with it's own readme re: what are the scripts, what software do you need etc). This main readme should focus on how to get started to run the MVP, e.g. what tools, where to get them etc, how to use the scripts (or direct users to a subfolder readme if there).

Looks good to me - just noting that the main readme will need updating with a note if/when the R work all gets moved over to another folder.

Yep. I was thinking of waiting for @jonjunduan to merge those changes into the dbt branch and then I will pull those changes into this branch before merging.

@stephhazlitt you should be good to merge the changes into your branch now. Assuming that works for you, this should be good to merge back into dbt.


The DisplayInformation class is missing DPI related functionality

Describe the bug
The Microsoft.Graphics.Display.DisplayInformation class introduced in the Windows App SDK 1.2 is missing all functionality related to DPI that was available in the UWP version.
Specifically, the following APIs are missing:

DpiChanged event
LogicalDpi property
RawDpiX property
RawDpiY property

The missing properties could be compensated by using the approach outlined in #4205. However, without the DpiChanged event is is unclear how a Windows App SDK app would react to DPI changes.
Will the DisplayInformation class be extended or do you consider that class to be complete?
Steps to reproduce the bug
Use the Microsoft.Graphics.Display.DisplayInformation class in a Windows App SDK class and attempt to retrieve DPI information from it.
Expected behavior
The same DPI related APIs as in the UWP version should be available. If that is not feasible please provide/document how DPI changes shall be handled by Windows App SDK apps.
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.2.1: 1.2.221116.1
Windows version
No response
Additional context
No response

The missing properties could be compensated by using the approach outlined in #4205. However, without the DpiChanged event is is unclear how a Windows App SDK app would react to DPI changes.

You can handle WM_DPICHANGED
(tested on Windows 10 21H1, Windows App SDK 1.1.0)

Interestingly, Windows.Graphics.Display.DisplayInformation was made available to use in desktop applications. (Note this is the UWP version here, not the Windows App SDK version.) Unfortunately, it was only just made available, so the minimum client listed is Windows 11 22H2.
According to the Windows Runtime APIs not supported in desktop applications page, you can't use the GetForCurrentView member function, but you can use the IDisplayInformationStaticsInterop interface to get an instance of the class.
Of course, there is the whole "not available on previous versions" thing, but I find it interesting that this is another case of the UWP WinRT APIs being opened up to desktop applications.

If you're using XAML in your app, a recommended option is to use XamlRoot.RasterizationScale with the XamlRoot.Changed event to listen for potential changes (you'd need to check if the RasterizationScale changed, since the Changed event may fired for multiple reasons).


Build error

Hello, we trying take some pallets from substrate through git, like frame-support = {git = https://github.com/paritytech/substrate}, but we have this error
error[E0407]: method `remove_leaf_block` is not a member of trait `ClientBackend`
   --> /home/substrate/.cargo/git/checkouts/substrate-7e08433d4c370a21/bc43ad4/client/light/src/backend.rs:249:2
    |
249 | /     fn remove_leaf_block(
250 | |         &self,
251 | |         _hash: &Block::Hash,
252 | |     ) -> ClientResult<()> {
253 | |         Err(ClientError::NotAvailableOnLightClient)
254 | |     }
    | |_____^ not a member of trait `ClientBackend`

error: aborting due to previous error
``

You can not mix git with crates.io versions


kernel 3.19 images and overlay fs.

Setting the base images with kernel 3.19
Setting overlay as fs.
Paving the way for overlay network.

I may have missed it, but is there a corresponding set of packer commits to upgrade the base images to 3.19?

@pmbauer we are using now http://cloud-images.ubuntu.com/trusty/current/hwe-v/

@enxebre what I mean is, I don't see any changes here https://github.com/Capgemini/Apollo/commits/master/packer

yeah there is no change in packer because the new base amis (upated in variables.tf) include the kernel 3.19 already :) http://cloud-images.ubuntu.com/trusty/current/hwe-v/trusty-server-hwe-v-cloudimg-amd64-kernel-info.txt

Ah; sorry; I didn't realize 3.19 was coming in 14.04 LTE. Thanks.


server: Implement non-blocking IO for VR bridges

I forgot FILE_FLAG_OVERLAPPED and it was actually blocking at ReadFile on Windows.
Now it implements non-blocking IO for both receiving and sending, for the UnixSocketBridge too. It was tested on Linux, but only on synthetic data as I don't own a Linux-compatible VR headset. On Windows it felt slightly snappier and reduced hiccups, compared to the #607.
Latency became more consistent on Linux across different update rates. On Windows it's clear that it was receiving data with a delay of 1 frame.
Time delta measurements between sending headset position and receiving a new tracker position:

Platform / Update rate / Num frames
Old (ms)
New (ms)

   p1   avg    p99
   p1   avg   p99

Windows / 90hz / 5000
0.089 0.134  0.203
0.538 1.746 4.103

Windows / 100hz / 5000
0.086 0.130  0.213
0.538 1.721 4.094

Windows / 120hz / 5000
0.076 0.192  0.253
0.550 1.844 4.123

Linux / 90hz / 5000
0.117 1.685 11.799
0.172 1.305 2.196

Linux / 100hz / 5000
0.102 0.764 10.710
0.157 1.086 1.815

Linux / 120hz / 5000
0.087 0.622  8.701
0.161 1.288 1.835

Please review and test.

what are you using to collect those statistics

100 lines of JS and plotly, it was faster to write a crossplatform tester in JS. Though it's always async and buffered (and slower than native), so doesn't represent the driver latency exactly. But should be close enough.

the difference between windows old and windows

In the old version it's a delayed tracker frame, which comes instantly when the thread unblocks after receiving a headset frame. The new latency is what it should have been instead.

I suspect this might still be handling some details w/ overlapped io incorrectly (i.e. we should be inserting explicit waits/getoverlappedresult anywhere we want a blocking-style call)

I'm calling ReadFile with 0 bytes to read, and GetOverlappedResult is only used to get amount of bytes read. ReadFile should always instantly return false with a GetLastError either 0 or IO pending. We're doing a select with WaitForMultipleObjects on auto-resetting events, so technically we should be fine here.
Indeed, there could be something I missed, so I want more people to review this to make sure all of it is implemented correctly, as I haven't touched Java and WinAPI in years.

So does it make it worse, I don't get it...

So does it make it worse, I don't get it...

It looks to me like it's taking about 1 ms to get the update now, but the old one was taking the time to get the update and it was one frame behind, so for 100 Hz, that's 10 ms + the extra time. I think this is how to interpret it? Please let me know if I got something wrong.

I think this is how to interpret it?

Yes, this should reduce latency in Windows SteamVR by ~8-11ms or more (1 frame) depending on your headset's update rate. Because the bridge thread was waking up only after receiving data, the new tracker positions were stuck until the next frame. This implementation signals the thread when to send too, so it no longer happens.
The previous implementation works and is okay because frame time consistency is more important than latency when presenting frames.
The lower half of the bar chart is the new version, now it takes 1-2ms on average to receive a response by the (simulated) driver on both platforms, if I'm measuring it right. Please do test if there is any perceivable difference.

yeah, I havent been able to test this yet, but I will try it tomorrow

we should be inserting explicit waits/getoverlappedresult anywhere we want a blocking-style call

Ah, I just realized you were talking about the part where we read the data, not the waits. Indeed the docs say:

If hFile is opened with FILE_FLAG_OVERLAPPED, the lpOverlapped parameter must point to a valid and unique OVERLAPPED structure, otherwise the function can incorrectly report that the read operation is complete.

As we're peeking and know how many bytes are available, the read operation returns true immediately with data. If not - pipe will close. Maybe we should handle the async case indeed, but JNA doesn't seem to expose GetOverlappedResult.

No measurable difference after the last 2 commits:

Platform / Update rate / Num frames
2cc1201 / before (ms)
9f45d9d / after (ms)

   p1   avg   p99
   p1   avg   p99

Windows / 90hz / 5000
0.539 1.835 4.110
0.543 1.803 4.111

Windows / 100hz / 5000
0.536 1.764 4.098
0.543 1.825 4.103

Windows / 120hz / 5000
0.540 1.806 4.116
0.545 1.791 4.108

JNA doesn't seem to expose GetOverlappedResult.

I ran into that when I was doing my own try at overlapped. This was the approach I took to gain access to GetOverlappedResult: https://github.com/kitlith/SlimeVR-Server/blob/unpoll_openvr_thread/server/src/main/java/dev/slimevr/platform/windows/Kernel32.java
Should really get around to submitting an upstream issue.

Looking deeper, it appears that the sleep precision in the VRServer thread is the limiting factor now.
Replacing Thread.sleep(1) with LockSupport.parkNanos(1000000) improves the latency on Windows to 0.6-1.6ms, but also results in slightly higher CPU usage.
As of now, I believe the latency is already acceptable.
This is how long it sleeps for:

OS
Method
p1
avg
p99

Windows
Thread.sleep(1)
1.008800 ms
2.511393 ms
4.000000 ms

Windows
LockSupport.parkNanos(1000000)
0.862300 ms
0.985986 ms
1.477500 ms

Linux
Thread.sleep(1)
1.078178 ms
1.456425 ms
1.627606 ms

Linux
LockSupport.parkNanos(1000000)
1.073479 ms
1.468993 ms
1.597515 ms

Now with VRServer waking on data the p99 latency is under 1ms and it looks smoother under load in VR. Still only properly tested on Windows.

Implemented async IPC and pose updates in the driver, this nuked the remaining 1 frame of latency - body is perfectly in sync with the head now. Will make another PR after cleaning up and more code validation.

This seems working nicely, only problem is that apparently the Linux driver has some issues :sob:


Migrate ipv4_missing_enabled deviation to metadata proto files

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 47.896%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1320

Relevant Lines:
2756

💛  - Coveralls

Validated on ATE/OTG versions of:

DUT Daemon Failure
Telemetry Basic Check Test
Singleton Test
Default Address Families

Updated remaining tests that I missed in the first version of the patch. Tested additionally on both ATE/OTG versions of

Backup NHG
Route Installation
Route Removal on non-default VRF


qt: Fix default font weight values if selected values aren't supported by the font

Fixes #3848

So basically it's smth like "should guess font weights for all font families, not only the default one, and also do this regardless of the number of font weights they support". Right?

Ops, did not see this comment before 🙈
Yeah now it's done for all fonts selected after running first time or running with -resetguisetting which probably wouldn't be required but doesn't hurt in the end. The problem in the issue was obviously that the OS default font didn't support one or both of the font weights we have as default but still had more than 4 weights so the adjustments i reworked there didn't correct the weight values to the available ones.. hard to say for sure but i think it should be that case 🤔

Is this a potential backport candidate?


[FTL] Could not create runner

Hey There,
I cant find provider-config.yaml file in .config/notify/config.yaml
             __  _ ___
  ___  ___  / /_(_) _/_ __
 / _ \/ _ \/ __/ / _/ // /
/_//_/\___/\__/_/_/ \_, / v1.0.1
                   /___/

                projectdiscovery.io
Use with caution. You are responsible for your actions
Developers assume no liability and are not responsible for any misuse or damage.
Using default provider config: /home/littleboy_pugazh/.config/notify/provider-config.yaml
[FTL] Could not create runner: could not open provider config file: open /home/littleboy_pugazh/.config/notify/provider-config.yaml: no such file or directory

Any Solution?

@litt1eb0yy you need to create one as per example shared in the readme - https://github.com/projectdiscovery/notify#provider-config


1.20.2 Crash on trying to get party pokemon

Produced on forge build 48.0.49, the mod will crash the server when attempting to get a users party pokemon, I don't have any logs however I'm sure it wouldn't be hard to get some if you needed them

If you want support please use the Discord


[KED-1121] Add MatplotlibWriter S3 DataSet

Notice

[ ] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

Motivation and Context
Why was this PR created?
How has this been tested?
What testing strategies have you used?
Checklist

[ ] Read the contributing guidelines
[ ] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Updated the documentation to reflect the code changes
[ ] Added a description of this change and added my name to the list of supporting contributions in the RELEASE.md file
[ ] Added tests to cover my changes
[ ] Assigned myself to the PR

@lorenabalan I could not figure out how to mock an encrypted bucket on moto. It seems to do nothing. I guess moto only mocks the calls..?

Also before we merge don't forget to tick the checkboxes in the PR description about contribution guide & legal agreement. :)

Also before we merge don't forget to tick the checkboxes in the PR description about contribution guide & legal agreement. :)

Ah I missed those.. got buried. Addressing now


Merge pull request #459 from MicrosoftDocs/master

Deploy October 2021 publish

Docs Build status updates of commit 6f2cadd:
:warning: Validation status: warnings
For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


[Testing] use line number / filename directive for better line number reporting

OCaml has this really neat feature of setting the line number / filename context of an executed script:
# 200 "some/file.mdx"

let a 1;

which will report an error like this:
File "some/file.mdx", line 201, characters 7-8:
Error: Syntax error

Here, the line number is correctly reported as 201.
This can potentially replace the weird regex magic done in scripts/test-examples.js.
\cc @rizo for giving the tip

this is not needed anymore, we got a way better way on testing all snippets that's at least 20 times faster