unable to update Scenario Outlines on Jira

I have a test which is a Scenario outline:
e.g.
  Given there are <start> cucumbers
  When I eat <eat> cucumbers
  Then I should have <left> cucumbers

  Examples:
    | start | eat | left |
    |    12 |   5 |    7 |
    |    20 |   5 |   15  |

When I run that as part of a test suite I get the following error:
Uploaded results to JIRA XRAY Test Execution: xx-1234
Exception IndexError: list index out of range
Traceback (most recent call last):
  File "C:\Python310\lib\runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Python310\lib\runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "D:\git\testing\windows-application-framework\venv\Scripts\behave.exe\__main__.py", line 7, in <module>
  File "D:\git\testing\windows-application-framework\venv\lib\site-packages\behave\__main__.py", line 183, in main
    return run_behave(config)
  File "D:\git\testing\windows-application-framework\venv\lib\site-packages\behave\__main__.py", line 127, in run_behave
    failed = runner.run()
  File "D:\git\testing\windows-application-framework\venv\lib\site-packages\behave\runner.py", line 804, in run
    return self.run_with_paths()
  File "D:\git\testing\windows-application-framework\venv\lib\site-packages\behave\runner.py", line 824, in run_with_paths
    return self.run_model()
  File "D:\git\testing\windows-application-framework\venv\lib\site-packages\behave\runner.py", line 626, in run_model
    failed = feature.run(self)
  File "D:\git\testing\windows-application-framework\venv\lib\site-packages\behave\model.py", line 350, in run
    formatter.eof()
  File "D:\git\testing\windows-application-framework\venv\lib\site-packages\behave_xray\formatter.py", line 221, in eof
    self.collect_tests()
  File "D:\git\testing\windows-application-framework\venv\lib\site-packages\behave_xray\formatter.py", line 235, in collect_tests
    testcase.status = self._get_xray_status(tc_status.statuses[0].name)
IndexError: list index out of range

I cannot reproduce. Here is my outline scenario: https://github.com/fundakol/behave-xray/blob/b6a1035d5cd15f525bb2e1b7318633f088d8bbf3/tests/features/calculator.feature#L26-L34
which worked with this command:
behave tests -f behave_xray:XrayCloudFormatter
Did I miss something?


Add GitHub links to changes of release 1.0.0

Add links according to the new changelog guidelines

@orenbm Thanks for taking care of this <3


User should be able to connect accounts to single login

User should be able to connect multiple social logins, so in case he is locked out of one they can use the other logins.

@patpedrosa
This bounty has been completed.

Need to decide where to put the link to it. currently it is simply in the menu
Currently it is simply a toggle for each login method, but actually after playing with it, I understood it should probably be an "Add", because you can have multiple google login. Under every login type, should be a list of connected accounts with an X to remove each one.


Alter rhacs-* tasks to rely on workspaces rather than secrets

Changes

Change the existing rhacs-* tasks in order to rely on workspace and file path rather than secret ref to access theAPI token value.
The goal here is to open for a new task that exchanges service account token against a short-lived token for use in RHACS commands.
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Follows the authoring recommendations
[ ] Includes docs (if user facing)
[ ] Includes tests (for new tasks or changed functionality)

See the end-to-end testing documentation for guidance and CI details.

[ ] Meets the Tekton contributor standards (including functionality, content, code)
[ ] Commit messages follow commit message best practices
[ ] Has a kind label. You can add one by adding a comment on this PR that
contains /kind <type>. Valid types are bug, cleanup, design, documentation,
feature, flake, misc, question, tep
[ ] Complies with Catalog Organization TEP, see example. Note An issue has been filed to automate this validation

[ ] File path follows  <kind>/<name>/<version>/name.yaml

[ ] Has README.md at <kind>/<name>/<version>/README.md

[ ] Has mandatory metadata.labels - app.kubernetes.io/version the same as the <version> of the resource

[ ] Has mandatory metadata.annotations tekton.dev/pipelines.minVersion

[ ] mandatory spec.description follows the convention
  ```

  spec:
    description: >-
      one line summary of the resource

      Paragraph(s) to describe the resource.
  ```

See the contribution guide for more details.

Upstream PRs for rhacs-* tasks were sent and merged.


"Hide Sidebar" menu item not updating to "Show Sidebar"

Git @ 911e6b0
Steps:

Evergreen -> View -> Hide Sidebar
Evergreen -> View

What happened:
Menu item is still named "Hide Sidebar".
Expected result:
Menu item is named "Show Sidebar".

I checked with git @ 765a91d and the bug is still there.

Deleting the derived folder and building from scratch fixed it...


[youtube:comments] Move comment extraction to new API

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[X] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[X] Searched the bugtracker for similar pull requests
[X] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[X] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[X] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Mostly just a draft at the moment. Likely will not merge until YouTube completely moves over to the new API for comment extraction on web client.
Fixes https://github.com/yt-dlp/yt-dlp/issues/438
Currently uses the old ajax API for the initial comment section load, but then uses new API for subsequent requests. This is due to this particular token not being compatible with the new API (see https://github.com/yt-dlp/yt-dlp/issues/438#issuecomment-869262106). This will be removed of course when YouTube finalizes the change.
ISSUES:
clickTrackingParams
In requests in the browser, this is located under the context/clickTracking and changes (likely based on the value given for the continuation).
Currently, in yt-dlp, we have this messy behavior (mostly TabIE continuation requests):

same clickTrackingParams for all requests since it is included in the context of ytcfg (hence what extract_context gets)
Updated clickTrackingParams provided in top level payload next to continuation
e.g.

query = {
    'context': {'clickTracking': {'clickTrackingParams': ctp_from_ytcfg}, 'client': {...}},
    'continuation'': token,
    'clickTrackingParams': ctp_for_token
}

???
TODO:

[ ] Handle new API initial continuation
[ ] Refactor build_continuation_query to be of new API format
[ ] Investigate where clickTrackingParams should be
More to come...

YouTube appears to be testing different response structures atm.
New API with old response:

New API with new response:

Noticed they've completely changed the location of the data we want, so we'll have to rework the relevant extraction code.


[Windows] Fix mouse cursor not shown after multiple mode changes.

Prevent overwriting old cursor handle on multiple mouse_mode changes
Ensure mouse_mode is set before cursor_set_shape is called to restore cursor, otherwise it will be ignored until WM_SETCURSOR is received (on window focus change).
Tested only on Wine running on macOS.
Fixes #37103

Thanks!

Cherry-picked for 3.2.2.


Add from_subquery to read models

This adds a from_subquery to read models that allows specifying
a subquery as the from for a query instead of always starting from
a single table. This is partially setup for adding set operators
because from subqueries are useful when using set operators. They
allow you to combine a number of queries and then order or filter
all of the results.
This adds a from expression to read model Relation objects so
they can dynamically build their initial active record relation
depending on if they are using a from subquery or not. By default
the from expression is a NullFromExpression which handles raising
nice errors explaining that a from or from_subquery needs to
be used. The FromExpression is mostly a struct wrapping the
record class that is provided when calling from. The
FromSubqueryExpression takes a block and will lazily build a
FromSubquery. By making the building of the from subquery lazy
the block isn't evaluated until query is called on a read model.
The subquery is used to build an initial activerecord relation and
set its from to the subquery's sql.
The FromSubquery now also knows its record class by asking its
read model class for its record class. This can cause a recursive
call through multiple from subqueries if a read model nests from
subqueries. This allows the outer most read model to use a deeply
nested record class to build an activerecord relation.
Closes #214
@kellyredding - Ready for review.

@jcredding this seems legit to me man 👎


Memory leak when using Socket.readString

The following code results in a big amount of memory being allocated by the process that is not being released.
After 20k connections, the used memory of the process is more than 100MB. After removing the call to readString, the memory usage does not increase anymore. I tested this with BlueSocket 1.0.52.
var i = 0
while true {
    let socket = try Socket.create()
    try! socket.connect(to: "localhost", port: 31337)
    let serviceResponse = try! socket.readString()!

    i += 1
    print(i, String(describing: serviceResponse))
    Thread.sleep(forTimeInterval: 0.001)
}

The socket connects to a service that first responds with "foobar" and then closes the connection.
The code for reproducing this can be found in the repository https://github.com/johannes87/TestBlueSocketMemoryLeak
The target TestService inside the Xcode project is the "foobar" service mentioned above.

I noticed that when I run dispatchMain() after the loop (and changing the while condition to e.g. "i < 20000"), the process uses less memory again. Before the connection loop, the memory usage is about 6MB, and after calling dispatchMain it is about 25MB. The maximum amount of memory used was ~135MB.


fix buf redeclared in scrapeloop

buf Has been declared in line 785, I think it is unnecessary to declare a new variable again here.
Signed-off-by: arugakiWei<EMAIL_ADDRESS>

👍 thanks!


Make widget modular

Now widget contains bitcoin, solana and evm support. Sadly all this support shipped on one package.  In some cases (a lot of cases i believe)  only one network is needed and other just a lot of unused bytes
Will be amazing to install only necessary network support by moving it to optional-dependencies and widget can detect what options should be enabled
Thank you!

Hey, we also had this idea, just didn't have time to properly address it. You can disable chains you don't want to see, though.
https://docs.li.fi/integrate-li.fi-widget/configure-widget#configure-allow-and-deny-options
The footprint should be minimal (100-200kb per ecosystem). Anyway, I completely agree with you and we will make it more modular :) Thank you!


Cannot set bytesused using meta struct when queueing data.

Hi,
I'm not sure if this is me misunderstanding something but when I fill the mmap output buffer with data I do not see a way to set the bytesUsed. Im streaming mjpeg and depending of the encoding the data size changes. As far as I can see now always the max buffer size is set. I tried using the meta struct to set meta.bytesused but that does not seem to be used in the library. As a workaround i did this in src/io/mmap/stream.rs:
diff --git a/src/io/mmap/stream.rs b/src/io/mmap/stream.rs
index 8a30846..13b21fc 100644
--- a/src/io/mmap/stream.rs
+++ b/src/io/mmap/stream.rs
@@ -190,9 +190,7 @@ impl<'a, 'b> OutputStream<'b> for Stream<'a> {
             v4l2_buf.type_ = self.buf_type as u32;
             v4l2_buf.memory = Memory::Mmap as u32;
             v4l2_buf.index = index as u32;
-            // output settings
-            let bytes = self.arena.get_unchecked_mut(index);
-            v4l2_buf.bytesused = bytes.len() as u32;
+            v4l2_buf.bytesused = self.buf_meta[index].bytesused;
             v4l2_buf.field = self.buf_meta[index].field;

             v4l2::ioctl(

However I do not know if this breaks other usage of the OutputStream. Am I missing something? Or am I on the right track and should my change be expanded to cover all situations.
Thanks in advance!

I recall people recently reported issues about the same thing. I attempted to fix those (runtime) crashes with https://github.com/raymanfx/libv4l-rs/commit/4d003b69f1dfa8580ff3127cfca03fc6888da509, but that only prevents copying a larger range of input data to a smaller output buffer.
I assume your patch is still needed for semantical correctness. Honestly I did not test output streaming for some weeks now, perhaps I should do that again soon. Could you send a PR for this one as well? Then we'll review it and I'll run some tests with MJPG frame encoding.

Done: https://github.com/raymanfx/libv4l-rs/pull/33
One issue that I can see is when bytesused is not set on the meta struct. So there should be some check there and then perhaps default to the max size?


warmup: run tiling scheme negotiation w/ roi

This fixes cache warmup for HDF5 files, that have different requirements for the tiling scheme depending on roi/non-roi runs.
Also run warmup with both SumUDF and PickUDF
Contributor Checklist:

[ ] I have added or updated my entry in the creators.json file
[ ] I have added a changelog entry for my contribution
[ ] I have added/updated documentation for all user-facing changes
[ ] I have added/updated test cases
[ ] I have included the rebuilt production build of the client (only if changes were made to the GUI)

Reviewer Checklist:

[ ] /azp run libertem.libertem-data passed

/azp run libertem.libertem-data

Thx, merging!


Gradle plugin move to MIrah organisation?

Disclaimer: I know this might not be exacyly the right place to ask, but I don't think the Groogle Groups for Mirah might have the traction to ask this.
It concerns the Gradle plugin for Mirah which I have written and a potenital move under Mirah organisation on Github, where I think (and at least @felxivf) it rightfully belongs.
It is basically in 1.0-SNAPSHOT right now and I would hope to publish a 1-0-beta1 soon containing the latest fixes from @felixvf.
Once moved I can continue to help with maintenance, although it will probably require very little for the time being, besides the occasional check whether it works with newer versoins of Gradle.
@baroquebobcat any thoughts?

I think that makes sense.
On Nov 12, 2016 5:59 AM, "Schalk W. Cronjé"<EMAIL_ADDRESS>wrote:

Disclaimer: I know this might not be exacyly the right place to ask,
but I don't think the Groogle Groups for Mirah might have the traction to
ask this.
It concerns the Gradle plugin for Mirah
https://github.com/ysb33r/gradle-mirah-plugin which I have written and
a potenital move under Mirah organisation on Github, where I think (and at
least @felxivf) it rightfully belongs.
It is basically in 1.0-SNAPSHOT right now and I would hope to publish a
1-0-beta1 soon containing the latest fixes from @felixvf
https://github.com/felixvf.
Once moved I can continue to help with maintenance, although it will
probably require very little for the time being, besides the occasional
check whether it works with newer versoins of Gradle.
@baroquebobcat https://github.com/baroquebobcat any thoughts?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mirah/mirah/issues/447, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAc0ryCL-kNDTyTJM0mwQR7_wE6cMAgks5q9bhGgaJpZM4KwaXd
.


support markdown

n order to support all of those other documentation generators, we will support general documentation tokens first. Essentially unilang tokens to generate markdown.

the model is complete.  Generalized to literature.
LITERATURE
􏱭LITERARY_DEPTH􏱮
􏱕LITERARY_SENTENCE_PROPERTIES􏱖
􏱫LITERARY_TITLE􏱬

    LITERATURE_SENTENCE
    􏱟LITERARY_SENTENCE_NAME􏱠
    􏱩LITERARY_SENTENCE􏱪
    􏱧LITERARY_SENTENCE_AUTHOR_NOTE􏱨

        LITERATURE_SENTENCE_TAG
        􏱝LITERARY_SENTENCE_TAG􏱞

        LITERATURE_SENTENCE_IMAGE
        􏱥LITERARY_IMAGE_PATH􏱦
        􏱣LITERARY_IMAGE_ALTERNATIVE_TEXT􏱤
        􏱡LITERARY_IMAGE_HOVER_TEXT􏱢
        􏱓LITERARY_IMAGE_WIDTH􏱔
        􏱑LITERARY_IMAGE_HEIGHT􏱒

        LITERATURE_SENTENCE_TABLE
        􏱛LITERARY_SENTENCE_TABLE_TITLE􏱜

            LITERATURE_SENTENCE_TABLE_HEADER
            􏱙LITERARY_SENTENCE_TABLE_HEADER􏱚

            LITERATURE_SENTENCE_TABLE_ROW
            􏱗LITERARY_SENTENCE_TABLE_ROW􏱘

ok, we have a decent markdown exporter


[add] make visual studio project support wasm intellisense

Add wasm intellisense support to visual studio project,this feature only support "vsmake" plugin.
It "cheat" visual studio because visual studio does not support wasm platform.Replace the platform "wasm" to  "windows" and arch name "wasm32" to "x86".Finally， add "ClangCL" PlatformToolSet. Since we use vsmake,the operations have no risk,and the intellisense works well.
THIS PATCH LOOKS UGLY,PLEASE REVIEW IT AND GIVE ME SOME SUGGESTIONS TO IMPROVE IT!

先试下我刚说的方案，按理不需要改什么的，这种太过 hack


brew cli client cannot auth

👋 Hello friends.
Given #230 is closed and about the go version, I decided to create a new issue rather than necro the old one. But I suspect it's the same or similar issue.
Installed cli via homebrew as per the README.md: brew install axiomhq/tap/axiom
Tried to authenticate: axiom auth login
I logged in, in the browser. But it 404's.
I was already logged in, in another tab, where I created a dataset, so I know my account is fine.
? You need to login to Axiom. Should I open your default browser? Yes
Waiting for authentication...
⣻Error: API error 404: Not Found

I tried a few more times...
 ✘ alan@Alans-MacBook-Pro ~/Downloads $ axiom version
Axiom CLI, release 0.13.0 (revision: 3147c4a)
  build user:       goreleaser
  build date:       2024-06-12T15:31:42Z
  go version:       go1.22.4

 alan@Alans-MacBook-Pro  ~/Downloads $ axiom auth status
? This seems to be your first time running this CLI. Do you want to login to Axiom? Yes
? You need to login to Axiom. Should I open your default browser? Yes
Waiting for authentication...
⡿Error: API error 404: Not Found
Usage:
  login [(-a|--alias) <alias>] [(-o|--org-id) <organization-id>] [-f|--force]

Examples:
# Interactively authenticate against Axiom:
$ axiom auth login

# Provide parameters on the command-line:
$ echo $AXIOM_TOKEN | axiom auth login --alias="axiom-mycompany" --org-id="fancy-horse-1234" -f

Flags:
  -a, --alias string    Alias of the deployment
      --auto-login      Login through the Axiom UI (default true)
  -f, --force           Skip the confirmation prompt
  -h, --help            help for login
  -o, --org-id string   Organization ID

Error: API error 404: Not Found
 ✘ alan@Alans-MacBook-Pro  ~/Downloads $

echo $AXIOM_TOKEN | axiom auth login --alias="axiom-popey" --org-id="popey-towers-mmnk" -f
Error: API error 500: Internal Server Error

🥹

Confirmed broken on the go build too:
~/go/bin/axiom version
Axiom CLI, release - (revision: -)
  build user:       -
  build date:       -
  go version:       go1.22.4
 alan@Alans-MacBook-Pro  ~  ~/go/bin/axiom auth login
? You need to login to Axiom. Should I open your default browser? Yes
Waiting for authentication...
⡿Error: API error 404: Not Found

Hey friend @popey 👋
Thanks for the report!
Now that our new API backend is fully rolled out, it looks like a regression :/
Can you test this pre-release: It works for me.

@lukasmalkmus, thanks for the quick response!
Yup, that works!

I've had the pre-release CLI running for about 12 hours now. It's shipped nearly 10M rows, so that's working. Thanks again.

Perfect, I tagged a non-rc release.
Sorry for the inconvenience and thanks for the report 🙌


login page is not working

I have changed Index page to my home page and vice versa ,& landed up to some problem because of previous initialization of my code.

Change Line 104 in index.php
from
<input type="submit" **name="login"** value="index" class="btn btn-primary" />
to
<input type="submit" **name="index"** value="index" class="btn btn-primary" />


"Assign a role to a user-assigned managed identity" pargraph is not updated/wrong

The step 4 in this article refers to "Select Access control (IAM)" which leads to wrong "Add a role assignment" procedure.
The step should refer to "Azure role assignments".
[Enter feedback here]

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

ID: 1f28a934-ad9b-6f87-93ac-7dbd32363b61
Version Independent ID: 84f6b6c5-a50d-293f-0d94-1aba738a622f
Content: Manage a user-assigned managed identity in the Azure portal - Azure AD
Content Source: articles/active-directory/managed-identities-azure-resources/how-to-manage-ua-identity-portal.md
Service: active-directory
Sub-service: msi
GitHub Login: @barclayn
Microsoft Alias: barclayn

@BicoGH
Thanks for your feedback! We will investigate and update as appropriate.

@BicoGH
Thanks for your input!
Looking into your issue, the documentation is correct, since the Assign a role to a user-assigned managed identity section is specifically assigning a role to a managed identity.

The Azure role assignments option provides a list of roles assigned to the selected user or group at various scopes such as management group, subscription, resource group, or resource.
If you have any questions regarding this, please let me know.
Thank you for your time and patience!


Supporting Older Versions of React?

react-testing-library version: 2.3.0
node version: 8.1.1
npm (or yarn) version: 5.1.0
react version: 0.14.7
react-dom version: 0.14.7

Relevant code or config
const reactTestingLibrary = require('react-testing-library');

What you did:
Simply importing react-testing-library when React-DOM is less than version 0.15.5 when react-addons-test-utils was deprecated in favor of using react-dom/test-utils.
What happened:
Error message:
Error: Cannot find module 'react-dom/test-utils'
    at Function.Module._resolveFilename (module.js:485:15)
    at Function.Module._load (module.js:437:25)
    at Module.require (module.js:513:17)
    at require (internal/module.js:11:18)
    ...

Problem description:
I'm currently writing front end tests for some legacy code that had no existing tests in order to provide some security and confidence that the code will remain working as I upgrade to newer versions of libraries and improved app design.
Obviously, upgrading my version of React is a non-starter, since I'm writing these tests in preparation for doing just that.
Enzyme has failed me in multiple ways, simply being unable or unwilling to test what I need to test, though it does support React 14.
Suggested solution:
Adjust codebase to handle test-utils being retrieved from react-addons-test-utils and make any necessary adjustments to the available API under that condition. Or, provide an alternative plugin or import for react-testing-library that handles a specific older version.
At the very least, update the package.json file to warn against using versions of React prior to 0.15.5.

Hi @dwighthouse! Thanks for the issue!
What I recommend is create a file that you can import into any of your test files as if it's a node module like this one (specifically what the file is, not its contents, you can import that file anywhere in the tests as if it's a node module which is possible because of this config). With a utils file in place, you could create your own version of react-testing-library by basically copying the contents of this file and changing the things you need.
I don't want to complicate this library by adding support for very old versions of React. I think this is a pretty simple workaround.
If you find it overly complicated, feel free to fork this project and publish react-14-testing-library or something. The changes should be pretty minimal. Most of the power behind this library is in dom-testing-library anyway :)
Thanks! And good luck!


Indentation Issue in Test Example

In  https://github.com/gratifyguy/botkit-mock/blob/master/examples/convo-bot/indexController.js lines 2 and 3 and 33-43 should be indented one less.

Briefly checked this out but was unfortunately unable to see what you're referring to. The indentations look correct to me on first glance.
If you still believe they're off, please submit a PR and we'll take a look.


Accept multiple solutions

I have resolved the following issues.

https://github.com/QasimWani/LeetHub/issues/488
https://github.com/QasimWani/LeetHub/issues/427
https://github.com/QasimWani/LeetHub/issues/340
https://github.com/QasimWani/LeetHub/issues/280

@QasimWani
As this fix is straightforward and quick to review, could you please take a moment to review this pull request? I appreciate your time and consideration. Thank you!

why not timestamp instead of random number?


Selecting a map containing a string literal strips the string from select

Environment

Elixir version (elixir -v):

Erlang/OTP 21 [erts-10.1] [source] [64-bit] [smp:4:4] [ds:4:4:10] [async-threads:1] [hipe]

Elixir 1.8.1 (compiled with Erlang/OTP 21)

Database and version (PostgreSQL 9.4, MongoDB 3.2, etc.):

PostgreSQL 11.2

Ecto version (mix deps):

3.1.4

Database adapter and version (mix deps):

Postgrex 0.14.3

Operating system:

OS X 10.14.4
Current behavior
 query_1 =
      ProjectA
      |> select([sg], %{type: "project_group", value: sg.id, display: sg.name})

 ProjectB
 |> select([sg], %{type: "project_group", value: sg.id, display: sg.name})
 |> union(^query_1)
 |> Repo.all()

We are currently doing a union with a map containing a literal field in the select (similar to what is shown above), but in planner.ex, the inner "project_group" literal is being completely stripped out of the ProjectB select when being converted to a query, resulting in a non-matching number of columns. Example error below:
** (Postgrex.Error) ERROR 42601 (syntax_error) each UNION query must have the same number of columns

         query: SELECT m0."id", m0."name" FROM "project_a" AS m0 UNION (SELECT 'project_group', s0."id", s0."name" FROM "project_b" AS s0 )

I was digging around on how to fix this, but I don't think I am familiar enough with the internals.
The field is being removed in remove_literals/1 within normalize/4 because the query.select.fields entry for that column remains "project_group" without any other structure as opposed to the other fields that contain typing information.
I figure the fix is either collect_fields should be adding field metadata to the literal, or that remove_literals should be ignoring it (which seems wrong), especially because the union is preserving this field when converted to a query.
Restructuring the query as shown below produces no error.
val = "project_group"
 query_1 =
      ProjectA
      |> select([sg], %{type: ^val, value: sg.id, display: sg.name})

 ProjectB
 |> select([sg], %{type: ^val, value: sg.id, display: sg.name})
 |> union(^query_1)
 |> Repo.all()

Expected behavior
The query below should either not produce an error and function with project_group being preserved, or an error should be surfaced before the sql is created stating to pass in an interpolated variable instead.
 query_1 =
      ProjectA
      |> select([sg], %{type: "project_group", value: sg.id, display: sg.name})

 ProjectB
 |> select([sg], %{type: "project_group", value: sg.id, display: sg.name})
 |> union(^query_1)
 |> Repo.all()

Thanks @TylerWitt! Can you provide a failing test case? If you can do, it would speed up the bug fixing process a lot. You can add a unit test to test/ecto/query/planner_test.exs. :)

@josevalim I added a test to show the discrepancy I was seeing, though I'm not sure that it is necessarily the entire issue, or if we should just be interpolating our strings everywhere and it just worked coincidentally until now. We just updated from 3.0.0 to 3.1.4, and this caused some of our tests to fail.

Looking at 3.0.0, it seems that the bug is the literal not being removed from the union.

@josevalim I made a fix for that test in #2998 as well that seems to fix the issue in our tests as well.


Would be nice to extract absurd into more self-contained modules

I feel that absurd currently is a little "too big", trying to solve multiple use cases such as html component, templating and css via javascript. Would be nice to split it up into at least these three modules:
absurd-html
absurd-css
absurd-components
or something along those lines. I'm aware this would likely require quite a bit of refactoring, but I'm sure it would be worth it for ppl who use it... Now I will look into DaddaJS ;)

+1 Would love absurd.js much more if it just did CSS pre-processing. Other features should be separate projects.

+1 for splitting into dedicated projects like absurd-html etc.

Well, guys I'm thinking about rewriting absurd and indeed maybe creating new modules. Still keep the name "absurd".

:+1:


Hide play button until video has loaded

With a network video, sometimes the video can take a couple seconds to load. The play button still shows up during this time and the user can press it. I can use a placeholder to show a CircularProgressIndicator but the play button sits in front of it, hiding it.
It would be great if the play button was hidden until the video had finished loading/initializing.

Until there is some callback for this you have to do it by yourself. You could do something like this: ea8b62a577228145d79face28704dbb760459

This was fixed in Chewie version 0.7.0

@Sub6Resources where can I find Chewie version 0.7.0?

@hanns2002 The latest Chewie version includes these fixes. If you'd like to specifically use 0.7.0 in your app (which isn't recommended) you can set that in your pubspec.yaml instead of the most recent version.


Sidebar width is not persisted

The open state of the sidebar is persisted but the width is not. It recognized the width and parsed it correctly. It also doesn't overwrite the data.

Fixed in 51a7d8b8cb08300725a64a1d13f336449c8b1a78


Hover animation? Question not a issue..

Hello,
I'm new on GitHub and I immideatly wanted to ask something about this project. I now know how to animate things but I want to animate something if I ask on it.
BTW: This project is really in one word: awesome!
Kind regards,
Musa.

Welcome on GitHub,
The effect you want to chieve could be done by javascript(JQuery). For example you popup notification(#notification). If you click on that it fade out. You simply listen to click event. after that add desired effect with .addClass method. Do not forget add class animated before you add your desired effect.
It is also described in readme https://github.com/daneden/animate.css#usage
Hope it helps

check below sample.

$(ABC).on('hover',function(e){
$(target).toggleClass('animation...');
});
//'ABC' is the element where you want to hover
//'target' is the element you want to apply 'animations'

@CylonSK @Yeswanth-JG @Hero-Appelmoes it’s 2015 and we’re still talking about JS hover events?
Here’s how I’d recommend animating an element on hover:
.element:hover {
  animation: bounce 1s;
}

Don’t forget vendor prefixes (-webkit-animation, -moz-animation, etc.) may be necessary.

Yes it's 2015 an we are still talking about JS because he did not ask about hover, but how to implement animate.css , so JS is still the best answer.
Also I would recommend using hover with JS, because it is better to have all animation together for maintenance purposes.


Few fixes for wait_for_instantlock

Pls see individual commits for details.
Note: p2p-instantsend.py is expected to fail randomly now because of propagation issues which were not caught previously (should be fixed via #3124).
Note2: Also needs #3122 to be merged first to import wait_until, will rebase

Needs rebase


New package: OpenHPSDR-TAPR.PowerSDR version <IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Closes #115536.
Tested and works fine.
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/116358&drop=dogfoodAlpha

It looks like that all checks for both PowerSDR and Thetis have passed.
I have decided that it would be best to merge both of these packages since they all have their own slightly different feature set and hardware support. :-)


Service manager enhancements

Description
This PR includes several enhancements noted by @OmarElawady in #1547

Service jobs interval was not accurate. It was the period between finishing one job and running another one. Now it's the interval between 2 successive runs of the job. This is now fixed by separating scheduled greenlets from running greenlets.
Added support for crontab-format service intervals (day of the week, month, day of the month, hour, minute). Now services can set their interval to interger (seconds) / CronTab object (CronTab(" * * * * * ")) / CronTab-formatted string (" * * * * * ")
Added the option to stop greenlets gracefully or killing it without waiting
Handled exceptions raised by greenlets to raise alerts

Tests
Correct intervals
stellar service interval = 25
disk-check service interval = 15

CronTab intervals
stellar service interval = "* * * * *"  - string formatted like CronTab format
disk-check service interval = CronTab("* * * * *")  - CronTab object

Running and scheduled services

Stop service gracefully

Kill service without waiting

Related Issues
https://github.com/threefoldtech/js-sdk/issues/1547

Codecov Report

Merging #1554 into development will decrease coverage by 3.03%.
The diff coverage is 44.73%.

@@               Coverage Diff               @@
##           development    #1554      +/-   ##
===============================================
- Coverage        35.88%   32.84%   -3.04%
===============================================
  Files              127      127
  Lines            11606    11641      +35
===============================================
- Hits              4165     3824     -341
- Misses            7441     7817     +376

Impacted Files
Coverage Δ

jumpscale/packages/admin/actors/admin.py
32.51% <22.22%> (-1.06%)
:arrow_down:

jumpscale/tools/servicemanager/servicemanager.py
72.46% <51.72%> (-10.87%)
:arrow_down:

jumpscale/clients/explorer/pagination.py
11.76% <0.00%> (-82.36%)
:arrow_down:

jumpscale/tools/notificationsqueue/__init__.py
33.33% <0.00%> (-66.67%)
:arrow_down:

jumpscale/sals/zos/__init__.py
21.05% <0.00%> (-63.16%)
:arrow_down:

jumpscale/clients/explorer/auth.py
25.74% <0.00%> (-55.45%)
:arrow_down:

jumpscale/clients/explorer/explorer.py
51.21% <0.00%> (-43.91%)
:arrow_down:

jumpscale/clients/explorer/base.py
57.14% <0.00%> (-42.86%)
:arrow_down:

jumpscale/clients/explorer/workloads.py
27.50% <0.00%> (-41.25%)
:arrow_down:

jumpscale/packages/admin/services/stellar.py
53.84% <0.00%> (-34.62%)
:arrow_down:

... and 24 more

Continue to review full report at Codecov.

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

CI needs to pass first

Fixing reloading packages with services

Fix package reloading with services


allow the user variable to be leveraged

What is the purpose of this pull request?: Allows the $USER variable to be leveraged in the Dockerfile (was currently unused).
What issue does this pull request address?: Fixes # N/A
Notes for reviewers to consider: N/A
Specific reviewers for pull request: @gardlt

LGTM


startup.jl doesn’t get loaded every time

When I start Atom and it automatically loads the last project, startup.jl gets executed fine.
Since today I have two Julia projects an Atom. When I now open Atom and go for File ⇒ Reopen Project ⇒ <Some Project>, the startup.jl file doesn’t get executed when the terminal gets “restored”(?). I need to Ctrl + D to quit that initial Julia session and press Enter for opening a new Julia session – then my startup file gets executed. Can this be fixed, so that it also gets executed directly when reopening projects?
julia-client v0.7.7
uber-juno v0.2.0
Atom v1.31.1

startup.jl is definitely executed every time we start Julia. That works fine for me when using Reopen Project.
What OS are you on? I know that Atom handles projects differently on Macs, so maybe it doesn't kill the currently running Julia process or something like that?

Fedora 28. Do you need more information?


[Feature]: Gradient Boosting Algo

Feature Name
Gradient Boosting Algorithm
Feature Description
Gradient Boosting is a machine learning algorithm that iteratively trains weak models (often decision trees) and combines them to create a strong predictive model. It works by fitting a new model to the residuals of the previous models, focusing on the areas where the current model is making mistakes. This iterative process helps to improve the overall accuracy and performance of the model.
Motivation
Gradient Boosting is often known for its ability to achieve high accuracy, especially on complex datasets. By combining multiple weak models, it can capture intricate patterns and relationships that might be missed by a single model.
Implementation Suggestions (Optional)
No response
Feature Type
New Algorithm
Does this feature require additional resources?

[ ] Additional Documentation
[ ] Third-Party Libraries
[ ] UI/UX Design
[ ] Other

References (Optional)
No response

open document issues for docs


[5.2] Forget key when key exists

When using for example ip addresses as keys for an array and try to use array_forget() or Arr::forget() it wil not check if the key exists in the array before exploding the key.
This wil, like Arr::get() or Arr::has(), first check if the key exists.

Could you add tests please?


Invalid token - see #41

Environment
PNPM monorepo, default webpack.config.js.
Windows 10.
Description

See issue #41.
Works in release and in Chrome debug.
Reproducible Demo

Set up a PNPM monorepo with a react-native package.

Not a bug in repack - just didn't include my expo modules in the transpiling -_-


Create development.jc

_ No description provided. _

@Jyotsana2005-stack Please add problem statement and tech stack and used resources.

Yeah!! sure

Yeah!! sure

@Jyotsana2005-stack Can you show me the snapshot of output ?


ERROR  [HPM] Error occurred while trying to proxy request domain to api domain

I'm using proxy-module. When project mode PRODUCTION. I see error log.
ERROR  [HPM] Error occurred while trying to proxy request [path url domain api project] from [my domain] to https://domain/Api (ECONNRESET) (https://nodejs.org/api/errors.html#errors_common_system_errors)
How to fixed it?

@standfv did you fix it?
same problem here :(


Move sanction module into the provenance repo

Summary
Move sanction module into the provenance repo (from our fork of the SDK).

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Moving them into their own repo instead: #1768

Changed my mind. Gonna move it into the provenance repo. Re-opened as #1901


Retrying a download in chrome makes it download the index page

Not sure this is a droppy issue, but if you launch a download, stop it, then press retry, chrome will download a download.htm file from the path /
If droppy can do anything to fix this, it could be nice. But it's quite a minor issue...

Weird, download URLs should be static.

Accessing the URL again works fine, it's only when clicking retrying.
Perhaps the /!/ in the URL?


Stale metrics for prometheus_notifications_sent_total

What did you do?
If prometheus was connected to one alertmanager and successfully sent alerts, but after that the addresses of alertmanagers have changed (new kubernetes have been rolled out), then in the metrics prometheus_notifications_.* there will be metrics on old alertmanager.  Is it possible to remove metrics from unused alertmanager when reloading prometheus?
What did you expect to see?
When we re-create pods with new addresses, we want old alertmanager metrics to disappear from prometheus.
What did you see instead? Under which circumstances?
Prometheus continues to give metrics about old alertmanager's: http://joxi.ru/GrqqgplHk8dYnr?d=1
Because of the dependent metrics, we can't now understand if there are alertmanagers we use that we can't send the metrics through.
Environment

System information:

Linux 5.3.0-28-generic x86_64

Prometheus version:

prometheus, version 2.16.0 (branch: HEAD, revision: b90be6f32a33c03163d700e1452b54454ddce0ec)
  build user:       root@0fdd9501c22e
  build date:       20200220-10:44:36
  go version:       go1.13.8

Alertmanager configuration file:

alerting:
  alertmanagers:
  - kubernetes_sd_configs:
    - role: endpoints
      namespaces:
        names:
        - monitoring
    scheme: http
    path_prefix: /
    timeout: 10s
    api_version: v1
    relabel_configs:
    - source_labels: [__meta_kubernetes_service_name]
      separator: ;
      regex: madison-proxy
      replacement: $1
      action: keep
    - source_labels: [__meta_kubernetes_pod_container_port_number]
      separator: ;
      regex: "80"
      replacement: $1
      action: keep

Counters don't become invalid due to the thing they refer to no longer existing, using the erors metrics should catch this.

Hello from the bug scrub.
Revisiting this, we have to say that traditionally, it's a good practice that a binary never removes any metrics that it has once exposed. A counter that isn't moving anymore can just stay around. It's rate will be zero, and that's how you can see that nothing is happening there anymore.
Having said that, it has become quite common for more complex programs with several subsystems that come and go, to remove metrics related to a certain instance of a subsystem once that instance is deleted. So this could also be done for Alertmangers to avoid cluttering the metrics of a Prometheus servers with more and more AM's if the latter have a high churn. @bboreham has drafted a PR how to do this. Let's see what reviewers think.


[nrfconnect] Recommend using west flash --erase

Problem
Using bare west flash, without erasing the entire flash
area, may lead to weird behaviour when switching between
different nRF Connect SDK/Zephyr/OpenThread revisions.
It's safer and faster to use west flash --erase.
Summary of Changes
Recommend using west flash --erase in nRF Connect examples documentation.
Fixes #5782

Why does it cause weird behavior?
This will be an issue when we have OTA upgrades..

@mspang Usually the problem appears when someone flashes CHIP example to the device which was flashed with vanilla OpenThread example - that does not use NCS/Zephyr and has different non-volatile data format - but let me double-check if NCS revision change may also trigger the issue. We'll look if we can detect such a situation earlier and assert with the proper log.

That wasn't the case in #5782 though. Hence the concern.


Missing Fog

In vanilla Minecraft, there usually exists a layer of fog along the horizon, which helps to hide chunks loading.
This seems to have been broken or deliberately disabled back in the original Colored Lights mod:

I imagine it was planned to have fog render again at some point, but also that it wasn't a priority.
You can still see the fog in the skyline, just not on actual terrain.

Huh, never noticed. I'll look into it, but it's not high priority.

1.7.10 version is out of date, the rework is written from scratch, so I close this bug.


Name property not included in new column after insertColumn

When inserting a new column the 'name' is not included. That appears to be important for the display of data (ordering of columns). If the data is saved and try to reload on another sheet, it is not showing in the same way.
From the config:
"columns":[
  {"type":"text","name":"0","source":[],"options":[],"editor":null,"allowEmpty":false,"title":"","width":50,"align":"center"},
  {"type":"text","source":[],"options":[],"width":50,"align":"center"},
  {"type":"text","name":"2","source":[],"options":[],"editor":null,"allowEmpty":false,"title":"","width":50,"align":"center"},
  {"type":"text","name":3,"source":[],"options":[],"editor":null,"allowEmpty":false,"title":"","width":50,"align":"center"}

 ],

Above the second row was an insert and 'name' is not present

This behavior is correct, and the name should be added using oninsertrow() in case you need. This is because the name is not necessarily a sequence number. If you are using it as a number, I suggest you to remove the name at all.


Update vertical.md

Type of change

[x] Add new papers (Please tell us why you think this paper is awesome!)
[ ] Fix the category of an existing paper/papers (Please tell us the reasons)
[ ] Add a new tool/primitive/application with a new markdown page (Thank you! Also, please tell us more about this awesome thing!)

Description

Please include a summary of the change and the relevant motivation.

I'd like to introduce one of my works on vertical federated learning, which focused on Non-IID problem.
In many application domains, especially the financial domain we discussed in the paper, the data for binary classification are heavily imbalanced, which is an important cause of poor performance in vertical federated learning.
This paper utilized widely used cryptographic techniques and conducted FEDERATED data under-sampling in the PSI procedure,
the metrics of downstream modeling tasks showed that it is an effective approach for mediating the imbalance problem.

I have read the CLA Document and I hereby sign the CLA

recheck

well, okay


Add note about importing with Phoenix

Even if the Phoenix team decides not to import build by default it might make sense to include this since people that already have a generated Phoenix application would have the Eco.Model import.

One comment. Otherwise :+1:


Announcements

[ ] Create
[ ] Read
[ ] Mark as read
[ ] Delete

API complete for creating, reading, and marking as read.


[HELP] error: failed to merge profile data: process didn't exit successfully

I have been trying to add coverage part to a Rust project that runs tests with nextest. Tests are running correctly but I am not being able to debug the following error:
zero
ci_tests  | ------------
ci_tests  |      Summary [  27.398s] 33 tests run: 33 passed, 1 skipped
ci_tests  | warning: not found *.profraw files in /app/target/llvm-cov-target; this may occur if target directory is accidentally cleared, or running report subcommand without running any tests or binaries
ci_tests  | error: no input files specified. See llvm-profdata merge -help
ci_tests  | error: failed to merge profile data: process didn't exit successfully: `/usr/local/rustup/toolchains/1.70.0-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-unknown-linux-gnu/bin/llvm-profdata merge -sparse -f /app/target/llvm-cov-target/app-profraw-list -o /app/target/llvm-cov-target/app.profdata` (exit status: 1)
ci_tests  | [cargo-make] ERROR - Error while executing command, exit code: 1
ci_tests  | [cargo-make] WARN - Build Failed.

Simplified docker file:
ARG RUST_VERSION="1.70"
ARG RUST_IMAGE="rust:${RUST_VERSION}-bullseye"

FROM ${RUST_IMAGE} as install
RUN cargo install cargo-chef

FROM install as planner
COPY . .

RUN cargo chef prepare --recipe-path recipe.json

# -- Builder with cache layer --
FROM install as builder
COPY --from=planner /app/recipe.json recipe.json
RUN cargo chef cook --workspace --recipe-path recipe.json

COPY . .
RUN cargo build --workspace

# -- Builder with cache layer --
FROM ${RUST_IMAGE} as base
RUN cargo install cargo-make cargo-nextest

RUN rustup toolchain install $RUST_VERSION --component llvm-tools-preview
RUN cargo install cargo-llvm-cov

FROM base as runtime
WORKDIR /app
COPY --from=builder /app/target /app/target
COPY . .
RUN cargo nextest archive --workspace --archive-file test.tar.zst

ENTRYPOINT ["cargo", "make", "docker-ci"]

Command that is run
rm -r /app/target
cargo llvm-cov --json --output-path /app/docker/tests/cov.json nextest --archive-file test.tar.zst --test-threads 1

Versions
cargo-llvm-cov v0.6.4

RUN cargo nextest archive --workspace --archive-file test.tar.zst

You have to use cargo llvm-cov nextest-archive, like:
https://github.com/taiki-e/cargo-llvm-cov/blob/aa02184b4c999b05cc17ba7f92e2026d8fc4dc20/.github/workflows/ci.yml#L86-L87
(We should perhaps expand the documentation of nextest to explain this or enhance our own documentation on nextest.)

The issue with using cargo llvm-cov nextest-archive is that it recompiles everything again.
I have tried without success.
COPY --from=builder /app/target /app/target/llvm-cov-target/
COPY . .
RUN cargo llvm-cov nextest-archive --workspace --archive-file test.tar.zst

ENTRYPOINT ["cargo", "make", "docker-ci"]

Note that i mapped the build to /app/target/llvm-cov-target/
We use a custom build command for a package and using nextest-archive does throw error

The issue with using cargo llvm-cov nextest-archive is that it recompiles everything again.

This is a fundamental limitation of Rust's current code coverage (regardless of which way you use it). There are several RFCs and proposals to improve this, but none have been approved yet.

We use a custom build command for a package and using nextest-archive does throw error

I don't have any information about the project you are building, so I don't know the details, but it is possible that the old build artifact is doing something or the fact that rustflags is being applied to the build script is causing some problems. (The former can be addressed by running cargo clean first, the latter by passing --target option.)

Thanks for your responses, I will try to investigate a bit more about this topic & avoid recompiling


[DOC] A docker-compose example command to correct on OpenSearch docker installation web page

What do you want to do?
An existing example docker-compose command shown on https://opensearch.org/docs/latest/opensearch/install/docker/ is a bit mistakenly placing the arguments 'up' and '-f /path/to/your-file.yml' in an incoorect order. If user just do copy&paste, the command fails.
Insted of (current) :  docker-compose up -f /path/to/your-file.yml
Should change to  :  docker-compose -f /path/to/your-file.yml up

[X] Request a change to existing documentation

Tell us about your request. Provide a summary of the request and all versions that are affected.
Suggesting to correct the order of arguments placement
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.

@JeffH-AWS , if this is something you can help with ... Thanks!!

Hello @jordarlu and thanks for raising this typo! Fix pending.

Thank you, @JeffH-AWS


API isn't decoding properly in Python 3

I'm running the application on top of Python 3 and querying it via curl, but the application is crashing with the following error:
Error on request:
Traceback (most recent call last):
  File "/Users/grant/.virtualenvs/blocks/lib/python3.5/site-packages/werkzeug/serving.py", line 209, in run_wsgi
    execute(self.server.app)
  File "/Users/grant/.virtualenvs/blocks/lib/python3.5/site-packages/werkzeug/serving.py", line 197, in execute
    application_iter = app(environ, start_response)
  File "/Users/grant/.virtualenvs/blocks/lib/python3.5/site-packages/durable/interface.py", line 109, in __call__
    return endpoint(environ, start_response, **values)
  File "/Users/grant/.virtualenvs/blocks/lib/python3.5/site-packages/durable/interface.py", line 97, in _default_facts_request
    message = json.loads(request.stream.read())
  File "/Users/grant/.pyenv/versions/3.5.2/lib/python3.5/json/__init__.py", line 312, in loads
    s.__class__.__name__))
TypeError: the JSON object must be str, not 'bytes'

This seems to be a simple encoding issue, with Python 2/3 handling string/unicode types differently. It works fine on Python 2.7.
I was able to fix the issue on both Python 2/3 (at least on my machine) by just appending a simple .decode('utf8') to the API interface.
Is @jruizgit able to replicate and are you open to PR's? If so, I can write one up and submit it for review.

Hi, thank you for reporting the problem. Yes, please send me a pull request and I will merge it.

Hi @grantbachman, I could not repro the problem. I used python 3 in macos platform. What platform are you using? Would you mind pasting your curl command?
I have added the utf-8 decode and uploaded a new version to pypi.

I'm running Python 3.5.2 on Mac OS. I was just testing with the example located here. Your newest release (0.33.114) has fixed the decoding issue for me. Thank you!


add properities

This project implements a subset of CSS，can add other properities

This request doesn't make any sense, please formulate it better. Closing until you provide better description.


Is it a good idea to keep defect lists as separate values for all the ExitStatus branches?

It may be nice to keep consistency and allow the user to easily distinguish between defects and causes when the evaluation terminates. I've opened a P/R for this: #169
Also I'm curious what happens when computation succeds but bracket finalizers fail?
It seems logical to return the value computed. At the same time in case it happens user may wish to get a list of defects as well. So I think it may be a good idea to add defects list into ExitStatus.Completed as well. What do you think?

I think the redesign of ExitResult solves this issue?

Hmm, please correct me if I'm wrong, but I think we still can't tell which of the unchecked exceptions are the causes of termination and which of them were collected from finaliser failures ._.

Fixed in the new lossless error model


Add note regarding current disabled status of mDNS

#657

I'm also wondering if this is true of the android chrome implementation. I didn't have an mdns server to hand that would allow me to generate the structure that avahi would, as the one I had was too strict.
Across the board, it'd be nice to know what is and isn't supported by the chrome implementation, especially as the chrome implementation links to this repo in docs. It appears by experimentation that ssdp and bluetooth work in chrome the way they are described here.

The android chrome implementation doesn't have mdns support.  This is a test bed and will typically have always have features not in the chrome implementation.  ssdp also doesn't work in chrome...not sure why it appears to for you.
I'm not sure the docs in this repo are the best place to describe the feature sets of other PW clients.  It's something we can consider though.

e.g. https://support.google.com/chrome/answer/6239299 links to this repo.
I thought I saw something from developers.google.com too, but I may have got my wires crossed.


Body component

Codecov Report

Merging #67 into master will decrease coverage by 11.76%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           master      #67       +/-   ##
===========================================
- Coverage     100%   88.23%   -11.77%
===========================================
  Files           2        9        +7
  Lines          16       34       +18
  Branches        2        2
===========================================
+ Hits           16       30       +14
- Misses          0        4        +4

Impacted Files
Coverage Δ

src/components/Body.js
100% <100%> (ø)

src/components/HistoryCaseRevision.js
40% <100%> (ø)

src/components/BackButton.js
100% <0%> (ø)

src/components/StationName.js
100% <0%> (ø)

src/components/TickButton.js
66.66% <0%> (ø)

src/components/CaseTitle.js
100% <0%> (ø)

src/components/Header.js
100% <0%> (ø)

Continue to review full report at Codecov.

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


Initial level do not work

set initial level (doesn't matter if 1 or 100%)
close app (force close to be sure. doesn't change behaviour)
open app.
app will open, show the initial level correctly on the bar graph.

what happens:

led flash the initial level for a brief moment and remain off.

what i expected:

led to remain ON at the initial level

Work around:

simply interacting with the app in any way restore functionality.

device tested: pixel 6 pro
tested version: 1.4.0
Reverting to 1.3.7 fixes the issue.

Issue reproducible, will hopefully be fixed in a few days.

(Hopefully) Fixed in Version 2.0.0.
Please also read the breaking changes for this release.
Can you confirm?

finally had time to upgrade. Confirm it is working on the troubled devices with 2.0.0!
Also, I did not have to uninstall the previous version. It even kept my preferences (auto start, level). Only lost the shortcut i had pointing to 1.x version, but that's very minor.

@gcb Awesome!


Validated Cell Type Descriptions

This story captures the user stories for when cell type chatgtp descriptions are validated via our biocurator team.
User Stories:
As a user, I want to see validation info (icon and disclaimer) for the cell type description I am viewing.
As a user, I want to references that are attributed to the validated cell type description.
As a user, I want to see a link to Cell Ontology to view their cell type definition.
As a user, I want to see a logo of the Cell Ontology next to their link - https://user-images.githubusercontent.com/112839/107354015-9375bb00-6ac5-11eb-9394-7318df343ec2.png
As a user, I want to see only 1 type of description when a cell type description is validated.
As a user, I want do not want to see the chatgtp source for any validated cell types.
As a user, I want do not want to see the Header - Experimental Description for any validated cell types.
@hthomas-czi to add designs

As a user, I want to see a logo of the Cell Ontology next to their link

Do users really want that?

It's a request from David on the CL side, I'm thinking it might be a small
favicon store logo
On Fri, Jul 21, 2023, 11:53 AM Harley Thomas @.***>
wrote:

As a user, I want to see a logo of the Cell Ontology next to their link
Do users really want that?
—
Reply to this email directly, view it on GitHub
https://github.com/chanzuckerberg/single-cell-data-portal/issues/5301#issuecomment-1646114199,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2S5ZBBM2ZASX4RQPVZ2OHLXRLF35ANCNFSM6AAAAAA2SPQWIY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Sure. Just want to be clear about whether requirements are actually user needs.
I'm not sure that having a small logo next to the link will work though. Their logo doesn't work well at small sizes.
We could consider having an "Acknowledgments" section that has a list of contributors with their logos. That would be a good incentive for attracting partners.

Thanks, Harley - that works, an acknowledgments section would be great.
Probably a secondary ticket?
On Fri, Jul 21, 2023 at 1:08 PM Harley Thomas @.***>
wrote:

Sure. Just want to be clear about whether requirements are actually user
needs.
I'm not sure that having a small logo next to the link will work though.
Their logo doesn't work well at small sizes.
We could consider having an "Acknowledgments" section that has a list of
contributors with their logos. That would be a good incentive for
attracting partners.
[image: CL Logo]
https://user-images.githubusercontent.com/72052788/255274247-a33b21b9-182f-4ead-b911-699a8c9bccdd.png [image:
Acknowledgments]
https://user-images.githubusercontent.com/72052788/255274242-9f1ffa4d-e67b-430c-95ac-a20572c4f524.png
—
Reply to this email directly, view it on GitHub
https://github.com/chanzuckerberg/single-cell-data-portal/issues/5301#issuecomment-1646186785,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2S5ZBBNPUBRIRXEKYLTERTXRLOU5ANCNFSM6AAAAAA2SPQWIY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Update 8/31: 90 descriptions ready so far. Time to get them onto CellGuide.


Designing and developing the footer

the UI of the footer can be improved.

sir ,I want to work on this issue to improve the footer designing and developing. (GSSoc23)
thanks you


Revert PR#3663

Revert reason, the PR causes the new issue #3770 .
I think we need to revert and use DDL of encrypt to fix #2884 .

HI @xiyelife , After the PR merged, there are some error for you added in #3768 .
Can you update the test case so that it can run correctly?

@xiyelife Hi boy, we need your support. This issue blocks our coming release!
This PR rollbacks one commit to solve #3770 , however one unit test in (your PR)[https://github.com/apache/incubator-shardingsphere/pull/3768] can not run well.
Currenty we have to ignore this unit test, so could you give a look at it and make it run well again?
Looking forward to your reply, Thx.

@tristaZero  yeah, I can make  my pr https://github.com/apache/incubator-shardingsphere/pull/3768 run well, but how to do it with this revert PR and waiting for it has been merged to dev , then submit a new PR?

@tristaZero yeah, I can make my pr #3768 run well, but how to do it with this revert PR and waiting for it has been merged to dev , then submit a new PR?

Yes, after this pr merged, init table SQL INSERT INTO t_encrypt_contains_column VALUES(1, 'cipherValue', 'plainValue', 'cipherValue', 'plainValue') will execute error.

@KomachiSion when your revert PR is merged, I'll sumbit a new PR to fixed it and add my unit test back.

@KomachiSion when your revert PR is merged, I'll sumbit a new PR to fixed it and add my unit test back.

Thanks.

@xiyelife Hi so now It’s your turn. :）

@xiyelife Hi sir, i notice that you have do a few of contributions to shardingsphere community. Very appreciated your effort.
Just out of my curiosity, do you enjoy it and also want to do more to this community? From my side, you are energetic and have a bit clear understand of sharding sphere. So i am expecting your successive interactions in sharding sphere community and to be our  committer one day. Good luck and welcome!


{Misc.} Update Track2 status

Description

Update Track2 status in doc.
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Track2 status


invalid enum variants with dot (.) should be replaced

paperclip currently generates enums with a dot (.), which is not valid in rust:
#[derive(Debug, Clone, Serialize, Deserialize)]
#[allow(non_camel_case_types)]
pub enum ServiceOrderMigrationUseCase {
    #[serde(rename = "FTTH1.7")]
    Ftth1.7,
}

The dot should be replaced by an underscore:
#[derive(Debug, Clone, Serialize, Deserialize)]
#[allow(non_camel_case_types)]
pub enum ServiceOrderMigrationUseCase {
    #[serde(rename = "FTTH1.7")]
    Ftth1_7,
}

Fixed as part of #344


Lack of Search Function

The default theme does not appear to include a search function. Is there a way to add search?

Hi. As Hugo is a static site generator, there is no default way to add search functionality. However, Hugo docs have some recommendations on how to achieve this.

I am not planning on adding any of those soon. PRs are welcome! so if you end up adding one of those to your project please consider submitting it as a PR for the theme.


Update README.md

Include NVM caveats in macos install section.  This PR is rather plain and mac-only, but I thought a PR would be nicer than an issue report.  The issue likely applies to other platforms and perhaps you want to have a separate section for NVM or install caveats in general, rather than in-line in the install bit.
To clarify the issue, when I copy/paste the install commands, I get this:
chmod: /usr/local/lib/node_modules: No such file or directory

because my node_modules folder is actually at:
~/.nvm/versions/node/v14.2.0/lib/node_modules

Feel free to reject the PR and create an issue if that is more appropriate, or I can create the issue if you like.

closing this, as there are a lot of ways to install Node and to keep the instructions simple for non-tech-savvy users I'm not including information about anything other than the main installer. I will add some clarity to the README, though.


list entity age

functionality

list companies with age of last update
list projects with age of last update
list people with age of last update

implementation

this can run in the respective folders. The command simply looks at the last update date within the given child folder
this could be based on https://github.com/nushell/nushell

related

#34

This can be done with a Git command:
git ls-files -z | xargs -0 -n1 -I{} -- git log -1 --format="%ai {}" {}


Added additional fields provided by HubSpot

Hi - I've just added support for some more fields returned from Hubspot's API.  The metaData field is particularly important because it carries the GDPR / consent compliance form items.

Awesome!

Awesome!


Revert "Fix nulls in notification output for primary branch Docker stack deployments"

Reverts cfpb/cfgov-refresh#5889

We don't need to revert the previous PR, just iterate forward in #5898 .


FIX: ListItem icon align

LiveURL: https://orbit-components-fix-listitem-align.surge.sh

Resolved.


Add docker-compose file

Fixed with https://github.com/d-Rickyy-b/certstream-server-go/commit/c96f94c4c9fe4ec1f665d1bd7fcc5673c495708e?

Implemented in c96f94c4c9fe4ec1f665d1bd7fcc5673c495708e


Implement finite-difference pricers.

I'd take this one. Do you have some references for the implementation?

@avhz I also found a chapter in implementing quantlib. I'm reading, we will see soon.

You can take a look here, specifically at the PDE_European.jl and PDE_American.jl files. I worked on this very problem a while ago, albeit, in Julia. Feel free to translate this code to Rust.

You can take a look here, specifically at the PDE_European.jl and PDE_American.jl files. I worked on this very problem a while ago, albeit, in Julia. Feel free to translate this code to Rust.

Thanks. I'm a little bit busy at work but I'll take a look!

Hey @SimonG85
Would it be ok if I take this issue? I was able to make some progress on it. I would very happy for you to look at the PR once it is ready.

@yfnaji yeah you can take it for sure!

@yfnaji
In the latest commit, I tried ATM option tests, and there a number of failures. In particular, the explicit() method seems to have an issue.
---- instruments::options::finite_difference_pricer::tests_finite_difference_pricer::european_call_explicit stdout ----
thread 'instruments::options::finite_difference_pricer::tests_finite_difference_pricer::european_call_explicit' panicked at src/instruments/options/finite_difference_pricer.rs:503:9:

Left:           NaN,
Right:          2.179260421286685,
Precision:      0.000000014901161193847656

note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

---- instruments::options::finite_difference_pricer::tests_finite_difference_pricer::american_put_explicit stdout ----
thread 'instruments::options::finite_difference_pricer::tests_finite_difference_pricer::american_put_explicit' panicked at src/instruments/options/finite_difference_pricer.rs:488:9:

Left:<PHONE_NUMBER>3232840000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000,
Right:          1.74684769403327,
Precision:      0.000000014901161193847656

---- instruments::options::finite_difference_pricer::tests_finite_difference_pricer::american_call_explicit stdout ----
thread 'instruments::options::finite_difference_pricer::tests_finite_difference_pricer::american_call_explicit' panicked at src/instruments/options/finite_difference_pricer.rs:473:9:

Left:           inf,
Right:          2.179260421286685,
Precision:      0.000000014901161193847656

---- instruments::options::finite_difference_pricer::tests_finite_difference_pricer::european_put_explicit stdout ----
thread 'instruments::options::finite_difference_pricer::tests_finite_difference_pricer::european_put_explicit' panicked at src/instruments/options/finite_difference_pricer.rs:518:9:

Left:           NaN,
Right:          1.691554666293824,
Precision:      0.000000014901161193847656

---- instruments::options::finite_difference_pricer::tests_finite_difference_pricer::american_call_implicit stdout ----
thread 'instruments::options::finite_difference_pricer::tests_finite_difference_pricer::american_call_implicit' panicked at src/instruments/options/finite_difference_pricer.rs:478:9:

Left:           2.771814089100344,
Right:          2.179260421286685,
Precision:      0.000000014901161193847656

---- instruments::options::finite_difference_pricer::tests_finite_difference_pricer::european_call_implicit stdout ----
thread 'instruments::options::finite_difference_pricer::tests_finite_difference_pricer::european_call_implicit' panicked at src/instruments/options/finite_difference_pricer.rs:508:9:

Left:           2.771814089100344,
Right:          2.179260421286685,
Precision:      0.000000014901161193847656

---- instruments::options::finite_difference_pricer::tests_finite_difference_pricer::european_call_crank_nicolson stdout ----
thread 'instruments::options::finite_difference_pricer::tests_finite_difference_pricer::european_call_crank_nicolson' panicked at src/instruments/options/finite_difference_pricer.rs:513:9:

Left:           0.8579231906354501,
Right:          2.179260421286685,
Precision:      0.000000014901161193847656

@avhz thanks for bringing this to my attention - I am currently investigating and will update as soon as I find something


Top 10 things for a JS open source project should do from a security perspective

Using Node.js as an example, what would OpenSSF or other security experts advise a project to do from a security perspective.
Based on comments from @mhdawson from a recent meeting.

This issue has been replaced by https://github.com/openjs-foundation/security-collab-space/issues/151


Delete value thru dashboard and api console

I cant delete my data from my dashboard.. also in api console. everytime i try to delete this thing shows up
error: Error handling request: ParseError {
code: 107,
message: 'cannot route DELETE \classes\Shift\Rtyl9b8wQK' } code=107, message=cannot route DELETE \classes\Shift\Rtyl9b8wQK
here is my server log

same problem to me
did somebody solve it?


Permutations support required

Hula-hoop has an issue with the multiple permutations on Zeus projects where hybrid is set to true.
/Users/jherri1/master/pharmacy/node_modules/hapi/node_modules/hoek/lib/index.js:663
    throw new Error(msgs.join(' ') || 'Unknown error');
          ^
Error: Invalid route options ([object Object]) {
  "method": "GET",
  "path" [1]: {
    "children": {
      "$hybrid:android;": "android",
      "$hybrid:ios;": "ios",
      "$hybrid:browser;": "browser"
    }
  }
}

[1] path must be a string
    at Object.exports.assert (/Users/jherri1/master/pharmacy/node_modules/hapi/node_modules/hoek/lib/index.js:663:11)
    at Object.exports.assert (/Users/jherri1/master/pharmacy/node_modules/hapi/lib/schema.js:15:10)
    at new module.exports.internals.Route (/Users/jherri1/master/pharmacy/node_modules/hapi/lib/route.js:39:12)
    at internals.Connection._addRoute (/Users/jherri1/master/pharmacy/node_modules/hapi/lib/connection.js:344:17)
    at internals.Connection._route (/Users/jherri1/master/pharmacy/node_modules/hapi/lib/connection.js:336:18)
    at internals.Plugin._apply (/Users/jherri1/master/pharmacy/node_modules/hapi/lib/plugin.js:432:14)
    at internals.Plugin.route (/Users/jherri1/master/pharmacy/node_modules/hapi/lib/plugin.js:407:10)
    at Object.exports.register (/Users/jherri1/master/pharmacy/src/server/index.js:13:10)
    at /Users/jherri1/master/pharmacy/node_modules/hapi/lib/plugin.js:235:14
    at iterate (/Users/jherri1/master/pharmacy/node_modules/hapi/node_modules/items/lib/index.js:35:13)

Where the code at server/index.js line #13 is:

  server.route(
    HulaHoop.api.resourceLoader.routes().map(function(route) {
      return {
        path: route,
        method: 'GET',
        handler: HulaHoop.endpoints.page(
          appName,
          {resourceRoot: '/r'}
        )
      };
    })
  );

howdy @jherr. I'm starting to take a look at this now.

@jherr I can't find "children ..." in the pharmacy project. I see it in the pack  project in node_modules. What behavior is causing this error?

@nvcexploder the issue is the structure that is created when permutations are in play results in a circus.json at the root of the build dir that only references the children, which have the actual data needed by hula-hoop to load.
The issues are:

Making hula-hoop aware of the children linking when built this way.
How do we differentiate between the urls in these cases? We need some friendly, and configurable, way of differentiating between the routes. This could be path prefix or it could be via a query parameter, but we need to make sure that:
a. All routes can still be loaded
b. The browser urls don't have unnecessary cruft in their urls. I.e. if we use a prefix, /ios/foo and /android/foo are fine but we don't want /browser/foo, instead /foo.
Make sure that the server permutations are used in favor of the client ones, when available.

This might require injecting additional data into the circus.json files to make sure that we get all of the metadata that we need and if so, I can point you to examples of how to do this.

Initial takes on this make 'browser' look like a keyword, should we implement that way or should we go with a 'default', or both?

@nvcexploder I'd like to avoid parsing the configId, if possible. I think we can use the serverRender flag that is generated by our internal tooling to help with some of this, if need be. We can also add additional metadata to circus.json, if need be.

Taking a look at what needs to happen here. Basically we have to dedupe and provide namespaces for each permutation. The general rules are as follows:

The glue project are expected to set their prefix value. I.e. Pack internally
For a given prefix, multiple routes may exist only if they have different serverRender flags
For a given prefix + route, the serverRender flag will take precedence if set.

This allows for conditionals like:
Zeus.router();  // Run in client and server
if (!$serverSide) {
  Zeus.router(); // Only run in client, but server will send CJS response
}

Should the need for such behavior arise. If this is not atomic enough, then we can look at other attribute setups later.

So what is the 'glue' project in this architecture? Would that be the application project (e.g. pharmacy)? And I'm ok with server rendering not being part of an initial pass here if that helps.
Also, Ken Wheeler is working on our hybrid stuff now. He can certainly happily zing along without this support. But eventually, say a month from now, the need will be a little more urgent. So, thanks for looking into this now.


Initializing two different instances causes one to fail.

Expected Behavior
I expect both instances to work properly and set correct sizes.
Actual Behavior
One of the instances works perfectly but the height of the second instance is incorrect.
Step-by-step reproduction instructions

Place two different containers on a webpage.
Initialize both
Check the height of each container

I use the following configuration:
const macyInstance = Macy({
            container: `#${wrappedId}`,
            columns: 1,
            margin: {
                x: 32,
                y: 32,
            },
            mobileFirst: true,
            breakAt: {
                576: {
                    columns: 2,
                },
                992: {
                    columns: 3,
        },
    },
});

macyInstance.recalculate();
Screenshots, screen recordings, code snippets
If possible, please upload a screenshot or screen recording which demonstrates the bug.
This is the first instance:

This is the second instance:

Problem was solved with the recalculateOnImageLoad function.


"roles" table id column name error.

Describe the bug
The "roles" table id column name is "group_id". It should be "role_id".
To Reproduce
Steps to reproduce the behavior:
N/A
Expected behavior
The id column error is confusing and will cause extra labor to clarity.
Screenshots
@Entity
@AttributeOverride(name = "id", column = @Column(name = "group_id"))
@Table(name = "roles")
public class RoleEntity extends AbstractEntity {

  private static final long serialVersionUID =<PHONE_NUMBER>369269308L;

  @Column(name = "name", length = 128, nullable = false, unique = true)
  private String name;

  @Column(name = "description", length = 255, nullable = true)
  private String description;

  @ManyToMany(mappedBy = "roles", fetch = FetchType.LAZY)
  private Set<IdentityEntity> identities = new HashSet<>();

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
The local h2 db will be corrected by liquibase code.
While in other envs, the JAVA code will be corrected by a PR, meanwhile, the DB table column name will be modifyed to "role_id" by sql statement. @joshfischer1108

fixed with #195


Rename KemPrimitives to DhkemPrimitives.

Codecov Report

Merging #252 (072d86b) into main (f9619e1) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #252   +/-   ##
=======================================
  Coverage   96.96%   96.96%
=======================================
  Files          35       35
  Lines        2309     2309
  Branches      236      236
=======================================
  Hits         2239     2239
  Misses         69       69
  Partials        1        1

Flag
Coverage Δ

unittests
96.96% <100.00%> (ø)

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

Files Changed
Coverage Δ

src/interfaces/dhkemPrimitives.ts
100.00% <ø> (ø)

src/kems/dhkem.ts
100.00% <100.00%> (ø)

src/kems/dhkemPrimitives/ec.ts
93.64% <100.00%> (ø)

src/kems/dhkemPrimitives/x25519.ts
96.95% <100.00%> (ø)

src/kems/dhkemPrimitives/x448.ts
97.00% <100.00%> (ø)


Swedish language support

Support for Swedish language

I have extracted only the code related to Swedish language in #223 and merged it. I will now close this PR, because it has become obsolete.
Feel free to open a new one only for the ABC auto accept feature or the fix for Kyocera.


Lint line endings

This standardizes us on unix line endings (\n aka 0x0a). It prevents https://github.com/spine-generic/data-multi-subject/pull/57#issuecomment-703746602.
Here's an example of how this looks when it triggers a warning: https://github.com/spine-generic/data-multi-subject/pull/87/checks?check_run_id=2374745131#step:4:69

Of course, to make this work, I had to edit the current state of some of the the metadata. It looks like the balgrist parts were edited on Windows, and most lines in participants.tsv too.
(citing http://dataprotocols.org/linear-tsv/

It is permitted but discouraged to separate records with carriage-return-newline (0x0d and 0x0a)

so I just got rid of the 0x0ds)

What is the diff in all the JSON files? is it motivated by a BIDS-related rule? If there are issues in the way JSON files are produced by dcm2niix then we should probably raise an issue on their repos (otherwise this issue will keep happening)

Well that's why I guessed

It looks like the balgrist parts were edited on Windows,

I don't really know why they're like that. I could. Most of the json files were unix-ended, it was just that set that I needed to bring into compliance.
It looks like all of them were added by you, in f246510c4e4f432f195709e1130ee417606eba44:
$ git log origin/master -- sub-balgrist*/**/*.json
commit f246510c4e4f432f195709e1130ee417606eba44
Author: Julien Cohen-Adad<EMAIL_ADDRESS>Date:   Tue Jun 30 16:31:02 2020 -0400

    Added sub-amu -> sub-barcelona

So... I don't know. It was probably a fluke. Lots of other files that you also edited were unix-ended just fine.
The linter will catch it if it happens again!

You can run this to check that the diff made no change to the data:
(find . -name "*.json" | while read json; do if ! diff -u <(git show origin/master:$json | python -m json.tool) <(git show ng/lint-newlines:$json | python -m json.tool); then echo $json; fi; done) | less

it parses all the JSON through python and pretty-prints it back out again (which is another method of canonicalization) for both branches, and then diffs it. It came out empty for me, so I think it's good.

It doesn't seem like BIDS specifies line endings; the most it says is use UTF-8 JSON. Which is too bad, because it means the format isn't canonical. It doesn't impose ordering on the keys, or pretty-printing or conversely minifying.
But I'm saying we should specify it.


Openai agent json malformed 11921

Description
The OpenAIAgent no longer crashes when the function argument is not valid json. Furthermore, a tool_call_parser can be passed when creating the OpenAIAgent that allows the user to handle/ recover from malformed tool calls. For an example, see docs/examples/agent/openai_agent_tool_call_parser.ipynb
Fixes  #11921
Version Bump?

[x] Yes
[ ] No

Type of Change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

[x] Added unit/integration tests and executed them locally

Suggested Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added Google Colab support for the newly added notebooks.
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I ran make format; make lint to appease the lint gods

@logan-markewich just merged with current main and pushed the changes.

@JoelGotsch could you kindly pull latest from main and resolve conflicts?

@nerdai done


404s on opening app

internal support issue 589493
404 every time they open the app.

any luck parsing out what may be going on?

This is another case where the problem is very likely in the backend. The original ticket indicates that the user can't see their Hive contents in the web view, either.
(Unfortunately, the original ticket is in the old ticket system, and the clue about that is only contained in the new ticket's title. Unravelling that, hoping there would be salient info in the original, would have been  a lot simpler if a link to the original was included in the new ticket.  I can only imagine that each of us involved has to figure that out, and track it down, it would be good if we each reduced the next person's efforts by including a direct link - which I duly did in my griping comment, in the new internal issue.)

Thanks Ken - I don't have access to RT anymore so I can't track down those links. Any thoughts on what to tell user/ do I need to open a Trac ticket

I think some (object) server side investigation is going to be necessary, since the behavior happens with both web and mobile access.
(Seems like RT access is going to be necessary for you to wrangle tix, if the supporters are continuing to refer to old tickets there?)


The "Copy Value" selection in debugger is always grayed out and unusable

The "Copy Value" right-click selection in the debugger is always grayed out and unusable.
Previously closed related issue: https://github.com/microsoft/vscode/issues/67047
VSCode version:

Version: 1.42.0
Commit: ae08d5460b5a45169385ff3fd44208f431992451
Also tested on 1.41.x

Mac OSX Mojave

Steps to Reproduce:

Download my sample project:  https://github.com/djangofan/karate-api-sample
In VSCode, install the VSCode debugger plugin and Cucumber plugin
I include a debug runtime .vscode/launch.json in the above mentioned github repo
Put breakpoint on line 18 of this file: https://github.com/djangofan/karate-api-sample/blob/master/src/test/java/features/jph/getalltodos.feature
When debugger stops on that line, you will NOT be able to "Copy Value" from the debugger stack list.

Does this issue occur when all extensions are disabled?
Unable to reproduce since my Debug session relies on the debugger/Karate plugin

Please file this issue against the karate debugger which you are using as the debugger.
Tell them they need to provide the evaluateName property for the variables (in case you are talking about the Copy Value in the VARIABLES view).
VS Code uses this property to properly evlauate the variable which it will copy with that command.


Bug: SyntaxError when generating tests

Environment:
platform: Linux
node: v18.14.0
playwright-bdd: v5.8.0
@playwright/test: v1.42.1
ts-node: v10.9.1
typescript: v5.1.6
Hi @vitalets, would you be able to help me by checking the following error that I am getting within a remote Linux machine when running command npx bddgen.
Sorry due to company restrictions I am just able to paste it like picture. Locally on my windows machine with the same nodejs version I am not getting that error. Any thoughts by looking at the screenshot of what could cause this?

Ah I think I found the reason, will check now, but it is probably because of that line in my config file

I need to first create the env file as the parsing fails.

No problem, you are welcome!


Issues loading plugin with last update

tried running install and update. Also tried uninstalling and reinstalling the plugin but all roads lead me here:
[4/19/2017, 7:59:14 AM] ERROR LOADING PLUGIN homebridge-broadlink-rm:
[4/19/2017, 7:59:14 AM] /usr/local/lib/node_modules/homebridge-broadlink-rm/accessories/accessory.js:36
async setCharacteristicValue (props, value, callback) {
^^^^^^^^^^^^^^^^^^^^^^
SyntaxError: Unexpected identifier
at createScript (vm.js:56:10)
at Object.runInThisContext (vm.js:97:10)
at Module._compile (module.js:542:28)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (/usr/local/lib/node_modules/homebridge-broadlink-rm/accessories/aircon.js:1:92)
[4/19/2017, 7:59:14 AM] ====================

Update node - see README for version
On 20 Apr 2017, 1:03 AM +1000, webpablo<EMAIL_ADDRESS>wrote:

tried running install and update. Also tried uninstalling and reinstalling the plugin but all roads lead me here:
[4/19/2017, 7:59:14 AM] ERROR LOADING PLUGIN homebridge-broadlink-rm:
[4/19/2017, 7:59:14 AM] /usr/local/lib/node_modules/homebridge-broadlink-rm/accessories/accessory.js:36
async setCharacteristicValue (props, value, callback) {
^^^^^^^^^^^^^^^^^^^^^^
SyntaxError: Unexpected identifier
at createScript (vm.js:56:10)
at Object.runInThisContext (vm.js:97:10)
at Module._compile (module.js:542:28)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (/usr/local/lib/node_modules/homebridge-broadlink-rm/accessories/aircon.js:1:92)
[4/19/2017, 7:59:14 AM] ====================
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub (https://github.com/lprhodes/homebridge-broadlink-rm/issues/51), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAJ8rfsDMthuj_s5HG85TnNAwrlZfaW5ks5rxiJMgaJpZM4NB0JC).

updated node + reinstalled home bridge and the plugin. deleted my persist folder.
[4/19/2017, 8:26:18 AM] ERROR LOADING PLUGIN homebridge-broadlink-rm:
[4/19/2017, 8:26:18 AM] /usr/local/lib/node_modules/homebridge-broadlink-rm/accessories/accessory.js:36
async setCharacteristicValue (props, value, callback) {
^^^^^^^^^^^^^^^^^^^^^^
SyntaxError: Unexpected identifier
at createScript (vm.js:56:10)
at Object.runInThisContext (vm.js:97:10)
at Module._compile (module.js:542:28)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (/usr/local/lib/node_modules/homebridge-broadlink-rm/accessories/aircon.js:1:92)
[4/19/2017, 8:26:18 AM] ====================

Just ran a clean setup on a brand new OS X install.
installed node.js of OS X
installed homebridge
ran without issue
installed broad link plugin
[4/19/2017, 10:09:13 AM] ERROR LOADING PLUGIN homebridge-broadlink-rm:
[4/19/2017, 10:09:13 AM] /usr/local/lib/node_modules/homebridge-broadlink-rm/accessories/accessory.js:36
async setCharacteristicValue (props, value, callback) {
^^^^^^^^^^^^^^^^^^^^^^
SyntaxError: Unexpected identifier
at createScript (vm.js:56:10)
at Object.runInThisContext (vm.js:97:10)
at Module._compile (module.js:542:28)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (/usr/local/lib/node_modules/homebridge-broadlink-rm/accessories/aircon.js:1:92)
[4/19/2017, 10:09:13 AM] ====================

It's definitely the node version...check using the node -v
On 20 Apr 2017, 3:11 AM +1000, webpablo<EMAIL_ADDRESS>wrote:

Just ran a clean setup on a brand new OS X install.
installed node.js of OS X
installed homebridge
ran without issue
installed broad link plugin
[4/19/2017, 10:09:13 AM] ERROR LOADING PLUGIN homebridge-broadlink-rm:
[4/19/2017, 10:09:13 AM] /usr/local/lib/node_modules/homebridge-broadlink-rm/accessories/accessory.js:36
async setCharacteristicValue (props, value, callback) {
^^^^^^^^^^^^^^^^^^^^^^
SyntaxError: Unexpected identifier
at createScript (vm.js:56:10)
at Object.runInThisContext (vm.js:97:10)
at Module._compile (module.js:542:28)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (/usr/local/lib/node_modules/homebridge-broadlink-rm/accessories/aircon.js:1:92)
[4/19/2017, 10:09:13 AM] ====================
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub (https://github.com/lprhodes/homebridge-broadlink-rm/issues/51#issuecomment-295349779), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAJ8rbGmp4uf9mwzMRderITFgA-O7RBhks5rxkBAgaJpZM4NB0JC).

AHHHH
v6.10.2.
I didn't realize there was a "for most users" and "latest" download page for node. ¯_(ツ)_/¯
Sorry for the noise…


Fix the .list node for the StringArrayEditor

fixes the icon buttons on the stringArrayEditor/tagInput .list element so that they are inline, given the correct background, and aligned correctly.  Restores pink validation styling.
Description
Here it is broken, on Live:

Here it is fixed:

Related Issues or Discussions

gitter discussion

QA Instructions, Screenshots, Recordings
Try using the string array editor (aka "tags" and "authors" inputs) in the Properties Editor.  The three states should all display properly (non-editing 'badge', editing badge, and empty badge/input).
Add'l comments
Likely the metadataEditor.less file could be overhauled more completely, but I'm just focused on fixing this one thing.  In my TagInput PR there are significant changes to the .less file so I don't want to muck about too much in there right now if i don't have to.

Does this one conflict with #3894 at all? Do you have a preference of which gets merged first?

Does this one conflict with https://github.com/naturalcrit/homebrewery/pull/3894 at all? Do you have a preference of which gets merged first?

I would just go with 3894, and this can be closed, reopening if necessary.  The tagInput PRs have a few more in the pipe, and i can't remember right now which bits of styling happen where but I'm sure I didn't take anything from this PR and reuse it (intentionally).

Then closing this for now, as #3894 should fix the issue.


Add a slow mode to benchmarks

I added a slow mode to the benchmarks and removed the overcommit option, to highlight two issues that occur in the tezos benchmarks:

without the overcommit option, the merge blocks sometimes;
we need to add thread yields in the freeze thread to ensure fairness.

I'm not convinced by the call to Unix.sleep in the tests by tracking maxrss seems like a good idea!

LGTM

Is this ready to be merged?


small fix to remove the option of generating a wasm tm client

Description
part of the work in #4801
Commit Message / Changelog Entry
type: commit message

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

Codecov Report

Merging #4814 (5a62616) into cian/merge-main-wasm (e9c0862) will increase coverage by 0.08%.
Report is 4 commits behind head on cian/merge-main-wasm.
The diff coverage is n/a.

Additional details and impacted files

@@                   Coverage Diff                    @@
##           cian/merge-main-wasm    #4814      +/-   ##
========================================================
+ Coverage                 79.78%   79.87%   +0.08%
========================================================
  Files                       189      189
  Lines                     13264    13249      -15
========================================================
- Hits                      10583    10582       -1
+ Misses                     2254     2242      -12
+ Partials                    427      425       -2

see 1 file with indirect coverage changes

ACK, but there's other logic that can be removed as well, such as chain.SetWasm()

ACK, but there's other logic that can be removed as well, such as chain.SetWasm()

pulling on threads and unravelling -- ill push these changes to this pr as well!

closing as changes are included in #4804


Add email verification to user signup

Description
I need to add email verification to user signup so that I can ensure users are valid.
Requirements

[x] Add email field to user signup form
[x] On signup, user will be sent a verification email
[x] On user verification, they are granted 'verified' status
[x] Until the user is validated, they shall be 'unverified'

Created branch and pulled for development.

Moved USER_TYPES LIST to constants.py

need to add route user will click in their email.
build reverse() route using user + verify code, will activate the user in our system.
need to add something that will not allow the user to login if their account is unverified.

Updated :)


Add Persistent Rope

Hello @zyedidia, I was thinking of contributing a persistent rope to the project.
Does it fit in to the project? There is already a rope, but adding immutable methods to it would cause undefined behavior if you used them with the mutable ones.
Thank you for your time!

I think it might fit. What would the API look like, and how would it be different from the current rope?

It would be pretty similar to the current rope, except operations like insert would return a changed value instead of changing it. For example, instead of:
rope := Rope.New([]int{1,3,4})
rope.Insert(1, []int{2})
rope  // [1, 2, 3, 4]

You would have
rope := PRope.New([]int{1,3,4})
changedRope := rope.Insert(1, []int{2})
rope  // [1, 2, 4]
changedRope // [1, 2, 3, 4]

The difference is quite minimal, but it can be very useful, specially because creating a copy of the rope takes log(n) instead of n space. I have used this approach in the past for a text editor, and it does not seem to have trouble with slowing the GC either.

Alright sounds good, let's call the package prope.