data points in thegraph

@find out how to do a query into event.
This is an example (the address is mine) of withdrawing (claiming) 482 Uni token, observe the event:
https://cn.etherscan.com/tx/0x68a7c154b6dbc38505612f0c0738628f5f9d869da8372edf824feae7576026e8#eventlog
Address
    0x090d4613473dee047c3f2706764f49e0821d256e

Name

    Claimed (uint256 index, address account, uint256 amount)View Source

Topics

        0 0x4ec90e965519d92681267467f775ada5bd214aa92c0dc93d90a5e880ce9ed026

Data

        index :220687
        account :0xbc8dafeaca658ae0857c80d8aa6de4d487577c63
        amount :482974914000000000000

This address only withdrew 400 Uni in total, but there are others who repeatedly drew Uni.

Execute the following query on this endpoint: https://explorer.bitquery.io/graphql/CUAEuPFx2m
query(
  $from: ISO8601DateTime,
  $till: ISO8601DateTime,
){
  ethereum {
    smartContractEvents(
      options: {desc: "date.date"},
      ##smartContractEvent: {is: "NameRegistered"},
      smartContractAddress:
      {is: "0x090d4613473dee047c3f2706764f49e0821d256e"}
      date: {since: $from, till: $till},

    ) {
      date{
         date:startOfInterval(unit:day)
      }
      smartContractEvent{
        __typename
        name
      }
      times: count
      uniqueCallers: count(uniq: callers)
    }
  }
}

Variables:
{
  "from":"2019-01-01",
  "till":null
}

Will refine the query later.

The following query get zero result set. Investigating...
query(
  $from: ISO8601DateTime,
  $till: ISO8601DateTime,
){
  ethereum {
    smartContractEvents(
      options: {desc: "date.date"},
      ##smartContractEvent: {is: "NameRegistered"},
      smartContractAddress:
      {is: "0x090d4613473dee047c3f2706764f49e0821d256e"}
      date: {since: $from, till: $till},
      smartContractEvent: {is: "Cliam"},

    ) {
      date{
         date:startOfInterval(unit:day)
      }
      smartContractEvent{
        __typename
        name
      }
      times: count
      uniqueCallers: count(uniq: callers)
    }
  }
}


per proto snat programming in standard

@christiankuhtz : Thanks for your contribution! The author, @KumudD, has been notified to review your proposed change.


Features for targets

I think it would be beneficial to have a way to "attach" a compilation target to a feature, so it will be compiled only if that particular feature would be enabled. I know it's possible to use #![cfg(feature="foo")] to achieve a similar effect, but for some use cases this would be better. cargo-extras crate, for instance, could use it, as none of the included binaries actually reside in that crate.
[[bin]]
name = "foo"
path = "src/bin/bar.rs"
feature = "foo"

[features]
foo = []

I think that this is a dupe of https://github.com/rust-lang/cargo/issues/1570, so I'm gonna close in favor of that. Thanks for the report though!


nxp: patches: linux-imx: Read ACK data to host boot the ADSD3500 SPI/I2C

ADSD3500 SPI/I2C linux driver patches to support the read acknowledgement data on every header and payload packet of ADSD3500 host boot firmware

Spaces were added in the conditional statement and comments for opcode comparison. The ADSD3500 SPI and I2C Linux driver source files have been added.


Can't read local、independent Scalar variables.

Using fmi4j read BouncingBall.fmu :
AbstractFmu fmu = AbstractFmu.from(fmuFile); ModelDescriptionProvider md = fmu.getModelDescription();  ModelVariables variables = md.getModelVariables(); List<TypedScalarVariable<?>> scalarVariables = variables.getVariables();
But it seems like scalarVarables dose not contains variables which causality attribute is independent and local. Is this intentional or a bug?

I can't be sure without a deep dive into the code, but what use do you have for accessing the independent variable?

I can't be sure without a deep dive into the code, but what use do you have for accessing the independent variable?

Using a for loop like:
AbstractFmu fmu = AbstractFmu.from(fmuFile); ModelDescriptionProvider md = fmu.getModelDescription(); ModelVariables variables = md.getModelVariables(); List<TypedScalarVariable<?>> scalarVariables = variables.getVariables();  for (TypedScalarVariable<?> var : scalarVariables){      Causality varCausality = var.getCausality();      if(varCausality == Causality.Local){          //no variable which causality is local     } }
The BouncingBall.fmu has a local variable der(h) and der(v), but using code above can't read the two variables.

Not sure why the above does not work, but it should be easier to access the variable name or valueReference. And it should also work.


Temporary fix for #41 - memory issue with reminder e-mails

limit the number of entries handled at a time, drasticly reduces the
time and memory the entitymanager needs to process the updates.
command had a --time-limit option that will make the command stop after
a number of seconds.

eg. The following command could be used in combination with a crontab set
to run each minute
php app/console intracto:mails:reminders:send --time-limit=50 --force
Entries are processed in batches (default 10, configurable) and a single flush of the spool is done at the end of the batch. Keeping batches small is advised (run more often with smaller batches).
A permanent fix/rewrite of the batch e-mail system is also in the make (enhancement for 2016?).

Thanks Tom!
But, before this can be merged, we must first investigate why some people get a mail that "the wishlist is updated" but nothing is in the list (often reported). Or even get the wishlist of someone else (often reported). So must be checked what's going on there.
Apart of that, I'm only merging the really essential things right now. We have 2 weeks to go before this season ends. Let's just do that first ;-)

This is just to fix the current problem we are facing on production.
I agree on dropping the Wishlist notifications if these are not reliable (that's just a single line that need to be commented out), but are we also dropping the notifications to remind the user to actually look at the entry? I do believe that these are very relevant, definitely in these last few weeks of the season. Other than a simple reminder, it also provides the user with the link to their entry.

Ah. Agree with that. Can you merge / deploy, enable the cron again and follow up if all goes well? That would help me a lot right now. :-)

:+1:


[Improve][Mysql-CDC]Fix Mysql-CDC option rule.

Purpose of this pull request

table-names and table-pattern only require one parameter.
Does this PR introduce any user-facing change?

How was this patch tested?

Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

Thanks @lightzhao . Could you check other CDC connector too?

ok.


Mark &mut BLEAdvertising as Send?

Is it safe for &mut BLEAdvertising to be marked with Send similar to https://github.com/taks/esp32-nimble/commit/046882663252b08626744cb1ee3d9fffa5ef1fa2? I want to access the advertising from the on_write callback. Currently I'm using channels, but it would be more convenient if I could just use Arc.
This is related to #17

Is it possible to paste the main.rs source code?

Is it possible to paste the main.rs source code?

Here is the code which produces that error:
https://github.com/ChocolateLoverRaj/rust-esp32c3-examples/blob/ae197aef46464ed5950683d3d34ae638a13e2c79/sensor-connect/src/main.rs#L132
it's not a minimum example. Let me know if I should make a minimum example.

I found out a much simpler way to reproduce the error: https://github.com/ChocolateLoverRaj/esp32-nimble/blob/d56af552fe6f8def6c951012b18c08ebff5dea03/examples/ble_secure_server.rs#L25

Is it possible to paste the main.rs source code?

Here is the code which produces that error: https://github.com/ChocolateLoverRaj/rust-esp32c3-examples/blob/ae197aef46464ed5950683d3d34ae638a13e2c79/sensor-connect/src/main.rs#L132
it's not a minimum example. Let me know if I should make a minimum example.

I corrected the error in the project you attached.
Please check it.
https://github.com/taks/rust-esp32c3-examples/tree/fix-build-error/sensor-connect

I found out a much simpler way to reproduce the error: https://github.com/ChocolateLoverRaj/esp32-nimble/blob/d56af552fe6f8def6c951012b18c08ebff5dea03/examples/ble_secure_server.rs#L25

It should work if you change _server to server on line 16.

Thanks for fixing the error. I noticed that you used esp32_nimble::utilities::mutex::Mutex instead of std::sync::Mutex. Why?


Rename the package name of master to coordiantor

What changes are proposed in this pull request?
Rename the package name of master to coordiantor
Why are the changes needed?
Make a separate package for the lightweight coordinator in OS code
Does this PR introduce any user facing changes?
No

@jiacheliu3 do you think this will make your cherrypicks hard to merge?
@beinan i am good overall with this change. just want some opinions from others. @Xenorith @apc999 @yyongycy
@yyongycy i know you were not totally onboard with this style of changing packages. This gives us options of using entirely ee code or extending some os coordinator package.  By the release time, we should have no duplicated code.

@jiacheliu3 do you think this will make your cherrypicks hard to merge? @beinan i am good overall with this change. just want some opinions from others. @Xenorith @apc999 @yyongycy
@yyongycy i know you were not totally onboard with this style of changing packages. This gives us options of using entirely ee code or extending some classes in the OS version. By the release time, we should have no duplicated code.

Is there no longer issue regarding the presto local cache compatibility now?

@jiacheliu3 do you think this will make your cherrypicks hard to merge? @beinan i am good overall with this change. just want some opinions from others. @Xenorith @apc999 @yyongycy
@yyongycy i know you were not totally onboard with this style of changing packages. This gives us options of using entirely ee code or extending some classes in the OS version. By the release time, we should have no duplicated code.

Yeah this can make my cherry-pick very hard. Git can identify a code file location has changed and move that to the updated directory, but git is NOT able to change the package and import in the code for me. So probably i will then have to manually update those code otherwise nothing compiles.
That said, can we possibly hold this rename off by 1-2 weeks? By then likely I've finished my cherry picking, then merging this will only affect PRs merged later into master-2.x(not as many as the 200 commit diff now but i don't know how many more PRs will go in master-2.x in the coming 2 months).


ModuleNotFoundError: No module named '_simbody'

Hello everyone !
I am new to python and opensim, so... ,sorry if my questions are "simple".
I'am currently using OpenSim 4.2. (64 bit) and  Python 3.7.10 (64 bit).
When I try to import the opensim package into Python I have this error message :
import opensim
Traceback (most recent call last):
File "D:\Anaconda3\envs\OpenSim\lib\site-packages\opensim-4.2_2021_03_12_fcedec9-py3.7.egg\opensim\simbody.py", line 14, in swig_import_helper
return importlib.import_module(mname)
File "D:\Anaconda3\envs\OpenSim\lib\importlib_init_.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1006, in _gcd_import
File "", line 983, in _find_and_load
File "", line 967, in _find_and_load_unlocked
File "", line 670, in _load_unlocked
File "", line 583, in module_from_spec
File "", line 1043, in create_module
File "", line 219, in _call_with_frames_removed
ImportError: DLL load failed: Le module spécifié est introuvable.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 1, in
import opensim
File "D:\Anaconda3\envs\OpenSim\lib\site-packages\opensim-4.2_2021_03_12_fcedec9-py3.7.egg\opensim_init_.py", line 1, in
from .simbody import *
File "D:\Anaconda3\envs\OpenSim\lib\site-packages\opensim-4.2_2021_03_12_fcedec9-py3.7.egg\opensim\simbody.py", line 17, in
_simbody = swig_import_helper()
File "D:\Anaconda3\envs\OpenSim\lib\site-packages\opensim-4.2_2021_03_12_fcedec9-py3.7.egg\opensim\simbody.py", line 16, in swig_import_helper
return importlib.import_module('_simbody')
File "D:\Anaconda3\envs\OpenSim\lib\importlib_init_.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
ModuleNotFoundError: No module named '_simbody'

having a very similar error with python 3.6.1. Did you fix this?

Hi @oskarasbrink ,
I just managed this by changing the installation folder name (frome OpenSim 4.2 to Opensim4.2)  and then by installing twice the python setup... Weird but it worked

Did it work for both of you? I haven't managed to fix it, I'm working with my mac.

I tried with 2 different computers (both windows) and it worked.
You may have a problem with admin rights and/or on the installation folder (can't help more I'm not familiar at all with mac)


[Pixi] - Le Iene AMP First Site Homepage suffering from CLS on AMP Cache

Hi all, I've noticed on Pixi that this AMP First Site Homepage https://www.iene.mediaset.it/ is suffering from Field CLS 0.16.

By Testing the AMP Cache URL https://www-iene-mediaset-it.cdn.ampproject.org/c/s/www.iene.mediaset.it/ on Chrome DevTools LH Audit it seems like these elements are causing the Layout Shifts:

Can anybody advice if the issue is happening because of an AMP Core component or guidance on Pixi can be improved on similar outcomes?
/cc @ampproject/wg-performance

@gilbertococchi  Hi Gilberto, we ask that you not create issues in this repo directly. The Page Experience tool creates reports automatically when a page has no recommendations and does not meet the CWV standard. In your case I see the tool has some recommendations and suggest you try those. In particular your CLS issue seems to be related to font swapping (try removing the deprecated amp-font component and adding font-display: swap. If you find a general performance related issue with AMP, please create an issue in the amphtml repo rather than here. Thanks !


Canary Islands is not present in the library

Some users come to us from the Canary Islands, but we can't validate them because this library does not have Canary Islands ISo code representation (IC).
It would be good to add it.

It is missing because it is a 3166-2 subdivision code and this library only offers 3166-1 codes. I have no intention of supporting 3166-2 subdivision codes at the moment because then they should all be implemented/represented, and that is a lot of work. Perhaps if it could be automated by parsing official data sources. But that is not the case right now.


Can't pass value-less options?

Some ffmpeg options don't take any values.
In this example '-an' (it's the disable audio option).
So how can we pass them on?
with: input_stream = ffmpeg.input('input.mp4')
I tried:
output_stream = ffmpeg.output(input_stream, 'output.mp4', 'an')
output_stream = ffmpeg.output(input_stream, 'output.mp4', an='')
output_stream = ffmpeg.output(input_stream, 'output.mp4', **{'an': ''})
None of the 3 worked.

Try an=None.
We should probably add an example of that in the readme. And also add support for an=True, as I think that might be more intuitive.

And also add support for an=True, as I think that might be more intuitive.
Just as a note, I tried an=True just out of intuition
an=None worked though!

There is an example about this: Mono to stereo with offsets and video
There, shortest=None is used.
Maybe this could be made more explicit, otherwise the issue could be closed.

I, too, tried y=True before finding the answer here.


Upgrade to jinja2 3.0

Note: draft PR. Jinja2 3.x currently has a bug that breaks a lot of our templates. We need to wait for that to be fixed before proceeding with this upgrade.
This PR updates us to django-jinja 2.8.0 and Jinja2 3.0.x (yes, that's a confusing version number).
Not huge amounts has changed within these upgrades but until the above bug is fixed I can't test this properly with all our projects. So need to sit on hands for a bit with this, but hopefully once Jinja2 updates this will be good to go.

See #163.


Limit how long we can check other servers

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

Project coverage is 95.19%. Comparing base (87bfd23) to head (4648f82).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1138      +/-   ##
==========================================
+ Coverage   95.16%   95.19%   +0.03%
==========================================
  Files         102      102
  Lines       10236    10270      +34
==========================================
+ Hits         9741     9777      +36
+ Misses        346      345       -1
+ Partials      149      148       -1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Settings repo

Settings repo (as already discussed) → for separation between the settings and the device code (also see Plan for Service Structure
Also relevant for the repos that the Research Assistants are writing. The later it comes the more they need to rewrite afterwards.

I try to make all things configurable through the config module. The user can set the config directory using the environment variable $SERVICE_CONFIG_DIR which defaults to the "config" folder in the root folder of the service. This folder is used to store the web settings. You can also use this folder to store your personal service config (access to the path is provided through ServiceConfig().config_dir).
If you think there are still some things missing, feel free to reopen the issue.


GH-1087: improvement debug logging

Closes #1087
Debug information about the sent message was recorded in the log before processing by MessagePostProcessors, so the information on the console might not coincide with reality.
Added log of the message after processing by MessagePostProcessor

LGTM

Merged as https://github.com/spring-projects/spring-amqp/commit/3f65fc01cf43f1a5710066e83e457886e18a2b73.
@PlatonovLex ,
thank you very much for the contribution.
Please, pay attention how I modified a commit message.
Not all are looking into GitHub for understanding what is going on.
And not all tools can resolve properly issue numbers to their links.
So, it would be better to always explain the reason and fixes exactly in the commit messages.
When you open Pull Request here on GH, its first commit message becomes a description of the PR.
Therefore it is one more benefit to always write good commit messages.
See more info in Chris Beams' article: https://chris.beams.io/posts/git-commit/

OK, I understood.
And thanks for the good article. Next time I will definitely use your recommendations.
Thank you again, guys.


Make parameters a keyword-only argument

I was thinking about whether we should make params a keyword only argument. Perhaps yes, but then not in this PR (since we should then probably enforce it all at once for every Params-like object).

Originally posted by @N-Wouda in https://github.com/N-Wouda/PyVRP/pull/168#discussion_r1125506972

Decided not to implement this, see #174.


[modify] modified to turn edit mode off when click approve

Coverage decreased (-6.2%) to 78.447% when pulling 6420afad0ea717406d825e0f9c8bc59ffb9911da on HiraiY:fix/edit_mode_off_click_approve into d89912b6b11d83cd12ddc78fb771da19b22b0289 on shirasagi:master.


Error getting data

Hi Sergio,
Although all worked for quite a while, got some errors lately
Working on domoticz version 2022.01  build hash c956851b running via docker.
Have installed requests and simplejson
Any thoughts on what might go wrong?
Many thanks in advance!
2022-07-17 17:26:48.031  Error: Airco: 'onHeartbeat' failed 'JSONDecodeError':'Expecting value: line 1 column 1 (char 0)'.
2022-07-17 17:26:48.031  Error: Airco: Exception traceback:
2022-07-17 17:26:48.031  Error: Airco:  ----> Line 283 in '/config/plugins/domoticz_panasonic_CZ-TACG1/plugin.py'
2022-07-17 17:26:48.031  Error: Airco:  ----> Line 199 in '/config/plugins/domoticz_panasonic_CZ-TACG1/plugin.py'
2022-07-17 17:26:48.031  Error: Airco:  ----> Line 353 in '/config/plugins/domoticz_panasonic_CZ-TACG1/plugin.py'
2022-07-17 17:26:48.031  Error: Airco:  ----> Line 357 in '/usr/lib/python3.8/json/init.py'
2022-07-17 17:26:48.032  Error: Airco:  ----> Line 337 in '/usr/lib/python3.8/json/decoder.py'
2022-07-17 17:26:48.032  Error: Airco:  ----> Line 355 in '/usr/lib/python3.8/json/decoder.py'
2022-07-17 17:26:58.118  Error: Airco: 'onHeartbeat' failed 'JSONDecodeError':'Expecting value: line 1 column 1 (char 0)'.
2022-07-17 17:26:58.118  Error: Airco: Exception traceback:
2022-07-17 17:26:58.118  Error: Airco:  ----> Line 283 in '/config/plugins/domoticz_panasonic_CZ-TACG1/plugin.py'
2022-07-17 17:26:58.118  Error: Airco:  ----> Line 199 in '/config/plugins/domoticz_panasonic_CZ-TACG1/plugin.py'
2022-07-17 17:26:58.118  Error: Airco:  ----> Line 353 in '/config/plugins/domoticz_panasonic_CZ-TACG1/plugin.py'
2022-07-17 17:26:58.118  Error: Airco:  ----> Line 357 in '/usr/lib/python3.8/json/init.py'
2022-07-17 17:26:58.118  Error: Airco:  ----> Line 337 in '/usr/lib/python3.8/json/decoder.py'
2022-07-17 17:26:58.118  Error: Airco:  ----> Line 355 in '/usr/lib/python3.8/json/decoder.py'

Hi Sergio,
Here the same error since a couple of days, any ideas? Thx
curl -v 'https://accsmart.panasonic.com/auth/login' -s --http1.1                                  -X POST --no-alpn --no-npn --header 'Accept: application/json; charset=UTF-8' -                                 -header 'Content-Type: application/json' --header 'X-APP-TYPE: 0' --header 'X-AP                                 P-VERSION: 1.20.0' --header 'User-Agent: G-RAC' --data-raw '{ "loginId": "yyyy", "language": 0, "password": "xxxxx" }'

Trying <IP_ADDRESS>:443...
Connected to accsmart.panasonic.com (<IP_ADDRESS>) port 443 (#0)
successfully set certificate verify locations:
CAfile: /etc/ssl/certs/ca-certificates.crt
CApath: /etc/ssl/certs
TLSv1.3 (OUT), TLS handshake, Client hello (1):
TLSv1.3 (IN), TLS handshake, Server hello (2):
TLSv1.2 (IN), TLS handshake, Certificate (11):
TLSv1.2 (IN), TLS handshake, Server key exchange (12):
TLSv1.2 (IN), TLS handshake, Server finished (14):
TLSv1.2 (OUT), TLS handshake, Client key exchange (16):
TLSv1.2 (OUT), TLS change cipher, Change cipher spec (1):
TLSv1.2 (OUT), TLS handshake, Finished (20):
TLSv1.2 (IN), TLS handshake, Finished (20):
SSL connection using TLSv1.2 / ECDHE-RSA-AES128-GCM-SHA256
Server certificate:
subject: C=JP; ST=Osaka; L=Kadoma City; O=Panasonic Corporation; CN=accsmart.                                 panasonic.com
start date: Feb  9 00:21:54 2022 GMT
expire date: Mar  8 14:59:00 2023 GMT
subjectAltName: host "accsmart.panasonic.com" matched cert's "accsmart.panaso                                 nic.com"
issuer: C=JP; O=Cybertrust Japan Co., Ltd.; CN=Cybertrust Japan SureServer CA                                  G4
SSL certificate verify ok.

POST /auth/login HTTP/1.1
Host: accsmart.panasonic.com
Accept: application/json; charset=UTF-8
Content-Type: application/json
X-APP-TYPE: 0
X-APP-VERSION: 1.20.0
User-Agent: G-RAC
Content-Length: 81

upload completely sent off: 81 out of 81 bytes
Mark bundle as not supporting multiuse
< HTTP/1.1 403 Forbidden
< Server: awselb/2.0
< Date: Mon, 18 Jul 2022 18:42:55 GMT
< Content-Type: text/html
< Content-Length: 118
< Connection: keep-alive
<

403 Forbidden

403 Forbidden

trying to use this script for a friend but also receive the 403 Forbidden error in the logfile.
Did panasonic change there authentication or server address?

I have no problem with my installation.
I was on holiday, I will take a look ASAP.
Le 29 juil. 2022 à 10:53, Guy Verschuere @.***> a écrit :

trying to use this script for a friend but also receive the 403 Forbidden error in the logfile.
Did panasonic change there authentication or server address?
—
Reply to this email directly, view it on GitHubhttps://github.com/sdamasoc/domoticz_panasonic_CZ-TACG1/issues/5#issuecomment-1199036005, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACEWTDUL2JCCHBMDXHCPERDVWOLXLANCNFSM53Z5Q4HQ.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Can you  try to execute the follow command to get a token:
curl 'https://accsmart.panasonic.com/auth/login/' -s --http1.1 -X POST --no-alpn --no-npn  --header 'Accept: application/json; charset=UTF-8'  --header 'Content-Type: application/json'  --header 'X-APP-TYPE: 0'  --header 'X-APP-VERSION: 1.15.1'  --header 'User-Agent: G-RAC'  --data-raw '{     "loginId": "<EMAIL>",     "language": 0,     "password": "<PASSWORD>" }'
Replace  and  with your credentials.
Ensure that you use the X-APP-VERSION 1.15.1 (last version of confort cloud app).
Also try to login at least once with the app to ensure that you accept the panasonic licence.

That seems to work:
{"result":0,"uToken":"FjHaZWdSZCF3beM657gn7zjlBUzJNchs","country":"BE","extUsrId":"UDeQrwCoprBDExJMRzPsFRTofjESJaFt","clientId":"0X2yVtrukiZFngzaEQcHO8Zb3CL74xBb","language":0}
However, with the same credentials and app version in domoticz:
`2022-08-01 12:20:22.410  Status: Python Plugin System: (PanasonicAirco) Entering work loop.
2022-08-01 12:20:22.410  Status: Python Plugin System: (PanasonicAirco) Initialized version 1.1.0, author 'sdamasoc'
2022-08-01 12:20:22.411  Python Plugin System: (PanasonicAirco) Debug logging mask set to: PYTHON
2022-08-01 12:20:23.368  Python Plugin System: (PanasonicAirco) getDevices=
403 Forbidden

403 Forbidden

2022-08-01 12:20:23.368  Error: Python Plugin System: (PanasonicAirco) 'onStart' failed 'JSONDecodeError'.
2022-08-01 12:20:23.368  Error: Python Plugin System: (PanasonicAirco) ----> Line 248 in '/temp/domoticz/plugins/domoticz_panasonic_CZ-TACG1/plugin.py'
2022-08-01 12:20:23.368  Error: Python Plugin System: (PanasonicAirco) ----> Line 71 in '/temp/domoticz/plugins/domoticz_panasonic_CZ-TACG1/plugin.py'
2022-08-01 12:20:23.368  Error: Python Plugin System: (PanasonicAirco) ----> Line 329 in '/temp/domoticz/plugins/domoticz_panasonic_CZ-TACG1/plugin.py'
2022-08-01 12:20:23.368  Error: Python Plugin System: (PanasonicAirco) ----> Line 346 in '/usr/lib/python3.9/json/init.py'
2022-08-01 12:20:23.368  Error: Python Plugin System: (PanasonicAirco) ----> Line 337 in '/usr/lib/python3.9/json/decoder.py'
2022-08-01 12:20:23.368  Error: Python Plugin System: (PanasonicAirco) ----> Line 355 in '/usr/lib/python3.9/json/decoder.py'`

Ok, your problem seems to come from the getDevices method.
Can you try the follow command to get the device list:
curl -s --location --request GET 'https://accsmart.panasonic.com/device/group/'  --header 'Accept: application/json; charset=UTF-8'  --header 'Content-Type: application/json'  --header 'X-APP-TYPE: 0'  --header 'X-APP-VERSION: 1.15.1'  --header "X-User-Authorization: <TOKEN>"  --header 'User-Agent: G-RAC'  --data-raw ''
And replace <TOKEN> with a fresh token.

That also seems to work:
{"iaqStatus":{"statusCode":200},"uiFlg":false,"groupCount":1,"groupList":[{"groupId":852541,"groupName":"My House","deviceList":[{"deviceGuid":"CS-TZ42WKEW+4758609730","deviceType":"3","deviceName":"Living","permission":3,"deviceModuleNumber":"CS-TZ42WKEW","deviceHashGuid":"4d789feb3218a9fd4a463702b6f9299ce55cfaa74c91b118f8168440f084f19c","summerHouse":0,"iAutoX":false,"nanoe":false,"nanoeStandAlone":false,"autoMode":true,"heatMode":true,"fanMode":false,"dryMode":true,"coolMode":true,"ecoNavi":false,"powerfulMode":true,"quietMode":true,"airSwingLR":true,"autoSwingUD":false,"ecoFunction":0,"temperatureUnit":0,"modeAvlList":{"autoMode":1,"fanMode":1},"coordinableFlg":false,"parameters":{"operate":0,"operationMode":2,"temperatureSet":20.0,"fanSpeed":0,"fanAutoMode":3,"airSwingLR":2,"airSwingUD":3,"ecoMode":2,"ecoNavi":0,"nanoe":0,"iAuto":0,"actualNanoe":0,"airDirection":1,"ecoFunctionData":0,"lastSettingMode":0}},{"deviceGuid":"CS-MTZ16WKE+4426118311","deviceType":"3","deviceName":"Slaapkamer ","permission":3,"deviceModuleNumber":"CS-MTZ16WKE","deviceHashGuid":"1c99aa9abfc98d5fd94140a712d67db3a3a3e4e62d6805f9c994f05e750b4fbb","summerHouse":0,"iAutoX":false,"nanoe":false,"nanoeStandAlone":false,"autoMode":true,"heatMode":true,"fanMode":false,"dryMode":true,"coolMode":true,"ecoNavi":false,"powerfulMode":true,"quietMode":true,"airSwingLR":true,"autoSwingUD":false,"ecoFunction":0,"temperatureUnit":0,"modeAvlList":{"autoMode":1,"fanMode":1},"coordinableFlg":false,"parameters":{"operate":0,"operationMode":2,"temperatureSet":18.0,"fanSpeed":0,"fanAutoMode":1,"airSwingLR":6,"airSwingUD":0,"ecoMode":0,"ecoNavi":0,"nanoe":0,"iAuto":0,"actualNanoe":0,"airDirection":1,"ecoFunctionData":0,"lastSettingMode":0}},{"deviceGuid":"CS-MTZ16WKE+4426118330","deviceType":"3","deviceName":"Bureau ","permission":3,"deviceModuleNumber":"CS-MTZ16WKE","deviceHashGuid":"886202dc95f1964a3e8c14678f980ef45bcb94bb0d01c718c9b2d6d080af4237","summerHouse":0,"iAutoX":false,"nanoe":false,"nanoeStandAlone":false,"autoMode":true,"heatMode":true,"fanMode":false,"dryMode":true,"coolMode":true,"ecoNavi":false,"powerfulMode":true,"quietMode":true,"airSwingLR":true,"autoSwingUD":false,"ecoFunction":0,"temperatureUnit":0,"modeAvlList":{"autoMode":1,"fanMode":1},"coordinableFlg":false,"parameters":{"operate":0,"operationMode":2,"temperatureSet":18.0,"fanSpeed":0,"fanAutoMode":1,"airSwingLR":6,"airSwingUD":0,"ecoMode":2,"ecoNavi":0,"nanoe":0,"iAuto":0,"actualNanoe":0,"airDirection":1,"ecoFunctionData":0,"lastSettingMode":0}}]}]}%

I made a fix on the headers sent by the plugin.
Can you pull the changes and test it on domoticz?

Also, be sure to use 1.15.1 in API version parameter

Thank you, this is working!
24 devices created in domoticz.

Also, be sure to use 1.15.1 in API version parameter

Does this mean that for every update of the app I need to change this number in domoticz? How can we find the latest number?

I found on the internet that we can use a version greater than the current version to avoid to update it every time they release a new version (ex: 1.20.0)
To know the version I look for it in the App Store on my phone when I execute the Confort Cloud app updates.

@MikeZMssl and @JoostdeJonge123 does this new code also fix your issues?

Hi Sergio,
I cannot check, because I am on holiday. I will check when I am back home.
Regards, Joost
-------- Oorspronkelijk bericht --------
Van: Sergio @.>
Datum: 01-08-2022 19:47 (GMT+01:00)
Aan: sdamasoc/domoticz_panasonic_CZ-TACG1 @.>
Cc: JoostdeJonge123 @.>, Mention @.>
Onderwerp: Re: [sdamasoc/domoticz_panasonic_CZ-TACG1] Error getting data (Issue #5)
@MikeZMsslhttps://github.com/MikeZMssl and @JoostdeJonge123https://github.com/JoostdeJonge123 does this new code also fix your issues?
—
Reply to this email directly, view it on GitHubhttps://github.com/sdamasoc/domoticz_panasonic_CZ-TACG1/issues/5#issuecomment-1201518728, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZWN5DCRE22EDLMM6C6HINTVXAEQXANCNFSM53Z5Q4HQ.
You are receiving this because you were mentioned.Message ID: @.***>
--
Dit e-mailbericht is gecontroleerd op virussen met Avast antivirussoftware.
https://www.avast.com/antivirus

Hi Sergio,
Will check after my holiday ! :)

From: Sergio @.>
Sent: Monday, August 1, 2022 7:46:51 PM
To: sdamasoc/domoticz_panasonic_CZ-TACG1 @.>
Cc: MikeZMssl @.>; Mention @.>
Subject: Re: [sdamasoc/domoticz_panasonic_CZ-TACG1] Error getting data (Issue #5)
@MikeZMsslhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMikeZMssl&data=05|01||731c2dc531b24b35495408da73e5d176|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637949728136108606|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=AmSGZBLzjCDhgPgFVcaCQmCD%2FI69FHSIwAGfMxhEOU4%3D&reserved=0 and @JoostdeJonge123https://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FJoostdeJonge123&data=05|01||731c2dc531b24b35495408da73e5d176|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637949728136108606|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=AAp7ZIk%2FPMlnFqnUcovUJprE%2Bbgzihq9bl9UQYCXrGo%3D&reserved=0 does this new code also fix your issues?
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsdamasoc%2Fdomoticz_panasonic_CZ-TACG1%2Fissues%2F5%23issuecomment-1201518728&data=05|01||731c2dc531b24b35495408da73e5d176|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637949728136108606|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=WL3T8JmnBinHsg%2FhY9dmZYqiwYbPwF%2B91%2BSEFEeRIl4%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQMPJTCNLYXZO7GXS6RSMD3VXAEQXANCNFSM53Z5Q4HQ&data=05|01||731c2dc531b24b35495408da73e5d176|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637949728136108606|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=ycT6FpDFtEkXMUlmdw99s7akGafqh6CsCD5cUaBTO7Q%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

Ok, have a good holiday and please feel free to reopen the issue if the problem is still present.

Back from holidays… problem seems to be fixed! Thanks.
Regards, Joost
From: Sergio @.***
Sent: maandag 1 augustus 2022 20:36
To: sdamasoc/domoticz_panasonic_CZ-TACG1
Cc: JoostdeJonge123; Mention
Subject: Re: [sdamasoc/domoticz_panasonic_CZ-TACG1] Error getting data (Issue #5)
Closed #5https://github.com/sdamasoc/domoticz_panasonic_CZ-TACG1/issues/5 as completed.
—
Reply to this email directly, view it on GitHubhttps://github.com/sdamasoc/domoticz_panasonic_CZ-TACG1/issues/5#event-7103936911, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZWN5DDZQTEUHDLZ6GS2VSTVXAKKFANCNFSM53Z5Q4HQ.
You are receiving this because you were mentioned.Message ID: @.***>
--
Dit e-mailbericht is gecontroleerd op virussen met Avast antivirussoftware.
https://www.avast.com/antivirus

Hi Sergio,
I am afraid I am struggling. Maybe to do with domoticz running from containers ?
I have checked the versionno and set it correct in domoticz
2022-08-25 19:23:48.143 Error: Airco: 'onHeartbeat' failed 'KeyError':'('uToken',)'.
2022-08-25 19:23:48.143 Error: Airco: Exception traceback:
2022-08-25 19:23:48.143 Error: Airco: ----> Line 283 in '/config/plugins/domoticz_panasonic_CZ-TACG/plugin.py'
2022-08-25 19:23:48.143 Error: Airco: ----> Line 199 in '/config/plugins/domoticz_panasonic_CZ-TACG/plugin.py'
2022-08-25 19:23:48.143 Error: Airco: ----> Line 350 in '/config/plugins/domoticz_panasonic_CZ-TACG/plugin.py'
2022-08-25 19:23:48.143 Error: Airco: ----> Line 306 in '/config/plugins/domoticz_panasonic_CZ-TACG/plugin.py'
2022-08-25 19:23:58.091 Error: Airco: 'onHeartbeat' failed 'KeyError':'('uToken',)'.
2022-08-25 19:23:58.092 Error: Airco: Exception traceback:
2022-08-25 19:23:58.092 Error: Airco: ----> Line 283 in '/config/plugins/domoticz_panasonic_CZ-TACG/plugin.py'
2022-08-25 19:23:58.092 Error: Airco: ----> Line 199 in '/config/plugins/domoticz_panasonic_CZ-TACG/plugin.py'
2022-08-25 19:23:58.092 Error: Airco: ----> Line 350 in '/config/plugins/domoticz_panasonic_CZ-TACG/plugin.py'
2022-08-25 19:23:58.092 Error: Airco: ----> Line 306 in '/config/plugins/domoticz_panasonic_CZ-TACG/plugin.py'

It seems you cannot get a token, just before the exception do you have a token in the logs (getToken=…)?
You can also just use the Python script testPlugin,py (edit it to set your credentials) to check what’s going wrong.
Sergio
Le 25 août 2022 à 19:27, MikeZMssl @.***> a écrit :

Hi Sergio,
I am afraid I am struggling. Maybe to do with domoticz running from containers ?
I have checked the versionno and set it correct in domoticz
2022-08-25 19:23:48.143 Error: Airco: 'onHeartbeat' failed 'KeyError':'('uToken',)'.
2022-08-25 19:23:48.143 Error: Airco: Exception traceback:
2022-08-25 19:23:48.143 Error: Airco: ----> Line 283 in '/config/plugins/domoticz_panasonic_CZ-TACG/plugin.py'
2022-08-25 19:23:48.143 Error: Airco: ----> Line 199 in '/config/plugins/domoticz_panasonic_CZ-TACG/plugin.py'
2022-08-25 19:23:48.143 Error: Airco: ----> Line 350 in '/config/plugins/domoticz_panasonic_CZ-TACG/plugin.py'
2022-08-25 19:23:48.143 Error: Airco: ----> Line 306 in '/config/plugins/domoticz_panasonic_CZ-TACG/plugin.py'
2022-08-25 19:23:58.091 Error: Airco: 'onHeartbeat' failed 'KeyError':'('uToken',)'.
2022-08-25 19:23:58.092 Error: Airco: Exception traceback:
2022-08-25 19:23:58.092 Error: Airco: ----> Line 283 in '/config/plugins/domoticz_panasonic_CZ-TACG/plugin.py'
2022-08-25 19:23:58.092 Error: Airco: ----> Line 199 in '/config/plugins/domoticz_panasonic_CZ-TACG/plugin.py'
2022-08-25 19:23:58.092 Error: Airco: ----> Line 350 in '/config/plugins/domoticz_panasonic_CZ-TACG/plugin.py'
2022-08-25 19:23:58.092 Error: Airco: ----> Line 306 in '/config/plugins/domoticz_panasonic_CZ-TACG/plugin.py'
—
Reply to this email directly, view it on GitHubhttps://github.com/sdamasoc/domoticz_panasonic_CZ-TACG1/issues/5#issuecomment-1227559049, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACEWTDSEW4GRQLEUM44WHOLV26UGNANCNFSM53Z5Q4HQ.
You are receiving this because you modified the open/close state.Message ID: @.***>

Hi Sergio,
The test script does work.
I am working with domoticz version 2022.1 hash c9526851b ; Might that be different?

The testScript just send python commands to the Panasonic api without using Domoticz, you need to edit the script to setup your credentials.
What I need to understand is the full log to see what the getToken method returns because the plug-in cannot find the key uToken in the JSON response.
Le 25 août 2022 à 20:52, MikeZMssl @.***> a écrit :

Hi Sergio,
The test script does work.
I am working with domoticz version 2022.1 hash c9526851b ; Might that be different?
—
Reply to this email directly, view it on GitHubhttps://github.com/sdamasoc/domoticz_panasonic_CZ-TACG1/issues/5#issuecomment-1227641344, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACEWTDWDCO6D3CZGLQTZZ2LV266IDANCNFSM53Z5Q4HQ.
You are receiving this because you were mentioned.Message ID: @.***>

That’s the log, we can see the error:
getToken={"message":"New version app has been published","code":4106}
Go to the hardware configuration, click on the plug-in and set api version to: 1.15.1 (last version of Panasonic comfort cloud app)
[image0.jpeg]
Le 25 août 2022 à 21:20, MikeZMssl @.***> a écrit :
getToken={"message":"New version app has been published","code":4106}

That indeed does help. Strange, cause my app version is 1.15.0
But since it is working : you are the best 👍
Many thanks :)


fix: #6 Make the collection capped with a default size of 10Gb

Hey - nice work 👍
Couple questions:

I believe the TTL approach on indexes does not work with the capped collections - do we need to remove the TTL option from the index on the createdAt field?
Also wondering if we should accept MB or GB instead of bytes to simplify the usage for clients looking to override the default capped collection (i.e. we convert for them into bytes)?

Very good points.
The expiry on index has been removed. Units changed to MBs.
Please have a look again. Thanks!

:tada: This PR is included in version 1.0.7 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add Picture Dialog to browse picture tree

When in debug mode, this PR replaces the infobox that would previously show up with a dialog showing more detailed and readable information about a picture. Additionally, mousing over a shape or shape name (in picture browser) will cause that shape to be highlighted. This also adds user-defined symbols to the bottom of autocomplete (see commit c63fd5c).
This PR makes a lot of changes to Driver.hs, namely an intermediate structure Drawing is added which makes highlighting possible.
A demo should be available at http://<IP_ADDRESS>/.
Even though the Summer of Haskell working period has ended, I'm still committed to maintaining my contributions to Codeworld. If there's any problems with this PR or bugs that show up in the future I'm still happy to fix them.

This PR should fix #533, #530 and make #531 obsolete (or make fixing #531 a matter of adding keybindings).

Thanks for the pull request.
I like the ideas here, but I don't think I can merge it in its current form due to UI issues.  It comes down to a number of small things, but put together, the interface is just too clunky and confusing.
A few of these small things are:

The code draws a popup window that looks almost taken from a 1987-era early GUI, with black boxes on a white background.  The title text is gigantic and out of proportion to the rest of the UI.  We don't need anything fancy here, but this just doesn't fit the look and feel of the web site at all, and it's confusing.
The slash-separated picture "path" isn't something that will likely make a lot of sense.
It's not clear to me why the "tree" section of the dialog box repeats the shape name, and doesn't seem to look much like a tree.
The coloring on highlight on the hover is far too intrusive.  It's frustrating to try to select a shape, when I have to memorize its location before moving the mouse over the canvas, because it will be hidden as soon as I get near it.  Perhaps it might look nicer if the highlight were always white or black with 25% alpha?

I also noticed that the polygon coordinates are wrong!  I'm not at all sure how, but the x coordinates shown in the popup are different from those in the code.

http://<IP_ADDRESS>/#PdWWq7nFoR-TxEgt9JQRRtw (the tie) is an example where polygon coordinates are wrong.

Thanks for the feedback! These should be easy changes, and I'll update the test server as I commit. I just want to make sure I'm going in the right direction with 1-3.
1. The dialog is just a Jquery UI dialog. I styled it myself so I didn't have to include jquery's full stylesheet as a dependency. If I use jquery's stylsheet it'll look a bit more modern. http://jqueryui.com/resources/demos/dialog/default.html
2 and 3. My goal for the dialog was to make it look something like a filebrowser where users could navigate through the tree-like structure of a Picture. The path is the shape's location in a picture's tree. If you click on something up the path the tree will show a nested list of everything below it. Originally the picture browser only opened after clicking a button on a row of the infobox, so I could possibly remove the path and revert to that.
4. This is a good point. I'll try changing the highlight color to always be black with a low alpha.
The coordinate problem seems to be caused by my num2str method not working on negative numbers. Deceivingly it still manages to produce something that looks reasonable.

Okay, I completely missed the tree structure working out once you have selected a parent node.  That's actually pretty cool now.  I still think the combination of both path and tree is overkill, though, and that many of my students won't really have the concept of a file path to work with in the first place, so the metaphor will be lost.  What about just showing the collapsible tree (starting from the root) and pre-expanding just the nodes on the path all the way from the root to the selected node?
If that's done, the path will be gone.  The title bar of the popup was never useful.  The current selected shape name could be better indicated by visibly highlighting it in the tree, rather than repeating the name.
Then the question is what to do with the text description.  Initially, my thoughts are that if you can already select the relevant code (at least you used to be able to... I can't figure out how any more), then that information is already accessible that way.  It's true that the code doesn't label the meanings of the parameters as clearly, but I suspect that in the long run, the right solution to that will be to add the info in the editor instead of the debugger - e.g., in a tooltip that appears when the cursor is on a function application, as in http://lh5.ggpht.com/_0j4bzarlOBg/S3FQy-9UZDI/AAAAAAAAArM/hjVu0Y5xrXg/image_thumb[9].png?imgmax=800
So I guess I wonder if the best approach is to only show the collapsible tree, render the selection there, and ALSO render the same selection in the code.
On the other hand, here's what would be absolutely lovely: if, when a node is selected, you also see just that one picture drawn onto a smaller canvas with a coordinate plane.  That would be incredibly useful for helping students to build a compositional mental model for pictures, and understanding that the subexpressions describe complete pictures in their own right.  So those are the two things that I think pass muster here: the collapsible tree, and a mini-canvas showing the selected picture on its own with a coordinate plane.

As far as the UI style, there are a few things that seem less than ideal to me.  The easier problem is just that it looks flat, while the rest of the site doesn't.  You can probably go a long way just by adding the "dropbox" class to the window.
But I feel like modeless dialogs are just not something that you see any more in modern UI design, so it will feel foreign and strange.  I preferred the older approach with a temporary box -- but maybe it won't be as convenient when there's more to navigate through and you can't move the box out of the way.  (Another approach would be to replace or push down the message area; but the message area already gets chopped off on small screens, so that's probably not feasible.)

Eric, thanks for the fixes to numbers and highlight color.  Are you still working on other concerns, or did you need anything from me to proceed?

I'm working on drawing the selected picture to a smaller canvas. Sorry I'm a bit slow, class started for me this week and I had to worry about some school-related stuff.
I do have a question though. I think the tree would be difficult to navigate in the same box from before, even with most things collapsed, so I'm not quite sure where to go with the UI. For some pictures the tree can be quite large and deeply-nested. I could scrap the tree and go back to the stack list, with an extra button to show the picture with a coordinate plane. I don't see a good way to do the tree without covering up something else or using a dialog. Possibly a modal dialog with just the tree could be shown, but that loses some of it's interactivity.
It'll be easier to implement if I know what I'm aiming for, so I'm just looking for a direction to go in.

Okay, for now let's keep the modeless dialog.  But applying the default styles (though I like your close button better) and adding the "dropbox" class, which adds a drop shadow for consistency with the rest of the UI would be a great change.  Then I'd love to see the full tree all the way from the root, and for clicking on a tree item to select the relevant code and possibly draw that shape, rather than changing the dialog contents.  Do you think that would work?

The preview should be running in the dialog now (and live at http://<IP_ADDRESS>/). Hopefully I can add default styles and remove the path and description as well tonight. I'm not sure I can keep the current close button, although I've noticed hover doesn't show properly on Firefox which might be why jquery does it differently.
I think selecting the code and not changing dialog contents is a good idea. Should the picture show up on a different column of the dialog or by the mouse so students can still see it as their browsing the tree?

This looks much nicer, I think.  I've been swamped with other things, but I'll try to take another detailed look in the next day or so.

Cool!  There's a lot of stuff I'd love to try, but this definitely looks better than what is live now, so I'll merge and push other changes forward.


Moving PDI from C to C++

In GitLab by @croussel on Aug 31, 2017, 10:57
I think we should list the steps and agree on them.

To make PDI C++ ready who should merge an atomic branch whose purpose is solely to enable compiling with a C++ compiler.

Independently of this we produce classes that will replace the C structures.

In the C/CXX header (.h), we explain the purposes of each classes and we declare them.

What need to be kept from the .c we move into .cxx but a lot of the .cxx codes is produced from scratch to match the classes purposes.
During that step we perform some replacement when copying/pasting C code:

malloc/free -> new/delete
char* -> std::string
C array -> std::vector/ std::array
use of reference instead of pointer
use of std::cout...

EDIT: sentence was not clear.

In GitLab by @croussel on Aug 31, 2017, 11:14
regarding merge request !78, I was thinking that, in this branch, we just have minor modifications that compiles with both C and C++ for compatibility issues (step 1 above).
In a different process, we define classes whose purposes are evolving compared with those of the C struct thus the core goes through a major change and the C-like function (malloc/free) are "naturally" removed in this step.
@jbigot : what do you think ?

In GitLab by @croussel on Sep 18, 2017, 13:57
closed


History table shows price in incorrect currency

What happened?
When you open a history table you can see the price in BSX instead of KSM
Please reproduce in steps

Open any NFT with price
Toggle history table
the price is denominated in BSX instead of KSM

Expected Behavior
There should be KSM instead of BSX
Please use the proper solution (we have component for that)
What browsers are you seeing the problem on?
No response
At which address did you encounter bug?
No response
Are you logged in?
No response
Which wallet you are using?
No response
Screenshots

Relevant log output
No response
Payment link for reward
https://kodadot.xyz/transfer/?target=<My_Kusama_Address_check_https://github.com/kodadot/nft-gallery/blob/main/CONTRIBUTING.md#creating-your-ksm-address>
Code of Conduct

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

👋

@Jarsen136 just a note that this is concerning the renewed /bsx route, it's already working on snek

it's already working on snek

They are using the same code sir :)

They are using the same code sir :)

I assume that's the problem here sir 😄
snek nft history:

bsx nft history:

okay, this is not just the history part, but this one all over again:

https://github.com/kodadot/nft-gallery/issues/3896

related PR:

https://github.com/kodadot/nft-gallery/pull/3906

@Jarsen136 please let's do this one first and push quickly for small supplemental release? 😄


[release-1.6] Integration test for ProxyStatus and Experimental AuthZ Check

This is an automated cherry-pick of #23502

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio_release-1.6
52ecebdeebd9fffc1830592ff16d00978eb2d53e
link
/test lint_istio_release-1.6

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

@istio-testing: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio_release-1.6
52ecebdeebd9fffc1830592ff16d00978eb2d53e
link
/test lint_istio_release-1.6

integ-pilot-k8s-tests_istio_release-1.6
52ecebdeebd9fffc1830592ff16d00978eb2d53e
link
/test integ-pilot-k8s-tests_istio_release-1.6

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

@istio-testing: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio_release-1.6
52ecebdeebd9fffc1830592ff16d00978eb2d53e
link
/test lint_istio_release-1.6

integ-pilot-k8s-tests_istio_release-1.6
52ecebdeebd9fffc1830592ff16d00978eb2d53e
link
/test integ-pilot-k8s-tests_istio_release-1.6

integ-local-tests_istio_release-1.6
52ecebdeebd9fffc1830592ff16d00978eb2d53e
link
/test integ-local-tests_istio_release-1.6

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

@istio-testing: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio_release-1.6
52ecebdeebd9fffc1830592ff16d00978eb2d53e
link
/test lint_istio_release-1.6

integ-pilot-k8s-tests_istio_release-1.6
52ecebdeebd9fffc1830592ff16d00978eb2d53e
link
/test integ-pilot-k8s-tests_istio_release-1.6

integ-local-tests_istio_release-1.6
52ecebdeebd9fffc1830592ff16d00978eb2d53e
link
/test integ-local-tests_istio_release-1.6

integ-multicluster-k8s-tests_istio_release-1.6
52ecebdeebd9fffc1830592ff16d00978eb2d53e
link
/test integ-multicluster-k8s-tests_istio_release-1.6

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

@googlebot I consent.

Closing this because the code has changed too much to do an automated cherry-pick.  Will do another PR for the needed function.

Closing this because the code has changed too much to do an automated cherry-pick. Will do another PR for the needed function.

Let's wait to close this when the other is ready to merge. It helps us track cherry picks. Thanks!

Closing because it is not needed.


Introduction of Provider and PropertyState

Spec: https://github.com/gradle/gradle/blob/master/design-docs/internal-apis-made-public.md#story-identify-and-promote-first-class-support-for-mapping-between-extension-and-task-properties

PR: https://github.com/gradle/gradle/pull/1452


Account Settings Health Check

Critical Changes
Changes

Added a Health Check for the Default Account Model, Administrative Account Record Type, and the Household Account Record Type

Issues Closed
New Metadata
Unpackaged Metadata
Deleted Metadata
Testing Notes

let's update the variable names to avoid abbreviations.

nvm let's fix these later.

let's update the variable names to avoid abbreviations.

nvm let's fix these later.


KAFKA-15853: Move ClientQuotaManagerConfig outside of core

blocker for #15158
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@mimaison
@ijuma
This PR are ready for merge for me. Please, take a look.

Hello @OmniaGM
Can you, please, omit force push to the PR's? :)
This breaks commit history and reviewers (like I am) don't understand what have changed with force push.
PR will be squashed to the single nice looking commit during the merge, anyway.

@OmniaGM This needs rebasing too. Thanks

Hello @OmniaGM
Can you, please, omit force push to the PR's? :) This breaks commit history and reviewers (like I am) don't understand what have changed with force push.
PR will be squashed to the single nice looking commit during the merge, anyway.

I do this usually with medium/big prs but as this one small and we have a chain of prs relays on each others for this works this makes it easier to me to cherry-picking kinda for me.

@OmniaGM This needs rebasing too. Thanks

done


Master Table Tenancy needs a multi-level option

Post haste

Not actually a priority at the moment, never mind...


Building iOS with context receivers

Issue description:
https://youtrack.jetbrains.com/issue/KT-58516

Context receivers don't have a support for Kotlin/Native. Removed them for now


window registry

currently there's a hard coded list of windows. figure out a way to register available windows and just have them all displayed. (in python I'd do this with a decorator on the method responsible for drawing each window, but not sure what I'll do in rust. probably has to be a macro.)

Doesn't look like the Pythonic approach can be reasonably achieved in Rust due to lack of runtime introspection. So the new idea is a trait and a hard coded vector of windows.
This was easy enough for everything except the repl window. This one has a different signature for its ui() method since it takes a mutable reference to the command buffer. Once the input field is moved to the bottom bar in #41.

Actually it's still a problem. It was necessary and made sense to move the history buffer out of the UI since it's really application state, not UI. (If we didn't do this then command executions in the main application needed to interact with GUI code to update the history hidden inside the GUI structs.)
But now this means that the command history must be passed to the REPL history UI, so it still has a different signature to the rest. I'm going to leave this for now. After all, it is fundamentally different to other UI elements, since it shows application state whereas all the others show only machine state.


New version: DominikReichl.KeePass version 2.56

[ ] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree

When will this Pubilshed?


envDataSubset() results in files with "_gen" suffix instead of "_env" suffix

Due to this line: https://github.com/jromanowska/HaplinMethyl/blob/cd64db9cb6765496e83f416b11e6d7944ab7240b/R/envDataSubset.R#L35
envDataSubset() results in the following file names:

*file.out*_gen.ffData, and
*file.out*_gen.RData.

I think this can be easily fixed by adding , root = "env" to the arguments supplied to the Haplin:::f.make.out.filename() function on lines 35 and 36. Ref. line 61-63 in envDataRead():
	files.list <- Haplin:::f.make.out.filename(
		file.in, file.out, dir.out = dir.out,
		root = "env", overwrite = overwrite )
```

I just tested envDataSubset() on real EWAS data on the safe server and can confirm that this issue is fixed :)

Thank you, Ellisif!


Blog app - Views

Created the views for the URL
Created CSS with styling.

Thank you so much for th the review, @nedjwab ! Have a nice day!


feat: scalable project browsing

What new features does this PR implement?

more compact searchable table view for projects
remove performance problem when number of Projects are large.
switch to server-side pagination and free text search

What bugs does this PR fix?
improves the CSS problem with home-teaser component (now renders but layout is still not right).
Does this PR introduce any additional changes?
creates a server-side text search using Elasticlunr library (in memory full-text search solution).
How have you tested this PR?

unit test added for new ProjectsSearch capability in API server.

Additional information

:tada: This PR is included in version 9.24.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


(TENTATIVE) Project initiation form: Add new field to associate form with one or more indicators

HOLD FOR DISCUSSION WITH EMILY
This is a proposed solution for the problem raised by Marco in Afghanistan, wherein they now have no way to associate -- even indirectly via data records -- project initiation forms with indicators.
We could add a new field to the PI form, allowing you to select one or more indicators.

No longer developing Workflow.


Update models.data.ts

I'm doing the same on my fork, It worked after I removed the pricing: line:
//chatgpt-4o-latest
{
idPrefix: 'chatgpt-4o-latest',
label: 'chatgpt-4o-latest-one',
description: 'Advanced, multimodal flagship model that’s cheaper and faster than GPT-4 Turbo.',
contextWindow: 128000,
maxCompletionTokens: 16384,
trainingDataCutoff: 'Oct 2023',
interfaces: [LLM_IF_OAI_Chat, LLM_IF_OAI_Vision, LLM_IF_OAI_Fn, LLM_IF_OAI_Json],
benchmark: { cbaElo: 1286 },
hidden: false,
},

This was supported with the o1 patch today


Run on Request

I am using this to develop single Requests. I end up having to make a folder to put a single Request in for load testing. This is not a dealbreaker, but it would be nice if there was an option to just run on a single Request

We had this feature in the past. Then Insomnia changed the APIs. Let me have a look if that's still possible.

This should solve the issue. @shindig can you please review?


pam_mount: /run/wrappers/bin seems not to be in $PATH so fusermount fails

Describe the bug
When using the pam_mount module to mount a directory encrypted with gocryptfs the mount fails with mount failed: Operation not permitted. This seems to be the case because fusermount is executed directly from the store (instead of /run/wrappers/bin/fusermount).
Steps To Reproduce

Copy the file below in a flake.nix and run nix build .

{
  description = "Bug report for pam_mount";
  inputs = {
    nixpkgs.url = "github:NixOS/nixpkgs/nixos-unstable";
    flake-utils.url = "github:numtide/flake-utils";
    nixos-generators = {
      url = "github:nix-community/nixos-generators";
      inputs.nixpkgs.follows = "nixpkgs";
    };
  };

  outputs = { self, nixpkgs, nixos-generators, ... }: {
    packages.x86_64-linux = {
      default = nixos-generators.nixosGenerate {
        system = "x86_64-linux";
        modules = [
          ({ config, pkgs, ... }: {
            environment.systemPackages = with pkgs; [ gocryptfs ];

            users.extraUsers.test = {
              home = "/home/test/";
              password = "test";
              isNormalUser = true;
            };

            security.pam.mount.enable = true;
            security.pam.mount.debugLevel = 2;
            security.pam.mount.extraVolumes = [''
              <volume user="test" fstype="fuse" options="nodev,nosuid,quiet"
              path="${pkgs.gocryptfs}/bin/gocryptfs#/home/test/enc/" mountpoint="/home/test/plain/" />
            ''];

            system.stateVersion = "22.11";
          })
        ];
        format = "vm";
      };
    };
  };
}

start the VM with result/bin/run-nixos-vm
log in as user test (password test)
execute

mkdir {plain,enc}
gocryptfs -init enc #give password 'test'
gocryptfs enc plain
echo a > plain/a
fusermount -u plain

log out
log in again
ls plain gives no directory contents

Expected behavior
/home/test/plain should be mounted, there should be a file a inside.
Additional context
There is a hacky solution:
By adding ''<path>${pkgs.util-linux}/bin:/run/wrappers/bin</path>'' to security.pam.mount.extraVolumes the example works.
/run/wrappers/bin could already be added in nixos/modules/security/pam_mount.nix.
Notify maintainers
@davidak

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/limited-interface-to-system-nix-external-dependencies/25825/7


can't vote

i just bought toucan nft but still i am not able to caste my vote? why is this so?

@pank3458 Please see the discussion of this topic https://github.com/snapshot-labs/snapshot/discussions/767


Bulk consume (even with slow-bulk parameter) give the results one by one and not in big chunks

This is not nice a probably has some performance implications.
The issue lies in the line
https://github.com/thomsonreuters/CM-Well/blob/6bc1381a0444b9f37fe770ea51b8097852048d4b/server/cmwell-ws/app/controllers/BulkScrollHandler.scala#L342
This flow splits the result into one-by-one infotons.
The fix should be changing the infotonWriteable to handle also infoton chunks.
https://github.com/thomsonreuters/CM-Well/blob/6bc1381a0444b9f37fe770ea51b8097852048d4b/server/cmwell-ws/app/controllers/BulkScrollHandler.scala#L353

Let me know how to test this once you implement the fix (hopefully, without deep diving into tcpdump analysis).


activity listing html markup

Closing PR because it has been referenced in https://github.com/spencer-rafada/things-to-do-in-Rexburg/pull/12


https://github.com/sosy-lab/sv-benchmarks/blob/master/c/loops/ludcmp_false-unreach-call.i

Symbiotic says TRUE.
Without optimizations, it says FALSE
without slicing it says FALSE also

It is some optimization from 'before', if this is just 'after' optimization, it is FALSE

with 'before' optimizations, there's not the assert called at all, but slicer say that it found it...

this was fixed


Trigger time update after play on HLS playback

Refactoring requested by issue #1183
Changes

Change null initial value of _lastTimeUpdate to { current: 0, total: 0 }
Move _startTimeUpdateTimer() from constructor to play()

Coverage increased (+0.2%) to 76.512% when pulling f003f3c2543e35fce696d9688b7ae80faf114ac4 on joaopaulovieira:timeupdate-hls-refactor into 89082c20e03fafb190ae75b071716aad596cc58a on clappr:master.


Add the genre relationship for track.

Please help to review the changes.
I add the relationship of the track and genre.


Update Derivative API tutorial to reflect the HTTP API

Once the HTTP API becomes stable, I think we want to drive people towards using it, as opposed to directly calling our CLI (when possible).

LGTM
/approve

/lgtm


[Improvement] Use hideLocalCluster flag to determine whether a dataSourceId must be present

At present, the dataSourceId schema validation for a route is lax in the sense that a dataSourceId when not present, it falls back to "Local Cluster".
This validation should be updated to check for hideLocalCluster flag. If the flag is set to false, then check if dataSourceId is present and a non-empty string.

[Triage] @DarshitChanpura Should I assign this issue to you?


  Processes that are being spawned in other shells show up.  I also see processes spawned by the status bar for my window manager.
I'm not familiar with netlink.  How does st tell netlink what events it cares about (or filter out ones it doesn't care about)?

st monitors fork events here.
If the process is a child of a currently tracked PID, it is added to the tracked set of PIDs.
I may not have understood properly the meaning of each field (it is not as straight forward as it sounds as explained here.

Turns out it was just a display issue. Fixed in be51763e90f955d314aefcee7cb7af0248dbfedf.


Use Torch Hub format, GIMP handling refactor, clarify license and reference details

Hi! This PR introduces three significant overall changes.
Using Torch Hub for model management
The current approach to providing models and sharing their weights kind of works but has some serious limitations:

Adding new models is cumbersome. You would have to re-upload the full weights archive file and the users would have to re-download it in full accordingly.
The user has to download the full gigabytes of weights even if some models are not relevant to their needs.
The licenses for the models are often quite restrictive and bundling the model sources in the GIMP-ML repo makes it difficult to correctly apply a single license to the repo (such as MIT) without clashing with the model licenses.

With that in mind, PyTorch offers a convenient and consistent way to publish and use existing models + weights via their torch.hub format (with corresponding central PyTorch Hub). Loading a model is as easy as doing
model = torch.hub.load('pytorch/vision:v0.6.0', 'deeplabv3_resnet101', pretrained=True)

Unfortunately, publishing models in this format is not too popular yet and is quite underutilized. This can be worked around quite simply, though, by forking any interesting repo to refactor the code as needed, add hubconf.py and re-publish the model weights in a more suitable format as a release under the forked repo.
So, I did just that for all of the models used by GIMP-ML:

valgur/DeblurGANv2, diff, weights

predictor = torch.hub.load('valgur/DeblurGANv2:python2', 'predictor', 'fpn_inception', device=device)

valgur/neural-colorization, diff, weights

G = torch.hub.load('valgur/neural-colorization:pytorch', 'generator', pretrained=True, map_location=device)

valgur/monodepth2, diff

repo = "valgur/monodepth2"
pretrained_model = "mono+stereo_640x192"
encoder = torch.hub.load(repo, "ResnetEncoder", pretrained_model, map_location=device)
depth_decoder = torch.hub.load(repo, "DepthDecoder", pretrained_model, map_location=device)

valgur/face-parsing.PyTorch, diff, weights

net = torch.hub.load('valgur/face-parsing.PyTorch', 'BiSeNet', pretrained=True, map_location=device)

valgur/CelebAMask-HQ, diff, weights

model = torch.hub.load('valgur/CelebAMask-HQ', 'Pix2PixHD', pretrained=True, map_location=device, gpu_ids=gpu_ids)

and deeplabv3 was already availale on PyTorch Hub

model = torch.hub.load('pytorch/vision', 'deeplabv3_resnet101', pretrained=True)

With this, the bundled model source code dirs, downloading weights.zip and using moveWeights.sh is no longer needed. Instead, the models are downloaded on the fly as necessary and with no work required by the user. I also added a hook to tqdm to get a nice indication of weight download progress directly in GIMP:

So, this solves all of the mentioned issues and more:

The user only has to download the models they need and the process is convenient.
Adding new models is easy.
Clean and consistent approach to hosting the model code or weights.
The work needed to make any existing model code portable and re-usable is not isolated to only GIMP-ML and is readily available for use by the wider PyTorch community and ecosystem instead.
The model weights are stored in a general PyTorch cache dir and can can be easily re-used by other applications, if necessary.

Along with these changes I also improved and cleaned up the input/output handling code in the plugin scripts somewhat. Notably:

deeplab_v3 now outputs labels with the standard color map.
got rid of unnecessary dependencies, such as scipy, skimage and opencv.
with the exception of facegen, all of the models now support CPU inference correctly.
added @torch.no_grad() decorator to disable unnecessary autograd work and to avoid the need for doing .detach() on Tensors.
got rid of the misleading scale parameter for the super resolution plugin (fixes #5)

Refactoring of GIMP wrapper code
I consolidated the GIMP<->NumPy wrapper code to shared functions under _util.py and added correct handling of layers with alpha channels. Also got rid of from gimpfu import * as it was fragile and caused frequent clashes between PIL.Image and gimpfu.Image.
Alpha channels are now removed from input images for processing and re-applied to the result automatically. The plugin input format types are now also correctly specified as RGB* or GRAY* accordingly to avoid confusing silent errors for the users. Fixes #8.
License, authors info and improved model references
The repo is in sore need of a license (as also mentioned in #11). The lack of it prevents any serious re-use, extension and collaboration on the project. And I personally like the idea behind this project a lot and would like to see it flourish and the number and variety of included models grow.
With the individual models now residing in separate repos with this PR, applying a license to the repo becomes straightforward. I propose adding a permissive MIT license and I already included one in this PR. Any changes in this PR are also licensed accordingly.
With the considerable input from this PR in mind, which is closer to a re-write than a patch, really, I also added myself as one of the contributors to the project in the readme. I hope you find it fair. I would also like to continue the collaboration in the future as well anyway, particularly by adding support for even more DL models now that it is much simpler. Besides that, the contributor credits are needed by nearly all open-source licenses as well.
I also extended and re-organized the model references somewhat. The most notable difference is again the addition of license details, which is a quite important detail for end-users since many of the models only allow non-commercial or research-only use.
If you have any comments or would like to approach something differently, let me know. I am open to any discussion.

Sorry we were not accepting PR as GIMP was transitioning to python3 and python2 was already deprecated. If you are interested, please contribute to this branch which is meant for python 3.


adding ASCII Seamic controller and Hori Fighting Commander 3

ASCII Seamic : 1 stick
Hori Fighting Commander 3 : 3 sticks
windows only
It comes from a user, raine emulator.

Hi, thank you for these! Can you please refer to the mapping guide for button layout to ensure the mappings are correct?
https://github.com/gabomdq/SDL_GameControllerDB#mapping-guide
Thanks again.

Ok, finally updated the mapping for the ascii pad, sorry for the delay, a bit of misunderstanding here.
By the way, the hori controller actually has no sticks which makes a it a little extreme for these mappings, but it's still useful to recognize the d-pad.
Anyway that's the best we can do here !

Oh by the way, if you are interested, here is the photo of the ASCII Seamic controller : https://segaretro.org/Seamic_Controller
And here is the hori fighting commander 3 : https://i.ebayimg.com/d/w1600/pict/283353630372_/SONY-PS3-Controller-Hori-Fighting-Commander-3-Black.jpg

@zelurker not too sure on these.. could you please ask your user to refer to the Windows game controller dialogue and make careful note of what each of the buttons are registering to be sure ? for example A = 3, B = 5, C = 7, etc. thanks !

Errr, it's really becoming complicated just for 2 small mappings... Yeah
one of the mapping follows the sega controller on your picture because it
looks like a sega controller (same disposition for the buttons), and the
other follows the playstation convention (obviously on the picture since it
uses the same playstation logos on the buttons). I'll forward the message a
last time, but if you prefer to refuse the PR, just let me know, it will be
faster and easier !
Le ven. 17 déc. 2021 à 00:52, alynne o. @.***> a écrit :

@zelurker https://github.com/zelurker not too sure on these.. could you
please ask your user to refer to the Windows game controller dialogue
https://www.top-password.com/blog/open-game-controllers-settings-in-windows-10/
and make careful note of what each of the buttons are registering to be
sure ? for example A = 3, B = 5, C = 7, etc. thanks !
—
Reply to this email directly, view it on GitHub
https://github.com/gabomdq/SDL_GameControllerDB/pull/527#issuecomment-996282989,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAV3SD4CBZOCFYWFR5JERRDURJ3TBANCNFSM5KAPEUIQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

it's really becoming complicated

Sorry for the trouble but thank you for considering my request. We do our best to try and provide good oversight here for each and every mapping to the best of our ability so players have a good experience with their devices.

one of the mapping follows the sega controller on your picture because it
looks like a sega controller (same disposition for the buttons), and the
other follows the playstation convention

This tracks with my expectation, thank you. Looking carefully at the ASCII Seamic mapping I am not confident your user has correctly configured the buttons C, Z, L and R as per the Sega spec. If you could ask them to provide notes on the DInput (Windows Game Controller dialogue) results for those buttons, I would appreciate it. No hurry.

Hello! I am the owner of these pads. I mapped my ASCII pad based on the "Sega -> Microsoft" layout provided in this database. So when the SDL mapping program asked for "R trigger" and the "R (shoulder) button", they were respectively mapped to C and Z buttons on the ASCII pad. Perhaps you are in doubt because the Windows Game Controllers properties counts the buttons from 1, whereas the SDL program which I used counts them from 0. Anyway, here you have what the Windows reports:
ASCII Seamic Controller:
A = 1
B = 2
C = 3
X = 4
Y = 5
Z = 6
L = 7
R = 8
Select = 9
Start = 10
Hori Fighting Commander 3:
Square = 1
Cross = 2
Circle = 3
Triangle = 4
Left Shoulder = 5
Right Shoulder = 6
Left Trigger = 7
Right Trigger = 8
Select = 9
Start = 10
PlayStation Button = 13
What's odd about the Hori pad is that it doesn't start counting from the the X/Cross button, but from the Square one. Also, the driver reports 13 buttons but buttons 11 and 12 don't exist. As a side-effect the PS button becomes number 13.
Anyway, I hope this can help. 👍

@dezraj Thank you for the thorough follow up.

Windows Game Controllers properties counts the buttons from 1 SDL program counts them from 0
Hori pad is [..] doesn't start counting from the the X/Cross button, but from the Square
driver reports 13 buttons but buttons 11 and 12 don't exist
PS button becomes number 13

These are all as expected, no issue. Thanks !


[proof] fix limit on AccumulatorProof's sibling count.

Motivation
N bit Merkle tree accumulator can have at most N siblings. Consider the following case where N=3.
To proof node 2, we need the siblings including 3, a, b.
                 root_hash
                   /   \
                  /     \
                 o       b
                / \     / \
               a   o   o   x
              / \ / \ / \
              0 1 2 3 4 5

In the case of ledger version which is 64-bit, there will be 64 siblings if the max version is greater or equal to 2^63 (i.e. MSB == 1). Although this is unlikely to be an issue in the foreseeable future, I thought we'd prefer to correct the code.

Have you read the Contributing Guidelines on pull requests?
Yes. CLA signed and submitted.
Test Plan
cargo test
Related PRs
None.

It seems @bors-libra is on holiday? 😂

@bors-libra your PTO is not approved.  Please return to work ASAP.
@bors-libra r+

:pushpin: Commit 28e610f has been approved by zamsden

:hourglass: Testing commit 28e610fa271b0dfc50f92dc8eaa7d37fbeedf6dd with merge 9eb4fedd522de536f427bf61a2bc49040117968c...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: zamsden
Pushing 9eb4fedd522de536f427bf61a2bc49040117968c to master...

Yay, now we can have twice as many leaves!

IIRC the reason we set it to 63 was that if you take into account how we number the nodes in DB (either in-order or post-order numbering), there can be at most 2^63 leaves. Though I agree this is an implementation detail and the format of proof doesn't need to be tied with it.

Yeah, that's a good point.  While it is an implementation detail, it is also a constraint.  Maybe we should revert this and instead a comment explaining why that is so?


Provide a way to dynamically generate Vault authentication token

We are looking to integrate with Vault, and the current VaultConfiguration class does not have a way to generate the authentication token at runtime.
Our use case is that Vault is secured using Googles IAM authentication and service accounts. We need to be able to generate an IAM token to be used for authentication at runtime, and not have it hard coded in a property.
Looking at the current code, I would have to override and replace VaultClientConfiguration and shove our IAM token stuff in there. I think it's possible, but it's not very nice or flexible. Spring provides a simple ClientAuthentication interface you can implement to provide the token, and something similar would suffice.
If I have time I'll see about getting a more thorough proposal together, but wanted to throw this out there and see if maybe my approach has an obvious downsides.

Also, this is definitely a feature/improvement request, but doesn't look like I can add that label.

Is there any update on this?


"Help translate OFF" displayed twice in Preferences

Checklist

[x] Bug Report - The issue you are reporting might be there already. Please search on the issues tracker before creating or reporting an issue.
[x] Before reporting, please ensure (if you can) that you have tested it with the latest (dev) version of the software.
[x] Don't forget to tag your issue, it makes it easier to discover.

Summary:
'Help translate Open Food Facts' option is being displayed twice in preferences. It is displayed under the category "Contributing" as well as under the category "About".
Steps to reproduce:
Open Preference Screen.
Expected behavior:
Should have been displayed just once under the "Contributing" category.
Observed behavior:
Displayed twice.
Screenshots:

Working on this.


Fix content-delivery-repo list ordering

Looks good to me.
One question here why don't create a string with _ORDERING ?
I think then the join operation can remove.

@xiangge I think using str.join on list is the cleanest way to get a very long string. Another way is to use String literal concatenation.
_ORDERING = (
    'variant_arch__variant__release__release_id,'
    'variant_arch__variant__variant_uid,'
    'variant_arch__arch__name,'
    'service,'
    'repo_family,'
    'content_format,'
    'content_category,'
    'shadow,'
    'name'
)

But having commas inside strings like this seems bit strange and can be easily missed in IDE.
What do you think?

OK, thanks for the explanation, learned.


Modified story file

Modified story.txt file with first 5 fill in the blanks. Please review and merge.

Reviewed


Add sorting to /api/Logs/filter

Add parameters that would allow to front-end to specify which column the data should be ordered by and in which direction.
existing post budy:
{
  "logTypeIds": [
    0
  ],
  "areaIds": [
    0
  ],
  "subTypeIds": [
    0
  ],
  "unitIds": [
    0
  ],
  "searchText": "string",
  "fromDate": "2023-06-12T09:48:40.667Z",
  "toDate": "2023-06-12T09:48:40.667Z"
}

proposed new with two optional fields:
{
  "logTypeIds": [
    0
  ],
  "areaIds": [
    0
  ],
  "subTypeIds": [
    0
  ],
  "unitIds": [
    0
  ],
  "searchText": "string",
  "fromDate": "2023-06-12T09:48:40.667Z",
  "toDate": "2023-06-12T09:48:40.667Z"
  "sortBy": "string",
  "desc": "boolean"
}

completed