Command execution timeout for command: (SENTINEL GET-MASTER-ADDR-BY-NAME)

Hello,
first of all thanks for this great project!
On production, we are getting a lot of these errors, while running the Redis sentinel. The Redis nodes don't show any signs of errors or downtime. Is there any way I can maybe instruct Redisson to retry these commands or can someone point me to the right direction to debug this?
I suspect these errors for being cause of our cache is not being evicted correctly which is super painful hard to debug issue, since the Stack trace doesn't say much.
org.redisson.client.RedisTimeoutException: Command execution timeout for command: (SENTINEL GET-MASTER-ADDR-BY-NAME), params: [mymaster], Redis client: [addr=redis://<IP_ADDRESS>:26379]

org.redisson.client.RedisTimeoutException: Command execution timeout for command: (SENTINEL GET-MASTER-ADDR-BY-NAME), params: [mymaster], Redis client: [addr=redis://<IP_ADDRESS>:26379]
    at org.redisson.client.RedisConnection.lambda$async$0(RedisConnection.java:253)
    at io.netty.util.HashedWheelTimer$HashedWheelTimeout.run(HashedWheelTimer.java:715)
    at io.netty.util.concurrent.ImmediateExecutor.execute(ImmediateExecutor.java:34)
    at io.netty.util.HashedWheelTimer$HashedWheelTimeout.expire(HashedWheelTimer.java:703)
    at io.netty.util.HashedWheelTimer$HashedWheelBucket.expireTimeouts(HashedWheelTimer.java:790)
    at io.netty.util.HashedWheelTimer$Worker.run(HashedWheelTimer.java:503)
    at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
    at java.lang.Thread.run(Thread.java:833)
java.util.concurrent.CompletionException: org.redisson.client.RedisTimeoutException: Command execution timeout for command: (SENTINEL GET-MASTER-ADDR-BY-NAME), params: [mymaster], Redis client: [addr=redis://<IP_ADDRESS>:26379]
    at java.util.concurrent.CompletableFuture.encodeThrowable(CompletableFuture.java:332)
    at java.util.concurrent.CompletableFuture.completeThrowable(CompletableFuture.java:347)
    at java.util.concurrent.CompletableFuture$UniCompose.tryFire(CompletableFuture.java:1141)
    at java.util.concurrent.CompletableFuture.postComplete(CompletableFuture.java:510)
    at java.util.concurrent.CompletableFuture.completeExceptionally(CompletableFuture.java:2162)
    at org.redisson.client.RedisConnection.lambda$async$0(RedisConnection.java:254)
    at io.netty.util.HashedWheelTimer$HashedWheelTimeout.run(HashedWheelTimer.java:715)
    at io.netty.util.concurrent.ImmediateExecutor.execute(ImmediateExecutor.java:34)
    at io.netty.util.HashedWheelTimer$HashedWheelTimeout.expire(HashedWheelTimer.java:703)
    at io.netty.util.HashedWheelTimer$HashedWheelBucket.expireTimeouts(HashedWheelTimer.java:790)
    at io.netty.util.HashedWheelTimer$Worker.run(HashedWheelTimer.java:503)
    at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
    at java.lang.Thread.run(Thread.java:833)

Please refer to this section https://github.com/redisson/redisson/wiki/16.-FAQ#q-what-is-the-cause-of-redistimeoutexception

Thanks. I am still not sure why exactly is GET-MASTER-ADDR-BY-NAME executed? Is this happening periodically to get new master or does this signalize that there is a new master? Also, when I increase the retryAttempts  will it try the operation with other client or it will query the same client again?
The docs say there are 3 retryAttempts by default. We have 3 replicas running, so it seems unlikely that any replica doesn't respond in the default 3000ms in 3 attempts.

The same happens for us. Redis is deployed by bitnami helm chart. 3 instances.
example of sentinel.conf
sentinel known-replica de-sqcx-test-master redis-sentinel-node-0.redis-sentinel-headless.de-sqcx-test.svc.cluster.local 6379
sentinel known-sentinel de-sqcx-test-master redis-sentinel-node-2.redis-sentinel-headless.de-sqcx-test.svc.cluster.local 26379 5f4ff5b1f3024ce375ab027ac24adb717305f02c
sentinel known-sentinel de-sqcx-test-master redis-sentinel-node-1.redis-sentinel-headless.de-sqcx-test.svc.cluster.local 26379 5add24b6f14969527b26e2b3d62ebfa73e2de554
sentinel known-replica de-sqcx-test-master redis-sentinel-node-1.redis-sentinel-headless.de-sqcx-test.svc.cluster.local 6379
sentinel announce-hostnames yes
sentinel resolve-hostnames yes
sentinel announce-port 26379
sentinel announce-ip "redis-sentinel-node-0.redis-sentinel-headless.de-sqcx-test.svc.cluster.local"

and exception:
{"@timestamp":"2024-07-31T14:36:37.870Z","log.level":"ERROR","message":"Can't execute SENTINEL commands on <IP_ADDRESS>/<IP_ADDRESS>:26379","ecs.version": "1.2.0","service.name":"sqcx-s
ervice-snapshot-maker-trigger","event.dataset":"sqcx-service-snapshot-maker-trigger","process.thread.name":"redisson-timer-5-1","log.logger":"org.redisson.connection.SentinelConnectionManag
er","error.type":"java.util.concurrent.CompletionException","error.message":"org.redisson.client.RedisTimeoutException: Command execution timeout for command: (SENTINEL GET-MASTER-ADDR-BY-N
AME), params: [de-sqcx-test-master], Redis client: [addr=redis://<IP_ADDRESS>:26379]","error.stack_trace":"java.util.concurrent.CompletionException: org.redisson.client.RedisTimeoutExcept
ion: Command execution timeout for command: (SENTINEL GET-MASTER-ADDR-BY-NAME), params: [de-sqcx-test-master], Redis client: [addr=redis://<IP_ADDRESS>:26379]\n\tat java.base/java.util.co
ncurrent.CompletableFuture.encodeThrowable(CompletableFuture.java:332)\n\tat java.base/java.util.concurrent.CompletableFuture.completeThrowable(CompletableFuture.java:347)\n\tat java.base/j
ava.util.concurrent.CompletableFuture$UniCompose.tryFire(CompletableFuture.java:1141)\n\tat java.base/java.util.concurrent.CompletableFuture.postComplete(CompletableFuture.java:510)\n\tat j
ava.base/java.util.concurrent.CompletableFuture.completeExceptionally(CompletableFuture.java:2194)\n\tat org.redisson.client.RedisConnection.lambda$async$0(RedisConnection.java:254)\n\tat i
o.netty.util.HashedWheelTimer$HashedWheelTimeout.run(HashedWheelTimer.java:706)\n\tat io.netty.util.concurrent.ImmediateExecutor.execute(ImmediateExecutor.java:34)\n\tat io.netty.util.Hashe
dWheelTimer$HashedWheelTimeout.expire(HashedWheelTimer.java:694)\n\tat io.netty.util.HashedWheelTimer$HashedWheelBucket.expireTimeouts(HashedWheelTimer.java:781)\n\tat io.netty.util.HashedW
heelTimer$Worker.run(HashedWheelTimer.java:494)\n\tat io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)\n\tat java.base/java.lang.Thread.run(Thread.java:
1583)\nCaused by: org.redisson.client.RedisTimeoutException: Command execution timeout for command: (SENTINEL GET-MASTER-ADDR-BY-NAME), params: [de-sqcx-test-master], Redis client: [addr=re
dis://<IP_ADDRESS>:26379]\n\tat org.redisson.client.RedisConnection.lambda$async$0(RedisConnection.java:253)\n\t... 7 more\n"}

@JanCizmar was you able to solve the problem?

We have exactly the same problem.
Redis is deployed by bitnami helm chart. 3 instances.
Redisson 3.25.0

@ivan-sirosh Hi Ivan, sorry for the delay.
No, it still happens time to time and it's annoying since it may cause inconsistencies between the database and Redis, but we currently don't have the time budget to dig into it. I hope it will be fixed in future releases. 🚀

Fixed. Thanks for report.


Improve wandb tests

The tests of the wandb-logger are not as comprehensive as for the rest of the loggers. Currently, they just test if constructing a logger works.
Testing wandb in the same way as the other loggers works:
def test_wandb_logger(tmpdir):
    """Verify that basic functionality of wandb logger works."""
    tutils.reset_seed()

    wandb_dir = os.path.join(tmpdir, "wandb")
    logger = WandbLogger(save_dir=wandb_dir, anonymous=True, offline=True)

    hparams = tutils.get_hparams()
    model = LightningTestModel(hparams)

    trainer_options = dict(
        default_save_path=tmpdir,
        max_epochs=1,
        train_percent_check=0.05,
        logger=logger
    )

    trainer = Trainer(**trainer_options)
    result = trainer.fit(model)

    print('result finished')
    assert result == 1, "Training failed"

def test_wandb_pickle(tmpdir):
    """Verify that pickling trainer with wandb logger works."""
    tutils.reset_seed()

    wandb_dir = str(tmpdir)
    logger = WandbLogger(save_dir=wandb_dir, anonymous=True)
    trainer_options = dict(
        default_save_path=tmpdir,
        max_epochs=1,
        logger=logger
    )

    trainer = Trainer(**trainer_options)
    pkl_bytes = pickle.dumps(trainer)
    trainer2 = pickle.loads(pkl_bytes)
    trainer2.logger.log_metrics({"acc": 1.0})

But the CI tests on github for windows fail. The tests for Ubuntu and Mac work. Seems to be some problem related to file-access that I cannot recreate locally

@ghost mind send a PR? :raccoon:

I think you guys fixed it here: https://github.com/PyTorchLightning/pytorch-lightning/commit/f862d9f6915881e3f4048586a16ef607e5640d26

@Borda can we close this?

can we close this?

not yeat as wandb is not included in the stand test suite
https://github.com/PyTorchLightning/pytorch-lightning/blob/9e8716afe8bd535ecb468027a8a4c919a3ae16d1/tests/loggers/test_all.py#L29
https://github.com/PyTorchLightning/pytorch-lightning/blob/9e8716afe8bd535ecb468027a8a4c919a3ae16d1/tests/loggers/test_all.py#L76

The wandb logger is now included in the other tests as well and the todos that @Borda pointed out are gone.
It is tested also for pickling and folder structure / paths for logged files.
I would argue it is now at the similar level as other loggers. Shall we close?


Example usage of custom connection keys

Description of your changes
This PR is just a demonstration for https://github.com/crossplane-contrib/terrajet/pull/121
I have:

[ ] Read and followed Crossplane's contribution process.
[ ] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

@turkenh do you still want to get this merged?

@muvaf no, this was indeed for demonstration purposes.


doc: csi driver object options

          @iPraveenParihar  Can you document csidriver options in https://github.com/ceph/ceph-csi/tree/devel/charts/ceph-csi-rbd#configuration docs too(cephfs helm docs too) ?

Originally posted by @Rakshith-R in https://github.com/ceph/ceph-csi/issues/4519#issuecomment-2026743730
Checklist:

[x] Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
[x] Reviewed the developer guide on Submitting a Pull Request
[x] Pending release notes updated with breaking and/or notable changes for the next major release.
[x] Documentation has been updated, if necessary.
[x] Unit tests have been added, if necessary.
[x] Integration tests have been added, if necessary.

Show available bot commands
These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

/retest ci/centos/<job-name>: retest the <job-name> after unrelated
failure (please report the failure too!)

@mergifyio queue


[Feature Request] Android app > Use speech API to say last\best times

I think it will be great to hear last lap time once app got it from Chorus device.
Also it will be useful to hear best lap time right after last lap time.
smth like "Last lap: 23.6 sec... Best lap: 18.2 sec"
For the new best lap it will be just like "New Best lap: 15.8 sec"

I agree this is a great idea. It  is a very beneficial feature to hear your lap times called out as you are flying with the goggles on!
Another open source lap timer I built does this. It runs in the web browser though, and I much prefer the Android app interface you have built for this one.

https://github.com/voroshkov/Chorus-RF-Laptimer/pull/10


Update grafana content to reflect current practices

Why
https://trello.com/c/CxkcgEk2/453-update-our-grafana-documentation-for-our-users

It is rare that teams are using the GDS default dashboard because
they are now creating their own dashboards in their team folders and using those
We also have other sources of metrics meaning teams may also
use the container metrics dashboard or dashboards for tools like
elasticsearch.

What

Added a short explanation for how team folders are organised
Added definitions of the purpose of the dashboards in the
General folder
Moved the detailed explanation of the default application dashboard
into it's own page (but have made no content changes)
Removed some of the simpler content as I don't think we have seen
evidence of users struggling to use basic Grafana functionality so far

Approach

I have kept the section on team dashboard permissions brief
and vague - essentially contact us for help - as I think we need
to do more on team permissions. At the moment, teams will not
be able to manage the permissions for users of their team without
having admin privileges of the whole system as per
https://github.com/grafana/grafana/issues/11584. This may be something
we want to investigate further to avoid us needing to manage all
user permissions in terms of which teams they can access and instead
being able to put this burden on the teams themselves.
I moved the detailed explanation of the default GDS application dashboard to avoid
cluttering the overview of grafana page too much. It is also questionable
how useful this page is for our users (my hypothesis is that it is not)
but I am not confident enough to just remove it so have instead moved it.

@idavidmcdonald I have updated the PR with some edits and a few questions\suggestions.
My other thought was to just remove the detailed explanation of the default GDS application dashboard page as part of this PR. It would be a decisive way to test your hypothesis and it could always be added back easily enough if it's something people still want.

@idavidmcdonald I have updated the guidance based on feedback from 2i peer review, these suggestions are based on the house style and things we find work with other guidance. These are mostly small changes.
I do think we should remove the remaining three screen images from the guidance. I think what we have written is clear and the Grafana interface is easy to navigate. It also means there's no overhead keeping images up to date, for example, the login screen has already changed from the current image.
Thanks,
Pete


pixie: migrate to boneyard

Created with brew boneyard-formula-pr because build is broken due to outdated zlib, last release was 19 May 2009.

for reference here is a snippet of the failure:
...
In file included from rendererContext.cpp:77:
./ribOut.h:261:36: error: no matching function for call to 'gzwrite'
        if (outputCompressed)   gzwrite(outFile,scratch,l);
                                                 ^~~~~~~
.../MacOSX10.12.sdk/usr/include/zlib.h:1351:21: note: candidate function not viable: \
no known conversion from 'FILE *' (aka '__sFILE *') to 'gzFile' (aka 'gzFile_s *') \
for 1st argument
ZEXTERN int ZEXPORT gzwrite OF((gzFile file,
                    ^
/bin/sh ../../libtool --tag=CXX   --mode=compile clang++ -DHAVE_CONFIG_H -I. -I../.. -I.. \
-I/usr/local/Cellar/openexr/2.2.0/include/OpenEXR -I/usr/local/Cellar/ilmbase/2.2.0/include/OpenEXR \
-g -O2 -c -o rendererFiles.lo rendererFiles.cpp
2 errors generated.
make[3]: *** [rendererContext.lo] Error 1
...

@MikeMcQuaid ping you on this, it is low priority, but I think it can be killed with 🔥. The developer has not maintained it at all since 2009 and has said as much; furthermore it is completely broken and in x11 so it doesn't even have still functional bottles, and again, x11. 😸

@JCount 👍 to boneyarding from me.

Thanks again @JCount!


getPage

A get Page function is missing. For each comPort there should a variable for the recent page number. Also a callback for page change is needed and a dedicated setPage function.

Any news?

Not in the moment. I wrote some code but there are some bugs I need to sove.

I've updated my preinit solution (printh 0x24 pageID FF FF FF) and added getter functions. Maybe we can merge.


update tags

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
``
Errors:
Paste here the error message you have received. Make sure to remove all sensitive information,
such as user name, password, credential, subscription ID, etc.

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
``

Expected Behavior
Environment Summary
Windows-10-10.0.19045-SP0
Python 3.10.10
Installer:

azure-cli 2.47.0

Extensions:
interactive 0.4.6

Dependencies:
msal 1.20.0
azure-mgmt-resource 22.0.0

Additional Context

Thank you for opening this issue, we will look into it.

Please provide detailed CLI command and error log here.


Feat: Publisher - show incoming/outgoing transactions (like we do on home)

Is your feature request related to a problem?
Publisher transaction-list only shows outgoing txs:

Describe the solution you'd like
If possible it would be good to show incoming and outgoing txs per publisher.
Similiar to what we do on the home-screen:

Describe alternatives you've considered
none
Additional context
fyi: We do not know where the incoming ones came from

I did a lnurl-withdraw on lnmarkets and received the funds, but then I opened Alby and I only see funds sent

Are you working on this?
No

First finding:
The current getuserinvoices (LND) API call result is not showing anything related to the host:
{
    "r_hash": {
        "type": "Buffer",
        "data": [
            7,
            14,
            ...
        ]
    },
    "payment_hash": "123",
    "payment_request": "lnbc123",
    "description": "Withdrawal for [USER] on SN",
    "pay_req": "lnbc123",
    "timestamp":<PHONE_NUMBER>,
    "type": "user_invoice",
    "expire_time": 86400,
    "amt": 10,
    "ispaid": true,
    "keysend": false,
    "custom_records": null
}

Filtering incoming txs via a description sounds like a bad idea.


Commenting multiple line in visual mode inserts extra slashes

First of all, excellent project!
The visual mode adds multi cursors automatically, when I select multiple lines and commenting with cmd+/, two sets slashes will be inserted and cannot uncomment once.
I hardly use multi-cursors before, so I'm not sure adding multi-cursor as a default behavior is suitable for most users.

Related to #59 & #61 . Multi-cursors will be disabled by default in next version

Is there a way to comment with multi cursors enabled?

@codeincontext
Try these mappings:
xmap gc  <Plug>VSCodeCommentary
nmap gc  <Plug>VSCodeCommentary
omap gc  <Plug>VSCodeCommentary
nmap gcc <Plug>VSCodeCommentaryLine

then gc while being in visual mode should do the trick.

Unfortunately this is not fixable due to cursor pos differences. In vscode cursor pos is actual before the character, in vim it's on character. This is not problem for right-2-left visual selections but for left-2-right one character is not being selected due to this. I'm workarounding this problem by creating second selection for 1 right character, that's why commenting through vscode command produces multiple comments (because there are 2 cursors on line for left-2-right visual selection).
The solution is not to use builtin vscode comment key and use xmap gc  <Plug>VSCodeCommentary bound to some key.
As workaround i'll remap <C-/> to call VSCodeCommentary

@asvetliakov great, that's really helpful. Thank you for taking the time to investigate ⚡

May using vscode' s ctrl / in insert mode is a better way (without tweak)
I fail with this:

@codeincontext Try these mappings:
xmap gc  <Plug>VSCodeCommentary
nmap gc  <Plug>VSCodeCommentary
omap gc  <Plug>VSCodeCommentary
nmap gcc <Plug>VSCodeCommentaryLine

then gc while being in visual mode should do the trick.


[DO NOT MERGE] Spike - Using the public layout components in Static

This branch is the (ongoing) outcome of spiking implementing the public layout components from the govuk_publishing_components gem. This work was done as part of the drive to reduce legacy and adopt more of the design system across gov.uk to make the site comply better with accessibility standards and to ease the use of the design system and components. This was done by the Frontend and Accessibility mission team which, as of April 2020, is now paused.
The step by step of the work done is documented here:
https://docs.google.com/document/d/1GbuOlFpLxDNFbcC8QbnLdyZBxwu_Bfl5qGcAAXxqCr0/edit#
and it is covered under this ticket:
https://trello.com/c/uEIU7R4m/238-switch-layout-template-for-public-template-in-static
This branch is not intended for merge, but is meant to be a disposable example of an attempt at implementation. At this point tests are failing and haven't been addressed.
The google document linked has more detail but as a TLDR this branch:

Implements a parallel set of templates that can be called by setting a template variable in a frontend app.
Pulls in the public layout component from govuk_publishing_components (currently only implemented on a branch) which in turn uses a header and footer component
Removes chunks of the layout code from static's govuk_template implementation

More work is needed, again outlined in the linked document, mostly around:

how to ensure all relevant parameters pass through the chain and get rendered, and no spurious code is left outstanding.
how to apply correct css, removing outdated code and ensuring correct layout, spacing, fonts etc

Can we close this pull request? I think it's covered by https://github.com/alphagov/static/pull/2411.

Closing, superseded elsewhere.


[FLINK-18661][Kinesis] Stream consumer Registration/Deregistration

What is the purpose of the change
Adding StreamConsumerRegistrar that is responsible for registration and deregistration of EFO stream consumers. There are 3 strategies:
Registration

EAGER: Registration occurs in FlinkKinesisConsumer constructor
LAZY: Registration occurs in KinesisDataFetcher constructor
NONE: Application passes in ARN, no registration performed

Deregistration

EAGER|LAZY: Deregistration occurs in KinesisDataFetcher shutdown
NONE: No deregistration performed

Brief change log

StreamConsumerRegistrar

Added class to perform the registration and deregistration business logic

KinesisProxyV2

Updated to invoke AWS services related to stream registration:

describeStream
describeStreamConsumer
registerStreamConsumer
deregisterStreamConsumer

FlinkKinesisConsumer

Invokes StreamConsumerRegistrarUtil#eagerlyRegisterStreamConsumers in constructor

KinesisDataFetcher

Invokes StreamConsumerRegistrarUtil#lazilyRegisterStreamConsumers in constructor
Invokes StreamConsumerRegistrarUtil#deregisterStreamConsumers in shutdown

FanOutRecordPublisherConfiguration

Added new config for registration/deregistration timeout to prevent registrar spinning for too long

Added a new RuntimeException hierarchy for semantic errors thrown from Kinesis connector, rooted by FlinkKinesisException

Verifying this change
This change added tests and can be verified as follows:

Running the following unit tests:

KinesisDataFetcherTest
FanOutRecordPublisherConfigurationTest
StreamConsumerRegistrarTest
KinesisProxyV2Test
AwsV2UtilTest
KinesisConfigUtilTest
StreamConsumerRegistrarUtilTest

Fake Kinesis behaviours modelled to emulate server:

FakeKinesisFanOutBehavioursFactory

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): yes (backwards compatible change made to FlinkKinesisConsumer)
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? There is a follow up task for documentation

Merging this now ...


alias for schema tests macros

resolves #2397
Description
Added aliases to columns returned from schema_tests macros. This will make the macros work on adapters required columnnames in their driver such as dbt-sqlserver with it's pyodbc driver
Checklist

[X] I have signed the CLA
[X] I have run this code in development and it appears to resolve the stated issue
[X] This PR includes tests, or tests are not required/relevant for this PR
[X] I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

FYI I also cooked up this issue so that you could override the core schema tests in dbt-sqlserver directly if you needed to in the future https://github.com/fishtown-analytics/dbt/issues/2415

Thanks!


Update package.json

When you run pnpm lint, 'SKIP_ENV_VALIDATION' will throw a 'not recognized command' error, adding SET on the script of lint inside nextjs app package.json solves it.

Both yours and the original command are platform-specific solutions. Your command doesn't work on Linux and the original does not work on Windows. I guess dotenv-cli should be used like in the root package.json.

We can use this opportunity to solve another issue at the same time. Currently, running "pnpm --filter nextjs dev", as instructed in the README to run nextjs seperately, does not work because of env variables. Maybe we can add a dotenv script in each app's package.json to make them work individually and also solve this by running "dotenv -v SKIP_ENV_VALIDATION=1 next lint"

We can use this opportunity to solve another issue at the same time. Currently, running "pnpm --filter nextjs dev", as instructed in the README to run nextjs seperately, does not work because of env variables. Maybe we can add a dotenv script in each app's package.json to make them work individually and also solve this by running "dotenv -v SKIP_ENV_VALIDATION=1 next lint"

Yea I talked to Anthony from Turbo about this and they are cooking up something to fix our needs here. I had dotenv in the apps before but moved cause their docs suggested you keep it in root... Feel free to open a PR switching it back


Reccurrence seems not worked

Just downloaded ultralist via go get github.com/ultralist/ultralist
When i add a reccurrent task like in example from the docs ultralist add Daily standup recur:weekdays the task appears in listing like

4      [ ]                     Daily standup recur:weekdays

If i want to modify reccurrence, via ultralist e 4 recur:monthly all task text just replaced by recur:monthly

Seems like go get github.com/ultralist/ultralist doesnt use latest release. So i just build from latest sources and now recur works.


directory tree support

Support serving a full directory tree from the local filesystem (and eventually also from data baked into the binary for easy release builds). Some things I'd like it to do:

be fairly secure. At least avoid ..-style directory traversal attacks. Possibly an equivalent of nginx's disable_symlinks.
MIME types. At least extension-based guessing (there's a mime_guess crate for this), preferably something pluggable.
indefinite Expires headers for files with content-hashed filenames (see WebPack caching for example).
directory listings, probably in a customizable way
automatic gzip handling (if browser specifies Accept-Encoding: gzip, look for a file ending in .gz first and serve that with Content-Encoding: gzip)

I have a work in progress. It depends on nix-rust/nix#916.

I've finally committed a version of this on the dir branch. It works on Unix. Not on Windows (and I forgot to disable the feature in my .travis.yml there so the CI build is currently failing on that branch).
@lnicola is this a feature you're interested in working on Windows? You contributed the crate's initial Windows support in #10. I don't know how the Windows APIs compare. My Unix code opens the base directory's descriptor and uses libc::openat to find stuff within it. It also builds paths as C strings; I know Windows does some UTF-16 thing instead. And example/serve_dir.rs uses nix::dir which also wouldn't work so well.

Sorry, I haven't looked into this yet (but I did notice the recent changes on its feature branch).
I don't think I need it. My use case is a server that does directory listings and archiving for download (as TAR currently, because they can be streamed, have a relatively predictable size and I didn't really need compression yet).


ArrayHelper check if object key exist

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Tests pass?
❌

Fixed issues
comma-separated list of tickets # fixed by the PR, if any

It is expected to be as it is:

// this is expected to fail if the property does not exist, or __get() is not implemented
// it is not reliably possible to check whether a property is accessible beforehand


[Bug] Speculative decoding small draft doesn't work on macOS

🐛 Bug
I tried to use Mistral Small 7B Instruct v0.3 as draft model for Mistral Large 2407. When not served using "--mode server", the model(s) never respond. I think that's because only CPU is used, instead of GPU as well. When serving with "--mode server", I see that the first token is streamed in the frontend, but then I get the following error: Check failed: (!mstates[i]->draft_output_tokens.empty()) is false.
To Reproduce
Steps to reproduce the behavior:

Download Mistral Large 2407
Quantize model and gen config
Run Mistral Large to see if it works standalone
Run the speculative decoding with: python -m mlc_llm serve /Users/USER/LLM/Mistral-Large-Instruct-2407-MLC --additional-models "HF://mlc-ai/Mistral-7B-Instruct-v0.3-q4f16_1-MLC" --speculative-mode small_draft --port 9999 --device metal --mode server
First token is streamed, then error message

USER@MBPM3MVLB ~ % python -m mlc_llm serve /Users/USER/LLM/Mistral-Large-Instruct-2407-MLC --additional-models "HF://mlc-ai/Mistral-7B-Instruct-v0.3-q4f16_1-MLC" --speculative-mode small_draft --port 9999 --device metal --mode server
[2024-09-16 08:50:13] INFO auto_device.py:79: Found device: metal:0
[2024-09-16 08:50:13] INFO jit.py:43: MLC_JIT_POLICY = ON. Can be one of: ON, OFF, REDO, READONLY
[2024-09-16 08:50:13] INFO jit.py:158: Using cached model lib: /Users/USER/.cache/mlc_llm/model_lib/3826dfed383847636248c8e5e540102b.dylib
[2024-09-16 08:50:13] INFO download_cache.py:227: Downloading model from HuggingFace: HF://mlc-ai/Mistral-7B-Instruct-v0.3-q4f16_1-MLC
[2024-09-16 08:50:13] INFO download_cache.py:29: MLC_DOWNLOAD_CACHE_POLICY = ON. Can be one of: ON, OFF, REDO, READONLY
[2024-09-16 08:50:13] INFO download_cache.py:166: Weights already downloaded: /Users/USER/.cache/mlc_llm/model_weights/hf/mlc-ai/Mistral-7B-Instruct-v0.3-q4f16_1-MLC
[2024-09-16 08:50:13] INFO jit.py:43: MLC_JIT_POLICY = ON. Can be one of: ON, OFF, REDO, READONLY
[2024-09-16 08:50:13] INFO jit.py:158: Using cached model lib: /Users/USER/.cache/mlc_llm/model_lib/7bbcaf068957bbf173dbd8ad644faea6.dylib
[2024-09-16 08:50:13] INFO engine_base.py:192: The selected engine mode is server. We use as much GPU memory as possible (within the limit of gpu_memory_utilization).
[2024-09-16 08:50:13] INFO engine_base.py:200: If you have low concurrent requests and want to use less GPU memory, please select mode "local".
[2024-09-16 08:50:13] INFO engine_base.py:205: If you don't have concurrent requests and only use the engine interactively, please select mode "interactive".
[08:50:13] /Users/catalyst/Workspace/mlc-ai-package-self-runner/_work/package/package/mlc-llm/cpp/serve/config.cc:688: Under mode "local", max batch size will be set to 4, max KV cache token capacity will be set to 8192, prefill chunk size will be set to 2048.
[08:50:13] /Users/catalyst/Workspace/mlc-ai-package-self-runner/_work/package/package/mlc-llm/cpp/serve/config.cc:688: Under mode "interactive", max batch size will be set to 1, max KV cache token capacity will be set to 32768, prefill chunk size will be set to 2048.
[08:50:13] /Users/catalyst/Workspace/mlc-ai-package-self-runner/_work/package/package/mlc-llm/cpp/serve/config.cc:688: Under mode "server", max batch size will be set to 80, max KV cache token capacity will be set to 32768, prefill chunk size will be set to 2048.
[08:50:13] /Users/catalyst/Workspace/mlc-ai-package-self-runner/_work/package/package/mlc-llm/cpp/serve/config.cc:769: The actual engine mode is "server". So max batch size is 80, max KV cache token capacity is 32768, prefill chunk size is 2048.
[08:50:13] /Users/catalyst/Workspace/mlc-ai-package-self-runner/_work/package/package/mlc-llm/cpp/serve/config.cc:774: Estimated total single GPU memory usage: 86697.674 MB (Parameters: 69664.656 MB. KVCache: 15602.123 MB. Temporary buffer: 1430.894 MB). The actual usage might be slightly larger than the estimated number.
[08:50:13] /Users/catalyst/Workspace/mlc-ai-package-self-runner/_work/package/package/mlc-llm/cpp/serve/engine.cc:365: Warning: Hybrid prefill mode fallbacks to chunked prefill, due to speculative mode is enabled and not implemented with hybrid prefill yet.
INFO:     Started server process [69315]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     Uvicorn running on http://<IP_ADDRESS>:9999 (Press CTRL+C to quit)
INFO:     <IP_ADDRESS>:58406 - "POST /v1/chat/completions HTTP/1.1" 200 OK
libc++abi: terminating due to uncaught exception of type tvm::runtime::InternalError: [08:50:41] /Users/catalyst/Workspace/mlc-ai-package-self-runner/_work/package/package/mlc-llm/cpp/serve/engine_actions/batch_draft.cc:151: InternalError: Check failed: (!mstates[i]->draft_output_tokens.empty()) is false:
Stack trace:
zsh: abort      python -m mlc_llm serve /Users/USER/LLM/Mistral-Large-Instruct-2407-MLC
Expected behavior
The model streams the output to the provided prompt.
Environment

Platform (e.g. WebGPU/Vulkan/IOS/Android/CUDA): Macbook Pro
Operating system (e.g. Ubuntu/Windows/MacOS/...): macOS Sequoia
Device (e.g. iPhone 12 Pro, PC+RTX 3090, ...):  Both yield the same error.

Additional context

Both models work fine separately.

Thank you @vlbosch. We also ran into this and get it fixed in #2906. The nightly packages are under built and will be ready in a few hours. I'll report back when the nightly build is done.

Hi @vlbosch the nightly wheel is updated and could you please try upgrade?

@MasterJH5574 Thanks for the quick response! I just updated to the latest nightly and retried. Small draft-mode does work now, however the speed running with small draft is slower than with Mistral Large alone. I thought that the baseline would be the regular speed of the large model? Or does that only count for the other speculative modes like eagle and medusa?


Zigbee reconnect range shorter than working range (TZ-1306)

Question
When an esp32-C6(EndDevice) is paired with an esp32-H2(Coordinator), if both go to deep sleep (Same behavior if powered on from a shut down state), the range at which the reconnect occurs is much shorter than the operating range at which both devices keep communicating when already connected.
My guess is the End device does not reconnect to the network if the rssi ou LQI is below a certain range. Is this correct and if so, is there a way to configure the right threshold?
Additional context.
The ESP32-C6 is on a set position, while the H2 is battery powered and is mobile. If the reconnect occurs under a certain range and is successful, i can get further and still transmit/receive my data (At least twice the range). However, if one of the 2 device gets disconnected and the reconnect procedure restart below that range, they are unable to reconnect.
Note that this range changes with the parameter of esp_zb_set_tx_power(XX);

@Matthieu-Poulain ,
Please refer to https://github.com/espressif/esp-zigbee-sdk/issues/352#issuecomment-2141402806.

This solved my issue, many thanks!

@Matthieu-Poulain If the issue has been resolved, please consider closing it. Welcome to open other SDK issues.


Add install instructions

Hello :=)
Can you please add install instructions and/or a make install target? This would also help with packaging

Hello,
I can add some instructions and update the Makefile - also, feel free to send a PR.
In general, you just need to be able to call envload and protonhax from a terminal, and steam being visible to them*.
For Arch, the process can be seen here https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=protonhax-git
Something in the lines.

Clone the repo, or download from the release page
Open the folder in a terminal
make - it will build the envload
Copy envload and protonhax to where you want to call them from and give proper permissions. e.g /usr/bin (but not everyone wants to deploy there)

For Steam flatpak I need to look into creating a version on flathub.


Breakpoints are gone

I can no longer create a breakpoint. 😢

I can repro the issue :/

Same root cause as #15706


Add a detailed Trip Plan page.

Should display an Itinerary in a more expanded, vertical manner. Should show all stops, including intermediaries. Maybe fare price too?

Update: New design features a map behind the underlying route.


Revamp of our example building infrastructure

The in-app example page (#3191) relies on pre-build RRD for its example. These RRD must be up-to-date for the dev builds, and version-specific for the public release. These RRD are also costly to build, which means we currently have only 7 examples covered. This figure must be increased.
We had a short discussion with @jprochazk and have the following proposal:

We create a dedicated CI workflow to build RRD examples, based on the existing build_demo_app.py script.
This workflow will be heavy, so it'll be run on a nightly basis of origin/main.
This script will pull everything it needs from example's front matter, including:

whether or not a given example must be included in the build
any build args to use (e.g. specify dataset, etc.)
metadata (title, description, tags, thumbnail, etc.

Like now, the built examples will be uploaded to our Google storage

for each build, under a commit/XXX URL path
for each build, a symlink/redirect will be updated for release/nightly
for each release, under release/0.X.Y

We create a build script to build a manifest file which will be embedded into all rerun executables via include_str!. This manifest will then be used to render the examples page on the welcome screen.

It will aggregate all examples which run on CI (identified via build_args being present in the frontmatter)
It will contain links to .rrd, thumbnails, metadata, etc.
The version chosen for those links will be based on a compile-time MANIFEST_VERSION environment variable.

For local and main builds, this will just point to version/nightly.
For PR builds, this will point to commit/<latest PR commit>.
For release builds, this will point to version/<tagged version>.

We will remove the manifest.json file from the build_demo_app.py script and GCS uploads.

Related:

#3190

Relevant: https://github.com/rerun-io/rerun/pull/3204 introduces the _RERUN_TEST_FORCE_SAVE=<filepath.rrd> environment variable, that will force any RecordingStream to write to a specified file on disk, irrelevant of how it was set up (connect(), spawn(), etc...).

Let's try to get this in before 0.9, but start with just the minimal set of examples.

The viewer include_str!:s crates/re_viewer/data/examples_manifest.json which in turn points to .rrd urls at a specific commit. Currently we need to update examples_manifest.json manually.
For 0.9 we could just update it manually to point to .rrd's we've updated specifically for the 0.9 release.
…or we figure out a proper fix for this.


Update error handling in instance_create_surface

Also did a couple minor refactors.

There is some panic and fmt.panicf we can remove and return error.


About mask and artifact

Hi, Thanks for your excellent work. I have two questions.

Have you ever tried removing the mask operation during training and inference, and directly using a weight of 0.5 to fuse the  two warping results? I wonder how much impact will this have for the model.
When I infer some hard cases, there are some artifacts in results. Do you have any suggestions or measures for it?
Thanks a lot!

I believe the effect will be poor. I don't remember if I have done any related experiments
I don't know particularly good method, maybe you can calculate the similarity between warped_img0 and warped_img1 to delete some interpolated frame results and directly copy the previous frame

Thanks for your reply.


Need support of error to represent unrecoverable logical error

We probably need support of error just like the one in Haskell. It won't be able to write one by user, because that will cause a type mismatch error.
Example use case:
let car [A] (l : PolyList[A]) : A =
    case l of
        Nil         -> error "Call `car` on a empty list"
     |  Cons x xs   -> x;

If I define error like this
let error (msg : String) = {
    java.lang.System.err.println(msg);
    java.lang.System.exit(1);
    ()
};

It won't compile, because of type mismatch. So I think compiler should see error function as a special case, or provide mechanisms to support throwing exception.
In Haskell, the message that sent from error function could be caught by Control.Exception.catch.
The definition of error in Haskell is
error :: [Char] -> a

More over, if I define error like this
let error [A] (msg : String) : A = {
    java.lang.System.err.println(msg);
    java.lang.System.exit(1);

    -- What should I write here??
};

Perhaps we need an easy way to add primitives into the compiler, including error and tracing features. Maybe @bixuanzju can think a bit about this.


Add prototypical profiler

Adds a really basic prototypical profiler, based on top of the frame stack.
Only currently goes one level down and doesn't have good frame names. But shows a proof of concept.

This PR is too stale to merge now, sorry.


CORS error on 1.6.0

Hello after update to 1.6.0 (from 1.4.0) I have CORS error when load .css and .js files
Also i see soem files not found

*I have updated mineflayer to laetst version 3.9.0

Probably related to https://github.com/imharvol/mineflayer-web-inventory/issues/23 while it's solved, try using the solution mentioned there

Hi seems is related,
I installed the remove-path version
npm i https://github.com/imharvol/mineflayer-web-inventory.git#remove-path
this version don't show cors error

After a few test I reinstalled 1.6.0 and now no shows error,
I don't know why now works but seems is an issue from cache or random error from Express
I close the ticket because now works.. and i cant reproduce the error
Best!


[Bug][Config-UI]  subtask calculateChangeLeadTime unable to handle NULL values

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
The subtask for my devlake config-UI pipeline fails with the error :
level=error msg=" [pipeline service] [pipeline #5907] [task #47939] subtask calculateChangeLeadTime ended unexpectedly\n\tWraps: (2) error adding result to batch (500)\n\tWraps: (3) Error 1292 (22007): Incorrect datetime value: '0000-00-00' for column 'first_commit_authored_date' at row 43 (500)\n\tWraps: (4) Error 1292 (22007): Incorrect datetime value: '0000-00-00' for column 'first_commit_authored_date' at row 43\n\tError types: (1) *hintdetail.withDetail (2) *hintdetail.withDetail (3) *hintdetail.withDetail (4) *mysql.MySQLError"
What do you expect to happen
Successful pipeline run
How to reproduce
A new column named first_commit_authored_date is introduced in the latest devlake versions (I am using v1.0.2-beta5  ) and this column was not in V0.21 and after upgrade, I am getting the above mentioned error .
The table containing this column is :  project_pr_metrics
NULL values in this column are not being correctly handled by devlake.
Anything else
No response
Version
v1.0.2-beta5
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@klesh Can you help with this?

Will take a look.


showing decimal value

I want to show decimal value, but vue-slider only showing integer. And also i couldn't set decimal points to interval. Is there any way to flace decimal point?

Can set the decimal interval.
https://jsfiddle.net/2xy72dod/8064/

thanks.. it works. My problem were somewhere else, that's why it wasn't showing decimal value


Parse div block syntax

Pandoc now has a DIV syntax for content blocks. For example:
::: homepage
![logo](images/logo.svg){width=120px height=120px} \

::: content
# Title
## Subtitle

<EMAIL_ADDRESS>:::

:::

Using the latest version of Pandoc, the above produces:

Title

Subtitle

<a<EMAIL_ADDRESS>

This would be great to see in FlexMark as well.

@DaveJarvis, I will add it to the to do list but at the moment no PanDoc specific profile or extensions are implemented so it may take a while to add this extension along with other PanDoc specific ones.

Should anybody need this functionality, attached are a few classes that may help.

ClosingDivBlock.java.txt
DivBlock.java.txt
FencedDivExtension.java.txt
FencedDivRenderer.java.txt
OpeningDivBlock.java.txt

Here's a short example usage:
final var extensions = new ArrayList<Extension>();
extensions.add( FencedDivExtension.create() );

final IParse parser = Parser.builder().extensions( extensions ).build();
final IRender renderer = HtmlRenderer.builder().extensions( extensions ).build();

This solution works for nested fenced div elements, although it may be suboptimal because the opening and closing div elements are written separately, without knowledge of each other. If a Markdown document is missing the closing ::: then the resulting HTML document will be missing a closing div.
If you have questions, you may be best off asking on StackOverflow.

@DaveJarvis please add a pr...

@XenoAmess, a PR probably won't happen. If I were to make a flexmark extension at all, it'd be for parsing citations and cross-references, not for fenced DIVs. Here's the latest source code for fenced DIVs as is integrated into my text editor:
https://github.com/DaveJarvis/KeenWrite/tree/main/src/main/java/com/keenwrite/processors/markdown/extensions/fences


sort to sort_by

I noticed this conversion issue:
from:
  list.sort{ |s| s.id }

to:
  list.sort(&:id)

but it doesn't work. Should be
  list.sort_by(&:id)

Thanks

You're using sort wrong - it yields two arguments to its block, not one.


Vulnerability found in lodash < 4.17.13

Vulnerability Information

Package name:  lodash
Package version: 4.17.11
Package Manifest: https://github.com/azu/security-alert/blob/master/yarn.lock
Dependencies type: unknown
Vulnerable version: < 4.17.13
Patched version: 4.17.13
GitHub Alert: https://github.com/azu/security-alert/network/alert/yarn.lock/lodash/open

How to fix?
Upgrade lodash to version 4.17.13 or later. For example:
"dependencies": {
  "lodash": ">=4.17.13"
}

or…
"devDependencies": {
  "lodash": ">=4.17.13"
}

Details

https://github.com/lodash/lodash/pull/4336
https://nvd.nist.gov/vuln/detail/CVE-2019-10744

lol
Results

[ERROR] [js/xss] Cross-site scripting vulnerability due to \[user-provided value\]\(1\).

https://github.com/azu/security-alert/blob/master/examples/Xss.js#L4
https://github.com/azu/security-alert/blob/master/examples/Xss2.js#L4

Suppressed Results
Nothing here.
Rules information

Rules details
- js/xss [error]

> Client-side cross-site scripting

Tool information

Name: CodeQL command-line toolchain
Organization: GitHub
Version: 2.2.4


Upgrade Microsoft.Graph lib to fix vulnerability.

CVE-2024-21319

API change check
API changes are not detected in this pull request.


[FEAT] Allow configuration of written file sizes

Is your feature request related to a problem? Please describe.
When writing a dataframe into files, often users will want to set the output file sizes to be thresholded at some reasonable value (e.g. 256MB).
It's not going to be possible to know upfront what the compressed data sizes are going to be, so perhaps we should expose this as an option to be thresholded on the "uncompressed" data size.

[ ] Configure CSV writes
[ ] Configure Parquet writes

Acceptance Criteria
df.write_parquet(..., max_uncompressed_file_size=...)
df.write_csv(..., max_uncompressed_file_size=...)

Note that this was added but with the API of using daft.set_execution_config instead: https://www.getdaft.io/projects/docs/en/latest/api_docs/doc_gen/configuration_functions/daft.set_execution_config.html


Exclude<any,whatever> (another way of negated type #29317)

Hi
Search Terms

Exclude<any
type subtraction
any but
all types except
Checked: https://github.com/microsoft/TypeScript/issues/31579 https://github.com/Microsoft/TypeScript/pull/29317
Suggestion

type T00 = Exclude<any,whatever>

T00 should be composed of any (all) types, except those types that are assignable to whatever
Use Cases

I definded a bunch of functions according to the following interface:
interface ecafretni { (x:any, y:Q): any }

x literally takes any type, there's no possible narrowing down.  Think of a general-purpose deep-copy function, for example.  Q is some other interface I defined somewhere.  Here's the use case: I had to change the order of the arguments of ecafretni to (y:Q, x:any) and I was wishing that TypeScript would point out all the (hundreds of)  points where I was calling functions abiding by that interface, so that I woulnd't miss any.  But it wouldn't because, of course, type Q is assignable to any and type any is assignable to Q.
So I came up with another type to use in the place of any: R = Exclude<any,Q>, and redefine ecafretni argument types as f(y:Q, x:R).  But it's not working.  It seems that R behaves the same way as any.
Examples

const f = function( x : Exclude<any,Number> ){};
const x:Number = 4;
const v = f(x); // TYPE MISMATCH !!!! (not caught by TypeScript)

Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

Duplicate of #29317.

Duplicate of #29188


UserControl will_unmount() docs example

Hello! I'm going through the docs and came up to "User controls" in the python guides sections and I'm a bit confused with will_unmount() in countdown timer example.
I see that it is "called before the UserControl is removed from a page", but when is UserControl removed from the page in the example? From what I see the timer just goes down to 00:01 and stops, even with will_unmount() commented out. Would I just have to clear or change the page view to have will_unmount() be called automatically?
import flet
from flet import UserControl, Text
import threading
import time

# Mounts are automatically called.
# did_mount() - called after the UserControl added to a page and assigned transient id.
# will_unmount() - called before the UserControl is removed from a page.

class Countdown(UserControl):
    def __init__(self, seconds):
        super().__init__()
        self.seconds = seconds

    def did_mount(self):
        self.running = True
        self.th = threading.Thread(target=self.update_timer, args=(), daemon=True)
        self.th.start()

    def will_unmount(self):
        self.running = False

    def update_timer(self):
        while self.seconds and self.running:
            mins, secs = divmod(self.seconds, 60)
            self.countdown.value = "{:02d}:{:02d}".format(mins, secs)
            self.update()
            time.sleep(1)
            self.seconds -= 1

    def build(self):
        self.countdown = Text()
        return self.countdown

def main(page):
    page.add(Countdown(10), Countdown(5))

flet.app(target=main)

Thanks for trying Flet!
Yep, when the control is removed from a page (control tree to be recise as it could be nested within another control) its will_unmount() is called before that.
So, doing, for example this will call will_unmount():
page.controls.clear()
page.update()

This is the place: https://github.com/flet-dev/flet/blob/main/sdk/python/flet/control.py#L409-L415

Thanks! That cleared things up.


Test the go-client for 1.15 kubernetes cluster

In GitLab by @tmon546596046 on Jul 15, 2019, 15:14
The kubernetes cluster master ip: <IP_ADDRESS>

In GitLab by @sokril on Nov 8, 2019, 08:05
test support for the follow function list:

services (deployment, service, stateful)
pv/pvc
configmap
nodes

but there are tow problem:

get the node hostname when get the node list in board
need build the role rule in kubernetes

In GitLab by @sokril on Nov 8, 2019, 08:33
assigned to @sokril

In GitLab by @sokril on Dec 26, 2019, 03:14
changed the description

In GitLab by @luotian-github on Mar 4, 2020, 09:16
1.15 already in 2.8

In GitLab by @luotian-github on Mar 4, 2020, 09:16
closed


⚠ ClamAV has degraded performance

In 906be56, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 918 ms

Resolved: ClamAV performance has improved in d368bdb.


python gir['location'] KeyError: 'location'

@DrDaveD any ideas about this?  It started happening recently, not sure what changed. Upgrading to cvmfs-servermon-1.23-1.24.obs.el7.noarch and doing sudo cvmfs_server update-geodb  did not help.
[Tue Apr 11 12:10:24.508467 2023] [mpm_prefork:notice] [pid 2789] AH00163: Apache/2.4.6 (CentOS) mod_wsgi/3.4 Python/2.7.5 configured -- resuming normal operations
[Tue Apr 11 12:10:24.508490 2023] [core:notice] [pid 2789] AH00094: Command line: '/usr/sbin/httpd -D FOREGROUND'
[Tue Apr 11 12:10:32.701243 2023] [:error] [pid 20927] processed /etc/cvmfsmon/api.conf
[Tue Apr 11 12:10:32.701294 2023] [:error] [pid 20927] aliases: {'local': '<IP_ADDRESS>'}
[Tue Apr 11 12:10:32.701302 2023] [:error] [pid 20927] excludes: []
[Tue Apr 11 12:10:32.701313 2023] [:error] [pid 20927] limits: {'whitelist-warning': 48, 'updated-warning': 4, 'gc-warning': 2, 'gc-critical': 7, 'updated-critical': 18}
[Tue Apr 11 12:10:32.701320 2023] [:error] [pid 20927] subdirectories: {}
[Tue Apr 11 12:10:32.755886 2023] [:error] [pid 20926] cvmfs_geo: opened /var/lib/cvmfs-server/geo/GeoLite2-City.mmdb
[Tue Apr 11 12:19:40.528038 2023] [:error] [pid 20926] cvmfs_geo: opened /var/lib/cvmfs-server/geo/GeoLite2-City.mmdb
[Tue Apr 11 12:25:40.610251 2023] [:error] [pid 20926] cvmfs_geo: closed old /var/lib/cvmfs-server/geo/GeoLite2-City.mmdb
[Tue Apr 11 12:57:21.347786 2023] [:error] [pid 20926] [remote <IP_ADDRESS>:132] mod_wsgi (pid=20926): Exception occurred processing WSGI script '/var/www/wsgi-scripts/cvmfs-server/cvmfs-api.wsgi'.
[Tue Apr 11 12:57:21.347861 2023] [:error] [pid 20926] [remote <IP_ADDRESS>:132] Traceback (most recent call last):
[Tue Apr 11 12:57:21.347896 2023] [:error] [pid 20926] [remote <IP_ADDRESS>:132]   File "/var/www/wsgi-scripts/cvmfs-server/cvmfs-api.wsgi", line 18, in application
[Tue Apr 11 12:57:21.348006 2023] [:error] [pid 20926] [remote <IP_ADDRESS>:132]     return cvmfs_api.dispatch(api_func, path_info, repo_name, version, start_response, environ)
[Tue Apr 11 12:57:21.348039 2023] [:error] [pid 20926] [remote <IP_ADDRESS>:132]   File "/usr/share/cvmfs-server/webapi/cvmfs_api.py", line 23, in dispatch
[Tue Apr 11 12:57:21.348315 2023] [:error] [pid 20926] [remote <IP_ADDRESS>:132]     return cvmfs_geo.api(path_info, repo_name, version, start_response, environ)
[Tue Apr 11 12:57:21.348339 2023] [:error] [pid 20926] [remote <IP_ADDRESS>:132]   File "/usr/share/cvmfs-server/webapi/cvmfs_geo.py", line 313, in api
[Tue Apr 11 12:57:21.348541 2023] [:error] [pid 20926] [remote <IP_ADDRESS>:132]     onegood, indexes = geosort_servers(now, gir_rem, servers, trycdn)
[Tue Apr 11 12:57:21.348560 2023] [:error] [pid 20926] [remote <IP_ADDRESS>:132]   File "/usr/share/cvmfs-server/webapi/cvmfs_geo.py", line 205, in geosort_servers
[Tue Apr 11 12:57:21.348585 2023] [:error] [pid 20926] [remote <IP_ADDRESS>:132]     gir_server = name_geoinfo(now, server)
[Tue Apr 11 12:57:21.348598 2023] [:error] [pid 20926] [remote <IP_ADDRESS>:132]   File "/usr/share/cvmfs-server/webapi/cvmfs_geo.py", line 173, in name_geoinfo
[Tue Apr 11 12:57:21.348617 2023] [:error] [pid 20926] [remote <IP_ADDRESS>:132]     gir = gir['location']
[Tue Apr 11 12:57:21.348646 2023] [:error] [pid 20926] [remote <IP_ADDRESS>:132] KeyError: 'location'

Oh I misinterpreted the output.
Thanks for the tip, that should fix it, I'll give it a try.

But that being the case, if the error is with a cvmfs-server geolookup for a client with a cloudflare IP, I'm not sure why servermon would flag a critical geo error:
$ curl "http://localhost/cvmfsmon/api/v1.0/all&format=details"
{"CRITICAL": {"geo": [{"msg": "HTTP Error 400: Bad Request",
                       "repo": "cvmfs-config.computecanada.ca"}]},
...

I guess the python error was a red herring, the real issue I was trying to address was:
$ curl "http://localhost/cvmfsmon/api/v1.0/all&format=details"
{"CRITICAL": {"geo": [{"msg": "HTTP Error 400: Bad Request",
                       "repo": "containers.computecanada.ca"}]},

Since updating to 2.10.1 fixed the other issue, now I don't see any httpd errors so I am not sure what the issue is.
[Tue Apr 11 14:55:19.060111 2023] [mpm_prefork:notice] [pid 2789] AH00171: Graceful restart requested, doing restart
[Tue Apr 11 14:55:19.131302 2023] [lbmethod_heartbeat:notice] [pid 2789] AH02282: No slotmem from mod_heartmonitor
[Tue Apr 11 14:55:19.132634 2023] [mpm_prefork:notice] [pid 2789] AH00163: Apache/2.4.6 (CentOS) mod_wsgi/3.4 Python/2.7.5 configured -- resuming normal operations
[Tue Apr 11 14:55:19.132653 2023] [core:notice] [pid 2789] AH00094: Command line: '/usr/sbin/httpd -D FOREGROUND'
[Tue Apr 11 14:55:46.260873 2023] [:error] [pid 19679] processed /etc/cvmfsmon/api.conf
[Tue Apr 11 14:55:46.260927 2023] [:error] [pid 19679] aliases: {'local': '<IP_ADDRESS>'}
[Tue Apr 11 14:55:46.260936 2023] [:error] [pid 19679] excludes: []
[Tue Apr 11 14:55:46.260947 2023] [:error] [pid 19679] limits: {'whitelist-warning': 48, 'updated-warning': 4, 'gc-warning': 2, 'gc-critical': 7, 'updated-critical': 18}
[Tue Apr 11 14:55:46.260955 2023] [:error] [pid 19679] subdirectories: {}
[Tue Apr 11 14:55:46.324773 2023] [:error] [pid 19678] cvmfs_geo: opened /var/lib/cvmfs-server/geo/GeoLite2-City.mmdb
[Tue Apr 11 14:56:57.531524 2023] [:error] [pid 19679] processed /etc/cvmfsmon/api.conf
[Tue Apr 11 14:56:57.531579 2023] [:error] [pid 19679] aliases: {'local': '<IP_ADDRESS>'}
[Tue Apr 11 14:56:57.531592 2023] [:error] [pid 19679] excludes: []
[Tue Apr 11 14:56:57.531611 2023] [:error] [pid 19679] limits: {'whitelist-warning': 96, 'updated-warning': 4, 'gc-warning': 2, 'gc-critical': 7, 'updated-critical': 18}
[Tue Apr 11 14:56:57.531622 2023] [:error] [pid 19679] subdirectories: {}

The actual CVMFS geo API seems to be working okay:
$ curl http://localhost/cvmfs/containers.computecanada.ca/api/v1.0/geo/`hostname -f`/cvmfs-s1fnal.opensciencegrid.org,cvmfs-stratum-one.cern.ch,cvmfs-stratum-one.ihep.ac.cn,cvmfs-s1-arbutus.computecanada.ca,cvmfs-s1-beluga.computecanada.ca,`hostname -f`
6,5,1,4,3,2

Oh I think I see the problem.  The probe that the cvmfs-servermon geo test uses happens to use cvmfs-stratum-one.cern.ch in test query, and that's in the range of addresses that disappeared from the maxmind DB.  That was probably a bad choice.  If you replace your `hostname -f` with cvmfs-stratum-one.cern.ch do you see the failure?  Can you find an older copy of /var/lib/cvmfs-server/geo/GeoLite2-City.mmdb to install on your stratum 1 until the fixed one comes out on Friday?

You had cvmfs-stratum-one.cern.ch in the list of servers to sort, and in that case the geo api just assumes it is far away (that's why 2 is on the end of your list) but the cvmfs-servermon geo probe uses it also as the reference address (after the /geo/`, before the list of stratum 1s) and that is required to be able to be in the database.

Oh yeah:
$ curl http://localhost/cvmfs/containers.computecanada.ca/api/v1.0/geo/cvmfs-stratum-one.cern.ch/cvmfs-s1fnal.opensciencegrid.org,cvmfs-stratum-one.cern.ch,cvmfs-stratum-one.ihep.ac.cn,cvmfs-s1-arbutus.computecanada.ca,cvmfs-s1-beluga.computecanada.ca,`hostname -f`
Bad Request: remote addr not found in database

$ curl http://localhost/cvmfs/containers.computecanada.ca/api/v1.0/geo/`hostname -f`/cvmfs-s1fnal.opensciencegrid.org,cvmfs-stratum-one.cern.ch,cvmfs-stratum-one.ihep.ac.cn,cvmfs-s1-arbutus.computecanada.ca,cvmfs-s1-beluga.computecanada.ca,`hostname -f`
6,5,1,4,3,2

This was also more confusing due to https://github.com/cvmfs/cvmfs/issues/3255 since normally I would just test with an arbitrary proxy string but that didn't work anymore.
I don't have an older copy of the mmdb but we can wait until Friday. For some reason only one s1 server is affected and it seems to be fine aside from the geoapi monitoring.

The other s1 doesn't have an older copy of the mmdb?

It must have cleared up with a geodb update.


[Bug]: Ultrawide not working if monitor is set as secondary display in windows.

Issue Summary:
Ultrawide not working if monitor is set as secondary display in windows.
GBFRelinkFix & Game Version Numbers:
v1.0.4 of gran & your fix
Log Files:
I don't see a log file 😩
Reproduction:

Set ultrawide monitor as secondary display in windows
Change to ultrawide monitor in gran's settings

Restarting the game doesn't help.
Swapping the ultrawide to the primary display resolves the issue but is awful QOL
Thanks for any help and for making such a good mod ^^

Sorry for the late reply but has the mod worked at all for you? You should be getting a GBFRelinkFix.log file.

 New pc and I didn't have file extensions showing anymore X_X
GBFRelinkFix.log
There's a fresh log with the ultrawide set as secondary display in windows, and selected in gran's settings.

Okay so when iCustomResX and iCustomResY are set to 0 in the config file then your desktop resolution for your primary display is used. Since the function I use only pulls info from the primary display, a solution for you might be to enter the resolution of your ultrawide display in the config file.
For example if your ultrawide display is 5120x1440 then you would set it like this:
[Custom Resolution]
; Enter your chosen resolution.
; You can leave these at 0 if you want your desktop resolution used instead.
Enabled = true
Width = 5120
Height = 1440

That worked great. Thank you for the assistance and for your great work!


Node attempts to sync with a peer running wrong node version

Description
Node will at times try to sync from node 19313afe even though it can clearly see in processing initial status packet that its running the wrong node version...
2021-05-21 02:22:04.783228 SUMMARY INFO Currently syncing from node ##19313afe…
2021-05-21 02:22:04.783274 SUMMARY INFO Max peer PBFT chain size:      80705 (peer ##19313afe…)
2021-05-21 02:22:04.783323 SUMMARY INFO Max peer PBFT consensus round: 87923 (peer ##19313afe…)
2021-05-21 02:22:04.783363 SUMMARY INFO Max peer DAG level:            294866 (peer ##19313afe…)
2021-05-21 02:22:04.783768 TARCAP INFO Node ##19313afe… disconnected
2021-05-21 02:22:07.328666 TARCAP INFO Node ##19313afe… connected
2021-05-21 02:22:07.447971 TARCAP ERROR Incorrect node version: 0.01, our node major version1.06, host ##19313afe… will be disconnected
2021-05-21 02:22:07.465740 TARCAP INFO Node ##19313afe… disconnected

Environment
Node version
v1.6
Expected behavior
Status packet would be first packet processed and would disconnect immediately if node doesn't pass initial status packet
Actual behavior
Either we don't disconnect immediately, or more likely the initial status packet is NOT always the first packet processed for the node.

I think we should add a peer->passed_initial_ boolean flag for the peer and ignore non-initial status packets if the peer hasn't passed initial status packet check.

Yesterday I tried to simulate this, because I have also thought this is happening, but actually it never happens. I think it is impossible to send anything before disconnecting

Never mind, just happened to me


Offset argument to place chord Symbol in a more precise location inside a word

I think it would be useful to have a parameter that you can use to control where the chord is inside the word.
For example, supposing the chord on the word "misfortune" lands on the the last syllable "ne", I would want to place the chord symbol with an offset, so it lands on the right-most side of the word.
I know it is hard to calculate syllables and stuff, so a straightforward implementation would be to use a length parameter to off set the chord symbol, like:
#chord[misfortune][G] [0.2cm]

Hi, you can do this:
misfortu#chord[ne][G]
But this has a problem, when the chord name is long, the canvas above the syllable splits the word.
Is good your idea with the offset.

Hi, you can do this:
misfortu#chord[ne][G]
But this has a problem, when the chord name is long, the canvas above the syllable splits the word.
Is good your idea with the offset.

Maybe an even better API would be to pass in the number of letters you want to offset.
If typst has a way to measure the size of the letters(in current font and size) it will work.
This should be better because you would only need to count the letters and the result would be right without needing to adjust it.
Not sure if it's possible

I think this is possible doing something along the lines of:

#let position_in_word(word, index) = {
  style( styles => {
   let sum
    for i in range(4) {
      sum += measure(word.at(i), styles).width
    }
    let letter = word.at(index)
    [Position of letter #letter in #word is #sum]
  }
  )
}

#position_in_word("Hello World!", 6)

Then you would only need to place the chord relative to the word using the offset provided.
Not sure how to do that though.

I did tests, but the style function has a strange behavior (https://github.com/typst/typst/issues/766) and I can't get the measure value.
Also I tried to use the typst-canvas functions inside of style function but didn't work.
I keep on investigating.

Now is possible to set an offset to position the chord above a specific character of the word,
Example:
#chord[hello][C][2] //  C chord above "e" character
#chord[hello][C][] // C chord centered above word

Also, the first character of the chord name is centered with the target character of the word.


feat: add spark history server api

PR summary

Add api methods for managing spark history server.
Add submission_time and auto_termination_time fields in spark application list and details responses.

BREAKING: The type of timestamp fields in Application response models has been changed to datetime.
Fields in Application and ApplicationGetResponse objects like start_time, end_time, finish_time will now have values of type datetime instead of plain strings.
Signed-off-by: Subin Shekhar<EMAIL_ADDRESS>Fixes: <! -- link to issue -->
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[x] Yes
[ ] No

Fields in Application and ApplicationGetResponse objects like start_time, end_time, finish_time will now have values of type datetime instead of plain strings.
Review the code referencing these fields and update as required.
Other information

:tada: This PR is included in version 2.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


details tag boolean attribute bug.

Description
When using the  element, trying to set the open attribute using a state or property will always make it one step behind.
Steps to Reproduce

Copy the code into the lit.dev playground

import { LitElement, html } from 'lit';
import { customElement, state } from 'lit/decorators.js';

@customElement('cus-appendix')
class Appendix extends LitElement {

  @state()
  private open = false;

  render() {
    return html`

      <slot name="title">
        Appendix
      </slot>

        <slot></slot>

    `;
  }

  toggleOpen() {
    this.open = !this.open;
  }
}

Alter the index.html to have a <cus-appendix></cus-appendix> tag
click on the custom element.

Live Reproduction Link
I am using typescript, if anyone knows to do this on stackblitz please let me know!
Expected Results
The element should update the state and render the attribute open.
Actual Results

It toggles the attribute on, and then off for the first click.
Subsequent clicks toggles the div content, but the state variable is always the opposite upon render().

I found that immediately after the execution of this.update(changedProperties); from ReactiveElement.performUpdate() that
the attribute is rendered correctly but somewhere after that, it removes the attribute.
Browsers Affected

[x] Chrome
[x] Firefox
[x] Edge
[ ] Safari 11
[ ] Safari 10
[ ] IE 11

The underlying open state of the detail element is being changed outside of Lit by the browser. In order to get Lit to understand this and do change detection against the live value, you need to use the live() directive.
I would probably bind against the detail element's open property as well.
import { LitElement, html } from 'lit';
import { customElement, state } from 'lit/decorators.js';
import { live } from 'lit/directives/live.js';

@customElement('cus-appendix')
class Appendix extends LitElement {

  @state()
  private open = false;

  render() {
    return html`

      <slot name="title">
        Appendix
      </slot>

        <slot></slot>

    `;
  }

  toggleOpen() {
    this.open = !this.open;
  }
}```

Could remove the need for using the live directive by doing the following?
render() {
    return html`
      <summary class="appendix__title"
<EMAIL_ADDRESS>      <slot name="title">
        Appendix
      </slot>

        <slot></slot>

    `;
  }

Or would run into the same problem as before?

The solution you proposed does not work as expected in the playground environment either.
@justinfagnani

For the reproduction link, you can just link to the lit.dev playground directly.

Created a repro in the playground.
Logging this.open at the start of the render method. Does seem to be one step behind.
There is a single click no-op before it starts working as well.

Hi @Thomas-Langlais,
The issue is happening because the open state and event behavior are competing.
This can be resolved by preventing the default click behavior on the details tag.
Example in lit playground
Please let me know if this doesn't fix your issue.
Thank you!


x:Uid not implemented on Android and IOs?

Hi
is x:Uid implemented for Android and IOs… For my testing I see that is not implemented..
is there a workaround ?
Thanks

Uno supports x:Uid for localization in XAML, but requires some manual setup in addition to the steps described here.
You should already see a line in the .csproj head for each target platform similar to the following (assuming a solution created from the VSIX template as a starting point):
<ResourcesDirectory>..\MyApp.Shared\Strings</ResourcesDirectory>
You need to add an ItemGroup to the csproj containing a line for each localization file:
<ItemGroup>
    <XamlCodeGenerationResourceFiles Include="..\MyProject.Shared\Strings\en\Resources.resw" />
    <XamlCodeGenerationResourceFiles Include="..\MyProject.Shared\Strings\es\Resources.resw" />
    <XamlCodeGenerationResourceFiles Include="..\MyProject.Shared\Strings\fr\Resources.resw" />
   ...etc...
  </ItemGroup>

Once this is added it should work correctly.

Yes, you are right it works!!
Btw.
Can you do a fac file for localization support, including my tips that you have to delete the default strings.xml file on the shared project. It will help other.
thanks!


[docs] Add notice of project closure to README

Make it clear that the project is not being developed anymore with a notice at the beginning of the README file.

lgtm


Feat:bounty submission for user and pending bounty list for admin

PR Fixes:

1 Users can now submit Bounties and check the status of their bounties and the amount they received
2 Admins can see pending bounties and review them and update the amount paid to the user and theyll easily have access to the account addresses of whoever submitted the bounty request,
3 Admins can also view the total amount of bounties disbursed

Fixes https://github.com/code100x/cms/issues/1090
User:
https://github.com/user-attachments/assets/3aa6e10d-2099-484a-83b6-1ac168e673eb
Admin:
https://github.com/user-attachments/assets/326cb44d-3673-44bb-998c-a218f9ff5616

Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I assure there is no similar/duplicate pull request regarding same issue

@hkirat could you review this

Here are the videos with better clarity :
User:
https://drive.google.com/file/d/1VlcsJHGWfcxdxSUe96gNov8KYoKemUTG/view?usp=sharing
Admin:
https://drive.google.com/file/d/1RP2fLu_MnUI5uxruvT64uq3n1hgEqViH/view?usp=sharing

Ser , why are you letting users again fill up the payout details at bounty page which is already being done in payout methods right ?

Also when adding the PR link don't you think it must be validated so that users don't end up submitting others PR , which will make admin life hard to validate the user to confirm he/she is the one whose has got that much amount of bounty assigned .

Correct me if I am wrong ser .

Ser , why are you letting users again fill up the payout details at bounty page which is already being done in payout methods right ?

its a place to edit the address details quickly, we can remove it and redirect the user to the payout method page but i think its redundant although we can put a button which takes them there but its extra steps i believe. unless you have some other idea that can be worked on.(maybe i could combine bounty and payout method page but it would be too cluttered)

Also when adding the PR link don't you think it must be validated so that users don't end up submitting others PR , which will make admin life hard to validate the user to confirm he/she is the one whose has got that much amount of bounty assigned .

how do you propose we should validate? with links itll be tricky. maybe we can display the users github along with the link so that admin can check if its the same person who submitted the pr which is the easiest solution i think.

@nischal-shetty2 linting checks have been failed sir

its related to this #956 , i ran lint:fix and format:fix before committing ser don't worry!

@hkirat @siinghd could y'all review this

Added this for easier admin clarification
https://github.com/user-attachments/assets/b60ae8e7-cad3-4031-a4c4-47b77872a750

@siinghd @devsargam @hkirat review required

@siinghd @devsargam @hkirat guys check this out

@nischal-shetty2 I will check this today before eod

@nischal-shetty2 I will check this today before eod

Alright!

@devsargam ser let me bother you by reminding about this

@devsargam bhai 🥲

@nischal-shetty2 10pm today else again write msg 😄

@nischal-shetty2 10pm today else again write msg 😄

ser reminder @devsargam

@nischal-shetty2 pls check the above. I will check this later on got some other's to reivew.

@nischal-shetty2 pls check the above. I will check this later on got some other's to reivew.

@devsargam the above being?

The review sir

U need to add auth in all server actions

@devsargam exams are going on so couldnt do much, will reopen in 1-2 days,


Flash movement debuff doesn't go away

Description

After getting flashed, the debuff granted to you which SHOULD be temporary is, instead, permanent. This means getting flashbanged stops you completely.
Reproduction

Grab anything that can flash you (flashbang, flash)
Use it to flash yourself
Once the flash subsides, you should notice that the movement debuff granted to you while you're flashed is still there

I can't, for the life of me, figure out what's wrong.
RefreshMovementSpeedModifiers() get's called when SlowedDownComponent shutsdown, but for whatever reason the event fired inside doesn't have the default values of 1f, 1f to the Speed modifiers, and I think that's what's currently borking everything up.

I can't, for the life of me, figure out what's wrong. RefreshMovementSpeedModifiers() get's called when SlowedDownComponent shutsdown, but for whatever reason the event fired inside doesn't have the default values of 1f, 1f to the Speed modifiers, and I think that's what's currently borking everything up.

This seems related to the jetpack update the other day, #9023
It looks to be the changes to the MovementSpeedModifierSystem. Originally, it wasn't raising the RefreshMovementSpeedModifiersEvent in SharedStunSystem when called from OnSlowRemove(), so the movement speeds were reset to the default 1.
Now, upon shutdown, the event is being called in SharedStunSystem, and it is getting set to the SlowedDownComponent's modifiers.
It looks to me like sloth's changes are good, and the RefreshMovementSpeedModifiersEvent is supposed to be called in both cases, and SharedStunSystem just needs to update the OnRefreshMovespeed() to correctly handle being called both on init and remove?

I think you're onto something! I did some very crude tests and it does indeed "solve" the issue.
I just gotta figure out how to do what you mentioned without hardcoding the values inside OnRefreshMovespeed

Fixed in https://github.com/space-wizards/space-station-14/pull/9214


Does it work with rails-api?

Hi, I have my current project build with rails-api(https://github.com/rails-api/rails-api) but not rails. But somehow the rake swagger:docs task always shows "processed 0" to me. I tried the example with rails-api and it didn't seem work either. I wonder does it have something to do with it? Thanks!

@willawill did you make it work succesfully? I meet the same issue.


Make parse optional

Context: https://twitter.com/intrnl0/status/1438179142512504836
Two ideas:

Disable deserialization at all with parse: false
Allow providing another deserializer with parse: JSON.parse

This looks like a great first issue, can I pick this up later today?

Suee @chrstnfrrs


Inquiry Regarding Model Open Sourcing

Inquiry Regarding Model Open Sourcing

We are preparing the code. It will be released soon~

Wonderful. Please could you also confirm that the models and code are Apache 2.0?

Will the weights of Vim-S also be released?


skip_completing ignores true value and completes

Describe the Issue
skip_completing is being ignored when set to true
Action Configuration
  comment_parser:
    runs-on: ubuntu-latest
    steps:
      - name: Parse Issue Comment for Command
        id: branch_deploy
        if: github.event_name == 'issue_comment'
        uses<EMAIL_ADDRESS>        with:
          command: ".drift refresh"
          allowed_contexts: "issue"
          reaction: eyes
          permissions: write,admin,maintain
          skip_completing: "true" # we will complete the deployment manually

Relevant Actions Log Output
2024-10-09T16:59:20.5017204Z ##[group]Run<EMAIL_ADDRESS>2024-10-09T16:59:20.5017832Z with:
2024-10-09T16:59:20.5018198Z   command: .drift refresh
2024-10-09T16:59:20.5018779Z   allowed_contexts: issue
2024-10-09T16:59:20.5019230Z   reaction: eyes
2024-10-09T16:59:20.5019626Z   permissions: write,admin,maintain
2024-10-09T16:59:20.5020244Z   skip_completing: true
2024-10-09T16:59:20.5020940Z   github_token: ***
2024-10-09T16:59:20.5021340Z   status: success
2024-10-09T16:59:20.5021820Z   allow_drafts: false
2024-10-09T16:59:20.5022257Z   allow_forks: false
2024-10-09T16:59:20.5022623Z   skip_ci: false
2024-10-09T16:59:20.5023467Z   skip_reviews: false
2024-10-09T16:59:20.5023939Z   param_separator: |
2024-10-09T16:59:20.5024318Z   allowlist: false
2024-10-09T16:59:20.5024789Z   allowlist_pat: false
2024-10-09T16:59:20.5025217Z env:
2024-10-09T16:59:20.5025551Z   STACK_CONFIG_FILE: stack_config.yaml
2024-10-09T16:59:20.5026142Z   CHECKOUT_REF: main
2024-10-09T16:59:20.5026557Z ##[endgroup]
2024-10-09T16:59:21.2545858Z ✅ operation requested on an issue
2024-10-09T16:59:21.2548020Z 🚀 success!
2024-10-09T16:59:21.2844780Z Post job cleanup.
2024-10-09T16:59:22.0010498Z Evaluate and set job outputs
2024-10-09T16:59:22.0034672Z Set output 'continue'
2024-10-09T16:59:22.0035753Z Set output 'comment_id'
2024-10-09T16:59:22.0036329Z Set output 'initial_reaction_id'
2024-10-09T16:59:22.0037351Z Cleaning up orphan processes
Extra Information
Looks Like skip_completed is read as a bool, then a comparison is run against a string to determine whether to return, which always evaluates to false.
https://github.com/github/command/blob/a0f8c9c9a16aad0cced91b2b7693de599d4093a3/dist/index.js#L33632
    const skip_completing = core.getBooleanInput('skip_completing')

https://github.com/github/command/blob/a0f8c9c9a16aad0cced91b2b7693de599d4093a3/dist/index.js#L33646-L33650
    if (skip_completing === 'true') {
      core.info('⏩ skip_completing set, exiting')
      return
    }

Should treat as a bool as in branch-deploy action.
https://github.com/github/branch-deploy/blob/368e17cf44738ebe4ee145839973967ceaf4ae44/dist/index.js#L42642-L42648
    // Skip the process of completing a deployment, return
    if (skip_completing) {
      core.info(
        `⏩ ${COLORS.highlight}skip_completing${COLORS.reset} set, exiting`
      )
      return
    }

I will create a PR.

https://github.com/github/command/pull/44


[BUG] cellfinder_curate fails with "from PySide2.QtWidgets import QApplication ImportError: DLL load failed: The specified procedure could not be found."

Describe the bug
When running cellfinder_curate for training, I get the following error:
Traceback (most recent call last):
  File "c:\users\7057mehrav\appdata\local\continuum\anaconda3\envs\brainrender\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "c:\users\7057mehrav\appdata\local\continuum\anaconda3\envs\brainrender\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\Users\7057mehrav\AppData\Local\Continuum\anaconda3\envs\brainrender\Scripts\cellfinder_curate.exe\__main__.py", line 4, in <module>
  File "c:\users\7057mehrav\appdata\local\continuum\anaconda3\envs\brainrender\lib\site-packages\cellfinder\train\curation.py", line 7, in <module>
    from PySide2.QtWidgets import QApplication
ImportError: DLL load failed: The specified procedure could not be found.

To Reproduce
Following is the command I ran which produced this error:
cellfinder_curate "C:\Users\7057mehrav\Documents\new-file-structure\146\146-cells-rotated" "C:\Users\7057mehrav\Documents\new-file-structure\146\146-background-rotated" "C:\Users\7057mehrav\Documents\new-file-structure\146\output-rotated-rerun\cell_classification.xml" -o "C:\Users\7057mehrav\Documents\new-file-structure\146\output-rotated-rerun\training" -x 10 -y 10 -z 10

Expected behavior
Training data is generated
Desktop (please complete the following information):

OS: Windows 10

Additional context
I looked into this error and found this stackoverflow thread related to it. I have tried the accepted solution but it didn't change anything for me. Do I need to have specific versions of PySide2 and shiboken2?

Fixed it. So basically PySide2, shiboken2 and PyQt5 all three have to be the exact same version (in this case 5.15.0) so pip upgrade of PyQt5 fixed the issue

Sorry for the delay, and glad you found a solution.
I wonder if this is an issue with the various versions required by cellfinder, or whether it's an issue with other software you have installed in the environment. Does this same issue appear in a fresh conda environment when only cellfinder has been installed?
I ask because you shouldn't need PySide2 at all for cellfinder.


Preparing v0.8.11a2 pre-release

The build for this PR failed yesterday because the axisregistry package has a too strict dependency rule for the protobuf version.
I will relax the requirement on axisregistry's setup.py file and cut a new release so that it can be used by fontbakery


Normalize open*settings commands args

VSCode Version: 1.41.1
OS Version: Win10 1909

Steps to Reproduce:
In an extension:

commands.executeCommand('workbench.action.openWorkspaceSettings', 'telemetry');
Workspace settings open, but without a search filter

No, as there is nothing to run that code

Why do you need this? You can use workbench.action.openSettings but I don't really want to provide a ton of commands to control all the different options in the settings UI.

There is a difference between User settings or Workspace settings, there is different UI and comparable methods in the API to reach them. The openWorkspaceSettings has the same signature as openSettings, but they don’t behave the same way. That’s why I think this needs to be resolved to be consistent.
Martin
Martin Tirion  |  senior software engineer |  CSE | Agile Response Team | Spatial Computing | Microsoft
Tel +31 (6) 1250 6153  |  Twitter @mtirion  |  Blog mtirion.nlhttp://mtirion.nl/
From: Rob Lourens<EMAIL_ADDRESS>Sent: vrijdag 7 februari 2020 01:40
To: microsoft/vscode<EMAIL_ADDRESS>Cc: Martin Tirion<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoft/vscode] workbench.action.openWorkspaceSettings doesn't take the search parameter (#90086)
Why do you need this? You can use workbench.action.openSettings but I don't really want to provide a ton of commands to control all the different options in the settings UI.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fvscode%2Fissues%2F90086%3Femail_source%3Dnotifications%26email_token%3DAOG22BLAAA6ZYB4TFSDW2WTRBSUUPA5CNFSM4KQNYVJ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELBJ7TQ%23issuecomment-583180238&data=02|01|Martin.Tirion%40microsoft.com|c8f44b209b1c4428740508d7ab6634af|72f988bf86f141af91ab2d7cd011db47|1|0|637166327775961166&sdata=Z%2BPL3izfV5c4NvoO2FDJCM9LfxzZg1R6w%2BuKkv87dVY%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOG22BNM6MKSMRMEDZONFH3RBSUUPANCNFSM4KQNYVJQ&data=02|01|Martin.Tirion%40microsoft.com|c8f44b209b1c4428740508d7ab6634af|72f988bf86f141af91ab2d7cd011db47|1|0|637166327775971123&sdata=fNuRatI1TCvGZr%2BZc0785sdmaFUoVpoYEumFHApMcAo%3D&reserved=0.

Same issue. I just want to open the workspace settings and filter by my extension name.

Same here!

+1

+1

This is exactly what I need to programmatically open the workspace settings for my extension, allowing users to easily make changes.
For anyone who needs this feature, execute openSettings with your search, then execute openWorkspaceSettings. It will switch the current settings window over to workspace settings while keeping your search string.
Example
vscode.commands.executeCommand('workbench.action.openSettings', 'telemetry' ); vscode.commands.executeCommand('workbench.action.openWorkspaceSettings');

Thanks @SpenceV1  and @JensWallgren for documenting this workaround
I had this same need btw.


[FLINK-8801][yarn/s3] Fix jars downloading issues due to inconsistent timestamp in S3 Filesystem

What is the purpose of the change
This change reverts the original fix for FLINK-8801's due to its insufficient in some scenarios where S3AFilesystem does not implement setTimes methods.
Instead, it uses retry to wait for the remote file to be available and overwrite the local file timestamp.
Brief change log

Revert PR 5602
Add retry at FileNotFoundException

Verifying this change
This change is already covered by existing tests, such as YarnFileStageTestS3ITCase for the upload path via S3.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: Yarn
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@yanyan300300 thanks for your contribution. would you mind creating an issue for this, and update the title and description of this patch

Indeed, it looks like this is the better approach. Aside from some minor details in the code, I would only have one concern which I'm not sure about:

Are we allowed to change the local resource's time? What are the implications?
Yes, Flink will need to set the local resource's time explicitly. This is also the case before your change.

I guess, this is the local resource file that was uploaded to the JM and there are no further implications but I'm not 100% sure.

@tillrohrmann @aljoscha Could you kindly review this change? Thanks!

@tillrohrmann @aljoscha @NicoK Could I get a review for this PR? Thanks!

Indeed, it looks like this is the better approach. Aside from some minor details in the code, I would only have one concern which I'm not sure about:

Are we allowed to change the local resource's time? What are the implications?

I guess, this is the local resource file that was uploaded to the JM and there are no further implications but I'm not 100% sure.

Yes, Flink will need to set the local resource's time explicitly. This is also the case before your change.

Can you clarify a bit on this? Where was this done?
My problem is only that Utils#setupLocalResource is not only used for the JobManager uploading resources to remote paths but also by Utils#createTaskExecutorContext() to copy configuration files into the user's home directory. If we now change the timestamp of that config file, I don't know whether this would impact YARN or our own code regarding the integrity of that file.

Thanks @tillrohrmann, yes, I must have mixed things up there - you are right about setting the timestamp of the LocalResource. I added one more small comment and after addressing this, I would also be fine with merging this.

@NicoK Could you approve this PR if you don't have further concerns?

@yanyan300300 can you squash your commits so that we'll have one commit for reverting the old implementation and one with your changes? Please also tag your commit with "[FLINK-8801]" then.
I'll merge it afterwards.

@NicoK Thanks for the approval!
If I squash in my feature branch, this PR will be closed due to altered commit history. Could you do it at your end? https://help.github.com/en/articles/about-pull-request-merges

@yanyan300300 strange - I do this quite frequently in my PRs - squashing WIP git history, then force-push to my branch and the PR gets updated. But I can also do this on my end.

merged via 770a40476e7aea4c53babc39e3edc922a3d6a4e7

@yanyan300300 strange - I do this quite frequently in my PRs - squashing work-in-progress git history, then force-push to my branch and the PR gets updated. But I can also do this on my end.

Thanks @NicoK Will do this next time.


Docker: Debug-friendly docker images

... Add support to the Makefile for generating a "production" image that is based on scratch, and a "development" or "debug" image that is based on an alpine or bash image. For instance:
make docker-gen DEBUG=true

Originally suggested by @leviable in #23

A naive solution I can think of is to create another dockerfile that would be run for the debug builds (this one won't have multi-stage builds) - is there a better way to implement this?

For instance: make docker-gen DEBUG=true

For the command, creating a new Makefile command instead of adding a variable sounds better, something like make docker-debug (open to suggestions). Since most shells support tab-completion with Makefile commands, just three taps would be enough to get the command, with no chances of typos!
(Pulling in my comment from the linked issue to move the conversation here)

Going ahead with @leviable's suggestion to use build-args in the Dockerfile, passing these build args through the Makefile commands - that's definitely better than my plan of dealing with two separate Dockerfiles


Cf image enrichment overivew

[CI] Staging environment is at http://cap-staging-docs.codefresh.io.s3-website.us-east-1.amazonaws.com/cf-image-enrichment-overivew/docs/

[CI] Staging environment is at http://cap-staging-docs.codefresh.io.s3-website.us-east-1.amazonaws.com/cf-image-enrichment-overivew/docs/

[CI] Staging environment is at http://cap-staging-docs.codefresh.io.s3-website.us-east-1.amazonaws.com/cf-image-enrichment-overivew/docs/


Double Quotes in labels during Annotation

Adding double quotes in the labels while annotating makes the label look like “label" when I would expect it to be like "label".
See:

and

This is actually an autocorrect issue! Note, the label below actually has the correct quotations!

The label at first autocorrects to the secondary quote.

But the user can actually go back and recorrect them!

Good luck! Closing the issue for now!


Call convenience quick start

Where's the source code @ericgiannini?

Do you have screenshots of what we are trying to accomplish in the UI?
пятница, 25 мая 2018 г. пользователь Chris Guzman написал:

Where's the source code @ericgiannini https://github.com/ericgiannini?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Nexmo/stitch-ios-quickstart/pull/10#issuecomment-392075160,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMCHtYIMvI0SN_SpfQzSNtWNwAffAoGhks5t2BS_gaJpZM4UNdas
.

--
Отправлено с мобильного устройства

@ChrisGuzman After the PR I will port the .md.


Review the changes after updating an existing project

User Story
As a wrangler I want to "Review the changes" so that I can I can make sure the changes I have made look correct
Acceptance Criteria

closing as this was already possible with the approach to updates we took


configure max_item_size for queryFrontEnd and Store deployments

when maxItemSize is specified

/lgtm

we should have a pr in observatorium repo before this one, since it uses the jsonnet file there?

@subbarao-meduri we should have a pr in observatorium repo before this one, since it uses the jsonnet file there?

Per our conversation, max_item_size does not exist in upstream observatorium CRD, and this is an ACM extension. We forked community observatorium as stolostron/observatorium-deployment to drive this change.

/test e2e-kind

/test e2e-kind

/test e2e-kind


Add raylib-cpp to vcpkg

I noticed that raylib-cpp is not available in the vcpkg registry. I believe it would be beneficial for many users if this library could be added to vcpkg.

Feel free to add it. raylib-cpp itself is header-only, so all you really need is raylib.


Add recurringWindow to the GKE maintenancePolicy

This schema is missing the recurringWindow and this does not allow me to create a GKE cluster with a specific maintenance policy, as specified i the official documentation: https://cloud.google.com/kubernetes-engine/docs/reference/rest/v1/projects.locations.clusters#Cluster.RecurringTimeWindow
This simple edit will permit this (lines 909 to 930). All property descriptions are the same as the official documentation.
Thank you.
Sincerely,
Fausto

Hi there.
I'm not sure that I understood the status of this request. Shall I do any further action or add more info for you? Or is it just question of time to approve my code?
Please let me know if I can be helpful.
Thank you so much.
Regards,
Fausto

Hi any news on this PR? Also I need the recurring window! @
@morgante @ocsig @sjvanrossum


corrected a mistake.

.

Please can you send me a developer swag.
On Mon, 8 Jul, 2019, 6:33 AM Aron Eidelman<EMAIL_ADDRESS>wrote:

@h3xar0n requested changes on this pull request.
I think the intention was to have it in the past tense. By the time you
read it, it already wrote.
( @locao https://github.com/locao , want to check with you before
closing.)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Kong/docs.konghq.com/pull/1389?email_source=notifications&email_token=AKSZ5O52JXXR4UH4RA7DVJTP6KG73A5CNFSM4H6WHZH2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB5VL7ZA#pullrequestreview-258654180,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKSZ5O4S6CX6MWEBKXK3OSTP6KG73ANCNFSM4H6WHZHQ
.


Turn off when video stops

Hi, I have the min light set on Hyperion but when the video stops I would like all of the Leds to goto zero, can you do this via the plugin?
Also can you make this only work on Movies, not Live Tv etc?
Thanks

There will be a control plugin for kodi (future) available. To get more control over this.
/Closed


Add option to create gRPC async client without checking for cluster.

title: Add option to create a async gRPC client which defers cluster check
Description:
In issue https://github.com/envoyproxy/envoy/issues/2785 we discussed adding an option s.t when creating a gRPC client the check for the presence of the cluster is skipped.
This PR adds the boolean to be enable it.
As it will now be possible that the underlying cluster is not there I also added a check in the gRPC client implementation that if httpAsyncClientForCluster throws an exception then the session will be closed with status Unavailable.
Risk Level: Low
Low: Default behavior is not changed.
Testing:
Added corresponding UT's
Note: I know the repo is getting re-org'd and the merge won't happen. But I wanted to get the PR up so that we can get agreement that the implemented logic is what we want.
Signed-off-by: Saurabh Mohan<EMAIL_ADDRESS>

@mattklein123 @htuch fyi. related to issue https://github.com/envoyproxy/envoy/issues/2785#issuecomment-376677125

@saumoh please merge master to resolve merge conflicts.

@htuch Will ping u once i've addressed the existing feedback and have all the checks green.


[Bash] read with -r to not mangle backslashes

As proposed by static analysis tool:
https://github.com/koalaman/shellcheck/wiki/SC2162

Can one of the admins verify this patch?

Jenkins, test this please

Jenkins, test this please.


Add self-managed environment implementation

This does not rely on environ for anything other than the starting variables.

LGTM!

Fixed the CI failing
@K1ngmar