jQueryMobile loader gif not fetched when processing css

Have following test code to try JQM from CDN, whitout using AssetPack plugin (/woa), and using AssetPack plugin (/wa):
https://github.com/dmanto/mojo-plugin-assetpack-issue_whith_JQM
The only asset that is really used is the css non-minified file from JQM.
Running it with morbo and then accessing to:
localhost:3000/woa
will access to a simple welcome page without using any asset, and accessing to:
localhost:3000/wa
will access to the same welcome page but using the AssetPack plugin to load then JQM css.
The problem is with the ajax-loader.gif asset, it seems that it is not fetched from the JQM repo.
I tried to add the Fetch pipe to solve this, but I get the following error:
Can't load application from file "/home/daniel/assets_test/assets_test": Can't call method "name" on an undefined value at /home/daniel/.plenv/versions/5.24.0/lib/perl5/site_perl/5.24.0/Mojolicious/Plugin/AssetPack/Pipe/Fetch.pm line 32
I guess it has something to do with the fact that it is a GIF image, but I'm not sure about that.

I think the bug is url(data:....). A new release is on its way.

Thanks for the bug report! 👍

Jan thanks for the promt response
Still have a problem  when running with hypnotoad and chrome browser (although it doesn't seem to make much sense, I admit...)
Running with morbo everything works fine. I can open JQM css:

inspecting in chrome with dev tools, from line 926 on:

/* Loader */
.ui-icon-loading {
  background: url(../../asset/2edfb2ffe3/ajax-loader.gif);
  background-size: 2.875em 2.875em;
}

Some but in FF, also from line 926 on:

/* Loader */
.ui-icon-loading {
  background: url(../../asset/2edfb2ffe3/ajax-loader.gif);
  background-size: 2.875em 2.875em;
}

That seems and work ok on both cases. Now with hypnotad:

inspecting in chrome (un minimized with {}), from line 975:

.ui-icon-loading {
  background: url(images/ajax-loader.gif);
  background-size: 2.875em 2.875em
}

now with FF, from line 744:

.ui-icon-loading {
  background:url(../../asset/2edfb2ffe3/ajax-loader.gif);
  background-size:2.875em 2.875em
}

Just to be clear, case 3 is the one not working, as you can see in the url() sentence, it ends up looking for ajax-loader.gif asset in css directory, that is:
http://localhost:8080/asset/3907fc9b75/images/ajax-loader.gif
so it doesn't find it and you get the 404 Not Found from the server

I don't really care about the browser in this case. Try to use "curl" or "wget" instead. If they show the correct data, then it's probably just browser cache which is the issue. Try doing a hot reload (CTRL+R, CMD+R) or change the assets so you get a new checksum.

Cool! Glad it worked out 😄


Modify heartbeat local storage to use wide strings on Windows.

Description

Provide details of the change, and generalize the change in the PR title above.

Two changes in this PR:

Add Windows international path support to the heartbeat storage library.
Ensure that multiple desktop apps don't write to the same heartbeat file by adding the package name to the heartbeat filename. Previously, only the app name (which is usually "__FIRAPPDEFAULT") was used, so if there were multiple apps installed on a machine (even completely unrelated apps), they would all use the same heartbeat file.

Testing

Describe how you've tested these changes. Link any manually triggered Integration tests or CPP binary SDK Packaging Github Action workflows, if applicable.

The integration test in this PR was run with added international characters in the path. (That test change has since been reverted.) Additionally, it was run manually without the wstring fix to verify that writing the heartbeat data to an international path returned an error prior to this fix, and this change actualy fixes it.

Type of Change
Place an x the applicable box:

[x] Bug fix. Add the issue # below if applicable.
[ ] New feature. A non-breaking change which adds functionality.
[ ] Other, such as a build process or documentation change.

Notes

Bug fixes and feature changes require an update to the Release Notes section of release_build_files/readme.md.
Read the contribution guidelines CONTRIBUTING.md.
Changes to the public API require an internal API review. If you'd like to help us make Firebase APIs better, please propose your change in a feature request so that we can discuss it together.

Seems like an ok change, assuming it works correctly as tested.
It would probably be possible to have a type alias that is either wstring or string depending on platform to reduce the number of times that FIREBASE_PLATFORM_WINDOWS is checked, but then you would have to include whatever file defines this type alias, and it would be less obvious to the reader what type the variable is, so maybe not much benefit to that.

I thought about doing that, and I had an older PR that did (in a different part of the code), but it felt more straightforward to just do it this way. Once we support C++17 we can just use std::path_t.


Update QemuTransport#write() to match new write API contract.

suspect this should fix #8278
forgot to add a loop to write all the data

cc @gromero @junrushao1994

The flaky test in auto scheduler is fixed by #8754

@areusch Hi. For the records, I still see the tvm._ffi.base.TVMError: MicroSessionTimeoutError: failed to read reply message after timeout 5s  exception even with this fix (https://github.com/apache/tvm/pull/8761). I tried against the sine model. I'll continue to investigate more tomorrow.

@gromero i also see this but only on tests/micro/zephyr/test_zephyr.py::test_rpc_large_array[mps2_an521-(16*1024)] now

regardless i think we should still ensure that write() conforms to the API properly. can you let me know which test case you're running?

regardless i think we should still ensure that write() conforms to the API properly.
@areusch yep, I agree.

can you let me know which test case you're running?
It's pretty much the sine model in tutorials/micro_tflite.py. I have prepared a branch which reproduces it (after several runs, you know): https://github.com/gromero/tvm/commits/8728

Just clone && cd 8728 && ./run.sh.  The MLF sine.tar is the same model as in micro_tflite.py but already compiled. The run must crash with:
[...]
INFO:__main__:b'/tmp/x20/qemu-hack/qemu-system-arm -cpu cortex-m33 -machine mps2-an521 -nographic -m 16 -vga none -net none -pidfile qemu.pid -chardev pipe,id=con,mux=on,path=/tmp/tmpq4aigpet/fifo -serial chardev:con -mon chardev=con,mode=readline -icount shift=7,align=off,sleep=off -rtc clock=vm -kernel /tmp/x20/build/zephyr/zephyr.elf\n'
qemu-system-arm: warning: nic lan9118.0 has no peer
[02:19:49] /home/gromero/git/tvm/src/runtime/micro/micro_session.cc:367: remote: microTVM Zephyr runtime - running
B
INFO:__main__:b"make[3]: Leaving directory '/tmp/x20/build'\n"
INFO:__main__:b'[100%] Built target run\n'
INFO:__main__:b"make[2]: Leaving directory '/tmp/x20/build'\n"
INFO:__main__:b'/usr/bin/cmake -E cmake_progress_start /tmp/x20/build/CMakeFiles 0\n'
INFO:__main__:b"make[1]: Leaving directory '/tmp/x20/build'\n"
Traceback (most recent call last):
  File "/home/gromero/git/tvm/8728/./an.py", line 38, in <module>
    syslib = session.get_system_lib()
  File "/home/gromero/git/tvm/python/tvm/micro/session.py", line 88, in get_system_lib
    return self._rpc.get_function("runtime.SystemLib")()
  File "/home/gromero/git/tvm/python/tvm/rpc/client.py", line 73, in get_function
    return self._sess.get_function(name)
  File "/home/gromero/git/tvm/python/tvm/runtime/module.py", line 85, in get_function
    check_call(
  File "/home/gromero/git/tvm/python/tvm/_ffi/base.py", line 348, in check_call
    raise get_last_ffi_error()
tvm._ffi.base.TVMError: MicroSessionTimeoutError: failed to read reply message after timeout 5s
FAIL

Which points  to the timeout happening at https://github.com/gromero/tvm/blob/8728/8728/an.py#L38 , i.e. just after the handshake as you mentioned I believe.  So I don't believe it's related to the model itself. I'm also with that hypothesis in mind that you raised about something fishy in mps2_an521 drivers ... But can't make sense of it yet.

@areusch LGTM! Although I think it won't fix totally https://github.com/apache/tvm/issues/8728 this looks correct to me and also there is a chance we will at least reduce the frequency PR 8728 happening in the CI, which is great. Thanks!

@areusch btw! the commit message cites PR 8278 (* suspect this should fix #8278) but it's indeed instead 8728! If there is still time...

thanks @gromero!


chore: enable merge conflicts bot

The token will be set up via separately.

@semorrison You might have to accept the triage permission invite on behalf of the bot.


Change text color of StyledClassedTextArea with css

Hi,
Thank a lot for your lib, but i have a new problem with RichTextFx. I use v0.6.10 of your lib.
I have a StyleClassedTextArea and i need to change color of text by css styling.
With this java code :
myText.getStyleClass().add("markdown-editor");

I tried try many things in css.
.markdown-editor {
    -fx-background-color:#000;
    -fx-text-fill: #FFF;
}

or
.markdown-editor {
    -fx-background-color:#000;
    -fx-fill: #FFF;
}

.markdown-editor .label{
    -fx-background-color:#000;
    -fx-text-fill: #FFF;
}

Whatever css, my background is black, but my text stay black despite my attempts.
Do you know any solution for change text color ?

Assuming myText is an instance of StyleClassedTextArea, you would need something like
.markdown-editor .text {
    -fx-fill: #FFF;
}

Nice it works fine.
Thanks.


Sort imports for more consistent diff (ruff/isort)

Summary of changes
As title says.
I've made sure that _distutils_hack should be sorted before _distutils, and that setuptools gets sorted before distutils in tests as long as the hack is supported (ref: https://github.com/pypa/setuptools/issues/4137 )
Pull Request Checklist

[x] Changes have tests
[x] News fragment added in newsfragments/. (no user-facing changes)
(See documentation for details)

I think in the case of this PR it would be interesting in separating 2 commits

Certainly, now that I have confirmation this is indeed wanted, I'll be cautious about keeping it into 2 clean commits.

Thank you very much.
This might add some extra work for the currently opened PRs, but I think it is good in the long term.

some extra work for the currently opened PRs

The good news is it shouldn't be much: accept both on merge conflict concerning imports, then let Ruff do the rest.


Customizable logstash index format

By default, logstash uses the index format of logstash-%{+YYYY.MM.dd}, creating a single index per day. However, I think there's a reasonable case for wanting to be able to configure it, depending on the installation requirements:

for smaller or significantly larger logging requirements, it can be more efficient for both kibana usage and elasticsearch to use a different index date format (e.g. logstash-%{+YYYY-ww} for weekly)
instead of using separate deployments, it can be more effective to segment indices by some property (e.g<EMAIL_ADDRESS>for environment-specific indices)

We could expose this as a logstash_parser.index_format property, defaulting to the current setting of logstash-%{+YYYY.MM.dd}.

Resolved by #106.


Ci/show fork test status

Submission type

[ ] This submission adds a new dataset
[ ] This submission adds a new method
[ ] This submission adds a new metric
[ ] This submission adds a new task
[ ] This submission adds a new Docker image
[ ] This submission fixes a bug (link to related issue: )
[ ] This submission adds a new feature not listed above

Testing

[ ] This submission was written on a forked copy of openproblems
[ ] Nextflow test pipeline is passing on this base branch of this pull
request (include link to passed test on NF Tower found in GitHub Actions summary: )
[ ] If this pull request is not ready for review (including passing the Nextflow test
pipeline), I will open this PR as a draft (click on the down arrow next to the
"Create Pull Request" button)

Submission guidelines

[ ] This submission follows the guidelines in our
Contributing document
[ ] I have checked to ensure there aren't other open Pull Requests for the
same update/change

PR review checklist
This PR will be evaluated on the basis of the following checks:

[ ] The task addresses a valid open problem in single-cell analysis
[ ] The latest version of master is merged and tested
[ ] The methods/metrics are imported to __init__.py and were tested in the pipeline
[ ] Method and metric decorators are annotated with paper title, year, author, code
version, and date
[ ] The README gives an outline of the methods, metrics and datasets in the folder
[ ] The README provides a satisfactory task explanation (for new tasks)
[ ] The sample test data is appropriate to test implementation of all methods and
metrics (for new tasks)

Codecov Report
Base: 86.32% // Head: 94.93% // Increases project coverage by +8.61% :tada:

Coverage data is based on head (b1a1ca3) compared to base (8a6236e).
Patch coverage: 96.35% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
+ Coverage   86.32%   94.93%   +8.61%
==========================================
  Files          77      157      +80
  Lines        1696     4288    +2592
  Branches      100      227     +127
==========================================
+ Hits         1464     4071    +2607
+ Misses        201      142      -59
- Partials       31       75      +44

Flag
Coverage Δ

unittests
94.93% <96.35%> (+8.61%)
:arrow_up:

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

Impacted Files
Coverage Δ

...oblems/data/Wagner_2018_zebrafish_embryo_CRISPR.py
0.00% <0.00%> (ø)

...s/tasks/multimodal_data_integration/metrics/mse.py
82.35% <50.00%> (ø)

...egration/batch_integration_graph/methods/_utils.py
61.11% <61.11%> (ø)

openproblems/api/utils.py
84.61% <75.00%> (+1.28%)
:arrow_up:

openproblems/api/run.py
85.71% <81.25%> (-14.29%)
:arrow_down:

openproblems/api/parser.py
88.46% <82.35%> (-0.61%)
:arrow_down:

...blems/tasks/label_projection/methods/scvi_tools.py
82.35% <82.35%> (ø)

openproblems/data/tabula_muris_senis.py
83.11% <83.11%> (ø)

...lems/tasks/_cell_cell_communication/_common/api.py
84.76% <84.76%> (ø)

...ms/tasks/_cell_cell_communication/_common/utils.py
86.04% <86.04%> (ø)

... and 151 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[Storage] Blob Typing Final Revision

This PR sets out to accomplish the following:

Check for passing livetest

Address any hanging local MyPy errors

Address any hanging requests

This PR should serve as the final verification step before getting APIView approval and moving towards a release 😄
Resolves #32847

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-storage-blob

/azp run python - storage - tests

/azp run python - storage - tests

/azp run python - storage - tests

/azp run python - storage - tests

/azp run python - storage - tests

/azp run python - storage - tests

/azp run python - storage - tests

/azp run python - storage - ci


Release Tarball formatting

I noticed a couple minor things regarding the release tarballs:

The _. in the file name seems unneccesary (scicat-cli_.2.2.0_Linux_x86_64.tar.gz). Maybe remove the period before the version number?
It would be more convenient to have a top-level directory inside the tarball. This is expected by pmodules and removing it can result in overwriting existing files.

Note that the pmodule buildblock will need to be updated after these changes. In particular, the post_prep function can be deleted.


Update audio-device extension

Description

It adds an action to hide device and show the hidden devices.

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[ ] I checked that files in the assets folder are used by the extension itself
[ ] I checked that assets used by the README are placed outside of the metadata folder

Thanks for your contribution @arthur-fontaine 🔥
Is there a way to re-hide the hidden devices after showing them again, maybe it could be a toggle function instead?
@benvp could you check this?

Thanks for your contribution @arthur-fontaine 🔥
Is there a way to re-hide the hidden devices after showing them again, maybe it could be a toggle function instead?
@benvp could you check this?

Yes it is 100% doable and easy. I can do this this evening or tomorrow.
Btw, do you have an idea about the icon to use? There is a duplicate between "Show hidden" and "Hide this device"

What about Eye and EyeDisabled

Yes that's what it is currently, but there are two new actions:

show/hide device
show/hide hidden devices

For the first one, I used Eye for "Show device" and EyeDisabled for "Hide device".
But for the second one, if we show the same icons, it may be disturbing for the user, no?

What about adding a <ActionPanel.Section>, that might help with the confusion, you can even set a title to make it even more clear, something like this

@arthur-fontaine are you still around?

Yes sorry I didn't see your last reply. I will make the changes in the next days

It should be ok now! Let me know if you see something to change.

I accidentally tried to hide all my devices. Maybe we should add the action to show hidden devices in the list (or a List.EmptyView) since I can't get them back 😂

Like this? (if there is no device, hidden or not, it does not change from before, it should show "No Results")

Hi 👋
Looks good to me, approved 🔥


C4EV3 doesn't work anymore

during a hackathon we found, that the c4ev3 version of the ev3 brick doesn't work. The reason is unclear.
Please write comments about the result of an analysis into this ticket!

This problem appeared to me just recently (15th november 23) during my weekly robotics workshop. I investigated by using several platforms and I think I could locate the problem.
Tested systems

several versions of Win 10 Enterprise and Pro - 22H2 - Builds 19045.3636 & 19045.3208 & 19045.3570
Ubuntu Linux 22.04.3 LTS
Debian 12

tested browsers
Firefox 119.0 & 115.3.1.esr
Chrome 119.0.6045.124 and other recent versions
recent Brave browser
If you need more precise information about the systems, let me know. I stopped writing down details when I found the potential cause.
What does work, what does not?
Open Roberta lab 5.2.6 + "Save as" = works on none of the systems tested (Win10)
Open Roberta Lab 5.2.6 + upload-c4ev3 script (Linux) = does not work, error message during script
Lego Mindstorms app (1.5.2 & 1.5.0) on Win10 = works without problems
Open Roberta lab 5.2.6 + ev3dev uplaod via Wifi / internet = works without problems
uploading old (april 2023) uf2 programs = works on all linux (via script) and Win10 systems (via send to and direct copy to ev3)
last known working setup = 8th november 23 on Win10
I started a weekly robotics workshop where we used Open Roberta Lab. It worked without the issue until Wednesday 8th. On my workshop this wednesday (15th), none of the laptops and/or EV3 bricks did work. When I checked the issue at home, I realized that my other EV3 bricks which had no problems before showed the same problem [see tested systems].
symptoms
The usual upload process (save the program file from browser to brick) does not work. I don't get the beeping sound to confirm the upload. Although there is a "NEPOprog" visible after download (also on systems that did not have any present before), it seems to be transferred incorrectly. Instead of being placed inside a seperate folder (like NEPOprog/NEPOprog.uf2) it seems to be located directly at BrkProg_SAVE , next to the demo program, when I look for it via the EV3 screen. When I try to start the program (NEPOprog) the Mindstorms screen appears, but immidiately stops, so i can't even read the output.
When using the upload-c4ev3 script in linux, i get the following output:
linux console:

:~/Downloads$ upload-c4ev3 NEPOprog.uf2
Extracting UF2 file
UF2 file successfully extracted
Uploading program files ...
(ev3duder output follow)
USB connection established.
upload was successful.
USB connection established.
upload was successful.
USB connection established.
upload was successful.
USB connection established.
upload was successful.
(end of ev3duder output)
(ev3duder output follow)
USB connection established.
../prjs/BrkProg_SAVE/NEPOprog.elf: /lib/libc.so.6: version GLIBC_2.33' not found (required by ../prjs/BrkProg_SAVE/NEPOprog.elf) ../prjs/BrkProg_SAVE/NEPOprog.elf: /lib/libc.so.6: version GLIBC_2.34' not found (required by ../prjs/BrkProg_SAVE/NEPOprog.elf)
../prjs/BrkProg_SAVE/NEPOprog.elf: /lib/libc.so.6: version GLIBC_2.32' not found (required by ../prjs/BrkProg_SAVE/NEPOprog.elf) exec` status unknown.
(end of ev3duder output)
Program files uploaded
Done!
Wait for the robot to beep and then start the program

EV3 screen (while uploading):

executing shell cmd

This message is shown ~ half a second
suggested problem
It looks like a glibc related issue to me. My linux systems use glibc 2.36-9+deb12u3 or 2.35-0ubuntu3.4 respectively, but I assume my laptop's version is not the problem. Since the problem appeared just recently (Open Roberta Lab was released novemeber 14th) and I had no problems before, I suppose this happened due to recent changes.
workaround
Uploading older uf2 programs (april 2023) that I had still around worked just fine with the same laptops and without the error message. Using Open Roberta Lab 5.2.5 might just work.
to do

[ ] check if older version are affected

solved by redeployment


ABI

Options:

WASI seems promising, I need to look more into it.
Make my own?

If you have any questions about WASI, feel free to ask!


Add v and version arguments

I would like to v and version arguments to your program so when the user runs node filename.js v the program will print the current version of the command-line tool.

That would be great thank you.  Please proceed!


Round Image at Notification's Large-Icon

Hello,
I really love this package but I found myself struggling with the Large Icon because I am unable to make it round. I found out that you would normally do this by using an Bitmap ( https://stackoverflow.com/questions/28280615/make-notification-large-icon-round ).
Is there already an way to this?
Thank you in forward 🥇

This plugin will be availabe soon with awesome_notifications 0.7.0

Hello @rafaelsetragni,
I hope this feature will be available soon, I'm currently working on an app that would really benefit from this addition, so far I have been able to alter the source code so I can somehow make it work, but it would be awesome to have it within the package. Thank you and good luck.

is there another way that we pass a string url in large icon parameter and display it , bcz i try to implement it it give me largeicon is invalid error even notification layout big piture is set


[zshelf][0.2.0] - New Package

🤞

Thank you all! 😘

Maybe we want to wrap nvm or asdf or something similar?
asdf could actually be used to create a more generic toolchain image.

Maybe we want to wrap nvm or asdf or something similar?

This likely is what the package should be doing instead of using the npm/node packages.

Well, it's tricky because the node interpreter is required after build to run the app...unless it is packed with pkg or nexe. But then the binary is huge comparatively which isn't ideal on a disk constrained device.

Well, it's tricky because the node interpreter is required after build to run the app...unless it is packed with pkg or nexe. But then the binary is huge comparatively which isn't ideal on a disk constrained device.

Right, so require the node package, but for build time use nvm to pull down a known good one to build node_modules.

Right, so require the node package, but for build time use nvm to pull down a known good one to build node_modules.

Ah, gotcha. Agreed.

Is there anything preventing this from getting merged? Sure the way npm is pulled is not optimal, but same would go if we were to apt install it and it seems to build and run fine for now.

I don't believe so.


Added large_bin_attack which has the same affect as unsorted_bin_attack

Thanks for adding this attack:-)
Large-bins have really been missing from this repo!
Please add the small things I noted for 2.25 and 2.26.
Thanks!

Oh and even though not ideal, please rebase your changes to the build script to the fixes from #86
Thanks!

@m1ghtym0 thanks for checking up on the stuff. Glad to know it was helpful.

Not sure how I can rebase the changes to build script. Github "pull request" is weird lol


Suggestion: Could provide the detailed steps about how to enroll a device to group

In the document, we know how to create an Enrollment Group. But there are no detailed steps about how to enroll a device to the group, or there is no link about that.

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

ID: ee12ff08-add3-63c5-b5ca-2e15e4d6f8d4
Version Independent ID: a4f8563c-0993-ba18-7474-f3b6b24c9124
Content: Provision a device using Azure IoT Hub Device Provisioning Service (.NET)
Content Source: articles/iot-dps/tutorial-net-provision-device-to-hub.md
Service: iot-dps
GitHub Login: @BryanLa
Microsoft Alias: v-masebo

@v-liujxu Thanks for your feedback! We will investigate and update as appropriate.

@v-liujxu You can find the detailed steps to enroll device to a group under How-to guides > Manage device provisioning configuration > Manage enrollments - Portal
Alternatively you can go through the document using the below link directly
How to manage device enrollments with Azure Portal


ci: remove v18

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add some Icons/Symbols to every item

Hi
This is not an  issue just an enhancement to gui request
Can u add Icons or symbols before the output or after the Row  just like this guy did it
https://github.com/BenRoe/MMM-SystemStats
Btw love all ur work using all ur modules in my version
and the great thing i love about ur modules that they r interconnected
Thanks , Keep up the good work

I'm building new module to replace this module. wait a while.

Gr8 thanks for responding
But if possible can u give me a timeline 😁 only if possible so that I can follow up on it

Anyway, you can add/modify the label by manual.
Open translations/en.json https://github.com/eouia/MMM-Tools/blob/master/translations/en.json  (if you are using English) then modify the text.
By example;
"IP" : "IP",

to
"IP" : "<img src="..."> My IP",  // <-- Whatever you want.

Gr8 Thanks
Waiting for new Module


Show base class of python classes

I am testing mkdocs for the documentation of my python project and I am up until now impressed by its combination with mkdocstrings.
I was wondering if would be possible please to have the base class for derived classes displayed in the auto-generated documentation. In other words, if we have a class like:
class DerivedClass(BaseClass):
  pass

I would like to have somewhere in the documentation of DerivedClass something like: "Base class: BaseClass". I am open to any option or workaround to display this information: modifying the templates, etc.
Thank you in advance

Hello, glad to hear you enjoy this project :heart:
It would be a nice addition indeed. This could be relatively easy to achieve in pytkdocs, by returning the parent classes (from __mro__) in the serialized output. They could then be rendered by mkdocstrings. We would add an option which defaults to false.


AN in browser MVP: Provide mechanism for UI to retrieve data from DB

Part of the spike work for AN in browser. Question still remains if it should offer some sort of an internal service, graphQL, or the UI should directly access the pouch

Taking myself off this task, we still need to come up with these APIs but will fall to the backend team when time comes.

Closing this in lieu of epic https://app.zenhub.com/workspaces/augur-5bfed3424b5806bc2bf8d173/issues/augurproject/augur/1330


Default winget source unavailable?

Brief description of your issue
On this Windows 10 install that was just upgraded to Windows 11, winget only shows results from the msstore.
Example output from winget search 7zip:

Failed when searching source; results will not be included: winget
Name                                                                                Id           Version Source
-----------------------------------------------------------------------------------------------------------------------
Zip Quick View - rar 7zip uncompress drag/drop                                            9NKWR8FH731C Unknown msstore
Zip - Real Zip: Unzip & Extract Zip, Rar, 7z, 7zip & More, also a File Compression App (… 9NSXFD2VRPQG Unknown msstore
Unarchiver One Archive File Manager -Compress Extract RAR,7zip,zip,7z,ISO                 9P8405B66PZV Unknown msstore
Open Rar Zip 7Zip Unrar Tar Unzip : Extract & Archives All Files                          9NQ657WKQDXW Unknown msstore
Zip Unlock - Unpack RAR, open zip, 7zip, gzip                                             9N487GM876Q7 Unknown msstore
Easy Unrar - ZArchiver, Unzip & Zip, gzip, 7zip                                           9PLNC0QX58D9 Unknown msstore
9 ZIP - open rar, zip, 7zip, gzip                                                         9NBLGGH3TDHG Unknown msstore
WinRare+ .. preview, extract and create: rar, zip, 7zip, gzip                             9PG2VPDGMJ71 Unknown msstore
8 Zip - open and extract RAR, ZIP, 7zip, 7z, ZipX, Iso, Cab for free                      9WZDNCRFJB33 Unknown msstore

Some steps that were taken:

Install the current release of winget, which has the same version number as what was already installed
winget source reset winget, which actually deleted the winget source, as is noted in #1575
winget source add --name winget https://winget.azureedge.net/cache resulted in the following output:

Adding source:
  winget -> https://winget.azureedge.net/cache
  ██████████████████████████████  0%
Failed to open the added source.
Removing source: winget...
Done

Steps to reproduce
It is unclear how to reproduce this problem on other systems.
Expected behavior

Packages from the winget source would be displayed
The original winget source should be added back to the sources list with winget source add --name winget https://winget.azureedge.net/cache

Actual behavior
Only results from the msstore source are being displayed, as if the winget source is currently unavailable.
Environment
Windows Package Manager v1.1.12653
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22000.282
Package: Microsoft.DesktopAppInstaller v1.16.12653.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

@ItzLevvie: Thanks for your feedback! Unfortunately, I overlooked it due to busy work weeks with lot of e-mail traffic.
I think the main issue was not on the server side, but it is a bug in winget's implementation: In fact, winget is only able to update the winget source, when I am logged in with my administrator account and issue the command in a terminal. However, I have a differently named normal user account, which I use for my day-to-day work. When I open the Windows Terminal as Administrator and issue winget from that terminal, I consistently get an error that it cannot update the winget source. Maybe this is worth its own issue or it is already tracked as an issue?

A big thank-you to @maehne for that last comment - saved us much frustration.


[Bug] move-analyzer - named addresses - vscode

🐛 Bug
While using named_addresses:
[addresses]
hello_blockchain = "_"

The move-analyzer VSCode pacage is throwing error:
Unresolved addresses found: [ Named address 'hello_blockchain' in package 'Resources' ] To fix this, add an entry for each unresolved address to the [addresses] section of /workspace/services/move_learning/apps/move-examples/1_resources/Move.toml: e.g., [addresses] Std = "0x1" Alternatively, you can also define [dev-addresses] and call with the -d flag

and not showing any other code errors in VSCode.
There is no other way to make named addresses working in VSCode.
To reproduce
use any named-address with VSCode move-analyzer package.
Stack trace/error message
Unresolved addresses found: [ Named address 'hello_blockchain' in package 'Resources' ] To fix this, add an entry for each unresolved address to the [addresses] section of /workspace/services/move_learning/apps/move-examples/1_resources/Move.toml: e.g., [addresses] Std = "0x1" Alternatively, you can also define [dev-addresses] and call with the -d flag

Expected Behavior
It handles named-address like any other address, so we can use --named-addresses with cli, eg:
aptos move compile --named-addresses hello_blockchain=default

System information
Docker: aptoslabs/tools:mainnet
Additional context
none

How about hello_blockchain = "0x1234" ? Or is this not what you want ?

It doesnt work, at least for aptos:
Command:
aptos move compile --named-addresses hello_blockchain=default

Result:
Compiling, may take a little while to download git dependencies...
{
  "Error": "Move compilation failed: Unable to resolve packages for package 'Resources': Unable to resolve named address 'hello_blockchain' in package 'Resources' when resolving dependencies: Attempted to assign a different value '0x1234' to an a already-assigned named address '0xf854(...)'"
}

I see. How about hello_blockchain = "0xf854...YOUR DEFAULT ADDRESS"

What is the point of named-addresses then? XD
Yeah it works that way, but I just wanted to report the bug, which is names-addresses is not working correctly with move-analyzer.

I changed move/language/tools/move-package/src/source_package/manifest_parser.rs from line:226 into
if entry_str == EMPTY_ADDR_STR {
    // original codes
    // if addresses.insert(ident, None).is_some() {
    //     bail!("Duplicate address name '{}' found.", ident);
    // }
    // changed codes
    if addresses.insert(
        ident,
        Some(parse_address_literal("0x00").context(
            "Error with parsing stubbed address 0x00"
        )?),
    )
    .is_some() {
        bail!("Duplicate address name '{}' found.", ident);
    }
} else ...

and move-analyzer looks working for now.
Please test if you like.

is the fix be merged ?

no.


Add CI/CD workflow

Mostly just copied from the CN repo.
Here's all I changed:

diff course-content/.github/workflows/notebook-pr.yaml course-content-dl/.github/workflows/notebook-pr.yaml
4c4
<     branches: master
---
>     branches: main
9c9
<   NMA_MAIN_BRANCH: master
---
>   NMA_MAIN_BRANCH: main
41a42
>           python -m pip install --upgrade pip wheel
47,52d47
<       - name: Install dependencies
<         if: "!contains(env.COMMIT_MESSAGE, 'skip ci')"
<         run: |
<           python -m pip install --upgrade pip wheel
<           pip install -r requirements.txt
<
74c69
<           python ci/process_notebooks.py $nbs --execute
---
>           python ci/process_notebooks.py $nbs --check-execution

Note this won't be fully functional until https://github.com/NeuromatchAcademy/nmaci/pull/6 lands.

Workflow succeeded with a test notebook; good to merge @spirosChv
BTW you probably want to enable the repo setting that blocks merges to main without an approving review.


About zoom component for IOS 16

@yudielcurbelo I notice that only IOS 17 have Zoom component, IOS 16 do not show the Zoom?
I use IOS 16 to scan a small QR Code but can not. I hope there should be a zoom for IOS 16 or can I set a default zoom for that? Such as when I turn on the Scanner, it already zoom a bit?

it seems that this:
advanced: [
{
zoom: 2,
},
],
not work in IOS 16? is there a way to make it work? @yudielcurbelo


next major?: support for AsyncFunction / better function-arguments

First, we could support the new async function fn() {}.
And second: better arguments parser, because currently it returns false if you have function like function someFn(a, opts = { foo: 1 }, callback)
This is because function-arguments is pretty simple, intentionally and seems like it doesn't understand "default params", which already landed in the nodejs's core, that's why some of the fs tests were failing.
Or probably not, we have parse-function for more advanced stuff.

@ljharb, let's support both the old-school (a, b, callback) and async function? Or your thing is just for async/await? Cuz... in treality the old school is async function too so :laughing: Oh, i have is-callback-function too i think haahhaha

hm. okay, no problem.
but any ideas on names for the current thing? I may need it some time, and seems like others are using it too and they may not be satisfied with the coming v2.

ah hmm - so one option for them is to just stay on v1 - it won't be npm deprecated or anything - but another possible name is is-asynchronous-function?
Thanks! I'll update and publish a v2 later today.


Rename Tab to GenericTabs and Collapsible to GenericCollapsible

Release Type: non breaking changes
Fixes https://x-team-internal.atlassian.net/browse/XP-2838
Description
Rename the new Tab and Collapsible to GenericTabs and GenericCollapsible respectively.
As part of the AC of the XP-2838 ticket a cleanup issue was created https://github.com/x-team/auto/issues/1820
Checklist

[x] set yourself as an assignee
[x] set appropriate labels for a PR (In Review or In Progress depending on its status)
[x] move respective JIRA issue to the IN REVIEW column
[x] make sure your code lints (npm run lint)
[x] Flow typechecks passed (npm run flow)
[x] Snapshots tests passed (npm run jest)
[x] check cleanup tasks (https://github.com/x-team/auto/labels/cleanup) and take a suitable small one (if exists) in a related area of the current changes
[x] component's documentation (.stories.js file) is changed or added accordingly to reflect any new or updated use cases or variants usage
[x] if any snapshots have been changed, verify that component still works and looks as expected and update the changed snapshot
[x] manually tested the app by running it in several different browsers (Firefox, Chrome, Opera, Safari, MS IE/Edge, etc.) and checked nothing is broken and operates as expected!

Steps to Test or Reproduce

run npm start
go to http://localhost:9001/?selectedKind=UI Components%2FGenericCollapsible
go to http://localhost:9001/?selectedKind=UI Components%2FGenericTabs
make sure to check the Filters demo http://localhost:9001/?selectedKind=UI Components%2FFilters as well as it's one of the components that makes usage of both renamed components

Impacted Areas in Application

GenericCollapsible
GenericTabs

Screenshots
Checkout the Storybook stories for the renamed components and others that makes usage of them

Rebased with a feature branch to remove merge conflicts.


Remove disposable domains that were removed from mailchecker

I ran your disposable domain database against our own and found many false positives for Chinese email domains. I dug and found they'd been removed from mailchecker. So I modified and ran the update script locally to not keep the existing domains, filtered to just removed domains, and then searched the mailchecker history (with git log -p -S domain -- list.json) to verify they had been removed and where.
https://github.com/FGRibreau/mailchecker/commit/17af0c523187b37442896c9b5ab4a6b65fa1cf8d
126.com
163.com
sina.com
sohu.com
vip.163.com
vip.qq.com
vip.sina.com
https://github.com/FGRibreau/mailchecker/commit/fa7696d8cdf4e40d5092f2055c86af5bfd22b80a
disposableemailaddresses:emailmiser.com
https://github.com/FGRibreau/mailchecker/commit/df525b922362e35f58a4a0665ddb6a9b9107135d
yeah.net

Thanks!

I've release 1.2.15 to rubygems with this PR merged.

Thanks!


XFA - Support non-embedded fonts without a Widths entry

some pdf use some fonts which are not embedded or they don't have any width array or don't have any css info (e.g. for standard fonts or Arial).
so add widths arrays for Liberation fonts in order to compute the ones for other fonts in using scale factors array.

/botio test


Want to add some documentation

LICENSE.md
contributing.md
contributors profile section in README.md

Please assign this to me under JWOC

@Sowham-3098 Go for it. Do only LICENSE.md and contributing.md. Feature point 3 is already assigned to implement.


Do not mask ipam driver if no ip config is passed

Fixes libnetwork issue 677
Signed-off-by: Alessandro Boch<EMAIL_ADDRESS>

LGTM

@cpuguy83
Taken care of your comments, PTAL when you get a chance.
Thanks

LGTM

LGTM


test: fixed flaky tests

Description
Fixed below flaky tests

inline_editing_validations_spec.js
Button_onClickAction_spec.js
NavigateTo_spec.ts
GitSyncedApps_spec.js
Basic_spec.js
List4_spec.js
Form_With_CheckBox_spec.js
SmartSubstitutionWidgets_spec.js

Type of change

Flaky test fix

How Has This Been Tested?

Cypress test runs

Checklist:
QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=69dcd84

/ok-to-test sha=742a542

/ok-to-test sha=742a542

/ok-to-test sha=6a47721

/ok-to-test sha=3a4c94a

/ok-to-test sha=c0843a2

/ok-to-test sha=9cdca3c

/ok-to-test sha=e9e8cec

/ok-to-test sha=a85d25e

/ok-to-test sha=6715c53

/ok-to-test sha=7dac11e

/ok-to-test sha=96da6d1


Error: undefined method 'to_i32' for Array(String)

Crystal -v : 1.2.2
From the past two weeks I have been working on crystal language. Recently I decided to write a generic blockchain to not only get a better hang on how blockchain works but also to learn crystal language in depth.
This is my repo: https://github.com/d80ep08th/crystal_study_room/tree/blockchain
I was using this link as a guide : https://www.toptal.com/blockchain/crystal-programming-language-tutorial
After many hours of hacking I found an issue in the main crystal lang repo which is why I am posting this issue here.
In /home/linuxbrew/.linuxbrew/Cellar/crystal/1.2.2/src/hash.cr:226:48
226 | initial_capacity = (initial_capacity || 0).to_i32
^-----
Error: undefined method 'to_i32' for Array(String) (compile-time type is (Array(String) | Int32))

Without reproduction code such report is meaningless.

What if I updated the Readme.md?

Reduced:
Hash(String, String).new initial_capacity: (Array(Int32).new || 10)

In /usr/lib/crystal/hash.cr:226:48

 226 | initial_capacity = (initial_capacity || 0).to_i32
                                                  ^-----
Error: undefined method 'to_i32' for Array(Int32) (compile-time type is (Array(Int32) | Int32))

https://play.crystal-lang.org/#/r/cgmq/edit
Is a case of the Hash constructor's initial_capacity parameter not having a type restriction. I.e. can pass it whatever and since an array is truthy, it tries to convert that to an Int32, hence the error.
@d80ep08th I looked and don't see you using a Hash at all, so if you could try and reduce the code that produces this, that would be great.

Thank you @Blacksmoke16  , that really helped . I pushed new changes yesterday . Everything builds fine but sadly I dont have any output, everything just builds and that is it.  Kemal server doesnt start , I cant tell why. Is there something more versatile than : crystal run src/file.cr -- progress  ?

That would be a breaking change, having a type restriction with initial_capacity : Int32? (or Int?) would yield a better error, with a better documentation.

Thank you @Blacksmoke16 , that really helped . I pushed new changes yesterday . Everything builds fine but sadly I dont have any output, everything just builds and that is it. Kemal server doesnt start , I cant tell why. Is there something more versatile than : crystal run src/file.cr -- progress ?

Your code is stuck in proof_of_work because of Digest::SHA256.hexdigest("(message)") (instead of hexdigest(message))

WOAH WOAH WOAH  @konovod , that totally worked!


MINOR: Move Timer/TimingWheel to server-common

This patch rewrite Timer and the related classes in Java and moves them to server-common module. It is basically a one to one rewrite of the Scala code. Note that MockTimer is not moved as part of this patch. It will be done separately.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Hi @dajac , yes let me take a look first thing tomorrow.

@divijvaidya Thanks for your review. I have addressed all your comments.

Failed tests seem unrelated:
Build / JDK 11 and Scala 2.13 / testReplication() – org.apache.kafka.connect.mirror.integration.IdentityReplicationIntegrationTest
1m 45s
Build / JDK 11 and Scala 2.13 / testOffsetTranslationBehindReplicationFlow() – org.apache.kafka.connect.mirror.integration.MirrorConnectorsIntegrationBaseTest
1m 2s
Build / JDK 11 and Scala 2.13 / testRestartReplication() – org.apache.kafka.connect.mirror.integration.MirrorConnectorsIntegrationExactlyOnceTest
3m 8s
Build / JDK 11 and Scala 2.13 / testReplicateSourceDefault() – org.apache.kafka.connect.mirror.integration.MirrorConnectorsIntegrationSSLTest
1m 29s
Build / JDK 11 and Scala 2.13 / testDescribeAtMinIsrPartitions(String).quorum=zk – kafka.admin.TopicCommandIntegrationTest
6s
Build / JDK 11 and Scala 2.13 / [1] Type=ZK, Name=testNewAndChangedTopicsInDualWrite, MetadataVersion=3.4-IV0, Security=PLAINTEXT – kafka.zk.ZkMigrationIntegrationTest
14s
Build / JDK 8 and Scala 2.12 / testBumpTransactionalEpoch(String).quorum=kraft – kafka.api.TransactionsTest
1m 20s
Build / JDK 8 and Scala 2.12 / [1] Type=ZK, Name=testNewAndChangedTopicsInDualWrite, MetadataVersion=3.4-IV0, Security=PLAINTEXT – kafka.zk.ZkMigrationIntegrationTest
15s
Build / JDK 17 and Scala 2.13 / testBalancePartitionLeaders() – org.apache.kafka.controller.QuorumControllerTest
13s
Build / JDK 17 and Scala 2.13 / shouldWriteLatestOffsetsToCheckpointOnShutdown[exactly_once_v2] – org.apache.kafka.streams.integration.EosIntegrationTest


Dojox Gauges are not Included

Dojox Gauges are not being included into our layer files.  Check for Dashboard theme before building them out?

Hi,
I am having the same error as there:
https://github.com/steveoh/grunt-esri-slurp/issues/38
That is to say:

error(311) Missing dependency. module: esri/dijit/_ObliqueRotationWidget; dependency: dojox/dgauges/CircularGauge
error(311) Missing dependency. module: esri/dijit/_ObliqueRotationWidget; dependency: dojox/dgauges/CircularRangeIndicator
error(311) Missing dependency. module: esri/dijit/_ObliqueRotationWidget; dependency: dojox/dgauges/LinearScaler
error(311) Missing dependency. module: esri/dijit/ObliqueViewer/OVCircularScale; dependency: dojox/dgauges/CircularScale
error (311) Missing dependency. module: esri/dijit/ObliqueViewer/OVCircularValueIndicator; dependency: dojox/dgauges/ScaleIndicatorBase

As suggested on the above link, I tried adding the following to the _app.config.js file but it did not solve the issue:
map: {     '*':  { 'dojox/dgauges': 'dgauges'} }
Any idea ?
Thanks !

OK. Will have a look at it, thanks.
I don't understand though why is it bundled if it is not used in the app ?

For reference, here are the changes that were made for esri-slurp-example project:
https://github.com/tomwayson/esri-slurp-example/commit/673d98859ed76496b2c50b98693aeafaeceaf159

@Biboba Thanks for posting that example. I had a question though: in the example, why are you including a reference to the dgauges fork ( https://github.com/dmandrioli/dgauges.git) instead of the "official" repo (https://github.com/dojo/dgauges)?

I was just referencing the commit for information.
I believe you should ask @tomwayson !

If the issue involves dojo, bower, and/or grunt, you should most definitely not ask me... at least, not if you want any useful information. 😄

@gavinr Is #63 verified to fix this issue?  If so, feel free to close.


Typing into Find text box in Find/Replace loses selection and searches file for the text instead

Godot version:
3.2.3
OS/device including version:
Linux
Issue description:
Typing into the find part of the Find/Replace option loses your selection selection and searches entire file for the text.
When you are only searching for text then highlighting the first occurrence is fine, but when you want to search/replace and Selection Only is highlighted this behaviour is wrong.
Steps to reproduce:

Select some lines of text
Press CTRL-R (selection only should be ticked)
Type something into the top 'find' box
Your selection is gone and the IDE jumps to the first occurance

What you have to do is a sequence that isn't 'normal' by remembering to bring up the search bar, entering the find text then selecting the text.
I expect this is because the 'find' and 'find/replace' use the same code.
Minimal reproduction project:

cc @Paulb23

Probably need to skip this if selection only is ticked.


brand_new_login_&_signup

Felt like the login and signup page of sudoEMR need some new design for effective user experience.

Please add focus to the username input field @ysquareimperial

And please @ysquareimperial be removing unused component.


Fixing src bugs

in the handle files:
1-The callback [:653] provided to the map [:653] call on this array [:653] should return a value, otherwise map [:653] will always return an array of undefined values. If the desired behavior is to just iterate through all elements [:653] , then consider using forEach [:475] instead.
and declaring variables with let keyword would be beneficial and will prevent future scope bugs.
2- I fix it to let patterns = new Set(patternServer)
but patterns variable is only updated but do not seem to be ever queried.so I suggest changing it to let patterns; [:172]
in handle data:
3-Two subsequent branches of an if statement has duplicate bodies ( here [:871] and here [:874] ). This may be caused by a copy-paste error.  merging the branches would be a solution to avoid code duplication.

Thank you so much for the contribution, @Abderrahmenla! I accept your code and I made some refactorings. Added ESlint and Prettier as well. I'll add more comments yet.


read/elf: Exclude ARM mapping symbols from is_definition()

I agree this needs fixing, but I don't think we want to fix it with special case logic like this.
There's some previous discussion in https://github.com/gimli-rs/object/pull/447, but I haven't gotten back to it yet to decide what to do.

I think most other tools just hardcode the exclusion for mapping symbols though, e.g. LLVM: https://github.com/llvm/llvm-project/blob/6b3470b4b83195aeeda60b101e8d3bf8800c321c/llvm/include/llvm/Object/ELFObjectFile.h#L742
It seems like the simplest approach and I think this is appropriate; mapping symbols are a bit of a hack (fortunately well defined by the various ABIs) and need to be countered with hacks in the consuming programs.

See the last commit of #601 for an alternative fix for this.


LeCroy: add high speed serial trigger support (8B10B/NRZ/64b66b)

Known trigger types (SDA816Zi):

C64B66B
C8B10B


Issue #7190 fixed

Issue Reference
#7190

@gitesh-webkul
Not fixed, still getting the same issue.

@gitesh-webkul
Not working, please find the below screenshot.

Still not fixed, please find the below screenshot.
 Please find the screenshot Here

@gitesh-webkul
Not working, please find the screenshot Here

Closing this pull request. another pull request is created for the same: https://github.com/bagisto/bagisto/pull/7312


indents in formatted text, such as markdown, get cleared when starting a new chat or sending a new message

Expected behavior: The indents stay
Observed behavior: the indents get removed, unless editing a message
output.webm

Please update to the latest.


Fix nutcracker aim-walk on clients

Due to the setting of EnemyAI.inSpecialAnimation to true in NutcrackerEnemyAI.AimGun, clients would not sync the hosts movement throughout the nutcrackers aim, and thus would not see the aim-walk. Patched by setting it back to false.
Additionally, a second patch to improve the sync accuracy of the nutcracker host-side by halving the EnemyAI.updatePositionThreshold to 0.5 improves the smoothness of the aim-walk for clients, as the slow movement would result in a slightly inconsistent speed due to the somewhat large position threshold to sync.

the setting of EnemyAI.inSpecialAnimation to true in NutcrackerEnemyAI.AimGun

Wouldn't that cause the same issue on the host too? Since all clients including the host run AimGun which sets inSpecialAnimation to true for around 2 seconds then sets it back to false 🤔

the setting of EnemyAI.inSpecialAnimation to true in NutcrackerEnemyAI.AimGun

Wouldn't that cause the same issue on the host too? Since all clients including the host run AimGun which sets inSpecialAnimation to true for around 2 seconds then sets it back to false 🤔

inSpecialAnimation only prevents the syncing of the transform's position to the server position for clients - for host it does nothing, though some other entities like Masked do use the value of this variable in their logic.


Publishing helper modules seperately?

Have you considered publishing the jdict and jstrutils  modules as separate nimble packages?  I think they are very universal and I would like to import them in other projects.

 I wouldn't mind having just a dependency on Karax for now.

I think I might move them to Nim's stdlib instead. Any objections?

Also, jsrutils misses following:
proc `&`*(s: float): cstring {.importcpp: "((#)+'')", nodecl.}

You may want to add it in the same time.

Seems like this didn't make it into stdlib. I'm going to close this for now due to inactivity. Please feel free to re-open and comment if you want to discuss further!


Syntax error is thrown if there is any commented code

In my custom JS Console,

If we write a commented code in the last line, we are getting syntax error like this

But, if we write it before the last line, it is working fine

It would be of great help if you suggest a solution for this

Sounds like you've already got a solution for it, no?

:smile:  We are giving this to end users in our code playground.
My question is,
How to handle the above two cases if the user tries to write the code that way. Whereas, they are working fine in Chrome Dev Tools.

I strongly suspect a parsing fault in the jsconsole ast logic. Happy to take a PR to resolve this, and I can publish to npm so it's consumable again

As per our use case, we resolved this issue by just appending a newline character to the command after it is sent to the run method in the run.js file.
It is working as expected.
I Will keep a PR if the workaround is okay. :slightly_smiling_face:


Should have a way to evaluate variables in docstrings

For example, for update_action_button, the docstring includes this:

    Note
    ----
    {note}

The {note} is replaced with a string by a decorator on the function, which looks like this:
_note = "This is a note that is inserted into the docstring of multiple functions."

@doc_format(note=_note)
def update_action_button(
    ...

Code here: https://github.com/rstudio/py-shiny/blob/d2eecc35a0ed21ef3d8e1b23dfd28ee09a2a2a6d/shiny/ui/_input_update.py#L73-L104
The resulting output in the Sphinx docs like this (https://shiny.rstudio.com/py/api/reference/shiny.ui.update_slider.html):

But with quartodoc, it just prints "{note}", probably because the decorator isn't being applied (https://machow.github.io/quartodoc/examples/shiny/reference/ui.update_navs.html):

We don't necessarily have to insert the string at package load time using the @doc_format decorator, but we will need to have some way of achieving the same result.

Working on this right now. I'll add an option for quartodoc to dynamically import the function, and use that docstring.
This way the docstring will be equivalent to...
from shiny.ui import update_action_button

update_action_button.__doc__


json resopnse

where is the links of json responses?

I don't understand your question. Can you please elaborate?

I want to add other country to app
On Jul 19, 2016 10:06 PM, "Felipe Lima"<EMAIL_ADDRESS>wrote:

I don't understand your question. Can you please elaborate?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/EliFUT/android/issues/21#issuecomment-233708512, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AIhIpx9ORF5FqO6MLQoAGDpGP0LTv8uuks5qXQsygaJpZM4JPkRc
.

that comes from the API backend database. check this repo: https://github.com/EliFUT/backend


Number of ports reported by build_simulation_project of edb are always zero

Count of number of ports created by build_simulation_project is always reporting zero to logger even though 3DLayout displays them under excitations when we open the created cutout. Review the get_ports_number of edb_core.hfss.

fixed


[Bug] Tailwind animate classes not found

Version
1.1.0
Reproduction link
https://codepen.io/brunolorente/pen/yLpoOGe
Operating System
macOS
Device
apple m1 pro
Browser & Version
chrome Versión 99.0.4844.83 (Build oficial) (arm64)
Steps to reproduce

I'm trying to use css classes like animate-spin but are not found

What is expected?
Be able to use tailwind's animation classes
What is actually happening?
These classes are not found

Solution
Additional comments
I'm not an expert in tailwind but these classes are needed

My fault


Enable pipeline builds with user specified ci & builld scripts

The base pipeline build needs to pass down the USER_REMOTES to the build job when useAdoptBuildScripts=false, so that the RepoHandler.checkoutUser..scripts() resolves the correct repo.


Prediction ID must be different from task ID, otherwise labeling doesn't start

Description
Not sure if it's a bug or I am just using it wrong, but here's the case: when I use default bbox labeling config.xml,  very simple predictions for pre-labeling and I set task ID equal to 0 and prediction ID equal to 0, I get some JS errors in labeling tab and labeling is not working.
To Reproduce
Steps to reproduce the behavior:
label-studio init proj

config.json is default;
config.xml:
<View>
  <Image name="image" value="$image"/>
  <RectangleLabels name="label" toName="image">
    <Label value="Airplane" background="green"/>
    <Label value="Car" background="blue"/>
  </RectangleLabels>
</View>

tasks.json:
{
    "0": {
        "id": 0,
        "data": {
            "image": "https://htx-misc.s3.amazonaws.com/opensource/label-studio/examples/images/nick-owuor-astro-nic-visuals-wDifg5xc9Z4-unsplash.jpg"
        },
        "predictions": [
            {
                "id": 0,
                "result": [
                    {
                        "from_name": "label",
                        "to_name": "image",
                        "type": "rectanglelabels",
                        "value": {
                            "height": 10,
                            "rectanglelabels": [
                                "Car"
                            ],
                            "rotation": 0,
                            "width": 10,
                            "x": 10,
                            "y": 10
                        }
                    }
                ]
            }
        ]
    }
}

Then:
label-studio start proj

Expected behavior
When entering "Labeling" tab, there is a loading pic at the top, which goes over and over. Also, when opening developer's console in browser, the error rises:
mobx-state-tree.module.js:3128 Uncaught (in promise) Error: [mobx-state-tree] No matching type for union (string | null?)
    at ut (mobx-state-tree.module.js:3128)
    at t.instantiate (mobx-state-tree.module.js:5435)
    at mobx-state-tree.module.js:4875
    at mobx-state-tree.module.js:4997
    at Array.forEach (<anonymous>)
    at t.forAllProps (mobx-state-tree.module.js:4997)
    at t.initializeChildNodes (mobx-state-tree.module.js:4874)
    at new t (mobx-state-tree.module.js:1277)
    at Ye (mobx-state-tree.module.js:2913)
    at t.instantiate (mobx-state-tree.module.js:4866)
    at t.e.create (mobx-state-tree.module.js:1836)
    at be (mobx.module.js:928)
    at t.r (mobx.module.js:915)
    at t.create (mobx-state-tree.module.js:1909)
    at be (mobx.module.js:928)
    at t.r (mobx.module.js:915)
    at i (CompletionStore.js:781)
    at Object.addPrediction (CompletionStore.js:788)
    at be (mobx.module.js:928)
    at r (mobx.module.js:915)
    at mobx-state-tree.module.js:2593
    at Se (mobx-state-tree.module.js:2456)
    at Object.r (mobx-state-tree.module.js:2483)
    at AppStore.js:325
    at Array.forEach (<anonymous>)
    at Object.initializeStore (AppStore.js:325)
    at be (mobx.module.js:928)
    at r (mobx.module.js:915)
    at mobx-state-tree.module.js:2593
    at Se (mobx-state-tree.module.js:2456)
    at Object.r (mobx-state-tree.module.js:2483)
    at lsf-sdk.js:115

If you change ID in prediction from 0 to 1, labeling is possible just as usual.
Screenshots

Environment:

OS: MacOS Catalina 10.15.7
Browser: Safari and Yandex Browser
Version 0.7.5.post2

Try to make ids as strings everywhere. (0 => "0").
You need to use different ids within any task elements., it's our frontend requirement.


port of LongMaxUpdater

Hi Iulian, I've ported the LongMaxUpdater class from Doug Lea. Can you have a look at it and possibly merge and push to nuget if you think it's good enough?
Thanks, Jorrit

Not sure why the build fails (https://ci.appveyor.com/project/etishor/concurrencyutilities/build/1.0.23). Seems to have to do with nuget.


Infinite loop in upload file progress bar

I want to write simple local ply files viewer with viser. It must just open ply file in local machine. Only way that i found to achieve it - use upload button for filename (and save file in temp directory). After "uploading" file progress bar loop infinitly. Seems like files upload to fast) What is correct way to fix it?

Hi @SmthFail, thanks for reporting this bug!
It looks like the file upload notification was broken by an overzealous find-and-replace in #293. I've implemented a fix in #354.
It looks like this error also would have been caught by the type checker if we were using a newer version of some dependencies; I updated those to prevent future regressions.

@brentyi Thanks. Now it work correct!


HDDS-5910 Add additional verification for S3 Auth.

What changes were proposed in this pull request?
Ensure that if the client has asked for S3 style auth, each request
sent has S3 Auth information set.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-5910
How was this patch tested?
Set the validation to trip always and insure it catches the missing auth.

Adding annotations will not insure that the insertion will occur at the time of invocation of the endpoint without the need to modify each invocation. For now not adding that change. No additional unit tests are needed for signature parsing for now.

I think our discussion is to add method level annotation. But this fix will catch issue, we are safe guarded in a way. But I still believe lets have a Jira open to add during method invocation, instead of Object Creation.

I do not see an easy way without intercepting the invocation of all the HTTP operations for the annotations to take effect. We can discuss offline but it seems a bit redundant with this change.

Adding annotations will not insure that the insertion will occur at the time of invocation of the endpoint without the need to modify each invocation. For now not adding that change. No additional unit tests are needed for signature parsing for now.

I think our discussion is to add method level annotation. But this fix will catch issue, we are safe guarded in a way. But I still believe lets have a Jira open to add during method invocation, instead of Object Creation.

I do not see an easy way without intercepting the invocation of all the HTTP operations for the annotations to take effect. We can discuss offline but it seems a bit redundant with this change.

I see one way is adding annotation to each method which handles HTTP requests or use aspectj library which we are discussing offline with @avijayanhwx. But with the proposed fix it is good, we can catch errors.

Adding annotations will not insure that the insertion will occur at the time of invocation of the endpoint without the need to modify each invocation. For now not adding that change. No additional unit tests are needed for signature parsing for now.

I think our discussion is to add method level annotation. But this fix will catch issue, we are safe guarded in a way. But I still believe lets have a Jira open to add during method invocation, instead of Object Creation.

I do not see an easy way without intercepting the invocation of all the HTTP operations for the annotations to take effect. We can discuss offline but it seems a bit redundant with this change.

I see one way is adding annotation to each method which handles HTTP requests or use aspectj library which we are discussing offline with @avijayanhwx. But with the proposed fix it is good, we can catch errors.

Yeah. I don't think bringing in a whole new library to deal with this is prudent at the moment. The current check with protocol check PR https://github.com/apache/ozone/pull/2822 should be sufficient and straightforward

Adding annotations will not insure that the insertion will occur at the time of invocation of the endpoint without the need to modify each invocation. For now not adding that change. No additional unit tests are needed for signature parsing for now.

I think our discussion is to add method level annotation. But this fix will catch issue, we are safe guarded in a way. But I still believe lets have a Jira open to add during method invocation, instead of Object Creation.

I do not see an easy way without intercepting the invocation of all the HTTP operations for the annotations to take effect. We can discuss offline but it seems a bit redundant with this change.

I see one way is adding annotation to each method which handles HTTP requests or use aspectj library which we are discussing offline with @avijayanhwx. But with the proposed fix it is good, we can catch errors.

Yeah. I don't think bringing in a whole new library to deal with this is prudent at the moment. The current check with protocol check PR #2822 should be sufficient and straightforward

Yes, as said for now we should be good. Lets create a Jira if any one interested can pickup or we can work on when needed if we see any issue.

Thank You @kerneltime for the contribution


docs: changes in joss paper according to reviews

Ref: https://github.com/openjournals/joss-reviews/issues/6917#issuecomment-2230677031

@ChrisRackauckas, can this be merged?


Velero kubernete addon ...manage_via_gitops does not work

Description
I cannot install Velero using argocd. It always installs via helm
I am using the following...
argocd_manage_add_ons  = true
Other modules are working fine and install via argocd eg externaldns
The velero module  is missing this line...
manage_via_gitops = var.manage_via_gitops
Versions

Module version : v4.14.0

Terraform version:  1.1.9

Hey, if you are referring to using https://github.com/aws-samples/eks-blueprints-add-ons , velero is not there, that's why it's not being installed for you and others.

https://github.com/aws-samples/eks-blueprints-add-ons/pull/113 created, once merge feel free to give it another run

I'm currently doing a PoC with blueprints, managing add-ons via GitOps and this is still a present problem.
Even when adding locally manage_via_gitops = var.manage_via_gitops to https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/main/modules/kubernetes-addons/velero/main.tf I end up with Velero in both ArgoCD and a Helm release that Terraform tries to apply.
Moreover, I end up with the following error messages, despite the fields being specified, whenever I sync Velero with Argo. If I Sync with Apply Only, there are no issues being reported


[cxx-interop] Bail if the type a reference type is wrapping is unkown.

If we can't find the type a reference type is wrapping (the pointee type), then don't import that type instead of crashing later.

@swift-ci please smoke test and merge.

@swift-ci please smoke test Linux platform.

@swift-ci please smoke test Windows platform.

@swift-ci please smoke test Linux platform.


[Bug] create a new chart, the data is not cleared.

First chart

Create a new chart:

the data from the previous chart still exists.

solved


head models

Hi, thanks for the good work! is this model only compatible with the custom head model, or is it able to work with other models? i.e. 3DMM/FLAME?

Hi, thanks for your interest in our work! Though our current model is only compatible with the custom head model, placing the generated hairstyle onto 3DMM/FLAME is easy as we only need to compute the rigid transformation between our head mesh and 3DMM (like what we did for single-view hair reconstruction). To compute the rigid transformation, you can either write your own code or use the Align Tool in MeshLab to manually do it.


Fix Limit issue and merge with new BwTree

This PR optimizes the sql statement with order by limit 1 and also merged ziqi's new BwTree. A simple performance test is done and the result shows that the performance is improved a lot! Let's test the true performance after merging this PR with master.

Coverage decreased (-0.3%) to 65.148% when pulling fbc18bd14ac3f424f01c9f9d478c0ae0d64ec07e on Michael-Tieying-Zhang:master into 0f93bb804a9d0d305c513201e2147a246013f55f on cmu-db:master.

#457


display total no of downloads and ranking

This is to suggest/ask for the feature on UI which is to display total no of downloads and ranking of the project like in npm or github. It will be a good feature.

Download count is planned #787
Rating is discussed #991

Thanks


Fix the tool link for Kargo docs

Signed-off-by: Bogdan Dobrelya<EMAIL_ADDRESS>

This change is

@chenopis PTAL

@bogdando thanks for the fast response! Cheers


Use plt.rc_context for default styles

[x] Closes #7313
[x] Tests added
[x] User visible changes (including notable bug fixes) are documented in whats-new.rst

Seaborn did something similar as the original but had a little check if the marker was filled or not:
https://github.com/mwaskom/seaborn/blob/bf4695466d742f301f361b8d0c8168c5c4bdf889/seaborn/relational.py#L563-L570

This has stalled for long enough. I'll merge this at the end of next week unless someone disagrees, at least it is an improvement to the bugged main.


px4fmu-v4: Fix for HIL Unable to Set Control Surfaces. Fixes #5651

Issue Description
The sensor_rest command added to the sensors.rc startup script for px4fmu_v4 hardware was leaving the /dev/pwm_output0 driver open.  (See https://github.com/PX4/Firmware/commit/021f0840ae7f724bfdfd317dda3d57362017b835)  This would prevent the pwm_out_sim module from registering as a simulated driver.  The result would cause the system to properly boot in HIL mode, but you would not be able to set any control surfaces.
Since sensor_rest and peripheral_reset are only used on initialization, the drivers can be shut down after they have performed their reset functions.
Testing
This has been tested and verified in HIL with the following:
Hardware
Pixracer Based Board
Only USB Used for Connection (No Sensors)
Software/Firmware:
Windows 10
XPlane 10.51r2 (build 105101 64-bit)
XPlane Model: HiLStar17
QGroundControl: 3.1.3

Can you instead use the condition (g_fmu == nullptr) to determine before the fmu_start() call if FMU is already running or not and only stop it if it wasn't running before?
Running a sensor reset in a fully booted system otherwise has an unintended side effect.

@LorenzMeier I made the requested changes.
I didn't see anywhere that the reset_sensors and reset_peripherals were called other than the one line mentioned in the commit.  But I agree that it would be a nasty surprise later if someone tried to use a reset in the script after an initialization.

Thanks, perfect now!


PiMon API

Is it possible in the future to integrate an API (x86 DLL, TCP/IP, or similar) to your PiMon software?
Background: I would like to display the PiMon values in my own Smart Home HMI.
Thanks for your help!

PiMon get its values via IOCTLs to the RPIQ driver, the code is in data.c.

Thank you for your information. Worked fine :)


Cherry-pick #50724 to release-4.8

Cherry-pick of #50724.

@jakebailey Is this still going into 4.8.4?

I think so, yeah. I don't think I opened any PRs that I wouldn't want in 4.8; this one's a crash.


[wallet] Attempt to fix e2e

Description
Added misisng env variable and removed the always-passing return. Test are still flaky due https://github.com/wix/Detox/issues/1723 but passing consistently on the second attempt.
Tested
itself.
Other changes

Related issues

Fixes #1659

Backwards compatibility

stale


use_vendor_multimodal_model

Bug Description:
The vendor_multimodal_model_name parameter does not seem to work as expected when using the openai-gpt4o model and vendor_multimodal_api_key

can you say more about what the error is? would help us debug

can you say more about what the error is? would help us debug
[llama_parse]with vendor_multimodal_model_name not work return empty sting when i use vendor_multimodal_model_name
i do formData.append("vendor_multimodal_model_name", "openai-gpt4o");
formData.append("vendor_multimodal_api_key", process.env.OPENAI_API_KEY)

@almasri-ahmad , we can't reproduce the issue on our side.
Did you specify use_vendor_multimodal_model=True?
Can you maybe share an Job ID for us to triage?


kratix init from operator

Context
kratix init from an operator. Output of the command will be a working promise with api, dependencies, and resource configure workflow image generated based on the input operator that people can kubectl apply without any further customization.
Design
TBD
Tasks

[ ] Implement
[ ] Consider what's a reasonable e2e integration test and what operators we want to use for testing
[ ] Doc

We are not sure if we want split or no split on the kratix cli; for simplicity, this story will only output a single promise.yaml


fix: German translations and order of disciplines

I fixed some of the german translations and added missing ones.
I also fixed the order of the disciplines/departments, so they correspond with the character sheets (see also #25)

Sorry, there seems to be a problem with selecting the disciplines. I'm investigating.

Also fixed #97

All of these fixes are potentially great, but I would recommend splitting this PR into multiple different ones, so that each specific change can be reviewed and accepted separately by the maintainer.

A german translation PR
The damage calculation fix PR (or review the one already submitted)
The discipline ordering (including an explanation for the order change)
Fixing the smallcraft name dropdown

The explanation for the discipline ordering is simple: they were sorted alphabetically by the English names. With translated names they were out of order. Also the the tab order wasn't right and most of all the order didn't match with the official character sheets and source books. And that last part is a real pain in the behind.
Furthermore I don't think this repository is maintained anymore. This PR is almost a year old and this is the first feedback i've gotten. Don't take this the wrong way, but maybe the maintainer can weigh in on this.

It looks like some merge conflicts have been introduced by the inclusion of bug fixes for the 2e support in v1.2.1. Can these changes be reviewed and see if all the changes made in this PR are still necessary?


Unable to take "INTEGRAL" of a math-function

We store voltage and current in our influx cluster.
We can calculate power like this:
SELECT voltage*current as power FROM <db>.<rp>.<meas> WHERE <series-tag>
We want to be able to calculate energy like this:
SELECT integral(voltage * current) as energy FROM <db>.<rp>.<meas> WHERE <series-tag> and <time>
However such a query gives the followig:
ERR: SELECT integral(current * voltage) FROM <db>.<rp>.<meas> WHERE <series-tag> [panic:interface conversion: influxql.Expr is *influxql.BinaryExpr, not *influxql.VarRef]
A more standard query such as:
SELECT integral(voltage) as energy FROM <db>.<rp>.<meas> WHERE <series-tag> and <time>
works a expected.
Is it possible to achieve the result I want?
Is it possible to do math inside the new INTEGRAL function?

The way to do this "now" would be to create a continuous query.
From https://docs.influxdata.com/influxdb/v1.3/query_language/continuous_queries/ it might be something like this:
CREATE CONTINUOUS QUERY energy ON
BEGIN
SELECT current * voltage INTO power FROM .
then you can do
SELECT integral(power) ...
Not a very intuitive way to solve the problem but does it work???

Unless I'm mistaken this would create ~50% more data in the database in the form of a measurement called power.
For any "one-off" style analysis we'd have to have created the CQ in advance which seems a bit inflexible, additionallly we're using the clustered version of  Influx and therefore paying by the Gb so a solution that doesn't involve creating extra data that we don't strictly need would be ideal.
Is there any plan to allow mathematics inside these functions?

Not if you then created a retention policy to limit how long the data is held for to the length of time for 1 or 2 measurement intervals.
Of course this is a lot of complexity that would be nice to not have to deal with.

Even that wouldn't work as I want to be able to query the energy as well as raw current and voltage at any time so having the series continually delete itself would be unhelpful.
I think mathematics within measurements would be a neat way to solve this issue.

So I haven't fully read the issue (just skimmed it) so I apologize if this isn't helpful (and promise to read it more fully), but you can likely use subqueries for this. Part of the reason why we didn't add integral(max(value)) was because subqueries existed after we implemented integral while subqueries didn't exist when we were adding things like derivative().
If you need to do math like current * voltage, I think you can also do that with subqueries. I think something like this: SELECT integral(power) FROM (SELECT current * voltage as power FROM <db>.<rp>.<meas> WHERE <series-tag>)

@jsternberg Yes this works.  Perfect thank-you.


Could not find artifact com.spotify:dockerfile-maven-plugin:jar:1.7.3 in..

I am trying to use this plugin but it can not find jar file from the repository
may be something broken ?
Downloading: http://repo.spring.io/milestone/com/spotify/dockerfile-maven-plugin/1.7.3/dockerfile-maven-plugin-1.7.3.pom
Downloading: https://repo.maven.apache.org/maven2/com/spotify/dockerfile-maven-plugin/1.7.3/dockerfile-maven-plugin-1.7.3.pom
[WARNING] The POM for com.spotify:dockerfile-maven-plugin:jar:1.7.3 is missing, no dependency information available
Downloading: http://repo.spring.io/milestone/com/spotify/dockerfile-maven-plugin/1.7.3/dockerfile-maven-plugin-1.7.3.jar
Downloading: https://repo.maven.apache.org/maven2/com/spotify/dockerfile-maven-plugin/1.7.3/dockerfile-maven-plugin-1.7.3.jar
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 2.015 s
[INFO] Finished at: 2017-08-14T10:38:40+03:00
[INFO] Final Memory: 16M/217M
[INFO] ------------------------------------------------------------------------
[ERROR] Plugin com.spotify:dockerfile-maven-plugin:1.7.3 or one of its dependencies could not be resolved: Could not find artifact com.spotify:dockerfile-maven-plugin:jar:1.7.3 in spring-milestones (http://repo.spring.io/milestone) -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/PluginResolutionException

There is no version 1.7.3 yet of this plugin, the most recent release is [1.3.4].
We upload releases to Maven central, I'm not sure if repo.spring.io is caching things from there or not.


6hundreds/23803/instrumented injected service access

Fixes https://github.com/gradle/gradle/issues/23803
As soon as we can inject non-serializable Gradle model classes (like Project or Gradle), then it is possible to have an access to it at execution time.
This PR is augmenting injected service getter with epilogue like
IntstrumentedExecutionAccess.notifyInjectedServiceAccess(ServiceLookup, Class, String), with reporting a problem as a result

@bot-gradle test acc

I've triggered the following builds for you:

AllConfigCacheTestsPullRequestFeedback build

👍 LGTM! A little more user friendly report might also include the injected property name, but that does not seem to be necessary. By the way, does the report include the stack trace? If it does, then it should already provide a way to find the violating property.

@h0tk3y No, it seems like report doesn't contain a stacktrace. Should we mind to add it?

@bot-gradle test acc

@mlopatkin PTAL

Your PR is queued. See the queue page for details.

@bot-gradle test and merge

I've triggered a build for you.