Getting Text Out Of Epub File

Hey! I was wondering if I can get the text out of an epub file using your framework. I don't see an appropriate  method in the declaration, so if you could show me how can do it, I would appreciate it

For those who are looking to take the text out of the book: check this out firstly page. From this article, I have discovered that stonezhl parsed all of xhtml files in a variable called 'pages'. To display content, use WKWebView. Hope that will help you)


Remove duplicate NavigatorCookies

I was trying to install https://github.com/tmpvar/jsdom/pull/1994 from git and ran into this kind of puzzling error:
% npm install

><EMAIL_ADDRESS>prepare /Users/andrew/projects/jsdom
> yarn convert-idl

yarn convert-idl v0.23.4
$ node ./scripts/webidl/convert
WebIDLParseError {
  message: 'The name "NavigatorCookies" of type "interface" is already seen',
  line: 44,
  input: ' {\n  readonly ',
  tokens:
   [ { type: 'whitespace', value: ' ' },
     { type: 'other', value: '{' },
     { type: 'whitespace', value: '\n  ' },
     { type: 'identifier', value: 'readonly' },
     { type: 'whitespace', value: ' ' } ] }
error Command failed with exit code 1.

Some how there were two copies of that... I'm surprised I'm the only one to run into this though.

I'm pretty sure this came with the new webidl2 version: https://github.com/w3c/webidl2.js/blob/develop/CHANGELOG.md#v420-2017-10-16
This LGTM.


Add glossary term collector

In each document define a definitions table
defs <- tibble::tribble(~term, ~def,
                        "cake", "a sweet baked dessert made from flour",
                        "pancake", "flat cake made in a pan")

and add to it using tibble::add_case()
Then collect all the definitions together and load them into the glossary page
library(epoxy)
#all_defs <- row_bind(def1, def2,...)

Define an epoxy template
{term}

: {def}

Could also write the glossary to a temporary text file, which is then removed after knitting


Fix feedback link on web

Fixes #627
Creator

[ ] Build relevant tests if any apply
[ ] Ensure there are no new warnings
[ ] Upload an animated GIF showcasing the solution (if it applies)
[ ] Set a relevant reviewer

Reviewer

[ ] Run test and make sure all tests pass
[ ] Review function and form on iOS
[ ] Review function and form on Android
[ ] Review function and form on Web
[ ] Review new test logic

Good fix. I approve.


Deploy (optional) JSON-RPC-Relay

🆕🐥 First Timers Only
This issue is reserved for people who have never contributed to hedera-enterprise or any open source project in general.
We know that creating a pull request (PR) is a major barrier for new contributors.
The goal of this issue and all other issues labeled by 'good first issue' is to help you make your first contribution to Hedera.
👾 Description of the issue
Today, we support the (optional) installMirrorNode input parameter to start a Mirror Node instance as part of the setup. Next to a Mirror Node we can add a JSON-RPC-Relay to the installation. The relay can be installed by the solo relay deploy command. General information about the JSON-RPC-Relay can be found here.
Suggested solution
Provide an additional (optional) input parameter installRelay that is false by default. If the parameter is set to true the JSON-RPC-Relay will be added by the GitHub Action.
📋 Step by step guide to do a contribution
If you have never contributed to an open source project at GitHub, the following step-by-step guide will introduce you to the workflow.
A more detailed general documentation of the GitHub PR workflow can be found here.

[ ] Claim this issue: Comment below that you are interested in working on the issue
[ ] Wait for assignment: A community member with the given rights will add you as an assignee of the issue
[ ] Fork the repository: You can do that in GitHub (by simply clicking the 'fork' button).
[ ] Check out the forked repository
[ ] Create a feature branch for the issue. We do not have a hard naming definition for branches but it is best practice to prefix the branch name with the issue id.
[ ] Solve the issue in your branch.
[ ] Commit your changes: to your branch
[ ] Start a Pull Request (PR): in the hedera-enterprise repository
[ ] Check GitHub Actions: Several GitHub Actions will be triggered automatically for each PR. If a GitHub Action fails and you do not understand the cause of that error do not hesitate to add a comment to the PR and ask the community for support.
[ ] Wait for reviews: Members of the community will review your PR. If a reviewer finds any missing pieces or a problem, he or she will start a discussion with you and describe the next steps for solving the problem.
[ ] You did it 🎉: We will merge the fix in the develop branch. Thanks for being part of our community as an open-source contributor ❤

🎉 Contribute to Hacktoberfest
Solve this issue as part of the Hacktoberfest event and get a chance to receive cool goodies like a T-Shirt. 🎽
🤔 Additional informantion
If you have any questions, just ask us directly in this issue by adding a comment. You can join the Hedera community chat at Discord. A general manual about open-source contributions can be found here.

Hi, I'm interested in this one, please add me as assignee.

@hendrikebbers Hello, Can you assign this issue to me?


Top level $not on a join criteria fails silently

Describe the bug
Trying to use $not in a join results in missing sql criteria for the not clause
To Reproduce
    qb.select('*')
      .leftJoin('a.books', 'b', {
        $not: {
            'b.foo': '1',
        },
      });

Results in
select `a`.* from `author2` as `a` left join `book2` as `b` on `a`.`id` = `b`.`author_id`

The $not part is ignored.
Expected behavior
select `a`.* from `author2` as `a` left join `book2` as `b` on `a`.`id` = `b`.`author_id` and  not b.foo = 1;

Additional context
None
Versions

Dependency
Version

node
16.x

typescript
4.7.4

mikro-orm
5.4.2

your-driver
mysql

You could try this as a workaround:
    qb.select('*')
      .leftJoin('a.books', 'b', {
        'b.foo': {
            $ne: '1',
        },
      });

The workaround is not an option for me as the join clause is auto-generated from input that I do not control.  Is it intended that top-level $not should work?  Researching Knex a little, it's not clear to me that Knex can do this.

Hmm looks like this is not really supported by knex, you cant have a generic not condition in the on clause, so this will require quite a redo of how the joins are created. Basically removing the knex API in favour of joinRaw().

IMO it's a very odd thing for knex to omit.  Relevant issue: https://github.com/knex/knex/issues/4598
I ended up applying a recursive De Morgan's to the criteria and that's working fine so far.  For context, my criteria objects are machine generated so I don't have the possibility for human intervention here.
My code depends on long-hand criteria objects, meaning shortcuts like my example above are not handled, but in my case we only generate long-hand criteria so not an issue for us.  Pasting here in case anyone else needs it.
    public static demorganNot(where: any, invert = false): any {
        if (!where) return where;

        if (Array.isArray(where)) {
            const arr: Array<any> = [];
            for (const el of where) {
                arr.push(this.demorganNot(el, invert));
            }
            return arr;
        }

        if (typeof where !== 'object') return where;

        // A $not operator can only exist by itself.
        if (Object.keys(where).length == 1) {
            const v = where['$not'];
            if (v) {
                return this.demorganNot(v, !invert);
            }
        }

        // Recursive descent through the rest of the criteria
        const obj = {};

        for (const [k, v] of Object.entries(where)) {
            if (k.startsWith('$') && invert) {
                const newOp = this.inverseOp[k];
                if (!newOp) throw new Error('No such op: ' + k);
                obj[newOp] = this.demorganNot(v, invert);
                continue;
            }

            obj[k] = this.demorganNot(v, invert);
        }

        return obj;
    }

    private static inverseOp = {
        $and: '$or',
        $or: '$and',
        $in: '$nin',
        $nin: '$in',
        $between: '$notBetween',
        $notBetween: '$between',
        $eq: '$ne',
        $ne: '$eq',
        $gte: '$lt',
        $lt: '$gte',
        $gt: '$lte',
        $lte: '$gt',
    };

I will try to refactor the QB to be less dependent on knex here, I want to gradually move away from it (will keep it as a query runner, but the queries can be generated explicitly by the ORM and ran via knex.raw() to have absolute control). I am sure I will end up with simpler code, as mapping the ORM queries to the knex API was quite tedious - the API is very user oriented, so quite terrible for our use case).

I'll admit I was thinking the same thing, so that's very encouraging news.

Damn this was challenging, but I finally have the tests green.


Branch switcher reboot

This is a rewrite of the branch switcher work in two pieces:

a vendor-agnostic plugin in @tinacms/toolkit, and
an implementation of the plugin wrapped up in TinaCloudProvider (tinacms package)

My work thus far hasn't been tested and may not be working, so don't assume you've done something wrong if it doesn't.
The plugin is hidden behind a feature flag. To activate it, add this after creating the CMS object:
cms.flags.set('branch-switcher', true)

Todo

[ ] Connect to the cloud starter example to determine if screen plugin is working correctly
[x] implement createBranch and listBranches in the plugin invocation, found in TinaCloudProvider

presumably we now want to add these endpoints to Tina Cloud, and can call out to them instead of implementing over the Github Bridge

[ ] Re-query on branch change
[ ] consider making the UI look slightly less prototype-y

@jeffsee55
I've reopened this one and rebased latest changes from main. While all the check boxes haven't been deliberately addressed yet, the create and list functions have been added, and access to the branch switcher is hidden behind the feature flag work DJ implemented.
This could in theory be merged in without it breaking any existing functionality, and then further refined once the tina cloud endpoints are in place.


Modify Queue already add to User not change bug --  fix #1514

#1514  What is the purpose of the pull request
*fix bug: the field queue in table t_ds_user does not change with table t_ds_queue modify the field queue_name *
Brief change log

modify api/service/QueueService.java
modify dao/mapper/UserMapper.java
modify dao/mapper/UserMapper.xml

Verify this pull request
This pull request is already covered by existing tests QueueControllerTest

Codecov Report

Merging #1517 into dev will increase coverage by 0.01%.
The diff coverage is 0%.

@@           Coverage Diff            @@
##             dev   #1517      +/-   ##
========================================
+ Coverage   6.75%   6.77%   +0.01%
========================================
  Files        272     272
  Lines      13612   13616       +4
  Branches    2165    2167       +2
========================================
+ Hits         920     922       +2
- Misses     12628   12630       +2
  Partials      64      64

Impacted Files
Coverage Δ

...che/dolphinscheduler/api/service/QueueService.java
0% <0%> (ø)
:arrow_up:

...ache/dolphinscheduler/common/enums/ZKNodeType.java
100% <0%> (+100%)
:arrow_up:

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 38b0765...7968a07. Read the comment docs.


Error on opening most of the Storybook pages

Expected Behavior

All Storybook pages should open without errors.
Components' stories should look similar to Autocomplete docs.

Current Behavior
Error on opening most of the Storybook pages.
Context
The version of Storybook had been updated to 7.0.0-beta.64 and it broke most of the Storybook pages.

Project Version: 0.9.5
Operating System: MacOS 13.0.1 (Ventura)
Browser: Google Chrome (latest)

Steps to Reproduce

Run 'yarn docs' command from the root of the repo
Wait for Storybook to be loaded and opened in a browser
Select any component except Autocomplete in the sidebar

Failure Logs

Checklist

[x] I am running the latest version
[x] I checked the documentation and found no answer
[x] I checked to make sure that this issue has not already been filed
[x] I'm reporting the issue to the correct repository (for multi-repository projects)
[x] I have provided sufficient information for the team

Fixed in #185


Line for results path in osemosys code can change

the main_step script changes the results path in the osemosys file when running models with GLPK. This creates an error when the lines in the OSeMOSYS file have changed.

This is outdated as well if I'm not mistaken, right @trevorb1?
In the new scripts you don't use the ResultsPaths in the OSeMOSYS code right?

Correct, @HauHe! We just copy over the model file to get around this if we are solving via GLPK. I think this is a more cumbersome solution, but more stable since we dont have to modify a specific line number in the model file each time.

Perfect, thanks for the explanation had briefly looked at the code, but wasn't exactly sure how you solved, just got that you did it different, haha


Home Assistant reporting deprecated "constsants" which will be REMOVED in HA Core 2025.1

hi @tdragon and @alex255
Just thought i would let you know about the below. Realise it states it won't affect us until 2025.1 version but thought i would give you a heads up.
2024-04-06 14:52:29.379 WARNING (ImportExecutor_0) [homeassistant.const] TEMP_CELSIUS was used from reef_pi, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfTemperature.CELSIUS instead, please report it to the author of the 'reef_pi' custom integration
2024-04-06 14:52:29.383 WARNING (ImportExecutor_0) [homeassistant.const] TEMP_FAHRENHEIT was used from reef_pi, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfTemperature.FAHRENHEIT instead, please report it to the author of the 'reef_pi' custom integration

Thanks. I'll fix it


Running in a class library project

If you are reverse engineering a project into a class library, install these packages before running the Scaffold-DbContext command:
PM> Install-Package Microsoft.EntityFrameworkCore.Tools
PM> Install-Package Microsoft.EntityFrameworkCore.SqlServer

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

ID: 998197fe-077b-acee-c91c-12f074359e9e
Version Independent ID: 7542e87e-8f99-9b91-6df1-6983776ad1d2
Content: Getting Started on ASP.NET Core - Existing Database - EF Core
Content Source: entity-framework/core/get-started/aspnetcore/existing-db.md
Product: entity-framework
GitHub Login: @rowanmiller
Microsoft Alias: divega

@michael-lang Thanks for the suggestion -- I am adding links from this tutorial to the CLI and PMC tools reference articles, which show how to install the tools.  For ASP.NET 2.1+, the instructions in this tutorial are correct; you don't need to install these packages because they're in the ASP.NET Core metapackage.


Improve multisite documentation

Fix #278

Coverage decreased (-0.3%) to 96.186% when pulling f6000ef6234fd5f05a248ce1d27f56cab59648a9 on docs/multisite into 300dcebfbfb16c64ad01c617262d945941561ec9 on develop.

[Current coverage][cc-pull] is 92.70%

Merging [#279][cc-pull] into [develop][cc-base-branch] will not change coverage

@@            develop       #279   diff @@
==========================================
  Files            19         19
  Lines          1206       1206
  Methods           0          0
  Messages          0          0
  Branches        147        147
==========================================
  Hits           1118       1118
  Misses           42         42
  Partials         46         46

Powered by Codecov. Last updated by [300dceb...f6000ef][cc-compare]
[cc-base-branch]: https://codecov.io/gh/nephila/djangocms-blog/branch/develop?src=pr
[cc-compare]: https://codecov.io/gh/nephila/djangocms-blog/compare/300dcebfbfb16c64ad01c617262d945941561ec9...f6000ef6234fd5f05a248ce1d27f56cab59648a9
[cc-pull]: https://codecov.io/gh/nephila/djangocms-blog/pull/279?src=pr


Worker login

Mam can I also add mobile no field in login-worker form.I have also added it to the schema in the backend.Actually login with only password field would create problem if 2 or more people set the same password.
Participant under Gssoc’22

Did merge your PR @ayush25102001


⚠ Bofrost.fr has degraded performance

In 6bf00d4, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1111 ms

Resolved: Bofrost.fr performance has improved in 73110aa.


BaseDenseAttention now supports attention dropout

Added optional dropout applied to attention weights (scores)

@googlebot I signed it!

I'm not sure I'm the best person to review this.
Can you find the author of dense attention?
https://github.com/tensorflow/community/blob/master/rfcs/20190115-dense-attention.md

Can one of the admins verify this patch?

@roumposg
https://github.com/tensorflow/models/blob/master/official/transformer/v2/attention_layer.py
I know its not a paper, but the official TensorFlow implementation. I was just morroring it.
And the results are slightly different from the "more conventional" dropout, given that it's applied to the attention weights. Think of it as the recurrent dropout the RNNs have.
On the other hand, I was about to contribute the implementation of the Relative Encoding (https://arxiv.org/abs/1803.02155, and with a better explanation https://arxiv.org/abs/1809.04281). Would it be a good idea?

@roumposg Thank you, I appreciate it. I will fix them as soon as I can. Also thanks to @tanzhenyu .

@roumposg I think it should be finished now, let me know please.

@gbaned Is everything alright?

@claverru Sorry for the late reply. It is pending with internal merging. Thanks!

Any news?

Any news?

@gbaned ?

@arthurflor23, @claverru  This is failing some internal tests, please stay tuned. Thanks!

It broke internal tests so @roumposg submitted a separate commit here

Thank you @tanzhenyu , @roumposg !

Seems auto-merge is not happening but the changes are now committed so we can close this. Thank you for the PR.


minikube: 1.33.0 -> 1.33.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/kubernetes/minikube/releases.
meta.description for minikube is: A tool that makes it easy to run Kubernetes locally
meta.homepage for minikube is: https://minikube.sigs.k8s.io
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.33.1 with grep in /nix/store/0ndh4syv6nf49c0px91dscwkck896b0n-minikube-1.33.1
found 1.33.1 in filename of file in /nix/store/0ndh4syv6nf49c0px91dscwkck896b0n-minikube-1.33.1

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

docker-machine-kvm2
minikube

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/0ndh4syv6nf49c0px91dscwkck896b0n-minikube-1.33.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A minikube https://github.com/r-ryantm/nixpkgs/archive/39a61da255cc954d1e5760d5e907001d997f0fc9.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/39a61da255cc954d1e5760d5e907001d997f0fc9#minikube

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/0ndh4syv6nf49c0px91dscwkck896b0n-minikube-1.33.1
ls -la /nix/store/0ndh4syv6nf49c0px91dscwkck896b0n-minikube-1.33.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages built:

docker-machine-kvm2
minikube

Maintainer pings
cc @ebzzry @copumpkin @vdemeester @AtkinsChang @Chili-Man for testing.

Result of nixpkgs-review pr 311528 run on x86_64-linux 1

2 packages built:

docker-machine-kvm2
minikube


go/salt-learning Intro to Typography Foundation training

[x] Setup IEMS registration page
[x] Send invites
[ ] Run training
[ ] Distribute survey

To help educate Designers on the revised foundations, new patterns and updated components we are kicking of a Design System Knowledge Share series, starting with Typography foundation.
This series will occur every 3-4 weeks.

Invites have been distributed to DXD and UITK distro list, as well as publicised on Symphony. It was also announced in a global team meeting.

July 17 - first session tomorrow

Jul-21: survey distributed on the 18th; will close on the 28th. Request from team to encourage participation.


chore(apps/gcp/prow/release): bump image for hook to v20230922-c60c561dfe

tested in dev env.
Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

/approve


moved _slot property and _registerSlot method to an extension of Component

#feature#

This change is

Reviewed 5 of 5 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

tests/unit/initializers/component-block-slots-test.js, line 28 [r1] (raw file):
Can we add a component test as well to demonstrate that the expected variables are available on a new component?

Comments from the review on Reviewable.io

Approved

@juwara0 the linting is failing - if you run npm run lint ___ --fix for those directories it should clean up most of it and then you can run npm run lint to check for any remaining issues

Review status: 1 of 8 files reviewed at latest revision, 1 unresolved discussion.

tests/unit/initializers/component-block-slots-test.js, line 28 [r1] (raw file):
Done.

Comments from the review on Reviewable.io

Reviewed 7 of 7 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io

Approved


Doesn't respect git author per repo

This command doesn't seem to account for having different local git users per repo.
For instance, my global git user is different to many repos I work in professionally. If I run git standup in a parent of my work repos, I get zero results as global user != local user.
Instead this command should grab the git user for each repo, and use that to find commits. I can workaround this by doing git standup -a "Some name" but I believe this should be default.
Unfortunately my bash-fu is awful, and I can't help make a PR for this.

Turns out my bash-fu isn't as bad as I thought. PR made :)


[SPARK-16964][SQL] Remove private[sql] and private[spark] from sql.execution package [Backport]

What changes were proposed in this pull request?
This PR backports https://github.com/apache/spark/pull/14554 to branch-2.0.
I have also changed the visibility of a few similar Hive classes.
How was this patch tested?
(Only a package visibility change)

cc @rxin

Test build #63803 has started for PR 14652 at commit 7ba8abb.

Test build #63803 has finished for PR 14652 at commit 7ba8abb.

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

Merged build finished. Test PASSed.

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

LGTM - merging in branch-2.0.

hm damn - this one includes https://github.com/apache/spark/pull/14654 and I merged that one first. I ended up reverting the other one from branch-2.0 to merge this one.
Can you close this ticket now?

Thanks!


[PLUGIN REQUEST] Preview.nvim (+ md-tui)

Field
Description

Plugin
Preview.nvim

Homepage
https://github.com/henriklovhaug/Preview.nvim

Nixpkgs
false

Extra Information
Demo of markdown preview in terminal, which is pretty cool:
https://x.com/orhundev/status/1795080714192347445
Package used,
https://github.com/henriklovhaug/md-tui

I have packaged md-tui in nixpkgs: https://github.com/NixOS/nixpkgs/pull/318492
Let's wait for it to make its way in nixos-unstable.

https://github.com/NixOS/nixpkgs/pull/323735


Issue with different users from different origins but same username

We are working with two user bases. The cf admins are entered into uaa with uaac and the regular user base comes via ldap. UAA allows for two user entities with same username but different origin.
Login with the different usernames and password combination works.
However, changing the password for either of them does not work.
Deleting a user with uaac does not work at all. Would also be questionable which user to delete as the only argument you can pass is username, which is not unique. Currently the message is "User not found".
Deleting the user with cf cli works, but also here the behavior is not well defined. Only username as parameter can be passed. This leads to deletion of user who was created first in the uaa.
This behavior was already reported in March in Google group
My proposal would be to extend the api and/or uaac and cf cli to support deleting a user based on its username and origin. If no origin is passed and the username is included multiple times, an error should be returned and no user should be deleted until it is clear which one really should be deleted.

UAA API and uaac can support this today.  The API supports DELETE on /Users/{userId} which will delete a specific user BUT that assumes you know the ID of the user that you wish to delete.
The ID of a user can be found by issuing the following uaac command:-
uaac users 'username eq "marissa" and origin eq "saml"' -a ID,username

or, if the user you wish to find is in another zone:-
uaac users 'username eq "marissa" and origin eq "saml"' -a ID,username -z otherzone

(NB: you will need zone.{zoneId}.admin scope for the zone in question)
Once found the user can be deleted by issuing:-
uaac curl -XDELETE http://localhost:8080/Users/{userId} -H 'Accept: application/json'

or, if the user you wish to delete is in another zone:-
uaac curl -XDELETE http://localhost:8080/Users/{userId} -H 'Accept: application/json' -H 'X-Identity-Zone-Subdomain: otherzone'

(NB: but again you will need zone.{zoneId}.admin scope for the zone in question)
That said I also do agree that the CLI (uaac in our case) should have user management commands that allow a user to be uniquely identified (with perhaps a reasonable origin default of 'uaa') so I will leave this open and talk our PM about this.

Thanks for the detailed information. It is good that there is a way to delete specific users. However, it would be great to have this as integrated CLI user management commands.
Please keep me posted on the discussion with Sree.

@Kiemes Ideally both Admins and Regular users should be coming from one source which incase of most enterprises is either SAML or LDAP based stores.
We have capabilities today to map scopes to ldap or saml groups. This allows LDAP and SAML users to have admin privileges.
Is there any specific reason to have admin users in UAA and other users in LDAP?

@sreetummidi yes I know, ideally it would be the case, but in our setup the operations users (admins) come from our company and the regular CF users are users from our customer. There is no LDAP or SAML which holds all users from both companies.
However, regardless of the usage of the UAA, if the UAA supports the co-existence of multiple users from different sources at the same time, which it does, it would be great to have a more intuitive CLI command than using the uaac curl.

@Kiemes point noted. We will add the cross origin smarts into the CLI


Fix #4298 be less strict about Operations on IBulkRequest

IBulkRequest dictated that operations should be implemented using BulkOperationsCollection which effectively is a SynchronizedCollection.
Since then we also introduced the object inititalizer syntax. Here we can push back on this requirement. This PR fixes that so that users using that syntax are not forced to use the BulkOperationsCollection.
Some performance numbers attached:

Method
Mean
Error
StdDev
Gen 0
Gen 1
Gen 2
Allocated

Descriptors.V8
2.581 ms
0.0529 ms
0.1383 ms
-
-
-
1.7 MB

Descriptors
2.722 ms
0.0818 ms
0.2308 ms
-
-
-
1.7 MB

BulkCollection.AddRange
1.173 ms
0.0317 ms
0.0936 ms
-
-
-
1.09 MB

BulkCollection.const
1.096 ms
0.0217 ms
0.0583 ms
-
-
-
1.09 MB

'new List'
1.061 ms
0.0209 ms
0.0468 ms
-
-
-
1.09 MB

Do we want to also expose a way to set the IList type on BulkDescriptor?

Good call out will amend today!

Do we want to also expose a way to set the IList type on BulkDescriptor?

Revisiting this I reckon its not worth the hassle, it would require introducing a constructor argument which breaks our selector rules or a complicated .InitCollection() method.


fix: destroy dubbo server before cleaning repository

What is the purpose of the change
fix issue https://github.com/apache/dubbo/issues/12064
Brief changelog
fix: destroy dubbo server before cleaning repository
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

https://github.com/apache/dubbo/pull/12118 Would be a better solution.


Add TTL for datasync

This PR is meant for discussion of and eventually implementation of an approach to add TTLs to DataSync and possibly to add feasible infrastructure for running jobs in vanilla Graphback

A workaround to test it out:

replace index.ts in mongo template with this
Note that the second argument to createDataSyncMongoDbProvider is the TTL offset in seconds:
https://gist.github.com/ssd71/f9bf3a3865e95a958d95845a0d857269#file-index-ts-L31
add @datasync annotation to your model

The prune job is being run every 10 seconds:
https://github.com/aerogear/graphback/blob/f56a5adf2335b35dab6cee62a3d254aa5727bb39/packages/graphback-datasync/src/providers/createDataSyncMongoDbProvider.ts#L47
Would be nice if we could also agree on what to change this interval to :D

This looks like good prototype of what we should do.
Challenge I see now is how to make this easy to configure.
TTL per entity type or global? Separate config for the prune job trigger time etc.

Yes, I am working on it, if you have any pointers feel free to tell me :smile:

TTL is global for now, I am thinking it would be better to have it per entity, what do you think? That way we would have to specify it for every model
Yes, we really might need configurable trigger time

TTL is per entity now
prune Interval is configurable now

See here for testing, will keep updating this gist from here on out

@ssd71 Let's revisit this again

Seening this PR I'm glad we did not merged it. Closing this PR


timestamp synchronization

How do I synchronize the linux system time to the camera as the camera timestamp?

In high precision this only works for gigevision.
Read the chapter about PTP in the documentation: https://docs.baslerweb.com/precision-time-protocol.html


fix partial introspection of qualified table names

The connector was recently modified to allow for qualified table names, after it became apparent the previous assumptions that all clickhouse tables would always live in the default schema proved false.
Those changes missed that we also relied on this assumption when introspecting filtered tables, that is only fetching introspection for specific tables.
This fix changes the assumption from "table names are arrays with a single string" to "table names are arrays with at least one string". We filter using the last string in the array.
Note, this behavior is still incorrect. We now ignore the qualified part of the table name, so when asked to introspect table schema1.table1 we would also return introspection result for schema2.table1
We can probably fix that too if it's a problem?

This is not a breaking change. Rather this should allow the connector to work correctly for users affected by this bug


Add support for HomeKit

Description:
After reading about HAP-Python in the forum, I tried a few things and ended up with this. It should add the bare bone support for homekit to Home Assistant and is quite easy expendable. For now I just added support for two components: temperature sensors and covers (with set_position support).
Tested with: HAP-python==1.1.2
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):
homekit:

cover:
  - platform: demo
sensor:
  - platform: yr
    monitored_conditions:
      - temperature

Checklist:

[x] The code change is tested and works locally.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

Off to a great start! Will need some tweaking to get it working with Home Assistant. I think that once the foundation is in place, it will be easier for people to extend. Please don't add extra device support to this PR (beyond temp sensor and cover) and let's make sure the foundation is right first.

I just pushed a new version which should address all your comments.

Linting fails because of the imports. To work around this, add the files to the IGNORE_PACKAGES constant in gen_requirements_all.py.

We'll need to get tests added too to make sure that the integrations work as expected.

@cdce8p @balloob How important is this for the integration being done here? I am asking so I can prioritise the work - if it will significantly speed up things here, I will try to work on that sooner.

@ikalchev Shouldn't be too important. It was just meant as a suggestion to think about for the future. By default only the packages need are also imported, so their shoun't be an affect on the runtime. As for tests: Since the initial file is a little bit bigger, generating the environment could take a slight moment longer.

As long as those files are never imported, it's indeed not high priority as it is not consuming extra memory.
I wonder how many accessories you'll need once the Home Assistant integration is done and you can just forward any Home Assistant integration over Homekit?

This is ok to merge when some tests get added.

Oh snap, I just saw that the Py35 test failures actually were Homekit failures. @cdce8p can you have a look what is going on? Wouldn't want to add flaky tests
https://travis-ci.org/home-assistant/home-assistant/jobs/343531242#L962
ERROR:homeassistant.core:Error doing job: Future exception was never retrieved
Traceback (most recent call last):
  File "/opt/python/3.5.3/lib/python3.5/concurrent/futures/thread.py", line 55, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/home/travis/build/home-assistant/home-assistant/homeassistant/components/homekit/__init__.py", line 133, in stop_driver
    self.driver.stop()
  File "/home/travis/build/home-assistant/home-assistant/.tox/py35/lib/python3.5/site-packages/pyhap/accessory_driver.py", line 491, in stop
    self.run_sentinel.set()
AttributeError: 'NoneType' object has no attribute 'set'

core.py                     94 ERROR    Error doing job: Future exception was never retrieved
Traceback (most recent call last):
  File "/opt/python/3.5.3/lib/python3.5/concurrent/futures/thread.py", line 55, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/home/travis/build/home-assistant/home-assistant/homeassistant/components/homekit/__init__.py", line 133, in stop_driver
    self.driver.stop()
  File "/home/travis/build/home-assistant/home-assistant/.tox/py35/lib/python3.5/site-packages/pyhap/accessory_driver.py", line 491, in stop
    self.run_sentinel.set()
AttributeError: 'NoneType' object has no attribute 'set'

I'm on it. Missed that one, since I'm running 3.6.

@cdce8p we will also need docs for this.

@arsaboo Will work on that next

Just be clear, this doesn't add support for lights and switches yet, or am I mistaken?

Indeed, only cover and temperature sensors are supported right now.

@fattdev PR welcome 👍

@balloob Would love to!


Cannot log into web client on quickstart VM

Thanks for open-sourcing this! I'm trying to get the quickstart examples running in the VM, but I'm having problems getting logged into the web client. I can get the frontend and backend running, but when I go to log in to the web client running on 9008 it spins for a while and then I get Invalid username or password. The users table in mysql has an entry for wherehows with a stored has that matches sha1("wherehows"), so I don't think it's actually an auth issue. The backend console shows the following:
[error] application - Could not get JDBC Connection; nested exception is java.sql.SQLException: Unable to open a test connection to the given database. JDBC url = jdbc:mysql://<IP_ADDRESS>/wherehows?charset=utf8&zeroDateTimeBehavior=convertToNull, username = wherehows.

I've run /root/bin/enable-all-hadoop-service.sh and tried starting the frontend before the backend and vice versa. This seems to be a pretty straightforward "I can't find mysql" error, but I'm kind of at a loss for how to fix it. Any help greatly appreciated!
Full trace from the frontend:
cloudera@quickstart wherehows]$ ./runfrontend
Java HotSpot(TM) 64-Bit Server VM warning: ignoring option MaxPermSize=256m; support was removed in 8.0
Picked up _JAVA_OPTIONS: -Djava.net.preferIPv4Stack=true
Play server process ID is 15205
[info] play - Application started (Prod)
[info] play - Listening for HTTP on /<IP_ADDRESS>:9008
[error] application - Authentication failed for user wherehows
[error] application - Could not get JDBC Connection; nested exception is java.sql.SQLException: Unable to open a test connection to the given database. JDBC url = jdbc:mysql://<IP_ADDRESS>/wherehows?charset=utf8&zeroDateTimeBehavior=convertToNull, username = wherehows. Terminating connection pool (set lazyInit to true if you expect to start your database after your app). Original Exception: ------
com.mysql.jdbc.exceptions.jdbc4.CommunicationsException: Communications link failure

The last packet sent successfully to the server was 0 milliseconds ago. The driver has not received any packets from the server.
	at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:422)
	at com.mysql.jdbc.Util.handleNewInstance(Util.java:411)
	at com.mysql.jdbc.SQLError.createCommunicationsException(SQLError.java:1117)
	at com.mysql.jdbc.MysqlIO.<init>(MysqlIO.java:350)
	at com.mysql.jdbc.ConnectionImpl.coreConnect(ConnectionImpl.java:2408)
	at com.mysql.jdbc.ConnectionImpl.connectOneTryOnly(ConnectionImpl.java:2445)
	at com.mysql.jdbc.ConnectionImpl.createNewIO(ConnectionImpl.java:2230)
	at com.mysql.jdbc.ConnectionImpl.<init>(ConnectionImpl.java:813)
	at com.mysql.jdbc.JDBC4Connection.<init>(JDBC4Connection.java:47)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:422)
	at com.mysql.jdbc.Util.handleNewInstance(Util.java:411)
	at com.mysql.jdbc.ConnectionImpl.getInstance(ConnectionImpl.java:399)
	at com.mysql.jdbc.NonRegisteringDriver.connect(NonRegisteringDriver.java:334)
	at java.sql.DriverManager.getConnection(DriverManager.java:664)
	at java.sql.DriverManager.getConnection(DriverManager.java:247)
	at com.jolbox.bonecp.BoneCP.obtainRawInternalConnection(BoneCP.java:363)
	at com.jolbox.bonecp.BoneCP.<init>(BoneCP.java:416)
	at com.jolbox.bonecp.BoneCPDataSource.getConnection(BoneCPDataSource.java:120)
	at org.springframework.jdbc.datasource.DataSourceUtils.doGetConnection(DataSourceUtils.java:111)
	at org.springframework.jdbc.datasource.DataSourceUtils.getConnection(DataSourceUtils.java:77)
	at org.springframework.jdbc.core.JdbcTemplate.execute(JdbcTemplate.java:627)
	at org.springframework.jdbc.core.JdbcTemplate.query(JdbcTemplate.java:692)
	at org.springframework.jdbc.core.JdbcTemplate.query(JdbcTemplate.java:724)
	at org.springframework.jdbc.core.JdbcTemplate.query(JdbcTemplate.java:734)
	at org.springframework.jdbc.core.JdbcTemplate.query(JdbcTemplate.java:784)
	at org.springframework.jdbc.core.JdbcTemplate.queryForList(JdbcTemplate.java:889)
	at dao.UserDAO.authenticate(UserDAO.java:109)
	at security.AuthenticationManager.authenticateUser(AuthenticationManager.java:42)
	at controllers.Application.authenticate(Application.java:130)
	at Routes$$anonfun$routes$1$$anonfun$applyOrElse$4$$anonfun$apply$4.apply(routes_routing.scala:341)
	at Routes$$anonfun$routes$1$$anonfun$applyOrElse$4$$anonfun$apply$4.apply(routes_routing.scala:341)
	at play.core.Router$HandlerInvoker$$anon$7$$anon$2.invocation(Router.scala:183)
	at play.core.Router$Routes$$anon$1.invocation(Router.scala:377)
	at play.core.j.JavaAction$$anon$1.call(JavaAction.scala:56)
	at play.GlobalSettings$1.call(GlobalSettings.java:64)
	at play.core.j.JavaAction$$anon$3.apply(JavaAction.scala:91)
	at play.core.j.JavaAction$$anon$3.apply(JavaAction.scala:90)
	at play.core.j.FPromiseHelper$$anonfun$flatMap$1.apply(FPromiseHelper.scala:82)
	at play.core.j.FPromiseHelper$$anonfun$flatMap$1.apply(FPromiseHelper.scala:82)
	at scala.concurrent.Future$$anonfun$flatMap$1.apply(Future.scala:251)
	at scala.concurrent.Future$$anonfun$flatMap$1.apply(Future.scala:249)
	at scala.concurrent.impl.CallbackRunnable.run(Promise.scala:32)
	at play.core.j.HttpExecutionContext$$anon$2.run(HttpExecutionContext.scala:37)
	at akka.dispatch.TaskInvocation.run(AbstractDispatcher.scala:42)
	at akka.dispatch.ForkJoinExecutorConfigurator$AkkaForkJoinTask.exec(AbstractDispatcher.scala:386)
	at scala.concurrent.forkjoin.ForkJoinTask.doExec(ForkJoinTask.java:260)
	at scala.concurrent.forkjoin.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1339)
	at scala.concurrent.forkjoin.ForkJoinPool.runWorker(ForkJoinPool.java:1979)
	at scala.concurrent.forkjoin.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:107)
Caused by: java.net.ConnectException: Connection timed out
	at java.net.PlainSocketImpl.socketConnect(Native Method)
	at java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:350)
	at java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:206)
	at java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:188)
	at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
	at java.net.Socket.connect(Socket.java:589)
	at java.net.Socket.connect(Socket.java:538)
	at java.net.Socket.<init>(Socket.java:434)
	at java.net.Socket.<init>(Socket.java:244)
	at com.mysql.jdbc.StandardSocketFactory.connect(StandardSocketFactory.java:259)
	at com.mysql.jdbc.MysqlIO.<init>(MysqlIO.java:300)
	... 48 more
------

Hi,
When did you download the vm image?
It looks like the image is old.
From the log,
[error] application - Could not get JDBC Connection; nested exception is java.sql.SQLException: Unable to open a test connection to the given database. JDBC url = jdbc:mysql://<IP_ADDRESS>/wherehows?charset=utf8&zeroDateTimeBehavior=convertToNull, username = wherehows. Terminating connection pool (set lazyInit to true if you expect to start your database after your app). Original Exception: ------
com.mysql.jdbc.exceptions.jdbc4.CommunicationsException: Communications link failure
It looks like the application is trying to connect to mysql db host on <IP_ADDRESS>.
It should be connect to host localhost.
database.opensource.url = "jdbc:mysql://localhost/wherehows?charset=utf8&zeroDateTimeBehavior=convertToNull"
You can see it in file application.conf, under folder /home/cloudera/wherehows/wherehows-1.0-SNAPSHOT/conf/.
Thanks!

Hi,
I  downloaded the WM Saturday, and I also have the same problem where it hangs during the login and returns with an "Invalid user .." message. I checked the log and I can see it is referring to the same IP "<IP_ADDRESS>".
Where do I change this so it refers to Localhost - I'm far from a Linux expert - so please be precise ;-)
Regards,
Hans Christian

Thanks for the reply!  I downloaded it on Friday, and I have that exact line in the config file:
database.opensource.url = "jdbc:mysql://localhost/wherehows?charset=utf8&zeroDateTimeBehavior=convertToNull"

I did some grepping in the application folders for <IP_ADDRESS> and didn't find anything but logged errors.
Is there a hash of the VM image or some other way to check the version of the files?

I am seeing this on the VM as well. I grabbed the image on Friday 3/4.
Mine is also trying to connect to MySQL on IP <IP_ADDRESS>
Setting this in application.conf is ignored - even setting it to gibberish, e.g. database.opensource.url = "you:shall:not:connect" it attempts to connect to <IP_ADDRESS>

I see the same behavior as others in the VM, downloaded Friday 3/4. I grepped around and looked for a config file with that IP, but none turned up. I did see that IP referred to the MySQL database for an Azkaban server, (not MySQL).

2016-03-07 12:17:37,033 - [INFO] - from play in main
Application started (Prod)
2016-03-07 12:17:37,170 - [INFO] - from play in main
Listening for HTTP on /<IP_ADDRESS>:9008
2016-03-07 12:20:01,774 - [ERROR] - from application in play-akka.actor.default-dispatcher-2
Authentication failed for user wherehows
2016-03-07 12:20:01,774 - [ERROR] - from application in play-akka.actor.default-dispatcher-2
Could not get JDBC Connection; nested exception is java.sql.SQLException: Unable to open a test connection to the given database. JDBC url = jdbc:mysql://<IP_ADDRESS>/wherehows?charset=utf8&zeroDateTimeBehavior=convertToNull, username = wherehows. Terminating connection pool (set lazyInit to true if you expect to start your database after your app). Original Exception: ------
com.mysql.jdbc.exceptions.jdbc4.CommunicationsException: Communications link failure

We tried on different machines didn't see this error... let me try use my own computer outside of internal network.

Can you try below steps to restart the front server?

get the font server process id
cat /home/cloudera/wherehows/wherehows-1.0-SNAPSHOT/RUNNING_PID
kill the process
kill -9 $pid
remove the RUNNING_PID file
rm /home/cloudera/wherehows/wherehows-1.0-SNAPSHOT/RUNNING_PID
using different command to start the font service
/home/cloudera/wherehows/wherehows-1.0-SNAPSHOT/bin/wherehows -Dhttp.port=9008

Let see if it works.
Thanks!

Hi Jerry Bai
Just followed your instructions - but it didn't help.
The  file  "/home/cloudera/wherehows/runfrontend" contains the same line as you describe, and I assume that this is the file which is started automatically when the WM is launched?
Regards,
Hans Christian

Yes, followed the instructions from jerrybai2009 above and no change in behavior on login:
[ERROR] - from application in play-akka.actor.default-dispatcher-2
Could not get JDBC Connection; nested exception is java.sql.SQLException: Unable to open a test connection to the given database. JDBC url = jdbc:mysql://<IP_ADDRESS>/wherehows?charset=utf8&zeroDateTimeBehavior=convertToNull, username = wherehows. Terminating connection pool (set lazyInit to true if you expect to start your database after your app). Original Exception: ------
com.mysql.jdbc.exceptions.jdbc4.CommunicationsException: Communications link failure

Following those instructions, I got the same failure. This, however, works:
/home/cloudera/wherehows/wherehows-1.0-SNAPSHOT/bin/wherehows -Dhttp.port=9008 -Ddatabase.opensource.url="jdbc:mysql://localhost/wherehows?charset=utf8&zeroDateTimeBehavior=convertToNull"

Great - now it works ;-)

Thanks, @nbateshaus, that worked like a champ!

Same here, thanks!

Thanks, @nbateshaus https://github.com/nbateshaus!
We have a new image posted. I have also mentioned the
database.opensource.url override in the readme file for download.
On Tue, Mar 8, 2016 at 10:08 AM, Taylor Terry<EMAIL_ADDRESS>wrote:

Same here, thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/linkedin/WhereHows/issues/64#issuecomment-193895967.

Thanks for @nbateshaus help. We found that the VM before March. 3 did have a hard code host name in the configure file. And that's inside the jar file in lib folder, that's why you can't grep it out. If you download after March. 4 it should works fine.
Recently we have some new changes in the latest version, welcome to check it out!


[move] Rework of Call and Module Access syntax

Motivation

The grammar for call syntax and identifiers was extremely complicated
Additionally, methods were a sort of half baked features, and didn't let you add impl style methods for custom instantiations, i.e. no methods for impl Vec.T<Coin> or anything specific.
Module access is now :: instead of .
Removed MName vs Name restrictions on identifiers

e.g. structs can start with lower, functions can start with upper

Killed method syntax
Changed call syntax to require parens

Will change some typing rules around this later

Test Plan
Updated all tests

@bors-libra r=bob-wilson

:pushpin: Commit 9cb23f4 has been approved by bob-wilson

:hourglass: Testing commit 9cb23f4ff401d67d54432bb0117c3d56d5e13cf1 with merge 49788a86d8c3d4f9b8f83e796625e086adcc02d6...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: bob-wilson
Pushing 49788a86d8c3d4f9b8f83e796625e086adcc02d6 to master...


fix(moti): don't use css vars

fix #1520

This makes it slower on web unfortunately, but correctness worth it for now.


Fix infinite loop when loading DXIL.dll if library is not found.

Thanks! I guess we need a new release?


Update upgrade.md

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Thanks @scjane 👍


autocomplete

hi everyone,
My problem is that to use oro_entity_create_or_select_inline create don't work and say this route does not exists, but I know this route is exist and work good.
This problem was as same as fsabeti's problem in another issue.
thanks.

Hello, vafabandi.
Please recheck this in dev env. Also ensure that you cleared the cache before this, with app/console cache:clear env dev.
If I correctly understood, you have issue similar to the #711. When you click "+\create" you see next in browser's console log:

Uncaught Error: The route "your_rote" does not exist.

This error usually fired in prod env for wide range of issues with requested route. In dev env, you'll be able to see problematic request in browser's Network panel and investigate response, maybe here is some error during widget rendering or so.

@vafabandi,  I am now closing this issue as it has been inactive for a while. Please, feel free to reopen it if you have any questions.


Update v0.8.6

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender


Update lockfile for rails bump to <IP_ADDRESS>

@jrafanie Please review.

I'm not sure why the bot didn't mark this petrosian in the title

Green, codacy is buggy. nothing new there.  It just failed to scan.


Expose webpackFinal extension

Issue:
angular-cli preset should filter out all the styling rules in order to integrate its rules instead.
Currently, angular-cli preset is kicked-in in the middle of the presets chain because it's a framework preset.
The last preset in the chain adds default rules or execute the custom webpack config. We need the ability for other presets to clean the webpack config after all the manipulations.

Related #4296
What I did
I've added the webpackFinal step.

Looks like it should also fix the initial issue from #4230


[Maestro version v1.27] maestro test stops when a custom toast appears

Describe the bug
The maestro test process ends when a custom toast appears in the app.
To Reproduce
Steps to reproduce the behavior, e.g.:

Start the simulator
Run maestro test ...
The Flow fails when a custom toast is shown

Expected behavior
Keep running the maestro test.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment information (please complete the following information):

Maestro version v1.27
Platform: Android
Framework: Native Android
Device model and OS version: -
Simulator, emulator or physical device: tested on emulator, I didn't try it on a physical device
Host: Mac M1

Bugreport output (if applicable)

Additional context
Slack message.

CustomToast.zip
There's a test maestro_toast_test.yaml in the root folder.
Thanks!

@cenci7 have you tried to run on Maestro 1.28.0 ?

![Screenshot from 2023-06-08 19-18-18](https://github.co@cenci7 /mobile-dev-inc/maestro/assets/114213754/ba9e89bb-754a-4713-b076-47655afd7cff)
@cenci7 i have executed the issue on both the versions of maestro 1.27 and 1.28. Its working fine.

Hey @cenci7 can you try on 1.28.0 and let me know if it works? If it still fails can you help with following:

Do adb logcat -c
Run maestro test which fails
Do adb logcat -d > output_logs.txt
Attach this output_logs.txt here.

I tested it again with version 1.27 and 1.28 and everything seems to be working fine.
Also tried it with MacOS Ventura and Monterrey and everything ok.
I don't know what caused it, but seems it's not happening anymore 👌
I close the issue.
Thanks anyway por the research!!


Got hash can't render

I just got hash CQgKBIBgh3h3h3iHeIcAAAAAAA== which can't rendered by browser:

file, err := os.Open("...")
if err != nil {
	log.Fatal(err)
}
img, _, err := image.Decode(file)
if err != nil {
	log.Fatal(err)
}
hash := thumbhash.EncodeImage(img)
print(base64.StdEncoding.EncodeToString(hash)
// CQgKBIBgh3h3h3iHeIcAAAAAAA==

If you test the thumbhash program in the repository, you'll see that this hash is decoded without any issue:
make build
./bin/thumbhash decode-image tmp.png CQgKBIBgh3h3h3iHeIcAAAAAAA==

I get the following image:
![](file:///home/galdor/dev/go/go-thumbhash/tmp.png)
There might be an issue with the way you use the Javascript library.

@galdor I expected that the resulting hash could be immediately saved to the database and later substituted into the src attribute, which the browser itself would decode. Your hash example also fails when used like this:

Ah I understand the confusion. You cannot do that. As the Thumbpage page explains, the hash contains the binary representation of a set of numeric parameters that can be used to recreate an image.
You can see these parameters with the dump-hash command I just added to the thumbhash program.
In the browser, you can use the official Javascript library to decode the hash and generate the image. There is a good example in the source code of the github.io page.

I'm closing the issue seems this is not a bug.

I believe that such processing should not be done in the browser. Especially in this context, because one of the tasks that the display of thumbnails solves is to improve UX. I will move further in the direction of doing it on the backend and putting it in storage.
Thank you for your response.

To anyone who got here with a task similar to mine:
file, err := os.Open("...")
if err != nil {
	log.Fatal(err)
}
img, _, err := image.Decode(file)
if err != nil {
	log.Fatal(err)
}
_ = file.Close()
hash := thumbhash.EncodeImage(img)
thumb, err := thumbhash.DecodeImageWithCfg(hash, thumbhash.DecodingCfg{
	BaseSize: 12, // result width
})
if err != nil {
	log.Fatal(err)
}
buf := new(bytes.Buffer)
if err := png.Encode(buf, thumb); err != nil {
	log.Fatal(err)
}
thumbEncoded := base64.StdEncoding.EncodeToString(buf.Bytes())
// thumbEncoded is your gem <3

@AlexanderMatveev thank you, I think this should be maybe part of the library


Update Setup.vdproj

After installling the CAT Instrumentation Framework Controller using the MSI, the tool fails to start with the error "Cannot Find ETWTools\tracefmt.exe".   The tool is expecting four files to be in the ETWTools sub-folder under the root install folder.   The MSI is not loading the files in the right place.   I updated the Setup project to load the projects to the proper location.   The changes are as follows:

Under Application Folders, add sub-folder called ETWTools

Modify Content Files for the Application Folder directory by adding the following two entries in the Exclude Filters:
default*.*
trace*.*

Add a Content File to the ETWTools sub-folder, and add one entry to the Exclusion Filters:
License*.*

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[Feature Request] [Minor] Copy timeline output as HTML

I think having the ability to copy the curl timeline output as HTML would be beneficial when using chat clients like Microsoft Teams. This feature would bring the color highlighting with it. If implemented it should probably be a preference.

I think it's fine to enable rich copy by default. You can always use Cmd+Shift+V (or Windows/Linux alternative) to paste unformatted if needed.


Lifecycle 0.6.1

Also fixes a mistake in the Gradle group where the wrong heroku/procfile version is used

I keep running into problems with the buildpacks, and I think we're going to need to update the cnb-shim so that it runs the v2 buildpacks in a tmp (writeable) dir

This update to cnb-shim introduces https://github.com/heroku/cnb-shim/commit/eece77504e9fd3fdf78b029c0bd804373c53d02d


On the User tab, I would like to know "WHY" that user has one of the four system permission.

On the User tab, I would like to know "WHY" that user has one of the four system permission.

something like this?

or maybe better this one?


Simplify memory management for dbs, conns, and statements

Hrm, CI is failing on Windows, and it may be a race condition if we garbage collect the database before we GC the connection?

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


On site support combined filter

Combine 'yes' and 'yes extra cost' options for onsite support question, plus some fixes for filter labels.
https://trello.com/c/BvnMwJZa/431-combining-web-chat-and-email-support-filter-options

I also approve 👍


Updated responses for Icelandic language.

Updating Icelandic responses.

Thanks, @schizza
I have been translating various open-source projects into Icelandic for decades now. Home Assistant is one of them. Looking forward to see this merged ;-)


Need help - newbie here

hi! i would like to ask how can i make the nightbot answer to the commands instead of having a message of my main account sayin "done!" and other responses. i probably have missed something while i was reading the instructions on how to install the timer for my stream, if that's the case i'm sorry, i just can't get the hold of streaming tools but i'm trying!
sincerely yours, Luschka

@luschka73 it's not possible to connect it with nightbot, you can create a new twitch account just for the bot and I believe you can also make nightbot use that account to send messages too. That way both will be using the the new twitch "bot" account.
You can make it so that it has a name unique for your channel, e.g. my channel name is MohFocus and my bot name is MohManager


Added Search

This PR added the search functionality required.

Doesn't Really if even it come. It is the same thing to bundle debounce from lodash or lodash.debounce. We will not ship whole lodash anyways.
Or if you really want to refactor this do that later.

Got it!
Nice terminal 👍 . What OS are you on?

In am on Ubuntu 16.04 LTS
I am using hyper as my terminal
Zsh as the shell
agnoster theme for oh my zsh
firacode font
you can get this on bash on windows or mac too!

I really want to use Ubuntu for developing, but after a while it gets slow and sometimes even freezes... And only running 2 terminals (one with server), a few Chrome tabs and Code editor. With the same things or more on Windows, no issues.

It has got better after linux 4.10 i have same issues in linux 4.8

Thanks @pantharshit00 !


Update to support new lib-common storage structs

One more fix is needed here for this to pass: https://github.com/openstack-k8s-operators/lib-common/pull/576

/test openstack-operator-build-deploy-kuttl

kuttl test failures on ci/prow/openstack-operator-build-deploy-kuttl  look valid

recheck

/test openstack-operator-build-deploy-kuttl

recheck

Seems to be the same error here:
❯ curl -s "https://storage.googleapis.com/test-platform-results/pr-logs/pull/openstack-k8s-operators_openstack-operator/1158/pull-ci-openstack-k8s-operators-openstack-operator-main-openstack-operator-build-deploy-kuttl/1852159512341909504/build-log.txt" | grep "missing from
 map"
    case.go:380: resource OpenStackDataPlaneNodeSet:openstack-kuttl-tests/edpm-extramounts: .spec.nodeTemplate.extraMounts.volumes.persistentVolumeClaim: key is missing from map
    case.go:380: resource Job:openstack-kuttl-tests/test-service-edpm-extramounts-edpm-extramounts: .spec.template.spec.volumes.persistentVolumeClaim: key is missing from map
    case.go:380: resource Job:openstack-kuttl-tests/test-service-edpm-extramounts-edpm-extramounts: .spec.template.spec.volumes.persistentVolumeClaim: key is missing from map

/test openstack-operator-build-deploy-kuttl

recheck

There seems to be genuine failure in the glance adoption:

There appears to be a genuine failure in the glance adoption process:
{"controller": "glanceapi", "controllerGroup": "glance.openstack.org", "controllerKind": "GlanceAPI", "GlanceAPI": {"name":"glance-default-internal","namespace":"openstack"}, "namespace": "openstack", "name": "glance-default-internal", "reconcileID": "26a4908c-b5c4-40ff-8e4e-bddcbe4c3e87", "error": "not all pods have interfaces with ips as configured in NetworkAttachments: [storage]"}

/test openstack-operator-build-deploy-kuttl

note: my last kuttl check might result in the same failure of the previous run [1], because go.mod should be refreshed to reflect the last hash associated to the lib-common update.
[1] https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/test-platform-results/pr-logs/pull/openstack-k8s-operators_openstack-operator/1158/pull-ci-openstack-k8s-operators-openstack-operator-main-openstack-operator-build-deploy-kuttl/1852671147426975744/artifacts/openstack-operator-build-deploy-kuttl/openstack-k8s-operators-kuttl/build-log.txt

recheck

/test precommit-check

@dprince from [1] I do not see an explicit kuttl failure like before. I assume we need to simply recheck.
[1] https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/test-platform-results/pr-logs/pull/openstack-k8s-operators_openstack-operator/1158/pull-ci-openstack-k8s-operators-openstack-operator-main-openstack-operator-build-deploy-kuttl/1853449635142569984/artifacts/openstack-operator-build-deploy-kuttl/openstack-k8s-operators-kuttl/build-log.txt

/test openstack-operator-build-deploy-kuttl

/test openstack-operator-build-deploy-kuttl

/test openstack-operator-build-deploy-kuttl

/test openstack-operator-build-deploy-kuttl


Add description about allocatableMemory.available in node eviction

/sig node
Add description about allocatableMemory.available in node eviction.  allocatableMemory.available signal is enable by default.

I see the Netlify build failed due to a timeout. Could you try to trigger the build again using
git commit --amend --no-edit && git push --force-with-lease

Hi @chenk008 Please rebase your branch

PR needs rebase.

@chenk008 Please resolve the conflicts.

Okay. Let's close this for now.
Two reminders sent to the author for resolving the conflicts and we got no feedback.
/close


Implemented additional delegates.

Implemented additional delegates in the Objective-C adapter.

Hi @tilltue  please review this PR for Objective-C branch. I have added additional delegate handling.

Thanks @tilltue


Fixed typo ad also added three back-ticks

Fixed typo on line 25, 'developer' was spelt 'developper'. Three back-ticks to indicate end of code highlighting was missing on line 41, so i added it.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you @archyonrails for your contribution!


AppointmentStore API is broken (AGAIN)

Every time Microsoft publishes an update to the Microsoft Calendar app, there is a high risk they brake the AppointmentStore API. This has happened again and the API has been unusable for 4 weeks now.
So the problem is that AppointmentStore.ShowEditNewAppointmentAsync does not work. It should open Microsoft calendar app window for adding a new appointment. Currently it just shows the splash screen and then closes immediately.
My app published in Microsoft Store does not work at all due to this issue and users are contacting me to fix the issue.
When do you fix this?
Could you please test your changes before publishing updates to the Microsoft Calendar app. You should understand that there are many 3rd party apps in Microsoft Store that depend on the AppointmentStore API so it must not be broken.
Test environment: Windows 10.0.19045.3208 (Calendar app 16005.14326.21508.0)

Please log this issue in the Feedback Hub.

Feedback Hub is just a waste of time. This needs to be fixed ASAP.

@kinex Posting in a repo that isn’t responsible for the specific issue (you said yourself the app breaks it, not the Sdk) is even more waste of time though?

@dotMorten This is the closest related repo I could find. If this is not the correct repo, at least Calendar app / AppointmentStore API developers could find this anyway, and someone knowing the correct team could point them out this issue (developers of this SDK and the Calendar app developers work in the same company, right?). I need to get this fixed in a way or another as my users are angry and they blame me about this of course. Only Microsoft can fix this.
I am not sure if the AppointmentStore API is part of this SDK or not, I suppose it might be or maybe at least should be. It is just an implementation detail of the AppointmentStore API that it launches the Microsoft Calendar app for several operations. And it is a very problematic implementation detail as we all 3rd party calendar app developers know.
And as a side note, if I have understood correctly the current Microsoft Calendar app is deprecated and is being replaced with a similar PWA app. This raises even more concerns. I hope they have not forgotten that the AppointmentStore API depends on the Calendar app currently. So what is your plan regarding the AppointmentStore API? I wonder if it is even technically possible to integrate the AppointmentStore API with their new PWA Calendar app. Maybe you should consider implementing fully independent AppointmentStore API in this SDK?

So a little terminology & history...
The AppointmentStore was designed as a brokering system between calendar readers and calendar providers.  The Windows calendar app is a reader of information in the user's appointment stores - you could imagine other apps like social calendaring systems also reading the appointment stores and doing something useful with the information. Mail & Calendar (and Outlook, and maybe others) are providers of appointment data.
Appointment readers aren't allowed to directly create an appointment on behalf of a provider, so we added methods for a reader to request that the provider show its own "edit and modify" experience.  That's the ShowEditNewAppointmentAsync method you linked - calling that invokes the provider to show a create-and-edit experience.
I'll hunt around for the Mail & Calendar owner to see if they know what's up.

@dotMorten The app ships with Windows inbox, so I'd argue it's closer to a platform API than not. And that's where posting here made sense--this repository was once accepting platform API bugs.

For the record, the Mail & Calendar teams are looking into this.  Cc @kentmai and @bencaratms


Try upgrading bundler on travis

THis is a test to see if it fixes the 1.9 issue.

Strange, not sure why this works (bundler is the same version before and after), but it does so ... strange

Oh, yea, 1.7 vs 1.15, alrighty then.

FYI, the underlying issue here is that Bundler only introduced required_ruby_version in Bundler 1.11. As a result, with Bundler 1.7 you could get a bundle that didn't work with you Ruby version (e.g., a ActiveSupport 5.1.4 on Ruby 1.9).


perf(css): small improvements

Description:
More atoms
BREAKING CHANGE:
No
Related issue (if exists):
No

CI failed

Thank you for fix :+1:


Detection boxes for something-somethingv2 dataset

I have found the detected bounding box for the something somethingv2 dataset here (https://drive.google.com/drive/folders/1XqZC2jIHqrLPugPOVJxCH_YWa275PBrZ). But they are stored in the split of 4 json files namely
bounding_box_smthsmth_part1.json
bounding_box_smthsmth_part2.json
bounding_box_smthsmth_part3.json
bounding_box_smthsmth_part4.json
Could you please help me are these the files I am required to give in the place of detected boxes?. If it is a different file, could you please provide the link to download the files?
I guess the detection boxes have to be stored according to their filename. Because I got the error as
No such file or directory: '../database/something-something/something-something-v2/detected_boxes/74225'
Is there any script to split the detected boxes of each file separately?
Thanks in advance

Hi, please check out the instructions here, you can now download these boxes.
Feel free to re-open this issue if you encounter any further issues.


port to python3

some basic testing has been done in downloader and restreamer,
but mostly this is just eyeballing and grepping for required changes.
Fixes #221

I have looked over your changes and I do not see any problems. I think the only way we are going to find any subtle bugs lurking in the code is by using the code so please merge when you are ready.


[DNM]bug 1959976: ceph: retry once before mon failover if mon pod is unscheduled

Events like node drain can take more than 10-15 minutes. If the node is not up before the default monTimeOut of 10 minutes, then rook will attempt to fail over the mon. This failover won't work as the node is still down.
This PR retries once before the mon failover if the mon pod is not scheduled
Signed-off-by: Santosh Pillai<EMAIL_ADDRESS>(cherry picked from commit f38829b57d0c14e28a203a8b7dd4325036d03eaf)
(cherry picked from commit 11e1d7d1ca0fbd623c6d7c23dc53c7e1e07e0e28)

Description of your changes:
Which issue is resolved by this Pull Request:
Resolves #
Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
[ ] Skip Tests for Docs: Add the flag for skipping the build if this is only a documentation change. See here for the flag.
[ ] Skip Unrelated Tests: Add a flag to run tests for a specific storage provider. See test options.
[ ] Reviewed the developer guide on Submitting a Pull Request
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.
[ ] Pending release notes updated with breaking and/or notable changes, if necessary.
[ ] Upgrade from previous release is tested and upgrade user guide is updated, if necessary.
[ ] Code generation (make codegen) has been run to update object specifications, if necessary.

Use draft if you don't want to see the PR merged.


feat: add default timeouts to Guzzle Client

Hi @mohammed-elhaouari,
Thanks for this awesome package.
It is recommended to have timeouts to HTTP clients, recently Laravel has added timeouts to their HTTP client too.
Read more:

https://divinglaravel.com/always-set-a-timeout-for-guzzle-requests-inside-a-queued-job
https://github.com/laravel/framework/pull/40466
https://github.com/laravel/framework/pull/40187

Thanks.

@mohammed-elhaouari
GitHub actions are failing due to composer  version 2

Yes, you can re-run the actions, which should take care of composer 2

Thanks, it will pass after merging for sure.
Please tag a new version.


Routing regression in 1.4.7

Routing behavior for variable paths has changed in 1.4.7 compared to 1.4.6. The variable path can no longer match multiple sub-paths.
For example, I have this route:
"/foo/bar/:path"
I expect it to be able to match anything after "foo/bar/", such as "foo/bar/a/b/c". As of 1.4.6, the HttpRouter behaved as expected, but this change (line 116 of HttpRouter.swift) that was released with 1.4.7 added a new behavior that requires the path to be the same length as the route.
Now, the route will only match on a path such as '"foo/bar/a"' and will not match on '"foo/bar/a/b"', since the path goes deeper than the length of the route.
Was this an intentional change or can this be reverted to the old behavior?

You can read more about it here https://github.com/httpswift/swifter/issues/405.


Uploading an empty file crashes RDMO

Description / Beschreibung
Uploading an empty file crashes RDMO and you don't even get a response. Not sure why.
Steps to reproduce / Schritte zum Reproduzieren

Go to /projects/
Click on 'Select file' (under "Import from file")
Select an empty file
"This site can’t be reached"

Context / Kontext
Chromium and Firefox on Debian 11

can not reproduce this, I only get this:

Import error
Files of this type cannot be imported.

The current dev-2.2.0 with the new projects overview does not crash for when import fails.
It just does not show anything about the failure, the request returns a 400.
Is this issue resolved like this or you wanna add some error handling on the import at /projects?

I think/hope it's resolved :smile:


npm install error

I've just cloned this repo and run npm install. Then I got this error
ERROR in unknown: Don't use path.replaceWith() with a source string, use path.replaceWithSourceString()
I'm using node 8.1.1 with npm 5.0.3
I have tried to follow the steps in Installation Debugging Solutions, but still got the error.
Then I tried to clear everything on my machine by delete all npm cache and uninstall node. Still got the error.

Getting this as well. +1

@Moai- Using yarn has solved this issue for me. Thank you.

This issue is actually caused by babili. You can temporarily disable it in your webpack production configs

I have the same problem, I tried to use yarn and also to disable babili without success.
I'm using node 8.1.1 and npm 5.0.3.

@fernandomantoan did you disable it in both webpack.renderer.prod.js and webpack.main.prod.js?

@amilajack Sorry, I had disabled only on webpack.main.prod.js, now everything is working, thanks.


AxiosError: Request failed with status code 401

When i run superbridge app local i meet this problem, i had add the .env file from .env.example, has something i need to config?

You need an API key to hit the backend. We were getting flooded with requests so now we restrict access.
Do you want to ping Alex_B_H on Telegram and I can help get you setup with one?

已收到

How to get an API key ?

已收到


Fix enumerate parallel via DiCE

Fixes #846 :tada:
Addresses #847

Simplifies TraceEnum_ELBO to use the DiCE gradient estimator.
Completely fixes parallel + sequential + MCMC enumeration logic (in all combinations).

Joint work with @eb8680 who wrote the first version of Dice_ELBO from which much of this is copied.

Nice!  FYI my less-tested version with extra Rao-Blackwellisation is here.

another question i have is the following;
let's say i only wanted it to be once differentiable. would it be simple to support that case and if so would it lead to a a performance gain? seems like the answer should be yes on both points?

let's say i only wanted it to be once differentiable

I think you could wrap it in a torch.autograd.Function with a @once_differentiable .backward().
Actually I think we could be more efficient by wrapping the entire (log_prob - log_prob.detach()).exp() op in a torch.autograd.Function that avoids computing log_prob, but that seems relatively low priority.

i'm unclear if rsvi will work correctly

I believe RSVI will work correctly, but will fail to take advantage of sticking-the-landing.

wouldn't it be sufficient to add a detach in one or two places if only_once_differentiable=True?

let's say i only wanted it to be once differentiable
wouldn't it be sufficient to add a detach

Actually the way we're using this in loss_and_grads should already be nearly as cheap as a once-differentiable version, since we're not passing create_graph=True to .backward(). In that sense, we're only using it in a once-differentiable way.


Possibility to use the same default classes in custom renderers

the current context passed to renderItem={({ title, arrow, depth, context, children }) does not contain the related active CSS classes that apply to the button according to the state change.
Would it be possible to get an example of how to reproduce the same exact output as in the default complex tree, using a renderItem override?

yes, this is what i meant. I would like to use Material UI as basis to style the sidebar, thus i would like to override some classes directly from Javascript.
The only catch in the code above is "where can i get the renderers  from within the renderItem callback?

i have just noticed that using this style object:
style={{                 paddingLeft:${(depth + 1) * 10}px,               }}
without the call to the renderers object seems to lead to the same default behavior. so that would be enough for my use case i guess

The renders prop contains the render logic, if you write your own renderers, you define those values yourself. The variable renderDepthOffset just defaults to the number 4, so if that's all you need, you can just inline the number or use a custom one as you mentioned. You can also use the default renderers by calling createDefaultRenderers({}) anywhere.

is it possible to pass a reference to the custom renderer to the tree item that the current node represents?
what i am trying to achieve here is  to open a menu on right click of a tree node and use a renameNode callback when the specific context menu item is selected.
I am having problem firing the "startRenamingItem" method from the current reference to the root tree. So my thought was to handle it directly like we do in the custom interaction object, i.e. accessing the "actions" object and using it in the custom renderer.
Maybe i should document that in a separate issue, but if i pass the TreeRef and call the startRenamingItem method from the ref, i get to see that only the onAbortRenamingItem is called, and not all other on* methods

Sorry, but I'm not completely sure what the problem is. In the render method of the item, one of the props should be a startRenamingItem method that you should be able to directly call (https://rct.lukasbach.com/docs/api/interfaces/treeitemrendercontext/#startrenamingitem). But using the imperative ref to the tree should also work...

hi @lukasbach  i have been experimenting with the complex-tree library and i have managed to create a custom ListItem that reacts to right clicks using the onContextMenu handler and displays a context menu:

now, i have 2 problems here:
1 - the underlying tree node of the sidebar is still selected and it is still listening to onclick events, so each time i click on a field in the contextmenu the event gets bubbled to the selected item, and this is definitely not what i want
2 - if i try to add a Rename Item field in the context menu and use the startRenaimingItem function, that will not work, i think for the same reason as in (1), i.e. that the renaming action is called immediately, perhaps because the selectedItem is "clicked" again
how can i force the selectedItem to still be focused (visually, so the user knows it has been selected) but disconnecting it from the onclick event listener?

just as extra information:

most of the component in the sidebar come from MaterialUI v4
i have tried adding event.stopPropagation() in the MenuItem onClick handler, but that does not prevent the eventListener on the sidebar node to fire
the MenuItem is a DOM child  of the sidebar node (thats why the event bubbles to it)

on problem number 2:
this is the log i get when trying to start renaming the item:

right after that i see the log of the renamedAborted callback

To avoid bubbling up click events from the context menu to the item renderer, I would first try to call e.stopPropagation() directly in the handler of the click events on the menu items, that should avoid the event bubbling altogether. If that doesn't work, you can try to implement a custom interaction manager that ignores events when the menu is open.
Regarding the form element being a descendant of a p element: If you start renaming, the item is rendererd in a form container. Looks like some material component you use for something further up renders in a paragraph element, maybe you can prevent that further up. In an accessible tree, you shouldn't have any paragraph elements further up in the tree structure anyways.

I'm having the same issue here, onFocusItem is being fired every time you trigger Drag, Drop or createInteractiveElementProps: onContextMenu and is being fired before them, so e.stopPropagation() won't work

Will close this since I don't see anything actionable on the original issue. chalscc, if this is still an issue for you, please create a new issue.


Resolve unique key issue in showcase and builder tools page

Checklist

[x] I have read the How to Contribute.
[x] I have run yarn build after adding my changes without getting any errors.

Updating documentation or Bugfix

Resolve unique key issue in showcase and builder tools page
Resolve Favourite SVG icon issue

tested locally & all seems to work as before ✔ ... out of curiosity, what was the issue?

classic react issue where each element in map should have specific key to distinguish elements from each other to increase performance, forgot to add those a year ago :D


Async generator functions plus generics cause confusing errors about Awaited on return

Bug Report
🔎 Search Terms
"asyncgenerator"
"async generator awaited"
🕗 Version & Regression Information
When checking the Typescript playground, the behaviour exists in 4.5.5 but not 4.4.4 and is still present in Nightly as of 5 May 2022.
⏯ Playground Link
Playground link with relevant code
💻 Code

async function* test<T, TReturn, TNext>(): AsyncGenerator<T, TReturn, TNext> {
  // const foo: Awaited<TReturn> = null as any // this would one be accepted as the return type in more recent typescript
  const bar: TReturn = null as any // Although this matches TReturn exactly, it is not accepted as a return value
  return bar
}

🙁 Actual behavior
Typescript rejects a return from an async generator that matches the return type argument from a generic AsyncGenerator, instead expecting an Awaited. This is confusing when creating generic helper methods dealing with generators, and also unexpected as presumably an async generator should be allowed to return both a T and Promise.
🙂 Expected behavior
Returning with a value matching the generic return type from AsyncGenerator should be accepted as value.

@rbuckton why is this? I understand why the IteratorResult would be of Awaited<TReturn> but not why the return value inside would have to be assignable to it.

Hmm. This shouldn't be an error since an async generator will adopt the return value whether or not it's a Promise. In other words, the following works but you shouldn't need the await:
async function* test<T, TReturn, TNext>(): AsyncGenerator<T, TReturn, TNext> {
  const bar: TReturn = null as any
  return await bar
}

I took a moment to analyze this as it seemed to be pretty interesting.
At first, I thought that unwrapReturnType is somewhat broken because Awaited<T> is just left as-is after passing through it (so it isn't really unwrapped). And arguably - it could probably be unwrapped here. However, I'm unsure how this would affect other things, but IIRC a simple & incorrect unwrapping, type.aliasTypeArguments[0], performed in the getAwaitedTypeNoAlias didn't break much. The only difference was the error reported here:
https://github.com/microsoft/TypeScript/blob/53343dd94d894ba141f53a1b2dde45d91e6b3a14/tests/baselines/reference/doNotElaborateAssignabilityToTypeParameters.errors.txt#L11
It changed from:
Type 'Yadda | Awaited<T>' is not assignable to type 'T'.
  'T' could be instantiated with an arbitrary type which could be unrelated to 'Yadda | Awaited<T>'.

to
Type 'T | Yadda' is not assignable to type 'T'.
  'T' could be instantiated with an arbitrary type which could be unrelated to 'T | Yadda'.

I don't think this was an incorrect change of the error fixture here but this kind of unwrapping definitely missed a lot of cases as it didn't actually instantiate this conditional type.
I think that instantiating it there, within getAwaitedTypeNoAlias, would probably yield good results and that it might be worth exploring.
However, I took a step back and analyzed how this compares to simple async functions and stuff. So the difference is that with async functions the return type of Promise<T> is indeed "unwrapped" and we operate on T as the "target" type. We can also try the reverse situation in the context of a simple async function - trying to assign Awaited<T> to T. This works because Awaited<T> is a "source" type in this scenario and structuredTypeRelatedTo can handle this situation (trying to assign a source conditional type to a target type param).
The problem is that the reverse isn't true - structuredTypeRelatedTo isn't capable of assessing that something is assignable to a target conditional type here. And this is what happens in the case of async generator (trying to assign T to Awaited<T>). A successful assignment can currently only be performed when the target type is conditional when if !c.root.inferTypeParameters && !isDistributionDependent(c.root) and this conditional type is distribution dependent (and it has to be).
So... I could try to instantiate this specific conditional type in getAwaitedTypeNoAlias and it would probably work. And maybe this is even worth doing as optimization or something, but a better solution (conceptually) would be to improve the algorithm around assignability when a target type is conditional. If you have any thoughts on how the improved algorithm could work - I could try to take a stab on this, but I'm incapable of figuring this out on my own.
Let me know what you think about this.


无法运行

python extract.py -o a 8013_1.fsb
Traceback (most recent call last):
File "extract.py", line 131, in
main()
File "extract.py", line 127, in main
exit(app.run(sys.argv[1:]))
File "extract.py", line 121, in run
self.handle_file(f)
File "extract.py", line 110, in handle_file
for sample_fakepath, sample_name, sample_data in self.read_samples(fakepath_prefix, fsb, ext):
File "extract.py", line 86, in read_samples
yield sample_fakepath, sample.name, fsb.rebuild_sample(sample)
File "C:\1\fsb5jb\fsb5_init_.py", line 211, in rebuild_sample
from . import vorbis
File "C:\1\fsb5jb\fsb5\vorbis.py", line 12, in
vorbis = load_lib('vorbis')
File "C:\1\fsb5jb\fsb5\utils.py", line 76, in load_lib
raise LibraryNotFoundException("Could not load the library %r" % (names[0]))
fsb5.utils.LibraryNotFoundException: Could not load the library 'vorbis'

From the README:

libogg and libvorbis are required to decode ogg samples. For linux simply install from your package manager. For windows ensure the dlls are avaliable (ie. in System32 or the directory you are running the script from). Known working dlls are avaliable as part of the release.

Whether you downloaded the Win64 or Win32 version (and not the source code tagged b7bf605), you should've obtained libvorbis.dll and libogg.dll. Just have them in the folder you're running extract.py from. Or, rename them vorbis.dll and ogg.dll and move them to C:\Windows\System32 (assuming you're running Windows).

Similar thing on my end:
  File "C:\Downloads\Carto - PC, PS4, NSW\NSW\Data\StreamingAssets\extract.py", line 131, in <module>
    main()
  File "C:\Downloads\Carto - PC, PS4, NSW\NSW\Data\StreamingAssets\extract.py", line 127, in main
    exit(app.run(sys.argv[1:]))
         ^^^^^^^^^^^^^^^^^^^^^
  File "C:\Downloads\Carto - PC, PS4, NSW\NSW\Data\StreamingAssets\extract.py", line 121, in run
    self.handle_file(f)
  File "C:\Downloads\Carto - PC, PS4, NSW\NSW\Data\StreamingAssets\extract.py", line 110, in handle_file
    for sample_fakepath, sample_name, sample_data in self.read_samples(fakepath_prefix, fsb, ext):
  File "C:\Downloads\Carto - PC, PS4, NSW\NSW\Data\StreamingAssets\extract.py", line 86, in read_samples
    yield sample_fakepath, sample.name, fsb.rebuild_sample(sample)
                                        ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Downloads\Carto - PC, PS4, NSW\NSW\Data\StreamingAssets\fsb5\__init__.py", line 211, in rebuild_sample
    from . import vorbis
  File "C:\Downloads\Carto - PC, PS4, NSW\NSW\Data\StreamingAssets\fsb5\vorbis.py", line 12, in <module>
    vorbis = load_lib('vorbis')
             ^^^^^^^^^^^^^^^^^^
  File "C:\Downloads\Carto - PC, PS4, NSW\NSW\Data\StreamingAssets\fsb5\utils.py", line 76, in load_lib
    raise LibraryNotFoundException("Could not load the library %r" % (names[0]))
fsb5.utils.LibraryNotFoundException: Could not load the library 'vorbis'``

Registering the provided DLLs, placing them into System32 or SysWOW64, nothing works.

you need to move libogg.dll and libvorbis.dll to the folder where extract.py is located.

you need to move libogg.dll and libvorbis.dll to the folder where extract.py is located.

Aaah, gotcha. Thanks. Somehow this isn't noticeable enough in the documentation. That said, I've stopped doing data mining, but at least others now know how to fix the issue.