[Refactor] define type length constants in thrift

What type of PR is this：

[ ] bug
[ ] feature
[ ] enhancement
[x] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #6482
Problem Summary(Required) ：

Remove the redundant code for default type length, and try best to make it consistent.

[FE PR Coverage check]
:heart_eyes: pass : 6 / 6 (100.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/catalog/PrimitiveType.java
3
3
100.00%

:large_blue_circle:
com/starrocks/catalog/ScalarType.java
3
3
100.00%

[FE PR Coverage check]
:heart_eyes: pass : 6 / 6 (100.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/catalog/PrimitiveType.java
3
3
100.00%

:large_blue_circle:
com/starrocks/catalog/ScalarType.java
3
3
100.00%

[FE PR Coverage check]
:heart_eyes: pass : 6 / 6 (100.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/catalog/PrimitiveType.java
3
3
100.00%

:large_blue_circle:
com/starrocks/catalog/ScalarType.java
3
3
100.00%

[FE PR Coverage check]
:heart_eyes: pass : 6 / 6 (100.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/catalog/PrimitiveType.java
3
3
100.00%

:large_blue_circle:
com/starrocks/catalog/ScalarType.java
3
3
100.00%

[FE PR Coverage check]
:heart_eyes: pass : 6 / 6 (100.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/catalog/PrimitiveType.java
3
3
100.00%

:large_blue_circle:
com/starrocks/catalog/ScalarType.java
3
3
100.00%

The constant generated by thrift is very hard to use, cause it's a plain global variable and has undetermined initialization order.
So temporarily close this PR, until we find a better way to fix it.


Release a new version to pypi?

Can you release a new version with the python3 fixes (#1 and #2)?
Thank you!

I upgraded the lsd algorithm version from 1.5 to 1.6, and solved the FileNotFoundError. Now it works on python3.
You can install it with pip install pylsd2


historicdata API

Add the ability to download files using the historicdata API, this can then feed into the 'create stream' function.

.get_my_data()
list all data purchased

.get_item_data_size(plan, sport, to_year...)
size of file to download

.get_collection_options(plan, sport, to_year..)
lists breakdown of market types / ids etc.

.download([plan, sport, to_year...], dir)
downloads file to dir

After mentioning this to betfair they have made the endpoints 'public':
https://historicdata.betfair.com/#/apidocs


Replace SciPy DOP853 with numbakit-ode

https://github.com/hgrecco/numbakit-ode/blob/main/nbkode/dop853.py

Hello, I discovered your project, and I find it very interesting. It would be my pleasure to contribute to this project. I am therefore interested in this issue.

Hi @Martinviv, welcome! Take your time to read the code, especially the cowell function in poliastro, which is the one using DOP853, and don't hesitate to ask questions.

That's what I'm already doing and I think I already have something that works (it passes the tests "test_propagation.py" on my computer). But I prefer to be sure and take the time to better understand the code before sending the PR

Great to know! :)

In fact I was mistaken the tests were working on the library and not on my code. But I'm sorry, I'm blocking pretty hard, so I'm going to abort  this issue and take time to better understand this project

No rush! :)

I am really happy to see numbakit-ode being adopted. I am really interested to know how it works for other people in other domains. Have you try to benchmark it? If you do, I usually split the benchmark in 3 parts

Solver instantiation
First step
Rest of the steps

The first two are still slow due to numba's inability at the moment to cache LLVM code and have function points. But you should get a speed boost in (3) which pays off for long simulations.

We haven't tried to benchmark it yet, but we'll keep your tips in mind :) Thanks!

For newcomers who want to try this labeled good first issue, here is more information about it:

In order to propagate an orbit inside an environment where natural or artificial perturbations exist (air drag, J2, solar radiation pressure...) the Cowell method is applied. This integrates the equations of motion using some numerical method, in this case DOP853, which is a Runge-Kutta based one.
This propagator is located under poliastro/twobody/propagation.py. Notice that the DOP853 is the one coming from scipy. You should modify the code in such a way that, the DOP853 class from numbakit-ode is the one being used inside the cowell propagator.
Finally, remember to add the numbakit-ode library as a dependency within pyproject.toml. Do not forget to run the tests locally, so you can be nothing broke.

Upon Implementing numbakit-ode library in the same ... I am facing some issues regarding the fact that the method nbkode.DOP853 need a function as an input parameter but previous implementation of a partial class as in the case with  functools.partial, the object thus returned is a class object so can't passed on as a parameter... Also the events parameter becomes null & void as it can't be called directly in the DOP853 function & hence there are errors in the computation of the answer ... any leads on how can this be resolved ?

@DhruvSondhi I think it would be easier to discuss over a draft PR :)

This proved not to be a "good first issue" at all :)

numbakit-ode dev here. It is great to see this, we will be happy to help. I am moving the more techincal discussion to hgrecco/numbakit-ode#11

Thanks @hgrecco ! Absolutely, hgrecco/numbakit-ode#11 is the place to ask for the stuff we need. In the meanwhile, we are trying to figure out how far we can go with the current state of affairs in #1049.

I tried with this diff:
diff --git a/pyproject.toml b/pyproject.toml
index d5e0b355..f352f6b3 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -38,6 +38,7 @@ requires = [
     "jplephem",
     "matplotlib >=2.0,!=3.0.1",
     "numba >=0.46 ; implementation_name=='cpython'",
+    "numbakit-ode",
     "numpy",
     "pandas",
     "plotly ~=4.0",
diff --git a/src/poliastro/core/propagation/__init__.py b/src/poliastro/core/propagation/__init__.py
index 34dc9177..7b8e4b57 100644
--- a/src/poliastro/core/propagation/__init__.py
+++ b/src/poliastro/core/propagation/__init__.py
@@ -36,7 +36,7 @@ __all__ = [
 ]

-def func_twobody(t0, u_, k):
+def func_twobody(t0, u_, params):
     """Differential equation for the initial value two body problem.

     This function follows Cowell's formulation.
@@ -51,6 +51,8 @@ def func_twobody(t0, u_, k):
         Standard gravitational parameter.

     """
+    k, = params
+
     x, y, z, vx, vy, vz = u_
     r3 = (x ** 2 + y ** 2 + z ** 2) ** 1.5

diff --git a/src/poliastro/twobody/propagation.py b/src/poliastro/twobody/propagation.py
index 634b350e..06a6f2b1 100644
--- a/src/poliastro/twobody/propagation.py
+++ b/src/poliastro/twobody/propagation.py
@@ -25,7 +25,7 @@ different propagators available at poliastro:
 import numpy as np
 from astropy import units as u
 from astropy.coordinates import CartesianDifferential, CartesianRepresentation
-from scipy.integrate import DOP853, solve_ivp
+from nbkode import DOP853

 from poliastro.core.propagation import (
     danby as danby_fast,
@@ -87,33 +87,19 @@ def cowell(k, r, v, tofs, rtol=1e-11, *, events=None, f=func_twobody):

     u0 = np.array([x, y, z, vx, vy, vz])

-    result = solve_ivp(
+    solver = DOP853(
         f,
-        (0, max(tofs)),
+        0,
         u0,
-        args=(k,),
-        rtol=rtol,
-        atol=1e-12,
-        method=DOP853,
-        dense_output=True,
-        events=events,
-    )
-    if not result.success:
-        raise RuntimeError("Integration failed")
-
-    t_end = (
-        min(result.t_events[0]) if result.t_events and len(result.t_events[0]) else None
+        params=np.array([k]),
+        # t_bound=max(tofs),  # Produces empty result!
+        # rtol=rtol,  # https://github.com/hgrecco/numbakit-ode/issues/16
+        # atol=1e-12,
     )
+    ts, us = solver.run(tofs)

-    rrs = []
-    vvs = []
-    for i in range(len(tofs)):
-        t = tofs[i]
-        if t_end is not None and t > t_end:
-            t = t_end
-        y = result.sol(t)
-        rrs.append(y[:3])
-        vvs.append(y[3:])
+    rrs = us[:, :3]
+    vvs = us[:, 3:]

     return rrs * u.km, vvs * u.km / u.s

However, I found several problems:

Running with NBKODE_NONUMBA=1 NUMBA_DISABLE_JIT=1 makes most tests pass, but many of them fail because of small numerical errors. I am not sure if the root cause of such errors is https://github.com/hgrecco/numbakit-ode/issues/11 or something else.

     This function follows Cowell's formulation.
@@ -51,6 +51,8 @@ def func_twobody(t0, u_, k):
         Standard gravitational parameter.

     """
+    k, = params
+
     x, y, z, vx, vy, vz = u_
     r3 = (x ** 2 + y ** 2 + z ** 2) ** 1.5

diff --git a/src/poliastro/twobody/propagation.py b/src/poliastro/twobody/propagation.py
index 634b350e..06a6f2b1 100644
--- a/src/poliastro/twobody/propagation.py
+++ b/src/poliastro/twobody/propagation.py
@@ -25,7 +25,7 @@ different propagators available at poliastro:
 import numpy as np
 from astropy import units as u
 from astropy.coordinates import CartesianDifferential, CartesianRepresentation
-from scipy.integrate import DOP853, solve_ivp
+from nbkode import DOP853

 from poliastro.core.propagation import (
     danby as danby_fast,
@@ -87,33 +87,19 @@ def cowell(k, r, v, tofs, rtol=1e-11, *, events=None, f=func_twobody):

     u0 = np.array([x, y, z, vx, vy, vz])

-    result = solve_ivp(
+    solver = DOP853(
         f,
-        (0, max(tofs)),
+        0,
         u0,
-        args=(k,),
-        rtol=rtol,
-        atol=1e-12,
-        method=DOP853,
-        dense_output=True,
-        events=events,
-    )
-    if not result.success:
-        raise RuntimeError("Integration failed")
-
-    t_end = (
-        min(result.t_events[0]) if result.t_events and len(result.t_events[0]) else None
+        params=np.array([k]),
+        # t_bound=max(tofs),  # Produces empty result!
+        # rtol=rtol,  # https://github.com/hgrecco/numbakit-ode/issues/16
+        # atol=1e-12,
     )
+    ts, us = solver.run(tofs)

-    rrs = []
-    vvs = []
-    for i in range(len(tofs)):
-        t = tofs[i]
-        if t_end is not None and t > t_end:
-            t = t_end
-        y = result.sol(t)
-        rrs.append(y[:3])
-        vvs.append(y[3:])
+    rrs = us[:, :3]
+    vvs = us[:, 3:]

     return rrs * u.km, vvs * u.km / u.s

However, I found several problems:

Running with NBKODE_NONUMBA=1 NUMBA_DISABLE_JIT=1 makes most tests pass, but many of them fail because of small numerical errors. I am not sure if the root cause of such errors is https://github.com/hgrecco/numbakit-ode/issues/11 or something else.


Merge mre/idiomatic-rust?

I have a feeling that you're doing a much better job maintaining this project than I do with https://github.com/mre/idiomatic-rust.
Do you think it makes sense to move the articles over to your place and create a new category called "idiomatic code" on Read Rust?
If so, feel free to copy over the contents from my repo. I can also create a PR for that if you like. 😊

Hi Matthias, seems like it would be relatively straightforward to port the page over to Read Rust. I'll look at doing it in the next few days. I'll raise a PR and add you as a reviewer when it's done. I also like your CI link checker. I may incorporate that too.


how to connect the link-state when we already started the createHttp link?

how to connect the link-state when we already started the createHttp link?
my snipped goes like this
const link = createHttpLink ({
   uri: "http://localhost: 4000/graphql",
   link: withClientState(schema),
   credentials: "include"
});

const client = new ApolloClient ({
   link,
   cache: new InMemoryCache()
});

but it still does not work!

@luizhenriquesoares you can compose links!

const http = createHttpLink ({
   uri: "http://localhost: 4000/graphql",
   link: withClientState(schema),
   credentials: "include"
});
const state = withClientState({ /* your code */ });
const link = state.concat(http)

@jbaxleyiii  thank you, my friend


[BUG] Search by project title doesn't appear to work

Software version
Current live version of the browser
Expected behavior
When typing keywords or the exact project title one would expect that project to be available to be selected
Actual behavior
When typing the exact title the message 'oops we can't find that project..' appears
Steps to reproduce the behavior

Go to the main data.humancellatlas.org browser or in the search bar of the 'explore tab'
Start typing the exact project title such as 'Ischaemic sensitivity...'

I think this kind of search used to work but I am not really sure when it stopped working

It would be very useful to search by either the "Project Title" or the "Project Label" (e.g. TissueStability).


[stable/airflow] flower dashboard cannot be accessed and liveness probe failure after upgrade or new install

Describe the bug
A clear and concise description of what the bug is.
Flower dashboard cannot be accessed in port 5555 and flower pod stuck in restart loop
Version of Helm and Kubernetes:
helm version
Client: &version.Version{SemVer:"v2.14.0", GitCommit:"05811b84a3f93603dd6c2fcfe57944dfa7ab7fd0", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.14.0", GitCommit:"05811b84a3f93603dd6c2fcfe57944dfa7ab7fd0", GitTreeState:"clean"}

kubectl version
Client Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.3", GitCommit:"721bfa751924da8d1680787490c54b9179b1fed0", GitTreeState:"clean", BuildDate:"2019-02-01T20:08:12Z", GoVersion:"go1.11.5", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.5", GitCommit:"2166946f41b36dea2c4626f90a77706f426cdea2", GitTreeState:"clean", BuildDate:"2019-03-25T15:19:22Z", GoVersion:"go1.11.5", Compiler:"gc", Platform:"linux/amd64"}

Which chart:
stable/airflow
What happened:
Flower dashboard cannot be accessed using port-forward at both service or directly from pod at port 5555 and liveness probe failed. After 5 fails it gets restated. Flower pod gets stuck in restart loop
Problem is in version 3.0.6 (current lastest) and not in 3.0.2, after reinstalling 3.0.2 everything works fine.
What you expected to happen:
liveness probe passed and flower dashboard accessible
How to reproduce it (as minimally and precisely as possible):
helm install --name air -f airflow/values.yaml --namespace airflow stable/airflow --version 3.0.6
kubectl logs -f air-flower-55cb567664-hxhm6 -n airflow
Anything else we need to know:
Logs
[2019-07-25 06:50:58,740] {{settings.py:182}} INFO - settings.configure_orm(): Using pool settings. pool_size=5, pool_recycle=1800, pid=1
[2019-07-25 06:50:59,080] {{__init__.py:51}} INFO - Using executor CeleryExecutor
[I 190725 06:50:59 command:136] Visit me at http://<IP_ADDRESS>:5555
[I 190725 06:50:59 command:141] Broker: redis://air-redis-master:6379/1
[I 190725 06:50:59 command:144] Registered tasks:
    ['celery.accumulate',
     'celery.backend_cleanup',
     'celery.chain',
     'celery.chord',
     'celery.chord_unlock',
     'celery.chunks',
     'celery.group',
     'celery.map',
     'celery.starmap']
[I 190725 06:56:14 command:44] SIGTERM detected, shutting down

Am new to using this chart and deployed it many times in last week, but also having exact same issue, with no idea how to debug from here. But going to try 3.0.2 now.

Having exact same issue!

Having the same issue here. For me the version 3.0.2 also fails to open flower dashboard.

After some investigation I found out the pods were having some issues connecting to redis and postgres and thats why Flower dashboard wasn't reachable.
What I did was to create the secrets for redis and postgres and included the environment variables on the values.yaml file. As follows:
Create secrets:
kubectl create secret generic airflow-postgres --from-literal=postgres-password=$(openssl rand -base64 13)
kubectl create secret generic airflow-redis --from-literal=redis-password=$(openssl rand -base64 13)
on values.yaml:
postgres:
  existingSecret: airflow-postgres

redis:
  existingSecret: airflow-redis

...
extraEnv:
  - name: POSTGRES_PASSWORD
    valueFrom:
      secretKeyRef:
        name: airflow-postgres
        key: postgres-password
  - name: REDIS_PASSWORD
    valueFrom:
      secretKeyRef:
        name: airflow-redis
        key: redis-password

This worked for me and I'm now able to reach Flower dashboard.

@mohannadbanayosi Your latest PR could also solve this issue?

@will-beta Yes I think it should solve it.
@deepaksood619 Can you try out version 4.0.5 of the chart? It has a fix for this.
The keys used for the secrets should be airflow-postgres and redis-password.

@mohannadbanayosi
I was having issues with the Airflow web server failing to start due to being unable to resolve the host name airflow-postgres. I tried installing 4.0.5 and now get the error
ModuleNotFoundError: No module named 'werkzeug.wrappers.json'; 'werkzeug.wrappers' is not a package

@benrifkind
Are you using the default values? I ran the following commands and all components were initialised properly:
helm repo update
helm install --name=airflow --version 4.0.5 stable/airflow
But if you use the docker image tag 1.10.2 it will break.

@mohannadbanayosi
Thanks for the response. Unfortunately I still get the same error. Full stack trace
executing webserver...
Traceback (most recent call last):
  File "/usr/local/bin/airflow", line 21, in <module>
    from airflow import configuration
  File "/usr/local/lib/python3.6/site-packages/airflow/__init__.py", line 40, in <module>
    from flask_admin import BaseView
  File "/usr/local/lib/python3.6/site-packages/flask_admin/__init__.py", line 6, in <module>
    from .base import expose, expose_plugview, Admin, BaseView, AdminIndexView  # noqa: F401
  File "/usr/local/lib/python3.6/site-packages/flask_admin/base.py", line 6, in <module>
    from flask import Blueprint, current_app, render_template, abort, g, url_for
  File "/usr/local/lib/python3.6/site-packages/flask/__init__.py", line 21, in <module>
    from .app import Flask
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 69, in <module>
    from .wrappers import Request
  File "/usr/local/lib/python3.6/site-packages/flask/wrappers.py", line 14, in <module>
    from werkzeug.wrappers.json import JSONMixin as _JSONMixin
ModuleNotFoundError: No module named 'werkzeug.wrappers.json'; 'werkzeug.wrappers' is not a package

Any other ideas?

Okay then just use version 4.0.7 of this chart and it should fix it. The latest version uses Airflow docker image 1.10.4 which fixes this issue (check this link)
So just run the following:
helm repo update
helm install --name=airflow --version 4.0.7 stable/airflow
and should be okay. Just make sure you delete any previous chart installations to have a fresh one.

@benrifkind
Okay then just use version 4.0.7 of this chart and it should fix it. The latest version uses Airflow docker image 1.10.4 which fixes this issue (check this link)
So just run the following:
helm repo update
helm install --name=airflow --version 4.0.7 stable/airflow
and should be okay. Just make sure you delete any previous chart installations to have a fresh one.

@benrifkind
Okay then just use version 4.0.7 of this chart and it should fix it. The latest version uses Airflow docker image 1.10.4 which fixes this issue (check this link)
So just run the following:
helm repo update
helm install --name=airflow --version 4.0.7 stable/airflow
and should be okay. Just make sure you delete any previous chart installations to have a fresh one.

@mohannadbanayosi
Just to close the loop on this. That install got rid of the ModuleNotFoundError but I am still running into the following error on the web pod:
sqlalchemy.exc.OperationalError: (psycopg2.OperationalError) could not translate host name "airflow-postgresql" to address: Temporary failure in name resolution

I execed into the pod to check if I could curl that pod and I was able to. This error only occurs when I am running this helm chart locally on Minikube. Installing it on EKS on Amazon works.

@benrifkind I replied to issue #16816 to move the discussion there.


Selection Highlight issue when selected text is followed by '-' in long files

When using pointers in C/C++ code the text would usually look like this:
ptr->member
In long files (I started seeing it from line 13840), when highlighting "ptr", matching strings in the file are not highlighted. However, when highlighting "ptr-" only then the matching strings are highlighted. Also, if ptr is mentioned earlier in the code and is not followed by '->', highlighting that string will not highlight the ptr occurrence which is followed by a "->" later in the code.

Can you please provide a full example of the behavior you're seeing?

Here's an example: unitPtr is selected but its occurrences few lines below are not highlighted.

Here's what happens when I also select the hyphen after unitPtr:

This time, the next occurrences are highlighted,
I would expect that selecting only unitPtr will highlight all its occurrences whether they are followed by hyphen or not.

Sorry, I wasn't clear enough, can you please proved a full example that I can copy/paste?

For that I need to share my workspace which I can't do.  You should try this with very long files and see if it reproduces.


Players cannot edit their inventories

What is the output link of /viaversion dump?
https://gist.github.com/anonymous/7d13b3e367df7548ed3c45f2c50e90b9
Are you using any additional software like BungeeCord? If so, what software and version? (Please list your plugins as well)
Plugins (39): icJukeBox, CoreProtect, AsyncWorldEditInjector, TopLite, WorldEdit, ClearLag, RSCommandSigns, TeleportationTools, Chairs, EasySetSpawn, Broadcaster, OpenTerrainGenerator, MultiWorld, AsyncWorldEdit, iDisguise, Reloader, ProVanish, PermissionsEx, NoteBlockAPI, MapLoader, ChatWithStaff, ViaVersion, GUIBanner, Tablist, Vault, ViaBackwards, GriefPrevention, ViaRewind, Essentials, Autorank, ProtocolLib, NoCheatPlus, SEARCH, TrollCommandsPlusPlus, PAC, ChatEx, ServerListPlus, AuthMe, HolographicDisplays
All are up to date.
How does this error happen? login? Using an item?
when on 1.12.2 my players can open their inventories but they cannot move objects in them in any way, in 1.11.2 it works just fine, and in older versions works too.
Is there an error in the console? Use pastebin.com. Is there a kick message?
there's only an error from my anticheat: PAC: user failed Inventory check [VL: 1] [Ping: 49] [Info: missing packet]

oh also every plugin is up-to-date. i checked myself

This is not an issue with ViaVersion, test without your anticheat.
ViaVersion only modifies versions that aren't your server version, since the issue happens on 1.12.2 this means it's down to another plugin (most likely PAC given the error).

it's not the anticheat, i tested it without.

I'd suggest removing plugins and testing to locate the cause.

oh it was actually pac AND search. 2 anticheats out of 3 don't work with viaversion. pac was saying missing packet, couldn't you fix it by emulating the missing packet somehow?

ViaVersion doesn't change anything on the version your on.
The older versions you have on your server are provided by ViaBackwards / ViaRewind so if there is a problem with a specific version you should post it on the respective github. (ViaVersion only provides new versions, eg. 1.12 on 1.11 server)


Introduce a center param to PCA (#734)

Associated GitHub Issue(s): https://github.com/dask/dask-ml/issues/734.
Tests/black/pyflakes passing: Yes.
Tests added to the following files:

tests/test_pca.py
tests/test_incremental_pca.py

tl;dr of design choices made:

IncrementalPCA doesn't get affected by the new parameter and IncrementalPCA(..., center=False ,...) is unsupported.
whiten=True does not imply, nor enforce center=True. Instead it warns about possible unexpected behavior, if the underlying data hasn't been centered already.

Once the maintainers confirm approval of the proposed changes, I'll be happy to update the .rst docs accordingly, to document the updated API, in a separate commit (as part of this PR).

would reformat /home/vsts/work/1/s/tests/test_pca.py
Oh no! 💥 💔 💥
1 file would be reformatted, 99 files would be left unchanged.

Oops - pretty sure that I did this check locally. Will re-run black to confirm.

Hi @TomAugspurger  - I'm writing to check if you've had a chance to look into the further changes that I pushed since your review, and whether you'd like any additional refinements to be done on top of that.
Thanks!

It'll be a week or so before I can take a look. Thanks for your patience!
In the meantime, perhaps @jameslamb has a chance to glance through this (no worries if you don't though)?

It'll be a week or so before I can take a look. Thanks for your patience!

It's all good - just wanted to check if this is still on your radar. I'm happy to wait as long as necessary, and incorporate potential further feedback.
Thanks for your confirmation, @TomAugspurger!

I had some time to go through and review today. Overall, I'm in favor of the change and I can understand how it could be a nice optimization for experienced users.
Please consider some of the review comments I left.
I also think it would improve understanding for people who find this pull request from search or see it in a changelog if you change the PR title to something like "Make automatic centering in PCA methods optional". By itself, the fact that there is now a parameter called center doesn't tell you much about the functional benefit of the changes in this pull request.

Thanks for your review, @jameslamb! I'll try to address them and make changes, wherever applicable, ASAP.


JSON accountid_other is me ?!

I'm trying to get the steam ID of the partner of my trade offer. The problem is that the 'accountid_other' is my bot account id !
At first, I tought it was my fault, but after debuging a bit, I found the "Offer" class inside the "TradeOfferWebApi" class and noticied that the property retourned by JSON is definitly my bot ID !
Can anyone confirm this ?

Problem is definitely on your end. The SteamID mapping in SteamBot TradeOffer class is correct.

Alright, well, I will redownload the bot and see if I have the same bug. Anyway, this shoul be correct, right ?

I invite someone to trade
We both add something to trade
I accept the trade offer
I confirm through my emails
"accountid_other" is the id of the guy I invited to trade with.

"accountid_other" is the id of the guy I invited to trade with

That's expected. (?)

Just to be sure...

Well, I don't really think it's my fault, because ID is wrong inside OnTradeAwaitingEmailConfirmation(long tradeOfferID) !
public override void OnTradeAwaitingEmailConfirmation(long tradeOfferID)
        {
            Log.Warn("Trade ended awaiting email confirmation");
            SendChatMessage("Please complete the email confirmation to finish the trade");

            SteamTrade.TradeOffer.TradeOffer offer = null;
            bool success = Bot.TryGetTradeOffer(tradeOfferID.ToString(), out offer);
            if(success)
            {
                SteamKit2.SteamID test = offer.PartnerSteamId;
            }
        }

I didn't edited anything...

Well, you already know other user's SteamID - it's OtherSID, inside your userhandler.
If PartnerSteamId equals bot's SteamID, it's Steam sending this information.

Alright, that's why. Thanks. But, actually, there is no way to get the PartnerSteamID inside the "Bot" class, right ? Because I'm doing the code above to get the trade offer inside the "Bot" class and I need to get the other player inventory to check wich items have been traded. Is their a way to do that ?

The snippet above won't work very well. You are retrieving the trade offer right after regular trading session ends. The trade offer will be in pending state.
Btw. You already know the list of items from the trading session.

Btw. You already know the list of items from the trading session.
Yeah, but I need to retrieve it when the user confirmed the trade via his emails.

That's OK, but the snippet is still invalid.

Why do you need the same list of items? It's doable of course.

the part 1. is working fine, so I guess, I shouldn't update my sniper (do I have to ? I can.)
2. I don't really get it, I just try to get wich items has been traded between users after trade offer has been confirmed via emails. That's it.

Also, this is working fine http://api.steampowered.com/IEconService/GetTradeOffer/v1/?key=AFCE0097A6C1DA09D0DD671CC9338542&tradeofferid=757379315&language=en
but this isn't : http://api.steampowered.com/IEconService/GetTradeOffer/v1/?key=AFCE0097A6C1DA09D0DD671CC9338542&tradeofferid=757657228&language=en
Why ?


Investigate release pruning

We are releasing the notebooks with meta ~23megs.
These may not be necessary here but we could point to notebooks in the individual packages.

We can close this now, correct?


Require separator between identifier and value

The recent change (https://github.com/sirwart/ripsecrets/commit/c40ddbd63c7bf6f08c7e7fe11bfe2a751cf23368) made the separator between identifier and value optional, which results in more false positives. I think this change makes the behavior more reasonable.
Alternatively, we can go back to requiring an assignment operator.

From BIND 9 config: https://bind9.readthedocs.io/en/v9_16_5/advanced.html#loading-a-new-key
That's a pretty niche use case though and I don't have any more examples.


Add option to rotate the calibration canvas

Some people have setups that are flipped from their computer. Someone asked to have an option to rotate the canvas to make calibration more intuitive.

Will this even be possible? Like, how are they going to use a mouse to find the correct corner? If they use keys, then shouldn't the color-coded corners solve the problem?
...I'm having trouble visualizing this one technically

I think you're right! The only way it would make sense is by remapping arrow keys but then I'd be scared someone else would stumble into it and be confused by what's happening. Maybe close this and suggest changing the display settings on their desktop to make it more intuitive.

Maybe close this and suggest changing the display settings on their desktop to make it more intuitive.

That, or maybe we try to get them to use colored corners and see if they can figure it out from there. I'm not even sure if desktops have a setting like that 😂
Do they not have this problem in projector programs (Adobe, Inkscape, etc) already today?

Hmmm, maybe during calibration...they can "swap" the colored corners of the grid when the corner highlights are on? This will of course only affect arrow keys...but maybe?
And then keep that transform for PDFs? Maybe that would work...

On a mac you can rotate the display to match your setup so I think we should just leave it at that. It seems like a pretty special case! I guess we could return to this if it keeps coming up.

Yes! It seems like people so far prefer keyboard based movement.

Ok, if that's the case, then I can work on this if you want. I actually think I get it now 😂

You'll notice I opened a PR for key management and not for rotation.  Toggling the corners wasn't too bad - it was "flipping" the calibration grid while not actually flipping the calibration grid.
My plan was not to apply css rotate, but to modify the height/width parameter on the grid itself by swapping them.
I tried swapping height and width of the grid before calling draw(), but that turned out to only partially work. Some of the grid rotated, but the black background did not (and honestly, maybe some of the other parts of the grid didn't either, it was hard to tell and pretty wonky).
After that, I decided to focus on the precursor to rotation instead, the key handlers.

Lol, I think that's a good idea! Sounds like more trouble than its worth. I suggested changing the rotation for the projector display and she seemed happy with that.


Truss element boundary conditions

Closes #424
Closes #423
Preparation for #422

Codecov Report

Merging #425 (1e09a27) into main (dcddb42) will decrease coverage by 0.35%.
The diff coverage is 54.54%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #425      +/-   ##
==========================================
- Coverage   86.48%   86.14%   -0.35%
==========================================
  Files          43       43
  Lines        1635     1638       +3
==========================================
- Hits         1414     1411       -3
- Misses        221      227       +6

Impacted Files
Coverage Δ

src/Entities/Trusses.jl
92.85% <0.00%> (-5.88%)
:arrow_down:

...rc/StructuralModel/StructuralBoundaryConditions.jl
87.87% <100.00%> (-0.47%)
:arrow_down:

... and 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Will a RegExp be allowed as the first argument?

The current split method accepts a RegExp as the first argument.
In that case the matched text is included in the returned array.
const a = 'a|b|c|d';
a.split(/|/);    // ['a', '|', 'b', '|', 'c', '|', 'd']
a.split(/|/, 2); // ['a', '|']

Will splitn allow RegExp arguments?
If so, how will it treat them?

The matched text is not included: you need to escape | in your regular expressions.

Doh!
nevermind sorry.

Well, it still needs to be clarified whether RegExps are allowed, right?

Wait!, I'm not crazy after all
The "include the separators" behavior is triggered by the expression including groups.
a.split(/(\|)/);
(7) ['a', '|', 'b', '|', 'c', '|', 'd']

Oh well, I didn't know about that! Then the initial question still holds completely.

Yikes. Every group is included.
const b = 'a:sep1:b:sep2:c:sep3:d:sep4:e';
b.split(/((:)(\w+)(:))/); // ['a', ':sep1:', ':', 'sep1', ':', 'b', ':sep2:', ':', 'sep2', ':', 'c', ':sep3:', ':', 'sep3', ':', 'd', ':sep4:', ':', 'sep4', ':', 'e']

This definitely breaks the goal of having split be the opposite of join.

Will include this. Yes, it should support regexp, just like perl does for example.

I believe it depends on the method name we finally choose.
If it is the name "splitn" or "splitN" I think we should support regexp. If choose another name like "explode", supporting regexp is not a must.
Precedent: String.prototype.replaceAll, the early version of it only support string for the best perf and the main pain point (s.replace("foo") doesn't work as expect ) in usage, but committee require it must support regexp unless it change to some other name like "substitute" .


Update zio-test to 1.0.12

Updates dev.zio:zio-test from 1.0.9 to 1.0.12.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio", artifactId = "zio-test" } ]

labels: library-update, semver-patch

Superseded by #65.


Error in sampling the template from uniform distribution?

Thanks for making the code available for this.
Looking through the RandStainNA implementation I was wondering if there is a typo here: https://github.com/yiqings/RandStainNA/blob/master/randstainna.py#L153
tar_std = np.random.uniform(
    low=self.channel_avgs.avg[idx] - 3 * self.channel_avgs.std[idx],
    high=self.channel_avgs.avg[idx] + 3 * self.channel_avgs.std[idx],
)

Shouldn't this be?
tar_std = np.random.uniform(
    low=self.channel_stds.avg[idx] - 3 * self.channel_stds.std[idx],
    high=self.channel_stds.avg[idx] + 3 * self.channel_stds.std[idx],
)

@amtsak Hi Anna, exactly. Thanks so much for pointing out the typo. We'll fix this very shortly.


Feature #17 eclass application classification classes

How to reference an ECLASS application or classification class? #17

Check again, after talking with ECLASS experts

Check wether the asset category of the Technical Data submodel template can be used.

on hold -> waiting for a joint publication of ECLASS and PI4.0

Q: Should one use application classes in define semantics of a submodel?
A: No, see the cited document (https://eclass.eu/fileadmin/Redaktion/pdf-Dateien/Broschueren/2021-06-29_Whitepaper_PlattformI40-ECLASS.pdf). New defined elements in ECLASS to define Submodels should be used once available.
The only exception is to use application/classification classes in submodels is to denote the category of a product (see ProductClassId property example within the TechnicalData Submodel template).

question will be re-formulated based on the discussion on 1.8.


style: Home page pop-ups removed

Actual Behaviour

Expected Behaviour

easy to reuse in the future, pull out as a component stand alone
Remove the style from the current home page

Steps to reproduce it

Screenshots of the issue

Would you like to work on the issue?

Hi, @Baoyuantop Do you have time to solve the problem?

Hi, @Baoyuantop Do you have time to solve the problem?

sure ～


Add count votes function

Click on the link to the issue where the voting markdown was used, and count votes easily! :)

I bet we could embed this in a custom GHA to generate the vote template and then post results. Maybe you did that already?

Nope, didn't do that. It's a good idea, but I don't think I can work on that in the near future.


⚠ Bofrost.es has degraded performance

In 1058682, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1458 ms

Resolved: Bofrost.es performance has improved in 34d62bd after 20 minutes.


UglifyJs: Unexpected token: punc (})

Description
When I try to uglify the build I'm getting this:
ERROR in bundle.js from UglifyJs
Unexpected token: punc (}) [./node_modules/@inlet/react-pixi/dist/react-pixi.development.es5.js:9765,19][bundle.js:59126,28]

That confusingly points to line:
      console.log({hostFiber});
It didnt happen before i started using @inlet/react-pixi
Steps to reproduce

clone the repo (https://github.com/smith-chris/poke/tree/d6d07a556accbef4962037789049a6cc592cb681) and follow the instructions (you probs just need to npm i
run npm run prod to see the error appear

Additional info
I'll just link specific tree to reproduce: https://github.com/smith-chris/poke/tree/d6d07a556accbef4962037789049a6cc592cb681
You can also see that commit before that used react-pixi-fiber and it does not have the same problem: https://github.com/smith-chris/poke/tree/aa73b342232c83acb15a70abdf3e6b2150da0abb
I hope that all the info you need to reproduce that issue. BTW I really like how painless it was to switch to this lib from react-pixi-fiber and I find yours ~30% faster. Good stuff!
Cheers,
Chris
EDIT:
I've updated all my dependencies and error still appears.
Now I think about it the problem is that you have object literal property value shorthand which is ES6. I guess my uglifyjs is setup in a way that it expects es5 and I could change that but giving that the name of the file is react-pixi.development.es5.js it should be fixed somewhere within react-pixi build setup.

Hi @smith-chris,
I accidentally pushed a modified reconciler to v0.1.3 😱
Just pushed a new version, please update react-pixi to 0.1.4 and try again.
Let me know if this solved it, thanks!

Tested, uglifying works now :) thanks @inlet
Still .es5 files are not es5 compatible. This is what es-check gives as a reason:
          · see the printed err.stack below for context```

If thats not intended then it's worth looking at pr I raised https://github.com/inlet/react-pixi/pull/36

Hi @smith-chris,
I think we need to redefine the output formats. Get rid of the es5 format as the umd versions is basically the same. Only having these:

cjs
umd
es (module)

What do you think?

Just tested and it seems like umd can be used as a script tag (for those 1% of people) so yeah, I think its a good idea :)


[local_auth][ANDROID] localizedReason should be optional

Use case
localizedReason  should be optional for Android. There could be use case where we only want to show a single piece of text on the biometric prompt. Currently showing 2 pieces of texts is mandatory
Proposal
Only set the description parameter to the BiometricPrompt if a non empty String is passed in the call.
code:
AuthenticationHelper.java
use
.setDescription(((String) call.argument("localizedReason")).isEmpty() ? null : (String) call.argument("localizedReason"))

instead of
.setDescription((String) call.argument("localizedReason"))

before:

after:

Very interested in this,
Am currently finding and modifying the BiometricPrompt.PromptInfo.Builder in the CD pipeline and thats very error-prone.

This would be really helpful. A lot of existing native applications don't use this, the reason can be implicitly explained to the user before showing the fingerprint / face ID popup anyway.


请问这个pow这个参数是啥

请问这个pow这个参数是啥，请问这个代码相对于nips原文里面的算法有什么改变

pow corresponds to the 2 in the denominator in Equation (2).
Usually pow=2 is good enough for most datasets.
We made pow as a tunable parameter for users who would like to obtain even better performance.


Fix CVE-2022-32149

Signed-off-by: Joseph Sheng<EMAIL_ADDRESS>
What this PR does / why we need it:
Special notes for your reviewer:
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

@JohnJAS please try to fix ci issue.

@yxxhero It looks like the circleci wants the AZURE_STORAGE_CONNECTION_STRING to deploy.
I've compared the config.yml with another successful build(https://app.circleci.com/pipelines/github/helm/helm/4368/workflows/6f607eb6-b32c-4bf6-b6ad-bfe641e6585f/jobs/15945), but I didn't find out the difference. Is there anything I've missed?
ci logs:
.circleci/deploy.sh: line 23: AZURE_STORAGE_CONNECTION_STRING: AZURE_STORAGE_CONNECTION_STRING environment variable is not se

@JohnJAS try to rebase your code.

@yxxhero Ahhh, I found circleci was run under my cloned helm repo. Now it is fixed and passed, please take a review.

@JohnJAS thanks very much. move to 3.10.3

Closing as a fix for this has already been merged. The dependency was updated in a separate PR that's been merged.
Thanks for the PR.


handle integration tests

currently the integration tests DO NOT cover all the data fetched from A+/course config file -> they should

auth token should be created in the DB to be persistent (that python script)


Hive ignores number of node configured for a cluster provision

When configuring a cluster, hive ignores the number of nodes and assign its own configuration (3 master nodes).

See https://github.com/openshift/installer/issues/545


Remove ILRepack

This commit removes ILRepack from the build. ILRepack is used in
the 6.x branch to internalize Json.NET into the Nest assembly, and
was being used in 7.x and master branch only to sign the versioned
canary packages. Since assembly-rewriter can now sign a rewritten
assembly (works cross-platform) in nullean/assembly-rewriter#7,
there is no need for ILRepack at all.

This PR is a dependency for tests passing in https://github.com/elastic/elasticsearch-net/pull/4679. Once this is merged, 4679 should pull this in


Buffersize too small

Hi,
thanks for providing this library. I had a problem were the buffersize in the ssc_device.py was too small. If a query with a long answer is performed, parts of the answer will be returned in the answer of further queries. This was the case for me with queries to /osc/schema. Changing the buffersize to 512 solved the problem.
    def send_ssc(self,
                 command: str,
                 interface: str = "%eth0",
                 buffersize: int = 512,
                 wait_time_seconds: float = .001,
                 port: int = 45):
        self.port = port

I have published a small tool with which I can query my Neumann KH 80 loudspeakers.
Thanks,
Thorsten

Hi Thorsten,
thank you for your feedback. The parameter 'buffersize' is a variable with a default value  of 64. You can set any other integer value.
I'll add an example script to illustrate the relation between buffersize, performance and maximum RX string length.
Would you allow me to add a slightly modified version of your khtool.py to the pyssc examples? I'd make sure to name you as the author in the file header of course.
Best regards
JJ

Hi JJ,
ok, that makes sense. I'm not very skilled with python. I just pushed a new version of the tool a changed buffersize setting.
Feel free to add a changed version.
Best regards,
Thorsten

Hi,
there's no need for a sleep command in the send_ssc function. The socket read is blocking and waits until the data is received, no matter how big the buffer is. So it's fine to have a big buffer byy default. Please try the code from below. It's just setting the socket timeout and removing the sleep. This should not have an impact on short answers of the speaker.
Best regards,
Thorsten
import socket
import time
import logging
from .Ssc_transaction import Ssc_transaction

class Ssc_device():
    """
    :param name: device name
    :param ip: ip address of ssc device

    .. code-block:: python
        :caption: todo

        >>> todo

    """
    def __init__(self,
                 name: str,
                 ip: str = None,
                 port: int = 45):
        self.name = name
        self.ip = ip
        self.socket = None
        self.port = port

    def connect(self, interface: str = "%eth0", port: int = 45):
        self.port = port
        self.socket = socket.create_connection((self.ip + interface, port))
        self.socket.setblocking(True)
        self.socket.settimeout(1)

    def disconnect(self):
        self.socket.close()

    def send_ssc(self,
                 command: str,
                 interface: str = "%eth0",
                 buffersize: int = 64,
                 wait_time_seconds: float = .001,
                 port: int = 45):
        self.port = port
        request_raw = f'{command}\r\n'.encode('utf-8')
        try:
            self.socket.sendto(request_raw, (self.ip + interface, self.port))
        except Exception as e:
            logging.warning('socket connection closed. Reopening. ' + str(e))
            self.connect(interface=interface, port=port)
            self.socket.sendto(request_raw, (self.ip + interface, self.port))
        data = self.socket.recv(buffersize)
        ssc_transaction = Ssc_transaction()
        ssc_transaction.TX = command
        ssc_transaction.RX = data.decode('utf-8')
        return ssc_transaction

Thanks Thorsten - that makes a lot of sense. I ran some experiments this morning just to be sure and it looks like I can indeed drop wait_time_seconds and increase the default buffersize. Looks like the buffersize has no direct effect on the execution time.
buffersize: 32768
Execution time: 38 milliseconds

buffersize: 32768
Execution time: 47 milliseconds

buffersize: 512
Execution time: 56 milliseconds

buffersize: 512
Execution time: 42 milliseconds

I'll test and push the changes in the next days. You are also welcome to create a pull request if you feel like it.

Hi JJ,
thanks for implemting the pull request.
I wrote a test Python app for the macOS menu bar that uses the pyssc library to dim the speakers. For this i changed some socket calls in the library to catch exceptions and control the reconnect better. Now the app doesn't freeze as often and for a long time when a speaker is unreachable.
I still need to clean up the code a bit and test it further. If you are interested, I can send you the changes next week for review. I think a pull request is not the right way to discuss the changes.
Cheers,
Thorsten

Sounds great - I'm very interested! You can PR or send me the changes. Looking forward to it.

@schwinn - this might be interesting to you: https://github.com/jj-wohlgemuth/pyssc-level-controller


[feature request] Add a field in UI indicating when a container was created/launched

It would be useful to have some indication in the UI of times that containers were upgraded/updated for deployments, etc.  A field in the stack/service view for each container as well as in the container view might make sense?

Any update on this?  It look a like there are time stamps in the API, would it be feasible to expose them in the UI?

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


Add officer detail page

Ideally, we should have a secondary page that the officer lookup links to which pulls up all available info we have on that particular officer: photos, complaints, OPA violations, etc.
This information should not be part of the normal lookup, since we don't want to slow the primary search down.

@AetherUnbound I'll work on this 😁

Oops - hold off for now! We have another service we'll be using for this actually. Thanks so much though! I'll actually close this issue.


use travis to upload the extension

http://stackoverflow.com/questions/33070611/how-to-continuously-deliver-app-updates-to-the-chrome-store-travis-ci

obsolete, extension functionality is part of native papertrail


[bn] localize content/en/docs/concepts/cluster administration/ index.md

content/en/docs/concepts/cluster administration/ index.md in Bangla
Fixes Issue : #44993
Related PR : #45009
Tracking Issue: #31677

/lgtm


[DEVOPS-2931] Upgrade to Node 18

:ticket: JIRA ticket: DEVOPS-2931 :ticket:

@dairyisscary

what the heck? why is node_modules checked in? is that a requirement of github actions?
thoughts on removing this directory from the repo?

Github actions requires these to be available somewhere.
The original instructions from GitHub whence this originated suggests this approach.
They've since advocated other options, but still, even those include checking in build artifacts.
I decided to stick with a light touch here, and just keep checking in the node_modules.


Fix: Add type support for Swiper/React ref object

Problem:
When using Swiper/React with Typescript, and when one attempts to assign a ref to the Swiper component, TS gives an error similar to this:
Type '{ children: Element[]; ref: RefObject<any>; className: string; }' is not assignable to type 'IntrinsicAttributes & SwiperProps & { children?: ReactNode; }'. Property 'ref' does not exist on type 'IntrinsicAttributes & SwiperProps & { children?: ReactNode; }'

This was also discussed in this issue: https://github.com/nolimits4web/swiper/issues/5241
Solution:
This happens because, although the Swiper component uses forwardRef, it is not defined in the .d.ts file.
The proposal is to create an interface that represents the element injected in the ref, such as
interface SwiperRef extends React.HTMLAttributes<HTMLElement> {
  swiper: SwiperClass;
}

And use it in the Swiper declare with React.RefAttributes, this way:
declare const Swiper: React.FunctionComponent<React.RefAttributes<SwiperRef> & SwiperProps>;

Usage:
With this solution in place, devs could then use the SwiperRef in such a way:
import React, { useRef } from 'react';
import { Swiper, SwiperRef, SwiperSlide } from 'swiper/react';

() => {
 const swiperRef = useRef<SwiperRef>(null);

 const swiperApi = swiperRef.current?.swiper;

  return (
    <Swiper ref={swiperRef}>
      <SwiperSlide>Hello1</SwiperSlide>
      <SwiperSlide>Hello2</SwiperSlide>
    </Swiper>
  );
};

This is wrong as Swiper ref is a HTML Element, not an instance of Swiper

Yes, missed that. Merged, thanks!

Hi, I cannot find SwiperRef in swiper-react.d.ts. Should I install it? thanks

Probably because it's still not released @ibrvimv
Thank you @dasilvaluis for this PR. Do you know when is it expected to be released?

Probably because it's still not released @ibrvimv
Thank you @dasilvaluis for this PR. Do you know when is it expected to be released?

I don't know man, I'm just some other guy

swiperRef doesn't exist

SwiperRef doesn't exist yet

Is there a solution on this case? I tried almost all the solutions but non of them worked. I'm working on create-react-app --template typescript. When I was working without --template typescript there was no problem like this.

@yusufcode hard to help without seeing your code.

@dasilvaluis hey man, thank you for return. Actually, there is no difference between typescripted and not typescripted codes. These are my codes:
<Swiper
        spaceBetween={24} // <- problem accours here
      >
        {
          swiperItems.map((swiperItem,i)=>
            <SwiperSlide key={i}>
              <Box display="flex" flexDir="column" bg="#fff" border="1px solid #E0D8D7" borderBottom="4px solid #ED1C24" borderRadius="2px">
                <Text>{swiperItem.title}</Text>
                <Text>{swiperItem.description}</Text>
                <Box>
                  <Button variant="second">Learn More</Button>
                  <Button variant="first">Find A Dealer</Button>
                </Box>
              </Box>
            </SwiperSlide>
          )
        }
      </Swiper>

By the way I SOLVED the problem. I downgraded the swiper from v10.0.4 to v9.4.1 and type problems gone. higher versions from v9.4.1  has these type problems. So I advise you guys to use lower versions of swiper.


Support virtual fields for insert/update

I will start researching this issue to see what work is needed to add this support to the library.

Awesome, thank you!

Can you provide me with an example of what you are trying to do?


user MergingMediaSource but Legacy decoding is disabled

Version
Media3 main branch
More version details
No response
Devices that reproduce the issue
piexel6
Devices that do not reproduce the issue
No response
Reproducible in the demo app?
Not tested
Reproduction steps

use HlsMediaSource ,and use MediaItem.Builder().setSubtitleConfigurations.
just player.setMediaSource(mergedSource). it not subtitle. but DefaultFactory.create, ok.
<IP_ADDRESS> ，I use SingleSampleMediaSource to other load my subtitle.but error 1004 and have this.
Caused by: java.lang.IllegalStateException: Legacy decoding is disabled, can't handle application/x-subrip samples (expected application/x-media3-cues).
3.I must user hlsSource , so what i can do?

Expected result
error 1004
Actual result
Caused by: java.lang.IllegalStateException: Legacy decoding is disabled, can't handle application/x-subrip samples (expected application/x-media3-cues)
Media
xxx
Bug Report

[ ] You will email the zip file produced by adb bugreport to<EMAIL_ADDRESS>after filing this issue.

The problem is not MergingMediaSource, it's SingleSampleMediaSource. You either need to switch to DefaultMediaSourceFactory (which will do the merging of subtitles for you and supports HLS), or instantiate your own ProgressiveMediaSource for subtitles like we do in DefaultMediaSourceFactory - see https://github.com/androidx/media/issues/1569#issuecomment-2255506625 for more info.


What's the project goal?

Hi,I'm interested in hacking pocketgo too,mainly for improving displaying.

This is a fix for a speaker noise issue: https://byteporter.com/pocket-go-pwm-script-install/


Wrong documentation about the gradle-parameter servletContainer

Documentation: http://akhikhl.github.io/gretty-doc/Gretty-configuration.html
Sourcecode: https://github.com/akhikhl/gretty/blob/master/libs/gretty/src/main/groovy/org/akhikhl/gretty/ServletContainerConfig.groovy
I was trying to use jetty 9.4
According to documentation, i have to use the following setting in build.gradle: servletContainer = "jetty94"
But the correct setting would be: servletContainer = "jetty9.4"
The difference is the dot between the first and second number.

This was fixed in https://github.com/akhikhl/gretty-doc/commit/dd3b0f8fe53af1cf122f57bdfb78b9cd5622b42c but the site is not republished.
In the fork this is published at https://gretty-gradle-plugin.github.io/gretty-doc/Gretty-configuration.html .


IL2CPP support for Standalone

Describe the solution you'd like
I would like to be able to build my project with IL2CPP
And you could also add an option to delete the _BackUpThisFolder_ButDontShipItWithYourGame folder.
Describe alternatives you've considered
I went back to a Mono build
Additional context
With / Without IL2CPP

This will need to be supported in the docker images. Here's a related issue:
https://gitlab.com/gableroux/unity3d/-/issues/66

@knxrb With they, you mean the person you're talking to above? 😄
Anyway, the linked issue seems to be clear and the last comment indicates it's working.
I guess the next step is for Unity Builder to allow for a parameter that allows selecting il2cpp images. Probably the image functionality has to be extended for this. Anyone is free to send a PR with the required changes.

Oh yeah >.< I didn't notice that it's the same guy for the images. I've only just started looking at and using these actions this afternoon.
I'll have a look at what's needed for this tomorrow and see if I can get it working.
Thanks for the info :+1:

@webbertakken I've had a quick look at this and from what I understand: this can't be done yet as there's only linux IL2CPP images?
The image suffix is being set to windows when building StandaloneLinux64:
https://github.com/webbertakken/unity-builder/blob/master/src/model/image-tag.js#L54

Linux support for IL2CPP should work out of the box, same for Android and iOS builds, whereas macOs and Windows will require these platforms to be used in the base image. This is currently not planned but a welcome addition to the repo that holds the docker images here.


Browse Google Music

Hello,
first - thanks for this frontend, brilliant work.
I wanted to know whether it is possible to include a browse feature for your library (including Google Music). Apparently, mopidy and the gmusic extension provide that functionality now.
Thanks
Chris

Browsing was added with PR #21


Cannot retrieve plugin for Play 2.3.8

Following instructions from https://github.com/play2war/play2-war-plugin/wiki/Configuration, update of my Play 2.3.8 project fails.
I added following line to project/plugins.sbt
addSbtPlugin("com.github.play2war" % "play2-war-plugin" % "1.3-beta3")

But activator update returns the following error:
[info] Resolving jline#jline;2.11 ...
[warn]  ::::::::::::::::::::::::::::::::::::::::::::::
[warn]  ::          UNRESOLVED DEPENDENCIES         ::
[warn]  ::::::::::::::::::::::::::::::::::::::::::::::
[warn]  :: com.github.play2war#play2-war-plugin;1.2: not found
[warn]  ::::::::::::::::::::::::::::::::::::::::::::::
[warn]
[warn]  Note: Some unresolved dependencies have extra attributes.  Check that these dependencies exist with the requested attributes.
[warn]          com.github.play2war:play2-war-plugin:1.2 (scalaVersion=2.11, sbtVersion=0.13)
[warn]
[trace] Stack trace suppressed: run last *:update for the full output.
[error] (*:update) sbt.ResolveException: unresolved dependency: com.github.play2war#play2-war-plugin;1.2: not found
[error] Total time: 21 s, completed 20 mai 2015 10:50:27

I guess expected version is not available on Maven Central Repo. How can I recover from this issue.
Thanks.

addSbtPlugin("com.github.play2war" % "play2-war-plugin" % "1.3-beta3")
^^^^^^^^^^^^^^
[warn]  :: com.github.play2war#play2-war-plugin;1.2: not found
^^^^
Both versions don't match. Something wrong in your config files ?

maybe a reload in sbt is necessary to reload the plugin configuration?

Well done.
The reload command fixes the issue. I guess I had a wrong configuration as I first set the 1.2 version of the plugin, before to see that 1.3-beta3 is necessary for Play 2.3.x.
@dlecan @yanns many thanks for your support.


remove _rev property from docs & fix doc mutation

closes #2 and makes some improvements to the readme. this would require a major version bump.

I tend to oppose this commit for a few reasons:

checking the _rev of the existing document is valid for a lot of use cases. e.g. here's how I use it in pouchdb-find - I want to do a particular action if the document exists vs. if it doesn't.
I could fix the above by making the diffFun pass in two options (e.g. diffFun(doc, docRev)), but arguably that's much more confusing, because users probably expect the _rev to be on the doc itself
You didn't add any tests ;)

I totally understand where you're coming from with wanting to do your own deep comparison and not have to worry about _rev, but I think the structured cloning fix along with delete doc._rev inside of your diffFun is a fine solution to that. So I'd like to merge your cloning fixes and README fixes, but not the other stuff. I'll propose a commit on top of yours this morning.

For doing a particular action if the document exists vs. if it doesn't, couldn't you just check if doc._id exists?

Yes, but to me, it's clearer to check the _rev, since that's what manages the revision history.

Alright, I added a test to demonstrate the 2nd kind of doc mutation that the _.clone is there to eliminate.
As for the removal of _rev, I find it clearer to check for the existence of _id because I'm asking if the doc exists, not if the doc has history... Of course, that's just semantics and either way would be understandable. Also, I might just be used to _id from my years using mongo.
But, if we got rid of _rev then it would be easier to compare new docs with old docs using _.isEqual, which is something I find myself doing quite often.

That's a good point, however I think your ''should not mutate local vars' test can be fixed without omitting the _revs. I understand that you want to do deep comparisons, but I'd like to optimize for performance and clarity here (ideally you shouldn't even need to deep comparison/cloning), and I just can't imagine that users will find it intuitive that the input doc is not identical to the one they would get from get(). It also removes the possibility that users can write functions that, for instance, react differently for first-gen docs 1- vs second-gen docs 2-.
I understand where you're coming from, but I think I'm going to be that jerk and close this pull request in favor of the other solution. I encourage you to submit a pull request to fix the 'should not mutate local vars' test, or if you don't mind I can simply borrow your test and write my own fix (happy to give you credit in the commit log, of course; I can change the author to you).
Thanks for your input!

Foillow-up: https://github.com/pouchdb/upsert/issues/7

Yeah, feel free to use the test case (as well as any other code you want).
And yeah clone can be used without omitting _rev, in fact that test should still work in exactly the same way if _rev is included in doc since it only checks doc.value !== newDoc.value, and so long as newDoc isn't mutated the test will pass.

As for changing behaviour based on the _rev - is the format of _rev actually standardized / guaranteed?

@slang800 Yep, all CouchDB implementations have the format <integer>-<hash>. So it's guaranteed that 1-x refers to the 1st generation, 2-y to the second generation, through 10-z and 100-w etc. We actually use it a lot in the PouchDB code to optimize for special cases with first-gen docs (e.g. during replication and map/reduce, where we know there's zero history, and so we can take shortcuts).
The only part that's quasi-standard is what comes after the hyphen. PouchDB just generates a random number, whereas CouchDB calculates a checksum of the entire document in order to dedup duplicate changes across multiple nodes. There's very little harm in us implementing it the way we do, and in fact it would be impossible, since the checksum implementation requires Erlang strings. But that's a whole other discussion; there's an open issue somewhere in CouchDB to make the revs truly deterministic. :)


Update gretel-demo-conditional-generation.ipynb

update collab

Are we planning to update this to use the new interface?

Are we planning to update this to use the new interface?

yes, good call.


Add Maria Shaldybin as Networking approver

I'd like to be added as an approver in the Diego working group.
@ameowlia would you be willing to nominate me?
Submitted substantial PRs:

https://github.com/cloudfoundry/routing-release/pull/275
https://github.com/cloudfoundry/silk-release/pull/61
https://github.com/cloudfoundry/silk-release/pull/54
https://github.com/cloudfoundry/silk-release/pull/53
https://github.com/cloudfoundry/cf-networking-release/pull/123
https://github.com/cloudfoundry/cf-networking-release/pull/117
https://github.com/cloudfoundry/cf-networking-release/pull/113
https://github.com/cloudfoundry/silk-release/pull/40
https://github.com/cloudfoundry/envoy-nginx-release/pull/4
https://github.com/cloudfoundry/haproxy-boshrelease/pull/285

Involved in technical discussions:

Dynamic ASGs
C2C over TLS

Authored Cloud Foundry blog post: Securing Container-to-Container Networking with TLS

👍 I approve Maria's nomination.
Besides the things she listed she was also one of the two architects behind the dynamic ASG feature and she was the sole architect for TLS for c2c. Both of these large features spanned  cf-networking, silk, and diego.


Misleading info in README

You can register a type implementing IFactory as a Factory Registration. This will automatically register it as both IFactory and T.

[FactoryRegistration] has been renamed to [RegisterFactory]. Is Factory Registration naming outdated here?

[RegisterFactory] doesn't register factory, only T. Is it bug in code or outdated info in docs?:

[RegisterFactory(typeof(Factory))]
public partial class Container : IContainer<IFactory<int>> { }

public class Factory : IFactory<int> { public int Create() => 0; }

new Container().Run(_ => { });

// leads to error:
// [SI0102] Error while resolving dependencies for 'StrongInject.IFactory<int>': We have no source for instance of type 'StrongInject.IFactory<int>'
//
// if we add [Register(typeof(Factory), typeof(IFactory<string>))] it works fine, but there is a warning now:
// [SI1001] 'Factory' implements 'StrongInject.IFactory<string>'. Did you mean to use FactoryRegistration instead?

FactoryRegistration is outdated in warning too.

In readme:

[RegisterFactory(typeof(InterfaceArrayFactory), scope: Scope.SingleInstance, factoryTargetScope: Scope.InstancePerResolution, typeof(IFactory<IInterface[]>))]

scope parameter should be factoryScope instead
and [RegisterFactory] doesn't have parameter Type registerAs.

Optional Parameters
If a parameter to a type or method is optional, StrongInject will not error if it cannot be resolved, and will instead just use the default value.
An example of where this can be useful is for providing a default instance of an interface if none is registered:
public class DefaultImplementation : IInterface {}
public interface IInterface {}

[Register(typeof(DefaultImplementation))]
public class Module
{
  [Decorator] GetIInterface(IInterface? impl = null, DefaultImplementation defaultImpl) => impl ?? defaultImpl;
}

Not working for me:
[Register<Service>]
[Register(typeof(DefaultImplementation))]
public partial class Container : IContainer<Service>
{
    [DecoratorFactory] IInterface GetIInterface(DefaultImplementation defaultImpl, IInterface? impl = null) => impl ?? defaultImpl;
}

public class DefaultImplementation : IInterface { }

public record Service(IInterface _);

//  [SI0102] Error while resolving dependencies for 'Service': We have no source for instance of type 'IInterface'

Thanks for all the issues. I will look into them when I get a chance.


Feature/influxdb format

Hi,
this adds an ingester for the InfluxDB format.

Coverage remained the same at 100.0% when pulling 56865d57c4961ac5b9177ba33fd7bcdd2803e642 on phxql:feature/influxdb-format into 740812cbc55d6da38ad644c2299b73e9a513745e on ChronixDB:master.

Coverage remained the same at 100.0% when pulling 56865d57c4961ac5b9177ba33fd7bcdd2803e642 on phxql:feature/influxdb-format into 740812cbc55d6da38ad644c2299b73e9a513745e on ChronixDB:master.

Coverage remained the same at 100.0% when pulling 56865d57c4961ac5b9177ba33fd7bcdd2803e642 on phxql:feature/influxdb-format into 740812cbc55d6da38ad644c2299b73e9a513745e on ChronixDB:master.

Coverage remained the same at 100.0% when pulling 56865d57c4961ac5b9177ba33fd7bcdd2803e642 on phxql:feature/influxdb-format into 740812cbc55d6da38ad644c2299b73e9a513745e on ChronixDB:master.


allow datafusion-cli to take file param

Which issue does this PR close?
We generally require a GitHub issue to be filed for all bug fixes and enhancements and this helps us generate change logs for our releases. You can link an issue to this PR using the GitHub syntax. For example Closes #123 indicates that this PR will close issue #123.
Closes #282.
Relies on #283
Rationale for this change
Why are you proposing this change? If this is already explained clearly in the issue then this section is not needed.
Explaining clearly why changes are proposed helps reviewers understand your changes and offer better suggestions for fixes.
Allow datafusion-cli to take file as a param.
What changes are included in this PR?
There is no need to duplicate the description in the issue here but it is sometimes worth providing a summary of the individual changes in this PR.
Are there any user-facing changes?
If there are user-facing changes then we may require documentation to be updated before approving the PR.
If there are any breaking changes to public APIs, please add the breaking change label.

Codecov Report

Merging #285 (24ba9a0) into master (9dface6) will decrease coverage by 0.11%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #285      +/-   ##
==========================================
- Coverage   76.19%   76.07%   -0.12%
==========================================
  Files         140      140
  Lines       23595    23631      +36
==========================================
  Hits        17978    17978
- Misses       5617     5653      +36

Impacted Files
Coverage Δ

datafusion-cli/src/main.rs
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

need to fix https://github.com/apache/arrow-datafusion/pull/286 first

need to fix https://github.com/apache/arrow-datafusion/pull/286 first


Updated documentation for snippted istio_installation_minimal and for pilot

Hello,
I added the key ior to the snippet istio_installation_minimal.adoc, because the key is in the default ServiceMeshControlPlane template. Furthermore I moved the table for pilot->request to pilot and added a key jwksResolverExtraRootCA to this table.
Olaf

/ok-to-test

/cherrypick maistra-1.1


vscodevim plugin breaks ionide

Problem
Ionide doesn't work when the vscodevim plugin is installed
How to reproduce

Install vs code
Install the ionide plugin
Open an fsharp file. E.g. the ionide hover-over works, and shows type signatures and documentation.
Install the vscodevim plugin
Now ionide is broken. It might work for a few seconds at most, then it will crash and never properly resume again.

Version info
VS Code version: 1.18.1
vscodevim plugin version: 0.10.6
OS: Windows 10 (version 1511, OS Build 10586.1295)

Note: My settings.json file is empty

Cannot repro.
can you please attach some logs?
info about how enable it in https://github.com/ionide/ionide-vscode-fsharp/blob/master/README.md#how-to-get-logs-for-debugging--issue-reporting
The Help -> Toggle Developer Tools -> console tab log will be nice too.


feat(logger): add logger so all console.log messages can be muted

We want to have a predictable output to the user. So all output from
used plugins is muted. By using the logger, only our messages get
rendered

Suggestions:

make logger a constructor new Logger({ level: 'silent' })
make logLevel a config setting, which defaults to silent
make bin/fastatic run fastatic({ logLevel: 'verbose' })
next step could be make logLevel a CLI option fastatic --log-level silent


create nuget packages for Jasiri & Jasiri.OpenTracing

Split current Jasiri nuget package into Jasiri and Jasiri.OpenTracing packages.

Done


Question about the syntax

https://github.com/willtebbutt/BayesianLinearRegressors.jl/blob/c7c782d9e20bda92c095ffcbcf531d8f9291c069/src/bayesian_linear_regression.jl#L29-L31
I've tried finding the syntax in the Julia docs but I can't find it and don't know the name. Would you mind explaining what's happening on these lines?

From a Julia perspective, this is "calling" the BayesianLinearRegressor object.
From a BayesianLinearRegressors perspective, you should interpret this as specifying the inputs X at which you're going to observe blr, and the covariance Σy  of the observation noise under which they will be observed -- if you provide a real value σ² it is assumed that the covariance is σ² I.
So let f be some BayesianLinearRegressor, then you can write
# Specify locations to make observations
fx = f(x, σ²)

# sample from the prior:
y = rand(fx)

# compute the log marginal probability of `y` under the prior:
logpdf(fx, y)

Does this answer your question?

Thanks so much for clarifying . I understand the Bayesian perspective but not the Julia syntax very well.
I assume you're function overloading on lines 29-31?

Just read the link. makes total sense now, appreciate it.


Message not showing shareViaFacebookWithPasteMessageHint.

window.plugins.socialsharing.shareViaFacebookWithPasteMessageHint(
'Hi, Checkout this app.IHere is the link to download the apps Google Play Store Image and Apple Play Store Image',
null /* img /,
null / url */,
function() {console.log('share ok')},
function(errormsg){alert(errormsg)}
)"
Am not able to see the message in the facebook popup.. Also can we add any variable which returns any value in the message.

Have you found a solution ?


Move NumberFormat to thread-local variable

The NumberFormat class isn't thread safe (StackOverflow, Docs), and sharing one instance across threads allowed for races which resulted in otherwise valid OSM tags not being parsed correctly.
This surfaced as some rather strange exceptions during Planetiler runs that I wasn't able to reproduce when running each input individually. For example, both of these exceptions were not reproducible in unit tests:
java.lang.NumberFormatException: For input string: ""
java.lang.NumberFormatException: For input string: "1."

Given that it's a race condition, I'm not sure how to write a proper test for this, but this change reliably fixes the issue for me during local runs, and the existing tests for Parse should at least confirm that there aren't regressions.

SonarCloud Quality Gate failed.
  1 Bug

From my reading of the issue, I believe this warning can be safely suppressed, but would like a second opinion. Parse is a static utility class that has no obvious lifecycle, and as far as I can tell we'd want the NumberFormat instance available for the duration of the thread.

Yikes! Good catch.  I'm a bit concerned about other locations that do this as well.  It looks like both Format and Parse classes use number format instances. Could you update those as well? Looking at a recent run on the planet there's only a small amount of time spent in any of those, so the overhead of getting the thread local variable should be fine.

SonarCloud Quality Gate failed.
  1 Bug

From my reading of the issue, I believe this warning can be safely suppressed, but would like a second opinion. Parse is a static utility class that has no obvious lifecycle, and as far as I can tell we'd want the NumberFormat instance available for the duration of the thread.

I think that's fine, can you put @SuppressWarnings("java:S5164") on that thread local definition with a short comment explaining that?


Correct debug output

I'm not sure I understand the reason for this change. The old link would take you to the admin page (which is useful if you want to understand or debug the chart), but the new link will just show the JSON used.

Before
After

If you want to see the JSON isn't the "Network" tab in Developer Tools a better option?

I'm not sure I understand the reason for this change. The old link would take you to the admin page (which is useful if you want to understand or debug the chart), but the new link will just show the JSON used.

I thought the debug output was just meant to log the URL that is subsequently gonna be fetched from the ETEngine server. But that would be a different domain anyways... which I did not notice because I am running everything locally... so my mistake. I will close this. And sorry for bothering you :slightly_smiling_face:

If you want to see the JSON isn't the "Network" tab in Developer Tools a better option?

Yes.


Render queue not preserved

One of the materials on my avatar's render queue is not preserved when generating the optimised shaders and materials. I have it set to a render queue of 1600, however upon optimising, the optimised variant's render queue option is reset to 2000.

Any more info on how this happened? Maybe some special shaders or shader inspectors?

I'll have a look and see if I can notice any patterns regarding this.

I've just come across this issue myself, so I hope I can add a bit more info.
Using Silent's shader, I noticed on a certain opaque material, default render queue, and with the bake button activated (pre-baked?), I would notice the material being semi-transparent in certain conditions. Shown in the image below, and 2nd image is what it should look like instead.
I've noticed the issue fixes itself when I do any of the following, independently of each other:

manually setting the render queue very high
undoing the "bake" option
enabling GPU instancing (?)

My optimizer settings for this are only:

Optimize on Upload
Merge Different Property Materials (with both sub-options selected)
Although the material in question doesn't show as merging with any other material.

The world is "White tile" by amanek: https://vrchat.com/home/world/wrld_1544f2bc-e6db-4810-b450-483376745f7a

I've included the broken shader of the post above.
shaders.zip


Unify Dockerfiles

There should be one and only one Dockerfile on this project.  I've now lost a couple of hours twice over this.

Fixed in 982c4f04e4ac719587a82119bf5fad30c744be4a


overlay2 driver: randomly ineffective write operations

Problem description

I start several (like 15) similar (almost identical) containers in a raw, from python script (docker module)
run's command arg invokes python scripts inside the containers as well, before starting monit as process supervisor
these scripts make modification on some config files by merging json files, overwriting one of them
the "overwritten" files are left intact in some (random!) of the almost identical containers (best case: one, worst case: 8) -  although script finished with no error.
running the same command by hand does the json merge fine

it only happens using overlay2 driver. btrfs does the job well. tried with xfs, ext4, and btrfs under overlay2, all faulty. also tried several OSs, like

centos8: xfs - problem arose here since no btrfs driver anymore - and ext4 - errors
centos7: btrfs, no problem
alpine: ext4 - errors, btrfs/onerlay2 (just moved the docker dir to btrfs) - errors, btrfs/btrfs (reinstalled docker)- fine

I have tried to play with timing and insert sync calls after writes, with no visible effect

Thanks for sharing the issue. If the issue is with the driver and not the docs, please create an issue at https://github.com/moby/moby
If you need support, you can try the Docker Community Slack at https://dockercommunity.slack.com/


18DS

This was the only example I found that didn't match the other titles. So the db has them all matching, but the proof on this one specific line has it different. See related #1250 #1255 #1256 and commit https://github.com/ShabadOS/database/pull/1253/commits/5bb7ff8b1b285a4d477e8eaf0aef20e4e22086ca

Key
Value

View
https://database.shabados.com/sources/7/page/10/line/2

Source
Ganj Nama

Page
10

ID
18DS

Line
vwihgurU jIau hwzr nwzr hY [

Keep the jIE in this line as well.


Automated export?

As I understand it this repo is about exporting translation pairs from tldr-pages to OPUS for a high-quality translation pairs dataset. Given that opus claims the last export was ~August 2023, is it possible to automate the export via e.g. GitHub actions etc?
Then a) we don't have to worry about it, and b) opus get a nice updated dataset regularly.

Hi, @sbrl we already provide exported datasets under the latest release that is automatically updated every month through GitHub actions (i.e. https://github.com/tldr-pages/tldr-translation-pairs-gen/releases/latest).
Additionally, I also publish the CSV dataset officially under our org in Kaggle at https://www.kaggle.com/datasets/tldr-pages/tldr-pages-translation-pairs-dataset.
Regarding the OPUS Corpus, they seem to update the dataset based on releases made to the repo here, so IG I will create quarterly releases so that the dataset is up to date upstream. (Will do one now)

Sounds good!
Yeah, I did see the kaggle dataset there, but hadn't explored it yet.


Signal Graphs

Having some sort of graph that can record data points of signal on every refresh would be helpful.
Right now, there don't seem to be any Compose Multiplatform graphing libraries that target Android, JVM, and iOS, so this isn't currently possible without trying to make an intermediate adapter for multiple libraries.

How about logging to csv - especially in the desktop environments.  We could create our own from there.


Redux setup

In this Branch I have done the followings:

Wrapped the app with redux.
Added the store to the application.
Created a slice for the application.

You have done a great job @Mhamad-Raad All what is left is to merge it

[ ] Descriptive PR

[ ] No linter Errors


Tag v2.3.1 as latest

Better, don't use the latest flag in Github :)

Thanks for reporting! We forgot to publish the release on github last time. Done now and added to the release process instructions for next releases.
https://github.com/apache/pulsar/releases/tag/v2.3.1


Build Docker image using Ploutos

This PR configures Ploutos to build an linux/amd64 Docker image and sanity check it by invoking the krill-sync --version command.
The primary reason for adding this was to exercise/validate the Dockerfile as it was recently discovered not to be working with the version of Rust contained in the outdated Alpine base image it was using but this went unnoticed due to the build process not building the Docker image.
This PR does NOT publish the produced image to Docker Hub. That can be configured if wanted.

No, Ploutous can handle that, but needs secrets adding to the repo.

Thanks!
So, if we want to publish the image on release, then that should be a separate triggered action no? We can do it, but I think it's not too urgent.

So, the PR has been updated to refer to Docker Hub credentials and those credentials have been created in Docker Hub and stored in GitHub as Variable DOCKER_HUB_ID and Secret DOCKER_HUB_TOKEN.
However, I had forgotten that Ploutos doesn't give you control over when it publishes to Docker Hub, rather it will publish images for builds on main or on release tags vN but not for any other branches or tags.
Is this okay @timbru?

@timbru: Should the README be updated to say how to run with Docker?


Race Track Enclosure

As a player, I would like to be constrained to drive the car mainly in the space of the racetrack so that I am not able to cheat.
Acceptance criteria:

When the car reaches the limit of the track and the beginning of another surface, the game should reduce the car speed by at least two thirds of its previous speed.
The game strategy is preserved and inforced.
The game environment can run smoothly in the emulator and app with no notable lags longer than 0.0001 seconds.

At the moment navigation is made difficult because of the enclosure because the enclosure is a mere wall. The next step is to improve the enclosure to only guide the car back to the track and not stop it completely when it crosses to the grass area. Related to #28

This issue is not critical for now because it requires a lot of effort and doesn't provide much value to the project. Will reopen it if there is time later on.


fix: requiement -> requirements typo in add.md

Just fixing a small typo with the text: requiement -> requirement

Thanks!


Generalizing Toolbar items for Repository and Organization view

This PR takes care of generalizing toolbar components of the Repository list and Organization list pages. This includes fixing filtering, select checkbox and generalizing the remaining tool bar components into separate components.

Looking good. Can you add some comments to the newer functions and wherever else you think they're warranted?
I spun this up locally and the happy paths work. A couple things that could use some more attention:

Select All checkbox doesn't work, only the dropdown. Can this be updated so that when the checkbox is clicked it selects/deselects the items on the current page?
The Select All checkbox state persists when changing paths between /organizations and /repositories on the navbar. Any filters or selections should change on path change.


AR 7.1 update affected test for AbstractAdapter#transform_query

A recently-merged update to ActiveRecord now causes the adapter context to be brought in when query transformers are being called.  Here is the relevant method in AbstractAdapter, note the addition of the parameter self:
def transform_query(sql)
  ActiveRecord.query_transformers.each do |transformer|
    sql = transformer.call(sql, self)
  end
  sql
end

It's part of this commit:
https://github.com/rails/rails/commit/d3b41750fb308662a9871139e6cd39849b0c03d8

Thank you!


fix: pip install

REQUIRED: Add a summary of your PR here, typically including why the change is needed and what was changed. Include any design alternatives for discussion purposes.

Attempt fix for issue 2108

REQUIRED: Fill out the below checklists or remove if irrelevant

If you are opening a PR for Official Notebooks under the notebooks/official folder, follow this mandatory checklist:

[ ] Use the notebook template as a starting point.
[ ] Follow the style and grammar rules outlined in the above notebook template.
[ ] Verify the notebook runs successfully in Colab since the automated tests cannot guarantee this even when it passes.
[ ] Passes all the required automated checks. You can locally test for formatting and linting with these instructions.
[ ] You have consulted with a tech writer to see if tech writer review is necessary. If so, the notebook has been reviewed by a tech writer, and they have approved it.
[ ] This notebook has been added to the CODEOWNERS file under the Official Notebooks section, pointing to the author or the author's team.
[ ] The Jupyter notebook cleans up any artifacts it has created (datasets, ML models, endpoints, etc) so as not to eat up unnecessary resources.

If you are opening a PR for Community Notebooks under the notebooks/community folder:

[ ] This notebook has been added to the CODEOWNERS file under the Community Notebooks section, pointing to the author or the author's team.
[ ] Passes all the required formatting and linting checks. You can locally test with these instructions.

If you are opening a PR for Community Content under the community-content folder:

[ ] Make sure your main Content Directory Name is descriptive, informative, and includes some of the key products and attributes of your content, so that it is differentiable from other content
[ ] The main content directory has been added to the CODEOWNERS file under the Community Content section, pointing to the author or the author's team.
[ ] Passes all the required formatting and linting checks. You can locally test with these instructions.

I think -q -U are just alias for --quiet --upgrade so this wouldn't change anything AFAIK.


Test: Alleviate the WaitForPods constrains about terminated pods.

This commits will allow pods that are in terminated state to clean
meanwhile some pods are installed. The main reason to allow this
behaviour is that nowadays we use ExpectAllPodsTerminated in the
AfterAll functions and this behaviour is not longer normal.
This change will return false for WaitForPods, but will not block the
test at all.
Signed-off-by: Eloy Coto<EMAIL_ADDRESS>

This change is

test-me-please

Coverage increased (+0.004%) to 42.685% when pulling 2f4cfaa9177bbf713b6baff74d06ccd47e15ee31 on eloycoto:TerminatedPods into 9a76fe52cb68189de03302cfd7590f757d2d596f on cilium:master.

Build failure: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/11134/

test-me-please

test-me-please

test-me-please


Maximum height of widgets docked in either the 'top' or 'bottom' areas is not adequate

🐛 Bug
The width of widgets docked in either the 'left' or 'right' areas with viewer.window.add_dock_widget is very flexible (provided that the widgets themselves are flexible, of course). This is not the case with widgets docked in the 'top' or 'bottom' areas: the maximum obtainable height is not sufficient for auxiliary visuals, only text controls (e.g. the built-in console).
To Reproduce
Steps to reproduce the behavior:
The tight constraints to which I am referring can easily be seen by trying to resize the console widget in its default position. In fact, on my desktop (Windows 10, 4K display, 250% scale), the console widget's minimum height is too large to fit in the provided space, which pushes the bottom of the window below the top of the taskbar even when maximized. For whatever reason, this creates the odd behavior where if the console widget is enabled when the window is maximized, restoring followed by re-maximizing the window aligns the bottom of the window of the taskbar without shrinking the console widget (i.e. everything is moved to the ideal position). Attempting to resize the console widget again, however, reverts the window to the previous state. It is worth noting that none of these behaviors occur if the widget does not have a minimumHeight() that is greater than the available space.
Aside from this, the following script demonstrates the limited utility of such a constrained docking area for auxiliary visuals.

Script
import napari
import numpy as np
from matplotlib.backends.backend_qt5agg import FigureCanvas
from matplotlib.figure import Figure

x = np.linspace(-50, 50)

viewer = napari.Viewer()
window = viewer.window

right_figure = Figure()
right_figure.gca().plot(x, x ** 2)
viewer.window.add_dock_widget(FigureCanvas(right_figure), area='right')

bottom_figure = Figure()
bottom_figure.gca().plot(x, x ** 3)
viewer.window.add_dock_widget(FigureCanvas(bottom_figure), area='bottom')

napari.run()

Screenshots:
Based on these screenshots alone, one can surmise that the minimum size of at least one built-in Qt widget is based on scaled screen coordinates, and on some setups, this minimum size is quite large. (likely a widget related to the vispy.scene.widgets.viewbox.ViewBox, e.g. the Camera, but I haven't had time to trace the origin of the issue).
Maximum widget sizes with my display scale set to 250% (the default for my display):

Maximum widget sizes with my display scale set to 200%:

Environment

About

napari: 0.4.11
Platform: Windows-10-10.0.19043-SP0
Python: 3.9.7 | packaged by conda-forge | (default, Sep 14 2021, 01:11:01) [MSC v.1916 64 bit (AMD64)]
Qt: 5.12.9
PyQt5: 5.12.3
NumPy: 1.21.2
SciPy: 1.7.1
Dask: 2021.09.1
VisPy: 0.8.1
OpenGL:

GL version: 4.6.0 - Build <IP_ADDRESS>49
MAX_TEXTURE_SIZE: 16384

Screens:

screen 1: resolution 1280x720, scale 3.0

Plugins:

NIRS Widget
console: 0.0.4
scikit-image: 0.4.11
svg: 0.1.5

Hi @QuaternionsRock, thanks for the report.
Could you please add a screenshot, it helps in understanding the issue at a glance.

Updating this issue as a UI/UX consideration.


Support for Symfony 4.3 EventDispatcher->dispatch definition

Fix #77 : Huge memory consumption
The definition of the Eventdispatcher->dispatch function has been changed in Symfony 4.3, throwing a deprecation warning.
For large data sets, this means that this deprecation warning is thrown a whole lot of times which takes up a surprising amount of memory.

@viniychuk Any chance of updating this merge request/commit to packagist?

I asked on the original issue and no answer for now.
Your can use this line in your composer.json to be up-to-date with master:
        "youshido/graphql-bundle": "dev-master#5f192e7cab00710b03e4dd24169e5cdb32b5fcb6"

Hi, it has been updated, but Packagist hooks didn't work — had to re-connect them.
Any of you guys interested in getting write permissions to repo?

Cheers


change to sha1 from md5

Description
Switching to sha1 to be FIPS compliant, which might be a future concern.
Going off the approved algorithms list I found: https://csrc.nist.gov/projects/hash-functions

@HubertStefanski I'm working on a branch with similar changes but found a few other issues that I would like to address. There seems to be a problem with importing the same dashboards from different namespaces. Some of them do not get imported when already present in another namespace. I'm currently debugging this.

@HubertStefanski it turned out that custom UIDs override folders. The same UID can't be present even in two different folders.

contained in #247


Add UI to set talk & notifications

from presets

length: 60

notifications: 40, 45, 50

length: 30

notifications: 15, 20

supported by d24492457ea78e8ab5b9e5e6c50f95af48828a74


Plugin: Math - Support European-style decimal separator

Autodetect input mode - UK-style decimal separator "." (current) or European-style decimal separator "," (new) - and ensure the calculation and output matches what the user expects.

Follows the suggested implementation from the MathJS library documentation: http://mathjs.org/examples/browser/custom_separators.html.html
Implements #246

I also bolded the calculation result!

looks good to me 👍


Is it possible to set different y axis for each track shown on fluff profile image ?

Hi,
I would like to use fluff profile to show my tracks, is it possible to manually set different vertical viewing range(y axis) for each track like UCSC genome browser ?
By the way, can I set different colors for each track by -c option ?
Thanks for your help !

Yes, both are possible! For example, let's say I run the following command:
fluff profile -i chr1:167100000-167200000 \
 -d H1_H3K27ac.bam mesendoderm_H3K27ac.bam neuronal_progenitor_H3K27ac.bam  \
 -o profile.png

This would result in the following picture:

Now I can change both scale as well as color.
fluff profile -i chr1:167100000-167200000 \
 -d H1_H3K27ac.bam mesendoderm_H3K27ac.bam neuronal_progenitor_H3K27ac.bam \
 -c purple,#386cb0,orange -S 100,300,300 -o profile2.png

With this as result:

For colors you can use color names, or hex code if you want to have more control. If you specify one of the ColorBrewer palette names fluff will cycle through this palette for every track.

That's odd...When I run, I got the error message:
biofluff                  2.1.0                    py27_0    bioconda
fluff profile -i chr18:14184022-14546879 -d ../bwfiles/K27ac_smooth.bw ../bwfiles/K27Ac_145.bw \ ../bwfiles/K27ac_155.bw \
../bwfiles/K27Ac_190.bw ../bwfiles/K27ac_142.bw ../bwfiles/K27Ac_237.bw \ ../bwfiles/K27Ac_255.bw ../bwfiles/K27ac_195.bw \
../bwfiles/k27ac_348.bw \
-o K27ac_snapshot \
-S 100,60,60,40,40,40,60,60,100 \
-a /database/genome/zebrafish_refFlat.bed \
-c #32cd32,purple,purple,#386cb0,#386cb0,#386cb0,orange,orange,orange \
-b white

The error message:
usage: fluff [-h] [options] profile [-i INTERVAL] -d [FILE [FILE ...]] -o name
                                    [-n] [-a FILE] [-t GROUPS] [-s GROUPS]
                                    [-S INT] [-f FRAGMENTSIZE] [-D] [-R] [-r]
                                    [-c NAMES] [-b BACKGROUND]
                                    [-T TEXTFONTSIZE] [-H] [-h]
fluff [-h] [options] profile: error: argument -c: expected one argument

What's the possible problem ?

Can you try the same command, but with the -c argument quoted:
-c "#32cd32,purple,purple,#386cb0,#386cb0,#386cb0,orange,orange,orange" \

OH! It worked! Thanks for your kindly help!
I love fluff to plot figures, it is easy to use and you guys have superb taste for beauty :)  The only problem is its documentation... Sometimes it has too little information to be understood. I recommended it to many my non-bioinfo classmates while they kept asking me detail usage. I think if its documentation could be more user-friendly, it will definitely attract many people. Deeptools is a good example. Please don't let fluff be buried because of something could be avoided.
Thanks again.

I'm glad it works. Thanks for your feedback on the documentation, it is a valid point.


Cosyvoice2.0 is released!

Are you interested in doing reverse engineering for the v2 tokenizer ？

Yes, don't worry, I'll do it when I'm free these two days.

https://github.com/xingchensong/S3Tokenizer/pull/18

#18

it works ! in  https://github.com/xingchensong/S3Tokenizer/pull/18

pip install -U S3Tokenizer

世上还是好人多啊QAQ老哥你开直播吧我给你刷火箭

#18

it works ! in #18

Thank you for sharing. However, why does it report the following error when I load the speech_tokenizer_v2_25hz version? The model used is S3TokenizerV2, and the onnx is the speech_tokenizer_v2.onnx provided by the official. Could you please tell me where the problem lies?

cc @Mddct

#18

it works ! in #18

Thank you for sharing. However, why does it report the following error when I load the speech_tokenizer_v2_25hz version? The model used is S3TokenizerV2, and the onnx is the speech_tokenizer_v2.onnx provided by the official. Could you please tell me where the problem lies?

I feel very convenient to use and haven't encountered this problem. Maybe you can check if your s3tokenizer code is up-to-date?

#18

it works ! in #18

Thank you for sharing. However, why does it report the following error when I load the speech_tokenizer_v2_25hz version? The model used is S3TokenizerV2, and the onnx is the speech_tokenizer_v2.onnx provided by the official. Could you please tell me where the problem lies?

I feel it very convenient to use and haven't encountered this problem. Maybe you can check if your s3tokenizer code is up-to-date?

I'm using the latest S3Tokenizer. However, the speech_tokenizer_v2.onnx is provided by the official of Cosyvoice2 instead of being automatically downloaded by S3Tokenizer (it couldn't be automatically downloaded due to certain reasons). Could this be related?

官方12.17更新过onnx

官方12.17更新过onnx，本仓库用的是最新的onnx

原来如此！我重新下载了更新后的模型，确实没有问题了，感谢!