Using @bazel/runfiles doesn't work

I'm using bazel/runfile to resolve files inside my build worktree and it doesn't work.
By investigating a bit deeper I realized that both envs in js_binary are not set:
- BAZEL_TARGET
- BAZEL_WORKSPACE

Setting them using the envs property on js_binary almost works!
    env = {
        "BAZEL_TARGET": "$(TARGET)",
        "BAZEL_WORKSPACE": "$(WORKSPACE)",
    },

For some reason the TARGET is resolved as @//mytarget:bin making it not work with @bazel/runfiles
(See here: https://github.com/bazelbuild/rules_nodejs/blob/stable/packages/runfiles/runfiles.ts#L52)
I've now hardcoded my target and everything works well, but it seams like a big hack to me.
Hope you can fix it in a near future :)

Thanks for the link, I think that's a bug in the logic in runfiles.ts, @//mytarget:bin should be treated the same as //mytarget:bin.
We should maybe ALSO figure out why the $(TARGET) expands that way.

What's your bazel version? 5.3.0 just came out and it changed the label syntax a bit. I'm curious if that makes any difference.

I'm using 5.2.0, will try with 5.3.0 later today :)

What about the ENV variables not being set properly by js_binary ?

I'm using 5.2.0, will try with 5.3.0 later today :)

I've tested with 5.3.0 and it's still preceded with an '@': BAZEL_TARGET: '@//proxy:bin'

https://github.com/bazelbuild/rules_nodejs/pull/3542 plus https://github.com/aspect-build/bazel-lib/commit/b5653b6037cf993acc889cfd8430ce000a167812 should fix it, pending a release of both

Perfect thanks!


VehicleWheel3D Sinks Through Gaps Smaller Than Itself & VehicleBody3D Slides on Slopes

Tested versions
4.3-stable
I'm pretty sure this is the case across all Godot 4 versions. Not sure about Godot 3.
System information
System info not needed.
Issue description
When you drive a VehicleBody3D through a gap smaller than its wheels, they sink down and get stuck. Sometimes forever, and then you have to restart the game.
When you drive a VehicleBody3D to a slope, then brake, it's supposed to stay like that. In this case however, it slides down until it's on a perfectly flat surface, then loses momentum and stops.
I've thought of reporting these separately but since these nodes are related, I think it's fine.
Changing physics engine to Jolt or Rapier does not fix these issues.
Steps to reproduce
1- Download the project and open it up.
2- Press play. You can drive the vehicle with WASD or Arrow keys and look around with the mouse.
3- Drive towards the higher slope right in front of you. Then stop driving. Vehicle slides down, which is not realistic. You can hold Space key while doing it to apply the brakes and watch it slowly slide down.
4- Drive towards the lower slopes and drive slowly over them. I don't know how to describe what will happen.
Minimal reproduction project (MRP)
vehiclebody_sliding_down.zip

When you drive a VehicleBody3D through a gap smaller than its wheels, they sink down and get stuck. Sometimes forever, and then you have to restart the game.

VehicleWheels work with infinitely thin raycasts, which means they can get stuck between small gaps. In most games, this is avoided at level design time by adding clip brushes in thin gaps so that the wheels can't get in. It is possible to use multiple raycasts or cylinder-based collision for the wheel, but it's significantly more complex and demanding in terms of performance.

When you drive a VehicleBody3D to a slope, then brake, it's supposed to stay like that. In this case however, it slides down until it's on a perfectly flat surface, then loses momentum and stops.

This is likely due to gravity being applied when the vehicle is on the ground. Increasing Physics Ticks per Second in the Project Settings will decrease this effect (and generally make vehicle simulation more accurate), at a performance cost. For most racing games, I'd increase this above the default 60 Hz to get more reliable simulation, especially at high speeds.

Increasing Physics Ticks per Seconds did not work for me. Still the same behavior. I have done the same test in Unreal and Unity. It's fine there.

I would love a Vehicle system with more customization options to combat these errors. I'm in the middle of making my own vehicle class and it feels a bit silly when the built-in class is right there, and does everything the same (if not better), but it won't work for me because I cant just change it to a shapecast.
Unless I can, and I just don't know how :P

@p-planet No you can't. The built-in class is a port of Bullet's, and as @Calinou mentioned it's mostly meant for arcade games

First part is not an issue, but an expected outcome of how the VehicleWheels are implemented (basically just hover-rays)
Second is a duplicate of https://github.com/godotengine/godot/issues/88998
which I am trying to workaround for weeks now.
The only working solution I found was to rely on the VehicleBody to fall asleep, which surprisingly, it does, even if it continues to slide.
The sleep state of the vehicles Rigidbody can act a perfect trigger for when you want to freeze or activate your vehicle manually however you like, preventing it from sliding


Rock talk refactoring

I try to greatly simplify the rock dialogues. It was a few hard-coded settings.
Now I try to make sure it is easy to add new dialogues, new graphics - only working with JSON files, without creating a dozen custom views.

OK, we have a working Rock Talk!
Right now:

in rock_talk.json you can add an entry (id should be unique, dialogue is an array of 1-2 strings)
levels there is an optional field rockTalkId in which you can use the id above

For concrete examples, see classicLevels.json.
Also, added Mach-Zehnder interferometer:


Allow building with vector-0.13.*

Fixes #8.

Forgot to reply to original ticket. Is a Hackage revision more convenient or is a new minor release better for you?

A Hackage revision would suffice. Thanks!

@fumieval seems no-one that's left in Tsuru has package maintainer access, can you make the cabal revision and/or add someone from Tsuru to maintainers on Hackage?
I guess I should also add you to this repo as collaborator.

@Fuuzetsu I thought I added maoe as a maintainer, but not this one apparently... I've just added MateuszKowalczyk and MitsutoshiAoe as maintainers.
I made a Hackage revision too: https://hackage.haskell.org/package/vector-th-unbox-0.2.2/revisions/
I missed the invitation and it has expired; can you re-invite me?

@Fuuzetsu I thought I added maoe as a maintainer, but not this one apparently... I've just added MateuszKowalczyk and MitsutoshiAoe as maintainers.

I see it, thanks!

I made a Hackage revision too: https://hackage.haskell.org/package/vector-th-unbox-0.2.2/revisions/

Great.

I missed the invitation and it has expired; can you re-invite me?

Sent. Feel free to close the issue once you accept.

Thanks! I accepted the invitation.


Question: Can I use this to view orchestrations built using Durable Task Framework

I have built some orchestrations using the durable task framework which run locally on one of our servers. Just wondering if this can be used to visualise these or does it only support durable functions? If that is the case, is there any ways of being able to visualize these orchestrations

It was discovered before that it should be absolutely possible: https://twitter.com/cgillum/status/1415470831136739328 .
But it highly depends on which persistence store you're using. With DurableTask.AzureStorage DfMon should work seamlessly. With other stores it might only work in injected mode.
Which store are you using?

It was discovered before that it should be absolutely possible: https://twitter.com/cgillum/status/1415470831136739328 . But it highly depends on which persistence store you're using. With DurableTask.AzureStorage DfMon should work seamlessly. With other stores it might only work in injected mode.
Which store are you using?

I'm using the DurableTask.SqlServer store at the moment.

Then you can just try to enter your SQL connection string in DfMon VsCode ext:

Please, let us know if it doesn't work.


kics: init at v1.5.12

Description of changes

Things done

Built on platform(s)

[X] x86_64-linux
[X] aarch64-linux
[X] x86_64-darwin
[X] aarch64-darwin

[X] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

Is your nixpkgs revision old? When I check all-packages.nix the default version of buildGoModule is 1_18

@azahi now better 🎉 ?

@winterqt fixed


HA 2025.11 warning...

2024-11-11 13:06:53.071 WARNING (MainThread) [homeassistant.helpers.frame] Detected that custom integration 'o365' uses async_config_entry_first_refresh, which is only supported for coordinators with a config entry and will stop working in Home Assistant 2025.11 at custom_components/o365/helpers/setup.py, line 87: await sensor_coordinator.async_config_entry_first_refresh(), please create a bug report at https://github.com/RogerSelwyn/O365-HomeAssistant/issues
Is this something to worry about?

It does say it is only a warning and it will stop working in a years time.
I will make the required change in due course.

It is possible that this warning will not be possible to remove, it seems that HA core is moving towards all integrations (certainly those that use co-ordinators) having a config entry. Because o365 is setup from yaml, it doesn't have a config entry.
I'm not sure this may not be solvable without removing the co-ordinators. Given that I have written replacement integrations, which don't have this problem, a wholesale migration to those may be required (in particular when they finally make it onto HACS default).


Use GitHub Status API

Here's a test of a success status, and a failure status.
cc @williamkapke & @phillipj

Fixes: https://github.com/nodejs-github-bot/github-bot/issues/14

rebased

LGTM

You mentioned on IRC that we should be polling SHA for all repos. Any reason you haven't deleted pollByPrThenComment()?
IIRC this means we would do status updates for readable-stream, but still do comments on nodejs.org and citgm.

Have you tested updates on PRs and how that affects previously set statuses? Could we end up with misleading success status on a PR if the first build succeeded, but new commits/pushed rebase results in brokens builds?

@phillipj I left that because I wasn't sure if it was still necessary.
Afaik statuses tie to commits, so if the commits are updated GitHub automatically resets it.

I'll give it a test when I get home later though.

Hmmm, looks like this no-longer works after being rebased on #8. Investigating.

Got it working. My bad.
Looks like this doesn't actually handle PR updates, working on it. (Example: https://github.com/TestOrgPleaseIgnore/test-github-bot/pull/17)

Updated, now works for PR updates: https://github.com/TestOrgPleaseIgnore/test-github-bot/pull/17

going to open a new PR for some of the other bits


ZSH config file is really heavy

Hi,
I tried to source $(pew shell_config) in my .zshrc to have a feedback on the venv I am using, in my prompt. However, it adds 2 extra seconds or so to spawn a new ZSH shell, which is a lot for such a basic feature.
Is it normal? Expected?
Thanks
EDIT: Plus it messes with my PS1, breaking new lines etc.

The PS1 problem was #69
That one has been fixed a while ago, but I didn't release a new version until yesterday.
I'm not sure about the delay, it's definitely not expected, but maybe it's related? Does it still happen with the latest version?
Thank you

@berdario Indeed, PS1 problem seems to be #69 and is fixed.
Delay is still present however =( I can help you in debugging it, but I do not really know what could be useful to you.
Thanks

Let me know if you have any test in mind that I could do to debug it further.
Here is a quick comparison on another laptop, same issue:
Without the source line:
% time zsh -c "source /home/lucas/.zshrc"
zsh -c "source /home/lucas/.zshrc"  0,05s user 0,01s system 103% cpu 0,060 total

With the source line:
% time zsh -c "source /home/lucas/.zshrc"
zsh -c "source /home/lucas/.zshrc"  0,38s user 0,11s system 99% cpu 0,494 total

Thanks
The difference seems to be only 0.43s, definitely not ideal, but not as bad as 2 extra seconds
I would try to pin down if the culript is the Python startup, compinit, or something else... remove the source $(pew shell_config) line from your .zshrc and let's try something like this:
dario@curie ~> time zsh -ic 'pew shell_config'
/home/dario/.local/venvs/pew/lib/python3.5/site-packages/pew/shell_config/init.fish
0.19user 0.02system 0:00.24elapsed 89%CPU (0avgtext+0avgdata 23144maxresident)k
0inputs+0outputs (0major+13131minor)pagefaults 0swaps

dario@curie ~> time zsh -ic 'compinit'
0.05user 0.02system 0:00.10elapsed 77%CPU (0avgtext+0avgdata 5732maxresident)k
0inputs+0outputs (0major+6831minor)pagefaults 0swaps

dario@curie ~> time zsh -ic 'source /home/dario/.local/venvs/pew/lib/python3.5/site-packages/pew/shell_config/complete.zsh; compinit'
_arguments:comparguments:312: can only be called from completion function
0.06user 0.02system 0:00.10elapsed 78%CPU (0avgtext+0avgdata 5780maxresident)k
0inputs+0outputs (0major+6863minor)pagefaults 0swaps

I just had to install zsh to try out these commands (I don't usually use zsh) and I'm getting this
 _arguments:comparguments:312: can only be called from completion function

error, so the completion script is not loaded successfully and the timings will obviously be skewed... anyhow, I suspect that the culprit might be compinit, if a lot of other completion functions have been loaded before

Ok, here are the results.
% /usr/bin/time -p zsh -ic "source /tmp/.zshrc_with_source"
real 0.86
user 0.65
sys 0.21

% /usr/bin/time -p zsh -ic "source /tmp/.zshrc"
real 0.37
user 0.28
sys 0.10

% /usr/bin/time -p zsh -ic "pew shell_config"
/usr/lib/python2.7/site-packages/pew/shell_config/init.zsh
real 0.25
user 0.18
sys 0.07

% /usr/bin/time -p zsh -ic "compinit"
real 0.14
user 0.09
sys 0.04

% /usr/bin/time -p zsh -ic "source /usr/lib/python2.7/site-packages/pew/shell_config/init.zsh; compinit"
real 0.40
user 0.30
sys 0.11

I used the standard GNUTime as I think it is a more standard way to measure time and it seems to return times which are much more in agreement with the things I observe.
Actually, even if the difference in start time with or without sourcing the pew shell config seems to be small, without the source line I have directly a shell when I open a terminal, whereas when I add the source line, there is a noticeable time between loading the terminal and having a working shell. I may be able to record a screencast if you want to see the difference by yourself.

Ok
The blame seems to be for 2/3 on the completion script/compinit and for 1/3 on the Python/Pew startup time itself
If the only reason you're sourcing the shell helper is to know which venv you're currently using, you can simply lift only the bare minimum needed code from /usr/lib/python2.7/site-packages/pew/shell_config/init.zsh:
Which should be
function virtualenv_prompt_info() {
    if [ -n "$VIRTUAL_ENV" ]; then
        local name=$(basename $VIRTUAL_ENV)
        echo "($name)"
    fi
}
PS1="$(virtualenv_prompt_info)$PS1"

I'm not sure if something else could/should be done... One day I'll try to have a look if there's a difference in the additional startup time added to the different shells, and if zsh if out of line here.
The main purpose of the shell_config is for autocompletion, and thus disabling it does not really make sense. I'm not the original author of complete.zsh and I'm not a zsh user, so I'm not sure if there's any obvious improvement that could be done there.
Adding the source $(pew shell_config) line is optional, and the first_run_setup will default to not add it. Maybe I could add a message in the warning about the potentially increased startup time? Or maybe if people see the source $(pew shell_config) as a bit of a black-box something could be added in the README to invite users to adapt it to their needs?

On another machine:
19:51 phyks@Phyks-laptop ~ % time zsh -ic "source ~/.zshrc"
zsh -ic "source ~/.zshrc"  1,07s user 0,12s system 96% cpu 1,235 total

19:51 phyks@Phyks-laptop ~ % time zsh -ic "source ~/.zshrc"
zsh -ic "source ~/.zshrc"  0,23s user 0,04s system 95% cpu 0,277 total

(first one is with the source line)
After further tests, major source of overhead is the pew shell_config line which invokes a Python shell. Replacing source $(pew shell_config) by source PATH_TO_INIT_ZSH gets a lot better.
Finally, using compinit in init.zsh might be useless as users already run it in their .zshrc. I am not 100% sure about the reason it is here.
Anyway, putting the actual content you mentionned in my zshrc is already what I do and it works just fine.

Yeah, the path outputted by shell_config differs depending on how/where pew has been installed...
The idea is also that when pew is updated, the new version of any completion script will be thus used in the user's shell config
That's pretty much a case of:

All problems in computer science can be solved by another level of indirection ... except for the problem of too many layers of indirection

I guess I could offer an option:

add source $(pew shell_config)
add (hardcoded) source PATH_TO_INIT_ZSH
[default] don't modify shell config file

Do you have any other idea?

Apart from turning pew shell_config into a sh script, or something like this, I do not see many other solutions. Sorry…

An old issue, but I totally agree with @Phyks.
As a solution, I found prezto's python module to be much more efficient. What's more, the virtualenv information is displayed on the right side of the terminal. Somehow it works with pew, although it only supports virtualenvwrapper and conda.
Here are some timings:
with source $(pew shell_config):
~ ❯❯❯ time zsh -ic "source ~/.zshrc"
zsh -ic "source ~/.zshrc"  1.12s user 0.49s system 99% cpu

with prezto's python module
~ ❯❯❯ time zsh -ic "source ~/.zshrc"
zsh -ic "source ~/.zshrc"  0.11s user 0.10s system 99% cpu

If you're using a framework like Oh My Zsh or Prezto that loads compinit, you can see a large speedup by deleting compinit from line 2 of PEW's init.zsh file. For example, mine is located at "/usr/local/lib/python3.6/site-packages/pew/shell_config/init.zsh" As long as pew is sourced after the framework (at the end of .zshrc in my case) this will result in a speedup of roughly 0.7 seconds without losing completion. Loading completion is one of the biggest slowdowns with shell startup.


CI: doc build broken due to new DeprecationWarning from jsonschema

This is happening in a third-party dependency, nbformat, and will be fixed by https://github.com/jupyter/nbformat/pull/368. A release with that fix should be out early next week.

Do we want to silence this warning for now to fix the docs build?

nbformat 5.9.1 has been released, and the CI here is green again, closing.


[Feature] footer misaligned on some pages

Problem
The footer of the pages should normally be at the footer(fixed at the bottom of the page) for all screens and it has to be taken into consideration.
Description
The current footer not well aligned on all the device.
Case study
Device: laptop with MIDP
Page: 404 page
Illustration:
Alternatives
As an alternative an image with some following thew design of the website can be the best.
Implementation

[X] I would be interested in implementing this feature.

@TimidRobot I'm on this now!

@TimidRobot Please this issue is linked to the components on the components repository. My suggestion now is that, Please transfer this issue on the cc-components repository for me to work it but there.

Hey sir @TimidRobot, I made a mistake in linking the PRs to the issue, This issue is not the one related to the previous PR at #315.
Currently making some research on how to solve this issue.

Hi @Tapo41, Hope you're doing great?
I think you should try reading through these lines

Currently, the 404 page is only served by Django (it is not published to creativecommons/cc-legal-tools-data/)--it only impacts the local development environment.


Department responsibilites in the handbook

Description
There's uncertainty what exactly the responsibility is per department. This creates uncertainty in the organisation on who creates what decision. There's a DRI section in the handbook, which however isn't always applied. While work on that, let's also work on the following:

Every department lists their responsibilities in the index for the departments handbook page. And who's ultimately responsible for reaching that goal

[ ] @knolleary for engineering
[ ] @MarianRaphael for product
[ ] @iskerrett for marketing
[ ] @zackwasli for sales
[ ] @joepavitt for UX/Design

Epic/Story
No response
Have you provided an initial effort estimate for this issue?
I have provided an initial effort estimate

Have assigned this issue to those tagged based on the feedback in the Retrospective this morning

Added a short bit on Engineering dept responsibilities via as a part of https://github.com/FlowFuse/website/pull/1263
https://github.com/FlowFuse/website/blob/943726915f3e269aa5eca3f43078abad1647d1ae/src/handbook/development/index.md#engineering


Godot 4.3 export to Android release and debug issue aab files

Tested versions
Reproduce on Godot 4.3stable version.
System information
windows 10 - Godot 4.3
Issue description
When exporting to Android aab file I have got the issue. The release file is bigger than debug one. When I extracted the release file I found dir "BUNDLE-METADATA" containing com.android.tools.build.debugsymbols and big *.so files
Steps to reproduce
1 Create new project
2 Install Android build template
3 Project-Export add android export
4 Check: Use gradle build
5 Choose: Export AAB
6 Export project
7 Export with Debug create 2 variants using .aab extension
8 * to release export use keystore

Minimal reproduction project (MRP)
issue_test.zip
readme.txt file contains the keystore user and password
I cannot download my .aab files because they are 45 MB for debug and 84 MB for release

same issue here :
apk is 100 Mo, aab is 180 Mo (both in release mode)

I believe that that is intentional.
Android release AAB images contain debug symbols and mapping tables that are split off by the Google Android Play Console after upload. That way, you do not have to manually upload them. In regular Android apps these are mostly the ProGuard deobfuscation files.
The final APK/AAB deployed to users will not contain those files.
Case in point: I created a simple Godot Android app and exported it as a debug and release AAB.

Size of Debug AAB: 35.9 MB
Size of Release AAB: 49.7 MB (containing a debug-symbols file in BUNDLE-METADATA)

Once uploaded to the App store, the App Bundle explorer says:

Size for new installs: 35.9 MB

Most of the extra debug symbols come from "libgodot_android.so.sym". If you really don't want those, you probably will need to compile the Android export yourself and remove all debug symbols.


Support for async / await pattern

...so that i/o bound handlers become more resource-efficient.
Had a preliminary chat with @payman81 and decided to go for something along the lines of adding IAsyncHandler<T> interface, similarly to IHandler<T>.
This way, the change will be non-breaking and it will be up to users to pick whether they want to go async in a particular scenario.

@ILikePies  - did you ever had time to work on this?

@dariusdamalakas - I attempted to do that around Feb but failed as this would require substantial refactoring. I reckon things have improved since then but haven't had a look yet.

Thats what I thought too - if that hasn't been done from the ground up, than adding async is not that trivial. Thanks

I second this. I have seen code that implements IHandler, and then needs to do http requests while processing the message, so (ab)uses the HttpClient class by calling .Result() immediately on returned tasks. Ack.
A rule of thumb is that if you need to re-synchronise (.e.g. when using async/await in a commandline app)  then you should do it once as high up as possible. Supplying an IAsyncHandler interface would push these waits up into the JustSaying Framework, even if the handling of them wasn't great.

General comments on a way to approach this here: https://gist.github.com/AnthonySteele/44804be39f19a871b0a6

@AnthonySteele , good stuff. This is also roughly in the direction that we have discussed this with @payman81 and @brainmurphy . I.e. follow B) without the sync support internally, and this will be a major version bump.

@dariusdamalakas Thanks. I agree that it should be a major version bump.
The take-away from that gist (aside from the basics of async ... await ) is that if you code to prefer async handlers, then it only takes a dozen or so lines of code to also support existing sync handlers. This would allow existing code to be upgraded and still compile and run. But then you might need to mark the sync version as deprecated in order to encourage going async.

@AnthonySteele thanks for that thourough analysis. I agree with your conclusion that solution B is preferable given how easy it is to support existing handlers.
After all, Messaging is inherently asynchronous.

This PR: https://github.com/justeat/JustSaying/pull/203 is for this issue.

Fixed in 4.0


WebBreaker Exception Handling for file types

Uncaught exception handling on unknown filetypes for...
webbreaker webinspect scan --settings=test.xml
webbreaker fortify upload --scan_name=test.fpr
Additionally, remove these fortify ssc events logged to console.
Starting new HTTPS connection (1): localhost

remediated with WebBreaker v2.0.08


fix adminjs panel donation creation issue

https://github.com/Giveth/impact-graph/issues/813
Upgraded adminjs
Used component loader instead of adminjs bundler

https://github.com/Giveth/impact-graph/issues/813

@aminlatifi  I has fixed it in https://github.com/Giveth/impact-graph/pull/630/commits/1e8b4139c0f61324e5ff5830b932e99269155762 we can close this PR (I directly pushed my commit in f_572_update_typeorm_and_adminbro branch, by mistake)

We fixed the issue in another PR so I and @aminlatifi  decided to close this PR


JSON Formatting error on light turn on

I have been able to setup a bulb and connect it. Turning the bulb off and setting the brightness works, but if I use the switch in homeassistant to turn it on, I'm receiving the following error.
2022-02-02 23:19:18 ERROR (MainThread) [homeassistant.components.websocket_api.http.connection] [2964029176] Error Code: -1003, JSON formatting error
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/websocket_api/commands.py", line 185, in handle_call_service
await hass.services.async_call(
File "/usr/src/homeassistant/homeassistant/core.py", line 1495, in async_call
task.result()
File "/usr/src/homeassistant/homeassistant/core.py", line 1530, in _execute_service
await handler.job.target(service_call)
File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 209, in handle_service
await self.hass.helpers.service.entity_service_call(
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 663, in entity_service_call
future.result() # pop exception if have
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 896, in async_request_call
await coro
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 700, in _handle_entity_call
await result
File "/usr/src/homeassistant/homeassistant/components/light/__init__.py", line 494, in async_handle_light_on_service
await light.async_turn_on(**filter_turn_on_params(light, params))
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 948, in async_turn_on
await self.hass.async_add_executor_job(ft.partial(self.turn_on, **kwargs))
File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 52, in run
result = self.fn(*self.args, **self.kwargs)
File "/config/custom_components/tapo_p100_control/light.py", line 89, in turn_on
self._l530.setBrightness(newBrightness)
File "/usr/local/lib/python3.9/site-packages/PyP100/PyL530.py", line 42, in setBrightness
raise Exception(f"Error Code: {errorCode}, {errorMessage}")
Exception: Error Code: -1003, JSON formatting error

I've also noticed that changing the brightness does not work a lot. I'm going to troubleshoot this later on.


Restore local storage state from backup file

Minimum restore backup functionality. It works.

Need to add an upload button with select dialog for mobile users!

Awesome, I will get on to test this today!
Can you refactor the stuff from mint.js into a new file?


Try to fix repeating close dialog on macOS and qt 5.12

Description

Try to solve (from @tlambert03)

if you press cmd-q, and then press it again (as the dialog instructs), it doesn't quit... it just opens more dialogs

https://user-images.githubusercontent.com/1609449/201672279-161cef46-63c7-4e6f-879f-82fd2a930d7f.mov
Type of change

[x] Bug-fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

References

closes #5335
How has this been tested?

[ ] example: the test suite for my feature covers cases x, y, and z
[ ] example: all tests pass with my change
[ ] example: I check if my changes works with both PySide and PyQt backends
as there are small differences between the two Qt bindings.

Final checklist:

[ ] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If I included new strings, I have used trans. to make them localizable.
For more information see our translations guide.

I can't say I get the code, but it does fix the issue in my macOS 12.6 Rosetta-emulated x86 env with pyqt 5.12.3.
👍 @Czaki

I add comments with explanations.

OK, I think I see it now. I was writting a question and explained it to myself.
My hangup was that quit_app is actually a bool so either False or True so it works with the dict you defined at the beginning in L118.
I'd still rather someone else review this just to be safe,

I could change bool to enum to increase readability if needed.

No no, it's me being an imbecile and not comprehending: self._is_close_dialog[quit_app]

nice job @Czaki  :)

Would you vote for 0.4.18 bugfix release or could it wait until 0.5.0?

don't have an opinion

Ok. Will merge after 24h.


Add equal pay task to lesson 01

As discussed, I added a simple and (imho) informative task to the wage dataset exercise.

looks good to me. Merci!


not working well on other datasets

I've been trying to train this code on other datasets like ImageNet but the results are very bad. The images turn into almost black and grey blobs. However, using the original author's theano implementation, results on ImageNet are good. Have you tried out other datasets?

@kracwarlock hi, do you know how to put my dataset? I alwanys faild to read the image files.
--train_data_root F:\pychqarmtestdemo\dataset\face
face fold includes many *.jog
Output:Renew dataloader configuration, load data from F:\pychqarmtestdemo\dataset\face.
it didn't begin to train.
do you know why it seems like this? Would like your reply. Thanks.


GATT Server is not sending response in onDescriptorWriteRequest callback

@aritchie It looks like the mGattServer.sendResponse method needs to be called onDescriptorWriteRequest callback when notification enabled on a characteristic.
I tried the below Sample Android GATT server code with this method implementation and client is not getting 133 error in client side callback onDescriptorWriteResult.
I don't see responseNeeded implemented in this class.
https://github.com/aritchie/bluetoothle/blob/master/Plugin.BluetoothLE/Platforms/Android/Server/Internals/GattServerCallbacks.cs
Which method needs to be implemented in this library to send the response to GATT client to fix this issue?
@Override
public void onDescriptorWriteRequest(BluetoothDevice device, int requestId,
BluetoothGattDescriptor descriptor,
boolean preparedWrite, boolean responseNeeded, int offset, byte[] value) {
Log.d("HELLO", "Our gatt server descriptor was written.");
super.onDescriptorWriteRequest(device, requestId, descriptor, preparedWrite, responseNeeded, offset, value);
Log.d("DONE", "Our gatt server descriptor was written.");
        //NOTE: Its important to send response. It expects response else client will disconnect
        if (responseNeeded) {
            mGattServer.sendResponse(device,
                    requestId,
                    BluetoothGatt.GATT_SUCCESS,
                    0,
                    value);

        }
    }

Originally posted by @mmkrishna2008 in https://github.com/aritchie/bluetoothle/issues/308#issuecomment-494501878

Please submit a PR - I'm not having this issue

This is duplicate of #313


Migrations Are Published Automatically

Seems the migrations are getting publish when I'm running my unit tests. This could be a potential bug.

Don't think so? You need to actually call the vendor publish command.

Weird, wonder why they get published then? Could it be something when we publish the migrations?
/**
 * Setup the migrations.
 *
 * @return void
 */
protected function setupMigrations()
{
    $source = realpath(__DIR__.'/../database/migrations/');
    $this->publishes([$source => database_path('migrations')], 'migrations');
}

Weird. Wonder why they get published then? Could it be something when we publish the migrations?
/**
 * Setup the migrations.
 *
 * @return void
 */
protected function setupMigrations()
{
    $source = realpath(__DIR__.'/../database/migrations/');

    $this->publishes([$source => database_path('migrations')], 'migrations');
}

The code above is taken from our service provider.

@lucadegasperi Do you have any ideas about this?

This was something on my side.


Enable new travis hotness

I trust you that this works assuming travis passes. We are still merging internally first, right?

I trust you that this works assuming travis passes. We are still merging internally first, right?

yup yup


docs(breaking): Adding a ViewChild entry

Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[V ] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[ ] Build (npm run build) was run locally and any changes were pushed
[ ] Lint (npm run lint) has passed locally and any fixes were made for failures

Pull request type

Please check the type of change your PR introduces:

[ V] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[V ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

When declaring a ion-slides...
HTML
<ion-slides #slider pager>

TS
@ViewChild('slider') slider:Slides;

I receive an Angular error :

Expected 2 arguments, but got 1.
[ng] 23    @ViewChild(slider) slider: slider:Slides;

Tested on Ionic 4.11.5. , with Angular 8 ONLY : according to the Angular.io API :

explaining that change needed : https://v8.angular.io/api/core/ViewChild#description ;
explaining that no more needed : https://v9.angular.io/api/core/ViewChild#description

Issue Number: N/A
What is the new behavior?

Any more error message

Does this introduce a breaking change?

[V ] Yes
[ ] No

Other information

Thanks for the PR! What issue does this fix? At first glance, this looks more like an Angular-specific breaking change than an Ionic one.

It's true, but does it deserve a PR for the Breaking Change Documentation
beetween v3 and v4 (not v5 because it's Angular >8) ?
[image: photo]
François Gardien
Développeur | Chef de projet at  FG Solutions
A  64 rue du Breuil, 17300 Rochefort
M  <IP_ADDRESS>.43  <<IP_ADDRESS>.43>
http://www.linkedin.com/in/françois-gardien-15496160/
http://github.com/Youtch
Notre environnement est fragile, merci de n’imprimer cet email qu’en cas de
nécessité.
Create your own WiseStamp email signature
https://www.wisestamp.com/signature-in-email/?utm_source=promotion&utm_medium=signature&utm_campaign=create_your_own&srcid=5021717640249344
Le lun. 29 juin 2020 à 18:10, Liam DeBeasi<EMAIL_ADDRESS>a
écrit :

Thanks for the PR! What issue does this fix? At first glance, this looks
more like an Angular-specific breaking change than an Ionic one.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ionic-team/ionic/pull/21631#issuecomment-651217576,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACD4ZOAWHC6TZD7DHTUAEWLRZC4F5ANCNFSM4OJJEAJA
.

Hmm what would be the benefit of noting this in the Ionic Framework v3-->v4 breaking changes document? v3 never supported Angular 9, so users would not have run into this issue.

Hmm what would be the benefit of noting this in the Ionic Framework v3-->v4 breaking changes document? v3 never supported Angular 9, so users would not have run into this issue.

Not exactly : This PR concerns Ionic 4 with Angular 8 (coming from Ionic 3 with Angular<8)

Ah ok I see, thanks for the clarification. The purpose of the breaking changes document is to note breaking changes to the Ionic Framework APIs. While this particular change may be required when migrating from Angular <8 to Angular 8, it is not a change to an Ionic Framework API. I spoke with the team, and we feel it may be confusing to note Angular breaking changes here. As a result, we have decided not to merge this PR. Thank you!

Thank you for your consideration.


Git clone command updated > Docs article "Opening a pull request"

Description:
The PR updates the git clone command required to follow along the documentation article "Opening a pull request". Yes, this is a PR to correct how to perform a PR. :-)
The command: git clone<EMAIL_ADDRESS>...no longer works.
See example error message: "fatal: repository<EMAIL_ADDRESS>does not exist"
This is the correct "git clone" command: git clone https://github.com/<github_username>/website

I signed it

/recheck

/assign
Hi @dkypuros did you check out the CLA troubleshooting tips in the CLA description link? They are here:
https://help.github.com/en/github/committing-changes-to-your-project/why-are-my-commits-linked-to-the-wrong-user

Hi @dkypuros  Also, you may want to make sure the git commit has the proper email address associated with it doing something like the following:
git commit -s --amend --author="Author Name<EMAIL_ADDRESS>git push -f

Waiting on<EMAIL_ADDRESS>My account is locked for some reason. https://support.linuxfoundation.org/
https://help.github.com/en/github/setting-up-and-managing-your-github-user-account/setting-your-commit-email-address

Hello,
Excuse me for this way of contacting you, I don't think it was a coincidence that I contacted you. It is good after several days of intense prayer that I do it is that the Holy Spirit guided me towards you by the grace of God. I donate my goods to an honest and needy person of French origin and I live in France. I suffer from a serious illness which condemns me to certain death, it is throat cancer, and I have a sum of 700,000 euros which I would like to make a donation to a person of trust and honest so let him make good use of it in order to help his family and start a new life in happiness and peace. I donate these funds for free to change your life. I donate this because after all the love of neighbor is the basis of all my Christian life.
I would then like to know if you can benefit from this donation?
Contact by email for the procedures to obtain 700,000 euros for free.


Can I use this in a prod app?

Hello, awesome launch screen :)
But is it safe to use in a production app?
Thanks,

Hello, can we use your approach in our code?

@kiwo12345 and @aatalyk, yes, it should be safe to use it in the production app as it should not contains and 3rd party framework and must follow the iOS App Development Guidelines.
A twitter app is using the similar animation on launches, so you are good to go!


Where to get the Quilt-VQA json file?

Thanks for your excellent work!
I'm wondering where to get the "quiltvqa_test_w_ans.json", "quiltvqa_test_wo_ans.jsonl", "quiltvqa_red_test_wo_ans.jsonl", and "quiltvqa_red_test_w_ans.json" for evaluation? I can only find "quiltvqa_nored_test_wo_ans.jsonl" in the huggingface, the other files are missing.

Thank you for the observation, we traced the commission to a private HF dataset.
We have added the JSON files to HF, for both Quilt_VQA and the Quilt_VQA_RED subset, and updated the Evaluation documentation in the codebase.
Apologies, if this affected your research in any way. Thanks for bringing it to our attention.


Hand or Wrist in affordance data?

Hi,
I am a little confused about the affordance data. There is no hand in the 263-length joint vector.  But in the paper, you point out that the affordance data includes hands.
So my question: Are the hand labels in affordance data wrist labels?
Your reply will be very helpful.

Yes, in the affordance data, the term 'hands' refers to the wrist joints. We will clarify this more explicitly in both the paper and the README.


perf-regression: workaround hyperfoil-maven-plugin issue with JDK 17 …

…#4016

@jamesnetherton It's working locally with:
openjdk 17.0.4 2022-07-19
OpenJDK Runtime Environment Temurin-17.0.4+8 (build 17.0.4+8)
OpenJDK 64-Bit Server VM Temurin-17.0.4+8 (build 17.0.4+8, mixed mode, sharing)

Time permitting, it would be great to have this PR checked on another machine.


typescript

add support to typescript

I am not used to typescript. I thought you can use every JS-lib in ts. What needs to be done to support typescript?

Basically "someone" has to write a Typescript Definition File for your Library as outlined in the Typescript Handbook (http://www.typescriptlang.org/Handbook#writing-dts-files). But i didn't tried that.

I did, but I'm not sure it's 100% correct. It works for my basic tasks:
interface Options {
  endpoint: string
  json?: boolean
  version?: number
  strictMode?: boolean
  start?(): void
  ready?(): void
  error?(): void
  headers?: string[]
  username?: string
  password?: string
  isAsync?: boolean
}

interface OHandler {
  inlinecount: number
  data: Object[]
  config(options?: Options): OHandler
  get(callback?: (data: Object[]) => void): Q.Promise<{}>
  save(callback?: (data: Object[]) => void): Q.Promise<{}>
  route(path: string, callback?: (data: Object[]) => void)
  find(selector: string|number): OHandler
  top(quantity: number): OHandler
  take(quantity: number): OHandler
  skip(quantity: number): OHandler
  first(): OHandler
  filter(filter: string): OHandler
  where(filter: string): OHandler
  any(filter: string, resource: string): OHandler
  search(columns: string[], term: string): OHandler
  orderBy(column: string, direction?: boolean): OHandler
  orderByDesc(column: string): OHandler
  count(): OHandler
  inlineCount(paramName?: string): OHandler
  batch(resource: string): OHandler
  expand(resource: string): OHandler
  ref(resource: string, id: string|number): OHandler
  post(params: Object): OHandler
  patch(params: Object): OHandler
  put(params: Object): OHandler
  remove(params: Object): OHandler
}

declare function o(options?: string|Options): OHandler

Please have a try and let me know, we can add it to the Typings registry.

I created a pull request with a slightly modified version of your def, @IceOnFire
see DefinitelyTyped/DefinitelyTyped#9209

Nice, thanks!

I think that this issue can be closed, since the feature has been implemented and is related to another project anyway.

Sorry for appending more question on this closed issue. I use npm to add @types/odata, and this works for below:
var query = o("****");
but how should I declare an interface that contains a field of the OHandler?
import * as odata from "odata" interface myData<T> {     source: odata.OHandler<T>; }
this doesn't work.
Thanks

Mhh, not sure about that. Haven't used o.js with typescript. Mayb just use the source: o ? @IceOnFire any hints on this?

I'm sorry, I too don't write TypeScript code.
I tried fiddling around on a sample project but came up to the conclusion that there are some issues on odata's type definition.
import * as o from 'odata'

interface Market {
  Name: string
  Description: string
}

const query = o('http://localhost:3000/Markets')

const promise: Promise<OHandler<Market>> = query.get<Market>()

promise.then((oHandler: OHandler<Market>) => {
  console.log(oHandler.data)
})

I'm quite sure that the promise should have type Promise<OHandler<Market>> because if I try with another type then TypeScript says that this is the correct one. However, the interface OHandler is not exported inside the module definition and I couldn't manage to export it correctly after some tries.
I'm afraid we need someone more skilled with TypeScript, or I just need some more time to learn it well enough.

Ok this seems to work :P
import Q = require('q')

declare namespace odata {
  interface Options {
    endpoint: string
    format?: string
    autoFormat?: boolean
    version?: number
    strictMode?: boolean
    start?(): void
    ready?(): void
    error?(): void
    headers?: string[]
    username?: string
    password?: string
    isAsync?: boolean
  }

  export interface OHandler<T> {
    inlinecount: number
    data: T[]

    config<T>(options?: Options): OHandler<T>
    progress<T>(callback: () => any): OHandler<T>

    get<T>(callback?: (data: T) => void): Q.Promise<OHandler<T>>
    save<T>(callback?: (data: T) => void): Q.Promise<OHandler<T>>

    post<T>(params: any): OHandler<T>
    patch<T>(params: any): OHandler<T>
    put<T>(params: any): OHandler<T>
    remove<T>(params?: any): OHandler<T>

    routes<T>(path: string, callback?: (data: T) => void): OHandler<T>
    route<T>(path: string, callback?: (data: T) => void): OHandler<T>
    triggerRoute(hash: string): OHandler<T>
    beforeRouting(callback: (routeParams: any) => boolean): OHandler<T>

    isEndpoint(): boolean
    loading<T>(startFn: () => any | boolean, endFn: () => any): OHandler<T>

    find<T>(selector: string | number): OHandler<T>

    top<T>(quantity: number): OHandler<T>
    take<T>(quantity: number): OHandler<T>
    skip<T>(quantity: number): OHandler<T>
    first<T>(): OHandler<T>

    include<T>(column: string, data: string): OHandler<T>
    exclude<T>(column: string, data: string): OHandler<T>
    filterByList<T>(column: string, data: string): OHandler<T>

    filter<T>(filter: string): OHandler<T>
    where<T>(filter: string): OHandler<T>
    any<T>(filter: string, resource: string): OHandler<T>
    search<T>(columns: string[], term: string): OHandler<T>

    orderBy<T>(column: string, direction?: boolean): OHandler<T>
    orderByDesc<T>(column: string): OHandler<T>
    select<T>(selectStr: string): OHandler<T>

    count<T>(): OHandler<T>
    inlineCount<T>(paramName?: string): OHandler<T>

    batch<T>(resource: string): OHandler<T>
    expand<T>(resource: string): OHandler<T>
    ref<T>(resource: string, id: string | number): OHandler<T>
    removeRef<T>(resource: string, id: string | number): OHandler<T>
    deleteRef<T>(resource: string, id: string | number): OHandler<T>
  }

  export function o(options?: string | Options): OHandler<{}>
}

@janhommes cuold you please check it and upload it if everything seems right to you?

Ok, so basically just export the OHandler as well? Not sure if this part export function o(options?: string | Options): OHandler<{}> would allow using the jquery like shorthand function still (o('some/url'))

thank you for your help, should Options be exported either?

@janhommes The difference is also in the import, which is outside the module, and the module, that now is a namespace.
I'm trying with the following code and every line is giving me 0 errors:
/// <reference path="odata.d.ts" />

import * as o from 'odata'

interface Market {
  Name: string
  Description: string
}

// promise-based
const query = o('http://localhost:3000/Markets')
const promise: Promise<o.OHandler<Market>> = query.get<Market>()
promise.then((oHandler: o.OHandler<Market>) => { console.log(oHandler.data) })

// callback-based
o('http://localhost:3000/Markets').get((data: [Market]) => console.log(data))

// with config
o().config({ endpoint: 'http://localhost:3000/' })
o('Markets').get((data: [Market]) => console.log(data))

@cbfrank Options seem to work too: I just tried this without errors:
const options: o.Options = { endpoint: 'http://localhost:3000/' }
o().config(options)

Actually with the namespace definition it seems there's no need for exports, it just works under the namespace o.

Nice. I will update the definitions for odata. Can we use namespaces for the o.js types as well? is this possible?
declare namespace 'o.js' {}

This worked fine for me until I got tired of configuring o() everywhere and decided to put it into a service like so:
import 'rxjs/add/operator/map';
import 'rxjs/add/operator/catch';

import { Injectable } from '@angular/core';
import { Http, Response } from '@angular/http';
import { Observable } from 'rxjs/Observable';

import { AuthenticationService } from '../core/authentication/authentication.service';
import { OHandler, Options } from 'odata/parts';
import * as o from 'odata';
import * as _ from 'lodash';

import { environment } from '../../environments/environment';

@Injectable()
export class VendorService {

  constructor(private authenticationService: AuthenticationService) {}

  odata(value: string): OHandler<{}> {
    const token = this.authenticationService.credentials.token;
    const config: Options = <Options>_.defaults({
      headers: [
        { name: 'Authorization', value: `Bearer ${token}` }
      ]
    }, environment.odata.connect);
    o().config(config);
    return o(value);
  }

}

Because I couldn't get access to Options or OHandler since they aren't publicly exported, I had to use my own mappings and separate out the type exports into their own module (as shown above) called 'odata/parts'.
The full index.d.ts I'm using looks like this now (shown below), and it works for my needs. I'm not a typescript expert so there may be a better way to handle this, but I was unable to find it. This shouldn't have any impact on those that are using it the old way since the interfaces were never exported to begin with.
// Type definitions for odata v0.3.3
// Project: https://github.com/janhommes/odata
// Definitions by: Jan Hommes <https://github.com/janhommes>
// Definitions: https://github.com/DefinitelyTyped/DefinitelyTyped

declare module 'odata/parts' {
    import Q = require("q");

    export interface Options {
        endpoint : string
        format ?: string
        autoFormat ?: boolean
        version ?: number
        strictMode ?: boolean
        start ?: () => any
        ready ?: () => any
        error ?: () => any
        headers ?: { name: string, value: string }[]
        username ?: string
        password ?: string
        isAsync ?: boolean
    }

    export interface OHandler<T> {
        inlinecount : number
        data : T

        config<T>(options ?: Options) : OHandler<T>
        progress<T>(callback : () => any) : OHandler<T>

        get<T>(callback ?: (data : T) => void) : Q.Promise<OHandler<T>>
        save<T>(callback ?: (data : T) => void) : Q.Promise<OHandler<T>>

        post<T>(params : any) : OHandler<T>
        patch<T>(params : any) : OHandler<T>
        put<T>(params : any) : OHandler<T>
        remove<T>(params ?: any) : OHandler<T>

        routes<T>(path : string, callback ?: (data : T) => void) : OHandler<T>
        route<T>(path : string, callback ?: (data : T) => void) : OHandler<T>
        triggerRoute(hash : string) : OHandler<T>
        beforeRouting(callback : (routeParams : any) => boolean) : OHandler<T>

        isEndpoint() : boolean
        loading<T>(startFn : () => any | boolean, endFn : () => any) : OHandler<T>

        find<T>(selector : string|number) : OHandler<T>

        top<T>(quantity : number) : OHandler<T>
        take<T>(quantity : number) : OHandler<T>
        skip<T>(quantity : number) : OHandler<T>
        first<T>() : OHandler<T>

        include<T>(column : string, data : string) : OHandler<T>
        exclude<T>(column : string, data : string) : OHandler<T>
        filterByList<T>(column : string, data : string) : OHandler<T>

        filter<T>(filter : string) : OHandler<T>
        where<T>(filter : string) : OHandler<T>
        any<T>(filter : string, resource : string) : OHandler<T>
        search<T>(columns : string[], term : string) : OHandler<T>

        orderBy<T>(column : string, direction ?: boolean) : OHandler<T>
        orderByDesc<T>(column : string) : OHandler<T>
        select<T>(selectStr : string) : OHandler<T>

        count<T>() : OHandler<T>
        inlineCount<T>(paramName ?: string) : OHandler<T>

        batch<T>(resource : string) : OHandler<T>
        expand<T>(resource : string) : OHandler<T>
        ref<T>(resource : string, id : string | number) : OHandler<T>
        removeRef<T>(resource : string, id : string | number) : OHandler<T>
        deleteRef<T>(resource : string, id : string | number) : OHandler<T>
    }

    export interface OFn<T> extends OHandler<T> {
        (options ?: string | Options) : OHandler<T>
    }
}

declare module 'odata' {
    import { OFn } from 'odata/parts';

    var o : OFn<{}>;

    export = o
}


Allow .web.jsx, .web.ts, .web.tsx to be ignored in RSD ESLint Plugin

Why
At the moment, only .web.js is ignored for the 'valid-styles' rule but in TypeScript projects, you'd want to ignore .web.ts etc. files as well.
How
Renaming ALLOWED_FILE to ALLOWED_FILES and making it an array of allowed web file extension suffixes.
.web.jsx, .web.ts, .web.tsx are all allowed now because styles can be defined both in pure JS/TS files and JSX counterparts.

LGTM. Thanks


Optimize geometry conversion code

I tried to optimize the code at the expense of some readability, but it's still quite slow.
It might be worth looking into https://docs.unity3d.com/Manual/JobSystem.html and refactoring code to work with that. The biggest performance hit comes from a big nested loop that should be easy to parallelize.
Performance breakdown:

GetDuplicateEdges: 66.2%

Self: 32%
List.get_Item: 15.8% (8,297,838 calls)
VectorUtilities.DistanceSqr - 12.8% (5,528,820 calls)
List.get_Count: 2.6% (1,385,660 calls)

RemapDuplicates: 18.6%
RemoveDuplicates: 3.3%
RemoveInvalidDuplicates: 0.7%
GetDuplicateVerticesMap: 0.3%

 Optimize geometry conversion code


Which branch to use for latest BoundingBox implementation

Overview
I have not looked into the MRTK Git Repo for a while and am quite confused about all the Branches without any vNext available and such. I tested the BoundingBox from the Master branch but this can't be right as it's handling is wrong using the MR Motion Controller.
Which Branch is the latest with a working BoundingBox in it?
Unity Editor Version
2018.2.6f1

Master is the way to go. We have not finished porting this feature to vNEXT yet.

@StephenHodgson ok who is in charge of the bounding box then? It's still broken when used with MR Headset. Movement is slow and not always the same direction as the motion controller goes. Even holding the controller still and moving the head moves the object. Scaling is the worst as you never get what you want, scaling down when you effectively scale up and the other way around, having a hard time to scale big really.
The box is not good at all. Who can I talk to?

Ok, I fixed it in my project, but the MRTK there is old and a mess. Nevertheless, the reason why the box is broken is that the code only works for the Hololens hand. The calculations only use the hand position, since there is no beam or cursor. The calculations need to done differently for the controller, as the relevant position is where the controller is aiming at, the offset from it's position in pointing direction.

I'm unsure as to who owns this feature. But we def want to port it as soon as possible. (There's been one or two attempts made already)
If you were able to fix some things, we'd appreciate any contributions back into the HTK dev branch.
If you were able to port to MRTK dev branch already, that'd be awesome to see as well.

BoundingBox and AppBar were originated by @cre8ivepark / @johnppella .
Those guys have begun porting already, see #3044 . I think it might be wise to hold off on attempting to contribute anything to vNext as their work seems to be ongoing and there's not much back and forth going on.

Bounding box is in mrtk_development. Closing out.


Always complete RequestBodyPipe.Reader

This is part of an effort to stop leaking memory pool blocks.
Semi related: https://github.com/dotnet/corefxlab/pull/1601

CI is upset. But the change itself looks good.


[BUG] Not the right output in DM's

What happened?

This is in DM

What should happen?
-I guess display your MC name and the right prefix
Steps to reproduce

I have no idea

Version

1.16.4
[ ] 1.12
[ ] 1.16

Platform

[x ] Forge
[ ] Spigot

Mod version: dcintegration-2.0.9-1.16
Config file
REMOVE YOUR BOT TOKEN BEFORE UPLOADING!!!
Discord-Integration.txt
Crash report if available (use https://hastebin.com)
-no crashes
Additional info

Will fix that in the next update

Will fix that in the next update


Implement viewcontact command to view student contact in an existing group

Codecov Report

Merging #92 (137540d) into master (c876e92) will decrease coverage by 1.12%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master      #92      +/-   ##
============================================
- Coverage     57.62%   56.49%   -1.13%
  Complexity      488      488
============================================
  Files           102      104       +2
  Lines          1909     1947      +38
  Branches        221      226       +5
============================================
  Hits           1100     1100
- Misses          745      783      +38
  Partials         64       64

Impacted Files
Coverage Δ

...edu/address/logic/commands/ViewContactCommand.java
0.00% <0.00%> (ø)

...a/seedu/address/logic/parser/AddCommandParser.java
100.00% <ø> (ø)

...du/address/logic/parser/AddGroupCommandParser.java
70.00% <ø> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
84.00% <0.00%> (-3.50%)
:arrow_down:

...eedu/address/logic/parser/DeleteCommandParser.java
100.00% <ø> (ø)

.../seedu/address/logic/parser/EditCommandParser.java
92.59% <ø> (ø)

.../seedu/address/logic/parser/FindCommandParser.java
31.42% <ø> (ø)

...address/logic/parser/ViewContactCommandParser.java
0.00% <0.00%> (ø)

...du/address/logic/parser/ViewTaskCommandParser.java
90.00% <ø> (ø)

src/main/java/seedu/address/model/AddressBook.java
60.00% <0.00%> (-1.12%)
:arrow_down:

... and 3 more

Continue to review full report at Codecov.

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


[FLINK-25065][docs] Update  document for jdbc connector

What is the purpose of the change
The pr updates documentation for lookup.cache.caching-missing-key config in jdbc connector
Brief change log

jdbc.md

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (docs)

@gaurav726 , could you update the content for Chinese documentation as well? The file is located in docs/content.zh/docs/connectors/table/jdbc.md.

@wuchong  done, please review

@wuchong  pipeline got succeed


xfce4-sensors-plugin: hddtemp permission denied

Issue description

Steps to reproduce

Install xfce.xfce4-sensors-plugin
add sensors plugin to Xfce panel
double click on plugin

Technical details

system: "x86_64-linux"
host os: Linux 4.9.70, NixOS, 17.09.2430.ed3b1bc8448 (Hummingbird)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 1.11.16
channels(root): "nixos-17.09.2430.ed3b1bc8448"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

This can be solved by:
security.wrappers.hddtemp.source = "${pkgs.hddtemp}/bin/hdtemp";

Nixpkgs alone can't setuid programs (that would be a security problem), so it can't be handled in user profile. For system profile, there are two options: programs.hddtemp.enable option or setuid wrapping hddtemp in services.xserver.desktopManager.xfce module when xfce4-sensors-plugin is in environment.systemPackages.

There is way better (sane) solution then just setuid'ing hddparm. I got same problem in ubuntu, which I installed to friend laptop.
Found solution mentioned here: https://askubuntu.com/questions/280440/how-to-get-xfce4-sensors-plugin-to-show-hdd-temperature-in-xfce-panel
At my own system (gentoo) xfce4-sensors-plugin has netcat support enabled together hddparm if emerged with 'hddparm' use flag. In contrast, rules file in ubuntu package doesn't have it. As result - even if I start hddparm daemon manually with all options (as mentioned in replies at above link), it's useless.

Still relevant.

It's still not working. Sensors plugin is declared in System Packages (along with the other XFCE plugins) but I still get that permission error.


reduce git commands when get many commits info

Codecov Report

Merging #145 into master will increase coverage by 0.04%.
The diff coverage is 68.42%.

@@            Coverage Diff             @@
##           master     #145      +/-   ##
==========================================
+ Coverage   37.28%   37.33%   +0.04%
==========================================
  Files          28       28
  Lines        1832     1843      +11
==========================================
+ Hits          683      688       +5
- Misses       1055     1059       +4
- Partials       94       96       +2

Impacted Files
Coverage Δ

commit_info.go
62.6% <68.42%> (+0.93%)
:arrow_up:

repo_commit.go
23.77% <0%> (-1.51%)
:arrow_down:

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 2fa2b63...72226f7. Read the comment docs.


Remove an infinite loop within the reverse geocoding job

Summary
After noticing that the workers were always busy locally, I found, what I believe is, some kind of infinite loop. This PR aims at fixing it.
Détails

We do the User#reverse_geocode thanks to the after_commit,
We trigger a job that recompute the lat and lon,
We randomize the lat and lon, thanks to the before_save,
The job save and commit the updated resource and... we go back to 1. :recycle:

To avoid that, we skip the after_commit :reverse_geocode when we're about to save from the reverse geocode job itself.

I don't understand the standardrb error. It works locally :tm: :see_no_evil:


UX: Change the "Model" button text to something more action oriented

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1435
Originally created by @iroussos on 2019-12-11 21:36:30

We have had users reporting that the "Model" button in the Pipelines Schedule View is not self explanatory.
We should change it to something more action oriented, e.g. "Create Reports" or "Analyze Data"?
This is the result of our debriefing discussion with @dmor after the first call for meltano/account-management#1

View 4 previous comments from the original issue on GitLab


Sync providers conflict when both have the same branch selected due to shared access token

Describe the bug
When users are utilising two sync providers with distinct branches, they may inadvertently activate the same branch for both providers within the settings due to various reasons. This results in both providers becoming linked and active, likely due to the shared access token. Consequently, deleting one provider leads to the unintended deletion of the other due to their linked status.
https://github.com/tokens-studio/figma-plugin/assets/93133012/3055a8e8-5521-4a8a-becf-499920af84f7
To Reproduce
Steps to reproduce the behavior:

Add a new provider
The new provider should use the same repo as the original provider, but use a different branch
Make sure both the providers are using the same access token
When switching the branch of the new one to the same branch as the original provider, both should be active in the settings
After deleting one, both are consequently deleted.

Expected behavior
To avoid having both being active, maybe a modal that indicates that this is not possible since there is already a provider with the same details active, and that the user can't have two providers active with the same branch.

This request has been moved to our new feedback tool. Go check out https://tokensstudio.featurebase.app/ to create a free account. You can upvote and comment on existing posts, make a new request, and subscribe to be notified when requests move into our roadmap.

Added to Featurebase Roadmap : 🧑💻 Sync to external token storage enhancements


does it support aws_db_option_group?

i can't find it, please support it, thanks

Pull requests #55 and #24 are related to this issue.

v1.17.0 has been released with this feature.


⚠ Cioos National CKAN has degraded performance

In 1c11232, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 15971 ms

Resolved: Cioos National CKAN performance has improved in baf72de after 11 minutes.


Allow on-the-fly transcoding via ffmpeg

An option to allow transcoding via ffmpeg would greatly increase usability in remote streaming situations.
FLAC files could be transcoded to e.g. medium/lowish-bitrate ${favourite-lossy-codec} and stream on mobiles without insane bandwidth usage.  Although many in the west have unlimited plans, this would be particularly useful in India and other parts of the world where connectivity is comparatively poor.
Subsonic does this on the fly as it uses stdout instead of temporary files with only minimal delay. A similar implementation would make this a fantastic Subsonic killer -- for music at least.
Thanks.

Closing as this is an enhancement and has been added to trello.
Trying to keep github issues to just bugs and in-progress development.
Trello card for reference: https://trello.com/c/WmudxWuo/53-allow-on-the-fly-transcoding-via-ffmpeg


Blog Post: Using 6to5 with JSX/Flow

I am using browserify and 6to5 (via 6to5ify) to build a React.js app. I am using the JSX syntax as well as Class, const, let, array destructuring,  etc.
All was running fine until I tried to add Facebook flow to my setup and suddendly it's a nightmare :) Is this blog post almost ready?
flow is mainly complaining about the use of const:

Unsupported variable declaration: const

What is the state of flow support in 6to5 exactly? 6to5 reads annotations and interprets them during compilation? Then flow check should be run only on the compiled file?
Thanks!

@tbassetto It's better to ask questions like this in the gitter room


slow odo describe

odo describe  takes a lot of time
▶ time odo describe
Component Name: myjava
Type: java
Source: file:///Users/tkral/tmp/odo-examples/fuse-rest-http-booster

Environment variables:
 - foo=bar

Storage:
 - myjava-zakj of size 1G mounted to /data

odo describe  0.20s user 0.05s system 5% cpu 4.542 total

20 seconds for simple description of a component is too much

It's not 20 seconds, it's 0.2 seconds… It might still be too slow for you but I find it acceptable…

Ah, you are right :flushed: , my bad. But it felt like 20 :grimacing:

I've noticed slowness yesterday on a very bad network where commands would hang despite the result being already returned and displayed… Didn't look into it, though… 😞

fixed
/close


Add ellipses to the highlighted snippets

See https://quarkusio.zulipchat.com/#narrow/stream/187038-dev/topic/search.2Equarkus.2Eio/near/431900183

BTW, I wouldn't want anyone to think I'm not a nitpicker, so... There's a dedicated unicode character for an ellipsis, which is different from three dots: https://www.toptal.com/designers/htmlarrows/punctuation/horizontal-ellipsis/

Still think this is a presentation problem that should be handled in the UI

That's why I've opened it as a draft 😃 so you can decide which way to go 🙈

Still think this is a presentation problem that should be handled in the UI

That's why I've opened it as a draft 😃 so you can decide which way to go 🙈

Right, thanks for opening it. Not in the mood to fight, so let's do this :)

BTW, I wouldn't want anyone to think I'm not a nitpicker, so... There's a dedicated unicode character for an ellipsis, which is different from three dots: https://www.toptal.com/designers/htmlarrows/punctuation/horizontal-ellipsis/

WOW there's also VERTICAL ELLIPSIS 🙈  never even thought that there was something like that 🙈. let me update the PR.

BTW, I wouldn't want anyone to think I'm not a nitpicker, so... There's a dedicated unicode character for an ellipsis, which is different from three dots: https://www.toptal.com/designers/htmlarrows/punctuation/horizontal-ellipsis/

WOW there's also VERTICAL ELLIPSIS 🙈 never even thought that there was something like that 🙈. let me update the PR.

And diagonal :grin: Unicode is wild.

Thanks @marko-bekhta :)


Model usage for Property Prediction

Hi,
Thanks for releasing the code. I was trying out the trained model for Property Prediction - Clintox. I used the following snippet (as suggested).
from generation import LlaSMolGeneration

generator = LlaSMolGeneration('osunlp/LlaSMol-Mistral-7B')
generator.generate('Is <SMILES> COC[C@@H](NC(C)=O)C(=O)NCC1=CC=CC=C1 </SMILES> toxic?')

It runs; however, it always responds with "No" for all the molecule in "data/raw/test/property_prediction-clintox.jsonl". Same issue was observed when I tried on some other molecules. Can you please look into this? Please let me know if I need to change something while running.
Thanks!

Thank you for bringing this to our attention. Since most compounds in the training data are not toxic, the model may have inevitably learned the bias and tend to respond "No". Larger-scale and more balanced training data would alleviate this problem. Thanks again for your interest.

Alright. Thanks for looking into this.


conan: fix Windows build and upgrade to 1.60.2 for macOS

This PR was initially conceived to fix the build on Windows after #1593, but then macOS job started to fail intermittently 😭 so, in order to restore full CI green status, it now contains the following changes:

fix Windows build broken after #1593 by not enabling option asio_no_deprecated
update Conan to version 1.60.2 on macOS CI job to avoid this error occurred here
downgrade Python to version 3.11 because version 3.12 (now present in macos-13 GA image) triggers another error, which will be fixed in not-yet-available Conan 1.62.0

Shall we update the Conan version in the README as well?

Sure, thanks!

I have updated the other jobs as well 🤞


Add button for resetting symbol in symbol explorer

When we did this, having a "Restart" or "Start-Over" button was a useful feature:


XFAIL Sourcery 4.0 hash on master, swift-4.2-branch

A recent Swift bug fix has disallowed code that was previously allowed, causing some source breakage. (See the following bug for more information: SR-7273)
This PR adds an XFAIL to the 4.0 configuration of Sourcery (on master and swift-4.2-branch), until an updated hash is submitted.
Note: The XFAIL from SR-6556 is no longer valid, and is also removed as part of this PR.

@swift-ci test

@swift-ci test

@swift-ci test

PR testing failed because another project is failing in the master suite. #171 should resolve this failure. Will test again once #171 is merged.

@swift-ci please test

@swift-ci please test

@swift-ci please test


Fix typo on last enqueued

Coverage remained the same at 91.858% when pulling 31e316d05e5d242be113c596346b4bc1c223ecd1 on cabello:last-enqueued-typo into e8f5a627731a192f24dce9c1fa0536cd91a904d4 on ondrejbartas:master.

@ondrejbartas any chance you could take a look at this PR please?


lixee expose EASF01 (HC) and  EAS02(HP) as home assistant statistics

Been running Zlinky_TIC for a week now - all perfect - thankyou.
Currently only the total energy is exposed as a statistic and so is easy to add home assistant energy dashboard.
// For HomeAssistant
const indexEnergy = () => exposes.numeric('energy', ea.STATE).withUnit('Wh').withDescription('Index Base').withProperty('currentSummDelivered');

In my case the metrics EASF01 and EASF02 seem to correspond the Wh accumulated in off-peak (HC) and peak(HP)
If these were both statistics  I could add these two statistics as independent energy inputs and improve the visualisation.
No clue how standard that mapping is, a quick google suggests it is not.
Thanks again.
Steve.
HA Statistic - https://www.home-assistant.io/more-info/statistics/

In fact this can be achieved with a customization to add attributes to the
sensor of choice.
homeassistant:
  customize:
    sensor.linky_easf01:
       friendly_name: "Total Off-Peak Rate Energy (HC)"
       state_class: total
       device_class: energy
    sensor.linky_easf02:
       friendly_name: "Total Peak Rate Energy (HP)"
       state_class: total
       device_class: energy

As you said, it can be done from HA. I'm not sure that is something related to Z2M, since last version already publish Wh/kWh attributes with correct device_class (energy)


Fix package entry point and default end position value on Android

Closes #52
Technically, only 0fd376a004c209479a67499a08f2e9ee7912811f is needed to close #52. b63f00ab0bf77b275703c9eef3664ecafc30f502 is another Android problem I've found while working on this issue. Let me know if I should open a separate pull request for it.

Looks good to me! Thanks @dgladkov

Pushed to npm in 1.3.2 :)


How to sort endpoints?

I apologize if I have missed something obvious but I'd like to sort the endpoints alphabetically. Is this possible?

At the moment, but I am working on this for the next release

I've done this on the add customizations method wanna a PR for this?

Basic SwaggerUI settings are exposed (#519)


add SPIKE Essential hub

Did you read this from the advertising data and/or the hub properties?

By asking the hub for a update on its HubProperty hub_kind:
>>> HubPropertyRequestUpdate(HubProperty(11))
[22:32:46.022] INFO: sending: HubPropertyRequestUpdate(<HubProperty.HUB_KIND: 11>)
[22:32:46.230] INFO: received: HubPropertyUpdate(<HubProperty.HUB_KIND: 11>, <HubKind.TECHNIC_SMALL: 131>)

I am surprised by the missing 0x82 too..

I am surprised by the missing 0x82 too..

Maybe it is supposed to be Robot Inventor (eventually)?

Merged, thanks!


Uncaught Error: Command failed: cd /media/solarpolarman/hdd/code/MushBot/.git

[Enter steps to reproduce below:]

Have "Hide VCS Ignored Files" and "Hide Ignored Names" ticked in tree-view's settings.
Untick them.

Atom Version: 1.6.0
System: elementary OS
Thrown From: git-time-machine package, v1.2.6
Stack Trace
Uncaught Error: Command failed: cd /media/solarpolarman/hdd/code/MushBot/.git

Are you on windows?

I have similar error on Windows

I'm on elementary OS which is based off Ubuntu 14.04


Helpers to benchmark a model

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.19% :tada:

Comparison is base (085124e) 79.82% compared to head (a60c26a) 80.02%.

:exclamation: Current head a60c26a differs from pull request most recent head 95575bd. Consider uploading reports for the commit 95575bd to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #77      +/-   ##
==========================================
+ Coverage   79.82%   80.02%   +0.19%
==========================================
  Files          31       32       +1
  Lines        2825     2853      +28
==========================================
+ Hits         2255     2283      +28
  Misses        570      570

Files Changed
Coverage Δ

onnx_extended/tools/test_onnx.py
100.00% <100.00%> (ø)

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


logging

When running embulk with --log parameter output is bit inconsistent(especially when running embulk with crontab where I'm trying to avoid unnecessary output).
So what happens currently

In errorless run

Embulk version (Embulk v0.8.14) is printed to stdout, everything else goes to --log file

Errorfull run

Embulk version is printed to stdout
Actual error message/stacktrace in printed to stdout
Everything else goes to --log file

The following 2016-10-28 22:10:20.281 +0900: Embulk v0.8.14 part is the output before setup
logger.
The first version output is the co-developer policy.
It is because of the following reason.

JRuby setup is very slow. This output is the sign Embulk working...
We want to know version number when someone asks questions.

embulk run --log /dev/null config.yml
2016-10-28 22:10:20.281 +0900: Embulk v0.8.14

Please redirect to /dev/null like the following
embulk run --log /dev/null config.yml > /dev/null

Ok. So this an feature. Closing


Throw SQLException from ResultSetAdapter#close()

This method currently throws Exception, but could be refined to throw the more specific SQLException.

This is done.


Duplicated Oncotree Codes: IDCS FDCS MPN

These codes are duplicated in the new "Edited Oncotree" worksheet. IDCS is "Interdigitating Dendritic Cell Sarcoma (IDCS)" ... and on one row it has parent Dendritic Cell Sarcoma (DCS) and in another row it has parent Histiocytic and Dendritic Cell Neoplasms (HDCN)
This would no longer be a tree if a node has two separate parents.
The same situation exists for Follicular Dendritic Cell Sarcoma (FDCS)
MPS is simply duplicated on lines 785 and 789 .. or some info is missing from 789 to create another lower level node.

Resolved by removing MPS duplicate, and by setting MainType for IDCS and FDCS and removing the duplicate from the SOFT_TISSUE tree


Fix Equals of AutoCompleteResponse and simplify some code

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: ShuiRuTian sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Who is actually still using the AutoComplete endpoint, and can they move to the completion endpoint?

OmniSharp-vim uses it still. We can look at moving to the new completion endpoint but it won't be today :smile:


NCCL backend in DeepSpeed not yet implemented

Warning in training log：
[WARNING] [comm.py:152:init_deepspeed_backend] NCCL backend in DeepSpeed not yet implemented
[INFO] [comm.py:594:init_distributed] cdb=None
is this normal?

Yes. It's a generic warning in DeepSpeed that my pure NCCL comms backend (i.e. without going through torch distributed) hasn't been merged in yet.
Safe to ignore.


New version: NordSecurity.NordLocker version <IP_ADDRESS>

Created by 🥟 Dumplings in workflow run #6330.

Log
Updated: <IP_ADDRESS> → <IP_ADDRESS>
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

/azp run

https://github.com/microsoft/winget-pkgs/issues/142854


Update deployments list for the multi-task setup

Accommodate the UI for changes in https://github.com/galaxyproject/cloudlaunch/pull/85.

I feel this is ready go be merged unless there are comments?


potential security vulnerability

Currently we have no src/rest-server/package-lock.json and we are using morgan 1.9.1 now

deployed a bot to fix vulnerability automatically.


Can't read attribute exif:UserComment

magick-wasm version
0.0.28
Description
Hello, thanks for writing a great library. I've been testing reading metadata written by exiftool-vendored and I have an issue when reading exif:UserComment. It returns the string ASCII rather than the set value.
I also tried to install the package myself and debug but got the following issue when running npm install:
 401 Unauthorized - GET<EMAIL_ADDRESS>- unauthenticated: User cannot be authenticated with the token provided.

Steps to Reproduce
To reproduce:
const outputPath = '~/Desktop/img.png'
// write a comment using exiftool
  exifTool.write(outputPath, {
        UserComment: JSON.stringify({
          x: 'info'
          // more JSON
          }),
      })

then
ImageMagick.read(new Uint8Array(text), async (image) => {
  console.log(image.getAttribute('exif:UserComment')) // prints 'ASCII'
})

Images

Can you share a file that can be used to reproduce this issue. This project uses a native library that you will need to debug instead. If you would want to debug this I would advise you to debug the ImageMagick command line tools instead.
p.s. This project gets the native library from GitHub packages and that requires a token.

This project uses a native library that you will need to debug instead. If you would want to debug this I would advise you to debug the ImageMagick command line tools instead.
p.s. This project gets the native library from GitHub packages and that requires a token.

Sorry for my late response, I missed the notification! It is indeed a problem with ImageMagick itself and I can recreate it with the command line too

Can you share a file that can be used to reproduce this issue.

One is included in the original post under images (the command line image)
I'm going to close this issue as it's not an issue with the wasm package 🙂


Fix layout when there are no active instances

Fixes #38. Hopefully.

Why'd you rename bot-instances-ul to bot-instances?  Is it alright if I change it back?  It's broken the navbar on the bot instances page, because the body has the bot-instances class.

@NobodyNada I changed it because it seemed clearer. Feel free to change it to whatever you’d like 😄


From private repo

@v-alje : Thanks for your contribution! The author, @abhijeetgaiha, has been notified to review your proposed change.


pylyzer: 0.0.66 -> 0.0.67

Things done
Changelog: https://github.com/mtshiba/pylyzer/releases/tag/v0.0.67
cc @natsukium

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 349731

x86_64-linux

:white_check_mark: 1 package built:

pylyzer

aarch64-linux

:white_check_mark: 1 package built:

pylyzer

x86_64-darwin

:white_check_mark: 1 package built:

pylyzer

aarch64-darwin

:white_check_mark: 1 package built:

pylyzer


[Feature Request] Show resources only to some players not all

I'd like to keep track of some ressources which are only important for me and some of my players, but not all of them. I don't want the others to see them at all times and clutter their window.
Could you add a permission based filter, instead of the binary "Hide/Reveal", similar to the default foundry permissions for actors, items, journals, ...?
The module Award XP creates a nice pop-up that asks, which players you want to select, if you need inspiration.

@gsterling From the FAQ in the readme:

What about player-specific resources, can they be tracked too?
No, the intent for this module was to have global numeric values everyone could keep an eye on. However, ardittristan/5eSheet-resourcesPlus would probably serve your players better by having custom resources assigned to their character sheet.

That said however, I'd be willing to take a look into it if I can lift it from Award XP. If I can plug it in in an easy way, it might be worth reviewing the idea of player-based permissions.


Update dependencies

Codecov Report
Patch and project coverage have no change.

Comparison is base (91a5c09) 100.00% compared to head (2da51b6) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #255   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          314       314
  Lines         9778      9778
  Branches       948       948
=========================================
  Hits          9778      9778

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


There was nothing detect?

I tested the sample video and had no problem recognizing and tracking it ok.
But when I use my phone to record a video about traffic flow on an overpass, it seems that the recognition and tracking don't work, also I used YOLOv8 from the reference repository, and after compiling it, I found that the video is able to recognize some content, I don't know why this is the case?

Look at scripts from line 92 to ``line 97inyolov8_deepsort_trt.py`
    for (bbox, score, label) in zip(bboxes, scores, labels):
        if label == 0 and score.item() > 0.3:
            bbox = bbox.round().int().tolist()
            cls_id = int(label)
            cls = CLASSES[cls_id]
            detections.append((bbox[0], bbox[1], bbox[2] , bbox[3], cls, score.item()))

The pre-trained model has 80 classes but I only used class 0 (person) to do in the sample video. So I used if label == 0 to filter.

Thanks.


How I can develop/contribute with a angular cli project?

Hi. I have a angular-cli project and I use ng2-daterangepicker. I found some error and fix it, but I can't compile and use new version.
What steps are needed for develop for angular cli project?
I try:

Clone repository.
Manually install jquery and other peerDependencies.
Run ngc-build script
Install this repository locally into my angular-cli project (npm install ../ng2-daterangepicker)
Remove ng2-daterangepicker/node_modules folder manually because I have errors.
Then I can't compile my project with follow errors:

ERROR in ../ng2-daterangepicker/lib/daterangepicker/daterangepicker.module.js
Module not found: Error: Can't resolve '@angular/core' in 'C:\GIT\ng2-daterangepicker\lib\daterangepicker'
ERROR in ../ng2-daterangepicker/lib/daterangepicker/daterangepicker.component.js
Module not found: Error: Can't resolve '@angular/core' in 'C:\GIT\ng2-daterangepicker\lib\daterangepicker'
ERROR in ../ng2-daterangepicker/lib/daterangepicker/config.service.js
Module not found: Error: Can't resolve '@angular/core' in 'C:\GIT\ng2-daterangepicker\lib\daterangepicker'
ERROR in ../ng2-daterangepicker/lib/daterangepicker/daterangepicker.component.js
Module not found: Error: Can't resolve 'bootstrap-daterangepicker' in 'C:\GIT\ng2-daterangepicker\lib\daterangepicker'
ERROR in ../ng2-daterangepicker/lib/daterangepicker/daterangepicker.component.js
Module not found: Error: Can't resolve 'jquery' in 'C:\GIT\ng2-daterangepicker\lib\daterangepicker'
ERROR in ../ng2-daterangepicker/lib/daterangepicker/config.service.js
Module not found: Error: Can't resolve 'jquery' in 'C:\GIT\ng2-daterangepicker\lib\daterangepicker'

Is it right way?
If I install module from github, source code not installed, and lib folder not exist. Why?

Closed due to inactivity


Another take on a component router

I was inspired by react-router to create something similar (but simpler) for hyperapps. The main difference between this router and the old router is that there is very little state/actions here, it is almost entirely Component based.
Highlight features include:

Dynamic nested routes.
No need to pass actions.router.go to Link.
Route guards using Redirects.
Shiny new documentation.
Route params and queryString parsing.

It works! No tests yet though 🙃 not a complete PR but looking for feedback 🙇

Looking good so far.
Do you have an example prepared for data fetching?

@SkaterDad I know what you mean about the Popstate component, there was a big thread on slack today about it and for now (since the purge of init) we think it is our best option. The alternative is adding addEventListener('popstate', actions.router.update) after your app that returns actions. Technically you can do either, but having everything as a component feels more uniform and easier to explain.
So route views are just functions that return a vnode, so you can wire then with whatever state/actions you like. Personally I would curry them (like actions):
const Home = (state, actions) => ({ path, params, query }) => h1(state.message)
const Away = ({ path, params, query }) => h1('Bye')

app({
  modules: { router },
  view: (state, actions) =>
    <main>
      <Route path="/" exact="true" view={Home(state, actions)} />
      <Route path="/adventure" view={Away} />
    </main>
  ,
})

I have not got an example of data fetching.. although I am starting to thing that this might not be the routers responsibility.. rather that of a dedicated Fetch component. that can live inside of route views. That will probably be the next component I try build.

@lukejacksonn Can you fix the git conflict?

What's up with this? GitHub? 😆

This branch has conflicts that must be resolved.


support for removing mingw32

I removed mingw32 from my /etc/pacman.conf since I have no need for it.
How can I make pacboy never try the mingw32 version so I don't need to always add :x?

https://github.com/msys2/pactoys/pull/6 will change default translation to the current active environment.


Using the 'real' module should be configurable

This pull request resulted in the 'real' module being loaded if it wasn't mocked out https://github.com/plasticine/inject-loader/pull/7
Ideally this should be configurable - and present a warning that the module isn't being mocked out.
There should also be some documentation around this as it's not clear the changes on upgrading from 1.0.0 to 2.0.1.

+1. Is call-thru to the original dependency supported at all currently? I'm migrating from proxyquire and would like to make use of this to reduce the amount of work I need to do.

Hey there. 👋  Bit overdue, but trying to get back to these.
I’m not sure I understand — currently if a replacement is not passed to the created module inject function then the original is used. This is the current behaviour, but could certainly use better docs (the whole module could!).
If I’m missing something then please feel free to clarify and re-open.