Reimport ZTF with larger ~100 mb parquet file sizes.

Reimport ZTF with larger ~100 mb parquet file sizes (on epyc, PSC). This is to reduce the number of partitions.

Note ztf_zource instead of ztf_source.
catalog_dir = "/data3/epyc/data3/hipscat/catalogs/ztf_axs/ztf_zource/"

healpix orders: [2 3 4 5 6 7 8 9]
num partitions: 41679
------
min size_on_disk: 0.00
max size_on_disk: 0.46
total size_on_disk: 8343.36

small-ish 	: 2779 	(6.7 %)
sweet-spot 	: 38900 	(93.3 %)

and a histogram of leaf parquet file sizes (units are GB)


Add github token check

I use a script to retrieve the token from a keychain, I have fumbled with the command and assigned a wrong value to the github API token. The GitHub queries then failed but the error returned by the cli tool was confusing, took me a bit to understand where the problem was.
This check ensures that the Github token retrieved is syntactically correct (a-z0-9).
cc: @spastorino

I'm not sure if I'd "validate" the token in that way but :+1:, I meant, whatever :)
cc @Mark-Simulacrum

I'm not sure if I'd "validate" the token in that way but +1, I meant, whatever :)

right, only "syntactically correct".
What would you otherwise suggest to validate the token (without performing an actual API call)?
Anyway, it's just a papercut so it's ok to simply close it :+1:  (I just thought to point it out)


cpptools needs an override

cpptools downloads a binary and it's not executable or able to run.
❯ readlink -f /home/fmzakari/.vscode/extensions/ms-vscode.cpptools/debugAdapters/bin/OpenDebugAD7

/nix/store/cx9451yl9icy2zyh9shggq5vq8j0gy1c-vscode-extension-ms-vscode-cpptools-1.21.1/share/vscode/extensions/ms-vscode.cpptools/debugAdapters/bin/OpenDebugAD7

Nixpkgs has special handling for it (https://github.com/NixOS/nixpkgs/blob/nixos-24.05/pkgs/applications/editors/vscode/extensions/ms-vscode.cpptools/default.nix)

@fzakaria, there's quite a lot of code, and I don't want to duplicate it. I suggest to use the version from nixpkgs. Alternatively, make a fork of nixpkgs, update the extension there, and use that version.

okay

Should the extension be removed at least so people don't install it?
On Fri, Aug 23, 2024, 9:38 AM Danila Danko @.***> wrote:

Closed #69
https://github.com/nix-community/nix-vscode-extensions/issues/69 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/nix-community/nix-vscode-extensions/issues/69#event-13995515713,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAETXDRNIWUMJJAL44ZAIOLZS5QPJAVCNFSM6AAAAABLBK6FBSVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTHE4TKNJRGU3TCMY
.
You are receiving this because you were mentioned.Message ID:
<nix-community/nix-vscode-extensions/issue/69/issue_event/13995515713@
github.com>

@fzakaria, nice idea! I'll provide a mechanism for removing particular extensions and will remove this extension.

Good call on removing the extension. However, it would be nice if upon trying to install it, nix would throw an error message during eval indicating what's wrong. error: attribute 'cpptools' missing isn't super descriptive as it stands.

Good call on removing the extension. However, it would be nice if upon trying to install it, nix would throw an error message during eval indicating what's wrong. error: attribute 'cpptools' missing isn't super descriptive as it stands.

+1 just ran into this and wouldn't have figured it out without searching cpptools in this repo

+2, just got hit with this


firefox: Clicking on nav bar doesn't work

"percent must be a number. got: NaN" core.js:38:8
"target slide is same as current slide" core.js:56:6

Fixed


Debugger breakpoint not hit in Chrome

Do you want to request a feature or report a bug?
Bug

What is the current behavior?
Debugger breakpoint is only hit in Firefox.
If the current behavior is a bug, please provide the steps to reproduce.

Clone webpack-source-maps (files shamelessly stolen from https://github.com/webpack/webpack/issues/4423#issue-211890851)
run npm install && npm start
Set a breakpoint in browser inspector on line 10 in webpack://index.js?2645 (the console.log statement).

What is the expected behavior?
The breakpoint should be hit in Chrome too.
As mentioned in #3165  there are some issues with source maps in Chrome, and this seems to be one of them.
If this is a feature request, what is motivation or use case for changing the behavior?
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.

Firefox 53
Chrome 58
webpack 2.5.1
node 6.10.1
Windows 10

I don't think that this is a webpack issue..

@vankop Can you clarify?

Looks like it was chrome bug, in Version 85.0.4183.83 (Official Build) (64-bit) works fine, let's close issue, anyway if you faced with this issue again feel free to open the new issue with reproducible test repo, feel free to feedback


Include yParity field in Transaction objects (type 1 and 2)

Describe the bug
Seems like a lot (all?) of the clients, reth included, are returning v instead of yParity in their JSON representation of EIP-2718 type 0x1 (EIP-2930) and 0x2 (EIP-1559) transactions.
Recent discussion on Eth R&D discord here: https://discord.com/channels/595666850260713488/745077610685661265/1129684798340878457
The execution-api schemas actually do point this out: https://github.com/ethereum/execution-apis/blob/main/src/schemas/transaction.yaml#L157 but this wasn't noticed until recently, seems like most EL clients followed geth's impl and it wasn't until recently that when an EL correctly followed the spec it was noticed by Micah that this breaks Prysm (and presumably other CLs as it expects the v field).
Micah also opened a tracking issue in geth here: https://github.com/ethereum/go-ethereum/issues/27727
As mentioned there, I think the safest short-term behavior is to return both v and yParity on these tx types, then eventually drop v for these tx types (keeping it of course for Legacy and EIP-155 txs).
I suspect this will require customizing the serde (de)serialization of Transaction type here: https://github.com/paradigmxyz/reth/blob/085a703d7c8f7d7bbb310f8cbcda3632b7f85ad9/testing/ef-tests/src/models.rs#L392
Steps to reproduce
curl any RPC that returns a type 1 or 2 transaction object, for example this random Sepolia tx https://sepolia.etherscan.io/tx/0xbad8ca05c9f95ea37cdb9f6353579fcf9a8c6b9604fe8188585ef0252eba1c90:
curl -X "POST" http://localhost:8545 \
     -H 'Content-Type: application/json' \
     -d $'{
  "id": 1,
  "method": "eth_getTransactionByHash",
  "jsonrpc": "2.0",
  "params": [
    "0xbad8ca05c9f95ea37cdb9f6353579fcf9a8c6b9604fe8188585ef0252eba1c90"
  ]
}'
{"jsonrpc":"2.0","result":{"hash":"0xbad8ca05c9f95ea37cdb9f6353579fcf9a8c6b9604fe8188585ef0252eba1c90","nonce":"0x156","blockHash":"0xa7d4c8ca72c74e54e0f3629949320b52bd679e699f90b264b545f8897b48d1a9","blockNumber":"0x1621a1","transactionIndex":"0x4","from":"0x6cc9397c3b38739dacbfaa68ead5f5d77ba5f455","to":"0x855847fc07879844c64ee1af1c65d80b0d6893b6","value":"0x1bc16d674ec80000","gasPrice":"0x59682f07","gas":"0x5208","maxFeePerGas":"0xee6b2800","maxPriorityFeePerGas":"0x59682f00","input":"0x","r":"0xc569c92f176a3be1a6352dd5005bfc751dcb32f57623dd2a23693e64bf4447b0","s":"0x1a891b566d369e79b7a66eecab1e008831e22daa15f91a0a0cf4f9f28f47ee05","v":"0x1","chainId":"0xaa36a7","accessList":[],"type":"0x2"},"id":1}

The signature portion should be: { ... "r":"0xc569c92f176a3be1a6352dd5005bfc751dcb32f57623dd2a23693e64bf4447b0","s":"0x1a891b566d369e79b7a66eecab1e008831e22daa15f91a0a0cf4f9f28f47ee05","yParity":"0x1" ... } rather than "v":"0x1".
Node logs
No response
Platform(s)
No response
What version/commit are you on?
90b6d006d1fc660c4883a4335e42f8e760c4adbe (latest main)
What database version are you on?
No response
If you've built Reth from source, provide the full command you used
No response
Code of Conduct

[X] I agree to follow the Code of Conduct

Looks like we would need to change the Signature type and add a yParity flag for non-legacy transactions. We mirrored geth's behavior for this and just put the y parity bit in the v field:
https://github.com/paradigmxyz/reth/blob/99a8e0f98211be6afb49e8e04ca61ddd7e352583/crates/rpc/rpc-types/src/eth/transaction/signature.rs#L22-L39


Error decoding firmware version E1.00

Error decoding firmware version E1.00.
Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/goodwe/es.py", line 197, in read_device_info
    self.dsp1_version = int(self.firmware[0:2])
                        ^^^^^^^^^^^^^^^^^^^^^^^
ValueError: invalid literal for int() with base 10: 'E1'

Name: Grid-Tied PV Inverter
Type: GW10K-DT

The problem is most likely not in decoding, butin inverter identification.
You have DT inverter, but the ES inverter code is being executed.
Does this problem  still exist ?

Just for refence, most likely the same problem as here - https://github.com/home-assistant/core/issues/95767


Same ssh host key on all images?

Hi,
I was looking at the image setup script and it looks like the ssh host key is not deleted, this means someone who has enabled ssh can not be sure that he is always connected to the same machine. The risk is medium, because this kiosk system is most of the time probably not reachable via the internet, but a notice in the docs would be great.
An instruction on how to regenerate host keys can be found on https://www.cyberciti.biz/faq/howto-regenerate-openssh-host-keys/.
Cheers

Hey,
Thanks for paying attention to security, these tips are always welcome!
In addition to the systems usually not being reachable over the internet, the temporary SSH key is in fact removed by the script:
https://github.com/futurice/chilipie-kiosk/blob/818a81335f8ec75e367b60339d1a481613e5db63/docs/image-setup.sh#L178
Similarly, SSH is actually disabled entirely, until the user specifically opts into re-enabling it.

I'm talking about the host key which is used to identify the server. Not the client key. If someone knows the host key, he could fake the server, aka man in the middle. I think the risk that someone is interfering the connection is low, but the fix should be easy, so maybe I create a pr later.

Ah, sorry, read your original comment in a hurry! That's indeed the case.
You're right, regenerating the key on first boot might be a good idea!

I need to verify this, but it should be enough to enable the regenerate_ssh_host_keys service.
/bin/systemctl enable regenerate_ssh_host_keys

I will create a pr after I gave it a try.

Merged, thanks a lot! :bowing_man:


Where are do Admin LTE folder for step 6 installation

Hi bro, where are do the admin lte folder for step 6. I do not see. Please help.
Publish config file & generator template files.
php artisan vendor:publish --provider="Goszowski\CrudGeneratorAdminLte\CrudGeneratorAdminLteServiceProvider::class"
Note: You should have configured database for this operation.
Copy the Admin LTE folder to public path and rename it to "admin-lte"

You must download it from admin LTE website


FR: add sendEmailVerification, sendPasswordResetEmail, sendSignInLink, etc.

Environment

Operating System version: OSX 10.13.3 and Debian 9
Firebase SDK version: firebase-tools -V shows 3.18.4
Library version: 2.10.0
Firebase Product: auth

Hey all, I would like my server to be able to trigger/send those various email authentication links.
See related: https://github.com/firebase/firebase-admin-node/issues/46

Also related: https://github.com/firebase/firebase-admin-node/issues/241

Anyone know if there are plans to implement the email actions?

Email action links API was implemented in #258

@hiranya911 Question - are the 'send' going to be implemented soon? Those only seem to generate links. Or am I off?

These APIs are only for generating links. We don't have plans in the short term to send customized emails from the Admin SDK.

@bojeil-google What about a default email? Is there a way to send an email (I do not care about customization) directly?

There are no plans for that as you can already do it with the client SDK. The Node.js client SDK can do that. The value here is in adding the capability to build customized emails which is not possible in our client SDKs. A lot of developers want this capability which is now possible through these new APIs. Ideally it would be great if we can send the customized emails for the developer too but we are not there yet.

@bojeil-google also, is there a way to send a password reset email (not link generation) through the python sdk? I do not care about customization in any way.

BTW, the REST API is documented here: https://firebase.google.com/docs/reference/rest/auth/#section-send-password-reset-email

Thank you very much @bojeil-google . So we would just need to make a REST API call from Python? For example using 'http://docs.python-requests.org/en/master/' or urllib.request ?

It's up to you.

Is this still relevant? I'm happy to work on this if it is.

I made an initial PR: #368

Thanks for looking into this Caleb. I've assigned your PR to one of our Auth team members. Stay tuned for an update.

Any updates?

This is pretty bizarre that it's not in the official API.


This example does not work after exporting on Keil µVision 5

The example works well on Arm mbed Online Compiler but after exporting on Keil and building, it says at least two errors:

/filer/workspace_data/exports/8/8bbbb9b5d0fe7814b65b507b54bb7d92/mbed-os-example-blinky/mbed_config.h contains an incorrect path

After building it make at least one error which is underlined in #include "mbed.h". It says: error in include chain (mbed_assert.h): invalid token at start of a preprocessor expression

ARM Internal Ref: MBOTRIAGE-1514

It is the same issue on nRF52-DK as well as on STM32 Discovery kit IoT node. So we can't use Keil for importation of Mbed projects : This is a major issue !

@Navis-Raven Have you tried using mbed-cli to export mbed-os-example-blinky to uVision 5?
It's strange that this isn't working for you considering we explicitly test exporting examples to uVision 5.

I have a similar issue while trying to use it with IAR IDE for Arm 8.30.1 (and for STM32 Discovery kit IoT node). Similar issue, it does not compile, and we have also 145 errors. This issue is quite annoying we cannot go anywhere else with this (changing the hardware or the IDE does not change anything). (I'm on Windows 7)

@cmonr
I didn't unederstand what mbed-cli is used for, and its usage is more complex. Adding to that the installation of mbed-cli crashed at the end of it, so, I decided to uninstall it.
I opened this issue here about mbed-cli installation problem
https://github.com/ARMmbed/mbed-cli/issues/733

It does not work on VSCode (ARM-GCC) either but not for the same reason, I have to open a new issue.
At this time I have no IDE option to run mbed API for any of these two hardware (nRF52-DK and STM32 Discovery kit IoT node)

@Navis-Raven Would you be able to check if this works with earlier releases and/or different targets?

I'm working only with earlier releases, because I've just installed these tools last week in order to try them.
As you can see I've tried with two different targets and I have similar issues. I have no more Arm based Target.

Some additionnal info Keil µVision 5 with nrF52DK

Hi, I will review the 2 issues regarding exporters, and will report back the status.

@Navis-Raven The workaround is to change the path of "mbed_config.h" to be in the root of your project. I'm looking at why you're getting the filer path within your project.

Hey @theotherjimmy didn't we talk about this issue with the online compiler the other day?

@theotherjimmy Here's the GitHub issue we were talking about: https://github.com/ARMmbed/mbed-os/issues/7866

I am pretty sure that I'm watching this git issue only because I raised a jira on it, just recently. Is this best done in jira?

Remove the * to fix assert error. I am not sure if thats the offical solultion but its no longer showing that error.

@EngSocProject Yes that's strange. I'm not sure either what the problem even is there. The header file is just trying to construct a version number. Further, if you're using uVision, you're probably using Arm Compiler 5. That section looks like it's setup for GCC.

Hi @EngSocProject , since I've also seen similar issues, I can "try" help.

You want to go into linker options, and turn on "Use memory layout from target dialog", and make sure you have debugging info turned off.
But you will get stuck again on running out of flash/RAM anyway, so I suspect there is an issue with the exporter, and if already familiar with Keil, you might do better mileage creating a project virtually from scratch. (try run > mbed export -m LPC1768 -i uvision5 --profile RELEASE)

What worked for me was using the commandline however:
c:\mbed-os-example-blinky-oob> mbed compile -m LPC1768 -c -t GCC_ARM
...
| Module           |         .text |       .data |        .bss |
|------------------|---------------|-------------|-------------|
| [fill]           |     615(+615) |       4(+4) |     12(+12) |
| [lib]\c.a        | 18671(+18671) | 2472(+2472) |     89(+89) |
| [lib]\gcc.a      |   3864(+3864) |       0(+0) |       0(+0) |
| [lib]\misc       |     208(+208) |     12(+12) |     28(+28) |
| main.o           |     198(+198) |       4(+4) |     24(+24) |
| mbed-os\drivers  |       56(+56) |       0(+0) |       0(+0) |
| mbed-os\hal      |   2005(+2005) |       4(+4) |     68(+68) |
| mbed-os\platform |   2903(+2903) |   260(+260) |   133(+133) |
| mbed-os\rtos     |   8312(+8312) |   168(+168) | 6053(+6053) |
| mbed-os\targets  |   3538(+3538) |       4(+4) |   241(+241) |
| Subtotals        | 40370(+40370) | 2928(+2928) | 6648(+6648) |
Total Static RAM memory (data + bss): 9576(+9576) bytes
Total Flash memory (text + data): 43298(+43298) bytes

I understand you will want to use Keil and the debugger, but hope this gives you some route forward. Core of the matter is the online/offline exporter is problematic on some targets, for now export to targets I have in my box of targets here work, so mileage will vary.

...same problems...
in main.cpp errore in include chain (mbed_assert.h): invalid token at start of a preprocessor expression
in mbed_assert.h line 100 and 103 error invalid token at start of a preprocessor expression

Just ran into this issue as well. The weird thing is, two of my newly exported projects were working totally fine, then all of a sudden, after diner, when I resumed work, both of them started to have this issue.

The original issue was that the exported Keil project file had errors in it, and so one needs to separate case of the file paths being hard-coded at export-time from other cases of missing files which may have a cause other than the exporter. @zhiyong80 If you di find how this went wrong as a separate cause, please do raise a fresh issue.
As noted, in this thread, its not just Keil exports, the same issue occurs for export gcc makefile.  @Navis-Raven ? Did the editing the project solution resolve this for you? If So I'd like to close this ticket, and refer people going forward to the more current ticket https://github.com/ARMmbed/mbed-os/issues/8277 , so we can focus effort in one place going forward.

@ConradBraam
Hi, I don't use Keil µV5 (or ARM mbed online compiler) because of these problems I had. I've uninstalled Keil µV5 since then. I've switched to VSCode with platformIO. So unfortunately I can't help you for the tests. If you want to test a solution you just have to repeat the protocol

@ConradBraam you are right there was indeed something else. I deleted a very old bin file on mbed drive thinking of it was not needed because I was downloading code from Keil anyway. That turned out to be the cause of my problem which was downloaded code didn't run after downloading.
Regarding the issue of this thread, I did and still do see the error, but that seems not to affect building nor built code. So I will leave with it for now.

Project HSP_RPC_GUI for MAX32620. Mbeg creates a huge single   string include record (about 2.5kB) for Keil include paths. Obviously, compiler could not handle it and that results in many "file not found" errors.
If manually recreate all include records one record per include it works.

Mbed sets incorrect include paths:
;/HSP;/HSP/Devices;/HSP/Devices/BMP280;/HSP/Devices/BMP280/BMP280;/HSP/Devices/HspLed;/HSP/Devices/HspLed/HspLed;/HSP/Devices/LIS2DH;/HSP/Devices/LIS2DH/LIS2DH;/HSP/Devices/MAX14720;/HSP/Devices/MAX14720/MAX14720;/HSP/Devices/MAX30001;/HSP/Devices/MAX30001/MAX30001;/HSP/Devices/MAX30101;/HSP/Devices/MAX30101/MAX30101;/HSP/Devices/MAX30205;/HSP/Devices/MAX30205/MAX30205;/HSP/Devices/PushButton;/HSP/Devices/S25FS256;/HSP/Hsp_BLE;/HSP/Hsp_BLE/BluetoothLE;/HSP/Interfaces;/HSP/LoggingService;/HSP/QuadSPI;/HSP/RpcServer;/HSP/System;/HSP/Test;/USBDevice;/USBDevice/USBAudio;/USBDevice/USBDevice;/USBDevice/USBHID;/USBDevice/USBMIDI;/USBDevice/USBMSD;/USBDevice/USBSerial;/usr/src/mbed-sdk;mbed-os;mbed-os/cmsis;mbed-os/drivers;mbed-os/events;mbed-os/events/equeue;mbed-os/features;mbed-os/features/FEATURE_BLE;mbed-os/features/FEATURE_BLE/ble;mbed-os/features/FEATURE_BLE/ble/services;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE/hci;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE/hci/maxwsn;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE/stack;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE/stack/cfg;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE/stack/include;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE/util;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE/wsf;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE/wsf/generic;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE/wsf/include;mbed-os/features/frameworks;mbed-os/features/frameworks/greentea-client;mbed-os/features/frameworks/greentea-client/greentea-client;mbed-os/features/frameworks/unity;mbed-os/features/frameworks/unity/unity;mbed-os/features/frameworks/utest;mbed-os/features/frameworks/utest/utest;mbed-os/features/mbedtls;mbed-os/features/mbedtls/inc;mbed-os/features/mbedtls/inc/mbedtls;mbed-os/features/mbedtls/platform;mbed-os/features/mbedtls/platform/inc;mbed-os/features/netsocket;mbed-os/hal;mbed-os/hal/hal;mbed-os/hal/storage_abstraction;mbed-os/platform;mbed-os/rtos;mbed-os/rtos/rtx/TARGET_CORTEX_M;mbed-os/targets/TARGET_Maxim;mbed-os/targets/TARGET_Maxim/TARGET_MAX32620;mbed-os/targets/TARGET_Maxim/TARGET_MAX32620/TARGET_MAX32620HSP;mbed-os/targets/TARGET_Maxim/TARGET_MAX32620/device
Correct paths is:
.\HSP\Interfaces;.\HSP\QuadSPI;.\HSP\LoggingService;.\HSP\Devices\MAX30001\MAX30001;.\HSP\Devices\LIS2DH\LIS2DH;.\HSP\Devices\HspLed\HspLed;.\HSP\RpcServer;.\HSP\System;.\HSP\Devices\MAX30205\MAX30205;.\HSP\Devices\S25FS256;.\USBDevice\USBDevice;.\HSP\Hsp_BLE\BluetoothLE;.\HSP\Devices\PushButton;.\HSP\Hsp_BLE;.\HSP\Devices\LIS2DH;.\HSP\Devices\MAX30001;.\HSP\Devices\BMP280;.\HSP\Devices\HspLed;.\HSP\Test;;/HSP;/HSP/Devices;/HSP/Devices/BMP280;/HSP/Devices/BMP280/BMP280;/HSP/Devices/HspLed;/HSP/Devices/HspLed/HspLed;/HSP/Devices/LIS2DH;/HSP/Devices/LIS2DH/LIS2DH;/HSP/Devices/MAX14720;/HSP/Devices/MAX14720/MAX14720;/HSP/Devices/MAX30001;/HSP/Devices/MAX30001/MAX30001;/HSP/Devices/MAX30101;/HSP/Devices/MAX30101/MAX30101;/HSP/Devices/MAX30205;/HSP/Devices/MAX30205/MAX30205;/HSP/Devices/PushButton;/HSP/Devices/S25FS256;/HSP/Hsp_BLE;/HSP/Hsp_BLE/BluetoothLE;/HSP/Interfaces;/HSP/LoggingService;/HSP/QuadSPI;/HSP/RpcServer;/HSP/System;/HSP/Test;/USBDevice;/USBDevice/USBAudio;/USBDevice/USBDevice;/USBDevice/USBHID;/USBDevice/USBMIDI;/USBDevice/USBMSD;/USBDevice/USBSerial;/usr/src/mbed-sdk;mbed-os;mbed-os/cmsis;mbed-os/drivers;mbed-os/events;mbed-os/events/equeue;mbed-os/features;mbed-os/features/FEATURE_BLE;mbed-os/features/FEATURE_BLE/ble;mbed-os/features/FEATURE_BLE/ble/services;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE/hci;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE/hci/maxwsn;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE/stack;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE/stack/cfg;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE/stack/include;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE/util;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE/wsf;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE/wsf/generic;mbed-os/features/FEATURE_BLE/targets/TARGET_Maxim/exactLE/wsf/include;mbed-os/features/frameworks;mbed-os/features/frameworks/greentea-client;mbed-os/features/frameworks/greentea-client/greentea-client;mbed-os/features/frameworks/unity;mbed-os/features/frameworks/unity/unity;mbed-os/features/frameworks/utest;mbed-os/features/frameworks/utest/utest;mbed-os/features/mbedtls;mbed-os/features/mbedtls/inc;mbed-os/features/mbedtls/inc/mbedtls;mbed-os/features/mbedtls/platform;mbed-os/features/mbedtls/platform/inc;mbed-os/features/netsocket;mbed-os/hal;mbed-os/hal/hal;mbed-os/hal/storage_abstraction;mbed-os/platform;mbed-os/rtos;mbed-os/rtos/rtx/TARGET_CORTEX_M;mbed-os/targets/TARGET_Maxim;mbed-os/targets/TARGET_Maxim/TARGET_MAX32620;mbed-os/targets/TARGET_Maxim/TARGET_MAX32620/TARGET_MAX32620HSP;mbed-os/targets/TARGET_Maxim/TARGET_MAX32620/device;.\HSP\Devices\BMP280\BMP280;.\USBDevice\USBSerial;.\HSP\Devices\MAX14720\MAX14720;.\HSP\Devices\MAX30101\MAX30101;.\HSP\Devices\MAX30101;.\HSP\Devices\MAX30205;.\HSP\Devices\HspLed;.\HSP\Devices\MAX14720

@Kabron287 Thanks for sharing. What is difference in the above? I see only  .\ added  ? I recall we used to have it but was refactored (shorten paths, not needed).

If manually recreate all include records one record per include it works.

What is "one record" in uvision project file?

@Martin
I do not know what the difference. I am quite satisfied that it works.
And with mbed notation, even attempt to edit this record causes Keil crash.
"One record" means the following

So, the fast solution is:

Copy the whole include path string from the C/C++ tab to clipboard(RightMouseClick Select All, Copy)
Paste the sting into editor
in Editor Ctrl+H search for ;/ replace to *;.* Copy string back to clipboard
Paste clipboard content back in the C/C++ tab
Enjoy

@Kabron287 Can you report this to Mbed OS repository, this is useful. I haven't seen this error yet, so would like to step-by-step how to reproduce and fix as above. Tools team will review

I did it.

Looks like @Kabron287 opened this one:
https://github.com/ARMmbed/mbed-os/issues/10682
I'll be closing this issue as a duplicate. We'll track it under mbed-os issue.


Collection.watch is not implimented

hi there. just wondering if i could get some help adding collection and database watching as per https://docs.mongodb.com/manual/changeStreams/
its functionality i need for my project and i am already using this library elsewhere and would like to continue to use as it seems quite reliable and extensive.

This would be a great addition.


Feature/integrationtest

Write integration tests with Capybara gem.
Secure app from n+1 problems.
Users Before N+1 fix and After N+1 fix.
Users before and after fixing n+1

Posts before fixing the n+1

Posts after fixing the n+1

Hi @Izebiafe, it would be nice to resolve the merge conflct error on your pr


Example: Add porting example for CUPS generic PCL driver to PAPPL

One of the goals for 1.0 is to provide an example printer application for the CUPS rastertohp filter and corresponding PPD files (laserjet.ppd, deskjet.ppd, and generpcl.ppd). This example should include:

[x] A new PAPPL-based PCL driver
[x] Code to load and save the configuration (call existing APIs)
[x] Command-line and web-based configuration (web-based UI is mostly done already)
[ ] Snapcraft packaging template
[ ] systemd service file/support

OK, thanks to @JaiLuthra1 and PRs #26 and #32, the bulk of this is now done. Still need the templates for snapcraft and systemd, and to do the documentation portion that walks the reader through the transformation.

OK, all of this is now in the hp-printer-app project!


About Video-LLaVA

Thank you for your work.
I was wondering if there is any chance to open-source video-related tasks.
Thank you!

Thanks for your interest in our work. Our FasterVLM with Video-LLaVA has been released at the video branch. Feel free to check it out, and if you have any further questions, please don't hesitate to ask! 😁


I get  a error when run the  train.py   Layer weight shape (3, 3, 3, 64) not compatible with provided weight shape (64, 3, 3, 3)

Layer weight shape (3, 3, 3, 64) not compatible with provided weight shape (64, 3, 3, 3)

My keras version is 2.0.5 . who can help solve de problem

whats ur backend for keras there?
I guess u should try keras version 2.0.9 or later?


Analyzer should warn when "dart:io" and "dart:html" are both available in the same library

Isn't that supported now on server-side code?
dart:io on browser code doesn't make sense of course.

I haven't seen yet an embedder that expose both of these library.
For the context I got an issue reported on a package (https://github.com/a14n/dart-js-wrapping/issues/8#issuecomment-215124967) where the issue was caused by an import of a server-side library in a web endpoint. There should be something to avoid this mistake.

I can't claim that I understand what's going on here https://groups.google.com/a/dartlang.org/forum/#!topic/reviews/-HJKjoBqz44, just remembered seeing it.

Is the request here just to report a diagnostics when a library imports both dart:io and dart:html directly? Or something more about indirect imports?


Could not resolve imports/virtual on 0.10.0

Sorry to put an issue right after you pushed the update, but I can't run my app anymore after going from 0.9.1 to 0.10.0
✘ [ERROR] Could not resolve "virtual:vuetify-configuration"

    node_modules/vuetify-nuxt-module/dist/runtime/plugins/config.mjs:2:44:
      2 │ import { isDev, vuetifyConfiguration } from "virtual:vuetify-configuration";
        ╵                                             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

  You can mark the path "virtual:vuetify-configuration" as external to exclude it from the bundle,
  which will remove this error and leave the unresolved path in the bundle.

✘ [ERROR] Could not resolve "#imports"

    node_modules/vuetify-nuxt-module/dist/runtime/plugins/config.mjs:3:27:
      3 │ import { useNuxtApp } from "#imports";
        ╵                            ~~~~~~~~~~

  You can mark the path "#imports" as external to exclude it from the bundle, which will remove this
  error and leave the unresolved path in the bundle.

The page does not render properly and theres 404 warnings of sass files on each reload.

upps, I didn't change the package exports, I'm going to check the playgroung in SB

@daniandl  it is working, you need to updated Nuxt to 3.9.0+, check the SB reproduction in the docs on in the readme file.

installing in real app and failing, I'll try to fix ASAP.

fixed in v0.10.1

@daniandl I need to review the new approach, my tests working but not sure if the new plugin template can be statically analysable.
I'm working on a new approach...


Does it support vs2017？

I have installed it with vs2017. The main components of ROS, such as rviz works well. but when I create my package including "ros::init", there will be "unresolved external symbols" error .
And the catkin_make command fails too.  It seems that the static lib of ROS is not properly configured. So does rosonwindows support vs2017? In fact I want to directly use ros on my existing project, so is there any way to use it in vs?

Hello there,
The binaries we release for melodic are not compatible with Visual Studio 2017.
The melodic chocolatey packages are built with the v141 tool chain, which is included in vs2019. v141 is not binary compatible with v140 which was included in vs2017.
If you want to use ROS with v140, it is feasible, but you will have to build ROS and include it in your installation. We have instructions on how to build ROS here: https://ms-iot.github.io/ROSOnWindows/Build/source.html
Another solution I'd offer is to use the rosserial interface, which is a ROS-like over WinSocket. We used this approach with UWP as demonstrated in our WinML ROS Arm demo https://github.com/ms-iot/ros_winml_arm, where we used a ML pose detector and command and control from a UWP application.
Please reopen if you have issues.


jamovi: Get-FixVersion introduces extra 0's

The usage of fix version here changes version <IP_ADDRESS> to <IP_ADDRESS>.
Which is technically not true, and it means users have to type --version=<IP_ADDRESS> instead of the more intuitive --version=<IP_ADDRESS> to get the right version. The normal Get-Version does not do this.
Is there a reason that I am unaware of doing this? Maybe it caused #15 .

I agree. Should be more intuitive with the next jamovi updates.

With Get-FixVersion there's always at leat 1 appended extra zero.


intellisense not working , ctrl + not goto function definition react native

intellisense not working , ctrl + not goto function definition react native

@jawadg
You could reinstall a lower version
same issue

Me, too.

/needsMoreInfo


Boost compatibility issue

I'm building on macOS and some Boost library has been deprecated in v1.63.0 and Graphene seemingly won't work with it anymore:
Scanning dependencies of target fc
[  8%] Building CXX object libraries/fc/CMakeFiles/fc.dir/src/uint128.cpp.o
[  9%] Building CXX object libraries/fc/CMakeFiles/fc.dir/src/real128.cpp.o
[  9%] Building CXX object libraries/fc/CMakeFiles/fc.dir/src/variant.cpp.o
[  9%] Building CXX object libraries/fc/CMakeFiles/fc.dir/src/exception.cpp.o
[ 10%] Building CXX object libraries/fc/CMakeFiles/fc.dir/src/variant_object.cpp.o
[ 10%] Building CXX object libraries/fc/CMakeFiles/fc.dir/src/thread/thread.cpp.o
In file included from /Users/thor/Documents/Code/BitSpace/graphene/libraries/fc/src/thread/thread.cpp:5:
In file included from /Users/thor/Documents/Code/BitSpace/graphene/libraries/fc/src/thread/thread_d.hpp:5:
In file included from /Users/thor/Documents/Code/BitSpace/graphene/libraries/fc/src/thread/context.hpp:10:
In file included from /usr/local/include/boost/coroutine/stack_context.hpp:14:
/usr/local/include/boost/coroutine/detail/config.hpp:17:4: warning: "Boost.Coroutine is now deprecated. Please switch to Boost.Coroutine2. To disable this warning message,
      define BOOST_COROUTINES_NO_DEPRECATION_WARNING." [-W#warnings]
#  warning                  "Boost.Coroutine is now deprecated. Please switch to Boost.Coroutine2. To disable this warning message, define BOOST_COROUTINES_NO_DEPRECATI...
   ^
In file included from /Users/thor/Documents/Code/BitSpace/graphene/libraries/fc/src/thread/thread.cpp:5:
In file included from /Users/thor/Documents/Code/BitSpace/graphene/libraries/fc/src/thread/thread_d.hpp:5:
/Users/thor/Documents/Code/BitSpace/graphene/libraries/fc/src/thread/context.hpp:215:9: error: no type named 'fcontext_t' in namespace 'boost::context'
    bc::fcontext_t               my_context;
    ~~~~^
/Users/thor/Documents/Code/BitSpace/graphene/libraries/fc/src/thread/context.hpp:69:23: error: no member named 'make_fcontext' in namespace 'boost::context'
     my_context = bc::make_fcontext( stack_ctx.sp, stack_ctx.size, sf);
                  ~~~~^
In file included from /Users/thor/Documents/Code/BitSpace/graphene/libraries/fc/src/thread/thread.cpp:5:
/Users/thor/Documents/Code/BitSpace/graphene/libraries/fc/src/thread/thread_d.hpp:401:17: error: no member named 'jump_fcontext' in namespace 'boost::context'; did you mean
      'boost::context::detail::jump_fcontext'?
                bc::jump_fcontext( &prev->my_context, next->my_context, 0 );
                ^~~~~~~~~~~~~~~~~
                boost::context::detail::jump_fcontext
/usr/local/include/boost/context/detail/fcontext.hpp:31:35: note: 'boost::context::detail::jump_fcontext' declared here
transfer_t BOOST_CONTEXT_CALLDECL jump_fcontext( fcontext_t const to, void * vp);
                                  ^
In file included from /Users/thor/Documents/Code/BitSpace/graphene/libraries/fc/src/thread/thread.cpp:5:
/Users/thor/Documents/Code/BitSpace/graphene/libraries/fc/src/thread/thread_d.hpp:443:17: error: no member named 'jump_fcontext' in namespace 'boost::context'; did you mean
      'boost::context::detail::jump_fcontext'?
                bc::jump_fcontext( &prev->my_context, next->my_context, (intptr_t)this );
                ^~~~~~~~~~~~~~~~~
                boost::context::detail::jump_fcontext
/usr/local/include/boost/context/detail/fcontext.hpp:31:35: note: 'boost::context::detail::jump_fcontext' declared here
transfer_t BOOST_CONTEXT_CALLDECL jump_fcontext( fcontext_t const to, void * vp);
                                  ^
1 warning and 4 errors generated.
make[2]: *** [libraries/fc/CMakeFiles/fc.dir/src/thread/thread.cpp.o] Error 1
make[1]: *** [libraries/fc/CMakeFiles/fc.dir/all] Error 2
make: *** [all] Error 2

I'm attempting to build commit ae4bc619e45efd2010ccaad059ee305d1e8c1f59 (head of the master branch at the time of writing) and it's not happy.

This repository is no longer maintained. https://github.com/bitshares/bitshares-core/ is effectively the reference Graphene implementation now.
Only Boost 1.57 to 1.60 is supported with no plans for change. You will need to build against a manual Boost build of one of these versions if your system version is newer.


Where is the local storage folder?

I've just installed this app on a Pixel 3a (Android 12) and I've managed to mount the smb share without issues. I have tested it like this:

Create a new email on gmail.
Attach a document.
Select "Open From" (three bars menu).
Chose CIFS Documents Provider.

When I chose it, it shows the IP address of my NAS and when I tap on it, I get access to the files that I previously configured on the app.
My problem is that I'm trying to figure out where is the local folder in my device. I thought the point of this app was to mount an SMB share on a local folder of your choosing, but there is no option to select a local folder, even though I have ticked "Use as local folder" box and rebooted. Does the app create one folder inside my phone with that share? And if that is the case, where is it?

@SiLenCe-WB Unfortunately, it doesn't do what you want. The setting Use as local storage means that any 3rd party application expects a local storage from Android's SAF (Storage Access Framework), a.k.a, the default Files app.
If this setting is Off and the 3rd party application can't see your SMB share from the Files app when you are trying to open it. Then that application expects a local storage files "only". Turning this setting On, is just to emulate that behavior, and you might be surprised that the same 3rd party app now see your files.
This is not perfect, but it works very well on most applications I have found; minus some that require input/output stream like video recording applications.

Yeah, it works very nicely 👌👌
I guess that in other to mount a SMB share on a folder you need root permissions anyway, right?

Correct. If you are rooted, and If you want to go down the rabbit hole, look into RClone sync for android.

@RebelliousX
Thank you!
@SiLenCe-WB
I don't think this app is for the purpose you expect.
I hope this app is useful for you.


ignore nil chan , or it may cause close nil error

As submited in #70. I modify this again.
I have already finished running the sh test_codecov.sh .
It passed all.

Codecov Report

Merging #74 (2302bfa) into master (b48b2ee) will increase coverage by 0.98%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #74      +/-   ##
==========================================
+ Coverage   89.43%   90.42%   +0.98%
==========================================
  Files           6        6
  Lines         303      355      +52
==========================================
+ Hits          271      321      +50
- Misses         17       19       +2
  Partials       15       15

Impacted Files
Coverage Δ

graph.go
83.33% <100.00%> (-0.35%)
:arrow_down:

factory.go
100.00% <0.00%> (ø)

component.go
100.00% <0.00%> (ø)

graph_connect.go
92.26% <0.00%> (+0.96%)
:arrow_up:

graph_ports.go
87.50% <0.00%> (+2.31%)
:arrow_up:

graph_iip.go
86.66% <0.00%> (+3.33%)
:arrow_up:

Continue to review full report at Codecov.

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

@v1xingyue nice catch! Thanks a lot for contribution!


Create api-reference.md

Hello, I am a PR summary agent on flows.network.
It could take a few minutes for me to analyze this PR. Relax, grab a cup of coffee and check back later. Thanks!


Enhanced Footer Design and Functionality

Objective: Improve the footer design to provide additional value to users and enhance visual appeal.
Current Footer:
The current footer simply includes text: "Join us in building CodeCache with notsoocool. The source code is available on GitHub."
Enhancement Suggestions:
Add Social Media Links:
Include icons linking to other social media platforms like LinkedIn, Twitter, or Discord to build a community.
Place them neatly aligned on the right side of the footer.
Add Additional Links:
Add links to important sections like "About Us," "Documentation," "Contribute," and "Privacy Policy" to provide easy access to users.
Redesign Footer Layout:
Use a two-column layout: one for the call-to-action and one for quick links.
Apply subtle color contrasts and a divider to separate sections and improve readability.
Add an Icon with Tooltip:
Beside "GitHub" text, add a GitHub icon that reveals a tooltip saying “View Source Code” when hovered over.
CSS Styling:
Add hover effects on links and icons to improve interactivity.
Use subtle animations to enhance user experience.

@Priyanshu282003 we have a branch for enhanced footer for now, if you want to tweak it a lil bit, you can do so, also keep in mind to raise a pr for that branch only.


build_type included in documentation, but being reported as incorrect in fastlane-plugin-emerge 0.10.0

We have recently upgraded our Fastlane plugins for emerge and are receiving the following error in a Fastfile that previously worked -
Could not find option 'build_type' in the list of available options: api_token, file_path, linkmaps, pr_number, branch, sha, base_sha, build_id, base_build_id, repo_name, gitlab_project_id, tag, order_file_version, config_path, app_id_suffix

fastlane action emerge also indicates that build_type is no longer accepted -
Loading documentation for emerge:

+----------------------------+
|           emerge           |
+----------------------------+
| Fastlane plugin for Emerge |
|                            |
|                            |
| Created by Emerge Tools    |
+----------------------------+

+------------------------------------------------------------------+
|                          emerge Options                          |
+------------------+------------------+------------------+---------+
| Key              | Description      | Env Var(s)       | Default |
+------------------+------------------+------------------+---------+
| api_token        | An API token     | EMERGE_API_TOKE  |         |
|                  | for Emerge       | N                |         |
| file_path        | Path to the      | EMERGE_FILE_PAT  |         |
|                  | zipped           | H                |         |
|                  | xcarchive or     |                  |         |
|                  | app to upload    |                  |         |
| linkmaps         | List of paths    |                  |         |
|                  | to linkmaps      |                  |         |
| pr_number        | The PR number    |                  |         |
|                  | that triggered   |                  |         |
|                  | this upload      |                  |         |
| branch           | The current git  |                  |         |
|                  | branch           |                  |         |
| sha              | The git SHA      |                  |         |
|                  | that triggered   |                  |         |
|                  | this build       |                  |         |
| base_sha         | The git SHA of   |                  |         |
|                  | the base build   |                  |         |
| build_id         | **DEPRECATED!**  |                  |         |
|                  | Replaced by      |                  |         |
|                  | `sha` - A        |                  |         |
|                  | string to        |                  |         |
|                  | identify this    |                  |         |
|                  | build            |                  |         |
| base_build_id    | **DEPRECATED!**  |                  |         |
|                  | Replaced by      |                  |         |
|                  | `base_sha` - Id  |                  |         |
|                  | of the build to  |                  |         |
|                  | compare with     |                  |         |
|                  | this upload      |                  |         |
| repo_name        | Full name of     |                  |         |
|                  | the respository  |                  |         |
|                  | this upload was  |                  |         |
|                  | triggered from.  |                  |         |
|                  | For example:     |                  |         |
|                  | EmergeTools/Eme  |                  |         |
|                  | rge              |                  |         |
| gitlab_project_  | Id of the        |                  |         |
| id               | gitlab project   |                  |         |
|                  | this upload was  |                  |         |
|                  | triggered from   |                  |         |
| tag              | String to label  |                  |         |
|                  | the build.       |                  |         |
|                  | Useful for       |                  |         |
|                  | grouping builds  |                  |         |
|                  | together in our  |                  |         |
|                  | dashboard, like  |                  |         |
|                  | development,     |                  |         |
|                  | default, or      |                  |         |
|                  | pull-request     |                  |         |
| order_file_vers  | Version of the   |                  |         |
| ion              | order file to    |                  |         |
|                  | download         |                  |         |
| config_path      | Path to Emerge   |                  |         |
|                  | config path      |                  |         |
| app_id_suffix    | A suffix to      |                  |         |
|                  | append to the    |                  |         |
|                  | application ID   |                  |         |
|                  | to               |                  |         |
|                  | differentiate    |                  |         |
|                  | between          |                  |         |
|                  | different        |                  |         |
|                  | builds of the    |                  |         |
|                  | same app         |                  |         |
+------------------+------------------+------------------+---------+
* = default value is dependent on the user's system

+-------------------------------------------------------------+
|                     emerge Return Value                     |
+-------------------------------------------------------------+
| If successful, returns the upload id of the generated build |
+-------------------------------------------------------------+

More information can be found on https://docs.fastlane.tools/actions/emerge

Is this an intentional deprecation ? Examples in the README in this repo list it as optional.

Hi @gemmakbarlow, thanks for reporting, we updated this field to be named tag. I'll add it back to the params list with the deprecation notice.

Ah got it. OK, sounds great ✅ 🎉 🙏🏻


Add multiple node support for Jenkins slaves

Create a dictionary and permit ability to add multiple Jenkins slaves from a list. Add functionality to provide ability to delete Jenkins slave nodes using absent key term.

 Add multiple node support for Jenkins slaves

Resolved by @s1061123 via https://github.com/redhat-nfvpe/ansible-cira/commit/59662f43aafee5e3c94cd119e4ea43287e138362


Add wait_for_connection() method to FixServer

Occasionally there is a race when using FixServer in a testcase when the first thing the testcase does is try to receive a message from the FixServer before the connection has been established:
msg = env.fixserver.receive()

which results in:
Exception: Cannot use default connection since more connections active

(This error is not entirely correct as there are no connections active yet (server checks if number of connections != 1) - but that does not really concern me).
This patch adds a wait_for_connection() method which fixes the race.
BTW. I was wondering if it would be desirable to do this in the FixServer's receive() method. I did not want to impact other users/uses of receive() so opted to expose this function instead.

Hi,
Thanks for the contribution. Can you clarify the expected use of this method within a testcase? Maybe if the method should always be called before the first message is receive()d it would make sense to make it a private method and have the FixServer automatically wait for the connection to be established before receiving the first message.
I do think we should also fix the error message in the case of 0 connections being active, thanks for spotting that!
Regards,
Ryan

Can we first agree that the expected setup with FixServer and FixClient is:

FixServer starts
FixClient starts and connects
server.accept_connection() is being called in a post_start() MultiTest step or in a setup() in a testsuite.
NOTE: if you cannot accept the connection you can sleep and retry.
And then you have your connection!

Example:
https://testplan.readthedocs.io/en/latest/download/Transports.html#fix
..but I would definately put the accept connection in post_start() of MultiTest.
Check my_tests/mtest.py file of:
https://testplan.readthedocs.io/en/latest/download/Interactive.html#basic
If you think this scenario does cover your race-condition case please provide more information.
Otherwise I don't think this PR is needed and I would advise before making a PR to raise an issue topic so we have a conversation as a very first thing.

Hi! I hope you are all well!
@ryan-collingham having it private and called from receive() would be ideal. I was hoping that is an option, I will look into doing that and will update this PR a bit later. In that case the error message would be correct (I think) as receive() would block until there is a connection.
@JohnChiotis First just to clarify do you prefer me raising an issue first before opening a PR as a starting point? I certainly can do that next time.
My test at the moment starts a FixServer and our binary application (eventually the test will start several FixServers, our binary application and some clients of our binary application). I have a driver for our binary application that in its started_check() method waits for the binary to log the connection details used for clients. At some later point the binary starts connecting to the FixServer(s) depending on configuration (schedule).
In this particular test I am working on now I want to test the FIX logon sequence used by a specific gateway in our binary to a FixServer. I also plan to test disconnection and re-connection at various points of processing.
So when my test case starts I want to assert correctness of the message received by the FixServer, but sometimes the connection has not occurred yet leading to the failure I described in my first post.
Regarding your suggestions, accept_connection() is for TcpServer and not FixServer it seems.
Up to now I used a polling loop to wait for connections but that is a bit unsatisfactory, I am looking for a cleaner solution that does not impose arbitrary delays on test suites by sleeping, i.e. proceed as soon as the awaited event occurs.
many thanks for helping!
kind regards.

Hi,
I think it looks very useful so we will merge it.
However I want to point out this shouldn't be called from the receive method. The drivers should have finished setting up, including ensuring all connections have been made, before the tests have started to run. You should call this method from a function given to Mutitest's after_start parameter. See here for an example of using Multitest's after start parameter. This will ensure that the connection (or multiple connections if you decide) are active before your tests start. Perhaps we should add this to our FIX downloadable example.
Also I'll just confirm a couple of points:

If you have an issue like this in future we would prefer that you raise an issue with us before opening a PR. We really appreciate PRs but this allows us to work with you on the best approach to take from the start.
Next time you submit a PR could you also submit a testcase to cover the new functionality. Our tests are under the test directory. We can add the testcase this time.

Thanks again for contributing!

Thank you @lambchr , I will reach out via an issue next time.
Regarding unit tests - I am very happy to add one.
I had difficulty running tests - do they only work with python 2?  (We use python 3.7.2).

We currently test against python 2.7 & 3.4. We also test on Linux and Windows, however these builds are done internally (so no link to share).
We should change our builds to run the tests against 3.7 (and fix any compatibility issues), I'll have to add that to the do list. Thanks for raising the issue. Can you try running the tests on Python 3.4 for now?

I had difficulty running tests - do they only work with python 2? (We use python 3.7.2).

I can run the tests fine with Python 3.7.2. What problems are you seeing?

I have a lot of deprecation warnings for various things and 2 errors. Should I be concerned about the deprecation warnings like:
DeprecationWarning: invalid escape sequence \
DeprecationWarning: inspect.getargspec() is deprecated since Python 3.0, use inspect.signature() or inspect.getfullargspec()
PytestWarning: cannot collect test class 'Testplan' because it has a __init__ constructor
PytestWarning: cannot collect test class 'TestReport' because it has a __init__ constructor
Too many to paste, should I be concerned about these?
Looking at that now maybe my environment lacks some modules:
ModuleNotFoundError: No module named 'testplan' and ModuleNotFoundError: No module named 'flask'. I will create another conda environment just for running tests I think, can use python 2 for that maybe.

The warnings are expected. We should fix the DeprecationWarnings at some point, the warnings about init constructors are benign though.
ModuleNotFoundError indicates that you probably haven't set your environment up correctly. Did you follow the instrucitons on https://testplan.readthedocs.io/en/latest/getting_started.html - in particular you need to run:
pip install -r requirements.txt
python setup.py develop

We also now have a dockerised development environment if you would like to try that, you can run with docker run -it ryancollingham/dev_env:testplan.

I did set it up before using those instructions and started to use that environment for my own tests. (Mixing conda and pip tripped me up where pip installs packages into conda environment where pip is installed and not the currently active one. I also switched to fish shell recently.)
I created a new conda environment for unit testing only and now it works! Thank you.
At the moment I cannot find any tests for FixServer or FixClient. I guess it is going to be a challenge to satisfy their msgclass and codec requirements in unit tests which is probably why there are no tests for it yet.

I need this method or some other mechanism to use in the after_start specified method for FixServer as @lambchr said. All examples and documentation I can find only uses it for TCPServer.
I considered subclassing FixServer but that does not seem feasible for something as fundamental as this. FixServer delegates to fix.server.Server which will need to be subclassed as well.

Currently we have no unit or functional test coverage of FixServer/Client as far as I can tell, though there is some coverage in the example under examples/Transports/FIX. I think it would make sense to update the example to explicitly wait for the connection to be established in a post_start function.
You are right that accept_connection is a method of the TCPServer and not the FixServer. So this PR is really adding an equivalent of TCPServer.accept_connection to the FixServer. Even though wait_for_connection is a better description of what the method is actually doing I think we should call the new method accept_connection for consistency.
I think that waiting for the threading.Condition is more elegant than the polling approach used in TCPServer, nice one :)

Thanks @ryan-collingham , I renamed it and changed the timeout default to 10 seconds too as that is what TCPServer use for a timeout.
I will try to add some unit tests, it is going to take me a while. Would adding pyfixmsg as a dependency for unit testing be an option? Or would it be better to mock it for FixServer testing?

There should be no dependency to pydixmsg,

Closing due to no activity.


Replace American spelling with British spelling

[X] Have you followed the guidelines in our Contributing document?
[X] Have you checked to ensure there aren't other open Pull Requests for the same change?
[X] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Wed Oct 30 18:05:17 GMT 2019
tl;dr:  -ize -> -ise
This PR replaces words spelt in American english with their British equivalents.

Wed Oct 30 18:13:22 GMT 2019
@dawidd6
https://github.com/Homebrew/brew/blob/eda0220e5f807482e4feb327e34375de308ea0fc/docs/Prose-Style-Guidelines.md#style-and-usage

You'll need to run brew man.


error: invalid cast from type 'simde__m128' to type 'simde__m128i'

Hi hi. While making a first pass at patching https://github.com/mengyao/Complete-Striped-Smith-Waterman-Library to use simde, I got the following error while compiling for i386:
/usr/bin/i686-linux-gnu-g++-9 -Wdate-time -D_FORTIFY_SOURCE=2 -g -O2 -fdebug-prefix-map=/build/libssw-1.1=. -fstack-protector-strong -Wformat -Werror=format-security -fPIC -shared -rdynamic -Wl,-soname,libssw.so.0 -o libssw.so.0 ssw.c ssw.h ssw_cpp.h ssw_cpp.cpp -Wl,-z,relro -Wl,-z,now
In file included from ../debian/include/simde/x86/../simde-common.h:27,
                 from ../debian/include/simde/x86/mmx.h:28,
                 from ../debian/include/simde/x86/sse.h:32,
                 from ../debian/include/simde/x86/sse2.h:34,
                 from ssw.c:38:
../debian/include/simde/x86/sse2.h: In function 'simde__m128i simde_mm_castps_si128(simde__m128)':
../debian/include/simde/x86/../hedley.h:1358:69: error: invalid cast from type 'simde__m128' to type 'simde__m128i'
 1358 | #  define HEDLEY_REINTERPRET_CAST(T, expr) (reinterpret_cast<T>(expr))
      |                                                                     ^
../debian/include/simde/x86/sse2.h:821:7: note: in expansion of macro 'HEDLEY_REINTERPRET_CAST'
  821 |   r = HEDLEY_REINTERPRET_CAST(simde__m128i, a);
      |       ^~~~~~~~~~~~~~~~~~~~~~~
../debian/include/simde/x86/sse2.h: In function 'simde__m128d simde_mm_castsi128_pd(simde__m128i)':
../debian/include/simde/x86/../hedley.h:1358:69: error: invalid cast from type 'simde__m128i' to type 'simde__m128d'
 1358 | #  define HEDLEY_REINTERPRET_CAST(T, expr) (reinterpret_cast<T>(expr))
      |                                                                     ^
../debian/include/simde/x86/sse2.h:838:7: note: in expansion of macro 'HEDLEY_REINTERPRET_CAST'
  838 |   r = HEDLEY_REINTERPRET_CAST(simde__m128d, a);
      |       ^~~~~~~~~~~~~~~~~~~~~~~
../debian/include/simde/x86/sse2.h: In function 'simde__m128 simde_mm_castsi128_ps(simde__m128i)':
../debian/include/simde/x86/../hedley.h:1358:69: error: invalid cast from type 'simde__m128i' to type 'simde__m128'
 1358 | #  define HEDLEY_REINTERPRET_CAST(T, expr) (reinterpret_cast<T>(expr))
      |                                                                     ^
../debian/include/simde/x86/sse2.h:857:7: note: in expansion of macro 'HEDLEY_REINTERPRET_CAST'
  857 |   r = HEDLEY_REINTERPRET_CAST(simde__m128, a);
      |       ^~~~~~~~~~~~~~~~~~~~~~~

Here's the patch https://salsa.debian.org/med-team/libssw/blob/simde/debian/patches/simde.patch
Adding -march=i686 (the baseline mode for Debian's 32-bit x86 release) gives the same error. If I instead specify -march=pentium-m or a more advanced processor with SSE2 then the error goes away.

Must be a C++ issue given the positive 32-bit i686 test at https://github.com/nemequ/simde/pull/53 :-)

Argh, I should really add a C++ build to the tests to catch stuff like this.  06ccbb42a7d8e2a531de6ef4f5b2c851ec1dc77a should fix this.  Your patch didn't apply cleanly against git master, so here is what I used for testing: https://paste.centos.org/view/fd666e71
On an unrelated note, I see an UNLIKELY macro in there… I'm not sure you care about more exotic compilers, but and FWIW Hedley (another one of my projects, so take the suggestion with a grain of salt) has some pretty solid likely/unlikely macros.

Thanks! I can confirm this fix


Dimension inconsistent?

Maybe it's something I missed. I am very new to dask. But here's a reproducible example:
Dataset: dbpedia_csv.tar.gz
https://drive.google.com/folderview?id=0Bz8a_Dbh9Qhbfll6bVpmNUtUcFdjYmF2SEpmZUZUcVNiMUw1TWN6RDV3a0JHT3kxLVhVR2M
data = dd.read_csv('dbpedia_csv/train.csv', header=None)
data.divisions = tuple(range(1, len(data.divisions) + 1))  # assign divisions other than None

data.count().compute()
0    560000
1    560000
2    560000
dtype: int64

which is consistent with
data[1].count().compute()

560000

But when data.compute()
I got this:
0                                         E. D. Abbott Ltd
1                                           Schwan-Stabilo
2                                               Q-workshop
3                        Marvell Software Solutions Israel
4                              Bergan Mercy Medical Center
5                                       The Unsigned Guide
6                                        Rest of the world
7                                               Globoforce
8                                                Rompetrol
9                                          Wave Accounting
10                                      Angstrem (company)
11                                         I-innovate (UK)
12                                                     JVC
13                                                Toei Bus
14                                       Tear Drop Records
15       Presses polytechniques et universitaires romandes
16                                                Websense
17                                 Adventist Health System
18                                                CIB Bank
19                                    Orfanato Music Group
20                                        SCAN Health Plan
21                                            Rudolf Kämpf
22                                        De Wendel family
23                              Mansfield Building Society
24                                          Witcomb Cycles
25                                     Goldilocks Bakeshop
26                              Guardian Assurance Company
27                                                  Shiply
28                         Orange Music Electronic Company
29                                       Rytec Corporation
                               ...
18629                                                  NME
18630                                         The Londoner
18631                          Formal Aspects of Computing
18632                                            Hayom Yom
18633                        Swords Against the Shadowland
18634              Aruvu Rezuru: Kikaijikake no Yōseitachi
18635                                    Tintin (magazine)
18636                       The Tenth Man (Chayefsky play)
18637                                    Dramatical Murder
18638                 A Dictionary of Modern English Usage
18639                                              Assayad
18640                                     Loveless (manga)
18641                                  The American Hebrew
18642                                  Montanan (magazine)
18643                                    Black Out (novel)
18644                                 Gold Digger (comics)
18645                               Dealer's Choice (play)
18646                                       Phuket Gazette
18647                            If I Forget Thee Oh Earth
18648                        The Lincoln Conspiracy (book)
18649                                   Endocrine Research
18650                             A Thousand Splendid Suns
18651                                       Western People
18652                              Mademoiselle (magazine)
18653                                  Scoliosis (journal)
18654                                     Barking in Essex
18655                                     Science & Spirit
18656                               The Blithedale Romance
18657                                  Razadarit Ayedawbon
18658                             The Vinyl Cafe Notebooks
Name: 1, dtype: object

Why isn't it 560000 but 18658?

A few things

You should check the count of the result, it may be longer than you think.  I wouldn't place much faith in the index values of a large CSV file.
Generally it's hard to determine exactly how many elements are in each partition of the CSV file, we're just throwing 20MB or so of text at pd.read_csv and letting it produce what it can.
What was your objective by assigning divisions=range(1, ...)?

But more generally, what is your goal here?  It's very hard to determine how many rows are within a CSV file without actually going through and reading the entire thing (which, if it's important, we could just do).

Thanks for your super quick response. My original goal was: given X and y, two different Series, I want to sample them both but firstly get samples from X and use indices from X's samples to get y's samples.
Alternatively, is there a way to column bind X and y? (preserving the row index)

I think append will do the job. Anyways, I am going to try other work-around. Thanks for the help.

@mrocklin Actually, do you mind giving a simple example to combine two dd.Series into a dd.DataFrame, such that they are two different columns in the new DataFrame?

Generally you should be aware that the dask.dataframe API is a subset of the Pandas API, which is much more heavily documented.  So a quick googling of "combine a pandas series to a pandas dataframe" is very likely to yield constructive results.
You may find the following Pandas doc page valuable http://pandas.pydata.org/pandas-docs/stable/merging.html .  I think that you're looking for concat with axis=1.
Alternatively, you can use the same random seed in each of the sample calls.
In [1]: import pandas as pd

In [2]: df = pd.DataFrame({'x': list(range(100)), 'y': [i * 10 for i in range(100)]})

In [3]: df.head()
Out[3]:
   x   y
0  0   0
1  1  10
2  2  20
3  3  30
4  4  40

In [4]: df.x.sample(frac=0.1, random_state=123)
Out[4]:
8      8
70    70
82    82
28    28
63    63
0      0
5      5
50    50
81    81
4      4
Name: x, dtype: int64

In [5]: df.y.sample(frac=0.1, random_state=123)
Out[5]:
8      80
70    700
82    820
28    280
63    630
0       0
5      50
50    500
81    810
4      40
Name: y, dtype: int64

Thanks. Got it.

Although actually, if you're using CSV files it'll be tough to align things correctly.  I suspect that this needs an initial read through in order to collect the row counts of each partition.

Those problems would go away if you were to convert your CSV data to HDF5

I see. Will check it out and let you know if I have further questions. Thanks a lot.


core: internal component support

We need to set up the project to support internal components to share between public facing components. Example cds-internal-close-button
Shared components in the library will be properly deduped due to JS modules. For components that we dont want to support externally yet and are solely for internal should be prefixed. cds-internal-close-button. However there is a non trivial amount of work to set this up. The internal components cant exist in @clr/core/internal as they would cause a circular dependency icon-button depends on icon which depends on internal. We also have to be careful to ensure we dont break tree shaking by keeping each component as a separate entry point. Set up would look something like this.
/clr-core
  /internal
  /internal-components
    /close-button
      close-button.element.ts
      entrypoint.tsconfig.json
      ...

// modal.element.ts
import '@clr/internal-components/close-button';

i'll be tackling this as part of the alerts work...


/check on specific char breaks bot

There seems to be an issue with the house ID for the house Tunnel Gardens 2 (Kazordoon).

2017-12-11 14:40:18,928:INFO: Command by Fritspizza in #glorious_bot (Exura Vita (Reddit)): /check vlurr
2017-12-11 14:40:19,138:ERROR: Exception in command: whois
Traceback (most recent call last):
File "/usr/local/lib/python3.6/site-packages/discord/ext/commands/core.py", line 62, in wrapped
ret = yield from coro(*args, **kwargs)
File "/root/NabBot/cogs/tibia.py", line 67, in whois
char_string = self.get_char_string(char)
File "/root/NabBot/cogs/tibia.py", line 1524, in get_char_string
house_url = url_house.format(id=char.house["id"], world=char.world)
KeyError: 'id'

This was fixed by PR #78, thanks @Tschis


Confusing statement - App Service - Seperate Plan for Production and Test

We used to say all deployment slots with default as Production and other slots like Pre-Prod etc. This brings good cost saving for customer - I can understand from security and management good to keep 2 seperate but can they be merged and we can configure authentication and authorization seperately.
I can understand to keep performance environment out because of common shared infra but can we just re-evaluate this statement.
[Enter feedback here]

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

ID: f14bdf5c-81b0-76df-4c61-3b4eca2435ae
Version Independent ID: 433531d0-bedf-ce5c-a747-ce1d6b6bea5f
Content: Resiliency checklist for services - Azure Design Review Framework
Content Source: docs/checklist/resiliency-per-service.md
Service: architecture-center
Sub-service: well-architected
GitHub Login: @Petertaylor9999
Microsoft Alias: pnp

Thanks for reporting - this issue is under review. This is a Microsoft Internal DevOps Bug ID # 503563


chalice: Expected no arguments to "Blueprint" constructor

Environment data

Language Server version: Pylance language server 2021.3.1
OS and version: macOS Catalina v10.15.7
Python version (& distribution if applicable, e.g. Anaconda): 3.8.7

Expected behaviour
No reportGeneralTypeIssues error. See AWS Chalice Blueprint
Actual behaviour
Expected no arguments to "Blueprint" constructor

Code Snippet / Additional information

Can you hover over Blueprint to see what we are inferring its __init__ signature as? If you go-to-declaration, does it take you to a stub file that may have defined this initializer differently?

Does this help?

It appears that the chalice package includes type stubs, and the declaration for the Blueprint class doesn't define a constructor.
Pylance is honoring the type stub information shipped with the chalice package. If you think the type information is incorrect (and it appears to be based on the implementation of Blueprint), I recommend filing a bug report or PR with the chalice repo.

I'm going to close this as there's nothing we can really do on our end here; we respect the stubs that come with the library (and we've previously closed these sorts of upstream issues).
You can feasibly suppress the error yourself for your usage, but it's best to let the library know that their types are incorrect. Those stubs look pretty bare.


Fix logo in docs.

Fixes #3919.
Screenshots

Pull Request Checklist

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] Database migration files are up to date.
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[ ] GraphQL schema and type definitions are up to date.
[x] Changes are mentioned in the changelog.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@371e6e3). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #3928   +/-   ##
=========================================
  Coverage          ?   90.61%
=========================================
  Files             ?      266
  Lines             ?    14562
  Branches          ?     1456
=========================================
  Hits              ?    13195
  Misses            ?      940
  Partials          ?      427

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 371e6e3...614ab45. Read the comment docs.


Inefficient Regular Expression Complexity in nth-check

Contents require login.
https://github.com/bcgov/nr-spar/security/dependabot/1
https://github.com/bcgov/nr-spar/security/code-scanning/1

Issues:

Inefficient Regular Expression Complexity in nth-check -> Closed with https://github.com/bcgov/nr-spar/pull/409
Inefficient regular expression complexity -> Also closed with https://github.com/bcgov/nr-spar/pull/409


DOC: small doc fix for tf.transpose_v2

Found some inconsistencies while going through the docs. Hope this helps!

@googlebot I signed it!

We will not be encouraging one liner grammatical changes as this is expensive process,please include more such changes so that we can review it, thank you for your interest.
CC @mihaimaruseac


Not working anymore

After installing zip file connection error showing for jiotv as well as hotstar

come on help me :)) @vivin-neelankavil

check out my latest issue
Thanks


v1.0.0 fixes

Change type size from 17px to 16px (#889)
All line heights are not unit-less, so they inherit/cascade better (#847)
base line-height is now 1.5 instead of 1.55, this keeps type spacing at whole pixels most of the time (#826)
Fixed a couple bugs with pre/post as well
Explicitly set all vars to false in imports to reduce the chance of multiple inclusions of built CSS when importing mixins

Looks good! only comments re: font changes would be the mark/label and code items are now a little tough to read. Other than that 👍

@macandcheese do we want to merge this in? I can open a new pr to give the label/code/mark elements a bit more size


Minimal Viable Dataspace - EDC Gaps for V1

Description
This feature serves as a place holder to collect gaps that need to be closed on the EDC side to implement a first version of a Minimal Viable Dataspace (MVD).
Collection of Known Gaps

Identity hub: though EDC offers such a functionality, it does not seem to bring value without a concept for verified credentials and claims (below), and policies to enforce the claims provided by the hub
Policies: does not seem to be implemented in EDC. We would like to start with a "simplest possible" authorisation rule, e.g. "the caller must have a claim that region=EU" (see also https://github.com/eclipse-dataspaceconnector/DataSpaceConnector/issues/855)
Federated Catalog: though EDC offers such a functionality, it's not clear if that really fits the concept of a distributed data space, and how to implement access control. (see also https://github.com/eclipse-dataspaceconnector/DataSpaceConnector/issues/995)
Verifiable Credentials and Claims: needs design and implementation (see also https://github.com/eclipse-dataspaceconnector/DataSpaceConnector/issues/1070)
Data Management API for data transfer (fixed in upstream https://github.com/eclipse-dataspaceconnector/DataSpaceConnector/issues/1066)

Acceptance Criteria

[ ] The EDC provides the required functionality to create a first version of a very basic MVD.

Stories

Discard this feature as the gaps will be planned for and implemented by dedicated features that will deliver a specific outcome.


Add table aws_iot_thing_group Closes #1994

Integration test logs

Logs
No env file present for the current environment:  staging
 Falling back to .env config
No env file present for the current environment:  staging
customEnv TURBOT_TEST_EXPECTED_TIMEOUT undefined

SETUP: tests/aws_iot_thing_group []

PRETEST: tests/aws_iot_thing_group

TEST: tests/aws_iot_thing_group
Running terraform
data.aws_partition.current: Reading...
data.aws_region.alternate: Reading...
data.aws_caller_identity.current: Reading...
data.aws_region.primary: Reading...
data.aws_region.alternate: Read complete after 0s [id=us-east-2]
data.aws_region.primary: Read complete after 0s [id=us-east-1]
data.aws_partition.current: Read complete after 0s [id=aws]
data.aws_caller_identity.current: Read complete after 0s [id=333333333333]
data.null_data_source.resource: Reading...
data.null_data_source.resource: Read complete after 0s [id=static]

Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # aws_iot_thing_group.example will be created
  + resource "aws_iot_thing_group" "example" {
      + arn               = (known after apply)
      + id                = (known after apply)
      + metadata          = (known after apply)
      + name              = "turbottest71983"
      + parent_group_name = "parentturbottest71983"
      + tags              = {
          + "terraform" = "true"
        }
      + tags_all          = {
          + "terraform" = "true"
        }
      + version           = (known after apply)

      + properties {
          + description = "This is my thing group"

          + attribute_payload {
              + attributes = {
                  + "One" = "11111"
                  + "Two" = "TwoTwo"
                }
            }
        }
    }

  # aws_iot_thing_group.parent will be created
  + resource "aws_iot_thing_group" "parent" {
      + arn      = (known after apply)
      + id       = (known after apply)
      + metadata = (known after apply)
      + name     = "parentturbottest71983"
      + tags_all = (known after apply)
      + version  = (known after apply)
    }

Plan: 2 to add, 0 to change, 0 to destroy.

Changes to Outputs:
  + account_id           = "333333333333"
  + aws_partition        = "aws"
  + aws_region           = "us-east-1"
  + parent_resource_name = "parentturbottest71983"
  + resource_aka         = (known after apply)
  + resource_name        = "turbottest71983"
  + version              = (known after apply)
aws_iot_thing_group.parent: Creating...
aws_iot_thing_group.parent: Creation complete after 2s [id=parentturbottest71983]
aws_iot_thing_group.example: Creating...
aws_iot_thing_group.example: Creation complete after 1s [id=turbottest71983]

Warning: Deprecated

  with data.null_data_source.resource,
  on variables.tf line 43, in data "null_data_source" "resource":
  43: data "null_data_source" "resource" {

The null_data_source was historically used to construct intermediate values
to re-use elsewhere in configuration, the same can now be achieved using
locals or the terraform_data resource type in Terraform 1.4 and later.

(and one more similar warning elsewhere)

Apply complete! Resources: 2 added, 0 changed, 0 destroyed.

Outputs:

account_id = "333333333333"
aws_partition = "aws"
aws_region = "us-east-1"
parent_resource_name = "parentturbottest71983"
resource_aka = "arn:aws:iot:us-east-1:333333333333:thinggroup/turbottest71983"
resource_name = "turbottest71983"
version = 1

Running SQL query: test-get-query.sql
Warning: terminal options has been deprecated and will be removed in a future version of Steampipe.
These can now be set in a steampipe workspace.
(/Users/parthas/.steampipe/config/default.spc:34,20-42,2)
[
  {
    "arn": "arn:aws:iot:us-east-1:333333333333:thinggroup/turbottest71983",
    "group_name": "turbottest71983"
  }
]
✔ PASSED

Running SQL query: test-list-query.sql
Warning: terminal options has been deprecated and will be removed in a future version of Steampipe.
These can now be set in a steampipe workspace.
(/Users/parthas/.steampipe/config/default.spc:34,20-42,2)
[
  {
    "arn": "arn:aws:iot:us-east-1:333333333333:thinggroup/turbottest71983",
    "group_name": "turbottest71983",
    "parent_group_name": "parentturbottest71983",
    "version": "1"
  }
]
✔ PASSED

Running SQL query: test-notfound-query.sql
Warning: terminal options has been deprecated and will be removed in a future version of Steampipe.
These can now be set in a steampipe workspace.
(/Users/parthas/.steampipe/config/default.spc:34,20-42,2)
null
✔ PASSED

Running SQL query: test-turbot-query.sql
Warning: terminal options has been deprecated and will be removed in a future version of Steampipe.
These can now be set in a steampipe workspace.
(/Users/parthas/.steampipe/config/default.spc:34,20-42,2)
[
  {
    "akas": [
      "arn:aws:iot:us-east-1:333333333333:thinggroup/turbottest71983"
    ],
    "title": "turbottest71983"
  }
]
✔ PASSED

POSTTEST: tests/aws_iot_thing_group

TEARDOWN: tests/aws_iot_thing_group

SUMMARY:

1/1 passed.

Example query results

Results
> select * from aws_iot_thing_group
[
 {
  "_ctx": {
   "connection_name": "aws"
  },
  "account_id": "333333333333",
  "akas": [
   "arn:aws:iot:us-east-1:333333333333:thinggroup/test12"
  ],
  "arn": "arn:aws:iot:us-east-1:333333333333:thinggroup/test12",
  "attribute_payload": {
   "Attributes": {
    "foo": "bar"
   },
   "Merge": false
  },
  "creation_date": "2023-12-05T17:44:57+05:30",
  "group_name": "test12",
  "index_name": null,
  "parent_group_name": null,
  "partition": "aws",
  "query_string": null,
  "query_version": null,
  "region": "us-east-1",
  "root_to_parent_thing_groups": null,
  "status": "",
  "tags": {
   "foo1": "bar1"
  },
  "tags_src": [
   {
    "Key": "foo1",
    "Value": "bar1"
   }
  ],
  "thing_group_description": "testing",
  "thing_group_id": "99df2181-c05d-4c74-8c35-57b6cb7989de",
  "title": "test12",
  "version": 1
 }
]

Time: 109ms. Rows fetched: 1 (cached). Hydrate calls: 0.
> select * from aws_iot_thing_group where group_name = 'test12'
[
 {
  "_ctx": {
   "connection_name": "aws"
  },
  "account_id": "333333333333",
  "akas": [
   "arn:aws:iot:us-east-1:333333333333:thinggroup/test12"
  ],
  "arn": "arn:aws:iot:us-east-1:333333333333:thinggroup/test12",
  "attribute_payload": {
   "Attributes": {
    "foo": "bar"
   },
   "Merge": false
  },
  "creation_date": "2023-12-05T17:44:57+05:30",
  "group_name": "test12",
  "index_name": null,
  "parent_group_name": null,
  "partition": "aws",
  "query_string": null,
  "query_version": null,
  "region": "us-east-1",
  "root_to_parent_thing_groups": null,
  "status": "",
  "tags": {
   "foo1": "bar1"
  },
  "tags_src": [
   {
    "Key": "foo1",
    "Value": "bar1"
   }
  ],
  "thing_group_description": "testing",
  "thing_group_id": "99df2181-c05d-4c74-8c35-57b6cb7989de",
  "title": "test12",
  "version": 1
 }
]

Time: 167ms. Rows fetched: 1 (cached). Hydrate calls: 0.

The PR has a dependency on the PR: https://github.com/turbot/steampipe-plugin-aws/pull/1997 so the Golang lint is failing.


Add support for PHP 8.1

Opened in draft as a test is failing. I haven't figured out what changed in 8.1 for this to happen. I know html encoding and decoding functions handle single quotes differently in 8.1, but I don't think they are used anywhere?
Fyi, it's the first ' in '这也不会,那也不会' that is different.
While searching for a solution on this, I quickly tried to migrate zbateson/mb-wrapper to Github Actions, but for some reason that's not working as suspected. Feel free to fix or just close that PR: https://github.com/zbateson/mb-wrapper/pull/5

Hi @Nielsvanpach --
Thanks for this.  It seems there was  NUL char at the beginning of the string for some reason, and for some reason php before 8.1 would just ignore that.
Thanks for this :)


Settings are saved and loaded from 2 different locations

Saved settings are loaded from a different location they are saved. It is impossible to load non-default settings.

#6 also appears to resolve this

Thanks for the heads up, looks like the issue is fixed now.


Meeting assistant error

This is what I see after authorizing the app

Looks like you also need to enable Calendar api on GCP...


TokenRegistry's unregisterToken need refractory

This unregisterToken now takes O(n) time. We really need to change it to O(1) otherwise when the list is too long, we'll eventually run out of gas before a token near the end of the list can be unregistered.
We will probably support thousands of tokens in the future.

One idea is to change
mapping (address => bool) tokenMap;
to:
mapping (address => TokenInfo) tokenMap;
where TokenInfo is defined as:
struct TokenInfo {
    uint index; // index of the address in the array
    uint8 tokenStandard; // 20 or 223
    string symbobl;
}

then if a token is removed, we can always move the last token in the list to fill the empy spot in the middle. The order tokens are stored in tokens doesn't really matter.

See PR #176. I actually did exactly what you proposed. Though I'm not sure if making index == 0 as unregistred is the most logical because index = 0 is valid. Maybe symbol should be used for this if possible?

On a side note, does symbol need to be a string? If it's just the ticker of the token it may make more sense to just store a bytes4 or equivalent, should save some gas as well.

@Brechtpd the symbol can be 2 to 6 letters and can be even longer.


Split RethNodeCommandExt into two traits

Describe the feature
split RethNodeCommandExt
https://github.com/paradigmxyz/reth/blob/main/bin/reth/src/cli/ext.rs#L35
into two:
RethNodeCommandConfig with the existing functions
and RethNodeCommandExt: RethNodeCommandConfig +  clap::Args so it can be plugged into cli
this gives us more flexibility when integrating custom config
Additional context
No response

I will fix this.


Why isn't the current date loading when the page loads

I put the minimum needed example on my page and it keeps coming up to the March 2013 date. How in the heck do I reset that and clear out the test event?
Thanks
Ty

Open app.js and comment out the line:-   day: "2016-09-26". The calendar will then load today's date.

Thanks Joshreade!


Make most network constants configurable and remove TODOs that won't be done

Motivation
As a part of doing some cleanup of TODOs and ensuring the code is in a good state, I've gone ahead and removed a bunch of TODOs that were not going to get done / didn't apply.
In addition, I've made it such that NetworkConfig is the almost one true source of all constants and configuration for Networking.  There were also some test NetworkBuilder improvements as a result.
The new location of the constants is less than ideal, but it keeps the constants related to their insertion point (defaults for the network config)

/land


Make sure we can compare a MatrixGate and a ControlledGate

We need to be able to say that CNOT defined as a MatrixGate is equal to the ControlledGate equivalent

Code: __eq__ in gate derived classes squirrel_ir.py


timeout in entry_point_waiter

Timeout in entry_point_waiter doesn't trickle down into wait_for_worker_idle. This becomes a problem is entry_point_waiter timeout is higher than the one in wait_for_worker_idle, as only the lowest timeout value will apply. This causes the entry point waiter to timeout  too soon.
https://github.com/onefinestay/nameko/blob/master/nameko/testing/services.py#L100

The issue here is that entrypoint_waiter requires the container to be idle before inserting itself as a dependency. It wouldn't be correct to pass the timeout value into wait_for_worker_idle, because the insert needs to happen before the waiter even starts counting.
This is a limitation of the way entrypoint_waiter is designed. If there's a long-running worker you can't use it at all. Furthermore if it's configured to wait on an entrypoint that fires but never completes, it will hang forever (doing wait_for_worker_idle again, before removing itself as a dependency).
I think the correct solve for this is to redesign the entrypoint_waiter so it's no longer a dependency.


when parsedStmt isDDL,Database is should not ignore

if we ignore the Database,Explicit assignment database name in sql is in need,That  maybe unreasonable.
DDL语句需要注入数据库名
sql example:
String sql="ALTER TABLE HITS02 ADD COLUMN Added1 UInt32 FIRST";

Hi @xj20305, did you try latest code on develop branch?

Hi @xj20305, did you try latest code on develop branch?
Hello,zhicwu
when i use version clickhouse-jdbc-0.3.1.jar I find this problem,
I try it in master last friday,and  fix it here , it work well in my project.

Thanks @xj20305. The issue has been fixed in PR #651. 0.3.1-patch will be available shortly.


Investigate slow outliers

Sometimes when running the benchmarks I get like 8k enqueued and dequeued instead of 80k. Something to look into.

Not sure how to do this


oauth-proxy: raise CPU request and limit.

With a CPU limit of 100m, oauth-proxy seems unable to cope with the load associated with its own liveness probes, leading to the model mesh pods being restarted every so often once a certain number of routes (inference services) are created.
Raise the CPU limit from 100m to 2.
Raise the CPU request from 100m to 0.5.
Related-to: https://github.com/opendatahub-io/modelmesh-serving/issues/62
Related-to: https://github.com/opendatahub-io/modelmesh-serving/issues/16
Signed-off-by: François Cami<EMAIL_ADDRESS>Motivation
Fix https://github.com/opendatahub-io/modelmesh-serving/issues/62
Modifications
diff --git a/config/internal/base/deployment.yaml.tmpl b/config/internal/base/deployment.yaml.tmpl
index c048971..7c4c18f 100644
--- a/config/internal/base/deployment.yaml.tmpl
+++ b/config/internal/base/deployment.yaml.tmpl
@@ -162,10 +162,10 @@ spec:
             failureThreshold: 3
           resources:
             limits:
-              cpu: 100m
+              cpu: 2
               memory: 256Mi
             requests:
-              cpu: 100m
+              cpu: 500m
               memory: 256Mi
           volumeMounts:
             - mountPath: /etc/tls/private

Result

/ok-to-test

Please hold merging until I get the chance to test this change.


[ADLS Gen2] Path - Create support in storage SDK

https://docs.microsoft.com/en-us/rest/api/storageservices/datalakestoragegen2/path/create
Current blocker:
Setting headers from Request to Http client.
Getting headers from Http client to Response.
Http pipeline support.

Scope:
Validation and Documentation API support is included.

Done.