User can send ETH to the same address the user is sending from

Should it be possible to send ETH back to the same address as shown in the video?
This is probably allowed at a protocol level but do you want to allow that in Fuel wallets, it can result in the following negative user experience:

The user will simply burn fees for no reason
The user might do this accidentally, in this case, at least a warning would be useful

https://github.com/FuelLabs/fuel-connectors/assets/114662397/f103485a-d948-4950-a500-bbd30a6cecd5
Expected behaviour:

Block users from being able to send assets to the same address they're sending from, or
Warn users that they're about to send assets to the same address they're sending from

I don't think we should fix this as:

protocol allows you to send to yourself
wallet is clearly showing "From" and "To" as the same account


Automated Package Upgrade

:tada: This PR is included in version 5.0.421 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update hotfix branch

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


Performance is worse when  using read type as CACHE_PROMOTE than NO_CACHE

Alluxio Version:
2.1.1
Describe the bug
Deploy alluxio in kubernetes with CACHE_PROMOTE and NO_CACHE, and I found the NO_CAHCE performance is much better than CACHE_PROMOTE. The file is about 130MB, and 1154 files in all.
To Reproduce

Deploy Alluxio with helm chart, the values file uses NO_CACHE

image: alluxio/alluxio
imageTag: "2.1.1"
imagePullPolicy: Always

properties:
    alluxio.user.metrics.collection.enabled: false
    alluxio.user.file.passive.cache.enabled: false
    alluxio.user.ufs.block.read.location.policy: alluxio.client.block.policy.LocalFirstAvoidEvictionPolicy
    # It should be great than (120)*0.01 = 2GB
    alluxio.user.block.avoid.eviction.policy.reserved.size.bytes: 2GB
    alluxio.fuse.debug.enabled: "false"
    alluxio.user.file.writetype.default: MUST_CACHE
    alluxio.user.block.write.location.policy.class: alluxio.client.block.policy.LocalFirstAvoidEvictionPolicy
    alluxio.worker.allocator.class: alluxio.worker.block.allocator.GreedyAllocator
    alluxio.user.block.size.bytes.default: 32MB
    fs.oss.endpoint: oss-cn-shanghai-internal.aliyuncs.com
    # set OSS endpoint
    fs.oss.accessKeyId: xxx
    fs.oss.accessKeySecret: yyy
    alluxio.user.file.readtype.default: NO_CACHE
    alluxio.master.mount.table.root.ufs: oss://imagenet/
    #alluxio.user.file.master.client.threads: "128"
    #alluxio.user.block.master.client.threads: "128"
    # alluxio.underfs.oss.socket.timeout: 3000sec
    alluxio.user.network.reader.chunk.size.bytes: 32MB
    alluxio.worker.file.buffer.size: 320MB
    alluxio.security.stale.channel.purge.interval: 365d
    alluxio.user.metadata.cache.enabled: "false"
    alluxio.user.metadata.cache.max.size: "1000000"
    alluxio.user.direct.memory.io.enabled: "true"
    alluxio.fuse.cached.paths.max: "1000000"
    alluxio.job.worker.threadpool.size: "64"
    alluxio.worker.block.master.client.pool.size: 128

tieredstore:
  levels:
  - alias: MEM
    level: 0
    path: /dev/shm
    type: hostPath
    # quota: 120GB
    high: 0.99
    low: 0.7

fuse:
  image: alluxio/alluxio-fuse
  imageTag: "2.1.1"
  imagePullPolicy: Always
  args:
    - fuse
    - --fuse-opts=direct_io

Run the training jobs without any cache
Performance result is :

10000	images/sec: 897.3 +/- 2.1 (jitter = 37.6)	4.466
----------------------------------------------------------------
total images/sec: 7171.94
----------------------------------------------------------------

Deploy the alluxio without setting alluxio.user.file.readtype.default

image: alluxio/alluxio
imageTag: "2.1.1"
imagePullPolicy: Always

properties:
    alluxio.user.metrics.collection.enabled: false
    alluxio.user.file.passive.cache.enabled: false
    alluxio.user.ufs.block.read.location.policy: alluxio.client.block.policy.LocalFirstAvoidEvictionPolicy
    # It should be great than (120)*0.01 = 2GB
    alluxio.user.block.avoid.eviction.policy.reserved.size.bytes: 2GB
    alluxio.fuse.debug.enabled: "false"
    alluxio.user.file.writetype.default: MUST_CACHE
    alluxio.user.block.write.location.policy.class: alluxio.client.block.policy.LocalFirstAvoidEvictionPolicy
    alluxio.worker.allocator.class: alluxio.worker.block.allocator.GreedyAllocator
    alluxio.user.block.size.bytes.default: 32MB
    fs.oss.endpoint: oss-cn-shanghai-internal.aliyuncs.com
    # set OSS endpoint
    fs.oss.accessKeyId: xxx
    fs.oss.accessKeySecret: yyy
    alluxio.master.mount.table.root.ufs: oss://imagenet/
    #alluxio.user.file.master.client.threads: "128"
    #alluxio.user.block.master.client.threads: "128"
    # alluxio.underfs.oss.socket.timeout: 3000sec
    alluxio.user.network.reader.chunk.size.bytes: 32MB
    alluxio.worker.file.buffer.size: 320MB
    alluxio.security.stale.channel.purge.interval: 365d
    alluxio.user.metadata.cache.enabled: "false"
    alluxio.user.metadata.cache.max.size: "1000000"
    alluxio.user.direct.memory.io.enabled: "true"
    alluxio.fuse.cached.paths.max: "1000000"
    alluxio.job.worker.threadpool.size: "64"
    alluxio.worker.block.master.client.pool.size: 128

tieredstore:
  levels:
  - alias: MEM
    level: 0
    path: /dev/shm
    type: hostPath
    # quota: 120GB
    high: 0.99
    low: 0.7

fuse:
  image: alluxio/alluxio-fuse
  imageTag: "2.1.1"
  imagePullPolicy: Always
  args:
    - fuse
    - --fuse-opts=direct_io

Run the training job
Get the performance result:

----------------------------------------------------------------
1000	images/sec: 620.6 +/- 7.8 (jitter = 264.4)	4.721
----------------------------------------------------------------
total images/sec: 4963.33
----------------------------------------------------------------

Expected behavior
A clear and concise description of what you expected to happen.
Urgency
Describe the impact and urgency of the bug.
Additional context
Add any other context about the problem here.

@ggezer as we talked offline, do you see this issue is consistent with what you found too or the problem you mentioned is different?

@apc999, It does not seem so looking at cluster configuration.
@cheyang, Do you see anything interesting in worker logs during CACHE_PROMOTE scenario?

The additional information is updated here:
Alluxio status:

1 master, 1 worker, 1 alluxio-fuse, all launched in the same machine through helm kubernetes
only 1 tier of alluxio worker memory large enough to hold the whole dataset

machine status:

Cpus: 82
Memory：336G
GPU：8 * NVIDIA V100
Network: 20GiB/s

dataset:

multiple layers of directory holding a large number of photos
about 144GB

training job:

deep learning training job cnn
will launch multiple processors (e.g. 8 processor for 8 GPU in this test)

The tests:
| readtype                    |     data cached     |  training speed          |
| NO_CACHE                |      False                |  about 990 images/s |
| NO_CACHE                |     True                |  about 1040 images/s |
| CACHE_PROMOTE   |     True                   |  about 630 images/s |

Observations:

This is a powerful machine, it has strong computation ability (Cpus: 82,GPU：8 * NVIDIA V100 )
The network interaction with alluxio UFS is fast (The network receive data amount is about 1500MB/s during training)
NO_CACHE nearly used all the GPU, while CACHE_PROMOTE didn't fully utilize the GPU （used about 70%)

According to @cheyang, the CACHE_PROMOTE performance is worse than NO_CACHE only occurs when the machine has strong computation ability (strong GPU). When the machine is not strong enough, this issue doesn't occur.

Would you mind pasting the full worker property configuration instead of summarizing it.

Also if you haven't already spotted anything interesting in worker logs, is it possible running the cache-promote scenario with debug traces for alluxio.worker.block.TieredBlockStore class?


Android Basics: SQL Basics

This sentence was included in the codelab section Primary Key:
"While you won't be using databases with more than one table in this course, having a unique ID which will also help with querying, updating, and deleting existing items in a table."
Problem with it is that it is an incomplete sentence and confusing to understand.

@hebadi
Thank you for reaching out to us. We fully understand your concern so the issue was fixed and published.


FileNotFoundError: [Errno 2]protein.faa

Thank you for making this convinient tool!But when I use VirBot I met this problem:
`$ VirBot.py --input ERR1992810.contigs.fa --output ERR1992810_vb --sen
Input contig: ERR1992810.contigs.fa
Output directory: ERR1992810_vb
Mode: sensitive mode
Predicting the encoded proteins...
Proteins prediction finished.
Scanning the protein by hmmsearch...
Error: Failed to open sequence file ERR1992810_vb/tmp/protein.faa for reading
HMMER finshed.
Scanning the protein by DIAMOND...
DIAMOND finshed.
Traceback (most recent call last):
File "/home/tool/VirBot/VirBot.py", line 379, in
sen=args.sen, taxa_mode = args.taxa)
File "/home/tool/VirBot/VirBot.py", line 309, in predict
proteins = parse_protein(temp_dir + "protein.faa")
File "/home/tool/VirBot/VirBot.py", line 171, in parse_protein
with open(filename,'r') as f:
FileNotFoundError: [Errno 2] No such file or directory: 'ERR1992810_vb/tmp/protein.faa'
`
Do you know how to solve it?And I have one more question want to know is will you update the ref database file? Thank you very much!

Hello CBT,
Thanks for your comment. I suppose this is because the gene prediction step is not fulfilled and VirBot does not get any proteins as input. Please check the dependant package "Prodigal". if it is not accessible, we will get this error. By the way, have you ever checked the example shown in the guideline? That would be a better test case.
You could type prodigal in your command line to check. And then have a try in the test case.
Guowei

Thank you for your advice! I try it and it works!

Do you know how to export the result of sequences of ncbi taxid?Thanks again!


Cannot register account through Bitwarden-web

I have a docker container for Bitwarden-go, the binary running inside is based on my pull request VictorNine/bitwarden-go/pull/8 that is started this way.
I also have a docker container for Bitwarden Vault which is told to point to http://localhost:8000 as there is were Bitwarden-go listens requests.
Bitwarden-go can response to requests perfectly fine:
jorge@morpho ~ % wget <IP_ADDRESS>:8000/api/accounts/register
--2018-01-05 04:13:34--  http://<IP_ADDRESS>:8000/api/accounts/register
Connecting to <IP_ADDRESS>:8000... connected.
HTTP request sent, awaiting response... 400 Bad Request
2018-01-05 04:13:34 ERROR 400: Bad Request.

However, Bitwarden Vault tells me that "Errors have occurred. · An unexpected error has occurred." and I cannot register my account (or login, I put the credentials I cannot register and try to login but bitwarden vault displays the exact error message).
I don't have any output from the bitwarden-go binary indicating an error.
What am I doing wrong?

Looks like thats the problem. My setting.json file looks like this:
{
"appSettings": {
"apiUri": "/api",
"identityUri": "/identity",
"iconsUri": "https://icons.bitwarden.com",
"stripeKey": "pk_test_KPoCfZXu7mznb9uSCPZ2JpTD",
"braintreeKey": "sandbox_r72q8jq6_9pnxkwm75f87sdc2",
"whitelistDomains": [
"localhost",
"localhost:8080"
]
}
}

So far, I can register at the web-vault. But I get this error: Missing auth header; I am using your neweset version and the newest version of the web-vault. It is 1.22

@shackra
Thanks, I referenced your dockerfiles to build my own alpine-based containers.
I got this to work using reverse proxy with the vaultURL parameter.
The command I used is as follows. The Vault container IP is in docker's default 172.17../16 subnet.
bitwarden-go -init -location /data -vaultURL <vault container IP>:4001
In the Vault container, I copied settings.Production.json to settings.json and changed to env tag in package.json. Access is via port 8000 on the bitwarden-go container.

@thoundernousx Just tried with the latest version. Could you post your settings.json?

Thanks, I tried it yesterday. Now everything works like a charm.

Weird, suddenly I was unable to access Bitwarden-web from the outside running inside a docker container.
I just will use @leonghui alpine versions of both containers and that is.


add mavsdk_server restart if already running at connection

I have a lot of issues trying to connect to my autopilot. Sometimes, it fail because the autopilot boot is not complete or serial port is locked elsewhere. If my connection fail, I cannot retry because mavsdk server is blocked in the error.
On connection, I added a restart of the server if it is already running and was launch by this instance of mavsdk-python. It allows me retry to connect.
It also gives an access to stop a server internally launched.
I am not sure this is a good way to do this. Are you ok with that or do you have suggestions ?

I am all for it if there is a need. Question now is: why does mavsdk_server hang when the serial connection fails? Can't we solve that from C++ by just returning an error and stopping gracefully?

Yes that could be a good solution.

Yes that could be a good solution.

Since you can reproduce that error, I would be super glad if you could have a look at the C++ side. And if we don't find an easy solution there, then we can totally go for this solution :+1:. I'm just thinking that this wrong behavior in C++ will also affect other users in other languages :innocent:.

Yes that could be a good solution.

Since you can reproduce that error, I would be super glad if you could have a look at the C++ side. And if we don't find an easy solution there, then we can totally go for this solution +1. I'm just thinking that this wrong behavior in C++ will also affect other users in other languages innocent.

I had a look on that. We can focus on I of my issues: reboot command.
When, using serial,  mavsdk send a reboot command, serial connection is lost (that is expected). Then I would like to reconnect mavsdk to the autopilot but I cannot. mavsdk does not try to reopen the serial port.
What is the expected behavior:

Should I start a new mavsdk_server ? In this case, I think something similar to what I propose in this PR should be added.
Should mavsdk auto reconnect to the autopilot ? I am not sure how to do that as the autopilot does not use the same port after reboot on Linux (I suppose this is due to mavsdk_server still using the previous one).
Should mavsdk_server exit when serial port is closed ?
Something else ?

Those are interesting points, thanks! To be honest, I don't have much experience with serial connections, so let's try to find something acceptable together :slightly_smiling_face:.

Should I start a new mavsdk_server ? In this case, I think something similar to what I propose in this PR should be added.

I think that you will have the same issue here: you don't know the serial port to be used for the new connection, right?

Should mavsdk auto reconnect to the autopilot ?

I seems a bit tricky here: say mavsdk is connected over a SiK radio. That's a serial connection to the SiK radio, but it could be interrupted (i.e. we could lose the HEARTBEAT messages), while the serial connection is actually still valid. In this case, we don't want to disconnect, we just want to wait for new heartbeats.
But if we disconnect and reconnect the SiK radio from the computer (i.e. unplug and replug the USB), then that may create a new serial device, and mavsdk probably cannot reliably guess that.
Question: would there be a way to detect, from userspace, that the serial device disappeared? Even if that's just working on a desktop Linux, that would help for development. But if that's not possible, I'm afraid that maybe the solution is to manually restart mavsdk_server, or maybe have some kind of custom script (outside from mavsdk_server that would do just that on your setup).
What do you think?

Those are interesting points, thanks! To be honest, I don't have much experience with serial connections, so let's try to find something acceptable together slightly_smiling_face.

I don't have experience with serial connection neither...

Should I start a new mavsdk_server ? In this case, I think something similar to what I propose in this PR should be added.

I think that you will have the same issue here: you don't know the serial port to be used for the new connection, right?

That is not a problem: In my python program, if I can start a new mavsdk_server, I can search a device in serial connections, try all serials while I haven't found the one a want, ask the user to choose, etc.

Should mavsdk auto reconnect to the autopilot ?

I seems a bit tricky here: say mavsdk is connected over a SiK radio. That's a serial connection to the SiK radio, but it could be interrupted (i.e. we could lose the HEARTBEAT messages), while the serial connection is actually still valid. In this case, we don't want to disconnect, we just want to wait for new heartbeats.
But if we disconnect and reconnect the SiK radio from the computer (i.e. unplug and replug the USB), then that may create a new serial device, and mavsdk probably cannot reliably guess that.
Question: would there be a way to detect, from userspace, that the serial device disappeared? Even if that's just working on a desktop Linux, that would help for development. But if that's not possible, I'm afraid that maybe the solution is to manually restart mavsdk_server, or maybe have some kind of custom script (outside from mavsdk_server that would do just that on your setup).

I think that is possible. I don't have sik radio right now to be verify we can distinguish radio disconnection and a serial disconnection but I think that ok and I will check when I can.
It's ok for me to use a custom script to manage mavsdk_server but, just to understand, as mavsdk-python is able create a  mavsdk_server in a  thread, what would be the drawback in allowing mavsdk-python to manage this instance of mavsdk ?

what would be the drawback in allowing mavsdk-python to manage this instance of mavsdk

No I think it makes sense, I was first hoping we could do something nicer at the C++ level, but it seems more complicated than I thought. Let's go with your changes!

tested with this script
import asyncio
from mavsdk import System
import time;
import serial.tools.list_ports

system_address = "serial://" + serial.tools.list_ports.comports()[0].device;

drone = System()
asyncio.run(drone.connect(system_address))

while True:
    time.sleep(1)

    try:
        asyncio.run(drone.action.reboot())
    except:
        pass

    time.sleep(10)
    system_address = "serial://" + serial.tools.list_ports.comports()[0].device;

    asyncio.run(drone.connect(system_address))

My autopilot reboot and reconnect endlessly

what would be the drawback in allowing mavsdk-python to manage this instance of mavsdk

No I think it makes sense, I was first hoping we could do something nicer at the C++ level, but it seems more complicated than I thought. Let's go with your changes!

I will try to improve the serial loss issue in C++ but not will not be able to reconnect so I think this will be helpful


SimpleConsumer: add deprecated to docstring

SimpleConsumer: add deprecated to docstring.
SimpleConsumer constructor params: generation_id, consumer_id are marked as deprecated in docstring
Also extend docstring for UnknownMemberId to explain one cause of the error
https://github.com/Parsely/pykafka/issues/630

Current coverage is 85.65% (diff: 100%)

Merging #631 into master will increase coverage by 0.02%

@@             master       #631   diff @@
==========================================
  Files            27         27
  Lines          3263       3262     -1
  Methods           0          0
  Messages          0          0
  Branches        519        519
==========================================
  Hits           2794       2794
+ Misses          327        320     -7
- Partials        142        148     +6

Powered by Codecov. Last update c3c5507...7817467

Thanks @bootandy, this is great if we decide to actually remove these parameters. See my comment for why I'm not sure we want to do that. Also, if we do decide to merge this, it has an unrelated change in client.py that I think should be removed from this PR.


mistake

how

yes I noticed it only in studio
it's in the Context things
WaitForChil

also the NOTT


Quick.AppService Starting with Param

Hi,
I install the Application with a parameter.
Example: MyApplicaiton MyParam1 /install
After the installation, When i try to start the application ParamStr(1) gives me empty value.
When i make a normal service application without using Quick.AppService, ParamStr(1) gives me the parameter which i give on installation.
How can i get params on start?
Best Regards,
Serhan.

MyServiceConsole.zip
MyServiceConsole.exe TEST /install
Give a parameter when you are installing the application.
(I'm setting the ImagePath at the end of the code)
But i can't get paramtr(1) when i start the application even it's not starting.
On the other hand this approch is working in classic service application.

!! NOTE !!
Also you can't remove the service if you install my service with a Parameter
You need to delete it from registry.


Call for Testing: No more ExceptT

Magicbane git master is now ExceptT-free. (More explanation in the 209cfbf8484c2ef5b5b20eb4a54b728ef45b547f commit message.) The new method of throwing errors is throwIO.
Please test your applications!
/cc @tejon

Building with lts-10.9 (old conduit) I get the following errors:

I'm probably not going to support old conduit…

I'm probably not going to support old conduit…
Good enough for me, just making a thorough report. :)

On the positive side: switching throwError to throwIO fixed my build, didn't even need to change any signatures.


no support for device orientation change?

I installed the demo apk and added the Counter widget to my tablet's home screen. Looks good in portrait, but rotating the tablet in landscape mode results in the widget display being incorrectly sized. Is there a way to handle that in the current code?

As far as I know this is not possible to achieve by using this library.
This is the same or related to #90 and #36
I'll close this since it is not possible (or I couldn't find a way), but feel free to reopen it if you find a way and I can try to implement it.

@sAleksovski A solution could be to specify a port and land layout for each widget as per https://developer.android.com/guide/topics/resources/providing-resources#AlternativeResources . Looks like it also requires removal of onConfigurationChanged() and android:configChanges from the manifest.
Do you think that approach could work?

This looks like configuration for an Activity.
I already tried with separate port and land layout files, but that does not call any method from AppWidgetProvider, which is needed in order to update the image.
Regular Android widgets (defined using xml) can be updated because the data does not change, only the layout depending on the port/land files, but in this library we are rendering to an image and the layout is the same. We need some method from AppWidgetProvider to be called in order to render a new image.


PageModel.GetXpmMarkup() should be split XPM page markup and the script

Sdl.Web.Common.Models.PageModel.GetXpmMarkup() has too many  responsibilities. It should be split into three public methods:

GetXpmPageMarkup()
GetXpmScriptMarkup()
GetXpmScriptUrl()

This is because the XPM page markup  i.e. <!-- Page Settings: {"PageID":"tcm:59-758-64","PageModified":"2017-04-25T14:18:34","PageTemplateID":"tcm:59-585-128","PageTemplateModified":"2017-08-23T10:09:38"} -->
and the script i.e. <script type="text/javascript" language="javascript" defer="defer" src="https://dev/WebUI/Editors/SiteEdit/Views/Bootstrap/Bootstrap.aspx?mode=js" id="tridion.siteedit"></script>  are two distinct things.
HtmlHelperExtensions.DxaPageMarkup() should separately call page.GetXpmPageMarkup() and page.GetXpmScriptMarkup() to render both.
GetXpmScriptMarkup() will in turn call GetXpmScriptUrl() to get the URL.
By making those changes an implementor can extend the way the Page Markup and the script markup is rendered on the page. This is useful when for example delay loading the XPM initialization.

Can you elaborate on the use cases for customizing the XPM markup?  Given your proposed design, it seems you have several use cases in mind.
Would it be an option to let PageModel.GetXpmMarkup() call GetXpmScriptMarkup(), which calls GetXpmScriptUrl() in turn (and let DxaPageMarkup() only call PageModel.GetXpmMarkup()) ?

A customization I am working now is to delay loading the XPM script.
Usually the XPM script loads after the page is parsed using <script type="text/javascript" language="javascript" defer="defer" src="https://server/WebUI/Editors/SiteEdit/Views/Bootstrap/Bootstrap.aspx?mode=js" id="tridion.siteedit"></script>. We are using Vue (a frontend scripting framework) to render some views on the frontend. To make sure that XPM components and fields which are inside those Vue views are rendered correctly, we load the XPM script when Vue has rendered all Vue components and the mounted() hook is called.
The XPM page metadata (the XPM HTML comments) can be loaded like usual, but the XPM JavaScript script cannot be loaded with the <script> tag outputted by GetXpmMarkup(). For now I have a custom view helper which renders just the XPM page metadata. A PageModel extension provides the XPM script URL which is passed to the Vue configuration.

@rpannekoek: about the DxaPageMarkup() and other methods. Why not separate the XPM page metadata and the script tag? They do not have to be rendered by the same helper method because they are separate things. The XPM page metadata is similar to the XPM component metadata, it is fine to leave it to the page view or the _Layout view. The XPM script tag could be loaded elsewhere, depending on the implementation.

The XPM page markup will be generated by the DxaPageMarkup() helper, calling PageModel.GetXpmMarkup()
The XPM script markup will be generated by the DxaScriptMarkup() helper, calling PageModel.GetXpmScriptMarkup(), which calls PageModel.GetXpmScriptUrl()


Docu Fixes

See: https://github.com/ArcadeData/arcadedb/discussions/1231 .

Fixed by https://github.com/ArcadeData/arcadedb-docs/pull/196


Updates

The primary goal of this is removal of core-js dependency decreasing install size from 16mb to 7.7mb.
I did the visitorKeys sync in a separate commit to show that the out-of-range updates did not require any changes to test fixtures.

It would be awesome if this PR was merged. Warning messages on installing software is not nice to see :(

@twada, can you please merge this?


problem in testing

I can only get high dice when training in" fold all",but it doesn't work when testing. What should I do to find the best configuration? Just
turning "(0,1,2,3,4)" into "all," in "find_best_configuration.py" and pythoning it don't work.

Hi @gumayusi3 ,
We didn't train so many folds.
Did you test this function with nnunet? If it works for nnunet, it should also work for u-mamba.

Thank you,I have solved this problem. The reason of low dice is that dataset is too small.I get high dice just because training dataset is same as validation when training in "fold all" .

Hi, I'm having the same problem.
May I ask, what is the number of datasets before and after your expansion? What are the dice values after expansion?
Thank you very much!

The dataset had about 50 plots before augmentation, and after augmentation to 1000 plots, Dice increased from about 0.3 to about 0.7, but still far from the theoretical optimum


Running Rancher 2.5 (Single Install) in LXC container fails

What kind of request is this (question/bug/enhancement/feature request): Question
Steps to reproduce (least amount of steps as possible):

Create LXC Container using Ubuntu Focal image with relevant config settings (see https://github.com/lxc/lxd/issues/4902)
Install docker.io in container
Docker run command:

root@lxc:~# docker run -d --privileged --restart=unless-stopped -p 80:80 -p 443:443 rancher/rancher

Result:
root@ubuntu4:~# docker logs --follow rancher
ERROR: Rancher must be ran with the --privileged flag when running outside of Kubernetes
ERROR: Rancher must be ran with the --privileged flag when running outside of Kubernetes
ERROR: Rancher must be ran with the --privileged flag when running outside of Kubernetes
ERROR: Rancher must be ran with the --privileged flag when running outside of Kubernetes
ERROR: Rancher must be ran with the --privileged flag when running outside of Kubernetes
ERROR: Rancher must be ran with the --privileged flag when running outside of Kubernetes
ERROR: Rancher must be ran with the --privileged flag when running outside of Kubernetes
ERROR: Rancher must be ran with the --privileged flag when running outside of Kubernetes

Expected Result:
Container starts, Rancher starting up.
Other details that may be helpful:
Other privileged Docker containers start correctly:
root@lxc:~# docker run --privileged hello-world
Unable to find image 'hello-world:latest' locally
latest: Pulling from library/hello-world
b8dfde127a29: Pull complete
Digest: sha256:9f6ad537c5132bcce57f7a0a20e317228d382c3cd61edae14650eec68b2b345c
Status: Downloaded newer image for hello-world:latest

Hello from Docker!
This message shows that your installation appears to be working correctly.

To generate this message, Docker took the following steps:
 1. The Docker client contacted the Docker daemon.
 2. The Docker daemon pulled the "hello-world" image from the Docker Hub.
    (amd64)
 3. The Docker daemon created a new container from that image which runs the
    executable that produces the output you are currently reading.
 4. The Docker daemon streamed that output to the Docker client, which sent it
    to your terminal.

To try something more ambitious, you can run an Ubuntu container with:
 $ docker run -it ubuntu bash

Share images, automate workflows, and more with a free Docker ID:
 https://hub.docker.com/

For more examples and ideas, visit:
 https://docs.docker.com/get-started/

So it look to me as if the LXC container itself would allow to run the privileged Rancher container, however something more seems to be required by Rancher. But what? Nothing can be seen in the logs.
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): Tested with Rancher latest and 2.5.8
Installation option (single install/HA): single install
Docker version:

root@lxc:~# docker version
Client:
 Version:           20.10.2
 API version:       1.41
 Go version:        go1.13.8
 Git commit:        20.10.2-0ubuntu1~20.04.2
 Built:             Tue Mar 30 21:24:57 2021
 OS/Arch:           linux/amd64
 Context:           default
 Experimental:      true

Server:
 Engine:
  Version:          20.10.2
  API version:      1.41 (minimum version 1.12)
  Go version:       go1.13.8
  Git commit:       20.10.2-0ubuntu1~20.04.2
  Built:            Mon Mar 29 19:10:09 2021
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.4.4-0ubuntu1~20.04.2
  GitCommit:
 runc:
  Version:          1.0.0~rc95-0ubuntu1~20.04.1
  GitCommit:
 docker-init:
  Version:          0.19.0
  GitCommit:

Found it!
In commit https://github.com/rancher/rancher/commit/b0919dd45d1833bcf38db342178a6a3f2718a24f, the path /dev/kmsg is explicitly checked in the entrypoint.sh script.
This path needs to be mounted from the LXC host into the LXC container which serves as Docker/Kubernetes Host. See https://discuss.linuxcontainers.org/t/rancher-2-5-privileged-docker-container-in-privileged-lxc/11416.

Hi How to swtich crgoup version? I stuck in switching cgroup.   #35201


Official F-Droid release

Are there any plans to release this application on the official F-Droid repository?

Not really but I'll look into it when I'm free.

At least it's available on IzzyOnDroid for now

Where we'd like to lift it up one level and confirm it's a Reproducible Build (see: Reproducible bulds, special client support and more in our repo) – but I could not figure out how to build the specific APK we use there. The build spits out several APKs, just not that one.
@LittleTrickster could you give me a hint on how to build that? Obviously, ./gradlew assembleRelease doesn't do the trick.
We'd appreciate if you could help making your build reproducible. We've prepared some hints on reproducible builds for that.
Looking forward to your reply!

@LittleTrickster any word? Thanks in advance!

OK, I've managed to build the specific APK now (by altering the build.gradle using sed -r '/abi \{/,/}/ { s/include.*/include "arm64-v8a"/ }' -i app/build.gradle), but it is not RB. Here's the APK diff:
-------------------------------
This can probably be ignored if the above error was
'Error: APK Signing Block offset < central directory offset.'
and the following shows the signature files as the only difference:
-------------------------------
--- /dev/fd/63  2024-08-08 10:02:21.267780725 +0200
+++ /dev/fd/62  2024-08-08 10:02:21.267780725 +0200
@@ -1,11 +1,11 @@
   META-INF/com/android/build/gradle/app-metadata.properties
   32-bit CRC value (hex):                         fadd38bc
   assets/dexopt/baseline.prof
-  32-bit CRC value (hex):                         6f3b3aa9
+  32-bit CRC value (hex):                         b67e3364
   assets/dexopt/baseline.profm
   32-bit CRC value (hex):                         59be0682
   classes.dex
-  32-bit CRC value (hex):                         03370966
+  32-bit CRC value (hex):                         f7ff0ade
   classes2.dex
   32-bit CRC value (hex):                         83d303cb
   lib/arm64-v8a/libc++_shared.so
@@ -20,6 +20,18 @@
   32-bit CRC value (hex):                         d17f9abb
   lib/armeabi-v7a/libopencv_java4.so
   32-bit CRC value (hex):                         10e8443d
+  lib/x86/libc++_shared.so
+  32-bit CRC value (hex):                         346622e6
+  lib/x86/libimage_processing_util_jni.so
+  32-bit CRC value (hex):                         8031e066
+  lib/x86/libopencv_java4.so
+  32-bit CRC value (hex):                         e58e5240
+  lib/x86_64/libc++_shared.so
+  32-bit CRC value (hex):                         3e046e92
+  lib/x86_64/libimage_processing_util_jni.so
+  32-bit CRC value (hex):                         afc57b58
+  lib/x86_64/libopencv_java4.so
+  32-bit CRC value (hex):                         6fca9e01
   assets/com/tom_roush/fontbox/resources/cmap/83pv-RKSJ-H
   32-bit CRC value (hex):                         506463a6
   assets/com/tom_roush/fontbox/resources/cmap/90ms-RKSJ-H
@@ -1395,9 +1407,9 @@
   META-INF/kotlinx_coroutines_core.version
   32-bit CRC value (hex):                         10dbda9d
   META-INF/services/a6.y
-  32-bit CRC value (hex):                         845a12c4
+  32-bit CRC value (hex):                         755dff9c
   META-INF/services/kotlinx.coroutines.internal.i
-  32-bit CRC value (hex):                         861cac9d
+  32-bit CRC value (hex):                         5e70ac5f
   kotlin-tooling-metadata.json
   32-bit CRC value (hex):                         955d2730
   kotlin/annotation/annotation.kotlin_builtins
@@ -1418,9 +1430,3 @@
   32-bit CRC value (hex):                         71c94b37
   org/bouncycastle/x509/CertPathReviewerMessages_de.properties
   32-bit CRC value (hex):                         8357ba7f

@IzzySoft Sorry for late reply
did some changes to target specific abi or just arm or universal using Project property singleAbi. Default still splits to many apks.
singleAbi param values arm, x86 x86_64 arm64-v8a armeabi-v7a universal
example
gradlew -PsingleAbi="arm" assembleRelease

By RB what do you mean?

By RB what do you mean?

Scroll up a little: Reproducible Builds, special client support and more in our repo. And I needed to build the "universal ARM" APK which is commented out in your build.gradle, so I could compare it against the one attached to your releases. Unfortunately, the builds were not identical, see the diff above. So I don't know if you maybe built from a different commit, or if my way to achieve the "double arm" just causes different results than yours. Unfortunately, in your build.gradle you don't offer a target I could call directly, without altering the file.
For RB, in short: a successful RB basically confirms your APK was really built from the source the tag points to, without any "local modifications" which might have "slipped something in". So it's an additional security indicator.

@IzzySoft  Changed app build gradle in latest commit
For arm ./gradlew -PsingleAbi="arm" assembleRelease will add both arm64-v8a and armeabi-v7a to apk

@IzzySoft Sorry for late reply did some changes to target specific abi or just arm or universal using Project property singleAbi. Default still splits to many apks. singleAbi param values arm, x86 x86_64 arm64-v8a armeabi-v7a universal example gradlew -PsingleAbi="arm" assembleRelease

Thanks! Then I can verify again once a new release is available (as verification needs 2 APKs built from the very same commit: one by you, one by the verification builder). Are there any plans for a new release yet? Once that day comes, please remember the "first basic rule" from our hints on reproducible builds – and build the APK from a clean tree at exactly the commit the tag will point to :wink:

And btw, gradlew -PsingleAbi="arm" assembleRelease produces the following APKs:
+ find . -name '*.apk'
./app/build/outputs/apk/release/app-arm64-v8a-release-unsigned.apk
./app/build/outputs/apk/release/app-x86_64-release-unsigned.apk
./app/build/outputs/apk/release/app-universal-release-unsigned.apk
./app/build/outputs/apk/release/app-x86-release-unsigned.apk
./app/build/outputs/apk/release/app-armeabi-v7a-release-unsigned.apk

I don't see the APK combining the two ARM ABis there. But that was at tag 1.03– I see that singleAbi has only been introduced after that. So any ETA for a release including this?


feature request: converter for cargo-deny

It would be great to have a deny-sarif tool to convert the output of cargo-deny (it supports json) to sarif format.

Agree it would be nice, open to additions, or if I find some time in the future I can look into it.


Reference persistence in forEachSegment can confuse consumers

Calling the forEachSegment callback with the same array over and over with only with its elements reassigned new values led to some tricky bugs to track.  By copying into new arrays in the callback call, we dodge having the same array reference presented to the callback on every iteration, so it can do what it likes with what is passed.
Here I have made the absolute minimum change to the segment.js file, and modified a test to be functionally identical, but reveal the tricky effect if the patch is not applied.

Out of curiosity: where did you run into problems using this function?
It is absolutely intentional that this function reuses the output array, because it is called very frequently and would generate a lot of garbage otherwise. Furthermore, it is not meant to be used by consumers of the library. It is not part of the official API and should not be used in applications.

I ran into problems when I was attempting to iterate down a lineString, on
which we are placing near each segment of the string an overlay that
provides further interaction capabilities with our application.  I needed
to obtain the sequence of segments - so I went to the api and found that
there the only sequence-of-segments evidently available was a publically
available method on the LineString object called 'forEachSegment'.  I used
it to aggregate up the resulting callbacks (with the confusion of a
'forEach' named thing that is actually a 'some' logic processor) so that I
could filter and process that resulting list in surrounding code, which
allow me to identify my overlays (which I've named by ids corresponding to
their point-to-point definition) and manage them in relation to the
LineString at the proper time.  It was something like this:
const segments = Coordinate[][]
lineString.forEachSegment((first, last) => segments.push([first, last]) &&
false)
segments.map.filter.forEach.other.functional.chains.on.the.list
which, as you can see, is a problem.
I am skeptical that it is called very frequently. In fact, my own debugging
(in which I instrumented forEachSegment directly with a console.log) found
no unexpected not-called-by-me utilizations on a map on which I had placed
a number of linestrings - admittedly, it was a test map and there weren't
tons of other features that could have similarly used the forEachSegment -
I can only say for sure that LineString certainly doesn't call it
frequently.
I am skeptical that creating a new Array object would be a performance
issue. This is the absolute fastest form of array - an array of - and a
very small array at that. The garbage collection is specifically optimised
for things going out of scope, which is exactly how the forEach callback
arguments will be handled - they'll go out of scope as the flow exits the
callback() args making no garbage overhead.
The code structure may be valid if these reasons have merit - do you have
any tests or experiments that have been done to validate these
assertion, such that my patch would perform measurably differently in even
the most complex situations your performance tests may devise?
It feels like premature optimization to me, but I could be wrong.
Also, If the list of public methods listed in the api reference
https://openlayers.org/en/latest/apidoc/module-ol_geom_LineString-LineString.html#forEachSegment
is
not where I am to find 'available operations to consume', where is the
blessed list?
Also, I'm working on a support library for geoinformatics display within
the organization I work, so it may be that I'm more likely than your
typical utilizer to be reaching into api functions...?
David
On Fri, Jul 31, 2020 at 1:46 AM Andreas Hocevar<EMAIL_ADDRESS>wrote:

Out of curiosity: where did you run into problems using this function?
It is absolutely intentional that this function reuses the output array,
because it is called very frequently and would generate a lot of garbage
otherwise. Furthermore, it is not meant to be used by consumers of the
library. It is not part of the official API and should not be used in
applications.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/openlayers/openlayers/pull/11371#issuecomment-666938852,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFB4LQ5FQ5R62O2H4WGLQLR6JLCDANCNFSM4PPF7DSA
.

--
David Ihnen
Voice contact<PHONE_NUMBER>

@DavidIAm Ok, now I see what you mean! The function you changed (ol/geom/flat/segments.forEach() is an internal one. The one your application calls (ol/geom/LineString#forEachSegment()) is definitely public. And I agree that for this public function, the behavior is unexpected. What about just modifying the callback directly in forEachSegment()? Something like this:
  forEachSegment(callback) {
    return forEachSegment(
      this.flatCoordinates,
      0,
      this.flatCoordinates.length,
      this.stride,
      function(coord1, coord2) {
        return callback.call(this, coord1.slice(), coord2.slice());
      }
    );
  }

Note that this library does not use ES6 features except class and modules yet.
You may be right that the way we reuse small arrays may be premature optimization. I have not verified this technique myself, and I do not know if those who originally wrote these functions (we have several written that way) have.

I can certainly amend my pull request to do what you suggest.  Confirm and I'll do just that as I do agree this is a startling behavior for consumers.  Do you think there should be a comment there about it? I can amend it that way if you like.  But I do want to keep talking about it. (thank you for engaging so politely with me!)
Were this code I was part of owning, I'd probably be arguing for adding a Symbol.Iterator to LineString that when called yields a sort of simple segment object (which are a {first: Coordinate; last: Coordinate;} object for instance, or the simple Coordinate[] you were using before). Now maybe that should be on a sub field (Linestring.segments?) but that's what I'd be discussing, the whys and wherefores of that.
I'm not quite sure to make of 'uses some es6 features' with the implied 'but not all of them'.  You literally have features enabled that make your code nicer and you... intentionally - not as 'we haven't gotten around to refactoring all the other code yet' but literally intentionally not using them in new code you write.  wat?
So in the end you just use array.slice() instead of [...array] it seems, so you don't use the es6 feature?
Well, your suggestion is a valid way to address the 'unexpected consumer behavior' it adds a layer abstraction but to what benefit? The only other place this flat/segment.js foreach seems to be called is in the intersectsExtent logic, which appears to have a complexity of O((n+1)*2) arrays created and garbage collected as a differential from pre-patch condition. I would hypothesize that even worst case - a freehand thousand segment line - you wouldn't be able to detect the overhead of the array builds. This really has a narrow scope- linestring and the extents intersector.
Were I opining on 'how I would prefer to have seen this' I would hope for something like... (and I do hope you forgive the typescript es2019 as that is the world I run in)
implied here is that my code where my overlay objects have listeners that self delete.  I wouldn't have to subvert forEachSegment as i have an iterator handy... which I can use and abuse as I please - and of course the internal utilizer-of-iterator (flat/segment.js#forEach) would similarly use the iterator to do its job
[...lineString] // or maybe [...lineString.segments]
   .filter(segment => segment.length())
   .map(segment => MyOverlayClass.makeId(segment))
   .map(id => this.getMap().getOverlayById(id))
   .forEach(overlay => overlay.dispatchEvent("delete"))


node: skip non-latest odd majors

When running with type majors, it doesn't seem likely that the user is interested in odd-numbered majors except current.
For example, running today with a major range of >=16, the results of deprecated v17 seems irrelevant.
@ljharb If you think this sounds reasonable I could take a stab it at

On all my projects I'm definitely interested in odd majors being required; being odd or even doesn't have any bearing on the semver of what a package supports.
An enhancement I would like to see on node/install particularly is one that reads the range from engines.node in package.json, though.


Added a test file for game_board.py

can you move this to tests directory?

also this is work in progress PR right?

yup
On Tue, Dec 10, 2019 at 10:48 PM Mayank Singh<EMAIL_ADDRESS>wrote:

also this is work in progress PR right?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/code-monk08/connect4/pull/22?email_source=notifications&email_token=AMU74CQHX4MQNOEHQGL6P33QX7FPVA5CNFSM4JZB6TC2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGQBFLI#issuecomment-564138669,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMU74CQDQVWAFIACWEXD6OTQX7FPVANCNFSM4JZB6TCQ
.

@code-monk08 I'm having trouble importing game_board if i put the test file in a separate directory.Any suggestions on what to do?

nvm got it sending a pr


Help needed: Flexx and watchdog

Hello all,
I'm posting here since my question on SO doesn't seem so popular and there is very little going on about flexx there (which is probably good!).
I hope this is allowed and if not, then feel free to delete this issue.
I am using watchdog to monitor for new files inside a directory. This library calls for an event handler (subclass of their FileSystemEventHandler) which I inherit from. In my event handler method (which gets triggered when a new file is created), I would like to call a PyComponent's method in order to mutate its state.
So far I've found that I can get a reference to the app's root component instance with from flexx.app import manager; manager.get_connections('MyAppName')[0].app but if I try to call one of its methods, I get the following trace:
  File "myfile.py", line 37, in on_created
    manager.get_connections('MyAppName')[0].app.update_verbose(False)
  File "lib\site-packages\flexx\event\_action.py", line 150, in __call__
    if loop.can_mutate(ob):
  File "lib\site-packages\flexx\event\_loop.py", line 85, in can_mutate
    active = self.get_active_component()
  File "lib\site-packages\flexx\event\_loop.py", line 103, in get_active_component
    if len(self._local._active_components) > 0:
AttributeError: '_thread._local' object has no attribute '_active_components'

Note that update_verbose is decorated with @flx.action.
I tried a different approach where instead of subclassing FileSystemEventHandler I was subclassing flx.PyComponent and re-implementing the methods required for watchdog (and instantiating this event handler from my main PyComponent's init function), but it didn't help, the stack trace was essentially the same.
Any help will be greatly appreciated! Thanks

Yes, you are allowed and encouraged to post questions here :)
The direction that you are taking could work, but is probably not what you want. If you have a desktop application, it's easy:
root = flx.launch(MyPyComponent)  # there is your instance!

If you are making more of a web server, it's probably easiest to make use of a regular Component. These are unaware of any Py-JS syncing, so they can be instantiated as a global object. But because they are components, they can be hooked up to PyComponents. Something like this:
from flexx import flx

class GlobalComp(flx.Component):

    foo = flx.StringProp("hello", settable=True)

global_instance = GlobalComp()

class PyComp(flx.PyComponent):

    def init(self):

        self.label = flx.Label()

    @flx.reaction
    def _update_foo(self):
        self.label.set_text(global_instance.foo)

if __name__ == "__main__":
    flx.serve(PyComp)
    flx.run()

Hey, thanks for the quick reply! It's indeed a desktop app, so I tried getting the root instance from the launch call and the result is the same:
AttributeError: '_thread._local' object has no attribute '_active_components'
I think the core of the problem is that the FileSystemEventHandler is running on a different thread maybe? Is it weird that self in self._local._active_components is actually _thread or is that normal?
  File "myfile.py", line 36, in on_created
    root.update_test('asdf')
  File "lib\site-packages\flexx\event\_action.py", line 150, in __call__
    if loop.can_mutate(ob):
  File "lib\site-packages\flexx\event\_loop.py", line 85, in can_mutate
    active = self.get_active_component()
  File "lib\site-packages\flexx\event\_loop.py", line 103, in get_active_component
    if len(self._local._active_components) > 0:
AttributeError: '_thread._local' object has no attribute '_active_components'

Not self but the _local of a Loop object stores active components per thread.

I think the core of the problem is that the FileSystemEventHandler [...] is running on a different thread maybe?

That's very likely. You'll need to invoke a call in the event loop. I think flx.loop.call_soon() can do this for you.

My man!! that did the trick very nicely! Calling flx.loop.call_soon(root.my_action, 'foo') from the FileSystemEventHandler instance totally works. Thanks!


docs: fix typo in gatt_client example readme (IDFGH-11030)

Fix typo of 'UIID16' found in gatt_client example documentation README.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for your contribution.

fixed
bce1715dfdfe791ab950f7fcb94d537b74c90025


Dutch translation work

Fix a small typo, missing 'c' in project in en.json;
Add Dutch translation files.

Thanks for that!


IndigoException: element: can not calculate valence on C, charge 0, connectivity 6

Steps to Reproduce

Place chain on the canvas
Change some bonds to triple
Click Layout

Expected behavior
The structure is changed their position
Actual behavior
Error is shown
Screenshots

Desktop (please complete the following information):

OS: Chrome Version 93.0.4577.63, Firefox Version 92.0, Microsoft Edge Version 93.0.961.47

Related issue: https://github.com/epam/Indigo/issues/533

Fixed, tested v.2.4


[BUG] Microsoft.Azure.Services.AppAuthentication 1.2.0 breaks compatibility

Microsoft.Azure.Services.AppAuthentication 1.2.0 breaks compatibility with apps that depend on a previous version. For example, I'm using Microsoft.Extensions.Configuration.AzureAppConfiguration which specifics >= 1.0.3.
A CancellationToken was added to AzureServiceTokenProvider.GetAccessTokenAsync. The existing method signature was modified with an optional parameter, instead of adding a new method. If breaking changes like this are added, shouldn't the version bump to 2, or was this a mistake?
The following exception is thrown:
Method not found: 'System.Threading.Tasks.Task`1<System.String> Microsoft.Azure.Services.AppAuthentication.AzureServiceTokenProvider.GetAccessTokenAsync(System.String, System.String)

To repro:
Follow steps here and update to Microsoft.Azure.Services.AppAuthentication 1.2.0: https://docs.microsoft.com/en-us/azure/azure-app-configuration/howto-integrate-azure-managed-service-identity

Thank you for opening this issue! We are routing it to the appropriate team for follow up.

@markgould Thank you very much for bringing this to our attention! We will be fixing this ASAP, targeting early next week.

@nonik0, any update for the issue?

@markgould Thank you for providing the repro steps. AppAuth 1.2.1 was just released that has the fix for this issue.
@erich-wang FYI.

This looks like it was addressed and never closed. I'm going to close it now.


Split *Dense into *Dense and *MaskedDense

@kabaka0 thoughts?  The idea is to as per #128 - the tensor package holds the data structures, and the essential methods (basically implementing Tensor), while things like Add will get moved to tensor/ops
Tasks

[ ] Write tests to ensure that all the new methods and functions in internal/stdeng can be accomplished using FlatMaskedIterator
[ ] If yes, then split into *Dense and *MaskedDense

I can see the logic behind the split of tensor - the package is currently quite large, which made it harder to locate methods. As for the split of *Dense, The one problem I can see with having *MaskedDense is that a quite a bit of code would be duplicated. I suppose that we could embed *Dense in *MaskedDense to minimize this. It would not have worked with my initial attempt at Masked (in which the mask was allowed to have an iterator with different stride from the data), but given the decision to drop that, it can be done.
Modifying genlib to support such a change would not need much code editing though, given that there already are checks for whether the object is masked or not.
Can you see any other downside to the embedding?

I was thinking precisely embedding
    *Dense
    mask []bool
...
}```

(I'll do the heavy lifting.. given I'm working on the sparse and colmajor Tensors) you just provide the struct you have in mind @kabaka0

That works.
type MA struct {
    *Dense
    mask []bool
    fillValue interface{}

...
}

I am happy to help with it, but if you are actively editing the package then I can leave it to you for now, but feel free to ask me to do stuff whenever needed.

This issue was moved to gorgonia/tensor#8


Better score font/thing

Grows then shrinks?
Changes color then back again?

Star icon instead of words


Copy collections in the DynamicAdapter

Use copied collection instead of original to prevent exceptions related to concurrent modification.

Fixed


Remove submission get-file CLI command

It has already been removed from the OpenAPI definition.

https://github.com/ddbj/ddbj-repository/commit/58a6479e552e6b336b5eeb746e586190bedbb2b3

Instead of following the API formally, we should consider user-friendliness.

There is no API definition, but this is accomplished by using Validation's API as with the Web, rather than deleting commands.


[Alert manager] Add firing status to the cert expiration checker alert

Add "status": "firing" to the alert.
So the alert will be shown on the webportal home page.

Coverage remained the same at 33.818% when pulling 3c5e425e42e738df5640552b11b3101c60909d8e on yiyi/fix_alert_from_cert_expiration_checker into ce9b957018dae40418eb82ae375ef6d883501678 on master.


Shiny reactivity of data source for treemap causes multiple treemaps to appear

Please run this shiny app to see the problem. Simply change the input and new treemaps will appear, a new one for every update.
Best regards,
Stephen
# As you modify the rows of data being used from GNI (reactivity),
# multiple objects appear on the web page, one treemap per update

library(d3treeR)
library(treemap)

server <- function(input, output) {

   # example 1 from ?treemap
   data(GNI2010)

   GNIR <- reactive({ GNI2010[1:input$rows, ] })

   output$tree1 <- renderD3tree3({

      tm <- treemap(GNIR()
           ,index=c("continent", "iso3")
           ,vSize="population"
           ,vColor="GNI"
           ,type="value"
      )

      d3tree3(tm, rootname = "World")

   })

}

ui <- fluidPage(
   numericInput("rows", "Rows from GNI?", 20, min = 5, max = 208),
   d3tree3Output('tree1', height = "600px")
)

shinyApp(ui = ui, server = server)

Thanks for the report.  The fix should be easy, but I am away from my computer this week.  This was a contributed version, and I guess my code review was not thorough enough.  Sorry for the trouble.

Glad to hear it should be a straightforward fix. Excellent package BTW.
Looking forward to fix. Cheers!
Much appreciated,
Stephen


goland debug page 500

goland  after  debug or run
Unable to find status/500 template
2022/01/24 19:02:18 ...ers/common/logger.go:21:1() [I] Started GET / for <IP_ADDRESS>:34106
2022/01/24 19:02:18 ...s/context/context.go:206:HTML() [E] Render failed: html/template: "install" is undefined
2022/01/24 19:02:18 ...s/context/context.go:290:PlainTextBytes() [E] PlainTextBytes: Unable to find status/500 template
2022/01/24 19:02:18 ...ers/common/logger.go:30:1() [I] Completed GET / 500 Internal Server Error in 469.837µs

https://docs.gitea.io/en-us/hacking-on-gitea/


[1] - redesigned Service-Providers, courses and videos page

Addresses: https://github.com/a2i-code-For-Govstack/nise3-skill-and-certification/issues/1
Service-Providers, courses and videos page redesigned

Correct "Skill Service Providers". The cards size is inconsistent across all the pages also some have card shadows some don't.
Replace the line under the header with  tag and leave some space between the card borders and  tag.

fixed issues

replaced card-header border with <hr> tag
fixed card size

** code pushed on trainee branch


Remove CHAR_INFO munging for raster fonts

RealUnicodeToFalseUnicode was described as:

This routine converts a unicode string into the correct characters
for an OEM (cp 437) font. This code is needed because the gdi glyph
mapper converts unicode to ansi using codepage 1252 to index font.
This is how the data is stored internally.

In other words, it takes a UCS2 string, translates it to the current
codepage and translates it back to UCS2 in the US version of Windows.
In the "eastern" DBCS version it "reinterprets" the DBCS string as
CP_USA (a particularly weird quirk).
The original implementation used to do this translation at every
opportunity where text went into or out of conhost.
The translation was weird, but it was consistent.
In Windows 10 RS1 conhost got a new UCS2-aware text buffer and
this translation was removed from most places, as the text buffer
was converted to store proper UCS2. This broke the entire concept
of the translation though. Whatever data you previously wrote with
something like WriteConsoleOutputCharacter now came back with
something entirely else via ReadConsoleOutput.
In other words, I believe past RS1 there was technically never any
point in "munging" CHAR_INFOs, as this only covered 2 API functions.
Still, this does mean that this PR represents an API breaking change.
It's a minor one though, because it only affects 2 API functions.
And more importantly, it's a necessary breaking change as we move
further and further away from correlating codepoint and column counts.
Validation Steps Performed

Remaining tests pass ✅

Given that I was wrong in my initial PR description, I've done some more & better due diligence:

This essentially uses WriteConsole as a test. The test string is:

カキクケコガギグゲゴサシスセソザジズゼゾタチツテトダヂヅデドハヒフヘホバビブベボパピプペポ
｛カキクケコ｝
（カキクケコ）
［カキクケコ］
【カキクケコ】
「カキクケコ」
『カキクケコ』
〝カキクケコ〟
⟨カキクケコ⟩

This basically shows what I meant with that "munging" was removed from anything but these two CHAR_INFO APIs. I believe this means we've already done a significant breaking change in the past and I think it's fine to break the remaining 10% as well. This does run the risk of breaking an old application, but it at least means that the behavior is consistent and predictable across all APIs.
I'm considering writing some additional tests to run on each version, but merely looking at the code I know that any version past the first one lacks support for the Unicode "falsification" and it's certainly not in our main branch right now either.


Add Multipass

Add Multipass
Microsoft Reviewers: Open in CodeFlow

It seems that you can only commit one yaml file per one pull request.

You should add package manifests in different branches.


Discussion: merge vault-configurer/vault-sidecar-unsealer with vault-operator

Is your feature request related to a problem? Please describe.
Wondering if we really need to have operator/configurer/unsealer as operator can actually do all 3.
Describe the solution you'd like
Operator does what a human operator would do in an automated way. I would think operator can do all the work it doesn't have to be a separate pod/container for each of these tasks. If there is a good reason to keep it that way then it is fine. I just wanted to understand/check on the design

Hi @raoofm, this is because bank-vaults can be used as a CLI tool and we don't want to repeat the same code in the operator, the other issue is that if all Vault instances would be handled by the same Pod (the operator) it would be hard to give proper access resolution, since in this setup all Vaults would be controlled by one instance (unseal, root token...).

@bonifaido To add to the list of pros: the permission handling for the Vault Operator obtained through https://github.com/operator-framework/community-operators/tree/master/upstream-community-operators/vault would be more effective with Operator Lifecycle Manager if there would be a single Operator.
Currently OLM cannot deploy bank vault in a way where it watches all namespaces, as it creates the service account vault in the namespace where the operator is deployed, not where the CR is instantiated.

this is because bank-vaults can be used as a CLI tool and we don't want to repeat the same code in the operator

can be reused not repeated.

if all Vault instances would be handled by the same Pod (the operator) it would be hard to give proper access resolution, since in this setup all Vaults would be controlled by one instance (unseal, root token...).

May be makes sense as sidecar simplifies for unseal/init (though an operator can still do it) but configurer seems like a fit for operator - no?

@bonifaido feel free to close it if merging configurer to operator doesn't make sense.

No, currently there are no plans to do that but I would keep this issue open for future reconciliation.


[Sign In] 'Email' and 'Password' fields are not highlighted after trying to sign in with registered email and incorrect password

Environment: Windows Server 2012 R2 Standard x64, Google Chrome 81.0.4044.129,
Mozilla Firefox 76.0 (64-bit)
Reproducable: always
Build found: commit 7439e88
Preconditions
There is at least 1 registered user.
Steps to reproduce

Go to GreenCity https://ita-social-projects.github.io/GreenCityClient/#/welcome
Click on 'Sign-in' link.
Set valid registered email address into 'Email' field.
Set incorrect password into 'Password' field (e.g. "Qw123456.").
Click on Sign-in button.

Actual result
'Password' and 'Email' fields are not highlighted in red color and error message is displayed under 'Password' field: "Bad email or password".

Expected result
'Password' and 'Email' fields are highlighted in red color and error message appears under 'Password' field: "Bad email or password".

User story #471
Test case https://jira.softserve.academy/browse/GC-30

Fixed

Verified.


Adding LineSeg class and relevant methods in other classes.

Added a class for representing straight line segments that form parts of street edges.

Woohoo, first pull request merged!


Accessing column with column name instead of indexe

I am trying to get the column with index successfully as below:
var col1 = this._grid.fields[1]; //Working
But the problem is when in future as per requirement we are adding new column to grid that will change the indexes of columns, is there any way I can access the fields by it name instead of its index ?
var col1 = this._grid.fields["Col1"]; //Not Working

Fields is a list of objects. The field name is one of the object keys, so you have to iterate over the list entries until you find the field you are looking for. (see https://stackoverflow.com/a/12462414)
var col1 = $("#jsGrid").jsGrid("option", "fields").find(o => o.name === "Col 1")
In your case:
var col1 = this._grid.fields.find(o => o.name === "Col 1")

@TimoBekon Thanks for the reply but I am getting syntax error in javascript when using bellow line of code:
var col1 = this._grid.fields.find(o => o.name === "Col 1")
And it is not sowing what syntax is erroneous

@vyasrohit1983
try this one
var col1 = $("#jsGrid").jsGrid("option", "fields").filter(o => o.name === "Col 1")


pronunciation button covering the text

OS: Android
Device name: Realme GT NEO2

would you mind if i fix this issue?
that's gonna be my last PR for hacktoberfest :)


ref #32: add as_of parameter for previous_version, next_version, current...

..._version
as_of defaults to 'end'

closing because it seems something went wrong with travis build, will re-submit pr


memory error when deferred_load = False in fieldset

Hi! I had similar problem than #658. Following @delandmeterp suggestions I set deferred_load=False . A short simulation was working for parcels version 2.1.2 but not for version 2.1.4, where fieldset crashes with MemoryError.  I tried to use field_chunksize to auto, False and a number without success.
So my question is if there is somehow a better way to solve the calendar issue without scarifying the deferred_load option.
Thank you!

Have you tried using the timestamps keyword argument, as explained in https://nbviewer.jupyter.org/github/OceanParcels/parcels/blob/master/parcels/examples/tutorial_timestamps.ipynb? Does that fix this issue?

Thanks Erik!, It works but I had to split my netcdf files into single time-step files to make it work.
But  now some another errors appeared again (#712) : recovery_kernel = recovery_map[p.state]; KeyError: 2.
First I added recovery kernel as you suggested in #737, only for ErrorCode.Error:
def DeleteParticle(particle, fieldset, time):
    particle.delete()
pset.execute(..., recovery={ErrorCode.Error: DeleteParticle}))

Then I had a ThroughSurfaceError with  Through-surface sampling by particle at (-7.472253, -4.896896, 21.116211). Note that depth value is 21.11m which is obviously not at the surface.
Then I set recovery={ErrorCode.Error: DeleteParticle, ErrorCode.ErrorOutOfBounds: DeleteParticle}) and the same recovery kernel error appears.
This is always happening in the first time-step of my simulation.

we're working on the KezError bug right now - we keep you informed when it's solved.


eth_feeHistory not fully working in 0.1.0

Describe the bug
Note that the latest behavor has already been fixed in main via https://github.com/paradigmxyz/reth/pull/3288 but since I raised in on Slack gakonst asked that I log it here for completeness.
See the table below in repro steps for more details.
Steps to reproduce
Here's a table of RPCs and their results, w/ notes.

RPC
Result
notes

{"method":"eth_feeHistory","params":["0x1","latest",[50]],"id":"latest","jsonrpc":"2.0"}
{"jsonrpc":"2.0","error":{"code":-32602,"message":"Invalid block range"},"id":"latest"}
this should work but didn't in 0.1.0

{"method":"eth_feeHistory","params":["0x1","0x10d05cf",[50]],"id":"blocknumber","jsonrpc":"2.0"}
{"jsonrpc":"2.0","result":{"baseFeePerGas":["0x9e1e3d77c","0xb1dfd6718"],"gasUsedRatio":[0.9997843333333334],"oldestBlock":"0x10d05cf","reward":[["0x0"]]},"id":"blocknumber"}
works as expected in 0.1.0

{"method":"eth_feeHistory","params":["0x0","latest",[50]],"id":"latest-w/-zero","jsonrpc":"2.0"}
{"jsonrpc":"2.0","result":{"baseFeePerGas":[],"gasUsedRatio":[],"oldestBlock":"0x0","reward":null},"id":"latest-w/-zero"}
zero-length should either be an error or behave the same as 0x1?

Node logs
No response
Platform(s)
Linux (x86)
What version/commit are you on?
0.1.0
What database version are you on?
reth db version
Current database version: 1
Local database version: 1

If you've built Reth from source, provide the full command you used
No response
Code of Conduct

[X] I agree to follow the Code of Conduct

As discussed this is already fixed, thank you for reporting for completeness.


Create Jetpack Compose versions of the quickstarts

We're currently working on creating Jetpack Compose versions of each quickstarts. This public issue should serve as an Implementation Tracker for it.
We're moving all the calls to Firebase/Google Services from Kotlin Activities/Fragments to a ViewModel, where possible. The idea is that the current  Kotlin+XML quickstarts should share the same ViewModel as the Compose ones.
All the work is currently being done in the compose branch and will be merged into the main branch once all quickstarts are done.
If you'd like to contribute to this effort, feel free to comment bellow which quickstart you'd like to work on and I'll assign it to you.

Quickstart
Assignee(s)
PR(s)
Status ( ⬜  = not assigned, 🔄 = work in progress, ✅  = done)

admob
@LokiFrostGiant
#1452
✅

analytics

⬜

appdistribution

⬜

auth
@thatfiredev
#1459
🔄

config
@millandavid
#1450
✅

crash
@millandavid

🔄

database

⬜

dynamiclinks
@argzdev
#1477
✅

firestore
@Joel-K-Muraguri

🔄

functions
@AlbertoYabeda

🔄

inappmessaging

⬜

messaging

⬜

perf

⬜

storage
@LokiFrostGiant

🔄

I would like to work on crash please!

I would like to work on storage, please 😎

@millandavid @LokiFrostGiant Assigned both of you. Go for it :)

I would like to work on firestore, please.

@Joel-K-Muraguri Sure, assigned to you. Be sure to checkout the  compose branch

I would like to work on storage.

@Eff4real I'm afraid Storage is already taken. You may choose a quickstart that doesn't have any assignee.

Hi @thatfiredev, I would like to work on dynamiclinks, if that's okay.

@argzdev Sure thing, assigned it to you :)

Hello @thatfiredev, I would like to work on Functions.

@AlbertoYabeda sure, assigned to you

@argzdev go for it :)

Hi @thatfiredev, I would like to work on messaging, if that's okay?

@lehcar09 For sure! Assigned it to you

Hi @thatfiredev, I would like to work on appdistribution, if that's okay.

@b1no0019 @amueler for sure, assigned you both. Thanks!

Simple commenting to update that I am now resuming work on storage!

Hi @thatfiredev, I'd like to work on Analytics, please.

@lehcar09 assigned to you. I have yet to review your FCM PR, I'll find some time next week, but you can go ahead and get started with Analytics.

Hi @thatfiredev, can I work on database?

@b1no0019 For sure, go for it


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 883 ms

Resolved: Bofrost.be performance has improved in 88f34e5 after 8 minutes.


java.util.concurrent.TimeoutException: com.android.internal.os.BinderInternal$GcWatcher.finalize() timed out after 10 seconds

We got this Crash report:
ANDROID_VERSION=5.0.2
APP_VERSION_NAME=19.2
BRAND=motorola
PHONE_MODEL=XT1072
STACK_TRACE=java.util.concurrent.TimeoutException: com.android.internal.os.BinderInternal$GcWatcher.finalize() timed out after 10 seconds
        at java.lang.Object.wait(Native Method)
        at java.lang.Object.wait(Object.java:422)
        at java.lang.ref.ReferenceQueue.remove(ReferenceQueue.java:101)
        at java.lang.ref.ReferenceQueue.remove(ReferenceQueue.java:72)
        at java.lang.Daemons$FinalizerDaemon.run(Daemons.java:173)
        at java.lang.Thread.run(Thread.java:818)

As this discussion suggests http://stackoverflow.com/questions/24021609/how-to-handle-java-util-concurrent-timeoutexception-android-os-binderproxy-fin
" The issue cannot be avoided - you will encounter it if your app runs in the background. You can mitigate by taking a wakelock, and prevent the device from sleeping, but that is a different story altogether, and a new headache, and maybe another talk in another con. You can minimize the problem by reducing GC calls - making the scenario less likely. tips are in the slides." ...

Looks like this is Android system problem. And it's being fixed by Android core team.

Hi yvolk,
Is this issue fixed from Android core.
Because we are still seeing this issue on API23 onwards on emulators especially. Or can you please guide me what to do in-order to get rid off this issue ?
-Srinu

@srinukodi As link from my first post tells: the problem was solved in Android firmware.
I personally never encountered such a crash, so I cannot tell otherwise.


Read me and explanations

Hi,
I'm trying to put my ruby project on the app engine, but I'm struggling a lot with the configuration and don't know how to troubleshoot / debug the issue.
I actually created a Stackoverflow post with the explanation of the problem.
https://stackoverflow.com/questions/51233614/google-cloud-ruby-deployment-and-ruby-docker
Would it be possible to have a little run down on how to use the tools that you guys are providing in this repo?
I'm not an expert with docker (Yet!) and a little clarity on how to use the different tools would be very useful for me and I'm sure for other people in my situation.
Thank you very much for your awesome work!

I posted you an answer on StackOverflow. Feel free to reach out to me directly if you're still having problems.
I know the code in this repo is difficult to follow. It's the implementation behind the App Engine Ruby Runtime, and I don't expect most users will need to interact with it directly. You're right that the README in this repo could be a bit clearer on that; I'll see about updating it.

I'm closing this case, and leaving the stackoverflow ticket open for now. I'll try and play a bit with it, if I can't find a solution I'll reach out and if it's solved, then I'll just close that case then 👍
Thank you very much for your help, it's very useful and well described!


Add todo FormGroup class variable

This has already been fixed via https://github.com/driftyco/ionic-site/commit/95d0b4e2f99349fef821f10ca1a61d90662735d8


west build -b nrf52dk_nrf52832 samples/boards/nrf/clock_skew failed

when I tried command:west build -b nrf52dk_nrf52832 samples/boards/nrf/clock_skew it throws the problem:
-- west build: generating a build system
Loading Zephyr default modules (Zephyr base (cached)).
-- Application: /home/xp/zephyrproject/zephyr/samples/boards/nrf/clock_skew
-- Cache files will be written to: /home/xp/.cache/zephyr
-- Zephyr version: 3.0.99 (/home/xp/zephyrproject/zephyr)
-- Found west (found suitable version "0.12.0", minimum required is "0.7.1")
-- Board: nrf52dk_nrf52832
CMake Error at /home/xp/zephyrproject/zephyr/cmake/modules/verify-toolchain.cmake:79 (find_package):
Could not find a package configuration file provided by "Zephyr-sdk"
(requested version 0.13.1) with any of the following names:
Zephyr-sdkConfig.cmake
zephyr-sdk-config.cmake

Add the installation prefix of "Zephyr-sdk" to CMAKE_PREFIX_PATH or set
"Zephyr-sdk_DIR" to a directory containing one of the above files.  If
"Zephyr-sdk" provides a separate development package or SDK, be sure it has
been installed.
Call Stack (most recent call first):
/home/xp/zephyrproject/zephyr/cmake/modules/zephyr_default.cmake:121 (include)
/home/xp/zephyrproject/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:51 (include)
/home/xp/zephyrproject/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:81 (include_boilerplate)
CMakeLists.txt:4 (find_package)
-- Configuring incomplete, errors occurred!
but actually I have edited the ~/.bashrc file like
export ZEPHYR_TOOLCHAIN_VARIANT=zephyr
export ZEPHYR_INSTALL_SDK_DIR=/home/xp/zephyr-sdk-0.13.2
why would this happened?

Have you done west zephyr-export?

Have you done west zephyr-export?
Thanks for your answer.
I have done west zephyr-export.All right it's my fault，my virtual machine has no enough space so that the sdk I downed is not complete.After I larged the disk space and redownloaded the sdk,build command works well.
But now the new problem occurs, I'm trying to flash the board with command west flash but it met another problem :

-- west flash: rebuilding
ninja: no work to do.
-- west flash: using runner nrfjprog
FATAL ERROR: required program nrfjprog not found; install it or add its location to PATH

Do I need to install the nrfjprog to fix the problem？But I have found nrfjprog.py file in zephyrproject/zephyr/scripts/west_commands/runners/.

Have you done west zephyr-export?

1

Do I need to install the nrfjprog to fix the problem?

Yes. Please see https://docs.zephyrproject.org/latest/boards/arm/nrf52dk_nrf52832/doc/index.html#programming-and-debugging

Have you done west zephyr-export?

Thanks for your answer. I have done west zephyr-export.All right it's my fault，my virtual machine has no enough space so that the sdk I downed is not complete.After I larged the disk space and redownloaded the sdk,build command works well. But now the new problem occurs, I'm trying to flash the board with command west flash but it met another problem :
-- west flash: rebuilding
ninja: no work to do.
-- west flash: using runner nrfjprog
FATAL ERROR: required program nrfjprog not found; install it or add its location to PATH

Do I need to install the nrfjprog to fix the problem？But I have found nrfjprog.py file in zephyrproject/zephyr/scripts/west_commands/runners/.

how to fix the required program nrfjprog not found? I also have this issue.

how to fix the required program nrfjprog not found? I also have this issue.

https://docs.zephyrproject.org/latest/develop/flash_debug/nordic_segger.html


Add configurable crossbreeding stat inheritance

Hello,
Here is a small change that adds configurable stat inheritance for cross breeding. By default it is disabled. If enabled, the crossbred offspring will inherit the average stats of the parents. This is with ratio set to 1.0. With ratio set to 0.5 the offspring gets 50% of the parent average for each stat.

I will play around with it. I've got a lot of projects going atm. but the code looks good.


RSpec/ReceiveMessages is marked as a safe autocorrect, but is not

Aside from the fact that receive_messages and similar could be overridden (which in my mind means it should never be considered a safe cop), it is unsafe:
allow(Foo).to receive(:bar).and_return(1)
baz_double = instance_double(Baz)
allow(Foo).to receive(:cat).and_return(baz_double)

autocorrects to
allow(Foo).to receive_messages(bar: 1, cat: baz_double)
baz_double = instance_double(Baz)
# errors as the double is created after its use

I should be able to confidently run safe rubocop autocorrects without even checking them, and this doesn’t pass that test. Even with the change to reorder the lines better, this is still an inherently unsafe fix

I should be able to confidently run safe rubocop autocorrects without even checking them

THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND

I don’t have much to add to that.

I don't mean in a legal sense. I mean that is the attitude taken by the rest of rubocop. Other "safe" autocorrects are ones which have zero effect on the actual behaviour of the program and instead just visual. This affects the behaviour

Consecutive allow(Foo).to receive(:bar).and_return(baz) limiting offenses to only should eliminate cases of unintended violations and automatic modifications, so it may be a good to change it that way. WDYT?

I was thinking that for a bit, until I just ran into one of this form:
allow(object).to receive(:message1).and_return([{ some_complicated_config}])
allow(object).to receive(:message2).and_return(object.message1.first)

so... consecutive expectations and none of those expectations' return value makes a call to the receiving object? but what about if it calls some other method, which would then call the receiving object? I think it should just be marked as unsafe, with some of these examples in the docs for why it's unsafe.

I don’t mean in a legal way. I mean bugs and deficiencies take place, and “should confidently without checking” is an aspiration, but we almost never can make sure.
We do our best by running autocorrection on new/modified cops against our current project’s code and check if it works as expected.
But we don’t have the capacity to ensure it is so for e.g. doing this on all real-world-rspec repos.
in RuboCop, a cop is safe to auto-correct if it is “by design”, not “by implementation”. This is where bugs and design flaws manifest the difference between the expectation and reality.

Is the example with using the result of one call as the return value of another realistic?

I mean, I found it in an actual code base, and it caused my specs to fail until I figured out why.
As to if it’s well-written code… that could be up for debate.

#1687 marks the cop’s autocorrection as unsafe. Reviews are welcome.

This can be solved by autocorrected code to the last position of previous code, not first.

Let’s try to do this in a later PR.


Add documentation about integrations

Description

Add global usage documentation of event, sfml and user_action integrations.

Close #154
What type of PR is this? (check all applicable)

[ ] Refactor
[ ] Feature
[ ] Bug Fix
[ ] Optimization
[x] Documentation Update

Added/updated tests?

[ ] New tests written
[ ] Existing tests updated
[x] Tests are not required because this is a documentation update
[ ] I need help with writing tests

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

Project coverage is 83.21%. Comparing base (5eda2ad) to head (ea35f92).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #156   +/-   ##
=======================================
  Coverage   83.21%   83.21%
=======================================
  Files          63       63
  Lines        1090     1090
  Branches      345      345
=======================================
  Hits          907      907
  Misses        127      127
  Partials       56       56

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


'Take' operator should support non-zero axis values

The current implementation of Take operator is incomplete. As mentioned in the documentation, It supports slicing along only axis=0 and in ‘clip’ mode.
The operator is intended to support other negative and non-negative values for axis. THe current implementation of the operator needs to be modified to meet multiple axis values.
@piiswrong @szha @zheng-da @haojin2 @eric-haibin-lin

Supported in #11326, can close once that is merged.

@nswamy Could you please change the labels to "Feature Request"? I feel like that's a better description of this issue. Our team is tracking bugs so this kind of issue could be confusing for us.

@anirudhacharya #11326 has been merged, so this issue should be closed?


How are these roles intended to be installed?

Since using the ansible-galaxy method is not supported (even via direct from git) what is the recommend way for using these roles? Just copy and modify?

I'm struggling with the same issue. Could those roles be made available through galaxy?

I'm struggling with the same issue. Could those roles be made available through galaxy?

Yes, you're supposed to clone the repository down to your Ansible controller machine. Then make a copy of the inventory folder, and modify that hosts.ini to match your setup. Then run:
ansible-playbook -i inventory/{your inventory}/hosts.ini site.yml

Check these videos out:

https://www.youtube.com/watch?v=N4bfNefjBSw


Adding link to features section

[X] Have you followed the guidelines in our Contributing document?
[X] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Merged. Thanks for the improvement, Swapnil! I'll update the live web page with the next release of Linuxbrew.


(PDB-2099) Fix test-latest-version shadowing

This commit merges two tests called test-latest-version. One of the
tests was shadowing the other and upon merge it became clear that one of
the tests that was being shadowed was broken. This commit also fixes the
misnamed config key that was causing issues and test failures.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-master/221/
Test PASSed.


How can I use GPU in test_frcnn.py?

Cause it uses cpu

Can you comment that?

that's weird because keras should grab the GPU when the  device is available.
can you post the logs?

Thanks. That was related with PyCharm. It didn't find the needed libraries. Under the terminal it launches ok.


How to use this plugin?

Hi guys
how to use this plugin?

Hello,
I just wasted time looking for this :)
So for you or others who wants to load this plugin you just have to :

create a folder named dragresize in the CKEDITOR plugins directory
put the plugin.js file inside
configure CKEDITOR to load this plugin, for example in config.js : config.extraPlugins = 'dragresize';

PS : Thanks @sstur for this plugin, it's very incredible that the webkit issue has never be solved (https://bugs.webkit.org/show_bug.cgi?id=12250)


Capture profiling statistics

Almost closes issue #686 - we just need to decide on the parameters that we should pass to the scale_run script itself, and to capture the extra statistics that are listed in the issue itself.
ipysession files are no longer produced
This alleviates the issue of filesizes of the .pyisession outputs. They are no longer pushed to the profiling repository - instead we push the stats.json files (discussed below) and optionally the rendered HTML files of the session results.
The HTML outputs are significantly smaller than the raw .pyisession outputs (of the order 10/100s kBs rather than 100s/1000s MBs).
Profiling captures additional information on top of the profiling output
The profiling runs are now setup to output a .stats.json file upon completion, which captures information from the profiling session as well as additional information about the simulation itself. The output file is just a json file and can be parsed as such. Currently we are capturing:

Start time of the profiling session
Duration of the profiling session
CPU time used by the profiled program
Information about the final state of the population DataFrame from the simulation:

Number of rows and columns
Size in MBs
Number of times it was extended

Disk I/O stats:

Number of read/writes
Size (in MB) of read/writes
Time spent reading/writing from disk

The name of the HTML file produced, provided it was produced

There is scope to include additional statistics that are of interest, which can be done by adding additional fields to the dictionaries that are produced in the record_XXX_statistics functions.
The run_profiling.py script can also be passed the --html flag to also produce a HTML file containing the results of the profiling session (as rendered by pyinstrument) if we so choose.
Additionally, run_profiling.py can also be called with an --additional-stats flag, which can be used to pass shell or workflow variables to the program which will then also be included in the .stats.json output. For example,
python run_profiling.py --additional-stats foo="bar bar" "fave number"=5

will result in the following key/value pairs appearing in the output:
"foo": "bar bar",
"fave number": "5",

Sample output:
Run on commit 3f3507a23483fa5e373d9b518968aefb8c84cbff, manually passing the sha and trigger keys.
tox -vv -e profile -- --additional-stats \
          sha=3f3507a23483fa5e373d9b518968aefb8c84cbff \
          trigger="manual trigger"

producing the .stats.json file:
{
  "sha": "3f3507a23483fa5e373d9b518968aefb8c84cbff",
  "trigger": "manual test",
  "html_output": "manual_test_0_3f3507a23483fa5e373d9b518968aefb8c84cbff.html",
  "start_time":<PHONE_NUMBER>.2109582,
  "duration":<PHONE_NUMBER>394165,
  "cpu_time":<PHONE_NUMBER>4,
  "pop_df_rows": 51000,
  "pop_df_cols": 448,
  "pop_df_mem_mb": 107.226341,
  "pop_df_times_extended": 1,
  "disk_reads": 96,
  "disk_writes": 6097,
  "disk_read_MB": 5.726208,
  "disk_write_MB": 441.458688,
  "disk_read_s": 0.031,
  "disk_write_s": 2.343
}

I've updated branch to change default values of profiling run parameters to be for 5 years, 50k initial population in mode 2, and now have the arguments to scale_run saved to profiling results directory as a JSON file along with the actual simulation outputs (by default only log output, with logging level set to WARNING to keep minimal). I've also done a bit of refactoring to make the run_profiling script a bit more flexible for using at the command line by exposing some of the key simulation parameters (e.g. simulation period, initial population) and added back an option (disabled by default) to save raw .pyisession output for when running locally. From my perspective assuming tests pass this is now good to go in. @tamuri tagging you in case you want to review given I've a series of changes on top of Will's.


Occasional PhantomJS start-up issues on Travis

I'm using the phantomjs driver for my browser tests and am having some intermittent problems on Travis (I may switch some of the tests to simple as I don't always need JavaScript, but I'd like to get this stable first).
Occasionally PhantomJS hasn't started when it should have, and so Spiderling crashes out with an error:
Configuration read from /home/travis/build/halfer/awooga-app/phpunit.xml

...........................................................E..... 65 / 71 ( 91%)

......

Time: 43.07 seconds, Memory: 27.75Mb

There was 1 error:

1) Awooga\Testing\Browser\BrowseTest::testBrowse

Openbuildings\Spiderling\Exception_Driver: Curl "url" throws exception couldn't connect to host

/home/travis/build/halfer/awooga-app/vendor/openbuildings/spiderling/src/Openbuildings/Spiderling/Driver/Phantomjs/Connection.php:263

/home/travis/build/halfer/awooga-app/vendor/openbuildings/spiderling/src/Openbuildings/Spiderling/Driver/Phantomjs/Connection.php:209

/home/travis/build/halfer/awooga-app/vendor/openbuildings/spiderling/src/Openbuildings/Spiderling/Driver/Phantomjs.php:273

/home/travis/build/halfer/awooga-app/vendor/openbuildings/spiderling/src/Openbuildings/Spiderling/Page.php:33

/home/travis/build/halfer/awooga-app/vendor/openbuildings/phpunit-spiderling/src/Openbuildings/PHPUnitSpiderling/Testcase/Spiderling.php:222

/home/travis/build/halfer/awooga-app/test/browser/tests/BrowseTest.php:18

/home/travis/build/halfer/awooga-app/test/browser/tests/BrowseTest.php:18

FAILURES!

Tests: 71, Assertions: 204, Errors: 1.

In the example there is one 'E' failure, but Spiderling/PhantomJS has worked fine for another ten or so. And if I restart the build in Travis, it then works fine!
I've added logging code to see if I can see anything about why PhantomJS might fail to start up, thus:
/**
 * Let's set add some logging here, to see why PhantomJS is flaky on Travis
 */
public function driver_phantomjs()
{
	// We can supply a log location here (or omit to use /dev/null)
	$logFile = '/tmp/phantom-awooga.log';
	$connection = new \Openbuildings\Spiderling\Driver_Phantomjs_Connection();
	$connection->start(null, $logFile);

	$driver = new \Openbuildings\Spiderling\Driver_Phantomjs();
	$driver->connection($connection);

	return $driver;
}

However this doesn't capture anything useful. In the same run, it captures normal PhantomJS traffic, e.g.:
$ cat /tmp/phantom-awooga.log

Starting server on port 4818

Recieved command POST /url http://localhost:8090/browse?search=phppot.com

Executing command POST /url http://localhost:8090/browse?search=phppot.com

http://localhost:8090/browse?search=phppot.com success

Recieved command POST /elements .//descendant-or-self::*[@id = 'addressSearch' and (@value = 'phppot.com')]

Executing command POST /elements .//descendant-or-self::*[@id = 'addressSearch' and (@value = 'phppot.com')]

Recieved command POST /elements .//descendant-or-self::h3[contains(string(.), '11 results')]

Executing command POST /elements .//descendant-or-self::h3[contains(string(.), '11 results')]

Recieved command POST /elements .//descendant-or-self::*[@id = 'paginator']

Executing command POST /elements .//descendant-or-self::*[@id = 'paginator']

Recieved command DELETE /cookies

Executing command DELETE /cookies

    ....

I might try adding a delay in setUp() in case the PhantomJS binary is still making connections. Also, I am relying on the copy provided in the default Travis build, but locally I use the one from jonnyw/php-phantomjs. I might give that a go, in case it is a later version. It never fails on my local machine (making this much harder to debug!).
I don't know if this is a support issue or a bug - if it is the former I'll happily move it to Stack Overflow. However I wonder if it is rather too niche to get much attention there! (Would be nice to see more eyeballs on this project - I've found Selenium slow and flaky in the past, so it is nice to see alternatives popping up).

This has been very reliable for the past few weeks, so I wonder whether a new version of PhantomJS is responsible. I'll revisit in a month, and if I don't get any more failures I'll put it down to a fixed bug.

@halfer Hi there! Have you had a chance of confirming this is now working as expected? We didn't have much issues lately.

I've not touched this for a few months. I think it did settle down, but I don't recall - I'll give it a try in due course, and will let you know.

I've run a few more tests, and I think this has settled down. I don't know if it is due to a new version of PhantomJS or extra wait code I added in back in December/January.
I'll close for now, it can always be reopened if the issue resurfaces. Thanks for chasing up!


Wrong Value Cell

I will take value in cell, when I take value in cell. the value that I get is not appropriate. Please help.

after xlsx read
"A887":{"t":"s","v":"ARC0008.0146"},"B887":{"t":"s","v":"F004"},"C887":{"t":"n","v":37623.00013888889,"w":"1/2/03"},"D887":{"t":"s","v":"ZARC"},"E887":{"t":"s","v":"ARC0008"},"F887":{"t":"s","v":"LE"}

That looks super weird, can you share the file?  If you don't want to make it public, feel free to email<EMAIL_ADDRESS>

Same as #2196 :
> new Date("SHOWER SCREEN TOWER 1,2,3")
< Thu Jan 02 2003 00:00:00 GMT-0500 (Eastern Standard Time)


Feature Request: jams.convert(a, 'namespace'): from 'note_midi' to 'pitch_contour'

I feel like this can be a unlossy mapping that doesn't need any arguments.
Further more, I think it's legal right now to convert from 'pitch_midi' to 'pitch_contour'
and 'pitch_midi' and 'note_midi' is basically two versions of the same.
It would help my work flow in evaluating these annotations...

I think we can call this one resolved.


Error when deleting a user as an org admin

Context
As an org admin when I delete a user from the dashboard UI and try to create another with the same username or email, I get an error saying the username or email is taken
Steps to Reproduce

Login as an org admin
Create a user from Your Organization > Users > Create user
Delete that user
Create another user with the same username or password

Current Result
You get an error
Expected result
It should work
Additional info
I've checked that the user is removed from cartodb but not from central.
This maybe related to the recent upgrade to Rails 4.
Additional traces provided:

https://rollbar.com/carto/CartoDB-Central/items/4006/
https://rollbar.com/carto/CartoDB-Central/items/13209/

Summoning @ramiroaznar as RT 🤠 and @alonsogarciapablo as dashboard owner for triaging

@alrocar good catch! next time remember to use @CartoDB/rt-managers

cc @javitonino I think this is on your roof

We've been taking a look at the traces and it seems an error that has been there for some weeks, so maybe the traces are unrelated.
We are going to deploy to production tomorrow morning the branch were I found that error. Let's wait for it, and I'll test it again this time in production to see if it still happens.
I'll update this issue accordingly.

@jgoizueta why did you move this one to Infra?

Hey, I've checked this with current production and couldn't reproduce it.
@alrocar I followed your steps and I worked well, no error. Do you think this is OK now?

It should be OK then. Closing.