fix hanging get traversal

gets were hanging for up to 15 minutes

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

Project coverage is 54.71%. Comparing base (f5a1022) to head (80e08f7).

Files
Patch %
Lines

impl/internal/dht/getput.go
64.58%
14 Missing and 3 partials :warning:

impl/pkg/service/pkarr.go
39.28%
15 Missing and 2 partials :warning:

impl/pkg/dht/logging.go
20.00%
4 Missing :warning:

impl/cmd/cli/identity.go
0.00%
1 Missing :warning:

impl/pkg/dht/dht.go
50.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #176      +/-   ##
==========================================
+ Coverage   54.43%   54.71%   +0.28%
==========================================
  Files          31       31
  Lines        2370     2427      +57
==========================================
+ Hits         1290     1328      +38
- Misses        955      970      +15
- Partials      125      129       +4

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


lr_scheduler step() called after every epoch instead of after every batch

🐛 Bug
When using a learning rate scheduler, the scheduler's step() method is called once at the start of every epoch() and not, as mentioned in the docs, at the start of every batch. This leads to a constant learning rate per epoch when using for instance the CyclicLR scheduler.
Please reproduce using the BoringModel
BoringModel
To Reproduce
Run the notebook. See the comments before the test is being done and examine the output.
Expected behavior

lr_scheduler step() is called every batch and not every epoch
learning rate changes every batch.

Environment

CUDA:

GPU:

Tesla T4

available:         True
version:           10.1

Packages:

numpy:             1.19.5
pyTorch_debug:     True
pyTorch_version:   1.7.0+cu101
pytorch-lightning: 1.1.5
tqdm:              4.41.1

System:

OS:                Linux
architecture:

64bit

processor:         x86_64
python:            3.6.9
version:           #1 SMP Thu Jul 23 08:00:38 PDT 2020

Additional context

https://pytorch-lightning.readthedocs.io/en/stable/lightning_module.html#configure-optimizers
I fixed it for you:
https://colab.research.google.com/drive/1yHaDmHSFFWjv2R90U7Tvp9Ek1KgYA2-c?usp=sharing

https://pytorch-lightning.readthedocs.io/en/stable/lightning_module.html#configure-optimizers
I fixed it for you:
https://colab.research.google.com/drive/1yHaDmHSFFWjv2R90U7Tvp9Ek1KgYA2-c?usp=sharing

Thanks for your quick response. So it is not a code issue.
I guess the confusion is between the docs of pytorch that mention for CyclicLR that  step() must be called every batch, the generatl pytorch docs that talk about iterations (not batches/epochs), and the pytorch lightning docs that have generalized a bit and allow update every n batches or epochs.

Thanks for your quick response. So it is not a code issue.
I guess the confusion is between the docs of pytorch that mention for CyclicLR that  step() must be called every batch, the generatl pytorch docs that talk about iterations (not batches/epochs), and the pytorch lightning docs that have generalized a bit and allow update every n batches or epochs.

Yes you can also call the scheduler manually yourself and not rely on Lightning.
However keep in mind that you may have to write much more boilerplate code to cover all cases.
One just has to know the terminology, some lr_schedulers define an epoch as the number of steps it takes before an update is made, this is then different from a training epoch.
It's just a matter of defintion.

Yes you can also call the scheduler manually yourself and not rely on Lightning.
However keep in mind that you may have to write much more boilerplate code to cover all cases.
One just has to know the terminology, some lr_schedulers define an epoch as the number of steps it takes before an update is made, this is then different from a training epoch.
It's just a matter of defintion.

I want to avoid writing boilerplate. It is now working as expected. I am coming from tensorflow/keras, where I ran into a lot of issues (memory leaks, memory usage in general, tensorflow addons not working, complicated dataset code, etc.). I could net get cyclic and differential learning rates to work with tensorflow/keras. So far, I am liking the pytorch lightning APIs very much. The model of pytorch/pyrtoch lightning is much more clean and provides lots of ways to debug. Debugging is easier since the existing code is easier to understand and extend. It looks like pytorch lightning is at the right abstraction level.

I want to avoid writing boilerplate. It is now working as expected. I am coming from tensorflow/keras, where I ran into a lot of issues (memory leaks, memory usage in general, tensorflow addons not working, complicated dataset code, etc.). I could net get cyclic and differential learning rates to work with tensorflow/keras. So far, I am liking the pytorch lightning APIs very much. The model of pytorch/pyrtoch lightning is much more clean and provides lots of ways to debug. Debugging is easier since the existing code is easier to understand and extend. It looks like pytorch lightning is at the right abstraction level.


Modified submixes/submixes.py fix errors

Hi ethman,
While I utilize the submixes utils, I found some bugs and fixed. I think it's all about the variable names, thus straight-forward.
btw, Thanks for sharing such a great & usable dataset

Awesome. I'm a bit swamped right now, but I will look this over and merge next week. Thanks for checking it out!


The permmissions doc doesn't explain why removing a permmission may silently fail

It seems possible to make a delete request that succeeds but does nothing
Owners role permission can't be removed in sharepoint and is silently failing for me.

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

ID: fbac80e5-ae35-b804-addd-53ccaf49e693
Version Independent ID: b2a13be1-7783-38c2-c318-1261847240fa
Content: List who has access to a file - Microsoft Graph v1.0
Content Source: api-reference/v1.0/api/driveitem-list-permissions.md
Product: sharepoint
Technology: microsoft-graph
GitHub Login: @JeremyKelley
Microsoft Alias: JeremyKelley

As part of a repository clean up effort we are closing older issues. If this issue remains, please: open a new issue, reference this issue, and provide any additional details that may help in resolution. Thank you for your understanding as we work to improve our responsiveness.


Custom spawn points

Custom spawn points are deactivated for the Burglar class, because due to a bug in a previous version of PZ, custom spawn points didn't work correctly.
Will have to check if they work in the current PZ version. If not, the code should be removed.

Looks like there is no clean way of adding custom spawnpoints, because they are now hidden away a bit ... instead of being kept in a global table, they are returned by a global function which recreates the whole table each time it is called.
This means, that we can't inject a new spawnpoint without overriding the function which returns the table.
The other idea would've been to intercept the point at which the SpawnRegions() function is called, but this would also require some overriding of vanilla behaviour.

function OnSpawnRegionsLoaded(regions)
    for i=1,#regions do
        local rgn = regions[i]
        print('OnSpawnRegionsLoaded: '..rgn.name)
    end
    local r = {
        name = 'Mall', points = {
            unemployed = {
                {worldX=46, worldY=19, posX=138, posY=236},
            },
        }
    }
    table.insert(regions, r)
end

Events.OnSpawnRegionsLoaded.Add(OnSpawnRegionsLoaded)

Tim Baker provided the above solution. Will try to implement it ASAP.


Fix typo in ServiceDiscoveryRecipe

What changes are proposed in this pull request?
Fixed a typo
Why are the changes needed?
Does this PR introduce any user facing changes?
No

alluxio-bot, merge this please


Facebook ID Token login

Sign-in with ID Token should be supported when using the Facebook OAuth provider, this is useful for implementing Facebook login with the native Facebook SDK which provides a better UX compared to the browser-based login.
Additional context

i think we need https://dev.to/meatboy/how-to-add-limited-login-facebook-on-ios-and-android-on-server-side-and-client-side-4616
and maybe need use with https://www.better-auth.com/docs/plugins/oidc-provider
hi @Bekacru do you think it's possible?


Use miekg/dns instead of go's builtin resolver, so that we can set edns0

In doing this we've had to import a bunch of code from go's net package to read
the /etc/resolv.conf file, since miekg/dns needs to be given a nameserver to
communicate with

LGTM


Cannot dynamically provide properties containing '_' with environment variable

Example :

docker run -e pipeline.unsafe_shutdown=true

In env2yaml.go, this environment variable will be passed to normalizeSetting to be transformed as pipeline.unsafe.shutdown and then rejected by isValidSetting.

Here is a PR with a test to reproduce the problem and a fix : https://github.com/elastic/logstash-docker/pull/32

Thanks for the PR!
Fixed by e8eaa11, and a806a7c.


Removed header from module

It was not updating with config.js values.

Will close in favor of #14


Sales-Channel migration from Magento

Please make sure that every store is properly migrated to a new sales-channel.

[public] Shopware automated response: This issue is linked to the internal issue: MIG-782
[created from MIG-782, comment 497205]


Passing Decoded PCM data clicks

Utilizing this to encode PCM audio straight from user input i.e. microphone audio, results in a clicking noise present in audio in between packets which is clearly audible when when played back through an Audio Worklet Node which is most noticeable when maintaining a note while singing.
This persists even if a large section of packets are encoded, decoded and then added to a buffer lasting a few seconds prior to being sent for playback through the Audio Node.
I have tested with a bufferSize being posted out of an AudioWorkletProcessor from the low end of 640 to the upper end of a full second of 44100 / 48000 with the result being the same, a clicking noise present in between packets only after encoding the individual Float32Array's that are provided, and promptly decoding them for audio playback.
Alternative Encoder methods did not present this clicking when feeding to the Audio Worklet for playback so I am hesitant to say that the code for playback is the culprit.
Any guidance or input on a possible resolution would be greatly and deeply appreciated, thank you.

Is each packet encoded as a separate buffer? I am not sure if there is a specific thing that must change in the QOA format to enable seamless streaming. Do you have a simple test case that I could try? Unsure at this point whether it's my implementation or a limitation of QOA itself.

Thank you for reaching out, each packet is encoded as a separate buffer, the intention was to see if QOA could be used for very specific live audio transmission.
I saw a few references in the base QOA that could have been alluring to being possible solutions, one in particular with sine waves and weights associated with it's compression located at this issue: https://github.com/phoboslab/qoa/issues/25
Very busy with work this week but I will try to get a simple test case assembled for use ASAP to hopefully help diagnose.


[Enhancement] Icon search performance

Is your feature request related to a problem? Please describe.
Loading the VoiceActivity plugin takes upwards of 15 seconds on my machine, spending the vast majority of time iterating over Webpack modules for each requested icon. This makes using Discord at startup an unpleasant wait.
Describe the solution you'd like
Replace individual search calls with a pattern matching any required icons, to reduce interation of all modules to a single-pass. The resulting subset can then be trivially searched.
Describe alternatives you've considered

Bulk fetching modules describing an SVG image, then searching within the resultant list per requested icon

An improvement, but still slower and more memory-intensive than the proposed feature

Dissecting the internals of Discord to find a direct mapping of icon to module

Requires an incredible time investment for something which may be a fragile solution

Additional context
Profile of plugin loading

Thanks for the phenomenally quick turnaround on this! 🤩

I've been trying to figure out the massive speed-up, but the diff suggests it's something already solved within BetterDiscord?


feat(deisrel): Add ability to stage standalone router chart

LGTM, https://github.com/deis/deisrel/pull/73#discussion-diff-65579966 is not blocking this PR


swiftenv.fuller.li is down

Running eval "$(curl -sL https://swiftenv.fuller.li/install.sh)" has been working for my projects, but today, it instead returned a Cloudflare error site. I was unable to capture the site before it appeared to return to serving the correct website, so I restarted my failed builds. However, they now say curl: (51) SSL: no alternative certificate subject name matches target host name 'swiftenv.fuller.li' and visiting the site in Safari says that Safari can't establish a secure connection to the server "swiftenv.fuller.li". Using curl from my computer allows me to read the site, but I don't want to do so insecurely from Travis.
Is there anything I can do besides waiting for the site to be fixed?

I think in the interim you can use eval "$(curl -sL https://raw.githubusercontent.com/kylef/swiftenv/master/docs/install.sh)", but I have also run in to this problem and would not like to point at the GitHub-hosted file in the long run, unless it become recommended.

I'm using eval "$(curl -sL https://swiftenv.fuller.li/install.sh || curl -sL https://raw.githubusercontent.com/kylef/swiftenv/master/docs/install.sh)", which tries the main website but uses the GitHub hosted one if the main site is unavailable. It seems to be working

Unfortunately, ReadTheDocs (the service I use to host the documentation) has had some implementation detail changes which have broken compatibility with the CDN used (CloudFlare).
As per the following in their documentation:

Due to a limitation, a domain cannot be proxied on Cloudflare to another Cloudflare account that also proxies. This results in a “CNAME Cross-User Banned” error. In order to do SSL termination, we must proxy this connection.

This has caused a couple of problems, namely also breaking HTTPS for the service. I've managed to restore this by tweaking configuration.


Escape markup tokens

I would like to send markupv2 and dont have to worry about escaping markup tokens like;
String escapedMsg = toEscapeMsg
.replace("", "\")
.replace("", "\")
.replace("[", "\[")
.replace("", "\\");
Without this I get bad requests back ;)

Duplicate #1176


Matplotlib backend should be forced to Agg

Recently we've started to see this error appear:
matplotlib.use() must be called *before* pylab, matplotlib.pyplot,
or matplotlib.backends is imported for the first time.

  warnings.warn(_use_error_msg)
/var/build/user_builds/astropy/checkouts/latest/docs/analytic_functions/index.rst:83: WARNING: Exception occurred in plotting index-1
 from /var/build/user_builds/astropy/checkouts/latest/docs/analytic_functions/index.rst:
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.7/matplotlib/sphinxext/plot_directive.py", line 515, in run_code
    exec code in ns
  File "<string>", line 14, in <module>
  File "/usr/lib/pymodules/python2.7/matplotlib/pyplot.py", line 1046, in subplots
    fig = figure(**fig_kw)
  File "/usr/lib/pymodules/python2.7/matplotlib/pyplot.py", line 423, in figure
    **kwargs)
  File "/usr/lib/pymodules/python2.7/matplotlib/backends/backend_tkagg.py", line 79, in new_figure_manager
    return new_figure_manager_given_figure(num, figure)
  File "/usr/lib/pymodules/python2.7/matplotlib/backends/backend_tkagg.py", line 87, in new_figure_manager_given_figure
    window = Tk.Tk()
  File "/usr/lib/python2.7/lib-tk/Tkinter.py", line 1767, in __init__
    self.tk = _tkinter.create(screenName, baseName, className, interactive, wantobjects, useTk, sync, use)
TclError: no display name and no $DISPLAY environment variable

when using the matplotlib plot_directive. It seems something has changed in the RTD configuration and the backend is defaulting to something interactive.
(here's the Astropy issue where someone first reported this: https://github.com/astropy/astropy/issues/3600)

@ericholscher - any idea what could be causing this? All our plots are currently broken due to this issue, which seems beyond our control and related to RTD. Are other projects not having this issue?

We haven't changed anything that I know of. Did you try this workaround?
http://stackoverflow.com/a/3054314
On Tue, Mar 24, 2015 at 1:48 AM, Thomas Robitaille<EMAIL_ADDRESS>
wrote:

@ericholscher https://github.com/ericholscher - any idea what could be
causing this? All our plots are currently broken due to this issue, which
seems beyond our control and related to RTD. Are other projects not having
this issue?
—
Reply to this email directly or view it on GitHub
https://github.com/rtfd/readthedocs.org/issues/1195#issuecomment-85406365
.

--
Eric Holscher
Maker of the internet residing in Portland, Or
http://ericholscher.com

@ericholscher - yes, we already call this but we see:
/usr/lib/pymodules/python2.7/matplotlib/__init__.py:1173: UserWarning:  This call to matplotlib.use() has no effect
because the backend has already been chosen;
matplotlib.use() must be called *before* pylab, matplotlib.pyplot,
or matplotlib.backends is imported for the first time.

which suggests that the backend has already been set beforehand.

Any suggestions? I don't know much about matplotlib, and we haven't done
anything that I know should effect it. I'm happy to upgrade it, it seems
we're using 1.3.1, but I don't really know how to debug issues since I
don't know the software at all.
On Tue, Mar 24, 2015 at 9:01 AM, Thomas Robitaille<EMAIL_ADDRESS>
wrote:

@ericholscher https://github.com/ericholscher - yes, we already call
this but we see:
/usr/lib/pymodules/python2.7/matplotlib/init.py:1173: UserWarning:  This call to matplotlib.use() has no effect
because the backend has already been chosen;
matplotlib.use() must be called before pylab, matplotlib.pyplot,
or matplotlib.backends is imported for the first time.
which suggests that the backend has already been set beforehand.
—
Reply to this email directly or view it on GitHub
https://github.com/rtfd/readthedocs.org/issues/1195#issuecomment-85576963
.

--
Eric Holscher
Maker of the internet residing in Portland, Or
http://ericholscher.com

Maybe @mdboom has some ideas - I'm also a bit puzzled by the behavior currently. Basically I don't understand how it ever changes to use TkAgg (since it seems that default on RTD is Agg).

Ah, sorry, I think I have it. Check out this minimalist project:
https://github.com/astrofrog/test-rtd
The RTD log (https://readthedocs.org/builds/test-rtd-astrofrog/2482239/) shows:
Making output directory...
Running Sphinx v1.2.2
TkAgg

so the default backend is set to TkAgg.
@mdboom, do you have any suggestions on how to make sure the default backend is Agg, not TkAgg, system-wide?

Looks like there's a config in /etc/matplotlibrc that sets it.
On Tue, Mar 24, 2015 at 9:14 AM, Thomas Robitaille<EMAIL_ADDRESS>
wrote:

Ah, sorry, I think I have it. Check out this minimalist project:
https://github.com/astrofrog/test-rtd
The RTD log (https://readthedocs.org/builds/test-rtd-astrofrog/2482239/)
shows:
Making output directory...
Running Sphinx v1.2.2
TkAgg
so the default backend is set to TkAgg.
@mdboom https://github.com/mdboom, do you have any suggestions on how
to make sure the default backend is Agg, not TkAgg, system-wide?
—
Reply to this email directly or view it on GitHub
https://github.com/rtfd/readthedocs.org/issues/1195#issuecomment-85586195
.

--
Eric Holscher
Maker of the internet residing in Portland, Or
http://ericholscher.com

@ericholscher - ah great, can you change it to Agg? There's no reason it should be anything else on a display-less server like RTD.

@ericholscher - it seems to be working now, did you make the change? If so, this issue can be closed :)

Yea, should be changed. Will keep this open because it needs to be fixed in
our deployment scripts as well, so it doesn't break again when we rev a new
build server.
On Tue, Mar 24, 2015 at 10:53 AM, Thomas Robitaille <
<EMAIL_ADDRESS>wrote:

@ericholscher https://github.com/ericholscher - it seems to be working
now, did you make the change? If so, this issue can be closed :)
—
Reply to this email directly or view it on GitHub
https://github.com/rtfd/readthedocs.org/issues/1195#issuecomment-85620289
.

--
Eric Holscher
Maker of the internet residing in Portland, Or
http://ericholscher.com

Believe this should be fixed.

Hello, i meet the same problem
"because the backend has already been chosen; matplotlib.use() must be called before pylab, matplotlib.pyplot,or matplotlib.backends is imported for the first time."
I am wondering that how to config  /etc/matplotlibrc to solve this problem.
Thanks !!!

Hi anyone could comment how to trouble shoot this issue. how to configure /etc/matplotlibrc to solve this problem.

Well putting these lines at the beginning of the Py code solves the purpose:
import matplotlib
matplotlib.use('Agg')

Hi,
this didn't worked for me
import matplotlib
matplotlib.use('Agg')
so instead I used:
import matplotlib.pyplot as plt.
plt.switch_backend('agg')
that worked. maybe it's helpful to someone else.

Hello, I am working in spyder in windows and this FINALLY worked for me:
Go to Tools>Preferences>IPython terminal>Graphs  (Menu names are an approximation since my spyder is in spanish)
Once here, uncheck the box "activate support".
Reestart and then type:
import matplotlib
matplotlib.use('TkAgg')
import matplotlib.pyplot as plt
#Example of plot saved to pdf file
a = [1, 2, 3, 4]
b = [2, 4, 6, 8]
plt.plot(a, b)
plt.savefig('test.pdf')
I hope this help you.

Hello.
In spyder to to
Tools>Preferences>IPython Termina>Graphs
Uncheck the box "Activate support"
Re-start spyder
After including:
import matplotlib
matplotlib.use('TkAgg')
import matplotlib.pyplot as plt
I FINALLY was able to obtain a pdf file that was not in blank after doing:
a = [1, 2, 3, 4]
b = [2, 4, 6, 8]
plt.plot(a, b)
plt.savefig('test.pdf')

Could it be that this problem resurfaced in the Python 3.6 image? Since it get a
No module named '_tkinter'
for a module importing matplotlib.

@Tillsten did you try specifying that requirement on your requirements.txt file? If you still think the problem is rtd related, please open a new issue with your rtd project link.

Yes, I did. The problem seems to be identical to this issue: matplotlib is installed, but the default backend is set to tkagg (using tkinter), which is not available on the headless rtd-image. This problem can be fixed by changing the rc file like above or upgrading matplotlib to 3.0, which should fix that issues.

My current workaround is just import and setting the matplotlib backend in the sphinx conf.py  :
 import matplotlib
 matplotlib.use('agg')

Thanks for sharing the workaround! I'll take a look to the docker images later

I didn't see anything else on the docker images related to this, so, not sure how this was solved (maybe rtd wasn't using the docker images in that time?). We have a workaround, so, not sure if we should reopen. Maybe adding this to the faq?


RunAs=CurrentUser is not supported in .NET Core

Please update the documentation and make clear that the connection string option "RunAs=CurrentUser" is not available in .NET Core.
Also make clear that "Azure AD Integrated Authentication" is therefore also not available in .NET Core.

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

ID: 17824597-7e9b-4ca5-12f7-310b0ce4bdf4
Version Independent ID: 49e1c9ee-fabf-df55-5bd9-1019abda22f1
Content: Service-to-service authentication to Azure Key Vault using .NET
Content Source: articles/key-vault/general/service-to-service-authentication.md
Service: key-vault
Sub-service: general
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@Rookian Thanks for your feedback! We will investigate and update as appropriate.

Hello. The only connection string that is unsupported for .NET Core is "RunAs=CurrentUser", and we now specify this in the table.  Thank you for your feedback! #please-close


Bugfix router login redirect

Currently if auth provider is set to dispatch-auth-provider-basic if the requestor is not authenticated, the requestor will remain hanging trying to access /login page.
e.g
User is unauthorized gets redirect to "/login" from L143 after this the hook on routerBefore will be called again but this time the path.to is "/login" thus the router will actually have no resolve at all, and next() is never called.

Thanks @stefanm8

Ah, I had fixed this in #266 but you beat me to it @stefanm8, thanks!


Node 5, Babel 6 and npm 3

Investigate whether these cause any problems for Wowser.

I have Node 5 & npm 3 (Win7 x64), so I tried it and everything seems to work.
I had to manually compile ffi bindings and install some module dependencies manually as well (was getting require errors all over the place).
I don't fully understand the consequences of moving from previous to this new flat module system.
Also there are a couple of Riot related JS errors, but the whole pipeline is working without any issues. How cool is that :smiley:

Excellent, thanks for checking! :+1:
Will give this a go later this week. Am planning on swapping out Riot with React.

Seem to run into https://github.com/node-ffi/node-ffi/issues/259 immediately on Node v5. Perhaps that doesn't apply to Windows?
Adding support for Node v4.x at the very least, seeing as it's long-term support.

Node 5.1.0 was just released and supposedly includes a fix!

Node 5.1 and npm3 confirmed working. :+1:


Appdex score and active requests not working

Using ASP.NET Core 3.1 with this configuration, Appdex is always 1 and Active requests is always 0. Am I doing something wrong or is this a bug?
They use application.httprequests__active and application.httprequests__apdex in grafana
public void ConfigureServices(IServiceCollection services)
{
    ...
    var metrics = AppMetrics.CreateDefaultBuilder()
        .Report.ToInfluxDb(options =>
        {
           options.InfluxDb.BaseUri = new Uri(appSettings.InfluxDbUri);
           options.InfluxDb.Database = appSettings.InfluxDbName;
           options.InfluxDb.CreateDataBaseIfNotExists = true;
        })
        .Build();

    services.AddMetrics(metrics);
    services.AddMetricsTrackingMiddleware(x =>
    {
        x.ApdexTrackingEnabled = true;
        x.ApdexTSeconds = 0.01;
    });
    services.AddMetricsReportingHostedService();
    services.AddAppMetricsCollectors();
    services.AddControllers().AddMetrics();
    ...
}

public void Configure(IApplicationBuilder app, IWebHostEnvironment env)
{
    ....
    app.UseMetricsAllMiddleware();
    ....
}

<PackageReference Include="App.Metrics.AspNetCore.All" Version="4.2.0-preview.0.27" />
<PackageReference Include="App.Metrics.Extensions.Collectors" Version="4.2.0-preview.0.27" />
<PackageReference Include="App.Metrics" Version="4.2.0-preview.0.27" />
<PackageReference Include="App.Metrics.AspNetCore" Version="4.2.0-preview.0.27" />
<PackageReference Include="App.Metrics.AspNetCore.Mvc" Version="4.2.0-preview.0.27" />
<PackageReference Include="App.Metrics.Reporting.InfluxDb" Version="4.2.0-preview.0.27" />

EDIT: Seems like Health is also not working application.health__results but application.health__score shows healthy

Same for me.
<PackageReference Include="App.Metrics.AspNetCore.All" Version="4.1.0" />
<PackageReference Include="App.Metrics.Formatters.Prometheus" Version="4.1.0" />

I'm pretty sure this has been fixed. Could you try a package update? A stable 4.2.0 will be released soon

Same for me on 4.2.0 & 4.3.0-preview.0.30
<PackageReference Include="App.Metrics.AspNetCore.All" Version="4.3.0-preview.0.30" />
<PackageReference Include="App.Metrics.Formatters.InfluxDB" Version="4.3.0-preview.0.30" />


Delete ServiceMonitor does not re-update configuration

Hi
perhaps i'm missing something but seems like when deleting ServiceMonitor the operator does not reload the configuration and update prometheus.
after deleting serviceMonitor i dont see the change refelected on the prometheus configuration, even after along time.
adding new ServiceMonitor being added as  a new Target, but what about the opposite?
doesnt deleting ServiceMonitor should be reflected on the configuration and remove that job?
thanks
Chen

How long did you wait? Could you watch the secret that gets generated and see if that gets updated? Did prometheus reload successfully?

i've waited long time, 5-10 minutes.
i'm not sure what was the problem, i ended up redeploying the stack and problem didnt appear again, it might be it was an issue on the deployment, not sure.
anyway , closing this.


Support onAction and drag and drop in listbox

While making the docs examples for drag and drop, I realized it wouldn't work with listbox because an explicit focusable drag affordance was required to activate accessible drag and drop mode and listbox options don't support focusable children. This is because the interactions conflict with selection and other actions on the items themselves. It would be useful to support drag and drop without a separate drag button in these cases.
This is accomplished by using Enter to trigger dragging, while reserving Space for selection. If row actions are also present, then it becomes Alt + Enter. For touch screen readers, a long press is used to trigger dragging (this actually fires the native onDragStart event). When actions are present, the first long press enters selection mode as before, and long pressing again drags (same behavior as without a screen reader on touch).
A new hasDragButton option is added to useDrag to opt-into a separate drag button as before, which keeps the interactions off the main draggable element and only activates accessible dnd through the drag button. This is used in ListView.

Yeah it seems VO on iOS has a bug where setting aria-hidden on an option in a listbox doesn't cause the a11y tree to be updated properly so it thinks the items are still there when you swipe but they aren't. If you tap outside the list and then go back in it starts working. I'm not exactly sure if we can work around this or not but we can look into it separately.
As for TalkBack, it seems to work fine on my device so not sure.


resize is now a public method

@fnicollet had the (much better) idea to just check for the existence of _resize() to support both the public and private method simultaneously.
https://github.com/mapbox/mapbox-gl-leaflet/commit/43e888604a91f5dddc1b5b215fbf5607a44320a9

Yes, that's what I ended up doing
Le 6 déc. 2016 04:07, "john gravois"<EMAIL_ADDRESS>a écrit :

@fnicollet https://github.com/fnicollet had the (much better) idea to
just check for the existence of _resize() to support both the public and
private method simultaneously.
mapbox@43e8886
https://github.com/mapbox/mapbox-gl-leaflet/commit/43e888604a91f5dddc1b5b215fbf5607a44320a9
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/patrickarlt/mapbox-gl-leaflet/pull/1#issuecomment-265048210,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABLtoWUYxJqJ43BuPaAO7Y1VyFNKb5Apks5rFNFUgaJpZM4K5scs
.

Just in case mapbox changes his mind again :)
Le 6 déc. 2016 08:49, "Fabien Nicollet"<EMAIL_ADDRESS>a écrit :

Yes, that's what I ended up doing
Le 6 déc. 2016 04:07, "john gravois"<EMAIL_ADDRESS>a écrit :

@fnicollet https://github.com/fnicollet had the (much better) idea to
just check for the existence of _resize() to support both the public and
private method simultaneously.
mapbox@43e8886
https://github.com/mapbox/mapbox-gl-leaflet/commit/43e888604a91f5dddc1b5b215fbf5607a44320a9
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/patrickarlt/mapbox-gl-leaflet/pull/1#issuecomment-265048210,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABLtoWUYxJqJ43BuPaAO7Y1VyFNKb5Apks5rFNFUgaJpZM4K5scs
.


Implement code path for re-opening databases

This means we can't actually reopen DBs yet, which is a bit of a snag.
pub fn ensure_current_version(conn: &mut rusqlite::Connection) -> Result<DB> {
    if rusqlite::version_number() < MIN_SQLITE_VERSION {
        panic!("Mentat requires at least sqlite {}", MIN_SQLITE_VERSION);
    }

    let user_version = get_user_version(&conn)?;
    match user_version {
        0 => create_current_version(conn),
        // TODO: support updating or re-opening an existing store.
        v => bail!(ErrorKind::NotYetImplemented(format!("Opening databases with Mentat version: {}", v))),
    }
}

Misfire with hub.


Options are not updated if already open

Options (for example, callbacks) are not updated if pop up is already open
Because of this:
https://github.com/dimsemenov/Magnific-Popup/blob/master/src/js/core.js#L167
For callbacks I fixed it like this:
if(mfp.isOpen) {
  mfp.st.callbacks = data.callbacks;
  mfp.updateItemHTML();
  return;
}

Came across the same issue, fixed by just removing this condition altogether (the suggested addition didn't work).


Update readme to reflect parity repository moved

Update readme to reflect parity repository moved (renamed from ethcore to paritytech)

Also, feel free to submit a PR to parity to include the musicoin chain configuration.

Is this still maintained?

Also note, --dapps-port will be deprecated after parity 1.7


java.net.ConnectException: Connection refused error on 0.16 version

Hello,I am so sorry to tell that I met java.net.ConnectException on 0.16 version

    <dependency>
        <groupId>Azure</groupId>
        <artifactId>mmlspark</artifactId>
        <version>0.16</version>
    </dependency>

this is my command to start spark
spark2-submit --class  HuoyingLightGBM --master yarn --executor-memory 8G --num-executors 2 --driver-memory 8g --executor-cores 1  --conf spark.yarn.executor.memoryOverhead=1024 --conf spark.sql.shuffle.partitions=20 /data/recommend.jar
this is my code:
val executor = ss.sparkContext.getConf.getInt("spark.executor.instances", 2)
println("lucky spark.executor.instances is " + executor)**//this is 2**
val finamInput = trainInput.repartition(executor)

val classifierModel = new LightGBMClassifier()
  .setLabelCol("label")
  .setFeaturesCol("features")
  .setRawPredictionCol("rawPrediction")
  .setObjective("binary")
  //      .setParallelism("voting_parallel")
  .setLearningRate(0.1)
  .setMaxBin(128)
  .setNumLeaves(500)
  .setMaxDepth(10)
  .setFeatureFraction(0.8)
  .setNumIterations(10)
  .setBaggingFraction(1.0)
  .setBaggingFreq(5)
  .setDefaultListenPort(24001)
  .setTimeout(100)
  .fit(finamInput)

val testRank = classifierModel.transform(testInput)
testRank.show(100, false)
val binaryClassificationEvaluator = new BinaryClassificationEvaluator()
  .setMetricName("areaUnderROC")
  .setRawPredictionCol("rawPrediction")
  .setLabelCol("label")
val classificationMetric = binaryClassificationEvaluator.evaluate(testRank)

val epsilon = 1e-8
println(s"LightGBMClassifier ${binaryClassificationEvaluator.getMetricName} = $classificationMetric")

and this is driver and worker logs.
worker1.txt
worker2.txt
driver.txt

@land1725 I looked at the logs but didn't see any error message that might help me understand what is causing the errors.  The connection refused errors could be due to a number of issues.  For the driver logs, you sent me one file but usually there are three files which contain different debug statements: 1.) stdout 2.) stderr 3.) log4j logs
Would you be able to send me the other driver logs?

@imatiach-msft hello,the driver log has already contains 1.) stdout 2.) stderr 3.) log4j logs

@land1725 hmm, perhaps it does have stderr/stdout, I'm not sure, but I do see the log4j logs.
I never see this line printed however:
https://github.com/Azure/mmlspark/blob/0b84a230d1556ced87be9139dd798237711c1158/src/lightgbm/src/main/scala/LightGBMUtils.scala#L115
It seems the workers aren't able to make a socket connection to the driver.  I wonder why that is.

I do see this line in the driver log:
19/04/10 14:48:09 INFO spark.LightGBMClassifier: driver waiting for connections on host: <IP_ADDRESS> and port: 41185
So it seems like the driver is waiting on this host/port but the workers are not able to connect to it.
But as to why the connection is refused from the workers to the driver I'm not quite sure.

@imatiach-msft hello,how can I help you to find how to solve this problem.Do you need to add more Log Info and Build a new PR for me?

@land1725 might you have some cluster settings that prevent the workers from connecting to the driver node?  Did lightgbm work on this cluster in an earlier version of mmlspark before?

@imatiach-msft it is not possible about cluster settings that prevent the workers from connecting to the driver node.Because I use LightGBMRegressor,it is fine.
my mmlspark version is  latest 0.16
Azure
mmlspark
0.16

you can try to run some simple scala code in a notebook against the cluster.  For example, you can do something similar to method createDriverNodesThread:
https://github.com/Azure/mmlspark/blob/0b84a230d1556ced87be9139dd798237711c1158/src/lightgbm/src/main/scala/LightGBMUtils.scala#L97
specifically, create a server socket:
val driverServerSocket = new ServerSocket(0)
and then get host and port and wait for workers to connect:
val host = getDriverHost(df)
val port = driverServerSocket.getLocalPort
Then, do a mapPartitions and verify that the workers can connect to the driver using those settings.
One strange thing I see in the logs is that your host is <IP_ADDRESS>:
ApplicationMaster host: <IP_ADDRESS>
However, in the logs it says:
driver waiting for connections on host: <IP_ADDRESS> and port: 41185
It seems strange that those hosts are different?

@land1725 another useful thing may be to go to the yarn UI, go to the failing task and you should see an error for each worker.  However, it looks like you have already given me the logs for both workers and they had the same error (connection refused).  I am a bit surprised because usually that error is a red herring and the real error is on the other worker.  Were those logs from the 2 workers for the same run?

@land1725 if LightGBMRegressor works fine than something is very strange, because both regressor and classifier use the same method to connect the workers to driver.  The differences only appear after the initial connection has been established.

@imatiach-msft I have did a simple test ,it seems good
scala>     val host = getDriverHost(df)
host: String = <IP_ADDRESS>
scala> val port = driverServerSocket.getLocalPort
port: Int = 40678
scala>     val port = driverServerSocket.getLocalPort
port: Int = 40678
scala>     log.info(s"driver waiting for connections on host: ${host} and port: $port")
scala> df.repartition(2)
res21: org.apache.spark.sql.Dataset[org.apache.spark.sql.Row] = [vid: int, features: array]
scala> res21.rdd.count
res22: Long = 410040

@land1725 one thing you could try is call mapPartitions on the df, and inside it try to connect to the driver from the workers, similar to the code here:
https://github.com/Azure/mmlspark/blob/master/src/lightgbm/src/main/scala/TrainUtils.scala#L242
and then send some info like here:
https://github.com/Azure/mmlspark/blob/master/src/lightgbm/src/main/scala/TrainUtils.scala#L250
the workers should be able to connect to the driver using the driver's host and port and send some information over the socket.

@land1725 one thing I noticed, you are setting the port to be:
  .setDefaultListenPort(24001)

could you leave it as the default value:
val defaultLocalListenPort = 12400
probably won't make any difference, but worth a try.

@imatiach-msft hello ,I am happy to tell that this problems has been solvec.
in My code you can see I set timeout 100s
.setTimeout(100)
when I delete this code it is normal again,thanks


replace stat for creates

Fixed homebrew play to use creates arg, it's cleaner.

nice improvement


Max Number of Shapes Renderable with Mesh

When rendering over 1k PolyLines on a single mesh, as the number of lines increased, I saw other lines disappear and a gradient streak appear, stretching from the top-left corner towards the middle where some line was rendered. See the below image:

Given that my code works for a lower number of lines perfectly, I tried to allocate more than 1 mesh for drawing all the lines. Distributing the total number of lines among several meshes seems to have fixed the bug. I assume that this bug relates to some max bound for how many shapes a mesh can hold, but have not been able to find evidence of that in the documentation. If this bound does exist, it should be documented.

Is this on Vulkan, Metal, or Direct3D? I think this is most probably fixed in master by https://github.com/hecrj/coffee/commit/79651405673ecd16da21b887a622f6650b29f38f.
I backported the fix and released a new patch version: https://github.com/hecrj/coffee/releases/tag/0.3.2. Let me know if it fixes the issue!

Metal. Will try the fix and report back. Thanks

Seems fixed on my end. Thanks for the fast turn around!

Awesome! I've been meaning to backport those fixes for a while :sweat_smile:
Thank you for the detailed report!

It seems that the issue of losing vertices is still present, though the situation improved since the last fix as it takes more of them being drawn to notice this. I'm noticing a loss of Polylines when more than ~16,400 are drawn. The loss is fairly noticeable when ~16,800 are drawn. There is no gradient streak this time. If lines are distributed over multiple meshes then the issue is improved i.e. doubling the number of meshes drawn shows a loss of lines at ~16,40 x 2 lines.

Not sure what i couldt be now. I will take a look.
By any chance, could you provide an SSCCE so I can easily reproduce the issue?

I have been able to reproduce the issue.
Here is an SSCCE:
use coffee::graphics::{
    Color, Frame, HorizontalAlignment, Mesh, Point, Shape, Window,
    WindowSettings,
};
use coffee::load::Task;
use coffee::ui::{
    slider, Column, Element, Justify, Renderer, Slider, Text, UserInterface,
};
use coffee::{Game, Result, Timer};

fn main() -> Result<()> {
    <Lines as UserInterface>::run(WindowSettings {
        title: String::from("Lines - Coffee"),
        size: (1280, 1024),
        resizable: false,
        fullscreen: false,
    })
}

struct Lines {
    mesh: Mesh,
    amount: u32,
    slider: slider::State,
}

impl Game for Lines {
    type Input = ();
    type LoadingScreen = ();

    fn load(_window: &Window) -> Task<Self> {
        Task::succeed(|| Lines {
            mesh: Mesh::new(),
            amount: 0,
            slider: slider::State::new(),
        })
    }

    fn draw(&mut self, frame: &mut Frame<'_>, _timer: &Timer) {
        frame.clear(Color::BLACK);

        self.mesh.draw(&mut frame.as_target());
    }
}

enum Message {
    AmountChanged(f32),
}

impl UserInterface for Lines {
    type Message = Message;
    type Renderer = Renderer;

    fn react(&mut self, message: Message, window: &mut Window) {
        match message {
            Message::AmountChanged(amount) => {
                self.mesh = Mesh::new();
                self.amount = amount.round() as u32;

                for i in 0..=self.amount {
                    let x = (i % window.width() as u32) as f32;
                    let y = (i / window.width() as u32) as f32;

                    self.mesh.stroke(
                        Shape::Polyline {
                            points: vec![
                                Point::new(x, y),
                                Point::new(x + 3.0, y),
                            ],
                        },
                        Color::WHITE,
                        2.0,
                    );
                }
            }
        }
    }

    fn layout(&mut self, window: &Window) -> Element<Message> {
        Column::new()
            .width(window.width() as u32)
            .height(window.height() as u32)
            .justify_content(Justify::End)
            .padding(50)
            .spacing(20)
            .push(Slider::new(
                &mut self.slider,
                0.0..=20_000.0,
                self.amount as f32,
                Message::AmountChanged,
            ))
            .push(
                Text::new(&self.amount.to_string())
                    .horizontal_alignment(HorizontalAlignment::Center),
            )
            .into()
    }
}

The example renders lines to fill the screen. As you described, the mesh strops drawing after reaching ~16,400 lines.

I have found the issue:
Mesh is currently using an index buffer of u16 values. Each polyline has 4 vertices, so the total amount of polylines supported by the current Mesh implementation is 2**16 / 4 = 16384.
The most simple fix is to promote the indices to u32.

Great thanks! I assume this wouldn't impact performance in any meaningful way?

It's hard to say without actual benchmarks (we should tackle #13 soon), but my intuition says the impact should be mostly negligible for most workloads/platforms.
In any case, if performance is actually noticeably worse, we could try to improve it by using different strategies (automatically batching huge meshes, etc.) without any breaking changes.


ContentDialog visual update

Updating the brushes, animations, layout logic

Could you post a Light and Dark image showing the new visual style?
Also what do your layout logic changes mean for developers?

Could you post a Light and Dark image showing the new visual style?
Also what do your layout logic changes mean for developers?

Could you post a Light and Dark image showing the new visual style?
Also what do your layout logic changes mean for developers?

Added screenshots.
The logic changes nothing for the developers. Just makes 2-3 column layout logic cleaner with additional fixed columns for spacing, which is stored as a resource and can be easily updated in one spot should we need this later.

Could you post a Light and Dark image showing the new visual style?
Also what do your layout logic changes mean for developers?

Added screenshots.
The logic changes nothing for the developers. Just makes 2-3 column layout logic cleaner with additional fixed columns for spacing, which is stored as a resource and can be easily updated in one spot should we need this later.

Could you post a Light and Dark image showing the new visual style?
Also what do your layout logic changes mean for developers?

Added screenshots.
The logic changes nothing for the developers. Just makes 2-3 column layout logic cleaner with additional fixed columns for spacing, which is stored as a resource and can be easily updated in one spot should we need this later.

Thank you for the visuals, that bottom darker bar with the buttons makes a huge difference (design and aesthetic wise)
Has some stuff in common with the Win32 Dialogs too.

Could you post a Light and Dark image showing the new visual style?
Also what do your layout logic changes mean for developers?

Added screenshots.
The logic changes nothing for the developers. Just makes 2-3 column layout logic cleaner with additional fixed columns for spacing, which is stored as a resource and can be easily updated in one spot should we need this later.

Thank you for the visuals, that bottom darker bar with the buttons makes a huge difference (design and aesthetic wise)
Has some stuff in common with the Win32 Dialogs too.

/azp run

/azp run

Currently increasing the width of the dialog also increases the width of the button. Say, at the width 1000 for content dialog, a single button looks odd (too stretched). Are we going to look into that too here?

Currently increasing the width of the dialog also increases the width of the button. Say, at the width 1000 for content dialog, a single button looks odd (too stretched). Are we going to look into that too here?

Currently increasing the width of the dialog also increases the width of the button. Say, at the width 1000 for content dialog, a single button looks odd (too stretched). Are we going to look into that too here?

This will be a separate bug as it's not a regression. @stmoy is there an open issue at the moment?

Currently increasing the width of the dialog also increases the width of the button. Say, at the width 1000 for content dialog, a single button looks odd (too stretched). Are we going to look into that too here?

This will be a separate bug as it's not a regression. @stmoy is there an open issue at the moment?

/azp run

/azp run

Currently increasing the width of the dialog also increases the width of the button. Say, at the width 1000 for content dialog, a single button looks odd (too stretched). Are we going to look into that too here?

This will be a separate bug as it's not a regression. @stmoy is there an open issue at the moment?

This could be done as a behaviour on the ContentDialog.  To have the bottom buttons stretch to fill the width, or  to align to the right (or the left with Right To Left flow)

Currently increasing the width of the dialog also increases the width of the button. Say, at the width 1000 for content dialog, a single button looks odd (too stretched). Are we going to look into that too here?

This will be a separate bug as it's not a regression. @stmoy is there an open issue at the moment?

This could be done as a behaviour on the ContentDialog.  To have the bottom buttons stretch to fill the width, or  to align to the right (or the left with Right To Left flow)

Currently increasing the width of the dialog also increases the width of the button. Say, at the width 1000 for content dialog, a single button looks odd (too stretched). Are we going to look into that too here?

This will be a separate bug as it's not a regression. @stmoy is there an open issue at the moment?

This is "by design" so no bug is filed - the main reason is that ContentDialogs are expected to always have at least two buttons link. In that context, it's not a huge deal that a single button dialog looks strange because there shouldn't be any single button dialogs.
That said, @mdtauk posted a reasonable one-button dialog so it might be reasonable to reassess the guidance. This should be reassessed separately from this PR though.

Currently increasing the width of the dialog also increases the width of the button. Say, at the width 1000 for content dialog, a single button looks odd (too stretched). Are we going to look into that too here?

This will be a separate bug as it's not a regression. @stmoy is there an open issue at the moment?

This is "by design" so no bug is filed - the main reason is that ContentDialogs are expected to always have at least two buttons link. In that context, it's not a huge deal that a single button dialog looks strange because there shouldn't be any single button dialogs.
That said, @mdtauk posted a reasonable one-button dialog so it might be reasonable to reassess the guidance. This should be reassessed separately from this PR though.

Currently increasing the width of the dialog also increases the width of the button. Say, at the width 1000 for content dialog, a single button looks odd (too stretched). Are we going to look into that too here?

This will be a separate bug as it's not a regression. @stmoy is there an open issue at the moment?

This is "by design" so no bug is filed - the main reason is that ContentDialogs are expected to always have at least two buttons link. In that context, it's not a huge deal that a single button dialog looks strange because there shouldn't be any single button dialogs.
That said, @mdtauk posted a reasonable one-button dialog so it might be reasonable to reassess the guidance. This should be reassessed separately from this PR though.

What about a single button, but also beside it, a Hyperlink Button?

Currently increasing the width of the dialog also increases the width of the button. Say, at the width 1000 for content dialog, a single button looks odd (too stretched). Are we going to look into that too here?

This will be a separate bug as it's not a regression. @stmoy is there an open issue at the moment?

This is "by design" so no bug is filed - the main reason is that ContentDialogs are expected to always have at least two buttons link. In that context, it's not a huge deal that a single button dialog looks strange because there shouldn't be any single button dialogs.
That said, @mdtauk posted a reasonable one-button dialog so it might be reasonable to reassess the guidance. This should be reassessed separately from this PR though.

What about a single button, but also beside it, a Hyperlink Button?

@tashatitova  There is possibly an issue with the new ContentDialog design.  The bottom part of the dialog, with the buttons in, seems to darken whilst the dialog transitions on and off screen.

Stills from a screen recording

@tashatitova  There is possibly an issue with the new ContentDialog design.  The bottom part of the dialog, with the buttons in, seems to darken whilst the dialog transitions on and off screen.

Stills from a screen recording


environment: add qt6 plugin prefix to profileRelativeSessionVariables

Description of changes
This helps qt 6 to find plugins (imput methods, for example) installed in profiles. We patched qt 5 to derive plugin paths from PATH for the same thing.

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

A better approach might be adding this to the qt app wrapper, together with the removal of qt 4, would eliminate QT_PLUGIN_PATH from the global variables.

cc @milahu @ttuegel

This was a horrible mistake that has caused nothing but pain. We should make sure that Qt 6 packages are pure.

However input methods is a thing crucial to those who need it, and without a unified API for them, Qt plugins is the only way to make them work. Qt do have support for ibus bundled, but for the rest like fcitx5, there's no such luck. Another choice can be adding these as a dep of wrapQtAppsHook, given that they normally have a small closure size.

However input methods is a thing crucial to those who need it, and without a unified API for them, Qt plugins is the only way to make them work.

Input methods won't even work as plugins: every time there is a new Qt release, this will break every installed program. As a temporary measure to get fcitx working, you could add that plugin only to QT_PLUGIN_PATH when fcitx is enabled.

Closing in favor of https://github.com/NixOS/nixpkgs/pull/174218 as a temporary fix.


Instructions to build from source on MacOS using MacPorts

Summary of building c3c with llvm-17 and clang-17 ports from MacPorts based on https://github.com/c3lang/c3c/discussions/1701

Thank you


x86: mitigate L1 Terminal Fault vulnerability

During speculative execution, non-present pages are treated
as valid, which may expose their contents through side
channels.
Any non-present PTE will not have its address bits zeroed,
such that any speculative reads to them will go to the NULL
page.
Signed-off-by: Andrew Boie<EMAIL_ADDRESS>

Codecov Report

Merging #13999 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #13999      +/-   ##
==========================================
- Coverage   52.27%   52.23%   -0.04%
==========================================
  Files         307      307
  Lines       45447    45447
  Branches    10514    10514
==========================================
- Hits        23757    23741      -16
- Misses      16900    16912      +12
- Partials     4790     4794       +4

Impacted Files
Coverage Δ

subsys/testsuite/ztest/include/ztest_assert.h
38.88% <0%> (-38.89%)
:arrow_down:

subsys/testsuite/ztest/src/ztest.c
65.28% <0%> (-7.44%)
:arrow_down:

Continue to review full report at Codecov.

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

Looks simple. Seems like you could take the kconfig out, though. The performance won't be meaningfully different (it's just changing masks in an existing bitwise operation), and it guarantees that we'll never goof and try to use non-present-but-unzeroed PTEs as a design point in a future VM system or something.

Good idea!


Migrate the IDR information to the server-setup-examples section

Point to the IDR deployment page for more information about the architecture
--depends-on https://github.com/IDR/idr.openmicroscopy.org/pull/148

:+1: much more understandable, thank you. Happy to go with it.
Likely out of scope:

I'm unsure of the ordering logic for the page
If we ever get to the point that we have screenshots on this page, I could see re-generating the IDR metrics table from source ;)

Merging but I concur there is likely some restructuration to be done around this page/content especially if we had more examples. If there were enough exampes, I could probably imagine this metadata made available via a map representation with the different sites and their specifications.


Web Container to have location access

Hi, i have a single page web app, that i want to load in a web container. So far, it loads it well.
Now, some of the page needs to access location data, but for some reason the webview doesn't ask me to allow/block the location access, and rejects it immediately.
i have added some permissions but still doesn't seem to work
    <uses-permission android:name="android.permission.ACCESS_GPS" />
    <uses-permission android:name="android.permission.ACCESS_ASSISTED_GPS" />
    <uses-permission android:name="android.permission.ACCESS_LOCATION" />
    <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />
    <uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION" />
    <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION" />

Might be related to this but i'm not sure how or where to inject the android code
Can anyone give me some leads on how to do this properly? Will appreciate any feedback and thanks in advance!

Normally you could use and agent https://jasonelle.com/jasonette-old-docs/agents/
to fetch the location using the native apis and then inject the result in your webview.

I, along with another friend, are having the exact same issue and it is really frustrating. we think we've narrowed it down to a function called "onRequestPermissionsResult" in the app>src>main>com.jasonette.seed>Core>JasonViewActivity.java file. it seems like it has a line of code in the if statement for camera but nothing for location/gps. we don't know if this is the source of the issue for sure or what lines to add after the cameraManager one.

https://github.com/jasonelle/jasonette-android/pull/19 maybe that pr helps


pre-emptive 0.6 compatability fix

Not urgent

Since this package supports Julia 0.5, wherein isa isn't parsed as an infix operator, that conditional should probably also have a version check. That is, something like
!(¬t in ops || ¬t === :in || (VERSION >= v"0.6.0-dev.whatever" && ¬t === :isa))

It might work like you have it in 0.5 but I'm not sure.
Would also be good to add a test if possible.

Will do, how can i find the whatever in v"0.6.0-dev.whatever from the commit code?

See https://github.com/JuliaLang/Compat.jl#developer-tips

Cheers!

Any plan for merging this?


json_rpc_2 is swallowing vmservice's exceptions

See https://github.com/flutter/flutter/pull/28530 and https://github.com/flutter/flutter/issues/28374
The exceptions are currently swallowed in https://github.com/dart-lang/json_rpc_2/blob/master/lib/src/server.dart#L178

json_rpc_2 v 2.1.0 has been published and should enable us to fail on these exceptions now (or at least print an error about them).

json_rpc_2 v 2.1.0 has been published and should enable us to fail on these exceptions now (or at least print an error about them).


add $columnsMs macro,

fix https://github.com/Altinity/clickhouse-grafana/issues/430

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

54 of 70 (77.14%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 75.945%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/datasource/scanner/scanner.ts
2
4
50.0%

src/datasource/sql-query/sql-query-macros.ts
17
20
85.0%

pkg/eval_query.go
35
46
76.09%

Totals

Change from base Build<PHONE_NUMBER>1:
-0.2%

Covered Lines:
3526

Relevant Lines:
4436

💛  - Coveralls


Consider using TypeScript

TypeScript is powerful, well-tested, and well-typed, and could take this library to the next level for many developers. I'd be happy to open a PR if you'd like me to do so (just let me know!)

Hey, I appreciate your interest, but this library should just remain in basic maintenance mode at this point. Not going to do any big build changes or even publish new versions (unless bugs are found). Nothing against typescript though; it's nice.


”Failed to ingest your data“

I am trying to use the local vector database chroma, but I have encountered many problems. Can anyone give me some advice?

creating vector store...
error RequiredError: Required parameter collectionName was null or undefined when calling getCollection.
at Object.getCollection (/Users/jinyi/gpt4-pdf-chatbot-langchain/node_modules/chromadb/dist/module/generated/api.js:240:23)
at Object.getCollection (/Users/jinyi/gpt4-pdf-chatbot-langchain/node_modules/chromadb/dist/module/generated/api.js:723:78)
at ApiApi.getCollection (/Users/jinyi/gpt4-pdf-chatbot-langchain/node_modules/chromadb/dist/module/generated/api.js:1045:45)
at ChromaClient.getCollection (/Users/jinyi/gpt4-pdf-chatbot-langchain/node_modules/chromadb/dist/module/ChromaClient.js:176:14)
at Chroma.addVectors (/Users/jinyi/gpt4-pdf-chatbot-langchain/node_modules/langchain/dist/vectorstores/chroma.js:76:45)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at Chroma.addDocuments (/Users/jinyi/gpt4-pdf-chatbot-langchain/node_modules/langchain/dist/vectorstores/chroma.js:39:9)
at Function.fromDocuments (/Users/jinyi/gpt4-pdf-chatbot-langchain/node_modules/langchain/dist/vectorstores/chroma.js:120:9)
at run (/Users/jinyi/gpt4-pdf-chatbot-langchain/scripts/ingest-data.ts:42:7)
at  (/Users/jinyi/gpt4-pdf-chatbot-langchain/scripts/ingest-data.ts:56:3) {
field: 'collectionName'
}
/Users/jinyi/gpt4-pdf-chatbot-langchain/scripts/ingest-data.ts:51
throw new Error('Failed to ingest your data');
^
Error: Failed to ingest your data
at run (/Users/jinyi/gpt4-pdf-chatbot-langchain/scripts/ingest-data.ts:51:11)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at  (/Users/jinyi/gpt4-pdf-chatbot-langchain/scripts/ingest-data.ts:56:3)

I'm running into the same issue. Pinecone env variables are set, had to set PINECONE_NAME_SPACE to PINECONE_NAME_SPACE="" to get to this point.

I'm running into the same issue. Pinecone env variables are set, had to set PINECONE_NAME_SPACE to PINECONE_NAME_SPACE="" to get to this point.
I'm also seeing this error:
error [PineconeError: PineconeClient: Error calling upsert: PineconeError: metadata size is 68702 bytes, which exceeds the limit of 40960 bytes per vector] /Users/icebearlabs/projects/private/abschlussarbeit-research-bot/scripts/ingest-data.ts:46 throw new Error('Failed to ingest your data');

Indeed, the code will display "Failed to ingest your data" whenever it comes across any pertinent issues. Based on your log information, I believe that the root of our errors differs.

For me the solution was to get a paid pinecone account. With the free starter, it just doesn't work.


Does the plugin work properly in godot 4

It does not right now, but I have local work to make it happen pretty soon :)

thanks à toi baba des bois ;)

mais ducoup, je suis obligé d'utiliser l'animation tree :'(


Provide an option to use Alpha to Coverage (aka A2C) when MSAA is enabled

The inclusion of this feature would effectively allow alpha-tested materials to appear anti-aliased. This is especially useful for foliage, and when used carefully, for hair. Considering how modern realtime hair is made, it's extremely difficult to avoid collisions and alpha sort correctly, and a two-pass system (rendering alpha-test first, rendering the remainder as alpha-sorted afterward) can introduce unwanted artifacts.
A robust A2C implementation effectively allows us to circumvent these issues and avoid the performance impact associated with alpha sorting multiple layers of geometry. While a quick, straightforward implementation would be wonderful, this article by Ben Golus notes some additional considerations that could elevate the usefulness of an implementation (namely 'sharpening' the result and preventing mipmaps from discarding useful details).
This is one of very few remaining obstacles to using state of the art character rendering techniques in Godot, not to mention improving other areas such as environment art. Given Godot's robust implementation of forward rendering, adding this technique to the native MSAA support should be achievable and provide additional motivation to utilize it.

Here are some images of a basic character using a barebones implementation of the technique in the Blender Game Engine, including a zoomed version of the results at some distance so you can appreciate the subpixel variation. Keep in mind that, with a less orderly-looking alpha for the hair cards and the sharpening from the previous article, the difference in quality would be more apparent, so this isn't necessarily a perfect example of the benefits.

Yes id love to see this! I believe it also helps with sorting issues vs traditional alpha?

@Gibbz Yes, as the article linked points out:

Okay, so with 4x MSAA and Alpha to Coverage you get 5 levels of opacity you can play with, 4/4 through 0/4 samples. What good is that? With traditional alpha blending you get at least 256 levels, so why would you ever want to use this? Because with Alpha to Coverage those coverage samples are opaque. That means they can be depth sorted! You can have a mesh of polygon soup and have intersecting triangles accurately sort at better than per pixel accuracy. This is no different than opaque objects or even alpha test with MSAA, but you also get soft edges.

@Calinou  It can be, but transparent objects go to the transparent pipeline and causes all kinds of sorting issues (example in the article). So avoiding the alpha blending completely gives us no issues with sorting, but allows pretty nice edges. The main issue is, that we have to have MSAA enabled for this. It's used by a lot of AAA games, but probably isn't very popular feature request due to lack of AAA games being developed in Godot.

Regarding use cases / motivation for this feature. On mobile tile-basec graphics chips this is probably higher performance then blending and especially for mobile VR it is almost essential for good quality: https://developer.oculus.com/blog/common-rendering-mistakes-how-to-find-them-and-how-to-fix-them/

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!

Note that this is being implemented in https://github.com/godotengine/godot/pull/40364.


pre-commit: add mypy

…while replacing the call also in Makefile

I'll cc @mvo5 on this as we are the mypy fans. I think we wanted to keep mypy optional so not adding it as a required check (this will make it run in ci as well right?) is probably better.


Export NotesDisplay from @medplum/react

This will add NotesDisplay as an exported member of the medplum react package.

Fixes #3663


Upgrade Meilisearch project to .NET Standard 2.1

Pull Request
What does this PR do?
Fixes #221
Upgrade to .NET Standard 2.1
PR checklist
Please check if your PR fulfills the following requirements:

[X] Does this PR fix an existing issue?
[X] Have you read the contributing guidelines?
[X] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to MeiliSearch!

bors merge


password clears on /account/edit save if left blank

the user password clears if the two password input fields are left blank on account save

What do you mean by clears?
If you mean that the password fields are blank, then that's intention. WordPress doesn't store a plaintext password to put in the value and generally you don't expose sensitive information like that in a form field anyway.
Or do you mean the user password is changed?

Sorry, I should have been more descriptive: the user password is changed

From what's going on in the DB, save with blank password input fields sets the password to a completely different value (hash in wp_users table changes), so the user is forced to reset if if he/she signs out.

Thanks, I'll take a look tonight

Sure, I'm digging into Accounts.php now. will let you know if I see anything funky

Here's the password change stuff. Pull requests are always welcome if you find a fix :+1:

Another bug left on edit profile.

Fixed in 8d4bdbe7872b495e920121e0f71d9562b8e0879d


POST broadcasting/auth 403 (Forbidden)

I use private channel, and got error like this:
POST http://<IP_ADDRESS>:8000/broadcasting/auth 403 (Forbidden)
Pusher :  : ["Unable to retrieve auth string from auth endpoint - received status 403 from /broadcasting/auth. Clients must be authenticated to join private or presence channels
please help me

Without more info it just looks like the channel was not authorized correctly.

https://laravel.com/docs/7.x/broadcasting#authorizing-channels

Remember that when you define an authorizer like:
    return $user->id === Order::findOrNew($orderId)->user_id;
});

You do not need to prefix the private- part to the channel name (that might be causing confusion).

In my case it throws 403 forbidden altough i'm dealing with a public channel  .


TypeError: undefined is not an object (evaluating 'dataset.color')

Hi,
I'm just trying to create a line chart using this library but getting following error. The example I'm trying is copied straight from the README document as follows,
My environment:
Platform: Mac OS (iOS)
node: 10.17.0
react-native-cli: 2.0.1
$ npm install --save react-native-chart-kit
$ npm install --save react-native-svg prop-types
$ cd ios; pod install (Was this needed?)
...
Downloading dependencies
Installing RNSVG (9.13.6)
..
Source code:
render() {
  const chartConfig = {
  backgroundGradientFrom: "#1E2923",
  backgroundGradientFromOpacity: 0,
  backgroundGradientTo: "#08130D",
  backgroundGradientToOpacity: 0.5,
  color: ( opacity = 1 ) => `rgba(26, 255, 146, ${opacity})`,
  strokeWidth: 2, // optional, default 3
  barPercentage: 0.5
};

const data = {
  labels: [ "January", "February", "March", "April", "May", "June" ],
  datasets: [
    {
	data: [ 20, 45, 28, 80, 99, 43 ],
	color: ( opacity = 1 ) => `rgba(134, 65, 244, ${opacity})`, // optional
	strokeWidth: 2 // optional
    }
  ],
  legend: [ "Rainy Days", "Sunny Days", "Snowy Days" ] // optional
  };
}
return (
  <View style={styles.container}>
    <Text>Bezier Line Chart</Text>
    <LineChart
	data={data}
	width={screenWidth}
	height={256}
	verticalLabelRotation={30}
	chartConfig={chartConfig}
	bezier
    />
  </View>
)

Any help is appreciated. TIA.

Try to remove legend param, it worked for me.
legend: [] // optional

@kythanh Thanks. With suggested change, I'm getting different error,
Invariant Violation: [5815,"RNSVGLine",11,{"matrix":[1,0,0,1,0,0],"propList":["stroke","strokeWidth","strokeDasharray"],"opacity":1,"fill":[0,4278190080],"fillRule":1,"fillOpacity":1,"stroke":[0,857407378],"strokeOpacity":1,"strokeLinecap":0,"strokeLinejoin":0,"strokeDasharray":["5","10"],"strokeWidth":1,"strokeDashoffset":null,"strokeMiterlimit":4,"vectorEffect":0,"x1":"<>","y1":0,"x2":"<>","y2":208}] is not usable as a native method argument
This error is located at:
in RNSVGLine (at Line.tsx:26)
in Line (at abstract-chart.js:201)
in RNSVGGroup (at G.tsx:28)
in G (at line-chart.js:348)
in RNSVGGroup (at G.tsx:28)
in G (at line-chart.js:315)
in RNSVGGroup (at G.tsx:28)
in G (at Svg.tsx:170)
in RNSVGSvgView (at Svg.tsx:157)
in Svg (at line-chart.js:302)
in RCTView (at line-chart.js:301)
in LineChart (at VDetail.js:225)

@kythanh After cleaning up project directory, the example worked. Thanks,


how to generate Path signature?

Hello, I wonder which py file or function in py file can generate Path signature for JHMDB dataset or my own dataset? Did I miss any details in your py file?

Hi @XinruiYuan
An example for this is given in the readme (https://github.com/kschlegel/PSFDataset#psfdataset-class), let me try to clarify this:
Lets assume you are working with JHMDB and you have an iterator that at each iteration will give you the keypoint array of one of the videos (of the form [frame, 15, 2] for the 15 keypoints in 2D in JHMDB) and the action label of the video. Now you can define the transformations you want to apply to this data:
transform = transforms.Compose([
    transforms.SpatioTemporalPath(disintegrate=False),
    transforms.temporal.DyadicPathSignatures(dyadic_levels=0,
                                             signature_level=3)
])

This example would take the signature of level 3 of your 30 (=15*2) dimensional path. To generate your dataset you would now do
dataset = PSFDataset(transform=transform)
for keypoints,label in JHMDB:
        # keypoints is a numpy array of shape [num_frames,15,2]
        # label is the integer action class label for this video
	dataset.add_element(keypoints, label)

If you want more than just the signature of the entire path you can add more transformations. The paper linked in the readme describes a lot of transformations you might want to consider and why. All transformations described in the paper are implemented here and have in-code documentation (I will hopefully create a separate documentation at some point).
Hope this helps, let me know if you have any further questions.


[SPARK-34074][SQL][TESTS][FOLLOWUP] Fix table size parsing from statistics

What changes were proposed in this pull request?
Fix table size parsing from the Statistics field which is formed at: https://github.com/apache/spark/blob/c3d81fbe79014f693cf93c02e31af401727761d7/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala#L573 . Before the fix, getTableSize() returns only the last digit. This works for Hive table in the tests because its size < 10 bytes, and accidentally works for V1 In-Memory catalog table in the tests.
Why are the changes needed?
This makes tests more reliable. For example, the parsing can not work in AlterTableDropPartitionSuite when table size before partition dropping:
+---------+
|data_type|
+---------+
|878 bytes|
+---------+

After:
+---------+
|data_type|
+---------+
|439 bytes|
+---------+

at:
        val onePartSize = getTableSize(t)
        assert(0 < onePartSize && onePartSize < twoPartSize)

Does this PR introduce any user-facing change?
No
How was this patch tested?
By existing test suites:
$ build/sbt -Phive-2.3 -Phive-thriftserver "test:testOnly *.AlterTableAddPartitionSuite"
$ build/sbt -Phive-2.3 -Phive-thriftserver "test:testOnly *.AlterTableDropPartitionSuite"

Test build #134206 has started for PR 31237 at commit e91bb5a.

Test build #134206 has started for PR 31237 at commit e91bb5a.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38791/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38791/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38791/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38791/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38791/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38791/

Test build #134206 has finished for PR 31237 at commit e91bb5a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #134206 has finished for PR 31237 at commit e91bb5a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/134206/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/134206/


lid dictionary 1024 is not downloadable - permissions?

🐛 Bug
At page https://github.com/facebookresearch/fairseq/tree/main/examples/mms
there is a table under heading LID that has a matrix of downloadable items.
The entry for row 1024, column Dictionary is not downloadable.
To Reproduce
Go to page https://github.com/facebookresearch/fairseq/tree/main/examples/mms
Right-click it, select 'Save As' from the pop-up menu, and choose a filename.
The download fails.
If you left-click on it (rather than right-click, 'save as'), it returns the following:

AccessDenied
Access Denied
KAXWZ1HRVXJ5GFWY
SVDZwLefcs6cPXqxIOYB4aA7y+CrL2lybWtOveXVd3tCjewZw77YzUKvwEwlR0uQwvpyNixpzv0=

Code sample
see above
Expected behavior
I expected to be able to download the file, as works with the other entries in column Dictionary.
Environment
Chrome Version 113.0.5672.93 (Official Build) (64-bit)
Windows 10 22H2 v 19045.2965
Additional context

Fixed now!


Charts are getting cutoff at the right end

The bars on the right most corner are cutoff (see next image for the correct number of bars) - Dec 23 data is cut off at the right side of the chart -

After manually adding an empty row for an extra month, to create extra space on the right side -

If it helps - Viewing this on a standard 27" 1140p monitor, maximized window as well.

Thanks for reporting this !
Can you please provide a small sql file that reproduces this ?

Ok, I can reproduce it with the following SQLite snippet:
select
    'chart'    as component,
    'bar'      as type,
    TRUE       as toolbar,
    TRUE       as time;

with recursive integers(i) as (
    select 1 as i
    union all
    select i + 1
    from integers
    where i < 100
)
select
    'S' || (i%10) as series,
    format('%d-01-01', 2010 + (i/10))     as x,
    (random() % 10)   as value
from integers;

I reported the issue upstream, since the bug seems to be in apexcharts, not in SQLPage itself: https://github.com/apexcharts/apexcharts.js/issues/4178

I have similar problems with long series.
I want series as stacked values for each classes.
It's ok for 1st et 2nd and after it's horrible. Some classes are a lot of times on x axys.
For the 3rd, instead of the "34" we have the "41" !

There is the code :
`-- Différents Dispositifs en place par Classe
select
'chart'               as component,
'Différents dispositifs par classe' as title,
'bar'             as type,
500 as height,
TRUE as labels,
TRUE as stacked,
'Classes' as xtitle,
'Nombre de dispositifs' as ytitle;
select
Nom_dispositif as series,
classe as x,
coalesce(Nombre,0) as value
FROM stats01 WHERE etab=$id ORDER BY classe;
This is the beginning of the debug with "34" as 3rd position... Debug output
{"component":"debug","title":"Différents dispositifs par classe","type":"bar","height":500,"labels":true,"stacked":true,"xtitle":"Classes","ytitle":"Nombre de dispositifs"}
{"series":"SEGPA","x":"31","value":5}
{"series":"UEE","x":"31","value":2}
{"series":"ULIS","x":"31","value":2}
{"series":"Mat. Pédagogique","x":"33","value":2}
{"series":"PAI","x":"33","value":1}
{"series":"PIAL","x":"33","value":2}
{"series":"PPRE","x":"33","value":2}
{"series":"SEGPA","x":"33","value":1}
{"series":"ULIS","x":"33","value":2}
{"series":"FLS","x":"34","value":3}
{"series":"Mat. Pédagogique","x":"34","value":1}`

Is this a different issue than https://github.com/lovasoa/SQLpage/issues/172 ? If it's the same problem, I don't think we need to track it twice. It has already been reported to the graph library upstream: https://github.com/apexcharts/apexcharts.js/issues/4178

ApexCharts just merged a fix for this issue. It will be included in SQLPage's next version :confetti_ball:


⚠ Rumah123 has degraded performance

In fa66953, Rumah123 (https://www.rumah123.com) experienced degraded performance:

HTTP code: 504
Response time: 20970 ms

Resolved: Rumah123 performance has improved in 2169b2f.


Still no audio... help please ASAP.

I've installed all the one liner scripts all using this  2018-03-13-raspbian-stretch.zip
but getting the same thing:
$ cat /proc/asound/modules
0 snd_bcm2835
1 snd_usb_audio
Number 1 is missing.
Please can someone tell me exactly why the file is missing?
Is there any chance (with telling me exactly what to do) to get the file put on there?
Unless it's meant to be like that seeing as it's just using a Mono USB Speaker???
Thank you.
Do I need to have the USB mono speakers connected when installing the Phoniebox?
Is that why it's missing?
By the way, I am using a powerbank if it's any help- https://www.amazon.co.uk/Anker-PowerCore-Portable-Double-Speed-Recharging/dp/B01JIWQPMW/ref=sr_1_55?keywords=power+bank&qid=1566401090&s=gateway&sr=8-55
Though, that powers everything fine, as the web app loads, USB card reader loads...
I've also looked at the troubleshooting.
RPi 3 Mobel B+ is the model... nothing extra fitted.
Just using a mono USB powered speaker, nothing else.
Clicking on one of the available audios, it plays for around 15 seconds then stop. 0% sound.
In headphones I get the "mmm" sound, also hear the startup sound. No audio from the streams.
Streams 100% work in VLC player & browser.
No Spotify installed as don't have it.
Would it help if I changed
echo "PCM" > /home/pi/RPi-Jukebox-RFID/settings/Audio_iFace_Name
from "PCM" to Speaker? Considering it's just one USB mono speaker.
INFO from that:
Simple mixer control 'PCM', 0
Capabilities: pvolume pvolume-joined pswitch pswitch-joined
Playback channels: Mono
Limits: Playback -10239 - 400
Mono playback -7575 [25] [75.75db] [on]
Will not be using  GPIO buttons
I do the "start" and "enable" thing for them two when promoting in the instructions.
I've never done a manual install.

Issue solved!
It was the audio output I was meant to do.

Well that didn't work.
Tried this:
https://support.screenly.io/hc/en-us/articles/214547803-How-do-I-enable-analogue-audio-output-for-my-Raspberry-Pi-
But I don't see the "Manage Screen" anywhere.
Tried right clicking the sound icon and electing Analouge from HDMI and after rebooting, from headphones I didn't get the "mmm" sound and I heard the startup sound very clearly.
Though however when registering a card and swiping it it does NOT come up as the ID number where the "available audio" is, even though it did when I first installed it.
I am using the one liner for and classic version without spottily.
So why exactly is the ID number not coming up on the "Available Audio
Tried this -
Start - Perferances - Audio Device Settings - Select Controls - Tick PCM. OK. Swipe card, nothing.
BUT I had it with the audio still checked via the sound control at the top.... then I get the "mmm" sound and I can just hear the startup sound.
Then tried this, amixer cset numid=3 2
getting the "mmm" sound and no startup sound.
Using a 3.5mm jack via the headphones.
Please, please, please can someone tell me exactly what is wrong?
I have been trying to get this working every day since last Wednesday.

Well I done this
Start - Perferances - Audio Device Settings - Select Controls - Tick PCM
Rebooted then I still got the "mmm" noise in headphones then when selecting the analogue via the sound at the top right then rebooted I head no "mmm" and heard the startup sound very clearly.
Then when I unplugged the HDMI from the tv and turned it on again via the powerbank I get the "mmm" noise again and hear the startup sound.
I register and swipe a card and I still get NO audio.
Do I also need to change the settings to force it to play via 3.5mm jack as I am using headphones?

Hi @MarkIpswich
do you get any other audio on the Pi? Like playing YouTube? Or other audio files (outside of Phoniebox)?

Downloaded YouTube audio, swiped a card, and nothing. No audio

Playlist folder is there in correct directory.

Did you use the install script for buster on the stretch OS?

I tried them all and I still get the problem.
At the moment, using the classic version without spotify.

I just noticed that the Youtube folder, there is no files in there as it says in web app.

 I didn't mean downloading youtube in Phoniebox. I meant simply: without Phoniebox, is there any sound coming from your raspberry pi? Using the browser and go to youtube or play a file in an audio player?
And is there any chance that your Pi is offline? Can you visit websites? In other words: did you type in the right IP address and the like during the install procedure or Phoniebox?
The ERROR failed to decode is an issue with the MPD (music player daemon) and is not part of the Phoniebox problem you have. https://www.raspberrypi.org/forums/viewtopic.php?t=97755
Did you try the sample files delivered with Phoniebox?
https://github.com/MiczFlor/RPi-Jukebox-RFID/wiki/MANUAL#sample-content

Not I did not try the sample files.
When buring the image, I load the stretch, put the wifi details in, get the code and paste from browser. Then when rebooting, I can access the phoniebox.de site fine (takes a bit longer than usual) and any other site I can't. But before rebooting, I can.
Yes, all the IP address's are correct. Router IP, static IP... http://192.168.x.xx
Well I looked to see if I have a static IP address and it says dymantic.
But the web app via http://192.168.x.xx loads fine without any problems.

I just looked up the IP address via System Configuration and the last two numbers are different.
Would it be because of that?

Well yes I got it working through headphones and now trying for the second time.
No audio!

Hi @MarkIpswich
sorry to ask you this: but please do a fresh install :) I just made some changes to the install script.

Re: audio. You got it working through the headphones. Then you say "no audio!". Can you explain what you mean?
So you registered a card with a stream and that works. Ok, that's good. Then you say "I haven't done card ID with that one yet" - again, what do you mean?

To remove a folder in the terminal, type:
rm -rf /home/pi/RPi-Jukebox-RFID/shared/audiofolders/folderName

Replace folder name with your folder name. Note: if there are spaces in the folder name, you have to "escape" them. Like: Folder Name becomes Folder\ Name - you add a backslash before the space.
Editing a stream (hopefully doable in the web app soon). For now, in the terminal type
nano /home/pi/RPi-Jukebox-RFID/shared/audiofolders/folderName/livestream.txt

I mean I put the web stream link in (Sunshine Live) then tested it via headphones. 100% works.
Then I had to go out so about an hour later or so, it doesn't play.
I mean for Sunshine Live, I haven't made it available yet for swiping. Haven't set it up that way yet.

Ok, but the Phoniebox is online, you can for example
ping phoniebox.de

?

Yes. I know that you can use the terminal on PC.
Is there anyway with doing it without a enternal cable?

Yet another problem.
I removed the folder as suggested, now I can't change the volume in web app. Just stays at 0% when setting it.
Help please.


Modularize reusable style workflow

This removes the logic from lint.yaml which is only for this repo, to reusable_style.yaml and modularizes out the logic into composite actions.
A follow up will be made to combine the old reusable style.yaml workflow with this new one.

Seems like the github action linter does not like composite actions with inputs :/

yeah, https://github.com/rhysd/actionlint has many changes since last version has been cut

/lgtm


Lowercase "a" in PMD desktop page banner.

The “A” in PMD should be lowercase in the desktop page banner.  Probably will make this more generic anyway...
Trivial change to the text on this page (although we will want to make this support more generic as we support multiple organization).

Fixed this typo across the source in general.  In the end, these will likely be replaced by references to the database if that is possible...


[SPARK-6477][Build]: Run MIMA tests before the Spark test suite

This moves the MIMA checks to before the full Spark test suite such that, if new PR's fail the MIMA check, they will return much faster having not run the entire test suite. This is preferable to the current scenario where a user would have to wait until the entire test suite completes before realizing it failed on a MIMA check in which case, once the MIMA issues are fixed, the user would have to resubmit and rerun the full test suite again.

Agree, I like this one. Fail-fast checks should go first.

Test build #29031 has started for   PR 5145 at commit 12b0aee.

This patch merges cleanly.

Test build #29031 has finished for   PR 5145 at commit 12b0aee.

This patch passes all tests.

This patch merges cleanly.

This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29031/
Test PASSed.

/cc @pwendell as well for any thoughts

LGTM

LGTM too