Remove unused dependencies from classpath

I noticed that some weird/oudated dependencies were showing up in my classpath like Ktor CIO and Ktor Android clients. I use OkHttp as my ktor backend. Since Kamel is a library its not supposed to implement things like Ktor Logging and CIO or other ktor client backends and leave that to the user.
I cleaned up any other unneeded depencencies I found in the files.

While I agree with you, don't you think there should be some default clients added? So, the user can only add the dependency for the library and can immediately use it, without having to look for other dependencies.

No, libraries that use ktor are meant to only include the dependencies it needs. Adding engines only make Ktor confused.
When using Ktor in a muliplatform way, the default HttpClient constructor(with no engine specified) uses whatever engine is implemented on the current platform, but this is broken when libraries start including thier own engines. You could add an engine example to the readme to make up for this.

Okay, no problem then. Could you update the README by adding a link to the Ktor page containing the engines?
Thanks.

Done 👍


Bug: Playground comments have inconsistent timestamps in different window contexts

When adding collaborative comments, the timestamps are inconsistent in different window contexts.
Lexical version: 0.17.0
Steps To Reproduce

Clone this repo and do relevant npm i.
In the playground package, run npm run dev.
Open http://localhost:3000/?isCollab=true in 2 different tabs.
Add some text and add a comment.
Wait a minute until the comment time stamp says "1 min. ago".
Reload only one browser tab.
Observe the comment time says "Just now" which is now different from the other tab.

Link to code example: use the playground in this repo (see reproduction steps above).
The current behavior
Comment timestamps are inconsistent in different window contexts.
The expected behavior
Comment timestamps are the same in different window contexts.
Impact of fix
This isn't affected if you test with the Split Screen option as it's always the same window context. But for all other usual scenarios the inconsistent timestamps are disconcerting.

Comments in the playground are just a playground tech demo, not an exported package suitable for re-use. The timestamp uses the performance counter without basing it on the timeOrigin (or just using Date.now) so it really doesn't make a lot of sense when used in more than one tab, browser or window. Could certainly be fixed if you're inclined to write a PR.


You should not use  or withRouter() outside a

Hi Team,
I'm facing below issue in react-router-dom.
"You should not use  or withRouter() outside a "
I'm using following dependencies.
"react": "^16.4.0",
"redux": "4.0.0",
"react-dom": "16.4.1",
"react-redux": "5.0.7",
"react-router-dom": "^4.4.0-alpha.1",
"react-router-redux": "5.0.0-alpha.6",
Also see attached screen-grab.
Thanks,
BSR

This is a bug tracker, not a support system. For usage questions, please use Stack Overflow or Reactiflux where there are a lot more people ready to help you out. Thanks!


Make delete planner independent of parameters

This removes the "bound analyze" variant for delete statements.
The AnalyzedDeleteStatement and all delete plan variations are no
longer using any parameter values. All plans can be used for single or
bulk execution. The parameter binding happens in the executor.
There is a bit of adapter logic necessary to still support the other
statement types where we have bulk-bound analyzed statements.
In addition, this also tunes the DeleteById execution a bit: Instead
of executing 1 delete request per id, it'll execute 1 delete request per
shard. This should improve the performance for queries like delete from t where pk in (...)

Codecov Report

Merging #6561 into master will increase coverage by 0.05%.
The diff coverage is 92.16%.

@@             Coverage Diff              @@
##             master    #6561      +/-   ##
============================================
+ Coverage     82.89%   82.95%   +0.05%
- Complexity    13216    13227      +11
============================================
  Files          1515     1517       +2
  Lines         49076    49140      +64
  Branches       5908     5911       +3
============================================
+ Hits          40682    40764      +82
+ Misses         5597     5585      -12
+ Partials       2797     2791       -6

Impacted Files
Coverage Δ
Complexity Δ

...ava/io/crate/analyze/AnalyzedStatementVisitor.java
50% <ø> (+1.02%)
24 <0> (ø)
:arrow_down:

...e/executor/transport/task/ShowCreateTableTask.java
66.66% <ø> (ø)
2 <0> (ø)
:arrow_down:

...main/java/io/crate/planner/node/dql/CountPlan.java
89.65% <ø> (ø)
14 <0> (ø)
:arrow_down:

...e/planner/node/management/ShowCreateTablePlan.java
85.71% <ø> (ø)
3 <0> (ø)
:arrow_down:

...src/main/java/io/crate/executor/task/NoopTask.java
83.33% <ø> (ø)
3 <0> (ø)
:arrow_down:

...java/io/crate/planner/node/dcl/GenericDCLPlan.java
100% <ø> (ø)
4 <0> (ø)
:arrow_down:

...java/io/crate/planner/node/ddl/GenericDDLPlan.java
100% <ø> (ø)
4 <0> (ø)
:arrow_down:

sql/src/main/java/io/crate/planner/Merge.java
88.73% <ø> (ø)
26 <0> (ø)
:arrow_down:

...src/main/java/io/crate/analyze/DeleteAnalyzer.java
100% <ø> (+8%)
3 <0> (-6)
:arrow_down:

...operation/projectors/sharding/ShardedRequests.java
100% <ø> (ø)
4 <0> (ø)
:arrow_down:

... and 96 more

Continue to review full report at Codecov.

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


Configuration guidelines for some membership schemes are missing

In the document [1], it has mentioned multiple membership schemes as following.
WSO2 supports the following membership schemes for clustering

Well-known address (WKA)
Multicast membership
AWS membership
AWS ECS membership
Kubernetes membership

But only WKA and AWS ECS are described in the configurations section.
Affected versions
IS 5.9.0 upwards
Additional info
Documents on the Kubernetes membership scheme & AWS membership scheme for the previous configuration model can be found at [2] & [3].
[1] https://is.docs.wso2.com/en/latest/setup/deployment-guide/#clustering-related-configurations
[2] https://docs.wso2.com/display/CLUSTER44x/Setting+Up+a+Cluster+in+Kubernetes+Mode
[3] https://docs.wso2.com/display/CLUSTER44x/Setting+Up+a+Cluster+in+AWS+Mode

Current status

[x] Well-known address (WKA)
[ ] Multicast membership
[ ] AWS membership
[x] AWS ECS membership
[x] Kubernetes membership

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


fix: enhance VNode logic and support multiple Trace IDs in span's ref

Fixes https://github.com/apache/skywalking/issues/11805
Video (Mock Data)
https://github.com/apache/skywalking-booster-ui/assets/20871783/b12439b8-602f-48d1-9727-3fe42ec33344
Signed-off-by: Qiuxia Fan<EMAIL_ADDRESS>

We need to reset all query conditions but the trace ID.

Updated
https://github.com/apache/skywalking-booster-ui/assets/20871783/3a4e551b-d16c-4c44-a9bc-6b62e806c664


Cannot find user settings

Followed instructions to get to User Settings and there's nothing there

"My profile".


arm64-v8a/libsuper_native_extensions.so, arm64- v8a/libirondash_engine_context_native.so, armeabiv7a/ libirondash_engine_context_native.so (MOBSF - SHARED LIBRARY BINARY ANALYSIS Reports).

After Testing my Application in MOBSF it throws some Warnings so how to resolve SHARED LIBRARY BINARY ANALYSIS warnings.
i've attached full testing report:
@knopp , Could you please look at this issue?

reports.pdf

The stack protection can probably be enabled through a rustc flag. Symbol stripping can be done at later stage so it's not really relevant (I don't think we should strip symbols by default) and I'm not sure how much fortified functions are relevant for Rust. I don't think that the code in super_native_extensions calls any unsafe glibc functions directly.


Menu elements become stretched or squashed

When changing window size, the table that stores the menu elements becomes stretched or squashed causing the items to be the wrong size and not fill the screen
Affects latest version

Fixed in most recent version by setting the stage's viewport bounds to start after black bars (gutters) and only include the game window.


Javadocs for Riak Search [JIRA: CLIENTS-34]

Need to document the search command and its associated classes.

Finished during previous Sprint.
[posted via JIRA by Alexander Moore]


Bug: 'variable' is defined but never used (@typescript-eslint/no-unused-vars) but is actually used.

Hey 👋,
I had a bug with ts-standard, I have a file called global.d.ts that look like this :
import { UserRequest } from '../models/User'

declare global {
  namespace Express {
    interface Request {
      user?: UserRequest
    }
  }
}

Where UserRequest is a simple interface, with that setup as I said in the title I've got  'UserRequest' is defined but never used. (@typescript-eslint/no-unused-vars) error, but I shouldn't have this error because I'm actually using it.
For the moment my workaround (not really pretty) is to do something like this :
// eslint-disable-next-line
import { UserRequest } from '../models/User'

Looking forward to discuss the bug, thank you.

So ts-standard is just a wrapper around https://github.com/standard/eslint-config-standard-with-typescript and https://github.com/typescript-eslint/typescript-eslint
This seems like a parsing/linting issue. I would recommend posting your issue on the https://github.com/typescript-eslint/typescript-eslint project where the typescript parser lives.
if you think this might instead be a configuration issue with how ts-standard passes options to eslint, the best way to test that out would be to install eslint, typescript-eslint, and the eslint-config-standard-with-typescript, configure it all manually, and run it to see if the error still exists. If the error is gone, then posting your configuration would be extremely helpful and we can figure out which options need to be passed to eslint from ts-standard 😄

It's a known issue with typescript-eslint for quite some time : https://github.com/typescript-eslint/typescript-eslint/issues/1856
Apparently the issue will be fixed in the 4.0 release : https://github.com/typescript-eslint/typescript-eslint/pull/2330.
Is ts-standard will upgrade typescript-eslint for the 4.0 version when ready ?

Yes, whenever typescript-eslint releases a new update, ts-standard will be updated appropriately 😄

Alright, since it's not really an issue related to ts-standard, I'll close this for now.


Fix test warning.

python/tests/test_async_client.py::TestCommands::test_expireat_pexpireat_ttl_with_positive_timeout[ProtocolVersion.RESP2-True]
python/tests/test_async_client.py::TestCommands::test_expireat_pexpireat_ttl_with_positive_timeout[ProtocolVersion.RESP2-False]
python/tests/test_async_client.py::TestCommands::test_expireat_pexpireat_ttl_with_positive_timeout[ProtocolVersion.RESP3-True]
python/tests/test_async_client.py::TestCommands::test_expireat_pexpireat_ttl_with_positive_timeout[ProtocolVersion.RESP3-False]
  /home/ubuntu/repositories/glide/python/python/tests/test_async_client.py:984: RuntimeWarning: coroutine 'check_if_server_version_lt' was never awaited
    if not check_if_server_version_lt(redis_client, "7.0.0"):
  Enable tracemalloc to get traceback where the object was allocated.
  See https://docs.pytest.org/en/stable/how-to/capture-warnings.html#resource-warnings for more info.

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@barshaul @shohamazon maybe we should run the python tests with some "no warning" flag, so that these issues will stop popping up?

@barshaul @shohamazon maybe we should run the python tests with some "no warning" flag, so that these issues will stop popping up?

I can do that


Rename partial folder

Renamed views/jobseekers/application/_account_header.html.slim (Rails default) to views/jobseekers/dashboard/_account_header.html.slim.
This is to avoid the similarity of names between the folders 'jobseekers/job_applications' vs 'jobseekers/application'.
Open to differing opinions: is 'dashboard' the most meaningful/succinct name possible? dashboard_partials?

Review app deployed to https://teaching-vacancies-review-pr-2907.london.cloudapps.digital


Simple_form Compatibility

If possible, can you extend the gem to include simple_form support.  I had to recreate my whole entire form, just use it. Otherwise, you did a great job!

@RailsCod3rFuture did you end up adding simpleform support yourself?
We are considering using this, and would add in simpleform support, it would be required for us.


tetragon: Add pprof http support

This PR adds support for pprof http endpoints.
Fixes: https://github.com/cilium/tetragon/issues/550
Signed-off-by: Andzej Maciusovic<EMAIL_ADDRESS>

@kkourt Rebased and added Signed-off-by text.

@kkourt Rebased and added Signed-off-by text.

Could you also please add it in the git commit as well? git commit --amend --signoff should do the trick.
Thanks!

Check patch still complains. Also it would have been great if you added ina  separate patch some docs under docs/debugging/README.md a section for profiling Tetragon agent with examples and output, so we can run them.
Thank you, LGTM!

@kkourt Ah, sorry about that. Added signoff to git commit.

@kkourt Ah, sorry about that. Added signoff to git commit.

Great! Many thanks!


[REF] Use partial functions and remove if statements for MP calls

Closes # .

Changes proposed in this pull request:

Pool.imap's documentation says the following:

For very long iterables using a large value for chunksize can make the job complete much faster than using the default value of 1.

I'm not sure what constitutes a "long iterable", but I can't help but wonder if using a larger chunksize value might be worth it.

Joblib can work with memmaps: https://joblib.readthedocs.io/en/latest/parallel.html#working-with-numerical-data-in-shared-memory-memmapping


Support secret store csi driver for cosign

The secret store csi driver mounts .crt files that works for Notary, we need to investigate configuration/chart changes needed to mount cosign store at a different path to notary validation certs

https://azure.github.io/secrets-store-csi-driver-provider-azure/docs/configurations/getting-certs-and-keys/
Here is some documentation on how to get cert and key using the driver. If we want to get the public key, we probably just need to specify objectType: key , instead of cert

Closing as we are moving away from CSI secret store and using built in CertificateStore


Multiple update schedules

The update system is far too noisy, but reducing the noise means that no data is reported for a while on some data points.
Multiple schedules should be created for updating different kinds of data, and all data that hasn't been updated in a while should be pushed on one of those schedules.

Should probably use schedule - so that plugins can specify their update frequency appropriately.


Adding "start" and "end" to createReadStream() options definition.

See node documentation here: https://nodejs.org/api/fs.html#fs_fs_createreadstream_path_options

please resolve conflict.

Hi Vvakame,
I am not sure how and I don't know how to see which conflicts we do have here.
Any help pointing me on how to address this would be appreciated.
Thanks

Ok, Let me know if that fixed it.

https://help.github.com/articles/resolving-a-merge-conflict-from-the-command-line/ ?

Thanks for the contribution. This PR contains merge conflicts and can not be merged at the moment. Please refresh this PR and address the merge conflicts.

Hello and thank you for your contribution!
Due to an excessively long queue of pull requests that have become stale, we are "declaring bankruptcy" and closing all PRs opened before May 1, 2016. If you'd still like to merge this code in, please open a new PR that has been merged and rebased with the master branch.
Going forward, we are committing to review or merge all PRs on a regular basis so this bankruptcy will not occur again. We apologize for the incovenience and hope you will continue to contribute to DefinitelyTyped in the future.
Thanks
The TypeScript and DefinitelyTyped teams


handle case when schema info is present but value is null

Actually I'm unsure if that is the best solution.
In case of a tombstone it might be actually better to forward the record without any changes.

Hey @twiechert , thanks for the contribution!

might be actually better to forward the record without any changes

Agreed. Could you give an example of when the schema is non-null but the value is null?
Also, do you think the message key logic should also allow for null values to be passed though in a similar fashion, rather than having the skip configuration?

hi @cricket007 ,
consider for example the comcast mirrortool:
records.add(new SourceRecord(sourcePartition, sourceOffset, destinationTopic, null, Schema.OPTIONAL_BYTES_SCHEMA, krecord.key(), Schema.OPTIONAL_BYTES_SCHEMA, krecord.value(), krecord.timestamp(), destinationHeaders));

replicating a tombstone, thus value null but setting the schema member to  Schema.OPTIONAL_BYTES_SCHEMA

Also, do you think the message key logic should also allow for null values to be passed though in a similar fashion, rather than having the skip configuration?

I think a null-key record is rarely useful but I also think the schema registry smt should not exclude such records by default (option would be fine).

should not exclude such records by default

Right, thus why the config option is currently true, however I am throwing an exception when the keys are included and null, so I would accept if this PR adjusted that logic as well for setting the updatedKey to be null for when includeKeys == true && key == null
        if (includeKeys) { // default = true
            if (key == null) {
                throw new ConnectException("Unable to copy record key schemas for null keys. Set '" + ConfigName.INCLUDE_KEYS + "=false'.");
}

Or maybe having to manually set include.message.key = false is good enough and having the logic for includeKeys ? updatedKey : r.key() is adequate. Personally, I can't really decide what is most user-friendly.

Well, I think the name include.message.key is a bit misleading, since it it always included. Question is if it is transformed or not.
I would personally say to never skip a record. While it might not be best practice to mix records with and without schema inside one topic, I don't think the smt should (by default) simply skip the records without schema in such case.
One could think of including options like skipNullKey/skipNullValue which would explicitly allow for that.

Incorporated parts of this change in #8


Fix tflite_runtime pip package

Standalone pip package of tflite was broken at least since interpreter_wrapper refactoring.
This commit adds the missing file to the python extension sources.
It also fixes an invalid import in the tflite_runtime/lite/__init__.py file created by build_pip_package.sh, relates to #23082
Not sure how to add tests for this...

@mayeut  Could you please resolve the conflicts? Thanks!

I'm going to go ahead and close this PR, because it seems to have stalled. If you're still interested in pursing this (and responding to my comments), please feel free to reopen!


Implement Least Common Ancestor query

Implement Least Common Ancestor query, e.g. from Susceptible Asymptomatic and Susceptible Symptomatic find Symptomatic

There is a branch now: https://github.com/indralab/mira/tree/lca-query

Done in #130


I'm unable to mock inherited methods in a subclass when the parent is also mocked

I'm using TestSlide version: 2.5.7
Given: https://gist.github.com/nsheridan/a08dc79a84cc2974710b7a02b78ff77b

When I run:

% testslide test.py

I expected this to happen:
The test test_parent_and_child_no_override should pass

But, instead this happened:
When both the child and parent classes are mocked  I get a NonExistentAttribute exception when I attempt to mock a non-overridden method in the child class:
% testslide test.py
test.TestThing
  test_child
  test_parent
  test_parent_and_child_no_override: NonExistentAttribute: 'a_method' can not be set.
  test_parent_and_child_no_override_careful_ordering
  test_parent_and_child_override

Failures:

  1) test.TestThing: test_parent_and_child_no_override
    1) NonExistentAttribute: 'a_method' can not be set.
    <StrictMock 0x10EF3CBD0 template=test.AChild test.py:32> template class does not have this attribute so the mock can not have it as well.
    See also: 'runtime_attrs' at StrictMock.__init__.
      File "/Users/nsheridan/homebrew/Cellar/python/3.7.7/Frameworks/Python.framework/Versions/3.7/lib/python3.7/contextlib.py", line 119, in __exit__
        next(self.gen)
      File "/Users/nsheridan/homebrew/Cellar/python/3.7.7/Frameworks/Python.framework/Versions/3.7/lib/python3.7/unittest/case.py", line 59, in testPartExecutor
        yield
      File "/Users/nsheridan/homebrew/Cellar/python/3.7.7/Frameworks/Python.framework/Versions/3.7/lib/python3.7/unittest/case.py", line 628, in run
        testMethod()
      File "test.py", line 38, in test_parent_and_child_no_override
        self.mock_callable(achild, "a_method").to_return_value(None)

Finished 5 example(s) in 0.2s: .
  Successful: 4
  Failed: 1

As you can see from the example, it is possible to

mock a method in a child class before the parent is mocked
mock a method in a child class after the parent is mocked if the method is overridden in the child

However if the parent and child classes are both mocked it is not possible to mock non-overridden methods in the child class

Thanks for reporting this!
This bug is a result of the extremely hacky way that mock_constructor is implemented due to an upstream Python bug...
The gist is:
When self.mock_constructor(sys.modules[__name__], "AClass").to_return_value(aclass) is called:

It must empty all attributes of AClass,
Create a subclass of it,
Put all attributes back at the subclass (but mocked),
Finally put this subclass in place of AClass at the module.

So, when achild = testslide.StrictMock(AChild) runs, it gets a reference to the original AChild class. But when self.mock_constructor(sys.modules[__name__], "AChild").to_return_value(achild) runs, it creates the new mocked class, which should be accessed via getattr(sys.modules[__name__], "AChild") exclusively, however, achild still has the reference for the old empty class.
mock_constructor has safeguards in place against similar cases, and it'll refuse to work if there are pre-existing instances of the mocked class. But the check, is currently not covering this case and needs improvement.
I drafted a fix:
diff --git a/testslide/mock_constructor.py b/testslide/mock_constructor.py
index a85fa6a..0b12ef5 100644
--- a/testslide/mock_constructor.py
+++ b/testslide/mock_constructor.py
@@ -315,13 +315,21 @@ def mock_constructor(target, class_name, allow_private=False, type_validation=Tr
         instances = [
             obj
             for obj in gc.get_referrers(original_class)
-            if type(obj) is original_class
+            # FIXME exclude all allowed cases, eg: parent class, references at this module.
+            if id(obj) != id(target)
         ]
         if instances:
+            references_str = "> " + "\n\n > ".join(f"{id(obj)}: {type(obj)}\n  {repr(obj)}" for obj in instances)
             raise RuntimeError(
-                "mock_constructor() can not be used after instances of {} were created: {}".format(
-                    class_name, instances
-                )
+                "mock_constructor() can not be used with multiple references to the class:\n"
+                "Due to an upstream Python bug, mock_constructor() implementation contains "
+                "a lot of workarounds "
+                "(https://testslide.readthedocs.io/en/master/patching/mock_constructor/index.html#implementation-details). "
+                "As a result, it is impossible to support its usage in scenarios where:\n"
+                "- Instances of the class have already been created\n."
+                "- Multiple references to the class exist.\n"
+                f"The following live objects have references to {class_name}:\n\n"
+                + references_str
             )

         if not inspect.isclass(original_class):

But as the FIXME comment tells, it still needs some love, this is gonna be tricky, not only to add all exceptions, but to also, give a meaningful error. Eg: the object with the reference can be a dictionary of another class (eg: OtherClass.whatever = AChild), and just printing this dictionary at the error, won't make it obvious that it belongs to OtherClass.


order statuses empty

Expected behavior
Bo shoud show correct order statuses
Actual behavior
BO show empty fields, in database the current_state column is "0"
Steps to reproduce the behavior
Create an order as a customer, complete the checkout process and come back to back office to see the empty status
Environment

PrestaShop version: <IP_ADDRESS>
Mollie module version: 4,4,2
PHP version: 7.1.26

Message sent, thanks for the support

Hi @powolnymarcel ,
The credentials that were previously sent are blocked, we can't log in, should we proceed with this case and wait for the new credentials with a new access?
Looking forward to your reply!
--
Best regards,
Invertus Support team

Hi @powolnymarcel ,
Due to inactivity I will mark this issue as closed for now.
Let me know if you have any other questions!
--
Best regards,
Invertus Support team


Updated sumeval for surprise

Adding surprise languages for the sumeval shared task.

This is a very small change so I'll just merge it.


[Documentation] [Unsafe Configuration] Added carbon.unsafe.driver.working.memory.in.mb parameter to differentiate between driver and executor unsafe memory

Added carbon.unsafe.driver.working.memory.in.mb parameter to differentiate between driver and executor unsafe memory

[ ] Any interfaces changed?
Yes, added a new parameter carbon.unsafe.driver.working.memory.in.mb
[ ] Any backward compatibility impacted?
No
[ ] Document update required?
Yes. Updated
[ ] Testing done
Verified manually
[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
NA

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/6436/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/6103/

Why driver needs this unsafe memory?

@xuchuanyin ....Usually in production scenarios driver memory will be less than the executor memory. Now we are using unsafe for caching block/blocklet dataMap in driver.  Current unsafe memory configured fo executor is getting used for driver also which is not a good idea.
Therefore it is required to separate out driver and executor unsafe memory.
You can observe the same in spark configuration also that spark has given different parameters for configuring driver and executor memory overhead to control the unsafe memory usage.
spark.yarn.driver.memoryOverhead and spark.yarn.executor.memoryOverhead

LGTM


No documentation

Please publish some documentation.
My personal use case is getting a sense of what the plugin does and how it works without installing it, but presumably it would benefit users, too.

Will start writing docs after the key features are done. There are temporary docs on our discord although I know that doesn’t quite answer your question.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Colin Tedford @.>
Sent: Friday, December 23, 2022 6:43:40 PM
To: Make-md/makemd @.>
Cc: Subscribed @.***>
Subject: [Make-md/makemd] No documentation (Issue #24)
Please publish some documentation.
My personal use case is getting a sense of what the plugin does and how it works without installing it, but presumably it would benefit users, too.
—
Reply to this email directly, view it on GitHubhttps://github.com/Make-md/makemd/issues/24, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAFHKNWW22VNAOKAOTTV3DTWOY2KZANCNFSM6AAAAAATIEHSHA.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

OK. Maybe specify in the README which features are done and which are planned, and note that documentation will follow?

The features in the readme are all in production but we have a few major features were at the late stages of finishing. Will update soon.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Colin Tedford @.>
Sent: Friday, December 23, 2022 7:36:59 PM
To: Make-md/makemd @.>
Cc: JP Cen @.>; Comment @.>
Subject: Re: [Make-md/makemd] No documentation (Issue #24)
OK. Maybe specify in the README which features are done and which are planned, and note that documentation will follow?
—
Reply to this email directly, view it on GitHubhttps://github.com/Make-md/makemd/issues/24#issuecomment-1364423076, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAFHKNQGQJBXGDBQSGJTPLDWOZASXANCNFSM6AAAAAATIEHSHA.
You are receiving this because you commented.Message ID: @.***>


Create batchamalick.yaml

hi @batchamalick so sorry I didnt see this sooner ... normally these requests process automatically, but in this case it did not because the file name you submitted ends with .yaml, and the workflow only triggers if your filename ends in ,yml
If you could please resubmit your pull request with the .yml extension, the workflow shoudl trigger for you automatically and process within a few minutes.
Thanks!


Import the notification banner macro as part of the layout

Import the notification banner so that users can include it in the prototype kit without needing to import it themselves, as we tell them to do in the ‘Getting Started’ documentation in the Design System.

Raised an issue to avoid this in the future: #967


Tapping on menu item does nothing?

I just push the version with the Provider implementation of this. However, it is just UI challenge.


Starter code

I'm submitting a ...

[x] feature request

Scaffolding and Starter code

Resolved in https://github.com/RommelTJ/rn-hooks-starter/pull/4


Support updating the gamepad mapping.

This is only supported on desktops yet (on mobile standard layout isn't
implemented yet, and on the web this is the browser's responsibility).
Fixes #1723.

FYI - not tested yet; for now just sent for early review. Going to test early next week.

On Sat, Aug 21, 2021, 02:14 Hoshi, Hajime (星一) @.***>
wrote:

@.**** commented on this pull request.
Thank you!
In input.go
https://github.com/hajimehoshi/ebiten/pull/1775#discussion_r693310717:

@@ -234,6 +234,15 @@ func IsStandardGamepadLayoutAvailable(id GamepadID) bool {
return uiDriver().Input().IsStandardGamepadLayoutAvailable(id)
}

+// UpdateStandardGamepadLayoutMappings updates the gamepad layout definitions using a set of definitions in SDL_GameControllerDB format.
+//
+// This functionality is only supported on desktop systems and will just return an error elsewhere.
+//
+// UpdateStandardGamepadLayoutMappings must be called on the main thread before the main loop.
Is this true? We should be able to make this concurrent-safe.

GLFW docs say it must be run on the main thread. I could do the dance to
run it on the main thread from anywhere like many other functions do - did
not see it as necessary though.
What I do not yet know is when mapping apply when you do this when the game
is already running - e.g. does one need to reconnect the gamepad to
actually apply changes to it.
Will try, then do this change and document it properly.

In input.go
https://github.com/hajimehoshi/ebiten/pull/1775#discussion_r693311425:

@@ -234,6 +234,15 @@ func IsStandardGamepadLayoutAvailable(id GamepadID) bool {
return uiDriver().Input().IsStandardGamepadLayoutAvailable(id)
}

+// UpdateStandardGamepadLayoutMappings updates the gamepad layout definitions using a set of definitions in SDL_GameControllerDB format.
+//
+// This functionality is only supported on desktop systems and will just return an error elsewhere.
+//
+// UpdateStandardGamepadLayoutMappings must be called on the main thread before the main loop.
+func UpdateStandardGamepadLayoutMappings(mappings string) error {

return uiDriver().Input().UpdateStandardGamepadLayoutMappings(mappings)

Just returning a boolean value should be fine. As some platforms (js)
don't support it on purpose, this should not be an 'error'.

Will do; the reason why I went with the error is to distinguish the two
reasons "not needed here" and "failed to parse".
So to fix this, are you OK if I make it return false on mobile (not
implemeted yet), true on JS (not needed) and forward the value on desktop
(i.e. indicating parse error)?
Or would you prefer true on mobile too where standard layout is not
implemented at all yet?
—

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hajimehoshi/ebiten/pull/1775#pullrequestreview-735470964,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB5NMDXY643GIFYCBQOYN3T5474FANCNFSM5CRCRDVA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

What I do not yet know is when mapping apply when you do this when the game
is already running - e.g. does one need to reconnect the gamepad to
actually apply changes to it.
Will try, then do this change and document it properly.

Ah good point. I appreciate your trial.

So to fix this, are you OK if I make it return false on mobile (not
implemeted yet), true on JS (not needed) and forward the value on desktop
(i.e. indicating parse error)?

Hmm, I see. What about returning (bool, error)? The mobile will always return false, nil.

BTW should I leave out the "take effect immediately" comment? It is true for GLFW according to its source, but I can neither tell if it's gonna stay this way in GLFW in the future, nor if the not yet written mobile support will have this property too.
Very likely semantics possible everywhere is applying it to all gamepads if run before the main loop, and any newly connected gamepads when run inside the main loop.

Another thing is - will the mobile support use the same format, or would you prefer a differently formatted string when implementing this for mobile? Could make the exact format of the string explained as platform dependent instead, and just say that on desktop platforms it's gonna be a SDL_GameControllerDB string.

Thank you for updating!

BTW should I leave out the "take effect immediately" comment? It is true for GLFW according to its source, but I can neither tell if it's gonna stay this way in GLFW in the future (as the documentation nowhere states this), nor if the not yet written mobile support will have this property too.
Very likely semantics possible everywhere is applying it to all gamepads if run before the main loop, and any newly connected gamepads when run inside the main loop.

Hmm, let's keep the comment and the current behavior for usability. We have to be careful about GLFW's update. And we should emulate exactly the same behavior for mobiles.
BTW, does "take effect immediately" mean that the mapping for an already-connected gamepad is also updated?

Another thing is - will the mobile support use the same format, or would you prefer a differently formatted string when implementing this for mobile? Could make the exact format of the string explained as platform dependent instead, and just say that on desktop platforms it's gonna be a SDL_GameControllerDB string.

I plan to use exactly same format for mobiles. Actually the database already includes Android and iOS data. I don't want to bother Ebiten users with forcing different formats for other platforms.

BTW, does "take effect immediately" mean that the mapping for an already-connected gamepad is also updated?

Yes; just confirmed this. Moved my code to update gamepad handling in the jump key handling, and provided a mapping with left/right directions swapped. Was hilarious, on the first jump the directions reversed.
So GLFW does properly apply this.
Does BTW mean that when wanting to add multiple mappings, one better adds them with one call than with many call - otherwise this becomes O(n^2) as every time this function is called, all mappings are evaluated for all connected gamepads.

FYI - not tested yet; for now just sent for early review. Going to test early next week.

Change now tested properly and working. Ready for full review / merge now.

js build failure fixed, thanks for the automated workflows!

Sorry, there was another js failure. Fixed too now.

Thanks!

Does BTW mean that when wanting to add multiple mappings, one better adds them with one call than with many call - otherwise this becomes O(n^2) as every time this function is called, all mappings are evaluated for all connected gamepads.

Hm, I don't think this function is so often called. Then I think we don't have to care about the computational cost.

Yeah - just wondering if I should explicitly state in the comment that
multiple mappings separated by newlines are permitted.
On Sun, Aug 22, 2021, 10:34 Hoshi, Hajime (星一) @.***>
wrote:

Thanks!
Does BTW mean that when wanting to add multiple mappings, one better adds
them with one call than with many call - otherwise this becomes O(n^2) as
every time this function is called, all mappings are evaluated for all
connected gamepads.
Hm, I don't think this function is so often called. Then I think we don't
have to care about the computational cost.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hajimehoshi/ebiten/pull/1775#issuecomment-903278396,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB5NMBYEYMBXJRNJZWH4H3T6EDHHANCNFSM5CRCRDVA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Yeah - just wondering if I should explicitly state in the comment that
multiple mappings separated by newlines are permitted.

We should permit this explicitly. Please fix the comment. Thanks!

I expanded upon the comments to describe this, and also now explain that it is valid but not necessary to pass an entire gamecontrollerdb.txt file, that this feature ideally should be exposed to the user in some form, and that people who have hardware needing this facility should send a pull request to have it included in the official database.


Modify to allow alternate ripping commands via optional scripts

Addresses Issue #16 by checking for the presence of executable scripts in the same directory as ripper.sh:

BLURAYrip.sh to be used as an alternate to the BluRay DVD ripping commands from ripper.sh,
DVDrip.sh to be used as an alternate to the DVD ripping commands from ripper.sh,
CDrip.sh to be used as an alternate to the audio CD ripping commands from ripper.sh,
DATArip.sh to be used as an alternate to the DATA CD ripping commands from ripper.sh

If the relevant script is found, it will be executed as an override to the associated command(s) within ripper.sh; if none of the defined scripts is found, no changes to previous ripper.sh operation occurs.
Additionally includes some minor code re-structuring to eliminate duplicate code and to assure that the defined variables are used reliably and consistently throughout the script.

Thanks for the pr - this looks good.

Great - glad you like the change!  I'd be happy to make updates to the
README.md file to document this new functionality.  I'm thinking the answer
to the "How do I set ripper to do something else?" FAQ question could have
an additional paragraph for this purpose.  Does that make sense to you?
On Thu, Feb 28, 2019 at 6:31 AM RiX<EMAIL_ADDRESS>wrote:

Thanks for the pr - this looks good.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rix1337/docker-ripper/pull/17#issuecomment-468273219,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXIWhm-d3VWZMpvOW_m_jHY1j_0anYguks5vR9pDgaJpZM4bO2I_
.

Totally, please do so by sending another PR - I hardly ever grant repo edit rights.

Works for me!
On Thu, Feb 28, 2019, 09:57 RiX<EMAIL_ADDRESS>wrote:

Totally, please do so by sending another PR - I hardly ever grant repo
edit rights.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rix1337/docker-ripper/pull/17#issuecomment-468350329,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXIWhuuKpFCSjBIdXIa6uNiT1dTRogijks5vSApjgaJpZM4bO2I_
.


Version check is not work for csharp_ls and fautocomplete

Problem description
The version check is only work for lua and rust, but others not work
Neovim version (>= 0.6)
NVIM v0.6.1
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by builduser
Features: +acl +iconv +tui
See ":help feature-compile"
system vimrc file: "$VIM/sysinit.vim"
fall-back for $VIM: "/usr/share/nvim"
Run :checkhealth for more info
Operating system/version
Linux cht-PC 5.16.13-arch1-1 #1 SMP PREEMPT Tue, 08 Mar 2022 20:07:36 +0000 x86_64 GNU/Linux
I've recently downloaded the latest plugin version of both nvim-lsp-installer and nvim-lspconfig

[X] Yes

Affected language servers
csharp_ls fautocomplete
Steps to reproduce

run LspInstallerInfo
csharp_ls and fautocomplete always not checked

Actual behavior
Not check
Expected behavior
check the new release
LspInfo
all is health

Healthcheck
nvim-lsp-installer: require("nvim-lsp-installer.health").check()
========================================================================
## nvim-lsp-installer report
  - OK: neovim version >= 0.6.0
  - WARNING: **Ruby**: not available
  - WARNING: **RubyGem**: not available
  - WARNING: **Composer**: not available
  - WARNING: **PHP**: not available
  - OK: **tar**: `tar (GNU tar) 1.34`
  - OK: **gzip**: `gzip 1.11`
  - OK: **curl**: `curl 7.82.0 (x86_64-pc-linux-gnu) libcurl/7.82.0 OpenSSL/1.1.1m zlib/1.2.11 brotli/1.0.9 zstd/1.5.2 libidn2/2.3.2 libpsl/0.21.1 (+libidn2/2.3.0) libssh2/1.10.0 nghttp2/1.47.0`
  - OK: **wget**: `GNU Wget 1.21.2 built on linux-gnu.`
  - OK: **julia**: `julia version 1.7.2`
  - OK: **python3**: `Python 3.10.2`
  - OK: **node**: `v17.6.0`
  - OK: **Go**: `go version go1.17.8 linux/amd64`
  - OK: **bash**: `GNU bash, version 5.1.16(1)-release (x86_64-pc-linux-gnu)`
  - OK: **sh**: `Ok`
  - OK: **java**: `Ok`
  - OK: **npm**: `8.5.3`
  - OK: **javac**: `javac 17.0.3`
  - OK: **pip3**: `pip 21.0 from /usr/lib/python3.10/site-packages/pip (python 3.10)`

Screenshots or recordings
No response

Hello! This is because these servers (csharp_ls, fsautocomplete) are installed via dotnet tool. It does not seem like dotnet provides any ability to check which version has been installed, I'm happy to be corrected though. The vast majority of servers supported by nvim-lsp-installer will support checking its version


Parallel log spew for events which are also reported back to the SDK user

We have very little depth of detail in the features spec around what should (or should not) be logged by SDKs.
We cover where under TO3c logHandler and RSC2.
We offer classic, coarse granularity of choice under TO3b logLevel and RSC3.
There are lots of scenarios where there is an event which might be interesting to someone debugging an application reliant upon our SDK, especially one of our support team looking at logs received from a customer, that it is helpful for us to emit that event to the log handler (whether that log handler was explicitly supplied by the customer, or is the default log handler - e.g. STDOUT).
Our default SDK behaviour, albeit not formally documented (as far as I can tell), is to log key events (whether indicative of success or failure) even when we then go on to tell the application code via formal API as well. Hence why I describe this as 'parallel log spew'.
Application developers (our users) will often have strong opinions about how they handle events and in what manner they are logged or reported elsewhere. There are many reasons why such strong opinions may exist - both relating to debugging in development and how their application runs in production on their customer devices (e.g. Android LogCat volume/noise). We should consider these strong opinions from our users in situations where the event that our SDK is emitting to the log is also being reported back to the application code via formal API (e.g. via either synchronous method throw or error return value; or callback of some sort). If the application developer takes the natural choice to write their own log messages as a result of handling those formal APIs they they are very likely to get event reported twice in their logs - which can be just as confusing when debugging (for the user and us in receipt of logs).
I equally recognise that there are also plenty of application developers who do not have such strong opinions or are perhaps less strong at debugging their own code, let alone taking into account SDK issues. For those developers it is perhaps helpful, in respect of how we can support them remotely, that our SDKs default to over/parallel logging. Sometimes it's as simple as the application developer not having used our formal API correctly and perhaps not listening to thrown exceptions, logging them to void or not handling callbacks. Let's not forget the ever faithful customer support first port of call: "switch logs to verbose". 🤓
The outcome of working on this issue could be as simple as documenting what our default SDK behaviour is, accepting this as a behaviour exhibited by Ably SDKs and moving on... However, I think we can do better here and would like to explore whether we can offer an API via client options that would allow the application developer to disable parallel log spew emissions from the SDK, if they are confident that their code is correctly and sufficiently using the formal APIs. This would be a new SDK feature.
Creation of this issue was triggered by https://github.com/ably/ably-asset-tracking-android/pull/813#pullrequestreview-1184716102. There is also this Slack thread (internal).
Relates to:

#14
#40
#60

Application developers (our users) will often have strong opinions about how they handle events and in what manner they are logged or reported elsewhere. There are many reasons why such strong opinions may exist - both relating to debugging in development and how their application runs in production on their customer devices (e.g. Android LogCat volume/noise). We should consider these strong opinions from our users in situations where the event that our SDK is emitting to the log is also being reported back to the application code via formal API

I agree with this. Ultimately, if they want to exercise control over what happens to the logs, then they can do this via logHandler etc. However, there are equally situations where the application developer consuming our library isn't actually the same party who has an opinion about the management of logs - especially for server code for example (where the developer who wrote the code isn't the same as the SRE who now has to operate it, or where the developer writing the code can't anticipate what issues might arise in the future, and so can't realistically make reliable filtering decisions on what gets logged and what doesn't. The other situation that we see is where the code that consumes our code is itself a library, and some other party eventually is the one that wants to control the log output. So the recipient of an error indication from an API is definitely not always exclusively the party who wants to observe those issues in fielded systems.
We are expected, and commonly asked, to help users of the service to help work out what is happening when they encounter an anomaly. For this, we really need to see relevant logs, and we have a right to decide what those contain, and these need to be consistent and comprehensive, so that we can provide that support. Of course ultimately the developer consuming the library is free to discard the logs, and the level of support that they can expect in that case is lower.
So I agree a more overt and explicit policy should be stated. But logging is there for a reason, and we (and others) legitimately have a need to be able to observe relevant events (errors and others), aside from the immediate needs of the direct consumer of the API.


Docs for how to create a module with composer

https://docs.silverstripe.org/en/4/developer_guides/extending/modules/ and https://docs.silverstripe.org/en/4/developer_guides/extending/how_tos/publish_a_module/ don't describe how to actually create a module. With composer, there's a bit of a chicken-and-egg situation: You need to have a module to install it via composer on your site, get autoloading going, and then start developing.
Given that we want people to create modules for SilverStripe, this should be far more obvious.

Relevant: https://github.com/silverstripe/silverstripe-module

Not sure about others, but I frequently use https://github.com/franzliedke/studio now for getting a module up and running. It’s similar to adding in a “path” repo in your root composer.json, it just does it automatically for you.


OrderType is null after deserializing in GetOrder

In Bittrex.Api.Client.Models.HistoricAccountOrder the field name is "OrderType", but inside the json response from the exchange it's just "Type".
For this reason JsonConvert.DeserializeObject doesn't assign the value of this field.
(line 306, BittrexClient.cs)
I renamed "OrderType" to "Type" in HistoricAccountOrder and it worked!

Thanks. This should be fixed now thanks to grepsy.


cx aes: fix some warnings for the OSU

Description
Fix some warnings needed for OSU build
Changes include

[X] Bugfix (non-breaking change that solves an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)
[ ] Tests
[ ] Documentation
[ ] Other (for changes that might not fit in any category)

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

Comparison is base (e050ce5) 59.47% compared to head (ac046e0) 59.47%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #470   +/-   ##
=======================================
  Coverage   59.47%   59.47%
=======================================
  Files          12       12
  Lines        1683     1683
=======================================
  Hits         1001     1001
  Misses        682      682

Flag
Coverage Δ

unittests
59.47% <ø> (ø)

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

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


Test loggers with NOTSET level having the parent level changed after instantiation

Assert that:

You can create a logger with level NOTSET
You can have a parent with a high level, e.g. WARNING
You can raise a DEBUG message that is not handled
You can change the level of the parent to DEBUG
You can raise a DEBUG message in the child that IS handled

I might be missing something but even the standard library logging doesn't assert this:
def test_case():
    logger = logging.Logger("test")
    logger.parent = logging.Logger("parent", logging.WARNING)

    stream = io.StringIO()
    handler = logging.StreamHandler(stream)
    logger.addHandler(handler)

    logger.log(logging.DEBUG, "Hello World")
    assert stream.getvalue() == ""

    logger.parent.setLevel(logging.DEBUG)

    logger.log(logging.DEBUG, "Hello World")
    assert stream.getvalue() == "Hello World\n"  # Failing

And then the test with picologging has the same behaviour and does not pass either.

I've actually added the suggested situation in my latest PR, https://github.com/microsoft/picologging/pull/108/files#diff-7c31002b037ae4bc2888350bb25d9b9621c6aff7e98d6e6871a8a649d088e27bR516
I changed the test to use getLogger and dots in the name for parent setting, as the logging library does not encourage directly using the constructor or directly setting the parent. With those changes, the test passes in stdlib and my branch.

Change merged!


Make the Future.join tryComplete function usable outside of the Future context

We removed Promise in favor of Future.join(...) and made the tryComplete function accessible in a managed context only.
We have an example that might show the necessity of making the tryComplete function accessible from the outside (like Promise did before): Concurrent recursive function memoization.
The current API:
Future.join(tryComplete -> {
    final boolean isCompleted;
    if (ok()) {
         isCompleted = tryComplete.test(Try.success("ok"));
    } else {
         isCompleted = tryComplete.test(Try.failure(new Error()));
    }
    logger.log(isCompleted);
}

Current possibility to externalize tryComplete:
AtomicReference<CheckedConsumer<Predicate<Try<? extends T>>>> promise = ...;
Future<?> future = Future.join(tryComplete -> promise.set(tryComplete));

Now we take a look, if this is sufficient to re-implement #2096 (e.g. using a data structure instead of AtomicReference, it was only an example).
The solution to add more flexibility is that tryComplete returns the self reference to the underlying Future instead of a boolean complete status.

Our Future implementation will be dropped. There will be no Vavr 1.0 module vavr-concurrent.
Reason: Implementing our own concurrency library is too error-prone, concurrency is a hard problem. There are many(!) alternatives, first of all Java's native CompletableFuture


Add support for custom CSRs

Tracking issue for adding custom CSR support to RISCV-DV.

#657 adds support for custom CSRs to riscv-dv.
A virtual hook function init_custom_csr(...) is exposed to the user to add any specific setup routines with the core's custom CSRs within the <corename>_asm_program_gen.sv extended class in the user's dv environment.
User must also fill out a bit [11:0] custom_csr[] array defined in the appropriate riscv_core_setting.sv file  to make the generator aware of the addresses of any custom CSRs.
From the riscv-dv side there is not much else work that needs to be done here, as it is up to the user to initialize their custom CSR implementations however befits the RTL implementation - this issue will be marked as closed, but will be re-opened should any glaring issue come up.


maximum timing snap precision isn't precise enough

i need 1/16

also it's weird that i can't even type it in.  even if i type 16 it resets to 1/12

finally increased the max resolution in e3bd903faa406fd4b1bcf426d4284098bfbf268d.
it does affect the scoring... i think, but from a mapping standpoint if you find yourself needing to go below even 1/16ths it's possible that the more fitting option would be to double the BPM and compensate for it via the note spacing and the time signature (number of beats per white line)


Am I the only one getting his Custom effects deleted ?

Describe the bug
My Customs effects keep getting deleted
To Reproduce
Steps to reproduce the behavior:
I got no idea of how this happen, but sometimes when I come back they're just no where to be found.
Expected behavior
Well to not delete every custom things i've made ?
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
I am hosting on Oracle Cloud.
We've tried with the community to reproduce the behavior by creating a new world with only that module, creating an effect and trying to make it happens. Nothing. I created another world and put every modules that I had on my original one as suggested by the community, but nothing happened either.
My version of Foundry is : 9.269
I'm using : Dnd5e 1.6.3
Module version : Last one to date.
Found the issue, and I don't understand why it's working this way.

So it seems where you create for the first time a custom effect and ITEM names Custom Convenient Effects is created and if this one is deleted, everything you're created is gone too. Why is this working this way ? Or not even written in the description of the module ?
So I tried to reinstall clearly the module and this kept appearing, however when I tried to move it in a "Not delete" folder I have, it disappeared, leaving nothing but stuggle behind it. I don't know if it's fixed but for now it seems to work correctly ?
The issue here was that I, somehow, managed to delete that file that serve as a database.
Ah yes, when I moved the item somewhere it disappeared like it should have from the beginning, So might have been just a random bug.

Foundry defines ActiveEffects as things that can only exist on an Item or a Character. Thus, this module creates an item behind the scenes and adds all of the effects you create to it. It then hides this item from your list of items to not clutter anything up. It's not explicitly stated that it's doing this anywhere because it shouldn't matter from the user perspective.
Somewhere along the line, your original Custom Convenient Effect item was deleted. I don't know how you did that or why it might have happened... Also, I thought I had code around this to handle when it happens, so I'll double check it. But yeah, I have no other answer for you because I've never seen it happen and you're the only one who has reported it.
My only real suggestion at the moment would be to delete the entry in the settings.db for the world about Convenient Effects to reset all of Convenient Effect's settings. Hopefully, that resets the item ID it was using to determine what item to use to store effects.
I'll close this if I find that deleting the custom effect is handled correctly.

Could it happen with a macro deleting items maybe not in folders or something like that?

Foundry defines ActiveEffects as things that can only exist on an Item or a Character. Thus, this module creates an item behind the scenes and adds all of the effects you create to it. It then hides this item from your list of items to not clutter anything up. It's not explicitly stated that it's doing this anywhere because it shouldn't matter from the user perspective.
Somewhere along the line, your original Custom Convenient Effect item was deleted. I don't know how you did that or why it might have happened... Also, I thought I had code around this to handle when it happens, so I'll double check it. But yeah, I have no other answer for you because I've never seen it happen and you're the only one who has reported it.
My only real suggestion at the moment would be to delete the entry in the settings.db for the world about Convenient Effects to reset all of Convenient Effect's settings. Hopefully, that resets the item ID it was using to determine what item to use to store effects.
I'll close this if I find that deleting the custom effect is handled correctly.

You can close it, everything is working fine now and I got no more weird items going around in my folders.

Could it happen with a macro deleting items maybe not in folders or something like that?

It might be the case. I did a clean up before this actually happen, didn't expect it to be the issue. Well, it was all my fault at first, i'll stop tweaking weird stuff again haha


Fill the delfin driver template matrix data

Issue/Feature Description:
Delfin has heterogeneous storage driver support matrix template data has yet to be filled, below are the supported drivers

[ ] Dell Emc VMAX
[ ] Dell Emc Unity
[ ] Hitachi VSP
[ ] HP 3PAR StoreServ
[ ] Huawei OceanStor
[ ] IBM Storwize

Why this issue to fixed / feature is needed(give scenarios or use cases):
How to reproduce, in case of a bug:
Other Notes /  Environment Information: (Please give the env information, log link or any useful information for this issue)

@Shruthi-1MN can we have a summary of these activities and results/docs, here? Easy to trace and support.

Filled VMAX and OceanStor driver matrix data in the PR #140.


--verbose doesn't show anything

I'm on 1.5.2
If I use autoxtrabackup -v -l DEBUG --backup  it fills the console with information
But if I remove the -l DEBUG part there is nothing
If I change DEBUG to CRITIAL/INFO etc, there is nothing either !!!

ok but it's confusing :D maybe make it simpler until future implementation :P

:)) haha) just skip it. use -l DEBUG and that's it)

??? no -l DEBUG prints nothing to the console
Imo, there is a need to unify command names etc

Okay sorry, -v -l DEBUG )

I have changed the default log to DEBUG to make it clear that it will show necessary messages.


Geojson name on file listing does not fit in the box

(When using the Box connector, which is the only one right now that can retrieve GeoJSON files)
Probably not worthy, but at least to take into account if a similar file listing is to be implemented/is implemented.
cc @xavijam

it is worth it, thanks @iriberri :).


JRTB-13: implement deployment process

Implement deployment process using docker image.
Acceptance criteria:

dockerfile added to the project
deployment process with providing environment variables is added to the project

Implemented in https://github.com/RobTakh/Javarush-Telegrambot-part2/pull/2


Update README.md

Needed to add the S3RVER key to the S3 config when working offline, I believe its required since all the examples also use it

@purefan
Thanks for your great work! I merged it.


Changed Dockerfile to reflect helm support for 1.7

Updated/commented out some ENV vars based on current helm support for 1.7

Also, general question: Do we still use these ENV vars?
ENV     K8S_VERSION=v1.7.1
ENV     K8S_HELM_VERSION=v2.5.1

need to add something to this PR to set the helm_override to not use helm.  will probably want to edit the sed command in 'fetch-credentials.sh' to also include a newline "\n" and then 'helm_override: true' or something like that.


Batch job table cleanup documentation update

This PR was created in the public repo which is not allowed for employees. This PR has been closed and replaced with this PR: https://github.com/MicrosoftDocs/Dynamics-365-Operations/pull/17381


Add labels and fix argo

Revises: #1236
Fixes error related to null labels trying to get added to a dictionary in argo workflows.

fyi, I test this with a vanilla argo-workflows create command and everything was fine.

As an additional note, the newly added Sensors for argo-workflows from #1271 would also require labeling. If you have time, this could be done in the scope of this PR.

I personally would like to get this merged as quickly as possible.  I can look into this but I haven't used Sensors so I would also need to look into that.  If it's easy, I can do it, but otherwise I would like to create a new PR.

We can do another PR for the sensors, they are just another template that is being compiled as part of argo-workflows create so should be straightforward to add the labels support for those. I might have time today to look into it.
Could you resolve the conflicts with this branch so it is ready for merging?

We can do another PR for the sensors, they are just another template that is being compiled as part of argo-workflows create so should be straightforward to add the labels support for those. I might have time today to look into it.
Could you resolve the conflicts with this branch so it is ready for merging?

Conflicts resolved.  Thanks for your help.

Tested the latest with:
METAFLOW_KUBERNETES_LABEL="" python hello.py run --with kubernetes
METAFLOW_KUBERNETES_LABEL="label1=val1" python hello.py run --with kubernetes
METAFLOW_KUBERNETES_LABEL="label1=val1,label2=val2" python hello.py run --with kubernetes
METAFLOW_KUBERNETES_LABEL="label=creation" python hello.py argo-workflows create
METAFLOW_KUBERNETES_LABEL="label=trigger" python hello.py argo-workflows trigger  # label has no effect here, the creation label is used
METAFLOW_KUBERNETES_LABEL="" python hello.py argo-workflows create
python hello.py run --with kubernetes:labels=l1=v1  # this also fails as expected

Sensor Labels: dhpollack/metaflow#1


Implement a UDP client

closes #4647

Looks good to me.
Should NewClient now be renamed to NewHTTPClient? If this v2 client code is not yet frozen, I think you should consider that. It feels more Go-like.

@corylanou @jwilder @otoolep My 1st precision implementation wasn't working right, so I've added a function to models.Point, RoundedString(d time.Duration) string. Main use case here is so that we can write UDP points with precision.

@otoolep That's a good question, maybe I could have NewClient log a warning DEPRECATED message, and call NewHTTPClient. What do you think?

@sparrc -- have we made any promises re V2 yet?

@otoolep I'm not sure what you mean by promises, but we do have this note telling people to use it already: https://github.com/influxdb/influxdb/tree/master/client#description

OK, that is what I mean.
OK, I would scrap the idea of renaming and leave NewClient as is, and add NewUDPClient like you are doing. A HTTP client is what most people should be using anyway.

@otoolep I made some significant changes based on what we were talking about in Slack. Now the UDP client will write a UDP packet before we hit the maximum size (64kb) and then reset the buffer

@otoolep I've pushed the changes you suggested about the buffer size before writing, and I also made changes to make the godoc examples look better, as well as an update to the README

Don't forget the CHANGELOG.

I think this looks fine. I do think you need to actually test that packets sent using this code actually works -- code a throwaway client that uses UDP and sends data to an InfluxDB service, and check that the data is in InfluxDB. Since it's UDP, this unit testing doesn't guarantee it is working. Just to be sure.
Once you do that, +1


Entering deep sleep breaks USB JTAG

It looks like stalling the cpu (e.g. by esp-backtrace's halt-cpu) can trigger the same issue.

Current state: replugging USB fixes JTAG for that power cycle.
It should be possible to not power down JTAG and related stuff if the board goes to sleep while plugged in/connected.

Fixed by https://github.com/esp-rs/esp-hal/pull/660


fix annotation did not added

Summary
Related issue(s)
Fixes #
Tests

[ ] Unit/function tests have been added and incorporated into make unit-tests.
[ ] Integration tests have been added and incorporated into make integration-test.
[ ] E2E tests have been added and incorporated into make e2e-test-all.
[ ] List other manual tests you have done.

/cc @clyang82

/cherry-pick release-2.12


bump modin version

Fixes https://github.com/bigdatagenomics/mango/issues/

Coverage remained the same at 80.27% when pulling e526ffd0846b5fb117f0c51cbe518efc14d99272 on akmorrow13:bump-modin into 4cfcd2a029c49f132cd8151c44bc18999e8c9e00 on bigdatagenomics:master.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/mango-prb/919/
Test PASSed.


Deny rust_2018_idioms and unused_qualifications in crate roots

Suggestion
When using a local path reference to the windows-rs crate, from a project that denies a bunch of lints globally (via rustflags in .cargo/config.toml until workspace lints get implicit/forced inheritance), there are quite a few warnings from the windows crate which would be trivial and neat to resolve.
Most/all originate from the rust_2018_idioms lint (and a few more like unused_qualifications, I think https://github.com/ash-rs/ash/blob/e6d80badc389d94e2a747f442e5ed4189b66d7d3/ash/src/lib.rs#L1-L8 might be a good starting point). This lint isn't enabled by default despite windows already setting edition = "2021", and certain things like extern crate might be nice to clear out.
Some samples:
warning: unused extern crate
  --> windows-rs\crates\libs\windows\src\lib.rs:11:1
   |
11 | extern crate self as windows;
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove it
   |
   = note: `-W unused-extern-crates` implied by `-W rust-2018-idioms`
   = help: to override `-W rust-2018-idioms` add `#[allow(unused_extern_crates)]`

warning: unnecessary qualification
  --> windows-rs\crates\libs\core\src\strings\bstr.rs:87:21
   |
87 |     fn from(value: &std::string::String) -> Self {
   |                     ^^^^^^^^^^^^^^^^^^^
   |
help: remove the unnecessary path segments
   |
87 -     fn from(value: &std::string::String) -> Self {
87 +     fn from(value: &String) -> Self {
   |

warning: hidden lifetime parameters in types are deprecated
   --> windows-rs\crates\libs\implement\src\lib.rs:241:34
    |
241 |     fn parse(cursor: syn::parse::ParseStream) -> syn::parse::Result<Self> {
    |                      ------------^^^^^^^^^^^
    |                      |
    |                      expected lifetime parameter
    |
    = note: `-W elided-lifetimes-in-paths` implied by `-W rust-2018-idioms`
    = help: to override `-W rust-2018-idioms` add `#[allow(elided_lifetimes_in_paths)]`
help: indicate the anonymous lifetime
    |
241 |     fn parse(cursor: syn::parse::ParseStream<'_>) -> syn::parse::Result<Self> {
    |                                             ++++

Happy to tighten this up if there's a way to do so without adding more attributes to the various crates. Perhaps with a clippy config file, like rustfmt.toml, or through the yml workflows.
Note that the warning about the "unused extern crate" is a false positive and it is in fact needed.

Happy to tighten this up if there's a way to do so without adding more attributes to the various crates. Perhaps with a clippy config file, like rustfmt.toml, or through the yml workflows.

Exactly that exists in the latest 1.74 Rust release: https://blog.rust-lang.org/2023/11/16/Rust-1.74.0.html#lint-configuration-through-cargo
It has one fatal flaw, which is having to opt-in every crate manually. That might work for windows-rs as there are not many crates and it's relatively easy to audit (and you probably don't care about having strict linting on most crates, like tools and tests).

Lints set in a workspace will be inherited by every crate.

@ChrisDenton:
https://github.com/rust-lang/cargo/issues/12208
https://github.com/rust-lang/cargo/issues/12984

Right you need to add workspace = true to the crate's lints table but that's more annoying than a "fatal flaw", imho.

For a workspace split across over 200 crates, with new ones being added on a weekly basis?

Tbh, in that case I'd be minded to automate it. There's probably a bunch of boiler plate stuff you need in any case.
But I admit I find that to be an unusual workflow, unless you're doing some kind of monorepo-like thing.

Thanks, workspace lints works nicely - I'll roll that into #2740.

Both rust_2018_idioms and unused_qualifications are now enforced as of #2741 and #2743.


Real-world example: Odd multiline array?

Here is an excerpt from a brunch config:
({
  processors: [
    require("autoprefixer", {
      browsers: ["> 1%", "last 2 versions", "ie >= 11", "Firefox ESR"]
    }),
    require("postcss-url")({
      url: url =>
        url.startsWith("/") || /^[a-z]+:/.test(url) ? url : `/static/${url}`
    })
  ]
})

Output:
({
  processors: [require("autoprefixer", {
      browsers: ["> 1%", "last 2 versions", "ie >= 11", "Firefox ESR"]
    }), require("postcss-url")({
      url: url =>
        url.startsWith("/") || /^[a-z]+:/.test(url) ? url : `/static/${url}`
    })]
});

Does this look the way we want it to? I'd expect something more similar to the input.
https://jlongster.github.io/prettier/#{"content"%3A"({\n  processors%3A [\n    require(\"autoprefixer\"%2C {\n      browsers%3A [\"> 1%25\"%2C \"last 2 versions\"%2C \"ie >%3D 11\"%2C \"Firefox ESR\"]\n    })%2C\n    require(\"postcss-url\")({\n      url%3A url %3D>\n        url.startsWith(\"%2F\") || %2F^[a-z]%2B%3A%2F.test(url) %3F url %3A `%2Fstatic%2F%24{url}`\n    })%5Cn%20%20%5D%5Cn%7D)%22%2C%22options%22%3A%7B%22printWidth%22%3A80%2C%22tabWidth%22%3A2%2C%22singleQuote%22%3Afalse%2C%22trailingComma%22%3Afalse%2C%22bracketSpacing%22%3Atrue%2C%22doc%22%3Afalse%7D%7D

The issue is that both values are calls with expansion as last argument which is treated as fitting one line. We need to manually break them by doing
shouldBreak: printed.some(willBreak)


Training with pytorch 2.0

PyTorch 2.0 is now available which now includes two major things that may be useful for faster training:

TorchDynamo, which can be used by calling torch.compile(model)
FLASH attention, by using torch.backends.cuda.enable_flash_sdp(True).

I have built k2 with the new PyTorch (it requires minimum CUDA 11.7), and am trying to see if we can leverage the speed-ups. I am trying to train a small streaming Zipformer-transducer in this setup.
Flash attention seems to be working out-of-the-box without any changes. TorchDynamo, on the other hand, has some issues. It is clear that any custom k2 objects (such as RaggedTensor) cannot be "compiled", so I do the following: instead of torch.compile(model), I only do torch.compile(model.encoder) since the encoder does not contain any k2 objects (which are mainly used in the decoder and loss computation. I get the following error:
Traceback (most recent call last):
  File "pruned_transducer_stateless7_streaming/train.py", line 1268, in <module>
    main()
  File "pruned_transducer_stateless7_streaming/train.py", line 1261, in main
    run(rank=0, world_size=1, args=args)
  File "pruned_transducer_stateless7_streaming/train.py", line 1139, in run
    train_one_epoch(
  File "pruned_transducer_stateless7_streaming/train.py", line 834, in train_one_epoch
    scaler.scale(loss).backward()
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_tensor.py", line 487, in backward
    torch.autograd.backward(
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/autograd/__init__.py", line 200, in backward
    Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/autograd/function.py", line 274, in apply
    return user_fn(self, *args)
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_functorch/aot_autograd.py", line 2348, in backward
    out = call_compiled_backward()
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_functorch/aot_autograd.py", line 2319, in call_compiled_backward
    CompiledFunction.compiled_bw = aot_config.bw_compiler(
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_dynamo/backends/common.py", line 38, in _wrapped_bw_compiler
    return eval_frame.disable(eval_frame.disable(bw_compiler)(*args, **kwargs))
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_dynamo/eval_frame.py", line 209, in _fn
    return fn(*args, **kwargs)
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_dynamo/utils.py", line 163, in time_wrapper
    r = func(*args, **kwargs)
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_inductor/compile_fx.py", line 441, in bw_compiler
    return inner_compile(
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_dynamo/debug_utils.py", line 595, in debug_wrapper
    compiled_fn = compiler_fn(gm, example_inputs)
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_inductor/debug.py", line 239, in inner
    return fn(*args, **kwargs)
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/contextlib.py", line 75, in inner
    return func(*args, **kwds)
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_inductor/compile_fx.py", line 176, in compile_fx_inner
    graph.run(*example_inputs)
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_dynamo/utils.py", line 163, in time_wrapper
    r = func(*args, **kwargs)
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_inductor/graph.py", line 194, in run
    return super().run(*args)
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/fx/interpreter.py", line 136, in run
    self.env[node] = self.run_node(node)
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_inductor/graph.py", line 466, in run_node
    if isinstance(result, TensorBox) and result.has_exceeded_max_reads():
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_inductor/ir.py", line 3932, in has_exceeded_max_reads
    self.num_reads() > config.realize_acc_reads_threshold
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_inductor/utils.py", line 212, in wrapper
    setattr(self, key, fn(self))
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_inductor/ir.py", line 3970, in num_reads
    read_writes = ComputedBuffer(
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_inductor/utils.py", line 212, in wrapper
    setattr(self, key, fn(self))
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_inductor/ir.py", line 2095, in get_read_writes
    return extract_read_writes(
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_inductor/dependencies.py", line 300, in extract_read_writes
    fn(*args)
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_inductor/ir.py", line 434, in store_output
    return ops.store(output_name, indexer(vars), self.inner_fn(vars))
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_inductor/lowering.py", line 1593, in inner_fn
    src_val = ops.masked(
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_inductor/virtualized.py", line 104, in inner
    line = getattr(self.parent_handler, name)(*args, **kwargs)
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_inductor/virtualized.py", line 75, in masked
    return f"masked({mask}, {body()}, {other})"
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_inductor/lowering.py", line 1595, in <lambda>
    lambda: src_loader(src_idx),
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_inductor/ir.py", line 762, in fn
    return functools.reduce(
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_inductor/ir.py", line 765, in <genexpr>
    value_fn(index, rindex)
  File "/home/hltcoe/draj/.conda/envs/torch2/lib/python3.8/site-packages/torch/_inductor/lowering.py", line 3398, in loader
    assert all(index[i] == 0 for i in reduced_idx)
AssertionError: While executing %slice_scatter : [#users=1] = call_function[target=torch.ops.aten.slice_scatter.default](args = (%full, %sum_3, 0, 864,<PHONE_NUMBER>854775807), kwargs = {})
Original traceback:
  File "/exp/draj/mini_scale_2022/icefall/egs/librispeech/ASR/pruned_transducer_stateless7_streaming/zipformer.py", line 1548, in forward
    src_extra = src[src.shape[0] - 1 :].expand(pad, src.shape[1], src.shape[2])

I will try to debug this a bit more, but just wanted to open this issue here in case someone has already come across this.

I could avoid the error by changing that line to:
src_extra = src[-1, ...].unsqueeze(dim=0).expand(pad, -1, -1)

I suppose TorchDynamo doesn't allow indexing with shapes. However, I keep getting OOM during compilation even after decreasing batch size significantly.

However, I think even without torch.compile(), just using the FLASH attention (by setting torch.backends.cuda.enable_flash_sdp(True)) improves memory efficiency. For LibriSpeech training setup (described in my original post), I am able to increase batch size from 500 to 800 on RTX GPUs.

After digging deeper, it seems flash attention was not being used after all. Here is a summary:

The scaled_dot_product_attention() in PyTorch 2.0 contains 3 implementations internally: FLASH attention, efficient attention, and a C++ attention of the regular SDP attention.
The first two implementations provide memory savings, but they do not yet work when an arbitrary attention mask is provided.
In this case, the function falls back on the C++ implementation I suppose.

It seems support for attention masks may be available soon. But still, it would not be usable directly in the Zipformer encoder. This is because Flash attention does not explicitly compute attention weights, which is require for the Zipformer (since it re-uses these weights in the second self-attention in the block). I think it should be fine to just re-compute the SDP attention for a second time with the fused kernel, since it is apparently quite fast (mainly due to much less read/write between the HBM and SRAM). Anyway, I will try this once flash attention supports attention masks.

Yeah I was talking about their implementation in PyTorch, not in icefall.


Bugs Solved

PR - UI Bugs Solved

Fixes Issue #42

Changes Made

[X] Updated the image sending feature to ensure it looks good on both light and dark screens.
[X] Resolved the issue of slow-loading icons for a smoother user experience.
[X] Set the body height to 100vh, ensuring a fully structured and responsive layout.

This pr is ready for review .. Kindly review it .. Changes are always welcomed :)
Ping @pooranjoyb @sristy17
Thank you 👍

@mahenoorsalat resolve the conflicts

Hey @mahenoorsalat I guess you haven't pulled the latest changes from master branch.
Please make sure you do the same.
Also please note.

Most probably you also fixed the bugs for cards (in dark-mode)
However they aren;t being reflected as due to some unwanted conflicts in the src coded. I'd suggest to create a fresh PR with all the changes from the master branch.

We'll help if needed.


healthcheck db required write permission?

Hello,
I have RDS with only read permission and i am checking health-check db and i found my status was  failed .so health-check db required the write permission?

Hi @nishit539687 yes it does. But you could easily write you own plugin inheriting from the existing database health check that doesn't do a write but only a read action.

Thank you @codingjoe .


prepare release

In this issue, we'll update documentation for a new release. Version number pending.

[ ]  Update home page
[X]  Update version statements in ado files (done in https://github.com/JMSLab/xtevent/commit/6408fdc18b5af9899a83af33a012304a1f85f459)
[X]  Check examples in home page (note changes to cohort and control_cohort syntax in #179 (done in https://github.com/JMSLab/xtevent/commit/3553273802d820e9dcf098eb24e2ac1567167e31)
[ ]  Check examples in test file and run
[ ]  Check examples in help files and run
[ ]  Check that help files display properly
[ ]  Update acknowledgments
[ ] Update install.txt and release.txt
[ ] Update version dependencies in dependency.do

Dear Jorge Eduardo Perez Perez,
I hope this message finds you well.
My name is Pan xy, and I am currently a Ph.D. student at Tsinghua University. I am encountering an issue when setting norm() to the first stage, norm(0). At the end of the process, I receive an r(198) error indicating that "_k_eq_m0 not found." Could you please advise me on how to address this situation?
Thank you very much for your help.
Best regards,
Pan xy

Dear Pan.
Thank you for your email. This was a previous issue in xtevent that was
fixed in issue #186
Error when normalizing 0 coefficient · Issue #186 · JMSLab/xtevent
(github.com) https://github.com/JMSLab/xtevent/issues/186
Can you please install the latest development version of xtevent and let us
know if the problem persists?
Thank you,
JPP
To install the last version in this repository, use the github command:
https://github.com/JMSLab/xtevent#to-install-the-last-version-in-this-repository-use-the-github-command
First, install the github command:
net install github, from("https://haghish.github.io/github/")
Then execute:
cap github uninstall xtevent
github install JMSLab/xtevent
On Thu, Jun 20, 2024 at 3:35 AM pxygithub2024 @.***>
wrote:

Dear Jorge Eduardo Perez Perez,
I hope this message finds you well.
My name is Pan xy, and I am currently a Ph.D. student at Tsinghua
University. I am encountering an issue when setting norm() to the first
stage, norm(0). At the end of the process, I receive an r(198) error
indicating that "_k_eq_m0 not found." Could you please advise me on how to
address this situation?
Thank you very much for your help.
Best regards,
Pan xy
—
Reply to this email directly, view it on GitHub
https://github.com/JMSLab/xtevent/issues/184#issuecomment-2180243416,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEBUYUDG4VC5YTDHJZEDCNDZIKO6BAVCNFSM6AAAAABGTU34DWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBQGI2DGNBRGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Dear Mr.  Perez.
I followed the steps you mentioned and the error displayed is as follows:
. net install xtevent, from("
https://raw.githubusercontent.com/JMSLab/xtevent/master")
checking xtevent consistency and verifying not already installed...
could not copy https://raw.githubusercontent.com/JMSLab/xtevent/master/.
/xtevent/_eventivstatic.ado
(no action taken)
I download the files from your github,and cover the old files in
my personal directory, But still don't work.Maybe I am doing something
wrong?
Thank you very much for your help.
Pan xy
Jorge Eduardo Perez Perez @.***> 于2024年6月25日周二 05:05写道：

Dear Pan.
Thank you for your email. This was a previous issue in xtevent that was
fixed in issue #186
Error when normalizing 0 coefficient · Issue #186 · JMSLab/xtevent
(github.com) https://github.com/JMSLab/xtevent/issues/186
Can you please install the latest development version of xtevent and let
us
know if the problem persists?
Thank you,
JPP
To install the last version in this repository, use the github command:
<
https://github.com/JMSLab/xtevent#to-install-the-last-version-in-this-repository-use-the-github-command>
First, install the github command:
net install github, from("https://haghish.github.io/github/")
Then execute:
cap github uninstall xtevent
github install JMSLab/xtevent
On Thu, Jun 20, 2024 at 3:35 AM pxygithub2024 @.***>
wrote:

Dear Jorge Eduardo Perez Perez,
I hope this message finds you well.
My name is Pan xy, and I am currently a Ph.D. student at Tsinghua
University. I am encountering an issue when setting norm() to the first
stage, norm(0). At the end of the process, I receive an r(198) error
indicating that "_k_eq_m0 not found." Could you please advise me on how
to
address this situation?
Thank you very much for your help.
Best regards,
Pan xy
—
Reply to this email directly, view it on GitHub
https://github.com/JMSLab/xtevent/issues/184#issuecomment-2180243416,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AEBUYUDG4VC5YTDHJZEDCNDZIKO6BAVCNFSM6AAAAABGTU34DWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBQGI2DGNBRGY>

.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/JMSLab/xtevent/issues/184#issuecomment-2187400471,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHLZKBRI6VK37LK6FVYGIWTZJCCZVAVCNFSM6AAAAABGTU34DWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBXGQYDANBXGE
.
You are receiving this because you commented.Message ID:
@.***>

Dear Mr.  Perez.
I've solved the problem by following the steps you mentioned, it looks like
there should have been a problem with the network before, and I already had
to add the ",replace" option after the command. Thank you very much for
your help!
Pan xy
xingyu pan @.***> 于2024年6月27日周四 23:55写道：

Dear Mr.  Perez.
I followed the steps you mentioned and the error displayed is as follows:
. net install xtevent, from("
https://raw.githubusercontent.com/JMSLab/xtevent/master")
checking xtevent consistency and verifying not already installed...
could not copy https://raw.githubusercontent.com/JMSLab/xtevent/master/.
/xtevent/_eventivstatic.ado
(no action taken)
I download the files from your github,and cover the old files in
my personal directory, But still don't work.Maybe I am doing something
wrong?
Thank you very much for your help.
Pan xy
Jorge Eduardo Perez Perez @.***> 于2024年6月25日周二
05:05写道：

Dear Pan.
Thank you for your email. This was a previous issue in xtevent that was
fixed in issue #186
Error when normalizing 0 coefficient · Issue #186 · JMSLab/xtevent
(github.com) https://github.com/JMSLab/xtevent/issues/186
Can you please install the latest development version of xtevent and let
us
know if the problem persists?
Thank you,
JPP
To install the last version in this repository, use the github command:
<
https://github.com/JMSLab/xtevent#to-install-the-last-version-in-this-repository-use-the-github-command>
First, install the github command:
net install github, from("https://haghish.github.io/github/")
Then execute:
cap github uninstall xtevent
github install JMSLab/xtevent
On Thu, Jun 20, 2024 at 3:35 AM pxygithub2024 @.***>
wrote:

Dear Jorge Eduardo Perez Perez,
I hope this message finds you well.
My name is Pan xy, and I am currently a Ph.D. student at Tsinghua
University. I am encountering an issue when setting norm() to the first
stage, norm(0). At the end of the process, I receive an r(198) error
indicating that "_k_eq_m0 not found." Could you please advise me on how
to
address this situation?
Thank you very much for your help.
Best regards,
Pan xy
—
Reply to this email directly, view it on GitHub
https://github.com/JMSLab/xtevent/issues/184#issuecomment-2180243416,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AEBUYUDG4VC5YTDHJZEDCNDZIKO6BAVCNFSM6AAAAABGTU34DWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBQGI2DGNBRGY>

.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/JMSLab/xtevent/issues/184#issuecomment-2187400471,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHLZKBRI6VK37LK6FVYGIWTZJCCZVAVCNFSM6AAAAABGTU34DWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBXGQYDANBXGE
.
You are receiving this because you commented.Message ID:
@.***>

Per SemVer the new version is 3.1.0

Summary: In this issue we updated the home page and the repository for the 3.1.0 release.
Thread continues in XX


Error executing service <ServiceCall climate.set_operation_mode: operation_mode=Manual, entity_id=['climate.downstairs']>

Home Assistant release with the issue:
0.65.5

Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):
Windows

Component/platform:
https://www.home-assistant.io/components/tado/

Description of problem:
Home Assistant shows status of Tado entities but cannot control them.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):
Log Details (ERROR)
Sat Mar 31 2018 19:12:59 GMT+0100 (GMT Summer Time)

Error executing service <ServiceCall climate.set_operation_mode: operation_mode=Manual, entity_id=['climate.downstairs']>
Traceback (most recent call last):
  File "C:\Users\domwe\AppData\Local\Programs\Python\Python36\lib\site-packages\homeassistant\core.py", line 1006, in _event_to_service_call
    await service_handler.func(service_call)
  File "C:\Users\domwe\AppData\Local\Programs\Python\Python36\lib\site-packages\homeassistant\components\climate\__init__.py", line 392, in async_operation_set_service
    await climate.async_set_operation_mode(operation_mode)
  File "C:\Users\domwe\AppData\Local\Programs\Python\Python36\lib\asyncio\futures.py", line 327, in __iter__
    yield self  # This tells Task to wait for completion.
  File "C:\Users\domwe\AppData\Local\Programs\Python\Python36\lib\asyncio\tasks.py", line 250, in _wakeup
    future.result()
  File "C:\Users\domwe\AppData\Local\Programs\Python\Python36\lib\asyncio\futures.py", line 243, in result
    raise self._exception
  File "C:\Users\domwe\AppData\Local\Programs\Python\Python36\lib\concurrent\futures\thread.py", line 56, in run
    result = self.fn(*self.args, **self.kwargs)
  File "C:\Users\domwe\AppData\Local\Programs\Python\Python36\lib\site-packages\homeassistant\components\climate\tado.py", line 228, in set_operation_mode
    self._control_heating()
  File "C:\Users\domwe\AppData\Local\Programs\Python\Python36\lib\site-packages\homeassistant\components\climate\tado.py", line 357, in _control_heating
    self.zone_id, self._current_operation, self._target_temp)
  File "C:\Users\domwe\AppData\Local\Programs\Python\Python36\lib\site-packages\homeassistant\components\tado.py", line 127, in set_zone_overlay
    self.tado.setZoneOverlay(zone_id, mode, temperature, duration)
  File "C:\Users\domwe\AppData\Roaming\.homeassistant\deps\Python36\site-packages\PyTado\interface.py", line 278, in setZoneOverlay
    data = self._apiCall(cmd, "PUT", post_data, True)
  File "C:\Users\domwe\AppData\Roaming\.homeassistant\deps\Python36\site-packages\PyTado\interface.py", line 81, in _apiCall
    response = self.opener.open(req)
  File "C:\Users\domwe\AppData\Local\Programs\Python\Python36\lib\urllib\request.py", line 532, in open
    response = meth(req, response)
  File "C:\Users\domwe\AppData\Local\Programs\Python\Python36\lib\urllib\request.py", line 642, in http_response
    'http', request, response, code, msg, hdrs)
  File "C:\Users\domwe\AppData\Local\Programs\Python\Python36\lib\urllib\request.py", line 570, in error
    return self._call_chain(*args)
  File "C:\Users\domwe\AppData\Local\Programs\Python\Python36\lib\urllib\request.py", line 504, in _call_chain
    result = func(*args)
  File "C:\Users\domwe\AppData\Local\Programs\Python\Python36\lib\urllib\request.py", line 650, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 422: Unprocessable Entity

Additional information:

Duplicate of https://github.com/home-assistant/home-assistant/issues/13548.


Render from String as opposed from defining a template struct pointing to a file?

Is is possible to render directly from a String using dynamic attributes?

Doubt it. This crate is compile time based.

Although sailfish can't parse runtime template, Rendering based on compile-time string should be possible like this:
#[derive(TemplateOnce)]
#[template = "
<%= name %>
"]
struct Template {
    name: String
}


Hot reload

is there any way to implement hot reload? the development is really slow this way
Implementation notes:
https://pdm.fming.dev/latest/pyproject/pep621/#console-scripts
https://api.flutter.dev/flutter/dart-io/FileSystemEntity/watch.html
https://github.com/dart-lang/sdk/issues/30918
https://api.dart.dev/stable/2.17.6/dart-io/Process-class.html

There is a package for this called jurigged, but unfortunately, it does not seems to work, it prints in the terminal that the file was changed but did not update the UI

Can you try this solution: https://flet.dev/docs/guides/python/hot-reload/

@FeodorFitsner This watchexec works for me, it closes and reopens the application, not the perfect world but it is ok for now. I'm entering the desktop and mobile development right now (just studying), isn't there something more "perfect" for desktop hot reaload like ViteJS?

Hot reload has been just released and documented here: https://flet.dev/docs/guides/python/hot-reload

This is very helpful. Thank you for implementing.