Start accumulate gradients schedule at epoch 0

What does this PR do?
Fixes #2480. If the pl.Trainer option accumulate_grad_batches was an integer then the first epoch (epoch 0) would use accumulate_grad_batches=1 while the remaining epochs would use accumulate_grad_batches=<user_value>. This was caused by #2289.
Before:

After:

Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
[x] Did you make sure to update the documentation with your changes?
[x] Did you write any new necessary tests?
[x] Did you verify new and existing tests pass locally with your changes?
[x] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

The tests/trainer/test_lr_finder.py::test_accumulation_and_early_stopping test failed because the _total_batch_idx assert value was changed in #2375 from 100 * 2 to 190. I think this change was a mistake. The _total_batch_idx changed to 190 because the first epoch had accumulate_grad_batches set to 1. Since this pull request fixes the accumulate_grad_batches bug for the first epoch, I have changed this test back to the previous value.
        assert before_lr != after_lr, \
            'Learning rate was not altered after running learning rate finder'
        assert len(lrfinder.results['lr']) == 100, \
            'Early stopping for learning rate finder did not work'
>       assert lrfinder._total_batch_idx == 190, \
            'Accumulation parameter did not work'
E       AssertionError: Accumulation parameter did not work
E       assert 200 == 190
E         +200
E         -190

Hi, thanks for the PR. The test that you changed is for LRFinder, it looks ok, but do you have an idea why the other test test_gradient_accumulation_scheduling in test_trainer.py did not fail? It seems like an important test but it is invariant to this change. It might be because this test has a very specific example in which this is not detected. I wonder if it could be parameterized to cover several cases.

The tests/trainer/test_lr_finder.py::test_accumulation_and_early_stopping test failed because the _total_batch_idx assert value was changed in #2375 from 100 * 2 to 190. I think this change was a mistake. The _total_batch_idx changed to 190 because the first epoch had accumulate_grad_batches set to 1. Since this pull request fixes the accumulate_grad_batches bug for the first epoch, I have changed this test back to the previous value.

this test has a bug, it shall be 200, pls correct it :]

Hi, thanks for the PR. The test that you changed is for LRFinder, it looks ok, but do you have an idea why the other test test_gradient_accumulation_scheduling in test_trainer.py did not fail? It seems like an important test but it is invariant to this change. It might be because this test has a very specific example in which this is not detected. I wonder if it could be parameterized to cover several cases.

Prior to this pull request the test_gradient_accumulation_scheduling test did not fail because the schedule is {1: 2, 3: 4}, which falls under the dict section of the if...else block in configure_accumulated_gradients(). In the GradientAccumulationScheduler callback the line scheduling.update({0: 1}) set the accumulate_grad_batches to 1 for the first epoch, which is tested in test_gradient_accumulation_scheduling in the line assert self.trainer.accumulate_grad_batches == 1 which will pass. This did not raise an error because the 0th epoch in the {1: 2, 3: 4} schedule is not set and because a dict was tested, not an int. If an int were tested (lets say 3) and the asserts were updated to always check for a value of 3, then the test would have failed because epoch zero would have accumulate_grad_batches=1. Epoch zero would have accumulate_grad_batches=1 because the line schedule = {1: accumulate_grad_batches} in configure_accumulated_gradients() did not define the first epoch, which scheduling.update({0: 1}) in GradientAccumulationScheduler sets to 1.
After this pull request the test_gradient_accumulation_scheduling test did not fail either because I modified the code in the if...else block in configure_accumulated_gradients() that handles ints, but a dict is tested in this test.
The test_gradient_accumulation_scheduling test should check an int as the value for accumulate_grad_batches. This addition to the test would have caught this error.

@HHousen would you like to modify that test so that it would fail on master and pass here? Otherwise the error could sneak back in the future without us noticing :)

@HHousen would you like to modify that test so that it would fail on master and pass here? Otherwise the error could sneak back in the future without us noticing :) I can help if you need

Sure. I'll parametrize the test so an int can be tested as well as a dict.

In the else block of _optimizer_step in the current test, why is 3 added to self.prev_called_batch_idx. Shouldn't 4 be added since the accumulate_grad_batches is 4?

Hello @HHousen! Thanks for updating this PR.

In the file tests/trainer/test_trainer.py:

Line 138:120: E501 line too long (152 > 119 characters)
Line 143:44: E226 missing whitespace around arithmetic operator
Line 144:62: E226 missing whitespace around arithmetic operator
Line 154:44: E226 missing whitespace around arithmetic operator
Line 155:62: E226 missing whitespace around arithmetic operator
Line 164:44: E226 missing whitespace around arithmetic operator
Line 165:62: E226 missing whitespace around arithmetic operator

In the else block of _optimizer_step in the current test, why is 3 added to self.prev_called_batch_idx. Shouldn't 4 be added since the accumulate_grad_batches is 4?

Nevermind, it should be 4.

I updated the test_trainer.py::test_gradient_accumulation_scheduling test. It had a few issues that prevented it from properly testing what it was designed to test. Namely, the batches per epoch was too low and only two epochs were processed but there were tests designed for greater than 3 epochs. I have updated and parametrized the test. The master branch should now have 2 fail and 1 succeed, whereas this pull request should have 3/3 succeed.

@HHousen solid catch!!

Sorry. I tried to remove the two dataset files that were downloaded when I ran the tests since I accidentally committed them. This did not work since force-pushing closed this pull request. I undid the force-push by reverting back to e0cb028, but I still can't reopen since GitHub says the branch was "force-pushed or recreated."
git reflog show bugfix/2480_grad-accumulate-0-epoch:
e0cb0285 (HEAD -> bugfix/2480_grad-accumulate-0-epoch, origin/bugfix/2480_grad-accumulate-0-epoch, refs/original/refs/heads/bugfix/2480_grad-accumulate-0-epoch) bugfix/2480_grad-accumulate-0-epoch@{0}: reset: moving to origin/bugfix/2480_grad-accumulate-0-epoch@{1}
c3b8cca4 bugfix/2480_grad-accumulate-0-epoch@{1}: filter-branch: rewrite
e0cb0285 (HEAD -> bugfix/2480_grad-accumulate-0-epoch, origin/bugfix/2480_grad-accumulate-0-epoch, refs/original/refs/heads/bugfix/2480_grad-accumulate-0-epoch) bugfix/2480_grad-accumulate-0-epoch@{2}: commit: Fix pep8 formatting
6420265c bugfix/2480_grad-accumulate-0-epoch@{3}: commit: Update test_trainer.py::test_gradient_accumulation_scheduling
f1f9b6f8 bugfix/2480_grad-accumulate-0-epoch@{4}: commit: Undo change in #2375
2e199c84 bugfix/2480_grad-accumulate-0-epoch@{5}: commit: Start accumulate gradients schedule at epoch 0
e5a97999 (origin/master, origin/HEAD, master) bugfix/2480_grad-accumulate-0-epoch@{6}: branch: Created from HEAD

It looks like everything should have been reverted. Any suggestions?

I suggest you simply submit an new PR.

 Then we reopen this PR (it would be a sad to lose all this valuable discussion and contribution)
git checkout bugfix/2480_grad-accumulate-0-epoch
git reset upstream/master
.... add the changes......
git commit....
git push -f

I have to open a new pull request since GitHub will not let me reopen this one even if the branch is at the state right before the request was automatically closed. Moved to #2513. It is strange though since it's the same exact branch in both requests.

I have to open a new pull request since GitHub will not let me reopen this one even if the branch is at the state right before the request was automatically closed.

It was closed because the PR state was indetical with target branch... So if there is any difference it allows you to reopen the PR...

I have to open a new pull request since GitHub will not let me reopen this one even if the branch is at the state right before the request was automatically closed.

It was closed because the PR state was indetical with target branch... So if there is any difference it allows you to reopen the PR...

I have to open a new pull request since GitHub will not let me reopen this one even if the branch is at the state right before the request was automatically closed.

It was closed because the PR state was indetical with target branch... So if there is any difference it allows you to reopen the PR...

I rewrote history by accident which closed the PR because there were no commits in common with master. I then reset back to the commit directly before the rewrite, so everything was in the same state as it was before the rewrite. It would still not let me open after this. There are currently 5 additional commits on this branch, which #2513 recognizes but this PR will not. GitHub should let me reopen a PR if the branch to be merged is in the same state that it was in directly before the closure. Since it would not let me do this, I added a commit to remove the accidentally committed files and opened #2513. Both PRs are for the same branch, which should not be possible to my understanding. The only reason it is possible is because this one was closed and is no longer tracking changes.
Essentially, this PR would not recognize that the state of the branch was reverted and thus would not let me reopen.


Add DeliverMax field in API responses; hide/remove Amount field for api_version 2

As of 2023-10-03, this is in-development for rippled, so the details may change

https://github.com/XRPLF/rippled/issues/3484
https://github.com/XRPLF/rippled/pull/4733

Needs investigation:

Might not need to make changes (other than updating libxrpl) if libxrpl generates the JSON

The PR of rippled has not finished yet. From the current implementation, there is direct change of handlers , Clio will need to update.

submitted #979


Invalid parameters are appended when using the navigation URL

When a table is configured, and the navigation URL is used such as: /dynamic/TestModule/nav-test-details?id={{selectedRow.id}}, upon right clicking/hovering an additional parameter get appended after the selected row if such as: undefined: https://function-adminportal-test.shesha.dev/dynamic/TestModule/nav-test-details?id=20bada52-3033-45b1-95ee-c5ce3f5f06a4?undefined
Upon updating the configurations, maybe using form then back to Url, upon right clicking an additional question mark gets appended: https://function-adminportal-test.shesha.dev/dynamic/TestModule/nav-test-details?id=3c28d1d9-5f9b-4c80-90e2-0461feedbefd?

Steps to reproduce the behavior:

Go to 'function site'
Configure a new table from scratch and from navigating to details view, use URL option
Save your changes and render the form
Hover on the navigation icon and check the url/right click to see the full URL
Actual Results: undefined is appended at the end of the id and ? after updating the configurations

Expected behavior
The navigation link feature should append the parameters configured only

Related Item #1595


1% Performance drop Apollo vs Retrofit

Question
Hello,
Background:
We are migrating to GraphQL. This is what our code does:

user press a button
launch a new screen fragment with transition animation. At the same time, make a prefetch call to some API. In control group, we use Retrofit. In enabled group, we use Apollo.
the time it takes for the screen transition to finish is 1% longer in enabled group.

Issue investigation:
A. I don't quite understand how would this GraphQL vs Retrofit affect screen transition. The prefetch call is a fire and forget that does not affect screen transition in anyway.
B. What I am suspecting is Apollo is doing some kind of work possible on the main thread. Since Android animation usually runs in main thread, maybe Apollo running code in main thread is causing this 1% drop of performance.
C. 2nd very wild theory is the worker thread Apollo is running on may be sharing the same core with main thread, hence the drop.
D. 3rd theory is we are using the default fetch policy, maybe it is checking Apollo store cache on main thread and takes some extra time? (In our Retrofit implementation, we check our own custom cache first. In our Apollo implementation, we rely on Apollo's cache).
Question:
What's the heaviest thing Apollo internally does? Like payload preparation, Json parsing, encrypting/encoding http traffic, runtime reflection or something else? Does Apollo do anything heavy on main thread?

Hi!

Request composition happens on the thread a query was executed (typically, the main thread)
For anything network related Apollo uses OkHttp (same as Retrofit), and calls are executed on Dispatchers.IO by default
Json parsing also happens on Dispatchers.IO
There is no runtime reflection in Apollo thanks to the code generation (unlike Retrofit)

(Not mentioning caching as I am guessing you are not using it for now.)
You may also find interesting information about threading in this design doc (but a warning that it may not be up to date as we have been trying to tweak things in this PR).

@BoD Thank you! That's very educative.
For now, I will disable apollo cache with .fetchPolicy(FetchPolicy.NetworkOnly) and see if it helps speed up performance in our case.

@ynkm169 you can also disable the cache completely (as in not calling ApolloClient.Builder.normalizedCache()) and it will save a few more CPU cycles. This skips an interceptor annd removes a few allocations and method calls. It's most likely very negligeable compared to network I/O but I'd thought I'd mention it for your experimentations.
As with everything there are multiple factors at play:

GraphQL POST data is typically larger than a REST call. You can look into Automatic Persisted Queries to improve this
GraphQL merged fields require to buffer the Json stream which can introduce latency for some queries. You can look into responseBased codegen to improve this
Because GraphQL Json field order is deterministic, it opens up a few optimisation that make parsing potentially a bit faster (although I haven't been able to witness that just yet)
The time for your backend to process a REST vs GraphQL request might be different

Overall, we're aiming to be on par with Retrofit. And more importantly, we're aiming to do all processing/parsing amortized while waiting for network I/O. So the time it takes to do a query should be roughly the same as the time it takes to execute the network call.
In these conditions, the most important contributors to any difference are be 1. and 4. which are outside Apollo Kotlin's control.
I would consider 1% to be negligible at this point but if it's important to your use case, 2 way forward are:

Enable AutoPersistedQueries
Probe processing time on your backend for 2 equivalent queries
Look into responseBased codegen. This codegen comes with caveats and trade offs so I'd recommend you do that last as it'll require some time to setup correctly.

Hope this helps!

Thanks all! That's great amount of info for us to dig deeper into the performance difference!
I just added a bug ticket/feature request here https://github.com/apollographql/apollo-kotlin/issues/4604


Can't create Ignition on AWS

Hi Gents, I was not able to create Ignition at AWS, for some reason, I am following the docs, but stack is constantly rolling back...
missing dependencies. Should be template, YAML updated or something?
Regards, Ilya.

I am having this problem as well. The rollback reason is always:
"Parameter validation failed: parameter value for parameter name KeyPairName does not exist. Rollback requested by user."

I am having this problem as well. The rollback reason is always:
"Parameter validation failed: parameter value for parameter name KeyPairName does not exist. Rollback requested by user."

Hey Jason, I gave up.. I think that you need supply many parameters in template, there are some examples, but hard to understand what needed... I had same messages as you experienced.. but more often I had dependencies can't be resolved or something.. given fact that new releases and all stuff.. I gave up... I just fired up a new VM in EC2..  and then having access I build environment. Manually,  I think that if you could have a ready stack tested , would be great..  I had similar experience with PTC ThingWork, it just ready in couple minutes, in Singapore data center. Deploying stack to Australian or US datacenter, I had similar issue.. so I decided just use windows 22 stack + SQL express + Ignition 26


How to install this?

Hi,
how can I install this package?
Would be nice if there would be an example in the readme.
Good job!

@sowinski see #47, does that help?

Yes thank you!


Update query API

In this PR we update the library with the new version of core:

JS library is updated to support the latest changes in the Query format;
VBuilders are removed and substituted for vBuild()/buildPartial() calls.

Additionally, some test output is now muted, since it prints errors from negative case tests.

@armiol, @alexander-yevsyukov, PTAL.


autogenerate invoice number & bugfixes

What

Implemented autogenerate invoice number for mobile
Bugfixes in clientSelction on both web & mobile
Refactored the client_list partial

@prasanthchaduvula I've fixed it. Please check.


WIP Elevator Dependent Drive Speed

Reduces Drive Speed with Elevator height, please review code as well as "Will this work?" (Thought Testing.)

"Will this work?" (Thought Testing.)

@sMyn42  What does this mean?

Also, this PR has merge conflicts, please revise and push


Replace kubectl setting with commands.

I resolved the DCO
also update the kuttl docs page with the same: https://github.com/kudobuilder/kuttl.dev/pull/3


List all available tags in the system

Hello,
how can I retreive all the tags that are available in the current system?
i've various elements that have tags, i was looking for a function that gives me the set of all the tags that were used.

If you are using just an out-of-the-box setup, your tags are storred in the Tag model (found in taggit.models). If this is a custom model (for example you have your own models derived from ItemBase), then you'll need to query that one instead.
So if you are using the standard setup, Tag.objects.all() will give you the tags.


r4.large is not listed

For some reason, the table leaves off the r4.large. This instance exists on the Amazon EC2 page: https://aws.amazon.com/ec2/instance-types/. It has 2 vCPUs and 15. 25 GB RAM.

Can't reproduce this, see the image below:


[BUG] Cannot disable logging when using compiler.watch or compiler.run

What is the current behavior?
Too many logging messages in console related to process. I am unable to disable these messages. Tried using quiet: false, no-info: true, 'info-verbosity':'none', infoVerbosity:'none', stats : false in options
If the current behavior is a bug, please provide the steps to reproduce.
Try to write own script to require web-pack and run watcher.
webpack(config).watch(options, (err, stats)=>{
//err is null
})

What is the expected behavior?
One should be able to configure logging level.
Please mention other relevant information such as the browser version, Node.js version, webpack version, and Operating System.
"webpack": "^4.6.0",
"webpack-cli": "^2.0.15"

Node.js 9.11.1
Microsoft Windows 10

Update
Logging is being done by one of the hooks (error in afterCompile), but still error text is too much, cannot even scroll to the run command in cmd.


.bazelignore the modules/ dir.

With the addition of overlays (#1566) it now can make sense to have BUILD and .bzl files inside the module version directories. However they don't necessarily make sense on their own. They are only supposed to work in the context of the source archive they overlay.
To prevent issues like not being able to load() from a //:def.bzl that does not exist in the registry repo, let's just .bazelignore the modules/ dir.

We had Bazel test targets to validate the metadata.json files against the metadata.schema.json file. These are no longer functional because the glob at https://github.com/bazelbuild/bazel-central-registry/blob/main/BUILD#L3 no longer finds any inputs :(
I think we need a more nuanced answer. In other repos I use --deleted_packages in .bazelrc to mark certain BUILD files as "not denoting a Bazel package in this folder". I'm going to send a PR to switch to that.

And of course while the tests were disabled, two new packages were added that violate the schema:
FAIL: //:test_metadata.modules/marisa-trie (see /private/var/tmp/_bazel_alexeagle/76616cfd9dcc8a6cdc22c2cc69d5859f/execroot/_main/bazel-out/darwin_arm64-fastbuild/testlogs/test_metadata.modules/marisa-trie/test.log)
INFO: From Testing //:test_metadata.modules/marisa-trie:
==================== Test output for //:test_metadata.modules/marisa-trie:
modules/marisa-trie/metadata.json invalid
[
  {
    instancePath: '/maintainers',
    schemaPath: '#/properties/maintainers/minItems',
    keyword: 'minItems',
    params: { limit: 1 },
    message: 'must NOT have fewer than 1 items'
  }
]
================================================================================
FAIL: //:test_metadata.modules/libcpuid (see /private/var/tmp/_bazel_alexeagle/76616cfd9dcc8a6cdc22c2cc69d5859f/execroot/_main/bazel-out/darwin_arm64-fastbuild/testlogs/test_metadata.modules/libcpuid/test.log)
INFO: From Testing //:test_metadata.modules/libcpuid:
==================== Test output for //:test_metadata.modules/libcpuid:
modules/libcpuid/metadata.json invalid
[
  {
    instancePath: '/maintainers',
    schemaPath: '#/properties/maintainers/minItems',
    keyword: 'minItems',
    params: { limit: 1 },
    message: 'must NOT have fewer than 1 items'
  }
]
================================================================================


Summary of time for the current day

Is your feature request related to a problem? Please describe.
Not related to any issue
Describe the solution you'd like
A summary of how above/below the current time you are for the current day
Describe alternatives you've considered
No alternatives considered
Additional context
Mockup of feature request:

Related to #22

Related to #22

Think this may duplicate #22 as well? Will display time per day alongside daily time delta against schedule.

Duplicate of #22


feat(coverage): TypeScript transpile runner prototype

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#3660  👈
#3659
#3658
main

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

Join @Boshen and the rest of your teammates on  Graphite


lsd on Fedora is outdated

The Fedora 35/36/Rawhide repos are still on lsd 0.20.1. Any chance to update these?
Cheers!

None of the distribution specific packages are maintained by us. You will have to reach out to the Fedora package maintainers.
https://packages.fedoraproject.org/pkgs/rust-lsd/lsd/


2.6.6 - Adding inline horizontal volume control

I've added the inline horizontal volume control I created for a project I was working on.

Hello, its says there is code attached but i can't see any code or file for downloading. Regards, Michael

Thanks so much for this. I've used your ideas to introduce the feature into the main code branch. I can't merge this pull request but your work was great!

No problem John, glad to help


RNChat npm run android failed

Got this error:
Could not get unknown property 'mergeResourcesProvider' for object of type com.android.build.gradle.internal.api.ApplicationVariantImpl.

Add this to  /node_modules/react-native-push-notification/android/build.gradle
def DEFAULT_GOOGLE_PLAY_SERVICES_VERSION = "+"
def DEFAULT_FIREBASE_MESSAGING_VERSION = "+"

@YoungDu does it resolve your issue?

@YoungDu does it resolve your issue?

nope

@YoungDu ok, could you provide more info re this issue
Dod you run the build using the command npm run android ?
Please post the complete output of the log error
We do not face this issue at your side, need to get more info to understand what is the reason  and how we can help

Add this to /node_modules/react-native-push-notification/android/build.gradle
def DEFAULT_GOOGLE_PLAY_SERVICES_VERSION = "+"
def DEFAULT_FIREBASE_MESSAGING_VERSION = "+"

That is not the solution
your app has so many issues likes:
-missing packages
-not using androidx
and still using the old deprecated compile instead of implementation
for this particular issue, the solution if found here
https://github.com/wix/react-native-navigation/issues/4757#issuecomment-468133753
I will make a push request where i resolved those issues if you wish so

@gilons we will be much appreciated 🥇

hello @DaveLomber i cannot push ot repo i'm getting 403 http error

@gilons could you please create a fork of the repo, and then make a PR from your fork to this main repo?

ok

ReactNativeJS ▶ TypeError: Invalid attempt to spread non-iterable instance
│
│ This error is located at:
│ in AppRoot (created by ConnectFunction)
│ in ConnectFunction (at App.js:10)
│ in Provider (at App.js:9)
│ in App (at renderApplication.js:40)
│ in RCTView (at AppContainer.js:101)
│ in RCTView (at AppContainer.js:119)
└ in AppContainer (at renderApplication.js:39)

I have resolved most of the errors and finally was able to run the sample cod but after running the above mentioned error occurred


Add Telemetry sample

Use RateLimiter.GetStatistics() + OpenTelemetry.Metrics

https://github.com/dotnet/aspnetcore/pull/47758/files + https://github.com/dotnet/aspnetcore/issues/47745


Openshift4

PR Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] Title of the PR starts with installer/product name (e.g. [ansible/artifactory])
[ ] CHANGELOG.md updated
[ ] Variables and other changes are documented in the README.md

@vinayagg @peters95  I have committed the changes for RT v7.17.5 and Xray v3.18.1
Please review the changes and merge them.

I have read the CLA Document and I hereby sign the CLA


"Enter Authorization header value" input field missing

We are facing similar issue with v1.3.0. Pls help to resolve this issue.

This repository has been closed and moved to laminas-api-tools/api-tools-documentation-swagger; a new issue has been opened at https://github.com/laminas-api-tools/api-tools-documentation-swagger/issues/5.


Post logs for MS and PFS services

Logs of the PFS and MS hosted services should be made available in the same way as the logs of the scenarios do. This can help to speed up the debugging process and make it easy for anyone to get access to the logs.

Requires #44, #59


Support TLS_PSK_WITH_AES_128_CCM_8 for IoT usecases

Summary
I'd like to request inclusion of at least one RFC 6655 cipher suite, TLS_PSK_WITH_AES_128_CCM_8. This suite is often used for IoT products, most notably IKEA Tradfri uses it for CoAP over DTLS.
Motivation
I'd like to use a Go native DTLS library in a number of my IoT projects. Some of these projects involve talking to (or exposing as) gateways that leverage this particular cipher suite as it's fairly common in IoT products. Currently bocajim/dtls exists which implements just that cipher.
Describe alternatives you've considered
I can always use this and the other DTLS library when I happen to need more than just TLS_PSK_WITH_AES_128_CCM_8. But I would much prefer to have one maintained library with all capabilities instead of being split between multiple packages from different maintainers.
Checklist

[x] Add CCM support #46
[ ] Add PSK support
[ ] Add TLS_PSK_WITH_AES_128_CCM_8

@bocajim Would it be feasible to contribute TLS_PSK_WITH_AES_128_CCM_8 to this library instead?

Hey @daenney
That would be fantastic! I am open to any/all improvements to the library. We are missing features just because they were not needed for pions/webrtc. I don't think I will have a lot of time, but would love to help this land.
This change won't be trivial, but we should totally do it! We don't support PSK, so we will need to handle that. The actual encrypt/decrypt is easy to update we already support CBC and GCM so adding CCM shouldn't be bad!
Hopefully we don't have the break the API for existing users, but if it is needed I understand.
If you want to join our Slack I can help and will try to explain a little bit here also.
The communication is split up into 'flights', I designed it this way so we can handle retransmissions/out of order packets.
each side accepts incoming messages, and when we get everything we want we start processing code
each side just keeps re-sending until it gets to the next flight [code (https://github.com/pions/dtls/blob/master/server_handlers.go#L218)

I started digging into this, the good news is OpenSSL does support PSK! The bad news is no CCM :/
I am working on this now, I will add PSK-AES256-CBC-SHA and then we can work on CCM after that lands.
thanks!
./openssl s_client -dtls -connect <IP_ADDRESS>:5000 -psk abc123
./openssl s_server -dtls1_2 -accept 5000 -nocert -psk abc123

Funky. I couldn't get it to show up in openssl ciphers on my side:
~ $ openssl ciphers | tr ':' "\n" | sort | grep -i psk
DHE-PSK-AES128-CBC-SHA
DHE-PSK-AES128-CBC-SHA256
DHE-PSK-AES128-GCM-SHA256
DHE-PSK-AES256-CBC-SHA
DHE-PSK-AES256-CBC-SHA384
DHE-PSK-AES256-GCM-SHA384
DHE-PSK-CHACHA20-POLY1305
ECDHE-PSK-AES128-CBC-SHA
ECDHE-PSK-AES128-CBC-SHA256
ECDHE-PSK-AES256-CBC-SHA
ECDHE-PSK-AES256-CBC-SHA384
ECDHE-PSK-CHACHA20-POLY1305
PSK-AES128-CBC-SHA
PSK-AES128-CBC-SHA256
PSK-AES128-GCM-SHA256
PSK-AES256-CBC-SHA
PSK-AES256-CBC-SHA384
PSK-AES256-GCM-SHA384
PSK-CHACHA20-POLY1305
RSA-PSK-AES128-CBC-SHA
RSA-PSK-AES128-CBC-SHA256
RSA-PSK-AES128-GCM-SHA256
RSA-PSK-AES256-CBC-SHA
RSA-PSK-AES256-CBC-SHA384
RSA-PSK-AES256-GCM-SHA384
RSA-PSK-CHACHA20-POLY1305

Glad you found it though!

@daenney what version of OpenSSL do you have, I was able to get this with OpenSSL 1.1.1b  26 Feb 2019
I also had to do openssl ciphers -V ALL@SECLEVEL=0; to see it

Same: OpenSSL 1.1.1b  26 Feb 2019. I had no idea about -V ALL@SECLEVEL=0 .

Nice, I think that adds a whole new level of usefulness :)
Thank you so much for taking time for the reviews guys @daenney @igolaizola! If you guys have any opinions on things the library needs/where we should go next I am all ears :) I am back on Pion WebRTC for a while, but always ready to jump in


Feature Request: Prevent backspace from going back in the browser's navigation

In some browsers, moving the cursor out of the editor and pressing backspace will go back in the browser's navigation instead of deleting text in the editor.

I'm tempted to investigate this further. Which browsers are you referring to?


Fix disappearing ammo

May have to do with min mode being enabled.

May not actually have to do with minmode. One person reporting it stated it happens on W10, can't imagine it's that though?

It's with Mac aswell, I could try testing with different settings they changed

Okay, I can pretty much confirm it's due to having minmode on. I think I may move away from having minmode solely affect the scoreboard in order to open up other possibilities in the future. Not sure though.

I nuked all _minmode entries in _tf2hud (Find in Files > replace all _minmode with something else) to fix this, in case anyone would like to fix it themselves now.  I'm likely going to wait until I have a large portion of the hud updated before actually committing any changes.


Introduce date literals

The lexing could probably be improved a little, but it works fine.

fair enough :+1:

might be worth adding a CLI test to serve as example

cool :tropical_drink:


Wrong vote result with general approval/rejection

With voting method Yes per candidate and General approval enabled you get wrong vote results.
Here are two test cases demonstrating this:
Wahlgang (1)
Administrator voted explicitly for f and g. I expected General approval to be 0. However, it mistakenly says (200 %) 2.
Wahlgang (2)
Administrator voted for General Approval. Amount of votes was 2 but only one vote was counted - for h.
Solution proposal: If amount of votes < number of candidates, general approval does not make sense and should be disabled. If amount of votes >= number of candidates, all candidates MUST be elected if general approval is checked.
Similar issue with No per candidate and General rejection.
Wahlgang (3)
Administrator voted explicitly against f. General approval says (100 %) 1 but should be 0.
Wahlgang (4)
3 votes, 3 candidates. Administrator voted General rejection. Only candidate f was rejected. I expect ALL 3 candidates to be rejected.

This is due to the way the general vote is saved. This is fixed in OS4, but I see that wrong results should not be shown. One solution would be to set global_yes to false on votes method Y and the same for n with global_no, right?

This is due to the way the general vote is saved. This is fixed in OS4, but I see that wrong results should not be shown. One solution would be to set global_yes to false on votes method Y and the same for n with global_no, right?

I like to have global_yes for votes method Y but as I pointed it should only be enabled if number of votes >= number of candidates. And for named voting the single votes should indicate that a global_yes was cast. The same is true for global_no and votes method N.

I like to have global_yes for votes method Y but as I pointed it should only be enabled if number of votes >= number of candidates. And for named voting the single votes should indicate that a global_yes was cast. The same is true for global_no and votes method N.

Sorry, I can't really follow your problem. Since it seems not to be a technical problem but rather a logic one, maybe talk directly to emanuel.

Sorry, I can't really follow your problem. Since it seems not to be a technical problem but rather a logic one, maybe talk directly to emanuel.

Thanks, Finn. I've already started to discuss this with Emanuel.

Thanks, Finn. I've already started to discuss this with Emanuel.

outdated


Place API weekday hours support?

I couldn't find it on the docs, but is there support for retrieving business hours in this gem?
Thanks!

Geocoder supports the Google Places Details API, so it will give you access to whatever information is provided there.


docs(i18n): New Crowdin updates

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Support different filename for root .terragrunt

I have terragrunt config files per account-region. The root config file should also have the matching region info and a different bucket name. Could you please support a different root config file instead of operating under the assumption that the root file will be named .terragrunt?
Thanks for the app, it's been super useful!

@evanstachowiak If you have different files, I guess you would still like to point to them case by case.
wouldn't https://github.com/gruntwork-io/terragrunt#cli-options

--terragrunt-config: A custom path to the .terragrunt file. May also be specified via the TERRAGRUNT_CONFIG environment variable. The default path is .terragrunt in the current directory.

solve your case?

@josesa, unfortunately not. That switch sets the config only for the current working directory. When looking for the parent config, terragrunt still uses the default filename of .terragrunt and this doesn't seem to be modifiable. This is problematic because the top-level config file contains the region, bucket name, etc whereas each .terragrunt-custom-name file only contain an include.

Could you provide a bit more info so we can understand the use case? What does your folder structure look like? Why does a root .terragrunt file not work for it? What solution are you proposing? Thanks!

hey @brikis98, sorry for the delay, I've been on vacation.
I have a similar structure to what you outlined in your blogpost: https://blog.gruntwork.io/how-to-create-reusable-infrastructure-with-terraform-modules-25526d65f73d#.tgyv0iyus
However, I don't have a separate folder for each environment, I just have a top-level services directory and then handle the configuration based on the environment.
The issue specifically is: I'm using a different bucket per account & region to store the terraform state. In this case, sharing a top-level .terragrunt file does not work for me.
My proposed solution: Respect the --terragrunt-config flag for the parent .terragrunt as well. Then it can support different s3 buckets using includes.

Would using environment variables work for you? That is, instead of setting the include path to a hard-coded value or find_in_parent_folder(), you set it to an environment variable Terragrunt can look up. You could then set that variable accordingly to whatever value you need.
If so, then once https://github.com/gruntwork-io/terragrunt/pull/77 is merged, you should be good to go.

Yes, this could work but it still seems inconsistent to specify a terrgrunt_config file but this is only for the current directory and for some reason the parent config ignores this flag. Maybe this is just me.
Another drawback of using get_env is that there is no way to check this into source control. Using this I will have to somehow coordinate the env variables across developer machines.

Yes, this could work but it still seems inconsistent to specify a terrgrunt_config file but this is only for the current directory and for some reason the parent config ignores this flag.

The --terragrunt-config flag tells Terragrunt where to find your .terragrunt file. The command-line flags have no relation to what that .terragrunt file may choose to do. For example, if that file happens to use the include keyword, it's up to the file, and not the command-line flags, to figure out the right way to specify the path to the included file. You can do that by hard-coding it, by having it found automatically using find_in_parent_folder(), or, hopefully soon, via env vars.

Another drawback of using get_env is that there is no way to check this into source control. Using this I will have to somehow coordinate the env variables across developer machines.

If your developers had to pass in a special --terragrunt-config flag anyway, then isn't that something you'd have to do across all developer machines too? I guess you could create a wrapper script to do that automatically, but then a wrapper script could just as easily set an env var.

Ah, perhaps then modifying find_in_parent_folder() to find the matching filename instead of .terragrunt? Otherwise, you are correct I can just hardcode the value, that's a good idea.
As for the --terragrunt-config, yes, there is some knowledge needed there but then the appropriate .terragrunt* files will already be checked into source control, so it is self-documented in a way.
Thanks for all the feedback!

Ah, perhaps then modifying find_in_parent_folder() to find the matching filename instead of .terragrunt?

Yea, this would make sense. The API would be simple too: find_in_parent_folder(filename). If you don't specify a filename, the default is .terragrunt.

@evanstachowiak I was just trolling through open issues and came across this. I'm wondering how you might have changed your workflow/directory structure now that Terragrunt has deprecated support for the .terragrunt configuration file and instead expects configuration to be in the terraform.tfvars file? See here for details.
Would you still lobby for updating the find_in_parent_folder() function to find_in_parent_folder_(filename)?

@conorgil now that terraform has native remote locking support I've just migrated away from terragrunt. I already had a wrapper script to handle multiple environments and profiles so now I don't need terragrunt for the locking.

@evanstachowiak  Thanks for the response and glad you found a workflow that works for you. Do you think this issue can be closed?

Seems like this issue is no longer relevant/needed. Closing.


Look at TODOs in W01

I added some todos to W01.lhs in commit 49aff10

Acted on most visible TODOs. The rest "demoted" to comments to handle in a later pass.
959df2f7a7539951707fb31b5076d07862485757


Binary value of [32]byte hash is calculated incorrectly

Hi guys, there is an issue: I have two 32-byte hashes. For one of them bit representation is calculated correctly and for second one is incorrectly. Here is my circuit:
type BinaryRepresentationCheckCircuit struct {
	ExpectedResult [256]frontend.Variable
	Hash           frontend.Variable
}

func (circuit BinaryRepresentationCheckCircuit) Define(api frontend.API) error {
	bits := api.ToBinary(circuit.Hash, 256)
	res := [256]frontend.Variable{}
        for i, j := len(bits)-1, 0; i >= 0; i, j = i-1, j+1 {
		res[i] = bits[j]
	}
	for i := range res {
		api.AssertIsEqual(res[i], circuit.ExpectedResult[i])
	}
        return nil
}

This test passes successfully:
func TestBinaryRepresentationCircuitSuccess(t *testing.T) {
	var circuit, witness BinaryRepresentationCheckCircuit

        // 32 bytes hash
	data := common.FromHex("af6b80f7c6b8d2e5ce1cfa3a58c7c8530a7f75bc4f73569a8dcffbde3efc075")
	witness.Hash = data
	setBitsToWitness(data, &witness)
	assert := NewAssert(t)
	assert.SolvingSucceeded(
		&circuit,
		&witness,
		WithBackends(backend.GROTH16),
		WithCurves(ecc.BN254),
		WithCompileOpts(frontend.IgnoreUnconstrainedInputs()),
	)
}

This test case fails even though it's exactly the same and just hash value is different (still 32 byte variable though):
func TestBinaryRepresentationCircuitFailed(t *testing.T) {
	var circuit, witness BinaryRepresentationCheckCircuit

        // 32 bytes hash
	data := common.FromHex("af6b80f7c6b8d2e5ce1cfa3a58c7c8530a7f75bc4f73569a8dcffbde3efc0753")
	witness.Hash = data
	setBitsToWitness(data, &witness)
	assert := NewAssert(t)
	assert.SolvingSucceeded(
		&circuit,
		&witness,
		WithBackends(backend.GROTH16),
		WithCurves(ecc.BN254),
		WithCompileOpts(frontend.IgnoreUnconstrainedInputs()),
	)
}

As you may see failed hash is exactly the same but contains one more number (3 at the end). But it's still 32 bytes hash so it's not clear for me what's the reason.
I played with different 32 byte hash values and the result is always the same, some test cases pass, some fails.
You can check the tests and code here - https://github.com/ruslangm/gnark/pull/2 .
Thanks.

Ok, my fault, 2nd hash value is bigger than p value and it causes field overflow - that's why it fails. Will split my hash into 2 variables and it should be good.


Breaking backwards compatible change log_printfv

This change breaks trying to link to older versions of arduino-esp32. This method was added in this change.
However, esp-insights is not versioned. So now it is not possible to build the release tagged 2.0.5 of arduino-esp32 using esp32-arduino-lib-builder. This fails on linking log_printfv.
Is there any way to know the versions (I.e. head commit hash) of this library when the v2.0.5 version of arduino-esp32 was generated?
I'm not really sure the correct way to map this across all projects. I am trying to make a 1-line change to arduino-esp32 at the version I am currently using without changing versions and risking further breaks.

If you don't want to use esp-insights in arduino, you can remove it from the cmakelists.txt file here. Can you try and let me know if this solves the linking issue?


Update README.md

Added papers to all sessions (1, 2, 3, 4, 5, 6, 7) and fixed major/minor errors.
See my fork for PRs to your repo.

Hi @DmitryRyumin, it's amazing!!
Thank you for your great effort!!


Add FeaturesDroppedLoader to extract header value

Description
Shortcut

Story: https://app.shortcut.com/cartoteam/story/268367/header-reading-loader-mode-in-deck-gl
Update loaders.gl to include header reading functionality
Use custom loader to parse out header to avoid having to invoke custom fetch function

Type of change

Refactor

Acceptance
I have struggled to link the C4R codebase to another project, and had to test by copying files across into a modified project. For full acceptance it would be good for someone with more experience working with C4R to try linking in this modified code into an existing app to verify the feature dropping behavior works as expected

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 4 (0.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.2%) to 71.802%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/react-api/src/hooks/useTileFeatures.js
0
1
0.0%

packages/react-api/src/hooks/FeaturesDroppedLoader.js
0
3
0.0%

Files with Coverage Reduction
New Missed Lines
%

packages/react-api/src/hooks/useTileFeatures.js
1
27.94%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
1878

Relevant Lines:
2428

💛  - Coveralls


Argo UI does not show complete logs

Summary
Getting similar problems like in #3065
The logs are wholly seen in Minio saved logs and in the pod logs with kubectl, but not in the Argo UI. Neither of them show any errors, only the Argo UI output is truncated when I use a workflow that incorporates Tensorflow output.
Could be the particular formatting of the training that stops the output to the UI? The logs are properly shown in the Argo UI exactly to the point before Tensorflow training output starts, (example training line from minio logs below):
1501/1501 [==============================] - 57s 33ms/step - loss: 0.8667 - accuracy: 0.7131 - precision: 0.8383 - recall: 0.6788 - f1_score: 0.7311 - val_loss: 0.1472 - val_accuracy: 0.9567 - val_precision: 1.0000 - val_recall: 0.9786 - val_f1_score: 0.9890
Diagnostics
What Kubernetes provider are you using?
Kubeadm 1.19.7
What version of Argo Workflows are you running?
v2.12.8

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

Might very well be that our current loggin infrastructure doesn't deal well with \r, which is what is used to replace the console output line to get that nice "progress effect" that tensorflow displays in lines like those.
I'll look into it

In fact, thinking this a bit more I think this is likely the case

Will have to see if this is something we can deal with or if K8s' Pod log streaming interface is the problem

Bug missing reproduction step. Can you please attach a workflow that reproduces the problem that we can run?

My suspicions were right: the issue is the use of \r and K8s itself does not handle it. Since we only display logs that are passed to use by K8s, I'm not sure what we can do about this.
See below for reproduction steps:
Workflow
# script templates provide a way to run arbitary snippets of code
# in any language, to produce a output "result" via the standard out
# of the template. Results can then be referenced using the variable,
# {{steps.<stepname>.outputs.result}}, and used as parameter to other
# templates, and in 'when', and 'withParam' clauses.
# This example demonstrates the use of a python script to
# generate a random number which is printed in the next step.
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: scripts-python-
spec:
  entrypoint: python-script-example
  templates:
  - name: python-script-example
    steps:
    - - name: generate
        template: gen-random-int

  - name: gen-random-int
    script:
      image: python:alpine3.6
      command: [python]
      source: |
        import time
        print("Starting job")
        for i in range(10):
          print("Progress", i, "/ 10", end="\r")
          time.sleep(1)
        print("Ended job")

K8s doesn't display any logs while pod is still running:

Final K8s Logs:

Final UI Logs:

Hey @Atharex I actually got something for you to try. When running python, try the -u flag: python -u. Example from another community member: https://github.com/onepanelio/templates/blob/d23d2e1a2e49dce37a4e9a428e3205b930a190c2/workflows/tensorflow-mnist-training/template.yaml#L29
Let me know if this works

Thanks for the hint, seems to work now! Turned out the problem was also a bit trickier. The node I was running the workload on was in a different subnet than other nodes. After it was transferred to the same subnet, the logs started showing up normally.

Closing as invalid. Please comment if incorrect.

As a "nice to have" it might be good if argo logs gave a warning that this behavior existed. I find it very non-intuitive. When I ask for logs, I expect to get all of stdout, so even though it is a K8s problem, if there is a workaround, or at least a warning that points to workarounds, that might make the experience of using argo nicer.


feat: created JDBCToJDBC template

Please update README and java/README files

Please update README and java/README files

Updated

@poojabasker20 I can see some changes are not related to this issue. Can you please revert those back?


Ability to rerun tests, markdown results

Changes
I found that I wanted to re-run failed tests so I could debug, so I added that.

Tests are clickable when not  in the running state

Regarding the command, it is not required to add anything to package.json here since we aren't making it accessible from any UI elements (including the command palette)

Test hover content is now in markdown to make it just that little easier to read

Checklist

[x] have tested my change
[ ] updated relevant documentation
[ ] Remove any/all console.logs I added
[ ] eslint is not complaining
[ ] have added myself to the contributors' list in CONTRIBUTING.md
[x] for feature PRs: PR only includes one feature enhancement.

I have been testing this all day today and I am happy with it. I am going to merge so I can use these features to work on my other tests.


Remove gcov usage until it can be used.

Until tests can be run successfully (everest-framework depends on liblog), removing dependencies can help simplify our efforts at automating builds and tests.

I'm a bit confused about this change. There is currently no CI here, no GitHub actions are running. It sounds like the tests pass when run locally in a dev environment.
* Why are we not putting the build + test into CI?

* Which dependency are we trying to remove here? gcov? Computing code coverage is generally a Good Thing to do.

I agree, but as it doesn't currently do so, removing these lines makes it easier to establish the automated builds in the first place. We can put the gcov lines back when we've got something that can use gcov. Otherwise, we have a failure when trying to do similar work in everest-framework, which depends on this project.

@shankari I am working on adding CI/CD for this repo. The card is APIT-107.


Filecoin.ChainPutObj

Filecoin.ChainPutObj is not used by any software I can find. Furthermore, looking at the Lotus source code, I doubt this RPC method can ever be called.


Log-out Button

User story

As a {HR-employee}
I want / need {a button to leave DIdentity}
So that {log-out from DIdentity}

Acceptance criteria

Create log-out button with an Icon (closing door icon)
Embed the button right corner in line with menu
Check mock-up design for inspiration

Definition of done (DoD)

code peer-reviewed (by at least one team member)
acceptance criteria met
integrated into a clean build
documentation updated
code is well-formatted (according to our norms) and structure
Code supports all functions/cases and all responses defined in the API
no severe bugs during main functionalities (program should not stop executing due to these bugs, e.g. runtime exception in Docker)
Bill of Materials updated if necessary
70% of the tests pass
important functions shall be tested
code has been deployed via Docker to the server

DoD general criteria

Feature has been fully implemented
Feature has been merged into the mainline
All acceptance criteria were met
Product owner approved features
All tests are passing
Developers agreed to release

Done with #246


No web interface for data source configuration

Your README.md for hassio-addon-teslamate shows a webUI for the Postgres HA addon (For data source configuration), but the installed addon does not have a WebUI button on the info tab.

I have tried [homeassistant_URL]/29b65938_postgres which returns a 404
I also tried [homeassistant_URL]/hassio/ingress/29b65938_postgres but it seems this plugin does not use ingress.
I also tried using a postgres client (Heidi) to connect to [homeassistant_URL]:5432 with the configured credentials, but the connection is refused.

Can you please advise the method you use to access the Postgres webUI as shown in the hassio-addon-teslamate README.mb

There is no web UI for Postgres. That screen shot is from Grafana.


Not getting all historical message of conversation

Description
hey there thanks advance for you time and help. i calling fetchHistoryMessages() but not getting all historical message specially the new one's also i have getLatestMessage() its working fine and showing me the latest message but when i call fetchHistoryMessages() i am not getting all the history message following i my code
let  paginationLimit = 45
let lastEvaluatedKey = ''
let convType = 0;
// getting messages from server
const conversation = await chatManager.fetchHistoryMessages(
convId,
convType,
paginationLimit,
lastEvaluatedKey,
);
console.log('RESPONSE[RETRIVE_CONVERSATION_MESSAGE]', conversation);
Note: just for testing i again switch back to version 1.1.1 its working fine i am getting all message currenlty i am using 1.1.2beta-4
Version
1.1.2beta-4
Output of npx react-native info
System:
OS: macOS 13.0.1
CPU: (8) arm64 Apple M1
Memory: 99.14 MB / 8.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 18.9.0 - /opt/homebrew/bin/node
Yarn: 1.22.19 - /opt/homebrew/bin/yarn
npm: 8.19.1 - /opt/homebrew/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.11.3 - /opt/homebrew/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.4, iOS 16.4, macOS 13.3, tvOS 16.4, watchOS 9.4
Android SDK: Not Found
IDEs:
Android Studio: 2021.2 AI-212.5712.43.2112.8815526
Xcode: 14.3/14E222b - /usr/bin/xcodebuild
Languages:
Java: <IP_ADDRESS> - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.1.0 => 18.1.0
react-native: 0.70.6 => 0.70.6
react-native-macos: Not Found
npmGlobalPackages:
Steps to reproduce
npx react-native run-android
yarn instal react-native-agora-chat
  // getting messages from server
  const conversation = await chatManager.fetchHistoryMessages(
    convId,
    convType,
    paginationLimit,
    lastEvaluatedKey,
  );

Snack, code example, screenshot, or link to a repository
https://github.com/
Platform

[X] All
[ ] Android
[ ] iOS

SimulatorOrDevice

[X] All
[ ] Simulator
[ ] Device

iOSVersion

[ ] All
[ ] iOS Earlier than 8
[ ] iOS 8
[ ] iOS 9
[ ] iOS 10
[ ] iOS 11
[ ] iOS 12
[ ] iOS 13
[ ] iOS 14
[X] iOS 15
[X] iOS 16 or above
[ ] Android Earlier than API Level 23
[ ] Android API Level 23
[ ] Android API Level 24
[ ] Android API Level 25
[ ] Android API Level 26
[ ] Android API Level 27
[ ] Android API Level 28
[ ] Android API Level 29
[ ] Android API Level 30
[ ] Android API Level 31
[X] Android API Level 32
[X] Android API Level 33 or above

hey there i am still facing this issue in 1.1.2-beta-5

The fetchHistoryMessages interface mainly added the direction parameter between version 1.1.1 and version 1.1.2. If you switch versions, ios recommends cleaning the cache. Problems can be checked by comparison. For example: start xcode or android studio, check the input parameters in debug mode, and compare the results.


custom styled is not function

I followed the guidelines in order to make my own style, However, I am getting the following error: TypeError: (0 , _reactTabtab2.default) is not a function
import {styled} from 'react-tabtab';
let {TabListStyle, ActionButtonStyle, TabStyle, PanelStyle} = styled;
TabListStyle = styled(TabListStyle)  // write css;
In order to fix the issue. I have to write like the following:
import styled from 'styled-components';
import {styled as themeStyled} from 'react-tabtab';
let {TabListStyle, ActionButtonStyle, TabStyle, PanelStyle} = themeStyled;
TabListStyle = styled(TabListStyle)  // write css;
"react": "^16.8.6", "react-dom": "^16.8.6", "styled-components": "^4.0.0" "react-tabtab": "^2.0.0",

Here is a working example that might help you:
https://kjkjjang.wordpress.com/2019/07/08/reacttypescript-모듈-추가-react-tabtab-2편/

Here is a working example that might help you:
https://kjkjjang.wordpress.com/2019/07/08/reacttypescript-모듈-추가-react-tabtab-2편/

Thanks, This helps me a lot.


Events Category

Event driven projects and tools, ie CDEvents

This would be the current "Protocols and Messaging" category. Maybe it should be renamed? Or merged with the proposed "Measuring and Reporting" category as proposed in #195

Hi @sbtaylor15, further to Emil's comment, please see #195 and comment further there or on #203, which is the prototype for the section.
Closing in favour of #195.


Develop

Coverage increased (+0.03%) to 90.265% when pulling 0f0c07185d657a7839b6711e608b9fcb9e909c6e on develop into 4052e3477a314829f34c91662737a0d5bf33e26a on master.

Thanks!


Hidden custom view when neither of title, message or text field is used

From version 5.0.0 it is not possible to see the custom view if I don't set also the title or message.
I try to use it like this
let alertView = ...
let alert = AlertController(view: alertView, style: .alert)
let cancelAction = ...
let addAction = ...
alert.addAction(cancelAction)
alert.addAction(addAction)
present(alert, animated: true, completion: nil)

But I can see only the buttons.
I located the problem to be here (and next line also):
https://github.com/KyoheiG3/SimpleAlert/blob/057e8330941c8cac798489ce3fbebe8441543476/SimpleAlert/AlertContentView.swift#L29

It's happened to me, too.


Allow config-cache customisation via custom bootloader/app object

WAS: ConfigManifest doesn't use custom cache backend
Due to #2933 on a hosting platform that uses NFS we're using the following in our _config.php
SS_Cache::add_backend("default", "File", array(
    "cache_dir" => TEMP_FOLDER . DIRECTORY_SEPARATOR . 'cache',
    "file_locking" => false,
));

I've noticed that this isn't picked up until after the ConfigManifest is built which uses the default cache backend (File, without file_locking disabled).

Hm... Yes, as the config manifest likely runs before the _config is found and included... probably needs the bootloader to be refactored before this is likely to change :/
I take it putting this in an _ss_environment.php file will be too soon?

I haven't really looked into this issue but would defining SS_MANIFESTCACHE solve your issue? It allows for different caching mechanisms as can be seen here: http://api.silverstripe.org/3.1/source-class-ManifestCache_File_PHP.html#9

@dhensby putting it in _ss_environment.php is the temporary solution I've used, but it's a bit hacky, we're using the following
define('TEMP_FOLDER', __DIR__ . DIRECTORY_SEPARATOR . "silverstripe-cache");
require_once 'framework/cache/Cache.php';
$cachedir = TEMP_FOLDER . DIRECTORY_SEPARATOR . 'cache';
if (!is_dir($cachedir)) {
	mkdir($cachedir);
}
SS_Cache::add_backend("default", "File", array(
	"cache_dir" => $cachedir,
	"file_locking" => false,
));

@micmania1 Unfortunately not, SS_MANIFESTCACHE is only used by ClassManifest, ConfigStaticManifest and TemplateManifest.

Yes, that's the kind of horrible thing I'd have expected you needed to use.
Unfortunately I don't think this issue is going to be fixed until there is a more robust boot loader for SS and that's going to be quite some way off

Related: https://github.com/silverstripe/silverstripe-framework/issues/6647

Amended title to reflect the fact that this is an enhancement that depends on another.

Related: https://github.com/silverstripe/silverstripe-framework/issues/6681

The app object is done now I think we can close this.


BUG: stats: fix typo in stable rvs per gh-12870

It's looking like gh-9523 might not make it in the next release. In case not, I thought we could at least fix gh-12870.
@bsdz does this address that issue?
Closes gh-12870

It fixes the typo but not the related parameterization bug #12871. Also bear in mind #9523 also introduces tests around both of these issues. IMHO I think it would be better to focus reviewing #9523; if fixing these long standing issues is important we should perhaps push back on next scipy release until #9523 is reviewed / adjusted and merged.

Yes, I know that gh-9523 is more important, overall. Thank you for your patience on that. I only started working on scipy.stats in the past few months, so I did not enter that conversation, but I will consider it once I get some other things out of the way.
I don't think this will take focus away from gh-9523, though; this is a one-line change that can easily be merged before 1.6.0.

I think you might have missed my first point. Fixing the typo doesn't fix #12871. So AFAICT the numbers generated for when alpha == 1 will still be wrong.

I see. OK, closing.


Bug/descw 1297 fix billing add new

I'm seeing an error on my machine when I start the frontend on this branch with yarn start:


ESP-IDF 5 release date? (IDFGH-8108)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
Hello-
I was wondering if there's a release date planned for esp-idf v5?
Thanks!
Noah

Thanks for reporting, IDF v5.0-beta1 has released a while ago at https://github.com/espressif/esp-idf/releases/tag/v5.0-beta1, will keep you updated on further release. Thanks.


Resolvers

@ethicalhackingplayground would be good if it have a flag to resolve domains using list of dns resolvers.

Hi there,
Thanks for your feature request, yes, I've currently got this listed on the things to implement :) please stay tuned for more updates.
Regards,
Blake

I've just added in the option to specify a list of dns resolvers, would you be able to test it out please?
Thanks.

@ethicalhackingplayground thank you for this fast, I have tested and found that resolving with dnsresolver is working fine.
One more thing I would like to add that there is a diff from previous one where dnsresolver is also showing status of the reosolved domains but in this version it is not having the said feature.
It would be good if we have another flag for the user to use --status to show status along with the resolved one.
About vhost feature which is a bonus in this, right now if user want to detect vhost by using --vhost flag it won't show any information like if the host is vhost or not it only show the domain name and also not showing the ip associated with that.
I would like that if user tried to resolve the domains from the list and also using --vhost then dnsresolver output would be like:
cat domains.txt | ./dnsresolver --vhost --resolvers trustedns.txt -r 25 -c 25
example.com <IP_ADDRESS>
test.com [vhost] <IP_ADDRESS>
new.com <IP_ADDRESS>
old.com [vhost] <IP_ADDRESS>

Thanks for your reply,
I'm a little confused about what you mean about the status of the resolved hosts, when you say the status of the resolved hosts, do you mean it was showing the hosts in this format  https:// or http:// if port 80,443 is opened because if so, this feature is now optional with the --ports flag. Concerning the virtual host enumeration feature, this is experimental at the moment but I have had some great feedback on this and I will be spending some time fixing up the virtual host enumeration feature, so far if you specify the --vhost flag it will find which subdomains belong to each IP address and there should be a folder called vhosts which contains all the IPs as subfolders and in each folder there is a vhosts.txt file which contains all the subdomains that belong to the IP address.
Also, just keep in mind some IPs may rotate so you may find that IP addresses are not always the same, you can validate if it's a virtual host by checking the subdomain on this site https://mxtoolbox.com/dnslookup.aspx to see if it's the same IP but also note that websites can point to one or more IP addresses.
I hope this helps, I will make sure to improve the virtual host discovery.
Regards,
Blake

Thank you @ethicalhackingplayground for this detailed information and hope to get more from dnsresolver

Hi @ethicalhackingplayground Thank you for the ping, I tested the vhost functionality, the one thing which I would like to take control on is default waf detection, can we give this control to the user so they can use the flag -waf along with vhost and or domain resolving functionality.
like only if user need to test if the host is behind a waf while resolving domain or while testing vhost functionality and by default it wont check the waf.

Thanks for your response @jaikishantulswani,
These are some excellent suggestions, I will make these changes as soon as possible.
Regards,
Blake.


build: release candidate

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


dataset label question

Within the dataset you are mentioning, the mask pixel values are more than 1, even though the number of classes is one.
I though that there would be 0 and 1 values for the pixel of mask image.
Is it OK?
Could you explain??

Hi, is the mask pixel value for the positive case, to which you are referring, 255? If so, this is just for visualisation of the mask in uint8 format, so that the negative class appears black and the positive class appears white. If the value was 1, it would be almost black and difficult to see. We map these values to 1 using the ToTensor() transform, which scales the values from [0, 255] to [0, 1] if the data is in uint8 format.

I meant, for the mask image, the pixel value should be one or zero, but there are other values too. Since the dataset has binary classes (background and polyp), I think there should be two pixel value 1 or 0 in mask image.
However, when I print the values of mask image from the dataloader in the code, there are many other values.
Could you explain?

Apologies, I had forgotten about this issue. Some pixels in the masks in the raw format do indeed have values other than 0 and 255, and after being mapped to [0, 1] through the ToTensor() transform are also not 0 or 1. Unfortunately, I do not know exactly why this is, you would need to ask the authors of the dataset. We concluded that since the values are always very close to 0 or 1, that this would cause little issue during training. However, in both the evaluation performed during validation (using our implementation of the Dice score given in Metrics/performance_metrics.py) and in the final evaluation of the model (eval.py), we binarized both the prediction and the ground truth using x > 0.5.

I am new to annotation. For tags, what is the appropriate format for me to use labelme, such as TXT or JSON? Could you please take some time to answer, thank you.


Add metricFilters to filter only the metrics we want to send

Add metricFilters to filter only the metrics we want to send
If no filters are specified, nothing is filtered to keep
current behaviour

There is more and more cloud services that allow people to easily send metrics, but by default, even while filtering collectors with the -f option, there is still way to many metrics sent by default
I think it would be great to be able to send only the few metrics we really want
What do you think?

@captncraig In which release, we can get this change? Do we have daily binary build for scollector to try  this change?

It will be in the 0.5.0 release which will probably not be out until a few weeks from now, due to the redis conversions I am working on.
I can make a prerelease build for this and some other things.

It would be good if u can make a prerelease, so we can start testing.

https://github.com/bosun-monitor/bosun/releases/tag/0.5.0-alpha
On Tue, Nov 3, 2015 at 3:07 PM, Digvijay Richhariya <
<EMAIL_ADDRESS>wrote:

It would be good if u can make a prerelease, so we can start testing.
—
Reply to this email directly or view it on GitHub
https://github.com/bosun-monitor/bosun/pull/1367#issuecomment-153504608.

Thanks @captncraig

Just adding an example here for reference:
you can use ^ and $ to match the start and end of a string, ( a | b ) to say a or b, and .* to match anything.
MetricFilters = ['^(win\.cpu|win\.system\..*)$', 'free'] would match win.cpu, win.system.* or any metric that includes the word free


0.10.0 - createConvexMeshShape example provided in manual doesn't work

Given that height map mesh generation is broke for me in 0.10.0 (See: Issue #378) I'm trying to migrate my height map based objects to use convex mesh shapes instead.
I'm trying to create a convex mesh shape as the user manual describes. https://www.reactphysics3d.com/usermanual.html#x1-4300010.1 under "10.1.4 Convex Mesh Shape".
Issue 1
Copy pasting the exact code provided in the manual, the one that creates a simple box with 24 vertices, ends up causing inconsistent behavior. 99% of the time when I run it, the shape fails to build and I get this error message:
Message (error): Error during initial hull creation in QuickHull: vertices too close to each other
However, 1% of the time it seems to not throw that error and let me continue, although the mesh isn't active in the world like I'd expect it to be.  So there's maybe some sort of floating point related edge case that rarely causes it to behave differently?
Issue 2
In the example, when the vertices are specified, the third and fourth rows of vertices are identical. The example is missing a negative sign before the x value on the fourth line.
However, both with and without making that fix, I still get the "vertices too close to each other" error from Issue 1.
Issue 3
In the documentation on the website for Convex Mesh Shape, the documentation seems to have the same description and code example twice, one on top of the other, nearly identical except for a slightly different comment in the second version. I'm guessing the one example was supposed to show how to create a convex mesh from a PolygonVertexArray, and one version how to make one from a VertexArray? Both examples show the same VertexArray example code.
As for documentation, I also find it confusing what values we're supposed to provide for "number of vertices" parameters. In the box example, there's 8 vertices, where each vertex has an x, y, and z value, but we're supposed to pass in "24" for "nbVertices Number of vertices in the array"? There's 24 floats that make up the vertices, but there's only 8 actual vertices, so it seems wrong that we're supposed to pass in 24 for the number of vertices. (Unless the example in the documentation is wrong?)
Issue 4
I was also initially trying to create a convex mesh shape from a PolygonVertexArray but I could not find a single working combination of arguments that didn't cause a crash due to a bad map access within rp3d. I guess whenever Issue 3 is fixed and there's sample code for the PolygonVertexArray path and I can test whether it works or not.

Thanks a lot for your feedback.
As you said, there are an multiple issues in the documentation. First that's right there should not be two identical vertices coordinates in the array. Secondly, because it's a cube, the number of vertices is 8 (not 24). You need to use 8 in the parameters when you create the VertexArray.
I will need to fix the documentation.
In the meantime, you can take a look at how a ConvexMeshShape is created in the testbed application.

Using a VertexArray (with QuickHull algorithm) take a look here
Using a PolygonVertexArrray take a look here

Note that if you want to use a ConvexMeshShape, your shape have to be convex. If you are moving from a HeightFieldShape to a ConvexMeshShape, maybe your HeightFieldShape was not convex. In this case, you should use a ConcaveMeshShape instead.
Again, thanks a lot for reporting this issue. I will take a look at the HeightFieldShape issue you have reported as soon as I have some time.

I'm trying to use Irrlicht to generate the bounding box since Irrlicht makes rectangles that more closely resemble the mesh. Since this can't do rectanlges, I'm trying to make it work with ConvexMeshShape. I made it so that my program gets Irrlicht to return the bounding box and im trying to make it convert that to a ConvexMeshShape.
Changing the vertex count from 8 to 24 allowed it to work a lot better but some rectangles cause the game to crash when collided. There is this one model whos collision box always causes a crash when an object bumps into it but my other models don't. The error is: "reactphysics3d/collision/shapes/ConvexMeshShape.cpp:106: virtual reactphysics3d::Vector3 reactphysics3d::ConvexMeshShape::getLocalSupportPointWithoutMargin(const reactphysics3d::Vector3&) const: Assertion `maxDotProduct >= decimal(0.0)' failed."
I had to go into the source code for this one. In ConvexMeshShape.cpp in the getLocalSupportPointWithoutMargin() function, there is a line of code "assert(maxDotProduct >= decimal(0.0));". I commented that out. The game no longer crashes when object collide into that one object. I don't know what the implications of this hack are but if I'm doing this wrong and that's the reason it's crashing, I would love to find out what the correct way is.
If I can use this to avoid implementing my own special colliders by building off of the plane class, I will.I just want to know if its deprecated and we're not supposed to be using it or if this is user error.

I'm trying to use Irrlicht to generate the bounding box since Irrlicht makes rectangles that more closely resemble the mesh. Since this can't do rectanlges, I'm trying to make it work with ConvexMeshShape. I made it so that my program gets Irrlicht to return the bounding box and im trying to make it convert that to a ConvexMeshShape.
Changing the vertex count from 8 to 24 allowed it to work a lot better but some rectangles cause the game to crash when collided. There is this one model whos collision box always causes a crash when an object bumps into it but my other models don't. The error is: "reactphysics3d/collision/shapes/ConvexMeshShape.cpp:106: virtual reactphysics3d::Vector3 reactphysics3d::ConvexMeshShape::getLocalSupportPointWithoutMargin(const reactphysics3d::Vector3&) const: Assertion `maxDotProduct >= decimal(0.0)' failed."
I had to go into the source code for this one. In ConvexMeshShape.cpp in the getLocalSupportPointWithoutMargin() function, there is a line of code "assert(maxDotProduct >= decimal(0.0));". I commented that out. The game no longer crashes when object collide into that one object. I don't know what the implications of this hack are but if I'm doing this wrong and that's the reason it's crashing, I would love to find out what the correct way is.
If I can use this to avoid implementing my own special colliders by building off of the plane class, I will. I was just wondering if ConvexMesh is out of date or borked and we're not supposed to be using it or if this is user error.

I have created a new issue here because it doesn't seem to be related to the current one.

This issue should now be fixed in the release of version v0.10.1. The documentation has also been updated.
Thanks again for reporting this issue.


HtmlTargetElement ParentTag doesn't work with @tagHelperPrefix

Given the following TagHelpers:
[HtmlTargetElement("row", ParentTag = "grid")]
public class RowTagHelper : TagHelper
{
}

public class GridTagHelper : TagHelper
{
}

Both of the following should work, but don't:

Hi. Thanks for contacting us.
We're closing this issue as there was not much community interest in this ask for quite a while now.


LuaHBTeX

I don't think that \ifluahbtex is the right test for normal usage. It does tell you if you can use luahbtex, but in most cases this shouldn't be important: The important question is if you can use luaharfbuzz. Why would it matter if you can use it because it is build into the engine or because it is available as an external library? Why shouldn't this be true for some future fork of LuaHBTeX? So I would prefer a test which is true iff require'luaharfbuzz' succeeds (or maybe if luaharfbuzz is defined, but I prefer the require variant because it's more flexible).

@zauguin  yes, you are probably right ...
the note about forks is true in general of course but mostly forks have maintained the old names and added new ones, so ptex is a fork of etex and  uptex is a fork of ptex and the tests here for uptex make the tests for etex, ptex and uptex all true.
For a fork of luahbtex it would be a matter of how radical the fork was whether you wanted the existing \ifluahbtex  to be true or false on the fork....
Coming back to \ifluahbtex  does require("luaharfbuzz") affect the behaviour of luatex (if you don't explicitly use harf mode?  That is, can I do the test without changing the behaviour?  Obviously just testing if luaharfbuzz is defined doesn't define anything but as you say it may not be such a good test.

No, require'luaharfbuzz' loads the luaharfbuzz binary if it is available, but as long as you don't use it it doesn't have any effect (except for slightly higher memory consumption). You could call Lua's searchers yourself, but that would be more complicated and introduce edge-cases. Also in the foreseeable future luaotfload will do require'luaharfbuzz' anyway, so if you call it again it will just return the already loaded instance.

@zauguin  OK so something like
\directlua{
if pcall(require, 'luaharfbuzz') then print("NO") else print("YES") end
}

@davidcarlisle I would switch YES and NO but otherwise yes.


[BUG] Only the last user in the list can be deleted

The bug
If I created more than one user they are displayed in a list. EG. User1 and User2. The list is alphabetically sorted, so the User1 is on top. If I now try to delete this first User1 a message appears which confirms to delete User2 which is the last of the list.
The OS that UpSnap is running on
Ubuntu 22.10
Version of UpSnap
docker image ghcr.io/seriousm4x/upsnap:latest
Your docker-compose.yml content
version: "3"
services:
  upsnap:
    container_name: upsnap_test
    image: ghcr.io/seriousm4x/upsnap:latest
    network_mode: host
    restart: unless-stopped
    volumes:
      - ./UpSnap:/app/pb_data
# # To use a non-root user, create the mountpoint first (mkdir data) so that it has the right permission.
# user: 1000:1000
    environment:
      - TZ=Europe/Berlin # Set container timezone for cron schedules
      - UPSNAP_INTERVAL=@every 10s # Sets the interval in which the devices are pinged
      - UPSNAP_SCAN_RANGE=<IP_ADDRESS>/23 # Scan range is used for device discovery on local network
#   - UPSNAP_SCAN_TIMEOUT=500ms # Scan timeout is nmap's --host-timeout value to wait for devices (https://nmap.org/book/man-performance.html)
#   - UPSNAP_PING_PRIVILEGED=true # Set to false if you don't have root user permissions
      - UPSNAP_WEBSITE_TITLE=PC Aufwecken # Custom website title
# # dns is used for name resolution during network scan
# dns:
#   - <IP_ADDRESS>
#   - <IP_ADDRESS>
# # you can change the listen ip:port inside the container like this:
    entrypoint: /bin/sh -c "./upsnap serve --http <IP_ADDRESS>:5000"
    healthcheck:
      test: curl -fs "http://localhost:5000/api/health" || exit 1
      interval: 10s

Reproduction steps
1. Go to "User"
2. Create User1
3. Create User2
4. Try to delete User1 -> User2 will be deleted
...

Additional information
No response

Thanks, fixed in 4.2.3


[9.4] ISPN-14183 Support protobuf message having size > 64M

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

Merged, thanks

Thank you!


Register to update and draw

registerCallbacks didn't register to update and draw, since those are not in the event handlers, the fix is simple just added them to the callbacks table manually.

Thanks 👍


chore(typescript): untrack tsconfig.tsbuildinfo

This PR stops tracking changes in tsconfig.tsbuildinfo cache file. The file is in .gitignore already, but was tracked earlier.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[stable/selenium] Selenium hub fails health probes and is restarted.

Is this a request for help?:
Nay.
Is this a BUG REPORT or FEATURE REQUEST? (choose one):

Feature Request (Change I would like to see)
Version of Helm and Kubernetes:
Helm:
Client: &version.Version{SemVer:"v2.9.1", GitCommit:"20adb27c7c5868466912eebdf6664e7390ebe710", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.9.1", GitCommit:"20adb27c7c5868466912eebdf6664e7390ebe710", GitTreeState:"clean"}

Kubernetes:
Server Version: version.Info{Major:"1", Minor:"8+", GitVersion:"v1.8.4+coreos.0", GitCommit:"4292f9682595afddbb4f8b1483673449c74f9619", GitTreeState:"clean", BuildDate:"2017-11-21T17:22:25Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}

NOTE: Our cluster uses t2.medium AWS instances. This issue might not appear on higher performance instance types.
Which chart:
charts/tree/master/stable/selenium
What happened:
When scaling down the number of chrome-nodes:
kubectl --kubeconfig=kubeconfig scale --replicas=1 deploy selenium-chrome
selenium-hub fails its health-checks and is restarted:
Warning  Unhealthy              43s (x2 over 1m)  kubelet, ip-172-22-30-239.ec2.internal  Liveness probe failed: Get http://<IP_ADDRESS>:4444/grid/console: net/http: request canceled (Client.Timeout exceeded while awaiting headers)
Warning  Unhealthy              39s (x2 over 1m)  kubelet, ip-172-22-30-239.ec2.internal  Readiness probe failed: Get http://<IP_ADDRESS>:4444/grid/console: net/http: request canceled (Client.Timeout exceeded while awaiting headers)

this can cause subsquent/concurrent tests to fail because the nodes are disconnected from the hub.
What you expected to happen:
Normally I would expect to be able to scale down the number of chrome nodes without any failures.
How to reproduce it (as minimally and precisely as possible):

Install the selenium stable chart.
Set the number of chrome-node replicas to a integer greater than 1. e.g. 3: kubectl --kubeconfig=kubeconfig scale --replicas=3 deploy selenium-chrome
Sleep 60
Scale the chrome-node deployment back down to 1. e.g. kubectl --kubeconfig=kubeconfig scale --replicas=1 deploy selenium-chrome
kubectl --kubeconfig=kubeconfig get po | grep selenium-hub
Sleep 45
kubectl --kubeconfig=kubeconfig get po | grep selenium-hub should have a higher restart count than it did before Step 4.
kubectl --kubeconfig=kubeconfig describe po selenium-selenium-hub-ab123456c-abcde should show failing health checks in the events log.

Anything else we need to know:
Selenium Developers reccomended using the health endpoint: /wd/hub/status and NOT: /grid/console.
This is because: "I do notice /grid/console hangs for a bit after a node loses connection." and "...when a node dies the grid server stalls for a while..." and "...registering and unregistering lots nodes is an expensive operation for the Grid, it tends to block it."
Additionally I added readinessProbe.initialDelaySeconds: 15 which also eliminated health check failures.
NOTE: Our cluster uses t2.medium AWS instances. This issue might not appear on higher performance instance types.
These are the changes I made to our template:
  livenessProbe:
    httpGet:
      path: /wd/hub/status
      port: {{ .Values.hub.port }}
    initialDelaySeconds: 30
    periodSeconds: 5
    timeoutSeconds: {{ .Values.hub.livenessTimeout }}
  readinessProbe:
    httpGet:
      path: /wd/hub/status
      port: {{ .Values.hub.port }}
    initialDelaySeconds: 15
    timeoutSeconds: {{ .Values.hub.readinessTimeout }}

Would you like to provide a PR?

@davidkarlsen I'd love to! Embarrassingly I cannot find out how to contribute to the project. 😳

I still have an open PR here:
https://github.com/helm/charts/pull/7554


Resolves #13 use pills for mobile menu

Some whitespace formatting changes included courtesy of Calva defaults that match the style guide, including single-space indentation of function arguments. I'm looking into how to configure vscode to use two spaces to match the project formatting.

 If I can figure out the right git command :).
I'll open another issue for whitespace/style stuff. I've just been going with vim defaults, with some adjustments so that calls to submit-tx are compressed horizontally.


[Vaults UI] Add developer tools in PP (Profile Picture) Menu

This menu should contains "Developer Tools" which opens a full screen Modal and display tabs "Providers" and "App Secrets" from the current "Developer Tools" menu item.

Closing since we don't want to do that, creating new cards for the dust apps tasks discussed here: https://dust4ai.slack.com/archives/C050SM8NSPK/p1724686850587629


Segmentation fault (core dumped) when trying to train or overtrain a model

Hello, I'm trying to use the train.py function to train a RIFE model with my own database. It all goes well till the last epoch where it gives me a "Segmentation fault (core dumped)". I'm using --nproc_per_node=1 and --world_size=1, so maybe that's the issue? It doesn't matter how many epoch I use it always gives this error at the last epoch. Also by launching the code like this: sudo -E /usr/bin/python3 -m torch.distributed.launch --nproc_per_node=1 train.py --epoch=1 --world_size=1 the error becomes simply "Segmentation fault" without the "(core dumped)" part. Any ideas of why is giving me this issue?

Have you solved this problem?


I am not able to receive mail in my inbox

When i am sending mail it shows that mail was sent i.e. queued as follows:-
[<-] 220 mail.monstorm.com ESMTP Postfix (Ubuntu)
[->] HELO monstorm.com
[<-] 250 mail.monstorm.com
[->] MAIL<EMAIL_ADDRESS>[<-] 250 2.1.0 Ok
[->] RCPT<EMAIL_ADDRESS>[<-] 250 2.1.5 Ok
[->] DATA
[<-] 354 End data with .
[->] Received: by monstorm.com (sSMTP sendmail emulation); Tue, 16 Jun 2015 14:09:44 +0000
[->] Date: Tue, 16 Jun 2015 14:09:44 +0000
[->]<EMAIL_ADDRESS>[->]<EMAIL_ADDRESS>[->] Subject:Test Dev V
[->]
[->] Hi All,
[->]        Release will start on DEV and NCDEV server in next 15 min.
[->] Please Commit your files in next 10 min.
[->] .
[<-] 250 2.0.0 Ok: queued as F21133DA3
[->] QUIT
[<-] 221 2.0.0 Bye
But i am not able to receive mail in inbox nor in my spam. Please give me any suggestions it's really urgent.

Try investigate a logs with "docker logs" when you sending email. Also you can try send email on other email host i.e. gmail and check spam folder.

@justforkidding Sorry for late reply. I'm a amateur programmer and don't have enough time to Q&A. Please refer to #1 which may be helpful. Thanks.

@DmitryINdig0  thanks. I have tried sending email i.e. gmail but it will not received in inbox. And i've also checked "docker logs" but it is not displaying any error.

@catatnight is there any other solution for this issue??

Seems like you want to use docker as relay only. Your actual mail domain is hosted on some other service provider. If you want to try you can build docker from https://github.com/gdvyas/docker-postfix/ and  and add -e YOURDOMAIN.TLD:relay:YOUR_ISP_OUTGOING_SMTP_SERVER  in docker run option.


More Secured Password Input

Short Description:
More Secured Password Input (interactive input instead of command-line [which might have history])
Still Support command-line argument, but if no password is given in arguments, the config initializing stage asks for it.

And rebase your branch from dev too. Thanks!

Will do (I don't have much time atm, but will try..)

so for now cancel this request right?

nope, just go pull from our dev branch and fix the conflict.

Someone (rarshonsky) has bitten me to it :)
https://github.com/PokemonGoF/PokemonGo-Bot/commit/cf0bab5e85ea369962f22ef26a6bf7b8e9f59d21


Multiple NICs

For NetFx and RT, added support to scan multiple connected NICs. This is
the default behavior. To scan only the "best" NIC, set the 6th argument
to the call to ResolveAsync() to "true".

Looks like this PR includes #20, correct?

Thanks!

Correct.
Sent from Windows Mail
From: Oren Novotny
Sent: Friday, October 23, 2015 4:31 PM
To: onovotny/Zeroconf
Cc: Allan Chin
Looks like this PR includes #20, correct?
—
Reply to this email directly or view it on GitHub.

Oh, the online documentation should probably be updated to describe the new argument.
Sent from Windows Mail
From: Oren Novotny
Sent: Friday, October 23, 2015 4:32 PM
To: onovotny/Zeroconf
Cc: Allan Chin
Merged #23.
—
Reply to this email directly or view it on GitHub.

Just update the docs. It may be a day or two before I can publish the update as I need to validate on iOS/Android. For now, you can grab the nupkg on the CI artifacts: https://ci.appveyor.com/project/onovotny/zeroconf/branch/master

Hey Oren,
I’m trying to build a PCL using Visual Studio 2015 to wrap Zeroconf, and I specify support for targets Windows 10, Xamarin.Android, and Xamarin.iOS. But when I try to install Zeroconf from the project’s Nuget Package Manager, I get this error. Is Zeroconf not supported on these target platforms, or am I missing something?
Severity
Code
Description
Project
File
Line
Error
Could not install package 'Zeroconf 2.3.3'. You are trying to install this package into a project that targets '.NETPortable,Version=v4.5,Profile=Profile7', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.
0
Also, do you know when Zeroconf 2.3.4 will get published?
Thanks,
Allan
From: Oren Novotny<EMAIL_ADDRESS>Sent: Friday, October 23, 2015 5:35 PM
To: onovotny/Zeroconf<EMAIL_ADDRESS>Cc: Sparksteam<EMAIL_ADDRESS>Subject: Re: [Zeroconf] Multiple NICs (#23)
Just update the docs. It may be a day or two before I can publish the update as I need to validate on iOS/Android. For now, you can grab the nupkg on the CI artifacts: https://ci.appveyor.com/project/onovotny/zeroconf/branch/master
—
Reply to this email directly or view it on GitHub https://github.com/onovotny/Zeroconf/pull/23#issuecomment-150727745 .  https://github.com/notifications/beacon/AL--vIsvSfkRHbqB96tylqK_Kl6-oA11ks5o-smagaJpZM4GU5Fb.gif

There was one NRE that I wanted to track down first before releasing 2.4. In the mean time, can you use the CI nuget feed as an additional source: https://ci.appveyor.com/nuget/zeroconf-n5mk74lp3y4p


Print all sheets of paper

I have 1 file pdf on server and when i preview or download to preview i see 2 pages with order like this:

page 1: have content
page 2: have content

But when i print this file i saw 3 sheets of page with order like this:

page 1: have content
page 2: empty page
page 3: have content
I use Vue2 and update check i see have and error with pdf 1.5

Hi @GitHubPhongNguyen,
Could you share the document you are having trouble with?


Not possible to add multiple kernel parameters with the same name.

I need to add the kernel parameters console=tty0 console=ttyS0 to my kernel boot options. I tried this:
kernel_parameter { "console":
value => "tty0",
}
kernel_parameter { "console":
value => "ttyS0",
}
But this does not work because I am not allowed to define kenel_parameter["console"] twice.

Pass an array of values instead: value => ['tty0', 'ttyS0']
https://github.com/hercules-team/augeasproviders_grub/#manage-parameter-with-multiple-values
http://augeasproviders.com/documentation/examples.html#kernelparameter-provider

You are right, I just discovered this in the documentation after submitting this. Sorry, I should have checked this earlier.