Confusions about checks in PolynomialTensor

I've been looking over PolynomialTensor after seeing some PRs touch it, and had a few questions. I'm not very familiar with the class, so I think one of them might be a misunderstanding, but the others appear to be problems that are a) not sufficiently tested and b) could give errors in the code.
My questions have to do with several places where a TypeError is raised, or tensors are considered unequal, if self.n_body_tensors.keys() != other.n_body_tensors.keys() in https://github.com/quantumlib/OpenFermion/blob/master/src/openfermion/ops/_polynomial_tensor.py.
The questions I have are

keys() doesn't guarantees order in its output, so I don't think these checks are particularly meaningful to begin with (they should at least use set(x.keys()) for comparison - if that's what we really want),
it's unclear to me that that should be what we want anyway - if a key exists in self but not in other with the corresponding value in self within EQ_TOLERANCE of zero, the tensors should be equal, not unequal (shouldn't they?),
TypeError says that arguments are the wrong type, not that something else is wrong with them. So TypeError is not the right error to be raising here. But as in 2. I don't think this condition should lead to an error at all.

Thoughts?

The keys() method of dictionaries actually returns a key view, which is a set-like object. See https://docs.python.org/3/library/stdtypes.html#dict-views.
I suppose you're right, but I think a valid comparison between two PolynomialTensors with different sets of keys is an extraordinary circumstance and it may be worth keeping it as is for the sake of code simplicity. I say extraordinary because usually when you're dealing with PolynomialTensors, the tensor type is fixed. For instance, InteractionOperator will have the keys (1, 0) and (1, 1, 0, 0), and that will never change. So all InteractionOperators will have the same key sets, and similarly for other PolynomialTensor subclasses I can imagine.
However, this does lead me to another question: why is equality comparison here implemented with an __eq__ method, while for SymbolicOperators it is implemented with the isclose method? I don't see a reason to do it differently for both.
TypeError is generally raised to indicate that an operation is not meant to be performed in some way on some objects; it appears to me that this is how it's being used here. The returned error message can contain more specifics about the problem.

This is true in Python 3 but not in Python 2, see e.g. https://stackoverflow.com/questions/5629023/key-order-in-python-dictionaries .

This is a fair question in general - I don't have a good solution really. I agree that isclose is a better way to handle things rather than __eq__ at least for the purposes of consistency.

I think you're misinterpreting a point about functions and methods from the docs (I think you might be looking at the second paragraph of https://docs.python.org/3/library/exceptions.html#TypeError ?). The paragraph below clarifies this - "Passing arguments of the wrong type (e.g. passing a list when an int is expected) should result in a TypeError, but passing arguments with the wrong value (e.g. a number outside expected boundaries) should result in a ValueError." In this case the arguments would be of the correct type but incorrect value (the dictionaries x.n_body_tensor and y.n_body_tensor have different keys), and a ValueError or similar would be the appropriate error to raise.

We can handle this by adding the line from builtins import dict and initializing our dictionaries with dict().
Okay, I agree that we should raise ValueError in some of these cases then.

@idk3 I made a new issue at #274 to implement the changes discussed here. Do you think that covers everything? If so I think we can close this issue.


Virtual Machines

Can I use this to create virtual machines using virtualbox or vmware?

Not possible because virtualization isn't enabled.

Thank you!


2017.1 support?

Will you be adding 2017.1 support to Unity studio?

supported after commit 14ec7c8259c87e81676982b7b36259c0b1570faa


"mkdir /etc/config: permission denied" on AdminServer start

I have deploy harbor.v1.3 on my Kubernetes1.9.1
There has an error on deploy adminserver.deploy.yaml, like this:
2018-01-29T02:35:53Z [INFO] initializing system configurations...
2018-01-29T02:35:53Z [INFO] the path of json configuration storage: /etc/config/config.json
2018-01-29T02:35:53Z [DEBUG] [driver_json.go:46]: path of configuration file: /etc/config/config.json
2018-01-29T02:35:53Z [INFO] the configuration file /etc/config/config.json does not exist, creating it...
2018-01-29T02:35:53Z [FATAL] [main.go:46]: failed to initialize the system: mkdir /etc/config: permission denied

How can solve this problem？ rebuild the adminserver images?

config
harbor.cfg
hostname = index.myharbor.com
ui_url_protocol = https
db_password = GDBFskBh
max_job_workers = 8
customize_crt = on
ssl_cert = /data/admin.crt
ssl_cert_key = /data/admin.key
secretkey_path = /data
admiral_url = NA
clair_db_password = password
log_rotate_count = 50
log_rotate_size = 200M
email_identity =
email_server = smtp.mydomain.com
email_server_port = 25
email_username =<EMAIL_ADDRESS>email_password = abc
email_from = admin<EMAIL_ADDRESS>email_ssl = false
email_insecure = false
harbor_admin_password = 6!ANUD@21Ntc
auth_mode = ldap_auth
ldap_url = ldaps://ldap.myldap.com
ldap_searchdn = cn=PamService,ou=Services,dc=myldap,dc=com
ldap_search_pwd =
ldap_basedn = ou=People,dc=myldap,dc=com
ldap_filter =
ldap_uid = uid
ldap_scope = 2
ldap_timeout = 5
ldap_verify_cert = true
self_registration = on
token_expiration = 30
project_creation_restriction = everyone
db_host = mysql
db_port = 3306
db_user = root
redis_url = harborrediss
clair_db_host = postgres
clair_db_port = 5432
clair_db_username = postgres
clair_db = postgres
uaa_endpoint = uaa.mydomain.org
uaa_clientid = id
uaa_clientsecret = secret
uaa_verify_cert = true
uaa_ca_cert = /path/to/ca.pem
registry_storage_provider_name = filesystem
registry_storage_provider_config =

I have run  k8s-prepare script to make the kubernetes deployment file.

harbor before 1.3.0 running in root priviledge, while in 1.3.0 it's running at normal user, so it's the error comming. As the document state, the code running kubernetes is only tested with harbor1.2.0,  to work with harbor 1.3.0, some configuration must be adjusted by youself.
For this problem, you can setup deployment yaml like  'command:  ["sh", "-c", "mkdir /etc/config; chown harbor:harbor /etc/config; /harbor/start.sh"]" , of cause, you also can rebuild the image to solve it.

Closing as the question's answered.


DIRAPI-398 / java.lang.IllegalStateException: Returned object not currently part of this pool

Wrapped the underlying connection only when the pool.getConnection() method is called. Added test for validating and non-validating connection factory. Remove pool ref from connection factory.

Hi,
when applying the patch, and running the tests, I gt 2 errors:
`[ERROR] Tests run: 2, Failures: 0, Errors: 2, Skipped: 0, Time elapsed: 0.148 s <<< FAILURE! -- in org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest
[ERROR] org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest.testPoolWithStartTls -- Time elapsed: 0.125 s <<< ERROR!
java.lang.RuntimeException: java.lang.IllegalStateException: Returned object not currently part of this pool
at org.apache.directory.ldap.client.api.LdapConnectionPool.releaseConnection(LdapConnectionPool.java:184)
at org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest$PoolTester.execute(ValidatingPoolableLdapConnectionFactoryTest.java:535)
at org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest.testPoolWithStartTls(ValidatingPoolableLdapConnectionFactoryTest.java:182)
at java.lang.reflect.Method.invoke(Method.java:498)
at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:289)
at java.util.concurrent.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1056)
at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1692)
at java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:175)
Caused by: java.lang.IllegalStateException: Returned object not currently part of this pool
at org.apache.commons.pool2.impl.GenericObjectPool.returnObject(GenericObjectPool.java:1031)
at org.apache.directory.ldap.client.api.LdapConnectionPool.releaseConnection(LdapConnectionPool.java:171)
... 7 more
[ERROR] org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest.testPoolWithBind -- Time elapsed: 0.006 s <<< ERROR!
java.lang.RuntimeException: java.lang.IllegalStateException: Returned object not currently part of this pool
at org.apache.directory.ldap.client.api.LdapConnectionPool.releaseConnection(LdapConnectionPool.java:184)
at org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest$PoolTester.execute(ValidatingPoolableLdapConnectionFactoryTest.java:535)
at org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest.testPoolWithBind(ValidatingPoolableLdapConnectionFactoryTest.java:58)
at java.lang.reflect.Method.invoke(Method.java:498)
at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:289)
at java.util.concurrent.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1056)
at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1692)
at java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:175)
Caused by: java.lang.IllegalStateException: Returned object not currently part of this pool
at org.apache.commons.pool2.impl.GenericObjectPool.returnObject(GenericObjectPool.java:1031)
at org.apache.directory.ldap.client.api.LdapConnectionPool.releaseConnection(LdapConnectionPool.java:171)
... 7 more
[INFO]
[INFO] Results:
[INFO]
[ERROR] Errors:
[ERROR]   ValidatingPoolableLdapConnectionFactoryTest.testPoolWithBind:58 » Runtime java.lang.IllegalStateException: Returned object not currently part of this pool
[ERROR]   ValidatingPoolableLdapConnectionFactoryTest.testPoolWithStartTls:182 » Runtime java.lang.IllegalStateException: Returned object not currently part of this pool
[INFO]
[ERROR] Tests run: 46, Failures: 0, Errors: 2, Skipped: 0
[INFO]
`

Hi,
I reviewed the test and tried to analyze.
I committed a fix to the branch:
https://github.com/github-rmslash-com/directory-ldap-api/commit/b6a5c2234bf0623ec7021862f85724733f646be1
in LdapConnectionPool i added the check if a PooledLdapConnection is
passed (to make sure that old code which uses the releaseConnection
method in the pool) is also working.
The problem with the test was that the validator operates on the
underlying object in the pool (The MonitoredLdapConnection). The test
checked for a validation of the PooledLdapConnection, but the wrapped
object is being validated.
By unwrapping the PooledLdapConnection during the validation test, the
test finishes successfully.
I also tried locally a full build using mvn package verify - everything
green now....
[INFO]
[INFO] Reactor Summary for Apache Directory LDAP API 2.1.8-SNAPSHOT:
[INFO]
[INFO] Apache Directory LDAP API .......................... SUCCESS [
4.979 s]
[INFO] Apache Directory LDAP API I18n ..................... SUCCESS [
0.458 s]
[INFO] Apache Directory LDAP API Utilities ................ SUCCESS [
1.080 s]
[INFO] Apache Directory API ASN.1 Parent .................. SUCCESS [
0.067 s]
[INFO] Apache Directory API ASN.1 API ..................... SUCCESS [
0.809 s]
[INFO] Apache Directory API ASN.1 BER ..................... SUCCESS [
1.244 s]
[INFO] Apache Directory LDAP API Parent ................... SUCCESS [
0.062 s]
[INFO] Apache Directory LDAP API Model .................... SUCCESS [
6.034 s]
[INFO] Apache Directory LDAP API Codec Parent ............. SUCCESS [
0.062 s]
[INFO] Apache Directory LDAP API Codec Core ............... SUCCESS [
4.811 s]
[INFO] Apache Directory LDAP API Net Parent ............... SUCCESS [
0.043 s]
[INFO] Apache Directory LDAP API Network MINA ............. SUCCESS [
0.295 s]
[INFO] Apache Directory LDAP API Extras Parent ............ SUCCESS [
0.047 s]
[INFO] Apache Directory LDAP API Extras Codec API ......... SUCCESS [
0.461 s]
[INFO] Apache Directory LDAP API Extras Codec ............. SUCCESS [
1.445 s]
[INFO] Apache Directory LDAP API Codec Standalone ......... SUCCESS [
0.445 s]
[INFO] Apache Directory LDAP API DSML Parent .............. SUCCESS [
0.037 s]
[INFO] Apache Directory LDAP API DSML Parser .............. SUCCESS [
1.646 s]
[INFO] Apache Directory LDAP API Extras ACI ............... SUCCESS [
1.185 s]
[INFO] Apache Directory LDAP API Schema Parent ............ SUCCESS [
0.039 s]
[INFO] Apache Directory LDAP API Schema Data .............. SUCCESS [
8.320 s]
[INFO] Apache Directory LDAP API Client Parent ............ SUCCESS [
0.039 s]
[INFO] Apache Directory LDAP API Client API ............... SUCCESS [
2.386 s]
[INFO] Apache Directory LDAP API DSML Engine .............. SUCCESS [
0.399 s]
[INFO] Apache Directory LDAP API Extras Util .............. SUCCESS [
0.954 s]
[INFO] Apache Directory LDAP API Extras Stored Procedures . SUCCESS [
0.351 s]
[INFO] Apache Directory LDAP API Extras Trigger ........... SUCCESS [
0.962 s]
[INFO] Apache Directory LDAP API Schema Converter ......... SUCCESS [
1.037 s]
[INFO] Apache Directory API All ........................... SUCCESS [
0.806 s]
[INFO] Apache Directory LDAP API Client All ............... SUCCESS [
6.655 s]
[INFO] Apache Directory API Integration Tests ............. SUCCESS [
2.989 s]
[INFO] Apache Directory API OSGi Integration Tests ........ SUCCESS [
27.740 s]
[INFO] Apache Directory LDAP API Distribution ............. SUCCESS [
0.412 s]
[INFO]
[INFO] BUILD SUCCESS
[INFO]
[INFO] Total time:  01:18 min
[INFO] Finished at: 2024-12-22T11:08:16+01:00
[INFO]
@.***:~/git/rmslash/directory-ldap-api$
Thanks for your patience and your support.
Andreas
Am 22.12.24 um 08:59 schrieb Emmanuel Lécharny:

Hi,
when applying the patch, and running the tests, I gt 2 errors:
`[ERROR] Tests run: 2, Failures: 0, Errors: 2, Skipped: 0, Time elapsed:
0.148 s <<< FAILURE! -- in
org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest
[ERROR]
org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest.testPoolWithStartTls -- Time elapsed: 0.125 s <<< ERROR!
java.lang.RuntimeException: java.lang.IllegalStateException: Returned
object not currently part of this pool
at
org.apache.directory.ldap.client.api.LdapConnectionPool.releaseConnection(LdapConnectionPool.java:184)
at
org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest$PoolTester.execute(ValidatingPoolableLdapConnectionFactoryTest.java:535)
at
org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest.testPoolWithStartTls(ValidatingPoolableLdapConnectionFactoryTest.java:182)
at java.lang.reflect.Method.invoke(Method.java:498)
at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:289)
at
java.util.concurrent.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1056)
at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1692)
at
java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:175)
Caused by: java.lang.IllegalStateException: Returned object not
currently part of this pool
at
org.apache.commons.pool2.impl.GenericObjectPool.returnObject(GenericObjectPool.java:1031)
at
org.apache.directory.ldap.client.api.LdapConnectionPool.releaseConnection(LdapConnectionPool.java:171)
... 7 more
[ERROR]
org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest.testPoolWithBind -- Time elapsed: 0.006 s <<< ERROR!
java.lang.RuntimeException: java.lang.IllegalStateException: Returned
object not currently part of this pool
at
org.apache.directory.ldap.client.api.LdapConnectionPool.releaseConnection(LdapConnectionPool.java:184)
at
org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest$PoolTester.execute(ValidatingPoolableLdapConnectionFactoryTest.java:535)
at
org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest.testPoolWithBind(ValidatingPoolableLdapConnectionFactoryTest.java:58)
at java.lang.reflect.Method.invoke(Method.java:498)
at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:289)
at
java.util.concurrent.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1056)
at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1692)
at
java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:175)
Caused by: java.lang.IllegalStateException: Returned object not
currently part of this pool
at
org.apache.commons.pool2.impl.GenericObjectPool.returnObject(GenericObjectPool.java:1031)
at
org.apache.directory.ldap.client.api.LdapConnectionPool.releaseConnection(LdapConnectionPool.java:171)
... 7 more
[INFO]
[INFO] Results:
[INFO]
[ERROR] Errors:
[ERROR] ValidatingPoolableLdapConnectionFactoryTest.testPoolWithBind:58
» Runtime java.lang.IllegalStateException: Returned object not currently
part of this pool
[ERROR]
ValidatingPoolableLdapConnectionFactoryTest.testPoolWithStartTls:182 »
Runtime java.lang.IllegalStateException: Returned object not currently
part of this pool
[INFO]
[ERROR] Tests run: 46, Failures: 0, Errors: 2, Skipped: 0
[INFO]
`
—
Reply to this email directly, view it on GitHub <https://github.com/
apache/directory-ldap-api/pull/165#issuecomment-2558367690>, or
unsubscribe <https://github.com/notifications/unsubscribe-auth/
ATI27QJGI34PXQ6XYUKMPH32GZWPNAVCNFSM6AAAAABUAUW6ESVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJYGM3DONRZGA>.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi,
I checked and fixed the failed test. Synced the forked, rebased the
ranch and reopened the pull request :). Feel free to test.
https://github.com/apache/directory-ldap-api/pull/165
ZTank you for your support
Andreas
Am 22.12.24 um 08:59 schrieb Emmanuel Lécharny:

Hi,
when applying the patch, and running the tests, I gt 2 errors:
`[ERROR] Tests run: 2, Failures: 0, Errors: 2, Skipped: 0, Time elapsed:
0.148 s <<< FAILURE! -- in
org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest
[ERROR]
org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest.testPoolWithStartTls -- Time elapsed: 0.125 s <<< ERROR!
java.lang.RuntimeException: java.lang.IllegalStateException: Returned
object not currently part of this pool
at
org.apache.directory.ldap.client.api.LdapConnectionPool.releaseConnection(LdapConnectionPool.java:184)
at
org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest$PoolTester.execute(ValidatingPoolableLdapConnectionFactoryTest.java:535)
at
org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest.testPoolWithStartTls(ValidatingPoolableLdapConnectionFactoryTest.java:182)
at java.lang.reflect.Method.invoke(Method.java:498)
at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:289)
at
java.util.concurrent.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1056)
at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1692)
at
java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:175)
Caused by: java.lang.IllegalStateException: Returned object not
currently part of this pool
at
org.apache.commons.pool2.impl.GenericObjectPool.returnObject(GenericObjectPool.java:1031)
at
org.apache.directory.ldap.client.api.LdapConnectionPool.releaseConnection(LdapConnectionPool.java:171)
... 7 more
[ERROR]
org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest.testPoolWithBind -- Time elapsed: 0.006 s <<< ERROR!
java.lang.RuntimeException: java.lang.IllegalStateException: Returned
object not currently part of this pool
at
org.apache.directory.ldap.client.api.LdapConnectionPool.releaseConnection(LdapConnectionPool.java:184)
at
org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest$PoolTester.execute(ValidatingPoolableLdapConnectionFactoryTest.java:535)
at
org.apache.directory.ldap.client.api.ValidatingPoolableLdapConnectionFactoryTest.testPoolWithBind(ValidatingPoolableLdapConnectionFactoryTest.java:58)
at java.lang.reflect.Method.invoke(Method.java:498)
at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:289)
at
java.util.concurrent.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1056)
at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1692)
at
java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:175)
Caused by: java.lang.IllegalStateException: Returned object not
currently part of this pool
at
org.apache.commons.pool2.impl.GenericObjectPool.returnObject(GenericObjectPool.java:1031)
at
org.apache.directory.ldap.client.api.LdapConnectionPool.releaseConnection(LdapConnectionPool.java:171)
... 7 more
[INFO]
[INFO] Results:
[INFO]
[ERROR] Errors:
[ERROR] ValidatingPoolableLdapConnectionFactoryTest.testPoolWithBind:58
» Runtime java.lang.IllegalStateException: Returned object not currently
part of this pool
[ERROR]
ValidatingPoolableLdapConnectionFactoryTest.testPoolWithStartTls:182 »
Runtime java.lang.IllegalStateException: Returned object not currently
part of this pool
[INFO]
[ERROR] Tests run: 46, Failures: 0, Errors: 2, Skipped: 0
[INFO]
`
—
Reply to this email directly, view it on GitHub <https://github.com/
apache/directory-ldap-api/pull/165#issuecomment-2558367690>, or
unsubscribe <https://github.com/notifications/unsubscribe-auth/
ATI27QJGI34PXQ6XYUKMPH32GZWPNAVCNFSM6AAAAABUAUW6ESVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJYGM3DONRZGA>.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Mit freundlichen Grüßen
Andreas Schufft
mobile: +49-171/8279949
phone: +49-7371/9347132


fix mission information padding

The button border is being slightly cutoff, specifically on the mission information overlay of level 2.
The container has overflow-y: auto on it, which is causing the issue. Can be fixed by adding padding-bottom: 0.25rem to the button-area div.

Looks good on edge, chrome, and firefox:

Moving to done.


[FLINK-27185][connectors] Convert connector-base module to assertj

Cherry-picked commits from #19660 to get the individual changes merged faster

@flinkbot run azure


Add key for Job information

Adding a specific key for job information will allow us to easily log this information using context

@lblanch it feels this is not a generic information we want to carry on through the chain of calls in the context, may be I miss a point

It's true that it is not generic to all logs, but important in logs about jobs. In a similar way that the Tool key is only important to enrichment logs, but by adding it as a key in logger, it allows us to set it once in the context and be sure that it will be included in all logs generated from that point onwards. I am aiming for the same with jobs: once the job is triggered and is being processed, the related job information will be added to the context with this key, and this will ensure that the information is included in all related logs -> this will make it easier to debug job related issues

@piotrowski I've created a new branch v1 so we can release changes to this major version for server. Will this work?


Feature/python3 compatibility3

initial stab at Python3 compatibility
proxy unit tests cause segfault, but other unit tests pass
Tested on OSX 10.9 / (pyenv) Python 2.6, 2.7, 3.4
issue #119

ack, this needs to be rebased on master

@kristi thanks for putting effort on this issue! Where exactly is the starting base of your refactoring? This PR is unmergeable and maybe you've created it against the wrong branch. Eg. try to rebase your stuff against master


Can't initialize array_view using initializer list for some runtimeclass types

In some runtimeclass types, I get this compiler error when trying to initialize/pass array_view from an initializer list:
'winrt::array_view<winrt::Develop::ShellView>::array_view(winrt::Develop::ShellView *,winrt::array_view<winrt::Develop::ShellView>::size_type) noexcept': cannot convert argument 1 from 'const _Elem *' to 'winrt::Develop::ShellView *'

Can you share a minimal self-contained repro?

This is by design. An initializer list is an array of const objects, but you are trying to create an array_view<winrt::Develop::ShellView> which is a view of mutable objects.
Change the array_view<winrt::Develop::ShellView> to array_view<winrt::Develop::ShellView const>.


Update bm_serial Commit Reference To Latest HEAD On Develop

Merged bm_protocol before bm_serial and needed to fix the commit pointer.

🤕 😫


Support dump subcommand in ctl

Add subcommand 'dump' in ctl utils.
Close #151

@tabokie All discussions have been resolved, please take time to look again.


Mixpanel Mapping Updates

Description of the change

Description here

Type of change

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Related issues

Fix #1

Checklists
Development

[ ] Lint rules pass locally
[ ] The code changed/added as part of this pull request has been covered with tests
[ ] All tests related to the changed code pass in development

Code review

[ ]  This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
[ ] "Ready for review" label attached to the PR and reviewers mentioned in a comment
[ ] Changes have been reviewed by at least one other engineer
[ ] Issue from task tracker has a link to this pull request

Update the image with this.

@ItsSudip @utsabc the doc required significant updates along with the proposed changes, so we have refreshed the entire content. Highlighting a few sections where we will need your inputs.


Adds Mesh Tutorial.

Signed-off-by: Franco Cipollone<EMAIL_ADDRESS>🎉 New feature
Summary
The mesh tutorial here created is migrated from Google Doc to an actual tutorial in ign-rendering
Tutorial Author: @RafiAbdullah
Some pics:

Test it
I share here the generated HTML  in case you want to take a look:
tutorial_model_and_optimize_meshes.zip
After extracting the files you can use your favorite browser by doing: (e.g. using google chrome)
google-chrome model_and_optimize_meshes.html

Checklist

[x] Signed all commits for DCO
[ ] Added tests
[x] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

Thank you for the tutorial, @francocipollone ! I don't remember if I was the one who suggested putting it in ign-rendering, but looking at it now, since it touches simulation concepts like collision, I think the tutorial should be in ign-gazebo instead. What do you think?

Thank you for the tutorial, @francocipollone ! I don't remember if I was the one who suggested putting it in ign-rendering, but looking at it now, since it touches simulation concepts like collision, I think the tutorial should be in ign-gazebo instead. What do you think?

Yeah I asked you about it on Slack. :smile: . It it true it mentions some simulation concepts, I could close it and replicate it in ign-gazebo then

Yeah I asked you about it on Slack. smile

Ha I thought so :see_no_evil:

I could close it and replicate it in ign-gazebo then

Thanks! I must have not realized earlier that it touched simulation-specific concepts :bowing_woman:

Closed in favor of https://github.com/ignitionrobotics/ign-gazebo/pull/915


Getting error @shopify/hydrogen doesn't appear to be written in CJS,

@shopify/hydrogen doesn't appear to be written in CJS, but also doesn't appear to be a valid ES module (i.e. it doesn't have "type": "module" or an .mjs extension for the entry point). Please contact the package author to fix.
Please check it.
Frontend seeing only logo and cart icon

I believe this is partially a duplicate of #341, specifically regarding the error you're seeing in the console.
However, it also sounds like you can't get the rest of the app to load. If you would like help diagnosing / fixing that issue, please follow the issue template so that we can get the necessary information to help you out.
Thanks!

I am getting warn as below also and not frontend load.
You have enabled the JIT engine which is currently in preview.

Looks like you've opened a discussion about the JIT engine warning here https://github.com/Shopify/hydrogen/discussions/389. 👍
With that, I think we've addressed everything we can here. I'm going to close this issue; if you have additional issues, please feel free to ask here or open a new issue while ensuring you use the issue template. Thanks!


Add TT ep 0025

@bryanfriedman This is the one where the speaker is external, thanks!

Added an "external" tag to front matter and using that to filter out from team list page. I think this should cover this situation for now.

@bryanfriedman Looks good to me! I should dig into that layout script so you don't have to keep doing work for me. Thanks!


Null Pointer Excepion on PortScan

Here in this line sometimes i get NPE - cannot set boolean
public void cancel() {
        this.cancelled = true;
    }

i suggest to fix it by :
public void cancel() {
try {
        this.cancelled = true;
} catch (Exception ignored) {
}
    }

I don't think the crash is from the library but from your code, can you share your code here so I can help?
this - cannot be null in java
cancelled - is a primitive boolean, which cannot be null.
So the NPE can't be coming from the library.


Could not find file 'clammicrobench_nasm_rob.asm'.

PS G:\git\Microbenchmarks\AsmGen\bin\Release\net7.0-windows> ./AsmGen.exe autocopy
Automatically copying files, based on default VS paths
Moved clammicrobench_nasm_zen3rob.asm
Moved clammicrobench_nasm_rob.asm
Unhandled exception. System.IO.FileNotFoundException: Could not find file 'G:\git\Microbenchmarks\AsmGen\bin\Release\net7.0-windows\clammicrobench_nasm_rob.asm'.
File name: 'G:\git\Microbenchmarks\AsmGen\bin\Release\net7.0-windows\clammicrobench_nasm_rob.asm'
   at System.IO.File.Move(String sourceFileName, String destFileName, Boolean overwrite)
   at AsmGen.Program.CopyFiles(String targetDir, String[] fileNames) in G:\git\Microbenchmarks\AsmGen\Program.cs:line 212
   at AsmGen.Program.Main(String[] args) in G:\git\Microbenchmarks\AsmGen\Program.cs:line 196

Build with latest clone and just changed the target framework to 6.0 or 7.0(because 5.0 would cause 'Metrics' does not exist in the namespace 'System.Diagnostics' error).

VS/NASM not used anymore for structure size tests


Connecting monaco with Yang language server

Hi,
I am having trouble to use this in my example. I am trying to connect to Yang language server (via web socket based) . I just modified few things to connect to Yang-LSP.
In client.js harcoding my yang-lsp server
function createUrl(path: string): string {
const protocol = location.protocol === 'https:' ? 'wss' : 'ws';
return normalizeUrl(${protocol}://localhost:5007);
}
languageclient does try to connect to my language server. However, the language server throwing below error...
`java.lang.IllegalStateException: Missing header Content-Length in input "GET / HTTP/1.1
Host: localhost:5007
Connection: Upgrade
Pragma: no-cache
Cache-Control: no-cache
Upgrade: websocket
Origin: http://localhost:3000
Sec-WebSocket-Version: 13
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/75.0.3770.100 Safari/537.36
Accept-Encoding: gzip, deflate, br
Accept-Language: en-US,en;q=0.9
Sec-WebSocket-Key: 0ssCC+n3Mw6RpB2c+3COYw==
Sec-WebSocket-Extensions: permessage-deflate; client_max_window_bits
"
at org.eclipse.lsp4j.jsonrpc.json.StreamMessageProducer.listen(StreamMessageProducer.java:91)
at org.eclipse.lsp4j.jsonrpc.json.ConcurrentMessageProcessor.run(ConcurrentMessageProcessor.java:99)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)`
its asking for Content-Length
I have already gone through the whole discussion around HTTP and LSP protocol and stuff
I didn't find any fix or workaround mentioned any where.
Does it require some intermediate parser logic from yang file to stream?
Any help/guidance will be appreciated
Thank you
Om Reddy

@OmReddy Have you tried connecting with other language servers? Do those work?

@rcjsuen the examples given with the Monaco language server are working for me
languageClient. Both examples are on JSON server.
I did my changes in json-server-launcher. So instead of JSON i am starting my Yang LSP here.
It appears everything is hooked, but when i try to send request(by modifying something in monaco editor) getting above error on server side.

Perhaps you could try launching the YANG language server from the command line and sending things directly in from there or try connecting to it with a different language client? Or maybe not use web sockets?
The VS Code extension for YANG doesn't seem very complicated so it seems like it shouldn't be hard to start the server.

@cdietrich @tocjent
Hi Folks, sorry to bother you guys... i read the discussion you guys had on similar issue few months back 189
Can you suggest what is the work around for this

either you can add/trim the header on the server.
or you use the upcoming lsp 0.8.0-SNAPSHOT which has support for websockets directly by avoiding using StreamMessageProducer/Consumer
https://github.com/eclipse/lsp4j/pull/314


Crash in leanback BrowseFragment

I have this strange crash happening in leanback BrowseFragment. It occurs when fast scrolling through the row. Do you have any idea how to prevent this crash ?
java.lang.IllegalArgumentException: View android.support.v17.leanback.widget.ShadowOverlayContainer{cee235a V.ED..... ........ 1854,334-2268,851} is not a direct child of android.support.v17.leanback.widget.HorizontalGridView{81dc26e VFE...... ........ 0,0-1920,605 #7f12014c app:id/row_content}
                                                   at android.support.v7.widget.RecyclerView.getChildViewHolder(SourceFile:4224)
                                                   at android.support.v17.leanback.widget.GridLayoutManager$3.createItem(SourceFile:1488)
                                                   at ct.a(SourceFile:78)
                                                   at android.support.v17.leanback.widget.Grid.j(SourceFile:346)
                                                   at android.support.v17.leanback.widget.GridLayoutManager.m(SourceFile:1734)
                                                   at android.support.v17.leanback.widget.GridLayoutManager.j(SourceFile:2070)
                                                   at android.support.v17.leanback.widget.GridLayoutManager.scrollHorizontallyBy(SourceFile:2005)
                                                   at android.support.v7.widget.RecyclerView$d.run(SourceFile:4722)
                                                   at android.view.Choreographer$CallbackRecord.run(Choreographer.java:871)
                                                   at android.view.Choreographer.doCallbacks(Choreographer.java:683)
                                                   at android.view.Choreographer.doFrame(Choreographer.java:616)
                                                   at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:857)
                                                   at android.os.Handler.handleCallback(Handler.java:751)
                                                   at android.os.Handler.dispatchMessage(Handler.java:95)
                                                   at android.os.Looper.loop(Looper.java:154)
                                                   at android.app.ActivityThread.main(ActivityThread.java:6120)
                                                   at java.lang.reflect.Method.invoke(Native Method)
                                                   at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:865)
                                                   at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:755)

It happened to me too.

This sample has been migrated to a new location and updates are happening there (check README for more information).
As recommended by GitHub, we are closing all issues and pull requests now that this older repo will be archived.
If you still see this issue in the updated repo, please reopen the issue/PR there. Thank you!


--sc does not work if SUBCASE contains commas

Description
I have a subcase which contains a comma and I can't filter it with --sc. Moreover it was very troubling because the wrong test was run.
Steps to reproduce
Here's a simple program:
#define DOCTEST_CONFIG_IMPLEMENT_WITH_MAIN
#include "doctest/doctest.h"

TEST_CASE("tc") {
    SUBCASE("ab") {
        std::cout << "ab" << std::endl;
    }
    SUBCASE("ab, c") {
        std::cout << "abc" << std::endl;
    }
}

Run it with
clang++ test.cpp -o test -std=c++14 && ./test --sc='ab, c'

You get "ab" instead of "ab, c"
Extra information
This seems to be a feature to run multiple subcases, but it was quite misleading to me, I lost some time just to see that the wrong test was run.
I have two alternative suggestions to fix that:

if the string is "ab, c" run all "ab", " c", and "ab, c" subcases
or change the argument syntax so that if you want to run multiple subcases, you would type --sc='ab' --sc='cd'

doctest version: master cd1d747
Operating System: debian unstable
Compiler+version: clang 5.0.0

the same issue is true for filtering test cases by name - not just about subcases...
the reasoning behind this was to have a simple way of specifying multiple test cases (or subcases) to be executed - with a comma-separated string... but I guess having commas in the names should either:

just work - including filtering
trigger a warning/error at runtime (or perhaps I could use a constexpr parse of the test case/subcase name to check for ,)

I guess I should make it work - but I'm not sure how - without changing the semantics of the current filtering system...

it seems that in Catch commas can be escaped and included in such searches - see this:
https://github.com/catchorg/Catch2/issues/1327#issuecomment-402970532

I just thought of a workaround for the time being - use ? instead of the commas (,)

a PR which allows commas to be escaped was just merged, however I do think that this solution:

or change the argument syntax so that if you want to run multiple subcases, you would type --sc='ab' --sc='cd'
would be better in the long term, so I'll keep this open

Well, now the comma as a separator is part of the doctest binary interface, we can't change it without a breaking change.
This would be a change for version 3.xx, if this is even going to be considered.
Personally, I think the backslash escaping solution does the job just fine.
I suggest we either

Close this
Mark it for 3.xx with a new label


Incorrect AzureApplicationInsightsProvider reference

The first paragraph in the How It Works section should be refer to Handler="Web.TPEngine.Providers.AzureApplicationInsightsProvider, Web.TPEngine, Version=<IP_ADDRESS> rather than Handler="Web.TPEngine.Providers.UserJourneyContextProvider, Web.TPEngine, Version=<IP_ADDRESS>.

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

ID: b323b269-e228-4e1f-265c-1b47831cc871
Version Independent ID: 9248541d-dc78-4cd6-01eb-2d459df058fe
Content: Track user behavior by using events in Application Insights from Azure Active Directory B2C
Content Source: articles/active-directory-b2c/active-directory-b2c-custom-guide-eventlogger-appins.md
Service: active-directory
GitHub Login: @davidmu1
Microsoft Alias: davidmu

@chrispadgettlivecom
Thanks for your feedback! We will investigate and update as appropriate.

Thanks, @chrispadgettlivecom
I created a pull request to fix this. #57909
CC: @davidmu1


Hardware API: Create HTTP handlers, client, and commands for move, move_rel, and send_command

A core capability for the hardware API is the ability to move the robot as a six-axis motion controlled machine. Implement the handlers, the commands, and client bindings for the hardware control methods move_to, move_rel, and a special method send_command that accepts a raw gcode string and sends it to the motor controller. In addition, implement it in the client.
Also, add elements to the status for the position of the robots' axes.

No longer applicable for this issue tracker


chore: release 1.6.4

:robot: I have created a release *beep* *boop*
1.6.4 (2021-07-13)
Bug Fixes

oxshibboleth syntax error (fdf73e5)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/GluuFederation/cloud-native-edition/releases/tag/v1.6.4 :sunflower:


Session Feedback

Thanks for taking a couple of minutes to give feedback, which will help me improve for next time. Before doing anything else, hit the green button "Submit new issue" right now to save this issue content, rather than try to complete the feedback in this raw form. Then go through the questions and mark a single checkbox for each, to represent your answer. Finally, in the empty comment box below this one, please describe what you liked and what you didn't like.
What was your experience of CAP before this session?

[ ] None
[x] Basic
[ ] Intermediate
[ ] Advanced

Did this session meet your expectations?

[ ] Not really
[ ] Somewhat
[ ] Mostly
[x] Fully

Was the time allotted to each exercise enough for you to work through them?

[ ] Not really
[x] On the whole, yes

What did you think of the extra information, explanations and narratives in the exercises?

[ ] Too much information
[ ] Didn't really read it, didn't really bother me though
[x] Found it helpful as context and background

How did you find the way we all moved at the same pace through the exercises?

[ ] Would have preferred to go through them on my own at my own speed
[ ] Didn't mind, no strong feelings either way
[x] Found it useful to keep pace and discuss the exercise questions, and reflect

How did you find the actual individual (👉) tasks in the exercises?

[ ] Not relevant enough
[ ] Hard to do
[x] About right
[ ] Helpful and useful

How confident are you in being able to start mashing up services in CAP?

[ ] Still don't understand how things fit together
[x] I get it now in theory, just need a bit more time
[ ] Confident enough to start integrating services

Which environment(s) did you use?

[ ] A Dev Space in SAP Business Application Studio
[x] VS Code + dev container

If you have time, please add a comment below to write free-form what you liked and what you disliked about the session. Thanks!

Very fun session thanks DJ! One comment from my end: I think a little bit more time to read on the longer exercises could be helpful, but maybe that's just me :-)
Until next time!


Add cloud function to send silent push notification

Make an HTTP POST to Firebase, with a payload that causes the notification to be silent
Add folder for server code

Currently only contains Google Cloud Function code
Each function's main.py and requirements.txt files follow the convention of the autogenerated source files when creating a Google Cloud Function in their GUI.

Add scripts to deploy cloud function
Add Python ignore boilerplate from GitHub
Add Google Cloud ignore boilerplate from gcloud functions deploy

(Squashed, to a single commit matching the PR title and description.)

You're welcome. The .gitignore and .gcloudignore mostly came from me inspecting the Source tab of the Google Cloud Function, and seeing unwanted files. I didn't see the Python files to ignore during local development. So the ignore files were to make sure only the necessary files ended up in the cloud platform. I hope it helps local development too.
(I wish the ignore files weren't in so specific a directory, but the specific directory was necessary for the ignores to be picked up by gcloud functions deploy. Later, if there winds up being a lot of Python code in this repo, maybe we could share these ignore files in more folders via symlink.)


New PR to test SignCLA at 2019-07-15 19:29:04.608572335 -0700 PDT m=+5.157646504

Body of PR

unsigned-fail-staging: test succeeded


Bad orthophoto after running soil mask ratio with 0.90 ratio

The current behavior or issue
The Alpha mask after soil masking ratio is run, for 0.90 ratio, is quite incorrect when run over /iplant/home/shared/commons_repo/curated/tamu_corn_2017_CS17_G2F_20190305/CS17_G2F_20170614_RC/CS17_G2F_20170614_RC_Mosaic.tif. This does not happen with the TERRA REF-based algorithm or when the ratio is 1.0.
The steps taken to reproduce the behavior or issue, or specify a location where the steps were recorded
Run the soilmask-ratio code with a ratio of 0.90. Load the resulting image in a view that respects the alpha channel
Expected behavior
The alpha channel is unaffected by the soil mask algorithm
Screenshots
N/A
Add other supporting information that may be useful
N/A
Completion criteria
This is done when the alpha channel is maintained

This is related to https://github.com/AgPipeline/issues-and-projects/issues/420. The resulting file was too large without specifying BIGTIFF=YES (see https://gdal.org/drivers/raster/gtiff.html). Better compression by specifying PREDICTOR=2 (see previous link) almost halved the file size, keeping it away from needing BIGTIFF


NaN rate setpoints when switching from acro to manual

The log https://logs.px4.io/plot_app?log=89ab485c-7768-4fca-b7e9-5ed9d09152b3 contains NaN setpoints when switching from acro to manual. Seems to be a race condition when the position controller did not yet publish a valid attitude setpoint but the attitude controller already got the mode switch.
However the position controller does publish an invalid setpoint with constant timestamp before that, as seen in the CSV file:
timestamp,roll_body,pitch_body,yaw_body,yaw_sp_move_rate,q_d[0],q_d[1],q_d[2],q_d[3],thrust,landing_gear,q_d_valid,roll_reset_integral,pitch_reset_integral,yaw_reset_integral,fw_control_yaw,disable_mc_yaw_control,apply_flaps
[...]
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
10039547,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,0.0,-1.0,0,0,0,0,0,0,0
234508568,0.0,-0.0,1.90531,0.0,0.579523,0.0,0.0,0.814956,0.118343,-1.0,1,0,0,0,0,0,0
234529628,0.0,-0.0,1.90531,0.0,0.579523,0.0,0.0,0.814956,0.118343,-1.0,1,0,0,0,0,0,0
234550563,0.0,-0.0,1.90531,0.0,0.579523,0.0,0.0,0.814956,0.118343,-1.0,1,0,0,0,0,0,0
234571562,0.0,-0.0,1.90531,0.0,0.579523,0.0,0.0,0.814956,0.120316,-1.0,1,0,0,0,0,0,0
234592561,0.0,-0.0,1.90531,0.0,0.579523,0.0,0.0,0.814956,0.123274,-1.0,1,0,0,0,0,0,0
234613560,0.0,-0.0,1.90531,0.0,0.579523,0.0,0.0,0.814956,0.12426,-1.0,1,0,0,0,0,0,0
234634566,0.0,-0.0,1.90531,0.0,0.579523,0.0,0.0,0.814956,0.12426,-1.0,1,0,0,0,0,0,0
234655562,0.0,-0.0,1.90531,0.0,0.579523,0.0,0.0,0.814956,0.126233,-1.0,1,0,0,0,0,0,0
234676562,0.0,-0.0,1.90531,0.0,0.579523,0.0,0.0,0.814956,0.12426,-1.0,1,0,0,0,0,0,0
234697562,0.0,-0.0,1.90531,0.0,0.579523,0.0,0.0,0.814956,0.125247,-1.0,1,0,0,0,0,0,0
234718560,0.0,-0.0,1.90531,0.0,0.579523,0.0,0.0,0.814956,0.12426,-1.0,1,0,0,0,0,0,0
234739571,0.0,-0.0,1.90531,0.0,0.579523,0.0,0.0,0.814956,0.125247,-1.0,1,0,0,0,0,0,0
234760563,0.0,-0.0,1.90531,0.0,0.579523,0.0,0.0,0.814956,0.126233,-1.0,1,0,0,0,0,0,0
234781563,0.0,0.167662,1.90531,0.0,0.577487,-0.0682384,0.0485249,0.812094,0.127219,-1.0,1,0,0,0,0,0,0
234802565,0.0,0.206141,1.90531,0.0,0.576447,-0.0838494,0.059626,0.810631,0.127219,-1.0,1,0,0,0,0,0,0
234823563,0.0,0.241872,1.90531,0.0,0.57529,-0.0983176,0.0699145,0.809004,0.127219,-1.0,1,0,0,0,0,0,0

log001_vehicle_attitude_setpoint_0.zip
@MaEtUgR @Stifael can you look into this please?

It might be due to MPC operating without a position.

 The hash is not known on master. I'm asking because this might have changed with #9731... or was it already including these changes?
It's hopefully reproducible in simulation.

It might be due to MPC operating without a position.

Is a good idea.

I'm asking because this might have changed with #9731

No it was before that. I was wondering that as well, it could be fixed already. I can report back when I see it again.

@bkueng Does this persist or can we close?


Create folders when needed, rather than at beginning?

Right now we create an empty zarr archive folder structure at the destination before starting softcopy. However this means that, if the user says they're gonna acquire 1000 timepoints but only acquires 3, we have lots of folders on the destination that don't neccessarily exist at the source. Although I'm not sure if tensorstore creates folders on demand either - this might be a moot point.

This is actually useful in either case for tensorstore - it takes a long time to create ~10^4 folders on our cluster, and so it would probably be a lot faster to create them on demand


Create Kubernetes node deleted experiment

A number of delete node experiments have been created (delete-all-nodes, delete-named-node, delete-random-node)


Upgrade to a new version of Makie, GLMakie and WGLMakie

This PR is the next step in supporting the new Makie/AbstractPlotting interface.
We are using the new Version of Makie: 0.14 and the new versions of<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Upgrade<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Added some utility functions for the documentation - currently added them in NotebooksUtilities as they are very similar to the utility functions to support Pluto.
Removed support for older version of Makie due to the fact that we are limiting the version in the project.toml
Seems that the new Makie draw arrows in figures a bit differently than previous versions which cause some changes in some debug features in the Emitters module and a few function in the Vis module.
Changed the support for the 3D, 2Dx and 2Dy buttons in the Makie figures - need to be checked well.
Added first try at producing documentation figures that are interactable - currently just in the Emitters documentation page for 3D drawings.

Codecov Report

Merging #228 (de7aec3) into main (e2e6ae2) will decrease coverage by 4.06%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #228      +/-   ##
==========================================
- Coverage   56.69%   52.62%   -4.07%
==========================================
  Files          68       64       -4
  Lines        6420     6390      -30
==========================================
- Hits         3640     3363     -277
- Misses       2780     3027     +247

Impacted Files
Coverage Δ

src/NotebooksUtils/NotebooksUtils.jl
0.00% <0.00%> (ø)

src/OpticSim.jl
87.50% <ø> (-3.41%)
:arrow_down:

src/Vis/Emitters.jl
0.00% <0.00%> (ø)

src/Vis/Visualization.jl
0.00% <0.00%> (-6.12%)
:arrow_down:

src/Geometry/CSG/CSG.jl
31.08% <0.00%> (-52.50%)
:arrow_down:

src/Geometry/Primitives/Qtype.jl
75.00% <0.00%> (-8.34%)
:arrow_down:

src/Geometry/Primitives/Sphere.jl
84.90% <0.00%> (-7.55%)
:arrow_down:

src/Geometry/Primitives/Cylinder.jl
79.62% <0.00%> (-7.41%)
:arrow_down:

src/Optical/LensAssembly.jl
48.93% <0.00%> (-7.24%)
:arrow_down:

... and 15 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 e2e6ae2...de7aec3. Read the comment docs.


Test 1

Can i work on your project, by adding GUI interface

done with addinng the gui interface!


Thermostat Setpoints Failing

I am having trouble with local thermostat calls again.
This is my test code with DeviceID and LocalKey redacted:
from tinytuya import Contrib
import time

tstatdev = Contrib.ThermostatDevice( '{DEVICE_ID}', '<IP_ADDRESS>', '{LOCAL_KEY}' )

tstatdev.setUnits( 'f' )

#test 1 works
#tstatdev.setMiddleSetpoint( 72, 'f' )

#test 2 fails (TypeError: setMiddleSetpoint() takes from 2 to 3 positional arguments but 4 were given)
#tstatdev.setSetpoint( 72, 'f' )

#test 3 fails (Requested key 'cooling_setpoint_f' not found!)
#tstatdev.setMode( 'cool' )
#tstatdev.setCoolSetpoint( 72, 'f' )

#test 4 fails (Requested key 'cooling_setpoint_f' not found!)
#tstatdev.setMode( 'heat' )
#tstatdev.setHeatSetpoint( 72, 'f' )

data = tstatdev.status()
print('Device status: %r' % data)

Tests were ran one at a time by uncommenting the associated lines of code.
The first test works with setMiddleSetpoint()
The second test fails with:
Traceback (most recent call last):
  File "/home/user/projects/xeal/tuya/new_test/tinytuya/examples/test_stat.py", line 12, in <module>
    tstatdev.setSetpoint( 72, 'f' )
  File "/home/user/.local/lib/python3.9/site-packages/tinytuya/Contrib/ThermostatDevice.py", line 306, in setSetpoint
    return self.setMiddleSetpoint( self, setpoint, cf )
TypeError: setMiddleSetpoint() takes from 2 to 3 positional arguments but 4 were given

The last 2 tests fail similarly with:
Requested key 'heating_setpoint_f' not found!
Traceback (most recent call last):
  File "/home/user/projects/xeal/tuya/new_test/tinytuya/examples/test_stat.py", line 20, in <module>
    tstatdev.setHeatSetpoint( 72, 'f' )
  File "/home/user/.local/lib/python3.9/site-packages/tinytuya/Contrib/ThermostatDevice.py", line 314, in setHeatSetpoint
    return self.setValue( k, setpoint )
  File "/home/user/.local/lib/python3.9/site-packages/tinytuya/Contrib/ThermostatDevice.py", line 353, in setValue
    dps, val = self.parseValue( key, val )
TypeError: cannot unpack non-iterable bool object

I'm not sure why the keys are not found, when they are clearly in the table.
@uzlonewolf, I know you've used the thermostats before. Have you ran into this, or have any insight?

Test # 2 was resolved by updating my install with:
python3 -m pip install --upgrade tinytuya
Tests # 3 and # 4 are still failing.

Ok, I think I see what's going on.  There are 2 different firmware versions: one which uses low-resolution temperatures on one set of DPs, and the other which uses high-resolution on a different set of DPs.  Until it receives a .status() response it has no idea which set it needs to use.  So, simply move that tstatdev.status() to above the tests.

Thank you @uzlonewolf. That worked!

As a side note, I've found that these thermostats get very confused if you send certain commands too quickly.  For example, a .setMode(...) followed immediately by a .set____Setpoint(...) will result in it reporting and displaying that new setpoint, however internally it will still be using the old setpoint.  In my program I ended up waiting for the status update before sending another command after either a mode change or setpoint change.

@uzlonewolf
The setFanRuntime() function has a similar error, but it is a typo bug.
On line 350, it uses the 'fan_runtime' key.
But on line 252 the table adds an underscore with 'fan_run_time'

Is this a bug related to issue #327?

Doesn't look like it.  Are any of those tests still failing?

the tests work with the status call ran first

I'm going to close this one out as it looks like everything has been fixed.  @elockman please re-open if you're still having issues.


Job scheduler's options specified in shell script are ignored by snakemake command

Is your feature request related to a problem? Please describe.
I usually write shell script with grid engine's options as follows:
sleep.sh
#!/bin/bash

#$ -l nc=4
#$ -p -50
#$ -r yes
#$ -q large.q

#SBATCH -n 4
#SBATCH --nice=50
#SBATCH --requeue
#SBATCH -p node03-06
SLURM_RESTART_COUNT=2

sleep 20
echo Hi > out.txt

, where #$ means the option row of open grid engine and #SBATCH means the option row of Slurm.
Specifying such comments, I can easily submit the job like
qsub sleep.sh

or
sbatch sleep.sh

with the options above.
However, when using snakemake command, I found that snakemake ignores such options written in shell script, so I have to specify the options in the option --cluster like below.
snakemake -j 1 --cluster "qsub -l nc=4 -p -50 -r yes -q large.q"

This is very stressful for me, because I have to memorize such long option.
Describe the solution you'd like
Please don't ignore the option rows in shell script.

Hi @kokitsuyuzaki,
I've personally dealt with this situation by placing my snakemake command in a bash script. Whenever I want to run snakemake, I just run my wrapper script instead. There's no need to remember the contents of the cluster argument because they're in the script.
#!/usr/bin/env bash

snakemake \
-j 1 \
--cluster "qsub -l nc=4 -p -50 -r yes -q large.q" \
"$@"

The use of "$@" at the end let's me pass any additional arguments to snakemake as arguments to the wrapper script.


CHEBI:80729 and CHEBI:121196, please CHECK

Hi Gareth,
Please check CHEBI:80729, toxoflavin and CHEBI:121196, reumycin (1-demethyltoxoflavin), since I need the two for a Rhea reaction.
pmid:27070241
Cheers,
Kristian

Hi Kristian,
These 2 entries have now been checked.
Best Regards,
Adnan


Add Deprecation to AddressSettings in favour of brokerProperties

The non extensible address settings can be fully replaced with brokerProperties, which aromatically supports any future settings

https://github.com/artemiscloud/activemq-artemis-operator/pull/881

[#1][#2][#3] format for the commit message will tie the commit to the issue, will try that.


Generating and publishing documentation

[x] include documentation publishing package
[x] generate documentation
[x] publish documentation

Basic documentation made and published.
Closing


@react-native-communit/eslint-config not found

Description
https://registry.yarnpkg.com/@react-native-communit%2feslint-config
https://registry.npmjs.org/@react-native-communit%2feslint-config
giving not found error while running yarn and npm i --f
also
https://github.com/facebook/react-native/tree/HEAD/packages/eslint-config-react-native-community#readme
It has not found error
React Native Version
0.71.12
Output of npx react-native info
System:
OS: macOS 13.5
CPU: (10) arm64 Apple M2 Pro
Memory: 80.22 MB / 16.00 GB
Shell:
version: "5.9"
path: /bin/zsh
Binaries:
Node:
version: 20.2.0
path: /opt/homebrew/bin/node
Yarn:
version: 1.22.19
path: /opt/homebrew/bin/yarn
npm:
version: 9.6.7
path: /opt/homebrew/bin/npm
Watchman:
version: 2<IP_ADDRESS>
path: /opt/homebrew/bin/watchman
Managers:
CocoaPods:
version: 1.12.1
path: /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms:
- DriverKit 22.4
- iOS 16.4
- macOS 13.3
- tvOS 16.4
- watchOS 9.4
Android SDK: Not Found
IDEs:
Android Studio: 2022.3 AI-223.8836.35.2231.10406996
Xcode:
version: 14.3.1/14E300c
path: /usr/bin/xcodebuild
Languages:
Java:
version: 11.0.19
path: /usr/bin/javac
Ruby:
version: 2.6.10
path: /usr/bin/ruby
npmPackages:
"@react-native-community/cli": Not Found
react: Not Found
react-native: Not Found
react-native-macos: Not Found
npmGlobalPackages:
"react-native": Not Found
Android:
hermesEnabled: Not found
newArchEnabled: Not found
iOS:
hermesEnabled: Not found
newArchEnabled: Not found
Steps to reproduce
run npm i or yarn
It's installation issue
Snack, screenshot, or link to a repository

yarn-error.log

Your error looks like you typo the package name no? @react-native-communit/eslint-config

As @iBotPeaches mentioned, you have a typo in the package name + the correct URL is:
https://github.com/facebook/react-native/blob/main/packages/eslint-config-react-native/README.md


dep task_name -> you need to specify at least one host or stage

Q
A

Issue Type
Question

Deployer Version
6.4.3

Local Machine OS
ubuntu

Remote Machine OS
ubuntu

The documentation doesn't say I need to pass a stage or host when running tasks, can you not have standalone tasks without requiring a host or stage for general commands you want to run before staging and all that jazz?
I've also tried calling ->local() on the task and I still get the same error.

Actually, it's a bit hidden on this page.
You have to add the following code to your deploy.php file:
set('default_stage', 'staging');

I've found that option useful for as the same need. Reference: #1517
https://deployer.org/docs/hosts.html#localhost
Here is my simple solution:
set('default_stage', 'local');

task('test', function ()
{
  $result = run('uptime');
  writeln('uptime: '. $result);
});

localhost()
  ->stage('local');

dep test command will use default_stage as stage default by itself. Also you don't need to use  runLocally or ->local() anymore for non-stage tasks.

Fixed!
<?php

namespace Deployer;

task('test', function () {
    run('date');
});


Move images; Change the order

Are there any plans to make the functionality to change the order of images?
Example: jQuery Sortable

Would be nice but it's not on my roadmap sorry @vodniciarv . If anyone wanted to have a go and submit a pull request feel free to go wild :)

Closing due to inactivity


Size : Rounding errors and property naming

For size the property size should be called sizeInDp.
val Size.sizeDp: Int should be named sizeInPx as it's actually the pixels. It should not be casted toInt() as it's inaccurate. As it's only used in Confetti as  private var width = size.sizeDp.toFloat() it should just be left as a float.
Also mass tells me quite nothing about what it does. What does negative mass mean? What does positive mass mean?
What does mass 0 mean? (it actually crashes the app as in Vector you are dividing by that value). So better validate the argument:
  data class Size(val sizeInDp: Int, val mass: Float = 5f) {

    internal val sizeInPx: Float
      get() = sizeInDp * Resources.getSystem().displayMetrics.density

    init {
      require(mass != 0F) { "mass=$mass must be != 0" }
    }
  }

Thanks for the feedback @PaulWoitaschek
I've created a pull request, if you have some time to review I would be very happy with that.


Two Cell Can Swipe simultaneously

Hey I want to stop two cell swiping simultaneously?

Fixed ;)

Glad to see this fix but per my testing this issue cannot be fixed completely.
I can still repro this issue sometimes.

Hello. This problem is repeated at me as well. If you open a menu of two cells with a small interval of time, no such problem. But if you pull the two cells at the same time, the problem persists. Is it possible to solve?

The same issue. I still can swipe two cells at the same time


Use Factory to Return HTTP Response within a Stump

Refactor Stumps to use a factory to generate an HTTP response for a stump rather than having a Stump directly tied to a static response.  This allows Stumps to dynamically modify the response.  Further, providing the originating HTTP request to the factory can allow the response to be modified based on the content of the incoming request.
Example Code
Before
class Stump {

  // Other Code Here ...

  public IStumpsHttpResponse Response
  {
      get => _response;
      set => _response = value ?? throw new ArgumentNullException(nameof(value));
  }
}

After
class Stump {

  // Other Code Here ...

  public IHttpResponseFactory ResponseFactory
  {
      get => _responseFactory;
      set => _responseFactory = value ?? throw new ArgumentNullException(nameof(value));
  }
}

interface IHttpResponseFactory {
IStumpsHttpResponse CreateResponse(IStumpsHttpRequest request);
}

Completed and added to v0.4

This has been updated with #93 wherein the interface is now called IStumpResponseFactory.


Added the 'spo sitedesign task list' command solving #781

Added the 'spo sitedesign task list' command solving #781

Coverage remained the same at 100.0% when pulling 150dac2a38faec0b17188002222c642ef95a1023 on waldekmastykarz:spo-sitedesign-task-list into 8354ad6d7fb2cfe21f428f988f98886e9ac0e626 on pnp:dev.

Merged manually.


Add excludes to sources.kubernetes

Following up on #17, add exclusions found in sources.container

thanks for this, I will review it later today.

@flah00 : First off thank you for the pull request, we always appreciate the help.  After thinking about this more, I think there are a few things we should do.    The sources.containers.conf uses a regex pattern primarily because there will be multiple fail paths under the containers log directory and you may want to exclude some but not all.
With the kubernetes sources however these are all concrete paths.  To avoid unnecessary processing of the logs in fluentd I think we should take advantage of the exclude path option in the fluentd plugin. We currently support this in containers by specifying the EXCLUDE_PATH environment variable.
exclude_path "#{ENV['EXCLUDE_PATH']}"
I think we should be able to to do the same thing here, but with all the sources in the kubernetes.sources.conf.  The advantage of this is being able to exclude logs that are not running as a pod and we do not send the logs to fluentd at all.  Do you want to take a stab at that approach?  I realize my comments in #17 likely led you to this path and after thinking about my comments, I think it was not the best direction so apologies for that.

@frankreno Thanks for the detailed explanation. I can't believe I excluded EXCLUDE_PATH! If you don't mind, I'm going to update the README, so it explicitly states what each exclude can affect.

Just so I'm clear, the EXCLUDE_PATH env var is dependent on the absolute path of a given log file. If I wanted to exclude all of the kubernetes log files, I would set the env var to EXCLUDE_PATH="[\"/mnt/log/*.log\"]". If I wanted to only exclude the api server, I would set it to EXCLUDE_PATH="[\"/mnt/log/kube-apiserver.log\"]". If I wanted to exclude the docker logs, I would set it to EXCLUDE_PATH="[\"/var/lib/docker.log\"]".

One final thought, it seems like it might also be sensible to include exclude_namespace in sources.kubernetes. If I wanted to exclude all of the kube-system namespace I'd be surprised to find out I had to use exclude_path instead.

@flah00 : totally fine to update the readme, thank you!  You are correct on the EXCLUDE_PATH, and I believe the tail plugin can accept a comma separated list.   The reason I like this approach is just to save unnecessary processing in fluentd.  With regards to adding the namespace, that is completely sensible and I agree with you.  Thank you again for your help on this.

@flah00 thank you so much for this.   I will update when it is pushed to our Docker Hub.


t2 version: chock full of unnecessary text

t2 version only deals with one connected (either explicitly specified or implied) Tessel 2 at a time. With that in mind, the additional text: Tessel [name] is unnecessary clutter that can be removed. Compare with t2 wifi

Suggesting:


Pointing information is not read anymore from files produced with ctapipe versions older than 0.21:

Due to the changes in the HDF5EventSource here:
https://github.com/cta-observatory/ctapipe/pull/2438
telescope pointing information is not read anymore from files produced with older versions of ctapipe than 0.21 using newer versions of ctapipe.
This is silent and results in pointing being nan, which also doesn't create an exception, but results in geometry['HillasReconstructor'].is_valid == True with all reconstructed values being nan.

This affects for example the public dl1/dl2 dataset, which was produced with 0.17


App should provide some feedback (spinner) while waiting for data to load

Feedback
Here is example, look e.g. at 00:00:06 there is missing picture and no indication that app is loading it, and it is isn't until 00:00:15 (and that is on the fast wifi!) until picture is loaded and the user is left wondering in the meantime what is happening.
https://github.com/commons-app/apps-android-commons/assets/156656/949ce642-17f8-4fce-b199-b1be464b0a41
Another situation is e.g. waiting for categories (see https://github.com/commons-app/apps-android-commons/issues/5316)
In general, procedure when app tries to do some user-initiated (i.e. UI thread) potentially-time-consuming-task (e.g. network-oriented) should be:

show the "loading..." spinner image (nicer if animated, but not a requirement, can be simple "Please wait..." text on the middle of the screen too)
start the (network, or other potentially time consuming) task
wait for task to finish
remove the spinner

Wiki username
Mnalis
Device name
Samsung Galaxy S23+
Android version
Android 14 (OneUI 6.1)
Commons app version
5.0.1~af028cbdd (latest F-droid)

I would like to work on this with @karyotakisg. Could we get assigned?

@vtalos @karyotakisg It is yours, thanks! :-)


Use mysql user and password from args.

Needed when "copy_data_from" database has username and password.

Coverage decreased (-0.2%) to 82.258% when pulling 08c979ff1ca5868069355eb408fec806c1a64975 on kovacsbalu:feat-user-passwd into 815394f7fde48cd2faacdafa74e8bff7372d1d63 on tk0miya:master.

As you know, Mysqld.dsn() can take username and password as argument:
with testing.mysqld.Mysqld() as mysqld:
    db = _mysql.connect(**mysqld.dsn(user='user', passwd='passw0rd'))

So this change is not needed, I think. What do you think?

I have my own database with user and password and this changes needed for "copy_data_from" argument. Without this fix I got:
pymysql.err.OperationalError: (1045, u"Access denied for user 'root'@'localhost' (using password: NO)")

See 439b57a

Coverage increased (+0.2%) to 82.581% when pulling 439b57aaf02203937120dda4c99a54084917f635 on kovacsbalu:feat-user-passwd into 815394f7fde48cd2faacdafa74e8bff7372d1d63 on tk0miya:master.

Sorry for late response.
I understand what you did. Merged just now.
Thank you for contribution!

No problem. Thanks for the merge! :)


Option value from initial select is unavailable in multiselect ?

Say i have my initial select :
<select multiple="multiple" id="my-select" name="my-select[]">
<option value="12">A</option>
<option value="20">B</option>
<option value="33">C</option>
</select>

When i initialize with the basic
(function ($) {
$('#my-select').multiSelect();
}(jQuery));

i have :

A

B

C

and

A

B

C

I want to dynamically (with provided javascript conditions) disable / enable some selectable or selection items, but i can only access the option value : 12, 20, 33. The problem is they are not anywhere in the
, even as a data attribute !
I would like to do something like :
$('#my-select option[value="20"]').attr("disabled", true);
but to make itbahave like this i have to do :
$('#1598-selection').addClass('disabled');
which is impossible because i can't match 20 and 1598-selection.

How come we can't access the initial option attribute in the list ?
How to dynamically target a , after the initialization, if i only know the option value of the original select ?

I've seen in the code that it should be "targetable" :
https://github.com/lou/multi-select/blob/master/js/jquery.multi-select.js#L117
and
https://github.com/lou/multi-select/blob/master/js/jquery.multi-select.js#L122
but in reality, for some reason the data attribute is not set.
I've created a fiddle
https://jsfiddle.net/agg393fn/


Training function

Todo

[x] Create a for loop for epoch
[x] Change optimizer function to class
[x] Change loss return type to a dictionary of all losses
[x] Add saving network code
[x] Wrap the code inside a train function.
[x] Make a wrapper class for scheduler.
[x] Add the mermaid flow chart to docs
[x] Add loss graph
[x] Fix the bug for epochs in the loss history
[x] Fix eval dictionary
[x] Plot mAP scores

Flow chart
flowchart TD
    Z[CNN] -- outputs --> A[network_head] --pred_head --> B[head_activations]
    B -- pred_head_act --> C[best_box]
    C -- best_pred_head --> D[extract_head]
    D -- pred_data --> E[non_max_suppression]
    D -- pred_data --> F[mean_average_precision]
    E -- pred_data_nms --> F

Find a way to sum up running_;loss to compute epoch_loss for each of the types of losses in each batch. Then, add the epoch losses to the dictionary of all_losses

Maybe include a variable running_total_loss, running_box_loss, ... such that they are computed in each epoch and then after the epoch, they are appended to the all_losses dictionary.

In the training on dev dataset, mAP gives 0.0 for all the train and valid epochs. Though, the unit tests are running correctly, and this could be the issue due to small train and valid datasize.
I will commit the training code in its present state as it is completing the required functions.


Please un-use scons

It's a bit annoying to get that stuff working in a python3 world in 2016.

I agree and plan to do this. No timeframe.

This was completed; Scons is deprecated, and there are OS-specific builds in build/.

thanks much


Defect: src/tests/unit/simple/test1Caf.F90 AKA increment_my_neighbor fails

Avg response time

Defect/Bug Report
src/tests/unit/simple/test1Caf.F90 AKA increment_my_neighbor fails (at least when oversubscribed @ 32 cores)
I have spent some time looking at this and can't convince myself that this is not a logic error in the test itself. So it could be a bug in the test or a bug in the library, likely due to a race condition, if that's the case.

OpenCoarrays Version: 1.9.0-5-g232d234
Fortran Compiler: GFortran 7.1
C compiler used for building lib: GCC 7.1
Installation method: FC=gfortran-7 CC=gcc-7 cmake ..
Output of uname -a: Darwin IBBs-MBP.local 14.5.0 Darwin Kernel Version 14.5.0: Tue Apr 11 16:12:42 PDT 2017; root:xnu-27<IP_ADDRESS>.3~1/RELEASE_X86_64 x86_64
MPI library being used: MPICH 3.2
Machine architecture and number of physical cores: Intel_64 @ 4 cores
Version of CMake: 3.8.2

Observed Behavior
Test fails when oversubscribed at 32 images
Expected Behavior
Test passes
Steps to Reproduce
Uncomment relevant line in CMakeLists.txt, L568 currently.

I have noticed several tests failing when over prescribed. Failing in the sense of running forever and I manually terminated. This could be because the test make no sense at higher numbers of processes. I will post what I am seeing later.

Some tests require a specific number such as power of two

I have been running some tests on this case. With -np 4 I can get a fail about every 100 runs or so.  The fail rate goes up with increased -np.
I can eliminate failures by inserting a 'sync all' here:
me = this_image()
np = num_images()
sync all
left  = merge(np,me-1,me==1)
right = merge(1,me+1,me==np)

Building with (MPICH 3.3, GCC/GFortran 8.3 all from mac Homebrew):
export FC="$(which gfortran-8)"
export CC="$(which gcc-8)"
cmake -Wdev -DCMAKE_BUILD_TYPE:STRING=Debug -DCMAKE_Fortran_FLAGS:STRING="-g -fbacktrace -fcheck=bounds,pointer" -DCMAKE_C_FLAGS:STRING="-g -fstack-check" ..
make -j

and then testing with:
bin/cafrun -np 50 bin/OpenCoarrays-2.6.1-11-g84ea96a-tests/increment_my_neighbor

Reliably causes failures on my work iMac
intel core i-5 4690 @ 3.5 GHz, 4 cores, 4 threads.

I have attached a full debug log of the runtime failure.
increment_my_neighbor.50img.fail.txt


[js] Add guidance about avoiding subclassing across package boundaries.

I attempted to explain the problem clearly, but I feel like the language can be cleaned up a bit. Suggestions are welcome.

@bterlson could you please review so we can try to get this PR in?

@bterlson plz

@joheredi @witemple-msft can both of you take a look at this and give me your thoughts?


Add a page of the types of jobs of a professional and add tests

… tests
A list of the types of jobs each professional has been created.
Every professional can update and delete the type of job
The changes that were made:
1.I added a type of job page
2.I added a page where you can edit job details
3.I added a page to cancel a type of job
4.Add base.html
5.Add tests
Resolve #95

Please split the form tests validity from the behavior of data flow

Done @Yarboa


docker 25: MANIFEST_BLOB_UNKNOWN: blob unknown to registry

I'm using create image-bundle command and getting this error:
 ✓ Parsing image bundle config
 ✓ Creating temporary directory
 ✓ Starting temporary Docker registry
 ✗ Pulling requested images [                                    0/13] (time elapsed 09s)
PUT http://<IP_ADDRESS>:33571/v2/XXX/manifests/sha256:9a4a8f7234957bc76c3a8b062070e429a1b8a23c36c24e5d6a26e9b1d91bc410: multiple errors returned:
MANIFEST_BLOB_UNKNOWN: blob unknown to registry; sha256:064b40dd2dfb098da5b4d43876da5d481047121e1ed560129383b74d30b5fe99;
MANIFEST_BLOB_UNKNOWN: blob unknown to registry; sha256:6626c9b5b2de04d53cccd1235e03fcb18f812284b61ce453859d282ce8d19cd0;
MANIFEST_BLOB_UNKNOWN: blob unknown to registry; sha256:7b92e5eddc133237670f4c75659800f8bd0c3384a713318c79d0531f91b696f2;
MANIFEST_BLOB_UNKNOWN: blob unknown to registry; sha256:5ace0b795b80b845f07ef9bb3cc42b3fcab0ec334dd92b68c7d9d710c5986ed4;
MANIFEST_BLOB_UNKNOWN: blob unknown to registry; sha256:0ca3d0f7e3e1281d6a3d1ce1a872b6f0761d70842e6e57a02ba6806e9f5557e6;
MANIFEST_BLOB_UNKNOWN: blob unknown to registry; sha256:b70bbbe162477aeb68d5c19a3d7d4da389c537b04d11eeabbe35b651314b8a89;
MANIFEST_BLOB_UNKNOWN: blob unknown to registry; sha256:b5f71c00b65306805e43337cc17e101eaf23e050afe4f3a0c6647f87f48f6fe6;
MANIFEST_BLOB_UNKNOWN: blob unknown to registry; sha256:b5f71c00b65306805e43337cc17e101eaf23e050afe4f3a0c6647f87f48f6fe6;
MANIFEST_BLOB_UNKNOWN: blob unknown to registry; sha256:20bfa07587c5c05b57c64bce2a61ca831ae4641161f5c717ceb233579847a76b;
MANIFEST_BLOB_UNKNOWN: blob unknown to registry; sha256:255c33047a850954ffbd0de23449674d8dce49b748e2af90afd1b8a1f1824a0a;
MANIFEST_BLOB_UNKNOWN: blob unknown to registry; sha256:fb97d22de48e2b66fbc1a59d651f7cb9e169c9f4a2008320a78012b3fe31f7b4;
MANIFEST_BLOB_UNKNOWN: blob unknown to registry; sha256:ee1d465b6ef5a03fccc178453de8d3b619fbe53635fb9e7c9f8893b98c5e0af9

mindthegap version: v1.13.0
OS: ubuntu 22.04
Docker:
Client: Docker Engine - Community
 Version:           25.0.5
 API version:       1.44
 Go version:        go1.21.8
 Git commit:        5dc9bcc
 Built:             Tue Mar 19 15:05:10 2024
 OS/Arch:           linux/amd64
 Context:           default

Server: Docker Engine - Community
 Engine:
  Version:          25.0.5
  API version:      1.44 (minimum version 1.24)
  Go version:       go1.21.8
  Git commit:       e63daec
  Built:            Tue Mar 19 15:05:10 2024
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.6.28
  GitCommit:        ae07eda36dd25f8a1b98dfbf587313b99c0190bb
 runc:
  Version:          1.1.12
  GitCommit:        v1.1.12-0-g51d5e94
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

Downgraded my docker to this and it worked:
Client: Docker Engine - Community
 Version:           24.0.9
 API version:       1.43
 Go version:        go1.20.13
 Git commit:        2936816
 Built:             Thu Feb  1 00:48:39 2024
 OS/Arch:           linux/amd64
 Context:           default

Server: Docker Engine - Community
 Engine:
  Version:          24.0.9
  API version:      1.43 (minimum version 1.12)
  Go version:       go1.20.13
  Git commit:       fca702d
  Built:            Thu Feb  1 00:48:39 2024
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.6.28
  GitCommit:        ae07eda36dd25f8a1b98dfbf587313b99c0190bb
 runc:
  Version:          1.1.12
  GitCommit:        v1.1.12-0-g51d5e94
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

Thanks for the report! I'm a bit confused why the Docker version matters because under the hood we don't use a container runtime at all 🤔
Could you share an example images config file so I can try to reproduce?

hello, thanks for asking, but I am assuming the project is subject to this issue here: https://github.com/google/go-containerregistry/issues/1870

i think this problem will go away for me, if you make a new release using v0.19 or later of that library

Aha thank you again! I'll get that fixed asap

Released v1.13.1! Please can you try it out?

Fixed in v1.13.1 - closing. Please reopen if this is still causing you issues.

confirmed fix after I updated to docker 26, thanks!

Nice! Thanks for confirming @coopstah13 !


Messages being split - unnaturally in Japanese

Reproduction link
https://stackblitz.com/edit/ng-zorro-antd-ivy-9v7u7r?file=src%2Fapp%2Fapp.component.ts
Steps to reproduce
Create an NZ notification in Japanese, then type any sentence in Japanese.
What is expected?
It is expected that a word in Japanese would be properly segmented and not be cut off mid-word when continuing on a new line when there is no space left in the notification box. Using the example in Stackblitz, the entire last word 日本語 should be in a new line instead of being cut off like:
日本
語
or
日
本語
What is actually happening?
The message in the notification is displayed on a new line, regardless of if it is in mid-word or not.

Environment
Info

ng-zorro-antd
15.0.0

Browser
Edge/Firefox/Chrome

https://stackblitz.com/edit/ng-zorro-antd-ivy-zngvdt?file=src/app/app.component.ts
Hi @DepickereSven. Here is a modified version of your demo. The text wrapping is not determined by ng-zorro-antd but CSS. You can use word-break: keep-all to avoid break the word "日本語".
This issue will be closed since it has nothing to do with zorro.

Thank you for explaining, I wasn't fully sure if it would be something zorro offers or not.


Letsencrypt Support

I would like to use letsencrypt to generate an ssl certificate for a passcore site.
Letsencrypt needs to browse to "http://password.yourdomain.com/.well-known/acme-challenge/x" to verify that a specific extentionless file exists on the web server.
The issue is that passcore is redirecting back to to the web root "http://password.yourdomain.com/" instead of returning the file at "http://password.yourdomain.com/.well-known/acme-challenge/x".
This results in the letsencrypt acme challenge failing and no ssl certificate being issued.
Is there something I can do in IIS to make this work or is it a limitation of passcore?

Is HomeController.cs causing my issue?
https://github.com/unosquare/passcore/blob/84349e9663cffbcf43ed492c3dd66f8431eee68e/src/Unosquare.PassCore.Web/Controllers/HomeController.cs

Hi,
Here is my workaround for this issue.

Stop passcore site
Create a new site in a different path with http(80) binding for your hostname.
Request a certificate for newly created site
Edit passcore site enable HTTPS only and start


Update Japanese translations

Added missing Japanese translations
Change some translations because some Japanese letters had broken

@DekoKiyo Pleae check the lastest version, and update your translation.


profile: when username is too long, it blocks some of the page content

Fixed by @filipesilva - thank you! :)


Unify the casing in all file names

Is your maintenance request related to a problem? Please describe.
We've been back and forth with the casing in our file names. We started out with PascalCase for all files. Later, we diverged for files that contain only a function, or for files that contain multiple things, to cammelCase.
In recent times, we see more and more projects to only use kebab-case for files. I've started doing that in the new instrumenter package.
Describe the solution you'd like
Align with all js projects and use only kebab case.
Describe alternatives you've considered
-
Additional context
I don't know if there is a way to automate this process. However, it has the potential to be a huge merge conflict thing. It is a one time pain and I want to do this AFTER our 4.0 release.

I think it can be checked in linter... I could setup it later :)


Style/dayview

feat: Basic responsive calender day view page
Added dayview template with css and router.
An tempreroy event class with,
processing function for the day view to work.
fixed after taking notes, and lint

Codecov Report

Merging #93 (15baa62) into develop (53d226d) will increase coverage by 0.26%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop      #93      +/-   ##
===========================================
+ Coverage    98.55%   98.81%   +0.26%
===========================================
  Files            8        9       +1
  Lines          207      253      +46
===========================================
+ Hits           204      250      +46
  Misses           3        3

Impacted Files
Coverage Δ

app/main.py
94.11% <100.00%> (+0.36%)
:arrow_up:

app/routers/dayview.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 53d226d...15baa62. Read the comment docs.

Codecov Report

Merging #93 (15baa62) into develop (53d226d) will increase coverage by 0.26%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop      #93      +/-   ##
===========================================
+ Coverage    98.55%   98.81%   +0.26%
===========================================
  Files            8        9       +1
  Lines          207      253      +46
===========================================
+ Hits           204      250      +46
  Misses           3        3

Impacted Files
Coverage Δ

app/main.py
94.11% <100.00%> (+0.36%)
:arrow_up:

app/routers/dayview.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 53d226d...15baa62. Read the comment docs.


Converting keras (.h) models to .tflite fails with error Windows fatal exception: access violation

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
TensorFlow installed from (source or binary):
TensorFlow version (or github SHA if from source):

Command used to run the converter or code if you’re using the Python API
If possible, please share a link to Colab/Jupyter/any notebook.
# Copy and paste here the exact command

The output from the converter invocation
# Copy and paste the output here.

Also, please include a link to the saved model or GraphDef
# Put link here or attach to the issue.

Failure details
If the conversion is successful, but the generated model is wrong,
state what is wrong:

Producing wrong results and/or decrease in accuracy
Producing correct results, but the model is slower than expected (model generated from old converter)

RNN conversion support
If converting TF RNN to TFLite fused RNN ops, please prefix [RNN] in the title.
Any 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.

shit, hit enter too early.. butterfingers :)


Add to PyPI

Love the module and have been using it a lot, but find it really annoying to have to Google dfgui to find the install URL everytime. Any idea when you'll get this puppy onto PyPI? Anything I could do to help with that?

The main reason I didn't do it was because I wasn't sure how I should express the wxpython dependency. Afaik wxpython is not pip installable, so manual installation notes are required anyway, right?

wxpython is on PyPI - https://pypi.org/project/wxPython/
and to be installed, you need to put to setup/install requirements

I know, but pip install wxpython never worked for me, most likely due to its native build dependencies, which simply cannot be handled by pip.

Personally I never had an issue doing pip install wxpython. I've done it on three or so computers now (Windows and Linux) and they've worked fine.

https://github.com/pypa/pip/issues/3627#issuecomment-214298817 states wx on pypi is merely a place holder and redirects to https://wiki.wxpython.org/How to install wxPython for installation instructions

The inability to get wx from pypi has actually inspired me to work on a successor. It might be a bit too early to announce it because it's nowhere near finished, but if you want to take an early peek: tabloo.
The idea is the same import tabloo; tabloo.show(df), but it runs a UI in a browser so all dependencies are easily pip installable. The currently available features are still very basic

but I plan to bring back more of the dfgui features in the next weeks. At least it's on already on PyPI.


Support authorization code flow for Google OAuth

I have a requirement to store the access token and refresh token issued by Google when a user signs into my application in order to be able to perform requests to Google Classroom API on behalf of the user, after they've granted the relevant scopes to my app.
In order to do so, I would need the frontend application to send an authorization code, as opposed to a simple access token (which I cannot refresh on my backend) https://developers.google.com/identity/protocols/oauth2/web-server#exchange-authorization-code
However, I can see that the way the convert-token endpoint works, it requires a token parameter to be in the request.
Is there any way to support the authorization code flow, where the request simply contains an authorization code, I exchange it on my server for an access token + refresh token from Google, store them, then create the normal in-house access token and return it to the user? The last part is the same exact flow as the normal convert-token endpoint, but I first need to exchange the authorization code for access + refresh token.
Thank you in advance.

@samul-1 please be more concrete in your question. Explain how your pipeline works.
Your link is broken, it's just pointing to this repo's issues, please verify your link again.

@samul-1 please be more concrete in your question. Explain how your pipeline works. Your link is broken, it's just pointing to this repo's issues, please verify your link again.

I fixed the link.
This issue has little to do with how my pipeline works.
Google OAuth2 authentication provides two flows:

client-side authentication (the one supported by this library), where the frontend is redirected to Google's consent page, completes the login, and obtains an access token, which is then sent to drf-social-oauth2, which verifies it and generates an in-house token
server-side authentication, or authorization code flow: in this case, the frontend completes the login and just obtains an authorization code from Google, which it then sends to the application backend. The application backend then exchanges that code for an access token and a refresh token. This allows the backend to store & use those tokens for making API calls to Google using the user's credentials I haven't been able to find a way to do this in drf-social-oauth2, despite the docstring in the ConvertTokenView says the authorization code flow is supported.

This is not a new request. Please see this PR made on the repo this project forked from: PR, and the issue mentioned here: issue.
I hope this is clearer now. I doubt this issue should've been closed to begin with, as I asked about the "authorization code flow", which is a standard term, not something I made up in my comment.

Hey all, any progress on this?

I also needed to store the refresh_token on the backend, and manage to do it with this workaround
class ConvertTokenSerializer(Serializer):
    grant_type = CharField(max_length=50)
    backend = CharField(max_length=200)
    client_id = CharField(max_length=200)
    token = CharField(max_length=5000)
    refresh_token = CharField(max_length=5000)  # <----- we add the refresh token to the serializer inputs

class ConvertTokenView(BaseConvertTokenView):

    def post(self, request: Request, *args: Any, **kwargs: Any) -> Response:
        serializer = ConvertTokenSerializer(data=request.data) # <---- we use our custom serializer
        serializer.is_valid(raise_exception=True)
        # Use the rest framework `.data` to fake the post body of the django request.
        request._request.POST = request._request.POST.copy()  # type: ignore
        for key, value in serializer.validated_data.items():
            request._request.POST[key] = value  # type: ignore

        try:
            url, headers, body, status = self.create_token_response(request._request)
        except InvalidClientError:
            return Response(
                data={"invalid_client": "Missing client type."},
                status=HTTP_400_BAD_REQUEST,
            )
        except MissingClientIdError as ex:
            return Response(
                data={"invalid_request": ex.description},
                status=HTTP_400_BAD_REQUEST,
            )
        except InvalidRequestError as ex:
            return Response(
                data={"invalid_request": ex.description},
                status=HTTP_400_BAD_REQUEST,
            )
        except UnsupportedGrantTypeError:
            return Response(
                data={"unsupported_grant_type": "Missing grant type."},
                status=HTTP_400_BAD_REQUEST,
            )
        except AccessDeniedError:
            return Response(
                {"access_denied": "The token you provided is invalid or expired."},
                status=HTTP_400_BAD_REQUEST,
            )
        except IntegrityError as e:
            if "email" in str(e) and "already exists" in str(e):
                return Response(
                    {"error": "A user with this email already exists."},
                    status=HTTP_400_BAD_REQUEST,
                )
            else:
                return Response(
                    {"error": "Database error."},
                    status=HTTP_400_BAD_REQUEST,
                )
        except Exception as e:
            return Response(
                {"error": str(e)},
                status=HTTP_500_INTERNAL_SERVER_ERROR,
            )

        return Response(data=json_loads(body), status=status)

class ServiceOAuth2(OpenIdConnectAuth):
    name = "service"
    ...

    def user_data(self, access_token: str, *args: Any, **kwargs: Any) -> UserData:
        data: UserData = self.get_json(
            "https://service.net/core/connect/userinfo",
            headers={"Authorization": f"Bearer {access_token}"},
        )
        return {**data, "refresh_token": self.data.get("refresh_token")} # <---- we add the refresh token to the user data, so this is stored in DB


Use link checker csv service for rake task

With the new integration with the Link Checker API, and the addition of the nightly check for broken links on public editions in https://github.com/alphagov/whitehall/pull/3574 (the results from these checks are saved in the LinkCheckReports table), the BrokenLinkReporter is no longer required.
The rake task now uses the LinkReporterCsvService class added in https://github.com/alphagov/whitehall/pull/3576 which uses the edition's LinkCheckReports to populate the CSV file with the broken links for each edition.

We also probably shouldn't merge or deploy this until #3574 is running.


Converter to use TaskMerger not TaskMaker

Finally was able to do some validation to make sure of some unknowns:

Both Merger and Maker pass validation for data integrity
The Maker version also takes 6 times as long to run compared to Merger.
Resultant segment sizes were comparable but not equal.

So this reduces the execution time by ~80%
This depends on PRs

~~https://github.com/druid-io/druid/pull/1351~~
~~https://github.com/druid-io/druid/pull/1403~~

in my mind, switching to indexmerger instead of maker should be just changing some calls. What do the rest of the changes encapsulate?

@fjy Dependent PRs.
I didn't do qualifications on the PRs independently, I did all my checks against this particular set of commits because I figured the others would be going in.
Once the others are in I'll rebase and this one will only be the simple change you are speaking of.

@fjy Here's the commit that this PR will end up being once all the dependent PRs are merged:
https://github.com/metamx/druid/commit/d5660d44d6878d863c7ea96d5e695bcb38b89ca9

@drcrallen that makes sense, cool, going through the dependent PRs right now

:+1:

@fjy @drcrallen it worries me a bit that segment sizes were not identical. It would be nice to get to the bottom of this.

@xvrl I think it is probably treatment of nulls/empty strings, in any case IndexMerger is what we use for everything prod like

ok, we probably need better tests for this, but if everyone's comfortable that's fine with me.


anime results is strange

I crop part of the example image as input without other modification, the result becomes very strange. Any idea?

The anime model is experimental because we use fixed resolution narrow domain images in training, which are 512x768 portraits only. Thus performance may degrade when other resolutions and other domain images are used.