Vendor common dependencies into the repository

We are currently working on shipping a browser build. #6202 fixes one of the issues @paullewis found; we need file extensions on our imports to match the ESM spec.
The next challenge is the import of dependencies that we use in any environments. The only dependency we have currently that's common to all environments is Mitt. Right now our bundled code ships as import mitt from 'mitt' but a browser has no idea what mitt refers to and how to resolve it.
importMaps may provide a solution in the future but they are not at a point to be relied on yet.
I'm proposing that we vendor any common dependencies into this repository. This means we can import them as import mitt from './common/third_party/mitt/mitt.es.js (exact path TBC) and that will work in the browser and Node equally well.
The downside is we have to manage these dependencies but I think our list of dependencies will be very small; the vast majority of Pptr's dependencies are for extracting files from zips/DMGs/etc and I think as a rule we'll be very considered about adding new common dependencies in the future. If needed we can introduce some scripts to make managing them and updating them easier (like we have in DevTools-Frontend).

@mathiasbynens @paullewis @OrKoN @hanselfmu PTAL :)

Do you expect this to only apply to mitt, or other dependencies?
I don't know of any other way of doing this besides rolling in a third_party or lib folder, and certainly if the number of deps is low, and it allows the codebase to be fully agnostic I'm personally very supportive.

It looks like Mitt does not have any dependencies, and seems like a single build can work in both the browser and Node, so this solution seems quite promising, at least for this interim period.
And I agree that we should be careful about adding other common dependencies, because not all the libraries out there are as self-contained as Mitt 😃

Do you expect this to only apply to mitt, or other dependencies?

It would apply to any common dependencies - that is, ones that are used in both the browser and Node version of Puppeteer.

It looks like Mitt does not have any dependencies, and seems like a single build can work in both the browser and Node, so this solution seems quite promising, at least for this interim period.

This is a good point, if Mitt had 5 dependencies, we'd have a bigger challenge on our hands. I hope we can avoid this and keep our dependencies down to a minimum. In our case Mitt being tiny really helps.
As @paullewis says I can't think of another way to do this; I'll put together a proof of concept to see how hard/easy/painful/etc it is.

I believe this one is fixed.


Add missing generic parameter to Codec.fuse method.

To fix https://github.com/dart-lang/sdk/issues/27722.

CC @leafpetersen @lrhn @floitschG

I think this needs conflicts resolved. But changes LGTM.

@har79 Would you rebase this on the latest head?


Add keyboard accelerators for Attach commands.

Keyboard accelerators for Attach commands are
bound to Ctrl+[, Alt+[, and Ctrl+Alt+[. These are
picked for mnemonic harmony with Ctrl+Alt+P.
Use names that start with "Attach" for these
commands in the menus, so that they're easier
to find in the keyboard customization menu.
The names in that menu take the form
Debug.WhateverTheButtonTextIs, so we were
winding up with Debug.Attach

vs-chromium is not just for chromium project, it works well on my own project.


TEPP Fixes

hey @wsmelton  - you can add on to this branch

See this PR:
https://github.com/sqlcollaborative/dbatools/pull/1241
Might be an idea to do #1241 first, then add it to the files

hey @wsmelton - please go back and add
Register-DbaTeppArgumentCompleter -Command THE-DBACOMMANDNAME -Parameter Database, Exclude to the end of each function file (outside of the Function {} block because it has to run on module load)


feat(monitor): monitor alert policy add mobile notify

这个 PR 实现什么功能/修复什么问题:
1.报警策略增加短信通知
是否需要 backport 到之前的 release 分支:

release/3.7

/cc @zexi
/area monitor

Codecov Report

Merging #10037 (005779f) into master (4cd9813) will increase coverage by 2.08%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           master   #10037       +/-   ##
===========================================
+ Coverage    5.97%    8.05%    +2.08%
===========================================
  Files         849      107      -742
  Lines      132261    12992   -119269
===========================================
- Hits         7897     1047     -6850
+ Misses     123663    11825   -111838
+ Partials      701      120      -581

Flag
Coverage Δ

aFlag
8.05% <0.00%> (+2.08%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/apigateway/handler/auth.go
0.00% <0.00%> (ø)

pkg/compute/models/groups.go

pkg/compute/guestdrivers/ucloud.go

pkg/compute/models/loadbalanceraclresource.go

pkg/mcclient/options/dbinstance_skus.go

pkg/mcclient/options/dns_trafficpolicy.go

pkg/compute/models/secgroupresource.go

pkg/mcclient/options/k8s/secret.go

pkg/compute/models/loadbalancerawscachedlbb.go

pkg/compute/models/route_table_associtaion.go

... and 730 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #10037 (005779f) into master (4cd9813) will increase coverage by 2.08%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           master   #10037       +/-   ##
===========================================
+ Coverage    5.97%    8.05%    +2.08%
===========================================
  Files         849      107      -742
  Lines      132261    12992   -119269
===========================================
- Hits         7897     1047     -6850
+ Misses     123663    11825   -111838
+ Partials      701      120      -581

Flag
Coverage Δ

aFlag
8.05% <0.00%> (+2.08%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/apigateway/handler/auth.go
0.00% <0.00%> (ø)

pkg/compute/models/groups.go

pkg/compute/guestdrivers/ucloud.go

pkg/compute/models/loadbalanceraclresource.go

pkg/mcclient/options/dbinstance_skus.go

pkg/mcclient/options/dns_trafficpolicy.go

pkg/compute/models/secgroupresource.go

pkg/mcclient/options/k8s/secret.go

pkg/compute/models/loadbalancerawscachedlbb.go

pkg/compute/models/route_table_associtaion.go

... and 730 more

Continue to review full report at Codecov.

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


SparkR Support

What changes were proposed in this pull request?
Initial Spark R support
How was this patch tested?

[x] Initial submission step
[x] Unit Tests
[x] Docker files (tested)
[x] Integration Tests

Upon merging, this PR closes #506

To run integration tests in a proper R environment, it is required that R_HOME is defined in the testing environment which means everyone would need to install R. Is that something that would be an issue or something that can be assumed if someone is building out a full dev environment for Spark?

Hmm, interesting. The submitting node has an R dependency? or the driver?

The R dependency is because there is a need to mimic the make-distribution environment in target/docker/R so that when we run ADD R opt/spark/R it is already packaged in the Docker environment; this is similar to how we setup PySpark.

@ssuchter @varunkatta Integration test will pass after R is installed and R_HOME is defined in the jenkins environment.
PR is otherwise ready for review

rerun integration tests please

rerun integration tests please

rerun integration tests please

PR is ready for review @foxish @erikerlandson

@ifilonenko you also need to update sbin/build-push-docker-images.sh to add the new images.

rerun unit tests please

Ready for merging to branch-2.2 unless any other concerns @erikerlandson @foxish @liyinan926

LGTM. Thanks for the work!

rerun unit tests please

Unless there are any further comments, I think this is ready to merge

All set to merge? @foxish @erikerlandson @liyinan926

I think it's all good.

hi - what's left for this work? this https://github.com/apache-spark-on-k8s/spark/pull/507#discussion_r143595440?

I left some comments. Also would like to see this tested in a production environment, but maybe we can just merge it and follow up as feedback comes in.

@mccheah as per your last comment. Is this okay to merge then?

Can merge when CI passes - I just updated the branch.

ready for merging: @foxish

Nvm. Please ignore last comment. That was in the merge commit.


Timer added and switches to next question

Was sick so wasn't able to solve it early...The  timer counts now and switches to the next question. Can be modified to add an action when all questions has been exhausted

Thanks for contributing ⚙ I will verify and merge it as soon as possible!


Add EditorConfig generator

Task
TR#89
Why?
Add .editorconfig generator along with the VSCode extensions.json to recommend appropriate setup.
What Changed

[x] Add generator for .editorconfig
[x] Add plugin for EditorConfig for VS Code
[x] @jdmcleod - Verify that there is no need for an extension for RubyMine

Screenshots
VSCode Recommendation

Output

I guess we probably would want a version bump with this...?


Implemented a workaround to support node v18

Azure pipelines has updated windows-latest to run on node v18.x.x on Febuary 17th.
Node 18 has bumped some requirements about open-ssl and Webpack 4 will not be updated to support this use case.
The correct solution is to upgrade to webpack 5 but this has the potential to be a massive task. In the meantime one can pass openssl-legacy-provider for node v18 to support the legacy ssl methods. This PR is a workaround to make our current build work and be able to release DNN v9.11.1 I'll create an issue to do this in a more proper manner after I can confirm this PR fixes our issue in CI.

Summary

This is a release management task, as per our policy, we are self-approving it.

Glad it worked!


Replace feather icons with lucide icons

While feather icons are great, it has a limited set of icons to use and the collection is not growing fast enough. However lucide icons is a fork of feather icons and is community-driven and has a lot more icons to work with. https://github.com/lucide-icons/lucide/tree/main/packages/lucide-angular#lucide-angular
Considering lucide icons is a fork of feather icons, I expect it to be an easy swap of the two libraries without issues.

Hi @imolorhe, I'll setup the project here and check how the app behaves with the lucide icons

Hi @imolorhe, I'll setup the project here and check how the app behaves with the lucide icons

Awesome!

It's been a while I developed Angular apps (worked with Angular 8) and I found some problems that I worked around on the PR above.

ngcc could never find the node_modules folder inside the node_modules/lucide-angular directory, so I couldn't install the dependency. Going back to an older version of lucide-angular fixed the problem. Versions 0.87.0 and above have this problem. Do you know if this something related with the Angular version? On the PR I'm using 0.84.0.

Importing the icons from lucide-angular/icons didn't work during the compilation, so I imported them from lucide-angular directly.

yarn local-verify finished successfully and the icons that I look are identical before and after the changes

Thanks for the fast turnaround time! It sounds weird that newer versions of the library will have issues with ngcc. Typically it's the older versions that have issues. It is fine to merge the PR with the older version. I can take a look afterwards to update the version.


Create context target label when it doesn't exist in PDS page

Create context target label when it doesn't exist in https://pds.nasa.gov/data/pds4/context-pds4/target/

Implemented and merged into master.


DOMException on disabled cookies in browser

When customer click on a payment, should be open popup with payment form, but thrown exceptions in the browser console:

Problem can be reproduced with disabled browser cookie:
https://www.chromium.org/for-testers/bug-reporting-guidelines/uncaught-securityerror-failed-to-read-the-localstorage-property-from-window-access-is-denied-for-this-document
I would suggest some custom event on which we could display message for customer.

Hi, si-dge!
Thank you for reporting this issue. Let us take some time to investigate your problem. As soon as we have any news, we'll update this issue.

Hi, si-dge!
Thank you for your patience. We have found the cause of this exception. However, it can take a while to fix this. We have plans to release the patch in the end of April.


root filesystem and chroot example not working

Hello Liz,
Fantastic session . howeever when i am trying to following the instructions for setting rootfs using chroot, not able to make it work
root@simpleLinuxVM:/# go run main.go run /bin/bash
Running [/bin/bash]
Running [/bin/bash]
panic: fork/exec /bin/bash: no such file or directory
Appreciate your advice.

Hi thanks for your talks !
I've got the same problem, even though, I tried the same code of multiple of your differents talks on "building container from scratch". Finally I tried the code of your repo and I still have this issue :
panic: fork/exec /bin/bash: no such file or directory
My filesystem that I use as a folder is supposed to work, I tried the bash binary which is in it and it works from the host.
(For information I took this from this repo : )
Thank you in advance for the help

@saikovvuri @vincentchemouny
docker run -d --rm --name ubuntufs ubuntu:20.04 sleep 1000
docker export ubuntufs -o ubuntufs.tar
docker stop ubuntufs
mkdir -p /home/liz/ubuntufs
tar xf ubuntufs.tar -C /home/liz/ubuntufs/

@saikovvuri @vincentchemouny
docker run -d --rm --name ubuntufs ubuntu:20.04 sleep 1000
docker export ubuntufs -o ubuntufs.tar
docker stop ubuntufs
mkdir -p /home/liz/ubuntufs
tar xf ubuntufs.tar -C /home/liz/ubuntufs/


GitPlugin causes crash

Tested versions
4.4.dev
System information
Windows 11 - Redot Engine v4.4.dev.custom_build.5e7e5fc60
Issue description
Redot crashes when trying to open an existing Godot project with the following message
ERROR: Cannot get class 'GitPlugin'.
   at: (core\object\class_db.cpp:552)
ERROR: Received a nullptr VCS extension instance during construction.
   at: (editor\plugins\version_control_editor_plugin.cpp:151)

================================================================
CrashHandlerException: Program crashed
Engine version: Redot Engine v4.4.dev.custom_build (5e7e5fc60f1e75cb272ebf650fd37c51e0fa16fa)
Dumping the backtrace. Please include this when reporting the bug to the project developer.
[0] <couldn't map PC to fn name>
[1] <couldn't map PC to fn name>
[2] <couldn't map PC to fn name>
[3] <couldn't map PC to fn name>
[4] <couldn't map PC to fn name>
[5] <couldn't map PC to fn name>
[6] <couldn't map PC to fn name>
[7] <couldn't map PC to fn name>
[8] <couldn't map PC to fn name>
[9] <couldn't map PC to fn name>
[10] <couldn't map PC to fn name>
[11] <couldn't map PC to fn name>
[12] <couldn't map PC to fn name>
[13] <couldn't map PC to fn name>
[14] <couldn't map PC to fn name>
[15] <couldn't map PC to fn name>
[16] <couldn't map PC to fn name>
[17] <couldn't map PC to fn name>
[18] <couldn't map PC to fn name>
-- END OF BACKTRACE --

Steps to reproduce
Open existing project from Godot 4.3
Minimal reproduction project (MRP)
N/A

Hm, we have not had this issue yet. Could you try opening the same project in Godot 4.4dev and see if that causes the same error?

Same with godot 4.4-dev3

Roger, that's an indicator that Godot's 4.4dev work broke it and we just need to leave it to them to fix it so we don't cause merge conflicts.


headword wrapping for examples no longer works (reported by Ivana)

The authors of the Croatian dictionary of Idioms used to be able to mark idioms or keywords (using ) in examples, but this feature is no longer available. Was it removed with the latest update?

I think this must have been left out when moving to Python.

This was caused by bad formatting in default dictionary template. Newly created dictionaries will have correct formatting.
To fix this in existing dictionaries, go to Configure-Entry formatting and choose style for h


Rake Task: avoid calling super with arguments to avoid an ArgumentError

Rake's RakeLib base class does not define a specific initializer.
In the initializer for our RakeTask creation class, we call super sans args.
When we do call it with a specific name, super fails like:
ArgumentError: wrong number of arguments (given 1, expected 0)
/home/travis/build/dev-sec/chef-os-hardening/vendor/bundle/ruby/2.6.0/gems/github_changelog_generator-1.16.0/lib/github_changelog_generator/task.rb:34:in `initialize'
/home/travis/build/dev-sec/chef-os-hardening/vendor/bundle/ruby/2.6.0/gems/github_changelog_generator-1.16.0/lib/github_changelog_generator/task.rb:34:in `initialize'

Fixes #942

Tested this (with super()) and it works for me now.

Thanks! Should I expect a new release soon ?

Everything depends on things not going bad when releasing. If they do, I lose hope and then things take more time.
Now, I had to:

use the invalid label and redo the changelog generation to exclude "an Issue which was not about this repo"
due to Branch Protection, I had to spend time on making a PR, to bump version & generate changelog
then I had to break MORE rules of Branch Protection due to other maintainers not being there
then I could build and release to RubyGems
And then, resetting Branch Protection

So, this is a hassle. And I've already quit this maintainership.

@mfortin PS: the 1.16.1 is out. Try it!

Thank you @olleolleolle

Thank you very much @olleolleolle 🙌
I see your pain with this. lmk, how we can we improve this.
I propose to keep release rules simple.
Probably we can remove release branch protection things? In order to make releases faster.
Btw, @mfortin @kennyadsl wants to point out that this repo needs collaborators. so if some want to participate - you are very welcome! https://github.com/github-changelog-generator/github-changelog-generator/issues/727
If you are using this repo, it would be great if someone can help to help to maintain and release new things for this repo.

Updated the Settings to remove the "requires a review to be merged". All other protections remain.


[BLE] Callback function calls three or four times

Hello,
Thank you so much for providing these very usefull C++ classes!
I am mostly using the BLE classes. Lately, a bug appeared. When I click on the read button on nRF Connect, the callback function is called three or four times without any reason. Maybe, you can help me figure it out this bug.
Thank you!
Leo
My code :
void run_server(void) {
   MainBLEServer* pMainBleServer = new MainBLEServer();
   pMainBleServer->setStackSize(20000);
   pMainBleServer->start(static_cast<void*>(&gasValue));

}

My Bluetooth server:
void MainBLEServer::run(void *data) {

	BLEDevice::init("MYDEVICE");
	BLEServer* pServer = BLEDevice::createServer();

	GasValue* gasPtr = static_cast<GasValue*>(data);

	BLEService* pServiceValue = pServer->createService("91bad492-b950-4226-aa2b-4ede9fa42f59");

	BLECharacteristic* pCharacteristicValue = pServiceValue->createCharacteristic(
		BLEUUID("0d563a58-196a-48ce-ace2-dfec78acc814"),
		BLECharacteristic::PROPERTY_READ
	);
	pCharacteristicValue->setCallbacks(new CallbackValue(*gasPtr));
	pCharacteristicValue->setValue("READ");

	BLECharacteristic* pCharacteristicRaw = pServiceValue->createCharacteristic(
		BLEUUID("01bf2093-f2d3-4fcc-8d2b-eda1667d96f4"),
		BLECharacteristic::PROPERTY_READ
	);
	pCharacteristicRaw->setCallbacks(new CallbackRawValue(*gasPtr));
	pCharacteristicRaw->setValue("READ_RAW");
	pServiceValue->start();

	BLEService* pServiceWrite = pServer->createService("3978601c-634d-41c1-a33f-dbdd92b68dbf");
	BLECharacteristic* pCharacteristicWrite = pServiceWrite->createCharacteristic(
		BLEUUID("16ce95bb-8ce1-48ff-8043-680cb323324d"),
		BLECharacteristic::PROPERTY_WRITE
	);
	pCharacteristicWrite->setCallbacks(new CallbackWrite());
	pCharacteristicWrite->setValue("WRITE");
	pServiceWrite->start();

	BLEAdvertising* pAdvertising = pServer->getAdvertising();
	pAdvertising->addServiceUUID(BLEUUID(pServiceValue->getUUID()));
	pAdvertising->addServiceUUID(BLEUUID(pServiceWrite->getUUID()));
	pAdvertising->start();
	delay(1000000);
}

My callback function (executed three or four times):
void CallbackValue::onRead(BLECharacteristic *pCharacteristic) {
	cout << "********** READ VALUE " << endl;
	gasValue = m_gasVal.get_gasValue();
	cJSON *gasJSON;
	gasJSON = cJSON_CreateObject();
	cJSON_AddNumberToObject(gasJSON, "CO",(float)gasValue.CO);
	cJSON_AddNumberToObject(gasJSON, "CO2",(float)gasValue.CO2);
	cJSON_AddNumberToObject(gasJSON, "NO2",(float)gasValue.NO2);
	cJSON_AddNumberToObject(gasJSON, "VOC",(float)gasValue.VOC);
	string gasStr(cJSON_Print(gasJSON));
	cJSON_Delete(gasJSON);
	pCharacteristic->setValue(gasStr);
}

Can you paste logs that shows all read in callback?

Test plan: Create a BLE Server that invokes a callback when a read request is made by a client and see how often it gets invoked.

Thank you for your responses.
I use ESP32 to read gas sensor and transmit the value via BLE.
The example provided in cpp_utils works properly. However, since I changed MyCallBackHandler class in order to have access to the gas measurements, it starts to call OnRead function several times as you can see on the following screenshot.

(@chegewara, if you want ESP32 logs, I don't really know how to access/activate them...)
One possibility is that the Task running the server is  lauchned several times since I did my modification. I looked at the code executed in the function and I don't see why.
@nkolban: unless I don't understand what you suggest, I've already done that running your example. The problem appeared after I did a pretty simple modification on the example.
I attached the source code of my classes with comments on what I changed (based on the examples).
MyBLE.zip
Thank you.
Leo

Sorry Leo ... my apologies ... the last post was for ME ... as I make my notes ... I'm working on the puzzle right now and trying to get into the habit of logging what I do :-)

Sorry its not english, but you could have turn on logs verbose:

I tested with two characteristics ... and still only saw the callback run once for any given characteristic.  So I then went back to your code and read it carefully and noticed that you have created TWO service.  I'm not saying that is a problem ... but that hasn't been in my test.  Am testing that scenario now.

Having 2 services with the same UUID is not allowed, at least in this library

@nkolban I tested to create several services before and it was not an issue (see ESP32 project attached)
ble_demo.zip
@chegewara I don't use Arduino IDE but thanks to your message, I found something under make menuconfig! Thank you! I will be back soon with some information!

I put verbose mode and that's not good... Nothing is executed, the binary seems too big to fit! (I have a pretty heavy project). I'm going to try to make it lighter.

No need, just create custom partition table like this one https://github.com/nkolban/ESP32_Explorer/blob/master/partitions.csv and setup it in menuconfig

Thank you, it worked. Please find attached the logs when I connect and read a value with my phone.
D (94769) BLEDevice: gattServerEventHandler [esp_gatt_if: 3] ... ESP_GATTS_CONNECT_EVT
D (94769) BLEUtils: GATT ServerEvent: ESP_GATTS_CONNECT_EVT
D (94769) BLEUtils: [conn_id: 0, remote_bda: 4c:6c:3b:02:ed:8e]
D (94769) BLEServer: >> handleGATTServerEvent: ESP_GATTS_CONNECT_EVT
V (94779) FreeRTOS: Semaphore giving: name: ConfEvt (0x3ffdf454), owner: <N/A>
V (94789) FreeRTOS: Semaphore giving: name: ConfEvt (0x3ffdeb94), owner: <N/A>
V (94799) FreeRTOS: Semaphore giving: name: ConfEvt (0x3ffded7c), owner: <N/A>
D (94799) BLEServer: << handleGATTServerEvent
D (95449) BLEUtils: Received a GAP event: ESP_GAP_BLE_UPDATE_CONN_PARAMS_EVT
D (95449) BLEUtils: [status: 0, bd_addr: 4c:6c:3b:02:ed:8e, min_int: 0, max_int: 0, latency: 0, conn_int: 6, timeout: 2000]
D (95459) BLEServer: BLEServer ... handling GAP event!
D (95799) BLEUtils: Received a GAP event: ESP_GAP_BLE_UPDATE_CONN_PARAMS_EVT
D (95799) BLEUtils: [status: 0, bd_addr: 4c:6c:3b:02:ed:8e, min_int: 0, max_int: 0, latency: 0, conn_int: 39, timeout: 2000]
D (95809) BLEServer: BLEServer ... handling GAP event!
D (98139) BLEDevice: gattServerEventHandler [esp_gatt_if: 3] ... ESP_GATTS_READ_EVT
D (98139) BLEUtils: GATT ServerEvent: ESP_GATTS_READ_EVT
D (98139) BLEUtils: [conn_id: 0, trans_id: 1, bda: 4c:6c:3b:02:ed:8e, handle: 0x2a, is_long: 0, need_rsp:1]
D (98149) BLEServer: >> handleGATTServerEvent: ESP_GATTS_READ_EVT
********** READ VALUE
D (98159) BLECharacteristic: >> setValue: length=48, data=7b0a0922434f223a0932352c0a0922434f32223a09362c0a09224e4f32223a093137392c0a0922564f43223a09300a7d, characteristic UUID=0d563a58-196a-48ce-ace2-dfec78acc814
D (98179) BLECharacteristic: << setValue
D (98189) BLECharacteristic: Sending a response (esp_ble_gatts_send_response)
D (98189) BLECharacteristic:  - Data: length=22, data=7b0a0922434f223a0932352c0a0922434f32223a0936, offset=0
D (98199) BLEServer: << handleGATTServerEvent
D (98209) BLEDevice: gattServerEventHandler [esp_gatt_if: 3] ... ESP_GATTS_RESPONSE_EVT
D (98209) BLEUtils: GATT ServerEvent: ESP_GATTS_RESPONSE_EVT
D (98219) BLEUtils: [status: ESP_GATT_OK, handle: 0x2a]
D (98229) BLEServer: >> handleGATTServerEvent: ESP_GATTS_RESPONSE_EVT
D (98229) BLEServer: << handleGATTServerEvent
D (98289) BLEDevice: gattServerEventHandler [esp_gatt_if: 3] ... ESP_GATTS_READ_EVT
D (98289) BLEUtils: GATT ServerEvent: ESP_GATTS_READ_EVT
D (98289) BLEUtils: [conn_id: 0, trans_id: 2, bda: 4c:6c:3b:02:ed:8e, handle: 0x2a, is_long: 1, need_rsp:1]
D (98299) BLEServer: >> handleGATTServerEvent: ESP_GATTS_READ_EVT
********** READ VALUE
D (98309) BLECharacteristic: >> setValue: length=48, data=7b0a0922434f223a0932352c0a0922434f32223a09362c0a09224e4f32223a093137392c0a0922564f43223a09300a7d, characteristic UUID=0d563a58-196a-48ce-ace2-dfec78acc814
D (98329) BLECharacteristic: << setValue
D (98329) BLECharacteristic: Sending a response (esp_ble_gatts_send_response)
D (98339) BLECharacteristic:  - Data: length=22, data=2c0a09224e4f32223a093137392c0a0922564f43223a, offset=22
D (98349) BLEServer: << handleGATTServerEvent
D (98359) BLEDevice: gattServerEventHandler [esp_gatt_if: 3] ... ESP_GATTS_RESPONSE_EVT
D (98359) BLEUtils: GATT ServerEvent: ESP_GATTS_RESPONSE_EVT
D (98369) BLEUtils: [status: ESP_GATT_OK, handle: 0x2a]
D (98369) BLEServer: >> handleGATTServerEvent: ESP_GATTS_RESPONSE_EVT
D (98379) BLEServer: << handleGATTServerEvent
D (98429) BLEDevice: gattServerEventHandler [esp_gatt_if: 3] ... ESP_GATTS_READ_EVT
D (98439) BLEUtils: GATT ServerEvent: ESP_GATTS_READ_EVT
D (98439) BLEUtils: [conn_id: 0, trans_id: 3, bda: 4c:6c:3b:02:ed:8e, handle: 0x2a, is_long: 1, need_rsp:1]
D (98449) BLEServer: >> handleGATTServerEvent: ESP_GATTS_READ_EVT
********** READ VALUE
D (98459) BLECharacteristic: >> setValue: length=48, data=7b0a0922434f223a0932352c0a0922434f32223a09362c0a09224e4f32223a093137392c0a0922564f43223a09300a7d, characteristic UUID=0d563a58-196a-48ce-ace2-dfec78acc814
D (98469) BLECharacteristic: << setValue
D (98479) BLECharacteristic: Sending a response (esp_ble_gatts_send_response)
D (98489) BLECharacteristic:  - Data: length=4, data=09300a7d, offset=44
D (98489) BLEServer: << handleGATTServerEvent
D (98499) BLEDevice: gattServerEventHandler [esp_gatt_if: 3] ... ESP_GATTS_RESPONSE_EVT
D (98499) BLEUtils: GATT ServerEvent: ESP_GATTS_RESPONSE_EVT
D (98509) BLEUtils: [status: ESP_GATT_OK, handle: 0x2a]
D (98519) BLEServer: >> handleGATTServerEvent: ESP_GATTS_RESPONSE_EVT
D (98519) BLEServer: << handleGATTServerEvent

I see some issues here, when did you last time update cpp_snippets library?

This morning! (git clone and copy-paste into my project)

You see this logs:
********** READ VALUE
D (98309) BLECharacteristic: >> setValue: length=48, data=7b0a0922434f223a0932352c0a0922434f32223a09362c0a09224e4f32223a093137392c0a0922564f43223a09300a7d, characteristic UUID=0d563a58-196a-48ce-ace2-dfec78acc814
D (98329) BLECharacteristic: << setValue
D (98329) BLECharacteristic: Sending a response (esp_ble_gatts_send_response)
D (98339) BLECharacteristic:  - Data: length=22, data=2c0a09224e4f32223a093137392c0a0922564f43223a, offset=22
D (98349) BLEServer: << handleGATTServerEvent
D (98359) BLEDevice: gattServerEventHandler [esp_gatt_if: 3] ... ESP_GATTS_RESPONSE_EVT
D (98359) BLEUtils: GATT ServerEvent: ESP_GATTS_RESPONSE_EVT
D (98369) BLEUtils: [status: ESP_GATT_OK, handle: 0x2a]
D (98369) BLEServer: >> handleGATTServerEvent: ESP_GATTS_RESPONSE_EVT
D (98379) BLEServer: << handleGATTServerEvent
D (98429) BLEDevice: gattServerEventHandler [esp_gatt_if: 3] ... ESP_GATTS_READ_EVT
D (98439) BLEUtils: GATT ServerEvent: ESP_GATTS_READ_EVT
D (98439) BLEUtils: [conn_id: 0, trans_id: 3, bda: 4c:6c:3b:02:ed:8e, handle: 0x2a, is_long: 1, need_rsp:1]
D (98449) BLEServer: >> handleGATTServerEvent: ESP_GATTS_READ_EVT

We can see that setValue is 48 bytes long and it should be send with 3 segments, but is send only one segment with 22 bytes. Even if here we have set flag is_long:
D (98439) BLEUtils: [conn_id: 0, trans_id: 3, bda: 4c:6c:3b:02:ed:8e, handle: 0x2a, is_long: 1, need_rsp:1]
I dont know why is this happened, i have to check with some code but until now it seemed to works fine to me.

Indeed, weird enough... Maybe the value to send is too long?
However, I receive a complete and correct JSON on nRF Connect...

Give us some time to make tests.

Or just let mr @nkolban to fix the issue 👍

What a great bug!!!   Thank you @Granyy ... perfect report and perfect debug traces.   The problem has been found and resolved and a fix applied to the code base.
For our interest and records, here is the back story.
When we are running a BLE Server on the ESP32, the ESP32 is passively sitting there waiting for read requests to arrive.  These will be requests from the partner BLE client saying "Hello, please give me the value of the characteristic".  This results in an event being raised asynchronously inside the BLE C++ classes which says "Here is a request to read a characteristic".  In that processing, we say "Do we have a user callback that will be invoked to get the value ... if yes ... call it".   That all sounds sane and fair.  However, there was a scenario we missed.
When a read request arrives, we don't send back the WHOLE data ... instead the data is chunked into MTU units ... this is the biggest chunk of data that can be transmitted in one time.  For example, if our data to send back is 60 bytes and our MTU is 25, then we will need to send back 3 units ... 25 + 25 + 10.  When we send back the first unit, the client will then come back and say "Thank you, now send me the next unit".   And in there was the problem.  We were invoking the user callback function for EVERY request to read data ... not just the first request to read data in a chunked sequence.  The code has now been changed so that we invoke the user callback only for the FIRST unit of a chunked chain and not every one.
Please get the latest code and see if all is now good.  If not, PLEASE let us know as soon as possible.

It works great! :D
Thank you @nkolban!
(One question out of context: is there a way to catch the  connection/deconnection  event of a device? -  If you could just named the class or the function and I'll figure it out myself how to do this)
Thank you again!

@Granyy There is BLEServer callback function with onConnect() and onDisconnect()

I just noticed, that the same bug (or a very similar one) appears on the callbacks for descriptors as well.
Same things are happening:
Read the descriptor once
Got 3 callbacks


feat(outbound-redis): add incr

resolves https://github.com/fermyon/spin/issues/668
Run rust-outbound-redis for testing:
> cargo build
> cd examples/rust-outbound-redis
> ../../target/debug/spin build
> ../../target/debug/spin up

> curl -i http://<IP_ADDRESS>:3000/publish
HTTP/1.1 200 OK
content-length: 0
date: Thu, 04 Aug 2022 15:20:13 GMT

It appears that all your commits are not signed but once you do that, we can merge 🚀

Updated it. Thank you.


Fix logo behavior on mobile

Personal logo not turning red after being selected, inconsistent with the rest of the social media logos.

For some reason, this has fixed itself


fix: update broken link for MonitoRSS website logo

Right now, the link to the website logo seems to be for an image that was uploaded to Discord some time ago. However, that image is now inaccessible for some reason (error 403 - forbidden).
This updates the link for the website logo to the publicly accessible one that's already in the public folder.
For self-hosted instances, it will point to that instance's publicly accessible favicon image.

Apologies for missing this - this should now be fixed.


Levenshtein distance for metrics widget

Is your feature request related to a problem? Please describe.
It would be nice to have more metrics available in the metrics widget.
Describe the solution you'd like
An additional metric should be added to the metrics widget displaying the Levenshtein distance between two columns.
Additional context
arxiv-summarization

Hi, I'd like to work on this. Could you please assign it to me?

Hello, I want to work on this. Could you please assign it to me?

Hi @BansalRaman, I've already started working on this issue.

FYI: a better dataset to test Levenshtein distance on: https://huggingface.co/datasets/vishnun/SpellGram

@druzsan @steffenslavetinsky I'd like a clarification regarding this issue: Is this metric intended for strings or boolean arrays? I noticed that the input data type for all the metrics in this file is restricted to boolean/number arrays, so I wrote the Levenshtein distance function for boolean arrays. However, this function is generally defined for strings, and the algorithm itself can easily be extended to work on strings, but I'm not sure how to fit it into this particular file. Please let me know what you think. Thank you!

@swetha3456 originally the metric was meant to be applied onto strings (it is a valid datatype). Furthermore, metrics are applied to the whole columns/pair of columns, i.e. Levenshtein distance should be applied to every pair of ith strings and then aggregated (for example, through mean). For example, see accuracy metric.
Feel free to ask if something is still unclear.

@druzsan Thank you for the explanation. I'm still a little confused since the allowed function types of parameters for the "compute" function (as defined in types.ts) does not include an array of strings. Should I add that here?

I think this would be the way to go, simply extend the ValueArray type to also accept strings.

@steffenslavetinsky Thanks for your input! I've submitted a PR after making this change. Please review it and let me know if I should make any changes.


memory

memory

Codecov Report
Patch coverage has no change and project coverage change: -0.14 :warning:

Comparison is base (9dd8728) 15.92% compared to head (38525e8) 15.78%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff              @@
##             master      #15      +/-   ##
============================================
- Coverage     15.92%   15.78%   -0.14%
  Complexity        7        7
============================================
  Files            11        9       -2
  Lines           113      114       +1
  Branches          9        9
============================================
  Hits             18       18
- Misses           95       96       +1

Impacted Files
Coverage Δ

...rc/main/java/ru/milan/parser/AnnotativeMemory.java
0.00% <0.00%> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@rultor merge

@rultor merge

@l3r8yj OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@l3r8yj Done! FYI, the full log is here (took me 2min)


Create Blog “2022-03-09t17-07-25-140z-creating-a-new-post”

Automatically generated by Netlify CMS

👷 Deploy Preview for quizzical-bartik-bcbdff processing.
🔨 Explore the source changes: 1526902ab266f00291709ee98ee959b13f8207f3
🔍 Inspect the deploy log: https://app.netlify.com/sites/quizzical-bartik-bcbdff/deploys/6228deff8d3ac50008ae21cc


Removing Vectorshape Usage

After CircuitPython 7.0.0 the usage of vectorshape has been deprecated. The current library has a rectangle_helper that uses it.
https://github.com/adafruit/Adafruit_CircuitPython_DisplayIO_Layout/blob/29fe3f0bdda917c6c8efbdcba94885eeb6963fa3/adafruit_displayio_layout/widgets/__init__.py#L62
However after CP 7.0 this function becomes obsolete as the API for vectorio needs to add the shape to the group using the corresponding palette. Open for discussion, that having only the fill_area from bitmaptools will be easier than refactoring the cartesian to draw with both libraries. See documentation for vectorio.recatngle. So maybe there is a workaround but I could not find it during my tests.
group = displayio.Group()

palette = displayio.Palette(1)
palette[0] = 0x125690

rectangle = vectorio.Rectangle(pixel_shader=palette, width=40, height=30, x=55, y=45)
group.append(rectangle)

Cartesian is the only library using this, and is already hardcoded to use bitmaptool library only.
https://github.com/adafruit/Adafruit_CircuitPython_DisplayIO_Layout/blob/29fe3f0bdda917c6c8efbdcba94885eeb6963fa3/adafruit_displayio_layout/widgets/cartesian.py#L360

Thank you for the fix @jposada202020.
I tested this out on a Feather ESP32-S2 TFT. The changed version does no longer crash. But I noticed that it doesn't actually draw the rectangle either.
I think it's because it creates a vectorio rectangle object but doesn't return it or do anything else with it, so it's never added to the display or shown.
I think I am in favor of removing the bitmaptool argument altogether instead of fixing it to work with vectorio optionally. The only current usage of this helper has it hardcoded for bitmaptools anyhow, and the function would end up being kind of awkward if it had to return or add a vectorio shape since it behaves differently from the bitmap behavior which can "draw into" a given bitmap without needing to return or do anything else with it.

Yes, Agreed with you, I have even tried to create a new palette object and the color_index passed to the function with no luck. I will work on the PR  for the cartesian widget this week and put this one as a draft. Thanks :)

see #84


NotImplementedError in MacOSX10.15

➜ ~ $ pip3 install pywifi
Collecting pywifi
  Downloading https://files.pythonhosted.org/packages/89/55/170a52685eeefeab36bfaf26bdca629263a0a28efffc3b556b2cc86ab966/pywifi-1.1.12-py3-none-any.whl
Installing collected packages: pywifi
Successfully installed pywifi-1.1.12
You are using pip version 19.0.3, however version 19.3 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
➜ ~ $ python3
Python 3.7.4 (v3.7.4:e09359112e, Jul  8 2019, 14:54:52)
[Clang 6.0 (clang-600.0.57)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import pywifi
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/pywifi/__init__.py", line 15, in <module>
    from .wifi import PyWiFi
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/pywifi/wifi.py", line 15, in <module>
    from .iface import Interface
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/pywifi/iface.py", line 15, in <module>
    raise NotImplementedError
NotImplementedError
>>> exit()

it seems like pywifi only support win or linux, and pywifi also can not run in my macbook

it seems like pywifi only support win or linux, and pywifi also can not run in my macbook

我也遇到了相同的问题

https://github.com/awkman/pywifi/blob/719baf73d8d32c623dbaf5e9de5d973face152a4/pywifi/iface.py#L15 The 10th to 15th  line of ifcace.py file tells that the pywifi only supports Windows and Linux platform but in MacOS it doesn't implement.

aghhhhh why exclude mac peeps :'(((

直接pip install pywifi中的iface.py文件在导包的时候判断系统是没有Mac OS X
pip3 uninstall pywifi
git clone -b macos_dev https://github.com/awkman/pywifi
cd pywifi
pip3 install .

在进行import pywifi就不会报NotImplementedError了

thanks!


Content is not sized correctly for the viewport

Bug report

What is the current behavior?
The viewport size is 424px, whereas the window size is 412px.  This can be seen on mobile phone size screens when you swipe left the page moves.
If the current behavior is a bug, please provide the steps to reproduce.
 We shouldn't render those unnecessary divs in the first place. I'm unsure how to fix it since Gatsby (and React) is putting those divs in the rendering tree.
I'm not gonna be able to look into this in the next couple of weeks so Any suggestions and/or PRs are very welcome!

overflow-x: hidden on body solves the visual issue by removing the horizontal scrollbar, but has other undesirable side effects in that it breaks shouldUpdateScroll if you happen to want to use it. (see https://github.com/gatsbyjs/gatsby/issues/7454) After a little more digging, it looks like a bulma .columns style with negative margins is pushing out the width causing the issue above. Instead of the overflow-x workaround, if you override the .columns margin style, that should do the trick (without breaking shouldUpdateScroll)
While not a great solution, it would have saved me a few hours of headaches if I knew about it.
/src/components/all.sass
...
 // apply to .columns or add another class for specificity
.columns
  margin-left: 0 !important
  margin-right: 0 !important
...

This is an issue within Bulma. Dropped a comment that will hopefully lead to resolution: https://github.com/jgthms/bulma/issues/1540#issuecomment-512969132
In the meantime, we need to provide a container with padding to absorb the negative margins of .columns.

Gotcha. It seems there are some things to be resolved before adding the plugins. For now I have made the PR to fix the footer at least (was quick): https://github.com/netlify-templates/gatsby-starter-netlify-cms/pull/574


Remove no-longer-needed C shims

The published version of libc now has all that's necessary to natively
read these fields!

The libc version specified in Cargo.toml hasn't been updated, is 0.2.60 sufficient or did you forget to commit that?

Ah indeed, can do


Users-permissions routes permissions support ?

Hi, firstly let me thank you for your plugin, people like you that develop and maintain deps like this keep the world spinning, great work! 🙌🏼
Now for my question, is there support or method to target users-permissions routes?
Is it possible to define routes permissions for the plugin's routes?
Let me know if you have tried this! and if I explained myself...
Thanks!

Hello 😊😊😊
Yes its possible, you need to override the plugin users-permissions router.
I'll send you an example tomorow, need 20 hour (french hour)
:)

Thanks Paul, thank you so much, that would be great!
With an example I'll be able to take it were I need 👌

try this (not tested, i have low time today) but come back to me if you need.
file: extensions/users-permissions/strapi-server.js
`
'use strict';
/**

extend User.
*/

const routes = require('./routes');
module.exports = (plugin) => {
//register routes
Object.keys(routes).forEach(key => {
if (!plugin.routes[key]) {
plugin.routes[key] = routes[key];
return;
}
const declaredRoutes = routes[key]
if (!declaredRoutes?.routes || !Array.isArray(declaredRoutes?.routes)) return;
declaredRoutes?.routes?.forEach(route => {
plugin.routes[key].routes.push(route);
});
});
return plugin;
};
`
file: extensions/users-permissions/routes/index.js
const contentApi = require('./content-api'); module.exports = {   "content-api": contentApi, };
file: extensions/users-permissions/routes/content-api.js
module.exports = {   routes: [     {      ...       config: {         prefix: '',       },     },   ] }
In fact we recreate the route config from the plugin users-permissions
I can try saturday (max delay) and give you a full example.

You can try this, i tested and work fine: click me

Thank you for you time and help in this matter, I'll be checking out the recommended plugin, thank for helping a junior dev! 🌱

FYI: The example did kind of work but I am in need of more customization and the recommended plugin works with what I already have, once again thank you for you kind help!

Let me a star, if you want, that encourage me to continue. have fun


New measures and a new stakeholder need to be added for heat

The new stakeholder:

heat_producer

The new measures:

number_of_connections_to_heat_network
kWh_of_heat_consumed
kWh_of_heat_produced (only available for heat_producer)

I have pushed these, but the measure for heat production is temporarily disabled. In isolation, the code works and passes the tests, but in a real network the measure never gets applied due to the lack of a "Heat Network" node in the electricity network. I'll deal with setting the correct heat buffering volume first, then come up with a workaround for this.
In the short-term, you can work around this by measuring heat consumption on the endpoints. Of course this won't work once buffering and loss are enabled.

kWh_of_heat_produced (only available for heat_producer)

Is it genuinely useful to have a measure which says "stakeholder one" pays "stakeholder two" for heat produced by technologies owned by "stakeholder three"? Or does it suffice to price the heat by the marginal cost of the producer, and show that as a cost for the associated stakeholder (much like other costs which aren't explicitly defined in the market model)?
The reason I ask is that this measure does not fit well at all with the way the market model currently works, which measures things happening at the level of nodes and not technologies. It is not impossible to change this, but I think I have to be reassured that it really is worth rewriting half the market model code to do it...

Is it genuinely useful to have a measure which says "stakeholder one" pays "stakeholder two" for heat produced by technologies owned by "stakeholder three"?

I'm afraid so yes. We need to be able to give subsidies to a stakeholder based on its heat production for example.

The reason I ask is that this measure does not fit well at all with the way the market model currently works, which measures things happening at the level of nodes and not technologies.

Could we not add some 'virtual nodes' to the topology: one for every source of heat?
We could even show them in the topology and make them clickable at some point (show the load of that source):

I'm afraid so yes. We need to be able to give subsidies to a stakeholder based on its heat production for example.

Thanks for that, I'll get to work on the changes.

Could we not add some 'virtual nodes' to the topology: one for every source of heat?

While our graph library is perfectly capable of that, much of Moses is predicated on the network being a tree. Orphaned nodes will likely cause problems elsewhere, and would require extra book-keeping during the calculation (to duplicate the load of the heat producers onto these nodes).


static private variables initialization bug

Inconsistent problems with initialized private static variables after asjsc compilation:
ActionScript3 code (SomeClass.as):
class SomeClass {

  private static var x = new AnotherClass();

  public function someMethod() {
    x.anotherMethod();
  }

}

Private accessor prefix for static variable x is missed in initialization code.
Everywhere else that prefix is present.
Javascript code (SomeClass.js):
pac.kage.SomeClass.x = new pac.kage.AnotherClass();

Expected javascript code  (SomeClass.js):
pac.kage.SomeClass.pac_kage_SomeClass_x = new pac.kage.AnotherClass();

I also created minimal example project to reproduce that issue at https://github.com/mrchnk/royale-compiler-problems/tree/master/private-static-variable

Hi @mrchnk , What version of Royale are you using ?

latest npm distribution<EMAIL_ADDRESS>

@mrchnk I'm not an expert from compiler, but since release there were some changes, so you may want to try nightly build and report back whether you have still problems.


Add text embedding processor to neural search

Signed-off-by: Zan Niu<EMAIL_ADDRESS>Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[x] New functionality includes testing.

[x] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed as per the DCO using --signoff

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

Signed-off-by: Zan Niu<EMAIL_ADDRESS>Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[x]  New functionality includes testing.

[x]  All tests pass

[ ]  New functionality has been documented.

[ ]  New functionality has javadoc added

[x]  Commits are signed as per the DCO using --signoff

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

Update the description and issue link.

Signed-off-by: Zan Niu<EMAIL_ADDRESS>Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[x]   New functionality includes testing.

[x]   All tests pass

[ ]   New functionality has been documented.

[ ]   New functionality has javadoc added

[x]   Commits are signed as per the DCO using --signoff

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

Update the description and issue link.

@zane-neo please fix this

How about adding more details in description? Suggest adding some examples there so people know what feature this PR is building.


Must select 24 words in latest "next" Fuel wallet

What version of Wallet are you using?
Version: 0.15.2 (next version)
What is happening?
If the user is simply copying and pasting their seed phrase in the the recover wallet edit fields, there's is no error or notification or recognition that there are 24 words being pasted from the clipboard when the seed phrase screen is set to 12 words.  Therefore the user can blindly and easily recover a completely different wallet from the first 12 words rather than the wallet they want to recover.
steps to reproduce
Install Chrome wallet from https://next-wallet.fuel.network/docs/install/

Paste in the 24 words

The wallet setup accepts a 24 word paste into a 12 word dialog without any complaints
How it should behave?
The dialog should automatically expand to 24 words and we see all 24 words in the dialog. It has been indicated on slack that this is in development but just to be sure and to follow up afterwards, and to retest, I'm creating this bug.
What os are you seeing the problem on?
Mac/Apple Desktop
What browser?
None
What is your browser version?
Version 122.0.6261.112 (Official Build) (x86_64)

This is fixed on version 0.16.2 by #1099

This has been released to the chrome web store.


Simod v3

@Mcamargo85,
These are the recent changes:

Simod restructuring
New replayer integration
CLI interface: discoverer and optimizer as two different commands
Simod now accepts a model as input as well, then it doesn't discover the model's structure and starts with parameters extraction
Pre-processing: removing outliers
Post-processing: Orlenys' recent changes
Memory optimization
GitHub Actions workflow is added for automatic testing (but SplitMiner doesn't work yet because of Xserver issue)

Hi @iharsuvorau , I just checked the commits, and I agree with the changes. I am just wondering if you test the tool on windows, If you remember, we had the issue with the routes, and you reverted the change; I didn't see if you managed to solve it

Hi @iharsuvorau , I just checked the commits, and I agree with the changes. I am just wondering if you test the tool on windows, If you remember, we had the issue with the routes, and you reverted the change; I didn't see if you managed to solve it

Hi,
No, I haven't changed the paths for Windows yet. I'm stuck with the X server issue for SplitMiner now for automated tests to work. The fix for Windows paths is straightforward, I'll add it this week.


Update packages

Summary
Better be on the safe side...

@cpoile FYI I bumped the runner image instead so we keep it stable.


Error Contents: No module named inputstreamhelper

Hi !
I've downloaded your plugin and installed it from the zip, and I'm trying to use it in my plugin as in your example.
But this fires an error :
    import inputstreamhelper
ImportError: No module named inputstreamhelper
-->End of Python script error report<--

Can you help ?
Thanks.

Hi!
Have you imported the module in your addon.xml file?
<requires>
    <import addon="script.module.inputstreamhelper" version="0.3.3"/>
</requires>

@rickybiscus Is this still an issue or can we close it?

I am going to close this as it is more than one year old.
If you still have an issue, feel free to report back.
Thanks!


NIFI-1678

Push heartbeats to ZooKeeper instead of NCM

This is a resubmission of #317 which was closed when we removed a temporary 1.x branch which it was referencing. Please see that PR for additional commentary.

I updated the PR with a new commit to address an issue that I ran into. When we create 'container' nodes in ZooKeeper whose paths are not sensitive, we will mark the node as wide-open, rather than applying ACLs. This allows us to not have to worry about if we have state management and heartbeats using different ACLs.
Also removed use of username/passwords from ZooKeeper configuration. This was done because ZooKeeper indicates that these should be used only for local testing and not for production. In production, security should be administered via Kerberos. Username and password were always sent over the wire unencrypted. Additionally, we are now using Apache Curator to handle Leader Election, and Curator does not appear to support using digest auth via username/password.

I updated the PR with a new commit to address an issue that I ran into. When we create 'container' nodes in ZooKeeper whose paths are not sensitive, we will mark the node as wide-open, rather than applying ACLs. This allows us to not have to worry about if we have state management and heartbeats using different ACLs.
Also removed use of username/passwords from ZooKeeper configuration. This was done because ZooKeeper indicates that these should be used only for local testing and not for production. In production, security should be administered via Kerberos. Username and password were always sent over the wire unencrypted. Additionally, we are now using Apache Curator to handle Leader Election, and Curator does not appear to support using digest auth via username/password.

Running the updated PR I ran into an issue when testing node disconnection while running an embedded ZK. It appears that by shutting down a majority of the nodes, we can no longer write the heartbeats and consequently the last node that was shut down still appears as connected.

Reviewing...

Functionally this is much better than before. Now when running clustered and we start losing nodes we are able to see the current cluster participation. However, once we have lost the quorum we are no longer able to update the Primary Node. This behavior is expected, however. Once the quorum is re-established the Primary is updated accordingly. As we move towards zero master clustering, when there is no quorum we will return appropriate response codes.
Looks good. Thanks!


User managed list of skills

It's common practice to modify the skill list for particular games.
Fate Sheets should support this by allowing users to manage the list of skills.

Need an ability to reset the skills list back to the default, then I'll consider this completed


0.11.0

Release 0.11.0

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@380b902). Click here to learn what that means.

@@            Coverage Diff            @@
##             master      #53   +/-   ##
=========================================
  Coverage          ?   13.49%
=========================================
  Files             ?       14
  Lines             ?     1052
  Branches          ?        0
=========================================
  Hits              ?      142
  Misses            ?      901
  Partials          ?        9

Impacted Files
Coverage Δ

base/settings/settings.go
0% <ø> (ø)

base/settings/startup.go
100% <ø> (ø)

base/pm/runnerhook.go
0% <ø> (ø)

base/pm/pm.go
0% <ø> (ø)

base/pm/runner.go
0% <ø> (ø)

core0/builtin/monitor.go
2.05% <2.05%> (ø)

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 380b902...8de6126. Read the comment docs.


Ray seems not be running at http://localhost:23517

Describe the bug
Ray seems not be running at http://localhost:23517
Request takes time to process and fails to this message. If in this time I open ray app, everything works just fine.
Happening on Windows 10 with laragon.
Versions
Ray version: <IP_ADDRESS>
spatie/ray package version: 1.1.3
spatie/laravel-ray package version (if applicable): 1.1
Laravel version (if applicable): 8.12
Media Library Pro version: N/A
To Reproduce
Steps to reproduce the behavior:

Create a new laravel app
Install laravel-ray package
Add one ray() statement in web.php
Quit ray Windows App
Open laravel default home route.

Expected behavior
As this is an optional developer tool, if ray app is not running, Laravel application should run just fine without considering ray helper just like it runs in prpduction.
Screenshots

Desktop (please complete the following information):

OS: Windows
Version 10

Additional context
Add any other context about the problem here.

Got this same error when running on Homestead on a Mac.. Does it work with homestead?

Got this same error when running on Homestead on a Mac.. Does it work with homestead?

I'll try to reproduce this soon.
@ronnievisser could you try this: https://twitter.com/JoePFerguson/status/1347302653127237634

I'll try to reproduce this soon.
@ronnievisser could you try this: https://twitter.com/JoePFerguson/status/1347302653127237634

@freekmurze It does not make any difference!

@freekmurze It does not make any difference!

I have the same error running on mac with Valet+ even without adding the ray() statement but just with an (known) error in my project. Starting the Ray App also solved it and shows the error in Ray.

I have the same error running on mac with Valet+ even without adding the ray() statement but just with an (known) error in my project. Starting the Ray App also solved it and shows the error in Ray.

...after uninstall and install again it work (1.0.3).

...after uninstall and install again it work (1.0.3).

for homestead:
"Create a ray.php config file at project root and copy the boiler config from the ray docs and then set the host in the config to <IP_ADDRESS> and it should work."
works

for homestead:
"Create a ray.php config file at project root and copy the boiler config from the ray docs and then set the host in the config to <IP_ADDRESS> and it should work."
works

Closing this as people are reporting that it works now.

Closing this as people are reporting that it works now.

If this is still a problem using the latest versions, let me know!

If this is still a problem using the latest versions, let me know!

@freekmurze did anyone test on Windows 10? I'm still unable to run laravel application if ray is not running

@freekmurze did anyone test on Windows 10? I'm still unable to run laravel application if ray is not running

Still having this with Valet+
"spatie/laravel-ray": "^1.1",
Version 1.0.3 (1.0.3)
Laravel Framework 8.16.1
When my application is trying to throw a error message now shows "Ray seems not be running at http://localhost:23517"
When Ray App is running I receive the expected error "InvalidArgumentException View [] not found."

Still having this with Valet+
"spatie/laravel-ray": "^1.1",
Version 1.0.3 (1.0.3)
Laravel Framework 8.16.1
When my application is trying to throw a error message now shows "Ray seems not be running at http://localhost:23517"
When Ray App is running I receive the expected error "InvalidArgumentException View [] not found."

...after uninstall and install again it work (1.0.3).
oh my ray.php config
'host.docker.internal', /* * The port number used to communicate with the Ray app. */ 'port' => 23517, ];

Doesn't work for me ... For some reason it can't communicate to my local computer.
Ray seems not be running at http://host.docker.internal:23517

...after uninstall and install again it work (1.0.3).
oh my ray.php config
'host.docker.internal', /* * The port number used to communicate with the Ray app. */ 'port' => 23517, ];

Doesn't work for me ... For some reason it can't communicate to my local computer.
Ray seems not be running at http://host.docker.internal:23517

Opening the specific port Ray is using in Homestead.yaml configuration file
ports:
     - send: 23517
       to: 23517
       protocol: tcp

Which is opened during the initialisation when restarting Homestead (and making vagrant provision before restarting it)
homestead-research: 23517 (guest) => 23517 (host) (adapter 1)

Retrieve the external ip of the host machine (from this SO : https://stackoverflow.com/a/44873282/2018745)
➜  route -n | grep 'UG[ \t]' | awk '{print $2}'
<IP_ADDRESS>

Copied the ray.php configuration file which is set to
<?php
[
    /*
     * The host used to communicate with the Ray app.
     * For usage in Docker on Mac or Windows, you can replace host with 'host.docker.internal'
     */
    'host' => '<IP_ADDRESS>',

    /*
     * The port number used to communicate with the Ray app.
     */
    'port' => 23517,
]

Checking from outside Homestead if something is listening to the port (Which must be Ray App)
➜  sudo netstat -an | grep LISTEN | grep 23517
tcp46      0      0  *.23517                *.*                    LISTEN
tcp4       0      0  *.23517                *.*                    LISTEN

Another way to watch if it's listening from outside of Homestead. And this command show the name of the app listening !
➜  sudo lsof -nP -iTCP:23517 | grep LISTEN
VBoxHeadl 46934 dib258   19u  IPv4 0x346ea245002b6341      0t0  TCP *:23517 (LISTEN)
Ray       47451 dib258   52u  IPv6 0x346ea244e5e0ab51      0t0  TCP *:23517 (LISTEN)

When I am loggued into the Homestead machine and everything of the above is working, I try to curl the Ray app and here is what I get :
➜  curl -X GET <IP_ADDRESS>:23517
curl: (52) Empty reply from server

Is it normal ?
Shouldn't the app return something ?
Could the VBoxHeadl intercept the value in some way ?

Is there another Curl command that I should try to see if it's getting ray to react ?
Thanks for you help ;)

Opening the specific port Ray is using in Homestead.yaml configuration file
ports:
     - send: 23517
       to: 23517
       protocol: tcp

Which is opened during the initialisation when restarting Homestead (and making vagrant provision before restarting it)
homestead-research: 23517 (guest) => 23517 (host) (adapter 1)

Retrieve the external ip of the host machine (from this SO : https://stackoverflow.com/a/44873282/2018745)
➜  route -n | grep 'UG[ \t]' | awk '{print $2}'
<IP_ADDRESS>

Copied the ray.php configuration file which is set to
<?php
[
    /*
     * The host used to communicate with the Ray app.
     * For usage in Docker on Mac or Windows, you can replace host with 'host.docker.internal'
     */
    'host' => '<IP_ADDRESS>',

    /*
     * The port number used to communicate with the Ray app.
     */
    'port' => 23517,
]

Checking from outside Homestead if something is listening to the port (Which must be Ray App)
➜  sudo netstat -an | grep LISTEN | grep 23517
tcp46      0      0  *.23517                *.*                    LISTEN
tcp4       0      0  *.23517                *.*                    LISTEN

Another way to watch if it's listening from outside of Homestead. And this command show the name of the app listening !
➜  sudo lsof -nP -iTCP:23517 | grep LISTEN
VBoxHeadl 46934 dib258   19u  IPv4 0x346ea245002b6341      0t0  TCP *:23517 (LISTEN)
Ray       47451 dib258   52u  IPv6 0x346ea244e5e0ab51      0t0  TCP *:23517 (LISTEN)

When I am loggued into the Homestead machine and everything of the above is working, I try to curl the Ray app and here is what I get :
➜  curl -X GET <IP_ADDRESS>:23517
curl: (52) Empty reply from server

Is it normal ?
Shouldn't the app return something ?
Could the VBoxHeadl intercept the value in some way ?

Is there another Curl command that I should try to see if it's getting ray to react ?
Thanks for you help ;)

Homestead.yaml updated just like @dib258 and same ray.php file.

Ray 1.0.5
spatie/ray: 1.3.2
php 7.4.5

Simple script. I include the package, I call ray('test') but I get nothing in the Ray app. Originally I was on spatie/ray 1.1 and I would get Ray seems not be running at http://<IP_ADDRESS>:23517 but now I don't even get that error.

Homestead.yaml updated just like @dib258 and same ray.php file.

Ray 1.0.5
spatie/ray: 1.3.2
php 7.4.5

Simple script. I include the package, I call ray('test') but I get nothing in the Ray app. Originally I was on spatie/ray 1.1 and I would get Ray seems not be running at http://<IP_ADDRESS>:23517 but now I don't even get that error.

No changes in the Homestead.yaml or the ray.php config. It's working now.

Ray 1.0.7
spatie/ray: 1.3.7

No changes in the Homestead.yaml or the ray.php config. It's working now.

Ray 1.0.7
spatie/ray: 1.3.7

I just tested removing the opening of the port in the Homestead.yaml (then vagrant provision, vagrant halt and vagrant up) file and it's actually working, so forcing Homestead to open the port is kind of adding some other tools listening to that port apparently that catch or block the request to Ray app.
Thanks for the solution @sevenpixels ! ;)
If someone has an explanation why, it would be amazing for me to learn why the heck it was not working ! :p

I just tested removing the opening of the port in the Homestead.yaml (then vagrant provision, vagrant halt and vagrant up) file and it's actually working, so forcing Homestead to open the port is kind of adding some other tools listening to that port apparently that catch or block the request to Ray app.
Thanks for the solution @sevenpixels ! ;)
If someone has an explanation why, it would be amazing for me to learn why the heck it was not working ! :p

If there is anything that can be improved around this in our docs, let me know!

If there is anything that can be improved around this in our docs, let me know!

Hi Freek, can you please fix the issue that Ray app needs te be running when working local?

Hi Freek, can you please fix the issue that Ray app needs te be running when working local?

@laemol  afaik this is not a problem in the latest version of the package. Can you create a small app to recreate the problem and share that with me?

@laemol  afaik this is not a problem in the latest version of the package. Can you create a small app to recreate the problem and share that with me?

@freekmurze I don't think there's a need to update the documentation since you created a command to create the config file with the default configuration of the host machine's ip.
The only thing that could go wrong if someone touched the configuration of Homestead/Vagrant and could have another host IP address, but maybe if they did that they should know the correct IP and change it in the config file.
I found that command to show the host machine's IP like this :
Retrieve the external ip of the host machine (from this SO : https://stackoverflow.com/a/44873282/2018745)
➜  route -n | grep 'UG[ \t]' | awk '{print $2}'
<IP_ADDRESS>

But that's maybe too unreadable to put in the documentation ^^'
And also regarding my interrogation on opening the 23517 port on Homestead. I think it's irrelevant since we are trying to access the host machine on the 23517 port not specially passing the request through Homestead 23517 port.

@freekmurze I don't think there's a need to update the documentation since you created a command to create the config file with the default configuration of the host machine's ip.
The only thing that could go wrong if someone touched the configuration of Homestead/Vagrant and could have another host IP address, but maybe if they did that they should know the correct IP and change it in the config file.
I found that command to show the host machine's IP like this :
Retrieve the external ip of the host machine (from this SO : https://stackoverflow.com/a/44873282/2018745)
➜  route -n | grep 'UG[ \t]' | awk '{print $2}'
<IP_ADDRESS>

But that's maybe too unreadable to put in the documentation ^^'
And also regarding my interrogation on opening the 23517 port on Homestead. I think it's irrelevant since we are trying to access the host machine on the 23517 port not specially passing the request through Homestead 23517 port.

@freekmurze the latest version of the package has fixed it, thanks! Really like Ray btw nice job!

@freekmurze the latest version of the package has fixed it, thanks! Really like Ray btw nice job!


SNOW-878562 - Release 1.6.0

Please answer these questions before submitting your pull requests. Thanks!

What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
Fixes #NNNN

Fill out the following pre-review checklist:

[ ] I am adding a new automated test(s) to verify correctness of my new code
[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency

Please describe how your code solves the related issue.
Please write a short description of how your code change solves the related issue.

I'm hoping to merge https://github.com/snowflakedb/snowpark-python/pull/964 before the release.

I'm hoping to merge #964 before the release.

#964 is merged and I have rebased this branch.


Restructure data

After some experimentation with the options page for #47 I realise that the data stored in localStorage is messy. Although organised it doesn't look nice and creates restrictions while lifting others. The solution would be to reorganise all the misc entries in to an array of objects which would then be parsed in to JSON, instead of doing this for each entry and then building the objects from these values.
The primary areas that require this rework are templates and URL shorteners (meta/config information). However, this should be done with the greatest care and testing as these are central to the extension and nothing should be overlooked. Consider the reasons that this was not done originally and fix those restrictions another way.
This should be done either before or alongside #48 as that issue will be adding an additional entry for each template.
Ensure that the objects are future-proof and that any new properties can be assigned to them in any future releases.

This could also include a new statistics/stats object stored in localStorage which could contain statistical information based on their usage. For example;

templateCount - Total number of templates
customTemplateCount - Total number of templates (excl. read-only)
topTemplate - Access to the most used template (will require a new property on templates to capture and store usage)
topShortener - Access to the most used URL shortener (again, will require a new property on shorteners)

And probably more...
Properties of this object could then be made available in templates through one or more tags.

This change is now included in the latest code so I just have to try and break it. Things to be tested;

Update process (from <IP_ADDRESS> to 1.0.0) doesn't break the settings for existing installations.
Properties of templates can be changed (via options page).
Predefined templates cannot have certain read-only properties changed (via options page).
New properties can be added to existing templates during initialization.
Changes to predefined templates are picked up during initialization.
New predefined templates are picked up during initialization.
Importing/exporting template(s) still works.

I'm extremely happy with this and glad I finally got around to doing it. Closing this off now.


Problem with CMIP6 global plots for single models

What happened?
Hi Mattia,
We modified the trend and the clima_and_bias notebooks, adding comments and making small variations to the code.
In particular we implemented a new set of conditions to treat the significance of ensemble results, according to IPCC guidance.
I want to ask you two things:

We are encountering a problem plotting global results for single models in the facet plot, for both notebooks: the maps are anomalously cutted both at poles and at longitude boundaries.

We would like to change the label for the ensemble from "model = CMIP6 " to "CMIP6 ensemble".  Same for CORDEX. I think that we can, in general, remove the statement "model =" from all the titles, keeping just the name of the dataset.

I attach the latest version of the notebook I'm working on.
trend_cmip6_t2m_annual_v2.ipynb.zip
Thank you
Minimal Complete Verifiable Example
No response
Relevant log output
No response
Anything else we need to know?
No response
Environment

Hi @paololanteri,

We are encountering a problem plotting global results for single models in the facet plot, for both notebooks: the maps are anomalously cutted both at poles and at longitude boundaries.

Fixed. Looks like we need to use set_global rather than set_extent for global maps.

We would like to change the label for the ensemble from "model = CMIP6 " to "CMIP6 ensemble". Same for CORDEX. I think that we can, in general, remove the statement "model =" from all the titles, keeping just the name of the dataset.

Done.
Could you please check whether the latest templates work OK?
These are the changes I made: https://github.com/bopen/c3s-eqc-toolbox-template/commit/37fbdf92548d46ead1b95cf76992c33013af1ecf

We modified the trend and the clima_and_bias notebooks, adding comments and making small variations to the code.
In particular we implemented a new set of conditions to treat the significance of ensemble results, according to IPCC guidance.

When you are done editing the notebooks, could you please open a PR or share the latest version. Make sure the latest changes are also implemented in the notebooks.

Hi Mattia,
yes, it is fixed,
thank you
Paolo


Add Parse Ethernet frame Operation, allow Parse IPv4 Header to cascade

This adds a new operation+tests that parses Ethernet frames.

This operation adds a basic information output and a 'packet data'-only which allows cascading to other Networking-related parsers (ie IPv4). I am still playing with the idea of updating those to allow the return of encapsulated data as well - allowing a quick/basic packet analysis through a recipe.

An accidental push before switching branches broke UI tests. These are now fixed.
The last few commits also allow the pre-existing "Parse IPv4 Header" to cascade its data down the line instead of only presenting a HTML table. Together with the Ethernet frame operation and the other IP-related parsers this allows analysis and decoding of IP packets (for example as logged by Suricata) and their contents.

@a3957273 Could you take another look at this before it disappears into page 2?


fix(diff): add “doc” related context in GH diff comments

Before this commit all comments published to GH on the diff feature
would be independent from the potential documentation/hub id (or slug)
given to the GH action inputs.
However, some of our users may want to use the GH action on multiple
different specification files (attached to different docs).
Thus with this commit we add the “context” of the documentation in the
comment signature. In this way we can remove only previous comments
for the given documentation (and avoid deleting any of the other Bump
comments)

but I didn't find out if this 'doc' is visible in this comment ?
Would you have an example of such comment please ?

It's not visible on Github, it's an html comment insde a github comment:
<!-- this is not visible if you paste it in a GitHub comment -->

Would you have an example of such comment please ?

Here is an example of two bump comments on the same PR:
https://github.com/bump-sh/examples/pull/5
Seems to work like a charm :blush:


When does BotOnInlineQueryReceived works?

Hi
When does the method BotOnInlineQueryReceived called?
I tried all kind of playing with Telegram but it never got called.
Thanks

the method BotOnInlineQueryRecieved Call when You call your robot on send message . for calling this method you shouldn't go to your bot directly

please check updated examples. you could discuss this question in our group chat

@karb0f0s
this group https://t.me/tgbots_dotnet
is not accessible,

@fatememosleh222
Do you mean this event raised whenever bot mentioned  in chat?
I didn't understand,
could you please explain it in more details?
thanks


update stubs for couchbase 3.1.2

Note that braces were moved to new line after I run cs-fix on the file

Note that braces were moved to new line after I run cs-fix on the file

@avsej Probably you should rebase on master and perform composer cs-fix after that as there is exactly vice-versa cs-fixer rule to place braces on the same line like foo() {}

this is what I did. maybe the issue in docker image or something like that.

@avsej I've just tried to run docker-compose -f docker-compose.yml run php composer cs-fix -d /opt/project/phpstorm-stubs on your fork and file has been formatted as expected

If it is not too hard for you, could you please update my PR with your result? I'm using CentOS 8, and when I run podman-compose -- it raises errors, so I've used just image without compose and it seems like it does not pick up correct settings.

@isfedorov, also I wonder if it is possible to swap directories like this:

couchbase -> couchbase_v2
couchbase_v3 -> couchbase
Because V2 is not maintained anymore, and reached EOL, so people are getting confused if they install current version, but get stubs for old one. The old version V2 still makes sense if users are in process of migrating, but for new users it is really confusing.

@isfedorov, also I wonder if it is possible to swap directories like this:

@avsej I think it's possible. Please do it in a separate PR but please note that generate-stub-map should be updated as well as it uses hardcoded couchbase_v3.

Ok, I will do it once this patch will be merged

@isfedorov I guess this PR is ready, or there is some pending changes or open questions?

@avsej PR seems to be ok to merge and will be merged soon. The only extra item for us is some of our internal tests that also use couchbase and they also should be updated.


Gradle Plugin 7.0.1 - Could not find matching constructor for: java.io.File(File)

Describe the bug
I can't run the scan after upgrading to version 7.0.x
Version of dependency-check used
The problem occurs using version 7.0.0 and 7.0.1 of the gradle plugin
Log file
Error: Could not find matching constructor for: java.io.File(File)
Log files:
https://gist.github.com/rewgoes/e04eb1c2bedccfeefd8fd8e5f53ebdd1
https://gist.github.com/rewgoes/01a39d30062cada20d1a3256287c9066
To Reproduce
Just run ./gradlew dependencyCheckAnalyze --info
Expected behavior
It should not fail

I also confirm this on Gradle 7.2 and 7.4.1 using Dependency Check 7.0.0 and 7.0.1.
As a workaround, I downgraded Dependency Check to 6.5.3

Same for me: Gradle 7.4.1 and dependency-check 7.0.1
* What went wrong:
Execution failed for task ':dependencyCheckAggregate'.
> Could not find matching constructor for: java.io.File(File)

Just updated from dependency-check 6.5.3 to 7.0.1 yesterday. (Needed to skip 7.0.0 due to #247)

This is a dupe of https://github.com/dependency-check/dependency-check-gradle/issues/258 reported against the plugin rather than the core project. The bug is on the plugin rather than the core I believe.


Sorting option for tags

This PR will add an util function which can sort the tags in two ways, and Openebs changed to OpenEBS in some articles

Sorted by length:

Sorted by name:

Signed-off-by: Rakesh PR<EMAIL_ADDRESS>

also @rakeshPRaghu ,
can we change Cas => CAS?

Can you please change OpenSource to Open Source in tags? @rakeshPRaghu


Add modelCard to Service definition

More and more "AI" services are being offered as endpoints (generation, summarization, analysis, etc.); it would a valid consideration to be able to declare which AI models were being used behind such services.  That is, these are not necessarily packaged or offered as top-level "components" per-se.

Again, we should re-visit what makes a component definition and service definition unique as both will have similar information (shared) from a BOM-perspective (i.e., read common base type).

Agreed. You should be able to accomplish this two different ways today.

An external reference on the service that specifies the SBOM that makes up that service
An external reference on the service that specifies the model card, which can appear in the same BOM or in an external BOM.


SurvivorSortOrder: False Son's position doesn't change with a custom sort order

Seems he wants to stay somewhere around 16-17, even though I've set him to 20

This is my custom sort order:
CommandoBody:1, NemCommandoBody:1.2, HuntressBody:2, Bandit2Body:3, ToolbotBody:4,HANDOverclockedBody:5, EngiBody:6, EnforcerBody:7, NemesisEnforcerBody:7.1, MageBody:8, MercBody:9, RobRavagerBody:9.1, NemMercBody:9.2, TreebotBody:10, LoaderBody:11, MinerBody:12, CrocoBody:13, CaptainBody:14, MoffeinPilotBody:15, RailgunnerBody:16, VoidSurvivorBody:17, SeekerBody:18, ChefBody:19, FalseSonbody:20, RobPaladinBody:20.1, RocketSurvivorBody:21, ArsonistBody:22, Executioner2Body:24, ChirrBody:25, SS2UCyborgBody:26, SS2UNucleatorBody:27

nvm the error from the mod is right and i didn't capitalize the b in FalseSonBody


Enable Background Video for new Playnite 9 Version

Since updating to Playnite 9, the PS5ish theme no longer allows for a background video to play on the main game selection screen.
The instructions here do not seem to work for the new version: https://github.com/davidkgriggs/PS5ish/wiki#background-image-background-video-background-music--profile-picture-installation

You are correct the wiki is out of date. I have updated it to show it is just for Playnite 8. Video backgrounds have not been implemented in v4 for Playnite 9. I will add it in and then close this issue once its done.

I am not able to get background videos to run following these steps. https://github.com/davidkgriggs/PS5ish/wiki/Playnite-8:-Extra-Metadata-Tools-Integration


Make repository contribution friendly.

See: https://wikijs.creativecommons.org/tech/github-repo-guidelines

[x] A LICENSE file.
[x] Code owners should be defined in the CODEOWNERS file in the .github directory in the project root.
[x] Add Code of Conduct in a CODE_OF_CONDUCT.md file in the root directory. We can use the standard file from our .github repository.
[x] Add project-specific contributor guidelines in a CONTRIBUTORS.md file in the root directory. The standard file from our .github repository can be used as-is or as a starting point.
[x] Add the standard CC metadata YAML file.
[ ] Update README.md file in the root directory of the repository to document how to install and build the project locally and document a high level overview of the project and code structure. It should also link to any other available documentation.

We don't know yet how to build this project locally so I leave the README.md intact

I'm sorry I Don't know how


Update langchain dependencies

Description
Update langchain dependencies
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[x] Bump-up dependent library

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.41%. Comparing base (d4394c7) to head (3672270).
Report is 13 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #796      +/-   ##
==========================================
- Coverage   94.78%   94.41%   -0.37%
==========================================
  Files          55       55
  Lines        2012     2025      +13
==========================================
+ Hits         1907     1912       +5
- Misses        105      113       +8

/retest

/retest

/lgtm


Portable Terminal usable while crouching

The portable terminal seems to be usable while crouching, resulting in the character standing up with the terminal at it's feet outside the fixed camera's field of vision.
To Reproduce

Crouch / Sneak
Press the 'Portable Terminal' keybind

Expected behavior
Either for the terminal to be properly visible, or to be unable to open it while crouching
Screenshots
(Captured forcing the camera downwards)

Logfiles
Nothing interesting
[Message:AdvancedCompany] Opening terminal
[Message:AdvancedCompany] Player 0 (local) opening terminal
Add a list of all of your mods
Tested with only the following:
BepInEx-BepInExPack-5.4.2100
Rune580-LethalCompany_InputUtils-0.6.3
PotatoePet-AdvancedCompany-1.0.149
Default settings, new profile, new save.
With this setup, leaving the terminal fixes the player and makes it crouch again.
However, with one or more mods present in the following list, the bug extends to bugging the player movement permanently until restarting, causing it to move a step back everytime it stops moving:

Additional bug mod list
 | BepInEx-BepInExPack-5.4.2100
 | RugbugRedfern-Skinwalkers-4.0.1
 | x753-Mimics-2.3.2
 | tinyhoot-ShipLoot-1.0.0
 | AlexCodesGames-AdditionalContentFramework-1.0.3
 | AlexCodesGames-AdditionalSuits-2.0.0
 | Rune580-LethalCompany_InputUtils-0.6.3
 | sfDesat-ExpansionAssets-1.1.0
 | HolographicWings-LethalExpansion-1.3.36
 | sfDesat-Orion-1.3.0
 | Drakorle-MoreItems-1.0.2
 | Pooble-LCBetterSaves-1.7.3
 | FlipMods-LetMeLookDown-1.0.1
 | TwinDimensionalProductions-CoilHeadStare-1.0.6
 | sfDesat-Aquatis-1.1.1
 | Hexnet111-SuitSaver-1.1.4
 | Evaisa-HookGenPatcher-0.0.5
 | Evaisa-LethalLib-0.14.2
 | IAmBatby-LethalLevelLoader-1.1.4
 | scoopy-Scoopys_Variety_Mod-1.1.0
 | Backrooms-Backrooms-0.1.3
 | AinaVT-LethalConfig-1.3.4
 | Gemumoddo-LethalEmotesAPI-1.4.3
 | Gemumoddo-BadAssCompany-1.1.5
 | Ceelery-Kast-1.0.3
 | Zingar-Atlas_Abyss-1.1.6
 | PotatoePet-AdvancedCompany-1.0.149
 | jockie-LethalExpansionCore-1.3.15
 | sfDesat-Celest-0.2.1
 | AudioKnight-StarlancerMoons-0.5.4
 | sfDesat-ViewExtension-1.0.1
 | Tolian-Celestria-0.3.2
 | Major_And_Skiz-MoreInteriors-2.2.0
 | Lordfirespeed-OdinSerializer-2022.11.9
 | xilophor-LethalNetworkAPI-2.1.4
 | Piggy-LC_Office-1.0.3
 | Hardy-LCMaxSoundsFix-1.0.0
 | Magic_Wesley-Wesleys_Moons-1.0.2
 | itsmeowdev-DoorFix-1.0.0
 | x753-More_Suits-1.4.1
 | Norman-GlowStickSuits-1.0.4
 | Monkeytype-HideChat-1.0.0
 | FlipMods-TooManyEmotes-1.8.6
 | Ozone-Runtime_Netcode_Patcher-0.2.5
 | Sligili-More_Emotes-1.3.3
 | HomelessGinger-MaskedEnemyOverhaul-3.1.2
 | LethalMoonTeam-Ether-1.6.5
 | LethalMatt-Outpost_31-1.1.2
 | KayNetsua-E_Gypt_Moon-2.0.14
 | Suskitech-AlwaysHearActiveWalkies-1.4.4
 | quackandcheese-MirrorDecor-1.3.2
 | Owen3H-CSync-1.0.7
 | NotAtomicBomb-TerminalApi-1.5.1
 | fivetoofive-DetailedScan-1.2.2
 | IntegrityChaos-Diversity-1.1.10
 | Evaisa-LethalThings-0.9.4
 | Badham_Mods-SCPFoundationDungeon-2.3.1
 | Zingar-SecretLabs-3.3.5
 | FEB-SCP_Suits-1.1.0
 | Verity-TooManySuits-1.0.5
 | mrov-LethalRichPresence-0.5.8
 | 5Bit-VoiceHUD-1.0.4
 | EladNLG-EladsHUD-1.2.1
 | EliteMasterEric-SlimeTamingFix-1.0.2
 | BlueAmulet-LCBetterClock-1.0.3
 | NutNutty-SellTracker-1.2.1
 | blink9803-DimmingFlashlights-1.0.1
 | ViViKo-NoSellLimit-1.0.1
 | NiceHairs-NuclearLib-1.0.4
 | NiceHairs-Symbiosis-1.0.5
 | Spyci-CozyImprovements-1.2.2
 | akechii-DiscountAlert-2.3.0
 | Bobbie-FixRPCLag-1.0.1
 | taffyko-NameplateTweaks-1.0.6
 | loaforc-FacilityMeltdown-2.3.0
 | SillySquad-TakeThatMaskOff-2.0.0
 | bigmcnugget-VanillaContentExpansion-0.1.8
 | BGN-PizzaTowerEscapeMusic-2.3.0
 | LethalResonance-LETHAL_RESONANCE_MELTDOWN-1.1.5
 | Clementinise-CustomSounds-2.3.2
 | no00ob-LCSoundTool-1.5.1
 | LethalResonance-LETHALRESONANCE-4.0.6
 | Zaggy1024-PathfindingLagFix-1.2.1
 | veri-ShipWindows-1.0.6
 | x753-Peepers-0.9.3
 | Boniato-Ganimedes-1.9.0
 | TeamClark-TheMostLethalCUSTOMISATION-1.3.0
 | Gloveman23-MoonPriceDisplay-1.0.1

Mod Manager Code:
018d8d6d-d353-804e-390a-563117a5c7f6
Thanks in advance

Will be fixed in 1.1.1


pg:backups schedule doesn't work, auto backup feature is dead

check this out
» heroku pg:backups schedule
 !    You must specifiy a time to schedule backups, i.e --at '04:00 UTC'
» heroku pg:backups schedule --at '04:10 UTC'                  1 ↵
Invalid schedule format: expected '<hour>:00 <timezone>'
» heroku pg:backups schedule --at "04:10 UTC"                  1 ↵
Invalid schedule format: expected '<hour>:00 <timezone>'
» heroku pg:backups schedule --at 04:10 UTC                    1 ↵
Invalid schedule format: expected '<hour>:00 <timezone>'

I'm following what the guide is saying, but it doesn't work.
I also DON'T have a web interface to set the backup schedule, so from my perspective, the auto backup feature is completely broken if I can't set a schedule.

okay just realized that you can only schedule backup on the :00 hour, not arbitrary time


Active days up

If possible would be nice to see information about "active" days up since churned in

Thanks for this idea!
Please help me out:
Is there any way to see the timestamp when a node churned in?

I believe it is shown as "Status Since".
I think it is since "block", so you would need to see the "current block" minus "since block", multiple by block time and divide by /minutes/hours/days...
.....

Status
active

Status Since
32403

.....


bash (and possibly other script resources) documentation problem

I'm reporting what seems to be a documentation problem with the bash resource (and probably others) found by @lamont-granquist while reviewing an issue I have (#2342)
I was using bash resource as what I thought a good workaround for the problem I had. The bash resource was like that:
bash "user1 reads file with owner user2 and group user2 with mode 0440" do
  command "cat /tmp/testfile"
  user "user1"
  group "user1"
end

Running this was succeeding, while it was failing with a permission denied problem when using the execute one.
But it was only succeeding because the command attribute is not used for bash resource and it was simply creating an empty script file which was run correctly because the code attribute was empty. This was an error on my part because using code "cat /tmp/testfile" was exhibiting the same permission denied problem.
The documentation should be more clear about this behavior because. @lamont-granquist even suggested to add some validation for this. Here a copy of his write-up about this:

What we probably need to do is have the execute resource run a @provider.command which load_current_resource in the provider sets to @new_resource.command. Then we stop mutating @new_resource.command in the script provider. We need to add validation on the resource that @new_resource.code is not nil and I think we can even override the validation on @new_resource.command in the script resource to throw an error if its not nil. Then we also need to be careful about other thing which may inherit from those like powershell.

Regards,
Matt

@jamescott we need to update the command parameter on all the script resources -- everything bash or perl or whatever -- but not execute -- to more clearly show that the command parameter must not be set on those resources, its for internal use only.

https://github.com/chef/chef-docs/commit/40b96bd1286a8eb5d1b85181f322d65bead8dcd7
What about powershell_script and batch?

Those are the same.
The dsc_script resource is a unique script resource which does not inherit from script/execute and the command property there does work to inject a user-specified command to run -- so dsc_script is its own special snowflake (sigh) and should not have its docs changed.


Timelines will save over each other

Describe the bug
If you click between different timelines fast then one timeline will show up as that timeline for example if I have timeline A and B; and I switch between them fast then timeline A will become timeline B, if you reload the project it's not an issue but if you save then it'll save timeline A as B
To Reproduce
Steps to reproduce the behavior:
Just spam click between them this is the fastest way I am able to recreate it (not that I spam between timelines)
Expected behavior
Timeline A to show Timeline A and Timeline B to show B
System (please complete the following information):

OS: Linux
Godot Version: 4,3
Dialogic Version: 2.0 Alpha 14

Solutions
just reload don't save

Also exists in Alpha 15 on Windows 10.
Relevant error messages when this occurs
 res://addons/dialogic/Editor/TimelineEditor/VisualEditor/timeline_editor_visual.gd:162 - Invalid access to property or key 'ready' on a base object of type 'null instance'.
  res://addons/dialogic/Editor/TimelineEditor/VisualEditor/timeline_editor_visual.gd:139 - Invalid type in function 'create_end_branch_event' in base 'MarginContainer (timeline_editor_visual.gd)'. The Object-derived class of argument 2 (previously freed) is not a subclass of the expected argument class.

This luckily doesn't appear to actually be a critical bug, as I tested saving everything and this does not actually overwrite anything in this case, it simply won't change out the loaded timeline from memory in the visual editor.

Also exists in Alpha 15 on Windows 10.
Relevant error messages when this occurs
 res://addons/dialogic/Editor/TimelineEditor/VisualEditor/timeline_editor_visual.gd:162 - Invalid access to property or key 'ready' on a base object of type 'null instance'.
  res://addons/dialogic/Editor/TimelineEditor/VisualEditor/timeline_editor_visual.gd:139 - Invalid type in function 'create_end_branch_event' in base 'MarginContainer (timeline_editor_visual.gd)'. The Object-derived class of argument 2 (previously freed) is not a subclass of the expected argument class.

This luckily doesn't appear to actually be a critical bug, as I tested saving everything and this does not actually overwrite anything in this case, it simply won't change out the loaded timeline from memory in the visual editor.

it might only be a critical bug for linux or the one I have Ubuntu which is 22.04 since it happened twice to me on two separate occasions

I guess what I mean by it not being a critical bug is that it likely isn't going to ever overwrite your timelines; it might appear that way in engine, but from my testing it appears that the issue is just the user interface having issues reflecting that you've selected a new file

oh that's interesting my two timelines were overwritten no matter what I tried, maybe I missed something cause I would reload the project, quit and not save, even turned my pc on and off lol. I'm still new to this so I probably missed something

It is still possible that the behavior is different on linux and more of an issue. The big thing I have ran into is that sometimes the parser enters into a bad state, and if you try to update the timeline via the visual editor, you get errors for trying to access a null instance.
Once that happens, on Windows machines at least, clicking on different timelines looks like all the timelines are being overwritten, but if you save and close nothing changes to the timeline files. It also doesn't seem like the whole plugin is crashing, just the visual editor

Hey, can you try this fix I made and see if it helps ya'll?
https://github.com/Invertex/dialogic/tree/fix-timeline-window-content-mismatch-bug
This bug is extremely hard for me to reproduce on my system, so I can't easily verify if it fixes the problem for sure. But I couldn't seem to get it to happen after these small changes, so hopefully it does fix it. Let me know and then I can PR it.

Can't recreate this at all unfortunately. We are gonna assume that Invertex's fix addresses it for now, but feel free to reopen this if you hit it again after patching.


pdfHtml5 exportable don't work

Defining column as exportable=false and printable=false don't work on pdfHtml5 or excelHtml5 .
I'm setting a column as printable=false and exportable=false . That works on Pdf, Print and Excel buttoms, but doesn't   work on excelHtml5 or pdfHtml5. Column is still beeing exported

[
'data' => 'editar',
'title' => '',
'orderable'      => false,
'searchable'     => false,
'exportable'     => false,
'printable'      => false,
],
System details

Operating System
PHP Version Version 7.2.21-1
Laravel Version  5.8.35
Laravel-Datatables Version  laravel-datatables-oracle v9.7.2
laravel-datatables-editor v1.17.1
laravel-datatables-buttons v4.8.0

Exportable and printable is for serverSide implementation. pdfHtml5 is for client side and I'm not sure how it works. Better check the official js docs.

Thanks for the quick answer, Yajra
I'll take a view on js docs


Full viewport size

Is it possible to set the width and height props to match the viewport width and height and have within enabled?
My use case is a responsive fullscreen modal, with a image.

@anandpatel Hi, I will try to implement this in several days

Has this gap been addressed?