How to invoke an instance method

When I try to invoke a method :
const CSharp = Il2Cpp.Domain.assembly("Assembly-CSharp").image;
const moneyDroper=CSharp.class("moneyDropper")
const dropMoney=moneyDroper.method("dropMoney").invoke()
I get an error :
cannot invoke a non-static method dropMoney: must be invoked throught a Il2Cpp.Object, not a Il2Cpp.Class
and when i try to initate an object instead using :
const object =moneyDroper.new().dropMoney()
i get another error saying :
TypeError: not a function
And when i try to invoke the method using :
className.method("dropMoney").invoke();
Failed to load script: script(line 14): SyntaxError: unexpected token in expression: '>'
The class I am trying to hook is that :
class moneyDropper : UnityEngine.MonoBehaviour
{
UnityEngine.GameObject[] dropPosition; // 0x18
UnityEngine.GameObject smallMoneyPrefab; // 0x20
System.Boolean canDrop; // 0x28
System.Void dropMoney(); // 0x009e9c44
System.Void stopDrop(); // 0x009e9cc4
System.Void invokedDrop(); // 0x009e9c48
System.Void throwMoney(); // 0x009e9ccc
System.Void .ctor(); // 0x009e9fcc
}
I am currently trying to attach to the address directly ,but the function might not be called .

The method you are trying to invoke is not static, hence you need an object in order to call it:
const = moneyDroper.new().method("dropMoney").invoke();

However this will (probably) have no effect - you are creating a new object that is not part of the game.
So, you need to obtain an existing moneyDropper instance.
You can do so by either:
1- Intercepting a method that has a moneyDropper as parameter or whose instance has a moneyDropper field (wiki);
2- Scan the heap to find every moneyDropper instance (wiki).

Oh okay . Thank you so much
It gives me a different error  "Abort was called " , but you are right i should try your suggestions .  I tried with the interceptor to attach to the native library and add the offset but no luck . I will read about the heap scanning code and try again with the intereptor .

Hi ,sorry for another question . I am running your _.js script instead of the index ,so i put all my code in it .  I currently have a a sytnax error on line 23  ?
Il2Cpp.perform(() => {
// 모든 dll 파일 덤프 > dump.cs 파일로 추출
//Il2Cpp.dump();
//Il2Cpp.dump("DiceClient.dll");
const CSharp = Il2Cpp.Domain.assembly("Assembly-CSharp").image;
const moneyDroper=CSharp.class("moneyDropper")
//moneyDroper.method("dropMoney").invoke()
moneyDroper.new().method("dropMoney").invoke();
const mscorlib = Il2Cpp.Domain.assembly("mscorlib").image;
const SystemType = mscorlib.class("System.Type");

// +++ heap scanning using class descriptors +++
Il2Cpp.GC.choose(SystemType).forEach((instance: Il2Cpp.Object) => {
    // instance.class.type.name == "System.Type"
	console.log("Hey")
});
// --- heap scanning using class descriptors ---

// +++ heap scanning using a memory snapshot +++
const snapshot = Il2Cpp.MemorySnapshot.capture();
snapshot.objects.filter(Il2Cpp.Filtering.IsExactly(SystemType)).forEach((instance: Il2Cpp.Object) => {
    // instance.class.type.name == "System.Type"
});
snapshot.free();
// --- heap scanning using a memory snapshot ---

const addr=ptr(CSharp.class("EVP.Wheel").fields[4])

});`
It says syntax

SyntaxError: expecting '(' specifically on Line 1 or Line 2 (depending on if you're using use strict I just learned is due - in my case - not including the --bundle option on esbuild.  I kept tweaking the package.json and tsconfig.json only to realize I accidentally stripped that argument out thinking it didn't really matter.


Revert "Bump actions/setup-python from 4.1.0 to 4.2.0"

Reverts GERSL/pycold#5

@Erotemic Jon, I kept encountering errors for 'lint_job' when merging this branch. Could you help double check it? It seems to be GDAL issue.

It looks like the error is real. The linter is saying that gdal and gdal_array are not defined.
./src/python/pycold/imagetool/prepare_ard.py:168:16: F821 undefined name 'gdal'
    driver_t = gdal.GetDriverByName(gdal_driver)  # http://www.gdal.org/formats_list.html
               ^
./src/python/pycold/imagetool/prepare_ard.py:171:12: F821 undefined name 'gdal'
    inDs = gdal.Open(file_name, gdal.GA_ReadOnly)

Use the "./run_linter.sh" script locally. It will tell you that you have an undefined variable and any other linting issues. (Note: most editors support adding some form of linting directly in the editing interface, I use the ALE plugin with flake8 for vim to do my linting. VSCode also has some great plugins to get flake8 results directly in the editor).
Adding from osgeo import gdal and from osgeo import gdal_array to the top of prepare_ard.py fixes the issue.

can't believe that these imports were removed by accident.
I also added import fiona package in prepare_ard.py. I found that fiona was missing in the requirement. But the CI on my recent push is still successful. It is weird. Did I miss something?
Su

can't believe that these imports were removed by accident.

This is likely because you merged devel into your branch before merging it back into devel. I've always had issues doing it that way (although I do believe it is a valid way to do it, but I don't have enough experience with it to know what the pitfalls are). What I usually do when I have a branch that is outdated from whatever the "main" branch is (in this case 'devel'), is I rebase my changes on top of that branch.
I wrote a small doc with references on how to do this in the WATCH repo: https://gitlab.kitware.com/smart/watch/-/blob/main/docs/rebasing_procedure.md

I also added import fiona package in prepare_ard.py. I found that fiona was missing in the requirement. But the CI on my recent push is still successful. It is weird. Did I miss something?

Yes, fiona also got incorrectly removed. I missed that when I made my first comment. I don't see any new changes on the CI. Where is your recent push?

This is likely because you merged devel into your branch before merging it back into devel. I've always had issues doing it that way (although I do believe it is a valid way to do it, but I don't have enough experience with it to know what the pitfalls are). What I usually do when I have a branch that is outdated from whatever the "main" branch is (in this case 'devel'), is I rebase my changes on top of that branch.

That was exactly what I was doing. It is good to know that rebasing the change could resolve the issue.  I will

I directly pushed it to the devel repo, just for convenience. I will create a new branch for future.

I do that all the time (usually hotfixes). It's NBD, but feature branches do help to ensure nothing breaks :)
If you're interested, we can also turn on cibuildwheel for other Python versions and get a Github Action step in there to publish pycold to PYPI whenever a new tag is created (e.g. via the Github Release Interface). It would make watch integration a lot nicer if we can pip install a specific known working version of pycold.
It looks like nobody has taken the name yet: https://pypi.org/project/pycold/  I would register a PYPI account and claim that name. Not sure if you can claim a name from the web interface.You might need to upload a package.
It wouldn't be too hard to manually upload one of the wheels the CI built: https://github.com/GERSL/pycold/suites/8158007680/artifacts/354178612
You can do that via twine (pip install twine): and then run:
# For testing: TWINE_REPOSITORY_URL=https://test.pypi.org/legacy
TWINE_REPOSITORY_URL=https://upload.pypi.org/legacy/
twine upload --username "$TWINE_USERNAME" --password=$TWINE_PASSWORD \
                --repository-url "$TWINE_REPOSITORY_URL" \
                "$WHEEL_PATH" --skip-existing --verbose || { echo 'failed to twine upload' ; exit 1; }

If you want to do auto publishing you will need to create a github secret with a pypi token for your account as well. If you tell me what the environment variable is named I'll add the action to publish to pypi whenever a new tag is made.

I like the idea of 'tagging' a certain version of pycold, so that my continued development won't break your system, though I never published package in PYPI before. I am studying from the PYPI website right now; it will take a while. But sure, I certainly hope that pycold could be released through pypi. If my understanding is correct, if we put PYCOLD in pypi, the users could directly install it through 'pip install'? (even for window users?)

Yes, uploading to pypi will allow people to pip install it. However, there are caveats because this package has a binary distribution: the CI needs to build a wheel for each platform / operating system.
It is possible to release a source tarball on pypi, which means if the user has all of the binary dependencies and a compiler, pip install will built it for them, but that is often very difficult to work with (that's why we don't pip install gdal here, and instead we use wheel published on Kitware's girder server). So often source and binary distributions are released together. For example, see the packages I released for networkx_algo_common_subtree. You will see there is a wheel for different operating systems (windows, osx, linux), python versions: cp37, cp38, etc..., and currently only one architecture (x86_64 i.e. 64 bit x86 or amd64 processors, but other architectures like arm64 (i.e. rasberry pi CPUs) or i686 (32 bit CPUs) could be supported, but they would have a specific machine build those wheels.
With this repo, I want to mainly target manylinux wheels, because that's a lot easier to work with, but we can try to build windows wheels as well. That is possible on Github Actions CI.
For PYPI, all you need to do is create an account. Then go to account settings:

Scroll down to the API tokens section and click Add API token:
.
Copy down this token and store it somewhere safe. If you lose it you will have to revoke it and make a new one.
Then go to github on this project and click the "..." option for the repo and go to Settings. Then under security click "Secrets" and then "Actions".

Click the button New Repository Secret: and set the "Name" to the name of the token, and then set the "Secret" to the secret value that you copied down. And finally hit "Add Secret".
Once you've done this, tell me the name of your secret, and I will setup the github action to publish to pypi.
You should also do this on the https://test.pypi.org/ site, which is a completely distinct site from pypi.org, but is used to test that your uploads are good. So make an account there, another secret token (with a different name, i.e. prefixed with TEST_ and then do the same process to add that as a second repo secret. That will allow PRs to test that they can publish to test.pypi, but when you make a tag it will publish to real pypi.

@Erotemic thank you for such detailed instruction. Yes, I already added 'PYPI' (for pypi) and 'PYPI_TEST' (for test.pypi) in repo secrets. Let me know if it works.


RPM SDK performance issues

RPM SDK has multiple performance issues to be resolved:

[x] It copies ALL packages of repository to temporary directory, repository may be huge (e.g. > 500 Gb), so it's very expensive operation, we need to get rid of copying of temporary files
[x] For optimization, the SDK user can provide only new packages (not all repository) and all repodata files, so we can update metadata based on provided packages. We can add update(Set<Key>) method to perform batch update only on selected keys
[x] Cleanup without providing packages: SDK can provide a method for removing packages based on checksums (to be discussed)

@g4s8 the puzzle #397 is still not solved.

Done, @olenagerasimova thanks


Adds Tests to Runner Implementation

Also reveals bugs when running flakeguard on test packages with panics in them.

Below is a summarization created by an LLM (gpt-4-0125-preview). Be mindful of hallucinations and verify accuracy.
Why
The changes enhance the Flakeguard tool's capabilities, making it more robust and useful for identifying flaky tests in a CI environment. They add functionality to exclude certain test packages, improve test result parsing, and ensure output files handle sensitive information correctly, all aimed at providing a better understanding and management of test flakiness.
What

tools/flakeguard/.gitignore

Added entries for tmp_test_flaky_state and flaky_test_results_*.json to ignore temporary test states and flaky test result files. This prevents unneeded files from being tracked by Git.

tools/flakeguard/Makefile

Modified test_unit target to exclude flaky_test_package during testing, ensuring flaky tests don't interfere with unit test results.
Added a new test target installing gotestfmt and running tests with coverage reports, excluding flaky_test_package. This improves the testing process by formatting test outputs and excluding known flaky tests.

tools/flakeguard/README.md

Added a newline for better readability under the main commands section.

tools/flakeguard/cmd/run.go

Added annotations to suppress GoSec warnings when writing files, acknowledging the security implications of the chosen file permissions.

tools/flakeguard/git/git_test.go

Added t.Parallel() to tests for concurrent execution, speeding up the test suite.

tools/flakeguard/reports/reports.go

Expanded TestResult struct to include Failures, Successes, Panics, and Skips fields for a more detailed test outcome analysis.
Adjusted JSON writing functions to include GoSec annotations for file writing operations, addressing potential security concerns.

tools/flakeguard/reports/reports_test.go

Adjusted a test helper function with a GoSec annotation to suppress warnings, ensuring the handling of file operations is acknowledged.

tools/flakeguard/runner/flaky_test_package/flaky_test.go

Introduced a new file with example tests, including passing, failing, flaky, skipped, and panic-inducing tests to simulate different scenarios in test execution.

tools/flakeguard/runner/runner.go

Added a CollectRawOutput field and corresponding logic in Runner to capture raw output from test runs, aiding in debugging and analysis.
Improved test execution functions to handle output collection and error analysis more robustly, including cleanup operations and detailed test result parsing.

tools/flakeguard/runner/runner_test.go

Added new unit tests for the Runner struct, covering scenarios with default settings, including handling panics and testing against a mock runner setup for controlled test execution verification.

Tests are still failing, need to fix the bugs first.


Rerun gen.py tests and do comparison

After #724.
Compare old LS to new LS, pass/fail/not applicable, and report bugs.
Splitting this into a second issue for tracking.

Done; will compile and submit bugs with repros after discussion.


[EPIC] A collection of items to improve developer / CI speed

Is your feature request related to a problem or challenge?
As DataFusion becomes more mature and gets more features and tests, the amount of time it takes to build and test the system has been increasing
This has several downsides:
Barrier to new contributors is higher
The resources required to build / link DataFusion are now very large which means some people may not be able to run them. For example @Alexhuszagh reports on https://github.com/apache/datafusion/pull/13693#issuecomment-2526394913

I only tested this locally with the failing datasource::physical_plan::json::tests::test_chunked_json tests, but it should be the same with all the remaining errors. My test machine doesn't have enough RAM to link the entire core test suite,

We are likely wasting lots of resources running CI tests
By my count the CI checks on the most recent checkin (link) require over 200 hours of runner time
We are lucky the ASF gets many runner credits from github, but this level is very wasteful in my opinion and likely unsustainable as we contemplate adding additional testing such as

https://github.com/apache/datafusion/issues/13812

Larger binary size
I have noticed that the datafusion-cli binary is now almost 100MB it used to be 60MB
Also, people such as @g3blv have noticed that the WASM build has increased 50%:
https://github.com/apache/datafusion/discussions/9834#discussioncomment-11574581
See

https://github.com/apache/datafusion/issues/13816

Decreased developer productivity due to longer cycle times
As DataFusion gets bigger, I have noticed that recompiling datafusion for me takes longer and longer
cargo test ...
Describe the solution you'd like
I would like to make the building and testing of DataFusion "easier"  and leaner. This ticket tracks work to improve things in this area
Describe alternatives you've considered

[ ] https://github.com/apache/datafusion/issues/13726
[ ] https://github.com/apache/datafusion/issues/13814
[ ] https://github.com/apache/datafusion/issues/13816

Additional context
No response

If this could be improved it would be wonderful. Arrow and Datafusion are very important projects using lexical and I want to test all my code against them and other high impact projects as an additional precaution in addition to our own checks internally, so being able to run these tests locally would be a huge benefit on my end.

Is there a way to test changes to the ci pipeline without actually checking in files to DF?

Is there a way to test changes to the ci pipeline without actually checking in files to DF?

We could in theory change the triggering rules for the the jobs to run on any changes to .github as well

If no one beats me to this I can likely take a look at the ci pipeline during the holiday break. I still want to finish the sqlite test integration first if possible though.

I also hope to hack on various parts of this EPIC during thebreak

Is there a way to test changes to the ci pipeline without actually checking in files to DF?

@Omega359 you can enable the workflow(s) on your fork

I've managed to knock some time off of the ci runs on a sidebranch (need to rebase off of main before pushing a PR) bu here is what I've managed to get to so far:
an example of a PR run now:

my sidebranch on the second run (cache populated)

cargo test doc and hash collisions are the two ones right now that take the most time and much of that is DF compilation that can't really be cached (would blow out the 10GB cache limit I think).
Most of the improvement is better caching, splitting up the linux build test into multiple jobs (it's all cargo checks), and switching to nextest for many tests. I also disabled the intel mac build test.
I hope to submit a PR this weekend for this.

I've managed to knock some time off of the ci runs on a sidebranch (need to rebase off of main before pushing a PR) but here is what I've managed to get to so far:

LOL your screenshots show 37m --> 19m (which is a 2x improvement by my accounting) so that is a somewhat modest description :)

cargo test doc and hash collisions are the two ones right now that take the most time and much

I think the hash collisions one fails so irregularly, that we should consider simply running it on main (not on all PRs) as discussed in https://github.com/apache/datafusion/issues/13845. While that does indeed defer the potential for finding issues I think the tradeoff might be worth it in this specific case


pkg/build/netbsd: use mknod rather than MAKEDEV

MAKEDEV may not be up to date in the instances

Did it work? We had similar issues with tap/tun.


Fix control vars

This PR addresses #46.

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

In the file tests/unit_tests/test_evaluation.py:

Line 50:80: E501 line too long (80 > 79 characters)
Line 99:80: E501 line too long (80 > 79 characters)

Thanks for the suggested variable name updates @yardasol. As for the test suite changes,
input_test_render_jinja_template_python.py: I updated the variable name to use a more generic naming convention.
test_backend.py: Again, I updated the variable name to use a more generic naming convention. And in lines 37-38, I updated the control_dict to follow the change I made in the rollo source code to be {control variable: list of solvers}. The changes in line 78 are to remove the legacy code where I had a nested dictionary in which control_dict[ind][1] had a value that determined whether a variable had multiple values (I don't use this anymore).
test_evaluation.py:  In lines 50-51 and 100-101 and 400-401, I updated the control_dict to follow the change I made in the rollo source code to be {control variable: list of solvers}. Then I also updated the variable name to use a more generic naming convention and removed the legacy code.
test_executor.py: In lines 62-63, I updated the control_dict to follow the change I made in the rollo source code to be {control variable: list of solvers}.
test_toolbox_generator.py: In lines 11, I removed the legacy code. In lines 17, 24, 88, I used more generic variable name. In line 54, 82-23, 101-102, I updated the control_dict to follow the change I made in the rollo source code to be {control variable: list of solvers}.

@yardasol I updated OpenMC to mamba install, so all tests are passing now. Merge?


update maintainers file

ping @docker/core-swarmkit-maintainers @mrjana follow up to docker/docker#31580

Codecov Report

Merging #2010 into master will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2010      +/-   ##
==========================================
+ Coverage   53.79%   53.84%   +0.05%
==========================================
  Files         109      109
  Lines       18997    18997
==========================================
+ Hits        10219    10229      +10
+ Misses       7549     7541       -8
+ Partials     1229     1227       -2

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 ac35889...a16512e. Read the comment docs.

LGTM

LGTM


Port all the robots' configuration to the TemperatureSensorType parameter

@AntonioConsilvio pointed me out that this PR:

https://github.com/robotology/icub-main/pull/911

removes the old parameter HasTempSensor replacing it with TemperatureSensorType, this means that on icub-main@devel several robots cannot start right now, see:

https://github.com/search?q=repo%3Arobotology%2Frobots-configuration HasTempSensor&type=code

cc @valegagge @MSECode

Since the code is retro-compatible, it should start even if using the old parameter HasTempSensor if this one is set to zero. As shown by the tests done here:
https://github.com/icub-tech-iit/five/issues/178#issuecomment-1785322184

Since the code is retro-compatible, it should start even if using the old parameter HasTempSensor if this one is set to zero. As shown by the tests done here: icub-tech-iit/five#178 (comment) So if we do not have the type of sensor on the motor or do we not need it, we can leave the configurations as they are But i'll check it

it is correct! we don't need to update the configuration of all robots unless the user want to read the temperature sensor in case the robot has it.
So I think we can close this issue


dependency list should show aliases

Preconditions
My Chart.yaml lists multiple dependencies. Some are aliased:
$ yq -y '.dependencies' < Chart.yaml
- name: ingress-nginx
  version: 3.25.0
  repository: https://kubernetes.github.io/ingress-nginx
- name: cp-helm-charts
  version: 0.5.0
  repository: https://confluentinc.github.io/cp-helm-charts
  alias: cp
- name: postgresql
  version: 10.3.17
  repository: https://charts.bitnami.com/bitnami
  alias: a-postgresql
- name: postgresql
  version: 10.3.17
  repository: https://charts.bitnami.com/bitnami
  alias: b-postgresql

Actual
When I run helm dependency list, I cannot identify aliased dependencies easily:
$ helm dependency list
NAME            VERSION REPOSITORY                                     STATUS
ingress-nginx   3.25.0  https://kubernetes.github.io/ingress-nginx     ok
cp-helm-charts  0.5.0   https://confluentinc.github.io/cp-helm-charts  ok
postgresql      10.3.17 https://charts.bitnami.com/bitnami             ok
postgresql      10.3.17 https://charts.bitnami.com/bitnami             ok

Expected
IMO, if there is at least one dependency with a defined alias, there should be an "ALIAS" column in the output.
$ helm dependency list
NAME            VERSION  REPOSITORY                                     ALIAS       STATUS
ingress-nginx   3.25.0   https://kubernetes.github.io/ingress-nginx     n/a         ok
cp-helm-charts  0.5.0    https://confluentinc.github.io/cp-helm-charts  cp          ok
postgresql      10.3.17  https://charts.bitnami.com/bitnami             a-postgres  ok
postgresql      10.3.17  https://charts.bitnami.com/bitnami             b-postgres  ok

Implementation
Should probably be around
https://github.com/helm/helm/blob/43853ea772e779b617b011a8d4e69205636fa4f9/pkg/action/dependency.go#L181-L190
I can PR, but first need to know how you want to handle dependencies that have no alias. I used "n/a" above, but we all know that representing absence/emptiness is always tricky. What's the convention in helm here?

Output of helm version:
version.BuildInfo{Version:"v3.2.1", GitCommit:"", GitTreeState:"", GoVersion:"go1.14.4"}

Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.5", GitCommit:"6b1d87acf3c8253c123756b9e61dac642678305f", GitTreeState:"archive", BuildDate:"1980-01-01T00:00:00Z", GoVersion:"go1.16.2", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.0", GitCommit:"e19964183377d0ec2052d1f1fa930c4d7575bd50", GitTreeState:"clean", BuildDate:"2020-08-26T14:23:04Z", GoVersion:"go1.15", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.): Irrelevant.

Thanks for the suggestion. I do agree that providing more visibility as to why a chart would be listed twice under helm dependency list (as an alias) would be beneficial to the end user.

but first need to know how you want to handle dependencies that have no alias. I used "n/a" above, but we all know that representing absence/emptiness is always tricky. What's the convention in helm here?

Good question. I know that dependencies can come from local files and not from a remote repository. Can you look at that use case and see what's the output there? I believe it should be an empty string.
I would also look at EncodeTable and check its output when provided an empty string as input for a table column.
Hope this helps!

Please reopen.


Leverage AMP Optimizer if possible

Feature Description
Follow-up to #967.
We should leverage the new SSR capabilities when the AMP plugin is active. Once the Optimizer is available as its own PHP package, we can also include it via Composer; we just need to avoid conflicts.
Prior art: https://github.com/ampproject/amp-wp/blob/85e45e895629bddf674e8ca2181128a1bcc6ad8c/includes/class-amp-theme-support.php#L2054-L2084
For this to work, the current output needs to be wrapped in an AmpProject\Dom\Document instance.

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria
QA Instructions

For this to work, the current output needs to be wrapped in an AmpProject\Dom\Document instance.

If you don't already us a DOM object for your own processing, you can use TransformationEngine::optimizeHtml( string $html ) instead of TransformationEngine::optimizeDom( Document $document ).

Currently blocked by https://github.com/ampproject/amp-toolbox/issues/567 it seems
Right now we get:
AMP optimization could not be completed due to the following:
 - CannotRemoveBoilerplate: Cannot remove boilerplate because of an unsupported layout: amp-story

The server side rendering is blocked, but the other transformers will do their optimizations.

Started running into this/similar issue, further explained here: https://wordpress.org/support/topic/only-super-admin-can-preview-or-publish-web-stories/

@brianpetro Thanks, I just replied, but note that it's not related to this ticket here at all.


Website issue: Users should be able to download the drawn sketchboard

Description
When clicking on the download icon, users are not able to download the drawn sketchbook in different formats like "jpg" , "png" and "jpeg" format. Add feature that when clicking on the download icon, users should be able to download the drawn sketchbook in different formats like "jpg" , "png" and "jpeg" format.
Screenshots

🥦 Browser
Google Chrome
Checklist ✅

[X] I checked and didn't find similar issue
[X] I have read the Contributing Guidelines
[ ] I am participating in JWOC
[X] I am participating in IWOC
[ ] I am willing to work on this issue (blank for no)

Hello @ItsRoy69 Can you please assign this to me. I really want to contribute.


As a user, I want to see a simple timeline visualization of works by publication year so that I can get a sense of how the materials are distributed by time.

testing notes
design links
search results: L / M / S
notes

is it responsive?
test with a small publication date range (e.g. 1700-1707)
test with a large range
test with only supplying one end of the range and leaving the other blank
does it seem to match up with your results?

I'm just testing the functionality for the desktop. The up and down arrows work really well!  Gives me an error message if I try to search with reversed years (e.g., 1915-1560). Works with large ranges. Small ranges work too, boxes are taller/bigger. The whole space is shadowed if I enter the same year in both boxes (e.g., 1900-1900). Visualization seems to correspond with search results. If you fill in the first box and leave the second limit blank, search defaults to 1922 on the other end. Vice versa with 1559. Looks good!

given resolution of #78 I think basic functionality is done for this, so closing. will probably open a new issue to add interactivity, e.g. hover states, in another release.


TikTok(?) update now redirects video pages

Hello, as with some update, TikTok seems to redirect the TeleTok user agent/cookies.
When running TeleTok with env var HTTPX_LOG_LEVEL=debug, with let's say video url: https://vm.tiktok.com/ZMRJq1M2c/ I get logs like:
DEBUG [2021-12-14 22:36:06] httpx._client - HTTP Request: GET https://vm.tiktok.com/ZMRJq1M2c/ "HTTP/1.1 301 Moved Permanently"
Moved Permanently.

TeleTok is not able to process these videos. I could not get around this change, would've submitted a PR otherwise.. Hope you can fix it!

Hi
Thank you for informing me about the issue, very much appreciate it.
Just checked if my bot can handle this tiktok and seems like it's
So I have researched the error Moved Permanently and I can assume that you have the latest version of httpx library installed. They recently did an update in which you need manually set following_redirects=True to your requests.
I did this change to the code, so you can pull the latest version and check if it's fixed your issue

Haha, it seems like so! Come to think of it, the bot broke after I did a system upgrade on my box :) Thanks for the quick fix! Much appreciated! This can be closed now.
(ofc a nice test case needed to be inserted :-) )


Yi1080 BFUSY31 - restarting after 10 sec of activity - loop

Hi everyone.
My camera is stuck in a loop where it powers on, works for 10 seconds and then restarts. I's a result of running unhack on it.
As I can log in to www interface for 10 seconds I tired everyghing (disabling snapshot etc.).
Running unbrick also did not work (maybe I have a wrong backup files or build it in a wrong way). Could anyone help me please. Highly appreciated. Thank you.

oh. I have this right now. unhacked. i've soldered it up and have ssh in. how did you fix?

What's your model suffix?


crouton not loading from sudo startunity

I have posted about this before but still cant seem to fix the problem, i get the following message from the command in the shell
Welcome to crosh, the Chrome OS developer shell.
If you got here by mistake, don't panic!  Just close this tab and carry on.
Type 'help' for a list of commands.
crosh> shell
chronos@localhost / $ sudo startunity
Entering /mnt/stateful_partition/crouton/chroots/trusty...
Unknown username "whoopsie" in message bus configuration file
X.Org X Server 1.15.1
Release Date: 2014-04-13
X Protocol Version 11, Revision 0
Build Operating System: Linux 3.2.0-61-generic x86_64 Ubuntu
Current Operating System: Linux localhost 3.8.11 #1 SMP Mon Apr 27 08:37:50 PDT 2015 x86_64
Kernel command line: cros_secure console= loglevel=7 init=/sbin/init cros_secure oops=panic panic=-1 root=/dev/dm-0 rootwait ro dm_verity.error_behavior=3 dm_verity.max_bios=-1 dm_verity.dev_wait=1 dm="1 vroot none ro 1,0 2506752 verity payload=PARTUUID=4aa30189-29c0-1640-af81-1abb4eb40031/PARTNROFF=1 hashtree=PARTUUID=4aa30189-29c0-1640-af81-1abb4eb40031/PARTNROFF=1 hashstart=2506752 alg=sha1 root_hexdigest=19983e99694f43f58c9b7fe794a36276d6631a9b salt=c1f66bbcb95439a8ee2d0617298166203029c4b74e1da8c2eb976c10a5fbe491" noinitrd vt.global_cursor_default=0 kern_guid=4aa30189-29c0-1640-af81-1abb4eb40031 add_efi_memmap boot=local noresume noswap i915.modeset=1 tpm_tis.force=1 tpm_tis.interrupts=0 nmi_watchdog=panic,lapic iTCO_vendor_support.vendorsupport=3
Build Date: 09 December 2014  11:14:53PM
xorg-server 2:1.15.1-0ubuntu2.5 (For technical support please see http://www.ubuntu.com/support)
Current version of pixman: 0.30.2
Before reporting problems, check http://wiki.x.org
to make sure that you have the latest version.
Markers: (--) probed, (**) from config file, (==) default setting,
(++) from command line, (!!) notice, (II) informational,
(WW) warning, (EE) error, (NI) not implemented, (??) unknown.
(==) Log file: "/var/log/Xorg.1.log", Time: Sat May  2 14:28:47 2015
(==) Using system config directory "/usr/share/X11/xorg.conf.d"
(EE)
(EE) Backtrace:
(EE) 0: /usr/bin/X (xorg_backtrace+0x48) [0x7f1a95253338]
(EE) 1: /usr/bin/X (0x7f1a950aa000+0x1ad029) [0x7f1a95257029]
(EE) 2: /lib/x86_64-linux-gnu/libpthread.so.0 (0x7f1a941a7000+0x10340) [0x7f1a941b7340]
(EE) 3: /usr/bin/X (0x7f1a950aa000+0xb5346) [0x7f1a9515f346]
(EE) 4: /usr/bin/X (xf86BusProbe+0x9) [0x7f1a95132d69]
(EE) 5: /usr/bin/X (InitOutput+0x74d) [0x7f1a951412bd]
(EE) 6: /usr/bin/X (0x7f1a950aa000+0x5985b) [0x7f1a9510385b]
(EE) 7: /lib/x86_64-linux-gnu/libc.so.6 (__libc_start_main+0xf5) [0x7f1a92be6ec5]
(EE) 8: /usr/bin/X (0x7f1a950aa000+0x44eae) [0x7f1a950eeeae]
(EE)
(EE) Segmentation fault at address 0x0
(EE)
Fatal server error:
(EE) Caught signal 11 (Segmentation fault). Server aborting
(EE)
(EE)
Please consult the The X.Org Foundation support
at http://wiki.x.org
for help.
(EE) Please also check the log file at "/var/log/Xorg.1.log" for additional information.
(EE)
(EE) Server terminated with error (1). Closing log file.
/usr/bin/xinit: giving up
/usr/bin/xinit: unable to connect to X server: Connection refused
/usr/bin/xinit: server error
Unmounting /mnt/stateful_partition/crouton/chroots/trusty...
chronos@localhost / $

i am having a similar problem

@glennismade Try updating your chroot see https://github.com/dnschneid/crouton/wiki/Common-issues-and-reporting

@divx118  i have and to no avail... i have downloaded the latest crouton file also and nothing seems to be working.

@glennismade Does the update produce an error? Can you post the output of the update preferably on gist or pastebin?

@divx118 yeah it give me an error, something to do with -y being used without force and something to do with username whoopsi, which if i remember is the Ubuntu error reporting system.

https://gist.github.com/glennismade/72ab556f48c488effb07

@glennismade You don't have the key installed from the spotify repo
W: GPG error: http://repository.spotify.com stable InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY 082CCEDF94558F59
See http://howtoubuntu.org/how-to-install-spotify-in-ubuntu maybe this can help you.
You can also remove the spotify ppa and try updating your chroot again. Then you can at least use it for now and resolve installing the spotify client later.

@divx118 thank you, that seems to have fixed it, although the performance using the new system seems to have taken a massive hit.. is that normal?

Nope.  Could you post the output of croutonversion inside the chroot?

@dnschneid the performance just seems to have dropped since the chromeos update using the chroot integration plugin..
but here is the output
Entering /mnt/stateful_partition/crouton/chroots/trusty...
Unknown username "whoopsie" in message bus configuration file
(trusty)glenn@localhost:~$ croutonversion
crouton: version 1-20150430123324~master:90ad47ff
release: trusty
architecture: amd64
xmethod: xiwi
targets: xiwi,unity
host: version 6812.83.0 (Official Build) stable-channel peppy
kernel: Linux localhost 3.8.11 #1 SMP Mon Apr 27 08:37:50 PDT 2015 x86_64 x86_64 x86_64 GNU/Linux
freon: yes
(trusty)glenn@localhost:~$

@glennismade You switched to XMETHOD xiwi now which has no gpu acceleration. Try starting your chroot with sudo startunity -X xorg If you want xorg to be the default again then update your chroot with sudo sh ~/Downloads/crouton -t xorg -n trusty

@divx118  @dnschneid  thank you so much.... that is finally back to normal, didnt want to have to go back reinstall if i didnt have to. cheers.


Added Base64 encode and decode string extension methods

Some base64 encoding methods for consideration. Feel free to move around, rename, suggest changes etc

Hi, no need to add Bas64 related methods to this project. I have an entire project/package relating to Encoding here: https://github.com/ByteDev/ByteDev.Encoding which covers Base64 as well as other encodings.
Thanks

Oops, didn’t notice that


Exclude standalone hosts as cluster

I have multiple clusters (icon with 3 PCs). I set the parameter cluster_include_filter = DAC and it works. However, I still have standalone hosts (one PC icon) that contain virtual machines. So I want to exclude them. Are they bypassing the exception check for now? and are created as a cluster. How can i do this?
I've tried adding like this:
host_exclude_filter = <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, esxi-vrn3.srv.local, esxi-vrn4.srv.local

Hey,
host_exclude_filter is an regex filter. try:
host_exclude_filter = <IP_ADDRESS>|<IP_ADDRESS>|<IP_ADDRESS>|<IP_ADDRESS>|<IP_ADDRESS>|esxi-vrn3.srv.local|esxi-vrn4.srv.local

Hello. Yes fine. This filter has worked. Thank you.


Would it be possible to keep the HTTP API responses cached for chat commands?

I would love the ability to track peoples' duel arena kills from the HTTP API but unfortunately it seems to cache the chat command for a couple minutes or so.
Can you make the last update permanently cached? I'm not sure if the other requests are but they seem to last longer at least.
Alternatively, maybe you could add webhooks to allow the client to post data to other websites kinda like the xp tracker already does but allow the user to customize the url address.

We could change it to cache responses, but most users like to have up-to-date stats shown when performing chat commands. What is the use case for cached responses?

I would like to make a Discord bot to show duel arena stats and the chat API is the closest thing I've found so far that would allow me to do that. I understand it wouldn't be up-to-date until the user does "!duels" from looking at the source code and if there was a way to auto-update current duel stats after a duel that would be great.
On a semi-related note: how does the /xp/get API work? It shows responses for some of the accounts I've tried but not all of them. I'm using the latest RL for my account and am not seeing any stats for that API.

Chat commands which rely on stored config data—such as the !duels command you referenced—do not store persistent values on our servers. They are stored for a short duration (5 minutes iirc) and then purged, so you cannot rely on being able to query that data at any time. We're not interested in serving player data that way, so that's not likely to change.
The XP API is used to store player XP data to display on runelite.net using its xp tracker.

But you do serve XP data like that? No interest in webhooks or custom HTTP POSTs plugin either so that the site doesn't have to serve these requests?
Also tell me if I should make another issue because I have all the XP trackers on but I am not seeing my stats in /xp/get and I am not seeing a section for xp tracking on the website. If you could quickly help me understand what to do to make it work and how to see it on runelite.net I'd appreciate it.

Our xp tracker hasn't worked for awhile due to a combination of Jagex throttling the hiscore lookups heavily and Jagex's hiscores not being online.

I have also run into that issue in my app. Could you not just upload hiscores directly from the client on logout in the same vein as the !duels command does (POSTs then GETs)? I understand you do not want to serve data this way but it seems like you are already doing it with these APIs so why not make a POST on logout of the users stats and fix the xp tracker?

That is an option, but we have problems with users submitting fake data for themselves and other people, and it isn't clear how to resolve it.

That makes sense. Is there an issue for this for potential resolutions?

The only 100% solution right now is to scrape hiscores. Jagex offers no other form for service except the hiscores to see xp gains.
Since Friday the hiscores have actually worked, so the xp tracker should be back up and running again now.
An alternative would be if Jagex gave 3rd parties the option to validate a RSN belongs to someone, but that's not currently possible. Even if so, the xp gains could still be fake, but it would not allow someone else to push fake xp gains for them, so the player would only be cheating their own stats,
The other alternatives would be downright horrible and a nightmare to maintain, for example manual verification of RSNs.

As has been mentioned, we're not interested in assuming the role of a hiscore provider or serving long lasting configuration-based values, so I am closing this issue.


Environment-specific logger channels

We need a way to define environment-specific logger channels.
My idea was to add an environments array to all channels, an add getEnvironments(): ?array on the interface.
return new LoggerConfig(
    channels: [
        new DailyLogChannel(
            path: 'path/to/log',
            environments: [Environment::PRODUCTION],
        ),
        new AppendLogChannel(
            path: 'path/to/log',
            environments: [Environment::DEV],
        ),
    ],
)

@przemyslaw-przylucki what do you think?

@brendt sorry - I was on a rather long business trip :/
I kinda like this approach but I feel like we could benefit more with env-specific config support like logger.production.php as we'll need more and more of that exentisivility.
Another cool thing would be to make initializers work like that too - it creates a bit more magic but also creates standards for how things are done. Not sure if I love it or not but it's an idea

I kinda like this approach but I feel like we could benefit more with env-specific config support like logger.production.php as we'll need more and more of that exentisivility.

Oh I think I like that! Let's have a discussion about it on discord as well


Filter subnets when listing subnets, creating instance and ocp

Signed-off-by: xieqiang2020<EMAIL_ADDRESS>Fixes #241

Design:

When creating instance or OCP, user (admin and general users) can choose 'public' subnets or the subnets that are shared in his/her organization;
When listing subnets from the left navigation menu, admins can see all the subnets, include the ones created by general users; general user can only see the subnets he/she created.

Design:

When creating instance or OCP, user (admin and general users) can choose 'public' subnets or the subnets that are shared in his/her organization;
When listing subnets from the left navigation menu, admins can see all the subnets, include the ones created by general users; general user can only see the subnets he/she created.


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Inputs do not zero out when used with New Input System

Environment

Code source: VRTK Github
Code version: July 23, 2021 (a1b6108df1233816bece20cd759e66c7210a73f2)
Platform version: Windows 10 x64, Unity 2020.3.18. Valve Index
3rd party dependencies:  SteamVR plugin 2.7.3 (sdk 1.14.15)

Steps to reproduce

Set up Tilia with Unity's New Input system as per this issue.
Add Tilia.SDK.SteamVR.Unity and SteamVR package from Unity Asset Store
Setup SteamVR default inputs per package instructions
Replace UnityXRPluginFramework with CameraRigs.SteamVR in the farm scene
Update Left/right controller references under Generic Actions and Locomotors to use SteamVR default input mappings accordingly
Set locomotion to any teleport, teleport pointers to axis control
Teleport around with both controllers

Expected behavior
Teleport pointer appears when thumbstick is off center, disappears when thumbstick is released to neutral.
Current behavior
Teleport pointer appears, but very often will fail to disappear as if the axis 2D coordinates were not zero. This is also true to other 1D axis actions like grabbing (index controller) and trigger axis.
I believe this is related to providing a deadzone value for float as per this fix in Tilia.Input.UnityInputSystem.
Not sure if it belongs there since it only occurs when using Tilia.SDK.SteamVR.Unity

This can also be recreated with the latest Tilia Farm project, without the use of SteamVR, so this issue might need to moved.
Steps to reproduce

Download Tilia Farm example for New Input System (627344c)
Open in Unity 2020.3.24f1 (Tested in Android platform)
XR Plugin: OpenXR (PC), OpenXR (Android)
Connect Oculus Quest to PC via Oculus link cable
Hit Play.
Build and deploy to Oculus Quest standalone android
Detach cable
Run in Quest

Current behavior
Near zero float values seem to not register correctly, causing pointers to not disappear, and movement axes to keep moving when joystick is at rest, and triggers to not reset.

just to confirm, you can only recreate when building for quest? you cannot recreate in the editor?

I think this issue is still part of the Tilia.Input.UnityInputSystem package and the sample mappings don't actually contain a better deadzone value and the callback to context for vector2 and vector3 also need a deadzone setting


[NEW STOCK] I suggest add a bot for the Gala (GALA).

Hiii, thanks for all yur bor ;)
if I can allow myself a suggestion, it would be to add the gala please! : D
https://coinmarketcap.com/fr/currencies/gala/
Thanks, see yu.

I am not making more free bots, but you can host your own version using the code in this repo.

Ok, thanks for the info. I will try later :D


MMM-Selfieshot autoupload using MMM-GooglePhotos

Hi, thank you for this module. This is great!
I am trying to get the autoupload using MMM-GooglePhotos for MMM-Selfieshot. I have followed the steps and have set up a album in GooglePhotos called MyMagicMirrorAlbum. I have also ran the below notification. However I still cannot see the photos uploaded.
PLease can you let me know if my config is correct. Also do i have to run this notification once or after each restart of MM or after each restart of my Pi, or after each photos.? please can you provide an example of the correct configuration
`      {
module: 'MMM-Selfieshot',
debug: true,
},  `

{   module: "MMM-GooglePhotos",   position: "top_right",   config: { 		albums: [], // Set your album name. like ["My wedding", "family share", "Travle to Paris"] 		updateInterval: 1000 * 60, // minimum 10 seconds. 		sort: "new", // "old", "random" 		uploadAlbum: "MyMagicMirrorAlbum", // Only album created by create_uploadable_album.js. 		//condition: { 			//fromDate: null, // Or "2018-03", RFC ... format available 			//toDate: null, // Or "2019-12-25", 			//minWidth: null, // Or 400 			//maxWidth: null, // Or 8000 			//minHeight: null, // Or 400 			//maxHeight: null, // Or 8000 			//minWHRatio: null, 			//maxWHRatio: null, 			//// WHRatio = Width/Height ratio ( ==1 : Squared Photo,   < 1 : Portraited Photo, > 1 : Landscaped Photo) 		//}, 		showWidth: 1080, // These values will be used for quality of downloaded photos to show. real size to show in your MagicMirror region is recommended. 		showHeight: 1920, 		timeFormat: "YYYY/MM/DD HH:mm", // Or relative can be used.   } },

You haven't set which album it should show
	config: {
		albums: [], // Set your album name. like ["My wedding", "family share", "Travle to Paris"]
Should be:
	config: {
		albums: [MyMagicMirrorAlbum], // Set your album name. like ["My wedding", "family share", "Travle to Paris"]

im not sure if you are still having the same issue, but if so, check out this ticket . I had to edit node_helper.js to get the upload feature to work


Add downloads badge to the README

Any thoughts on adding a download badge to the README?

Seems unnecessary to me, especially for a project that's just starting. Npm badge is useful information because it shows version number and links to the package. Travis badge because it's an indicator of project health. What purpose does download badge serve?

I like that it makes information symmetrical to what is on npmjs.com, which can avoid a trip there for users who may be vetting the package. I agree it's not integral.

I guess I'm a little skittish about it because I don't want to overload every readme with badges. I can see how it does provide an idea of how much the package is getting used -- but if you're just initializing a new module that number is always going to be zero.

Cool, makes sense.


VersionOverrides link is broken in samples

URL of sample
URL
Describe the bug
Unable to get the schema that is listed in the above link
To Reproduce
Steps to reproduce the behavior:

_Go to http://schemas.microsoft.com/office/taskpaneappversionoverrides
See: The resource you are looking for has been removed, had its name changed, or is temporarily unavailable.

Expected behavior
Not sure, but I would think it would be something valid at least
Screenshots
If applicable, add screenshots to help explain your problem.
Environment

Platform: Mac
Host: Word
Office version number: 16.73
Operating System: MacOS

Additional context
Add any other context about the problem here.

@Kevin-Wallace-Uptime Where are you seeing this link?

It's in pretty much every manifest file: https://github.com/OfficeDev/Office-Add-in-samples/blob/8274b3f1aa54c24eab26d7de06053c280192a313/Samples/excel-insert-file/manifest.xml#L2

Those aren't URLs. Those are XML namespace URIs. They uniquely identify an XML schema, but they aren't necessarily URLs that you can navigate to. The schemas are at: https://learn.microsoft.com/en-us/openspecs/office_file_formats/ms-owemxml/c6a06390-34b8-4b42-82eb-b28be12494a8


Update ci configuration.

Tested on my fork:
StoglRobotics-forks/ros2_control_demos#1
StoglRobotics-forks/ros2_control_demos#2
StoglRobotics-forks/ros2_control_demos#3
Scripts for generating this CI setup are from StoglRobotics/ros_team_workspace#30. To use them, check doc

@bmagyar as this is a test repository. I will continue with merging. I am very certain that it works and we can bulletproof it for other repositories.

@mergifyio backport galactic foxy


Undefined port on ionic run with livereload

Short description of the problem:
Command ionic cordova run android -l writes config.xml and put undefined port on content index.html tag. This is the automatically inserted text on config.xml
<content original-src="index.html" src="http://<IP_ADDRESS>:undefined" />
What behavior are you expecting?
This problems dont let the app entirely load on real device. Here is the print from the app after it is opened.
Steps to reproduce:

call livereload from ionic command
wait build and app launch

Ionic Info
global packages:

    @ionic/cli-utils : 1.0.0
    Cordova CLI      : 7.0.1
    Ionic CLI        : 3.0.0

local packages:

    @ionic/cli-plugin-cordova : 1.0.0
    @ionic/cli-plugin-ionic1  : 1.0.0
    Ionic Framework           : ionic1 1.3.3

System:

    Node       : v6.9.2
    OS         : macOS Sierra
    Xcode      : Xcode 8.3.2 Build version 8E2002
    ios-deploy : 1.8.6
    ios-sim    : 5.0.8

Thanks, this was fixed in this commit: https://github.com/driftyco/ionic-cli/commit/fbaccd5d16c50fe0ce15f6cca6a5734a33133028
It will be in 3.1.0.


Remove axios and detox from package.json in the root folder

Summary
the dev dependencies for axios and detox are not really needed in the root project but only in the detox folder. this will help with future dependency scans to not being flagged.
Also updated the deps for detox.
Ticket Link
N/A
Release Note
NONE

@enahum can you check the error seen here when running the tests.

sure.. is giving some TS errors.. I'll fix that


Can anyone provide some insight on the "context" argument in the methods?

As stated I have reviewed the code and the RFC and I guess I am confused on what the "context" is and how to utilize it?
caldav.listEvents({   "filename": "" }, **context**)

context is an optional argument. In particular, it can contain authentication details if those weren't passed into the constructor. If you're building your own custom actions, it will have details about the triggering event (e.g. HTTP headers)


Upwards Offset

Describe the bug
An offset going up making some buttons not visible
To Reproduce
Copy and paste a wall of text and it'll offset it
Screenshots
If applicable, add screenshots to help explain your problem

Infomation (please complete the following information)
Discord channel: Stable
OS: Windows
Mod: BetterDiscord
Discord language: English

duplicate of #35

Fixed


scrollview for items

If you have a lot of items you can set
listStyle={height: 100} and scroll them

Sweet! Thanks again for working on this.

Thanks!
But I found an issue with this.
When you click on ScrollView (any item) - TextInput will lose focus and hence hide rendered items and then you can't detect what item have been clicked. I'm working on it.
So probably you should rollback for now

I think the issue you described is going to be fixed with keyboardShouldPersistTaps={true}. Right now I replace the for loop which renders the items with a ListView and I will add the keyboardShouldPersistTaps prop too.

Yeah! Just did the same :)

It's all in the repo and published on npm

Thanks man!


"Undefined variable: self" when calling vim-fugitive commands

Since Neovim commit https://github.com/neovim/neovim/commit/0f681c80e1e9c9060394365dae12f8ebd5736176 I get an "Undefined variable: self" message when running :Gpush followed shortly by a Neovim core dump. Disabling vim-dispatch-neovim so that it runs synchronously fixes the issue.

Should be fixed by #10


Securing Node-RED with user and password

Hi,
Seems like a bug, steps to reproduce at commit 75855d54504f28677947659857ae5ea906487001:

Clone the repo, npm install and npm run build.
Modify settings.js: un-comment the adminAuth block
Start node red and go to http://localhost:1880
Login

Unexpected result: cannot load editor.

Hi, we will need some more details to understand what's happening here.

Did you confirm you can load the editor before you enable adminAuth?
Did you just uncomment adminAuth or did you also configure it with proper values?
What error do you get when trying to load the editor?

Recreated - investigating.

Thanks for spotting this!


Add document for Fairing APIs

/kind feature
Describe the solution you'd like
Kubeflow Fairing should have baisc documents for Fairing APIs, so that the user can just reading the documents when calling APIs, instead of reading code.
Anything else you would like to add:

I prefer to use the sphinx to generate the docs for Fairing APIs. I would like have a demo for the Fairing community to show my thoughts by the PR #338. Thanks.

We can use this to track the docs
/assign @brightfly
/assign @hamedhsn
/assign @jinchihe
/assign@swiftdiaries
/cc @karthikv2k

To be avoid duplicating efforts, with all due respect, I suggest the assignment can be as following to finish the docStrings for functions/class. You can tell me or append to change if you are interested in other.

Owner: @brightfly
backends/
builders/
cloud/

Owner: @hamedhsn
config.py/
deployers/
frameworks/

Owner: @jinchihe
functions/
http_utils.py
kubernetes/
utils.py

Owner: @swiftdiaries
ml_tasks
notebook
preprocessors
runtime_config.py [Seems no need]

Generate docs using the sphinx and update #338 if needed after finished docstrings. Owner: @jinchihe
Reviewer for the docstring changes: @karthikv2k @abhi-g
Target date:

Finishing draft (send PR for reviewing): Aug 16
Megering PR(updated to address reviewing comments): Aug 23
Generating final Fairing API docs: Aug 30

Comments?  Thank you very much for the cooperation.

Hello @jinchihe did you mean to tag me above ?  Maybe you accidentally tagged the wrong person?  LGTM except the dates I likely need an additional week

Hello @jinchihe did you mean to tag me above ? Maybe you accidentally tagged the wrong person? LGTM except the dates I likely need an additional week

@hamelsmu Updated. Thanks for the reminder. Sure, that's depends on your working, just proposal date, not deadline :-)

Hello @hamedhsn @swiftdiaries @@brightfly,
I recommend you use the pyment to generate the DocString format and then just need to add description for function and agrs. I did like this in the PR #352 for my owned parts. Thanks.

@jinchihe I really apologize but I have to drop out of this effort temporarily.  I was supposed to be using fairing at work so this was a natural way for me to document the library while using it, but unfortunately, I have been assigned to something different.  However, I do plan on coming back and contributing hopefully in the near future.  Sorry about this 🙇

@hamelsmu No problem. Plan couldn't catch up with change sometimes, haha. I will try to take your parts once get chance. Thanks.

@jinchihe I ll take care of those part, as you tagged me originally ;)

@jinchihe I ll take care of those part, as you tagged me originally ;)

@hamedhsn Wow, Exciting! Welcome. Great thanks for your contribution in advance. :-)

@jinchihe I ll take care of those part, as you tagged me originally ;)

@hamedhsn Wow, Exciting! Welcome. Great thanks for your contribution in advance. :-)

No problem. PR: https://github.com/kubeflow/fairing/pull/354

@xauthulei @QxiaoQ  @zoyun @wzhanw
Thanks for your docString contribution in advance!

Please install pyment and then use the tool to generate the docstring framework for your parts,
And then add docStrings for each function and class manually, please see the PR https://github.com/kubeflow/fairing/pull/346 for examples.

To be avoid duplicating efforts, with all due respect, I suggest the assignment for left parts can be as following to finish the docStrings for functions/class. You can tell me or append to change if you are interested in other.
Owner: @wzhanw
kubeflow/fairing/builders/
Owner: @zoyun
kubeflow/fairing/cloud/
kubeflow/fairing/backends/
Owner @xauthulei @QxiaoQ
kubeflow/fairing/ml_tasks/
kubeflow/fairing/notebook/
kubeflow/fairing/preprocessors/
kubeflow/fairing/runtime_config.py (Seems no need?)

@jinchihe , I will take action later, Thanks

@jinchihe Me too.

My parts are done:

kubeflow/fairing/runtime_config.py (Seems no need?)

This python file is too simple without complex logic, we could ignore that.

kubeflow/fairing/preprocessors/

For this part, I have committed the PR and associated with this issue, you could review them when you are free. Thanks.

@jinchihe

@xauthulei Really great! Thanks!

@wzhanw Any plan to finish this?  thanks a lot!

For this changes, we need to enable the pylint  when using the VScode to edit. so that we could avoid thekubeflow-fairing-presubmit  job failed.


NIFI-8614 Corrected Cluster Node Firewall for Spring 5

Description of PR
NIFI-8614 Updates NodeClusterCoordinatorFactoryBean to support a null instance of ClusterNodeFirewall as expected prior to upgrading from Spring Framework 4 to Spring Framework 5.  Changes include adjusting FileBasedClusterNodeFirewallFactoryBean to return null when the firewall configuration file is not found.  These changes are necessary to support existing system integration tests and clusters running without TLS configured.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically main)?

[X] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[X] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[X] Have you written or updated unit tests to verify your changes?
[X] Have you verified that the full build is successful on JDK 8?
[X] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

Thanks @exceptionfactory this looks good. Ran system tests and saw that they are now successful whereas the cluster-based ones previously were failing. +1 merged to main


esp32-c3 freezing after for tasks over 5800ms with _TASK_SLEEP_ON_IDLE_RUN. Regular esp32 works fine.

It appears the use of _TASK_SLEEP_ON_IDLE_RUN with the esp32 is discouraged and it doesn't appear to reduce power usage much but maybe this will help diagnose the issue?
//This works on the esp32-c3. Reg esp32 works.
Task t2(5800, TASK_FOREVER, &t2Callback);
//This works on the esp32-c3 for 2 runs then freezes. Reg esp32 works.
Task t2(5850, TASK_FOREVER, &t2Callback);
//This freezes the esp32-c3 after 1 run.. Reg esp32 works.
Task t2(5900, TASK_FOREVER, &t2Callback);
I verified this is the case with the real hardware as well as this slick simulator that also freezes in the same fashion.
The full esp32-c3 example code used to test
https://wokwi.com/projects/373706302321604609
The regular esp32 with the same code except working with longer delay
https://wokwi.com/projects/373350195861220353

@beachmiles - there is a good reason why the use of _TASK_SLEEP_ON_IDLE_RUN is discouraged.
The current implementation (which may have an overflow bug indeed) uses light sleep mode which is incompatible with WiFi and BT.
https://docs.espressif.com/projects/esp-idf/en/latest/esp32c3/api-reference/system/sleep_modes.html#wi-fi-bluetooth-and-sleep-modes
You are more than welcome to play with the sleepMode implementation to find a better solution here:
https://github.com/arkhipenko/TaskScheduler/blob/4ccca1ac3d63a2b6fa853df3c93147fa4e8980da/src/TaskSchedulerSleepMethods.h#L48
but I would not use _TASK_SLEEP_ON_IDLE_RUN  if you use WiFi, definitely not on a single-core C3.

Verified the old TaskScheduler 3.6.0 version does not freeze my esp32-c3 where 3.7.0 does freeze with the longer tasks.
It looks like there was maybe no sleeping function set for esp32 with _TASK_SLEEP_ON_IDLE_RUN in this 3.6.0 version so that would explain it working unless Im not seeing the code correctly.
I could have sworn I saw a lower power draw with _TASK_SLEEP_ON_IDLE_RUN being set on my esp32 though.
https://github.com/arkhipenko/TaskScheduler/compare/v3.6.0...v3.7.0


Remove redundant CI job

Codecov Report

Merging #12433 (3d1de67) into main (aa54ef1) will increase coverage by 0%.
The diff coverage is n/a.

:exclamation: Current head 3d1de67 differs from pull request most recent head 019d7ae. Consider uploading reports for the commit 019d7ae to get more accurate results

@@          Coverage Diff           @@
##            main   #12433   +/-   ##
======================================
  Coverage     69%      69%
======================================
  Files        496      496
  Lines      31458    31458
  Branches    4928     4928
======================================
+ Hits       21952    21967   +15
+ Misses      7653     7642   -11
+ Partials    1853     1849    -4

Impacted Files
Coverage Δ

src/platform/common/process/condaService.node.ts
67% <0%> (-2%)
:arrow_down:

src/platform/api/pythonApi.ts
66% <0%> (-2%)
:arrow_down:

src/kernels/kernelDependencyService.node.ts
82% <0%> (-1%)
:arrow_down:

src/kernels/common/baseJupyterSession.ts
81% <0%> (-1%)
:arrow_down:

src/notebooks/debugger/debuggerVariables.ts
69% <0%> (ø)

...c/platform/interpreter/interpreterPackages.node.ts
74% <0%> (+<1%)
:arrow_up:

...c/kernels/execution/cellExecutionMessageHandler.ts
80% <0%> (+1%)
:arrow_up:

src/notebooks/controllers/kernelRanking/helpers.ts
61% <0%> (+1%)
:arrow_up:

...tive-window/debugger/jupyter/kernelDebugAdapter.ts
62% <0%> (+1%)
:arrow_up:

src/platform/common/application/extensions.node.ts
86% <0%> (+2%)
:arrow_up:

... and 4 more


[fix] make <li> <code> elements in docs scrollable

Continuation of #8034 and #8056
I missed code blocks in list-items when I did the last PR.
Problem can be seen in this section:
https://kit.svelte.dev/docs/adapters#writing-custom-adapters
(view the page in a mobile-screen sized browser-window)
This little addition should take care of that.
Note:
Thought about scrolling the whole<ul>. But that did not look right when I tried it. Just doing the same scroll as in paragraphs had a better feel.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

Linting checked out on my machine with pnmp lint from the monorepo root.

I guess close this and apply code change to https://github.com/sveltejs/site-kit/blob/3fa010ffcb4a8c059e899c50d2802cc4122aeadd/packages/site-kit/src/lib/styles/text.css#L59 instead, if wanted. I leave this to you guys.


[on hold] Prepare to publish Spanish and Japanese runtime docs

Summary
This PR is on hold until the translation team completes QA of the translated docs
Details and comments
Fixes #

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 62.412%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2851

Relevant Lines:
4568

💛  - Coveralls


Define writing style

It seems there are some inconsistencies in the cookbook's writing style.
For example:

verb tense in titles: "Write a Parquet file" vs. "Reading a Parquet file"
capitalization: "IPC" vs. "ipc", "parquet" vs. "Parquet", "numpy", "Arrow array" vs. "Arrow Array", etc.

Should this be consistent across the different implementations of the cookbook, or only matters to be consistent within each implementation?
In the case of the capitalization point, I think it'd be really helpful to have a style guide which is applicable across all versions, though I'm not sure with the verb tense point whether this should be up to each individual implementation to decide.

Ideally, very general concerns like this should probably get a consistent answer, IMHO.
Besides writing style, there's the topic of trying to define the typical structure of a cookbook entry (for example: problem + solution + optional elaboration), so that searching through it yield a consistent experience.

cc @amol-

That makes sense; I absolutely agree that consistency is important for writing good docs (and something we can do with more of).
I just want to be careful not to prioritise consistency above the needs of the different audiences for the different cookbooks; the typical Arrow C++ user is likely a very different person with different skills and aims than a typical Arrow R user, who in turn is different from a typical Arrow Python user etc etc, and I want to make sure each implementation is able to cater best to the needs of their audiences.

I think that for the titles it makes sense to stick to a verb standard. Mostly to make sure the index of the cookbook doesn't look too odd.
I'm personally less concerned about capitalization as there will be cases where you want to use Arrow Array because you mean specifically the Array class or Arrow array when you are talking more in general of arrays as a data structure.


Intraday Live Scanner

Hello,
Thank you for the good  and exceptional work.
I tried the  Intraday Live Scanner ie, Option E.  The output always comes out empty with the 60sec update cycle (see below). Could you verify that this is not the case and that I am doing something wrong.  Thanks.
OUTPUT
[+] 5-EMA : Live Intraday Scanner 	Last Scanned: 14:26:30
+--------+---------------+----------+------+----------+-------+
| Time   | Stock/Index   | Action   | SL   | Target   | R:R   |
|--------+---------------+----------+------+----------+-------|
+--------+---------------+----------+------+----------+-------+
[+] 5-EMA : Live Intraday Scanner 	Last Scanned: 14:26:30
+--------+---------------+----------+------+----------+-------+
| Time   | Stock/Index   | Action   | SL   | Target   | R:R   |
|--------+---------------+----------+------+----------+-------|
+--------+---------------+----------+------+----------+-------+

Hi @sumancology
Live scanner will display 5-EMA based entries for Nifty and BankNifty whenever the conditions occur in real time.
Unless the conditions occur, the results will be empty and that's why it keeps scanning the markets periodically.
Here is an example for your reference.

Hi @sumancology
Live scanner will display 5-EMA based entries for Nifty and BankNifty whenever the conditions occur in real time. Unless the conditions occur, the results will be empty and that's why it keeps scanning the markets periodically.
Here is an example for your reference.

Hello,
Thank you for the quick response.  The issue is resolved.
Appreciated.


Not very helpful

I'm looking for information on the various parameters/flags that I can use in the commands. The descriptions for commands that correspond to actions like "Remove a migration" are very anemic in this document.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6e28b0a8-b7e4-4c07-9e60-0df403bbee28
Version Independent ID: 12300b31-6f7b-b75c-970d-693365e77013
Content: Migrations - EF Core
Content Source: entity-framework/core/managing-schemas/migrations/index.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @bricelam
Microsoft Alias: bricelam

See the following articles:

Entity Framework Core tools reference - Package Manager Console in Visual Studio
Entity Framework Core tools reference - .NET CLI

Note from team triage: make sure the sections are cross-linked appropriately.


the definition of selection frequency is correct?

I used LogisticRegression to test the code. I set C (which is the inverse of lambda) as np.logspace(-2,2,30), I observed that the index of the highest selection frequency is always from the highest C (lowest lambda). It seems that the Lasso method is not working?
According to the definition of fj, when a feature j enters the Lasso path at a specific lambda  (lets say L0, from high to low value), For L1<L0, the coefficient beta of the feature is non-zero. Theoretically speaking, if L2<L1<L0, we can say that the fj(L2)>fj(L1)>fj(L0), which means that smaller lambda produces higher selection frequency? It seems that there is no need to test multiple lambda values and if we set lambda to an extreme small value, we will be getting the best selection frequency?
I also tries to rerun the code with different random seed, it seems that the results at each run is totally different because the knockoffs generated from each run is different.

Hello @BigBroKuang ,
Thank you for your interesting questions and your observation.
To answer your first two questions, I prefer to warn you that solution paths for the lasso estimator are not always monotonic. It is possible to have a lambda L1 where the coefficient of a feature is not equal to 0, whereas it is 0 for a lambda L0 and L2 with L2 < L1 < L0. I invite you to dig in this thread https://stats.stackexchange.com/questions/154825/what-to-conclude-from-this-lasso-plot-glmnet
Also, you touch a second explanation of the use of multiple lambda with your third point. Because of the artificial feature generation random process, the selection process of Stabl is seed dependent. As we bootstrap on a subset of the whole dataset, the coefficient value is not the same for all bootstraps.
Tell me if you have other questions,
Xavier

Thank you so much for your reply!
I tried to test the method on the entire dataset.  Stabl produces multiple number of features with different random seed. My question is how should we determine the best seed or result?

Intuitively speaking, the reason that we use LASSO for feature selection is: after a feature j enters the lasso path at a specific lambda L0, for any L1<L0, the coefficient beta of j is theoretically greater than 0, and shows a monotonical increasing property. Yes, it is true that some coefficients are not monotonically increasing after L0 in real real experiments, but I think this is phenomenon is due to the randomization of initalization of fitting parameters or the introduction of knockoffs. If we assume that: it is theoretically true beta could be 0 or non-zero after L0, we cannot trust on LASSO anymore, since it cannot produce patterned results on beta. According to the definition of selection frequency in your paper, you assumed that beta could be either 0 or non-zero.

Hello @BigBroKuang,
You are right saying that Stabl is dependent of the random seed. The knockoff generation has an impact on the number of selected variable. The number of selected variable might change a little but the set of really informative feature is theoretically selected for every random seed.
In practice:

for research purpose, we fix the random seed to an arbitrary number (42 for example).
In a production setting, we run a hundred of Stabl with a different random seed and select the most selected features by Stabl. There should be a gap in frequency between informative and non informative features.
There is still room for improvement with Stabl, but the set of informative feature is much smaller and reliable than Lasso or other kind of feature selection algorithm.

For your second comment, the non-increasing behavior of the frequency path (frequency path of feature across multiple lambda) can be observed if you decrease the number of bootstrap. The increase in the number of bootstrap reduces this effect as well.


Update create-a-phone-system-call-queue.md

This public documentation does not have any reference to Hybrid scenarios, this only describes the steps that should be followed for Microsoft Teams, even though at the beginning of the page it's mentioned: "Applies to: Skype for Business, Microsoft Teams"
Customer requested to be added a reference in this article where it is stated that the resource account in hybrid scenario needs to be created on-premises, and a reference to this article as well: https://docs.microsoft.com/en-us/skypeforbusiness/hybrid/plan-call-queue

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@MikePlumleyMSFT PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Add Azure Custom Vision

@serenaz : Thanks for your contribution! The author, @garyericson, has been notified to review your proposed change.

@serenaz - Hi Serena, thanks for offering this suggestion. We've decided not to include each Cognitive Services service individually in this article, but to send the user to the central Cognitive Services page where they can get all the details on services like Custom Vision.
#please-close


GaussianRasterizationSettings.new() got an unexpected keyword argument 'kernel_size'

got prompt
[SPARSE][CONV] spconv algo: auto
[ATTENTION] Using backend: flash_attn
using MLP layer as FFN
Sampling: 100%|██████████| 12/12 [00:01<00:00, 10.35it/s]
Sampling: 100%|██████████| 12/12 [00:03<00:00,  3.03it/s]
Before postprocess: 612412 vertices, 1225374 faces
Decimating Mesh: 100%|██████████[00:06<00:00]
After decimate: 30300 vertices, 61268 faces
Rasterizing: 100%|██████████| 1000/1000 [00:00<00:00, 1299.53it/s]
Found 122 invisible faces
Dual graph: 91690 edges
Mincut solved, start checking the cut
Removed 716 faces by mincut
WARNING- Some cuts were necessary to cope with non manifold configuration.
INFO- Loaded 29950 vertices and 60552 faces.
100% done After remove invisible faces: 30076 vertices, 60616 faces
Rendering: 0it [00:00, ?it/s]
Rendering: 0it [00:00, ?it/s]
!!! Exception during processing !!! GaussianRasterizationSettings.new() got an unexpected keyword argument 'kernel_size'
Traceback (most recent call last):
File "/root/autodl-tmp/ComfyUI/execution.py", line 348, in execute
output_data, output_ui, has_subgraph = get_output_data(obj, input_data_all, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
File "/root/autodl-tmp/ComfyUI/execution.py", line 218, in get_output_data
return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
File "/root/autodl-tmp/ComfyUI/execution.py", line 189, in map_node_over_list
process_inputs(input_dict, i)
File "/root/autodl-tmp/ComfyUI/execution.py", line 178, in process_inputs
results.append(getattr(obj, func)(**inputs))
File "/root/autodl-tmp/ComfyUI/custom_nodes/ComfyUI_TRELLIS/TRELLIS_node.py", line 107, in sampler_main
glb=image_to_3d(model,img,preprocess_image,covert2video,trial_id,seed,cfg,steps,slat_cfg,slat_steps,mesh_simplify,texture_size,mode)
File "/root/autodl-tmp/ComfyUI/custom_nodes/ComfyUI_TRELLIS/app.py", line 48, in image_to_3d
glb = postprocessing_utils.to_glb(
File "/root/autodl-tmp/ComfyUI/custom_nodes/ComfyUI_TRELLIS/trellis/utils/postprocessing_utils.py", line 455, in to_glb
observations, extrinsics, intrinsics = render_multiview(app_rep, resolution=texture_size, nviews=100)
File "/root/autodl-tmp/ComfyUI/custom_nodes/ComfyUI_TRELLIS/trellis/utils/render_utils.py", line 110, in render_multiview
res = render_frames(sample, extrinsics, intrinsics, {'resolution': resolution, 'bg_color': bg_color}, **kwargs)
File "/root/autodl-tmp/ComfyUI/custom_nodes/ComfyUI_TRELLIS/trellis/utils/render_utils.py", line 76, in render_frames
res = renderer.render(sample, extr, intr, colors_overwrite=colors_overwrite)
File "/root/autodl-tmp/ComfyUI/custom_nodes/ComfyUI_TRELLIS/trellis/renderers/gaussian_render.py", line 222, in render
render_ret = render(camera_dict, gausssian, self.pipe, self.bg_color, override_color=colors_overwrite, scaling_modifier=self.pipe.scale_modifier)
File "/root/autodl-tmp/ComfyUI/custom_nodes/ComfyUI_TRELLIS/trellis/renderers/gaussian_render.py", line 72, in render
raster_settings = GaussianRasterizationSettings(
TypeError: GaussianRasterizationSettings.new() got an unexpected keyword argument 'kernel_size'
Prompt executed in 67.95 seconds

try reinstall diff-gaussian-rasterization，
git clone https://github.com/autonomousvision/mip-splatting.git ./tmp/extensions/mip-splatting
pip install ./tmp/extensions/mip-splatting/submodules/diff-gaussian-rasterization/

尝试重新安装 diff-gaussian-rasterization，git 克隆https://github.com/autonomousvision/mip-splatting.git ./tmp/extensions/mip-splatting pip install ./tmp/extensions/mip-splatting/submodules/diff-gaussian-rasterization/

"Is there a specific version requirement?"
** Python version: 3.10.15 (main, Oct  3 2024, 07:27:34) [GCC 11.2.0]
Total VRAM 24210 MB, total RAM 1031718 MB
pytorch version: 2.4.1+cu121
xformers version: 0.0.28.post1
Set vram state to: NORMAL_VRAM
Device: cuda:0 NVIDIA GeForce RTX 4090 D : cudaMallocAsync
Using xformers cross attention
[Prompt Server] web root: /root/autodl-tmp/ComfyUI/web
Total VRAM 24210 MB, total RAM 1031718 MB
pytorch version: 2.4.1+cu121
xformers version: 0.0.28.post1
Set vram state to: NORMAL_VRAM
Device: cuda:0 NVIDIA GeForce RTX 4090 D : cudaMallocAsync
Loading: ComfyUI-Impact-Pack (V7.14)
Loading: ComfyUI-Impact-Pack (Subpack: V0.6)
[Impact Pack] Wildcards loading done.

确认去除白底后的图片是否正常，有些脱底节点输出的图片格式不对

确认去除白底后的图片是否正常，有些脱底节点输出的图片格式不对

有推荐的抠图节点吗？抠完图保存在上传回去就正常了

你开启preprocesse image 会自动处理的


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 2033 ms

Resolved: Bofrost.at performance has improved in 534e0f1.


[Device Support Request] Bosch Motion Detector RFDL-ZB-MS (battery percentage = unknown)

Is your feature request related to a problem? Please describe.
Does not display remaining battery percentage for HA integration: Zigbee HubZ Smart Home Controller - Zigbee Home Automation
Describe the solution you'd like
Would like to see battery percentage remaining battery as I am using several of these in my HA setup.
Device signature - this can be acquired by removing the device from ZHA and pairing it again from the add devices screen. Be sure to add the entire content of the log panel after pairing the device to a code block below this line.
{ "node_descriptor": "NodeDescriptor(logical_type=<LogicalType.EndDevice: 2>, complex_descriptor_available=0, user_descriptor_available=0, reserved=0, aps_flags=0, frequency_band=<FrequencyBand.Freq2400MHz: 8>, mac_capability_flags=<MACCapabilityFlags.AllocateAddress: 128>, manufacturer_code=4403, maximum_buffer_size=82, maximum_incoming_transfer_size=82, server_mask=0, maximum_outgoing_transfer_size=82, descriptor_capability_field=<DescriptorCapability.NONE: 0>, *allocate_address=True, *is_alternate_pan_coordinator=False, *is_coordinator=False, *is_end_device=True, *is_full_function_device=False, *is_mains_powered=False, *is_receiver_on_when_idle=False, *is_router=False, *is_security_capable=False)",   "endpoints": {     "1": {       "profile_id": 260,       "device_type": "0x0402",       "in_clusters": [         "0x0000",         "0x0001",         "0x0003",         "0x0020",         "0x0400",         "0x0402",         "0x0500",         "0x0b05"       ],       "out_clusters": [         "0x0019"       ]     }   },   "manufacturer": "Bosch",   "model": "RFDL-ZB-MS",   "class": "zigpy.device.Device" }
Additional context

The other features: motion detection, temperature, and illuminance work properly. Only the battery is unknown.  Unpairing and repairing, fully rebooting, and reconfiguring device did not fix the issue.  Also for what it's worth, this model has 2 new batteries inserted but shows as in developer tools below as "battery_quantity: 1".

Developer tools shows this under State attributes (YAML, optional):

state_class: measurement battery_size: Other battery_quantity: 1 battery_voltage: 2.8 unit_of_measurement: '%' device_class: battery friendly_name: Motion Sensor battery

This is some what similar to issue #25361 but that was a different model but was fixed by a custom quirk pull #285.

Please note this is my first request, so if more info is needed just tell me what to provide and possibly where to find it. Thank you in advance!

If you like getting the battery reporting working you can using the old quirk but adding one new device class with updated signature for the new version.
https://github.com/zigpy/zha-device-handlers/blob/b0bbbfb6b1f267b7d87d833b1ba5a5b0b127a9a1/zhaquirks/bosch/motion.py#L27-L70
It shall being 2 empty lined between the first (original) device class and the new and no one after the new (end of file).
Changing the name and signature and if cluster is changed also you need getting then matching the new device.
If all is working ZHA shall loading it then you is restarting HA.
Easiest is using local quirk for testing https://github.com/zigpy/zha-device-handlers/discussions/693#discussioncomment-857274.
Fixing battery quantity and adding batter types is possible but need more work.
You can looking in the IKEA quirk INIT that i have adding and fixing there remotes batteries how its don.
PS: IKEA remotes is sending false batter % (1/2) and the BOCH is not sending % only voltage and its prity command "defects" in devices.

@MattWestb After making some changes to that quirk I am now able to see the battery level as shown in the code below but have not pursued the battery quantity since they are in parallel and able to operate with either 1 or 2 installed.  I really appreciate your help and response as I was also able to create a working quirk to also show battery level for the Visonic MCT-350 SMA door/window contact sensors that are from Xfinity/Comcast if that is of interest to anyone I can upload as well.
`"""Device handler for Bosch motion sensors."""
from zigpy.profiles import zha
from zigpy.quirks import CustomDevice
from zigpy.zcl.clusters.general import Basic, Identify, Ota, PollControl
from zigpy.zcl.clusters.homeautomation import Diagnostic
from zigpy.zcl.clusters.measurement import TemperatureMeasurement, IlluminanceMeasurement
from zigpy.zcl.clusters.security import IasZone
from zhaquirks import PowerConfigurationCluster
from zhaquirks.bosch import BOSCH
from zhaquirks.const import (
DEVICE_TYPE,
ENDPOINTS,
INPUT_CLUSTERS,
MODELS_INFO,
OUTPUT_CLUSTERS,
PROFILE_ID,
)
class BoschPowerConfiguration(PowerConfigurationCluster):
"""Bosch power configuration cluster for motion sensor."""
MAX_VOLTS = 3.0
MIN_VOLTS = 1.9

class RFDLZBMS(CustomDevice):
"""Custom device representing Bosch motion sensors."""
signature = {
    #  <SimpleDescriptor endpoint=1 profile=260 device_type=402
    #  device_version=0
    #  input_clusters=[0, 1, 3, 20, 400, 402, 500, b05]
    #  output_clusters=[19]>
    MODELS_INFO: [(BOSCH, "RFDL-ZB-MS")],
    ENDPOINTS: {
        1: {
            PROFILE_ID: zha.PROFILE_ID,
            DEVICE_TYPE: zha.DeviceType.IAS_ZONE,
            INPUT_CLUSTERS: [
                Basic.cluster_id,
                Diagnostic.cluster_id,
                IasZone.cluster_id,
                Identify.cluster_id,
                IlluminanceMeasurement.cluster_id,
                PollControl.cluster_id,
                PowerConfigurationCluster.cluster_id,
                TemperatureMeasurement.cluster_id,
            ],
            OUTPUT_CLUSTERS: [Ota.cluster_id],
        }
    },
}

replacement = {
    ENDPOINTS: {
        1: {
            INPUT_CLUSTERS: [
                Basic.cluster_id,
                Diagnostic.cluster_id,
                IasZone.cluster_id,
                Identify.cluster_id,
                IlluminanceMeasurement.cluster_id,
                PollControl.cluster_id,
                BoschPowerConfiguration,
                TemperatureMeasurement.cluster_id,
            ],
            OUTPUT_CLUSTERS: [Ota.cluster_id],
        }
    }
}`

GREAT !!
IKEAs motion sensors is also having 2 batteries in parallel and the attribute is shown in ZHA and is for easter replacing batteries so having the right "in stock" and its not saying if they is in serie or parallel.
Can you making one PR for adding the updated quirk so other user can using it in the future ?
Also peas put ``` on one empty  line before and after the code so its not losing the formation and user can copy it easier.

Sure, will do.  I updated the code format above with ```

@Pau1ey I have been looking for an updated quirk for this Visonic MCT-350 SMA door/window contact sensors that are from Xfinity/Comcast.
I would greatly appreciate if you could share it. As mine is connecting as URC4460BC0-X-R by Universal Electronics Inc and I have not yet figured out the location of the basic file to try and update it.

Is the custom quirk above still working with Home Assistant 2022.4.5?
(Someone could then PR the changes to this repo, so the custom quirk would be merged into Home Assistant.)


Fixed issue #2806

@AtharvaDevasthali,
I'd addressed this and therefore no other action is required.

I am closing this pull request.
Thank you.


pkgsMusl.jdk8: mark broken for musl

pkgsMusl.jdk8: mark broken for musl
Tracking issue: https://github.com/NixOS/nixpkgs/issues/281618

error: ‘isnanf’ was not declared in this scope

CC @edwtjo

I'm not sure what is going on with CI:

@infinisil

Sorry for that! Already fixed since https://github.com/NixOS/nixpkgs/pull/281412, it's really easy to mess up the GitHub workflows since they're not tested by default :sweat_smile:. You'll have to rebase unfortunately.

pkgsMusl.jdk8: mark broken for musl

I'm not sure if it makes sense to start marking everything as broken on musl that doesn't build. You are going to add a lot of broken = stdenv.isMusl in that case...

I'm not sure if it makes sense to start marking everything as broken on musl that doesn't build. You are going to add a lot of broken = stdenv.isMusl in that case...

Better than building it everytime and finding out some inner dependency is broken. Also Musl doesn't have cache, which makes it harder to build. Every PR I have opened to report breakage it is to the specific package and not some downstream breakage. So, it is not all packages that are being marked broken but just the packages that need to be marked broken to avoid long builds.


Height and Width?

Hi Ivo! I use your lazy sizes module and works perfectly for me. Ivo there is way to add width and height to my images? Right now I use Picture tag and in Lighthouse it show no width height but I set.
<picture>

  <source
    type="image/webp"
    media="(max-width: 640px)"
<EMAIL_ADDRESS>
  <source
    type="image/webp"
    media="(max-width: 1024px)"
<EMAIL_ADDRESS>
  <img<EMAIL_ADDRESS>    width="640"
    height="640"
    alt="Blue 1"
    class="lazyload">

</picture>

Is doing right?

Closing this due to inactivity. Feel free to reopen if you have issues.


Relocate Univariate Domain, Vanishing Polynomial, Lagrange Interpolation

Summary
Relocate Univariate Domain, Vanishing Polynomial, Lagrange Interpolation to ark-poly
Problem Definition
As a result of #53 native implementations of Univariate Domain, Vanishing Polynomial, Lagrange Interpolation in this repo. However, they should actually be at ark-poly.
4fbdc2b
Proposal
Move them to ark-poly and adjust the corresponding constraint structs (if there is any at this moment).

For Admin Use

[x] Not duplicate issue
[x] Appropriate labels applied
[x] Appropriate contributors tagged
[x] Contributor assigned/self-assigned

ark-poly already includes native variants of these, no? We already have EvaluationDomain, SparsePolynomial, and various FFT algorithms there.


ch: LoadScript with fake script filename fails on xplat

Some of our debug related tests provide fake filenames for scripts.
For example;
var x = WScript.LoadScript("function foo(){}", "samethread", "dummyFileName.js");

In case the dummyFileName.js is not exist on the actual folder, this particular call returns undefined back to JS land.
Expected behavior is given below;
Get current full path => join full path with this fake name => wholeName.toLowerCase => use the resulting string as a filename to run script. In other words, do not fail script run.

This one is also fixed


Update AssemblyInfo.cs of iOS project

modify AssemblyInfo, let (1) AssemblyTitle is same as project name (2) AssemblyProduct is same as project name (3) AssemblyCompany is same as registered organization (4) AssemblyCopyright is  Copyright © +  registered organization  + year
GitHub Issue (If applicable): #

PR Type
What kind of change does this PR introduce?

What is the current behavior?

What is the new behavior?

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tested code with current supported SDKs
[ ] Docs have been added/updated which fit documentation template. (for bug fixes / features)
[ ] Tests for the changes have been added (for bug fixes / features) (if applicable)
[ ] Contains NO breaking changes
[ ] Updated the Release Notes
[ ] Associated with an issue (GitHub or internal)

Other information

Internal Issue (If applicable):

You change was merged as part of https://github.com/nventive/Uno/pull/671, thank you for your contribution!


[$250] Migrate ReportScreen.js to function component

Class Component Migration
Filenames

[ ] src/pages/home/ReportScreen.js

type: React.Component
has state values: true
has refs: false
has context: false
lifecycle methods: componentDidMount,componentDidUpdate

Task

We currently have some class components in our codebase that we would like to refactor to a function component.
Here's a link with some general advice on how to refactor a class component to a function component: https://react.dev/reference/react/Component#alternatives
If you need additional guidance, please ask in #expensify-open-source
Test for any regressions and verify that there are no breaking changes

Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0184e14afba16fa311
Upwork Job ID:<PHONE_NUMBER>953848833
Last Price Increase: 2023-08-01
Automatic offers:

aimane-chnaif | Reviewer | 25887921

⚠ Could not update price automatically because there is no linked Upwork Job ID. The BZ team member will need to update the price manually in Upwork.

Heads up! The pricing for this issue has been adjusted based on the scope of the work and the fact that no complex bug investigations or proposal are required.

I'd like to work on this issue

I am ready to work on this ticket. :)

@dayana7204 Kindly refrain from spamming all the issues that are on HOLD. Once they are off hold you will have an opportunity to apply.

Hi Marc,
Really sorry for the inconvenience.
Just wanted to get assigned the tasks once those are opened.
Let me stop it now :)
Regards,
Dayana.
On Tue, Jun 27, 2023 at 4:08 PM Marc Glasser @.***>
wrote:

@dayana7204 https://github.com/dayana7204 Kindly refrain from spamming
all the issues that are on HOLD. Once they are off hold you will have an
opportunity to apply.
—
Reply to this email directly, view it on GitHub
https://github.com/Expensify/App/issues/16255#issuecomment-1610144661,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BATBWIWAM33KHWFAIYEM3P3XNM4THANCNFSM6AAAAAAWAY4MYI
.
You are receiving this because you were mentioned.Message ID:
@.***>

I am able to work on this task.

i will work on this

Would love to take this

The proposal aims to migrate the existing ReportScreen.js file from a class component to a function component with React hooks. This migration will enhance code readability, maintainability, and adhere to modern React best practices. The steps include refactoring the class component to a function component, removing unnecessary state and lifecycle methods, introducing React hooks for state management and side effects, handling props, and updating event handlers. The benefits of this migration include improved code readability, better performance, and easier future upgrades. The goal is to provide a more robust and efficient ReportScreen component for the application.

Would love to work on this

Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~019b0a755f7bda1754

I would love to work on this.

Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~010d8b5416650c0aa1

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~010d8b5416650c0aa1

I would like to work on this.

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~0169022423ba77b59f

I would love to work on this issue

@aimane-chnaif  waiting for a review :)

@marcaaron Can we close this issue as PR was merged a while ago? 😄

Closing this.


fix: i18n and dynamic routes

This PR fixes a NextJs application that uses dynamic routes and i18n at the same time.
Using both types of routes at the same time was causing route conflicts and always returning the content of the dynamic route.

:tada: This PR is included in version 2.6.0-stage.9 :tada:
The release is available on:

npm package (@stage dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Avoid any collision of id selection.

The selecting method of event id is used random,
I suggest a little fragment of code to avoid any collision even that seems seldom or never happen.

Thank you!!