what is "Would you like to enable auto fetching of Git repositories?"

VSCode Version: 1.19
OS Version: osx

Include info on what is "Would you like to enable auto fetching of Git repositories?" along with the alert

Feature Explanation: Code can automatically and periodically fetch refs from your remotes. It calls git fetch periodically. That way it can show you how many incoming/outgoing commits you might have. This is the feature that the popup asks about.

I was just Googling for the same :)

I was just Googling for the same

Same here. Does anyone know what this is?

I've added an explanation in the first comment of this issue:

Feature Explanation: Code can automatically and periodically fetch refs from your remotes. It calls git fetch periodically. This is the feature that the popup asks about.

how would i disable/enable this in after making a selection?

Same question as @taivu. Plus, will anything on the current VSC set-up change if I select "no" to the prompt?

Firstly I don't think this should have been raised as an issue.
See https://github.com/Microsoft/vscode/issues/34684
Think this setting was previously true by default, now it's false. The prompt is just to remind you to enable if you wish.
Configured via "git.autofetch": false, so can be easily undone

Verification
I've improved the message to:

Would you like Code to periodically run git fetch?

I've also added a Read More link to https://go.microsoft.com/fwlink/?linkid=865294 which will soon talk a bit more about this setting: https://github.com/Microsoft/vscode-docs/pull/1307


Unnecessary parameter in demo

As the title says, the below lines should be removed. (it does not cause any problem now since they are removed afterward by this line)
https://github.com/ros-simulation/gazebo_ros2_control/blob/master/gazebo_ros2_control_demos/config/cartpole_controller_effort.yaml#L15
https://github.com/ros-simulation/gazebo_ros2_control/blob/master/gazebo_ros2_control_demos/config/cartpole_controller_velocity.yaml#L15

I had a similar comment for the ros2_control demos. might be a copy-paste error?
cc @destogl

Yes, this is definitely a copy-paste error. @JaehyunShim would you open a PR?

PR https://github.com/ros-simulation/gazebo_ros2_control/pull/68

@ahcorde  Thanks!


Correct JTAC sensor_frame param

Hmm, looks like main got force pushed to in this repo so I can't merge this PR now without fixing some things...I'll have more time to look at this Tuesday

Closing this and opening https://github.com/PickNikRobotics/moveit_studio_ur_ws/pull/258 instead since the change this was correcting got bumped from main so this isn't really relevant anymore


Where should I put 'TAGGIT_CASE_INSENSITIVE'

The documentation says:

If you want django-taggit to be CASE-INSENSITIVE when looking up existing tags, you’ll have to set the TAGGIT_CASE_INSENSITIVE setting to True

It says nothing about where this line should go.
Where should I put this line? In settings.py or in models.py?

Based on the tests it looks like it should be in the settings. https://github.com/jazzband/django-taggit/blob/b21c94be8b6b2da9516ee1f635d87f1323a88a43/tests/tests.py#L723
I think some clearer docs would help clear this up for other users.

According to _TaggableManager on 211 line
python case_insensitive = getattr(settings, "TAGGIT_CASE_INSENSITIVE", False)
we need to add this parameter in settings.py

Yep, this is currently a global setting (tho personally I think it would make sense for this to be overridable on tag-based basis, depending on your use-case)

(keeping this open for now, will close when we tweak the documentation)

@rtpg Did you want me to update the docs? Is there an issue or card i should pick up?

Yeah if you could send in a PR that makes the originally cited text a bit clearer we can merge that in, and close out this issue

This is in the getting started docs. I suspect we can close this without any modifications.
See https://github.com/jazzband/django-taggit/blob/master/docs/getting_started.rst#getting-started

alright just modified that sentence, closing this one out now


Release Models of Mixups and MogaNet and Update Features in V0.2.6

Updateing new features:

Fix the classification heads and update implementations and config files of AlexNet and InceptionV3.

Uploading Benchmark Results (release):

Updating mixup benchmarks on ImageNet as provided in AutoMix and update results in Model_Zoo_sup.md.
Release pre-trained models and logs of MogaNet.

Updating documents:

Update documents of mixup benchmarks on ImageNet with new backbones in Model_Zoo_sup.md.
Update awesome lists in Awesome Mixups and Awesome MIM and provide teaser figures of most papers as illustrations.

We will refactor the implementation of mixup augmentations and upload pretrained models of mixup benchmarks in V.0.2.7.


[14.0.x] ISPN-15230 Server tests unable to run due to use of JUnit 4 imports

https://issues.redhat.com/browse/ISPN-15230

merged! thanks @ryanemerson !


Lines and polygons add limits(Minimum length and area) / 3D point cloud supports custom colors in annotation

:tada: This PR is included in version 1.7.0-alpha.6 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.14.0-alpha.8 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


MAINT: more consistent behaviour on serialisation annotated seqs

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1123 unchanged lines in 24 files lost coverage.
Overall coverage increased (+0.02%) to 91.522%

Files with Coverage Reduction
New Missed Lines
%

.nox/test-3-10/lib/python3.10/site-packages/cogent3/util/deserialise.py
16
90.77%

.nox/test-3-10/Lib/site-packages/cogent3/util/deserialise.py
16
90.77%

.nox/test-3-11/lib/python3.11/site-packages/cogent3/util/deserialise.py
16
90.77%

.nox/test-3-11/Lib/site-packages/cogent3/util/deserialise.py
16
90.77%

.nox/test-3-8/lib/python3.8/site-packages/cogent3/util/deserialise.py
16
90.77%

.nox/test-3-8/Lib/site-packages/cogent3/util/deserialise.py
16
90.77%

.nox/test-3-9/lib/python3.9/site-packages/cogent3/util/deserialise.py
16
90.77%

.nox/test-3-9/Lib/site-packages/cogent3/util/deserialise.py
16
90.77%

.nox/test-3-10/lib/python3.10/site-packages/cogent3/core/sequence.py
26
94.26%

.nox/test-3-10/Lib/site-packages/cogent3/core/sequence.py
26
94.26%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
235471

Relevant Lines:
257284

💛  - Coveralls


Allow joke !!/alives in CHQ

Currently, there are several joke responses to the !!/alive command which only show in SOCVR and the Tavern. I'd like to make CHQ great again by bringing them back to Charcoal HQ.
Does anyone have any objections? It's quite obvious by the fact that Smokey's posting a message that the bot must be alive, so there shouldn't be any concerns from that perspective.

Meh, sure. We don't really need new joke stuff, but I'm okay with existing stuff being used across the board


Switch chain to rmrk2 on rmrk2.kodadot.xyz

We have new domains prepared for testing rmrk2 implementation, so users have pleasant times, it would be nice to redirect them to rmrk2 automatically, as now they land on bsx

https://rmrk.kodadot.xyz
https://rmrk2.kodadot.xyz

can we still switch chains from here in these urls? @yangwao

can we still switch chains from here in these urls? @yangwao

yes I guess so!

hey @Jarsen136 can you have a look? We would like to have it ready this week for rmrk2.kodadot.xyz and bugs catching :)
Along with

#5536


config-sample.yaml instead of config.yaml

Hi Chris,
please ship a config-sample.yaml instead of config.yaml so extracting the release zip will not overwrite an existing config file.
Best regards
Joerg

Hej @lingej,
quite nice idea. I've fixed that issue in f0612c6.
You'll find that enhancement in Release v0.2.0. 😄
Best,
Christian


Separate out platform-specific CSS

CSS should be separated out into "shared" and "platform" (or "mac"/"ios") style sheets, since there are a ton of duplicated rules.

Fixed a while ago!


Inconsistent context menu item for Merge Parent Into

Checkout and Rebase Onto Parent but Checkout Selected Branch And Merge Parent Into It
I requested it myself, for the sake of clarity... but I already don't like it 😅 It looks really awkward in context menu.
Maybe change the latter into:
Checkout And Merge With Parent

This is less precise (after all, it could be interpreted as "merge this branch into parent")... but putting it close to Rebase makes the purpose quite clear.
Or alternatively! How about changing both captions:
Checkout and Sync to Parent by Rebase
Checkout and Sync to Parent by Merge

❓
This way it's clear that both actions achieve the same goal ⚽

Good points made here and I agree with your concerns about the first option.
I would go for the second - it reflects the common thing; making the branch to be in sync.
And we can provide a more extensive explanation of git operation going under the hood in the action description.

NOTE: Changing the action names will result with inconsistency to the scenarios and recordings from README (and the other places probably).


Better quote randomization

Adds an imperfect way to reduce the amount of duplicate quotes (from >quote get ) without messing with the db or doing complex math.
Also fixes a gazillion clippy warnings

note: RANDOM_INDEX_SIZE could be removed if you want to hold the entire length in memory, which will fully remove the chance of getting duplicates until reshuffled; but there currently is no call to the force_shuffle method anywhere.

note: RANDOM_INDEX_SIZE could be removed if you want to hold the entire length in memory

It looks like nothing is stopping it from using the entire length either way.


[WFLY-5732] Fix export-journal operation with custom paths

if the paths resources are not defined (as is the case by default),
use their default values to resolve the actual paths
check if the path value is absolute to determine whether the
relative-to path must also be resolved.

JIRA: https://issues.jboss.org/browse/WFLY-5732

Looks good, but holding off merging until the downstream issue has been acked


XML_parser ability parse comments form oval-definition

Append XML_parser ability parse comments form oval-definition.
This is first PR of appending of comments form oval-definition to graphs.

Hello @Honny1! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file graph/client.py:

Line 97:100: E501 line too long (114 > 99 characters)
Line 132:1: W293 blank line contains whitespace
Line 155:100: E501 line too long (114 > 99 characters)

In the file graph/oval_graph.py:

Line 179:1: W293 blank line contains whitespace
Line 206:23: E231 missing whitespace after ':'
Line 207:23: E203 whitespace before ':'
Line 207:24: E231 missing whitespace after ':'
Line 208:24: E231 missing whitespace after ':'
Line 208:34: E231 missing whitespace after ':'
Line 212:19: E203 whitespace before ':'
Line 213:20: E231 missing whitespace after ':'
Line 213:30: E231 missing whitespace after ':'

In the file tests/test_client.py:

Line 89:1: E302 expected 2 blank lines, found 1
Line 102:13: E117 over-indented
Line 102:49: E711 comparison to None should be 'if cond is None:'
Line 178:1: E302 expected 2 blank lines, found 1

Looks great, merging!


Add terms of use

Add terms of use.
Partially based on https://foundation.wikimedia.org/wiki/Terms_of_Use/en
For https://github.com/ambuda-org/ambuda/issues/127

Wonderful!


Release Google.Cloud.Deploy.V1 version 2.18.0

Changes in this release:
New features

Add properties for nested resource name references (commit 15eec4d)

Created release for Google.Cloud.Deploy.V1-2.18.0


Typescript Visual Studio 2015 Update 3 isn't showing up anywhere

How do i get typescript back in visual studio??  I have resinstalled several times cleared data and component cache everything....

I actually removed VS PREVIEW "15" which might has hosed up everything.  Just a clue.

if you open a new TS file, what do you see? do you see colorization working?
can you uninstall all version of TS from add/remove programs, install update 3 from (https://msdn.microsoft.com/en-us/library/mt752379.aspx), then reinstall TS 2.0.2

No colorization is working and when I try to update with that link it says cannot install visual studio update 3 is not installed.  I downloaded the visual studio 2015 update 3 iOS and web installer both of which failed.
Sent from my iPhone

On Sep 20, 2016, at 10:39 PM, Mohamed Hegazy<EMAIL_ADDRESS>wrote:
if you open a new TS file, what do you see? do you see colorization working?
can you uninstall all version of TS from add/remove programs, install update 3 from (https://msdn.microsoft.com/en-us/library/mt752379.aspx), then reinstall TS 2.0.2
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Can you share install logs? it should be the last installer in your %TEMP% folder with dd_ prefix

I zipped up all loges starting with dd_
vsLogs.zip

I am wiping out my machine tomorrow so any help would be great.


Fix slow InlineInlineable

Fixes https://github.com/numba/numba/issues/9204
Avoid deepcopying Loc in Var.__deepcopy__. This fixes slow IR copying in InlineInlineable.
In one case, time spend in InlineInlineable went from 1second to <300ms.

Thanks for the patch. Borrowing the _gen_involved function from numba.tests.test_ir_inlining and modifying it slightly demonstrates the issue and fix.
from numba import njit
import numpy as np
import timeit

# this forces the inliner to recursively inline 5x then stop.
_count = 0
def cost_model(*args):
    global _count
    if _count < 5:
        _count += 1
        return True
    return False

# used in _gen_involved
_GLOBAL = 1234

def _gen_involved():
    _FREEVAR = 0xCAFE

    @njit(inline=cost_model)
    def foo(a, b, c=12, d=1j, e=None):
        f = a + b
        a += _FREEVAR
        g = np.zeros(c, dtype=np.complex64)
        h = f + g
        i = 1j / d
        # For SSA, zero init, n and t
        n = 0
        t = 0
        if np.abs(i) > 0:
            k = h / i
            l = np.arange(1, c + 1)
            m = np.sqrt(l - g) + e * k
            if np.abs(m[0]) < 1:
                for o in range(a):
                    n += 0
                    if np.abs(n) < 3:
                        break
                n += m[2]
            p = g / l
            q = []
            for r in range(len(p)):
                q.append(p[r])
                if r > 4 + 1:
                    s = 123
                    t = 5
                    if s > 122 - c:
                        t += s
                t += q[0] + _GLOBAL

        fortran = int((f + o + r + t + r + a + n).real)
        if fortran < 0:
            return foo(fortran, fortran, fortran, fortran, fortran)
        else:
            fortran

    return foo

involved = _gen_involved()

@njit
def foo():
    involved(1, 1, 1, 1, 1)

print(timeit.timeit(lambda: foo.compile(()), number=10))

Locally, this patch makes the above compile about 2x faster.


Update the opensource charts to be compatible with k8s v1.22+

This mirrors #9 but modifies the opensource deploy charts.
Didn't get a chance to actually deploy this set of changes, since we only run the enterprise version. Should work the same but would benefit from a quick test on a cluster before merging.

Thanks for doing this @vihangm!


Selenium 4 support without page factory

Description
After migration of Selenium 4, mobile scripts are getting fail. What is the Appium .NET Driver version supporting Selenium 4 without page factory and without Selenium Extra package.
Element Format :IWebElement btnCancel => FindElement(By.XPath("//button/span[contains(.,'Cancel')]"));
Environment

.NET client build version or git revision if you use some shapshot: .NET 6
Appium server version or git revision if you use some shapshot: Latest
Desktop OS/version used to run Appium if necessary:
Node.js version (unless using Appium.app|exe) or Appium CLI or Appium.app|exe:
Mobile platform/version under test: Andriod/iOS
Real device or emulator/simulator: Real Device

@pro-pintuyadav did you tried this version?
v5.0.0_beta1

@akinsolb, I see no response. So I think we can close it for now.


Disable RuboCop style warning for multiline 'expect { ... }'

By convention, RSpec uses 'expect { ... }'' rather than 'expect do ... end', regardless
of whether the block spans multiple lines or not.

Just added another commit here. Let me know if I should split them up.

As I kept working through RuboCop style warnings on the RSpec codebase, I found a number of others which are not useful.

Thanks for taking a look at this!
We have up to now only applied rubocop to our lib directory -- that is, the code we ship that users use.  You can see the rubocop call in our build process:
https://github.com/rspec/rspec-dev/blob/3a2bba6f8c1dcb116958da07b839187022b727d0/travis/script/functions.sh#L180
If we're going to start setting guidelines and rules for the spec files, we'll want to change our build script to start running it against them as well.  However, I'm not sure that the extra work of maintaining rubocop compliance in our specs is worth it. Rubocop compliance in the specs are far less important (IMO).  And I'm not sure the effort to keep up rubocop compliance in general has been a worthwhile tradeoff.
What do other @rspec/rspec folks think?

The first time I ran Rubocop on some of my source code, I absolutely hated it; I felt like this really stupid computer program was telling me how to write code, without understanding the subtle reasons why I chose to format a certain line this way or that way.
Having used it a bit more now, I see that it can be a really useful tool, if you customize its configuration to only warn about things which are really, clearly, unequivocally bad style. By default, it produces far too many warnings. If you are finding it more trouble than it is worth, it may be that you could profitably disable some warnings which are not useful for you.
It is very reasonable for you to apply a different standard of coding style in your specs. That's why this PR disables a number of style checks in spec code, but keeps them working on the main library code. Actually, there are more warnings which still need to be disabled on the spec code.
I am willing to do the up-front work to make all your specs comply with style guidelines. Once that is achieved, maintaining it should be easy.

I'm ok with this, generally we're pretty good at this, if there's too many exceptions (to the rules) we can always turn off a rule for just the specs right?

If someone is willing to do the initial setup work, I'm ok with applying a (potentially different) set of rubocop rules against spec directory.
(as long as I get to disable all the ones I don't like :P)

also hello @alexdowad nice to see you on another project! :)

@xaviershay Hello.

Shouldn't all these new rules only be applied to the spec directories?

RSpec maintainers, do you like:
method { another { } }
or:
method { another { }}?

Also, what do you think of aligning things like this:
expect(a).to   eq(1)
expect(abc).to eq(1)

Another thing. The latest version of RuboCop detects code like this:
args.map { |s| s.to_sym }
...and suggests this instead:
args.map(&:to_sym)
I like the terseness of the latter version, but am not sure if it could hurt performance. Which style do you prefer?

@myronmarston said:

I'm not sure the effort to keep up rubocop compliance in general has been a worthwhile tradeoff.

I'm coming around to this opinion on rubocop in RSpec also. I do use rubocop in several projects, but the amount of effort that has been going into rubocop in RSpec seems out of proportion to its usefulness.
Changing {:a => 1} to { :a => 1 } (or vice-versa) is a pretty pointless change, as both are readable. Making that sweeping change across the project will create a ton of code churn and make useful git blaming/history more difficult.
I don't think RSpec has had any problem socially policing contributed code that's in poor style - I can understand that having a tool define the standard can make that easier to be consistent, but it's also raising the barrier to contributing. I would not want to fail a build or reject a contribution because {:foo => :bar} didn't have padding spaces, or their code didn't line up in neat columns (which is a style I personally dislike btw).
I'm not suggesting throwing out rubocop completely, but I'd rather disable any cops that would result in sweeping changes to many files across the codebase than get really strict about a 100% consistent style. Rather than talking about what style people prefer in the abstract, I'd rather address specific pieces of code that stick out as poor quality - I'm not sure several of the rubocop rules discussed here are useful for identifying those spots.
I would rather have just a few 'critical' cops enabled, and use good judgement with simple non-tool-enforced guidelines than create a minefield of rules.

Compare https://github.com/alexdowad/rspec-dev/blob/disable_block_style_warning/travis/.rubocop_rspec_base.yml to http://guides.rubyonrails.org/v3.2.21/contributing_to_ruby_on_rails.html#follow-the-coding-conventions

I'm not suggesting throwing out rubocop completely, but I'd rather disable any cops that would result in sweeping changes to many files across the codebase than get really strict about a 100% consistent style. Rather than talking about what style people prefer in the abstract, I'd rather address specific pieces of code that stick out as poor quality - I'm not sure several of the rubocop rules discussed here are useful for identifying those spots.

I would rather have just a few 'critical' cops enabled, and use good judgement with simple non-tool-enforced guidelines than create a minefield of rules.

I agree with this.  Rubocop is a really neat tool but many of the existing cops identify issues I don't care about at all (such as the hash spacing issue or single vs double quotes) and keeping in compliance takes additional effort from everyone plus it increases the difficulty of contributing to RSpec.  IMO, generally consistent code style helps reduce the cognitive load on maintainers and code reviewers by reducing "out of place" bits.  What Rubocop is doing now with its built-in cops is far beyond that, though; it's nit-picking at things I would never notice and really don't care about and just creating busy work.
@soulcutter Do you want to take a stab at trimming our list of enabled cops down to something more reasonable?

PR updated with a number of cops which cause excessive numbers of warnings disabled.

Trying to be as helpful as possible here: Would one of the maintainers like to write up a document describing what standard of coding style you would like to maintain, and I will come up with a set of cops which enforce exactly what is described in that document (no more and no less)?
If everybody likes @xaviershay's style guide, that could even be used.

Closed due to lack of activity.


operators [N] zookeeper-operator (0.10.3)

Signed-off-by: fswang<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[x] Are you familiar with our contribution guidelines?
[x] Have you packaged and deployed your Operator for Operator Framework?

[x] Is your submission signed?
[x] Is operator icon set?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

/merge possible

/merge possible

/merge possible

/merge possible

/merge possible


Added quick fix which should allow the model to run only the single-i…

If the user sets the number of two-way interactions to 0, the model output file used by the wrapper outputs blanks under the pairwise interaction label. Under the current version, this raises a ValueError in the wrapper because wrapper assumes there will always be a compatible float following the two-way interaction header.
I added a quick fix which catches this ValueError and sets the two-way interactions to 0 if an incompatible (non-numerical) value is parsed. This allows users to input the number of two-way interactions (k2) as 0, which is useful for running the one-way or global-bias models as a baseline.
This is more closely inline with the libfm Binary input behavior, which, when the number of two-way interactions is set to 0, models only the global or one-way biases.
There may be a way to handle the k2 input as 0 earlier in the setup phase of the wrapper, but this fix works. Catching the exception as a ValueError may create issues if for some reason the temporary model output file outputs other incompatible values which don't appear as a result of the user inputting 0 for k2.

Quick and easy 👍   When you feel like improving the solution you are free to open another PR.
Thanks for the contribution!


{'code': 88, 'message': 'Rate limit exceeded.'} is not handled

This is what is currently returned. No further information is provided.
2024-09-05 10:56:20,013 [ERROR] :: Couldn't find guest token
2024-09-05 10:56:20,013 [ERROR] :: Couldn't find guest token
2024-09-05 10:56:20,014 [ERROR] :: Couldn't generate a new session.

However the actual reason was because of a rate limit. The response was
{'code': 88, 'message': 'Rate limit exceeded.'}
It is not currently handed in util.check_for_errors() or util.check_api_rate_limits()

Hey @burruplambert,
Which line or url does return the given rate limit exceeded response?


Added fromTimestamp() and fromTimeString() methods

Fix for #1394. Added documentation for fromTimestamp() and fromTimeString() methods. Also reordered methods a bit.

Merged into develop at zendframework/zf2-documentation@8d8508f36aa3c63608cd8b85e7192bfb466dbbce
Thanks @Martin-P!


Contact point

Hi! Is there a way to get the contact point between two polygons?
Thank you

Sadly there's nothing in the lib that will give you this. There could be many (or infinite) collision points depending on how the polygons are oriented (e.g. think of two squares sitting on top of each other - they contact along the entire line segment of the top of the one box and the bottom of the other)
You might be able to individually test each edge of the polygon to see which one(s) collide and then use the overlap vector in the Result to determine the exact point - but i'm not 100% sure that would work in all cases.

I think this issue should open back up, its important for integrating physics


🐞[bug]: the cursor animation is bugged

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
the cursor animation is still visible even after user's cursor is moved to another display or not at the site,
Expected ;
the cursor animation should disapper if the user's cursor is not on the site.
I will fix this issue. Assign me this
Expected behavior

Add ScreenShots

What browsers are you seeing the problem on?
Firefox, Chrome, Safari, Microsoft Edge
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24EXTD contributor
[X] I have starred the repository

l would like to work on this issues


Change more usages of SemanticModel::is_builtin to use resolve_builtin_symbol or match_builtin_expr

Summary
This PR switches more callsites of SemanticModel::is_builtin to move over to the new methods I introduced in #10919, which are more concise and more accurate. I missed these calls in the first PR.
Test Plan
cargo test. I haven't introduced any new tests in this PR, because I added quite a few in #10919, and this is just doing the same thing as that PR. One existing violation in the test fixtures becomes newly fixable, though.

I've added the internal label because we don't need two changelog entries for https://github.com/astral-sh/ruff/pull/10919 and this.

Performance is overall neutral, but some linter benchmarks are showing regressions of 1-2%. I think that's probably okay, given that #10982 was also overall neutral, and the final version of that PR had improvements of 2-3% on some of the same benchmarks.


[WIP] chore: migrate to supabase auth helpers RC

Thanks for doing this @thorwebdev!  Helping me get a head start on how to update auth with the new Supabase v2 lib.


Absolute Position Encoding：Why are the two tensors not alternately merged？

https://github.com/tensorflow/tensor2tensor/blob/bafdc1b67730430d38d6ab802cbd51f9d053ba2e/tensor2tensor/layers/common_attention.py#L453
In the orginal paper, the position_embedding is like this:  [..., sin i, cos i, ...]

See #177 and #1591 (and #1677).

just different orderings of the same set of channels,  The effects of both are consistent theoretically.


[formMixin?] State updates (probably from reference loading) reset all fields

Since 0.7.1, when I click for the first time on the 'Edit' button, and start right away to edit any field value, it is resetted a few times. It doesn't happen again when i cancel and reclick 'Edit'.
For some reason it was extremely hard to capture this in gif, but I got here a sample:

I have a better one :

:+1:

You might be re-rendering the form, which gives new empty props to the fields and clears your inputs. Can you please try to debug and investigate who's responsible for the potential re-render.
Thank you

:+1:

I think this is the best description I can give:

('projet' is the form store node, the others are the reference lists)

The real problem is the form should be in a loading state until the references are loaded as for other actions.

@JabX @pierr @Bernardstanislas is it still relevant ?
@pierr or @Bernardstanislas could you please qualify this issue ?

Yup.

@JabX Is it still an issue ?

Probably.

This is due to the current implementation of the form.
This won't be solved on this version, but with the use of Redux.


remove la_known

I think you might have to do the same for prevloc

I think you might have to do the same for prevloc (previous_la_known)

@smeny prevloc is still optional!


Problem with detected cards

Hey there,
I'm trying to install birdnetapp on my raspberry pi for I find the solution with Telegram and InfluxDB very interessting! I had some problems, which I was able to solve (install missing modules, load secrets.py-file), but unfortunately my very rudmientary skills in python and the alsaaudio module are not good enough for the following problem. It seems that birdnetapp is not able to find my microphone and use it. When I try to start the main.py it stops with the following error:

INFO:birdnetapp:detected cards ['Headphones', 'Microphones', 'vc4hdmi0', 'vc4hdmi1'] configuring: Device
Traceback (most recent call last):
File "/home/pi/birdnetapp/main.py", line 2, in
main()
File "/home/pi/birdnetapp/birdnetapp/init.py", line 276, in main
stream.open()
File "/home/pi/birdnetapp/birdnetapp/init.py", line 190, in open
card_i = cards.index(self.card)
ValueError: 'Device' is not in list

How can I solve this?
Thanks in advance and kind regards!

Well, I just realized that I have to specify "card" in config.py. Sorry for the inconvenience...!

Hi!
Although they are in the requirements.txt, I had to install pyalsaaudio (and previously libasound2-dev), influxdb-client and pyTelegramBotAPI individually. I didn't overlook the requirements install session, so I cannot tell if there has been an error with these modules during that install... I'm trying to finalize the setup in the coming days, then I'll know if I did the previous steps of the readme correctly :)

Alright, I made the birdnetapp run. The Microphone was inside and thus there were no birdsounds. The App ran about twenty seconds and ended with this error:

INFO:birdnetapp:detected cards ['Headphones', 'Microphones', 'vc4hdmi0', 'vc4hdmi1'] configuring: Microphones
INFO:birdnetapp:started
Traceback (most recent call last):
File "/home/pi/birdnetapp/main.py", line 2, in
main()
File "/home/pi/birdnetapp/birdnetapp/init.py", line 278, in main
runner(args, stream)
File "/home/pi/birdnetapp/birdnetapp/init.py", line 252, in runner
assert len(futures) < 10 , "Processing not keeping up with incoming data"
AssertionError: Processing not keeping up with incoming data

For your information, I have an Raspberry Pi 4B with 2gb RAM. Could this model be too slow for this app?
I then ran the app again and played a ring necked pheasant to see, if it is able to detect it. And it was. I still got the following error:

INFO:birdnetapp:detected cards ['Headphones', 'Microphones', 'vc4hdmi0', 'vc4hdmi1'] configuring: Microphones
INFO:birdnetapp:started
INFO:birdnetapp:Ring-necked Pheasant /home/pi/birdNet/Ring-necked Pheasant/22-07-23_08-32-21.mp3 conf: 0.9997
INFO:birdnetapp:Ring-necked Pheasant /home/pi/birdNet/Ring-necked Pheasant/22-07-23_08-32-24.mp3 conf: 0.9998
Traceback (most recent call last):
File "/home/pi/birdnetapp/main.py", line 2, in
main()
File "/home/pi/birdnetapp/birdnetapp/init.py", line 278, in main
runner(args, stream)
File "/home/pi/birdnetapp/birdnetapp/init.py", line 255, in runner
res = f.result()
File "/usr/lib/python3.9/concurrent/futures/_base.py", line 433, in result
return self.__get_result()
File "/usr/lib/python3.9/concurrent/futures/_base.py", line 389, in __get_result
raise self._exception
File "/usr/lib/python3.9/concurrent/futures/thread.py", line 52, in run
result = self.fn(*self.args, **self.kwargs)
File "/home/pi/birdnetapp/birdnetapp/init.py", line 222, in process
return upload_result(ts, fname, SAVEDIR, res, args.min_confidence, args.dry)
File "/home/pi/birdnetapp/birdnetapp/init.py", line 137, in upload_result
df = query_api.query_data_frame(query)
File "/usr/local/lib/python3.9/dist-packages/influxdb_client/client/query_api.py", line 126, in query_data_frame
_generator = self.query_data_frame_stream(query, org=org, data_frame_index=data_frame_index, params=params)
File "/usr/local/lib/python3.9/dist-packages/influxdb_client/client/query_api.py", line 146, in query_data_frame_stream
response = self._query_api.post_query(org=org, query=self._create_query(query, self.default_dialect, params),
File "/usr/local/lib/python3.9/dist-packages/influxdb_client/service/query_service.py", line 281, in post_query
(data) = self.post_query_with_http_info(**kwargs)  # noqa: E501
File "/usr/local/lib/python3.9/dist-packages/influxdb_client/service/query_service.py", line 307, in post_query_with_http_info
return self.api_client.call_api(
File "/usr/local/lib/python3.9/dist-packages/influxdb_client/_sync/api_client.py", line 343, in call_api
return self.__call_api(resource_path, method,
File "/usr/local/lib/python3.9/dist-packages/influxdb_client/_sync/api_client.py", line 173, in __call_api
response_data = self.request(
File "/usr/local/lib/python3.9/dist-packages/influxdb_client/_sync/api_client.py", line 388, in request
return self.rest_client.POST(url,
File "/usr/local/lib/python3.9/dist-packages/influxdb_client/_sync/rest.py", line 293, in POST
return self.request("POST", url,
File "/usr/local/lib/python3.9/dist-packages/influxdb_client/_sync/rest.py", line 243, in request
raise ApiException(http_resp=r)
influxdb_client.rest.ApiException: (404)
Reason: Not Found
HTTP response headers: HTTPHeaderDict({'Date': 'Sat, 23 Jul 2022 06:32:23 GMT','Content-Type': 'application/json; charset=utf-8', 'Transfer-Encoding': 'chunked', 'Connection': 'keep-alive', 'trace-id': 'e769ffcc187e1373', 'trace-sampled':'false', 'vary': 'Accept-Encoding', 'x-platform-error-code': 'not found', 'Strict-Transport-Security': 'max-age=15724800; includeSubDomains', 'X-Influxdb-Request-ID': '27f7e54fed205cc16c835cc96135e5c8', 'X-Influxdb-Build': 'Cloud'})
HTTP response body: b'{"code":"not found","message":"failed to initialize execute state: could not find bucket \"main\""}'
I defined the bucket in my config.py, but the app still tried to put in a "main" bucket on influxdb. I then opened a "main" bucket but I wasn't able to start the app again since the first run, because it always gives me the first error that the processing could not keeping up with incoming data. I didn't get any telegram message from this run.
What do these errors say? Can I do something to make the app run or is my 2gb system not fast enough for BirdNET-Analyzer?

Can you please post your config.py?

My config.py:

#Mic settings
#PS-Eye
RATE = 48000
CHANNELS = 2
CARD = 'Device'
CHUNK = RATE * 1 # 1 second buffer
#Files saved here
SAVEDIR = '/home/pi/birdNet'
#Number of most recent files to keep per bird
KEEP_FILES = 300
#whether store longterm results in wav or mp3
EXPORT_FORMAT = '.mp3'
#how long to wait before sending a notification
NOTIFICATION_DELAY_SECONDS = 60*5
#birdNET settings
#Note: recording length has impact on conf_thresh
#longer recording could lower CONF_THRESH to achieve the same detection rate
RECORD_SECONDS = 6
CONF_TRHRESH = 0.55
LAT=53.XXXX
LON=8.XXXX
OVERLAP = 0
#birdNet server
HOST='<IP_ADDRESS>'
PORT = 8080
#influx
ORG = "Altburgbirding"
BUCKET = "Altburgbirding"
SEEN_TIME = '14d'

I installed tflite-runtime, yes.
I run the following os with 64 bit:

Static hostname: raspberrypi
Icon name: computer
Operating System: Debian GNU/Linux 11 (bullseye)
Kernel: Linux 5.15.32-v8+
Architecture: arm64

Here is the log of the BirdNET-Analyzer server.py:

{'lat': 53.XXX, 'lon': 8.XXX, 'num_results': 3, 'overlap': 0, 'week': 29}
Analyzing /tmp/tmp77k2zp47.wav
Finished /tmp/tmp77k2zp47.wav in 0.89 seconds
####################  2022-07-23 17:18:11.059596  ####################
{'lat': 53.XXX, 'lon': 8.XXX, 'num_results': 3, 'overlap': 0, 'week': 29}
Analyzing /tmp/tmpno0ixxjt.wav
Finished /tmp/tmpno0ixxjt.wav in 0.87 seconds
####################  2022-07-23 17:18:12.017010  ####################
{'lat': 53.XXX, 'lon': 8.XXX, 'num_results': 3, 'overlap': 0, 'week': 29}
Analyzing /tmp/tmpz3v3xk58.wav
Finished /tmp/tmpz3v3xk58.wav in 0.88 seconds
####################  2022-07-23 17:18:12.971397  ####################
{'lat': 53.XXX, 'lon': 8.XXX, 'num_results': 3, 'overlap': 0, 'week': 29}
Analyzing /tmp/tmpbgk9lg_x.wav
Finished /tmp/tmpbgk9lg_x.wav in 0.88 seconds
####################  2022-07-23 17:18:13.932491  ####################
{'lat': 53.XXX, 'lon': 8.XXX, 'num_results': 3, 'overlap': 0, 'week': 29}
Analyzing /tmp/tmpl79xbhnf.wav
Finished /tmp/tmpl79xbhnf.wav in 0.89 seconds
####################  2022-07-23 17:18:14.892353  ####################
{'lat': 53.XXX, 'lon': 8.XXX, 'num_results': 3, 'overlap': 0, 'week': 29}
Analyzing /tmp/tmp2_w8u94_.wav
Finished /tmp/tmp2_w8u94_.wav in 0.88 seconds

When I start the server.py, I do it with screen, so I am able to start the birdnetapp while the server.py is still running:
screen python3 server.py
I don't know if this is the right solution, is there another way to start the server.py and afterwards the birdnetapp?
Thank you for your help!

screen is not an issue (tried it myself, works fine). Also your logs and config look good.  birdnetapp sends 6 seconds of buffer to process every 3 seconds, so any processing time of > 3 seconds is an issue. Your logs show <1 second. no problems there.
I see an issue where on cold start, birdnetAnalyzer would take closer to 60 seconds to produce the first result on my RPI-3, which causes the same error you are seeing,  As a quick solution to try, is to just restart birdnetapp without touching birdnetAnalyzer server  upon this error, and see if it continues to work normally.
As for starting the app and the server in the background, there are systemd service configs in the root of repo:
(Note: systemd service runs as root, so make sure you sudo pip3 install -r requirements.txt
sudo cp birdnet_server.service  /lib/systemd/system/
sudo systemctl daemon-reload
sudo systemctl enable birdnet_server.service
sudo systemctl start birdnet_server.service
sudo systemctl status birdnet_server.service

Also, if the problem persists, please  git pull the latest version of birdnetapp  and  post the log here when run with python3 main.py --debug

And one more thing that could be an issue: RATE and CHANNELS setting might not be compatible with your microphone/card. To test,  run
arecord -l

to see a list of devices, then in my case it returns
**** List of CAPTURE Hardware Devices ****
card 1: Device [USB Audio Device], device 0: USB Audio [USB Audio]
  Subdevices: 1/1
  Subdevice #0: subdevice #0

so I would type:
arecord -f cd --device="hw:1,0" --rate 48000 --channel 2 foo.wav

If this command records audio successfully, then it is not an issue, however, watch if it prints something like this:
Warning: rate is not accurate (requested = 48000Hz, got = 44100Hz)
         please, try the plug plugin

Then you would need to adjust config.py accordingly

Nevermind the arecord checks. I added  Mic setting configuration error checks in the code.  Just get the latest code from  birdnetapp  repo, and run python3 main.py

Nice, now it works! I get data from the detected birds into my InfluxDB. Unfortunately, I don't get a message via telegram. At first, I made a channel with my bot in it and put the channel_id into the chatid value. Now I changed it into a chatid (of a group chat where the Bot and me is in) and revoke the token of the bot but I still don't get a message from telegram. There is no error, so I really don't know why it doesn't work... I'm still happy and will run the app during the coming night! Thank you for your help!

Please note, telegram messages are delayed by about 5 minutes after the bird stops tweeting,  this allows collection of 'best'  audio sample, not just first sample, to be sent over telegram. You can control this delay in config.py (NOTIFICATION_DELAY_SECONDS) or on command line python3 main.py --notification_delay 0
Also, you wont get a telegram notification unless the bird is new.  all 'seen' birds in a time window will not trigger a telegram.  app queries influx to see if bird is already there for a specified time window. You can control this via a setting in config.py : SEEN_TIME  -> default is 2 weeks, you can set it to whatever you want.

INFO:birdnetapp:sending telegram message for Northern Cardinal
If  so, to debug telegram specifically, I just pushed some test-code. Please  get the latest code for birdnetapp, and run the following command from birdnetapp folder:
PYTHONPATH=. python3 tests/test_telegram.py

If all goes well, you should get a telegram message about a Cardinal.
P.S.  Assuming you cloned the repo with lfs , if not, do git lfs pull, as it needs example/cardinal.wav to be about 1.3M

Yepp, you were right. Telegram messages started a few minutes later and keep coming since. But it doesn't matter if the species was already detected today.. I have no idea why, the config.py has the 14d value in it (see above). Do you have any ideas?
Except for this issue, I'm very happy with the results! Thank you very much for helping me setting it up!

hm, I am unable to reproduce your telegram messages spam.  It works fine with my setup.  One thing to try is see what this query returns in your influx instance . It should contain already seen birds.
                import "influxdata/influxdb/schema"
                schema.fieldKeys(
                    bucket: "main",
                    predicate: (r) => r["_measurement"] == "birdnet",
                    start: -14d,
                )

Well, I didn't overthing the influxdb query process 'cause I am quite happy with the current state to get telegram messages and all my data to my InfluxDB Cloud instance... There's no local influxdb yet, this is why the query doesn't work, right? Sorry for that! I'm going to try it in the next days! Thanks again for your help! :)

It works for me with Influx2 cloud instance from (https://cloud2.influxdata.com/signup).  There should be no difference between cloud and local setup.
P.S. One thing I changed in the latest code is move  the bucket name and influx org into the secrets.py file, since those parameters seem to be all related.

I didn't have the time to update my pi the past week. Just to let you know, now that I have, it all works out!


Issues with terraform apply after release of terraform-provider-zpa_v3.33.5

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Terraform Version
1.5.7
Affected Resource(s)
zpa_application_segment
Debug Output
Error: Plugin did not respond
│
│   with zpa_application_segment.NAME_HERE,
│   on zpa_application_segment.tf line 7350, in resource "zpa_application_segment" NAME_HERE:
│ 7350: resource "zpa_application_segment" "NAME_HERE" {
│
│ The plugin encountered an error, and failed to respond to the
│ plugin.(*GRPCProvider).ApplyResourceChange call. The plugin logs may
│ contain more details.
Panic Output
Stack trace from the terraform-provider-zpa_v3.33.5 plugin:
panic: interface conversion: interface {} is []interface {}, not *schema.Set
goroutine 53 [running]:
github.com/zscaler/terraform-provider-zpa/v3/zpa.expandApplicationSegmentRequest(_, , {, _})
github.com/zscaler/terraform-provider-zpa/v3/zpa/resource_zpa_application_segment.go:566 +0x1157
github.com/zscaler/terraform-provider-zpa/v3/zpa.resourceApplicationSegmentUpdate(0xc0005d0d00, {0xc7e540, 0xc000564480})
github.com/zscaler/terraform-provider-zpa/v3/zpa/resource_zpa_application_segment.go:360 +0x214
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*Resource).update(0xf3e9f0?, {0xf3e9f0?, 0xc00010dd40?}, 0xd?, {0xc7e540?, 0xc000564480?})
github.com/hashicorp/terraform-plugin-sdk/v2@v2.34.0/helper/schema/resource.go:828 +0x15f
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*Resource).Apply(0xc0001775e0, {0xf3e9f0, 0xc00010dd40}, 0xc0000cc8f0, 0xc0005b6f80, {0xc7e540, 0xc000564480})
github.com/hashicorp/terraform-plugin-sdk/v2@v2.34.0/helper/schema/resource.go:947 +0x83a
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*GRPCProviderServer).ApplyResourceChange(0xc00038e5d0, {0xf3e9f0?, 0xc00010dc80?}, 0xc000620410)
github.com/hashicorp/terraform-plugin-sdk/v2@v2.34.0/helper/schema/grpc_provider.go:1153 +0xd5c
github.com/hashicorp/terraform-plugin-go/tfprotov5/tf5server.(*server).ApplyResourceChange(0xc0002574a0, {0xf3e9f0?, 0xc00066db00?}, 0xc0006675e0)
github.com/hashicorp/terraform-plugin-go@v0.23.0/tfprotov5/tf5server/server.go:865 +0x3d0
github.com/hashicorp/terraform-plugin-go/tfprotov5/internal/tfplugin5._Provider_ApplyResourceChange_Handler({0xdbd6a0, 0xc0002574a0}, {0xf3e9f0, 0xc00066db00}, 0xc0005b6500, 0x0)
github.com/hashicorp/terraform-plugin-go@v0.23.0/tfprotov5/internal/tfplugin5/tfplugin5_grpc.pb.go:518 +0x1a6
google.golang.org/grpc.(*Server).processUnaryRPC(0xc0001a3000, {0xf3e9f0, 0xc00066da70}, {0xf43248, 0xc0002dc180}, 0xc0006c5440, 0xc000405860, 0x14e91f8, 0x0)
google.golang.org/grpc@v1.63.2/server.go:1369 +0xdf8
google.golang.org/grpc.(*Server).handleStream(0xc0001a3000, {0xf43248, 0xc0002dc180}, 0xc0006c5440)
google.golang.org/grpc@v1.63.2/server.go:1780 +0xe8b
google.golang.org/grpc.(*Server).serveStreams.func2.1()
google.golang.org/grpc@v1.63.2/server.go:1019 +0x8b
created by google.golang.org/grpc.(*Server).serveStreams.func2 in goroutine 27
google.golang.org/grpc@v1.63.2/server.go:1030 +0x125
Error: The terraform-provider-zpa_v3.33.5 plugin crashed!
This is always indicative of a bug within the plugin. It would be immensely
helpful if you could report the crash with the plugin's maintainers so that it
can be fixed. The output above should help diagnose the issue.
Important Factoids
We are using Spacelift to run terraform. Since terraform-provider-zpa_v3.33.5 was released today we started to have this issue. We locked to the version 3.33.4 and it's back to work.

@yaroslav-skvortsov Can you please indicate which port configuration format are you currently using?
tcp_port_ranges   = [ "80", "443" ]

or
  tcp_port_range = [
    {
    from = "1"
    to   = "52"
  }
]

@yaroslav-skvortsov Can you please indicate which port configuration format are you currently using?
Hi! In all new segments we are using:
tcp_port_range = [
{
from = "443"
to   = "443"
}
]

But in some segments (old segments) we have like this:
tcp_port_range = [
{
from = "443"
to   = "443"
},
{
from = "22"
to   = "22"
},
{
from = "80"
to   = "80"
}
]
tcp_port_ranges = ["443", "443", "22", "22", "80", "80"]
I guess that's how it was retrieved with zscaler terraformer tool (last example). I don't know if this matters, but we have all zpa segments in 1 terraform file (so 2 types are mixed), also all the policies in one another tf file scim groups in different tf file etc...
ק

Got. We know what the problem is.
Please give us a couple of hours and we'll put a new fix in place via v3.33.6 version.
Thanks for your patience.
Zscaler DevRel.

Thank you so much!

@yaroslav-skvortsov
We've released a minor fix via version v3.33.6.
Please test and let us know.
Zscaler DevRel

Resolved in v.3.36.6


1.1.0 scaling behavior issue

Hello! First of all, thanks for your great package.
I have an issue after upgrading to 1.1.0.
I use onScaleStart, onScaleUpdate, onScaleEnd callbacks, and it seems like onScaleUpdate does not work properly (and sometimes I get the error, stack trace below). The other thing is, onScaleEnd is not called at all.

Stack trace

The following RangeError was thrown while dispatching a pointer event:
RangeError (index): Invalid value: Only valid value is 0: 1
When the exception was thrown, this was the stack:
#0      List.[] (dart:core-patch/growable_array.dart:281:36)
#1      _XGestureDetectorState.onPointerMove (package:gesture_x_detector/gesture_x_detector.dart:181:65)
#2      RenderPointerListener.handleEvent (package:flutter/src/rendering/proxy_box.dart:2892:29)
#3      GestureBinding.dispatchEvent (package:flutter/src/gestures/binding.dart:419:22)
#4      RendererBinding.dispatchEvent (package:flutter/src/rendering/binding.dart:322:11)
#5      GestureBinding._handlePointerEventImmediately (package:flutter/src/gestures/binding.dart:374:7)
#6      GestureBinding.handlePointerEvent (package:flutter/src/gestures/binding.dart:338:5)
#7      GestureBinding._flushPointerEventQueue (package:flutter/src/gestures/binding.dart:296:7)
#8      GestureBinding._handlePointerDataPacket (package:flutter/src/gestures/binding.dart:279:7)
#12     _invoke1 (dart:ui/hooks.dart:170:10)
#13     PlatformDispatcher._dispatchPointerDataPacket (dart:ui/platform_dispatcher.dart:331:7)
#14     _dispatchPointerDataPacket (dart:ui/hooks.dart:94:31)
(elided 3 frames from dart:async)
Event: PointerMoveEvent#b33dd(position: Offset(185.1, 510.2))
  position: Offset(185.1, 510.2)
Target: RenderPointerListener#5c4b8 relayoutBoundary=up11
  needs compositing
  parentData: <none> (can use size)
  constraints: BoxConstraints(0.0<=w<=392.7, 0.0<=h<=600.4)
  size: Size(392.7, 600.4)
  behavior: deferToChild
  listeners: down, move, up, cancel

Logs

[✓] Flutter (Channel stable, 2.10.5, on macOS 11.5.2 20G95 darwin-arm, locale en)
  • Flutter version 2.10.5 at /Users/papertigify/tools/flutter
  • Upstream repository https://github.com/flutter/flutter.git
  • Framework revision 5464c5bac7 (10 days ago), 2022-04-18 09:55:37 -0700
  • Engine revision 57d3bac3dd
  • Dart version 2.16.2
  • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
  • Android SDK at /Users/papertigify/Library/Android/sdk
  • Platform android-31, build-tools 31.0.0
  • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
  • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
  • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 12.5.1)
  • Xcode at /Applications/Xcode.app/Contents/Developer
  ! Flutter recommends a minimum Xcode version of 13.
    Download the latest version or update via the Mac App Store.
  • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
  • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
  • Android Studio at /Applications/Android Studio.app/Contents
  • Flutter plugin can be installed from:
    🔨 https://plugins.jetbrains.com/plugin/9212-flutter
  • Dart plugin can be installed from:
    🔨 https://plugins.jetbrains.com/plugin/6351-dart
  • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] Connected device (2 available)
  • iPhone 11 Pro (mobile) • ios • com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
  • Chrome (web)• chrome • web-javascript • Google Chrome 100.0.4896.127

[✓] HTTP Host Availability
  • All required HTTP hosts are available

! Doctor found issues in 1 category.

New version released: https://pub.dev/packages/gesture_x_detector/versions/1.1.1
Please let me know if it solved this issue.

Solved at version 1.1.1


Support more languages

From a discussion here (https://discuss.atom.io/t/spelling-in-english-only/11819) I guess that node-spellchecker does not support other languages beside en_US
Please add more languages.

:+1:

Could webFrame.setSpellCheckProvider in atom-shell be used for that?

:+1:

It seems @paulcbetts changes will solve this, see https://github.com/atom/node-spellchecker/pull/6 and https://github.com/atom/node-spellchecker/pull/7
Unless I misunderstood the PRs. I wonder if it also works on linux :worried:

@despairblue It will work on Linux, but you've got to manage shipping the dictionaries yourself (or more likely, track down where on the system they're storing the Hunspell dictionaries)

@paulcbetts awesome, that's still way better the status quo :clap:

hunspell -D shows that I use the german dictionary, but in Atom I still have the english dictionary only.

A support for other languages (e.g. German) would be great

This is fixed in this package, now Atom needs to use the multi-language features that node-spellchecker exposes

So is this being worked on in Atom to integrate it? I'm not sure I get what needs to be done for this to be usable. Thank you.

:thumbsup:


SignalR-Client-Cpp in release mode asks for cpprest140d_2_6.dll

I am testing with Visual Studio 2015 and the nuget package for SignalR-Client-Cpp.
Installing the package also installs Cpp Rest as expected. Running my test application in debug mode works fine. However, when I switch to release mode I get an error for a missing Cpp Rest debug DLL (cpprest140d_2_6.dll). Is this a problem of the nuget package?

Are you installing this package http://www.nuget.org/packages/Microsoft.AspNet.SignalR.Client.Cpp.v120.WinDesktop/1.0.0-alpha1? I think the issue might be that this package is meant to be used with the compiler that was included in VS2013 and not VS2015. I am actually a bit surprised things worked when compiling the Debug version with VS2015 - last time I tried it did not work.
I did make necessary changes and the client now compiles with VS2015 however the package has not been shipped yet. You can however create the package on your machine - from VS 2015 command prompt run msbuild /t:CreatePackage /p:Configuration=Release /p:Platform=x64 (Configuration and Platform are optional and default to Debug and Win32 respectively). Once you have a package you can copy it to local folder and add this folder as a NuGet source. This way you will be able to install the package easily to your project.

I was using the package from the https://www.myget.org/F/aspnetvnext/ feed:
Microsoft.AspNet.SignalR.Client.Cpp.v140.WinDesktop
OK, I will try to build it myself. Thanks.

Sorry, I think I don't quite get it. I downloaded the entire source tree to a local directory, started the VS2015 command prompt, changed to the solution directory and ran:
c:\MyDir>msbuild signalrclient.sln /t:signalrclientdll /p:Configuration=Release /p:Platform=x64
The build works fine but how is the nuget package supposed to be created and where? Is this an extra step? Thanks Nicolas

If you are using the Microsoft.AspNet.SignalR.Client.Cpp.v140.WinDesktop package then you don't need to build your own package - this is the version for VS2015 (v140 indicates toolset used in VS2015) but it was not yet published to NuGet.  and there must be something wrong with the targets file. I will look into this.
I just noticed that the build command I provided was missing the project file which was probably the reason why you couldn't make it work. It should have been msbuild build\build.msbuild /t:CreatePackage /p:Configuration=Release /p:Platform=x64 (you run it from the project root folder). As I said - you don't need to build the package on your own since you are using the correct one and the bug appears to be somewhere in one of the targets file.

Just out of curiosity I built the package myself using your command and created a local nuget feed. With this package I don't experience the problem. I did not change anything in the targets file!

I have the same issue, running dependency walker on the package currently on nuget.org (VS2013), the release dll appears to depend on the debug version of the rest dll.

@woodmeister123 - I just checked and I don't see this. For the package on NuGet Debug version depend on Debug version of Casablanca but Release versions depend on Release version of casablanca:
PS C:\Source\SignalR> dumpbin /dependents "C:\Temp\SignalRCpp\microsoft.aspnet.signalr.client.cpp.v120.windesktop.1.0.0-alpha1.nupkg\build\native\dll\x64\v120\Debug\signalrclient.dll"
Microsoft (R) COFF/PE Dumper Version 14.00.23506.0
Copyright (C) Microsoft Corporation.  All rights reserved.

Dump of file C:\Temp\SignalRCpp\microsoft.aspnet.signalr.client.cpp.v120.windesktop.1.0.0-alpha1.nupkg\build\native\dll\x64\v120\Debug\signalrclient.dll

File Type: DLL

  Image has the following dependencies:

    cpprest120d_2_5.dll
    KERNEL32.dll
    MSVCP120D.dll
    MSVCR120D.dll

  Summary

       10000 .data
       16000 .pdata
       6F000 .rdata
        3000 .reloc
        1000 .rsrc
       EF000 .text
PS C:\Source\SignalR> dumpbin /dependents "C:\Temp\SignalRCpp\microsoft.aspnet.signalr.client.cpp.v120.windesktop.1.0.0-alpha1.nupkg\build\native\dll\Win32\v120\Debug\signalrclient.dll"
Microsoft (R) COFF/PE Dumper Version 14.00.23506.0
Copyright (C) Microsoft Corporation.  All rights reserved.

Dump of file C:\Temp\SignalRCpp\microsoft.aspnet.signalr.client.cpp.v120.windesktop.1.0.0-alpha1.nupkg\build\native\dll\Win32\v120\Debug\signalrclient.dll

File Type: DLL

  Image has the following dependencies:

    cpprest120d_2_5.dll
    KERNEL32.dll
    MSVCP120D.dll
    MSVCR120D.dll

  Summary

        F000 .data
       24000 .rdata
        A000 .reloc
        1000 .rsrc
       BB000 .text

PS C:\Source\SignalR> dumpbin /dependents "C:\Temp\SignalRCpp\microsoft.aspnet.signalr.client.cpp.v120.windesktop.1.0.0-alpha1.nupkg\build\native\dll\x64\v120\Release\signalrclient.dll"
Microsoft (R) COFF/PE Dumper Version 14.00.23506.0
Copyright (C) Microsoft Corporation.  All rights reserved.

Dump of file C:\Temp\SignalRCpp\microsoft.aspnet.signalr.client.cpp.v120.windesktop.1.0.0-alpha1.nupkg\build\native\dll\x64\v120\Release\signalrclient.dll

File Type: DLL

  Image has the following dependencies:

    cpprest120_2_5.dll
    KERNEL32.dll
    MSVCP120.dll
    MSVCR120.dll

  Summary

       10000 .data
        7000 .pdata
       34000 .rdata
        2000 .reloc
        1000 .rsrc
       58000 .text
PS C:\Source\SignalR> dumpbin /dependents "C:\Temp\SignalRCpp\microsoft.aspnet.signalr.client.cpp.v120.windesktop.1.0.0-alpha1.nupkg\build\native\dll\Win32\v120\Release\signalrclient.dll"
Microsoft (R) COFF/PE Dumper Version 14.00.23506.0
Copyright (C) Microsoft Corporation.  All rights reserved.

Dump of file C:\Temp\SignalRCpp\microsoft.aspnet.signalr.client.cpp.v120.windesktop.1.0.0-alpha1.nupkg\build\native\dll\Win32\v120\Release\signalrclient.dll

File Type: DLL

  Image has the following dependencies:

    cpprest120_2_5.dll
    KERNEL32.dll
    MSVCP120.dll
    MSVCR120.dll

  Summary

        F000 .data
       18000 .rdata
        7000 .reloc
        1000 .rsrc
       54000 .text

This is fixed. The packages on nightly feed have now correct dependencies.


Get a circular image from cropping

Is there a way to output a circular image from cropperjs? I used the code provided in issue #545 but it doesnot work. The circle does appear but the output is a rectangle. Thanks!

@fengyuanchen The above link doesn't seem to work anymore. Do you know where I might find it?


add M1 Edition

please add M1 Edition ☺

@catalinaLi I have it on my m1 mac running on monterey no issues. are you sure you downloaded latest one?

just for update: but on macOs Ventura v. 13.1 still working fine

It works fine but the build is running as Intel through Rosetta.

请支持M系列芯片支持，谢谢

Anyone knows how to use it on M1. It dont work on my M1 Ventura v. 13.5 using version 12.1


Issue with Html-Pdf for node js

I've recently encountered a bug that is causing an error when I use a font awesome icon in my html that is being used to convert to pdf.   I haven't been able to find anything when searching for this issue so not sure if this is new or not enough people have experienced this.  Below are the error codes:
html-pdf: Evaluation - Font Awesome Kit: Error
Stack: at https://kit.fontawesome.com/b46fe6b39d.js
at ChildProcess.respond (frontend/node_modules/html-pdf/lib/pdf.js:121:31)
at ChildProcess.emit (events.js:315:20)
at Process.ChildProcess._handle.onexit (internal/child_process.js:277:12)
I hope that a resolution can be found for this as I would like to use the icons I was using.

Hello, i'm having the same issue.
@jschaeppi  did you find any solution to this issue?

How did you go about this error @davideavanzi ?


Mock API on Dashboard Cypress Test

Update the main dashboard tests in frontend/cypress/e2e/smoke-test/dashboard-page.cy.ts
The test currently is testing the favourite and unfavourite functionality for the following pages:

Register A-class seedlot
My seedlot
Seedlot Dashboard

Tasks

Remove the Seedlot Dashboard page test cases, this is to reduce the amount of the tests and mocked api needed
Update the description of tests so they are more concise (where they are needed), for example:

should be able to favourite the a class seedlot page

Change that to "should be able to favourite the Register A-class seedlot page."

Use cy.intercept() to intercept all calls related to the favourite activity endpoints

Intercept them within each it() as needed, basically we should not perform any CRUD through the actual backend
the results of CRUD actions should be reflected through the data returned from the mocked endpoint

closed due to username/password leak, also not sure whether cy.reload() is needed, I will take it over and have a look.


Stop property-sort-order and no-value-vendor-prefix from throwing with css variables

What do the changes you have made achieve?

Prevent the property-sort-order from throwing errors when css variables are present

Are there any new warning messages?

No

Have you written tests?

I haven't added a new test, but I've added css variable declarations to the test file to show it throwing before the fix and not throwing after.

Have you included relevant documentation

N/A

Which issues does this resolve?
Fixes #1224
DCO 1.1 Signed-off-by: Justin Anastos<EMAIL_ADDRESS>

Coverage increased (+0.003%) to 97.533% when pulling abbbbe19d2e324723a94606c42858582126328fa on justinanastos:fix/1224/property-sort-order-throwing-css-variables into be43c0bd231774873668b0560a8a893d89ef10d0 on sasstools:develop.

Coverage increased (+0.003%) to 97.533% when pulling abbbbe19d2e324723a94606c42858582126328fa on justinanastos:fix/1224/property-sort-order-throwing-css-variables into be43c0bd231774873668b0560a8a893d89ef10d0 on sasstools:develop.

I would also like this merged :)

@justinanastos there is a merge conflict.
@DanPurdy is there additional effort or comments to be made on this PR beyond the merge conflict?

Is there anything holding this up from being merged other than the conflict?


Use private IP, not public IP?

Is it possible to extract the private IP, instead of public IP?  I'm using TF + TF-Inventory, but accessing the servers via a bastion host.  It's been a very useful tool so far, thanks!

I don't know anything about Go, but a quick poke around, and removing the 'public_ip' option in 'resources.go' did the trick :-)

@completerevolution Editing the source code is not the best option for your need.
What you want to do is:
export TF_KEY_NAME=private_ip

and then the inventory will output private ips instead of public ones.

I tried:
export TF_KEY_NAME=private_ip
and it didn't work for me.  I got a blank list.  However, as of version 0.7.7, if I did this:
export TF_KEY_NAME=ipv4_address_private
It worked fine.


[⛔] NT-1050 Adding ErroredBackingViewHolder

📲 What
Adding ErroredBackingViewHolder and ViewModel
🤔 Why
So users can view their errored backings in the Activity feed  🔜
🛠 How
XML

Added layout for ErroredBacking item_errored_backing.xml
Added icon ic_icon_alert.xml (it still needs a11y copy)

ErroredBackingViewHolder

ErroredBackingViewHolderViewModel

Added inputs erroredBacking and manageButtonClicked
Added outputs projectFinalCollectionDate, projectName, and notifyDelegateToStartFixPaymentMethod

Tests

Added ErroredBackingFactory helper

👀 See

📋 QA
How's it look? 👀 It'll be added to the Activity feed in NT-1051
Story 📖
NT-1050

@colleenmacd Updated the screenshot!


Remove obsolete packages from mvnrepository?

On mvn repository https://mvnrepository.com/artifact/io.netty there are some outdated packages
io.netty » netty
Netty/All In One
Last Release on Dec 3, 2012
And for the newer packages the "last release date" shows the last netty 5 alpha:
io.netty » netty-all
Netty/All In One
Last Release on Mar 3, 2015
March 2015 is the last alpha of the discontinued 5.0.0 branch.
Maybe removing that would help users find the right netty version for them.

IMHO removing jars is a bad practice as it can break builds

Even alpha jars?

Yes... everything that is not SNAPSHOT may break stuff

everything that is not SNAPSHOT may break stuff

:100:


CASSANDRA-17241: Improve audit logging documentation for production

After using audit logging in production, it turns out that documentation is lacking some important information.
Namely:

roll_cycle can be overriden by Chronicle Queue
max_log_size is ignored if archive_command option is set
archive_command prevents Chronicle Queue Shrink Manager from working

Rebased from trunk + formatting in a separate commit

closing as this work will be included under umbrella PR https://github.com/apache/cassandra/pull/2062


Assistants Beta V2

OpenAI has released a new version of the Assistants, which significantly changes the API surface.
Subtasks:

#267
#284

I wasn't planning on tracking these individually.

I wonder if there is some way we could consume the OpenAI API spec so that when new properties get released it can be partially automated?
https://github.com/openai/openai-openapi/blob/master/openapi.yaml

I already have something locally that I use, but it's not good enough for all the edgecases.

For anyone wondering how to patch this package to support Assistants Beta V2, here's what I did

Have the author's package in the asset folder so it's editable.
To patch the header without changing the author's package, there's a small reflection trick that can be done

var dictionary = (Dictionary<string, string>)client.DefaultRequestHeaders;
dictionary["OpenAI-Beta"] = "assistants=v2";
client.GetType().GetProperty(nameof(client.DefaultRequestHeaders)).SetValue(client, dictionary);

3- Now you have to use the tool_choice field otherwise your assistant will just ignore the knowledge files you provide. To do this,
modify the CreateRunAsync method under ThreadExtensions.cs to
public static async Task<RunResponse> CreateRunAsync(this ThreadResponse thread, string assistantId, object toolChoice = null, CancellationToken cancellationToken = default)
            => await thread.Client.ThreadsEndpoint.CreateRunAsync(thread, new CreateRunRequest(assistantId, toolChoice), cancellationToken);

And CreateRunRequest.cs class, modify the bigger constructor to be
[Preserve]
        public CreateRunRequest(string assistantId, object toolChoice = null, string model = null, string instructions = null, IEnumerable<Tool> tools = null, IReadOnlyDictionary<string, string> metadata = null, float? temperature = null) {
            AssistantId = assistantId;
            ToolChoice = toolChoice;
            Model = model;
            Instructions = instructions;
            Tools = tools?.ToList();
            Metadata = metadata;
            Temperature = temperature;
        }

and add the property
[Preserve]
        [JsonProperty("tool_choice")]
        public object ToolChoice { get; }

I chose the "object" type because ToolChoice can be a string, or a dictionary, and JSONNet just takes care of the rest.
Hacky but works.

@StephenHodgson I set up a test project and it's having issues, would changes to the Assistant be the reason I'm getting this error:
System.ObjectDisposedException: 'Cannot access a disposed object.
Object name: 'System.Net.Http.HttpClient'.'

when it gets to await assistant.CreateThreadAndRunAsync(threadRequest); in this:
  var assistantName = _configuration["OpenAIAssistant"];
        var assistant = (await ListAssistants()).FirstOrDefault(x => x != null && x.Name == assistantName);
        if (assistant == null)
        {
            assistant = await CreateAssistant();
        }
        else if (assistant.Instructions != _instructions)
        {
            var modifyRequest = new CreateAssistantRequest(
                name: assistant.Name,
                model: assistant.Model,
                instructions: _instructions
            );
            assistant = await assistant.ModifyAsync(modifyRequest);
        }

        if (assistant != null)
        {
            using var api = new OpenAIClient();
            var messages = new List<Message>() { "hello" };
            var threadRequest = new CreateThreadRequest(messages);
            await assistant.CreateThreadAndRunAsync(threadRequest);
        }

Or should I raise this as a new issue?

Hi @StephenHodgson
Thank you for the great library!
OpenAI has released gpt-4o model and it's not compatible with Assistants v1 API. Throwing error:
{     "message": "The requested model 'gpt-4o' cannot be used with the Assistants API in v1. Follow the migration guide to upgrade to v2: https://platform.openai.com/docs/assistants/migration.",     "type": "invalid_request_error",     "param": "model",     "code": "unsupported_model"   }
Is it possible to generate a library with Assistants v2?

Sorry I haven't had time to get around to it.
I'll see what I can do this weekend.

I'm also happy to help but I might need a quick kick off with you on the best way to approach some of these changes. I'll ping you on Discord

Most of the work is done. Just finishing up streaming support and fixing a few bugs with tool cache

Большая часть работы сделана. Просто завершаю поддержку потоковой передачи и исправляю несколько ошибок с кешем инструментов.

Hello! When can we expect GPT-4o support? Considering that most of the work has already been completed.


Review changes: textflow; fix link to conda page.

Mostly follows suggestions from @tkknight

I've made some comments, but I think these changes here are mostly a matter of personal preference and not worth getting to hung up over. So rather than also adding my own opinion of how I like things to be laid out... I'm just gonna merge this.


Refresh docs for beta19

Refresh docs for beta19

My branch is a mess. Closing PR. Will fix and resubmit.


Fix issues with [Docker Pulls Count] image display in the documentation

Hi @yimogit thanks for the huge effort but I have bad news. Those files are auto-generated by crowdin
https://github.com/verdaccio/website#translation
if you think a modification is needed it, you need to work only in this directory https://github.com/verdaccio/website/tree/master/docs

I just created this https://github.com/verdaccio/website/blob/master/CONTRIBUTING.md to avoid this happen again.

Thank you, the problem will be solved after publishing the website.


Utility for monitoring mayastor IOPS using influxdb and grafana.

This is the grafana work I did today for the demo. The code was taken from the old mayastor repo and it was polished to catch up with the changes in mayastor. One change though is that all the source code for the monitoring is kept aside from the rest of the mayastor code in monitor subdirectory (source code, docker file, k8s deployment file, configuration files, ...). It is not part of the product. Though it is generally useful when we want to present the mayastor to others.

I'll not comment on the code itself, other than to say a quick scan didn't show any obvious problems.
However, going forward, it is easier to write or configure an agent for an aggregator, like telegraf, and have that handle forwarding the metrics to influxdb, prometheus, or whatever. If the metrics themselves are easily grokked from an API call, then it could just be a simple matter of configuring
the inputs.httpjson agent for telegraf.

@richardelling , Thanks for the suggestions, telegraf seems very useful for collection not just CSI volume metrics, will keep that on the list to look into for sure!

@richardelling : thanks for your comment! That looks like exactly what we need.
I will close this PR and eventually submit a new one with telegraf. I will do changes to our HTTP server to make the format of json data telegraf-friendly and create deployment yaml for telegraf. IMHO that's what mayastor should provide. Users are then free to choose what database and what graphing tool they want to use. That said it would be nice if we had something basic & simple in the repo for displaying the stats. Not sure if influxdb + grafana is the best choice. It's open for comments.

@jkryl there are several options to implement metrics, perhaps we can open a discussion on slack (NB, I don't see you in the contributor's channel) or email... issues don't work well for these discussions.


Improve the Code Organization Documentation

https://github.com/Microsoft/vscode/wiki/Code-Organization

[x] Injection, services, and service identifier
[ ] editor/contrib and workbench/parts


Missing Microsoft.Testing.Extensions.CodeCoverage 17.13.1

Describe the bug
MSTest.Sdk/3.6.4 references Microsoft.Testing.Extensions.CodeCoverage 17.13.1 which is not on NuGet
Steps To Reproduce

Create a new project with Project Sdk=MSTest.Sdk/3.6.4
Build

Expected behavior
Builds
Actual behavior
error NU1102:
Unable to find package Microsoft.Testing.Extensions.CodeCoverage with version (>= 17.13.1)
- Found 12 version(s) in nuget.org [ Nearest version: 17.13.0 ]
- Found 0 version(s) in Microsoft Visual Studio Offline Packages

Hi @sehra,
Thanks for raising this issue. It's my bad, I didn't pay attention the Code Coverage extension wasn't published to NuGet, we will fix that right now.

Fixed.

Microsoft.CodeCoverage.MSBuild is also still missing.

@martincostello it's good now https://www.nuget.org/packages/Microsoft.CodeCoverage.MSBuild/17.13.1


Cleanup top level tarball folders

There are too many top level folders in the tarball.  The # and names hurt the approachability of the tarball.
/ArcadeOverrides - should be removed - https://github.com/dotnet/source-build/issues/2295
/scripts - Proposed removing in https://github.com/dotnet/installer/pull/13843
/keys - Move under eng.
/Tools vs tools-local - Why do we need two tools folders?  Can these be combined and moved under eng?
/repos - The name of this is a bit confusing because it could lead the consumer to think this is where the repo source lives.  This should have a more meaningful name.  I question if this should be a top level folder.  Would it make sense to move under eng?
/packages - This is confusing because it doesn't contain any of the resulting packages rather it is a collection of packages used by the build.  This should have a more descriptive name.  Would it make sense to move this under artifacts?

/repos, I think, is the odd one out. That's one of the few things I sometimes end up modifying as part of distro builds. There's some value to it being a top-level visible directory too.
What do you think about moving the contents of repos into /src?

/repos, I think, is the odd one out. That's one of the few things I sometimes end up modifying as part of distro builds. There's some value to it being a top-level visible directory too.
What do you think about moving the contents of repos into /src?

Thanks for the input.  I can live with it at the root of the repo if it had a more descriptive name that helped differentiate it between /src.  repo-projects  or even projects would be more meaningful to me.  Putting it in /src would make it the odd one out there IMO.

Originally, Tools was the SDK, Arcade, anything else we needed to bootstrap and tools-local were source-build-specific things built in the source-build.  I don't know that separating them has huge value now that most consumers will supply their own bootstrap SDK and previously-source-built packages.  I could also see combining repos and tools-local into a local-src or similar to separate source-build itself vs contributing-repo-sources.  Or /src/source-build (/src/orchestration?  we used that in 1.x for a bit) in addition to /src/sdk etc.
I guess really the distinction to me seems to be:

repo sources
source-build sources itself
stuff to bootstrap source-build

So spitballing a completely new organization could be:

All the /src/repos/ - moved from /src/ so we can retain the different dir.props and dir.targets for contributing repos vs our code
/src/source-build/tools - tools-local
/src/source-build/build - /repos
src/source-build/... - anything else we end up needing
/prereqs - git-info, toolset, keys, Tools.  prep.sh puts the bootstrap SDK, PSB, and SBRP tarballs here.  we look for smoke-test-prereqs here.
/eng - as required
/test - remains as is
/artifacts - should go away for the shipped tarball (unless empty for convenience)
/packages - still used for extracted package tarballs but not shipped unless empty for convenience

[Triage] Discussing this in triage, the following proposal was formulated:
/ artifacts - contains all build output
/ eng
/ common - common arcade
/ tools - source build tasks and tools - this is the existing tools-local directory
/ prereqs - requirements necessary to build the tarball
/ git-info
/ keys
/ packages
/ tools - source-built tools - this is the existing Tools directory
/ src - repo source
/ test - scenario tests
The one folder that still has varying opinions is the current /repos.  Everyone seems to agree it should be renamed to better indicate is contains the repo projects. There are three options on the table.
/ eng / repo-projects
/ repo-projects
/ src / repo-projects

What's the purpose of the folder?

What's the purpose of the folder?

It contains a project for each product repo that is part of source-build.  The projects encapsulate the logic to invoke the product build.  Most of the logic is defined shared props/targets but some repos require varying degrees of specialization.

Would you expect this to go away in the "one true source build land"?

Discussing this with @crummel, we have come up with the following proposal.  One of the primary changes it includes is moving all directories that are modified by the build into artifacts.
/ artifacts - contains all build output
  / packages - existing top-level packages folder; archiveArtrifacts.txt is replaced with content from eng/versions.props
  / source-build-sdks - this existing Tools directory
/ eng
   / common - common arcade
   / tools - source build tasks and tools - this is the existing tools-local directory
/ prereqs - requirements necessary to build the tarball
   / git-info - existing top-level git-info folder
   / keys - existing top-level keys folder
/ repo-projects - renames repos
/ src - repo source
/ test - scenario tests

What is the recommended process to follow in order to make the suggested change to the git-info folder?

Change https://github.com/dotnet/arcade-services/blob/4b97ba07fa0b7cbda645aa94f32099c4e3ef0710/src/Microsoft.DotNet.Darc/src/DarcLib/VirtualMonoRepo/VmrInfo.cs#L54 to prereqs/git-info
Bump DarcLib in installer
Let VMR sync
Remove /git-info manually in the VMR

@premun - the git-info renaming will also require changes to the source-build infra.  Those will have to be made in the same PR where the DarcLib flows in.  I will prepare those changes and coordinate w/you.

Alright. I don't expect problems as essentially the git-info files are just a dump during the VMR sync, a simple XML projection of the source-manifest.json that is not ever read again by the VMR tooling.

We should also probably move the the directories in installer.

@premun - I agree.  FYI there is a readme.md at src/SourceBuild that should be preserved at src/VirtualMonoRepo.  The readme needs some updating.  That can be done with a follow-up commit.  Doing this next week sounds great while there is a lull in activity as many will be on holiday break.

Yeah, I imagine we can put there the instructions around vmr-sync.sh

@MichaelSimons feel free to re-assign this to me and I will take care of that

@MichaelSimons feel free to re-assign this to me and I will take care of that

I logged a separate issue for this because it is not really part of this issue which is just about the root tarball/VMR folder.
https://github.com/dotnet/source-build/issues/3184

@MichaelSimons should we remove https://github.com/dotnet/dotnet/blob/main/packages/archive/archiveArtifacts.txt?

@MichaelSimons should we remove https://github.com/dotnet/dotnet/blob/main/packages/archive/archiveArtifacts.txt?

I will do that once https://github.com/dotnet/installer/pull/15140 is completed.  The entire packages directory will be removed.

Closing as completed.  Thanks @premun for your help on this.  Overall this was a nice improvement.


Builds with ament_python broken with sphinx >= 7.0.0

The following way of obtaining a list of available commands from setup.py is broken on sphinx v7.0.0 and later.
https://github.com/colcon/colcon-core/blob/99c03262ad7edd4af5954c424c9248e0c4fe7307/colcon_core/task/python/build.py#L170-L185
This occurs because the setuptools integration is deprecated:
~~https://github.com/sphinx-doc/sphinx/issues/9595#issuecomment-1135018951~~
The end result is that ament_python continues as normal, but installs the package as an egg without the egg-info.
https://github.com/colcon/colcon-core/blob/99c03262ad7edd4af5954c424c9248e0c4fe7307/colcon_core/task/python/build.py#L92
When building ros, the build will fail on ament_cmake_core because it cannot find ament_package. Egg modules are not imported even if the site-packages directory is in the python-path. It's possible to add something like this to make the egg import
import site
import os
site.addsitedir(os.path.split(__file__)[0])

However, there's further breakage later in the build process in cmake, as it expects template files in a certain location (and not inside the compressed egg).

Funnily enough, this actually seems to be a bug with Sphinx. I've filed a PR with them: sphinx-doc/sphinx#11418

Resolved in sphinx 7.0.1


Helios Viewer (Not Responding)

Hello! Wanted to take this project for a spin on my Geforce 3090, it looks really impressive! But alas.. after what seems to be messages of success, I am presented with a frozen application with one CPU core pegged at 100%.

********************************************************************************************************
Sat Jan  2 18:50:02 2021
Log Started.
********************************************************************************************************
[ 18:50:02 | INFO    ] : Successfully initialized platform! , FILE : application.cpp , LINE : 117
[ 18:50:02 | INFO    ] : (Vulkan) Vendor : NVIDIA , FILE : vk.cpp , LINE : 4281
[ 18:50:02 | INFO    ] : (Vulkan) Name   : GeForce RTX 3090 , FILE : vk.cpp , LINE : 4282
[ 18:50:02 | INFO    ] : (Vulkan) Type   : VK_PHYSICAL_DEVICE_TYPE_DISCRETE_GPU , FILE : vk.cpp , LINE : 4283
[ 18:50:02 | INFO    ] : (Vulkan) Driver :<PHONE_NUMBER> , FILE : vk.cpp , LINE : 4284
[ 18:50:02 | INFO    ] : (Vulkan) Number of Queue families: 3 , FILE : vk.cpp , LINE : 4321
[ 18:50:02 | INFO    ] : (Vulkan) Family 0 , FILE : vk.cpp , LINE : 4330
[ 18:50:02 | INFO    ] : (Vulkan) Supported Bits:  , FILE : vk.cpp , LINE : 4331
[ 18:50:02 | INFO    ] : (Vulkan) VK_QUEUE_GRAPHICS_BIT: 1 , FILE : vk.cpp , LINE : 4332
[ 18:50:02 | INFO    ] : (Vulkan) VK_QUEUE_COMPUTE_BIT: 1 , FILE : vk.cpp , LINE : 4333
[ 18:50:02 | INFO    ] : (Vulkan) VK_QUEUE_TRANSFER_BIT: 1 , FILE : vk.cpp , LINE : 4334
[ 18:50:02 | INFO    ] : (Vulkan) Number of Queues: 16 , FILE : vk.cpp , LINE : 4335
[ 18:50:02 | INFO    ] : (Vulkan) Family 1 , FILE : vk.cpp , LINE : 4330
[ 18:50:02 | INFO    ] : (Vulkan) Supported Bits:  , FILE : vk.cpp , LINE : 4331
[ 18:50:02 | INFO    ] : (Vulkan) VK_QUEUE_GRAPHICS_BIT: 0 , FILE : vk.cpp , LINE : 4332
[ 18:50:02 | INFO    ] : (Vulkan) VK_QUEUE_COMPUTE_BIT: 0 , FILE : vk.cpp , LINE : 4333
[ 18:50:02 | INFO    ] : (Vulkan) VK_QUEUE_TRANSFER_BIT: 1 , FILE : vk.cpp , LINE : 4334
[ 18:50:02 | INFO    ] : (Vulkan) Number of Queues: 2 , FILE : vk.cpp , LINE : 4335
[ 18:50:02 | INFO    ] : (Vulkan) Family 2 , FILE : vk.cpp , LINE : 4330
[ 18:50:02 | INFO    ] : (Vulkan) Supported Bits:  , FILE : vk.cpp , LINE : 4331
[ 18:50:02 | INFO    ] : (Vulkan) VK_QUEUE_GRAPHICS_BIT: 0 , FILE : vk.cpp , LINE : 4332
[ 18:50:02 | INFO    ] : (Vulkan) VK_QUEUE_COMPUTE_BIT: 1 , FILE : vk.cpp , LINE : 4333
[ 18:50:02 | INFO    ] : (Vulkan) VK_QUEUE_TRANSFER_BIT: 1 , FILE : vk.cpp , LINE : 4334
[ 18:50:02 | INFO    ] : (Vulkan) Number of Queues: 8 , FILE : vk.cpp , LINE : 4335

This is coming from the downloadable distribution [here])https://github.com/diharaw/Helios#usage).
Any ideas what one could do?

Hi! that's quite strange, are you running the latest NVIDIA drivers? Because at the moment the application doesn't perform a graceful exit if the required KHR raytracing extensions aren't available. It could be that.

Hi! that's quite strange, are you running the latest NVIDIA drivers? Because at the moment the application doesn't perform a graceful exit if the required KHR raytracing extensions aren't available. It could be that.

I'm using drivers from the 9th of December, but I see there's one from the 15th. Could that be it? I'll give it another try next time around. Thanks for the quick reply.

I'm using drivers from the 9th of December, but I see there's one from the 15th. Could that be it? I'll give it another try next time around. Thanks for the quick reply.

Yes, you need the drivers from the 15th of December (460.89). It was the first release with the final set of KHR extensions required.

Yes, you need the drivers from the 15th of December (460.89). It was the first release with the final set of KHR extensions required.

Yes, that works. This is really impressive. :) Thanks

Yes, that works. This is really impressive. :) Thanks


*: only print out major.minor version for cluster version

Fix #3284
The patch version has no meaning for cluster version. And it might confuse people who is running a 2.2.3 version of etcd but has a cluster version printed as 2.2.0.
Now
etcdserver: set the initial cluster version to 2.2

Before
etcdserver: set the initial cluster version to 2.2.0

/cc @polvi

lgtm


[FR]: DB Folder Overview with Duplicate Option

Contact Details
No response
Present your request
It would be great to get an overview of the databases and their associated fields in a separate view.  Here 3 direct parameters can be added like:
Add Database Option
Open Database Setting Option
Duplicate as empty with same properties option
via this view, the management of properties should also become easier.

For which platform do you request this request??
Desktop

hi @GaboCapo ,
I think it could be interesting.
Instead of a new view (too complicated to develop) It should be a new modal just like the database creator helper.
Its not trivial because we need to search all the databases inside your vault, load them (just the metadata) and create the UI. I will add it to the roadmap but as long term

I've been able to reproduce most of your requests by setting the source of the database with Dataview query WHERE database-plugin (each database file as property database-plugin) or FROM -"exclude folder" WHERE database-plugin.


Unable to compile

Same game entity. When I use MPT to start, I can compile the trainer, but when I switch to SIT, it will tell me that the compilation has failed. The game version is 29197. This is the link to SIT.
https://drnotdq.net/d/284-0315-这个档 -Spt-380-29197-% E7% 89% 88% E6% 9C% AC% E6% 9B% BF% E6% 8D% A2% E5% 8C% 85--% E6% B8% B8% E6% 88% 8F% E6% 9C% AC% E4% BD% 93

SPT-AKI is patching binaries during the first run, and we need to compile against those patched binaries. If you install this trainer on stock binaries (so even for this exact same version), you'll hit issues.

Ah sorry I think I misunderstood your issue.
Please have a look at #490
You should just need to rerun the installer


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1478 ms

Resolved: Bofrost.at performance has improved in fb2e02b after 16 minutes.


Reify account creation intent in transaction (due to creation-fee)

Exchanges and other parties that are very careful to keep a ledger of balance transfer changes off-chain desire the property that every transaction deterministically produces the same N balance transfers or fails. If we allow implicit account creation, then it's not clear just by looking at a transaction how funds move between accounts. It is also impossible to implement the rosetta spec properly without this.
We should add a field to txns in the relevant places to opt-in to account creation explicit, and add this to GraphQL and CLI wherever necessary. When this bit is missing, account creation should unconditionally fail with an error even if enough funds are moved that an account creation fee would be covered.

My preferred solution to this is:

always show the account creation fee as charged to the receiver (when TID=1)
show this sub-transaction as failed if the account already existed.

Brandon to see which of the 3 possible solutions is best for auditing

According to some third parties, what is preferred from an integration perspective is actually pulling the account creation-fee implicitly from the fee field of the transaction. This way if you are uncertain an account exists you can just fudge your fee a bit, but don't have to rely on wallet balances.

To bring our OOB discussion into the open:

Changing fees to include account creation is a big switch, it means block producers can never look at a transaction and know whether it will be worth including or not.

There's now an explicit create account command, so they should be able to always set that flag and not worry about fees, if that helps with any tensions


feat: CE-1028 - Add Close Button to Desktop Filters

Description
Adding a close button to the filter drawer for desktop.
Fixes #CE-1028
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] Manually Tested
[ ] Test B

Checklist

[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Further comments

Thanks for the PR!
Deployments, as required, will be available below:

Frontend
Backend

Please create PRs in draft mode.  Mark as ready to enable:

Analysis Workflow

After merge, new images are deployed in:

Merge Workflow

Hey Scarlett,
Not too worried about the mobile styling given this view is only available in small viewports which for the most part do not support 'hover' events'. The button styling is a global pattern previously established that I am leveraging.

@jeznorth urrr it's also different with the existing style of the close button in the modal...
I mean the mobile and modal are in sync in style rn. Only this new close button looks different.

@jeznorth urrr it's also different with the existing style of the close button in the modal... I mean the mobile and modal are in sync in style rn. Only this new close button looks different.

We're splitting hairs here. One could argue the out of the box 'close' buttons should be updated appropriately to support appropriate hover states. I'll create a custom style to accommodate this then for this one-off.


Filter search for 'VR' should bring up WebXR docs

When I was first getting started with Babylon.js, the way I jumped into setting up a VR scene as a brand new user was to go to the "How To..." tab of the doc website and to search for "VR".
This search had two results, the WebVR camera and WebVR experience helper docs, which in turn led me down a rabbithole of linked WebVR-focused docs. This was despite the fact that I was doing this well after the WebXR experience helper had been created and was the preferred method. (See: #1974)
In an ideal world, searching for "VR" on the "How To..." filter screen should bring up all WebXR focused docs instead of just WebVR ones. I don't know if this is easily possible with the way the filter box is implemented, I suspect it might currently just be a plaintext search of article titles. This is probably less of a critical issue if #1974 gets merged, but wanted to raise it anyway.
FWIW, the global search in the top-right corner of the docs UI does properly return WebXR content for a search for the term "VR". What I've described specifically only happens in the filter search on pages, such as the "How To..." list.

Also related issue, searching examples for WebXR doesn't return any results. Thanks!

I think this is because there is no WebXR example yet :)
We can add them pretty easily here: https://github.com/BabylonJS/Documentation/blob/master/examples/list.json

Need to address both of those.
Regarding the search - I will make sure to add a large disclaimer on VR-related documentation to go try WebXR instead, and will add VR as a keyword to the XR pages.
I'll also add a few examples to the examples page (and remove the WebVR exampl(s)

Closing per Raanan for #1 and for  #2 (webXR Search) is now working

Closing per Raanan for #1 and for  #2 (webXR Search) is now working


[sonarque] - question: email/smtp support

checked the charts/tpl and values and did not find support to allow us to configure [configuration->general->email[smtp]] which is used by sonar in order to, among other things, notify us (administrators) that we are close to max # lines of code (for licensed version) and other important notifications. any plans to support that? thanks

this could be done via sonar.properties which is supported already by the chart. thanks


Wacom Pressure Sensitivity not working

I use a wacom Intuous 4.
Pen Pressure doesn't seem to work under windows 7.

9a9f03c5d76707082fd8c854de121750b18a49c9 :(

so wacom can't be used now anymore? there's no way to enable it?

@naomik chrome now supports PointerEvent (since chrome 55. current stable version is 56). so we can make sculptgl to support tablet pen pressure by using PointerEvent instead of NPAPI.

Thanks for the heads up, didn't know it was released, I just added the support!
I don't have a wacom tablet, so I can't really test.
I tested on my nexus 5x, and it does have an impact, though the pressure is infered from the surface area rather than true pressure.