Added division functionality

Hi @GOPAL-YADAV-D, can you review this PR?

Hi @DNA5769, please check this PR.

Hi @GOPAL-YADAV-D, can you review this PR?

It is under review,

@docjon09 ,Please resolve the conflicts too

Hi @GOPAL-YADAV-D , I have done the required corrections. Please review them.

@docjon09 ,Thank you for your valuable contribution!! . Do check out other repositories from IEEE-VIT.


Panics in bitvector reasoning

Came across two panics with bitvector reasoning.
This succeeds:
#[proof] #[verifier(bit_vector)]
fn f() {
    ensures(forall(|i: u64| i < 64 >>= ((1 as u64) << i > 0)));
}

This panics:
#[proof] #[verifier(bit_vector)]
fn f() {
    ensures(forall(|i: u64| ((1 as u64) << i > 0)));
}

thread 'rustc' panicked at 'internal error: generated ill-typed AIR code: error 'in call to has_type, argument #1 has type "BitVec64" when it should have type "Poly"' in expression '(has_type i@ (UINT 64))'', rust_verify/src/verifier.rs:354:21
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
error: aborting due to previous error

This succeeds:
#[proof] #[verifier(bit_vector)]
fn f() {
    ensures((1 as u64) << (1 as u64) > 0);
}

This panics:
#[proof] #[verifier(bit_vector)]
fn f() {
    ensures((1 as u64) << 1 > 0);
}

thread 'rustc' panicked at 'internal error: generated ill-typed AIR code: error 'error 'in call to <<, arguments should have the same width, but got 64 and 32' in expression '(bvshl #x0000000000000001 #x00000001)'' in expression '(bvugt (bvshl #x0000000000000001 #x00000001) #x0000000000000000)'', rust_verify/src/verifier.rs:354:21
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

ping @channy412

Sorry again that I didn't fix that already. I will get back to this issue after fixing the recommends checking part(hoping that it will fix this too). Thanks a lot for the posting!

The second example is about improper error handling in the bit-vector translation. https://github.com/secure-foundations/verus/pull/193 will fix this, and report VIR error instead of panic


Unexpected behavior change from pixi v.0.18 to v0.19 for github pypi dependencies

Checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest version of pixi, using pixi --version.

Reproducible example

Issue description
We have recently switched our MODFLOW continuous integration approach to use pixi.  We had it set up and working with pixi version 0.18 and things were working well.  Our CI workflow started failing today in response to a change introduced to version 0.19.  This is the error message we see:
Run<EMAIL_ADDRESS>  with:
    pixi-version: latest
  env:
    PIXI_BETA_WARNING_OFF: true
Downloading Pixi
  Pixi installed to /home/runner/.pixi/bin/pixi
Restoring pixi cache
pixi install --locked
  /home/runner/.pixi/bin/pixi install --locked --manifest-path pixi.toml --color always
    × lock-file not up-to-date with the project

The problem stems from the use of github pypi dependencies in our pixi.toml file.
[pypi-dependencies]
flopy = { git = "https://github.com/modflowpy/flopy.git" }
modflowapi = { git = "https://github.com/MODFLOW-USGS/modflowapi.git" }
modflow-devtools = { git = "https://github.com/MODFLOW-USGS/modflow-devtools.git" }

The pypi dependency hashes in pixi.lock are not up to date. Running pixi install --locked with 0.18 succeeds, and pixi install does not update locked hashes. With 0.19, pixi install --locked fails with the error message above, and pixi install updates the hashes.
It's unclear to us why pixi 0.19 is issuing the "lock-file not up-to-date with the project", whereas pixi 0.18 did not.  I'm working on this with @wpbonelli and @Hofer-Julian.
Expected behavior
pixi version 0.19 should behave the same as 0.18 w.r.t. github pypi dependencies.

He quick reply from my phone, this was actually a fix in v0.19.0, your lockfile is probably incorrect because it misses purls.

Running pixi install with v0.19.0 should fix the lockfile. And should be backwards compatible with v0.18.0.

Thanks @ruben-arts I take it https://github.com/prefix-dev/pixi/pull/1148 is the fix you are referring to. I wonder if you would help clarify some things.
If we update the lockfile as you suggest, will the setup-pixi action fail (as it does now) whenever a github dependency is found to be out of date? Do we need to set frozen to avoid this, since the action defaults to locked if a pixi.lock file is found?
How would you suggest proceeding if we don't want to lock github dependencies? Historically we have elected to depend on github branches for projects which are developed more or less in lockstep with MODFLOW 6. Is there a post-install hook or some way to install them on pixi install? Or is a custom task (that must be run separately) the only way?

That is indeed the fix.
I'm not sure why the GitHub repos would trigger the invalid lockfile. I'll have to look into that, as we don't check if the revisions in the lockfile are up-to-date in the --locked check.
We implemented the same behavior as cargo. So I guess we should create an update command so you could do pixi update pkgx pkgy pkgz on a ci run if you want that.

I think something is going wrong, haven't figured out what but with running -vv you can see why a lock-file is considered out-of-date.
 INFO pixi::config: Global config not found at /Users/tdejager/.config/pixi/config.toml
 INFO pixi::config: Global config not found at /Users/tdejager/Library/Application Support/pixi/config.toml
 INFO pixi::config: Global config not found at /Users/tdejager/.pixi/config.toml
 INFO pixi::environment: verifying prefix location is unchanged, with prefix file: /private/tmp/modflow6/.pixi/envs/default/conda-meta/pixi_env_prefix
 INFO pixi::lock_file::outdated: the pypi dependencies of environment 'default' for platform win-64 are out of date because the requirement 'pandas' could not be satisfied (required by 'modflowapi')
 INFO pixi::lock_file::outdated: the pypi dependencies of environment 'default' for platform linux-64 are out of date because the requirement 'pandas' could not be satisfied (required by 'modflowapi')
 INFO pixi::lock_file::outdated: the pypi dependencies of environment 'default' for platform osx-arm64 are out of date because the requirement 'pandas' could not be satisfied (required by 'modflowapi')
 INFO pixi::lock_file::outdated: the pypi dependencies of environment 'default' for platform osx-64 are out of date because the requirement 'pandas' could not be satisfied (required by 'modflowapi')
 ...

We are investigating further.

It's a bigger problem with missing packages from the new pypi mapping, we are trying to fix this now.

The current issue is that our map is out of date with conda-forge @nichmor is working on that to fix v0.19.0. I'm going to remove the use of that map from main and make a patch release as soon as we can.

We fixed the map locally I get the correct versions for modflow6. However, you need to regenerate the whole lock file once, sorry about that.

We fixed the map. Locally I get the correct versions for modflow6. However, you need to regenerate the whole lock file once, sorry about that.

Thanks, @tdejager.
@mjr-deltares and I already wondered why updating the lockfile without deleting it first didn't work.
I've already regenerated it with pixi 0.19 and that seems to work fine. Or should I regenerate it with 0.18 to be sure?

Would be good to see if there is now a difference between the two pixi versions, so if you could share that, that would be nice! @Hofer-Julian

I ran the following:
rm pixi.lock
./pixi018 install
mv pixi.lock pixi018.lock
./pixi019 install
mv pixi.lock pixi19.lock

The lockfiles differ and that's how they look like:
modflow6_lock_files.zip

Yeah, okay that's looking good, so the new lock file is only including the pypi source dependencies, which is what you want to achieve:
      - pypi: git+https://github.com/modflowpy/flopy.git@9e87acddcf0740aac491826ef86fd45ae2304dac
      - pypi: git+https://github.com/MODFLOW-USGS/modflow-devtools.git@08eff72b73da66541f7a08ddf398298210ce3758
      - pypi: git+https://github.com/MODFLOW-USGS/modflowapi.git@0c632e511fb54068d14dcdff44c572ccb6e8831f

The rest is conda so this seems to be working. Also the 0.19 one seems to include purl's which we now use for the conda to pypi comparison.

Close as fixed


No respone in gmail

In gmail, I typed in some markdown stuff, and then both clicked on the text box then select the markdown toogle button, and also highlight those syntax and click the button, still no response, even I typed in the example code
print('You get syntax highlighting...')
print('...if you include the language name')

Please help me

In gmail on Chrome in Mac OS I had similar results until restarting Chrome.

I was on windows but still face this issue.


Relative path in custom watchers

This was missing in #3633.

Codecov Report

Merging #3673 into dev will decrease coverage by 0.09%.
The diff coverage is 87.5%.

@@            Coverage Diff            @@
##              dev    #3673     +/-   ##
=========================================
- Coverage   98.94%   98.85%   -0.1%
=========================================
  Files          18       18
  Lines        1139     1135      -4
  Branches      306      304      -2
=========================================
- Hits         1127     1122      -5
- Misses         12       13      +1

Impacted Files
Coverage Δ

lib/builder/builder.js
97.84% <87.5%> (-0.58%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b4ab277...d466b37. Read the comment docs.

Indeed! Will update.
On Fri, Aug 10, 2018 at 2:52 AM Alexander Lichter<EMAIL_ADDRESS>wrote:

@manniL commented on this pull request.
In lib/builder/builder.js
https://github.com/nuxt/nuxt.js/pull/3673#discussion_r209150738:

@@ -631,10 +633,10 @@ export default class Builder {
})

 const nuxtRestartWatch = _.concat(

 this.options.watch,

 this.options.watch.map(path => r(path)),

Could this work as this.options.watch.map(r) as well?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nuxt/nuxt.js/pull/3673#pullrequestreview-145110420,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAwA5-no9ZLGBPeGVxva_L-MQEmerPkks5uPR-FgaJpZM4V3km6
.


Implement live streaming for execute resources

This brings live streaming of execute resource output to the
output formatters. It also adds a mechanism for checking to
see if an output formatter is in use through the event dispatch
system.
It adds a new configuration option, "always_stream_execute", which
does what it says on the tin.

Please have a look at the failing tests.

We should fix this up and get it merged.

Yes, please. I've been too busy to do it myself and also have failed at cajoling people into picking this up.

Yeah I'll eventually get to it if nobody beats me to it.


Feat: Display output id in output

Description of change

Display output id in outputs.
Fix memory leak warning in associated output table.

Aims to fix https://github.com/iotaledger/explorer/issues/358
Type of change

Enhancement (a non-breaking change which adds functionality)

Change checklist

[x] My code follows the contribution guidelines for this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas

This is how it looks now:

LGTM.
Needs links to be updated to point to "Output page" which we don't have atm. So it will be finished after https://github.com/iotaledger/explorer/issues/370.


Move repo to SECQUOIA

I'll move this repo to SECQUOIA for all of us to contribute to it

I realized you have to move the repo yourself Albert


"unknown field SkipMagic" when installing cli mcap from github url, works fine in cloned source

Description

Version: 8bc1ef7ae4eb949bdcce8a8a61f1a036819847fc  tag: releases/python/mcap-ros2-support/v0.5.3
Platform: Ubuntu 23.04, apt installed golang-go go1.20.3 linux/amd64

Steps To Reproduce

sudo apt install golang-go
$ go install github.com/foxglove/mcap/go/cli/mcap@latest
# github.com/foxglove/mcap/go/cli/mcap/utils
go/pkg/mod/github.com/foxglove/mcap/go/cli/mcap@v0.0.0-20231023205342-8bc1ef7ae4eb/utils/mcap_amendment.go:189:3: unknown field SkipMagic in struct literal of type mcap.WriterOptions
go/pkg/mod/github.com/foxglove/mcap/go/cli/mcap@v0.0.0-20231023205342-8bc1ef7ae4eb/utils/mcap_amendment.go:259:17: writer.WriteChunkIndex undefined (type *mcap.Writer has no field or method WriteChunkIndex)
go/pkg/mod/github.com/foxglove/mcap/go/cli/mcap@v0.0.0-20231023205342-8bc1ef7ae4eb/utils/mcap_amendment.go:401:3: unknown field SkipMagic in struct literal of type mcap.WriterOptions

Cloning locally works:
git clone git@github.com:foxglove/mcap.git  # probably should do shallow clone instead
go install go/cli/mcap
$HOME/go/bin/mcap
🔪 Officially the top-rated CLI tool for slicing and dicing MCAP files.
...

(mcap version just outputs a blank line though)
Maybe the real difference is not having the @latest- does the mcap binary behave any differently without it?
Expected Behavior
mcap binary is put into $HOME/go/bin

I think that’s why we say “Installing via 'go install' is not recommended, and is not guaranteed to work.” 😉

Some context: https://github.com/foxglove/mcap/pull/826

Ok, it looks like from https://github.com/foxglove/mcap/blob/main/.github/workflows/ci.yml make is the real way to build it:
cd go
make -C cli/mcap build
...
./cli/mcap/bin/mcap version
releases/python/mcap-ros2-support/v0.5.3


Made the installation flow for Kueue Grafana dashboard smoother

Fixes #191.
Propagated environment variables from the Cloud Build yaml into the installation script. Added timeout to the wait for CRD creation.

@arueth Would appreciate a review!


Empty results on large queries when using fetchall()

Hello there.
I've found that these two examples will not always return the same thing on large queries:

fetchall

cur.execute(large_query)
rows = cur.fetchall()
print(rows)

iterator

for r in cur.execute(large_query):
  print(r)

The fetchall approach ends up returning an empty list every time I use it when running large queries, but the iterator approach prints the result successfully
Does anyone know why this may be happening?
I don't know how I should supply a working example to debug this problem, as my queries all contain domain logic of my company. Nevertheless, I can say that I'm connecting around 8 Postgres databases with 100k rows per table and equal schemas, and I union the tables from all databases together with UNION views, like this:
CREATE OR REPLACE VIEW hive.default."table_unioned_view" AS
SELECT 'database1' AS database, "other_column" FROM database1.public.table
UNION ALL
SELECT 'database2' AS database, "other_column" FROM database2.public.table
UNION ALL
SELECT 'database3' AS database, "other_column" FROM database3.public.table;

And the big query that I'm running uses these type of views

Would you like to try to come up with a test case reproducing the problem?
See https://github.com/prestosql/presto-python-client/blob/a7438556213bcf954f8a7a5063c7823bbb177edb/integration_tests/test_dbapi.py#L59-L68 for some example test using dockerized Presto


CAMEL-20852: Fix CamelCatalogTest

Motivation
The build is failing because of the test CamelCatalogTest since the addition of an option within the context of CAMEL-20852
Modifications:

Adapt the test consequently

failure due to not regenerated code related to Elasticsearch

Let's merge this so at least the PRs won't fail there but later.


Added animation on scroll feature

🛠 Fixes Issue
Closes #79
👨💻 Changes proposed
I have added animation on scroll behaviour
🔧 Check List (Check the type of issue it fixes)

[ ] Bug Fix
[ ] New Extension
[x] Enhancement
[ ] Documentation

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] Any dependent changes have been merged and published in downstream modules
[x] I have done git pull before adding my new changes
[x] The title of my pull request is a short description of the requested changes.

@Arjun31415 Kindly review and merge the PR

LGTM


feat(transformer/commonjs): support dynamic import and esModule flag.

Based on the issue #5368, since the size of the PR is too large, I opt to implement additional features using other PRs (including this) or align the behavior with Babel.

stale


Beta Realease!!!

Codecov Report

Merging #27 (512ebb0) into main (84c20a9) will decrease coverage by 21.13%.
The diff coverage is 6.81%.

@@             Coverage Diff              @@
##              main      #27       +/-   ##
============================================
- Coverage   100.00%   78.86%   -21.14%
============================================
  Files           13       13
  Lines          305      388       +83
============================================
+ Hits           305      306        +1
- Misses           0       82       +82

Impacted Files
Coverage Δ

src/LocalSearchSolvers.jl
100.00% <ø> (ø)

src/solver.jl
98.96% <0.00%> (-1.04%)
:arrow_down:

src/state.jl
97.43% <0.00%> (-2.57%)
:arrow_down:

src/utils.jl
100.00% <ø> (ø)

src/models/sudoku.jl
15.78% <1.23%> (-84.22%)
:arrow_down:

src/model.jl
100.00% <100.00%> (ø)

src/models/cut.jl
100.00% <100.00%> (ø)

src/models/golomb.jl
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 84c20a9...512ebb0. Read the comment docs.

Codecov Report

Merging #27 (512ebb0) into main (84c20a9) will decrease coverage by 21.13%.
The diff coverage is 6.81%.

@@             Coverage Diff              @@
##              main      #27       +/-   ##
============================================
- Coverage   100.00%   78.86%   -21.14%
============================================
  Files           13       13
  Lines          305      388       +83
============================================
+ Hits           305      306        +1
- Misses           0       82       +82

Impacted Files
Coverage Δ

src/LocalSearchSolvers.jl
100.00% <ø> (ø)

src/solver.jl
98.96% <0.00%> (-1.04%)
:arrow_down:

src/state.jl
97.43% <0.00%> (-2.57%)
:arrow_down:

src/utils.jl
100.00% <ø> (ø)

src/models/sudoku.jl
15.78% <1.23%> (-84.22%)
:arrow_down:

src/model.jl
100.00% <100.00%> (ø)

src/models/cut.jl
100.00% <100.00%> (ø)

src/models/golomb.jl
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 84c20a9...512ebb0. Read the comment docs.


Add installation instructions

Add pip installation instructions.  It seems like such an obvious step, but it's not always a given that some project is available on pypi or through pip install.

thanks so much.


Add context key for when comments are visible

I was trying to get escape collapsing all comments (https://github.com/microsoft/vscode/issues/131419) but this would need an extra context key. Here are the current context keys:

I think ideally we would have commentsAreVisibleInViewport or commentsAreVisible, the latter isn't quite as good since it would hide comments which are currently off screen and might seem like nothing happened.

Not quite what you're looking for, but we have workspaceHasCommenting now in case that's useful to someone else who comes across this issue.

I wanted a keybinding like this:
{ "key": "escape", "command": "comments.collapse", "when": "commentsAreVisibleInViewport" }

workspaceHasCommenting wouldn't work in this case because I want escape only to do that if there are visible comments in the editor, so it still works when notifications show, the hover is focused, etc.


react-server-cli doesn't hot reload or watch components that require react-server-gulp-module-tagger

When working on #222, I had to take down the server and restart it every time I changed a component because I used the logger, and that requires tagging, which react-server-cli doesn't do.  We should either switch from using gulp to using a babel plugin for our source transforms, or we could update react-server-cli to use react-server-gulp-module-tagger as part of its compile step.

update react-server-cli to use react-server-gulp-module-tagger as part of its compile step

I totally thought it did that.


fence.i required after modification of the trap vector table?

Should CLIC also specify fence.i is required after modification of the trap vector table like what is specified in the Zc proposed extension?
Zc extension - Table Jump Fault handling section:
"Memory writes to the jump vector table require an instruction barrier (fence.i) to guarantee that they are visible
to the instruction fetch."
Created pull #206.

The note added in https://github.com/riscv/riscv-fast-interrupt/pull/206/files#diff-3eb538450d2c1f1119dfc066c0ed20f286527c0e4f995a9196c77dfd30b8f7b4R1072 states that a FENCE.I can be used to address the coherency issue that can exist between a store to the trap vector table and the load to retrieve the function pointer from the vector table.
This does not sound correct as "the memory access to retrieve the function pointer for vectoring is treated as a load" whereas a FENCE.I instruction "provides explicit synchronization between writes to instruction memory and instruction fetches on the same hart".
Had the CLIC pointer retrieval been handled as an instruction fetch (as done for the table jump in Zc, see https://github.com/riscv/riscv-fast-interrupt/issues/191), then a fence.i would work.
It should not matter that an implementation uses the fetch stage/infrastructure/interface to perform the pointer fetch related memory access, it is still a load. (We have now implemented this in the CV32E40X and handling this pointer access as a load is a bit awkward).
Instead of recommending FENCE.I maybe a regular FENCE can be recommended?

closed with pull #227


[BUG] All documents get dumped into "Documents" collection since 1.14.0

Describe the bug
Since 1.14.0 all my documents get inserted into one collection named "Documents". All documents also got a _class_id value with .UserEntry
To Reproduce
class UserEntry(Document, ABC):
    id: int
    value: Optional[str] = None
    ...

    class Settings:
        name = 'user_data'
        use_state_management = True

user_data = await UserEntry(...)
user_data.value = 'Test'
await user_data.insert()

Expected behavior
The document get's inserted into a "user_data" collection.
Additional context
https://github.com/roman-right/beanie/compare/1.13.1...1.14.0

I tried this against 1.13.1, 1.14.0, and the current 1.15.3. It inserts into the collection user_data
Full script:
import asyncio
from abc import ABC
from typing import Optional

from motor.motor_asyncio import AsyncIOMotorClient

from beanie import Document, init_beanie

class UserEntry(Document, ABC):
    id: int
    value: Optional[str] = None

    class Settings:
        name = 'user_data'
        use_state_management = True

async def main():
    cli = AsyncIOMotorClient("mongodb://test:test@localhost:27017")
    db = cli["test_find_one"]
    await init_beanie(db, document_models=[UserEntry])

    await UserEntry(id=2, value="Test").insert()

asyncio.run(main())

Please, correct my code to reproduce this error.

@roman-right
Hi, I've got this issue as well.
Before ~1.13 I used the following pattern:
Have and extended base document class:
class BetterDocument(Document):
    @classmethod
    async def get_or_create(cls, data):
        record = await cls.find_one(data)
        if not record:
            record = cls(**data)
            await record.insert()
        return record

    @classmethod
    async def get_or_update(cls, lookup, data):
        record = await cls.find_one(lookup)
        all_data = {**lookup, **data}
        if not record:
            record = cls(**all_data)
            await record.insert()
        else:
            await record.update(all_data)
        return record

    @classmethod
    async def get_edged(cls, lookup=None, default=None, direction=DESCENDING, sort_by="created"):
        umongo_cursor = cls.find(lookup if lookup else {}).sort([(sort_by, direction)]).limit(1)
        async for record in umongo_cursor:
            return record
        return default

    @classmethod
    async def get_latest(cls, lookup=None, default=None):
        return await cls.get_edged(lookup, default, direction=DESCENDING)

    @classmethod
    async def get_earliest(cls, lookup=None, default=None):
        return await cls.get_edged(lookup, default, direction=ASCENDING)

Then I inherited it for all my models in a project:
...

class PlayerIdentity(BaseModel):
    external_id: Indexed(str, unique=True)
    username: Indexed(str, unique=True)
    demo_player: bool = False

class PlayerStatistics(BaseModel):
    total_bets: int = 0
    total_wins: int = 0

class Players(PlayerIdentity, PlayerStatistics, BetterDocument):
    created: datetime = Field(default_factory=datetime.utcnow)
    integration: Link[Integrations]
    game_settings: PlayerGameSettings = Field(default_factory=PlayerGameSettings)

...

Everything worked perfectly until (I guess... didn't notice exactly) - v13.2.
After that version, I've got created a document called BetterDocument
Note: I didn't put this document to the document_models list into the init_beanie call
Fields of that document are all fields from all documents inherited from it + field called _class_id with the name of the childer document with . prefix, e.g.:

Thank you in advance! love beanie very much!

Juggerr thanks for sharing your case. I simplified my example strongly, so this might be the reason for not causing the issue.
I also subclassed the Document class (overwrite save_changes ti insert the doc if not exists) and called it Document. Seems like it's connected with that. I'll investigate that later.

Thank you for your input @Juggerr ,
Could you guys make a script that I can run to see the problem? This bug looks pretty important but I can not reproduce it.
I use your snippets - put them into my dummy inserter and don't see this bug:
import asyncio
from datetime import datetime

from beanie import Document, init_beanie
from motor.motor_asyncio import AsyncIOMotorClient
from pydantic import BaseModel, Field

class BetterDocument(Document):
    ...

class PlayerIdentity(BaseModel):
    one: str = "one"

class PlayerStatistics(BaseModel):
    total_bets: int = 0
    total_wins: int = 0

class Players(PlayerIdentity, PlayerStatistics, BetterDocument):
    created: datetime = Field(default_factory=datetime.utcnow)

async def main():
    cli = AsyncIOMotorClient("mongodb://beanie:beanie@localhost:27017")
    db = cli["test_wrong_collection"]
    await init_beanie(db, document_models=[Players])

    await Players().insert()

asyncio.run(main())

Beanie versions: 1.13.1, 1.14.0, 1.15.x

Here is my example:
import asyncio
from pydantic import BaseModel, Field
from motor.motor_asyncio import AsyncIOMotorClient
from beanie import Document, init_beanie

class Mixin(BaseModel):
    id: int = Field(..., ge=1, le=254)

class MyDoc(Document):
    class Settings:
        is_root = True
        use_state_management = True

class Test(Mixin, MyDoc):
    name: str

class Test2(MyDoc):
    name: str

async def main():
    client = AsyncIOMotorClient()
    await init_beanie(database=client.b, document_models=[Test, Test2])
    t1 = await Test(id=1, name='test').create()
    print('Created', t1)
    t2 = await Test2(name='test2').create()
    print('Created', t2)
    await t1.delete()
    await t2.delete()

if __name__ == '__main__':
    asyncio.run(main())

Beanie version 1.15.3. Using mongosh I see only "MyDoc collection" and no Test & Test2.

Thank you, @Vitalium !
Reproduced. This will be fixed today

@Luc1412 , @Juggerr , @Vitalium please try 1.15.4

works fine. Thanks for fixing it.

@roman-right It works, thank you very much!


Step 8 - Changing the selected output

Step 8: Changing the selected output
In the previous step, you found invocations of the macros we are interested in. Modify your query to find the top-level expressions these macro invocations expand to.
Note: An expression is a source code element that can have a value at runtime. Invoking a macro can bring various source code elements into scope, including expressions.

:keyboard: Activity: Find the expressions that correspond to macro invocations
As before, if you don't know how a piece of source code is represented in the library, you can use the auto-completion and contextual help to discover the classes and predicates you need.

Edit the file 8_macro_expressions.ql with the previous query
Use the getExpr() predicate in the select section, to return the wanted expressions.
Once you're happy with the results, submit your solution.


Error: Unsupported platform: aarch64-linux-android

I run deno 1.35.0 for aarch64-linux-android (adb shell): https://github.com/denoland/deno/issues/19759
violet:/data/local/tmp/deno-test/5 $ ./deno --version
deno 1.35.0 (release, aarch64-linux-android)
v8 <IP_ADDRESS>
typescript 5.1.6

Fresh can create a project but can not run:
violet:/data/local/tmp/deno-test/5 $ export HOME=$(pwd)
violet:/data/local/tmp/deno-test/5 $ export PATH=$(pwd):$PATH
violet:/data/local/tmp/deno-test/5 $ deno run -A -r https://fresh.deno.dev

  🍋 Fresh: the next-gen web framework.

Project Name [fresh-project]
Let's set up your new Fresh project.

Fresh has built in support for styling using Tailwind CSS. Do you want to use this? [y/N] y
Do you use VS Code? [y/N] n
The manifest has been generated for 3 routes and 1 islands.

Project initialized!

Enter your project directory using cd fresh-project.
Run deno task start to start the project. CTRL-C to stop.

Stuck? Join our Discord https://discord.gg/deno

Happy hacking! 🦕
violet:/data/local/tmp/deno-test/5 $ cd fresh-project/
violet:/data/local/tmp/deno-test/5/fresh-project $ ls -l
total 36
-rw-rw-rw- 1 shell shell  363 2023-07-08 11:33 README.md
drwxrwxrwx 2 shell shell 3488 2023-07-08 11:33 components
-rw-rw-rw- 1 shell shell  767 2023-07-08 11:33 deno.json
-rwxrwxrwx 1 shell shell  130 2023-07-08 11:33 dev.ts
-rw-rw-rw- 1 shell shell  604 2023-07-08 11:33 fresh.gen.ts
drwxrwxrwx 2 shell shell 3488 2023-07-08 11:33 islands
-rw-rw-rw- 1 shell shell  454 2023-07-08 11:33 main.ts
drwxrwxrwx 3 shell shell 3488 2023-07-08 11:33 routes
drwxrwxrwx 2 shell shell 3488 2023-07-08 11:33 static
-rw-rw-rw- 1 shell shell  111 2023-07-08 11:33 twind.config.ts
violet:/data/local/tmp/deno-test/5/fresh-project $ deno task start
Task start deno run -A --watch=static/,routes/ dev.ts
Watcher Process started.
The manifest has been generated for 3 routes and 1 islands.

 🍋 Fresh ready
    Local: http://localhost:8000/

An error occurred during route handling or page rendering. Error: Unsupported platform: aarch64-linux-android
    at install (https://deno.land/x/esbuild@v0.17.19/mod.js:1843:11)
    at https://deno.land/x/esbuild@v0.17.19/mod.js:1852:29
    at ensureServiceIsRunning (https://deno.land/x/esbuild@v0.17.19/mod.js:1975:7)
    at Module.initialize (https://deno.land/x/esbuild@v0.17.19/mod.js:1743:9)
    at initEsbuild (https://deno.land/x/fresh@1.2.0/src/build/esbuild.ts:120:19)
    at EsbuildBuilder.build (https://deno.land/x/fresh@1.2.0/src/build/esbuild.ts:40:13)
    at ServerContext.#buildSnapshot (https://deno.land/x/fresh@1.2.0/src/server/context.ts:402:31)
    at Object.default (https://deno.land/x/fresh@1.2.0/src/server/context.ts:772:49)
    at handler (https://deno.land/x/fresh@1.2.0/src/server/router.ts:177:34)
    at Object.next (https://deno.land/x/fresh@1.2.0/src/server/context.ts:436:34)

esbuild has support for arm64 android: https://www.npmjs.com/package/@esbuild/android-arm64

The current workaround of this problem is to manually download esbuild binary:
> wget<EMAIL_ADDRESS>> tar -xvf android-arm64-0.17.19.tgz
> cd package/bin
> adb push esbuild /data/local/tmp/deno-test/5
esbuild: 1 file pushed, 0 skipped. 53.3 MB/s (9371648 bytes in 0.168s)
> adb shell
violet:/ $ cd /data/local/tmp/deno-test/5
violet:/data/local/tmp/deno-test/5 $ export HOME=$(pwd)
violet:/data/local/tmp/deno-test/5 $ export PATH=$(pwd):$PATH
violet:/data/local/tmp/deno-test/5 $ esbuild --version
0.17.19
violet:/data/local/tmp/deno-test/5 $ export ESBUILD_BINARY_PATH=$(pwd)/esbuild
violet:/data/local/tmp/deno-test/5 $ cd fresh-project/
violet:/data/local/tmp/deno-test/5/fresh-project $ deno task start
Task start deno run -A --watch=static/,routes/ dev.ts
Watcher Process started.
The manifest has been generated for 3 routes and 1 islands.

 🍋 Fresh ready
    Local: http://localhost:8000/

Now it works well on Android.

The PR has been merged: https://github.com/evanw/esbuild/pull/3217
so this will fix after upgrade to new version of esbuild :-)

Resolved by https://github.com/denoland/fresh/pull/1841

I have the shame issues:(
 < ~/deno/fresh > -> deno task start
Task start deno run -A --watch=static/,routes/ dev.ts
Watcher Process started.
The manifest has been generated for 5 routes and 1 islands.

 🍋 Fresh ready
    Local: http://localhost:8000/

An error occurred during route handling or page rendering.
Error: Unsupported platform: aarch64-alpine-linux-musl
    at install (https://deno.land/x/esbuild@v0.19.11/mod.js:1878:11)
    at https://deno.land/x/esbuild@v0.19.11/mod.js:1887:29
    at ensureServiceIsRunning (https://deno.land/x/esbuild@v0.19.11/mod.js:2010:7)
    at Module.initialize (https://deno.land/x/esbuild@v0.19.11/mod.js:1777:9)
    at EsbuildBuilder.build (https://deno.land/x/fresh@1.6.3/src/build/esbuild.ts:54:21)
    at async ServerContext.buildSnapshot (https://deno.land/x/fresh@1.6.3/src/server/context.ts:304:26)
    at async ServerContext.#bundleAssetRoute (https://deno.land/x/fresh@1.6.3/src/server/context.ts:637:22)
    at async handler (https://deno.land/x/fresh@1.6.3/src/server/context.ts:288:14)
    at async ext:deno_http/00_serve.js:448:1


Fix code scanning alert

Tracking issue for:

[x] https://github.com/ConsenSys/quorum-explorer/security/code-scanning/4

Fixed in https://github.com/ConsenSys/quorum-explorer/pull/39


Clarify that this is not supported for Azure SQL

@srdan-bozovic-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @srdan-bozovic-msft - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. Would you make this content update in the private repo instead? Thank you! We'll close this PR.


Soundcloud Problem

when play a soundcloud song the bot still Loading please wait... message and not work anything

i fix soon the next update!


ProbabilitySampler doesn't honor W3C sampled flag

What version of OpenTelemetry are you using?
0.4.0
What version of Node are you using?
12.16.0
What did you do?
const spanContext = {
  traceId: 'd4cda95b652f4a1592b449d5929fda1b',
  spanId: '6e0c63257de34c92',
  traceFlags: TraceFlags.UNSAMPLED,
};
const span = tracer.startSpan("main", { parent: spanContext });

What did you expect to see?
I would expect that above span is a NoRecordingSpan as the incoming sampling flag indicates that trace is unsampled.
What did you see instead?
The ProbabilitySampler ignores the sampling flag and a normal, recording Span is created.
Additional context
I'm not sure if this is really a bug or intended behavior of the ProbabilitySampler.
I found following in W3C spec regarding this: Probability sampling (sample 1 out of 100 distributed traces by flipping a coin).
My interpretation of this is that the first tier flips the coin an tells the others about it's decision via the sampling flag.
I think it doesn't make much sense to re-decide on each tier as this would result in each distributed trace having gaps.

That is also my interpretation.
It looks like Python defers to the parent context if it is there.
Ruby tries to implement respecting the parent's decision, but it has a bug where it will decide again if the parent sampled is false.


Fix #380: Align share buttons vertically.

Thanks for the review.
I don't really know. The all buttons have the same height, which I guess is 10px? It does work, and I just thought it is relatively better solution than other answers in Stackoverflow.


Improve error handling

The script should raise click exceptions to communicate errors to the user instead of raising generic errors and stack trace
https://click.palletsprojects.com/en/7.x/api/#click.ClickException

resolved in #11


container not able to start.

Hi, Im very much a proxmox and linux newbie.     I ran the script to create the container.    I can see in my promox dashboard that vm101 was created.     When I go to start the container I receive the following error.   Any thoughts?    Again, I apologize if this is a user error.
Nov 16 11:39:04 nuc systemd[1]: Starting PVE LXC Container: 101...
Nov 16 11:39:04 nuc lxc-start[17655]: lxc-start: 101: lxccontainer.c: wait_on_daemonized_start: 856 No such file or directory - Failed to receive the container state

I echo what ktoms71 says.  I have installed in proxmox and it will not start.

I would also like to add that the following error was logged:
Job for<EMAIL_ADDRESS>failed because the control process exited with error code.
See "systemctl status<EMAIL_ADDRESS>and "journalctl -xe" for details.
TASK ERROR: command 'systemctl start pve-container@101' failed: exit code 1

same issue here, and it was working on this server with an older version of your script
if it helps i run it with debugging log on and i get this:
root@pve:~# lxc-start -n 108 -l DEBUG -F lxc-start: 108: conf.c: run_buffer: 335 Script exited with status 25 lxc-start: 108: start.c: lxc_init: 861 Failed to run lxc.hook.pre-start for container "108" lxc-start: 108: start.c: __lxc_start: 1944 Failed to initialize container "108" lxc-start: 108: tools/lxc_start.c: main: 330 The container failed to start lxc-start: 108: tools/lxc_start.c: main: 336 Additional information can be obtained by setting the --logfile and --logpriority options
i've also tried updating promox, disk space fine. Im able to mount container to.

if you remove apt upgrade -y from install_tuya-convert.sh it worked for me
bash -c "$(wget -qO - https://raw.githubusercontent.com/matbrewer396/proxmox_tuya-convert_container/master/create_container.sh)" -s local-lvm

I'm not sure why it would not start the container. If you have updated Proxmox recently, make sure you reboot Proxmox if you are having problems. If you still receive this error, run journalctl -xe and post output from Starting PVE LXC Container: to the last error related to the container.

Have the same problem. any progress on this? or use matbrewer396 link?
or can you update yours?

Yes.  After updating ProxMox it started right up.

Idd it works :) only cant connect to samba for uploading the bin fil :(


ControlPlane does not add itself to Notification Center

ControlPlane is not adding itself to Notification Center on Yosemite.

Double check your ControlPlane config, general tab, to ensure Use Notifications is enabled.
Also double check that you haven't disabled notifications for ControlPlane in the notification center section of System Preferences.


ensure the session cookie for docs is different to the bk/bk session cookie

In production both bk/bk and docs are hosted under the buildkite.com domain, so in theory they can read each others cookies.
However, the rails session cookie is encrypted and we use different encryption keys for both apps. If we configure them to use the same session cookie, they'll only both be able to read the data if we use the same encryption key. I've done that before elsewhere and it works, but coupling codebases like that is awkward and brittle.
The docs app has no logged in state anyway, so for now there's no need for it to read the bk/bk session. The safest path is to just give them distinct session cookies until that becomes a problem.
In any case, the docs app currently doesn't write to the session anywhere. Setting this is a defensive move so if we start writing to the session in the future, we don't break things.

/cc @matthewd @toolmantim


[EPIC] Balance system

Tasks

[ ] Modify existing views and add balance components
[x] #296
[x] #302
[x] #300

Yeehaw mvp completed


[CHEC-796] Updating Modal Overlay

Added Box Shadow
Added light overlay with option to use either light or dark.

Note:
This PR does not include removing the close button as I need to verify with the design first. Since we are removing the ability to close by clicking off the modal we may be getting ourselves into a bit of a UX footgun.

@blazepollard and I discussed the close buttons and have decided that they will be staying.


igt-gpu-tools: Fix patch error

Remove the patch for scripts/run-tests.sh.
The test list will be passed through the shell environment
variable IGT_TEST_ROOT.
Signed-off-by: Arthur<EMAIL_ADDRESS>

We always build the latest version of igt-gpu-tools. So, this patch is needed by all the branches.

Adding @petegriffin :
I could also propagate this fix to sumo and thud, but it turned out that these two branches don't have igt-gpu-tools at all.

@andrey-konovalov makes sense for Thud, I guess Sumo is a bit old now and not sure it is used by anyone


serveStatic() does not work

What version of Hono are you using?
3.12.7
What runtime/platform is your app running on?
Bun
What steps can reproduce the bug?
I have recently started learning this framework and I am very excited. Today I understood serveStatic() in Bun does not work for me when I use it.
My code:
import { Hono } from 'hono'
import { serveStatic } from 'hono/bun'

const app = new Hono()

app.use('/static/*', serveStatic({ root: './static' }))

I was able to do my work with the following code (in a way contrary to the documentation):
import { Hono } from 'hono'
import { serveStatic } from 'hono/bun'

const app = new Hono()

app.use('*', serveStatic({ root: `${import.meta.dir}/static` }))

export default app

I also changed serveStatic() code to:
// 33 - 45 lines
    let path = getFilePath({
      filename: options.rewriteRequestPath ? options.rewriteRequestPath(filename) : filename,
      root: options.root,
      defaultDocument: DEFAULT_DOCUMENT,
    })

    if (!path) return await next()

    path = `/${path}` // previously path = `./${path}`

    if (existsSync(path)) {
      const content = file(path)
      if (content) {

And now it works fine for me. What do you think is the reason for this problem? Is it a bug or am I making a mistake?
Additional information
Also, I noticed that NodeJS modules are used in this file, I am interested in contributing and rewriting with Bun.

This was a foolish mistake on my part :) However, I made a few changes #2115


Insert dummy annotations in place of optimized out resources

FileOptimizer now should insert empty annotation meta commands in place of the resources that are removed as not referenced in a trimmed capture range. This should keep the block index when replaying an optimized trimmed capture in alignment with the block index calculated at capture time.

CI gfxreconstruct build queued with queue ID 182909.

CI gfxreconstruct build # 4138 running.

CI gfxreconstruct build # 4138 passed.

CI gfxreconstruct build queued with queue ID 186558.

CI gfxreconstruct build # 4184 running.

CI gfxreconstruct build # 4184 passed.


add Box property to discriminate standard box from custom box

Issue:
Custom user-defined box type ids can overlap with standard ones, but we need to discriminate them.
For example in the lambo registry app the CarBox has typeId = 1 , the same  as the standard RegularBox : due to this the following code is not enough to select only the boxes of type CarBox:
if (currentBox.boxTypeId() == CarRegistryBoxesIdsEnum.CarBoxId.id()) {
  ...
}

furthermore, in the wallet/allBoxes endpoint result list is not very clear the difference between the two elements:

           {
               "typeId": 1,
               "value": 42000,
               "nonce":<PHONE_NUMBER>234175154,
                "id": "ac54c0dc93e6cecb4e6cb80efd817b878d9027fca5e2cce14b79b5ec92e0115e",
                "proposition": {
                    "publicKey": "4a2437a1106a9b5912dfc2162180fe5758cfe449f45c72bfdee7ad928a570c0e"
                }
            },
            {
                "typeId": 1,
                "year": 2018,
                "color": "Red",
                "vin": "QWEFGFWRGWSGWR341",
                "model": "Lamborghini Aventador",
                "nonce":<PHONE_NUMBER>242333935,
                "id": "52c120fbe847e9864c8ccad7f27a7cd0f2d1cb38ce7646bb17133c0879425595",
                "proposition": {
                    "publicKey": "4a2437a1106a9b5912dfc2162180fe5758cfe449f45c72bfdee7ad928a570c0e"
                }
            }`

Possible solution:
We need a new  boolean property on the Box interface to discriminate the default box types from the custom ones.
The property should be returned also in the wallet/allBoxes endpoint result list.

It's allowed to keep the same type ids between core types and custom types. The only restriction is that core type ids must be unique between themselves. And the same between custom types. We need it for serialization.
I agree with you, that it's confusing to see such case as you described, so additional boolean property can be a good solution here.

Add 2 fields in json:

"typeName" : , like "ZenBox"
isCustom: true | false
Remove:
"typeId" from json, use only for internal byte serialization needs.
Note: check the usage of type id in gamechain, lambo and zkAudit.

Done


“Error in checkGrep(grep(".A.txt", files))

Hi! I ran into this error:
“Error in checkGrep(grep(".A.txt", files)) :
Improper folder specification; file missing / extra file present. See documentation
Calls: importMito -> checkGrep
Execution halted”
To Reproduce
Steps to reproduce the behavior:
mgatk tenx -i possorted_bam.bam -n atac-08 -o atac-08 -c 30 -bt CB -b cellranger/atac-08/filtered_peak_bc_matrix/barcodes.tsv --keep-temp-files
The same if I use scRNA data:
mgatk tenx -i possorted_bam.bam -n scRNA-09 -o scRNA-09 -c 30 -ub UB -bt CB -b cellranger/scRNA-09/filtered_feature_bc_matrix/barcodes.tsv --keep-temp-files
Thank you very much for your help!

I have the same issue, any work arounds?

Hello there! I ran into the same error. After checked the output folder, I found that  a few of folders that should be created as in line ~302-308 in cli.py were missing. what I did is I changed '/.internal' to /internal' and in OneSample.py, I changed line ~17 config = yaml.load(stream, Loader=yaml.Loader) to from ruamel.yaml import YAML; d = YAML(typ='rt'); config = d.load(stream).         Leave --keep-temp-files in the command so that u can know whether all the folders were made.
HTHs!!!

Hello, I tried doing both the above option, but keep getting the same error. Below is the code I used:

(mgatk_env) abhinavsoni@crtd-sieweke-mbp07 ASAP_seq % mgatk tenx -i possorted_bam.bam -n ASAP_Seq_young -o mgatk_young -b barcodes.tsv -bt CB -g mm10
Output:
Mon Sep 30 15:37:36 CEST 2024: mgatk v0.7.0
Mon Sep 30 15:37:37 CEST 2024: Found bam file: possorted_bam.bam for genotyping.
Mon Sep 30 15:37:37 CEST 2024: Found file of barcodes to be parsed: barcodes.tsv
Mon Sep 30 15:37:37 CEST 2024: User specified mitochondrial genome matches .bam file
Mon Sep 30 15:40:18 CEST 2024: Finished determining/splitting barcodes for genotyping.
Mon Sep 30 15:40:19 CEST 2024: Genotyping samples with 8 threads
Fehler in checkGrep(grep(".A.txt", files)) :
Improper folder specification; file missing / extra file present. See documentation
Ruft auf: importMito -> checkGrep
Ausführung angehalten
`
Is there a workaround for this? or I am doing something weird. Any help will be appreciated. Thank you!

I figured it out, in the end troubleshooting using the log file "ASAP_Seq_young.snakemake_tenx.log" was quite helpful.
For my particular first it showed some error in pulp package, and downgrading with pip3 install --force-reinstall -v "pulp==2.7" did the trick as mentioned in issue #93
There were some other issues post this, however they could be easily resolved by following the log files.


Clicking on link on "October statement"opens the app in web instead of opening it within the current app

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open NewDot
Go to Concierge chat
Click on "October 2022 expensify statement"

Expected Result:
Opens within the app
Actual Result:
Opens in web
Workaround:
unknown
Platform:

Where is this issue occurring?

Web
Desktop App

Version Number: 1.2.22-1
Reproducible in staging?: Need reproduction
Reproducible in production?: Need reproduction
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Expensify/Expensify Issue URL:
Issue reported by: @puneetlath
Slack conversation:
https://expensify.slack.com/archives/C01GTK53T8Q/p1667333672813869
View all open jobs on GitHub

I believe this issue will also be taken care of by https://github.com/Expensify/Expensify/issues/234459

Ah cool. Let's close this then. Thanks y'all.


need console subsystem abstraction for console syscalls

We currently have the printk/stdout/stdin hook registration mechanism which, among other things, doesn't have a good place to implement system calls.
We put something together specific to printk(), but we need to do this for various C library functions as well that emit to the console in both newlib and minimal libc.
A first attempt added syscalls to the C libraries (see #6815 ) but it would be MUCH better if there was a dedicated syscall layer independent of the C library, or specific device drivers. A subsystem like other drivers have, with perhaps some optimizations since we know it is a singleton.
This is both a bug and a feature since the underlying issue is that user threads calling printf(), or newlib read/write, crash.

@andrewboie , what's the status?


Kanban Board Setup

Project link
There are 2 people on our team

Team Member 1: Islam Bahnas
Team member 2: Victor Ebube

Hi @Banstein and @vickymarz,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request ❌

[ ] You have submitted a project, that does not have the link to your Github project. Kindly include that 👍
[ ] Also, the list of GitHub usernames should be assigned to: Team member #1, Team member #2

Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Hi @team,
Status: Approved ✔
Your project is complete!  There is nothing else to say other than... it's time to merge it  🚢
Congratulations! 🎉
Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
You can also consider:

N/A

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Add debug code for the VSOCK test

What this PR does
Before this PR: Flake VSOCK test in #12930
After this PR: Check the if the guest agent executable is used by another process using lsof. This is only used to gather more information for debugging.

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

@zhuchenwang there's a fix for this here that was merged: https://github.com/kubevirt/kubevirt/issues/12930
/close
Please chime in if you disagree


Sphinx doesn't find the theme after being installed

Download the theme via pip and following the docs proposal on:
https://pypi.org/project/sphinx-rtd-theme/
https://sphinx-rtd-theme.readthedocs.io/en/latest/installing.html
this is the result.

Little question:
But I hadn't already installed it via pip ???
No doubt about it, hahahahahahaha works just fine.

There does seem to be some sort of problem with your setup. Even in your screenshot, it looks like Sphinx 1.8.5 is installed but when you run make html it is using Sphinx 1.6.7.

In the screen I show you all the sphinx packages installed in my system with version numbers.
I have no idea where I can get that version 1.8.5 that does not exist in my system or in the LinuxMint repositories.
Okay thanks for the reply. I give up and look for another documentation generator.

@adrix71 you should use a virtual environment to install your dependencies https://docs.python.org/3/library/venv.html

This looks to be an issue with the path of the binary that you are running locally, and not an issue with the theme. Virtualenv installation is likely to solve this issue. We don't support or maintain versions of this theme that were packaged for distributions, package installation is the correct method of installation.
The fact that your make html pass references an old Sphinx version likely means that the system level sphinx-build is being used, and sphinx_rtd_theme is not installed as a system level dependency.
Closing this as it doesn't seem to be a bug with the theme.


No CHANGELOG for this repo

It's time to start a changelog file on this repo for each release. Perhaps we could backfill it between v1.1 and the latest release for v1.4?

Added, as of https://github.com/solidusio/solidus_auth_devise/commit/a236e8b721dfe5c7ccda38e62d548eb6c3bd7ce8. Thanks for the suggestion!.

Thank you!


Simplify label rendering on scene tree tab hover

When we hover over items in the scene tree tab, a label is rendered over the threejs object that's being hovered over.
This is done super manually using a vanilla threejs CSS2DRenderer:

https://github.com/nerfstudio-project/viser/blob/main/viser/client/src/LabelRenderer.tsx
https://github.com/nerfstudio-project/viser/blob/bd2e5e8ffbb185140bcce19f4d44649acdd340d0/viser/client/src/ControlPanel/SceneTreeUI.tsx#L55-L109

I think we can completely delete LabelRenderer.tsx, and dramatically simplify everything by taking advantage of the <Html /> tag in drei.
This would also let use delete the nasty global objFromSceneNodeNameRef (https://github.com/nerfstudio-project/viser/blob/bd2e5e8ffbb185140bcce19f4d44649acdd340d0/viser/client/src/index.tsx#L44-L46).

ill take a crack at it


"Use of undeclared identifier 'SORT_VERSION'" in sort.c

I'm getting Use of undeclared identifier 'SORT_VERSION' when trying to compile the latest ios_system master (with the sort command) in OpenTerm.

That's odd. Lines 3026 to 3028 of text_cmds.patch should say:
+//#ifdef __APPLE__
+//				printf("%s (%s)\n", VERSION, strlen(SORT_VERSION) ? SORT_VERSION : "0");
+//#else

Oh, I should probably re-run the get-frameworks script.


Hyperion fails to build with on RPi2

When compiling the source on the latest Raspbian using the instructions provided, it fails to build here:
[ 46%] Building CXX object libsrc/leddevice/CMakeFiles/leddevice.dir/moc_LedDevicePhilipsHue.cxx.o
In file included from /home/pi/hyperion/build/libsrc/leddevice/moc_LedDevicePhilipsHue.cxx:10:0:
/home/pi/hyperion/build/libsrc/leddevice/../../../libsrc/leddevice/LedDevicePhilipsHue.h:137:72: error: expected primary-expression before ‘{’ token
libsrc/leddevice/CMakeFiles/leddevice.dir/build.make:143: recipe for target 'libsrc/leddevice/CMakeFiles/leddevice.dir/moc_LedDevicePhilipsHue.cxx.o' failed
make[2]: *** [libsrc/leddevice/CMakeFiles/leddevice.dir/moc_LedDevicePhilipsHue.cxx.o] Error 1
CMakeFiles/Makefile2:832: recipe for target 'libsrc/leddevice/CMakeFiles/leddevice.dir/all' failed
make[1]: *** [libsrc/leddevice/CMakeFiles/leddevice.dir/all] Error 2
Makefile:113: recipe for target 'all' failed
make: *** [all] Error 2

The culprit is the following line: LedDevicePhilipsHue.h:137
It could be fixed by replacing it with
			false, int transitiontime = 1, std::vector<unsigned int> lightIds = std::vector<unsigned int>());

Maybe I'll create a pull request later.

Ok, this requires C++0x compatibility which should be supported by Raspbian by now. But keeping it downwards compatible does not hurt.

@ntim I was using the latest Raspbian, I downloaded it yesterday. It still uses a relatively outdated GCC4.6 though, and doesn't fully support c++11.
Either the instructions should be fixed to include updating to some newer gcc version, or (which would be simpler) keep gcc4.6 compatibility.

Extended initializer lists are supported by version 4.6. Can you check if you compiled using the --std=c++0x option?

https://gist.github.com/mrgreywater/33949c82dd356b6bcdb8

Ok, maybe then the old gcc has not the full menu yet regarding the extended initializer lists although the manual suggests something else.

Is this now fixed?

I assume so, since this was the only change I had to do to get it to compile. Thanks for the reminder to close the issue.


fix: complete arguments for each event and export the types of props

All events of this component are missing parameters which making it impossible to obtain the context information when the events are triggered.
For example, the event onStringTyped should have two parameters, arrayPos and self, but it is missing.
// typed.js README.md

/**
 * After each string is typed
 * @param {number} arrayPos
 * @param {Typed} self
 */
onStringTyped: (arrayPos, self) => {},

I have completed parameters to all the events, and then exported the configuration type TypedOptions for this component(re export from typed.js).

Thx for your contribution!


liquidstakeibc - stargaze-1

There was an issue reported via an user about stargaze unbonded token not being distributed post claiming.
Short Analysis:
Unstaked tokens were not relayed in time for stargaze for epoch 582 and timedout,  stargaze timeout tx msg index 2
persistenceCore q lsibc unbondings stargaze-1 --home ~/networks/core --height 17304049
- burn_amount:
    amount: "60621412694"
    denom: stk/ustars
  chain_id: stargaze-1
  epoch_number: "582"
  ibc_sequence_id: channel-197-sequence-1494 // edit this to empty while upgrading
  mature_time: "2024-06-20T12:39:28.817137090Z"
  state: UNBONDING_MATURED // edit this to maturing while upgrading
  unbond_amount:
    amount: "62810179898"
    denom: ustars

And while epoch 591 unstaked tokens were being returned to persistence chain it marked epoch 582 as claimable as well.
This leads to cascading effects as new epochs will be completed and those tokens will be used to fill older requests and so on. To plug this, a Samaritan address deposited enough tokens to the unbonding address ( and the protocol should refund it to the address whenever it can )
Ideally epoch 582 should be artifically re-added via upgrade handler and the tokens receiver should be set to the Samaritan address like
- address: persistence1fp6qhht94pmfdq9h94dvw0tnmnlf2vutnlu7pt // address which refilled the unbonding amount
  chain_id: stargaze-1
  epoch_number: "582"
  stk_amount:
    amount: "60621412694"
    denom: stk/ustars
  unbond_amount:
    amount: "62810179898"
    denom: ustars

while this might hinder some auditability of liquidstakeibc, it ensures smooth operations at chain level.
tx - persistence refill unbonding wallet tx
unbonding module account - persistence1fqr7xxtav8zq2ld4d35py5qyfcqae8qhuvu8zy
user account - persistence1fp6qhht94pmfdq9h94dvw0tnmnlf2vutnlu7pt
stargaze ICA address - stars1pjshpkzt57rl4chmj9zsmhc4409c9tj5d4pekmyfphmlqg0w7w5sge46d8

error logs of unbonding claims failed -
1:50PM ERR could not send unbonded tokens from module account to delegator error="spendable balance<PHONE_NUMBER>ibc/AD8E1D4AC4EA8FC79CC46E33319A3791477D4DEBFC30D5D874074B993422B41B is smaller than<PHONE_NUMBER>ibc/AD8E1D4AC4EA8FC79CC46E33319A3791477D4DEBFC30D5D874074B993422B41B: insufficient funds" epoch=582 host_chain=stargaze-1 module=x/liquidstakeibc workflow=DoClaim
1:50PM ERR could not send unbonded tokens from module account to delegator error="spendable balance<PHONE_NUMBER>ibc/AD8E1D4AC4EA8FC79CC46E33319A3791477D4DEBFC30D5D874074B993422B41B is smaller than<PHONE_NUMBER>8ibc/AD8E1D4AC4EA8FC79CC46E33319A3791477D4DEBFC30D5D874074B993422B41B: insufficient funds" epoch=591 host_chain=stargaze-1 module=x/liquidstakeibc workflow=DoClaim
1:50PM ERR could not send unbonded tokens from module account to delegator error="spendable balance<PHONE_NUMBER>ibc/AD8E1D4AC4EA8FC79CC46E33319A3791477D4DEBFC30D5D874074B993422B41B is smaller than<PHONE_NUMBER>3ibc/AD8E1D4AC4EA8FC79CC46E33319A3791477D4DEBFC30D5D874074B993422B41B: insufficient funds" epoch=591 host_chain=stargaze-1 module=x/liquidstakeibc workflow=DoClaim

claim successful logs after the rifilling of unbond account.
1:50PM INF Claimed unbonding. claimed_amount=4635688132ustars host_chain=stargaze-1 module=x/liquidstakeibc workflow=DoClaim
1:50PM INF Claimed unbonding. claimed_amount=47967734708ustars host_chain=stargaze-1 module=x/liquidstakeibc workflow=DoClaim
1:50PM INF Claimed unbonding. claimed_amount=12531907253ustars host_chain=stargaze-1 module=x/liquidstakeibc workflow=DoClaim

same with two more unbondings
https://www.mintscan.io/persistence/tx/D08E61501CE83B6AFFBA8B9C9C35EB802B1C6FDCD191A409D51D225E84966E1D?height=17755719
https://www.mintscan.io/persistence/tx/6B711087B54333CC493AB62C2C328864EA5AF3944C8A77AA3816C03450A5B93A?height=19393488


Log errors if directives do not start at beginning of line.

Updated tests to validate expectations.
Added two additional tests to validate extensible directives and built-in directives get start at line verification.

#1201

Ya we do. That was the initial goal of the fix. That way there's more structure to directives which enable us and users to understand where they can occur. A great example of this is the @page diretive; for a while we were hand parsing everything and having the white space in front of it didn't do anyone any favors.

Not sure about this one. If we have some custom directive that makes sense only in a scope, it would suck to not allow it to be indented. Or if there is no such thing as a scoped directive, should we probably restrict all directives to be at the start of the file?

Not sure about this one. If we have some custom directive that makes sense only in a scope, it would suck to not allow it to be indented. Or if there is no such thing as a scoped directive, should we probably restrict all directives to be at the start of the file?

@rynowak thoughts?

Not sure about this one. If we have some custom directive that makes sense only in a scope, it would suck to not allow it to be indented.

We should not be whitespace sensitive

Or if there is no such thing as a scoped directive, should we probably restrict all directives to be at the start of the file?

Yeah we need a feature like this. I think there are three parts:

file scoped - one per file, has overriding behavior implemented by lowering (like @model)
multiple - top-level scoped - appears in 'leading' directives, but multiple are allowed (like @inject)
inline - allowed anywhere (like @section)

Enable file scoped extensible directives
Enable multiple occurring extensible directives
Enable inline extensible directives

🆙 📅  and rebased this guy ontop of nimullen/parsercontext

ping.

@DamianEdwards signed off on the breaking change. @rynowak / @muratg this ok to get in for preview2?

@NTaylorMullen Yup, if @DamianEdwards signed off on this, it's OK to check in. Please make sure to tag the original bug with breaking-change.

@NTaylorMullen did you create a breaking change announcement?

Did not, investigating a VS crash issue then I'll create one.


[BUG] mcdev status bar, context menu and dependencies should only be activated in a mcdev project

Describe the bug
mcdev in status bar, dependencies check and the context menu should only be activated if the project is a mcdev or there are in the same folder multiple mcdev projects. If not an error is showing up in the output console when the extension is not in a mcdev project.

is this really blocking for someone that wants to use mcdev? if it's just annoying but non-blocking, please move it to a bug release (1.0.x)


Test static analysis

Maybe related to a cppcheck issue ?
https://trac.cppcheck.net/ticket/13269
Anyway, lets fix it

the auto can be const auto& while you're in there

I'm curious about why it managed to find its way through the CI to the master.


@decorate fails in 0.12.1

in 0.12.0 this code works
"use strict";

import {decorate} from "core-decorators";

function decorator(fn) {
	return function inner(...args) {
		console.log("fn", fn);
		return fn.bind(this)(...args);
	};
}

const obj = new class {

	@decorate(decorator)
	@decorate(decorator)
	test(...args) {
		console.log(...args);
	}

};

obj.test();

in 0.12.1 it throws
test.js:43
                return fn.bind(this).apply(undefined, arguments);
                         ^

TypeError: Cannot read property 'bind' of undefined
    at _class.inner (test.js:8:10)
    at Object.<anonymous> (test.js:22:5)
    at Module._compile (module.js:413:34)

This is a toughy...
The issue is that we switched to lazily applying the decorator using a getter so that we can apply it to the instance instead of the prototype which resolved another bug. The test suite missed it because I wasn't testing chaining multiple @decorate calls. I'm experimenting with some solutions.

@TrejGun I believe I've fixed it with #74, gonna let the code stew for a day. Feel free to try if you can.
You can try it out either by building core-decorators yourself or you can download this zip and extract the contents into your node_modules/core-decorators/*
core-decorators.zip

LGTM, thanks!

will this work with something like
    @autobind
    @decorate(decorator)
    test(...args) {
        console.log(...args);
    }

and
    @decorate(decorator)
    @autobind
    test(...args) {
        console.log(...args);
    }

?

@TrejGun not at the moment (one breaks the other invokes the method pre-emptively internally so be careful)
This whole thing has been an awful juggling between different requests (e.g. #65), so I'm considering reverting back to the old way of decorating the prototype instead and then introducing an option or a new decorator that will then opt-into decorating the instance instead.


Lost focus on 'select2:open' when using 'minimumResultsForSearch' option.

I was having an issue similar to #3279 : I hit enter, dropdown is open, but I'm not able to navigate or search. The focus seems to go to another "place". I had to click and then it becomes navigable/search-able with keyboard.
But I think is the weird think. I another page, this works fine.
So, I noticed that only difference was the minimumResultsForSearch option:
options: {
  minimumResultsForSearch: 7,
  ajax: {
    // . . .
  }
}

When I remove minimumResultsForSearch, then works fine.
For now, I'm disabling minimumResultsForSearch option

PD: Sorry for the incomplete/bare report, I've much time right now. But I will make some tests later and update this report with falling tests to discard or reproduce, because this option is very useful.

Prerequisites

[x] I have searched for similar issues in both open and closed tickets and cannot find a duplicate
[ ] The issue still exists against the latest master branch of Select2
[ ] This is not a usage question (Those should be directed to the community)
[ ] I have attempted to find the simplest possible steps to reproduce the issue
[ ] I have included a failing test as a pull request (Optional)

Environment
Browsers

[x] Google Chrome
[x] Mozilla Firefox
[ ] Internet Explorer

Operating System

[x] Windows
[ ] Mac OS X
[x] Linux
[ ] Mobile

Libraries

jQuery version: 2.2.4
Select2 version: 4.0.3

Without an example to look at here, I'm going to guess that the focus is going to the search box, and then the search box is being hidden while it waits for results?

Can you expand upon the issue you are facing by adding a few more pieces of information to your bug report?

What steps can be used to reproduce the issue?
Does the bug happen consistently across all tested browsers?
Can you reproduce this issue on the examples page?
Are you using Select2 with other plugins?

The contributing guide contains a useful section with a few questions which can help us track down the bug and speed up the process of finding a fix for it.  The creation of an isolated test case would also be useful.

Reproduction of the above issue (I think) with 4.0.5 in Chrome: http://jsfiddle.net/BrfSf/404/
and with 4.0.0: http://jsfiddle.net/BrfSf/405/
Steps to reproduce the defect:

Create a Select2 thang with the minimumResultsForSearch set to fewer than the number of options.
Open the Select2 thang with the 'open' method of the Select2 API.

Expected: If we had a visible search box, we would focus that, but since we don't, I think we should focus the selected element in the results list, but I don't know how to use a screen reader, so what do I know? But I do know that I should be able to use up and down keys to select an item, and be able to Tab / Shift + Tab to other tabbable items on the form.
Actual: The body is focused, up and down do nothing, and users have to tab 1.4 million million times to get back to the selection field (estimation based on modern web site average).
Notice I'm opening the Select2 box with $('select').select2('open'), whereas if I click on the drop-down, or if I Tab to the drop-down and hit Space, focus is not lost and all is well.

If someone can help me understand the strategy for shifting focus, I may be able to fix this. Looking through the code, I don't see any obvious division of responsibilities regarding what components define what event listeners to pass focus onto what elements. That makes the problem seem complicated to me. :laughing:

@shark-horse I think you are missing the $(document).ready(function () { ... } );
Mmm... btw, fiddles seems to work for me in Chrome. Maybe my issue was because I used the Bootstrap theme?
I don't remember. I'l check my project in witch I was using this.

Updated: http://jsfiddle.net/BrfSf/407/
Opening box with search works--I can type in the box and use the up and down arrow keys to change items:

Opening a box without search fails. I have to click the Select box itself, or tab to it and press Space, to start using the arrow keys:

This is with Firefox 56.0 on Ubuntu 16.04. Same behavior (slightly different appearance) in Chrome 62.0.3202.89, same OS.
@nelson6e65 To confirm, you're not clicking on the Select2 drop-down, right? You're just calling Select2's open and then immediately using the arrow keys?
I think I can fix this. The only place where the condition to show or not show search is handled is here:
var showSearch = self.showSearch(params);
if (showSearch) {
  self.$searchContainer.removeClass('select2-search--hide');
} else {
  self.$searchContainer.addClass('select2-search--hide');
}

The whole codebase neither queries for the .select2-search--hide class nor calls showSearch anywhere else. We should at least check whether the search box is visible (or if showSearch()) before focusing it.

@nelson6e65 @kevin-brown Failing test in a branch based on master submitted. I think that satisfies all the prerequisite checkboxes in the original post, but I cannot edit it.

I've been looking at and playing with select2's focus code and event handling today. It looks like focusing the Selection allows keypresses (e.g. up, down, Enter) to work as expected; that's why opening the Select2 field by UI interaction works: Whether we tab to the selection and hit Space, or click on the Select2 element, we are focusing the selection element. So it seems the solution for this defect is to focus the selection element.
It looks like the responsibility of focusing is given the component with the DOM element to focus, not e.g. the element the focus event is triggered on, i.e. everyone steals focus as they see fit. This seems problematic in the case of the dropdown search input (dropdown/search.js) specifically because it handles its own appearance / disappearance, but it has no way of knowing that the element to focus is the selection element--that could break valid extensions of Select2. Seems to me we need another way to determine who gets focus.
I could use some guidance here. Off the top of my head, I like the idea of "parent" components delegating to their "children" via API call, with Select2 (core.js) being the topmost parent since it "knows about" all the other select field-related elements (the select, the selection, and the dropdown).  (Focus can be passed the other way via event triggering, perhaps?) In this example, we might say Select2.prototype.focus would call focus on either its dropdown or its selection based on the multiple option and whether the search element is visible... but I don't think we want Select2 to know that much, eh? What can we do?

I pushed a fix where the dropdown search will call .focus() on the selection element, which it gets by JQuery-ing its $container. :-1: But what else can we do?

I did not figure out how to fix and reopen my previous PR, so I made a new one. Please see above.

@shark-horse can you please rebase off of 4.0.6-rc.0?  This will get folded in to the next pre-release for 4.0.6.

I rebased. Should I submit a PR to merge into the develop branch? I see that 4.0.6-rc.0 is not merged into master, so merging to master would make the PR include a lot of unrelated commits.

Thanks!  Also, I see you've been pretty active on the repo - do you want to drop in to my chat (https://chat.userfrosting.com/channel/select2) to talk about plans for the next release?

This is a workaround I came up with for this issue (coffeescript).
$(el).data('select2').on 'results:all', (e) =>
    context = $(el).data('select2')
    searchContainer = context.dropdown.$searchContainer
    if searchContainer.hasClass('select2-search--hide')
        context.$selection.focus()


Notifications screen is empty for some users

Description
This one is hard to reproduce because it only affects some users. We have not yet been able to determine the cause of the issue, but it is definitely confirmed by multiple users.
Repro steps:

Open the Notifications screen

Expected behavior:

Notifications show up

Actual behavior:

Empty screen shows up with no notifications
Loading indicator doesn't show up either
The notification dot in the bottom nav does show up
Some users have reported that notifications all of a sudden start working, without any special action on their part. When notifications start working, they continue to work reliably; in other words, the problem disappears.

This issue has been resolved on the backend.


Visual Studio crashes when debugging IE

When using angular-dateparser.js version 1.1.1 we get a crash of Visual Studio 2015 when starting Internet Explorer 11 from within the development environment using the 'Start debugging' (F5) command.
We have retrieved the library from the Bower repository and the problem probably has something to do with the angular-dateparser.js.map file (which references the TypeScript sources that are not in the dist folder). Putting the  DateParser.ts and DateParserDirective.ts in the dist folder solves the problem.

@henkdehaan This is most likely due to the sourceroot path pointing to the wrong location. I've added source content into the sourcemaps so you should no longer have this issue. Let me know if you're still having trouble with this.


Meta: address all current content: correction issues

this meta issue address all current content: correction issues
planned commits:

[ ] Correction: Add note about required Npgsql version (fixes https://github.com/crate/crate-npgsql/issues/42)

closing this as a part of my board cleanup. there is only one subtask so no need for a meta issue. I have assigned the subtask to @infoverload as it relates to the client libraries (integrations)


vendor/CodeMirror should be vendor/codemirror

On a fresh checkout, make will fail because it can't find vendor/codemirror. Issue is probably introduced from a case-insensitive FS.

The only reason make is still there is because the tests haven't been moved out. I'll keep this as a cleanup or  remove make issue.

Yeah, I'm not sure how relevant this ticket is. I was a bit confused when I was setting up our new Strider instance.
I was cloning from git directly and ran into this issue. I later just installed from npm and didn't have any problem.
I just thought I should mention it, as it seemed like a mistake related to case sensitivity.
Close at will :)


catalina fixes

Task/Issue URL:
Tech Design URL:
CC:
Description:

Steps to test this PR:
1.
2.

Copy Testing:

[ ] Use of correct apostrophes in new copy, ie ’ rather than '

Orientation Testing:

[ ] Portrait
[ ] Landscape

Device Testing:

[ ] iPhone SE (1st Gen)
[ ] iPhone 8
[ ] iPhone X
[ ] iPhone 14 Pro
[ ] iPad

OS Testing:

[ ] iOS 14
[ ] iOS 15
[ ] iOS 16

Theme Testing:

[ ] Light theme
[ ] Dark theme

Internal references:
Software Engineering Expectations
Technical Design Template

Current dependencies on/for this PR:

develop

PR #1990   👈

This comment was auto-generated by Graphite.


Ability to override Get-UnitySetupInstance's BasePath with an environment variable

It seems like unitysetup.powershell is looking for editors to be installed in very specific locations:  https://github.com/microsoft/unitysetup.powershell/blob/90be58fbaa233a3bb0fec6ce33ca00c20ee509bf/UnitySetup/UnitySetup.psm1#L1204
I have mine configured so they are installing on a different drive. As such, when I run Get-UnitySetupInstance, it doesn't return all the installed editors.

Hey, it's funny how the world works. From what I can tell, I added support for this just 4 days ago in #244, and its available in 5.5.152-develop. Set $env:UNITY_SETUP_INSTANCE_BASEPATH_DEFAULT to a comma separated list of paths, which will be used instead of that default. Let me know if that doesn't work for you for any reason and we can re-open this issue.

Thank you for the speedy response. This is the sort of fix I had in mind.

@jwittner For clarification, I was trying to use the same pattern as $BasePath, using and array of strings like: $env:UNITY_SETUP_INSTANCE_BASEPATH_DEFAULT = @('C:\Program Files*\Unity*', 'C:\Program Files\Unity\Hub\Editor\*'). It wasn't clear that this should be a string of comma separated paths like this: $env:UNITY_SETUP_INSTANCE_BASEPATH_DEFAULT = "C:\Program Files*\Unity*,C:\Program Files\Unity\Hub\Editor\*". I think this should actually support both. If it is an array, it should take the strings like $BasePath already does. The array input seems like it would be more powershell based on how those paths might be created.

If you run help Get-UnitySetupInstance the parameter and environment var support is documented as:
 -BasePath <String[]>
    Under what base patterns should we look for Unity installs?
    Defaults to Unity default locations by platform
    Default can be configured by comma separated paths in $env:UNITY_SETUP_INSTANCE_DEFAULT_BASEPATH

    Required?                    false
    Position?                    1
    Default value
    Accept pipeline input?       false
    Accept wildcard characters?  false

I'm open to improved phrasing to clarify this of course.
Regarding the array, I think I'm ok with supporting $env:UNITY_SETUP_INSTANCE_DEFAULT_BASEPATH as both a csv as string and an array of strings, but if I'm hoping you'll make a new issue for that so it's clear that we have support for what this issue describes in the title so I'll close this again with that understanding. =)

I didn't realize that I could have just used Get-UnitySetupInstance -BasePath @('C:\Program Files*\Unity*', 'C:\Program Files\Unity\Hub\Editor\*'). 🤦 Still, having $env:UNITY_SETUP_INSTANCE_DEFAULT_BASEPATH is a tremendous help and I define it as one of my system variables to make it so that I don't have to manage it each session. Winning.
I was setting it up in PowerShell when I was trying to figure things out, and I assigned the array of strings to $env:UNITY_SETUP_INSTANCE_DEFAULT_BASEPATH and it returned no results. In hindsight, this makes perfect sense. I was effectively trying to use $env:UNITY_SETUP_INSTANCE_DEFAULT_BASEPATH to accomplish what the -BasePath parameter does.
So now I'm not sure what I'm asking makes sense or needs to be considered. The idiomatic PowerShell way would be to create a String[] and pass that in as an argument. Allowing the $env:UNITY_SETUP_INSTANCE_DEFAULT_BASEPATH environment variable to support a PowerShell String[], only makes sense when you create the environment variable from within a shell or as part of the $PROFILE. I still think a case could be made that $env:UNITY_SETUP_INSTANCE_DEFAULT_BASEPATH should take both String[] and StringOfCsv, because for platforms other than Windows modifying your $PROFILE might still be the way to create environment variables, and a string array is easier to manage in a PowerShellesque way.
I'll open it, and you can decide if you want to close it, understanding more of my request.


How to open/close programmatically

What is the preferred way to programmatically  expand/collapse the views? I see no method to do so and i see no way to retrieve the views to fake click events on them. The overall design is very "closed" so i can't get access to anything that could help me, as far as i discovered this lib.

thanks @pythoneer, this functionality is something we are working on. Right now there isn't a public method available to do so. As a hack, you can modify expandParent(ParentObject parentObject, int position) to be public instead of private, but the actual change will require some reworking that we will get to very soon.

Excellent! :D .. i somehow feared that (i noticed that function in the first place). But i use your lib through gradle (compile 'com.bignerdranch.android:expandablerecyclerview:1.0.3'). Just for the record if other ppl needs that til you fixed that – i used reflections to do that.
            final int voteNum = 0; // number of element in the recycler view

            WsGetVotings_Response.WsVoting vote = currentVotings.get(voteNum);
            VotingObject vobj = new VotingObject(vote); // extends ParentObject

            Class<?>[] classArray = new Class<?>[2];
            classArray[0] = ParentObject.class;
            classArray[1] = int.class;

            Object[] parameters = new Object[2];
            parameters[0] = vobj;
            parameters[1] = voteNum;

            Method m = ExpandableRecyclerAdapter.class.getDeclaredMethod("expandParent", classArray);
            m.setAccessible(true);
            m.invoke(votingAdapter, parameters); // votingAdapter extends ExpandableRecyclerAdapter

i hope this helps someone in need :) ... and thank you very much for you quick help!

No problem, good workaround in the meantime. Hopefully we'll get the changes in soon to get rid of that reflection!

Fixed in #67. Will be included in 2.0.0.


Move some common Go stuff up into a new 'ct' package.

Also fixes a few linty errors while I'm at it.

LGTM


Watcher not uploading

Do you read the FAQ?

[ X ] Yes.
[ ] I am going to read now.

Describe the bug
Easy less compiling the css files to the local css folder. Watcher is not uploading. The manual upload works fine.
To Reproduce
Filesystem:
Server:
root

httpdocs

css

Local:
root

css

Settings:
{
  "name": "Name",
  "host": "domain.de",
  "protocol": "sftp",
  "port": 22,
  "username": "**",
  "password": "**",
  "remotePath": "/httpdocs",
  "uploadOnSave": false,
  "watcher": {
    "files": "css/*.css",
    "autoUpload": true,
    "autoDelete": false
  }
}

Thanks for help!

css／＊.css
Pfeifakopf<EMAIL_ADDRESS>于 2019年4月29日周一 00:24写道：

Do you read the FAQ?

[ X ] Yes.
I am going to read now.
https://github.com/liximomo/vscode-sftp/blob/master/FAQ.md

Describe the bug
Easy less compiling the css files to the local css folder. Watcher is not
uploading. The manual upload works fine.
To Reproduce
Filesystem:
Server:
root

httpdocs

css

Local:
root

css

Settings:
{
"name": "Name",
"host": "domain.de",
"protocol": "sftp",
"port": 22,
"username": "user",
"password": "*****
", "remotePath": "/httpdocs", "uploadOnSave": false, "watcher": {
"files": "css/.css",
"autoUpload": true,
"autoDelete": false
}
}
Thanks for help!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/liximomo/vscode-sftp/issues/547, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACK4RIJGRMYFCT3PXX3XFZTPSXFVPANCNFSM4HI63MBA
.

css／＊.css
Sorry, i don't understand...

files: "css/*.css"
Pfeifakopf<EMAIL_ADDRESS>于 2019年4月29日周一 18:33写道：

css／＊.css
Sorry, i don't understand...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/liximomo/vscode-sftp/issues/547#issuecomment-487531294,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACK4RIMYTPSAERB5YHFJ63TPS3FGXANCNFSM4HI63MBA
.

If found the problem now. Watcher is not working, if there is no open Windows Explorer window of the watching directory.  If you open a window of the watching direcotory, all works fine. Perhaps this helps you.

Hi! I also have some trouble auto uploading a compiled CSS file. I have an extension that compiles a css file based on my less file. But when the css file is generated (less file is saved) the less file is uploaded OK (if that's what you want...) but the css file does not get uploaded. IS this because the less file is generated and nog saved by the user?
Is there a way to auto upload the generated css file?

I think I've got it working... In stead of the 'uploadOnSave' (now set to false) I used the watcher setting to wacht for file changes. This seems to work. Great.

Hi! I think I now ran into the same issue Pfeifakopf had...  I am starting out with VS code and set up a testing project to test some features / extensions like less compiling andSFTP upload. In my test project everythink works fine and the SFTP extension watches the CSS folder (with the compiled css form the less extension) and uploas it on change. Exactly as I wanted. Great.
So now I wanted to use this in an exsisiting project.  But the wachted folder (css) did nt trigger the SFTP upload when a new css file was compiled. I did exactly what I did with my test project.
But then I realised there is a difference: The VSC project folder is on a shared folder (Window 10 / Synology NAS). And that somehow cuases issues with the watcher.
The I read Pfeiakopfs comments again and tried this:
In windows, I openend an explorer window exactly on the folder where the wactcher was set to watch. And nog the css does get uploaded! As soon as I select a different folder in the windows explorer, it stopped working.
How weird is that ;-)
So I can get this to work, but I need to have a woindows explorer opened on the watched folder every time I want to use my project.
So my question is: Can this be somehow changed / fixed in the SFTP extension (or in a different way) so we don't have to open this explorer window every time we work on our projects?

I stoped using the NAS shared folder for the local project because of this issue. I use the lcal HD now. This works fine.
@rafaelperozin Are you storing your local file son a shared drive. If so that might be the issue. If not it something else.

toring your local file son a shared drive. If so that might be the issue. If not it

@jjnxpct sorry, I'm setting a wrong remotePath.

Thanks, @Pfeifakopf, @liximomo, and @jjnxpct! These tips were helpful; I was able to resolve this same issue for myself.
What really seemed to solve the issue for me was to clear my cache.
I use Cloudflare for all my sites (it's a great, freemium service). I set my Cloudflare to dev mode so I can see my code changes on a browser right away.
Hope this helps! Onelove.

Hey there,
having the same problem on my Mac as well as on my Windows laptop.
Here's my config:
"watcher": {
  "files": "**/Resources/Public/Frontend/css/theme.min.css",
  "autoUpload": true,
  "uploadOnSave": true,
  "autoDelete": false
}

Any idea?

This worked very well for me
"uploadOnSave": true,
"watcher": {
    "files": "**",
    "autoUpload": true,
    "autoDelete": true
},

for future reference if you came from google like me:
"watcher": {
"files": "**/css/*{.css,.map}",
}

Source

I'm new to the SFTP extension and have configured it as recommended and then using pearlboy's suggestion. Neither seems to be working. It works great when going from the PC to the host but does not recognize anything created on the host. Open to suggestions...

I'm new to the SFTP extension and have configured it as recommended and then using pearlboy's suggestion. Neither seems to be working. It works great when going from the PC to the host but does not recognize anything created on the host. Open to suggestions...

well, setting up a watcher on the server isn't a task here.
@liximomo should close this issue

@JeffTeter  This extension is deprecated. Use the SFTP extension instead.
Maintained and updated version by @Natizyskunk 😀
(Forked from the no longer maintained liximomo's SFTP plugin)

Thanks for the response. I thought I was on the right extension.
I still have a question... I have added the watcher section per issue 136.
But, if I modify an item on the remote, the change does not appear in the
local until I refresh. Is that the expected behavior?
On Fri, Feb 10, 2023, 7:09 PM Md. Sumon Islam @.***>
wrote:

@JeffTeter https://github.com/JeffTeter This extension is deprecated.
Use the SFTP https://github.com/Natizyskunk/vscode-sftp extension
instead.
Maintained and updated version by @Natizyskunk
https://github.com/Natizyskunk/ 😀
(Forked from the no longer maintained liximomo's SFTP plugin
https://github.com/liximomo/vscode-sftp.git)
—
Reply to this email directly, view it on GitHub
https://github.com/liximomo/vscode-sftp/issues/547#issuecomment-1426551038,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADDSGWXAULKHZU67DJO5NVTWW3RDBANCNFSM4HI63MBA
.
You are receiving this because you were mentioned.Message ID:
@.***>


ARROW-190: Python: Provide installable sdist builds

This will need to include the license and some other things, or are those picked up automatically? here's the manifest for kudu-python for example
https://github.com/apache/incubator-kudu/blob/master/python/MANIFEST.in

LICENSE and README were missing, everything else was picked up.
Also added the exclude list from Kudu.

+1, thank you


How do I merge the audio and video?

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2<IP_ADDRESS> (update instructions) or later (specify commit)
[X] I've searched the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Question
youtube-dl [link] or youtube-dl --merge-output-format mkv [link] downloads the audio and video files separately, can't this youtube-dl merge the files?
Verbose log
No response

Install ffmpeg if you haven't already. Then show verbose log

Also, this issue tracker is for yt-dlp. If you would like to report a bug for youtube-dl, you can report it at https://github.com/ytdl-org/youtube-dl/issues

Also, this issue tracker is for yt-dlp. If you would like to report a bug for youtube-dl, you can report it at https://github.com/ytdl-org/youtube-dl/issues

I renamed yt-dlp.exe to youtube-dl.exe.

Install ffmpeg if you haven't already. Then show verbose log

That fixed it