Jackson JSON string escaping causing issues with OPA server V1 data api

I am using Jackson JSON output to submit to an OPA server with the following SDK code:
public static void validate(String json, String policyPath) throws Exception {
        OpaApiClient sdk = OpaApiClient.builder().build();

        ExecutePolicyWithInputRequest req = ExecutePolicyWithInputRequest.builder()
                .path(policyPath)
                .requestBody(ExecutePolicyWithInputRequestBody.builder()
                        .input(Input.of(json))
                        .build())
                .build();

        ExecutePolicyWithInputResponse res = sdk.executePolicyWithInput()
                .request(req)
                .call();

        if (res.successfulPolicyResponse().isPresent()) {
            System.out.println(res);
        }
    }

If I save the Jackson JSON output to a JSON file, wrap it with an input object, and then POST it to the OPA server with curl, then there are no issues. The curl post looks like so:
curl "localhost:8181/v1/data/devnexus/java/imports/validation/deny" \
	--data-binary @parsed-java-input.json \
	| jq .

Input object:
"req_body":"{\n  \"input\": {\n    \"className\": \"Customer\",\n    \"packageName\": \"org.jimmyray.mongo.data.model\",\n    \"access\": \"PUBLIC\",\n    \"annotations\": [\n      \"Document\"\n    ],\n    \"imports\": [\n      \"org.jimmyray.mongo.framework.Strings\",\n      \"org.springframework.data.mongodb.core.index.Indexed\",\n      \"org.springframework.data.mongodb.core.mapping.DBRef\",\n      \"org.springframework.data.mongodb.core.mapping.Document\",\n      \"io.jimmyray.thisimportisbad\"\n    ],\n ...

The result is:
"resp_body":"{\"result\":[{\"imports\":{\"import\":\"io.jimmyray.thisimportisbad\",\"msg\":\"Class contains bad import.\",\"severity\":\"high\"}}]}\n"

However, if I take the same JSON string and pass it to the input.of() method, there are additional escape back-slashes added to the JSON, and the OPA policy then returns an empty result, as if the policy cannot find the imports elements. The policy is listed below.
"req_body":"{\"input\":\"{\\n  \\\"className\\\" : \\\"Customer\\\",\\n  \\\"packageName\\\" : \\\"org.jimmyray.mongo.data.model\\\",\\n  \\\"access\\\" : \\\"PUBLIC\\\",\\n  \\\"annotations\\\" : [ \\\"Document\\\" ],\\n  \\\"imports\\\" : [ \\\"org.jimmyray.mongo.framework.Strings\\\", \\\"org.springframework.data.mongodb.core.index.Indexed\\\", \\\"org.springframework.data.mongodb.core.mapping.DBRef\\\", \\\"org.springframework.data.mongodb.core.mapping.Document\\\", \\\"io.jimmyray.thisimportisbad\\\" ],\\n ...

The result is empty:
"resp_body":"{}\n"

OPA Policy:
package devnexus.java.imports.validation

import rego.v1

known_bad_imports := {"io.jimmyray.thisimportisbad"}

deny contains result if {
	some i in input.imports
	some kbi in known_bad_imports
	contains(lower(i), kbi)

	result := {"imports": {
		"severity": "high",
		"msg": "Class contains bad import.",
		"import": i,
	}}
}

Is there a way to post the Jackson JSON output string, mapper.writeValueAsString(output);, without the extra escapes added? Perhaps a --data-binary type setting for the OPA SDK input.of() method?

So, I tried to use a Map<String, Object> instead of a string with:
Map<String, Object> map = mapper.convertValue(output, new TypeReference<Map<String, Object>>() {});

Then passing the map object to the input.of() method:
ExecutePolicyWithInputRequest req = ExecutePolicyWithInputRequest.builder()
                .path(policyPath)
                .requestBody(ExecutePolicyWithInputRequestBody.builder()
                        .input(Input.of(map))
                        .build())
                .build();

The input looks better:
"req_body":"{\"input\":{\"className\":\"Customer\",\"packageName\":\"org.jimmyray.mongo.data.model\",\"access\":\"PUBLIC\",\"annotations\":[\"Document\"],\"imports\":[\"org.jimmyray.mongo.framework.Strings\",\"org.springframework.data.mongodb.core.index.Indexed\",\"org.springframework.data.mongodb.core.mapping.DBRef\",\"org.springframework.data.mongodb.core.mapping.Document\",\"io.jimmyray.thisimportisbad\"],

However, there is still something wrong:
"resp_body":"{}\n"

Hi! I haven't checked this from a computer yet, but this line isn't correct:
contains(lower(i), kbi)

Try using in,
lower(i) in kbi

Thanks, @srenatus! Regal didn't pick that issue up. I changed the policy like so:
package devnexus.java.imports.validation

import rego.v1

known_bad_imports := {"io.jimmyray.thisimportisbad", "io.jimmyray.thisimportisworse"}

deny contains result if {
	some i in input.imports
	lower(i) in known_bad_imports

	result := {"imports": {
		"severity": "high",
		"msg": "Class contains bad import.",
		"import": i,
	}}
}

It still worked fine in opa exec and posting to the OPA server with curl:
{
  "result": [
    {
      "path": "parsed-java.json",
      "result": [
        {
          "imports": {
            "import": "io.jimmyray.thisimportisbad",
            "msg": "Class contains bad import.",
            "severity": "high"
          }
        },
        {
          "imports": {
            "import": "io.jimmyray.thisimportisworse",
            "msg": "Class contains bad import.",
            "severity": "high"
          }
        }
      ]
    }
  ]
}

However, the SDK calls are still returning an empty result:
"resp_body":"{}\n"

I am new to the OPA Java SDK, so I am most likely doing something wrong. However, the SDK is sending to the same OPA server and policy bundle where the curl POST works just fine. Thanks for your help!

Can you share the debug logs of the opa server? Something must be off, maybe the path...

Yep, I think you are on to something. When I post via curl, I see this:
"req_method":"POST","req_path":"/v1/data/devnexus/java/imports/validation/deny"

However, when I use the SDK, I see:
"req_method":"POST","req_params":{},"req_path":"/v1/data/devnexus%2Fjava%2Fimports%2Fvalidation%2Fdeny",

Note the %2F hex encoding in the policy path. Does that make any difference?
The call to the Java program is:
java -jar ~/.m2/repository/io/jimmyray/parser/1.0-SNAPSHOT/parser-1.0-SNAPSHOT-shaded.jar \
    -f ~/code/Customer.java -p devnexus/java/imports/validation/deny -v -d

The policy I pass to the OPA SDK call is (provided by debug statements):
Policy-query path = devnexus/java/imports/validation/deny

ExecutePolicyWithInputRequest req = ExecutePolicyWithInputRequest.builder()
                .path(policyPath)
                .requestBody(ExecutePolicyWithInputRequestBody.builder()
                        .input(Input.of(map))
                        .build())
                .build();

It looks like the issue, though I don't know where the hex encoding is happening. This curl returns the same empty result when the policy path is hex encoded:
curl "localhost:8181/v1/data/devnexus%2Fjava%2Fimports%2Fvalidation%2Fdeny" \
	--data-binary @parsed-java-input.json \
	| jq .
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 10508  100     3  100 10505    918  3142k --:--:-- --:--:-- --:--:-- 3420k
{}

Note the %2F hex encoding in the policy path. Does that make any difference?

Most definitely the problem. We've had extra hooks in the other sdks, and I thought we had taken the same steps here, but it looks like that's not the case. I'll check the code more thoroughly tomorrow.
In the mean time, you could set up a default decision in your OPA server and not use a path in the SDK? That could work...

So checking the code real quick, we have a test indicating that a call to the policy policy/hello works.
However, that test is using our high-level porcelain API. Could you try that? Something along these lines should do it:
        OPAClient opa = new OPAClient(address, headers);

        try {
            result = opa.evaluate("devnexus/java/imports/validation/deny", map);
        } catch (OPAException e) {
            System.out.println("exception: " + e);
            assertNull(e);
        }

@jimmyraywv Did this unblock you / solve your issue if you use the high-level API? If so, I might close this issue or re-title it to make it clear that it's only when using the low-level API.


Title strings with enclosing # will truncate permalinks

I found that using a title in my permalinks would truncate the link created if the title contained the # symbol - the url parser deems it to be valid as expected being a fragment. Therefore I've proposed this fix to remove the value on pageToPermalinkTitle() call.
I'm not sure how far up the chain this should go - eg are there .Site.Permalink values I should be concerned with also? I'm not sure but altering this method fixes the issue so far as I can tell.
Also as a seperate issue, you can create new files with # values in the filename. I think they should also be stripped. I can create this PR also if required.
My setup
# config.toml
[permalinks]
    test    = "/:title/"

Then alter any frontmatter content file to have a # in the title
+++
categories = []
date = "2016-08-29T07:50:07+03:00"
draft = true
tags = []
title = "a new #1 file post that stinks like #2s"
+++

Then ensure that you are spitting out the permalink
{{ .Title | markdownify }}

Before with url truncation.

After - no truncation on href and a happy url =]

go test -run TestPermalink -v shows permalink tests are aok.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Kyle Clarke seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.

NOTE: I have already submitted my CLA - where the Kyle Clarke seems not to be a GitHub user is coming from is anybody's guess. How else did I create this PR without being a Github user?

GitHub uses email addresses to identify users. It seems you committed with an email address that is unknown to Github. Just add this to the emails in the settings and confirm it. This way, GitHub and the CLA bot can associate the commit with you.
This solved the problem for previous pull request submitters.

Thanks @digitalcraftsman - I have moved to Gitlab now so that address was my default and assigned in the PR. I have verified now =]

Ok this PR needs more work, I managed to break permalinks again by having a ' in the title of my blog. So all non url chars AND the # should be stripped.
I also think I found another bug where using a number with Taxonomy failed to compile with
[taxonomies]
    tag = "tags"
    category = "categories"
    series = "series"

  {{ range .Site.Taxonomies.series.29days }} <-- Goes kaboom due to 29

{{ .Name }}

  {{ end }}

I will confirm and lodge/PR.

@kylehq I think that you can create a new function to strip special characters, and then remove # and any other character that may break the links.
I don't know if it's related, but if I create a post titled "Hey Mr. John Doe", the {{ .Permalink }} contains the dot, but, of course, I get a 404 when I click it. But removing it manually from the url at the address bar gets me to the correct post.

@kylehq,
I believe your issue with .29days is a characteristic of Go templates.  You need to do something like this:
{{ range (index .Site.Taxonomies.series "29days") }}

@thiagomgd that's exactly related, I managed to break my posts again by describing America as  `Murica =] I will create (or confirm a method doesn't already exist) to do just that.

@moorereason I didn't know that thanks - but I suggest we change the docs to reflect this https://github.com/spf13/hugo/blob/master/docs/content/taxonomies/displaying.md#2-listing-content-with-the-same-taxonomy-term
I will create another PR.

I'm going to have to park this for a bit - first first thoughts were to create a regexp to remove any chars & html entities that are not url safe. Excluding spaces in order to get a text representation of a title value that could be urlize'd.
But then good ol unicode comes along and title: ラーメン hmmmm
A bit blocked on how best to tackle this 😒

Ok so CR is required. But I learned that you can do unicode regexp's today - awesome!

@kylehq,
Can you squash and resolve merge conflicts?


impossibility to pick hero if you passed the pick time.

If you pass the picking time you cant pick the builder when the game starts becouse its impossible now (after patch 7.00).
Fix it please

people now random after picking time


Resource Page

Issue #378
Iteration 1:

[x] Themes from current iteration will be added (footer/navbar)
[x] Text from Figma will be added
[x] Links Will be added using Flexbox

Images also need to be added

Iteration 2:

[ ] Navbar will be reflected as in the design
[ ] Menu will be reflected as in the design
[ ] Styling will better reflect Figma Design

Changes completed, but to note this PR isn't for styling!!!

@discombobulateme


New commands

Opening this to begin reviewing the changes and make sure that enough tests are written.

cbascom Told me that he was not sure about the "avoid blocking read" commit, and why there was not a change to the tests at the same time.  But, since it was years ago, the memories have faded.

The code in the "avoid blocking read" looks sane - it prevents the destructor from trying to read the reply (again). This is similar to why the "tmp = conn; conn = NULL; tmp->readXXXReply();" pattern is there.
This repo could really use some clang-format love :)


Rotation tool in 2D Editor inverts its direction.

Godot version:
cab97f4b504a6
OS/device including version:
Win10 64-bit
Issue description:
When setting negative scaling in a node, it will influence how the rotation dragging works in its children.
Making the tool rotate in an inverted direction.  I understand it's from the transform inheritance, but for artists It's very unintuitive, especially since the parent node isn't affected by this change.
The tool should apply a global rotation.

Steps to reproduce:
Create a hierarchy of 2D nodes, flip the scaling of the parent, and try using the editor rotation to rotate it's children.
Minimal reproduction project:
Rotation Issue.zip

In the 3D editor, there is a button that's used to toggle "local mode." I think it would make most sense to add this button to the 2D editor as well.

I can look into this one. I'll try the solution I mentioned above.

Thanks, would be much appreciated.

Is this because scale -1 flips the node and its children around the corresponding axis?  If so, I think the fix for this would be to invert the rotation tool's applied rotation by -1 if an odd number of scale values for the parents of the node being manipulated are negative.

There is no need to implement a local mode in 2D. It simply requires the global transform to be checked, and revert the rotation direction if the transform flips the object.

@groud You're right. After giving it some more thought, making a local transformation mode would imply a global transformation mode as well. I don't think that's accurate to how the tools work. I think I'll implement your solution, as it sounds like the simplest way to fix this issue.


[WIP][Fix] rubocop is now working

I thought we would create a new command ruby_rubocop for this, no?

@yugui @kigster deprecated this for https://github.com/bazelruby/rules_ruby/pull/38


fix(client): remove url parameters from forum href

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Prevents a situation like this: https://forum.freecodecamp.org/t/problem-with-the-fcc/484974
Where I assume the behaviour comes about from:

Signing-in to freeCodeCamp
NOT dismissing the flash
Clicking any Create a Forum Post button

I see a commit that adds tests for this, but the tests don't seem to actually be in this final PR. 😕

I see a commit that adds tests for this, but the tests don't seem to actually be in this final PR. 😕

I removed the tests, because I removed the function parsing the output.
I suppose there could be tests for this, but, as I mentioned to Oliver, I do not like mocking things, and I think it would be overkill to e2e this.

If you want to test and not mock, you can just create a function and pass window as an argument. I.e. something that returns all the text.
Up to you, it's not a critical component.


Devise Master returns 200 on user create but Devise 3.5.2 returns 201

I have a Rails-based API. For a previous project (Rails 4.2.2, Ruby 2.2.2, Devise 3.5.2), I normally get 201 with the full User resource.
SQL (1.2ms)  INSERT INTO "users" ("email", "encrypted_password", "first_name", "last_name", "ui_language", "original_language", "target_language", "open_id", "created_at", "updated_at", "confirmation_token", "confirmation_sent_at") VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)  [["email"<EMAIL_ADDRESS>["encrypted_password", "$2a$10$Fg1TOz97G1LCRlBx1yLhLeXHs2K.ZTgn/k9wi4okkHoZTQotBKbOa"], ["first_name", "anh"], ["last_name", "nguyen"], ["ui_language", "hi"], ["original_language", "hi"], ["target_language", "hi"], ["open_id", "-2ueX7V2hQQX51s12zqT"], ["created_at", "2016-02-10 13:14:12.848088"], ["updated_at", "2016-02-10 13:14:12.848088"], ["confirmation_token", "T6K-MVx2Hx_SxtPAp-P6"], ["confirmation_sent_at", "2016-02-10 13:14:13.081561"]]
  Rendered devise/mailer/confirmation_instructions.html.erb (4.8ms)

Devise::Mailer#confirmation_instructions: processed outbound mail in 280.1ms

Sent mail to<EMAIL_ADDRESS>(19.6ms)
Date: Wed, 10 Feb 2016 20:14:13 +0700
From<EMAIL_ADDRESS>Reply-To<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>Subject: Confirmation instructions
Mime-Version: 1.0
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

Welcome test15@gmail.com!

You can confirm your account email through the link below:

Confirm my account

   (0.8ms)  commit transaction
   (0.1ms)  begin transaction
  User Exists (0.1ms)  SELECT  1 AS one FROM "users" WHERE "users"."open_id" = ? LIMIT 1  [["open_id", "dJFaZGxMv8jrRyvWczth"]]
  SQL (0.4ms)  UPDATE "users" SET "last_sign_in_at" = ?, "current_sign_in_at" = ?, "last_sign_in_ip" = ?, "current_sign_in_ip" = ?, "sign_in_count" = ?, "open_id" = ?, "updated_at" = ? WHERE "users"."id" = ?  [["last_sign_in_at", "2016-02-10 13:14:13.400264"], ["current_sign_in_at", "2016-02-10 13:14:13.400264"], ["last_sign_in_ip", "<IP_ADDRESS>"], ["current_sign_in_ip", "<IP_ADDRESS>"], ["sign_in_count", 1], ["open_id", "dJFaZGxMv8jrRyvWczth"], ["updated_at", "2016-02-10 13:14:13.402382"], ["id", 1]]
   (0.8ms)  commit transaction
Completed 201 Created in 686ms (Views: 7.5ms | ActiveRecord: 5.2ms)

However on the latest project (Rails 5 beta 2, Ruby 2.3.0, Devise master), I get 200 with blank response.
SQL (0.9ms)  INSERT INTO "users" ("email", "encrypted_password", "confirmation_token", "confirmation_sent_at", "created_at", "updated_at", "first_name", "last_name", "open_id") VALUES ($1, $2, $3, $4, $5, $6, $7, $8, $9) RETURNING "id"  [["email"<EMAIL_ADDRESS>["encrypted_password", "$2a$10$GZnQLSKhoWRRopUlI6OSy.7glelY6qk.D6x5WruuHx53hOTZR4mVC"], ["confirmation_token", "Zeb-SuiTU3mekzusipuh"], ["confirmation_sent_at", 2016-02-10 12:39:31 UTC], ["created_at", 2016-02-10 12:39:31 UTC], ["updated_at", 2016-02-10 12:39:31 UTC], ["first_name", "anh"], ["last_name", "nguyen"], ["open_id", "cW9AS-MTLx2w9zfycSL-"]]
  Rendered /Users/jackof1trade/.rvm/gems/ruby-2.3.0/bundler/gems/devise-7c75ebe672f1/app/views/devise/mailer/confirmation_instructions.html.erb (8.2ms)
Devise::Mailer#confirmation_instructions: processed outbound mail in 223.2ms
Sent mail to<EMAIL_ADDRESS>(13.7ms)
Date: Wed, 10 Feb 2016 19:39:31 +0700
From<EMAIL_ADDRESS>Reply-To<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>Subject: Confirmation instructions
Mime-Version: 1.0
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

Welcome test9@gmail.com!

You can confirm your account email through the link below:

Confirm my account

   (0.6ms)  COMMIT
Completed 200 OK in 619ms (Views: 0.2ms | ActiveRecord: 11.8ms)

Since a resource was created successfully the best practice is for 201 to be returned with at least the resource ID. Any idea why this might have changed?

I found some other inconsistencies in response code. I'm not sure if it's Devise or Rails 5

Can you please provide a sample application that reproduces the error?

@lucasmazza
Please try the failing RSpec tests in this sample (Rails 5 beta 2, Ruby 2.3.0, Devise 4.0.0RC1): https://github.com/adnguyen1989/rails_api_template/tree/user_management
A working Rails 4.2.2, Ruby 2.2.2, Devise 3.5.2 sample:
https://github.com/adnguyen1989/yomiapi

This is an incompatibility with the responders gem and ActionController::API, as the responders default rendering relies too much on the controller implementation.
There is a proposal fix at plataformatec/responders#146.


Add shift days for Doctors

Will involve the addition of command to adjust shift days

Will move the addition of command to v1.3.


Use junitparser instead of homegrown parser

We should probably just use junitparser instead of our internal homegrown solution for parsing j/xUnit XML. This may help with issues like #92

Looking into this one

After a review, we realised that the junitparser doesn't support custom attributes like "xpassed" or "xfailed"


Added gke-scheduled-autoscaler

Moved from github.com/GoogleCloudPlatform/gke-scheduled-autoscaler

Hi @rbarberop! Could you make sure that snippet names follow this hierarchy: $PRODUCT_$SAMPLE_$TAG, for example in your case something along those lines: gke_scheduled_autoscaler_hpa.
What is the end-goal of the github.com/GoogleCloudPlatform/gke-scheduled-autoscaler repo? Is it going to be archived in favour of this / is the tutorial going to point here instead?

Thanks for pointing that out @bourgeoisor
The original repo will have just a README with the new address and the tutorial will point at this one in the end too.

Could you add the following in a new file called .github/workflows/gke-scheduled-autoscaler-ci.yml:
name: gke-scheduled-autoscaler-ci
on:
  push:
    branches:
      - main
    paths:
      - '.github/workflows/gke-scheduled-autoscaler-ci.yml'
      - 'gke-scheduled-autoscaler/**'
  pull_request:
    paths:
      - '.github/workflows/gke-scheduled-autoscaler-ci.yml'
      - 'gke-scheduled-autoscaler/**'
jobs:
  job:
    runs-on: ubuntu-20.04
    steps:
      - uses: actions/checkout@v3
      - name: build gke-scheduled-autoscaler container
        run: |
          cd gke-scheduled-autoscaler
          docker build --tag gke-scheduled-autoscaler .


Text only uses theme inside components

Checklist

[x ] Check other issues and make sure that it is not reported yet.

Versions and Environment
Elixir: 1.8.1-otp-21
Erlang: 21.2.2
Scenic: 0.10.0
OS: MacOS
Steps to reproduce
Graph.build(font_size: 22, font: :roboto_mono, theme: :light)
  |> text("System")
  |> button("System")

Expected Behavior
Text would default to a dark color for the light theme and be visible.
Actual Behavior
Text is not a component, it is a primitive, which means it ignores the theme and renders white by default. The text ends up invisible.
The text inside the button is handled as expected and visible.
Additional Comments
After some discussion with Emilio I believe this works as intended but I'd say it doesn't work the way you'd expect. I think following the theme is the work of the component, not the primitive. But the default way of putting arbitrary text on the screen is using the primitive. So we get no good default for the theming. Would it be reasonable for text to pick up the theme for default text color?
I think there is a potential for UI developer incredulity at why text just doesn't respect the theme setting. Do we want a component that simply wraps text and does the theming? What is that called? A Label? A ThemedText?

I see the point you are making. Good question on if it should be a change to the text primitive to pick up the default from them or a label control.
I'm leaning toward updating the primitive itself.

I took a brief look at it hoping I could throw a PR in here for that case. But I didn't quite follow where that piece of action would have to happen from my brief attempt.
If you could point me in the right direction I'd be happy to make a proof-of-concept we could consider.

This isn't so straight-forward. Text is just a primitive and thus doesn't have a natural place to add code telling to pick up the contents of a theme. Instead it just inherits whatever the current font styles are form above it in the graph tree.
Probably the best place is if a theme is set during the Graph.build(...) function, then extract and set the font styles. Assuming they are not already set explicitly by the dev.
Memory is fuzzy. May have already done this? Need to take a look.


ValueError: list.remove(x): x not in list

How can the issue be reproduced
Windows 10.
The custom config file is in
Sublime Text 3\Data\Packages\User\MarkdownTOC.sublime-settings

and has the following content

{
  "defaults": {
    "autolink": true,
    "bracket": "square",
    "levels": [1,2,3],
    "indent": "    ",
    "remove_image": false,
    "bullets": "*",
    "style": "ordered",
    "lowercase": "only_ascii",
    "autoanchor":true,
    "link_prefix": "ant-markdowntoc-",
    "bullets": ["-"],
    "lowercase": "only_ascii",
    "style": "unordered",
    "uri_encoding": true,
    "markdown_preview": "github"
  },
  "id_replacements": [
    {
      "pattern": "\\s+",
      "replacement": "-"
    },
    {
      "pattern"<EMAIL_ADDRESS>      "replacement": ""
    }
  ]
}

Try to save any markdown file with a TOC section
What was expected
It should update the TOC and the links alongside the headers
What actually occurred
Up to a few days ago it worked fine, however today I noticed that it doesn't update the TOC. The console shows:
Traceback (most recent call last):
  File "C:\R\ANT10_VERBATIM\Z_PROGRAMMI\Sublime Text 3\sublime_plugin.py", line 1088, in run_
    return self.run(edit)
  File "C:\R\ANT10_VERBATIM\Z_PROGRAMMI\Sublime Text 3\Data\Installed Packages\MarkdownTOC.sublime-package\markdowntoc/markdowntoc_update.py", line 6, in run
  File "C:\R\ANT10_VERBATIM\Z_PROGRAMMI\Sublime Text 3\Data\Installed Packages\MarkdownTOC.sublime-package\markdowntoc/markdowntoc_insert.py", line 85, in find_tag_and_insert
  File "C:\R\ANT10_VERBATIM\Z_PROGRAMMI\Sublime Text 3\Data\Installed Packages\MarkdownTOC.sublime-package\markdowntoc/markdowntoc_insert.py", line 62, in get_toc_open_tag
  File "C:\R\ANT10_VERBATIM\Z_PROGRAMMI\Sublime Text 3\Data\Installed Packages\MarkdownTOC.sublime-package\markdowntoc/base.py", line 23, in defaults
  File "C:\R\ANT10_VERBATIM\Z_PROGRAMMI\Sublime Text 3\Data\Installed Packages\MarkdownTOC.sublime-package\markdowntoc/base.py", line 13, in settings
ValueError: list.remove(x): x not in list

I know that the v 4. release notes suggest to delete the replacement_id, but in my case it did not change anything. I even deleted the whole content of my preferences.
What was the version of the involved component
I suppose it is version 4. I cannot be sure because the List Packages command is not working on my sublime now.

Sorry for late reply...
I tried to reproduce but couldn't. Does the problem still occur?
Of course you tried reinstalling, right?

Err..., ok, reinstalling worked.
By the way it didn't allow me to uninstall the plugin using "package control", because it showed the same error as when I tried to run the plugin: list.remove(x):...
However, the action of manually deleting the package folder, immediately triggered an automatic reinstallation by pacakageControl, and it started working fine!
Thanks!

HI, happy to hear that!
It seems to be something wrong outside of plugins...


geoJSON in geocoder

Is it possible to use geoJSON data for region layers, like this one https://github.com/codeforamerica/click_that_hood/blob/master/public/data/amsterdam.geojson ?

Hi @chrisvel
I'm not sure what you mean by using geoJSON data? If by that you mean use an API that returns geoJSON data format. Then yes it would be possible to use geocoder in such a way. Our library is easily extendable to include more APIs.
Sorry if I misunderstood you. If so could you describe your use case to us?
Thanks,
Thu

Closing due to inactivity.


Update to Clang v9.0.0

Clang v9.0.0 was just released and both libClang and the bindings should be updated to use it.

Most, but not all, of the pre-built binaries are up now (http://releases.llvm.org/download.html#9.0.0), so I can likely start on this soon.

The libclang packages have been uploaded. Will regenerate the bindings and have a PR up momentarily.

PR is up here https://github.com/microsoft/ClangSharp/pull/86


chore(avm): bump SRS to 1 << 20

#7575  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @fcarreiro and the rest of your teammates on  Graphite

Benchmark results
Metrics with a significant change:

protocol_circuit_simulation_time_in_ms (private-kernel-tail-to-public): 6,733 (+57%)
avm_simulation_time_ms (Token:mint_public): 92.8 (+62%)
avm_simulation_time_ms (Token:_increase_public_balance): 83.5 (+430%)
avm_simulation_time_ms (FPC:pay_refund_with_shielded_rebate): 111 (-37%)
app_circuit_proving_time_in_ms (AuthRegistry:set_authorized (avm)): 2,412 (+17%)
app_circuit_proving_time_in_ms (FPC:prepare_fee (avm)): 2,517 (+17%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Proof generation
Each column represents the number of threads used in proof generation.

Metric
1 threads
4 threads
16 threads
32 threads
64 threads

proof_construction_time_sha256_ms
5,723 (+1%)
1,542 (+1%)
710 (+1%)
810 (+8%)
890 (+15%)

proof_construction_time_sha256_30_ms
11,735
3,170 (+1%)
1,407
1,436
1,461

proof_construction_time_sha256_100_ms
47,550 (+9%)
12,090 (+1%)
5,411
5,534 (+3%)
5,358

proof_construction_time_poseidon_hash_ms
89.0 (+14%)
39.0 (+15%)
41.0 (+21%)
64.0 (+10%)
94.0 (+7%)

proof_construction_time_poseidon_hash_30_ms
1,530 (+1%)
416
200 (+1%)
222 (-2%)
269 (+1%)

proof_construction_time_poseidon_hash_100_ms
5,627 (+1%)
1,515
678
801 (+12%)
822 (+10%)

L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
4 txs
8 txs
16 txs

l1_rollup_calldata_size_in_bytes
708
708
708

l1_rollup_calldata_gas
6,576
6,586
6,600

l1_rollup_execution_gas
583,038
583,199
583,364

l2_block_processing_time_in_ms
764 (-1%)
1,447 (+3%)
2,685 (-1%)

l2_block_building_time_in_ms
13,116
25,688
50,748 (-1%)

l2_block_rollup_simulation_time_in_ms
13,116
25,688
50,748 (-1%)

l2_block_public_tx_process_time_in_ms
10,982 (-1%)
23,351
48,388 (-1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 8 txs.

Metric
3 blocks
5 blocks

node_history_sync_time_in_ms
7,197 (+2%)
10,003 (+1%)

node_database_size_in_bytes
12,443,728
16,441,424

pxe_database_size_in_bytes
16,254
26,813

Circuits stats
Stats on running time and I/O sizes collected for every kernel circuit run across all benchmarks.

Function
time_ms
bytecode_size_in_bytes

GasToken:_increase_public_balance
100
13,790

GasToken:set_portal
12.0 (-22%)
3,339

Token:constructor
123 (-2%)
23,692

FPC:constructor
94.3 (+2%)
13,592

GasToken:mint_public
82.8 (+2%)
10,158

Token:mint_public
:warning: 92.8 (+62%)
19,034

Token:assert_minter_and_mint
69.6 (+1%)
12,925

AuthRegistry:set_authorized
46.8 (+4%)
7,812

FPC:prepare_fee
102 (-7%)
11,068

Token:transfer_public
36.9 (+10%)
31,218

FPC:pay_refund
132 (-1%)
21,266

Benchmarking:increment_balance
1,301
11,281

Token:_increase_public_balance
:warning: 83.5 (+430%)
15,006

FPC:pay_refund_with_shielded_rebate
:warning: 111 (-37%)
22,361

Public DB Access
Time to access various public DBs.

Function
time_ms

get-nullifier-index
0.158 (+2%)

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
256 leaves
512 leaves
1024 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.5 (+1%)
17.5 (+5%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.7
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.606 (+1%)
0.539 (+5%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
49.9 (+4%)
75.6
131 (+1%)
248 (+1%)
468

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
95.9
159
287
543
1,055

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.510 (+4%)
0.465
0.449
0.449 (+1%)
0.436

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
61.5 (+3%)
111
182
363 (+3%)
690

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
109
207
355
691
1,363

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.518 (+3%)
0.499
0.483 (+1%)
0.491 (+3%)
0.474

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
73.9 (+2%)
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
133
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
0.526 (+2%)
N/A
N/A
N/A
N/A

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
77,362
668,542

Transaction size based on fee payment method
| Metric |  |
| - |  |


NETOBSERV-926: Fix kafka writer configuration ... [1.4 backport]

Backporting https://github.com/netobserv/netobserv-ebpf-agent/pull/195

/ok-to-test

/lgtm

/label qe-approved


Add inform_on decorator for python.

@lynncyrin OK now this is looking more like Real Python. Mind doing a sanity check for me around weirdness?

@zspencer thoughts

I don't have any specific suggestions about where it would be effective, but for a module like this I feel there should be strong integration with the logging module
I'm going to want to use @inform_on(level='info' as @inform_on.info( to mirror logger.info the calls I use currently
I'm not sure about a general python api for settings, but for good django integration you are definitely going to want FakeInformant to be defined with a settings value
putting the last two items together, I would ideally be using this module like

@inform_on.info
def do_something(self, force=False):

Python actually has a really good logging framework, it probably would be easier to use it rather than trying to re-invent it, and you'd get redirect to a file etc... for free:
import logging
import time

logging.basicConfig(format='==> %(module)s, %(funcName)s %(message)s', level=logging.ERROR)
# with fancy time
# logging.basicConfig(filename='/tmp/test.log', format='%(asctime)sZ pid:%(process)s module:%(module)s %(message)s', level=logging.ERROR)

logging.info('blah blah')
logging.debug('you won\'t see this')

.../...

if __name__ == '__main__':
    parser = argparse.ArgumentParser()
    parser.add_argument('--debug', '-d', action='store_true')

.../...

if args.debug:
    logging.getLogger().setLevel(logging.DEBUG)

Yep! The next steps I have are to actually hook into the logging library in the same (or similar) way the Ruby one does.
The goal here is to make it easy to instrument entrances and exits to a function with the surrounding context for debugging purposes and to populate event based analysis systems.
This way instead of people having to decide exactly what message to log they state they want to inform on a given method and include particular context and it sends that.
Next few commits will replace the fake informant with one that delegates to the standard logging library.
Sent from my iPhone

On Nov 27, 2017, at 7:23 AM, Yves Dorfsman<EMAIL_ADDRESS>wrote:
Python actually has a really good logging framework, it probably would be easier to use it rather than trying to re-invent it, and you'd get redirect to a file etc... for free:
import logging
import time
logging.basicConfig(format='==> %(module)s, %(funcName)s %(message)s', level=logging.ERROR)
with fancy time
logging.basicConfig(filename='/tmp/test.log', format='%(asctime)sZ pid:%(process)s module:%(module)s %(message)s', level=logging.ERROR)
logging.info('blah blah')
logging.debug('you won't see this')
.../...
if name == 'main':
parser = argparse.ArgumentParser()
parser.add_argument('--debug', '-d', action='store_true')
.../...
if args.debug:
logging.getLogger().setLevel(logging.DEBUG)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@lynncyrin  - Good point on inform_on.info I was initially attempting to mirror the design of the ruby DSL, but I could see an inform_on.info working there as well, and if it would feel more natural to you I think it's a good suggestion!

@lynncyrin + @dorfsmay - This now integrates tightly with the built in logging module. The example.py file demonstrates how to instrument a class.
I've split out a feature request for @inform_on.info instead of using @inform_on(level=...


Implement email blacklist

Currently the UI suggests<EMAIL_ADDRESS>Create a blacklist file that can be loaded and put this email address in it. Disallow email addresses from the blacklist as a destination for the output file. Also add<EMAIL_ADDRESS>to the blacklist.

Implemented as hard-coded in api.py.


New Feature: Also output current cluster

Just some thoughts. Wouldn't it be nice that debug also prints out the cluster.worker.id if it is running/called within a cluster (cluster.fork())?

https://github.com/visionmedia/debug/issues/370


add podcastdx-client to known node libraries

Add an additional nodejs client to the docs

Thank you Ryan!


Use imagemin-zopfli for PNGs?

sharp uses pngquant under the hood. Among recent tools, zopflipng seems to give the best lossless compression rate.
Would it make sense for this project to transition to imagemin-zopfli at some point?
It’s slower, but I guess that shouldn’t be an obstacle if the optimisation is meant to be one-off.

Thanks @nisbet-hubbard! Generally interested in best results, too, but it seems that the whole imagemin-* family (with the exception of independent packages like imagemin-guard) are unmaintained. imagemin-zopfli is at 3 years now (for their workflows), which makes me wonder if it would be an exception.
If there’s a signal that this or other imagemin-* packages are being maintained again, I’d have a look, but if not, I’d prioritize future improvements and likely better security with maintained packages (where possible).


Automatic close results after desired time

Current Implementation
After a certificate read the app show the result until the close button is pressed
Suggested Enhancement
An option that allows you to choose a viewing time of your choice
Expected Benefits
In order to use the app with an automatic recognition system.
It would also be useful to be able to use the front camera

Salve, è già stata effettuata questa funzione? Possiamo attivarla in qualche modo? Avrei necessità che si auto chiudesse.

aggiornamenti?

Hi
we are experimenting on a feature like this but the front camera seems a bit low quality in order to achieve a fast QR scanning.
So at the moment we are thinking whether this feature would be useful or not.

Hi we are experimenting on a feature like this but the front camera seems a bit low quality in order to achieve a fast QR scanning. So at the moment we are thinking whether this feature would be useful or not.

hi actually i am wondering regarding automatic closing after scan and if possible go back directly to scan and not to ACTIVITY FIRST


Improve download verification

Adds a verification by checking the HTTP response headers when downloading a shapefile or CSV to ensure the content is in the expected format.

https://github.com/urbanogilson/SICAR/actions/runs/5457822260/jobs/9932249459


Feature/makefiles and cli

Correct ,

These changes look like they are all to support local development, right?

Yep that is right,


bugfix: fix rados_read_op_omap_get_vals deprecated

Signed-off-by: zhangyue<EMAIL_ADDRESS>fix both #76 and #79

@noahdesu  PTAL, I really need this pr.

looks good. out of curiosity, why is this patch required? the deprecation is only a warning, not an error.

Yep, Maybe you're right. But I got different output of program(simply connect to ceph cluster) after this patch.
Before patch only got this deprecation, after patch not occur this deprecation error and got msg(I print in the end of program)


Change KafkaBinderMetrics to a tagged TimeGauge

Using a TimeGauge rather than a plain gauge ensures that the value is scaled to a base unit of time that matches what each monitoring system expects.
I'm a little distressed by the implementation in Kafka, because it appears to use System#currentTimeMillis to arrive at a millisecond latency. currentTimeMillis is not a monotonically increasing clock. It can actually move back/forward on NTP shifts or local time discontinuities. But oh well.
Also moved group and topic to tags.

@artembilan Oops! KafkaBinderActuatorTests fixed.

@artembilan while I agree to move KafkaBinderMetrics to a new class given that MR is optional, i would prefer it to be a separate PR/Issue for tracking purposes.

@jkschneider you missing an @author tag. I'll add it during the merge (just wanted to let you know for the future)


caso de uso 15

Codecov Report

Merging #10 (3595cc7) into dev (1586090) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##               dev     #10   +/-   ##
=======================================
  Coverage     0.83%   0.83%
  Complexity       5       5
=======================================
  Files           90      90
  Lines         1193    1193
  Branches        67      67
=======================================
  Hits            10      10
  Misses        1183    1183

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


Dart VM crash during web framework tests in Engine CI

On the Dart SDK roll here: https://github.com/flutter/engine/pull/31106
../../runtime/vm/object.cc: 4429: error: Unable to use class<PHONE_NUMBER>3116 which is not loaded yet.
version=2.17.0-54.0.dev (dev) (Mon Jan 24 15:02:20 2022 -0800) on "linux_x64"
pid=15784, thread=15848, isolate_group=main(0x564b788d9000), isolate=main(0x564b788da800)
isolate_instructions=564b76043ac0, vm_instructions=564b76043ac0
  pc 0x0000564b762ae0dc fp 0x00007f5520cfdf60 dart::Profiler::DumpStackTrace(void*)+0x7c
  pc 0x0000564b76043c74 fp 0x00007f5520cfe040 dart::Assert::Fail(char const*, ...) const+0x84
  pc 0x0000564b762420ac fp 0x00007f5520cfe060 /b/s/w/ir/x/w/flutter/bin/cache/dart-sdk/bin/dart+0x20740ac
  pc 0x0000564b761b3be3 fp 0x00007f5520cfe180 dart::ClassFinalizer::LoadClassMembers(dart::Class const&)+0xd3
  pc 0x0000564b76239b53 fp 0x00007f5520cfe1d0 dart::Class::EnsureIsFinalized(dart::Thread*) const+0xc3
  pc 0x0000564b762fe30f fp 0x00007f5520cfe230 dart::PatchableCallHandler::ResolveTargetFunction(dart::Object const&)+0x16f
  pc 0x0000564b762fe420 fp 0x00007f5520cfe2a0 dart::PatchableCallHandler::ResolveSwitchAndReturn(dart::Object const&)+0x20
  pc 0x0000564b762ff059 fp 0x00007f5520cfe4c0 dart::DRT_SwitchableCallMiss(dart::NativeArguments)+0x2d9
...

Command and full backtrace in:
https://ci.chromium.org/ui/p/flutter/builders/prod/Linux SDK Drone/104340/overview
Marking P2 for investigation. Will bump up or down depending on whether this repeats on Engine CI rebuilds.

/cc @alexmarkov

I tried running the tests locally a few times, but the crash doesn't appear.
@zanderso Is there a core dump for this crash?

No - we don't have a core dump from this test

To clarify, it's unlikely that that Dart roll introduced the flake. The roll only had some dead code removal and an analysis server change in it. It was a coincidence that the VM happened to crash on that Dart -> Engine roll.

Considering that the crash is not easily reproducible, could we lower the priority of this issue.

Considering that the crash is not easily reproducible, could we lower the priority of this issue.

Yeah, I haven't seen this come up again, so dropping to P4.


syntax parsing error when declaring const hook

If you were to declare a top level constant hook and use it. You will get a build time error.
E.G.
const myHook = ChainedHook([]);

@MappableClass(hook: hook)
class Foo with FooMappable{
  final String? bar;
  Foo(this.bar)
}

The error was:
Could not format because the source could not be parsed:

line 122, column 40 of .: Expected to find '('.
    ╷
122 │         final MappingHook hook = const myHook ;
    │                                        ^^^^^^^^^^^^^^^^^
    ╵

There code here is not referring to the source code, it's referring to the code that dart_mappable created, before throwing it away once it detected a syntax error.
The issue here is that const should only appear when instantiating a Hook, not when referencing a constant.

@schultek Seems like a simple check here would suffice:

Indeed a bug, let me look into it.


Opensearch support as alternative to Elasticsearch

Verify future versions of Opensearch as this is becoming a very viable option as an Elastic replacement and it is fully open source as opposite from Elastic that has started to limit the usage. Right now version 7 and Opensearch 1.x are quite compatible but who knows if things will diverge in the future.
We have several Opensearch clusters available to use for indexing but installing and maintaining Elasticsearch and purchase licenses just for Arkime usage might not be a viable option, and I think others might have the same issue. Especially if you require things like AD logins and such.
Br
Jan

Wrong repo, didn't notice I was in arkimeweb :)


Mistake in the definition of "Key Trust"

"Key Trust"
"The key trust model uses the user's Windows Hello for Business identity to authenticate to on-premises Active Directory. The certificate trust model is supported in hybrid and on-premises deployments and requires Windows Server 2016 domain controllers."
-> I though that for the "certificate trust" model only 2008 R2 DC's are required ?

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

ID: 52a63c1d-79e8-8e28-88b8-38b9984cdeda
Version Independent ID: b88c3dc4-c4aa-4b98-e4b9-ab182975b4c0
Content: How Windows Hello for Business works - Technology and Terms
Content Source: windows/security/identity-protection/hello-for-business/hello-how-it-works-technology.md
Product: w10
GitHub Login: @MikeStephens-MS
Microsoft Alias: mstephen

in the "key trust" section, the second sentence should say "The KEY trust model is supported in hybrid and on-premises deployments and requires Windows Server 2016 domain controllers"; but it says Certificate by mistake. please fix it.

@officedocsbot assign @mypil

@mypil Please  check my PR #3174

@BartJocque - Thank you for submitting feedback.
From our understanding, the issue has been resolved but it may take a few days for the merged content to appear in the article. If you feel it hasn't been resolved, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!

@officedocsbot close


Feature/led driver bugfix

Missing volatile qualifier resulted in this loop away being optimized into an infinite loop:
// wait for current transmission to complete
while(current_driver_idx_ >= 0) {};

Nice catch.


Team Interest Management - clients receive all teams' objects when connecting

Describe the bug
Objects spawned with a team are synced to clients that connect after the object has spawned, even if the new client and existing object are assigned to different teams.
I believe this is the same issue: https://github.com/vis2k/Mirror/issues/3170
[IMPORTANT] How can we reproduce the issue, step by step:
Please tell us how to reproduce your issue, STEP BY STEP, with one of our built in examples.

Set player prefab to have Force Shown checked
Assign an object Team A
Spawn the object when server starts
Join the game as a client where the player object has Team B assigned

When Team B client connects it will receive the Team A object.
Expected behavior
Team B clients shouldn't be sent Team A objects when connecting.
Desktop (please complete the following information):

OS: Windows
Unity version: 2022.1.10
Mirror branch: 2022.10

Additional context
NetworkServer.SpawnObserversForConnection:  NetworkServer.cs#L1367
This is calling into TeamInterestManagement.OnCheckObserver correctly. That method looks at the player object's team's forceShown before checking if the teamIds match or not. TeamInterestManagement.cs#L140
Since my player objects have forceShown = true this is causing all of the objects to be synced to the client, ignoring the teams that are set. I commented out the line I linked and it doesn't seem to immediately break anything and fixes this issue.

Closing as "by design".  ForceShown is intended to make the object visible to all clients.  Team Int. Mgmt. is designed to hide team resources from other teams, and turning on ForceShown is typically done on players so everyone can see each other for combat or whatever.  If you don't want players to see other team players at all then turn ForceShown off.
Perhaps a different flavor of Interest Management might suit your needs better.
For additional support, please join our Discord.

@MrGadget1024 The objects being replicated to the wrong team do not have ForceShown enabled.
Imagine a game where 2 teams (A, B) chase their own objective that shouldn't be visible to the other team. So team A chases objective A and team B chases objective B.

Both teams' player objects (spawned by Mirror on connection) have ForceShown enabled so they can see each other
Objective objects have ForceShown disabled so that only their team can see them

There's a different behavior for Objective objects being replicated:

Player on Team B connects before Objective A is spawned: it only gets replicated to players on team A
Player on Team B connects after Objective A is spawned: it gets replicated to Team A and the Team B player that is connecting

The second behavior is what this bug report is for. Is that the designed behavior? That kind of makes Team Interest Management only useful if you refuse new connections after the game starts.

Alright I think I understand better now - reopening for investigation.


Mesh interaction

Mesh Interaction and optimizing spaces (I'm sorry)

This commit will fix pixi-spine issue https://github.com/pixijs/pixi-spine/issues/38

:+1: Looks good to me, @GoodBoyDigital thoughts?

Wouldn't it be nice to have some tests on this?
It may have some assertions that containsPoint does what it is supposed to do. It seems implemented perfectly, but having tests it will ensure that future changes doesn't break existing functionality.

I 100% agree, I don't think it should stop the merge of the feature, but we should definitely add some if at all possible.

Ok, I'll make some tests :) Btw, I only did test with vertex indices, other branch of "if" wasnt tested.

looks good to me! happy to roll in!


Yet another bump issue.

Describe the bug
Tx was bumped by the user two times, still pending.

Additional context
txid: https://amarok.connextscan.io/tx/0x54337ba4e48ff701a25f7d10c7db6ee748f2288ef1694abfb7d8ebe5f34fc802

one more

https://amarok.connextscan.io/tx/0xed87b24c1c4bd5f9063b2eea77655684998f6bec042521b68d0e417bbe2bbb09

https://amarok.connextscan.io/tx/0xbccc5fc11674973c2201d43e10cee7bfe1bd18e621de6014834de407bcf38091?src=search

one more
https://amarok.connextscan.io/tx/0x9439ca342a02df02fdcafa783a880de2470925516c339fb750e07225088cde44?src=search

We'll be fixing this in https://github.com/connext/monorepo/issues/3360


Update SUMMARY.md

Fixed TOC with proper path so content is displayed in prod.

Thanks, @lieut-data! If you could merge this that'd be awesome (I don't have authorization to merge).

Build failure here looks to be unrelated -- merging anyway.


Simple extension to set the log level for exiv2

I am using this package for my terminal-based picture cataloging system and I am going crazy with how much output I get when reading data from a huge image collection. I could not find another way to suppress all the output.
Since I did never use CGO I was just more or less guessing how this would be implemented in a safe way. I tried to fiddle with the C++ Enums but I gave up and just "made it work". Implementing the callback would be over my head right now and I actually don't care too much for the errors and warnings while parsing.
I would love it if this or something similar could go into the official package so I can get rid of my fork.

Thanks for the contribution!
This looks fine to me, though I don't have experience with cgo either.

Could you please set a new release tag so I can use the new version?

done


Question on protecting an API using AbstractRestfulController with OAuth2

I've finally managed securing my controller (extending the AuthController class) as mentioned in the documentation ( https://github.com/zfcampus/zf-oauth2#how-to-protect-your-api-using-oauth2 ) :
class MyTestController extends AuthController {

    ...

    public function __construct($serverFactory, UserIdProviderInterface $userIdProvider) {

        parent::__construct($serverFactory, $userIdProvider);
    }

    public function onDispatch(\Zend\Mvc\MvcEvent $e) {

        $server = call_user_func($this->serverFactory, "oauth");

        if (!$server->verifyResourceRequest($this->getOAuth2Request())) {

            $response = $server->getResponse();
            return $this->getApiProblemResponse($response);
        }

        return parent::onDispatch($e);
    }

    ...

As my API is RESTful, I would like to use the AbstractRestfulController as base for my controller. Could you please give me a hint/code example, how to secure a controller which extends the AbstractRestfulController? I have not found any useful information so far regarding this topic (and as I'm still quite new to ZF2, there's a lot to learn).
Any help would be greatly appreciated!

hi,
i have the same problem, have you solved it and can give me a hint?

Well, detailing a thorough answer on OAuth2 workflows and architecture would take ages, and here is definitely not the right place.
Briefly, AuthController is not meant to be extended for "business controllers". Its purpose is to handle authentication requests:

generating / revoking tokens
handling authorization code...

and upon success, redirect to the requested routes (controller/action).
What you can do indeed, as referred in https://github.com/zfcampus/zf-oauth2#how-to-protect-your-api-using-oauth2, is to inject the server instance (OAuth2\Server) into your controller from its factory and call its methods from your actions ($this->server->verifyResourceRequest...)
 Its location should be in a dedicated listener and called from you 'onBootstrap' callback in your module class ;)

@jguittard
Can you please share any code example for injecting the server instance (OAuth2\Server) into controller from its factory and call its methods from actions ?

This repository has been closed and moved to laminas-api-tools/api-tools-oauth2; a new issue has been opened at https://github.com/laminas-api-tools/api-tools-oauth2/issues/12.


asset() helper returns HTTP but not HTTPS

I'm testing project at Heroku and using asset() to generate urls for my styles. Heroku automatically uses SSL, but asset() generates urls with HTTP, It makes Chrome to block these scripts.
Project url: http://slovolit.herokuapp.com

Check out trusted proxies. :)


[Loc] added continue message to sql-migration

This PR fixes #

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
26434

Relevant Lines:
57517

💛  - Coveralls


Index.d.ts should contains comment. Otherwise, development is difficult,

Please follow the issue template.


cmd/link: "-X main.version" when version has a value and is later used in a global struct literal doesn't work

What version of Go are you using (go version)?

$ go version
go version go1.13.1 darwin/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="<snip>/Library/Caches/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/segev/go"
GOPROXY=""
GORACE=""
<EMAIL_ADDRESS>GOTMPDIR=""
<EMAIL_ADDRESS>GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/b_/c6yh0ksn63d1yy192x2p9f4c0000gn/T/go-build796234501=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

Steps

Save main.go, go build and run it.
You get "Version: dev"
Run go build -ldflags "-X main.version=foo"
You get "Version: dev", instead of "Version: foo".

If you change var version = "dev" to var version string, it works. It also works if you just fmt.Println("Version:", version) without going through the struct.
The documentation says:
	This is only effective if the variable is declared in the source code either uninitialized
	or initialized to a constant string expression. -X will not work if the initializer makes
	a function call or refers to other variables.

Which doesn't suggest to me that this should somehow not work.
This will be a common scenario when using https://github.com/spf13/cobra which uses a global struct like that with a Version field, and is where I encountered this originally.
main.go
package main

import "fmt"

var version = "dev"

type cmd struct {
	Version string
}

var rootCmd = cmd{
	Version: version,
}

func main() {
	fmt.Println("Version:", rootCmd.Version)
}

What did you expect to see?
"Version: foo" when running go build -ldflags "-X main.version=foo".
What did you see instead?
"Version: dev" when running go build -ldflags "-X main.version=foo".

@cherrymui @rsc @mdempsky @ianlancetaylor

@ianlancetaylor explained in https://github.com/golang/go/issues/26042#issuecomment-400056126

The issue is that for something like:
var A = "foo"
var B = A

the compiler recognizes that A is statically initialized to a constant, so it's compiled as:
var A = "foo"
var B = "foo"

Then later when we link with -X mypkg.A=bar, it only changes it to:
var A = "bar"
var B = "foo"

because the linker doesn't know it needs to modify B as well.
I see at least a few options here:

Change cmd/link's documentation to only guarantee -X works for uninitialized string variables. cmd/compile already pessimistically optimizes references to uninitialized package-scope variables, assuming that they might be initialized externally (e.g., assembly code or the linker).

Change cmd/compile to not apply the optimization mentioned above to string-typed variables. This might produce slightly worse code in some cases, but I think should be limited to emitting slightly less efficient program initialization code (i.e., we need to dynamically initialized some variables instead of statically initialize them).

Change cmd/compile to track how string-typed variables are used and where the values get copied so that cmd/link can patch up everywhere it's copied to. I'm worried this will be a lot of complexity for little value.


bun 1.0.3

Why
like recent bun
What changed

update bun to 1.0.3

Test plan

bun --version with module installed prints 1.0.3

Rollout
Describe any procedures or requirements needed to roll this out safely (or check the box below)

[x] This is fully backward and forward compatible

13b1054 reverted the module config to use the package-json-runner script/config from v3. the only delta is the path for the bun derivation


Add Weight Prediction

Dataset Link
Dataset
Description
Prediction of weight based on height.

Please assign me this issue, I would like to work on this.

Yeah sure , You can work on it.


chore(release): v5.9.0

This PR prepares the release of version v5.9.0.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 61.80%. Comparing base (2081e94) to head (0ea41c2).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #939   +/-   ##
=======================================
  Coverage   61.80%   61.80%
=======================================
  Files          35       35
  Lines        1152     1152
=======================================
  Hits          712      712
  Misses        440      440

Flag
Coverage Δ

win32
59.95% <ø> (ø)

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


Don't set cache-control and expires headers.

Let users set caching related headers as per their needs.

Hmm, don't you think it makes sense to have some sensible defaults, and then the user can remove/update them further if they'd like to?

I would prefer it to be opt-in instead of opt-out but I guess sensible defaults are fine. I'll just use my own response factory :)

Out of curiosity, what headers did you not want?

None, during app development :)

I would suggest adding a check for \Cake\Core\Configure::read('debug') and set the caching headers only if that call returns false.


Possible relay discovery defect

Version: 1.9.2

Platform: Darwin / Linux

Subsystem: Circuit relay transport

Severity: High

Description:
I have a p2p network of some relay and non-relay nodes. The non-relay nodes are configured to discover 3 relays, configured both with and without reservation concurrency. For some reason (maybe connection issues, or anything else - it's irrelevant for the main issue), the connection between nodes and relays breaks, and then after a time, the node is disconnected from all of the relays.
After checking the logs and digging into the implementation of the circuit relay transport, I found that there is an already-implemented relay discovery mechanism, and it works this way in simple terms:

When a relay disconnects, check the number of connected relays against discoverRelays
If not enough relays are connected, start the discovery process, and grab discovery lock, by setting a running flag in RelayDiscovery instance
After discovering relays, try to connect them until we reach discoverRelays
If enough relays are connected, release the lock, letting the discovery to be run again on relays disconnection

I think there is a critical issue here. If for any reason, the discovery doesn't discover enough relays, or we cannot connect the discovered ones, the relays count keeps under the discoverRelays, while the discovery is locked and cannot be run again. Over time, relays disconnect one by one, and we don't reconnect to them.
As an example:

Suppose a discoverRelays of 3
We connect to 3 relays
We disconnect from 2 for some reason
The discovery starts, and finds 4 new relays
For some reason, we only connect to 1 of 4, resulting in 1 new + 1 old connected relays, which is below 3 threshold, and so the discovery won't be run again
The process continues, until no relays are connected

I wonder why other possible solutions are not implemented, for example, changing the condition with which the lock is released, or running the discovery periodically.

Steps to reproduce the error:

It is clarified in description. There are some other configurations that may (or may not) affect the scenario, though. As an example, the auto dial should be disabled.

I found out that the random walk part of the discovery is essentially an infinite async iterable (until the relevant signal is aborted), so it's not exactly the case that I described in my previous comment. By the way, shouldn't we add peer-routing to the list of the transport dependencies if discoverRelay > 0, as of identify? In addition, the peer store search part of the discovery is still run once, which I think, may cause problems. It's possibility is probably quite rare, though.

Over time, relays disconnect one by one, and we don't reconnect to them.
the random walk part of the discovery is essentially an infinite async iterable

Yes, so it should continue to discover peers until it eventually finds enough relays to bring the number with reservations up to discoverRelays.
It sounds like you're seeing something different?

By the way, shouldn't we add peer-routing to the list of the transport dependencies if discoverRelay > 0, as of identify?

I think that would make it easier for users to discover misconfigurations, yes.

In addition, the peer store search part of the discovery is still run once, which I think, may cause problems

Could you expand on this a little?

It sounds like you're seeing something different?

What I'm saying here is that, in my opinion, it's wrong to suppose random walk always succeeds. If any error is thrown inside of the random walk, peer routing, or any related code, we get a failed when finding relays on the network log and the whole relay discovery process ends forever, and there is no mechanism to restart it automatically. Or maybe I'm wrong about it?

I think that would make it easier for users to discover misconfigurations, yes.

If it's as simple as adding it to the list of transport serviceDependences symbol , I can open a PR.

Could you expand on this a little?

Based on what I understood from the relay discovery code, it has two parts: It first tries to find some relays in the peer store, and then starts the random walk. Putting my answer to the first question aside and supposing random walk always succeeds, the peer store search is still only run once. It's neither an infinite iterable like random walk, nor is re-run through some other mechanism. We may not be able to connect to peer store relays at the moment of disconnection for a reason (say a simple network partition), but we may connect them a moment after if we try. In other word, there may be no need to run random walk for a long time at all: we simply need to give the peer store another chance.

My previous comment contains the answers to @achingbrain questions.


GitProjector from tags not working

I've been experimenting with the GitProjector and ObjectTemplate based on the examples in the launch blog post in order to create new environments for specific commits based on git tags.  I've been able to successfully get the example working with spec/ref/branch: ... but when switching over to tags (spec/ref/tag) reconciliation fails with an error.
My GitProjector:
apiVersion: templates.kluctl.io/v1alpha1
kind: GitProjector
metadata:
  name: environment-projector
  namespace: flux-system
spec:
  interval: 1m
  url<EMAIL_ADDRESS>  secretRef:
    name: repo-secret
  ref:
    tag: PR-(\d)+

After this has been pushed to the cluster, I don't see any errors in the template-controller pod but I do see this error on the GitProjector itself:
Status:
  Conditions:
    Last Transition Time:  2024-02-02T17:39:37Z
    Message:               could not determine ref type for refs/tags/PR-1444
    Observed Generation:   1
    Reason:                Error
    Status:                False
    Type:                  Ready

I expected something similar to the using branch, being able to pull the value in my ObjectTemplate with {{ matrix.git.ref.tag }}.
I've been digging through the source code for that error string "could not determine ref type" and it looks like maybe the tags check should be checking for hasPrefix(...) similar to the branch logic above, but is instead checking hasSuffix(...) instead.
Other details of my environment:
My git repository is hosted in Bitbucket (which is why I'm using the normal GitProjector and tags for discovering pull requests, instead of something like ListGithubPullRequests.  I'm also deploying all of this using FluxCD, so Flux deploys the GitProjector and ObjectTemplate.  The GitProjector is looking at my application repository though, not the FluxCD config repo.

Thanks so much - safe travels and enjoy the event!

@mtthwcmpbll latest release should fix the issue you encountered.

Thanks so much!  I've deployed the most recently release and validated it working as expects 🎉


history of transactions is not updated according to token

see USDT history or TALAO history on ethereum address -> same as ETH history

Size: M


Layout and Design "Get Involved"

Split Page into 3 Sections (Home, Business, Community)

"Description"
"Link to sandboxsolar.com"

"Description"
"Link to http://sandboxsolar.com/commercial-solar-installation"

"Description"
"Link to http://sandboxsolar.com/community"

Can we get the text that explains how to add solar at Home, Business, and Community?


devenv: use explicit package paths for executables

As a follow-up from #545 I thought there might be more instances where executable were used from the global system. This turned out to be true.
I'm not sure whether any of the currently used executables result in incompatiblities, but I thought to avoid such potential problems in the future.
I also took the liberty to align the nix and devenv-yaml calls so they look the same as calls to other packages.
I checked for usage of other external commands by using PATH="" in the top of the devenv-script and look for command not found errors.

Hmm CI is failing because it tries to run:
devenv ci
with the new changes, which results in:
/nix/store/00i4b6rg45xinc8jhf4cfdamnpisn02l-devenv-profile/bin/devenv: line 228: /nix/store/isia7synl0mdbcvfwav5ywnrwmwbwvxv-source/bin/nix: No such file or directory

There seems to be a mixup of src/devenv.nix with an older flake.nix. This is because src/devenv.nix is imported from other places than just /flake.nix.
I have now made sure the original call signature is used for src/devenv.nix, so it stays compatible. I did the resolve nix to nix.packages.${pkgs.stdenv.system}.nix inside src/devenv.nix instead of inside /flake.nix.

The python-poetry test should've been resolved by https://github.com/cachix/devenv/pull/552


change RxSwift to version 5

This change is already part of https://github.com/quickbirdstudios/XCoordinator/pull/87, however there might be some changes to how references are held in XCoordinator, which we first want to discuss before supporting RxSwift 5 (because a change from RxSwift 4 to 5 will require a major version update).


Where to run script https://github.com/CESNET/netopeer2/blob/master/cli/sample_script.sh

Hi,
This is not an issue but a question where I can execute this file from? I guess netopeer2-server needs to be started first. Can I run it from a unix shell?
br,
//mike

Yes, the server should be running and then you run it in a shell or simply execute it, the system knows it should be run by a shell because of this line.


Can't read books in creative

Players aren't able to read books while in creative mode. Even if pages it's inside whitelistedNBT and removeCreativeNBTTags = false
Mod version: 0.5.1

So book loses all the pages once you switch to creative?

This is likely related to the item sanitiser always being ran on creative to avoid accidentally clearing the inventory by giving known hashes to fetch the correct item against, although it does end up giving clients semi-broken items that aren't actually broken to the server (at least, until the item gets inadvertently overwritten by other means). Creative really isn't sane to deal with here. Fun fact: The creative client will resend every item that has been updated since the last time the inventory was opened back to the server, overwriting what was there before. This had the effect of clearing the entire inventory of, what was to GolfIV, denied NBT when removeCreativeNBTTags was set to true.
A quick workaround for now would be:

Go into Survival or Adventure (or any other non-creative gamemodes)
Click the book in the inventory to force the server to send the correct NBT
Use the book as you would normally.

An idea that I have in mind is to try to send the correct NBT for the item when one holds something in the hotbar to open it. This would mainly be useful for books, but could be applied to other items that may require their NBT intact on the client in order to be used. This can also be extended to survival since there maybe a case where the player's inventory was sanitised for being oversized, but the individual item could be fine.
Should also be possible to make this just not happen at all when removeCreativeNBTTags is set to false, and to only append to the NBT when the inventory isn't oversized, since the mechanism is meant for that.


Could not find CIFs in /mmcif_files

I have been working to get alphafold running with a singularity image on my schools cluster. There was an issue using Rsync to get the whole PDB downloaded in cif format so we used FTP and manually put them in the right folder. All of the files exist in .cif format in the proper directory and the directory is specified correctly in my run script, but every time it fails with this error:
Could not find CIFs in Directory that has all the cif files
Any clue what may be wrong or how I could go about fixing it?

Is it possible the cif files in the directory are still gzipped?

They are fully unzipped and are in the XXXX.cif format.

Could you try running the following code snippet that does the check on the directory?
import glob
import os

mmcif_dir = "<path to the directory with the mmCIF files>"

if not glob.glob(os.path.join(mmcif_dir, '*.cif')):
  print('Could not find CIFs in %s', mmcif_dir)

If it reports the same problem, then it could be also a permission problem – are the files world readable?

I ran that and got no errors and all of the files are world readable.
Here is the full error when I try to run alphafold:
WARNING: Skipping mount /etc/localtime [binds]: /etc/localtime doesn't exist in container /opt/alphafoldenv/lib/python3.8/site-packages/absl/flags/_validators.py:203: UserWarning: Flag --preset has a non-None default value; therefore, mark_flag_as_required will pass even if flag is not specified in the command line!   warnings.warn( E0902 16:20:15.371900<PHONE_NUMBER>70944 templates.py:860] Could not find CIFs in /gpfs/share/apps/alphafold/2.0.0/pdb_mmcif/mmcif_files Traceback (most recent call last):   File "/app/alphafold/run_alphafold.py", line 283, in <module>     app.run(main)   File "/opt/alphafoldenv/lib/python3.8/site-packages/absl/app.py", line 312, in run     _run_main(main, args)   File "/opt/alphafoldenv/lib/python3.8/site-packages/absl/app.py", line 258, in _run_main     sys.exit(main(argv))   File "/app/alphafold/run_alphafold.py", line 210, in main     template_featurizer = templates.TemplateHitFeaturizer(   File "/app/alphafold/alphafold/data/templates.py", line 861, in __init__     raise ValueError(f'Could not find CIFs in {self._mmcif_dir}') ValueError: Could not find CIFs in /gpfs/share/apps/alphafold/2.0.0/pdb_mmcif/mmcif_files

I seem to be having the same issue
File "/app/alphafold/alphafold/data/templates.py", line 861, in init
raise ValueError(f'Could not find CIFs in {self._mmcif_dir}')
ValueError: Could not find CIFs

Are you running AlphaFold using Docker?
If so, do you see
Mounting <path to your mmcif dir> -> <...>

in the logs? I also see you are mounting a gpfs filesystem. Do you know whether it is supported by Docker?

We are not running AlphaFold with Docker. We have been trying to use a Singularity Image so that it will run on our HPC.

I am experiencing the same problem, @Augustin-Zidek do you have a time estimate as to when this issue may be resolved. Thanks in advance.

I seem to have got it working locally with Singularity and have written a basic shell script to setup the database locations at /ARCCA/alphafold/tree/singularity/docker/run_singularity.sh but I have so far had to fix permissions on files and seems its very sensitive and when having issues example command being run would help.

It's just the bindings for Singularity to point to the databases. I could refactor a bit to remove the need for a file but helped me debug. Realised the whole script is a bit overkill but was copying the docker way of running it. Ideally I can see that to run on clusters it needs converting to nextflow or similar to deal with different resource requirements for each step but that's a future project.

Its created in the script - basically just the SINGULARITY_BIND environment variable where bindings can be sourced to set it before running Singularity e.g.
export SINGULARITY_BIND="${SINGULARITY_BIND},$source_path:$target_path"
echo "export SINGULARITY_BIND=\"$SINGULARITY_BIND\"" > bindings.txt

Bit blunt but assumed container needed databases in certain locations but didn't register the arguments are basically setting locations and can be anywhere.

why is this issue closed when there is no clear answer? I am facing same problem.

Hello, I have finally figured out a solution for the issue. In my bash script that calls singularity, there were issues binding all the database directories. I was able to solve the issue by putting soft links to each database directory inside of one folder. I then directed the singularity script to bind to that directory such that all of them would be visible and everything is running now. Hope this helps!

could you please elaborate on that solution? are you still binding each database separately? can you maybe provide your binding commands? thanks!


Multipart form submission "bug" (application/octet-stream vs. application/pdf)

HtmlUnit (3.5.0) is sending a content type of "application/octet-stream" rather than "application/pdf" for PDF files, as form-multipart posts (using BrowserVersion.CHROME).
This is not how modern browsers or other modern HTTP tools appear to behave. E.g., they seem to all send "application/pdf". Obviously a backend can therefore react differently, if it considers this this piece of information.
Example:
Content-Type: multipart/form-data; boundary=AaB03x

   --AaB03x
   Content-Disposition: form-data; name="submit-name"

   Larry
   --AaB03x
   Content-Disposition: form-data; name="files"; filename="file1.pdf"
   Content-Type: application/octet-stream

   ... contents of file1.pdf ...
   --AaB03x--

... could/should be:
Content-Type: multipart/form-data; boundary=AaB03x

   --AaB03x
   Content-Disposition: form-data; name="submit-name"

   Larry
   --AaB03x
   Content-Disposition: form-data; name="files"; filename="file1.pdf"
   Content-Type: application/pdf

   ... contents of file1.pdf ...
   --AaB03x--

If it were a JPG or other well-known file type, the Content-Type is correct for binary files (i.e., not application/octet-stream).
This can be adjusted/introduced via:
getBrowserVersion().registerUploadMimeType("pdf", "application/pdf");
However, I think this should be default behaviour (like the real Chrome).
Easy to fix, just needs to be added to BrowserVersion, like the others:
CHROME.registerUploadMimeType("pdf","application/pdf");

Thanks. I will have a look after being back from vacation.

Thank you!


expose public_sharing property for eligible users

Forgot this one. This is required to be able to modify the visibility of the user from the UI.
In addition to this:

Updated response for Nextcloud directory list, so it won't return HTTP/400 when no path is provided or when NC is not configured. This will reduce the noise in the UI.
Fixed custom exception behavior to handle the case when errors are not a dictionary
Refactored tests. Removed duplicate cases and disabled broken test for now. All current tests passed!

Looks good to me!


Added branding values (icon + color)

Change-type: patch
Signed-off-by: Miguel Casqueira<EMAIL_ADDRESS>

@balena-ci I self-certify!


how to integrate function calling

greta work Brandon! How do you integrate function calling into it

I am using queue.put_nowait() and it seems to work ok with a demo assistant with two functions to create a response. I modified the on_event in event_handler.py for action required. With each iteration of tools (functions) I built the tools output. Because I am learning Python, I have yet to figure out how to call client.beta.threads.runs.submit_tool_outputs_stream … my temporary solution is to call the api via hpptx.AsyncClient() and look for “thread.message.completed” and add the message to the queue. I self.done.clear() when I get a 200 response after the API call. Not sure that helps.


[DCOS-32730]Added rlimit_memlock parameter to the elastic configuration

In order to make bootstrap.memory_lock: true working in Elastic, we need rlimit_memlock to be set to infinity.

Elastic is no longer in monorepo


Removes the kubernetes_dashboard element from google_container_cluste…

Removes the kubernetes_dashboard element from google_container_cluster
#71

Thanks for the PR @craigedmunds !
I'll kick off the tests now.

The tests passed, so merging this in!


Comment out 'use TB hierarchy' code

It doesn't work so maybe don't expose it to users until it's fixed. I can volunteer to fix this in January if nobody else does

Boy do I have good news for you!


custom charts: metrics listing out of date

Looking at our custom charts page, I notice a lot of the listed "Available Metrics" are out of date. For e.g. any and everything command queue related is no longer a thing (replicas.commandqueue.combinedqueuesize, replicas.commandqueue.combinedreadcount, etc.). That aside, other metrics also no longer exist, such as txn.abandons.
To determine what exists and what doesn't, I was looking at the metrics I'm able to track on our custom charts page (/debug/chart).

This also seems to be a thing for our prior releases. I only checked as far back as 19.1, but wouldn't be surprised if it's been around for longer (the command queue was removed ages ago).

bump


Testing super-linter

You configured linter incorrectly, it runs on branch "master", but we have "main" instead


[Feature]: Integrate Triton page attention to vllm

paged_attention_v2_kernel(FlagAttention)
paged_attention_v2_reduce_kernel(FlagAttention)

Replace vLLM CUDA kernels with Triton kernels:
https://bytedance.larkoffice.com/wiki/HHoiwbHbui1ZaKkYK3yclrXInvJ

page attention and flash attention for prefill triton support completed. Decode with KV cache needs to rewrite a new triton version method.

Needs thorough testing after https://github.com/bd-iaas-us/vllm/issues/43 completion. Now the code is ready for branch trion-npu-support.

The thorough testing will be done in https://github.com/bd-iaas-us/vllm/issues/36.


Use JsonProcessingException instead of JacksonException in Test

Description
Use JsonProcessingException in testing instead of JacksonException to maintain compatibility with Jackson 2.10 and above in test code as JacksonException was introduced in 2.12.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


404 Not Found [npx fix-dependencies]

What happened?
I was following the docs as mentioned https://microsoft.github.io/rnx-kit/docs/guides/dependency-management
Although while running the command npx fix-dependencies I got the error of which I am attaching the screenshot below.

Affected Package
fix-dependencies
Version
"react-native": "0.72.6"
Which platforms are you seeing this issue on?

[ ] Android
[ ] iOS
[X] macOS
[ ] Windows

System Information
System:
  OS: macOS 14.5
  CPU: (8) arm64 Apple M1
  Memory: 196.88 MB / 8.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 16.20.1
    path: ~/.nvm/versions/node/v16.20.1/bin/node
  Yarn:
    version: 1.22.22
    path: /opt/homebrew/bin/yarn
  npm:
    version: 6.14.18
    path: ~/.nvm/versions/node/v16.20.1/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.15.2
    path: /opt/homebrew/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.5
      - iOS 17.5
      - macOS 14.5
      - tvOS 17.5
      - visionOS 1.2
      - watchOS 10.5
  Android SDK:
    API Levels:
      - "31"
      - "32"
      - "33"
      - "34"
    Build Tools:
      - 30.0.3
      - 31.0.0
      - 33.0.0
      - 33.0.2
      - 34.0.0
    System Images:
      - android-28 | Google APIs ARM 64 v8a
      - android-31 | Google APIs ARM 64 v8a
      - android-33 | Google APIs ARM 64 v8a
      - android-33 | Google APIs ATD ARM 64 v8a
      - android-34 | Google APIs ARM 64 v8a
      - android-34 | Google Play ARM 64 v8a
    Android NDK: Not Found
IDEs:
  Android Studio: 2024.1 AI-241.15989.150.2411.11948838
  Xcode:
    version: 15.4/15F31d
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.10
    path: /Library/Java/JavaVirtualMachines/jdk-17.jdk/Contents/Home/bin/javac
  Ruby:
    version: 2.6.10
    path: /usr/bin/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.72.6
    wanted: 0.72.6
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: Not found
  newArchEnabled: Not found

Steps to Reproduce
In the react-native project at root run the command npm add --save-dev @rnx-kit/align-deps
Then add the following scripts command in package.json:
 "scripts": {
    "check-dependencies": "rnx-align-deps",
    "fix-dependencies": "rnx-align-deps --write"
  }

Then run the command npx rnx-align-deps --init app
At last run npx fix-dependencies
Code of Conduct

[X] I agree to follow this project's Code of Conduct

To run a script, you need to run npm run <script>. You should read the npm documentation here: https://docs.npmjs.com/cli/v10/commands/npm-run-script
As this issue is unrelated to rnx-kit, I'm closing it.


Ability to work with a list of configured values of single values

As an app devleoper, I want to read a list or array of configured values of single values. An example would be a list of valid keys, or a list of specially handled business keys.

list.1=
list.2
list=apple, orange

Two sides to this story:

configuration files (probably depends also on file type - yaml, properties)
API to retrieve this value (probably a single way to read an array/list)

2. API to retrieve this value (probably a single way to read an array/list)

Add a built-in converter for this.

Consider adding built-in converters for  Collection, Map, and array type as key.
config.alist=apple,orange
config.amap=a,apple,o,orange
config.array=apple,orange

For the properties format, my recommendation to represent Collections and Maps is the following:
# Collections
config.list[0]=foo
config.list[1]=bar

#Maps (the Map key is the last segment of the property)
config.map.key=foo
config.map.anotherKey=bar

This avoids issues with escaping the comma used to split the values, and in my opinion, it is also more explicit and clear. It has some more pros for advanced stuff. For instance, complex types:
server.host=localhost
server.port=8080
server.endpoints[0].path=/hello
server.endpoints[0].method=GET

@ConfigMapping will provide support for this.


[BUG] Fix issues with the enhancement issue template

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When opening an Issue of type enhancement there are two issues

There is a mandatory box to describe the current behaviour
The checkbox with the id: work_on_it is set to mandatory so you can only open an enhancement if your prepared to do the work

Expected Behavior
Remove the current behaviour text box completely , and reword the next textbox to
"label: Please describe how this enhancement will improve Egeria"
and secondly make the "work_on_it" checkbox to optional
Steps To Reproduce
try to open an Enhancement Issue
Environment
- Egeria:
- OS:
- Java:
- Browser (for UI issues):
- Additional connectors and integration:

Any Further Information?
No response

This has now been merged & the changes look good - thanks


Change language to English and set recording duration

I cannot change language to English and set recording duration

#31


the problem of installing pangolin with vcpkg for windows x64

I am installing the pangolin library in the windows 10(x64) with vcpkg ,but I have suffered the following problem:
-- Downloading http://www.nasm.us/pub/nasm/releasebuilds/2.14/win32/nasm-2.14-win32.zip...
-- Downloading http://www.nasm.us/pub/nasm/releasebuilds/2.14/win32/nasm-2.14-win32.zip... Failed. Status: 22;"HTTP response code said error"
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:156 (message):
Failed to download file.
If you use a proxy, please set the HTTPS_PROXY and HTTP_PROXY environment
variables to "https://user:password@your-proxy-ip-address:port/".
Otherwise, please submit an issue at https://github.com/Microsoft/vcpkg/issues
Error: Building package libjpeg-turbo:x64-windows failed with: BUILD_FAILED
could you give me some help? thanks!

Looks the www.nasm.us website is sometimes down making it impossible to download the file

@wulang584513 Thanks for the feedback. I did not reproduce this issue anymore. Please reopen a new issue if it is still a problem for you!


python310Packages.pontos: 23.3.1 -> 23.3.3

Changelog: https://github.com/greenbone/pontos/releases/tag/v23.3.3
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 219370 run on x86_64-linux 1

2 packages built:

python310Packages.pontos
python311Packages.pontos


[Feature Request]: Something like Tray Application.

It would be awesome if I could set the application as a "tray application".
Whenever this application is minimized, it would be minimized to the toolbar. Something like automatic "Minimalize to tray".

Sorry for the late reply, but yes, looks a good idea.
To be honest this program needs a revamp with new features too, like move to another virtual desktop (Windows 10).
Not sure when I will start working on this again, but consider it on my list.
[]'s

I was going to ask for the same thing. There's also a slight variation that I hope would be easy to implement at the same time: override the close button so that instead of closing the app, it minimizes it to the system tray.
As an example use case, I use Teams and Outlook at work, and the way they behave is annoyingly different. Teams automatically does what I suggested, where closing the window leaves the app in the system tray. Outlook, OHOH, shuts down completely if you press the close button, so you have to remember to click the minimize button instead if you want to keep getting notifications about things like upcoming meetings. Right now I'm using an AutoHotKey script to re-open Outlook if I close it by accident, but that feels like a pretty ugly hack.


proposal(contrib/openstack): python replacement for shell script

The existing provisioning script for openstack is fairly brittle and relies
on a bunch of software that the user needs to install ( python-novaclient etc )
This is a working [just] prototype that uses the openstack-infra shade library
which is designed to be more interopable between different openstack clouds.
The basic syntax remains the same,  however by using argparse we can start to
extend it to support more complicated openstack environments.
It also removes the need to run make discovery_url as it grabs the user-data template
and fetches the URL in code.
Usage:
$ pip install -r contrib/openstack/requirements.txt
$ contrib/openstack/provision-openstack-cluster.py --help
usage: provision-openstack-cluster.py [-h] [--action {build}]
                                      [--flavor FLAVOR] [--network NETWORK]
                                      [--floating-ip-pool FLOATING_IP_POOL]
                                      [--secgroup SECGROUP]
                                      [--instances INSTANCES]
                                      [--instance-prefix INSTANCE_PREFIX]
                                      [--os-auth-url OS_AUTH_URL]
                                      [--os-username OS_USERNAME]
                                      [--os-password OS_PASSWORD]
                                      [--os-project OS_PROJECT]
                                      image keypair

Installer for DEIS on Openstack

positional arguments:
  image                 The name or ID of the glance image to use. Should
                        exist in glance
  keypair               The name of the ssh keypair to use. Should exist in
                        nova

optional arguments:
  -h, --help            show this help message and exit
  --action {build}      Action to perform
  --flavor FLAVOR       The Openstack flavor to use
  --network NETWORK     The Openstack network to use, Should exist in Neutron
  --floating-ip-pool FLOATING_IP_POOL
                        The Openstack floating IP pool to use, Should exist in
                        Neutron
  --secgroup SECGROUP   The Openstack secgroup to use. Will create if not
                        there.
  --instances INSTANCES
                        The number of instances to start
  --instance-prefix INSTANCE_PREFIX
                        The name for your instances
  --os-auth-url OS_AUTH_URL
                        The Openstack Auth URL
  --os-username OS_USERNAME
                        The Openstack Username
  --os-password OS_PASSWORD
                        The Openstack Password
  --os-project OS_PROJECT
                        The Openstack Project/Tenant Name
$ contrib/openstack/provision-openstack-cluster.py coreos-717.3.0 deis \
	  --network internal --floating-ip-pool external

Your Pull Request can earn you credit at DigitalOcean! See http://docs.deis.io/en/latest/contributing/hacking/#digitalocean-credit for details.
Thanks for the contribution! Please ensure your commits follow our style guide. This code will be tested once a Deis maintainer reviews it.

this is not meant for immediate merging ...   but to open a discussion and see if this is wanted, and/or what other features would be interesting.    Documentation would also need to be updated in the correct place before/after merge.

I really appreciate the effort put into this PR - I think this work could integrate very nicely into rigger which is intended to provide a common user interface (currently only CLI) on top of our cloud provider provisioning scripts. Take a look at how I've defined the interaction between rigger and the deis provisioning scripts in this design document.
At the current time we don't test the OpenStack-oriented Deis provisioning process, but it would be most excellent if we started doing that, and this PR could jumpstart that effort on our end. :+1:

Yeah we can probably use terraform if its openstack support is up to scratch.   do we have an example where we're using it for another platform that I can borrow style etc from ?

@paulczar #4315 :)

Accidentally closed the PR.

Thanks for the contribution! Please ensure your commits follow our style guide. This code will be tested once a Deis maintainer reviews it.
Your Pull Request can earn you credit at DigitalOcean! See http://docs.deis.io/en/latest/contributing/hacking/#digitalocean-credit for details.

Thanks for the contribution! Please ensure your commits follow our style guide. This code will be tested once a Deis maintainer reviews it.
Your Pull Request can earn you credit at DigitalOcean! See http://docs.deis.io/en/latest/contributing/hacking/#digitalocean-credit for details.

@paulczar Did you get a chance to take a look at Terraform support for OpenStack? Docs are here: https://terraform.io/docs/providers/openstack/index.html
I'm not familiar enough with OpenStack to judge the comprehensiveness of support, but I agree with @bacongobbler that if we're going to rewrite a provider, we should probably do it in Terraform whenever possible.

I hope to find the time this week to take a shot at doing it with terraform.

first pass at terrorform support.   seems pretty straightforward.   not fully tested yet ... but I do end up with working coreos instances...    updated the PR to provide status.
@sgoings can you glance over the rigger bits?  I just blank copy and pasted from the DO section and made a best guess.

Sweet! Thanks @paulczar for putting time into this! I'll take a closer look tomorrow and see what I can do to add OpenStack as a CI-tested provider soon...

ping @sgoings for re-review -- looks like @paulczar made your last requested change

Jenkins, add to whitelist.

rebased it to remove all the clutter from the original python stuff.

@sgoings can you give this a look-see when time permits? I'm trying to fish or cut bait on remaining LTS release issues soon. :clock1130:

OpenStack testing efforts are now targeted at Deis v2. It's unlikely we'll revive this PR so I'm closing it. Thanks again @paulczar!