Flutter For Web

Can you use it for the Web?

@shinriyo
Can't be used for Flutter for Web
I would like to support Flutter for Web, but it will take some time.

@0maru I can make the contribution and implement web part(although I am a mobile developer) based on approach from the article
The article is about Twitch and OAuth 2.0 but it could be easily rewritten to Twitter. What we really need from it is Open an External Window part and handling redirects.
What do you think?

@shorben07
Please contribute if you can implement it according to Android and iOS processing.
I'll be waiting for Pull Requests.
Thank you.

@0maru After doing more research it turned out that there is another plugin that already supports web https://pub.dev/packages/oauth2_client so, I withdraw my contribution commitment for now


comparison vs map

Hi,
It would be useful to also include a map in the comparison, to understand at what point the skiplist provides faster inserts
Thanks

Do you mean populate a list with n items and then record the time to insert at every position? Like so:

create list of n items
for every index [0,n], record time to insert an element
remove the new element so the list stays the same size

Will look into adding more insightful benchmarks to record. For now, you can imagine that the insert timing would look the same at each position as it would in the "worst case inserts" for a list of that size, which I've already recorded.

Hi,
I apologize for not explaining better. I am interested in using a skip-list as a dictionary (string->int) for fast inserts O(log(n)) vs O(n*log(n)) for map.
Does your skip-list or any other support string keys? If not, what other structure do you suggest. I might also check ART (adaptive radix tr.)
Thanks

I note the accepted key types in the README. The best of the bunch for ease of use, accepting strings as keys, and reliable speed is https://github.com/huandu/skiplist -- look at the tests to see how to use string->int.


fix: use f*_algebraic intrinsics, instead of f*_fast intrinsics

f*_algebraic intrinsics are safer than f*_fast intrinsics since they do not assume inputs are not NaNs or Inf.

it generates same asm: https://godbolt.org/z/bndTTGrEb


The arrows icon is no longer animated when git push/pull is in progress.

The arrows icon is no longer animated when git push/pull is in progress.
Two versions ago (I think) that little icon at the bottom was animated while a git push/pull was in progress. Now it's just static. Any idea what happened?  Was it removed on propose or it's a bug?

VSCode Version: 1.11.2
OS Version: 10.12.4

Steps to Reproduce:

Edit some files
Commit them
Do a Git push

https://github.com/Microsoft/vscode/issues/23975


Package for installing

It is troublesome to install egzact.
It is worth providing packages to let users install Egison and egzact.
Homebrew
I would like to create following formulas.

homebrew formula to install Egison
homebrew formula to install egzact (This formula refers to Egison's one).

Debian package
I would like to create following debian packages.

deb package to install Egison (individual package)
deb package to install egzact (individual package)

installers are already provided. closed.


Added Alpine image support

Added Dockerfile for Alpine linux image support.
Reduced by half the image size.
Tested.

Hi,
Could you put the Dockerfile inside a path called alpine please?
Thank you

Sure. Here you have it.

Now you can pull the image using:
docker pull logitravel/spark-docker:alpine


"Socket had a problem and will reconnect" after RGBTW lamp is turned off, then back on

Hey, I have a problem with my RGBTW lamp after I turn it off using the power switch (like you would with a normal lamp). I'm using 1.5.0-rc.12. When I turn the lamp off, obviously tuya-lan loses connection and tries to reconnect and therefor spams the homebridge.log with [TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (Error: ERR_CONNECTION_TIMED_OUT)
First question: is there a way to disable the retries after a certain time or number of tries and wait for the next UDP packet from the lamp instead?
Now my second problem is that if I turn the lamp back on using the power switch, it seems like tuya-lan tries to reconnect to the lamp, but fails for some reason.
The lamp is powered on and homebridge just started, so all is working normally:
[TuyaAccessory] Sending Küche Deckenlampe {"22":1000}
[TuyaAccessory] Sending Küche Deckenlampe {"20":true}
[TuyaAccessory] Heard back from Küche Deckenlampe with command 8
[TuyaAccessory] Heard back from Küche Deckenlampe with command 8

Now I turn the lamp off using the switch.
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (Error: ERR_PING_TIMED_OUT)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (EHOSTUNREACH)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (EHOSTUNREACH)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (EHOSTUNREACH)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (EHOSTUNREACH)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (Error: ERR_CONNECTION_TIMED_OUT)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (EHOSTUNREACH)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (Error: ERR_CONNECTION_TIMED_OUT)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (EHOSTUNREACH)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (Error: ERR_CONNECTION_TIMED_OUT)

Again, this seems normal to me, but this would soon turn into these error messages filling up the log file. Now I turn the lamp back on and would expect it to become available again, but various errors occur:
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (ECONNREFUSED)
[TuyaAccessory] Sending first query to Küche Deckenlampe (3.3)
[TuyaAccessory] Heard back from Küche Deckenlampe with command 10
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (Error: ERR_CONNECTION_TIMED_OUT)
[TuyaAccessory] Sending first query to Küche Deckenlampe (3.3)
[TuyaAccessory] Heard back from Küche Deckenlampe with command 10
[TuyaAccessory] Sending first query to Küche Deckenlampe (3.3)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (EPIPE)
[TuyaAccessory] Sending first query to Küche Deckenlampe (3.3)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (EPIPE)
...
[TuyaAccessory] Sending first query to Küche Deckenlampe (3.3)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (ECONNRESET)
...
[TuyaAccessory] Sending first query to Küche Deckenlampe (3.3)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (EPIPE)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (Error: ERR_CONNECTION_TIMED_OUT)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (Error: ERR_CONNECTION_TIMED_OUT)
...
[TuyaAccessory] Sending first query to Küche Deckenlampe (3.3)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (EPIPE)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (ECONNRESET)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (EALREADY)
[TuyaDiscovery] UDP from <IP_ADDRESS>:6667 0x000055aa...0x0000aa55
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (Error: ERR_CONNECTION_TIMED_OUT)
[TuyaAccessory] Sending first query to Küche Deckenlampe (3.3)
[TuyaAccessory] Heard back from Küche Deckenlampe with command 10
[TuyaAccessory] Sending first query to Küche Deckenlampe (3.3)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (EPIPE)
[TuyaAccessory] Sending first query to Küche Deckenlampe (3.3)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (EPIPE)
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (ECONNRESET)

and so on... If needed, I can provide the whole log file.
If I restart homebridge, everything is working fine again and the lamp is working properly.
[TuyaDiscovery] Discovery started on port 6666.
[TuyaDiscovery] Discovery started on port 6667.
[TuyaDiscovery] UDP from <IP_ADDRESS>:6667 0x000055aa...0x0000aa55
[2020-2-29 10:10:09 PM] [TuyaLan] Discovered Küche Deckenlampe (<id>) identified as RGBTWLight (3.3)
[TuyaAccessory] Changing ping gap for Küche Deckenlampe to 9s
[2020-2-29 10:10:09 PM] [TuyaLan] Connected to Küche Deckenlampe
[TuyaAccessory] Sending first query to Küche Deckenlampe (3.3)
[TuyaAccessory] Heard back from Küche Deckenlampe with command 10
[2020-2-29 10:10:10 PM] [TuyaLan] Ready to handle Küche Deckenlampe (RGBTWLight:3.3) with signature {"20":true,"21":"colour","22":1000,"23":579,"24":"0000000003e8","25":"000e0d0000000000000000c80000","26":0}

My homebridge config.json looks like this:
{
    "platform": "TuyaLan",
    "devices": [
        {
            "name": "Küche Deckenlampe",
            "type": "RGBTWLight",
            "manufacturer": "Oeegoo",
            "model": "YY-CL16-220S-WF(RGB+)",
            "id": "<id>",
            "key": "<key>",
            "dpPower": 20,
            "dpMode": 21,
            "dpBrightness": 22,
            "dpColorTemperature": 23,
            "dpColor": 24,
            "minWhiteColor": 140,
            "maxWhiteColor": 400,
            "colorFunction": "HSB"
        }
    ]
}

Thanks for your help!

I was able to somewhat solve this issue by adding
"pingTimeout": 2147483,  "connectTimeout": 2147483,
to the device config. Now when I turn the lamp off, I still get
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (EHOSTUNREACH)
every few seconds, but it recovers fine when I turn the power back on.<PHONE_NUMBER> ms should last me approx. 25 days, which should be plenty, so I‘m closing this issue for now.

I was able to somewhat solve this issue by adding
"pingTimeout": 2147483,
"connectTimeout": 2147483,

to the device config. Now when I turn the lamp off, I still get
[TuyaAccessory] Socket had a problem and will reconnect to Küche Deckenlampe (EHOSTUNREACH)
every few seconds, but it recovers fine when I turn the power back on.<PHONE_NUMBER> ms should last me approx. 25 days, which should be plenty, so I‘m closing this issue for now.

Thank you. You solved one of my big problem

I'm not sure what is wrong with my setup,I have two problems first I get the same connection error timeout. After adding your findings it has went away but my GU10 RGBAW bulb show up as a regular bulb on homekit
{
            "platform": "TuyaLan",
            "devices": [
                {
                    "name": "wifi+BLE",
                    "type": "RGBTWLight",
                    "manufacturer": "MOKO",
                    "model": "wifi+BLE Smart bulb",
                    "id": "eb7e9d0b37b987ec2djwef",
                    "key": "170f64ac3335c631",
                    "ip": "<IP_ADDRESS>",
                    "dpPower": 20,
                    "dpMode": 21,
                    "dpBrightness": 22,
                    "dpColorTemperature": 23,
                    "dpColor": 24,
                    "minWhiteColor": 140,
                    "maxWhiteColor": 400,
                    "colorFunction": "HSB",
                    "pingTimeout": 2147483,
                    "connectTimeout": 2147483

here is my logs
[10/1/2020, 18:39:10] Homebridge v1.2.3 is running on port 51155.
[10/1/2020, 18:39:11] [TuyaLan] Unregistering wifi+BLE
[10/1/2020, 18:40:10] [TuyaLan] Failed to discover wifi+BLE (################) in time but will connect via <IP_ADDRESS>.
[10/1/2020, 18:40:10] [TuyaLan] Connected to wifi+BLE
[TuyaAccessory] Disconnected from wifi+BLE
[TuyaAccessory] Closed connection with wifi+BLE

I'm not sure what is wrong with my setup,I have two problems first I get the same connection error timeout. After adding your findings it has went away but my GU10 RGBAW bulb show up as a regular bulb on homekit

{

            "platform": "TuyaLan",

            "devices": [

                {

                    "name": "wifi+BLE",

                    "type": "RGBTWLight",

                    "manufacturer": "MOKO",

                    "model": "wifi+BLE Smart bulb",

                    "id": "17088888888",

                    "key": "17088888888",

                    "ip": "<IP_ADDRESS>",

                    "dpPower": 20,

                    "dpMode": 21,

                    "dpBrightness": 22,

                    "dpColorTemperature": 23,

                    "dpColor": 24,

                    "minWhiteColor": 140,

                    "maxWhiteColor": 400,

                    "colorFunction": "HSB",

                    "pingTimeout": 2147483,

                    "connectTimeout": 2147483

}

Maybe try taking out all the parameters for the pin numbers (dpPower etc.) and color settings. These just apply to my specific lamp and are probably different for yours.
Also check if RGBTWLight is the correct type for your lamp. There's a list of all types in the wiki.

@jnuk are there any documentation on how to setup DpPower? Type: RGBTWLight is correct.

If I remember correctly, I googled the light's name and found the settings in one of the issues on this repo. I don't think there is a guide and as long as you don't have access to the schematics of your lamp or someone else has found out the correct pins, you might have to find it out yourself by trial and error...


fix(cli): fix oxlint --format json yields 0 files to lint

closes #2930

#2940  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Boshen and the rest of your teammates on  Graphite


Fix warning introduced in #1240 (08a1752)

unless does not accept a Sensitive parameter, and passing one result in a warning being output:
(/Postgresql_psql[ALTER ROLE puppetdb ENCRYPTED PASSWORD ****]) Unable to mark 'unless' as sensitive: unless is a parameter and not a property, and cannot be automatically redacted.

This is not really an issue here since this query is not displayed in the logs, even in debug mode.

Codecov Report

Merging #1249 (3947d95) into main (08e3a51) will increase coverage by 0.27%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##             main    #1249      +/-   ##
==========================================
+ Coverage   65.81%   66.09%   +0.27%
==========================================
  Files          14       14
  Lines         351      348       -3
==========================================
- Hits          231      230       -1
+ Misses        120      118       -2

Impacted Files
Coverage Δ

...ns/postgresql/postgresql_acls_to_resources_hash.rb
0.00% <0.00%> (ø)

...b/puppet/functions/postgresql/postgresql_escape.rb
0.00% <0.00%> (ø)

...puppet/functions/postgresql/postgresql_password.rb
0.00% <ø> (ø)

lib/puppet/functions/postgresql_escape.rb
0.00% <ø> (ø)

lib/puppet/functions/postgresql_password.rb
0.00% <ø> (ø)

lib/puppet/provider/postgresql_conf/parsed.rb
100.00% <ø> (ø)

.../puppet/provider/postgresql_conn_validator/ruby.rb
80.00% <ø> (ø)

lib/puppet/type/postgresql_conf.rb
93.33% <ø> (ø)

lib/puppet/type/postgresql_conn_validator.rb
100.00% <ø> (ø)

lib/puppet/type/postgresql_replication_slot.rb
100.00% <ø> (ø)

... and 4 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 587fc12...3947d95. Read the comment docs.

Cc authors of #1240 : @DavidS @sheenaajay

@smortex Thanks alot for the quick fix.Much appreciated for your time.
Just wondering if we need a sanitisation mechanism  to avoid shell injection for the values passed for username/password.

Since that never was in place, perhaps this should be merged now to avoid the regression (warning). I can't think of a solution that does work.

Thanks @ekohl and @smortex .Just had a quick discussion with team and we are looking into it. Will get the fix in. Thank you.

Following ticket is opened for sensitive data support needed for unless command
https://tickets.puppetlabs.com/browse/PUP-10950. Thank you.

Following changes added to fix the warning message. Thank you.
https://github.com/puppetlabs/puppetlabs-postgresql/pull/1258

Replaced by #1258


Adds method to list nodes on a wire

Summary
Fixes #1502
Adds a method nodes_on_wire() to the DAGCircuit. This method yields all the nodes in the DAG which affect the given wire. The method takes 2 parameters -
wire : the wire to be looked at
only_ops(optional) : default to False, if True then only the op nodes are returned, otherwise all nodes on the wire are returned.
This will be affected by #1815 and will be updated to return DAGNodes instead.

@maddy-tod while you are working on the file, can you rename qiskit/dagcircuit/_dagcircuit.py to qiskit/dagcircuit/dagcircuit.py (ie. removing the underscore) at a convenient time (for example, as the last commit in this PR)? Since you are one of most active contributors working on the file, it would make it smoother if you took care of it during one of your PRs, to avoid some conflicts.

I have also renamed qiskit/dagcircuit/_dagnode.py to qiskit/dagcircuit/dagnode.py after #1815 .

This looks good, there were some conflicts that I resolved.
However during the review, I noticed that the DAGNode.__eq__ method is defined so that it can compare DAGNode against int. This was probably done due to some convenience, but I think it is a bit of an unexpected behavior to compare objects of different types. It took me a while to understand why the test was comparing against ints, when the nodes_on_wire() method is returning DAGNodes. Can we fix this in a follow up PR?

@ajavadia That was done to allow for easier testing (there was a comment in the code about it at some stage but it must have been removed). I think this was mainly because the older version of DAGNode.__eq__ only allowed for identical DAGNode objects to compare to equal. This meant that we couldn't test that, for example, a method returned the nodes in the right order without looking at the node_id, as if we simply made identical objects to what we were expecting they wouldn't compare as equal. It was neatest to put it in DAGNode.__eq__ as otherwise all the testing would have to access the private variable _node_id.
The work @1ucian0 is doing should fix this but we could also refactor it out by having a method to return the  _node_id/compare a given val with the _node_id to see if it is equal.


SX1262 support

You mentioned in another issue you'd like some kind of coordination before others submitted PRs. I have some SX1262 modules arriving soon, and I'd like to add support to this library (I can see where, and how). Just wanted to hear your thoughts on contributing this upstream before I start.
I share your frustration with the (over)complexity of the other implementations, after looking at either using semtech's reference which has SX1262 support already, or adapting arduino-lmic.

The new radios look great, I would be very happy to accept code to support them.
For contributing a radio driver I think the guidelines to observe are:

must be contributed with the same license (it's still your code, of course)
must have an opt-in build option (e.g. LDL_ENABLE_SX1262)
must be selectable at run-time (like the existing radio drivers)
must keep consistent style (mainly concerned about indentation and spaces)

Hopefully the run-time selection feature doesn't cause too many problems. It was easy with the SX1272/6 since it's almost the same chip. I like it because it will make it possible have universal wrappers on Arduino style platforms that have plenty of flash.
I am interested to see if the existing Mac->Radio interfaces make sense for a new type of radio. They can be modified if necessary.
I'm not currently using any well thought out branching strategy. Assume that I will accept a pull-request to master if it passes review. If people start depending on this library I might shift to something like gitflow to stage releases.
 A lot of it is silly stuff like making the prefixes consistent (no more mixed lora/ldl). There is a slight change to how the Radio interrupt lines connect back to the MAC. Once this is done I hope it can remain stable for a long time.

The guidelines are sane, and what I would have done anyway :)

Arduino style platforms that have plenty of flash

I'm gonna be using an STM32l431, so not a problem for me!

I'm not currently using any well thought out branching strategy. Assume that I will accept a pull-request to master if it passes review. If people start depending on this library I might shift to something like gitflow to stage releases.

Just a side note, but I personally don't really like gitflow's develop and master split - releases should use git tags and development should happen on master.

 A lot of it is silly stuff like making the prefixes consistent (no more mixed lora/ldl). There is a slight change to how the Radio interrupt lines connect back to the MAC. Once this is done I hope it can remain stable for a long time.

Thanks for the heads up! I'm not likely to start working on this until the module works it's way to me from China, and even then I'm quite busy working on a cubesat, so this PR may never materialize.

I'm gonna be using an STM32l431, so not a problem for me! I'll test the module on arduino before possibly porting the arduino wrapper to STM32duino.

I like Arduino wrappers. Aside from being easy to use, they are a good chance to sanity check the interfaces and portability features. I'd like to have more of them.

Just a side note, but I personally don't really like gitflow's develop and master split

I've actually never used it. A policy of "use a tagged release" is much simpler.

PR may never materialize.

No worries.

0.5.0 supports SX1262 :tada:

0.5.0 supports SX1262 :tada:


rp_tags command can not get "tags RP Analysing" output

I login Harbor and   run /harbor-go-client rp_tags -d 0 -m 2000 -n xxx
I get this output :
[root@localhost /]# /harbor-go-client rp_tags -d 0 -m 200 -n ths/website-datacenter-back
===============================
==  Start tags RP Analysing  ==

==> GET http://xxx.xxx.xxx/api/search?q=ths/website-datacenter-back
==> only on Repo [ths/website-datacenter-back], max-days-untouched: 0   max-keep-num-after-Ndays: 200
My Harbor version : 1.5.3 & 1.3.0
It seems that the data returned by the api has changed.

Thanks for filing this issue. @mr1way
I doubt that there may be something abnormal with data structure definition.
I use harbor-go-client to analyse tags on harbor v1.5.0-d59c257e , it works fine.
I will upgrade my harbor version to v1.5.4 and do some tests.
I will ping you back tomorrow.


make sure there's a newline before each 'File: ' heading

Currently, the file tests/venv/postBuild/README.rst is missing a newline at the end of the last line, and so samples.html in the generated docs fails to properly format the File: postBuild heading. This fixes the code in docs/source/conf.py to add an extra newline Just In Case.

(This seemed better than just fixing README.rst, which is also an OK fix; but this fixes any future such problem.)

Thanks for this! Restarted the one failing travis job, so waiting on that. After that: LGTM!


Is there a pretrained mobilenet weights in model zoo?

I am looking for a pretrained mobilenet weights. But it seems not exist in the model zoo. Will mxnet provide it in the model zoo?

For the regular Mxnet API there is a mobilenet pretrained model convert from caffe model here

gluon have a pretrained mobilenet, and also can convert to symbol pretrained model.

Are you sure there is pretrained mobilenet for gluon ?
I was not able to get it:
>>> import mxnet as mx
>>> from mxnet import gluon
>>> gluon.model_zoo.vision.mobilenet1_0(pretrained=True)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.5/dist-packages/mxnet-0.12.1-py3.5.egg/mxnet/gluon/model_zoo/vision/mobilenet.py", line 115, in mobilenet1_0
    return get_mobilenet(1.0, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/mxnet-0.12.1-py3.5.egg/mxnet/gluon/model_zoo/vision/mobilenet.py", line 100, in get_mobilenet
    net.load_params(get_model_file('mobilenet%s'%version_suffix, root=root), ctx=ctx)
  File "/usr/local/lib/python3.5/dist-packages/mxnet-0.12.1-py3.5.egg/mxnet/gluon/model_zoo/model_store.py", line 80, in get_model_file
    short_hash=short_hash(name))
  File "/usr/local/lib/python3.5/dist-packages/mxnet-0.12.1-py3.5.egg/mxnet/gluon/model_zoo/model_store.py", line 58, in short_hash
    raise ValueError('Pretrained model for {name} is not available.'.format(name=name))
ValueError: Pretrained model for mobilenet1.0 is not available.

Sorry, the is no mobilenet params file in model zoo.
You can use the model convert from caffe. Just load the params file with mx.ndarray.load, and change key values to the name of symbols.

thx, @shuokay
I'll try to use that one convert from caffe.

This will be resolved in #9215

mobilenet pre-trained model is now provided thanks to @hetong007


Roadmap to JUnion 1.2.1: Info, features, bugs

General information about this project on its website.
Status of JUnion 1.2.2:
JUnion 1.2.2 brings almost complete redesign of the system. The result of which will be support to allocate struct types on both Java heap and off-heap. Since many APIs require the data to be present in either a primitive array or a java.nio.Buffer, if JUnion did not support modifying either as a struct, then the programmer would have to resort to copying the data to suitable format, which would result in diminishing or completely negating the benefits gained from using struct types. By being able to allocate struct types with a backing primitive array or Buffer or off-heap this can be resolved.
In JUnion 1.2.1 and before, an allocated struct array implemented automatic memory management.  During finalization of a struct array, the underling memory was freed. Indexing into the struct array resulted in returning pointer to the struct as a native address represented by type 'long'. To keep using the returned pointer it was necessary to keep a strong reference to the allocated stuct array, because otherwise the GC can finalize it and the memory will be freed. In JUnion 1.2.2, this problem has been addressed by representing struct pointers as 'long' and a reference to struct array.
One of the common data structures is an ArrayList which supports resizing. A support for resizable collection is planned.
JUnion 1.2.1 pre-release is out.
This topic is to present the status of JUnion 1.2.1.
You are welcome to post feature requests / report bugs.
The plan is to accomplish the tasks below(some or all), and release JUnion 1.2.1 iteratively as features/fixes are completed. Later, after some proper tests, the new version will be added to maven, etc.
Current features are listed at https://tehleo.github.io/junion/features.html
Planned features:

[x] - multidimensional arrays

Fixes/Other:

[x] - fix path issue
[x] - fix class fields & initializers are not compiled with junion
[x] - fix increment/decrement operators on fields in a struct
[x] - fix assignment operators on fields in a struct
[x] - update eclipse ASTParser to JLS12

Tests:

[ ] - check support with Java 11
[ ] - check if java modules work correctly

As 1.2.1 is out, will you publish this version to maven repostory? Thanks.

Hi, version 1.2.1 is indeed out, but as a pre-release. Since, version 1.2.2 will be shortly out, I will publish to maven, once I also wrote test cases. Also version 1.2.2 is now optimized for Java 9+ as well.


⚠ AlphaRatio has degraded performance

In 31d1eca, AlphaRatio ($AR) experienced degraded performance:

HTTP code: 200
Response time: 3629 ms

Resolved: AlphaRatio performance has improved in b314d0c after 10 minutes.


Does not work with Ionic

Describe the bug
It seems to setup in typescript that causes the error but I cannot find the solution. I cannot reproduce the same error with the Stackblitz but it does throw a different error.
Error while collecting story [PATH]/src/components/Input.story.vue:
[PATH]/node_modules/@ionic/core/components/ion-accordion.js:4
import { proxyCustomElement, HTMLElement, h, Host } from '@stencil/core/internal/client';
^^^^^^

SyntaxError: Cannot use import statement outside a module
    at Object.compileFunction (node:vm:360:18)
    at wrapSafe (node:internal/modules/cjs/loader:1094:15)
    at Module._compile (node:internal/modules/cjs/loader:1129:27)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1219:10)
    at Module.load (node:internal/modules/cjs/loader:1043:32)
    at Function.Module._load (node:internal/modules/cjs/loader:878:12)
    at Module.require (node:internal/modules/cjs/loader:1067:19)
    at require (node:internal/modules/cjs/helpers:103:18)
    at Object.<anonymous> ([PATTH]/node_modules/@ionic/vue/dist/index.js:6:25)

Reproduction
https://stackblitz.com/edit/histoire-vue3-starter-wwoeu5
System Info
System:
    OS: macOS 12.6.2
    CPU: (8) arm64 Apple M1
    Memory: 99.70 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.19.0 - ~/.volta/tools/image/node/16.19.0/bin/node
    Yarn: 1.22.19 - ~/.volta/bin/yarn
    npm: 6.14.18 - ~/.volta/tools/image/npm/6.14.18/bin/npm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Browsers:
    Chrome: 111.0.5563.64
    Firefox: 110.0.1
    Safari: 16.3
  npmPackages:
    @histoire/plugin-vue: ^0.15.8 => 0.15.8
    @vitejs/plugin-vue: ^4.0.0 => 4.0.0
    histoire: ^0.15.8 => 0.15.8
    vite: ^4.1.0 => 4.1.1

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.
[X] The provided reproduction is a minimal reproducible example of the bug.

Since Ionic v7 is official announced.
The removal of CommonJS resolved the issue.

Improved Vite Compatibility – We’ve removed the CommonJS entry points for Ionic React and Ionic Vue which makes Ionic easier to use with Vite and Vitest.

Since Ionic v7 is official announced.
The removal of CommonJS resolved the issue.

Improved Vite Compatibility – We’ve removed the CommonJS entry points for Ionic React and Ionic Vue which makes Ionic easier to use with Vite and Vitest.


[stdpar] Update explicit memory management through CUDA API to smart pointers

This pull request aims to update CUDACore and CUDADataFormats libraries and plugins to remove all explicit memory management through the CUDA API and replace it with the use of std::unique_ptr /  std::shared_ptr which will be instrumented by nvc++ during compilation to use unified memory.
This will include changes such as:

Removal of classes providing portability between Host and Device which are no longer needed
Removal of most CUDA runtime API calls
Plugins update to be compatible with the updated CUDACore and CUDADataFormats libraries
Renaming of files, classes, libraries to reflect the changes

Most notably, plugins will still use CUDA kernels but these changes will help in porting kernels to stdpar algorithms as most the framework will already be standard c++.

@makortel The pull request is starting to become quite large even though most changes are deletion so we should probably review it now.
As mentioned in the first comment, the core changes are to memory management; allocate memory on the heap through std::unique_ptr and let nvc++ instrument the unified memory usage. This led a lot of extra changes as a lot of classes were designed to be generic over the allocation method / pointer type, which is not needed in the stdpar version

@esseivaju Just to confirm, from your side this PR is ready to be merged, right?

@makortel Yes it should be ready to be merged


[MotiPressable]: rendering breaks using animate prop

Is there an existing issue for this?

[X] I have searched the existing issues

Do you want this issue prioritized?

[ ] Yes, I have sponsored
[X] Not urgent

Current Behavior
After bumping the reanimated to 3.3.0 MotiPressable animate throws an error(see screenshot) when rendering the component.
This is ONLY happening in dev mode, release mode shows no crash/error from what we've seen.
Nothing fancy was used as the animation, just the opacity animation when pressed.
animate={useMemo(
() =>
  ({ hovered, pressed }) => {
      return {
        opacity: hovered || pressed ? 0.5 : 1,
      }
    },
  []
)}

### Expected Behavior

MotiPressable animates without breaking.

### Steps To Reproduce

This PR https://github.com/artsy/palette-mobile/pull/134 commit fixes the error by removing the animation, trying MotiPressable with reanimated v3.3.0, and MotiPressable `animate` throws the error.

### Versions

```markdown
- Moti: 0.24.2
- Reanimated: 3.3.0
- React Native: 0.69.12

Screenshots

Reproduction
https://stackblitz.com/edit/nextjs-wfhfwc?file=pages%2Findex.tsx
only happening in native, looks fine on web.

Remember to  add ‘worklet’ at the top of the animate function.


Dynamically alter placeholders

Just wondering if there's a way to dynamically alter placeholders; i.e. "Everything" when nothing is typed, then "Select more" as soon as a selection is made?

The placeholder will be now invisible once selections are made similar to how it works with native inputs.


"create experiment table" button

In GitLab by @jakebeal on Feb 4, 2019, 16:49
Allow people to readily generate experiment tables following our desired parsing template by a push of a button (or similar)

In GitLab by @jakebeal on Aug 1, 2019, 13:26
assigned to @nwalczak

In GitLab by @jakebeal on Aug 1, 2019, 13:26
Script menu item, probably.

In GitLab by @nwalczak on Aug 14, 2019, 08:42
created branch 34-create-experiment-table-button

In GitLab by @nwalczak on Aug 14, 2019, 08:42
mentioned in merge request !41

In GitLab by @nwalczak on Aug 19, 2019, 16:24
closed via merge request !41

In GitLab by @nwalczak on Aug 19, 2019, 16:24
closed via commit ecd49c77d21d53e1f2412382084a2a21bca1164c

In GitLab by @nwalczak on Sep 16, 2019, 11:47
mentioned in commit ecd49c77d21d53e1f2412382084a2a21bca1164c


Feat: Onboard dataset Mimic-III

Description
This dataset includes 25 pipelines.
Checklist

[x] (Required) This pull request is appropriately labeled
[x] Please merge this pull request after it's approved

Use the sections below based on what's applicable to your PR and delete the rest:
Data Onboarding

[x] I'm adding or editing a dataset
[x] The Google Cloud Datasets team is aware of the proposed dataset
[x] I put all my code inside  datasets/<DATASET_NAME> and nothing outside of that directory

Code cleanup or refactoring

[x] I'm refactoring or cleaning up some code

This pull request is to delete the existing dataset named mimicIII and add the dataset mimic_iii

Please review @adlersantos @nlarge-google


Can't Authorize YAML Build Task to Use Service Connection

copy from: https://github.com/Microsoft/azure-pipelines-agent/issues/1809
When I try to build my source using the yaml below;
pool:
vmImage: 'Ubuntu 16.04'
variables:
imageName: 'xxxredactedxxx'
projectfolder: 'xxxredactedxxx'
steps:

script: docker build -f $(projectfolder)/Dockerfile -t $(imageName):$(build.buildId) -t $(imageName):latest .
displayName: 'docker build'

task: AmazonWebServices.aws-vsts-tools.ECRPushImage.ECRPushImage@1
displayName: 'Push Image: '
inputs:
awsCredentials: 'AWS'
regionName: 'eu-west-1'
sourceImageName: '$(imageName)'
repositoryName: '$(imageName)'
autoCreateRepository: true
I get the following error;

Job phase1: Step input awsCredentials references service connection AWS which could not be found. The service connection does not exist or has not been authorized for use. For authorization details, refer to https://aka.ms/yamlauthz
I confirm I have a service connection named AWS.
I tried reading the link in the error message. However, the article doesn't show any directions on how to authorize the build to use the service conneciton.

Hello @TingluoHuang
I can't solve my problem
https://github.com/Microsoft/azure-pipelines-agent/issues/1809#issuecomment-434274247
Thanks

@CesarRN can you share your AzurePipeline(VSTS) account name or the pipeline definition url that facing the problem? i can try check our telemetry see what error i can find.

@vijayma ?

I came across the same issue and posted in stackoverflow - assuming that this is an intended behaviour or a bug that won't fix easily - is there a chance to extend the troubleshooting section here https://aka.ms/yamlauthz - this is the shortcut which appears in the error message - so that people using build pipelines have a chance to fix it - e.g. by adding or changing variables ?

Is there any way to authorize and serverEndpoint for a FtpUpload@1 in a JobTemplate? I cannot get this to work with all the known workarounds.

Any update?

Any movement on this?
I've just tried adding this task to my build.yml and i'm getting an error in by PR Build Unable to queue Build
Job Job: Step input azureSubscription references service connection My Subscription which could not be found. The service connection does not exist or has not been authorized for use. For authorization details, refer to https://aka.ms/yamlauthz.

- task: AzureResourceGroupDeployment@2
  displayName: 'Validate ARM Template'
  inputs:
    azureSubscription: 'My Subscription'
    resourceGroupName: 'dev'
    location: 'West Europe'
    csmFile: 'src/Arm/azuredeploy.json'
    csmParametersFile: 'src/Arm/azuredeploy.parameters.json'
    deploymentMode: Validation

You now have an option in the service connection/variable group/secure file management experience to "authorize all pipelines". If you select that option, then you won't see this error any more.
If you do not want to authorize all pipelines, then we are rolling out another fix for this now/next week. The build that failed will have a banner giving you an option to fix it right there by authorizing the pipeline (and then start another build).

The "Authorize" Button did nothing to me. After queuing the same error appeared...

Check whether the service endpoint from the error message shows up in the list of service endpoints (under project settings -> Pipelines -> service connections).
Check whether you are a member of the user role on that service connection.

I had a same issue. Yaml pipeline reference an environment. After clean up of service connections I have deleted service connection related to the environment. Than the error during the pipeline was confusing. I had to recreated the environment, which was referenced by Yaml pipeline and it solved the issue.

I just ran into this issue and fixed it by doing this work-around, which says to edit the pipeline in classic view. I simply set my agent to clean the source before building and saved.
@vijayma I have authorized the service connection for all pipelines, but I still had this issue, so I'm not sure that fixed the issue.

I've also run into this issue. Unfortunately the work arounds mentioned didn't work. I had to recreate a new pipeline using the Docker Registry template and let the wizard to generate the service connection and the yaml. After that I've edited the yaml to suit my old one and I also was able to move it in a subfolder.
Probably is worth mentioning that my setup involves a template for building containers in a subfolder and pipeline in the repo's root which uses the templates.

I Just went through all of the work arounds as well with no luck.

Why have an "authorize resource" button if it doesn't work?
Also is there a "yaml" sandbox where I can work through connection issues like this without having to re invoke the entire pipeline?

I've the same issue here as well. I've altered my YAML files by dividing two region based servers into two deployments instead of one and now it complains about this.
Encountered error(s) while parsing pipeline YAML: Job job_deploy_server: Step  input azureSubscription references service connection $(ServicePrincipalRegion) which could not be found. The service connection does not exist or has not been authorized for use. For authorization details, refer to https://aka.ms/yamlauthz.
Adding a variable doesn't do the trick like this work-around, and retricting permission on the pipeline and giving it back the permission didn't solve it either.
Also, the authorize button appears, but doesn't do anything other then give a message that it gave the authorization, which it didn't.

By the way, it's easy to miss the button that allows you to authorize!

Try change the service connection name to the sub name and try again. I worked for me.

There should be an option to configure the Pipeline to the new or different Service Connection.
Scenario: I delete an existing Service Connection used by Pipeline. Created a new Service Connection with a different name. But the Pipeline is still referring to old name of the Service Connection object.
As mentioned by @HollyGram9307 , creating the new Service Connection with the old name is a workaround. May not be a suitable workaround for all cases.

By the way, it's easy to miss the button that allows you to authorize! Clicking this can fix the issue

This doesn't work. When you rerun failed jobs or run the pipeline again, the same error message (and the same button) appears.

By the way, it's easy to miss the button that allows you to authorize! Clicking this can fix the issue

This doesn't work. When you rerun failed jobs or run the pipeline again, the same error message (and the same button) appears.

I agree with @FaCoffee1984  ... i created a fresh pipeline for xamarin. and facing the same issue. I get the notification. i click the button authorize resources. we get notification resources authorized. if we queue the build again - same issue occurs again ... how to proceed ?

I agree with @FaCoffee1984  ... i created a fresh pipeline for xamarin. and facing the same issue. I get the notification. i click the button authorize resources. we get notification resources authorized. if we queue the build again - same issue occurs again ... how to proceed ?

Like other people here, the "Authorize resources" button didn't work in my case. Nevertheless, the workaround that fixed it for me was editing the service connection and reverting changes:

Edit Service connection: change name (copy the old name) and uncheck "Grant Access permission to all pipelines"
Click on "Verify and save"
Edit Service connection: revert name (paste old name) and check "Grant access permission" to all pipelines
Click on "Verify and save"

There must be a bug somewhere that when someone clicks "Grant access permission to all pipelines" it only grants it to the existing pipelines at that point in time.

I also can't get this working. I've tried all the suggested fixes above and still doesn't work. This issue was originally posted 3 years ago, so I guess it's not going to be resolved. I'm using GitHub as my repo, with the Pipelines integration app. I can get it working with a DevOps Git repo, but not with GitHub. Looks like I'll need to ditch Pipelines and move to GitHub Actions.

By the way, it's easy to miss the button that allows you to authorize! Clicking this can fix the issue

This doesn't work. When you rerun any failed jobs or run the pipeline again, the same error message (and the same button) appears. It says that all resources have been authorised after clicking, but the error message is still there.

Same here.
The subscription exists in service connections, it has all pipeline permissions, yet this authorization error still happens.
I figured out Microsoft wants us to suffer and that's the reason.
And a way of fixing their issie is not using their platform.

By the way, it's easy to miss the button that allows you to authorize! Clicking this can fix the issue

I was using my  subscription Id on the yaml, changing to the same name to the service connection just worked, go figure!, Thanks for the post, I was in a real sucking pickle and crazy loop

Like other people here, the "Authorize resources" button didn't work in my case. Nevertheless, the workaround that fixed it for me was editing the service connection and reverting changes:
1. Edit Service connection: change name (copy the old name) and uncheck "Grant Access permission to all pipelines"
2. Click on "Verify and save"
3. Edit Service connection: revert name (paste old name) and check "Grant access permission to all pipelines"
4. Click on "Verify and save"

Editing the service connection and just modifying the description fixed the issue for me.
("Grant Access permission to all pipelines" is unchecked in my case.)

None of the workarounds or proposed solutions work for me. Referencing a service connection through YAML templating. The name resolution works correctly, but it can't find the SC from the pipeline - authorizations are all set correctly.

@damnedOperator If you are using group variables, could you try this? It has fooled me more than once:

In DevOps, go to Pipelines
Hover over the three dots on the right hand side of your pipeline and click 'Edit'
Hover over the three dots in the top-right corner, next to Variables and Run, and click 'Triggers'
Click on 'Variables', which is to the left of 'Triggers'
Click on 'Variable Groups' and then, in the middle of the screen, click on 'Link variable group'
Choose the relevant variable group which includes your Service Connection and select 'Link' at the bottom
Re-run your pipeline and it should work

@damnedOperator If you are using group variables, could you try this? It has fooled me more than once:
In DevOps, go to Pipelines
Hover over the three dots on the right hand side of your pipeline and click Edit
Hover over the three dots in the top-right corner, next to Variables and Run, and click Triggers
Click on Variables, which is to the left of Triggers
Click on Variable Groups and then, in the middle of the screen, click on Link variable group
Choose the relevant variable group which includes your Service Connection and select Link at the bottom
Re-run your pipeline and it should work

We use the Service connections from the Project Properties so this is of no use to us - but deleting the SC and recreating it seemed to work.

@damnedOperator If you are using group variables, could you try this? It has fooled me more than once:

In DevOps, go to Pipelines
Hover over the three dots on the right hand side of your pipeline and click Edit
Hover over the three dots in the top-right corner, next to Variables and Run, and click Triggers
Click on Variables, which is to the left of Triggers
Click on Variable Groups and then, in the middle of the screen, click on Link variable group
Choose the relevant variable group which includes your Service Connection and select Link at the bottom
Re-run your pipeline and it should work

Caught me out plenty of times with Release pipeline, didn't event think of it for Build pipeline.  Working, thank-you!

Microsoft should've changed the naming scheme for task.inputs.azureSubscription.
Example:
steps:
- task: AzureCLI@2
  inputs:
     azureSubscription: <SERVICE_CONNECTION_NAME> # Note: azureSubscription != subscriptionName or subscriptionId
    scriptType: bash
    scriptLocation: inlineScript
    workingDirectory: src
    inlineScript: |
      set -ex
      echo "Listing resource groups..."
      az group list
  displayName: 'List resource groups'

Azure docs highlights the following:  azureSubscription: # string. Alias: connectedServiceNameARM. Required. Azure Resource Manager connection.
They connectedServiceNameARM alias can be used instead of azureSubscription. Not sure why they just didn't name it serviceConnectionName. Either way, to solve this The service connection does not exist or has not been authorized for use ensure you use the service connection name instead of your subscriptionName or subscriptionID.
Personally, to make things clearer, I'd go with connectedServiceName rather than azureSubscription:
steps:
- task: AzureCLI@2
  inputs:
     connectedServiceNameARM: <SERVICE_CONNECTION_NAME> # Note: azureSubscription != subscriptionName or subscriptionId
    scriptType: bash
    scriptLocation: inlineScript
    workingDirectory: src
    inlineScript: |
      set -ex
      echo "Listing resource groups..."
      az group list
  displayName: 'List resource groups'


Issue with stereochemistry

Hello,
I am performing docking experiments with various sugars (glucose, galactose and mannose) using the web server. I see inversion of the stereochemistry, which I do not see with other docking tools (Chai-1, diffdock). As such, I was wondering if there is a way to set up protenix to avoid this inversion. I do not have a computational background and may just be unaware of how to properly configure protenix.
Thanks!

@karlyy123 can you give some detailed information or provide the results to us. cc @cloverzizi

Hi @karlyy123 , even when the input 'ref pos' contains specific chiral indications, the output might still display chirality inversion, as noted in the AF3 paper. We're thinking about adding information to the output confidence JSON indicating whether an inversion has occurred. This would help users select the correct sample without inversion. Alternatively, we might explore other solutions to address this issue. Currently, our model cannot prevent inversion.


A more clever way to ignore only createSanctuary body indentation

Instead of disabling indent rule for a range of lines, disable it
for direct children of createSanctuary function, thus enabling
indent rule checks for it's deeper children.
For example, the following code if put inside createSanctuary would
have been considered correct by eslint before this commit:
  def('fromPairs',
    {f: [Z.Foldable]},
    [f($.Pair($.String, a)), $.StrMap(a)],
    fromPairs);

After this commit, code like this whould have been reported as
indent rule violation.

Very cool!
My one concern is that we're duplicating all the existing indentation settings in order to add a special case for createSanctuary. Is it possible to specify "ignoredNodes" but inherit the other settings?

I had this concern too.
Here is a rather cumbersome implementation https://github.com/futpib/sanctuary/pull/1/files . It's either that I can't find a proper way to inherit eslint config the way we would like, or that eslint indeed does not provide an easy way to do that.
An easier solution would be to directly require eslint-common.json, but that would bring in the assumption that indent rule is defined exactly there. I could require eslint-es3.json and join in whatever it extends, but that's like reimplementing eslint internal config resolution.
All these are clumsy (but the first solution at least seems correct), that's why I just duplicated the indent rule options.

futpib/sanctuary#1 would introduce more of a maintenance burden than the small amount of duplication introduced in this pull request.

An easier solution would be to directly require eslint-common.json, but that would bring in the assumption that indent rule is defined exactly there.

I'm willing to make that assumption. The result is quite reasonable:
diff --git a/.eslintrc.js b/.eslintrc.js
new file mode 100644
index 0000000..3db1526
--- /dev/null
+++ b/.eslintrc.js
@@ -0,0 +1,22 @@
+/* eslint-env node */
+/* eslint-disable quote-props */
+
+'use strict';
+
+var common = require('sanctuary-style/eslint-common.json');
+
+
+var indent = common.rules.indent.slice();
+indent[2] = Object.assign({}, indent[2], {
+  ignoredNodes: indent[2].ignoredNodes.concat([
+    'FunctionDeclaration[id.name=createSanctuary] > BlockStatement.body'
+  ])
+});
+
+module.exports = {
+  root: true,
+  extends: ['./node_modules/sanctuary-style/eslint-es3.json'],
+  rules: {
+    indent: indent
+  }
+};
diff --git a/.eslintrc.json b/.eslintrc.json
deleted file mode 100644
index 741f63f..0000000
--- a/.eslintrc.json
+++ /dev/null
@@ -1,7 +0,0 @@
-{
-  "root": true,
-  "extends": ["./node_modules/sanctuary-style/eslint-es3.json"],
-  "rules": {
-    "indent": ["error", 2, {"SwitchCase": 1, "FunctionDeclaration": {"parameters": "first"}, "FunctionExpression": {"parameters": "first"}, "CallExpression": {"arguments": "first"}, "ArrayExpression": "first", "ObjectExpression": "first", "ignoredNodes": ["ConditionalExpression", "MemberExpression", "FunctionDeclaration[id.name=createSanctuary] > BlockStatement.body"]}]
-  }
-}
diff --git a/Makefile b/Makefile
index 62c6e8f..fa4304c 100644
--- a/Makefile
+++ b/Makefile
@@ -31,7 +31,7 @@ README.md.tmp: index.js

 .PHONY: lint
 lint:
-	$(ESLINT) -- index.js karma.conf.js scripts/version-urls test
+	$(ESLINT) --ignore-pattern !.eslintrc.js -- .eslintrc.js index.js karma.conf.js scripts/version-urls test
 	$(REMEMBER_BOWER) $(shell pwd)
 	rm -f README.md
 	VERSION=0.0.0 make README.md

I'm not sure whether the patch above is an improvement on this pull request. What do you think?
Perhaps we should stick with the current approach (which is by far the simplest) and have make lint assert that these commands produce the same output:
$ node --print 'var indent = require("./.eslintrc.json").rules.indent; indent[2].ignoredNodes.pop(); JSON.stringify(indent)'
["error",2,{"SwitchCase":1,"FunctionDeclaration":{"parameters":"first"},"FunctionExpression":{"parameters":"first"},"CallExpression":{"arguments":"first"},"ArrayExpression":"first","ObjectExpression":"first","ignoredNodes":["ConditionalExpression","MemberExpression"]}]

$ node --print 'JSON.stringify(require("sanctuary-style/eslint-common.json").rules.indent)'
["error",2,{"SwitchCase":1,"FunctionDeclaration":{"parameters":"first"},"FunctionExpression":{"parameters":"first"},"CallExpression":{"arguments":"first"},"ArrayExpression":"first","ObjectExpression":"first","ignoredNodes":["ConditionalExpression","MemberExpression"]}]

Why do you have this exceptional indentation in the first place? Why not indent the createSanctuary function the way indent expects it to be indented?

Why do you have this exceptional indentation in the first place?

I believe that when I added createSanctuary I decided not to indent its body for three reasons:

indenting would have necessitated rewrapping descriptions (to adhere to the 79-character limit);
indenting would have made git blame less convenient; and
I have an aesthetic preference for flat code.

I'll close this pull request for the reason given in https://github.com/sanctuary-js/sanctuary/pull/467#issuecomment-352295669. I appreciate the suggestion, @futpib.


Entry for grammar updates in pull request template

I'm hoping having an explicit entry for it in the template will reduce the number of grammar maintainers opening pull requests for that.
Template removed as it doesn't apply.

In the extent of this PR; is there a known release schedule? Is a new version also immediately deployed to GitHub? I'd like to know when I can expect certain features to get live ^_^

There's no strict schedule, but @lildude does a new release per month on average I think.

In the extent of this PR; is there a known release schedule? Is a new version also immediately deployed to GitHub? I'd like to know when I can expect certain features to get live ^_^

What @pchaigno said 😁 I plan to make a release this week or early next.


Android JUnit Instrumentation tests output parser

Android JUnit Instruments tests provide text based output. We need parser for the same to upload results  on hive, testrail, etc

Moved issue here https://github.com/bbc/res/issues/52


documentReady block is triggered before configure has had a chance to run

We're loading ahoy up like this (from GTM):
<script type="text/javascript" src="//path_to_ahoy/ahoy-0.3.2.js"></script>

<script>
  ahoy.configure({
    cookieDomain: "ourdomain.com"
  });
  ahoy.trackAll();
</script>

We execute this both on our naked domain and a subdomain.  So, obviously setting cookieDomain is important or we'd get double cookied and/or bad things™  would happen.
It appears to me that this thing: https://github.com/ankane/ahoy.js/blob/master/src/index.js#L438-L442 is triggered before the configure block has had a chance to execute.
I've currently 'fixed' this by hardcoding our domain into the lib itself here: https://github.com/ankane/ahoy.js/blob/master/src/index.js#L8 and that seems to fix things up nicely.
Would it make sense to remove the documentReady block from the lib itself and give control of that to the end user to execute after configure has had a chance to run?

Hey @jsharpe-mr, I think a vast majority of users either don't need to configure Ahoy or are able to configure it before the document is ready, so I don't think it makes sense for all of those users to have to call it manually.

Agree with @jsharpe-mr that would be nice to have more granular control over documentReady().
In our case we install & use ahoy via npm/yarn. However, importing the module automatically executes documentReady() and starts it, before we get a chance to set up ahoy with variables necessary for its correct function (e.g. urlPrefix).
At least when building as a node module, would be nice to skip documentReady() to make the library more standalone.

I'm using ahoy.js in webpacker installed via yarn. What is the recommended practice for configuring ahoy.js with this setup?
I think I'm running into the same issue where documentReady runs before I'm able to configure the library.

I'm also using ahoy.js in a webpacker environment and am facing the same issue. The way I'm handling this is by temporarily assigning the config object to window.ahoy, allowing ahoy to pull that object in on instantiation (see relevant ahoy logic), and resetting window.ahoy to its original value:
initAhoy() {
  // desired configuration
  let ahoyConfig = { cookies: false };

  // back up current window.ahoy value
  let oldAhoy = window.ahoy;

  // assign window.ahoy to the config object,
  // ahoy.js will use it for configuration once instantiated
  window.ahoy = ahoyConfig;
  this.ahoy = require('ahoy.js');

  // restore previous window.ahoy value
  window.ahoy = oldAhoy;
}

I'm not sure if this is the best approach, but it works fairly well for my use case.


bluetooth: audio: BAP Unicast Client only discovers 1 of each type of PAC record

https://github.com/zephyrproject-rtos/zephyr/blob/23fb3b77a2b2cf53eff1697888f443236069b30c/subsys/bluetooth/audio/bap_unicast_client.c#L4405
PACS servers may have multiple PAC characteristic of each type source/sink:

PACS v1.0.1 Section 3.1
(..)
When the server supports reception of audio data, the decision to expose all supported PAC records
within a single Sink PAC characteristic or multiple Sink PAC characteristics is left to the implementation.

unicast_client_pac_discover_cb() always returns BT_GATT_ITER_STOP so only the first one is found.

This looks like a duplicate of https://github.com/zephyrproject-rtos/zephyr/issues/70826. Can you confirm @Chris-StJohn-Bose ?

This looks like a duplicate of #70826. Can you confirm @Chris-StJohn-Bose ?

Yes I think this is a duplicate @Thalley

Closing as duplicate then :)

thanks @Thalley I'll submit a PR if I get around to fixing it and tag #70826

Sounds good. There are also some planned effort from @jthm-ot to make make the PACS client usable for the broadcast assistant as well, so just be aware that the PACS client will likely need to be moved out from the BAP unicast client and into a standalone client usable from both BAP Unicast Client and BAP Broadcast Assistant (or have it's own public API). It does not prevent the above from being added, but just be aware of any changes that may cause merge conflicts


Fixed an embedded scrolling bug (Resolves #496)

Fixed an embedded scrolling bug (Resolves #496)
@miguelcmedeiros please let me know if this solves your issues in Superlist, and let me know if the PR looks OK.
Most of the work in the PR is to create a scrolling minimap to help us diagnose scrolling problems in the future. It still needs a bit of work, and we probably need some tools beyond that, but I wanted to unblock you.
There are some failing golden tests due to some kind of timing issue. I'll come back tomorrow to fix those tests and merge in, if this solves your problem.

I removed the golden tests for the scrolling minimap because I couldn't seem to get the minimap connected to a scrollable in the test. I left the minimap code in super_editor and the example app because it seems to work OK in a real app. We should come back and improve the minimap code, and add tests.
@miguelcmedeiros I'm going to merge this so that I can move forward with other work, but you should verify this fix in Superlist ASAP.


Access line in event listeners (to read and write)

Hi,
I'm playing with tty-reader, tty-cursor and tty-screen and would like to do a simple thing: when someone presses TAB while writing text in TTY::Reader prompt, move cursor to other place on the screen and put there e.g. number of currently read characters. I can hack it somehow with the tty-reader API, but I think it would be much easier if I could access the line object inside the reader's event code, e.g. like this:
reader.on(:keytab) do |event|
  print cursor.save
  print cursor.move_to(1, 1)
  puts "=> #{event.line.to_s.size}"
  print cursor.restore
end

It would give me also possibility to remove the TAB from the line to avoid placing it in the output string.
I believe it would make tty-prompt more flexible. It requires minor rearrangements in read_line while loop (especially the callback would be called after the line was updated).
Maybe I'm plainly wrong and just overlooked a simple solution? But if I'm right I can prepare a pull request.

Hey,
Thanks for using the tty libraries!
The tty-reader is a direct extract from tty-prompt. The event callbacks style has been implemented as a way of decoupling different prompts from the underlying input logic. I haven't announced this library around since I wanted to give myself some time to build it up so that it can also be used for creating REPLs and other shell funky stuff. The road ahead is long and windy 😄
I'm currently rewriting the keys functionality which basically changes majority of internals and especially how #read_line works. Therefore, I would probably wait with any changes until I do this rewrite. It's important as some keys on different systems are working wrongly or are not recognised at all.
As far as your suggestion goes of exposing the line inside the even object. I think this is a matter of taste about designing apis. I'm not sure if exposing line object through the event is a good thing or not. I would need more time to think it through. It may well be that line could be a second yielded parameter? I really like your idea of increasing the flexibility though.
I really hope that I get time this weekend to work on the keys update.

I found tty-reader on gem list in tty README, so I didn't think it's internal lib ;-)
I proposed to expose the line through event because I thought it easier to implement, but my first thought was to add second yielded parameter ;-)
OK, if it's still significantly evolving it's probably better not to mess with your concepts with such small PRs. I'm waiting (im)patiently for the developments! ;-)

The tty-reader isn't really internal per say but at the same time I haven't invested enough time to make sure it works in variety of scenarios outside of tty-prompt context. I'm fully aware that this library needs more work before it reaches flexibility of readline. However, I'm really glad you took the interest. It gave me extra motivation to work on it!
I have changed how the keys work but still want to spend some time to extract logic for line editing. After that it will be safe to think of new apis etc... One of the major things I plan to add is a completion feature. I need it for tty-prompt.

Hey,
The newest tty-reader has been released for some time now. If you fancy taking another look at this functionality be all means submit PR and let's have a discussion.

Thanks for this! ;-)
A lot happened in the meantime and for now I had to set my project aside. I hope to be able to come back and play more with the great tools you develop for cli in Ruby!

Sure! I hope you didn't take my comment as a form of criticism. I just
excuse myself for not providing any further value... ;-)
I really liked wroclove_rb, maybe we will meet there ;-)
sob., 24 lis 2018 o 16:24 Piotr Murach<EMAIL_ADDRESS>napisał(a):

I always revisit the issues but given I'm maintaining quite a few Ruby
gems it takes time to circle back 😅 Hope we get to chat at some Ruby
conference in the near future!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/piotrmurach/tty-reader/issues/5#issuecomment-441375228,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABa5i5wgrcHOaNtZqDjJv1oFA-6Ep92bks5uyWStgaJpZM4Qu1We
.

--
Wojciech Rząsa

I've never been to wroclove_rb. Just checked the site and it seems there is nothing for the next year sorted yet. Thanks for letting me know I will keep a close watch on this one in case I can make it.


test/CMakeLists.txt: Use an explicit list instead of GLOB

Using GLOB is slow and considered bad practice.
From https://cmake.org/cmake/help/latest/command/file.html:

We do not recommend using GLOB to collect a list of source files from
your source tree. If no CMakeLists.txt file changes when a source is
added or removed then the generated build system cannot know when to ask
CMake to regenerate. The CONFIGURE_DEPENDS flag may not work reliably on
all generators, or if a new generator is added in the future that cannot
support it, projects using it will be stuck. Even if CONFIGURE_DEPENDS
works reliably, there is still a cost to perform the check on every
rebuild.

Close #1686.

Coverage remained the same at 100.0% when pulling eb6fe421ae821a04899066613702eebfb39998e6 on t-b:avoid-using-glob-in-cmake into d187488e0db0533bdd7c53ec0c687ca1745b8b9e on nlohmann:develop.

Thansk!

🔖 Release item
This issue/PR will be part of the next release of the library. This template helps preparing the release notes.
Type

[ ] ✨ New Feature
[ ] 🐛 Bug Fix
[ ] ⚡ Improvement
[x] 🔨 Further Change
[ ] 🔥 Deprecated function

Description

Avoid usage of GLOB in CMake


Add new kind of toastr

When doing an ajax request, it is sometime usefull to display a notification to the user. This new kind of toastr is used for this purpose.

Not going to add new toasts for the time being, sorry.


magetic structure in phono3py

Dear Togo  Sensei,
This is more of query not an issue with phono3py.  I was wondering if it possible to include symmetry breaking due to non-zero magnetic moments  while creating displacements in phono3py. The crystal i am investigating in AFM type, and  at least within phonopy I see that inducing magnetic lowers the overall symmetry of the cell.
Thanks
Alex
PS. Sorry about posting this here.  I am having trouble with my email.

I've never tried, so I realized it is not supported by the phono3py command. But I think it should work in the API level, i.e., just there is no route to pass magmom information to phono3py command. Do you have an example of crystal structure with magnetic moments?

Togo Sensei, Thank you for your response.
I was hoping to do some magnetic structure like MnTe, and MnSe..  But now reading the lietraure carefully it seems that in these structures magentic moments can be canted (non collinear), which I think cannot be handled within phonopy ( also the DFT runs for these would be changeling).
For collinear magnetic structures, I think phonopy works well as long as I specify a unit-cell with correct magnetic ordering (via magmom).  I think the same way it done for phno3py as well, I will take a look for possibility of doing it within API.
Once I have  working example ready, I will share it. I will close this for now.
Thanks and Regards
Alex

OK, I see. I implemented non-collinear type symmetry treatment in spglib. So maybe some day in the future.


[REQ] GitHub Action for copa

What kind of request is this?
New feature
What is your request or suggestion?
We should design a GitHub action to utilize copa. I think that would live in a separate repo in the same org

@salaxander can i work on this issue?

Hey @reetasingh! I think we need to talk a little bit more on the design of this before we're ready to start work on it, but I will absolutely keep you posted!

Hey @reetasingh! I think we need to talk a little bit more on the design of this before we're ready to start work on it, but I will absolutely keep you posted!

Thanks for the update @salaxander , meanwhile I am also open to any other issue

WIP https://github.com/project-copacetic/copa-action/pull/1

Available now: https://github.com/marketplace/actions/copacetic-action
Opened #245 for docs update follow up


INS Documentation

Checklist

[x] Interact with INS

[x] Registry document
[x] Registrar document
[x] Resolver document

[x] INS Library support

[x] Javascript
[ ] Python
[ ] Command-line

[x] Implementation guide of INS

[ ] Add INS into decentralized application
[x] Using INS libraries/SDK
[x] Resolving INS
[x] Updating INS records

Python library and command line will move to other repository
Integration with INS into decentralized application will create another repository


chore: add owner

What this PR does / why we need it:
chore: add owner
Which issue(s) this PR is related to (optional, link to 3rd issue(s)):
Fixes #
Reference to #

Special notes for your reviewer:
/cc @your-reviewer

Release note:

NONE

/lgtm


Add org Sentience Institute

I created the file and added 9 incomplete positions. Not ready for review yet.

I completed 16 positions. Ready for review


[Hub Generated] Publish private branch 'validate'

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
ARM (Control Plane) API Specification Update Pull Request
Purpose of this PR
What's the purpose of this PR? Check all that apply. This is mandatory!

[ ] New API version. (Such PR should have been generated with OpenAPI Hub, per this wiki doc.)
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix swagger quality issues in S360.
[x] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I have reviewed the general guidance on the spec PR review process: https://aka.ms/specprreview.
[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data-plane related specifications.
[ ] I commit to follow the Breaking Change Policy.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can request review from an ARM API Review board.

ARM API changes review

If you want for the ARM team to review this PR, you must add the ARMReview label.
The automation may automatically add the ARMReview label, if appropriate.
If this happens, proceed according to guidance given in GitHub comments also added by the automation.

Breaking change review
If you have any breaking changes as defined in the Breaking Change Policy,
follow the process outlined in the High-level Breaking Change Process doc.
Getting help

For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For additional help, see https://aka.ms/azsdk/support/spectools.

The PR is created based on the updates in the private branch. The updates in the PR has already been reviewed and approved with this PR Azure/azure-rest-api-specs-pr/12081

Issue for management client library name :
https://github.com/Azure/azure-sdk-pr/issues/1187

Hi @RBANSALMS This PR was flagged for introducing a new RP namespace that is not being onboarded with RPaaS. Merging this PR to the main branch will be blocked as RPaaS is required for new RPs. To resolve this and allow the PR to be merged into the main branch, please use RPaaS to onboard the new RP.

New rp namsespaces must be onboarded with rpaas. Can you please fix this ?

@raosuhas
Regarding your comment regarding "New rp namsespaces must be onboarded with rpaas", we are already onbroaded to RPaas.
I have other concerns wrt to this PR:
1. I had opened this PR via Openapihib by providing the latest PR for our RP from the Private repo.
https://openapihub.azure-devex-tools.com/tools/publishpullrequest
Latest PR from Private Repo:  https://github.com/Azure/azure-rest-api-specs-pr/pull/12081
But it seems that this PR does not include all the files from the private repo, it does include only one file "specification/scom/resource-manager/Microsoft.SCOM/preview/2022-09-13-preview/scom.json", but all other files from the private repo are not included in this PR.
All files from the private repo are shown below:

Ideally all the files from the private repo should be part of this PR to be merged into the public repo, right?
Please guide me how should I make include all the files in the PR.

I think, as all the files are not included in the PR, so you are seeing this RP as not being onboarded to RPaas. Once the above point 1 will get fixed, then this error will also go away.

I assume that the same should be the way for other validation errors as well (Avacado, ModelValidation , SemanticValidation etc)

Please feel free to redirect.

@raosuhas , please help me in getting unblocked.

Opened manually generated PR.  https://github.com/Azure/azure-rest-api-specs/pull/24803
Closing this PR.


Frontend-homework-4

Metric
Value

alert_status
b:white_check_mark: OK/b

bugs
2

code_smells
0

reliability_rating
D (at least 1 Critical Bug)br/Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)br/Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
b:white_check_mark: OK/b

bugs
2

code_smells
0

reliability_rating
D (at least 1 Critical Bug)br/Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)br/Note: A being best, E being worst

vulnerabilities
0


InitMouseInput Signature

The InitMouseInput signature only accepts unit for 'data'. It should accept uint to allow for the mouse scroll down event.

@vflame
Thanks for bringing this up :)
The mouseData field is variable. It contains mouse scroll event only when dwFlags contains MOUSEEVENTF_WHEEL. That's why isn't not in the structure.
Ref: https://msdn.microsoft.com/en-us/library/windows/desktop/ms646273(v=vs.85).aspx
However, what I'd like to do is like the KeyboardInput structure, it should include helper methods that extract it.  Could you try it? A PR would be awesome :)

I'm not sure if this is 'kosher' or not but I just changed the signature from uint to int and scrolling up and down is working as expected.
I'm not following what you mean by using a helper method extract the variable. I'm not very familiar with winapi.

@vflame  - I'm guessing you're facing problems because you're somehow losing the sign bit during the int-uint conversion. You probably might want to look at the application code.

I'm not following what you mean by using a helper method extract the variable. I'm not very familiar with winapi.

Essentially, another overloaded InitMouseInput with the proper scroll values, which automatically set's the dwFlags to 1 as well. That should be the correct approach.
PS: For a quick solution though, the MouseInput structure is transparent, so you can just create it manually and do whatever you want, as a workaround. :)

I also found casting the int to uint using unchecked (permitting the overflow) works as well without any changes to the existing source. Thanks for the library.


Is it possible to add a custom header in the RequestBuild or have a funcation like AddHeaders()?

Agreed, we could add AdditionalHeaders to ClientConfig and then add them in setCommonHeaders() like this
...
if len(c.config.AdditionalHeaders) > 0 {
		for k, v := range c.config.AdditionalHeaders {
			for _, val := range v {
				req.Header.Add(k, val)
			}
		}
	}
...

This would make this lib a perfect connector via VPNs that require additional headers or even OpenRouter that can connect to multiple other AIs like meta-llama and google.


connection error on Victron Smart Shunt - ERROR [sampling] solarCharger error: <class 'AssertionError'>

I am successfully monitoring two Liontron batteries at the same time using your app. thanks again for your work!
I now tried to connect a new victron smart shunt that i recently bought. I always get the same error. It seems like its connecting successfully but than fails to parse the data froms the device.
what i tried:

tried to fetch data from all three devices at the same time
tried to fetch data only from victron smart shunt
renamed the device
change the pin code
bt_power_cycle: true/false
install_newer_bleak: true/false

any ideas?
Paste log output between BEGIN and END:
BEGIN

   return runner.run(main)
  File "/usr/lib/python3.11/asyncio/runners.py", line 118, in run
    return self._loop.run_until_complete(task)
  File "/usr/lib/python3.11/asyncio/base_events.py", line 640, in run_until_complete
    self.run_forever()
  File "/usr/lib/python3.11/asyncio/base_events.py", line 607, in run_forever
    self._run_once()
  File "/usr/lib/python3.11/asyncio/base_events.py", line 1922, in _run_once
    handle._run()
  File "/usr/lib/python3.11/asyncio/events.py", line 80, in _run
    self._context.run(self._callback, *self._args)
  File "/app/main.py", line 282, in main
    await fetch_loop(fn, period=sample_period, max_errors=max_errors)
  File "/app/main.py", line 35, in fetch_loop
    await fn()
  File "/app/main.py", line 275, in fn
    await t()
  File "/app/bmslib/sampling.py", line 67, in __call__
    return await self.sample()
  File "/app/bmslib/sampling.py", line 89, in sample
    async with bms:
  File "/app/bmslib/bt.py", line 250, in __aenter__
    await self.connect()
  File "/app/bmslib/victron.py", line 64, in connect
    await super().connect(timeout=timeout)
  File "/app/bmslib/bt.py", line 153, in connect
    await self._connect_client(timeout=timeout)
  File "/app/bmslib/bt.py", line 138, in _connect_client
    self.logger.debug("Pairing %s using psk '%s'...", self._psk)
Message: "Pairing %s using psk '%s'..."
Arguments: ('123456',)
20:16:19 DEBUG [client] BLE device @ ED:B9:01:26:9D:35 already paired with hci0
20:16:19 DEBUG [victron] write keep_alive b'`\xea'
20:16:19 DEBUG [client] received D-Bus signal: org.freedesktop.DBus.Properties.PropertiesChanged (/org/bluez/hci0/dev_ED_B9_01_26_9D_35/service0029/char003d): ['org.bluez.GattCharacteristic1', {'Value': <dbus_next.signature.Variant ('ay', b'\xff\xff\xff\x7f')>}, []]
20:16:19 DEBUG [client] Read Characteristic 6597eeff-4bda-4c1e-af4b-551c4cf74769 | /org/bluez/hci0/dev_ED_B9_01_26_9D_35/service0029/char003d: bytearray(b'\xff\xff\xff\x7f')
20:16:19 DEBUG [client] Write Characteristic 6597ffff-4bda-4c1e-af4b-551c4cf74769 | /org/bluez/hci0/dev_ED_B9_01_26_9D_35/service0029/char002a: b'`\xea'
20:16:19 DEBUG [client] received D-Bus signal: org.freedesktop.DBus.Properties.PropertiesChanged (/org/bluez/hci0/dev_ED_B9_01_26_9D_35/service0029/char003d): ['org.bluez.GattCharacteristic1', {'Notifying': <dbus_next.signature.Variant ('b', True)>}, []]
20:16:19 DEBUG [client] received D-Bus signal: org.freedesktop.DBus.Properties.PropertiesChanged (/org/bluez/hci0/dev_ED_B9_01_26_9D_35/service0029/char0035): ['org.bluez.GattCharacteristic1', {'Value': <dbus_next.signature.Variant ('ay', b'\xf9\xff')>}, []]
20:16:19 DEBUG [client] Read Characteristic 6597ed8e-4bda-4c1e-af4b-551c4cf74769 | /org/bluez/hci0/dev_ED_B9_01_26_9D_35/service0029/char0035: bytearray(b'\xf9\xff')
20:16:19 DEBUG [client] received D-Bus signal: org.freedesktop.DBus.Properties.PropertiesChanged (/org/bluez/hci0/dev_ED_B9_01_26_9D_35/service0029/char0035): ['org.bluez.GattCharacteristic1', {'Notifying': <dbus_next.signature.Variant ('b', True)>}, []]
20:16:19 DEBUG [client] received D-Bus signal: org.freedesktop.DBus.Properties.PropertiesChanged (/org/bluez/hci0/dev_ED_B9_01_26_9D_35/service0029/char0031): ['org.bluez.GattCharacteristic1', {'Value': <dbus_next.signature.Variant ('ay', b')\x05')>}, []]
20:16:19 DEBUG [client] Read Characteristic 6597ed8d-4bda-4c1e-af4b-551c4cf74769 | /org/bluez/hci0/dev_ED_B9_01_26_9D_35/service0029/char0031: bytearray(b')\x05')
20:16:19 DEBUG [client] received D-Bus signal: org.freedesktop.DBus.Properties.PropertiesChanged (/org/bluez/hci0/dev_ED_B9_01_26_9D_35/service0029/char0031): ['org.bluez.GattCharacteristic1', {'Notifying': <dbus_next.signature.Variant ('b', True)>}, []]
20:16:19 DEBUG [client] received D-Bus signal: org.freedesktop.DBus.Properties.PropertiesChanged (/org/bluez/hci0/dev_ED_B9_01_26_9D_35/service0029/char0039): ['org.bluez.GattCharacteristic1', {'Value': <dbus_next.signature.Variant ('ay', b'\xf4\xfd\xff\xff')>}, []]
20:16:19 DEBUG [client] Read Characteristic 6597ed8c-4bda-4c1e-af4b-551c4cf74769 | /org/bluez/hci0/dev_ED_B9_01_26_9D_35/service0029/char0039: bytearray(b'\xf4\xfd\xff\xff')
20:16:19 DEBUG [client] received D-Bus signal: org.freedesktop.DBus.Properties.PropertiesChanged (/org/bluez/hci0/dev_ED_B9_01_26_9D_35/service0029/char0039): ['org.bluez.GattCharacteristic1', {'Notifying': <dbus_next.signature.Variant ('b', True)>}, []]
20:16:19 DEBUG [client] received D-Bus signal: org.freedesktop.DBus.Properties.PropertiesChanged (/org/bluez/hci0/dev_ED_B9_01_26_9D_35/service0029/char002d): ['org.bluez.GattCharacteristic1', {'Value': <dbus_next.signature.Variant ('ay', b'\xff\xff')>}, []]
20:16:19 DEBUG [client] Read Characteristic 65970fff-4bda-4c1e-af4b-551c4cf74769 | /org/bluez/hci0/dev_ED_B9_01_26_9D_35/service0029/char002d: bytearray(b'\xff\xff')
20:16:19 DEBUG [client] received D-Bus signal: org.freedesktop.DBus.Properties.PropertiesChanged (/org/bluez/hci0/dev_ED_B9_01_26_9D_35/service0029/char002d): ['org.bluez.GattCharacteristic1', {'Notifying': <dbus_next.signature.Variant ('b', True)>}, []]
20:16:19 INFO [sampling] connected bms SmartShuntBt(ED:B9:01:26:9D:35)!
20:16:19 ERROR [sampling] solarCharger error: <class 'AssertionError'>
20:16:19 INFO [sampling] Bleak version 0.13.1a1
20:16:19 ERROR [main] 1 exceptions occurred fetching BMSs
20:16:19 ERROR [main] Error (num 1, max 200) reading BMS:
20:16:19 ERROR [main] Stack: Traceback (most recent call last):
  File "/app/main.py", line 35, in fetch_loop
    await fn()
  File "/app/main.py", line 280, in fn
    raise exceptions[0]
  File "/app/main.py", line 275, in fn
    await t()
  File "/app/bmslib/sampling.py", line 67, in __call__
    return await self.sample()
           ^^^^^^^^^^^^^^^^^^^
  File "/app/bmslib/sampling.py", line 95, in sample
    sample = await bms.fetch()
             ^^^^^^^^^^^^^^^^^
  File "/app/bmslib/victron.py", line 86, in fetch
    sample = BmsSample(**values)
             ^^^^^^^^^^^^^^^^^^^
  File "/app/bmslib/bms.py", line 59, in __init__
    assert math.isfinite(soc)
AssertionError
20:16:20 ERROR [sampling] solarCharger error: <class 'AssertionError'>
20:16:20 INFO [sampling] Bleak version 0.13.1a1
20:16:20 ERROR [main] 1 exceptions occurred fetching BMSs
20:16:20 ERROR [main] Error (num 2, max 200) reading BMS:
20:16:20 ERROR [main] Stack: Traceback (most recent call last):
  File "/app/main.py", line 35, in fetch_loop
    await fn()
  File "/app/main.py", line 280, in fn
    raise exceptions[0]
  File "/app/main.py", line 275, in fn
    await t()
  File "/app/bmslib/sampling.py", line 67, in __call__
    return await self.sample()
           ^^^^^^^^^^^^^^^^^^^
  File "/app/bmslib/sampling.py", line 95, in sample
    sample = await bms.fetch()
             ^^^^^^^^^^^^^^^^^
  File "/app/bmslib/victron.py", line 86, in fetch
    sample = BmsSample(**values)
             ^^^^^^^^^^^^^^^^^^^
  File "/app/bmslib/bms.py", line 59, in __init__
    assert math.isfinite(soc)
AssertionError
20:16:21 DEBUG [client] received D-Bus signal: org.freedesktop.DBus.Properties.PropertiesChanged (/org/bluez/hci0/dev_ED_B9_01_26_9D_35/service0029/char0039): ['org.bluez.GattCharacteristic1', {'Value': <dbus_next.signature.Variant ('ay', b'\x12\xfe\xff\xff')>}, []]
20:16:21 DEBUG [victron] msg current 0.494

END

I'll fix this with the next update

I just tried it and it works with the latest version. thanks a lot for your quick response 👍


Remove redesign-for-ui branch

I... actually don't know how this happened.

Will be removed once no longer needed

Wait, so it WASN'T GitHub doing this?

Then what exactly DID do it!??!?!!

No.

It wasn't GitHub.


yask: Fixes to make the acoustic wave equation test take off

This will have to be rebased and should go after #352

Rebased; this is the next one in the YASK line

Addressed comments and rebased. Will merge once travis is happy

Comments addressed and approved, so merging...

Merged.


[composer] if the project is already using twig then it downgrades to v1.x

php-phantomjs downgrades packages if they have newer ones.
D:\Work\Ad-Server V3>composer update
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - jonnyw/php-phantomjs v4.5.1 requires twig/twig ~1.16 -> satisfia
ble by twig/twig[1.x-dev, v1.16.0, v1.16.1, v1.16.2, v1.16.3, v1.17.0,
 v1.18.0, v1.18.1, v1.18.2, v1.19.0, v1.20.0, v1.21.0, v1.21.1, v1.21.
2, v1.22.0, v1.22.1, v1.22.2, v1.22.3, v1.23.0, v1.23.1, v1.23.2, v1.2
3.3, v1.24.0, v1.24.1, v1.24.2, v1.25.0, v1.26.0, v1.26.1, v1.27.0, v1
.28.0, v1.28.1, v1.28.2, v1.29.0, v1.30.0, v1.31.0, v1.32.0] but these
 conflict with your requirements or minimum-stability.
    - jonnyw/php-phantomjs v4.5.0 requires twig/twig ~1.16 -> satisfia
ble by twig/twig[1.x-dev, v1.16.0, v1.16.1, v1.16.2, v1.16.3, v1.17.0,
 .....

my composer file has twig 2.* already
"require": {
   ....
    "twig/twig": "2.*",
   ....
    "jonnyw/php-phantomjs": "4.*"
  }

Unfortunately Twig 2.* requires php 7 whereas this library supports earlier versions of php. The next major release of the library will be php 7+ so I will upgrade Twig at that point.


Use bukkit in-built ChatColor method for color code replacement.

It's possible people use & as part of the message and not for colour code. This fix will then only replace & with § when its an actual supported colour code.

Didn't think of that :), also wasn't aware this existed. Thanks for the PR! Could you merge into the dev branch next time? keeps it a bit neater for me :P

Could you merge into the dev branch next time? keeps it a bit neater for me :P

So you want me to set the target merge branch to development instead of master?

Could you merge into the dev branch next time? keeps it a bit neater for me :P

So you want me to set the target merge branch to development instead of master?

Yes, I merged it now though so it's fine :), I just wanted to make a separate branch to master so that master is always stable

You can just alter the target branch before merging, click on the "into" under the PR title to do so.

Oh I was looking for it but couldn't find it, is it the 'rebase' thing?

Sorry, "Edit" right next to the title, then change the target branch.

Alright will remember it for next time :)


New at Python - downloading a list of friends from a list of accounts

Hello, as mentioned in the title I am entirely new at Python, basically know how to adapt scripts, create loops and using pandas...
I am trying to obtain the complete list of friends and followers from a specific set of accounts as part of my research project (I am a PhD student). Is it possible with Tweepy ? I really don't get how to do it from the documentation and feel quite dumb each time I look at it.
I have my tokens (I generated a standalone app) but apart from authenticating myself, I cannot do anything with Tweepy...
Thanks in advance to the (very) patient person who will manage to help me.
access_token='*'
access_token_secret='*'
consumer_key='*'
consumer_secret='*'
bearer_token='*'
auth = tweepy.OAuthHandler(consumer_key, consumer_secret)
auth.set_access_token(access_token, access_token_secret)
api = tweepy.API(auth, wait_on_rate_limit=True)
try:
    redirect_url = auth.get_authorization_url()
except tweepy.TweepError:
    print('Error! Failed to get request token.')
api.update_status('tweepy + oauth!')

def limit_handled(cursor):
    while True:
        try:
            yield cursor.next()
        except tweepy.RateLimitError:
            time.sleep(15 * 60)

for friend in tweepy.Cursor(api.friends).items():
    # Process the friend here
    process_friend(friend)

for follower in limit_handled(tweepy.Cursor(api.followers).items()):
    print(follower.screen_name)

Is it possible with Tweepy ?

Yes
Is there a specific issue you're encountering? What is process_friend?

Is it possible with Tweepy ?

Yes
Is there a specific issue you're encountering? What is process_friend?

Actually I cannot manage to write a script to scrap all these friends and followers. "process_friend" was in the original example provided in the doc, I thought it was mandatory to include it smh but it's actually just an undefined variable 👍

From this, process_friend is an undefined function, yes.
Otherwise, this is fine for iterating through friends and followers.
Feel free to ask about any specific issues you have.


Return list of products (even if empty)

There is no way to know if the user has purchased or not the product. The restorePurchase method only fire the event IF the user has made a purchase. How to validate if the user has bought or not a product?

Hey @rafaelcmrj , there is no way to get this. You should work with the restorePurchases available functions. For example your app can initialize an empty array and save it in local settings. Then you can call restorePurcahses and in the callback add the bought products to the array. Then in the rest of your application, where you want to check if a product is bought, you can check the array in the local settings to see if the product is there or not. Hope this helps.

@PeterStaev the problem is that the restorePurchases method doesn't call the callback if you have no products in your account, so I will never know if the method was executed or not... The correct way to do that was to put my application initializer inside the callback function, so I will make sure I only start my application AFTER verify if you have or not the purchase. Makes sense?

@rafaelcmrj this is not how I would implement it. Like I have outlined in the step by step guide in the readme, you should pertain info about what products are purchased and what not in the application-settings provided by NativeScript. Once your user purchases a product you should save this locally in the application settings, so you can use it directly after the purchase is complete. You should not call resotrePurcahses every time you want to know if the user bought a product or not. restorePurchases should be triggered by the user and is basically usable only when the user uninstall the app or installs it on another phone. In this case they trigger purchase restoring and in the callback you again set the application settings just like a normal purchase.
So to sum it up - your app should start in "unpaid" mode with all paid functions hidden/disabled. If the user buys a product you save this in the application-settings and enable the corresponding function. If the user changes their phone they press some button/menu to restore purchases. Then you wait in the callback and if you receive info about some product you save that again in the application-settings and again enable the paid function for that product.

@rafaelcmrj this is not how I would implement it. Like I have outlined in the step by step guide in the readme, you should pertain info about what products are purchased and what not in the application-settings provided by NativeScript. Once your user purchases a product you should save this locally in the application settings, so you can use it directly after the purchase is complete. You should not call resotrePurcahses every time you want to know if the user bought a product or not. restorePurchases should be triggered by the user and is basically usable only when the user uninstall the app or installs it on another phone. In this case they trigger purchase restoring and in the callback you again set the application settings just like a normal purchase.
So to sum it up - your app should start in "unpaid" mode with all paid functions hidden/disabled. If the user buys a product you save this in the application-settings and enable the corresponding function. If the user changes their phone they press some button/menu to restore purchases. Then you wait in the callback and if you receive info about some product you save that again in the application-settings and again enable the paid function for that product.

@PeterStaev I was doing on this way, but if we only check the app-settings, how can we guarantee that the user didn't cancel the subscription after he/she activated the product? Doing in this way, someone can buy a product, activate the new version and cancel the purchase. He will be able to access the paid version after canceling the product.

Subscription are not yet supported by the plugin (#2). And from what I know a normal in-app product cannot be cancelled.

I've found that non-renewable subscriptions do work - most of the time.
Sometimes - I don't get some purchase notifications - but that could be something on this end.
But - yes users can purchase and then cancel and there's no way to know of that. Apple iTunes doesn't give you any info on such transactions.


[Explore]: Prevent duplicate records (both name and GUID)

Background
We are currently taking several different approaches to avoid duplicate records:

We guard against multiple apps in a space with the same name using a validating web hook
Several of our repo fetch methods in the api codebase will error if they find duplicate results
We have a "leases" proposal to enforce uniqueness, but I don't believe it's implemented for any resources yet. It's also fairly heavy weight.
There's a proposal to use CF names as K8s names, which would ensure uniqueness of name within a namespace. However, this has problems, as per the comments in the proposal

Generally at creation time we're not ensuring uniqueness of GUIDs (metadata.name) across namespaces, and of names (spec.name) within namespace.
Acceptance Criteria
Explore the current solutions to this problem and synthesize a few options for how we can enforce global uniqueness of GUIDs and uniqueness of names within a space. We may use several different solutions at once depending on the type of resource, how dangerous it is to have duplicates, and how expensive and error prone the solution is.
Number of days this exploration is expected to take. This is what is “pointed”.
3 days.
Dev Notes

Ideally we will prevent the resources from being created in the k8s API. Failing that, we should transition the duplicate record into an error state in the k8s API and ignore it in the CF API shim.

The solution should prevent duplicates when using kubectl directly, not just when using the CF API

Consider Open Policy Agent

Just to clarify we have the leases solution, wrapped in NameRegistry, for orgs, spaces and apps currently.

Here's a link to a spike done on OPA for the authorization work: https://github.com/eirini-playground/auth-explore/tree/master/opa-gatekeeper, with some words here.

TL;DR: We think the lease mechanism looks like a good approach; if we can find a way to share code across validating web hooks instead of copy/pasting, that would be better.
@acosta11 & I walked through all of the various entity types that it is possible to create today with the CF cli and double checked against the current state of our product. As it stands today, orgs, spaces, and apps are all protected with the lease/NameRegsitry mechanism above.
The only types that are implemented, but not currently protected with the lease mechanism are CFRoute and CFDomain. These types currently have no checks for duplicate names.
We did a quick POC to see how difficult it would be add a web hook to detect duplicate routes. It's in this branch
This implementation took about 2ish hours to get working, so not too bad. (we skipped tests of any kind, so maybe it would have been a day to make it perfect)
We copy-pasted a lot of code from the CFApp webhook. Maybe we want to consider some mechanism to make the checking logic generic? We would need some interface to find a "lock name" string for each type, and to produce error messages for the various error cases, but otherwise the implementation is very standard.

I like the sound of a generic, parameterisable webhook that can act on all the types.
The org/space logic is slightly different at the moment, as you have to look at a label on a subnamespaceanchor to tell whether it's an org or a space. But once we have CRDs for orgs and spaces that distinction vanishes.

Thank you, @kieron-dev, for the info and for weighing in. The leases solution is much farther along than I'd realized and it's fantastic that we a generic solution on top of it is feasible!
Thanks for the work and the write-up, @julian-hj and @acosta11. I'll write a follow up story to implement the missing uniqueness constraints, with a dev note about generalizing it.

@julian-hj @acosta11 I just realized that you didn't discuss GUID uniqueness in your write-up. Is there anything currently insuring that we don't have duplicate GUIDs across spaces?

We talked via Slack, and they decided not to focus on GUID uniqueness since it isn't an issue when using the API shim rather than kubectl directly. I'm going to spin off that exploration into a separate, ice-boxed story.
Looking at the resources in previous comments, it looks like you could use Leases to ensure resource GUID uniqueness across spaces by making the Leases in the root namespace. This would prevent situations where /v3/apps/some-app-guid matches multiple resources.


Update Composer dependencies (2021-05-20-00-10)

Visual regression test passed!
View the visual regression test report


Enter key yields Input::Character('\n'), not Input::KeyEnter

Similar to #50
Observed in pancurses 0.16.1 (on Arch Linux)

Same for me, setting window.keypad(use_keypad) to either true or false does nothing. Tried on Windows, Mac and Linux (Ubuntu 22.04)