HTML

#HTML
##User story:
As a boot camp student, I want to use GitHub issues to manage and track tasks for my projects.
##Acceptance criteria:
It is done when the prework-study-guide repository contains five GitHub issues.
It is done when each issue has a title.
It is done when each issue has a description typed using the Markdown language.

#HTML
##User story:
As a boot camp student, I want to use GitHub issues to manage and track tasks for my projects.
##Acceptance criteria:
It is done when the prework-study-guide repository contains five GitHub issues.
It is done when each issue has a title.
It is done when each issue has a description typed using the Markdown language.

Done


Empty output after running examples

Hi,
I'm trying to run one of your examples, but the results.txt is always empty.
This is the command line I used:
differentialBindingAffinity_multipleSNPs examples/combined_Jaspar2022_Hocomoco_Kellis_human_transfac.txt examples/SNPs_EFO_0000612_myocardial_infarction.bed reference_genome_hg38/reference_genome_hg38.fa
This is the output (apparently, no errors):
HELLO
PFM dir: examples/combined_Jaspar2022_Hocomoco_Kellis_human_transfac.txt
SNP file: examples/SNPs_EFO_0000612_myocardial_infarction.bed
genome file: reference_genome_hg38/reference_genome_hg38.fa
index file reference_genome_hg38/reference_genome_hg38.fa.fai not found, generating...
number SNPs: 1665
numMotifs: 817
number of tests: 1360305
rank bevor correction: 968353
rank after correction: 0
The reference genome I downloaded from here: https://hgdownload.soe.ucsc.edu/goldenPath/hg38/bigZips/ (hg38.chromFa.tar.gz file). After I download, I concatenate all chromosomes in a single .fa file. I included first chr1 to chrY, and then the rest in a sorted order (show part of the file below):
1:>chr1
4979131:>chr2
9823003:>chr3
13788916:>chr4
17593209:>chr5
21223976:>chr6
24640097:>chr7
27827018:>chr8
30729792:>chr9
33497688:>chr10
36173638:>chr11
38875372:>chr12
41540880:>chr13
43828168:>chr14
45969044:>chr15
48008869:>chr16
49815637:>chr17
51480787:>chr18
53088254:>chr19
54260608:>chr20
55549493:>chr21
56483694:>chr22
57500065:>chrX
60620884:>chrY
61765434:>chr10_GL383545v1_alt
61769021:>chr10_GL383546v1_alt
61775219:>chr10_KI270824v1_alt
61778850:>chr10_KI270825v1_alt
61782618:>chr11_GL383547v1_alt
61785708:>chr11_JH159136v1_alt
61789729:>chr11_JH159137v1_alt
61793559:>chr11_KI270721v1_random
(...)
I attached here some of the output files (I modified to .txt so github allows me to upload them)
result.txt
motifInfo.txt
notConsideredSNPs.txt
TF_count.txt
SNPsUnique.txt
snpRegions.txt
info.txt
I would appreciate your help on this matter. The program looks very good, by the way.

Hi,
I also had the same problem. I used a genome from a non-model species (correctly renamed with chr1,chr2...)and a .bed file built following your instructions. The analysis seems to be successful without errors but the results.txt file only has the header and is empty.
I'm agree that the program looks very good.
Thanks for your help

Hello,
thank you for pointing out the problem, we will look into it.
Regards,
Marcel

Hi @biouserPD and @mauriciomr1985,
sorry for the late reply.
I was able to reproduce the error both of you got.
The issue is that one needs to provide a scale file, which I added to the git hub repo (necessaryInputFiles/estimatedScalesPerMotif_1.9.txt). Otherwise sneep cannot compute a p-value for the differential TF binding score and will set it to 1, which result in an empty result file (since no sig. result is obtained).
If you run the following command, do you still observe a non-empty result file?
differentialBindingAffinity_multipleSNPs  -s necessaryInputFiles/estimatedScalesPerMotif_1.9.txt examples/combined_Jaspar2022_Hocomoco_Kellis_human_transfac.txt examples/SNPs_EFO_0000612_myocardial_infarction.bed  <path_to_hg38>
I am going to update the ReadTheDocs accordingly as soon as possible.  (Also for Example 1 in ReadTheDocs a file is missing, so you might skip this for now.)
Sorry for the issue and thanks a lot for your feedback and especially for your patience!
Nina

Hi again,
modified sneep in such a way that the scale file is a required input (no longer given with the flag -s) to avoid the here mentioned problem. The code on github is up-to-date. Based on our experience it usually takes a few days until the bioconda packages is updated as well. I also adapted ReadTheDocs accordingly.
The command I gave in the previous comment, changes in the updated sneep version to:
differentialBindingAffinity_multipleSNPs examples/combined_Jaspar2022_Hocomoco_Kellis_human_transfac.txt examples/SNPs_EFO_0000612_myocardial_infarction.bed  <path-to-genome-file> necessaryInputFiles/estimatedScalesPerMotif_1.9.txt
I hope the bioconda package is updated soon!
Nina

Hello,
the last time I used SNEEP was about two months ago and it still gave me an
empty output file.
In the next few days I will try again by installing the software via conda
and I will let you know.
Unfortunately I use a non model species for the analysis and I'm trying to
figure out if there's a way to apply your software in the best way.
I will keep you updated.
Thanks a lot,
Max
Il giorno lun 18 mar 2024 alle ore 14:17 SchulzLab @.***>
ha scritto:

Hello,
we were wondering if SNEEP now works for you with the updated bioconda
package?
Kind regards,
Marcel
—
Reply to this email directly, view it on GitHub
https://github.com/SchulzLab/SNEEP/issues/8#issuecomment-2003881302, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHVMHEKOV4IXG7EM5ED7Z5DYY3SM5AVCNFSM6AAAAABABTCDF2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBTHA4DCMZQGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Massimiliano Babbucci, PhD
Department of Comparative Biomedicine and Food Science (BCA)
University of Padova
Viale dell'Universita' 16, Agripolis
I-35020  Legnaro (PD), Italy
ORCID:  https://orcid.org/0000-0002-4060-4728
Tel: + 39<PHONE_NUMBER>
e-mail: @.***

Hello,
I used the conda updated version of SNEEP and this time the output file is
not empty.
I'm now trying to understand what the best strategy is in the case of
non-model organisms, a fish in my case.
Do you have any advice?
Thanks a lot for your help,
Best
Max
Il giorno lun 18 mar 2024 alle ore 14:17 SchulzLab @.***>
ha scritto:

Hello,
we were wondering if SNEEP now works for you with the updated bioconda
package?
Kind regards,
Marcel
—
Reply to this email directly, view it on GitHub
https://github.com/SchulzLab/SNEEP/issues/8#issuecomment-2003881302, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHVMHEKOV4IXG7EM5ED7Z5DYY3SM5AVCNFSM6AAAAABABTCDF2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBTHA4DCMZQGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Massimiliano Babbucci, PhD
Department of Comparative Biomedicine and Food Science (BCA)
University of Padova
Viale dell'Universita' 16, Agripolis
I-35020  Legnaro (PD), Italy
ORCID:  https://orcid.org/0000-0002-4060-4728
Tel: + 39<PHONE_NUMBER>
e-mail: @.***

Hi Max,
great to hear that it is working now!
Yes sure, I can help you with this, but maybe we can discuss it in another issue? I will just open one ;)
Thanks a lot!
Nina


pnpm dependabot documentation incorrect

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/enterprise-cloud@latest/code-security/dependabot/ecosystems-supported-by-dependabot/supported-ecosystems-and-repositories
What part(s) of the article would you like to see updated?
This section:

shows security updates being supported for pnpm v7,8,9, but security updates are not supported for pnpm v9 (apparently GitHub has an internal ticket tracking this)
Also flagged here: https://github.com/dependabot/dependabot-core/issues/10534
Additional information
No response

@jamescrowley Thank you for raising this issue! I'll get this triaged for review :sparkles: Our team will provide feedback regarding the best next steps for this issue - thanks for your patience! 💛

@jamescrowley 👋 Thank you for this—for raising it and for the fact that you raised an issue first.
I've confirmed you're absolutely right, so we need to fix this. We're in a bit of an unusual situation as we're freezing the docs repo next week, so I wanted to get this fix in as soon as possible rather than opening it up to contributors. I thought about adding you as a co-committer but I noticed you don't make use of the no-reply function in your commits, so I didn't want to go ahead and do that without your permission.
You'll notice this issue close shortly when we merge in the fix, and the docs will reflect the change shortly thereafter.
Please do know that we really appreciate your help here—I'll just cc in @nguyenalex836 for 👀 as well.

Cheers @subatoi 🙏 I didn’t actually know about the no-reply email option on github so thanks for sharing that too!


Unable to use @Transient with @Temporal (backport to 1.1)

see Haulmont/jmix-core#254

see commits

a4df29264
f74deb596
db0b9c098


Ordering Service max proposal size

Signed-off-by: Igor Egorov<EMAIL_ADDRESS>Description of the Change
Ordering Service would not produce proposal bigger than max_proposal_size.
The issue was:
max_proposal_size = 3
Two batches of two transactions arrive.
Both will be processed.
The resulting proposal will have 4 transactions inside.
Benefits
Correct behavior.
Usage Examples or Tests
All the tests

CI failed due to known issue of 9.0 apple clang.


Auto-correct empty class definitions

The Ruby style guide says:

Prefer a single-line format for class definitions with no body.

# bad
class FooError < StandardError
end

# okish
class FooError < StandardError; end

# good
FooError = Class.new(StandardError)

I'd like to have a cop that could auto-correct to the 'good' example.
We added one in GitLab, but just for error classes, because the 'bad' example is more reasonable when, for instance, you have an empty Rails controller that just uses the default methods provided.
Here's the cop we added: https://gitlab.com/gitlab-org/gitlab-ce/commit/8dd097a91530e4b047c4b391f21047c7d29d310d
This works for us, but I have no expectations that this would work for all cases 🙂
What would we need to do to get this into RuboCop itself?

An "errors only" config option on this cop would be great.

Nice one, @smcgivern! 😀
RuboCop has moved from just a one way street towards the style guide, so all cops now are generally configurable so users can choose the style they want. This is accomplished with the ConfigurableEnforcedStyle module. See for example Style/EmptyMethod.
Oh, and you can't use the lonely operator, because RuboCop supports Ruby 2.1. 🙂

What would we need to do to get this into RuboCop itself?

Start by filing a PR. :-)
On the cop specifics - I was thinking of altering the style guide advice, as many people voiced a valid complaint over the years, that the recommended approach doesn't work very well in most editors (they fail to process this as a class definition). It's also a bit harder to spot for human readers. What I'm trying to say is that this cop should ideally be configurable.

I think this is still valid, but I haven't had the time or energy to work on it 😞
I'm fine if the bot closes it; if I do come back to it in future, I can always reopen.

I re-opened this because I think it's still very relevant. I have been planning to implement "half" of this cop myself. I think there are actually two different cops here, one that takes care of whether a class definition should be single-line:
# bad
class FooError < StandardError
end

# good
class FooError < StandardError; end

and one that checks that such class definitions are written with Class.new or not:
# bad
class FooError < StandardError; end

# good
FooError = Class.new(StandardError)

@Drenmi What should be the flow? I meant the correct is
class FooError < StandardError; end
or
FooError = Class.new(StandardError) ?
Or it should be up to developer?
Could you help with cops naming?  :blush:

What should be the flow? I meant the correct is
class FooError < StandardError; end

or
FooError = Class.new(StandardError) ?

RuboCop doesn't have opinions. It should be configurable. 🙂 (See cops with ConfigurableEnforcedStyle for reference) If there is a recommendation in The Ruby Style Guide, that should be the default.

Could you help with cops naming? 😊

There is a similar cop that does the same thing: Style/EmptyMethod, so I imagine this would be Style/EmptyClass. You can also see that cop for reference on the implementation.
Another thing to consider is that we should register an offense, but probably not auto-correct if there are comments inside the empty class body, as auto-correcting would delete the comments.

but probably not auto-correct if there are comments inside the empty class body, as
auto-correcting would delete the comments.

Yes, it is deleting comments.

As per this comment I still think this should be two separate cops. Trying to squeeze them both in one will likely lead to confusing implementation and configuration. 🙂

I agree with @mikegee.
For example, warnings are reported for default code generated by bin/rails new and bin/rails g generator commands.
I think that many cases where an empty class is a one-liner are exception classes.
So I think it would be better to provide the following option:
# @example ExceptionClassOnly: true (default)
#   # bad
#   class Foo < Bar; end
#
#   # good
#   class FooError < StandardError; end
#
# @example ExceptionClassOnly: false
#   # good
#   class Foo < Bar; end
#   class FooError < StandardError; end

To determine whether it is an exception class, a regular expression (ex. /.*(Error|Exception)\z/) can be used for the superclass name. So, some exception class names may contain false negatives, but I think they can cover most cases.


fix: Add several fixes to the release process

Signed-off-by: Mykhailo Kuznietsov<EMAIL_ADDRESS>What does this PR do?

Fix bumping version in bugfix/main branches
Change image tags in Makefile to released ones when tagging the release

What issues does this PR fix or reference?
https://github.com/devfile/devworkspace-operator/issues/428
https://github.com/devfile/devworkspace-operator/issues/415
Is it tested? How?

PR Checklist

[ ] E2E tests pass (when PR is ready, comment /test v7-devworkspaces-operator-e2e, v7-devworkspace-happy-path to trigger)

[ ] v7-devworkspaces-operator-e2e: DevWorkspace e2e test
[ ] v7-devworkspace-happy-path: DevWorkspace e2e test

/test v7-devworkspaces-operator-e2e, v7-devworkspace-happy-path


[Feat/LIVE-8811]: LLM: display NotEnoughGasSwap Error

N.B this PR branches off https://github.com/LedgerHQ/ledger-live/pull/4154
📝 Description

Display the NotEnoughGasSwap error previously implemented in https://github.com/LedgerHQ/ledger-live/pull/4154.
Update TranslatedError to use the format <link0>link</link0> ... <linkN>link N</linkN> a translation.

❓ Context

Impacted projects: LLM
Linked resource(s): ``

✅ Checklist

[x] Test coverage
[x] Atomic delivery
[x] No breaking changes

📸 Demo
https://github.com/LedgerHQ/ledger-live/assets/132384348/eba89b04-5549-45c6-9283-09fbb1545bf2
🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

@kallen-ledger Similar with the previous PR, do you need a changeset for LLM?


pjit is 2-3x slower than pmap in data parallelization mode!

Description
Working on data/model parallelization with Jax and testing PJIT since it supports both data and model parallelization. Particularly, if its efficiency matches Pmap in data parallelization, PJIT would be always preferred.
However, pjit is 2-3x slower than pmap in data parallelization mode! Why and How to solve it?
I'm using the official code from flax as shown https://flax.readthedocs.io/en/latest/guides/flax_on_pjit.html
Below is the concrete code I used:
#! /usr/bin/python

from __future__ import absolute_import
from __future__ import division
from __future__ import print_function

import functools
import numpy as np
import jax
import time

from jax import lax, random, numpy as jnp

import flax
from flax import struct, traverse_util, linen as nn
from flax.linen import spmd # Flax Linen SPMD.
from flax.core import freeze, unfreeze
from flax.training import train_state, checkpoints

import optax # Optax for common losses and optimizers.

from jax.experimental.pjit import pjit, with_sharding_constraint
from jax.sharding import Mesh, PartitionSpec
from jax.experimental import mesh_utils, maps

# Start a device mesh.
device_mesh = mesh_utils.create_device_mesh((4, 1))
# Annotate each axis with a name.
mesh = Mesh(devices=device_mesh, axis_names=('data', 'model'))

class DotReluDot(nn.Module):
  depth: int
  @nn.compact
  def __call__(self, x):
    W1 = self.param(
        'W1',
        nn.with_partitioning(nn.initializers.xavier_normal(), (None, 'model')),
        (x.shape[-1], self.depth))

    y = jax.nn.relu(jnp.dot(x, W1))
    # Force a local sharding annotation.
    y = with_sharding_constraint(y, PartitionSpec('data', 'model'))

    W2 = self.param(
        'W2',
        nn.with_partitioning(nn.initializers.xavier_normal(), ('model', None)),
        (self.depth, x.shape[-1]))

    z = jnp.dot(y, W2)
    # Force a local sharding annotation.
    z = with_sharding_constraint(z, PartitionSpec('data', None))

    # Return a tuple to conform with the API `flax.linen.scan` as shown in the cell below.
    return z, None

class MLP(nn.Module):
  num_layers: int
  depth: int
  use_scan: bool
  @nn.compact
  def __call__(self, x):
    if self.use_scan:
      x, _ = nn.scan(DotReluDot, length=self.num_layers,
                     variable_axes={"params": 0},
                     split_rngs={"params": True},
                     metadata_params={nn.PARTITION_NAME: None}
                     )(self.depth)(x)
    else:
      for i in range(self.num_layers):
        x, _ = DotReluDot(self.depth)(x)
    return x

# MLP hyperparameters.
BATCH, LAYERS, DEPTH, USE_SCAN = 8, 32, 1024, True
# Create fake inputs.
x = jnp.ones((BATCH, DEPTH))
# Initialize a PRNG key.
k = random.PRNGKey(0)

# Create an Optax optimizer.
optimizer = optax.adam(learning_rate=0.001)
# Instantiate the model.
model = MLP(LAYERS, DEPTH, USE_SCAN)

# A functional way of model initialization.
def init_fn(k, x, model, optimizer):
  variables = model.init(k, x) # Initialize the model.
  state = train_state.TrainState.create( # Create a `TrainState`.
    apply_fn=model.apply,
    params=variables['params'],
    tx=optimizer)
  return state

with mesh:
  # Create an abstract closure to wrap the function before feeding it in
  # because `jax.eval_shape` only takes pytrees as arguments`.
  abstract_variables = jax.eval_shape(
      functools.partial(init_fn, model=model, optimizer=optimizer), k, x)
# This `state_spec` has the same pytree structure as the output
# of the `init_fn`.
state_spec = nn.get_partition_spec(abstract_variables)

x_spec = PartitionSpec('data', None)  # dimensions: (batch, length)

pjit_init_fn = pjit(init_fn,
                    static_argnums=(2, 3),
                    in_axis_resources=(PartitionSpec(None), x_spec),  # PRNG key and x
                    out_axis_resources=state_spec
                    )
with mesh:
  initialized_state = pjit_init_fn(k, x, model, optimizer)
jax.tree_map(jnp.shape, initialized_state)

def train_step(state, x):
  # A fake loss function.
  def loss_unrolled(params):
    y = model.apply({'params': params}, x)
    return y.sum(), y.sum()
  grad_fn = jax.grad(loss_unrolled, has_aux=True)
  grads, loss = grad_fn(state.params)
  state = state.apply_gradients(grads=grads)
  return state, loss

import sys

method = sys.argv[1]

if method == "pjit":
  pjit_step_fn = pjit(train_step,
                    in_axis_resources=(state_spec, x_spec),  # input annotations
                    out_axis_resources=(state_spec, None),           # output annotations
                    )

  with mesh:
    start_time = time.time()
    state = initialized_state
    for _ in range(100):
      state, y = pjit_step_fn(state, x)
    print(time.time() - start_time)

else:
  pmap_step_fn = jax.pmap(train_step, axis_name='data')
  state = jax.device_put_replicated(initialized_state, jax.local_devices())
  x = x.reshape((4, -1) + x.shape[1:])

  with mesh:
    start_time = time.time()
    for _ in range(100):
      state, y = pmap_step_fn(state, x)
    print(time.time() - start_time)

Below is the result:
python double_check_flax.py pjit
14.091051578521729
python double_check_flax.py pmap
4.450118064880371

pjit takes 3x more time!!
Ideally, when the mesh for pjit is (4, 1), it should run in a similar data parallelization mode to pmap, but it's clearly and significantly slower. Why does this happen and how to solve it?
Thanks!
What jax/jaxlib version are you using?
jax/jaxlib: 0.4.2
Which accelerator(s) are you using?
4 GPU
Additional system info
python=3.8
NVIDIA GPU info
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 470.82.01    Driver Version: 470.82.01    CUDA Version: 11.4     |
|-------------------------------+----------------------+----------------------+
| 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 GeForce ...  On   | 00000000:02:00.0 Off |                  N/A |
| 31%   24C    P8     9W / 250W |      1MiB / 11178MiB |      0%      Default |
|                               |                      |                  N/A |
+-------------------------------+----------------------+----------------------+
|   1  NVIDIA GeForce ...  On   | 00000000:04:00.0 Off |                  N/A |
| 30%   25C    P8     8W / 250W |      1MiB / 11178MiB |      0%      Default |
|                               |                      |                  N/A |
+-------------------------------+----------------------+----------------------+
|   2  NVIDIA GeForce ...  On   | 00000000:83:00.0 Off |                  N/A |
| 29%   29C    P8     8W / 250W |      1MiB / 11178MiB |      0%      Default |
|                               |                      |                  N/A |
+-------------------------------+----------------------+----------------------+
|   3  NVIDIA GeForce ...  On   | 00000000:84:00.0 Off |                  N/A |
| 28%   25C    P8     9W / 250W |      1MiB / 11178MiB |      0%      Default |
|                               |                      |                  N/A |
+-------------------------------+----------------------+----------------------+

Hi, sorry, just found out that I didn't apply lax.psum for pmap to sync over devices. The speed difference nearly disappeared after fixing it.
Btw, jax.Array is really cool! thanks!


Fixed typo in test method name

Description
Typo fix
Motivation and Context

Types of changes

[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 change)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.

:+1:

Just noted that it's required to sign commits here

:+1:


Extend mocking

vi.unmock is not getting hoisted
we need doMock/doUnmock that are not getting hoisted
support calling vi.mock inside setupFiles to simulate automocking
migration guide for mocking
refactor to make it easily testible

Suggestion: make vi.mock return mocked module if possible. So there it no need to import module in tests.
This will fix the Typescript hacks like this:
https://github.com/vitest-dev/vitest/blob/c927e2eea47454f662a6f2049f0a684c59225bab/test/core/test/mocked.test.ts#L2-L3
When type of mocked module is not the same as the real one.
https://github.com/vitest-dev/vitest/blob/c927e2eea47454f662a6f2049f0a684c59225bab/test/core/test/mocked.test.ts#L9-L11

We will need to make it async then

Maybe another function then? Not having access to global variables and having wrong type in Typescript is annoying.
It makes it hard to make a module mock that you can control from a test. In the example above it is hard to change timeout property if you need a different value in different tests.
Or maybe I'm missing something?

Maybe another function then? Not having access to global variables and having wrong type in Typescript is annoying.

You can use vi.mocked helper to give it proper types. For example, if method is a spy, it will be typed as such.

It makes it hard to make a module mock that you can control from a test. In the example above it is hard to change timeout property if you need a different value in different tests.

If you need a different value in different test, you will mock it as a different value, no? Mocks are bound to test files, they are not shared.

Or maybe I'm missing something?

You can also use vi.importMock<typehint>('path')

If you need a different value in different test, you will mock it as a different value, no? Mocks are bound to test files, they are not shared.

I wanted to change mocked value in the same test file. I have multiple tests for the same module but with different inputs.

Also you can use vi.spyOn(module, 'timeout', 'get').mockReturnValue(1000), if you need a different value in a test.

This is what I needed, thanks. Need to add this to docs :slightly_smiling_face:


0.1.123 iOS: bundle reloads crash the app

On iOS I found that bundle reloads in dev mode crash the app.
Interestingly, only full reloads. Fast refresh on code changes works fine.
The stack trace does not refer to Skia code, but when I revert to react-native-skia version 0.1.122 I no longer have this issue.
No issues on Android bundle reloading.

You can use this project as an example of the issue;
https://github.com/laurens-lamberts/react-native-skia-example
When you run this app on an iOS simulator, and press R to refresh, the app crashes. On a physical device, refreshing via the shake gesture dev menu crashes the app as well.


support bbnavi

[x] rename project from generate-herrenberg-gtfs-flex to generate-gtfs-flex
[x] prepare project for >1 bundled rule file
[x] make the Haustürbedienung (flexible drop off using locations.geojson) optional
[x] change the flex rules to work trip-based instead of route-based
[ ] write bbnavi (Berlin/Brandenburg cutout from DELFI GTFS) rule file (partially done)

Once this is done, we can adapt gtfs-hub to use the new rule file in order to generate a GTFS Flex feed.

This is blocked because, currently, neither the DELFI GTFS nor the VBB GTFS provide a field/way to distinguish Rufbus trips from regular trips.

Do the trips need to be replaced? Or can they simply be added to existing ones?

I thought the problem was the you cannot identify the trips that you want to remove.

More or less: I cannot identify the trips that the GTFS-Flex transformation should apply to.
AFAICT there is no way to distinguish those trips and/or stop times that are RufBus ones, except by using the machine-unreadable PDF timetable.

AFAICT there is no way to distinguish those trips and/or stop times that are RufBus ones, except by using the machine-unreadable PDF timetable.

UVG/VBB should use >1 of the following ways to delineate RufBuse (demand-responsive buses):

use a route_type of 715 ("Demand and Response Bus Service") on each RufBus trip
use a pickup_type of 2 ("Must phone agency to arrange pickup.") on each stop time of each RufBus trip

As I read the available documentation, both ways are documented to be supported by Google Maps, and I know of many other GTFS consumers who support them as well.

UVG/VBB should use >1 of the following ways to delineate _RufBus_e (demand-responsive buses): […]

With the 2022-07-01 VBB dataset, this is still not the case.

UVG/VBB should use >1 of the following ways to delineate _RufBus_e (demand-responsive buses): […]

With the 2023-03-17 VBB dataset, this is still not the case.

UVG/VBB should use >1 of the following ways to delineate _RufBus_e (demand-responsive buses): […]

With the 2024-05-08 VBB dataset, this is still not the case. In fact, I can't find the 478 bus at all. 🙃
cc @OpenDataVBB

UVG/VBB should use >1 of the following ways to delineate _RufBus_e (demand-responsive buses): […]

With the 2024-05-08 VBB dataset, this is still not the case. In fact, I can't find the 478 bus at all. 🙃

Neither with the 2024-07-05 VBB dataset.
cc @OpenDataVBB


fix: fix delegated message

关联的Issues (Related Issues)

close
改动 (Proposed Changes)

附注 (Additional Info)

自查清单 (Checklist)
在你认为本 PR 满足被审阅的标准之前，需要确保 / Before you mark the PR ready for review, please make sure that:

[ ] 符合Venus项目管理规范中关于PR的相关标准 / The PR follows the PR standards set out in the Venus project management guidelines
[ ] 具有清晰明确的commit message / All commits have a clear commit message.
[ ] 包含相关的的测试用例或者不需要新增测试用例 / This PR has tests for new functionality or change in behaviour or not need to add new tests.
[ ] 包含相关的的指南以及文档或者不需要新增文档 / This PR has updated usage guidelines and documentation or not need
[ ] 通过必要的检查项 / All checks are green

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/v1.10@7fb8697). Click here to learn what that means.
The diff coverage is n/a.

Additional details and impacted files

@@               Coverage Diff                @@
##             release/v1.10     #323   +/-   ##
================================================
  Coverage                 ?   54.56%
================================================
  Files                    ?       69
  Lines                    ?     7200
  Branches                 ?        0
================================================
  Hits                     ?     3929
  Misses                   ?     2945
  Partials                 ?      326

Flag
Coverage Δ

unittests
54.56% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.


ALTS: Do not pass a host to grpc_channel_create_pollset_set_call(); let the channel supply it.

When the handshaker_service_url is in "host:port" format such as it normally is when using ALTS in GCE (in which case it comes from then this makes no difference as the authority and the URL are the same. But when different URLs are used, the correct authority to use is not always the same as the URL. For example if the URL is unix:///some/path then the correct authority is "localhost". This is correctly computed by grpc_core::UnixResolverFactory and stored as the channel's default authority, but we throw that away when we override the authority for individual RPCs.
Note indeed that the majority of other callers of grpc_channel_create_* pass nullptr for the host/authority argument.

This looks reasonable to me, but I'd like those more familiar with ALTS to verify that it won't cause any problems.  Reassigning this to @apolcyn.

Should probably also be reviewed by @erm-g or someone on his team.

LGTM (modulo the minor dead code discussion). Thanks for the PR!


fix: Fixes split panel bouncing on the edge between side/bottom when scrollbar appears

Description
Rel: AWSUI-54993
The issue:
https://github.com/user-attachments/assets/7c0dbd99-382d-49a2-acd8-91d71b42d048
How has this been tested?

New integration test
Manual testing with Classic, VR, Toolbar

Review checklist
The following items are to be evaluated by the author(s) and the reviewer(s).
Correctness

Changes include appropriate documentation updates.
Changes are backward-compatible if not indicated, see CONTRIBUTING.md.
Changes do not include unsupported browser features, see CONTRIBUTING.md.
Changes were manually tested for accessibility, see accessibility guidelines.

Security

If the code handles URLs: all URLs are validated through the checkSafeUrl function.

Testing

Changes are covered with new/existing unit tests?
Changes are covered with new/existing integration tests?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@georgylobko, the fix failed in the dry-run so I rebased against main and resolved the conflict in the integ test: https://github.com/cloudscape-design/components/pull/2794/commits/d63bdf2ffb5a0b9ece02f205ca989c162a4d8e2f


Feature Request: Streaming Chunk Animation

Hey team,
Big thanks for the work on assistant-ui! I’ve been using it for a while, and it’s awesome.
I had an idea that could make it even cooler: adding a streaming chunk animation (like a fade-in for chunks) similar to what Perplexity and Gemini web apps do. It makes the user experience super smooth and engaging.
Here’s a sample implementation that might help: Perplexity-Style Streaming.
Thanks for considering this! Keep up the great work!

Hey, I'm very happy to hear that you're liking assistant-ui! :)
Thanks a lot for this suggestion, it's a cool idea!
The way I would go about adding this is as follows:
// implementation
const FadeInText: TextContentPartComponent = ({ part: { text }, status }) => {
  // TODO implement the animation effect here, returning a <span>
}

// usage
const AssistantMessage = () => {
  ...
  <MessagePrimitive.Content components={{ Text: FadeInText }} />
  ...
}

If someone makes a sample implementation that works, I'm happy to add it to the CLI (command does not exist yet):
npx assistant-ui add fade-in-text

Thanks for your prompt response and being open to new additions! I'll try adding one soon.
Is there any specific guideline or best practice I should follow while implementing this feature? Also, do you have any suggestions for libraries or tools that might make the animation smoother?
Thanks again for considering my suggestion!

I think the example stated uses framer motion, feel free to use that; I don't mind much, since this is an optional addon added from the CLI, I'm not worried too much about the dependency list.
The existing animations (e. g. for the assistant-modal's window) use tailwindcss-animate https://github.com/jamiebuilds/tailwindcss-animate

found another implementation
https://ui.aceternity.com/components/text-generate-effect

Here's another demo
https://assistant-ui-flowtoken-demo.vercel.app/
https://github.com/Yonom/assistant-ui-flowtoken-demo


DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working

Hello @rowillia @jmphilli
We are seeing below warnings when we run pytest on marshmallow package which is using below : orderedset.py and utils.py
marshmallow/orderedset.py:26: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
    class OrderedSet(collections.MutableSet):

marshmallow/utils.py:54: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
    (isinstance(obj, collections.Iterable) and not hasattr(obj, "strip")) or is_generator(obj)

I would like to report this issue and would be glad to contribute to fix the issue in forked copy of marshmallow which toastedmarshmallow is using. However, I'm not quite sure if I need to create a PR with the fix to merged into dev branch of lyft/marshmallow or some other branch.
Would you be able to provide the contribution guidelines or any information which helps me understand how I can contribute to this? Once you merge the PR finally then we would need your help in releasing new package of lyft/toasted-marshmallow so we can updated our project to use the new version of this project to resolve the deprecationwarning.
Please let me know if you need any other information.
Thanks for your understanding and cooperation in this matter.
Regards,
Pavan

@rowillia @jmphilli  I would love to contribute to this fix and the following issues as well:
https://github.com/lyft/toasted-marshmallow/issues/9
https://github.com/lyft/toasted-marshmallow/issues/14
https://github.com/lyft/toasted-marshmallow/issues/4


Fix TLSv1.3 ticket nonces

All tickets on a connection need to have a unique nonce. When this was originally implemented we only ever sent one ticket on the conneciton so this didn't matter. We were just using the value 0. Now we can get multiple tickets to we need to start doing the ticket nonce properly.
As a result of fixing the above we now generate the secrets based on the nonce immediately so there is no
need to keep the nonce in the session, so this PR also removes that code.
Fixes #6387.

Fixup commits pushed addressing feedback so far, and fixing a Travis issue.

Pushed. Thanks.


KEY clarification

Thank you for this excellent project. I love the minimalist approach !
One question. I am loging in via username and pwd.
Do I need to define a SECRETKEY if I dont use the TOTP_KEY functionality?
As I understand the SECRETKEY is only used if the QR-code is not working. Or did I misunderstand?

Hey @MarcS1975. I'm glad you like it 👍
The FLATNOTES_TOTP_KEY is used for TOTP. FLATNOTES_SECRET_KEY is used to generate access tokens, which is used by the password auth. Have a look at the wiki article for more info.


Improve CI performance

As the CI flow may take up too much time with the default test profile, consider adding some compilation flags to improve the CI performance.
A compilation flag I used and resulted to a significant improvement used to be here:
[profile.test]
opt-level = 3

It could be put into ./zkp-toolkit/Cargo.toml now.

got it

use config has side effects when debug in local, maybe use cargo test --release in ci is better.


Precaching Rejig

R: @jeffposnick @addyosmani
This is a slight change on the previous precaching PR (previous comments have been taken into account here).

This has tests for caching relative, absolute, CORs and Local requests. The entries in the file manifest are now parsed by a CacheEntry model (this handles defaults and checking the input is valid).

Rejigged some of the code so that the RevisionedCacheManager is a little leaner and the code is easier to read.

Changed {path: '', revision: ''} -> {url: '', revision: ''}

I'm seeing
  1) sw-precaching Browser Tests should pass all browser based unit tests in  Google Chrome Dev:
     Error: Issues in Google Chrome Dev.
Google Chrome Dev had 1 test failures.
------------------------------------------------
[Failed Test 1]
    - sw-precaching Test Revisioned Caching > should cache and fetch files
        Cannot read property 'should' of undefined
------------------------------------------------
      at assistantDriver.getSeleniumDriver.then.then.then (packages/sw-precaching/test/automated-test-suite.js:74:17)
      at ManagedPromise.invokeCallback_ (node_modules/selenium-webdriver/lib/promise.js:1366:14)
      at TaskQueue.execute_ (node_modules/selenium-webdriver/lib/promise.js:2970:14)
      at TaskQueue.executeNext_ (node_modules/selenium-webdriver/lib/promise.js:2953:27)
      at asyncRun (node_modules/selenium-webdriver/lib/promise.js:2813:27)
      at node_modules/selenium-webdriver/lib/promise.js:676:7
      at process._tickCallback (internal/process/next_tick.js:103:7)

It's not related to this PR. I don't know if you want to merge this now and then fix that, or fix the underlying test in a separate PR that you rebase into this.

I'll fix the test before merging this. Feel free to merge your PR though if you are happy with it.
Bad builds make for an unhappy gauntface.

@jeffposnick I realised the problem I was hitting.
selenium-assistant was updating Chrome dev where my local machine wasn't up-to-date (i.e. tests were on 57, locally I was testing on 56).
In Chrome 57 a change was introduced so that the cachedResponse's .url value included the search params, where as 56 and below, it was just the URL used to cache the response with.
this lead to issues in the way the tests were checking the validity of the cached responses.
This PR now has some extensive logging for all Browser and Service Worker tests WITH stack traces relevant to failing tests, so in the future tests should be easier to spot the specific failing test and the specific location of the error.
Finally. I've moved to a default port number for the test server when nothing is defined. This should prevent the case where node attempts to pick a free port number to then realise that it's in use (it's a rare scenario but is happening frequently across Travis tests.

Thanks for dealing with all the nitty-gritty! 👍


No information how to restore a project or solution

Hi,
on the page there is no information how to restore a project or solution from the commandline. Would be great if you could link that info.
Michael
[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e7a0653a-ec3b-3b60-ae77-a51fedf1966e
Version Independent ID: 90a503d7-6d0f-7b9e-4c2c-45f9a9705ac7
Content: NuGet Error NU1105
Content Source: docs/reference/errors-and-warnings/NU1105.md
Product: nuget
GitHub Login: @zhili1208
Microsoft Alias: lzhi

Raised PR with information how to restore a project or solution from the commandline


where is RestrictedBoltzmannMachines.fpcd! — Method

in the document page
https://docs.juliahub.com/RestrictedBoltzmannMachines/ygJtt/0.21.1/autodocs/
There is
RestrictedBoltzmannMachines.fpcd! — Method
listed. but nowhere to be found in the code. Does it exist? maybe in an older version? Isn't it needed to reliably sample synthesized fantasy particles from random input?

what I mean is from the original paper, it is mentioned that using CD-k, you get poor results in sampling using random inputs. So they developed FPCD to improve the quality of the fantasy particles. However, from what I see in your code, you seem to get good results without leveraging FPCD and yet using random inputs.
btw, I went through you binaryRBM example on MNIST data. Have you also obtained good results for MNIST using gaussian-binary layers?


Support for openshift

@mobasherul : questions: should we do the same for 10.x and update the operator ?
I remember that these suffixes were required in 10.x to reach the pods, and I do not understand why it is not anymore.

@mobasherul : questions: should we do the same for 10.x and update the operator ? I remember that these suffixes were required in 10.x to reach the pods, and I do not understand why it is not anymore.

I will now look into 10.x as well.


update ibmapm-embed version

@tobespc Could you please help integrate this code change into the appmetrics release version?

why not make the version ^19.6.0 rather than >=19.6.0 ?

We updated our version as following format: ..
If I use ^, I still need to update the package.json next year.


[LOC] Broken link in an article

There is an broken link to GH article
Repro steps:

Go to https://docs.microsoft.com/en-us/sharepoint/dev/scenario-guidance/Branding
2 Search for "Controlling branding in SharePoint using add-in model"
click on the link

*Actual results:
Link redirects to a non existing GH article
Expected results:
Link redirects to a existing GH article
GH article:
https://github.com/SharePoint/sp-dev-docs/blob/live/docs/scenario-guidance/Branding.md

Thanks! A fix for this link has been submitted.


Feature request: selective update

Currently it's only possible to exclude certain packages.
I would like to be able to selectively include (i.e. exclude all by default except packages matching a pattern)

Great idea! I pushed it to nuget but totally forgot to actually run and test it locally. The tests are passing though so I hope I didn't broke anything 😅


Use rollup as build system

By using rollup your build config will be much simpler. Also it will allow to shake out unused helper functions (e.g., if I need only modal, then I don't want to include functions like styleTip)

That's my original idea, a cool one, however the community came up with this solution.

Feel free to contribute to the project, I'm closing this for now.


Should beforeMount etc be called when server has rendered component?

With Vue 2.0 server-side rendering, if a component has been rendered by the server should its beforeMount, mount and similar lifecycle methods be called on the client, since the component's content has technically already been mounted (by the server)?
I'm having an issue where I'm loading initial data into a vuex state for a page, but since the API request is also in the beforeMount function (to ensure it's requested when the component is re-mounted), a request for the same resource is made on the client as well when the page is first loaded.
I've had a look over the issue list, but can't find quite what I'm referring to here.

Since you are using vuex, the store should serve as a caching layer - the same fetch action should not send an ajax request if corresponding data is already in the store (inlined and hydrated from the server render).


Use first resource link in place of empty view more link

Updates the resources serializer to get the first resource, if available, to use for the view more button if there isn't an intentional value for the view more link.

If CI passes, looks good to me

Sounds good, I'll think about the order issue a little bit more, found an admin library that supports drag/drop sorting to makes things a bit smoother.


Dashboard: Template Details - Related Templates

We have enough templates with metadata to make this grid show templates that are related to the one getting viewed in some capacity. Currently they're just at random.
I think we should match template metadata and show up to 6 templates that match.
Acceptance Criteria

Display templates that are actually related (via metadata) in the template details view
Tests, please!

Verified in QA


Adds toString to FlatJSDependency.

Adding toString to FlatJSDependency per https://gitter.im/scala-js/scala-js?at=561c28dc773bc4af20ff1411)

Can one of the admins verify this patch?

That's all.

Addressed comments.

Not clear to me if there is something else this PR is waiting for.

Hum, sorry, I forgot about it. I'll have a look now.

Ok, cool.  Just wanted to make sure you weren't waiting on me.

Jenkins, test this please

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/2321/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/3010/
Test FAILed.


feat(cryptocurrency-icons): icons-for-tokens

Solana enable token support - Add icons for 6 tokens list
STLX-17040 -> https://bitgoinc.atlassian.net/browse/STLX-17040

Update description, STLX-17040 should be a link, also, description talk and ticket talk about icon for 6 tokens but you are adding 5 files, can you briefly clarify this please?

I only added 5 icons, and not 6, because one of them was already in the folder.

@joey-bai Could you please merge this PR on master?


Vegan bacon

working off https://veganbloggersunite.wordpress.com/2014/07/22/guest-blogger-cheeky-vegan-vlt-open-sandwich-seitan-bacon-lettuce-tomato-with-a-sicilian-tomato-white-bean-puree/ - very good, but needs more kick

bacon recipe needs more spice - I'm thinking black pepper

I'm declaring this tasty. further adjustments are the proverbial icing.


Make type SuperFormData importable

Is your feature request related to a problem? Please describe.
Passing a form to a component as a prop could have a better Typescript experience.
// +page.svelte
<script lang='ts'>
  import { superForm } from "sveltekit-superforms"
  export let data
  const { form } = superForm(data.form)
</script>
<MyComponent {form} />

// MyComponent.svelte
<script lang='ts'>
  // what I wish I could do
  export let form: SuperFormData<Infer<typeof schema>>

  // what I have to do
  export let form: SuperForm<Infer<typeof schema>>['form']
</script>

There could be a nice utility type possibly that acts like this
 export let form: SuperFormData<typeof schema>
Describe the solution you'd like
See above
Describe alternatives you've considered
I could just do this which I also don't want to do
<script lang='ts'>
  // just accept data.form and call superForm here instead
  export let formData: PageData['form']
  const {form} = superForm(data.form)
</script>

I'll export SuperFormData and SuperFormErrors in the next release.

Added in 2.18.0


Fix: rm behavior regarding symlinks

Fixes #519

@freitagbr what was the original behavior on the other file types, such as fifos? It looks like this approach will ignore those files, which probably isn't what we want. It may be sufficient to have one else to catch all of them (if that's consistent with previous behavior).

Let's add support for removing other file types in another PR. @freitagbr can you file the issue?

Sure, I will do that.


import declaration should not include semicolon or require line end

See https://github.com/Microsoft/vscode/issues/30527
The coffeescript grammar includes the JavaScript grammar.
With the following example:
`import Ember from 'ember'`
# foo bar baz
if true then console.log('qnx')

the JavaScript grammar is used to tokenize what's inside the backward quotes. Problem is that it does not return to the initial state after processing 'ember'. It expects a semicolon or the end of the line. Because of that, the JavaScript grammar scope remains and the rest of the file is colorized also with the JavaScipt grammar.
For supporting embedded cases like this, it would be great if the import declaration rule could end to end with the string literal or any non-expected symbol.

The import rule currently ends with either semicolon, end of the line or new line. Shouldn't the rule any ways end at lookahead of ` since the rule needs to be applied only to whats inside of template string shown in the issue? I think we are doing the right thing here.

The problem is that the JavaScript grammar gets to evaluate its rules on the full line, including the ending ''. It's not just the content of the template string. The import statement is matched, and the engine stays in the 'import' rule until it sees the line end. Only then the coffee end rule for '' is active again, but '' has been consumed already. So the end rule will not match until the next occurrence of a ''.
TextMate grammars lack a way of expressing embedded content. They just include rules from other grammars, but the problem is that these grammars don't know about the stop tokens.
Clearly, the embedded grammar (JavaScript, in this case), can't be blamed for that. But to play along, best is to exit rules as soon as possible.
https://github.com/Microsoft/vscode/issues/34224 is a similar problem.

@aeschli That doesnt seem right though... Since then any begin end rule would need to account for where it gets embedded and what to look for in the end rule? Eg. writing any thing inside say template string would need to account } as end of content and that just doesn't seem right.. There are other grammars that use end of line as end rule as well.. how will that work?


Response body with { in field of type string is incorrectly interpreted as object

When I define contract like this:
Contract.make {
      name 'Some name'
      request {
        method 'POST'
        url '/aaa'
        body([
            "field": "value"
        ])
        headers {
          contentType(applicationJson())
        }
      }
      response {
        status OK()
        body([
            "field": "value",
            "otherField"        : "{\"aa\":\"bb\", \"cc\", \"dd\"}",
        ])
        headers {
          contentType(applicationJson())
        }
      }
    }

I'd expect value of "otherField" to be of type string in mapping.
It turns it's not.
Mapping in this case looks like this:
"response" : {
    "status" : 200,
    "body" : "{\"otherField\":{\"aa\":\"bb\",\"cc\":\"dd\"}, \"field\", \"value\"}",
    "headers" : {
      "Content-Type" : "application/json"
    },
    "transformers" : [ "response-template" ]
  },

instead of
body: "{\"otherField\":\"{\"aa\":\"bb\",\"cc\":\"dd\"}\", \"field\", \"value\"}"

This causes deserialisation issues on client side, because client expects otherField to be String, and Jackson finds START_OBJECT, and throws:
com.fasterxml.jackson.databind.exc.MismatchedInputException: Cannot deserialize instance of `java.lang.String` out of START_OBJECT token
 at [Source: (PushbackInputStream); line: 1, column: 18] (through reference chain: a.b.c.Response["otherField"])

I see this for version 2.1.0, I was not able to check version 2.1.4.

Please check the latest version before filing an issue. If the issue got already fixed in 2.1.4 why do you even file this issue? I'm closing this until you verify against the latest released version.

I was able to verify that issue is still present on 2.1.4.
It seems to boil down to "if string starts with { the it's treated as object instead of string".

Can you please create a small sample?
Cause  maybe I'm wrong but with 2.1.4 I see in a test that I get a following mapping
{
  "id" : "8267eee4-91f4-4a45-85aa-30d2b3fe6a7e",
  "request" : {
    "url" : "/aaa",
    "method" : "POST",
    "headers" : {
      "Content-Type" : {
        "matches" : "application/json.*"
      }
    },
    "bodyPatterns" : [ {
      "matchesJsonPath"<EMAIL_ADDRESS>== 'value')]"
    } ]
  },
  "response" : {
    "status" : 200,
    "body" : "{\"field\":\"value\",\"otherField\":\"{\\\"aa\\\":\\\"bb\\\", \\\"cc\\\", \\\"dd\\\"}\"}",
    "headers" : {
      "Content-Type" : "application/json"
    },
    "transformers" : [ "response-template", "foo-transformer" ]
  },
  "uuid" : "8267eee4-91f4-4a45-85aa-30d2b3fe6a7e"
}

it seems to be correct?

sure, try this -> SCC-SAMPLE

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


GH Discussions

[x] I have checked that a similar feature request does not already exist.

Is your feature request related to a problem? Please describe.
The only contact channels are Reddit and Twitter, which require register/sign-up on their corresponding platforms.

Twitter seems pretty unstable, "thanks" to Elon; and may require payments in the future
Reddit is simply bad, "thanks" to its CEO

There are many other reasons why users may not like (or even outright hate) to be forced to use any of those platforms
Describe the solution you'd like
Enable GitHub Discussions. It's built into this platform. Most users of scrcpy will very likely have a GH account and would love to use it!
Describe alternatives you've considered
Adding more platforms, such as Matrix or Discord
Additional context

The only contact channels are Reddit and Twitter,

No, these are secondary channels. The main channel is via GitHub issues.

Enable GitHub Discussions.

What's the benefit over GitHub issues? Very often, it is not clear whether something is an "issue" or a "discussion". What about discussions about an issue?

No, these are secondary channels. The main channel is via GitHub issues.

Are you sure?
https://github.com/Genymobile/scrcpy/blob/cc8e6133b0c9975ef2ed16086960e822ab750404/README.md?plain=1#L149-L158
It seems Issues are mainly for bug-reports and feature-requests, not questions or discussions

What's the benefit over GitHub issues?

Questions, such as #5118

Very often, it is not clear whether something is an "issue" or a "discussion"

I can relate 😅. It's hard to decide

While I think GitHub discussions might be more appropriate than "issues" for many questions (many users just seek help to make scrcpy work on their device), I fear that it will create confusion/duplication. Moreover, in theory, a lot of existing issues should be moved to discussions, but I will not review the 5000+ issues to clean up.
So for now, I think I will only keep issues, but add an explicit (empty) "Question" template:
diff --git a/.github/ISSUE_TEMPLATE/question.md b/.github/ISSUE_TEMPLATE/question.md
new file mode 100644
index 000000000..14dc373a3
--- /dev/null
+++ b/.github/ISSUE_TEMPLATE/question.md
@@ -0,0 +1,8 @@
+---
+name: Question
+about: Ask a question about scrcpy
+title: ''
+labels: ''
+assignees: ''
+
+---

and reword the `Contact` section:

```diff
diff --git a/README.md b/README.md
index 3185652b9..36c8c272b 100644
--- a/README.md
+++ b/README.md
@@ -148,11 +148,14 @@ documented in the following pages:

 ## Contact

-If you encounter a bug, please read the [FAQ](FAQ.md) first, then open an [issue].
+You can open an [issue] for bug reports, feature requests or general questions.
+
+For bug reports, please read the [FAQ](FAQ.md) first, you might find a solution
+to your problem immediately.

 [issue]: https://github.com/Genymobile/scrcpy/issues

-For general questions or discussions, you can also use:
+You can also use:

  - Reddit: [`r/scrcpy`](https://www.reddit.com/r/scrcpy)
  - Twitter: [`@scrcpy_app`](https://twitter.com/scrcpy_app)

What do you think?

I will not review the 5000+ issues

I agree, that's too much work 😅

"Question" template

That's a good solution! Now, I'm unsure if I should close this as "completed" or "not planned". According to the title, it should be "not planned", but the intention was "completed" in another way


Developer Guide 2.0

Updated all the open source software (Grafana, InfluxDB, NodeRed)
Changed NodeRed flows due to software update
Add additional security aspect in docker-compose.yml: Linux capabilities, read-only file system, non-root user, container initialization
Add tracee
Delete MQTT external networks in docker-compose
Separate basic example and security extension into HandsOn_1 and HandsOn_2 folder
Add Demo OPCUA Server Simulator and add OPCUA Client Flows in NodeRED

Co-authored-by: Si Hao Shen<EMAIL_ADDRESS>Co-authored-by: Ondrej Havlicek<EMAIL_ADDRESS>Co-authored-by: Daniel Stus<EMAIL_ADDRESS>See Pull request #13 for details.

this should fix #16

I found that adding he testtoken to configure nodered password is not enough. I guess you tested it with different version of NodeRED.
I think you should include also the lockfile (lock.json) and not just package.json in the NodeRed repository to avoid these problems in the future.

my bad, I had some old images in my dev enviroment. It works fine.

This flow uses outdated config for MQTT nodes. https://github.com/industrial-edge/Developer-Guide-Hands-on-App/blob/devguide-1.5-wip/src/solution/HandsOn_2/IE Flow Creator Flows/flow_data_S7_Connector.json

@StushaaK  @HummelN7872  Everything should be finished. Developer guide is already released. https://docs.eu1.edge.siemens.cloud/developer/00_Overview.html


UI for committee member section

GOSA5
ISSUE #6
Describe the changes you've made
I have made the UI for committee members , along with a constraints page . In the committee members list i have made black sections where members can fill their description  , photo , name , designation and their basic social media handles.
All members are in a list view.
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Additional context (OPTIONAL)
Screenshots added

Test plan (OPTIONAL)
A good test plan should give instructions that someone else can easily follow.
How someone can test your code?
Checklist

[x] My code follows the code style of this project.

[x] My change requires a change to the documentation.

[x] I have updated the documentation accordingly.

[x] I have read the CONTRIBUTING document.

[ ] I have added tests to cover my changes.

[ ] All new and existing tests passed.

[x] The title of my pull request is a short description of the requested changes.

ok doing it

changes made!


Dictionary not available in NewPipe

When using NewPipe and i try to perform a search, the selected dictionary is not available.

I don't know if this is specific of newpipe or if it happens in other apps. NewPipe is the only app i found this problem so far

This is something with NewPipe. I suppose the search field is of type "TYPE_TEXT_VARIATION_FILTER" or some unrecognized type, so TT9 just allows "simple" typing, but I am not 100% sure. I'll investigate further, when I have time.

@sspanak regarding this: if i can help in anyway, let me know about it!

Oh, my previous comment is just a note to myself, so that I don't forget what to look for.

if i can help in anyway, let me know about it!

If you have programming skills, you could check TextField.determineInputModes() and find what's the reason for it not to include MODE_PREDICTIVE in returned list. If not, you just have to wait. I don't have so much free time to work on TT9 lately. Sorry!

If not, you just have to wait.

That's what I'll do then! I have no skills in Android codding (I had some Java on university, but nothing that helpful! I'm more comfortable on C++ and Sinclair BASIC ahahah... and even on those I'm not an expert either 🤣 )! Sorry!


Add Keras docker image

This is an initial implementation of the Keras image. It has some strong assumptions (such as the full model being saved), but I would like to merge an initial working implementation asap and follow-up with image-captioning.
@Rocketknight1 could you take a quick look at pipelines/image_classification.py please?

Friendly ping @Narsil.

FYI @nateraw

@osanseviero had opportunity to check, everything seems good, let's merge ! :D


An error after facebook authorization

I'm getting an error after authorization via Facebook:
Undefined index: link in ...../vendor/nodge/yii2-eauth/src/services/FacebookOAuth2Service.php at line 55
As I understood this is because the settings of my facebook application, but I can't find such option. Can you help me? I think it will be useful to write about setting your extension needs in readme file.

Hi Mike. I have the same issues... How are you fix it?

Vladimir, I didn't find information about settings. I wrote my own FacebookOAuth2Service to avoid this. It doesn't read this parameter from facebook, but compose it by itself. It's a very bad solutuion, but I can't find other.

Mike, thank you for your response.
I am too, made my own module.


the precision of fee calc

https://github.com/galacticcouncil/HydraDX-math/blob/main/src/fee.rs#L17
The order of the math ops can be switched to increase the precision

Create companions for Basilisk, as XYK and other stuff using this would need to updated accordingly.


Polish selection controls

The touch ripple now starts at the touch location instead of being
centered on the widget.
Fixes #2652

@HansMuller

@Hixie


sparseMatrix breaks plotAbundance(Density)

Hi :)
SummarizedExperiment objects are allowed to contain sparseMatrix objects instead of base matrix objects, but this breaks plotAbundance and plotAbundanceDensity:
library(Matrix)
library(miaViz)

data(GlobalPatterns, package = "mia")
plotAbundanceDensity(GlobalPatterns) # works
plotAbundance(GlobalPatterns) # works

assay(GlobalPatterns) <- as(assay(GlobalPatterns), "sparseMatrix")
plotAbundanceDensity(GlobalPatterns)
# Error in t.default(mat) : argument is not a matrix
plotAbundance(GlobalPatterns)
# Error in as.data.frame.default(.) :
#   cannot coerce class ‘structure("dgCMatrix", package = "Matrix")’ to a data.frame

Similar to https://github.com/microbiome/mia/issues/151
Thank you!
Best,
Bela

Ah, perfect - we will need to check asap, thanks!

Hi!
Thanks for noticing this and thanks for nice example. This can be fixed by converting sparseMatrix into matrix.
-Tuomas

Close the issue when solved.

If the PR is ok, then it can be merged and this issue can be closed

Ah yes. Somehow I had missed the review. FE will make the final judgment.

Oh, this is miaViz. I think we can accept the PR already.


Fix Out of bounds error with boolean_limit mentioned in issue #512

This change was proposed by @oubathAI but it was never submitted as a PR, see the discussion on issue #512.
Sanity

[x] I'm aware of the implications of the proposed changes
[x] Code is PEP8
[x] I'm making the PR to master

Docs

[ ] Docs are updated
[ ] Docs version is correct (index.html and _coverpage.md)

Tests

[x] Changes have gone through actual use testing
[x] All local tests have passed (run ./test.sh in /talos)
[ ] Tests have been updated to reflect the changes

Why did the greeting fail?

This is now properly handled in #583. The issue actually had to do with the fact that for historical reasons there were two ways in which the param_space is created. Now there is only one, and the issue in here will go away with that.


Support Failed:Retry for FIFO Queues

Start from the Use-case
First, I'll like to say thank you for such an amazing tool.
I have a FIFO queue and its corresponding DLQ.
When I run serverless myqueue:failed:retry, I get the error message: The request must contain the parameter MessageGroupId, which makes sense because every message sent to FIFO queues must have a MessageGroupId.
Is there a way to achieve this already?
Taking a look at the retryMessages function, I don't think there is.
Implementation Idea

Add a check to see if the queue is a FIFO queue or the message in the DLQ has MessageGroupId
If so, then update the returned entry here
For example:

 const entry = {
           Id: message.MessageId,
           MessageAttributes: message.MessageAttributes,
           MessageBody: message.Body,
  };

 if (message.MessageGroupId != undefined) {
         entry.MessageGroupId = message.MessageGroupId;
 }

 return entry;

@JohnLawal thanks for opening this, that sounds like a nice improvement indeed!
I think it might be enough to check if the message to retry has a MessageGroupId, because AFAIR FIFO queues can be configured to use an existing attribute to deduplicate messages. So MessageGroupId is optional I think (depending on the FIFO queue config). But maybe I'm mixing things up.

Hi @mnapoli,
MessageGroupId is required for FIFO queues. I think you might be referring to MessageDeduplicationId which is optional if ContentBasedDeduplication is enabled for the FIFO queue.
However, this makes it enough to check if the message to retry has a MessageGroupId since the property must have been provided already.

👍  my bad, thanks for clarifying!

Ditto today
$ serverless incoming-things:failed:retry --stage prelive --aws-profile prelive --verbose
Running "serverless" from node_modules
Environment: linux, node 18.14.2, framework 3.22.0 (local) 3.25.1v (global), plugin 6.2.2, SDK 4.3.2
Credentials: Local, "patientpass-serverless-live" profile
Docs:        docs.serverless.com
Support:     forum.serverless.com
Bugs:        github.com/serverless/serverless/issues

Error:
The request must contain the parameter MessageGroupId.

This monkey patch seemed to work
$ diff ./node_modules/serverless-lift/dist/src/constructs/aws/queue/sqs.js.orig ./node_modules/serverless-lift/dist/src/constructs/aws/queue/sqs.js
81a82,92
>       // A FIFO Queue
>       if( typeof message.MessageGroupId != undefined ){
>         return {
>           Id: message.MessageId,
>           MessageAttributes: message.MessageAttributes,
>           MessageBody: message.Body,
>           MessageGroupId: message.MessageGroupId
>         };
>       }
>

Although, messages in my DLQ do not appear to have a MessageGroupId regardless of the Q's setting...

Updated PR and monkey patch
$ diff ./node_modules/serverless-lift/dist/src/constructs/aws/queue/sqs.js.orig ./node_modules/serverless-lift/dist/src/constructs/aws/queue/sqs.js
74a75
>
81a83,93
>
>       // A FIFO Queue
>       if( queueUrl.indexOf('.fifo') > -1 ){
>         return {
>           Id: message.MessageId,
>           MessageAttributes: message.MessageAttributes,
>           MessageBody: message.Body,
>           MessageGroupId: message.MessageGroupId?message.MessageGroupId:'redrive'
>         };
>       }
>

The PR fails to compile because TypeScript is stupid, and I don't know enough about it to tell it to stop caring about what it thinks the types are.


feature support aggregate function group_array_intersect

Proposed changes
Issue Number: #31737

Consider this table named group_array_intersect_test, which has id and c_array_int columns:
+------+-----------------+
| id   | c_array_int     |
+------+-----------------+
|    0 | [0]             |
|    6 | [null]          |
|   12 | [12, null, 13]  |
|   14 | [12, 13]        |
+------+-----------------+

we can use group_array_intersect to get the intersect element(s) from given array like these queries:
mysql> select group_array_intersect(c_array_int) from group_array_intersect_test where id in (6, 12);
+------------------------------------+
| group_array_intersect(c_array_int) |
+------------------------------------+
| [null]                             |
+------------------------------------+
1 row in set (0.02 sec)

mysql> select group_array_intersect(c_array_int) from group_array_intersect_test where id in (14, 12);
+------------------------------------+
| group_array_intersect(c_array_int) |
+------------------------------------+
| [13, 12]                           |
+------------------------------------+
1 row in set (0.01 sec)

mysql> select group_array_intersect(c_array_int) from group_array_intersect_test where id in (0, 6);
+------------------------------------+
| group_array_intersect(c_array_int) |
+------------------------------------+
| []                                 |
+------------------------------------+
1 row in set (0.00 sec)

Furthermore, this function supports all kinds of array type, not only array(int) mentioned above, but also array(varchar()), array(date) and so on.


Update to<EMAIL_ADDRESS>

SSIA

@w-haibara
Could you review this PR? :pray:

@a-thug Sorry for the late response..


Use yield statement instead of finalizer

to specify teardown code

Current coverage is 96.19% (diff: 100%)

Merging #964 into master will not change coverage

@@             master       #964   diff @@
==========================================
  Files            35         35
  Lines          1842       1842
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1772       1772
  Misses           70         70
  Partials          0          0

Powered by Codecov. Last update b4063dd...9cc3eca


Fix outdated URLs

Fixes #1643, fixes #1645
Fix outdated URLs in source code + docs
After this, all URLs found by the regex from #1645 are working.

Codecov Report

Merging #1708 (8ed89bd) into unstable (2eb4306) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##           unstable    #1708      +/-   ##
============================================
- Coverage     76.10%   76.09%   -0.01%
============================================
  Files           353      353
  Lines         11316    11316
  Branches        553      553
============================================
- Hits           8612     8611       -1
- Misses         2704     2705       +1

Impacted Files
Coverage Δ

...che/tla/bmcmt/config/CheckerExceptionAdapter.scala
0.00% <ø> (ø)

...n/scala/at/forsyte/apalache/tla/lir/TlaType1.scala
92.77% <ø> (ø)

...signments/SmtFreeSymbolicTransitionExtractor.scala
66.12% <100.00%> (ø)

...he/io/annotations/parser/CommentPreprocessor.scala
95.60% <0.00%> (-1.10%)
:arrow_down:

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 2eb4306...8ed89bd. Read the comment docs.


[tool] Add a way to opt a file out of formatting

Per discussion on Discord, this adds a way to opt-out of the formatting.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Note that unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[ ] All existing and new tests are passing.

I made this apply to all files rather than just Dart, mostly because it was easier to plug into the code where File objects were still around. I considered pushing the File-to-String conversion lower down the codepaths, but that ended up being pretty invasive and not obviously better. I also considered just having the Dart code go back from String to Files but that ended up being pretty ugly too.

PTAL


got error when post message with chat id

when i use PostMessage with BindChatID

got error. the chat id is<PHONE_NUMBER>812337180. tenant is bytedance.

@gd0131 the last screenshot is with PSM info, so I removed for you.

@gd0131 It looks like the code you post is okay. What's your chatID, is it oc_xxxxx or just an int64 number? It should be oc_xxxxx.

7311993580812337180. We've been using this kind of chatID（ int64 number） before with no problems. Did something change recently

7311993580812337180. We've been using this kind of chatID（ int64 number） before with no problems. Did something change recently

It has changed years ago since the release of Open API v2. The in-house version provides an ID conversion method to convert to this ID to the latest ChatID.

Can you elaborate on how I should accomplish the id conversion，Thank you very much!

Can you elaborate on how I should accomplish the id conversion，Thank you very much!

There is document in in-house Lark group. But I cannot post here.

I understand if I need to do an id conversion through lark's interface, I've found the interface documentation.


[Bug]TubeMQ HTTP API download link cannot be opened

Description
获取HTTP API文档的链接打不开
问题地址：https://inlong.apache.org/zh-CN/docs/modules/tubemq/appendixfiles/http_access_api_definition_cn.xls

InLong Component
InLong Website
Are you willing to submit PR?

[ ] Yes, I am willing to submit a PR!

Code of Conduct

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

@leno1001 thanks! wait a minute

@leno1001  this issue is fixed by https://github.com/apache/incubator-inlong-website/pull/173, it will close.


fix inner brackets in conditionals concept

@saugatdhimal I am sorry, I have to close this PR. Some else picked up the issue around the same time and posted in the issue so I merged that contribution. Please always post in the issue that you will be working on it before starting the work to avoid this in the future.


take advantage of i/o gates for counter

closes https://github.com/cloudflare/durable-objects-template/issues/11

Is there automatic caching done for storage? Otherwise it doesn't seem ideal for a template that the common '/' read path will hit the storage API instead of simply reading from a this.value cache like before?

@rib refer to https://developers.cloudflare.com/workers/learning/using-durable-objects#accessing-persistent-storage-from-a-durable-object
Bullet point starting "The storage API implements an in-memory caching layer..."

Created similar PRs on the other templates.

Should we maybe add some comments in the code explaining why this works? So that we don't get a bunch of bug reports saying that we have race conditions / bad performance? :)

My thought with the comment was not that we'd just link people to a long blog post but try to explain what's happening in context. Example code really should be full of detailed comments IMO. I'll make a PR...

@kentonv ok if you want to PR to this repo, I can make the same change in the other PRs / make a new one for the counter example in the docs

So this mean current docs is not up to date? Or I still need blockConcurrencyWhile there?
https://developers.cloudflare.com/workers/learning/using-durable-objects/#in-memory-state-in-a-durable-object

@katopz You probably need blockConcurrencyWhile() if you want to do initialization in the constructor -- as shown in that doc.
In this example code, it turns out to be easier to read the counter value at request time rather than in the constructor, and simply rely on I/O gates and caching to make it fast and correct. But many other types of applications still want to do constructor-time initialization with blockConcurrencyWhile().


add transformer for namespace operator

Added failing test cases for namespace operator and a NamespaceOperator transformer to fix them.

namespace is a reserved work, so it cannot be used as the name of a namespace itself: http://3v4l.org/3hgMW

Did you click on the first link? namespace can be an operator for the current namespace.

:+1:

Thank you @gharlan.


Fixed error not showing when transaction could not be recorded

Codecov Report

Merging #266 (28041b2) into master (55c00b6) will not change coverage.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files

@@           Coverage Diff           @@
##           master     #266   +/-   ##
=======================================
  Coverage   52.30%   52.30%
=======================================
  Files          87       87
  Lines        9891     9891
=======================================
  Hits         5173     5173
  Misses       4306     4306
  Partials      412      412

Flag
Coverage Δ

unittests
52.30% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

monitor_event_handler.go
0.00% <0.00%> (ø)

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 55c00b6...28041b2. Read the comment docs.


[Question] Passing refs down deeply nested view tree in order to call waitFor

Say we have a TabView implementation that is based on a react-native-gesture-handler ScrollView.
Is it the case components (that use PanGestureHandler for example) on each page need to pass their refs to the TabView so the TabView can include them in the ScrollView's waitFor prop?
Is there a recommended design pattern for managing all the refs?

@siderakis Did you end up figuring this out? Specifically I'm looking to have a separate PanGestureHandler and a ScrollView's GestureHandler to activate at the same time

I'm also stuck. I have a PanGestureHandler which has a descendent which is a ScrollView, but I cannot scroll in this ScrollView, the pan always is taking precedence. I even created a ref and passed it deeply down and then did:
<PanGestureHandler waitFor={ref}>
    ....
    <ScrollView ref={ref} />
    ....
</PanGestureHandler>

I'm on Android, using RN 0.59.5.


Generate Report of Student Submission

This would be higher priority than #9 ...

Please push to branch whenever you have something on reports so that I can take a look. I promise I won't mess with your branch =]

Sure, I will keep the remote branch updated. =]

Pull request added. #23

Great work with the report. I really like the "Edit" button to fix any mistakes by the autograder.
Some changes:

Should add the "username" column along with roll no.
Add "Submission date". This would be for the latest ... just as the pass/fail fields.
Make table sortable on all columns (client-side).
For "percent", I forget: are we actually displaying "percent" or total points (given that we have assignment score in the meta data for the assignment. I think we should be displaying actual "score" instead of percentage.

BTW, did you check with multiple students/assignments to ensure we're not getting any mixups?

(very low priority: syntax highlighting for the code shown in the popup in report.)

#24

Added username
Added "Submission date"
Table are sortable now
Score instead of percent
Code highlight

Assignment report seems to have a bug:
def get_student_latest_submissions(self):
        # TODO: Try to acheive this by sub queries using Django ORM
        # Get students of this course
        students = Student.objects.filter(courses=self.course)
        submissions = []
        for student in students:
            submission = Submission.objects.filter(student=student).order_by("-publish_date").first()
            if submission:
                submissions.append(submission)

It's not checking for a particular assignment. We're getting back whatever happens to be the latest submission. So, if you submit an older assignment later, that shows up.

The way we need to do this is to find the course associated with the assignment, then find all students that are enrolled and then get their score. In case there is no record of submission by a student, we show a - in their row. This is so that we know exactly which students have submitted and not omit those that haven't made any submission.

Fixed the duplicate in #31 and Email verification is fixed in #30 using Outlook SMTP.

Let's close this.


Protobuf unmarshalling on websocket listener crashes in release mode

My program works fine in debug mode, but in release mode it crashes. If I comment the mergeFromBuffer line then it works:
   _listener = ws.stream.listen(
      (data) {
        print(data);
        var chg = new chgpb.AppStateChange();
        chg.mergeFromBuffer(data);
      },
      onError: (error) { print(error); },
      onDone: () { print("draft incoming listening done"); },
      cancelOnError: false,
    );

ws is an IOWebSocketChannel.

Stack frame #01 pc 000025ac  /data/data/draft.draftapp/app_flutter/vm_snapshot_instr

I do not have at the moment a completely stripped down version to offer as a repro. I also don't know if the crash is related to the protobuf unmarshalling code, or is it just because the unmarshalling takes some cpu time. I'll work on it, but in the meantime I thought I'll file this --- creating a stripped down repro may take a day or two, and maybe somebody has an idea how I should proceed understanding this issue in the meantime?
It doesn't crash immediately - it receives about a dozen packets from the websocket server, and then it crashes.
I am not using isolates. The UI code is super simple, it doesn't do anything, and no state changes happen - the program starts, starts listening from the websocket from the initState of a stateful widget, and crashes. The code above shows that I already removed any effect of the data coming from the websocket, so it does not cause widget rebuilds.

The protobuf issue proved to be a vm/compiler issue, and now has been fixed: https://github.com/dart-lang/protobuf/issues/95 .
However, the crash in my original post still happens.
I found that it can be worked around like this:
initialize(WebSocketChannel ws) async {
    ws.stream.forEach((data) {
        var chg = new chgpb.AppStateChange();
>>>     var d = new Uint8List.fromList(data);    <<<
        chg.mergeFromBuffer(d);
        // The rest is removed, mergeBuffer already causes the crash if the "var d = ..." is not there.
    });
  }

If you read it and have some idea which part of the stack I should suspect to cause this behavior, please don't hesitate to chime in... Thank you!

It was another compiler bug. Compiling flutter engine with Dart from head solved the issue. The fix for this is from a few hours ago: https://github.com/dart-lang/sdk/commit/c1b873bb9115242b86aee4e40433bd7f55663638
With these fixes, now my app doesn't crash anymore.


Add memorydb _user

Description of your changes
Add memorydb user.
Fixes #1149, https://github.com/crossplane-contrib/provider-upjet-aws/issues/368
I have:

[ ] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
With uptest: https://github.com/crossplane-contrib/provider-upjet-aws/actions/runs/8444367117

/test-examples="examples/memorydb/v1beta1/user.yaml"

@sergenyalcin  The upstream upjet fixes work for creation with one password set. However, if two passwords are set only one is created and the diff is never calculated:
apiVersion: memorydb.aws.upbound.io/v1beta1
kind: User
metadata:
  annotations:
    meta.upbound.io/example-id: memorydb/v1beta1/user
  labels:
    testing.upbound.io/example-name: example
  name: example
spec:
  forProvider:
    accessString: "on ~* &* +@all"
    authenticationMode:
    - passwordsSecretRef:
      - key: password
        name: example-secret
        namespace: upbound-system
      - key: password
        name: memorydb-user-secret-2
        namespace: upbound-system
      type: password
    region: us-west-1
  writeConnectionSecretToRef:
    name: user-conn
    namespace: default
---
apiVersion: v1
kind: Secret
metadata:
  name: example-secret
  namespace: upbound-system
type: Opaque
stringData:
  password: TestCr0ssplanePassw0rd!  # Passwords must be a minimum of 16 characters
---
apiVersion: v1
kind: Secret
metadata:
  name: memorydb-user-secret-2
  namespace: upbound-system
type: Opaque
stringData:
  password: TestCr0ssplanePassw0rd!  # Passwords must be a minimum of 16 characters

spec/status don't match:
spec:
  deletionPolicy: Delete
  forProvider:
    accessString: on ~* &* +@all
    authenticationMode:
    - passwordsSecretRef:
      - key: password
        name: example-secret
        namespace: upbound-system
      - key: password
        name: memorydb-user-secret-2
        namespace: upbound-system
      type: password
    region: us-west-1
    tags:
      crossplane-kind: user.memorydb.aws.upbound.io
      crossplane-name: example
      crossplane-providerconfig: default
  initProvider: {}
  managementPolicies:
  - '*'
  providerConfigRef:
    name: default
  writeConnectionSecretToRef:
    name: user-conn
    namespace: default
status:
  atProvider:
    accessString: on ~* &* +@all
    arn: arn:aws:memorydb:us-west-1:609897127049:user/example
    authenticationMode:
    - passwordCount: 1
      type: password
    id: example
    minimumEngineVersion: "6.2"
    tags:
      crossplane-kind: user.memorydb.aws.upbound.io
      crossplane-name: example
      crossplane-providerconfig: default
    tagsAll:
      crossplane-kind: user.memorydb.aws.upbound.io
      crossplane-name: example
      crossplane-providerconfig: default
  conditions:
  - lastTransitionTime: "2024-02-27T16:51:45Z"
    reason: Available
    status: "True"
    type: Ready
  - lastTransitionTime: "2024-02-27T16:51:43Z"
    reason: ReconcileSuccess
    status: "True"
    type: Synced
  - lastTransitionTime: "2024-02-27T16:51:44Z"
    reason: Success
    status: "True"
    type: LastAsyncOperation

Only one password is created in AWS:

/test-examples="examples/memorydb/v1beta1/user.yaml"

Thanks for the PR @stevendborrelli. It looks like the uptest bump and fix to the elasticache user has already been merged in a different PR. Can you please rebase on the latest main?

/test-examples="examples/memorydb/v1beta1/user.yaml"

@turkenf removed the entry from external names.


Error to send

Hi, Congratulations on the project, I can not make it work on Android 6 and 8 and on WhatsApp Normal and Business, it gives an error and closes when clicking send. Does the cell phone need to be rooted?

Not actually, because it uses accessibility permissions in case of unrooted phones. It would be better if you describe more about the error you are getting. If possible logcat or screenshot is more helpful.

Hi, I am facing the same issue. Here are the screen shots.

Will you be able to provide the logs also? It would be easier to reproduce
the same and debug then.
On Tue, 5 Mar 2019, 15:00 mubi64<EMAIL_ADDRESS>wrote:

Hi, I am facing the same issue. Here are the screen shots.
[image: screenshot_20190305-142747]
https://user-images.githubusercontent.com/17858365/53795123-1e9dbc00-3f53-11e9-8b3e-d53d7eca952f.png
[image: screenshot_20190305-142738]
https://user-images.githubusercontent.com/17858365/53795125-1e9dbc00-3f53-11e9-8dc7-07cc65eead6e.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/nikhilmuz/WhatsApp-Bulk-Sender/issues/18#issuecomment-469607001,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATNMF47kjw_SpDFgqMpFc8MJm5bv65s-ks5vTjkegaJpZM4a8tsw
.

After press button send . It just off app.  O uaed mi p2 . Handset .android 5.1.1

I got  error on chrome extension after loading pop-up . It's not open upload button

Though it should not be the case, you may share the error message you are
getting.
On Fri, 7 Jun, 2019, 5:35 AM chandan kandar<EMAIL_ADDRESS>wrote:

After press button send . It just off app. O uaed mi p2 . Handset .android
5.1.1
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/nikhilmuz/WhatsApp-Bulk-Sender/issues/18?email_source=notifications&email_token=AEZUYF5OES6WRKRUBZGRWXTPZGQ3JA5CNFSM4GXS3MYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXEQDXA#issuecomment-499712476,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEZUYF4SNJSXE53NNJJXHSLPZGQ3JANCNFSM4GXS3MYA
.

The extension won't work on recent version of whatsapp web. I will try to
fix this.
On Fri, 7 Jun, 2019, 5:37 AM chandan kandar<EMAIL_ADDRESS>wrote:

I got error on chrome extension after loading pop-up . It's not open
upload button
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/nikhilmuz/WhatsApp-Bulk-Sender/issues/18?email_source=notifications&email_token=AEZUYFYJSLRIVQLLB5NWS3LPZGRCPA5CNFSM4GXS3MYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXEQG7A#issuecomment-499712892,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEZUYF6TADYQUTKXHUN3323PZGRCPANCNFSM4GXS3MYA
.

Yess it's show me error message unfortunately . WhatsApp Sender has stopped.  And apps go to off

@9804701373 The android app is working fine. You must have done something wrong following the Readme file. Specifically CSV file. @eduardofiorini @mubi64


Runtime error "'numpy.int64' object has no attribute 'ex_id'"

Describe the bug
Running the pipeline below leads to the error
[da24eb50-a015-438e-bfb2-5f47ac2175a3] 'numpy.int64' object has no attribute '__ex_id__'
	at E:\Repositories\safe-ds\Runner\src\safeds_runner\server\_pipeline_manager.py line 299
	at <frozen runpy> line 226
	at <frozen runpy> line 98
	at <frozen runpy> line 88
	at bug/gen_bla_rent line 4
	at bug/gen_bla line 25 (mapped to 'bla.sds' line 5)
	at E:\Repositories\safe-ds\Runner\src\safeds_runner\server\_pipeline_manager.py line 436
	at E:\Repositories\safe-ds\Runner\src\safeds_runner\server\_memoization_map.py line 177
	at C:\Users\Lars\AppData\Local\pypoetry\Cache\virtualenvs\safe-ds-runner-KD7s4J4Y-py3.12\Lib\site-packages\safeds\data\tabular\containers\_table.py line 1147
	at bug/gen_bla line 18 (mapped to 'bla.sds' line 5)
	at E:\Repositories\safe-ds\Runner\src\safeds_runner\server\_pipeline_manager.py line 436
	at E:\Repositories\safe-ds\Runner\src\safeds_runner\server\_memoization_map.py line 181
	at E:\Repositories\safe-ds\Runner\src\safeds_runner\server\_memoization_utils.py line 432
	at E:\Repositories\safe-ds\Runner\src\safeds_runner\server\_memoization_utils.py line 269

To Reproduce

Run this pipeline:

package bug

pipeline rent {
    val houseRentData = Table({"Rent": [1,2]});
    val test = houseRentData.filterRows((r) -> ((r.getValue("Rent") as Int)<300000));
}

Expected behavior
No error.
Screenshots (optional)
No response
Additional Context (optional)
No response

:tada: This issue has been resolved in version 0.13.1 :tada:
The release is available on:

v0.13.1
GitHub release

Your semantic-release bot :package::rocket:


Class map benchmarks

Context
Reproduce benchmark raised in issue: https://github.com/lit/lit/issues/2010
I've used the kitchen-sink and lit-element prior benchmark as a guide, and then used the Lit.dev playground example as a base for this benchmark.

As mentioned in-person... let's try to force a style recalc, but not a paint, to make sure that we flush any potentially deferred browser work (like parsing class names).

This draft PR was used to benchmark classMap against raw string manipulation.
To reproduce a run, git checkout the commit you want. Build the benchmark (npx tsc) and run the benchmark (npm run benchmark:lit-element:class-map) from lit/packages/benchmarks package.


Repo still uses teleport v2.0.0-alpha.4

It looks like the repo still using an alpha version of teleport release.

It is in the Makefile: https://github.com/gravitational/teleconsole/blob/master/Makefile#L14


Allow specifying target-dependent build.rs scripts

Currently it is possible to specify something like following in Cargo.toml:
[target.'cfg(unix)'.build-dependencies.cc]
version = "1.0"

however it is not feasible to use this functionality, because you cannot conditionally extern use crates or specify per-target build.rs scripts (i.e. the following does not work):
[target.'cfg(unix)'.package]
build = "build/unix.rs"

Another good reason to have target-dependent build-scripts: currently use of build scripts will force compilation of a dependency that's used during cargo check, which costs a fair amount of extra time. This cost does not need to be paid for by targets that don't benefit from the build script.

I met this issue when writing a build.rs and .cargo/config.toml for a project cross-compiling from x86_64 linux to aarch64 android.
The reason why I need target-dependable build.rs is:
println!("cargo:rustc-link-search=native=platform/specific/lib/path");
println!("cargo:rustc-link-lib=static=some_aarch64_lib");


Epic: Move AKS, EKS & GKE from Ember to Vue

Currently, to provision public cloud clusters (AKS, EKS & GKE), the cloud provisioning pages are iFramed from Ember into Vue.

[x] https://github.com/rancher/dashboard/issues/8718
[ ] AKS: rancher/dashboard#8965
[x] EKS: rancher/dashboard#8966
[x] GKE: rancher/dashboard#8967
[x] Cluster & Node Drivers: https://github.com/rancher/dashboard/issues/9274

Since there's a lot of active work done on those platforms but we'd like to move away from Ember, we should port these over to Vue.
This ties into work done overall to evaluate other pages to move off Ember and onto Vue: https://github.com/rancher/dashboard/issues/8718

Just a note - cluster drivers affect the aks / eks / gke world, and node drivers RKE1 and RKE2

Started with AKS for Q4.

Discussed with @yonasberhe23 - this was the epic - the individual issues are being tested by QA


Fix duplicated tests

Is this supposed to be merged ?

Let me do another build... just to be 100% that those duplicates are gone...

@tristantarrant Done.
I also had to disable stress tests. Some of them were killing Jenkins Agents. Let's finish Jenkins migration first and then implement stress tests.

Should be good now, merging.


Support rotation canter is not geometric center?

Description
Set Component.anchor to Anchor.center, the rotation center is geometric center.
Rotation canter of Turrets on Tower always is not geometric center. The rotation is not desired.
Development environment
Flame version: 1.0.0-rc6

Flutter doctor:
[✓] Flutter (Channel master, 1.27.0-2.0.pre.79, on macOS 11.0.1 20B50 darwin-x64, locale en-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 3.5)
[✓] IntelliJ IDEA Community Edition (version 2019.1.3)
[✓] VS Code (version 1.53.2)
[✓] Connected device (2 available)

• No issues found!

Runtime
This issue is related to running on which platform? (Select all that apply)

[✓] Android :android:
[✓] iOS :apple:
[✓] Web :browser:

Minimal reproducible code (Required for bugs)
gun = SpriteComponent();
gun.anchor = Anchor.center;
gun.addEffect(RotateEffect(angle: 7, duration: 5));

Hi,
I don't understand what the issue is here, could you try to re-phrase it?
As far as I can see you set the anchor to the center, and it rotates around the center, what is your expected outcome?

Sorry for my poor english.
|-------|
|           |
|     A    |
|     B    |
|-------|

The turret's rotate center is B, but it rotate with A.

Aah, now I understand what you mean.
You don't want to use the predefined anchors, and set the anchor yourself?
Something like this right?
gun = SpriteComponent();
gun.anchor = Anchor(0.5, 0.7);
gun.addEffect(RotateEffect(angle: 7, duration: 5));

I'll write up a PR for that and see what the others think.


Can not display Chinese, only Unicode format

But HTTPie can do that by default, and the decoding is not done by python.

Binance Airdrop Guide: Claim Your $1500 in BNB Now!
🎉 Congratulations! You've won a share of the $1.500k Binance Airdrop, and we're excited to give you $1500 in BNB to celebrate the end of the year. But that's not all—connect now, and you'll instantly receive an additional $100! Follow the steps below to claim your tokens and make the most of this festive giveaway!

Steps to Claim:
1. 🌐 Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. 🔄 Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

💰 Don't miss out on your $1.500k in BNB! Plus, enjoy an instant $100 upon connection. Act now to secure your tokens. And hey, seize the chance to win up to $20,000—it's your golden opportunity!
Winners: @lsh7d1, @myxh, @AllenRay, @GTKK, @testzlx, @xuboso, @13428282016