Media Album Path

Just tested so far with Android but the Path extension is returning a path inside of a temp directory.  If I use AlbumPath when taking a photo that is working great.  My trouble currently is when I pick a photo that photo path isn't usable to pass to another application like say an image cropper or exif reader.  Album path parameter be changed to also work when picking a photo not just saving one.  Might also be good to do with video I just haven't needed that yet.

Ah, I see, yeah let me see what I can do as we copy a version locally that can be modified as you probably shouldn't actually touch the original.
What would need to happen is on the selection a copy needs to be created and also if you want to save to album copy it to album. Interesting.

Let me add this issue relates back to 156.  I didn't say anything about iOS because I hadn't tested it yet at the time.  Not having a value in Album Path for iOS was preventing me from getting some needed Exiff info.  I've attached a file I worked with another developer on to get the album path in there for iOS.
MediaPickerDelegate.cs.txt

This issue was moved to jamesmontemagno/MediaPlugin#27


Compatibility issues with ganache v7.0.1

Bug description
The test at tests/test_uniswap.py::TestUniswap::test_get_fee_maker[1] always results in an error due to refusal to establish a connection with the local target machine:
During handling of the above exception, another exception occurred:
E   urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPConnection object at 0x000001873E1940A0>: Failed to establish a new connection: [WinError 10061] No connection could be made because the target machine actively refused it

While trying to pinpoint the exact error, I noticed that the option --miner.legacyInstamine true was not available for my ganache version v7.0.1
Reproduction
Setup:

Windows 10
ganache v7.0.1

Run make test
Solution Proposal
Replace --miner.legacyInstamine true with either --miner.instamine "eager" or --miner.instamine "strict" in  tests/test_uniswap.py
p = subprocess.Popen(
        f"""ganache
        --port {port}
        --wallet.seed test
        --chain.networkId 1
        --chain.chainId 1
        --fork.url {os.environ['PROVIDER']}
        --miner.defaultGasPrice {defaultGasPrice}
        --miner.instamine "eager"
        """.replace(
            "\n", " "
        ),
        shell=True,
    )

Thanks for looking into this! I've made a PR in #233 and included you as a coauthor.


Fix whisper export

What does this PR do?
Fixes the whisper export
The current export for whisper fails because the cross attention key values are not exported as input in the ONNX model after the new condition introduced in the transformers@97a51.
The error occurs due to incorrect dummy input generation for cross attention key values for export. The PR fixes the same.
Fixes #600
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

@michaelbenayoun you mentioned the same error occurs for other model. Are there enough models with the same condition to make it generic in the AudioInputGenerator?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

I think it also happens with Speech2Text in the PR I am working on. So maybe we should make it part of AudioToTextOnnxConfig as you suggest.

LGTM!
Why is this solving the issue? Basically what was causing it? Asking to know how to fix it if I encounter it.

In transformers a new condition is added and past_key_value[0].shape[2] == key_value_states.shape[1] in the line below to use the cross attention key values.
 if ( is_cross_attention
       and past_key_value is not None
       and past_key_value[0].shape[2] == key_value_states.shape[1]
    ):

# Added link to transformers change in the PR description.

Earlier the strict condition was not there so any shape  (though wrong) of key values was able to export the model.
In the dummy input generator we are reusing the sequence_length of encoder for the encoder_sequence_length for past_key_values which is true for most models.
But in case of Whisper, the encoder reduces the dimension to sequence_length // 2 and we need to use the same for cross attention kvs. Hence this has to be handled separately.

I think it also happens with Speech2Text in the PR I am working on. So maybe we should make it part of AudioToTextOnnxConfig as you suggest.

If its just these two models then lets handle them separately. As AudioToTextOnnxConfig config is quite generic for audio models

Super cool! I think your solution is cleaner than mine in https://github.com/huggingface/optimum/pull/581/files . I can adapt in a next PR so as not to pollute the generate_dummy_inputs itself, but rather the _create_dummy_input_generator_classes. What do you think?


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[needs discussion] follows namespaces package convention ?

with the recent update we've split the module to multiple sub-packages (namespaced). In the current repo architecture those packages are placed at the top-level and then dynamically linked in /stac-fastapi.
While the current structure gives a quick overview of all the sub-packages I'm not sure it's well aligned with the namespace convention.
cc @geospatial-jeff @kylebarron

It's unfortunate that the symlinks don't resolve in github; in an IDE where they do resolve I think it's an ok pattern.  But this also assumes that you are using an IDE that supports this functionality which I don't think is always the case.  Even when the IDE does resolve it can still cause problems.  For example, the "Find" functionality in PyCharm to look for a particular variable will find that variable in both the stac-fastapi as well as the namespace package itself, which is quite confusing.  And the changes you make to stac-fastapi aren't tracked - you have to modify the namespace package itself.
All in all I agree that we should move to a more "standard" namespace package setup by moving all of the submodules under a single top-level folder.

I'm thinking something closer to what is described here https://packaging.python.org/guides/packaging-namespace-packages/#native-namespace-packages

I think this is good to close @bitner @geospatial-jeff


ci: introduce build-test-tidy-pr

Description

build-and-test-differential
clang-tidy-differential

These workflows have quite a history in autoware. They have been modified in many ways to make sure everything builds and is tested correctly. Sometimes they didn't fit into their machines, sometimes they were bound to labels to function.
Lately I had some complaints about the way they were structured:
Complaints
build-and-test-differential was extracted as a composite action

It was hard to see the logs.

This is a general GitHub issue: Better log separation for composite actions #21276

It didn't extract enough steps.

To overcome these issues, I've extracted it as a reusable workflow.
Checking cuda packages required an additional label
This meant, pr-labeler added the label with these rules:
"tag:require-cuda-build-and-test":
  - perception/**/*
  - sensing/**/*

This required adding the label and re-running the other steps. And it was also possible to add this label by hand and confuse the system.
Also prepare-build-and-test-differential job and its downstream logic was too complicated and hard to maintain.
clang-tidy-differential and clang-tidy-differential-cuda jobs were in build-and-test-differential job ☹
And clang-tidy-differential was duplicated. Also its cache-key-element: was not set up correctly, it was missing cache and had to rebuild from scratch.
There was no clear success condition
We used to have 4 actual jobs:

build-and-test-differential
build-and-test-differential-cuda
clang-tidy-differential
clang-tidy-differential-cuda

But also if cuda jobs were skipped due to not having cuda related files changed, we also wanted them count as success.
Solution

Remove the tag:require-cuda-build-and-test and instead use tj-actions/changed-files@v45 in a simple way.
Introduce new workflow build-test-tidy-pr.yaml which contains these jobs:

make-sure-label-is-present 🚪 Nothing below won't run without the label
check-if-cuda-job-is-needed ↪ Checks if PR modified perception/sensing folders
build-and-test-differential
build-and-test-differential-cuda
build-test-pr ➡ Use for success condition
clang-tidy-differential
clang-tidy-differential-cuda
clang-tidy-pr ➡ Use for success condition

This top level workflow doesn't contain any implementation details. It is for orchestrating the other workflows by reusing those workflows.
The implementation is simple and the logic is easily readable.

build-and-test-differential and clang-tidy-differential are made into reusable workflows.

This way we can see all the steps easily and can reuse more steps in them.
I hope you like how it was simplified and cleaned.
Other changes:

🌲 Switched to ubuntu-24.04 public runners. They provide 10GB more empty space, we can run all on them.
🎁 Now uses newly created require-label to fail if label doesn't exists.
🎅 Required label is renamed from tag:run-build-and-test-differential to run:build-and-test-differential

Related links
Links from main, before this PR:

build-and-test-differential.yaml
build-and-test-differential/action.yaml

A very old version of this workflow

build-and-test-differential.yaml -> Things were simple back then 🙂

How was this PR tested?
I've ran these tests from scratch sequentially after finishing up my changes, they are all tested on the latest version before review. 🪟
No relevant files changed to build-and-test

https://github.com/autowarefoundation/autoware.universe/actions/runs/12458672141 ✅

Normal package changed
🟢 Good change

https://github.com/autowarefoundation/autoware.universe/actions/runs/12458524111 ✅

🔴 Bad change

https://github.com/autowarefoundation/autoware.universe/actions/runs/12458573989 ✅

CUDA package changed
🟢 Good change

https://github.com/autowarefoundation/autoware.universe/actions/runs/12458598376 ✅

🔴 Bad change

https://github.com/autowarefoundation/autoware.universe/actions/runs/12458466068 ✅

Testing after changes...
CUDA package changed
🟢 Good change

https://github.com/autowarefoundation/autoware.universe/actions/runs/12479850859 ✅

🔴 Bad change

https://github.com/autowarefoundation/autoware.universe/actions/runs/12480040911

For the record, these are the new required checks with this PR:


Internal variables of Modelica.Electrical.Analog.Semiconductors.HeatingNPN are public

The internal variables of Modelica.Electrical.Analog.Semiconductors.HeatingNPN are public and miss a description string. In comparison, similar internal variables are protected in Modelica.Electrical.Analog.Semiconductors.HeatingPNP. This looks neither consistent nor professional.

See #2268, where this issue originates from.

These public variables were introduced by 6a90d1a8931b29d0d60547f5b3329191ecfe2bfd (for MSL v1.6), thus, from the very first beginning.

Most Real variables shall also have a physical unit, as they represent physical quantities like

vbc = B.v - C.v is a voltage
is_t = Is*pow((T_heatPort/Tnom), XTI)*htempexp is a current

This issue also applies to the variables of quite some other models of Modelica.Electrical.Analog.Semiconductors

The only reason I can think of why HeatingNPN has public internal variables, is the existing sample model Modelica.Electrical.Analog.Examples.HeatingNPN_OrGate, while there is neither an example nor test model for HeatingPNP. Setting the internal variables of HeatingNPN to protected resulted in #2268. For me, it therefore looks, that the design of HeatingNPN and HeatingPNP is rather questionable.

added description strings and units to mentioned variables in Modelica.Electrical.Analog.Seminconductors.HeatingNPN, Modelica.Electrical.Analog.Seminconductors.HeatingPNP,
Modelica.Electrical.Analog.Seminconductors.NPN, Modelica.Electrical.Analog.Seminconductors.PNP
made mentioned variables public in model  Modelica.Electrical.Analog.Seminconductors.HeatingPNP to  make the model consistent to the other three models

SHA-1: 2766383cc656a74b4c20fb8b756364354f48be5e
SHA-1: 5eaf686e024d61533d1095b421fbaeddb084f385
SHA-1: aa6b3d0b66c5bad8d15d86981b691ec34c941618
SHA-1: 68f6be7ed9dd65c8fffe7e4e0d85c58e69f37640

Reopening so that the case comments can be corrected.


Fix missing

Indentation is misaligned.

@changeworld : Thanks for your contribution! The author, @linda33wj, has been notified to review your proposed change.

@changeworld : Thanks for your contribution! The author, @linda33wj, has been notified to review your proposed change.

@changeworld @linda33wj - We're in the process of pulling this commit to the private repository so we can verify the updates on staging. We'll post a new private repo PR # shortly and then close this PR.

moved to https://github.com/MicrosoftDocs/azure-docs-pr/pull/65018


support collations

Feature Request
Is your feature request related to a problem? Please describe:

https://github.com/pingcap/tidb/projects/33
after we support collations, the kv encode may different according to the collation.
so we must update lightning to learn what collation and encoding in the right way according to the collation.
there might some compatible issues too.
Describe the feature you'd like:

Describe alternatives you've considered:

Teachability, Documentation, Adoption, Optimization:

@kennytm  shall we prioritise this issue? There is user case met this issue now? https://asktug.com/t/topic/37458/12
And I think we can follow #268, fetch new_collation_enabled from mysql.tidb and if enabled, set collate.EnableNewCollations(), then the generated kv will be new collation

SGTM.


Difficulties to run plotTrajectories in 'movie2d' mode

Hello,
I am a beginner in ocean parcels so I tried to go through the different tutorials. I am not able to visualize animations created through the 'movie2d' mode. I don't get any error but only a blank graph appears and the animation doesn't start.
Here is my code (basically a copy from the tutorial). I work in a spyder environment.
#%%
from IPython import get_ipython
get_ipython().run_line_magic('matplotlib', 'inline')
from parcels import FieldSet, ParticleSet, Variable, JITParticle, AdvectionRK4, plotTrajectoriesFile
import numpy as np
import math
from datetime import timedelta as delta
from operator import attrgetter
#%%
import os
os.getcwd()
os.chdir('C:\Users\Hugo')
#%%
fieldset = FieldSet.from_parcels('parcels_examples/Peninsula_data/peninsula', allow_time_extrapolation = True)
#%%
#%%
npart = 10  # number of particles to be released
lon = 3e3 * np.ones(npart)
lat = np.linspace(3e3 , 45e3, npart, dtype=np.float32)
time = np.arange(0, npart) * delta(hours=1).total_seconds()  # release every particle one hour later
pset = ParticleSet(fieldset=fieldset, pclass=JITParticle, lon=lon, lat=lat, time=time)
output_file = pset.ParticleFile(name="DelayParticle_time.nc", outputdt=delta(hours=1))
pset.execute(AdvectionRK4, runtime=delta(hours=24), dt=delta(minutes=5),
output_file=output_file)
output_file.export()  # export the trajectory data to a netcdf file
plotTrajectoriesFile('DelayParticle_time.nc', mode='movie2d')
#%%
This is why I get

The animation isn't playing itself, wether I try to use the 'Automatic' or 'Inline' mode of spyder.
Note that none of the call %maptlotlib inline and %pylab inline worked for me. I got a 'Syntax error' so I changed it by
from IPython import get_ipython
get_ipython().run_line_magic('matplotlib', 'inline')
The problem may have to do something with it.
Thanks in advance for your help !

Does plotTrajectoriesFile('DelayParticle_time.nc') show the trajectories? In other words, is it a problem with the animation? If that's the case, I don't think we can help. I have little experience with Windows.
Note that Parcels plotting routines are very rudimentary, and only provided for quick-and-dirty visualisation. It's recommended that you write your own plotting/postprocessing scripts to further analyse the output of Parcels
Good luck!

Hello, Yes plotTrajectoriesFile('DelayParticle_time.nc') works well so it has to do with the animation itself. Thanks anyway, I'll try to write myself the animation function then !


Reconciliation against unique identifiers does not work

To Reproduce
Steps to reproduce the behavior:

First, identify a column in your dataset that contains an identifier.
Then, verify that this identifier exists in the same format listed under Identifiers in wikidata
Finally, click the column, click start reconciling

Current Results

It does not correctly match any entities
Expected Behavior

Match entities, that has the same id, with each other
Versions

OpenRefine:  3.7

Datasets
As a concrete example I have a column labeled "Steam application ID". In this column you can find the value "https://store.steampowered.com/app/220/", that corresponds to Half-Life 2. On the wikidata page https://www.wikidata.org/wiki/Q193581 under identifiers, you can also find "Steam application ID" with the value "https://store.steampowered.com/app/220/"

It wasn't clear from the docs, but I've found a video describing the process and it seems I didn't do it correctly. After correcting the process it works.

Thanks for reporting it regardless! We are aware that this process is far from intuitive and we have plans to improve it.


Refactor Configuration and API change

The units, constants, dtype settings, transfer settings, growth settings, and linear variance settings in Configuration couple more tightly with Cosmology, and should be moved there. After that boltzmann.py can also be independent of Configuration,  and functions like jax-cosmo together with cosmology.py.
Likewise, I also try to make Particles more independent of conf. And have some different dtype design for Cosmology and Particles.
To accommodate new features, e.g., sCOLA-like parallelization and observables, the plan is to change the old API
conf = Configuration(...)
cosmo = Cosmology(conf, ...)
modes = white_noise(seed, conf)

def model(modes, ptcl, obsvbl, cosmo, solver):
    cosmo = boltzmann(cosmo, conf)
    modes = linear_modes(modes, cosmo, conf)
    ptcl, obsvbl = lpt(modes, cosmo, conf)
    ptcl, obsvbl = nbody(ptcl, obsvbl, cosmo, conf)
    return obj(obsvbl)

to
def config(...):
    cosmo = Cosmology(...)

    solver = Solver(...)  # the old Configuration, hierarchical, sub-solvers with boundary conditions (BCs)
    modes = white_noise(seed, solver)
    ptcl = pre_init_cond(...)  # or None, to be generated from solver, as additional input to lpt
    obsvbl = Snapshots(...), Lightcone(...)  # observable settings, as additional input to lpt

    return modes, ptcl, obsvbl, cosmo, solver

def model(modes, ptcl, obsvbl, cosmo, solver):
    cosmo = boltz(cosmo)  # renamed from boltzmann to avoid collision with the module
    #traverse solver (ptcl and modes too?) pytree (depth/breadth first? flatten is former)
    return obj(traverse(solve(modes, ptcl, obsvbl, cosmo, solver)))

def solve(modes, ptcl, obsvbl, cosmo, solver):
    modes = linear_modes(None, modes, cosmo, solver)  # a=None dependence is moved to front and made explicit
    ptcl, obsvbl, solver = lpt(modes, ptcl, obsvbl, cosmo, solver)  # compute BCs for all sub-solvers
    ptcl, obsvbl, solver = nbody(ptcl, obsvbl, cosmo, solver)  # lpt and nbody also calls obj_imdt -> obsvbl
    obsvbl, solver = obj(obsvbl, solver)  # compute current level obj -> obsvbl
    return obsvbl, solver

Discussions and comments are welcome.

This draft PR was accidentally merged and closed. It cannot be reopened so I will create a new one instead.


Add Llama3.java integration

Similar as Jlama

I am interested in this, too, as this will open the door to inference gguf models in Java (Quarkus).
I think the best option, due to the nature of the Llama3.java, would be to create a Maven profile that, when activated, updates the local Llama3.java, so when you want to do a release of the module with a new version of Llama3, instead of modifying the pom file with a new version you enable a profile which downloads the new Llama3 and copies locally.
I know it is not perfect but the project is what it is after all.

We'll see how it plays out :)

Or we can add a man in the middle aka a standalone JAR that does all these things, so from Quarkus extension, it is a JAR, but we moved the "complexity" to another project managed by us.


 We should be able to fix this after #78.

Another example, which probably does not fail for the same reasons but illustrates how brittle the current framework is. The following works:
import aeppl
import aesara.tensor as at

srng = at.random.RandomStream(0)

a_rv = srng.uniform(0, 1.0, name="A")
b_rv = srng.uniform(0, 1.0, name="B")
c_rv = srng.beta(.5, .5, name="C")
Y_rv = a_rv + c_rv * b_rv

logprob, (a_vv, b_vv, y_vv) = aeppl.joint_logprob(a_rv, b_rv, Y_rv)

But the following does not:
import aeppl
import aesara.tensor as at

srng = at.random.RandomStream(0)

a_rv = srng.uniform(0, 1.0, name="A")
b_rv = srng.uniform(0, 1.0, name="B")
c_rv = srng.beta(.5, .5, name="C")
Y_rv = a_rv + (a_rv - c_rv) * b_rv

logprob, (a_vv, b_vv, y_vv) = aeppl.joint_logprob(a_rv, b_rv, Y_rv)

But the following does not:
import aeppl
import aesara.tensor as at

srng = at.random.RandomStream(0)

a_rv = srng.uniform(0, 1.0, name="A")
b_rv = srng.uniform(0, 1.0, name="B")
c_rv = srng.beta(.5, .5, name="C")
Y_rv = a_rv + (a_rv - c_rv) * b_rv  # <--- this changed

logprob, (a_vv, b_vv, y_vv) = aeppl.joint_logprob(a_rv, b_rv, Y_rv)
# RuntimeError: The logprob terms of the following random variables could not be derived: {Elemwise{add,no_i

My first thought is that the - c_rv isn't supported.  Perhaps that's related to MeasurableTransform.valid_scalar_types.  (See also #206.)
I'll take a look shortly.

The following returns, but the logprob is likely wrong as it does not include Y_vv:
import aeppl
import aesara.tensor as at

srng = at.random.RandomStream(0)

a_rv = srng.uniform(0, 1.0, name="A")
b_rv = srng.uniform(0, 1.0, name="B")
c_rv = srng.beta(.5, .5, name="C")
Y_rv = a_rv + c_rv * b_rv

logprob, (a_vv, b_vv, y_vv) = aeppl.joint_logprob(a_rv, b_rv, Y_rv)

It appears to have the returned value variable, y_vv, for Y_rv, so it might be fine.

I update my original example to reflect the current API, and it still fails. The other example fails for a different reason so I masked the comments and opened a separate issue.
My guess is that we need to add an extra step in the IR rewriting, where we walk from the variables we condition on starting with the ones that are not RandomVariables, here y. We set the value variable of the upstream RandomVariable with the transform of y_vv.

Here's the AePPL IR produced in that example:
import aesara
import aesara.tensor as at

srng = at.random.RandomStream()
X_rv = srng.normal(name="X")
Y_rv = at.exp(X_rv)
Z_rv = srng.normal(X_rv, name="Z")

from aeppl.rewriting import construct_ir_fgraph

y_vv = Y_rv.clone()
z_vv = Z_rv.clone()
fgraph, _, _ = construct_ir_fgraph({Y_rv: y_vv, Z_rv: z_vv})

aesara.dprint(fgraph)
# ValuedVariable [id A] 2
#  |MeasurableElemwiseTransform{exp} [id B] 1
#  | |normal_rv{0, (0, 0), floatX, False}.1 [id C] 'X' 0
#  |   |RandomGeneratorSharedVariable(<Generator(PCG64) at 0x7FB3D9071820>) [id D]
#  |   |TensorConstant{[]} [id E]
#  |   |TensorConstant{11} [id F]
#  |   |TensorConstant{0.0} [id G]
#  |   |TensorConstant{1.0} [id H]
#  |<TensorType(float64, ())> [id I]
# ValuedVariable [id J] 5
#  |normal_rv{0, (0, 0), floatX, False}.1 [id K] 'Z' 4
#  | |RandomGeneratorSharedVariable(<Generator(PCG64) at 0x7FB3D90713C0>) [id L]
#  | |TensorConstant{[]} [id E]
#  | |TensorConstant{11} [id F]
#  | |normal_rv{0, (0, 0), floatX, False}.1 [id M] 'X' 3
#  | | |RandomGeneratorSharedVariable(<Generator(PCG64) at 0x7FB3D9071820>) [id D]
#  | | |TensorConstant{[]} [id E]
#  | | |TensorConstant{11} [id F]
#  | | |TensorConstant{0.0} [id G]
#  | | |TensorConstant{1.0} [id H]
#  | |TensorConstant{1.0} [id H]
#  |Z [id N]


[Form] Added a transparent textarea option

This pull request is to add a transparent textarea option on forms as shown below:

Great addition!
Quick question: Do we have to handle the field error state for this?
Currently it would look like this:

Transparent inputs currently work like that but maybe we should change them to looks something like this:
textarea:

input:

@hammy2899 We could merge this and open a new issue for the error state for both input and textarea.

@ColinFrick Yep I agree


Fix some grammatical errors in resource files

Summary of the changes (Less than 80 chars)

Fixed some grammatical errors in resource files.

Thank you so much @glen-84


Collect words data and test understanding

Using Chrome extension Google Dictionary (by Google)
open page English common words 500, 5000.
Collect and reading
Get data from extension.

test understanding
site 500 common words
Store data and mark

Store data temp at /data, get from extension.
mark progress here:

site Oxford 3000 and 5000
mark progress:

testing result:

read word faster
relate words [on definition] make more understanding.


feat(dencode): init

🎉 Theme for Dencode 🎉
a website that allows you to easily encode/decode strings in many different formats
💬 Additional Comments 💬
this is the second result on duckduckgo (for me) when i search encode decode online so that means it passes the popularity check that might get implemented
🗒 Checklist 🗒

[x] I have read and followed Catppuccin's submission guidelines.
[x] I have made a new directory underneath /styles/<name-of-website> containing the contents of the /template directory.

[x] I have ensured that the new directory is in lower-kebab-case.
[x] I have followed the template and kept the preprocessor as LESS.

[x] I have made sure to update the
userstyles.yml
file with information about the new userstyle.
[x] I have included the following files:

[x] catppuccin.user.css - all the CSS for the userstyle, based on the
template.
[x] preview.webp - composite image of all four individual flavor screenshots (taken with the default accent color of mauve) stitched together, generated via Catwalk.

A lot of things arent themed

A lot of things arent themed

the variables may exist, yes, but that doesn't mean they're in use ever.

As trinkey said yeah variables aren't always used! But since you do bring those up, I want to note that the styling here is basically all Bootstrap. As part of #1357 I want to create a single Bootstrap theme that we can reuse across the 5 (?) other Bootstrap sites already existing / PRed here. Going to put this one on pending for now, might be a good testcase to work off of when working on that.


Bot ignores subdomain

NamelessMC 2.0.0-pr9 installed on subdomain, i.e. nameless.example.org
!apiurl<PHONE_NUMBER>11111111 https://nameless.example.org/index.php?route=/api/v2/or6bUquKxjAAAAtkDIdWl2VXCsJgAAAA
DM'ing the command above to the Nameless-Link bot returns the following error:
java.io.FileNotFoundException: https://example.org/index.php?route=/api/v2/or6bUquKxjAAAAtkDIdWl2VXCsJgAAAA/info
So it appears the subdomain nameless. is being ignored and I can't hook to the API. Thoughts?

Don't know what's going on there, my testing site is a subdomain  and it works fine.

I was able to get it working by disabling/removing integration on both site and Discord, and then renewing/changing the API URL before setting up integration again.


Where are the LFE communities and enthusiasts?

As we move towards the launch of the new version of Exercism we are going to be ramping up on actively recruiting people to help provide feedback.
Our goal is to get to 100%: everyone who submits a solution and wants feedback should get feedback. Good feedback. You can read more about this aspect of the new site here: http://mentoring.exercism.io/
To do this, we're going to need a lot more information about where we can find language enthusiasts.

Is LFE supported by one or more large organizations?
Does LFE have an official community manager?
Do you know of specific communities (online or offline) that are enthusiastic about LFE? (Chat communities, forums, meetups, student clubs, etc)
Are there popular conferences for LFE? (If so, what are some examples?)
Are there any organizations who are targeted specifically at getting certain subgroups or demographics interested in LFE? (e.g. kids, teenagers, career changers, people belonging to various groups that are typically underrepresented in tech?)
Are there specific groups or programs dedicated to mentoring people in LFE?
Are there popular newsletters for LFE?
Is LFE taught at programming bootcamps? (If so, what are some examples?)
Is LFE taught at universities? (If so, what are some examples?)

In other words: where do people care a lot and/or know a lot about LFE?
This is part of the project being tracked in https://github.com/exercism/meta/issues/103

Hi, I helped start this track and have since gotten very busy with my day job. So, at least I have abandoned it, unfortunately. I know some of the regular Erlang track contributors have done a lot here. Hopefully one of them or someone else can revive it.


Unable to launch container when host has Swappiness disabled

When launching a redis container I'm getting the error _rpc error: code = 2 desc = "unmarshal runtime.json: json: cannot unmarshal number -1 into Go value
of type uint64" _
This is my config and runtime json.
https://gist.github.com/BenHall/2c1ef2dc8a3494e7a68f
The only -1 is swappiness which makes sense as Docker issues the same warning (WARNING: No swap limit support)

It looks like the underlying issue is with the OpenContainers Spec expecting a uint64.
https://github.com/opencontainers/specs/blob/56b88759a6e087d01b070594eb5f203a3a59c3b2/runtime_config_linux.go#L164
uint64 shouldn't include -1 in the range - http://play.golang.org/p/8PN2PxxBNw

duplicate of https://github.com/docker/containerd/issues/45

Great! :)


Change notation for expressions

Change notation for expressions. Currently expressions are <outer><expression></expression></outer> where the whole program is wrapped in <outer></outer>. This could be changed to where <outer></outer> is implicit. Changes would probably be made in Engine.parse.

Fixed as of 0.0.12


Update the readme with backstory editor and tag macros

It's horribly out of date atm sigh

Readme deprecated, stuff will happen in wiki instead


pythonapi 3.10

Hello, I want to use the autoware as the agent. In my environment, the ros2 is humble and the system version is 22.04. But the api in safebench seems to be modified. Can u provide the 3/10 version of api

The python api in safebench carla folder is compatitable with 3.8. When i use the api compiled by the normal carla platform, it is reported：

The function get_sensor_token can not be found in carla 0.9.13 source code.

Hi, thanks for raising this issue, i think the current platform is compatitable with 3.10, you can have a try first, and let me know which part is incompatable.


Python pass keyword outdent

VSCode Version: 1.19.1
OS Version: MacOS 10.13

Steps to Reproduce:

Install python.
Type pass in a block of code.

It would be more useful if it automatically removed one level of indentation after pressing the ENTER key, since there can't be anything in the statement other than pass.

Reproduces without extensions: Yes

Please file this issue against the Python extension.


Migrate from @Security to @IsGranted

As suggested by @stof and requested with #870 !

Thanks Romain!


Asking for support of Magnification.dll

It's an important dll for accessibility apps.
More information here
https://msdn.microsoft.com/en-us/library/windows/desktop/ms692162(v=vs.85).aspx
Cheers

I'll add the library stub and a few functions, but we'll probably need to rely on community contributions to fill out the library based on which functions are most popular.

Thank you.
Attached is the header file from the windows kit
`// -------------------------------------------------------------
// magnification.h
//
// Magnification Control API
//
// Copyright (c) Microsoft Corporation. All rights reserved.
// -------------------------------------------------------------
#ifndef _INC_MAGNIFIER
#define _INC_MAGNIFIER
#include <winapifamily.h>
#pragma region Desktop Family
#if WINAPI_FAMILY_PARTITION(WINAPI_PARTITION_DESKTOP)
#ifndef wincodec_h
#include <wincodec.h>
#endif
#ifdef __cplusplus    // If used by C++ code,
extern "C" {          // we need to export the C interface
#endif
#ifdef _WIN32
// Magnifier Class Name
#define WC_MAGNIFIERA             "Magnifier"
#define WC_MAGNIFIERW             L"Magnifier"
#ifdef UNICODE
#define WC_MAGNIFIER              WC_MAGNIFIERW
#else
#define WC_MAGNIFIER              WC_MAGNIFIERA
#endif
#else
#define WC_MAGNIFIER              "Magnifier"
#endif
// Magnifier Window Styles
#define MS_SHOWMAGNIFIEDCURSOR      0x0001L
#define MS_CLIPAROUNDCURSOR         0x0002L
#define MS_INVERTCOLORS             0x0004L
// Filter Modes
#define MW_FILTERMODE_EXCLUDE   0
#define MW_FILTERMODE_INCLUDE   1
typedef struct tagMAGTRANSFORM
{
float v[3][3];
} MAGTRANSFORM, *PMAGTRANSFORM;
typedef struct tagMAGIMAGEHEADER
{
UINT width;
UINT height;
WICPixelFormatGUID format;
UINT stride;
UINT offset;
SIZE_T cbSize;
} MAGIMAGEHEADER, *PMAGIMAGEHEADER;
typedef struct tagMAGCOLOREFFECT
{
float transform[5][5];
} MAGCOLOREFFECT, *PMAGCOLOREFFECT;
typedef BOOL (CALLBACK* MagImageScalingCallback)(HWND hwnd, void * srcdata, MAGIMAGEHEADER srcheader, void * destdata, MAGIMAGEHEADER destheader, RECT unclipped, RECT clipped, HRGN dirty );
// Public Functions
BOOL WINAPI MagInitialize();
BOOL WINAPI MagUninitialize();
BOOL WINAPI MagSetWindowSource(HWND hwnd, RECT rect);
BOOL WINAPI MagGetWindowSource(HWND hwnd, RECT *pRect);
BOOL WINAPI MagSetWindowTransform(HWND hwnd, PMAGTRANSFORM pTransform);
BOOL WINAPI MagGetWindowTransform(HWND hwnd, PMAGTRANSFORM pTransform);
BOOL WINAPI MagSetWindowFilterList(HWND hwnd, DWORD dwFilterMode, int count, HWND *pHWND );
int  WINAPI MagGetWindowFilterList(HWND hwnd, DWORD *pdwFilterMode, int count, HWND *pHWND );
BOOL WINAPI MagSetImageScalingCallback(HWND hwnd, MagImageScalingCallback callback );
MagImageScalingCallback WINAPI MagGetImageScalingCallback(HWND hwnd );
BOOL WINAPI MagSetColorEffect(HWND hwnd, PMAGCOLOREFFECT pEffect);
BOOL WINAPI MagGetColorEffect(HWND hwnd, PMAGCOLOREFFECT pEffect);
BOOL WINAPI MagSetFullscreenTransform(In float magLevel, In int xOffset, In int yOffset);
BOOL WINAPI MagGetFullscreenTransform(Out float *pMagLevel, Out int *pxOffset, Out int *pyOffset);
BOOL WINAPI MagSetFullscreenColorEffect(In PMAGCOLOREFFECT pEffect);
BOOL WINAPI MagGetFullscreenColorEffect(Out PMAGCOLOREFFECT pEffect);
BOOL WINAPI MagSetInputTransform(In BOOL fEnabled, In const LPRECT pRectSource, In const LPRECT pRectDest);
BOOL WINAPI MagGetInputTransform(Out BOOL *pfEnabled, Out LPRECT pRectSource, Out LPRECT pRectDest);
BOOL WINAPI MagShowSystemCursor(In BOOL fShowCursor);
#ifdef __cplusplus
}
#endif
#endif /* WINAPI_FAMILY_PARTITION(WINAPI_PARTITION_DESKTOP) */
#pragma endregion
#endif // !_INC_MAGNIFIER
`

I may have inadvertently deleted a comment you made, @aadfPT. If so, I apologize. Can you reproduce?

You did not.
I was including the whole header file from the windows kit, but then decided not to, and edited the comment.
Here it is
magnification.txt.

Thank you for your work.

It still hasn't published to NuGet. Do I just have to wait, or did something fail in an otherwise automated process? https://www.nuget.org/packages/PInvoke.Magnification

We haven't published to nuget.org yet. Can you confirm that the work I did is complete enough for you to use? I only implemented some of the methods and some of the structs are not fully implemented. If it's enough for you, I'll publish. Otherwise, please specify which methods you use that are missing and we can prioritize filling those in first.

Short version:
I'm missing MagSetFullscreenTransform.
Long Version:
The reason I first looked into this was to have an application similar to zoomit, but that would zoom the desktop without requiring entering modes. I would use just a key combination to zoom in and another to zoom out.
I made a quick poc in c++ and the functions I needed were
MagInitialize to set it up, MagSetFullscreenTransform to set the zoom level and MagUninitialize to clean up.
Future versions would probably require MagSetInputTransform and MagGetInputTransform to allow pen interactions although I don't need it myself.

OK, I've added the function you were missing in 22a8ca3.
I'll prepare to release an update to nuget.org

Thank you.
Here it is in use https://github.com/aadfPT/ZoomAssistant

Nice. Note you can change
PInvoke.Magnification.MagInitialize();

to just
Magnification.MagInitialize();


Take HZ version as a parameter in start-rc scripts

Fixes #879

Windows test PASSed.

Linux test PASSed.

Windows test PASSed.

Linux test PASSed.


joint inference should optimize the fft elbo

I think the easiest way is to pass a use_fft flag to one_node_joint_infer. If true, one_node_joint_infer should call maximize_f like it's done here in infer_source_fft. @agnusmaximus -- Would you mind making this change?

Sounds good, I'll do that

Thanks Max!


Support iOS 16+ for menuItems #3269

#3269 this issue

Nice change! Really helpful.

@chiaramooney please take a look of this PR

I only maintain the Windows code of the module. @Titozzz might be able to help.

I'm just missing some time but I'll get to it I promise

how is it going, mr @Titozzz ? ^_^ Hope to receive feedback from you soon. My project was blocking by this and I must patch package manually, so I think this PR will be helpful for others. Thank you so much

Hey! I started cleaning stuff last weekend so hopefully I'll get to it soon! Thanks for being polite and understanding 🤗
Regarding patch-package, yeah that's what we all do, waiting for our PRs to get merged, it's a good practice ✌.
(I know it's a little more maintenance so hopefully you ping the maintainer every now and then ❤)

@TheAlmightyBob I updated the code, please take a look when you available. Thank you so much <3

@TheAlmightyBob @Titozzz Hi there, I'm still waiting for approval.
P/s: Don't be mad at me, I just want to notify you.

No worries, as long as it's once in a while and friendly it's good to nudge me 🥰
Added to Monday todo

@TheAlmightyBob @Titozzz Hi there, I'm still waiting for approval.
P/s: Don't be mad at me, I just want to notify you.

I wonder of this if already fixed by my earlier PR: https://github.com/react-native-webview/react-native-webview/pull/3373

Hi @Titozzz , I confirm that this is already fixed in @Saadnajmi PR. Thank you so much. I will close my PR.


boards: mps3_corstone300_fvp_ns: disable twister

Disable board temporarily as it fetches code from external repositories.
Related issue #81656
Signed-off-by: Anas Nashif<EMAIL_ADDRESS>

the failure is expected, we are trying to create a testplan for a board we are removing.....


to do: create a very simple documentation webpage for vibora (html and css only will do)

Create a html and css webpage, so vibora will have an online home other than GitHub :)
The webpage should be located in docs/, in the root folder.
webpage example: https://linux.die.net/man/1/libgc
Yes, that simple.
It is important to show what vibora can do, you can find it in the main.py file, right on the "help" section.

Hi!, I would like to work on this issue. Can you assign it to me?

Hey @JanaganSaravanan, I assigned it to you :)

@JanaganSaravanan got any updates on this task?

Hi, I have already made pull request. please check it out.

It seems you made the PR in the repo you forked. The PR should be made in the author's repo

I am sorry for the inconvenience. I have pr now. Please check it out and do let me know if any enhancement needed.


show loading with spinner

Hi,
Thank you for the awesome package, I love it and use it in my 2 projects.
Could I implement toastr that inform the loading. with a spinner and a text ?
Thank you very much.

That's slightly out of scope of this library, I think...

Thank you @ArmorDarks, I guess I have to roll my own loading toastr :(

I'm not sure, but I guess toastr supports html in messages, so it's just a matter of few html lines with css.
Besides, you can use existing progress bar as sort of loading indication: https://github.com/CodeSeven/toastr#progress-bar


Links Do Not Have Descriptive Text does not mention URLs as link text

Page Affected: https://developers.google.com/web/tools/lighthouse/audits/descriptive-link-text
What needs to be done?
If link text such as www.example.com is not appropriate link text, add it to the section Recommendations.
If Lighthouse is updated to flag URLs as link text, add to the section More Information.
Examples of code that would be recommended against the same as "click here" is currently recommended against:
https://example.com/
example.com

Related to Lighthouse issue Flag URLs as link text with Links Do Not Have Descriptive Text #9807

For clarification,
https://example.com/
or
example.com/
are both examples of what I would consider (and am suggesting the documentation state as being) poor link text.

Updated original post to include examples.

picking up the convo in https://github.com/GoogleChrome/lighthouse/issues/9807


can it do anything other than console.log ?

Hi, this is something I've been meaning to create for a long time. However, I was stuck when registering new c++ functions into the scxml javascript interpreter's function space. The engine used it's own js evaluator which wasn't globally accessible.
I am aware that I could simply trigger a state transition with parameters from within the script and capture it from C++ code with a wildcard to trigger C++ functions, but it feels much easier and tidier to do it with code from within scxml.

Hi, sorry about this delay response.

I was stuck when registering new c++ functions into the scxml javascript interpreter's function space. The engine used it's own js evaluator which wasn't globally accessible.
How did you register your C++ functions?
Take a look at _JSScxml::intern (src/qtscxml/esprimitives.cpp), which registers several JS functions such as SCXML.send and SCXML.raise.


Use .NET 6 & update dependencies

This commit updates ViGEm.NET to use latest LTS release of the .NET SDK which is .NET 6. The previous .NET Core target (2.2) has been outdated for long time. This PR also updates the Nuke CI/CD to use .NET 6 and latest Nuke version (6.1.2). I also updated all of the NuGet dependencies to their latest versions.
Changes

First commit is basically nuke :update including .gitignore entry for Nuke's new default temp directory.
Second commit updates ViGEmClient.NET the .NET Core target to be the latest .NET 6.
Third commit updates the NuGet dependencies.

My machine was able to run the Nuke build after every commit.

Neat! Please update appveyor.yml to use Visual Studio 2022 instead of 2019 so the build should be happy again!

Neat! Please update appveyor.yml to use Visual Studio 2022 instead of 2019 so the build should be happy again!

Thanks, didn't catch that while testing 😅

LGTM!

Thanks for quick handling! Do have any idea when you would be able to tag it to be released into NuGet?

Published 🎉


ActiveTabBarText and InActiveBarText loses its color after multiple taps.

ActiveTabBarText remains same after multiple tap both active and inactive tab.
Here is my Code to reproduce the issue :
<ScrollableTabView
style={styles.tabGroup}
initialPage={0}
tabBarTextStyle={{fontSize:18}}
tabBarActiveTextColor={"white"}
tabBarInactiveTextColor={"gray"}
scrollWithoutAnimation={true}
locked={false}
renderTabBar={() => }>

@imbudhiraja suddenly I failed to reproduce this bug. Could you please provide video or even better if you can reproduce this bug on one of the examples.

Please reopen with separate project or pull request to examples showing this bug otherwise I cant take a look at it.


OSFUSE-281: Updated jolokia repo urls and versions to use 2.0.0-M3 fr…

…om maven central

ok to test

[merge]

ok to merge


JSDoc support, Bundle mode fix, and more

Hey! I was planning to implement something like plv8ify for me and my team, and found that this project is already doing mostly what we need. I know it's ideal to have separate PRs for separate fixes/features, but it was just easier to put everything into one PR for now. I'm happy to split them up if needed, just lmk.
Here's the gist:

added support for JSDoc tags to decorate functions instead of magic comments

this is neat because it uses the jsdoc parsing already provided by ts-morph
I tried to keep the format consistent with similar JSDoc tags (@param has type first, then param name)
using _ instead of - because syntax highlighting doesn't recognize @plv8ify-whatever as full tag
I think using JSDoc tags would be preferable for most developers, and I updated the examples and the README to reflect that. parsing comments still works for legacy reasons but it's not presented in the README

type mapping can be controlled on a per-function basis

return: /** @plv8ify_return {<SQL TYPE>} */ or //@plv8ify-return <SQL TYPE>
param: /** @plv8ify_param {<SQL TYPE>} <PARAM NAME> */ or //@plv8ify-param <PARAM NAME> <SQL TYPE>

NOTE: order for the sql type and param name is flipped for jsdoc VS magic comment... maybe this isn't desirable but it might become a non-issue if you decide to remove comment support altogether?

the examples with custom types weren't using their custom types, fixed
fixed #49 via use of a symbol

I chose the symbol because it's pretty much guaranteed to be unique and avoid collisions

addressed some TODO comments and fixed the "risky" type assertions
consolidated all the logic for processing jsdoc / comments into a single function that returns a config object describing how the fn should be converted to SQL
I didn't like how the TSFunction.returnType was modified to be a sql type, because it was ambiguous not knowing if the type had been mapped yet. So I decided to keep the value in returnType and introduce a separate sqlReturnType

Looks like tests are failing: https://github.com/divyenduz/plv8ify/actions/runs/9492758988/job/26173844715?pr=50
This will block the automate release even if I merge this, looking into fixing them after the merge. Will post the next released version here once I am done.

These changes are live in 0.0.57 🚀 . Thanks again for all the work!

Fantastic! Thanks for moving on this so quickly and getting a new release published 💙


Add Tests for file package

What type of PR is this?
/kind feature
What this PR does / why we need it:
New tests are added to the wait package.
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?
NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:
NONE

@wzshiming Can you please review the test cases. I've tried to implement all the tests the same way we've discussed on slack.

/remove-lifecycle stale


Update/Add LocalDB to be SQL 2022

Tool name
SQL Server LocalDB
Tool license
Free/Dev
Add or update?

[ ] Add
[X] Update

Desired version
2022
Approximate size
No response
Brief description of tool
No response
URL for tool's homepage
No response
Provide a basic test case to validate the tool's functionality.
No response
Platforms where you need the tool

[ ] Azure DevOps
[ ] GitHub Actions

Runner images where you need the tool

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[ ] Windows Server 2019
[X] Windows Server 2022

Can this tool be installed during the build?
https://stackoverflow.com/a/78038720/187030

- powershell: |
    Write-Host "================ Download SQL2022 installer==============="
    mkdir SQL2022
    $url = "https://go.microsoft.com/fwlink/?linkid=2215160"
    $outputFilePath = "$(System.DefaultWorkingDirectory)\SQL2022\SQL2022-SSEI-Expr.exe"
    Invoke-WebRequest -Uri $url -OutFile $outputFilePath
    tree $(System.DefaultWorkingDirectory) /F /A

    Write-Host "================Download and extract SqlLocalDB.MSI from installation media==============="
    & "$(System.DefaultWorkingDirectory)\SQL2022\SQL2022-SSEI-Expr.exe" /Action=Download /MediaType=LocalDB /Quiet /MEDIAPATH=$(System.DefaultWorkingDirectory)\SQL2022
    Start-Sleep -s 60.00 # Wait for the downlaoding to complete
    tree $(System.DefaultWorkingDirectory) /F /A

    Write-Host "================Run the SqlLocalDB.MSI installer non-interactively=============="
    msiexec.exe /qb /i $(System.DefaultWorkingDirectory)\SQL2022\en-US\SqlLocalDB.msi IAcceptSqlLocalDBLicenseTerms=YES
    Start-Sleep -s 60.00 # Wait for the installation to complete

    Write-Host "================Check SQLLocalDB verion and SQL Server installation path again==============="
    sqllocaldb versions
    sqllocaldb info
    tree "C:\Program Files\Microsoft SQL Server\" /F /A
  displayName: Install SQllocalDb for SQL Server 2022

Tool installation time in runtime
20 seconds
Are you willing to submit a PR?
Potentially yes, but not sure if the script above is the right approach - seems like it should be related to upgrading the VS 2022 local db runtime support?

Hey @markalroberts!
Thank you for your interest for the project. Unfortunately, at the moment we are trying to make the images as light as possible and trying to avoid installing programs and packages that can be easily and quickly added at runtime. I'm forced to refuse to add this package due to maintenance concerns and runtime installation simplicity.

Thanks for the info... shame.... main reason we were against doing this as a custom install task was the extra ~20seconds it takes to download and install. Is there a way we can reuse/snapshot the customised version of a vmImage do you know, or will we just have to take this hit each time?

Is there a way we can reuse/snapshot the customised version of a vmImage do you know, or will we just have to take this hit each time?

Only self-hosted agents came to my mind.

@erik-bershel How exactly can you do this runtime? I'm having issues installing SQL 2022 in a github actions task.


provide cpptoml vendor package

Introducing a vendor package for cpptoml which resolves #67
Signed-off-by: Karsten Knese<EMAIL_ADDRESS>

Fine with colcon now but breakes the build with the ./iceoryx_build_test.sh script in /tools
CMake Error at CMakeLists.txt:184 (find_package):
By not providing "Findcpptoml_vendor.cmake" in CMAKE_MODULE_PATH this
project has asked CMake to find a package configuration file provided by
"cpptoml_vendor", but CMake did not find one.
Could not find a package configuration file provided by "cpptoml_vendor"
with any of the following names:
cpptoml_vendorConfig.cmake
cpptoml_vendor-config.cmake

can you try to change find_package(cpptoml_vendor REQUIRED) to find_package(cpptoml_vendor QUIET)?

QUIET solves it so far.  Maybe there is a better solution @elBoberido


Sign up issue in user.py

There is problem while signing up in user section.User can't sign up.
Please assign me this issue under gssoc24 label

this issue is already raised  #118.


Error when serving conecction: body size exceeds the given limit

My actions before raising this issue

[ X] Read/searched the docs
[ X] Searched past issues

I got a custom Darknet Model running on CVAT, capable to annotate images automatically, however, when I try to use the same model to annotate a video, an error its thrown which says the same as the title.
I tried to remove the maxRequestBodySize parameter in my function.yaml but the error still persist.
Expected Behavior
The video should get annotated.
Current Behavior
Video it's not annotated

Serverless functions in CVAT do not work with videos.
When you want to annotate the whole video (a task, I assume),

it sends frame by frame to the serverless function, get results and saves them.

Doing so still throws the same error as the title, just for claritifaction, the model I'm running uses this configuration:
triggers:     myHttpTrigger:       maxWorkers: 1       kind: 'http'       workerAvailabilityTimeoutMilliseconds: 1000       attributes:         maxRequestBodySize: 33554432         readBufferSize: 33554432

Hello，

I check the docker logs:
error when serving connection "<IP_ADDRESS>:8080"<->"<IP_ADDRESS>:33830": body size exceeds the given limit

@zhoupan163  did you manage to solve the issue?
I'm stuck with the same problem

@zhoupan163 did you manage to solve the issue? I'm stuck with the same problem
Yes, I have already resolved it，Thank for your help


handlers not firing with version 0.9.0

Using version 0.8.0, everything works great. When I upgrade to 0.9.0, I can't get any of my handlers to fire. Any idea why that might be?
Roughly my components look like this:
const keyMap = {
  onEnter: 'return'
};

const handlers = {
  onEnter: this.doStuff
};

<HotKeys keyMap={keyMap} handlers={handlers}>
    ...
</HotKeys>

I tried attached focused and attached={window} because I thought maybe they weren't in focus, but that didn't work, either.

Spent 2 hours on this, and I had the same exact problem.


Update for API 1.19 (Docker 1.7)

Change port mapping to a map from ports to empty structures. This
seems to be the new format even though it's not documented.
Update test for the new error message

++


moov: raise query params to their noun level types

This restricts the query params usage to endpoints where they will affect the response. callArg is generic and could be passed anywhere.
Goes in line with some changes from https://github.com/moovfinancial/moov-go/compare/main...ledg-2674#diff-caf877c03a3ddfb0c7c4f1e3621afac6eb86c2d3de6a85db762d9fbcb9db8b73

Codecov Report
Attention: Patch coverage is 38.46154% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 32.32%. Comparing base (47b3427) to head (a9959a2).
 Consider uploading reports for the commit 7fd4a2d to get more accurate results

Files
Patch %
Lines

pkg/moov/disputes.go
20.00%
8 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #105      +/-   ##
==========================================
+ Coverage   28.00%   32.32%   +4.32%
==========================================
  Files          20       21       +1
  Lines        1025     1058      +33
==========================================
+ Hits          287      342      +55
+ Misses        702      671      -31
- Partials       36       45       +9

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


Update from skeleton

🗣 Description
This PR updates this lost sheep to integrate our glorious cisagov/action-lineage automated update system. This includes bringing the testing up-to-date and migrating from TravisCI to GitHub Actions for the CI pipeline.

💭 Motivation and Context
This repository is used by other repositories that have been brought up-to-date, so it should also be up-to-date. Also, no repository left behind.

🧪 Testing
Automated testing passes.

✅ Checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All new and existing tests pass.

Great work bringing this one into the light! I have two minor changes to request:
1. Shouldn't `molecule.yml` be a symlink to `molecule-no-systemd.yml` instead?

2. If all the platforms in `molecule-no-systemd.yml` pass then we should update `meta/main.yml` accordingly.

You were right about which configuration molecule.yml should symlink to for this project. I had systemd on the brain and misread sysctl as systemctl when I was updating everything. Please see 643e478 for the change.
Unless I am missing something the meta/main.yml should now have all platforms in it since tests pass.


Peer-to-peer code review

Hi @astroboyReloaded!
Your project is really good designed, but i think there are some things that can be improved

I notice that you added a hover effect on the recent works cards, i think it would look so much better if add a transition for the hover effect to appear
Also in the form there are no placeholder on the inputs, you could add it to improve the user experience
I like how you do the mobile version but there something that could be improved in the form, i think would be better if you add more gap between the textarea and the button
Icons in the footer don't follow the figma template, they are spaced out and on figma they are more to the right

Solved! Thanks!!!


Update Respond JS to latest version

Looks like Respond JS 1.4.0 is included - can it be update to 1.4.2?
REF: https://github.com/scottjehl/Respond/blob/master/dest/respond.min.js

Good eye. I've got this setup as a build tool that will download a fresh Bower copy, but the respond.src.js it keeps getting is 1.4.0... :[
I suspect someone there updated some stuff and didn't do a new tag.
I've included the minified version which purports to be 1.4.2. /shrug


refactor(app): Apply updated UI styles

Overview
Addresses #9528 but applying all style update requests except the new dropdown component which will be covered in a follow up PR.
Changelog

refactor(app): Apply updated UI styles

Review requests
note: still using old dropdown component (TODO in next PR along with switching all Text to StyledText)

[ ] typography matches design
[ ] tertiary button matches design
[ ] modal matches design
[ ] new alert (for software update) now a banner, matches design
[ ] rest of app is not adversely affected by these styling updates
[ ] LL and PD are not affected by changes to radiogroup css optional changes (new prop)

Risk assessment
Medium only because the changes here can touch other products and or screens in app

Hey Katie,
Since I don't have my dev environment set up I can only review tertiary buttons and fonts.
Links for review:

Loom video explaining my notes
Figma link to my notes
Link to the ticket I'm referring to
Link to type sheet in design system


question exonuclease/phosphodiesterase

How do we correctly describe:
Uniprot
3'-5' exoribonuclease that posttranscriptionally removes uridine and adenosine nucleosides from the 3' end of spliceosomal U6 small nuclear RNA (snRNA), directly catalyzing terminal 2', 3' cyclic phosphate formation
SGD
Putative phosphodiesterase specific for U6 snRNA 3' end modification; trims the 3' poly(u) tract to leave a terminal 3' phosphate; human homolog, hUSB1 (aka C16orf57) produces a 2',3' cyclic phosphate;
I see it variously described as:
GO:0000175 3'-5'-exoribonuclease activity http://www.uniprot.org/uniprot/Q9BQ65 USB1 human
GO:0008081 phosphoric diester hydrolase activity  SGD USB1
Should there be a more specific child term to capture both of these terms?
Should there be a grouping term here for nucleotide phosphodiesterases?
http://www.ebi.ac.uk/QuickGO/GSearch?q=phosphodiesterase
Reported by: ValWood
Original Ticket: geneontology/ontology-requests/11346

So it seems that this one can be closed now?

My query in this one  was fixed independently by Rob Nash yesterday.
Hi Val,
I took a look at your annotation and the paper. It seems that the older paper we had used came to a different conclusion based on the evidence that had at the time. However, I think the evidence presented in the paper you used was solid and I think your choice of term was good, so I have removed our current function annotation, and replaced if with an ISS to mpn1. I have also added the paper to our db and updated our description. I don’t think the two terms should be related, so I don’t think one should be an ancestor of the other.
Cheers,
Rob


Fix tmp directory startup error

Fixes madmax startup error
Pulls the latest image and removed version numbers

Added version and signedBy back in - should be ready to merge now


Marshalling object

Can I do this for a case object?
trait Protocols extends DefaultJsonProtocol {
   implicit val ipInfoFormat = jsonFormat5(IpInfo.apply)
   implicit val ipPairSummaryRequestFormat = jsonFormat2(IpPairSummaryRequest.apply)
  implicit val ipPairSummaryFormat = jsonFormat3(IpPairSummary.apply)
}

Hi, no, not really, but have a look at https://github.com/lloydmeta/enumeratum - there's support for some JSON formats there, not spray-json unfortunately but it's pretty easy to add.


Can inference_sft use the cosyvoice 2.0 model?

This is the code I am currently using
@asynccontextmanager
async def lifespan(app: FastAPI):
model_dir = os.getenv("MODEL_DIR", "/workspace/CosyVoice/pretrained_models/CosyVoice-300M-SFT")
if model_dir:
logging.info("MODEL_DIR is {}", model_dir)
app.cosyvoice = CosyVoice(model_dir)
# sft usage
logging.info("Avaliable speakers {}", app.cosyvoice.list_avaliable_spks())
else:
raise LaunchFailed("MODEL_DIR environment must set")
yield
app = FastAPI(lifespan=lifespan)
@app.post("/api/inference/sft")
@app.get("/api/inference/sft")
async def sft(tts: str = Form(), role: str = Form()):
start = time.process_time()
output = app.cosyvoice.inference_sft(tts, role)
end = time.process_time()
logging.info("infer time is {} seconds", end-start)
return buildResponse(output['tts_speech'])
I would like to use your cosyvoice model and call it directly using the sft interface will it work?

可以，但不建议


selection.appendAll

What if this
svg.append("g")
  .appendAll("rect", data)

were shorthand for this
svg.append("g")
  .selectAll()
  .data(data)
  .enter()
  .append("rect")

The latter is a common special case of the data-join, and it seems worthwhile to support it explicitly? Some optimizations:

You don’t need a selector (pass undefined to selection.selectAll, which is already optimized)
You don’t need a key function (because the initial selection is necessarily empty)
You only need the enter selection (because the initial selection is necessarily empty)
You don’t need to call selection.order (because the initial selection is necessarily empty)

Besides optimization, the benefit (as I understand it) would be a more straightforward expression of intent in that quite common case; a drawback might be that this new method doesn't allow the user to move from "create" to the "update" pattern if that's what they need.

this new method doesn't allow the user to move from "create" to the "update" pattern if that's what they need.

That's a great point. Migrating code that is designed to be run only once to code that can be run multiple times is one of the biggest pain points of D3 (in my experience at least, this is what students especially struggle with the most).
Maybe the "update" pattern version of the shorthand could look something like this:
one(svg, "g")
  .joinAll("rect", data)

shorthand for
svg
  .selectAll("g")
  .data([null])
  .join("g")
  .selectAll()
  .data(data)
  .join("rect")

Yes, that’s historically why D3 hasn’t included a special API for this case and encourages you to use selection.join instead. But I think it’s totally reasonable to special-case the enter selection, as people already do when they chain enter-append instead of using selection.join.


Make openssl/rsa.h coexist with complex.h.

Use 'i' as parameter name not 'I'.
The latter causes problems when complex.h is #included.

[x] documentation is added or updated
[x] tests are added or updated

Fixes: #7224

Wait hold on, Travis fault relevant

Fix for Travis is in, waiting on the builds.

Merged, thanks for the corrections.

Should this go to 1.1.1?

+1 for cheery picking to both 1.1.1 and 1.1.0

Merged to 1.1.0 and 1.1.1.


benchmark result

I found there is a benchmark: http://calvin.cs.uni-freiburg.de/
Is this a public open benchmark?
But I can not find the  submission  place .Where can we submit  the evaluation results?

Yes, you can run the CALVIN evaluation as described in the README here and if you get a performance that beats our baselines (incl. HULC), then contact Oier to add your model to the leaderboard.


fix: [M3-8758] - Use default v8 reporter in coverage:summary job

Description 📝
Since merging https://github.com/linode/manager/pull/11049 I noticed an issue with the coverage failing often due a potential memory leak when the job starts. After doing a lot of profiling with vitest I wasn't able to find a culprit, and all tests are running just fine and it appears coverage:summary is the only one throwing this error (which could be a red herring too).

Either way, removing the junit reporter and using default v8 reporter (like all other vitest jobs) gets rid of the issue and allows the coverage script to run smoothly.
Changes  🔄

Use default v8 reporter in coverage:summary job

Target release date 🗓
Please specify a release date to guarantee timely review of this PR. If exact date is not known, please approximate and update it as needed.
How to test 🧪
Reproduction steps

run yarn coverage:summary in current develop branch, notice the error after a few seconds of running the job, and sometimes getting test errors as a result

Verification steps

run yarn coverage:summary in this branch, notice no error and job succeeding reliably with v8 reporter

As an Author I have considered 🤔
Check all that apply

[ ] 👀 Doing a self review
[ ] ❔ Our contribution guidelines
[x] 🤏 Splitting feature into small PRs
[x] ➕ Adding a changeset
[ ] 🧪 Providing/Improving test coverage
[ ] 🔐 Removing all sensitive information from the code and PR description
[ ] 🚩 Using a feature flag to protect the release
[x] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

well - womp womp, putting this back into draft, this is strange, NodeTable.test.tsx fails only on the coverage job, and of course not locally.

I do not think https://github.com/linode/manager/pull/11049 was actually responsible for this showing up. it may have been happening for a while. I will keep an eye on future failures and close this non-fix


Looking for contributors and maintainers

I could use some help maintaining or adding features to rust_gpiozero. It is intended to be a port of the python-based gpiozero library. I would be grateful for any kind of help. Thanks

I was actually thinking of starting something like this, as I have some python code I wanted to re-write in Rust - but I haven't got around to it yet.  I suppose instead, I could help maintain this project.  Is there a reason it's licensed GPL?  This might limit the number of people who use the library, rather than something like the ZLib license which is made specifically for statically-linked libraries.

I’m learning and could also pitch in!

I'd be happy to contribute here as well!

hey @AldaronLau @karisair @azjkjensen thanks a lot!
Any suggestions on how to proceed?

@AldaronLau no particular reason for choosing that license. perhaps might make more sense to got with MIT/Apache-2.0 as other rustlang crates? thoughts?

I think I'll have more ideas on how to proceed with the direction of the crate after I start using it in a personal project.  I will review all of the code and see if anything stands out.  I think for the current open issues and the one current pull request, we could assign one of us to work on each depending on who's interested in what.  As for adding additional maintainers (which I'm defining as people who merge pull requests into master, respond to issues, and can release new versions on crates.io), you'll probably want to make sure that they're people who have similar goals with you for this project before giving collaborator status.  That way you won't always have to be involved with every change made to this library (which I'm guessing is your goal).
As for the license, I think MIT/Apache-2.0 is fine and consistent with the rest of the ecosystem, but I personally choose Zlib/Apache-2.0 for crates I've started - as it's essentially the same with the minor exception that Zlib/Apache-2.0 doesn't require the license to be included in binary applications that depend on the library.  But, either one is good.

I'm fine with dividing up the existing work. Let me know which issue/PR to take.

thanks @azjkjensen @AldaronLau
I will add issues for further work to be done. feel free to add issues you see relevant

Feel free to comment on those you would be willing to work on
@azjkjensen perhaps you can help test the current open PR?

Sounds great, I'll check it out.

I'm going to look into implementing bounce_time for button presses...

I'm going to look into implementing bounce_time for button presses...

I'd be happy to help move from Travis CI to GitHub Actions and get the builds working

thanks @dcroote that would be great. we should also automatically publish to crates.io

@rahul-thakoor - if you're willing, I'd be happy to assist in basic maintenance and some PR reviews when I have the availability. I recently used this package to build a small project on my pi zero w and would like to support others wishing to contribute and do the same.

hey @dcroote awesome! i would really appreciate that :)


Rigidbody. Number of contacts reported < 0?

Godot version:
3.X
OS/device including version:
Any?
Issue description:
It can be set -1? Or in editor settings not set min-max value?
This not critical, but ... strange.
Or if it set -1, then one substract from contacts reported? (Don`t know how ^_^)
If it set -X... unpredicteble logick results.

yep, just need some tweaks in editor - min value.
i missclick and not see "-" before number.

But I agree that it shouldn't be allowed to be negative.


operation response type with headers

#746
#1034
#1037
~remove special handling for single responses~
adjust non-single responses to be like single responses. See analysis in  https://github.com/ctaggart/azure-sdk-for-rust/pull/17
return new type operation::Response(azure_core::Response)

typed headers() body() functions
allow getting the azure_core::Response. may be implement AsRef<azure_core::Response>

https://github.com/Azure/azure-sdk-for-rust/blob/2eadbde60c2653a9dc172552ca617450679fd512/services/svc/blobstorage/src/package_2020_10/operations.rs#L407-L412
specification\storage\data-plane\Microsoft.BlobStorage\preview\2020-10-02\blob.json

    "/?restype=account&comp=properties": {
      "get": {
        "tags": [
          "service"
        ],
        "operationId": "Service_GetAccountInfo",
        "description": "Returns the sku name and account kind ",
        "parameters": [
          {
            "$ref": "#/parameters/ApiVersionParameter"
          }
        ],
        "responses": {
          "200": {
            "description": "Success (OK)",
            "headers": {
              "x-ms-client-request-id": {
                "x-ms-client-name": "ClientRequestId",
                "type": "string",
                "description": "If a client request id header is sent in the request, this header will be present in the response with the same value."
              },
              "x-ms-request-id": {
                "x-ms-client-name": "RequestId",
                "type": "string",
                "description": "This header uniquely identifies the request that was made and can be used for troubleshooting the request."
              },
              "x-ms-version": {
                "x-ms-client-name": "Version",
                "type": "string",
                "description": "Indicates the version of the Blob service used to execute the request. This header is returned for requests made against version 2009-09-19 and above."
              },
              "Date": {
                "type": "string",
                "format": "date-time-rfc1123",
                "description": "UTC date/time value generated by the service that indicates the time at which the response was initiated"
              },
              "x-ms-sku-name": {
                "x-ms-client-name": "SkuName",
                "type": "string",
                "enum": [
                  "Standard_LRS",
                  "Standard_GRS",
                  "Standard_RAGRS",
                  "Standard_ZRS",
                  "Premium_LRS"
                ],
                "x-ms-enum": {
                  "name": "SkuName",
                  "modelAsString": false
                },
                "description": "Identifies the sku name of the account"
              },
              "x-ms-account-kind": {
                "x-ms-client-name": "AccountKind",
                "type": "string",
                "enum": [
                  "Storage",
                  "BlobStorage",
                  "StorageV2",
                  "FileStorage",
                  "BlockBlobStorage"
                ],
                "x-ms-enum": {
                  "name": "AccountKind",
                  "modelAsString": false
                },
                "description": "Identifies the account kind"
              },
              "x-ms-is-hns-enabled": {
                "x-ms-client-name": "IsHierarchicalNamespaceEnabled",
                "type": "boolean",
                "description": "Version 2019-07-07 and newer. Indicates if the account has a hierarchical namespace enabled."
              }
            }
          },
          "default": {
            "description": "Failure",
            "headers": {
              "x-ms-error-code": {
                "x-ms-client-name": "ErrorCode",
                "type": "string"
              }
            },
            "schema": {
              "$ref": "#/definitions/StorageError"
            }
          }
        }
      },
      "parameters": [
        {
          "name": "restype",
          "description": "restype",
          "in": "query",
          "required": true,
          "type": "string",
          "enum": [
            "account"
          ]
        },
        {
          "name": "comp",
          "description": "comp",
          "in": "query",
          "required": true,
          "type": "string",
          "enum": [
            "properties"
          ]
        }
      ]
    },

Fixed by #1084.


why VsyncWaiterAndroid need call postFrameCallback on platfrom thread?

i'm learning Flutter engine source code recently, but one thing really make me confused.
Android request vsync on platrform thread.
void VsyncWaiterAndroid::AwaitVSync() {
  auto* weak_this = new std::weak_ptr<VsyncWaiter>(shared_from_this());
  jlong java_baton = reinterpret_cast<jlong>(weak_this);

  task_runners_.GetPlatformTaskRunner()->PostTask([java_baton]() {
    JNIEnv* env = fml::jni::AttachCurrentThread();
    env->CallStaticVoidMethod(g_vsync_waiter_class->obj(),     //
                              g_async_wait_for_vsync_method_,  //
                              java_baton                       //
    );
  });
}

why not call directly asyncWaitForVsync like ios platform？
Choreographer.getInstance() .postFrameCallback support to be called on any thread.

Keeping it open and labeling it for further input from the team.
/cc @chinmaygarde

We did try that on Android, https://github.com/flutter/engine/pull/29889.
But it was reverted for some reason, we will try to reland again after this issue https://github.com/flutter/flutter/issues/94371 fixed.


the accuracy on the OCID_grasp dataset and the problem of the stack

Hello! thanks for your excellent work!
I have some questions:
1.when I run the test part, I only got the visible results—sets of images with grasp candidates and segmentation results. How can I get the grasp accuracy and seg . IoU? I can't find them.
2.can you tell me what the operation of the stack is in the grasp refinement head?
Thank you very much!

Hi,
really sorry for the late reply!
About 1): The evaluation script is currently not part of the repository. We plan to add it in the next days.
About 2): The input for the MLP is a 4d tensor of shape (NxCxHxW), where W,H are width and height of the feature map. C is the dimension where we stack the two feature maps, and we use the torch.cat()
operation to do so. We generate these feature map pairs N-times, where N defines the number of grasp candidates that we want to refine.
@emptybottle, the grasp refinement head is currently not part of this repository.

Hi, really sorry for the late reply! About 1): The evaluation script is currently not part of the repository. We plan to add it in the next days. About 2): The input for the MLP is a 4d tensor of shape (NxCxHxW), where W,H are width and height of the feature map. C is the dimension where we stack the two feature maps, and we use the torch.cat() operation to do so. We generate these feature map pairs N-times, where N defines the number of grasp candidates that we want to refine.
@emptybottle, the grasp refinement head is currently not part of this repository.

thank you!

Evaluation script for calculating Jaccard index and segmentation IOU was added in commit https://github.com/stefan-ainetter/grasp_det_seg_cnn/commit/2492d5ec78f831c327e817246e822cdfce9e16ad.


VS2015 Support

Is it possible to you could have a copy of projects in VS2015 format?
I tried opening, and it naturally failed on VS2017 project format.

Hi @datasmithtech
No problem. There is already a branch called VS2015 which is in Visual Studio 2015 format.
One thing to note is that Microsoft say that they only support .NET Core projects in VS2017. Yes, such as the ASP.NET Core version of my project will work in VS2015, but you might run into problems if you try to update it.


[feature request] add heatmap

could we add a feature for showing the heatmap of daily questions solved too

I agree, that will be helpful.

@PatternFinder
You can now use ?ext=heatmap to enable heatmap.

PR: #64

that's awesome bro!


Updated bad link.

The previous link went to the docs, with the faq tag. But nothing is tagged as faq in the docs. I updated the link to the osv.dev about page, which is presented as an faq.

/gcbrun


Quickbooks Online Linked Service - Connection Failed - Cannot Resolve Hostname

Followed the information on this page which was very helpful.  I gathered all the pieces required on the QBO side; however the connection is failing with:
ERROR [HY000] [Microsoft][QuickBooks] (20) Error while attempting to use REST API: Couldn't resolve host name
Is this a problem with the end point? (currently noted as quickbooks.api.intuit.com in the information page) or something else within ADF that needs to be configured to reach the host?

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

ID: 998a28b0-5278-73e2-7e27-b01997807797
Version Independent ID: c7a902d0-b098-0e3d-ef13-3f5f3082382d
Content: Copy data from QuickBooks Online using Azure Data Factory (Preview) - Azure Data Factory
Content Source: articles/data-factory/connector-quickbooks.md
Service: data-factory
GitHub Login: @linda33wj
Microsoft Alias: jingwang

@syntrixsystems  Since this issue is not related directly to the doc, I would recommend you to create a thread on the forums - Microsoft Q&A or Stack Overflow
Once you post your issue on forums, it will have visibility across the community which is a better suited audience for such types of issues.

It is still related to the document.  The info in the document was used to create the linked service which isn't working.  Perhaps the documents should outline prerequisites in ADF for the connection to succeed as well.

@syntrixsystems  Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

#reassign:jianleishen

@syntrixsystems thank you for your feedback. for the error mentioned above, please file a ticket if the connection doesn't work for root cause. without that, it's difficult to improve the doc or identify if it's a specific config issue, etc.
@CHEEKATLAPRADEEP-MSFT please close this given the info here is not sufficient to improve the doc.


feat(proxy): add proxy

set proxy from env

Thanks the PR. Could you help to add the proxy for all the request? Such as the function createClient().
Your PR would be serve when the default environment proxy was set.
And I have a idea that media-get can provide the options to let the user set diffrent proxy per website/source.
For example. I would like to set the a proxy for youtube and set the b proxy for douyin.
I need to make a friendly options design for the feature. Both of the following is need.

default config file for media-get. Place to /$HOME/.media-get.json
cmd options

Not just for the user, after the feature launched, the CI would be run well 😄

The feature has been implemented by #12
Thx for your working anyway, bro.
Closed the PR.


fix(examples/coap): Various fixes

Description
These are simple fixes taken out of https://github.com/future-proof-iot/RIOT-rs/pull/348 – that PR is going back a few steps because I took a wrong turn, and this now I'm consolidating things into manageable changes.
This does introduce one FIXME, but the need for getting that piece of information was already there and just hidden by the broken code.

[x] I have followed the Coding Conventions.
[x] I have performed a self-review of my own code.
[x] I have made corresponding changes to the documentation.

Depends on #469.

I'm on that in parallel, but that's running through tests time and again. I did do a full local build test of the CoAP example, and exceeding what CI did also with and without IPv6 enabled.
Code that is not in CI has a tendency to rot, let's fix this in parallel. Even with no CI running, this PR makes the code better, and reduces the warnings I'm sifting through while looking for hints at what is keeping #469 back.

I'm on that in parallel, but that's running through tests time and again. I did do a full local build test of the CoAP example, and exceeding what CI did also with and without IPv6 enabled.

ok!

LGTM.


ArgumentError: unknown keyword: fallback

I'm adding refile to a new project and when trying to use attachment_url(@user, :profile_image, :fill, 300, 300, fallback: "default.png"). I'm getting this error, if I remove the fallback keyword it works.

Ok I found the problem, I was on refile 0.5.5 and even when I ran bundle update refile it wouldn't update because refile-s3 required refile version 0.5.5. So updating first refile-s3 made it work!.


Some minor improvements to command line arguments parsing

Two commits, mostly minor improvements, feel free to pick or drop either or close the whole MR.

cli: construct reduce duplication a bit
cli: use FromStr and fallback to parse MachineFormat and IRFormat

Not accepting PRs to this repo, sorry man. I do appreciate the help though!

No worries. Feel free to copy-paste the code and merge it as your own if you like. Just curious - what's the reasoning?

I just use GitHub as a code-hosting/CI solution for personal projects and as a portfolio of sorts, not for collaboration. Because of that, I like to make sure that all the code hosted under my account is actually my own.


Helm: refactoring and optimization

Main theme color is defined in index.html. Remove constant MAIN_THEME_COLOR from code generally
Add checking, that FeedsPanels and FeedsSelectors aren't null
Background image and Card background image are defined in version/ path. If these images are absent, we use theme main color instead of image.

Why I did it?
Because it should work, if we remove all non-core plugins from version/
I found these problems when worked on Scotland-version.

I started work with Scotland, and removed all non-core elements from version/ path.
After this I found some problem in Core/ part. So, these changes - to solve these problems.
If it is necessary, I can explain each change

@fzaninotto
Francois, I fixed all notes, besides contrast mode via material-ui theme. Can we do this later?
I need in time to understand, which styles settings should we create (because current styles are more flexible in comparisom with initial material-ui styles)

as for the styles, it's up to @tony-shannon to prioritize.

Also, you can put custom elements in a material ui theme, to theme your own elements.

@fzaninotto
Francois, I changed color defenition via Material-UI theme settings. I created two stylesheets - for light and for dark mode, and removed all redundant styles.

@fzaninotto
What I did:

Update material-ui to the latest version;
Fix all styles, which were broken because of it;
Create theme object in window.config and merge it with default theme in createTheme()

PR was updated.

@fzaninotto
Francois, I changed version of Material-UI to 1.5.0 and updated the PR.
I tried to solve the issue with overriding default theme by window.config. But in this case we loose possibility to toggle contrast mode and to set background for cards, if card image is absent.
Besides this, we should use window.config information in UserTour component, because react-joyride library doesn't work with Material-UI


Remove traces of AB3

closes #254

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


[FR] Add --offline

There are some inconvenient workarounds for using this software without making an internet connection (which adds considerable latency on unstable networks). This use case should see official support. I propose adding the latest versions of the tokenizers to the pip package, and just using them without checking for updates when the user supplies --offline. Of course, the tokenizers can be updated any time the user doesn't use this flag.
To summarize, I propose two changes:

Include the needed tokenizers in the initial download, such that after installing the package with pip, it is ready to go.
Auto-update these tokenizers whenever the user does NOT supply --offline.

PS: I have skimmed the workaround, and while it works for offline usage, I am not sure it would solve the latency issue on an online machine. The workaround needs lots of manual steps, too, and it's not just some script we can run and be done with it.
Related:

https://github.com/openai/tiktoken/issues/279
https://github.com/openai/tiktoken/issues/232

After taking a look at the code, it seems just setting TIKTOKEN_CACHE_DIR on an online machine should be enough to avoid redownloading these files. But the latency is still high. I use this script:
#!/usr/bin/env python3
##
import os
import tiktoken
import sys

##
HOME = os.environ["HOME"]

tiktoken_cache_dir = f"{HOME}/tmp/tiktoken_cache"

os.environ["TIKTOKEN_CACHE_DIR"] = tiktoken_cache_dir
##

def num_tokens_from_message(message, model="gpt-4"):
    encoding = tiktoken.encoding_for_model(model)
    num_tokens = len(encoding.encode(message))
    return num_tokens

message = sys.stdin.read()

num_tokens = num_tokens_from_message(message, "gpt-4")

print(num_tokens)

And I have ~0.26s latency for counting a single token!
❯ echo | time openai_token_count.py
1
openai_token_count.py  0.28s user 0.04s system 94% cpu 0.339 total; max RSS 93200

This kind of latency is terrible.

@NightMachinery The issue of .tiktoken file getting deleted on Linux has been fixed. Now, I'm able to use tiktoken in offline mode by setting the environment variable TIKTOKEN_CACHE_DIR.
import os
os.environ["TIKTOKEN_CACHE_DIR"] = "path/to/tiktoken_dir"

# rest of tiktoken code

@nkilm Please report latency. The latency is still terrible even with the file not deleted, isn't it?


maybe_idx allocating for heterogeneous systems

for heterogeneous systems, we are using Arrays of abstract type VertexFunction. In this case maybe_idx is allocating.
Because the f! of each Vertex may be different, we get a type instability :Any
Maybe we could take the inital vertexlist and keep only the unique elements, as well as a set of node indices for each element and then iterate over those

you mean that instead of
struct nd_ODE_Static{G, GD, T1, T2}
    vertices!::T1
    edges!::T2
end

where vertices <: Array{<ODEVertex} we would get something like
vertices <: Tuple{ODEVertex{type1}, ODEVertex{type2}, ODEVertex{type3}, ...}

together with some indices
vierticeidx <: NTuple{nr_off_diff_vertex_functions, Vector{Int}}

and than we'd have a separate homogenious core loop over every "type" of vertex function?
for (i, odevert) in enumerate(vertices)
   indices = verticeidx[i]
   # do core loop for odevert and indices
end

I'm pretty sure vertices[i] is not type stable unless i is a compile time constant. We have to try whether the type inference works through for odevert in vertices.... otherwise one might need to use a @generated function to manually unroll the loop over the N different vertex types...

i already checked it, and the tuple loop is not type stable :(

at the moment i manually unrolled the tuple (just testing with 2 different vertices), but good to know that @generated is how you would do such a thing normally.

Well it is one possible solution the information is there to make it typestable. We could also use a package like https://github.com/StephenVavasis/Unroll.jl or just write a littel @unroll macro ourselfs. I haven't thought this throug yet but if it works it is much more straight forward than using @generated functions

is it typstable with the manually unrolled loop? I am not 100% sure that getindex(::Tuple, i) is stable for compile time constant i

its type stable

there are still allocations though, i just have no idea why

partially solved with #91. ODE ODE and DDE Static cases remain open. Remove maybe_idx when this is finished.

https://github.com/PIK-ICoNe/NetworkDynamics.jl/pull/91#discussion_r687443366

solved by #98


Stop mocking rest-hooks

There are a few instances[1] where we test components by using jest.mock on rest-hooks, which is not recommended: https://resthooks.io/docs/guides/unit-testing-components
PR #5587 adds a test helper for setting up rest-hooks fixtures in component tests, but it can't go in a shared location until we stop mocking rest-hooks.
Acceptance Criteria

The renderWithResolver method is available from CustomRenderUtils.tsx.
All component unit tests that need canned responses to rest-hooks calls are refactored to use renderWithResolver rather than jest.mocking rest-hooks.

[1] GitHub search is bad at this sort of thing, so here are the references:

https://github.com/CDCgov/prime-reportstream/blob/0fe7fbd220b93889a2f4473adf8db4f3f7ed1935/frontend-react/src/components/Admin/EditSenderSettings.test.tsx#L14
https://github.com/CDCgov/prime-reportstream/blob/0fe7fbd220b93889a2f4473adf8db4f3f7ed1935/frontend-react/src/pages/admin/AdminOrgNew.test.tsx#L14
https://github.com/CDCgov/prime-reportstream/blob/0fe7fbd220b93889a2f4473adf8db4f3f7ed1935/frontend-react/src/pages/daily/Table/ReportsTable.test.tsx#L34
https://github.com/CDCgov/prime-reportstream/blob/0fe7fbd220b93889a2f4473adf8db4f3f7ed1935/frontend-react/src/pages/submissions/SubmissionDetails.test.tsx#L26

going to close this as we are moving off of rest hooks, and this testing business will come out in the wash


fix for Logarithmic Scale Tick Bug for values of 0

Fix for iss#3917 and iss#2966.
The minNotZero value is based on fontSize now (like recomended here) (i closed the previous pr because of too many conflicts).

The proposed fix doesn't work for data not containing zero values.
I left a comment with test cases and a fix in the issue: #4790

A fix for this was implemented in https://github.com/chartjs/Chart.js/pull/4913


Update HRSC links in documentation

Beginning from section 4.2.1 in the POMM_AFIDS_User_Guide_v1.pdf document, there are several links mentioned pointing to http://i-mars.eu/web-gis/. I suggest changing these to the direct links https://maps.planet.fu-berlin.de as the i-mars links are just i-framed versions of the direct links, with a reduced window size.

Links updated.


Add allowed_shorthands option to Shorthands linter

Fixes #392
Uses @sds's recommended API:
linters:
  Shorthand:
    allowed_shorthands: [1, 4]

Thanks for the pull request!


Punching through uniffi to expose the cursor API

exposing the Cursor API

Nice! The code LGTM. We may want a little more context in the docs about when one might need a cursor etc, @heckj thoughts?

Question about Cursor - I get how it works based on the tests provided, but wanted to verify a few details:

Cursor works for both .Text objects and .List objects - it just happens to be more obviously useful for Text cursor tracking
Cursor also updates its position if/when you invoke "insert" to add an item to a list
There is no "default" cursor that you can retrieve from either Text or List objects - you have to manually set it at some point in time, and it updates with the changes from there
If a cursor is near the end of a list, and you spliceText/delete the last word, including where the cursor previously pointed, the cursor position updates to the UTF-8 index end of the text

And a question - as much for @munhitsu as @alexjg - right now we leave the whole tracking of a UTF-8view of a Swift String to the developer to manage the positioning in and out, ignoring the Character indexes (which point to full glyphs, not UTF8 index positions). Would it make sense to have an initializer on Cursor, referencing the Character Index within the String of the .Text object, that we automagically convert to a UTF-8 position when initializing the cursor to better developer ergonomics?

Correct on all four points. The idea of a "default" cursor is interesting, that would be the start of the sequence?
As to the character index stuff, that makes a lot of sense. Can it be done efficiently? Do you not need to track the state of the "current" version of the text in order to translate to UTF-8 indexes?

The idea of a "default" cursor is interesting, that would be the start of the sequence?

That was sort of the idea that I was starting with, and maybe exposing Cursor as an additional property on the higher-level AutomergeText object, initializing it to index 0 as a starting point. That said, I'm not sure if that makes sense in the scenario where you've created a .Text object, but not added any data into it yet. The cursor would, at that point, be at an invalid position as far as I could see. (Haven't tried anything experimental yet, just thinking through it)

As to the character index stuff, that makes a lot of sense. Can it be done efficiently? Do you not need to track the state of the "current" version of the text in order to translate to UTF-8 indexes?

My biggest concern is the whole "one glyph being multiple UTF-8 code points" scenario with String and UTF-8 positions. I think it makes a HUGE amount of sense to let the cursor shift with Automerge's updates, but when initializing a cursor from a Swift string value, using the character index from the current-value of string to convert into the UTF-8 view to get that appropriate index. If I recall correctly, there's a conversion method within String that lets you convert String.Index to String.UTF8View.Index pretty easily.

@munhitsu I'm closing this PR and replacing it with https://github.com/automerge/automerge-swift/pull/70, which targets the main branch. I cherry picked your commits over to that branch, then added some additional documentation.

If ever you want to get back to the idea of having the default cursor then bear in mind that UITextField thinks in terms of NSRange. This means 2 cursors.
Using the Cursor in anger made me also realise that spliceText would benefit from having variant with cursor as an argument, but this work should start from rust implementation...
Thank you for the merge!

I think I'll snag that into a future enhancement issue, thanks @munhitsu


Add test results matrix

Include summary test result matrix.

@shepazu Hi, Doug. I put the summary percentage for testing is in the Discussions tab.


Support for Dirham(AED)

Hello, are there any plans to get AED working with Fixer?

The source is the European Central Bank: http://www.ecb.europa.eu/stats/policy_and_exchange_rates/euro_reference_exchange_rates/html/index.en.html
Currently there's no AED.


MediaControl.PlayStateListener.onError() called instead of onSuccess()

Hello,
We have an issue regarding this library (we use 1.6 version):
By selecting any type of media (photo, media or music), we successfully connect to a Roku device, the MediaPlayer.LaunchListener.onSuccess(MediaPlayer.MediaLaunchObject object) gets called.
Here we check if we have a MediaPlayer.MediaLaunchObject.mediaControl, and if we do, we register a MediaControl.PlayStateListener to be informed of the media play status
The onSuccess(MediaControl.PlayStateStatus object) callback should be called, but instead the onError(ServiceCommandError error) gives us the "com.connectsdk.service.command.NotSupportedServiceCommandError" error object.
The error.getCode() returns 0 (zero value) and error.getPayload() returns null, so I don't have any way of knowing what the error means
However, the media selected continues to play, we can select another media and that will also be casted correctly, but each time we select another media, we get the onError(), instead of the onSuccess() callback and we need to know that the Status has changed to PlayStateStatus.Playing
Can you please tell me why this is happening,
Thank you

Hello again,
The issue described above for Android library is still a problem for us, as the Play/Pause behavior on a Roku Streaming Stick 3500X is also affected by the faulty onError(ServiceCommandError error) callback.
By not calling the onSuccess(MediaControl.PlayStateStatus object) with the adequate state, we cannot modify our UI accordingly to user actions. I must remind you that the media is played correctly, is just that the callbacks we receive from the library are not called correct, so we get onError instead of onSuccess
Can you please investigate this matter
Thank you


added new query IAM policies without groups

Closes #864

Fix the failing unit test and it's ready to be approved

Fix the failing unit test and it's ready to be approved

@ruigomescx - fixed :)


Remove price mock + Price history setup

Remove price mock
Fix tests open handles (writeSDLFile and httpServers)
Add price history (1 year)

This is the 2nd split of Price refactor https://github.com/GaloyMoney/galoy/pull/662

Please rebase on top of #709 when it's merged


Add a parameter to specify the maximum loop execution time

Right now GML-CLI has all safeguards off. It'd be nice to allow specifying the maximum time an application can loop before it throws an exception

Added in  acb53fc. Default timeout is now set back to 1000 ms, and supports inf to set to infinity.
Usage: -timeout [ms] or -timeout inf