Dask: @subflow-decorated functions break concurrency when WORKFLOW_ENGINE == "dask"

Details about the quacc environment

quacc version: 0.4.6
Python version: 3.9+

What is the issue?
When the Dask workflow engine is used, @subflow-containing workflows break concurrency. This is an upstream Dask bug, as described in https://github.com/dask/dask/issues/10733 and the link therein.
How can we easily reproduce the issue?
quacc set DEBUG true
quacc set WORKFLOW_ENGINE dask

from ase.build import bulk
from dask.distributed import Client

from quacc.recipes.emt.slabs import bulk_to_slabs_flow

client = Client()

atoms = bulk("Cu")
delayed = bulk_to_slabs_flow(atoms)
result = client.gather(client.compute(delayed))

Note how each relaxation runs sequentially rather than concurrently by monitoring stdout.

Here is the minimal example in pure Dask:
import time
from dask import delayed

@delayed
def relax_job(atoms):
    print("Sleeping")
    time.sleep(5)
    return True

def make_slabs_from_bulk(atoms):
    return [1] * 3

@delayed
def bulk_to_slabs_subflow(atoms, job):
    slabs = make_slabs_from_bulk(atoms)
    results = []
    for slab in slabs:
        result = job(slab)
        results.append(result)
    return results

def bulk_to_slabs_flow(atoms):
    return bulk_to_slabs_subflow(atoms, relax_job)

As noted by Zack, the following helps alleviate the concurrency problem but not for our specific case in quacc where Delayed objects are being passed around. This is much closer to the solution, however.
elif SETTINGS.WORKFLOW_ENGINE == "dask":
    from dask import delayed
    from dask.distributed import worker_client

    @wraps(_func)
    def wrapper(*args, **kwargs):
        with worker_client() as client:
            futures = client.compute(_func(*args, **kwargs))
            results = client.gather(futures)
            return results

    return delayed(wrapper)


torchpipe与torchgpipe有什么区别么？

torchpipe与torchgpipe有什么区别么？ 两个库名字很相似

torchgpipe主要是用于训练时的流水线并行；torchpipe用于推理服务的吞吐最优化，支持tensorrt等第三方加速后端，并且主要关注最简化的ensemble语法
torchgpipe is mainly used for pipeline parallelism during training; torchpipe is used for optimizing throughput of inference services, supporting third-party acceleration backends such as TensorRT, and mainly focuses on simplified ensemble syntax.

got it


The entry 'RVC1006Nvidia\run.sh cannot be extracted

When I extract the .7z file, the folder is deleted after the prompt message.
Error: The entry 'RVC1006Nvidia\run.sh cannot be extracted from the archive because it is an unsupported filt type.

I'm also having this exact issue.

When I extract the .7z file, the folder is deleted after the prompt message. Error: The entry 'RVC1006Nvidia\run.sh cannot be extracted from the archive because it is an unsupported filt type.

I tried to download the latest version of "7-zip" and I was able to unzip it successfully. I opened the archive using the "7zFM.exe" shortcut and clicked "Extract". It unpacked perfectly.


fix: Dynamic modal copy based on context

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix

Needs QA check

@kodadot/qa-guild please review

Context

[x] Closes #8134
[ ] Requires deployment <snek/rubick/worker>

Did your issue had any of the "$" label on it?

[x] My DOT address: Payout

Screenshot 📸

[ ] My fix has changed UI

Copilot Summary

🤖[deprecated] Generated by Copilot at fbb483f
This pull request enhances the user interface and localization of the auto-teleport feature for NFT transactions. It adds different confirmation messages for the action button, updates the localization file with new and improved strings, and adds fallback values for the modal dialog.

🤖[deprecated] Generated by Copilot at fbb483f

Sing, O Muse, of the skillful coder who devised
A wondrous feature for the auto-teleport action,
That lets the users swiftly move their NFTs across
The blockchain realms with ease and satisfaction.

listing - ok
minting item - ok
minting collection - ok
buying - ok
transfer nft - i don't think we have teleport enabled for it

Thanks!
pay 30 usd

😍 Perfect, I’ve sent the payout
💵 $30 @ 5.13 USD/DOT ~ 5.848 $DOT
🧗 16SjUbGKSdjCdWTy3NNT3JxbRVGGqD4mwkHpc6BD9U2Rp29Z
🔗 0x49ebbcab1275264ca396c4e5731937279771816baaab0d0b61393032480da9ea
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Server side only certs - openssl client - wrong certificate type

Your environment.

Version: 2.1.0
Browser: What?
tldr: Server side only certificates - using openssl dtls client - fails

What did you do?
I've created production letsencrypt certs that are valid.
I configure my listener like so:
certificate, err := util.LoadKeyAndCertificate(dtls_key, dtls_cert)
util.Check(err)

rootCertificate, err := util.LoadCertificate("getssl-docker/certs/isrgrootx1.pem")
util.Check(err)
certPool := x509.NewCertPool()
cert, err := x509.ParseCertificate(rootCertificate.Certificate[0])
util.Check(err)
certPool.AddCert(cert)

// Prepare the configuration of the DTLS connection
config := &dtls.Config{
	Certificates: []tls.Certificate{*certificate, *rootCertificate},
	// Create timeout context for accepted connection.
	ConnectContextMaker: func() (context.Context, func()) {
		return context.WithTimeout(ctx, 30*time.Second)
	},
}

Test my server (openssl 3.0.2)
openssl s_client -dtls1_2 -connect valid.dns.name.for.cert:port -CAfile lecerts/chain.pem -debug
Output:
Note the cert Verification: OK, yet the client expects a different certificate type.

CONNECTED(00000007)
depth=2 C = US, O = Internet Security Research Group, CN = ISRG Root X1
verify return:1
depth=1 C = US, O = Let's Encrypt, CN = R3
verify return:1
depth=0 CN = valid.dns.name.for.cert
verify return:1
0016900F01000000:error:0A00017F:SSL routines:tls_post_process_server_certificate:wrong certificate type:ssl/statem/statem_clnt.c:1914:
Certificate chain
0 s:CN = valid.dns.name.for.cert
i:C = US, O = Let's Encrypt, CN = R3
a:PKEY: rsaEncryption, 4096 (bit); sigalg: RSA-SHA256
v:NotBefore: Mar 31 04:25:40 2022 GMT; NotAfter: Jun 29 04:25:39 2022 GMT
no peer certificate available
No client certificate CA names sent
SSL handshake has read 2415 bytes and written 809 bytes
Verification: OK
New, (NONE), Cipher is (NONE)
Secure Renegotiation IS supported
Compression: NONE
Expansion: NONE
No ALPN negotiated
SSL-Session:
Protocol  : DTLSv1.2
Cipher    : 0000
Session-ID:
Session-ID-ctx:
Master-Key:
PSK identity: None
PSK identity hint: None
SRP username: None
Start Time:<PHONE_NUMBER>
Timeout   : 7200 (sec)
Verify return code: 0 (ok)
Extended master secret: no

What did you expect?
I expect a successful handshake
What happened?
The last packet sent in this handshake is from the openssl client; Alert - Fatal - Illegal Parameter (47)
The openssl client rejects the certificate as "wrong type". Here's the exact line of openssl code that's faulting https://github.com/openssl/openssl/blob/openssl-3.0.2/ssl/statem/statem_clnt.c#L1914

If you append a IP-Capture, there may be a chance, to see what fails.

You're loading your TLS key and the root CA separately, but that doesn't mean that Go will build the certificate chain for you. Now you just have 2 disjoint certs in your pool. The first one being offered is now a root CA, which your client really can't do anything with and probably errors out.
You don't want to do it this way server side. You need to create a cert bundle with your server cert + any intermediate CA (without the root CA, the client should have that one already because otherwise it can't verify anything). Concatenate those into a file, and load that instead.


Weak Password Requirements

It was observed the product does not require that users should have strong passwords, which makes it easier for attackers to compromise user accounts. We tried using “12345678” as a password and were able to create a user using signup api as well as able to change the password using change-password api.

Duplicate of #1130


kie-issues#1213: configure for dockerhub registry

Fixes:

apache/incubator-kie-issues#1213

Depends on docker.io/apache/incubator-kie-kogito-ci-build:main-latest being released first (after #1197 is merged and pipeline execution succeeds).
Part of PR group:

apache/incubator-kie-kogito-pipelines#1198
apache/incubator-kie-drools#5952
apache/incubator-kie-optaplanner#3087

@jstastny-cz I'm not sure why the dsl test is failing. Can you verify?

Those tests don't work well when my fork name is different than the actual repo :/

Tests passed when running locally with correct overrides. The test scripts would need much time investment, I might rather rename my forks :-D


Projection and redundant order by with Navigation propery

When using navigation property the generated sql query ignores the projection requested and bring all columns (for both tables).
Also an order by clause is added to the query - but I guess this maybe relates to https://github.com/aspnet/EntityFramework/issues/6861 right?
Example:
Models:
public class Player
{
        public string PlayerId { get; set; }

        public string Nickname { get; set; }

        public string CountryId { get; set; }
        public Country County { get; set; }
}

public class Country
{
        public string CountryId { get; set; }
        public string Name { get; set; }
}

The query::
var playerInfo =
                    (from p in ctx.Players
                        where p.PlayerId == "123"
                        select new
                        {
                            Player = p.PlayerId,
                            Name = p.County.Name
                        })
                    .FirstOrDefaultAsync().Result;

The SQL generated:
SELECT p.PlayerId, p.CountryId,  p.Nickname, p_County.CountryId, p_County.Name
FROM Players AS p
LEFT JOIN Countries AS p_County ON p.CountryId = p_County.CountryId
WHERE p.PlayerId = '123'
ORDER BY p.CountryId
LIMIT 1

Is this a known issue?

dupe of #6647

Thanks


Support Locale in Bar Chart

The Britecharts library introduced options to set the locale for the bar chart.
This enables overriding the standard number formatting and for example use different decimal separators and/or currency symbols in the axis labels and the bars labels.
Expected Behavior
Being able to change the locale, e.g. allow GBP or EUR sign be used for number formatting.
Difference from Current Behavior
No locale setting(s) available in the bar chart component.
Possible Implementation
Introduce a new property to pass the locale information as an object into Britecharts.
Context
Format monetary values e.g. as "1.3k Euro" or "1325 €" in the bar labels.
Link to relevant commit in Britecharts repo:
Dec 12, 2019 - 96c665753fd31f265f31a095f6fea95200fdaaa2

Included on the RC1: https://github.com/britecharts/britecharts-react/commits/release/1.0.0-rc1


Update README.md

Changed to the correct git repository.
Pull request checklist:

[ ] CHANGELOG.md was updated, if applicable

Description

Unnecessary diff


[torchvision] - MNIST datasets

Source: https://github.com/pytorch/vision/blob/master/torchvision/datasets/mnist.py
Using configen, create a subset of the torchvision datasets for MNIST. Pair with tests.
These should be along the same lines as @tkornuta-nvidia prototype in NeMo:
https://github.com/NVIDIA/NeMo/blob/main-vis-res/nemo/collections/cv/datasets/configs.py

@omry The PR for this will also give us a chance to think about how to logically structure managing 'two projects' - both torch and torchvision.

Another question is whether torchvision should become a dependency. It will need to be at least for tests.

We need to decide if we want to have a different config distribution (pip pacakge) or if we are good with putting all subprojects in the same artifact.
Is torchvesion released with it's own versioning or is it tied to PyTorch's version?

please note there is more than torchvision, i.e. torchaudio (that we are using in NeMo) and torchtext (that I don't really have experience)...

We need to decide if we want to have a different config distribution (pip pacakge) or if we are good with putting all subprojects in the same artifact.
Is torchvesion released with it's own versioning or is it tied to PyTorch's version?

Yap, all those projects have their own versioning.
My proposition is to do it as optional package, with optional dependencies
so one can install

torch-configs = torch-configs[torch] -> (torch) [default]
torch-configs[all] -> torch + torchvision + torchtext + torchnlp
or any subset e.g. torch-configs[torch,torchvision]

here's how we did it in NeMo when it comes to requirements:
https://github.com/NVIDIA/NeMo/blob/90775abc3a413baa0a6c8e68beb6160c7ebd108d/setup.py#L90
(still, I think we can make it better when it comes to actual "installation", as far as I know NeMo tries to install all collections disregarding the user choice during installation)

Okay, so if we do go with different pip packages per target library we support we end up with:
For each package:

What Hydra version (or versions?) a package is supporting.
What version (or versions?) of the library the package is supporting.

I added some text to the google doc about versioning, let's discuss there.


Make CI information, like tag, available

ci_detective as a source of variables

Getting the version from cargo (see #16) is probably good enough for now.  We still might want more for the "just in case" scenarios where a tarball is being generated from a commit besides the tagged one.


Add community health files

CONTRIBUTING.md
Issue templates
Code of conduct.

Actually, should add the last two centrally in https://github.com/supabase-community/.github


Score calculation as a product of the bbox and maskiou probability

@dbolya Hi Daniel, I found something strange in the postprocess function (in layers/output_utils.py). I'm  using the yolact_plus_base_config settings.
In the first section of the function there is a filtering on the score_threshold (line 43). Later in the function there is a recalculation of the scores using the maskiou_p (line 86).
Shouldn't it be better that the filtering on the score_threshold is done after this calculation (applying the filer on the product of the bbox and maskiou probability)?

Good catch, but that threshold was only meant to be a preliminary one to reduce processing time. The real threshold happens in prep_display, which in the YOLACT++ case would threshold again using the reweighted score. Since it's not possible that the score increases from rescoring, this shouldn't be an issue.
Though, there is a question of how all-in-one the postprocess function should be. I intended score_threshold to be there just for extra speed and for the user to threshold the score themselves, but looking back that makes no sense huh.
Also, I think your fix is missing the application of keep to the masks, since they get generated before the score reweighting (since they're necessary for that). You probably need to change


[AMQ-8464] JMS 2.0 JMSConsumer receiveBody method implementations

Overall my impressions are that this seems more complicated than it appears to be at first glance because you need to properly handle rolling things back and making sure that in certain modes the client acts like the receive was never called. I am not sure that simply re-adding the message to the unconsumed list is enough. There's a lot of code in rollback() to handle things like duplicate detection so the message can be received again.
So that being said I think this needs more work and a lot more testing and might need to wait for 6.1.0


Updated SugarDevice to add platform methods

Added methods to detect the current platform in an activity: isApp, isWebApp, isElectron, isMobile.

Good. Thanks.


Timeout error on elasticsearch bulk operation

 Traceback (most recent call last):
   File "/usr/lib/python3.5/multiprocessing/pool.py", line 119, in worker
     result = (True, func(*args, **kwds))
   File "/usr/lib/python3.5/multiprocessing/pool.py", line 44, in mapstar
     return list(map(*args))
   File "./ethdrain.py", line 123, in setup_process
     helpers.bulk(elasticsearch, out_actions)
   File "/usr/local/lib/python3.5/dist-packages/elasticsearch/helpers/__init__.py", line 194, in bulk
     for ok, item in streaming_bulk(client, actions, **kwargs):
   File "/usr/local/lib/python3.5/dist-packages/elasticsearch/helpers/__init__.py", line 162, in streaming_bulk
     for result in _process_bulk_chunk(client, bulk_actions, raise_on_exception, raise_on_error, **kwargs):
   File "/usr/local/lib/python3.5/dist-packages/elasticsearch/helpers/__init__.py", line 91, in _process_bulk_chunk
     raise e
   File "/usr/local/lib/python3.5/dist-packages/elasticsearch/helpers/__init__.py", line 87, in _process_bulk_chunk
     resp = client.bulk('\n'.join(bulk_actions) + '\n', **kwargs)
   File "/usr/local/lib/python3.5/dist-packages/elasticsearch/client/utils.py", line 73, in _wrapped
     return func(*args, params=params, **kwargs)
   File "/usr/local/lib/python3.5/dist-packages/elasticsearch/client/__init__.py", line 1167, in bulk
     doc_type, '_bulk'), params=params, body=self._bulk_body(body))
   File "/usr/local/lib/python3.5/dist-packages/elasticsearch/transport.py", line 318, in perform_request
     status, headers, data = connection.perform_request(method, url, params, body, ignore=ignore, timeout=timeout)
   File "/usr/local/lib/python3.5/dist-packages/elasticsearch/connection/http_urllib3.py", line 122, in perform_request
     raise ConnectionTimeout('TIMEOUT', str(e), e)
 elasticsearch.exceptions.ConnectionTimeout: ConnectionTimeout caused by - ReadTimeoutError(HTTPConnectionPool(host='localhost', port=9200): Read timed out. (read
  timeout=10))
 """

 The above exception was the direct cause of the following exception:

 Traceback (most recent call last):
   File "./ethdrain.py", line 151, in <module>
     POOL.map(setup_process, CHUNKS)
  File "/usr/lib/python3.5/multiprocessing/pool.py", line 260, in map
    return self._map_async(func, iterable, mapstar, chunksize).get()
  File "/usr/lib/python3.5/multiprocessing/pool.py", line 608, in get
    raise self._value
elasticsearch.exceptions.ConnectionTimeout: ConnectionTimeout caused by - ReadTimeoutError(None: None)

helpers.bulk line seems to timeout in some cases

Should probably:

lower the CHUNK_SIZE to a more reasonable number (500 Blocks could be HUGE depending on how many tx there is in).
increase the timeout value (default is 10 sec).

fixed by fec31426f605eb75659df3de0e90c969f72a565b


Not working on Arch

Hi,
thanks for the enhancement! Unfortunately I couldn't make it work on Arch with Gimp 2.818:

The theme is listed and can be selected in the Themes section, but doesn't really do anything. Would appreciate some help if I misunderstood something.
Cheers,
Szabolcs

Thanks for opening an issue. I'm using Arch too. Not sure if the desktop would make a difference but I'm using Gnome.

Thanks for opening an issue. I'm using Arch too. Not sure if the desktop would make a difference but I'm using Gnome.

Not sure, I'm using fvwm, but I'm also running gnome apps, and GTK themes also work.

For what it's worth, it doesn't work here either. I'm using Ubuntu with Cinnamon. I put Cinnamon in HiDPI mode but did not make other changes. Loading this theme appears to change something, but it does not make anything bigger. Gimp version: 2.8.16.

I'm on Arch with Gnome also, and it's not working for me either.
When I run it from the command line, I see some errors on startup, but I see these errors even when I start it up using the default theme:
$ gimp
GEGL-geglmodule.c-Message: Module '/usr/lib/gegl-0.2/openraw.so' load error: libopenraw.so.7: cannot open shared object file: No such file or directory
/usr/lib/gimp/2.0/plug-ins/web-page: error while loading shared libraries: libwebkitgtk-1.0.so.0: cannot open shared object file: No such file or directory

(gimp:14328): LibGimpBase-WARNING **: gimp: gimp_wire_read(): error
/usr/lib/gimp/2.0/plug-ins/help-browser: error while loading shared libraries: libwebkitgtk-1.0.so.0: cannot open shared object file: No such file or directory

(gimp:14328): LibGimpBase-WARNING **: gimp: gimp_wire_read(): error

Nothing new gets logged when I change the theme.  Like @LucidTaZ, when I choose it something (maybe the border sizes) changes, but the icons don't.
Here's the detailed version info:
 $ gimp-console --version --verbose
GNU Image Manipulation Program version 2.8.18
git-describe: GIMP_2_8_16-114-gcd2bbf7

using GEGL version 0.2.0 (compiled against version 0.2.0)
using GLib version 2.50.2 (compiled against version 2.50.2)
using GdkPixbuf version 2.36.4 (compiled against version 2.36.1)
using Pango version 1.40.3 (compiled against version 1.40.3)
using Fontconfig version 2.12.1 (compiled against version 2.12.1)
using Cairo version 1.14.8 (compiled against version 1.14.8)

Thanks for adding detail everyone. I'm time poor at the moment so please feel free to open a PR if you think you've got a solution.

Didn't work on Fedora 25 for me, either. Here's the fix:
cd ~/.gimp-2.8/themes/gimp-hidpi
mv gimp-hidpi-master/* .
rmdir gimp-hidpi-master
Theme should now be working.

Yep. I hope it's a simple unzip mistake. In the installation instructions I'm using the -d option of unzip which extracts the files of the zip directly into the target.

I created a pull request for the readme that contains a fix. Note that the zip should always contain a top level directory (as it does now), but I believe GitHub always adds that.
When following the new instruction, the directory is called "gimp-hidpi-master", not "gimp-hidpi". It's not an issue for the functionality though, and it might actually be preferable.

Worked for me too to get the icons in place. Thanks @cwells. However, even though the icons are SVGs, they're still small (as well as the dialogs, and toolbox, the layers window, etc). Seems more like a GIMP issue than a theme issue though. Do you all know if there's something else I could to scale things up? My monitor is 3200x1800px @ 276ppi (it's a Dell XPS 13).

Apologies for the unzip confusion here. The README has been updated accordingly.


Mouse and Keyboard lagging while XOutput open

when i open XOutput i notice a lag of mouse especially when i play videogames.
is there a way to shadow keyboard and mouse so to not have input lag?

You can solve the problem by
Taskmanager>Details>Xoutput.exe>Rightclick>Set Preference>High>Change of priority
If Xoutput.exe doesn't appear open it and try

You can solve the problem by Taskmanager>Details>Xoutput.exe>Rightclick>Set Preference>High>Change of priority
If Xoutput.exe doesn't appear open it and try

Hasn't worked for me.


Fix case of emptyString option

Code expects camel case option, readme specified option as lowercase and was consequently ignored.

Closed, duplicate of PR #10


What's difference between View::render and View::renderTemplate

They do the same things - include template file. Isn't be better to remove View::renderTemplate, add some addtitonal arguments to the View::render?

renderTemplate goes to the active theme while render goes to the views folder so they are different.
Whilst the render method could be changed to do both or even use an array instead of params I prefer to keep them separate.


PCA and RAMBO benchmarks should be migrated to new infrastructure

PCA and RAMBO benchmarks have to be migrated to new infrastructure.

[x] initialize function
[x] numpy/baseline function for validation

PCA and Rambo available in dpbench.


Tooltip - Performance

In my app I can often have over 1000 tooltips on my page,
And in the current state, it seems to get really laggy.
Perhaps have a Tooltip service that listens to mouse hover events and displays tooltips wherever you need to instead of creating more and more listeners?

Nice edge case, I will think how to overcome this

BTW I would love to look on this crazy page :D

any plan for this?

I will get here but not sure when

tried several options, with lazy injection and so on
no reasonable changes in performance
amount of listeners, can not be reduced Unfortunately (at least at the moment)

I hope thing gets fixed early. :(

performance will be one of the main prios for v2 of ngx-bs
we actually already did improvement to registering event handlers in v1.9.1
but some changes require breaking changes :(

its ok we have moved to another third party tooltip provider which works well !!

Is this still actual? Tested it here with 2000 tooltips, no lags. Closing it for now.

based off llyaSurmay's mock up, i've constructed something close to what im seeing here.
the difference between when the table is using the tooltip vs a simple title is obvious. Please reopen

In the newest version of ngx-bootstrap it LAGS as hell... PLEASE reopen

Just watched this - the answer is in the video https://youtu.be/Tlmx1PbP8Qw?t=761
Hope this helps


Consider orchestra integration

Orchestra is a Laravel base CMS with a lot of features and useful extras. May be worth working around building on top of this platform aswell as supporting base Laravel installs.

The project is good but the CMS is lacking in approachability from a consumer standpoint. Instead of directly integrating into this project, will instead attempt to organize the generated application into a similar structure while avoiding any direct dependencies.


add codecov .toml

We are missing codecov dependency in pyproject.toml. And need to check overall that our implementation with poetry is working

Done


move the dark mode to bottom in mobile devices

Naming Rules
Name your PR like this: ISSUENUMBER-TITLE-YOURNAME
Description
make map section responsive.
#33
Make sure you include the issue number with a hash sign # so Github can link this PR to the right issue, like this:
Fixes
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have carefully reviewed my own code
[ ] I have commented my code
[ ] I have updated any documentation


Bug: Cursor disapear when hold right arrow key

Screen: Champion selector
Action: Hold RIGHT button
Actual: The cursor is disapear and program is hanging

Seems to have been caused by https://github.com/sweetriverfish/LeagueAutoAccept/commit/883e7ec134746fa455705cf8edd3835505e78398

Issues solved, thank you

Fastest issue resolved I ever posted on github xD
Thanks.


sitemapindex with child-sitemaps returns always the same sitemap

Using the following configuration
const defaultOptionsExample = {
    hostname: 'http://localhost:3000',
    exclude: ['/**'],
    //gzip: true
}

export const sitemap = async () =>  {
    return {
        exclude: ['/**'],
        path: '/sitemapindex.xml',
        sitemaps : [
            {
                ...defaultOptionsExample,
                path: `sitemap-1.xml`,
                routes: ["/bla", "/fasel"],
            },
            {
                ...defaultOptionsExample,
                path: `sitemap-2.xml`,
                routes: ["/xxx", "/yyy"],
            }
        ]
    }
}

I get this for sitemap-1.xml
<url>
<loc>https://localhost:3000/bla</loc>
</url>
<url>
<loc>https://localhost:3000/fasel</loc>
</url>
</urlset>

and the exact same content for sitemap-2.xml
<url>
<loc>http://localhost:3000/bla</loc>
</url>
<url>
<loc>http://localhost:3000/fasel</loc>
</url>
</urlset>

If I load sitemap-2.xml first I get for both sitemaps.
<urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9" xmlns:news="http://www.google.com/schemas/sitemap-news/0.9" xmlns:xhtml="http://www.w3.org/1999/xhtml" xmlns:mobile="http://www.google.com/schemas/sitemap-mobile/1.0" xmlns:image="http://www.google.com/schemas/sitemap-image/1.1" xmlns:video="http://www.google.com/schemas/sitemap-video/1.1">
<url>
<loc>http://localhost:3000/xxx</loc>
</url>
<url>
<loc>http://localhost:3000/yyy</loc>
</url>
</urlset>

It seems like the first content that gets generated is also show as the other sitemaps or some kind of caching issue.

Hey sorry for the late response!
Would be nice if you could provide a  Vitest test,  would save me some time.
Examples can be found in https://github.com/funkenstudio/sitemap-module-nuxt-3/blob/experimental/test/module.test.js
Anyways I will try to look at this start of next week :)

Reading the mentioned file I saw that the vitests for my issue are already implemented.
On my system they all fail. See this screenshot
bash-5.1# yarn nuxt-build-module && yarn test
yarn run v1.22.19
$ /app/data/sitemap-module-nuxt-3/node_modules/.bin/nuxt-build-module
ℹ Building @funken-studio/sitemap-nuxt-3                                                                                08:11:51
✔ Build succeed for @funken-studio/sitemap-nuxt-3                                                                                08:12:01
  /app/data/sitemap-module-nuxt-3/dist/runtime (chunks: 16 files)                                                                             08:12:01
  dist/module.mjs (size: 69.6 kB, exports: default)                                                                                08:12:01
                                                                                08:12:01
Done in 12.24s.
yarn run v1.22.19
$ yarn lint && yarn unit
$ eslint . --ext js,vue
$ vitest run

 RUN  v0.15.2 /app/data/sitemap-module-nuxt-3

I have the same issue

Should be finally fixed with version 4.0.4


Migrate primary branch from master to main

See https://github.com/Senzing/stream-loader/issues/283

Pull requests:

https://github.com/Senzing/mapper-icij/pull/29


Feature request: MCUXpresso IDE integration

The NXP MCUXpresso IDE is probably the most widely used development environment for NXP micro-controllers. It is free to use and available for Linux, macOS and Windows. It seems like there is no support for it in TensorFlow Lite for Microcontrollers yet. Would be great to have support for it.

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


Remove feature flag from NCRC content

[ ] Confirm duplicates have disappeared
[ ] Remove feature flag
[ ] Check live in Production

Duplicates have not disappeared:

https://staging.sciety.org/articles/activity/10.1101/2<IP_ADDRESS>43836
https://staging.sciety.org/articles/activity/10.1101/2<IP_ADDRESS>49023
https://staging.sciety.org/articles/activity/10.1101/2<IP_ADDRESS>47381
https://staging.sciety.org/articles/activity/10.1101/2<IP_ADDRESS>55632


Onboarding Request

Your Twitch username
snKarma

So, it looks like in my db you successfully made one request in the past and I'm not entirely sure how. I'm onboarding your account now

What do I have to do next so it works?

The onboarding step here is really just because Spotify only lets me allow 25 users to have access to my service. Everything other than the Spotify API access should be set up when you go through the onboarding steps. I added you, so it should all work now.

Perfect thanks a lot


Server side Rendering React

How can i server side rendering react?

Please see: https://github.com/davezuko/react-redux-starter-kit/issues/805


Feat/payout

Takes #673 closer to the final stage

How long this feature should be merged into?

@thaibao-incognito, there is no timeline.  If you want to use this today, here is a complete example: https://github.com/TENK-DAO/tenk/blob/main/contracts/tenk/src/payout.rs

FYI, Payout NFT standard was aligned with the implementation: https://github.com/near/NEPs/pull/276
This PR is almost there, and it seems only the last mile is left, so I hope NEAR community can make this final push here

@frol I think this can now be closed in favour of https://github.com/near/near-sdk-rs/pull/1077


Tidy up profiles and add help

Just cleaning up our profile work,
adding profiles to help and refactoring file/function names.

Can another maintainer take a look and merge this please?


[Bug]

The entity IDs are not correct/changed with the latest version.
With the latest version 1.2.5, the entity IDs are missing the integration/device name in it. For example, the CPU usage entity ID now is sensor.cpu_usage, whereas before it was sensor.truenas_system_cpu_usage. This makes it difficult to identify while searching for an entity in cards or automations.
How to reproduce the issue
Steps to reproduce the behavior:

Go to the new entities created
Click on any of the entities
Go to the settings and look at the entity ID.

Expected behavior
Expect to see the device name as part of the entity id.
Screenshots

Software versions
All fields in this sections are required.

Home Assistant version: HA 2023.09.2
TrueNAS integration version: v1.2.5
TrueNAS version: TrueNAS-SCALE-<IP_ADDRESS>

thats intended change, as it was causing issues everywhere except system device.

see #101


⚠ Bofrost.at has degraded performance

In aa6353a, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1253 ms

Resolved: Bofrost.at performance has improved in 2ef423f.


Adds C++ testing user manual chapter

Context

Fixes https://github.com/gradle/gradle-native-private/issues/239
Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

Here is the rendering of the Testing C++ project chapter: https://builds.gradle.org/repository/download/Gradle_Check_BuildDistributions/22162146:id/distributions/gradle-5.5-all.zip!/gradle-5.5-20190423152654%2B0000/docs/userguide/cpp_testing.html


Set Up API Documentation Hosting

Host the API documentation on platforms like SwaggerHub in GitHub Pages for accessibility.
Push the documentation to a Category on Masa Developer docs using a GitHub Action
Add a Domain for the EC2 instance where we have been running validator/miner/healthcheck
Make sure the healthcheck API running on port 4000 has an SSL certificate ( requirement to be called from the vercel instance )

Theoretically this ticket can be closed if we are able to access the /docs from the deployed validator.  @hide-on-bush-x is the validator still deployed at: http://<IP_ADDRESS>:8000/docs?  seems like it may have moved...

@grantdfoster Devnet lost state, I need to go through the whole process on that machine again
Will do it later today

@hide-on-bush-x hey - how this went? Can we close the issue?

@Luka-Loncar Not yet, ill double check that the docs are accessible today

@5u6r054 can you help here maybe?

All clear now, validator is running on http://<IP_ADDRESS>:8000
To access docs everyone can go to http://<IP_ADDRESS>:8000/docs and it will be accessible, currently the validator is running on pm2

what is left here is to have a DNS for the swagger API's and link these to the masa developer docs for an example of the API endpoints.

There are still items left on this card to do

We decided that this issue is not needed and therefore its closed.


test: deal with lockfile checksums dynamically

When generating local registries in the testsuite, the specific package
checksums may vary from the original simply due to different zlib
versions. For example, Fedora 40 is now using zlib-ng-compat. We can
account for this dynamically by either formatting the checksum in the
expected output string, or using generate-lockfile on projects and
their expected output.

Alternatively, I think setting Compression::none() in the cargo-test-support registry will sufficiently avoid zlib differences. At least, it appears to be consistent at first glance, but I'll do more testing and send a new PR if that works out.

Closing in favor of #13744.

:umbrella: The latest upstream changes (presumably #13744) made this pull request unmergeable. Please resolve the merge conflicts.


feat: support use with statement to create a ftpserver temporarily

@giampaolo the continuous-integration test failed. But the log shows that this is caused by pyftpdlib.test.test_servers.TestCornerCasesThreadMixin.test_quick_connect . I think it's not caused by my code. Could you review this pull requre where you have spare time?

@giampaolo , Please review this pull request when you have any free time. Thanks a lot.

consider rerunning with current CI

I don't think this is necessary (or at least not in this form). Do you need this to have a test FTP server to connect to during unt tests? If that's the case, then I think it makes more sense to add __enter__ / __exit__ to the test FTP server  class:
https://github.com/giampaolo/pyftpdlib/blob/278466048b5f1612ba178b35243925be2e6717ac/pyftpdlib/test/__init__.py#L430

I have a simple script that need to start a ftpserver and call other web service to connect the ftpserver. It was inconvinent to first start a ftpserver and than run the simple script. I want to achieve this using with statement
with FtpServer(ftpusername, ftppassword) as tmp_ftpserver:
    requests.post("http://otherserver/download", json={
        ftpusername, ftppassword, filepath
    })

after the script, the ftpserver can be closed automatically.

Other wise, I will have to open two terminal.
First terminal
python -m ftpdlib --username=admin --password

Second terminal
python3 main.py

and I can only stop the ftpserver after the main script finished.

I forgot I added it, but FTPServer class already supports the context manager protocol:
https://github.com/giampaolo/pyftpdlib/commit/327f3e0
That is different than your context manager though: on  __enter__ it does NOT automatically call server_forever() in a thread.

I forgot I added it, but FTPServer class already supports the context manager protocol: 327f3e0 That is different than your context manager though: on __enter__ it does NOT automatically call server_forever() in a thread.

The function automatically call server_forever() in a thread is what I really want. That's why I create the FTPServerContext. I want to create a ftpserver temporally and use it in afterword scripts.

I tried use the blocking=False parameter, It didn't work as I wanted. My code looks like this:
with FTPServer(("localhost", 2121), handler) as f:
    f.serve_forever(blocking=False)
    print("start success")
    with FTP() as ftp:
        ftp.connect(host='localhost', port=2121)
        ftp.login()
        print(ftp.dir())

there is no start success printed on the screen, neither will the later code been executed.


Chunked Parquet Boto Client Token Expiration due to boto3_to_primitives

Describe the bug
When concurrently chunking large parquet datasets using to_parquet,  the passed boto3_session param ultimately gets decomposed into the underlying token/session/etc using boto3_to_primitives. This effectively causes a race condition between the TTL of the original session and the write operations.
For most other operations wrangler, the boto3 session does not seem to be converted to primitives, and therefore can utilize automatic credential refreshing using boto's DeferredRefreshableCredentials. In my case, the boto session being given to to_parquet is created in this way because it is assuming a cross-account AWS IAM role which normally have a 1 hour TTL.
Is there a reason that boto3_to_primitives is necessary rather than passing the session itself?
To Reproduce
Python 3.7.7 and awswrangler 2.3.0
Steps to reproduce the behavior:

Create a boto session with a specified TTL
Call to_parquet using the boto session, on a dataframe large enough to saturate the time window.

I think it's probably related to the development guide of boto3, which basically recommends to create a separate session for each thread and process: https://boto3.amazonaws.com/v1/documentation/api/latest/guide/resources.html?highlight=multithreading#multithreading-and-multiprocessing
So maybe it could be a solution to be able to provide a Callable which produces a boto3 session in a new thread or process. However I'm not quite sure if that would be the best setup. What do you think about it?

I think passing a callable could be a reasonable approach. I wonder if it could also be possible to pass the S3 client itself since it's thread-safe to each of the concurrent writes. It looks like those creds are ultimately getting used in _S3ObjectBase to create the S3 client.

Unfortunately boto sessions aren't thread-safe :/ Creating a session per thread, however, has another, less obvious downside - for every session, boto has to load service definition from the disk which may add up to quite a number. This happens irrespective of whether the session is created from scratch or from primitives.
Client, according to the docs, is thread-safe and doesn't suffer from the above, in fact, we're already reusing the client across threads for read requests. I think we're safe to do the same for writes, too.


MockServer issue since new contribution

OpenUI5 version:
Hi,
There is a contribution by @olirogers  to the MockServer (check url):
https://github.com/SAP/openui5/pull/922
I have a customer that have issues since this change and some data is not saved when using the 'setEntitySetData' method.
They did some debugging and this is what they wrote:

I have debugged the method "setEntitySetData" and it seems the problem is in the new "_enhanceWithMetadata" method. In the end of each loop over the navigation property you override the existing property:
o[k] = {
__deferred: {
 uri: r + E + "(" + t._createKeysString(e, o) + ")/" + k
}  };
However, we have already the data loaded for this navigation property:
o[k] is an object {results:[...]}. I guess you need to check if the navigation property is already resolved.

Can someone please take a look on this and and check the issue?
@olirogers maybe you have some insight about this.
Thanks,
Avishai.

SO actually the change was reverted :)
For more details, https://github.com/SAP/openui5/pull/922


Add some style package options

Such as centered or not, min width, max width.
By default have max width at 720px. That's 80 char width on my screen. Check that on another screen.
<style>
	.markdown-body {
		box-sizing: border-box;
		min-width: 200px;
		max-width: 980px;
		margin: 0 auto;
		padding: 45px;
	}

	@media (max-width: 767px) {
		.markdown-body {
			padding: 15px;
		}
	}
</style>

Add these to the css right before the rest of the document

https://github.com/kylebarron/atom-gmail/commit/53350afa61b4aa2aed961ce54a057a3d70920b7e


Rule has list of actions to execute

[ ] Extend RuleInterface and add actions there
[ ] Remove rule instance from ActionInterface (see point 4)
[ ] Probably remove ActionInterface complitally and replace it with callable type
[ ] Get rid of ActionsAction and CallbackAction
[ ] Update Engine::execute to be able to handle new actions functionality

I am on it ;)


Dependency updates

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

Project coverage is 76.96%. Comparing base (01adaad) to head (eb267d7).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1092   +/-   ##
=======================================
  Coverage   76.96%   76.96%
=======================================
  Files          68       68
  Lines         673      673
  Branches      120      100   -20
=======================================
  Hits          518      518
- Misses        139      155   +16
+ Partials       16        0   -16

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


🪄 Enhancement - Fix mypy typing issues

Summary
Currently running the python -m mypy src throws many warnings and errors.

Most of the issues are caused because of the missing stubs or py.typed marker of the social_core dependency and there are no critical errors that can ruin the user authentication workflow in production.


PlexServer() no longer working

Hi,
I was using plexapi successfully for a while and then moved to a new place, havent touched it in a while.
Now I can no longer connect to plex server using plexapi.server.PlexServer. I get the following error:
In [1]: from plexapi.server import PlexServer

In [2]: ps = PlexServer()
---------------------------------------------------------------------------
NotFound                                  Traceback (most recent call last)
<ipython-input-2-553a4ba3c216> in <module>()
----> 1 ps = PlexServer()

/Users/eric/miniconda3/envs/plex/lib/python2.7/site-packages/plexapi/server.pyc in __init__(self, baseurl, token, session)
     25         self.token = token
     26         self.session = session or requests.Session()
---> 27         data = self._connect()
     28         self.friendlyName = data.attrib.get('friendlyName')
     29         self.machineIdentifier = data.attrib.get('machineIdentifier')

/Users/eric/miniconda3/envs/plex/lib/python2.7/site-packages/plexapi/server.pyc in _connect(self)
     48         except Exception as err:
     49             log.error('%s: %s', self.baseurl, err)
---> 50             raise NotFound('No server found at: %s' % self.baseurl)
     51
     52     @property

NotFound: No server found at: http://localhost:32400

Note however that if I'm using the MyPlexAccount method, it works. Interestingly, after I create an account with MyPlexAccount, connect through the account, then plex=PlexServer() works fine.
Any idea/suggestion?
I'm using plexapi 2.0.2 with python 2.7
Thanks!

If you are trying to use the local access, it looks like you are missing some arguments. I use it as follows:
token = "tokengoeshere"
url = "serverurlgoerhere"
plex = PlexServer(url, token)

Thanks, I will try that as well. However, PlexServer() with no argument used to work, and it's still referred in the tutorial.
Was there a change from Plex then? If it's deprecated, then the tutorial should reflect this.

edurand is right in that the following code snippet used to work.  I haven't had a chance to test it yet to see if Plex changed something or not. The scary part is that you say it works after first connecting with a username and pass?
In [2]: ps = PlexServer()```

Yes, it does work after connecting first with passwd.

For what it's worth, its not working for me either as edurand explained. They must now require a username and password to connect locally.  I'll have to update the documentation accordingly.

Did you try adding your local network as an allowed network without authorization? I had to make use of that option some months ago when they added it to keep it working in my setup.

Is that a new Plex setting?
Le 30 oct. 2016 11:08 PM, "amazingr4b"<EMAIL_ADDRESS>a écrit :

Did you try adding your local network as an allowed network without
authorization? I had to make use of that option some months ago when they
added it to keep it working in my setup.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mjs7231/python-plexapi/issues/68#issuecomment-257183139,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABI7Kr-1UE9H4tZ4LCVjZz-IzjOQRe9cks5q5RVAgaJpZM4KV_93
.

Been around for at least a few months. You'd have to check the plex sever changelog to see when it happened.

Closing this one. As far as I can tell you can't access the API urls anymore without specifying an auth token, which is what was required to use plexapi.server.PlexServer() without any credentials. I tried messing around with the "List of IP addresses and networks that are allowed without auth" option in the server settings, but nothing seemed to be working for me.
Reopen if you feel I should research this more.


Moving out of an Allocated

Currently impossible but should be easily implemented.
into_inner(), move_out(), take() are all possible names.

I've implemented this as take()


Complexity digit on card overlaps summary text.

I created a branch with a fix.  Likely not a very good fix... but might help.
https://github.com/aaron832/kanboard/tree/feature/ComplexityOverlapFix

Can we get a pull request made from this?
@fguillot isn't this a bug instead of an improvement?

It's fixed in the development branch. I moved the complexity to the bottom right.


Backport "HBASE-26520 Remove use of db.hbase.namespance tracing attribute (#4015)" to branch-2.5

The HBase-specific attribute db.hbase.namespace has been deprecated in favor of the generic
db.name. See also https://github.com/open-telemetry/opentelemetry-specification/issues/1760
Signed-off-by: Duo Zhang<EMAIL_ADDRESS>Signed-off-by: Tak Lon (Stephen) Wu<EMAIL_ADDRESS>

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 12s
Docker mode activated.

-0 :warning:
yetus
0m  7s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2.5 Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m  9s
branch-2.5 passed

+1 :green_heart:
compile
0m 50s
branch-2.5 passed

+1 :green_heart:
shadedjars
7m  9s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 47s
branch-2.5 passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 57s
the patch passed

+1 :green_heart:
compile
0m 51s
the patch passed

+1 :green_heart:
javac
0m 51s
the patch passed

+1 :green_heart:
shadedjars
7m  5s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 44s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 33s
hbase-common in the patch passed.

+1 :green_heart:
unit
1m 44s
hbase-client in the patch passed.

32m  9s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4047/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4047

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 5bbf5022a092 4.15.0-153-generic #160-Ubuntu SMP Thu Jul 29 06:54:29 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.5 / 0ee15e0865

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4047/1/testReport/

Max. process+thread count
317 (vs. ulimit of 12500)

modules
C: hbase-common hbase-client U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4047/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 44s
Docker mode activated.

-0 :warning:
yetus
0m  7s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2.5 Compile Tests _

+0 :ok:
mvndep
0m 17s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m 56s
branch-2.5 passed

+1 :green_heart:
compile
1m  2s
branch-2.5 passed

+1 :green_heart:
shadedjars
7m 54s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m  0s
branch-2.5 passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 28s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m 35s
the patch passed

+1 :green_heart:
compile
1m  3s
the patch passed

+1 :green_heart:
javac
1m  3s
the patch passed

+1 :green_heart:
shadedjars
7m 45s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 56s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 48s
hbase-common in the patch passed.

+1 :green_heart:
unit
1m 39s
hbase-client in the patch passed.

36m 58s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4047/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4047

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 0bd87c51cf07 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.5 / 0ee15e0865

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4047/1/testReport/

Max. process+thread count
363 (vs. ulimit of 12500)

modules
C: hbase-common hbase-client U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4047/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Have command to format the output of the audit command

The future of the audit command is going to be to have it output more data as json. This is going to be harder for people to read.  Need to have the audit command output json and then another command convert that to html/latex/pdf.  Doing this this way also allows for people to build on cloudmapper as a continuous scanner (ex. Security Monkey), such that every hour or day they'd run collect -> audit and then push any results to their Slack or other alerting.

Resolved in #323


WIP: Use cookiejar to get updated session cookies in e2e_bookinfo

This PR attempts to fix the monthly issue of session expiration in the e2e
tests for bookinfo. Instead of hard-coding the cookie values, this PR uses
cookiejars + login requests to store session cookies for the two test users.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: sebastienvas
If they are not already assigned, you can assign the PR to them by writing /assign @sebastienvas in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@douglas-reid: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@douglas-reid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
6de4e1cc8c5e6b8279bf4a816182e19e1812e815
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@douglas-reid: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
6de4e1cc8c5e6b8279bf4a816182e19e1812e815
link
/test istio-unit-tests

prow/istio-presubmit.sh
6de4e1cc8c5e6b8279bf4a816182e19e1812e815
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@douglas-reid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
95d54fed5b3e251624bfc2c4299b57095524a0fc
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: hklai
If they are not already assigned, you can assign the PR to them by writing /assign @hklai in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@douglas-reid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
27d4e344f8343b40cb1cfc1bb922e65d4b0e9f2b
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@douglas-reid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
f83cd3f4d7a8f90ffd78f0976ac46a85d353eb96
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdake sure. as soon as this is in, i'll open the release-1.0 PR.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: rshriram
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [rshriram]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Backfill Russian cache data

Russia cache is never used
With old code, Russia would never actually hit the cache, because the URLs changed for every hit depending on what the CSRF token was.
We need to update the cache files so that Russia uses the cache every day, when Russian files exist.

Documenting this and putting my name on it as discussed in today's meeting.

Finished PR in https://github.com/lazd/coronadatascraper-cache/pull/4 (different repo).  With that PR, RU scrapes using only the cache.  :tada:

Done and merged, RU caching fixed.


kie-server: upgrade to WildFly 10/EAP 7 + Tomcat 8

remove old configs for WildFly 8, EAP 6
and Tomcat 7

adjust profile names to synch with the kie-wb-smoke-tests
(e.g. tomcat8x -> tomcat8; eap7x -> eap7)

remove ancient jbossas71x profile

+1 with just small comment to include KIE.SERVER.SIGNAL (disabled) queue definition

@sutaakar could you please take a look as well? (since this is also tests + cargo related)

+1

jenkins retest this (wrong job config caused that the kie-server tests weren't executed :/)


Multiple GPU training on nuScenes

Congratulations! I find it costs a lot of time for training with single GPU. Would you please release code for multiple GPU training? (or multiple GPU for Cylinder3D) Thank you!

The Cylinder3D-DDP is under testing. We will release it soon. Stay tuned.

You can refer to this code to achieve multi-gpu training.


Installation Problems

When executing python setup.py install  I get an error which says that a newer version of GCC is needed. The warning references to this  issue. However the suggested approaches from the issue did not work for me. I am using conda and don't have sudo rights.
Finally, I resolved the error by running: conda install -c anaconda gxx_linux-64
I hope this helps anyone who has the same problem.

I had the same problem with my Windows Desktop.. Just curious are you using windows too?

No, I tried this on linux machines only.

now it fails when pytorch > 1.7.0

I had the same problem with my Windows Desktop.. Just curious are you using windows too?

Have you ever solve this problem?


#1784 line chart custom color initialized

Description
If you change the line chart to Custom Color and then add dimension, the color will return to the initial palette.
Related Issue : #1784

How Has This Been Tested?

create line chart
color setting > cusom color setting
change color
add dimension

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project. it will be added soon
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document. it will be added soon
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.


Trim long container names and labels in details

fixes #418

Can confirm row labels are truncated as expected.
However, table headers are still wrapping:

I thought that was feature ;)

@davkal Maybe this should be addressesed somewhere else, but #418 also mentions there should be a way to display the full names (I suggested to do it on hover-on).

@fons that works already.

all right, lets keep this only for #418

Can confirm this seems to all work as expected.  LGTM


ADD Map for css in build doc

It's add module in npm gulp-sourcemaps
When genrate the doc it add sourcemap for the css

Are the source maps not supposed to be written in the same folder as the original file? (e.g., dist/assets/css/app.css.map)

I have tested and apparently Sourcemap directory is the root of the destination directory ('_build/assets/css').
With ('.') it is written in the same directory as the file css.
(it's translate by google translate sorry for my bad english)


Update substrate slides for HK 2024

Updates to:

Merklized storage slides.
Interaction with substrate chains.
Other minor updates to substrate slides.

Please make a better pr title so we know what changed when looking back.
Maybe something like "Substrate: Update slides for Hong Kong" or "Substrate module: discuss merklized storag" or something.

@Ank4n please fix conflicts

Thank you @Ank4n


fix(AIP-158): fix too-broad isPaginatedResponseMessage filter function

isPaginatedResponseMessage currently catches ALL messages with a field named next_page_token. After some discussion, we figured that should change to messages starting with List or Search and ending with Response, or fields ending with Response and containing a next_page_token field, to prevent next_page_tokens from being used in contexts outside of list/search.

Please change your commit message to be fix(AIP-158): fix filter function or something in that format fix(AIP-158): message

+1 fix/add tests

Closing as inactive. Please reopen and address comments if this is still something worth considering.


feature(build): mirate from yarn to pnpm

related to #370

I also worked on this yesterday, but fortunately I did changes you did not, and you did change I did not.
I’ll get your branch and add my changes

The Vercel docs is failing because the Vercel configuration in the Vercel build settings is using yarn.
Once pnpm works and lands on main I can change the Vercel build settings.


Expose language display names to extension API

VSCode Version: 1.8.1
OS Version: Irrelevant

Currently it seems it is not possible to get the name of the language of a document in an extension. The document object contains a property languageId which has an internal ID for the language. vscode.languages.getLanguages() can also be used to get all registered language IDs. There is no method to get the actual names of the languages.
The editor displays the name of the language in the status bar and in the language mode menu, but the names are not exposed to extensions. Personally I am working on an extension for my web service which requires the actual language names and not the IDs.

The extension https://marketplace.visualstudio.com/items?itemName=juha-ristolainen.code-stats-vscode requires this feature to be implemented for proper functionality. Without it, it can only work with a hardcoded mapping of language IDs to language names, which will obviously fail when new programming languages and other syntaxes are created.

need it

Is this problem still unsolved?
I am also having trouble with this problem now.

Here is the code I wrote for someone who is having the same problem.
This code seems to be generally fine, but I can't get the localized plaintext name.
const getLanguageName = (languageId: string) => vscode.extensions.all
    .map(i => <any[]>(i.packageJSON as any)?.contributes?.languages)
    .filter(i => i)
    .reduce((a, b) => a.concat(b), [])
    .filter(i => i.id === languageId && 0 < (i.aliases?.length ?? 0))
    .map(i => i?.aliases?.[0])?.[0] ?? languageId;

Came here to say this would be a nice to have.

/duplicate of https://github.com/microsoft/vscode/issues/2871


[Feature] Add title attribute to buttons in the Note Menu Bar and Bring the Note Menu Bar to Top in order to follow Fitt's Law of Human Computer Interaction UX Design

Added tooltips for the icons in the note menu bar.
Placed the note menu bar on top of the editor because of the universal design practices and to follow Fitt's law of human-computer interactions. According to this law, the time required for a person to move a pointer to the note menu bar is high if it is placed bottom of the editor. Most web applications place this kind of menu bar on top of the editor.
Closes # #456
Browser checklist
This PR has been tested in the following browsers:
Chrome
Firefox
Safari
Tested for the above browsers.
Tested for the above browsers.

After improvements.


Related Project

Just created a similar project, but different interface and supports more variety of environments (Windows, Linux, Docker container).  And was wondering if you'd be willing to create a Related Projects section above your license in the readme and linking over.
https://github.com/tracker1/node-ms-sqlcmd

Yeah, just submit a PR and I'll accept. I don't use/maintain this anymore so would be great to have an alternative. Thx!

Oh and feel free to add it at the top of the readme so people don't miss it.


Unable to resolve resource on Windows

Original fix #1715
Regression from #2072
Related https://github.com/graphql/vscode-graphql/issues/254

After some digging, I've confirmed that the bad path is indeed present in the GraphQLCache for the type.
Supposedly the uri is wrong when caching the schema (I wasn't able to debug at that stage because I'm attaching too late to the process)
https://github.com/graphql/graphiql/blob/484c0523cdd529f9e261d61a38616b6745075c7f/packages/graphql-language-service-server/src/MessageProcessor.ts#L827-L845
I suspect some vscode-uri.URI.parse(uri).toString() should use vscode-uri.URI.parse(uri).fsPath instead? but my initial attempt failed miserably and I'm a little out of my comfort zone at this point

Alright I gave it another try, from #2072 I put back all the replaced occurrences of pathToFileURL and it works
I think the problem is because it shouldn't be using vscode-uri.URI.parse on filesystem paths, it should only be used on fileURI
I get the following on Windows
> require("vscode-uri").URI.parse("C:\\projects\\graphiql\\packages\\graphql-language-service-server").toString()
'C:%5Cprojects%5Cgraphiql%5Cpackages%5Cgraphql-language-service-server'

> require("url").pathToFileURL("C:\\projects\\graphiql\\packages\\graphql-language-service-server").toString()
'file:///C:/projects/graphiql/packages/graphql-language-service-server

Actually, it looks like the proper fix is to use vscode-uri.URI.file().toString() in place where we have a filesystem path
As shown here https://github.com/microsoft/vscode-uri

> require("vscode-uri").URI.file("C:\\projects\\graphiql\\packages\\graphql-language-service-server").toString()
'file:///c%3A/projects/graphiql/packages/graphql-language-service-server'

I tried it real quick and it seems to work


Update sbt to 1.5.5

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

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

labels: library-update, semver-patch

Codecov Report

Merging #135 (031e700) into master (56a0cbc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #135   +/-   ##
=======================================
  Coverage   80.35%   80.35%
=======================================
  Files           6        6
  Lines         112      112
=======================================
  Hits           90       90
  Misses         22       22

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 56a0cbc...031e700. Read the comment docs.


Requiring UI Pack with Webpack

Is there any specific reason the Velocity UI pack isn't required by default? What is best practice for adding this to a webpack project? Importing it globally doesn't help as the package pulls directly from 'velocity-animate', not the global Velocity object I create within my project.
Currently I'm aliasing 'velocity-animate' to a internal module and adding the UI pack manually but this seems messy.
Thanks, this project looks awesome.

I noticed that too, the velocity-animate package pulled by velocity-react's components is different from the one you would normally include globally in your project, since it comes bundled as its own dependency, effectively causing the UI pack to register on an object different that the one used by velocity-react.
My current workaround is to require globally the correct velocity-animate module
require('velocity-react/node_modules/velocity-animate')
require('velocity-react/node_modules/velocity-animate/velocity.ui')

Is this a case where you have Velocity already as a dependency in your project? In that case, does npm dedupe do what you need to collapse the dependency tree?

FWIW the demo app uses webpack, though it's obviously quite toy. We have this at the top of the top-level component:
require('velocity-animate');
require('velocity-animate/velocity.ui');

But will https://github.com/twitter-fabric/velocity-react/blob/master/velocity-component.js#L42 use the version with UI pack added here: https://github.com/twitter-fabric/velocity-react/blob/master/demo/main.jsx#L6?
At least in my testing, it will pull the Velocity global object directly from the npm_modules folder. Again, there may be a webpack config I'm unaware of that fixes this

Hrm. I noticed that the demo isn't the best case because it doesn't NPM-in the velocity-react bits.
Regardless, in a separate toy project I still couldn't repro. I see that in ./node_modules/velocity-react/node_modules that there's no velocity-animate package of its own. NPM didn't install one there, and it's instead letting the one in ./node_modules/velocity-animate be used.
This is with NPM 2.14.4.
My package.json is:
{
  "name": "velocity-test",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "author": "",
  "license": "ISC",
  "dependencies": {
    "react": "^0.13.3",
    "velocity-animate": "^1.2.3",
    "velocity-react": "^1.0.0"
  },
  "devDependencies": {
    "babel-core": "^5.8.25",
    "babel-loader": "^5.3.2",
    "react-hot-loader": "^1.3.0",
    "style-loader": "^0.12.4",
    "webpack": "^1.12.2",
    "webpack-dev-server": "^1.12.0"
  }
}

And do the require for velocity-animate and velocity-animate/velocity.ui in an app.js, followed by a require for the MainComponent that requires velocity-react/velocity-component.

Here's the gist of my packaging test: https://gist.github.com/phopkins/d127287ffa950d3b0d2c

Nice.  This appears to have solved my First Argument error. Similar to https://github.com/julianshapiro/velocity/issues/327

Closing, since I think npm dedupe pretty much solves this. Please re-open with more info if that's not the case.


Planet Saved States

Describe your changes
In this branch I will be working out how to have saved and consistent states for the planets so you can know what is on them from space, and so you can enter and leave the planet as often as you want.
Issue number and link
#168
Checklist before requesting a review

[ ] I have performed a self-review of my code
[ ] I have added any applicable tests
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] This complies with the product coding guidelines

Messed up opening the PR. Reopening.


ML-405 Remove typedefs to fix compiler warning

Signed-off-by: johnholt<EMAIL_ADDRESS>

Merged, though I wonder if version should have been increased?


0.7 Turbo problem

new 0.7 version default Turbo pin is -1 , I try to use up or down arrow button restore to 0.62 ver Pin in 14,
up arrow button give me red prompt : 0 is already assigned!
but older version u can continue choose to a empty pin, and this 0.7 ver I can not use it. (so this is bug 1)
next I use keyboard input 14 (after this u can use up or down arrow button choose number, but if u let it to -1 bug 1 will show again)
The most important question is now I am setting to:
Turbo Pin: 14
Turbo Pin LED: -1
Turbo Shot Count: 15
Turbo Dial (ADC ONLY): -1
and let Turbo Add On Enable , when I reinsert usb cable, OLED displayer can show T15
but I press Turbo Button (in Pin 14)
None of the buttons that could be turbo in the previous version did not work (This is bug 2)
I did someting wrong or is there really a BUG?

This is a bug.The turbo cannot be started. PicoFightingBoard.uf2 It's the same.After the web configuration is enabled, it will still show the closed state when you enter it again.

Yes we are aware of the bug and will work to address it.


Fix performance issues in graph

The graph does not scale linearly. This is due to a number of factors:

The graph is currently persisted and updated in a service. This means that updates will be put on it, even when the graph is out of view. This means that performance issues are not only felt on the graph, but the table view (which can support large quantities of data) is also suffering.

This could be resolved in #574

Elements are being added one at a time. Every time an element is added, cytoscape performs multiple sorts. This is the reason for the non linear scaling - As more get added, the longer the sort takes.

Elements have CSS applied to them one at a time once they are added - This causes a second sort thereby worsening the effect of the above issue.

Blocked until #574 is merged.

Merged into develop


Feature Request: Add slider-carousel

Currently slider exists in https://app.pluralsight.com/library/
It would be great to have something similar in the Design System.
In our own practice we used Slick slider which works well, but not ideal, especially dots.

thanks for the request!
duplicate of #411


Replaced some strange punctuation

The first dash char is Unicode U+2014, not the normal dash.
The second quotation marks are U+201C and U+201D.
Is it done intentionally?

No, it's not intentional. Thanks!


Remote Table Replication

Tutorials: https://developers.sap.com/tutorials/hana-cloud-mission-extend-09.html
create virtual table  refresh definition;
should be
ALTER virtual table  refresh definition;

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.


Add info on protecting the branch

Why:
I've tried to use auto-merge, but the button did not appear even though I was running CI jobs on my PR and enabled "Allow auto-merge". Also, I protected the branch by setting "Require status checks to pass before merging". However, only after also setting "Require pull request reviews before merging", the button appeared!
What's being changed:
Update documentation.
Check off the following:

[ ] I have reviewed my changes in staging (look for the latest deployment event in your pull request's timeline, then click View deployment).
[ ] For content changes, I have completed the self-review checklist.

I've updated the text, because I just figured that I needed to explicitly set which tests need to pass.

@rikhuijzer Thanks so much for opening a PR!! I am not sure whether you need to have branch protection set. This will be a great question for our wonderful  support team. Can you please check in with them and let us know if a docs update is needed 👍 ✨

I rather not be part of this bureaucracy. I‘m trying to fix your documentation, not mine.

For me it works now, so I don’t see why I should contact support

Sure of your wonderful focus, thank you
On Fri, Jul 9, 2021, 9:23 PM Ramya Parimi @.***> wrote:

@rikhuijzer https://github.com/rikhuijzer Thanks so much for opening a
PR!! I am not sure whether you need to have branch protection set. This
will be a great question for our wonderful support team
https://support.github.com/contact. Can you please check in with them
and let us know if a docs update is needed 👍 ✨
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/8034#issuecomment-877374093, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKPHZDKMBVE55XX263Y7BKDTW45CFANCNFSM5ABK4PCQ
.


Do not modify the current selection when trying to input an invalid character

Hello, first of all nice work!
An issue I encountered is when you select input text using CTRL+A and release "CTRL" just before the "A" the selection will be removed.  The following will only detect select all if the CTRL+A keys are released simultaneously:
if ((ctrlKey || cmdKey) && kdCode === 65) { /** if select all (a=65)*/
    return true;
}

A solution may be to replace this with something like :
if (getElementSelection(e.currentTarget).length > 0) {                 return true;             }
EDIT : Initial title : "CTRL+A selection removed"

@jannunezz Hello, and thank you for your feedback.
The behavior you are describing is the normal behavior you can observe in other inputs.
Using the ctrl+a keyboard shortcut means that you have to hit the keys in that order :

press ctrl
press a
release a
release ctrl

If you invert step 4 and 3, you are really trying to input some a characters in the input, which is prevented, dropping the current selection.
You can test that behavior in any other input and you'll find that most of the time this will indeed input some a characters in those.
The way I see it is that we'd need either :

to create a small debounce to check when we get a a key pressed, if ctrl was pressed a few milliseconds ago. I'm not sure this level of complexity has its place in autoNumeric though, or
do not drop the current selection if the input character does not change the input value.

@BobKnothe Would you see any drawback to trying to keep the current selection when dropping any invalid key input?

The last commit fixes that issue by preventing any 'bad' key to modify the current selection.
This can be found for now in the next branch.


Ability to run a GGML file without registering it

I realized it would be really convenient if you could execute a new GGML file without having to first do the registration dance.
I could do this with a model called ggml that takes a required path option.

Got this working:
llm -m gguf -o path ~/Downloads/una-cybertron-7b-v2-bf16.Q8_0.gguf 'Instruction: Write a poem about an otter
Response:'

Which outputs:

Otters are elusive, playful creatures
Swift in the water and crafty in their features
Hiding beneath kelp or drifting on logs
With a smile that stretches wide beyond our dogs
They dance gracefully underwater, chasing fish
Astonishing us all with such splendid bliss
Otters are known for their love of toys
Balls are tossed into the water, where they joyously play
These marvelous mammals leave an everlasting impact
Their presence bringing life to our waters and rivers unquenched
Dazzling spectacles for all who watch
A magical moment for some fortunate souls to catch
Otters embody a spirit unbounded by boundaries
Inspiring us to embrace the beauty of nature's wonders
Leaving footprints on our hearts that will always be etched
With lessons from these graceful, otter-ly loved creatures


A11y_AzureToolsStorageExplorer_Blob File_BlobContainerAcquireLease_ScreenReader:NVDA does not announce lease status changes or current locked status for selected blobs.

Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
GitHubTags: #A11y_AzureToolsStorageExplorer; #A11yMAS; #A11YTCS; #DesktopApp; #WCAG1.3.1; #NVDA; #Win32;
Environment Details:
Desktop App: Windows Version: Windows10
NVDA Version: 2019.1.1
Steps to Reproduce:

Launch Storage Explorer
Navigate to "Open Connect dialog" pane and Sign in to Azure.
Go to "Manage Accounts" pane and Select the subscriptions you will use. Once selected, click on "Apply" button.
Navigate to "Blob Containers" of the selected subscription, right click and hit enter on "Create Blob container" to Create a new Blob container.
Once completed, Navigate to "Upload" button and Upload a blob/file by using  "Upload Files" control.
Right click on blob and select "Acquire Lease" control and hit enter.
Open NVDA and check whether it reads anything about the locked status for which do the Acquire lease.
Actual:
NVDA does not read about the locked status for which after click on the acquire lease.
Same issue is observed in JAWS also.
Expected:
NVDA should provide the information about locked symbol for which do the acquire lease

User Impact:
Screen reader user unable to know that why the locked symbol is provided.
Attachment for Reference
1906_A11y_AzureToolsStorageExplorer_BlobFile_BlobContainerAcquireLease_ScreenReader.pptx

@nirmalsahoo Is the problem that we don't announce the change to item, or that we don't say anything about the lock when the item is selected?

Same root cause as #1983.

@Craxal

After selecting Item when hit enter on "Acquire Lease" screen reader not announces anything about the Change to Item.
And also Move to next item and again go back to "Locked" Item, Observe screen reader not announces any information about the Lock.

@MRayermannMSFT  Verified on version 1.12.0 and NVDA 2019.2.1. The issue is still reproduced. NVDA does not read the locked status as after click on acquire lease.NVDA should read the locked status comes after click on acquire lease.

So, it looks like the Acquire/Break Lease commands don't trigger the table ARIA label update. I'll need to investigate how to do that.

GitHubTags:#BM-TCS-StorageExplorer-Win32-Sep2019;


change buildkite output folder

Purpose
closes https://github.com/CliMA/ClimaCoupler.jl/issues/713
Removes all references to the /groups/esm/slurm-buildkite/climacoupler-ci  folder.
To Do

check that /groups/esm/slurm-buildkite/climacoupler-ci is still empty after CI runs

@Sbozzolo It looks like these were the only places this folder was used in the repo. Do you know if there's anything else that needs to be changed?

More generally, the folder was being used to save some file across runs, but the folder on the cluster is empty, meaning that files are not being saved. So, something is not working as expected. We can still move forward with this PR.


Add the concept of admin users

Fixes #20
Signed-off-by: Miquel Sabaté<EMAIL_ADDRESS>

I had some doubts about what and admin user can do (mostly when it comes to admin vs owner of a team), but this is probably something that will be more obvious as this project goes forward :)
This PR simply adds a new column admin to the users table and offers the possibility to seed the DB with a new admin user. This seeding has to be done when deploying for the first time on production. I've updated the Vagrant config to just seed the DB with the defaults.

The code looks fine. Do you think it would make sense to expand the specs to ensure users created by sign up are not admin?

AFAIK, the only way in which that could happen is if we explicitly permitted the admin parameter (e.g. by adding it in the configure_sign_up_params method from the Auth::RegistrationsController). Since we don't do that, it's not necessary to add it in the spec. However, I can add it if you think it would add an extra layer of security :)

You are right, we can omit it.


Module federation should declare the remote entries with its location

Feature request

Let's start from this example.
The app1 (shell/main/host) depends on app2.
This dependency is declared in the webpack.config.js, but the location of app2 is declared in the /public/index.html
What is the expected behavior?
I'd like to see them togheter, in the plugin options
    new ModuleFederationPlugin({
      name: "app1",
      library: { type: "var", name: "app1" },
      remotes: {
        app2: {alias: "app2", remoteEntry: "http://localhost:3002/remoteEntry.js"}
      },
      shared: ["react", "react-dom"],
    }),

What is motivation or use case for adding/changing the behavior?
Easier to maintain
How should this be implemented in your opinion?
It should not change the core behaviour. It's like a syntax sugar: just copy the remotes.{key}.remoteEntry to the index.html as <script>
Are you willing to work on this yourself?
I'd love to. Not sure if I know enough about webpack's implementation.

"I'd like to see them togheter" is not a good sentence.
It would be better to say: "I doesn't feel right to put this kind of metadata in the index.html".
It's true that the index.html is the place where it should end up, but doesn't mean it should be declared there.
A second option would be to declare it in the package.json since it's the place to declare the dependencies:
{
    "name": "app1",
    "version": "0.0.0",
    "dependencies": {...},
    "devDependencies": {...},
    "remoteEntries": {
        "app2": "http://localhost:3002/remoteEntry.js"
    }
}

I partly agree with u.  Declare in the index.html is not a good idea.
But if declaring in the config file, the url of app2 will just be static.
For example, we want to get the url of app2 from an admin backend.
Actually, I want webpack do the favor for me to finish the last step, not fetching remoteEntry.js by my code.

u can define remoteEntry in your js files.

This was been added via the script external

remoteEntry

how? could u please give  an example


Added troubleshooting information for Mac users

Fixes #86

@jo-mueller,
thanks for updating the documentation and making us aware of this problem :)


Now recheck when changes made while typesetting

Made changes as per dpvc's suggestions.
Added a flag mjPending to signal that the content has changed while the typesetting was happening. Setting the flag requeues CreatePreview(). This flag is then used for early exit instead of mjRunning This can close #1132.

The changes are good, but there is still a problem with the pull request.  You have it targeted to develop, which is great, but the problem is that it is branched from master, where it should be branched from develop.  Merges go from develop to master (creating a release), never the other direction.  You can see that yours is the wrong direction because it includes a commit at the top that is not one of the ones that you made.  Your commit history should include only your own changes, not ones due to merging a different branch into the develop branch.
So at this point there are two possible solutions:  one would be to check out develop, create a new branch from there, copy your updated sample file and commit that fresh into the new branch, and then make another new pull request for that branch having only the one commit.
The other is to rebase the current branch from master to develop using the git rebase command.
I suspect the first approach is easier, though it loses the earlier commit history.  In this case, that's probably fine, however.

Okay, I understand. I'll fix that and make a new PR. Thanks for your patience with this, I'm new to git and open-source contributing, and I really appreciate it.

Closing this in favor of #1148


How do avoid racing conditions with is_subscribed

Hi!
I feel somewhat uncomfortable with is_subscribed. There is a sequence point after calling is_subscribed. So another thread might unsubscribe the subscriber and then s.on_next() gets called on an unsubscribed object? Is this safe? And if yes, why do we call is_subscribed anyway?
auto published_observable =
    rxcpp::sources::create<int>([](const rxcpp::subscriber<int>& s)
    {
        for (int i = 0; i < 1000; i++)
        {
            if (!s.is_subscribed())
                return;
            s.on_next(i);
        }
    })

Yes, cross-thread cancellation is always a race. In the Rx model this is type of race is safe.
Only one call to on_error/on_completed/unsubscribe is delivered no matter how many calls are made. And once one of those calls is made no further calls to on_next will be delivered.
The subscriber uses is_subscribed() to filter calls to on_next/on_error/on_completed and calls unsubscribe when on_error/on_completed have been delivered.
calls to is_subscribed() are used to interrupt loops. unsubscribe functions passed to add() are used to teardown resources and connections. they are both needed to cover the diversity of sources. without is_subscribed() there would be so many unsubscribe functions passed to add() that just set a bool.

OK, I see I still have a lot to learn. Nonetheless my first impulse is that I would prefer an interface where this method is missing. If this type of race is safe (I assume s.on_next() will then throw or lead to a noop) why check anyway? And sorry if I do not grok it immediately.


Include exported library in prefixed import suggestions

Given a library
library libA;
class A { }

and another that exports the first library
library libB;
export '/libA.dart';

then class A should be suggested whenever libB is imported
import '/libB.dart' as b;
main() { b.^ } // <<<< Should suggest A here

Also see No autocomplete on some aliased libraries

https://codereview.chromium.org/1263093003/


Stream browser defaulting to UTF8 when using a custom Raw Stream type

I have narrowed down our data encoding issue in Stroom7 as mentioned above for Windows Sec events to be related to Stream Type. If I import using "Raw Events" everything works as expected. When I import using " Raw Events" which we use I 100% always get the "Unable to display source [12345:1:1] as text - Error fetching data: unable to decode a UTF-8 character....." error. No matter what data encoding I try it ALWAYS thinks it's UTF-8 and it's not, it is UTF-16LE and flushing the Feed Doc Cache makes no difference. It looks like the change to data encoding isn't registering for anything other than "Raw Events".
We have all of events set to " Raw Events" so this is a problem for us. Is there something we can do to fix this or is there an underlying change required?

Have added a new property stroom.data.meta.rawMetaTypes to categorise stream types as being types for receipt of raw data.  Any types included in this property will use the Data Encoding value set on the feed.
Thus if you have a custom stream type Custom Raw you need to add this value to both stroom.data.meta.metaTypes and stroom.data.meta.rawMetaTypes.

Showing utf32 data with a custom stream type

Showing the info tab that now displays the encoding (from the feed settings and appropriate to the stream type)

Fixed in >v7.0.6


Add params filtering on new column.

This is to maintain security of any params that are not supposed to be logged. As well as a way to exclude params that may bust the size of a text field.

I love it. I would love if there was some tests though. Would you be up for that?


Allow roles to be configured

A new shared input should be introduced to allow users to specify the roles (comma separated) to be used for certificates generated.
At the moment, the behaviour is to use all roles associated with the bot, which may not be conducive to following the principle of least privilege.

Transferred to https://github.com/teleport-actions/root/issues/1


Introduce a maxwarning option. To accompany the maxerr option.

#3562 Introduced some new issues. I have a situation where I have potentially an unlimited number of warnings. And I would like to stop collecting warnings at somepoint. Before #3562 I was using the maxerr option. Could we consider implementing a maxwarning option.

Yes, I'm certainly open to discussing that!
Something I mentioned in gh-3444 (the feature request which precipiated the change) seems relevant here:

Errors describe problems in parsing, so they often appear in large numbers following a single syntax error. That's because JSHint attempts to recover from invalid syntax as a convenience to the user. Recovery involves some guesswork, so there's no foolproof way to do it. If JSHint guesses incorrectly, then it will end up reporting a large number of irrelevant errors.
Warnings, on the other hand, are reported in response to discrete patterns in the code. They're far less likely to exhibit the same kind of "avalanche" that sometimes occurs for errors.

This makes me wonder: are each of the warnings that you are seeing distinct? What is it that makes some more important than others?

Are each of the warnings that you are seeing distinct? Yes? maybe? I'll try and answer this. Let me know if you are asking for something different though.
So in this situation a user is using a code editor in our app. And said user is potentially entering in minified code. (I know this sounds really silly but it makes sense in our case). Sometimes due to the minified code, we end up with things like a = new Date throwing warnings. These warnings are very individualized to their own cause. Totally understandable and good, but when you have thousands of these individual warnings it can be painful and slow to let the linter keep chewing cpu cycles. This also slows down our test suit pretty dramatically. I figure in this case if the user has this many warnings they probably don't really care to fix them. So I would like to limit the cpu waisted on them.
What is it that makes some more important than others? The case I described above hopefully gives some more context. It's not that any warning is more valuable than another. It's just the ability to limit cpu usage in this case.

Got it. That's a pretty rarefied use case, but I can see how you can't really work around it without explicit support from JSHint.
Implementing a maxwarning option shouldn't take too much code. Do you have any interest in writing a patch?


benchmark, charlatan (logical-conflict)

This pull request was generated by the 'mq' tool

/trunk merge


Fix bitrot in unrelated test, packaging 22.0 was just released.

I broke this out of #2007 since it's also affecting #2010.


Add iconresizer.com to the tools list

Project URL
https://iconresizer.com/#!/
Description
Added iconresizer.com to the tools list. It comes in handy when generating all icon files for the app store.
Why it should be included to awesome-ios (optional)
It speeds up the App Store submission process for developers and designers.
It is actively maintained and supports the latest iOS 10 icon sizes (as well as iOS 9  and before). I left the last 3 checkboxes unchecked as they are not applicable in this case.
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 or later
[ ] Supports Swift 3
[ ] Has a commit from less than 2 years ago
[ ] Has a clear README in English

          1 Error

:no_entry_sign:
Found 1 link issue

          1 Warning

:warning:
Too many changes (when adding, please keep it to one project per Pull Request)

Link issue by awesome_bot

Line
Status
Link

2080
302
https://swiftlang.ng.bluemix.net/#/repl redirects tohttps://swiftlang.ng.bluemix.net/verify

  Generated by :no_entry_sign: danger

There were 4 projects with trailing whitespace that were changed due to my editor settings. I committed these as the contribution guidelines specify that text editors should trim trailing whitespace. Let me know if you'd prefer if I only commit my addition and I'll make the change.

Hey @DKirwan!
Don't worry about the message. Thank you for contributing (and getting rid of those trailing whitespaces)! 🎉


Eloquent model pulls and paginate

I tried some variations to use getRecords with Eloquent and paginate, but I was unsuccessful. Has anyone tried to paginate search terms - whether through getRecords or getResults?

I'd be interested in this as well, generally how to use pagination in Larasearch.
I see we can pass a limit, padding and offset as an option to the search(), but didn't dig deeper how it's plugging into the Laravel pagination.
BTW the link in the wiki (https://github.com/iverberk/larasearch/wiki/Searching#pagination) is dead.
Cheers!

I did it like this
    $params['offset']  = $offset;
        $params['limit']  = $limit;
        $params['sort'][ Input::get('sort', '_score') ]['order'] = Input::get('order', 'desc');

        $results = Series::search(\Input::get('title', '*'), $params);

        $series = \Paginator::make($results->getResults()->toArray(), $results->getTotal(), $limit);

Paginator on L5 ?