Adding more options for Single Table Inheritance in Doctrine 2 Annotation

Hello,
here is my attempt to make Single Inheritance more customizable.
I had to make possible to force generation of array keys in MwbExporter/Object/Annotation.php in order to be able to specify integer discrimanator values. When defining php array keys as string representation of integer, php interprets those as integer, then Annotation::isKeysNumeric is not suficient to determine wether or not to write these keys. In this case (and when discriminator is an integer), discriminatorMap Annotation was not able to write keys, as specified by user. Not sure it was the best way to achieve this.

After further reading, I'm figuring out that it may be possible to make it entirely though single table inheritance (it seems I misunderstood the «SINGLE» word in thinking that hierarchy tree could only have a unique parent entity) :
BasePerson (@Entity, @InheritanceType("SINGLE_TABLE"), @DiscriminatorMap({"base_person" = "BasePerson", "person" = "Person", "base_man" = "BaseMan", "man" = "Man", "base_woman" = "BaseWoman", "woman" = "Woman"}))
Person extends BasePerson (@Entity)
BaseMan extends Person (@Entity) - BaseWoman extends Person (@Entity)
Man extends BaseMan (@Entity) - Woman extends BaseWoman (@Entity)
Maybe it should be more suitable and simple ?

how the annotation in the comment table should look in the BasePerson in order to obtain the
BasePerson (@Entity, @InheritanceType("SINGLE_TABLE"), @DiscriminatorMap({"base_person" = "BasePerson", "person" = "Person", "base_man" = "BaseMan", "man" = "Man", "base_woman" = "BaseWoman", "woman" = "Woman"})) ? Can you please share it ?

6 months later, I commited and rebased my enhancements to handle single inheritance while benefits from "base" classes. I added in the same time some functionalities I was needing while developping and testing.
See example/data/inheritance.mwb for the example. The extraction of resulting code can be found on a branch of mine (https://github.com/molaux/mysql-workbench-schema-exporter/blob/test-inheritance/test) in the test folder. Have a look at the config file mwb-exporter.test.json
Notes :

I had to make directories more customizable and so added the possibility to manage different directories for Entities, Ropositories and BaseEntities. For that purpose, the "dir" main config points now to the target bundle base directory (instead of entities one) : subdirectories for Entities, Ropositories and BaseEntities are now deduced from subnamespaces in configuration (entityNamespace, repositoryNamespace and baseEntityNamespace). With a lot of tables, it's much comfortable.
I also made base classes abstract as they should not exist in database. In the same idea, it's not referenced in discriminator map as 'base' anymore
I added few configuration options like overwriteExtendedEntities or generateSingleInheritance (this one generates a singleton discrimanator map even if there is no discriminator map in table comment and no extendable entity is configured) that are not yet documented
the default main entity discriminator is "extended" (to stay backward compatible) if not supplied through comment and if discriminator type is string
getters and setters involves now BaseEntities whereas doctrine annotations (relations, etc...) use leaves (Entities)
I did my best to follow conventions. Tell me if I'm still wrong.

To do:

Abstract "base" classes being not referenced in discriminator map anymore, it should be checked if single inheritance is still needed to achieve extendableEntities mode when no single inheritance is expected;
in my example, I tested adding getters and setters to Violonist and Violon child entity to see if it works as expected. In a next session I plan to generate it directly in Violonist and Violon base class by simply configuring foreign key comment in MWB;
at present time, single inheritance generates only a one level deep tree where leaves are all descedent from main entity. It would be great to be able to describe a more complexe tree through the table dicriminator comment; It is not a very hard work.

@malutanpetronel, sorry to answer your question so late but I had to finish this before, and so much other things even before. Have a look at my example (particularly, the comments on Tools, Persons, and Skills) and the README.md file.

thx... no hurry :) anytime an answer is good ;)

thx... no hurry :) anytime an answer is good ;)

The recent project refactor makes my pull request obsolete. No idea on how to maintain this...


[0.6/dx11] Bugfixes

Fixes compilation issues in release, errant warnings, and creation of DeferredContext1.
Waiting for @m4b to confirm if the context fix works.

Going ahead with this PR despite it not fixing the rendering bug as it doesn't make it worse.

bors retry

bors retry


ci: ensure default value is correct in workflow ui

Ensure that the default value showed in user interface remains
consistent with the slash command.
Closes: #5046

/ok-to-merge CI and doc only changes


Error after installing tensorflow

NVIDIA-Drive PX2
[Required] Details of the work you did before the problem occurred:
 Installed tensorflow, pip list |grep tensorflow shows the correct version
 I had set the environment variables:LD_LIBRARY_PATH  as /usr/local/cuda-9.2/lib64

[Required] Error message:
 ImportError: libcublas.so.9.0: cannot open shared object file: No such file or directory

[Required] Overview of problems and questions:

As it is displayed in the error message.
The version of CUDA is different.
If you want to use CUDA version 9.2, please build Tensorflow yourself.

Close because there is no reply for 2 months or more.


Support for TMSCHEMA tables to test SSAS tabular model

Currently Tabular model metadata is also getting verified against MDSCHEMA tables which are mainly meant for multidimensional model.
Tabular model metadata information is stored in TMSCHEMA tables which should be used to test the model.
Thanks

You're right but could you be a bit more specific on what's going not well with the current implementation?

Thanks for responding. Few scenarios where current implementation will not work:

In Tabular model to get mapping between Technical column name (Database field example FIRST_NM) and business column name (example - First Name), we need use TMSCHEMA_COLUMNS. Same information can't be retrieved by DBSCEHMA_COLUMNS.
To get perspective details in case of tabular model, we need use TMSCHEMA_PERSPECTIVE_TABLES and TMSCEHMA_PERSPECTIVE_COLUMNS table. Same can't be retrieved from MDSCHEMA_DIMENSIONS which I believe is getting used currently,
Please let me know if any other information/scenario is required.


Should menu and UI macros work?

Should these two features: menu:View[Zoom > Reset] and btn[label] display correctly in rendered jekyll html output?

There are two parts this. First is whether the macros themselves are processed. That's enabled by defining the experimental attribute in the Jekyll config (if you don't want have to set it per document). The second part is whether the resulting HTML is styled properly. That requires an addition to the stylesheet that you include with your site. You can snag these lines of CSS for a quick solution: https://github.com/asciidoctor/asciidoctor/blob/1cde78f91aca9b7c9ca542ad118f6a736a12c693/data/stylesheets/asciidoctor-default.css#L117-L128
It might be a good idea to include what I just wrote under this section in the README: https://github.com/asciidoctor/jekyll-asciidoc#circled-callout-numbers

I got this working with your suggestions. the formatting looks odd though, with down and right arrows between menu options:

Looks like there is some sort of conflict going on with existing styles in your site.

Thanks, I will investigate! 👀🤔


Jax hangs when using PositionalSharding and fft2.

Description
I would like to use positional sharding to batch some computation (involving fft2) across gpus. The program hangs at jax.numpy.fft.fft2. As a minimal example:
import jax
import jax.numpy as jnp
from jax.experimental import mesh_utils
from jax.sharding import PositionalSharding
key = jax.random.PRNGKey(0)
x = jax.random.normal(key, (8, 64, 64, 64, 2))
@jax.jit
def f(x):
    y = jnp.fft.fft2(x, axes=(0, 1))
    return y

n_devices = jax.local_device_count()
sharding = PositionalSharding(mesh_utils.create_device_mesh(n_devices,)).reshape((n_devices, 1, 1, 1, 1))
_x = jax.device_put(x, sharding)

# This works fine
print('Non-sharded', jax.vmap(f)(x).shape)

# This does not
jax.vmap(f)(_x)

Weirdly, the GPU utilization stays at 100% with no deviation.
What jax/jaxlib version are you using?
jax v0.4.14, jaxlib v0.4.14+cuda12.cudnn89
Which accelerator(s) are you using?
GPU
Additional system info
Python 3.10, Ubuntu 22.04.2 LTS, cuDNN v8.9.4
NVIDIA GPU info
+---------------------------------------------------------------------------------------+
| NVIDIA-SMI 530.30.02              Driver Version: 530.30.02    CUDA Version: 12.1     |
|-----------------------------------------+----------------------+----------------------+
| GPU  Name                  Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf            Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|                                         |                      |               MIG M. |
|=========================================+======================+======================|
|   0  NVIDIA RTX A6000                On | 00000000:4F:00.0 Off |                  Off |
| 30%   37C    P2               91W / 300W|   2855MiB / 49140MiB |    100%      Default |
|                                         |                      |                  N/A |
+-----------------------------------------+----------------------+----------------------+
|   1  NVIDIA RTX A6000                On | 00000000:52:00.0 Off |                  Off |
| 30%   42C    P2               98W / 300W|    915MiB / 49140MiB |    100%      Default |
|                                         |                      |                  N/A |
+-----------------------------------------+----------------------+----------------------+
|   2  NVIDIA RTX A6000                On | 00000000:53:00.0 Off |                  Off |
| 30%   40C    P2               76W / 300W|   4955MiB / 49140MiB |     39%      Default |
|                                         |                      |                  N/A |
+-----------------------------------------+----------------------+----------------------+
|   3  NVIDIA RTX A6000                On | 00000000:56:00.0 Off |                  Off |
| 33%   61C    P2              208W / 300W|  38695MiB / 49140MiB |     99%      Default |
|                                         |                      |                  N/A |
+-----------------------------------------+----------------------+----------------------+
|   4  NVIDIA RTX A6000                On | 00000000:57:00.0 Off |                  Off |
| 35%   65C    P2              211W / 300W|  38695MiB / 49140MiB |      7%      Default |
|                                         |                      |                  N/A |
+-----------------------------------------+----------------------+----------------------+
|   5  NVIDIA RTX A6000                On | 00000000:CE:00.0 Off |                  Off |
| 36%   64C    P2              200W / 300W|  38695MiB / 49140MiB |     99%      Default |
|                                         |                      |                  N/A |
+-----------------------------------------+----------------------+----------------------+
|   6  NVIDIA RTX A6000                On | 00000000:D1:00.0 Off |                  Off |
| 49%   73C    P2              287W / 300W|  37403MiB / 49140MiB |     94%      Default |
|                                         |                      |                  N/A |
+-----------------------------------------+----------------------+----------------------+
|   7  NVIDIA RTX A6000                On | 00000000:D2:00.0 Off |                  Off |
| 33%   63C    P2              188W / 300W|  48431MiB / 49140MiB |     99%      Default |
|                                         |                      |                  N/A |
+-----------------------------------------+----------------------+----------------------+
|   8  NVIDIA RTX A6000                On | 00000000:D5:00.0 Off |                  Off |
| 30%   26C    P8               13W / 300W|  10097MiB / 49140MiB |      0%      Default |
|                                         |                      |                  N/A |
+-----------------------------------------+----------------------+----------------------+
|   9  NVIDIA RTX A6000                On | 00000000:D6:00.0 Off |                  Off |
| 30%   29C    P8               20W / 300W|    663MiB / 49140MiB |      0%      Default |
|                                         |                      |                  N/A |
+-----------------------------------------+----------------------+----------------------+

If you trying to Shard your array across GPU/CPUor TPU or any other accelerator that you have the best and most efficient way to do is using pjit and I recommend you to use

import os
os.environ['XLA_PYTHON_CLIENT_PREALLOCATE'] = 'false'
os.environ['XLA_PYTHON_CLIENT_ALLOCATOR'] = 'gpu'

and if you want to use pjit the easiest and fastest method for your code is

import jax
import jax.numpy as jnp
from jax.experimental import mesh_utils, pjit

from jax.sharding import Mesh, PartitionSpec as PS
from fjutils import with_sharding_constraint  # smarter and more stable version
# or
from jax.experimental.pjit import with_sharding_constraint

key = jax.random.PRNGKey(0)
x = jax.random.normal(key, (8, 64, 64, 64, 2))

@jax.jit
def f(x):
    y = jnp.fft.fft2(x, axes=(0, 1))
    return y

n_devices = jax.local_device_count()
mesh = mesh_utils.create_device_mesh(
    jnp.ones((1, n_devices)).shape, ('dp', 'fsdp', 'mp')
)

with mesh:
    x = with_sharding_constraint(
        x, PS('fsdp')  # the array shape is (8, 64, 64, 64, 2) and you have 10 devices this won't work
        # I recommend to set visible devices to 8 or reshape the array like (n_devices,-1,2)
    )
    res = pjit.pjit(
        f,
        in_shardings=PS(),
        out_shardings=PS()
    )(x)
    print(x)

and the reason that this code of your
jax.vmap(f)(_x)

is not working because you have to pass the full array variable name to the func in case you are using the single machine I guess

Hi! Thanks for the response!
Sorry, I should clarify. We've ensured that when batching, the size of the large batch always evenly divides the number of devices (doing something like .reshape(n_devices, -1,...). Even when restricted to the case of 2 GPUs, the code I've listed in my original post fails.
I tried running the code that you've provided and have a few questions:
mesh = mesh_utils.create_device_mesh(
    jnp.ones((1, n_devices)).shape, ('dp', 'fsdp', 'mp')
)

This line produces the following error:
AttributeError: 'str' object has no attribute 'device_kind'

Q1: The documentation for create_device_mesh states lists the second argument as "devices: optionally, the devices to construct a mesh for. Defaults to jax.devices()." It seems like this argument should be the GPU or TPU object representing the accelerator? Can you elaborate on what the string args are doing?
Q2: Is there a difference between pjit and jit? The jax documentation has the line "NOTE: This function is now equivalent to jax.jit please use that instead" so I'm a little confused on why this would have different behavior vs. jit.
Q3: What does mesh as a context manager do in this case? Does it automatically apply jax.device_put to arrays created within the context block? When I try to use the resulting mesh as a context manager, an attribute error is thrown:
AttributeError: __enter__

Removing the context manager does throw the following error:
RuntimeError: pjit requires a non-empty mesh if you are passing `PartitionSpec`s or `None` to in_shardings!

Q4: You mentioned vmap(f)(_x) is not working because

you have to pass the full array variable name to the func in case you are using the single machine

I'm not sure I understand. _x should be sharded after using device_put (to my understanding from the tutorial notebook). I thought the vmap takes into account the input argument sharding?
Regardless, in order to make the code semi-run, I had to make a few changes:
import os
os.environ['CUDA_VISIBLE_DEVICES'] = '6,7' # Restricted to 2 GPUs for simplicity
os.environ['XLA_PYTHON_CLIENT_PREALLOCATE'] = 'false'

# Commented out - Including this line prevents jax from seeing the GPUs? Throws an error "falling back to CPU"
# os.environ['XLA_PYTHON_CLIENT_ALLOCATOR'] = 'gpu'

import jax
import jax.numpy as jnp
from jax.sharding import NamedSharding, PositionalSharding, Mesh
from jax.experimental import mesh_utils, pjit
from jax.sharding import Mesh, PartitionSpec as PS
from fjutils import with_sharding_constraint

n_devices = jax.local_device_count()
sharding = PositionalSharding(mesh_utils.create_device_mesh(n_devices,)).reshape((n_devices, 1, 1, 1, 1))
key = jax.random.PRNGKey(0)
x = jax.random.normal(key, (2, 64, 64, 64, 2))

@jax.jit
def f(x):
    y = jnp.fft.fft2(x, axes=(0, 1))
    return y

devices = mesh_utils.create_device_mesh(
    jnp.ones((1, n_devices, 1)).shape,
)
mesh = Mesh(devices, ('dp', 'fsdp', 'mp'))

with mesh:
    x = with_sharding_constraint(
        x, PS('fsdp')
    )
    res = pjit.pjit(
        f,
        in_shardings=PS(),
        out_shardings=PS()
    )(x)
    print(x)

Which results in the following error:
ValueError: Sharding passed to pjit does not match the sharding on the respective arg. Got pjit sharding: NamedSharding(mesh={'dp': 1, 'fsdp': 2, 'mp': 1}, spec=PartitionSpec()),
arg sharding: NamedSharding(mesh={'dp': 1, 'fsdp': 2, 'mp': 1}, spec=PartitionSpec('fsdp',)) for arg shape: (2, 64, 64, 64, 2)

Any advice is appreciated. Thanks!

sorry for wrong code this one will work
import os

os.environ['CUDA_VISIBLE_DEVICES'] = '6,7'  # Restricted to 2 GPUs for simplicity
os.environ['XLA_PYTHON_CLIENT_PREALLOCATE'] = 'false'

# Commented out - Including this line prevents jax from seeing the GPUs? Throws an error "falling back to CPU"
# os.environ['XLA_PYTHON_CLIENT_ALLOCATOR'] = 'gpu'

import jax
import jax.numpy as jnp
from jax.sharding import NamedSharding, PositionalSharding, Mesh
from jax.experimental import mesh_utils, pjit
from jax.sharding import Mesh, PartitionSpec as PS
from fjutils import with_sharding_constraint

def main():
    n_devices = jax.local_device_count()
    sharding = PositionalSharding(mesh_utils.create_device_mesh(n_devices, )).reshape((n_devices, 1, 1, 1, 1))
    key = jax.random.PRNGKey(0)
    x = jax.random.normal(key, (2, 64, 64, 64, 2))

    devices = mesh_utils.create_device_mesh(
        jnp.ones((1, n_devices, 1)).shape,
    )
    mesh = Mesh(devices, ('dp', 'fsdp', 'mp'))

    with mesh:
        def f(x):
            x = with_sharding_constraint(
                x, PS('fsdp')
            )
            y = jnp.fft.fft2(x, axes=(0, 1))
            return y

        res = pjit.pjit(
            f,
            in_shardings=PS(),
            out_shardings=PS()
        )(x)
        print(res)

if __name__ == "__main__":
    main()

Q1 Answer
for using jax.devices() you have to pass the backend name that  you want to get the list of device of that for example in your case you have 10 RTX A6000 if you use jax.devices('gpu') you will get list of your gpus their position and ... but if you try to get tpu it will raise an error cause you don't have any tpu on your machine jax.devices('tpu') and if you want to have a use case example imaging you want to use offload method in flax to load params on CPU and then shard them across all of the devices you can use

with jax.default_device(jax.devices('cpu')[0]):
    .... # load your params here

Q2 Answer
Yes, there is a difference between pjit and jit. pjit is a deprecated function that was used to compile functions for parallel execution on multiple devices. jit is the newer function that should be used instead.
The main difference between pjit and jit is that pjit requires the user to specify the sharding of the input and output arguments. Sharding is a way of dividing up a computation across multiple devices. The sharding of an argument specifies how many times it should be replicated across the devices.
For example, if you have a function that takes an input array of size 100 and you want to run it on 2 devices, you would need to specify the sharding of the input array as (2, 50). This means that the input array would be replicated 2 times across the devices, with each device getting a slice of size 50.
The sharding of the output arguments is also important because it determines how the outputs of the function are distributed across the devices. If you do not specify the sharding of the output arguments, then jit will try to infer the sharding automatically. However, this may not always be possible, and it can lead to suboptimal performance.
For these reasons, it is generally recommended to use jit instead of pjit. jit is more flexible and easier to use, and it does not require the user to specify the sharding of the input and output arguments.
also keep in mind that pjit is still in the experimental state
Q3 Answer
pjit required to be used in mesh

Hi, thanks for your response! The code that you provided still does not run. Crucially, it hangs on lines:
        res = pjit.pjit(
            f,
            in_shardings=PS(),
            out_shardings=PS()
        )(x)

And never prints res. Similar to the script that I posted, the GPU utilization peaks at 100% and hangs. Even giving it sufficient time to complete (~1hr) yields no difference. Do you have a suggestion for how I would go about finding the source of the hang?

Hi, thanks for your response! The code that you provided still does not run. Crucially, it hangs on lines:
        res = pjit.pjit(
            f,
            in_shardings=PS(),
            out_shardings=PS()
        )(x)

And never prints res. Similar to the script that I posted, the GPU utilization peaks at 100% and hangs. Even giving it sufficient time to complete (~1hr) yields no difference. Do you have a suggestion for how I would go about finding the source of the hang?

actually i ran this code twice on Kaggle TPUs and GPUs and the code worked on both of them
and this was output
[[[[[ 3.24833870e+00+0.00000000e+00j -1.67170668e+00+0.00000000e+00j]
    [ 1.13448925e+01+0.00000000e+00j -1.56441584e+01+0.00000000e+00j]
    [-2.06614799e+01+0.00000000e+00j -4.72019482e+00+0.00000000e+00j]
    ...
    [-8.73501062e-01+0.00000000e+00j  1.61715186e+00+0.00000000e+00j]
    [-6.35988188e+00+0.00000000e+00j  4.83073378e+00+0.00000000e+00j]
    [ 6.58679754e-02+0.00000000e+00j  1.09692688e+01+0.00000000e+00j]]

   [[ 1.50113926e+01+0.00000000e+00j  1.85722128e-01+0.00000000e+00j]
    [ 1.44338236e+01+0.00000000e+00j  1.20921860e+01+0.00000000e+00j]
    [-2.31441736e+00+0.00000000e+00j -2.15576148e+00+0.00000000e+00j]
    ...
    [-5.78798234e-01+0.00000000e+00j -1.50790095e+00+0.00000000e+00j]
    [-8.80465508e+00+0.00000000e+00j  2.43676052e+01+0.00000000e+00j]
    [ 1.34009323e+01+0.00000000e+00j  2.30402994e+00+0.00000000e+00j]]

This seems fixed to me with jax/jaxlib 0.4.20. Closing, on the belief that it's fixed. If it's still a problem with an up to date JAX, leave a comment and I'll reopen the issue. Hope that helps!


datapath: add controller manage to monitor cilium iptabels rules

Fixes #8856

This change is

Coverage decreased (-0.02%) to 45.813% when pulling 3efe5bdc9fdaf4e461207c7537f90de228e814a6 on fristonio:fristonio/iptables-manager-fix into 70c2eae518329ddc5f1b6e44fe416cf8b54700ba on cilium:master.

@fristonio Great work so far! One suggestion - perhaps you could use iptables-save output to check for rules. That would allow us to avoid multiple exec's for iptables -C, also then you could check that order of the rules is correct.

Hey @brb, yeah that makes sense. I will update the Pull Request.

Can this PR be re-opened and merged?

Can this PR be re-opened and merged?

@mskrocki I suggest to discuss this either in the community meeting or in the #development channel.


[Release-1.26] E2E Domain Drone Cleanup

Backport https://github.com/k3s-io/k3s/pull/8579

Cleanup inactive vm domains
Have e2e depend on amd64 pipeline

Linked Issues
https://github.com/k3s-io/k3s/issues/8379

User-Facing Change

Further Comments

S390x is having issues, all other tests are passing. Merging.


Web/lpa fixes

Describe your changes
Case officer bug fixes for web.
Issue ticket number and link
https://pins-ds.atlassian.net/browse/BOCM-423
https://pins-ds.atlassian.net/browse/BOCM-425
https://pins-ds.atlassian.net/browse/BOCM-426
Type of change 🧩

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Other (please explain in the description section above)

Checklist before requesting a review

[x] I have performed a self-review of my own code
[x] I have double checked this work does not include any hardcoded secrets or passwords
[ ] I have made corresponding changes to the documentation
[ ] I have provided details on how I have tested my code
[x] I have referenced the ticket number above
[ ] I have provided a description of the ticket
[ ] I have included unit tests to cover any testable code changes

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

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

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

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

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


Cannot use Local or Both validation modes when embedding IDS3

When using a ValidationModel.Local or ValidationMode.Both. Per the docs the discovery endpoint is hit to look up the IssuerName and SigningCertificate.
However, when hosting IDS3 in the same Katana app the discovery endpoint (obviously) isn't available. UseIdentityServerBearerTokenAuthentication throws:

An exception of type 'System.InvalidOperationException' occurred in IdentityServer3.AccessTokenValidation.dll but was not handled in user code

Additional information: IDX10803: Unable to create to obtain configuration from: 'http://localhost:14869/identity/.well-known/openid-configuration'.

I understand this probably isn't a common use-case, but is a valid one? Small repro (modified Simplest OAuth2 Walkthrough sample): https://gist.github.com/zsims/f15b11de7b96c61f2ef5

you need to set issuername and cert manually on the validation middleware in that case.

Thanks for the reply, docs so hint at this. I can raise a PR to clarify if it helps? If not feel free to close this :smile:

Yes do that please
Sent from my iPhone

On 13.11.2015, at 09:58, Zachary Sims<EMAIL_ADDRESS>wrote:
Thanks for the reply, docs so hint at this. I can raise a PR to clarify if it helps? If not feel free to close this
—
Reply to this email directly or view it on GitHub.

Noticed you did this in https://github.com/IdentityServer/Documentation/commit/fc0ed3a7194b4ec0bf024f6d210ddd2dce59f43a
Thanks for the update!


Add a license to the repository

I recently came across your project surface-code-generator and I noticed that the repository github.com/alexnguyenn/surface-code-generator currently does not have a license specified.
Why Adding a License is Important?
As GitHub documents

Public repositories on GitHub are often used to share open source software. For your repository to truly be open source, you'll need to license it so that others are free to use, change, and distribute the software.

Would be possible to add a valid license to this repository? Probably something really OpenSource such as the Apache 2.0 license would fit really well.
Thanks!

Thanks!


swtpm: store state in tmpfs

swtpm stores state in the directory specified in the --tpmstate argument
If the same state directory/files are used for multiple installations, the available space can be filled, and the installer will fail with the error "insufficient space for NV allocation".
Move swtpm state to tmpfs to create new state files every run.
Change-type: patch

I self-certify!


[help] can not load techbench module ?

Error message
❯ .\redbean-build.com -vvmbag
D2022-06-07T17:39:50.627197:tool/net/redbean.c:5341:redbean-build:8132] (lua) LuaRun("/.init.lua")
E2022-06-07T17:39:50+000145:tool/net/redbean.c:1034:redbean-build:8132] (lua) failed to run lua code: /.init.lua:1: module 'techbench' not found:
        no field package.preload['techbench']
        no file 'E:\scoop\global\apps\lua\current'
        no file 'E:\scoop\global\apps\lua\current'

zip -sf redbean-build.com
Archive contains:
  usr/share/ssl/root/
  usr/share/ssl/root/amazon.pem
  usr/share/ssl/root/certum.pem
  usr/share/ssl/root/comodo.pem
  usr/share/ssl/root/digicert.pem
  usr/share/ssl/root/geotrust.pem
  usr/share/ssl/root/globalsign.pem
  usr/share/ssl/root/godaddy.pem
  usr/share/ssl/root/google.pem
  usr/share/ssl/root/isrg.pem
  usr/share/ssl/root/quovadis.pem
  usr/share/ssl/root/redbean.pem
  usr/share/ssl/root/starfield.pem
  usr/share/ssl/root/verisign.pem
  usr/share/zoneinfo/
  usr/share/zoneinfo/Beijing
  usr/share/zoneinfo/Berlin
  usr/share/zoneinfo/Boulder
  usr/share/zoneinfo/Chicago
  usr/share/zoneinfo/GST
  usr/share/zoneinfo/Honolulu
  usr/share/zoneinfo/Israel
  usr/share/zoneinfo/Japan
  usr/share/zoneinfo/London
  usr/share/zoneinfo/Melbourne
  usr/share/zoneinfo/New_York
  usr/share/zoneinfo/Singapore
  .ape
  help.txt
  .init.lua // ------------------------------------- added
  favicon.ico
  redbean.png
  .lua/
  .lua/fullmoon.lua // -------------------------- added
  .lua/techbench.lua  // ------------------------ added
Total 35 entries (255284 bytes)

unzip -lc .\redbean-build.com .init.lua
Archive:  ./redbean-build.com
 extracting: .init.lua
require "techbench"  // -------------------- .init.lua content

@lost4git, everything looks correct in your project setup, but for some reason redbean is not looking in any of the /zip/ folders. This is how the same message looks for me:
E2022-06-07T18:38:56+001281:tool/net/redbean.c:1043:cosmobin:13188] (lua) failed to run lua code: /.init.lua:1: module 'techbench' not found:
        no field package.preload['techbench']
        no file '/zip/.lua/techbench.lua'
        no file '/zip/.lua/techbench/init.lua'
        no file '/zip/.lua/techbench.so'
        no file '/zip/.lua/loadall.so'
        no file './techbench.so'
stack traceback:
        [C]: in function 'require'
        /.init.lua:1: in main chunk

Notice that in my case it checks /zip/.lua/techbench.lua, which is where the file is in your case, but in your stacktrace these paths are not listed. Did you build your redbean copy yourself?

My build script file (powershell)

build.ps1
$ErrorActionPreference = "Stop"

$redbeanUri = "https://justine.lol/redbean/redbean-latest.com"
$redbeanPath = "./redbean.com"
$redbeanBuildPath = "./redbean-build.com"
$luaPath = "./.lua"
$initLuaPath = "./.init.lua"

if(Test-Path $redbeanBuildPath)
{
  Remove-Item $redbeanBuildPath
}

"Checking redbean.com whether exists..."
if(-not $(Test-Path $redbeanPath))
{
  "Downloading redbean.com from $redbeanUri to $redbeanPath"
  Invoke-WebRequest -Uri $redbeanUri -OutFile $redbeanPath
}

"Creating ./.lua folder..."
if($(Test-Path $luaPath))
{
  Remove-Item -Recurse $luaPath
}
mkdir $luaPath

"Copying fullmoon.lua to ./.lua folder..."
Copy-Item "./fullmoon.lua" "$luaPath/fullmoon.lua"

"Copying techbench.lua to ./.lua folder..."
Copy-Item "./examples/techbench.lua" "$luaPath/techbench.lua"

"Generating .init.lua..."
Set-Content -Value "require 'techbench'" -Path $initLuaPath

Write-Host "> Whether to package the resources to redbean.com and start readbean.com (Y/N): " -ForegroundColor Yellow -NoNewline
$toPackage = Read-Host
if(($toPackage -ne "Y") -and ($toPackage -ne "y"))
{
  return
}
"Packaging to redbean.com..."
Copy-Item $redbeanPath $redbeanBuildPath
zip  $redbeanBuildPath "$luaPath/fullmoon.lua" "$luaPath/techbench.lua"
zip $redbeanBuildPath $initLuaPath

"Starting redbean.com..."
. $redbeanBuildPath -vvmbag

I suspect it's the same issue as #2, as you appear to be using redbean 1.5. I'd expect this to be fixed when redbean 2.0 is out (or if you build redbean with any version 1.6+).

@lost4git, can you re-test with the current redbean version: https://justine.lol/redbean/redbean-latest.com? I expect the issue to be resolved.

@pkulchenko
I found that it is already v2.0, but the problem still exists 😂
D2022-06-14T00:00:45.102047:tool/net/redbean.c:4922:redbean-build:1084] (lua) LuaRunAsset("/.init.lua")
E2022-06-14T00:00:45+000000:tool/net/redbean.c:1047:redbean-build:1084] (lua) failed to run lua code: /.init.lua:1: module 'techbench' not found:
        no field package.preload['techbench']
        no file '/E/scoop/global/apps/lua/current'  // still try to load module from my local lua home?
        no file '/E/scoop/global/apps/lua/current'
stack traceback:
        [C]: in function 'require'
        /.init.lua:1: in main chunk

redbean-build.zip
my build script output

There may be an issue with your build script, try this, (note I use: bash -c './redbean.com -v', because I am using zsh.)
git clone git@github.com:pkulchenko/fullmoon.git \
  && cd fullmoon \
  && curl https://redbean.dev/redbean-latest.com >redbean.com \
  && chmod +x redbean.com \
  && mkdir .lua \
  && cp fullmoon.lua .lua \
  && cp examples/techbench.lua .lua \
  && echo 'require "techbench"' >.init.lua \
  && zip redbean.com .init.lua .lua/fullmoon.lua .lua/techbench.lua \
  && bash -c './redbean.com -v'

then to confirm working:
curl http://<IP_ADDRESS>:8080/json

@imaitland same problem in WSL 😂

I can't reproduce this and the script that @imaitland posted works for me.
I don't understand where /E/scoop/global/apps... path is coming from and why there is no /zip/... paths that should be added by default: https://github.com/jart/cosmopolitan/blob/master/tool/net/redbean.c#L5199.
In any case, it doesn't seem like it's a fullmoon issue, as your copy of redbean for some reason doesn't set proper lua paths, so the modules are not loaded. I'm a bit at a loss here. Can you run redbean with --ftrace option and check if LuaStart and GetDefaultLuaPath functions are on the list?

it is work for me now 😂

Thanks to @pkulchenko  for your patient answer

Maybe redbean needs to modify the priority of the lua path

Yes, we'll have to check on that in redbean; thank you for the update!


Enable Depositor withdrawal to LP

At the moment Depositor withdraws to itself. To avoid an extra tick, make sure it withdraws to the LP contract directly.

Addressed in https://github.com/timewave-computer/covenants/pull/18


Need help :)

Sorry, after done setup, I tried to register a user but it asks for 4 digit invitation code. Do I need to figure it out myself or there is actually 4 digit code stated? Thanks!

The invitation code can be configured in app/config.php. The default code is 3702 :-)

Thank you for the fast response. :)

Sir without know default code, how to bypass this section


CMIS_STRING

does not exist in 702
https://abapedia.org/steampunk-2302-api/cmis_string.dtel.html
just use the builtin type instead?

ah yes thanks, just fixed it:


Ban API does not show if a permaban is revoked or if a ban is explicitly expired.

unbanned_datetime isn't ever polled for permanent bans, meaning that an in-effect permanent ban and a revoked one are indistinguishable.

@qwertyquerty


bugfix: add presence check for childResourceType

This PR resolves an issue where sometimes navigating within the GUI causes a blocking error with .fields being undefined.
How to run locally ?
In the project where moquerie is being used update the npm script as follows to point towards your fork
    "start-moquerie": "node /home/user/Projects/github.com/project-name/moquerie/packages/moquerie/bin.mjs",

PR now targets upstream


[BUG] PlasmoMessaging Request type is never

What happened?
Typescript throws an error for this code whenever you use the sendToBackground function from @plasmohq/messaging.
It shows this: PlasmoMessaging.Request<never, {}>

Not sure where the fix of this bug should come in, but would be happy to raise a PR

Version
Latest
What OS are you seeing the problem on?
Windows
What browsers are you seeing the problem on?
No response
Relevant log output
No response
(OPTIONAL) Contribution

[X] I would like to fix this BUG via a PR

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

Try to restart the typescript server and also restart the dev

Try to restart the typescript server and also restart the dev

thanks will try that

Yea, this happens when you haven't re-built the extension, bc it generates new types.

Yea, this happens when you haven't re-built the extension, bc it generates new types.

@rahulraikwar00 tip helped. Thanks


Make Editer support touch select text

Issue Type: Feature Request
Please make Edter support touch screen selection text
VS Code version: Code 1.37.1 (f06011ac164ae4dc8e753a3fe7f9549844d15e35, 2019-08-15T16:17:55.855Z)
OS version: Windows_NT x64 10.0.17763

/duplicate of #47770


ActionSheets

I read in the readme that PR with alternatives to ActionSheets are welcome.  What did you all have in mind?  Why aren't ActionSheets supported?  Happy to lend a hand.

Thank you for asking.
cc @aksonov.

@wootwoot1234 You may check existing Modal, TabBar and do the same for ActionSheet.

no actionSheet in 3.x ?

You can use this https://github.com/exponentjs/react-native-action-sheet

@aksonov ActionSheets should not be a responsability of this package

@sibeliusseraphini this makes sense. My App uses both RNRF and Expoent's react-native-action-sheet with no issues. Benefit of this setup is that is compatible with Both IOS and Android.
@wootwoot1234 PR's are still welcome, although I'm not sure if this will be a duplicated effort considering that Expoentjs/react-native-action-sheet works with RNRF

@rturk I was under the impression it didn't work.  @aksonov why does it say it doesn't work in the readme?  Glad to hear it does.

Maybe here is some misunderstanding. Sure any component will work with RNRF, but it would be great to have something like Actions.sheet({...}) and see actionsheet, without wrapping anything like exponent's requires. version 2 allowed such easy usage, v3 doesn't.

Ok, yeah.  There was some misunderstanding on my part.  Can you update the readme so that's clearer?  I would say something like:
"No built in Action Sheet like v2.  If you want to use Action Sheets you'll need to use a 3rd party module."

@wootwoot1234 please submit a PR to update the readme, please also add a link to react-native-action-sheet there.
I think we can close this issue cc @rturk

@rturk Can you briefly show how you got react-native-action-sheet to work with RNRF? When trying to integrate react-native-action-sheet with RNRF, I get errors, on PropTypes not being defined, and it goes away when I add PropTypes to the react-native imports, but another error on func, "Cannot read property 'func' of undefined"
I followed their docs, and when it comes to the part where you wrap your app:
render() {
    return (
      <ActionSheet ref={component => this._actionSheetRef = component}>
        // Render the rest of your app here.
      </ActionSheet>
    );
  }

I wrapped the Router with all its scenes like this:
render() {
    return (
      <ActionSheet ref={component => this._actionSheetRef = component}>
        // Render the rest of your app here.
       <Router createReducer={reducerCreate} getSceneStyle={getSceneStyle}>
         // all scenes here
       </Router>
      </ActionSheet>
    );
  }

But still getting the issue "Cannot read property 'func' of undefined" in the actual component where I am trying to use the action sheet. The component uses exactly the react-native-action-sheet method of integration their documentation showed. I think the issue is related to an incorrect integration by me with RNRF wrapping the app part.
Can you please share how you integrated it correctly?


fix(material-experimental/mdc-slider): throw error when thumb is missing

Adds a proper error when the MDC slider is missing a thumb. Currently we eventually hit a runtime error which isn't easy to debug.
Fixes #24057.

I didn't add one since the other validation logic didn't have test either.


problem with Russian language

When trying to import excel file just in routes.php I got an array with abracadabra.
It seems that there is a problem with Russian language
Do you know how to fix it?
Route::get('xls', function ()
{
Excel::load('app/database/xls/categories.xlsx', function($reader)
{
$result=$reader->select(array('id', 'parent_id','level','title','sef','body',
'image','meta_title','keywords','description'))->get();
var_dump($result);
})->get();
}
);

I think this has to do with the dump rather than the package. I think we have had a similiar issue with Chinese characters before. Try echo'ing <meta charset="YOUR CHARSET"> on top of the routes.php file, that fixed it for dumping the data back then.

You are right.
echo ''; just before the dump solved the issue
thanks.


[Diagnostics]: Add diagnostics for incorrect property wrapper references in function arguments

Adds recording of UsePropertyWrapper and UseWrappedValue fixes in function arguments. Additionally refactors a lot of the logic for determining the correct diagnostic away from the diagnostic site and into fixPropertyWrapperFailure which gets used both here, and in the same diagnostic for member references.

Resolves SR-10928.

@swift-ci please smoke test

@swift-ci please test source compatibility

@swift-ci please test source compatibility

@swift-ci please smoke test

I'll make those final changes and get them pushed as soon as I have access to WiFi 👍

@swift-ci please smoke test

@swift-ci please smoke test


one issue when execute terraform init script

[opc@console-1 oke-terraform]$ terraform init
Initializing modules...

auth in modules/auth
base in modules/base
base.bastion in modules/base/bastion
base.vcn in modules/base/vcn
network in modules/okenetwork
oke in modules/oke
There are some problems with the configuration, described below.

The Terraform configuration must be valid before initialization so that
Terraform can determine which modules and providers need to be installed.
Error: Missing key/value separator
on modules/okenetwork/security.tf line 57, in resource "oci_core_security_list" "workers_seclist":
51:
52:
53:
54:
55:
57:       tcp_options {
Expected an equals sign ("=") to mark the beginning of the attribute value.

Looks like you're using TF 0.12. This not supported yet.


Changed the bootstrapper selector to show script type instead of platform.

This looks good to me! :-)


Add Swiss German

Adding Swiss German so I can pick my native tongue on our company internal profile.
Code see https://en.wikipedia.org/w/index.php?title=ISO_639:gsw

@adlawson any chance we could get eyes on this? Thank you for your work on this package!


access log filters: various enhancements

Implement LE comparison filter.
Runtime filter additions (configurable default, divisor, and independent
randomness).

Risk Level: Low
Testing: Unit
Docs Changes: https://github.com/envoyproxy/data-plane-api/pull/490
API Changes: https://github.com/envoyproxy/data-plane-api/pull/490
Release Notes: Added
Fixes https://github.com/envoyproxy/envoy/issues/2570

@dnoe @danielhochman updated


Bug: Chapter 6 "fashionMNIST_foolbox_boundary.ipynb"

Main issue
I think the code was written with an older version of foolbox or tensorflow. Because while running
cell
x_adv = attack(input_or_adv=x,
               label = y,
               starting_point = starting_point_image,
               unpack = False,
               iterations = 2000,
               log_every_n_steps = 500)

This error arises:
TypeError  Traceback (most recent call last)
<ipython-input-20-f0698dcd1aa3> in <module>()
      4                unpack = False,
      5                iterations = 2000,
----> 6                log_every_n_steps = 500)

TypeError: __call__() missing 2 required positional arguments: 'inputs' and 'labels'

I tried renaming input_or_adv  to inputsand label to np.ndarray([y]) but it doesnt work. the labels lengths and inputs length should be equal.
here is a link of what i had done on google colab
Colab link

Hi @JafarBadour
I haven't been able to recreate this problem. Please could you let me know your versions of tensorflow and  FoolBox? They should be as per the requirements.txt file.
Many thanks,
Katy

I have solved it by making virtual environment and installing older version of tensorflow ... problem was because of new version. Thank you for reply.


linted existing pixels

I've already added a pixel, but there were issues caused because someone submitted a pull request without running the test and didn't lint the file. I'm attempting to correct that with this pull.

Thank you for the help! The changes have been fixed in the meantime, so I'll close this one but the help is appreciated :)


local import statements

Since it doesn't look like #5763 is going to be solved anytime soon, and since users are starting to run into problems writing cross-platform modules with npm dependencies, I'm experimenting with allowing import statements to be nested inside a block.
void useCeylonCollections() {
    import ceylon.collection {
        HashMap,
        ArrayList
    }
    HashMap();
    ArrayList();
}

void useJavaCollections() {
    import java.util {
        HashMap,
        ArrayList
    }
    HashMap();
    ArrayList();
}

It turns out that this has been super-simple to implement and has zero impact on the backends as far as I can discern. And it does solve the problem of not being able to have an import scoped with native, as well as being orthogonally-useful.
So I'm inclined to put the finishing touches on this and release it as part of 1.3.1. WDYT?

I think this is a good enough solution for most cases. +1 from me.

I have several questions related to IDE support:

does code folding still work with nested import lists?
should code completion add new imports to the nested list instead of the toplevel list, if there is one?
would it make sense to have a quick assist that moves an import from a nested list to an "outer" list?

does code folding still work with nested import lists?

Yep, seems to be working.

should code completion add new imports to the nested list instead of the toplevel list, if there is one?

If there is already an import statement for the package, then it should add it to the existing import, wherever it is. Otherwise it should add a new import statement to the toplevel list.
Surprisingly, this already seems to work.

would it make sense to have a quick assist that moves an import from a nested list to an "outer" list?

Rrm, shrug, I guess, but it's not that crucial.
What I have noticed with respect to IDE support is that autocompletion seems to add duplicate imports to nested import statements. We need to fix that.

As of my latest commit, this seems to be working nicely. I still need:

tests
spec

This kinda reminds me an old proposal that never became true: fully qualified references.
Something like:
foo.bar::Baz calcBaz() {
    return foo.bar::Baz();
}

I recall Gavin had no objections, and maybe still useful.

@someth2say it can be done, but it's always looked pretty ugly to me. I prefer local imports personally.

I've always hated to find local imports in other languages like Scala or Perl, for no particular rational reason. Other than perhaps that I don't find it tidy, and people abuse it.
OTOH it's a pretty cheap solution for the problem at hand, even though I'd prefer annotating imports at the top. The only down-side is having to repeat those imports in every native block rather than have them apply to the unit as a whole.
I'm curious why annotating imports was harder/worse? I didn't glean that from looking at the other issue.

@someth2say it can be done, but it's always looked pretty ugly to me. I prefer local imports personally.

Qualified types are nice if you have a single one of them. If you start having more than 2 at the same place, I agree a local import (or an aliased global import) is better.

I'm curious why annotating imports was harder/worse?

Well it's a much bigger, more impactful change. I mean, currently we can't even represent a package import in an annotation constraint, much less read it from the metamodel, since there's just no metamodel-level representation of these things.
I mean, sure, we could hack them into the grammar as this totally special case thing that only DeclarationVisitor understands but .... yew.

And the other thing is that apart from native, I have never seen any other single usecase for annotating a package import.

Hell, package imports don't even belong to something with a representation in the metamodel: there is no metamodel for a Unit.

Why do we even have imports in units? Just do an implicit import foo { ... } on every module import in the descriptor! :D ducks and runs

So to summarise:

No annotation constraint for import annotations: we could make a special-case metamodel declaration type that we can't read (dubious),
No way to read them from metamodel: well, that'd be our first compile-time-only annotation I suppose, the metamodel would not read them, there's no point,
No other single usecase for annotating a package import (super special).

Yeah, the import annotations are harder to do and harder to justify. I just hope people won't abuse local imports.

I just hope people won't abuse local imports.

I think we can revisit the decision if we see they do. As a feature in and of itself I think it's nice to have and if at the same time it fixes this problem for very little effort then that's certainly an added advantage.

I just hope people won't abuse local imports.

My guess is that since imports are generally written automatically by the IDE, and not by hand, then that's not likely. But if it really turns out to be a problem in practice, we could add a compiler warning for usage that we consider "abuse", that is, use of a local import for a name that:

is not coming from a native module import, and
is not importing a name that is already imported somewhere else.

It should be pretty easy to determine that an import could be safely moved to the toplevel and warn, it seems to me.

Untested, but I suspect the backends will need to account for this regarding imports of members of top-level objects. See #5892 and the fix that searches expr.getUnit().getImports().

@jvasileff it seems to work for me, with both the Java and JS backends.

Ok, but there's a trick: the member has to be inherited for it not to work.
This fails on three backends (add to a module named simple):
shared void run() {
    import simple {
        obj { xval = xval }
    }
    print(xval);
}

interface X {
    shared default String xval => "x";
}
object obj satisfies X {}

Should there be more restrictions? The code below works, but the ordering and nesting of nodes in the AST doesn't match the source code.
shared void run() {
    b();
    import simple { f = foo }
    f();
    import simple { b = bar }
}
void foo() => print("foo");
void bar() => print("bar");

Duplicate import aliases aren't being detected. For example:
import ceylon.language { parse = parseFloat }
shared void run() {
    import ceylon.language { parse = parseInteger }
    Integer? i = parse("1.0");
}

@jvasileff I don't especially see why that should be disallowed. You're allowed to hide names in outer scopes with a local import.

Makes sense.

I have updated the spec, so now I'm closing this issue.


It should be possible to flexibly exclude projects and plugins from the build

Please use https://discuss.lightbend.com/c/tooling including a specific user story instead of posting them to the issue tracker.
Right now SBT in very inconvenient for big monorepos.
For example, I may have ten software components but at some point I may wish to work with just couple of them.
At this point there is no easy to exclude several subprojects from the project. In some cases projectmatrix/siracha can help. But things are getting worse in case we have a cross-build wish to build all the artifacts of a component for one of the platforms. One of the cases is when we use IntelliJ -- it has a lot of bugs in ScalaJS support, at this point it can't build any projects referencing sbt-crossprojects. And there are other similar cases.
It would be great if we can add tags to projects and then specify exclusion rules in commandline, for example
sbt --exclude "(platform:javascript && tagname) || platform:native"

More details about my motivation

Please use https://discuss.lightbend.com/c/tooling including a specific user story instead of posting them to the issue tracker.

umm. Let's discuss on Discuss.


Issue in installation process

I could not install this package (1.14) using the instructions.
Running composer require laravel/cashier-mollie succesfully installs the required dependencies, but then at the end, the following is automatically run:
> Illuminate\Foundation\ComposerScripts::postAutoloadDump
> @php artisan package:discover --ansi

   Mollie\Api\Exceptions\ApiException

  Invalid API key: ''. An API key must start with 'test_' or 'live_' and must be at least 30 characters long.

  at ...\vendor\mollie\mollie-api-php\src\MollieApiClient.php:353
    349|     {
    350|         $apiKey = trim($apiKey);
    351|
    352|         if (!preg_match('/^(live|test)_\w{30,}$/', $apiKey)) {
  > 353|             throw new ApiException("Invalid API key: '{$apiKey}'. An API key must start with 'test_' or 'live_' and must be at least 30 characters long.");
    354|         }
    355|
    356|         $this->apiKey = $apiKey;
    357|         $this->oauthAccess = false;

  1   ...\vendor\mollie\laravel-mollie\src\Wrappers\MollieApiWrapper.php:92
      Mollie\Api\MollieApiClient::setApiKey("")

  2   ...\vendor\mollie\laravel-mollie\src\Wrappers\MollieApiWrapper.php:67
      Mollie\Laravel\Wrappers\MollieApiWrapper::setApiKey()

So then I set the API key in .env (and running php artisan config:cache), and retried the installation. I still got the same issue. I don't know, maybe this is something due to my own installation, ... in any case, I could finally circumvent the issue by removing bootstrap/cache/config.php.
Even if that's an issue on my end, I'd say there's at least a documentation issue, because the mollie key should be set before even trying to install the package. Alternatively, that kind of initiatlisation or check should be delayed until a later time?

Did you solve it? I am facing the same issue

Hi @maksudur-rhmn, you can circumvent it like this:

First set the MOLLIE_KEY env variable
Then ensure that there is bootstrap/cache/config.php file (you can delete it) if it's there
Try installing this module again

Another workaround, I figured out later, is to edit the "scripts" section at the bottom of composerjson, and temporarily remove the "post-autoload-dump" there. You'll want to put it back after installing this package though, and once everything is setup and working, run composer dump-autoload to make sure everything is in order.

If you set the key in .env and the problem persists, clearing the config cache should fix it.

Part of the issue may be that mollie/laravel-mollie is booted before this Cashier package.
Easiest fix is to mention in Cashier's readme that the key should be set in .env before running composer require.

Fixed it in the readme.
Thanks for reporting @JorisDebonnet

Hi @maksudur-rhmn, you can circumvent it like this:

First set the MOLLIE_KEY env variable
Then ensure that there is bootstrap/cache/config.php file (you can delete it) if it's there
Try installing this module again

Another workaround, I figured out later, is to edit the "scripts" section at the bottom of composerjson, and temporarily remove the "post-autoload-dump" there. You'll want to put it back after installing this package though, and once everything is setup and working, run composer dump-autoload to make sure everything is in order.

Thank you :)


Made following modifications to bring the datamodels up to date with …

…the latest changes to the enumeration.json file in data model version 1.1.0.

organisationType: changed “commerical” spelling error to “commercial”
personRole: removed 11 (publicationAuthor) and added 22 (depositingUser)
resourceType: Removed 12 (metadataRecord)

Hi @LondonAppDev checked and approved. Please merge.


Add C Support.

C Support is nearly there as python has to talk though a C like interface.

Do you mean C++ right ? Because the readme indicates C support.

We are about 99% on the C support. We still have like 1% more to do.

For the C++ support it will be a simple formality ?

Well C++ can use C APIs yes, but I would not call them C++ APIs. For C++ APIs I expect name spaces, classes, operator overloading, polymorphism, exception throwing and maybe RAII. C++ should be treated as a different language. More like Nim or Java.

I totally agree with you. I hope your project will motivate developers to use Nim to create powerful libraries for other languages. And finally enlarge the community.

Thank you again for your incredible contributions.
This is probably a naïve question. A proc returning a string gives a char* return type in the generated C header (after .cstring is called in internal.nim). Whas is the lifetime of this memory? Isn't the parent string vulnerable to garbage collection at Nim's will, and doesn't this affect the cstring's storage? Isn't it necessary to export memory management procs to accompany it (somewhat like system/allocCStringArray)?

We are aware of the problem with returning strings. We will probably implement some sort of buffer type or make ref/unref work with strings.
For now it happens to work because pixie is single threaded, returning the string still keeps the data around for C to deal with. But yes as soon as you make another call the data can get clobbered.
We will fix this.


DirectivesAttributes.java:36-39: Add Unit Tests for...

The puzzle 589-249cdd88 from #589 has to be resolved:
https://github.com/objectionary/jeo-maven-plugin/blob/7a800a4c6d30052ea10cc8d96493b1762c7a786d/src/main/java/org/eolang/jeo/representation/directives/DirectivesAttributes.java#L36-L39
The puzzle was created by rultor on 14-May-24.
Estimate: 30 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 589-249cdd88 has disappeared from the source code, that's why I closed this issue.


Proposed fix for startup command when using AWS SSO

In the Readme, the current AWS startup command doesn't work if you are using SSO. This is because the Docker container needs more of the ~/.aws folder exposed in order to access full data about the current SSO session.
The command should read:
export AWS_PROFILE=<your_aws_profile>
docker run \
    -e API_PROVIDER=bedrock \
    -e AWS_PROFILE=$AWS_PROFILE \
    -e AWS_REGION=us-west-2 \
    -v $HOME/.aws:/home/computeruse/.aws \
    -v $HOME/.anthropic:/home/computeruse/.anthropic \
    -p 5900:5900 \
    -p 8501:8501 \
    -p 6080:6080 \
    -p 8080:8080 \
    -it ghcr.io/anthropics/anthropic-quickstarts:computer-use-demo-latest

I'd be happy to PR this but don't have push access to the repo.
Thanks for a great demo.

Thanks! Would you mind making a fork and sending PR from there? Appreciated!

What error are you seeing with that?
I am seeing 'anthropic.PermissionDeniedError: Error code: 403 - {'message': "You don't have access to the model with the specified model ID."}' even with Bedrock:*
Wondering if that is my issue.

@joerawr there's an explicit model access request process on bedrock, which I believe is what you're running into.

Yes Sir, that was it.  6 of 7 Anthropic models were selected, and the V2 that the reference image uses is V2.  I enabled it, and we're in.  Thanks!


Enhance "Verify That XXXXXX Image Is A CPaaS Built Image" tests

Tests were failing sometimes because prometheus pod wasn't running yet after installation. Adding a wait until the prometheus pod is running

Tested successfully in local and rhods-ci-pr-test/2351 (only oauth-proxy test, as the others were skipped)


CF : Added support for get template summary

CF : Added support for get template summary

Coverage decreased (-0.05%) to 94.269% when pulling 32509a2575213f1376f92845010a130bc7c30652 on usmangani1:LISSUEEXT32 into 936c7c80f193ca5c7662a2b25742ac142bac5419 on spulec:master.

This is now part of moto >= 1.3.15.dev1001


Validate custom domains during app creation

No more than 5 conditions can be added to a rule https://docs.aws.amazon.com/elasticloadbalancing/latest/application/load-balancer-limits.html

For now, we should not allow more than 4 custom domains to be added to an app stack. In the future, we could split them, but that would require changes to the cloudformation stacks as well.


Enable camel_case_cursor_sensitivity in a searchbox

Is there way to enable this extension's feature even in a search box?
Here:

I don't see any extension points for that in the docs, no. But I've been foiled by them before. I'd assume it's pretty regular textbox but I could have a little look through the source.
Don't hold your breath though.

I would also be interested to know if it could be enabled in this search box, that appears when renaming an identifier in TS:

Man I'd be all over this if I found it to be possible. Still doesn't seem to be though. The docs are quite awful but even the latest typings doesn't give anything away. Maybe we should make an enquiry over at the vscode repo.


Error: provider.mailgunv3: fork/exec /tmp/workspace/terraform-provider-mailgunv3: exec format error

I got this error message:
Error: provider.mailgunv3: fork/exec /tmp/workspace/terraform-provider-mailgunv3: exec format error

Could this be happening because of the binary rename?
https://github.com/alexissavin/terraform-provider-solidserver/issues/3
This person seems to have experienced similar issues for a different provider, but was able to resolve it by recompiling source.
That did not resolve it for me. Do you have any ideas, by any chance?
It had been working a couple days ago 😕
Additional information:
+ terraform-provider-mailgunv3
  File: terraform-provider-mailgunv3
  Size: 39238     	Blocks: 80         IO Block: 4096   regular file
Device: 3dh/61d	Inode: 66705       Links: 1
Access: (0777/-rwxrwxrwx)  Uid: (    0/    root)   Gid: (    0/    root)
Access: 2019-06-12 01:05:28.000000000
Modify: 2019-06-12 01:05:28.000000000
Change: 2019-06-12 01:05:28.000000000

+ terraform plan '-input=false' '-out=tf-plan'

Error: provider.mailgunv3: fork/exec terraform-provider-mailgunv3_v0.3.0: no such file or directory

Estimated Date it Stopped Working: 2-3 days ago?
Terraform Version: v0.11.13

Sorry to hear there are issues. Just to confirm, what version of terraform
and the plugin and OS are you using?
On Tue, Jun 11, 2019 at 8:59 PM Nicole Yson<EMAIL_ADDRESS>wrote:

I got this error message:
Error: provider.mailgunv3: fork/exec /tmp/workspace/terraform-provider-mailgunv3: exec format error
Could this be happening because of the binary rename?
alexissavin/terraform-provider-solidserver#3
https://github.com/alexissavin/terraform-provider-solidserver/issues/3
This person seems to have experienced similar issues for a different
provider, but was able to resolve it by recompiling source.
That did not resolve it for me. Do you have any ideas, by any chance?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/phillbaker/terraform-provider-mailgunv3/issues/14?email_source=notifications&email_token=AAAXCKIWRJXN4C3ROKIMRI3P2BC55A5CNFSM4HXEO432YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GY6G5QA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAXCKJNRU7SUKJIO662VGLP2BC55ANCNFSM4HXEO43Q
.

What OS are you running this on? Are you compiling the provider or using the binary on the release?

MacOS and I tried both the binary on the release and compiling the provider.

It seems to be indicating it's looking for the correct provider. Nothing else has changed since a couple days ago :/
Things I've tried:

switching the name of the binary to just mailgunv3 instead of terraform-provider-mailgunv3_v0.3.0
changing the permissions of the binary chmod +x chmod 700 chmod 777
bumping the version down to 0.2.0

Behavior:

can terraform init and validate with terraform-provider-mailgunv3_v0.3.0 but not terraform-provider-mailgunv3 (renamed binary)
cannot plan with terraform-provider-mailgunv3_v0.3.0

stat before running the plan to ensure it exists:
+ stat  <redacted full path to provider>/terraform-provider-mailgunv3_v0.3.0
  File: <redacted full path to provider>/terraform-provider-mailgunv3_v0.3.0
  Size: 33631504  	Blocks: 65688      IO Block: 4096   regular file
Device: 3dh/61d	Inode: 66799       Links: 1
Access: (0755/-rwxr-xr-x)  Uid: (    0/    root)   Gid: (    0/    root)
Access: 2019-06-13 17:21:53.000000000
Modify: 2019-06-13 17:21:50.000000000
Change: 2019-06-13 17:21:52.000000000

running the plan:
[Pipeline] sh
+ cd infra/acquisitions
+ terraform plan '-input=false' '-out=tf-plan'

provider.mailgunv3: fork/exec <redacted full path to provider>/terraform-provider-mailgunv3_v0.3.0: no such file or directory

Ah, @NicoleYson I think the last detail is the important one. Related to #10 - the current binary on the release page is dynamically compiled for linux, but alpine needs a static build.
I've updated the release to have static builds: https://github.com/phillbaker/terraform-provider-mailgunv3/releases/tag/v0.3.1. Can you try the appropriate file from there or compiling to have a static build locally?

Awesome!!! Thanks! I'll try it.

YES!!! I think it's working! 😄 Thank you!! 🎉

Confirmed working for sure.
terraform apply and everything. Thank you so much for your fast resolution @phillbaker !!! 😄


CompositeByteBuf#newComponent() failed to resolve source index

Motivation:

CompositeByteBuf#newComponent(ByteBuf buf, int offset) checks whether buf is instance of ( AbstractUnpooledSlicedByteBuf, PooledSlicedByteBuf)  to adjust srcIndex.
If we use -Dio.netty.leakDetection.level=SIMPLE, PARANOID, ..,  to find out memory leakage, (Simple|Advanced)LeakAwareByteBuf wraps ByteBufs mentioned above based on samplingInterval. so ByteBuf checking logic skips to unintentionally.

Modification:

Add more instance check logic: if buf is instance of WrappedByteBuf, unwrap it first.

Result:

srcIndex is resolved correctly.

@jingene please provide a unit test as well

@normanmaurer Oh sorry, I added a unit test just now. https://github.com/netty/netty/pull/9398/commits/184d750766b061664a765255b49cc7e846f90293

@netty-bot test this please

@netty-bot test this please

I fixed to do unwrap() if buf is an instance of SimpleLeakAwareByteBuf and unwrapped buf is not type of sliced buffer(AbstractUnpooledSlicedByteBuf or PooledSlicedByteBuf),
create Component instance using initial LeakAwareByteBuf.
(https://github.com/netty/netty/pull/9398/commits/31c0d2f055b4adc90dc6e9fa14f68bbcbb215f21)

@netty-bot test it please

@netty-bot test this please

@jingene there seems to be a leak... please check with ./mvnw -Pleak clean package and fix.

@jingene just to understand, this is not a fix for a leak or functional error, is that correct? The unwrapping and index adjustment done in newComponent is really only an optimization to reduce subsequent indirection. So I'm not sure this is really needed given that leak aware buffers are rare by design.
If we do it then it seems we should do it generally for WrappedBuffers, what was the reason for the latest commit to make it more specific? The important thing is that when buf is set to an unwrapped buffer, the slice field is set to the original buffer. This will ensure existing ref counting semantics are preserved.
And it would also cover UnreleasableBuffers which could be more prevalent/significant than the leak-aware ones in certain use cases.

@njhill
Actually, I don't have deep understanding of ByteBuf family concepts. sorry.
The import thing is the behavior of CompositeByteBuf#duplicate() changed after the netty v4.1.32.Final  released(related PR; https://github.com/netty/netty/pull/8437/files).
I wrote a testcase to prove  CompositeByteBuf#component() works as intended if added component is (*SlicedByteBuf wrapped with *LeakAwareByteBuf)
https://github.com/netty/netty/pull/9398/files#diff-88454e0c917aafd9378476d4cf035969R38
Using before the v4.1.31.Final: TC PASS
Using after the v4.1.32.Final : TC FAILED
Please tell me if there is wrong with the verification. Thanks.

Thanks @jingene, I only read the PR description and should have looked closer at your unit test. So the problem is really that the behaviour of CompositeByteBuf#component(int) changed, right? This does indeed look like a bug, so thank you for raising it.
I think it was caused by my mis-assumption about the behaviour of ByteBuf#duplicate() - specifically that duplicating a slice does not result in another slice and doesn't preserve the reader/writer indices.
As mentioned, unwrapping sliced buffers before they're added was really just an optimization and shouldn't be required for correctness. It just happens that it circumvented the bug for the most common cases. I will open a PR with a general fix and your unit test (which looks great) and make sure you are included as a co-author.
We could separately consider extending the pre-unwrapping optimization to include some of these other buffer variants like wrapped slices, and possibly duplicates.

@njhill Thank you for your explanation :) I'll close this PR and I hope it will be discussed further in the PR that you will open.

Thanks @jingene, I've now opened #9416, PTAL!


vscode was error closed while saving the file

Type: Bug

Click on the file header to create a new file but use Ctrl + s to save it
This problem will occur when the file name is index/component/view or others, but using default name Untitled-1 will not

VS Code version: Code - Insiders 1.88.0-insider (b356b209a5a41743b45f25d569a7afb733d347a9, 2024-03-18T16:36:07.814Z)
OS version: Windows_NT x64 10.0.19045
Modes:
https://github.com/microsoft/vscode/assets/102725920/5cf8799c-d25b-49ea-8d64-5c7068054aa6

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 x 2808)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
15.87GB (7.03GB free)

Process Argv
--crash-reporter-id 5db62d60-a6db-471f-9a20-2cd2cbe2e849

Screen Reader
no

VM
0%

Extensions: none
A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805cf:30301675
vsaa593cf:30376535
py29gd2263:30784851
vscaat:30438846
c4g48928:30535728
962ge761:30841072
pythongtdpath:30726887
welcomedialog:30812478
pythonidxpt:30768918
pythonnoceb:30776497
asynctok:30898717
dsvsc013:30777762
dsvsc014:30777825
dsvsc015:30821418
pythontestfixt:30866404
pythonregdiag2:30926734
pyreplss1:30879911
pythonmypyd1:30859725
pythoncet0:30859736
2e7ec940:30992800
pythontbext0:30879054
accentitlementst:30870582
dsvsc016:30879898
dsvsc017:30880771
dsvsc018:30880772
d6fi2104:30968146
pydisww2:30991134
cppperfnew:30980852
d34g3935:30961436
fegfb526:30952798
bg6jg535:30979844
ccp2r6:30958154
pythonait:30973460
gee8j676:30988845

Can you please follow the steps in https://github.com/Microsoft/vscode/wiki/Native-Crash-Issues to get at more details around the crash and attach the result here? Thanks!

has this d7e82eac-8811-4b22-9289-b151e1386519.dmp

It looks like the crash occurs either because of Sogou IME or because of the interaction between Sogou IME and the file dialog. Do you encounter these crashes on other programs when trying to save files with custom names?
I have attached the symbolicated crash dump.
symbolicated-d7.log

I haven't experienced these crashes on other programs. Everything works fine when I use notepad++.
When I use chrome to access a js link and save it with Sogou IME , it is unresponsive but does not crash and exit.
And there is really no problem when i don’t use Sogou IME.


[local_auth] Android throws auth_error.notAvailable instead of showing Dialog to enable pin-code

Is there an existing issue for this?

[X] I have searched the existing package issues
[X] I have read the guide to filing a bug

What package does this bug report belong to?
local_auth
What target platforms are you seeing this bug on?
Android
Have you already upgraded your packages?
No
Dependency versions
pubspec.lock
[Paste file content here]

Steps to reproduce

Make sure pincode and biometrics are disabled
Call localAuth.authenticate()
It will crash and throw auth_error.notAvailable

This reminds me a lot of #96646 and might very well be the same issue.
Expected results
It should show the dialog for the user to add pin-code
Actual results
It throws an error notAvailable.
Code sample
Code sample
bool _authenticated = await localAuth.authenticate(
      localizedReason:
          isSetup ? Translations.biometrics.setUp.localization : Translations.biometrics.login.localization,
      authMessages: <AuthMessages>[
        iosStrings,
        androidString,
      ],
      options: const AuthenticationOptions(
        useErrorDialogs: true,
        stickyAuth: false,
        sensitiveTransaction: true,
        biometricOnly: true,
      ),
    );

Screenshots or Videos

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.3.4, on macOS 13.4.1 22F770820d darwin-x64, locale
    en-DK)
    • Flutter version 3.3.4 on channel stable at
      /Users/lue/Documents/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision eb6d86ee27 (11 months ago), 2022-10-04 22:31:45 -0700
    • Engine revision c08d7d5efc
    • Dart version 2.18.2
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/lue/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Ultimate Edition (version 2023.1.3)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 74.0.4
    • Dart plugin version 231.9161.14

[✓] VS Code (version 1.80.0)
    • VS Code at /Users/lue/Downloads/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (3 available)
    • SM G960F (mobile) • 2174a6b8150d7ece • android-arm64  • Android 10 (API 29)
    • macOS (desktop)   • macos            • darwin-x64     • macOS 13.4.1
      22F770820d darwin-x64
    • Chrome (web)      • chrome           • web-javascript • Google Chrome
      117.0.5938.62

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Hi @LueNC, this is indeed a duplicate of https://github.com/flutter/flutter/issues/96646.
Kindly subscribe to the original issue for updates.
Closing from here, thank you.


Support summaries in the headline

I would love if the KLG format could support free text in the headline (the line where the date goes).
Personally I think it would improve readability, e.g. in the following having day-of-the-week in the headlines makes it easier for me to remember what the context was for the work I did that day:
2021-02-28 Sunday
   06:00-7:00  #writing
   08:00-10:00 #writing (pareto frontier)
   12:00-12:40 #implicit preferences

2021-03-01 Monday
   06:00-07:00 #writing
   09:10-11:44 #implicit
   14:20-15:50 #implicit
   16:00-17:00 garden
   21:00-22:00 looking at text file time-trackers

2021-03-02 Tuesday
    5:40-07:00 #writing
    9:00-12:20 #implicit -
   12:50-13:10 #implicit jon
   13:30-14:00 talking to eduardo about jobs
   14:46-16:12 #implicit working on foundations
   19:14-20:15 try out Pareto frontier with R
   20:25-21:55 #implicit

My initial reasoning to not support summaries in the headline was this:

The headline can contain a the “should total” property 2018-03-25 (7h30m!) and I was worried that allowing multiple optional things to appear might lead to confusion or ambiguity.
The headline is the only place where additional properties or information might go, if ever needed in the future. The lines below are already reserved for free text summaries, so I can’t change anything there without breaking the file format. Curiously, the spec currently allows multiple properties to appear in order to account for extensibility, although it’s of theoretical nature at the moment.
Strictly speaking I think that putting the weekday labels along the dates is a view-concern, not a concern of the data format. So a powerful syntax highlighter could generate these labels on the fly and display them inline. (See mockup below) That way you wouldn’t need to spell out the weekday yourself, and you also could never make a mistake like labelling it with the wrong weekday. Actually having such a syntax highlighter is a different story of course, maybe a potential idea for @vladdeSV to consider for the VSCode highlighter?

That being said, I still want to rule this out categorically, but unfortunately it’s not super trivial either. Let’s keep this issue open, so that we can refer to it.

(EDIT: “I still want to rule this out categorically” → “That being said, I still don’t want to rule this out categorically”)

(Not strictly related to the issue, but I have plans to add some sort of inline labels of sorts to my VS Code syntax highligher. So far I had only come up with the idea of having the total duration for the day, but showing the weekday is a great idea.)


Fix prepared statement params

What type of PR is this?
Bug Fix

What does this PR do / why is it needed ?
Resolved a bug that prevented execution of prepared statements with parameters.

Which issue(s) this PR fixes:

Fixes #
Other notes for reviewers:
Does this PR introduce a user-facing change?

LGTM

This PR is stale because it has been open for 30 days with no activity. Please remove stale label or add any comment to keep this open. Otherwise this will be closed in 5 days.

This PR was closed because it has been inactive for 35 days. Please re-open if this PR is still relevant.


[Question] - Huge Live stream architecture

Hi,
I wanna Deploy a big cluster of SRS (Now I'm considering scalability for the next 10 years )
I was thinking about the following Architecture:

Client is publishing Stream B on origin servers and origin servers are sending this stream to 3 servers (server A,B,C)
If in the middle of streaming event , the number of clients became more than bandwidth of this 3 servers (for instace, with 10Gb NIC on each server and 2mb bandwidth of each stream I can just handle 5000 stream for playback)
So for this 3 servers I can just handle specific number of live stream playback and I don't know which stream would have more clients for playback to push it to 5 server or more
Can SRS handle this problem dynamically?
What is best solution to solve this issue?

Server A,B,C is not a server, it's servers or region. The region A,B,C contains more than one servers.
The edge cluster is design for huge players, for example, Origin(1 player) => Edges(each 100 players) => Edges(each 1000 streams), which serves at most 100*1000=100k players, and there is only one player in origin server.
The origin cluster is design for huge streams(publishers), where each publisher publish one stream to origin.

Please try this arch, do some research and demos. And bellow is about the load balance, and never forget the scalability of system depends on GSLB first, not only about media servers:

Create multiple origin-edge clusters, where GSLB can dispatch client to different origin-edge cluster, for example, by url hash, the client which play x://xxx/x/x is dispatched to cluster A, and client plays y://yyy/y is dispatched to cluster B.
The GSLS is only a load balance API, for example, an HTTP RESTful API, to get the server to publish or play.
The common tech for GSLB is DNS or HTTP-DNS, please do some research.


Zoom value for google api

Hi, I was looking for an appropriate zoom value for google static map API. I found zoom value 14 in the planet downloader script. Should I use the same for google map API?
Thanks.

Use zoom = 16.
You can test out the download in the jupyter notebook on a given lat/lon/zoom as well to make sure the image seems of decent quality.

These are two files I tested with two zoom (14 and 16) values. Which do you
think will be best for our task?
Thanks.
On Thu, Sep 3, 2020 at 11:44 PM Jatin Mathur<EMAIL_ADDRESS>wrote:

Closed #6
https://github.com/jmather625/predicting-poverty-replication/issues/6.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jmather625/predicting-poverty-replication/issues/6#event-3726097503,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALK7L5ZBWQ7A5ZC57HQ4I2TSD7PYPANCNFSM4QVGUKRA
.

16 if using Google

Thanks
On Fri, Sep 4, 2020 at 2:31 AM Jatin Mathur<EMAIL_ADDRESS>wrote:

16 if using Google
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jmather625/predicting-poverty-replication/issues/6#issuecomment-686774811,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALK7L52QGZG6EI2Q74LWPVDSEADKBANCNFSM4QVGUKRA
.

Hey Ehsan, here are the correct versions:
pytorch==1.3.0 torchvision==0.4.1
I've pushed a readme update and added a comment to help anyone else with these issues

Thanks
On Sun, Sep 6, 2020 at 10:16 PM Jatin Mathur<EMAIL_ADDRESS>wrote:

Hey Ehsan, here are the correct versions:
pytorch==1.3.0 torchvision==0.4.1
I've pushed a readme update and added a comment to help anyone else with
these issues
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jmather625/predicting-poverty-replication/issues/6#issuecomment-687846094,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALK7L54GKAXFG6OPT5KSWELSEO7YNANCNFSM4QVGUKRA
.


Accordion Content Text Missing Randomly

Framework7 version: 3.6.6
React.js version: 16
Platform and Target:Chrome 72

Has anyone bumped into this before?

I've a similar problem in accordion list. There is a visual glitch when opened.

JSFiddle: https://jsfiddle.net/amfyw5rp/1/
Framework7 version: 3.6.6
Vue.js version 3.6.6
Platform: Chrome, Android, iOS

Also happening on 4.0.1

I've a similar problem in accordion list. There is a visual glitch when opened.

JSFiddle: https://jsfiddle.net/8zhr3vkc/3/
Framework7 version: 3.6.6
Vue.js version 3.6.6
Platform: Chrome, Android
On Safari & Firerox 65 & Edge this does not seem to happen

Find any work arounds for this? It seems to be pretty random for me. 4.0.1 does seem to do it less often.

When TouchRipple shuts down, visual failure is more noticeable: https://jsfiddle.net/mc1k6gL5/
Some items don't render until scroll or click another item.

I have been able to fix it by removing the 'swipeout' attr: https://jsfiddle.net/z7fs9n04/

Ok here is a clear example of it.
For framework7 React. It seems to be related to having the ListItem as a link or href
https://jsfiddle.net/quallman1234/vcme6snq/

Any hints of a fix? Its only happening in chrome :(. Dont have a large user base so can shift them to firefox for the time being.

have the same problem with react and chrome

Still happening in 4.3.1


fix: Don't install recommended packages in base image

Get rid of pynvim, which is flagged as malicious in XRAY.
Resolves #313.

Tried the change with the t4c/client/remote image and works fine so far. Quickly checked the list of installed packages. The only relevant missing one was gnupg, added it to the list.

Apparently this fix has broken the image for me: I have relied on the presence of ssh inside the container and have used it along with other components in order to use the identity keys present on the host (mounting gpg-agent and ssh-agent sockets inside the container). What would be your recommended approach to mitigating this? On the host I use KeePassXC and its ssh-agent feature in order to not store private keys on the disk. I require maintaining this state, therefore it would be necessary for the Eclipse inside the container to fetch the keys from the agent running on the host.
I would be grateful for any insight regarding this issue, either some solution I haven't thought of or simply adding ssh to the container explicitly.

Apparently this fix has broken the image for me: I have relied on the presence of ssh inside the container and have used it along with other components in order to use the identity keys present on the host (mounting gpg-agent and ssh-agent sockets inside the container). What would be your recommended approach to mitigating this? On the host I use KeePassXC and its ssh-agent feature in order to not store private keys on the disk. I require maintaining this state, therefore it would be necessary for the Eclipse inside the container to fetch the keys from the agent running on the host.
I would be grateful for any insight regarding this issue, either some solution I haven't thought of or simply adding ssh to the container explicitly.

Hey @kacper-ka, definitely an interesting use case that I didn't consider yet. Until there is a solution, you can continue to use version v2.5.2.
I mainly see two options:

You create a small Dockerfile that inherits from our image and installs an ssh-agent on top. This would be the easiest solution.
FROM [IMAGE_THAT_YOU_CURRENTLY_USE]
USER root
RUN apt-get update && apt-get install openssh-client
USER techuser

Then, you have to build the image with docker build .. Please note that you have to repeat the step after each update of the base image.

We add openssh-client to the image again. I just see the risk that we don't want to have ssh installed in the remote image for security reasons. The remote image inherits from the base image, so that would be affected as well. This situation will improve with buildpacks because we can individually combine layers, but this will not be happen anytime soon (https://github.com/DSD-DBS/capella-dockerimages/pull/300).

Hey @kacper-ka, a short update from my side. We decided in the team that we want to support SSH officially in the image. Thanks for raising the nice use-case. The client was added in https://github.com/DSD-DBS/capella-dockerimages/pull/319 and will be part of the next release (this week).

Great, thanks for the update @MoritzWeber0 !

Great, thanks for the update @MoritzWeber0 !

Took a bit longer than expected because we had to fix the pipeline first. The update is released in v2.6.0. Feel free to try it out.