Can't print when connected to printer "MHT-P80A", "FK-POS80-AT", on android

MHT-P80A

FK-POS80-AT

Sir, How to fix this

For MHT-P80A I didn't see any error logs.
But for FK-POS80-AT I notice that there is an error: "can't not found writable characteristic". When printing with a Bluetooth printer,the printer device should have a writable characteristic. This error is throw when the plug-in couldn't find a writable characteristic.
I didn't have those two printers so I couldn't test for you. You may install "nRF connect" App(or other Bluetooth kits) to connect to printer and try to find the problems.

https://github.com/NordicSemiconductor/Android-nRF-Connect

I noticed that your package is similar to this one. Both of my printers are
working properly with this package. However, since this package uses
connect function but not await as well as your package, I just wanted to
fix the bug on your package.
Vào 18:13, Th 6, 17 thg 2, 2023 xiaozhizhong @.***> đã
viết:

https://github.com/NordicSemiconductor/Android-nRF-Connect
—
Reply to this email directly, view it on GitHub
https://github.com/xiaozhizhong/flutter_simple_bluetooth_printer/issues/1#issuecomment-1434492646,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY5GZ2BU47YUQAIFK2NV7VDWX5MN7ANCNFSM6AAAAAAU7INBE4
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

Did you forget to attach the package link? I didn't see the package link or name in your reply

Here Sir,
https://pub.dev/packages/flutter_pos_printer_platform
Vào 18:39, Th 6, 17 thg 2, 2023 xiaozhizhong @.***> đã
viết:

Did you forget to attach the package link? I didn't see the package link
or name in your reply
—
Reply to this email directly, view it on GitHub
https://github.com/xiaozhizhong/flutter_simple_bluetooth_printer/issues/1#issuecomment-1434520502,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY5GZ2CKEJL6WD4GVY6JZOTWX5PQRANCNFSM6AAAAAAU7INBE4
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

I need this package's "https://pub.dev/packages/flutter_pos_printer_platform"
connect function to return true when BLE connection is successful but it
doesn't, it always return false.  I need it to wait until
setCharacteristic() is done before return boolen
Vào 18:08, Th 6, 17 thg 2, 2023 xiaozhizhong @.***> đã
viết:

For MHT-P80A I didn't see any error logs.
But for FK-POS80-AT I notice that there is an error: "can't not found
writable characteristic". When printing with a Bluetooth printer,the
printer device should have a writable characteristic. This error is throw
when the plug-in couldn't find a writable characteristic.
I didn't have those two printers so I couldn't test for you. You may
install "nRF connect" App(or other Bluetooth kits) to connect to printer
and try to find the problems.
[image: image]
https://user-images.githubusercontent.com/18508389/219633238-05f1e670-0b06-4261-ab4d-53e00fe1b213.jpeg
—
Reply to this email directly, view it on GitHub
https://github.com/xiaozhizhong/flutter_simple_bluetooth_printer/issues/1#issuecomment-1434486330,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY5GZ2HSU2U65B3DDW2JCXTWX5L3VANCNFSM6AAAAAAU7INBE4
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

@chuvanhoang888 I've made some enhancements on finding writable characteristic and create the branch v0.1.5, could you please give it a try by replacing following codes in your pubspec.yaml?
  flutter_simple_bluetooth_printer:
    git:
      url: https://github.com/xiaozhizhong/flutter_simple_bluetooth_printer.git
      ref: v0.1.5

Changes:

Add writable characteristic condition BluetoothGattCharacteristic.PROPERTY_SIGNED_WRITE;
If no writable characteristic found, use characteristic with CLIENT_CHARACTERISTIC_CONFIG descriptor (Same as package flutter_pos_printer_platform);
Add 'characteristicUuid' param to writeText & writeRawData. (If 'can't not found writable characteristic' error still occurs, you can specific the characteristicUuid to match the characteristic to write )

Thanks for your great support for the package.  I will try the code again
on Monday and get back to you
Vào 23:33, Th 6, 17 thg 2, 2023 xiaozhizhong @.***> đã
viết:

@chuvanhoang888 https://github.com/chuvanhoang888 I've made some
enhancements on finding writable characteristic and create the branch
v0.1.5, could you please give it a try by replacing following codes in your
pubspec.yaml?
flutter_simple_bluetooth_printer:
git:
url: https://github.com/xiaozhizhong/flutter_simple_bluetooth_printer.git
ref: v0.1.5
Changes:

Add writable characteristic condition
BluetoothGattCharacteristic.PROPERTY_SIGNED_WRITE;
If no writable characteristic found, use characteristic with
CLIENT_CHARACTERISTIC_CONFIG descriptor (Same as package
flutter_pos_printer_platform);
Add 'characteristicUuid' param to writeText & writeRawData. (If
'can't not found writable characteristic' error still occurs, you can
specific the characteristicUuid to match the characteristic to write )

—
Reply to this email directly, view it on GitHub
https://github.com/xiaozhizhong/flutter_simple_bluetooth_printer/issues/1#issuecomment-1434886450,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY5GZ2D4OK5XXMF65T5NA3LWX6R5FANCNFSM6AAAAAAU7INBE4
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

For MHT-P80A I didn't see any error logs. But for FK-POS80-AT I notice that there is an error: "can't not found writable characteristic". When printing with a Bluetooth printer,the printer device should have a writable characteristic. This error is throw when the plug-in couldn't find a writable characteristic. I didn't have those two printers so I couldn't test for you. You may install "nRF connect" App(or other Bluetooth kits) to connect to printer and try to find the problems.

@chuvanhoang888 I've made some enhancements on finding writable characteristic and create the branch v0.1.5, could you please give it a try by replacing following codes in your pubspec.yaml?
  flutter_simple_bluetooth_printer:
    git:
      url: https://github.com/xiaozhizhong/flutter_simple_bluetooth_printer.git
      ref: v0.1.5

Changes:

Add writable characteristic condition BluetoothGattCharacteristic.PROPERTY_SIGNED_WRITE;
If no writable characteristic found, use characteristic with CLIENT_CHARACTERISTIC_CONFIG descriptor (Same as package flutter_pos_printer_platform);
Add 'characteristicUuid' param to writeText & writeRawData. (If 'can't not found writable characteristic' error still occurs, you can specific the characteristicUuid to match the characteristic to write )

Hello Sir,
Have a good day
I have tested v0.1.5 version on FK-POS80-AT printer, printing was done successfully, however out of 4 print times, 1 time printing failed.

Hello Sir,
Have a good day

I have tested v0.1.5 version on FK-POS80-AT printer, printing was done
successfully, however out of 4 print times, 1 time printing failed.
[image: image.png]
For the MHT-P80A printer, the printer is still not printing even though
the writeText function returns true
However, I see 1 Uuid feature: "BEF8D6C9-9C21-4C9E-B632-BD58C1009F9F"
can make both machine "FK-POS80-AT" and "MHT-P80A" successful.
[image: image.png]
[image: image.png]
Have you added this "characteristicUuid" property for ios. On ios,
printer "MHT-P80A" works normally and printer "FK-POS80-AT" gets error like
on android

Vào Th 6, 17 thg 2, 2023 vào lúc 23:33 xiaozhizhong <
@.***> đã viết:

@chuvanhoang888 https://github.com/chuvanhoang888 I've made some
enhancements on finding writable characteristic and create the branch
v0.1.5, could you please give it a try by replacing following codes in your
pubspec.yaml?
flutter_simple_bluetooth_printer:
git:
url: https://github.com/xiaozhizhong/flutter_simple_bluetooth_printer.git
ref: v0.1.5
Changes:

Add writable characteristic condition
BluetoothGattCharacteristic.PROPERTY_SIGNED_WRITE;
If no writable characteristic found, use characteristic with
CLIENT_CHARACTERISTIC_CONFIG descriptor (Same as package
flutter_pos_printer_platform);
Add 'characteristicUuid' param to writeText & writeRawData. (If
'can't not found writable characteristic' error still occurs, you can
specific the characteristicUuid to match the characteristic to write )

—
Reply to this email directly, view it on GitHub
https://github.com/xiaozhizhong/flutter_simple_bluetooth_printer/issues/1#issuecomment-1434886450,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY5GZ2D4OK5XXMF65T5NA3LWX6R5FANCNFSM6AAAAAAU7INBE4
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

@chuvanhoang888

For "I have tested v0.1.5 version on FK-POS80-AT printer, printing was done
successfully, however out of 4 print times, 1 time printing failed", I noticed that your output error shows 'status 133', which might be a bug of Android. Please refer to https://github.com/dariuszseweryn/RxAndroidBle/wiki/FAQ:-Cannot-connect and https://issuetracker.google.com/issues/37121040. Maybe you can try add durations to each printing, or not calling disconnect after each printing but after all printing is done.
For "For the MHT-P80A printer, the printer is still not printing even though
the writeText function returns true", Some devices might has more than 1 writable characteristic, but not all those characteristic can handle your printing commands. This plugin will find out all writable characteristics but only use the first one. But even we use a wrong characteristic, the printing callback result might be still success. In this case, you should specific the characteristicUuid to tell this plugin which characteristic should be used.
For "Have you added this "characteristicUuid" property for ios. On ios,
printer "MHT-P80A" works normally and printer "FK-POS80-AT" gets error like
on android". Not yet, I'm a little busy now and it's planned to the day after tomorrow.

Thanks for your feedback.
Vào 12:04, Th 2, 20 thg 2, 2023 xiaozhizhong @.***> đã
viết:

@chuvanhoang888 https://github.com/chuvanhoang888

For "I have tested v0.1.5 version on FK-POS80-AT printer, printing
was done
successfully, however out of 4 print times, 1 time printing failed", I
noticed that your output error shows 'status 133', which might be a bug of
Android. Please refer to
https://github.com/dariuszseweryn/RxAndroidBle/wiki/FAQ:-Cannot-connect
and https://issuetracker.google.com/issues/37121040. Maybe you can try
add durations to each printing, or not calling disconnect after each
printing but after all printing is done.
For "For the MHT-P80A printer, the printer is still not printing
even though
the writeText function returns true", Some devices might has more than
1 writable characteristic, but not all those characteristic can handle your
printing commands. This plugin will find out all writable characteristics
but only use the first one. But even we use a wrong characteristic, the
printing callback result might be still success. In this case, you should
specific the characteristicUuid to tell this plugin which characteristic
should be used.
For "Have you added this "characteristicUuid" property for ios. On
ios,
printer "MHT-P80A" works normally and printer "FK-POS80-AT" gets error
like
on android". Not yet, I'm a little busy now and it's planned to the
day after tomorrow.

—
Reply to this email directly, view it on GitHub
https://github.com/xiaozhizhong/flutter_simple_bluetooth_printer/issues/1#issuecomment-1436334284,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY5GZ2D3LCDTY3NOR24XE5TWYL3NTANCNFSM6AAAAAAU7INBE4
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

For ios: printer "MHT-P80A" works normally, but with printer "FK-POS80-AT"
get error "Failed to write value for the characteristic. Error:The value's
length is invalid."
[image: Screen Shot 2023-02-20 at 14.54.34.png]
Vào Th 2, 20 thg 2, 2023 vào lúc 12:04 xiaozhizhong <
@.***> đã viết:

@chuvanhoang888 https://github.com/chuvanhoang888

For "I have tested v0.1.5 version on FK-POS80-AT printer, printing
was done
successfully, however out of 4 print times, 1 time printing failed", I
noticed that your output error shows 'status 133', which might be a bug of
Android. Please refer to
https://github.com/dariuszseweryn/RxAndroidBle/wiki/FAQ:-Cannot-connect
and https://issuetracker.google.com/issues/37121040. Maybe you can try
add durations to each printing, or not calling disconnect after each
printing but after all printing is done.
For "For the MHT-P80A printer, the printer is still not printing
even though
the writeText function returns true", Some devices might has more than
1 writable characteristic, but not all those characteristic can handle your
printing commands. This plugin will find out all writable characteristics
but only use the first one. But even we use a wrong characteristic, the
printing callback result might be still success. In this case, you should
specific the characteristicUuid to tell this plugin which characteristic
should be used.
For "Have you added this "characteristicUuid" property for ios. On
ios,
printer "MHT-P80A" works normally and printer "FK-POS80-AT" gets error
like
on android". Not yet, I'm a little busy now and it's planned to the
day after tomorrow.

—
Reply to this email directly, view it on GitHub
https://github.com/xiaozhizhong/flutter_simple_bluetooth_printer/issues/1#issuecomment-1436334284,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY5GZ2D3LCDTY3NOR24XE5TWYL3NTANCNFSM6AAAAAAU7INBE4
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

Can you please provide the commands? it seems that your printing commands are too long

Can you please provide the commands? it seems that your printing command

I use the package "esc_pos_utils" and only print a short piece of Unit8List code

s are too long

Please copy and attch the bytess

Please copy and attch the bytess

`void _print2X1() async {
if (selectedPrinter == null) return;
final codes = "Chu Van Hoang";
List bytes = [];
// Xprinter XP-N160I
final profile = await CapabilityProfile.load(name: 'XP-N160I');
// PaperSize.mm80 or PaperSize.mm58
final generator = Generator(PaperSize.mm80, profile);
bytes += generator.setGlobalCodeTable('CP1252');
bytes += generator.text('Test Print',
    styles: const PosStyles(align: PosAlign.center));
bytes += generator.text('Product 1');
bytes += generator.text('Product 2');
bytes += generator.text('Product 3');
bytes += generator.cut();
Uint8List bytess = Uint8List.fromList(bytes);
try {
  await _connectDevice();
  if (!_isConnected) return;
  final isSuccess = await bluetoothManager.writeRawData(bytess);
  if (isSuccess) {
    await bluetoothManager.disconnect();
  }
} on BTException catch (e) {
  print(e);
}

}`

Please copy and attch the bytess

void _print2X1() async {     if (selectedPrinter == null) return;     //final codes = "Chu Van Hoang";     List<int> bytes = [];     // Xprinter XP-N160I     final profile = await CapabilityProfile.load(name: 'XP-N160I');     // PaperSize.mm80 or PaperSize.mm58     final generator = Generator(PaperSize.mm80, profile);     bytes += generator.setGlobalCodeTable('CP1252');     bytes += generator.text('Test Print',         styles: const PosStyles(align: PosAlign.center));     bytes += generator.text('Product 1');     bytes += generator.text('Product 2');     bytes += generator.text('Product 3');     bytes += generator.cut();     Uint8List bytess = Uint8List.fromList(bytes);     try {       await _connectDevice();       if (!_isConnected) return;       final isSuccess = await bluetoothManager.writeRawData(bytess);       if (isSuccess) {         await bluetoothManager.disconnect();       }     } on BTException catch (e) {       print(e);     }   }

Hello Sir,
I just found an error when doing print on android that is: printout data is
mising
final codes = "^XA\r\n^MMT\r\n^PW384\r\n^LL0253\r\n^LS0\r\n
^BY2,3,81^FT375,92^BCI,,N,N\r\n^FDILP-107661^FS\r\n^FT375,191^A0I,45,45^FH\
^FDILP-107661^FS\r\n^FT374,27^A0I,23,24^FH\^FD^FS\r\n^FT372,59^A0I,23,24^FH
\^FD^FS\r\n^PQ1,0,1,Y^PQ1^XZ\r\n";
Vào Th 2, 20 thg 2, 2023 vào lúc 15:31 xiaozhizhong <
@.***> đã viết:

Please copy and attch the bytess
—
Reply to this email directly, view it on GitHub
https://github.com/xiaozhizhong/flutter_simple_bluetooth_printer/issues/1#issuecomment-1436542820,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY5GZ2DLKLIIWQFU5CGFN5DWYMTU7ANCNFSM6AAAAAAU7INBE4
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

I googled "printout data is mising" but nothing came up, is this the original error text?
The printing codes in example is in ZPL format for Zebra printers. If your printer dose not support ZPL, you should not use this codes for testing. See https://supportcommunity.zebra.com/s/article/ZPL-Command-Information-and-DetailsV2?language=en_US

can i contact you
Vào 8:51, Th 4, 22 thg 2, 2023 xiaozhizhong @.***> đã
viết:

I googled "printout data is mising" but nothing came up, is this the
original error text?
The printing codes in example is in ZPL format for Zebra printers. If your
printer dose not support ZPL, you should not use this codes for testing.
See
https://supportcommunity.zebra.com/s/article/ZPL-Command-Information-and-DetailsV2?language=en_US
—
Reply to this email directly, view it on GitHub
https://github.com/xiaozhizhong/flutter_simple_bluetooth_printer/issues/1#issuecomment-1439324882,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY5GZ2D3KZDZLPTI66OXCNDWYVWK7ANCNFSM6AAAAAAU7INBE4
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

Sure. Email<EMAIL_ADDRESS>

@chuvanhoang888

For "I have tested v0.1.5 version on FK-POS80-AT printer, printing was done
successfully, however out of 4 print times, 1 time printing failed", I noticed that your output error shows 'status 133', which might be a bug of Android. Please refer to https://github.com/dariuszseweryn/RxAndroidBle/wiki/FAQ:-Cannot-connect and https://issuetracker.google.com/issues/37121040. Maybe you can try add durations to each printing, or not calling disconnect after each printing but after all printing is done.
For "For the MHT-P80A printer, the printer is still not printing even though
the writeText function returns true", Some devices might has more than 1 writable characteristic, but not all those characteristic can handle your printing commands. This plugin will find out all writable characteristics but only use the first one. But even we use a wrong characteristic, the printing callback result might be still success. In this case, you should specific the characteristicUuid to tell this plugin which characteristic should be used.
For "Have you added this "characteristicUuid" property for ios. On ios,
printer "MHT-P80A" works normally and printer "FK-POS80-AT" gets error like
on android". Not yet, I'm a little busy now and it's planned to the day after tomorrow.

update: Added characteristicUuid for iOS

Hello,
Can you test this case on android
void _print2X1() async {
if (selectedPrinter == null) return;
List bytes = [];
// Xprinter XP-N160I
final profile = await CapabilityProfile.load(name: 'XP-N160I');
// PaperSize.mm80 or PaperSize.mm58
final generator = Generator(PaperSize.mm80, profile);
bytes += generator.setGlobalCodeTable('CP1252');
bytes += generator.text('Test Print',
styles: const PosStyles(align: PosAlign.center));
bytes += generator.text('Product 1');
bytes += generator.text('Product 2');
bytes += generator.text('Product 3');
bytes += generator.cut();
Uint8List bytess = Uint8List.fromList(bytes);
try {
await _connectDevice();
if (!_isConnected) return;
final isSuccess = await bluetoothManager.writeRawData(bytess);
if (isSuccess) {
await bluetoothManager.disconnect();
}
} on BTException catch (e) {
print(e);
}
}`
Vào 21:22, Th 5, 23 thg 2, 2023 xiaozhizhong @.***> đã
viết:

@chuvanhoang888 https://github.com/chuvanhoang888

For "I have tested v0.1.5 version on FK-POS80-AT printer, printing
was done
successfully, however out of 4 print times, 1 time printing failed", I
noticed that your output error shows 'status 133', which might be a bug of
Android. Please refer to
https://github.com/dariuszseweryn/RxAndroidBle/wiki/FAQ:-Cannot-connect
and https://issuetracker.google.com/issues/37121040. Maybe you can try
add durations to each printing, or not calling disconnect after each
printing but after all printing is done.
For "For the MHT-P80A printer, the printer is still not printing
even though
the writeText function returns true", Some devices might has more than
1 writable characteristic, but not all those characteristic can handle your
printing commands. This plugin will find out all writable characteristics
but only use the first one. But even we use a wrong characteristic, the
printing callback result might be still success. In this case, you should
specific the characteristicUuid to tell this plugin which characteristic
should be used.
For "Have you added this "characteristicUuid" property for ios. On
ios,
printer "MHT-P80A" works normally and printer "FK-POS80-AT" gets error
like
on android". Not yet, I'm a little busy now and it's planned to the
day after tomorrow.

update: Added characteristicUuid for iOS
—
Reply to this email directly, view it on GitHub
https://github.com/xiaozhizhong/flutter_simple_bluetooth_printer/issues/1#issuecomment-1441882681,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY5GZ2AVKYB77PL2PP3LRILWY5XDJANCNFSM6AAAAAAU7INBE4
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

I changed write withResponse to withoutResponse on iOS, Please try again to see if it's fixed.

[image: image.png]
[image: image.png]

I still can't print from the bytecode generated from the
"flutter_esc_pos_utils" package on Android. However i tried splitting the
bytecode and it prints successfully

https://pub.dev/packages/flutter_esc_pos_utils

Printing on ios via FK-POS-80-AT printer still failed even though
writable characterictis was found

service = '49535343-FE7D-4AE5-8FA9-9FAFD205E455';
bakeCharacteristic = '49535343-8841-43F4-A8D4-ECBE34729BB3';
                service = "E7810A71-73AE-499D-8C15-FAA9AEF0C3F2";
                bakeCharacteristic =

"BEF8D6C9-9C21-4C9E-B632-BD58C1009F9F";
[image: image.png]
Vào Th 5, 23 thg 2, 2023 vào lúc 22:59 xiaozhizhong <
@.***> đã viết:

I changed write withResponse to withoutResponse on iOS, Please try again
to see if it's fixed.
—
Reply to this email directly, view it on GitHub
https://github.com/xiaozhizhong/flutter_simple_bluetooth_printer/issues/1#issuecomment-1442028921,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY5GZ2BFGGB4TUW5KBI3CLTWY6CN5ANCNFSM6AAAAAAU7INBE4
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

The problem is that the detected device is getting a wrong configuration. Is returning a BLE device instead of Classic Device.
So the only solution I found for now its when it's detected as BLE in Android, set it as Classic Bluetooth with v0.1.5. I haven't tried with more printers, but this way it's working with Android & iOS:

Likewise, nowadays almost all Android devices accept classical Bluetooth or at least are compatible with both versions.


Cannot add package

I am trying to install SFSafeSymbols version 5.2, and Xcode is having a wee bit of a problem.
I select the package:

Dependency Rule: Up to Next Major Version (5.2.0 < 6.0.0)
Add to Project: (my project)

Result:
Package Resolution Failed:

SFSafeSymbols

Updating from https://github.com/SFSafeSymbols/SFSafeSymbols
the package manifest at '/Packages.swift' cannot be accessed (Package.swift doesn't exist in the file system)

(Screen captures below

Selecting the package:

Error message:

I understand that this could be an issue with my environment and not an issue with SFSafeSymbols,  but any assistance with this would be greatly appreciated.

As it turns out, this was an issue with my own environment and not an issue with SFSafeSymbols. It seems that Xcode + SPM is still a bit of a work in progress at Apple, perhaps.


Stuck in Provisioning after update

**What kind of request is this (question/bug/enhancement/feature request):**bug/question
Steps to reproduce (least amount of steps as possible):

create cluster with cluster authorized endpoint (via ui)
update cluster-config, and disable the endpoint

Result:

cluster stuck in provisioning, displaying:

This cluster is currently Provisioning.

Exit status 1, Error from server (NotFound): error when deleting "management-statefile_path_redacted": daemonsets.apps "kube-api-auth" not found

Other details that may be helpful:
cluster still accessible and working
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.3.3
Installation option (single install/HA): HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Custom
Machine type (cloud/VM/metal) and specifications (CPU/memory): metal
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.0", GitCommit:"70132b0f130acc0bed193d9ba59dd186f0e634cf", GitTreeState:"clean", BuildDate:"2019-12-07T21:20:10Z", GoVersion:"go1.13.4", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.3", GitCommit:"b3cbbae08ec52a7fc73d334838e18d17e8512749", GitTreeState:"clean", BuildDate:"2019-11-13T11:13:49Z", GoVersion:"go1.12.12", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

Client:
 Version:           18.06.2-ce
 API version:       1.38
 Go version:        go1.10.3
 Git commit:        6d37f41
 Built:             Sun Feb 10 03:47:56 2019
 OS/Arch:           linux/amd64
 Experimental:      false

Server:
 Engine:
  Version:          18.06.2-ce
  API version:      1.38 (minimum version 1.12)
  Go version:       go1.10.3
  Git commit:       6d37f41
  Built:            Sun Feb 10 03:46:20 2019
  OS/Arch:          linux/amd64
  Experimental:     false

regards,
strowi

seems to be related to https://github.com/rancher/rancher/issues/22409  except disabling and re-enabling doesnt fix the issue.

Are you absolutely sure you have the resources to deploy in whatever cloud? aka capacity. There are a lot of older tickets with this similar issue if you search through. Hope it helps

This is bare-metal and the cluster is only barely used right now:

Can you share the logging from the rancher/rancher container after you enabled the endpoint?

i tried again to disable + enable the endpoint, but the rancher-pods don't give any output. And debug mode is just A LOT, and i couldn't find anything out of the ordniary. Anything special to look for?
Righ now, when i try to disable the endpoint and hit save, the ui doesn't take the changes, and still shows it as enabled...


Allow users to set weights for different loss terms from outside

As part of this PR, also two functions to convert between internal (e.g. energy_strain_gradients) and external (e.g. virial) names.

📚 Documentation preview 📚: https://metatensor-models--193.org.readthedocs.build/en/193/

The internal/external naming isn't changing anything that we didn't have before... I just extracted a function for the name changes that were being done both for the logger and eval, so we can handle them in one single place


ParallelGZIPInputStream

I'm currently working on parallelizing this in my own fork. Do you have any advice or resources to share? Didn't get very far unfortunately.
How do you detect how many bytes to "chop off" and pass to a thread for decompression?

Making progress.. I'm thinking we can store the segment sizes for splitting up the inflation in the FEXTRA header field. The problem is that we can only write them once the file has been written... Wish there was a better way to append information.
My testing works nicely so far, just have to find a better way to communicate the block sizes. Any suggestions?

I don't actually know a good answer to your question. TBH, I wrote this as a tutorial exercise for some other programmers in how to write and profile good code in the JUC APIs. However, I have been using it in anger in various projects, and it's been a great boon. Those projects tend to be typified by very large data sets, so one option is just to hand off multimegabyte blocks to something like an FJP and pay the cost of passing the leading/trailing bits between threads... although can a thread "resync" if it's given the second megabyte at random?
Again, I confess, I didn't read the spec, I didn't do dictionary pre-seeding or anything. This was really about 30 minutes' work as a tutorial exercise which turned out to be publishable. I'll help as much as I can.

Makes sense. I really like this paper on the topic:
http://prof.icc.skku.ac.kr/~jaewlee/pubs/lctes13_vld.pdf

Right now, gzip decompression is costing me 45 seconds per unit test in one of my products, and my system has 4 cores. The data might or might not have been compressed using this parallelgzipoutpustream, but either way, would I love to get that down to 12 seconds: "yes!"

 I'll try to get the parallel part going next weekend. Boundary guessing is then a separate task.

Also, for linear scaling, the largest boxes I have are a 4(8 w/HT) core E5620 and a similar Core2. It seems not to get much benefit from the HT cores. I'd be very interested in any results from larger boxes. I did get some suggestions from concurrency-interest at some point. That said, my primary use case is saving human time on 4-core laptops, not saving real-time on 64-core servers.

:+1:


学术引用

请问如何引用这篇论文？目前是还没发表吗？

您好，邮件已收到。

Google scholar has no search information yet. You can check the relevant information on the paper page received by icdm2021 at https://icdm2021.auckland.ac.nz/accepted-papers/
DM1049 “Addressing Exposure Bias in Uplift Modeling for Large-scale Online Advertising”
Wenwei Ke, Chuanren Liu, Xiangfu Shi, Yiqiao Dai, Philip Yu, and Xiaoqiang Zhu
发件人: @.*** @.> 代表 yuyihe-1216
发送时间: 2022年1月18日 13:25
收件人: aifor/eeuen @.>
抄送: Subscribed @.***>
主题: [aifor/eeuen] 学术引用 (Issue #1)
请问如何引用这篇论文？目前是还没发表吗？
—
Reply to this email directly, view it on GitHub https://github.com/aifor/eeuen/issues/1 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AAYV2LFINSYL4MPDCOT6RYLUWT2Q5ANCNFSM5MGHDSWA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .
You are receiving this because you are subscribed to this thread.  https://github.com/notifications/beacon/AAYV2LAYRFYUGGMUSAQHCTTUWT2Q5A5CNFSM5MGHDSWKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4QPTW6RA.gif Message ID: @.*** @.***> >

now you can quota this work with
@inproceedings{ke2021addressing,   title={Addressing Exposure Bias in Uplift Modeling for Large-scale Online Advertising},   author={Ke, Wenwei and Liu, Chuanren and Shi, Xiangfu and Dai, Yiqiao and Philip, S Yu and Zhu, Xiaoqiang},   booktitle={2021 IEEE International Conference on Data Mining (ICDM)},   pages={1156--1161},   year={2021},   organization={IEEE} }


Error during tests

I create Angular 6 app. I run
npm install --save object-fit-images
and add this to polyfills.ts :
import objectFitImages from 'object-fit-images'; objectFitImages();
Page work ok, but when i try test app - i get this :
27 08 2018 15:37:17.921:WARN [karma]: No captured browser, open http://localhost:9876/
27 08 2018 15:37:17.980:INFO [Chrome 68.0.3440 (Mac OS X 10.13.6)]: Connected on socket qcouXxMJ1_pUqUGyAAAA with id 73542196
Chrome 68.0.3440 (Mac OS X 10.13.6) ERROR
  Uncaught TypeError: object_fit_images_1.default is not a function
  at http://localhost:9876/_karma_webpack_/polyfills.js:12909
Chrome 68.0.3440 (Mac OS X 10.13.6) ERROR
  Uncaught TypeError: object_fit_images_1.default is not a function
  at http://localhost:9876/_karma_webpack_/polyfills.js:12909
Chrome 68.0.3440 (Mac OS X 10.13.6) ERROR
  Uncaught TypeError: object_fit_images_1.default is not a function
  at http://localhost:9876/_karma_webpack_/polyfills.js:12909

Chrome 68.0.3440 (Mac OS X 10.13.6): Executed 0 of 0 ERROR (0.148 secs / 0 secs)

Do you know reason ?

Yes, use require in Typescript because this is not a ES Module, just CommonJS


cog mediatype at iana

Has there been any discussion on the use of a dedicated mediatype for COG?
Does it make sense to have a dedicated mediatype, or is it better to tag COG files as regular image/tiff files?
If relevant, are there initiatives to register the cog mediatype at iana?
Question comes from the catalogue world, which wants to reference COG files from metadata records indicating the proper mediatype.

It should be image/tiff; application=geotiff; profile=cloud-optimized
Not sure where it comes from exactly but this has been discussed quite some time ago. FYI it's the one used in STAC https://github.com/stac-utils/pystac/blob/main/pystac/media_type.py#L7
I'm not sure if this was ever registered to Iana. maybe @cholmes knows

Thanx @vincentsarago, doesit make sense to add this aspect to the specification document?

would be a nice addition yes

I would point this discussion is redundant with https://github.com/opengeospatial/CloudOptimizedGeoTIFF/issues/1 and should maybe be closed in favor of it. The addition of new keys is apparently a no-go for IANA since you can only use keys that were foreseen in the image/tiff registration, but profile might be a special case due to another spec.

While reading those threads; conclusion should be:

the tiff mediatype as defined at iana does not allow to add alternative parameters then application, so the convention to use profile=cloud-optized is not correct
suggestion by @EmDevys to replace application=geotiff; profile=cloud-optimized for  application:geotiff+cog makes sense
using a dedicated parameter to differentiate geotiff from tiff can not be registered as such at iana (use of parameters is implicit to the image/tiff definition)


Tweak for firmware 1.3.1448 and up. The devices seems to return a tup…

Tweak for firmware 1.3.1448 and up. The devices seems to return a tuple of 6 instead of 3 when performing the sync gesture.
The first 3 seem to be correct, the remaining 3 are unknown.

Indeed, this is the way to go. Thanks!


StackOverflowError

AtomGrid.jl: Error During Test at /Users/yingxing/Projects/IntegralGrid.jl/test/test_atomgrid.jl:783
Got exception outside of a @test
StackOverflowError:

It appears that the error is caused by the use of a large vector in the cat function. It is important to note that this error is not specific to any particular version of Julia. I have personally tested the code on both julia-1.9.0 and julia-1.9.2 using MacOS M2 Pro and Ventura 13.5.


Warn if Update-Database finds no migrations

We just create an empty database, but we should also write a warning in case there are migrations somewhere but our commands just couldn't find them.

Triage, I don't think this is needed now that #4098 is in.

Sounds good to me.

:+1:


Parsing of generic ResourceObjects with ObjectParser.

The ObjectParser was limited to the Annotation-type only. The new implementation allows parsing of generic ResourceObjects using the new method parse(Class, String, URL, RDFFormat, boolean).
The old method for parsing Annotations was preserved to ensure compatibility, but rewritten to use the new generic implementation.
In order to keep the old implementation compatible the new one also doesn't guarantee that the result is duplicate-free.
Example usage:
ObjectParser parser = new ObjectParser();
// Specify the type to get all instances of it:
List<Person> persons = parser.parse(Person.class, doc, baseUri, RDFFormat.TURTLE, true);

// Use ResourceObject to get all resources in the document:
List<ResourceObject> allResources = parser.parse(ResourceObject.class, doc, baseUri, RDFFormat.TURTLE, true);

The ObjectParser now has two different implementations for the parse()-method (cf. fc1023b). The new one now guarantees a duplicate-free result. The old one keeps its behaviour of returning non-unique objects and was marked deprecated.


Picking multiple images from Photos

Hi,
Is it possible to choose multiple images using this valuable plugin? It's working well with single file selection option, I need to pick multiple photos and send to the server. If so please provide the suggestion here It would be great for manage this plugin.

Thanks @jamesmontemagno


[BUG] Always thinks that combat is enabled

Describe the bug
Unable to start time controls. Error message "There is an active combat" appears although there is no active combat.
To Reproduce
I have disabled all modules except for Tidy UI and Simple Calendar. The time settings/calendar was originally imported from Calendar/Weather

Go to the calendar
Click on the play button to start time
See error

Expected behavior
Time starts passing.
Browser (please complete the following information):

OS: Mac
Browser: Chrome
Browser Version: 90

Additional context
Note that I created a new world (I use the HarnMaster 3 system) and there it works as expected, however I'm using the default calendar in this case.

This message can appear if any scene has an active combat running. So the first thing I would check is to see if there is any scene in the world has an active combat.
This behaviour happens because advancing combat rounds actually changes the foundry game time and running a real world clock in to of that would advance the time too much.

I wasn't aware that combat was per scene. Thanks for pointing that out. I finally found the scene which had an active combat.
Suggestion: Either limit your check to the currently active scene, or perhaps in the error message mention which scene has an active combat so the GM doesn't have to hunt.

Yeah those are good points I will look into limiting it to the currently active scene or at the very least mention which scene it is.


Update README.md

Updated getCSS to getCss

CLA is valid!

Coverage remained the same at 100.0% when pulling 8f91a7d8a56a2f786a3e82b1434feea09b2f5728 on dylanharrington:patch-1 into eb8e10ee224e2cf4aca29720d7d756f1b808d7fd on yahoo:master.

Thanks @dylanharrington!


Problem with laravel 5.4

Hi, I hope you can help me to use this package with Laravel 5.4....I have tested with Laravel 5.3.30 and there aren't problems....but when I try to use in my project ( laravel 5.4) the url -> 'links' gives this error:
ReflectionException in Container.php line 719:
Class links.middleware does not exist
in Container.php line 719
at ReflectionClass->__construct('links.middleware') in Container.php line 719
at Container->build('links.middleware') in Container.php line 598
at Container->resolve('links.middleware') in Container.php line 567
at Container->make('links.middleware') in Application.php line 708
at Application->make('links.middleware') in Pipeline.php line 138
at Pipeline->Illuminate\Pipeline{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing{closure}(object(Request)) in SubstituteBindings.php line 41
at SubstituteBindings->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing{closure}(object(Request)) in VerifyCsrfToken.php line 65.........................
Can you help me to find a solution?
Thanks

Try with 1.0.3, just released ^^

Now all perfect .... thanks

You're welcome ^^


[path_provider] win32 package is being included on Android

Details

Target Platform: Android
Target OS version/browser: Android 11 ARM64 SDK 30
Devices: Emulator
I've tried out the new flag to analyze the app size and it seems like Windows only package win32 is being included by path_provider, even though it's clearly an Android release mode build.
Logs
[√] Flutter (Channel stable, 1.22.0, on Microsoft Windows [Version 10.0.18363.418], locale en-US)
    • Flutter version 1.22.0 at C:\Progs\flutter
    • Framework revision d408d302e2 (7 days ago), 2020-09-29 11:49:17 -0700
    • Engine revision 5babba6c4d
    • Dart version 2.10.0

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at C:\Progs\AndroidSDK
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = C:\Progs\AndroidSDK
    • Java binary at: C:\Progs\AndroidStudioBeta\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[√] Android Studio (version 4.0)
    • Android Studio at C:\Progs\AndroidStudioBeta
    • Flutter plugin version 50.0.1
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] Connected device (1 available)
    • sdk gphone x86 (mobile) • emulator-5554 • android-x86 • Android 11 (API 30) (emulator)

• No issues found!

I'm currently using the latest version:
path_provider: ^1.6.18
Here is the analysis of the app size using the flutter build apk --release --target-platform android-arm64 --analyze-size flag:
  lib/
    arm64-v8a                                                               6 MB
    Dart AOT symbols accounted decompressed size                            6 MB
      package:flutter                                                       3 MB
      package:timezone                                                    496 KB
      dart:core                                                           400 KB
      dart:typed_data                                                     274 KB
      dart:io                                                             214 KB
      dart:async                                                          205 KB
      dart:ui                                                             204 KB
      package:flutter_localizations                                       157 KB
      dart:collection                                                     154 KB
      package:dio                                                          78 KB
      dart:convert                                                         74 KB
      package:source_span                                                  68 KB
      package:intl                                                         60 KB
      package:alpha_education                                              58 KB
      package:win32                                                        42 KB
      package:auto_route                                                   39 KB
      dart:isolate                                                         37 KB
      package:vector_math                                                  35 KB
      package:easy_localization                                            34 KB
      package:flushbar                                                     25 KB

@ac130kz
Athough I don't have Android 11 emulator, but I tried on Android api 29 by adding the plugin and ran on latest stable and master, but don't see the win32 package in the list.

flutter doctor -v
[✓] Flutter (Channel stable, 1.22.0, on Mac OS X 10.15.4 19E2269, locale en-IN)
    • Flutter version 1.22.0 at /Users/dhs/documents/Fluttersdk/flutter
    • Framework revision d408d302e2 (7 days ago), 2020-09-29 11:49:17 -0700
    • Engine revision 5babba6c4d
    • Dart version 2.10.0

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.5, Build version 11E608c
    • CocoaPods version 1.9.3

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 46.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.49.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.11.0

[✓] Connected device (1 available)
    • SM A260G (mobile) • 5200763ebcfa861f • android-arm • Android 8.1.0 (API
      27)

• No issues found!

Probably since you are on windows and I am on Mac, maybe that's why I am not seeing it.
Thanks,

This is currently the expected behavior, due to a combination of the workaround for https://github.com/flutter/flutter/issues/52267 (requiring a direct dependency from path_provider to path_provider_windows) and https://github.com/flutter/flutter/issues/14233 (platform checks aren't cut points for tree shaking).
@amirh FYI: another slightly different data point against the proposed solution of generating the Dart registration for all platforms at once.


VoD preprocessing results in gt_mask being only 1.0

I want to take your previous approach RaFlow as a baseline for my master thesis on self-supervised moving object segmentation of radar point clouds. During evaluation of the models provided in the repo I found that the results were suspicious in the sense that the moving IoU of the vod model was exactly zero. Upon further inspection I found that the gt_masks against which the results are measured only contain ones meaning every point is considered static. This can't be correct since the original dataset explicitly contains moving objects.
Is there any way you could help me find the error in the preprocessing?

Hi, could you visualize and observe the gt_masks for more frames? Did you use our provided code to generate the gt_masks?

For generation of the export I simply followed the instructions in the GETTING_STARTED.md.
I iterated through all the exported pairs and counted the ones where the gt_masks only contained ones. This number of pair is exactly the size of the respective partition. Please find the code for this in the attached zip-file.
VoD_inspection.zip

Hi, could you show me which frames only contained ones in their gt_masks. Note that for our training samples, we do not generate gt_masks as our method rely on cross-modal supervision signals.

The distinction between train, val and test is clear. For the training samples the groundtruth masks and labels are zero since for training of CMFlow the pse ones are used. But actually for every single sample in the validation and test partition the gt_masks seem to be only ones.

Hi, could you try some debugging to find out from which step the generation of gt_masks becomes abnormal? I think this should help to solve your problem as our users could succesfully generate valid gt_masks and reproduce our results.


fix(ffe-buttons): Increased margin between buttons to give them som air

This PR solves #515
New distance between buttons in a button group:

Implemented according to sketch from @adidick

Coverage remained the same at 81.121% when pulling 8e0f9d36bd4f37aa341328f3bc2b038b53a2e396 on hdaljord:515-button-group-margin into 57fd626c6d771c487e07cc95ee1a33165d04cd6a on SpareBank1:develop.


Proxy Server

Our severs behind firewall, how to setup the proxy server for the API client

Any solutions for this?

You need to use the superagent-proxy.   Patched the API client file to support proxy.

Hello,
An update regarding this ticket. We have an internal branch adding proxy support with superagent-proxy. However, in order to maintain support for Node version 4 and 5 we are unable to include this at this time. This ticket has been targeted for the next major upgrade of this package.

A branch with proxy support has now been added with v5.1.0.
Please take a look at branch v5.1.0-Proxy for this. Keep in mind, that in order to add this capability, support or Node.js v4 and v5 has been dropped on this branch.


chore(main): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2023-05-08)
Features

add cgroup and test (238cab0)
add dockerfile (7101064)
add filesystem and test (e4d20ca)
add main (1c5a3a1)
add network and test (a4bd683)
add process and test (0791854)
add run func (e957f32)
add util (724984f)

Bug Fixes

add error handling (43b1e48)
add logs (9afb1ab)
add sudo to MustSetHostname (de4798f)
bulk improvements (ac41e22)
comment out defer (27a0858)
comment out for build pass (be4913e)
Deferring unsafe method "Close" on type "*os.File" (8407f43)
errors unhandled (e1eb232)
GetAvailableIP IP address range calculation (8263bfc)
modularize the container module (e4a39cc)
potential file inclusion via variable (ef09b70)
reaplce ioutil (17830ca)
subprocess launched with a potential tainted input or cmd arguments (ab2530c)
tests (4aff97f)
update (288db87)
update (de68b01)
update cgroup code/tests (265d8b9)
update cgroup tests (7ae6021)
update dockerfile golang image (701762f)
update filesystem and tests (a190f9f)
update GetAvailableIP (7beb82d)
update mod sum (6dd8ea6)
update packages (d0139bc)
update permission (8baa0e0)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/elispeigel/spocker/releases/tag/v1.0.0 :sunflower:


Fail to connect, while using firewall app on android.

Describe the bug
I using RethinkDNS as firewall for android.

i also allow your app to connect 192.168..
also dosable Block connections without VPN
but still fail, need to disable all VPN firewall to make it work.

To reproduce

install RethinkDNS
check RethinkDNS allow LocalSend
make windows PC as Hotspot
try to send or recieve file

Expected behavior
i saw some app can connect to local network even enabled firewall, you app may be possible soon.
Screenshots
No response
Desktop (please complete the following information)

Windows 11

Smartphone (please complete the following information)

Android 13

Additional context
No response

I can send to the guest device when lockdown mode is enabled. But can't receive with lockdown mode enabled.

I can send to the guest device when lockdown mode is enabled. But can't receive with lockdown mode enabled.

I found a solution at least for RDS.
You can enable Don't Route Private IPs , while momentarily disabling VPN lockdown mode, otherwise it'll not allow you to enable it. (RDS) ver 055e)


Falco76 patch 1

ripulitura dell'esempio di discovery

README-DEV.env.md doesn't exists anymore
disco.html should be improved with the web link I commented, and then you removed. It should be simply commented out and improved with a specialized logo, different from which already used for spid-saml-check.
Feel free to resubmit your PR

README-DEV.env.md doesn't exists anymore
disco.html should be improved with the web link I commented, and then you removed. It should be simply commented out and improved with a specialized logo, different from which already used for spid-saml-check.
Feel free to resubmit your PR


TypeError with array or src globs since v0.6.4

In version 0.6.4, I get the following error: TypeError: Arguments to path.resolve must be strings at configureFiles (index.js:58:27).
I believe it's because I set src to be an array of globs:
src: [
	'core/templates',
	'theme/templates'
]

It was working fine in version 0.6.1, so I assume the responsible commit is 806b34a0ae9472be312e2a6403a3346cdab50df7.

Whoops! Sorry about that. I'll fix it as soon as possible.

OK, I reverted the changes made in v0.6.4 in order to produce v0.6.5. So, it should be fixed now.
I didn't get around to adding tests for every scenario, so that's why that happened.

No worries, thanks for the quick turnaround! :+1:


sparsehash is slower on a release build with asserts

What steps will reproduce the problem?
1. Building with ASSERTS on in a Release environment.

What is the expected output? What do you see instead?
I expected dense_hash_set to beat the unordered_set even on an asserted release
build.

What version of the product are you using? On what operating system?
Sparsehash 2.0.2

Please provide any additional information below.
64-bit Linux, GCC 4.7.2, -O2 -std=c++11

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 9 Jun 2015 at 3:44

What may happen is that the original poster may include another file which inconditionally defines NDEBUG to 1, which would enable the asserts even in release. For example, Google's own cpp-btree headers (see btree.h) has the following:
#ifndef NDEBUG
#define NDEBUG 1
#endif
so wherever btree.h is included, asserts are enabled even in release builds. This is a bug in cpp-btree IMHO.


Cut a new release for iOS

The last version of mumble available for iOS is from 2014, but it's clear that you've made updates to the codebase since then.  This is a request for you to cut a new release and upload it to the itunes store.

Yes, the app really needs a new release.
Unfortunately, all my time right now is spent getting a 1.3.0 release of desktop Mumble out the door.
Hopefully I'll get some time during the month to be able to cut a release.
Here's a checklist/braindump of the necessary steps. Any help is greatly appreciated.
If anyone is interested, I think we need to go through the various warnings we're getting with Xcode 8.3 and fix 'em. Also, we should consider deprecated APIs in the process, since we'll probably be targetting iOS 9 or 10 as the deployment target.
Also, we should check whether we need to update icons, add a .xib launch screen for big devices, etc. etc.

I hope for a new release too. Specially the PTT function and settings panes after connected server. I wish to help but I'm not a expert in iOS. However I can help to design the settings UI(and maybe the whole app UI). I will upload my works later.

Mumble for iOS 1.3.1 released.


Use latest rustc-serialize and increment version for crates.io

Closes #86

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

Hmm trav seems to be erroring on this line, going to update to the latest nightly and see if I can reproduce the error on my end and fix it.

The issue is that the .from_str_radix method has been "moved" - the docs mention this but they do not mention where it has been moved too.

#88 is related

The issue was that rustc was choosing the recently re-implemented, unstable version of the from_str_radix method rather than the local Num implementation. Using Num on the culprit line seems to have fixed it.

Ok, as alex pointed out my last attempt was just causing an infinite loop.
This last commit copies the implementation from std into the Num implementation as a temporary avoidance of using the unstable feature. I've left a FIXME indicating it should be replaced once the std from_str_radix is stabilised.
@alexcrichton is this ok? (currently waiting on travis)


Permit merging of conflicting orders

Attempts to address #751.
Work in progress. Still needs:

UI, or a way to post to this view
Redirect back to the proper place (the attendee assignment page?)
Merging of EventHousing data (this is potentially tricky)
Message reporting what happened.

I think we can probably not try to transfer any attendee or housing info (for now), as long as we direct people to verify that their registration info is correct. The priority for this ticket is making sure people have access to everything they paid for...
But I keep flip-flopping on this. Maybe it would be good to transfer attendees, and transfer EventHousing if one doesn't already exist?

Having to re-enter data would also be pretty frustrating.

@melinath @harrislapiroff I think this is ready to review -- currently it is not transferring attendees or eventhousing.

This is looking good!
A couple things:

Could the "Merge" button go on the right like the "Claim" button does?

Now that merging is a thing, the count in the menu should probably include the mergeable count as well.

I'm feeling more and more like we should transfer attendees (and hosting info if possible). Otherwise we'll just have another data loss bug down the line where someone does the transfer and doesn't re-enter their attendees, resulting in purchased items that aren't attached to anyone – and potentially attendees who vanish from the database! Event admins will be able to deal much better with occasional duplicates.
We can still ask people to go through and verify their data; just less danger this way.

@melinath by "hosting info" do you mean EventHousing objects? If so I think that might be hard to do because an Order can only have one EventHousing, so it's not clear which one should supercede the other. There's going to be some data lost either way.

@chigby Clarification: I think we should transfer it if the one it's being merged into doesn't have an EventHousing object attached, and just let it get deleted otherwise. It's data loss, but not as bad? If we allowed multiple hostings per order (which we should someday) then I'd want to switch to copying it over. But until then this should be fine.

The modal can even give a specific warning if the EventHousing conflicts & will be lost, and otherwise just give a general notice that folks should verify their data.

@melinath Okay, I think this is ready to review

This looks great. Merging.


Implemented project requirments

[ ] Created a new Rails app called 'hello-rails-react'.
[ ] Initialized your project with Git.
[ ] Set up your project with webpacker the react-rails gems as described in the previous lesson's tutorial.
[ ] Has a Postgres database set up.
[ ] Created a table for storing your messages and created 5 different greetings.
[ ] Created an API endpoint that selects a random greeting from your table (you will need a controller with an action and Rails route).
[ ] Created a static view that will be the root of your app.

This project is an introduction for Ruby on Rails working with React independently (in separate repositories). Click here to go to the frontend repo.

STATUS: APPROVED  ✔

Hi @Dagic-zewdu ,
Your project is complete! Great job for nailing it right on the first attempt 🥇. There is nothing else to say other than... it's time to merge it 🍾🚢 :shipit: . Congratulations!  🎉💯🌟
Good Points 👍

Project is set up correctly ✔
Descriptive Pull Request ✔
No linter errors ✔
Professional documentation ✔

Cheers and Happy coding! :clap: :clap: :clap:
Feel free to leave any questions or comments in the PR thread by mentioning my Github username if something is not 100% clear.

As described in the Code reviews limits policy you have  limited number of  reviews per this project. If you think that the code review was not fair, you can request a second opinion using this form.


Updated contribution documentation.

Added notes about 3.4 LTS version.
Moved development documentation to a new page.
Corrected commit-squashing example in the documentation.

Coverage increased (+0.6%) to 89.528% when pulling 9d57db8604e135b101854b96c13beec6900be311 on evildmp:docs-contribution into af20a561b26cd32b4aafb21b5feccab96d2b1582 on divio:release/3.3.x.

Made several notes, but looks really great!

Coverage increased (+0.6%) to 89.528% when pulling 9d57db8604e135b101854b96c13beec6900be311 on evildmp:docs-contribution into af20a561b26cd32b4aafb21b5feccab96d2b1582 on divio:release/3.3.x.


Create Medium Polymer Item

Create a Polymer spool item. Should probably just look like a spool of PLA
2x2 size, max stacks 10

[x] Model
[x] Texture
[ ] ItemInformation
[x] Item3D
[ ] ItemControl/Icons
[ ] Spawn pool

PR: https://github.com/Menacing/unnamed-extraction-shooter-roguelike/pull/374


Improve System module types

Closes #197

While I was at it, I figured implementing #197 was straight-forward to fix.

@JumpLink Would you take a look?

@swsnr LGTM, thank you!


Add a LICENSE.md file to buzzkill

Use the MIT license

Closing, want to use a different license, but I chose the branch name poorly.


fix/docs: dated TLS auth link in KubeEnforcer docs' ToC

2d3b1eb109ab447f8453e5c21de34073cb170c64 changed the heading slightly,
but didn't change the respective entry in the ToC

this updates the ToC entry to match the heading

@agilgur5 done

Noting that this was a re-opening of #107


swagger version should not be hard written

swagger version needs to read from the package.json file

I think this is now fixed https://github.com/EddieHubCommunity/api/blob/main/src/swagger.ts#L6


Bug: KindeClient ignores environment variables

Prerequisites

[X] I have searched the repository’s issues and Kinde community to ensure my issue isn’t a duplicate
[X] I have checked the latest version of the library to replicate my issue
[X] I have read the contributing guidelines
[X] I agree to the terms within the code of conduct

Describe the issue
The documentation for kinde-java-sdk states that it can be configured via environment variables such as KINDE_DOMAIN, KINDE_CLIENT_ID etc; but this currently does not work.
The error can be traced to an imperfectly configured use of the dotenv library. The intention appears to be to allow kinde-java-sdk to be configured either via environment variables or via a .env file - or by some combination of both. (It is also possible to configure it programmatically, but I ignore that case here). In order to allow this, the project makes use of the io.github.cdimascio.dotenv-java library within KindeClientBuilder.
While the dotenv-java library can be configured to read from environment variables (by setting its systemProperties attribute), by default this is set to false and by default, the library requires the presence of a .env file, in the absence of which it throws an exception.
This problem should have a simple solution:

To ensure that dotenv can be configured from environment variables, it must be configured with systemProperties = true
To ensure that dotenv does not require a .env file, it must be configured with ignoreIfMissing = true.

Library URL
kinde-core/src/main/java/com/kinde/KindeClientBuilder.java
Library version
2.0.0
Operating system(s)
Other Linux
Operating system version(s)
Debian 12.7
Further environment details
No response
Reproducible test case URL
Simply, try to configure a KindeClient through environment variables with no .env file
Additional information
No response

Hi @ghferrari ,
Thank you for raising, and your detailed issue description.
I will get this checked out

Thanks for the detail fix, description, much appreciated


Relational Databases data types handling

Data Types for relational tables are different depending on database product.

Not sure what the implication is, given that the dataType property is just a String...  What's the underlying impact we're trying to address?  (Or is this issue stale, given it's more than 6 months old?)

I think this is stale ...


UIWebView are no longer accepted

Describe the bug
Apple rejected the build because of UIWebView, and I found some references in your framework.
UIWebView are formally deprecated and new apps containing these frameworks are no longer accepted by the App Store, and should adopt the replacement framework, WKWebView.
To Reproduce
Steps to reproduce the behavior:

Download the repo
Open Terminal app and execute: grep -R UIWebView *
See the result:

Binary file AdyenWeChatPayInternal.xcframework/ios-i386_x86_64-simulator/AdyenWeChatPayInternal.framework/AdyenWeChatPayInternal matches
Binary file AdyenWeChatPayInternal.xcframework/ios-arm64_armv7/AdyenWeChatPayInternal.framework/AdyenWeChatPayInternal matches
debug-symbols/iOS-Device/2FF47549-9319-3D43-887B-A0C518F72AD4.bcsymbolmap:-[WXUIWebViewControll initWithDelegate:]
debug-symbols/iOS-Device/2FF47549-9319-3D43-887B-A0C518F72AD4.bcsymbolmap:-[WXUIWebViewControll viewDidLoad]
debug-symbols/iOS-Device/2FF47549-9319-3D43-887B-A0C518F72AD4.bcsymbolmap:-[WXUIWebViewControll refresh:]
debug-symbols/iOS-Device/2FF47549-9319-3D43-887B-A0C518F72AD4.bcsymbolmap:-[WXUIWebViewControll webView:shouldStartLoadWithRequest:navigationType:]
debug-symbols/iOS-Device/2FF47549-9319-3D43-887B-A0C518F72AD4.bcsymbolmap:-[WXUIWebViewControll webViewDidStartLoad:]
debug-symbols/iOS-Device/2FF47549-9319-3D43-887B-A0C518F72AD4.bcsymbolmap:-[WXUIWebViewControll webViewDidFinishLoad:]
debug-symbols/iOS-Device/2FF47549-9319-3D43-887B-A0C518F72AD4.bcsymbolmap:-[WXUIWebViewControll webView:didFailLoadWithError:]
debug-symbols/iOS-Device/2FF47549-9319-3D43-887B-A0C518F72AD4.bcsymbolmap:-[WXUIWebViewControll .cxx_destruct]
debug-symbols/iOS-Device/2FF47549-9319-3D43-887B-A0C518F72AD4.bcsymbolmap:l_OBJC_PROTOCOL_$_UIWebViewDelegate
debug-symbols/iOS-Device/2FF47549-9319-3D43-887B-A0C518F72AD4.bcsymbolmap:l_OBJC_LABEL_PROTOCOL_$_UIWebViewDelegate
debug-symbols/iOS-Device/2FF47549-9319-3D43-887B-A0C518F72AD4.bcsymbolmap:l_OBJC_PROTOCOL_$_WXUIWebViewControllDelegate
debug-symbols/iOS-Device/2FF47549-9319-3D43-887B-A0C518F72AD4.bcsymbolmap:l_OBJC_LABEL_PROTOCOL_$_WXUIWebViewControllDelegate
Binary file debug-symbols/iOS-Device/AdyenWeChatPayInternal.framework.dSYM/Contents/Resources/DWARF/AdyenWeChatPayInternal matches
debug-symbols/iOS-Device/8E3708B7-FBE9-3612-8234-EB60A10873E4.bcsymbolmap:-[WXUIWebViewControll initWithDelegate:]
debug-symbols/iOS-Device/8E3708B7-FBE9-3612-8234-EB60A10873E4.bcsymbolmap:-[WXUIWebViewControll viewDidLoad]
debug-symbols/iOS-Device/8E3708B7-FBE9-3612-8234-EB60A10873E4.bcsymbolmap:-[WXUIWebViewControll refresh:]
debug-symbols/iOS-Device/8E3708B7-FBE9-3612-8234-EB60A10873E4.bcsymbolmap:-[WXUIWebViewControll webView:shouldStartLoadWithRequest:navigationType:]
debug-symbols/iOS-Device/8E3708B7-FBE9-3612-8234-EB60A10873E4.bcsymbolmap:-[WXUIWebViewControll webViewDidStartLoad:]
debug-symbols/iOS-Device/8E3708B7-FBE9-3612-8234-EB60A10873E4.bcsymbolmap:-[WXUIWebViewControll webViewDidFinishLoad:]
debug-symbols/iOS-Device/8E3708B7-FBE9-3612-8234-EB60A10873E4.bcsymbolmap:-[WXUIWebViewControll webView:didFailLoadWithError:]
debug-symbols/iOS-Device/8E3708B7-FBE9-3612-8234-EB60A10873E4.bcsymbolmap:-[WXUIWebViewControll .cxx_destruct]
debug-symbols/iOS-Device/8E3708B7-FBE9-3612-8234-EB60A10873E4.bcsymbolmap:l_OBJC_PROTOCOL_$_UIWebViewDelegate
debug-symbols/iOS-Device/8E3708B7-FBE9-3612-8234-EB60A10873E4.bcsymbolmap:l_OBJC_LABEL_PROTOCOL_$_UIWebViewDelegate
debug-symbols/iOS-Device/8E3708B7-FBE9-3612-8234-EB60A10873E4.bcsymbolmap:l_OBJC_PROTOCOL_$_WXUIWebViewControllDelegate
debug-symbols/iOS-Device/8E3708B7-FBE9-3612-8234-EB60A10873E4.bcsymbolmap:l_OBJC_LABEL_PROTOCOL_$_WXUIWebViewControllDelegate
Binary file debug-symbols/iOS-Simulator/AdyenWeChatPayInternal.framework.dSYM/Contents/Resources/DWARF/AdyenWeChatPayInternal matches

Expected behavior
No matches of UIWebView

Hey @poborp
Thanks for reaching out!
As far as I can tell WXUIWebViewControll   is part of the Wechat SDK. The issue is on their side.
Others reported it too
We will update it as soon as possible. For now I can't see anything related to WebView issue in release notes

So this problem will happen to all customers, it's possible to remove the WeChat SDK until they fix the bug? because we are blocked and Apple reject all new updates

it's possible to remove the WeChat SDK

what dependency management system are you using?
Just removing AdyenWeChat from podfile, Cartfile or packages.swift should be enough..

I used Swift Package Manager. I deleted the linked framework in the target and is working.
Thanks a lot @descorp :)


iOS and Samsung device contacts issue

I am using the version from 0.3.10 to 0.4.5 but in iPhone6 the contact avatar display as color circle but not displaying the name. Please help on this previously it was working but after upgrading this issue occur then again I downgrade the library and now I am not able to see the contact list on iphone6.
Also on the Samsung A70 it not showing the contacts in the listing.
I am using this method to fetch the contacts of the phone
await ContactsService.getContacts(withThumbnails: false);

@sperochon related to your PR?

@lukasgit   I don't know. The example app works fine on both my Android phone and my iPhone. It displays initials name in colored circles.
@mobiledev18  Do you reproduce this issue with the example app? If yes, please give some details about your Android and iOS versions. Open your app or the example Android or iOS app directly in Android Studio or Xcode, run it from here and see if there is an error message displayed in the console. Thanks.


Live-VQC dataset

Thanks for your great work. It helps me so much! I tried to download the Live-VQC dataset, but failed to do so every time. Could you please provide me with another way to download the live-VQC dataset? Thank you very much.

Hi, thanks for your interest in our work.
We can chat further on email, please contact me at<EMAIL_ADDRESS>


rename modules and directories to start with hawkular-wildfly-agent

@jmazzitelli this is the second reverted commit. Would you please review and eventually merge?


Support Debounce

The goal with this PR is to support Livewire model debounce modifier and to take control over the debounce value in general.
When ->reactive() is to fast, and .lazy is to slow... you want to reach for debounce.
Non breakable change.
New method examples
SomeField::make()->debounce(delay: '700ms')

x-on:input.debounce.{{ $getDebounce() }}="updateState"  // for blade files

Fields that are entangled doesn't support .lazy or .debounce therefore the applyStateBindingModifiers() has been updated to return different values based on if the field is entangled or not. (The $except parameter from "fake entangle" has been removed, it becomes obsolete)
Debounced alpine fields have been updated to use a dynamic value instead of fixed "500ms", to let the developer decide the delay.  (Only applied to KeyVal field because it is not applicable in the DatePicker)
The new $getDebounce() method can be used to pass the debounce value to alpine components.
Any field that has a wire.model="foo" attribute, can now use the new debounce method.
The default value is set to '500ms'
If one tries to use the .debounce() method on an entangled field the modifier is ignored.
(Replaces #3002 because I merged two PRs by mistake)

Please mark this as "ready for review" when the formatting changes have been removed :)

I've fixed the formatting changes, so this PR is now ready for review.

Thanks


ci: Disable dart-package-analyzer on release branches

axel-op/dart-package-analyzer is using flutter pub upgrade instead of get, which ignores pubspec.yaml dependency_overrides.
Because of that, all release/* branches are failing, because the package cannot find the "about to be released" version of our sentry-dart package that it depends on.
#skip-changelog

@marandaneto we can restore those once we find out how to properly do that.

Codecov Report

Merging #770 (1217851) into main (d878777) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #770   +/-   ##
=======================================
  Coverage   90.01%   90.01%
=======================================
  Files         105      105
  Lines        3376     3376
=======================================
  Hits         3039     3039
  Misses        337      337

Continue to review full report at Codecov.

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


SAM local commands misbehave for CDK Templates synthesized in PowerShell

Description
Operating on a Windows machine and using a Powershell terminal, when a user synthesizes a template from an AWS-CDK application, the output template does not behave as expected when invoking sam local commands.
Steps to reproduce

Open a Powershell terminal
Set the default out file encoding to utf8. See #1628 for details.

$PSDefaultParameterValues['Out-File:Encoding'] = 'utf8'

Synthesize a template from a CDK application.

cdk synth > template.yml

Invoke a sam local command

sam local invoke MyFunction3BAA72D1 --debug

Observed result
The sam local commands fail with the error message

Error: [InvalidTemplateException("'Resources' section is required")] 'Resources' section is required

Expected result
I expected the lambda function to be executed in a container.
Additional environment details (Ex: Windows, Mac, Amazon Linux etc)

OS: Windows 10 Enterprise
sam --version: 0.41.0
node: 12.14.1
python 3.6.5

Interestingly when I synthesized the template from Git Bash instead of PowerShell, the sam local commands appeared to work as expected.

Have you checked for the line endings?
Can you open the same file in notepad++ maybe, and convert line ending to LF

@sriram-mv yeah I tried that. In VS Code I switched the line endings from CRLF to just LF, but the result was the same.

Error: [InvalidTemplateException("'Resources' section is required")] 'Resources' section is required

Any update on this?
I get the following error when synthesizing a CDK template to be consumed by AWS SAM with Powershell via VSCode (on Win10)
Error: Failed to parse template: unacceptable character #x0000: special characters are not allowed   in "<unicode string>", position 3
Works fine running the same commands in command prompt

Possible Causes:
Error: [InvalidTemplateException("'Resources' section is required")] 'Resources' section is required

UTF-8 BOM is included in the first line with "Resources".
The section name becomes "0xEF 0xBB 0xBF Resources:".

Error: Failed to parse template: unacceptable character #x0000: special characters are not allowed in "", position 3

Template is encoded in UTF-16 instead of UTF-8

Any update on this?
I get the following error when synthesizing a CDK template to be consumed by AWS SAM with Powershell via VSCode (on Win10)

I had the same issue. I have been able to fix this in VS Code by installing a newer version of Powershell (I had version 5 on Win10) and changing the version of Powershell used by VS Code.
The issue (as I have understood it) is that Powershell 5 doesn't have the option to use utf8 with no BOM when redirecting output to a file. ">" is actually the same as using | Out-File in Powershell:
Use redirection operators (>, >>, 2>, 2>>, and 2>&1) to send the output of a command or expression to a text file. The redirection operators work like the Out-File
(from here: https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.core/about/about_operators?view=powershell-7)
And looking at the docs you can see that in version 5 default value for -Encoding is unicode (https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.utility/out-file?view=powershell-5.1):
unicode Uses UTF-16 with the little-endian byte order.
And there is no option for utf8 with no BOM.
So here is what I did:
Start by downloading version 6 or 7 (both have option to use utf8 with no BOM) from here: https://github.com/PowerShell/PowerShell
When in VS Code, go to File > Preferences > Settings. Type "powershell" in the searchbar to filter settings related to Powershell. Then look for Powershell Additional Exe Paths and click "edit settings.json". Make sure it contains this (or the path to where you have installed Powershell 6/7):
`"powershell.powerShellAdditionalExePaths": [
    {
        "exePath": "C:\\Program Files\\PowerShell\\7\\pwsh.exe",
        "versionName": "PowerShell 7"
    }
],
"powershell.powerShellDefaultVersion": "PowerShell 7",
"terminal.integrated.shell.windows": "C:\\Program Files\\PowerShell\\7\\pwsh.exe",`

The last line is added to make Terminal select Powershell 7 by default. After this I have been able to create template.yaml like this:
cdk synth --no-staging | Out-File -FilePath template.yaml -Encoding utf8NoBOM
or simply:
cdk synth --no-staging > template.yaml
since Powershell 7 uses utf8NoBOM by default.


⚠ ClamAV has degraded performance

In b0f6987, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 604 ms

Resolved: ClamAV performance has improved in 7a76a7c after 6 minutes.


Methods with dependencies are passed to IMethodInterceptor

TestNG Version
<IP_ADDRESS>
Expected behavior
As per IMethodInterceptor documentation :

Only methods that have no dependents and that don't depend on any other test methods will be passed in parameter.

Actual behavior
Methods with dependencies, e.g. marked with dependsOnMethods are passed in parameter.
Is the issue reproductible on runner?
I tested just on maven

[ ] Maven

Test case sample
https://github.com/abaxanean/testng-dependsOn

Why is it a problem? What if we just remove it from the documentation ?
@cbeust What was the idea behind this doc line ?


fix: changed size to boxSize

Closes #407
Description
size has been deprecated and has been modified.

Current behavior (updates)

New behavior

Is this a breaking change (Yes/No):

Additional Information

Thanks for review.
I fixed it.
Please review again.


AIRFLOW_83_add_mongo_hooks_and_operators (new new)

New PR for old PR that was closed with no merge (#2962)
Thanks for reviewing. I would appreciate any/all feedback as this is my first serious apache incubator-airflow PR.
Make sure you have checked all steps below.
JIRA
[ x ] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"
https://issues.apache.org/jira/browse/AIRFLOW-XXX
Description
[ x ] Here are some details about my PR, including screenshots of any UI changes:
AIRFLOW-83 requests a mongo hook and operator contribution. We had one laying around that I expanded to be more in line with Airflow practices.
Tests
[ x ] My PR adds the following unit tests OR does not need testing for this extremely good reason:
Adds 6 unit tests for the hook
Would love more insight on proper Operator testing if having no tests for it is unacceptable
Commits
[ x ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":
Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"
[ x ] Passes git diff upstream/master -u -- "*.py" | flake8 --diff
added small fix in setup.py as well for code not touched by this PR

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@3ed25a9). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master    #3440   +/-   ##
=========================================
  Coverage          ?   76.67%
=========================================
  Files             ?      203
  Lines             ?    15065
  Branches          ?        0
=========================================
  Hits              ?    11551
  Misses            ?     3514
  Partials          ?        0

Impacted Files
Coverage Δ

airflow/utils/db.py
33.33% <0%> (ø)

Continue to review full report at Codecov.

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

Thanks for picking this back up. Please use format "[AIRFLOW-XXX] My Airflow PR" for your commit.

@jgao54 Thanks for reviewing - I have updated code.rst

@jgao54 Any more comments on this? If not I will merge it.

lgtm, thanks!

@kaxil Excited to get this merged. Thanks for the review.

Thanks for your contribution @andscoop . Really appreciate it.


reply to allowed

hi there, i have try this bot and. awesome for self chat, but, when allow number to PM there’s no reply from the bot, and i check on mongoDB there’s no document/data insert into mongoDB, please advice me. many thanks. from me

Give more information. If possible attach screenshots.

{"status":"found","number":"628XXXXXXXXXX","times":1,"permit":true}
this files on modules/tempdata/628XXXXXXXXXX.json

Ping will not work for all, and it is perfectly fine, this use local db (Using fs 😁) and only update the database for some specified reasons.

Ping will not work for all, and it is perfectly fine, this use local db (Using fs 😁) and only update the database for some specified reasons.

i try another command but still no reply, in PM or inside of group

It is userbot bro

Read the command's wiki
https://github.com/TheWhatsBot/WhatsBot/wiki/Commands


Retry DB connection on startup

Got this failure:
lakefs_1    | panic: could not open DB: failed to connect to `host=postgres user=lakefs database=postgres`: server error (FATAL: the database system is starting up (SQLSTATE 57P03))

This looks like a race during startup, so need to retry at least on this error. This will also be useful for simplifying system automation.

lakeFS got more than one way to connect to the database, during lakefs run and lakefs setup the first call in case of the run goes to BuildDatabaseConnection and in case of lakeFS setup it goes using Migrate (migrate.NewWithSourceInstance).
The current retry connects works for our BuildDatabaseConnection but not for the setup process.


Base logic doesn't allow adding workflow templates

There is an event (OnInsertWorkflowTemplates) in codeunit 1502 (Workflow Setup) in the InsertWorkflowTemplates function. I've subscribed to this event, and after copying a number of base functions (see issue #928) it seems as though I will be able to add my own workflow templates.
Not so...............
The function InsertWorkflowTemplates is ONLY called when there are no existing workflow templates, which pretty much renders it impossible to easily add a new workflow template via extension as there will always be existing entries that are generated as part of the base code.

New event OnAfterInitWorkflowTemplates has been added to solve this issue


unable to fetch adoptopenjdk:11-jre-hotspot during publishing Docker image to Jfrog artifactory

Environment:

Jib version: 3.2.1
Build tool: Apache Maven 3.8.5
OS: Ubuntu

Description of the issue:
Facing issue while authenticating from jfrog artifactory for adoptopenjdk:11-jre-hotspot

Expected behavior:
The authentication should be through and maven build should publish Docker image to Jfrog artifactory. Sample below:

Steps to reproduce:

Maven build command: mvn -B -V -e -Dstyle.color=always clean package jib:buildTar
-Djib.from.image=jfrog-repo-path/adoptopenjdk:11-jre-hotspot
-Djib.outputPaths.tar=/home/runner/work/github-repo-name/docker-image.tar

jib-maven-plugin Configuration:
<plugin>
	<groupId>com.google.cloud.tools</groupId>
	<artifactId>jib-maven-plugin</artifactId>
	<version>3.2.1</version>
	<configuration>
		<from>
			<image>docker.org-name.com/adoptopenjdk:11-jre-hotspot</image>
		</from>
		<to>
			<image>docker-snapshots.org-name.com/org-name/path-to-repo/${project.artifactId}/${project.artifactId}:${project.version}-local</image>
		</to>
		<container>
			<jvmFlags>
				<jvmFlag>-noverify</jvmFlag>
				<jvmFlag>-XX:+AlwaysPreTouch</jvmFlag>
				<jvmFlag>-Djava.security.egd=file:/dev/./urandom</jvmFlag>
			</jvmFlags>
			<entrypoint>
				<command>bash</command>
				<arg>-c</arg>
				<arg>/entrypoint.sh</arg>
			</entrypoint>
			<environment>
				<SPRING_OUTPUT_ANSI_ENABLED>ALWAYS</SPRING_OUTPUT_ANSI_ENABLED>
			</environment>
			<creationTime>USE_CURRENT_TIMESTAMP</creationTime>
		</container>
		<extraDirectories>
			<paths>
				<path>src/main/jib/</path>
			</paths>
			<permissions>
				<permission>
					<file>/entrypoint.sh</file>
					<mode>755</mode>
				</permission>
			</permissions>
		</extraDirectories>
		<outputPaths>
			<tar>${project.build.directory}/${project.artifactId}-image.tar</tar>
			<digest>${project.build.directory}/${project.artifactId}-image.digest</digest>
			<imageId>${project.build.directory}/${project.artifactId}-image.id</imageId>
		</outputPaths>
	</configuration>
</plugin>

Log output:
Additional Information:

Hi, Any updates will be appreciated.

Hey @amanarch96, if Jib was able to retrieve credentials then it will say something like “Using credentials…” as described here: https://github.com/GoogleContainerTools/jib/blob/c419354aa42c4c8a2a32f93c1524d0b31b598979/docs/faq.md#what-should-i-do-when-the-registry-responds-with-unauthorized. You may also find this section on Authentication methods to be helpful. Additionally, we seem to be getting a 404 error (not found) instead of 401 (unauthorized) so it may not be an authentication issue. Please verify if the base image path has been entered correctly. To troubleshoot further, perhaps you could also try doing docker pull <image reference> to see if you're able to retrieve the image via docker cli.

I don't this is a Jib issue but something wrong in the jfrog artifactory.

Closing as part of cleanup. Please re-open if it's still relevant.


Fix expired listener and emitter

Checklist

[x] yarn run dist:ci passes on my machine
[x] I have followed the commit message guidelines, with messages suitable for appearing in the changelog

Description
My team has found this issue when investigating why file writes would fail with the following message after certain tests:

Turns out that any request that is marked as expired is causing a timeout to happen due to requestsCompleted never resolving. This happened mainly due two factors:

The event name for expired had a trailing space, which effectively made the listener wait for a non-existent event to happen. Removing the trailing space has fixed this issue;
The expired event was emitted before the relevant event was removed from the activeRequestIds, which would cause the listener to think there were still pending requests, and wait for the remaining requests to be resolved. Moving the event emitting to after the removal of the request from the array has fixed this issue.

Thanks in advance!

good spot, thanks for pull request @Yurickh
i’ll get it merged and released tomorrow

sorry I missed this as I was away at a conference!


v2.0.0

Refactor & update test scripts
Add support Referrer-Policy

Coverage decreased (-13.08%) to 86.916% when pulling c5de73ea276a79bf96a2c03311626620bd4d28f0 on dev into d06e5d758ba4e61ffcb355d33a285476c364054c on master.

Coverage decreased (-11.2%) to 88.785% when pulling 5c4f469624129841a2033779b5b1e2f7bda079ab on dev into d06e5d758ba4e61ffcb355d33a285476c364054c on master.


Unable to get the library to work with Kotlin 1.7.0

Is this library designed to be used with Kotlin 1.9.0? I get the following error when I add ohteepee dependency and compile my project.
~/.gradle/caches/modules-2/files-2.1/org.jetbrains.kotlin/kotlin-stdlib-common/1.9.0/cd65c21cfd1eec4d44ef09f9f52b6d9f8a720636/kotlin-stdlib-common-1.9.0.jar!/META-INF/kotlin-stdlib-common.kotlin_module:
Module was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.9.0, expected version is 1.7.1.

Hi @ashwinbhaskar thanks for trying our library.
I think this issue is not related to this library, appereantly this issue can happen when your kotlin-gradle-plugin's version is different than com.android.tools.build:gradle's plugin version.
Please check out this answer on stackoverflow to see if it works https://stackoverflow.com/a/74425347
As for now i'm closing this issue but the link i provide doesn't solve the issue you can open a new issue and we can investigate it more.


support :after and :before for insertion at particular place within an array

resolved and documented: https://github.com/cloudfoundry/docs-bosh/commit/350771bc1753e9eb28556202a6e21d2fe61c0bff


MixedRealityDictationSystem: Add ability to get audio clip for current dictation session

Overview
Adds a property AudioClip to IMixedRealityDictationSystem to allow devs to obtain an audio clip recording of the current dictation session. Also adds a few properties to dictation handler (IsListening, AudioClip)
Changes

Fixes: #8226
Breaking change: Addition of property AudioClip in IMixedRealityDictationSystem
Documents the breaking change in draft release note

/azp run

/azp run

@MaxWang-MS can you also add a note to the breaking changes section:
https://github.com/microsoft/MixedRealityToolkit-Unity/blob/mrtk_development/Documentation/ReleaseNotes.md#breaking-changes
This way we don't lose track of it during the 2.5 release

/azp run

@MaxWang-MS can you also add a note to the breaking changes section:
https://github.com/microsoft/MixedRealityToolkit-Unity/blob/mrtk_development/Documentation/ReleaseNotes.md#breaking-changes
This way we don't lose track of it during the 2.5 release

Done!


Provide an API to read/write Image pixels with borrows.

What problem does this solve or what need does it fill?
Today, the primary (only?) way to access the pixels of an image (reading or writing) is to first remove the Image from the Assets<Image>, call Image::try_into_dynamic, then read or mutate the image, then put the image back into the Assets<Image> resource.
What solution would you like?
We can create our own ImageView struct which will store a borrow (or mutable borrow) to an Image (for "reasonable" image types - most of what DynamicImage currently supports). We can then implement the image::GenericImage and image::GenericImageView traits which will allow the image crate to do its job. We can also probably have our own functions get_color and set_color to make it easier to use with Bevy stuff.

Do you mean a trait that makes it possible to read,write and modify images with the image crate?
If that's so i think it shouldn't be hard to implement
(New to contributing please if I am doing something wrong I apologize)

Do you mean a trait that makes it possible to read,write and modify images with the image crate?
If that's so i think it shouldn't be hard to implement
(New to contributing please if I am doing something wrong I apologize)

The image crate already contains the trait we should implement: image::GenericImageView and image::GenericImage. What we need is something like a wrapper type around &Image or &mut Image that implements these traits, so we can reuse all the stuff the image crate provides, while also avoiding the need to use ImageBuffer or DynamicImage.
It doesn't seem too hard to implement, I started it a little bit and got a little confused about how the image crate expects those traits to be implemented. Feel free to pick this up @Cedar-81!

 I worked with the image crate recently maybe I could help.

I didn't get very far at all, but here's roughly what I was thinking: https://gist.github.com/andriyDev/c846454c87869963f51131de8ec38bdf

I think this is basically just fixed by #10392

@andriyDev  - What I'd like to be able to do to modify a image using the image crate is
let buffer: &mut image::ImageBuffer = images.get_mut(&draw_image_id).expect("image not present").into_buffer();

That would let me modify the image using methods like subimage() and pixels_mut() and know they were stored back into images without a copy of the whole image being made. I don't think that's possible currently, is it? Using try_to_dynamic() copies, and sticking to the bevy::Image limits the functionality available.
Am I wrong? If not, can we reopen this issue?


introduce bingo and rework makefiles

Description
This PR does following:

introduce bingo for versioning of our tooling
move common Makefile parts to .make folder and include it into all other Makefiles
replace rsync step in doc generation with mutagen in order to enable hot reload during doc editing
change from protoc to buf protoc in order to make this also manageable with bingo

Related Issue
Motivation and Context
Every developer uses a different version of tooling, this is solved by bingo.
Makefiles share a lot of code and changing a line is a lot of work to ensure it is changed in every Makefile, this is solved by includes.
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

My last idea was that maybe only the files/lines which were changed in this PR are analyzed but this also doesn't seem to be true...

The golangci-lint stuff works in my example project.
My next attempt would be to execute the sonar-scanner locally with the generated reports to see if the reports are incorrect or something else.


Landing Page

@CodiumAI-Agent /review

@CodiumAI-Agent /review

PR Review

 ⏱ Estimated effort to review [1-5]
3, because the PR involves both CSS and JSX changes across multiple files, including significant styling updates and structural changes in the JSX components. Reviewing these changes requires a good understanding of CSS best practices, responsiveness, and React component structure.

 🧪 Relevant tests
No

 🔍 Possible issues

Possible Bug: The removal of <Navbar /> and the commented-out <Routes> in App.jsx might affect navigation and routing within the application. Ensure that the application's navigation works as expected without these components.

Performance Concern: The use of high-resolution images as background images in CSS might slow down page loading times, especially on slower internet connections. Consider optimizing these images or using a CSS gradient as a fallback.
 🔒 Security concerns
No

 Code feedback:
relevant fileFrontend/src/App.css
suggestion

Consider using CSS variables for colors and fonts to improve maintainability and readability of your CSS code. For example, define --primary-color: #F3B9BE; and use it as color: var(--primary-color); [important]

relevant linecolor: #F3B9BE;
relevant fileFrontend/src/App.css
suggestion

For better responsiveness and mobile-friendliness, consider using relative units like vw, vh, em, or % instead of px for margins, padding, and font sizes. [important]

relevant linepadding: 20px;
relevant fileFrontend/src/App.jsx
suggestion

If the <Navbar /> component is no longer needed, it's a good practice to also remove its import statement to keep the code clean and avoid confusion. [medium]

relevant line-import Navbar from "./components/Navbar"
relevant fileFrontend/src/components/Home.jsx
suggestion

Use className instead of class for defining CSS classes in JSX to follow React conventions and avoid console warnings. [important]

relevant lineLOGIN

 ✨ Review tool usage guide:
Overview:
The review tool scans the PR code changes, and generates a PR review. The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on any PR.
When commenting, to edit configurations related to the review tool (pr_reviewer section), use the following template:
/review --pr_reviewer.some_config1=... --pr_reviewer.some_config2=...

With a configuration file, use the following template:
[pr_reviewer]
some_config1=...
some_config2=...

  Utilizing extra instructions
The review tool can be configured with extra instructions, which can be used to guide the model to a feedback tailored to the needs of your project.
Be specific, clear, and concise in the instructions. With extra instructions, you are the prompter. Specify the relevant sub-tool, and the relevant aspects of the PR that you want to emphasize.
Examples for extra instructions:
[pr_reviewer] # /review #
extra_instructions="""
In the 'possible issues' section, emphasize the following:
- Does the code logic cover relevant edge cases?
- Is the code logic clear and easy to understand?
- Is the code logic efficient?
...
"""

Use triple quotes to write multi-line instructions. Use bullet points to make the instructions more readable.

  How to enable\disable automation

When you first install PR-Agent app, the default mode for the review tool is:

pr_commands = ["/review", ...]

meaning the review tool will run automatically on every PR, with the default configuration.
Edit this field to enable/disable the tool, or to change the used configurations

  Auto-labels
The review tool can auto-generate two specific types of labels for a PR:

a possible security issue label, that detects possible security issues (enable_review_labels_security flag)
a Review effort [1-5]: x label, where x is the estimated effort to review the PR (enable_review_labels_effort flag)

  Extra sub-tools
The review tool provides a collection of possible feedbacks about a PR.
It is recommended to review the possible options, and choose the ones relevant for your use case.
Some of the feature that are disabled by default are quite useful, and should be considered for enabling. For example:
require_score_review, require_soc2_ticket, require_can_be_split_review, and more.

  Auto-approve PRs
By invoking:
/review auto_approve

The tool will automatically approve the PR, and add a comment with the approval.
To ensure safety, the auto-approval feature is disabled by default. To enable auto-approval, you need to actively set in a pre-defined configuration file the following:
[pr_reviewer]
enable_auto_approval = true

(this specific flag cannot be set with a command line argument, only in the configuration file, committed to the repository)
You can also enable auto-approval only if the PR meets certain requirements, such as that the estimated_review_effort is equal or below a certain threshold, by adjusting the flag:
[pr_reviewer]
maximal_review_effort = 5

  More PR-Agent commands

To invoke the PR-Agent, add a comment using one of the following commands:

/review: Request a review of your Pull Request.
/describe: Update the PR title and description based on the contents of the PR.
/improve [--extended]: Suggest code improvements. Extended mode provides a higher quality feedback.
/ask <QUESTION>: Ask a question about the PR.
/update_changelog: Update the changelog based on the PR's contents.
/add_docs 💎: Generate docstring for new components introduced in the PR.
/generate_labels 💎: Generate labels for the PR based on the PR's contents.
/analyze 💎: Automatically analyzes the PR, and presents changes walkthrough for each component.

See the tools guide for more details.
To list the possible configuration parameters, add a /config comment.

See the review usage page for a comprehensive guide on using this tool.