upgrade prototype test CI from 3.7 to 3.10

@ejguan @NivekT starting for Python >= 3.8 we are seeing a lot of I/O warnings that are turned into errors by our test suite. I was under the impression that we don't need to handle file closing. Has this changed?

@pmeier Sorry for jumping on a draft PR. To clarify, is this PR an investigation or you eventually propose to merge? Typically we want to pin all tests on the earliest supported version, so if indeed the problems disappear we probably don't want to merge this but rather investigate and fix the underlying issue. Probably that's what you have in mind as well but just confirming. Thanks!

For now, this is just an investigation. @NicolasHug saw some failures locally that neither I nor CI could reproduce. Turns out, the errors (or better warnings turned into errors) only pop up for Python >= 3.8. This PR confirms that this is indeed the case.
It should not be merged in the current state, but maybe will be the basis for further investigations or fixes. I'll either close it or mark it as "ready to be reviewed".

Checking it

I will spend more time on this issue.

I may find the culprit.
For example of cub200, when we construct data pipeline using _datapipe(resources), we load a few data via
https://github.com/pytorch/vision/blob/467b841ac2650c962952bbee5a542edb835c98bc/torchvision/prototype/datasets/_builtin/cub200.py#L188-L190
But, before creating the dict image_files_map, we have a Demux to hold file handlers. Before the iteration of data-loading pipeline, the Demux has been iterated and the buffer will be filled with unclosed file handlers. Then, for these file handlers, we can never close them as the reference count is always larger than 1.
I only narrowed down the root cause of such error. I haven't got enough time to verify whether or not we change image_files_map to IterToMapConverter to support lazy loading would solve the issue
https://github.com/pytorch/data/blob/fd942eec986db373f76f521528570bfef2f1d22f/torchdata/datapipes/map/util/utils.py#L20
cc: @NivekT

But, I couldn't find the code to extract files from the tar file. Could you please provide the code pointer?

To complement Kevins comment in https://github.com/pytorch/vision/pull/5801#issuecomment-1096915679: the loading happens in
https://github.com/pytorch/vision/blob/467b841ac2650c962952bbee5a542edb835c98bc/torchvision/prototype/datasets/utils/_resource.py#L62-L72

Folder are loaded with FileOpener(FileLister(...))
Files are loaded with FileOpener(IterableWrapper(...))
Archives are additionally loaded with the respective loader datapipe to have the same behavior for folders and archives

But, how about the custom file? What kind of files are you referring to?

We have a failing test for
https://github.com/pytorch/vision/blob/467b841ac2650c962952bbee5a542edb835c98bc/torchvision/prototype/datasets/_builtin/semeion.py#L29
and that uses a .data file (csv in disguise) file and features no Demultiplexer. Upon closer inspection, it seems like the warning is raised at the wrong time:
failed on setup with "pytest.PytestUnraisableExceptionWarning: Exception ignored in: <_io.FileIO [closed]>
[...]
Traceback (most recent call last):
  File "/home/circleci/.pyenv/versions/3.10.4/lib/python3.10/unittest/mock.py", line 2153, in __init__
    self.name = name
ResourceWarning: unclosed file <_io.FileIO name='/tmp/pytest-of-circleci/pytest-0/test_num_samples_sbd_train_nov0/sbd/benchmark.tgz' mode='rb' closefd=True>

So actually SBD is the culprit here. I'll go through all of them again to find the actual datasets that are causing this.

Rechecking our and torchdata logs, it turns out these datasets are the culprits:

clevr
cub200
dtd
food101
gtsrb
sbd
voc

This eliminates point 1 from https://github.com/pytorch/vision/pull/5801#issuecomment-1096825604. My guess is that the warning is emitted during garbage collection and thus happens at a non-deterministic time.

As for the other similarities: all culprits feature a Demultiplexer, but cub200 is the only dataset that starts the iteration during the construction of the datapipe. Another pointer is that we have other datasets that feature a Demultiplexer

coco
imagenet
emnist

but don't seem to emit this warning.

but cub200 is the only dataset that starts the iteration during the construction of the datapipe

Yeah. It's the reason that only cub200 fails for the case of test_smoke.

Adding this patch seems solve all the CI issue:
+    def __del__(self):
+        for q in self.child_buffers:
+            q.clear()
+

@NivekT Do you mind taking a lead to see why StreamWrapper does not close the inner file handler automatically? And how can we add a test to prevent such scenario?

I had a look and tried to reproduce the error locally with torchvision. I played around with cub200 and demux.
This is the simplest test case that I have and it seems to work fine.
dp = IterableWrapper([f'{i}.tar.gz' for i in [1, 2, 3]])
dp = FileOpener(dp, mode='rb').load_from_tar()
dp1, dp2 = dp.demux(2, lambda _: 0)

When the demux DataPipes is no longer needed, the file handles inside its buffer get closed correctly by StreamWrapper.
cub200 seems to be working locally for me as well. Let me know what code snippet you run to have a ResourceWarning.
I have been trying to figure out the following:

If _DemultiplexerIterDataPipe not cleaning up properly, what is the specific situation to cause the buffer to not clean up properly?

Are there references to those DataPipes inside the buffer elsewhere, such that they don't get deleted?

Is StreamWrapper's __del__ logic causing the issue here?

I am leaning towards the first thing is the issue here, therefore, I think @ejguan's suggestion to specifically clear out the buffer is reasonable and we should do that unless we can figure out what specifically cause the buffer not to clear.

You can reproduce the Error from pytest test/test_prototype_builtin_datasets.py -v -k <pattern of test> @NivekT

I'll try to compile a minimal reproduction outside of our test environment.

I was able to pinpoint it to cases where the buffer of the Demultiplexer was not empty. For the datasets in https://github.com/pytorch/vision/pull/5801#issuecomment-1097630775 the buffer is empty and thus we are not seeing a failure.

After devoting quite a significant time, I'm fairly confident it is not worth sinking any more time into this. My hunch is that this is pytest issue, but I'm not sure.
Minimal steps to reproduce:

Use Python >= 3.8
Test a dataset that internally uses a Demultiplexer
Have any data stored in the _Demultiplexer().child_buffers

Even with that, sometimes the error shows up, but sometimes it doesn't. If someone else wants to debug further:

If you try to reproduce outside of the test suite, make sure ResourceWarning's are visible. They are ignored by default by Python. pytest enables them.
It seems that the error is pointing the original archive for being not closed rather the file handles extracted from it.

After some offline discussion with @NicolasHug, we think it is best to just ignore the warnings. I tested on

Python 3.8
Python 3.9
Python 3.10

and CI was green.

After https://github.com/pytorch/pytorch/pull/76345 is landed into latest PyTorch nightly, the Error caused by ResourceWarning is eliminated in the TorchVision domain test for TorchData.
But, an Error pops up due to the lambda function from https://github.com/pytorch/vision/blob/1ac6e8b91b980b052324f77828a5ef4a6715dd66/test/test_prototype_builtin_datasets.py#L138
See: https://github.com/pytorch/data/runs/6195902638?check_suite_focus=true

But, an Error pops up due to the lambda function from

I can't reproduce that in our own CI: https://app.circleci.com/pipelines/github/pytorch/vision/17117/workflows/612d3491-b1ec-4b2f-9dee-860a078c65c3/jobs/1387558

The failures in torchdata CI happen for Python 3.[89] on macOS and Python 3.[789] on Windows. The failure is
AttributeError: Can't pickle local object 'TestCommon.test_data_loader.<locals>.<lambda>'

which is correct, but this leads to more questions:

Why aren't we seeing this failure on Linux and even more peculiar not macOS when using Python 3.7?
Why aren't we getting a warning about the lambda usage?

Why aren't we seeing this failure on Linux and even more peculiar not macOS when using Python 3.7?

This is because fork is the default mp method for Linux and macOS (3.7). If you change starting method to spawn, the lambda function won't work.

Why aren't we getting a warning about the lambda usage?

collate_fn is an argument for DataLoader or for DataPipe. So, we didn't get chance to update it. We can add a warning to DataLoader when collate_fn is lambda and num_workers>0.

Superseded by #5914.


Is compression working when node server is running on a container?

We have the package working fine on local development machines (Mac Catalina), but it doesn't seem to compress anything when we deploy on Kubernetes in a container.  The base container we are using is the standard node 10.16.3 from Docker Hub, but that's a rather minimal Linux distro and I'm wondering if there are some OS dependencies that we need to install.

I haven't had issues with this module inside kubernetes, personally, and there are no dependencies to install outside the module.

Closing as no response.


Use HTTPS for eslint.org links

eslint.org now supports HTTPS:

eslint.org  is now served over HTTPS! Please let us know if you encounter any issues.

Published in v1.0.17. 🎉


Should a wallet's list of Supported Chains allow Testnet or Upcoming chains?

Some wallets may want to include testnets in their list of supported chains.

There seems to be some value in including this in some way, but there might be some confusion when mixing them up with mainnet chains.
We could consider a way to indicate that those chains are testnets. Note that not all testnets are registered as "[mainnet chain net] + 'testnet" (e.g., 'osmosistestnet'), so the name might not be sufficient. We could use a separate array (e.g., 'supported_testnet_chains')?
We could also just leave it up to the chain registry to clarify whether a chain is a mainnet or testnet.

Also, some submissions may want to submit PRs listing chains they will soon be adding to save on the time.

My initial opinion on this is the not allow 'soon' info, but only current info.

Want to confirm opinion that we should require that all chains be registered to the chain registry.

@pyramation , thoughts?

Some wallets may want to include testnets in their list of supported chains.

I can't see why they would support testnets officially? I'd suggest against it. But this is just my initial thought! Maybe others would disagree.

Want to confirm opinion that we should require that all chains be registered to the chain registry.

Definitely and names should map to an existing chain_name

IMHO, the inclusion of testnets has more to do with technical capability than it does with public perceived of official support. So I'm thinking when a dApp offers interaction with testnets, that should still be able to query for compatible wallets.
With the recent shift at the chain registry to suffix testnet chain names with 'testnet', the potential confusion brought up in point (1) should no longer be a worry.


[#94078878] Not able to save features and benefits with only one feature or one benefit.

https://www.pivotaltracker.com/story/show/94078878
This also fixes how the API client response is being used. The client now returns the entire response object. This feels a bit clunky but it is how the API is currently returning. I'm going to look at returning a response object that gives access to the metadata and links while giving access to the data in a more natural way.

Closing while I address @quis's comment.


Allow fast way of setting xlabel/ylabel location (and yticks for 2D)

For example for 2D spectra, all of this could be bundled into one function.
ax.yaxis.tick_right()
ax.yaxis.label.set_rotation(0)
ax.yaxis.label.set_horizontalalignment("left")
ax.yaxis.label.set_verticalalignment("top")
ax.yaxis.set_label_coords(1.02, 1)

That would give something like this.

Done to a certain extent (good enough for the example in the README), but needs to be fleshed out and documented.


Add a collection of non-physics based controls

At the moment, the 3D controls are all physics based (i.e. they use joints)
These provide a great realism but can also be quite intensive (if many are in the scene) and sometimes may be overkill.
An alternative set of controls could be built that don't use joints and just provide an approximation of the control mechanisms.
Ideal controls are:

Button
Slider
Lever
Drawer
Door (maybe?)

https://github.com/thestonefox/VRTK/issues/1409


Submenu in a dropdown truncated by the viewport

Reproduction link
https://stackblitz.com/edit/ng-zorro-antd-start-d4izon
Steps to reproduce

make a dropdown with submenus
positionnate the dropdown near the viewport
open the dropdown and submenu

What is expected?
The submenu is repositioned to avoid collision with the viewport.
What is actually happening?
The submenu collides with the viewport, in the example, only 2 elements are visible.

Environment
Info

ng-zorro-antd
8.5.2

Browser
Chrome Version 83.0.4100.0

Has fixed it in 9.0


.race question

Isn't the winner supposed to get all betted flowers? When someone wins a race, they just get their flowers back, or don't get anything If they did not bet, even though there are other players that bets flowers. ??

As far as I know that isn't in the use case of .race (if anything, you should use .rafflecur for that.)
This can be a FeatHub ticket too.

People can bet different amounts so it wouldn't really be right if someone
bet two on a pot of a couple hundred and got it all.
On Tue, Nov 28, 2017, 11:04 AM Midoriin<EMAIL_ADDRESS>wrote:

Isn't the winner supposed to get all betted flowers? When someone wins a
race, they just get their flowers back. ??
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Kwoth/NadekoBot/issues/1884, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABGad8sVuqXFQWVrYqq0EfreggbLiMumks5s7C8cgaJpZM4Qte44
.

.h .jr

Thanks kwoth 👍


Can't download encrypted files larger than 1 MB from cloud storage

I'm using upload_from_file with encryption_key to store files inside a Google Cloud Storage bucket. This works very well.
Now I'm trying to download these files again. Using download_to_file with the same encryption_key. This crashes with the following stack trace:
Traceback (most recent call last):
  File "restore.py", line 40, in <module>
    main()
  File "restore.py", line 37, in main
    download_from_storage(args.file, args.bucket, args.credentials, key)
  File "restore.py", line 17, in download_from_storage
    blob.download_to_file(my_file, encryption_key=encryption_key)
  File "/Users/daniel/Projects/python/gcs-backup/env/lib/python2.7/site-packages/google/cloud/storage/blob.py", line 354, in download_to_file
    download.initialize_download(request, client._connection.http)
  File "/Users/daniel/Projects/python/gcs-backup/env/lib/python2.7/site-packages/google/cloud/streaming/transfer.py", line 390, in initialize_download
    self.stream_file(use_chunks=True)
  File "/Users/daniel/Projects/python/gcs-backup/env/lib/python2.7/site-packages/google/cloud/streaming/transfer.py", line 614, in stream_file
    response = self._process_response(response)
  File "/Users/daniel/Projects/python/gcs-backup/env/lib/python2.7/site-packages/google/cloud/streaming/transfer.py", line 528, in _process_response
    raise TransferRetryError(response.content)
google.cloud.streaming.exceptions.TransferRetryError: The target object is encrypted by a customer-supplied encryption key.

I'm using google-cloud-storage==0.20.0.
This only occurs when downloading files larger than 1mb. Any file less than 1mb is downloaded and decrypted as expected. A file larger than 1mb might be downloaded in chunks and only the first 1mb chunk is saved to disk.
Steps to reproduce

Upload a file larger than 1mb using a custom encryption key
Try to download this file using the same encryption key
Find a 1mb chunk of your file at the destination

demo code
#!/usr/bin/env python

from tempfile import NamedTemporaryFile
import os

from google.cloud import storage
from google.cloud.storage import Blob

CREDENTIALS = './credentials.json'
BUCKET_NAME = '< SET BUCKET >'
ENCRYPTION_KEY = 'v3CtoFyEJmj52RGsSqze7C8fD6mzgpnd'
FILE_NAME = 'enc-test-file'

client = storage.Client.from_service_account_json(CREDENTIALS)
bucket = client.get_bucket(BUCKET_NAME)

def upload_to_storage(filename):
    blob = Blob(FILE_NAME, bucket)

    with open(filename, 'rb') as my_file:
        blob.upload_from_file(my_file, encryption_key=ENCRYPTION_KEY)

def download_from_storage(filename):
    blob = Blob(FILE_NAME, bucket)

    with open(filename, 'wb') as my_file:
        blob.download_to_file(my_file, encryption_key=ENCRYPTION_KEY)

def main():
    size = 2097152  # 2mb file

    f = NamedTemporaryFile(delete=False)
    f.write("\0" * size)
    f.close()

    print('Uploading {} to Google Cloud Storage...'.format(f.name))
    upload_to_storage(f.name)

    print('Downloading {} from Google Cloud Storage...'.format(f.name))
    download_from_storage(f.name)

    os.remove(f.name)

if __name__ == '__main__':
    main()

This might be because the encryption headers aren't added to the request for the next chunk.

@dbanck thanks for reporting!
@dbanck, I think that the encryption key doesn't get passed to download_to_file() but rather is passed to the Blob constructor.
See: https://github.com/GoogleCloudPlatform/google-cloud-python/blob/491519dc54cf2204703374ff374cabbd4cd14476/storage/google/cloud/storage/blob.py#L310-L311

This might be true for the current master, but the docs have a different example: https://googlecloudplatform.github.io/google-cloud-python/stable/storage-blobs.html#google.cloud.storage.blob.Blob.download_to_file
The files inside my local site-packages are looking like this: https://github.com/GoogleCloudPlatform/google-cloud-python/blob/umbrella-0.20.0/storage/google/cloud/storage/blob.py#L287

@dbanck, thanks! I forgot that it changed since the last release.

This might be because the encryption headers aren't added to the request for the next chunk.
I'll take a look at this and see if I can track down the issue.

@dbanck sorry for the slow update on this. I've been playing with it and I thought it was completely fixed on master but it appears that it may not be. I'm confirming that and then I'll try and get a fix in.

Don't worry, take your time. Thanks for the feedback and looking into it :)


registerCard method does not comply with Registration Flow diagramm

Hello,
following the registration flow it seams as this js kit does not follow the steps as it should.
Mainly it is taking over the steps 7 as it should, but also 8 and 9.
Currently the _tokenizeToken is the actual function (step 7) that should be used (only) and steps 8 and 9 should be done on the server.
While we can do that by just calling the function, it should be rewritten in a way, that it is not marked as private and also so that it foster the flow.

Hi @theluk,
I agree the workflow isn't ideal, we are reworking these assets over the next quarter
I'll see if we can fix something here.


[GSSOC'21] Heap Class

I am Manas Tole GSSOC 2021 participant.
I want to contribute in Heap data structure.
I will code Heap Class program in Java programming language including :
1)Max-heap
2)Min-heap
Please assign this task to me :)

I would like to contribute to this. Please assign this to me

Hello,
I am Vatsala, GSSoC 2021 participant. I want to contribute the code for the implementation of heap-sort using C++.
I will make sure the code is clean and also I will write comments to make it more understandable.
Please assign this task to me

@Upasana1216 language ?


Did you try you impl BoxConv on Unet(Resnet encoder)?

subj
How to set max height, max width?
Single-process box gen takes too long.

I didn't, but ENet and ERFNet in our paper are very very similar. They are all typical encoder-decoders of residual blocks with max-poolings and upconvolutions. Anyway, I'll soon upload example model definitions I used in the experiments.
Set max_input_h and max_input_w to maximum height/width of that module's input tensors.
What is box gen? What exactly do you run?

320х320 WxH, box gen method is def reset_parameters(self) box_convolution_module.py

Right, it's currently not very fast, I will speed it up. Still, I don't expect that you will run reset_parameters() very often. Maybe just once before training.

It's work ok on small HxW with one layer. Pls try add it on UNET-like architecture. Thx for repo it's nice!

Here's an example of application: ENet. As far as I know, it's UNet-like, except skip connections.

Thx a lot. I'll try it.


Stone age theme collapse icon missplaces

as you can see collapse icon move to the code

I missed this report when i was on vacation, so sorry!

@Jacse Please upload the new version.

Thank you :D i fixed it myself next day after I posted it :)
but if i donwload it on other pc it would help me :) and others too
have a nice day! and thank you for your work!
On 20 July 2015 at 23:59, Eric J<EMAIL_ADDRESS>wrote:

@Jacse https://github.com/Jacse Please upload the new version.
—
Reply to this email directly or view it on GitHub
https://github.com/Jacse/themes-for-brackets/issues/128#issuecomment-123060318
.

--
Chceme lámať okovy, chceme rozbiť kamene nášho mesta,
áno, to sme my, chceme otvoriť brány Kráľa Krista.
Chceme vidieť národy, ako na každom rohu chvália Pána,
tak to sme my, generácia, ktorá práve vstáva.
eSPé -cd On - Bláznovstvo

Currently on vacation as well @wormeyman, will get to a computer and upload asap

Uploaded :+1:


LinkedCellEditor manual input does not get set

When you enter a value yourself via keyboard, the LinkedCellEditor does not send its value to the Server

Enter key should send the value and not every key press...
There's an option for such cases: every Editor class has the option: savingImmediate -> means that every key press should send the value


Backend working in production (endpoints tested)

Backend Functionality in Production Confirmed
The backend is now fully functional in the production environment. All endpoints have been thoroughly tested, and their responses align with the expected behavior.
Feel free to review.

We delete this PR due to an error related to the destination branch.


[maintenance] December link fixes

This PR addresses broken links found using the Screaming Frog SEO Spider.

This change is

@zacharysarah This should cover all the broken links I found in our current docset. I've ignored any broken links in previous version branches.
P.S. Make sure you update the following lines in /_redirects to refect v1.9:
/docs/api-reference/v1/definitions/     /docs/api-reference/v1.8/ 301
/docs/api-reference/v1/operations/     /docs/api-reference/v1.8/ 301


Frontend redirects are weird

Few issues:

Going to nonexisting page will keep app reloading instead of showing 404
We fixed going to existing page by adding custom error redirect on our cloudfront distribution

Depends on https://github.com/bugout-dev/moonstream/issues/182


Custom port + SSL install seems broken in redirection and LetsEncrypt request

I just tested enabling SSL via LetsEncrypt and enabling custom port on my instance. It appears that when enabling both options there are two problems:

Normally, nginx is configured to redirect a http:// to https:// when SSL is enabled. It still does that but it redirects to the same http port. So for example http://michiel.intercity.io:8880 is redirected to https://michiel.intercity.io:8880 but that should be to port 8843.
The SSL certificate file under /shared/ssl/michiel.intercity.io.cer seems empty. Making nginx trigger the following error:

root@michiel:/shared/ssl# nginx -t
nginx: [emerg] PEM_read_bio_X509_AUX("/shared/ssl/michiel.intercity.io.cer") failed (SSL: error:0906D06C:PEM routines:PEM_read_bio:no start line:Expecting: TRUSTED CERTIFICATE)
nginx: configuration file /etc/nginx/nginx.conf test failed

Perhaps LetsEncrypt needs the server to be available on :80 or we need to set a custom port number when requesting the cert at:8880 or something?

@michiels The reason it is empty, is because the cert was not issues yet. Also, how many times did you try with the michiel.intercity.io domain in the past week?

@jvanbaarsen Ok, I tried 4 times or so. Shall I try and reproduce with a new domain name?

From: https://letsencrypt.org/docs/rate-limits/

We also have a Duplicate Certificate limit of 5 certificates per week. A certificate is considered a duplicate of an earlier certificate if they contain the exact same set of hostnames, ignoring capitalization and ordering of hostnames. For instance, if you requested a certificate for the names [www.example.com, example.com], you could request four more certificates for [www.example.com, example.com] during the week. If you changed the set of names by adding [blog.example.com], you would be able to request additional certificates.

And yeah, their rate limiting is awful

@jvanbaarsen Great, thanks for the pointer. I'll try and reproduce with a new domain name. I think point 1: the redirection issue is still applicable tho.

@michiels You can use a different subdomain :-) Also, is the domain reachable for Let's Encrypt? I'll look into the first point

@jvanbaarsen in the first test it was, because on the subsequent installs (without custom port setting) everything worked perfect. I'll now test with michiel2.intercity.io should also be reachable for LetsEncrypt in the DNS.
If it succeeds for me, I think the user that reported this issue initially, hit the rate limiting by LetsEncrypt too. For a future improvement: would it be an option to first see if we can request the cert via LetsEncrypt before we set up nginx to use it?

@michiels There is no real way of checking that :( Every check will also count as a request.

@jvanbaarsen Okay, but how are you receiving/putting the certificate in the file at the moment? That can be the final verification right?

We tell acme.sh where it needs to store the SSL cert, and acme.sh does all the magic under the good (https://github.com/Neilpang/acme.sh)

@jvanbaarsen ok, but then we could periodically poll for the contents of the expected certificate file, right?

@michiels Where do you want to poll this? We have nothing running on the system to check this?

I think we have concluded that custom ports + SSL are currently not supported and this bug should be turned into some kind of enhancement/feature request that takes a bit more work than just fixing a bug.

@michiels Yes, sounds about right!

@michiels I have been playing around with Docker / Let's encrypt a little, and came to the following idea. Let me know what you think:
We install Nginx on the host machine, rip out the nginx config out of the Intercity-docker container. That way we can easily manage restarts of the SSL etc on the host machine.
We create custom Let's Encrypt docker image (Already 70% done), and have that take care of the cert creation.
The benefit of having NGINX on the host machine, is that we can add a nginx.conf file to tell nginx to redirect traffic to http://domain.com/.well-known/acme/uniq-code (The path let's encrypt checks). So we can have cert issuing without down time.

@michiels See: github.com/intercity/letsencrypt

@jvanbaarsen I think it sounds sensible. How will you combine the nginx on the host machine with what Dokku installs as host nginx? Is that the same thing? Eventually you want Dokku to manage the nginx after it being installed, or at least not clash with the custom letsencrypt setup that you will make. If they can live alongside :+1:

@michiels I just create a custom nginx config file (The same as Dokku does) for Intercity. We might need to build some plumbing around it to make it all click correctly. But I think this is the way to go :)

@jvanbaarsen great! If it's just a matter of adding a sites-available or a new file in conf.d and Dokku also does that they can perfectly run alongside! I also agree that having a main nginx router on the host makes lots of these things handy dandy

I second this, seeing
nginx: [emerg] PEM_read_bio_X509_AUX("/shared/ssl/[REDACTED].cer") failed (SSL: error:0906D06C:PEM routines:PEM_read_bio:no start line:Expecting: TRUSTED CERTIFICATE)

when inspecting the container.

There's a dokku plugin nowadays for Letsencrypt that works, with a few caveats. The most important issue is that you need to have your app deployed already before requesting a certificate.
I don't think it'd be all that difficult to add support for LE this way.

@ariejan I wonder if there is a way we can check if the app is deployed or not. Because there is a very low number throttle on the amount of request you can do to LE (At least that used to be the case).

There is, using ps:report <app>. Also, checking if DNS is working correctly for each domain should be possible. When all checks are green, you can push the 'Letsencrypt' button.
root@ubuntu-8gb-nbg1-1 ~ # dokku ps:report wordcraft-io
=====> wordcraft-io process information
       Processes:           0
       Deployed:            false
       Running:             false
       Restore:             true
       Restart policy:      on-failure:10

Oh nice! We also need to have some check in place to see if the domain is reachable, right?
So the following checks need to pass:

Custom domain is added
ps:report returns all green
domain is actually reachable

If those pass, LE can be enabled for that specific app.

A custom domain is not a requirement IMHO. You could use the (ugly) generated subdomain vhost.

@ariejan Are you sure? I know in the past that didn't work (for reasons that I don't know anymore). I have the feeling a lot has changes with LE since the last time we tried this, so it might indeed be good to give the whole LE plugin another go :)


characterbody3d doesnt interact with rigidbodies

Godot version
4.0 alpha5
System information
linux
Issue description
characterbody3d doesnt interact with rigid bodies
it treats them like kinematic bodies
Steps to reproduce
try walking with a characterbody3d up to a dynamic rigidbody and it wont move
Minimal reproduction project
No response

@djmaesen Please upload a minimal reproduction project to make this easier to troubleshoot.

This seems like the expected result. Kinematic bodies are not supposed to influence any physic driven object unless you deliberately implement it. This predictability is the key element in kinematic bodies.
Godot 4.0 include the possibility to automatically apply the velocity of a kinematic or physic body on the character as is a really common use case for moving platforms and kinda easy to implement.
On the other hand, a lot of things can happen inside a kinematic body depending on your implementation, so is better to make this kind of things in the parts on the code you want to.

it worked in godot 3
my player interacted with rigidbodies. when calling move_and_slide

it worked in godot 3 my player interacted with rigidbodies. when calling move_and_slide

Ok, i understand you know.  you are talking about the infinite inertia property, which is not in godot 4.0.

IIRC you can switch it on, it's just not on by default unlike 3.x

IIRC you can switch it on, it's just not on by default unlike 3.x

ok , how?

IIRC you can switch it on, it's just not on by default unlike 3.x

I may be wrong, but i remember talking about that feature being removed and i cant find any reference in the new docs:
new documentation as i can find it on the old kinematic body documentation
So, maybe in this case @djmaesen would like to make a new issue in Godot proposals to discuss about the implementation of an equivalent feature if you like it. But is not technically a bug, but a missing feature.

Ooh, you may be right. Haven't used 4.x that much. I know most of the old flags were moved out of the move_and_collide() and move_and_slide() calls to elsewhere and thought it was the case with this too.

Ooh, you may be right. Haven't used 4.x that much. I know most of the old flags were moved out of the move_and_collide() and move_and_slide() calls to elsewhere and thought it was the case with this too.

I'm really deep in testing movement and sliding right now so i didn't get to the point of working on physics interactions, but thinking about it now it sounds like it could be a recurrent cause of doubts. While is not technically a bug, due to the amount of changes and the implementation of the feature to apply velocity for moving platforms, it would be great to have a section in the docs explaining that infinite_inertia has been removed, and ideally (But not so urgent) a little explanation about how to implement this things without mess with the built in feature for platforms.
I don't know ho is currently in charge of the documentation, but it would be great to, at least, let that person now about this.

IIRC you can switch it on, it's just not on by default unlike 3.x

I may be wrong, but i remember talking about that feature being removed and i cant find any reference in the new documentation as i can find it on the old kinematic body documentation
So, maybe in this case @djmaesen would like to make a new issue in Godot proposals to discuss about the implementation of an equivalent feature if you like it. But is not technically a bug, but a missing feature.

so its not a bug, but a design decision? i think this is a bad idea. adding a function to the move_and_ slide() code would really be helpfull.

Infinite inertia was removed from master by https://github.com/godotengine/godot/pull/51490.

Uuugh. Yet another case where it's eminently clear Godot 4 needs better docs, and fast, before more people pick it up and run into problems such as this.

so how do u apply "infinite inertia " in godot4 ? i dont understand the one directional collision layers

I don't either - it's one of the features that urgently need better docs.

please revert back to infinte inertia for 3d kinematic bodies. or at least add an option to enable it
i dont understand the tuse of directional collisionlayers.

please revert back to infinte inertia for 3d kinematic bodies. or at least add an option to enable it i dont understand the use of directional collisionlayers.

Here is a temporary solution:
func _push_rigidbody() -> void:
	if !enable_push_rigidbody: return

	var slideCollision
	for i in get_slide_collision_count():
		slideCollision = get_slide_collision(i)

	if slideCollision == null:
		return

	for i in slideCollision.get_collision_count():
		var collider = slideCollision.get_collider(i)
		if collider is RigidBody3D:
			collider.apply_central_impulse(-slideCollision.get_normal(i) * push_force)

In my case I get better results using a ShapeCast3D(CapsuleCast):
func _push_rigidbody() -> void:
	if !enable_push_rigidbody: return

	if _wall_helper_capsule != null:
		for i in _wall_helper_capsule.get_collision_count():
			var collider = _wall_helper_capsule.get_collider(i)
			if collider is RigidBody3D:
				collider.apply_central_impulse(-_wall_helper_capsule.get_collision_normal(i) * push_force)

i've already managed to get it working, it was as simple as setting up a correct collision layer interaction.

i've already managed to get it working, it was as simple as setting up a correct collision layer interaction.

Could you explain the collision layer interaction? I have a RigidBody3D with Collision mask and layer set to 1 and a CharacterBody3D set to the same but they are not interacting.
Obviously there are no docs on collision layers for Godot 4 (this is honestly a serious problem, I am going to start contributing there because it will be the death of the engine like this).

i've already managed to get it working, it was as simple as setting up a correct collision layer interaction.

How precisely do you set this up?
Have the player collision mask set to FALSE for rigidbodies you want to move (but the rigidbody mask for the player set to true)
Cause then the issue arises that the player doesn't interact with the rigidbody at all if the rigidbody is frozen.

FWIW, this is the only way I was able to get this working, and it seems to work pretty well:
  public void ProcessCollisions()
    {
        for (var i = 0; i < GetSlideCollisionCount(); i++)
        {
            var collision = GetSlideCollision(i);
            for (var k = 0; k < collision.GetCollisionCount(); k++)
            {
                var body = collision.GetCollider(k) as RigidBody3D;
                if (body == null)
                    continue;
                var point = collision.GetPosition(k) - body.GlobalPosition;

                var force = 5; // put whatever force amount you want here

                body.ApplyImpulse(-collision.GetNormal(k) * force, point);
            }
        }
    }

i've already managed to get it working, it was as simple as setting up a correct collision layer interaction.

How precisely do you set this up? Have the player collision mask set to FALSE for rigidbodies you want to move (but the rigidbody mask for the player set to true)
Cause then the issue arises that the player doesn't interact with the rigidbody at all if the rigidbody is frozen.

it was relatively simple set world collision layer to 1, player to layer2 with mask to layer1(world) and then set the rigidbodies collision to layer 4 and mask to 1,2 and 4

I just spent 3 hours trying to figure this out after upgrading. Thanks for the clear solution!
That said, At minimum this needs better documentation. Ideally I'd like to see a more straightforward workaround before the launch of 4.0

it was relatively simple set world collision layer to 1, player to layer2 with mask to layer1(world) and then set the rigidbodies collision to layer 4 and mask to 1,2 and 4

I cant manage to stand on the rigid body though, it just slides oout.

it was relatively simple set world collision layer to 1, player to layer2 with mask to layer1(world) and then set the rigidbodies collision to layer 4 and mask to 1,2 and 4

I cant manage to stand on the rigid body though, it just slides oout.

me neither

Sooo I guess the characterbody just has to manually push the rigid body.

Yes this is how it's always been, you have to choose one or the other or some people implement these interactions themselves

Haha thank you a lot for this, saved me a bunch of headaches trying to figure out why the heck this worked fine in my project on Godot 3.x and suddenly broke. Not sure why others are posting needed hacks?

@iamjoeysox

RigidBody2D sliding under a CharacterBody2D

Thats the entire reason the "hacks" are needed.
The hacks are actually the new intended method.

@iamjoeysox

RigidBody2D sliding under a CharacterBody2D

Thats the entire reason the "hacks" are needed. The hacks are actually the new intended method.

Haha yeah, I got myself super confused and figured out today I had it wrong lol. Thanks again!


multi_analysis with group_by only gives the data for 1st group

I have this query:
new Keen.Query('count', {
  event_collection: 'pageView',
  timeframe: 'this_30_days',
  interval: 'daily',
  group_by: 'user.ip',
});

which results in the expected output:
[ { value:
     [ { 'user.ip': '<IP_ADDRESS>', result: 1159 },
       { 'user.ip': '109.x.x.145', result: 0 },
       { 'user.ip': '88.x.x.170', result: 0 } ],
    timeframe:
     { start: '2016-05-13T18:30:00.000Z',
       end: '2016-05-14T18:30:00.000Z' } },
  { value: ...

But I wanted this query as a part of a multi_analysis. So I have this query now:
new Keen.Query('multi_analysis', {
  event_collection: 'pageView',
  analyses: {
      all: {
          analysis_type: 'count',
      },
      unique: {
          analysis_type: 'count',
          group_by: 'user.ip',
      }
  },
  interval: 'daily',
  timeframe: 'this_30_days',
});

This however results in this:
[ { value: { all: 1159, unique: 1159 },
    timeframe:
     { start: '2016-05-13T18:30:00.000Z',
       end: '2016-05-14T18:30:00.000Z' } },
  { value: { all: 1948, unique: 1948 },
    timeframe:
     { start: '2016-05-14T18:30:00.000Z',
       end: '2016-05-15T18:30:00.000Z' } },
  { value: { all: 1256, unique: 1256 },

There's all: 1159, unique: 1159 results in which the unique part seems to only have the data from the first result in the first query
 [ { 'user.ip': '<IP_ADDRESS>', result: 1159 },   <<
   { 'user.ip': '109.x.x.145', result: 0 },  << where is this data??
   { 'user.ip': '88.x.x.170', result: 0 } ],  << and this

Other results than the first one are apparently not being included in multi_analysis.
Is this by design or is it a bug?

@laggingreflex sorry for the oversight here – sharing with our platform team to investigate further 👍

@laggingreflex for multi-analysis queries, group_by is a top-level parameter -- a sibling to analyses and timeframe, and is ignored if passed in as you have it. In this case I would suggest defining running two parallel queries, like so:
var client = new Keen({ /* Configure */ });

var query1 = new Keen.Query('count', {
  event_collection: 'pageView',
  interval: 'daily',
  timeframe: 'this_30_days'
});

var query2 = new Keen.Query('count', {
  event_collection: 'pageView',
  group_by: 'user.ip',
  interval: 'daily',
  timeframe: 'this_30_days'
});

client.run([query1, query2], function(err, res) {
  // Handle res[0] and res[1]
});


Pygame detect and classify: SystemError: /dev/video16 is not a video capture device

Hello, I am having difficulty with the following error message when I try to run either detect.py or classify_capture.py through the pygame folder of the google coral examples-camera folder. I am using a USB webcam on Raspberry Pi 4.
pi@raspberrypi:~/coral/examples-camera/pygame $ python3 detect.py
pygame 1.9.4.post1
Hello from the pygame community. https://www.pygame.org/contribute.html
Loading ../all_models/mobilenet_ssd_v2_coco_quant_postprocess_edgetpu.tflite with ../all_models/coco_labels.txt labels.
By default using camera:  /dev/video16
Traceback (most recent call last):
File "detect.py", line 154, in
main()
File "detect.py", line 120, in main
camera.start()
SystemError: /dev/video16 is not a video capture device
I am able to successfully run the opencv example without an issue so the installation and configuration seems to be correct.  I did run the pygame install requirements bash file and there are no errors.  There seems to be some difficulty with how the usb webcam is detected.

@FunkyGandalf
This is an opencv issue, basically trying to open /dev/video16 to read inputs form the camera at this line.
You can check which camera index is the correct one and run it using this:
python3 detect.py --camera_idx {idx}


please release latest version to rubygems

i don't think the 1.0.3 version has been bumped since january of 2014?

poke?

poke again?

@gnowxilef Done but this gem is no longer maintained as none of us uses Hipchat anymore.
https://rubygems.org/gems/hipbot/versions/1.0.4


Don't rely on double spends to deduplicate transactions

Fix for #971
Will also make it easier to determine the failed txs for including in the consensus history.

Codecov Report
Base: 65.40% // Head: 65.42% // Increases project coverage by +0.02% :tada:

Coverage data is based on head (dde6318) compared to base (f361cb3).
Patch coverage: 100.00% of modified lines in pull request are covered.

:exclamation: Current head dde6318 differs from pull request most recent head d5b8d43. Consider uploading reports for the commit d5b8d43 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #998      +/-   ##
==========================================
+ Coverage   65.40%   65.42%   +0.02%
==========================================
  Files         116      116
  Lines       17399    17399
==========================================
+ Hits        11380    11384       +4
+ Misses       6019     6015       -4

Impacted Files
Coverage Δ

fedimint-server/src/consensus/mod.rs
92.91% <100.00%> (ø)

fedimint-server/src/lib.rs
80.46% <0.00%> (-0.26%)
:arrow_down:

modules/fedimint-wallet/src/lib.rs
85.42% <0.00%> (-0.09%)
:arrow_down:

fedimint-server/src/net/peers.rs
90.58% <0.00%> (+1.76%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Once this is merged I can remove d0c7a38abeb8e3ac73737f92ad6374cc7f4e725e from #975.


Terragrunt does not mix parent terragrunt.hcl content into .terragrunt-cache

Hi, I don't know if this is a bug or maybe a misconfiguration issue, but this schema worked the last time I used terraform and I don't find any other reason why it does not works.
Versions:
Terraform v0.12.3
Terragrunt v0.19.6
My directory structure
./
├── bin
│   ├── terraform
│   └── terragrunt
├── do-kubernetes-creds.json
└── live
    ├── sandbox
    │   └── terraform-instance
    │       └── terragrunt.hcl
    ├── stage
    └── terragrunt.hcl

cat ./live/sandbox/terraform-instance/terragrunt.hcl
terraform {
  source = "../../../../tf-modules//bastion-host"
}

include {
  path = find_in_parent_folders()
}

cat ./live/terragrunt.hcl
remote_state {
  backend = "gcs"
  config = {
    bucket         = "devops"
    prefix         = "terraform-state/${path_relative_to_include()}/terraform.tfstate"
  }
}

$ terragrunt plan
[terragrunt] [/Users/monkey/tf-test/live/sandbox/terraform-instance] 2019/06/27 13:03:30 Running command: terraform --version
[terragrunt] 2019/06/27 13:03:30 Reading Terragrunt config file at /Users/monkey/tf-test/live/sandbox/terraform-instance/terragrunt.hcl
[terragrunt] 2019/06/27 13:03:30 Cleaning up contents of download folder /Users/monkey/tf-test/live/sandbox/terraform-instance/.terragrunt-cache/C7klnJ0JOvtrV3YBXXBqrdYa60k/RtsJHkaVlj16ypBH01STcx8XS2k
[terragrunt] 2019/06/27 13:03:30 Downloading Terraform configurations from file:///Users/monkey/Directories/dial-once/tf-modules into /Users/monkey/tf-test/live/sandbox/terraform-instance/.terragrunt-cache/C7klnJ0JOvtrV3YBXXBqrdYa60k/RtsJHkaVlj16ypBH01STcx8XS2k
[terragrunt] 2019/06/27 13:03:30 Copying files from /Users/monkey/tf-test/live/sandbox/terraform-instance into /Users/monkey/tf-test/live/sandbox/terraform-instance/.terragrunt-cache/C7klnJ0JOvtrV3YBXXBqrdYa60k/RtsJHkaVlj16ypBH01STcx8XS2k/bastion-host
[terragrunt] 2019/06/27 13:03:30 Setting working directory to /Users/monkey/tf-test/live/sandbox/terraform-instance/.terragrunt-cache/C7klnJ0JOvtrV3YBXXBqrdYa60k/RtsJHkaVlj16ypBH01STcx8XS2k/bastion-host
[terragrunt] [/Users/monkey/tf-test/live/sandbox/terraform-instance] 2019/06/27 13:03:30 Initializing remote state for the gcs backend
[terragrunt] 2019/06/27 13:03:30 Missing required GCS remote state configuration project
[terragrunt] 2019/06/27 13:03:30 Unable to determine underlying exit code, so Terragrunt will exit with error code 1

ls .terragrunt-cache/C7klnJ0JOvtrV3YBXXBqrdYa60k/RtsJHkaVlj16ypBH01STcx8XS2k/bastion-host
main.tf        terragrunt.hcl vars.tf

cat .terragrunt-cache/C7klnJ0JOvtrV3YBXXBqrdYa60k/RtsJHkaVlj16ypBH01STcx8XS2k/bastion-host/terragrunt.hcl
terraform {
  source = "../../../../tf-modules//bastion-host"
}

include {
  path = find_in_parent_folders()
}

As you can see terragrunt does not mixed the root terragrunt.hcl remote state config file with the child one, I know this does not not represent the documentation neither terragrunt v0.18.x behavior.
Thanks in advance for the time anyone invest in looking in my issue.

I found that using Terragrunt v0.19.0 neither merge the terragrunt.hcl file contents in .terragrunt-cache but now the initialization backend works. So actually something is wrong with v0.19.6

@robmorgan Could this be a bug with the recently added GCP/GCS support?

Hi i have the same problem with Azure.
terragrunt: 0.19.6
terraform: 0.12.3
this is my tree:
.
├── README
├── terragrunt.hcl
├── vars.tfvars
└── westus2
    ├── mgmt-subnet
    │   └── terragrunt.hcl
    ├── mgmt-vm-diego-lab-01
    │   └── terragrunt.hcl
    ├── mgmt-vnet
    │   └── terragrunt.hcl
    ├── rg
    │   └── terragrunt.hcl
    └── terragrunt.hcl

This is my root terragrunt.hcl
remote_state {
  backend = "azurerm"

  config = {
    resource_group_name  = "${get_env("RG_NAME_INFRA", "diego-infra")}"
    storage_account_name = "${get_env("TERRAFORM_STORAGE_ACCOUNT_NAME", "diegotf")}"
    container_name       = "${get_env("TERRAFORM_CONTAINER_NAME", "diego-tf")}"
    key = "development/${path_relative_to_include()}/terraform.tfstate"
  }
}

This is my root/westus2/terragrunt.hcl
inputs = {
    location = "westus2"
}

And finally this is my live hcl file root/westus2/rg/terragrunt.hcl
terraform {
  source = "${find_in_parent_folders()}/../../../diego-azure-tf-modules//resource_group"
}

# Include all settings from the root terraform.tfvars file
include {
  path = find_in_parent_folders()
}

with terragrunt plan into root/westus2/rg/terragrunt.hcl, i got this is the error:
diego@AC-DMlagos2:/mnt/c/code/public.cloud/terraform-lab/diego-azure-tf-live/westus2/rg$ terragrunt plan
[terragrunt] [/mnt/c/code/public.cloud/terraform-lab/diego-azure-tf-live/westus2/rg] 2019/06/28 12:25:09 Running command: terraform --version
[terragrunt] 2019/06/28 12:25:12 Reading Terragrunt config file at /mnt/c/code/public.cloud/terraform-lab/diego-azure-tf-live/westus2/rg/terragrunt.hcl
[terragrunt] 2019/06/28 12:25:12 Cleaning up existing *.tf files in /mnt/c/code/public.cloud/terraform-lab/diego-azure-tf-live/westus2/rg/.terragrunt-cache/zAZxgP0FApqtssVNnTa-yrc98c4/RbFKWdQdmvbTH-w05FIEKUUoxKo
[terragrunt] 2019/06/28 12:25:12 Downloading Terraform configurations from file:///mnt/c/code/public.cloud/terraform-lab/diego-azure-tf-modules into /mnt/c/code/public.cloud/terraform-lab/diego-azure-tf-live/westus2/rg/.terragru
nt-cache/zAZxgP0FApqtssVNnTa-yrc98c4/RbFKWdQdmvbTH-w05FIEKUUoxKo
[terragrunt] 2019/06/28 12:25:12 Copying files from /mnt/c/code/public.cloud/terraform-lab/diego-azure-tf-live/westus2/rg into /mnt/c/code/public.cloud/terraform-lab/diego-azure-tf-live/westus2/rg/.terragrunt-cache/zAZxgP0FApqts
sVNnTa-yrc98c4/RbFKWdQdmvbTH-w05FIEKUUoxKo/resource_group
[terragrunt] 2019/06/28 12:25:12 Setting working directory to /mnt/c/code/public.cloud/terraform-lab/diego-azure-tf-live/westus2/rg/.terragrunt-cache/zAZxgP0FApqtssVNnTa-yrc98c4/RbFKWdQdmvbTH-w05FIEKUUoxKo/resource_group
[terragrunt] [/mnt/c/code/public.cloud/terraform-lab/diego-azure-tf-live/westus2/rg] 2019/06/28 12:25:12 Running command: terraform init

Initializing the backend...
container_name
  The container name.

temporary solution
But if i put the configuration of root/terragrunt.hcl (so the backend configuration) into the file root/westus2/terragrunt.hcl, everithing works as expected.
I think that terragrunt was not able to completely reconstruct the whole folder hierarchy.

@robmorgan Could this be a bug with the recently added GCP/GCS support?

I've got something strange happening at the moment with GCS config in the same folder and its due to some of the extended configuration not being passed in correctly, but I'm still debugging it.

I've got something strange happening at the moment with GCS config in the same folder and its due to some of the extended configuration not being passed in correctly, but I'm still debugging it.

Thanks.
@brikis98 I set TF_LOG=DEBUG with my plan command and noted that remote state backend is passed as arguments to terraform init as:
-backend-config=prefix=terraform-state/sandbox/terraform-instance/terraform.tfstate", "-backend-config=bucket=devops-dialonce", "-backend-config=credentials=/Users/monkey/tf-test/live/../creds.json"

Was this always the behavior to remote state backend instead of mixing the content in .terragrunt-cache as I thought it should be?

I have a feeling it may be related to: https://github.com/gruntwork-io/terragrunt/commit/2dd80cf37de675266bb21c89319e4cddf4e18d0c, but so far I have not been able to reproduce it consistently.


Resizing image while using app

Hello,
dithering images is often used in conjunction with shrinking the image, in order to reduce the perceptual color loss while drastically reducing file size. Is there a way this could be added into the app?

There is already a way to do shrink images, though you are not able to specify the exact image dimensions. If you load an image and increase the pixelation in the image tab and then export with the size setting set to "actual", you will get a smaller image.
Also as well, if a large image is imported and the "shrink large images" setting is checked in the settings tab the image will be shrunk proportionally so the largest dimension (width or height) is 1200 pixels.

Hmm, looking at this now, there appears to be a bug with the size setting when exporting, it is supposed to function as described in my previous comment, but it doesn't look like it is working, I will work on a fix.


add itemProperties openNested option for nested schema

Add the option to have a nested schema on mount, this way the user doesn't have to click the + button in order to add the first item, he/she can just start typing in the data.
itemProperties: {
   openNested: true
}

Hi, does passing prefilledProps directly to the SmartForm works? I'd rather go for the solution that require less changes in the code if this use case is already covered by props

Didn't work for me, the item stays closed. Please let me know if I missed anything.

Also in my use case I'd prefer it to stay empty, simply open, so the user doesn't have to click the +

@yairtal can you please send me a snippet with prefilledProps that did not work?
In the form example I can have this:

With those props (used in a Datatable):
![image](https://user-images.githubusercontent.com/7983005/86384352-e5807380-bc8e-11ea-88d1-f9fd9fbeaf39.png)

Closing in the meantime so I don't forget but of course I'll reopen if you have any additional info

Maybe it messes up placeholders though? But your solution would have the same limitation as you programmatically do the same thing as having prefilledProps with an empty object.

This didn't work in my case since my case is an Edit/Update case, I have a documentId prop in my SmartForm.

Ok I get the use case that makes sense.

I am not a huge fan of adding more and more options to the form but I know this one is pretty common. I hope to make it easier to build custom forms based on the Vulcan components instead of systematically having to tweak the whole form behaviour to add such features, it should be easier in the first palce.


"Reopen Closed Editor" has strange behavior with split editor

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

VS Code Version: 1.79.1
OS Version: macOS Ventura 13.4

Steps to Reproduce:

Open then close a file (file1)
Open two other files (file2 and file3
With File2 selected, split the editor
There should be File2 on both the left and right editors. Click on the right editor so your cursor is in that editor
Close File2 on the left (original) editor
Hit CMD + Shift + T (File -> Open Recent -> Reopen Closed Editor). It will open some file that was closed in the right editor  (or do nothing if you've never used this workspace before)
Click on the left editor (which should be showing File3)
Hit CMD + Shift + T

Expected Behavior:
It opens File2 again, since that was the last file opened in that editor
Actual Behavior:
It opens File1.
This seems to have something to do with the file being in both editors and being closed in one. Note that though the reproduction steps seem convoluted, this happens to me when I have several files open across a split editor and I'm cleaning them up.

I have my cursor in the right editor
I close several tabs on the left side, closing one by accident
I hit CMD + Shift + T to reopen it, but I reopen something in the right editor instead
Click in the left editor and hit CMD + Shift + T
Get some other file instead of the one I had just meant to get back

To clarify: "Reopen closed editor" is not aware of editor groups, it works in a very simple way:

listen to editors that close
remember the closed editor in a stack
when "Reopen closed editor" is called, pick the most recent editor from that stack
if the editor is not already opened, open it
otherwise continue in a loop until an editor was opened

I think what you are asking here is scope "Reopen closed editor" to the editor group the editor was in? Essentially allowing to reopen editors exactly how they were in each group?

I'm sorry @bpasero I think I'm getting the terminology confused. I guess what I was calling a "file" is really an "editor" and what I was calling an "editor" was more like a tab group (when you do "Split editor" and it makes two things, those "things").
What I'm describing is not non-ideal behavior but what I think is an actual bug. There are circumstances where "Reopen closed editor" does not open an editor that was closed but "skips" it. I'm pretty sure it's not correct behavior because in some circumstances it does reopen it and in others doesn't.
Here's a video:
https://github.com/microsoft/vscode/assets/5677803/7c830d79-2c36-4eea-9f96-932af47aacff

I have a situation where I have file2.txt open on the left and right
With my cursor on the right side, I close "file2.txt" on the left side
When I hit CMD + Shift + T, it opens an unrelated file that is NOT the last one I closed on the right side
When I move my cursor to the left side and hit CMD + Shift  + T, it opens another unrelated file, and that original file2.txt I want back is no longer on the stack

Thanks, this is helpful and matches what I had said: "Reopen closed editor" does not take editor groups into account. It will reopen the last editor that was closed across all groups and does not take individual editor groups into account.
Previously https://github.com/microsoft/vscode/issues/50695

@dustin-relicx what bug? In my testing I did not find any bugs. If you have more specific steps that show a bug, please post them here.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Invalid Domain

I'm trying to load my config file for TrustKit with an internal domain that ends in .qa, for internal testing purposes. However it keeps throwing "Terminating app due to uncaught exception 'TrustKit configuration invalid', reason: 'TrustKit was initialized with an invalid domain MYDOMAIN.qa"

Hello,
Domains that are not using a public TLD (https://wiki.mozilla.org/Public_Suffix_List) are not supported.

Have there been any changes on this? Usually when using an in-house CA you also want to pick your own domain names for the servers.


realsense2[openni2] build failure

Host Environment

OS: Windows 10.0.18363
Compiler: MSVC v142

To Reproduce
Steps to reproduce the behavior:
./vcpkg install realsense2[openni2]
I have VCPKG_DEFAULT_TRIPLET = x64-windows
Failure logs
PS C:\vcpkg> vcpkg install realsense2[openni2]
Computing installation plan...
The following packages will be built and installed:
    realsense2[core,openni2]:x64-windows
Starting package 1/1: realsense2:x64-windows
Building package realsense2[core,openni2]:x64-windows...
-- Using cached C:/vcpkg/downloads/IntelRealSense-librealsense-9f99fa9a509555f85bffc15ce27531aaa6db6f7e.tar.gz
-- Using source at C:/vcpkg/buildtrees/realsense2/src/aaa6db6f7e-746a483df4
-- Configuring x64-windows
-- Building x64-windows-dbg
-- Building x64-windows-rel
CMake Error at ports/realsense2/portfile.cmake:79 (file):
  file COPY cannot find
  "C:/vcpkg/buildtrees/realsense2/x64-windows-rel/_out/rs2driver*".
Call Stack (most recent call first):
  scripts/ports.cmake:90 (include)

Error: Building package realsense2:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: realsense2:x64-windows
  Vcpkg version: 2020.02.04-nohash

install-x64-windows-dbg-out.log
install-x64-windows-rel-out.log
config-x64-windows-out.log
(most other logs are empty)
Additional context
I already had openni installed, so I was doing vcpkg install realsense2[openni2] --recurse the first time I got this error.

@FSund, thanks for reporting this issue!
I will summit a PR for fixing this issue.

@FSund, the PR has been merged, please get latest source and try again.

Nice, thanks!


Where has UMI count for AIR chains gone?

Describe the bug
I am looking to access UMI count for each chain of a set of antibodies read into the scirpy anndata frame. It appears that read count is present in the awkward array associated with the airr anndata obsm awkward array under the title 'consensus_count' like so:
adata["airr"].obsm['airr']['consensus_count']
but the UMI count is nowhere to be found. None of the available fields point to the UMI count.
where has this field gone (it is present in the original 10X filtered_contig_annotation.csv input file)?
System

OS: Linux
Python version: 3.10.3
Versions of libraries:
anndata             0.10.4
awkward             2.5.1
matplotlib          3.8.2
mudata              0.2.3
muon                0.1.5
numpy               1.26.2
pandas              2.1.3
scanpy              1.9.3
scirpy              0.14.0

Additional context
Add any other context about the problem here.

Hi Adam,
thanks for bringing this up!
The UMI count goes to adata["airr"].obsm["airr"]["duplicate_count"].
I always found this a bit confusing, and I've just seen that there has been an update to the AIRR Rearrangement standard that introduces a dedicated umi_count field which is a lot clearer. I'll update scirpy to use this field instead in the future.
Best,
Gregor


xarray_open_dataset should use open_zarr not open_dataset

I'm not sure why, but open_zarr determines the chunks of the dataset whereas open_dataset does not.

closing as i think we want to be flexible to other engines which are xarray readable (e.g. netcdf)


azblob: Upgrade to STG 85-86

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] MIT license headers are included in each file.

/azp run go - azblob


Css problem

css problem and nothing is displayed when we type any message or create room

check your developer tool logs - I need some error messages to be able to help you out.

Developer tool shows error like this:
Failed http://<IP_ADDRESS>:8000/components/font-awesome/css/font-awesome.css
to load resource: the server responded with a status of 404 (Not Found)

you probably miss the dependencies for the project - have you run bower install?

No, how to run bower install?

As stated in my earlier comment bower install. If you don't have bower installed install it by npm install -g bower. For further info: http://bower.io/#getting-started

Still those errors are coming out.I think that's because in zip file there was no css file named font-awesome.css and also no folder named components

after bower install do you get a folder 'bower_components'? if so, rename that to 'components'. If the error with the font-awesome.css still persist, get the css manually from https://fortawesome.github.io/Font-Awesome/

I have installed bower but i haven't got folder bower_components.
May you explain  how to configure bower.json?

there's already a bower.json for this project - you don't need to add one. In the project's root folder you can go ahead and execute bower install and that should install the project dependencies. Please make sure you don't receive any errors while running the aforementioned command.

Actually 5 errors are still coming out. Two for bootstrap.css , font-awesome.css and other two are for jquery.js, bootstrap.js. I have installed bower but these errors are still coming out

when you say 'I have installed bower' do you mean that you have actually installed bower (by executing npm install bower -g) or that you have installed the bower packages (by executing bower install)?
If you still have the errors fetch those dependencies manually and save them to your project - also go ahead and change the references in the index.html file.

I have installed bower both way globally and locally but there is no folder named bower_components.
Now i am updating bower.json by using command
$ bower install --save jquery bootstrap
Error comes like this...
bower ENOGIT        git is not installed or not in the PATH
what should i do?

that's your issue at the first place - you need git installed as well. please google that.

thanks a lot, now everything is working perfectly.

awesome - enjoy the app! :)


add check quota

add check quota

Codecov Report

Merging #74 into develop will decrease coverage by 0.07%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##           develop      #74      +/-   ##
===========================================
- Coverage    41.86%   41.78%   -0.08%
===========================================
  Files          129      129
  Lines        22773    22825      +52
===========================================
+ Hits          9533     9537       +4
- Misses       13240    13288      +48

Impacted Files
Coverage Δ

tx_pool/src/pool.rs
82.39% <ø> (ø)
:arrow_up:

libproto/src/protos/auth.rs
0% <0%> (ø)
:arrow_up:

libproto/src/protos/executor.rs
0% <0%> (ø)
:arrow_up:

ethcore-io/src/worker.rs
76.47% <0%> (+7.84%)
:arrow_up:

Continue to review full report at Codecov.

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


Added submitted mixin to schemas missing it

Added the submitted mixin to the following schemas:
award, biosource, construct, quality_metric*, software, summary_statistic*, treatment*, user, vendor and workflow_run.
Will separately add to the new schema in the add_capture_hic branch.

Coverage remained the same at 37.857% when pulling 4b7561b41dc91fb9a377e56ca850fbcb2bfbf6ed on add_submitted into 3e384bfd24988f8a2cc6d4b2c72c851db53ddaf8 on master.


When viewing extension details: Error loading webview: Error: Could not register service workers

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

VS Code Version:1.56.2 (user setup)
Commit: 054a9295330880ed74ceaedda236253b4f39a335
Date: 2021-05-12T17:13:13.157Z
Electron: 12.0.4
Chrome: 89.0.4389.114
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS Version: Windows_NT x64 10.0.19043

Steps to Reproduce:

Launch VS Code
Search for an extension in the extension sidebar
Click on the extension
Error displayed:
Error loading webview: Error: Could not register service workers: TypeError: Failed to register a ServiceWorker for scope ('vscode-webview://extensioneditor/') with script ('vscode-webview://extensioneditor/service-worker.js?platform=electron&id=extensionEditor&vscode-resource-origin=https%3A%2F%2FextensionEditor.vscode-webview-test.com'): ServiceWorker cannot be started.

(Expected):

Launch VS Code
Search for an extension in the extension sidebar
Click on the extension
Extension details displayed

Screen capture:

Duplicate of #122951

Thanks for the reference. Tried searching but didn't find that one. Following


JAMES-3911 JPA: position messages index correctly

@chibenwa do you happen to have/plan to run same tests as in https://github.com/apache/james-project/pull/1593 and publish the results as well?

A bit busy lately but yes I definitly should!


removing duplicate URL so this is copy and pastable

fixing issues in readme as promised

Hello @ksigler7,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


toxiproxy: 2.5.0 -> 2.6.0

Description of changes
This PR updates toxiproxy to its latest release: https://github.com/Shopify/toxiproxy/releases/tag/v2.6.0
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.


nixos/tests/systemd-boot: fix option bootCounting test evaluation

Description of changes
Option name was changed in #330017 but the test was not updated.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg test systemd-boot


to implement profile 04

can you help to implement profile4 in the autosar official document?
https://www.autosar.org/fileadmin/standards/R20-11/FO/AUTOSAR_PRS_E2EProtocol.pdf

Try this branch: https://github.com/zariiii9003/autosar-e2e/tree/p04

It works, many thanks


Added to possibility to manually set sign and digest alg. as well the possibility to change the default values for sign and digest alg.

View example sp-wsgi and dip how the default values can be changed.

Needs a change to setup


Added How To Play Page and Navbar

I have added a Rules and FAQ's page. It consists of the basic rules to play the game.
It also consists of the general FAQ's regarding the game.
In addition to this, I have also added a Navbar to the website.
Thank You

send screenshots

send screenshots

Here you go
For the Home page :

For the How to Play Page :

Pull for the same changes again

Pull for the same changes again

Pull for the same changes again

I didn't get you. Could you please clarify more.


Update common/ai_templates/generic.txt

Updated for 1.5, we need to update too.

Updated in #478 (removed the blocked_for = { [list of tags] })


multipel 'RobotModel' cannot have different alpha

I have two RobotModel (A and B) added to RViz, which show the same robot model at different configurations (A, B). Both widget have separate settings for the Alpha value. However, if I change the alpha for one of the RobotModel (A), the other one changes (B) too:

The alpha value is not updated in the parameter overview of the other widget B, but it seems as if both models share the same visual properties.
Your environment

OS Version: Ubuntu 18.04
ROS Distro: Melodic
RViz, Qt, OGRE, OpenGl version as printed by rviz:

rviz version 1.13.3
compiled against Qt version 5.9.5
compiled against OGRE version 1.9.0 (Ghadamon)

Duplicate of #1385. Fixed in #1387, which was released into 1.13.4.

Thanks for fixing this. I didn't notice that duplicate bug since I only searched in the open issues.
Looking forward to a binary release :-)


emmmmm...Why?

main.py
from fastapi import FastAPI
from child import child_router

app = FastAPI()

# want: /child/info -> √ /child/ws -> √
# got: /child/info -> √ /child/ws -> ×  /ws -> √
# app.include_router(child_router)

# want: /child/info -> ×, /child/ws -> ×, /child/child/info -> √  /child/child/ws √
# got: /child/info -> ×, /child/ws -> √, /child/child/info -> √ /child/child/ws ×
app.include_router(child_router, prefix=child_router.prefix)

child.py
from fastapi import APIRouter, WebSocket

child_router = APIRouter(
    prefix="/child"
)

@child_router.websocket("/ws")
async def child_websocket(websocket: WebSocket):
    await websocket.accept()
    while True:
        try:
            await websocket.receive()
        except:
            pass

@child_router.get("/info")
async def get_child_info():
    return "child information"

I understand your issue, but do you really think this is a way to express yourself?
Anyway, it's a bug. You should not add the prefix on the APIRouter if using websockets (until it's solved).
A PR to solve that was already created and is waiting for review: https://github.com/tiangolo/fastapi/pull/2640

我了解您的问题，但是您真的认为这是表达自己的方式吗？
无论如何，这是一个错误。APIRouter如果使用websockets ，则不应在前缀上添加前缀（直到解决）。
已解决的PR已创建，正在等待审查：＃2640

orz, sorry, my poor English.  It's a good way for me to express myself

没关系。不用担心
如果您需要任何帮助，请告诉我
:)

At the very least you should provide a readable issue title.

At the very least you should provide a readable issue title.

受教了. > - < .      -- 菜鸟敬上

Thanks for the help here everyone! 👏  🙇
If that solves the original problem, then you can close this issue ✔

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


Integrate LLVM at llvm/llvm-project@cedc53254a5d

Update BUILD files and submodule for
cedc53254a5d

You didn't actually update the submodule. In fact the description says this is for the same commit we're already at?


Feat/semantic improvement v1

Query is spell checked and corrected by ChatGPT.
5 additional semantically similar queries merged with original query.
6 queries goes through the search_index func.
Dynamic sentences used for creating dynamic context.
New context used in prompt for ChatGPT response generation.

1- assign reviewer
2- assign project
3- assign asignee


docs: add links to available languages

Took me a while to figure out the available options, this should help anyone else new to this repo.

thanks!


[Ruby] Support newer releases of the FFI gem

Describe the feature request
Can the Ruby bindings be made to work with newer releases of the ffi gem? Its preventing us from upgrading the unleash ruby client to 6.0 due to dependency conflicts.
Background
The unleash ruby client version 6.0.5 depends on yggdrasil-engine ~> 0.0.6 which depends on ffi ~> 1.15.5 and we depend on ffi ~> 1.16.3
Solution suggestions
While I haven't dug into the Ruby bindings to see if there's a reason for not allowing newer versions of the ffi gem, it should be as simple as changing which versions of the ffi gem are allowed.

Hey @Th3-M4jor, thanks for reporting this. There shouldn't be any reason why we can't upgrade, I'll try patch that today

@Th3-M4jor, we've upgraded and released, you should be able to pick up the latest Unleash ruby client. Thanks again for reporting this!


[UpdatePlugin]  Martin Petran ghinja 1.3

Please update my plugin:
Author: Me
Repo name: ghinja
Release: https://github.com/Martyx00/ghinja/releases/tag/1.3

Done, thanks!


Using absolute imports (was relative imports)

Based on #49

@yanoz: it looks like project's authors (@magopian and @Natim) don't have permissions on this Github repository. Can you grant them admin privileges on this repo?

nop... we already discussed that few monthes ago. No solution has been validated nor implemented.
Thing is, I don't want to add @magopian and @Natim as member of Novapost organization.
An idea was to create Novapost OSS organization.
You tell me. Of course my point is not to "close" the project.
@rodo @lisael your idea?

Hopefully github has got another solution for you. It is to set collab people on a project basis so you do not need to create another organisation.
The thing is it was the case on this project last time we checked it and today I wanted to restart the travis for this PR and I was not allowed anymore.
I don't understand what prevent you from adding some contributors to the project they are still maintaining? Why don't you put them private if you don't want the community to help improve them?

@Natim it seems that you misunderstood the problem, we want to open projects and in first maintain orginal contributor/maintainer, but we can't include external people in corporate organization, sure you understand these point. If the old way things were done does'nt work (it's the case when you loose your rights) we have to change them, a new organization called something like NovaFloss or other may be a good ideau, this organization will contain all public projects.
I need more information in why just stand alone project does'nt work for us ? Why do we need to keep django-mail-factory and other project under Novapost organization ?

"Of course my point is not to "close" the project. "
Please, read carefully before writing stupid things.

but we can't include external people in corporate organization, sure you understand these point

No that's exactly the point.

Why do we need to keep django-mail-factory and other project under Novapost organization ?

I am not at Novapost anymore so I wont argue more than this statement about this.
If you want to create a new org for opensource project that's a solution but here is why I think it is not a good solution:

Having all project in the same organisation is easier for Novapost workers because they don't have to think about if it should be a public project or not upfront, they can start it private and discover later that it can be made public
Having all project under the novapost organization helps the community to decide if the project is worth using. It is a also a good advertisement for the Novapost / Peopledoc brand
Having another organisation is fine I think but it will bring the same problems as for the novapost organisation because you will have more rights managements to do (adding people on both orgs) and you will still need to define collab on a per project basis (because all collaborators should not be able to commit on all projects)

"Of course my point is not to "close" the project. "

I never said such a thing.

lol, no I did.

Again, I won't add you as Novapost organization member. That's a fact.
Let's find another solution.

I won't add you as Novapost organization member

This is not what I ask for.
I was thinking about: https://help.github.com/articles/adding-or-inviting-members-to-a-team-in-an-organization/
If you create a team per project it will let you give access to this project to some people that are not members of the Novapost organization.
If you want we can host public project with external collab on the spiral-project org.

Having all project in the same organisation is easier for Novapost workers because they don't have to think about if it should be a public project or not upfront, they can start it private and discover later that it can be made public

You make me scared, you don't start by reading the licence before contribute to a project ?
More seriously be sure we work hard to find the best solution, I'll read about Github Teams, I discover this github functionnality, thanks for that. I always worked on public project I'm not comfortable when I have to close doors.

You make me scared, you don't start by reading the licence before contribute to a project?

All project starts private (Authorship) until you put a Licence on them.

But you're ok that if you become member of this specific project's team, then you become member of Novapost Organization, right?

You mean that our pictures may appear on the first page of the organization?
Apparently yes but we will have only rights of the teams we are on.
I don't understand what is your definition of organization's member.

What truly matters is PyPI and readthedocs. Code repository can be moved. So at Novapost we should not block project development because of permissions on code reporitory. It is "fix permissions" VS "move repository".
Nobody at Novapost owns this project on PyPI or readthedocs... so at the moment Novapost cannot pretend to actually maintain the project.
Authors can upload to readthedocs and PyPI, but they don't have permissions on code repository.
Weird.

 That's a fact.
Let's find another solution.

@Natim, @magopian: in the case Novapost does not implement another solution (novafloss, teams or whatever) quick enough (your appreciation), then you can (that's another fact) clone the repository in a place you own, change repository URL in docs+setup.py, then update PyPI and readthedocs. Imho you may not wait for months again.

In the case Novapost does not implement another solution (novafloss, teams or whatever) quick enough (your appreciation), then you can (that's another fact) clone the repository in a place you own, change repository URL in docs+setup.py, then update PyPI and readthedocs. Imho you may not wait for months again.

Yes but we love Novapost too much for that. I prefer to take the time to solve this problem and let Novapost own her projects than to fork and take control. Also I will add people that want to release new version as soon as we find a solution.
If the solution is create a new org, @yanoz it is your call and that's not really my problem. I just say I won't do that for the reasons I explained earlier. But if you think it is more important that people doesn't see my face on the Novapost organization page, then I don't have anymore arguments.

It's very funny because I thought (I was wrong...) that you @benoitbryon  would be one of the person that would have helped us to find a good solution to fix this problem...

I rather think it is in fact a good move to make sure the conversation keep going in the right direction with all the arguments on the table.

Enough solutions have been mentioned to stop discussing and actually do something.
I don't have enough permissions at Novapost organization to actually implement one of them.

But if you think it is more important that people doesn't see my face on the Novapost organization page, then I don't have anymore arguments.

well... let's end this dicussion on this clever remark. We'll find a solution and we will notify you when it's done.

Cool thanks.

@Natim please let us share a beer at Fosdem to exchange on this, I need your github's knowledge on team, @magopian indeed if you are in Brussels this week end too I'll enjoy to see you at the table.

Yes it will be my pleasure, plus I owe some beers to @boblefrag for its Django book.

@benoitbryon I added you on pypi as owner of the project as well as @hobbestigrou as maintainer.

Sorry for closing this, but it's way to old, please re-open if this still matters to you.


global: jinja2 friendlier console warnings

Since the introduction of https://github.com/inveniosoftware/invenio/commit/552d1c4cca765911f5df100abb862e8b39d8862f
the devserver console is full of messages like:
/home/jmartinm/.virtualenvs/pu-inspire2/local/lib/python2.7/site-packages/jinja2/environment.py:966: RuntimeWarning: Convert string '/home/jmartinm/.virtualenvs/pu-inspire2/local/lib/python2.7/site-packages/jinja2/environment.py:966: RuntimeWarning: Convert string '.eJxtkNEKgjAUhl9lCd1J0QN0YbnoohJmEGEiyy064Ka4U-Tbl2Fzhpf_9_0czjmJF9MdXR_JrawVxxkIEsRdyED4P665ko5po3V5KVzXRuuENHkNFUKpnYpDnSkapcYMm2o4rce2q0DJ_6JltvUEA1coABun1kPbK7jB7FEJjtK9foAvesOifac-6bSljI6uPlmSqSHBIRxZ46siFlJGVmf3tZ6feChfOL-jKjx_kaZvaAWTqg.bqPhXDl7sl1Y16GkEl8OK5-4L5M' in template to unicode.
  return concat(self.root_render_func(self.new_context(vars)))
' in template to unicode.
  return concat(self.root_render_func(self.new_context(vars)))

/home/jmartinm/.virtualenvs/pu-inspire2/local/lib/python2.7/site-packages/jinja2/environment.py:966: RuntimeWarning: Convert string '/home/jmartinm/.virtualenvs/pu-inspire2/local/lib/python2.7/site-packages/jinja2/environment.py:966: RuntimeWarning: Convert string '0.00059700012207' in template to unicode.
  return concat(self.root_render_func(self.new_context(vars)))
' in template to unicode.
  return concat(self.root_render_func(self.new_context(vars)))

/home/jmartinm/.virtualenvs/pu-inspire2/local/lib/python2.7/site-packages/jinja2/environment.py:966: RuntimeWarning: Convert string '/home/jmartinm/.virtualenvs/pu-inspire2/local/lib/python2.7/site-packages/flask_debugtoolbar/templates/panels/sqlalchemy.html:31: RuntimeWarning: Convert string '

SELECT format.id AS format_id, format.name AS format_name, format.code AS format_code, format.description AS format_description, format.content_type AS format_content_type, format.mime_type AS format_mime_type, format.visibility AS format_visibility, format.last_updated AS format_last_updated
FROM format
WHERE format.content_type != %s AND format.visibility = %s ORDER BY format.name

' in template to unicode.
  {% trans "Line" %}
' in template to unicode.
  return concat(self.root_render_func(self.new_context(vars)))

I understand the unicode issue in the templates is worth fixing, but it:

Makes it difficult to work with, as all those messages are mixed up with messages you might be interested it while working on other features.

Don't seem helpful, as I don't see any pointer to the template that has the problem and what exactly the problem is. (well i know it is a unicode issue, but not exactly what to do about it)

So, can we either hide those messages or make them more explicit so that people that see them can actually fix the problem? I don't think in the current state they are going to have the desired effect.

Yeah, and in addition to several deprecation warnings, when loading search result pages, the console often crashes and reboots itself so you have to refresh a few times to load the page etc.
Can we redirect this output elsewhere or opt in for it?

I think @jirikuncar is working on fixing the stack trace issue.
invenio.ext.jinja2hacks was initially a dirty temporary measure to be able to decode 8-bit string to unicode using utf8 instead of ascii. I know @jirikuncar have spent several days trying to fix jinja2hacks so that we could continue as normal, but the fact is that there's no easy solution.
The reason that these warnings are loud is because they will break your production service and that they must be dealt with. I think we need all hands on deck to fix this issue and I'm aware it's interrupting existing work but travis builds is failing because of this so we are largely blind for integration as it is now.
To disable it simple remove the the warnings.warn from invenio.ext.jinja2hacks.
The fix for the problem is quite simple: {{ var }} -> {{ var|u }}

@lnielsen Thanks for the info. @jmartinm and I was discussing with @jirikuncar  and I got the understanding we could solve it also outside of the templates from the source of the variable - avoiding adding | u everywhere? Or will that just be about adding unicode() or  str.decode everywhere?
@jirikuncar also noticed that the flask toolbar produced several of these outputs, so disabling it with DEBUG_TB_ENABLED=False in the config helped locate the "real" issues.

Yes, you're right it can also be fixed with s.decode('utf8') (which I think is about the same as {{ var|u }}). I think I prefer {{ var|u }} over s.decode('utf8'), but naturally the very best would be to ensure we deal with a unicode string in the first place (I think that's however a lot harder to fix).

@jmartinm you see from where the errors are coming e.g. flask_debugtoolbar/templates/panels/sqlalchemy.html:31. I think we can close this issue as we discussed it IRL.


Pairing Initiated from Central Device

While using the nRF SoftDevice for Bluetooth Low Energy (BLE) applications, there is a need to initiate the pairing process from the central device when establishing a connection with a peripheral device. However, no official documentation or examples demonstrating how to achieve central-initiated pairing with the nRF SoftDevice have been found.

I don't think anyone has needed central pairing yet, so it hasn't been implemented. Should be reasonably straightforward to add, but you'd need to refer to the softdevice sequence diagrams and API to add the missing bits. PRs welcome!


Failed to complete chroot setup.

**```
Please paste the output of the following command here: sudo edit-chroot -all
name: kali-rolling
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/kali-rolling...
crouton: version 1-20181001133934~master:38012fdf
release: kali-rolling
architecture: amd64
targets: xfce
host: version 11151.59.0 (Official Build) stable-channel snappy
kernel: Linux localhost 4.4.159-15346-gc5e629939ae0 #1 SMP PREEMPT Tue Dec 11 00:13:48 PST 2018 x86_64 GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/kali-rolling...

name: kali-rolling
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/kali-rolling...
crouton: version 1-20181001133934~master:38012fdf
release: kali-rolling
architecture: amd64
targets: xfce
host: version 11151.59.0 (Official Build) stable-channel snappy
kernel: Linux localhost 4.4.159-15346-gc5e629939ae0 #1 SMP PREEMPT Tue Dec 11 00:13:48 PST 2018 x86_64 GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/kali-rolling...

#### Please describe your issue:
Hello, so I am trying to download Kali Linux. I've downloaded crouton and ran ' sudo sh -e ~/Downloads/crouton -r kali-rolling -t xfce' and I got the error: 'Installing CRAS...
/prepare.sh: 1166: /prepare.sh: cannot create /usr/share/alsa/alsa.conf.d/10-cras.conf: Directory nonexistent
Failed to complete chroot setup.
Unmounting /mnt/stateful_partition/crouton/chroots/kali-rolling...'
I have power washed my chromebook multiple times and I've kept trying again and I keep getting the same error message. I also know this issue has been solved in other sections, but I still can't seem to figure it out. And by the way I have an Intel chromebook.

#### If known, describe the steps to reproduce the issue:
Power wash my chromebook. Enable Developer Mode. Install crouton. Type shell. Run the code listed above.

**ALL CROSH CODE:**
`Preparing to unpack .../libasound2-dev_1.1.7-2_amd64.deb ...
Unpacking libasound2-dev:amd64 (1.1.7-2) ...
Setting up libasound2-dev:amd64 (1.1.7-2) ...
Reading package lists... Done
Building dependency tree
Reading state information... Done
patch is already the newest version (2.7.6-3).
patch set to manually installed.
The following packages were automatically installed and are no longer required:
  libasound2-dev:i386 libc6-dev:i386 libspeex-dev:i386 libspeex1:i386 libspeexdsp-dev:i386 linux-libc-dev:i386
Use 'apt autoremove' to remove them.
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Compiling CRAS (native)...
Installing CRAS...
/prepare.sh: 1166: /prepare.sh: cannot create /usr/share/alsa/alsa.conf.d/10-cras.conf: Directory nonexistent
Failed to complete chroot setup.
Unmounting /mnt/stateful_partition/crouton/chroots/kali-rolling...`

@fox-lucas,
See my reply here: https://github.com/dnschneid/crouton/issues/2333#issuecomment-450400332
I'm closing this issue since it's being tracked above.
-DennisLfromGA


Is there any way I can get custom image album name list from iOS?

CameraRoll.getPhotos(fetchParams)       .then((data) => this._appendImages(data, changeGallery), (e) => console.log(e));
I used above code and as a result, in data there's a parameter called 'group_name'.
In Android, I can get all the album names.
In iOS, all the group_name is same as 'All Photos'.
So I can sort albums in Android, but not in iOS.
How can I get custom image album name list...?
p.s. Last time I asked, 'DanielMSchmidt' closed my issue without asnwering;;;
I need a comment from an author!
Thanks a lot.

The issue tracker is meant to be used to track bugs that need to be fixed. "How do I do X?" types of questions are best handled in Stack Overflow, which is a purpose-built Q&A forum. Would you like to post this question to Stack Overflow with the tag #react-native?
As of this writing we have 789 open issues in the repository, so we need to be a bit more proactive about closing issues. I'm afraid I'll have to close this one out for now!

Okay I understood.
Thanks..

It seems like there's no way to do this with RN. We can get a list of images that are in albums:
CameraRoll.getPhotos({
  first: 10,
  groupTypes: 'Album',
})

Which is the first 10 images that are in albums, not the first image in the first 10 albums (which would be much more convenient). But we can't get a list of albums.
Seems like a feature request.

Feel free to post it to Product Pains as a feature request: https://productpains.com/product/react-native/ - it's been helpful when it comes to prioritizing what people work on next.

@DylanVann , @hramos
Gracias gentlemen...
for Android, I fixed CameraRoll as my taste and getting album list pretty well.
But since I strongly think this feature will be used a lot, I'll request on there for potential better work !
Have a good day !

@DylanVann @BROUDING did you guys find a workaround for this? I managed to get a list of albums using this module https://github.com/shimohq/react-native-albums getAlbumList method but the getImageList isn't implemented in the iOS version. I then tried to use the built-in CameraRoll.getPhotos with groupName set to the previously retrieved album name, but it doesn't match with what's acceptable:

Just leaving this here as that's as far as I dug it for now.


deprecated v2 yaml package

deprecated v2 yaml package in config_import.go, and in order to align project yaml package version

@CyanChan  Thanks for this PR!


added my name in team.md

Please resolve the merge conflict and remove the label when resoled. @Harsha-Ambati Thanks.


Embed functionality is missing

No embed code is being generated, nor are chart embeds available.

Need input on priority of date slider/adjustment; drastically reduces complexity to omit.


fix: substring issue with trailing wildcard

closes #2
I'll look at publishing a release tomorrow

Thx for fixing so fast :-)


🐛 BUG: Routes in wrangler.toml doesn't respect env

What version of Wrangler are you using?
0.0.17
What operating system are you using?
Mac, Linux
Describe the Bug
How to reproduce
Create different routes for production env in wrangler.toml file
routes = ["example-test.blabla.ch/*"]
[env.production]
routes = ["example.blabla.ch/*"]

execute
wrangler publish ./src/example.ts --env production --config ./wrangler.toml

Worker will be deployed to example-test.blabla.ch/* instead of example.blabla.ch/*.
This because here https://github.com/cloudflare/wrangler2/blob/9534c7fd1351daacaed63b3a3e2fafa884b515a8/packages/wrangler/src/publish.ts#L303 env params is not used at all (its used for vars, durable objects etc, but not for routes) see bindings above.

good catch, sorry about that! I'll fix this asap.

This is fixed in wrangler@alpha, we'll do a @beta release by the end of the week. Thanks for reporting this!


Make PubSubRouter a Startable to be started by the Service

Coverage decreased (-0.03%) to 74.654% when pulling 17ca018e1148f0b77fe666a7b9b3f055751df837 on refactor/guble-31-pubsubrouter-service into 3f6b47f0e97df278646d103d16b72e68f979fbd9 on master.

Good refactoring!
But now the conficts have to be resoved before merging.


use docker

we have a docker \o/ https://github.com/versatiles-org/versatiles-docker/pkgs/container/versatiles-tilemaker
Now we need to use it.


[Feature]: "Embargo status" is confusing

What would you like to see added to the NWB GUIDE?
I think the embargo status section of the form is a bit confusing.

Can we have a string next to the checkbox that says "embargo this Dandiset"
Can you make it so the NIH Award Number text field is inactive by default and becomes active when "embargo this Dandiset" is checked/inactive when unchecked? Also, when it is checked, please add a red asterisk indicating that this field is required.

Do you have any interest in helping implement the feature?
Yes.
Code of Conduct
Yes
Did you confirm this feature was not already reported?
Yes

@bendichter Is this resolved now?