Practices for Hiding Suggestion

Currently, when a user starts typing, the recyclerview is hidden by setting its visibility to gone (Source)

There is no animation this way, should we clear the adapter instead?
Once a new query is submitted, there is sometimes a blink of the old data before the new data is added and shown
I haven't tested this anywhere but my app, but the first adapter change doesn't show for me; there is something with the data changes, though it looks like it should work.

I am going to take a look on this for next version.


Auto adjust height of the row

Description
is there any way  to adjust the row height automaticly according to content of the cell?
Reproduction URL
No response

Thank you for your comment.
There is no option that automatically adjusts the height, but this may be achieved by using options.onSave.
Since onSave accepts (table, position), you can get {[address]: Cell} by writing the following.
const diff = table.getObjectFlatten({
  filter: (cell) =>
    ! !cell?.changedAt &&
    cell.changedAt > table.lastChangedAt!
});

For example, you could get the number of rows from Cell.value and multiply by line-height to determine the height.
Or you could get the HTML height by identifying the HTML element td[data-address="${address}"] .gs-cell-rendered from position.
This might be applied like table.update({diff: {[y]: {height}}, partial: true}) to change the height.
Please give it a try.


Documentation page tracking

[x] When 'should be tracked' is checked, the answers are only shown on the results page if the download button at the end of the documentation page is clicked. This button still says download on it even though its function now seems to be submit instead. I suggest there should be two buttons - 'download' will always download a PDF and 'submit' will only appear when tracking is on and the answers will be submitted to the results page. Perhaps there should be also an option to remove the download button when tracking is on as a submit & download button might not always be required together

[x] 'Should be tracked' option should be reworded - I suggest 'Submit answers' might be more appropriate if a submit button is added. This could also do with a tooltip to explain that this will track answers and surface them on the results page / adaptive content page

Additional comments on this functionality:

[ ] The conditions related to the 'show download button' are reversed - file orientation, font size and file name are hidden when the box is checked and are shown when the box is checked. Also, need to check how this worked before recent changes. If the download button was always shown by default then these options should be shown even if the show download button optional property has not been added
[ ] There should be some on screen confirmation after the submit answers buttons has been clicked. At the moment the button just disappears. There should be something that at least says 'your answers have been submitted' This should be a language string that is editable if you click the 'show language options' box in the editor
[ ] The submit button disappears after it's clicked - should it? If you edit your answers after submitting, will these be shown on the results page or just the version that was there when you clicked submit?
[ ] Even when submit answers is on and I click submit, no info appears on the results page. Not sure what is going wrong but I think this worked the first time I tested this?

@torinfo the issues I previously found here are all working well now.
There is something odd on the data shown on the results page though - if an image or description block has been added to the page then these show listed in the interactivity results even though there is nothing to associated with them that can be answered or submitted.


Can you draw a heart shape wall

All the examples are square walls. Can you draw a heart shape

💗 this shap

Sure, take a look at the concave and SVG examples which should help you start.

how to put balls into the Wall which is   a heart shape  ;  can do it

also circle shape

So is the circle shape，

thank you very much


typo: CONTRIBUTING.md

Double word "of"
sytactically -> syntactically
properities -> properties

Automation for azure-sdk-for-python

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-java

Automation for azure-sdk-for-js

Automation for azure-sdk-for-go

Automation for azure-sdk-for-node

Can one of the admins verify this patch?


Fix to JSMA attack where all_feat was not updating

Description
The value of gamma in saliency_map.py did not affect the generation of adversarial examples. Regardless of the value, the attack would run until either the attack was successful or the maximum number of features were modified. This was due to all_feat not updating
Fixes #196
Type of change
Modify lines 123-124 in saliency_map.py
all_feat[active_indices, feat_ind[:, 0]] = 1
all_feat[active_indices, feat_ind[:, 1]] = 1

Please check all relevant options.

[X] Bug fix (non-breaking)
[ ] New feature (non-breaking)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Testing
The bug was verified with the following code. The fix shows that the bug no longer occurs and the attack behaves as expected
import warnings
warnings.filterwarnings('ignore')
from keras.models import load_model
from art import DATA_PATH
from art.utils import load_dataset, get_file
from art.classifiers import KerasClassifier
import numpy as np

(x_train, y_train), (x_test, y_test), min_, max_ = load_dataset('mnist')
path = get_file('mnist_cnn_original.h5', extract=False, path=DATA_PATH,
                url='https://www.dropbox.com/s/p2nyzne9chcerid/mnist_cnn_original.h5?dl=1')
classifier_model = load_model(path)
classifier = KerasClassifier(clip_values=(min_, max_), model=classifier_model, use_logits=False)
from art.attacks import SaliencyMapMethod

#First we try with gamma=1, this means that the attack should run until it fools the classifier or uses all of the features
print("--------- Gamma = 1 ---------------")
print("Original Predicted Label:", np.argmax(classifier.predict(np.array([x_test[0]])),axis=1))
attack = SaliencyMapMethod(classifier=classifier, gamma=1)
x_adv = attack.generate(np.array([x_test[0]]), y=np.array([9]))
print("Adv Predicted Label:", np.argmax(classifier.predict(x_adv[:2]),axis=1))
print("Number of features changed", np.sum(np.array(x_adv[0],dtype=np.float32) != np.array(x_test[0],dtype=np.float32)))

#Now we set gamma to a very small value (close to 0). It should stop after the first iteration
print("--------- Gamma = 0.0000000000001 ---------------")
print("Original Predicted Label:", np.argmax(classifier.predict(np.array([x_test[0]])),axis=1))
attack = SaliencyMapMethod(classifier=classifier, gamma=.0000000000001)
x_adv = attack.generate(np.array([x_test[0]]), y=np.array([9]))
print("Adv Predicted Label:", np.argmax(classifier.predict(x_adv[:2]),axis=1))
print("Number of features changed", np.sum(np.array(x_adv[0],dtype=np.float32) != np.array(x_test[0],dtype=np.float32)))

Checklist

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Looks good - thank you @keykholt for spotting this issue and providing the fix!

@keykholt Thank you for your first contribution to ART!


Extend cluster_autoscaling fields in gke-cluster-standard

feat(gke-cluster-standard): Add feature to setup cluster_autoscaling

Set the autoscaling_profile if OPTIMIZE_UTILIZATION is needed
Add the possibility to set disk_size and disk_type
Add configuration of management and shielded_instance_config

Each of these options is optional and set to their default value if not set.
feat(gke-cluster-standard): Add GPUs setup feature for cluster_autoscaler
Add the possibility to setup gpu_resources into node auto-provisioning.

Checklist

I applicable, I acknowledge that I have:

[x] Read the contributing guide
[x] Ran terraform fmt on all modified files
[x] Regenerated the relevant README.md files using tools/tfdoc.py
[x] Made sure all relevant tests pass

@anthonyhaussman ;can you fix the failing tests?

@anthonyhaussman ;can you fix the failing tests?

@juliocc It should be good now :pray:

@juliocc I had some trouble finding the problem and reproducing it.
But I have found the problem and pushed the fix in https://github.com/GoogleCloudPlatform/cloud-foundation-fabric/commit/e74e0caced62edcb6c351e3f79eb4cb9f300d7bb.
Sorry for the trouble. :pray:


Fixes in ECAPA Voxceleb recipe

Hello, this PR includes a few fixes in ECAPA Voxceleb recipe about s-norm implementation, enrol/test namings and passing device parameter.

Thank you for this PR! @Jenthe, could you please double check the s-normalization part?

Thank you for the fixes. I tested it and everything looks fine.


provision: Clean up interface for template_database_status

Following 84edb5c5162e9299ab5fb6a0a59c402484caa98c, it's clear that we should have a cleaner database for how the arguments to template_database_status are passed to correctly check whether the target database (whether zulip_test_template, used by test-backend, or the ).  I think we could do significantly better by defining a class or TypedDict defining the settings for the zulip_test_template and zulip databases and accessing those. (Or maybe just having there be a single argument for which database we're checking?).
Fundamentally, there's only two valid ways to call template_database_status, so it shouldn't take like 5 arguments with defaults, it should take one argument that specifies which database we want to access.

@zulipbot claim


lathe machine upgrading

About the PR
adds two upgrades:

speed via manipulators
material use via matter bins

Screenshots

Changelog

:cl:

add: Lathes can now be upgraded to print faster and to consume fewer materials per print.

test got cunked :pensive:


Field name aliases

In some cases, when we have huge collection of relatively small objects, there will be a huge storage overhead, caused by human-friendly naming.
It will be great, to have aliases or `, for example:
class Measure
    field :measured_at, type: DateTime, name: "d"
    field :instrument_id, type: Integer, name: "i"
    field :value, type: Integer, name: "v"
    field :quality, type: Integer, name: "q"
end

After explicitly setting name in field, mongoid will automatically change original field names in criteria, to real one.

You can already add the as: "new_name" option to field right now... You you just want the functionality extended into criteria as well?

@y8 Yeah, the as option simply creates aliases for the field's accessors.


Inconsistent query type

The hypermedia generator produces records for queries but the framework provides QueryBase as abstract class.
Records cannot inherit abstract classes. Either the generator needs to be changed to emit classes or the framework should provide QueryBase as an abstract record.

@MathiasReichardt Can we consider using the record type for QueryBase? The only problem with that approach is that the current Member Clone is a reserved name for records.


FIX RedisRegistry's problems

modify the FileConfiguration in the RedisRegistry.Let it use registry.conf
fix uese redis registry, if restart or close the redis coudn't unregister the server.But still can't prevent kill -9.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.lizhao01 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #669 into develop will decrease coverage by 0.01%.
The diff coverage is 0%.

@@              Coverage Diff              @@
##             develop     #669      +/-   ##
=============================================
- Coverage      32.22%   32.21%   -0.02%
  Complexity       877      877
=============================================
  Files            227      227
  Lines           8787     8791       +4
  Branches        1058     1058
=============================================
  Hits            2832     2832
- Misses          5633     5637       +4
  Partials         322      322

Impacted Files
Coverage Δ
Complexity Δ

...r/discovery/registry/RedisRegistryServiceImpl.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

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 edb4e71...0ca4d77. Read the comment docs.

Thanks for the work.please sign the CLA first.so we can accept your PR.


🐛 internal error: entered unreachable code: The node can't have the same range of the operator.

Environment information
CLI:
  Version:                      1.4.1
  Color support:                true

Platform:
  CPU Architecture:             aarch64
  OS:                           macos

Environment:
  BIOME_LOG_DIR:                unset
  NO_COLOR:                     unset
  TERM:                         "xterm-256color"
  JS_RUNTIME_VERSION:           "v21.3.0"
  JS_RUNTIME_NAME:              "node"
  NODE_PACKAGE_MANAGER:         "bun/1.0.15"

Biome Configuration:
  Status:                       Loaded successfully
  Formatter disabled:           false
  Linter disabled:              false
  Organize imports disabled:    true
  VCS disabled:                 true

Workspace:
  Open Documents:               0

What happened?

Biome encountered an unexpected error

This is a bug in Biome, not an error in your code, and we would appreciate it if you could report it to https://github.com/biomejs/biome/issues/ along with the following information to help us fixing the issue:

Source Location: crates/biome_js_analyze/src/utils.rs:110:13
Thread Name: biome::worker_2
Message: internal error: entered unreachable code: The node can't have the same range of the operator.

Expected result
It runs and doesn't hang
Code of Conduct

[X] I agree to follow Biome's Code of Conduct

@niklasarnitz Thank you for the report. Unfortunately, we don't have enough information to help, we would need to have a snippet of code that triggers this error.
cc @victor-teles you might want to look at this. Even though the rule you implemented in this configuration, we currently have a small bug that enables all the nursery rules. That's why we see the error. Let's wait a response from OP

I'm getting the same failure after upgrading from 1.4.x to 1.5.0.
The reproduce clone this repo on this branch: https://github.com/EvHaus/haus.gg/pull/996
Then run:
bun i
bun biome:check
I end up with all sorts of failures like this:
This is a bug in Biome, not an error in your code, and we would appreciate it if you could report it to https://github.com/biomejs/biome/issues/ along with the following information to help us fixing the issue:

Source Location: crates/biome_js_analyze/src/utils.rs:110:13
Thread Name: biome::worker_12
Message: internal error: entered unreachable code: The node can't have the same range of the operator.

Biome encountered an unexpected error

This is a bug in Biome, not an error in your code, and we would appreciate it if you could report it to https://github.com/biomejs/biome/issues/ along with the following information to help us fixing the issue:

Source Location: crates/biome_js_analyze/src/semantic_analyzers/nursery/no_misleading_character_class.rs:310:68
Thread Name: biome::worker_19
Message: byte index 7 is not a char boundary; it is inside '\u{80}' (bytes 6..8) of `^[  -~-ÿ]+$`

Biome encountered an unexpected error

This is a bug in Biome, not an error in your code, and we would appreciate it if you could report it to https://github.com/biomejs/biome/issues/ along with the following information to help us fixing the issue:

Source Location: crates/biome_js_analyze/src/semantic_analyzers/nursery/no_misleading_character_class.rs:310:68
Thread Name: biome::worker_5
Message: byte index 28 is not a char boundary; it is inside '–' (bytes 27..30) of `\s|(?![\'])[\!-\@\[-\`\{-\~–-‼]`

Interestingly, these failures don't happen when running the same code on GitHub Actions. Only happens when running on my machine.

For me the issue was that something in the glob patterns changed in Biome 1.5.0. I had to make this config change in order to ensure the .next folder is ignored in my Next.js project:
## Before
"files": {
	"ignore": [".next"]
}

## After
"files": {
	"ignore": ["./.next"]
}


[SPARK-47144][CONNECT][SQL][PYTHON] Fix Spark Connect collation error by adding collateId protobuf field

What changes were proposed in this pull request?

Collated expression SELECT 'abc' COLLATE 'UCS_BASIC_LCASE' is failing when connecting to sever using Spark Connect (in this case with pyspark):
pyspark.errors.exceptions.connect.SparkConnectGrpcException: (org.apache.spark.sql.connect.common.InvalidPlanInput) Does not support convert string(UCS_BASIC_LCASE) to connect proto types.

When using default collation UCS_BASIC, the error is not occurring.
Fix the error by making StringType protobuf compliant with Spark datatype which now includes collation id.
Why are the changes needed?

To fix the error which collations introduce in Spark connect.
Does this PR introduce any user-facing change?

Yes, when previously running
$ ./bin/pyspark --remote 'sc://localhost'
...
>>> spark.sql("SELECT 'abc' COLLATE 'UCS_BASIC_LCASE'")
...

the described error would appear. These changes remove the error.
How was this patch tested?

Changes include test which simulates described behavior by utilizing SparkConnectServerTest to establish client-server connection. Test is invoked by build/sbt 'connect/testOnly *SparkConnectServiceE2ESuite'.
Was this patch authored or co-authored using generative AI tooling?

No.

You will need to run ./dev/connect-gen-protos.sh to re-gen the proto files for Python (this is required whenever you updates the source proto).

thanks, merging to master!


Fix open or close drawer when gesture should be cancelled

According to https://facebook.github.io/react-native/docs/panresponder.html, onPanResponderTerminate should be used to cancel an active pan.
Currently, when we use something like a scrollview as side menu, when we start swiping we can break it by touching the scrollview inside the menu which can be focused because there is no state changing and then pointerEvents is still set to "auto".

👍


Fatal error: Invalid handle returned - for all connections if one fails

After one connection using pdo_sqlsrv v4.0.8629.2 fails, for example because of incorrect user/pass, All the other connections also are failing with PHP Fatal error: Invalid handle returned.
Environment: Win10, Apache/2.4.23 (Win64) OpenSSL/1.0.2h PHP/7.0.9 using php_pdo_sqlsrv_7_ts_x64.dll.
Reproducing:
2 php scripts that are creating DB connections using php, like:
file1.php:
<?php try{     $dbconnection = new PDO('sqlsrv:Server=localhost;Database=cdr41', 'existinguser', 'existingpassword');     echo 'OK'; }catch(PDOException $e){     echo $e->getMessage(); } ?>
file2.php:
<?php try{     $dbconnection = new PDO('sqlsrv:Server=localhost;Database=cdr41', 'fakeuser', 'fakepassword');     echo 'OK'; }catch(PDOException $e){     echo $e->getMessage(); } ?>
I can run file1.php multiple times without any problems, but after running even once file2.php, first time file2.php generates and exception, like it should, but after that file1.php and file2.php are failing with Fatal error: Invalid handle returned.
When switching back to 32bit apache + php + pdo SQL driver 3.2 all works as expected.

@eziitiss We are looking into this. We will keep you posted.

:-1: m2

@Yahasana we are actively investigating this. Do you have a repro script that you can share with us?

@meet-bhagdev I have already posted my two test scripts. Nothing more is needed, just a local MSSQL server (in my case: v13.0.1708) with existing DB name "CDR41" and existing user "existinguser" with password "existingpassword" and not existing user "fakeuser". Sure - you can rename the DB and users how you like - just rename them also in PDO connection string :) .
What I did not mention in my original post - to recover from this failure I need to restart my Apache server. If you need some more info or some debug - let me know.

Now I remember that nearly have the same issue but not about password.
Probably after getting php timeout while pdo query, the other instances starts to give the same error till I restart the IIS 8.5. A little bit busy here but try to reproduce the error too.

I have the same problem here.
4.0.2 - Linux
4.1.0 -windows

I get this error like every 10 connections. Turning connection pooling off seems to make the error go away.

@ethanhann-tr @super-vip @eziitiss @mlhkr We are looking into this and will aim to fix this asap. Stay tuned :)

This same thing is happening to me.
I'm using:

Ubuntu 16.04
PHP 7.0.9
Nginx 1.11.1
Laravel 5.2.43
msphpsql 4.0.2 - Linux

Important Note: It's happening exactly every 3 requests.

@lancepioch @all We are able to repro this bug in our lab. We are fixing it as we talk and will aim to fix it in our next release. Our next release should be out very soon.

Just for the record, someone also reported this on Drupal.org:
https://www.drupal.org/node/2772037

@all, we are actively fixing this bug. Can you provide a repro script similar to the one @eziitiss shared? We would like to test against it to validate our fix.

@meet-bhagdev I am seeing this issue while using Laravel, so I don't have a simple script I can extract and share. Were you able to validate your fix? Can we expect a release with it soon?

@ethan-tr , in your config/database.php add `'pooling' => false,' to your database parameters, as follow:
'connections' => [
        'sqlsrv' => [
            'driver'   => 'sqlsrv',
            'host'     => env( 'DB_HOST' ),
            'port'     => env( 'DB_PORT', 1433 ),
            'database' => env( 'DB_DATABASE' ),
            'username' => env( 'DB_USERNAME' ),
            'password' => env( 'DB_PASSWORD' ),
            'prefix'   => '',
            'charset'  => 'UTF-8',
            'appname'  => 'App name',
            'pooling'  => false,
        ],
],

@meet-bhagdev This issue is really annoying. If you already have a fix, please consider doing a release even if we have to wait longer for other fixes.

@david-garcia-garcia @rodrigopedra @lancepioch @eziitiss @Yahasana @super-vip @mlhkr : We just released our next preview: https://github.com/Azure/msphpsql/releases/tag/v4.0.3-Linux. We have attempted to fix this bug. Please help us validate. If there are more issues, update this thread and we will take a look at it.

@meet-bhagdev: Any schedule on Windows version ?

It's already released:
https://github.com/Azure/msphpsql/releases/tag/v4.1.1-Windows

Can confirm issue solved in my case using 4.1.1.
Thank you!

The 4.0.3 driver seems to have fixed the issue in my tests.

@meet-bhagdev changed our app today and it seems ok with the new version. thanks!

@rodrigopedra @ethan-tr @eziitiss Glad to hear the issue is fixed 👍 Going to close this issue. @All feel free to re-open the issue if you need to.

I spent an hour. But when I removed the port number, it worked out.
'connections' => [
        'sqlsrv' => [
            'driver'   => 'sqlsrv',
            'host'     => env( 'DB_HOST' ),
            'port'     => '',
            'database' => env( 'DB_DATABASE' ),
            'username' => env( 'DB_USERNAME' ),
            'password' => env( 'DB_PASSWORD' ),
            'prefix'   => '',
            'charset'  => 'UTF-8',
            'pooling'  => false,
        ],
],


add new Myanmar (my) language resource for numbers.

Note: Myanmar scripts are properly rendered in most fonts. Use proper font, such as Halvetica in os x, to display them correctly.

Burmese Orthography for those interested https://www.scribd.com/document/141354924/Burmese-Orthography

Thanks for sending. I did some quick tests and it seems that "32" doesn't work. Can you look at it?
(play :my$core "သုံးဆယ်နှစ်ပါး" nil {:reference-time (time/t -2 2016 11 16 21 10 0)})

After updating to HEAD, unit test passed here.
Kyaws-Mac-mini:duckling mbikyaw$ lein test

lein test duckling.core-test
Nov 17, 2016 5:17:14 PM clojure.tools.logging$eval470$fn__474 invoke
INFO: Extracting from 'january 2014' with targets [{:module "en$core", :dim "time", :label "T"}]
Nov 17, 2016 5:17:14 PM clojure.tools.logging$eval470$fn__474 invoke
INFO: Extracting from '2 brasses' with targets [{:module "en$core", :dim "quantity", :label "T"}]
Nov 17, 2016 5:17:14 PM clojure.tools.logging$eval470$fn__474 invoke
INFO: Extracting from 'Oct. 12 from 2 to 5 p.m. Monday-Friday, Sept. 7-March 1, 8 a.m.-noon, 1-5 p.m.' with targets [{:module "en$core", :dim "time", :label "T"}]
Nov 17, 2016 5:17:14 PM clojure.tools.logging$eval470$fn__474 invoke
WARNING: @pass-all reached maximum stash size for sentence 'Oct. 12 from 2 to 5 p.m. Monday-Friday, Sept. 7-March 1, 8 a.m.-noon, 1-5 p.m.'

lein test duckling.ml.naivebayes-test

lein test duckling.time.obj-test

lein test duckling.time.pred-test

lein test duckling.util-test

Ran 8 tests containing 30 assertions.
0 failures, 0 errors.
Kyaws-Mac-mini:duckling mbikyaw$


CGB won't install project properly

Hi,
while trying to create a project with cgb using npx create-guten-block my-block on a current node/npm install on win10 pro 64, the process is constantly aborting after step 2 with the exact same error documented below (and described in this thread: https://github.com/ahmadawais/create-guten-block/issues/34):

The node module fs-extra is trying to execute a file operation on a directory
which leads to en EISDIR error that ends execution of cgb half way through the process without copying the plugin files.

I'm bringing this up despite the previous thread because this is happening in a new install after a long time, and because there was no hint at a solution in the previous thread. Maybe some new information has emerged since then. However, the only relevant information I could find was the issue-thread indicated above.
If so, I'd be grateful for some info and help.

node -v: 12.13.1

npm -v: 6.12.1

npm ls cgb-scripts (if you haven’t ejected):--<EMAIL_ADDRESS>

Operating system: WIN 10 PRO 64

This is the error message:
- 3. Creating plugin files...ERROR on templatePromise → Error: EISDIR: illegal operation on a directory, readlink 'O:\xampp\htdocs\my_wp\wp-content\plugins\my-block'     at Object.readlinkSync (fs.js:934:3)     at checkDest (O:\xampp\htdocs\my_wp\wp-content\plugins\my-block\node_modules\fs-extra\lib\copy-sync\copy-sync.js:183:23)     at onDir (O:\xampp\htdocs\my_wp\wp-content\plugins\my-block\node_modules\fs-extra\lib\copy-sync\copy-sync.js:110:24)     at getStats (O:\xampp\htdocs\my_wp\wp-content\plugins\my-block\node_modules\fs-extra\lib\copy-sync\copy-sync.js:48:32)     at startCopy (O:\xampp\htdocs\my_wp\wp-content\plugins\my-block\node_modules\fs-extra\lib\copy-sync\copy-sync.js:41:10)     at Object.copySync (O:\xampp\htdocs\my_wp\wp-content\plugins\my-block\node_modules\fs-extra\lib\copy-sync\copy-sync.js:36:10)     at O:\xampp\htdocs\my_wp\wp-content\plugins\my-block\node_modules\cgb-scripts\scripts\init.js:41:7     at new Promise (<anonymous>)     at copyTemplateFiles (O:\xampp\htdocs\my_wp\wp-content\plugins\my-block\node_modules\cgb-scripts\scripts\init.js:36:26)     at module.exports (O:\xampp\htdocs\my_wp\wp-content\plugins\my-block\node_modules\cgb-scripts\scripts\init.js:168:8) {   errno: -4068,   syscall: 'readlink',   code: 'EISDIR',   path: 'O:\\xampp\\htdocs\\my_wp\\wp-content\\plugins\\my-block' }

I eventually managed to solve this: it's an error that occured when I tried to use create-guten-block from within a directory that is not the home drive. In other words creating the block scaffolding on c:/ works, creating it on o:/ throws errors, for some unknown reason.


Tapo P115's not working after HA update

Hi experts,
Just started with HA couple months ago on RPi4. Very simple setup. 3 identical wifi powerswitches Tapo P115. After updating HA today I see the entities are mixed ! (every device had "1 device and 8 entities" before) (only the "Smart-plugRESERVE" is still correct) Ofcourse nothing works.

After a full reboot the second device came back normal, but the first is still mixed

Tape "SmartPlugCar" has unchanged IP <IP_ADDRESS> (mac 48-22-54-4A-6C-69)
I see in the logs :
2023-11-29 11:25:20.802 INFO (MainThread) [custom_components.tapo.setup_helpers] Trying to track ip address of 48-22-54-4A-6C-69, last known ip is <IP_ADDRESS>
2023-11-29 11:25:20.805 DEBUG (MainThread) [custom_components.tapo] Migrating from version 6
2023-11-29 11:25:20.805 INFO (MainThread) [custom_components.tapo] Migration to version 6 successful
2023-11-29 11:25:20.806 INFO (MainThread) [custom_components.tapo.setup_helpers] Trying to track ip address of 9C-53-22-2B-1D-30, last known ip is <IP_ADDRESS>
2023-11-29 11:25:29.664 DEBUG (MainThread) [custom_components.tapo.setup_helpers] Creating new API to create a coordinator for 80220AE8E788B439A2DE1D1F39E1356B20F80EC4
2023-11-29 11:25:29.736 DEBUG (MainThread) [custom_components.tapo.setup_helpers] Creating new API to create a coordinator for 8022885F12229596C51AD31B87A0509320F8D425

It's due to wrong tracking mac address. This feature actually is too unstable, I will disable it in the next release while I'm working on a true auto-discovery mechanism

Thank you for the feedback ! (and the good work)


Fix off-by-one error in column calculation for popup_setpos in mouse (and bugs in visual line and also one-char selection)

This fixes an off-by-one error arbitrarily introduced in the popup_setpos calculation (in get_fpos_of_mouse()) code in mouse.c. I'm not really sure why it's there as we intentionally decrement the column by 1, but from looking at the code I cannot see why it's there at all, considering the column is already 0-indexed.
Testing this in GTK and MacVim confirms that it fixes the issue, and the function isn't used outside of popup_setpos.

Here's a video (taken under Ubuntu GTK, with latest Vim 9.0.735):
https://user-images.githubusercontent.com/1217449/195467169-7c8e0a6f-5fdb-47a7-9995-261ee9278957.mov
Note that right clicking one column to the right still kept the selection, but somehow right clicking at the left-most column of the selection would lose it. This shows the off-by-one error, but I'm not 100% sure why the original code was there. Probably just an oversight.

I pushed an additional commit to fix other bugs i found in popup_setpos mode, where apparently visual line mode basically doesn't work properly, and if you have normal visual mode with only one character selected it also wouldn't work.
Copy-pasting the commit message below:
Fix popup_setpos bugs in visual line mode, and when selecting only one char
Fix two bugs related to popup_setpos:

When selecting only one character in visual mode, right-clicking away
wouldn't deselect the selection. That's because the code was only
using less-than operators to check w_cursor against VIsual, instead
of using less-than-or-equal.
Visual line didn't really work properly. For one, it suffered the
above issue of only one character selected, and second, the logic
didn't really account for line selection (so if you click at the
beginning or end of line, you would end up deselecting even though
you still right clicked on the selection). Fix this by adding a
special check for visual line mode to make sure it works by line
number instead of cursor pos.

Also added tests. Should be good to review now.


Event Subscriptions - 500 error - ngrok

Description
I'm not sure what changed from last week to this week or if it is something with my system. Regardless I cannot use ngrok to expose my locally hosted slackbot. When entering the ngrok URL into New Request URL I receive the information in the label of the input 'Your URL didn't respond with the value of the challenge parameter.' Additionally, I get the notification Your request URL gave us a 500 error. Update your URL to receive a new request and challenge value.
I tried the same with localtunnel and there it works fine. (I don't like using localtunnel because it tends to crash more often than ngrok.) I uninstalled and reinstalled ngrok with npm and brew for both it doesn't work.
What type of issue is this? (place an x in one of the [ ])

[x] bug
[ ] enhancement (feature request)
[x] question
[ ] documentation related
[x] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
 I'm running into the same problem and struggling to figure out if I'm doing something silly.

For anyone else running into this issue, I was able to fix it by running ngrok http <IP_ADDRESS>:3000 instead of ngrok http 3000.

both didn't work for me... I'm still getting a 502 Bad Gateway

Did you able to solve this issue I have the same error


Notebook shows up at the wrong time after changing editor setting

Open a notebook in a tab in the background
Open settings.json
Toggle the editor.scrollBeyondLastLine setting
The notebook shows up on top even though its editor tab is not visible

Peng says editor config change triggers a layout

https://github.com/microsoft/vscode/blob/master/src/vs/workbench/contrib/notebook/browser/notebookEditorWidget.ts#L124 which tries to layout the list view after the setting is changed and we should only update the widget's visibility when it's visible https://github.com/microsoft/vscode/blob/master/src/vs/workbench/contrib/notebook/browser/notebookEditorWidget.ts#L724-L726


Update README AboutUs page

README.adoc: Edit badge
AboutUs.adoc: Edit description and change file format.
docs/images: Add my profile image in png format.

Pull Request Test Coverage Report for Build 21

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

Totals

Change from base Build 20:
0.0%

Covered Lines:
1433

Relevant Lines:
1544

💛  - Coveralls


Issue: No calendar popup when selecting 'Defer Until..."

Hi,
When I select the 'defer until...' option in the Nudge UI, the calendar does not pop up for users to choose a custom date/time. Nudge is closing and will reopen at the hour's 0- or 30-minute mark.  It feels as if the app is crashing.
The video below is a screen recording of the issue, and the steps to reproduce it are to deploy Nudge and then click 'Defer until..."
https://github.com/macadmins/nudge/assets/83666991/6958386f-8547-4a68-9f13-a53eaf66aabd
I have the following set:

Required install date set as 3/28/24
Approaching Refresh Cycle is 6 hours
Approaching Window Time is 4 days
Imminent Refresh Cycle is 4 hours
Imminent Window Time  is 2 days
All User Quit Deferrals is set to True

I am using Post-Install Script 0.0.22 and Nudge <IP_ADDRESS>549 pkg.
Thanks!

I also have crash reports in Console logs for Nudge.

Please send me the crash reports and your entire preferences set. I cannot recreate the calendar issue.

please try https://github.com/macadmins/nudge/releases/tag/v<IP_ADDRESS>554


Added the location of the export zone file

Added the location to where the export zone file can be found (C:\Windows\System32\DNS) and that the location is on the DNS server.  Also added a note that -FileName is a filename and not a filepath.  This should clarify how users can use this command and where they can locate the output.

Docs Build status updates of commit bf90148:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2019-ps/dnsserver/Export-DnsServerZone.md
:white_check_mark:Succeeded
View (WindowsServer2019-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 6086632:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2019-ps/dnsserver/Export-DnsServerZone.md
:white_check_mark:Succeeded
View (WindowsServer2019-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Tested this, ok, can you please also correct the description for FileName parameter? Thank you!

I think I have made the requested change to -FileName.  Please review.

Docs Build status updates of commit bec3639:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2019-ps/dnsserver/Export-DnsServerZone.md
:white_check_mark:Succeeded
View (WindowsServer2019-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 2c42ae9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2019-ps/dnsserver/Export-DnsServerZone.md
:white_check_mark:Succeeded
View (WindowsServer2019-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


[silgen] Change SwitchEnumBuilder to invalidate the insertion point u…

…nconditonally after emitting a block, rather than asserting so.
If we perform an exit and do not use a terminator, then we will still have a set
insertion point. By unconditionally unsetting it, we gain the property that we
want and are nice to the normal exit case.

@swift-ci smoke test and merge

Same with this one.


[MEGATHREAD] ECONNECT errors and unresponsive plugin

Overview
I'm creating a megathread for us to pool thoughts and spread information. For the past few months, more and more people have been experiencing errors in their logs to the affect of:
Error: GET https://www.alarm.com/web/api/devices/sensors?idsxxxxx failed, reason: read ECONNRESET
    at /homebridge/node_modules/homebridge-node-alarm-dot-com/node_modules/node-alarm-dot-com/dist/index.js:470:15
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
    at async Promise.all (index 1)
    at async Promise.all (index 0)

This error may or may not be accompanied by your accessories becoming unresponsive.
Why is this happening?
I believe what has happened is that Alarm.com has started ratelimiting calls to their APIs. This is completely fair, and likely to prevent DDOS attacks, but it means that our plugin is having trouble due to how "chatty" it is.
What can I do right now?
My testing has led me to believe that they're limiting peoples' IP addresses once a threshold of requests has been reached. This means you can (likely) get around the restriction by doing two things:

Do this first! Increase the values authTimeoutMinutes and pollTimeoutSeconds. I would try setting authTimeout to 60 and pollTimeout to 300. This will reduce how chatty the plugin is while also decreasing how responsive it is to external changes. Doing this may help prevent you from getting banned again.
Restart your homes internet modem. For most people, this will get you a new external IP address so that you're no longer banned.

How can we fix this long-term?
Assuming this is the actual cause of the problem, the fix would be increasing the defaults for the above values. Since this will reduce how responsive the plugin is, we will need to build dynamic code so local device changes are properly reflected. Unfortunately I'm not sure if there's any way around making the plugin less responsive to external changes.
We should also at some point comb through the code to see if it's feasible to reduce the amount of API calls it makes. This may help prevent people from getting banned.
I'm sorry to the people this has inconvenienced.

I reinstalled to give your advice a try, but I have been unable to change my public IP, so I'm now experiencing a new issue: only my panel and garage door are being discovered, they both work though.

I change the values for authTimeoutMinutes and pollTimeoutSeconds to the recommended values and have let run for the last 2 days. Consistently getting the following error every 5 minutes and I am not able to control Alarm.com:
[5/16/2021, 7:42:14 AM] [Security System] Error: GET https://www.alarm.com/web/api/systems/systems/2736084 failed: [object Object]
    at /usr/local/lib/node_modules/homebridge-node-alarm-dot-com/node_modules/node-alarm-dot-com/dist/index.js:470:15
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
    at async Promise.all (index 0)

Also have same issue as @ifeign where I cannot change my public IP as this seems to be locked to my modem's MAC address.
Question, if I have been blocked by Alarm.com and I cannot change my public IP am I basically out of luck using this plugin?

In the latest beta (1.7.2-beta.4) I added some randomness to the device refresh, so whatever value you put will have between 0-5 minutes added. I don't think any one thing will solve this problem, but eventually we'll have enough variance to not get caught.
Eventually your MAC address will switch, and if your mobile app still works we can assume there's a way around this. The issue is that I don't have enough time or energy to troubleshoot this forever...

For those of us with Qolsys panels, which is probably quite a few, addressing my issue would fix polling of sensors, meaning potentially less server load - you’d just have to poll when something like a garage door or lock is used, vs constantly checking sensor status https://github.com/node-alarm-dot-com/homebridge-node-alarm-dot-com/issues/71

Chase, first and foremost, thank you for all the work and time you are putting in to this. These recent errors I imagine have been nothing but a headache for you. I was running on 1.7.2-beta.4 for awhile. I was having the constant ECONNRESETS but was still able to arm/disarm for the most part, which was really all I need. I upgraded to the latest beta, 1.7.2-beta.7 and now can not connect at all. I am receiving the error below. I have tried deleting and reinstalling and have re-obtained by two factor authentication from a web browser and updated it. Wanted to share in case it was a beta opportunity vs. related to the ongoing issues with alarm.com.
Also, is there a way to simplify this plugin to only handle arm/disarm/stay and not pull in sensor info? If so, would that eliminate the issues with the econnreset?
[5/17/2021, 12:59:00 PM] [Security System] UNHANDLED ERROR: Error: No afg cookie: ASP.NET_SessionId=2j5cwe0n0wb2vhy5fouckb3r; cookieTest=1; twoFactorAuthenticationId=E41FAA1436B1FE4D43E5998EC8BFB384225F419160F683122AD4A478DF7573ED; IsFromNewSite=1; fromASP=; IsFromSeamlessLogin=; login=; ST=; SeamlessLoginEnabled=; BIGipServer~AlarmApplication~Alarm_WEBADC_Alarm_HTTPS=!/JMvL1D5Bxx0DuvNzUFX+zqT5KjmGEm9aGsXXHJPy7lr2+IC5RA50VArb6pgxg9KGDHl5B+3qGRpyQ8=
at /usr/lib/node_modules/homebridge-node-alarm-dot-com/node_modules/node-alarm-dot-com/dist/index.js:93:23
at processTicksAndRejections (internal/process/task_queues.js:95:5)

Chase, first and foremost, thank you so much for all the work you have done on this plugin. It has been one of my favorites on homebridge but I can’t imagine the headache it’s caused you since this error.
I was wondering, is it possible to strip the plug-in down to just having the ability to set the arm state of the system and not pull in all of the accessories? Having the arm state connected to “good night” scenes, away from home geofencing etc is absolutely awesome. My thought was perhaps it would only need to log in when an arm/disarm command is sent.

I change the values for authTimeoutMinutes and pollTimeoutSeconds to the recommended values and have let run for the last 2 days. Consistently getting the following error every 5 minutes and I am not able to control Alarm.com:

[5/16/2021, 7:42:14 AM] [Security System] Error: GET https://www.alarm.com/web/api/systems/systems/2736084 failed: [object Object]

    at /usr/local/lib/node_modules/homebridge-node-alarm-dot-com/node_modules/node-alarm-dot-com/dist/index.js:470:15

    at runMicrotasks (<anonymous>)

    at processTicksAndRejections (internal/process/task_queues.js:95:5)

    at async Promise.all (index 0)

I too am now seeing this after changing the settings as suggested.
(I only noticed seeing this in my logs when I recently upgraded to a much newer version of nodejs.)

I will add to the thanks for this plug-in. It worked flawlessly for quite some time which is pretty amazing!
I have Homebridge running on a Mac Mini. What I find really strange is that I can login to alarm.com using Safari on this Mac, however, the alarm-dot-com plugin running on the same Mac fails at login.

@chase9 Hi Chase,  I installed beta 9 this morning.  I was able control a door lock 1 time before getting the following:
[5/18/2021, 10:15:57 AM] [Security System] Error: GET https://www.alarm.com/web/api/systems/systems/5349741 failed: [object Object]
at /usr/lib/node_modules/homebridge-node-alarm-dot-com/node_modules/node-alarm-dot-com/dist/index.js:464:15
at processTicksAndRejections (internal/process/task_queues.js:97:5)
at async Promise.all (index 0)
Plugin and all devices have become unresponsive now.   Not sure how but ADC appears to be detecting plugin connections vs app or web interface connections almost immediately.

Hi Chase, similar behavior as @ngori. Installed beta 9, configured and rebooted. On initial boot, login was successful returning registered panel and devices as well as initial state (motion detected). First sample loop returned the ECONNECT error and devices no longer updated.
I only have alarm system with no additional devices such as lights and garage door. Login through webpage and app occur without issue.

Hello.  I just wanted to drop back by and report on this comment mentioning the Home Assistant plugin.
uvjustin/alarmdotcom and uvjustin/pyalarmdotcomajax also suffer from this issue for me.  The difference is, as best I can tell, this integration only supports arming and disarming so it kinda brute forces the operation.
raise client_error(req.connection_key, exc) from exc
aiohttp.client_exceptions.ClientConnectorError: Cannot connect to host www.alarm.com:443 ssl:default [Connection reset by peer]
2021-05-20 22:08:01 WARNING (MainThread) [homeassistant.helpers.entity] Update of alarm_control_panel.alarm_com is taking over 10 seconds
2021-05-20 22:08:07 ERROR (MainThread) [pyalarmdotcomajax.pyalarmdotcomajax] Can not load state data from Alarm.com
2021-05-20 22:08:31 WARNING (MainThread) [homeassistant.helpers.entity] Update of alarm_control_panel.alarm_com is taking over 10 seconds
2021-05-20 22:08:37 ERROR (MainThread) [pyalarmdotcomajax.pyalarmdotcomajax] Can not load state data from Alarm.com

I think we can pretty conclusively say we are triggering some sort of defensive DDoS protection on their side.

It’s very unlikely to work, but has anyone tried reaching out to the Alarm.com dev team?

I believe we had a former ADC dev assisting for a while. To your point it's worth a shot though.
Couple other thoughts/brainstorms:
I thought there was a version of an ADC plugin at one point doing a screen scrape. Very inefficient vs the api's but possibly an option that wouldn't get blocked.
The other thought was routing this plug-in traffic through a vpn service and varying the end points. If the denial is just based on ip (seems like this is possibly given there has been limited success with new WAN ips) this might work. If ADC is rate limiting at the account level it wouldn't help.  Not sure how feasible it would be to implement though given you would need a commercial VPN service.

Hello,
Bit of feedback the other way around: I use 1.7.1, I have 30+ sensors/detectors and while I have a lot of the ECONNRESET errors, generally speaking the system is working and reporting very well.
I have authTimeoutMinutes set to 30 and I didn't set pollTimeoutSeconds so it's set to whatever the default is.
I do have an issue with the arming/disarming of the system but I worked around it by creating dummy switches (homebridge-dummy). Instead of arming/disarming from the homebridge-node-alarm-dot-com alarm accessory, my automations interact with the dummy switches. The dummy switches automatically turn off after 30 seconds and I have additional automations that turn the alarm on (or off) when the dummy switches are turned on AND off.
That works well. Often it doesn't change the alarm state when the dummy switch is turned on, but it does work after 30 seconds when it turns off.
So it seems to me that the problem is that something somewhere needs to be refreshed, and it's refreshed by the initial attempt at changing the status of the alarm.
If I can help troubleshoot this let me know.
Great plugin for me, thanks for all the hard work!

Has any progress been made on this? Just my luck that I moved into a house that came with an Alarm.com system only for this to happen lol

@jfmach , how do you control the alarm on and off states with dummy switches?  That would solve this problem for me.  Thanks.

So, I decided to give the plugin another try. As of this writing, my alarm system, lock and garage door work. BUT I don't have any door, window or motion sensors. This isn't the biggest deal, but it's a little strange. I cleared my accessory cache and it didn't change anything.

I unloaded the plugin a couple of weeks ago. Retried yesterday and initially thought that the issue had cleared on the Alarm.com end. Within 30 minutes the ECONNECT errors returned and the plugin would not respond. Started looking over the code just for understanding. As webpage and iOS app work just fine wondering if there is a problem in the way the ‘node-fetch’ node_module queries the Alarm.com servers.

I unloaded the plugin a couple of weeks ago. Retried yesterday and initially thought that the issue had cleared on the Alarm.com end. Within 30 minutes the ECONNECT errors returned and the plugin would not respond.

Are you using the beta release? It’s been almost perfect for me. Sure, things are a little slower due to the reduced polling, but it’s been 90% reliable. I haven't seen ECONNECT errors, but have seen other random timeouts that eventually resolved themselves

Yes, I am using the beta. What do you have your sampling settings at? Default or higher?

Default, timeout 10, polling 60
On Sun, Jun 13, 2021 at 9:10 AM Doug B @.***> wrote:

Yes, I am using the beta. What do you have your sampling settings at?
Default or higher?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/node-alarm-dot-com/homebridge-node-alarm-dot-com/issues/72#issuecomment-860217359,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHWLHYEWF7HTNPHC4SQPTQLTSS367ANCNFSM445CTDQQ
.

Thanks. I have homebridge running on an M1 Mac Mini. Generated a new Safari mfaCookie for the Mini and reloaded beta-9 of the plug-in with default values. Will see if errors are generated and if so impact on functionality.

Thanks. I have homebridge running on an M1 Mac Mini. Generated a new Safari mfaCookie for the Mini and reloaded beta-9 of the plug-in with default values. Will see if errors are generated and if so impact on functionality.

IDK if it helps, but I made a secondary alarm.com account specifically to use with this plugin and to reduce server activity from my primary account - added bonus is you can revoke that account should you suspect any security breach

I thought of giving that a try. With loging level at 4, I am getting the following error:
[6/13/2021, 4:35:18 PM] [Security System] Error: GET https://www.alarm.com/web/api/systems/systems/2736084 failed: [object Object]     at /usr/local/lib/node_modules/homebridge-node-alarm-dot-com/node_modules/node-alarm-dot-com/dist/index.js:464:15     at runMicrotasks (<anonymous>)     at processTicksAndRejections (internal/process/task_queues.js:95:5)     at async Promise.all (index 0)
problem or not?

That looks like an error I’ve been getting, that seems to make the plugin unresponsive briefly. It seems to resolve itself though, usually a child bridge restart (or homebridge restart if you haven’t set up child bridges) fixes it if I’m not patient enough.

Set up a secondary account just for homebridge with full control. All sensors are visible.
Do you recommend running plugin in a child bridge?

Set up a secondary account just for homebridge with full control. All sensors are visible.
Do you recommend running plugin in a child bridge?

I like the convenience of child bridges, I can restart one platform without restarting the whole server. You’ll have to re-add all your related accessories if you do move to a child bridge, they also use more RAM, the more you have. Thankfully I’ve got an 8gb Pi.

Using the secondary account I am still occassionally getting the same error but interaction with Alarm.com seems to be working. Will run with this configuration for now. Would like to try out child bridges but leave for another day.

I have no knowledge of coding but hope this will help or give some clue.
I reinstalled v. 1.7.2-beta. 4 (the first beta that supposed 2FA) and I have no error message so far.

This is a great observation. I loaded v1.7.2-beta.4 and also do not have any error messages. Log is showing that security system is polling every 10 minutes. This was not true with beta.9.

This still happens

On which release? I’m kinda wondering if we’ll see some of the people who rolled back to the earlier beta will get flagged by the server and go back to econnect errors

I am trying to see if I can start with beta-4 and add in the code changes stepwise to see where the problem occurs.
The first item I found was that the randomized timer was I believe 10x higher than planned resulting in a very long polling interval. Have submitted comment to this code change.

I believe I found the issue with beta-9 but want to do more testing to make sure.

Secondary accounts are not the answer.  Some of us cannot create one.  Alarm companies that work with alarm.comhttp://alarm.com have that option and restrict it.  I for now cannot create a secondary account.  I was trying to help test
On Jun 14, 2021, at 9:05 AM, Doug B<EMAIL_ADDRESS>wrote:
Using the secondary account I am still occassionally getting the same error but interaction with Alarm.comhttp://Alarm.com seems to be working. Will run with this configuration for now. Would like to try out child bridges but leave for another day.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnode-alarm-dot-com%2Fhomebridge-node-alarm-dot-com%2Fissues%2F72%23issuecomment-860804517&data=04|01||b1311bf652ad4c029bbd08d92f4e3346|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637592835176698046|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Y6CvZ6mqi1WCg0saqhkxIsUZ%2F7GZhGY7SqMTOti0LJg%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAT2J4BX6KBLK7G7HZNXGLHLTSYSDXANCNFSM445CTDQQ&data=04|01||b1311bf652ad4c029bbd08d92f4e3346|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637592835176708042|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Qml2KDDnavelzxqAUVypnfxcHH4z%2FXGFapOoY8fYhPs%3D&reserved=0.

Secondary accounts were more of a theory anyway. My main reason for using it is if my homebridge passwords ever leak, I dont leak the info from my primary account

Well I realized that.  I gathered that from the e-mail chain/.  Just pointing out my observations for your theory
On Jun 16, 2021, at 6:42 AM, ifeign<EMAIL_ADDRESS>wrote:
Secondary accounts were more of a theory anyway. My main reason for using it is if my homebridge passwords ever leak, I dont leak the info from my primary account
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnode-alarm-dot-com%2Fhomebridge-node-alarm-dot-com%2Fissues%2F72%23issuecomment-862390834&data=04|01||2be9721a2cbd4e0906b208d930cc85ea|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637594477235345935|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Al86MZitr4fjZ1Dw8eToVDf7Gemu5T3fuBjKKHUawRQ%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAT2J4BWNQZUNJRHZF5DXDETTTCS2VANCNFSM445CTDQQ&data=04|01||2be9721a2cbd4e0906b208d930cc85ea|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637594477235355931|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=EHEHdClgm%2BBogCLsCjY43JV7a8KQZX70dB1VP5w5dJ8%3D&reserved=0.

Hi all,
The changes from @DMBlakeley have been merged into the latest beta (Beta 11). Could you please update to the latest beta and let me know if you run into any problems? I've been running it for a day now that have seen the plugin successfully recover from an ECONNECT error.

Hi, I’ve been running beta 11 since it was pushed last week. Have seen significant improvements- at best, I receive a refresh devices error a couple of times a day that quickly resolves itself.
Anthony

On Aug 18, 2021, at 8:37 AM, Chase Lau @.***> wrote:

Hi all,
The changes from @DMBlakeley have been merged into the latest beta (Beta 11). Could you please update to the latest beta and let me know if you run into any problems? I've been running it for a day now that have seen the plugin successfully recover from an ECONNECT error.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

How do I go about loading and installing the latest beta (Beta 11) into Homebridge?

If you are using Homebridge UI for plugin configuration, go to the Plugins page and select the "wrench" symbol and then "Install previous version". You will find beta.11 as an option.


Implement sshoogr main app that will be capable of running *.sshoogr scripts

Those scripts are just plain Groovy with few magic imports added

You can use groovy with Grab.
Execute with groovy scriptfile.groovy
@Grab('com.aestasit.infrastructure.sshoogr:sshoogr:0.9.18')
import static com.aestasit.infrastructure.ssh.DefaultSsh.*

options.trustUnknownHosts = true
options.failOnError = false
options.execOptions.failOnError = false

remoteSession(...) {
//       sshoogr magic
 }

The idea was to have executable that can be called like that:
  sshoogr --hosts=list_of_hosts.txt --trust=true --failOnError=false magic.sshoogr


How to choose the vaule of weight when using the GPS data to fix frame ?

It seems the accuracy and covariance of GPS data are not used when optimizing. So what is the relationship between GPS data accuracy and frequency and weight? And I think if change the weight according to GPS data, we can get a better result. Is my idea feasible?

@daijicheng
Do  you have test the cartographer in 3D with GPS?

Hi @daijicheng ,

No, I don't test the cartographer in 3D with GPS yet, but I plan to do this.

So now hava you tested cartographer in 3D  with GPS?
I'm planning to do this. I have got my gps topic by  using nmea_navsat_driver and assigned  "use_nav_sat = 1", What else should do?
Hoping for your reply!
Thank you very much!

@Fanny-one
What you need to do is publishing the nav_sat message with the topic name "fix".
I have not finished test cartographer with GPS. If you have any progress, I hope you can show me the result.

What you need to do is publishing the nav_sat message with the topic name "fix".
I have not finished test cartographer with GPS. If you have any progress, I hope you can show me the result.
thank your reply!
ok!
My  colleagues are testing cartographer without GPS . Now I want to test it with GPS but don't know how to do that. Maybe we can communicate with each other through qq which is 827938633 or wechat.


Utilize Promise.all for Efficient Asynchronous Operations 🚀

PR Checklist
Please check to confirm your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/angular/angular-cli/blob/main/CONTRIBUTING.md#-commit-message-guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
This pull request refactors the code to leverage Promise.all for running multiple asynchronous functions concurrently. By utilizing Promise.all, we achieve improved performance and readability when dealing with asynchronous operations.
Benefits of Promise.all:

Improved Performance: Promise.all allows all the asynchronous functions to execute concurrently, potentially leading to significant performance gains, especially when dealing with independent tasks.
Simplified Code: By using Promise.all, we can group multiple asynchronous operations into a single line, making the code more concise and easier to maintain.

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Thanks for the contribution but closing as PR feedback has not been addressed. Please feel free to open a new PR.

https://github.com/angular/angular-cli/pull/28056 Sorry for the delay


bug in SHOW DATA/FULL triggered presumably by redefining dataset time axis

Reported by steven c hankin on 13 Apr 2015 19:26 UTC
This bug has shown up while experimenting with FMRC forecast collections from TDS.  These datasets have no 1D time axis.  The "should" have a lag axis, representing the time interval between time steps of each run.  So in the script below we define such a lag axis and replace the original (index only) axis with it.  Something -- TBD -- is not done thoroughly in this replacement, resulting in a crash when SHOW DATA/FULL is executed.
The bug has been in there at least since V6.92 -- possible much longer.  The dataset fmrc_hawaii.nc is in /home/users/hankin/FERRET32/fer.  Any other TDS FMRC output may be used instead to demonstrate the problem.
use fmrc_hawaii.nc

! no problem
show data/full

! no redefine the time axis
! none defined in file, so index only axis at present
define axis/t=0:1368:3/unit=hour/t0="06-may-2011" t1d
define axis/units=days/t=0:`.125*64`:.125 time

! now the show data crashes
show data/full

quit

! here's the error log:
show data/full
     currently SET data sets:
    1> ./fmrc_hawaii.nc  (default)

 name     title                             I         J         K         L         M         N
 TIME     Forecast time for ForecastModel  ...       ...       ...       1:65      ...       1:50
             hours since 2011-05-06T00:00:00Z on grid GOL1 with -1.E+34 for missing data
             F=05-MAY-2011 12:00:00:25-JUN-2011 00:00:00
 DLWRFSFC net downward longwave radiation  1:101     1:41      ...       1:65      ...       1:50
             W m-2 on grid GOL2 with -9.99E+08 for missing data
             X=140.3W:89.8W  Y=10.3S:10.3N  F=05-MAY-2011 12:00:00:25-JUN-20
Fortran runtime error: Array reference out of bounds for array 'line_modulo', lower bound of dimension 1 exceeded (in file 'tm_world.F', at line 84)

Note that if you try these commands on a released Ferret, compiled with optimizations on and bounds checking off, it will not crash.  It will instead produce the "???" output below.
        NOAA/PMEL TMAP
        FERRET v6.92
        Linux 2.6.18-371.9.1.el5PAE 32-bit - 07/10/14
        13-Apr-15 12:08
.
.
.
! now the show data crashes
show data/full
     currently SET data sets:
    1> ./fmrc_hawaii.nc  (default)

 name     title                             I         J         K         L         M         N
 TIME     Forecast time for ForecastModel  ...       ...       ...       1:65      ...       1:50
             hours since 2011-05-06T00:00:00Z on grid GOL1 with -1.E+34 for missing data
             F=05-MAY-2011 12:00:00:25-JUN-2011 00:00:00
 DLWRFSFC net downward longwave radiation  1:101     1:41      ...       1:65      ...       1:50
             W m-2 on grid GOL2 with -9.99E+08 for missing data
             X=140.3W:89.8W  Y=10.3S:10.3N  F=05-MAY-2011 12:00:00:25-JUN-20

  time range: ???

Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2271

Comment by ansley.b.manke on 14 Apr 2015 16:27 UTC
I think this bug goes back forever; it's about redefining the record axis.
yes? define axis/x=1:5:1 xax
yes? define axis/y=2:10:2 yax
yes? define axis/t=1:7:1 tax
yes? let var = x[+ y[gy=yax](gx=xax]) + t[save/file=xyz.nc var

cancel definitions or start a new session
yes? use xyt
yes? define axis/t=1:7:1/units=days/t0=1-jan-2000 tax
Replacing definition of axis TAX
           *** NOTE: grid  used by data set xyz
           *** NOTE: Redefinition may alter apparent contents of data set
yes? sh dat/full
     currently SET data sets:
    1> ./xyz.nc  (default)

 name     title                             I         J         K         L         M         N
 VAR      X[GX=XAX](gt=tax]
yes?) + Y[GY=YAX] + T[GT=TA  1:5       1:5       ...       1:7       ...       ...
               on grid GBM1 with -1.E+34 for missing data
             X=0.5:5.5  Y=1:11

  time range: ???

or in a Ferret compiled with bounds checking, the runtime error.
The DEFINE AXIS command defines a new axis, and it replaces the information in the grids but not in the list of dataset record axes,  ds_time_axis(dset).
SHOW AXIS/FULL reports the coordinate range of all the dimensions, skipping the time axis, then goes and looks for the time axis,
* describe the time range (uses "grid" from last variable displayed)
	taxis = ds_time_axis(dset)

but taxis is pointing to the original time axis from the dataset and so it's not in any of the grids.
-- a second question is about record axis in 6-D Ferret. We punted on this a bit; saving data from the file in the original report, we do this:
yes? save/i=1:5/j=5/file=a.nc/clobber dlwrfsfc
 LISTing to file a.nc
           *** NOTE: Saving E or F axis with time axis uses /RIGID in time

Comment by ansley.b.manke on 14 Apr 2015 17:54 UTC
All that's needed is a change in purge_mr_axis.F, where grid are updated, to also check for the old axis in ds_time_axis(iset) and replace it with the axis number of the new axis.
I also fixed a small bug in show_axis_full.F which was chopping off the output; it may be longer for 6-D grids. This results in a change in the output of benchmark script bn_syntax.


incomplete sentence in readme?

see the description in "Usage" and somehow it ends with ". The":

You have to know how many integers were coded when you decompress. You can store this information along with the compressed stream. The

Fixed.


Add windows and macos to CI

@codetalker7
This PR adds Windows and MacOS to the CI.

@ShouvikGhosh2048 @codetalker7 - this looks like a old PR - if we have already closed it, shall we close the PR?


Two datepickers on the page both open

If you have two pop up datepickers on the page, both will open when either of the input buttons are focused.
I set up a plunker where you can see the changes: http://plnkr.co/edit/c28JnZfcuU0Gy5mX0max?p=preview
All I did was duplicate the existing html input field and the error occurs

Having the same issue as @arook where my datepickers are in ng-repeat so I have many of them and when I click one they all open. They have unique id's but currently share the same model. Is there a way to handle this?

@enjareyes you might try using the latest build or build from master as we've decoupled the date picker popup from the actual date picker directive.

i am using add new button and each form have from and to .
html :
`

`
IN CONTROLLER :
`$scope.dateFormat = 'dd-MM-yyyy';
    $scope.from = {
        opened: [false],
    };

    $scope.to = {
        opened: [false],
    };

    $scope.open_from = function (index) {
        for (var i = $scope.foodArray.length - 1; i >= 0; i--) {
            if (i === index) {
                $scope.from.opened[i] = true;
            } else {
                $scope.from.opened[i] = false;
            }
        }
    };

    $scope.open_to = function (index) {
        for (var i = $scope.foodArray.length - 1; i >= 0; i--) {
            if (i === index) {
                $scope.to.opened[i] = true;
            } else {
                $scope.to.opened[i] = false;
            }
        }
    };`


Small change to allow zbox to compile currently

Looks like the newest std library doesn't have a .emerge field in std.log.Level. So I changed it to .err. Also updated gitignore.
 $ zig build
./examples/log_handler.zig:23:9: error: enum 'std.log.Level' has no field named 'emerg'
    log(.emerg, .examples, "{s}", .{msg});
        ^
/usr/lib/zig/lib/std/log.zig:75:19: note: 'std.log.Level' declared here
pub const Level = enum {

I've made a mistake. I need to resend the pull request from the fixes branch. Sorry, I'm inexperienced with contributing!


400502660: ValueError: No objects to concatenate

Error in get_game_pbp

pandas sometimes reads PBP table column 'time' as 'time_of_day' even though on the site the column is clearly labeled 'time' ????????? wtf

323150087 also giving same error because of 0th half table in PBP


scripts: extract_dts_includes.py: Adhere to PEP8

This patch PEP8-ifies the extract_dts_includes.py script
Signed-off-by: Andy Gross<EMAIL_ADDRESS>

Awesome!
There are just a few minor issues that flake8 is still reporting:
$ flake8 scripts/extract_dts_includes.py
scripts/extract_dts_includes.py:10:1: F401 'pprint' imported but unused
scripts/extract_dts_includes.py:250:5: F841 local variable 'node' is assigned to but never used
scripts/extract_dts_includes.py:372:5: F841 local variable 'target_node' is assigned to but never used
scripts/extract_dts_includes.py:377:9: F841 local variable 'pin_entry' is assigned to but never used
scripts/extract_dts_includes.py:390:17: F841 local variable 'node_label' is assigned to but never used
scripts/extract_dts_includes.py:439:5: F841 local variable 'node' is assigned to but never used
scripts/extract_dts_includes.py:507:21: F841 local variable 'label' is assigned to but never used
scripts/extract_dts_includes.py:508:21: F841 local variable 'storage' is assigned to but never used
scripts/extract_dts_includes.py:577:5: F841 local variable 'compatible' is assigned to but never used
scripts/extract_dts_includes.py:770:9: F841 local variable 'flash_base' is assigned to but never used

Fixed Marti's comments.  fluke8 now reports no issues.


Chinese Pīnyīn instead of Chinese characters

Please provide a setting to display Chinese Pīnyīn (with tonemarks) in the HanziGraph instead of Chinese characters. This would be helpful to get a better feeling for the homonymes and thus the true character of the Chinese language. My reasoning and reason for this feature is the following:
My mother tongue is German and I have been studying Chinese language for almost 20 years, in all settings: university courses in China, living in China with native Chinese romantic partners, private courses, tutors, online tutoring, apps, graded readers (of which I read about 20), CDs, Youtube, everything in all configurations imaginable… yet I still can't even follow a simple conversation. However, last year

I have read several dozen books on Chinese grammar and history,
as well as on typography and historical script development in Europe,
as well as having read about the (real) history of romanisation in Vietnam (pushed by the Vietnamese revolutionaries and Vietnamese leaders, not the European foreigners) and the modern Vietnamese script,
as well as having studied Vietnamese for 6 months for 2 hours daily and purchased almost 50 books in Vietnamese language
as well has having studied the work of John Taylor Gatto and [moderated], which has provided me with the necessary emotional power

thus I have made the monumental and upright revolutionary choice to disregard Chinese characters forever on, regard them as a specialised study subject such as ancient Chinese history and herbarium science, and to continue studying with Chinese Pīnyīn only. Since then I have made REMARKABLE strides in studying Chinese language in a short time, as it suddenly is as easy as studying Spanish or Italian or any other language. It is truly suddenly a joy and much fun. The only drawback is the lack of reading materials in Chinese Pīnyīn (and the lack of "allies"), and the many writing mistakes Chinese tutors make since most tutors don't know yet the orthography rules of Chinese Pīnyīn, as of the CN Gov Pinyin Rules GB/T 16159-2012 (update of the 1996 version, which is already 28+ years available but still largely unknown.) Luckily ChatGPT is quite strong in Chinese Pīnyīn.
All that said to help your motivation :) Thank you for considering this feature.

To check if I'm understanding the idea:

the nodes in the graph would be pinyin syllables, e.g., diǎn or guān
clicking one would then show the most common characters with that pronunciation, along with definitions, example sentences, etc.
edges in the graph would be words formed by connecting the two pronunciations, so as an example, guān is connected to diǎn because of 观点, and other words with pronunciation guāndiǎn or diǎnguān would be shown.
the graph's edge behavior would continue as-is, showing definitions and examples for the words.

I should note that showing pinyin in the example sentences is already supported (there's an option in the menu accessible from the upper right), and all definitions include pinyin.
I think adding an option for showing pinyin in the graph could be interesting, though, so I can look into that.

Glad you might look into it! There's a couple of things that probably need some consideration, since suddenly we will be looking at the vernacular, spoken Chinese language. In Hanzi the morpheme (one character) is the smallest meaningful unit, but in the spoken Chinese language it's the 词 (cí) the word, that forms the smallest meaningful unit.
The rules of Chinese Pīnyīn are defined in GB/T 16159-2012, I'll attach a Chinese Simplified text version of this document that I made (including the corrections of Mark Swofford / pinyin.info).
Pinyin Rules GB:T 16159-2012 simplified formatted.txt
Tone Marks According to the rules in GB/T 16159-2012, Chinese Pīnyīn is written with diacritics, not numbers.
Nodes. Yes, a node would display a syllable in Chinese Pīnyīn. How would you plan to deal with ambiguity? For example 行 (xíng / háng) In the Chinese language the majority of words have either 1 or 2 syllables, so there's a good chance that a node is a word.
Edges Yes, edges would display 2 syllable words. According to the rules the diacritics are always true to the syllable, they do not change when syllables are combined. However, one exception, sometimes the diacritic on the 2nd syllable is dropped (like in 看看 kànkan) but there is no explicit rule for that in GB/T 16159-2012. In other cases it's 2 different words, for example 东西 can mean dōngxī (East-West) or dōngxi (thing).
Examples There's Pin1yin1 with numbers in the examples, but just for the node names, and missing an option to show the correct spelling with diacritics. The example sentences seem to be not written in Chinese Pīnyīn. Unfortunately there's currently no library that implements the rules GB/T 16159-2012 to produce compliant Pīnyīn from Hanzi. Furthermore, even large language models make many mistakes producing Pīnyīn. I guess there's just not a sufficient volume of Pīnyīn text available yet to train models.
Using Chinese Pīnyīn would indeed be very interesting to see on HanziGraph, and to engage in a research and to get a feeling for the vernacular, spoken language, as opposed to the more academic, abstract Chinese Simplified writing system.

I made a mockup how it could look like, first the original HanziGraph:

Then my mockup in Pīnyīn. Maybe not as "impressive" and "culturally rich" at first glance, but for sure easier to read (and containing more information about how words are currently spelled and pronounced):

I don't know how the edges are made, but diànshāng (diànzǐ shāngwù) would be a nice word too: e-commerce. Which touches on the subject of which meaning is chosen for the word, as diàn can mean "shop, store, location", or "electricity", amongst many others (the PLECO built-in dictionary has 16 entries for "diàn").
Furthermore I'm quite excited about the edges, they really make the short words shine and demonstrate how Standard (Mandarin) Chinese depends on context and a certain amount of syllables/words to be spoken before intended meaning becomes apparent. It also really shows the difference between the vernacular (spoken, living) language and the two very different writing systems. While Pīnyīn aims to accurately record the spoken language for writing and reading, Simplified seems to have completely different goals.

yeah, that mockup is what I had imagined would be interesting to build. I'll try to get it prototyped in the next few weeks.


current version drops cex when not set

for
usrDef <- gsplot(mar=c(4,4,4,4), xaxs='r', yaxs='r') %>%
     points(x=1, y=2, side=c(3,2), legend.name="Points 1", cex=3, xlab='cat') %>%
     points(x=3, y=4, side=c(1,4), legend.name="Points 2", pch=5, col="red", ylab=expression(paste("Discharge in ",ft^3/s))) %>%
     lines(x=c(3,4,3), y=c(2,4,6), legend.name="Lines 1", lty=5, col="orange") %>%
     lines(x=c(1,2,5), y=c(1,8,5), legend.name="Lines 2", lwd=3, ylab='Science!', xlab='dogs') %>%
     legend(x=1.5,y=4)
usrDef

in legendParams:
$pt.cex
[1]  3 NA NA NA

so pt.cex gets NA instead of the default (probably 1)

This whole method needs to be revisited - filling in NA is also messing with density specifications (see #219

curious @lindsaycarr , why did we move from a list to a data.frame for these legend styles? What problem did that solve?

since I was trying to add all legend arguments at one time, and call legend at the very end, there needed to be fillers when something wasn't specified so that it would still align with its corresponding args. For instance,
gs <- gsplot() %>%
points(1:5, col="blue", pch=21, legend.name="points 1") %>%
points(5:1, col="red", legend.name="points2") %>%
points(7:10, pch=5, legend.name="points3") %>%
legend()
gs

If it was a list, since points2 does not specify pch it needs a filler, otherwise the pch=5 from points3 would end up applying to points2. So I chose a data frame because it automatically filled in NAs. Obviously, this is not a sound approach


Support a tagged template constructor for JSONPath syntax

To some library consumers, it would be convenient if natural-lenses supported construction of a lens from JSONPath syntax.
Example:
// Instead of:
const lens = require('natural-lenses');
const lensOrig = lens('x', 'y', 0, 'z', 'odd key');

// Tagged template JSONPath construction could look like:
const { A } = require('natural-lenses');
const lensTT = A`$.x.y[0].z['odd key']`;

The syntax for constructing the lens looks like the JavaScript that would be used for getting or setting (if on the left hand side of =) the target slot if it could be assumed all intermediate containers existed and were JSON types.

Ideally, JavaScript values could be interpolated within square brackets for more complicated values, including customized steps:
const customStep = new lens.Step(...);
const lensOrig = lens(customStep);
const lensTT = A`$[{customStep}]`;

This would facilitate use of the tagged template construction with ES6 Map containers using non-string keys.  This also suggests it might be good to expose an A on the lens.Factory to fully facilitate use of Map containers.


s3.merge_upsert_table to work on empty destination

Describe the bug
I wonder why the merge upsert is not working if destination s3  is empty, when no files in the table's s3 bucket created yet. Somehow I expected to be able to merge into empty database:
{
  "errorMessage": "Parquet file size is 1 bytes, smaller than the minimum file footer (8 bytes)",
  "errorType": "ArrowInvalid",
  "requestId": "03b9e81d-879d-4155-845d-86c4c0957e6b",
  "stackTrace": [
    ...
    "  File \"/var/task/triggers-lambda.py\", line 147, in realy_save_to_avro\n    wr.s3.merge_upsert_table(delta_df= ds,\n",
    "  File \"/var/task/awswrangler/s3/_merge_upsert_table.py\", line 113, in merge_upsert_table\n    existing_df = wr.s3.read_parquet_table(database=database, table=table, boto3_session=boto3_session)\n",
    "  File \"/var/task/awswrangler/_config.py\", line 461, in wrapper\n    return function(**args)\n",
    "  File \"/var/task/awswrangler/s3/_read_parquet.py\", line 972, in read_parquet_table\n    df = read_parquet(\n",
    "  File \"/var/task/awswrangler/s3/_read_parquet.py\", line 777, in read_parquet\n    return _read_parquet(\n",
    "  File \"/var/task/awswrangler/s3/_read_parquet.py\", line 540, in _read_parquet\n    table=_read_parquet_file(\n",
    "  File \"/var/task/awswrangler/s3/_read_parquet.py\", line 480, in _read_parquet_file\n    pq_file: Optional[pyarrow.parquet.ParquetFile] = _pyarrow_parquet_file_wrapper(\n",
    "  File \"/var/task/awswrangler/s3/_read_parquet.py\", line 42, in _pyarrow_parquet_file_wrapper\n    return pyarrow.parquet.ParquetFile(\n",
    "  File \"/var/task/pyarrow/parquet.py\", line 230, in __init__\n    self.reader.open(\n",
    "  File \"pyarrow/_parquet.pyx\", line 991, in pyarrow._parquet.ParquetReader.open\n    check_status(builder.Open(rd_handle, properties, c_metadata))\n",
    "  File \"pyarrow/error.pxi\", line 99, in pyarrow.lib.check_status\n    raise ArrowInvalid(message)\n"
  ]
}

How to Reproduce

Create glue table
Try to call wr.s3.merge_upsert_table(

Expected behavior
success with the first data written to table files
Your project
No response
Screenshots

OS
AWS lambda linux
Python version
3.9
AWS SDK for pandas version
2.6.1 IMO
Additional context
No response

Hey,
Thanks for submitting this issue. I agree with your idea of the expected behavior. If a Glue table is already defined but empty, we should simply perform an insert.
However, I was not able to replicate the exact error you received. In my testing, the error I received was:
NoFilesFound: No files Found on: s3://aws-sdk-pandas-base-awssdkpandas2c9af251-2xcd1unijfnt/table/.

Can you double check whether you have an empty metadata file in your path that's causing this different error?
Best regards,
Leon

AWESOME! 10x guys for speed!

Thank you for bringing this to our attention. We will plan this release for version 2.17.x.


v-tooltip.esm.js?e37d:564 Uncaught (in promise) TypeError: Cannot read properties of null (reading 'theme')

v-tooltip.esm.js?e37d:564 Uncaught (in promise) TypeError: Cannot read properties of null (reading 'theme')
at _default (v-tooltip.esm.js?e37d:564:1)
at resolvePropValue (runtime-core.esm-bundler.js?5c40:2611:1)
at setFullProps (runtime-core.esm-bundler.js?5c40:2592:1)
at initProps (runtime-core.esm-bundler.js?5c40:2437:1)
at setupComponent (runtime-core.esm-bundler.js?5c40:6299:1)
at mountComponent (runtime-core.esm-bundler.js?5c40:4224:1)
at processComponent (runtime-core.esm-bundler.js?5c40:4199:1)
at patch (runtime-core.esm-bundler.js?5c40:3791:1)
at ReactiveEffect.componentUpdateFn [as fn] (runtime-core.esm-bundler.js?5c40:4339:1)
at ReactiveEffect.run (reactivity.esm-bundler.js?a1e9:160:1)

Fix proposal here : https://github.com/Akryum/v-tooltip/pull/815


[ERROR] Throw error explicitly on param value not in range

Motivation
The code checks if value is in valid range. If yes, sets it else ignores param and returns silently doing nothing.
Modifications
Explicitly throw IllegalArgumentException if param value not in valid range.
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): ( no)
The public API: (no)
The schema: ( no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation

If yes, how is the feature documented? (not applicable )

rub java8 tests

run java8 tests

run java8 tests

Hi @sijie Please guide me with the failure here.
Its seems there is some unrelated test that is failing on the jenkins:

2019-03-30\T\20:50:21.995 [ERROR] org.apache.pulsar.client.api.BrokerServiceLookupTest.testWebserviceServiceTls(org.apache.pulsar.client.api.BrokerServiceLookupTest)

run java8 tests


Update sql-database-paas-overview.md

Reduce ambiguity

Docs Build status updates of commit eee7f55:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azure-sql/database/sql-database-paas-overview.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@WilliamDAssafMSFT
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Thanks @verargulla for taking the time to submit this PR! Your commit will be merged and this article update is in flight. After the updated article is published, GitHub will list you as an article contributor.
Thanks again for this feedback, your suggestions help improve our documentation.
#sign-off


⚠ Bofrost.be has degraded performance

In cefdb35, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1367 ms

Resolved: Bofrost.be performance has improved in 7d52694 after 22 minutes.


[CT-718] Add Grants to Materializations

blocked by: https://github.com/dbt-labs/dbt-core/issues/5263
Once the above work is merged into core, there are some tweaks that need to be made in the spark adapter to fully expose the feature:

[ ] add a call to apply_grants(..., revoke=True) to all custom materializations

Important clarification that this functionality may only be available on the Databricks Runtime today, since I can't find OSS Apache Spark docs about it. I imagine (hope) it would become available in a future version, though, so I think dbt-spark is still probably the right place to put it. Open to feedback from Databricks folks if that's not the case.

get_grant_sql: https://docs.databricks.com/spark/latest/spark-sql/language-manual/security-grant.html (default should work)
get_revoke_sql: https://docs.databricks.com/spark/latest/spark-sql/language-manual/security-revoke.html (default should work)
get_show_grants_sql: https://docs.databricks.com/spark/latest/spark-sql/language-manual/security-show-grant.html (default should work — even though the docs only include show grant (singular), I've tested show grants (plural) and it works equivalently)


Filter WebSocket connections before open

Simple usage:
h.onHttpUpgrade([](uWS::HttpSocket<uWS::SERVER> s, uWS::HttpRequest req) {
    return false;
});

In this example WebSocket connections are not allowed.
If you provide no httpUpgradeHandler the connections are allowed by default.
Yep, s.terminate() would do the same. But consider this example:
h.onHttpUpgrade([](uWS::HttpSocket<uWS::SERVER> s, uWS::HttpRequest req) {
    return req.getHeader("origin").toString() == "http://example.com";
});

If the Origin header isn't http://example.com, connection is rejected.
Why this is better than checking it after the connetion is open? Because instead of adding it to the pool and removing it, we can close it before.
FYI, I didn't know how to name this feature properly, feel free to change it.

I need to rethink how the upgrade path should look like at some point. Maybe something like this will be added, maybe not. I don't know yet. Currently there are about 3 different paths and I would like to merge them all into one common path (Node.js addon from HTTP -> WS, Node.js addon from built-in HTTP -> WS, default upgrade path).

This PR is on hold.


Reproduce results for CGConv

Hello,
Another related question about CGConv. I looked at the original github repo for CGConv and compared it with your implementation. Your implementation is consistent with your documentation (all the math are right and same with the original paper) but different from code here. For example, the batch normalization is used in forward of the ConvLayer class but not in your CGConv class.
I was wondering when you implement a method, will you check with the original implementation, or do you mainly stick with the math in the paper? Sometimes some design choices may not be mentioned in the paper and this may cause some difference.
Hopefully, my question makes sense to you. I would like to use pytorch-geometric but also want to make sure I can replicate the result in the original paper.

Generally, I check for the original implementation, but must have missed the batch normalization here. Feel free to submit a PR to fix this if you are interested :)


[MXNET-969] Fix buffer overflow in RNNOp (v1.3.x)

Description
A port of https://github.com/apache/incubator-mxnet/pull/12603 to the 1.3.x release branch.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@KellenSunderland @nswamy

@mxnet-label-bot ["pr-awaiting-merge"]

@mxnet-label-bot ["pr-awaiting-merge"]


feat: display "license, language, and last updated at" in repo pages

Description
PR adds pills that show metadata (license, language, last updated) for the repository page.

Related Tickets & Documents
Closes #3706

Mobile & Desktop Screenshots/Recordings

Mobile + Language Explore page
https://github.com/user-attachments/assets/f02408d3-a42e-4378-95e4-58fb21992181
If there's no license or language

Steps to QA

Go to any repo page
See metadata pills under the repo avatar

Tier (staff will fill in)

[ ] Tier 1
[ ] Tier 2
[x] Tier 3
[ ] Tier 4

[optional] What gif best describes this PR or how it makes you feel?

unrelated:
It would be cool to index repos in starsearch based on license. I am not curious to find projects that popular and GPL.
related:
Checking repos that have no license or topics and it still displays the logos. We should remove it if none is present. I am assuming last commit date would be the repo created_at date if no code is present. Assuming a repo without a readme or code would not be interesting to look at, but I am curious if the app would 500 if there are no commits.

When I try it out on smaller screens there are scrollbars. This is in dev tools, but it happens on my phone as well.

It's supposed to be on scroll. I can wrap it instead if needed.

@zeucapua it should be the last_pushed_at field instead of the pushed_at field.

@zeucapua it should be the last_pushed_at field instead of the pushed_at field.

Ah I see my confusion. I didn't see last_pushed_at in the /repos/<repo>/info endpoint. Let me get the value off of /repo/<repo> endpoint instead.

For

When I try it out on smaller screens there are scrollbars. This is in dev tools, but it happens on my phone as well.

For this I would remove scrolling, add flex and let it wrap.

I talked to @isabensusan during the stand-up and since there's a chance we might add more information, it could get crowded on mobile if we wrap the data. And so the solution we arrived at was to add a gradient to show the user it's overflowing and can be scrolled on.

@zeucapua it should be the last_pushed_at field instead of the pushed_at field.

last_pushed_at isn't available in the /repo/<org>/<repo>/info endpoint which is being used in the repo pages. I'll make a quick issue and PR to add it in the API repo!

@zeucapua it should be the last_pushed_at field instead of the pushed_at field.

last_pushed_at isn't available in the /repo/<org>/<repo>/info endpoint which is being used in the repo pages. I'll make a quick issue and PR to add it in the API repo!

I see. That might introduce some more loading time to gather that data, so pushed_at may have to do for now

For

When I try it out on smaller screens there are scrollbars. This is in dev tools, but it happens on my phone as well.

For this I would remove scrolling, add flex and let it wrap.

I talked to @isabensusan during the stand-up and since there's a chance we might add more information, it could get crowded on mobile if we wrap the data. And so the solution we arrived at was to add a gradient to show the user it's overflowing and can be scrolled on.

I still find horizontal scrolling odd for this, but lets try it out.


Samples during adaption phase

Maybe we should make the sample function not return those samples by default (and support an optional keyword for it)?

yes, but let's consider fix these issues altogether in https://github.com/TuringLang/Turing.jl/issues/723

Is this issue resolved in AHMC now?

AHMC currently doesn't deal with dropping adaption phase samples at all. This is done in Turing side. I'm not sure if we want to support it here at all? How do you think?

We can also implement it here. So Turing side can just droping adaption samples via passing a keyword argument to AHMC. I just don't know which one is better (this or the current one).

Maybe  we can create a keyword argument in AHMC, the use it optionally in other packages like Turing.

I think Turing should maintain it's current behavior by dropping the samples at the end of sampling. If it's done strictly on the AHMC side, then the NUTS/HMC code won't work so well in Gibbs.
It is a good idea to include a keyword for AHMC though, if only to give users more options for raw AHMC.

Also, if AHMC knows those samples will be dropped, it doesn't have to store them at all, which could be a potential performance improvement.

... by dropping the samples at the end of sampling

Maybe Turing side should not keep the samples during adaptation at all if they were to be dropped in the end? @cpfiffer

That's really a design choice. You could move the adaptation stuff to sample_init! if you wanted to. A question I'd have is related to Gibbs: Do you need to run the other component samplers while HMC is running, or can it adapt independently? If it can adapt independently then we can just put the adaptation phase in sample_init! and everything's cool. Alternatively, if they need to be run together, we'd have to modify the Gibbs code to run the adaptation cycle ahead of time with all the component samplers.

question I'd have is related to Gibbs: Do you need to run the other component samplers while HMC is running, or can it adapt independently? If it can adapt independently then we can just put the adaptation phase in sample_init! and everything's cool.

It's likely this won't work well, because it'll only adapt a local covariance conditioned on variables not updated by HMC.

Maybe Turing side should not keep the samples during adaptation at all if they were to be dropped in the end? @cpfiffer

I think it's better to keep the samples during adaption, and optionallly drop them when sampling finishes. This is simpler to implement and maintain, and won't lose any efficiency other than some memory cost for storing the samples.

I think it's better to keep the samples during adaption, and optionallly drop them when sampling finishes.

What I mean here is that if drap_warmup=true is set, we do not need to first store them then drop them. Instead, we can avoid storing them completely. See my PR https://github.com/TuringLang/AdvancedHMC.jl/pull/108.

Closed by #108.