Added support for SSM connections

Unsure if you want this configurable by a variable? Considering SSM may become the only method to connect by default (https://github.com/AndrewGuenther/fck-nat/pull/48) I figured there was no harm in attaching the SSM policy by default.
For now, this will only be relevant for users building their own AMI based on your PR to add SSM support.

Thanks for the PR @kieranbrown . Indeed, adding SSM permissions would be necessary once this becomes the default in the upstream fck-nat project. However I am a bit uncomfortable with using the AWS' managed policy for this project as the policy has too many permissions.
When SSM becomes the default, I think it would be better either to make a trimmed policy containing only the IAM actions for SSM-messages required for remote shell, or alternatively at the very least exposing a variable controlling the policy ARN to let the user chose a policy that suit their needs.

@RaJiska I've updated the PR to use least privilege and added a variable to disable for those who don't want it.

LGTM, thanks for the contribution @kieranbrown .
I think outputting the role name and ARN is definitely a good idea regardless of whether the user wishes to add policies or not as the end user may make alternative use of it (e.g: storing in parameter store).
Though I'm more for adopting a minimalist approach in regards of the number of variables exposed to the user as in my opinion, the goal of this project is to mimic NAT Gateway, and make this as dumb as possible, therefore removing as much complexity as possible. In the future I may wish to remove the attach_ssm_policy variable and just have it enabled by default and make SSH off by default.


sqlalchemy_inspect - module 'time' has no attribute 'clock' where I don't use time library

Describe the bug

I am not using time library and I got an error (module 'time' has no attribute 'clock') on my package sqlalchemy_inspect - repo https://github.com/brunonascimento89/sqlalchemy-inspect
Expected behavior

Download the database schema
To Reproduce

pip install -i https://test.pypi.org/simple/ sqlalchemy-inspect
write the following code

from sqlalchemy_inspect import database
from decouple import config

conn = database.SQL(
    host=config('DB_HOST'),
    port=config('DB_PORT'),
    database=config('DB_NAME'),
    user=config('DB_USER'),
    password=config('DB_PASS'),
)
conn.extract_model(filename='test_model.py', engine='mysql+pymysql', engine_args=True)

where .env have
DB_NAME=db_name
DB_USER=db_user
DB_PASS=db_pass
DB_HOST=db_host
DB_PORT=db_port

Additional context

Error:
Exception has occurred: AttributeError
module 'time' has no attribute 'clock'
  File "project\directory\test_my_package\test.py", line 1, in <module>
    from sqlalchemy_inspect import database

Hi, this issue tracker is for issues with the codebase behind https://pypi.org itself, not the projects hosted on PyPI.
You should report this issue to the tracker for the project in question instead.

Hello @di. I got this reference on https://test.pypi.org/help/#feedback where they wrote "If you're experiencing an issue with PyPI itself, we welcome constructive feedback and bug reports via our issue tracker."
I cannot find another reference in pypi. Might you help me please? Thank you in advance


ts0601_trv_maxsmart  ZHA  can't load zhaquirks

Hi ,
Any chances to solve this problem ?
Home Assistant 2023.7.3
Supervisor 2023.07.1
Operating System 10.3
Logger: zhaquirks
Source: /usr/local/lib/python3.11/site-packages/zhaquirks/init.py:456
First occurred: 16:31:25 (1 occurrences)
Last logged: 16:31:25
Unexpected exception importing custom quirk 'ts0601_trv_maxsmart'
Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/zhaquirks/init.py", line 454, in setup
importer.find_module(modname).load_module(modname)
File "", line 605, in _check_name_wrapper
File "", line 1120, in load_module
File "", line 945, in load_module
File "", line 290, in _load_module_shim
File "", line 721, in _load
File "", line 690, in _load_unlocked
File "", line 936, in exec_module
File "", line 1074, in get_code
File "", line 1004, in source_to_code
File "", line 241, in _call_with_frames_removed
File "/config/custom_zha_quirks/ts0601_trv_maxsmart.py", line 91
zha_quirks/ts0601_trv_maxsmart.py at main · jacekk015/zha_quirks · GitHub
^
SyntaxError: invalid character '·' (U+00B7)

Better download that file again.
There's no line:
<title>zha_quirks/ts0601_trv_maxsmart.py at main · jacekk015/zha_quirks · GitHub</title>
in the code.
You can see it yourself:
https://github.com/jacekk015/zha_quirks/blob/17033c8de6ef036af6f5b148b046e3f8e4076548/ts0601_trv_maxsmart.py#L91

Hi ,
Thanks , it worked , looks like linux did something when I  downloaded file or uploaded to HA,  did same operation on Windows and all working.

Glad it worked.


Error handling in setup plugin

Clear the form and submit it, there is no errors.

Resolved in #1322

duplicated #1391


See if we can remove Chrome requirement for CI

We were using the chrome engine for CI, to run Jasmine using Grunt. We're no longer using Grunt, and are just running Jasmine directly. I don't think this needs the Chrome engine anymore, as I think it uses Node's.
This should be tested and, if not required, removed from the Travis build as it slows CI down.

This has been removed and will be released in v4.0.0


Debug button named "Attach" instead of Start

Minor issue here, VS2017 15.4.0 Python 3.0.17262.1
Open up a new Python project, select the code editor window. Debug button becomes "Attach..." instead of "Start".

Thanks! We're already tracking that one here:
https://github.com/Microsoft/PTVS/issues/2423


”TÂRGU MUREȘ” is listed in the drop-down menu

Description:
"TÂRGU MUREȘ" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "TÂRGU MUREȘ" in the search bar.

Expected results
The location is listed in the drop-down menu as "Oraș TÂRGU MUREȘ, MUREȘ (TÂRGU MUREȘ)".


Generic /association/find route returning 500

https://api.monarchinitiative.org/api/association/find/?rows=0

that's odd, I think rather than increasing the timeout we should explore why a basic query like this takes so long. We do similar things with the GO one all the time
E.g. the call underlying this should be pretty much the same:
http://amigo.geneontology.org/amigo/search/annotation
cc @kltm

I'd first check used memory versus size of store on the machine.
If you can skype me the underlying machine name, I can check myself.

Okay, I took a quick look at this. Some notes:

the solr index more-or-less given 30G

I'm not sure if the heap allocation in java will allocate more, but it may well

the machine has had some swapping history

free -m

the machine is using a rotational disk

cat /sys/block/sr0/queue/rotational

the index is well north of 250G
load is low

Given all of this, my best guess is that you have insufficient RAM to accomplish the task or that you are so close to the machine RAM limits that you occasionally manage to trigger the need for swap, slowing things down.
My recommendation would be to start by dropping the RAM given to Solr by a couple of G (to prevent possible swapping) and see if things improve. Otherwise, you may want to look in to a beefier machine.
That said, I'm not sure this is the actual cause, but obviously not good either way. More information could be gotten from the logs at the time of the 500 and seeing what it is about the queries around then, and trying to duplicate the issue from the machine.

Tagging @kshefchek

There is something fishy here. I have queried the monarch solr directly
many times and it's fairly responsive, not sure why the basic fetch all
documents query would be so slow, it's possible the code is constructing
an ultra-dumb query, maybe it's retrieving every single facet value for
counts..?

Here is the query generated by BioLink-API/Ontobio:
https://solr.monarchinitiative.org/solr/golr/select/?q=*:*&facet=on&facet.field=subject_taxon_label&facet.field=object_closure&facet.limit=25&facet.mincount=1&fl=id,is_defined_by,source,subject,subject_label,subject_taxon,subject_taxon_label,relation,relation_label,object,object_label,object_taxon,object_taxon_label,evidence_object,evidence_graph,reference,qualifier,is_redundant_for,type,evidence,evidence_label,evidence_type,evidence_type_label,evidence_with,evidence_closure,evidence_closure_label,evidence_subset_closure,evidence_subset_closure_label,evidence_type_closure,evidence_type_closure_label&rows=10&wt=json
There are two closures for facet fields:

subject_taxon_label
object_closure

Do we need facets for object_closure in this query? Removing that from facet.field reduces the query run from 8 seconds to 0.5 second.

It's good to have a test case; it's worrying that it works every time--that should be getting cached.
I would note that looking at the above query running on the machine, it seems to peg a couple of CPUs for the duration of the query, but little I/O activity (that I've found so far).
I would still start poking the memory settings here--no matter what, history of swapping would indicate a problem. Also, this is apparently not a real machine (KVM); the CPU here may be hiding the I/O to the real system or similar. It might we worth checking the status of the host machine.

(although at the moment the default upper level terms are too broad to be interesting)

Yes, thats what made me wonder if its useful to have object_closure facet field. But it definitely makes sense to have in in there, by default, since that would be the expectation for this call.


Next exercise?

I really love the way you set this whole DDD exercise up. That said, how about exercise 4, is it still in the pipeline?

@Expecho, thank you for the kind words. Exercises 4 and 5 are still in the pipeline.
I don't want to promise anything, but I'm working on getting the 4th done. 3rd one wasn't polished enough to be engaging, so I don't want to repeat the same mistake.
There will be a newsletter update when the next exercise is out.


lm buddy ragas draft

What's changing
Added ragas entrypoint. Still need to run a full test on it but it code is generally ready for review.
How to test it
Related Jira Ticket
https://mzai.atlassian.net/browse/RD2024-5
Additional notes for reviewers

Would also be helpful to write some documentation here under evaluation_concepts: https://github.com/mozilla-ai/lm-buddy/tree/main/docs/source

Thanks for this @imtihan ! LGTM once tests pass 🚀


pnpm update

Using a simple pnpm update didn't fix any of the issues in #1900 but it fixes some vulnerabilities shown with pnpm audit. I didn't try to force-update semver to fix the security vulnerability there.

@qwerty287 ci failed related


Add token count for diffs with tiktoken

The code is moved to GitInfo class but I can fix it for you too

@antonakospanos fixed in another PR


Update metadata.json

According to the specs (https://puppet.com/docs/puppet/7.4/modules_metadata.html) name should use dash as separator not slash. This triggers an error when doing a metadata-json-lont on this module. It als prevents our IDE (RubyMine) from correctly resolving module dependencies. Also allow latest dependencies.

@crayfishx could you give this a short review and release the fix?


Switch joycons misbehave on Android

Thumbsticks are orientated for vertical hold
Left face buttons (dpad) are non-functional
SL/SR non-functional
L+R pairing to join two joycons is unsupported when tested (though should work apparently?)

Android kernel is using a version of the Linux drivers that aren't quite production ready (dkms-hid-nintendo). Eventually (as in, with an Android update), all of the auto-pairing etc. should be taken care of. For now we're blacklisting:
https://gist.github.com/offalynne/07f78774e3d9ff00fc79200951ac1883#file-sdl2-guid-and-android-device-description-gamepad-blocklist-L7-L8
If come v4 this is still a problem (hint: it will be) we'll do internal pairing for JoyCon couples on Android.


Fix for arcgiscache layers

Layer url used for ArcGISCache and ol3 were producing invalid xyz urls.

TileArcGISRest source is still flagged as experimental as of 3.9.0. Once it's know this is staying, probably best to update the createArcGISCacheLayer function to use a TileArcGISRest


Check low balance before send

Check balance before sending transaction
https://github.com/w3f/Grant-Milestone-Delivery/pull/713#issuecomment-1426085300
Steps:

Scan QR code for receive, send with low balance

Expected: Should alert user of low balance
Actual: Tries transaction

Reference:
https://polkadot.js.org/docs/api/start/api.tx.subs/#payment-information
the illogically named "partialFee" is actually ... the fee.

In my case, this didn't work as expected.

Although my account balance is sufficient:

It would display this error:


Fixed typo in sentence

Summary
Describe your changes here.
Fixes #Issue_Number (if available)

@mairaw cleaned up the other commits. Now one commit for one file change.


want for python 3

please we want it for python 3

I am not a professional coder. I only do a little bit of hacking. My suggestions are:

You can try and port the code to Python 3 and send a pull request or
Say which code is priority for you and what you are trying to accomplish and someone else may be able to help.
Good luck!

Thanks.
Can you please suggest me the path to start learning hacking
On Fri, Feb 8, 2019, 10:42 AM இரா. அசோகன்<EMAIL_ADDRESS>wrote:

I am not a professional coder. I only do a little bit of hacking. My
suggestions are:

You can try and port the code to Python 3 and send a pull request or
Say which code is priority for you and what you are trying to
accomplish and someone else may be able to help.
Good luck!

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/AshokR/TamilNLP/issues/6#issuecomment-461692930, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AfOsaa3F-y_eei5sAm8kyiZrJRDN0AAFks5vLQdJgaJpZM4anajC
.

Please say what you are trying to accomplish if the code is available for Python 3. This will help me, and others, to recommend the best way for you to reach that goal.


🐛 [firebase_analytics] Missing google_app_id. Firebase Analytics disabled.

Bug report
Describe the bug
I get the following error while trying to use firebase analytics
Missing google_app_id. Firebase Analytics disabled. See https://goo.gl/NAOOOI

Steps to reproduce
I have created a new project.
I have setup firebase using the command
flutterfire configure

But when a  call is made to Firebase Analytics like
FirebaseAnalytics.instance.logEvent(name: 'mycustomevent');

it throws the following error in the screen
Missing google_app_id. Firebase Analytics disabled. See https://goo.gl/NAOOOI

Expected behavior
Should not throw error
Sample project
Relevant code.
              FirebaseAnalytics.instance.logEvent(name: 'mycustomevent');

Firebase setup code in main.dart
  // Initialize Firebase.
  WidgetsFlutterBinding.ensureInitialized();
  await Firebase.initializeApp(
    options: DefaultFirebaseOptions.currentPlatform,
  );
runApp(MyApp());

Additional context
Generated firebase_options.dart is as follows
// File generated by FlutterFire CLI.
// ignore_for_file: lines_longer_than_80_chars
import 'package:firebase_core/firebase_core.dart' show FirebaseOptions;
import 'package:flutter/foundation.dart'
    show defaultTargetPlatform, kIsWeb, TargetPlatform;

/// Default [FirebaseOptions] for use with your Firebase apps.
///
/// Example:
/// ```dart
/// import 'firebase_options.dart';
/// // ...
/// await Firebase.initializeApp(
///   options: DefaultFirebaseOptions.currentPlatform,
/// );
/// ```
class DefaultFirebaseOptions {
  static FirebaseOptions get currentPlatform {
    if (kIsWeb) {
      throw UnsupportedError(
        'DefaultFirebaseOptions have not been configured for web - '
        'you can reconfigure this by running the FlutterFire CLI again.',
      );
    }
    // ignore: missing_enum_constant_in_switch
    switch (defaultTargetPlatform) {
      case TargetPlatform.android:
        return android;
      case TargetPlatform.iOS:
        return ios;
      case TargetPlatform.macOS:
        throw UnsupportedError(
          'DefaultFirebaseOptions have not been configured for macos - '
          'you can reconfigure this by running the FlutterFire CLI again.',
        );
    }

    throw UnsupportedError(
      'DefaultFirebaseOptions are not supported for this platform.',
    );
  }

  static const FirebaseOptions android = FirebaseOptions(
    apiKey: '***masked***',
    appId: '***masked***',
    messagingSenderId: '***masked***',
    projectId: '***masked***',
    storageBucket: '***masked***',
  );

  static const FirebaseOptions ios = FirebaseOptions(
    apiKey: '***masked***',
    appId: '***masked***',
    messagingSenderId: '***masked***',
    projectId: '***masked***',
    storageBucket: '***masked***',
    iosClientId: '***masked***',
    iosBundleId: '***masked***',
  );
}

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.8.1, on Ubuntu 21.10 5.13.0-27-generic, locale en_IN)
[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0-rc1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.1)
[✓] VS Code (version 1.63.2)
[✓] Connected device (2 available)

• No issues found!

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
$ flutter pub deps -- --style=compact
Dart SDK 2.15.1
Flutter SDK 2.8.1
myapp 1.0.0+1

dependencies:
- android_intent_plus 3.1.0 [flutter platform meta]
- another_flushbar 1.10.28 [flutter]
- bottom_sheet 1.0.5 [flutter]
- cached_network_image 3.2.0 [flutter flutter_cache_manager octo_image cached_network_image_platform_interface cached_network_image_web]
- card_swiper 1.0.4 [flutter]
- cupertino_icons 1.0.4
- dart_geohash 2.0.1
- dio 4.0.4 [http_parser path]
- dio_http_cache 0.3.0 [crypto dio flutter json_serializable json_annotation path quiver sqflite]
- easy_debounce 2.0.2+1
- equatable 2.0.3 [collection meta]
- f_logs 2.0.0 [flutter sembast path_provider intl xxtea meta path stack_trace]
- firebase_analytics 9.1.0 [firebase_analytics_platform_interface firebase_analytics_web firebase_core firebase_core_platform_interface flutter]
- firebase_core 1.12.0 [firebase_core_platform_interface firebase_core_web flutter meta]
- firebase_crashlytics 2.5.0 [firebase_core firebase_core_platform_interface firebase_crashlytics_platform_interface flutter stack_trace]
- flutter 0.0.0 [characters collection meta typed_data vector_math sky_engine]
- flutter_cache_manager 3.3.0 [clock collection file flutter http path path_provider pedantic rxdart sqflite uuid]
- flutter_hooks 0.18.2 [flutter]
- flutter_map 0.14.0 [async collection flutter http intl latlong2 meta positioned_tap_detector_2 proj4dart transparent_image tuple vector_math]
- flutter_map_location_marker 2.0.0 [flutter flutter_compass flutter_map geolocator latlong2]
- flutter_svg 1.0.3 [flutter meta path_drawing vector_math xml]
- galleryimage 1.1.0 [flutter cached_network_image photo_view]
- geolocator 8.0.5 [flutter geolocator_platform_interface geolocator_android geolocator_apple geolocator_web]
- get_storage 2.0.3 [flutter get path_provider]
- latlong2 0.8.1 [intl]
- location 4.3.0 [flutter location_platform_interface location_web]
- logger 1.1.0
- material_design_icons_flutter 5.0.6595 [flutter]
- material_dialog 0.0.9 [flutter meta]
- material_floating_search_bar 0.3.6 [flutter meta]
- path_provider 2.0.8 [flutter path_provider_android path_provider_ios path_provider_linux path_provider_macos path_provider_platform_interface path_provider_windows]
- provider 6.0.2 [collection flutter nested]
- share 2.0.4 [meta mime flutter]

dev dependencies:
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters charcode collection matcher meta source_span stream_channel string_scanner term_glyph typed_data]

transitive dependencies:
- _fe_analyzer_shared 22.0.0 [meta]
- analyzer 1.7.2 [_fe_analyzer_shared cli_util collection convert crypto glob meta package_config path pub_semver source_span watcher yaml pedantic]
- args 2.3.0
- async 2.8.2 [collection meta]
- boolean_selector 2.1.0 [source_span string_scanner]
- build 2.2.1 [analyzer async convert crypto glob logging meta path]
- build_config 1.0.0 [checked_yaml json_annotation path pubspec_parse yaml]
- cached_network_image_platform_interface 1.0.0 [flutter flutter_cache_manager]
- cached_network_image_web 1.0.1 [flutter flutter_cache_manager cached_network_image_platform_interface]
- characters 1.2.0
- charcode 1.3.1
- checked_yaml 2.0.1 [json_annotation source_span yaml]
- cli_util 0.3.5 [meta path]
- clock 1.1.0
- collection 1.15.0
- convert 3.0.1 [typed_data]
- crypto 3.0.1 [collection typed_data]
- dart_style 2.1.1 [analyzer args path pub_semver source_span]
- fake_async 1.2.0 [clock collection]
- ffi 1.1.2
- file 6.1.2 [meta path]
- firebase_analytics_platform_interface 3.0.5 [firebase_core flutter meta plugin_platform_interface]
- firebase_analytics_web 0.4.0+6 [firebase_analytics_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins js]
- firebase_core_platform_interface 4.2.4 [collection flutter meta plugin_platform_interface]
- firebase_core_web 1.5.4 [firebase_core_platform_interface flutter flutter_web_plugins js meta]
- firebase_crashlytics_platform_interface 3.1.13 [collection firebase_core flutter meta plugin_platform_interface]
- flutter_blurhash 0.6.0 [flutter meta pedantic]
- flutter_compass 0.7.0 [flutter]
- flutter_web_plugins 0.0.0 [flutter js characters collection meta typed_data vector_math]
- geolocator_android 3.0.2 [flutter geolocator_platform_interface]
- geolocator_apple 2.0.1 [flutter geolocator_platform_interface]
- geolocator_platform_interface 4.0.1 [flutter plugin_platform_interface vector_math meta]
- geolocator_web 2.1.4 [flutter flutter_web_plugins geolocator_platform_interface]
- get 4.6.1 [flutter]
- glob 2.0.2 [async collection file path string_scanner]
- http 0.13.4 [async http_parser meta path]
- http_parser 4.0.0 [charcode collection source_span string_scanner typed_data]
- intl 0.17.0 [clock path]
- js 0.6.3
- json_annotation 4.1.0 [meta]
- json_serializable 4.1.4 [analyzer build build_config collection json_annotation meta path source_gen]
- lists 1.0.1 [meta]
- location_platform_interface 2.3.0 [flutter meta plugin_platform_interface]
- location_web 3.1.1 [flutter flutter_web_plugins http_parser js location_platform_interface meta]
- logging 1.0.2
- matcher 0.12.11 [stack_trace]
- meta 1.7.0
- mgrs_dart 2.0.0 [unicode]
- mime 1.0.1
- nested 1.0.0 [flutter]
- octo_image 1.0.1 [flutter flutter_blurhash]
- package_config 2.0.2 [path]
- path 1.8.0
- path_drawing 1.0.0 [vector_math meta path_parsing flutter]
- path_parsing 1.0.0 [vector_math meta]
- path_provider_android 2.0.11 [flutter path_provider_platform_interface]
- path_provider_ios 2.0.7 [flutter path_provider_platform_interface]
- path_provider_linux 2.1.5 [ffi flutter path path_provider_platform_interface xdg_directories]
- path_provider_macos 2.0.5 [flutter path_provider_platform_interface]
- path_provider_platform_interface 2.0.3 [flutter platform plugin_platform_interface]
- path_provider_windows 2.0.5 [ffi flutter path path_provider_platform_interface win32]
- pedantic 1.11.1
- petitparser 4.4.0 [meta]
- photo_view 0.12.0 [flutter]
- platform 3.1.0
- plugin_platform_interface 2.1.2 [meta]
- positioned_tap_detector_2 1.0.4 [flutter]
- process 4.2.4 [file path platform]
- proj4dart 2.0.0 [mgrs_dart wkt_parser meta]
- pub_semver 2.1.0 [collection meta]
- pubspec_parse 1.1.0 [checked_yaml collection json_annotation pub_semver yaml]
- quiver 3.0.1+1 [matcher]
- rxdart 0.27.3
- sembast 3.1.2 [meta path synchronized]
- sky_engine 0.0.99
- source_gen 1.0.3 [analyzer async build dart_style glob meta path pedantic source_span yaml]
- source_span 1.8.1 [collection path term_glyph]
- sqflite 2.0.2 [flutter sqflite_common path]
- sqflite_common 2.2.0 [synchronized path meta]
- stack_trace 1.10.0 [path]
- stream_channel 2.1.0 [async]
- string_scanner 1.1.0 [charcode source_span]
- synchronized 3.0.0
- term_glyph 1.2.0
- test_api 0.4.3 [async boolean_selector collection meta source_span stack_trace stream_channel string_scanner term_glyph matcher]
- transparent_image 2.0.0
- tuple 2.0.0 [quiver]
- typed_data 1.3.0 [collection]
- unicode 0.3.1 [lists]
- uuid 3.0.5 [crypto]
- vector_math 2.1.1
- watcher 1.0.1 [async path]
- win32 2.3.8 [ffi]
- wkt_parser 2.0.0
- xdg_directories 0.2.0 [meta path process]
- xml 5.3.1 [collection meta petitparser]
- xxtea 2.1.0
- yaml 3.1.0 [collection source_span string_scanner]

@the100rabh
Are you trying to run this on Android or iOS ?
According to the documentation, the dart-only initialization is not supported by all plugins on all platforms. See this table for your reference which shows firebase_analytics plugin isn't yet supported on Android platform for dart-only initialization.
https://firebase.flutter.dev/docs/cli/#usage

I was trying for Android. Got it thanks, I guess I will have to go the manual route for this. The documentation is misleading there for firebase analytics in that case.

I just got the exact same error after ditching the old method in favor of flutterfire configure.


[50-52, mysql] Fix CI encoding problems

Encoding of the test DBs was wrong...
Fixes #879

very nice, thanks!


Cache-Control wildcards in citizen config

You can set the Cache-Control headers currently for specific static assets within the config like so:
  "control": {
    "/forum/themes/default/production.css": "max-age=31536000",
    "/forum/themes/default/production.js": "max-age=31536000",
    "/forum/themes/default/fonts/open-sans/opensans-regular-webfont.woff2": "max-age=31536000",
    "/forum/themes/default/fonts/open-sans/opensans-semibold-webfont.woff2": "max-age=31536000",
    "/forum/themes/default/fonts/open-sans/opensans-bold-webfont.woff2": "max-age=31536000",
    "/forum/themes/default/fonts/glyphs/icomoon-subset.woff": "max-age=31536000"
  }

Add support for wildcards so that entire directories or files matching the specified pattern receive the same header:
  "control": {
    "/forum/themes/default/production.*": "max-age=31536000",
    "/forum/themes/default/fonts/open-sans/opensans-*": "max-age=31536000",
    "/forum/themes/default/fonts/glyphs/icomoon-subset.woff": "max-age=31536000"
  }

Or:
  "control": {
    "/forum/themes/**/production.*": "max-age=31536000",
    "/forum/themes/**/fonts/**/*": "max-age=31536000"
  }

Added in 8ca1a895e129636e086c128f33dfdb9a843a3046. Uses regular expressions for more flexibility.


Age Calculator App

#335
This is the app for Age Calculator App Issue referenced above.

@avishekdas128 Your app is fabulous. Thank you so much for your contribution. Happy Coding! and Keep Contributing

@avishekdas128 Your app is fabulous. Thank you so much for your contribution. Happy Coding! and Keep Contributing

Thanks alot, can the hacktoberfest-accepted label be added?

@avishekdas128 yes, but it would accepted even if we merge it without adding a label

@avishekdas128 Give a star to our repo if you liked it.

@avishekdas128 Thanks and keep contributing!


.track should use this.events to retrieve matching events

Affected LOC
Should instead use the new array-handling events() functionality introduced in https://github.com/segmentio/analytics.js-integration/pull/45; as written currently it's case-sensitive so it's more restrictive than it should be.

added to jira https://segment.atlassian.net/browse/INT-182


TITLE_2: H1 must/should match part of the TITLE

Desc: Same issue as HEADING_1 when page has no H1
Data: "page: no page title" in Element column of Element Results table
Tool: AInspector Sidebar
Evaluation Library Version: <IP_ADDRESS>
Ruleset: HTML4 Legacy Techniques
URL: https://news.ycombinator.com/

Updated TITLE rule 2, it had a number of issues including not being very forgiving for more than 2 visible H1 elements

updated messaging for checking title with up to 2 H1 elements for consistency of H1 content with TITLE content
If more than 2 visible H1 elements the rule returns a manual check for verifying the H1 elements identify and describe a major section of the page, this clause though creates a mismatch with the rule summary and description

Remove rule from rulesets.  The rule has problems when there are more that 2 H1 elements on the page.


Modify library to accept partition path

Fixing the library issue
Changes to format specific partition
Consider lvm partitions for Swift in local
Change name for OS shared partition var
Fix the syntax issues

@j2sol Can we get this merged?


Importing chain file from the threeML BayesianAnalysis results over to Xspec

Question:
After an Ultranest BayesianAnalysis() run, which is the chain file than can be imported outside the threeML environment for external processing from the Monte Carlo samples? If I'm not wrong, it should be the same file that corner_plot() calls.
There are three text (not FITS) files available with parameter samples inside, after the BayesianAnalysis sampling with UltraNest:

equal_weighted_post.txt
weighted_post.txt
weighted_post_untransformed.txt.

The other file is the run.txt, which contains the statistical bits.
The last is the AR container itself (a FITS file). But it has a key issue that does not allow importing over Xspec. This is the error I get if I try to load it on Xspec with the chain load myARfile.fits:
***XSPEC Error: Chain IO Error: while attempting to read CHAIN extension in myARfile.fits
I changed the EXTNAME keyword from "ANALYSIS_RESULTS" to "CHAIN" in the header file of a copy of the AR container. But the chain load after that returns an error I could not fix:
***XSPEC Error: At least 1 pair of TEMPR and STROW keywords are required in file
Requirement:
Knowing which file is the correct one is essential, since I could then try to sample on Xspec to compute the equivalent width of my line model component. The eqwidth command on Xspec allows that from MCMC chain samples.

This is a question about ultranest and should be directed there.

This is a question about ultranest and should be directed there.

Understood. Thank you.

I shall provide an update here on the solution, if I find one over the Ultranest repository. It may help someone else later. So, I request to keep this open until then.

Question:
After an Ultranest BayesianAnalysis() run, which is the chain file than can be imported outside the threeML environment for external processing from the Monte Carlo samples? If I'm not wrong, it should be the same file that corner_plot() calls.
There are three text (not FITS) files available with parameter samples inside, after the BayesianAnalysis sampling with UltraNest:

equal_weighted_post.txt
weighted_post.txt
weighted_post_untransformed.txt.

The other file is the run.txt, which contains the statistical bits. The last is the AR container itself (a FITS file). But it has a key issue that does not allow importing over Xspec. This is the error I get if I try to load it on Xspec with the chain load myARfile.fits: ***XSPEC Error: Chain IO Error: while attempting to read CHAIN extension in myARfile.fits I changed the EXTNAME keyword from "ANALYSIS_RESULTS" to "CHAIN" in the header file of a copy of the AR container. But the chain load after that returns an error I could not fix: ***XSPEC Error: At least 1 pair of TEMPR and STROW keywords are required in file
Requirement: Knowing which file is the correct one is essential, since I could then try to sample on Xspec to compute the equivalent width of my line model component. The eqwidth command on Xspec allows that from MCMC chain samples.

Solution:
It's the equal_weighted_post.txt file.
https://johannesbuchner.github.io/UltraNest/performance.html#output-files


Form v2

For RadioButtons:

RadioButton receives native click
It creates proper synthetic event with correct value
BaseInputComponent receives event and can safely extract value
Value propagates up to form handler


The named parameter 'config' isn't defined

Trying to add   config: NumberPaginatorUIConfig(     // default height is 48     height: 64,     buttonShape: BeveledRectangleBorder(       borderRadius: BorderRadius.circular(8),     ),
and getting 'The named parameter 'config' isn't defined' / 'The method NumberPaginatorUIConfig isn't defined for type...'

@gliitterpuke which version of number_paginator are you using?

Closing due to inactivity. @gliitterpuke feel free to reopen if the issue persists.


Revisit startPushScalers for ScaledJob

There is this TODO comment in the scale_handler.go, I wonder whether it is still relevant:
https://github.com/kedacore/keda/blob/b6c60a088a7ac5683638feba3692b40b0b613f29/pkg/scaling/scale_handler.go#L168-L170

@TsuyoshiUshio can you please take a look and let me know if you have any questions?

Hi @zroubalik and @ahmelsayed
Sorry, I forget to implement the push scaler case. The reason the comment is there is, You can see the hard corded value. It is scaleTo and MaxScale. I need to write one more method on scale_handler.go.
I have a question. For the startPushScaler method, why it calls RequestScale method for each loop? For the scaling, there are several scaler exists at the same time, that is why scale logic is calculated from all scalers metrics. So that checkScalers passing scalars array to the RequestScale. https://github.com/kedacore/keda/blob/v2/pkg/scaling/scale_handler.go#L181
However, startPushScalers method looping with go routine. for each go routine request scale in parallel. That means, if the 2 push scaler is there, then it is going to exceed the maxscale. https://github.com/kedacore/keda/blob/v2/pkg/scaling/scale_handler.go#L142  What I'd like to know is, why do you change de design between checkScalers and startPushScalers. I'd like to know the context and background.

The behavior of a push scaler is to push isActive state whenever there is an external event. Multiple push scalers don't have to coordinate that push with each other because they are not pushing a particular number to scale to, but rather just the isActive state. if multiple scalers push isActive at the same time, KEDA will only attempt to activate the deployment (scale to 1) for the first and debounces the rest. How does scaledJob handles multiple triggers reporting isActive on every pollingInterval. You can think of push scalers as if the pollingInterval is 0.

Thank you @ahmelsayed  Then for the scaled job for the Push Scaler, can I assume something like this? I can't understand the meaning of the debounces. Is it only the first one is executed?  If I assume so,

MaxScale is applyed for the first push scaler. If the other pushscalers are fired, it will be ignored.
to scale is just one.  (That mean, if the push scaler is trigged, then 1 job is created for consume it.

Sorry for the basic question, could you tell me some usecase for Scaled Job with Push Scaler?

Thank you @ahmelsayed to share the info with me offline.
I noticed that ScaledJob doesn't fit for the current implementation for using the go routine.
https://github.com/kedacore/keda/blob/b6c60a088a7ac5683638feba3692b40b0b613f29/pkg/scaling/scale_handler.go#L168
PushScaler could be more than 1,
My suggestion is,

startPushScaler method is called, if the object is scaled jobs, Loop through the PushScalers. (not to loop on the current go routine)
Calculate the scaleTo by the scaler's metadata by the all PushScalers metadata so that if there is multiple push scalers were there, the amount of the job doesn't exceed the Max Scale.

If it is ok, I'll implement it and find time to do it.

That sounds good to me.

I think this issue ok for now.


🌐 Add Chinese translation for docs/zh/docs/advanced/advanced-dependencies.md

as title

📝 Docs preview for commit 799a5cd55cfd3c16f7c5c20c7dc5290af9c494fd at: https://6499e2cac058f30345225e75--fastapi.netlify.app

Thank you for your work and effort! @jaystone776 🎉
And thanks for the review @TimorChow ✌


Make the block coding system work in game, not just in the editor

What Happened
There are uses for this drag-and-drop coding system within a running game system to give it a kind of extreme configurability.  For example, a runtime version of this module this module was made for the Heroes card game two stands down from the Endless stand at Godotcon2024.
An in-game version of Block Coding would also provide the capability to run this feature under Godot's multiplayer system for the purpose of remote collaboration/coaching/teaching, since the editor itself is unlikely ever to be upgraded to multiplayer.
How to Reproduce
When I instantiate res://addons/block_code/ui/main_panel.tscn into a game it doesn't work due to numerous references to EditorInterface.get_inspector(), a single global instance of BlockEditorContext, and so forth.
Block Coding Plugin Version
v0.7
Godot Engine Version
v4.3
Operating System
Nixos
Logs or command-line output
No response
Anything else?
No response

Features that rely on the editor (like dragging nodes from the scene tree) would have to be removed to enter it in a game environment. Instead, why not a multiplayer version of this plugin with a pull request? I haven't made any multiplayer games yet, but it seems that a multiplayer plugin should be possible

Duplicate of https://github.com/endlessm/godot-block-coding/issues/196


Depsy

Because we should have more tags. And to also better track utilization and statistics. Came across Depsy as matplotlib (depsy link) is tied into it. The project itself is NSF funded, so may be of broader @biocore interest.
I'm filing this under CI but not sure if that is the most appropriate.

You can actually already access to the scikit-bio link:
http://depsy.org/package/python/scikit-bio

Nifty...

This looks really cool, nice find! Somehow @gregcaporaso is showing up as the only scikit-bio contributor, not sure where they're getting those stats from.
I'll tag this as a quick fix in case anyone's interested in adding the badge to scikit-bio's README.


SetChargingProfile not implemented

Hi,
When I was doing the test, I got the notification as below:
[MO] Recv: [2,"c613ba53-331d-4c08-9068-d50d4d2dcfb7","SetChargingProfile",{"connectorId":0,"csChargingProfiles":{"chargingProfileId":0,"stackLevel":0,"chargingProfilePurpose":"TxDefaultProfile","chargingProfileKind":"Relative","chargingSchedule":{"chargingRateUnit":"W","chargingSchedulePeriod":[{"startPeriod":0,"limit":7000.0}],"minChargingRate":0.0}}}]
[MO] ERROR (Operation.cpp:38): Unsupported operation: processReq() is not implemented
[MO] ERROR (Operation.cpp:42): Unsupported operation: createConf() is not implemented
[MO] Send: [4,"c613ba53-331d-4c08-9068-d50d4d2dcfb7","NotImplemented","",{}]
I assume that the SetChargingProfile service already has the built-in processReq() and createConf() functions.
Do I need to implement these two functions for SetChargingProfile ?
Thanks very much.
Steven

Never mind.
I figured it out. I need to call setSmartChargingXXOutput() after mocpp_initialize.
Thanks very much.

Resolved


After copying views I think internationalize is broken

Monologue internationalization strings yaml file don't get migrated after the views get used. ie. if you have a t('.title') call it will default to 'Title' rather than what I would expect of migrating a internationalized strings locale file.

@msevestre ever seen that? Doesn't ring a bell on my end.


Stack refactor

Currently with this code every time you want different items to be stackable you need to add them to an enum on the shared stack component which is icky and means anything dealing with stack code needs to be aware of it which is especially dirty in construction.
I'm not sure what the ideal system would look like but ideally
a) you'd just be able to mark something as stackable and be able to stack them with the same prototype(?). Whether there is a wrapper entity for the stack or whatever I don't know.
b) be able to have common stack groups for stuff that really needs it (e.g. different types of boards).
c) Have a helper so you can easily spawn either an individual entity or a stack as required. #2675 has helpers for this but it requires the stacktype enum.
Material has some oldcode that could probably do with a lookover at the same time (e.g. it has blunt damage on it).

Make StackType a string, not an enum. Job done.

Ironically, StackType is already of type object, which means it could be an enum, a string or something else

Oh well the entire point is moot then, you don't have to update the enum to make things stack.

Indeed. Although we should probably refactor stack code anyway and use prototypes or something...

Do not check prototype IDs for equality.

By my last comment, I meant having some sort of stack prototype with IDs. Those would hold certain useful information (icon to refer to the stack/material, default entity prototype ID perhaps, max count possible, etc) and stack components would set their stack type to those.

Please please fix this.

Pretty sure this was fixed by Smug. Closing.


(PUP-3724) Use ffi 1.9.6

Update ffi requirement to use at least 1.9.6 because Ruby 2.1.5
requires it.

CLA signed by all contributors.


readinessProbe failing with "separate Port" setup

Bug description
To test the readinessProbe with Istio mTLS enabled I'm following the instructions of
https://istio.io/docs/ops/app-health-check/
However when I move the readinessProbe to a different port by applying "samples/health-check/liveness-http.yaml" the readinessProbe still fails.
I suspect that separate policies/destination rules need to be created for that port that are not mentioned in the documentation. What are those rules and should they be mentioned in the https://istio.io/docs/ops/app-health-check/ documentation?
Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[ x] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Expected behavior
both containers should be in "ready" status
Steps to reproduce the bug
Follow instructions in https://istio.io/docs/ops/app-health-check/ for "Separate port"
Run
kubectl apply -f <(istioctl kube-inject -f samples/health-check/liveness-http.yaml)
Version (include the output of istioctl version --remote and kubectl version)
istioctl version --remote
client version: 1.3.1
control plane version: 1.3.1
kubectl version
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.3", GitCommit:"2d3c76f9091b6bec110a5e63777c332469e0cba2", GitTreeState:"clean", BuildDate:"2019-08-19T11:13:54Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"14+", GitVersion:"v1.14.6-gke.2", GitCommit:"c9de33b5439df6e206d7ba646787c6ace92d737b", GitTreeState:"clean", BuildDate:"2019-09-06T18:30:33Z", GoVersion:"go1.12.9b4", Compiler:"gc", Platform:"linux/amd64"}
How was Istio installed?
Followed instructions in https://istio.io/docs/ops/app-health-check/
Environment where bug was observed (cloud vendor, OS, etc)
google gke
Additionally, please consider attaching a cluster state archive by attaching
the dump file to this issue.

This works for me, tried on 1.3.3 release.
https://github.com/istio/istio/blob/release-1.3/samples/health-check/liveness-http.yaml#L34
We don't have a readiness probe defined in the liveness-http.yaml. Only liveness probe. And that works for me as well.
It's using port 8002, which is not listed on the liveness service definition. so mTLS configuration(authentication policy) is not covered there. You shouldn't need to configure Istio API specifically, at least for this example.

I'm closing but feel free to reopen.

I have this issue too (Istio 1.4.3).
It works for all my microservices, except for frontend pods and the API pod (those that are directly linked to a gateway). That's really strange.


Fix task overview style

Can you rebase this on master? 🙏 I added vertical overview styles - I could go check how the borders look there and disable them in vertical mode if needed

It's rebased now!


Introduce ...rest parameters in the IR.

The magic arguments variable is now banned from the IR. It is replaced by its cleaner ES6 replacement: ...rest parameters.
Rest params are only allowed in exported methods and constructors, and only as last parameter. They are desugared into a loop preceding the normal body of the method, copying the elements from arguments. But until desugaring, we do not have to deal with it.
When eventually targeting ES6, a rest parameter need not be desugared at all.
To preserve backward binary compatibility, a deserialization hack identifies uses of arguments, and replaces it by a rest param. All normal parameters are also removed, and replaced by indexed access into the rest parameter.

CI only for now. A second commit must come.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/1552/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2003/
Test FAILed.

@gzm0 I have an issue with the bootstrapping test. The JS deserializer does not link anymore, because it depends on hashMethodDef which depends on java.security. And even if we ignored the linking error, it would actually blow up at runtime for the first commit, because we would indeed try to patch up a MethodDef using arguments, meaning we need to re-hash it.
I see 3 solutions:

When patching a MethodDef, do not hash it, but put None instead. This is always valid, but a patched MethodDef would always appear as modified for the later stages. Question: since it has to be an exported def, it is not optimized, so what are the things that are invalidated?
Keep the original hash. This is not strictly speaking valid, but it will "work" for all the current usages of these hashes. Indeed, the patching is deterministic, so if the original methods were equivalent, so will the patched methods. But the hash does not correspond to the new MethodDef.
Hack the JS version to produce None, and rehash properly on the JVM version. This is impractical to implement.

My favorite is the second option, despite the relative ugliness.
WDYT?

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/1553/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2004/
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/1554/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2005/
Test PASSed.

How about solution 3, by implementing and using the MessageDigest.getInstance properly:

In the Scala.js implementation, throw a NoSuchAlgorithmException for everything
In the TreeHasher, only hash if we actually have "SHA-1" available

I'm not sure that is less ugly. That means we'd have something that silently doesn't work, and that we would keep forever. At least solution 2 will vanish with the next major release.

That's true... Let's go with 2 then and document this on the MethodDef (as a hack). Just so noone runs into it.
On the other hand, solution 1 would not bring major performance impact: Only the desugaring would be invalidated each time IIUC. Let me check..

It seems that the rest of the pipeline does not even optimize exports based on their hash, just the class version (i.e. file date/time). We can therefore just go with 1.
Please have a look at the pipeline and verify that the export hashes are indeed not used.

You're right. I have changed that part to None. I also verified "experimentally" that reoptimizing the test-suite code base twice in a row did not invalidate anything this way.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/1555/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2010/
Test PASSed.

Added the second commit which emits ...rest params in the compiler.
Review by @gzm0

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/1556/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2011/
Test PASSed.

Updated.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/1559/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2020/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/1560/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2021/
Test PASSed.

LGTM. However, please test this with a real 0.6.0 artifact locally, since there is one codepath we do not test with the two commits now (deserialization of old ParamDef).

OK verified, it works :-)


[ISSUE #4755] Some improvement about branch management

Fixes #4755

Looks good.
+1

Coverage increased (+0.04%) to 48.899% when pulling 8248539a517bff6e713357cc80b7515412711edf on aaron-ai:pr into 6a95661291a7a94e55b6813d5298b58a779aea2b on apache:develop.

Coverage increased (+0.04%) to 48.899% when pulling 8248539a517bff6e713357cc80b7515412711edf on aaron-ai:pr into 6a95661291a7a94e55b6813d5298b58a779aea2b on apache:develop.

Codecov Report

Merging #4756 (8248539) into develop (6a95661) will increase coverage by 0.25%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #4756      +/-   ##
=============================================
+ Coverage      44.60%   44.85%   +0.25%
- Complexity      7544     7582      +38
=============================================
  Files            977      977
  Lines          67777    67777
  Branches        8959     8959
=============================================
+ Hits           30230    30403     +173
+ Misses         33798    33627     -171
+ Partials        3749     3747       -2

Impacted Files
Coverage Δ

...etmq/namesrv/routeinfo/BatchUnRegisterService.java
94.73% <0.00%> (-5.27%)
:arrow_down:

...ava/org/apache/rocketmq/filter/util/BitsArray.java
59.82% <0.00%> (ø)

...e/rocketmq/namesrv/routeinfo/RouteInfoManager.java
65.94% <0.00%> (+0.15%)
:arrow_up:

...mq/client/impl/producer/DefaultMQProducerImpl.java
44.74% <0.00%> (+0.37%)
:arrow_up:

...cketmq/store/ha/autoswitch/AutoSwitchHAClient.java
75.84% <0.00%> (+0.37%)
:arrow_up:

...rocketmq/broker/processor/PopMessageProcessor.java
38.73% <0.00%> (+0.54%)
:arrow_up:

.../org/apache/rocketmq/proxy/config/ProxyConfig.java
49.45% <0.00%> (+0.54%)
:arrow_up:

...ent/impl/consumer/DefaultLitePullConsumerImpl.java
70.48% <0.00%> (+1.01%)
:arrow_up:

.../apache/rocketmq/logging/inner/LoggingBuilder.java
64.71% <0.00%> (+1.26%)
:arrow_up:

...mq/store/ha/autoswitch/AutoSwitchHAConnection.java
74.93% <0.00%> (+1.36%)
:arrow_up:

... and 15 more

Help us with your feedback. Take ten seconds to tell us how you rate us.

LGTM~, Maybe more labels can be attached to RocketMQ, Messaging, Streaming, Eventing, CloudNative, etc.

done.

I was wondering if this "required_approving_review_count" would be more appropriate to set up to 2 and wanted to hear from other people.

@vintagewang
Actually, 2 is practically the number Google takes internally. Here is reason they give

In practice, most code reviews that require more than one approval usually go through a two-step process: gaining an LGTM from a peer engineer, and then seeking approval from appropriate code owner/readability reviewer(s). This allows the two roles to focus on different aspects of the code review and saves review time. The primary reviewer can focus on code correctness and the general validity of the code change; the code owner can focus on whether this change is appropriate for their part of the codebase without having to focus on the details of each line of code. An approver is often looking for something different than a peer reviewer, in other words. After all, someone is trying to check in code to their project/directory. They are more concerned with questions such as: “Will this code be easy or difficult to maintain?” “Does it add to my technical debt?” “Do we have the expertise to maintain it within our team?”

If all three of these types of reviews can be handled by one reviewer, why not just have those types of reviewers handle all code reviews? The short answer is scale. Separating the three roles adds flexibility to the code review process. If you are working with a peer on a new function within a utility library, you can get someone on your team to review the code for code correctness and comprehension. After several rounds (perhaps over several days), your code satisfies your peer reviewer and you get an LGTM. Now, you need only get an owner of the library (and owners often have appropriate readability) to approve the change.

See https://abseil.io/resources/swe-book/html/ch09.html#code_review-id00002


feat(upgrade): upgrade wharfie cli to v0.6.8 (backport #887)

Problem:

Backport of https://github.com/harvester/harvester-installer/pull/887
Solution:

Related Issue:
Test plan:

PTAL @bk201


Centralised PSake build scripts and tasks

Hi PSake-VSTS,
We have an onpremise set of psake build and deploy scripts that we share across all of our repositories,
The set of scripts are in the build script repo.
So 100+ code repos with one set of scripts.
Each 100+ repo is different, but by convention we have webapps, services, consoleapps etc building, deploying and installing across many different machines.
I've just setup of first repo in VSTS and installed the psake task from the marketplace, it's asking me to configure the psake file path which is not possible as there is no psake files at all in our code repos.
Our environment configuration settings are stored in another repo, which our psake looks up for settings, so i'm thinking i need to use a feature inside VSTS to handle settings and configuration.
Any advice our guidance as to how I can configure a similar situation in VSTS with reusing our conventions inside psake and vsts?
We've invested a lot of time into our psake scripts and i'm hoping to not have to abandon them, or use a copy inside each repo, or completely rebuild our conventions inside.
I know i need to completely redesign the deploy side of these scripts, but i'd like to reuse the conventions of the build scripts to abstract and simply this transition.
Any advice or pointers to the concepts that I need to understand inside VSTS would be greatly appreciated.
Regards,
Matt

Hi Matt,
If i understand correctly you have locally 1 repos containing all your psake common build scripts and then 100+ repos containing your code which you build using your common build scripts. Locally when you build, do you run the scripts from your build repos location or do you copy them in your code repo first?
If you need to have both repos on your server to compile unfortunately this will not work in VSTS builds as you can only configure one repo to get source from. One possible solution would be to use git sub module feature to include your build repo as a sub module in your code repo then you would have both repos file on your build server.

Hi Guillaume,
We don't copy them into the repo first. We have the psake build scripts in the PATH of the machine so they can appear at the command line in each repo. Saves on mb's on the disk too.
Submodules is a great idea and i could test that locally as well as on VSTS.
We might have some issues with making the same exact set of scripts work as we run our "build" command on the root of the folder next to the solution, i suppose we could run
cmd> buildscripts\build.bat at root of sln
Cheers! Plenty to think about now!
Matt


Add article on how to contribute samples

As we continue standardizing sample layouts and onboarding Quantum Development Kit samples to the Samples Browser, it will be helpful to document how we write samples and what content needs to be there for working with the Samples Browser. This PR begins that effort by documenting what I look for as I port samples to that format and prepare them for publication at docs.microsoft.com/samples.

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/contributing/code.md
78
57
0
link
:white_check_mark:

articles/contributing/docs.md
83
53
2
link
:white_check_mark:

articles/contributing/index.md
87
67
0
link
:white_check_mark:

articles/contributing/samples.md
85
67
1
link
:white_check_mark:

articles/contributing/toc.yml
100
100
0
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit 6470ec0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/contributing/code.md
:white_check_mark:Succeeded
View

articles/contributing/docs.md
:white_check_mark:Succeeded
View

articles/contributing/index.md
:white_check_mark:Succeeded
View

articles/contributing/samples.md
:white_check_mark:Succeeded
View

articles/contributing/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Thank you so much @cgranade I really appreciate it. I've reviewed and approved but would like @efratshabtai to take a look at it as well, before merging.

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/contributing/code.md
78
57
0
link
:white_check_mark:

articles/contributing/docs.md
83
53
2
link
:white_check_mark:

articles/contributing/index.md
87
67
0
link
:white_check_mark:

articles/contributing/samples.md
85
67
1
link
:white_check_mark:

articles/contributing/toc.yml
100
100
0
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit 5ade662:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/contributing/code.md
:white_check_mark:Succeeded
View

articles/contributing/docs.md
:white_check_mark:Succeeded
View

articles/contributing/index.md
:white_check_mark:Succeeded
View

articles/contributing/samples.md
:white_check_mark:Succeeded
View

articles/contributing/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 262e411:
:clock10: Pending: waiting for processors (10 builds ahead of you)
:warning: Docs Build is busy, currently there are 10 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 262e411:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/contributing/code.md
:white_check_mark:Succeeded
View

articles/contributing/docs.md
:white_check_mark:Succeeded
View

articles/contributing/index.md
:white_check_mark:Succeeded
View

articles/contributing/samples.md
:white_check_mark:Succeeded
View

articles/contributing/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Unbound not resolving certain domains

I am facing a strange problem. I have installed unbound on a raspberry pi 4 following the pi-hole instructions https://docs.pi-hole.net/guides/unbound/
It worked fine for years and without any further problem. Since approx 2 weeks I have troubles to resolve Facebook related domains (sometimes) as well as microsoft.
here a dig example
`dig www.instagram.com @<IP_ADDRESS> -p 5335
;; Warning: Message parser reports malformed message packet.
; <<>> DiG 9.11.5-P4-5.1-Raspbian <<>> www.instagram.com @<IP_ADDRESS> -p 5335
;; global options: +cmd
;; Got answer:
;; ->>HEADER<<- opcode: QUERY, status: NOERROR, id: 31420
;; flags: qr rd ra; QUERY: 1, ANSWER: 2, AUTHORITY: 0, ADDITIONAL: 1
;; OPT PSEUDOSECTION:
; EDNS: version: 0, flags:; udp: 1472
;; QUESTION SECTION:
;www.instagram.com.		IN	A
;; ANSWER SECTION:
www.instagram.com.	3600	IN	CNAME	z-p42-instagram.c10r.facebook.com.
z-p42-instagram.c10r.facebook.com. 27739 RESERVED0 A # 4 1F0D53AE
;; Query time: 30 msec
;; SERVER: <IP_ADDRESS>#5335(<IP_ADDRESS>)
;; WHEN: Thu May 21 10:20:15 CEST 2020
;; MSG SIZE  rcvd: 106
`
yesterday for example I was not able to access teams.microsoft.com when I then switched to google dns it resolved without any further problem.

The output of the dig shows that the A record is from an rrset class that is in the reserved range, this is wrong it should be IN.  Unbound likely echoes that from the upstream server message.  It is therefore likely that if you enable verbosity 4 and look at unbound's logs of the resolution, that unbound logs receiving that awkward record from the upstream server when it asks for that.
In any case if you switch on verbosity 4 you can see that happens, and also the individual, dig-style messages and output in the logs when answers are received.  Then you can skip through those logs and look at what unbound received when it asked that question.  This can help uncover what happened.
Best regards, Wouter

Hi,
here are the results from Syslog Unfortunately the logging did not work well on the initial test. FYI the first lookup was successful it was the second one which had the problems.
Syslog.txt

So from that syslog.txt, you receive, twice records with CLASS value 0 rrsets in them.  Also the root trust anchor fails to prime with signature verification failure.  That would make all queries SERVFAIL with DNSSEC failure.  Also there are timeouts, where no response is received.
From <whatsapp.com.> <IP_ADDRESS>
May 25 08:45:46 pihole unbound[27513]: web.whatsapp.com.#01130736#011CLASS0#011C
NAME#011mmx-ds.cdn.whatsapp.net.
And later from <cdn.whatsapp.net.> <IP_ADDRESS>
May 25 08:45:46 pihole unbound[27513]: mmx-ds.cdn.whatsapp.net.#01130780#011CLASS0#011A#01<IP_ADDRESS>
In that above #011 is a tab, syslog puts that instead.
Unbound is asking for the CLASS IN records, but gets replies in with CLASS0 (a value of zero in the class field, which is not supposed to happen in ordinary query answer packets).
Additionally, I see May 25 08:45:45 pihole unbound[27513]: [1590389145] unbound[27513:0] info: failed to prime trust anchor -- DNSKEY rrset is not secure . DNSKEY IN
with before that the DNSSEC fail reason: verify: signature mismatch
And you have timeouts in response to queries.  So the upstream servers are not responding to you, or only intermittently.
Unbound fails to look up, but it looks like everything is wrong.  Not getting answers.  Answers with simply some sort of format error, class 0.  And wrong answers, in the DNSSEC sense, the data is not correct.
If I try to fetch data from whatsapp, I do not receive CLASS0 or timeouts from them.  You could try to perform a dig manually for the packets that seem to have responses that unbound cannot work with, it looks like that would then also give the wrong data to that dig output manually.  Unbound cannot really create the correct answer for you if it only gets answers with the wrong contents.

;; Query time: 1 msec
;; SERVER: <IP_ADDRESS>#53(<IP_ADDRESS>)
;; WHEN: Mon May 25 09:47:53 CEST 2020
;; MSG SIZE  rcvd: 4157`
It is a bit bizarre to me. for your answer, what do you suggest I do ?

I guess it worked before for you because you enabled TLS for unbound upstream connection and that stops the DNS packets from getting changed, simply because it uses a different transport mechanism.
But I do not know what causes your answers to get mangled.  Does not really look like it is a wrong answer, I get the same CNAME answer, but not malformed, here.  If you are on a network where you need to sign up, sometimes they do this to stop people that have not paid from accessing the network, and the intent may be that 'everything works' after the sign up, but perhaps some buggy software is not so good, eg. does not send DNSSEC signatures is what I have seen before.  Not the malformed packets that you have.

Could you tell me how to enable TLS for upstream? I would like to try to see what happens if I enable.
I am working on my "normal" home network. I am wondering if there might be something wrong with my router?

TLS upstream with tls-upstream: yes (for all upstreams) or forward-tls-upstream: yes (for the servers in that forward statement).  There are more servers that support TLS if you search for that.
The packets are wrong, I do not know what device could have done it.

I tried to add the tls-upstream: yes or the alternative to unbound.conf however I get an error when starting
`May 25 11:53:31 pihole systemd[1]: unbound.service: Failed with result 'exit-code'.
-- Subject: Unit failed
-- Defined-By: systemd
-- Support: https://www.debian.org/support
-- The unit unbound.service has entered the 'failed' state with result 'exit-code'.
May 25 11:53:31 pihole systemd[1]: Failed to start Unbound DNS server.
-- Subject: A start job for unit unbound.service has failed
-- Defined-By: systemd
-- Support: https://www.debian.org/support
-- A start job for unit unbound.service has finished with a failure.
-- The job identifier is 3539 and the job result is failed.
May 25 11:53:31 pihole systemd[1]: unbound-resolvconf.service: Start request repeated too quickly.
May 25 11:53:31 pihole systemd[1]: unbound-resolvconf.service: Failed with result 'exit-code'.
-- Subject: Unit failed
-- Defined-By: systemd
-- Support: https://www.debian.org/support
-- The unit unbound-resolvconf.service has entered the 'failed' state with result 'exit-code'.
May 25 11:53:31 pihole systemd[1]: Failed to start Unbound DNS server via resolvconf.
-- Subject: A start job for unit unbound-resolvconf.service has failed
-- Defined-By: systemd
-- Support: https://www.debian.org/support
-- A start job for unit unbound-resolvconf.service has finished with a failure.
-- The job identifier is 3600 and the job result is failed.
May 25 11:53:34 pihole kernel: Voltage normalised (0x00000000)
May 25 11:53:36 pihole sudo[1359]:      xxx : TTY=pts/0 ; PWD=/etc/unbound ; USER=root ; COMMAND=/bin/journalctl -xe
May 25 11:53:36 pihole sudo[1359]: pam_unix(sudo:session): session opened for user root by xxx(uid=0)`

Try unbound-checkconf for syntax errors.  Or look at the syslog to see if it printed the syntax errors in the config file.  If your unbound version is very old, the TLS upstream is not there, upgrade or use a package like getdns or stubby.

I tried to downgrade the firmware of my router. For the moment it seems like it is working. I will monitor and post back


Cats plugin not working with wrench

I been trying to get Cats plugin not working with wrench  to work i used steam and the website version and its till showing
Python Traceback ( Most recent call last)
I'm using 0.19.0 cats and blender latest version still having issues if someone knows how to fix please help and thanks you

#379

Thank you  for the information, I still haven't tried the new one I went back to an older version and it worked


Added RodriguesVec to the rotation parameterizations

Rodrigues vector parametrization have come up (OpenCV, Ceres, etc) so I thought I'd add them before I forgot about them.
Other changes:

Migrated test framework to the v0.5 style
Changed rotate() implementations to parametrization specific versions (mostly)

Looks good, I'd suggest cutting down on exports and making some exported names more explicit but other than that I'm happy.

Changes Unknown when pulling 0ee8680b178c5d8a31acd408d8522a1ff0bd0bdf on RodriguesVec into ** on master**.

Changes Unknown when pulling f4df372661669fe349d3138ce1b00de90ca109e5 on RodriguesVec into ** on master**.


select could accept both column number[s] and dimlabel[s]

Now that dimlabels(x::IndexedTable) exists, it would be helpful to have select become dimlable aware.
hitemps = Table(Columns(city = [fill("New York",3); fill("Boston",3)],
                        date = repmat(Date(2016,7,6):Date(2016,7,8), 2)),
                [91,89,91,95,83,76])

# suggesting

select(hitemps, 1)    == select(hitemps, :city)
select(hitemps, 1, 2) == select(hitemps, :city, :date)
select(hitemps, 2, 1) == select(hitemps, :date, :city)

it alreay works!


make vsphere pool variable optional

Some people might not have the license to enable DRS that is required to create resource pools. So, make pool an optional variable.

Thanks for this pull request. @chrislovecnm, can you review this PR so that we can get it merged?

There are significant changes coming in after the pr gets approved by terraform, and they release.
@prudhvitella We are moving towards using the builtin terraform vsphere provider. If you want assistance using the new terraform provider please let me know.
@stevendborrelli you want me to push a branch with the new changes?  My PR with terraform has not been merged....  I would not recommend making updates at this time.  Your call.

For reference: https://github.com/hashicorp/terraform/pull/3867

Given that PR with terraform will change lot of things, lets close this PR. I don't see much value in it.

@prudhvitella I just want to say thanks for submitting this, we really appreciate contributions!


Citra VS JIT

Minor changes that I use for Citra's shader JIT... Merge plz

Please pick a more suitable PR title in the future.


Combine object and connection ID in keyspace

This should save us some bits in the keyspace.

Yey! I'm contributing...


How to set scmProvider filter based on repository group or path?

We would like to have different argoCD instances to handle different set of repos using scmProvider. Currently the repository filter only works on repo name, is there any way to make it work on the full path or have another filter to match the repo group?
e.g. how can we set one scmProvider to sync https://gitlab/group_1/xxxx projects and another to sync https://gitlab/group_2/xxxx projects?

Missed the group param :)


[Restore YouTube Username] it doesn't work

Some accounts work fine, some don't.

Thanks for reporting. Fixed now.

@wiful10 Could you please confirm the issue is resolved or not?

Solved. Thank you.


[Bug][rubysrc2cpg] do keyword issue

Describe the bug
getting parser error -  mismatched input 'do' expecting <EOF> at line 1
To Reproduce
RSpec.configure do |config|
  config.before(:suite) do
    DatabaseCleaner.clean_with :truncation, except: %w(ar_internal_metadata)
  end
end

@xavierpinho i'm not confident about whether this issue is related to do keyword or not, but this fragment of code throwing error.

@xavierpinho i'm not confident about whether this issue is related to do keyword or not, but this fragment of code throwing error.

Indeed, the issue is still #3040, so I'll be closing this one. But thanks for the sample either way!


Version pro: I can't find the CAPTCHA...

I attempted to login fastapi-admin on pro version, but I found there was no CAPTCHA showing on the login page.

Was this a bug? Or I missed something?

Fixed it


when i npm i, there is such an error, can you help me

I want to npm i in this project.But there is such an error.It looks like I don’t have such a local package.There is no way to directly run a clone project directly.
npm i
npm ERR! code ENOLOCAL
npm ERR! Could not install from "../tiptap-collab-server" as it does not contain a package.json file.

The same problem occurs in tiptap-extension-collaboration. Can I install this package directly once?

Did your clone the repo as explained in the linked article : https://www.naept.com/en/blog/easy-collaborative-editor-with-tiptap-and-prosemirror/ ?
package.json is present in the project directory.


Minor - BuyAndSellSignalsToChart example setPaint typo

within BuyAndSellSignalsToChart.java  buyMarker.setPaint is invoked multiple times where as sellMarker.setPaint is never called. lines 86 and 92. pull request to follow this ticket. Cheers

Whoops! Thank you. :)


Add luandy64 to Contributor list

Hi  @luandy64, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added social share  to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.

The tutorial was very clear, but maybe that's because I have used git and GitHub before. I was just getting ready to find some projects I really want to dig into and commit a change to, so I was using this one as a way to brush up on how people do things.
A pain point I can anticipate is that documentation can be hard if you're not used to reading it. Actually, it's hard even when you know what you doing, I think. Sometimes I'll read something, not understand it, go to stack overflow, read it again, and suddenly it's clear. So a good next step for this project might be a "Next Steps" section for a new person, where you walk them through common ways to familiarize themselves with a new project; how to get help when they are stuck (and how to use things like slack, gitter, etc).
Overall, great job with the tutorial! Maybe we can actually work together one day on a project!

I’d urge you to continue contributing to other open source projects.
We also have a slack team to help/mentor people trying to contribute to open source projects.
You can join the team by going to https://goo.gl/PM1Jdh


Handle doctrine resolved interfaces

Im using doctrine's resolve interfaces (resolve target entity) ability  to provide abstraction in my entities. The problem when I create a user this way:
It gives me an error when I do this:
Question\MyQuestionImplementaion\Question:
    question{1..3}:
        title: <realText(100)>
        author: 1
        text: <realText(2000)>

Question association looks like this:
@manyToMany(targetEntity="User\MyInterface\User")
Interface User\MyInterface\User maps to User\MyImplementation\User
error I get is:
No ObjectManager for class User\MyInterface\User
Seems like issue is in h4cc\AliceFixturesBundle\ORM\Doctrine.php
How can I handle resolved interfaces?

Hi there,
i have never heard of the configuration you did with @manyToMany(targetEntity="User\MyInterface\User"), because there does not seem to be a concrete class defined as targetEntity.
All i can do is ask the doctrine managerRegsitry for the manager of a given class:
https://github.com/h4cc/AliceFixturesBundle/blob/master/ORM/Doctrine.php#L118-L122
To me it sounds more like a problem with doctrine, because doctrine does not seem to know about your User\MyInterface namespace, so it can not provide a manager for that.

Please provide more information, if this issue still persists.


Don't add credHelper if there's no GCP credentials

Please provide a description of this PR:
As we know, gcloud auth configure-docker adds the Docker credHelper
entry to Docker's configuration file, or creates the file
if it doesn't exist.
But if there's no GCP credential file(defined by env
GOOGLE_APPLICATION_CREDENTIALS), which is often met in internal CI,
it would lead to a failure when doing
the normal docker operation:
"
ERROR: error getting credentials - err: exit status 1, out:
`You do not currently have an active account selected.
See https://cloud.google.com/sdk/docs/authorizing for more information
"
So just ignore the credHelper adding operation if there's no
GCP credentials.
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[x] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[ ] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

Hi @TrevorTaoARM. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

@TrevorTaoARM: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
6f544de89d68009f1b1ba3b74cf87d5a50d9db46
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


added c++ basic topics

@UjjwalSharma01 hey , I am a beginner in open source right now, could you please elaborate on why the PR was marked as spam when it was written in the code itself to add more C++ topics .
Thank you !


Prediction

-How can i make a prediction for an image?
-How can i train the model with my own data?
Thanks.

Hi!
The simplest thing to do is to arrange your data as a MVTec dataset.
Here is an example of how it's done: https://github.com/rvorias/ind_knn_ad#custom-datasets
You can also look at the datasets that the repo automatically downloads.
If you want to predict something, I've also added a little snippet to the readme:
from indad.model import SPADE

model = SPADE(k=42, backbone="hypernet")

# feed healthy dataset
model.fit(...)

# get predictions
img_lvl_anom_score, pxl_lvl_anom_score = model.predict(...)

The first ... are a dataset. Like I've said before, it's easy when you use the MVTec dataset structure
The second ... is a sample, so a 4-d torch tensor of size [1, 3, 224, 224].
Hope that helps!

I get an output image but it doesn't give a result similar to the examples :(
**model.fit(train_ds)
PATH = "/home/Desktop/DL/AnomalyDetection/ind_knn_ad-master/model/"+str(cls)+".pth"
torch.save(model.state_dict(), PATH)
print("One image testing ...")
pil_img = Image.open("/home/Desktop/DL/AnomalyDetection/ind_knn_ad-master/datasets/carpet/test/cut/009.png")
pil_img = pil_img.resize((224, 224))
pil_to_tensor = transforms.ToTensor()(pil_img).unsqueeze_(0)
img_lvl_anom_score, pxl_lvl_anom_score = model.predict(pil_to_tensor)
trans = torchvision.transforms.ToPILImage()
out = trans(pxl_lvl_anom_score)
#print(pxl_lvl_anom_score)
out.save("asd.png")**

You are missing some normalization.
Try this:
out = trans(pxl_lvl_anom_score[0]/torch.max(pxl_lvl_anom_score))

Your image will be in greyscale though.
If you want to visualize it in a notebook, you can also work with matplotlib:
import matplotlib.pyplot as plt

plt.imshow(pxl_lvl_anom_score[0])

You can also work with an overlay effect, e.g.:
def save_image(sample, fmap):
    timestamp = datetime.now().strftime("%d_%m_%Y_%H_%M_%S")

    plt.imshow(sample.permute([1,2,0]))
    plt.imshow(fmap, cmap='jet', alpha=0.5)
    plt.gca().set_axis_off()
    plt.subplots_adjust(top = 1, bottom = 0, right = 1, left = 0,
                hspace = 0, wspace = 0)
    plt.margins(0,0)
    plt.gca().xaxis.set_major_locator(plt.NullLocator())
    plt.gca().yaxis.set_major_locator(plt.NullLocator())
    plt.savefig(f"results/image_{timestamp}.png", bbox_inches = 'tight',
        pad_inches = 0)

Gonna close this issue soon as it has nothing to do with the repo.


attr_accessible in token.rb

Hello all,
I'm not sure if this is specific to my use, but I found I had to add attr_accessible :user_id to the model.class_eval block in oauth/consumers/token.rb
model.class_eval do
  attr_accessible :user_id
  validates_presence_of :user, :token
end

Otherwise, the token were being saved without user ids.

a better solution from here : http://unhandledexpression.com/2011/06/28/rails-and-oauth-plugin-part-2-the-consumer/#comment-205
You should add this to your ApplicationController:
def current_user=(user)
current_user = user
end

I've tried that solution
def current_user=(user)
current_user = user
end
and it works well when I am in Rails 3.0.7 but am still getting user_id=nil in 3.1. Anyone else encountered this?

It worked for me in 3.1. I had to delete some part of the code which was useless for me. I can't help more...

octopusinc, i've found some other craziness specifically to how the gem behaves in 3.1
i've outlined it here
https://github.com/pelle/oauth-plugin/issues/96
No solutions in there, but thought you might find it interesting. Basically, as far as I can tell, the user_id is passed along to the gem all the way till it tries to save the token. funny thing is, if I try to save it again one line later in the same method, it works.
completely baffles me.

Thanks for sharing your research @yutai.  I'm with you, baffling...

I had this in the ApplicationController (I don't know why I wrote that here):
def logged_in?
true
end
That wasn't working until I removed it ... and maybe another similar part.
@octopusinc : the solution I wrote is better because I don't have to change the code in the Gem. As I will put my app in production later, I can't change any line of the Gems...

In the ConsumerToken model I commented out:
#before_create :create_user

because of something I read over here (https://github.com/rails/rails/issues/1594) & IT WORKED.  @pixeltrix says that the belongs_to association triggers a before_save too.  Is this method nuking the user variable by creating a new one?


GA4 + UA dual setup: Two libraries/container loaded

Universal Analytics (UA) will be deprecated in summer of 2023, so while switching my setup from UA to GA4 I am having a dual implementation of both GA properties in a single GTM container. I have two tags for each pageview/event, one for UA and one for GA4.
The problem right now is that the GTM container loads two container for each of the tags.

gtm.js for UA
gtag.js for GA4

This really slows down page speed score. Not sure if this is an issue with the module or with the Google Tag Manager. How to handle this situation because switching from setup to another could take some weeks?

I did some research and it turns out that vue-gtag plugin supports UA (gtm events) and GA4 events simultaneously.
Moreover it's not mentioned in the plugin docs but the v1 of this plugin supports Vue 2 while v2 doesn't - Vue 3 only.


Injection Failing on M1 Macbook

The following error occurs when injecting on a M1 MacBook Pro running big Sur:

The injector works fine on other programs which would make me believe that it is a problem with the cheat. I am sorry for causing any problems, I don't have much knowledge with c++.
Any solution would be greatly appreciated.

1- First of all, we don't have M1 MacBook Pro. It's probably about compilation arguments. The one which in releases, compiled for x86_64. You should compile for your M1 system.
2- Compile OSXInj & Bootstrap for your M1. They have diffrent arch.
(Sorry for my english btw)
(If OSXInj works for another dylibs, make 1. one)
Btw, i'm gonna make an injector from scratch.

Yah I did, what is weird is that the m1 macs have Rosetta, a program that lets them emulate x86_64 programs, I’ll be excited to try in the future with your new injector to see if it works any better.

So, is it working right now? I mean can you see the menu?
Key is insert or you can just use the touchbar button...
I'm asking because i'm gonne close the issue if it is worked...

You should download the project and compile for your M1 for smoother experience and fix the error.

I did that, it still gives the same error, very weird.

osxinj.zip
tell me if this works,
compiled on my machine

I did that, it still gives the same error, very weird.

can you also try recompile osxinj & bootstrap for M1?
Is there any errors while compiling? At least it should give something...
And i'm going to try your dylib right now...

The compiled one for M1 didn't worked for me (and it's very normal). But it should work for you. As i said before (you can search on google) they say it's about osxinj. I didn't looked to much on it so idk why it happens but, yeah. It's not about our cheat. Tried on (2020, 2019)

Can you see the ButOSX button on touchbar after you inject?

Ahh maybe the injector is the problem, I am unable to compile as arm64 though, I have to compile as x86_64, feel free to close the issue, I will just wait till you guys write one.

No problem. It'll be very hard to write an injector to the computer which we don't have but we'll gonna try.
Closing...

Hello again. Can you test with compiling https://github.com/Lyceion/CroXinj and using it with? (Plase compile both ButOSX and CroXinj for m1 and test like that. If it works, it'll be awesome.)

Thank you, I will tonight or tomorrow:)

Sadly it still doesn't compile for arm. I would be happy to talk further on another platform so we don't have to converse thourgh GitHub issues. Feel free to reach out to me on discord: jakep#8028 :)


AWQ inference is 50% slower than GPTQ

Hey Casper,
System:  Ubuntu 22.04 RTX3090 CUDA 118 Python 3.10 AutoAWQ 0.1.0
I'm only seeing 50% of the performance of a GPTQ model in ExLlamaV2 which is surprising.
Also the in device memory use is 15% higher for the same model, AWQ loaded in AutoAWQ  vs GPTQ loaded in ExLlamaV2.  Here's a few stats with a max_seq_len of 4096.
model                                   max seq      memory     tokens/s
TheBloke_Llama-2-7B-chat-AWQ               4096      7560MB         60.7
TheBloke_Llama-2-7B-chat-GPTQ              4096      6622MB        139.9

TheBloke_Llama-2-13B-chat-AWQ              4096     12532MB         38.7
TheBloke_Llama-2-13B-chat-GPTQ             4096     10828MB         84.9

TheBloke_Phind-CodeLlama-34B-v2-AWQ        4096     21162MB         21.5 (would not complete)
TheBloke_Phind-CodeLlama-34B-v2-GPTQ       4096     18370MB         40.2

How did you measure performance?

Hey @casper-hansen for these metrics I took a shortcut and ran them using text-generation-webui.
If I can find the time today I'll make a second attempt using a plain python script.

You should try the benchmark script that I included in examples. GEMV achieves 90% of ExLlamaV2 speed and GEMM achieves 70%.
Also do note that AutoAWQ is a more broad framework. We aim for high speed but make compromises that does ExLlama does not.

I cobbled together a version of the script to gauge performance and you're absolutely right, the implementation in text-generation-webui is slowing things down.
I can confirm that for the first run AutoAWQ approaches 90% of ExLlamaV2 performance at least for TheBloke/Llama-2-7b-Chat-AWQ.
What interesting is that the inference performance goes down for every run.  This was run against the standard 0.1.0 version so it errored out after the last run shown.

(textgen) alansrobotlab@goliath:~/Documents/brayden/langchain$ python testawq.py
Fetching 13 files: 100%|██████████████████████████████████████████████████████████████████| 13/13 [00:00<00:00, 226248.76it/s]
Replacing layers...: 100%|████████████████████████████████████████████████████████████████████| 32/32 [00:01<00:00, 20.25it/s]
Tokens per second:<PHONE_NUMBER>8403186
Tokens per second:<PHONE_NUMBER>8754163
Tokens per second:<PHONE_NUMBER>9892934
Tokens per second:   95.48121527004926
Tokens per second:   99.72065805469065
Tokens per second:   98.25381852653379
Tokens per second:   94.36816405132929
Tokens per second:   90.7243856285012
Tokens per second:   88.71160320470709

from awq import AutoAWQForCausalLM
from transformers import AutoTokenizer
import time

#model_name_or_path = "TheBloke/CodeLlama-34B-AWQ"
model_name_or_path = "TheBloke/Llama-2-7b-Chat-AWQ"
#model_name_or_path = "casperhansen/vicuna-7b-v1.5-awq"

# Load model
model = AutoAWQForCausalLM.from_quantized(
    model_name_or_path,
    #"awq_model_w4_g128.pt",
    fuse_layers=True,
    trust_remote_code=False,
    safetensors=True,
    max_new_tokens=4096,
)
tokenizer = AutoTokenizer.from_pretrained(model_name_or_path, trust_remote_code=False)

tokens = tokenizer(
    "# What are all fo the different quarks in nuclear physics and when and where were they discovered?\n\n", return_tensors=>
).to("cuda")

# Generate output
for _ in range(10):
    start = time.time()
    generation_output = model.generate(
        **tokens,
        do_sample=True,
        temperature=0.2,
        top_p=0.95,
        top_k=0,
        max_new_tokens=512,
    )
    elapsed = time.time() - start
    count = len(generation_output[0])
    #print("Elapsed:  ",elapsed)
    #print("Tokens:  ",count)
    print("Tokens per second:  ",(count/elapsed))
    #print(tokenizer.decode(generation_output[0], skip_special_tokens=True))


[Snyk] Upgrade com.fasterxml.jackson.dataformat:jackson-dataformat-yaml from 2.14.1 to 2.14.2

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade com.fasterxml.jackson.dataformat:jackson-dataformat-yaml from 2.14.1 to 2.14.2.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released a month ago, on 2023-01-29.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Will probably be reopened with the correct versions now that Swagger is less used in Hilla


listing applications in graph is a pageable operation

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.
[ ] 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.
[ ] 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 ] Validation errors from the Linter extension for VS Code have all been fixed for this spec. (Note: for large, previously checked in specs, there will likely be many errors shown. Please contact our team so we can set a timeframe for fixing these errors if your PR is not going to address them).
[ x] The spec follows the patterns described in the Swagger good patterns document unless the service API makes this impossible.

 https://github.com/Azure/azure-sdk-for-python/pull/808

 https://github.com/Azure/azure-sdk-for-node/pull/1916

 https://github.com/Azure/azure-sdk-for-ruby/pull/481


Add reference information for stable metrics

This is a Feature Request
Following on issue https://github.com/kubernetes/website/issues/24915, create reference information for
the list of stable metrics.
The list of stable metrics is mentioned in PR: https://github.com/kubernetes/website/pull/27421.
Investigate creating reference information in the docs for https://github.com/kubernetes/kubernetes/blob/master/test/instrumentation/testdata/stable-metrics-list.yaml.

This one can be closed. Pls feel free to reopen if the issue still persists.
/close

Also see issue #32302


[Do Not Merge] increasing timeout value for rgw 300 sec -> 1800 sec

Signed-off-by: Sachitanand Shelake<EMAIL_ADDRESS>Problem Statement

Problem statement

Design

For Bug, Describe the fix here.
For Feature, Post the link for design

Coding
Checklist for Author

[ ] Coding conventions are followed and code is consistent

Testing
Checklist for Author

[ ] Unit and System Tests are added
[ ] Test Cases cover Happy Path, Non-Happy Path and Scalability
[ ] Testing was performed with RPM

Impact Analysis
Checklist for Author/Reviewer/GateKeeper

[ ] Interface change (if any) are documented
[ ] Side effects on other features (deployment/upgrade)
[ ] Dependencies on other component(s)

Review Checklist
Checklist for Author

[ ] JIRA number/GitHub Issue added to PR
[ ] PR is self reviewed
[ ] Jira and state/status is updated and JIRA is updated with PR link
[ ] Check if the description is clear and explained

Documentation
Checklist for Author

[ ] Changes done to WIKI / Confluence page / Quick Start Guide

exploration is done from motr side hence this patch is no longer required.. hence closing this PR


Auto fetch science.json\article from Wikipedia

science.json\article is a list of some articles from Wikipedia.
You can fetch it's automaticaly from Wiki Data by group, for example, in Elizabeth generating session.
Is it idea?

@yn-coder Yeah, it's good idea but our priority is data generating. Articles are not worth such a complexity of the library.

:(

@yn-coder I'm sorry, but it requires too much effort.