CanvasLayer moves Area2D collision only visually, but not actually

Godot version
3.4.2 stable, 3.4.3 stable, 4.0 alpha3
System information
Win 64 Nvidea 765M
Issue description
The whole purpose of the CanvasLayer node is often to just move it's children along with the Camera obove gameplay.
This works, however if there is a Area2D child of the CanvasLayer node, the collision detection area does not move, even though it is visually represented as moving along as it should:
https://user-images.githubusercontent.com/47016402/155624974-3262abb9-0fdf-45cc-9c06-bd8c560e0c75.mp4
This is very problematic for my card game project where I have a large table spanning over multiple screens and therefore need a camera which moves, but also need cards on that table to interact with Area2Ds "sticking" to the camera, like it already visually does, not as it does here in the bug where it actually sticks to the world position.
Steps to reproduce
1.) Add a Camera2D which moves when press SPACE
2.) Add a Area2D which moves with the mouse
3.) Add a CanvasLayer
4.) Add a Area2D as child to the CanvasLayer to interact with the mouse-Area2D.
5.) Move the Camera2D to observe the stuck collision detection
possibly related to:
https://github.com/godotengine/godot/issues/35965
https://github.com/godotengine/godot/issues/35052
Minimal reproduction project
Godot 3.4.2 stable and 3.4.3 stable minimal project:
CanvasLayer_Area2D_bug.zip
Godot 4.0 alpha3 minimal project:
CanvasLayer_Area2D_bug_G4_a3.zip

extends Control

var cam_x_position = 0

func _physics_process(delta):
	$mouseArea2D.global_position = get_global_mouse_position()
	if Input.is_action_just_pressed("ui_accept"):
		cam_x_position += 50
	$Camera2D.global_position.x = lerp($Camera2D.global_position.x, cam_x_position, 0.1)

func _on_canvaslayerArea2D_area_entered(area):
	$CanvasLayer/stateLabel.text = str(area.name," << entered")

func _on_canvaslayerArea2D_area_exited(area):
	$CanvasLayer/stateLabel.text = str(area.name," >> exited")

@Calinou I've also tested it in current stable version and Godot4 apha3, with the exact same result. Does it need more testing still (regarding the "needs testing" label)?

Does it need more testing still (regarding the "needs testing" label)?

needs testing means that someone else should try to confirm this issue.

Still happening in Godot Engine v4.1.1.stable.official

Restested this bug on Godot 3.5.3 RC1 and Godot 4.2 Dev4 to see if https://github.com/godotengine/godot/pull/70310 changed anything. But nope. Bug is still present.

I can confirm that the bug still occurs. In fact, I think it happens with all collision shapes or shapecasts, not just children of Area2D. Any collision shape or shapecast that is a descendant of a CanvasLayer seems to be affected.
Has anyone found a workaround? Somehow my objects seem to be jerky when making them a child of my camera.

In my opinion the bug is referred to the signals to manage collisions in Area2D, that signals allows to detect collisions on objects placed in different spatial contexts with the same positions, but the same position in different context are different positions.
To explain better: the canvasLayerArea2D in the CanvasLayer have specific position for example Vector2(100, 200), the mouseArea2D is not placed in the same CanvasLayer and it's takes the position of the mouse by get_global_mouse_position(), but this is the position based on the main CanvasLayer and I supposed returns viewport global coordinates based on the game window.
This means, at the start the positions are the same, but when you move along the sub CanvasLayer with Camera2D, the positions are not longer comparable, because the canvasLayerArea2D is shifted, but the area_entered signal is always incorrect triggered at the position Vector2(100, 200) of the viewport global coordinates.
Probably is necessary to rework Area2D signals to manage objects from different spatial context
Possible workaround in the example (not tested):
Move mouseArea2D in the CanvasLayer and gets the mouse coordinates referred to the local context :
$CanvasLayer/mouseArea2D.position = $CanvasLayer.make_canvas_position_local(get_global_mouse_position())

I'm currently running into this issue in a drag-and-drop scenario:

tilemapped environment with a camera in canvaslayer A, which follows the viewport
UI in canvaslayer B, which doesn't
item being carried with the mouse from the UI in B, onto the tilemap in A, using areas to validate the drop location

Based on the notes in the reverted fix, this sounds like a relatively deep bug in godot's various coordinate systems & transforms... so until that is untangled, I'm curious: does anyone have an elegant workaround?
My current plan is to remove all the canvaslayer and cameras, and instead emulate a "camera" by translating the tilemapped environment layer. I'm sure that's inefficient, since moving the whole map probably invalidates lots of things (physics?), but I'm not sure what the real penalty would be. Alternatively, I'm considering hacking it by cloning the Area2D into the environment, but that will couple together several different components that really shouldn't know about one another.

Godot Engine v4.3.stable.official it still happens


onDoubleTap should add pause playback and resume playback

I tried to use it, but it doesn't work when going to full screen.
AspectRatio(                         aspectRatio: 16 / 9,                         child: GestureDetector(                           onDoubleTap: () {                             if (_controller.playerStatus.playing) {                               _controller.pause();                             } else {                               _controller.play();                             }                           },                           child: MeeduVideoPlayer(                             controller: _controller,                           ),                         ),                       )

I tried to use it, but it doesn't work when going to full screen.
AspectRatio(                         aspectRatio: 16 / 9,                         child: GestureDetector(                           onDoubleTap: () {                             if (_controller.playerStatus.playing) {                               _controller.pause();                             } else {                               _controller.play();                             }                           },                           child: MeeduVideoPlayer(                             controller: _controller,                           ),                         ),                       )

You are wrapping the main screen not the Fullscreen too,

in 4.2.24
you can add the overlay like this, will work on both fullscreen and normal mode
MeeduVideoPlayer(
           controller: _meeduPlayerController,
           videoOverlay: (context, controller, responsive) => Container(
             color: Colors.red,
             child: SizedBox(
               height: 50,
               width: 50,
             ),
           ),
           header: (context, controller, responsive) => header,
         ),


Add ProtectedFileStream::WriteAsync()

ReadAsync() is currently available but not WriteAsync(). WriteAsync() wasn't added because writes aren't thread-safe. After discussion, we decided we should still provide a non-thread-safe WriteAsync() which sets up a worker thread calling Write(). It is understood that only one thread at a time should be working with one target stream and that neither Write() nor WriteAsync() is thread-safe.

Done by https://github.com/AzureAD/rms-sdk-for-cpp/pull/26

Thanks!


Support for setting locale

Currently locale is hardcoded to en_GB. It is possible to "monkeypatch" this using code like the following, but it would be nicer to expose the internationalization support built into the faker library from your public API.
import anonymizedf.anonymizedf as adf
adf.fake = Faker(locale="sv_SE")

Sure, I could update that to just have en_GB as a default parameter but allow for setting the locale.
I guess  like something like this 🤔
anonymize(df, locale="sv_SE")


fix reading preshared keys

lookup('file') does not work with binary files and causes failures depending on the content of a preshared key file

centos-8 is expected to fail, a fix for that is in #11

@tomjelinek any ideas why we're getting the rhel-8-y failures?
TASK [linux-system-roles.ha_cluster : Install cluster packages] ****************
task path: /tmp/tmp98asuavs/tasks/install-and-configure-packages.yml:30
fatal: [/cache/rhel-8-y.qcow2]: FAILED! => {"changed": false, "failures": [], "msg": "
Depsolve Error occured:
Problem 1: conflicting requests
  - nothing provides corosynclib(x86-64) = 3.1.0-4.el8 needed by corosync-3.1.0-4.el8.x86_64
Problem 2: package pacemaker-2.0.5-9.el8.x86_64 requires corosync >= 2.0.0, but none of the providers can be installed
  - conflicting requests
  - nothing provides corosynclib(x86-64) = 3.1.0-4.el8 needed by corosync-3.1.0-4.el8.x86_64
", "rc": 1, "results": []}

We're using the nightly/latest-RHEL-8/compose/HighAvailability repo.

Several cluster components (including corosync) are split into multiple rmps. Some of those rpms go to rhel-appstream repo, while the others go to rhel-HighAvailability. Therefore, these two repos must point to the same release. If rhel-appstream is 'latest-RHEL-8.4', then rhel-HighAvailability must be 'latest-RHEL-8.4' as well.

[citest bad]

[citest pending]

[citest pending]

[citest pending]

[citest bad]

[citest pending]

[citest bad]

[citest pending]


Option to rerun only requested tasks

Expected Behavior
As a developer or DPE engineer, I want to rerun a given task, regardless if it's up to date or is cacheable. Typical scenarios are for benchmarking compilation tasks, debugging flaky tests, refreshing broken cache artifacts, etc.
I would like a flag (or equivalent) allowing me to run ./gradlew :path:to:my:task --rerun-requested-tasks. The applicable task(s) should be subject to the regular Gradle task matching and abbreviations, so that tas would match all task instances in all included projects.
Current Behavior
Using --rerun-tasks runs all the tasks up to and including the requested tasks, which wastes time and resources.
The closest approximation of the requested feature is to invoke ./gradlew :path:to:my:cleanTask :path:to:my:task --no-build-cache which is both tedious and error-prone.
Context
I have a need for this feature on at least a weekly basis, based on the given use-cases.
When a developer in a large repo has broken cache artifacts for a given task, it's performance prohibitive to tell the developer to run with --rerun-tasks, but it's also inconvenient to tell them to use ./gradlew :path:to:my:cleanTask :path:to:my:task --no-build-cache, as the cache artifacts won't be refreshed.

I agree this would be nice, where I work I added a property which is a comma separated list of tasks, and there's some configuration that marks the output of those tasks as upToDateWhen { false }.
This has the effect of the cache not being used, and the new cache entry being pushed.
That being said, if you need this feature on a weekly basis, maybe you should identify where your broken caches are coming from. Though I appreciate sometimes resolving it is out of your hands, e.g. if a plugin is broken.

Oh, my "weekly basis" is usually not due to broken cache artefacts. Benchmarking and rerunning tests are more common. I agree that addressing cache problems should be a separate activity. It's usually related to underspecified inputs, orphaned output files or out-of-order task execution.


Update connection.py

Refactor of get_connection() in order to fix the SSL problem when run the restore-backup.py script:
self._sslobj.do_handshake()
ssl.SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:590)

Thanks for the patch! The refactor does clean up the connection handling.


Basic level parsing

Current coverage is 88.26% (diff: 88.88%)

Merging #7 into master will decrease coverage by <.01%

@@             master         #7   diff @@
==========================================
  Files             5          5
  Lines           179        213    +34
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            158        188    +30
- Misses           21         25     +4
  Partials          0          0

Powered by Codecov. Last update 3714cde...aa5167e


Normalizer issue #46

Simplified normalizer.

From doctrine/doctrine2#1332 , maybe we should extends AbscractHydrator directly.

Thanks for this PR ! merged


Update 'SprtFtrs' Testing filter values for Windows 11

[Can you please look at updating the 'SprtFtrs' Testing filter values for Windows 11, as they appear to be HEX now when you run the 'fltmc instances -v' command. Value example "0000000f".]

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

ID: 2eb725b8-9082-fd0e-c616-6e7477d1864d
Version Independent ID: 5bb5bdde-fb2e-f99f-f59e-f3e0e0acbd57
Content: Windows Offloaded Data Transfers - Windows drivers
Content Source: windows-driver-docs-pr/ifs/offloaded-data-transfers.md
Product: windows-hardware
Technology: ifs
GitHub Login: @lorihollasch
Microsoft Alias: loriwhip

@LHammondAritzia , I've changed the SprtFtrs values from decimal to hex!


BUG: The forgot password page has significant layout issues.

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?

The page's css needs to be corrected. It has Misaligned Elements and broken layout.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Assigned!
Make sure to attach before & after screenshots in your PR and include the issue number.
Also, please star the repo before working on it. Happy open-source contributing!


Fix GoogleMapsLoader.VERSION override problem

ISSUE: https://github.com/Carrooi/Js-GoogleMapsLoader/issues/58
Comments:

Now, the lib always append the default version to the url. (Now the dafault is v=3.27)
Now, if the user set the version (GoogleMapsLoader.VERSION = '999') without specify the client, the version override it anyway.
I've updated all the tests about #createUrl because now all urls will end with a version. Also, I've added a default version variable in test file like in lib src code.

Maybe we should update to 3.31 because is last release version

Hello and thank you.
Do you want to update the current version too? It could be done in this pull request (another commit).

@davidkudera Done. Now the default version is 3.31

Nice, thanks again :beers:

Hi @TheBearSO ;
I'm not able to override to use 3.27 version.
I'm using webpack like that
// ...
var GoogleMapsLoader = window.GoogleMapsLoader || require('google-maps');

// ... some stuff
GoogleMapsLoader.KEY = apiKey;
GoogleMapsLoader.LIBRARIES = ['places'];
GoogleMapsLoader.VERSION = '3.27';

Then in browser console :
GoogleMapsLoader.createUrl()
"https://maps.googleapis.com/maps/api/js?callback=__google_maps_api_provider_initializator__&key=AIzaSyBQczvNz0CTGtdZLjk0Z7Dt8RyGowMY0Lo&libraries=places"

So always loading lastest version (v3.31) :(
Any help ?
Regards

Merged, thank you :+1:


Secondary Local Index query support?

I see that this supports GSI, but am having trouble (null pointer) trying to have it query using a Local Secondary Index (LSI) field I have in my DynamoDB.  Is this supported, and if so is there an example?

Do you have the stacktrace of the NullPointerException?

when I query by Local Index Query， the mode is still SCAN mode


[NDTensors] Improve checks in Combiner for undefined contractions

Description

Fixes #1034 by adding better checks for catching that case and throwing an informative error (in both the Dense and BlockSparse cases).
Add a new test file for Combiner in NDTensors, including testing this new behavior.
Fixes an issue in BlockSparse Combiner uncombining contraction where it was assuming the indices had unique identifiers (i.e. implicitly assumed they were Index), so failed with more general block dimensions.

So now:
using ITensors
i, j = Index.((2, 2))
T = randomITensor(i)
C = combiner(i, j)
C * T

outputs:
ERROR: Trying to contract a tensor with indices:

((dim=2|id=69),)

and labels:

(-1,)

with a combiner tensor with indices:

((dim=4|id=989|"CMB,Link"), (dim=2|id=69), (dim=2|id=866))

and labels:

(2, -1, 3).

This is not a valid combiner contraction.

If you are combining, the combined index of the combiner should be the only one uncontracted.

If you are uncombining, the combined index of the combiner should be the only one contracted.

By convention, the combined index should be the index in position 1 of the combiner tensor.

Stacktrace:
  [1] error(s::String)
    @ Base ./error.jl:35
  [2] invalid_combiner_contraction_error(tensor::DenseTensor{Float64, 1, Tuple{Index{Int64}}, Dense{Float64, Vector{Float64}}}, tensor_labels::Tuple{Int64}, combiner_tensor::Tensor{Number, 3, Combiner, Tuple{Index{Int64}, Index{Int64}, Index{Int64}}}, combiner_tensor_labels::Tuple{Int64, Int64, Int64})
    @ NDTensors ~/.julia/dev/ITensors/NDTensors/src/combiner/combiner.jl:237
[...]

@kmp5VT @christopherdavidwhite

Codecov Report

Merging #1074 (4555cc7) into main (76db1d5) will decrease coverage by 26.31%.
 Consider uploading reports for the commit 02e55d8 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##             main    #1074       +/-   ##
===========================================
- Coverage   67.81%   41.50%   -26.31%
===========================================
  Files          95       94        -1
  Lines        9999     9362      -637
===========================================
- Hits         6781     3886     -2895
- Misses       3218     5476     +2258

Impacted Files
Coverage Δ

src/adapt.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/mps/adapt.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/mps/projmpo.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/mps/observer.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/physics/autompo/matelem.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/physics/autompo/qnmatelem.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/physics/autompo/opsum_to_mpo.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/physics/autompo/opsum_to_mpo_qn.jl
0.00% <0.00%> (-97.62%)
:arrow_down:

src/physics/autompo/opsum_to_mpo_generic.jl
0.00% <0.00%> (-95.75%)
:arrow_down:

src/mps/mps.jl
0.00% <0.00%> (-90.38%)
:arrow_down:

... and 69 more

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


Build IREE for an arm platform

Hi,
I am trying to build IREE compiler into and aarch64 platform. When I compile it I faced the above error. Any ideas?

It looks like your sdcard is formatted in a way that doesn't support symlinks. Most likely fat i would guess. Under Linux, we assume that symlinks can be created.
You'll know if this is the issue if the following command from an sdcard directory fails:
ln -s foobar .

(you will need to build on a native Linux filesystem like ext4)

It looks like your sdcard is formatted in a way that doesn't support symlinks. Most likely fat i would guess. Under Linux, we assume that symlinks can be created.
You'll know if this is the issue if the following command from an sdcard directory fails:
ln -s foobar .

Thanks for your answer. That is correct, the issue is the one you mentioned. But how I will solve this if I don't have enough space on a native Linux file system (only 32 gb)?

A few other hints (but I don't know what you are trying to do so may not apply):

Smaller platforms often just need the runtime and not the compiler (which is the most expensive part of the build). Disable it with -DIREE_BUILD_COMPILER=OFF
Debug information bloats the build size. Building with -DCMAKE_BUILD_TYPE=Release should help.
Enabling these two options in Linux can save a lot of disk space in the build directory for debug builds: https://github.com/iree-org/iree/blob/663b1a7b541e7d511675c45667cf39304e6b9eec/CMakeLists.txt#L269

A few other hints (but I don't know what you are trying to do so may not apply):

Smaller platforms often just need the runtime and not the compiler (which is the most expensive part of the build). Disable it with -DIREE_BUILD_COMPILER=OFF
Debug information bloats the build size. Building with -DCMAKE_BUILD_TYPE=Release should help.
Enabling these two options in Linux can save a lot of disk space in the build directory for some builds: https://github.com/iree-org/iree/blob/663b1a7b541e7d511675c45667cf39304e6b9eec/CMakeLists.txt#L269

Yes that is a very good advice thanks !!


Send my custom snippets

Your extension is awesome and very easy to use. There is any way to send you my custom snippets?

Yes please! You could send your snippets to my email address crazyzyt#gmail.com (# for @) to simply leave them here.

I forgot to say... Now there's no way to export the snippets, so you could click Debug menu -> Open Preferences File to open brackets.json, and send the contents of edc.brackets-snippets.hints to me.

Ok, but I have a problem with your plugin. Some times don't load. There is any shortcut to open the snippet manager?

No shortcuts because I think there's already a bulb button on the edge right. Do you mean the button

didn't work? If so, I think there must be some errors I haven't encountered with.
If you like to share some error logs(which could help me fix this), you could click Debug -> Show Developer Tools, and the logs would be listed on Console tab :smiley:

You have an email :).


Support constructor in protectedMethods

I would like the generated constructor to be protected and require me to implement a constructor in my partial implementation. Can this be supported?

+1... the scenario I'm looking at is making the c'tor protected, then create an overload in the partial class in order to create a constructor that ASP.NET Dependency Injection can use.

Can you propose a PR with this change?
Do we need a new config to avoid breaking changes?

+1 one for the feature


Problem with Sponge on my server

Forge version: 1.8-<IP_ADDRESS>7
Sponge: 1.8-1371-2.1DEV433
I have a problem when i put sponge on my server (into the mods folder)
When i have a mod in it: it's ok but when it s sponge, i suddenly cannot make anything with the RCon or something else. The error message when i try to make any commands in the rCon:
"Error executing: help (net.minecraft.network.rcon.RConConsoleSource cannot be cast to org.spongepowered.api.util.command.CommandSource)"
I don t uderstand what i need to do. Can you help me please

Rcon support hasn't been implemented from the API.

Excuse me but i'm belgian, so i don't understand very well what you mean in your sentence.
(and I'm sorry for every mistake I can do)

@lncogmito
Its a bug with Sponge and we'll get it fixed up as soon as we can.

Ok, i just understood your first answer. Can we know how we will know when this bug will be fixed up?

@lncogmito
Depends on people's availability. You shouldn't be trying to run Sponge as your live server, it should only be used for testing.

Ok thank you
I hope this report will help you in the sponge's conception

I don't understand what are the reference you are giving zml :/

I guess he referenced this Pr in the commits that (partially or fully) fixes this issue.

That is the commit that fixed this issue. Guess @Zidane accidentally stripped that out when merging the PR.


Care to craft a release?

The latest release is more than two years old and a couple of fixes have piled up but can't be used as there's no official release containing those.

Besides your change for 'todo' the rest are mostly cosmetic changes, but sure I will do the release today in the evening.

Awesome, thanks!


Change giganto_ingest_addrto giganto_ingest_srv_addr

Close: #471

Codecov Report
Attention: Patch coverage is 0% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 21.45%. Comparing base (00a78a8) to head (e2dad9a).

Files
Patch %
Lines

src/producer.rs
0.00%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #474      +/-   ##
==========================================
+ Coverage   21.05%   21.45%   +0.39%
==========================================
  Files          47       47
  Lines        8810     8643     -167
==========================================
- Hits         1855     1854       -1
+ Misses       6955     6789     -166

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


MarkdownString.appendCodeblock is not escaping code block border.

Does this issue occur when all extensions are disabled?: Yes

Version: 1.82.2 (user setup)
Commit: abd2f3db4bdb28f9e95536dfa84d8479f1eb312d
Date: 2023-09-14T05:55:25.390Z
Electron: 25.8.1
ElectronBuildId: 23779380
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
Steps to Reproduce:
console.log(new vscode.MarkdownString().appendCodeblock('```', txt")).value

Got:
```txt
```
```

Expected(code block border is escaped with more ` or ~):
````txt
```
````

I would like to work on this.


[Event Request] CU 1303 "Correct Posted Sales Invoice" procedure TestCorrectInvoiceIsAllowed

Hello,
We have invoices with zero amount and we need to create correction.
We like to request the following Event:
Object: CorrectPostedSalesInvoice.al
Where is the event placed:
procedure TestCorrectInvoiceIsAllowed(var SalesInvoiceHeader: Record "Sales Invoice Header"; Cancelling: Boolean)
    var
        IsHandled: Boolean;
    begin
        CancellingOnly := Cancelling;

        SalesInvoiceHeader.CalcFields(Amount);
        // >> ZS
        IsHandled := false;
        OnTestCorrectInvoiceIsAllowedOnBeforeTestAmount(SalesInvoiceHeader, Cancelling, IsHandled);
        if not IsHandled then begin
            // << ZS
            SalesInvoiceHeader.TestField(Amount);
            // >> ZS
        end;
        // << ZS
        TestIfPostingIsAllowed(SalesInvoiceHeader);
        TestIfInvoiceIsCorrectedOnce(SalesInvoiceHeader);

Event Code:
// >> ZS
    [IntegrationEvent(false, false)]
    local procedure OnTestCorrectInvoiceIsAllowedOnBeforeTestAmount(var SalesInvoiceHeader: Record "Sales Invoice Header"; var Cancelling: Boolean; var IsHandled: Boolean);
    begin
    end;
    // << ZS

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


install chaincode error

when I up network.I get error when install chaincode.
# Run the chaincode install script on cli container ***********
  non-zero return code
  go<EMAIL_ADDRESS>Get<EMAIL_ADDRESS>dial tcp <IP_ADDRESS>:443: i/o timeout
  Error: failed to normalize chaincode path: 'go list' failed with: go<EMAIL_ADDRESS>Get<EMAIL_ADDRESS>dial tcp <IP_ADDRESS>:443: i/o timeout: exit status 1
  Error: failed to read chaincode package at 'simple_go_1.0.tar.gz': open simple_go_1.0.tar.gz: no such file or directory

Check if you have internet access so it can be downloaded the protobuf package

Check if you have internet access so it can be downloaded the protobuf package

Please add more detailed steps for recreating the error. Other wise, there is not much to go on with.

Please add more detailed steps for recreating the error. Other wise, there is not much to go on with.

I have downloaded the latest version of Minifab docker image, which is on Jan 27th. When I install a contract in go language, there is still an error about the go proxy.

I have downloaded the latest version of Minifab docker image, which is on Jan 27th. When I install a contract in go language, there is still an error about the go proxy.

It looks like the GOPROXY env is not set when running installcc docker run.

It looks like the GOPROXY env is not set when running installcc docker run.

@Mavlarn as I have asked earlier, you need to detail what you have done and in what kind of environment, normally you would need to describe each step from beginning so that ones who like to help can possibly know what you've been through. For this kind of errors, my guess is that you are in an environment, access to google site is restricted. To help with that, you will need to create your own spec.yaml file and specify the go proxy as the default spec.yaml file already has but commented out. If the sample proxy is not working for you, you will need to find a proxy which works for you.

@Mavlarn as I have asked earlier, you need to detail what you have done and in what kind of environment, normally you would need to describe each step from beginning so that ones who like to help can possibly know what you've been through. For this kind of errors, my guess is that you are in an environment, access to google site is restricted. To help with that, you will need to create your own spec.yaml file and specify the go proxy as the default spec.yaml file already has but commented out. If the sample proxy is not working for you, you will need to find a proxy which works for you.

I updated with latest shell and docker images, when I run up, the default simple chaincode contract of go version can be installed.
But, after I created a new channel, and prepared custom chaincode, when I try to install the new chaincode, there is always and error as this:
TASK [Run the chaincode install script on cli container] ***********************
fatal: [minifab]: FAILED! => {"changed": true, "cmd": ["docker", "exec", "d5ec399859_cli", "/vars/run/ccinstall.p1.bc.starlets.com.cn.sh"], "delta": "0:00:31.181495", "end": "2021-01-29 16:45:29.588752", "msg": "non-zero return code", "rc": 1, "start": "2021-01-29 16:44:58.407257", "stderr": "Error: chaincode install failed with status: 500 - failed to invoke backing implementation of 'InstallChaincode': could not build chaincode: docker build failed: docker image build failed: docker build failed: Error returned from build: 1 \"go<EMAIL_ADDRESS>Get<EMAIL_ADDRESS>dial tcp <IP_ADDRESS>:443: i/o timeout\n\"", "stderr_lines": ["Error: chaincode install failed with status: 500 - failed to invoke backing implementation of 'InstallChaincode': could not build chaincode: docker build failed: docker image build failed: docker build failed: Error returned from build: 1 \"go<EMAIL_ADDRESS>Get<EMAIL_ADDRESS>dial tcp <IP_ADDRESS>:443: i/o timeout", "\""], "stdout": "", "stdout_lines": []}

PLAY RECAP *********************************************************************
minifab                    : ok=24   changed=0    unreachable=0    failed=1    skipped=26   rescued=0    ignored=0

And I also tried with the sample chaincode in fabric-sample, it can be installed successfully.
I am not sure if there is any error in my chaincode. I created it in vscode, and installed related extensions. There is no error in vscode.  At least it should compile successfully.
So, is there any way I can check the reason it fails? Thanks.

I updated with latest shell and docker images, when I run up, the default simple chaincode contract of go version can be installed.
But, after I created a new channel, and prepared custom chaincode, when I try to install the new chaincode, there is always and error as this:
TASK [Run the chaincode install script on cli container] ***********************
fatal: [minifab]: FAILED! => {"changed": true, "cmd": ["docker", "exec", "d5ec399859_cli", "/vars/run/ccinstall.p1.bc.starlets.com.cn.sh"], "delta": "0:00:31.181495", "end": "2021-01-29 16:45:29.588752", "msg": "non-zero return code", "rc": 1, "start": "2021-01-29 16:44:58.407257", "stderr": "Error: chaincode install failed with status: 500 - failed to invoke backing implementation of 'InstallChaincode': could not build chaincode: docker build failed: docker image build failed: docker build failed: Error returned from build: 1 \"go<EMAIL_ADDRESS>Get<EMAIL_ADDRESS>dial tcp <IP_ADDRESS>:443: i/o timeout\n\"", "stderr_lines": ["Error: chaincode install failed with status: 500 - failed to invoke backing implementation of 'InstallChaincode': could not build chaincode: docker build failed: docker image build failed: docker build failed: Error returned from build: 1 \"go<EMAIL_ADDRESS>Get<EMAIL_ADDRESS>dial tcp <IP_ADDRESS>:443: i/o timeout", "\""], "stdout": "", "stdout_lines": []}

PLAY RECAP *********************************************************************
minifab                    : ok=24   changed=0    unreachable=0    failed=1    skipped=26   rescued=0    ignored=0

And I also tried with the sample chaincode in fabric-sample, it can be installed successfully.
I am not sure if there is any error in my chaincode. I created it in vscode, and installed related extensions. There is no error in vscode.  At least it should compile successfully.
So, is there any way I can check the reason it fails? Thanks.

@Mavlarn your chaincode was not built. and most likely your chaincode did not compile. If the chaincode comes with minifabric worked fine, the error should be in your own chaincode, maybe missing dependencies or other issues.

@Mavlarn your chaincode was not built. and most likely your chaincode did not compile. If the chaincode comes with minifabric worked fine, the error should be in your own chaincode, maybe missing dependencies or other issues.

Thanks.
I also doubt about that. But I cannot find why. Because there is no error in my local env. If there is something missing in dependencies, there will be error in local.
And during the installing, before it fails, I got the docker inspect  for the ccenv container.  And I see the GOPROXY in env is not changed. So I thought maybe the env is not working. I attached the result of docker inspect.
inspect_ccenv.json.txt

Thanks.
I also doubt about that. But I cannot find why. Because there is no error in my local env. If there is something missing in dependencies, there will be error in local.
And during the installing, before it fails, I got the docker inspect  for the ccenv container.  And I see the GOPROXY in env is not changed. So I thought maybe the env is not working. I attached the result of docker inspect.
inspect_ccenv.json.txt

Thanks.
I also doubt about that. But I cannot find why. Because there is no error in my local env. If there is something missing in dependencies, there will be error in local.
And during the installing, before it fails, I got the docker inspect  for the ccenv container. And I see the GOPROXY in env is not changed. So I thought maybe the env is not working. I attached the result of docker inspect.
inspect_ccenv.json.txt

Have you checked the dependencies ?
The errors when you are trying to install a chain code will be display during the install command .
Also check if all the dependencies can be downloaded during installation from your first error it seems like chain code cannot find a package .
But this has nothing to do with minifabric.
Please also  install PORTAINER  and check the peer logs / orderer logs containers to see after the failed installation of the chain code .
Also try to install the default chain code coming with  mini fabric and see what happens .

After several days working on this, It can work now. At first, reply the question above:

About the dependency, I can make sure it's ok.
From the docker logs, I cannot see any helpful message, just the error as I posted above.

At first, I also thought it was because of the dependency problems. So I took much time to modify the contract and try again and again.
At last, I tried to deploy from vscode. I setup the environment and wallet, and package and deploy from vscode. During installation, there is another error, same as that in this post .
Error: chaincode install failed with status: 500 - failed to invoke backing implementation of 'InstallChaincode': could not build chaincode: docker build failed: docker image build failed: docker build failed: Error returned from build: 1 "go: inconsistent vendoring in /chaincode/input/src/github.com/hyperledger/fabric-samples/chaincode/abstore/go:
<EMAIL_ADDRESS>is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt

My error message is about some other libs, but I think it is the same reason.
It said it is some problem in go1.14. So I changed the version to 1.13. Then the installation was ok.
And at last, thanks for your help, and providing these great tool.

Please feel free to close the issue if everything works now .


Fixed SHA2

I'd prefer this to be merged before #186 gets released. #186 could and very likely will break compatibility with other users of this SHA2 library.

Sweet, thanks!


WARN: Bid depth (0) too shallow to convert

Hi!
I've recently tried to run your program, and it looks like it works great! However, I've been looking at the logs and it seems like it's been spamming me with bid depth (0) too shallow to convert. I've tried to reduce the depth in the config to 5 or 10, however it errors out.  I've looked through all of the other posts in the issues section of this Github repo, but none of the solutions seem to work. (I'm running the latest version.)
WARN: Bid depth (0) too shallow to convert
INFO: WebSocket closed: xxxxxx@depth@100ms (1006)

Bid depth (0) errors are almost always caused by a disconnected websocket, which are almost always caused by a lack of cpu resources to handle so many concurrent connections AND the computations required by the bot

You can try bundling the websockets together to reduce the number of total outstanding ws connections via WEBSOCKET.BUNDLE_SIZE

Hmmm, it seem's like the error is still happening, even though I have increased the websocket bundle size. I'm pretty sure it's not getting bottlenecked by the CPU, since it's just chillin at around 35% utilization (I'm running it on GCP, maybe that's the issue?).

Ensure that 35% isn't your overall cpu usage. This bot is single threaded, so one core might be overworked. I haven't experienced any big performance tweaks bundling websockets but the option is there still.
Personally I always ran on aws, but gcp should be just fine so long as the specs on the box are adequate

Something to keep in mind too is that really low depth cache starting values can actually be exhausted by a lot of market takers. If this is the case, you will still see depth(0) errors, but not websocket disconnection errors

I've tried a bunch of depth cache levels, however it still just spams depth(0) errors. I haven't tried it on one of my computers at home, I will do that and see if it works.

Look in your performance logs and see if there is a line like this:
CPU clock usage for calculations: ...

Hi, I really admire your work in making this program. Regarding this error, I'm also experiencing the same thing and it certainly sounds like it's due to a lack of cpu resources on my end.

Personally I always ran on aws, but gcp should be just fine so long as the specs on the box are adequate

You mentioned that you use aws. What specs do you use?

c5n.large I think. Maybe xlarge but it's been awhile

Hi
I have the same error message
I have try to increase and decrease the WEBSOCKET.BUNDLE_SIZE but it still the same
I have some websocket deconnection but not too often
the cpu usage is about 60-75% (on one core)
Could you explain what "Bid depth (0) too shallow to convert" mean ?
THK

It can be caused by a few reasons but the most common cause is when your cpu (only single core) is overwhelmed and cannot handle the ping pong websocket events so they timeout and disconnect. If your websockets are not disconnecting, this could be from using too low of a depth value and the cached depth gets exhausted at a rate higher than new orders are posted

Hi,
I am getting the same error without disconnection of websocket. I sometimes catch profit but it does not execute.
[1/4/2022, 6:11:44 AM] WARN: Ask depth (0) too shallow to convert 10 BUSD to BNB using BNBBUSD
[1/4/2022, 6:11:44 AM] WARN: Bid depth (0) too shallow to convert 0.019 BNB to BUSD using BNBBUSD
[1/4/2022, 6:11:44 AM] WARN: Ask depth (0) too shallow to convert 0.019 BNB to XTZ using XTZBNB
[1/4/2022, 6:11:44 AM] WARN: Ask depth (0) too shallow to convert 263.834 UAH to BUSD using BUSDUAH


If can't import then can repo be added to cdnjs

It would be helpful if the releases were available on https://cdnjs.com. Then we could get the files from a cdn. cdnjs is free too!
I found I could not merely instantiate the plugin with a plugins object addition and an import statement like other plugins. I had to include the dist file in my head to get it to work.
Example:
plugins: {
            chromecast:{
              receiverAppID:'3x2083xxxx',
              addButtonToControlBar: false,
            },

followed by an import statement of:
import 'videojs-hotkeys';
// more plugins.
import '@silvermine/videojs-chromecast';

Helpful links.
https://github.com/cdnjs/cdnjs
https://github.com/cdnjs/cdnjs/blob/master/CONTRIBUTING.md

I guess I am looking for the plugin to either be able to be imported and no addition to head tag, or the repo get added to cdnjs.

@daveferrara1 I'm assuming you closed it because it's already available on unpkg.com? Or for some other reason?

Always for get that stuff is available o unpkg. I just realized if I wasn't going to use require I could just do:
import chromecast from '@silvermine/videojs-chromecast'; chromecast(videojs);


dbeaver/pro#1408 convert "string" data type to text during data transfer

A better data type for the string - is a string, but not all databases support it. Second choice - is "text" or a data type with the "text" name inside, like "longtext". The last option - varchar(100) - default string value.
In the case of datatransfer from Mongo to MariaDB/MySQL - "text" data type without parameters as a target type - the best choice, imho.

verified


Optional parameters in functions are being converted as required in client code.

Describe the bug
I have a Future<String?> function(String positionalArg, {bool optional = true}){...}, And it gives as _i2.Future<String?> function(String positionalArg, {required bool isAsset}) => caller... on generation.
To Reproduce
Steps to reproduce the behavior:
In endpoints functions,
Future<String?> getUrl(Session session, String path,
        {bool isAsset = false, bool isPublic = true}) async =>
    (await session.storage.getPublicUrl(
            storageId: isPublic ? 'public' : 'private',
            path: isAsset ? 'assets/$path' : path))
        .toString();

And generate the client code.
This is what you get
_i2.Future<String?> getUrl(
  String path, {
  required bool isAsset,
  required bool isPublic,
}) =>
    caller.callServerEndpoint<String?>(
      'assets',
      'getUrl',
      {
        'path': path,
        'isAsset': isAsset,
        'isPublic': isPublic,
      },
    );

Expected behavior
This is the expected client code.
_i2.Future<String?> getUrl(
  String path, {
  bool isAsset = false,
  bool isPublic = true,
}) =>
    caller.callServerEndpoint<String?>(
      'assets',
      'getUrl',
      {
        'path': path,
        'isAsset': isAsset,
        'isPublic': isPublic,
      },
    );

Serverpod versions

CLI Version: 0.9.22
Library version: 0.9.22

Platform information
Software:

    System Software Overview:

      System Version: macOS 13.1 (22C65)
      Kernel Version: Darwin 22.2.0
      Boot Volume: Macintosh HD
      Boot Mode: Normal
      Computer Name: Mr.’s MacBook Air
      User Name: Mr. Minnu (mr.minnu)
      Secure Virtual Memory: Enabled
      System Integrity Protection: Enabled
      Time since boot: 9 days, 15 hours, 53 minutes

Responsible for this bug / behavior are this lines: https://github.com/serverpod/serverpod/blob/1fee3bab10c73d04ac890c92e3a162e79ba95103/tools/serverpod_cli/bin/generator/protocol_analyzer.dart#L144-L148
I don't know if this behavior can be solved, since the default value might be one, that is not available at the client.

Example
Future<int> returnNamedOptionalIntWithDefault(Session session,
    {int namedOptionalIntWithDefault = defaultInt}) async {
  return namedOptionalIntWithDefault;
}

static const defaultInt = 1;

The only option I see, is setting the parameter as nullable at the client side and replacing null values with the default value at the server side. (But maybe even this solution would require this dart functionality.)
As an alternative, we could ban references from default values for named parameters.


ITMs in <IP_ADDRESS> patch version

There are 30 ITMs in the latest patch release

Details
Looks like 30 ITMs snuck into the latest patch per xEdit 4.1.5h.
Some examples (can't paste all because of the character limit):
Removing: MQ301EarthTerminal03SubMenu "Access Station Logs" [TMLM:000B11CF]
Removing: [INFO:002C33C5] ('Oh, I'm sorry. Here I am. Arguing with my partner, and you're just standing right here.' in GRUP Topic Children of [DIAL:002C334A])
Removing: [DIAL:002C334A]
Removing: [INFO:0022722D] ('No, I can't handle this. Dad... Dad is....' in GRUP Topic Children of [DIAL:00227278])
Removing: [DIAL:00227278]
Removing: [INFO:00208E54] ('What kind of operation are you running here?' in GRUP Topic Children of [DIAL:00208E60])
Removing: [DIAL:00208E60]
Removing: [INFO:0021B154] ('Blend needs Aurora. And Aurora is totally illegal outside of the Astral Lounge. And really really illegal off-world.' in GRUP Topic Children of [DIAL:0021B170])
Removing: [DIAL:0021B170]
Removing: [INFO:0012C618] ('Amp is... steroid. Very strong, very powerful. Legal for the moment.' in GRUP Topic Children of [DIAL:0012C632])
Removing: [DIAL:0012C632]
Removing: [INFO:0022D907] ('I've spoken to the people at Reliant Medical Corporate, I've talked to the Freestar Rangers... I've even gone to the Freestar Collective Council.' in GRUP Topic Children of [DIAL:0022DAC4])
Removing: [DIAL:0022DAC4]
Game Version
<IP_ADDRESS>
Language
English (EN)
New Game
Yes
Reproduction Steps

Download mod.
Launch xEdit in quickclean mode.
Select community patch ESM.
Review ITMs.

Please check the FAQ before reporting this
https://www.nexusmods.com/starfield/articles/7

Why does StarfieldCommunityPatch.esm contain Identical to Previous Override/Identical to Master (ITPO/ITM) records?
These "ITMs" are usually false positives. They exist because the patch provides text-based fixes for multiple locales. The SF1Edit tool - a
t time of writing - does not distinguish these and only compares the text shown for your current locale. In short, you may see an ITM in English because this record has been fixed in German/Spanish/French/etc. You may clean these records and it will not break your own personal game, but there is really no reason to do so. If you do clean them, please ensure you don't forward these edits in a Collection/Wabbajack list as it will cause problems for users in the language the fix is intended for.

Please check the FAQ before reporting this
https://www.nexusmods.com/starfield/articles/7

Why does StarfieldCommunityPatch.esm contain Identical to Previous Override/Identical to Master (ITPO/ITM) records?
These "ITMs" are usually false positives. They exist because the patch provides text-based fixes for multiple locales. The SF1Edit tool - a
t time of writing - does not distinguish these and only compares the text shown for your current locale. In short, you may see an ITM in English because this record has been fixed in German/Spanish/French/etc. You may clean these records and it will not break your own personal game, but there is really no reason to do so. If you do clean them, please ensure you don't forward these edits in a Collection/Wabbajack list as it will cause problems for users in the language the fix is intended for.


Deploy a postgresql database for registry

Note: let's create just one deployment of postgres and multiple schemas for the different components.


Userspace only tracing supported?

I tried decoding traces with the perf recording flag exclude_kernel = 1 set, however this computes invalid instruction pointer (Pointers to not mapped memory). Is mode supposed to work in the current state with the sideband library?

This should work.  Would you please provide more detail?  E.g. the command-line you were using to record and to decode the trace, the full error message, and a few packets around the error location to give some context.

Hello Jörg,
Do you have further questions on this?  If not, I would close it or you may close it, if you want.
regards,
markus.


Switch search-api-v2 workers to use new redis instance in production

Trello: https://trello.com/c/H4PW2hNC/1399-create-new-redis-instance-for-search-api-v2-and-move-search-api-v2-to-it-peri-peri-🌶

Closing as this was the wrong approach https://github.com/alphagov/govuk-helm-charts/pull/2674


Update dependabot.yml

It was not updating the packaging Dockerfiles which are in subdirectories

My only quibble is that "Update dependabot.yml" won't be very useful in git log output.

Hmm, it looks like this is not possible as dependabot does not support recur sing into subdirectories: https://github.com/dependabot/dependabot-core/issues/2178
I'm going to hardcode the paths for all current docker images for now. But it looks like generating a dependabot.yaml is the only solution at this stage.


Fix bad math expression: empty string error (OSX)

A duplicate for declaration results in the aforementioned error,
particularly on OSX. printf will throw an error due to $i
being empty for the last index.

I'm unsure if the duplication is intentional or if it is a mere typo. Nevertheless, this resolves the described error for OSX El Capitan (10.11.5). It is considered a syntax error in bash.

Hi! Thanks for this, it was a typo indeed.
Merging.
Just to stay on record in case anyone comes here with the same issue in the future: can you just describe how the issue happened to you? Was it always present when you enabled the prompt symbol color hashing? Also, for the same reason, could you provide the zsh and terminal emulator version?

Hi @frmendes,
I'm normally on a Lubuntu, and I started using this theme about a week or so ago, with a few modifications. Namely, the $PROMPT_GEOMETRY_COLORIZE_SYMBOL was enabled. It was perfectly fine on that machine (appearance wise at least...)
When I sync'd my dotfiles with my OSX machine, it started giving the aforementioned error upon starting my terminal up. There were two errors. One is the bad math expression described above. I can't remember the other one. They both pointed to prompt_geometry_hash_color as the source of the error. And that is when I noticed the double for. With the colorize symbol disabled, the error does not show up.
I plugged the code in https://www.shellcheck.net/ just to validate.
Also, as requested:

zsh 5.0.8 (x86_64-apple-darwin15.0)
iTerm 2 Build 3.0.10

Let me know if you have any additional questions :) Happy to help. Thanks!


Cannot write on cache store occasionally during use.

I'm using the IMAP proxy via fetchmail in daemon mode, with --cache-store. During the 24 hrs of use the log reports:
Error saving state to cache store file at <cache_filename> - is the file writable?
Apparently, it succeeds in connecting just one sec after. The log shows this message every 1/1.5 hour or so.

Do you have more than one account set up? I suspect this is due to concurrent access attempts. Improving this is next on the roadmap.

I setup a single box with the proxy service and a single fetchmail process on that. Apparently no.

Even, that was not the case when not  using the cache store.

To be clear, I mean more than one email address, not necessarily multiple processes. It could also I suppose be caused by a single address but multiple simultaneous access attempts (some clients often do this to monitor multiple folders).
Regardless, I can't think of anything else that could be causing this, but will be looking at improvements to concurrent usage in the relatively near future.

Here it is a single email addresss (on Office 365) and a single client (fetchmail) with a single process run AFAIK.

The concurrent-config branch has a proposed solution for this issue. Could you take a look and see whether the issue still occurs?

In the meantime a few things changed on this side, including a new Python version. Apparently the previous issue seems disappeared. Maybe something was broken in Python 3.9?

Thanks for following up. I'll keep this open until that branch is ready to merge - in the meantime please do let me know if it happens again.

I've decided to merge this branch, but please feel free to re-open the issue if it re-occurs. Thanks for the help with this.


Fail to deploy on Streamlit

Hi Dr. Wu, first thing first, many thanks for creating this fascinating web app.
I followed your blog, created a fork of this repo and tried to deploy a copy of the app to Streamlit. I followed the steps and created a token and added it as an environment variable.
However, the deploy log still requires me to authenticate GEE, and the error Earth Engine client library not initialized. appears when I try to upload the study boundary.
Following are the screenshots of the secret token configuration and the deploy log. Is there something I did wrong when configuring the app/authenticating GEE?

Setting token as environment variable

Error when adding study boundary
https://user-images.githubusercontent.com/29334677/201335429-5c5abb14-6ddb-4cc3-b0fa-4346ffe2ae53.mp4
Deploy log of error

The EARTHENGINE_TOKEN should look like this. Use ee.Authenticate() on your computer to generate a new token if needed.

Thanks! The app works perfectly now. Originally I thought the EARTHENGINE_TOKEN only includes the refresh_token part of the JSON.

Greetings! I too am trying to follow that blog and create a working app by forking that repo, but I am getting stuck earlier at the ee.Authenticate() step. I get an error that ends with:
Exception: gcloud command not found. Please ensure that gcloud is installed.
More information: https://developers.google.com/earth-engine/guides/python_install
I eventually landed here but I'm feeling a bit lost as I don't think this is correct (I'm building environments in Anaconda): https://cloud.google.com/sdk/docs/install
Also, I tried this package but it is no longer available from conda-forge according to my install attempt:
https://anaconda.org/conda-forge/google-cloud-sdk
Do you have a quick hint as to where I might look to fix the authentication issue with gcloud?
Thanks much!


NavMeshSurface - support for tree colliders

After sources are collected by calling NavMeshBuilder.CollectSources, go through and add tree colliders for each TerrainCollider encountered.
Supported tree colliders are: Sphere, Capsule, Box, Mesh.
Tree colliders are added with hard-coded area Non-walkable.
NavMeshModifier on trees is not supported (yet).
NavMeshModifierVolume should work as usual, even on trees.
In future I'd like to support NavMeshModifier and use default area instead of hard-coded Non-walkable
Resolves #30

Hello i have 4 errors with your code:  error CS1061: 'Collider' does not contain a definition for 'TryCast' and no accessible extension method 'TryCast' accepting a first argument of type 'Collider' could be found (are you missing a using directive or an assembly reference?)  and one with object.trycast.
How can i fix it? Thanx (sorry for english)

There was missing TryCast extension method, fixed

@turadr Could we please have a review on this? Would be so awesome to have this available in the NavMeshComponents. Thank you! :))

@turadr Could we please have a review on this? Would be so awesome to have this available in the NavMeshComponents. Thank you! :))

@hendriku We appreciate the work put into this PR but unfortunately we cannot take contributions to this repository right now. If that will become possible in the future we will then address this PR.

@hendriku We appreciate the work put into this PR but unfortunately we cannot take contributions to this repository right now. If that will become possible in the future we will then address this PR.


Hi,Is the yajl component being maintained?

Hi,
Is the yajl component being maintained? It seems like it hasn't been updated for a long time.
^_^

Hi,
can you answer my question, is the yajl component being maintained?and is there a new version to be updated?tks


How do you load model and cat_vocab_dict after training?

Thanks for your contributions! I have trained my model and I would like to know if there is anyway to load the trained from my saved_path?

Whenever I tried to load the model from my saved_model path, I will always receive this message:
ValueError: Unable to restore custom object of type _tf_keras_metric currently. Please make sure that the layer implements get_configand from_configwhen saving. In addition, please use thecustom_objectsarg when callingload_model().

very simple.
Your model is saved under the project_name folder in .pb (protobuf) format which is the new Tensorflow saved model format.
You can load the model back if you give the project_name folder. It will automatically find the saved model and load associated artifacts which will help in predictions. The keras_model_type below is the same as the type of model that you built above (earlier). That way, you can have multiple model types under different names. The model and cat_vocab_dict are not needed and you can set them as empty strings.
Here is the call:
y_preds = deep.predict(model="", project_name, test_dataset=test,                                  keras_model_type=keras_model_type, cat_vocab_dict="")


doc(Algebra/Group/Defs): add note on names SMul, VAdd; typo fix

Edited module docstring:

Fixed recent accidental damage
Added a note about the motivation for the names SMul and VAdd
Zulip discussion

My concern is that you are using "Euclidean space" in the way Euclid used it, which is apparently not the way it's used in mathlib. It looks like you should be talking about an additive torsor for a vector space,  rather than Euclidean space (which as Eric pointed out does have a zero)

I changed the wording after discussion on Zulip, so just for the sake of honesty I (re-)requested reviews from @YaelDillies, @grunweg and @kbuzzard. Sorry for the trouble!

I think removal of the ambiguous term makes it better. Now I think it reads a bit funny because you're not saying what the vector space is acting on but I don't want to cause any more fuss.
I know you've said that you don't want to use the word "torsor" for some reason (despite the fact that there are plenty of technical words in other docstrings), but the other side of this coin would be: who do you have in mind who wants to use VAdd but doesn't know what a torsor is? But honestly I don't really mind what this docstring says so if a maintainer is happy to merge this as it stands then that's fine.

I didn't think it was really about torsors, any more than SMul is. Maybe I'm misunderstanding.
In any case, "See the file Module.AddTorsor for a motivating example for the name VAdd (vector addition)" seems better all round, so I have made that change.

Thanks for bearing with me!

maintainer merge


Remove credentials from logs

Stuff like this:
[2023-11-15T14:46:15.176Z INFO  trustification_storage] Update config to StorageConfig {
        bucket: Some(
            "vexination",
        ),
        region: Some(
            Custom {
                region: "eu-central-1",
                endpoint: "http://localhost:9000",
            },
        ),
        endpoint: Some(
            "http://localhost:9000",
        ),
        access_key: Some(
            "admin",
        ),
        secret_key: Some(
            "password",
        ),
        validator: None,
    }

I think we should be good here for now. If we find additional ones, we need to fix them too.


Fix Test-Connection due to .NET 8 changes

PR Summary
.NET 8 made a change on how a custom buffer works on Linux, so we have to change the code to use a default empty buffer so that .NET uses its own default buffer.  A custom buffer size requires sudo on Linux.
PR Context
Fix https://github.com/PowerShell/PowerShell/issues/17018
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[ ] Not Applicable
OR
[x] Documentation needed

[x] Issue filed: https://github.com/MicrosoftDocs/PowerShell-Docs/issues/10458

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

If this cmdlet is running with elevated permissions on Linux, why not just check for those permissions for certain parameters and report an error if needed?
(As far as I remember there is a way to set up ping for a regular user too. I don't think it's necessary to complicate our installer, but all this could be simply reflected in the documentation. Or we could create something like Enable-PSRemoting.)

What pwsh version has this fix for Test-Connection ?
I tried 7.4.0-preview.6 but still requires sudo on Linux


fix sprintf and snprintf format string

There are some cases of printing unsigned integer with %d conversion
specificator and vice versa (signed integer with %u specificator).

the first change is in the example file of hiredis. Do you mind to submit it there?

@badboy thanks for pointing this out. I submitted https://github.com/redis/hiredis/pull/369 in hiredis upstream.

Thank you @jamel, merged.


[Color Picker] custom format implementation

Summary of the Pull Request
Extend the Color Picker module with the possibility of using custom color formats.

PR Checklist

[X] Closes: #8305
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments
The main changes are on the settings page, where there is a new button for creating new formats.
The created formats appear in the list of the formats among the predefined ones.
There is a possibility to edit and delete the custom formats.

Validation Steps Performed
Tested locally

Why do we need the = and ; in the format definition formula? Can't we go without it? Like only new(R=255,G=255,B=255).
What character is used a decimal seperator? Can we split the formula parts in the bracket with ; to allow , and . as decimal separator?

Hi @htcfreek It was just an example. You can define the formula as you like.
For example       Color ( R =%Re, G = %Gr, B = %Bl)
or (just as an example)         %Re ;  %Gr ;  %Bl   .......   %Na
You can have any characters in the format string, the valid %XX parameters will be replaced by the corresponding values.

But for what do = and especially ; (at the end) stands for?
I think we have a complex and a simple (color(r=3,g=2,b=5) example.

I think the new...; is implying a C# code -like thing

Hm, one idea/question home to mind: can those variables be used in the name to?

Sure @Jay-o-Way. But do we need to borrow a user with such things if it's not necessary. I think our main goal here should be to keep it short and simple for the user.
I think the best way for our users is that they only type the name, the formula part within the brackets and maybe a "short" name. And then the brackets and everything else we need is added automatically.

Oh heck yeah!!!! 🔥🔥🔥🔥🔥🔥

@donlaci
Are the custom formats only available in editor or for clipboard too? Can we synchronize clipboard and editor formats?

@donlaci

I think we should move the add custom button to the list view header action content.
In the settings group  befor the editor formats is a clipboard format setting. And there you can choose the format for direct copy (without using the editor). That's wat I mean.

@htcfreek
Thanks for the clarification, now I got it. We well consider your suggestions.

Awesome work @donlaci 🚀. I have a few questions looking at this screenshot:

Does this imply that the current formats (that come with PT) cannot be removed? Wouldn't it make more sense to remove them all and ship some of these as 'presets'? This feels a bit weird now. If not, I'd suggest splitting the format listviews.
Wouldn't make more sense if the list shows the actual preview of the format (i.e. 'what will I see in the ColorPicker UI)? Vs. the format itself?

There are a couple of minor styling tweaks we could do, but we can leave that to after merging this in :).

Yes, currently the old hard coded formats are kept fixed, removal is not implemented - as it was before. We could easily add the edit, remove buttons to them as well and allow the users to modify, delete them. I'm not sure whether it adds value to the app or it is better to have allways the preset formats. They can be hidden and it looks professional to have so many presets - but it is just my oppinion. :-)

Yeah.. I think adding those buttons (and from a UX perspective, make sure formats that are custom and ship with PT are 'equal'). I can imagine users going to start ask why this one format can be deleted, and the other cannot :).

Yes, we could have the preview of the format if it makes more sense. Just let me know and I'll update the code.

I'd say so? I think if we replicate the format that you'd see in the Color Picker Editor makes most sense?

FYI

I've merged main here due to the UI refactor that just went into main. Here's hope I didn't break anything 😅

Can we please get updated screenshot from the settings page.

Hi @htcfreek,
Your proposal, to add the custom color formats to the list of formats for direct copy, has been implemented. Here you have some screenshots.

Until now, the SettingsCard with "" does not actually have any setting (or controls) inside of it. I think it should be converted to text, below the group header.

I agree with @htcfreek 's point earlier - shouldn't the 'Add new format' button be in the top card, just like we do with Image Resizer?
That would look something like this:

@niels9001 @htcfreek
The current state is:

@niels9001 @htcfreek
The current state is:

How can we make clear that the custom formats not exclusive to the editor? Rename/reorder the settings?

How can we make clear that the custom formats not exclusive to the editor? Rename/reorder the settings?

Good point. At this point, I would say "drop the current header"

@donlaci
Please update the description of the formats list too. Something like "Configure the available formats". Currently it has a text reference to the editor.

Hi,
The latest state is shown on the pictures
What changed is

the description of the first panel
instead of the rules an exxample is shown as description for the available formats (also for the default format selection)
the presets (RGB, HEX, ...) are converted into 'custom' formats. ('Custom' is a new 'normal'). Every format is editable, deletable.

@donlaci
Can you please rename the group header from "Editor" to "Color formats".

I thought the default formats are protected against deletion.
Can we convert setting then from v1 to v2 and does it keep the old configuration?

I thought the default formats are protected against deletion.
Can we convert setting then from v1 to v2 and does it keep the old configuration?
Are the last screenshots up to date?

Hi @htcfreek
Now all formats are treated in the same way, all can be deleted.
Yes, settings conversion from v1 to v2 is implemented.
The screenshots are up to date except the help section. The list of available parameters has been extended (due different calculation of the 'same' parameters in different models, like saturation in HSI, HSB and HSL) The parameter list has been extended.

@donlaci
The settings page still has a left over: The settings group header "Editor".

Hi @htcfreek
I better put here new screenshots. The label "editor" has been replaced. I hope everything is fine.


tsocks: fix ./configure error

The tsocks package no longer compiles due to a bug in ./configure:
configure:774:1: error: return type defaults to ‘int’ [-Wimplicit-int]
  774 | main(){return(0);}

Replacing main() by int main() fixes the bug and allows the package to compile.

Things done

Built on platform(s)

[X] 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/)
25.05 Release Notes (or backporting 24.11 and 25.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.

Thank you, @aucub, for reviewing and proposing changes!
I've opened a new pull request that changes the target branch to master: https://github.com/NixOS/nixpkgs/pull/369474


edit CI toolchains

Limit Travis to 5 jobs (can avoid both Debug & Release Xcode+ASAN) and avoid long 3x {armv7, armv7a, arm64} iOS builds (arm64 is ok for day-to-day):

add ios-nocodesign-10-1-arm64-dep-9-0-device-libcxx-hid-sections-lto
remove Debug or Release variant of address sanitizer osx-10-12-sanitize-address-hid-sections

os-nocodesign-10-1-arm64-dep-9-0-device-libcxx-hid-sections-lto -> PR # https://github.com/ruslo/polly/pull/147

PR # ruslo/polly#147

Merged

Thanks.  Done.  #464.  In the future, CMake LTO customization that supports optional LTO for the various test, console and sample apps might help (w/ cmake LTO customization in the future).  LTO is most critical for the deployed library assets.


It didn't start from pwd?

I ran :pwd in a buffer and got ~/.config, while when I execute :lua require("FTerm").toggle() the path is at ~.
Update: I have tested other folders under ~, this bug only happens on ~/.config.

Never mind, it's my fault. I just realized that it's a bug from my plugin. I should detect the filetype with vim.bo.filetype, not vim.opt.filetype.


Error in file_get_contents

I have performed a test according to the code below
`<?php
require_once "MoodleRest.php";
$parameters = array('userlist' => array(array('userid' => 13, 'courseid' => 2), array('userid' => 13, 'courseid' => 2)));
$json = (new MoodleRest())->setServerAddress("http://dev.moodle.local/webservice/rest/server.php")->setToken('f0bb33743851adc4c2475240787c07b0')->setReturnFormat(MoodleRest::RETURN_JSON)->request('core_user_get_course_user_profiles', $parameters);
echo $json;`
And I got the following error

Can you help me?

I recommend a tool to test the web services called Postman, with it you can know how to test the functions and determine if you are having any error in the handling of the functions, a problem on the server or a configuration problem of the web service

Personally, I developed this file to handle the functions that I have enabled in my web service to call them from any php file where I am going to use them.
`<?php
require_once "MoodleRest.php";
class Moodle
{
private $Moodle;
private $token;
function __construct()
{
$this->Moodle = new  MoodleRest();
$this->token = $this->getToken();
}
//get web service token
function getToken()
{
	$url = 'https://url of moodle site/login/token.php?';
	$data = array(
			'username' 		=> 'webservice_user_name',
			'service'	 	=> 'webservicename',
			'password'	 	=> 'webservice_user_password'
		);
	$options = array(
	  'http' => array(
	    'header'  => "Content-type: application/x-www-form-urlencoded\r\n",
	    'method'  => 'POST',
	    'content' => http_build_query($data),
	  ),
	);
	$context  = stream_context_create($options);
	$result = file_get_contents($url, false, $context);
	$array = json_decode($result, true);
	return $array['token'];
}

//conenction to the web service using the token
function connection()
{
	$ip = 'url_of_moodle_site';
	$this->Moodle->setServerAddress("https://$ip/webservice/rest/server.php");
	$this->Moodle->setToken($this->token);
	return $this->Moodle->setReturnFormat(MoodleRest::RETURN_ARRAY);
}

/* any enabled function on web service for example core_user_get_users */
function get_course_profiles($userid,$courseid)
{
	if($this->connection()):
		$info = array
					(
						'userid'			=> $userid,
						'courseid' 		=> $courseid
					);
		$params = array( 'userlist' => array($info));
		$ARRAY = $this->Moodle->request('core_user_get_course_user_profiles', $params);
		return "result2:\n" . print_r($ARRAY, true);
	else:
		return 0; //or wherever you want
	endif;
}
function GetUserId($username)
{
	if($this->connection()):
		$username = array
					(
						'key' 	=> 'username' ,
						'value' => $username
					);
		$params = array( 'criteria' => array($username));
		$ARRAY = $this->Moodle->request('core_user_get_users', $params);
		return $ARRAY['users'][0]['id'];
		die();
	else:
		echo "Error de conexion";
	endif;
}
function UserPasswordReset($username,$password)
{
	if($this->connection()):
		$info = array
					(
						'id'			=> $this->GetUserId($username),
						'password' 		=> $password,
						'preferences' 	=> array($preferences = array
											(
												'type' 	=>'auth_forcepasswordchange',
												'value' => 0
											))
					);
		$params = array( 'users' => array($info));
		$ARRAY = $this->Moodle->request('core_user_update_users', $params);
		//print_r( $this->Moodle->getPrintOnRequest());
		return 1;
	else:
		return 0;
	endif;
}

/* end of function management */

}
?>`

there is an example of your code https://github.com/llagerlof/MoodleRest/wiki/MoodleRest-examples

@brunocs86 Hi Bruno.
I tested your code here, making the necessary changes to my environment and it worked.
Looks like you have a name resolution problem. This can be caused for some reasons.
Please, do this tests:
Try to wget the server.
$ wget http://dev.moodle.local/webservice/rest/server.php -O wget.txt
Open the wget.txt. Should be something like this:
<?xml version="1.0" encoding="UTF-8" ?>
<EXCEPTION class="moodle_exception">
<ERRORCODE>invalidtoken</ERRORCODE>
<MESSAGE>Invalid token - token not found</MESSAGE>
</EXCEPTION>

a. If wget cannot find the host wget.htm will be empty and you will receive the error wget: unable to resolve host address. In this case the problem it's in your network.
b. If wget find your server, it could be doing some redirection. In this case try using the IP of your Moodle server instead the host name. In setServerAddress() replace the host dev.moodle.local by the IP of your moodle and make the request.
c. If everything fails, just to cover most scenarios, try to wget using the IP instead the  hostname.
Please, report the results here so we can find a solution for your case.

there is an example of your code https://github.com/llagerlof/MoodleRest/wiki/MoodleRest-examples

thanks for your contribution, it was very enlightening

@brunocs86 Hi Bruno.
I tested your code here, making the necessary changes to my environment and it worked.
Looks like you have a name resolution problem. This can be caused for some reasons.
Please, do this tests:
Try to wget the server.
$ wget http://dev.moodle.local/webservice/rest/server.php -O wget.txt
Open the wget.txt. Should be something like this:
<?xml version="1.0" encoding="UTF-8" ?>
<EXCEPTION class="moodle_exception">
<ERRORCODE>invalidtoken</ERRORCODE>
<MESSAGE>Invalid token - token not found</MESSAGE>
</EXCEPTION>

a. If wget cannot find the host, wget.txt will be empty and you will receive the error wget: unable to resolve host address. In this case the problem it's in your network.
b. If wget find your server, it could be doing some redirection. In this case try using the IP of your Moodle server instead the host name. In setServerAddress() replace the host dev.moodle.local by the IP of your moodle and make the request.
c. If everything fails, just to cover most scenarios, try to wget using the IP instead the hostname.
Please, report the results here so we can find a solution for your case.

I will try to carry out your guidelines, thank you very much for helping me.

Due to inactivity, and because I am assuming that the author of this issue must have already solved it, I am closing this thread.


Invalid Account Verification key

I get an invalid key response when a user tries to verify ana ccount. Does the Verification key has an expiration period?

Yes, which is configurable.

Ok. Found that as the "VerificationKeyLifetime" settings. Thanks


Reproduction Bias

First of all, thx to you guys all for the stimulating insights! I have a question: When I try to follow the coding logic and reproduce the results with Megatron-LM, I encounter some biases in the final results. Specifically, the speedup of Eagle1 generally aligns with what is mentioned in the paper, but the performance of Eagle2 is unsatisfactory, as it can’t reach a 5.x speedup(only about 0.5 higher than Eagle1). Are there any tricks or details in the implementation that you guys could share(like some other inference optimizations)?

We haven't tried Megatron-LM before. But according to our experience, the speedup ratio depends on several factors, such as target model types, sizes, tasks (coding problem is faster than others), datasets, etc.

Sry, btw, when are the evaluation results for Qwen2 and LLaMA3 expected to be released?

LLaMA3's evaluation results have been released in the EAGLE2 paper. Qwen2's speedup ratio is similar to that of LLaMA3 (we have no plan to show it in the paper as Qwen2 is trained after our paper is written).

May I please ask for the specific figures or metrics for Qwen2?

Thank you, Prof. Zhang, for patiently answering my questions. 🙏


cd react-native && npm install   , then<EMAIL_ADDRESS>not compatibale with my OS .Anybody know how to fix this pro

npm ERR! Windows_NT 6.3.9600
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "install"
npm ERR! node v4.1.1
npm ERR! npm  v2.14.4
npm ERR! code EBADPLATFORM
npm ERR! notsup Unsupported
npm ERR! notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>npm ERR! notsup Valid OS:    darwin
npm ERR! notsup Valid Arch:  any
npm ERR! notsup Actual OS:   win32
npm ERR! notsup Actual Arch: x64
npm ERR! Please include the following file with any support request:
npm ERR!     D:\workspace\person-workspace\offical-demo1\react-native\npm-debug.log

fsevents should be optional so you shouldn't worry about it.

@gamedilong are you still encountering this issue? If not, please consider closing it - we're trying to keep the issue tracker easier to maintain.

try to install with this command: "npm install --no-shrinkwrap"
you can refer this issue: https://github.com/facebook/react-native/issues/4138


An Istio.io test for circuit breaker (#18287)

Istio.io test for circuit breaker - runs the example mentioned in circuit breaking - Istio.io page

Hi @su225. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

@su225: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
9750115b52220fb3c8ebf150ab0b7638db5ebaaf
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@brian-avery - anything else that needs to be fixed in this PR?

@su225 I'm afraid istio.io tests have been moved to istio/istio.io: https://github.com/istio/istio.io/tree/master/tests

Closing this PR as it's been superseded.


[AIRFLOW-2567] Extract result from the kubernetes pod as Xcom

Make sure you have checked all steps below.
JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title.

https://issues.apache.org/jira/browse/AIRFLOW-2567

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

In order to simplify the construction of DAG, it is useful to be able to extract the execution result of the container as XCom.
Users will be able to do this by writing the result as the result.json in /airflow/xcom directory mounted on the container automatically.
This function is realized by Pod's sidecar container.
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
Add KubernetesPodOperatorTest. test_xcom_push

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[x] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Codecov Report

Merging #3466 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3466   +/-   ##
=======================================
  Coverage   76.89%   76.89%
=======================================
  Files         203      203
  Lines       15120    15120
=======================================
  Hits        11627    11627
  Misses       3493     3493

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 817296a...63718ce. Read the comment docs.

@dimberman PTAL

@mrkm4ntr This looks great :). Just have some style/code-cleanup comments but overall LGTM.

Also thanks for the ping @kaxil

@dimberman Thanks for your review. PTAL.

@kaxil Resolved conflict.


Release Google.Cloud.Eventarc.Publishing.V1 version 2.0.0-beta05

Changes in this release:
New features

Change netstandard2.1 target to netstandard2.0 (commit 82bea85)

Created release for Google.Cloud.Eventarc.Publishing.V1-2.0.0-beta05


docs: deploying/publishing section of readme updated for semantic-rel…

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

v1.8.0
GitHub release

Your semantic-release bot :package::rocket:

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

v1.8.0
GitHub release

Your semantic-release bot :package::rocket:


run-errors

When I am running “bash scripts/zero_shot.sh mpvr gpt clip_b32 eurosat imagenet_r” , the following error appears. How can I download or train the missing one?
FileNotFoundError: [Errno 2] No such file or directory: './descriptions/mixtral/DescribableTextures.json'

Hi,
Thanks for your interest. Please follow the readme to get the data and run the experiments. All details are present there.
Best,


Bug: /messages call fails with error from etherscan (endpoint has changed)

Describe the bug
When I call the messages endpoint, I receive an error:
Endpoint:
http://<IP_ADDRESS>:8080/api/messages?protocol=ethereum&network=mainnet&address=0x92d8f10248c6a3953cc3692a894655ad05d61efb`

Error:
{"code":500,"message":"Get https://etherscan.io/api?action=txlist\u0026address=0x92D8f10248C6a3953CC3692A894655ad05D61Efb\u0026endblock=99999999\u0026module=account\u0026sort=desc\u0026startblock=0: auto redirect is disabled"}

Then, when calling the etherscan url (replacing \u0026 with &):
https://etherscan.io/api?action=txlist&address=0x92D8f10248C6a3953CC3692A894655ad05D61Efb&endblock=99999999&module=account&sort=desc&startblock=0

returns error:
{"status":"0","message":"NOTOK","result":"Invalid API URL endpoint, use api.etherscan.io"}

To Reproduce
Steps to reproduce the behavior:

Configure Mailchain with at least one Ethereum address
Go to 'https://inbox.mailchain.xyz/#/'
Inspect the developer console
See failed message endpoint call
See error

What version of Mailchain are you using ?
$ mailchain version
Version: v0.0.60
Expected behavior
It should fetch messages
Additional context
Add any other context about the problem here.

closed by #639


Relay moco message level data to Gambit Converstation

Notes per discussion with @aaronschachter:

Relay Twillio -> MoCo -> Blink -> Gambit Convers
Talk to @rapala61 when Gambit Convers staging endpoint is ready
Follow the same retry protocol as we have with Gambit Campaigns
Gambit Convers is using different auth header

cc @justkika

Point 2 is referring to import-message in this case?

Not sure. @aaronschachter do you know which one?

@aaronschachter said it should be recieve-messae

Note: traffic volume Blink is receiving from MoCo is relatively large, about 120 msg/s during broadcasts:

In order to send it to Gambit Convers safely, we're going to need a rate limiter.

We want this set up for staging only right now: POST https://gambit-conversations-staging.herokuapp.com/api/v1/receive-message. You'll need to include basic auth and can find the username and password in the Heroku settings for Conversations staging.
Nothing's going to import-message -- refs https://github.com/DoSomething/blink/issues/102#issuecomment-321088158


[Request] Angular Universal/Server Side Rendering Support

In the past month, Angular Universal has become increasingly important because of SEO. I would love to be able to use this package again, but I can't as it stands currently.
Even if you were to make a mock component that could be rendered on the server side and then replaced in the browser, one could import the mock in the server module and the actual module in their browser module. I have been researching Universal very closely for the past week so let me know if you have questions. Again, I think it will be important to this project!

A possible duplicate of https://github.com/SebastianM/angular-google-maps/issues/1052 (definitelly related, though).

Yeah, they are closely related (I didn't see that one, oops). The repo he forked is quite old though, because the error I am getting is on import, not export. Either way, very closely related.

I've done some more digging, and I am 99% sure that if this module was compiled to es5 it would work for universal.

@dockleryxk oddly enough I didn't realize this was an issue but made a PR for it already in #1114

Until this is resolved, I have it working with a workaround by dynamically loading a component that contains my agm-map based on if it's the browser or server: https://angular.io/guide/dynamic-component-loader

R.I.P.


Love / Copy notification WILL NOT GO AWAY!

As title - what's going on here?  Please make it go away.

Yeah this one is quite annoying. We should have the option to disable it.

Please choose to disable "Active App Notification" in power settings aka "Options"

"warning may skip scrobbles" seeing as that's the whole point of the app, it seems a bit odd to have scrobbles linked to this option

I assumed it was obvious, as I would not add notifications if they weren't necessary.
If disabled, some battery optimized devices may skip scrobbles.
If you permitted the app not to be battery optimized it should work.

When I disable this the notification still is there until the song changes. Thanks for the heads up though :+1:

When I disable this the notification still is there until the song changes. Thanks for the heads up though :+1:

This notification is persistent for me even when there music app I was using had been paused and exited (by which I mean the music app notification has been swiped). Is this notification supposed to be persistent even after closing the associated music app? So the notification will be permanent until I restart the phone?
I'm using GoneMad music player and simple scrobbler version 1.7.0 on Android 9.

@sn0skier
Please choose to disable "Active App Notification" in power settings aka "Options". After changing a song it will disable the notification.
Have you attempted this friend?

@sn0skier
Please choose to disable "Active App Notification" in power settings aka "Options". After changing a song it will disable the notification.
Have you attempted this friend?

Yes, I currently have this setting disabled, but I would like to enable it, but I only want to see the notification when a song is playing, not all the time even when no song is playing. If this isn't possible I guess that's OK because scrobbling seems to working even with this disabled. Thank you for your help.

This following apk should function well.
Not 100% certain it will fix the problem, but it's worth a try.
app-debug.zip

Remember to disable Active App Notification

This following apk should function well.
Not 100% certain it will fix the problem, but it's worth a try.
app-debug.zip

Sorry, but I don't have time to debug this right now and I don't know when it will. As I stated above, scrobbling is working correctly for me now even with the notification disabled, so fixing this isn't a priority for me right now. Thank you for your help and I'm sorry I won't be able to help debug this. I'm sure I'm not the only one that wanted this fixed though, so hopefully someone else will be able to confirm the fix.

No matter what I do, I can't remove this persistent notification. I've tried on my LG G6 and my Pixel 4XL. I'm getting to the point that I just want to give up on this once great and simple app.


Stack overflow in splitPath().

Brief Issue Summary
I noticed the following messages in the "Output/Log (Window)" window:
[2018-09-18 09:59:29.060] [renderer1] [error] Maximum call stack size exceeded: RangeError: Maximum call stack size exceeded
	at splitPath (C:\Users\igork\.vscode\extensions\vector-of-bool.cmake-tools-1.1.1\out\src\util.js:64:19)
	at splitPath (C:\Users\igork\.vscode\extensions\vector-of-bool.cmake-tools-1.1.1\out\src\util.js:71:21)
	at splitPath (C:\Users\igork\.vscode\extensions\vector-of-bool.cmake-tools-1.1.1\out\src\util.js:71:21)
	at splitPath (C:\Users\igork\.vscode\extensions\vector-of-bool.cmake-tools-1.1.1\out\src\util.js:71:21)
...

At the same time, the window "CMAKE: PROJECT OUTLINE" is empty.
Platform and Versions

Operating System: Windows 10
CMake Version: 3.9.2
VSCode Version: 1.27.2
CMake Tools Extension Version: 1.1.1
Compiler/Toolchain: VisualStudio 14.0 - amd64

I debugged it a bit and found that the issue appears when a full Windows path is passed to the function.
For example, it splits 'C:\some\dir' up to 'C:\', and at that point path.dirname('C:\') returns just 'C:\' which leads to infinity recursion.

Thanks for the diagnosis! I'll get this one.

Got a simple fix. Will be in 1.1.2


When I follow the step "git checkout --track -b deconv-deep-vis-toolbox yosinski/deconv-deep-vis-toolbox",I get back a error.please help solve it

when I follow the step "git checkout --track -b deconv-deep-vis-toolbox yosinski/deconv-deep-vis-toolbox".I get a error message error: Your local changes to the following files would be overwritten by checkout: 	Makefile Please, commit your changes or stash them before you can switch branches. Aborting
I don't konw how to deal with it.I know little about git .the caffe I complise is the lastest.

You can use "git checkout --track -f -b deconv-deep-vis-toolbox yosinski/deconv-deep-vis-toolbox" force to do the command. git checkout

thank you very much.It worked! @PieBoo


Fixed widbatpc for Bangle 2

Thanks! It'd actually be really nice to purge all the 16 bit color constants from the apps - they're just totally unreadable.


 As we have upgraded to angular 1.6.1, all of them got broken. It seems like the module registration for module "httpMock" fails as the test cases start up.
Bellow is a stacktrace:
'''
Failed: javascript error: [$injector:nomod] Module 'httpMock' is not available! You either misspelled the module name or forgot to load it. If registering a module ensure that you specify the dependencies as the second argument.
http://errors.angularjs.org/1.6.1/$injector/nomod?p0=httpMock
JavaScript stack:
Error: [$injector:nomod] Module 'httpMock' is not available! You either misspelled the module name or forgot to load it. If registering a module ensure that you specify the dependencies as the second argument.
http://errors.angularjs.org/1.6.1/$injector/nomod?p0=httpMock
at http://localhost:8080/forhandler/_public/gjensidigePluss/js/web.min.js?_ts=159f4e070bf:16:8
at http://localhost:8080/forhandler/_public/gjensidigePluss/js/web.min.js?_ts=159f4e070bf:16:17264
at ensure (http://localhost:8080/forhandler/_public/gjensidigePluss/js/web.min.js?_ts=159f4e070bf:16:16243)
at Object.module (http://localhost:8080/forhandler/_public/gjensidigePluss/js/web.min.js?_ts=159f4e070bf:16:16756)
at eval (eval at executeAsyncScript (:438:5), :3:26)
at eval (eval at executeAsyncScript (:438:5), :6:5)
at eval (eval at executeAsyncScript (:438:5), :6:31)
at executeAsyncScript (:438:26)
at :454:29
at callFunction (:346:33)
(Session info: chrome=55.0.2883.87)
(Driver info: chromedriver=2.26.436362 (5476ec6bf7ccbada1734a0cdec7d570bb042aa30),platform=Windows NT 6.1.7601 SP1 x86_64)
'''
We are using protractor v. 4.0.9.

Hi @atomcat1978 . I've ran all unit tests and examples using Angular 1.6.1 and protractor 4.0.14 with no major issues. I just had to adjust something for the unit tests to run with no issues. Are you still experiencing this problem?

I've pushed to master the updates to the unit tests and examples to use Angular 1.6.1

@atecarlos I am facing the same problem
My Angular version is 1.5.11
Protractor Version: 5.0

@sri85 did you get the latest version?


Upgrade action versions in the main workflow

The ones used currently are over 2 years old, some are deprecated.
Ref: https://github.com/jazzband/pip-tools/pull/1650#issuecomment-1266202584

Looks good, I made https://github.com/webknjaz/pip-tools/pull/1 to update this and upgrade the other actions too.

@hugovk done!