Suspected memory leak - when loading multiple models with tf.keras.models.load_model()

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Linux Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
NA
TensorFlow installed from (source or binary): binary wheel via PyPI
TensorFlow version (use command below):
2.1.0-dev20191231 (v1.12.1-21412-g3a094e6 2.1.0-dev20191231)
Python version:
Bazel version (if compiling from source):
NA
GCC/Compiler version (if compiling from source):
NA
CUDA/cuDNN version:
CUDA 10.0
GPU model and memory:
V100 32 GB
You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
I'm suspecting a CPU memory leak when loading multiple models.
When im running infinite loop that keeps loading the same model while using the same variable the memory (private bytes and working set) of the process keep increasing. At some points the working set seems to free some memory, but the trend is that the memory keeps on rising.
I used a simple model (attached).
This trend happens even though I call gc.collect() on every iteration and tf.keras.backend.clear_session().
the issue also happens in TF 2.0 (v2.0.0-rc2-26-g64c3d38 2.0.0).
for a specific model:
running in TF 2.0 each iteration adds 16 MiB
running in TF 2.1 each iteration adds 2 MiB
Describe the expected behavior
The memory shouldnt increase on each interation
Code to reproduce the issue
`
import os
import tensorflow as tf
import gc # garbage collector
import objgraph
from memory_profiler import profile

def mem_stat():
  objs = gc.get_objects()
  print("total objects count", len(objs))

@profile
def profile_own_model():
    model = tf.keras.models.Sequential([
        tf.keras.layers.Flatten(input_shape=(28, 28)),
        tf.keras.layers.Dense(128, activation='relu'),
        tf.keras.layers.Dropout(0.2),
        tf.keras.layers.Dense(10, activation='softmax')
    ])
    # model.save('my_model')
    tf.keras.backend.clear_session()
    del model
    gc.collect()

@profile
def profile_load_model(path):
    model = tf.keras.models.load_model(model_path, compile=False)
    tf.keras.backend.clear_session()
    del model
    gc.collect()

model_path = f'/my_model.hd5'
print("load model in loops:")

c = 1
while True:
    print("----------- iter", c)
    profile_load_model(model_path)

    print("mem stat after model creation:")
    mem_stat()
    objgraph.show_growth(limit=30)
    c += 1

`
Other info / logs

Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

@farotem, I tried reproducing the issue with Tf 2.1.but got different error message. Please find the gist and confirm. Thanks!

@gadagashwini hi I looked at the gist,
it looks like the profiler can't work under Ipython environment still you can look at the memory with your own tools..
to run the code just comment the memory_profiler part:
import os
import tensorflow as tf
import gc # garbage collector
import objgraph
#from memory_profiler import profile

def mem_stat():
    objs = gc.get_objects()
    print("total objects count", len(objs))

#@profile
def profile_own_model():
    model = tf.keras.models.Sequential([
        tf.keras.layers.Flatten(input_shape=(28, 28)),
        tf.keras.layers.Dense(128, activation='relu'),
        tf.keras.layers.Dropout(0.2),
        tf.keras.layers.Dense(10, activation='softmax')
    ])
    # model.save('my_model')
    tf.keras.backend.clear_session()
    del model
    gc.collect()

#@profile
def profile_load_model(path):
    model = tf.keras.models.load_model(model_path, compile=False)
    tf.keras.backend.clear_session()
    del model
    gc.collect()

model_path = f'/my_model.hd5'
print("load model in loops:")

c = 1
while True:
    print("----------- iter", c)
    profile_load_model(model_path)

    print("mem stat after model creation:")
    mem_stat()
    objgraph.show_growth(limit=30)
    c += 1

`

Installing tf-nightly-gpu with "pip install tf-nightly-gpu" fixed the issue for me

@farotem, I tried replicating the issue with commenting profile, looks like issue is related my_model file. Please find the gist. Can you try @retviews's suggestion and let us know how it progresses. Thanks!

Hi I tried installing tf-nightly-gpu, version: 2.1.0.dev20200107
still receives memory leaks in the CPU.
note 1:
the number of python objects doesn't seem to be the cause for the leak, after few iterations the number of python objects do not increase but the memory usage of the CPU is still increasing.
note 2:
I tried loading the same model from json and there are no memory leaks.
I think it means the memory leaks is somewhere in deserializing the model from folder.
I looked at the gist, below is the fixed code: (changed the path of file)
import os
import tensorflow as tf
import gc # garbage collector

def build_and_save_own_model():
    model = tf.keras.models.Sequential([
        tf.keras.layers.Flatten(input_shape=(28, 28)),
        tf.keras.layers.Dense(128, activation='relu'),
        tf.keras.layers.Dropout(0.2),
        tf.keras.layers.Dense(10, activation='softmax')
    ])
    model.save('my_model')
    tf.keras.backend.clear_session()
    del model
    gc.collect()

def profile_load_model(path):
    model = tf.keras.models.load_model(model_path, compile=False)
    tf.keras.backend.clear_session()
    del model
    gc.collect()

model_path = 'my_model'
build_and_save_own_model()
print("load model in loops:")
c = 1
while True:
    print("----------- iter", c)
    profile_load_model(model_path)
    c += 1

Is this still an issue with latest version of tensorflow (2.2)?

I seem to be having this same issue on TF 2.3.

tested on TF 2.3 same issue..

This seems to be reduced with the latest tf-nightly, tested with this colab notebook.
See also #40171 for more discussion.

It is reduced but not fixed sadly its still an issue for our team
I opened a new bug for nightly version (tf2. 4)
https://github.com/tensorflow/tensorflow/issues/45453#issuecomment-740755898


Updated functionality for add_rfree and copy_rfree

Making changes to rs.utils.add_rfree as per #169 and changes to rs.utils.copy_rfree as per #159
add_rfree takes seed
The seed is passed through to np.random.default_rng(seed), such that running add_rfree again with the same seed results in the same flags. seed defaults to None, and conveniently np.random.default_rng(None) is a valid call.
A test was added to test_add_rfree() to confirm that flags are the same when a seed is used, and are different when a seed is not used.
copy_rfree handles column names flexibly
The recent addition of the ccp4_convention parameter to add_rfree means that there are two possible names for an r-free column, "R-free-flags" (phenix convention) or "FreeR_flag" (ccp4 convention). copy_rfree will now by default search the provided dataset_with_rfree for a column named "R-free-flags", then "FreeR_flag", and throw a ValueError if neither is found.
The user can override this behavior via the custom_rfree_key argument if the r-free flag column uses any other naming convention.
A function test_copy_rfree() was added to test successfully copying in all combinations of column naming. A function test_copy_rfree_errors() was added to verify that ValueErrors are thrown when appropriate.
Old unittest code
There is an existing test for copy_rfree written with unittest. I believe that all key functionalities are tested in my new pytest code, making the unittest code redundant, but I left it in for now and labelled it in its docstring that it is a "legacy" version. Let me know if there's anything in the unittest code that I should additionally migrate, and/or if the unittest code should me removed altogether.

Codecov Report

Merging #170 (48b1e00) into main (2d279de) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #170   +/-   ##
=======================================
  Coverage   98.41%   98.42%
=======================================
  Files          44       44
  Lines        1704     1714   +10
=======================================
+ Hits         1677     1687   +10
  Misses         27       27

Flag
Coverage Δ

unittests
98.42% <100.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

reciprocalspaceship/utils/rfree.py
100.00% <100.00%> (ø)

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 facb76c...48b1e00. Read the comment docs.


Added localization files for brazilian portuguese (#431)

Signed-off-by: Thiago Romano<EMAIL_ADDRESS>Summary of Changes
Added the localization files needed to enable the brazilian portuguese language in Settings.
Related Issues
https://github.com/hyperledger/aries-mobile-agent-react-native/issues/431
Pull Request Checklist
Tick all boxes below to demonstrate that you have completed the respective task. If the item does not apply to your this PR check it anyway to make it apparent that there's nothing to do.

[x] All commits contain a DCO Signed-off-by line (we use the DCO GitHub app to enforce this);
[X] Updated LICENSE-3RD-PARTY.md for any added dependencies or vendored components;
[X] Updated documentation as needed for changed code and new or modified features;
[X] Added sufficient tests so that overall code coverage is not reduced.

If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!
Pro Tip 🤓

Read our contribution guide at least once; it will save you a few review cycles!
Your PR will likely not be reviewed until all the above boxes are checked and all automated tests have passed.

PR template adapted from the Python attrs project.

This looks good to me, although I do say I don't speak Portuguese, so if someone is able to review that does it might be good. Thanks for the contribution!

Hi, I am Brazilian and a native speaker of Portuguese and I did a review and approved

Just need rebase and  it should be good to merge

Just rebased the branch. I don't have write permission, so one of you guys will have to merge it for me.


Update pylint 2.17.16

https://pypi.org/project/pylint/2.17.6/
This is the latest stable version.

I'm closing this PR because we're archiving this repository. Please, fell free to fork it.


insufficient_quota for small prompt

I got the error insufficient_quota for this small prompt:
Prompt: "test"
Status code: 429
Error data: {"error":{"message":"You exceeded your current quota, please check your plan and billing details.","type":"insufficient_quota","param":null,"code":null}}

On your openai account you have exceeded your token free tier limit, that's why this error occurred.
For fixing this update your token in plugin or buy premium tier of openai or try hacky way....


ArduinoOSC requires Ethernet library even when only WiFi is used

Stumbled upon this when I tried your library for the first time. Not sure if you consider this a bug, but since Ethernet is not stated as a required dependency, I do :)
steps to reproduce in Arduino IDE:

make sure you have Ethernet lib NOT installed
load arduino/OscWiFi example project
select any board that supports Ethernet (e.g. ESP32)
try to compile

Expected result:
It should compile without errors.
Actual result:
In file included from /home/kartoffel/Arduino/libraries/ArduinoOSC/examples/arduino/OscWiFi/OscWiFi.ino:2:0:
/home/kartoffel/Arduino/libraries/ArduinoOSC/ArduinoOSC.h:54:22: fatal error: Ethernet.h: No such file or directory
compilation terminated.

A possible solution could be to split the header up into
ArduinoOSC_WiFi.h
ArduinoOSC_Ethernet.h
which set some defines and then include
ArduinoOSC.h

Just sharing, this also happened in my environment, where PlatformIO, esp-idf, and Arduino (as a component) is used.
So I tried installing the latest(v2.0.0) Ethernet Library, ending up with a compile error on it.

@superkartoffel @hana Thank you for reporting, fixed in v0.3.22!


Set up application period models

Create a application period class that will define our application periods, probably in a new package model
Blocked by #12
Needs an array of event IDs

Done in #88


Search Tab Opens Empty Search Page

When a user selects a service record from the Search results, then click on the the search icon from the navigation bar, it should open an empty search page. Currently it shows the last search results.
Acceptance Criteria

[ ]  When a user selects a service record from the Search results, then click on the the search icon from the navigation bar, it should open an empty search page.

Steps for QA:

Open the search page
Enter keyword and location, then click Search
Select a service record from the search results
Click the search icon on the navigation bar

[ ] You should see an empty search page, not the previous search result page

@jparlina  I'm not sure about the AC on this story. According to Algolia presentations I have seen (I can dig them up), the search result page should ideally never be empty.

@aron-chen-pg Any thoughts on this user experience?

From looking at other platforms, I agree with @rasmus-storjohann-PG. Ideally, we don't want search result page to be empty other than when users first access the feature per launch. Showing the last search result makes sense to me. One more conventional pattern though would be to simply remove the bottom navigation bar whenever users click on a service. That way the only way to get back is to click on the back arrow on the top of the header bar.

Thanks @aron-chen-pg and @rasmus-storjohann-PG, I'm closing this issue then.


Annoying pieces of shits

Including this one except it's closed for good.


Question on Copyright

If I don't use the mapstyle feature and use my own maps created with mapboxgl js do I need to purchase a plan?  I essentially just want to use the print features of the tool without the mapstyle component (which I can remove on my own).

I'm not entirely sure what you're asking. I didn't write Mapbox GL JS, and I don't know what plans you're referring to (I'm not selling anything). The code I've written (what's in this repository) is MIT licensed, so if you redistribute the code with or without modifications you need to keep the copyright statement with the code. However, the code license only covers the code, not the output produced by it.
The licensing of the output images is governed by the data provider, e.g. Mapbox or OpenMapTiles, depending on the style used. This is where the non-commercial use requirement and attribution requirements come from. You're certainly welcome to use whichever data provider you want.

My mistake.  I incorrectly assumed this was an official Mapbox product and that all maps whether self-generated or otherwise might require some kind of subscription.  By the way thanks for the great plugin.

I have no affiliation with Mapbox. As far as I know, the only location of official Mapbox software is their GitHub account.
When I originally made this tool, there was no easy way to generate print-resolution maps with OSM data, although Mapbox now has an official way of doing so.


SVG content get's truncated when persisted in DB

Look's like the data type should be text.

Hi, can you tell me the icon with which this problem occurred?

Hi, I actually have this problem with all the icons I tried. The issue is slightly different in different environments.
Localhost (Docker compose with mysql:latest for the linux/amd64


Cant establish MariaDB connection without SHOW DATABASE privilege

Description
I connect to a MariaDB database with a user with limited permissions, including the absence of the SHOW DATABASE privilege. In DBeaver, the connection is never established, and the following error is raised.
Access denied; you need (at least one of) the SHOW DATABASES privilege(s) for this operation
Other tools, such as the mariadb cli client, and MySQL Workbench allow connections and queries using the same user. I suspect DBeaver is trying to inspect the connection's Databases to show in the UI and is failing when it cant do so.
Is there an option to allow such a connection without trying to fetch the database list?
DBeaver Version
Community Edition <IP_ADDRESS>403091004
Operating System
macOS Sonoma 14.2.1 (23C71)
Database and driver
SERVER: MariaDB 10.6.12 (managed by https://aws.amazon.com/rds/)
DRIVER: mysql8 (mysql-connector-j-8.2.0.jar)
have also tried mariaDB (mariadb-java-client-2.7.1.jar)
Steps to reproduce

Create a local mariadb server
Create a test db, and a user without the SHOW DATABASES permission
Attempt to start a connection in DBeaver to the local mariadb server with the established user
Fail with:
Access denied; you need (at least one of) the SHOW DATABASES privilege(s) for this operation

Additional context
No response

Hello @seldonPlan
Thanks for the report.
It is not clear why you can't give such permission. Could you please explain it more?

Duplicate of https://github.com/dbeaver/dbeaver/issues/8010

Hello @seldonPlan
Thanks for the report. It is not clear why you can't give such permission. Could you please explain it more?

I'm not the original reporter, but I'm having the same problem.
I'm not the manager for the database server, I only have access to one of the many databases on that server, and I should not be able to list all the databases.
Past versions used to give this error but still would allow for queries.  I believe that would be a correct behavious.

Duplicate of #8010

this might be a different issue. Current version of dbeaver impedes the connection altogether. #8010 only reports of the list of tables to be missing.

Ok, thanks for the explanation.

Hello, we still encounter this issue, is there any development on this ?


Run e2e tests in Circle

This PR was successfully deployed to production by CircleCI build 43 at 2017-01-03 12:43 GMT.


Project build taking too long

Before merging the PR #12, the project building was taking 1 minute.
Now, it's taking 6 minutes!!.
I suspect the following directory C:\projetos\goleador\src\main\webapp\node_modules has something to do with it, because it is too big (~ 200MB) and it has a lot of small files.

The following steps are taking more time:
~110s mvn install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
~80s mvn clean cobertura:cobertura coveralls:report
~70s mvn clean package docker:build
~25s building image
~5s tests
The times above are the difference between the logs.

We can temporary remove the coveralls step, but see the time to build by maven before:
~9s mvn install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
~32s mvn clean package docker:build
~23% of actual time.
There is still something wrong!

Heroku maintains a cache that stores node_modules.
This cache is persisted between builds.
It is an option.
https://devcenter.heroku.com/articles/nodejs-support#cache-behavior

Yarn claims to be much faster and reliable than its predecessor.
Maybe we can start using it instead of npm.
Take a look at this performance comparison.

Ohh my god!
Maybe that's why JHipster chose it.
Living and learning with the opensource world!


intrinsic_matmul/matmul.c:2:10: fatal error: thead_matrix.h: No such file or directory

I run ./env.h and get the error:
intrinsic_matmul/matmul.c:2:10: fatal error: thead_matrix.h: No such file or directory
    2 | #include <thead_matrix.h>
      |          ^~~~~~~~~~~~~~~~
compilation terminated.
make: *** [Makefile:26: intrinsic] Error 1

and I cannot find <thead_matrix.h> header file in the gcc or any other directory.

fixed it with commit [e1a8f33]

the latest toolchain can be found here    https://www.xrvm.cn/community/download?id=4333581795569242112

I found the <thread_matrix.h> header file and compile successfully via the toolchain in the link you provided. Thank you.
BTW, I found that the toolchain works with SPEC version 0.3, but not with version 0.4, because the matrix-scalar instructions were removed in SPEC version 0.4. Is that right?

I found the <thread_matrix.h> header file and compile successfully via the toolchain in the link you provided. Thank you. BTW, I found that the toolchain works with SPEC version 0.3, but not with version 0.4, because the matrix-scalar instructions were removed in SPEC version 0.4. Is that right?

Where is the thread_matrix.h file?

I found the <thread_matrix.h> header file and compile successfully via the toolchain in the link you provided. Thank you. BTW, I found that the toolchain works with SPEC version 0.3, but not with version 0.4, because the matrix-scalar instructions were removed in SPEC version 0.4. Is that right?

Where is the thread_matrix.h file?

In the toolchain provided in the link above.


baresip: different hashes between the github and website releases

Hi Alfred,
I downloaded baresip-0.5.9.tar.gz from http://www.creytiv.com/pub and from the github release section. While there is no difference between the contents, the checksums differ.

6a4b4c0d26932aa9abcbbc2ad401285e  baresip-0.5.9.github.tar.gz
40fae16eca6f5fe3cdd3435b0cb07e99  baresip-0.5.9.www.tar.gz

This isn't very nice. People might get suspicious that something funky is going on only to find out that there's no difference. But it causes unnecessary extra efforts.
;-)
Kind regards,
Seb

Same for re/rem :)

I think the openBSD ports group discovered this recently:
https://marc.info/?l=openbsd-ports&m=151973450514279&w=2

These are not guaranteed to be stable, they can change as github
update software and caches expire (this has happened at some point over
the last few months so we have been seeing a number of hash failures
recently).

it looks like there is a small difference in the header:
-00000000: 1f8b 0800 543b db5a 0003 ec3c 6977 db46  ....T;.Z...<iw.F
+00000000: 1f8b 0800 0000 0000 0003 ec3c 6977 db46  ...........<iw.F

the tarball on the www.creytiv.com server is generated using this command:
$ make git_release

and also the SHA256 hash is generated from that binary.
do you think we should copy the Assets (tarball) from github instead ?
Alfred

Hi Alfred,
If I understand the text you provided correctly, it's possible to upload your own tarball to github, preventing regeneration by github scripts. That sounds like the way to go. You could even sign your releases cryptographically if you wanted.
Kind regards,
Seb

then I see two alternatives:

use github release tarball as the reference, and copy it to www.creytiv.com
as part of the release process.

use www.creytiv.com release tarball as reference, and upload to github.

I think I prefer alternative 1 since it means one less step for me.
current released files will not be changed. the next release of re/rem or baresip
will then follow this scheme.
sounds good @micmac1 ?
Thanks for reporting this BTW.

Am May 12, 2018 9:02:23 PM UTC schrieb "Alfred E. Heggestad"<EMAIL_ADDRESS>
then I see two alternatives:

use github release tarball as the reference, and copy it to
www.creytiv.com
as part of the release process.

use www.creytiv.com release tarball as reference, and upload to
github.

I think I prefer alternative 1 since it means one less step for me.
current released files will not be changed. the next release of re/rem
or baresip
will then follow this scheme.
sounds good @micmac1 ?
Thanks for reporting this BTW.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/alfredh/baresip/issues/388#issuecomment-388582637

The text sounds like github generated release tarballs do not guarantee a stable hash. The hash may change over time. So #2 would be great, as with #1 there is still a chance of differing hashes.

Aha, I get it now.
I uploaded the tarball from www.creytiv.com for baresip version 0.5.9 here:
https://github.com/alfredh/baresip/releases/download/v0.5.9/baresip-0.5.9.tar.gz
could you please test it and see if this is what you want ?
The link to "Source Code (tar.gz)" is still there, though.

Hello Alfred,
Now the checksums are the same. That looks good. Thank you very much!
I think it's normal that the github generated source still shows up. See
this project: https://github.com/fln/addrwatch/releases
It's the same. The author uploaded his/her own release and github
generated another tar.gz. The checksums differ. I don't see a problem
with that, as long as there's one reliable package up on github.
Again, thank you!
Kind regards,
Seb

I have also uploaded tarballs for re/rem:
https://github.com/creytiv/re/releases/tag/v0.5.8
https://github.com/creytiv/rem/releases/tag/v0.5.3
so I will change the release procedure a bit then; after the version
is released and tagged, I will generate a tarball and upload it to
www.creytiv.com and github.
older releases will just stay as they are.

hi @micmac1 okay to close this one now ?

Am May 13, 2018 12:34:14 PM UTC schrieb "Alfred E. Heggestad"<EMAIL_ADDRESS>
hi @micmac1 okay to close this one now ?
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/alfredh/baresip/issues/388#issuecomment-388623646

Hi Alfred,
Sure, all OK. Have a good Sunday!
Thanks,
Seb


minor fixes on unix environment

Hello,
I'm not PHP developer, I'm only able to modify and use foreign code :-) For this reason I will not send pull request :-)
There were problems with carriage return "%0D" char in $appid and $appsecret (probably caused by windows CRLF vs unix LF?)
url_https_post curl send body:grant_type=client_credentials&client_secret=blabla123blabla%0D&client_id=123456%0D
Solution in is trim() before usage.
Without this fix, the refresh token failed, result was null
Second problem:
it does not worked, it was not able to find and create instance of class TestPushMsgCommon()
include_once (dirname(__FILE__) . './push_common/test_sample_push_msg_common.php');
it works after change to:
include_once (dirname(__FILE__) . '/push_common/test_sample_push_msg_common.php');
environment:
Linux & PHP 5.2, macOS & PHP 7.1

Hello, we have updated the code  b243ffc to fix this issue,  please try again.


transmitting bluetooth 5 on Raspberry Pi 4

Hi there
I have tried this on my Raspberry Pi 4 with ubuntu 20.04
when I do sudo ./transmit 5 p I got
Command 0xA returned error 0xC
Command 0x39 returned error 0x1
Supported Low Energy Bluetooth features:
        Features: 0x3f 0x00 0x00 0x08 0x00 0x00 0x00 0x00
          LE Encryption
          Connection Parameter Request Procedure
          Extended Reject Indication
          Slave-initiated Features Exchange
          LE Ping
          LE Data Packet Length Extension
          Remote Public Key Validation
Command 0x36 returned error 0x1
The transmit power is set to 0 dBm
Command 0x35 returned error 0x1
Command 0x39 returned error 0x1
Command 0x37 returned error 0x1
Command 0x37 returned error 0x1
Command 0x37 returned error 0x1
Command 0x37 returned error 0x1
Command 0x37 returned error 0x1
Command 0x37 returned error 0x1
Command 0x37 returned error 0x1

are these errors typical?  is the BT5 Long Range + Extended Advertisements transmitting normally?
thanks

Readme says
"When tested on a Raspberry Pi 3B and 4B, the Extended Advertising HCI interface commands are not supported (option 4 and option 5 p). Probably due to the HW/driver only supporting BT4 Legacy Advertising. The older BT4 Legacy non-Extended Advertising HCI commands (option l) worked okay."
try with a transmit option other than 5 p

Oh, I see.
Can I use some USB BT5 dongles to send the message? do you have any successful cases using USB BT dongles? or a list of BT5 modules that are working well?

I have not tried any Bluetooth dongles. I don't see why that should not work though.
If you manage to find a suitable dongle that supports BT5 etc. and you can confirm that it works, please do drop a note here. We can update the documentation then.

I bought a BT5 USB adapter from Amazon. The brand is ONVIAN and the chipset is RTL8761B. This time I use a PC with Ubuntu 20.04
I received the following output from the terminal:
❯ sudo ./transmit 5 p
Supported Low Energy Bluetooth features:
        Features: 0xbd 0x5f 0x66 0x00 0x00 0x00 0x00 0x00
          LE Encryption
          Extended Reject Indication
          Slave-initiated Features Exchange
          LE Ping
          LE Data Packet Length Extension
          Extended Scanner Filter Policies
          LE 2M PHY
          Stable Modulation Index - Transmitter
          Stable Modulation Index - Receiver
          LE Coded PHY
          LE Extended Advertising
          Channel Selection Algorithm #2
          Connection CTE Request
          Connection CTE Response
          Antenna Switching During CTE Transmission (AoD)
          Antenna Switching During CTE Reception (AoA)
The transmit power is set to 0 dBm
Received unknown event: 0x3E
Received event with invalid opcode 0x200A. Expected 0x2039
Received event with invalid opcode 0x2039. Expected 0x203C

It looks working normall, but not sure about the meaning of invalid opcode.
How can I verify it is indeed transmitting the OpenDroneID?
Using a andriod receiver?
Sadly, the opendroneid-wireshark-dissector doesnot support BT5 yet.

additional testing results:
I have installed the OpenDroneID app on a Google Pixel 6a. it can detect both 4 and l but cannot get anything for option 5 p
it supports BT5 long-range and extended advertising, but I cannot confirm if the BT adaptor has an issue or the phone has an issue.

Probably the transmitter works as intended but I am not sure what those error codes are. I don't have the PC I originally tested the BT5 Long Range mode on anymore, so I can't dig into this.
There is an incomplete list of Android phones available and at least the Pixel 6 has been marked to not support BT5 Long Range.
Maybe you could try to check if any of your friends/family have one of the phones on the list that is known to support receiving BT5 Long Range and ask them to install the receiver app and test against the Bluetooth dongle transmitter?
It is actually strange that the Pixel 6 doesn't support receiving the signals, since the Android feature flags does claim that it supports both the Long Range and the Extended Advertising modes. Possibly the driver SW doesn't listen to primary and/or secondary advertisements on the Coded PHY but only on the regular 1 Mbit PHY.

tested with a Samsung Galaxy A21s and Oneplus 7(T?), both can detect BT5.
Great work, thanks.


version and device_class return wrong values

Whenever I try to get version and device class, I receive values which differ from those produced by ideviceinfo.
 let device = idevice::get_first_device().unwrap();
 info!("Got device: {:?}", device);

This returns
Got device: Device { udid: 00008020-001D2CD90128002E, network: false, ip_address: None, device_class: 5, version: 0 }

I believe 5 is Unknown, and version is 0.
ideviceinfo -k ProductVersion returns 13.3.1
Some other metrics:
ProductName: iPhone OS
ProductType: iPhone11,8
ProductVersion: 13.3.1
ProductionSOC: true
ProtocolVersion: 2

In addition, is version the iOS version?
Thanks!

So this is just from memory, I’ll be sure to get a definite answer later.
These values are returned by usbmuxd, and are inferred from the data the USB controller detects from it. The device class definitely seems to be wrong, as 5 means Apple Watch. I don’t remember what version means atm, but it’s not the iOS version. You can see how to fetch the iOS version in the ideviceinfo example here in the repo.


Add initial JDK20 packaging with default versions

Fixes #626
This PR adds the initial packaging for JREs & JDKs for version 20. The version numbers in the template files will be updated during the release ( as is typical ), place holder version of <IP_ADDRESS>.0+1 has been used.

Fine to have as placeholder and fix with correct release version later, and if really want, can even do dry-run for installer build

we might need a big rebase from https://github.com/adoptium/installer/pull/625/files if it gets merged first.


Transaction notifications

Would be great to receive notifications of incoming transactions

As an MVP, in app notification would already be a great improvement.
It's annoying when I show a QR code to receive money that I have to close it to see if the money has been received.
Also, sound notification is important. So that a vendor can just hear the confirmation.

Ended up doing both:

in-app notifications when receiving
push notifications


Missed translateZ support?

I want to make 3D animations like shown here:
https://desandro.github.io/3dtransforms/examples/cube-02-show-sides.html
I've tried translate(0,0,100) but it doesn't work.

Hey and thanks for reporting this!
There's an awesome place to ask question like this one: StackOverflow. It's the best system for Q&A. Many people from the community hang out there and will be able to see your question, you can vote on answers and mark question as answered etc. This lets us keep a list of bug reports and feature requests on github and especially Product Pains (again, with voting which is really nice).
I'm posting this here because github issues haven't been working very well for us and because StackOverflow is so much better. Thanks for reading! :)

Nobody answered my question there.
Pavel.

2 марта 2016 г., в 21:27, Satyajit Sahoo<EMAIL_ADDRESS>написал(а):
Hey and thanks for reporting this!
There's an awesome place to ask question like this one: StackOverflow. It's the best system for Q&A. Many people from the community hang out there and will be able to see your question, you can vote on answers and mark question as answered etc. This lets us keep a list of bug reports and feature requests on github and especially Product Pains (again, with voting which is really nice).
I'm posting this here because github issues haven't been working very well for us and because StackOverflow is so much better. Thanks for reading! :)
—
Reply to this email directly or view it on GitHub.

@aksonov Try asking in #reactiflux or discord channel may be?

This likely doesn't work because the translationZ property is ignored on Android. See here.
The "perspective" property also isn't yet implemented even though there is a pull request to implement it for quiet some time now.


Fix the COC!

Update the version to: https://www.contributor-covenant.org/version/2/1/code_of_conduct/
Contact method:

Hmm, if this goes stale once more after this message, I'll be working on this.

lol ok 👍
TBH I thought we were going to remove the stale bots too?

That will be so great 🙌


Localized resources from Razor component library not working in Blazor project

Strings from resources files (.resx) in a component library are not localized correctly when used by a Blazor project.
But they are ok when used by a Razor component project.
You can download the attached solution (WebApplication.zip) to reproduce this issue:

Run the "Blazor.Server" project
Go to /fetchdata page. The page content should be translated to Spanish
On the other side, if you run the "RazorComponent" project, the /fetchdata page is correctly translated to Spanish

WebApplication.zip

Thanks for contacting us, @gustavnavar.
This will be addressed as part of https://github.com/aspnet/AspNetCore/issues/5471.


add back weight for pod antiaffnity.

Resolve: https://github.com/istio/istio/issues/14034

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: morvencao
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [morvencao]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/assign @linsun

@morvencao: GitHub didn't allow me to assign the following users: linsun.
Note that only istio members and repo collaborators can be assigned.
For more information please see the contributor guide

In response to this:

/assign @linsun

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

@morvencao: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
0ebd2d6390bed28e72c8db5f0e7dbf088f7923ae
link
/test istio-pilot-multicluster-e2e

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: morvencao, sdake
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [morvencao,sdake]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/assign @duderino

@linsun: changing LGTM is restricted to assignees, and assigning you to the PR failed.

In response to this:

/lgtm
/approve

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: linsun, morvencao, sdake
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [linsun,morvencao,sdake]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@morvencao is this also needed for master and release-1.2?

@linsun This has been merged to new installer, not sure if we need to create a PR for release-1.2 in main repo here.

@morvencao I think so.  FYI, the TOC has decided that the old installer will be the default installer for release 1.2 so that we can collect more feedback on the new installer before we promote it to stable.  (sorry if I have ever communicated that to you but it just happened a week ago).

@linsun Got it. Have cherry pick the commit to release-1.2: https://github.com/istio/istio/pull/14418

At this point I'd like to keep all changes out of release-1.1 unless it's really clear that they address an important issue.  release-1.2 will be out in ~3 weeks so I think you should try to get this change merged into that.
Please reopen and argue if I have misunderstood the importance of this change.


How does this work in a larger app?

I have basic knowledge of Rx and some experience with Flux. Though I don't quite enjoy the tone of your presentation, I find the way you compose things in Cycle very interesting and would like to learn more.
My question is how a larger application would be structured. (I guess, a port of FB Flux chat example to Cycle would be very helpful.)
For example, if there are ten components that want to "follow user", is it correct that I would:

Inject them all into FollowUserIntent
Have them all export followUserClick$ with the same name so same intent works for them?

Or would I rather have different intents, each for a specific component, but have FollowUserIntent subscribe to these intents?

What happened to the FAQ that was linked to above?

Most questions and answers in the FAQ got deprecated as the framework evolved, and the useful Q&As got incorporated into the docs site as gray boxes placed in the middle of text, in proper context. @neverfox


Generated ID field is detected as a date field

Should be a text field so that Palladio doesn't try to treat it as a date or numbrer.

Fixed in 1.2.9


Support vectorized numpy operations

Example
seq 10 | pyline -N 'd**2'
seq 100 | pyline -N 'sum(d)'
seq 100 | pyline -N 'sum(d)'

This will likely be contentious:

numpy dependecy, interaction of numpy with virtualenvs, testing if we decide to only import numpy when used
Different mode of functionality: data has to be read ahead of time to work with numpy, this is potentially confusing since the behaviour is different with and without the numpy option
To be useable one might end up adding magic (special cases for different types of data), this may offend your sensibilties.

That said, all things being equal, the world might only want one command line tool to quickly run python from the command line.

See https://github.com/talwrii/pyline/tree/talwrii--numpy--2016-09-21--WIP--issue-27 for a reference implementation.
I wanted these features to exist, so I coded them; if this is deemed an ill fit for pyline then I can pull this into a separate tool without too much work.
Note that this is a work in progress. There is an unresolved question of the numpy dependency:

Are we happy for this tool to have a numpy dependency?
If not are we happy to have certain functionality to have a numpy dependency? If so how should tests work.

Opinions?

Are we happy for this tool to have a numpy dependency?

You could add it as an extras_require in setup.py; though I think expecting users to have numpy installed is a fair assumption.

If not are we happy to have certain functionality to have a numpy dependency? If so how should tests work.

try:
    import numpy as np
except ImportError:
    np = None

# ...

skipif(np is None, "NumPy is not installed")

unittest has skipif: https://docs.python.org/2/library/unittest.html#skipping-tests-and-expected-failures
pytest has skipif: http://doc.pytest.org/en/latest/skipping.html#id1
nose appears to not work w/ unittest.skipif: http://stackoverflow.com/questions/21936292/conditional-skip-testcase-decorator-in-nosetests

So, probably pytest
import pytest
@unittest.skipif(np is None,
                    reason= "NumPy is not installed")
def testfuncname():
    # ...

I'm sort of partial to using full imports in the command (for the sake of making it easier to copy and paste to a program), so, personally, I'd prefer e.g. np.sum() over just sum(); but there may be a justifiable reason to be sage-y?

(I am open to a (first!) PR)

I wanted these features to exist, so I coded them; if this is deemed an ill fit for pyline then I can pull this into a separate tool without too much work.

In terms of scope; IDK about other functions which consume the whole file

You could add it as an extras_require in setup.py; though I think expecting users to have numpy installed is a fair assumption.

If there's an ImportError in the pyline function, $ pyline should return a nonzero error code

Cool cool. It seems like you're open to the idea of this feature.
Numpy dependency while running
No one's going to use the extra_require :) . Yep just erroring out if numpy is missing and --numpy is used seems reasonable. Perhaps together with instructions to install numpy.
Numpy dependency while testing
So my concern about skipping tests is that it's an invitation for tests to not be run when
they need to be. There appears to be no way to automatically have different requirements for python setup.py test. test_requires would appear to exist solely to mislead! (http://stackoverflow.com/questions/9607565/how-do-i-force-setup-py-test-to-install-dependencies-into-my-virtualenv).
Some options:

Just have the tests fail if numpy is not installed (people will be able to work out how to fix these easily enough)
Do the above and add a special extra_requires for testing. (but people probably won't remember to use it)
Suck it up and add a numpy dependency (But when you add a numpy dependency to a virtualenv numpy gets built from source, which takes ages...)

Continuous integration
There seems to be some sort of continuous integration (CI) for this, but a brief inspection of the source code failed to tell me how this worked. Is there anything I should bear in mind here?
What numpy things to put into the namespace
Some comments

One could still refer to numpy.func even if we flood the namespace with the contents of numpy. But the argument is presumably to force / encourage to make a decision that results in more reusable code.
I don't know how confusing the alias np is for people new to numpy, though it does seems standard in a number of examples
I feel that long function names in formulas can make them difficult to read. Consider: numpy.sqrt(numpy.sum(d[,0]**2)) versus sqrt(sum(d[,0]**2)). But np.sqrt(np.sum(d[,0])) isn't too bad.

Opinions? I don't have strong opinions other writing out numpy.blah is a little wordy.

In terms of scope; IDK about other functions which consume the whole file

Clarification plz

In terms of scope; IDK about other functions which consume the whole file

More details please!

As-is, pyline iterates through the input without reading the whole file into RAM (thus avoiding issues with memory consumption).
The proposed changes build a list for each line of the whole input file (consumes the whole file) and then copies those into a numpy array.
I'm hesitant to expand the scope of this utility; though I do recognize the usefulness of a --numpy option.

[CI]

https://github.com/audreyr/cookiecutter-pypackage

https://github.com/westurner/pyline/blob/master/.travis.yml
https://github.com/westurner/pyline/blob/master/tox.ini

[long function names]

import numpy; np = numpy
from numpy import * could potentially cause namespace collision

numpy dependency

As-is, pyline requires zero dependencies. The pyline.py file can be copied to sys.path and runs fine.
There is obviously a tradeoff between {zero dependencies, fast-tests} and {third-party library features}.

(TBH, I don't need this functionality)
A fork/workalike would be cool
Otherwise

The optparse help text for --numpy should carry a warning like NOTE: This method consumes the whole input file
travis.yml could run a different tox env which includes numpy

Because I need/want to be able to run the tests without installing numpy

(Though I recognize that conda install numpy is really easy)

TBH, I don't need this functionality

Cool cool. You can have it if you want though :) (the "otherwise" approach seems reasonable).
Okay, unless you tell me otherwise I'm going to. Extract out a a tool called npcli which

Runs like npcli sum(d), possibly doing magic things to work out whether it needs input.
References this tool and includes you as a second author (feel free to request otherwise), but clearly not a maintainer. ( I wonder if this counts as endorsing according to the license)
Uses the same BSD license

If it becomes apparent that other people want this feature then I imagine we can easily finish off this branch and merge.

Cool cool. You can have it if you want though :) (the "otherwise" approach seems reasonable).

:) cool

Pyed piper may also be worth a look. It has some fancy/magical command parsing.

https://code.google.com/archive/p/pyp/

... https://github.com/westurner/pyline/issues/27#issuecomment-250038384

[ ] there could be a requirements-numpy.txt (with a -r requirements.txt) here https://github.com/westurner/pyline/blob/master/requirements.txt

[ ] install: pip install requirements.numpy here https://github.com/westurner/pyline/blob/master/.travis.yml#L12

I suppose, since the tests already require third party packages, requiring numpy for tests and --numpy would be fine

AFAIK, there's still not yet a numpy/__main__.py (for python -m numpy or python -m numpy.cli.
Now that I've added other optional dependencies to pyline, it's unlikely that it would be accepted for inclusion as a numpy CLI module


Role permission not working after update to 3.7.1

Hi all,
After upgraded to 3.7.1, the role permission is no longer working.

I need to explicitly Grant permission in Edit Permission :

Is this a bug or do I need to make any changes to make it work again?
Thanks.

Hi volkan,
Thanks for your quick reply.
Client side permission checking seems to behave differently,
        if (!Authorization.hasPermission("Core:Toc:Modify")) {
            this.toolbar.findButton('.add-button').addClass('disabled');
        }

it was working in older version where I only check the Role permission to users.
The client side hasPermission() seems not working with Role permissions anymore.  It will work only if I grant the permission again in Edit Permission.
Thanks.


Very Straight Unnatural Rivers

Found rivers that have very long straight sections.

This will be fixed in next version. I improved mountains that made it worse so had to fix it now.


FilterDialog: resizing the picklist causes the list to scroll to the far right bottom

Firefox is not affected, chrome is.
Also, it's not the resizing perse, doing a mousedown on the resizer widget trigges this behavior


Disable the submit button if required form fields are not filled out

Details
See issue for more context.
Fixed Issues
$ https://github.com/Expensify/Expensify/issues/171673
Tests/QA

Navigate to the company step in the VBA flow: <baseURL>/bank-account/company.
These are the fields that need to be filled out in order for the "Save & Continue" button to be enabled. Verify that the button is disabled if any of these fields aren't filled out. You don't need to input a valid value, any value will suffice as long as it isn't empty:

Legal Business Name
Company Address
City (for company address)
State (for company address)
Zip Code (for company address)
Company Website
Tax ID Number
Incorporation Date
Industry Classification Code
Expensify Password

Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[x] iOS
[ ] Android

Screenshots
N/A

@madmax330 Thanks so much for the review! I reverted the commit that removed the duplicate checks, should be good for another look!

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging         in version: 1.0.81-5🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production         in version: 1.0.82-7🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Support abstract fields in patterns

I have a PatternA with several ents using it, then I have PatternB with another set of ents using it. I want a fieldEdge from PatternB to PatternA, but there is a one-to-one mapping between which ent each edge will go to. So EntA1 (uses PatternA) will always match up with EntB1 (uses PatternB). I want a way to specify in the pattern that a field should be implemented in each Ent (ideally I can even specify some fields and the ent would use the union of the pattern and the ent schema.
So maybe an API like:
  // Inside PatternB
  abstractFields: {
    entFromPatternAID: UUIDType({
      fieldEdge: {
        // schema: <<This part would be filled in by each Ent>>,
        inverseEdge: {
          name: 'patternBs',
        },
      },
    }),
  }

Maybe that's a little bit of a pipe dream, as I'd still be happy with something like:
abstractFields: {
  entFromPatternAID: typeof Return<UUIDType>,
}

I should also state that I'd like to use that abstract field in a constraint on the pattern.


latest release broken

ERROR Autofac.Core.Registration.ComponentNotRegisteredException: The requested service 'Trash.CreateConfig.CreateConfigCommand' has not been registered. To avoid this exception, either register a component to provide the service, check for service registration using IsRegistered(), or use the ResolveOptional() method to resolve an optional dependency.   at Autofac.ResolutionExtensions.ResolveService(IComponentContext context, Service service, IEnumerable1 parameters)
at Autofac.ResolutionExtensions.Resolve(IComponentContext context, Type serviceType)
at Trash.Command.CliTypeActivator.ResolveType(IContainer container, Type typeToResolve)
at Trash.Program.<>c.b__1_0(Type type)
at CliFx.Infrastructure.DelegateTypeActivator.CreateInstance(Type type)
at CliFx.CliApplication.RunAsync(ApplicationSchema applicationSchema, CommandInput commandInput)
at CliFx.CliApplication.RunAsync(IReadOnlyList1 commandLineArguments, IReadOnlyDictionary2 environmentVariables)
`
Fix: use release previously

Same on Mac and Windows release

Thank you for the report and I apologize for overlooking this. This is resolved in v1.3.2.

wait hold on you're doing us a favor don't apologize. this software is awesome.


Fix enumerating through standalone lists whose objects have list prop…

…erties
Closes #1906.
\c @bdash

:+1:

:+1:


Fixed incorrect mm_sfence define

Was using an incorrect post-fix "_ps"

Codecov Report

Merging #139 into master will increase coverage by 6.22%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #139      +/-   ##
==========================================
+ Coverage   67.95%   74.18%   +6.22%
==========================================
  Files          34       34
  Lines        8185     7716     -469
==========================================
+ Hits         5562     5724     +162
+ Misses       2623     1992     -631

Impacted Files
Coverage Δ

simde/x86/sse.h
74.39% <ø> (-5.79%)
:arrow_down:

simde/x86/sse4.1.h
1.57% <0.00%> (-81.88%)
:arrow_down:

simde/x86/sse2.h
78.82% <0.00%> (-9.32%)
:arrow_down:

simde/x86/ssse3.h
81.34% <0.00%> (-0.74%)
:arrow_down:

simde/x86/mmx.h
67.20% <0.00%> (+1.99%)
:arrow_up:

simde/x86/avx512f.h
75.90% <0.00%> (+2.73%)
:arrow_up:

simde/arm/neon/int64x1.h
61.53% <0.00%> (+3.47%)
:arrow_up:

simde/arm/neon/uint64x1.h
61.53% <0.00%> (+3.47%)
:arrow_up:

simde/arm/neon/int64x2.h
62.96% <0.00%> (+3.58%)
:arrow_up:

... and 21 more

Continue to review full report at Codecov.

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

Merged, thanks!


Bug: Issue with file preview in column layout

Description
When in column view mode with the preview pane enabled, clicking on the first file displays its characteristics, folder logo, etc. However, when I click on the next file or folder, nothing appears and I have to click on another folder or in the blank space for the preview to display properly again.
Steps To Reproduce
Open "Files"
Choose a test category, such as "Documents"
Set the view to "Columns"
Enable the preview pane
Click on a folder, and then on a second one
Requirements

Fix the issue with the preview not displaying correctly when switching from one file/folder to another in column view mode.
Ensure that the preview displays smoothly and successfully for each selected file/folder without any interruption.

Files Version
Version : <IP_ADDRESS>
Windows Version
Windows 11 22H2 22621.1635
Log file
No need.

Thanks for the report, I was able to reproduce this.

@Lukiluc29 Just to make sure I am reproducing the issue correctly based on your description:

Open "Files"
Select "Documents"
Change View Layout to "Columns"
Enable Preview Panel
Click any items in the first level, the detail shows up correctly in the preview.
Select a folder and click on an item at the second level, the preview is not rendered. Select a different item or lose focus of the current selection and reselect the item will cause it to render correctly in the preview.

Does this match the issue you observed?
On a separated note, if you are at a lower-level (e.g. subfolder) and you try to select any folder on the upper level, the preview is also not rendered unless you select a different item or refocus the item. Is this the intended behavior?

This issue will be fixed in v2.5.


flag: ContinueOnError is not continuing

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?
go1.4.2 linux/amd64

What operating system and processor architecture are you using (go env)?

GOARCH="amd64"
GOBIN=""
GOCHAR="6"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="~/gocode"
GORACE=""
GOROOT="~/code/go"
GOTOOLDIR="~/code/go/pkg/tool/linux_amd64"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"

What did you do?

fs := flag.NewFlagSet("d", flag.ContinueOnError)
boolPtr := fs.Bool("fork", false, "a bool")
_ = fs.Parse([]string{"-unknown", "--fork"})

https://play.golang.org/p/93DWNF-53z

What did you expect to see?
fork: true because ContinueOnError is used

What did you see instead?

flag provided but not defined: -unknown
Usage of d:
  -fork
    	a bool
fork: false

I didn't expect to see Usage though

flag.ContinueOnError says "Return a descriptive error" [1]
Additionally, the 2nd parameter to fs.Bool() is the default value to use. Sinceyou set it to false, that will be the value of *boolPtr [2]
1: https://golang.org/src/flag/flag.go?s=6739:6761#L259
2: https://play.golang.org/p/7QJz9r6lo_

I think this is working as intended.  Certainly we are not going to change the behavior.  If you have a specific suggestion for how to improve the docs, please let us know.  Closing.


Bugfix/6.3/tcomp 333 enforcer improvement

Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[x] Tests for the changes have been added (for bug fixes / features)

What is the current behavior? (You can also link to an open issue here)
Components tests use di classes
What is the new behavior?
Components tests don't use di classes
Does this PR introduce a breaking change?

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:
Related to https://jira.talendforge.org/browse/TCOMP-333

Looks good -- I'm not sure why we were using the DI enforcer in these tests, so this is a good idea in every way!

Thanks for review

Pull request test result: success :star:

please merge it.


renaming node_modules directory with "javascript.updateImportsOnFileMove.enabled" option set to "always" causes crash

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.62.0
OS Version: Manjaro Linux

Steps to Reproduce:

set javascript.updateImportsOnFileMove.enabled to always
install and import multiple dependencies
change the directory name of the "node_modules" to anything else

When doing so VSCode attempts to update imports of many of the files inside the node_modules and if you have too many dependencies the amount of files it attempts to update is too high and causes the program to stop responding.
The worst part is that simply killing the application and starting it again doesn't necessarily fix the problem, in my case all of the updated files have opened in the editor as unsaved and persisted between closing and opening the application causing it to crash every time I opened it.
Perhaps the auto import update should never happen for any files in the node_modules, regardless of  the user settings?

Yes I think we should never try to update imports under node_modules (although someone on the TS team will have to confirm that there aren't weird edge cases around this)


H5082 dual smart plug

Govee Device SKU
H5082
Govee2MQTT Version
2024.01.13-b7277f05
Describe the issue
The device allows for independent control of each outlet but this govee2mqtt only pulls in a master switch that toggles both plugs together.
If I'm understanding the below logs correctly, this is a limitation of the API but I wanted to confirm. If that is the case, please let me know and then it's cool if you close this.
Startup Diagnostics
N/A
Additional Logs
{
  "meta": {
    "capabilities": [
      {
        "alarmType": null,
        "eventState": null,
        "instance": "powerSwitch",
        "parameters": {
          "dataType": "ENUM",
          "options": [
            {
              "name": "on",
              "value": 1
            },
            {
              "name": "off",
              "value": 0
            }
          ]
        },
        "type": "devices.capabilities.on_off"
      }
    ],
    "device": "2F:6C:60:74:F4:1F:37:D8",
    "deviceName": "Office Smart Plug",
    "sku": "H5082",
    "type": "devices.types.socket"
  },
  "state": {
    "capabilities": [
      {
        "instance": "online",
        "state": {
          "value": true
        },
        "type": "devices.capabilities.online"
      },
      {
        "instance": "powerSwitch",
        "state": {
          "value": 0
        },
        "type": "devices.capabilities.on_off"
      }
    ],
    "device": "2F:6C:60:74:F4:1F:37:D8",
    "sku": "H5082"
  }
}

Home Assistant Logs
No response
Anything else?
No response

Yeah, the platform API only provides one switch for this device.
It's possible that reverse engineering the IoT packets sent by the app will unlock individual control.
Please set the debug level to govee::service::iot=trace,info and capture and share the iot packets that are received when you activate the individual outlets.

  It looks like the on_off value in the packet isn't a binary value, but rather a range of 0-3.  It appears to just correspond to a 2 bit binary representation of the plug.  i.e....
00 = 0
01 = 1
10 = 2
11 = 3
Both Off (00)

[2024-05-29T15:05:45 TRACE govee::service::iot] GA/290d4e642f2e43efe3c3f1676384332d -> {"proType":2,"sku":"H5082","device":"60:DF:60:74:F4:B8:04:62","softVersion":"1.00.21","wifiSoftVersion":"1.00.21","cmd":"status","type":0,"transaction":"u_1717009545733","pactType":1,"pactCode":1,"state":{"onOff":0,"result":1},"op":{"command":["qhIAAAAAAAAAAAAAAAAAAAAAALg=","qrQAAAAAgAAAAIAAAACAAAAAgB4=","qrAAAQAAAAAAAAAAAAAAAAAAABs=","qrAAAAAAAAAAAAAAAAAAAAAAABo=","qhIBAAAAAAAAAAAAAAAAAAAAALk=","qrQBAAAAgAAAAIAAAACAAAAAgB8=","qrABAQAAAAAAAAAAAAAAAAAAABo=","qrABAAAAAAAAAAAAAAAAAAAAABs="]}}
[2024-05-29T15:05:45 DEBUG govee::service::iot] Packet { sku: Some("H5082"), device: Some("60:DF:60:74:F4:B8:04:62"), cmd: Some("status"), msg: None, state: StateUpdate { on_off: Some(0), brightness: None, color: None, color_temperature_kelvin: None, sku: None, device: None }, op: Some(OpData { command: [Base64HexBytes([AA, 12, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, B8]), Base64HexBytes([AA, B4, 00, 00, 00, 00, 80, 00, 00, 00, 80, 00, 00, 00, 80, 00, 00, 00, 80, 1E]), Base64HexBytes([AA, B0, 00, 01, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 1B]), Base64HexBytes([AA, B0, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 1A]), Base64HexBytes([AA, 12, 01, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, B9]), Base64HexBytes([AA, B4, 01, 00, 00, 00, 80, 00, 00, 00, 80, 00, 00, 00, 80, 00, 00, 00, 80, 1F]), Base64HexBytes([AA, B0, 01, 01, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 1A]), Base64HexBytes([AA, B0, 01, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 1B])], mode_value: [], sleep_value: [], wakeup_value: [], timer_value: [] }) }

Right Side On (01)

[2024-05-29T15:06:13 TRACE govee::service::iot] GA/290d4e642f2e43efe3c3f1676384332d -> {"proType":2,"sku":"H5082","device":"60:DF:60:74:F4:B8:04:62","softVersion":"1.00.21","wifiSoftVersion":"1.00.21","cmd":"status","type":0,"transaction":"x_1717009572489","pactType":1,"pactCode":1,"state":{"onOff":1,"result":1},"op":{"command":["qhIAAAAAAAAAAAAAAAAAAAAAALg=","qrQAAAAAgAAAAIAAAACAAAAAgB4=","qrAAAQAAAAAAAAAAAAAAAAAAABs=","qrAAAAAAAAAAAAAAAAAAAAAAABo=","qhIBAAAAAAAAAAAAAAAAAAAAALk=","qrQBAAAAgAAAAIAAAACAAAAAgB8=","qrABAQAAAAAAAAAAAAAAAAAAABo=","qrABAAAAAAAAAAAAAAAAAAAAABs="]}}
[2024-05-29T15:06:13 DEBUG govee::service::iot] Packet { sku: Some("H5082"), device: Some("60:DF:60:74:F4:B8:04:62"), cmd: Some("status"), msg: None, state: StateUpdate { on_off: Some(1), brightness: None, color: None, color_temperature_kelvin: None, sku: None, device: None }, op: Some(OpData { command: [Base64HexBytes([AA, 12, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, B8]), Base64HexBytes([AA, B4, 00, 00, 00, 00, 80, 00, 00, 00, 80, 00, 00, 00, 80, 00, 00, 00, 80, 1E]), Base64HexBytes([AA, B0, 00, 01, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 1B]), Base64HexBytes([AA, B0, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 1A]), Base64HexBytes([AA, 12, 01, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, B9]), Base64HexBytes([AA, B4, 01, 00, 00, 00, 80, 00, 00, 00, 80, 00, 00, 00, 80, 00, 00, 00, 80, 1F]), Base64HexBytes([AA, B0, 01, 01, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 1A]), Base64HexBytes([AA, B0, 01, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 1B])], mode_value: [], sleep_value: [], wakeup_value: [], timer_value: [] }) }

Left Side On (10)

[2024-05-29T15:05:14 TRACE govee::service::iot] GA/290d4e642f2e43efe3c3f1676384332d -> {"proType":2,"sku":"H5082","device":"60:DF:60:74:F4:B8:04:62","softVersion":"1.00.21","wifiSoftVersion":"1.00.21","cmd":"status","type":0,"transaction":"u_1717009515357","pactType":1,"pactCode":1,"state":{"onOff":2,"result":1},"op":{"command":["qhIAAAAAAAAAAAAAAAAAAAAAALg=","qrQAAAAAgAAAAIAAAACAAAAAgB4=","qrAAAQAAAAAAAAAAAAAAAAAAABs=","qrAAAAAAAAAAAAAAAAAAAAAAABo=","qhIBAAAAAAAAAAAAAAAAAAAAALk=","qrQBAAAAgAAAAIAAAACAAAAAgB8=","qrABAQAAAAAAAAAAAAAAAAAAABo=","qrABAAAAAAAAAAAAAAAAAAAAABs="]}}
[2024-05-29T15:05:14 DEBUG govee::service::iot] Packet { sku: Some("H5082"), device: Some("60:DF:60:74:F4:B8:04:62"), cmd: Some("status"), msg: None, state: StateUpdate { on_off: Some(2), brightness: None, color: None, color_temperature_kelvin: None, sku: None, device: None }, op: Some(OpData { command: [Base64HexBytes([AA, 12, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, B8]), Base64HexBytes([AA, B4, 00, 00, 00, 00, 80, 00, 00, 00, 80, 00, 00, 00, 80, 00, 00, 00, 80, 1E]), Base64HexBytes([AA, B0, 00, 01, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 1B]), Base64HexBytes([AA, B0, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 1A]), Base64HexBytes([AA, 12, 01, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, B9]), Base64HexBytes([AA, B4, 01, 00, 00, 00, 80, 00, 00, 00, 80, 00, 00, 00, 80, 00, 00, 00, 80, 1F]), Base64HexBytes([AA, B0, 01, 01, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 1A]), Base64HexBytes([AA, B0, 01, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 1B])], mode_value: [], sleep_value: [], wakeup_value: [], timer_value: [] }) }

Both Sides On (11)

[2024-05-29T15:06:39 TRACE govee::service::iot] GA/290d4e642f2e43efe3c3f1676384332d -> {"proType":2,"sku":"H5082","device":"60:DF:60:74:F4:B8:04:62","softVersion":"1.00.21","wifiSoftVersion":"1.00.21","cmd":"status","type":0,"transaction":"u_1717009600449","pactType":1,"pactCode":1,"state":{"onOff":3,"result":1},"op":{"command":["qhIAAAAAAAAAAAAAAAAAAAAAALg=","qrQAAAAAgAAAAIAAAACAAAAAgB4=","qrAAAQAAAAAAAAAAAAAAAAAAABs=","qrAAAAAAAAAAAAAAAAAAAAAAABo=","qhIBAAAAAAAAAAAAAAAAAAAAALk=","qrQBAAAAgAAAAIAAAACAAAAAgB8=","qrABAQAAAAAAAAAAAAAAAAAAABo=","qrABAAAAAAAAAAAAAAAAAAAAABs="]}}
[2024-05-29T15:06:39 DEBUG govee::service::iot] Packet { sku: Some("H5082"), device: Some("60:DF:60:74:F4:B8:04:62"), cmd: Some("status"), msg: None, state: StateUpdate { on_off: Some(3), brightness: None, color: None, color_temperature_kelvin: None, sku: None, device: None }, op: Some(OpData { command: [Base64HexBytes([AA, 12, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, B8]), Base64HexBytes([AA, B4, 00, 00, 00, 00, 80, 00, 00, 00, 80, 00, 00, 00, 80, 00, 00, 00, 80, 1E]), Base64HexBytes([AA, B0, 00, 01, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 1B]), Base64HexBytes([AA, B0, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 1A]), Base64HexBytes([AA, 12, 01, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, B9]), Base64HexBytes([AA, B4, 01, 00, 00, 00, 80, 00, 00, 00, 80, 00, 00, 00, 80, 00, 00, 00, 80, 1F]), Base64HexBytes([AA, B0, 01, 01, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 1A]), Base64HexBytes([AA, B0, 01, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 00, 1B])], mode_value: [], sleep_value: [], wakeup_value: [], timer_value: [] }) }


Add override for can.route.data

This might be better in done-autorender, putting an issue here for now.

I think your change did this, right @James0x57 ?

Yep - we had the chicken and egg problem so it doesn't look great but here's where the change happened:
https://github.com/canjs/can-ssr/blob/minor/lib/index.js#L93-L125

Right, this won't be necessary in can-wait 0.3.0 which i'm working on integrating into can-ssr now.

Closed by #143


fix lro/rpclegacy test case

pending on https://github.com/Azure/autorest.python/pull/2277

like we should update the service, not check for every call whether a body exists or not

It is from rpc-legacy testcase, which [return no body](https://github.com/Azure/cadl-ranch/blob/02a43e2c3f5732c30fd89b3e73f73b3921b89d85/packages/cadl-ranch-specs/http/azure/core/lro/rpc-legacy/mockapi.ts#L10-L22] for initial call.

@msyyc this is going to be fixed in typespec-azure, we should use their fix instead of always checking if a body is fixed Azure/typespec-azure#3972

It is not limit for TCGC but for Typespec, since Typespec doesn't have competence to describe the difference between initial call and final response.

After discussion, we all agree to optimize cadl-ranch test case to make it more precise about the lro calling process


updated pypianoroll.Multitrack function

@sumuzhao should I go ahead and update all the functions related to mentioned update?


Flatten chain of InnerExceptions to an Array of Exceptions

Hey!
I have a  JsonLayout with <attribute name="exception" layout="${exception: format=@}" /> inside it, that creates chain of Json objects, each of which representing an Exception. While this looks to be the desired behavior, when logging to Elasticsearch it just gives headaches.
There is no exact foreign key sort of relationship, hence I can't flatten a chain of exceptions to an array of exceptions, and this must be done because sometimes with for example 4 levels of chained exceptions, I should write initial lucene query like:
exception.Type: FooException OR exception.InnerException.Type: FooException OR exception.InnerException.InnerException.Type: FooException ....
There are some workarounds, like query string query which accepts wildcards as field name, but not every tool has support for it (e.g. Grafana).
I think there should be an option of flattening all these inner exceptions to an array of exceptions, like
{
    "message" :"something",
    "exceptions":
    [
       {"InnerMostException" },
       {"OuterException"},
// The rest
    ]
}

I searched the whole documentations and couldn't find such a thing. If there is no such model of formatting, can there be one at some point?
Thanks, this project has pretty nice and friendly collaborators. Kudos

Closing due to inactivity. PullRequests are ofcourse welcome


Understanding the splits

Thanks for sharing the code base and datasets! We enjoyed reading the paper.
I have some confusion regarding the splits used in the code. Can you please explain the difference between "pl_fixed", "fixed",
"pl_random" and "random"? Are they related to the low_label and high_label settings mentioned in the paper?
When we set the low_labels_test argument to 1 with random split it still directs to a 60-20 split of the dataset which is contrary to what is mentioned in the paper. Am I missing something here?
Thanks in Advance!

Hi, fixed refers to the low label rate mentioned in the paper (20 labels per class, 'fixed' means usually this is given as the official split) while random refers to 60% setting. pl is the pseudo labels given by the TAPE.  low_labels_test is related to the few-shot setting mentioned in the annotation part.


Where should we add vae ?

Which folder we have to add the vae ? The resulting images are washed out. So please tell me where to add vae's

I'd like to know too.

check in this repo https://github.com/AUTOMATIC1111/stable-diffusion-webui

sd/stable-diffusion-webui/models/VAE


Update ingameSounds-soundID.png roblox

emando
Changes

Checks
By submitting your pull request for review, you agree to the following:

[x] This contribution was created in whole or in part by me, and I have the right to submit it under the terms of this repository's open source licenses.
[ ] I understand and agree that this contribution and a record of it are public, maintained indefinitely, and may be redistributed under the terms of this repository's open source licenses.
[ ] To the best of my knowledge, all proposed changes are accurate.

This appears to be making no change to the file (replaces one image with the same image). If you'd like to open another pull request, please do so; I'll close this one.


feat: port GroupTheory.GroupAction.Opposite

mathlib SHA : fc2ed6f838ce7c9b7c7171e58d78eaf7b438fb0e

It seems like there are some serious issues with MulOpposite being a type synonym. See https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/rw.20seeing.20through.20type.20synonym.3F for discussion.

depends on: #1036

I think that merging master helped.

Darn it, too late! I'll open another PR with my golfs and fixes.


Can't connect via IP

My school blocks FCC and I used to be able to connect via IP using <IP_ADDRESS> and now it just shows a 404 error. I have pinged it again and the IP address hasn't changed. If this was done on purpose then close the issue and I will find out a way to get there somehow.

@PKMNinja We disable direct access through the ip. Otherwise someone could point their domain at it and server FCC as their own.
Why does your school block FCC?

They use just a general filter and the filter catches it for it being a forum. They do this with gitter, but at least gitter makes sense.

@PKMNinja Do you know what blocking software they're using?  Maybe we can appeal that classification.   Clearly not a forum.

@SaintPeter I believe it is http://www.iboss.com/ I can confirm for certain tomorrow at school.

@SaintPeter Yes, I just checked and it is iboss network security banned because of forum content.

I spent some time looking around the iboss site and there doesn't appear to be any sort of method for reporting an incorrectly calssified website.  Free Code Camp did used to have a forum, so I guess technically it was once a "forum".
My only suggestion would be to contact your school administration and ask them to whitelist this clearly educational site.
Sorry.

@SaintPeter thank you for checking it out. I will send a couple e emails around the IT department. See what I can do. Thank you once again for seeing what you could do.