Issue after Huawei's server update

Hi,
The server of huaweis was updated yerterday and move to a uni...eu5 one. I have also the 0.0.20 version but i'm getting this error:
`Traceback (most recent call last):
File "/home/pi/.local/lib/python3.9/site-packages/requests/models.py", line 971, in json
return complexjson.loads(self.text, **kwargs)
File "/usr/lib/python3/dist-packages/simplejson/init.py", line 525, in loads
return _default_decoder.decode(s)
File "/usr/lib/python3/dist-packages/simplejson/decoder.py", line 370, in decode
obj, end = self.raw_decode(s)
File "/usr/lib/python3/dist-packages/simplejson/decoder.py", line 400, in raw_decode
return self.scan_once(s, idx=_w(s, idx).end())
simplejson.errors.JSONDecodeError: Expecting value: line 6 column 1 (char 5)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/etc/openhab/scripts/huawei_web.py", line 13, in
client = FusionSolarClient("XXXXXX", "XXXXXX", huawei_subdomain="uni003eu5", captcha_model_path="/etc/openhab/scripts/captcha_huawei.onnx")
File "/home/pi/.local/lib/python3.9/site-packages/fusion_solar_py/client.py", line 232, in init
self._configure_session()
File "/home/pi/.local/lib/python3.9/site-packages/fusion_solar_py/client.py", line 385, in _configure_session
self._session.headers["roarand"] = r.json()[
File "/home/pi/.local/lib/python3.9/site-packages/requests/models.py", line 975, in json
raise RequestsJSONDecodeError(e.msg, e.doc, e.pos)
requests.exceptions.JSONDecodeError: Expecting value: line 6 column 1 (char 5)`
Can some one help me?
Thank you very much,
Kind regards,

Hi @ATNALDC
Thanks a lot for reporting this. Apparently this (known) issue triggers other exceptions depending on how you use the package.
Should be fixed now in version v0.0.21 (ad54570813c4663dccc27c29bee9768506a6f201). Please let me know it if isn't!

Hi @jgriss
Now it seems to log in but i get the following error related to get_power_status
`Traceback (most recent call last):
File "/home/pi/.local/lib/python3.9/site-packages/requests/models.py", line 971, in json
return complexjson.loads(self.text, **kwargs)
File "/usr/lib/python3/dist-packages/simplejson/init.py", line 525, in loads
return _default_decoder.decode(s)
File "/usr/lib/python3/dist-packages/simplejson/decoder.py", line 370, in decode
obj, end = self.raw_decode(s)
File "/usr/lib/python3/dist-packages/simplejson/decoder.py", line 400, in raw_decode
return self.scan_once(s, idx=_w(s, idx).end())
simplejson.errors.JSONDecodeError: Expecting value: line 6 column 1 (char 5)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/etc/openhab/scripts/huawei_web.py", line 22, in
stats = client.get_power_status()
File "/home/pi/.local/lib/python3.9/site-packages/fusion_solar_py/client.py", line 144, in wrapper
result = func(self, *args, **kwargs)
File "/home/pi/.local/lib/python3.9/site-packages/fusion_solar_py/client.py", line 408, in get_power_status
power_obj = r.json()
File "/home/pi/.local/lib/python3.9/site-packages/requests/models.py", line 975, in json
raise RequestsJSONDecodeError(e.msg, e.doc, e.pos)
requests.exceptions.JSONDecodeError: Expecting value: line 6 column 1 (char 5)
`
Thank you very much,
Kind regards

Hi again,
After some checks I have seen that this issue only occurs when ussing the session in FusionSolarClient call:
`session2 = requests.Session()
with open('/etc/openhab/scripts/session_huawei.plk', 'rb') as f:
session2 = pickle.load(f)
client = FusionSolarClient("XXXX", "XXXXX", huawei_subdomain="uni003eu5", captcha_model_path="/etc/openhab/scripts/captcha_huawei.onnx", session = session2)
get the stats
stats = client.get_power_status()
with open('/etc/openhab/scripts/session_huawei.plk', 'wb') as f:
pickle.dump(session2, f)
log out - just in case
client.log_out()`
If I not use the session option it works.
Other thing I have seen is that "coal", "co2", "coalyear" and "co2year" has been deprecated?
Thank you very much,
Kind regards,

Hi @ATNALDC
Unfortunately, my account was not yet moved to the new unixxx API. Therefore, I cannot debug the behavior.
In a quick test with my credentials, everything still worked.

Hi @jgriss
After some research I found, that ussing the code to reuse session produce an error on the login if it's the first time you use it and, the file with session information is empty. In this case the file to store session can't be set because _configure_session() it's only call if session doesn't exists. and during the whole process you use a "fake" session which doesn't let you to log in.
Maybe, as suggestion could be interesting to have a method to be able to read the session information, so you can write it into the session file the first time you use the code, and after that you can then set the code as specified on the readme.
Thank you very much!
kind regards


[BUG] debug_traceBlockByNumber throwing "insufficient funds" error for block 81772242 on pacific-1

Seid version
name: sei
server_name: <appd>
version: v5.5.4
commit: 07b5ad5b129bc1b3f47925cf1ec71b44287ff0bb
build_tags: netgo,
go: go version go1.21.7 linux/amd64
build_deps:
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>
Chain ID
pacific-1
Describe the bug
While calling the debug_traceBlockByNumber method for block 81772242, the nodes throw an error:
{
  "jsonrpc": "2.0",
  "id": 1,
  "error": {
    "code": -32000,
    "message": "tracing failed: insufficient funds for gas * price + value: address 0x7Ac6d25FD5E437cB7c57Aee77aC2d0A6Cb85936C have<PHONE_NUMBER>279949 want<PHONE_NUMBER>000000"
  }
}

This is the case case with multiple nodes.
Request Body
{
  "jsonrpc": "2.0",
  "method": "debug_traceBlockByNumber",
  "params": [
    "0x4dfbedb",
    {
      "timeout": "10m",
      "tracer": "callTracer",
      "tracerConfig": {
        "onlyTopCall": false,
        "withLog": true
      }
    }
  ],
  "id": 1
}

Additional context
I tried the same call on the Rhino Public API, it throws an error too, but a different one:
curl  -X POST \
  'https://evm-rpc.sei-apis.com' \
  --header 'Content-Type: application/json' \
  --data-raw '{
  "jsonrpc": "2.0",
  "method": "debug_traceBlockByNumber",
  "params": [
    "0x4dfbedb",
    {
      "timeout": "10m",
      "tracer": "callTracer",
      "tracerConfig": {
        "onlyTopCall": false,
        "withLog": true
      }
    }
  ],
  "id": 1
}'

{
  "jsonrpc": "2.0",
  "id": 2727,
  "error": {
    "code": -32000,
    "message": "tracing failed: nonce too high: address 0x7Ac6d25FD5E437cB7c57Aee77aC2d0A6Cb85936C, tx: 979341 state: 0"
  }
}

fixed in https://github.com/sei-protocol/sei-chain/commit/225a8bb656e853cc9a15b4cd9334c44254e39d22


cursor position is offset

Is there an existing issue for this?

[X] I have searched the existing issues.

Is this issue related to iced?

[X] My hardware is compatible and my graphics drivers are up-to-date.

What happened?
drawing a point at the cursor location results in the point being drawn to the lower right of the cursor.
let c = Path::line(_cursor.position().unwrap_or_default(), _cursor.position().unwrap_or_default());
frame.stroke(&c, thin_stroke());

scaling the canvas by changing the window size does not appear to affect the offset amount measured in pixels. Easiest way to replicate this is to paste the code snippet above into the clock example around l128. The code snippet below seems to fix the problem:
frame.translate(Vector::new(-20., -20.));

using iced version 0.8
What is the expected behavior?
The point should be drawn at the upper left tip of the cursor pointer by default.
Version
master
Operative System
Windows
Do you have any log output?
No response

Do you have an SSCCE? The coordinates of the mouse cursor are absolute window coordinates, while a Canvas uses relative coordinates.
In order to get the proper position, you can use cursor.position_in(bounds) where bounds are the bounds provided to Program::draw.

The clock example has a padding of 20, which would explain the offset you are experiencing.


Add CI coverage for l3_edge in eos_designs_unit_tests and eos_designs_unit_tests_v4.0

Add the same test of l3_edge input data to both scenarios

Use old data model (dicts) in eos_designs_unit_tests
Use new data model (lists) in eos_designs_unit_tests_v4.0

Check that the generated structured config is the same across both scenarios

Solved in #2327


Finishing the tagging fix

Argh, missed another "v" prefix.

Coverage remained the same at 100.0% when pulling 905c98cc6f5dade9eb11428de9f900529c765297 on jmbowman/another_tag_fix into 3c97fb44ff1b576ab1afa0161340217febd71e1b on master.


Unknown switch error when running example code

Executing the line
dotnet run --project /projects/proj1/proj1.csproj
returns an error saying unknown switch. I think it should be corrected to:
dotnet run --project ./projects/proj1/proj1.csproj

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

ID: fd3f4227-cd9b-7b1c-7e1b-ffb3d42ae496
Version Independent ID: 7e78f9f5-f77c-828d-ef76-8ef8593d5803
Content: dotnet run command - .NET Core CLI
Content Source: docs/core/tools/dotnet-run.md
Product: dotnet-core
GitHub Login: @mairaw
Microsoft Alias: mairaw

Thank you @ragnarstolsmark for your feedback. We'll investigate this issue.


Docker-compose configuration doesn't work out the box

Using the default docker-compose configuration the brick-server fails because to many connections to Postgres.
Same situation as here https://github.com/BrickSchema/brick-server/issues/5#issuecomment-884902733
My workaround is setting the max_connections parameters for the Postgres instance.
Here the snippet in my docker-compose.yml
 brickserver-postgres:
     container_name: brickserver-postgres
     image: "jbkoh/brickserver-postgresql:latest"
     command: postgres -c 'max_connections=300'
     environment: # Below secrets should be matched with the information in `configs.json` too.
       - POSTGRES_USER=bricker
       - POSTGRES_PASSWORD=brick-demo
       - POSTGRES_DB=brick

the interesting line is:
command: postgres -c 'max_connections=300'

Can confirm. Was cursing to the sky until I found this. Thank you!

Hmm, resuming cursing the sky now because querying by entity_id just hangs forever and I lack the chops (or patience) to figure it out. Sigh.

I think this issue is out of date. Did you have a chance to try the new version?


No documentation on setting up email alerts in Thirdeye

Hi,
I am having difficulties testing Thirdeye alerting and receiving emails on my test alerts.
There is not much documentation on the issue and coupled with other bugs that break the interface, it prevents me from testing the tool.
Is there a chance that a simple tutorial can be written that covers everything from importing jdbc mysql metric, creating an alert, adding smtp server and sending out an email to specific recepients?

I updated the configuration docs. Please take a look  @vasilije90
https://thirdeye.readthedocs.io/en/latest/configuration.html

Looks good @xiaohui-sun
It covers most of the issues I faced setting it up.
A phantomJS install step would be nice, or at least a pointer.
I assume most systems won't have it installed.


Update levels.js

I changed the wording for the Simplified Chinese and Traditional Chinese translations. Specifically:

Changed 么 to 吗
Added a question mark.
Added 跳蹦

@Jlemien, thanks for the improvements!


Explain when you should use saveLayer with a CustomPainter

Drawing with a BlendMode unfortunately doesn't give the expected results. In the following case the source Rect should be gone and the dest Rect (the red one) should be cut out in the bottom right corner. Instead the source Rect just gets black.
class TestPaint extends CustomPainter {
	@override
	void paint(Canvas canvas, Size size) {
		canvas.drawPaint(new Paint()..color = Colors.transparent);
		canvas.drawRect(
			new Rect.fromLTWH(50.0, 100.0, 200.0, 200.0),
			new Paint()..color = Colors.red
		);
		canvas.drawRect(
			new Rect.fromLTWH(150.0, 200.0, 200.0, 200.0),
			new Paint()..color = Colors.white
				..blendMode = BlendMode.dstOut
		);
	}
}

It doesn't depend on the color of the second Rect. Other BlendModes like srcIn or dstIn don't seem to work either. Is this a bug?

@HansMuller or @Hixie would know.

@chinmaygarde or @abarth are more likely to know. Blend modes are dark magic to me.

Blend modes that depend on the contents of the dst buffer will depend on how the scene is composited.  For example, what draws the yellow rectangle?  Is that part of the dst buffer for this custom painter?  The answer seems likely to depend on the compositing strategy.

The yellow rectangle comes from an other underlying widget. TestPaint is a child to that widget. It was my understanding that in this case the two canvases are independent from each other, are they not?
So in the dst buffer for the custom painter there's nothing more than the transparent background (although it doesn't alter the outcome if it's there or not) and the two rectangles. That's why I don't understand where the black color can possibly come from.

It was my understanding that in this case the two canvases are independent from each other, are they not?

Whether the two canvases are independent depends on the compositing strategy.  One way to think about a compositing strategy is as a plan for which canvases are independent from each other (and therefore can be recomposited in a different way at a later point in time).

That's why I don't understand where the black color can possibly come from.

I suspect the canvas contains the yellow color from the parent widget.  The painting operation changes that to transparent, revealing the black that is composited behind these widgets in a separate canvas.

I don't think the canvas is passed to the child. To figure this out I created two CustomPaints with a circle on each of the two canvases. If I skew the second canvas, only the second circle is distorted. So the second canvas seems to be independent (at least when skewing).
Is it possible that this BlendMode problem has to do with this issue #9627? It's mentioned there that the standard clear color is not transparent but black:
canvas->clear(SK_ColorBLACK);
So when applying a dstOut filter a certain region gets cleared in the process (the cut-out corner in the red rectangle), but instead of getting cleared to transparent, it gets the standard clear color black?

Skew just changes the current transformation matrix for subsequent commands, so that is as expected.

Okay, then skewing is different.
But the question remains: Is there a way to have a clean and independent transparent CustomPainter canvas? In classic Android, this is the standard behavior. When I add a custom View to a Layout, the View canvas is perfectly independent from its parent (via overriding onDraw).

I believe that's what saveLayer gives you.

I'm going to reopen this and use it to track making sure the documentation is clear on this point.
I want to update at least: CustomPaint, CustomPainter, Canvas

Hi, any news on this issue?

@imaNNeoFighT how is https://api.flutter.dev/flutter/dart-ui/Canvas/saveLayer.html ?

@Hixie
I've tried to solve @robertgirschick's sample by saveLayer(), but it didn't work, maybe I don't know how I should use it.
Could you please show me an example?

how is https://api.flutter.dev/flutter/dart-ui/Canvas/saveLayer.html ?

Frustrating. :smile: It takes some trial and error to figure out whether "nothing happened" is the result of painting white on white, or getting the size and offsets wrong.  The nuances of anti-aliasing seem like they should be secondary to more basic examples.

Leaving this open since CustomPaint still doesn't seem to have a reference to saveLayer docs. This seems fixed for CustomPainter and BlendMode.

Updating the labels since the CustomPaint class doesn't seem to have a reference to saveLayer docs


Add Color Mask Flags to Materials

Added the ability to manipulate the OpenGL color mask to materials. This allows for materials that only write to the dept buffer, or other special effects.
This should fix #18901
In essence it introduces the necessary plumbing to control glColorMask from shaders via four new render modes, and adds the necessary flags to SpatialMaterial to easily control these flags without using shaders directly.

I've updated the PR and fixed a bug it introduced. SpatialMaterial::MaterialKey was using a fixed-width bit-field to track the material flags, and that bit-field was not large enough to hold the new flags.
Also added a few comments to make sure this interdependency is more obvious for anyone else adding flags in the future.

p.s.: If desired, I could also reduce this to a single flag (DISABLE_COLOR_WRITE) that completely turns of RGBA at the same time. I only need this for my own use case, but added all four components separately for more flexibility for other users.

This is an interesting feature, though I would probably implement it a bit different..

Can you give me a hint how to make this more acceptable to merge?
Juan Linietsky<EMAIL_ADDRESS>schrieb am Do., 4. Apr. 2019,
22:15:

This is an interesting feature, though I would probably implement it a bit
different..
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/pull/26697#issuecomment-480046691,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABM_V2XDB0U7PUAsf_2GKEL9PZdsB5NKks5vdl1JgaJpZM4bgpqj
.

After talking with reduz, this will not work in GLES 3 since it's not compatible with MRTs and glColorMaski is only available in Desktop GL.

Yes, this will definitely be doable for Godot 4.0, but can't be done for 3.x branch.

@shartte Is this still desired? If so, it needs to be rebased on the latest master, and redone for Vulkan.
If not, abandoned pull requests will be closed in the future as announced here.

Actually, regardless of this PR's viability, @shartte's fork of Godot has been deleted, so rebasing this would require a new PR from a new branch in a new fork. Therefore, I'm closing this.


Alignment on the content of the button.

I believe other developers also need the alignment of icon and text.
I have edited

main.dart for example
auth_button_style.dart
button_content.dart
for adding the mainAxisSize and mainAxisAlignment.

Example of use. See the mainAxisSize and mainAxisAlignment.
AppleAuthButton(
  onPressed: () {
    // your implementation
    setState(() {
      isLoading = !isLoading;
    });
  },
  themeMode: themeMode,
  isLoading: isLoading,
  style: AuthButtonStyle(
    width: double.infinity,
    buttonType: buttonType,
    iconType: iconType,
    margin: const EdgeInsets.only(left: 24, right: 24, bottom: 8),
    mainAxisSize: MainAxisSize.max,
    mainAxisAlignment: MainAxisAlignment.start,
  ),
),
FacebookAuthButton(
  onPressed: () {
    // your implementation
    setState(() {
      isLoading = !isLoading;
    });
  },
  themeMode: themeMode,
  isLoading: isLoading,
  style: AuthButtonStyle(
    width: double.infinity,
    buttonType: buttonType,
    iconType: iconType,
    margin: const EdgeInsets.only(left: 24, right: 24, bottom: 18),
    mainAxisSize: MainAxisSize.max,
    mainAxisAlignment: MainAxisAlignment.start,
  ),
),

Hi @thruthesky nice job,
probably they would like, I'll check for other properties to add.

@elbeicktalat any chance to merge it?

please add this.

@thruthesky Thanks for your contribution,
but I'm not going to merge your code since there are too much non related changes.
Such as formatting and adding platform code to git. Anyway this issue has been solved in #60


Remove error condition from get_global_transform()

Removes the arbitrary error in get_global_transform() method.
The error makes no sense for a couple of reasons:

it only triggers in debug build, which means the method works differently in release build
even with the error, the transform may not be truly "global", because Node ancestors stop transform propagation

if node is not inside tree, it has null parent, which is effectively the same as having Node parent

Closes #30445

This change makes sense to me. Don't see any problems with it.

@reduz originally added this in https://github.com/godotengine/godot/commit/83ae9a5e2859a1dc2393aebeb9f9efdf12c53e73, thoughts?

@mhilbrunner
In 83ae9a5 CanvasItem::set_notify_transform is changed to call get_global_transform, but verifies beforehand, that the node is in the tree.
So this PR does not change anything for CanvasItem::set_notify_transform, as far as I can tell.

Thanks!

Cherry-picked for 3.6.

Just seen this has been cherry picked! Yay, had been waiting for this for a while (see #30455 ) .
Ah, I've just seen, this PR is for canvas item, and the issue is for Spatial.
Also a possible bug - I haven't checked whether this occurs yet. If you remove a branch from the scene tree then unless the global transform is made dirty then the results will be stale (i.e. still represent the transform within the tree). So when removing from the tree we would need to propagate dirty flags to ensure consistent result. I'm not sure if this is done already, will check.


Nightly failed on apiWebdavPreviews/previews.feature:219

Build: https://drone.owncloud.com/owncloud/ocis/14181/52/7
@skipOnOcV10.6 @skipOnOcV10.7 @skipOnOcV10.8.0
  Scenario: updates to a file should change the preview for both sharees and sharers                                      # /drone/src/oc10/testrunner/tests/acceptance/features/apiWebdavPreviews/previews.feature:219
    Given the administrator has set the default folder for received shares to "Shares"                                    # OccContext::theAdministratorHasSetTheDefaultFolderForReceivedSharesTo()
    And auto-accept shares has been disabled                                                                              # FeatureContext::autoAcceptSharesHasBeenDisabled()
    And user "Brian" has been created with default attributes and without skeleton files                                  # FeatureContext::userHasBeenCreatedWithDefaultAttributesAndWithoutSkeletonFiles()
    And user "Alice" has created folder "FOLDER"                                                                          # FeatureContext::userHasCreatedFolder()
    And user "Alice" has uploaded file with content "file to upload" to "/FOLDER/lorem.txt"                               # FeatureContext::userHasUploadedAFileWithContentTo()
    And user "Alice" has shared folder "FOLDER" with user "Brian"                                                         # FeatureContext::userHasSharedFileWithUserUsingTheSharingApi()
    And user "Brian" has accepted share "/FOLDER" offered by user "Alice"                                                 # FeatureContext::userHasReactedToShareOfferedBy()
    And user "Alice" has downloaded the preview of "/FOLDER/lorem.txt" with width "32" and height "32"                    # FeatureContext::userDownloadsThePreviewOfWithWidthAndHeight()
    And user "Brian" has downloaded the preview of "Shares/FOLDER/lorem.txt" with width "32" and height "32"              # FeatureContext::userDownloadsThePreviewOfWithWidthAndHeight()
    When user "Alice" uploads file "filesForUpload/lorem.txt" to "/FOLDER/lorem.txt" using the WebDAV API                 # FeatureContext::userUploadsAFileTo()
    Then the HTTP status code should be "204"                                                                             # FeatureContext::thenTheHTTPStatusCodeShouldBe()
    And as user "Alice" the preview of "/FOLDER/lorem.txt" with width "32" and height "32" should have been changed       # FeatureContext::asUserThePreviewOfPathWithHeightAndWidthShouldHaveBeenChanged()
    And as user "Brian" the preview of "Shares/FOLDER/lorem.txt" with width "32" and height "32" should have been changed # FeatureContext::asUserThePreviewOfPathWithHeightAndWidthShouldHaveBeenChanged()
    When user "Brian" uploads file with content "new uploaded content" to "Shares/FOLDER/lorem.txt" using the WebDAV API  # FeatureContext::userUploadsAFileWithContentTo()
    Then the HTTP status code should be "204"                                                                             # FeatureContext::thenTheHTTPStatusCodeShouldBe()
    And as user "Alice" the preview of "/FOLDER/lorem.txt" with width "32" and height "32" should have been changed       # FeatureContext::asUserThePreviewOfPathWithHeightAndWidthShouldHaveBeenChanged()
      HTTP status code 404 is not the expected value 200
      Failed asserting that 404 matches expected 200.
    And as user "Brian" the preview of "Shares/FOLDER/lorem.txt" with width "32" and height "32" should have been changed # FeatureContext::asUserThePreviewOfPathWithHeightAndWidthShouldHaveBeenChanged()

72 scenarios (72 passed)
1296 steps (1296 passed)
4m53.91s (19.93Mb)
runsh: Total 72 scenarios (72 passed, 0 failed)
runsh: Exit code of main run: 0

Tried it to reproduce locally but not reproducable. I will try ti fail it on CI.

Passing consistently in CI as well https://drone.owncloud.com/owncloud/ocis/14187/40/8

@SagarGi Re-appeared today: https://drone.owncloud.com/owncloud/ocis/14356/51/7

ok i will be looking into it today as well.

The above scenario is not reproducible but can be reproduce when we keep the both content same as the above error says.
May be When user "Alice" uploads file "filesForUpload/lorem.txt" to "/FOLDER/lorem.txt" using the WebDAV API this step could be replace with When user "Alice" uploads file with content "new uploaded content one" to "/FOLDER/lorem.txt" using the WebDAV API since others scenarios are using the update step like this. @saw-jan

@SagarGi Re-appeared today: https://drone.owncloud.com/owncloud/ocis/14356/51/7 But in different step:
  @skipOnOcV10.6 @skipOnOcV10.7 @skipOnOcV10.8.0
  Scenario: updates to a file should change the preview for both sharees and sharers                                      # /drone/src/oc10/testrunner/tests/acceptance/features/apiWebdavPreviews/previews.feature:219
    Given the administrator has set the default folder for received shares to "Shares"                                    # OccContext::theAdministratorHasSetTheDefaultFolderForReceivedSharesTo()
    And auto-accept shares has been disabled                                                                              # FeatureContext::autoAcceptSharesHasBeenDisabled()
    And user "Brian" has been created with default attributes and without skeleton files                                  # FeatureContext::userHasBeenCreatedWithDefaultAttributesAndWithoutSkeletonFiles()
    And user "Alice" has created folder "FOLDER"                                                                          # FeatureContext::userHasCreatedFolder()
    And user "Alice" has uploaded file with content "file to upload" to "/FOLDER/lorem.txt"                               # FeatureContext::userHasUploadedAFileWithContentTo()
    And user "Alice" has shared folder "FOLDER" with user "Brian"                                                         # FeatureContext::userHasSharedFileWithUserUsingTheSharingApi()
    And user "Brian" has accepted share "/FOLDER" offered by user "Alice"                                                 # FeatureContext::userHasReactedToShareOfferedBy()
    And user "Alice" has downloaded the preview of "/FOLDER/lorem.txt" with width "32" and height "32"                    # FeatureContext::userDownloadsThePreviewOfWithWidthAndHeight()
    And user "Brian" has downloaded the preview of "Shares/FOLDER/lorem.txt" with width "32" and height "32"              # FeatureContext::userDownloadsThePreviewOfWithWidthAndHeight()
    When user "Alice" uploads file "filesForUpload/lorem.txt" to "/FOLDER/lorem.txt" using the WebDAV API                 # FeatureContext::userUploadsAFileTo()
    Then the HTTP status code should be "204"                                                                             # FeatureContext::thenTheHTTPStatusCodeShouldBe()
    And as user "Alice" the preview of "/FOLDER/lorem.txt" with width "32" and height "32" should have been changed       # FeatureContext::asUserThePreviewOfPathWithHeightAndWidthShouldHaveBeenChanged()
    And as user "Brian" the preview of "Shares/FOLDER/lorem.txt" with width "32" and height "32" should have been changed # FeatureContext::asUserThePreviewOfPathWithHeightAndWidthShouldHaveBeenChanged()
    When user "Brian" uploads file with content "new uploaded content" to "Shares/FOLDER/lorem.txt" using the WebDAV API  # FeatureContext::userUploadsAFileWithContentTo()
    Then the HTTP status code should be "204"                                                                             # FeatureContext::thenTheHTTPStatusCodeShouldBe()
    And as user "Alice" the preview of "/FOLDER/lorem.txt" with width "32" and height "32" should have been changed       # FeatureContext::asUserThePreviewOfPathWithHeightAndWidthShouldHaveBeenChanged()
      WebDav::asUserThePreviewOfPathWithHeightAndWidthShouldHaveBeenChanged previous and current previews content is same but expected to be different
      Failed asserting that Binary String: 0xffd8ffdb008400080606070605080707070909080a0c140d0c0b0b0c1912130f141d1a1f1e1d1a1c1c20242e2720222c231c1c2837292c30313434341f27393d38323c2e333432010909090c0b0c180d0d1832211c213232323232323232323232323232323232323232323232323232323232323232323232323232323232323232323232323232ffc00011080020002003012200021101031101ffc401a20000010501010101010100000000000000000102030405060708090a0b100002010303020403050504040000017d01020300041105122131410613516107227114328191a1082342b1c11552d1f02433627282090a161718191a25262728292a3435363738393a434445464748494a535455565758595a636465666768696a737475767778797a838485868788898a92939495969798999aa2a3a4a5a6a7a8a9aab2b3b4b5b6b7b8b9bac2c3c4c5c6c7c8c9cad2d3d4d5d6d7d8d9dae1e2e3e4e5e6e7e8e9eaf1f2f3f4f5f6f7f8f9fa0100030101010101010101010000000000000102030405060708090a0b1100020102040403040705040400010277000102031104052131061241510761711322328108144291a1b1c109233352f0156272d10a162434e125f11718191a262728292a35363738393a434445464748494a535455565758595a636465666768696a737475767778797a82838485868788898a92939495969798999aa2a3a4a5a6a7a8a9aab2b3b4b5b6b7b8b9bac2c3c4c5c6c7c8c9cad2d3d4d5d6d7d8d9dae2e3e4e5e6e7e8e9eaf2f3f4f5f6f7f8f9faffda000c03010002110311003f00f62123e3ef374f5a52efc82ed8c64fcc6b4fec36ff00dc3ff7d1a058db8fe03ff7d1a00ccdce40cbb67fdea7c137972077deea3b6e3fe35a1f61b70a1769c0ff0068d392d2042484ce7fbc73fce8027a28a2800a28a2803fffd9
is not equal to <text>.
    And as user "Brian" the preview of "Shares/FOLDER/lorem.txt" with width "32" and height "32" should have been changed

Re-occurred at  build: https://drone.owncloud.com/owncloud/ocis/14706/51/7

This issue cannot be reproduce locally. And all the failures seems to be with the content check. It fails sometimes on upper upload(change content) or for the second change(change content). Upon investigation the failure seems to be due to same content (can be reproduce when we make the content same when updating). But the tests is checking status code that file has been updated. But still intermittently failing.

https://github.com/owncloud/ocis/pull/4435 this is a debug PR with step   And the content of file "/FOLDER/lorem.txt" for user "Alice" should be "updated file one" . And is passing consistently on debug build. But cant certainly say that it could be the main reason reason of failure. But might help for debugging if it fails in coming days.

Tried with uploading large file and problem does not seem to be on upload content. Very Hard to reproduce locally or in CI.

Adding the content assertion step is surely helpful, but just for the debugging purpose. Strange that the multiple runs missed the flakiness. Maybe try with some different scenarios (not only single multiple scenarios). Or we may have to wait for to see if it is resolved already.

Not seen in recent build i am putting this to backlog. Can be brought back if reoccured

As this issue has not been reproduced for a while now, @grgprarup is going to do a final check and maybe we can close this if it is not reproducible.

Tried to reproduce this issue and is not reproducible locally and passing on recent nightly builds. So closing this issue.


Add jstrieb/sytems-programming-cheat-sheet

What does this PR do?
Add resource(s)
For resources
Description
A dense cheat sheet of useful x86-64 Unix system programming stuff
Why is this valuable (or not)?
There is no other such cheat sheet that includes useful implementation details and behavior for systems programming.
How do we know it's really free?
I am the author and released it for free!
For book lists, is it a book? For course lists, is it a course? etc.
It is a cheat sheet, as per the list it was added to.
Checklist:

[x] Read our contributing guidelines
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction)

Followup

Check the output of Travis-CI for linter errors!

Nice!  Please feel free to add author name. Thanks for making your cheatsheet free! Consider adding it to Archive.org.
Welcome to free-programming-books!


Prevent concurrent uploads with same lbry name

Ticket
Closes #464
Currently, we check if we have any existing claims with the same name when uploading, i.e. "lbry://name". It does not include claims that you are still uploading, so you might end up with duplicate claims.
In the ticket, there is also the issue of 2 uploads sharing the same GUI slot, causing the progress indicator to jump between the uploads. That has been fixed by a prior PR using a guid instead of using name.
Aside
I think there is another request to allow the same name but on different channel ... next time, next time ....

You can even do same names on same channel,but it needs: https://github.com/OdyseeTeam/odysee-api/issues/399


Add partitioning_mode to image-builder customizations

Once https://github.com/osbuild/image-builder/pull/876 passes and is merged I can rebase this.

/retest


bugfix

Fix bugs persisting statemachines.
Adapted coding style and some comments

I have reassessed changes and they are not necessary

Hi @manuelbcd , thank you for your time and contribution.
Some feedback on the line "$statement->bindParam(":machine", $identifier->getMachine());"
that line will make the PDO library complain, since for some reason the second parameter of the bindParam method should not be an expression/evaluation.
Are you using the library and what is your experience with it?

Hey! Thanks for contacting me. This Pull request was mistaken. Apologise
for the inconvenience.  my experience is good with statemachine and redis
but Im facing some problems when I use mysql pdo.
Wbr
Manuel Boira
El 19/4/2016 22:00, "Rolf Vreijdenberger"<EMAIL_ADDRESS>escribió:

Hi @manuelbcd https://github.com/manuelbcd , thank you for your time
and contribution.
Some feedback on the line "$statement->bindParam(":machine",
$identifier->getMachine());"
that line will make the PDO library complain, since for some reason the
second parameter of the bindParam method should not be an
expression/evaluation.
Are you using the library and what is your experience with it?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/rolfvreijdenberger/izzum-statemachine/pull/6#issuecomment-212102562


Failed to create Gitlab identity provider

Hi,
I have compiled the JAR and load in Keycloak. But when I configure it and save changes, it crashes:
 04:30:41,835 ERROR [org.keycloak.services.error.KeycloakErrorHandler] (default task-6) Uncaught server error: java.lang.AbstractMethodError: Receiver class io.apicurio.kc.ext.social.LocalGitLabIdentityProviderFactory does not define or inherit an implementation of the resolved method 'abstract org.keycloak.models.IdentityProviderModel createConfig()' of interface org.keycloak.broker.provider.IdentityProviderFactory.

Start command:
command: ["-b", "<IP_ADDRESS>", "-Dkeycloak.profile.feature.docker=enabled", "-Dapicurio.hub.gitlab.url=https://gitlab.example.com"
Keycloak version: 9.0.2
Gitlab version: GitLab Community Edition 12.9.2
Thanks

What version of Keycloak are you using?  That identity provider class has not been updated for awhile, so it's probably a version mismatch issue.

Hi,
Keycloak version: 9.0.2

Yeah that's the issue.  Those identity provider classes were written for an older Keycloak version.  I can add this task to my to-do list, but if you have the ability to look into this yourself it'll probably get done sooner.  :)
Basically you would need to update the version of Keycloak in the pom.xml file and then fix whatever compiler problems that result.
Also apologies for asking you a question that you had already included in the summary of the original problem report!  I just noticed... sigh


Split special functions into separate package

I propose to move module Statistics.Math into separate package. It contains functions which are useful on they own and not only in context of statistics. As for package name it could be special-functions
Proposed module structure is: Numeric.Polynomial.Chebyshev for Chebyshev polinomials. It makes sense to put them into separate module since there are Lagrange polynomials etc. It could get too crowded in one module.  Numeric.SpecFunction for everything else.
I could handle split but I could you create repository and give me commit rights?
I think after split module Statistics.Math should be marked deprecated but still export functions for one or two releases.

On Sat, Dec 10, 2011 at 12:51 PM, Aleksey Khudyakov <
<EMAIL_ADDRESS>
wrote:

I propose to move module Statistics.Math into separate package. It
contains functions which are useful on they own and not only in context of
statistics. As for package name it could be special-functions

That's a good idea. I created "math-functions" and added you as an admin:
https://github.com/bos/math-functions

Proposed module structure is: Numeric.Polynomial.Chebyshev for Chebyshev
polinomials. It makes sense to put them into separate module since there
are Lagrange polynomials etc. It could get too crowded in one module.
Numeric.SpecFunction for everything else.
I could handle split but I could you create repository and give me commit
rights?
I think after split module Statistics.Math should be marked deprecated but
still export functions for one or two releases.

Sounds great. Thanks!

I'll start working on it but I'm quite busy right now so it'll probably take some time.
Also I want to remind that pull request which fixes serious performance problem with uniformR for integral types hangs for more than two months. (https://github.com/bos/mwc-random/pull/6). At least GHC7.0 is affected
.

Package math-functions is more or less release ready. Code, test suite and benchmarks are ported
I've had to copy half of Statistics.Constants to the Numeric.SpecFuntions in order to break dependency.

OK, thanks for working on this - I just tagged and released <IP_ADDRESS>.

I'm reopening this since statistics doesn't use function from math-functions yet. Also I think it makes sense to move Statistics.Constants to math-functions as well. As Numeric.MathFunctions.Constants or something like that

I'm reopening this since statistics doesn't use function from math-functions yet. Also I think it makes sense to move Statistics.Constants to math-functions as well. As Numeric.MathFunctions.Constants or something like that

Now splitting is complete. I forgot to add few functions so another releases of math-functions is needed.


Minus character not appearing

VSCode Version: 1.36.1
OS Version: 10.14.4

Steps to Reproduce:

Open a new file. As far as I can tell, the file type does not matter (I've tried with Plain Text, JSON, JavaScript, Python, and SQL files).
Try to type a - key. For me, nothing happens (i.e. no characters are produced), even when I'm trying to comment out SQL lines or subtract integers in JS or Python.
However, the _ character works fine.

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

Could you please follow our wiki https://github.com/microsoft/vscode/wiki/Keybinding-Issues to help provide more info?

Figured this out, there was a weird keybinding attached to my - key. Thanks


add "row" processing in neo4j result

Hi,
Because some cypher queries could return rows, sigma neo4j plugin should manage those kind of results. It's just few lines of code those not load data processing.
I use rows queries to retrieve some data to put in sigma dataviz human interface.
Cheers.

Hi Cyrille !
Your are modifying the data state of sigma.js by adding a new attribute (row) that the API doesn't manage, and doesn't need.
Can you tell what you want to do with the row (and can not do with the graph) ?
Cheers.

Hi Simon,
I do not remember the initial need, just that some results data were in this rows.
By the way I close the issue until I'll come back to the case.
Cheers.


Migrate to AndroidX ?

I'll probably do a PR

https://github.com/zagum/Android-ExpandIcon/releases/tag/1.3.0

thanks


Signin/signup page

I want to make a Signin/ SignUp page. The page would look great and would add more values to the whole website.

Hey @vanishasamriddhi I don't want to keep any login or signin page. I have already closed an issue on this, you can check.


docs: Add missing link to npm.

Referenced in line 250 of dependencies.md

Merged, thanks @tommyip!

I'm a bit confused why our tooling didn't catch this; tools/build-docs should in theory do so?


Add ppc64le and s390x support to bootc-image-builder

The fedora-bootc images include all four architectures supported by Fedora, however bootc-image-builder  currently supports  x86_64 and aarch64 only.
Opening this to request/discuss adding support for ppc64le and s390x .

Could you please elaborate what is not working? We have a smoke test in https://github.com/osbuild/bootc-image-builder/blob/main/test/testcases.py#L102 that will generate manifests for ppc64le/s390x which should be supported via --target-arch. Could you please paste the failing commandline?

Is this issue about us not providing bib container images for these arches on quay?


[$500]  IOU - Enter click does not deselect/select user in split bill

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.34-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4255773
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Open the app
Open any group conversation
Click on plus and click on split bill
Enter any amount and continue
Using arrow keys, navigate between users on final confirmation page
While any user is focus, click enter and observe that instead of deselecting the user, it trigger split bill

Expected Result:
When user is focused on split bill confirmation preview, enter click should deselect/select the user
Actual Result:
When user is focused on split bill confirmation preview, enter click triggers split bill
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/74441cc3-75f2-4b66-b572-d9a40fd92629
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01c60de4a237522ca5
Upwork Job ID:<PHONE_NUMBER>215609344
Last Price Increase: 2024-01-31

We think that this bug might be related to #vip-vsp
CC @quinthar

Proposal
Please re-state the problem that we are trying to solve in this issue.
IOU - Enter click does not deselect/select user in split bill
What is the root cause of that problem?
We are calling the onFinish function for types of IOU requests.
Therefore, the user will advance from the MoneyTemporaryForRefactorRequestParticipantsSelector page when addSingleParticipant is called.
https://github.com/Expensify/App/blob/2d2ea113f18d5e99e77ee645fe7fb22cb0a9f6bc/src/pages/iou/request/MoneyTemporaryForRefactorRequestParticipantsSelector.js#L183-L191
What changes do you think we should make in order to solve the problem?

Only call the onFinsh function if iouType !== CONST.IOU.TYPE.SPLIT) is true.
if (iouType !== CONST.IOU.TYPE.SPLIT) {
    return
}
onFinish();

Furthermore, the onFinish will be called when the user uses the keyboard shortcut CMD+ENTER.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Request money - Missing Tooltip for "Cancel" Icon Linked to "Request money, get $250
What is the root cause of that problem?
We are using the pressOnEnter prop even if the iou is a split request.
https://github.com/Expensify/App/blob/2d2ea113f18d5e99e77ee645fe7fb22cb0a9f6bc/src/components/MoneyTemporaryForRefactorRequestConfirmationList.js#L594-L595
Therefore, the user will confirm the request.
What changes do you think we should make in order to solve the problem?
Create a condition to use the  pressOnEnter prop properly.
For example.
pressOnEnter={iouType !== CONST.IOU.TYPE.SPLIT}

Proposal
Please re-state the problem that we are trying to solve in this issue.
IOU - Enter click does not deselect/select user in split bill
What is the root cause of that problem?
In the MoneyTemporaryForRefactorRequestConfirmationList, we are using the pressOnEnter prop even if the iou is a split request and the user can modify the participants.
https://github.com/Expensify/App/blob/2d2ea113f18d5e99e77ee645fe7fb22cb0a9f6bc/src/components/MoneyTemporaryForRefactorRequestConfirmationList.js#L594-L595
Therefore, the user will confirm the request and create a transaction when the ENTER and CMD+ENTER keyboard shortcuts.
What changes do you think we should make in order to solve the problem?
Create a condition to set the pressOnEnter prop appropriately.
For example:
<ButtonWithDropdownMenu
    pressOnEnter={!userCanModifyParticipants.current}

Furthermore, the user can use the CMD+ENTER shortcut to create the transaction.
Result
What alternative solutions did you explore? (Optional)
We could remove the pressOnEnter prop but depending on the onSelectRow pressing the enter button could make the user navigate to the profile page.
https://github.com/Expensify/App/blob/2d2ea113f18d5e99e77ee645fe7fb22cb0a9f6bc/src/components/MoneyTemporaryForRefactorRequestConfirmationList.js#L621

@Krishna2323 Makes sense!
I approve the proposal by @Krishna2323
C+ reviewed 🎀 👀 🎀

The PR that introduced the bug has been identified. Link to the PR:

It seems that the bug was present in the relevant components since they were created

The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:

N/A

A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

No need for additional discussion

Determine if we should create a regression test for this bug.

Up to the QA team

If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Open any group conversation
Press the "Actions" (small plus) button and select on "Split bill"
Enter any amount and continue
Using arrow keys, navigate between users on the final confirmation page
While any user is focused, press the "Enter" key and verify that that user got selected/deselected

Payment Summary:
Contributor: $500 @Krishna2323 paid in Upwork
C+: $500 @cubuspl42 paid in Upwork


Use more stable form of Haversine formula.

This PR would solve issue #5932. The new formula does not give a non-zero distance from a point to itself. For pairs of distinct points, the new formula gives the same results as the old one, to precision ~1e-7 (this large differences are basically only encountered with near antipodal points, more often the difference is much smaller, around 1e-9).
JSFiddle comparing the versions: https://jsfiddle.net/Ljacqhpj/1/
Performance-wise the formulas are pretty much identical.

Here's the Node snippet I used to evaluate performance:
const R = 6371000;
const distance = (latlng1, latlng2) => {
	var rad = Math.PI / 180,
	lat1 = latlng1.lat * rad,
	lat2 = latlng2.lat * rad,
	a = Math.sin(lat1) * Math.sin(lat2) +
      Math.cos(lat1) * Math.cos(lat2) * Math.cos((latlng2.lng - latlng1.lng) * rad);

	return R * Math.acos(Math.min(a, 1));
};

const distance2 = (latlng1, latlng2) => {
	var rad = Math.PI / 180,
	lat1 = latlng1.lat * rad,
	lat2 = latlng2.lat * rad,
	dLat = (latlng2.lat - latlng1.lat) * rad,
	dLon = (latlng2.lng - latlng1.lng) * rad,
	a = Math.sin(dLat / 2) * Math.sin(dLat / 2) +
      Math.cos(lat1) * Math.cos(lat2) * Math.sin(dLon / 2) * Math.sin(dLon / 2),
	c = 2 * Math.atan2(Math.sqrt(a), Math.sqrt(1 - a));
	return R * c;
};

const np = 100000;
let x1, x2, d1 = new Array(np), d2 = new Array(np);
console.time('d1');
for (let i = 0; i < np; i++) {
	x1 = {lat: 180 * Math.random() - 90, lng: 360 * Math.random() - 180};
	x2 = {lat: 180 * Math.random() - 90, lng: 360 * Math.random() - 180};
	d1[i] = distance(x1, x2);
}
console.timeEnd('d1');

console.time('d2');
for (let i = 0; i < np; i++) {
	x1 = {lat: 180 * Math.random() - 90, lng: 360 * Math.random() - 180};
	x2 = {lat: 180 * Math.random() - 90, lng: 360 * Math.random() - 180};
	d2[i] = distance2(x1, x2);
}
console.timeEnd('d2');

@mourner think your comments on this would be valuable. I don't really know the history behind selecting the current formula, but perhaps you remember?

I've set up a jsperf benchmark at https://jsperf.com/haversine-distance - it seems that @jussimattas is actually faster than the current implementation, even with the duplicated calls to Math.sin(), at least in FFX 57.

Updated JSFiddle: https://jsfiddle.net/Ljacqhpj/2/

Alright. Still needs @IvanSanchez's approval.


Create java

First of all, Thanks for collaborating with us!!! Please dont spam. give a valid code we will definately merge that.

Hello @biswabaibhab007 , first of all thanks for collaborating with me!!! Really feeling nice to have a conversation with you....But here I cannot see any file in your pull request....Can you please upload it again???


[Common Issue] Check here for some common issues you might encounter

Common Issue
More Questions will be added......
Training Related
Q : Can not finetune the existing LLaVA-Onevision checkpoints
A : We edit our model's config so that it is able to be served on sglang. To finetune on existing LLaVA-Onevision checkpoints, you might first need to download all the weight and change the model_type in the config.json from llava to qwen2
Q : Can not download LLaVA-NeXT data or LLaVA-Onevision Data?
A : Sometimes we found out that the version of the dataset can cause issue. You can try update your dataset before performing the load_dataset.
Q : Data is not complete?
A : Yes, LLaVA-NeXT data is not complete as we can not release the 15k user data. For other data such as video or multi-images, it is splited into different dataset repo on huggingface. For example, lmms-lab/M4-Instruct-Data, lmms-lab/LLaVA-ReCap-CC3M, lmms-lab/ShareGPTVideo, lmms-lab/LLaVA-ReCap-558K, lmms-lab/LLaVA-ReCap-118K

Q: About video data?
A: It's to be released in @ZhangYuanhan-AI next version of a more powerful video model.
Currently we released the data yaml used in onevision stage at onevision.yaml.
You can checkout the three subsets video data, (1) sharegpt4video_255000.json (checkout sharegpt4video) (2) 0718_0_30_s_academic_mc_v0_1_all.json (to be released) (3) academic_source_30s_v1_all.json (to be released).

Hi,
Thank you very much for your outstanding work, I have a question to ask:
What is the final batchsize? (num_gpu * batch_size_per_device * accumulation_steps). Including pretrain and fintune
https://github.com/LLaVA-VL/LLaVA-NeXT/blob/79ef45a6d8b89b92d7a8525f077c3a3a9894a87d/scripts/train/direct_finetune_clip.sh#L42
Looking forward to your reply
Thanks


[AA-1106] Finalize Home Index for .NET Core

@plioi @saa14
Please help review and merge the changes.
Thanks

At this point, I think the HomeController should be shared between the Web and Web.Core projects since they are (nearly) identical here. That would mean (not in Solution Explorer!), cutting the new HomeController from Web.Core and pasting it on top of the old one in Web, then using our csproj file sharing to make it available here in Web.Core as well. This would make it much more clear what the edge cases are. In this PR, it just looks like a whole new controller.

Be sure to investigate the build failure.

I made a few additions in order to get the tests to pass. When the tests run under .NET Core we needed to take a few extra steps to ensure that we were getting the new config value infrastructure initialized just like during a run of the web application. This is great, because it motivated some setup we'd like to start adopting more and more in the tests, so the first step of that appears as part of this PR.


Break out the CMS related code into a new class.

Break out the CMS related code (v2 related) into a new class and move tests out of cert_checker_test.

This is almost identical to code submitted last week. Just pulls it out and tweaks the API.

Naming things is hard! I would maybe go with CMSVerifier. It does verify the CMS signature of the message, but not that the signer cert is part of a valid chain as we use in CT.

I think that's all fixed up now.


web interface

we probably need one
Should it be:

A separate process getting data through the API and/or our own RPC, or
Part of the same process that serves the API and manages the data and actual communication with dcrd?

yes, we need a browsable web interface to complement the api. Id go with in the same process since the web interface code should be easily compartmentalized within the project and will share the rest of the code with the api. Should use the same functions (and data sources) for retrieving the data as the api. Likewise, everything viewable on the web interface should be accessible via the api. Minimal design but good hyperlinking connection of the data on the initial templates.

Closed by https://github.com/dcrdata/dcrdata/pull/14, but that's just a start.


Azure DevOps compatibility issues

In source-controller we use go-git to perform Git operations.
According to these issues https://github.com/go-git/go-git/issues/64, https://github.com/src-d/go-git/issues/335, https://github.com/src-d/go-git/issues/1058 Azure DevOps doesn't implement Git v1 protocol and go-git doesn't implement v2 that supposed to work with Azure. As shelling out to the git binary is not an option for us, we have to figure out a way to Azure DevOps.

Watching this - curious as to what solution you guys come up with

Leaving a comment here too. As we are also affected by this.
@stefanprodan : Would it mean, in worst case we need to extend the go-git code for v2 handling?

Would it mean, in worst case we need to extend the go-git code for v2 handling?

Either Azure DevOps implements Git v1 or we need to add support for v2 in go-git.

:wave:  hi, I'm the product manager for Azure Repos server (which is where protocol stuff lives). I'm afraid we have no plans to relax the requirement that clients support multi_ack. It looked from some of the related issues like go-git is not planning to implement multi_ack but is working on a v2 protocol implementation.

If anyone is interested following along the work to get this fixed. https://github.com/go-git/go-git/pull/200

With @vtbassmatt stance and @phillebaba PR getting rejected we are now investigating if it's possible to replace go-git with Go bindings for libgit2. Meanwhile if your are an Azure DevOps user, please consider switching to GitHub.

@stefanprodan That would be awsome!


carousel active when there is only one image

Hi,
I have a problem when for some movies only 1 image is fetched from API, but carousel is displaying right arrow and when I click it, of course next slide is empty. If I click even more times the whole carousel goes bananas...
I tried setting [overflowCellsLimit]="1" but that didn't help
Is there some way to disable carousel when there is only one image, but still display that image?
thanks

Try to install the latest version, is this problem still there?


Fix hovering state when target contains special characters

This PRs fixes the following issue:

#1794
#1687
#1083 (probably)

CSS class names are NOT allowed spaces but every other characters is fine (https://www.w3.org/TR/CSS21/syndata.html#characters).
Since they are all prefixed by c3-targets- we do not have to care about the first character.
We simply need to escape in querySelector or in CSS if any.
I don't know if c3-target-XXX  is considered public or private api but this PR will break css/js based on this css class when target contains special characters.

close #1687

I confirmed this fixes the example of #1687
This change breaks the case when the data name contains special characters other than - and the user uses c3-target-XXX directly from their code. I think that's very rare usage. (possibly none)
The current replacing rule has an obvious defect, so I think nobody wants that behavior and it's ok to merge this.


fix: DIA-1334: Catch and report pydantic ValidationError

Specifying to instructor/tenacity to not retry on pydantic ValidationError
When this case is hit, raising a custom exception to return to LSE - ContrainedGenerationError

Commit https://github.com/HumanSignal/infra/commit/afbddd94456f6d916e860c6e1e4d9b9e66559fe2 is created.

Codecov Report
Attention: Patch coverage is 53.84615% with 6 lines in your changes missing coverage. Please review.

Project coverage is 58.98%. Comparing base (ea653ae) to head (ab76a75).
Report is 3 commits behind head on master.

Files
Patch %
Lines

adala/runtimes/_litellm.py
55.55%
4 Missing :warning:

adala/utils/exceptions.py
50.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #184      +/-   ##
==========================================
- Coverage   59.34%   58.98%   -0.36%
==========================================
  Files          43       44       +1
  Lines        1894     1919      +25
==========================================
+ Hits         1124     1132       +8
- Misses        770      787      +17

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

Commit https://github.com/HumanSignal/infra/commit/122a0bcf1f963aaca329f335ca9040986e6212f6 is created.

Commit https://github.com/HumanSignal/infra/commit/34c9a2923d79c779a5dfbbc406d683d3018c13ff is created.


onConnected is not called all the time

i av used previous version and it was called maybe ur socket is not
comnected
SmileCs past3.com.ng
On 11 Feb 2016 21:29, "sk003cs"<EMAIL_ADDRESS>wrote:

—
Reply to this email directly or view it on GitHub
https://github.com/TakahikoKawasaki/nv-websocket-client/issues/34.


[Question]: How to print the number of tokens used on each query? I want to print that.

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
How to print the number of tokens used on each query? I want to print that.

this is not working. Can you please help me with a very clear code ?

@rahul311291 yea the bot tried its best there lol
Here's an actual notebook example https://gpt-index.readthedocs.io/en/stable/examples/callbacks/TokenCountingHandler.html

token_counter.total_embedding_token_count always returning 0

@uzumakinaruto19 As of now, Dosu only responds to the author of issues. If you have any feature requests for Dosu, let me know!

@uzumakinaruto19 I think you messaged in discord? But yea, token counting only works for streaming in newer versions of llama-index


Update Email address relationship #46

As stated in my updated comments in https://github.com/tipoff/forms/issues/31, tests will fail since the references in the AttachContactUsersCommand need to be updated to reflect how email is now setup

@chx2 Let's go ahead and remove the AttachContactUsersCommand from this package in this PR. With the new Email Address model (https://github.com/tipoff/authorization/issues/57), we will now know the User (https://github.com/tipoff/authorization/issues/20) for the Contact (#30) through that relationship since Email Address has a nullable user_id and will be updated when a user is created with that Email Address.

Removed AttachContactUsers Command


Left horizontal slide out menu not working properly in mobile

when i try to open slide out menu it works but when i try to scroll through it ,it unexpectedly closes

@sagar-shah94 if this is still an issue could you provide some reproduce steps?

This issue was moved to firebase/friendlychat-android#8


How does steps translate to epochs?

Does a step refer to the training of a number of images equal to the batch size? So if I have training size = 4200 and batch size = 8, is 1 epoch equal to 4200/8 = 525 steps?

Yes, you are right.

Thank you


OpenNMT does not create checkpoints => index out of bounds error when training

I cloned the repository and ran the argos-train-init script.
After that, when running the argos-train command and using en to de as the languages, the data gets downloaded and the script  continues until an index out of bounds error. Error is below. The part with [] is the print of the opennmt_checkpoints in argostrain/train.py:165

Ubuntu 22.04
Python 3.10.6

Try PYTORCH_CUDA_ALLOC_CONF=max_split_size_mb:512 to fix the CUDA out of memory error. https://forum.opennmt.net/t/training-a-multilingual-model-with-opennmt-py/5242/13
The list out of range is caused by the out of memory crash.


Support for Dynomite running over memcached

Hi,
Could someone please explain why do we need a specific dyno client for Dynomite running over memcached? I thought Dynomite has it's own api, so Dyno client talks directly to Dynomite (so it's should be back-end specific).
I would appreciate any advices on how to use DYno client with Dynomite running over memcached.
Thanks

Dyno does not have its own API. Dyno is a wrapper around other clients, like Jedis, Redisson, and Memcache. We mainly exercise the Jedis at Netflix, so we have not been paying much attention to the Memcache. However, the structure is adaptable and if somebody wants to use it, then we would love to assist.

OK. Several other questions related to memcached support:

When I start dynomite (I'm using code from v0.6 branch) it often fails (50% of the time) with:
[2017-11-20 12:57:52.619] conf_push_scalar:541 push 'memcached-4:8101:memcached-rack2:dc2:4294967294' [2017-11-20 12:57:52.620] conf_handler:1291 conf handler on dyn_seeds: memcached-4:8101:memcached-rack2:dc2:4294967294 [2017-11-20 12:57:52.622] _dn_alloc:259 malloc(24) at 0x1c53e30 @ ../dyn_array.c:64 [2017-11-20 12:57:52.623] conf_server_init:134 init conf server 0x1c538c0 [2017-11-20 12:57:52.660] dn_resolve_inet:520 address resolution of node 'memcached-4' service '8101' failed: Name or service not known [2017-11-20 12:57:52.660] _dn_free:305 free(0x1c53ed0) @ dyn_string.c:59 [2017-11-20 12:57:52.660] conf_handler:1302 conf: directive "dyn_seeds" has an invalid value [2017-11-20 12:57:52.660] conf_pop_scalar:564 pop 'memcached-4:8101:memcached-rack2:dc2:4294967294' [2017-11-20 12:57:52.660] _dn_free:305 free(0x1c53e50) @ dyn_string.c:59 dynomite: configuration file '/etc/dynomite/conf/memcached-3.yml' syntax is invalid [2017-11-20 12:57:52.660] conf_pop_scalar:564 pop 'dyn_seeds' [2017-11-20 12:57:52.660] _dn_free:305 free(0x1c53810) @ dyn_string.c:59 [2017-11-20 12:57:52.660] conf_pop_scalar:564 pop 'dyn_o_mite' [2017-11-20 12:57:52.660] _dn_free:305 free(0x1c53790) @ dyn_string.c:59 [2017-11-20 12:57:52.660] _dn_free:305 free(0x1c41b70) @ ../dyn_array.c:82 [2017-11-20 12:57:52.660] _dn_free:305 free(0x1c53710) @ dyn_string.c:59 [2017-11-20 12:57:52.660] dn_stacktrace:326 [0] /lib/x86_64-linux-gnu/libpthread.so.0(+0x11390) [0x7fa2eaa2c390] ??:0 [2017-11-20 12:57:52.664] dn_stacktrace:326 [1] dynomite(string_deinit+0x3) [0x429953] /dynomite/src/dyn_string.c:58 [2017-11-20 12:57:52.669] dn_stacktrace:326 [2] dynomite(conf_destroy+0x55) [0x410385] /dynomite/src/dyn_conf.c:146 [2017-11-20 12:57:52.674] dn_stacktrace:326 [3] dynomite(conf_create+0x170) [0x410680] /dynomite/src/dyn_conf.c:2233 [2017-11-20 12:57:52.681] dn_stacktrace:326 [4] dynomite(core_start+0x80) [0x415350] /dynomite/src/dyn_core.c:281 [2017-11-20 12:57:52.685] dn_stacktrace:326 [5] dynomite(main+0x64f) [0x40cdff] /dynomite/src/dynomite.c:574 [2017-11-20 12:57:52.690] dn_stacktrace:326 [6] /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf0) [0x7fa2ea368830] ??:0 [2017-11-20 12:57:52.693] dn_stacktrace:326 [7] dynomite(_start+0x29) [0x40d2a9] ??:? [2017-11-20 12:57:52.698] signal_handler:132 signal 11 (SIGSEGV) received, core dumping

meaning it can't discover another dynomite node.
Also, if one node goes down, and other node tries to replicate - it also goes down.
I'm running several dynomite nodes in docker, so there is definitely some race condition between nodes, but  guess dynomite should some how gracefully handle such cases (try to reconnect several times after some timeout?).
What would you suggest to do to avoid this problem?
Here is my config (for reference):
`dyn_o_mite:
datacenter: dc2
dyn_listen: <IP_ADDRESS>:8101
dyn_port: 8101
dyn_seed_provider: simple_provider
dyn_seeds:

memcached-4:8101:memcached-rack2:dc2:4294967294
listen: <IP_ADDRESS>:8102
rack: memcached-rack2
servers:
<IP_ADDRESS>:11211:1
timeout: 150000
tokens:<PHONE_NUMBER>
data_store: 1`

`dyn_o_mite:
datacenter: dc2
dyn_listen: <IP_ADDRESS>:8101
dyn_port: 8101
dyn_seed_provider: simple_provider
dyn_seeds:

memcached-3:8101:memcached-rack2:dc2:2147483647
listen: <IP_ADDRESS>:8102
rack: memcached-rack2
servers:
<IP_ADDRESS>:11211:1
timeout: 150000
tokens:<PHONE_NUMBER>
data_store: 1`

2)When I'm trying to use Spy Memcached client and save data - it works fine.
But when I'm trying to read I receive:
[2017-11-20 12:44:49.561] server_rsp_forward:834 <CONN_CLIENT 0x528b660 11 from '<IP_ADDRESS>:54290'> <REQ 0x556fc00 4124:0::1 REQ_MC_GET, len:14> RECEIVED <RSP 0x5b1ce00 4126:0 RSP_MC_END len:5> [2017-11-20 12:44:49.561] stats_server_to_metric:1800 metric 'read_responses' for server [2017-11-20 12:44:49.561] _stats_server_incr:1816 incr field 'read_responses' to 1 [2017-11-20 12:44:49.561] stats_server_to_metric:1800 metric 'read_response_bytes' for server [2017-11-20 12:44:49.561] _stats_server_incr_by:1846 incr by field 'read_response_bytes' to 5 [2017-11-20 12:44:49.561] msg_local_one_rsp_handler:1005 <REQ 0x556fc00 4124:0::1 REQ_MC_GET, len:14> SELECTED <RSP 0x5b1ce00 4126:0 RSP_MC_END len:5> [2017-11-20 12:44:49.564] dn_stacktrace:326 [0] /lib/x86_64-linux-gnu/libpthread.so.0(+0x11390) [0x7f07351b4390] ??:0 [2017-11-20 12:44:49.578] dn_stacktrace:326 [1] dynomite(memcache_post_coalesce+0x114) [0x4331a4] /dynomite/src/proto/dyn_memcache.c:1556 [2017-11-20 12:44:49.596] dn_stacktrace:326 [2] dynomite(req_done+0x174) [0x420994] /dynomite/src/dyn_request.c:148 [2017-11-20 12:44:49.612] dn_stacktrace:326 [3] dynomite() [0x40db52] /dynomite/src/dyn_client.c:282 [2017-11-20 12:44:49.622] dn_stacktrace:326 [4] dynomite(msg_recv+0x575) [0x420385] /dynomite/src/dyn_message.c:890 [2017-11-20 12:44:49.632] dn_stacktrace:326 [5] dynomite(core_core+0xff) [0x414aff] /dynomite/src/dyn_core.c:386 [2017-11-20 12:44:49.641] dn_stacktrace:326 [6] dynomite(event_wait+0xeb) [0x43a90b] /dynomite/src/event/dyn_epoll.c:259 (discriminator 2) [2017-11-20 12:44:49.713] dn_stacktrace:326 [7] dynomite(core_loop+0x166) [0x415996] /dynomite/src/dyn_core.c:650 (discriminator 4) [2017-11-20 12:44:49.732] dn_stacktrace:326 [8] dynomite(main+0x671) [0x40ce21] /dynomite/src/dynomite.c:585 [2017-11-20 12:44:49.743] dn_stacktrace:326 [9] /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf0) [0x7f0734af0830] ??:0 [2017-11-20 12:44:49.755] dn_stacktrace:326 [10] dynomite(_start+0x29) [0x40d2a9] ??:? [2017-11-20 12:44:49.774] signal_handler:132 signal 11 (SIGSEGV) received, core dumping /dynomite/startup.sh: line 6:     9 Segmentation fault      (core dumped) dynomite --conf-file=/etc/dynomite/conf/${DYN_CONFIG_FILE} --verbosity=11
is this some known problem? I saw an old branch (memcachedFix) - can you merge the fix to the main branch? what was the fix?
Thanks in advance!

I suggest you open an issue on Dynomite as this is not a Dyno issue. The fixes can be found here: https://github.com/Netflix/dynomite/commit/5f6cef310b495f48baaa179bae77ad3a74f141ce They are mostly taken out of the corresponding code in Twemproxy and slightly adapted for Dynomite: https://github.com/twitter/twemproxy/blob/master/src/proto/nc_memcache.c Generally speaking adding a protocol is a relatively easy effort, so feel free to fix the memcache portion.

Got it. Thanks for quick feedback


platform/api/equinixmetal: wipe disks before installing

platform/api/equinixmetal: wipe disks before installing
On EM disk names are not stable and may change across reboots. This is
generally not a problem, but since we started reusing machines, it is
much more likely that there will be a leftover flatcar installation on
one disk and we are now installing to the second. This messes with some
tests because partitions from both disks may be mixed up when mounting
by label. Fix this by wiping the partition tables from disks on
reinstallation.
This should resolve a flake we've seen with cl.ignition.oem.regular, and others where "expected /dev/sda found /dev/sdb" show up.
How to use
[ describe what reviewers need to do in order to validate this PR ]
Testing done
[Describe the testing you have done before submitting this PR. Please include both the commands you issued as well as the output you got.]

[ ] Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)

Let's keep the commit around, in case we completely misread the situation.


Update AEXML v4.6.1

Resolves ~~https://github.com/tuist/xcodeproj/issues/YYY~~
Short description 📝
Update AEXML version to v4.6.1
Solution 📦
I fixed XML's newline escaping.
https://github.com/tadija/AEXML/pull/177
Implementation 👩💻👨💻

[x] Step 1 Fork -> Clone
[x] Step 2 Executed open Package.swift
[x] Step 3 Updated AEXML version in xcodeproj.podspec , Cartfile , Carffile.resolved & Package.swift

Hi.
Is it possible to remove .swift-version from this repository?
Then, I was able to resolve dependencies in my environment.

@all-contributors add @nnsnodnb for code

Thanks a lot for helping with this @nnsnodnb. If CI is green I'll merge the PR.

This seems like it has broken the checks, it's probably worth some investigation.


Add some missing {% trans %}

IN HIS NAME
Dear kmmbvnr,
Hi,
I'm using your lovely template in another language (Farsi).
It seems some of the {% trans %} were missed, so I add them here.
May be it can be useful for you.
Regards,
Ali

Great! Thnks

you are welcome ;)


Git repository: %s

and

Git error occurred while getting commit. Please check if Gerrit is configured as remote for the currently used Git repository.

I have configured "pushInsteadOf" in git, with gerrit server.

@ArminVieweg: Please check out the following hint (3rd point):
https://github.com/uwolfer/gerrit-intellij-plugin#list-of-changes-is-empty
"pushInsteadOf" is not respected.

Do you mean with "add a remote" this?

Unfortunately I just get  a "Connection failed" error. "Failed to send channel request".
But maybe I am totally wrong.

Ah sorry.

Do you get any additional messages? Please also check the "Version control"-tab in IntelliJ after you get this error.

@ArminVieweg: Please see my last comment.

Oh sorry!
Here is what version control says:

23:44:07.774: cd D:\vhosts\typo3master.local\.git
23:44:07.774: git -c core.quotepath=false fetch --progress Packages/TYPO3.CMS refs/changes/23/36823/1
fatal: Project not found:
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
23:44:10.427: cd D:\vhosts\typo3master.local
23:44:10.427: git -c core.quotepath=false log b22d67df86b44be0cecc1631dea399e1b19253e2 -1 --pretty=format:%x01%H%x02%ct%x02%an%x02%at%x02%ae%x02%cn%x02%ce%x02%P%x02%s%x02%b%x02%B%x03 --encoding=UTF-8 -M --name-status -c --
fatal: bad object b22d67df86b44be0cecc1631dea399e1b19253e2

@ArminVieweg: Thanks. It looks like the first "cd" is wrong? It shouldn't change into "typo3master.local/.git". Would "typo3master.local/Packages/TYPO3.CMS" be correct?

No, this D:\vhosts\typo3master.local\ is the root of the cloned repository.

@ArminVieweg: Thanks. Just to clearify: D:\vhosts\typo3master.local\ is your clone directory of the project Packages/TYPO3.CMS (and not a subdirectory)?

Yes. Sounds like that the missing structure is the reason ;)

@ArminVieweg: I have just tried to rebuild your setup (with TYPO3) and it seems to work fine here.
The content I have added in .git/config:
[remote "Packages/TYPO3.CMS"]
        url = https://review.typo3.org/Packages/TYPO3.CMS
        fetch = +refs/heads/*:refs/remotes/origin/*

Please change your ssh url with the https one for this remote. Since you are still pushing to "origin" (I assume), it won't hurt. This remote is just used for the Gerrit plugin.
Btw, what does your origin look like? For me the plugin also worked without adding a remote.

@ArminVieweg: Have you had time to check things as written in my feedback above?

@ArminVieweg: Would be great if you could confirm that my last comment helped. If it still does not work for you, I'd like to look into this issue once again.

@uwolfer: I'm not the reporter, but I had a similar issue and your suggestion resolved it.

Sorry for not responding! I'm not into this topic anymore. But great to see, that this post helped nevertheless :)


Minor display glitch: line separating selection from search result doesn't go all the way down to the footer.

I've been wrestling with this for a while. For the next release, I am going to try to refactor the sidebar so it doesn't use 'position:fixed' (which is how it currently always stays visible even when you scroll down)
For today, I can either do what I'm doing now, which is allow room at the bottom so it doesn't cover the footer - which doesn't scale when you change your scrolling, hence the gap - or let the sidebar overlay the footer. Another option is actually getting rid of the footer altogether.

I would just leave it for now.
Carl
Sent from my iPhone
Dr. Carl Kesselman
Dean's Professor, Epstein Dept. Industrial and Systems Engineering
Fellow, Information Sciences Institute
Viterbi School of Engineering
Phone: +1<PHONE_NUMBER>
Email: carl<EMAIL_ADDRESS>On Oct 12, 2015, at 10:00 AM, Cris Williams <notifications<EMAIL_ADDRESS>wrote:
I've been wrestling with this for a while. For the next release, I am going to try to refactor the sidebar so it doesn't use 'position:fixed' (which is how it currently always stays visible even when you scroll down)
For today, I can either do what I'm doing now, which is allow room at the bottom so it doesn't cover the footer - which doesn't scale when you change your scrolling, hence the gap - or let the sidebar overlay the footer. Another option is actually getting rid of the footer altogether.

Reply to this email directly or view it on GitHubhttps://github.com/informatics-isi-edu/chaise/issues/123#issuecomment-147460809.

@crisaless as this been fixed?

this is a facebase-only issue because of that footer. My recent commit fixed this so go ahead and review on dev.facebase.org/data/search/ (you could only see the sidebar glitch once there were search results that push the footer down)


Make client logging configurable

Is your feature request related to a problem? Please describe.
The Python and Go client libraries log - a lot.  It's annoying especially if you're trying to use a form of structured logging, or just debug something you know is unrelated to Pulsar.
Describe the solution you'd like
At bare minimum, there should be a way to just turn it off.
A more complete solution would allow configuring the logger in a way idiomatic to each language.  For example, integrating with Python logging.  I'm not 100% sure how.
Another option would be to allow redirecting the logs to a file instead of polluting stderr.
Describe alternatives you've considered
Not sure if there's a good alternative.

@grantwwu In Go it's already possible to configure. Take a look at the Logger in ClientOptions:
https://godoc.org/github.com/apache/pulsar/pulsar-client-go/pulsar#ClientOptions
We should do something similar for Python as well and probably move the logger to Py logging by default.

C++ lib has a setLogger() method that we'd need to hook up in the Python bindings: https://pulsar.apache.org/api/cpp/classpulsar_1_1_client_configuration.html#aa0dc0968fea7621352aae5a82bda7222

Good to know for Go.  Definitely would love this in Python as well.

The logging module in the Python standard library is implemented in Python. It is not a built-in implemented in C.
I think that we could implement a log4cxx appender that uses boost::python::exec to call methods on a Python standard library Logger instance, with arguments passed as locals.

The client logging in python still seems non-configurable even after providing the log4cxx config file. Is it possible to configure to stop logging to stdout ?

It seems maybe this got held up due to the potential difficulty of sending log data from C++ up to python.  How about just providing options to silence logging, and/or set the log level in the meantime?


PATCH support for Spring in NEBA

Hi,
I've noticed that Neba extends SlingAllMethodsServlet which does not support PATCH method for REST API calls Any way to enabled it in NEBA ?
I'm delivering a fully REST API over Sling and therefore I need this method.
Tnks in advance,
Regards,
Ivo Leitão

Hi @ivoleitao
Absolutely - I'll open up a ticket and include it i the upcomign 5.0 release!
Kind regards,
Olaf

Hi Olaf,
Tnks a lot,
Regards,
Ivo Leitão

Implemented, done. Will be release with 5.0 shortl!


Default heapster settings when not provided

Fixes #615
When using an old plan file that is missing the add_ons section the validation should not fail and instead use default values.

LGTM


Clean up and document angular bindings

The current angular binding is messy and undocumented:

[x] When the library is used in angular we need to clean up the global ERMrest library.
[x] For the revised API sketch we do not need the factory, we just need ERMrest to be the factory.
[x] We need documentation. Ideally this can be generated from jsdocs but it may not be possible and may have to be manually written.

Merged the PR #107. Closing this issue.


[CPU][SVE] Enable scalable vectorization and tiling for non-padded matmuls

This patch implements a "vertical slice" that allows for the scalable vectorization of matmuls on AArch64 targets with SVE. This only updates lowerings along that path, so more is needed to enable scalable vectorization everywhere.
This required a few changes:

The default vector sizes of matmuls on AArch64+SVE are now (8, [32], 16)

That is a middle scalable dimension (i.e. 32 x vscale)

iree-llvmcpu-tile-and-fuse now generates vscale bounded loops for scalable tiles
TilingConfig now returns a pair of tile sizes and scalable flags (SizesAndScalableFlags) for vector sizes

This allows connecting the scalable sizes to the generic vectorizer (which passes them down to the linalg vectorizer)

A few unit tests have been added for this, but a complete e2e test is not possible without SVE testing infrastructure.

Looking good. I think if we switch to use bit vectors, this is landable now. Thanks!

Looking good. I think if we switch to use bit vectors, this is landable now. Thanks!

I'm still not sure BitVectors would be better here.
We would need to convert the BitVectors back into SmallVectors anyway, the vectors are also going to be pretty small (probably less than 10 elements). So if you compare the memory usage of a BitVector vs a SmallVector, the SmallVector appears to win. A SmallVector<bool> is 64 bytes and stores 38 elements inline, a BitVector is 72 bytes (as it's just a wrapper of a SmallVector<uint64_t> + some extra fields), so unless the vectors are quite large (and go onto the heap), I don't see any advantage to using BitVectors.
See: https://godbolt.org/z/ajnaaq36Y

Looking good. I think if we switch to use bit vectors, this is landable now. Thanks!

I'm still not sure BitVectors would be better here.
We would need to convert the BitVectors back into SmallVectors anyway, the vectors are also going to be pretty small (probably less than 10 elements). So if you compare the memory usage of a BitVector vs a SmallVector, the SmallVector appears to win. A SmallVector<bool> is 64 bytes and stores 38 elements inline, a BitVector is 72 bytes (as it's just a wrapper of a SmallVector<uint64_t> + some extra fields), so unless the vectors are quite large (and go onto the heap), I don't see any advantage to using BitVectors.
See: https://godbolt.org/z/ajnaaq36Y

Thanks for the analysis. I am not sure I am talking about necessarily byte level savings. Its more about norm. Having bitvector seems more canonical way of representing bit data structures?

Within the Vector dialect/MLIR scalability is (for better or worse) represented with array of bools, and it what the llinalg vectorizer expects. So using something different here, and then converting between the two makes things a little trickier


[backport] Refactor random tests

Backport of #4907

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 84dbbf02dbe9a5bbc2c2c6b1571801dd77fa9bd1, target branch v9) succeeded!


More API retiring

Targets some items from #804 and #716

It's currently failing because #880 is needed. Will rebase in a few minutes once it's merged


Cannot commit

Running Gittyup 1.1.0 under Mint 20.
My local git repos are on a Windows share running on OpenmediaVault 5.6.26
I can clone repos from that PC but I cannot commit back to it.
When I try, I get the error :
" Unable to push to 'origin' - failed to create temporary file '/run/flatpak/doc/553105de/wpSk.git/objects/pack/pack_git2_tfL4qe': Permission denied"
The git repo directory is shared via CIFS

Hi @tester0077, the flatpak does not allow to edit files outside of the home folder. But I think there is another problem with your configuration, because normally you should not need more than the home folder.
I am not sure If I understood correctly:

"origin" is on the OpenmediaVault OS?
You are cloning to your local pc (into the home folder or any subfolder of it)?
Are you able to commit, but not able to push?

Can you show me how your remote URL looks like? Gittyup -> Remote -> Configure Remotes

The URL looks like  /run/user/1000/doc/493432c5/wpDesk.git
After more work to change permissions on my repo, I get the error:      Unable to push to 'origin' - unsupported URL protocol
I can push  from Smartgit though, so I am not stuck

@tester0077 I can confirm your issue.
The problem is that you are cloning from a local repository where as the flatpak package is not allowed to access

see solution in #215


Define a default color for a Target if the color is empty

Define a default color for a Target if the color is empty

Working as expected


Add sensor platform for SolarEdge Monitoring API

Description:
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#18846
Example entry for configuration.yaml (if applicable):
sensor:
  - platform: solaredge
    api_key: YOUR_API_KEY
    site_id: YOUR_SITE_ID
    monitored_variables:
      - currentPower
      - lastDayData
      - lastMonthData
      - lastYearData

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Hello @GidoHakvoort,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

df98c0a73abf4618e921a6c84296e274e596147b - This commit has something that looks like an email address (gido@hakvoort.it). Maybe try linking that to GitHub?.
12e3d43cb3c91d2843ebe8231c64e9922ffebbe1 - This commit has something that looks like an email address (gido@hakvoort.it). Maybe try linking that to GitHub?.
b74cde316590f9528ae66474ad7dbfc6e14c21fd - This commit has something that looks like an email address (gido@hakvoort.it). Maybe try linking that to GitHub?.
4033255dd57de15728df6ad1f234cf5f32dd741c - This commit has something that looks like an email address (gido@hakvoort.it). Maybe try linking that to GitHub?.
3b4b13d1d492aa0aa6c7f1128166739bc8638c2e - This commit has something that looks like an email address (gido@hakvoort.it). Maybe try linking that to GitHub?.
85f9f36976e47fe366b72a0cfda9122952b48a6e - This commit has something that looks like an email address (gido@hakvoort.it). Maybe try linking that to GitHub?.
63d07d1b19ad953e3d2cbf30166611638b0c1ebc - This commit has something that looks like an email address (gido@hakvoort.it). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

@pvizeli Thanks, should be fixed

Hi @Mariusthvdb,
Thanks, nice suggestion, I will look into it. For the site details I could add an additional sensor which will be updated once a day (SolarEdge has a daily query limit of 300 requests). Details such as site id, name, status, peak power, etc. could then be added as attributes to a sensor (e.g. solaredge_site_details). For the inventory, I might be able to separate this per equipment type (e.g. meters, sensors, gateways, etc). This might however be a bit more difficult as these are lists. If you have any suggestions, let me know.


Task 1, Task 14, Task 15

{
"NoFile":  [
{
"Your organization requires your project to have an Automation Hub URL defined.":  "Use "Project Settings" to add the missing information. "
},
{
"Dependency package UiPath.Excel.Activities is not used.":  "Remove unused packages in order to improve process execution time. "
},
{
"Dependency package UiPath.Mail.Activities is not used.":  "Remove unused packages in order to improve process execution time. "
}
]
}

{
"NoFile":  [
{
"Your organization requires your project to have an Automation Hub URL defined.":  "Use "Project Settings" to add the missing information. "
},
{
"Dependency package UiPath.Mail.Activities is not used.":  "Remove unused packages in order to improve process execution time. "
}
],
"Main.xaml":  [
{
"Variable ExtractDataTable doesn\u0027t have DataTable prefix.":  "DataTable Variable should start with a dt_ prefix. "
},
{
"Variable Result doesn\u0027t have DataTable prefix.":  "DataTable Variable should start with a dt_ prefix. "
}
]
}

{
"NoFile":  [
{
"Your organization requires your project to have an Automation Hub URL defined.":  "Use "Project Settings" to add the missing information. "
},
{
"Dependency package UiPath.Mail.Activities is not used.":  "Remove unused packages in order to improve process execution time. "
}
],
"Main.xaml":  [
{
"WriteLine activity usage at Checking The FullName Value By Write Line Activity.":  "Write Line activity can be replaced with tracepoints during debug. They should be removed before publishing. Consider also changing WriteLine to Log Message if it is really needed for debugging purposes. "
},
{
"Variable Data doesn\u0027t have DataTable prefix.":  "DataTable Variable should start with a dt_ prefix. "
},
{
"Activity Log Message has a default name.":  "Check if it could be easier to understand what the activity does by adding a more descriptive title. "
}
]
}

@venkatvtu5419
Task -1
Code verified.
Task -14
Code verified.
Task -15
Code verified.
Provide Input file on necessary Tasks along with code


Merlin-destruct can't handle some GADT types

See https://github.com/janestreet/merlin-jst/pull/36 for the test demonstrating as such, and https://github.com/janestreet/merlin-jst/pull/38 for the sad workaround.

I could not reproduce the issue upstream. (So, while an analogous code issue could well exist upstream, it's not in a state where we can make a GitHub issue.)


Allow post-processing to append new layers.

Fix bug where post-processing would only replace existing layers, not add new ones, which we need for mapzen/vector-datasource#140.
@rmarianski could you review, please?

:+1:

Thanks!


Flushing to network

I am using the analytics plugin.
I can't work out how to control the flushing to a server. Should I do it here or in the analytics plugin.
My plane it to send the data to a golang server and index it so it's possible to track usage patterns.

@joeblew99 you can do it all under flutter_analytics, this one is just for the local buffering - when it fills up (or any other trigger condition) it's in charge of calling the callback received from analytics and ship the local batched data to a remote destination


Improve shutdown/load-balancer integration

At present, when a SIGTERM is received the following happens:

The HTTP stops processing new requests. (Any new requests get a 503 response.)
Existing HTTP requests that are underway are allowed to complete.
Once in-flight requests have completed, the HTTP server stops accepting new connections entirely.
Events in the mapping/sink pipeline are flushed.
The whole server stops.

This causes problems with some load-balancers, who continue to forward traffic to a node until the health check takes the node out of rotation. In addition, load-balancers don't always retry failed requests. This means that requests between the initial SIGTERM and the health check marking the node as unhealthy will fail: the request can be forwarded to a node that has shut itself down, yielding an error that will be propagated to the client.
In order to help with deployment in these situations, we plan to introduce a new feature allowing for delayed shutdown. This will consist of:

A new configuration item: global.server.delay_shutdown that defaults to 0 ms.
On SIGTERM:

Mark the /ping handler to start returning a 503 status code, and the body should change to something else that doesn't contain pong. (I kind of like no p*ng for you: ${DELAY_TIMEOUT}.)
Start a timer for the global.server.delay_shutdown duration.

When the timer lapses, start the current shutdown process.

This allows users to configure the delay interval such that requests continue to be served until their health check mechanism stops forwarding requests to the node.

Kubernetes is one of the things we're aiming to help with. In particular when stopping a pod it continues to deliver traffic to it between the SIGTERM and the next readiness probe. During this interval we respond with 503 (or don't accept the connection) and that hits the client.
The situation with HAProxy is the similar out-of-the-box: there's an option to have it retry requests that fail but it's not on by default.
The goal here is to help with any load-balancer that supports a readiness check, even if they don't have any extra knowledge about node availability.
Getting back to Kubernetes' liveness and readiness checks, I think as things currently stand the /ping endpoint is suitable for a readiness check and would remain so under this proposal but behave better. For liveness any of the javascript endpoints are better, and are unaffected by this proposal.
Have I missed something?

Would it then make sense to initiate the shutdown sequence only after the first invocation of readiness after SIGTERM has been responded with a 503.

That would work for Kubernetes, OpenShift and ELB (I think), although not out of the box: they default to 3 failing probes before marking it unhealthy.
It would work for HAProxy if there's a single server and it's configured to mark it as unhealthy after a single failing probe. In a HA setup each server does its own health check, so it wouldn't work there. It's also normal to have a few failing probes before you mark a node unhealthy.
We could make the thing ultra-configurable, but it seems rather brittle when all cases can be covered with a single "fail /ping for x seconds before really shutting down" setting.
…time passes…
I've been doing some reading, and I'm also not 100%-convinced that the interwebs understand the problem correctly with Kubernetes. Specifically, its documentation states that when terminating a pod the following happen in parallel:

Processes get a TERM signal.

Pod is removed from endpoints list for service, and are no longer considered part of the set of running pods for replication controllers. Pods that shutdown slowly can continue to serve traffic as load balancers (like the service proxy) remove them from their rotations.

I would assume this is racy, and there's at least one report that agrees. In particular I don't think the readiness probe is relevant here, except in the sense that it could also cause a pod to be marked as unavailable.
With this in mind I think the original proposal still stands. Right now though I'm going to sleep on it…

I took this as a reference: https://github.com/RisingStack/kubernetes-graceful-shutdown-example
The idea is to verify the graceful shutdown by performing a test. By firing a predefined set amount of requests and check if they all arrive properly. I'll let you guys know what the impact of the change will be.

@Fokko So I'm wondering if that reference describes Kubernetes as it actually works. Specifically, its description contradicts the Kubernetes documentation.

Good point, right now we're still on Kubernetes 1.7, so it might even change in 1.8 or 1.9


Assign a reserved range of mac addresses for UCI devices

Assign a reserved range of mac addresses for UCI devices to protect against conflicts with user defined Anchor addresses.

The mac address for ranging is defined by the APP config, UCI devices do not have an assigned mac address by default.


Prod deploy e2e needs to take place in different sub

Currently it tries to use the int sub, which doesn't work for obvious reasons.
Better would be for it to use a prod sub.  Best would be for it to detect the "InProgress" case too.

@JackQuincy do you have space for this?

I can take this up. FYI @amanohar

I actually separated the subs yesterday afternoon I'll just validate that is working correctly

prod e2e sub didn't have needed feature flags. I just added them. The e2es should work now


[BugFix] simplify expr hashCode calculating (backport #10954)

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Cherry-pick of https://github.com/StarRocks/starrocks/commit/601029cf37662b1e269a64c685a7d4f7848146a3
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_fe_unittest


Allow to define image input parameters with predefined values for selection as dropdown in "Run..."

When the possible values for the input parameter are known in advance, users would benefit from a dropdown (with a single choice).
At the moment, the popup doc string contains the hint for the possible values,  but it's not possible to copy from the popup because it disappears when hovering over it.  UPDATE: The latter case is addressed in slipstream/SlipStreamUI#334

"Top-level" issue: slipstream/SlipStream#83