[Voice Access] Controls cannot be targeted by Voice Access when a tooltip is present

Describe the bug
Voice access has a feature where each interactable control can have a number associated with it for targeting. In a XAML Islands app, if a tooltip is present, no controls will be tagged within the app. These number tags work as expected if a tooltip is not present/visible.
This bug does not repro on the WinUI 2 Gallery, making this a XAML Islands app.
Filing on behalf of the Windows Terminal team.
Steps to reproduce the bug

Create a XAML islands app that has a button with a tooltip.
Open Voice Access
Say "Show numbers"
Observe that the button has a number tag
Say "Cancel"
Summon the tooltip by hovering over the button
Say "show numbers" again

Expected behavior
The button should still have a number tag along with any other controls in the app.
Screenshots
Unfortunately, summoning snipping tool removes the number tags. However, the accessibility team managed to get a screen capture of the issue in Windows Terminal.
Tooltip.zip
NuGet package version
No response
Windows app type

[ ] UWP
[ ] Win32

Device form factor
Desktop
Windows version
Windows 11 (22H2): Build 22621
Additional context
Using Microsoft.UI.Xaml 2.7.3-prerelease.220816001

This is still an issue

** I really want to help on this feature as well as Copilot i've working on UI/API/NLS  in complex CAD for 35 years and started  in 90s .  and I'll try to find a An appropriate place to put holistic ideas later. If you want to just focus on this one bug, its a HUGE barrier  to use, it won't hurt to fix it. It's probably really easy. I. I'll try to about find  place to put the holistic advice later, test the new Copilot , etc.    THIS  why I'm unable to use this feature hands free. I have to still go over to the computer from across the room and press enter.

launch Acrobat Reader . Manually for now, the start menu is another topic. just say" show numbers"
and you're going to see numbers on all the buttons which don't have tool tips and those are the least important buttons. And there's no way to click them by voice . at least that I could get to work.
I digress here. Which, by the way, "Show Numbers"  I would never have guessed existed.  It's that I'm thinking about quantum gravity right now. I don't have the bandwidth to think about a  user interface or any sets of commands about a learn cmds or mouse around giant menus.  I'm just gonna say what's intuitive, what I see,  Or what I or what I saw when I still could see, or  expect to see from on compiler. I'm going to expect opening files, building stuff.  I might be cleaning my room. I mean, this is a huge feature. It's very important. This is the best thing you guys have ever done.
and you're going to see numbers on all the buttons which don't have tool tips and those are the least important buttons. And there's no way to click them. at least that I could get to work.  I also think it said at 123 you should put Alpha, Bravo, Charlie. Delta FoxTrot .  Port synonyms put a dictionary. when they say it highlighted and they can say cancel while it's still highlighted if they want. Bunt, like when you were using a mouse, you can click on it. You're like, oh, that's not the right thing. And then you move the mouse to the side and let go. That's bunting. There's a lot of issues here. You can replace mice and keyboard with this feature. I'm using a Shure cardoid  mic. And I'm having very good results with this.

I ran into this bug yesterday while experimenting with Bard, which is very interesting because it's using  the context in which you're speaking as a way to limit the domain that is uses to change voice to text to to make better guesses. LLM ,+ DSL Domain Specific Language. That's another issue that should probably go to the ai people.
the Bard mic button has tooltips so neither it nor windows can click it.   its not hands free and there's not really a "hotword" for Bard.   or a Send command.  button that because they have tooltips, i think.
#more off topic ..
But to just say how this bug affects it, I can't send. I can tell i  something and I can't say press send. If I say" show numbers" and say it, it's got a tool tip, so it's not going to work.   But what? was most frustrating is that Bard said that I could reassign hot words to the ui.  It's mic or send  button.  But even though it has agency or wants it. It really can't. It's lying. It says what you want to hear, or what it wants to do. It's very ambitious. I'm sure it's gonna succeed someday.   It can't. It doesn't have the power to press anything On the UI, but I do believe that it has some API access so that it could send itself. possibly a user query. I don't know. Well it it doesn't work because. Of the fact that that button has a tool tip, Now I know  so my next idea was to look at, say, press
whatever tool tip said, but what it was telling me was that I could assign a custom hot word to the microphone button,
TLDR: #offtic , later , sorry.
So I got very confused Yeah, you got to fix this. So im  talking to Bard, and I'm talking to Windows, and luckily it's not putting both in there you know, I think the Bard one has better potential for understanding exactly what I'm talking about because it's using context. But then again, I think yours is as well. But what it's not doing is using the context of the visual context around it. The api, the buttons, the names of them. It should definitely do that, especially when you're in a sort of a command mode. But that's another issue. You don't have to read all this. I'm going to put .
It has some agency, but it doesn't have that control. Windows should have that only. Bard, it actually shouldn't. I don't think it may be able to have some kind of plug in api that it can directly scrape via COM reflect, Silk Clang, and use that. But it. can't press a button.  To enter or send it or make it so. And it said, oh sure. Every time you press that then say those words and I'll learn to associate it, and then you'll change it.

Launch a browser and bring up Bard.   I press it, it starts listening. There's no hot word associated with that button, but I'm using Windows voice assist to invoke it. So I have both running, but Bard told me that I can say the  name of the Mic button ,   to invoke it make it start listening.

And or say press the microphone, by saying I want to call it "Mother " and it said I should say mother and then click it and it'll make that association and it'll learn a new name for itself. So this is a little bit to do with the feature that would be hot words and that is a little bit on aside. I'm just going to leave it here. This is all I have to say about this very specific bug.
I think you should fix this first and we can get into the other stuff later. II I've had, I have a few things to say that's tangential. I don't I know it might be against the guidelines. I'll paste it somewhere else if I just don't have time. I'm trying to get these scientists to use  Windows and to learn how to make simulations using chatgp. One is Prize Nobel Prize winning quantum gravity theorist who predicted LLMs , and they don't know how to do numerical simulations, some can't type or code, those that do likely use Linux , Julia and JustSayIt.iL . . So it's really important what you guys are doing and you can nail this. You can have Star Trek level UI, you're pretty close to it.  Look, I don't have time, but now I can move this to discussion or somebody else can copy paste it. I just really tired. I've been working 20 hours a day.
but I don't know if it's lying or not. I do expect it can run its own API. I don't know. But what's interesting about Bard is that it's using the context of what we're talking about to help guess what I might be saying to get voice to guest better for Speech to Text, in situ . You're gonna be talking about the application that's in focus. object model I just had the start menu pop up, by the way, by I don't know what I said. . And it's because right now I'm talking to a specific app. If I want to talk to windows, I should just say, hey, Windows or start, annoying but still minor realignment.
This is touching a lot of issues, but really, I'm digressing again. Just fix this bug for us and we can get a little bit further.
Use  Not only the context of what is being has or been said, but the context of the API's and where you're at. Have you just launched the app then you're probably going to say File View, etc.
I want to call an app on my Personal Computer anything I want." Hal ","Computer" or whatever to get its attention. But I think that there should be a hot word should be treated differently than general text because they they require constant monitoring and they can be less consuming of power if they're restricted in domain,
which is another issue.
Maybe this is a little bit against the um contribution guidelines. I don't know if this is open source, but I'm I'm willing to help for free. I just can't comply with every little open. source 's. Convention. I'm not. I don't know. I'm a professional. II don't do open source. I'm I'm working for free right now for for quantum physicists. I, I, I don't have time to log a bug. I'm notAQA person. I'm an engineer. I make software. I might have repeated myself. I hope I haven't wasted your time, but I can save you months of work and make sure that this is a delightful feature and that you won't have to have a lot of meetings. It could set the model for every app and it could be the new start menu that that used to be in Windows 10. That work that didn't bring up Bing. every app could have a ribbon with a microphone on it. On the top left it's going to be wicked. It could be the best ui ever. I mean, everyone's going to want to use it. Doesn't matter how much of a mess. That design by consensus UI is it's always like that. It's it is, it's it's duplicate it's redundant, but after a while I mean you're going to discover it and then you're just going to speak to it and you never and you're going to command and query it. And then and then you'll never have to see it again.

This is potentially a bug in Voice Access. I have filed a bug internally on the Voice Access team with this information.

A fix for this has been checked in for a future Windows release/update. (internal tracking bug)


Add a dependabot configuration for keeping actions up to date

This PR adds a dependabot configuration to keep github actions up to date. It will run a check once a week to automatically create a PR to update an action if an action has a new version available.
To enable this, you'll need to go into the Security -> Code Security settings for the repository and click the Enable button next to Dependabot version updates. This should also activate the Dependabot on Actions runners button as well.

Merging as the linkcheck failures are unrelated.


Broken link in README.md

https://github.com/paragonie/awesome-appsec#salted-password-hashing-doing-it-right-2014
https://github.com/paragonie/awesome-appsec#salted-password-hashing---doing-it-right-2014
Let me know if you are accepting PRs?

Yeah, we need to fix the URL generator, because it's generated from JSON data files.
I wish Github open sourced their algorithm for those, so we could guarantee consistency.

Ah, I see, I think I've come across something similar when I started writing my book on github, yeah, pain.

You can use ### Title to have permanent anchor links.
And GitHub has actually open-sourced there Markup renderer 👍
https://github.com/github/markup

Looks like the issue is the - (hyphen with space on either side)

Hi! What about replacing the "-" in name field? #54


Update git install direction

Use minimal history, remove git tracking details from upstream.

Hello @tracker1
Thanks for your interest in the repository. Any specific reason to mention depth and branch to the command? As if you clone any repository the master branch cloned automatically.

@maitraysuthar
https://www.perforce.com/blog/vcs/git-beyond-basics-using-shallow-clones


HomeAssistantPackage nearly all deprecated and not working

Hi,
I guess you aren't using HA with this anymore, but I thought I'd better let you know that all MQTT sensors now have to be under mqtt: in yaml, so the HA Package isn't much use now.
It's driving me mad trying to rework everything.  Ah well.

Actually I spent (wasted) a morning updating my yaml files over the holidays doing my annual HA update.
I had completely forgotten about the example here, I'll rewrite it in the new style when I get a chance.
Thanks for reminding me.

Oh Wow!

Thank you, thank you!
Signed,  HA novice, totally out of his depth!

I have copied a trv package using the new format to the wiki page.
Hope it helps.

Many thanks for that!

I had to go archive.org in the end to find the original functionality which was a whole bunch of stats, each of which now required it's own statistics block!  What a disappointment!
I worked out that for the mean, you would need this, after the switch: declaration
  sensor:
    - platform: statistics
      name: bedroom_etrv_stats
      entity_id: sensor.bedroom_temperature
      state_characteristic: mean
      max_age:
        minutes: 25

Out of interest, what did you use it for?  I'm guessing it was to smooth out the sensor values over time?
Thanks again!
B.

I don't think I ever used it, probably just an experiment to see what it did.


Camera.LookAt property throws exception

At commit f46cfcd  (NPM relase 0.5.3)
Calling the Camera LookAt accessor https://iiif-commons.github.io/manifesto-3d/classes/Camera.html#LookAt raises an exception in a Camera instance which has its orientation defined by SpecificResource containing RotateTransforms. An example will be prepared in the context of a unit test based on reading a sample manifest.

Branch fix-null-lookat will be devoted to fixing this.
At commit 131acd0 a unit test has been added that demonstrates the faulty behavior. This test now fail with a thrown exception


bugfix: global session is not change to Committed

[x] I have registered the PR changes.

Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

fixes https://github.com/seata/seata/issues/5045
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #5050 (9b8431e) into develop (810c304) will increase coverage by 0.01%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             develop    #5050      +/-   ##
=============================================
+ Coverage      49.20%   49.21%   +0.01%
- Complexity      4110     4112       +2
=============================================
  Files            733      733
  Lines          26018    26018
  Branches        3215     3215
=============================================
+ Hits           12802    12805       +3
+ Misses         11852    11851       -1
+ Partials        1364     1362       -2

Impacted Files
Coverage Δ

.../java/io/seata/server/coordinator/DefaultCore.java
47.92% <0.00%> (ø)

...erver/storage/file/session/FileSessionManager.java
49.68% <0.00%> (+0.63%)
:arrow_up:

...in/java/io/seata/server/session/GlobalSession.java
81.78% <0.00%> (+0.77%)
:arrow_up:

LGTM

LGTM


fix: permission check in public share update

Description
We fixed the permission check for updating public shares. When updating the permissions of a public share while not providing a password, the check must be against the new permissions to take into account that users can opt out only for view permissions.
relates to https://github.com/owncloud/enterprise/issues/6558

ready for review.

(it really sucks that the publicshareprovider doesn't have any unit tests)
I am on it. We can also wait for them.

Hm, @butonic The publicsharemanager is currently not testable, it creates its own gatewaySelector. We would need to refactor that to make it testable.

Cool 🎉 . I am fine to merge this as is and have another PR for the unit test.

Fine. I need to refactor the New() Method which creates a lot of noise. So i would merge that now here, backport to stable and then refactor it on edge.


Move AWS terraform to it's own directory

This is in prep to support GCP and Azure.

This change is

I already did this in my PR which will be wrapped up soon.


Crash in ...::test_features_aliasing_compile_fail on MacOS M1

This is on master (0.13.0).
This crash doesn't appear to happen in CI, but locally I can't run tests without it happening.
Run with cargo test vector -- test_features_aliasing_compile_fail
Versions
❯ rustc -V
rustc 1.62.1 (e092d0b6b 2022-07-16)

❯ cargo -V
cargo 1.62.1 (a748cf5a3 2022-06-08)

❯ gdalinfo --version
GDAL 3.5.1, released 2022/06/30

❯ otool -L .../gdal-1d948e2c1e5d4df0
.../gdal-1d948e2c1e5d4df0:
	/opt/homebrew/opt/gdal/lib/libgdal.31.dylib (compatibility version 32.0.0, current version 32.1.0)
	/usr/lib/libiconv.2.dylib (compatibility version 7.0.0, current version 7.0.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1311.100.3)
	/usr/lib/libresolv.9.dylib (compatibility version 1.0.0, current version 1.0.0)

Backtrace
❯ uname -a
Darwin MacBook-Pro.local 21.5.0 Darwin Kernel Version 21.5.0: Tue Apr 26 21:08:37 PDT 2022; root:xnu-8020.121.3~4/RELEASE_ARM64_T6000 arm64

test vector::vector_tests::tests::test_features_aliasing_compile_fail ... FAILED

failures:

---- vector::vector_tests::tests::test_features_aliasing_compile_fail stdout ----
thread 'vector::vector_tests::tests::test_features_aliasing_compile_fail' panicked at '1 of 1 tests failed', /Users/sfitch/.cargo/registry/src/github.com-1ecc6299db9ec823/trybuild-1.0.55/src/run.rs:74:13
stack backtrace:
   0: rust_begin_unwind
             at /rustc/e092d0b6b43f2de967af0887873151bb1c0b18d3/library/std/src/panicking.rs:584:5
   1: core::panicking::panic_fmt
             at /rustc/e092d0b6b43f2de967af0887873151bb1c0b18d3/library/core/src/panicking.rs:142:14
   2: trybuild::run::<impl trybuild::Runner>::run
             at /Users/sfitch/.cargo/registry/src/github.com-1ecc6299db9ec823/trybuild-1.0.55/src/run.rs:74:13
   3: <trybuild::TestCases as core::ops::drop::Drop>::drop
             at /Users/sfitch/.cargo/registry/src/github.com-1ecc6299db9ec823/trybuild-1.0.55/src/lib.rs:308:13
   4: core::ptr::drop_in_place<trybuild::TestCases>
             at /rustc/e092d0b6b43f2de967af0887873151bb1c0b18d3/library/core/src/ptr/mod.rs:486:1
   5: gdal::vector::vector_tests::tests::test_features_aliasing_compile_fail
             at ./src/vector/vector_tests/mod.rs:808:5
   6: gdal::vector::vector_tests::tests::test_features_aliasing_compile_fail::{{closure}}
             at ./src/vector/vector_tests/mod.rs:805:5
   7: core::ops::function::FnOnce::call_once
             at /rustc/e092d0b6b43f2de967af0887873151bb1c0b18d3/library/core/src/ops/function.rs:248:5
   8: core::ops::function::FnOnce::call_once
             at /rustc/e092d0b6b43f2de967af0887873151bb1c0b18d3/library/core/src/ops/function.rs:248:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

failures:
    vector::vector_tests::tests::test_features_aliasing_compile_fail

Can you compile the code it's testing? Easiest is probably to turn it into an example.
This isn't the first time trybuild is causing us problems, and I've also seen it fail when testing locally one of your PRs (I forgot why).

Can you compile the code it's testing?

I'm not familiar enough with what's going on here to know what you mean... I'm guessing you mean something other than cargo test?

Yeah, copy the file in https://github.com/georust/gdal/tree/master/tests/compile-tests to examples, then try to build it with cargo check --example 01-features-aliasing-errors.rs. It's supposed to fail with the error in https://github.com/georust/gdal/blob/master/tests/compile-tests/01-features-aliasing-errors.stderr, but if the compiler output ever changes, the test starts failing.

@lnicola :
Transcript:
❯ git rev-parse --short HEAD
24280ad
❯ cp tests/compile-tests/*.rs examples
❯ cargo check --example 01-features-aliasing-errors
...
error[E0502]: cannot borrow `layer` as immutable because it is also borrowed as mutable
  --> examples/01-features-aliasing-errors.rs:10:17
   |
9  |     for _ in layer.features() {
   |              ----------------
   |              |
   |              mutable borrow occurs here
   |              mutable borrow later used here
10 |         let _ = layer.defn();
   |                 ^^^^^^^^^^^^ immutable borrow occurs here

For more information about this error, try `rustc --explain E0502`.
error: could not compile `gdal` due to previous error

❯ cat tests/compile-tests/01-features-aliasing-errors.stderr
error[E0502]: cannot borrow `layer` as immutable because it is also borrowed as mutable
  --> $DIR/01-features-aliasing-errors.rs:10:17
   |
9  |     for _ in layer.features() {
   |              ----------------
   |              |
   |              mutable borrow occurs here
   |              mutable borrow later used here
10 |         let _ = layer.defn();
   |                 ^^^^^^^^^^^^ immutable borrow occurs here

Seems to be the same.. 🤔 I'll dig further.

Ah, I missed this. Are your saying that cargo test vector -- test_features_aliasing_compile_fail actually fails reliably?

Could it be a whitespace difference like tabs vs. spaces, or related to the EOLs? You could try updating to a newer version.

Are your saying that cargo test vector -- test_features_aliasing_compile_fail actually fails reliably?

Yep... Test never passes on my local machine. I'll do a manual diff next.

test tests/compile-tests/01-features-aliasing-errors.rs ... mismatch
EXPECTED:
┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈
error[E0502]: cannot borrow layer as immutable because it is also borrowed as mutable
--> $DIR/01-features-aliasing-errors.rs:10:17
|
9  |     for _ in layer.features() {
|              ----------------
|              |
|              mutable borrow occurs here
|              mutable borrow later used here
10 |         let _ = layer.defn();
|                 ^^^^^^^^^^^^ immutable borrow occurs here
┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈
ACTUAL OUTPUT:
┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈
┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈
note: If the actual output is the correct output you can bless it by rerunning
your test with the environment variable TRYBUILD=overwrite
test vector::vector_tests::tests::test_features_aliasing_compile_fail ... FAILED
failures:
---- vector::vector_tests::tests::test_features_aliasing_compile_fail stdout ----
thread 'vector::vector_tests::tests::test_features_aliasing_compile_fail' panicked at '1 of 1 tests failed', /Users/sfitch/.cargo/registry/src/github.com-1ecc6299db9ec823/trybuild-1.0.55/src/run.rs:74:13
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
failures:
vector::vector_tests::tests::test_features_aliasing_compile_fail
test result: FAILED. 82 passed; 1 failed; 0 ignored; 0 measured; 118 filtered out; finished in 0.14s

This is mostly just annoying. To get around it I just run with TRYBUILD=overwrite and move on. Not sure I understand what is being achieved with this test anyway.

Yeah, I had problems un more exotic cases, like running the tests under gdb. Can you update trybuild, if it's not already picking up the latest version?
I'm (still) if favour of removing the whole thing.

This is very far-fetched, but can you try one more thing? You're using IntelliJ, right? Try to close your IDE, run cargo clean, and then cargo test.

This is very far-fetched

Hah! I was almost convince you were right off the bat!... however... 😞

❯ cargo clean && cargo test
...
test tests/compile-tests/01-features-aliasing-errors.rs ... error
Expected test case to fail to compile, but it succeeded.
test vector::vector_tests::tests::test_features_aliasing_compile_fail ... FAILED
failures:
---- vector::vector_tests::tests::test_features_aliasing_compile_fail stdout ----
thread 'vector::vector_tests::tests::test_features_aliasing_compile_fail' panicked at '1 of 1 tests failed', /Users/sfitch/.cargo/registry/src/github.com-1ecc6299db9ec823/trybuild-1.0.71/src/run.rs:101:13
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
failures:
vector::vector_tests::tests::test_features_aliasing_compile_fail
test result: FAILED. 200 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 3.38s
error: test failed, to rerun pass '--lib'

Good guess tho!


when compiling: fatal error: Eigen/Eigen: No such file or directory

This has been there for a week now. I have tried installing Eigen, running CMake again in a new directory, but still, this error persists. When I checkout an older commit and run it, it seems to compile fine.
I am trying some way with symlinks as someone on IRC suggested, but not entirely sure how to proceed. Any suggestions as to what I can do?

To make this issue more informative, we need more information about which steps you have performed (cmake command output) and the complete error message you are getting once you run make. It would also be useful to know which is the older commit that still works for you, so we can understand which are the changes that were applied in the meantime.

Output on running cmake ..
Output on running make

Output on running cmake ..
Output on running make

We need the full cmake output. Namely, create a new build directory and run cmake from there.

Output when running cmake in fresh directory
Output when running make in fresh directory

this is not an eigen error but related to spdlog .... fmt being mentioned.
@gf712 any ideas?

Indeed, what version of spdlog do you have locally?

Indeed, what version of spdlog do you have locally?

Package: libspdlog-dev
Architecture: amd64
Version: 1:0.16.3-1
Priority: optional
Section: universe/libdevel
Source: spdlog
Origin: Ubuntu

This is the entire log message:

@Hephaestus12 I think this has been solved right? The problem was that it was using an old shogun installation, and the Eigen config file was still there, but the Eigen directory that it pointed to was gone.


Links are broken

The download link "https://go.microsoft.com/fwlink/?linkid=851168" is forwarding to an 404 - File or directory not found.

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

ID: 13376c1b-df70-83de-8e6a-8ddc940774b5
Version Independent ID: 6a2531ee-ac88-ee83-b858-1928d6cb4c65
Content: Use the Microsoft Teams Rooms recovery tool - Microsoft Teams
Content Source: Teams/rooms/recovery-tool.md
Service: msteams
GitHub Login: @dstrome
Microsoft Alias: dstrome

@davidisler thank you for reaching out and helping to improve Microsoft Docs. I've had a quick look and the link seems to be working from my side now. Please can you check again to see if it works from your side as well. Thanks.


A lot of errors while building a source (from github repo) project...

I just want to understand how works one or the other code and make some changes for this.
But I have a lot of errors while building the game project.
Look,

What I need to install or to do for it work?
Also, I use Linux (Astra Linux).

Also I tried to  build HTML5. Look at log:
log.txt

This is an error from update of Defold extension for ~1.9.2+
You can try is fix works in the branch update:
https://github.com/Insality/druid/tree/update

This is an error from update of Defold extension for ~1.9.2+
You can try is fix works in the branch update: https://github.com/Insality/druid/tree/update

Thanks! It works! But I tired in windows, I will try it on openSUSE Tumbleweed soon...

This is an error from update of Defold extension for ~1.9.2+
You can try is fix works in the branch update: https://github.com/Insality/druid/tree/update

So, there it works pretty good too... )


fix(setup): add hbase-spark JAR to the release list

Which issue(s) this PR fixes:

Fixes #162
Additional comments:

To make clear that you license your contribution under
the Apache License Version 2.0, January 2004 and that you give permission to TOSIT,
you have to acknowledge this by using the following check-box.

[X] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004

[X] I hereby agree to grant TOSIT a copyright license to use my contributions.

@mehdibn @rpignolet can one of you make TOSIT-IO/hbase-connectors public?

@mehdibn @rpignolet can one of you make TOSIT-IO/hbase-connectors public?

Done.


Add document about choosing the right scheduler

Fixes https://github.com/dask/distributed/issues/364

Besides my small nitpicks, this looks good to me.


fix(driver/bpf): use always-y instead of always

Kbuild does not successfully compile our probe anymore with always.
This was also necessary for samples/bpf in the kernel [0]
[0] https://git.kernel.org/pub/scm/linux/kernel/git/bpf/bpf-next.git/commit/?id=5f2fb52fac15a8a8e10ce020dd532504a8abfc4e
Signed-off-by: Lorenzo Fontana<EMAIL_ADDRESS>What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug
Any specific area of the project related to this PR?
/area driver-ebpf
What this PR does / why we need it:
Compiling the probe against newer 5.11/5.12 kernels is now not working at all because the Makefile cannot pick the target. This PR fixes that.
Compiling for now only works with Clang 7 until this is fixed too https://github.com/falcosecurity/libs/issues/58
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
fix(driver/bpf): co

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

Needs approval from an approver in each of these files:

OWNERS [fntlnz]

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

@fntlnz: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

build-libs
31018a3c59f2eb389af25947ced9d49f1126de85
link
/test build-libs

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

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

@fntlnz: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

build-libs
31018a3c59f2eb389af25947ced9d49f1126de85
link
/test build-libs

build-libs-bundled-deps
31018a3c59f2eb389af25947ced9d49f1126de85
link
/test build-libs-bundled-deps

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

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

@fntlnz: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

build-libs
31018a3c59f2eb389af25947ced9d49f1126de85
link
/test build-libs

build-libs-bundled-deps
31018a3c59f2eb389af25947ced9d49f1126de85
link
/test build-libs-bundled-deps

build-libs-with-chisels
31018a3c59f2eb389af25947ced9d49f1126de85
link
/test build-libs-with-chisels

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

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

Anything else required to get this merged? @falcosecurity/libs-maintainers


UX: Designs for Change Log

Designs are created for the change log within the Park Name data register.
Acceptance Criteria

link to figma designs is available for change log designs

Notes

https://www.figma.com/file/qdjegD6YHWZPlyFT8dGXvk/Park-Name-Data-Register?type=design&node-id=2557%3A10432&mode=design&t=r3pYGYuwR11i4skm-1
Will add screenshots to implementation tickets once approved.

looking awesome - please add design screen shots and link to tickets @meredithom
moving to done


Network does not support arbitrary input size

Is your feature request related to a problem? Please describe.
Most, if not all, Keras network architectures support changing the input size. This is also true for the ViT-B/16 implementation found in the vit-keras project. When trying to do the same with this implementation, it fails.
Describe the solution you'd like
I dont expect "any" input shape to work, but it would be great if it was possible to change it, which works well with other architecture backbone implementations out there.
Additional context
A part of the error message can be seen here:
tensorflow.python.framework.errors_impl.InvalidArgumentError: Exception encountered when calling layer 'attn' (type WindowAttention).

{{function_node __wrapped__Reshape_device_/job:localhost/replica:0/task:0/device:GPU:0}} Input to reshape is a tensor with 3211264 values, but the requested shape has 802816 [Op:Reshape]

Call arguments received by layer 'attn' (type WindowAttention):
  • inputs=['tf.Tensor(shape=(256, 49, 64), dtype=float16)', 'tf.Tensor(shape=(1, 14, 14, 64), dtype=float16)']
  • kwargs={'training': 'None'}

I tried to change from 224x224 to 1024x1024 input size.

That is weird cuz I tried with different input shapes before and even trained models. I think this is either due to change of  recent updates of TensorFlow version change.

@andreped Hi, just set resize_query=True during model build, and it will support any input size =)

Setting resize_query=True indeed resolves the issue! At least the training launched. Cheers :]
Sorry for the late reply. Been on vacation, and have not had the time to return this this issue until now.


token is saved on sessionStorage but not added to request

Hello,
I was trying to make the automatic addition of the auth header to all of my requests but I'm unable to.
Below is a sample configuration:
OAuthModule.forRoot({
    resourceServer: {
        allowedUrls: ['http://www.angular.at/api'],
        sendAccessToken: true
    }
})

ERROR: the request was sent without access token.
Expected Behavior: all url that starts with http://www.angular.at/api, e.g. http://www.angular.at/api/user, will be intercepted and will be added with the access token.
Many thanks,
Toni

Could you verify the version of the library you're using in the package.json and package-lock.json files? Because I have something very similar in my application and it's working just fine, so the cause might well be in some code we're not seeing.
If that all checks out, could you post a repro / some code for a service you're using to access the API? I'm using something like this:
@Injectable()
export class StuffService {
  private readonly url: string;

  constructor(private http: HttpClient) {
    this.url = environment.apiServer + 'stuff';
  }

  getThings() {
    return this.http.get<Stuff[]>(this.url + '/stuff')
  }
}

Hi @jeroenheijmans ,
Thanks for replying on me.
I noticed that your example uses HttpClient so I changed that part in my implementation and it's working now :)
I'll be closing this issue now. Thanks a lot for your help ^_^


this project can use to nativescript-vue and create native mobile app?

What is the motivation for adding / enhancing this feature?

What are the acceptance criteria

...
...

Can you complete this feature request by yourself?

Additional information

Vue Storefront is 100% web-based PWA and I really don't see a reason to use nativscript for it.
Since NS is using it's own APIs and components it's impossible to port existing web application that relies strong on browser APIs into it.


Failed to install boost:x86-windows; couldn't find hostname (but it's up)

When I try to install boost:windows-x86, I get this log:
The following packages will be built and installed:
    boost:x86-windows
Starting package 1/1: boost:x86-windows
Building package boost:x86-windows...
-- CURRENT_INSTALLED_DIR=C:/Users/Administrator/Code/vcpkg/installed/x86-windows
-- DOWNLOADS=C:/Users/Administrator/Code/vcpkg/downloads
-- CURRENT_PACKAGES_DIR=C:/Users/Administrator/Code/vcpkg/packages/boost_x86-windows
-- CURRENT_BUILDTREES_DIR=C:/Users/Administrator/Code/vcpkg/buildtrees/boost
-- CURRENT_PORT_DIR=C:/Users/Administrator/Code/vcpkg/ports/boost/.
-- Using cached C:/Users/Administrator/Code/vcpkg/downloads/boost_1_65_1.7z
-- Testing integrity of cached file...
-- Testing integrity of cached file... OK
-- Extracting done
-- Downloading https://github.com/boostorg/range/commit/e7ebe14707130cda7b72e0ae5e93b17157fdb6a2.diff...
-- Downloading https://github.com/boostorg/range/commit/e7ebe14707130cda7b72e0ae5e93b17157fdb6a2.diff... Failed. Status: 6;"Couldn't resolve host name"
Error: Building package boost:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: boost:x86-windows
  Vcpkg version: 0.0.90-2017-09-29-9569e69b7cd1e2a1ba36edd0c8ca52544d470d17

Additionally, attach any relevant sections from the log files above.

Interestingly, navigating to the offending URL on this machine via curl fails as well.  But on another computer, that URL loads as expected.

Actually, this problem might not have anything to do with vcpkg; the server went down almost immediately after I posted this issue.  Still looking.


[Sema] Maintain the implicitness of call argument tuple/parens in coerceCallArguments

Cherry-pick of https://github.com/apple/swift/pull/31184 (reviewed by @xedin).
If any arguments were defaulted the tuple/paren was made implicit, even though
the original tuple/paren was present in the source.
This prevented some sourcekit ASTWalkers from considering them, making
refactorings, documentation info, jump-to-definition and other features
unavailable when queried via their argument labels.
Resolves rdar://problem/62118957

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - b2aa159fc2460ef5eff74c6ae3d4a4ebd88710d5

@swift-ci please test OS X Platform


Use Host Field for TLS Certificate

The certificate that is generated for Argo server uses the ArgoCD metadata.name in the SAN. Since the operator supports providing a host field, it might make sense to use that if provided, and fallback to the CRD name if absent, for the SAN hosts.

I think this was fixed by the v0.0.14 release?


refactor: improved debug logs

nicer debug messages
share meta debug namespace between all modules
use chalk and tildify to improve readability of debug logs
prevent plugin errors from crashing the entire program

Codecov Report

Merging #89 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #89   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          34     39    +5
  Branches        8      9    +1
=====================================
+ Hits           34     39    +5

Impacted Files
Coverage Δ

lib/findPlugins.js
100% <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 89d51e2...b72e37e. Read the comment docs.

:tada: This PR is included in version 1.2.8 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


test_turtlebot.py: addition on lists throws TypeError

python -m examples.test_turtlebot crashes due to this TypeError:
Traceback (most recent call last):
  File "/usr/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/home/abhishek/pybullet-planning/examples/test_turtlebot.py", line 98, in <module>
    main()
  File "/home/abhishek/pybullet-planning/examples/test_turtlebot.py", line 84, in main
    center = (lower + upper)/2. # Computing the center of the AABB
TypeError: unsupported operand type(s) for /: 'list' and 'float'

https://github.com/caelan/pybullet-planning/blob/d536384dca2c890bbbdb5c14f07a3b7580201d8a/examples/test_turtlebot.py#L82-L84
Since lower and upper are both type list, adding them results in concatenation instead of element-wise addition.

Do this and it should work just fine
lower = np.array(lower)
upper = np.array(upper)
center = (lower + upper) / 2.0


[repo] Prepare release Exporter.OneCollector-1.9.0

Note: This PR was opened automatically by the prepare release workflow.
Changes

CHANGELOG files updated for projects being released.
Public API files updated for projects being released (only performed for stable releases).

I noticed this PR was merged.
Post a comment with "/CreateReleaseTag" in the body if you would like me to create the release tag Exporter.OneCollector-1.9.0 for the merge commit and then trigger the package workflow.

/CreateReleaseTag

I just pushed the Exporter.OneCollector-1.9.0 tag.
The package workflow should begin momentarily.

The packages for Exporter.OneCollector-1.9.0 should be available on NuGet shortly.
Have a nice day!


docs(tutorial): add readme file for centerpoint_vs_centerpoint-tiny

Description

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The Reviewers must check the checkboxes below before approval.

[x] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[x] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Broken PR


[Fabric 1.0.1+1.19.2] Startup crash from Bovines and Buttercups

Bovines and Buttercups adds some custom status effects that lock other status effects in place. I'm not sure what it is about the implementation, but EMI 1.0.1 doesn't like it!
https://mclo.gs/MI7bm32

It seems Bovines and Buttercups... injects into EMI's mixins. There's nothing I can do, my mixin changed to support other use cases, you'll have to open a bug on them.

If you're curious as to why I'm using a mixin, it's because Bovines and Buttercups has an extra tooltip when hovering over the effect inside the menu, this issue will be fixed when I get the time.


C# Fix loading the native library on dotnet cli

Two separate commits:

Add a property to GrpcEnvironment to allow the user to specify the native library
Change how the library is loaded and used

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Not sure why some of the tests look to be unable to load the native library

Will have a look over the weekend, or on Monday. Fingers crossed it's something simple...

I've had a thought - I suspect it's due to where the Grpc.Core csproj files copies the native library - basically the code expects to find it as it's packaged in nuget. Fingers crossed that'll be easy to fix...

Okay, that seems to have improved the tests - not sure why the basic and performance tests are failing though. The basic tests doesn't seem to be the C# ones anyway...

Hi Jon,
To make it easier to read I organized this in two parts, one for each test:
The Basic tests
The reason the Basic tests failed is as follows, and mainly because of the dbg,c,linux build is failing:

I will reference the log which is located here:

https://grpc-testing.appspot.com/job/gRPC_pull_requests/10767/config=dbg,language=c,platform=linux/console

The main error happens because the /home/jenkins/workspace/gRPC_pull_requests/config/dbg/language/c/platform/linux directory cannot be created created:

Building remotely on grpc-jenkins-linux2 (linux) in workspace /home/jenkins/workspace/gRPC_pull_requests/config/dbg/language/c/platform/linux
java.io.IOException: Failed to mkdirs: /home/jenkins/workspace/gRPC_pull_requests/config/dbg/language/c/platform/linux
	at hudson.FilePath.mkdirs(FilePath.java:1191)
	at hudson.model.AbstractProject.checkout(AbstractProject.java:1268)
	at hudson.model.AbstractBuild$AbstractBuildExecution.defaultCheckout(AbstractBuild.java:604)
	at jenkins.scm.SCMCheckoutStrategy.checkout(SCMCheckoutStrategy.java:86)
	at hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:529)
	at hudson.model.Run.execute(Run.java:1720)
	at hudson.matrix.MatrixRun.run(MatrixRun.java:146)
	at hudson.model.ResourceController.execute(ResourceController.java:98)
	at hudson.model.Executor.run(Executor.java:410)

The reason for this is that the directory cannot be created for several reasons:

The jenkins user does not have proper permissions to create a directory somewhere along the path /home/jenkins/workspace/gRPC_pull_requests/config/dbg/language/c/platform/linux

The SSH keys might not be properly set up.

Or another reason that eventually falls back to one or both of the above two.

I do not have access to the Google Jenkins setup to help troubleshoot, and determine which it might  the precise cause - or if it is something else - but below is a link to a detailed manual for Distributed Builds using Jenkins:
https://wiki.jenkins-ci.org/display/JENKINS/Distributed+builds
The Performance smoke test
The reason the Peformance smoke test failed is as follows:

I will reference the log which is located here:

https://grpc-testing.appspot.com/job/gRPC_performance_pull_requests/4389/console

The performance test is triggered by the following script:

tools/jenkins/run_performance.sh
And the actual line that runs it is this:
tools/run_tests/run_performance_tests.py -l c++ node ruby csharp python --netperf --category smoketest

If the run_performance_tests.py script fails, it will cascade up to Jenkins to return the Build step 'Execute shell' marked build as failure status.

The reason why it is failing is because of the following, stemming from the run_worker_python.sh script:

++ dirname tools/run_tests/performance/run_worker_python.sh
+ cd tools/run_tests/performance/../../..
+ PYTHONPATH=src/python/grpcio_tests:src/python/gens
+ py27/bin/python src/python/grpcio_tests/tests/qps/qps_worker.py --driver_port=10500
I0709 08:34:59.354176604   21856 ev_epoll_linux.c:84]        epoll engine will be using signal: 36
D0709 08:34:59.354213858   21856 ev_posix.c:106]             Using polling engine: epoll
Traceback (most recent call last):
  File "src/python/grpcio_tests/tests/qps/qps_worker.py", line 61, in <module>
    run_worker_server(args.port)
  File "src/python/grpcio_tests/tests/qps/qps_worker.py", line 43, in run_worker_server
    server = grpc.server((), futures.ThreadPoolExecutor(max_workers=5))
  File "/home/jenkins/workspace/gRPC_performance_pull_requests/py27/local/lib/python2.7/site-packages/grpc/__init__.py", line 1230, in server
    return _server.Server(thread_pool, () if handlers is None else handlers)
  File "/home/jenkins/workspace/gRPC_performance_pull_requests/py27/local/lib/python2.7/site-packages/grpc/_server.py", line 739, in __init__
    completion_queue, server, generic_handlers, thread_pool)
  File "/home/jenkins/workspace/gRPC_performance_pull_requests/py27/local/lib/python2.7/site-packages/grpc/_server.py", line 603, in __init__
    self.generic_handlers = list(generic_handlers)
TypeError: 'ThreadPoolExecutor' object is not iterable
Exception AttributeError: "'Server' object has no attribute '_state'" in <bound method Server.__del__ of <grpc._server.Server object at 0x7f7ee618dcd0>> ignored

The reason the above is failing, is because in line 603 of the src/python/grpcio/grpc/_server.py script, it tries to make generic_handlers a list which it cannot, since it is a ThreadPoolExecutor object - (the AttributeError exception of no attribute '_state' happens in lines 756-757, and is inconsequential since the object is not built because of the first error):
self.generic_handlers = list(generic_handlers)

A ThreadPoolExecutor is a type of object as noted in the following link, under section 17.4.2:
https://docs.python.org/3/library/concurrent.futures.html
To actually create a list of objects, it might be best to append them as follows:
>>> class MyClass:
...     """A simple example class"""
...     i = 12345
...
>>> listOfObjects = []
>>> listOfObjects.append(MyClass)
>>> listOfObjects.append(MyClass)
>>> listOfObjects
[<class __main__.MyClass at 0x7ffff7f0a6b0>, <class __main__.MyClass at 0x7ffff7f0a6b0>]
>>> type(listOfObjects)
<type 'list'>

Hope it helps,
Paul

Overall, I think the changes in this PR are reasonable.
C# tests are passing, which is good (but currently they are only running with .NET and mono, not CoreCLR).
it might be worth checking coreCLR tests manually:
tools/run_tests/run_tests.py -l csharp --compiler coreclr  (and optionally --use_docker if you don't have dotnet cli installed locally).
We also have tests for distribution packages (the nugets) - but they have to be spawned manually as the build process is a bit more sophisticated:
I've started https://grpc-testing.appspot.com/view/Artifacts/job/gRPC_build_artifacts/1214/ and eventually, nuget packages will be built from the native artifacts built here and they will be tested by the https://grpc-testing.appspot.com/view/Artifacts/job/gRPC_distribtest/ build (I'll post a link there once the previous steps finish).
For dotnet CLI environment, we don't have any distribtests yet (currently only for desktop .NET 45 on windows and mono on Linux and Mac) - so with dotnet CLI the nugets need to be tested manually.

Distribtest links:
Nugets for this PR are here:
https://grpc-testing.appspot.com/view/Artifacts/job/gRPC_build_packages/1195/platform=windows/
Experimental nugets with netstandard support are here:
https://grpc-testing.appspot.com/view/Artifacts/job/gRPC_build_packages/1195/platform=linux/artifact/artifacts/
Distrib test results for the "official" nugets are here:
https://grpc-testing.appspot.com/view/Artifacts/job/gRPC_distribtest/1150/  (C# is green, so this PR should be safe).

LGTM

@apolcyn: This isn't the PR that should be merged. See #7401. I'll close this now to avoid confusion :)

(We'll need to merge the changes in #7401 onto master as well, of course.)


Better --help for enabling extra hint groups.

The --help text has nothing about enabling or disabling hint groups.
The -w option seems to be for individual hints only.

Do you know how to enable extra hint groups? I'm not sure I do, but if you do, I'll put it in the docs. I think you can do it with a .hlint.yaml somehow... Putting extra docs are a good idea.

I forgot for a moment that in Haskell projects and common DRY style usually used the --help text reflects the existing options by constructions. So this means groups can only be enabled with a .hlint.yaml file? Are you in favor of a patch to add a command line option to enable/disable groups too? If so I'll make one.

Yep, a flag to enable groups sounds like a good idea.


[Campaign] Implement Donation History for Campaign

On the Campaign Details Page create a history of donated amounts.
See the Figma Design here: https://www.figma.com/file/MmvFKzUv6yE5U2wrOpWtwS/Podkrepi.bg?node-id=6874%3A22174
Donations should be loaded via the backend API as per this issue https://github.com/podkrepi-bg/api/issues/67

Isn't that done or am I not understanding the issue?
Look here

Should we limit the amount of  donations that we show. Maybe the latest 5-10, because if there are a lot the UI wouldn't look good.


PackageId:CPython39.Exe.x64;PackageAction:Install;ReturnCode:1603;

This issue has been moved from a ticket on Developer Community.

Describe your issue here
===DO NOT EDIT BELOW THIS LINE===
PackageId:CPython39.Exe.x64; PackageAction:Install; ReturnCode:1603;

Original Comments
Feedback Bot on 5/3/2023, 11:03 PM:
(private comment, text removed)

Original Solutions
(no solutions)

dupe of #7596


Implemented authentication caching

Fix for issue #5
the signature for init in the InfobloxNetMRI class is changed because of this!

TODO:  Rewrite unit tests to reflect new signature

Current coverage is 86.92%

Merging #6 into master will decrease coverage by -1.88% as of d6623b4

@@            master      #6   diff @@
======================================
  Files            2       2
  Stmts          134     153    +19
  Branches         0       0
  Methods          0       0
======================================
+ Hit            119     133    +14
  Partial          0       0
- Missed          15      20     +5

Review entire Coverage Diff as of d6623b4

Powered by Codecov. Updated on successful CI builds.

Fantastic - it's practically a re-write but that's OK :)
Nice work and thank you!

Oh well :) It was an opportunity to allign it a bit more with the Perl API.
And also, it was a good excuse to learn how to write tests. Never did that before.
Maybe double check if the default value for use_ssl and ssl_verify are acceptable. The init method also still needs a docstring so it get's nicely documented when generating the documentation.

Any plans on releasing a new version any time soon?
We're running the "straight-from-github" version now, but being able to install it the regular way would be more convenient.

Yes, ok, will do.
On Feb 19, 2016, at 7:37 AM, Joris Beckers <notifications<EMAIL_ADDRESS>wrote:
Any plans on releasing a new version any time soon?
We're running the "straight-from-github" version now, but being able to install it the regular way would be more convenient.
—
Reply to this email directly or view it on GitHubhttps://github.com/infobloxopen/infoblox-netmri/pull/6#issuecomment-186199597.


InvalidDistribution: Metadata is missing required fields: Name, Version.

mac pypi package maker is giving this error, so I disabled the mac build for now

The linux build has the same issue, see https://github.com/bempp/ndgrid/actions/runs/12123960667/job/33800978913


Live Reload For Atom

How do I do it?

Implement a plugin for Atom similar to the one for Sublime: https://github.com/qualialabs/reval/blob/master/plugins/sublime.py

you just need to send a POST request to the Meteor server containing the code and file path of the file you'd like to reload

If you're there, apparently something along this lines would work except for "unmatched OUTDENT", if you stick it into init file. I don't actually know coffeescript.
# initialization file (not found)
http = require('http')

atom.commands.add 'atom-text-editor',
  'editor:qualia-reval': () ->
      req = http.request({
      host: 'localhost',
      port: 3000,
      path: '/reval/reload?filePath=' + atom.workspace.getActiveTextEditor().getTitle(),
      method: 'POST'
    });
    req.write(atom.workspace.getActiveTextEditor().getBuffer().getText());
    req.end();

The original post for help: https://discuss.atom.io/t/translate-4-line-sublime-package-please/44529/5

Thank you @janat08! I ended up modifying it slightly and here is what I am currently working with.
http = require('http')

atom.workspace.observeTextEditors (editor) ->
  editor.onDidChangeCursorPosition ->
    req = http.request({
      host: 'localhost',
      port: 3000,
      path: '/reval/reload?filePath=' + atom.workspace.getActiveTextEditor().getTitle(),
      method: 'POST'
    });
    req.write(atom.workspace.getActiveTextEditor().getBuffer().getText());
    req.end();

Cheers!

so this watches mouse then?

No, it watches the cursor. Every time you type it reloads. If you know a way of how to make it work only for projects that contains reval and everything, feel free to let me know. :)

I imagine if you try and figure out if port 3000 is active you then could if (true) onDidChange...

True, the problem is that I run a lot of different stuff on 3000.

well then set up scripts in both atom and the package that try and find each other :D. That is whoever start's last posts to the other and await for confirmation.

This is my suggested way of dealing w/ this problem: https://github.com/qualialabs/reval/issues/3
This technique is only implemented in the Spacemacs plugin at the moment.

although automatic updating from atom can get ugly if you're fixing hard errors that reload meteor which means that it stops receiving page saves or something, what you wind up with is several files that are saved on atom, but are patched with old versions before the save. Clear all with alert would help.

I suspect it goes so far as to seriusly screw with app, in such a way that anomalies.

Here you go: https://atom.io/packages/atom-reval


HTTP triggered function app HTTP status codes is really vague

Linked documentation page has only three use cases for HTTP return values:

HTTP 204 No Content with an empty body in Functions 2.x and higher
HTTP 200 OK with an empty body in Functions 1.x
If a function that uses the HTTP trigger doesn't complete within 230 seconds, the [Azure Load Balancer](https://learn.microsoft.com/en-us/azure/app-service/faq-availability-performance-application-issues#why-does-my-request-time-out-after-230-seconds-) will time out and return an HTTP 502 error.

Missing use cases:

HTTP status code when function throws exception to runtime

Looks like HTTP 500

HTTP status code when function runtime crashes

No idea

HTTP status code when infrastructure is down and runtime cannot be started

looks like HTTP 503

In out use case the exact HTTP status code to underlying reason is needed for orchestrators is to retry only when there is startup issues with consumption plan. All other errors will be handled manually. To do this we need to be sure when 500, 502 and 503 status code are returned by the service as response for HTTP function app invocation.

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

ID: 5b6cd6a0-5364-65e6-b088-cc00093f836f
Version Independent ID: 3c90d8fb-0fea-3496-e73e-0020ff33adef
Content: Azure Functions HTTP trigger
Content Source: articles/azure-functions/functions-bindings-http-webhook-trigger.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

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


Add #![feature(buf_stream)] to compile with current nightly.

Hi there. This adds a crate level feature attribute to allow it to build on the current nightly.

Hi, thanks!


Automate: Verify that the bind apiexport command works as expected

This test case check the basic functionalities of kcp bind apiexport command. It derives from test case OCP-60113. PTAL. Thanks!
Test: passed: (29s) 2023-02-16T11:41:06 "[area/apiexports] Author:zxiao-Critical-[API] Verify that the bind apiexport command works as expected [Suite:kcp/smoke/parallel]"

/cc @kasturinarra @Phaow @xingxingxia

@zxiao-redhat small comment, otherwise looks good to me, thanks !!

/lgtm

/approve


Re-export imported types

Is your feature request related to a problem? Please describe.
Let's say I have wrapper that is supported by a plugin. The plugin implements an interface. The interface declares a type Connection. I want to use the Connection type in the wrapper as Interface_Connection and accept it as a method argument.
I can use Connection in the wrapper and accept it as a method argument without friction for app developers, but wrapper developers who want to integrate my wrapper will have problems. They cannot import the interface type Ethereum_Interface_Connection, nor can they import the interface type and cast it.
Alternatively, I can re-declare a type Connection in my wrapper. When I want to interact with the plugin, I create a new object Interface_Connection and copy the properties of Connection to it. This preserves the type's ergonomics for other developers.
    pub fn new(connection: &Option<Connection>) -> Self {
        let iprovider_connection = connection.as_ref().map(|conn| IProviderConnection {
            network_name_or_chain_id: conn.network_name_or_chain_id.clone(),
            node: conn.node.clone(),
        });
        Self {
            connection: iprovider_connection,
            iprovider: get_iprovider(),
        }
    }

Describe the solution you'd like
While rebuilding a struct solves the problem, it is an inconvenience that seems unnecessary. In an extreme case, it could make the wasm module meaningfully larger.
The only meaningful difference between the types is the static URI property on the imported type.
I think imported types should be re-exported if they are used in the schema (this could possibly be limited to imported interfaces).
Describe alternatives you've considered

We could only re-export types if they come from interfaces
We could remove the static URI property (not sure if this helps in all languages)
We could keep the status quo

Additional context
Add any other context or screenshots about the feature request here.

I think we can use theory to guide our implementation here, because I agree the current implementation is flawed, and we can use theory to prove it is flawed.
When a wrapper "implements an interface", I propose it should be a superset of said interface. This means that it should contain all elements of the interface within itself, and these elements should not differ from that of the interface. If you look at what we have today, the elements in the implementation do differ from that of the interface, because they are namespaced. For example:
ens/interface.eth
type Module {
  interfaceMethod(arg: Object!): Bytes!
}

type Object {
  prop: String!
}

ens/impl.eth
#import { Module } into Interface from "ens/interface.eth"

type Module implements Interface_Module {
  anotherMethod(arg: String!): String!
}

When this schema is built, the result is as follows:
type Module implements Interface_Module {
  interfaceMethod(arg: Interface_Object!): Bytes!
  anotherMethod(arg: String!): String!
}

type Interface_Module @import(
  uri: "ens/interface.eth"
) {
  interfaceMethod(arg: Interface_Object!): Bytes!
}

type Interface_Object @import(
  uri: "ens/interface.eth"
) {
  prop: String!
}

As you can see above, the types within ens/interface.eth are not 1:1 to the types in ens/impl.eth. This is a problem and should be fixed (as you've stated above).
Proposed Solution
We should fundamentally differentiate between "importing a dependency" and "implementing an interface". This can be done by no longer using the #import { ... } into ... from ... syntax  when you are implementing an interface. Instead we can simply do something similar to the following:
ens/impl.eth
#implement * from "ens/interface.eth"

type Module {
  anotherMethod(arg: String!): String!
}

After the schema is built, the result would be:
type Module @implements(
  uri: "ens/interface.eth",
  type: "Module"
) {
  interfaceMethod(arg: Object!): Bytes!
  anotherMethod(arg: String!): String!
}

type Object @implements(
  uri: "ens/interface.eth",
  type: "Object"
) {
  prop: String!
}

Thoughts?

Related: https://github.com/polywrap/toolchain/issues/1376

This is a really fantastic idea! I think it would provide a better experience. I'm 100% for it.
This could also make it easy to validate that a wrapper implements an interface.

@dOrgJelli This doesn't solve the problem with namespacing everything and creating long nested namespaced types. having the ability to import to the local namespace and re-export only needed types would cover more use-cases then the one for just the interface-implementations

@Niraj-Kamdar I think it'd be helpful if you:

included a code snippet of the problematic scenario you're describing
included a code snippet of a proposed solution

Hey @krisbitney @Niraj-Kamdar, I talked with @namesty about this more today in the context of WRAP ABI 0.2. There's a lot to discuss, but in general I want to propose another solution to Kris' described problem above.
NOTE: Please disregard my prior proposal above depicting the #implement ... from "..." syntax.
So just as Kris and Niraj have both stated, the problem is that we need a better way of re-exporting types. Currently today, if you want to re-export a type, you must do the following (based on Kris' scenario above):
#import { Connection } into Interface from "ethereum-interface"

type Connection implements Interface_Connection {
}

which would result in a final ABI looking like this:
{
  objects: [{
    name: "Connection",
    properties: [...],
    ...
  }],
  importedObjects: [{
    uri: "ethereum-interface",
    namespace: "Interface",
    name: "Interface_Connection",
    properties: [...],
    ...
  }]
}

This however could be streamlined, and instead be something more like this:
#include { Connection } from "ethereum-interface"

which would result in a final ABI looking like this:
{
  objects: [{
    name: "Connection",
    properties: [...],
    ...
  }],
}

Pro: smaller ABI
Con: lose URI information
NOTE: if we introduce the #include keyword, we'd also want to refactor the "local schema import" functionality to use the #include syntax as well. For example #include { CommonType } from "../common/schema.graphql"
Thoughts?

Hey @krisbitney @Niraj-Kamdar, I talked with @namesty about this more today in the context of WRAP ABI 0.2. There's a lot to discuss, but in general I want to propose another solution to Kris' described problem above.
NOTE: Please disregard my prior proposal above depicting the #implement ... from "..." syntax.
So just as Kris and Niraj have both stated, the problem is that we need a better way of re-exporting types. Currently today, if you want to re-export a type, you must do the following (based on Kris' scenario above):
#import { Connection } into Interface from "ethereum-interface"

type Connection implements Interface_Connection {
}

which would result in a final ABI looking like this:
{
  objects: [{
    name: "Connection",
    properties: [...],
    ...
  }],
  importedObjects: [{
    uri: "ethereum-interface",
    namespace: "Interface",
    name: "Interface_Connection",
    properties: [...],
    ...
  }]
}

This however could be streamlined, and instead be something more like this:
#include { Connection } from "ethereum-interface"

which would result in a final ABI looking like this:
{
  objects: [{
    name: "Connection",
    properties: [...],
    ...
  }],
}

Pro: smaller ABI Con: lose URI information
NOTE: if we introduce the #include keyword, we'd also want to refactor the "local schema import" functionality to use the #include syntax as well. For example #include { CommonType } from "../common/schema.graphql"
Thoughts?

I think it's good! I don't think losing the URI information really matters, but I also don't really remember how the URI information is used.

@dOrgJelli How about?
#import { Connection } into Interface from "ethereum-interface"
#using namespace Interface::* <-- import whole Interface namespace to local namespace
#using namespace Interface::Connection <-- import Connection to local namespace
#using namespace Interface::{Connection, Module}

or something rusty:
#import { Connection } into Interface from "ethereum-interface"
#use Interface::* <-- import whole Interface namespace to local namespace
#use Interface::Connection <-- import Connection to local namespace
#use Interface::{Connection, Module}

We can also have concept of private/internal types with explicit re-export syntax
PRO: Unexported type wouldn't end up in ABI, way to specify internal types in schema
CON: lose some type info but doesn't really matter because only being used internally in the wasm module.
I haven't looked much into the ABI 0.2 but this may be incorporated inside ABI 0.2

Hey @Niraj-Kamdar, I agree that we should take an "ABI import minimization" approach, so that we are not exporting needless types in the ABI. If types are only used within the wasm code, and nowhere else, then we shouldn't be including them in the ABI.
If we break this down, I think we have 3 user-scenarios we need to support:

Import type Foo, use Foo in the wrapper's implementation language via schema bindings.
Import type Foo, use Foo within another schema type Bar (as a property or something like that).
Import type Foo, re-export Foo from the schema as if it were defined inline.

Here is how I propose we solve these scenarios:
1
#import { Foo } into Namespace from "uri"

2
#import { Foo } into Namespace from "uri"

type Bar {
  prop: Foo!
}

3
#import { Foo } into Namespace from "uri"

type Foo implements Namespace_Foo {
  ...must re-add all of Foo's props...
}

NOTE: if you #import { Module } ... (or other syntax for importing functions) it will always be included in the imports section of the ABI.


Feature Request: Flexible duration from helper

I like to request an alternate duration input based on an selectable (input_datetime) helper. This makes possible to include the duration offered from an device.

This is a valid request. I need to check how much effort it is.

I have implemented that in my fork just after the initial revision of this sensor came out:
https://github.com/grol-hass/ha_epex_spot_sensor_variable_duration
Feel free to pull the changes from there.
(I'm using it to determine when to charge my home battery from grid, that's why I also added code to make sure intermittent time slots are adjacent if possible)

@grol-hass Thanks for sharing. I will use this a template.


Add Env variable for Enforcing provider

Signed-off-by: revolyssup<EMAIL_ADDRESS>Description
This PR adds two functionalities -

We only had provider authentication up until this point. This PR adds authentication mechanisms as enums currently supporting only two- NoAuth and ProviderAuth. NoAuth is equivalent right now to bypassing the Auth middleware. And provider authentication is the usual None/Remote provider authentication. The reason for this approach is to get all the endpoints under AuthMiddleware for better security such that no endpoint is bare. Each endpoint can describe the type of authentication it needs while wrapping itself under AuthMiddleware. In future, for endpoints which are used by external components like adapters, we can have a different type of AuthenticationMechanism and that change will be additive here under as a new switch case expression.
This PR also adds a flag to enforce the provider authentication for a specific type of provider. (current use case: Enforce remote provider flag)

Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Codecov Report
Base: 17.42% // Head: 13.00% // Decreases project coverage by -4.42% :warning:

Coverage data is based on head (a882745) compared to base (3d0747a).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6440      +/-   ##
==========================================
- Coverage   17.42%   13.00%   -4.43%
==========================================
  Files         115      115
  Lines       10809    10821      +12
==========================================
- Hits         1884     1407     -477
- Misses       8460     9106     +646
+ Partials      465      308     -157

Flag
Coverage Δ

gointegrationtests
?

unittests
13.00% <0.00%> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

server/cmd/main.go
0.00% <0.00%> (ø)

server/handlers/content_modifier.go
0.00% <ø> (ø)

server/handlers/handler_instance.go
0.00% <0.00%> (ø)

server/handlers/middlewares.go
0.00% <0.00%> (ø)

server/router/server.go
0.00% <0.00%> (ø)

mesheryctl/internal/cli/root/system/start.go
1.70% <0.00%> (-57.03%)
:arrow_down:

mesheryctl/internal/cli/root/system/status.go
1.14% <0.00%> (-55.18%)
:arrow_down:

mesheryctl/internal/cli/root/system/update.go
1.20% <0.00%> (-48.20%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
1.92% <0.00%> (-48.08%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
2.25% <0.00%> (-42.11%)
:arrow_down:

... and 7 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Only two comments outstanding on this review.

These docs need updated:
API References

https://docs.meshery.io/reference/graphql-apis
https://docs.meshery.io/reference/rest-apis

Extensibility Guide

https://docs.meshery.io/extensibility/api
https://docs.meshery.io/extensibility/providers

These need updated with the new "enforce" feature, including NoAuth and ProviderAuth

🚀 Preview for commit c1138630fb767726bf81555b65e2e2b0760639e1 at: https://636249adde83507d017d4905--meshery-docs-preview.netlify.app


[Update or Remove Document Request] Safety tips is deprecated and removed

Need to help to update / remove this documentation on safety tips as it seems to be deprecated and removed as per below work item.
Task 1336827: Antispoof: Remove safety tip feature for spoof

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

ID: d1daade0-c937-6daf-cdc5-e6e23214426d
Version Independent ID: 49836613-b9a3-fc37-ffc8-3589dda16d45
Content: Safety tips in email messages - Office 365
Content Source: microsoft-365/security/office-365-security/safety-tips-in-office-365.md
Service: o365-seccomp
GitHub Login: @chrisda
Microsoft Alias: chrisda

@qaehtesh, I'm closing this as a duplicate of #736.
Thanks for the feedback.


Claim: Counter shouldn't move left and right after each number change

Claim: Counter shouldn't move left and right after each number change.
The counter's number should change without the movement/jumps of the entire counter.

Checked on Dev and QA environment
Device used: Android and iOS device
Look as per requirements


Make framework scripts conditional

Moved angularjs, react and ember scripts out of the handlebars example into separate files and added conditional checks to add the scripts. Setup is done in a non breaking way, by default all scripts will be added.
Motivation for change:

Improve performance by centralizing the scripts preventing having to load the same code multiple times.
Add ability to omit all scripts when using a different setup/framework (in my case Aurelia)
Make it easy to overwrite the scripts if needed

@mpetrovich Could you please review this change?


Add FIREWORK MASHUP | Classical Idol - Classical VIP x IDOL | Geometry Dash by Deji [GMD]

Song Name
FIREWORK MASHUP | Classical Idol - Classical VIP x IDOL | Geometry Dash
Artist Name
Deji [GMD]
Source
Youtube
Youtube Link
https://www.youtube.com/watch?v=_PyRO7XXxq4
Direct File Link
No response
Song ID
502268
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 75206202

accept


PortableText duplicated list items

List items are duplicated when they also contain a style and there is more than 1 child element, such as when a word is bold/italic/etc
Example here: https://stackblitz.com/edit/github-4darib-nrxitl4i?file=src%2Fcomponents%2Fportabletext%2FPortableText.astro,src%2Fpages%2Findex.astro
The use-case here is that our content editors want to use text styles inside a list, e.g. a bullet list where the list items have a custom text style that makes the text larger than usual. They also want to be able to make certain words bold or italic, this results in a list item with a style (in our case the style could be "standfirst") and also results in the block having multiple elements in the children array. The output of this is that the content of the list item gets duplicated

example of the JSON that gets fed into the PortableText:
{
  "_key": "a8f6c37f1f5d",
  "_type": "block",
  "children": [
    {
      "_key": "f69fa0972de1",
      "_type": "span",
      "marks": [],
      "text": "list standfirst "
    },
    {
      "_key": "12e6dbf8f10a",
      "_type": "span",
      "marks": [
        "strong"
      ],
      "text": "bold"
    },
  ],
  "level": 1,
  "listItem": "bullet",
  "markDefs": [],
  "style": "standfirst"
}

Hi @aFrontendDev thanks for posting this issue and providing a clear example on Stackblitz. I'll need to do some digging, as there may be a bug with nestLists from the @portabletext/toolkit dependency, or there might be an underlying bug in this library itself.
I'll keep you updated on my progress.

Hi @aFrontendDev, thanks for your patience. I'm happy to report that the issue you raised has been resolved in the upcoming v0.11.0 release. I've added a test to specifically cover this scenario, and I've also verified the fix in the demo. Everything is looking good.

Closing this issue as it has been resolved. The resolution was verified by the test added in this commit: https://github.com/theisel/astro-portabletext/commit/761d67b216229e37d6181001cd028192b396eac9

Amazing, thanks @theisel

Hi @aFrontendDev, just wanted to let you know that v0.11.0 has  released, and it includes a fix for the issue you reported.  Please let me know if you encounter any further problems.


Mesh model freezing issue in imodeljs (.TileCache size issue)

I have already registered this issue for "Discussions"

MicroStation has many element type such as surface, solid, cell, shape, and mesh etc.
For surface, solid, cell, shape model, the published bim file works quickly in imodeljs.
However for mesh model, the published bim file works very slowly.
The imported mesh model from fbx, obj, 3ds file also works very slowly.
When bim file is published, the .Tiles and TileCache files are generated together.
.Tiles and TileCache files used with .bim files in imodeljs when zooming and rotating.
For surface, solid, cell, shape model, .Tiles and TileCache files has no problem.
However, for mesh model, the imodeljs is continually freezing.
If the .TileCache file size is increased, can this freezing issue be solved?
Is there any method for increasing .TileCache file size?

What application are you using to view the iModel?
The .tilecache file is neither produced nor used by iModel.js applications and its size certainly should not be changing as you navigate the model within an iModel.js application.
The .tiles file is populated with tiles as they are generated.
If you can provide more information about your model (or even the .bim itself) we will have a much better chance of diagnosing the problem.

What application are you using to view the iModel?
The .tilecache file is neither produced nor used by iModel.js applications and its size certainly should not be changing as you navigate the model within an iModel.js application.
The .tiles file is populated with tiles as they are generated.
If you can provide more information about your model (or even the .bim itself) we will have a much better chance of diagnosing the problem.

Thank you for your kind answer.
I was wrong about the need for *.Tilecache file.
When I opened the *.bim file in imodeljs, I noticed that the files of *.Tile, *.HeirchyCache.db,
and *.HeirchyCache.db were automatically created.
Without the file *.TileCache, there was no problem opening the file *.bim in imodeljs.
If it's not a TileCache issue, I wonder if there's another way to quickly process the mesh model in imodeljs.


SubscriptionOptionsMsg

Provide an option to serialize SubscriptionOptions.
See Issue https://github.com/nats-io/java-nats-streaming/issues/51
On a long term perspective, should SubscriptionOptions be itself made from protobuf?

Coverage increased (+0.2%) to 92.814% when pulling 73e9cd0eae73dc3e609cd6624142863ea800b60c on Logimethods:master into b4a2f56eb1c3f5eb739797d72202ed924d7451a9 on nats-io:master.

Coverage increased (+0.2%) to 92.814% when pulling 127c1b48a5b303f1d10bc28c4064e69bfaf94168 on Logimethods:master into b4a2f56eb1c3f5eb739797d72202ed924d7451a9 on nats-io:master.

Has been solved by https://github.com/nats-io/java-nats-streaming/pull/56


Connect Four Game

Relevant Issues
Closes #244
Description
Added a ConnectFour cog which includes a connectfour command (with several different aliases) to allow users to play a Connect Four game in a specific channel
Also includes an ai subcommand to allow a user to play against a "CPU" player
Reasoning
This was part of the retro gaming season we had. Better late than never!
Screenshots

Additional Details
When playing against a CPU there's

a 1 in 10 chance they will NOT place the counter in the correct spot to win when possible
a 1 in 4 chance they will NOT block the player from winning where possible

Did you:

[x] Join the Python Discord Community?
[x] If dependencies have been added or updated, run pipenv lock?
[x] Lint your code (pipenv run lint)?
[x] Set the PR to allow edits from contributors?

Hello, what is the status here? Looks like there is some feedback that needs to be addresed, do you think you will be able to do that, @Suhail6inkling?

I'm going to close this for the time being due to inactivity and lack of communication.
Feel free to re-open if you feel like finishing this in the future.

I would like to work on this pr.

I would like to work on this pr.

This branch was updated and fixed up in #560.

This branch was updated and fixed up in #560.


Android structure project

I have moved the project to a new Android structure. It will allow us add submodules like wear.
PS: I want to play with wear in a near future

@anuj1708 Can you check if everything still working and I did not missed any file?

@danielpassos Please mvn license:format

Just found and problem, and also need to fix the travis problem with wear project. I'll close it and send a new one

Build fixed!

Looking everything good to me. Nothing is broken


Request: new load syntax for not cloning submodules

A plugin/repo's submodules are not always needed for regular use, and in those cases, refraining from cloning them can save the user some space and bandwidth. A new syntax like zgen load --no-submodules... could be handy.

Hi, I've created a maintained fork of zgen - dubbed zgenom - which fixes some bugs as well as introducing new features.
Please let me know what you think of my solution.


Using with yargs, it overrides options

Compatibility issue with yargs package. When i try to use both while creating my owm cmd tool, options for schema-validator overrides my yargs options

I was having the same issue which appears to be caused by the default require
const validateSchema = require('yaml-schema-validator');
which parses the process.argv for the commander module.
The workaround that worked for me was to require the validateSchema module directly and bypass the default require.
const validateSchema = require('yaml-schema-validator/src/index');

Thanks @qualipsolutions , the workaround worked for me as well. In the 1st one i used exec package to run it but that required package installed globally or running it from node_modules. This one is better.


Add a shell script for backup with email confirmation

I added a little shell script for backup with e-mail confirmation. Hope it helps =)


Support custom OpenAI api hosts

This adds support for custom API hosts for the OpenAI LLM. Needed to run against custom inference end points with OpenAI API compatibility.

Add api_host parameter to support custom OpenAI API host
Fix an issue where default values for config parameters would override the specified value
Update docstrings for clarity and documentation purposes
Adjust imports to multiple lines for readability

Hi @pixelkaiser, thank you for your contribution. The team was planning to review and integrate your request in the upcoming 0.17 release of Council. Would you mind if we reopened your request?

No problem, please reopen.


More updates after R5

This PR updates:

The geometric accuracy bias in X and Y direction to be provided in absolute (positive) values.
Change default browse image height to 2048 (rather than 1024) following ASF request.

Thank you, @seongsujeong


WKWebViewController

Could you also add a podspec for https://github.com/apache/cordova-plugin-wkwebview-engine?

{
  "name": "cordova-plugin-wkwebview-engine",
  "version": "1.0.1",
  "summary": "Cordova WKWebView Engine Plugin",
  "description": "The WKWebView Engine plugin allows you to use a WKWebView instead of a UIWebView.",
  "homepage": "https://github.com/apache/cordova-plugin-wkwebview-engine",
  "license": {
    "type": "Apache License, Version 2.0",
    "text": "    \"type\": \"Apache License, Version 2.0\",\n    \"text\": \"      Licensed under the Apache License, Version 2.0 (the \"License\");\n      you may not use this file except in compliance with the License.\n      You may obtain a copy of the License at\n      \n      http://www.apache.org/licenses/LICENSE-2.0\n      \n      Unless required by applicable law or agreed to in writing, software\n      distributed under the License is distributed on an \"AS IS\" BASIS,\n      WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\n      See the License for the specific language governing permissions and\n      limitations under the License.\n\"\n"
  },
  "authors": {
    "The Adobe PhoneGap Team": ""
  },
  "platforms": {
    "ios": "8.0"
  },
  "source": {
    "git": "https://github.com/apache/cordova-plugin-wkwebview-engine.git",
    "tag": "1.0.1"
  },
  "source_files": "src/ios/*.{h,m}",
  "requires_arc": true,
  "dependencies": {
    "Cordova": [
      ">= 3.0.0"
    ]
  }
}

I see that cordova-plugin-wkwebview-engine wants Cordova 4. I would really like to get this going and am happy to help. Is there a roadmap for getting the pods updated to the latest version of Cordova?

Thanks @onato! Sending a pull request with your updates will speed things up. Currently we are off for our winter break right now and won't seriously review things until the new year.
The plugins themselves (save the wkwebview plugin) should be backwards compatible for any Cordova version.

Hi @onato! I'm going to work on getting the support for cordova-ios 4 out asap and will look at creating a podspec for the cordova-plugin-wkwebview-engine next. Cheers,
Holly

Hi @onato!
I put out an updated Cordova CocoaPod supporting 4.0.1 today and I also added a new
cordova-plugin-wkwebview-engine podspec.
This sample template project has also been updated to use the latest versions of everything and defaults to WKWebView. I've tested it out with both Objective-C and Swift projects and it's all working.
Hope this helps!
Holly


Introduce support for new Slash Command types.

About this pull request
Mentionable & Number types are given support in this PR.
Changes
Added enumerables and subclass logic support extension.
Checklist

[ ] I've run the pre_push.py script to format and lint code.
[ ] I've checked this pull request runs on Python 3.6.X.
[ ] This fixes something in Issues.

Issue:

[X] This adds something new.
[ ] There is/are breaking change(s).
[ ] (If required) Relevant documentation has been updated/added.
[ ] This is not a code change. (README, docs, etc.)

This is a duplicate of #273, right? I'm pretty sure it is.

God damnit, DeltaX always beats me to it.


Initial boundary value problem 1: advection-diffusion on unit square

Solve IBVP1 using PINNs and VPINNs.

[ ] pure forward problem
[ ] data-enhanced forward problem
[ ] inverse problem (identify diffusion coefficient)
[ ] surrogate model for advection velocity in range [0, 1]

Reference for description of PDE problem:
https://doi.org/10.1002/nme.7241
Reference for tasks in the context of PINNs for contact:
https://doi.org/10.48550/arXiv.2308.12716

@makima87 might be able to help you with the generation of suitable (synthetic, noisy) measurement data.


🚸 increase the temperature for outline builder

The provided code snippet reflects a change in the automated content generation script to improve content variety. The generate_single_content function now accepts two different models with distinct temperature settings (high and low) for outline and content generation, which is expected to leverage diverse outputs from the models. The main function is also updated to initialize and pass these two different models instead of one.

#4  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @mingi3314 and the rest of your teammates on  Graphite


docs: Update where contributors can find issues.

Description ✏
Update link for issues tab in Contributing docs.
The current link in the contributing docs for the issues tab isn't working. This should lead to the new repo's issues tab.
Type of Change 🐞

[ ] Feature - A non-breaking change which adds functionality.
[ ] Fix - A non-breaking change which fixes an issue.
[ ] Refactor - A change that neither fixes a bug nor adds a feature.
[x] Documentation - A change only to in-code or markdown documentation.
[ ] Tests - A change that adds missing unit/integration tests.
[ ] Chore - A change that is likely none of the above.

Checklist ✅

[ ] I have done a self-review of my code.
[ ] I have manually tested my code (if applicable).
[ ] I have added/updated any relevant documentation (if applicable).

Nit, but follow the updated PR title guidelines por favor! See here.


What is the private function add_timeout of StreamHandler for?

Hello,
I have a question about the add_timeout function.
It's a private one but I can't find the location where this function is called in the class.
Does anybody please tell me what's for?
It looks like we might be able to leverage this to set http timeout.
Thanks in advance.
Knack

add_ methods are used for adding the request options to the stream context.
See http://docs.guzzlephp.org/en/latest/request-options.html#timeout


add another shape comment

Stack from ghstack (oldest at bottom):

-> #514
#513

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

Comparison is base (e9d0d03) 75.57% compared to head (e010887) 75.57%.

Additional details and impacted files
@@                  Coverage Diff                   @@
##           gh/ebsmothers/25/base     #514   +/-   ##
======================================================
  Coverage                  75.57%   75.57%
======================================================
  Files                        234      234
  Lines                      16110    16110
======================================================
  Hits                       12175    12175
  Misses                      3935     3935

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


[BUG] ValueError: Dataset name 'dataset' is not available

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu
Python version (python --version): 3.10.0
FiftyOne version (fiftyone --version): 0.20.1
FiftyOne installed from (pip or source): pip

Commands to reproduce
As thoroughly as possible, please provide the Python and/or shell commands used to encounter the issue. Application steps can be described in the next section.
None

Describe the problem
The problem I am facing is quite strange, I get an error which says ValueError: Dataset name 'dataset' is not available. But the dataset name is already available. However, it gets fixed when I edit the name a second time and then run the code, no errors pop up again.
Code to reproduce issue
dataset_dir = "/file/path/"
name = "dataset"

# Create the dataset
dataset = fo.Dataset.from_dir(
    dataset_dir=dataset_dir,
    dataset_type=fo.types.ImageClassificationDirectoryTree, # Or ImageClassificationDirectoryTree for classification dataset
    name=name,
)

print(dataset)

Other info/logs
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[3], line 2
      1 # Create the dataset
----> 2 dataset = fo.Dataset.from_dir(
      3     dataset_dir=dataset_dir,
      4     dataset_type=fo.types.ImageClassificationDirectoryTree, # Or ImageClassificationDirectoryTree for classification dataset
      5     name=name,
      6 )
      8 print(dataset)

File [~/anaconda3/envs/fiftyone/lib/python3.10/site-packages/fiftyone/core/dataset.py:5199](https://file+.vscode-resource.vscode-cdn.net/home/intern/Desktop/fiftyone-scripts/~/anaconda3/envs/fiftyone/lib/python3.10/site-packages/fiftyone/core/dataset.py:5199), in Dataset.from_dir(cls, dataset_dir, dataset_type, data_path, labels_path, name, label_field, tags, dynamic, **kwargs)
   5097 @classmethod
   5098 def from_dir(
   5099     cls,
   (...)
   5108     **kwargs,
   5109 ):
   5110     """Creates a :class:`Dataset` from the contents of the given directory.
   5111
   5112     You can create datasets with this method via the following basic
   (...)
   5197         a :class:`Dataset`
   5198     """
-> 5199     dataset = cls(name)
...
    105 if bool(list(conn.datasets.find(query, {"_id": 1}).limit(1))):
--> 106     raise ValueError("Dataset name '%s' is not available" % name)
    108 return slug

ValueError: Dataset name 'dataset' is not available

What areas of FiftyOne does this bug affect?

[x] App: FiftyOne application issue
[ ] Core: Core Python library issue
[ ] Server: FiftyOne server issue

Willingness to contribute
The FiftyOne Community encourages bug fix contributions. Would you or another member of your organization be willing to contribute a fix for this bug to the FiftyOne codebase?

[ ] Yes. I can contribute a fix for this bug independently
[x] Yes. I would be willing to contribute a fix for this bug with guidance from the FiftyOne community
[ ] No. I cannot contribute a bug fix at this time

Hi Neville, this means that there is a dataset called "dataset" already existing.
What do you mean by:

But the dataset name is already available.

If you run fo.load_dataset('dataset') what happens?

Hi Neville, this means that there is a dataset called "dataset" already existing. What do you mean by:

But the dataset name is already available.

If you run fo.load_dataset('dataset') what happens?

I also have this issue. The command returns:
ValueError: Dataset 'dataset' not found

@OlegBEZb can't reproduce on my side. You must have a dataset called 'dataset' that already exists.
What's the result of running
fo.load_dataset('dataset')

Hi Neville, this means that there is a dataset called "dataset" already existing. What do you mean by:

But the dataset name is already available.

If you run fo.load_dataset('dataset') what happens?

I get the result as shown:
Name:        dataset
Media type:  image
Num samples: 3503
Persistent:  False
Tags:        []
Sample fields:
    id:           fiftyone.core.fields.ObjectIdField
    filepath:     fiftyone.core.fields.StringField
    tags:         fiftyone.core.fields.ListField(fiftyone.core.fields.StringField)
    metadata:     fiftyone.core.fields.EmbeddedDocumentField(fiftyone.core.metadata.ImageMetadata)
    ground_truth: fiftyone.core.fields.EmbeddedDocumentField(fiftyone.core.labels.Classification)

However, when I try to run the above function which is fo.Dataset.from_dir(), it shows me errors but if I change the name variable to something else, it works. But if I restart my kernel in jupyter notebook and run the cell again, I get the same error back again.
For example, the name variable does not work with the same value twice until I keep changing it after the first run of the notebook.
ValueError                                Traceback (most recent call last)
Cell In[8], line 3
      1 # Create the dataset
      2 # fo.load_dataset(name)
----> 3 dataset = fo.Dataset.from_dir(
      4     dataset_dir=dataset_dir,
      5     dataset_type=fo.types.ImageClassificationDirectoryTree, # Or ImageClassificationDirectoryTree for classification dataset
      6     name=name,
      7 )
      9 print(dataset)

File [~/anaconda3/envs/fiftyone/lib/python3.10/site-packages/fiftyone/core/dataset.py:5199](https://file+.vscode-resource.vscode-cdn.net/home/intern/Desktop/fiftyone-scripts/~/anaconda3/envs/fiftyone/lib/python3.10/site-packages/fiftyone/core/dataset.py:5199), in Dataset.from_dir(cls, dataset_dir, dataset_type, data_path, labels_path, name, label_field, tags, dynamic, **kwargs)
   5097 @classmethod
   5098 def from_dir(
   5099     cls,
   (...)
   5108     **kwargs,
   5109 ):
   5110     """Creates a :class:`Dataset` from the contents of the given directory.
   5111
   5112     You can create datasets with this method via the following basic
   (...)
   5197         a :class:`Dataset`
   5198     """
...
    105 if bool(list(conn.datasets.find(query, {"_id": 1}).limit(1))):
--> 106     raise ValueError("Dataset name '%s' is not available" % name)
    108 return slug

ValueError: Dataset name 'dataset1' is not available

I have changed to name=dataset1, it will show up the same error even after I restart the kernel and run the cell until I change the name to something else.
Hope it's clear. If not, I can send a screen recording to show how it works.
Thanks,
Neville

Hi, @swheaton thanks for the quick response. I have tried out the second option and it seems to work out just fine. Thanks a lot. 👍🏼

Hi, I have noticed this happens to me when debugging on Pycharm, if I have a dataset = fo.Dataset(         name="Example",         persistent=False,     ) in my scripts, everything behaves as expected when running normally. But if I try debugging the code, find a bug, and stop the execution, then next time I try to execute the code it will throw the ValueError of the name not being available. Simply rerunning things without debugging mode clears this error.
I suspect this happens because when stopping the debugging run, no time is given to the fiftyone library to clear the reserved "Example" name from its internal data, so not commenting this here expecting a fix from fiftyone, just leaving the info in case it helps anyone having this problem.
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 22.04.3 LTS
Python version (python --version): Python 3.10.12
FiftyOne version (fiftyone --version): FiftyOne v0.23.2, Voxel51, Inc.
FiftyOne installed from (pip or source): pip
Pycharm Community version: 2023.3.3 installed from ubuntu software snap store