Service Health query with multi tags

To support - https://github.com/hashicorp/consul/issues/294

This currently breaks all tests.
The param name should remain tag. I don't think hashicorp changed it?
If you just supply a string, tag should work as it did before. Just test the type to see if it's a list. If it is a list, encode it into the format expected by Consul.
You've made things less readable / maintainable by consolidating the handling of other params.
Your use of eval here is terrifying. Don't do that! haha.

Let me fix them.

I've created an issue #137


Add -fontforge flag to output UFO with FontForge comptatible glif names

I tried to import the UFO outputted from the command line into FontForge, however it did not recognize any of the glif names and added them as custom glyphs.
This change adds a command line argument to output glif names based on unicode values, which then imports correctly into FontForge.
I also added a requirements.txt file and updated the README to help make this a little easier for future new-comers.

Thanks for your work. I had no idea this could be useful for FontForge users too! ;-)
At first glance, I am reluctant to accept this, since this seems to be a workaround for a FontForge issue.
After some investigation, I noticed that Unicode values were not written into the glyph objects themselves (and therefore not present in .glif files). I updated the code to write the Unicode value, please let me know what FontForge does now.
Your edits to the README.md strip trailing spaces from various lines – those are needed for proper line breaks, and cannot be stripped. Probably that’s the editor at work, I set my editor to leave spaces alone when editing .md.
Let me know how it goes after this little update, and then we can work from there.

That seems to work much better than my hack! Thank you.
I admittedly know nothing about font software, have just been muddling my way through. You can see my attempt to add box drawing characters using FontForge here: https://github.com/vsalvino/terminex. I know it would be possible to script the entire process, just a matter of learning the right libraries and commands.

That looks great! Box Drawing Characters for the win! :-)
I am glad this small update improves the overall process. It seems that many coding projects are started out by just “muddling through” – it’s the result that counts!


CNV-38842: Create VM from upload file does not work

📝 Description
File was needed to be set in onFileInputChange property and when uploading a file from create VM flow, we upload the disk before the proccessed template's VM is getting populated with a namespace.
🎥 Demo
Before:
https://github.com/kubevirt-ui/kubevirt-plugin/assets/67270715/07acf543-ea54-4422-a01d-58a7d0576d5e
After:
https://github.com/kubevirt-ui/kubevirt-plugin/assets/67270715/2266f98c-c24c-4de5-b328-289ded248050

/lgtm

/cherry-pick 4.15

/cherry-pick release-4.15


Swift 5 update

Check List
Thanks for considering to open an issue. Before you submit your issue, please confirm these boxes are checked.

[ ] I have read the wiki page and cheat sheet, but there is no information I need.
[ ] I have searched in existing issues, but did not find a same one.
[ ] I want to report a problem instead of asking a question. It'd better to use kingfisher tag in Stack Overflow to ask a question.

Issue Description
What
[Tell us about the issue]
Reproduce
[The steps to reproduce this issue. What is the url you were trying to load, where did you put your code, etc.]
Other Comment
[Add anything else here]

The code should be already Swift 5 compatible. However, since Apple delayed the requirement of Xcode 11, there are still old versions of Xcode 10 being used to build apps with Kingfisher, we cannot drop set Swift version to 5 to the project setting yet.
We will update it as soon as Apple officially stops accepting old SDKs/Swift versions.


Consolidate common dependencies in sorald-parent/pom

We need to ensure that all common dependencies should be stored in sorald-parent/pom.xml. I noticed this because sorald and sorald-api use different versions of spoon-core.

Following the rules over here is recommended.


Side Scrolling doesn't work Logitech MX Master 3

Issue Type: Bug
Hi there,
the side scrolling doesn't seem to work reliable with the thumb wheel.
After switching through a lot of settings for the thumb wheel, till setting it back to horizontal scrolling it worked for a short period.
However after a short break it stopped working again.
I read that there was a similar problem with version 1.28.
My drivers and my vsc version are up to date.
cheers
PT
VS Code version: Code 1.45.1 (5763d909d5f12fe19f215cbfdd29a91c0fa9208a, 2020-05-14T08:27:35.169Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700K CPU @ 4.20GHz (8 x 4200)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
47.96GB (32.08GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (5)

Extension
Author (truncated)
Version

unity-code-snippets
kle
1.3.0

csharp
ms-
1.22.0

python
ms-
2020.5.80290

printcode
nob
3.0.0

unity-debug
Uni
2.7.5

/duplicate https://github.com/microsoft/vscode/issues/96986


Add protected SSH key support

An issue for this change is required

Description
This PR changes the functionality of the add key process to use an interactive shell so that the password prompt can be passed to the host/user to be entered manually.
I did some initial testing and research to try and use the existing --passphrase cli flag but couldn't find a secure way to either inject the passphrase or the decrypted key into the ssh-agent container. This PR removes the passphrase flag altogether in favor of forwarding the ssh-agent password prompt to the user.
I tested this and it works on Linux (intel, Ubuntu 23.04), MacOS (intel, Ventura), and Windows (intel, 11, WSL2+Docker Desktop). On windows using powershell/cmd.exe, the password is not hidden as you type, but at least it will not be saved in shell history. I also found an error with ssh key permissions on windows and included a fix for that.
Example usage where I enter an incorrect password the first time, but the correct password the second time:
➜  pygmy git:(ssh-private-key) go run . status
[*] amazeeio-mailhog: Running as container amazeeio-mailhog
[*] amazeeio-haproxy: Running as container amazeeio-haproxy
[*] amazeeio-dnsmasq: Running as container amazeeio-dnsmasq
[*] amazeeio-ssh-agent: Running as container amazeeio-ssh-agent
[*] Resolv Linux Resolver is properly connected
The agent has no identities.
 - http://docker.amazee.io/stats (amazeeio-haproxy)
 - http://mailhog.docker.amazee.io (amazeeio-mailhog)

➜  pygmy git:(ssh-private-key) go run . addkey --key ~/.ssh/pygmy-protected
Validation success for protected SSH key /home/brandon/.ssh/pygmy-protected
Enter passphrase for /home/brandon/.ssh/pygmy-protected:
Bad passphrase, try again for /home/brandon/.ssh/pygmy-protected:
Identity added: /home/brandon/.ssh/pygmy-protected (pygmy-protected)

➜  pygmy git:(ssh-private-key) go run . status
[*] amazeeio-mailhog: Running as container amazeeio-mailhog
[*] amazeeio-haproxy: Running as container amazeeio-haproxy
[*] amazeeio-dnsmasq: Running as container amazeeio-dnsmasq
[*] amazeeio-ssh-agent: Running as container amazeeio-ssh-agent
[*] Resolv Linux Resolver is properly connected
Q256 SHA256:tOuDj9Bqw6NsAZxxUfINhHD7ztoytOwNftGDepjNudA pygmy-protected (ED25519)
 - http://docker.amazee.io/stats (amazeeio-haproxy)
 - http://mailhog.docker.amazee.io (amazeeio-mailhog)

➜  pygmy git:(ssh-private-key) docker exec -it amazeeio-ssh-agent ssh-add -l
256 SHA256:tOuDj9Bqw6NsAZxxUfINhHD7ztoytOwNftGDepjNudA pygmy-protected (ED25519)

Resolves
This PR resolves issue #333
Non-standard tests for this change
If any non-standard testing is expected, please describe it here.

This one is testing well.
Unfortunately there's something triggering \ufffd to appear on the status output that can't be filtered out.
Otherwise looks good, and I'll merge it - all other testing is still passing.
https://github.com/golang/go/issues/48749

We should look to improve testing on this one in the future though.
By the way, thank you for the work on this.


Contracts deploy automation by using a continuous integration pipeline

As part of deploying new contracts to Mainnet, we have an internal checklist (aka "OUSD Deploy / Launch plans) which includes a battery of manual tests and verifications performed by the engineer doing the deploy.

Slither
Echidna
Smoke tests
Manual tests by invoking hardhat tasks from the cli
This is a labor intensive and error prone process.

By implementing a "CI" (Continuous Integration) pipeline, we could significantly reduce the amount of manual work necessary for deploys:
Phase I: Development and automated testing

Any commit on a branch and on master kicks off a series of tests using GitHub actions and runners. Including

Slither (we already have that)
Echidna. This is very CPU intensive (takes in the order of ~30min to run on my laptop). We could mitigate by using different strategies: parallelize the runs on multiple workers, only run for contract changes (vs DApp changes)
Smoke tests. Run those against a forked hardhat node. Notes:

We should expand those tests to also include the manual tests described in step 4 above.
When running in fork mode, the deploy script should also automatically execute the associated governance proposal that performs the upgrade - so that the tests run against the new contracts.

Phase II: Deployment (manual)

Once all the tests are green and the PR with the contract(s) change is merged to master, an engineer manually runs the deploy against Mainnet.
This does not take up much time. Though we might want to look into ways of improving recovery in case of failures (see #945 )

Phase III: Governance proposal validation (automated testing)

The deploy to Mainnet generates files (ABIs, hardhat deployment file) that must be committed to the repo via a PR. We can use this PR to run another round of validation to ensure the governance proposal works as expected on the newly deployed contracts
At this point, the upgrade hasn't been activated on Mainnet yet since the governance proposal associated with the deploy is still going thru approval, timelock (2 days) and execution
We could add a bit of logic so that when the tests run in fork mode, we check if there is any pending proposal (by querying the governor contract). If so, execute the proposal before running the tests.

For the recent deploy 38 of the staking contract upgrade, I went through the deploy checklist spreadsheet, with an eye to reordering or simplifying as needed. Not very much actually needed to be changed. It wasn't that bad.
I think the key thing that had made the past few deploys so painful is that I've spent an hour or two each time trying to get the smoke tests to run. Maybe there's something about my system that is causing the issue. I'm using a clean git clone for each deploy.
When I'm doing deploys and testing the deploy beforehand, I start a local forked node, then use brownie to play with the new contract changes and verify that they are behaving as expected. This always works very well. I'm wondering if we just add a smoke test function to our brownie world code, rather than continuing to work it in JS. I find this kind of testing in brownie much simpler to write and change, and we've already got all the pieces in place.

I don't think we need to echidna on each commit - only if we change the OUSD token, which happens very rarely. That's the only code that's currently covered by fuzzing tests.

Great data point.
I haven't run them for a while but I do remember the smoke tests being flaky.
If the brownie tooling can be used to have stable and equivalent in functionality smoke test vs the current JS then I don't think anyone will have any objection. Can JS and brownie coexist in the same repo?
Agreed Echidna could selectively run only when specific contracts are modified. Would be lot more efficient vs running for every single change.


Fix artifacts and publishing on AppVeyor.

Did not publish the NuGet packages to AppVeyor, so AppVeyor was unable to find the artifacts to publish to NuGet and GitHub.

There we go!
Artifacts are included in the build and are properly uploaded to the GitHub release and on NuGet.

I believe I marked the releases as draft so that you would have one final
chance to look at it before and edit any text.  Thanks for building this
library, it was exactly what we were looking for!
James Stumme
Pillar Technology
On Wed, Jan 4, 2017 at 7:55 AM, Felix Schröter<EMAIL_ADDRESS>wrote:

There we go!
Artifacts are included in the build and are properly uploaded to the
GitHub release and on NuGet.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/FelschR/aspnetcore-identity-documentdb/pull/11#issuecomment-270375592,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC6vvzkaX-LaPG4R6Cw_YqOkvHzfCkHgks5rO6TcgaJpZM4LaogD
.

An alternative would be what we do on our project.  We basically have the
production release kick off on a tag build.  So then the workflow become
create a github release which kicks off an appveyor build and deploys to
production.  There are a couple of yml changes, but it works great for us.
Let me know what you think.
On Jan 4, 2017 08:08, "Felix Schröter"<EMAIL_ADDRESS>wrote:
No problem, I needed this library as well, which is why I created it.
Opening up its source can only
benefit my project as well as the community.
The git releases are created as drafts, the NuGet package is directly
published (as prerelease) though.
I think there is unfortunately no way to create drafts on NuGet.
Maybe I should stop developing on the main (netcore) branch and only merge
my changes for actual releases/prereleases or similar. I'll think about
that.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/FelschR/aspnetcore-identity-documentdb/pull/11#issuecomment-270378276,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC6vv40ndtho_jaxLqD-ktsKLocBtlFgks5rO6fagaJpZM4LaogD
.


Implement to support tosa.depthwise_conv2d operation

implement for depthwise op

You'll need to update analyzeBlock in analysis.cpp as well.

Can we merge common parts of this and encodeOp(State &st, mlir::tosa::Conv2DOp op, ..)?

Build is broken?

Oops, the failures are from Z3 API updates. I will update them.

Ehh, no, they are compile failures in the Z3 header.

I think it's an internal error in z3


(Feature) - Replaces createProxy with createProxyWithNonce method no safe creation

Closes #1577 by:

Replacing the createProxy method with createProxyWithNonce that lets us add some salt for randomizing the created safeAddress

Travis automatic deployment:
https://pr1630--safereact.review.gnosisdev.com/volta/app

I'm finding quite odd that we are calculating the randomSalt value twice during the creation of the Safe.
I think we should pass the value as a parameter to the functions.

Yes make sense, I updated the code to do that

Travis automatic deployment:
https://pr1630--safereact.review.gnosisdev.com/volta/app

Travis automatic deployment:
https://pr1630--safereact.review.gnosisdev.com/rinkeby/app

Travis automatic deployment:
https://pr1630--safereact.review.gnosisdev.com/volta/app

Travis automatic deployment:
https://pr1630--safereact.review.gnosisdev.com/rinkeby/app

Travis automatic deployment:
https://pr1630--safereact.review.gnosisdev.com/volta/app

Travis automatic deployment:
https://pr1630--safereact.review.gnosisdev.com/rinkeby/app


Remove actions from sl-modal

hide() and show() actions don't seem to be used.  They both call method counter-parts which do get used.  There does not seem to be any reason to have the actions here.

@azizpunjani can you confirm this?

@notmessenger confirmed, we do not need those actions in there. open() and close() methods are called by service, show(), hide() actions are not being used anywhere.

@notmessenger confirmed, we do not need those actions in there. show() and hide() methods are called by service, show(), hide() actions are not being used anywhere.


Direct git to ignore commonly untracked Python and Jetbrains files.

This is a .gitignore file with files that should not be tracked by git.

the major parts of this were added in #264, thanks @MichaelWakahe !


#25 : added navbar in app.tsx; removed from homepage

resolves issue #25

@tymnec I have edited the comment. The PR resolves issue #25

Hmm.
Whenever you mention an issue you should write Resolves #issuenumber just Github rules to link PR with an issue.
Ok but still there are some conflicts. I will fix these conflicts.
@pradnyabhukan Thanks for your contribution.

Thank you for letting me know! I am still learning to contribute😅. Thank you 😄 @tymnec


Build master branch

As pointed out in @gianm's reviewing process 👍
Because of an old PR, the version was set hard at 0.9.2, but I prefer to build the master branch as it was before. This gives us the latest functionality and also the ability to easily try out the latest features.
Cheers, Fokko

Can we merge this one?


docs/networking: minor clarification to DHCP type

Make it explicit that the DHCP daemon being referred to is the one
that's part of the special CNI DHCP plugin.

I think the changes help a bit in the understanding the special nature of the dhcp daemon command. However, I think we can do better here. Adding that dhcp executable with daemon option is relying messages back and forth between another DHCP server on the network and the client inside the container.

@venekamp thanks for the feedback! What you are describing above sounds like a DHCP proxy, but I don't think it is the case for the current plugin. This one should just register itself as a DHCP endpoint (client) and hand out the assigned network config via CNI semantics (paging @squeed just to confirm I'm not wrong here).
If the above statement is correct, I think it is better described as a "DHCP (host) agent" more than a daemon or a proxy.

Well, it's a dhcp client daemon. It's a bit more than a proxy; it also keeps the leases alive while the container is running.
My suggested wording would be:

The DHCP IPAM plugin needs a client daemon to be running on the host.

@lucab, @squeed My first understanding was that it was a fully operational DHCP server, then I thought it was a proxying client and now I understand it to be more than just relaying DHCP messages. I would try to clarify what this "thing" does and what it expects. My try would be:

The DHCP IPAM plugin acts as a proxy between a dhcp client running inside the container and a dhcp service already running on the network. At the same time the plugin keeps the leases alive while the container is running. It can do this, because it still has the appropriate capabilities set.

You can omit the last sentence. At least I think this is why the plugin can renew leases and such things, while processes inside the container do no-longer have the capabilities to make changes to the interface.

updated, ptal

I think it reads well.


Improve type hints that were worsened by adding a second type argument to the Ref type

Vue version
3.5.6
Link to minimal reproduction
https://play.vuejs.org/#eNqlVNtu2kAQ/ZWpX5xIxCjcHqhDm1apmkq9KKXqQxxVxh7jTda71u4aiCj/3tk1AUMCrVQ/WPbOmTO3s7P0LssymFXoDb1QJ4qVBjSaqgQei+lF5BkdeaNIsKKUyoB5LBGWcJ1dikdYQaZkAf5bcm/rPFaY+q/3oTeYteCHmKu4pM+ND7n4G+gS1DNTJNpt8kukmKFgKNaEOaMvXMRFyVEPoZCVRpAzVFDo6XnLvjvu3YUzx+q8mIa0KjlLYoNpy1HPc5bkUMQPqAkXp0xMweRY44vK2qSieLFKMSVLLIAZmMcaJpiRJYhsbtq4uHBhY534H5FzCT+l4ukr/7SB6KwRS5gMYRcGqyaw2wS+wOnQroDP29L9WcypaVQxpUi1ZkxQDVIb/rhOU3IMuJye2GQDh14HbRg6hwzdjcEFPls/cCkgNgYLkoyRFHSxHZHeoOxjHZ2JFPCBLTANxyOqk/7CjTLoqKETC/gNYxJeVonEMClsMzp0fOJyGcL4dAi34zvSgkGRarglrzt4U0szHLccu+UkpuFO4GUkgNhMpRxpTUjFrZ43lorvOTX13Xuwp6kJUvmqBZPKzW4A2jDOSTMaKldJSdLxJ75zaAy5Vw+5c0Qw/SfIXxUz2EEelszeUHuHpt0/ZBg0ZBC2611BA6IfqwFOV4v+AML8fNRoYZjIFEf7lzNsu2OrmyTH5GF79Zx6grBNLMfYdsbyf2x1Wf/IEbYbxXotWo/UpYxNg3stBe1QJ67IS2RRMo7qa2m1Syt0WMvO2mIazvyTOzOqQlpH9bkL+8L5vV7Ys8j7plCjmmHkbWwmVlM0tfnq+xe6DA1jIdOKE/qI8QZpwpXNsYa9q0RKaTdwLttrt6dpRY71lb1v+qkom6hFrhw+8mh3vz9S+jbdbtBzfnTtqIu/aBqWkxrYDfrBwFv9ATlTNQE=
Steps to reproduce
Mouse over msg1, msg2, msg3 to see the type hints for each constant: you will see inner type several times instead of one, as before #11442 was merged.
What is expected?
Mouse over msg4, msg5, msg6 to see the type hints for each constant: you will see inner type just once, as it was before changes introduced in #11442.
What is actually happening?
After the changes introduced in #11442 the Ref type has 2 type arguments (for a long period of time it was just one). Type hints for both of  them are shown on mouse over.
They are usually the same, but still shown 2 times. This makes it much more harder to read the inferred type. Before the changes introduced in #11442 reading the types of Refs was much more convenient and I think it is important enough as ref is one of the core building blocks of every Vue application and reading complex Typescript types is hard enough already.
System Info
Vue Playground

Any additional comments?
In #11442 a new 2nd type parameter S was introduced to the Ref type.
This change makes type hints (like in VSCode) hard to read as the inferred type parameter is always duplicated.
Example:
const msg = ref('Hello World!')

Previously (during many years) when you mouse over on the msg in VSCode, you were seen this:
const msg: Ref<string>

Now you see this:
const msg: Ref<string, string>

This is not big deal when the type is simple like string, but when it is complex and multi-line, this duplication makes type reading much harder than it was before the change of the Ref type.
Is it possible to make the second type to be hidden in type hints when it is identical to the first type? Formally it is not identical as T will be UnwrapRef<T>, while S will be UnwrapRef<T> | T, but maybe there is a way to do it?
I'm not that good in TypeScript types, so can't suggest any solution, but I tried to just create an intermediate type like this:
type RefFixed<T> = Ref<UnwrapRef<T>, UnwrapRef<T> | T>

// An original `ref` declaration that is currently in Vue:
// export declare function ref<T>(value: T): [T] extends [Ref] ? IfAny<T, Ref<T>, T> : Ref<UnwrapRef<T>, UnwrapRef<T> | T>;

// A "fixed" declaration:
function ref2<T>(value: T): [T] extends [Ref] ? IfAny<T, Ref<T>, T> : RefFixed<T> {
  return ref(value)
}

(see the playground example above)
I.e. RefFixed<T> is used instead of Ref<UnwrapRef<T>, UnwrapRef<T> | T> in the return type of ref.
And looks like it helps with type hints.
So, maybe this or some more sophisticated solution could be implemented?

PR welcome.

LGTM! I agree with creating an intermediate type for better readability. Feel free to submit a PR if you’d like.

How about TypedRef?


SNOMEDCT codes in NuclearGradeVS may not be appropriate

The codes are children of 373375007 | Well differentiated histological grade finding (finding). It's confusing that the codes stem from histological grade, not nuclear grade.

The children of SCT#384736003 | Nuclear grade finding seem appropriate and align with CAP's guidelines on how to assign nuclear grade, despite describing characteristics beyond strict pleomorphism:
SCT#369781009 | Ductal carcinoma in situ nuclear pleomorphism, grade 1: monotonous nuclei, 1.5 - 2.0 red blood cells diameters, with finely dispersed chromatin and only occasional nucleoli (finding)
SCT#369782002 | Ductal carcinoma in situ nuclear pleomorphism, grade 2: neither nuclear grade 1 nor nuclear grade 3 (finding)
SCT# 369783007 | Ductal carcinoma in situ nuclear pleomorphism, grade 3: Markedly pleomorphic nuclei, usually >2.5 red blood cells diameters, with coarse chromatin and multiple nucleoli (finding)]
I will make the change.


Separate org CS score from normal accounts CS score

Harvest contract a contribution score (CS) table
Currently this table contains CS for organizations as well as accounts
Make it so it only contains CS for individuals
Organizations need their own CS table - could be scope

We might need to separate out all org tables from individual tables
See harvest contract here on this method
void harvest::ranktx(uint64_t start_val, uint64_t chunk, uint64_t chunksize, name table) {

This seems to be done already, CS seems to be calculate at:
void harvest::calccs(uint64_t start_val, uint64_t chunk, uint64_t chunksize) {
  require_auth(_self);

  check(chunksize > 0, "chunk size must be > 0");

  uint64_t total = utils::get_users_size();
  auto uitr = start_val == 0 ? users.begin() : users.lower_bound(start_val);
  uint64_t count = 0;

  while (uitr != users.end() && count < chunksize) {
    calc_contribution_score(uitr->account);
    count++;
    uitr++;
  }

So it seems it only iterates over users, not over orgs.

This seems to be already done:
ACTION harvest::setorgtxpt(name organization, uint64_t tx_points) {
  require_auth(get_self());

  tx_points_tables orgtxpoints(get_self(), "org"_n.value);

The table is scoped differently here.

Done


Bot "is typing...." for a very long time

Sometimes if I load a longer than usual playlist, the bot will be typing in the chat area. It may continue typing for more than hours sometimes. It kind of bug some of my members out as they don't know what the bot is doing.
I, myself, personally know it should be the bot processing the playlist. But sometimes for an hour plus, was what prompt me to wonder if there's a way around reducing the bot "is typing..." duration or disable it overall as a setting or sort. I never played anything longer than 200 songs. Not even once.
Just small quality of life things for some users.
To reproduce the issue mentioned:

Input a playlist URL from Spotify
Bot "joins" call, and begins "is typing..." process in the chat area

Latest Bot version
Ubuntu 20.04.3 LTS
VM Instance on Oracle Cloud

This bug still persist.

If the bot is still typing, then it's still processing the playlist.

I can put one-song only playlist, the same thing happens.

Please open a new issue with the correct issue template if still relevant


Branch developer guide update mail

This PR contains a partial update to Developer Guide for the Mail Functionality, I will be adding more to the part later.

Hi Sumanth! Might want to

Codecov Report

Merging #116 (bc74051) into master (bfe066b) will increase coverage by 0.65%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #116      +/-   ##
============================================
+ Coverage     62.86%   63.51%   +0.65%
- Complexity      570      606      +36
============================================
  Files           104      108       +4
  Lines          1909     2020     +111
  Branches        219      228       +9
============================================
+ Hits           1200     1283      +83
- Misses          642      662      +20
- Partials         67       75       +8

Impacted Files
Coverage Δ

.../java/seedu/address/logic/parser/TailorParser.java
72.00% <0.00%> (-6.27%)
:arrow_down:

...ava/seedu/address/logic/commands/ClearCommand.java
100.00% <0.00%> (ø)

...va/seedu/address/logic/commands/DeleteCommand.java
100.00% <0.00%> (ø)

...a/seedu/address/logic/commands/NewTaskCommand.java
100.00% <0.00%> (ø)

...eedu/address/logic/commands/RemoveTaskCommand.java
100.00% <0.00%> (ø)

...address/logic/commands/SetDefaultGroupCommand.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/model/Content.java
65.00% <0.00%> (ø)

...in/java/seedu/address/model/VersionedContents.java
71.73% <0.00%> (ø)

...java/seedu/address/logic/commands/RedoCommand.java
85.71% <0.00%> (ø)

...java/seedu/address/logic/commands/UndoCommand.java
85.71% <0.00%> (ø)

... and 3 more

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 bfe066b...bc74051. Read the comment docs.


Silent_fail that fails due to app running still reopens the app

When a run is set to silent_fail, if the app is running then it should just fail. Instead it is currently "reopening" the app that it already knows is open - this causes the app to grab focus, and potentially open a new window if the app was running with no windows visible.
Script result: 2022-12-13 14:55:38 : REQ   :  : shifting arguments for Jamf
2022-12-13 14:55:38 : REQ   : firefoxpkg : ################## Start Installomator v. 10.1, date 2022-12-02
2022-12-13 14:55:38 : INFO  : firefoxpkg : ################## Version: 10.1
2022-12-13 14:55:38 : INFO  : firefoxpkg : ################## Date: 2022-12-02
2022-12-13 14:55:38 : INFO  : firefoxpkg : ################## firefoxpkg
2022-12-13 14:55:39 : INFO  : firefoxpkg : BLOCKING_PROCESS_ACTION=silent_fail
2022-12-13 14:55:39 : INFO  : firefoxpkg : NOTIFY=success
2022-12-13 14:55:39 : INFO  : firefoxpkg : LOGGING=INFO
2022-12-13 14:55:39 : INFO  : firefoxpkg : LOGO=/System/Applications/App Store.app/Contents/Resources/AppIcon.icns
2022-12-13 14:55:39 : INFO  : firefoxpkg : Label type: pkg
2022-12-13 14:55:39 : INFO  : firefoxpkg : archiveName: Firefox.pkg
2022-12-13 14:55:39 : INFO  : firefoxpkg : App(s) found: /Applications/Firefox.app
2022-12-13 14:55:39 : INFO  : firefoxpkg : found app at /Applications/Firefox.app, version 107.0.1, on versionKey CFBundleShortVersionString
2022-12-13 14:55:39 : INFO  : firefoxpkg : appversion: 107.0.1
2022-12-13 14:55:39 : INFO  : firefoxpkg : Latest version of Firefox is 108.0
2022-12-13 14:55:39 : REQ   : firefoxpkg : Downloading https://download.mozilla.org/?product=firefox-pkg-latest-ssl&os=osx&lang=en-US to Firefox.pkg
2022-12-13 14:55:47 : INFO  : firefoxpkg : found blocking process firefox
2022-12-13 14:55:47 : INFO  : firefoxpkg : Telling app Firefox.app to open
2022-12-13 14:55:48 : INFO  : firefoxpkg : Reopened Firefox.app as username
2022-12-13 14:55:48 : INFO  : firefoxpkg : username
[message repeats 8 times]
2022-12-13 14:55:48 : ERROR : firefoxpkg : ERROR: blocking process 'firefox' found, aborting
2022-12-13 14:55:48 : REQ   : firefoxpkg : ################## End Installomator, exit code 12

Looks like this is a duplicate of 657 (and similar to 638).


Adding new table

Adding new table in voyager needs to make sure that the Primary ID has an auto_increment value otherwise you may receive an error when adding a new data type and not specifying an ID

+1 was getting an error when trying to add new entries into a table.

+1, I'm not able to find any option to add "auto_increment" value from admin panel.

I noticed that ID in new tables has auto_increment value, but it's removed when table is updated.

Seems to be fixed
Ping me if I'm wrong.


blend-file import does not import vertex colors

Godot version
4.1.1 stable-official
blender version 3.6.3
System information
macOS 13.5.2 (22G91)
Issue description
this issue is similar to https://github.com/godotengine/godot/issues/73056, but no longer on the blender side as of blender 3.5

vertex colors from .blend files are not imported.
weirdly, when exporting from blender as .glb, godot shows them correctly.

left: .glb import, right: .blend import

Steps to reproduce

create a blender file
create an object with vertex colors (face corner type)
save the blender file in a godot project and add to a scene
there are no vertex colors present
export the blender file as .glb
in the glb export there are vertex colors present

Minimal reproduction project
vertexcolorexample.zip

it seems the only issue was that the color checkbox in import settings was off.
not sure if this is a default, or i removed the ckeckbox...

but i think the issue can be closed.

Ran into a similar problem with Blender 4.1.1 and godot 4.2.1. Commenting here because google brought me here so many times when troubleshooting this, so others with the same problem might find this comment.  When using automatic blend-file import, vertex colors were not being imported. The solution ended up being to use the vertex color attribute as an input ANYWHERE in the material in Blender before exporting, basically to "tell" blender that you are using vertex colors. I think this is because, by default gltf exporter now only exports vertex colors if they are actually used in the shader.

@cottatheripper I suggest opening an issue on https://github.com/godotengine/godot-docs to track this, so it can be documented in the Importing 3D scenes documentation. You could also open a pull request that edits https://github.com/godotengine/godot-docs/blob/master/tutorials/assets_pipeline/importing_3d_scenes/available_formats.rst directly 🙂


Support color theme customizations specific to ColorThemeKind

Issue Type: Feature Request
Users can already set workbench.colorCustomizations and editor.tokenColorCustomizations and editor.semanticTokenColorCustomizations at two levels, either globally or for named themes.
I would also like to be able to set customizations based on the kind of color theme.
Possible syntax, using double square-brackets in place of the singles that denote a named theme:
"workbench.colorCustomizations": {
	"[[Dark]]": {
		"activityBar.activeBackground": "#222222"
	},
	"[[Light]]": {
		"activityBar.activeBackground": "#dddddd"
	},
	"[[HighContrast]]": {
		"activityBar.activeBackground": "#ffff00"
	}
}

One specific use case is for a semantic highlight provider extension that defines custom token types and would like to provide sensible default styling for these tokens in dark, light and high contrast themes.
VS Code version: Code - Insiders 1.47.0-insider<PHONE_NUMBER>0e48be84d86f5763e126c7c5bf22e0, 2020-06-30T09:00:22.588Z)
OS version: Windows_NT x64 10.0.18363

The suggested feature make sense, but I don't think the suggested use case is the right motivation.
It's ok for a user to define new theme colors, and override themes, but we don't want extensions to do it. I understand why semantic token provider would want that. But such colors will interfere with the work of theme authors.
Colors have to come from themes and the decision if/what to color also has to come from themes.
It's not possible to find a good default color that works with all themes. There are many things that have to work together, background color, various selection colors. If it doesn't look good, the theme will be blamed.

@aeschli I take your points re themes and theme authors, but if a language doesn't map well to the standard tokens and consequently the provider's SemanticTokensLegend contains mostly (or even exclusively) custom tokens, then either the language provider also has to provide one or more full themes (and convince users to use it/them), or else there will be little or no semantic coloring, right?
I wrote "full theme" because as far as I know it's not possible to subclass an existing theme and only provide additional color information.

I understand the dilemma. But actually it's no different with TextMate grammars/TextMate themes. A grammar can be as sophisticated as it wants, if themes don't assign colors to the scopes, there's no benefit to that.

have your custom tokens inherit from the standard tokens, as these are more likely covered by themes or have TextMate scope mapping
have your own theme to showcase the capabilities, and work with theme authors and we can improve the default themes
tell user how they can take advantage of semantic tokens through defining rules in editor.semanticTokenColorCustomizations

It turns out that editor.semanticTokenColorCustomizations and its friends aren't language-overridable, so an extension can't influence them through its configurationDefaults contribution. Probably the best (worst?) it can do is give the user a block of settings to add to their settings.json or XYZ.code-workspace file. Maybe even implement a command that adds this block. Then leave it to the user to tweak the colors if they don't play nice with the theme.
Being able to supply [[Dark]], [[Light]] and [[HighContrast]] variants would be nice. Otherwise I might use the ColorThemeKind of the active theme to pick the right variant, then either add it in a theme-specific section or add it where it applies to all themes. In the latter case if the user changed to a theme of a different ColorThemeKind they might want to rerun the command.

I'd like to add my +1 vote for this feature. My use case would be purely from a user's side, who sometimes switches to a different theme based on my mood, light conditions etc. I'd like my customisations to work with dark and light themes in general, as it's difficult to define them for each theme I might be using in the future.

Thanks, I've submitted my thumbs up. I feel, however, that the title may be too developer-specific and may not be clear to the average user. Something like "theme colour customisation for dark/light/high contrast theme kind in general" sounds clearer to me, though I'm sure that can be improved further.


Modified RDS DB Engine version

Issue #, if available:
The RDS MySQL DB EngineVersion is old and not available in the RDS console and hence the CLoudFormation Stack rollback.
Description of changes:
Modified the EngineVersion from 5.7.28 to 5.7.33.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The Engine Version 5.7.33 also does not exist now.
May be we can go for Engine version 5.7.37


Help

Operation not permitted - Note that ICMP messages can only be sent from processes running as root. What do I do?

Unfortunately you cannot create ICMP packets without root privileges, I have plans to address this using an alternative library soon


Set auth strategy in the auth rule

It would be nice to specify what strategy should be used to retrieving a specific role for a rule. Now if you have multiple auth stategies plt goes through all of them until find a correct one. This also cause some problems if a request contains data that match more than one stategy (might happpen when you moving from one stategy to another).

this would be a good option indeed. PR?


Refactoring gameStats

WIP on the opponent player methods onePointTeam and onePoint

WIP on twoPoints event

Done ThreePoints and Assists, BTW done with onePoint and twoPoints

WIP I done rebounds (off & def) offProvFouls and steals

WIP done blocks defProv fouls and onePointMiss

Done

Done with stats now we're going with summary

Summary done

Replacement done

Done with playerSettings


The Inferenservices can't working properly!

/kind bug
What steps did you take and what happened:
When I deploy a isvc, the ksvc is work properly, but isvc tip not ready, the error is
NAME    URL   READY     PREV   LATEST   PREVROLLEDOUTREVISION   LATESTREADYREVISION             AGE
craft         Unknown          100                              craft-predictor-default-zstwg  12h
the ksvc status is:
NAME                      URL                          LATESTCREATED                   LATESTREADY                     READY   REASON
craft-predictor-default   http://craft.example.com   craft-predictor-default-zstwg   craft-predictor-default-zstwg   True
when I describe isvc: the status is
Status:
Components:
Predictor:
Latest Created Revision:    craft-predictor-default-zstwg
Latest Ready Revision:      craft-predictor-default-zstwg
Latest Rolledout Revision:  craft-predictor-default-zstwg
Traffic:
Latest Revision:  true
Percent:          100
Revision Name:    craft-predictor-default-zstwg
Conditions:
Last Transition Time:  2021-03-11T07:26:30Z
Message:               Waiting for load balancer to be ready
Reason:                Uninitialized
Severity:              Info
Status:                Unknown
Type:                  PredictorConfigurationReady
Last Transition Time:  2021-03-11T07:26:30Z
Message:               Waiting for load balancer to be ready
Reason:                Uninitialized
Status:                Unknown
Type:                  PredictorReady
Last Transition Time:  2021-03-11T07:25:59Z
Severity:              Info
Status:                True
Type:                  PredictorRouteReady
Last Transition Time:  2021-03-11T07:26:30Z
Message:               Waiting for load balancer to be ready
Reason:                Uninitialized
Status:                Unknown
Type:                  Ready
What did you expect to happen:
How to make it to Ready
Environment:

Istio Version: 1.3.5
Knative Version: 1.18.0
KFServing Version: 0.5.0
Kubeflow version: 1.12.0
Kfdef:[k8s_istio/istio_dex/gcp_basic_auth/gcp_iap/aws/aws_cognito/ibm] istio_dex
Kubernetes version: (use kubectl version): 1.18.0
OS (e.g. from /etc/os-release): coreOs

That looks like your load balancer issue

That looks like your load balancer issue

@yuzisun I Loadbalancer can access, I don't think so.

@jidemoski were you able to figure this out? Same issue here and no clue what's going on

@jidemoski @amybachir
Is "example.com" resolves to your Loadbalancer?
The default config of knative has a default domain name of "example.com", which may not meets your cluster setting.
See this doc for how to customize it.

@jidemoski thanks for taking the time to look at this. I’m not using the default config. I have customized this config map for my domain name.
I’m tracking my issue here #1900


Enforce Policy Checking

Background
The current state of Red Hat’s deployment engine for Ansible Automation Platform in Microsoft Azure gives customers a containerized process of their Ansible Automation deployment.
Simply put, this engine deploys Ansible Automation on Microsoft Azure, walking through the entire deployment process, displaying deployment step completion in a serial manner in real time,  and failing at the end of the processes if any issues have been found.
If the deployment has failed, the customer then has to fix related issues, and redeploy the entire instance themselves.
As a Azure customer utilizing the deployment driver, I want the driver to policy check my deployment to verify and enforce my environment early on to prevent any late failures.

The terms "policy check my deployment" and "verify and enforce my environment" need to be specified. Also, this should either be an Epic or a User Story, not both at the same time I would say.

I am looking to address this is the cross-repo issue that I listed above.  I am understanding the "policy check my deployment to mean":

prior to deployment, execute a DryRun to determine what changes will be made during the deployment
if there is an Azure policy applied to my subscription/scope that prohibits the deployment, it will be caught

As an example, if I have an Azure policy applied to my resource group that says "Do not deploy this Azure resource type unless it has the property "publicNetworkAccess": "Disabled", and the deployment template has it set to "Enabled", the DryRun will catch the policy violation before attempting the deployment.
I am adding more implementation details and pre-requisites to the cross-repo issue listed above.


Release comes with Wasm files

Currently the follow wasm files are packaged with this project.
445K Aug  9 12:12 node_modules/near-api-js/examples/cookbook/utils/wasm-files/staking_pool_factory.wasm
108K Aug  9 12:12 node_modules/near-api-js/examples/cookbook/utils/wasm-files/status_message.wasm
86B Aug  9 12:12 node_modules/near-api-js/test/data/main.wasm
348K Aug  9 12:12 node_modules/near-api-js/test/data/multisig.wasm
348K Aug  9 12:12 node_modules/near-api-js/test/wasm/multisig.wasm

Which is about 1.3MB.
Given that their directories are cookbook and test I assume they are not needed.

Already fixed here: https://github.com/near/near-api-js/pull/631


feat: support cairo1 execute calldata

Motivation and Resolution
With Cairo1, Account contracts use simpler calldata for execute. The differences are as below:
func __execute__(
    call_array_len: felt,
    call_array: CallArray*,
    calldata_len: felt,
    calldata: felt*
)

struct CallArray {
    to: felt,
    selector: felt,
    data_offset: felt,
    data_len: felt,
}

Example:
call1 = { to: 0x8001, selector: 0x5E11, calldata: [0xCAA1, 0x1001] }
call2 = { to: 0x8002, selector: 0x5E12, calldata: [0xCAA1, 0x1000, 0x0002] }

0x0002 (call_array_len)
-- call_array
0x8001 (to1)
0x5E11 (selector1)
0x0000 (data offset1)
0x0002 (data len11)
0x8002 (to2)
--
0x5E12 (selector2)
0x0003 (data offset2)
0x0003 (data len12)
--
0x0005 (calldata_len)
-- (calldata)
0xCAA1
0x1001
0xCAA1
0x1000
0x0002

//////////////// CAIRO 1

fn __execute__(calls: Array<Call>) -> Span::<felt252> {

}

struct Call {
    to: ContractAddress,
    selector: felt252,
    calldata: Array<felt252>,
}

call1 = { to: 0x8001, selector: 0x5E11, calldata: [0xCAA1, 0x1001] }
call2 = { to: 0x8002, selector: 0x5E12, calldata: [0xCAA1, 0x1000, 0x0002] }

Example:

0x0002 (calls size)
    0x8001 (to 1)
    0x5E11 (selector 1)
    0x0002 (calldata 1 length)
       0xCAA1 (calldata 1)
       0x1001 (calldata 1)
    0x8002 (to 2)
    0x5E12 (selector 2)
    0x0003 (calldata 2 length)
        0xCAA11 (calldata 2)
        0x00000 (calldata 2)
        0x00002 (calldata 2)

The example implementation can also be found here
https://github.com/starkware-libs/cairo/blob/49637d619bbfdd339df29dae282e3a9f77014ebe/crates/cairo-lang-starknet/test_data/account.cairo#L69

Usage related changes

For dapps and users, there won't be any change. It's upto the wallet provider to detect correct Cairo version an Account is using and pass it in transactionDetails of execute. Currently supported via cairoVersion?: "0" | "1".
Note: if cairoVersion is not passed, it defaults to 0

Checklist:

[ ] Performed a self-review of the code
[ ] Rebased to the last commit of the target branch (or merged it into my branch)
[ ] Linked the issues which this PR resolves
[ ] Documented the changes
[ ] Updated the docs (www)
[ ] Updated the tests
[ ] All tests are passing

Suggestions for better function name than *_cairo1 will be appreciated


Could i help with this

Description
Is there a way i could contribute to this plugin?
Implementation ideas
No response

Hi @CoderGuyTedd there is always help wanted.  If you're good with C++,  you're welcomed to implement any bug fixes or suggestions found on Discord or here on the repository. If C++ isn't your jam, there is documentation in the https://github.com/CraterCrash/godot-orchestrator-docs that could always use some help or you can also implement and submit examples to https://github.com/CraterCrash/godot-orchestrator-examples


🐛 BUG:  Wrangler 3.69.0 breaks local dev with Error: Cannot find module<EMAIL_ADDRESS>

Which Cloudflare product(s) does this pertain to?
Wrangler
C3 (npm create cloudflare)
What version(s) of the tool(s) are you using?
3.69.0 [wrangler]
2.23.0 [c3]
What version of Node are you using?
18.20.4
What operating system and version are you using?
Mac Sonoma 14.5
Describe the Bug
Observed behavior
Please describe.
create a new project with
category: hello world example
lang: typescript
git: yes
deploy: no
immediately run
npm install
npm exec wrangler dev  <-- will fail with following error
✘ [ERROR] Error reloading local server: Error: Cannot find module<EMAIL_ADDRESS>Expected behavior
Please describe.
run dev server as usual
Steps to reproduce
Please provide the following:

A minimal working subset of your worker code
A minimal working subset of your wrangler.toml
Commands used to start your local dev server, including custom env and cli args
Steps to be performed in the browser, curl commands, or a test we can run that reliably fails (at least a percent of the time)

create a new project with
category: hello world example
lang: typescript
git: yes
deploy: no
immediately run
npm install
npm exec wrangler dev  <-- will fail with following error
✘ [ERROR] Error reloading local server: Error: Cannot find module<EMAIL_ADDRESS>100% replicable
A git repo we can clone and run a test suite on, or which has a README with step-by-step instructions, is even better. In this case, please use the field below to provide a link to the minimal repro.
Please provide a link to a minimal reproduction
No response
Please provide any relevant error logs
No response

Thanks for reporting, can reproduce, we are sorting out right now.

A fix is in review - just waiting for CI jobs to merge and then we'll run another release.


[FEAT] Add odemis as an openfibsem backend

Hello @patrickcleeve2! Thanks for opening this PR.


Recipe improvements

@doktoric

Recipes are associated with host groups instead of the cluster itself. Also one host group can contain multiple recipes (e.g.: spark and GCS connector).
Recipe plugins now have an execution type - one-node or all-nodes. If one-node is set, the plugin will only be installed on one of the nodes in the host group. It can be used for example to run HDFS commands in the plugin.

DB migration is needed, I'll send the PR in that repo soon.

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/904/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/907/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/908/


Adjust FSM handling of non-broadcast

Depending on how the calling sketch uses rf12_recvDone and thence rf69::recvDone_compat. It is possible to stall in TXIDLE after a uninteresting non-broadcast packet is delay discarded rather than discarded immediatley.

I've merged a somewhat altered version of this code - I think the whole problem can be ascribed to "rxstate = TXIDLE" being set too late in the "case TXRECV:" branch of the switch.

Does this not have the same problem outcome for rf12_canSend() since the packet isn't fully discarded until a call into rf12_recvdone happens. I think this code still has the stall..

Ah, I see your point. But hasn't this always been the case: when recvDone returns true, you get a brief hold on the data, and things resume on the next recvDone call. The RF12 code is written under the assumption of lots of polling to keep the FSM moving.
Hm.. need to think more about this, am re-opening this for now.

Yes, I think it has always been the case but I don't think it needs to be. Once it comes out of TXRECV it has to deliver something because rxfill != 0 if nothing else.


Untitled file won't close annoying bug

Hi there,
As a simple user, I'm reporting an annoying bug :)
How to reproduce it with Spyder v<IP_ADDRESS> on Windows (10):

Close all opened files.
Restart Spyder: there should be only the "temp.py" file, as expected.
Leave the temp.py file opened.
Open 3 or 4 files.
Close all files (Ctrl+Shift+W).
==> The temp.py file is renamed "untitled0.py" and Spyder ask if it should be saved before closed.
Ok, why not, but, when answering "No", the "untitled0.py" file is renamed "untitled1.py" and Spyder asks again if it should be saved before closed, and so on and so forth.

-Pierre

Hey @PierreRaybaut!! Good to see you're still using Spyder :-)
This is a duplicate of issue #2625. I'll fix it on 2.3.6.

Sorry, I forgot to mention that 2.3.6 will be released next week. It's main purpose is making Spyder compatible with IPython/Jupyter 4.0.

Yep. Still using it :)


⚠ Bofrost.de has degraded performance

In be9b382, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1935 ms

Resolved: Bofrost.de performance has improved in 2b6645a after 7 minutes.


Call ctrl.Finish() in every test

Pull Request Test Coverage Report for Build 120739006

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+8.2%) to 71.608%

Totals

Change from base Build 120738347:
8.2%

Covered Lines:
285

Relevant Lines:
398

💛  - Coveralls


Destroy Joyride in Foundation

Is there a way to destroy the joyride provided by Zurb Foundation problematically. I am trying to destroy the joyride like this
$('#joyride-list-location-creation').joyride('destroy');

But I am getting this error

Uncaught TypeError: $(...).joyride is not a function(…)

$('#joyride-list-location-creation').remove();

is the way to go.


with latest neovim 0.5.0, this error was noticed

With the latest Neovim 0.5.0, This error was noticed.
E5108: Error executing lua ...ny/.nvim/plugged/nvim-lspinstall/lua/lspinstall/util.lua:11: attempt to index a nil value

Please help fix this.

i see the same error, and it seems to have something to do with the setup of additional language servers.
i added a config for pyls before, and when i remove it, the error disappears.
-- pyls config
local config = require"lspinstall/util".extract_config("pyls")
config.default_config.cmd[1] = "./venv/bin/pyls"

require'lspinstall/servers'.python = vim.tbl_extend('error', config, {
  install_script = [[
  python -m venv ./venv
  ./venv/bin/pip install -U pip
  ./venv/bin/pip install -U 'python-language-server[all]'
  ./venv/bin/pip uninstall yapf autopep8
  ./venv/bin/pip install -U 'pyls-black'
  ./venv/bin/pip install -U 'pyls-mypy'
  ]]
})

this should look like the custom installer described in the readme - did the setup change somehow?

turns out, there was just no default config for pyls anymore.
(see https://github.com/neovim/nvim-lspconfig/pull/1074)
switched to pylsp, and its working again.
maybe its something similar for you @antonyr ?

@puhoy I was not using pyls. But this error didn't occur after I rebuilt neovim on my M1 from scratch.


Subject Area API regression after big refactor

Prior to the big refactor a delete of a term in the FVT was interpreted as a a soft delete, the response contained the deleted Term (and I assume the other nodes)
After the refactor the delete term does not contain the deleted term.
The API needs to be re instated and the FVTs amended to test this, the glossary author view service needs amending as well

@immmus is this something you want to look at?

@davidradl Hi, after refactoring, the removal is still soft. There is no remote term in the answer just for the reason that the remove method in RepositoryHadler is void. But the deleteEntity method from OMRSMetadataCollection is still used. And before they sent it, I scratched it all out, the method did not change in any way.

@planetf1 @davidradl  I've just ran test and we are can see the results. All test's without errors.

https://gist.github.com/immmus/e87b80b697e4208a9aa9779fecf8260b
And can see soft deleted.

The problem most likely is that by running tests from different methods, they run in parallel and therefore we always get different search results.

@planetf1 @davidradl
In this form, tests pass in parallel.

It's results:
https://gist.github.com/immmus/fbfc75da6a5bfd44a8c53a977ab506f9
In the future, there is an intersection of the logic of tests! As can be seen on the next slide.

So friends, let's objectively look at the situation and not come up with theories about some kind of regressions. In this case, the problem is that the logic of the tests is not isolated from each other and when they are run in parallel, they simply break.

Here are some more screenshots that I missed.

@immmus I have a look further into this; it is more subtle than I originally thought.

subject area api has 2 delete APIs for entities and relationships. One for soft and one for hard.
the big refactor change the implementation to use the repository handler removeIsolatedEntity; which issues a deleteEntity, which is escalated to a purge - so does not ever return Function not supported as it did before the big change.
I agree it is better to be using the repository handler, but this means that we need to remove the purges of nodes and relationships. I think we can keep restores.
I have been talking with @mandy-chessell, she is making changes to the repositoryHandler to have it check the providence. The repository handler will be enhanced to only allowed through API calls that target non external repositories.
Because of the last point. Mandy suggests that subject area  use the repository handler removeEntity rather than removeIsolatedEntity - this will check provenance when her change goes in.

@immmus I have a look further into this; it is more subtle than I originally thought.

subject area api has 2 delete APIs for entities and relationships. One for soft and one for hard.
the big refactor change the implementation to use the repository handler removeIsolatedEntity; which issues a deleteEntity, which is escalated to a purge - so does not ever return Function not supported as it did before the big change.
I agree it is better to be using the repository handler, but this means that we need to remove the purges of nodes and relationships. I think we can keep restores.
I have been talking with @mandy-chessell, she is making changes to the repositoryHandler to have it check the providence. The repository handler will be enhanced to only allowed through API calls that target non external repositories.
Because of the last point. Mandy suggests that subject area  use the repository handler removeEntity rather than removeIsolatedEntity - this will check provenance when her change goes in.

The difference between removeEntity and removeIsolatedEntity is that it checks Properties, but then calls removeIsolatedEntity anyway. Or mb, I don’t understand something?

@immmus yes you are right, but Mandy is making a change to the implementation of these methods (not yet in master) that means we should use removeEntity. So when her changes go in - it will work as expected.

I see

and

@immmus So removeEntity calls removeIsolatedEntity which calls deleteEntity which if it fails calls purgeEntity. I suggest you do not touch this code, Mandy is updating this code to cope with provenance.

@immmus So removeEntity calls removeIsolatedEntity which calls deleteEntity which if it fails calls purgeEntity. I suggest you do not touch this code, Mandy is updating this code to cope with provenance.

Yes I understood it. Let's to wait when @mandy-chessell will make a changes and we will make ours. Okay?

Okay.

Okay.

Thank you:)


[BUG] enabling webpush notification give SQL syntax error message

Describe the bug
When enabling push notifications on a profile "look and feel" page, the following error pops up in a red box:
(psycopg2.errors.SyntaxError) syntax error at or near "user" LINE 2: ... INSERT INTO webpush_subscriptions (endpoint, user, data... ^ [SQL: INSERT INTO webpush_subscriptions (endpoint, user, data, host) VALUES (%s, %s, %s, %s) ] [parameters: ('https://fcm.googleapis.com/fcm/send/dmHZy5d30lM:APA91bH8LPJc_gJMFqKMB7xH98o9OIbUkAacJPa-XIwAkycnNoI12k1TU9thlzCucpJiM4p_ucmsz2clq99rWnxsvcrqdfyz0xSVFqz6NNHIydJ4gofoY9loKo-4NJMMWj5-olpprc5d', '9ecf81fe79554109a8981b5e16a80dfb', '{"endpoint":"https://fcm.googleapis.com/fcm/send/dmHZy5d30lM:APA91bH8LPJc_gJMFqKMB7xH98o9OIbUkAacJPa-XIwAkycnNoI12k1TU9thlzCucpJiM4p_ucmsz2clq99rWnxs ... (67 characters truncated) ... me":null,"keys":{"p256dh":"BCdzHpFXe9yXTCnllgIenPFTxQecw-bvvIpWneuDtmP52BHXuaKFU1l9zvFb6SYX757eYIT2Z5u2yZ0wC70N0sE","auth":"0jFIovVeEb8T-FJhBq0eJA"}}', 'demo.lnbits.com')] (Background on this error at: http://sqlalche.me/e/13/f405)
500

To Reproduce
Steps to reproduce the behavior:

Use lnbits on mobile
Click on 'My account' -> 'Look and Feel' -> 'Notifications'
See error

Expected behavior
I certainly wouldn't expect a psycopg2 error with an SQLITE-based installation.
Desktop (please complete the following information):

LNbits version: 35bb3cc94b31b8d6fa83ee86be82e3045c17a24e
Database sqlite

Additional context
The same problem is seen with demo.lnibts.com.

Seeing that some SQL parser was offended by the key word user, I've tried to double-quote it in my instance's core/crud.py, and it seems to help.
diff --git a/lnbits/core/crud.py b/lnbits/core/crud.py
index 4ad1fca7..ba3cd892 100644
--- a/lnbits/core/crud.py
+++ b/lnbits/core/crud.py
@@ -1256,7 +1256,7 @@ async def get_webpush_subscription(
     endpoint: str, user: str
 ) -> Optional[WebPushSubscription]:
     row = await db.fetchone(
-        "SELECT * FROM webpush_subscriptions WHERE endpoint = ? AND user = ?",
+        'SELECT * FROM webpush_subscriptions WHERE endpoint = ? AND "user" = ?',
         (
             endpoint,
             user,
@@ -1269,7 +1269,7 @@ async def get_webpush_subscriptions_for_user(
     user: str,
 ) -> List[WebPushSubscription]:
     rows = await db.fetchall(
-        "SELECT * FROM webpush_subscriptions WHERE user = ?",
+        'SELECT * FROM webpush_subscriptions WHERE "user" = ?',
         (user,),
     )
     return [WebPushSubscription(**dict(row)) for row in rows]
@@ -1280,7 +1280,7 @@ async def create_webpush_subscription(
 ) -> WebPushSubscription:
     await db.execute(
         """
-        INSERT INTO webpush_subscriptions (endpoint, user, data, host)
+        INSERT INTO webpush_subscriptions (endpoint, "user", data, host)
         VALUES (?, ?, ?, ?)
         """,
         (
@@ -1297,7 +1297,7 @@ async def create_webpush_subscription(

 async def delete_webpush_subscription(endpoint: str, user: str) -> None:
     await db.execute(
-        "DELETE FROM webpush_subscriptions WHERE endpoint = ? AND user = ?",
+        'DELETE FROM webpush_subscriptions WHERE endpoint = ? AND "user" = ?',
         (
             endpoint,
             user,

thanks for the report! i wonder how the quotes got lost!. can you PR you fix?

Fixed here: https://github.com/lnbits/lnbits/pull/2533


Claiming @everett_toews nodes with thanks to @Rackspace

Thank you @everett-toews you and @Rackspace are my heroes !!
We simply cannot go this far without your support.
I'm so sorry that it's got conflicts somehow :-( :-( :-( :-( :-(
could you please rebase it?

@chanwit Rebased and ready to go. Thanks!

You rock !!!
LGTM


Benchmark is unrepresentative of results for some callers

Following up on concerns I raised when the preprint first went up based on @Addicted-to-coding ABACBS presentation

bwa mem -a

The -a setting has a huge effect on the FDR of many callers. If the caller doesn't recommend -a, you shouldn't be benchmarking it with -a since it's not the bwa default.

We weren't able to find any clear documentation on how to use QUAL, and FILTER so those were ignored

If there's no documentation in the caller, then you should use the specifications definitions of those fields:
6. QUAL — quality: Phred-scaled quality score for the assertion made in ALT. i.e. −10log10 prob(call in ALT is
wrong). If ALT is ‘.’ (no variant) then this is −10log10 prob(variant), and if ALT is not ‘.’ this is −10log10
prob(no variant). If unknown, the MISSING value must be specified. (Float)

7. FILTER — filter status: PASS if this position has passed all filters, i.e., a call is made at this position.
Otherwise, if the site has not passed all filters, a semicolon-separated list of codes for filters that fail. e.g.
“q10;s50” might indicate that at this site the quality is below 10 and the number of samples with data is below
50% of the total number of samples. ‘0’ is reserved and must not be used as a filter String. If filters have
not been applied, then this field must be set to the MISSING value. (String, no white-space or semi-colons
permitted, duplicate values not allowed.)

You should not penalise callers for following the VCF file format specifications.

Also, the raw data can't be downloaded due to constant git lfs errors about the repository exceeding it's lfs download quota.

We will consider this in the next release of the analysis
Thanks for pointing this out


Setting for automatically loading schema used in minDynatraceVersion

Summary
A setting in VSCode that will run the Load schema function automatically each time the minDynatraceVersion is changed.
Details
When developing, sometimes one forgets the minDynatraceVersion has changed and will perform changes in the extension YAML that are validated, and then notice that they are actually not valid at the moment of upload, possibly causing you to have to re-factor stuff. Having the schema be loaded automatically when the version changes could potentially make development faster and easier.
Use Cases
When developing extensions, not having to care about what schema the VSCode add-on is using, as it will always use the one I have as the minDynatraceVersion would make developing much easier.
Challenges
This should be a setting in case someone wants to work as it always have, by running the Load schema function. Maybe loading it very often could have some issues, maybe performance, but I wouldn't think so.

The recommendation is to not change minDynatraceVersion manually yourself, but always through the Load schemas command.

Nevertheless, should someone have the time to implement it, it would have to be done in the onChange listener that's already hooked up to the extension which parses the extension.yaml. Before we communicate the new parsedExtension object, we have the opportunity to check if the minDynatraceVersion has changed and if so trigger async the loadSchemas function with that version.


Python 3.7 pip install has an error

I am having a problem with installing the pysal on windows with Python 3.7. When I use the pip install:
pip install pysal

Everything is OK until the end, the command line threw an error:
INFO:root: Building on Windows requires extra options to setup.py to locate needed GDAL files. More information is available in the README.
  ERROR: A GDAL API version must be specified. Provide a path to gdal-config using a GDAL_CONFIG environment variable or use a GDAL_VERSION environment variable.
  ----------------------------------------
ERROR: Command errored out with exit status 1: 'd:\python37\python.exe' 'd:\python37\lib\site-packages\pip\_vendor\pep517\_in_process.py' get_requires_for_build_wheel 'C:\Users\BRIGHT~1\AppData\Local\Temp\tmpmny6kf2u' Check the logs for full command output.

It seems like it asks me to add the gdal to the PATH. But after adding that the error still occurs...Any idea? Thank you very much!

Can you try installing with conda and use the conda-forge channel? This seems to handle the python geospatial stack much more robustly than pip.

Can you try installing with conda and use the conda-forge channel? This seems to handle the python geospatial stack much more robustly than pip.

@sjsrey Thank you for your reply! Yes, I have successfully installed pysal using conda. I would close this issue.


embedded power bi results in microsoft login box

From @toy-opensource on March 27, 2018 0:19
there is an excellent example for how to embed powerbi App Owns Data here:
https://powerbi.microsoft.com/en-us/documentation/powerbi-developer-embed-sample-app-owns-data/
however this example is for running on the .net platform
and some of the routines used in the example dont exist for aspnetcore
there is another excellent post about someone attempting to do the same thing:
https://community.powerbi.com/t5/Developer/Embed-Power-BI-dashboard-in-ASP-Net-core/td-p/273279
so first i built the standard .net version according to the first example
and then i tried to migrate to aspnetcore incorporating fixes from the second link
here are the urls im using:
private static readonly string AuthorityUrl = "https://login.windows.net/common/oauth2/authorize/";
private static readonly string ResourceUrl = "https://analysis.windows.net/powerbi/api";
private static readonly string ApiUrl = "https://api.powerbi.com/";
private static readonly string EmbedUrl = "https://app.powerbi.com/";
and the result is -
the microsoft login box - not cool
can anyone tell me if they have actually embedded powerbi into aspnetcore?
if you have - can you give me a coupla hints on how to get real data back?
Copied from original issue: aspnet/Mvc#7555

Hi @toy-opensource
It looks like this is a question about how to use ASP.NET Core with Power BI. While we do our best to look through all the issues filed here, to get a faster response we suggest posting your questions to StackOverflow using the asp.net-core tag and the powerbi tag.

We periodically close 'discussion' issues that have not been updated in a long period of time.
We apologize if this causes any inconvenience. We ask that if you are still encountering an issue, please log a new issue with updated information and we will investigate.


add a cli command to generate a DBML dump

This has some pretty immediate "byproduct" wins:

We can map it: https://dbdiagram.io/d/645a51c8dca9fb07c4c56c19
We can generate docs: https://dbdocs.io/ewdurbin/PyPI

My usecase is to automatically generate and compare a fides dataset using the output though...

Example of how the result could be used to generate/merge-into an existing fides dataset: https://gist.github.com/ewdurbin/05914fee6eb4e53d4357fb3b27f76b36#file-0-dbml_to_fides_dataset-py


Unable to connect to a Hadoop cluster with Kerberos security configured.

Hi,
I am using the HBase UI tool 'H-Rider'. I was able to use it with a cluster which does not have Kerberos security configured but was getting 'failed to connect' errors when trying to connect to a cluster with Kerberos authentication. Is it possible to connect H-Rider with such a cluster? I couldn't find anything where I could set this settings or properties file. I am able to connect to this secured cluster through a Java HBase client so Kerberos on my machine is setup properly, I believe.
Thanks & Regards,
Shahab

Hi,
Instead of hardcoding System properties, you can also set it at launchtime
java  -Djava.security.krb5.conf=/etc/krb5.conf  -jar target/h-rider-1.0.9-SNAPSHOT.jar
and add core-site.xml,hdfs-site.xml and hbase-site.xml in the classpath.
Works great for me.

Thanks.
Feel free to prepare a merge request, and we will add it!
On Nov 10, 2017 15:22, "Damien Claveau"<EMAIL_ADDRESS>wrote:
Hi,
Instead of hardcoding System properties, you can also set it at launchtime
java -Djava.security.krb5.conf=/etc/krb5.conf -jar
target/h-rider-1.0.9-SNAPSHOT.jar
and add core-site.xml,hdfs-site.xml and hbase-site.xml in the classpath.
Works great for me.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/NiceSystems/hrider/issues/42#issuecomment-343471815,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACSLZDeYgtfI6Nnvq-YdN-eQ40hnWyaeks5s1E4AgaJpZM4AtAqY
.


feat: add custom marshaler and fix go mods for examples

closes #6

I will be working on #8 before merging this as well.


Change the way containerd uses custom configuration

How to categorize this issue?

/area quality
/kind enhancement
What would you like to be added:
Remove the import of conf.d folder that is added here
head -n 1 /etc/containerd/config.toml
# Output:
# imports = ["/etc/containerd/conf.d/*.toml"]

This causes overwrite of configuration added in /etc/containerd/config.toml like sandbox_image
grep -F -A 2 '[plugins."io.containerd.grpc.v1.cri"]' /etc/containerd/config.toml
# Output:
# [plugins."io.containerd.grpc.v1.cri"]
#   # use fixed sandbox image
#   sandbox_image = "registry.k8s.io/pause:3.7"
containerd config dump | grep sandbox_image
# Output:
# sandbox_image = "registry.k8s.io/pause:3.7"

# Now adding foo.toml

cat /etc/containerd/conf.d/foo.toml
# Output:
# [plugins."io.containerd.grpc.v1.cri".containerd]
#   discard_unpacked_layers = false
containerd config dump | grep sandbox_image
# Output:
# sandbox_image = "registry.k8s.io/pause:3.6"

Why is this needed:
Same issue has been hit here
The registry-cache extension that is under developement needs to change containerd configuration. If someone uses the import functionality and has enabled registry-cache for his shoots he will overwrite the registry-cache configuration and this will make the extension useless in those cases
This PR proposes the removal of the import functionality and proposes changes of the documentation for how to create custom containerd configuration
It seems in 2.0 of containerd merging will happen rather than overriding but if we extrapolate the time between minor releases from https://containerd.io/releases/ it seems 2.0 will come at least after a year

/cc @rfranzke @timebertt @danielfoehrKn

There is a new configuration option in containerd 1.5, which looks like a good replacement of the current containerd config imports and would simply the registry cache extension:
https://github.com/containerd/containerd/blob/main/docs/hosts.md
Also see https://kind.sigs.k8s.io/docs/user/local-registry/ for an example usage in kind.
I would try to rather use this configuration option instead of suggesting users and extensions to patch the main configuration file.

We are already aware of the new way of configuring containerd registries. See the updated details in https://github.com/gardener/gardener-extension-registry-cache/issues/3.

Great. I would vote for keeping the conf.d import nevertheless and simply use the new host config dir as an additional mechanism.
I.e., I would vote for closing https://github.com/gardener/gardener/pull/7837


GVCF output

Hello,
Would be possible to add a GVCF output to Clair? GLnexus could be used on the output GVCFs to make a joint VCF afterwards.
Thank you for your help.
Best regards,
Guillaume

GVCF output added in Clair2.

Great, thanks :) Will try it asap.

Clair2 will be available as soon as early 2021. Thank you for your suggestion!

Oh my bad, I thought that Clair was the v2 of ClairVoyante. Thank you for your work!

Hello,
I am trying to process PacBioCLR data with this version of Clair and hope to get GVCF. As mentioned above, GVCF output was added in Clair2, but I did not find the corresponding parameter in the help document. May I ask what parameters should I use.
Thanks

Clair doesn't support GVCF output, please use https://github.com/HKU-BAL/Clair3


Fix multiple building type tags

This happened in the initial days of importing when we don't have the combine LA building plugin in JOSM.

[ ] Extract all building with multiple tags with overpass.
[ ] Fix tags.

Issue
Query

building with ; in the value
http://overpass-turbo.eu/s/hqg

cc @maning

Thanks @ramyaragupathy lets keep this ticket open and do another pass after the import is done.

This is done.  All imported buildings have only one building value.
Thanks @ramyaragupathy !
Closing.


Warn me when I break AoT

Hello,
I would love if Codelyzer could warn me when I do something in my code that will break AoT.
The current error messages when you try to add AoT to an existing app are really cryptic and not useful at all.
Do you think that Codelyzer could check for that? I think that it would make this lib so useful that it would be included in all angular apps :-)
Let's start a list of the things that break AoT to see what could be checked (correct me if I'm wrong):

[ ] private / protected properties used in templates
[ ] using external values insides string templates (with ${...}) instead of class properties
[ ] using require() (instead of import) to load angular code that uses decorators, or to load template/styles
[ ] not exporting interfaces / types (even if you don't use them outside of the current file)
[ ] using "export default"
[ ] using form.controls.controlName instead of form.get('controlName')
[ ] using control.errors?.someError instead of control.hasError(‘someError’)
[ ] using functions in decorators (especially routes / ng modules) instead of using the reference to an exported function
[ ] using external angular libs that don't include *.metadata.json files

Do you know of any other things that break AoT? Do you think you could check for some (of most) of these?

See also this documentation gist by @chuckjaz https://gist.github.com/chuckjaz/65dcc2fd5f4f5463e492ed0cb93bca60

Interesting, thanks @teropa ! This explains a lot of the errors that I listed and should be helpful to @mgechev if he decides to do that :)

Nice list! We have partial support at the moment by warning when you try to bind to missing or non-public properties or methods.
I'm not sure to what extent it's worth implementing the above rules in case soon in development we switch to ngc from tsc and get all the errors from there. I will sync with Alex Eagle on that.

I think it'll be better to wait until ngc becomes the default in Angular CLI and we get the compiler errors during the incremental compilation.
This should happen at some point (hopefully soon). However, if anyone is willing to help with the listed rules I'll be happy to assist. Otherwise, there are a few more critical issues to take care about such as missing context for the template parser and incorrect symbol references.

dynamic-component-loader will not work but it will not throw any error in compile time.


[Fan sharing] Colorful background and float right styles

Hi Zhou Hua,
This is not an issue, but just that I would like to share a couple of Obsidian Sticky Headings tweaks.

Thank you for creating this plugin!

Thanks for sharing! I just added a section in the project's README to collect some interesting styles.


Can't login

I followed the quickstart for laravel, tried to implement it in 5.3.
When trying to login I always get the "These credentials do not match our records." message.
When trying to do an import on all the users I get this errorException
ldap_search(): Search: Operations error
Please help me :)

Found out the user I used didn't have search permissions.. Problem solved!

Glad you've solved your issue! Thanks for following up.


[optimize][txn] Optimize transaction lowWaterMark to clean useless data faster

Motivation
The current implementation mechanism of low water mark is to make a judgment after the end transaction, and only for the first transaction in ongoingTxns.
This makes useless data cleaning very slow.
Modification
Store lowWaterMark for tc in TB and TP.
Recursively calls lowWaterMark and cleans the data according to the stored lowWaterMarks
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation
Check the box below or label this PR directly.
Need to update docs?

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] no-need-doc
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-added
(Docs have been already added)

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks


Wrapping page-router-outlet with a layout causes crash when navigating to a page with tabs (iOS only)

Environment
Provide version numbers for the following components (information can be retrieved by running tns info in your project folder or by inspecting the package.json of the project):

CLI: 7.0.4
Cross-platform modules: 7.0.0
Android Runtime: 7.0.0
iOS Runtime: 7.0.0
Plugin(s): 0
NativeScript-Angular: 10.1.0
Angular: 10.1.0

Describe the bug
Tabs causes app to crash on navigating to it when the page-router-outlet is wrapped within a layout (GridLayout). This happens on both NS6 and NS7. When page-router-outlet is wrapped with a layout, it doesn't crash only when the page that contains the Tabs is the first page being navigated to by the app - subsequent navigation to pages containing Tabs causes the same crash.
The errors are different with different configuration of the Tabs (i.e. with the Tabstrip vs omitting the Tabstrip). Removing the layout that wraps page-router-outlet seems to fix both errors (in < NS7)
// WITHOUT TABSTRIP
***** Fatal JavaScript exception - application has been terminated. *****
NativeScript encountered a fatal error: Uncaught TypeError: Cannot set property 'hidden' of null
at
push.../node_modules/@nativescript/core/ui/tabs/index.js.UIPageViewControllerImpl.viewDidLayoutSubviews(file: node_modules/@nativescript/core/ui/tabs/index.ios.js:167:0)

// WITH TABSTRIP
***** Fatal JavaScript exception - application has been terminated. *****
NativeScript encountered a fatal error: Uncaught TypeError: Cannot read property 'setTitleColorForState' of null
at
setTabBarColor(file: node_modules/@nativescript/core/ui/tabs/index.ios.js:856:0)
at [color:setNative](file: node_modules/@nativescript/core/ui/tab-navigation-base/tab-strip/index.js:82:0)
at applyPendingNativeSetters(file: node_modules/@nativescript/core/ui/core/properties/index.js:1059:0)
at initNativeView(file: node_modules/@nativescript/core/ui/core/properties/index.js:1018:0)
at onResumeNativeUpdates(file: node_modules/@nativescript/core/ui/core/view-base/index.js:695:22)
at _resumeNativeUpdates(file: node_modules/@nativescript/core/ui/core/view-base/index.js:259:0)
at onLoaded(file: node_modules/@nativescript/core/ui/core/view-base/index.js:214:0)
at (file: node_modules/@nativescript/core/ui/core/view-base/index.js:297:0)
at callFunctionWithSuper(file: node_modules/@nativescript/core/ui/core/view-base/index.js:291:0)
at callLoaded(file: node_modules/@nativescript/core/ui/core/view-base/index.js:297:0)
at loadView(file: node_modules/@nativescript/core/ui/core/view-base/index.js:437:0)
at (file: node_modules/@nativescript/core/ui/core/view-base/index.js:216:0)
at eachChild(file: node_modules/@nativescript/core/ui/tab-navigation-base/tab-navigation-base/index.js:42:0)
at onLoaded(file: node_modules/@nativescript/core/ui/core/view-base/index.js:215:0)
at onLoaded(file: node_modules/@nativescript/core/ui/tabs/index.ios.js:412:0)
at (file: node_modules/@nativescript/core/ui/core/view-base/index.js:297:0)
at callFunctionWithSuper(file: node_modules/@nativescript/core/ui/core/view-base/index.js:291:0)
at callLoaded(file: node_modules/@nativescript/core/ui/core/view-base/index.js:297:0)
at l<…>

To Reproduce
The sample project (linked below) currently has the page-router-outlet wrapped in a GridLayout (app.component.html), notice that on tapping the button that navigates to the page containing the Tabs will cause a crash. It no longer crashes if the GridLayout is removed (in < NS7).
Expected behavior
App doesn't crash when navigating to a page that contains Tabs even when page-router-outlet is wrapped by a layout (which when not wrapped in a layout in NS7 results in an com.tns.NativeScriptException: Calling js method onCreate failed error as described here)
Sample project
https://github.com/williamjuan027/issues-repro/tree/master/tabs-router-layout
$ npm i && ns run ios --no-hmr

Hey, is there any updates on this? Im experiencing the same issue.
Environment.
"dependencies": {
    "@angular/animations": "~11.2.7",
    "@angular/common": "~11.2.7",
    "@angular/compiler": "~11.2.7",
    "@angular/core": "~11.2.7",
    "@angular/forms": "~11.2.7",
    "@angular/platform-browser": "~11.2.7",
    "@angular/platform-browser-dynamic": "~11.2.7",
    "@angular/router": "~11.2.7",
    "@nativescript/angular": "~11.2.0",
    "@nativescript/appversion": "~2.0.0",
    "@nativescript/core": "~7.3.0",
    "@nativescript/email": "~2.0.0",
    "@nativescript/fingerprint-auth": "~7.0.0",
    "@nativescript/firebase": "^11.1.3",
    "@nativescript/secure-storage": "~3.0.0",
    "@nstudio/nativescript-https": "^3.0.1",
    "@nstudio/nativescript-pulltorefresh": "3.0.1",
    "@types/node": "^12.12.12",
    "html-encoder-decoder": "^1.3.8",
    "jwt-decode": "^2.2.0",
    "nativescript-awesome-webview-with-custom-menu-items": "^70.1.1",
    "nativescript-carousel": "^7.0.1",
    "nativescript-lottie": "^5.0.3",
    "nativescript-ngx-fonticon": "^7.0.0",
    "nativescript-phone": "^2.0.0",
    "nativescript-rater": "^2.1.2",
    "nativescript-root-detection": "^1.0.0",
    "nativescript-taptic-engine": "^2.1.0",
    "nativescript-ui-gauge": "^7.0.2",
    "nativescript-ui-listview": "^9.1.0",
    "reflect-metadata": "~0.1.13",
    "rxjs": "^6.6.0",
    "rxjs-compat": "^6.5.3",
    "zone.js": "^0.9.1"
  },
  "devDependencies": {
    "@angular/compiler-cli": "~11.2.7",
    "@nativescript/android": "7.0.1",
    "@nativescript/ios": "7.2.0",
    "@nativescript/types": "~7.3.0",
    "@nativescript/webpack": "~4.1.0",
    "@ngtools/webpack": "~11.2.6",
    "@typescript-eslint/eslint-plugin": "^4.6.0",
    "@typescript-eslint/parser": "^4.5.0",
    "codelyzer": "~5.2.0",
    "eslint": "^7.11.0",
    "eslint-config-airbnb-typescript": "^12.0.0",
    "eslint-config-prettier": "^6.13.0",
    "eslint-plugin-import": "^2.22.1",
    "eslint-plugin-nativescript": "0.0.0",
    "eslint-plugin-prettier": "^3.1.4",
    "husky": "^4.3.7",
    "lint-staged": "^10.5.3",
    "node-sass": "^4.13.1",
    "prettier": "^2.1.2",
    "sonarqube-scanner": "^2.8.0",
    "typescript": "~4.0.0",
    "webpack-obfuscator": "^0.27.2"
  },

***** Fatal JavaScript exception - application has been terminated. *****
NativeScript encountered a fatal error: Uncaught TypeError: Cannot set property 'hidden' of null
at
push.../node_modules/@nativescript/core/ui/tabs/index.js.UIPageViewControllerImpl.viewDidLayoutSubviews(file: node_modules/@nativescript/core/ui/tabs/index.ios.js:168:0)


Error installing ember-page-title on ember-cli 2.8.0

root@1b981a35323d:/ember/ember-quickstart# ember install ember-page-title
Installed packages for tooling via npm.
installing ember-page-title
The ember generate <entity-name> command requires an entity name to be specified. For more details, use ember help.
ember -v output:
ember-cli: 2.8.0
node: 6.5.0
os: linux x64
npm version 3.10.3

I'm getting this error too, but my ember version is 2.4. I guess this is not a version related bug.
ember -v output:
ember-cli: 2.4.3
node: 4.4.1
os: darwin x64

Gotcha! Will try digging into this.

On Sep 15, 2016, at 12:35 PM, Rafael Santos<EMAIL_ADDRESS>wrote:
Hi @tim-evans,
I'm getting the same error as @gorbypark, but my ember version is 2.4. I guess this is not a version related bug.
ember -v output:
ember-cli: 2.4.3
node: 4.4.1
os: darwin x64
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I wonder if this is the harmless error that was a bug in ember-cli caused when there is no add-on blueprint (see ember-cli#3846). It should have been fixed by now, so I'm not sure why we're still seeing this. There was a workaround of implementing an empty normalizeEntityName, which might work again.

Ahh, yeah. That may be worth a try

I'm on 2.8.1 and getting the same error. First time using this addon, so I am unfamiliar with how it works. I'm assuming a generator is supposed to run in install? Because the addon isn't updating the title, which I assume is related to this error.

Fixed the blueprint. It seems like that ember-cli bug is back :(


[GJ-13] Refine docs on compiling and testing

This pr refined the docs on compiling and testing.
https://github.com/oap-project/gazelle-jni/issues/13

LGTM.


[vscode] Connect openInTextEditor Callback

[vscode] Connect openInTextEditor Callback
Connect the button & event added in https://github.com/lastmile-ai/aiconfig/pull/1142/files with the callback to reopen the aiconfig editor webview as a text editor to interact with the underlying file directly.
https://github.com/lastmile-ai/aiconfig/assets/5060851/76a25aa7-566f-491d-9949-b0f2f9ec3b1a
Note, this requires closing and reopening the file in order to re-initialize the aiconfig editor webview for the file. We could potentially add another command to handle this scenario (something like, "AIConfig: Open in Editor" which defaults to current file but shows others in current directory?)

@saqadri thoughts on the caveat?

Note, this requires closing and reopening the file in order to re-initialize the aiconfig editor webview for the file. We could potentially add another command to handle this scenario (something like, "AIConfig: Open in Editor" which defaults to current file but shows others in current directory?)

Not sure that it's a blocker for shipping this, or if it can be added as an Issue for later

@rholinshead to clarify you mean in order to reopen the file as an aiconfig editor webview after having opened it as text, you need to close and reopen the editor, right? IMO this is ok. I like your idea of adding a command but not P0 in my opinion

@rholinshead to clarify you mean in order to reopen the file as an aiconfig editor webview after having opened it as text, you need to close and reopen the editor, right? IMO this is ok. I like your idea of adding a command but not P0 in my opinion

Yes, correct - need to close and reopen. Sounds good, I'll ship this

Actually, come to think of it, that command could be pretty useful in other situations too -- for example one thing people struggled with creating a config was naming the file to match the expected extension *.aiconfig.json/yaml
Having a command could allow people to open arbitrary json/yaml files as aiconfig editors


divide by zero error in evaluation_DRBox.py with high confidence thresholds

While decoding using the evaluation_DRBox.py, it uses decode_detections function from the drbox_output_decoder.py. This is done for confidence threshold conf from 0.1 to 0.9. However, when say the conf is as high as 0.9, no boxes might cross the threshold in which case decode_detections return an empty array, and true_positive as well as false_positive remains 0. This in turn (obviously) return divide by zero error in precision.append(true_positive / (true_positive + false_positive)) since true_positive + false_positive = 0.
How do we handle such cases?
Any help will be really appreciated! Thank you

@ppontisso Any clue here? WIll really appreciate any help. TIA

Hello,
I think one way to handle it could be to add a very small number to the denominator of the division.
You could do : true_positive / (true_positive + false_positive + 1e-10), and if true_positive + false_positive = 0 then the result will be 0.
This will avoid the zero division error, while not changing the results of the other threshold.
Paul

@ppontisso That makes sense, good little trick! Thank you :)


startup/bootsnap: raise repeated LoadError.

If HOMEBREW_BOOTSNAP_RETRY was set then we should raise the LoadError the second time so that the failures are user visible for debugging.

Thanks @carlocab!