No alert in spring log for "Not enough privileges." error

Here are 2 scenarios:
1. Correct user/pasword but wrong database in URL
Steps to reproduce:

Configurations in application.yml. Correct user/pasword but wrong database in URL
Hardcode the correct database name in mapper.xml
We won't get any warnning info or alert when starting the spring application
Also no alert when we call the corresponding api interface thru postman or others

2. Correct user/pasword and correct database in URL, but wrong database name in mapper.xml
Steps to reproduce:

Configurations in application.yml. Correct user/pasword and correct database in URL
Hardcode the wrong database name in mapper.xml
Call the interface by postman or else, we got 500 error.
No alert in application log.

I assume you're using JDBC driver 0.3.0 but what's the version of ClickHouse?

I assume you're using JDBC driver 0.3.0 but what's the version of ClickHouse?

clickhouse-<IP_ADDRESS>

Tried scenario 1 using ClickHouse 20.3, and I got below exception when creating a connection:
Message:
ClickHouse exception, code: 81, host: localhost, port: 50093; Code: 81, e.displayText() = DB::Exception: Database nonExistDb doesn't exist (version <IP_ADDRESS> (official build))
Stack trace:
ru.yandex.clickhouse.except.ClickHouseException: ClickHouse exception, code: 81, host: localhost, port: 50093; Code: 81, e.displayText() = DB::Exception: Database nonExistDb doesn't exist (version <IP_ADDRESS> (official build))

I'm not sure what exactly you need and why you didn't get any error log - it may relate to your log configuration but from JDBC driver perspective I didn't find anything abnormal.

Tried scenario 1 using ClickHouse 20.3, and I got below exception when creating a connection:
Message:
ClickHouse exception, code: 81, host: localhost, port: 50093; Code: 81, e.displayText() = DB::Exception: Database nonExistDb doesn't exist (version <IP_ADDRESS> (official build))
Stack trace:
ru.yandex.clickhouse.except.ClickHouseException: ClickHouse exception, code: 81, host: localhost, port: 50093; Code: 81, e.displayText() = DB::Exception: Database nonExistDb doesn't exist (version <IP_ADDRESS> (official build))

I'm not sure what exactly you need and why you didn't get any error log - it may relate to your log configuration but from JDBC driver perspective I didn't find anything abnormal.

Please let me clarify a bit more, it's not a non-existing db, but a existing database, which doesn't hava access to with the current username/password.

Thanks @jisa6001 for the reporting the issue. I didn't get your point but I was able to reproduce the issue.

Here is my case:
ddl
create database if not exists non_privilege_db;
create table if not exists non_privilege_db.credentials(user String, password String) engine=Memory;

user.xml
<test>
    <password>123</password>
    <networks incl="networks" replace="replace">
        <ip>::/0</ip>
    </networks>
    <!-- Settings profile for user. -->
    <profile>default</profile>
    <!-- Quota for user. -->
    <quota>default</quota>
    <allow_databases>
        <database>system</database>
    </allow_databases>
</test>

application.yml
spring:
  datasource:
    url: jdbc:clickhouse://localhost:8123/non_privilege_db
    username: test
    password: 123

And then when I started the SpringBootApplication, I didn't get any error info.
but if thru CLI, I could get the same error as you.

Interesting... I still cannot reproduce the issue you met.
Below test case works for me(dba is the super user I mentioned above).
@Test
public void testAcl() throws Exception {
    ClickHouseProperties props = new ClickHouseProperties();
    props.setUser("dba");
    props.setPassword("dba");
    ClickHouseDataSource ds = ClickHouseContainerForTest.newDataSource(props);
    try (ClickHouseConnection c = ds.getConnection(); Statement s = c.createStatement()) {
        s.execute("create database if not exists secrets");
        s.execute("create table if not exists secrets.credentials(user String, password String) engine=Memory");
        s.execute("create user guest identified with sha256_password by '123'");
        s.execute("grant select on system.* to guest");
    }

    ClickHouseProperties newProps = new ClickHouseProperties();
    newProps.setUser("test");
    newProps.setPassword("123");
    ClickHouseDataSource newDs = ClickHouseContainerForTest.newDataSource("secrets", newProps);
    try (ClickHouseConnection c = newDs.getConnection(); Statement s = c.createStatement();
        ResultSet rs = s.executeQuery("select * from credentials")) {
        fail("Should fail as test user does not have access to secrets database");
    } catch (ClickHouseException e) {
        assertTrue(e.getErrorCode() == 497);
    }

    newProps = new ClickHouseProperties();
    newProps.setUser("guest");
    newProps.setPassword("123");
    newDs = ClickHouseContainerForTest.newDataSource("secrets", newProps);
    try (ClickHouseConnection c = newDs.getConnection(); Statement s = c.createStatement();
        ResultSet rs = s.executeQuery("select * from credentials")) {
        fail("Should fail as guest user does not have access to secrets database");
    } catch (ClickHouseException e) {
        assertTrue(e.getErrorCode() == 497);
    }
}

My suggestion is to enable Apache http client debug log to see what's going on there. Pay attention to the authorization header to double check if the user is test.
log4j.logger.org.apache.http=DEBUG

I debuged the souce code and compared two case: non-existing database & no privilege database;
And I assumed the connection url will be verified by  "ClickHouseConnectionImpl"- initConnection, and the below sql will be executed.
ResultSet rs = s.executeQuery("select timezone(), version()")
ClickHouseStatementImpl - executeQuery
InputStream is = getInputStream(sql, additionalDBParams, externalData, additionalRequestParams);
ClickHouseStatementImpl - getInputStream
            HttpResponse response = client.execute(post, httpContext);
            entity = response.getEntity();
            checkForErrorAndThrow(entity, response);

checkForErrorAndThrow
    private void checkForErrorAndThrow(HttpEntity entity, HttpResponse response) throws IOException, ClickHouseException {
        if (response.getStatusLine().getStatusCode() != HttpURLConnection.HTTP_OK) {
            InputStream messageStream = entity.getContent();
            byte[] bytes = Utils.toByteArray(messageStream);
            if (properties.isCompress()) {
                try {
                    messageStream = new ClickHouseLZ4Stream(new ByteArrayInputStream(bytes));
                    bytes = Utils.toByteArray(messageStream);
                } catch (IOException e) {
                    log.warn("error while read compressed stream {}", e.getMessage());
                }
            }
            EntityUtils.consumeQuietly(entity);
            String chMessage = new String(bytes, StandardCharsets.UTF_8);
            throw ClickHouseExceptionSpecifier.specify(chMessage, properties.getHost(), properties.getPort());
        }
    }

non-existing database：
response.getStatusLine().getStatusCode() = 404, and throw a ClickHouseException, and the druid connection pool will catch this exception. (btw, I used druid connection pool)
no privilege database：
response.getStatusLine().getStatusCode() = 404, and it won't throw any Exception
I'm not sure if this two errors should be captured in the same place, but it could explain the case i met.


[BUG] Reports are truncating values in Pdf display

What is the bug?
Download reports is causing the legend and certain reports to have truncated values, this is due to the ingestion of data from tesseract

Steps to reproduce the behavior:

Add Sample data for Flights
Click on reporting and Create report
Select the type as PDF and click on visualizations
Click create

Current workaround for this it to use reporting-cli

+1, any updates on this?


Hi grip looks nice! Will there be more documentation and examples?

I think the documentation is enough at the moment, for a bit more in detail look you can review the Kemal framework documentation


Travis: use jruby-<IP_ADDRESS> in CI matrix

This PR updates the CI matrix to use latest JRuby.

Thanks. Btw, is there any significant change between these JRubies?

This last one had two regressions fixed.

On 26 May 2017, at 17:53, Marcin Kulik<EMAIL_ADDRESS>wrote:
Thanks. Btw, is there any significant change between these JRubies?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I see, cool :)


Pre-Compiled Version?

You have instructions for both node and rails, but do you have a pre-compiled version of TL for a basic HTML site? I guess I could install node an compile it myself, but I feel like it has to exist already.

Yep! You can find the latest release version at this URL:
https://raw.githubusercontent.com/turbolinks/turbolinks/master/dist/turbolinks.js

Awesome, thanks for a fast response.

For anyone coming here via Google, you can switch "master" in the URL to the tag/version number you wish to lock to, e.g. https://raw.githubusercontent.com/turbolinks/turbolinks/v5.0.0/dist/turbolinks.js


Update to CycloneDDS 0.8.x release branch in Galactic

See ros2/ros2#1229

I take that no re-release is required ... ?

I think we should probably update to the official 0.8.0 tag, but it will require manual buildfarm intervention due to some reprepro nuances. I'll coordinate that happening.
No re-release is required, at this stage though.


Configuration conflict between template and plugin from plugin library

We're trying to test beta version of Conversational Insights. For that we had to install "Dual channel recording" plugin from plugin library in Flex admin:

However after redeploy of our custom plugins with this template we noticed that "Dual channel recording" plugin was disabled, because it's disabled in the repository configuration. So this means plugins from the library share configuration with template plugins.
Is there a way to use plugins from the library together with this template? How could I split their configuration in this case?

Hi @RusinovIG! Yes, unfortunately currently some plugin library plugins share the configuration with the template. When running the template, we recommend disabling the plugin library counterparts and instead using the template features, as the versions available in the template are usually more up-to-date and allow you to manage configuration more holistically.
We do have future plans to resolve these conflicts, but unfortunately that is pending some plugin library and Flex UI improvements.

Hi @dremin! Thanks for the answer. Ok, I'll check on the other side, if plugin from template could be used with "Conversational Insights" feature somehow.


Add the stop-id parameter

This fixes #1

Coverage remained the same at 80.0% when pulling ec4a1440a551857587f851e1dfe5b8e78cd8a6f2 on add-stop-id-parameter into dd73022fe9baa51b590098899b65ff271c4bd48f on master.

Coverage remained the same at 80.0% when pulling ec4a1440a551857587f851e1dfe5b8e78cd8a6f2 on add-stop-id-parameter into dd73022fe9baa51b590098899b65ff271c4bd48f on master.


uptime endpoint?

Noticed Jason's sys/clock updates and was reminded of this thread I made a few years ago on a way to query the uptime of a BIG-IP system over REST: https://devcentral.f5.com/questions/get-uptime-of-a-system-over-rest
Would this be something that could get added to the API? Do newer versions of REST API have a feasible way to obtain the uptime? Would it make sense to get it over the SOAP API?
Thanks.

@vsnine I'm not aware of such an endpoint. The method you have there would be the recommended way

@vsnine the endpoint for that does not exist as far as I can see, so using tmsh over rest would be your best bet here. closing


Add global shortcuts implementation

Draft because I don't have the setup to test it yet, and it needs some better docs.
EDIT: Turns out that it also wrongfully assumes that there is only one Vec<Shortcut> type, while in reality there is two, one for defining new shortcuts and one for reading them, so this is also TODO.

Thanks a lot for taking the time to implement this!

I am looking at doing a new beta in the upcoming days and wonder if you would have the time to update the PR by then? otherwise we can merge it as is and I would push other commits on top of it to finish the implementation/docs

Type mismatch with Vec<Shortcut> described in initial description is now fixed. So as far as functionality goes, it should be good now.


Wsl2 Ubuntu 18.04 - the virtual machine or container exited unexpectedly

Environment
Windows build number:  Win32NT 10.0.20226.0 Microsoft Windows NT 10.0.20226.0
Your Distribution version: Ubuntu 18.04
Whether the issue is on WSL 2 and/or WSL 1: WSL2

Steps to reproduce

I have problem with wsl2 ubuntu 18.04 start. Last two builds (20211, 20226) were broken due  remote procedure call failed. That should be fixed in new build 20226. But now my existing wsl2 ubuntu18.04 throws new error the virtual machine or container exited unexpectedly [process exited with code<PHONE_NUMBER>]
That seemed suspicious, so I tried clean install of ubuntu 20.04 from Microsoft Store and that works.
Expected behavior
Run existing wsl2 distro ubuntu 18.04 that I've been using for last few months.
Actual behavior
the virtual machine or container exited unexpectedly [process exited with code<PHONE_NUMBER>]
*note: this error message is translated by me. I have localized windows, I don't know its english equivalent.

I have the same issue (same localized Windows, same RPC Error, which makes me disable DNS Client) How did you solve this? And I think this issue is worth being reopened.


TCL: Remove TCL dependency from TSControl and EventControl

Looks like there is an issue. I'll look it up.

Cherry picked to 8.1.0


Feature/214/add-firebase-analytics

🔍Add Firebase Analytics to track how the website is used #214
🔍 What's Included (OLD)

Implemented Firebase Analytics to gather insights on user interactions within the website.
In Analytics.tsx, initialized Firebase Analytics.
Updated SocialIcons.tsx to log clicks on social media icons, capturing the specific platform clicked.
Updated Contact.section.tsx with analytics event logging for subscription button clicks, tagged with subscribe_button_click.
- Modified Landing.tsx to track page views on component mount and scroll percentage milestones (every 5%)
first_visit, session_start enabled on GA and handled by default.

🗒TODO

[x] Replace 5% milestone scroll tracking with anchor points the website ("seen_faq", "seen_about", etc)
[x] Add an analyticsEvent for each sponsor button
[x] Add an analyticsEvent for the MLH banner
[x] Add an analyticsEvent for the application dashboard button

🗒 Notes

Make sure environment variables for Firebase configuration are added in .env for testing and deployment.
We may want to customize our reports on GA to better represent the events collected.

📁 Files Affected:

src/components/Analytics.tsx
src/components/SocialIcons.tsx
src/components/sections/Contact.section.tsx
src/pages/Landing.tsx

WOOHOO CONFLICTS!!!


nuxt-link-active is case sensitive

Versions

nuxt: 2.15.6
node: 15.12.0

Problem

Example code

<template>
  <div>
    <NuxtLink to="/">Home</NuxtLink>
    <NuxtLink to="/about">About</NuxtLink>
  </div>
</template>

<style>
.nuxt-link-active { color: #8a1ec9 }
.nuxt-link-exact-active {  color: #ddc77f }
</style>

the styles for nuxt-link-active will show if you go to this route: /about
but it won't work if you go to: /About or /aBoUt
What is Expected?
links should get the nuxt-link-active class regardless of the word-casing of the route
What is actually happening?
routes with different caseing will not activate the nuxt-link-actice styles

also apparently the nuxt-link-active class won't be added if there are any url queries in the web address
example for this link -> /about
will match: /about
won't match: /about?tab=1
Note: to make changes take place you need to refresh the page.

This is an upstream issue in vue-router. If you're still experiencing this, please do file an issue in https://github.com/vuejs/router.


slack-vitess-r14.0.4-dsdefense: add retries to flaky ci, run self-hosted on Ubuntu

Description

Add retries to flaky CI tests
Run (disabled) self-hosted checks on ubuntu-20.04

Related Issue(s)

Checklist

[ ] "Backport to:" labels have been added if this change should be back-ported
[ ] Tests were added or are not required
[ ] Did the new or modified tests pass consistently locally and on the CI
[ ] Documentation was added or is not required

Deployment Notes

Merging CI-only change without review


Demo Broken

The demo is currently broken, probably because the repository owner name has changed (miguelao => yellowdoge).

Oh rats! Let me fix this...

Should be fixed!


Problems with exported geometry in gltf format (pc reception desk model)

Godot version:
Godot 3.0.6
OS/device including version:
Windows 10/Acer Aspire E 15
Issue description:
Let's take a look at this PC model (.blend file):
pc_reception.zip
Now let's export it into GLTF format using https://github.com/KhronosGroup/glTF-Blender-Exporter
pc_reception_gltf.zip
When viewing this model online via https://gltf-viewer.donmccurdy.com/ is looks okay:

But when exported to Godot, the result is very strange:

Using apply transforms, I was able to get it to import. I also set the units to cm and baked transforms.
<DichotomyMatt>
You can apply scale and rotation before you export the model. Just click Ctrl-A in object mode. I hope this is what you are asking for.
https://blenderartists.org/t/bake-transform-into-vertices/411086
This is 2.8 blender file.
pc_reception.zip
Converted as fbx and glb (gltf).
issue.zip

The normals are also flipped, I can change the cull mode, but that's not the correct way.

If you're not wedded to gltf, perhaps try the blender-to-godot exporter:
https://github.com/godotengine/godot-blender-exporter

@dodgyville , yeah, we will probably do this, thanks.
As a matter of fact, we have all possible model formats in our project. We start with dae, because it is oldest and most stable. If we encounter any problems with dae, we are trying to use gltf or escn.
We enjoy using Godot, but all this mess with model conversion is really REALLY frustrating :(
I'm trying to share such problem models with the community, because I believe that this can help the developers to fix the bugs in the converters and/or in the Godot :)

@fire , thanks for your help, too!
My idea was: if something is looking OK in Blender, then it should be easily exported to Godot.
Hopefully, something can be done on the Godot side too, to improve this...


Add python-opencv-pip

See https://pypi.org/project/opencv-python/

Two observations here:

Do you really want to add a Python2 key?
Do you really need the -pip key?  There is the debian package key right above, which is more useful for the buildfarm.

Neither question above prevents this from going in, but I'm curious on what the use-case here is.

No response in over a month.  Closing, but feel free to reopen and answer the questions above.


Investigate rclone bucket emptying delay

This issue was solved by reverting partially the change the introduced the regression, but the exact causes are still unknown.

https://github.com/iterative/terraform-provider-iterative/issues/452
https://github.com/iterative/terraform-provider-iterative/pull/453

The bucket emptying method introduced with #420 returns almost immediately, but files take a few minutes more to disappear. I suspect that rclone is using some trickery to empty the bucket in the server side (e.g. expiration time) and returns as soon as the deletion begins.
Can be reproduced with v0.10.0 in a storage.workdir filled with files:
mkdir --parents stress/{1..10}/{1..10}/{1..10}
yes | head | tee stress/{1..10}/{1..10}/{1..10}/file

I just got #452 on v0.10.0 despite having just 1 file (no folders) to sync (fixed by v0.10.3)


Pre-analysis data schema & validation

Pre-analysis data schema & validation
Over the last week, the HCA DCP and Gut Cell Atlas about whether it would be possible to use the schema for earlier, pre-publication data sharing and standardization within their networks. The use case is the same for both groups:

As a consortium, we want to encourage our teams to format their data matrices in a standard way, but groups often lack analysis results at the stage that they want to share data, so cannot generate data that adheres to the cellxgene schema.

We want to encourage the ecosystem to adopt a standard schema. If the single cell matrix schema does not fulfill this use case, toolchains and consortia that need to handle data pre-publication will not have access to single cell matrix schema validation tooling, and will not be able to use this schema to standardize their data until it is ready for publication.
I came up with a potential approach that would work (and which includes an analysis of changes required to cellxgene products) to illustrate, and am interested in feedback.
Potential solution

The schema is partitioned into
a. Base module
b. Visualization module
Validation is expanded to validate:
a. Base module
b. Base + Visualization module
Portal updated to utilize validation option 2.b (no changes to data entering portal)

Changes to schema
X (matrix data)

"strongly recommended" processed results eliminated from Base module. Processed results are REQUIRED in visualization module.

obs (cell metadata)

CL terms move to visualization module

obsm

required embeddings move to visualization module

uns

X_normalization moves to visualization module

Problems:

AnnData's decision to use .X as the required and default data slot make it very challenging to consistently represent raw data only (raw in .X) and raw + processed data (raw in .X.raw and processed in .X). Any user adding processed data to this schema needs to remake their AnnData object.
Theoretically, we currently accept RNA-seq data that includes only raw data, and we let that be visualized in the explorer. This is an almost universally bad experience. To address these cases, In the future, cellxgene could accept and enable download but NOT visualization of data that validates only the Base Module. Or, we could reject these cases.

@BAevermann - I'll let you close this issue if it will not be used further.

Closing @BAevermann - reopen if needed.


Global recheck time setting

Version: 0.36
Installation: Docker on Raspberry pi 4, linux/arm/v7
Browser: Firefox on Windows 10
I am not sure I understand the usage of "Maximum time in minutes until recheck" under main Settings.
I have set it to 5 minutes, but the recheck is done according to each individual page setting. The default setting for the individual pages seem to be 180 minutes and pages are checked every 180 minutes unless I change the individual setting.
Example below.
The main setting is set to 5 minutes.
Only the first page is updated separately to be checked every 7 minutes. That seems to work.
The remaining pages are checked every 180 minutes (default setting).
The last page is newly added, after setting the main recheck to 5 minutes. I would have guessed it would be set default to 5 minutes, but it is still set to 180 minutes

Version: 0.36
Installation: Docker on Raspberry pi 4, linux/arm/v7
Browser: Firefox on Windows 10
I am having the same issue with my global "Maximum time in minutes until recheck" setting being ignored. It seems to have been broken in the latest release as it was working previously.
Setting the "Maximum time in minutes until recheck" individually as a workaround seems to be working.

Hi, i see it..
I added some tests so it doesnt happen again (I hope!), and some UI cleanups
When you enter a custom time

When the watch is running on a default time

And global settings page has a better field comment

unfortunately this wont apply to your existing entries because they will already have had a value entered, but should fix all new ones :) just set their fields to blank to follow the defaults.
(maybe a function to set all back to default?)
building in a branch now

Thanks! This solves the issue

@roupenig anytime! remember, i write this application in my spare time when i should be working more hours for actual money and paying my rent and food, if you like it, please consider the BTC link on the README.md!


New dropdowns for Name, Label and Unit

Updated the ingredient name field to now use a dropdown and display options from an array.
Updated the label to to use a dropdown in a similar way.
Added a new Unit field and made it a dropdown.
TODO: Decide on how to display these units for item details.

added the csv that it's supposed to parse ingredients from.
i also defaulted the unit variable to count. I can revert that if that's not what we wanted, I just thought it may have gotten missed.
LGTM - Merging.


Enable go modules

Go modules are becoming a de-facto standard in the Go community and adding go.mod files with metadata helps projects which may consume this repository as a dependency in deciding how to pin transitive dependencies and how to resolve conflicts.
This is a result of the following commands:
go mod init
go get ./...
go mod tidy

in a clean Go 1.11.5 environment.

Coverage remained the same at 100.0% when pulling 8a1fbf18aa95e9c1f4cf90a7c32affc43e9397f4 on radeksimko:go-mod into 5f10fee965225ac1eecdc234c09daf5cd9e7f7b6 on agext:master.

Thank you, Radek.

Thanks for a quick review & merge @alex-alrux .
It would be great if you could cut v1.2.2 with this patch as Go Modules strictly follow semantic versioning and the new version (with this patch) otherwise isn't discoverable via go get -u and other ways.

Will do really soon. I'm hoping to include a few more changes (mostly nitpicks), but if I don't get to that in a day or so, I'll cut the v1.2.2 as-is.


include parameter files for monsoon_wang and diurnal_covey

@doutriaux1   OK I've created a branch (535_pjg_monsoon_diurnal_parameterfiles) with a test parameter file (examples/parameter_files/test_inparams-mpindex_compute.py).  This points to data on oceanonly.... better would be to automatically download file or use readily accessible test-case data.   I will add an example for the diurnal next.

@doutriaux1   There is now also a test parameter file for the first diurnal code.   This works on data avail on crunchy.   Like the monsoon example mentioned above (same branch and subdir), the test param for diurnal would be good to be "self contained", ie., somehow able to run with the test data we have in the repository, or perhaps hanging off a website.

@doutriaux1  We should discuss whether or not these test_parameter files rely on the small test data on the repo or if we pull down larger data on the fly to provide a more informative test case

@gleckler1 when we have tests relying on bigger dataset I'll add them to the suite, but skip them on travis

@doutriaux1  Right.   So your task with this branch ((535_pjg_monsoon_diurnal_parameterfiles) is to set up the parameter files I've added to work with the test suite.

@doutriaux1 currently the tests in this PR point to data in crunchy.   Can you fix them to point to the test data suite so that a user can run them?

@doutriaux1  diurnal param file no longer working.... changes to dask since you wrote this last September?
(pmp_0808_py2.7) bash-4.1$ compositeDiurnalStatisticsWrapped.py -p test_inparams_compositeDiurnalStatistics.py
done
MODPATH: /work/cmip5-test/new/amip/atmos/3hr/pr/
TEMPLATE: cmip5.IPSL-CM5A-LR.amip.r1i1p1.3hr.atmos.3hr.pr.000000.v*.xml
FILES: ['/work/cmip5-test/new/amip/atmos/3hr/pr/cmip5.IPSL-CM5A-LR.amip.r1i1p1.3hr.atmos.3hr.pr.000000.v20110427.xml']
PARAMS: [<pcmdi_metrics.diurnal.common.INPUT object at 0x7fd4cc4c9810>]
/export_backup/gleckler1/anaconda2/envs/pmp_0808_py2.7/lib/python2.7/site-packages/dask/context.py:23: UserWarning: The dask.set_options function has been deprecated. Please use dask.config.set instead
warnings.warn("The dask.set_options function has been deprecated. "
/export_backup/gleckler1/anaconda2/envs/pmp_0808_py2.7/lib/python2.7/site-packages/dask/base.py:835: UserWarning: The get= keyword has been deprecated. Please use the scheduler= keyword instead with the name of the desired scheduler like 'threads' or 'processes'
warnings.warn("The get= keyword has been deprecated. "
Traceback (most recent call last):
File "/export_backup/gleckler1/anaconda2/envs/pmp_0808_py2.7/bin/compositeDiurnalStatisticsWrapped.py", line 226, in
cdp.cdp_run.multiprocess(compute, params, num_workers=args.num_workers)
File "/export_backup/gleckler1/anaconda2/envs/pmp_0808_py2.7/lib/python2.7/site-packages/cdp/cdp_run.py", line 21, in multiprocess
results = bag.map(func).compute(num_workers=num_workers)
File "/export_backup/gleckler1/anaconda2/envs/pmp_0808_py2.7/lib/python2.7/site-packages/dask/base.py", line 156, in compute
(result,) = compute(self, traverse=False, **kwargs)
File "/export_backup/gleckler1/anaconda2/envs/pmp_0808_py2.7/lib/python2.7/site-packages/dask/base.py", line 402, in compute
results = schedule(dsk, keys, **kwargs)
File "/export_backup/gleckler1/anaconda2/envs/pmp_0808_py2.7/lib/python2.7/site-packages/dask/multiprocessing.py", line 177, in get
raise_exception=reraise, **kwargs)
File "/export_backup/gleckler1/anaconda2/envs/pmp_0808_py2.7/lib/python2.7/site-packages/dask/local.py", line 505, in get_async
raise_exception(exc, tb)
File "/export_backup/gleckler1/anaconda2/envs/pmp_0808_py2.7/lib/python2.7/site-packages/dask/local.py", line 274, in execute_task
result = _execute_task(task, data)
File "/export_backup/gleckler1/anaconda2/envs/pmp_0808_py2.7/lib/python2.7/site-packages/dask/local.py", line 255, in _execute_task
return func(*args2)
File "/export_backup/gleckler1/anaconda2/envs/pmp_0808_py2.7/lib/python2.7/site-packages/dask/bag/core.py", line 1554, in reify
seq = list(seq)
File "/export_backup/gleckler1/anaconda2/envs/pmp_0808_py2.7/lib/python2.7/site-packages/dask/bag/core.py", line 1714, in map_chunk
yield f(*a)
File "/export_backup/gleckler1/anaconda2/envs/pmp_0808_py2.7/bin/compositeDiurnalStatisticsWrapped.py", line 39, in compute
reverted = template.reverse(os.path.basename(fileName))
File "/export_backup/gleckler1/anaconda2/envs/pmp_0808_py2.7/lib/python2.7/site-packages/genutil/Filler.py", line 170, in reverse
raise "Invalid pattern sent"
TypeError: exceptions must be old-style classes or derived from BaseException, not str

is it new env?

@doutriaux1 just created with conda create -n pmp_0808_py2.7 -c cdat/label/nightly -c conda-forge -c cdat -c pcmdi/label/nightly -c pcmdi cdat pcmdi_metrics python=2.7

@doutriaux1  Need to get this one working again before release.   Seems like the dask stuff is still a moving target.   the --bash option you setup for clim-cmor is a functioning temporary solution.  I'm not sure if the better thing to do with diurnal (via param file) is to fix it or to overhall it to use parallelize_driver.py

on it should be done tomorrow (finger crossed)


Discussion: expect(object).toInclude(otherObject) - should use tmatch?

tmatch does a better job of deeply checking for a partial object in another.
Specifically with:

matching arrays and nested objects in arrays.
comparing dates
matching RegExp to a string

Should we change the code (as I PRed @ #80) to use it?
tmatch(
  {a: 1, b: 2, c: {d: 4, e: {f: 5, g: 6}, h: 7}, i: [{j: 8}, {k: 9, l: 10}], m: 'hello' },
  { b: 2, c: {d: 4, e: {g: 6}}, i:[ {j: 8}, {k: 9} ], m: /ll/ }
)

A PR that adds test cases that pass with tmatch but fail with the current implementation would be a good start.

We already have a toMatch method. Maybe we should augment that with tmatch-style support instead of further modifying toInclude? IMO "include" methods should check the expected value is a subset of the actual value, respecting types. "match" methods make it more obvious that we support things like RegExp matches, ala String.prototype.match.


refactor(client): unify create dataset and deprecate create_auth_dataset

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 6 (83.33%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 86.705%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tensorbay/client/gas.py
4
5
80.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.4%

Covered Lines:
6078

Relevant Lines:
7010

💛  - Coveralls


Maint/287 maintenance migrate from unplugin vue cesub style to vue 35 features

Description

Removed now unused dependencies because now part of Vue 3.5.x
Updated vite configuration

Reference
Issues #287

I tested this by including custom CSS in .ce components and normal Vue SFCs. Both custom CSS rules as well as the patternlab imports where included in the generated entrypoint file. Should be fine like this. :)


MPQ Editor: Bitmap Viewer crashes with fresh MPQ

MPQEditor <IP_ADDRESS>2, <IP_ADDRESS>8 Unicode
StormLib 9.24

Create Empty MPQ
Game profile: Warcraft III Maps
max number files: 4000
Go to Tools, open MPQ Bitmap
It will create a very long list
Scroll Down and crash

However when you save and reopen the same empty MPQ archive, it will show one blue (present) block only without crashing.

I'll see the crash, will fix MPQEditor a.s.a.p. Thanks for the heads up.

MPQ Editor fixed & available on my website.


jh names and tavern names

Added variety to names,json and taverns.json

Looks good to me, merging! 👍


MM-17119: Updates LDAP id attribute config help text to outline optio…

…n for Active Directory.
Summary

Updates help text for LDAP id attributes to outline a valid option for AD.
Updates Makefile regex to include i18n-extract in make help.

Ticket Link
https://mattermost.atlassian.net/browse/MM-17119

@mkraft Please help cherry-pick to v5.15.

@amyblais Cherry-pick complete.


Show contramap/etc

?

https://github.com/samhh/fp-ts-std/commit/3c6929bd269bd95ec039202e6b7c2644fcfe935f


WIP: Add development guide doc for dotnet & xamarin

Example project code links in Articles are not available yet. Please use following links for example code as PRs are under review from respective branches.

Desktop/Console app branch link: https://github.com/ravinderjangra/safe-getting-started-dotnet/tree/console-app
Xamarin.Forms app branch link: https://github.com/ravinderjangra/safe-getting-started-dotnet/tree/xamarin-app

Xamarin is still available to select

@nadiaburborough Please review the final content. I'll update the PR once discussed with @krishnaIndia.

Closing this and separate PR's will be raised for mobile and desktop.

Closing this and separate PR's will be raised for mobile and desktop.

Closing this. Separate PRs will be raised for mobile and desktop app developer using dotNet/Xamarin.

Closing this. Separate PRs will be raised for mobile and desktop app developer using dotNet/Xamarin.

Closing this. Separate PRs will be raised for mobile and desktop app developer using dotNet/Xamarin.

Closing this. Separate PRs will be raised for mobile and desktop app developer using dotNet/Xamarin.

Closing this. Separate PRs will be raised for mobile and desktop app developer using dotNet/Xamarin.

Closing this. Separate PRs will be raised for mobile and desktop app developer using dotNet/Xamarin.

Closing this. Separate PRs will be raised for mobile and desktop app developer using dotNet/Xamarin.


Add screenshots for readme

Main screen - Add new run tab

Main screen - Compare runs tab

Compare Multiple Runs screen


How to add Angular?

I'm new to MAUI but proficient in Angular. I need to access USB, Bluetooth, and NFC. I want to take my existing Angular project and add it to the resources/Raw folder. Can you guide me on how to get Angular running with MAUI. Do I need to run the ng server before I debug? I'll figure it out but maybe you have the answer already. Thanks.

What have you already tried? This issue implies that the main repo's code can't access external resources (although if you want to use the fork in that issue, you should be able to point it at your angular dev server), so with the main repo you would probably need to actually build your app with some subdirectory of Resources/Raw as the target in order to run it.

@sibiren-spins Thanks for mentioning that link. I was able to get the Angular app bundled as a microfrontend using what I learned here: https://blog.bitsrc.io/diving-deep-into-angular-elements-c17a868d6894
However, now the issue I am facing is the inability to route from one component to another. I'm guessing it's how the web element was created:
`export class ElementModule implements DoBootstrap {
constructor(private injector: Injector) {
}
ngDoBootstrap(appRef: ApplicationRef) {
if (!customElements.get('home-mfe')) {
const homeComponent = createCustomElement(HomeComponent, {
injector: this.injector
});
customElements.define('home-mfe', homeComponent);
}
}
}
`
but so far so good, I'm able to communicate from Angular to C#. I just need to figure out what I am doing wrong to communicate the other way around - C# to Angular.


Allow selecting remote branches when creating worktrees.

Amazing! Good work on executing the seemingly impossible.

see a list of worktrees
selecting a parent folder
allowing for worktree folder name to be different from branch name

One feature I'd ask for is that instead of only showing locally checked out branches as candidates for creating new worktrees from, also include remotes too. 🙏🏻

Thank you so much for your feedback! That is a great suggestion and something I overlooked. I've just released a new version that hopefully addresses this. v1.0.1 If you have any issues or other suggestions please let me know.

Legend 🦾 Great stuff!


feature: structured type declarations

Currently, annotations for 3rd party modules are provided as code:
-- file: [...]/lib/foo.lua

--- Foo objects are...
---@class foo
---@field name string
---@field attr string this is an attribute
---@field enabled boolean
local foo = {}

--- Options for creating a new foo object
---@class foo.new.opts : table
---@field count number
---@field enabled boolean

--- Updates foo.attr to the given value
---@param value string the new value
---@param force boolean ignore errors when changing the attr
---@return boolean ok whether or not the action succeeded
---@return string? error on failure, an error string
function foo:set_attr(value, force) end

--- Copies the foo object to a new one
---@param opts? foo.new.opts
---@return foo
function foo:copy(opts) end

return {
  --- Instantiate a new foo object
  ---@param opts? foo.new.opts
  ---@return foo
  new = function (opts) end,

  ---@type table<string, foo>
  registry = {},

  ---@type foo[]
  queue = {},
}

What if the language server provided a way of submitting annotations in a structured format that was much closer to the internal representation of the code?
For instance, here's how these annotations could look in a json sort of format:
{
  "name": "foo",
  "type": "module",
  "module": "lib.foo",
  "fields": {
    "new": {
      "type": "function",
      "description": "Instantiate a new foo object",
      "params": [
        {
          "type": "foo.new.opts",
          "default": null,
          "name": "opts",
          "optional": true
        }
      ],
      "returns": [
        { "type": "foo" }
      ]
    },
    "registry": {
      "type": "table",
      "table.keys": { "type": "string" },
      "table.values": { "type": "foo" }
    },
    "queue": {
      "type": "array",
      "array.values": { "type": "foo" }
    }
  },
  "types": {
    "foo": {
      "type": "class",
      "parent": "table",
      "description": "Foo objects are...",
      "fields": [
        {
          "name": "name",
          "type": "string"
        },
        {
          "name": "attr",
          "type": "string",
          "description": "this is an attribute"
        },
        {
          "name": "enabled",
          "type": "boolean"
        },
        {
          "name": "set_attr",
          "type": "method",
          "params": [
            {
              "description": "the new value",
              "name": "value",
              "type": "string"
            },
            {
              "default": false,
              "description": "ignore errors when changing the attr",
              "name": "force",
              "optional": true,
              "type": "boolean"
            }
          ],
          "returns": [
            {
              "description": "whether or not the action succeeded",
              "name": "ok",
              "type": "boolean"
            },
            {
              "description": "on failure, an error string",
              "name": "error",
              "optional": true,
              "type": "string"
            }
          ]
        },
        {
          "name": "copy",
          "type": "method",
          "params": [
            {
              "type": "foo.new.opts",
              "name": "opts",
              "optional": true
            }
          ],
          "returns": [
            {
              "type": "foo"
            }
          ]
        }
      ]
    },
    "foo.new.opts": {
      "type": "class",
      "fields": [
        {
          "name": "count",
          "type": "number"
        },
        {
          "name": "enabled",
          "type": "boolean"
        }
      ]
    }
  }
}

why?

make it easier to generate type definitions dynamically and programmatically because one doesn't need to write a template that generates lua code.
reduce startup cost for the language server since it doesn't have to parse any lua code to load types

I uses this method in 0.X and 1.X versions, but finally abandons this method:

The parser needs to support two sets of parsing rules at the same time, and will invade the business.
The user cannot jump to the meta file by go to definition.
The format will have potential compatibility problems.

In the version at that time, the advantage of this method is that it has faster parsing speed and relatively static cache.
But in the design of 3.X, lua files have static cache data as much as possible, so the only advantage is faster parsing speed.
The file parsing is only carried out once when the project is opened, and the advantage of parsing speed can not offset the above disadvantage.
As for the difficulty of generating lua code, as long as we can negotiate a format, I can easily provide a generation tool.

@sumneko thanks for your reply and explanation :) I understand your reasoning

As for the difficulty of generating lua code, as long as we can negotiate a format, I can easily provide a generation tool.

This would definitely be appreciated, though I wouldn't ask you to spend any special effort on it.


Password generator

Hello,
nice tool.
It would be cool to have a simple password generator like the UUID generator.
Something like this: http://passwortgenerator.org/index.php
In the preferences you can set the length and the characters to be used in the password.

Going to close this. Generating truly secure passwords would require more than just extracting secure bytes from a hardware chip and there are other apps that would do this better than Sol.


streamdeck-xl: cannot open device on linux

On linux companion can see the device, but fails to open it. This works fine on windows.
See issue https://github.com/bitfocus/companion/issues/677 for a better description. It is the same case, it tries to use hidraw3 to connect and fails.

Have you setup the udev rules? https://github.com/bitfocus/companion-satellite/blob/master/pi-image/50-satellite.rules

The beta builds now include a README and udev rules file in the linux build. Sourced from https://github.com/bitfocus/companion-satellite/tree/master/assets/linux
This will be in the 1.5.0 release


Issue/2379 facts overview

Description
closes #2379
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Code is clear and sufficiently documented
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Processing this pull request

An unexpected error occurred while processing this pull request. The request might have been partially processed. Caused by: Remote state does not match local state: got pull requests for branches master but change entry specifies branches master, iso5.
If this is a first stage pull request (not opened by the merge tool) this is probably caused by a race condition (inmanta/irt#650). The best way to resolve this is to make sure the merge tool can pick up this pull request again and start from a clean slate. To do this, take the following steps:

Close all pull requests opened by the merge tool with the same source branch as this one. Be careful not to close pull requests for source branches that start with merge-tool/. All pull requests created by the merge tool link to this pull request so you should be able to find them mentioned here.
Re-add the merge-tool-ready label to this pull request.

An unexpected error occurred while processing this pull request. The request might have been partially processed. Caused by: [Errno 2] No such file or directory: 'yarn': 'yarn'

Processing this pull request

An unexpected error occurred while processing this pull request. The request might have been partially processed. Caused by: [Errno 2] No such file or directory: 'yarn': 'yarn'

Processing this pull request

Merged into branches master, iso5 in c6c0f3f0b5ec4e863d12b43d9c97da0b7c01cb97


CIAO CSS @import error

Looks like not allowing import from google fonts?
Child extract-text-webpack-plugin:

    ERROR in ./~/css-loader!./~/sass-loader/lib/loader.js?{}!./src/assets/assets.css
    Module build failed: Unexpected formathttps://fonts.googleapis.com/icon?family=Material+Icons (1:1)

    > 1 | @import https://fonts.googleapis.com/icon?family=Material+Icons;
        | ^
      2 | @font-face {
      3 |   font-family: 'Roboto';
      4 |   font-style: normal;

@patrikx3 { loader: 'css-loader', options: { import: false } } 😛

thanks dude!

Worked ?

hello, i added it in, but now it just remove the
@import "https://fonts.googleapis.com/icon?family=Material+Icons";
and i want it :)
how can i keep it in?


Allow users to set up a special alert when their Monitor is erroring

 Issue by Jon-AtAWS
Wednesday Apr 03, 2019 at 01:03 GMT
Originally opened as https://github.com/opendistro-for-elasticsearch/alerting/issues/21

Current behavior is to send the user's alert text when their monitor or trigger is in the "Error" state. This is confusing, since I don't know whether the problem is captured by the data or whether the problem is evaluating the trigger.
I need some way to have Open Distro alert me that there's a problem in my setup.

 Comment by ghost
Friday Apr 05, 2019 at 04:51 GMT

The context passed to your scripts (both trigger script and action script) has details of the error in the ctx.error field.  You can send a different alert text if there is an error by writing your mustache template as follows:
{{^ctx.error}}
  The action ran successfully!
{{/ctx.error}}
{{#ctx.error}}
   Action failed. Here is the error message: {{ctx.error}}
{{/ctx.error}}

When evaluating a trigger script (painless) the ctx.error variable will contain the error capturing data if there was one.  When evaluating an action script (mustache) ctx.error will contain either the error capturing data or the error evaluating the script (in that order).

 Comment by Jon-AtAWS
Friday Apr 05, 2019 at 21:06 GMT

OK, I get it that I can write a conditional in my script. However, I have to know that I can do that.
More than that, I would like separate notifications for errors in my monitoring system. The team that responds to those issues could be different than the team that responds to the monitored condition.

 Comment by ghost
Sunday Apr 07, 2019 at 05:33 GMT

OK, I get it that I can write a conditional in my script. However, I have to know that I can do that.

We went back and forth on having separate actions for errors vs. using the inline conditional in the script. Ultimately, we decided to go with the inline conditional as it provides flexibility to programmatically implement different policies vs. a pre-defined error notification.   I agree that this needs to be better documented (perhaps in a blog or wiki post?)
/cc @aetter and @elfisher to explore how we can better surface this technique in documentation and UI.

More than that, I would like separate notifications for errors in my monitoring system. The team that responds to those issues could be different than the team that responds to the monitored condition.

I agree that this should be possible, but currently isn't.  One way to allow for this is suppress notificatinos for actions if they produce no output (right now it throws an error, if there isn't any output).  You can then achieve what you describe by splitting the conditional in the script into separate actions with different destinations for errors vs. normal monitoring.  Place the normal action in a {{^ctx.error}} block and the error action template in a  {{#ctx.error}} block.  /cc'ing @elfisher for his thoughts.

 Comment by ghost
Tuesday Apr 09, 2019 at 04:16 GMT

ctx.error is mainly useful to deal with errors that are query and trigger specific (for e.g. not enough data, script error etc.).  I'm not sure I understand what you mean by "2 different systems for alerting".
For errors related to the "monitoring system" itself (I assume you're thinking of something like being unable to send notifications or unable to save alerts to the alerting indices, multiple monitors being delayed),  I agree that these would normally be handled by a different team than the monitor creator.  That's why we logs those as errors/warnings to the ES log with the expectation that cluster administrators will be monitoring the logs.  We also have a detailed system status API that can be used by administrators to monitor the overall status of the alerting plugin: https://opendistro.github.io/for-elasticsearch-docs/docs/alerting/api/#monitor-stats .  This is intended to be integrated into whatever external cluster health monitoring is being used (similar to /_cluster/health).  So it's not expected that ctx.error would be used to handle this.
Going back to your original issue: Would having a per-action flag to suppress running the action  if the monitor is in an ERROR state solve your issue?  Under the hood this could just check ctx.error similar to the conditional above...

 Comment by Jon-AtAWS
Monday Apr 08, 2019 at 21:52 GMT

I'd way rather maintain 2 different systems for alerting than maintain 2 different actions per trigger.
That gets really messy if I have even a moderate amount of triggers.


Change playlist page count

This pull request...

[X] Fixes a bug
[ ] Introduces a new feature
[ ] Improves an existing feature
[ ] Boosts code quality or performance

Description
Increases playlist page count for YouTube playlists. Maybe this should be configurable? Is it worth making it configurable, or just set it to a huge number?
Purpose
Allows longer playlists
Relevant Issue(s)
This PR closes issue #49

Note: need to check that this works, and if it does, we should consider making this a config item instead of a hard-coded value


Filter DataGrid by Multiselect ComboBox with LoadData event defined

This block of code (or something similar) needs to be added to the QueryableExtension file to properly handle a multi-select filter. I have seen sample code for multi-select dropdowns filtering data grids, but they do not work. This is because the FilterValue is stored in an IEnumerable but there is nothing in QueryableExtension that handles that in the ToFilterString method. I was able to get my scenario to work by using this code.
This code works with a LoadData event handler that is used to appropriately filter the grid items by the filter string generated across all columns.

Hi @nstuller,
Thanks for the pull request!
I've tried the demo for custom filtering template with multiple selection however I've got the following error:

Happy to help. I have resolved the issue that @enchev pointed out. I also added a workaround because otherwise the multi-select doesn't get it's parameters set until after the reload that occurs in the OnChange method.

Hi @nstuller,
I've reviewed further your code and I've noticed that it will work only for IEnumerable<string> (PropertyAccess.GetElementType(column.GetFilterValue().GetType()) == typeof(string)) and only if the DataGrid is bound using LoadData (ToFilterString() method is used only in this case).
Unfortunately we will unable to accept this pull request. We will do our best to provide built-in filtering support for any IEnumerable<> in some of our next updates together with example.


fix: Fix order of parameters

Fixes the order of parameters. The parameters in the default api are - required params, optional params, depending on order in the spec file. The order in our api is - required path params, required body, required query params, optional path params, optional query params.
After I had already implemented that, I had an insightful conversation with @jjtang1985: We should probably have a parameter object instead of multiple parameters, because if this gets bigger the API gets quite ugly. That way the order wouldn't matter, we would have to only keep it to call the default api.
@FrankEssenberger @florian-richter - I left the reordering for now, but I will be happy to still improve this on this PR.

Hey @FrankEssenberger,
I implemented the variant with the object for parameters now. To be honest, I am not 100% happy with it yet, but I think it will do for this PR. The issue, that I have with the current implementation is, that there is no difference between path parameters, query parameters and the request body.
I could imagine something like this:
args: {
  pathParameters: {
    requiredPathParam: string,
    optionalPathParam? string
  },
  queryParameters: {
    requiredQueryParam: string,
    optionalQueryParam? string
  },
  body: BodyType
}

But then for simple cases the usage might not be optimal. Something in that direction:
fn({ pathParameters: { requiredPathParam: 'test' } })

Hey @FrankEssenberger,
I implemented the variant with the object for parameters now. To be honest, I am not 100% happy with it yet, but I think it will do for this PR. The issue, that I have with the current implementation is, that there is no difference between path parameters, query parameters and the request body.
I could imagine something like this:
args: {
  path: {
    requiredPathParam: string,
    optionalPathParam? string
  },
  query: {
    requiredQueryParam: string,
    optionalQueryParam? string
  },
  body: BodyType
}

I share your concern. For little parameters it is overkill. I think it would be best to resolve name conflicts via some postfix and keep the signature clear. I also hope that conflicts are the exception because it is not a good API if you have two parameters with the same name as path and query. I mean how distinguish them? But for this PR I agree it is good enough.


Add image support plus extra extension

Added an extension for ColorPicker3 to support Strides Color3 type.
Added rendering requirements to display images in ImGui using GCHandle (If there is a better way then please let me know)
This has been tested with both Diirectx and OpenGL and seems to work fine.

Example:
    public SceneInspectorWindow(Scene scene, IServiceRegistry services)
        : base(services)
    {
        Enabled = true;

        string imagePath = "StrideLogo.png";
        using var input = File.Open(imagePath, FileMode.Open);
        _test = Texture.Load(Game.GraphicsDevice, input);

        _texture = (IntPtr)GCHandle.Alloc(_test);
    }

    protected override void OnDraw(bool collapsed)
    {
        foreach (var entity in _entities)
        {
            DrawEntity(entity);
        }

        //StrideImage(_test);
        ImGui.Image(_texture, new System.Numerics.Vector2(_test.Width, _test.Height));
        ImGui.ImageButton("Test", _texture, new System.Numerics.Vector2(_test.Width, _test.Height));
    }

    protected override void Destroy()
    {
        base.Destroy();
        GCHandle.FromIntPtr(_texture).Free();
    }

I was looking into a better way for users to use this feature and I think I have a simpler set up:
public static class TextureExtensions
{

    // Dictionary to hold textures
    private static readonly Dictionary<IntPtr, Texture> _textureRegistry = [];
    private static readonly Dictionary<Texture, IntPtr> _pointerRegistry = [];
    private static nint _count;

    /// <summary>
    /// Gets a pointer to the Texture and adds it to the <see cref="_textureRegistry"/> if it was not previously added.
    /// </summary>
    /// <param name="texture"></param>
    /// <returns></returns>
    public static IntPtr GetPointer(this Texture texture)
    {
        if(_pointerRegistry.TryGetValue(texture, out var pointer)) return pointer;
        _count++;
        _textureRegistry.Add(_count, texture);
        _pointerRegistry.Add(texture, _count);

        return _count;
    }

    /// <summary>
    /// Attempts to convert a pointer to a texture if its in the <see cref="_textureRegistry"/>
    /// </summary>
    /// <param name="pointer"></param>
    /// <param name="texture"></param>
    /// <returns></returns>
    public static bool TryGetTexture(this IntPtr pointer, out Texture texture)
    {
        if(_textureRegistry.TryGetValue(pointer, out texture))
        {
            return true;
        }
        return false;
    }
}

Would this make more sense? I think this would allow the user to not have to worry about memory management if I am understanding things correctly.

Hey, nice work, just a couple of comments on the architecture of this one;
The two static dictionaries used there are never cleared, them being static means that they hold onto the texture objects for the lifetime of the application.
You could just clear those collections right after Draw() to fix this issue.
_textureRegistry doesn't need to be a dictionary since the nint you return is just incremented on each new item. You can use a list, every time you add to the list, use the list's length-1 as the id.
As for TextureExtensions

I would rather you move the methods in TextureExtensions into ImGuiExtension.
Make them internal, if there are parts of the API where users would need this, we should cover that part for them instead of providing them this obtuse method.
And rename GetPointer to something less misleading, they're not pointers to texture content after all

Thanks again for all the work you're putting in

The only problem with using a list is that the index doesnt work for the first item in the list. When I return the first index of zero, ImGui will treat it as a IntPtr.Zero and think there is no texture needed to be drawn and draw a broken texture. The process is a bit odd since ImGui is being used as the middle man from the developer to the renderer I have to pass it through as a valid non zero "pointer" even if it is just a reference to a list or dictionary.
The other thing Im wondering about is do we want to wipe the dictionary every frame? Would it not be better to keep it in the dictionary if the user reuses the same Texture, instead of having to add and remove it every frame?
I think still implemented everything you mentioned apart from the dictionary to a list due to the previously mentioned issue.
And not a problem! I have genuinely been loving working with ImGui when I have been able away from my main PC compared to all of the other UI options. Im hoping I can finally settle on this one and stop switching frameworks lol.

I guess what I could do if the list is still wanted is change the renderer to just check the zero index of the extension instead of just checking if the TextureId is zero.. but that may still be a bit weird if there is a Texture at index zero but it was actually supposed to draw the font texture?
I can play around with it tomorrow if you think this should still be changed.

I think this would work in your case ?
internal static nint GetTextureKey(Texture texture)
{
    if (_pointerRegistry.TryGetValue(texture, out var key)) return key;

    _textureRegistry.Add(texture);
    nint id = (nint)_textureRegistry.Length; // Notice how the value we're assigning here, and returning below cannot be zero, it is always >= 1
    _pointerRegistry.Add(texture, id);

    return id;
}

internal static bool TryGetTexture(IntPtr key, out Texture texture)
{
    int index = ((int)key-1); // Then, following the logic above, we just have to reduce by one here to get the index
    if (index >= 0 && index < _textureRegistry.Length)
    {
        texture = _textureRegistry[index];
        return true;
    }
    return false;
}

The other thing Im wondering about is do we want to wipe the dictionary every frame? Would it not be better to keep it in the dictionary if the user reuses the same Texture, instead of having to add and remove it every frame?

It would be nice to avoid paying the cost of dictionary insertions, sure, but in this case I would rather make sure we don't have out of scope or otherwise unused textures polluting the dictionary lookups. We could employ a weak table instead of a dictionary to avoid holding onto those references, and remove the clear after every draw, but I don't know how efficient those actually are compared to using a dictionary and clearing after draw.

There we go, just needed to come at it with fresh eyes. This should be good and only uses a single list now based on your example.

Looks good, thanks !


On comparing KLEE/Tracer-X with LLBMC

For programs that contain errors, KLEE/Tracer-X have to be run with -exit-on-error because LLBMC only reports the first error it finds.

Also, LLBMC should be run with assembly.ll generated by KLEE.


Sometimes stops reading the text from image, after sometime it starts reading.

I am using the tess-two:9.0.0 in my android project to read the MRZ code. It's working fine, but some time it stops reading the text from image. Then after some time it again starts reading the text.
Environment

Tesseract Version: tess-two:9.0.0
Commit Number:
Platform: android

Current Behavior:  Sometime don't read the text from image.
Expected Behavior: Should read the text every time.
Suggested Fix:

Please respect guidelines for posting issue: we do not provide support for 3rd party projects.


Removing ASP.NET and xunit from the tabs

We are not planning to implement benchmarks for AP.NET or xUnit (at least for now).

We are not planning to implement benchmarks for AP.NET or xUnit (at least for now).

I added Xunit and ASP to test my app config.... Then I forgot to receive before raising the PR 🥲
Sorry for this @marcin-krystianc ...


Parsing keys with spaces into a struct

Hi,
I have a piece of JSON like so:
{
  "Player count": 1234,
  "Games played": 5678
}

How would I go about parsing this into a struct, since struct field names (as far as I can tell), can't contain spaces?
Would I need to use the loosely typed serde mode?

You can use the rename or alias attributes of serde to change how the keys are matched during de/serialization.
https://serde.rs/field-attrs.html#rename

Exactly what I was looking for. Thanks!


Istanbul removes function name in coverage

This:
function square (x) { return x * x; }

Becomes this:
function (args) { var output;

var __cov_LQgId1OfxIM2Vvj2j1AaeQ = (Function('return this'))();

if (!__cov_LQgId1OfxIM2Vvj2j1AaeQ.$$coverage$$) { __cov_LQgId1OfxIM2Vvj2j1AaeQ.$$coverage$$ = {}; }
__cov_LQgId1OfxIM2Vvj2j1AaeQ = __cov_LQgId1OfxIM2Vvj2j1AaeQ.$$coverage$$;
if (!(__cov_LQgId1OfxIM2Vvj2j1AaeQ['.../istanbul/test/instrumentation/test-es6-arrow-fn.js'])) {
   __cov_LQgId1OfxIM2Vvj2j1AaeQ['.../istanbul/test/instrumentation/test-es6-arrow-fn.js'] = {"path":".../istanbul/test/instrumentation/test-es6-arrow-fn.js","s":{"1":1,"2":0},"b":{},"f":{"1":0},"fnMap":{"
1":{"name":"square","line":1,"loc":{"start":{"line":1,"column":0},"end":{"line":1,"column":20}}}},"statementMap":{"1":{"start":{"line":1,"col
umn":0},"end":{"line":1,"column":37}},"2":{"start":{"line":1,"column":22},"end":{"line":1,"column":35}}},"branchMap":{}};
}
__cov_LQgId1OfxIM2Vvj2j1AaeQ = __cov_LQgId1OfxIM2Vvj2j1AaeQ['.../istanbul/test/instrumentation/test-es
6-arrow-fn.js'];
function square(x){__cov_LQgId1OfxIM2Vvj2j1AaeQ.f['1']++;__cov_LQgId1OfxIM2Vvj2j1AaeQ.s['2']++;return x*x;}

return output;
}

It was named square, but after coverage it becomes anonymous

That instrumented code seems to be missing some essential components: output is never set, the instrumented square function is never exported out of the wrapping function, does it even work? Aside from the function name.

I have this same issue when creating snapshots in Jest with enzyme shallow rendering.
say for example I have this:
const MyComponent () => (
<NestedComponent/>
);
When I run with coverage the snapshot renders as
<Component />

When I run without coverage the snapshot renders as
<NestedComponent/>

this because https://github.com/airbnb/enzyme/blob/master/src/Debug.js#L26 default to Component in the specified case

@mikewoudenberg Yeah it's super annoying. Using function statements instead of expressions and arrow functions fixes it, but it shouldn't actually matter how you define your component. Unfortunately this particular issue has existed with istanbul for quite some time now..

Workaround idea. You could enforce using named function using the func-style eslint plugin.
func-style: ["error", "declaration"] should enforce using function declarations.

Any updates on this?

I've recently encountered this issue and I tried to fix it on the istanbul side. I forked babel-plugin-istanbul to fix this, and I put my discovery in here:
https://github.com/istanbuljs/babel-plugin-istanbul/issues/125
Could you folks let me know what you think about my proposal?


Options not saved? (or fetching JSON file buggy)

We're using your addon to cycle through dashboards in the office, and it's awesome!
Configuration is user friendly and can be done remotely by anyone from our internal project management system, which webhooks to integromat.com, which produces a JSON file and uploads to ftp, which is fetched by Chrome Enterprise Tab Rotate (CETR)  every XX secs. :)
We've had issues where the updated JSON file does not trigger an update in CETR.
Loading the JSON file in a browser confirms that the file is updated.
When looking in CETR settings we noticed that the options were not saved - or perhaps they are not loaded when opening Options a second time? Here's a short screen recording: http://somup.com/cYQUFLljU1
We can't be sure if the Options are not saved, which prevents loading the updated JSON file, or there's a bug with fetching the JSON file, but would be happy to help get rid of the bug in any way we can.
/f

Can you post me your JSON file that has issues with the update, if possible? You can remove the URLs if they are internal and not meant to be shared.
Also: is the bug reproducible? Does the JSON config fail to update every time?

Can you post me your JSON file that has issues with the update, if possible?
Sure: http://weboost.dk/tabcycle.json

Also: is the bug reproducible? Does the JSON config fail to update every time?
I was screenrecording the bug a few times, after confirming that config failed to update 5 times in a row. Of course, it suddenly works and I'm not able to get it to fail now. I believe what i did was change the remote URL to a file that doesn't exist on the server, and then back to the working one. That seems to have triggered a succesfull sequence of updating...

However, what IS reproducible is this:
I have several Chrome profiles for different work purposes (bookmarks, addons, settings, etc)
If a Chrome window of any profile is already opened, opening the CETR profile (clean Chrome profile, only CETR addon installed) will not start the tab rotation automatically.
The addon dropdown status shows "Stop, Pause, Reload" which means it's in status: Start.
Clicking Pause & Resume does nothing
Clicking Stop closes the window.
Clicking Reload actually start the tab rotation.
We're planning to run 4 different Chrome profiles from the same box Chromecasted to 4 TVs, but this won't be possible at the moment. If the workaround to the original issue (config not updating) works in general, i think this is a worthy issue to take on instead.
I should probably open a feature request for this though :)
/f

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

Chrome Web Store
GitHub release

Your semantic-release bot :package::rocket:


Several smaller bugfixes

fixes #554 - adds quotations around inputs for running pipelines.  It isn't an issue with ffmpeg.  Isn't an issue with training either because of the manifest input files.  Easy to replicate issue using a video file with spaces in the name.
fixes #541 - somewhere along the way we the exit condition for continuous mode change.  We now allow the user to perform two clicks to create a rect or click and drag.  I updated the documentation in the tool tip and checked the wiki but didn't see anything there.  I also made it so continuous detection mode is disabled by default to prevent confusing users.  The main issue was that single clicking wasn't getting users out of continuous mode.
fixes #477 - adding a trim feature when putting in type names.  Before you could have a type name of '       '.  Checked and this is already done for the attributes.
fixes #475 - this is the one that may have the most impact.  The issue was that clicking from focus on a track input box back to the geoJS area would cause an update in the tracks and call updateLayers in geoJS while you were in the middle of a annotation creation.  This would disable the creation mode and then it would process the mouse click indicating that you are clicking on nothing and deselect the newly created track.  I added in a check to see if you are in creation mode and the current annotation has no coordinates.  This should prevent it from kicking out of creation mode and resetting the selected track when this happens.  A standard Escape button will cancel just like before as well.  I would test this one more thoroughly than the others.
fixes #555 - some styling changes to the confidence pairs section for larger lists.

I spent a while messing with this, I couldn't find anything wrong with it.
Looks like it solves all those issues.  Thanks!


noop: Compute file table after typecheck, not after resolve

Motivation

This paves the way for us to get some information from infer into the file table
output.
Test plan

Existing tests

Current dependencies on/for this PR:

master

PR #6941   👈

PR #6919

This comment was auto-generated by Graphite.


Example Course

Expand the test course so that it can not only be used for more tests,
but also for documentation purposes as an example course.

Created:
https://github.com/eriq-augustine/cse-cracks-course
This includes a full assignment on regular expressions.


yabar: test and module are broken

Issue description
nixos/tests/yabar fails intermittently on hydra: https://hydra.nixos.org/job/nixos/trunk-combined/nixos.tests.yabar.x86_64-linux/all .  Affects master and 18.09, cc #45960.
From the logs, it looks like the yabar service exits because no bar is defined. But due to incorrect timing the test accidentally :smile:  succeeds sometimes because the test vm shuts down before the yabar service exits.
A quick look into the yabar module also lets me suspect that the config files it generates are syntactically incorrect (semicolons which shouldn't be there according to upstream doc), but I didn't have time to investigate further.
cc maintainer @Ma27  - if you don't have time to fix it we should at least remove the yabar test from release.nix to disable it on Hydra, it's not helpful in the current state.

I can have a look tonight (or tomorrow) and report back. IIRC the test wasn't entirely pure on Hydra (I never managed to reproduce it locally unfortunately).

I just had a short look at this. The module works perfectly fine with yabar-unstable, the stable version seems to have several issues. Let's see if I can fix it or yabar-stable as well, otherwise we may need to restrict the module for yabar-unstable (which is fairly usable as well AFAIK).

Thanks. What about the test? It looked like you would have to define a config with at least one bar for the test to succeed.

I had a short look at this yesterday, but I was way too tired to fix this.
Thanks a lot for finding the cause for the broken test, I'm still not sure why it succeeded from time to time though.
I found several segfaults when using yabar (I only use yabar-unstable which works wayy better), so I never noticed :/
I'll implement a warning in the module which recommends to use yabar-unstable for now and links to several issues upstream, let's see if this fixes the issue :)

I'm still not sure why it succeeded from time to time though.

systemd starts yabar.service, yabar reads its config file and exits with an error because there's no bar defined. waitForUnit('yabar.service') can only check if the systemd unit is in active state, so it succeeds it the short interval between unit startup and unit failure.
Here's a fix for the non-deterministic success:
$yabar->waitForUnit("yabar.service", "bob");   # start the unit
$yabar->sleep(10); # give it some time to fail
$yabar->waitForUnit("yabar.service", "bob");   # see if it's still there


blackbox_register_new_file stopped working after eab0418f

Still trying to wrap my head around this problem.
After doing a git pull, running blackbox_register_new_file shows this error:
% echo nevermind > whatever.txt
% blackbox_register_new_file whatever.txt
gpg: whatever.txt: encryption failed: No public key
I double-checked that all public keys had subkeys which were not expired.
When I step back from the latest release to eab0418f the command works.
Adding "set -x" to the top of blackbox_register_new_file reveals this output:
-rRECIPIENT2
-rRECIPIENT3' whatever.txt
gpg: RECIPIENT1
-rRECIPIENT2
-rRECIPIENT3: skipped: No public key
gpg: whatever.txt: encryption failed: No public key```

In release eab0418f and before, output looks like this:

```+ gpg --yes --trust-model=always --encrypt -o whatever.txt.gpg -rRECIPIENT1  -rRECIPIENT2 -rRECIPIENT3 whatever.txt```

All the recipients are on the same line.

What baffles me is that the code in _blackbox_common.sh is the same between the two versions:

```gpg --yes --trust-model=always --encrypt -o "$encrypted" $(awk '{ print "-r" $1 }' < "$BB_ADMINS") "$unencrypted"```

Anybody else seeing this error?

Didn't mean to close this without confirming.  Please try the most recent build to see if this is resolved for you.

@harrisonpage - I figured your issue would be resolved by 7d9c07678bc1266c6de03c999b796c8d104ea1d6  But it looks like you're not effected by the extra quotes.  Can you pull the lastest release and let me know if you're still seeing this issue?

@selfcommit Awesomes: It is fixed! Thanks for following up.


get_client_from_json_dict don't work anymore with ResourceManagementClient because of parameter change

Package Name:azure-common, azure-mgmt-resource
Package Version:1.1.25, 15.0.0
Operating System: docker python buster 3.8
Python Version: 3.8

Describe the bug
The function get_client_from_dict (azure-common) with the ResourceManagementClient Class (azure-mgmt-resources) does not work anymore as it defines the needed parameter credential and cretentials like it was before version 15.x

Hi @HerwigBachner
azure-common is not compatible with the latest version of SDK that uses azure-identity protocol (like azure-mgmt-resource indeed).
We need to discuss how the migration story would go here @chlowell @jongio @johanste

@nickzhums

cc @yonzhan @jsntcy

We discussed on our team and concluded we do not want to support the JSON or file factory for security purposes, and we don't want to encourage practices that feel unsecure.
We wrote a migration guide for scenario:
https://github.com/Azure/azure-sdk-for-python/blob/master/sdk/identity/azure-identity/migration_guide.md
Please let me know any feedback or questions related to this.


Implement Channel Reopening

Problem Summary
Once a channel is closed due to timeout, or other reasons. Add the ability to reopen it.
Description
Once a channel is closed due to timeout, or other reasons, there should be a path to reopen it. This reopened channel should have the same ica address as the closed channel.
Tasks

[x] Implement the feature in a feature branch.
[x] Write end to end tests for it.

No new code had to be added to the contract. The newly added end to end test verifies that channel reopening works using the same ica address.


simstartdate not working

Describe the bug
i can't seems to make the simstartdate to work;
it says it will start the sample at the date choosen, but not doing it (see screenshot below)
btw, the --sim fast-sample alone, not working as well (should be a random date sample)
To Reproduce
adding the --simstartdate argument, example :
python3 pycryptobot.py --market ETHBUSD --sim fast-sample --simstartdate 2021-05-07
Expected behavior
Sim starting at the date choosen
Screenshots

Same thing:
$ python3 pycryptobot.py --sim fast --simstartdate 2021-01-01
[clipped]
2021-06-03 10:00:00 | BTCUSDT | 1h | Close: 39259.2300 | EMA12/26: 39259.2300 = 39259.2300 | MACD: 0.0000 = 0.0000OBV: 4322.7874 (0.0000%)ERI: |  | WAIT | Last Action: SELL
2021-06-03 11:00:00 | BTCUSDT | 1h | Close: 38781.7500 | *v EMA12/26: 39185.7715 < 39223.8611 v* | *v MACD: -38.0896 < -7.6179 v*v OBV: -1117.5090 (-125.8500%) v | ERI: sell |  | WAIT | Last Action: SELL
2021-06-03 12:00:00 | BTCUSDT | 1h | Close: 38999.0100 | v EMA12/26: 39157.0389 < 39207.2054 v | v MACD: -50.1665 < -16.1276 vv OBV: 3675.6729 (-428.9200%) v | ERI: sell |  | WAIT | Last Action: SELL
2021-06-03 13:00:00 | BTCUSDT | 1h | Close: 38440.0100 | v EMA12/26: 39046.7268 < 39150.3761 v | v MACD: -103.6493 < -33.6320 vv OBV: -3793.8941 (-203.2200%) v | ERI: buy |  | WAIT | Last Action: SELL
2021-06-03 14:00:00 | BTCUSDT | 1h | Close: 38615.2100 | v EMA12/26: 38980.3396 < 39110.7342 v | v MACD: -130.3946 < -52.9845 vv OBV: -828.8382 (-78.1500%) v | ERI: buy |  | WAIT | Last Action: SELL
[clipped]

It's also the same for different --sim values.
The startdate is different when I change granularity, perhaps just some fixed number of intervals is used in each case.


Add country and asset type, when creating site

Detailed Description
Add option for county and asset type
Context

nice to have tool where we can add sites

Possible Implementation

add to code here and return in display object

Hey @peterdudfield , I'm interested in this issue. There is a request can you please tell me what exactly is the issue we are trying to fix in a bit detail. Since this would be my first open-source contribution. It will be quite helpful.

Thanks @Princccee , it would be here and below.
Currently we cant add country or asset_type we are properties of the Site object

Hi @peterdudfield,
I came across this issue and am interested in contributing. Could you let me know which parts of the project need attention or what needs to be modified?
Thanks!


nginx's enableACME fails because of ACMEv1 EOL

When enabling a new ACME site:
{
  "type": "urn:acme:error:unauthorized",
  "detail": "Account creation on ACMEv1 is disabled. Please upgrade your ACME client to a version that supports ACMEv2 / RFC 8555. See https://community.letsencrypt.org/t/end-of-life-plan-for-acmev1/88430 for details.",
  "status": 403
}

I think it would be fixed by an upgrade of simp_le
To Reproduce
Steps to reproduce the behavior:

add a new acme domain
nixos-rebuild switch

Metadata
Please run nix run nixpkgs.nix-info -c nix-info -m and paste the result.
Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute: nginx
# a list of nixos modules affected by the problem
module: nginx

More information here: https://community.letsencrypt.org/t/end-of-life-plan-for-acmev1/88430/3
This actually happened as part of a warning "brownout period" (I was hit by this yesterday too)- they'll be disabling v1 account creation in November, and this is a tactic to get people to notice and upgrade.
Preferably this upgrade should happen by the 16th (five days from now), and it should really happen before November, as the feature will stop working for all new users and will be really confusing.
New deployments will not be able to create the acme account or register domains after October 31, or during the October brownout period of 16th -18th.
Anyone with a deployment will be able to provision certificates for new domains until June of 2020 on that same server (using the same ACME "account").
Deployments + domains that have already provisioned a cert should be able to renew until June of 2021

Yes, bumping simpl_le to a version supporting v2 should work.
However, I'm a bit confused about 0.15.1 - this will probably include https://github.com/zenhack/simp_le/commit/cf0f006e0862ceea9e9259bd44a6382864db984f - We should probably wait for that 0.15.1 release (and maybe poke about it).

https://calendar.google.com/calendar/embed?src=letsencrypt.org_caqskun93lgiabjj4ba9cb1rek%40group.calendar.google.com  contains the timeline we have for fixing this. We should backport this to 19.09

Started this migration on #71291,
See https://github.com/NixOS/nixpkgs/pull/71291#issuecomment-543595091 for current status.

Fixed in #71291


Find Icons for 6 Relics created so far

Important points:

Use the Relic template image (#180) to make sure the images all have the right size.
All Relic images must have an image pair just with the outline.

List of relic images (original and outline)

Duffel Bag
Red Headband
Fighting Gloves
Spiky Bracers
White Boots
Handcuffs

DONE \O/


Top bar missing since upgrade to System Informer

Brief description of your issue
After updaing to System Informer from Process Hacker (nightly) top bar gone missing with no ability to recover it.
With that bar you could open defult process manager, start process as some user and so. If was so useful for me.
It was present from upgrading to System Informer and thought that it would get fixxed in next update.
Screenshot of both programs open:

Steps to reproduce (optional)
No response
Expected behavior (optional)
No response
Actual behavior (optional)
No response
Environment (optional)
System Informer (nightly), last update check 20.12.2022
Process Hacker (nightly) 3.0.4953 (37e0466)
Win10x64 Pro 21H2 19044.2364

Fix

It may be set as auto-hide. Press Alt and View > Toolbar > Main menu (auto-hide)

Thanks! No need to press ALT because the context menu will continue to disappear.

@ghGhost-cs
You should reset settings to defaults